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

Retention policy creation not enforcing minimum retention duration #2991

Closed
corylanou opened this issue Jun 15, 2015 · 4 comments
Closed

Retention policy creation not enforcing minimum retention duration #2991

corylanou opened this issue Jun 15, 2015 · 4 comments
Assignees
Labels
Milestone

Comments

@corylanou
Copy link
Contributor

You can currently create a RP with a duration of 1s. We used to have a minimum but I can't remember what it is. It should be more than 1s though I would think.

There is a test in cmd/influxd/run/server_test.go, testname: TestServer_RetentionPolicyCommands, table test: Ensure retention policy with unacceptable retention cannot be created that is currently skipped that is able to replicate the issue.

@otoolep
Copy link
Contributor

otoolep commented Jun 15, 2015

@corylanou - this minimum used to be 1 hour.

@beckettsean
Copy link
Contributor

@corylanou confirming the minimum RP duration was 1 hour: #2760.

Is max still correct? #2762

@beckettsean beckettsean added this to the 0.9.2 milestone Jun 16, 2015
otoolep added a commit that referenced this issue Jun 17, 2015
@otoolep
Copy link
Contributor

otoolep commented Jun 17, 2015

#3027

@otoolep otoolep modified the milestones: 0.9.1, 0.9.2 Jun 17, 2015
@otoolep otoolep self-assigned this Jun 17, 2015
@otoolep
Copy link
Contributor

otoolep commented Jun 17, 2015

All fixed for 0.9.1

@otoolep otoolep closed this as completed Jun 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants