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

Update documentation for default-user S3 credentials #309

Merged
merged 3 commits into from
Jan 16, 2023

Conversation

giubacc
Copy link

@giubacc giubacc commented Jan 10, 2023

Fixes: https://github.com/aquarist-labs/s3gw/issues/303
Signed-off-by: Giuseppe Baccini giuseppe.baccini@suse.com

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • CHANGELOG.md has been updated should there be relevant changes in this PR.

@giubacc giubacc requested a review from m-ildefons January 10, 2023 14:22
@giubacc giubacc self-assigned this Jan 10, 2023
@giubacc giubacc added the kind/documentation Improvements or additions to documentation label Jan 10, 2023
@giubacc giubacc added this to the v0.11.0 milestone Jan 10, 2023
Fixes: https://github.com/aquarist-labs/s3gw/issues/303
Signed-off-by: Giuseppe Baccini <giuseppe.baccini@suse.com>
Add information on how to obtain the randomly generated access key and
secret key, should the user not have provided them.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Copy link
Contributor

@m-ildefons m-ildefons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍 this looks good.
I'm missing information on how to obtain the randomly generated secrets though.
See: giubacc#1

@giubacc
Copy link
Author

giubacc commented Jan 13, 2023

Thanks +1 this looks good. I'm missing information on how to obtain the randomly generated secrets though. See: giubacc#1

merged!

Copy link
Contributor

@m-ildefons m-ildefons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT

@m-ildefons m-ildefons merged commit 2875c0c into s3gw-tech:main Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

charts: add existingSecret option
2 participants