Page MenuHomePhabricator
Differential D11115

[web] Check backup client wasm integrity buildkite
AbandonedPublic
Actions

Authored by michal on Feb 20 2024, 2:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 26, 11:06 PM
Unknown Object (File)
Wed, Jun 26, 6:55 AM
Unknown Object (File)
Wed, Jun 26, 1:56 AM
Unknown Object (File)
Tue, Jun 25, 11:12 PM
Unknown Object (File)
Tue, Jun 25, 3:31 AM
Unknown Object (File)
Thu, Jun 20, 1:26 AM
Unknown Object (File)
Fri, Jun 14, 6:19 AM
Unknown Object (File)
Thu, Jun 13, 8:11 AM
View All 56 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
atul
Summary

ENG-6873 : Add a buildkite job for checking integrity of backup client wasm file

Similarly to comm_query_executor we want a buildkite job to build the wasm file from source and make sure that it's the same. In this diff I'm modifiying the existing pipeline to also check for the backup client.

Test Plan

Check that the CI passes.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

michal created this revision.Feb 20 2024, 2:40 AM
michal held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptFeb 20 2024, 2:40 AM
michal added a parent revision: D11041: [backup-client] Compile to wasm.Feb 20 2024, 2:40 AM
Harbormaster failed remote builds in B26980: Diff 37339!Feb 20 2024, 2:54 AM
michal updated this revision to Diff 37340.Feb 20 2024, 3:00 AM

Install wasm-pack on CI

Harbormaster completed remote builds in B26981: Diff 37340.Feb 20 2024, 3:15 AM
michal updated this revision to Diff 37345.Feb 20 2024, 3:50 AM

Check if CI will fail if wasm file is not updated, but the code is changed

Harbormaster completed remote builds in B26986: Diff 37345.Feb 20 2024, 4:05 AM
michal updated this revision to Diff 37350.Feb 20 2024, 4:32 AM

Debugging

Harbormaster completed remote builds in B26990: Diff 37350.Feb 20 2024, 4:47 AM
michal updated this revision to Diff 37353.Feb 20 2024, 5:20 AM

Debugging

michal updated this revision to Diff 37355.Feb 20 2024, 5:32 AM

Check if my changes broke the pipeline

Harbormaster completed remote builds in B26993: Diff 37353.Feb 20 2024, 5:35 AM
Harbormaster completed remote builds in B26995: Diff 37355.Feb 20 2024, 6:00 AM
michal updated this revision to Diff 37539.Feb 23 2024, 9:09 AM

Update the new emscripten pipeline

michal updated this revision to Diff 37544.Feb 23 2024, 9:23 AM

Rebase

Harbormaster failed remote builds in B27122: Diff 37539!Feb 23 2024, 9:23 AM
Harbormaster failed remote builds in B27127: Diff 37544!Feb 23 2024, 9:46 AM
michal published this revision for review.Feb 23 2024, 10:10 AM
atul added a comment.Feb 23 2024, 11:35 AM

Passing two command flags seems weird, can we do like yarn build-db-wasm && yarn build-db-wasm?

michal updated this revision to Diff 37590.Feb 26 2024, 3:29 AM

Switch to one command. Check that CI fails if wasm wasn't updated.

Harbormaster failed remote builds in B27170: Diff 37590!Feb 26 2024, 4:56 AM
michal updated this revision to Diff 37591.Feb 26 2024, 5:02 AM

Switch to two command, as I had problems with doing both builds in one command.

Harbormaster failed remote builds in B27171: Diff 37591!Feb 26 2024, 5:09 AM
michal updated this revision to Diff 37596.Feb 26 2024, 5:20 AM

Check that CI passes if wasm is current.

michal updated this revision to Diff 37598.Feb 26 2024, 5:32 AM

Try cleaning before building

Harbormaster failed remote builds in B27172: Diff 37596!Feb 26 2024, 5:34 AM
Harbormaster failed remote builds in B27174: Diff 37598!Feb 26 2024, 5:53 AM
michal updated this revision to Diff 37611.Feb 26 2024, 6:58 AM

Remap paths

Harbormaster failed remote builds in B27179: Diff 37611!Feb 26 2024, 7:05 AM
atul requested changes to this revision.Feb 26 2024, 1:58 PM

Looks like the Emscripten workflow is failing? Sending back to your queue, feel free to re-request review if I'm missing something

This revision now requires changes to proceed.Feb 26 2024, 1:58 PM
Harbormaster completed remote builds in B27127: Diff 37544.Feb 26 2024, 2:18 PM
ashoat added inline comments.Feb 26 2024, 6:10 PM
.buildkite/emscripten.yml
5 ↗(On Diff #37611)

We should avoid committing commented-out code

michal added a comment.Feb 27 2024, 5:16 AM

Oh sorry, I forgot this was in review, sorry for the confusion. I'm having issues with generated WASM reproducibility on CI so this diff isn't ready yet and I'm still testing.

michal updated this revision to Diff 37653.Feb 27 2024, 6:06 AM

Rebase

Harbormaster failed remote builds in B27204: Diff 37653!Feb 27 2024, 6:18 AM
ashoat added a comment.Feb 27 2024, 8:53 PM

It's good practice to "Plan Changes" to take diffs off of your reviewers' queues if they're not ready for review

michal planned changes to this revision.Feb 28 2024, 5:44 AM

Sorry, I keep forgetting that for this diff, when testing CI I have to plan changes after every new revision.

michal abandoned this revision.Feb 28 2024, 7:01 AM

Canceling this in favour of just adding another build step to cleaninstall. See discussions in ENG-6873

Revision Contents
Changeset List

PathSize
.buildkite/
6 lines
web/
backup-client-wasm/
wasm/
scripts/
3 lines

Diff 37653

View Options

.buildkite/emscripten.yml

Loading...
View Options

web/backup-client-wasm/wasm/backup-client-wasm_bg.wasm

Loading...
View Options

web/scripts/run_wasmpack.sh

Loading...