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

BGD-6179 Handle orphaned drivers #253

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Conversation

HafdisE
Copy link
Contributor

@HafdisE HafdisE commented Dec 6, 2024

Jira ticket

https://spotinst.atlassian.net/browse/BGD-6179

Description

Introduces the new env vars required to enable the new cleanup mechanisms for orphaned drivers and stuck failed spark app CRs, as well as bumps the image tag to use the new version.

Demo

Screenshot 2024-12-06 at 16 20 05 After applying the chart.

Checklist

  • I have added a Jira ticket link
  • I have filled in the test plan
  • I have executed the tests and filled in the test results
  • I have updated/created relevant documentation

How to test

Apply the chart but set the image.tag to BGD-6179-handle-orphaned-driver-pods and the image.registry to the private registry.

Test plan and results

See PR in the watcher for more details.

Test Description Result Notes
1 Apply chart on cluster and see that it is configured correctly
2 Integration tests
3 Have stuck failed app CRs and observe that they get removed

@HafdisE HafdisE force-pushed the BGD-6179-handle-orphaned-driver-pods branch 3 times, most recently from 009ecac to 7b35b2a Compare December 13, 2024 08:27
@HafdisE HafdisE force-pushed the BGD-6179-handle-orphaned-driver-pods branch from 7b35b2a to 2f2d5ee Compare December 13, 2024 08:31
@HafdisE HafdisE marked this pull request as ready for review December 13, 2024 08:57
@HafdisE HafdisE requested a review from a team as a code owner December 13, 2024 08:57
@HafdisE HafdisE merged commit b08678d into main Dec 13, 2024
2 checks passed
@HafdisE HafdisE deleted the BGD-6179-handle-orphaned-driver-pods branch December 13, 2024 10:21
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.

2 participants