-
Notifications
You must be signed in to change notification settings - Fork 21
Conversation
@philbrookes FYI moving things around a bit nothing major. Will merge once it passes |
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.
Typo needs fixing
pkg/mobile/integration/service.go
Outdated
@@ -130,15 +117,16 @@ func (ms *MobileService) MountSecretForComponent(svcCruder mobile.ServiceCruder, | |||
} | |||
|
|||
//find the clientService secret name | |||
css, err := svcCruder.List(ms.filterServices([]string{clientService})) | |||
if err != nil || len(css) == 0 { | |||
cSeviceList, err := svcCruder.List(filterServices([]string{clientService})) |
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.
typo here: cServiceList
pkg/mobile/types.go
Outdated
@@ -94,6 +94,9 @@ func (at AppTypes) String() string { | |||
//ValidAppTypes is a list of valid app types | |||
var ValidAppTypes = AppTypes{"cordova", "android", "iOS"} | |||
|
|||
//ServiceTypes are the service types that we are aware of and support | |||
var ServiceTypes = []string{"fh-sync-server", "keycloak", "custom"} |
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'm uncomfortable with this in general - not specific to this PR; I don't like that we have to recompile, rebuild and redeploy the API server anytime we want to add a new mobile service.
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.
yes agree, however I am chatting with the service catalog guys around having labels on the secrets kubernetes-retired/service-catalog#1222 which I think will help. Also can be moved out to an env var at somepoint
moving client pkg out of mobile pkg. Some other general refactors and additional tests