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

TEST: remove content_origin and let pulp-operator auto set it. #306

Closed

Conversation

mikedep333
Copy link
Member

Do not merge this.

This is based on the following commit:

Set CONTENT_ORIGIN settings

The Travis environment needs content_origin set as a setting.

This also updates the tests to expect Pulp's Distribution.base_url to
provide absolute urls.

Required PR: pulp/pulpcore#358
Required PR: pulp/pulp-smash#1227
Required PR: pulp/pulp_installer#185

https://pulp.plan.io/issues/5629

@mikedep333 mikedep333 requested a review from a team November 5, 2019 22:48
This is based on the following commit:

Set CONTENT_ORIGIN settings

The Travis environment needs `content_origin` set as a setting.

This also updates the tests to expect Pulp's Distribution.base_url to
provide absolute urls.

Required PR: pulp/pulpcore#358
Required PR: pulp/pulp-smash#1227
Required PR: pulp/pulp_installer#185

https://pulp.plan.io/issues/5629

[noissue]
@mikedep333 mikedep333 force-pushed the operator_auto_content_origin_test branch from c75dd5a to 13bd067 Compare November 5, 2019 23:07
@ipanova
Copy link
Member

ipanova commented Nov 8, 2019

closing in favour of #308

@ipanova ipanova closed this Nov 8, 2019
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.

3 participants