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.

Sign up for GitHub

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

Jump to bottom

Bug 2051457: CCM PodDisruptionBudgets #174

Merged
merged 16 commits into from Apr 14, 2022
Merged

Bug 2051457: CCM PodDisruptionBudgets #174

merged 16 commits into from Apr 14, 2022

Commits on Apr 13, 2022

  1. rename vsphere cred request manifest

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    2a91c11 View commit details
    Browse the repository at this point in the history

  2. Unify labels across deployments and daemonsets

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    4303cd6 View commit details
    Browse the repository at this point in the history

  3. Introduce PodDisruptionBudget apply logic

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    2fa51b8 View commit details
    Browse the repository at this point in the history

  4. Add GetPlatformNameString method to operator config 

    For convinient obtain platfrom name as a string across operator
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    5bee43c View commit details
    Browse the repository at this point in the history

  5. Introduce new well known labels on ccm deployments/daemonsets 

    infrastructure.openshift.io/cloud-controller-manager label contains
platform name, intended to be used as selector for podDisruptionBudgets.

For consistency reasons infrastructure.openshift.io/cloud-node-manager
label was introduced for daemonsets
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    d2f73ea View commit details
    Browse the repository at this point in the history

  6. Add pod disruption budget for every deployment, except single node cl… 

    …usters
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    053c1b1 View commit details
    Browse the repository at this point in the history

  7. Update resourceapply test for using envtest and gomega matcher 

    Expected to fail, respective resourceapply changes for handling Deployment
coming next commit
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    e1f47b7 View commit details
    Browse the repository at this point in the history

  8. Add recreate procedure for deployment and daemonsets in case of chang… 

    …ed selectors

Selectors are immutable for Deployments and DaemonSets,
in case if it was changed, delete old and create new one with prior serverside validation
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    1bf44f1 View commit details
    Browse the repository at this point in the history

  9. Align pdb resourceapply logic with library-go, add respective test

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    d3be146 View commit details
    Browse the repository at this point in the history

  10. add check for deletion timestamp for break possible inf recursion, tests

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    d4b6c3f View commit details
    Browse the repository at this point in the history

  11. misc: inline err handling, change incorrect event descriptions, logging

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    5b17fa2 View commit details
    Browse the repository at this point in the history

  12. remove recursion, some err handling inlining 

    DRY principle does not work well in this world, sorry
    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    a496360 View commit details
    Browse the repository at this point in the history

  13. fix ccm replicas count for azure deployment

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    7993147 View commit details
    Browse the repository at this point in the history

  14. update error messages

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    9429f58 View commit details
    Browse the repository at this point in the history

  15. Fix rebase conflicts, missed powervs tests, adjust powervs assets

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    599c739 View commit details
    Browse the repository at this point in the history

  16. Generate namespace per test-suite for resource-apply tests

    @lobziik
    lobziik committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    7488432 View commit details
    Browse the repository at this point in the history