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.

Sign up for GitHub

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

Jump to bottom

Human-readable time durations in config files #56

Closed
dankraw opened this issue Feb 15, 2016 · 1 comment
Closed

Human-readable time durations in config files #56

dankraw opened this issue Feb 15, 2016 · 1 comment
Assignees
@janisz
Labels
enhancement help wanted

Comments

@dankraw
Copy link
Contributor

dankraw commented Feb 15, 2016

It would be nice if we had human-readable time durations in config files instead of the nanosecond integers, for example:

"Sync": {
  "Interval": 900000000000
}

would become

"Sync": {
  "Interval": "15m"
}
@janisz
Copy link
Contributor

janisz commented Feb 22, 2016

Also it will be great if file definition could be override with labels, and when value if not provided in file to use default.

@janisz janisz added this to the 1.0 milestone Aug 16, 2016
@janisz janisz removed this from the 1.0 milestone Sep 22, 2016
janisz added a commit that referenced this issue Dec 1, 2016
@janisz
Fixes #56 | Humman-readable interval in config
98a16cb 
Currently we allow user to specify intervals/timeouts in
human readable format only in command line flags. This commit
introduce allowign user to specify duration as inteager nanoseconds or
as formated string. String should have format supported by
time.ParseDuration.
@janisz janisz self-assigned this Dec 1, 2016
janisz added a commit that referenced this issue Dec 1, 2016
@janisz
Fixes #56 | Humman-readable interval in config
a008c36 
Currently we allow user to specify intervals/timeouts in
human readable format only in command line flags. This commit
introduce allowign user to specify duration as inteager nanoseconds or
as formated string. String should have format supported by
time.ParseDuration.
janisz added a commit that referenced this issue Dec 2, 2016
@janisz
Fixes #56 | Humman-readable interval in config
a9c9b7d 
Currently we allow user to specify intervals/timeouts in
human readable format only in command line flags. This commit
introduce allowign user to specify duration as inteager nanoseconds or
as formated string. String should have format supported by
time.ParseDuration.
@janisz janisz closed this as completed in 8353e73 Dec 2, 2016
@janisz janisz removed the in progress label Dec 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@janisz janisz

Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@janisz @dankraw