Skip to content

Add pkglistgen bot for git workflow project PRs#3287

Draft
g7 wants to merge 10 commits intoopenSUSE:masterfrom
g7:pkglistgen-bot
Draft

Add pkglistgen bot for git workflow project PRs#3287
g7 wants to merge 10 commits intoopenSUSE:masterfrom
g7:pkglistgen-bot

Conversation

@g7
Copy link
Copy Markdown
Member

@g7 g7 commented Jan 16, 2026

A more meaningful diff is available here: YoukouTenhouin/openSUSE-release-tools@379976a...g7:openSUSE-release-tools:pkglistgen-bot

This review bot runs pkglistgen on codestream QA projects.

pkglistgen only runs on successful builds, otherwise the request
is skipped and retried later.

Contributes to openSUSE/openSUSE-git#207

Comment thread git-pkglistgen.py Outdated
Comment thread git-pkglistgen.py
only_update_weakremovers=False,
stop_after_solve=False,
custom_cache_tag="git-pkglistgen",
)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on SLES we have to manually patch the default.productcompose file , what about having a confiuguration that allows to apply patches after pkglistgen has run?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are right... but perhaps we can fix it directly in pkglistgen. I'm not sure if we add support for cherry-picking commits/adding patches how long it will take to break (as most likely the 000productcompose output might change a lot and the patch might be rejected).

I'm not sure if we need those specific changes for staging at this moment

@g7 g7 force-pushed the pkglistgen-bot branch 5 times, most recently from 0edc059 to aceaca2 Compare January 26, 2026 17:19
@g7
Copy link
Copy Markdown
Member Author

g7 commented Jan 26, 2026

v2:

  • Rebased on top of feat: add branch info in gitea request implementation #3291.
  • Support forking repositories
  • Add support for Label retrieval on the gitea platform
  • Use an allowlist for organization/repo pairs
  • Only process In Progress PRs
  • Ship the osrt-git-pkglistgen both in the openSUSE-release-tools-pkglistgen package

g7 added 3 commits February 24, 2026 14:48
It is perfectly valid for a pkglistgen run to be identical to the
last one. Ensure we don't fail in that case.

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Populate Request.reviews with the reviews coming from the Gitea PR.

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
This review bot runs pkglistgen on codestream QA projects.

pkglistgen only runs on successful builds, otherwise the request
is skipped and retried later.

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
@g7 g7 force-pushed the pkglistgen-bot branch 2 times, most recently from bf83015 to 0773997 Compare February 24, 2026 14:02
@g7
Copy link
Copy Markdown
Member Author

g7 commented Feb 24, 2026

v3:

  • Fix lint issues
  • Fix allowlist that wasn't working properly
  • Add support for Labels
  • Allow passing status messages

g7 added 7 commits February 24, 2026 15:06
Available in Request._labels

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Fork organization should be supplied via the pkglistgen-git-staging-org-url
configuration.

The tip of the QA project branch will be forked to
pkglistgen-git-staging-org-url/<QA_PROJECT_NAME>, and its scmsync
will be set there.

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Use --git-allowed-repos to specify them. They need to be comma
separated.

If a repository is not there, the bot will ignore the request even if
it is added as a reviewer.

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
autogits gained support for Labels, where RelEngs can choose between
different templates for a different degrees of testing.

This commit adds support for it. The bot will not run (but will
accept the PR anyway) for "In Progress" stagings without any of the
allowed Labels being set. The labels are configurable, and they default
to:

QA-SLES-Basic QA-SLES-Reduced QA-SLES-Full

Signed-off-by: Eugenio Paolantonio <eugenio.paolantonio@suse.com>
@g7 g7 marked this pull request as draft March 19, 2026 16:54
@g7
Copy link
Copy Markdown
Member Author

g7 commented Mar 19, 2026

I'm moving this to draft. We will move to package monkey for the next milestone, and so this bot needs to be reworked.

Moved the Labels and Reviews support in #3307

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.

2 participants