-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update README with new API #59
Conversation
regenerate-token: true | ||
... | ||
``` | ||
After token rotation, the `spec.credentials` section will be deleted and `status.credentials.generationTimestamp` updated. |
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.
Would be good to also mention whether or not and what else will be done accordingly
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 didn't get your point, could you elaborate more on this?
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 mean, except the generationTimestamp
is updated, if anything else is also changed, it would be good to mention here as well.
|
||
The key difference from general purpose workflow are: | ||
- second robot account and corresponding secret created with read (pull) only access to the image repository. | ||
- the pull secret is propagated into all `Application` environments via `RemoteSecret`. |
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.
We know that the "environments" means individual deployment environment, staging and prod. Do we need to mention it here explicitly?
Is "Application environments" described in some other document? If yes, it should be good to link to 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.
I looked briefly and failed to find the definition. The closest mention of it is here. Don't think we should include the link above here.
No description provided.