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

influxdb 2.0.0-beta.9 setup wrong confirrmation of retention #17862

Closed
poulhs opened this issue Apr 24, 2020 · 3 comments · Fixed by #18017
Closed

influxdb 2.0.0-beta.9 setup wrong confirrmation of retention #17862

poulhs opened this issue Apr 24, 2020 · 3 comments · Fixed by #18017

Comments

@poulhs
Copy link

poulhs commented Apr 24, 2020

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

  1. ./influxdb_2.0.0-beta.9_linux_amd64/influxd
  2. ./influxdb_2.0.0-beta.9_linux_amd64/influx setup --org bug --bucket bug --retention 1h --username bug --password buggy

Expected behavior:
Welcome to InfluxDB 2.0!

You have entered:
Username: bug
Organization: bug
Bucket: bug
Retention Period: 1 hrs
Confirm? (y/n):
Actual behavior:
Welcome to InfluxDB 2.0!

You have entered:
Username: bug
Organization: bug
Bucket: bug
Retention Period: 3600000000000 hrs
Confirm? (y/n):
Environment info:

  • System Info: Linux x86_64

  • InfluxDB version: InfluxDB 2.0.0-beta.9 (git: eb6a552) build_date: 2020-04-24T20:14:53Z

  • System info: Run uname -srm and copy the output here

  • InfluxDB version: Run influxd version and copy the output here

  • Other relevant environment details: Container runtime, disk info, etc

Time matters....

@russorat
Copy link
Contributor

russorat commented May 8, 2020

@poulhs thanks for the issue. yeah this is a bug. we will take a look.

@jsteenb2
Copy link
Contributor

jsteenb2 commented May 8, 2020

@poulhs thanks for filing this bug! fix is in and should land in the next release 👍

@poulhs
Copy link
Author

poulhs commented May 9, 2020

but it was such a nice bug.... 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants