202: Add Support for PlainHTTP for flexibility in chart downloads

[ioannis-matzaris]

Add Support for PlainHTTP

Fixes #202

• Added PlainHTTP, which enables insecure HTTP connections for chart downloads
  • useful in specific scenarios where secure connections are unavailable.
• Extended support for PlainHTTP across all relevant operations:
  • Install (InstallAction)
  • Upgrade (UpgradeAction)
  • Pull (PullAction)
• Updated CRD schema (helm.crossplane.io_releases.yaml) to include the new PlainHTTP field in the spec.

I have:

• Read and followed Crossplane's contribution process.
• Run make reviewable to ensure this PR is ready for review. There is no reviewable step in the Makefile

How has this code been tested

• manually deployed thepackage/crds/*
• deployed the examples/sample/release.yaml by adding an extra line after L17 plainHTTP: true, without any errors. Before my changes, this line would have caused an error

[AbrohamLincoln]

@turkenh I just ran into this issue and found this PR. What can we do to get this reviewed and merged?

[haarchri]

Thanks for this enhancement, and sorry for the long delay in reviewing it. Since it’s been a long time, could you please rebase the PR? After that, we can merge it into the main branch and include it in the next maintenance release.

[ioannis-matzaris]

Thanks for this enhancement, and sorry for the long delay in reviewing it. Since it’s been a long time, could you please rebase the PR? After that, we can merge it into the main branch and include it in the next maintenance release.

@haarchri No problem at all. I’ve rebased the PR onto the latest main.

Thanks for reviewing! Happy to see it included in the next maintenance release.

[haarchri]

thanks, can you add this too in the namespaced controller ? Since crossplane V2 we have also the apis for namespaced with extra Controllers - thanks ;)

[ioannis-matzaris]

thanks, can you add this too in the namespaced controller ? Since crossplane V2 we have also the apis for namespaced with extra Controllers - thanks ;)

@haarchri Thanks for your feedback. It should be also working on the namespaced controller with the 07137e1, right?

[haarchri]

did you rerun make generate ? missing the update in one of the CRDs

[ioannis-matzaris]

did you rerun make generate ? missing the update in one of the CRDs

good catch! I updated it here

[sergenyalcin]

Thanks @ioannis-matzaris LGTM!