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

feat(cmd/influx): allow for setting v1 passwords via CLI args #20128

Merged
merged 2 commits into from
Nov 20, 2020

Conversation

danxmoran
Copy link
Contributor

Closes #20107

Backports #20123 to 2.0

  • CHANGELOG.md updated with a link to the PR (not the Issue)
  • Well-formatted commit messages
  • Rebased/mergeable
  • Tests pass
  • http/swagger.yml updated (if modified Go structs or API)
  • Feature flagged (if modified API)
  • Documentation updated or issue created (provide link to issue/pr)
  • Signed CLA (if not already signed)

@danxmoran danxmoran changed the title feat(cmd/influx): allow for setting v1 passwords via CLI args (#20123) feat(cmd/influx): allow for setting v1 passwords via CLI args Nov 20, 2020
### Features

1. [20128](https://github.com/influxdata/influxdb/pull/20128): Allow password to be specified as a CLI option in `influx v1 auth create`.
1. [20128](https://github.com/influxdata/influxdb/pull/20128): Allow password to be specified as a CLI option in `influx v1 auth set-password`.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gshif this look good to you, won't cause pain when you prep the release later? I kept the text from the original PR and just updated the numbers.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok

@danxmoran danxmoran merged commit d5fd456 into 2.0 Nov 20, 2020
@danxmoran danxmoran deleted the dm-backport-v1-auth-password-20107 branch November 20, 2020 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants