Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 1365353 - [RFE] run rpmdeplint and return it's results
Summary: [RFE] run rpmdeplint and return it's results
Keywords:
Status: MODIFIED
Alias: None
Product: rpm-test-trigger
Classification: Community
Component: general
Version: unreleased
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified vote
Target Milestone: ---
Assignee: beaker-dev-list
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-09 04:14 UTC by Roman Joost
Modified: 2018-11-09 23:17 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed:


Attachments (Terms of Use)

Description Roman Joost 2016-08-09 04:14:06 UTC
Two tasks here:

* execute rpmdeplint, check it's return status and
* return results to the Errata end point

For the acceptance tests we can use the fake Errata server which can simulate JSON api endpoints. It might need extending to write meaningful tests.

Comment 1 Tyrone Abdy 2016-11-18 05:43:09 UTC
As part of the discovery on this task it turns out that our assumptions on the repositories has been false.

So after discussion we have come that this patch will go through using the currently flawed assumptions that are present within rpm-test-trigger.

The current way we determine repositories is flawed see bug 1390801 for what the proposed solutions and issues encountered in the process of taking over this task from rjoost. As well as the follow up bug to have proper integration with PDC and pulp for 1395499

https://gerrit.beaker-project.org/#/c/5254


As an example to illustrate our current flawed assumptions.

We currently assume that the structure of a repository set works as such.

Base Repository (e.g Centos-Server) based on the paths.
/content/centos/6.8/server/x86_64/os

that it will only have optional/debug/devel that exist under
/content/centos/6.8/server/x86_64/devel/os
/content/centos/6.8/server/x86_64/optional/os
/content/centos/6.8/server/x86_64/debug/os

But in fact there are additional addons that can also be nested underneath the variant, that need the variant repo (that it is an addon to) to exist.

/content/centos/6.8/server/x86_64/special_variant/os
that would require the 
/content/centos/6.8/server/x86_64/os repo/path as well to successfully run an rpmdeplint run.

Comment 2 Tyrone Abdy 2016-11-18 05:44:59 UTC
As a note on the previous comment as not outlined explicitly.

The current idea is that we continue on in the patch with the current flawed assumption. Then in bug 1390801 fix up based on the new structure of repositories.

Comment 3 Tyrone Abdy 2016-11-30 23:19:34 UTC
The patch has been uploaded based on the repository changes, namely that we focus on the destination repositories opposed to the individual builds/rpms as the data structure.

The aim was to simplify the rpmdeplint stage and reduce as much assumptions at that point about the structure of advisories and their builds.

Patch is here.

https://gerrit.beaker-project.org/#/c/5254/


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