Skip to content

Conversation

@michelle-purcell
Copy link
Contributor

@michelle-purcell michelle-purcell commented Oct 9, 2024

This change is required to help customers understand some key enhancements about the Red Hat Insights component in OpenShift Container Platform and to help reduce the churn to technical support and other field teams who might get asked about some of the key changes that were introduced in 4.17.

Several enhancements were made to the Insights Operator component of Red Hat Insights for OpenShift in 4.17.

The biggest highlight was that a new data collection feature was introduced called Rapid Recommendations.

Several other enhancements were made to increase the data that gets collected and to provide more remedial recommendations.

Version(s):

4.17

Issue:

HCCDOC-3084

Link to docs preview:

https://83273--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-17-release-notes.html#ocp-4-17-insights-operator-enhancements_release-notes

QE review:

  • QE has approved this change.

Additional information:

The Red Hat Insights team are working through a backlog of JIRAs, hence the delay in processing this doc requirement.

Initiating the Change Management - Special Case process of the OCP docs project. Thank you.

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Oct 9, 2024
@openshift-ci
Copy link

openshift-ci bot commented Oct 9, 2024

Hi @michelle-purcell. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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 HCCDOC-3084-InsightOperatorReleaseNotes branch 2 times, most recently from 20a4655 to db9a0a6 Compare October 10, 2024 14:20
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 10, 2024
@michelle-purcell michelle-purcell changed the title HCCDOC-3084 Add release note about the new Rapid Recommendations feature HCCDOC-3084 Add missing release notes for Insights Operator enhancements including rapid recommendations Oct 10, 2024
@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from db9a0a6 to 374d34f Compare October 10, 2024 14:27
[id="ocp-4-17-insights-operator-enhancements_{context}"]
=== Insights Operator

* The Insights Operator now collects the `haproxy_exporter_server_threshold` metric. (link:https://issues.redhat.com/browse/OCPBUGS-36687[*OCPBUGS-36687*])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI anyone 👀 - This line item has actually been moved down to line 570.

* The Insights Operator now collects the `haproxy_exporter_server_threshold` metric. (link:https://issues.redhat.com/browse/OCPBUGS-36687[*OCPBUGS-36687*])
// Engineering reference CCXDEV-12582

* Previously, the Insights Operator gathered information about all Ingress Controller certificates, including their `NotBefore` and `NotAfter` dates. This data is now compiled into a `JSON` file located at `aggregated/ingress_controllers_certs.json` for easier monitoring of certificate validity across the cluster. (link:https://issues.redhat.com/browse/OCPBUGS-35727[*OCPBUGS-35727*])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

FYI anyone 👀 - This line item has actually been moved down to line 576.

@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from 374d34f to 1d787b0 Compare October 10, 2024 14:44
@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 Oct 10, 2024
@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from 1d787b0 to 56156a6 Compare October 15, 2024 22:55
@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 Oct 15, 2024
@michelle-purcell
Copy link
Contributor Author

/retest

@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 15, 2024

🤖 Tue Oct 29 15:19:52 - Prow CI generated the docs preview:

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

@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch 2 times, most recently from 10bdd6c to 276649b Compare October 16, 2024 10:43
@michelle-purcell
Copy link
Contributor Author

/retest

@michelle-purcell michelle-purcell changed the title HCCDOC-3084 Add missing release notes for Insights Operator enhancements including rapid recommendations HCCDOC3084 Add missing release notes for Insights Operator enhancements including rapid recommendations Oct 16, 2024
@openshift-ci
Copy link

openshift-ci bot commented Oct 16, 2024

@michelle-purcell: I was thinking about moving to Moscow but there is no point Russian into things.

In response to this:

/joke

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.

@openshift-ci
Copy link

openshift-ci bot commented Oct 16, 2024

@michelle-purcell: dog image

In response to this:

/woof

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.

@openshift-ci
Copy link

openshift-ci bot commented Oct 16, 2024

@michelle-purcell: My first time using an elevator was an uplifting experience. The second time let me down.

In response to this:

/joke

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 HCCDOC-3084-InsightOperatorReleaseNotes branch from 276649b to cd7f532 Compare October 16, 2024 12:59
@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 Oct 16, 2024
@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from cd7f532 to 6f96a46 Compare October 17, 2024 13:44
@michelle-purcell
Copy link
Contributor Author

/retest

@michelle-purcell
Copy link
Contributor Author

@dfitzmau - Thanks for your merge review. I think we are good to go.

@dfitzmau
Copy link
Contributor

@dfitzmau - Thanks for your merge review. I think we are good to go.

Hi @michelle-purcell . Would you be OK to squash the commits. I see 8 on this PR. This OCP Docs Manual section should help.

@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch 2 times, most recently from cd016f2 to aca1eee Compare October 23, 2024 16:15
@michelle-purcell
Copy link
Contributor Author

Hi @michelle-purcell . Would you be OK to squash the commits. I see 8 on this PR. This OCP Docs Manual section should help.

Thanks @dfitzmau - No problem. All done. Thanks

@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch 2 times, most recently from 1ce4867 to 52803c5 Compare October 24, 2024 06:46
@michelle-purcell
Copy link
Contributor Author

/retest

@kcarmichael08 kcarmichael08 added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Oct 24, 2024
@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from 52803c5 to 8d23c81 Compare October 29, 2024 11:46
@michelle-purcell
Copy link
Contributor Author

/retest

@michelle-purcell
Copy link
Contributor Author

@dfitzmau & @kcarmichael08 - Done! Thanks so much to you both. Sorry for the delay. I was OOO since last Thursday morning. Thanks again for all your help getting this through. 💚

@kcarmichael08
Copy link
Contributor

Hi @michelle-purcell , before I can merge, we just need to squash to one commit, but everything looks good otherwise!

@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from ec5d113 to e46642b Compare October 29, 2024 13:55
@michelle-purcell
Copy link
Contributor Author

Done. Thanks

@kcarmichael08
Copy link
Contributor

kcarmichael08 commented Oct 29, 2024

Done. Thanks

Unfortunately somehow it's pulling in one extra commit (Andrea's) and I'm still showing 2. I'll contact you on slack to see if we can fix.

Further notes added and eng references

fix

fix

Add SME fixes

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>

HCCDOC-3084 Add release note about the new Rapid Recommendations feature

Further notes added and eng references

fix

fix

Add SME fixes

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Sara Thomas <sarthoma@redhat.com>

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>

fix

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

Co-authored-by: Darragh Fitzmaurice <57954076+dfitzmau@users.noreply.github.com>
@michelle-purcell michelle-purcell force-pushed the HCCDOC-3084-InsightOperatorReleaseNotes branch from e46642b to b615c2c Compare October 29, 2024 15:02
@openshift-ci
Copy link

openshift-ci bot commented Oct 29, 2024

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

@kcarmichael08 kcarmichael08 merged commit 07b95d5 into openshift:enterprise-4.17 Oct 29, 2024
2 checks passed
@kcarmichael08 kcarmichael08 added branch/enterprise-4.17 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 Oct 29, 2024
@kcarmichael08 kcarmichael08 added this to the Continuous Release milestone Oct 29, 2024
@michelle-purcell michelle-purcell deleted the HCCDOC-3084-InsightOperatorReleaseNotes branch October 30, 2024 08:05
@michelle-purcell
Copy link
Contributor Author

/joke

@openshift-ci
Copy link

openshift-ci bot commented Dec 5, 2024

@michelle-purcell: You know what they say about cliffhangers...

In response to this:

/joke

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.17 needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. 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.

5 participants