Skip to content

rabbit-openqa: encode the product version in job name#3238

Draft
g7 wants to merge 2 commits intoopenSUSE:masterfrom
g7:openqa-version-name
Draft

rabbit-openqa: encode the product version in job name#3238
g7 wants to merge 2 commits intoopenSUSE:masterfrom
g7:openqa-version-name

Conversation

@g7
Copy link
Copy Markdown
Member

@g7 g7 commented May 15, 2025

The bot in the SUSE side crashed recently due to a name collision:

  File "/godata/pipelines/SUSE.openQA/./scripts/gocd/rabbit-openqa.py", line 284, in check_some_projects
    project.update_staging_status(staging)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
  File "/godata/pipelines/SUSE.openQA/./scripts/gocd/rabbit-openqa.py", line 175, in update_staging_status
    raise Exception(f'Names of job #{id} and #{taken_names[name]} collide: {name}')
Exception: Names of job #17092411 and #16972018 collide: Default-qcow-Updates-Staging-install_ltp+sle-micro+Default-qcow-Updates-Staging@aarch64

This is a bit of a niche situation, as the project is in the SUSE:SLFO:Kernel:1.0 project, where we share the kernel between SLFO 1.0 and 1.1.

This means that the same tests are run against both Micro 6.0 and Micro 6.1, and might thus collide.

Now, setting this as a Draft since I suppose we should discuss whether this should be applied to every project (where the version information is redundant, and might add entropy to already crowded test names), or maybe gate it to an attribute where it can be enabled on the project side, as it might be useful for just a bunch of projects. Suggestions welcome.

g7 and others added 2 commits May 15, 2025 23:22
The bot in the SUSE side crashed recently [0] due to a
name collision:

    File "/godata/pipelines/SUSE.openQA/./scripts/gocd/rabbit-openqa.py", line 284, in check_some_projects
      project.update_staging_status(staging)
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
    File "/godata/pipelines/SUSE.openQA/./scripts/gocd/rabbit-openqa.py", line 175, in update_staging_status
      raise Exception(f'Names of job #{id} and #{taken_names[name]} collide: {name}')
  Exception: Names of job #17092411 and #16972018 collide: Default-qcow-Updates-Staging-install_ltp+sle-micro+Default-qcow-Updates-Staging@aarch64

This is a bit of a niche situation, as the project is in the
SUSE:SLFO:Kernel:1.0 project, where we share the kernel between SLFO 1.0
and 1.1.

This means that the same tests are run against both Micro 6.0 and Micro
6.1, and might thus collide.

[0] https://botmaster.suse.de/go/tab/build/detail/SUSE.openQA/69498/Run/1/Run

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
@Vogtinator
Copy link
Copy Markdown
Member

Note that this will break all staging projects as the configured required_checks expect the current format.

Now, setting this as a Draft since I suppose we should discuss whether this should be applied to every project (where the version information is redundant, and might add entropy to already crowded test names), or maybe gate it to an attribute where it can be enabled on the project side, as it might be useful for just a bunch of projects. Suggestions welcome.

Hm. Maybe it should indeed be configurable per project. The format of the check name could be templated in an OBS prj attribute.

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