Skip to content

Conversation

rohennes
Copy link
Contributor

@rohennes rohennes commented Sep 24, 2024

OCPBUGS-31318: Fixing xref

Version(s):
4.13

Issue:
https://issues.redhat.com/browse/OCPBUGS-31318

Link to docs preview:
https://82326--ocpdocs-pr.netlify.app/openshift-enterprise/latest/scalability_and_performance/cnf-low-latency-tuning.html#node-tuning-operator-assigning-proper-node-selector_cnf-master

Additional information:
Typo, no QE.

This only effects 4.13 and 4.12. This module does not exist in later versions. So I am making the fix directly in the 4.13 branch. Is this ok? And can it be backported to 4.12 or do I need a separate PR? Thanks!

@openshift-ci-robot openshift-ci-robot added jira/severity-low Referenced Jira bug's severity is low for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. labels Sep 24, 2024
@openshift-ci-robot
Copy link

@rohennes: This pull request references Jira Issue OCPBUGS-31318, which is invalid:

  • release note text must be set and not match the template OR release note type must be set to "Release Note Not Required". For more information you can reference the OpenShift Bug Process.
  • expected Jira Issue OCPBUGS-31318 to depend on a bug in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), CLOSED (DONE-ERRATA), but no dependents were found

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

In response to this:

OCPBUGS-31318: Fixing xref

Version(s):
4.13

Issue:
https://issues.redhat.com/browse/OCPBUGS-31318

Link to docs preview:

Additional information:
Typo, no QE.

This only effects 4.13 and 4.12. This module does not exist in later versions. So I am making the fix in 4.13, can it be backported to 4.12 or do I need a separate PR?

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/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 24, 2024
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Sep 24, 2024

@openshift-ci-robot
Copy link

@rohennes: This pull request references Jira Issue OCPBUGS-31318, which is invalid:

  • expected the bug to be in one of the following states: NEW, ASSIGNED, POST, but it is MODIFIED instead
  • release note text must be set and not match the template OR release note type must be set to "Release Note Not Required". For more information you can reference the OpenShift Bug Process.
  • expected Jira Issue OCPBUGS-31318 to depend on a bug in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), CLOSED (DONE-ERRATA), but no dependents were found

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

In response to this:

OCPBUGS-31318: Fixing xref

Version(s):
4.13

Issue:
https://issues.redhat.com/browse/OCPBUGS-31318

Link to docs preview:
https://82326--ocpdocs-pr.netlify.app/openshift-enterprise/latest/scalability_and_performance/cnf-low-latency-tuning.html#node-tuning-operator-assigning-proper-node-selector_cnf-master

Additional information:
Typo, no QE.

This only effects 4.13 and 4.12. This module does not exist in later versions. So I am making the fix in 4.13, can it be backported to 4.12 or do I need a separate PR?

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.

@rohennes
Copy link
Contributor Author

/label peer-review-needed

@openshift-ci openshift-ci bot added the peer-review-needed Signifies that the peer review team needs to review this PR label Sep 24, 2024
@rohennes
Copy link
Contributor Author

FAO: Peer reviewer, please see additional info in description, thanks!

@lpettyjo lpettyjo self-requested a review September 24, 2024 13:27
@lpettyjo lpettyjo added peer-review-in-progress Signifies that the peer review team is reviewing this PR and removed peer-review-needed Signifies that the peer review team needs to review this PR labels Sep 24, 2024
@lpettyjo lpettyjo added this to the Continuous Release milestone Sep 24, 2024
Copy link
Contributor

@lpettyjo lpettyjo left a comment

Choose a reason for hiding this comment

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

LGTM!

@lpettyjo lpettyjo added peer-review-done Signifies that the peer review team has reviewed this PR and removed peer-review-in-progress Signifies that the peer review team is reviewing this PR labels Sep 24, 2024
@rohennes
Copy link
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Sep 24, 2024
@jeana-redhat jeana-redhat added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Sep 24, 2024
Copy link
Contributor

@jeana-redhat jeana-redhat left a comment

Choose a reason for hiding this comment

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

These don't quite conform to guidelines, but the fixes are straightforward :)

/remove-label merge-review-in-progress
/remove-label merge-review-needed

----

For more information, see link:https://access.redhat.com/documentation/en-us/openshift_container_platform/4.5/html-single/nodes/index#nodes-scheduler-node-selectors[Placing pods on specific nodes using node selectors].
For more information, see _Placing pods on specific nodes using node selectors_ in the _Additional resources_ section.
Copy link
Contributor

Choose a reason for hiding this comment

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

Per guidelines

Suggested change
For more information, see _Placing pods on specific nodes using node selectors_ in the _Additional resources_ section.
For more information, see _Placing pods on specific nodes using node selectors_.

Comment on lines 23 to 25
* For more information about recommended firmware configuration, see xref:../scalability_and_performance/ztp_far_edge/ztp-vdu-validating-cluster-tuning.adoc#ztp-du-firmware-config-reference_vdu-config-ref[Recommended firmware configuration for vDU cluster hosts].
* For more information about assigning a node selector, see xref:../nodes/scheduling/nodes-scheduler-node-selectors.adoc#nodes-scheduler-node-selectors[Placing pods on specific nodes using node selectors].
Copy link
Contributor

Choose a reason for hiding this comment

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

Better to fix the existing incorrect format here than propagate it :)

Suggested change
* For more information about recommended firmware configuration, see xref:../scalability_and_performance/ztp_far_edge/ztp-vdu-validating-cluster-tuning.adoc#ztp-du-firmware-config-reference_vdu-config-ref[Recommended firmware configuration for vDU cluster hosts].
* For more information about assigning a node selector, see xref:../nodes/scheduling/nodes-scheduler-node-selectors.adoc#nodes-scheduler-node-selectors[Placing pods on specific nodes using node selectors].
* xref:../scalability_and_performance/ztp_far_edge/ztp-vdu-validating-cluster-tuning.adoc#ztp-du-firmware-config-reference_vdu-config-ref[Recommended firmware configuration for vDU cluster hosts]
* xref:../nodes/scheduling/nodes-scheduler-node-selectors.adoc#nodes-scheduler-node-selectors[Placing pods on specific nodes using node selectors]

@openshift-ci openshift-ci bot 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 24, 2024
@rohennes
Copy link
Contributor Author

@jeana-redhat - thanks as always, I made those changes.

/label merge-review-needed

@openshift-ci openshift-ci bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Sep 25, 2024
Copy link

openshift-ci bot commented Sep 25, 2024

@rohennes: 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.

@skopacz1 skopacz1 added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Sep 25, 2024
Copy link
Contributor

@skopacz1 skopacz1 left a comment

Choose a reason for hiding this comment

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

Just have one question, once that's answered I can either merge as-is or wait for you to update the PR first

@skopacz1 skopacz1 merged commit c605473 into openshift:enterprise-4.13 Sep 25, 2024
2 checks passed
@openshift-ci-robot
Copy link

@rohennes: Jira Issue OCPBUGS-31318 is in an unrecognized state (MODIFIED) and will not be moved to the MODIFIED state.

In response to this:

OCPBUGS-31318: Fixing xref

Version(s):
4.13

Issue:
https://issues.redhat.com/browse/OCPBUGS-31318

Link to docs preview:
https://82326--ocpdocs-pr.netlify.app/openshift-enterprise/latest/scalability_and_performance/cnf-low-latency-tuning.html#node-tuning-operator-assigning-proper-node-selector_cnf-master

Additional information:
Typo, no QE.

This only effects 4.13 and 4.12. This module does not exist in later versions. So I am making the fix directly in the 4.13 branch. Is this ok? And can it be backported to 4.12 or do I need a separate PR? Thanks!

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.

@skopacz1 skopacz1 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 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.13 jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. jira/severity-low Referenced Jira bug's severity is low for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. peer-review-done Signifies that the peer review team has reviewed this PR size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants