Add DAG repo to airflow submission response messages #2103
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request updates the submission response message to include the GitHub URL that contains the uploaded Airflow DAG. This addition only occurs if the response object contains a
git_url
attribute.Note: this logic will need to be revisited once we decouple the platform runtimes from the front-end (as part of #2100), at which time I suspect the response object will also include the message text appropriate to the platform runtime.
How was this pull request tested?
The current automation tests don't cover submission, so this was manually verified using an appropriate pipeline and produced this output:
Resolves #2101
Developer's Certificate of Origin 1.1