-
Notifications
You must be signed in to change notification settings - Fork 282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: bootstrap_docker skips build it can pull from ecr. #2545
Merged
Merged
Changes from 109 commits
Commits
Show all changes
111 commits
Select commit
Hold shift + click to select a range
04d5136
Initial cci_gen work.
charlielye 27ef8da
Intial
charlielye 06ae9b1
more
charlielye 3d55c7c
Merge remote-tracking branch 'origin/master' into cl/ypb_cleanup
charlielye a26f1f4
wip
charlielye 61b2b80
wip
charlielye 14a334f
wip
charlielye 63a7d3c
wip
charlielye cdc2b65
fix
charlielye c7c5baf
Add yq bin.
charlielye d3f44c8
Move to native version of yq. Can also now remove jq from build-system.
charlielye 4d818d1
fix
charlielye d4f5a81
debug
charlielye 36c8251
fix: PATH and bb.js ref
ludamad 3d52670
fix: PATH and bb.js ref
ludamad 7a6e238
fix: PATH and bb.js ref
ludamad e72e347
Merge remote-tracking branch 'origin/master' into cl/ypb_cleanup
ludamad a447ffe
fix: pass debug env to spot [debug ci]
ludamad 054efef
tinkering [debug ci]
ludamad 7b93f83
Merge remote-tracking branch 'origin/master' into cl/ypb_cleanup
ludamad 648c910
fix: yarn-project-base-deps not in manifest [debug ci]
ludamad 617f0ac
fix: dont set CACHE_FROM if try_fetch_image fails
ludamad da5b6d9
Format private-token
ludamad ff0d592
Lint private-token
ludamad f1f649e
chore: speed up yarn-project step
ludamad daf365c
Revert change
ludamad 23c1a93
yarn-project Dockerfile update
ludamad 79ea225
yarn-project Dockerfile update
ludamad 7d491c2
Fix script call.
charlielye 0813f83
wip
charlielye c01660a
Use r6a instances to build. Higher cpu count for bb. Allow [ci rebuil…
charlielye ebe6bb3
Run cci_gen. [ci rebuild]
charlielye 36de7fb
Simplify ci cmd stuff. Rollback cpu count to see if it actually helpe…
charlielye 28da340
Forgot to cci_gen... [ci rebuild]
charlielye 5701341
Add timestamps. yolo [ci rebuild]
charlielye 46e468c
Add timestamps. yolo [ci rebuild]
charlielye 6d9378b
Remove redundant timestamps
ludamad 54f89f2
fix: rebuild flag unbound
ludamad 961dd5c
fix: use fetching commit message
ludamad 173b5ee
wip [ci rebuild]
charlielye 576c8f2
Merge remote-tracking branch 'origin/master' into cl/ypb_cleanup
charlielye 5018c7e
Remove cci_gen stuff for now.
charlielye 2128754
Fix
charlielye b87c3c1
wip
charlielye fe3390f
README
charlielye 0d227a8
Formatting
charlielye 951bf36
Disable blank tests as not passing.
charlielye a766965
fix
charlielye d22b550
fix
charlielye 914e2b9
fix
charlielye 4784413
fix
charlielye c4abeac
fix
charlielye 644c8f2
fix
charlielye 1e0e6ba
ypb down to 730MB
charlielye 3596ee8
Smaller builds using pnp (at last second), and some incantations.
charlielye 30e5e86
Comments.
charlielye c95c1f2
Bump ubuntu machine executor image to 2204
charlielye 5738974
Fix wait on missing image.
charlielye d8ecba6
Revert Dockerfile.dockerignore to work with Docker v20. Can move to m…
charlielye c8da4d2
Fix sandbox dockerfile
charlielye b3afd23
Fix canary Dockerfile.
charlielye 3e5a985
Add tests to own job.
charlielye 6d05422
Make tests actually run...
charlielye 2238e78
Add ecr_login
charlielye a10336c
Fix browser.
charlielye 0c7a83d
Make yarn-project use some rebuild patterns. May need tweaking. Fix a…
charlielye 2434d9a
Try and get browser test working again.
charlielye b1a8f4b
Fix canary.
charlielye a03f341
Merge remote-tracking branch 'origin/master' into cl/ypb_cleanup
charlielye 3b5a942
fix: ypb deploy (#2464)
ludamad 1a77721
chore: yarn project test breakup (#2484)
ludamad 47a83d2
Merge branch 'master' into cl/ypb_cleanup
ludamad d0446aa
fix: cimg
ludamad 46b3ab4
fix: cimg
ludamad 2037278
[ci debug]
ludamad 4e0cb82
Merge remote-tracking branch 'origin/cl/ypb_cleanup' into cl/ybp_smaller
charlielye c01cae3
fix: Use calculate_image_uri
ludamad 6322f5c
[ci debug]
ludamad 71ca071
ecr login [ci debug]
ludamad 4376ca9
Merge remote-tracking branch 'origin/cl/ypb_cleanup' into cl/ybp_smaller
charlielye a79cd39
fix: dont require tag
ludamad 5507089
fix: dont require tag
ludamad b8b23cd
Fix merge. yarn-project on spot.
charlielye 5fe0019
Merge remote-tracking branch 'origin/cl/ypb_cleanup' into cl/ybp_smaller
charlielye 50cf1a2
Fix canary deploy build.
charlielye 5901208
Merge remote-tracking branch 'origin/master' into cl/ybp_smaller
charlielye 8219860
Fix merge oops.
charlielye 12a07a2
Merge branch 'master' into cl/ybp_smaller
charlielye 03129c1
Parallel wasm/wasm-threads build.
charlielye a1b3ccc
Shift pnpify into final artefact, separate formatting again. [ci rebu…
charlielye eee400c
Merge [ci rebuild]
charlielye 0510b1e
[ci rebuild yarn-project]
charlielye fdde4b9
Simpler build that doesn't push/pull to leverage layer cache.
charlielye 9d6f022
Attempt to handle ' in commit msg.
charlielye 91fd342
[ci rebuild yarn-project]
charlielye d39a46f
[ci rebuild yarn-project]
charlielye 1f898e3
[ci rebuild yarn-project]
charlielye 05bb816
[ci rebuild]
charlielye d400647
Add yarn-project as doc dep.
charlielye a3f2fab
Fix docs
charlielye 2497b24
Add file.
charlielye a11e2d0
Remove pnp. Gains not worth it.
charlielye 21506f6
Merge remote-tracking branch 'origin/master' into cl/ybp_smaller
charlielye 6d827ce
Slim stage yarn-project.
charlielye 1ff4e45
Formatting.
charlielye d16ad40
Merge branch 'cl/ybp_smaller' into cl/no_build_layer_cache
charlielye b495771
boostrap_docker will skip builds it can pull from ecr
charlielye 7dd3c5a
Consider working tree when computing content hash (on request)
charlielye 736be1b
Merge remote-tracking branch 'origin/master' into cl/bootstrap_docker…
charlielye 6dcf540
Favour rebuilding changed over computing new hash.
charlielye b67799a
Color!
charlielye File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can I be the bash complexity police? : D
What about something like...
but you're the captain