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

add option to load prometheus data from a dir #66

Merged
merged 2 commits into from
Sep 29, 2016
Merged

Conversation

tzach
Copy link
Contributor

@tzach tzach commented Sep 29, 2016

This PR takes @kongove work in 496f701 and add it as a parameter yo start-all.sh, making it easier to use

@tzach
Copy link
Contributor Author

tzach commented Sep 29, 2016

@kongove @nirmaayan please review

sudo docker run -d -v $PWD/prometheus/prometheus.yml:/etc/prometheus/prometheus.yml:Z -p 9090:9090 --name aprom prom/prometheus:v1.0.0
else
echo "Loading prometheus data from $DATA_DIR"
sudo docker run -d -v $DATA_DIR:/prometheus -v $PWD/prometheus/prometheus.yml:/etc/prometheus/prometheus.yml:Z -p 9090:9090 --name aprom prom/prometheus:v1.0.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I needed to add flag Z to -v $DATA_DIR:/prometheus:Z, otherwise Prometheus complained that:

level=error msg="Error opening memory series storage: open /prometheus/VERSION: permission denied" source="main.go:143" 

@nirmaayan
Copy link
Contributor

@tzach, not working
using the following command
./start-all.sh -d /home/nir/1/metrics_performance_degradation/
Monitor is up, there is no metrics in the monitor

@tzach
Copy link
Contributor Author

tzach commented Sep 29, 2016

@pdziepak thanks, :Z added. It does work for me with and without it
@nirmaayan I suspect your metrics are from 25/9/2016 while Grafana default presentation is the last 5 min

@nirmaayan
Copy link
Contributor

@tzach working now - please merge

@tzach
Copy link
Contributor Author

tzach commented Sep 29, 2016

@pdziepak does the second commit satisfy?

@pdziepak
Copy link
Contributor

Yes, it does.

@tzach tzach merged commit 9d06008 into master Sep 29, 2016
@tzach tzach deleted the load-data-to-prometheus branch September 29, 2016 14:58
amnonh added a commit to amnonh/scylla-grafana-monitoring that referenced this pull request Feb 17, 2021
…ce/' changes from 7a541804..bc069204

ae9a7ff5 Release 1.0.0
a3da86be Merge pull request scylladb#94 from el-psycho/master
0d98ebfa Fix duplicate alert rows
b5e7ca5f Use correct logo file name
aed6dac0 set type returned by custom functions
25d9489d cleanup query function
81db37c5 do not perform hidden queries
1e4bc9c4 update README
b65e4b0c update README
0567785b allow filtering on receiver
e4ffd46d update README
f8a57a1f avoid error on empty records
a73e51c2 rewrite datasource for grafana 7.x
4fe2aadf Merge pull request scylladb#66 from timmow/add-credentials
9e99e6cf Add withCredentials if present
7c574f4f update dependencies
511236da Merge pull request scylladb#64 from Mobiliteitsfabriek/feature/query-filters-silenced-active-inhibited
36e5a6ca Merge pull request scylladb#78 from camptocamp/dependabot/npm_and_yarn/websocket-extensions-0.1.4
b1714a5f Merge pull request scylladb#82 from camptocamp/dependabot/npm_and_yarn/lodash-4.17.19
f3eff01e Merge pull request scylladb#90 from camptocamp/dependabot/npm_and_yarn/ini-1.3.7
c20d3170 Bump ini from 1.3.5 to 1.3.7
1e63c2d2 Bump lodash from 4.17.15 to 4.17.19
59b35f8e Bump websocket-extensions from 0.1.3 to 0.1.4
0d4803bd Rebuild after rebasing
e76612fc Support multiple filters
ea5df703 Fix default for queryActive
dc5b3e1d Fix formatting
2ee31740 Add support for querying inhibited or silenced too

git-subtree-dir: grafana/plugins/camptocamp-prometheus-alertmanager-datasource
git-subtree-split: bc069204ac8bc6e1b8bd713758470c02ee831b06
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

Successfully merging this pull request may close these issues.

3 participants