workstation-ostree-config

#501 CI: Add comps / manifest sync check on PRs
Merged by siosm. Opened by siosm.
Unknown source main

Download 501.patch

comps-sync.py: Return non-zero if changes are needed but not saved

This will let us use that in CI to check for pending comps-sync changes.

justfile: Add a recipe to check for comps sync changes

Will be used by Zuul CI.

zuul: Verify that the comps and manifests are synced

Best effort (as it will only run on PRs) check for comps / manifests
sync.

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/f22202422de646fdb2f57f71b4a5bd30

~~LGTM~~

Looks like the new task added to ci/validate.yaml is failing with

error: Justfile does not contain recipes&&,touch, or.zuulci.comps.

I think you need to use ansible.builtin.shell instead of ansible.builtin.command
Edited by miabbott

3 new commits added

  • zuul: Verify that the comps and manifests are synced
  • justfile: Add a recipe to check for comps sync changes
  • comps-sync.py: Return non-zero if changes are needed but not saved

Thanks, good catch.

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/c536850bf8564e2ab1e6241e283d2801

Pull-Request has been merged by siosm
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.