search

envoyproxy / java-control-plane

Java implementation of an Envoy gRPC control plane
Apache License 2.0
293 stars 136 forks source link

Fix gitHub action to upgrade envoy protos #197

Closed rulex123 closed 2 years ago

rulex123 commented 2 years ago

I had to update a couple of things to get the github action that upgrades the Envoy API to run successfully.

I have tested it in my fork for Envoy v1.21.0, so the next step would be to configure it to run automatically whenever a new Envoy version is released.

Here is the PR that was created through the github action -> https://github.com/rulex123/java-control-plane/pulls

@onemanbucket can you take a look when you get a chance?

codecov-commenter commented 2 years ago

Codecov Report

Merging #197 (c3ae231) into main (c8484a3) will increase coverage by 0.61%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #197      +/-   ##
============================================
+ Coverage     82.04%   82.65%   +0.61%     
  Complexity      277      277              
============================================
  Files            31       31              
  Lines           980      980              
  Branches         78       78              
============================================
+ Hits            804      810       +6     
+ Misses          143      137       -6     
  Partials         33       33
Impacted Files Coverage Δ
.../io/envoyproxy/controlplane/cache/SimpleCache.java 82.97% <0.00%> (-0.71%) :arrow_down:
...o/envoyproxy/controlplane/cache/TestResources.java 87.76% <0.00%> (+3.72%) :arrow_up:

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 c8484a3...c3ae231. Read the comment docs.