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

added documentation for s3_endpoint (#78 / 213e9a8 and #92) #93

Merged
merged 1 commit into from
May 8, 2019

Conversation

danielbodnar
Copy link
Contributor

Q A
Bug fix? no
New feature? no
API breaks? no
Deprecations? no
Related tickets Additional documentation for #78 (213e9a8) and banzaicloud/banzai-charts#528
License Apache 2.0

What's in this PR?

I added additional documentation for the s3_endpoint variable and will be submitting another PR to the banzaicloud/banzai-charts.

Why?

I want to use Wasabi instead of Amazon, and noticed the feature existed but appeared undocumented.

@CLAassistant
Copy link

CLAassistant commented May 7, 2019

CLA assistant check
All committers have signed the CLA.

@danielbodnar
Copy link
Contributor Author

Signed the CLA and submitted an issue (#92). Let me know if there's anything else ya need me to do.

@danielbodnar danielbodnar changed the title added documentation for #78 and 213e9a8 added documentation for s3_endpoint (#78 / 213e9a8 and #92) May 7, 2019
danielbodnar added a commit to danielbodnar/banzai-charts that referenced this pull request May 7, 2019
@asdwsda asdwsda requested a review from tarokkk May 8, 2019 09:51
ahma pushed a commit to banzaicloud/banzai-charts that referenced this pull request May 8, 2019
@kozmagabor kozmagabor requested review from ahma and orymate May 8, 2019 12:07
Copy link
Contributor

@ahma ahma left a comment

Choose a reason for hiding this comment

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

Thanks @danielbodnar

@ahma ahma merged commit 9931ed3 into kube-logging:master May 8, 2019
simonkey007 pushed a commit to simonkey007/logging-operator that referenced this pull request Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants