-
Notifications
You must be signed in to change notification settings - Fork 226
Update messages on service account to be explicit about GCP service account #570
Update messages on service account to be explicit about GCP service account #570
Conversation
/assign medinatiger |
) | ||
|
||
const ( | ||
ServiceAccountPrefix = "k8s-machine-controller-" | ||
ServiceAccount = "service-account" | ||
MachineControllerSecret = "machine-controller-credential" | ||
ServiceAccountNs = "kube-system" |
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.
This should be sth configurable in the future. It doesn't mean you should change this. Just FYI
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.
Agreed! Filed to track it: #573
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: medinatiger, rsdcastro The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
…ce-account-message Update messages on service account to be explicit about GCP service account
No description provided.