Skip to content

RHIDP-6585: Update ArgoCD plugin docs to include notification capabilities#1774

Open
jmagak wants to merge 9 commits intoredhat-developer:mainfrom
jmagak:RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities
Open

RHIDP-6585: Update ArgoCD plugin docs to include notification capabilities#1774
jmagak wants to merge 9 commits intoredhat-developer:mainfrom
jmagak:RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities

Conversation

@jmagak
Copy link
Member

@jmagak jmagak commented Feb 9, 2026

IMPORTANT: Do Not Merge - To be merged by Docs Team Only

Version(s): 1.9, main

Issue: RHIDP-6585

Preview:

additional-rhdh-capabilities

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Feb 9, 2026

@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from dad7712 to 59ef2df Compare February 9, 2026 21:06
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 59ef2df to 74fbd9b Compare February 10, 2026 08:19
Copy link
Member

@hmanwani-rh hmanwani-rh left a comment

Choose a reason for hiding this comment

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

@jmagak I’ve added some suggestions for the title, subtitle, and abstract. Please take a look when you get a chance and let me know if they look good to you.

I’ve also drafted a disclaimer and shared it with Emily for review. Once it’s approved, I’ll pass it along so we can incorporate it as well.

@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 45df61b to 2df26cd Compare February 10, 2026 15:21
@jmagak
Copy link
Member Author

jmagak commented Feb 10, 2026

@jmagak I’ve added some suggestions for the title, subtitle, and abstract. Please take a look when you get a chance and let me know if they look good to you.

I’ve also drafted a disclaimer and shared it with Emily for review. Once it’s approved, I’ll pass it along so we can incorporate it as well.

Thank you so much, Heena for these great suggestions and generally for the huge work with this.

@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 8f0c499 to 9896864 Compare February 11, 2026 09:47
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 9896864 to 761b0c2 Compare February 11, 2026 10:36
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 761b0c2 to aa8ed2f Compare February 16, 2026 22:11
@jmagak jmagak requested a review from djanickova February 17, 2026 11:35
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from aa8ed2f to 8f0c499 Compare February 17, 2026 12:54
Copy link
Member

@djanickova djanickova left a comment

Choose a reason for hiding this comment

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

Regarding the feedback from test day, to provide additional clarification:
Parts 1.1–1.4 are performed inside the RHDH application (source repo / image build), so changes to the RHDH instance itself.
Parts 2–3 (ArgoCD configuration and Application annotations) are cluster-level steps executed via oc and apply directly to the OpenShift environment hosting ArgoCD.

Possibly we could add a short section to the beginning about what will need to be updated and where - maybe a short list of the steps needed to be done and where they will be executed? However, this documentation is very much aimed at platform engineers (who will probably understand this even from the diagram), not just regular developers, as changes in the cluster need to be done to make this feature work. It would maybe be worth pointing this out in the beginning of the tutorial as well, just so that readers understand that this is quite a technical topic and not something that can be configured by anyone.

@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 535eb81 to 2852506 Compare February 17, 2026 15:38
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 4d6b278 to d3d1d7b Compare February 18, 2026 11:42
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from d3d1d7b to f619306 Compare February 18, 2026 12:31
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from f619306 to e0d6127 Compare February 18, 2026 12:39
@jmagak jmagak requested a review from djanickova February 18, 2026 12:53
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from e0d6127 to 4738452 Compare February 18, 2026 13:04
@jmagak jmagak added Technical review done ⛅ Any procedure has been succesfully tested and removed Technical review needed 🔩 Test all the procedures labels Feb 18, 2026
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 4738452 to 91ab878 Compare February 26, 2026 13:41
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 91ab878 to 939a002 Compare February 27, 2026 11:41
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from 939a002 to b854abd Compare February 27, 2026 11:41
@jmagak jmagak force-pushed the RHIDP-6585-Update-ArgoCD-plugin-docs-to-include-notification-capabilities branch from b854abd to 240bbb6 Compare March 2, 2026 15:44
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 2, 2026

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants