Project

General

Profile

Bug #33804

In rare cases upstream APT (deb content) repos can exceed size limited DB fields

Added by Quirin Pamp 7 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Repositories
Target version:
Katello 4.4.0
Difficulty:
trivial
Triaged:
Yes
Bugzilla link:
Pull request:
https://github.com/Katello/katello/pull/9927
Fixed in Releases:
Katello 4.4.0
Found in Releases:
Red Hat JIRA:

Description

The issue was reported here: https://community.theforeman.org/t/katello-value-too-long-for-type-character-varying-255/24953/26

It is being fixed on the pulp side under this issue: https://pulp.plan.io/issues/9277

However, Katello also stores some of the affected fields in the Katello DB, with the same limit originally set by pulp_deb.
Therefor, this will also need to be fixed on the Katello side.

I expect the pulp_deb fix to make it into a pulpcore 3.15 compatible version. Due to DB migrations, it will not be possible to backport the fix any further.

Associated revisions

Revision e75900ae (diff)
Added by Manisha Singhal 3 months ago

Fixes #33804 - Change size limit of deb attributes based on pulp deb (#9927)

History

#1 Updated by Samir Jha 7 months ago

  • Status changed from New to Need more information

Hey Quirin,

Based on the pulp fix, do you still see any changes needed in Katello for this?

#2 Updated by Markus Bucher 7 months ago

Hi Samir,

I am afraid we will have to extend the fields in Katello's DB as well, because we have the same data and the same field constraints in Katello.

#3 Updated by Justin Sherrill 6 months ago

  • Triaged changed from No to Yes
  • Difficulty set to trivial
  • Status changed from Need more information to New
  • Category set to Repositories

#4 Updated by Ian Ballou 4 months ago

I'll be moving this to 4.5.0 if it a PR doesn't pop up in the next couple of weeks. 4.4.0 is branching on Feb 10th.

#5 Updated by The Foreman Bot 3 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9927 added

#6 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases Katello 4.5.0 added

#7 Updated by Manisha Singhal 3 months ago

  • Status changed from Ready For Testing to Closed

#8 Updated by Ian Ballou 3 months ago

  • Fixed in Releases Katello 4.4.0 added
  • Fixed in Releases deleted (Katello 4.5.0)

Also available in: Atom PDF