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

Add apt-get update step into the clang-tidy job #79344

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

moxian
Copy link
Contributor

@moxian moxian commented Jan 26, 2025

Summary

None

Purpose of change

Clang-tidy runners are spuriously failing https://github.com/CleverRaven/Cataclysm-DDA/actions/runs/12967965940/job/36170475239?pr=79311

Describe the solution

The error message says

E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

So I thought I should maybe do that...

Describe alternatives you've considered

Unclear.
I'm not exactly sure what's the root of the problem because sometimes apt-get install works totally fine. E.g. https://github.com/CleverRaven/Cataclysm-DDA/actions/runs/12967965940/job/36170475187?pr=79311 which is part of the exact same matrix as the failing one in https://github.com/CleverRaven/Cataclysm-DDA/actions/runs/12967965940/job/36170475239?pr=79311
(I'm also a bit surprised that github actions don't come with apt-get update pre-run - doing so would save them a small amount of CPU cycles, but guess there're other complications with that)

Testing

Seems to be working in my fork https://github.com/moxian/Cataclysm-DDA/actions/runs/12970195966?pr=16
But, again, I'm not 100% sure this fixes the issue for good, or if i was just lucky here

Additional context

@github-actions github-actions bot added the Code: Tooling Tooling that is not part of the main game but is part of the repo. label Jan 26, 2025
@moxian moxian marked this pull request as draft January 26, 2025 01:27
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Jan 26, 2025
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Jan 26, 2025
@moxian moxian marked this pull request as ready for review January 26, 2025 01:35
@akrieger
Copy link
Member

akrieger commented Jan 26, 2025

the job that's passing fetched everything from an azure look aside mirror. The failures tried fetching directly from upstream sources. My bet is it's mostly bad luck with GitHub apt infra, and upstream proactively rejecting or otherwise failing apt fetches (got overloaded?)*

@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Jan 26, 2025
@moxian
Copy link
Contributor Author

moxian commented Jan 26, 2025

I would absolutely agree, but I find it sus that it's only fetching in clang-tidy job that fails, and not in any of the general build matrix jobs (although I only spot-checked those). The only difference (aside from the lack of apt-get update) between clang-tidy job and basic build is that the former has gettext explicitly in the apt install list, and the latter one doesn't.

@akrieger
Copy link
Member

Matrix builds are ubuntu 20.04 and tidy is 24.04.

@akrieger
Copy link
Member

IDK this probably doesn't break anything / regress much.

@akrieger akrieger merged commit 9ea772d into CleverRaven:master Jan 27, 2025
32 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Code: Tooling Tooling that is not part of the main game but is part of the repo. json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants