-
Notifications
You must be signed in to change notification settings - Fork 129
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
Added test file for cmd/armadactl/cmd/cancel_test.go #2418
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #2418 +/- ##
==========================================
+ Coverage 58.56% 58.60% +0.03%
==========================================
Files 231 231
Lines 28813 28813
==========================================
+ Hits 16875 16886 +11
+ Misses 10636 10624 -12
- Partials 1302 1303 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your PR. Can you get the lint issues fixed and then I’ll review?
Updated the armada/cmd/armadactl/cmd/cancel_test.go, fixing lint issues.
Updated cancel_test.go file added as test file for armada/cmd/armadactl/cmd/cancel.go, fixing lint issues.
Fixed lint issues in armada/cmd/armadactl/cmd/cancel_test.go
* Make fake executor support cancellation We now track which pods are allocated to each node, so we can deallocate them correctly on cancellation Previously cancelled pods would just remain allocated forever and block all capacity on the cluster * Formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you make sure your PR only has your additions?
I apologise for the inconvenience. The other account is not mine so I won't be able to remove its commits from this PR. Is it okay if I create another PR for this test file and close this one? |
Yea feel free to close and reopen. It’s no big deal |
Fixes #
Issue #2167
Special notes for your reviewer:
┆Issue is synchronized with this Jira Task by Unito