Hi Michael
Good catch. I guess we don’t execute these tests in CI by by gradle default
task (build) right ? We should include it.
I agree the tests have to pass.
I will cancel this vote and we have to fix the tests on main and on the
release branch.
If no volunteer I will fix these tests.
Thanks !
Regards
JB
Le jeu. 24 avr. 2025 à 20:36, Michael Collado <collado.m...@gmail.com> a
écrit :
> -1 (non-binding)
>
> There are test failures in the regression tests. A couple of them seem
> small - the spark_sql_basic.sh.ref file reports an incorrect response from
> the catalog creation.
>
> However, the AWS test_spark_sql_s3_with_privileges.py test also fails with
> this assertion:
>
> 2025-04-24 11:00:35 > assert 'Contents' in data_objects
> 2025-04-24 11:00:35 E AssertionError: assert 'Contents' in
> {'CommonPrefixes': [{'Prefix':
> 'polaris_test/snowflake_catalog/db1/schema/iceberg_test_table_4a2efdb543/obj_layout/0001/1101/'}],
> 'Delimiter': '/', 'EncodingType': 'url', 'IsTruncated': False, ...}
>
> I think the tests ought to be passing.
>
> Mike
>
> > On Apr 23, 2025, at 8:57 AM, Jean-Baptiste Onofré <j...@nanthrax.net>
> wrote:
> >
> > Hi
> >
> > I removed the index.yaml, I re-add it (for testing purpose).
> >
> > Regards
> > JB
> >
> > On Wed, Apr 23, 2025 at 5:16 PM Dmitri Bourlatchkov <di...@apache.org>
> wrote:
> >>
> >> It looks like index.yaml comes and goes randomly :|
> >>
> >> On Wed, Apr 23, 2025 at 11:12 AM Dmitri Bourlatchkov <
> >> dmitri.bourlatch...@dremio.com> wrote:
> >>
> >>> Now, in my env, this worked:
> >>>
> >>> helm repo add polaris
> >>> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/
> >>>
> >>> but:
> >>>
> >>> $ helm repo update
> >>> Hang tight while we grab the latest from your chart repositories...
> >>> ...Unable to get an update from the "polaris" chart repository (
> >>> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/):
> >>> failed to fetch
> >>>
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/index.yaml
> >>> : 404 Not Found
> >>>
> >>> Cheers,
> >>> Dmitri.
> >>>
> >>> On Wed, Apr 23, 2025 at 10:07 AM Alex Dutra
> <alex.du...@dremio.com.invalid>
> >>> wrote:
> >>>
> >>>> Hi JB,
> >>>>
> >>>> The right URL with the release tarball is:
> >>>>
> >>>>
> >>>>
> https://dist.apache.org/repos/dist/dev/incubator/polaris/0.10.0-beta-incubating/
> >>>>
> >>>> With that URL:
> >>>>
> >>>> +1 (non-binding)
> >>>>
> >>>> - Verified checksums and signatures
> >>>> - Verified the source distribution has no binary file
> >>>> - Verified "gradlew rat test" passes on the source distribution
> >>>> - Verified it is possible to import Polaris BOM + Polaris Core in a
> >>>> Gradle project
> >>>>
> >>>> However, I just tried to add the Helm chart repo and it failed:
> >>>>
> >>>> helm repo add polaris
> >>>> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/
> >>>> Error: looks like "
> >>>> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/";
> is
> >>>> not a valid chart repository or cannot be reached: failed to fetch
> >>>>
> >>>>
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart/index.yaml
> >>>> : 404 Not Found
> >>>>
> >>>> I don't think this would prevent the release though, but something we
> want
> >>>> to improve for the future.
> >>>>
> >>>> Thanks,
> >>>>
> >>>> Alex
> >>>>
> >>>> On Wed, Apr 23, 2025 at 2:37 PM Jean-Baptiste Onofré <j...@nanthrax.net
> >
> >>>> wrote:
> >>>>
> >>>>> Hi everyone,
> >>>>>
> >>>>> I propose that we release the following RC as the official
> >>>>> Apache Polaris 0.10.0-beta-incubating release.
> >>>>>
> >>>>> * This corresponds to the tag:
> apache-polaris-0.10.0-beta-incubating-rc0
> >>>>> *
> >>>>>
> >>>>>
> >>>>
> https://github.com/apache/polaris/commits/apache-polaris-0.10.0-beta-incubating-rc0
> >>>>> *
> >>>>>
> >>>>>
> >>>>
> https://github.com/apache/polaris/tree/138fe7ec2883a054c71728b83cdf0c5eb49af21b
> >>>>>
> >>>>> The release tarball, signature, and checksums are here:
> >>>>> *
> >>>>>
> >>>>>
> >>>>
> https://dist.apache.org/repos/dist/dev/incubator/polaris/apache-polaris-0.10.0-beta-incubating
> >>>>>
> >>>>> Helm charts are available on:
> >>>>> *
> https://dist.apache.org/repos/dist/dev/incubator/polaris/helm-chart
> >>>>> NB: the docker images (polaris-server and polaris-admin) will be
> >>>> published
> >>>>> on DockerHub once release vote passes.
> >>>>>
> >>>>> You can find the KEYS file here:
> >>>>> * https://dist.apache.org/repos/dist/release/incubator/polaris/KEYS
> >>>>>
> >>>>> Convenience binary artifacts are staged on Nexus. The Maven
> repositories
> >>>>> URLs are:
> >>>>> *
> >>>>>
> >>>>
> https://repository.apache.org/content/repositories/orgapachepolaris-1004/
> >>>>> *
> >>>>>
> >>>>
> https://repository.apache.org/content/repositories/orgapachepolaris-1005/
> >>>>>
> >>>>> Please download, verify, and test.
> >>>>>
> >>>>> Please vote in the next 72 hours.
> >>>>>
> >>>>> [ ] +1 Release this as Apache polaris 0.10.0-beta-incubating
> >>>>> [ ] +0
> >>>>> [ ] -1 Do not release this because...
> >>>>>
> >>>>> Only PPMC members and mentors 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.
> >>>>>
> >>>>> NB: if this vote passes, a new vote has to be started on the
> Incubator
> >>>>> general mailing
> >>>>> list.
> >>>>>
> >>>>> Thanks
> >>>>> Regards
> >>>>>
> >>>>> JB
> >>>>>
> >>>>
> >>>
>
>
