Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resource name on the provider side #42

Open
deusebio opened this issue Feb 14, 2023 · 0 comments
Open

Resource name on the provider side #42

deusebio opened this issue Feb 14, 2023 · 0 comments

Comments

@deusebio
Copy link
Contributor

Right now the interface for most of databases is such that requirer provides the resource name (being database or topic) on its request and the provider answers with the credentials (e.g. username, password, endpoints, etc).

This is somewhat inconsistent with the UX for s3 integrator, where the provider ALSO provides the bucket name and the requirer has a clear requirement listed that :

Is expected to tolerate that the Provider may ignore the bucket field in some cases (e.g. S3Proxy or S3 Integrator) and instead use the bucket name received.

Could we do this also for all database interfaces?

One of the things we are currently struggling with is that the resource name is present in the requirer application data bag, which can be read by the leader but not other units. In the use-case where the client applications want to scale out (adding more units), we need the resource name to be available to them as well. Of course one could include this info in the peer relations, but duplicating information (from application relation to peer relation) does not seem optimal to me. Beside, providing this on the requirer side would fix this, but also it would be more general and consistent with S3 if we want to provide the feature of overriding the requirer preference (similar to the requirement above for s3)

WRFitch pushed a commit to WRFitch/charm-relation-interfaces that referenced this issue Mar 7, 2023
jnsgruk pushed a commit that referenced this issue Mar 8, 2023
* updated opensearch doc to include adding the index to the provider side of the application

see #42 for reasoning

* Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant