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

[Enhancement Proposal] Enable setting multiple bucket names for each S3 integrator app #48

Open
michaeldmitry opened this issue Sep 6, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@michaeldmitry
Copy link

michaeldmitry commented Sep 6, 2024

Steps to reproduce

  1. Deploy s3-integrator, loki-coordinator-k8s, mimir-coordinator-k8s
  2. juju run s3-integrator/leader sync-s3-credentials access-key=<accesskey> secret-key=<secretkey>
  3. juju config s3-integrator endpoint=<minIO_endpoint>
  4. juju integrate s3-integrator loki-coordinator-k8s:s3 first (Note that: the loki charm passes loki as a bucket name in requirer app data)
  5. Then, juju integrate s3-integrator mimir-coordinator-k8s:s3 (Note that: the mimir charm passes mimir as a bucket name in requirer app data)
  6. jhack show-relation s3-integrator loki-coordinator-k8s
  7. jhack show-relation s3-integrator mimir-coordinator-k8s

Expected behavior

When multiple applications relate to the same s3-integrator while providing different bucket names from their requirer side (without setting bucket config option on the S3 integrator provider side), calling get_s3_connection_info from loki-coordinator-k8s should return {"bucket": "loki"} and calling get_s3_connection_info from mimi-coordinator-k8s should return {"bucket": "mimir"}

Actual behavior

get_s3_connection_info from loki or mimir returns {"bucket": "loki"}

Versions

Operating system: Ubuntu 22.04.4 LTS

Juju CLI: 3.4.5-genericlinux-amd64

Juju agent: 3.4.5

Charm revision: latest/edge rev: 41

Log output

Juju debug log:

Additional context

#49

@michaeldmitry michaeldmitry added the bug Something isn't working label Sep 6, 2024
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/DPE-5389.

This message was autogenerated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant