action #179314
Updated by tinita 13 days ago
## Observation
email report:
We got an email report about too many ```
opensuse.org :: openqa.opensuse.org :: Minion Jobs - see https://openqa.opensuse.org/minion/jobs?state=failed
WARNINGs: failed minion jobs: is 423.00 (outside range [:400]).
https://openqa.opensuse.org/minion/jobs?state=failed ```
One failed job: issue I found is https://openqa.opensuse.org/minion/jobs?id=5038421 showing
```
created: 2025-03-20T14:38:07.148564Z
...
result:
error: |-
<strong>Failed to save ibus_test_kr-ibus-korean-switch-hangul-20250319.</strong><br><pre>Unable to push Git commit (/var/lib/openqa/share/tests/opensuse/products/opensuse/needles): To github.com:os-autoinst/os-autoinst-needles-opensuse.git
! [rejected] master -> master (fetch first)
error: failed to push some refs to 'github.com:os-autoinst/os-autoinst-needles-opensuse.git'
hint: Updates were rejected because the remote contains work that you do not
hint: have locally. This is usually caused by another repository pushing to
hint: the same ref. If you want to integrate the remote changes, use
hint: 'git pull' before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.</pre>
```
There were 5 failed jobs like this around the same time: https://openqa.opensuse.org/minion/jobs?task=save_needle&state=failed&queue=¬e=
At some point the user was able to save the needle.
The problem is that `update_remote` and `update_branch` are not enabled on o3, so it can happen that a push to the needles repo happened while someone else is trying to save a new needle.
Only when those two configuration options are enabled the code will try to rebase on the upstream master.
## Acceptance Criteria
**AC1:** `save_needle` uses the same approach as `git_clone` to find out the upstream default branch (instead of having to configure `update_branch`
## Suggestions
* If git is enabled in the config and `git_push` as well, consider to make `update_remote` and rebasing on upstream the default behaviour.
Back