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

Fix serializer for erratum pkglists #1372

Merged
merged 1 commit into from
Jun 17, 2019
Merged

Conversation

goosemania
Copy link
Member

It didn't affect publish results, however it did affect applicability
calculations and erratum content which user queries via API.

Kudos to Hyu (reporter of the issue) for the patch.

closes #4880
https://pulp.plan.io/issues/4880

It didn't affect publish results, however it did affect applicability
calculations and erratum content which user query via API.

Kudos to Hyu (reporter of the issue) for the patch.

closes #4880
https://pulp.plan.io/issues/4880
@goosemania
Copy link
Member Author

Verification steps (2 repos with the same errata id but different pkglists are needed):

# create and sync one repo
$ pulp-admin rpm repo create --repo-id base_repo --relative-url base_repo --download-policy on_demand --feed $URL_TO_BASE_REPO
$ pulp-admin rpm repo sync run --repo-id base_repo

# create and sync other repo (which has at least one erratum with the same id as in base_repo but different pkglist)
$ pulp-admin rpm repo create --repo-id debuginfo_repo --relative-url debuginfo_repo --download-policy on_demand --feed $URL_TO_DEBUGINFO_REPO
$ pulp-admin rpm repo sync run --repo-id debuginfo_repo

# search for the erratum of interest, check that both pkglists are present and not only one like before
$ pulp-admin -vvv rpm repo content errata --repo-id base_repo --str-eq 'id=<errata-id-which-is-present-in-both-repos>'


@goosemania goosemania merged commit 8f36fb7 into pulp:2-master Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants