[velero] Remove bitnami nicely by inverting the original idea

[albundy83]

Special notes for your reviewer:

Try to fix #698

Checklist

• DCO signed
• Chart Version bumped, please refer to the chart version instruction
• Variables are documented in the values.yaml or README.md
• Title of the PR starts with chart name (e.g. [velero])

For moment, it's just an idea to remove bitnami kubectl and the need for sh :)

What do you think ?

[albundy83]

And I have used the kubernetes version way from this pull request here

[albundy83]

Ultimately, I use the work done here for this pull request.

[fcuello-fudo]

@albundy83 do you plan to resume this PR?

[albundy83]

Hello,
yes, once this pull has been merged, we should be able to use directly the velero binary to install crds, see this part in upgrade-crds.yaml

......
          command:
            - /velero
          args:
            - install
            - --crds-only
            - --apply
......

and kubectl binary to clean, see those parts in cleanup-crds.yaml:

......
{{/* Define the list of resources to clean up */}}
{{- $cleanupResources := list "restore" "backup" "backupstoragelocation" "volumesnapshotlocation" "podvolumerestore" -}}
......
          command:
            - /bin/kubectl
          args:
            - delete
            - {{ . }}
            - --all
......
          command:
            - /bin/kubectl
          args:
            - delete
            - crd
            - -l
            - component=velero
......

[jenting]

Is this PR ready to be review? If yes, could you please help address the CI failure? Thanks for your contribution :)

[albundy83]

Hello,
As I already explained, unfortunately I need this pull to be merged.

[aviallon]

@albundy83 the PR just got merged :)

[albundy83]

Hello,
yes I see it, very good news :)
Now let's wait for the release