Skip to content

Conversation

@srkethireddy
Copy link
Contributor

@srkethireddy srkethireddy commented Oct 27, 2025

Adding an option to override "metadata.google.internal.:8080" by setting a value for GCE_METADATA_HOST environment variable.

b/451639946

cc: @apolcyn , @anicr7

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 27, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@srkethireddy srkethireddy marked this pull request as draft October 27, 2025 19:48
@srkethireddy srkethireddy force-pushed the metadata-server-address branch from 4516b88 to f2eb129 Compare October 27, 2025 22:16
@srkethireddy srkethireddy marked this pull request as ready for review October 27, 2025 23:14
Copy link
Member

@ejona86 ejona86 left a comment

Choose a reason for hiding this comment

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

Is there an internal bug or such to link to here? I'd really want some breadcrumb to follow.

@srkethireddy
Copy link
Contributor Author

Is there an internal bug or such to link to here? I'd really want some breadcrumb to follow.

Here is a bug for context: b/451639946.

@ejona86 ejona86 changed the title Metadata Server Address Override with Environment Variable alts: Metadata Server Address Override with Environment Variable Oct 30, 2025
@srkethireddy srkethireddy marked this pull request as draft October 30, 2025 23:00
@ejona86
Copy link
Member

ejona86 commented Oct 30, 2025

/gcbrun

@ejona86 ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Oct 30, 2025
@grpc-kokoro grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Oct 30, 2025
@ejona86 ejona86 marked this pull request as ready for review October 30, 2025 23:13
@ejona86 ejona86 merged commit d89628c into grpc:master Oct 31, 2025
16 of 17 checks passed
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