Skip to content

Add documentation on values for the Supply Chain use case#92

Open
mlorenzofr wants to merge 21 commits intovalidatedpatterns:mainfrom
mlorenzofr:doc-values
Open

Add documentation on values for the Supply Chain use case#92
mlorenzofr wants to merge 21 commits intovalidatedpatterns:mainfrom
mlorenzofr:doc-values

Conversation

@mlorenzofr
Copy link
Copy Markdown
Collaborator

This PR adds documentation on which values ​​should be uncommented in the values-hub.yaml file to set up the Supply Chain use case.

I've also added a patch file so it can be applied and configured quickly.

Under review

@mlorenzofr mlorenzofr self-assigned this Dec 19, 2025
Copy link
Copy Markdown
Collaborator

@minmzzhang minmzzhang left a comment

Choose a reason for hiding this comment

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

I like the patch method, shall we create patch files for all the optional layer1 applications?

Comment thread docs/supply-chain.md Outdated
@mlorenzofr
Copy link
Copy Markdown
Collaborator Author

I like the patch method, shall we create patch files for all the optional layer1 applications?

This is an option I'm proposing. Creating patches is simple and quick to implement. The downside is that it will require more maintenance if we make a lot of changes to the values-hub.yaml file.

Copy link
Copy Markdown
Collaborator

@sabre1041 sabre1041 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. A few suggestions

Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated

### Setting up the values

We've seen the components we'll use in this use case. To configure them, we must add (uncomment) these parameters to the `values-hub.yaml` file
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What does "We've see the components we'll use in this use case" mean?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I am referring to the components described in the previous sections (Main and Supporting Components): RHTAS, RHTPA, ZTWIM, Quay, MCO, and Tekton.

Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
@mlorenzofr mlorenzofr requested a review from sabre1041 January 7, 2026 18:41
@mlorenzofr mlorenzofr marked this pull request as ready for review January 7, 2026 18:41
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
Comment thread docs/supply-chain.md Outdated
mlorenzofr and others added 14 commits April 4, 2026 17:33
Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
mlorenzofr and others added 6 commits April 4, 2026 17:33
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Co-authored-by: Andrew Block <andy.block@gmail.com>
Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
Copy link
Copy Markdown
Collaborator

@sabre1041 sabre1041 left a comment

Choose a reason for hiding this comment

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

@mlorenzofr We are going to need to update the sync waves to align with #109

Signed-off-by: Manuel Lorenzo <mlorenzofr@redhat.com>
@mlorenzofr mlorenzofr requested a review from sabre1041 April 7, 2026 09:29
@mlorenzofr
Copy link
Copy Markdown
Collaborator Author

@sabre1041 document updated with the latest sync-waves changes

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants