test: add tmt test to verify posttrans working

Author: HuijingHei

.packit.yaml

@@ -10,43 +10,40 @@ downstream_package_name: rust-bootupd
 
 specfile_path: contrib/packaging/bootupd.spec
 
+srpm_build_deps:
+  - cargo
+  - git
+  - libzstd-devel
+  - openssl-devel
+  - zstd
+
+actions:
+  # The last setp here is required by Packit to return the archive name
+  # https://packit.dev/docs/configuration/actions#create-archive
+  create-archive:
+    - bash -c "cargo install cargo-vendor-filterer"
+    - bash -c "cargo xtask spec"
+    - bash -c "cat target/bootupd.spec"
+    - bash -c "cp target/bootupd* contrib/packaging/"
+    - bash -c "ls -1 target/bootupd*.tar.zstd | grep -v 'vendor'"
+  # Do nothing with spec file. Two steps here are for debugging
+  fix-spec-file:
+    - bash -c "cat contrib/packaging/bootupd.spec"
+    - bash -c "ls -al contrib/packaging/"
+
 jobs:
   - job: copr_build
     trigger: pull_request
     targets:
-      - centos-stream-9-x86_64
-      - centos-stream-9-aarch64
-      - centos-stream-9-s390x
-      - centos-stream-10-x86_64
-      - centos-stream-10-aarch64
-      - centos-stream-10-s390x
-      - centos-stream-10-ppc64le
-      - fedora-43-x86_64
-      - fedora-43-aarch64
-      - fedora-43-s390x
-      - fedora-43-ppc64le
-      - fedora-rawhide-x86_64
       - fedora-rawhide-aarch64
-      - fedora-rawhide-s390x
-      - fedora-rawhide-ppc64le
+      - fedora-rawhide-x86_64
 
   - job: tests
     trigger: pull_request
     targets:
-      - centos-stream-9-x86_64
-      - centos-stream-9-aarch64
-      - centos-stream-10-x86_64
-      - centos-stream-10-aarch64
-      - fedora-43-x86_64
-      - fedora-43-aarch64
-      - fedora-rawhide-x86_64
       - fedora-rawhide-aarch64
+      - fedora-rawhide-x86_64
     tmt_plan: /tmt/plans/package
-    tf_extra_params:
-      environments:
-        - tmt:
-            context:
-              running_env: "packit"
 
   - job: propose_downstream
     trigger: release

tmt/plans/package.fmf

@@ -1,22 +1,38 @@
-# Common settings for all plans
+discover:
+  how: fmf
+execute:
+  how: tmt
 provision:
   how: virtual
-  image: $@{test_disk_image}
+  hardware:
+    boot:
+      method: uefi
+  image: $@{test_image}
+  user: root
 prepare:
-  # Run on package mode VM running on Packit and Gating
+  # Run on package mode VM especially on Fedora
   - how: shell
-    order: 98
-    script:
-      - mkdir -p bootc && cp /var/share/test-artifacts/*.src.rpm bootc
-      - cd bootc && rpm2cpio *.src.rpm | cpio -idmv && rm -f *-vendor.tar.zstd && zstd -d *.tar.zstd && tar -xvf *.tar -C . --strip-components=1 && ls -al
-      - pwd && ls -al && cd bootc/hack && ./provision-packit.sh
-    when: running_env != image_mode
-execute:
-  how: tmt
+    order: 10
+    adjust:
+      - when: distro == fedora
+        script:
+          - |
+            set -eux
+            if [ -d /boot/efi/EFI/fedora ]; then
+              for f in /boot/efi/EFI/fedora/grub*.efi; do
+                echo "Corrupting $f"
+                echo "test" > "$f"
+              done
+            fi
+  - how: install
+    order: 20
+    package:
+      - bootupd
+
 
 /plan-post-install:
   summary: Execute post install tests
   discover:
     how: fmf
     test:
-      - /tmt/tests/tests/test-post-install
+      - /tmt/tests/tests/test-post-install

tmt/tests/package/test-post-install.sh

@@ -0,0 +1,66 @@
+# number: 10
+# tmt:
+#   summary: Test posttrans on package mode
+#   duration: 10m
+#
+#!/bin/bash
+set -eux
+
+echo "Testing posttrans on package mode"
+
+bootupctl --version
+
+source /etc/os-release
+if [ "$ID" == "fedora" ] && [ "$VERSION_ID" -lt 44 ]; then
+    echo "Skip testing on F43 and older"
+    exit 0
+fi
+
+suffix=""
+get_grub_suffix() {
+    case "$(uname -m)" in
+        x86_64)
+            suffix="x64"
+            ;;
+        aarch64)
+            suffix="aa64"
+            ;;
+        *)
+            echo "Unsupported arch"
+            exit 1
+            ;;
+    esac
+}
+
+if [ "$TMT_REBOOT_COUNT" -eq 0 ]; then
+    echo 'Before first reboot'
+    # assume ESP is already mounted at /boot/efi
+    mountpoint /boot/efi
+    get_grub_suffix
+    grubefi="grub${suffix}.efi"
+
+    grub_source_path=$(find /usr/lib/efi/ -name "${grubefi}")
+    if [ -z "${grub_source_path}" ]; then
+        echo "Error: Source GRUB binary ${grub_source_path} not found."
+        exit 1
+    fi
+
+    grub_target_path=/boot/efi/EFI/fedora/${grubefi}
+    if [ ! -f "${grub_target_path}" ]; then
+        echo "Error: Could not find target GRUB binary ${grub_target_path}."
+        exit 1
+    fi
+    # get checksum from /usr/lib/efi/grub2/xx/EFI/fedora/grub.efi
+    source_checksum=$(sha256sum "${grub_source_path}" | cut -d' ' -f1)
+    # get checksum from target /boot/efi
+    target_checksum=$(sha256sum "${grub_target_path}" | cut -d' ' -f1)
+    # confirm that the target grub.efi is updated
+    [ "${source_checksum}" == "${target_checksum}" ]
+    tmt-reboot
+elif [ "$TMT_REBOOT_COUNT" -eq 1 ]; then
+    echo 'After the reboot'
+    # just confirm the reboot is successful
+    whoami    
+fi
+
+echo "Run posttrans test successfully"

tmt/tests/tests.fmf

@@ -1,4 +1,8 @@
 /test-post-install:
   summary: Execute installation on package mode 
   duration: 10m
-  test: bash package/post-install.sh
+  adjust:
+    - when: distro != fedora
+      enabled: false
+  test: bash package/test-post-install.sh
+  

xtask/src/main.rs

@@ -32,6 +32,7 @@ fn try_main() -> Result<()> {
             "vendor" => vendor,
             "package" => package,
             "package-srpm" => package_srpm,
+            "spec" => spec,
             _ => print_help,
         };
         f(&sh)?;
@@ -155,10 +156,9 @@ fn impl_package(sh: &Shell) -> Result<Package> {
         )
         .run()?;
     }
-    // Compress with gzip and write to crate
-    let srcpath: Utf8PathBuf = Utf8Path::new("target").join(format!("{namev}.crate"));
-    cmd!(sh, "gzip --force --best {p}").run()?;
-    std::fs::rename(format!("{p}.gz"), &srcpath)?;
+    // Compress with zstd
+    let srcpath: Utf8PathBuf = format!("{p}.zstd").into();
+    cmd!(sh, "zstd --rm -f {p} -o {srcpath}").run()?;
 
     Ok(Package {
         version: v,
@@ -243,6 +243,42 @@ fn package_srpm(sh: &Shell) -> Result<()> {
     Ok(())
 }
 
+fn update_spec(sh: &Shell) -> Result<Utf8PathBuf> {
+    let _targetdir = get_target_dir()?;
+    let p = Utf8Path::new("target");
+    let pkg = impl_package(sh)?;
+    let srcpath = pkg.srcpath.file_name().unwrap();
+    let v = pkg.version;
+    let src_vendorpath = pkg.vendorpath.file_name().unwrap();
+    {
+        let specin = File::open(format!("contrib/packaging/{NAME}.spec"))
+            .map(BufReader::new)
+            .context("Opening spec")?;
+        let mut o = File::create(p.join(format!("{NAME}.spec"))).map(BufWriter::new)?;
+        for line in specin.lines() {
+            let line = line?;
+            if line.starts_with("Version:") {
+                writeln!(o, "# Replaced by cargo xtask spec")?;
+                writeln!(o, "Version: {v}")?;
+            } else if line.starts_with("Source0") {
+                writeln!(o, "Source0: {srcpath}")?;
+            } else if line.starts_with("Source1") {
+                writeln!(o, "Source1: {src_vendorpath}")?;
+            } else {
+                writeln!(o, "{line}")?;
+            }
+        }
+    }
+    let spec_path = p.join(format!("{NAME}.spec"));
+    Ok(spec_path)
+}
+
+fn spec(sh: &Shell) -> Result<()> {
+    let s = update_spec(sh)?;
+    println!("Generated: {s}");
+    Ok(())
+}
+
 fn print_help(_sh: &Shell) -> Result<()> {
     eprintln!(
         "Tasks: