-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
cli: -namespace
should override job namespace
#10875
Conversation
When a jobspec doesn't include a namespace, we provide it with the default namespace, but this ends up overriding the explicit `-namespace` flag. This changeset uses the same logic as region parsing to create an order of precedence: the query string parameter (the `-namespace` flag) overrides the API request body which overrides the jobspec.
f3220b2
to
38af39e
Compare
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.
Thanks! I think I have been bitten by this before. We'll need to follow up with integration (or e2e) tests and ensure our documentation calls out ordering.
cli: `-namespace` should override job namespace
cli: `-namespace` should override job namespace
docs: add backward incompat note about #10875
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fixes #9893
When a jobspec doesn't include a namespace, we provide it with the default
namespace, but this ends up overriding the explicit
-namespace
flag. Thischangeset uses the same logic as region parsing to create an order of
precedence: the query string parameter (the
-namespace
flag) overrides theAPI request body which overrides the jobspec.