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

k8s_info now outputs whether the api was found #308

Merged
merged 5 commits into from
Nov 30, 2020

Conversation

fabianvf
Copy link
Collaborator

SUMMARY

Previously, an empty list of resources could mean two things:

  1. There were no instances of resources matching the api_version and
    kind found in the cluster.
  2. There was no API found that matched the provided api_version and
    kind.

This adds a new field to the k8s_info output, api_found, which is a
boolean that allows this ambiguity to be resolved by the user without
additional steps.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

k8s_info

ADDITIONAL INFORMATION

@codecov
Copy link

codecov bot commented Nov 18, 2020

Codecov Report

Merging #308 (2e7a8e0) into main (b97c5ce) will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
- Coverage   37.00%   36.95%   -0.06%     
==========================================
  Files           3        3              
  Lines         727      728       +1     
  Branches      144      144              
==========================================
  Hits          269      269              
- Misses        409      410       +1     
  Partials       49       49              
Impacted Files Coverage Δ
...ommunity/kubernetes/plugins/module_utils/common.py 35.85% <0.00%> (-0.07%) ⬇️

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 b97c5ce...2e7a8e0. Read the comment docs.

Previously, an empty list of resources could mean two things:
1. There were no instances of resources matching the api_version and
   kind found in the cluster.
2. There was no API found that matched the provided api_version and
   kind.

This adds a new field to the k8s_info output, `api_found`, which is a
boolean that allows this ambiguity to be resolved by the user without
additional steps.
@fabianvf fabianvf force-pushed the return-api-discovered branch from 864da95 to c32a14c Compare November 18, 2020 17:32
@tima tima self-requested a review November 19, 2020 18:28
Copy link
Collaborator

@tima tima left a comment

Choose a reason for hiding this comment

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

/lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants