I will dig into the AWS failure. The other two failures should simply require
copy/paste from the test output :)
Mike
> On Apr 24, 2025, at 11:46 AM, Jean-Baptiste Onofré <j...@nanthrax.net> wrote:
>
> 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
>>>>>>>
>>>>>>
>>>>>
>>
>>
