Skip to content

Conversation

@tlaurion
Copy link
Collaborator

@tlaurion tlaurion commented May 5, 2025

Fix bug for cache reuse introduce when switching from x230 -> t480 for first board being build (but not switching x230->t480 for musl-cross-make at the same time under #1908)

Test made from this PR:


As said under #1666, we need insights on best next strategies to save cache/pass workspace caches under CircleCI per above links. Short version:

tlaurion added 3 commits May 2, 2025 14:33
…: we add procps+psmisc in flake.nix to be able to troubleshoot processes and /proc related info from docker image

Signed-off-by: Thierry Laurion <insurgo@riseup.net>
… musl-cross-make from x230 to t480 (t480 is part of save_cache, not x230...)

Signed-off-by: Thierry Laurion <insurgo@riseup.net>
…t generating error since $$$$(MAKE) was resolved to MAKE in configure.log, but was still working correctly)

See: https://app.circleci.com/pipelines/github/tlaurion/heads/3314/workflows/e588480d-d13a-49e0-a1b6-78fed839b70b/jobs/67601?invite=true#step-102-1420_53

Signed-off-by: Thierry Laurion <insurgo@riseup.net>
@tlaurion tlaurion requested a review from Copilot May 5, 2025 14:37
@tlaurion tlaurion added the bug label May 5, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the CircleCI configuration to address caching issues related to board targets and includes updated Docker tooling for enhanced debugging. Key changes include:

  • Updating the Docker image version from tlaurion/heads-dev-env:v0.2.4 to v0.2.5 in several jobs.
  • Changing the target from x230-hotp-maximized to t480-hotp-maximized in the workflow.
Files not reviewed (2)
  • flake.nix: Language not supported
  • modules/musl-cross-make: Language not supported

@tlaurion
Copy link
Collaborator Author

tlaurion commented May 5, 2025

TODO:

@tlaurion
Copy link
Collaborator Author

tlaurion commented May 5, 2025

Merging as bugfix since needed by other PR failing on CircleCI builds, like #1966

@tlaurion tlaurion merged commit 569e44c into linuxboot:master May 5, 2025
1 check passed
@tlaurion tlaurion mentioned this pull request May 5, 2025
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant