Backport verify service accounts are set to stable 5 #8553
Conversation
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
|
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
| @@ -69,3 +69,19 @@ func MissingAdminUserID(service string) error { | |||
| "the config/corresponding environment variable).", | |||
| service, defaults.BaseConfigPath()) | |||
| } | |||
|
|
|||
| func MissingServiceAccountID(service string) error { | |||
There was a problem hiding this comment.
exported func MissingServiceAccountID should have a comment or be unexported
There was a problem hiding this comment.
this is a backport and we should not add something that is not in master
| service, defaults.BaseConfigPath()) | ||
| } | ||
|
|
||
| func MissingServiceAccountSecret(service string) error { |
There was a problem hiding this comment.
exported func MissingServiceAccountSecret should have a comment or be unexported
There was a problem hiding this comment.
this is a backport and we should not add something that is not in master
| @@ -83,6 +83,10 @@ type FrontendService struct { | |||
| ServiceAccount ServiceAccount `yaml:"service_account"` | |||
| } | |||
|
|
|||
| type OcmService struct { | |||
There was a problem hiding this comment.
exported type OcmService should have a comment or be unexported
There was a problem hiding this comment.
this is a backport and we should not add something that is not in master
|
|
This is now part of #8627 |
backport #8546 to stable 5