Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Fixes #250 by defining the FQCN connection plugin name #252

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

tima
Copy link
Collaborator

@tima tima commented Oct 7, 2020

SUMMARY

Fixes #250 by defining the FQCN connection plugin name to force using the one included in this collection

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugin/inventory/k8s.py

…name to force using the one included in this collection
@tima tima added has_issue This PR has a related issue it could close. priority/high type/bug Something isn't working labels Oct 7, 2020
@codecov
Copy link

codecov bot commented Oct 7, 2020

Codecov Report

❗ No coverage uploaded for pull request base (main@0377a89). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #252   +/-   ##
=======================================
  Coverage        ?   36.74%           
=======================================
  Files           ?        3           
  Lines           ?      724           
  Branches        ?      144           
=======================================
  Hits            ?      266           
  Misses          ?      409           
  Partials        ?       49           

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 0377a89...af0d026. Read the comment docs.

@tima tima merged commit b7ec6bf into ansible-collections:main Oct 7, 2020
@tima tima deleted the fix-250 branch October 7, 2020 19:58
@geerlingguy
Copy link
Collaborator

Just posting for reference, it looks like the bug was introduced in #208

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has_issue This PR has a related issue it could close. priority/high type/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI failing on downstream Ansible 2.9 test during inventory tests
3 participants