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

ci: fix nix sandbox filename by adding random string #13306

Merged
merged 1 commit into from
Apr 27, 2022

Conversation

jakubgs
Copy link
Member

@jakubgs jakubgs commented Apr 26, 2022

By using just the timestamp we increase the probability of hitting a race condition with another build due to same filename of sandbox file. Fixes the following error:

+ rm /tmp/nix-env-220426-120032.tmp
rm: cannot remove '/tmp/nix-env-220426-120032.tmp': No such file or directory

https://ci.status.im/job/status-react/job/platforms/job/android/529

Depends on: https://github.com/status-im/status-jenkins-lib/pull/40

@jakubgs jakubgs requested a review from a team April 26, 2022 12:38
@jakubgs jakubgs self-assigned this Apr 26, 2022
@status-github-bot status-github-bot bot added this to REVIEW in Pipeline for QA Apr 26, 2022
@status-im-auto
Copy link
Member

status-im-auto commented Apr 26, 2022

Jenkins Builds

Click to see older builds (12)
Commit #️⃣ Finished (UTC) Duration Platform Result
9bc3a54 #1 2022-04-26 12:41:05 ~1 min android-e2e 📄log
9bc3a54 #1 2022-04-26 12:41:13 ~2 min android 📄log
✔️ 9bc3a54 #1 2022-04-26 12:51:34 ~12 min ios 📦ipa 📲
4f409bc #2 2022-04-27 07:36:02 ~1 min android-e2e 📄log
4f409bc #2 2022-04-27 07:36:02 ~1 min android 📄log
✔️ 4f409bc #2 2022-04-27 07:45:45 ~11 min ios 📦ipa 📲
a711d04 #3 2022-04-27 07:46:00 ~1 min android-e2e 📄log
a711d04 #3 2022-04-27 07:46:02 ~1 min android 📄log
✔️ a711d04 #3 2022-04-27 07:56:44 ~11 min ios 📦ipa 📲
✔️ 5c06b7d #4 2022-04-27 07:58:04 ~11 min ios 📦ipa 📲
✔️ 5c06b7d #4 2022-04-27 07:58:26 ~11 min android-e2e 📦apk 📲
✔️ 5c06b7d #4 2022-04-27 07:58:42 ~11 min android 📦apk 📲
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ b883357 #5 2022-04-27 08:10:45 ~9 min android-e2e 📦apk 📲
✔️ b883357 #5 2022-04-27 08:11:26 ~10 min android 📦apk 📲
✔️ b883357 #5 2022-04-27 08:16:32 ~15 min ios 📦ipa 📲
✔️ f1cd115 #6 2022-04-27 09:15:17 ~9 min android 📦apk 📲
✔️ f1cd115 #6 2022-04-27 09:15:38 ~9 min android-e2e 📦apk 📲
✔️ f1cd115 #6 2022-04-27 09:17:27 ~11 min ios 📦ipa 📲

@jakubgs jakubgs force-pushed the nix/fix-sandbox-filename branch 4 times, most recently from b883357 to f1cd115 Compare April 27, 2022 09:05
By using just the timestamp we increase the probability of hitting a
race condition with another build due to same filename of sandbox file.

Depends on: status-im/status-jenkins-lib#40

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs merged commit 2bab132 into develop Apr 27, 2022
@jakubgs jakubgs deleted the nix/fix-sandbox-filename branch April 27, 2022 11:07
Pipeline for QA automation moved this from REVIEW to DONE Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants