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

Taught pulp-rpm about the 'static_context' attribute of modules. #1984

Closed
wants to merge 1 commit into from

Conversation

ggainey
Copy link
Contributor

@ggainey ggainey commented May 10, 2021

The included test relies on a new fixture, and will fail without it.

closes #8638
Required PR: pulp/pulp-fixtures#213

@pulpbot
Copy link
Member

pulpbot commented May 10, 2021

Attached issue: https://pulp.plan.io/issues/8638

@ggainey ggainey force-pushed the 8638_static_context branch 2 times, most recently from 0cdfb1e to 3daa658 Compare May 10, 2021 20:13
class Migration(migrations.Migration):

dependencies = [
('rpm', '0033_auto_publish'),
Copy link
Contributor

@dralley dralley May 11, 2021

Choose a reason for hiding this comment

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

Note to self that if we need to back out the autopublish changes, we need to come back and fix this

pulp_rpm/app/models/modulemd.py Outdated Show resolved Hide resolved
pulp_rpm/app/serializers/modulemd.py Outdated Show resolved Hide resolved
pulp_rpm/app/models/modulemd.py Outdated Show resolved Hide resolved
pulp_rpm/app/models/modulemd.py Outdated Show resolved Hide resolved
@ggainey ggainey force-pushed the 8638_static_context branch 2 times, most recently from 0ece85e to 5201f4b Compare May 12, 2021 18:09
The included test relies on a new fixture, and will fail without it.

closes #8638
Required PR: pulp/pulp-fixtures#213
@dralley dralley closed this May 13, 2021
@goosemania
Copy link
Member

Merged as a part of #1988.

@ggainey ggainey deleted the 8638_static_context branch March 31, 2023 20:09
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.

None yet

4 participants