Skip to content

Sometimes, WKTR installation does not succeed in the Simulator#35709

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pvollan:eng/Sometimes-WKTR-installation-does-not-succeed-in-the-Simulator
Oct 25, 2024
Merged

Sometimes, WKTR installation does not succeed in the Simulator#35709
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pvollan:eng/Sometimes-WKTR-installation-does-not-succeed-in-the-Simulator

Conversation

@pvollan
Copy link
Copy Markdown
Contributor

@pvollan pvollan commented Oct 25, 2024

642ca04

Sometimes, WKTR installation does not succeed in the Simulator
https://bugs.webkit.org/show_bug.cgi?id=281875
rdar://138351968

Reviewed by Elliott Williams.

This is caused by a daemon involved in the installation being suspended. This patch works around this
issue by creating a small utility that sends a XPC message to the daemon in order to resume execution.
This issue is flaky, so it is hard to confirm the fix is sufficient, but local testing looks good.

This patch was previously reviewed in #35544 and landed in
https://commits.webkit.org/285637@main. It is being relanded with a build fix for Debug mode.

* Tools/Scripts/webkitpy/xcode/simulated_device.py:
(SimulatedDevice.install_app):
* Tools/WebKitTestRunner/Configurations/WebKitEligibilityUtil.xcconfig: Added.
* Tools/WebKitTestRunner/WebKitEligibilityUtil/main.cpp: Added.
(main):
* Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:

Canonical link: https://commits.webkit.org/285704@main

601d5f6

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 win
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug 🧪 wpe-wk2 ✅ 🧪 win-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 🧪 api-mac 🧪 api-wpe
✅ 🧪 webkitpy ✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🛠 wpe-cairo
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🧪 api-gtk
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 🧪 unsafe-merge ✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@pvollan pvollan force-pushed the eng/Sometimes-WKTR-installation-does-not-succeed-in-the-Simulator branch from 99fde22 to 9f0fae5 Compare October 25, 2024 03:55
@pvollan pvollan self-assigned this Oct 25, 2024
@pvollan pvollan added the Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases label Oct 25, 2024
@pvollan pvollan force-pushed the eng/Sometimes-WKTR-installation-does-not-succeed-in-the-Simulator branch from 9f0fae5 to 601d5f6 Compare October 25, 2024 16:00
@pvollan pvollan added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Oct 25, 2024
https://bugs.webkit.org/show_bug.cgi?id=281875
rdar://138351968

Reviewed by Elliott Williams.

This is caused by a daemon involved in the installation being suspended. This patch works around this
issue by creating a small utility that sends a XPC message to the daemon in order to resume execution.
This issue is flaky, so it is hard to confirm the fix is sufficient, but local testing looks good.

This patch was previously reviewed in WebKit#35544 and landed in
https://commits.webkit.org/285637@main. It is being relanded with a build fix for Debug mode.

* Tools/Scripts/webkitpy/xcode/simulated_device.py:
(SimulatedDevice.install_app):
* Tools/WebKitTestRunner/Configurations/WebKitEligibilityUtil.xcconfig: Added.
* Tools/WebKitTestRunner/WebKitEligibilityUtil/main.cpp: Added.
(main):
* Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:

Canonical link: https://commits.webkit.org/285704@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Sometimes-WKTR-installation-does-not-succeed-in-the-Simulator branch from 601d5f6 to 642ca04 Compare October 25, 2024 17:27
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 285704@main (642ca04): https://commits.webkit.org/285704@main

Reviewed commits have been landed. Closing PR #35709 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 642ca04 into WebKit:main Oct 25, 2024
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants