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

Reword to let newcomers work with a release, not with HEAD. #91

Closed
mkevac opened this issue May 20, 2015 · 6 comments
Closed

Reword to let newcomers work with a release, not with HEAD. #91

mkevac opened this issue May 20, 2015 · 6 comments

Comments

@mkevac
Copy link

mkevac commented May 20, 2015

If doing everything from 'getting started' page, one will find out that config is wrong:

marko@marko-desktop:~/tmp/prometheus (master) $ ./prometheus -config.file=prometheus.conf
prometheus, version 0.13.3 (branch: master, revision: 572b280)
  build user:       marko@marko-desktop
  build date:       20150520-18:45:13
  go version:       1.4.2
E0520 18:56:16.629990   18370 main.go:224] Couldn't load configuration (-config.file=prometheus.conf): yaml: line 3: did not find expected ',' or '}'
E0520 18:56:16.630068   18370 main.go:225] Note: The configuration format has changed with version 0.14, please check the documentation.
@fabxc
Copy link
Contributor

fabxc commented May 20, 2015

So you pulled the master branch, into which we just merged a large set of changes.
Unfortunate timing as we have not officially made this a release candidate yet.

You can find docs matching those changes here:
https://github.com/prometheus/docs/blob/master/content/docs/operating/configuration.md

If you would like to run the latest version (which matches the documentation on prometheus.io) please checkout the 0.13.3 version tag and build from that.

On Wed, May 20, 2015 at 5:56 PM Marko Kevac notifications@github.com
wrote:

If doing everything from 'getting started' page, one will find out that
config is wrong:

marko@marko-desktop:~/tmp/prometheus (master) $ ./prometheus -config.file=prometheus.conf
prometheus, version 0.13.3 (branch: master, revision: 572b280)
build user: marko@marko-desktop
build date: 20150520-18:45:13
go version: 1.4.2
E0520 18:56:16.629990 18370 main.go:224] Couldn't load configuration (-config.file=prometheus.conf): yaml: line 3: did not find expected ',' or '}'
E0520 18:56:16.630068 18370 main.go:225] Note: The configuration format has changed with version 0.14, please check the documentation.


Reply to this email directly or view it on GitHub
#91.

@beorn7
Copy link
Member

beorn7 commented May 20, 2015 via email

@beorn7 beorn7 changed the title Config wrong Reword to let newcomers work with a release, not with HEAD. May 20, 2015
@juliusv
Copy link
Member

juliusv commented May 20, 2015

...or download a released binary build.

@beorn7 beorn7 closed this as completed in f0821ec May 22, 2015
@mkevac
Copy link
Author

mkevac commented May 22, 2015

Strange fix. Instead of fixing documentation, you introduced additional work for a newbie to do. Instead of him just copy-pasting commands, he must check and wonder which version he has.

@beorn7
Copy link
Member

beorn7 commented May 22, 2015

@mkevac ???? Documentation has been fixed.

@mkevac
Copy link
Author

mkevac commented May 23, 2015

Yes, it was. I am sorry. But not with mentioned change (beorn7 closed this in f0821ec 21 hours ago), but with f733434. This confused me.

aylei pushed a commit to aylei/docs that referenced this issue Oct 28, 2019
* Update migration.md

* op-guide: add skip-tz-tsc flag to mydumper prometheus#109
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

No branches or pull requests

4 participants