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
Copy link

sonarcloud 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 merged commit c85cc78 into k8ssandra:main Jun 11, 2024
1 check passed
Copy link

sonarcloud bot commented Jun 11, 2024

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