-
Notifications
You must be signed in to change notification settings - Fork 18
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃敟 Add external credentials #215
Conversation
api/v1alpha1/proxmoxcluster_types.go
Outdated
// CredentialsRef is a reference to a Secret that contains the credentials to use for provisioning this cluster. If not | ||
// supplied then the credentials of the controller will be used. | ||
// +optional | ||
CredentialsRef *corev1.ObjectReference `json:"credentialsRef,omitempty"` |
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.
Maybe SecretReference
or if you expect the secret to be in the same namespace LocalObjectReference
would be sufficient?
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.
SecretReference is okay,
c97859d
to
64c2870
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.
The code is good to go in but this PR needs to update the documentation to explain how to use external credentials instead of PROXMOX_{URL,TOKEN,SECRET}.
Please check the added docs. |
Quality Gate passedIssues Measures |
This PR adds external credentials to the Cluster.
To make it able to use multiple Proxmox data centers
Close #210