This vote has passed with three +1 votes (myself, Tomek and Kaxil) and no -1 votes.

Release to follow shortly.

On Wed, 25 Nov, 2020 at 11:21, Kaxil Naik <[email protected]> wrote:
+1 (binding)

On Sat, Nov 21, 2020, 13:52 Tomasz Urbaszek <[email protected] <mailto:[email protected]>> wrote:
Then I'm +1 binding. The command itself works as expected.

Tomek

On Sat, Nov 21, 2020 at 2:33 PM Ash Berlin-Taylor <[email protected] <mailto:[email protected]>> wrote:
Yes, this dist is for use with 1.10.13 plus, so installation will be a bit rough right now. If you install the wheel from Kaxil's vote email (_not_ from pypi) that would give you airflow 1.10.13 and then you should be able to install and use this as a user would once that is released.

On 21 November 2020 11:31:42 GMT, Jarek Potiuk <[email protected] <mailto:[email protected]>> wrote:
Thanks Tomek!

Re: version: I think it won't work until we release 1.10.13 (by definition >=1.10.13 will not install rc version until you specify it explicitly). So --no-deps` switch should be a recommended solution for now.

For the docs: I think we are moving fast with releases - and as Ash wrote some docs in dev - it needs updating still and things might get broken :). We are scrambling with the release and our bleeding edge is not as sharp (yet!). Thanks for understanding about it :)

Re: the docs about verification has now been split to airflow/providers and we will need one for upgrade check (or maybe combining them into one):

The right now is here for airflow and likely it is valid for the upgrade check with small modifications: <https://github.com/apache/airflow/blob/master/dev/README_RELEASE_AIRFLOW.md#verify-release-candidates-by-contributors>

J.


On Sat, Nov 21, 2020 at 12:03 PM Tomasz Urbaszek <[email protected] <mailto:[email protected]>> wrote:
Btw. this link is no longer valid:

<https://github.com/apache/airflow/blob/master/dev/README.md#vote-and-verify-the-apache-airflow-release-candidate>

 but the v1-10-test version works:
<https://github.com/apache/airflow/blob/v1-10-test/dev/README.md#vote-and-verify-the-apache-airflow-release-candidate>

 Tomek

On Sat, Nov 21, 2020 at 11:44 AM Tomasz Urbaszek <[email protected] <mailto:[email protected]>> wrote:
 >
> How can this package be installed in an existing Airflow environment? I tried breeze with 1.10.12, 1.10.13 and v1-10-test (+ appending rc1 suffix) and in each case I got:
 >
> root@81b7a417ce5b:/opt/airflow# pip install apache-airflow-upgrade-check
 > Collecting apache-airflow-upgrade-check
> Downloading apache_airflow_upgrade_check-1.0.0rc1-py2.py3-none-any.whl (44 kB) > |████████████████████████████████| 44 kB 1.2 MB/s > Requirement already satisfied: packaging in /usr/local/lib/python3.6/site-packages (from apache-airflow-upgrade-check) (20.4)
 > Collecting importlib-metadata~=2.0; python_version < "3.8"
> Downloading importlib_metadata-2.0.0-py2.py3-none-any.whl (31 kB) > ERROR: Could not find a version that satisfies the requirement apache-airflow<3,>=1.10.13 (from apache-airflow-upgrade-check) (from versions: 1.10.9-bin, 1.8.1, 1.8.2rc1, 1.8.2, 1.9.0, 1.10.0, 1.10.1b1, 1.10.1rc2, 1.10.1, 1.10.2b2, 1.10.2rc1, 1.10.2rc2, 1.10.2rc3, 1.10.2, 1.10.3b1, 1.10.3b2, 1.10.3rc1, 1.10.3rc2, 1.10.3, 1.10.4b2, 1.10.4rc1, 1.10.4rc2, 1.10.4rc3, 1.10.4rc4, 1.10.4rc5, 1.10.4, 1.10.5rc1, 1.10.5, 1.10.6rc1, 1.10.6rc2, 1.10.6, 1.10.7rc1, 1.10.7rc2, 1.10.7rc3, 1.10.7, 1.10.8rc1, 1.10.8, 1.10.9rc1, 1.10.9, 1.10.10rc1, 1.10.10rc2, 1.10.10rc3, 1.10.10rc4, 1.10.10rc5, 1.10.10, 1.10.11rc1, 1.10.11rc2, 1.10.11, 1.10.12rc1, 1.10.12rc2, 1.10.12rc3, 1.10.12rc4, 1.10.12, 1.10.13rc1, 2.0.0b1, 2.0.0b2, 2.0.0b3) > ERROR: No matching distribution found for apache-airflow<3,>=1.10.13 (from apache-airflow-upgrade-check)
 >
> It seems that upgrade check is trying to install airflow even when one is installed already. I was able to install the package using --no-deps flag, however this is not something I would recommend to users.
 >
> The command works fine, it found few problems in default Airflow configuration of 1.10.13.
 >
 > Btw. the right way to run it without 1.10.13rc is:
> python -c 'from airflow.upgrade.checker import __main__; __main__()'
 >
 > Mind the dot between upgrade and checker :)
 >
 > I'm -0.5 due to installation issues.
 >
 > Best,
 > Tomek
 >
> On Fri, Nov 20, 2020 at 11:28 PM Ash Berlin-Taylor <[email protected] <mailto:[email protected]>> wrote:
 >>
 >> Hey all,
 >>
>> This vote is a little bit different to our usual ones. This calls for the first release fo apache-airflow-upgrade-check -- a collection of rules to ease the upgrade from 1.10 to 2.0.0. These checks are not final or complete, and more can be released in future releases of this dist, and users will be notified that there is an update available.
 >>
>> This calls for the release of a new dist: apache-airflow-upgrade-check, version 1.0.0. This represents the contents of the airflow/upgrade/ tree (plus a few supporting files) as a separate dist, as called for in <https://github.com/apache/airflow/issues/11112>
 >>
>> This email is calling a vote on the release, which will last until 10:00 UTC on Wednesday 25th November (timed to coincide with the planned 1.10.13rc1 vote)
 >>
 >> Consider this my (binding) +1.
 >>
 >> The files can be downloaded from
>> <https://dist.apache.org/repos/dist/dev/airflow/upgrade-check/1.0.0rc1/>
 >>
>> *apache-airflow-upgrade-check-1.0.0rc1-source.tar.gz* is a source release containing the files that made up the binary and wheel releases.
 >>
>> * apache-airflow-upgrade-check-1.0.0rc1-bin.tar.gz* is the binary Python "sdist" release.
 >>
>> * apache_airflow_upgrade_check-1.0.0rc1-py2.py3-none-any.whl* is the binary Python pre-compiled wheel file.
 >>
 >> Public keys are available at:
 >> <https://dist.apache.org/repos/dist/release/airflow/KEYS>
 >>
>> This dist is also available PyPI <https://pypi.org/project/apache-airflow-upgrade-check/>
 >>
>> Only votes from PMC members are binding, but members of the community to test the release and vote with "(non-binding)".
 >>
>> The test procedure for PMCs and Contributors who would like to test this RC are described in >> <https://github.com/apache/airflow/blob/master/dev/README.md#vote-and-verify-the-apache-airflow-release-candidate>, but again, this time it is a little bit different.
 >>
>> To actually use this command, either wait for Kaxil to release the 1.10.13rc1, and then run `airflow upgrade_check`, or in the mean time you can run this with `python -c 'from airflow.upgradechecker import __main__; __main__()'`
 >>
>> Please note that the version number inside the archives exclude the `rcX` string, so it's now simply 1.0.0. This will allow us to rename the artifact without modifying the artifact checksums when we actually release.
 >>
 >> Thanks,
 >> Ash


--
Jarek Potiuk
Polidea <https://www.polidea.com/> | Principal Software Engineer

M: +48 660 796 129 <tel:+48660796129>
 <https://www.polidea.com/>



Reply via email to