Skip to content

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

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

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

Conversation

@mlorenzofr
Copy link
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
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?

@mlorenzofr
Copy link
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
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


### 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
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
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.

@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
mlorenzofr and others added 18 commits February 2, 2026 11:31
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>
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>
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