Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check that the dist tag uses the correct value #58

Closed
tyll opened this issue Oct 21, 2019 · 4 comments
Closed

Check that the dist tag uses the correct value #58

tyll opened this issue Oct 21, 2019 · 4 comments
Labels
depends-on-koji Depends on a change or feature in Koji question Further information is requested

Comments

@tyll
Copy link

tyll commented Oct 21, 2019

When checking the EVR allow to check that the dist tag uses the right value, for example builds for Fedora 31 should use fc31.

@dcantrell
Copy link
Collaborator

Koji doesn't give the expanded disttag in its output, but I think we can use the build target and derive a disttag from that. Here's what I'm thinking:

  1. Expand the Koji data gathering in rpminspect so it picks up the build target name when it gathers the rest of the data.
  2. Define rules in rpminspect.conf on how to transpose a build target to a disttag. Or just add a 1:1 mapping.
  3. When rpminspect is able to get a build target, fill in the release string in struct rpminspect as our expected disttag value. The -r command line option should still override this though.

Right now rpminspect works when comparing two builds within the same build target because it derives the disttag value from the Release tag in the RPM headers. If they don't match, then it knows you are comparing two builds from different build targets.

In rpminspect.conf, I am thinking it can either carry a 1:1 mapping of targets to disttags. Such as:
f32-build = fc32

Or a transpose rule, like:
f${RELEASE}-build = fc${RELEASE}

Either way, this mapping information will have to be maintained just like the rest of the data in rpminspect-data-fedora.

Thoughts?

@dcantrell dcantrell added the question Further information is requested label Mar 3, 2021
@dcantrell
Copy link
Collaborator

I would still like to implement something here, but I think it's going to require some contributions to the Koji API to have it report out the dist tag as expanded in the mock environment used for building. I don't think that's impossible, but that would be the first step before I can do anything in rpminspect.

I have opened https://pagure.io/koji/issue/2760 requesting this functionality in Koji.

@dcantrell dcantrell added the depends-on-koji Depends on a change or feature in Koji label Mar 15, 2021
@hroncok
Copy link
Contributor

hroncok commented Feb 23, 2022

In rpminspect.conf, I am thinking it can either carry a 1:1 mapping of targets to disttags. Such as: f32-build = fc32

Or a transpose rule, like: f${RELEASE}-build = fc${RELEASE}

Either way, this mapping information will have to be maintained just like the rest of the data in rpminspect-data-fedora.

Thoughts?

Just a note: A valid Fedora 37 dist tag might as well be fc37~bootstrap.

@dcantrell
Copy link
Collaborator

dist tags are pretty fluid and we have a rough pattern we follow, but we are deliberately not that strict. I don't think a check here can be implemented reliably after thinking about this for a while. If someone has a better idea, please let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depends-on-koji Depends on a change or feature in Koji question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants