-
Notifications
You must be signed in to change notification settings - Fork 81
Add variables for molecule prepare.yml to clone the branches #272
Conversation
Required PR: pulp/pulp_installer#272 re: #6547 [noissue]
d14fb80
to
a4a957c
Compare
…ey test against fixes: #6548 Required PR: pulp/pulp_installer#272
Does it still WIP? |
Kind of. I'm double-checking my work now (for proper design, and correctness. And checking CI output) before I ask you to review. Thank you for being so attentive. EDIT: I should also create another PR for pulp_rpm_prerequisites with branch 3.3 variables. I should technically create a branch anyway, but I need it to test this PR. |
1aaf7a3
to
0aae56b
Compare
So we can manually verify that the correct versions were installed. Also verifies that the status page is accessible; that pulpcore-api and nginx are basically working. fixes: #6585
giving it the capability to do mounts like PrivateTmp re: #6586 [noissue]
0aae56b
to
484a708
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is just a matter of rebasing
# pulp_install_plugins has the following molecule (CI) only subfields: | ||
# item.value.git_repo: The git repo to clone from for source test scenarios. | ||
# item.value.git_commitish: The commitish (branch, tag) to checkout for source test scenarios. | ||
# | ||
# pulp_install_plugins has the following not advertised subfield because it is | ||
# only used for CI purposes currently, and the related logic (conflicts with | ||
# version) are not done yet: | ||
# item.value.pip_version_specifier: One of more pip version specifiers, | ||
# separated by commas. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would move these comments to somewhere under molecule
dir
Closing this for now. Let's get back to this when we have time. |
fixes: #6547