Skip to content

Remove runtime go image and replace it with default debian image#403

Merged
paulosjca merged 4 commits intogrpc:masterfrom
ajinkyakulkarni75:go-runtime-image
Mar 27, 2025
Merged

Remove runtime go image and replace it with default debian image#403
paulosjca merged 4 commits intogrpc:masterfrom
ajinkyakulkarni75:go-runtime-image

Conversation

@ajinkyakulkarni75
Copy link
Copy Markdown
Contributor

Removed the runtime go image.

@paulosjca paulosjca added the release notes: yes Indicates that PR needs to be in release notes label Mar 27, 2025
@paulosjca paulosjca self-requested a review March 27, 2025 07:14
@paulosjca paulosjca changed the title removed docker file of runtime go image Remove runtime go image and replace it with default debian image Mar 27, 2025
export BUILD_IMAGE_PREFIX=gcr.io/grpc-testing/e2etest/init/build/
export RUN_IMAGE_PREFIX=gcr.io/grpc-testing/e2etest/init/runtime/
export GOCMD=go1.21.13
export TEST_INFRA_VERSION=v1.5.7-pre.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here it should be just v1.5.7 without -pre.1.

gcr.io/grpc-testing/e2etest/runtime/php7:v1.5.1
gcr.io/grpc-testing/e2etest/runtime/python:v1.5.1
gcr.io/grpc-testing/e2etest/runtime/ruby:v1.5.1
gcr.io/grpc-testing/e2etest/init/build/csharp:v1.5.7
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here the location and name of the images has changed.

  1. gcr.io/ becomes us-docker.pkg.dev/.
  2. init/build/ becomes init-build-.
  3. init- becomes init-.
  4. runtime/ is removed.

```shell
gcr.io/grpc-testing/e2etest/runtime/sidecar:v1.5.1
gcr.io/grpc-testing/e2etest/runtime/xds-server:v1.5.1
gcr.io/grpc-testing/e2etest/runtime/sidecar:v1.5.7
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same location changes apply here too.

@paulosjca paulosjca self-requested a review March 27, 2025 08:45
@paulosjca paulosjca merged commit 5f27f43 into grpc:master Mar 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: yes Indicates that PR needs to be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants