Skip to content

debug: allow debug from remote cache config #6076

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tonistiigi
Copy link
Member

follow-up to #6061

Allows debugging from remote config passed via POST request in addition to debugging local cache store.

@tonistiigi
Copy link
Member Author

Leaving draft for now as there seems to be an extra level of indirection when debugging remote source, that comes from output index being embedded into the cache key. I was aware of output index being stored as a separate value only in bolt but bit unexpected how these behave differently even in the CacheKeyStorage interface level that I want to investigate more.

This adds extra level of indirection to the cache keys debug,
making it more verbose but making sure that same digests are
reported for remote cache.

Signed-off-by: Tonis Tiigi <[email protected]>
@tonistiigi tonistiigi force-pushed the cache-debuginfo-db-remote branch from dbb57d8 to 80e6c06 Compare July 11, 2025 00:28
@tonistiigi tonistiigi marked this pull request as ready for review July 11, 2025 00:28
@tonistiigi
Copy link
Member Author

Leaving draft for now as there seems to be an extra level of indirection when debugging remote source, that comes from output index being embedded into the cache key. I was aware of output index being stored as a separate value only in bolt but bit unexpected how these behave differently even in the CacheKeyStorage interface level that I want to investigate more.

In order to fix this, I added the same indirection to the boltdb debug as well, so that both backends generate matching keys.

Also added extra commit for better output:

@tonistiigi
debug: combine links from parents intead of children
80e6c06
Grouping the link digests to parent side (links pointing
to current node) makes more sense than children side (links
pointing to next nodes). This also reduces the duplicates
in the output as previously same digest link could need
plaintext under multiple nodes.

Grouping the link digests to parent side (links pointing
to current node) makes more sense than children side (links
pointing to next nodes). This also reduces the duplicates
in the output as previously same digest link could need
plaintext under multiple nodes.

Signed-off-by: Tonis Tiigi <[email protected]>
@tonistiigi tonistiigi force-pushed the cache-debuginfo-db-remote branch from 80e6c06 to ac9317b Compare July 11, 2025 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant