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

Mac PW Pool: Restrict software installation/updates #197

Merged
merged 2 commits into from
May 29, 2024

Conversation

cevich
Copy link
Member

@cevich cevich commented May 28, 2024

For whatever reason, non-admin users are permitted to install and update software on Macs by default. This is highly undesirable in a CI environment, and especially so in one where the underlying resources are shared across testing contexts. Block this by altering system settings to require admin access.

Further through experimentation, it was found that rosetta (allows arm64 macs to run x86_64 code) ignores the admin-required settings. To give pause to any users trying to run softwareupdate, move it out of general reach. This isn't a perfect solution, but should at least discourage all simple usage.

Copy link

github-actions bot commented May 28, 2024

Successfully triggered github-actions/success task to indicate successful run of cirrus-ci_retrospective integration and unit testing from this PR's 52f0c1853e4f7cb6d4a9181dc5578732dfa732d6.

@cevich cevich marked this pull request as ready for review May 28, 2024 20:20
Copy link

github-actions bot commented May 28, 2024

Successfully triggered github-actions/success task to indicate successful run of cirrus-ci_retrospective integration and unit testing from this PR's 330ff4ab78cd4544c5a4e6029e9616fa2221b1df.

Specifically, detail the manual testing steps.

Signed-off-by: Chris Evich <cevich@redhat.com>
For whatever reason, non-admin users are permitted to install and update
software on Macs by default.  This is highly undesirable in a CI
environment, and especially so in one where the underlying resources are
shared across testing contexts.  Block this by altering system settings
to require admin access.

Further through experimentation, it was found that rosetta (allows arm64
macs to run x86_64 code) ignores the admin-required settings.  To give
pause to any users trying to run `softwareupdate`, move it out of general
reach.  This isn't a perfect solution, but should at least discourage all
simple usage.

Signed-off-by: Chris Evich <cevich@redhat.com>
Copy link

github-actions bot commented May 29, 2024

Successfully triggered github-actions/success task to indicate successful run of cirrus-ci_retrospective integration and unit testing from this PR's 3a39b5cafc5fc1cc6c2d9e31a771f6f8386cce23.

@cevich cevich merged commit 0eb6675 into containers:main May 29, 2024
9 checks passed
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

1 participant