e2e: Replace EOL CentOS 8 with CentOS Stream 9 in test templates#1502
e2e: Replace EOL CentOS 8 with CentOS Stream 9 in test templates#1502ashokpariya0 wants to merge 1 commit intok8snetworkplumbingwg:masterfrom
Conversation
Replace quay.io/centos/centos:8 with quay.io/centos/centos:stream9 in e2e test templates to address end-of-life concerns. Signed-off-by: Ashok Pariya <ashok.pariya@ibm.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates several E2E test templates to use CentOS Stream 9 instead of CentOS 8. The reviewer suggests using the "stream9-minimal" image variant instead of the full image across all templates to reduce resource usage and pull times in CI environments.
| containers: | ||
| - name: default-route-worker1 | ||
| image: quay.io/centos/centos:8 | ||
| image: quay.io/centos/centos:stream9 |
There was a problem hiding this comment.
The centos:stream9 image is unnecessarily large (approx. 200MB) for a simple sleep container used in E2E tests. Using the minimal version quay.io/centos/centos:stream9-minimal reduces pull times and resource usage in CI environments while still providing the necessary networking tools like ip.
image: quay.io/centos/centos:stream9-minimal
| containers: | ||
| - name: default-route-worker2 | ||
| image: quay.io/centos/centos:8 | ||
| image: quay.io/centos/centos:stream9 |
There was a problem hiding this comment.
The centos:stream9 image is unnecessarily large (approx. 200MB) for a simple sleep container used in E2E tests. Using the minimal version quay.io/centos/centos:stream9-minimal reduces pull times and resource usage in CI environments while still providing the necessary networking tools like ip.
image: quay.io/centos/centos:stream9-minimal
| containers: | ||
| - name: macvlan-worker1 | ||
| image: quay.io/centos/centos:8 | ||
| image: quay.io/centos/centos:stream9 |
There was a problem hiding this comment.
The centos:stream9 image is unnecessarily large (approx. 200MB) for a simple sleep container used in E2E tests. Using the minimal version quay.io/centos/centos:stream9-minimal reduces pull times and resource usage in CI environments while still providing the necessary networking tools like ip.
image: quay.io/centos/centos:stream9-minimal
| containers: | ||
| - name: macvlan-worker2 | ||
| image: quay.io/centos/centos:8 | ||
| image: quay.io/centos/centos:stream9 |
There was a problem hiding this comment.
The centos:stream9 image is unnecessarily large (approx. 200MB) for a simple sleep container used in E2E tests. Using the minimal version quay.io/centos/centos:stream9-minimal reduces pull times and resource usage in CI environments while still providing the necessary networking tools like ip.
image: quay.io/centos/centos:stream9-minimal
| containers: | ||
| - name: simple-centos1 | ||
| image: quay.io/centos/centos:8 | ||
| image: quay.io/centos/centos:stream9 |
There was a problem hiding this comment.
The centos:stream9 image is unnecessarily large (approx. 200MB) for a simple sleep container used in E2E tests. Using the minimal version quay.io/centos/centos:stream9-minimal reduces pull times and resource usage in CI environments while still providing the necessary networking tools like ip.
image: quay.io/centos/centos:stream9-minimal
1 participant
Replace quay.io/centos/centos:8 with quay.io/centos/centos:stream9 in e2e test templates to address end-of-life concerns.
Fixes #1500
Note to reviewers:
also quay.io/centos/centos:8 does not support the s390x architecture, which leads to failures on s390x arch.
https://quay.io/repository/centos/centos?tab=tags