-
Notifications
You must be signed in to change notification settings - Fork 44
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
Introduced the Pulp 3 'content' role. #1166
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rochacbruno
added a commit
to rochacbruno/pulp_rpm
that referenced
this pull request
Feb 6, 2019
1. gen_yum_config_file was generating repo file using `:` intead of `=` and that was causing dnf errors. 2. Consume and Download test cases are now using the host defined in the new `content` role if defined. Depending on: pulp/pulp-smash#1166 [noissue]
nixocio
reviewed
Feb 7, 2019
7631c96
to
3ca67c1
Compare
rochacbruno
commented
Feb 8, 2019
rochacbruno
commented
Feb 8, 2019
rochacbruno
commented
Feb 8, 2019
nixocio
reviewed
Feb 11, 2019
nixocio
reviewed
Feb 11, 2019
Added `content` as a valid role for config hosts Added helper methods in `cfg` to get content_host Base URL. Note: In cases where `content` role is not defined then the `api` host is assumed as the content host.
3ca67c1
to
c8155be
Compare
rochacbruno
added a commit
to rochacbruno/pulp_rpm
that referenced
this pull request
Feb 11, 2019
1. gen_yum_config_file was generating repo file using `:` intead of `=` and that was causing dnf errors. 2. Consume and Download test cases are now using the host defined in the new `content` role if defined. Depending on: pulp/pulp-smash#1166 Closes #4320 https://pulp.plan.io/issues/4320
nixocio
approved these changes
Feb 12, 2019
rochacbruno
added a commit
to rochacbruno/pulp_rpm
that referenced
this pull request
Feb 12, 2019
1. gen_yum_config_file was generating repo file using `:` intead of `=` and that was causing dnf errors. 2. Consume and Download test cases are now using the host defined in the new `content` role if defined. Depending on: pulp/pulp-smash#1166 Closes #4320 https://pulp.plan.io/issues/4320
rochacbruno
added a commit
to rochacbruno/pulp_rpm
that referenced
this pull request
Feb 12, 2019
1. gen_yum_config_file was generating repo file using `:` intead of `=` and that was causing dnf errors. 2. Consume and Download test cases are now using the host defined in the new `content` role if defined. Depending on: pulp/pulp-smash#1166 closes: #4320 https://pulp.plan.io/issues/4320
rochacbruno
added a commit
to rochacbruno/pulp_rpm
that referenced
this pull request
Feb 12, 2019
1. gen_yum_config_file was generating repo file using `:` intead of `=` and that was causing dnf errors. 2. Consume and Download test cases are now using the host defined in the new `content` role if defined. Depending on: pulp/pulp-smash#1166 closes: #4320 https://pulp.plan.io/issues/4320
rochacbruno
added a commit
to rochacbruno/pulp_file
that referenced
this pull request
Feb 13, 2019
Relates to: pulp/pulp-smash#1166 [noissue]
rochacbruno
added a commit
to rochacbruno/pulp_python
that referenced
this pull request
Feb 13, 2019
Relates to: pulp/pulp-smash#1166 [noissue]
rochacbruno
added a commit
to rochacbruno/pulp_file
that referenced
this pull request
Feb 13, 2019
Required PR: pulp/pulp-smash#1169 Relates to: pulp/pulp-smash#1166 [noissue]
daviddavis
pushed a commit
to daviddavis/pulp_file
that referenced
this pull request
May 14, 2019
Required PR: pulp/pulp-smash#1169 Relates to: pulp/pulp-smash#1166 [noissue]
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added
content
as a valid role for config hostsAdded helper methods in
cfg
to get content_host Base URL.Added
content
as a new option onpulp-smash settings create
Note: In cases where
content
role is not defined then theapi
host is assumed as the content host.
https://docs.pulpproject.org/en/3.0/nightly/components.html#content-serving-application