Re: [RESULT][VOTE] Airflow 1.9.0rc8

2017-12-27 Thread Jakob Homan
Sent out a call to arms...

-jg


On 27 December 2017 at 10:22, Sid Anand  wrote:
> Ugh... you're right. Hmn... what about our other 2 mentors?
>
> -s
>
> On Wed, Dec 27, 2017 at 4:14 AM, Bolke de Bruin  wrote:
>
>> Did you count 3? I only counted 2?
>>
>> Sent from my iPhone
>>
>> > On 27 Dec 2017, at 02:48, Sid Anand  wrote:
>> >
>> > I see that we received 3 +1 binding votes (assuming all IPMC members).
>> Per
>> > https://incubator.apache.org/guides/releasemanagement.html, shouldn't
>> that
>> > be sufficient?
>> >
>> > In the podling report, should I mention that we did our 4th release or
>> > still working toward it?
>> > -s
>> >
>> > On Tue, Dec 19, 2017 at 9:50 AM, Chris Riccomini 
>> > wrote:
>> >
>> >> I am waiting until incubator actually approves the release. Otherwise, I
>> >> might have to redo the tag if they reject us and we have to re-vote a
>> new
>> >> RC.
>> >>
>> >>> On Tue, Dec 19, 2017 at 8:19 AM, Bolke de Bruin 
>> wrote:
>> >>>
>> >>> Chris,
>> >>>
>> >>> Can you please tag 1.9.0 correctly (ie 1.9.0)? I see reports on gitter
>> >>> people have issues with it.
>> >>>
>> >>> Cheers,
>> >>> Bolke.
>> >>>
>>  On 18 Dec 2017, at 21:49, Chris Riccomini 
>> >> wrote:
>> 
>>  Thanks to all for helping validate it. :)
>> 
>>  On Mon, Dec 18, 2017 at 12:48 PM, Ash Berlin-Taylor <
>>  ash_airflowl...@firemirror.com> wrote:
>> 
>> > Woot! Great news, and thanks for all the time you've spent preparing
>> >> all
>> > the RCs.
>> >
>> > -ash
>> >
>> >> On 18 Dec 2017, at 20:13, Chris Riccomini 
>> >>> wrote:
>> >>
>> >> Hello,
>> >>
>> >> Apache Airflow (incubating) 1.9.0 (based on RC8) has been accepted.
>> >>
>> >> 5 “+1” binding votes received:
>> >>
>> >> - Maxime Beauchemin (binding)
>> >> - Chris Riccomini (binding)
>> >> - Bolke de Bruin (binding)
>> >> - Joy Gao (binding)
>> >> - Fokko Driesprong (binding)
>> >>
>> >> My next step is to open a thread with the IPMC.
>> >>
>> >> Cheers,
>> >> Chris
>> >>
>> >> On Mon, Dec 18, 2017 at 11:26 AM, Chris Riccomini <
>> >>> criccom...@apache.org
>> >>
>> >> wrote:
>> >>
>> >>> I believe so, yes.
>> >>>
>> >>> I will proceed with release. :)
>> >>>
>> >>> On Mon, Dec 18, 2017 at 11:20 AM, Bolke de Bruin <
>> bdbr...@gmail.com
>> >>>
>> >>> wrote:
>> >>>
>>  But it doesn’t stop us from releasing now right? You can just
>> >> update
>> > the
>>  tag. It is not connected to the artifacts.
>> 
>>  Bolke.
>> 
>>  Sent from my iPhone
>> 
>> > On 18 Dec 2017, at 18:51, Chris Riccomini > >
>>  wrote:
>> >
>> > https://github.com/apache/incubator-airflow/pull/2889
>> >
>> > On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini <
>> > criccom...@apache.org
>> >
>> > wrote:
>> >
>> >> Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll
>> >> have
>> > to
>> >> keep editing the tag every time there's a new RC. I agree that
>>  removing the
>> >> check seems like the best course of action.
>> >>
>> >> On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin <
>> >>> bdbr...@gmail.com>
>> >> wrote:
>> >>
>> >>> This is just matter of setting the tag in the repo right?
>> >>>
>> >>> We should remove that check or make it not fail at least. It is
>> >>> ridiculous.
>> >>>
>> >>> B.
>> >>>
>> >>> Verstuurd vanaf mijn iPad
>> >>>
>>  Op 17 dec. 2017 om 07:32 heeft Joy Gao  het
>> > volgende
>> >>> geschreven:
>> 
>>  Ahh, tested the build on a fresh virtualenv, which succeeded
>> >> *the
>>  first
>>  time* given gitpython was not installed and it skipped the
>> > assertion
>> >>> check
>>  > >>> setup.py#L68-L73>.
>>  Build fails on re-installs :(
>> 
>> > On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu
>> > > >
>> >>> wrote:
>> >
>> > +0.5 (non-binding)
>> >
>> > Looks like the version(1.9.0) and tag(1.9.0rc8) is
>> mismatched,
>> > > >>> setup.py#L87>
>> > which will cause the installation (pip install or python
>> >> setup)
>> >>> to
>> >>> error
>> > out and fail.
>> > nit: mind also updating the release log "
>> > 

Re: [RESULT][VOTE] Airflow 1.9.0rc8

2017-12-27 Thread Sid Anand
Ugh... you're right. Hmn... what about our other 2 mentors?

-s

On Wed, Dec 27, 2017 at 4:14 AM, Bolke de Bruin  wrote:

> Did you count 3? I only counted 2?
>
> Sent from my iPhone
>
> > On 27 Dec 2017, at 02:48, Sid Anand  wrote:
> >
> > I see that we received 3 +1 binding votes (assuming all IPMC members).
> Per
> > https://incubator.apache.org/guides/releasemanagement.html, shouldn't
> that
> > be sufficient?
> >
> > In the podling report, should I mention that we did our 4th release or
> > still working toward it?
> > -s
> >
> > On Tue, Dec 19, 2017 at 9:50 AM, Chris Riccomini 
> > wrote:
> >
> >> I am waiting until incubator actually approves the release. Otherwise, I
> >> might have to redo the tag if they reject us and we have to re-vote a
> new
> >> RC.
> >>
> >>> On Tue, Dec 19, 2017 at 8:19 AM, Bolke de Bruin 
> wrote:
> >>>
> >>> Chris,
> >>>
> >>> Can you please tag 1.9.0 correctly (ie 1.9.0)? I see reports on gitter
> >>> people have issues with it.
> >>>
> >>> Cheers,
> >>> Bolke.
> >>>
>  On 18 Dec 2017, at 21:49, Chris Riccomini 
> >> wrote:
> 
>  Thanks to all for helping validate it. :)
> 
>  On Mon, Dec 18, 2017 at 12:48 PM, Ash Berlin-Taylor <
>  ash_airflowl...@firemirror.com> wrote:
> 
> > Woot! Great news, and thanks for all the time you've spent preparing
> >> all
> > the RCs.
> >
> > -ash
> >
> >> On 18 Dec 2017, at 20:13, Chris Riccomini 
> >>> wrote:
> >>
> >> Hello,
> >>
> >> Apache Airflow (incubating) 1.9.0 (based on RC8) has been accepted.
> >>
> >> 5 “+1” binding votes received:
> >>
> >> - Maxime Beauchemin (binding)
> >> - Chris Riccomini (binding)
> >> - Bolke de Bruin (binding)
> >> - Joy Gao (binding)
> >> - Fokko Driesprong (binding)
> >>
> >> My next step is to open a thread with the IPMC.
> >>
> >> Cheers,
> >> Chris
> >>
> >> On Mon, Dec 18, 2017 at 11:26 AM, Chris Riccomini <
> >>> criccom...@apache.org
> >>
> >> wrote:
> >>
> >>> I believe so, yes.
> >>>
> >>> I will proceed with release. :)
> >>>
> >>> On Mon, Dec 18, 2017 at 11:20 AM, Bolke de Bruin <
> bdbr...@gmail.com
> >>>
> >>> wrote:
> >>>
>  But it doesn’t stop us from releasing now right? You can just
> >> update
> > the
>  tag. It is not connected to the artifacts.
> 
>  Bolke.
> 
>  Sent from my iPhone
> 
> > On 18 Dec 2017, at 18:51, Chris Riccomini  >
>  wrote:
> >
> > https://github.com/apache/incubator-airflow/pull/2889
> >
> > On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini <
> > criccom...@apache.org
> >
> > wrote:
> >
> >> Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll
> >> have
> > to
> >> keep editing the tag every time there's a new RC. I agree that
>  removing the
> >> check seems like the best course of action.
> >>
> >> On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin <
> >>> bdbr...@gmail.com>
> >> wrote:
> >>
> >>> This is just matter of setting the tag in the repo right?
> >>>
> >>> We should remove that check or make it not fail at least. It is
> >>> ridiculous.
> >>>
> >>> B.
> >>>
> >>> Verstuurd vanaf mijn iPad
> >>>
>  Op 17 dec. 2017 om 07:32 heeft Joy Gao  het
> > volgende
> >>> geschreven:
> 
>  Ahh, tested the build on a fresh virtualenv, which succeeded
> >> *the
>  first
>  time* given gitpython was not installed and it skipped the
> > assertion
> >>> check
>   >>> setup.py#L68-L73>.
>  Build fails on re-installs :(
> 
> > On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu
> >  >
> >>> wrote:
> >
> > +0.5 (non-binding)
> >
> > Looks like the version(1.9.0) and tag(1.9.0rc8) is
> mismatched,
> >  >>> setup.py#L87>
> > which will cause the installation (pip install or python
> >> setup)
> >>> to
> >>> error
> > out and fail.
> > nit: mind also updating the release log "
> > https://github.com/apache/incubator-airflow/blob/1.9.0rc8/
> >>> CHANGELOG.txt"
> >
> >
> > On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko
> >>> 
> > wrote:
> >
> >> 

Re: [RESULT][VOTE] Airflow 1.9.0rc8

2017-12-18 Thread Chris Riccomini
Thanks to all for helping validate it. :)

On Mon, Dec 18, 2017 at 12:48 PM, Ash Berlin-Taylor <
ash_airflowl...@firemirror.com> wrote:

> Woot! Great news, and thanks for all the time you've spent preparing all
> the RCs.
>
> -ash
>
> > On 18 Dec 2017, at 20:13, Chris Riccomini  wrote:
> >
> > Hello,
> >
> > Apache Airflow (incubating) 1.9.0 (based on RC8) has been accepted.
> >
> > 5 “+1” binding votes received:
> >
> > - Maxime Beauchemin (binding)
> > - Chris Riccomini (binding)
> > - Bolke de Bruin (binding)
> > - Joy Gao (binding)
> > - Fokko Driesprong (binding)
> >
> > My next step is to open a thread with the IPMC.
> >
> > Cheers,
> > Chris
> >
> > On Mon, Dec 18, 2017 at 11:26 AM, Chris Riccomini  >
> > wrote:
> >
> >> I believe so, yes.
> >>
> >> I will proceed with release. :)
> >>
> >> On Mon, Dec 18, 2017 at 11:20 AM, Bolke de Bruin 
> >> wrote:
> >>
> >>> But it doesn’t stop us from releasing now right? You can just update
> the
> >>> tag. It is not connected to the artifacts.
> >>>
> >>> Bolke.
> >>>
> >>> Sent from my iPhone
> >>>
>  On 18 Dec 2017, at 18:51, Chris Riccomini 
> >>> wrote:
> 
>  https://github.com/apache/incubator-airflow/pull/2889
> 
>  On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini <
> criccom...@apache.org
> 
>  wrote:
> 
> > Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have
> to
> > keep editing the tag every time there's a new RC. I agree that
> >>> removing the
> > check seems like the best course of action.
> >
> > On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin 
> > wrote:
> >
> >> This is just matter of setting the tag in the repo right?
> >>
> >> We should remove that check or make it not fail at least. It is
> >> ridiculous.
> >>
> >> B.
> >>
> >> Verstuurd vanaf mijn iPad
> >>
> >>> Op 17 dec. 2017 om 07:32 heeft Joy Gao  het
> volgende
> >> geschreven:
> >>>
> >>> Ahh, tested the build on a fresh virtualenv, which succeeded *the
> >>> first
> >>> time* given gitpython was not installed and it skipped the
> assertion
> >> check
> >>>  >> setup.py#L68-L73>.
> >>> Build fails on re-installs :(
> >>>
>  On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu
>  
> >> wrote:
> 
>  +0.5 (non-binding)
> 
>  Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
>   >> setup.py#L87>
>  which will cause the installation (pip install or python setup) to
> >> error
>  out and fail.
>  nit: mind also updating the release log "
>  https://github.com/apache/incubator-airflow/blob/1.9.0rc8/
> >> CHANGELOG.txt"
> 
> 
>  On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko
> >> 
>  wrote:
> 
> > +1 binding
> >
> > Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin <
> >>> bdbr...@gmail.com
> >>>
> >
> >> +1, binding
> >>
> >> Checked sigs, version, source is there (did not check build),
> bin
> >>> is
> > there.
> >>
> >> Bolke
> >>
> >> Verstuurd vanaf mijn iPad
> >>
> >>> Op 15 dec. 2017 om 23:31 heeft Joy Gao  het
> >> volgende
> >> geschreven:
> >>>
> >>> +1, binding
> >>>
> >>> Thank you Chris!
> >>>
> >>> On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
> > criccom...@apache.org>
> >>> wrote:
> >>>
>  Hey all,
> 
>  (Last time, I hope)^2
> 
>  I have cut Airflow 1.9.0 RC8. This email is calling a vote on
> >>> the
> >> release,
>  which will last for 72 hours. Consider this my (binding) +1.
> 
>  Airflow 1.9.0 RC8 is available at:
> 
>  https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9
> >>> .0rc8/
> 
>  apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
>  release
> >> that
>  comes with INSTALL instructions.
>  apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary
> >>> Python
> >> "sdist"
>  release.
> 
>  Public keys are available at:
> 
>  https://dist.apache.org/repos/dist/release/incubator/airflow/
> 
>  The release contains no new JIRAs. Just a version fix.
> 
>  I also had to change the version number to exclude the `rc6`
> >>> string
> 

[RESULT][VOTE] Airflow 1.9.0rc8

2017-12-18 Thread Chris Riccomini
Hello,

Apache Airflow (incubating) 1.9.0 (based on RC8) has been accepted.

5 “+1” binding votes received:

- Maxime Beauchemin (binding)
- Chris Riccomini (binding)
- Bolke de Bruin (binding)
- Joy Gao (binding)
- Fokko Driesprong (binding)

My next step is to open a thread with the IPMC.

Cheers,
Chris

On Mon, Dec 18, 2017 at 11:26 AM, Chris Riccomini 
wrote:

> I believe so, yes.
>
> I will proceed with release. :)
>
> On Mon, Dec 18, 2017 at 11:20 AM, Bolke de Bruin 
> wrote:
>
>> But it doesn’t stop us from releasing now right? You can just update the
>> tag. It is not connected to the artifacts.
>>
>> Bolke.
>>
>> Sent from my iPhone
>>
>> > On 18 Dec 2017, at 18:51, Chris Riccomini 
>> wrote:
>> >
>> > https://github.com/apache/incubator-airflow/pull/2889
>> >
>> > On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini > >
>> > wrote:
>> >
>> >> Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have to
>> >> keep editing the tag every time there's a new RC. I agree that
>> removing the
>> >> check seems like the best course of action.
>> >>
>> >> On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin 
>> >> wrote:
>> >>
>> >>> This is just matter of setting the tag in the repo right?
>> >>>
>> >>> We should remove that check or make it not fail at least. It is
>> >>> ridiculous.
>> >>>
>> >>> B.
>> >>>
>> >>> Verstuurd vanaf mijn iPad
>> >>>
>>  Op 17 dec. 2017 om 07:32 heeft Joy Gao  het volgende
>> >>> geschreven:
>> 
>>  Ahh, tested the build on a fresh virtualenv, which succeeded *the
>> first
>>  time* given gitpython was not installed and it skipped the assertion
>> >>> check
>>  > >>> setup.py#L68-L73>.
>>  Build fails on re-installs :(
>> 
>> > On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu > >
>> >>> wrote:
>> >
>> > +0.5 (non-binding)
>> >
>> > Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
>> > > >>> setup.py#L87>
>> > which will cause the installation (pip install or python setup) to
>> >>> error
>> > out and fail.
>> > nit: mind also updating the release log "
>> > https://github.com/apache/incubator-airflow/blob/1.9.0rc8/
>> >>> CHANGELOG.txt"
>> >
>> >
>> > On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko
>> >>> 
>> > wrote:
>> >
>> >> +1 binding
>> >>
>> >> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin <
>> bdbr...@gmail.com
>> 
>> >>
>> >>> +1, binding
>> >>>
>> >>> Checked sigs, version, source is there (did not check build), bin
>> is
>> >> there.
>> >>>
>> >>> Bolke
>> >>>
>> >>> Verstuurd vanaf mijn iPad
>> >>>
>>  Op 15 dec. 2017 om 23:31 heeft Joy Gao  het
>> >>> volgende
>> >>> geschreven:
>> 
>>  +1, binding
>> 
>>  Thank you Chris!
>> 
>>  On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
>> >> criccom...@apache.org>
>>  wrote:
>> 
>> > Hey all,
>> >
>> > (Last time, I hope)^2
>> >
>> > I have cut Airflow 1.9.0 RC8. This email is calling a vote on
>> the
>> >>> release,
>> > which will last for 72 hours. Consider this my (binding) +1.
>> >
>> > Airflow 1.9.0 RC8 is available at:
>> >
>> > https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9
>> .0rc8/
>> >
>> > apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
>> > release
>> >>> that
>> > comes with INSTALL instructions.
>> > apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary
>> Python
>> >>> "sdist"
>> > release.
>> >
>> > Public keys are available at:
>> >
>> > https://dist.apache.org/repos/dist/release/incubator/airflow/
>> >
>> > The release contains no new JIRAs. Just a version fix.
>> >
>> > I also had to change the version number to exclude the `rc6`
>> string
>> > as
>> >>> well
>> > as the "+incubating" string, so it's now simply 1.9.0. This will
>> > allow
>> >>> us
>> > to rename the artifact without modifying the artifact checksums
>> >>> when
>> >> we
>> > actually release.
>> >
>> > See JIRAs that were in 1.9.0RC7 and before (see previous VOTE
>> email
>> >> for
>> > full list).
>> >
>> > Cheers,
>> > Chris
>> >
>> >>>
>> >>
>> >
>> >>>
>> >>
>> >>
>>
>
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-18 Thread Chris Riccomini
I believe so, yes.

I will proceed with release. :)

On Mon, Dec 18, 2017 at 11:20 AM, Bolke de Bruin  wrote:

> But it doesn’t stop us from releasing now right? You can just update the
> tag. It is not connected to the artifacts.
>
> Bolke.
>
> Sent from my iPhone
>
> > On 18 Dec 2017, at 18:51, Chris Riccomini  wrote:
> >
> > https://github.com/apache/incubator-airflow/pull/2889
> >
> > On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini 
> > wrote:
> >
> >> Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have to
> >> keep editing the tag every time there's a new RC. I agree that removing
> the
> >> check seems like the best course of action.
> >>
> >> On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin 
> >> wrote:
> >>
> >>> This is just matter of setting the tag in the repo right?
> >>>
> >>> We should remove that check or make it not fail at least. It is
> >>> ridiculous.
> >>>
> >>> B.
> >>>
> >>> Verstuurd vanaf mijn iPad
> >>>
>  Op 17 dec. 2017 om 07:32 heeft Joy Gao  het volgende
> >>> geschreven:
> 
>  Ahh, tested the build on a fresh virtualenv, which succeeded *the
> first
>  time* given gitpython was not installed and it skipped the assertion
> >>> check
>   >>> setup.py#L68-L73>.
>  Build fails on re-installs :(
> 
> > On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu 
> >>> wrote:
> >
> > +0.5 (non-binding)
> >
> > Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
> >  >>> setup.py#L87>
> > which will cause the installation (pip install or python setup) to
> >>> error
> > out and fail.
> > nit: mind also updating the release log "
> > https://github.com/apache/incubator-airflow/blob/1.9.0rc8/
> >>> CHANGELOG.txt"
> >
> >
> > On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko
> >>> 
> > wrote:
> >
> >> +1 binding
> >>
> >> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin <
> bdbr...@gmail.com
> 
> >>
> >>> +1, binding
> >>>
> >>> Checked sigs, version, source is there (did not check build), bin
> is
> >> there.
> >>>
> >>> Bolke
> >>>
> >>> Verstuurd vanaf mijn iPad
> >>>
>  Op 15 dec. 2017 om 23:31 heeft Joy Gao  het
> >>> volgende
> >>> geschreven:
> 
>  +1, binding
> 
>  Thank you Chris!
> 
>  On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
> >> criccom...@apache.org>
>  wrote:
> 
> > Hey all,
> >
> > (Last time, I hope)^2
> >
> > I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
> >>> release,
> > which will last for 72 hours. Consider this my (binding) +1.
> >
> > Airflow 1.9.0 RC8 is available at:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/airflow/1.
> 9.0rc8/
> >
> > apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
> > release
> >>> that
> > comes with INSTALL instructions.
> > apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary
> Python
> >>> "sdist"
> > release.
> >
> > Public keys are available at:
> >
> > https://dist.apache.org/repos/dist/release/incubator/airflow/
> >
> > The release contains no new JIRAs. Just a version fix.
> >
> > I also had to change the version number to exclude the `rc6`
> string
> > as
> >>> well
> > as the "+incubating" string, so it's now simply 1.9.0. This will
> > allow
> >>> us
> > to rename the artifact without modifying the artifact checksums
> >>> when
> >> we
> > actually release.
> >
> > See JIRAs that were in 1.9.0RC7 and before (see previous VOTE
> email
> >> for
> > full list).
> >
> > Cheers,
> > Chris
> >
> >>>
> >>
> >
> >>>
> >>
> >>
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-18 Thread Bolke de Bruin
But it doesn’t stop us from releasing now right? You can just update the tag. 
It is not connected to the artifacts. 

Bolke. 

Sent from my iPhone

> On 18 Dec 2017, at 18:51, Chris Riccomini  wrote:
> 
> https://github.com/apache/incubator-airflow/pull/2889
> 
> On Mon, Dec 18, 2017 at 9:44 AM, Chris Riccomini 
> wrote:
> 
>> Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have to
>> keep editing the tag every time there's a new RC. I agree that removing the
>> check seems like the best course of action.
>> 
>> On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin 
>> wrote:
>> 
>>> This is just matter of setting the tag in the repo right?
>>> 
>>> We should remove that check or make it not fail at least. It is
>>> ridiculous.
>>> 
>>> B.
>>> 
>>> Verstuurd vanaf mijn iPad
>>> 
 Op 17 dec. 2017 om 07:32 heeft Joy Gao  het volgende
>>> geschreven:
 
 Ahh, tested the build on a fresh virtualenv, which succeeded *the first
 time* given gitpython was not installed and it skipped the assertion
>>> check
 >> setup.py#L68-L73>.
 Build fails on re-installs :(
 
> On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu 
>>> wrote:
> 
> +0.5 (non-binding)
> 
> Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
> >> setup.py#L87>
> which will cause the installation (pip install or python setup) to
>>> error
> out and fail.
> nit: mind also updating the release log "
> https://github.com/apache/incubator-airflow/blob/1.9.0rc8/
>>> CHANGELOG.txt"
> 
> 
> On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko
>>> 
> wrote:
> 
>> +1 binding
>> 
>> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin > 
>>> +1, binding
>>> 
>>> Checked sigs, version, source is there (did not check build), bin is
>> there.
>>> 
>>> Bolke
>>> 
>>> Verstuurd vanaf mijn iPad
>>> 
 Op 15 dec. 2017 om 23:31 heeft Joy Gao  het
>>> volgende
>>> geschreven:
 
 +1, binding
 
 Thank you Chris!
 
 On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
>> criccom...@apache.org>
 wrote:
 
> Hey all,
> 
> (Last time, I hope)^2
> 
> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
>>> release,
> which will last for 72 hours. Consider this my (binding) +1.
> 
> Airflow 1.9.0 RC8 is available at:
> 
> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> 
> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
> release
>>> that
> comes with INSTALL instructions.
> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
>>> "sdist"
> release.
> 
> Public keys are available at:
> 
> https://dist.apache.org/repos/dist/release/incubator/airflow/
> 
> The release contains no new JIRAs. Just a version fix.
> 
> I also had to change the version number to exclude the `rc6` string
> as
>>> well
> as the "+incubating" string, so it's now simply 1.9.0. This will
> allow
>>> us
> to rename the artifact without modifying the artifact checksums
>>> when
>> we
> actually release.
> 
> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email
>> for
> full list).
> 
> Cheers,
> Chris
> 
>>> 
>> 
> 
>>> 
>> 
>> 


Re: [VOTE] Airflow 1.9.0rc8

2017-12-18 Thread Chris Riccomini
Sigh. There's a catch 22 here. If we tag an RC as 1.9.0, we'll have to keep
editing the tag every time there's a new RC. I agree that removing the
check seems like the best course of action.

On Sun, Dec 17, 2017 at 12:01 AM, Bolke de Bruin  wrote:

> This is just matter of setting the tag in the repo right?
>
> We should remove that check or make it not fail at least. It is ridiculous.
>
> B.
>
> Verstuurd vanaf mijn iPad
>
> > Op 17 dec. 2017 om 07:32 heeft Joy Gao  het volgende
> geschreven:
> >
> > Ahh, tested the build on a fresh virtualenv, which succeeded *the first
> > time* given gitpython was not installed and it skipped the assertion
> check
> >  0rc8/setup.py#L68-L73>.
> > Build fails on re-installs :(
> >
> >> On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu 
> wrote:
> >>
> >> +0.5 (non-binding)
> >>
> >> Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
> >>  >
> >> which will cause the installation (pip install or python setup) to error
> >> out and fail.
> >> nit: mind also updating the release log "
> >> https://github.com/apache/incubator-airflow/blob/1.9.0rc8/CHANGELOG.txt
> "
> >>
> >>
> >> On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko  >
> >> wrote:
> >>
> >>> +1 binding
> >>>
> >>> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin 
> >>>
>  +1, binding
> 
>  Checked sigs, version, source is there (did not check build), bin is
> >>> there.
> 
>  Bolke
> 
>  Verstuurd vanaf mijn iPad
> 
> > Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende
>  geschreven:
> >
> > +1, binding
> >
> > Thank you Chris!
> >
> > On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
> >>> criccom...@apache.org>
> > wrote:
> >
> >> Hey all,
> >>
> >> (Last time, I hope)^2
> >>
> >> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
>  release,
> >> which will last for 72 hours. Consider this my (binding) +1.
> >>
> >> Airflow 1.9.0 RC8 is available at:
> >>
> >> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> >>
> >> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
> >> release
>  that
> >> comes with INSTALL instructions.
> >> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
>  "sdist"
> >> release.
> >>
> >> Public keys are available at:
> >>
> >> https://dist.apache.org/repos/dist/release/incubator/airflow/
> >>
> >> The release contains no new JIRAs. Just a version fix.
> >>
> >> I also had to change the version number to exclude the `rc6` string
> >> as
>  well
> >> as the "+incubating" string, so it's now simply 1.9.0. This will
> >> allow
>  us
> >> to rename the artifact without modifying the artifact checksums when
> >>> we
> >> actually release.
> >>
> >> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email
> >>> for
> >> full list).
> >>
> >> Cheers,
> >> Chris
> >>
> 
> >>>
> >>
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-17 Thread Bolke de Bruin
This is just matter of setting the tag in the repo right? 

We should remove that check or make it not fail at least. It is ridiculous. 

B.

Verstuurd vanaf mijn iPad

> Op 17 dec. 2017 om 07:32 heeft Joy Gao  het volgende 
> geschreven:
> 
> Ahh, tested the build on a fresh virtualenv, which succeeded *the first
> time* given gitpython was not installed and it skipped the assertion check
> .
> Build fails on re-installs :(
> 
>> On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu  wrote:
>> 
>> +0.5 (non-binding)
>> 
>> Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
>> 
>> which will cause the installation (pip install or python setup) to error
>> out and fail.
>> nit: mind also updating the release log "
>> https://github.com/apache/incubator-airflow/blob/1.9.0rc8/CHANGELOG.txt;
>> 
>> 
>> On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko 
>> wrote:
>> 
>>> +1 binding
>>> 
>>> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin 
>>> 
 +1, binding
 
 Checked sigs, version, source is there (did not check build), bin is
>>> there.
 
 Bolke
 
 Verstuurd vanaf mijn iPad
 
> Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende
 geschreven:
> 
> +1, binding
> 
> Thank you Chris!
> 
> On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
>>> criccom...@apache.org>
> wrote:
> 
>> Hey all,
>> 
>> (Last time, I hope)^2
>> 
>> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
 release,
>> which will last for 72 hours. Consider this my (binding) +1.
>> 
>> Airflow 1.9.0 RC8 is available at:
>> 
>> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
>> 
>> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
>> release
 that
>> comes with INSTALL instructions.
>> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
 "sdist"
>> release.
>> 
>> Public keys are available at:
>> 
>> https://dist.apache.org/repos/dist/release/incubator/airflow/
>> 
>> The release contains no new JIRAs. Just a version fix.
>> 
>> I also had to change the version number to exclude the `rc6` string
>> as
 well
>> as the "+incubating" string, so it's now simply 1.9.0. This will
>> allow
 us
>> to rename the artifact without modifying the artifact checksums when
>>> we
>> actually release.
>> 
>> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email
>>> for
>> full list).
>> 
>> Cheers,
>> Chris
>> 
 
>>> 
>> 


Re: [VOTE] Airflow 1.9.0rc8

2017-12-16 Thread Joy Gao
Ahh, tested the build on a fresh virtualenv, which succeeded *the first
time* given gitpython was not installed and it skipped the assertion check
.
Build fails on re-installs :(

On Fri, Dec 15, 2017 at 6:01 PM, Feng Lu  wrote:

> +0.5 (non-binding)
>
> Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,
> 
> which will cause the installation (pip install or python setup) to error
> out and fail.
> nit: mind also updating the release log "
> https://github.com/apache/incubator-airflow/blob/1.9.0rc8/CHANGELOG.txt;
>
>
> On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko 
> wrote:
>
> > +1 binding
> >
> > Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin 
> >
> > > +1, binding
> > >
> > > Checked sigs, version, source is there (did not check build), bin is
> > there.
> > >
> > > Bolke
> > >
> > > Verstuurd vanaf mijn iPad
> > >
> > > > Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende
> > > geschreven:
> > > >
> > > > +1, binding
> > > >
> > > > Thank you Chris!
> > > >
> > > > On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
> > criccom...@apache.org>
> > > > wrote:
> > > >
> > > >> Hey all,
> > > >>
> > > >> (Last time, I hope)^2
> > > >>
> > > >> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
> > > release,
> > > >> which will last for 72 hours. Consider this my (binding) +1.
> > > >>
> > > >> Airflow 1.9.0 RC8 is available at:
> > > >>
> > > >> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> > > >>
> > > >> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source
> release
> > > that
> > > >> comes with INSTALL instructions.
> > > >> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
> > > "sdist"
> > > >> release.
> > > >>
> > > >> Public keys are available at:
> > > >>
> > > >> https://dist.apache.org/repos/dist/release/incubator/airflow/
> > > >>
> > > >> The release contains no new JIRAs. Just a version fix.
> > > >>
> > > >> I also had to change the version number to exclude the `rc6` string
> as
> > > well
> > > >> as the "+incubating" string, so it's now simply 1.9.0. This will
> allow
> > > us
> > > >> to rename the artifact without modifying the artifact checksums when
> > we
> > > >> actually release.
> > > >>
> > > >> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email
> > for
> > > >> full list).
> > > >>
> > > >> Cheers,
> > > >> Chris
> > > >>
> > >
> >
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Feng Lu
+0.5 (non-binding)

Looks like the version(1.9.0) and tag(1.9.0rc8) is mismatched,

which will cause the installation (pip install or python setup) to error
out and fail.
nit: mind also updating the release log "
https://github.com/apache/incubator-airflow/blob/1.9.0rc8/CHANGELOG.txt;


On Fri, Dec 15, 2017 at 3:21 PM, Driesprong, Fokko 
wrote:

> +1 binding
>
> Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin 
>
> > +1, binding
> >
> > Checked sigs, version, source is there (did not check build), bin is
> there.
> >
> > Bolke
> >
> > Verstuurd vanaf mijn iPad
> >
> > > Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende
> > geschreven:
> > >
> > > +1, binding
> > >
> > > Thank you Chris!
> > >
> > > On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini <
> criccom...@apache.org>
> > > wrote:
> > >
> > >> Hey all,
> > >>
> > >> (Last time, I hope)^2
> > >>
> > >> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
> > release,
> > >> which will last for 72 hours. Consider this my (binding) +1.
> > >>
> > >> Airflow 1.9.0 RC8 is available at:
> > >>
> > >> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> > >>
> > >> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release
> > that
> > >> comes with INSTALL instructions.
> > >> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
> > "sdist"
> > >> release.
> > >>
> > >> Public keys are available at:
> > >>
> > >> https://dist.apache.org/repos/dist/release/incubator/airflow/
> > >>
> > >> The release contains no new JIRAs. Just a version fix.
> > >>
> > >> I also had to change the version number to exclude the `rc6` string as
> > well
> > >> as the "+incubating" string, so it's now simply 1.9.0. This will allow
> > us
> > >> to rename the artifact without modifying the artifact checksums when
> we
> > >> actually release.
> > >>
> > >> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email
> for
> > >> full list).
> > >>
> > >> Cheers,
> > >> Chris
> > >>
> >
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Driesprong, Fokko
+1 binding

Op vr 15 dec. 2017 om 23:39 schreef Bolke de Bruin 

> +1, binding
>
> Checked sigs, version, source is there (did not check build), bin is there.
>
> Bolke
>
> Verstuurd vanaf mijn iPad
>
> > Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende
> geschreven:
> >
> > +1, binding
> >
> > Thank you Chris!
> >
> > On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini 
> > wrote:
> >
> >> Hey all,
> >>
> >> (Last time, I hope)^2
> >>
> >> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
> release,
> >> which will last for 72 hours. Consider this my (binding) +1.
> >>
> >> Airflow 1.9.0 RC8 is available at:
> >>
> >> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> >>
> >> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release
> that
> >> comes with INSTALL instructions.
> >> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
> "sdist"
> >> release.
> >>
> >> Public keys are available at:
> >>
> >> https://dist.apache.org/repos/dist/release/incubator/airflow/
> >>
> >> The release contains no new JIRAs. Just a version fix.
> >>
> >> I also had to change the version number to exclude the `rc6` string as
> well
> >> as the "+incubating" string, so it's now simply 1.9.0. This will allow
> us
> >> to rename the artifact without modifying the artifact checksums when we
> >> actually release.
> >>
> >> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email for
> >> full list).
> >>
> >> Cheers,
> >> Chris
> >>
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Bolke de Bruin
+1, binding

Checked sigs, version, source is there (did not check build), bin is there.

Bolke

Verstuurd vanaf mijn iPad

> Op 15 dec. 2017 om 23:31 heeft Joy Gao  het volgende 
> geschreven:
> 
> +1, binding
> 
> Thank you Chris!
> 
> On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini 
> wrote:
> 
>> Hey all,
>> 
>> (Last time, I hope)^2
>> 
>> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the release,
>> which will last for 72 hours. Consider this my (binding) +1.
>> 
>> Airflow 1.9.0 RC8 is available at:
>> 
>> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
>> 
>> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release that
>> comes with INSTALL instructions.
>> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python "sdist"
>> release.
>> 
>> Public keys are available at:
>> 
>> https://dist.apache.org/repos/dist/release/incubator/airflow/
>> 
>> The release contains no new JIRAs. Just a version fix.
>> 
>> I also had to change the version number to exclude the `rc6` string as well
>> as the "+incubating" string, so it's now simply 1.9.0. This will allow us
>> to rename the artifact without modifying the artifact checksums when we
>> actually release.
>> 
>> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email for
>> full list).
>> 
>> Cheers,
>> Chris
>> 


Re: [VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Maxime Beauchemin
+1, binding

On Fri, Dec 15, 2017 at 2:31 PM, Joy Gao  wrote:

> +1, binding
>
> Thank you Chris!
>
> On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini 
> wrote:
>
> > Hey all,
> >
> > (Last time, I hope)^2
> >
> > I have cut Airflow 1.9.0 RC8. This email is calling a vote on the
> release,
> > which will last for 72 hours. Consider this my (binding) +1.
> >
> > Airflow 1.9.0 RC8 is available at:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
> >
> > apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release
> that
> > comes with INSTALL instructions.
> > apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python
> "sdist"
> > release.
> >
> > Public keys are available at:
> >
> > https://dist.apache.org/repos/dist/release/incubator/airflow/
> >
> > The release contains no new JIRAs. Just a version fix.
> >
> > I also had to change the version number to exclude the `rc6` string as
> well
> > as the "+incubating" string, so it's now simply 1.9.0. This will allow us
> > to rename the artifact without modifying the artifact checksums when we
> > actually release.
> >
> > See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email for
> > full list).
> >
> > Cheers,
> > Chris
> >
>


Re: [VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Joy Gao
+1, binding

Thank you Chris!

On Fri, Dec 15, 2017 at 2:30 PM, Chris Riccomini 
wrote:

> Hey all,
>
> (Last time, I hope)^2
>
> I have cut Airflow 1.9.0 RC8. This email is calling a vote on the release,
> which will last for 72 hours. Consider this my (binding) +1.
>
> Airflow 1.9.0 RC8 is available at:
>
> https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/
>
> apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release that
> comes with INSTALL instructions.
> apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python "sdist"
> release.
>
> Public keys are available at:
>
> https://dist.apache.org/repos/dist/release/incubator/airflow/
>
> The release contains no new JIRAs. Just a version fix.
>
> I also had to change the version number to exclude the `rc6` string as well
> as the "+incubating" string, so it's now simply 1.9.0. This will allow us
> to rename the artifact without modifying the artifact checksums when we
> actually release.
>
> See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email for
> full list).
>
> Cheers,
> Chris
>


[VOTE] Airflow 1.9.0rc8

2017-12-15 Thread Chris Riccomini
Hey all,

(Last time, I hope)^2

I have cut Airflow 1.9.0 RC8. This email is calling a vote on the release,
which will last for 72 hours. Consider this my (binding) +1.

Airflow 1.9.0 RC8 is available at:

https://dist.apache.org/repos/dist/dev/incubator/airflow/1.9.0rc8/

apache-airflow-1.9.0rc8+incubating-source.tar.gz is a source release that
comes with INSTALL instructions.
apache-airflow-1.9.0rc8+incubating-bin.tar.gz is the binary Python "sdist"
release.

Public keys are available at:

https://dist.apache.org/repos/dist/release/incubator/airflow/

The release contains no new JIRAs. Just a version fix.

I also had to change the version number to exclude the `rc6` string as well
as the "+incubating" string, so it's now simply 1.9.0. This will allow us
to rename the artifact without modifying the artifact checksums when we
actually release.

See JIRAs that were in 1.9.0RC7 and before (see previous VOTE email for
full list).

Cheers,
Chris