Skip to content

Commit

Permalink
Remove missing extras in Airflow 2.2 check (#25738)
Browse files Browse the repository at this point in the history
The Airflow 2.2 check install airflow with predefined set of extras,
but some of those extras were missing in 2.2 because the providers
were not existing yet then. There was also a 'dot' instead of coma
in the list of extras between apache.beam and apache.atlas.

It did not have too bad effect, because those dependencies are
pulled in when the providers get installed, but it did not correctly
test the apache.atlas and apache.beam upgrade scenarios - from the
versions that were installed in 2.2 to the latest version in main.

This PR fixes it.
  • Loading branch information
potiuk authored Aug 16, 2022
1 parent bc46477 commit 9033839
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -848,10 +848,10 @@ ${{ hashFiles('.pre-commit-config.yaml') }}"
# The extras below are all extras that should be installed with Airflow 2.2.0
AIRFLOW_EXTRAS: "airbyte,alibaba,amazon,apache.atlas.apache.beam,apache.cassandra,apache.drill,\
apache.druid,apache.hdfs,apache.hive,apache.kylin,apache.livy,apache.pig,apache.pinot,\
apache.spark,apache.sqoop,apache.webhdfs,arangodb,asana,async,\
celery,cgroups,cloudant,cncf.kubernetes,dask,databricks,datadog,dbt.cloud,\
apache.spark,apache.sqoop,apache.webhdfs,asana,async,\
celery,cgroups,cloudant,cncf.kubernetes,dask,databricks,datadog,\
deprecated_api,dingding,discord,docker,\
elasticsearch,exasol,facebook,ftp,github,github_enterprise,google,google_auth,\
elasticsearch,exasol,facebook,ftp,github_enterprise,google,google_auth,\
grpc,hashicorp,http,imap,influxdb,jdbc,jenkins,jira,kerberos,ldap,\
leveldb,microsoft.azure,microsoft.mssql,microsoft.psrp,microsoft.winrm,mongo,mysql,\
neo4j,odbc,openfaas,opsgenie,oracle,pagerduty,pandas,papermill,password,plexus,\
Expand Down

0 comments on commit 9033839

Please sign in to comment.