Tendrl / commons

Common code usable by all Tendrl components
http://www.tendrl.org
GNU Lesser General Public License v2.1
4 stars 23 forks source link

Import Cluster fails on "node X doesn't have network details populated" with Ansible 2.8 #1085

Closed GowthamShanmugam closed 5 years ago

GowthamShanmugam commented 5 years ago

bugzilla: 1702412 tendrl-bug-id: Tendrl/commons#1084

Signed-off-by: GowthamShanmugasundaram gshanmug@redhat.com

GowthamShanmugam commented 5 years ago

@shtripat @nthomas-redhat This solution will work in current supporting ansible version as well as ansible 2.8.

codecov[bot] commented 5 years ago

Codecov Report

Merging #1085 into master will increase coverage by 0.02%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1085      +/-   ##
==========================================
+ Coverage   80.79%   80.81%   +0.02%     
==========================================
  Files          91       91              
  Lines        3821     3826       +5     
  Branches      502      502              
==========================================
+ Hits         3087     3092       +5     
  Misses        631      631              
  Partials      103      103
Impacted Files Coverage Δ
tendrl/commons/utils/ansible_module_runner.py 100% <100%> (ø) :arrow_up:

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 d7aae16...0d18864. Read the comment docs.