Skip to content

Run the deploy job on Gimlets#10750

Open
emilyalbini wants to merge 1 commit into
mainfrom
ea-deploy-gimlet
Open

Run the deploy job on Gimlets#10750
emilyalbini wants to merge 1 commit into
mainfrom
ea-deploy-gimlet

Conversation

@emilyalbini

Copy link
Copy Markdown
Member

This should enable us to increase capacity on the job. I had to increase the timeouts for some of the retry loops to get the job to succeed.

Fixes #10054.
This is stacked on top of #10749.

Comment on lines +258 to +261
# The gimlet factory creates a zpool on /pool/bsu/0 backed by the first M.2
# disk. We use it to store large files that wouldn't fit in the ramdisk.
SCRATCH_DIR=/pool/bsu/0/scratch
pfexec mkdir $SCRATCH_DIR

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

FWIW, there are some other jobs that use Gimlet runners that are instead using pilot local disk list to find disks to make a zpool on, like this: https://github.com/oxidecomputer/propolis/blob/979b72896bc39a876c98fb32fe706da37a4fc408/.github/buildomat/phd-run-with-args.sh#L17-L21

Do you think we ought to use that to using the scratch dataset there too? It might shave a bit off the job's total runtime if we didn't go and initialize another zpool...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes that'd work!

Comment on lines +258 to +259
# The gimlet factory creates a zpool on /pool/bsu/0 backed by the first M.2
# disk. We use it to store large files that wouldn't fit in the ramdisk.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure what their DWPD is, but is the endurance of the M.2 drives a problem we should worry about?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That is something we were already doing before on the other systems. In practice the only thing we are writing to the M.2s is the vdevs used to store zones on, so I'm not too worried.

@emilyalbini emilyalbini force-pushed the ea-external-storage branch from 05b8416 to cf769f6 Compare July 8, 2026 15:06
Base automatically changed from ea-external-storage to main July 9, 2026 07:01
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.

3 participants