kyma-project / kyma-environment-broker

Kyma Environment Broker
Apache License 2.0
0 stars 22 forks source link

Adjust workflow skipping #1222

Closed MarekMichali closed 1 month ago

MarekMichali commented 1 month ago

Description

During release the kyma-gopher-bot creates a PR with bumps. For example, it changes the values.yaml of the KEB chart. We shouldn't run tests on exactly this PR and waste time, because we do tests in the release workflow. But we can't skip those jobs with path filtering either, because human developers also change those files during their daily work and then the tests need to be run.

A workaround is to add a check to workflows when they are triggered by kyma-gopher-bot and skip them. The All checks passed workflow only needs to be green to merge a PR, so in the case of kyma-gopher-bot we can just return a success status immediately without waiting for other jobs.

Note that if kyma-gopher-bot does more development PR's in the future, this behavior will need to be changed so that all workflow states are checked as in the case of a human developer PR.

Changes proposed in this pull request:

Related issue(s)

kyma-bot commented 1 month ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

github-actions[bot] commented 1 month ago

Add one of following labels

- kind/feature -> Use it when you want to submit a new feature

- kind/enhancement -> Use it when you modify or improve an existing feature

- kind/bug -> Use it when you fix a bug

MarekMichali commented 1 month ago

/kind enhancement