Skip to content

Conversation

tao12345666333
Copy link
Member

@tao12345666333 tao12345666333 commented Sep 22, 2025

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • PR is based off the current tip of the main branch.
    - [ ] Changes are documented under the "Unreleased" header in CHANGELOG.md
  • New or modified sections of values.yaml are documented in the README.md
  • Commits follow the Kong commit message guidelines

@tao12345666333 tao12345666333 self-assigned this Sep 22, 2025
Signed-off-by: Jintao Zhang <zhangjintao9020@gmail.com>
Signed-off-by: Jintao Zhang <zhangjintao9020@gmail.com>
@tao12345666333 tao12345666333 marked this pull request as ready for review September 29, 2025 08:39
@tao12345666333 tao12345666333 requested a review from a team as a code owner September 29, 2025 08:39
- name: Push kong-operator chart to DockerHub (OCI)
if: ${{ steps.kong_operator_oci.outputs.publish_oci == 'true' && env.DOCKERHUB_USERNAME != '' && env.DOCKERHUB_TOKEN != '' }}
run: |
# Save and push the chart explicitly to kong/kong-operator on DockerHub
Copy link
Member Author

@tao12345666333 tao12345666333 Sep 30, 2025

Choose a reason for hiding this comment

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

To avoid conflicts between the Helm chart and KO image tags, we will push to the kong/kong-operator-chart repo. However, there is a limitation here: when pushing to OCI, the path must match the name of the Chart. We may need to modify the name of the Chart.

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.

Add support for OCI format Helm chart
1 participant