-
Notifications
You must be signed in to change notification settings - Fork 4.7k
handle secret types #2424
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
handle secret types #2424
Conversation
@@ -20,9 +20,14 @@ import ( | |||
type CreateSecretOptions struct { | |||
// Name of the resulting secret | |||
Name string | |||
|
|||
// SecretTypeName is the type to use when creating the secret. It is checked against know types. |
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.
It is checked against known types
Fix godoc, lgtm |
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_origin/2049/) (Image: devenv-fedora_1593) |
Evaluated for origin up to da6a88b |
[Test]ing while waiting on the merge queue |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/2328/) |
Merged by openshift-bot
Adds support for specifying
Secret.Type
from the command line and automatic detection based on secret data content.@derekwaynecarr ptal