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

Control qpid-proton rpm version by installing from ManageIQ copr repo #409

Merged
merged 1 commit into from
Apr 8, 2020

Conversation

simaishi
Copy link
Contributor

@simaishi simaishi commented Apr 7, 2020

Disallow installing qpid-proton from epel repo (so it will come from ManageIQ copr repo), to ensure the version matches with the gem version.

Related to: ManageIQ/manageiq#19942 and ManageIQ/manageiq#20016

Building 0.30.0 rpm in ManageIQ-Master and ManageIQ-Jansa copr repo now. Until ManageIQ/manageiq#20016 is merged, bundle will fail on master/jansa appliances with or without this PR.

…ManageIQ

copr repo), to ensure the version matches with the gem version
@miq-bot
Copy link
Member

miq-bot commented Apr 7, 2020

Checked commit simaishi@8aca676 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@carbonin carbonin merged commit 9b72794 into ManageIQ:master Apr 8, 2020
@simaishi simaishi deleted the lock_qpid_rpm branch April 8, 2020 14:06
simaishi pushed a commit that referenced this pull request Apr 16, 2020
Control qpid-proton rpm version by installing from ManageIQ copr repo

(cherry picked from commit 9b72794)
@simaishi
Copy link
Contributor Author

Jansa backport details:

$ git log -1
commit f623eb899e413c17285243bf30b1127b53001d48
Author: Nick Carboni <[email protected]>
Date:   Wed Apr 8 09:20:43 2020 -0400

    Merge pull request #409 from simaishi/lock_qpid_rpm

    Control qpid-proton rpm version by installing from ManageIQ copr repo

    (cherry picked from commit 9b72794803c62527f14ad317c9c165aba165f6a9)

@jrafanie jrafanie mentioned this pull request Nov 30, 2023
7 tasks
jrafanie added a commit to jrafanie/manageiq-appliance-build that referenced this pull request Dec 1, 2023
Undo ManageIQ#409 now that
we're ready to use the upstream packages in el8 or el9 epel qpid proton 0.37.0+

To be merged with ManageIQ/manageiq#22271
Prerequisite to ManageIQ/manageiq#22698
Needed for ManageIQ/manageiq#22696
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants