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

Update CAPT and Tink stack components to latest release for consuming L3 provisioning changes #4115

Merged
merged 3 commits into from
Dec 12, 2024

Conversation

rahulbabu95
Copy link
Member

@rahulbabu95 rahulbabu95 commented Dec 10, 2024

Issue #, if available:
https://github.com/aws/eks-anywhere-internal/issues/2792

Description of changes:
This PR bumps CAPT and multiple Tink Services to the latest release.
Below are the list of version bumps:

  • CAPT to v0.6.1
    • Remove patch as the bootiing will be done through workflows and InUse label is directly attached to the hardware object once the workflow is complete from this change onwards
  • Smee to v0.15.1
  • Charts to v0.6.2
    • Remove patch to use smee default values for flags as in the bootstrap cluster, we do not use smee that's part of the chart deployment, but rather deploy smee as a docker container. Also, for the management cluster deployment once this we integrated DHCP relay, we access every tink service using the LB IP and not from the host IP.
    • Additionally, there's an upstream fix so that there's no defaulting needed anymore and is honored as is if the env vars are not set in the deployment.
  • Rufio to v0.6.1
  • Tinkerbell CRDS for Rufio and Tink
    • This project is just used to install the CRDs and do not need the values.yaml as nothing gets templated here. Modify helm_replace.sh script to run the search and replace on values.yaml file only if present to handle cases like this where a values.yaml might not be required.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@eks-distro-bot eks-distro-bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Dec 10, 2024
@eks-distro-bot
Copy link
Collaborator

@rahulbabu95: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

  • /test autoscaler-1-27-presubmit
  • /test autoscaler-1-28-presubmit
  • /test autoscaler-1-29-presubmit
  • /test autoscaler-1-30-presubmit
  • /test autoscaler-1-31-presubmit
  • /test aws-image-builder-presubmit
  • /test aws-otel-collector-tooling-presubmit
  • /test boots-tooling-presubmit
  • /test bottlerocket-bootstrap-tooling-presubmit
  • /test cert-manager-tooling-presubmit
  • /test cilium-tooling-presubmit
  • /test cloud-provider-aws-1-27-tooling-presubmit
  • /test cloud-provider-aws-1-28-tooling-presubmit
  • /test cloud-provider-aws-1-29-tooling-presubmit
  • /test cloud-provider-aws-1-30-tooling-presubmit
  • /test cloud-provider-aws-1-31-tooling-presubmit
  • /test cloud-provider-nutanix-tooling-presubmit
  • /test cloud-provider-vsphere-1-27-tooling-presubmit
  • /test cloud-provider-vsphere-1-28-tooling-presubmit
  • /test cloud-provider-vsphere-1-29-tooling-presubmit
  • /test cloud-provider-vsphere-1-30-tooling-presubmit
  • /test cloud-provider-vsphere-1-31-tooling-presubmit
  • /test cloudstack-cloudmonkey-tooling-presubmit
  • /test cluster-api-provider-aws-snow-tooling-presubmit
  • /test cluster-api-provider-cloudstack-tooling-presubmit
  • /test cluster-api-provider-nutanix-tooling-presubmit
  • /test cluster-api-provider-tinkerbell-tooling-presubmit
  • /test cluster-api-provider-vsphere-tooling-presubmit
  • /test cluster-api-tooling-presubmit
  • /test containerd-tooling-presubmit-amd64
  • /test containerd-tooling-presubmit-arm64
  • /test cri-tools-tooling-presubmit
  • /test dhcp-presubmit-amd64
  • /test dhcp-presubmit-arm64
  • /test distribution-tooling-presubmit
  • /test eks-a-admin-image-presubmit
  • /test eks-a-upgrader-image-presubmit
  • /test eks-anywhere-cli-tools-tooling-presubmit
  • /test eks-anywhere-diagnostic-collector-tooling-presubmit
  • /test eks-anywhere-packages-image-tooling-presubmit
  • /test emissary-tooling-presubmit
  • /test envoy-presubmit
  • /test etcdadm-bootstrap-provider-tooling-presubmit
  • /test etcdadm-controller-tooling-presubmit
  • /test etcdadm-tooling-presubmit
  • /test flux-tooling-presubmit
  • /test govmomi-tooling-presubmit
  • /test harbor-scanner-trivy-tooling-presubmit
  • /test harbor-tooling-presubmit
  • /test hegel-tooling-presubmit
  • /test hello-eks-anywhere-tooling-presubmit
  • /test helm-controller-tooling-presubmit
  • /test helm-tooling-presubmit
  • /test hook-tooling-presubmit-amd64
  • /test hook-tooling-presubmit-arm64
  • /test hub-tooling-presubmit
  • /test imagebuilder-1-27-presubmit
  • /test imagebuilder-1-28-presubmit
  • /test imagebuilder-1-29-presubmit
  • /test imagebuilder-1-30-presubmit
  • /test imagebuilder-1-31-presubmit
  • /test ipxedust-presubmit-amd64
  • /test ipxedust-presubmit-arm64
  • /test kind-1-27-tooling-presubmit
  • /test kind-1-28-tooling-presubmit
  • /test kind-1-29-tooling-presubmit
  • /test kind-1-30-tooling-presubmit
  • /test kind-1-31-tooling-presubmit
  • /test kube-rbac-proxy-tooling-presubmit
  • /test kube-vip-tooling-presubmit
  • /test kustomize-controller-tooling-presubmit
  • /test linux-bootconfig-presubmit-amd64
  • /test linux-bootconfig-presubmit-arm64
  • /test linuxkit-tooling-presubmit-amd64
  • /test linuxkit-tooling-presubmit-arm64
  • /test local-path-provisioner-tooling-presubmit
  • /test metallb-tooling-presubmit
  • /test metrics-server-presubmit
  • /test notification-controller-tooling-presubmit
  • /test prometheus-node-exporter-tooling-presubmit
  • /test prometheus-prometheus-tooling-presubmit
  • /test redis-tooling-presubmit
  • /test rolesanywhere-credential-helper-presubmit
  • /test rufio-tooling-presubmit
  • /test runc-tooling-presubmit-amd64
  • /test runc-tooling-presubmit-arm64
  • /test source-controller-tooling-presubmit
  • /test tink-tooling-presubmit
  • /test tinkerbell-chart-presubmit
  • /test tinkerbell-crds-presubmit
  • /test tinkerbell-stack-presubmit
  • /test trivy-tooling-presubmit
  • /test troubleshoot-tooling-presubmit
  • /test validate-generated-presubmit

Use /test all to run the following jobs that were automatically triggered:

  • autoscaler-1-27-presubmit
  • autoscaler-1-28-presubmit
  • autoscaler-1-29-presubmit
  • autoscaler-1-30-presubmit
  • autoscaler-1-31-presubmit
  • aws-image-builder-presubmit
  • aws-otel-collector-tooling-presubmit
  • boots-tooling-presubmit
  • bottlerocket-bootstrap-tooling-presubmit
  • cert-manager-tooling-presubmit
  • cloud-provider-aws-1-27-tooling-presubmit
  • cloud-provider-aws-1-28-tooling-presubmit
  • cloud-provider-aws-1-29-tooling-presubmit
  • cloud-provider-aws-1-30-tooling-presubmit
  • cloud-provider-aws-1-31-tooling-presubmit
  • cloud-provider-nutanix-tooling-presubmit
  • cloud-provider-vsphere-1-27-tooling-presubmit
  • cloud-provider-vsphere-1-28-tooling-presubmit
  • cloud-provider-vsphere-1-29-tooling-presubmit
  • cloud-provider-vsphere-1-30-tooling-presubmit
  • cloud-provider-vsphere-1-31-tooling-presubmit
  • cloudstack-cloudmonkey-tooling-presubmit
  • cluster-api-provider-aws-snow-tooling-presubmit
  • cluster-api-provider-cloudstack-tooling-presubmit
  • cluster-api-provider-nutanix-tooling-presubmit
  • cluster-api-provider-tinkerbell-tooling-presubmit
  • cluster-api-provider-vsphere-tooling-presubmit
  • cluster-api-tooling-presubmit
  • containerd-tooling-presubmit-amd64
  • containerd-tooling-presubmit-arm64
  • cri-tools-tooling-presubmit
  • dhcp-presubmit-amd64
  • dhcp-presubmit-arm64
  • distribution-tooling-presubmit
  • eks-a-admin-image-presubmit
  • eks-a-upgrader-image-presubmit
  • eks-anywhere-cli-tools-tooling-presubmit
  • eks-anywhere-diagnostic-collector-tooling-presubmit
  • eks-anywhere-packages-image-tooling-presubmit
  • emissary-tooling-presubmit
  • envoy-presubmit
  • etcdadm-bootstrap-provider-tooling-presubmit
  • etcdadm-controller-tooling-presubmit
  • etcdadm-tooling-presubmit
  • flux-tooling-presubmit
  • govmomi-tooling-presubmit
  • harbor-scanner-trivy-tooling-presubmit
  • harbor-tooling-presubmit
  • hegel-tooling-presubmit
  • hello-eks-anywhere-tooling-presubmit
  • helm-controller-tooling-presubmit
  • helm-tooling-presubmit
  • hook-tooling-presubmit-amd64
  • hook-tooling-presubmit-arm64
  • hub-tooling-presubmit
  • imagebuilder-1-27-presubmit
  • imagebuilder-1-28-presubmit
  • imagebuilder-1-29-presubmit
  • imagebuilder-1-30-presubmit
  • imagebuilder-1-31-presubmit
  • ipxedust-presubmit-amd64
  • ipxedust-presubmit-arm64
  • kind-1-27-tooling-presubmit
  • kind-1-28-tooling-presubmit
  • kind-1-29-tooling-presubmit
  • kind-1-30-tooling-presubmit
  • kind-1-31-tooling-presubmit
  • kube-rbac-proxy-tooling-presubmit
  • kube-vip-tooling-presubmit
  • kustomize-controller-tooling-presubmit
  • linux-bootconfig-presubmit-amd64
  • linux-bootconfig-presubmit-arm64
  • linuxkit-tooling-presubmit-amd64
  • linuxkit-tooling-presubmit-arm64
  • local-path-provisioner-tooling-presubmit
  • metallb-tooling-presubmit
  • metrics-server-presubmit
  • notification-controller-tooling-presubmit
  • prometheus-node-exporter-tooling-presubmit
  • prometheus-prometheus-tooling-presubmit
  • redis-tooling-presubmit
  • rolesanywhere-credential-helper-presubmit
  • rufio-tooling-presubmit
  • runc-tooling-presubmit-amd64
  • runc-tooling-presubmit-arm64
  • source-controller-tooling-presubmit
  • tink-tooling-presubmit
  • tinkerbell-chart-presubmit
  • tinkerbell-crds-presubmit
  • tinkerbell-stack-presubmit
  • trivy-tooling-presubmit
  • troubleshoot-tooling-presubmit
  • validate-generated-presubmit

In response to this:

/retest autoscaler-1-27-presubmit autoscaler-1-28-presubmit autoscaler-1-29-presubmit autoscaler-1-30-presubmit autoscaler-1-31-presubmit aws-image-builder-presubmit aws-otel-collector-tooling-presubmit boots-tooling-presubmit

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@rahulbabu95
Copy link
Member Author

/test bottlerocket-bootstrap-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test cert-manager-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test cloud-provider-aws-1-27-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test troubleshoot-tooling-presubmit trivy-tooling-presubmit tinkerbell-stack-presubmit tinkerbell-crds-presubmit tinkerbell-chart-presubmit tink-tooling-presubmit source-controller-tooling-presubmit runc-tooling-presubmit-amd64 rufio-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test rolesanywhere-credential-helper-presubmit redis-tooling-presubmit prometheus-prometheus-tooling-presubmit prometheus-node-exporter-tooling-presubmit notification-controller-tooling-presubmit metrics-server-presubmit metallb-tooling-presubmit local-path-provisioner-tooling-presubmit

@rahulbabu95
Copy link
Member Author

rahulbabu95 commented Dec 11, 2024

/test linuxkit-tooling-presubmit-amd64 kustomize-controller-tooling-presubmit kube-vip-tooling-presubmit kube-rbac-proxy-tooling-presubmit kind-1-31-tooling-presubmit kind-1-30-tooling-presubmit kind-1-29-tooling-presubmit kind-1-28-tooling-presubmit kind-1-27-tooling-presubmit hub-tooling-presubmit hook-tooling-presubmit-amd64 helm-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test kube-vip-tooling-presubmit kube-rbac-proxy-tooling-presubmit kind-1-31-tooling-presubmit kind-1-30-tooling-presubmit kind-1-29-tooling-presubmit kind-1-28-tooling-presubmit kind-1-27-tooling-presubmit hub-tooling-presubmit hook-tooling-presubmit-amd64 helm-tooling-presubmit helm-controller-tooling-presubmit hello-eks-anywhere-tooling-presubmit hegel-tooling-presubmit harbor-tooling-presubmit harbor-scanner-trivy-tooling-presubmit govmomi-tooling-presubmit flux-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test etcdadm-tooling-presubmit etcdadm-controller-tooling-presubmit etcdadm-bootstrap-provider-tooling-presubmit envoy-presubmit emissary-tooling-presubmit eks-anywhere-packages-image-tooling-presubmit eks-anywhere-diagnostic-collector-tooling-presubmit eks-anywhere-cli-tools-tooling-presubmit eks-a-upgrader-image-presubmit distribution-tooling-presubmit cri-tools-tooling-presubmit containerd-tooling-presubmit-amd64 cluster-api-tooling-presubmit

@rahulbabu95
Copy link
Member Author

/test eks-a-admin-image-presubmit cluster-api-provider-vsphere-tooling-presubmit cluster-api-provider-tinkerbell-tooling-presubmit cluster-api-provider-nutanix-tooling-presubmit cluster-api-provider-cloudstack-tooling-presubmit cluster-api-provider-aws-snow-tooling-presubmit cloudstack-cloudmonkey-tooling-presubmit cloud-provider-vsphere-1-31-tooling-presubmit cloud-provider-vsphere-1-30-tooling-presubmit cloud-provider-vsphere-1-29-tooling-presubmit cloud-provider-vsphere-1-28-tooling-presubmit cloud-provider-vsphere-1-27-tooling-presubmit cloud-provider-nutanix-tooling-presubmit cloud-provider-aws-1-31-tooling-presubmit cloud-provider-aws-1-30-tooling-presubmit cloud-provider-aws-1-29-tooling-presubmit cloud-provider-aws-1-28-tooling-presubmit

Copy link
Member

@jacobweinstock jacobweinstock left a comment

Choose a reason for hiding this comment

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

The CRDs need to have some CAPI annotations i believe. They also previously had some helm annotations that are removed in this. I suggest checking with @ahreehong on this.

@rahulbabu95
Copy link
Member Author

/approve

@eks-distro-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rahulbabu95

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@eks-distro-bot eks-distro-bot merged commit 7c04258 into aws:main Dec 12, 2024
94 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved lgtm size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants