Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync chart with bitnami/kubeapps chart (version 12.4.10) #6658

Merged
merged 4 commits into from
Aug 20, 2023

Conversation

github-actions[bot]
Copy link
Contributor

Description of the change

This is an automatic PR for synchronizing the changes performed externally at the bitnami/kubeapps official chart to the Kubeapps development version.

Benefits

The Kubeapps development chart will get the newest changes, including image version updates and other updates.

Possible drawbacks

Although unlikely, some changes implemented by the Bitnami team may not be 100% compatible with the current Kubeapps version.

Applicable issues

N/A

Additional information

This PR is marked as a draft until a Kubeapps maintainer manually reviews it.

Signed-off-by: kubeapps-bot <tanzu-kubeapps-team@vmware.com>
@netlify
Copy link

netlify bot commented Aug 18, 2023

Deploy Preview for kubeapps-dev canceled.

Name Link
🔨 Latest commit dfdeeb0
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/64dfa23f16b51600080bbc61

Comment on lines +7 to +21
images: |
- name: kubeapps-apis
image: docker.io/bitnami/kubeapps-apis:2.8.0-debian-11-r40
- name: kubeapps-apprepository-controller
image: docker.io/bitnami/kubeapps-apprepository-controller:2.8.0-debian-11-r13
- name: kubeapps-asset-syncer
image: docker.io/bitnami/kubeapps-asset-syncer:2.8.0-debian-11-r13
- name: kubeapps-pinniped-proxy
image: docker.io/bitnami/kubeapps-pinniped-proxy:2.8.0-debian-11-r40
- name: kubeapps-dashboard
image: docker.io/bitnami/kubeapps-dashboard:2.8.0-debian-11-r37
- name: nginx
image: docker.io/bitnami/nginx:1.25.2-debian-11-r0
- name: oauth2-proxy
image: docker.io/bitnami/oauth2-proxy:7.4.0-debian-11-r272
Copy link
Contributor

@antgamdia antgamdia Aug 18, 2023

Choose a reason for hiding this comment

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

We might need to check our sync scripts to ensure we are still performing the replacement properly.

Edit: or we can just ignore it, it is just an extra field they have added.

Copy link
Contributor

@absoludity absoludity Aug 20, 2023

Choose a reason for hiding this comment

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

Edit: or we can just ignore it, it is just an extra field they have added.

For now maybe, but longer term I assume we'd need to ensure this field also gets updated (probably related to being able to move/relocate the chart). EDIT: Ah, or you mean for our dev chart - yeah, I don't think we'll care there, true.

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia marked this pull request as ready for review August 18, 2023 16:56
Comment on lines -1853 to +1859
## @param dashboard.image.registry Dashboard image registry
## @param dashboard.image.repository Dashboard image repository
## @param dashboard.image.tag Dashboard image tag (immutable tags are recommended)
## @param dashboard.image.digest Dashboard image digest in the way sha256:aa.... Please note this parameter, if set, will override the tag
## @param dashboard.image.pullPolicy Dashboard image pull policy
## @param dashboard.image.pullSecrets Dashboard image pull secrets
## @param dashboard.image.debug Enable image debug mode
## @param ociCatalog.image.registry OCI Catalog image registry
## @param ociCatalog.image.repository OCI Catalog image repository
## @param ociCatalog.image.tag OCI Catalog image tag (immutable tags are recommended)
## @param ociCatalog.image.digest OCI Catalog image digest in the way sha256:aa.... Please note this parameter, if set, will override the tag
## @param ociCatalog.image.pullPolicy OCI Catalog image pull policy
## @param ociCatalog.image.pullSecrets Dashboard image pull secrets
## @param ociCatalog.image.debug Enable image debug mode
Copy link
Contributor

Choose a reason for hiding this comment

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

Woops, thanks.

@absoludity
Copy link
Contributor

Landing this as there's another one queued up now.

@absoludity absoludity merged commit c9a2e05 into main Aug 20, 2023
@absoludity absoludity deleted the sync-chart-changes-12.4.10 branch August 20, 2023 21:22
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.

4 participants