wooga / atlas-build-unity

92%

master: 78%

LAST BUILD ON BRANCH PR-111

branch: PR-111

CHANGE BRANCH

x

Reset

• PR-111
• 2692491593e162389c4d535c3a47d57a34e23640
• PR-10
• PR-100
• PR-101
• PR-102
• PR-103
• PR-104
• PR-106
• PR-107
• PR-11
• PR-113
• PR-114
• PR-115
• PR-117
• PR-118
• PR-119
• PR-12
• PR-120
• PR-122
• PR-124
• PR-125
• PR-126
• PR-127
• PR-128
• PR-129
• PR-13
• PR-130
• PR-131
• PR-133
• PR-134
• PR-135
• PR-136
• PR-137
• PR-139
• PR-14
• PR-140
• PR-141
• PR-143
• PR-144
• PR-145
• PR-146
• PR-147
• PR-148
• PR-150
• PR-152
• PR-154
• PR-16
• PR-17
• PR-18
• PR-20
• PR-21
• PR-24
• PR-25
• PR-26
• PR-27
• PR-28
• PR-29
• PR-30
• PR-31
• PR-32
• PR-35
• PR-36
• PR-37
• PR-39
• PR-40
• PR-42
• PR-44
• PR-45
• PR-46
• PR-48
• PR-49
• PR-50
• PR-53
• PR-54
• PR-55
• PR-56
• PR-57
• PR-58
• PR-59
• PR-6
• PR-60
• PR-61
• PR-62
• PR-63
• PR-64
• PR-65
• PR-66
• PR-67
• PR-68
• PR-7
• PR-70
• PR-71
• PR-72
• PR-73
• PR-74
• PR-75
• PR-76
• PR-77
• PR-78
• PR-79
• PR-8
• PR-80
• PR-81
• PR-82
• PR-84
• PR-85
• PR-86
• PR-87
• PR-88
• PR-89
• PR-9
• PR-90
• PR-91
• PR-92
• PR-93
• PR-94
• PR-95
• PR-96
• PR-97
• PR-98
• PR-99
• master
• release/1.x

Improve keychain cleanup after gradle process kill

Description
===========

This patch adds a shutdown handler to the `net.wooga.build-io` plugin
to cleanup created build keychains in case of abrupt build process termination.
It uses the `java.lang.Runtime#addShutdownHook` API and registeres a hook
after the `addKeychain` task was executed and removes said hook
when `removeKeychian` was executed. These shutdown hooks run even during normal
termination so it is crucial to only do the work when needed.

I setup a tests that runs a build with a very long task execution in a forked jvm.
The test framework will spawn a new gradle deamon which we will pin to a different
gradle version to find in the process list of macOS. The tests will run with
multiple kill signals to verify the shutdown hook execution.

This whole logic only works for normal kill signals like `TERM` but won't work
for a hard `KILL` signal.

I though about to add more tests to check if the behavior of adding and removing
the shutdown hook works as expected. But in the end it is only important that it
runs correctly during these process shutdown scenarios and that the plugin behaves
normal under normal execution which the other tests already cover.

Changes
=======

* ![IMPROVE] ![MACOS] keychain cleanup after gradle process kill