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

[bp/1.30] CI migration stack #36393

Open
wants to merge 9 commits into
base: release/v1.30
Choose a base branch
from
Open

[bp/1.30] CI migration stack #36393

wants to merge 9 commits into from

Conversation

asedeno
Copy link
Contributor

@asedeno asedeno commented Oct 1, 2024

No description provided.

phlax and others added 9 commits September 30, 2024 19:31
@phlax @asedeno
ci/rbe: Use GCR image for Engflow RBE (envoyproxy#36032)
3289634 
our existing RBE uses the GCR image rather than the dockerhub one

this may/not be related to dockerhub rate limiting that seems to have
started recently

Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
bazel/ci: Prepare repo for using Engflow RBE (envoyproxy#36293)
40dd9b9 
Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
bazel/ci: Use workspace rather than action envs for fake scm revision (…
710a5bb 
…envoyproxy#36298)

Signed-off-by: Ryan Northey <[email protected]>

Signed-off-by: phlax <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
repo/ci: Shift remaining CI -> github (envoyproxy#35904)
ddb9afa 
Signed-off-by: Ryan Northey <[email protected]>

Signed-off-by: phlax <[email protected]>
@wbpcode @asedeno
use bash source rather than dirname $0 (envoyproxy#35804)
0df4634 
Commit Message: use bash source rather than dirname $0
Additional Description:

Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

---------

Signed-off-by: wangbaiping <[email protected]>
Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
bazel/rbe: Rename engflow_pool -> rbe_pool (envoyproxy#36301)
c8c26cd 
Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
bazel/ci: Mark builds with Engflow pool requirements (envoyproxy#36300)
67e06b2 
Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
ci/format: Remove ~unused CI format test (envoyproxy#35986)
3a323b2 
this test was never really run as `CI_BRANCH` is not automatically set
in azp

now the thing its testing has moved to bazel so the test is not really
correct

also im in the process of moving the thing being tested to toolshed
(where it will be testsed more comprehensively)

Signed-off-by: Ryan Northey <[email protected]>
@phlax @asedeno
ci: Workaround flakey ext_proc test
6c3d57a 
Signed-off-by: Ryan Northey <[email protected]>
@asedeno
Copy link
Contributor Author

asedeno commented Oct 1, 2024

@phlax, we might have an easier time with DCO if you recreate this branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wbpcode wbpcode Awaiting requested review from wbpcode wbpcode is a code owner

@soulxu soulxu Awaiting requested review from soulxu soulxu is a code owner

@mattklein123 mattklein123 Awaiting requested review from mattklein123 mattklein123 is a code owner

@alyssawilk alyssawilk Awaiting requested review from alyssawilk alyssawilk is a code owner

@ravenblackx ravenblackx Awaiting requested review from ravenblackx ravenblackx is a code owner

@phlax phlax Awaiting requested review from phlax phlax is a code owner

@RyanTheOptimist RyanTheOptimist Awaiting requested review from RyanTheOptimist RyanTheOptimist is a code owner

@abeyad abeyad Awaiting requested review from abeyad abeyad is a code owner

@fredyw fredyw Awaiting requested review from fredyw fredyw is a code owner

@kyessenov kyessenov Awaiting requested review from kyessenov kyessenov is a code owner

@tyxia tyxia Awaiting requested review from tyxia tyxia is a code owner

@nezdolik nezdolik Awaiting requested review from nezdolik nezdolik is a code owner

@yanavlasov yanavlasov Awaiting requested review from yanavlasov yanavlasov is a code owner

@ggreenway ggreenway Awaiting requested review from ggreenway ggreenway is a code owner

@zuercher zuercher Awaiting requested review from zuercher zuercher is a code owner

@KBaichoo KBaichoo Awaiting requested review from KBaichoo KBaichoo is a code owner

@adisuissa adisuissa Awaiting requested review from adisuissa adisuissa is a code owner

@jmarantz jmarantz Awaiting requested review from jmarantz jmarantz is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@asedeno @phlax @wbpcode