Skip to content

[AIRFLOW-6402] SSH operator exception - NoneType object has no attribute startswith#6970

Closed
ohadmata wants to merge 3 commits intoapache:v1-10-stablefrom
ohadmata:v1-10-stable
Closed

[AIRFLOW-6402] SSH operator exception - NoneType object has no attribute startswith#6970
ohadmata wants to merge 3 commits intoapache:v1-10-stablefrom
ohadmata:v1-10-stable

Conversation

@ohadmata
Copy link
Contributor

@ohadmata ohadmata commented Dec 30, 2019

@codecov-io
Copy link

codecov-io commented Jan 2, 2020

Codecov Report

Merging #6970 into v1-10-stable will decrease coverage by <.01%.
The diff coverage is 75%.

Impacted file tree graph

@@               Coverage Diff               @@
##           v1-10-stable   #6970      +/-   ##
===============================================
- Coverage          81.5%   81.5%   -0.01%     
===============================================
  Files               526     526              
  Lines             36032   36034       +2     
===============================================
+ Hits              29368   29369       +1     
- Misses             6664    6665       +1
Impacted Files Coverage Δ
airflow/utils/log/logging_mixin.py 95.4% <100%> (-1.15%) ⬇️
airflow/contrib/operators/ssh_operator.py 84.14% <66.66%> (-0.86%) ⬇️
airflow/operators/postgres_operator.py 100% <0%> (ø) ⬆️
airflow/operators/mysql_operator.py 100% <0%> (ø) ⬆️
airflow/operators/mysql_to_hive.py 100% <0%> (ø) ⬆️
airflow/operators/generic_transfer.py 100% <0%> (ø) ⬆️
airflow/hooks/hdfs_hook.py 92.5% <0%> (ø) ⬆️
airflow/executors/celery_executor.py 88.88% <0%> (ø) ⬆️
airflow/utils/sqlalchemy.py 96.82% <0%> (ø) ⬆️
airflow/utils/log/wasb_task_handler.py 42.46% <0%> (ø) ⬆️
... and 19 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8597cb...17effb3. Read the comment docs.

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there are more commits than needed here? Also It would be great if you follow our new beautiful shiny template we have when creating the PR.

Propagate message removing escape codes.
"""
self.logger.log(self.level, remove_escape_codes(message))
# self.logger.log(self.level, remove_escape_codes(message))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's an overlook?

if(auto_tailing && checkAutoTailingCondition()) {
var should_scroll = true
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated change?

@ohadmata
Copy link
Contributor Author

ohadmata commented Jan 2, 2020

I made a mistake here. sorry. will fix that in another pull request

@ohadmata ohadmata closed this Jan 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments