Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add RHEL7 support for PE #57

Merged
merged 2 commits into from Mar 4, 2014

Conversation

Projects
None yet
5 participants
Contributor

mckern commented Feb 26, 2014

This currently requires more hackey bandaid logic (à la what we did for the FOSS mocks for EL7). We'll address this once EL7 is released.

@mckern mckern Add RHEL7 support for PE
This currently requires more hackey bandaid logic (à la what we did
for the FOSS mocks for EL7). We'll address this once EL7 is released.
c7ad72f
Member

melissa commented Feb 26, 2014

👍 However, what if we updated this to have a generic "beta" template, instead of the rhel7 bandaid? I'm not sure if that's worth the time, but that way there'd be less to add in and pull out every time something goes in/out of beta

Contributor

mckern commented Feb 26, 2014

Eh. Beta URLs are pretty much always going to be whatever sub-URI the maintainers got to stick to the wall. I don't mind the bandaid approach since new OS platforms don't come out that often.

@stahnma stahnma commented on an outdated diff Feb 26, 2014

templates/pe-el7-bandaid.erb
+debuglevel=1
+reposdir=/dev/null
+logfile=/var/log/yum.log
+retries=20
+obsoletes=1
+gpgcheck=0
+assumeyes=1
+syslog_ident=mock
+syslog_device=
+proxy=http://proxy.puppetlabs.lan:3128/
+
+# repos
+[beta-<%=@dist%>-<%=@release%>-<%=@arch%>]
+name=centos<%=@release%>-<%=@arch%>-os
+enabled=1
+baseurl=http://ftp.redhat.com/redhat/rhel/beta/7/x86_64/os/
@stahnma

stahnma Feb 26, 2014

Member

Why wouldn't we use our internal mirror?

@mckern mckern Update URL for RHEL7 PE mocks
Changed it to reference internal RHEL7 mirror.
5747eac
Contributor

mckern commented Feb 26, 2014

@stahnma Mirrors! We has 'em!

Member

haus commented Feb 26, 2014

👍

Member

stahnma commented Feb 26, 2014

So now comes the next question.... is this module designed for public consumption? I think so. If so, why do we have neptune in the list? Basically, is this for internal PL or public usage.

Member

haus commented Feb 26, 2014

The foss builder logic is absolutely for public consumption, so the FOSS mocks should be hitting the redhat repos, and not the mirror.

Contributor

mckern commented Feb 26, 2014

FOSS mock reset to point to public repo.

Member

melissa commented Mar 4, 2014

👍

@shrug shrug added a commit that referenced this pull request Mar 4, 2014

@shrug shrug Merge pull request #57 from mckern/pe_on_rhel7
Add RHEL7 support for PE
dca5fa7

@shrug shrug merged commit dca5fa7 into puppetlabs:master Mar 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment