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

Fixes #32553 - Remove md5 from allowed content types #356

Merged
merged 1 commit into from
Jun 7, 2021

Conversation

jlsherrill
Copy link
Contributor

Previously we had added md5 to maintain md5 across upgrades
such that the upgrade would pass, but it turns out pulpcore 3.11
includes a migration which would handle this for us, if we were
running the db migration before collectstatic

@jlsherrill
Copy link
Contributor Author

corresponds with theforeman/puppet-pulpcore#190 going to run an upgrade pipeline to verify this works as it should

@wbclark
Copy link
Contributor

wbclark commented May 7, 2021

This now appears to be waiting on https://pulp.plan.io/issues/8712

Previously we had added md5 to maintain md5 across upgrades
such that the upgrade would pass, but it turns out pulpcore 3.11
includes a migration which would handle this for us, if we were
running the db migration before collectstatic
@jlsherrill
Copy link
Contributor Author

This should be good to go now that 3.11.2 has been built and is available in the repos. If we merge this as is, this HAS to go into katello 4.1.

@jlsherrill
Copy link
Contributor Author

also ping @ekohl for awareness

@ekohl ekohl merged commit fb40194 into theforeman:master Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants