Skip to content

setup new pipeline to run kubelet config#592

Open
bravebeaver wants to merge 19 commits into
mainfrom
beileih/kubeletBenchmark
Open

setup new pipeline to run kubelet config#592
bravebeaver wants to merge 19 commits into
mainfrom
beileih/kubeletBenchmark

Conversation

@bravebeaver
Copy link
Copy Markdown
Member

adding a new pipeline for kubelet eval.

setup extra variables which allows for customised kubelet config later on.

…rt with.

- the variables are defined to be used by later steps in the pipeline to config cluster accordingly
Copy link
Copy Markdown
Collaborator

@alyssa1303 alyssa1303 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add code change to run test first and move the variable setup to matrix

@bravebeaver bravebeaver requested a review from alyssa1303 April 15, 2025 01:44
@bravebeaver
Copy link
Copy Markdown
Member Author

please add code change to run test first and move the variable setup to matrix

can i please have another review? added configuration files and the steps to configure clusters.

- stage: azure_swedencentral_evict_memory_100mi
dependsOn: []
jobs:
- job: SetKubeletVariables
Copy link
Copy Markdown
Collaborator

@alyssa1303 alyssa1303 May 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's schedule a meeting to go over this PR. An extra job is expensive and should be avoid unless really necessary. Also, I still don't see terraform files? Can you split this PR into smaller one for easier review?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants