Mailing-List: contact dev-help@airflow.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@airflow.incubator.apache.org
From: Bolke de Bruin <bdbruin@gmail.com>
Content-Type: text/plain;
	charset=utf-8
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (1.0)
Date: Wed, 22 Feb 2017 23:00:26 +0100
Subject: Re: [RESULT] [VOTE] Release Airflow 1.8.0 based on Airflow 1.8.0rc4
Message-Id: <50A8DA3D-B513-4898-B9E5-9B982787743B@gmail.com>
References: <0C96110D-2074-468A-BCAC-A7FEEE331EF8@gmail.com> <BF8C1D7F-17F3-447B-9F2C-B06288E11DD2@gmail.com> <CABYbY7c5-r4bco_QAL=ePdC18euRpXvKROdQNhkSC9-r19LC7g@mail.gmail.com> <CABekQG6xiex8c6M5ZMmYOckyZ8Dy9d0_TvJKQR7DGpb9OY-CBQ@mail.gmail.com> <CABekQG4OMFKgXzaKnON+zYJAZGYNiM=pvTJf8k-DFmmuAH6L+A@mail.gmail.com>
In-Reply-To: <CABekQG4OMFKgXzaKnON+zYJAZGYNiM=pvTJf8k-DFmmuAH6L+A@mail.gmail.com>
To: dev@airflow.incubator.apache.org
archived-at: Wed, 22 Feb 2017 22:00:35 -0000

So it is a database integrity issue? Afaik a start_date should always be set=
 for a DagRun (create_dagrun) does so  I didn't check the code though.=20

Sent from my iPhone

> On 22 Feb 2017, at 22:19, Dan Davydov <dan.davydov@airbnb.com.INVALID> wro=
te:
>=20
> Should clarify this occurs when a dagrun does not have a start date, not a=

> dag (which makes it even less likely to happen). I don't think this is a
> blocker for releasing.
>=20
>> On Wed, Feb 22, 2017 at 1:15 PM, Dan Davydov <dan.davydov@airbnb.com> wro=
te:
>>=20
>> I rolled this out in our prod and the webservers failed to load due to
>> this commit:
>>=20
>> [AIRFLOW-510] Filter Paused Dags, show Last Run & Trigger Dag
>> 7c94d81c390881643f94d5e3d7d6fb351a445b72
>>=20
>> This fixed it:
>> -                            </a> <span id=3D"statuses_info"
>> class=3D"glyphicon glyphicon-info-sign" aria-hidden=3D"true" title=3D"Sta=
rt Date:
>> {{last_run.start_date.strftime('%Y-%m-%d %H:%M')}}"></span>
>> +                            </a> <span id=3D"statuses_info"
>> class=3D"glyphicon glyphicon-info-sign" aria-hidden=3D"true"></span>
>>=20
>> This is caused by assuming that all DAGs have start dates set, so a broke=
n
>> DAG will take down the whole UI. Not sure if we want to make this a block=
er
>> for the release or not, I'm guessing for most deployments this would occu=
r
>> pretty rarely. I'll submit a PR to fix it soon.
>>=20
>>=20
>>=20
>> On Tue, Feb 21, 2017 at 9:49 AM, Chris Riccomini <criccomini@apache.org>
>> wrote:
>>=20
>>> Ack that the vote has already passed, but belated +1 (binding)
>>>=20
>>> On Tue, Feb 21, 2017 at 7:42 AM, Bolke de Bruin <bdbruin@gmail.com>
>>> wrote:
>>>=20
>>>> IPMC Voting can be found here:
>>>>=20
>>>> http://mail-archives.apache.org/mod_mbox/incubator-general/
>>> 201702.mbox/%
>>>> 3c676BDC9F-1B55-4469-92A7-9FF309AD0EC8@gmail.com%3e <
>>>> http://mail-archives.apache.org/mod_mbox/incubator-general/
>>> 201702.mbox/%
>>>> 3C676BDC9F-1B55-4469-92A7-9FF309AD0EC8@gmail.com%3E>
>>>>=20
>>>> Kind regards,
>>>> Bolke
>>>>=20
>>>>> On 21 Feb 2017, at 08:20, Bolke de Bruin <bdbruin@gmail.com> wrote:
>>>>>=20
>>>>> Hello,
>>>>>=20
>>>>> Apache Airflow (incubating) 1.8.0 (based on RC4) has been accepted.
>>>>>=20
>>>>> 9 =E2=80=9C+1=E2=80=9D votes received:
>>>>>=20
>>>>> - Maxime Beauchemin (binding)
>>>>> - Arthur Wiedmer (binding)
>>>>> - Dan Davydov (binding)
>>>>> - Jeremiah Lowin (binding)
>>>>> - Siddharth Anand (binding)
>>>>> - Alex van Boxel (binding)
>>>>> - Bolke de Bruin (binding)
>>>>>=20
>>>>> - Jayesh Senjaliya (non-binding)
>>>>> - Yi (non-binding)
>>>>>=20
>>>>> Vote thread (start):
>>>>> http://mail-archives.apache.org/mod_mbox/incubator-
>>>> airflow-dev/201702.mbox/%3cD360D9BE-C358-42A1-9188-
>>>> 6C92C31A2F8B@gmail.com%3e <http://mail-archives.apache.
>>>> org/mod_mbox/incubator-airflow-dev/201702.mbox/%3C7EB7B6D6-
>>> 092E-48D2-AA0F-
>>>> 15F44376A8FF@gmail.com%3E>
>>>>>=20
>>>>> Next steps:
>>>>> 1) will start the voting process at the IPMC mailinglist. I do expect
>>>> some changes to be required mostly in documentation maybe a license her=
e
>>>> and there. So, we might end up with changes to stable. As long as these=

>>> are
>>>> not (significant) code changes I will not re-raise the vote.
>>>>> 2) Only after the positive voting on the IPMC and finalisation I will
>>>> rebrand the RC to Release.
>>>>> 3) I will upload it to the incubator release page, then the tar ball
>>>> needs to propagate to the mirrors.
>>>>> 4) Update the website (can someone volunteer please?)
>>>>> 5) Finally, I will ask Maxime to upload it to pypi. It seems we can
>>> keep
>>>> the apache branding as lib cloud is doing this as well (
>>>> https://libcloud.apache.org/downloads.html#pypi-package <
>>>> https://libcloud.apache.org/downloads.html#pypi-package>).
>>>>>=20
>>>>> Jippie!
>>>>>=20
>>>>> Bolke
>>>>=20
>>>>=20
>>>=20
>>=20
>>=20