Skip to content

Conversation

jeana-redhat
Copy link
Contributor

@jeana-redhat jeana-redhat commented Sep 18, 2025

Version(s):
4.20+

Issue:
OSDOCS-16222

Link to docs preview:
Capacity Reservation configuration options

QE review:

  • QE has approved this change.

Additional information:

@jeana-redhat jeana-redhat added this to the Planned for 4.20 GA milestone Sep 18, 2025
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Sep 18, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 18, 2025

@jeana-redhat: This pull request references OSDOCS-16222 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target either version "4.21." or "openshift-4.21.", but it targets "openshift-4.20" instead.

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16222

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

1 similar comment
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 18, 2025

@jeana-redhat: This pull request references OSDOCS-16222 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target either version "4.21." or "openshift-4.21.", but it targets "openshift-4.20" instead.

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16222

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 18, 2025
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 18, 2025

@jeana-redhat: This pull request references OSDOCS-16222 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target either version "4.21." or "openshift-4.21.", but it targets "openshift-4.20" instead.

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16222

Link to docs preview:
Capacity Reservation configuration options

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@jeana-redhat
Copy link
Contributor Author

@BraeTroutman and @huali9 PTAL 🙇

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Sep 18, 2025

@BraeTroutman
Copy link
Contributor

@jeana-redhat same points as Huali from my end re: using PascalCase rather than kebab case for the preference enum values. Otherwise looks great to me!

@jeana-redhat jeana-redhat force-pushed the OSDOCS-16222-capi-aws-CapacityReservationPreference branch from b72987f to 5aafddc Compare September 22, 2025 14:15
@openshift-ci-robot
Copy link

openshift-ci-robot commented Sep 22, 2025

@jeana-redhat: This pull request references OSDOCS-16222 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target either version "4.21." or "openshift-4.21.", but it targets "openshift-4.20" instead.

In response to this:

Version(s):
4.20+

Issue:
OSDOCS-16222

Link to docs preview:
Capacity Reservation configuration options

QE review:

  • QE has approved this change.

Additional information:

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 openshift-eng/jira-lifecycle-plugin repository.

@jeana-redhat jeana-redhat added the merge-review-needed Signifies that the merge review team needs to review this PR label Sep 22, 2025
Copy link

openshift-ci bot commented Sep 22, 2025

@jeana-redhat: all tests passed!

Full PR test history. Your PR dashboard.

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.

@huali9
Copy link

huali9 commented Sep 23, 2025

Thank you @jeana-redhat
/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 23, 2025
@bergerhoffer
Copy link
Contributor

/label merge-review-in-progress

@openshift-ci openshift-ci bot added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Sep 23, 2025
Copy link
Contributor

@bergerhoffer bergerhoffer left a comment

Choose a reason for hiding this comment

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

Merge review LGTM!

@bergerhoffer bergerhoffer added ok-to-merge and removed merge-review-in-progress Signifies that the merge review team is reviewing this PR merge-review-needed Signifies that the merge review team needs to review this PR labels Sep 23, 2025
@jeana-redhat jeana-redhat merged commit bc570a8 into openshift:main Sep 23, 2025
2 checks passed
@jeana-redhat
Copy link
Contributor Author

/cherrypick enterprise-4.20

@openshift-cherrypick-robot

@jeana-redhat: new pull request created: #99576

In response to this:

/cherrypick enterprise-4.20

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.20 jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. ok-to-merge size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants