Skip to content

Add rmdepcheck and fedora-review tests#3089

Merged
centosinfra-prod-github-app[bot] merged 1 commit into
packit:mainfrom
LecrisUT:feat/more-tests
May 12, 2026
Merged

Add rmdepcheck and fedora-review tests#3089
centosinfra-prod-github-app[bot] merged 1 commit into
packit:mainfrom
LecrisUT:feat/more-tests

Conversation

@LecrisUT
Copy link
Copy Markdown
Collaborator

@LecrisUT LecrisUT commented Apr 1, 2026

TODO:

  • Write new tests or update the old ones to cover new functionality.
  • Make the related tests compatible with packit's contexts
    • rmdepcheck
    • fedora-review ready but postponing the enablement
    • license-validate

Merge after fedora-ci/rmdepcheck-pipeline#12

Also not sure how this would behave when run in prod when the corresponding tests are not yet enabled.

RELEASE NOTES BEGIN

Packit Fedora-CI now runs rmdepcheck and license-validate tests on PRs.

RELEASE NOTES END

@LecrisUT LecrisUT requested a review from a team as a code owner April 1, 2026 12:26
@LecrisUT LecrisUT requested review from betulependule and removed request for a team April 1, 2026 12:26
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces support for rmdepcheck and fedora-review tests within the Fedora CI workflow by adding dedicated payload generation methods and updating the corresponding integration tests. The review feedback identifies several necessary corrections: the TMT plan names in the implementation should include a leading slash to maintain consistency with conventions and test expectations; the integration tests must be updated to expect the stg branch for the ref field to match the non-production environment logic; and the deployment check should be refactored to compare the enum member directly rather than its string name for improved robustness.

Comment thread packit_service/worker/helpers/testing_farm.py Outdated
Comment thread packit_service/worker/helpers/testing_farm.py Outdated
Comment thread tests/integration/test_listen_to_fedmsg.py Outdated
Comment thread tests/integration/test_listen_to_fedmsg.py Outdated
Comment thread packit_service/worker/helpers/testing_farm.py Outdated
@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@LecrisUT
Copy link
Copy Markdown
Collaborator Author

LecrisUT commented Apr 29, 2026

@majamassarini this should be all ready right now, so let's move it forward. Btw among the packit roles, who is best to ping about moving some PRs forward? (picked the community-shepherd 🙂)

CC rmdepcheck and license-validate test owners, all good to start running it on PRs? @AdamWill @xsuchy

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@AdamWill
Copy link
Copy Markdown

AdamWill commented Apr 29, 2026

CC rmdepcheck and license-validate test owners, all good to start running it on PRs?

For rmdepcheck: with the caveat that we know it'll fail on the individual PRs of a multi-package update case, yeah.

@LecrisUT
Copy link
Copy Markdown
Collaborator Author

For rmdepcheck: with the caveat that we know it'll fail on the individual PRs of a multi-package update case, yeah.

Don't worry this is only one of 10 other different ways things fail there 😅

@LecrisUT
Copy link
Copy Markdown
Collaborator Author

LecrisUT commented May 7, 2026

@nforro Can we get this moving soon?

@xsuchy
Copy link
Copy Markdown
Contributor

xsuchy commented May 12, 2026

@LecrisUT can you rebase it please?

@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

Signed-off-by: Cristian Le <git@lecris.dev>
@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Member

@nforro nforro left a comment

Choose a reason for hiding this comment

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

LGTM

@nforro nforro moved this from New to In review in Packit pull requests May 12, 2026
@nforro nforro added the mergeit Merge via Zuul label May 12, 2026
@centosinfra-prod-github-app
Copy link
Copy Markdown
Contributor

@centosinfra-prod-github-app centosinfra-prod-github-app Bot merged commit da3f352 into packit:main May 12, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in Packit pull requests May 12, 2026
@LecrisUT LecrisUT deleted the feat/more-tests branch May 13, 2026 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mergeit Merge via Zuul

Projects

Development

Successfully merging this pull request may close these issues.

5 participants