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

Problem: pulp_rpm 3.3.0 cannot install (& F32 upgrade) #657

Closed

Conversation

mikedep333
Copy link
Member

on the pulp-operator pulp container because Fedora's createrepo_c
is too old

Solution: Temporarily replace the createrepo_c RPM with its
build-dependencies, and let createrepo_c get installed &
built from PyPI.

fixes: #6523

Developed on top of this change:

containers: Upgrade to Fedora 32

fixes: #6522

@pulpbot
Copy link
Member

pulpbot commented Apr 17, 2020

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

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

goosemania added a commit to goosemania/pulp_rpm that referenced this pull request Apr 20, 2020
@dralley
Copy link
Contributor

dralley commented Apr 20, 2020

@mikedep333 Needs a changelog entry

@mikedep333 mikedep333 force-pushed the 6523_6522_container_createrepo_c_deps branch from f979e74 to b54a9fa Compare April 20, 2020 13:32
on the pulp-operator pulp container because Fedora's createrepo_c
is too old

Solution: Temporarily replace the createrepo_c RPM with its
build-dependencies, and let createrepo_c get installed &
built from PyPI.

fixes: #6523
@mikedep333 mikedep333 force-pushed the 6523_6522_container_createrepo_c_deps branch from b54a9fa to 1e61111 Compare April 20, 2020 13:36
@mikedep333 mikedep333 changed the title Problem: pulp_rpm 3.3.0 cannot install Problem: pulp_rpm 3.3.0 cannot install (& F32 upgrade) Apr 20, 2020
@mikedep333
Copy link
Member Author

This seems to be failing because of a bug in the Travis scripts: It is trying to use the Travis Python version (3.7) in the containers. But going from Fedora 30 containers to Fedora 32 means going from Python 3.7 to 3.8.

@fao89
Copy link
Member

fao89 commented Jul 16, 2020

@mikedep333 do you think we can reopen it?

@mikedep333
Copy link
Member Author

@fao89 yes, we just have to fix the bug #657 (comment)

@mikedep333 mikedep333 reopened this Jul 16, 2020
@fao89
Copy link
Member

fao89 commented Jul 16, 2020

@fao89 yes, we just have to fix the bug #657 (comment)

the problem was with unit tests here:
https://github.com/pulp/plugin_template/blob/master/templates/travis/.travis/script.sh.j2#L91
As plugins don't use operator anymore, I think the bug is already addressed

@fao89
Copy link
Member

fao89 commented Jul 17, 2020

@mikedep333 maybe it is better to move this container thing out of pulpcore, to a repo we (CI/CD team) have more control

@mikedep333
Copy link
Member Author

@fao89 The plan has been to move it out. We agreed on pulp-operator, but we need to figure out the CI details and folder structure and all. There's probably a redmine ticket.

@goosemania
Copy link
Member

@fao89 , @mikedep333 , do we still need it? or should we close it?

@fao89
Copy link
Member

fao89 commented Sep 1, 2020

@fao89 , @mikedep333 , do we still need it? or should we close it?

it was moved to pulp-operator, we can close it

@goosemania
Copy link
Member

Closing as per #657 (comment)

@goosemania goosemania closed this Sep 1, 2020
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

6 participants