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

Add Pre-flight Check for OS version #31

Closed
mani2care opened this issue Jan 21, 2023 · 17 comments
Closed

Add Pre-flight Check for OS version #31

mani2care opened this issue Jan 21, 2023 · 17 comments
Assignees
Labels
enhancement New feature or request testing needed On-hold, awaiting feedback

Comments

@mani2care
Copy link

Hi Dan , really wonderful script to onboard the macOS via enrolment trigger or DEP .

one consent my location users macs are BYOD setup and some of talented users are having old version of Mac OS catalina and below. They will not read the instructions during the enrolment. Sometimes users are enrolled to JAMF and setup your mac script will be failed due to swift dialog compatibility.

Might be your able to add the limitation it will help us to use it without any error.
If in case users is old version just to run a policy trigger and exit with JAMF dialog as complete.

2)And we need a built in automation for the swift dialog version validation. If in case I am using old versions of swifdialog and install it latest version so it will help us to additional effort to do
Swiftdialogversion =$8any version /Autoupdate get always latest xyz version.

@mani2care mani2care added the enhancement New feature or request label Jan 21, 2023
@dan-snelson
Copy link
Owner

Thanks for the double Feature Request, @mani2care; will you please open a separate issue for Item No. 2?

As discussed in the MacAdmins Slack #setup-your-mac Channel, we're handling this use-case at the Jamf Pro Policy level; computers which enroll with OSes of macOS 10.15.7 Catalina and earlier only see two policies:

  1. Update Inventory
  2. Upgrade to macOS Ventura

In the meantime, you're welcome to use this and this.

@dan-snelson dan-snelson changed the title Restructuring macOS to avoid the error setup your mac script Add Pre-flight Check for OS version Jan 21, 2023
@dan-snelson
Copy link
Owner

Revised Issue Title

  • From: Restructuring macOS to avoid the error setup your mac script
  • To: Feature Request: Add Pre-flight Check for OS version

@dan-snelson
Copy link
Owner

@mani2care:

Please test-drive 1.7.0-rc4 (which is not production-ready):

Screen Shot 2023-01-21 at 10 48 01 AM

@dan-snelson dan-snelson added the testing needed On-hold, awaiting feedback label Jan 21, 2023
Copy link

I also think macOS validation would be awesome since we’re constantly chasing zero days and need to make sure new builds are at latest level.

@dan-snelson
Copy link
Owner

@SirDrSpiderpig: Currently, the check is just to confirm the Mac can run swiftDialog.

@dan-snelson
Copy link
Owner

@mani2care and @SirDrSpiderpig:

Please test-drive 1.7.0-rc5 which introduces the ability to specify a requiredMinimumBuild.

Screen Shot 2023-01-25 at 7 13 56 AM

Screenshot 2023-01-25 at 7 14 35 AM

@SirDrSpiderpig
Copy link

Will do and report back.

@mani2care
Copy link
Author

mani2care commented Jan 25, 2023 via email

@SirDrSpiderpig
Copy link

Downloaded it now and will tear on Jamf cloud beta instance.

If I can suggest to make the requireMinimumbuild variable added as part of the script variables in Jamf so it can be updated by the policy as opposed to going into the code. Just like debug and welcome screen are.

Will report back after testing

@dan-snelson
Copy link
Owner

Thanks for the prompt feedback, @SirDrSpiderpig.

How frequently do you see yourself updating requiredMinimumBuild ?

@SirDrSpiderpig
Copy link

Anytime there is a zero day. This year we’ve have plenty of those that kept us scrambling

@dan-snelson
Copy link
Owner

Ah, OK.

Currently, requiredMinimumBuild is at Line 44 and the Script Parameters don't start until Line 148.

@dan-snelson
Copy link
Owner

@SirDrSpiderpig:

In case it helps, 1.7.0-rc6 includes a yet-to-be-tested requiredMinimumBuild="${8:-"22D"}".

@dan-snelson
Copy link
Owner

Catalina

Screen Shot 2023-01-25 at 9 42 52 AM

Screen Shot 2023-01-25 at 9 42 57 AM

Ventura

Screenshot 2023-01-25 at 9 43 13 AM

Screenshot 2023-01-25 at 9 43 23 AM

@SirDrSpiderpig
Copy link

Looking good!

@dan-snelson
Copy link
Owner

Just uploaded a screencast for 1.7.0's new Outdated OS Action.

@dan-snelson
Copy link
Owner

Addressed in 1.7.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request testing needed On-hold, awaiting feedback
Projects
None yet
Development

No branches or pull requests

3 participants