You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by mo...@apache.org on 2023/08/04 12:56:01 UTC
[airflow] branch fix-openlineage-tests updated (9b790301f7 -> 66e20ab01a)
This is an automated email from the ASF dual-hosted git repository.
mobuchowski pushed a change to branch fix-openlineage-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
omit 9b790301f7 openlineage: remove eager test for auth headers, fix example dag
new 66e20ab01a openlineage: remove eager test for auth headers, fix example dag
This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version. This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:
* -- * -- B -- O -- O -- O (9b790301f7)
\
N -- N -- N refs/heads/fix-openlineage-tests (66e20ab01a)
You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.
Any revisions marked "omit" are not gone; other references still
refer to them. Any revisions marked "discard" are gone forever.
The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
tests/providers/openlineage/plugins/test_openlineage_adapter.py | 1 -
1 file changed, 1 deletion(-)
[airflow] 01/01: openlineage: remove eager test for auth headers, fix example dag
Posted by mo...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
mobuchowski pushed a commit to branch fix-openlineage-tests
in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 66e20ab01ae95ea1600e1e2ec6f027a617b5d523
Author: Maciej Obuchowski <ob...@gmail.com>
AuthorDate: Thu Aug 3 23:10:01 2023 +0200
openlineage: remove eager test for auth headers, fix example dag
Signed-off-by: Maciej Obuchowski <ob...@gmail.com>
---
airflow/example_dags/plugins/event_listener.py | 6 +++---
tests/providers/openlineage/plugins/test_openlineage_adapter.py | 4 ----
2 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/airflow/example_dags/plugins/event_listener.py b/airflow/example_dags/plugins/event_listener.py
index 2e2d01800b..16848c9ebc 100644
--- a/airflow/example_dags/plugins/event_listener.py
+++ b/airflow/example_dags/plugins/event_listener.py
@@ -111,7 +111,7 @@ def on_task_instance_failed(previous_state: TaskInstanceState, task_instance: Ta
# [START howto_listen_dagrun_success_task]
@hookimpl
-def on_dag_run_success(dag_run: DagRun, message: str):
+def on_dag_run_success(dag_run: DagRun, msg: str):
"""
This method is called when dag run state changes to SUCCESS.
"""
@@ -126,7 +126,7 @@ def on_dag_run_success(dag_run: DagRun, message: str):
# [START howto_listen_dagrun_failure_task]
@hookimpl
-def on_dag_run_failed(dag_run: DagRun, message: str):
+def on_dag_run_failed(dag_run: DagRun, msg: str):
"""
This method is called when dag run state changes to FAILED.
"""
@@ -142,7 +142,7 @@ def on_dag_run_failed(dag_run: DagRun, message: str):
# [START howto_listen_dagrun_running_task]
@hookimpl
-def on_dag_run_running(dag_run: DagRun, message: str):
+def on_dag_run_running(dag_run: DagRun, msg: str):
"""
This method is called when dag run state changes to RUNNING.
"""
diff --git a/tests/providers/openlineage/plugins/test_openlineage_adapter.py b/tests/providers/openlineage/plugins/test_openlineage_adapter.py
index f81974a93e..cec69e0662 100644
--- a/tests/providers/openlineage/plugins/test_openlineage_adapter.py
+++ b/tests/providers/openlineage/plugins/test_openlineage_adapter.py
@@ -42,8 +42,6 @@ def test_create_client_from_ol_env():
client = OpenLineageAdapter().get_or_create_openlineage_client()
assert client.transport.url == "http://ol-api:5000"
- assert "Authorization" in client.transport.session.headers
- assert client.transport.session.headers["Authorization"] == "Bearer api-key"
@conf_vars(
@@ -57,8 +55,6 @@ def test_create_client_from_config_with_options():
assert client.transport.kind == "http"
assert client.transport.url == "http://ol-api:5000"
- assert "Authorization" in client.transport.session.headers
- assert client.transport.session.headers["Authorization"] == "Bearer api-key"
@conf_vars(