-
Notifications
You must be signed in to change notification settings - Fork 2.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
Label tokens #22184
Label tokens #22184
Conversation
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.
LGTM This adds labels to tokens
cc @vincent99
37529a7
to
30cb12a
Compare
def test_auth_label(admin_mc, user_factory): | ||
user = user_factory() | ||
k8s_client = CustomObjectsApi(admin_mc.k8s_client) | ||
tokens = k8s_client.list_cluster_custom_object( |
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.
There should be a wait_for in here to ensure we actually get the users token before attempting to access it.
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.
Fixed now. Also re-vendored types, which included another change but it looks legit.
lgtm |
Depends on: Types PR: rancher/types#940
Adds a label to Rancher management created tokens to help categorize them. Examples:
Note