You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2021/08/10 06:53:22 UTC

[airflow] branch main updated: Remove hard-coded container name for trino (#17525)

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new bd0a3d5  Remove hard-coded container name for trino (#17525)
bd0a3d5 is described below

commit bd0a3d555e0175a1e56e84a12e6199f4782f5b41
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Aug 10 08:53:02 2021 +0200

    Remove hard-coded container name for trino (#17525)
    
    This had no negative effect because we only run trino during
    integration tests but it could actually prevent two integration
    tests run in parallel on the same machine if we choose otherwise
    and it will look better in logs:
    
    ```
    NAME
    airflow-integration-mysql_pinot_1
    airflow-integration-mysql_statsd-exporter_1
    airflow-integration-mysql_redis_1
    airflow-integration-mysql_mysql_1
    airflow-integration-mysql_cassandra_1
    airflow-integration-mysql_grafana_1
    airflow-integration-mysql_openldap_1
    trino
    airflow-integration-mysql_kdc-server-example-com_1
    airflow-integration-mysql_mongo_1
    airflow-integration-mysql_rabbitmq_1
    airflow-integration-mysql_airflow_run_908822c9c10f
    ```
---
 scripts/ci/docker-compose/integration-trino.yml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/scripts/ci/docker-compose/integration-trino.yml b/scripts/ci/docker-compose/integration-trino.yml
index e381ced..d6fa6eb 100644
--- a/scripts/ci/docker-compose/integration-trino.yml
+++ b/scripts/ci/docker-compose/integration-trino.yml
@@ -19,7 +19,6 @@ version: "2.2"
 services:
   trino:
     image: ghcr.io/apache/airflow-trino:359-2021.07.04
-    container_name: trino
     hostname: trino
     domainname: example.com