Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-18 Thread Chris Douglas
+1 (binding) Verified checksum/signature; LICENSE/NOTICE/DISCLAIMER look good. -C On Thu, Mar 16, 2017 at 9:32 AM, Bolke de Bruin wrote: > Hello Incubator PMC’ers, > > The Apache Airflow community has voted and approved the proposal to release > Apache Airflow 1.8.0 (incubating) based on 1.8.0

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-17 Thread Jitendra Pandey
Downloaded, verified signatures Built and installed using setup.py locally. NOTICE/DISCLAIMER/LICENSE look ok. +1 (binding) On 3/17/17, 12:39 PM, "Bolke de Bruin" wrote: >Thanks! I have created AIRFLOW-1000 >(https://issues.apache.org/jira/browse/AIRFLOW-1000) to track this. > >We have 2 +1 (

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-17 Thread Bolke de Bruin
Thanks! I have created AIRFLOW-1000 (https://issues.apache.org/jira/browse/AIRFLOW-1000) to track this. We have 2 +1 (binding) votes. Can someone be so kind and do another review and by that reach eternal glory by becoming part of Airflow’s first release under the Apache umbrella? Cheers! Bolk

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread John D. Ament
Looks good. Here's my +1 As I've said before, the package name doesn't block the release. If you can create a JIRA to track it, that would be great. On Thu, Mar 16, 2017 at 9:02 PM Bolke de Bruin wrote: > Sure. Done > > On distribution naming and deprecation can we do this for the next releas

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Bolke de Bruin
Sure. Done On distribution naming and deprecation can we do this for the next release as it requires source code changes? - Bolke > On 16 Mar 2017, at 16:53, John D. Ament wrote: > > Oh ok I see, thanks for the point. Can you delete everything in > https://dist.apache.org/repos/dist/dev/in

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread John D. Ament
Oh ok I see, thanks for the point. Can you delete everything in https://dist.apache.org/repos/dist/dev/incubator/airflow/ that isn't the release under vote? I'll re-review at this point. For the distribution, is it possible for you to publish both apache-airflow and just airflow, and somehow mar

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Arthur Wiedmer
Hi John, 1. The release candidate is 1.8.0rc5, is there a particular naming convention we should be using there? I will let Bolke comment on 2., but usually the Python pip package manager is sensitive to the naming of the archive, unfortunately. Best, Arthur. On Thu, Mar 16, 2017 at 4:38 PM, Jo

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Bolke de Bruin
Hey John, You seem to be unpacking BETA 1, instead of RELEASE CANDIDATE 5? https://dist.apache.org/repos/dist/dev/incubator/airflow/airflow-1.8.0rc5+apache.incubating.tar.gz On #2, people will in general use “pip install ” which is a package manager. Cheers Bolke > On 16 Mar 2017, at 16:38,

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread John D. Ament
Bolke, 1. Where do you see the DISCLAIMER? It should be in the root, but I see it no where in the archive. ~/Desktop/airflow-1.8.0b1+apache.incubating ☺ ls MANIFEST.in PKG-INFO airflow airflow.egg-info setup.cfgsetup.py tests ~/Desktop/airflow-1.8.0b1+ap

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Bolke de Bruin
Hi John, 1. I checked the tar ball and it does contain the DISCLAIMER, is it expected to be somewhere else? 2. Is the naming a hard requirement and a blocker for this release? The reason for naming it this way, it allows for upgrading from earlier versions of Airflow that were managed outside

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread John D. Ament
-1, release is missing necessary DISCLAIMER file. Also to the point that Jakob noted, the current artifact naming must be fixed (its a branding issue from my POV). It should be apache-airflow (or something like that) not simply airflow. John On Thu, Mar 16, 2017 at 12:40 PM Bolke de Bruin wrot

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread siddharth anand
+1 (non-binding) On Thu, Mar 16, 2017 at 12:46 PM, Jakob Homan wrote: > +1 binding. > > md5/asc/LICENSE/NOTICE/DISCLAIMER/headers all look good. > > NITs for next release: > * setup.cfg still has Maxime listed as author and email. > * usually the artifacts are named apache-whatever, rather than

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Jakob Homan
+1 binding. md5/asc/LICENSE/NOTICE/DISCLAIMER/headers all look good. NITs for next release: * setup.cfg still has Maxime listed as author and email. * usually the artifacts are named apache-whatever, rather than whatever-apache. Thanks, Jakob On 16 March 2017 at 10:46, Maxime Beauchemin wrote

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Maxime Beauchemin
+1 (non-binding) On Thu, Mar 16, 2017 at 9:59 AM, Jeremiah Lowin wrote: > +1 (non-binding) > > > On Thu, Mar 16, 2017 at 12:56 PM Arthur Wiedmer wrote: > > > +1 (non-binding) > > > > Best, > > Arthur > > > > On Thu, Mar 16, 2017 at 9:42 AM, Chris Riccomini > > wrote: > > > > > +1 (non-binding)

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Jeremiah Lowin
+1 (non-binding) On Thu, Mar 16, 2017 at 12:56 PM Arthur Wiedmer wrote: > +1 (non-binding) > > Best, > Arthur > > On Thu, Mar 16, 2017 at 9:42 AM, Chris Riccomini > wrote: > > > +1 (non-binding) > > > > On Thu, Mar 16, 2017 at 9:32 AM, Bolke de Bruin > wrote: > > > > > Hello Incubator PMC’ers

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Arthur Wiedmer
+1 (non-binding) Best, Arthur On Thu, Mar 16, 2017 at 9:42 AM, Chris Riccomini wrote: > +1 (non-binding) > > On Thu, Mar 16, 2017 at 9:32 AM, Bolke de Bruin wrote: > > > Hello Incubator PMC’ers, > > > > The Apache Airflow community has voted and approved the proposal to > > release Apache Airf

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-03-16 Thread Chris Riccomini
+1 (non-binding) On Thu, Mar 16, 2017 at 9:32 AM, Bolke de Bruin wrote: > Hello Incubator PMC’ers, > > The Apache Airflow community has voted and approved the proposal to > release Apache Airflow 1.8.0 (incubating) based on 1.8.0 Release Candidate > 5. We now kindly request the Incubator PMC mem

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-02-24 Thread Bolke de Bruin
Hi Jakob et al., Thanks for voting, based on your feedback we cancelled the vote to address the issues. The patch for this has been merged https://github.com/apache/incubator-airflow/pull/2098 (AIRFLOW-895). I am wondering how to deliver

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-02-23 Thread Jakob Homan
(Reposting from the Juneau thread, where I accidentally posted before)... * md5 and sig looks good * Spot check on licensing looks good * The PKG-INFO, setup.cfg, and setup.py refer to Maxime as the author and list his email address. This is incorrect for an Apache release. * Missing NOTICE, DISC

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-02-22 Thread Arthur Wiedmer
+1 non-binding On Wed, Feb 22, 2017 at 2:38 PM, Chris Riccomini wrote: > +1 non-binding > > On Mon, Feb 20, 2017 at 11:50 PM, Bolke de Bruin > wrote: > > > Hello Incubator PMC’ers, > > > > The Apache Airflow community has voted and approved the proposal to > > release Apache Airflow 1.8.0 (incu

Re: [VOTE] Release Apache Airflow 1.8.0 (incubating)

2017-02-22 Thread Chris Riccomini
+1 non-binding On Mon, Feb 20, 2017 at 11:50 PM, Bolke de Bruin wrote: > Hello Incubator PMC’ers, > > The Apache Airflow community has voted and approved the proposal to > release Apache Airflow 1.8.0 (incubating) based on 1.8.0 Release Candidate > 4. We now kindly request the Incubator PMC memb