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

Aurora style job names #1602

Closed
jjneely opened this Issue Apr 28, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@jjneely
Copy link
Contributor

jjneely commented Apr 28, 2016

Would folks consider adding / (forward slash) to the JobName regular expression? This would allow Aurora-style job names to be used as scrape target job names without mangling.

https://github.com/prometheus/prometheus/blob/master/config/config.go#L31

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Apr 28, 2016

In general we can't change our data model to match the permitted characters of every possible system, it's pretty much fixed by now.

There's nothing stopping you using something with a slash as a value for a job label, but you'll need to do it via relabelling.

I would generally recommend sticking to valid label names as job labels, as we might (might) have an easier way to use them in the future that'll only work for job names which follow that constraint.

@jjneely

This comment has been minimized.

Copy link
Contributor Author

jjneely commented Apr 28, 2016

So...closing.

With your advise, and digging further into my issue of monitoring the same job in QA vs PROD on a shared execution cluster I discovered that relabeling was the right solution here.

I have a Prometheus scrape job setup per Aurora job (ZooKeeper ServerSet) where I'm using relabeling to add qa/prod labels. Of course this needs unique job names in the Prometheus configuration to parse correctly. So I made the job_name unique.

This did cause a lot of issues especially with the prometheus jobs I was monitoring. Relabeling job back to the actual job name ("prometheus" in this case) made things work as they should.

@jjneely jjneely closed this Apr 28, 2016

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.