Hi all, I identified what was causing the email template files to show up in dist:
https://github.com/apache/polaris/pull/4163 Thanks, Alex On Fri, Apr 10, 2026 at 12:32 AM Dmitri Bourlatchkov <[email protected]> wrote: > > Hi Adnan, > > I'm not sure how the email template .txt files got added to "dist", but I > believe we should not actually "release" them to the users... meaning they > should not be in "dist". > > It should be ok to remove them manually from RC1, IMHO. Yet, if we're going > to make an RC2 we might as well fix the scripts to avoid publishing email > templates. > > WDYT? > > Cheers, > Dmitri. > > On Wed, Apr 8, 2026 at 11:03 PM Adnan Hemani via dev <[email protected]> > wrote: > > > Hi Dmitri, > > > > Thanks for the verifications. The email .txt files were generated as part > > of the automated release process - I'm really not sure why those are there > > either, but didn't change it since I didn't find it to be a big issue. I'm > > happy to make a change to remove them for the next release (or as part of > > this release if RC1 doesn't pass). > > > > I'm not familiar on the helm index background, but can confirm that there > > was not an "index.yaml" file generated as part of the GitHub Action. > > Robert/Pierre, is this something you can confirm that we had consensus on > > doing and if the current release should've done this? > > > > Best, > > Adnan Hemani > > > > On Tue, Apr 7, 2026 at 12:24 PM Dmitri Bourlatchkov <[email protected]> > > wrote: > > > > > Hi All, > > > > > > Voting +0 for now because of the two strange .txt files and helm index > > > issue (below) > > > > > > Verified: > > > * Checksums > > > * Signatures > > > * Build from source > > > > > > So far all clear. > > > > > > However: > > > > > > $ helm repo add polaris-rc > > > https://dist.apache.org/repos/dist/dev/polaris/helm-chart > > > Error: looks like " > > > https://dist.apache.org/repos/dist/dev/polaris/helm-chart"; > > > is not a valid chart repository or cannot be reached: failed to fetch > > > https://dist.apache.org/repos/dist/dev/polaris/helm-chart/index.yaml : > > 404 > > > Not Found > > > > > > IIRC, the idea was for the RCs to host a new helm index verbatim. The > > index > > > will then replace the old index at release publication time. This way > > we'd > > > be able to validate its exact contents before making it available to all > > > users. > > > > > > WDYT? > > > > > > Cheers, > > > Dmitri. > > > > > > > > > > > > On Tue, Apr 7, 2026 at 3:10 PM Dmitri Bourlatchkov <[email protected]> > > > wrote: > > > > > > > Hi Adnan, > > > > > > > > I see these two files in dist... Do we really need them there? > > > > > > > > apache-polaris-1.4.0.vote-email-body.txt > > > > apache-polaris-1.4.0.vote-email-subject.txt > > > > > > > > Thanks, > > > > Dmitri. > > > > > > > > On Mon, Apr 6, 2026 at 9:37 PM Adnan Hemani via dev < > > > > [email protected]> wrote: > > > > > > > >> Hi everyone, > > > >> > > > >> I propose that we release the following RC as the official Apache > > > Polaris > > > >> 1.4.0 release. (Do not worry, there is no RC0 for this release :) > > > >> > > > >> This corresponds to the tag: apache-polaris-1.4.0-rc1 > > > >> * https://github.com/apache/polaris/commits/apache-polaris-1.4.0-rc1 > > > >> * > > > >> > > > >> > > > > > https://github.com/apache/polaris/tree/70f7a0288412f80197fca71d5f635ca221eb6331 > > > >> > > > >> The release tarball, signature, and checksums are here: > > > >> * https://dist.apache.org/repos/dist/dev/polaris/1.4.0 > > > >> > > > >> Helm charts are available on: > > > >> * https://dist.apache.org/repos/dist/dev/polaris/helm-chart/1.4.0 > > > >> > > > >> NB: you have to build the Docker images locally in order to test Helm > > > >> charts. > > > >> > > > >> You can find the KEYS file here: > > > >> * https://downloads.apache.org/polaris/KEYS > > > >> > > > >> Convenience binary artifacts are staged on Nexus. The Maven > > repositories > > > >> URLs are: > > > >> * > > > >> > > > > > https://repository.apache.org/content/repositories/orgapachepolaris-1063/ > > > >> > > > >> Please download, verify, and test according to the release > > verification > > > >> guide, which can be found at: > > > >> * > > > >> > > > >> > > > > > https://polaris.apache.org/community/release-guides/release-verification-guide/ > > > >> > > > >> Please vote in the next 96 hours (extra time given due to Iceberg > > > Summit). > > > >> > > > >> [ ] +1 Release this as Apache Polaris 1.4.0 > > > >> [ ] +0 > > > >> [ ] -1 Do not release this because... > > > >> > > > >> Only PMC members have binding votes, but other community members are > > > >> encouraged to cast non-binding votes. > > > >> This vote will pass if there are 3 binding +1 votes and more binding > > +1 > > > >> votes than -1 votes. > > > >> > > > >> Best, > > > >> Adnan Hemani > > > >> > > > > > > > > >
