feat: make helm hooks mandatory

[jbouchery]

Description

I saw that this PR added the Helm Hooks to the migration job #70. I think it could be better to make them mandatory instead of default in the values (so it can't be overridden). In addition, i deleted the initContainer that is deprecated when using Helm Hooks.

References

The PR introducing the migration Helm Hooks #70

Review Checklist

• I have clicked on "allow edits by maintainers".
• I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
• The correct base branch is being used, if not main
• I have added tests to validate that the change in functionality is working as expected

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: jbouchery (86caad5)

[rhamzeh]

Thanks for the PR @jbouchery!

While the team reviews the PR, may I ask you to:

• Sign the Linux Foundation's EasyCLA? As a Cloud Native Computing Sandbox project, we cannot merge PRs that haven't signed the CLA.
• Resolve the merge conflicts - There seem to be some merge conflicts around your PR (Mostly the app version. You may have to bump this again if another PR is merged before yours)

[jbouchery]

Hi @rhamzeh,

Thanks for the feedback !

I Signed the Linux Foundation's EasyCLA as you said. I also reviewed the merge conflicts but my PR only involve a change of the Helm Chart, not the application. I don't think i have to bump de appVersion in this case.

[jon-whit]

I understand the other changes in this PR, but I don't understand why you dropped this. Could you elaborate on why you removed this?

[rhamzeh]

pinging @jbouchery :)

[supercairos]

Please don't male the hooks mandatory as it seams to be breaking on ArgoCD deployement...

I had to do this :

  migrate:
    annotations:
      helm.sh/hook: null
      helm.sh/hook-weight: null
      helm.sh/hook-delete-policy: null 

in order to make it helm on argocd