Skip to content

Comments

continue running the script even if management action hangs#808

Merged
scriptingosx merged 1 commit intoInstallomator:devfrom
dnikles:avoid-notification-hang
Jan 21, 2023
Merged

continue running the script even if management action hangs#808
scriptingosx merged 1 commit intoInstallomator:devfrom
dnikles:avoid-notification-hang

Conversation

@dnikles
Copy link
Contributor

@dnikles dnikles commented Dec 13, 2022

I've seen this happen in several logs

2022-12-03 09:13:11 : INFO  : firefoxpkg : notifying
2022-12-13 08:18:36 : INFO  : firefoxpkg : App not closed, so no reopen.
2022-12-13 08:18:36 : REQ   : firefoxpkg : All done!
2022-12-13 08:18:36 : REQ   : firefoxpkg : ################## End Installomator, exit code 0

Installomator notifies that the app is installed, and just hangs there, preventing any further jamf policies from running. In this example, 10 days later I killed "Management Action" and it proceeded to finish the script. Adding the & when Management Action is called should prevent this possible hang.
This won't prevent Management Action from hanging, but will prevent Installomator from hanging and preventing further jamf policies from running.

I've seen this happen in several logs
```
2022-12-03 09:13:11 : INFO  : firefoxpkg : notifying
2022-12-13 08:18:36 : INFO  : firefoxpkg : App not closed, so no reopen.
2022-12-13 08:18:36 : REQ   : firefoxpkg : All done!
2022-12-13 08:18:36 : REQ   : firefoxpkg : ################## End Installomator, exit code 0
```
Installomator notifies that the app is installed, and just hangs there, preventing any further jamf policies from running. In this example, 10 days later I killed "Management Action" and it proceeded to finish the script. Adding the `&` when Management Action is called should prevent this possible hang.
@scriptingosx scriptingosx added this to the v11 milestone Jan 6, 2023
@scriptingosx scriptingosx changed the base branch from main to dev January 21, 2023 08:28
@scriptingosx scriptingosx merged commit acff814 into Installomator:dev Jan 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants