Helm checks are mandatory for Manifests update, making PR fail. #3232
Description
Validation Checklist
- I confirm that this is a Kubeflow-related issue.
- I am reporting this in the appropriate repository.
- I have followed the Kubeflow installation guidelines.
- The issue report is detailed and includes version numbers where applicable.
- I have considered adding my company to the adopters page to support Kubeflow and help the community, since I expect help from the community for my issue (see 1. and 2.).
- This issue pertains to Kubeflow development.
- I am available to work on this issue.
- You can join the CNCF Slack and access our meetings at the Kubeflow Community website. Our channel on the CNCF Slack is here #kubeflow-platform.
Version
master
Detailed Description
When I use any Synchronization script on /scripts as requested by Manifest, these do not update the helm packages but only the Manifest, as intended
On the PR, Helm checks are MANDATORY
This makes periodic update FAIL.
Kindly consider either:
- making the Helm gi/GHA OPTIONAL
- updating WG leads instruction on how to update Manifests in this repo
Steps to Reproduce
./scripts/synchronize-model-registry-manifests.sh- open PR
running the required instructions to update Manifests, as above, make the PR fail. This is the second time so I can confirm the behaviour is reproducible and present.
Screenshots or Videos (Optional)
Thank you for your consideration.