-
Notifications
You must be signed in to change notification settings - Fork 527
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
New Ledger Bucket with Create Only Perms + Moving the CI to devnet from berkeley #15731
base: compatible
Are you sure you want to change the base?
Conversation
!ci-build-me |
4696e1c
to
f1f36c8
Compare
!ci-build-me |
!ci-build-me |
Aware there is a auth issue in CI for reading the bucket. Keeping an eye on progress 👀 |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
!ci-build-me |
1 similar comment
!ci-build-me |
7398784
to
fbd7a5e
Compare
!ci-build-me |
!ci-build-me |
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.
Please split this PR. Separately: create-only permissions and CI changes.
Some more suggestions/questions:
|
We have allocated a new read only bucket for the ledger tarball in CI. This is to prevent the file from accidentally being overwritten by another process pushing to the bucket. The approach here was to simply find and replace the old link. I still have to sanity check if this is correct and would appreciate feedback from @dkijania.