[CI:BUILD] Packit: set propose-downstream action type to pre-sync

Also address review concerns in pr#18675.

[NO NEW TESTS NEEDED]

Co-authored-by: Chris Evich <cevich@redhat.com>
Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>

.packit.yaml

@@ -33,11 +33,9 @@ jobs:
     update_release: false
     dist_git_branches:
       - fedora-all
-    #TODO use the right action to update the goimports
-    # For whatever reason, none of the actions seem to work
-    #actions:
-    #    create-patches:
-    #      - "cd rpm && bash update-spec-provides.sh"
+    actions:
+        pre-sync:
+          - "cd rpm && bash update-spec-provides.sh"
 
   - job: koji_build
     trigger: commit

rpm/Makefile

@@ -1,7 +1,7 @@
+.PHONY: rpm
 rpm:
 	$(shell /usr/bin/bash ./update-spec-version.sh)
 	spectool -g podman.spec
-	sudo dnf -y builddep podman.spec
 	rpmbuild -ba \
 		--define '_sourcedir $(shell pwd)' \
 		--define '_rpmdir %{_sourcedir}/RPMS' \

rpm/podman.spec

@@ -154,8 +154,6 @@ Recommends: %{name}-gvproxy = %{epoch}:%{version}-%{release}
 Provides: %{name}-quadlet
 Obsoletes: %{name}-quadlet <= 5:4.4.0-1
 Provides: %{name}-quadlet = %{epoch}:%{version}-%{release}
-# TODO: need to find the right action in packit propose-downstream to update
-# the goimports here. None of the actions seem to work so far.
 # DO NOT DELETE BELOW LINE - used for updating downstream goimports
 # vendored libraries
 

rpm/update-spec-provides.sh

@@ -4,6 +4,8 @@
 # packaging, via the `propose-downstream` packit action.
 # The goimports don't need to be present upstream.
 
+set -e
+
 SPEC_FILE=$(pwd)/podman.spec
 
 sed -i '/Provides: bundled(golang.*/d' $SPEC_FILE

rpm/update-spec-version.sh

@@ -4,6 +4,8 @@
 # default. Useful for local manual rpm builds where the Version needs to be set
 # correctly.
 
+set -e
+
 SPEC_FILE=$(pwd)/podman.spec
 LATEST_TAG=$(git tag --sort=creatordate | tail -1)
 LATEST_VERSION=$(echo $LATEST_TAG | sed -e 's/^v//')