Skip to content

(maint) Merge 6.x to main #8898

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

Merged
merged 6 commits into from
Mar 18, 2022
Merged

Conversation

joshcooper
Copy link
Contributor

No description provided.

joshcooper and others added 6 commits March 15, 2022 17:14
Starting with macOS 10.14 Mojave, Apple require full disk access
(FDA) to make changes to certain system functionality, including
user ID (UID) and the `/home` directory. Granting FDA means either
manual user intervention or creating a Privacy Preferences
Policy Control Payload (PPPCP) profile through mobile device
management (MDM).

This means that these two tests (`should_modify_home.rb` and
`should_modify_uid.rb`) don't work on any version of macOS that
Puppet supports (10.14-11), so we no longer need them.
This test excludes all versions of macOS starting from 10.15
Catalina. As Puppet dropped macOS 10.14 Mojave support last year,
we no longer have to use a regex matcher as every supported
version of macOS can be excluded.
* upstream/6.x:
  (maint) Expands test to include macOS 12
  (maint) Removes irrelevant macOS tests
  (maint) Remove houndci config

Conflicts:
	acceptance/tests/resource/user/should_modify_home.rb
	acceptance/tests/resource/user/should_modify_uid.rb

Deleted files
@joshcooper joshcooper requested a review from a team as a code owner March 18, 2022 03:59
@joshcooper joshcooper requested a review from a team March 18, 2022 03:59
@joshcooper joshcooper merged commit 1b8c8a1 into puppetlabs:main Mar 18, 2022
@joshcooper joshcooper deleted the merge_6x_main_mar17 branch March 18, 2022 16:15
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.

3 participants