Set up deployment pipeline for mz-debug #32117
Merged
Conversation
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
SangJunBak
changed the title
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
c1a3111 to
bc4b8c2
Compare
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
2 times, most recently
from
60975fd to
f393375
Compare
SangJunBak
force-pushed
the
jun/#8908/clap-fixes
branch
from
937d857 to
b95a37a
Compare
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
9fe5a40 to
5ba7fb7
Compare
SangJunBak
force-pushed
the
jun/#8908/clap-fixes
branch
from
b95a37a to
0726fc1
Compare
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
5ba7fb7 to
2e630f3
Compare
SangJunBak
force-pushed
the
jun/#8908/clap-fixes
branch
from
0726fc1 to
a6d2ad7
Compare
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
2e630f3 to
3846c6c
Compare
SangJunBak
force-pushed
the
jun/#8908/clap-fixes
branch
from
a6d2ad7 to
6bed857
Compare
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
3 times, most recently
from
e5c79da to
1e1b601
Compare
def-
requested changes
Apr 9, 2025
ci/deploy_mz-debug/README.md
Outdated
| @@ -0,0 +1,20 @@ | |||
| # Deploy the Materialize debug tool. | |||
|
|
|||
| The CI process will build and deploy the LSP server to the materialize-binaries S3 bucket. | |||
test/mz-debug/mzcompose.py
Outdated
| c.up("materialized") | ||
|
|
||
| # Get the container id of the materialized service | ||
| print(c.container_id("materialized")) |
test/mz-debug/license_key
Outdated
| @@ -0,0 +1 @@ | |||
| eyJ0eXAiOiJKV1QiLCJhbGciOiJQUzI1NiJ9.eyJzdWIiOiIwMDAwMDAwMC0wMDAwLTAwMDAtMDAwMC0wMDAwMDAwMDAwMDAiLCJleHAiOjIwNTg5NzQ2ODgsIm5iZiI6MTc0MzYxNDY4OCwiaXNzIjoiTWF0ZXJpYWxpemUsIEluYy4iLCJhdWQiOiIwMDAwMDAwMC0wMDAwLTAwMDAtMDAwMC0wMDAwMDAwMDAwMDAiLCJpYXQiOjE3NDM2MTQ2ODgsImp0aSI6ImVkZGFmMDA0LWRjMWUtNDhjZi05Y2MxLTQxZDE1NDNkOTQwYSIsInZlcnNpb24iOjEsIm1heF9jcmVkaXRfY29uc3VtcHRpb25fcmF0ZSI6MTAwMC4wLCJhbGxvd19jcmVkaXRfY29uc3VtcHRpb25fb3ZlcnJpZGUiOnRydWUsImV4cGlyYXRpb25fYmVoYXZpb3IiOiJXYXJuIn0.a-O0xEfwgKKWkKCUoKF_odY_-ZbMN8t-fkLLfVvYkprD-uI2mFtWwVup9rRNDPdXO6M4aDkmTuTaX5Wt9jv1pPHDicICqPITtwCisvVRzV4IRcP09DinN-vooTcQy-ehG4smlpZ3CpWdig3CiJI4Sjv_QMTr6G8ETslZHCAtUfNpglSDXDCIBBujmiAQB6LpgTzNV6Uuq6t-FPKGAfhs3NkxSV_KGm-4LcoqhrqiLv6IotanwTTf7ZUeyW0HApdTbfFw6oBh_RORFULHcLojI8n5kAy2VIqzgc_ft2EpFNsw_L0XqhvECIfr-6f1oXErD7YdXNuEC2jdfWnapg3k7i61IbHyU6bo35tNw3AFynsx29UeeClmHdYW_UsyZAQmwjvW3pFo9Itaz1v9cVggZew_pa-tt6qXVvr9T6MnCbjj-zxJhAWyR4vRms817t4VgqHW9paa9l9aw4adWAkpzK3W0D1U0_lsCM-_918znZkmn0c7WxAFHAvjxob9d1xcJna06grsmzvCcFdtdlveqZB-s4nxDAB5rM5rm6vrLBf81x8UCuwA76NJxAmSXvKpVqS7iet_Crow90UPCOUoxWl2ql6HUXqVj4dl7nS43v42rFZGonjE4pL-A-7QYXfoqly0LoR-kCeBwHpKGJtDzBYBRl14-YGjH-y9HfOvBkI | |||
There was a problem hiding this comment.
Ouch, I think we have to invalidate and recreate this key now for everyone. CC @doy-materialize
I'll add license_key to .gitignore so this doesn't happen so easily again.
There was a problem hiding this comment.
which key is this? (where did it come from?)
There was a problem hiding this comment.
It's the materialize dev license key in 1password.
SangJunBak
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
7366895 to
1ee28f5
Compare
def-
approved these changes
Apr 9, 2025
5 tasks
- I refactored mz_lsp_server's deploy_utils into a tarball_uploader. I plan to move the other binaries to use it too. - Tested macos and linux separately - Unclear if deploy_utils is necessary or if I should rename it. Not confident on where each variable lives as well. - Created a binary to run mz-debug
Fix lint errors Fix lint errors
- Implements cloud test and mzcompose test that checks if the tool generates a zip, which is a sign of success.
def-
force-pushed
the
jun/#8908/deploy-debug-tool
branch
from
ecaaefe to
829447a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Based on PR #32122
Chain of upstream PRs as of 2025-04-08
PR Fix interface bugs for mz-debug #32122:
main←jun/#8908/clap-fixesjun/#8908/clap-fixes←jun/#8908/deploy-debug-toolSee commit messages for details
Motivation
Tips for reviewer
Checklist
$T ⇔ Proto$Tmapping (possibly in a backwards-incompatible way), then it is tagged with aT-protolabel.