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

Cleanup install on ubuntu runners #730

Merged
merged 9 commits into from
Apr 5, 2024
Merged

Conversation

TylerJang27
Copy link
Collaborator

@TylerJang27 TylerJang27 commented Apr 5, 2024

Fix failures introduced by php setup. Successful run

Copy link

trunk-io bot commented Apr 5, 2024

⏱️ 1h 36m total CI duration on this PR
Job Cumulative Duration Recent Runs
CodeQL-Build 11m 🟩🟩🟩🟩🟩 (+2 more)
Windows Linter Tests 7m 🟥🟥🟩
Plugin Tests Main (Latest, macOS) 7m
Plugin Tests Main (Snapshots, macOS) 6m
Plugin Tests Main (Latest, windows-latest) 6m
Plugin Tests Release (Latest, windows-latest) 6m
Plugin Tests Main (Snapshots, windows-latest) 6m
Plugin Tests Release (Snapshots, windows-latest) 6m
Tool Tests Main (windows-latest) 5m 🟥
Trunk Check runner [linux] 5m 🟩🟩🟩🟩🟩 (+2 more)
Plugin Tests Release (Latest, macOS) 5m 🟥
Linter Tests macOS 4m 🟩🟩🟩🟩
Repo Tests / Plugin Tests 3m 🟩🟩🟩🟩🟩 (+2 more)
Linter Tests Linux 3m 🟥🟩🟩🟩
Linter Tests 2 macOS 2m 🟩🟩
Tool Tests Main (ubuntu-x64) 2m 🟩
Linter Tests 3 macOS 2m 🟩🟩
Linter Tests 2 Linux 2m 🟩🟩
Plugin Tests Release (Snapshots, macOS) 2m
Linter Tests 3 Linux 2m 🟩🟩
Upload Linter Test Results / Generate Snapshots PR 1m 🟩
Plugin Tests Main (Snapshots, ubuntu-x64) 1m 🟥
Action Tests Main 44s 🟩
Detect changed files 35s 🟩🟩🟩🟩🟩 (+2 more)
Repo Tests / Plugin Tests 27s 🟩
Upload Linter Test Results / Upload Test Results 26s 🟩
Upload Tool Test Results / Upload Test Results 22s 🟩
Aggregate Test Results 16s 🟥🟩🟩🟩🟩 (+2 more)
Plugin Tests Release (Latest, ubuntu-x64) 14s 🟥
Plugin Tests Release (Snapshots, ubuntu-x64) 13s 🟥
Plugin Tests Main (Latest, ubuntu-x64) 9s 🟥
Tool Tests Main (macOS) 1s

settingsfeedbackdocs ⋅ learn more about trunk.io

Copy link

github-actions bot commented Apr 5, 2024

Breakpoint trunk-plugins-linter-tests was triggered on exit. Click here to connect to the session.

Copy link

github-actions bot commented Apr 5, 2024

Breakpoint trunk-plugins-linter-tests was triggered on exit. Click here to connect to the session.

@@ -44,6 +44,11 @@ runs:

case "$RUNNER_OS" in
Linux)
# Cleanup any existing apt locks
sudo killall apt-get || true
sudo rm /var/lib/apt/lists/lock || true
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was a 2-day old apt-get process that still had the lock on one of the runners. This should rarely/never happen, so just clean up as part of the action runner process, because it will fail otherwise anyway

@TylerJang27 TylerJang27 marked this pull request as ready for review April 5, 2024 21:41
@TylerJang27 TylerJang27 requested review from laurit17 and det April 5, 2024 21:41
@TylerJang27 TylerJang27 merged commit 0cfcdb0 into main Apr 5, 2024
13 checks passed
@TylerJang27 TylerJang27 deleted the tyler/cleanup-install branch April 5, 2024 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants