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

feat(sdk): Enable setting OwnerReference on ResourceOps. Fixes #1779 #4831

Merged

Commits on Mar 12, 2021

  1. feat(sdk): Enable setting OwnerReference on ResourceOps. Fixes kubefl…

    …ow#1779
    
    Argo supports a field in the ResourceTemplate that makes the controller
    add an owner reference of the workflow to the created resource since
    v2.4.0 [1].
    With the upgrade of Argo client [2] and deployment [3] we are now able
    to exploit it.
    
    We set it to 'false' by default on all ResourceOps (actually, leave it
    empty).
    
    Setting the field to 'true' for VolumeOps allows the garbage collection
    of PVCs upon workflow cleanup [4].
    
    [1] https://github.com/argoproj/argo/blob/v2.4.0/pkg/apis/workflow/v1alpha1/workflow_types.go#L1044-L1045
    [2] kubeflow#4498
    [3] kubeflow#3537
    [4] kubeflow#1779
    
    Signed-off-by: Ilias Katsakioris <elikatsis@arrikto.com>
    elikatsis committed Mar 12, 2021
    Configuration menu
    Copy the full SHA
    9ba27af View commit details
    Browse the repository at this point in the history