Skip to content

fix(cirrus): use different glean data dir per pid#13120

Merged
relud merged 1 commit intomainfrom
relud-cirrus-glean-datadir
Jul 28, 2025
Merged

fix(cirrus): use different glean data dir per pid#13120
relud merged 1 commit intomainfrom
relud-cirrus-glean-datadir

Conversation

@relud
Copy link
Contributor

@relud relud commented Jul 28, 2025

Because

  • glean can only have one process per data dir

This commit

  • Change cirrus metrics_config.data_dir to be a Path like /var/glean/$PID

Fixes #13119

@relud relud force-pushed the relud-cirrus-glean-datadir branch from 8d81895 to 4197e6e Compare July 28, 2025 21:40
@relud relud enabled auto-merge July 28, 2025 21:41
@relud relud added this pull request to the merge queue Jul 28, 2025
Merged via the queue into main with commit 6dac4a3 Jul 28, 2025
15 checks passed
@relud relud deleted the relud-cirrus-glean-datadir branch July 28, 2025 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yashikakhurana yashikakhurana yashikakhurana approved these changes

@jaredlockhart jaredlockhart Awaiting requested review from jaredlockhart jaredlockhart is a code owner

@jeddai jeddai Awaiting requested review from jeddai

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Cirrus workers need separate glean data directories

2 participants

@relud @yashikakhurana