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.

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

ci: Add util to test structure to clean cloudv2 after testrun #14255

Merged
merged 5 commits into from
Oct 24, 2023

Conversation

savex
Copy link
Contributor

@savex savex commented Oct 18, 2023

To minimize code duplication, util is using the same RpCloudApiClient
class that is used in redpanda_cloud class. Also, to isolate it from
ducktape runtime, its placed in tests folder to utilize rptest as an
external module and prevent any CI-related modifications.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

Release Notes

  • none

  To minimize code duplication, util is using the same RpCloudApiClient
  class that is used in redpanda_cloud class. Also, to isolate it from
  ducktape runtime, its placed in tests folder to utilize rptest as an
  external module and prevent any CI-related modifications.
@savex savex marked this pull request as draft October 18, 2023 18:23
@savex savex force-pushed the cloudv2-8970-cleanup-resources branch from f5e709c to 1147291 Compare October 18, 2023 23:23
@savex savex requested review from andrewhsu and simonlord October 19, 2023 14:05
@savex savex force-pushed the cloudv2-8970-cleanup-resources branch from 1147291 to 87261d5 Compare October 19, 2023 14:08
@savex
Copy link
Contributor Author

savex commented Oct 19, 2023

Regex for date handling: https://regex101.com/r/EMVXKT/1

@savex savex marked this pull request as ready for review October 20, 2023 21:26
@vbotbuildovich
Copy link
Collaborator

Copy link
Member

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

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

LGTM

synced with alex at our htt phase 2 mtg on the approach for rp_cloud_cleanup.py script. is suitable for now to cleanup the redpanda cloud clusters and namespaces in the context of post-ducktape test runs.

@savex savex merged commit 39062df into dev Oct 24, 2023
28 checks passed
@savex savex deleted the cloudv2-8970-cleanup-resources branch October 24, 2023 16:40
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