Skip to content

fix: remove undefined EKS function calls in helm/operator jobs#4398

Merged
openshift-merge-bot[bot] merged 1 commit intoredhat-developer:release-1.9from
gustavolira:fix/remove-undefined-eks-functions-1.9
Mar 12, 2026
Merged

fix: remove undefined EKS function calls in helm/operator jobs#4398
openshift-merge-bot[bot] merged 1 commit intoredhat-developer:release-1.9from
gustavolira:fix/remove-undefined-eks-functions-1.9

Conversation

@gustavolira
Copy link
Copy Markdown
Member

Summary

  • Remove calls to aws_eks_verify_cluster and aws_eks_get_cluster_info in eks-helm.sh and eks-operator.sh
  • These functions were removed but their call sites were not updated, causing command not found errors in the eks-helm-nightly pipeline
  • Backport of the fix from main (PR chore(ci): use hardcoded container platform vars #3610) to release-1.9

Error

[2026-03-12T12:44:08Z] ℹ Starting EKS Helm deployment
/tmp/rhdh/.ci/pipelines/jobs/eks-helm.sh: line 20: aws_eks_verify_cluster: command not found

Test plan

  • Verify eks-helm-nightly pipeline passes without command not found errors
  • Verify eks-operator-nightly pipeline passes without command not found errors

🤖 Generated with Claude Code

…info calls

These functions were removed but their call sites in eks-helm.sh and
eks-operator.sh were not updated, causing "command not found" errors
in the eks-helm-nightly pipeline.

Backport of the fix from main (PR redhat-developer#3610) to release-1.9.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-ci openshift-ci Bot requested review from kadel and teknaS47 March 12, 2026 14:16
@sonarqubecloud
Copy link
Copy Markdown

@gustavolira
Copy link
Copy Markdown
Member Author

/test e2e-eks-helm-nightly

@gustavolira
Copy link
Copy Markdown
Member Author

/test ?

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Mar 12, 2026

@gustavolira: The following commands are available to trigger required jobs:

/test e2e-ocp-helm

The following commands are available to trigger optional jobs:

/test e2e-aks-helm-nightly
/test e2e-aks-operator-nightly
/test e2e-eks-helm-nightly
/test e2e-eks-operator-nightly
/test e2e-gke-helm-nightly
/test e2e-gke-operator-nightly
/test e2e-ocp-helm-nightly
/test e2e-ocp-helm-upgrade-nightly
/test e2e-ocp-operator-auth-providers-nightly
/test e2e-ocp-operator-nightly
/test e2e-ocp-v4-17-helm-nightly
/test e2e-ocp-v4-19-helm-nightly
/test e2e-ocp-v4-20-helm-nightly
/test e2e-ocp-v4-21-helm-nightly
/test e2e-osd-gcp-helm-nightly
/test e2e-osd-gcp-operator-nightly

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

pull-ci-redhat-developer-rhdh-release-1.9-e2e-ocp-helm
Details

In response to this:

/test ?

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-sigs/prow repository.

@github-actions
Copy link
Copy Markdown
Contributor

Image was built and published successfully. It is available at:

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Mar 12, 2026

@gustavolira: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-eks-helm-nightly d341480 link false /test e2e-eks-helm-nightly

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

@openshift-ci openshift-ci Bot added the lgtm label Mar 12, 2026
@openshift-merge-bot openshift-merge-bot Bot merged commit f6dee0f into redhat-developer:release-1.9 Mar 12, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants