You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "adutra (via GitHub)" <gi...@apache.org> on 2026/04/09 11:05:20 UTC

[PR] Releasey: fix missing index.yaml in dist/dev Helm repo [polaris]

adutra opened a new pull request, #4154:
URL: https://github.com/apache/polaris/pull/4154

   This PR fixes a missing index.yaml file in the `dist/dev` Helm chart repository when using the automated release process.
   
   The release-3 workflow currently performs the following steps:
   
   1. Copies the generated `index.yaml` into the SVN working copy
   2. Copies `artifacthub-repo.yml` into the SVN working copy
   3. Runs `svn add` for `artifacthub-repo.yml`
   4. Runs `svn commit`
   
   But it never runs `svn add` for `index.yaml`. This worked before because `index.yaml` was already tracked by SVN.
   
   But now, when the release-4 workflow runs `svn mv` to move `index.yaml` from `dist/dev` to `dist/release`, the file is no longer tracked in `dist/dev`. The next release-3 workflow run copies a new `index.yaml` into the working directory, but since it's untracked, `svn commit` silently ignores it.
   
   <!--
   ๐Ÿ“ Describe what changes you're proposing, especially breaking or user-facing changes. 
   ๐Ÿ“– See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
   -->
   
   ## Checklist
   - [ ] ๐Ÿ›ก๏ธ Don't disclose security issues! (contact security@apache.org)
   - [ ] ๐Ÿ”— Clearly explained why the changes are needed, or linked related issues: Fixes #
   - [ ] ๐Ÿงช Added/updated tests with good coverage, or manually tested (and explained how)
   - [ ] ๐Ÿ’ก Added comments for complex logic
   - [ ] ๐Ÿงพ Updated `CHANGELOG.md` (if needed)
   - [ ] ๐Ÿ“š Updated documentation in `site/content/in-dev/unreleased` (if needed)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Releasey: fix missing index.yaml in dist/dev Helm repo [polaris]

Posted by "adutra (via GitHub)" <gi...@apache.org>.
adutra merged PR #4154:
URL: https://github.com/apache/polaris/pull/4154


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org