Skip to content

Add node scaling to e2e tests #291

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Jul 4, 2022
Merged

Add node scaling to e2e tests #291

merged 2 commits into from
Jul 4, 2022

Conversation

dgkanatsios
Copy link
Collaborator

@dgkanatsios dgkanatsios commented Jul 2, 2022 •
edited
Loading

This PR

  • adds a LOG_LEVEL env variable to the controller manager, with default value "info"
  • configures the same value to "debug" on e2e tests
  • enhances the e2e tests with a worker Node becoming unavailable (cordon and drain) and available again.

@dgkanatsios dgkanatsios force-pushed the nodescalee2e branch 2 times, most recently from e4f29fd to 1b1eabe Compare July 2, 2022 03:40
@dgkanatsios dgkanatsios added the area/tests Make sure things don't break, unit tests. label Jul 2, 2022
@dgkanatsios dgkanatsios force-pushed the nodescalee2e branch 5 times, most recently from 31ce022 to 71ba338 Compare July 3, 2022 01:20
@dgkanatsios dgkanatsios marked this pull request as ready for review July 3, 2022 01:20
@dgkanatsios dgkanatsios requested a review from javier-op July 3, 2022 01:27
javier-op
javier-op reviewed Jul 4, 2022
View reviewed changes
javier-op
javier-op reviewed Jul 4, 2022
View reviewed changes
@dgkanatsios dgkanatsios merged commit 46e9ccd into main Jul 4, 2022
@dgkanatsios dgkanatsios deleted the nodescalee2e branch July 4, 2022 17:16
ghost
ghost reviewed Aug 21, 2023
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

op

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

@javier-op javier-op javier-op approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
area/tests Make sure things don't break, unit tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dgkanatsios @javier-op