Bug 2299789 - go-rpm-macros' CI test is broken
Summary: go-rpm-macros' CI test is broken
Keywords:
Status: MODIFIED
Alias: None
Product: Fedora
Classification: Fedora
Component: go-rpm-macros
Version: 42
Hardware: Unspecified
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Alejandro Sáez Morollón
QA Contact: Edjunior Barbosa Machado
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2024-07-25 01:08 UTC by Adam Williamson
Modified: 2025-02-26 13:05 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Adam Williamson 2024-07-25 01:08:46 UTC
go-rpm-macros has a CI test which tries to download a grafana .src.rpm from Fedora 38 and rebuild it (it apparently has some special property the test wants to check). Until today this was failing because F38 is EOL and has been archived, but the test was still looking for it at https://dl.fedoraproject.org/pub/fedora/linux/releases/38 . I fixed that in https://src.fedoraproject.org/rpms/go-rpm-macros/c/aff0d952b9cf407b2b71798326810a6004c59b71?branch=rawhide , but now the test fails because the spec syntax is obsolete: https://artifacts.dev.testing-farm.io/5a0dd3db-9eae-4e2e-8647-5ebb466b80b7/work-cimv8sx8b0/plans/ci/execute/data/guest/default-0/tests/Sanity/gobuild-grafana-1/data/grafana.rpmbuild.log - "error: %patchN is obsolete, use %patch N (or %patch -P N): %patch1 -p1"

I don't want to guess how you want to go about fixing that, so I'm reporting this bug...you probably will want to stash a modified version of the .src.rpm somewhere, or come up with a new test spec file, or something.

Reproducible: Always

Steps to Reproduce:
1. Do a build of go-rpm-macros
2. Watch it fail fedora-ci.koji-build.tier0.functional and get gated

Actual Results:  
It fails the test and the update is gated

Expected Results:  
It should pass the test and not be gated

Comment 1 Maxwell G 2024-07-31 16:55:48 UTC
Edjunior usually handles issues with these tests. I believe they're also shared with RHEL.

Comment 2 Edjunior Barbosa Machado 2024-08-07 10:37:16 UTC
A new test Sanity/gobuild_hello that has been added to https://src.fedoraproject.org/tests/go-rpm-macros

This PR removes the current failing test and adjusts the test plan accordingly:
https://src.fedoraproject.org/rpms/go-rpm-macros/pull-request/22

Comment 3 Aoife Moloney 2025-02-26 13:05:33 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 42 development cycle.
Changing version to 42.


Note You need to log in before you can comment on or make changes to this bug.