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

ceph: avoid multiple reconciles of ceph cluster #8638

Merged
merged 2 commits into from
Sep 2, 2021
Merged

ceph: avoid multiple reconciles of ceph cluster #8638

merged 2 commits into from
Sep 2, 2021

Commits on Sep 2, 2021

  1. ceph: avoid double reconcile with default value 

    When setting a default values with kubebuilder flags it seems that the
kube operator intercepts the API call, mutate the CR and inject it.
This results in two events, one for creation and one for edition and we
end up with 2 reconciles. Let's avoid this.

Signed-off-by: Sébastien Han <seb@redhat.com>
    @leseb
    leseb committed Sep 2, 2021
    Configuration menu
    Copy the full SHA
    ab16674 View commit details
    Browse the repository at this point in the history

  2. ceph: do not reconcile if the op is not ready 

    When the op is initializing, the ceph config is not ready and thus we
should reconcile. This avoids a double reconcile.

Signed-off-by: Sébastien Han <seb@redhat.com>
    @leseb
    leseb committed Sep 2, 2021
    Configuration menu
    Copy the full SHA
    072c8f4 View commit details
    Browse the repository at this point in the history