Skip to content
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

Refactor Delete Namespace test #3698

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Refactor Delete Namespace test #3698

merged 1 commit into from
Jul 27, 2023

Conversation

jmckulk
Copy link
Collaborator

@jmckulk jmckulk commented Jul 27, 2023

  • Allow runner to define a namespace to delete through the Makefile. This will be useful if two sets of Kuttl tests are running in the same env
  • Move from 2 replicas to 1 to speed up the test
  • Use single line volume claim specs

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change, as applicable?
  • Have you tested your changes on all related environments with successful results, as applicable?
    • Have you added automated tests?

Type of Changes:

  • New feature
  • Bug fix
  • Documentation
  • Testing enhancement
  • Other

What is the current behavior (link to any open issues here)?

What is the new behavior (if this is a feature change)?

  • Breaking change (fix or feature that would cause existing functionality to change)

Other Information:

@jmckulk
Refactor Delete Namespace test
6c9a3bc 
- Allow runner to define a namespace to delete through the makefile. This
  will be useful if two sets of kuttl tests are running in the same env
- Move from 2 replicas to 1 to speed up the test
- Use single line volume claim specs
@jmckulk jmckulk merged commit b7bafdb into CrunchyData:master Jul 27, 2023
15 checks passed
@jmckulk jmckulk deleted the delete-namespace-test-update branch July 27, 2023 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjaminjb benjaminjb benjaminjb approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jmckulk @benjaminjb