Project

Profile

Help

Issue #439

Uploaded errata don't have the same attribute types as sync'd errata

Added by rbarlow over 6 years ago. Updated over 1 year ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
1. Low
Version:
Master
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

Description

I learned during the fixing of #1100848[0] that some of the attributes for an uploaded erratum don't have the same types as the synchronized versions. The only attribute I know this for sure about is "pushcount". For an uploaded erratum, pushcount will be an integer, but for a sync'd erratum it will be a string (of an integer).

We needed to fix [0] quickly so we went for making sure we only have strings to ElementTree. I believe the correct fix is to make sure we are consistent about the types that land in our database between synchronizing errata and uploading them. I personally believe that pushcount should be an int in the database, so for that case we should fix the synchronization code.

I set the severity to low because I cannot think of a use case that this would cause an issue for.

[0] https://bugzilla.redhat.com/show_bug.cgi?id=1100848

+ This bug was cloned from Bugzilla Bug #1101728 +

Associated revisions

Revision bcea16d1 View on GitHub
Added by dkliban@redhat.com 4 months ago

Handle conflict when cherry-picking changelog

fixes: #439

History

#1 Updated by bmbouter over 6 years ago

  • Severity changed from Low to 1. Low

#2 Updated by bmbouter over 2 years ago

  • Status changed from NEW to CLOSED - WONTFIX

Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.

#3 Updated by bmbouter over 2 years ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF