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

Use backup namespace for clean job #1300

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

smutel
Copy link
Contributor

@smutel smutel commented Apr 26, 2024

What this PR does:
Change the target namespace for purge medusa tasks

Which issue(s) this PR fixes:
Fixes #1299

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

@smutel smutel requested a review from a team as a code owner April 26, 2024 09:34
@smutel smutel force-pushed the PurgeMedusaNamespace branch from 0ab3d78 to c70dfb9 Compare April 26, 2024 09:48
@smutel smutel force-pushed the PurgeMedusaNamespace branch from c70dfb9 to 359b3ae Compare May 7, 2024 09:50
Copy link

sonarqubecloud bot commented May 7, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@smutel
Copy link
Contributor Author

smutel commented May 7, 2024

@adejanovski Could you please tell me what do you think about this PR and tell me how to resolve CI issues ?

@adejanovski adejanovski force-pushed the PurgeMedusaNamespace branch from 359b3ae to d5347d5 Compare June 11, 2024 09:57
@adejanovski adejanovski merged commit c85cc78 into k8ssandra:main Jun 11, 2024
1 check passed
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
1.8% Duplication on New Code

See analysis details on SonarCloud

@adejanovski
Copy link
Contributor

Hi @smutel, I merged this change already in another PR yesterday (with some slight adjustments) but I was not feeling good about it because you're PR was pushed a while ago and we didn't take the time to review it. I've merged this PR so you can get credited for the contribution.
Thanks!!

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.

CronJob for medusa purge not in the correct namespace
2 participants