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

web.console.templates is not overridden by passed argument #1700

Closed
anioool opened this Issue Jun 2, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@anioool
Copy link

anioool commented Jun 2, 2016

Hi,

I planed to add node extractor and use the default templates for that. I tried to call the template but I am receiving the same error - files don't exist. I checked the command line flags and I found that flag web.console.templates contains default value instead of value which i passed in command line.

I am executing prometheus by using following commad:
/bin/prometheus -config.file=/etc/prometheus/prometheus.yml \ -storage.local.path=/prometheus \ -alertmanager.url=http://localhost:9093 \ -web.console.libraries=/etc/prometheus/console_libraries \ -web.console.templates=/etc/prometheus/consoles \ -logtostderr
Tested with versions : 0.18.0 and 0.19.2

@brian-brazil

This comment has been minimized.

Copy link
Member

brian-brazil commented Jun 2, 2016

This is working correctly for me on 0.18.0 and 0.19.2. Are you sure you're talking to the Prometheus you think you're talking to?

@anioool

This comment has been minimized.

Copy link
Author

anioool commented Jun 2, 2016

Strange... I am using the docker and deployments scripts for kubernetes cluster. For web.console.libraries flag I do not have any problems.

Dockerfile
`FROM prom/prometheus

COPY prometheus.yaml /etc/prometheus/prometheus.yml
COPY alerts_config /etc/prometheus/alerts_config

VOLUME [ "/prometheus" ]
WORKDIR /prometheus

EXPOSE 9090
CMD ["/bin/prometheus", "-config.file=/etc/prometheus/prometheus.yml", "-storage.local.path=/prometheus", "-logtostderr", "-web.console.libraries=/etc/prometheus/console_libraries", "-web.console.templates=/etc/prometheus/consoles" ]
`

Container
- name: prometheus-init image: registry.xxx/prometheus:1.2.0 command: - /bin/sh - -c - | while [ ! -f /data/lock/alertmanager.lock ] do echo 'Alertmanager is still not up and running. Sleep 10 seconds...' sleep 10 done echo 'Trying to launch prometheus app...' && /bin/prometheus -config.file=/etc/prometheus/prometheus.yml \ -storage.local.path=/prometheus \ -alertmanager.url=http://localhost:9093 \ -web.console.libraries=/etc/prometheus/console_libraries \ -web.console.templates=/etc/prometheus/consoles \ -logtostderr volumeMounts: - name: prometheus-data mountPath: /data/lock imagePullSecrets: - name: registry.xxx.com

@anioool

This comment has been minimized.

Copy link
Author

anioool commented Jun 3, 2016

Hi,

Maybe it will sound stupid by changed the order of passed parameters and after that web.console.templates flag has been overridden.

Sorry for raising not necessary ticket.

@anioool anioool closed this Jun 3, 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.