Skip to content

[OSDOCS-20065] Autonode preview - #116476

Open
EricPonvelle wants to merge 1 commit into
openshift:mainfrom
EricPonvelle:Autonode_Preview
Open

[OSDOCS-20065] Autonode preview#116476
EricPonvelle wants to merge 1 commit into
openshift:mainfrom
EricPonvelle:Autonode_Preview

Conversation

@EricPonvelle

@EricPonvelle EricPonvelle commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Version(s):
enterprise-4.22+

Issue:
https://redhat.atlassian.net/browse/OSDOCS-20065

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:
This PR combines #115023, #115388, and #115441 to serve as a complete preview of the Autonode Fast Follow Updates.

@EricPonvelle EricPonvelle added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. branch/enterprise-4.22 branch/enterprise-5.0 labels Jul 24, 2026
@openshift-ci openshift-ci Bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Jul 24, 2026
@ocpdocs-previewbot

ocpdocs-previewbot commented Jul 24, 2026

Copy link
Copy Markdown

🤖 Tue Jul 28 18:04:01 - Prow CI generated the docs preview:
https://116476--ocpdocs-pr.netlify.app
Complete list of updated preview URLs: artifacts/updated_preview_urls.txt

Comment thread modules/rosa-nodes-karpenter-reference.adoc
Comment thread rosa_cluster_admin/autonode/rosa-autonode-api-reference.adoc Outdated
Comment thread rosa_cluster_admin/autonode/rosa-autonode-setup.adoc

@bchandra-ocp bchandra-ocp left a comment

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.

(1/n) review. Working on the rest.

Comment thread rosa_cluster_admin/autonode/rosa-autonode-setup.adoc Outdated
Comment thread modules/rosa-nodes-autonode-managing-iam-setup.adoc Outdated
Comment thread modules/rosa-nodes-autonode-managing-iam-setup.adoc Outdated
@@ -93,11 +84,11 @@ $ aws iam attach-role-policy \
--role-name rosa-karpenter-controller-role-${CLUSTER_NAME} \
--policy-arn $POLICY_ARN

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.

Instead of $POLICY_ARN, let's just use arn:aws:iam::aws:policy/service-role/ROSAKarpenterControllerPolicy which is the ARN for the AWS IAM Managed Policy. So the complete command will look as follows:

aws iam attach-role-policy \
--role-name rosa-autonode-operator-role \
--policy-arn arn:aws:iam::aws:policy/service-role/ROSAKarpenterControllerPolicy

Comment thread modules/rosa-nodes-autonode-managing-tagging-setup.adoc
Comment thread rosa_cluster_admin/autonode/rosa-autonode-setup.adoc
--role-name rosa-karpenter-controller-role-${CLUSTER_NAME} \
--policy-arn $POLICY_ARN
--role-name rosa-autonode-operator-role \
--policy-arn arn:aws:iam::aws:policy/service-role/ROSAKarpenterControllerPolicy

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@bchandra-ocp When I used the suggested command, I got:

aws iam attach-role-policy --role-name rosa-autonode-operator-role --policy-arn arn:aws:iam::aws:policy/service-role/ROSAKarpenterControllerPolicy

aws: [ERROR]: An error occurred (NoSuchEntity) when calling the AttachRolePolicy operation: The role with name rosa-autonode-operator-role cannot be found.

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.

@EricPonvelle You need to use the role you have used previously in the documentation: rosa-karpenter-controller-role-${CLUSTER_NAME}

"rosa-autonode-operator-role" is MY role. AWS only has managed policies.

* link:https://karpenter.sh/docs/[Karpenter project documentation]
* xref:../../upgrading/rosa-hcp-upgrading.adoc#rosa-upgrade-options_rosa-hcp-upgrading[Update options for {product-title} clusters configured with {autonode}]
* xref:../../support/troubleshooting/rosa-troubleshooting-autonode.adoc#rosa-troubleshooting-autonode[Troubleshoot {autonode} configuration and operational issues]
* link:https://github.com/openshift/hypershift/tree/main/api[HyperShift API Definitions Repository]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🤖 [error] RedHat.TermsErrors: Use 'hosted control planes' rather than 'HyperShift'. For more information, see RedHat.TermsErrors.

@openshift-ci

openshift-ci Bot commented Jul 28, 2026

Copy link
Copy Markdown

@EricPonvelle: all tests passed!

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.

@jc-berger jc-berger left a comment

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.

/lgtm

- key: node.kubernetes.io/instance-type
operator: In
values:
- c5.xlarge

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.

Since this spec also includes the "key" value, I believe you want to include your missing content here. So the spec for yaml would be:

spec:
  template:
    spec:
      requirements:
      - key: node.kubernetes.io/instance-type
        operator: In
        values:
        - c5.xlarge
      - key: karpenter.k8s.aws/instance-cpu
        operator: Gte
        values: ["4"]
      - key: karpenter.sh/capacity-type
        operator: In
        values: ["on-demand"]

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.22 branch/enterprise-5.0 lgtm Indicates that a PR is ready to be merged. 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.

5 participants