Skip to content

Conversation

@Emyrk
Copy link
Member

@Emyrk Emyrk commented Nov 18, 2025

No description provided.

Copy link
Member Author

Emyrk commented Nov 18, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Emyrk Emyrk force-pushed the stevenmasley/second_graph branch 2 times, most recently from 0ee7480 to 6389238 Compare November 19, 2025 13:10
@Emyrk Emyrk changed the base branch from stevenmasley/wholetimings to graphite-base/20823 November 19, 2025 15:35
Second graph stage should be measured
@Emyrk Emyrk force-pushed the graphite-base/20823 branch from d8e078c to aff2080 Compare November 19, 2025 15:36
@Emyrk Emyrk force-pushed the stevenmasley/second_graph branch from 6389238 to 92e9f6a Compare November 19, 2025 15:36
@graphite-app graphite-app bot changed the base branch from graphite-base/20823 to main November 19, 2025 15:36
@Emyrk Emyrk force-pushed the stevenmasley/second_graph branch from 92e9f6a to 349338a Compare November 19, 2025 15:37
@Emyrk Emyrk force-pushed the stevenmasley/second_graph branch from 349338a to 97a86e0 Compare November 20, 2025 16:29
@@ -0,0 +1,2 @@
-- Currently have 2 graph stages in a workspace build
ALTER TYPE provisioner_job_timing_stage ADD VALUE IF NOT EXISTS 'graph_second';
Copy link
Member Author

Choose a reason for hiding this comment

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

I really dislike this name. I have another PR where I add a stage_seq number.

We only have the graph stage duplicated, and it should be the only one. So probably best to just hard code it 😢

(I also tried to remove the second graph, but that is really challenging with quotas)

@github-actions github-actions bot added the stale This issue is like stale bread. label Nov 28, 2025
@github-actions github-actions bot closed this Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale This issue is like stale bread.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants