[AIRFLOW-7044] Add host_key option to SSH connection extras#7714
Closed
aaronfowles wants to merge 97 commits intoapache:masterfrom
aaronfowles:AIRFLOW-7044_host_key
Closed
[AIRFLOW-7044] Add host_key option to SSH connection extras#7714aaronfowles wants to merge 97 commits intoapache:masterfrom aaronfowles:AIRFLOW-7044_host_key
aaronfowles wants to merge 97 commits intoapache:masterfrom
aaronfowles:AIRFLOW-7044_host_key
Conversation
* DAG serialization improvement: the DAG's source code is now stored in the dag_code table and is queried from here when the Code view is opened for the DAG. The webserver no longer needs access to the dags folder in the shared filesystem.
…7678) Each stage of the CI tests needs to pull our `ci` image. By removing java from it we can save 1-2minutes from each test stage. This is part of that work.
The original fix from the commit b4215f6 was wrong. It converted to utc initially but then make_naive had used TIMEZONE and converted it to the lcoal timezone rather than UTC.
Pausing and unpausing dags in the dags view is asynchronous, and there is currently no indication to the user if the operation fails. This patch updates the paused input and highlights it in red when pausing or unpausing fails.
added 23 commits
March 16, 2020 13:06
…LOW-7044_host_key
Codecov Report
@@ Coverage Diff @@
## master #7714 +/- ##
=======================================
Coverage 86.17% 86.17%
=======================================
Files 915 915
Lines 44196 44196
=======================================
Hits 38085 38085
Misses 6111 6111 Continue to review full report at Codecov.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
16 participants
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 PR adds a new option in the SSH connection extras where you can specify base64 ssh-rsa public key of a host. The SSHHook constructor will then add this key to ~/.ssh/known_hosts if not present.
NB - this is my first time submitting to this project.
Issue link: AIRFLOW-7044
Make sure to mark the boxes below before creating PR: [x]
[AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID** For document-only changes commit message can start with
[AIRFLOW-XXXX].In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.