airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <j...@apache.org>
Subject [jira] [Commented] (AIRFLOW-5829) Get rid of checklicence image
Date Wed, 06 Nov 2019 09:32:00 GMT

    [ https://issues.apache.org/jira/browse/AIRFLOW-5829?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16968218#comment-16968218
] 

ASF subversion and git services commented on AIRFLOW-5829:
----------------------------------------------------------

Commit d4ff52974373a6b4bf423917b8880c2492c51c0c in airflow's branch refs/heads/master from
Jarek Potiuk
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=d4ff529 ]

[AIRFLOW-5829] Get rid of the checklicence image (#6495)

This change is a further step of simplifying the set of scripts
used by CI. The separate checklicence image was implemented as an
optimisation of the licence check time. The image to download was
small and could be downloaded slightly faster in CI. However that
made all the management script more complex and lead to having
separate jobs for check licence and static checks. That lead to
actually longer time of Travis jobs - because new machine had to
be spun-off for checklicence check only.

With this change, the CI image is the only one left and it is slightly
bigger (with RAT tool added) but the same image is used for all the
tests - unit tests, static checks and checklicence checks.

This also makes it easier to manage the images and decreases update
overhead on the developers using Breeze.

> Get rid of checklicence image
> -----------------------------
>
>                 Key: AIRFLOW-5829
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-5829
>             Project: Apache Airflow
>          Issue Type: Sub-task
>          Components: ci
>    Affects Versions: 2.0.0, 1.10.5
>            Reporter: Jarek Potiuk
>            Assignee: Jarek Potiuk
>            Priority: Major
>
> We can get rid of checklicence image and run checklicence checks using the standard CI
image



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message