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

ci: [release/v0.14.x] images tool #926

Open
wants to merge 1 commit into
base: release/v0.14.x
Choose a base branch
from

Conversation

dlipovetsky
Copy link
Contributor

What problem does this PR solve?:
Backport of #822

Which issue(s) this PR fixes:
Fixes #

How Has This Been Tested?:

Special notes for your reviewer:

**What problem does this PR solve?**:
Adds a tool to fetch all images CAREN uses

**Which issue(s) this PR fixes**:
Fixes #

**How Has This Been Tested?**:
<!--
Please describe the tests that you ran to verify your changes.
Provide output from the tests and any manual steps needed to replicate
the tests.
-->
I ran the following command from `hack/tools/fetch-images`

```
$ go run main.go -chart-directory=../../../charts/cluster-api-runtime-extensions-nutanix/ --helm-chart-configmap=../../../charts/cluster-api-runtime-extensions-nutanix/templates/helm-config.yaml -caren-version=v0.13.1
ghcr.io/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix:v0.13.1
ghcr.io/nutanix-cloud-native/caren-helm-reg:v0.13.1
registry.k8s.io/sig-storage/snapshot-controller:v8.0.1
registry.k8s.io/provider-aws/cloud-controller-manager:v1.27.7
registry.k8s.io/provider-aws/cloud-controller-manager:v1.28.6
registry.k8s.io/provider-aws/cloud-controller-manager:v1.29.3
registry.k8s.io/provider-aws/cloud-controller-manager:v1.30.1
public.ecr.aws/ebs-csi-driver/aws-ebs-csi-driver:v1.33.0
public.ecr.aws/eks-distro/kubernetes-csi/node-driver-registrar:v2.11.0-eks-1-30-10
public.ecr.aws/eks-distro/kubernetes-csi/livenessprobe:v2.13.0-eks-1-30-10
public.ecr.aws/ebs-csi-driver/aws-ebs-csi-driver:v1.33.0
public.ecr.aws/eks-distro/kubernetes-csi/external-provisioner:v5.0.1-eks-1-30-10
public.ecr.aws/eks-distro/kubernetes-csi/external-attacher:v4.6.1-eks-1-30-10
public.ecr.aws/eks-distro/kubernetes-csi/external-resizer:v1.11.1-eks-1-30-10
public.ecr.aws/eks-distro/kubernetes-csi/livenessprobe:v2.13.0-eks-1-30-10
registry.k8s.io/autoscaling/cluster-autoscaler:v1.30.0
quay.io/metallb/speaker:v0.14.8
quay.io/frrouting/frr:9.1.0
quay.io/metallb/speaker:v0.14.8
quay.io/metallb/controller:v0.14.8
registry.k8s.io/nfd/node-feature-discovery:v0.16.1-minimal
registry.k8s.io/sig-storage/csi-node-driver-registrar:v2.9.3
docker.io/nutanix/ntnx-csi:3.0.0
registry.k8s.io/sig-storage/livenessprobe:v2.11.0
registry.k8s.io/sig-storage/csi-provisioner:v3.6.3
registry.k8s.io/sig-storage/csi-attacher:v4.4.3
registry.k8s.io/sig-storage/csi-resizer:v1.9.3
registry.k8s.io/sig-storage/csi-snapshotter:v3.0.3
docker.io/nutanix/ntnx-csi:3.0.0
registry.k8s.io/sig-storage/livenessprobe:v2.11.0
registry.k8s.io/sig-storage/csi-external-health-monitor-controller:v0.10.0
docker.io/nutanix/ntnx-csi-precheck:3.0.0
quay.io/cilium/cilium:v1.16.0
quay.io/cilium/cilium-envoy@sha256:bd5ff8c66716080028f414ec1cb4f7dc66f40d2fb5a009fff187f4a9b90b566b
quay.io/cilium/operator-generic:v1.16.0
quay.io/cilium/certgen:v0.2.0
docker.io/rancher/local-path-provisioner:v0.0.28
ghcr.io/nutanix-cloud-native/cloud-provider-nutanix/controller:v0.3.2
quay.io/tigera/operator:v1.34.0
```


**Special notes for your reviewer**:
<!--
Use this to provide any additional information to the reviewers.
This may include:
- Best way to review the PR.
- Where the author wants the most review attention on.
- etc.
-->

Depends on #896.
@dlipovetsky dlipovetsky changed the title ci: images tool (#822) [release/v0.14.x] ci: images tool (#822) Sep 27, 2024
@jimmidyson jimmidyson changed the title [release/v0.14.x] ci: images tool (#822) ci: [release/v0.14.x] images tool Sep 28, 2024
@jimmidyson
Copy link
Member

Updated pr title for conventional commits.

But more importantly wondering if we need this change, how is it used outside of CAREN at the moment, and if we do need it are there other changes that are also required for this backport (eg #896)?

@jimmidyson
Copy link
Member

@dlipovetsky Do we still need this?

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