Skip to content

RHIDP-9554: helm chart values overview#1768

Open
deerskindoll wants to merge 22 commits intoredhat-developer:mainfrom
deerskindoll:RHIDP-9554
Open

RHIDP-9554: helm chart values overview#1768
deerskindoll wants to merge 22 commits intoredhat-developer:mainfrom
deerskindoll:RHIDP-9554

Conversation

@deerskindoll
Copy link
Member

@deerskindoll deerskindoll commented Feb 9, 2026

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

Version(s):

main, release-1.9
Issue:

https://issues.redhat.com/browse/RHIDP-9554
Preview:

https://redhat-developer.github.io/red-hat-developers-documentation-rhdh/pr-1768/helm-chart-config-reference/

@deerskindoll deerskindoll changed the title RHIDP-9554: helm chart values overview DRAFT RHIDP-9554: helm chart values overview Feb 9, 2026
@deerskindoll deerskindoll added the Technical review needed 🔩 Test all the procedures label Feb 9, 2026
@deerskindoll deerskindoll changed the title DRAFT RHIDP-9554: helm chart values overview RHIDP-9554: helm chart values overview Feb 9, 2026




Copy link
Member

Choose a reason for hiding this comment

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

why so many blank lines?

Copy link
Member Author

Choose a reason for hiding this comment

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

because the includes are shy and need more space.

sorry, I was moving things around in the file and didn't notice. I'll fix it

----
+
.. OPTIONAL: View default values of the upstream PostgreSQL Chart, which is a dependency of the upstream Backstage Chart.
Relevant if using the local PostgreSQL database (recommended to switch to an external database in production).
Copy link
Member

Choose a reason for hiding this comment

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

this is weirdly worded.

maybe something more like:

 OPTIONAL: View default values of the upstream PostgreSQL Chart, which is a dependency of the upstream Backstage Chart.
While using the local PostgreSQL database is **not recommended for production**, as you should be using your own external database, this allows visibility into the local database.

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

@rm3l
Copy link
Member

rm3l commented Feb 10, 2026

/cc

@openshift-ci openshift-ci bot requested a review from rm3l February 10, 2026 20:25
@rm3l
Copy link
Member

rm3l commented Feb 12, 2026

@deerskindoll would you mind sharing a link to the preview? Looks like the workflow has failed: https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/actions/runs/21883008119/job/63171471408?pr=1768
Thanks.

…m-chart-values.adoc

Co-authored-by: Armel Soro <armel@rm3l.org>
|""

|upstream.backstage.image.pullPolicy
|Specifies the link:https://kubernetes.io/docs/concepts/containers/images/#image-pull-policy[`imagePullPolicy`].
Copy link
Member

Choose a reason for hiding this comment

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

Build error seem related to this line

rsync: [sender] link_stat "/home/runner/work/red-hat-developers-documentation-rhdh/red-hat-developers-documentation-rhdh/pr-content/images/#image-pull-policy" failed: No such file or directory (2)

@rhdh-bot
Copy link
Collaborator

rhdh-bot commented Feb 16, 2026

@deerskindoll
Copy link
Member Author

@deerskindoll would you mind sharing a link to the preview? Looks like the workflow has failed: https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/actions/runs/21883008119/job/63171471408?pr=1768 Thanks.

done

@sonarqubecloud
Copy link

@deerskindoll
Copy link
Member Author

/cherry-pick release-1.9

@openshift-cherrypick-robot
Copy link
Contributor

@deerskindoll: once the present PR merges, I will cherry-pick it on top of release-1.9 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-1.9

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.

----
$ helm pull redhat-developer-hub \
--repo https://charts.openshift.io \
--version {product-chart-version} \
Copy link
Member

Choose a reason for hiding this comment

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

This attribute doesn't seem to be substituted properly in the preview:

Image

Comment on lines +39 to +43
[NOTE]
====
This is an optional image for disconnected
environments.
====
Copy link
Member

Choose a reason for hiding this comment

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

This note does not render properly in the preview:

Image

|{"enabled":true,"image":{"registry":"quay.io","repository":"curl/curl","tag":"latest"},"injectTestNpmrcSecret":false}

|test.enabled
|Enables the test-connection pod used for testing the release using `helm` test.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
|Enables the test-connection pod used for testing the release using `helm` test.
|Enables the test-connection pod used for testing the release using `helm test`.

This is the actual command name.

Copy link
Member

Choose a reason for hiding this comment

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

Could these be merged with modules/helm-chart-config-reference/ref-helm-chart-additional-upstream-namespace-values.adoc? Not sure why we have an additional upstream Backstage chart values, as both are under the upstream scope.. Or did you want to separate upstream and upstream.backstage scopes?

|upstream.backstage.appConfig
|Generates a `ConfigMap` and configures it in the Backstage pods.
|object
|{}
Copy link
Member

Choose a reason for hiding this comment

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

This is the default value in the upstream backstage chart, but this is actually set in the RHDH chart, which is what RHDH customers would be interested about: https://github.com/redhat-developer/rhdh-chart/blob/release-1.9/charts/backstage/values.yaml#L56-L75

|upstream.backstage.extraEnvVars
|Backstage container environment variables.
|list
|[]
Copy link
Member

Choose a reason for hiding this comment

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

|upstream.backstage.extraVolumeMounts
|Backstage container additional volume mounts.
|list
|[]
Copy link
Member

Choose a reason for hiding this comment

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

|upstream.backstage.extraVolumes
|Backstage container additional volumes.
|list
|[]
Copy link
Member

Choose a reason for hiding this comment

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

|upstream.ingress.host
|Hostname to be used to expose the route to access the Backstage application, such as `backstage.IP.nip.io`.
|string
|""
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
|""
|"{{ .Values.global.host }}"

The default value is different in the RHDH chart: https://github.com/redhat-developer/rhdh-chart/blob/release-1.9/charts/backstage/values.yaml#L258C12-L258C37

|upstream.service.extraPorts
|Extra ports to expose in the Backstage service. Typically used with the sidecar value.
|list
|[]
Copy link
Member

Choose a reason for hiding this comment

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

@rm3l
Copy link
Member

rm3l commented Feb 24, 2026

/cc @OpinionatedHeron

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

Labels

Technical review needed 🔩 Test all the procedures

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants