Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

k8s loop flattening for template argument #49

Merged
merged 20 commits into from
Apr 22, 2021

Conversation

abikouo
Copy link
Contributor

@abikouo abikouo commented Apr 8, 2021

SUMMARY

k8s loop flattening

https://github.com/ansible-collections/community.kubernetes/pull/369
https://github.com/ansible-collections/community.kubernetes/pull/321

Closes: #19

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

template argument can now contains list of item (string or file path)
continue_on_error argument is introduced, used with template when multiple resource as defined, defines whether to continue or stop (default behavior) when creation/deletion error occurred.


abikouo and others added 2 commits April 8, 2021 17:25
@codecov
Copy link

codecov bot commented Apr 8, 2021

Codecov Report

Merging #49 (0ceef69) into main (edc48ee) will increase coverage by 0.84%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
+ Coverage   23.17%   24.02%   +0.84%     
==========================================
  Files           1        1              
  Lines         151      154       +3     
  Branches       24       29       +5     
==========================================
+ Hits           35       37       +2     
  Misses        111      111              
- Partials        5        6       +1     
Impacted Files Coverage Δ
...ections/kubernetes/core/plugins/action/k8s_info.py 24.02% <0.00%> (+0.84%) ⬆️

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 edc48ee...0ceef69. Read the comment docs.

@abikouo abikouo requested review from Akasurde and gravesm April 9, 2021 08:26
Copy link
Member

@gravesm gravesm left a comment

Choose a reason for hiding this comment

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

A few small requests, but this looks good. Nice work.

plugins/modules/k8s.py Outdated Show resolved Hide resolved
@gravesm
Copy link
Member

gravesm commented Apr 13, 2021

@Akasurde @fabianvf can you take a look at this?

@goneri goneri self-requested a review April 16, 2021 14:26
@goneri
Copy link
Member

goneri commented Apr 16, 2021

Thank you for the patch. @abikouo, do you observe a performance improvement with the patch?

@abikouo
Copy link
Contributor Author

abikouo commented Apr 16, 2021

Thank you for the patch. @abikouo, do you observe a performance improvement with the patch?

@goneri I did not really take the time to benchmark it, but it seems better that the previous way of using api

plugins/action/k8s_info.py Outdated Show resolved Hide resolved
plugins/action/k8s_info.py Outdated Show resolved Hide resolved
plugins/action/k8s_info.py Outdated Show resolved Hide resolved
@abikouo abikouo requested a review from goneri April 20, 2021 12:52
molecule/default/tasks/template.yml Outdated Show resolved Hide resolved
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.

Implement "loop flattening" in k8s
3 participants