[Pulp-dev] pulp_rpm and current backwards-compatibility problems

[Grant Gainey]

I'm sorry, this is going to be a noisy thread for a while - but this is
Complicated, and I want to make sure everyone stays on the same page as we
get ourselves out of this thicket...

On Tue, May 11, 2021 at 4:20 PM Grant Gainey <ggainey at redhat.com> wrote:

> Whew. So anyway, the sequence to unsnarl this goes something like so:
>
> 1. get pulp_rpm CI building against pulpcore-3.12
>   - PR#1985 <https://github.com/pulp/pulp_rpm/pull/1985>
>   - figure out how to pin django-2.2.20 before pip tries to install 2.2.21
>

This step is *completed*.  PR#1985
<https://github.com/pulp/pulp_rpm/pull/1985> was modified and built
successfully!

fao89 points out that the trick we're doing w/ requirements.txt won't work
for release-jobs because they install pulpcore from pypi.

Suggestion:

<fao891> an alternative would be pinning django on the container image -
https://github.com/pulp/pulp_rpm/blob/master/.ci/ansible/Containerfile.j2#L13-L22
<bmbouter> fao891: yeah that's what we need right there


So that can be something to look at going forward.

Next up is "revert The Things", but I'm not going to do that at the end of
a long day.

G


2. Revert the following commits in pulp_rpm:
>  - d0c9badd Refactor distribution migration 0032
>  - fbaadaca Add support for automatic publishing and distributing
>  - 3157ad Oracle ULN
> 3. get the static_context change updated (since a migration will have Left
> the Building) and get it merged
> 4. merge any other fixes that won't break 3.7-compat [OPTIONAL], and THEN
> 5. cut pulp-rpm/3.11 as compatible with pulpcore/3.7-thru-3.10
>
> Once 3.11 is in the can, then we :
> 1. Re-apply the reverted PRs and resolve conflicts (there will be some)
> 2. remove the dependency on enqueue_with_reservation()
> 3. Unpin pulp_rpm from 3.12/django-2.2.20
> 4 Release a pulp_rpm/3.12 that will be ready for pulppcore/3.13 to be
> released
>
> I feel like I've missed a step somewhere - dalley, bmbouter, was there
> something else form the call we just had? Or is this It?
>
> Whew!
>
> OK, I am off to adjust PR-1985 to do step-1 above.
>
> G
>
> On Tue, May 11, 2021 at 11:49 AM Grant Gainey <ggainey at redhat.com> wrote:
>
>> Hey folks,
>>
>> We've been talking about how we need a pulpcore/3.7-to-3.11-compatible
>> release of pulp_rpm. The static_context change requires a schema-change,
>> and it has to be available to katello-3.18 (and hence pulpcore-3.7)
>>
>> The static_context change is PR#1984
>> <https://github.com/pulp/pulp_rpm/pull/1984>
>>
>> Right now, pulp-rpm/master has changes that require pulpcore/3.12 or
>> later. Those changes are:
>>
>> d0c9badd Refactor distribution migration 0032
>> fbaadaca Add support for automatic publishing and distributing
>>
>> In addition, pulp_rpm/master is *currently broken* because it still
>> references the deprecated enqueue_with_reservation(), that just got removed
>> from pulpcore/master.
>>
>> As I understand it, what needs to happen #SOON, is the following:
>>
>>    1. revert the two commits above and merge,
>>    2. get the static_context change updated (since a migration will have
>>    Left the Building) and get it merged,
>>    3. merge any other fixes that won't break 3.7-compat [OPTIONAL], and
>>    THEN
>>    4. cut 3.11 as compatible with 3.7-thru-3.10 pulpcore
>>
>> Once pulp_rpm/3.11 is released, we can then:
>>
>>    1. re-apply the auto-pub/dist-schema changes,
>>    2. fix enqueue-problem,
>>    3. mark pulp_rpm/master as 3.12+ compat, and finally
>>    4. release pulp_rpm/3.12 to be ready for pulpcore-3.13
>>
>> And this all needs to happen by next week?
>>
>> Is there anything I'm missing here?
>>
>> G
>> --
>> Grant Gainey
>> Principal Software Engineer, Red Hat System Management Engineering
>>
>
>
> --
> Grant Gainey
> Principal Software Engineer, Red Hat System Management Engineering
>


-- 
Grant Gainey
Principal Software Engineer, Red Hat System Management Engineering
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/pulp-dev/attachments/20210511/2894be52/attachment.htm>