Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

build: update envoy-build-tools #24460

Closed
wants to merge 4 commits into from
Closed

build: update envoy-build-tools #24460

wants to merge 4 commits into from

Conversation

jpsim
Copy link
Contributor

@jpsim jpsim commented Dec 8, 2022 •
edited

Pulls in envoyproxy/envoy-build-tools#188 which adds Java and Android tools to the Docker image.

Signed-off-by: JP Simard jp@jpsim.com

Commit Message:
Additional Description:
Risk Level: Moderate to CI jobs, Low to Envoy
Testing: CI
Docs Changes:
Release Notes:
Platform Specific Features:
Fixes: envoyproxy/envoy-mobile#2614

@jpsim
build: update envoy-build-tools
010320c 
Signed-off-by: JP Simard <jp@jpsim.com>
@repokitteh-read-only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #24460 was opened by jpsim.

see: more, trace.

@repokitteh-read-only
Copy link

CC @envoyproxy/mobile-maintainers: FYI only for changes made to (mobile/).
envoyproxy/mobile-maintainers assignee is @alyssawilk

🐱

Caused by: #24460 was opened by jpsim.

see: more, trace.

@jpsim
Update envoy_build_tools in bazel/repository_locations.bzl
1ca84f6 
Signed-off-by: JP Simard <jp@jpsim.com>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Dec 8, 2022
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #24460 was synchronize by jpsim.

see: more, trace.

@jpsim
Copy link
Contributor Author

jpsim commented Dec 9, 2022

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #24460 (comment) was created by @jpsim.

see: more, trace.

@jpsim jpsim mentioned this pull request Dec 9, 2022
Merged
@jpsim
Copy link
Contributor Author

jpsim commented Dec 9, 2022

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #24460 (comment) was created by @jpsim.

see: more, trace.

@jpsim
Copy link
Contributor Author

jpsim commented Dec 14, 2022

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #24460 (comment) was created by @jpsim.

see: more, trace.

@jpsim
Merge remote-tracking branch 'origin/main' into jp-update-envoy-build…
2d8b10b 
…-tools

* origin/main: (59 commits)
  Make IsOkAndHolds matcher work with submatchers (#24498)
  ios: fix platform key value store (#24532)
  make ClusterInfo::traffic_stats_ a unique_ptr, so that later we can lazy-init it later. (#24406)
  quic: splitting into client and server (#24513)
  fixing coverage (#24529)
  ci: Add `examplesOnly` condition (#24465)
  ci: sonatype_nexus_upload.py: remove unused format argument (#24471)
  deps: Bump `build_bazel_rules_apple` -> 1.1.3 (#24527)
  deps: Bump `com_github_nghttp2_nghttp2` -> 1.51.0 (#24525)
  deps: Bump `rules_license` -> 0.0.4 (#24523)
  build(deps): bump sphinxcontrib-httpdomain from 1.8.0 to 1.8.1 in /mobile/docs (#24126)
  build(deps): bump github/codeql-action from 2.1.35 to 2.1.36 (#24473)
  build(deps): bump openpolicyagent/opa from 0.47.2-istio to 0.47.3-istio in /examples/ext_authz (#24514)
  build(deps): bump node from `80844b6` to `2770c78` in /examples/ext_authz/auth/http-service (#24515)
  build(deps): bump abseil-cpp to latest version (#24386)
  xDS: add xDS config tracker extension point (#23485)
  kafka: add shared consumer manager (#24494)
  coverage: Improve test coverage (#24355)
  deps: Bump `rules_python` -> 0.16.1 (#24344)
  ci: revert disable running the Objective-C integration app (#24478)" (#24496)
  ...

Signed-off-by: JP Simard <jp@jpsim.com>
@jpsim
Use leaner Docker image where possible
b5ff0c0 
Signed-off-by: JP Simard <jp@jpsim.com>
@jpsim
Copy link
Contributor Author

jpsim commented Dec 14, 2022

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit didn't fail.

🐱

Caused by: a #24460 (comment) was created by @jpsim.

see: more, trace.

@jpsim
Copy link
Contributor Author

jpsim commented Dec 14, 2022

Waiting on envoyproxy/envoy-build-tools#190

@alyssawilk alyssawilk removed their assignment Dec 15, 2022
@RyanTheOptimist RyanTheOptimist removed their assignment Jan 12, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Feb 12, 2023
@github-actions
Copy link

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot closed this Feb 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RyanTheOptimist RyanTheOptimist Awaiting requested review from RyanTheOptimist RyanTheOptimist will be requested when the pull request is marked ready for review RyanTheOptimist is a code owner

@alyssawilk alyssawilk Awaiting requested review from alyssawilk alyssawilk will be requested when the pull request is marked ready for review alyssawilk is a code owner

@abeyad abeyad Awaiting requested review from abeyad abeyad will be requested when the pull request is marked ready for review abeyad is a code owner

@fredyw fredyw Awaiting requested review from fredyw fredyw will be requested when the pull request is marked ready for review fredyw is a code owner

Assignees
No one assigned
Labels
deps Approval required for changes to Envoy's external dependencies mobile stale stalebot believes this issue/PR has not been touched recently
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

android/ci: run CI jobs on Linux
3 participants
@jpsim @alyssawilk @RyanTheOptimist