Skip to content

build: update envoy and fix ci build#1232

Closed
junr03 wants to merge 38 commits intomainfrom
use-container-linux
Closed

build: update envoy and fix ci build#1232
junr03 wants to merge 38 commits intomainfrom
use-container-linux

Conversation

@junr03
Copy link
Copy Markdown
Member

@junr03 junr03 commented Jan 8, 2021

Description: this PR updates envoy and makes the following changes:

  • linux CI jobs now use the envoy-build-ubuntu container published by envoy-build-tools
  • updates upload/download-artifact to v2.
  • specified using remote jdk toolchain for the coverage build.
  • temporarily moves java and kt unit tests to macOS. Subsequent PR will move as many CI jobs as possible to run on linux runners, as this saves time via container execution; and concurrency as the linux limit is higher than the macOS limit.
  • updates protobuf dep to 3.14.0
  • updates metrics transport to V3 Api
  • updates bazel build targets used in several places in the codebase
  • Fixes warnings in C++ tests.
    Risk Level: low for all jobs ran in CI. High for release and artifact jobs as they won't execute until a release is done, and they are most likely broken. Will fix after running.
    Testing: CI

Signed-off-by: Jose Nino jnino@lyft.com

Jose Nino added 30 commits January 8, 2021 15:14
Signed-off-by: Jose Nino <jnino@lyft.com>
fmt
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
fix
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
jdk
Signed-off-by: Jose Nino <jnino@lyft.com>
jdk
Signed-off-by: Jose Nino <jnino@lyft.com>
jdk
Signed-off-by: Jose Nino <jnino@lyft.com>
jdk
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Jose Nino added 8 commits January 12, 2021 12:27
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
This reverts commit 488e3c0.

Signed-off-by: Jose Nino <jnino@lyft.com>
This reverts commit c2de37e.

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
@junr03 junr03 force-pushed the use-container-linux branch from 1cafc2b to dd68b9f Compare January 13, 2021 16:08
@junr03 junr03 changed the title ci: use envoy build container for linux ci build: update envoy and fix ci build Jan 13, 2021
@junr03
Copy link
Copy Markdown
Member Author

junr03 commented Jan 13, 2021

merged onto #1230

@junr03 junr03 closed this Jan 13, 2021
@junr03 junr03 deleted the use-container-linux branch July 20, 2021 22:36
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.

1 participant