Skip to content

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
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
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