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

Wait conditions #35

Merged
merged 1 commit into from May 19, 2020
Merged

Conversation

willthames
Copy link
Collaborator

@willthames willthames commented Feb 21, 2020

SUMMARY

Improve k8s Deployment and Daemonset wait conditions

Ensure that Deployments and Daemonsets properly await
all replicas to be available. Correctly handles the
subtle edge case when a service account no longer exists.

Note that this will dramatically slow Daemonset updates

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

k8s

@willthames
Copy link
Collaborator Author

Tests failing would be fixed by #34

@codecov-io
Copy link

codecov-io commented Feb 21, 2020

Codecov Report

Merging #35 into master will decrease coverage by 0.10%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #35      +/-   ##
==========================================
- Coverage   43.33%   43.22%   -0.11%     
==========================================
  Files           3        3              
  Lines         540      539       -1     
  Branches      109      109              
==========================================
- Hits          234      233       -1     
  Misses        263      263              
  Partials       43       43              
Impacted Files Coverage Δ
...ctions/community/kubernetes/plugins/modules/k8s.py 90.00% <0.00%> (-0.91%) ⬇️
...s/community/kubernetes/plugins/module_utils/raw.py 39.16% <0.00%> (ø)

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 5f3d2fc...35ffd0e. 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.

Few comments, nothing blocking

molecule/default/tasks/apply.yml Outdated Show resolved Hide resolved
plugins/module_utils/scale.py Show resolved Hide resolved
plugins/module_utils/scale.py Show resolved Hide resolved
Ensure that Deployments and Daemonsets properly await
all replicas to be available. Correctly handles the
subtle edge case when a service account no longer exists.

Note that this will dramatically slow Daemonset updates
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.

lgtm

@fabianvf fabianvf requested a review from geerlingguy May 14, 2020 23:35
@willthames willthames mentioned this pull request May 16, 2020
Copy link
Collaborator

@geerlingguy geerlingguy left a comment

Choose a reason for hiding this comment

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

LGTM.

@fabianvf fabianvf merged commit 2f7b552 into ansible-collections:master May 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants