Skip to content

Conversation

@gerrod3
Copy link
Contributor

@gerrod3 gerrod3 commented Oct 20, 2022

fixes: #572
Required PR: pulp/pulpcore#3341

Copy link
Contributor

@dralley dralley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, however, I'm going to draft this because you will need to bump the minimum required pulpcore release once we know what that is.

I don't think we can do it yet because if you declare a minimum of 3.22.0 it'll break.

@dralley dralley marked this pull request as draft October 20, 2022 19:45
Repository
Repository,
)
from pulpcore.plugin.responses import ArtifactResponse
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we backport the pulpcore change to make it available in 3.21.z maybe earlier?

Copy link
Contributor

@dralley dralley Oct 21, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could but we'd have to call it not-a-feature or the CI will whine about it

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a release date set for 3.22? We'd love to get this fix released since it's currently biting us but I appreciate Pulp's desire to not backport features.

@gerrod3 gerrod3 marked this pull request as ready for review December 15, 2022 20:10
@gerrod3 gerrod3 merged commit 64e55c2 into pulp:main Dec 15, 2022
@gerrod3 gerrod3 deleted the artifact-response branch December 15, 2022 20:11
dralley added a commit that referenced this pull request Dec 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

500 error when pip installing packages and using object storage

4 participants