Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

airflow-provider-vdk: Job execution status and logs method #796

Merged
merged 5 commits into from
Apr 11, 2022

Conversation

gabrielgeorgiev1
Copy link
Contributor

@gabrielgeorgiev1 gabrielgeorgiev1 commented Apr 9, 2022

This change implements a method for the VDK connection hook which
allows a user to check the status of a job execution in the VDK
control-service. It also implements a method which allows a user to
download the stored job execution logs for a particular execution.

Testing done: locally tested and verified that the correct object
and correct log string are returned, implemented 2 unit tests

Signed-off-by: Gabriel Georgiev [email protected]

This change implements a method for the VDK connection hook which
allows a user to check the status of a job execution in the VDK
control-service.

Testing done: locally tested and verified that the correct object
is returned, unit test

Signed-off-by: Gabriel Georgiev <[email protected]>
Signed-off-by: Gabriel Georgiev <[email protected]>
Signed-off-by: Gabriel Georgiev <[email protected]>
Signed-off-by: Gabriel Georgiev <[email protected]>
@gabrielgeorgiev1 gabrielgeorgiev1 changed the title airflow-provider-vdk: Job execution status method airflow-provider-vdk: Job execution status and logs method Apr 10, 2022
Signed-off-by: Gabriel Georgiev <[email protected]>
@gabrielgeorgiev1 gabrielgeorgiev1 enabled auto-merge (squash) April 11, 2022 09:51
@gabrielgeorgiev1 gabrielgeorgiev1 merged commit 940b28a into main Apr 11, 2022
@gabrielgeorgiev1 gabrielgeorgiev1 deleted the person/gageorgiev/airflow-log-status branch April 11, 2022 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants