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

vrg: remove the use of captureStartConditionally #1532

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Commits on Sep 6, 2024

  1. drpc conditions in wide output

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    f2d9dbf View commit details
    Browse the repository at this point in the history
  2. makefile support for x86_64

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    efbf620 View commit details
    Browse the repository at this point in the history
  3. my env

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    5322b06 View commit details
    Browse the repository at this point in the history
  4. my makefile vars

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    6dbc5c1 View commit details
    Browse the repository at this point in the history
  5. my config for e2e

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    8df829d View commit details
    Browse the repository at this point in the history
  6. tests: keep all suite cleanup functions in suite_test.go

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    f49ca94 View commit details
    Browse the repository at this point in the history
  7. tests: dump the kubeconfig for the testenv server

    This is going to be useful to inspect the state of the cluster after a
    test fails.
    
    Set the env variable SKIP_CLEANUP to "true" or "1" to skip the cleanup
    of the environment to make use of the kubeconfig.
    
    The kubeconfig is dumped in the testbin directory.
    
    After debugging you will have to kill the testEnv. The two binaries that
    are used by the testEnv are etcd and kube-apiserver. Check for the
    running process with a command like
    
    ps aux | grep -e kube-apiserver -e etcd
    
    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    4244b77 View commit details
    Browse the repository at this point in the history
  8. hack: customize script for OS type

    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    59d0ece View commit details
    Browse the repository at this point in the history
  9. drenv: turn off check for broker certs

    On Mac, the check for certs is more strict and it fails for submariner
    service. Turning off the check for certs.
    
    More info: golang/go#51991
    
    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    bc6922d View commit details
    Browse the repository at this point in the history
  10. vrg: remove the use of captureInProgressStatusUpdate

    captureInProgressStatusUpdate was a function pointer that was an empty
    function when the VRG was being reconciled as Primary and a call to
    VRGConditionReasonUploading with status false when reconciling as
    Secondary in the relocate case.
    
    Simplify the code by removing the function pointer.
    
    * Remove the captureInProgressStatusUpdate parameter from
      kubeObjectsProtect
      kubeObjectsCaptureStartOrResumeOrDelay
      kubeObjectsCaptureStartOrResume
      kubeObjectsGroupCapture.
    
    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    5712c85 View commit details
    Browse the repository at this point in the history
  11. vrg: remove the use of captureStartConditionally

    captureStartConditionally was a function pointer that was
    * capturing kube objects one last time when the vrg was secondary
    * capturing kube objects if time since last capture was greater than the
      capture interval
    
    Refactored into one function that performs the capture based on the vrg
    spec.
    
    Signed-off-by: Raghavendra Talur <raghavendra.talur@gmail.com>
    raghavendra-talur committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    36ad6ee View commit details
    Browse the repository at this point in the history