Skip to content

Revert "ci: cache ~/.conan2 in build workflow" - #85

Merged
lia (lia-viam) merged 1 commit into
mainfrom
revert-84-ci-cache-conan
Aug 11, 2026
Merged

Revert "ci: cache ~/.conan2 in build workflow"#85
lia (lia-viam) merged 1 commit into
mainfrom
revert-84-ci-cache-conan

Conversation

@lia-viam

Copy link
Copy Markdown
Contributor

Reverts #84

lost this in my github notifications, but just recording for posterity that caching conan with github actions is basically always an error, for a couple reasons. the simplest one is that this is the exact purpose of the viam artifactory server, so it's an antipattern to be caching on top of a dedicated, purpose-built cache.

the more complex reason is that github actions caching is not fit to purpose for the level of granularity used by conan to cache binary artifacts, and attempts to distinguish github cache artifacts, eg by runner name or platform, will still lead to collisions and races which show up in basically any nontrivial CI matrix.

looking ahead it would be good if we had a canonical "good example" to point people at for how this should be done, which is that CI jobs should use a lockfile to pin dependency versions, and periodically upload dependencies to the viam conan server if they're slowing down CI

@JosephBorodach

Copy link
Copy Markdown
Contributor

Sorry I overlooked this. As we discussed offline, once we have that good example if you can point me to it that would be great because we have 3 other camera modules that are caching conan

@lia-viam
lia (lia-viam) merged commit ec29dda into main Aug 11, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants