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.

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

bazel: Bump -> 7.1.2 #37578

Merged
merged 1 commit into from
Dec 19, 2024
Merged

bazel: Bump -> 7.1.2 #37578

merged 1 commit into from
Dec 19, 2024

Conversation

phlax
Copy link
Member

@phlax phlax commented Dec 9, 2024

No description provided.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Dec 9, 2024
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: #37578 was synchronize by phlax.

see: more, trace.

@phlax phlax changed the title bazel: Bump -> 7.4.1 [WIP] bazel: Bump -> 7.4.1 Dec 9, 2024
@phlax phlax marked this pull request as draft December 9, 2024 17:14
@phlax phlax force-pushed the deps-bump-bazel branch 7 times, most recently from 05c7847 to eca9979 Compare December 10, 2024 21:25
@phlax phlax force-pushed the deps-bump-bazel branch 5 times, most recently from cf1ffc1 to fe07a68 Compare December 13, 2024 18:02
@phlax phlax changed the title [WIP] bazel: Bump -> 7.4.1 bazel: Bump -> 7.1.2 Dec 13, 2024
@phlax phlax marked this pull request as ready for review December 13, 2024 18:04
@phlax phlax changed the title bazel: Bump -> 7.1.2 [WIP] bazel: Bump -> 7.1.2 Dec 13, 2024
@phlax phlax marked this pull request as draft December 13, 2024 18:23
keith
keith previously approved these changes Dec 13, 2024
@phlax
Copy link
Member Author

phlax commented Dec 13, 2024

i got passed the original problem but its created new errors - so reWIPped and trying to debug

abeyad
abeyad previously approved these changes Dec 13, 2024
Copy link
Contributor

@abeyad abeyad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks @phlax this is awesome!

@phlax phlax force-pushed the deps-bump-bazel branch 7 times, most recently from 167163b to bbd9aa8 Compare December 17, 2024 20:43
@phlax phlax changed the title [WIP] bazel: Bump -> 7.1.2 bazel: Bump -> 7.1.2 Dec 17, 2024
@phlax phlax marked this pull request as ready for review December 17, 2024 20:49
@phlax
Copy link
Member Author

phlax commented Dec 17, 2024

/retest flaky tsan

Signed-off-by: Ryan Northey <[email protected]>
@phlax
Copy link
Member Author

phlax commented Dec 18, 2024

/retest

Copy link
Contributor

@RyanTheOptimist RyanTheOptimist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, very exciting!

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Dec 19, 2024
@phlax phlax merged commit 71aeee2 into envoyproxy:main Dec 19, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants