Skip to content

Add release notes for the Insights Operator for 4.18#88584

Merged
jeana-redhat merged 1 commit intoopenshift:enterprise-4.18from
michelle-purcell:InsightsOperatorRNHighlights
Feb 20, 2025
Merged

Add release notes for the Insights Operator for 4.18#88584
jeana-redhat merged 1 commit intoopenshift:enterprise-4.18from
michelle-purcell:InsightsOperatorRNHighlights

Conversation

@michelle-purcell
Copy link
Contributor

@michelle-purcell michelle-purcell commented Feb 13, 2025

This PR adds more enhancements that are in OCP 4.18 for the Insights Operator, including Rapid Recommendations and the capability for Red Hat to collect more data, should the customer choose. Plus a fixed bug that customers would be interested to know. about.

Version(s):

4.18 only

Issue:

HCCDOC-3441

Link to docs preview:

https://88584--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-18-release-notes.html

QE review:

  • QE has approved this change.

Additional information:

@michelle-purcell michelle-purcell marked this pull request as draft February 13, 2025 18:27
@openshift-ci openshift-ci bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 13, 2025
@michelle-purcell michelle-purcell marked this pull request as ready for review February 13, 2025 18:33
@openshift-ci openshift-ci bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 13, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Feb 13, 2025

🤖 Thu Feb 20 19:35:34 - Prow CI generated the docs preview:

https://88584--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-18-release-notes.html

@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch from 1c8e55d to 587e38f Compare February 18, 2025 08:18
@michelle-purcell
Copy link
Contributor Author

/label Insights Operator

@michelle-purcell
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 Feb 18, 2025
@cbippley
Copy link
Contributor

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

@openshift-ci openshift-ci bot 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 Feb 18, 2025
Copy link
Contributor

@cbippley cbippley left a comment

Choose a reason for hiding this comment

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

LGTM!

@cbippley
Copy link
Contributor

/label peer-review-done
/remove-label peer-review-in-progress

@openshift-ci openshift-ci bot 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 Feb 18, 2025
@michelle-purcell
Copy link
Contributor Author

Thank you, @cbippley 💚

@michelle-purcell
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 Feb 19, 2025
@JoeAldinger JoeAldinger added merge-review-in-progress Signifies that the merge review team is reviewing this PR and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Feb 19, 2025
@JoeAldinger JoeAldinger self-assigned this Feb 19, 2025
Copy link
Contributor

@JoeAldinger JoeAldinger left a comment

Choose a reason for hiding this comment

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

Some things to consider and fix before merge.

@JoeAldinger JoeAldinger removed the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Feb 19, 2025
@openshift-ci openshift-ci bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 20, 2025
@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch from 2577154 to e48b101 Compare February 20, 2025 13:44
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 20, 2025
@openshift-ci openshift-ci bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 20, 2025
@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch from c6a74b8 to 1307a84 Compare February 20, 2025 13:49
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 20, 2025
@michelle-purcell
Copy link
Contributor Author

Thanks so much, @JoeAldinger for a thorough review. Great suggestions. I fixed those attributes. I've deferred a couple for wider discussion for consistency across the other Insights doc deliverables. 💚
I think we are ready to merge now.

@michelle-purcell
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 Feb 20, 2025
@jeana-redhat jeana-redhat added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Feb 20, 2025
@jeana-redhat jeana-redhat added this to the Planned for 4.18 GA milestone Feb 20, 2025
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.

two li'l nits - otherwise LGTM

If you don't want to drop it back in the queue feel free to DM me :)

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

|Supported
|====

[id="ocp-release-notes-insights-operator-enhancements_{context}"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like this anchor was deleted inadvertently? Should add this back before merge

@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 Feb 20, 2025
@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch 2 times, most recently from 9eb3029 to 580b1bf Compare February 20, 2025 18:32
@michelle-purcell
Copy link
Contributor Author

Thanks @jeana-redhat! Great catch. Both items are now fixed. 💚

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.

Whenever the builds finish this is
/label ok-to-merge

@openshift-ci
Copy link

openshift-ci bot commented Feb 20, 2025

@jeana-redhat: The label(s) /label ok-to-merge cannot be applied. These labels are supported: acknowledge-critical-fixes-only, platform/aws, platform/azure, platform/baremetal, platform/google, platform/libvirt, platform/openstack, ga, tide/merge-method-merge, tide/merge-method-rebase, tide/merge-method-squash, px-approved, docs-approved, qe-approved, no-qe, downstream-change-needed, rebase/manual, cluster-config-api-changed, approved, backport-risk-assessed, bugzilla/valid-bug, cherry-pick-approved, cloud-experts, cnv, dev-tools, distributed-tracing, ims, jira/valid-bug, merge-review-in-progress, merge-review-needed, mtc, multi-arch, oadp, peer-review-done, peer-review-in-progress, peer-review-needed, rhacs, rhv, sd-docs, serverless, service-mesh, sme-review-done, sme-review-needed, staff-eng-approved, telco. Is this label configured under labels -> additional_labels or labels -> restricted_labels in plugin.yaml?

Details

In response to this:

Whenever the builds finish this is
/label ok-to-merge

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.

@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch from 580b1bf to 6b9bb5c Compare February 20, 2025 19:15
@openshift-ci openshift-ci bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 20, 2025
…4.18

Update release_notes/ocp-4-18-release-notes.adoc

Co-authored-by: Jeana Routh <61474374+jeana-redhat@users.noreply.github.com>

merge fix

fix escape
@michelle-purcell michelle-purcell force-pushed the InsightsOperatorRNHighlights branch from 6b9bb5c to f27e4fa Compare February 20, 2025 19:16
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Feb 20, 2025
@openshift-ci
Copy link

openshift-ci bot commented Feb 20, 2025

@michelle-purcell: 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.

@jeana-redhat
Copy link
Contributor

Yay! finally behaving 🎉

@jeana-redhat jeana-redhat merged commit dc2ab43 into openshift:enterprise-4.18 Feb 20, 2025
2 checks passed
@michelle-purcell
Copy link
Contributor Author

Thank you @jeana-redhat.

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

Labels

branch/enterprise-4.18 ok-to-merge peer-review-done Signifies that the peer review team has reviewed this PR size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants