Skip to content

[PE-42706]: Add support for installing Puppet agent on Solaris 10 SPARC hosts#289

Merged
joshcooper merged 1 commit into
mainfrom
PE-42706-update-beaker-pe-gem-to-be-able-to-install-solaris-10-sparc-puppet-agent-via-agent-downloads
Nov 19, 2025
Merged

[PE-42706]: Add support for installing Puppet agent on Solaris 10 SPARC hosts#289
joshcooper merged 1 commit into
mainfrom
PE-42706-update-beaker-pe-gem-to-be-able-to-install-solaris-10-sparc-puppet-agent-via-agent-downloads

Conversation

@span786

@span786 span786 commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

What's this PR do?

This PR introduces support for installing the solaris-10-sparc Puppet agent directly via agent-downloads. This approach is necessary because the build is not part of a standard release and was created for a specific customer. These changes allow us to validate the solaris-10-sparc Puppet agent with PE releases without publishing it to the general agent distribution locations.
Changes in this PR are similar to this PR #264 which was done for SLES 11 (x86_64) platform

Should any of this be tested outside the normal PR CI cycle?

No

@span786 span786 requested review from a team as code owners November 11, 2025 13:32
@span786 span786 force-pushed the PE-42706-update-beaker-pe-gem-to-be-able-to-install-solaris-10-sparc-puppet-agent-via-agent-downloads branch from 77adda8 to 32c98f1 Compare November 14, 2025 07:16
@span786 span786 force-pushed the PE-42706-update-beaker-pe-gem-to-be-able-to-install-solaris-10-sparc-puppet-agent-via-agent-downloads branch from 32c98f1 to de455bf Compare November 14, 2025 09:21

@joshcooper joshcooper left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM though it seems some of the checks are still failling. Not sure if those are broken and it's ok to merge? Thoughts @steveax

@span786

span786 commented Nov 18, 2025

Copy link
Copy Markdown
Contributor Author

LGTM though it seems some of the checks are still failling. Not sure if those are broken and it's ok to merge? Thoughts @steveax

The acceptance tests are failing because the system cannot download the puppet-agent due to the absence of a valid license. Below is the error snippet:

15:09:00 Last 10 lines of output were:
15:09:00 	No provider of 'puppet-agent=8.15.0.65.gd2a4b575d' found.
15:09:00 	+ '[' 104 -ne 0 ']'
15:09:00 	+ echo 'Failed to install puppet-agent. Please check that you have a PE license. Aborting...'
15:09:00 	Failed to install puppet-agent. Please check that you have a PE license. Aborting...
15:09:00 	+ exit 1
15:09:00 	+ remove_repo
15:09:00 	+ zypper rr puppet-enterprise
15:09:00 	Removing repository 'puppet-enterprise' [...done]
15:09:00 	Repository 'puppet-enterprise' has been removed.
15:09:00 	Error running install script /tmp/tmp.ruMXYdHTDz

@steveax steveax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should fix those tests by augmenting them with a license step, but seems like time is of the essence here so 👍

@steveax

steveax commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Opened PE-42803 to cover fixing those tests (they've been broken since February 2025.)

@joshcooper joshcooper merged commit cbd80e7 into main Nov 19, 2025
2 of 3 checks passed
@joshcooper joshcooper deleted the PE-42706-update-beaker-pe-gem-to-be-able-to-install-solaris-10-sparc-puppet-agent-via-agent-downloads branch November 19, 2025 23:30
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.

3 participants