Skip to content

Conversation

@jmagak
Copy link
Member

@jmagak jmagak commented Sep 30, 2025

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

Version(s): 1.7, main

Issue: RHIDP-9107

Preview:

1.1. Architecture overview
1.4. Enabling Orchestrator plugins components

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Sep 30, 2025

Copy link

@pkliczewski pkliczewski left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link

@pkliczewski pkliczewski left a comment

Choose a reason for hiding this comment

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

looks good to me

@jmagak jmagak added Technical review done ⛅ Any procedure has been succesfully tested and removed Technical review needed 🔩 Test all the procedures labels Oct 2, 2025
Copy link
Contributor

@deerskindoll deerskindoll left a comment

Choose a reason for hiding this comment

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

a couple of recommendations

@jmagak jmagak force-pushed the RHIDP-9107-Orchestrator-architecture branch from 3f7aef0 to 3fb03f7 Compare October 10, 2025 10:19
@Gerry-Forde Gerry-Forde merged commit 6b8b397 into redhat-developer:main Oct 10, 2025
3 checks passed
@Gerry-Forde
Copy link
Member

/cherry-pick release-1.7

@openshift-cherrypick-robot
Copy link
Contributor

@Gerry-Forde: #1422 failed to apply on top of branch "release-1.7":

Applying: Add architecture section
Using index info to reconstruct a base tree...
M	modules/orchestrator/con-supported-architecture-for-orchestrator.adoc
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): modules/orchestrator/con-supported-architecture-for-orchestrator.adoc deleted in Add architecture section and modified in HEAD. Version HEAD of modules/orchestrator/con-supported-architecture-for-orchestrator.adoc left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config advice.mergeConflict false"
Patch failed at 0001 Add architecture section

In response to this:

/cherry-pick release-1.7

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.

themr0c pushed a commit to themr0c/red-hat-developers-documentation-rhdh that referenced this pull request Oct 13, 2025
* Add architecture section

* Add architecture section

* Add images

* Incorporate technical review

* Incorporate technical review

* Update the section

* Apply technical reviews

* Apply technical reviews

* Apply Janas suggestions

* Apply Jana's suggestions

* Update texts

* Apply new suggestions

* Incorporate Fabrice's suggestions

* Incorporate Fabrice's suggestions

* Apply suggestions

* Apply suggestions

* Apply suggestions

---------

Co-authored-by: GitHub Actions <[email protected]>
Gerry-Forde pushed a commit to Gerry-Forde/red-hat-developers-documentation-rhdh-v2 that referenced this pull request Oct 14, 2025
* Add architecture section

* Add architecture section

* Add images

* Incorporate technical review

* Incorporate technical review

* Update the section

* Apply technical reviews

* Apply technical reviews

* Apply Janas suggestions

* Apply Jana's suggestions

* Update texts

* Apply new suggestions

* Incorporate Fabrice's suggestions

* Incorporate Fabrice's suggestions

* Apply suggestions

* Apply suggestions

* Apply suggestions

---------

Co-authored-by: GitHub Actions <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.