Skip to content

Commit

Permalink
Merge pull request #62 from useblacksmith/default-devdb
Browse files Browse the repository at this point in the history
backend: default to /dev/vdb when ExposeVolume response is empty
  • Loading branch information
adityamaru authored Dec 7, 2024
2 parents 96821d6 + 5ab7817 commit 15e5bef
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion dist/index.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

3 changes: 2 additions & 1 deletion src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -440,7 +440,8 @@ async function getBuilderAddr(inputs: context.Inputs, dockerfilePath: string): P
exposeId = stickyDiskResponse.expose_id;
device = stickyDiskResponse.device;
if (device === '') {
throw new Error('No device returned from ExportVolume request');
// TODO(adityamaru): Remove this once all of our VM agents are returning the device in the stickydisk response.
device = '/dev/vdb';
}
clearTimeout(timeoutId);
await maybeFormatBlockDevice(device);
Expand Down

0 comments on commit 15e5bef

Please sign in to comment.