Skip to content
This repository has been archived by the owner on Jun 26, 2024. It is now read-only.

Drop Support binding N services to application using single ServiceBinding resource #898

Closed
jasperchui opened this issue Mar 4, 2021 · 1 comment
Labels
spec to-be-groomed issues to be groomed
Milestone

Comments

@jasperchui
Copy link

Reference EPIC #892

Use Case:

ServiceBinding resource allows declaring multiple services.

Reasons:

Although this looks at first as a handy shortcut, using this might cause a few issues:

A binding item from one service can be shadowed by another from another service (when binding keys are the same). Likelihood is very high, given that expected key names are username/password/url/… The issue can be worked around by introducing binding key prefixes or some custom naming conventions so that the keys become unique, but then it might be that such bindings cannot be consumed by application, i.e. it needs to be modified to support that
Unbind operation becomes all or nothing, i.e. it is not possible to unbind single service
What about updating/editing impact? I want to edit one service’s binding resources, how does it impact the applications that are consuming it?

Requirements:

Update samples:
[SB] Binding a workload with a non-CR-based PostgrersDB: The service resources involved are (1)Route/Service to Postgres (2)Secret containing DB credentials.
[SB] Binding a workload with keycloak. The service resources involved are (1) Keycloak CR (2) KeycloakUser CR

@pedjak
Copy link
Contributor

pedjak commented Jun 17, 2021

This case is contained in #899 - by solved it, this is will be addressed as well. Hence closing.

@pedjak pedjak closed this as completed Jun 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
spec to-be-groomed issues to be groomed
Projects
None yet
Development

No branches or pull requests

3 participants