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.

Sign up for GitHub

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

Jump to bottom

Add the ability to set up a Fedora 32 system for subman development #8

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add the ability to set up a Fedora 32 system for subman development #8

wants to merge 7 commits into from

Conversation

cnsnyder
Copy link
Member

This PR adds a couple things, which collectively allow for the ansible role to set up a fedora 32 system to do subscription-manager development:

  • When yarn is available via the distro's package manager (as indicated in the distro specific vars) install it from that source
  • Remove unnecessary and old dependencies for doing subscription-manager dev on fedora
  • Allow the playbook to support multiple versions of the same distro specified by distro major version. This allows different sets of distro specific variables to be specified for say fedora 32 vs fedora33 if necessary. It was implemented this way so as to not break compatibility.

@cnsnyder cnsnyder mentioned this pull request Feb 15, 2021
Merged
@cnsnyder
Updates to do dev work on our cockpit plugin
4862576 
This no longer installs yarn (as it is no longer necessary for f32).
This installs npm via rpm and install dev deps and project deps
for the mounted dev environment.
@cnsnyder cnsnyder force-pushed the csnyder/support_f32 branch from 83eb136 to 4862576 Compare February 19, 2021 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cnsnyder