Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

traefik: Update chart, add servicemonitor label to prom service (#361) #362

Merged
merged 1 commit into from
Jul 15, 2020

Conversation

gracedo
Copy link
Contributor

@gracedo gracedo commented Jul 15, 2020

  • chore: Copy prev revision

  • chore: Update chart, add servicemonitor label to prom service

  • test: Remove prometheus from testing groups w/traefik

  • fix: Move servicemonitor label to correct block

  • chore: Bump chart

cherry-picked from ceac181

What type of PR is this?

Backport #361

What this PR does/ why we need it:

Which issue(s) this PR fixes:

https://jira.d2iq.com/browse/D2IQ-70138

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

Checklist

  • The commit message explains the changes and why are needed.
  • The code builds and passes lint/style checks locally.
  • The relevant subset of integration tests pass locally.
  • The core changes are covered by tests.
  • The documentation is updated where needed.

* chore: Copy prev revision

* chore: Update chart, add servicemonitor label to prom service

* test: Remove prometheus from testing groups w/traefik

* fix: Move servicemonitor label to correct block

* chore: Bump chart
@gracedo gracedo added ready backport This is a PR that backports to a branch labels Jul 15, 2020
@gracedo gracedo requested review from joejulian and dkoshkin July 15, 2020 20:36
@gracedo gracedo requested a review from a team as a code owner July 15, 2020 20:36
@gracedo gracedo self-assigned this Jul 15, 2020
@joejulian
Copy link
Contributor

konvoy failures all appear to be aws api related.

@joejulian joejulian merged commit 21a7f64 into stable Jul 15, 2020
@joejulian joejulian deleted the stable_backport361 branch July 15, 2020 22:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport This is a PR that backports to a branch ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants