Skip to content

Tag image fetch metrics with use_oci_fetcher experiment label#11411

Merged
dan-stowell merged 1 commit intomasterfrom
bbdan-instrument-oci-fetcher-exp
Feb 26, 2026
Merged

Tag image fetch metrics with use_oci_fetcher experiment label#11411
dan-stowell merged 1 commit intomasterfrom
bbdan-instrument-oci-fetcher-exp

Conversation

@dan-stowell
Copy link
Contributor

Add a 'use_oci_fetcher' label to the ImageFetchDurationUsec histogram so we can compare latency and error rates between the OCI fetcher experiment path and the standard path.

  • Add ImageFetchUseOCIFetcherLabel constant and add it to the histogram
  • Add useOCIFetcher bool parameter to PullImageIfNecessary and RecordImageFetchMetrics
  • Thread the value from each caller: ociruntime and firecracker pass their stored useOCIFetcher field, podman/docker pass false, runner passes PlatformProperties.UseOCIFetcher

Add a 'use_oci_fetcher' label to the ImageFetchDurationUsec histogram
so we can compare latency and error rates between the OCI fetcher
experiment path and the standard path.

- Add ImageFetchUseOCIFetcherLabel constant and add it to the histogram
- Add useOCIFetcher bool parameter to PullImageIfNecessary and
  RecordImageFetchMetrics
- Thread the value from each caller: ociruntime and firecracker pass
  their stored useOCIFetcher field, podman/docker pass false, runner
  passes PlatformProperties.UseOCIFetcher

Co-authored-by: Shelley <shelley@exe.dev>
@dan-stowell dan-stowell marked this pull request as ready for review February 25, 2026 20:58
@dan-stowell dan-stowell merged commit db36d6e into master Feb 26, 2026
14 of 16 checks passed
@dan-stowell dan-stowell deleted the bbdan-instrument-oci-fetcher-exp branch February 26, 2026 20:53
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