chore: move over the helm chart for the ui from the operator repo

[HarikrishnanBalagopal]

Signed-off-by: Harikrishnan Balagopal [email protected]

[github-actions]

Thanks for making a pull request! 😃
One of the maintainers will review and advise on the next steps.

[ashokponkumar]

Won't we need a copy of the helm chart here retained till the new operator is in place? Otherwise the current helm based operator is not valid. So we can only remove the GitHub action part alone. Also from operator perspective, in read me, the helm chart reference can possible be removed.

[ashokponkumar]

Also the prepare for release section from mahespur might need to be removed

[HarikrishnanBalagopal]

Also the prepare for release section from mahespur might need to be removed

done

[HarikrishnanBalagopal]

Won't we need a copy of the helm chart here retained till the new operator is in place? Otherwise the current helm based operator is not valid. So we can only remove the GitHub action part alone. Also from operator perspective, in read me, the helm chart reference can possible be removed.

aren't the release tar files enough?

[ashokponkumar]

The operator Dockerfile uses the helm chart from its context (the operator repo).

[HarikrishnanBalagopal]

The operator Dockerfile uses the helm chart from its context (the operator repo).

I commented that out https://github.com/konveyor/move2kube-operator/pull/55/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R6

[ashokponkumar]

The operator Dockerfile uses the helm chart from its context (the operator repo).

I commented that out https://github.com/konveyor/move2kube-operator/pull/55/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R6

Yes, that line is required. Without it, the operator does nothing.

[HarikrishnanBalagopal]

The operator Dockerfile uses the helm chart from its context (the operator repo).

I commented that out https://github.com/konveyor/move2kube-operator/pull/55/files#diff-dd2c0eb6ea5cfc6c4bd4eac30934e2d5746747af48fef6da689e85b752f39557R6

Yes, that line is required. Without it, the operator does nothing.

yes, but isn't it broken anyway?

[ashokponkumar]

We have not tested it extensively with the new UI. But technically, it should be working, since the operator does nothing but the equivalent of a helm install