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 etcd username and userpassword from config #4205

Merged
merged 3 commits into from
Jun 17, 2021
Merged

Added etcd username and userpassword from config #4205

merged 3 commits into from
Jun 17, 2021

Conversation

ymhhh
Copy link
Contributor

@ymhhh ymhhh commented May 20, 2021

To use etcd with user & password when it should auth

@ymhhh ymhhh changed the title [ENHANCEMENT] Added etcd username and userpassword from config Added etcd username and userpassword from config May 20, 2021
@pstibrany
Copy link
Contributor

Thank you for your PR. To make it into mergeable state, please do the following:

  • add two new fields into RegisterFlagsWithPrefix method in etcd.go
  • after doing that, run make BUILD_IN_CONTAINER=false doc to regenerate documentation
  • add entry to CHANGELOG file

Thank you!

Copy link
Contributor

@pstibrany pstibrany left a comment

Choose a reason for hiding this comment

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

LGTM, thank you.

CHANGELOG.md Outdated
@@ -2,6 +2,7 @@

## master / unreleased

* [ENHANCEMENT] Added etcd username and userpassword from config #4205
Copy link
Contributor

Choose a reason for hiding this comment

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

Order of entries in the changelog should be: [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].

Suggested change
* [ENHANCEMENT] Added etcd username and userpassword from config #4205
* [ENHANCEMENT] Etcd: Added username and password to etcd config. #4205

Copy link
Contributor

Choose a reason for hiding this comment

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

@ymhhh Could you fix this, please? We would be very happy to merge once done! Thanks!

CHANGELOG.md Outdated
@@ -2,6 +2,7 @@

## master / unreleased

* [ENHANCEMENT] Added etcd username and userpassword from config #4205
Copy link
Contributor

Choose a reason for hiding this comment

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

@ymhhh Could you fix this, please? We would be very happy to merge once done! Thanks!

Henry Huang and others added 3 commits June 17, 2021 18:02
Signed-off-by: Henry Huang <[email protected]>
Signed-off-by: Henry Huang <[email protected]>
Signed-off-by: Marco Pracucci <[email protected]>
@pracucci pracucci merged commit 9d049ab into cortexproject:master Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants