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-5896] bns set annotation when shutting down a kernel #230

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

epignot
Copy link
Contributor

@epignot epignot commented Sep 13, 2024

Jira ticket

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

Description

bigdata-notebook-service nows sets annotation kill-requested-at on SparkApp CR when kernel shutdown (or restart) is requested.
a ClusterRole has been added to allow this behaviour in a previous change.

Demo

Please add a recording/screenshot of the feature/bug fix in action

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

  • deploy chart helm upgrade bigdata-notebook-service-bdenv-v30 bigdata-notebook-service -n spot-system
  • run notebooks
  • kill and restart them
  • check CR contains kill-requested-at annotation

Test plan and results

Test Description Result Notes
1 Chart deployment Pass
2 Annotation on killed notebook Pass
3 Annotation on restarted notebook Pass

Screenshot 2024-09-13 at 15 57 32

Screenshot 2024-09-13 at 17 10 30
Screenshot 2024-09-13 at 17 10 45

@epignot epignot requested a review from a team as a code owner September 13, 2024 12:26
@epignot epignot changed the title bns set annotation when shutting down a kernel [BGD-5896] bns set annotation when shutting down a kernel Sep 13, 2024
bigdata-notebook-service nows sets annotation `kill-requested-at` on
SparkApp CR when kernel shutdown (or restart) is requested.
a ClusterRole has been added to allow this behaviour in a previous
change.
@epignot epignot force-pushed the bgd-5896-bns-kill-annotation-on-notebook-shutdown branch from 84a40e9 to 8afc99d Compare September 16, 2024 06:40
@epignot epignot merged commit 6402c01 into main Sep 16, 2024
1 check passed
@epignot epignot deleted the bgd-5896-bns-kill-annotation-on-notebook-shutdown branch September 16, 2024 06:42
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