Skip to content

.github: use create eks nodegroup action for l7-perf workflow#44706

Merged
fristonio merged 1 commit intomainfrom
pr/fristonio/l7-perf-eks-nodegroup
Mar 25, 2026
Merged

.github: use create eks nodegroup action for l7-perf workflow#44706
fristonio merged 1 commit intomainfrom
pr/fristonio/l7-perf-eks-nodegroup

Conversation

@fristonio
Copy link
Copy Markdown
Member

Fixes #42363

Since different workflows have different need of how node groups are configured add an option to provide the full node groups yaml block.

@fristonio fristonio added area/CI Continuous Integration testing issue or flake release-note/misc This PR makes changes that have no direct user impact. labels Mar 10, 2026
@fristonio
Copy link
Copy Markdown
Member Author

/test

@fristonio
Copy link
Copy Markdown
Member Author

/l7-perf

1 similar comment
@fristonio
Copy link
Copy Markdown
Member Author

/l7-perf

@fristonio fristonio marked this pull request as ready for review March 10, 2026 08:09
@fristonio fristonio requested review from a team as code owners March 10, 2026 08:09
Copy link
Copy Markdown
Member

@giorio94 giorio94 left a comment

Choose a reason for hiding this comment

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

/lgtm, thanks! Could you extend the commit message with a bit more details for future reference?

Copy link
Copy Markdown
Member

@nebril nebril left a comment

Choose a reason for hiding this comment

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

LGTM, I agree with Marco's nits, especially dropping mode parameter.

@fristonio fristonio force-pushed the pr/fristonio/l7-perf-eks-nodegroup branch from ae81872 to 125e3c7 Compare March 10, 2026 20:10
@fristonio fristonio requested a review from a team as a code owner March 10, 2026 20:10
@fristonio fristonio requested a review from tommyp1ckles March 10, 2026 20:10
Make `setup-eks-nodegroup` action generic by providing an option to
override the node group details. This allows users of the action to
define custom node group setup as per the workflow requirement.

Additionally this commit migrates `l7-perf.yaml` and
`scale-test-egw.yaml` workflows to use the updated action definition by
providing the custom node group configuration.

Signed-off-by: Deepesh Pathak <deepeshpathak09@gmail.com>
@fristonio fristonio force-pushed the pr/fristonio/l7-perf-eks-nodegroup branch from 125e3c7 to 374c99c Compare March 10, 2026 20:12
@fristonio
Copy link
Copy Markdown
Member Author

fristonio commented Mar 10, 2026

/scale-egw
EGW scale workflow seems to be failing but unrelated to the changes in this PR.

@fristonio
Copy link
Copy Markdown
Member Author

/l7-perf

@fristonio
Copy link
Copy Markdown
Member Author

/test

Copy link
Copy Markdown
Member

@giorio94 giorio94 left a comment

Choose a reason for hiding this comment

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

Thanks!

@fristonio
Copy link
Copy Markdown
Member Author

Ping @joestringer @tommyp1ckles

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 25, 2026
@fristonio fristonio added this pull request to the merge queue Mar 25, 2026
Merged via the queue into main with commit e91fe6e Mar 25, 2026
573 of 595 checks passed
@fristonio fristonio deleted the pr/fristonio/l7-perf-eks-nodegroup branch March 25, 2026 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: make setup-eks-nodegroup github action generic

6 participants