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

don't overwrite the ReplicaSet service label #1482

Merged
merged 1 commit into from
Mar 2, 2020

Conversation

gdbelvin
Copy link
Contributor

All pods were only declaring the 'cloud-sql-sidecar' service label, completely breaking service discovery

All pods were only declaring the 'cloud-sql-sidecar' service label, completely breaking service discovery
@gdbelvin gdbelvin requested a review from a team as a code owner February 28, 2020 20:02
@gdbelvin gdbelvin requested a review from AlCutter February 28, 2020 20:02
@codecov
Copy link

codecov bot commented Feb 28, 2020

Codecov Report

Merging #1482 into master will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1482      +/-   ##
==========================================
+ Coverage   67.83%   67.90%   +0.07%     
==========================================
  Files          54       54              
  Lines        4026     4026              
==========================================
+ Hits         2731     2734       +3     
+ Misses        893      891       -2     
+ Partials      402      401       -1     
Impacted Files Coverage Δ
core/sequencer/server.go 73.94% <0.00%> (+0.97%) ⬆️

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 b14c3eb...6fb386e. Read the comment docs.

@gdbelvin gdbelvin requested review from Mercurrent and removed request for AlCutter March 2, 2020 11:37
@gdbelvin gdbelvin added the bug label Mar 2, 2020
@gdbelvin gdbelvin merged commit 260f9dc into google:master Mar 2, 2020
@gdbelvin gdbelvin deleted the selector branch March 2, 2020 13:05
gdbelvin added a commit to gdbelvin/keytransparency that referenced this pull request Mar 25, 2020
* master: (109 commits)
  Upgrade Prometheus (google#1500)
  Run `go build ./...` with github action (google#1446)
  Replace tinkio.ProtoFile with keyset.BinaryReader (google#1492)
  Bump github.com/golang/mock from 1.4.1 to 1.4.2
  Bump github.com/prometheus/client_golang from 1.5.0 to 1.5.1
  Bump github.com/golang/protobuf from 1.3.3 to 1.3.5
  Bump github.com/google/tink from 1.3.0-rc4 to 1.3.0
  New Design Doc (google#1469)
  Acknowledgements (google#1490)
  Use cases (google#1489)
  Bump github.com/prometheus/client_golang from 1.4.1 to 1.5.0 (google#1486)
  README.md # Related (google#1485)
  Explain authorized keys (google#1484)
  Use new prometheus sidecar deployment (google#1483)
  don't overwrite the ReplicaSet service label (google#1482)
  Update encrypted creds (google#1481)
  Pickup DB_HOST env variable (google#1480)
  Fix credential encryption (google#1479)
  Use Cloud MySQL in GKE (google#1473)
  Update client_secrets.json.enc (google#1478)
  ...
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