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

Modulesync: use voxpupuli helper gems #734

Merged
merged 2 commits into from
Apr 14, 2020

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Apr 11, 2020

No description provided.

spec/spec_helper_acceptance.rb Outdated Show resolved Hide resolved
@mmoll
Copy link
Contributor

mmoll commented Apr 13, 2020

please rebase :)

@ekohl ekohl force-pushed the voxpupuli-helpers branch 2 times, most recently from 643e3a7 to 46f4128 Compare April 13, 2020 15:35
@@ -308,7 +308,7 @@
# it reports its $osreleasemajor as 2, not 6.
# thats why we're matching for '2' in both parts
# Amazon Linux is like RHEL6 but reports its osreleasemajor as 2017 or 2018.
$osreleasemajor = regsubst($::operatingsystemrelease, '^(\d+)\..*$', '\1') # workaround for the possibly missing operatingsystemmajrelease
$osreleasemajor = regsubst($facts['os']['release']['full'], '^(\d+)\..*$', '\1') # workaround for the possibly missing operatingsystemmajrelease
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure about this, #694 was intended to solve it in a cleaner way.

@mmoll mmoll merged commit 46c4031 into theforeman:master Apr 14, 2020
@mmoll
Copy link
Contributor

mmoll commented Apr 14, 2020

merged, bedankt @ekohl!

@ekohl ekohl deleted the voxpupuli-helpers branch April 14, 2020 23:14
@wbclark wbclark linked an issue May 12, 2020 that may be closed by this pull request
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.

Use modern facts
4 participants