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

3.1 support and updates #75

Merged
merged 18 commits into from
Dec 23, 2015
Merged

Conversation

detiber
Copy link
Contributor

@detiber detiber commented Dec 19, 2015

No description provided.

@detiber
Copy link
Contributor Author

detiber commented Dec 19, 2015

addresses: #74 #73 #72

@detiber
Copy link
Contributor Author

detiber commented Dec 19, 2015

@ryanj This should address the issues you've been running into.

@detiber
Copy link
Contributor Author

detiber commented Dec 22, 2015

@thoraxe @ryanj latest update to this should fix the issue with re-registration on subsequent runs.

- Rebase off of latest upstream changes
- Added ability to subscribe based on a list of pool_ids
- Fixed bug where module wasn't enabling the subscription_management yum
  plugin
- Fixed error where pools were being re-subscribed every subsequent run
@ryanj
Copy link
Contributor

ryanj commented Dec 22, 2015

👍 This is working really well. I'm in favor of merging this work into 'master'

@thoraxe
Copy link
Contributor

thoraxe commented Dec 23, 2015

I was finally able to make this work myself. The issues I had were not related to the demo-ansible script, but rather to problems with Docker starting which subsequently failed the rest of the provisioning stuff. I think this is probably good to go.

detiber added a commit that referenced this pull request Dec 23, 2015
@detiber detiber merged commit 50088b6 into 2015-Middleware-Keynote:master Dec 23, 2015
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

3 participants