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

deps: bump com_github_grpc_grpc -> 1.49.2 #25131

Merged
merged 4 commits into from
Jan 30, 2023

Conversation

RiverPhillips
Copy link
Contributor

@RiverPhillips RiverPhillips commented Jan 24, 2023

Signed-off-by: river phillips riverphillips1@gmail.com

Commit Message: deps: bump com_github_grpc_grpc -> 1.49.2
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]
I

Signed-off-by: river phillips <riverphillips1@gmail.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: #25131 was opened by RiverPhillips.

see: more, trace.

@RiverPhillips RiverPhillips marked this pull request as ready for review January 24, 2023 20:37
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Jan 24, 2023
@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 @htuch

🐱

Caused by: #25131 was opened by RiverPhillips.

see: more, trace.

Signed-off-by: river phillips <riverphillips1@gmail.com>
Signed-off-by: river phillips <riverphillips1@gmail.com>
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Jan 25, 2023
@htuch htuch enabled auto-merge (squash) January 25, 2023 04:25
@phlax
Copy link
Member

phlax commented Jan 26, 2023

/retest

@repokitteh-read-only
Copy link

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

🐱

Caused by: a #25131 (comment) was created by @phlax.

see: more, trace.

htuch
htuch previously approved these changes Jan 27, 2023
@phlax
Copy link
Member

phlax commented Jan 27, 2023

@RiverPhillips the fail looks genuine:

ERROR: /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/external/com_github_grpc_grpc/src/core/BUILD:75:16: in @bazel_compdb//:aspects.bzl%compilation_database_aspect aspect on cc_library rule @com_github_grpc_grpc//src/core:transport_fwd: 
Traceback (most recent call last):
	File "/build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/external/bazel_compdb/aspects.bzl", line 312, column 48, in _compilation_database_aspect_impl
		compile_commands = _cc_compile_commands(ctx, target, feature_configuration, cc_toolchain)
	File "/build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/external/bazel_compdb/aspects.bzl", line 157, column 73, in _cc_compile_commands
		compiler_options = cc_common.get_memory_inefficient_command_line(
Error in get_memory_inefficient_command_line: Invalid toolchain configuration: Cannot find variable named 'module_name'.

https://dev.azure.com/cncf/envoy/_build/results?buildId=126148&view=logs&j=430ab721-bbba-5b3a-3c7b-ff111cc657fe&t=b6238472-0f30-5195-100c-623a3015b9ef&l=63

Signed-off-by: river phillips <riverphillips1@gmail.com>
@RiverPhillips
Copy link
Contributor Author

Cheers @phlax, this errors seems to occur on versions >=1.50.0. I'll change this PR to bump it to 1.49.2 and investigate on my end why this is failing

auto-merge was automatically disabled January 27, 2023 20:45

Head branch was pushed to by a user without write access

@RiverPhillips RiverPhillips changed the title deps: bump com_github_grpc_grpc -> 1.51.1 deps: bump com_github_grpc_grpc -> 1.49.2 Jan 27, 2023
@htuch htuch merged commit 6a9cabe into envoyproxy:main Jan 30, 2023
VishalDamgude pushed a commit to freshworks/envoy that referenced this pull request Feb 2, 2023
Signed-off-by: river phillips <riverphillips1@gmail.com>
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.

3 participants