Skip to content
This repository was archived by the owner on Oct 11, 2024. It is now read-only.

Comments

Fix permissions on Prometheus config files#1429

Merged
gdbelvin merged 1 commit intogoogle:masterfrom
RJPercival:prom_cfg
Jan 8, 2020
Merged

Fix permissions on Prometheus config files#1429
gdbelvin merged 1 commit intogoogle:masterfrom
RJPercival:prom_cfg

Conversation

@RJPercival
Copy link
Contributor

Prometheus runs as nobody, which meant it couldn't read the config files in /etc/prometheus. Change ownership of the files we place there so that it can read them.

Prometheus runs as nobody, which meant it couldn't read the config
files in /etc/prometheus. Change ownership of the files we place there
so that it can read them.
@RJPercival RJPercival added the bug label Jan 8, 2020
@RJPercival RJPercival requested a review from a team as a code owner January 8, 2020 12:53
@codecov
Copy link

codecov bot commented Jan 8, 2020

Codecov Report

Merging #1429 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1429   +/-   ##
=======================================
  Coverage   66.31%   66.31%           
=======================================
  Files          54       54           
  Lines        4026     4026           
=======================================
  Hits         2670     2670           
  Misses        962      962           
  Partials      394      394

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 926eef1...0e30c92. Read the comment docs.

@gdbelvin gdbelvin merged commit 610bdd3 into google:master Jan 8, 2020
gdbelvin added a commit to gdbelvin/keytransparency that referenced this pull request Jan 8, 2020
* master:
  Fix scripts to not depend on KT being in $GOPATH (google#1428)
  Fix permissions on Prometheus config files (google#1429)
@RJPercival RJPercival deleted the prom_cfg branch January 8, 2020 14:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants