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

Revert "Enable automatic updates of test code and needles by default" #6045

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

Martchus
Copy link
Contributor

@Martchus Martchus commented Nov 5, 2024

This partially reverts commit a4047ef and e932030 as they likely cause problems with scheduling parallel clusters using the PARALLEL_ONE_HOST_ONLY=1 setting which is sometimes not taken into account anymore.

This is presumably not caused by the automatic Git updates themselves but by the the handling of related Minion jobs: If an openQA job has pending Minion jobs then the openQA job is not considered at all by the scheduler. This presumably leads to parallel clusters being only partially considered when assigning jobs to workers breaking the PARALLEL_ONE_HOST_ONLY=1 setting.

Related ticket: https://progress.opensuse.org/issues/168379

This partially reverts commit a4047ef and
e932030 as they likely cause problems with
scheduling parallel clusters using the `PARALLEL_ONE_HOST_ONLY=1` setting
which is sometimes not taken into account anymore.

This is presumably not caused by the automatic Git updates themselves but
by the the handling of related Minion jobs: If an openQA job has pending
Minion jobs then the openQA job is not considered at all by the scheduler.
This presumably leads to parallel clusters being only partially considered
when assigning jobs to workers breaking the `PARALLEL_ONE_HOST_ONLY=1`
setting.

Related ticket: https://progress.opensuse.org/issues/168379
Copy link

github-actions bot commented Nov 5, 2024

Great PR! Please pay attention to the following items before merging:

Files matching docs/*.asciidoc:

  • Consider generating documentation locally to verify it is rendered correctly using tools/generate-docs

This is an automatically generated QA checklist based on modified files.

@mergify mergify bot merged commit d806bde into os-autoinst:master Nov 5, 2024
46 checks passed
@Martchus Martchus deleted the git-update-default branch November 5, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants