Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

krel obs stage proceeds with exit code zero if there's a failed package build #3632

Closed
xmudrii opened this issue Jun 4, 2024 · 3 comments · Fixed by #3662
Closed

krel obs stage proceeds with exit code zero if there's a failed package build #3632

xmudrii opened this issue Jun 4, 2024 · 3 comments · Fixed by #3662
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.

Comments

@xmudrii
Copy link
Member

xmudrii commented Jun 4, 2024

What happened:

We have a waiting logic implemented in krel obs stage that's waiting for all package build processes to be done. However, this waiting mechanism doesn't check if the build is successful or not. We just had the following situation:

Step #3: rpm                  x86_64     succeeded
Step #3: rpm                  aarch64    succeeded
Step #3: rpm                  ppc64le    succeeded
Step #3: rpm                  s390x      succeeded
Step #3: deb                  x86_64     succeeded
Step #3: deb                  aarch64    succeeded
Step #3: deb                  ppc64le    failed
Step #3: deb                  s390x      succeeded

Even though the ppc64le build failed, the Cloud Build job finished successfully (i.e. with exit code zero).

What you expected to happen:

Failed builds should fail the Cloud Build job to make the user aware that something is wrong.

How to reproduce it (as minimally and precisely as possible):

It's a bit more complicated as this concrete failure was random/build system related, but you can try to run krel obs stage with a broken spec file.

@xmudrii xmudrii added kind/bug Categorizes issue or PR as related to a bug. sig/release Categorizes an issue or PR as relevant to SIG Release. area/release-eng Issues or PRs related to the Release Engineering subproject labels Jun 4, 2024
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 5, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
@saschagrunert
Copy link
Member

Something like this could help here: openSUSE/osc#1573

I think having it in osc as a feature is a cleaner way than parsing the status messages on our end.

saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 5, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 5, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 5, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 5, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 11, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
saschagrunert added a commit to saschagrunert/osc that referenced this issue Jun 11, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
dmach pushed a commit to saschagrunert/osc that referenced this issue Jun 20, 2024
This allows to exit with `1` in case any build fails to provide feedback
to end-user scripts.

Refers to: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
@saschagrunert
Copy link
Member

openSUSE/osc#1573 got merged, waiting for a new osc release to utilize the flag.

@saschagrunert
Copy link
Member

Requesting a new tag in openSUSE/osc#1587

saschagrunert added a commit to saschagrunert/packaging that referenced this issue Jul 1, 2024
This allows future krel versions to wait for the build results and
report the correct error.

Ref: kubernetes/release#3632

Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/release-eng Issues or PRs related to the Release Engineering subproject kind/bug Categorizes issue or PR as related to a bug. needs-priority sig/release Categorizes an issue or PR as relevant to SIG Release.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants