Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Handle set object retrieved from lookup plugin #118

Merged
merged 1 commit into from Jun 5, 2020

Conversation

Akasurde
Copy link
Member

@Akasurde Akasurde commented Jun 3, 2020

SUMMARY

self.client.resources.api_groups is a dict_keys and is not
handled correctly by default callback plugin while JSON serialization.

This fix will typecast it to list so that it can be JSON serialized for
further processing.

Fixes: #111

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

molecule/default/tasks/full.yml
plugins/lookup/k8s.py

self.client.resources.api_groups is a dict_keys and is not
handled correctly by default callback plugin while JSON serialization.

This fix will typecast it to list so that it can be JSON serialized for
further processing.

Fixes: #111

Signed-off-by: Abhijeet Kasurde <akasurde@redhat.com>
@codecov
Copy link

codecov bot commented Jun 3, 2020

Codecov Report

Merging #118 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #118   +/-   ##
=======================================
  Coverage   42.88%   42.88%           
=======================================
  Files           3        3           
  Lines         541      541           
  Branches      110      110           
=======================================
  Hits          232      232           
  Misses        266      266           
  Partials       43       43           

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 9a95269...b277f85. Read the comment docs.

Copy link
Collaborator

@fabianvf fabianvf left a comment

Choose a reason for hiding this comment

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

Nice catch

@fabianvf
Copy link
Collaborator

fabianvf commented Jun 4, 2020

I probably should fix this in openshift/openshift-restclient-python too

@Akasurde Akasurde merged commit 18dfb68 into ansible-collections:master Jun 5, 2020
@Akasurde Akasurde deleted the i111 branch June 5, 2020 06:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants