packit / dist-git-to-source-git

Converting dist-git to source-git
MIT License
4 stars 9 forks source link

Apply tags on updates in source-git #133

Closed csomh closed 3 years ago

csomh commented 3 years ago

In order to be able to track which dist-git commit did source-git update originate from, create a "conversion" tag from the dist-git branch name and commit hash (ex: convert/c8s/05262ab9019701b9d278b305017861200204f559) and apply it at the tip of the branch in source-git.

Before triggering the conversion, check if such a tag exists, and skip updating the source-git repo if it does.

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

csomh commented 3 years ago

recheck

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

csomh commented 3 years ago

@TomasTomecek please take another look. The systemd test that failed twice in a row here has passed for me locally, so if you agree, I would ignore the result of that here.

csomh commented 3 years ago

recheck

softwarefactory-project-zuul[bot] commented 3 years ago

Build failed.

csomh commented 3 years ago

Okay, so the systemd test fails at packit srpm, with the message:

packit.exceptions.PackitSRPMException: Preparing of the upstream to the SRPM build failed: Failed to parse SPEC file: error: Unable to open /home/packit/rpmbuild/SOURCES/triggers.systemd: No such file or directory

Which does not make too much sense. Maybe this is a packit issue after all? :thinking:

TomasTomecek commented 3 years ago

Seems like systemd dist-git got updated and that update break our conversion process (did they remove the file?)

csomh commented 3 years ago

recheck

softwarefactory-project-zuul[bot] commented 3 years ago

Build succeeded.

softwarefactory-project-zuul[bot] commented 3 years ago

Build succeeded (gate pipeline).