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

(maint) Update supported platforms #92

Merged
merged 3 commits into from
Mar 16, 2016
Merged

Conversation

MikaelSmith
Copy link
Contributor

Updates supported platforms for new package releases, AIX PowerPC 8
hardware, and platforms that were added but not noted as supported.

Note that specific upgrade scenarios will still depend on Puppet being
installed from packages prior to upgrade. puppet-agent packages should
exist for all supported platforms.

Michael Smith added 3 commits March 14, 2016 18:50
Updates supported platforms for new package releases and platforms that
were added but not noted as supported.

Note that specific upgrade scenarios will still depend on Puppet being
installed from packages prior to upgrade. `puppet-agent` packages should
exist for all supported platforms.
Checking for osfamily == Amazon was a mistake, the osfamily will be
RedHat. Repo names on Amazon instances need to be handled specially;
they use EL 6 repos. Adds testing for Amazon OS.
Clarifies Puppet 3 version requirements.
@@ -44,8 +44,6 @@
} elsif $::operatingsystem == 'aix' and $::architecture =~ /PowerPC_POWER[5,6,7]/ {
$aix_ver_number = regsubst($::platform_tag,'aix-(\d+\.\d+)-power','\1')
$_package_file_name = "${puppet_agent::package_name}-${puppet_agent::params::master_agent_version}-1.aix${aix_ver_number}.ppc.rpm"
} elsif $::operatingsystem == 'Darwin' and $::macosx_productversion_major =~ /10\.[9,10,11]/ {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This section was redundant, see lines 42-43 above.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hm, must have been a bad re-base.

@highb
Copy link
Contributor

highb commented Mar 16, 2016

👍 sane changes to me, travis is passing. We just need to ensure that we test the module out on AWS AMI images at some point.

@@ -116,6 +112,7 @@ Specifically in the 1.1.0 Release:
* For Windows, you must trigger an agent run after upgrading so that Puppet can create the necessary directory structures.
* Upgrading from 2015.2.x to 2015.3.x is not yet supported.
* Solaris 11 is not yet supported.
* AIX package names are based on PowerPC architecture version. PowerPC 8 is not yet supported.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We could fix this by dropping the exclusion of PowerPC_POWER8. Happy to do that here or another PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, either is fine to me. Maybe lets get it in another PR so we don't overload this one.

@MikaelSmith
Copy link
Contributor Author

I tested the Amazon AMI manually for OSS. It's a challenge to setup, they don't provide ruby(selinux), and the packages assume Ruby 1.8, so you have to mess with the environment a lot and manually install from RPMs. It also won't upgrade from Puppet3/Facter2, which are the Amazon-provided packages. So it doesn't really work for OSS, but that's not new.

I expect PE works, but that requires setting up a PE master in AWS to test against.

@highb
Copy link
Contributor

highb commented Mar 16, 2016

👍 Let's just make another PR for the Power8 change.

highb added a commit that referenced this pull request Mar 16, 2016
(maint) Update supported platforms
@highb highb merged commit c9b463f into puppetlabs:master Mar 16, 2016
@MikaelSmith MikaelSmith deleted the support branch March 17, 2016 23:14
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.

2 participants