Aditya Maru
6fe2467492
src: silence metric warnings for now
9 months ago
Aditya Maru
4759d93c12
src: use the plumbed BLACKSMITH_BACKEND_URL if present
9 months ago
Aditya Maru
def1585067
*: report metrics to the VM agent
9 months ago
Aditya Maru
4723a2a346
src: stop spurious warnings on buildkit shutdown
9 months ago
Aditya Maru
1672d6fbad
src: fix shutdown retry behavior
9 months ago
Aditya Maru
9302d2aea9
src: stop running process as nohup to avoid missing logs
9 months ago
Aditya Maru
ac42783fa9
src: cleanup flakiness in different parts of the action
9 months ago
Aditya Maru
54bc4e0788
src: refactor cleanup logic to expose buildkitd.log
...
Previosuly, we only killed the buildkitd process and unmounted
if builderInfo was non null. This was wrong cause we could have setup
builkdkitd, but failed after that step. This would then rely on the last
ditch effort by the post action to cleanup. We now change the proc kill
and unmount to happen on any build error.
9 months ago
Aditya Maru
d43ee61bb7
*: move to grpc backed communication for the agent
9 months ago
Aditya Maru
53000f0f59
ignore error when nothing is mounted
10 months ago
Aditya Maru
1df1b3c361
src: ignore error when theres nothing mounted
10 months ago
Aditya Maru
de0451e517
src: make post unmount even if buildkitd is no longer present
...
Also increase retries when trying to unmount the buildkit directory.
Retry up to 3 seconds now, previously we were only retrying 3 times
with a 100ms backoff.
10 months ago
Aditya Maru
0f99a0b1c7
src: start sending get request with query params
...
We are incorrectly using formData in a get request. To move
away from this we send both query params and formData until
the server is fully upgraded. After which we can stop sending
formData.
10 months ago
Aditya Maru
0186286e06
*: use axios-retry instead of handrolled retry methods
10 months ago
Aayush Shah
7b8642822f
src: make `getDockerfilePath` return the full path to the dockerfile ( #64 )
...
Previously we were just returning the path to the dir containing the dockerfile
in most cases.
10 months ago
Aditya Maru
f06a558c36
src: alert if an exception is thrown on cleanup
10 months ago
Aditya Maru
b76cd7bf3b
src: fix bug in conditional that zero'd out expose ID
10 months ago
Aayush
f9d1e150a9
*: unify error handling and add more unit tests
10 months ago
Aditya Maru
c71ad2dbef
*: refactor methods to support mocking
...
Additionally, write some tests to ensure the driver method
`startBlacksmithBuilder` handles all exceptions correctly in
both nofallback=true and nofallback=false configurations.
10 months ago
Aditya Maru
5ab78173d3
backend: default to /dev/vdb when ExposeVolume response is empty
10 months ago
Aditya Maru
edc01b36df
backend: use device returned in ExposeVolume response
10 months ago
Aditya Maru
72c7e93db9
src: send stickydisk key with commit
10 months ago
Aditya Maru
6d7db93fa2
src: bump socket creation timeout to 5s from 3s
10 months ago
Aditya Maru
c308f14958
src: report the correct sticky disk key
10 months ago
Aditya Maru
17d922af1a
src: shuttle a expose ID between expose and commit
10 months ago
Aditya Maru
a55bae5255
src: prevent top-level ref variable from being shadowed
10 months ago
Aditya Maru
9841eabab1
src: only resolve buildref on success
10 months ago
Aditya Maru
4938a7e10a
src: change arch to use BLACKSMITH_ENV
10 months ago
Aditya Maru
9336122050
src: add some idempotent cleanup safeguard in post action
10 months ago
Aditya Maru
bdf7f0bb37
src: wrap all steps after blacksmith builder in try catch
...
This ensures we always run cleanup if any step after creating the Blacksmith
builder errors out.
10 months ago
Aditya Maru
9b63433194
src: make blacksmith builder name unique
10 months ago
Aditya Maru
bda6587832
src: change sticky disk key to repo name
10 months ago
Aditya Maru
c33190b3c9
src: add local mirror to buildkit toml
10 months ago
Aditya Maru
1dee25cffd
src: fix the movement of cleanup to the main step
10 months ago
Aditya Maru
f16c36e819
src: add resize2fs call if the block device is formatted
10 months ago
Aditya Maru
e1da38ff9d
src: add petname and vmID to notification
10 months ago
Aditya Maru
ac4af6279b
src: move shutdown, cleanup, commit from post to after build
10 months ago
Aditya Maru
138e3a2a14
dist: cat buildkit log file if build fails
10 months ago
Aayush Shah
bdd6696880
Only use buildRef if it exists ( #42 )
10 months ago
Aayush Shah
658750494f
workflows: add workflow to avoid checking in ungenerated code ( #41 )
10 months ago
Aditya Maru
3cdd1d81fc
*: blkid returns non-zero code if no fs found
10 months ago
Aditya Maru
88ab597dfe
*: move formatting of the block device to the client
...
Previously, we were incorrectly formatting the block device
on the ceph server. This could cause corruption. As a mitigation
we now conditionally format the block device on the client. This
should only kick in if a previous parent clone has not already been
formatted.
10 months ago
Aditya Maru
b83388c4ca
*: inline headers
10 months ago
Aayush Shah
a6005ce994
record and report docker build duration seconds ( #33 )
10 months ago
Aayush Shah
7d2e9e7288
log the full request URL ( #32 )
10 months ago
Aayush Shah
3b6e3e7be4
update generated code ( #31 )
10 months ago
Aayush Shah
fc356a7e65
inline some logic ( #29 )
10 months ago
Aayush Shah
562ddafb4b
rename a method and add some debug logging ( #27 )
10 months ago
Aayush Shah
c50dbd0552
dont fail the build if we fail to contact backend ( #26 )
10 months ago
Aayush Shah
51601ec916
report builds to the backend ( #25 )
10 months ago