-
Notifications
You must be signed in to change notification settings - Fork 244
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
[hailtop] Remove namespace options in hailctl auth commands #14069
[hailtop] Remove namespace options in hailctl auth commands #14069
Conversation
a3ef087
to
8a114de
Compare
hail/python/hailtop/auth/auth.py
Outdated
json=body | ||
) | ||
url = get_deploy_config().url('auth', f'/api/v1alpha/users/{username}/create') | ||
async with hail_session() as session: |
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.
I think we maybe purposely gave extra timeout for this API call.
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.
I forgot these weren't immediate operations. I restored the timeouts.
deploy_config.url('auth', f'/api/v1alpha/users/{username}') | ||
) | ||
async def async_delete_user(username: str): | ||
url = get_deploy_config().url('auth', f'/api/v1alpha/users/{username}') |
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.
is this API call fast?
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.
no, restored the timeout
I'm trying to move our codebase away from relying on the notion of a K8s namespace for routing because this model does not hold in alternative deployment environments such as Terra. Further, having a
-n
option inhailctl auth
commands is awkward because it can only be used for dev environments yet is visible to users in the help menu. As such, this is a breaking change only for developers. The functionality is not really gone though because you can replace anyhailctl auth … -n dgoldste
withHAIL_DEFAULT_NAMESPACE=dgoldste hailctl auth …
.