-
Notifications
You must be signed in to change notification settings - Fork 380
chore(IDX): remove container installed packages #8654
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
Conversation
…73c3ad266a705450a720f6ab4cc19f Image tag: 19719d01eeb27669291c8edd33689940ca08bb4a55402e5f7b19868ec2404be2
|
Run URL: https://github.com/dfinity/ic/actions/runs/21626815087 New container image: |
…98822b0a93d84f24b16c9708f9b3e2 Image tag: 114e03637a54a7731cde3c63ca70dd2994480a918340ec3d92cdcdaf3531324e
Instead of adding a new comment we update the existing comment to avoid PR clutter. (Tested on this PR: #8654)
…1927b0852791b7c8021e1d3bc0e385 Image tag: 883c8019dfa7922fda5e45d470820d8dbdc19602357db42955fdfab5dfc612a2
basvandijk
left a comment
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.
Are the test failures just flakiness?
This #8654 broke cargo builds and I need to add it back. Cargo build run: https://github.com/dfinity/ic/actions/runs/21635223254/job/62358667256?pr=8663 --------- Co-authored-by: IDX GitHub Automation <> Co-authored-by: Bas van Dijk <[email protected]>
This removes a set of packages that are either not needed at all or installed via bazel. They include: - cocogitto - sdk - motoko Total container size reduction was ~400MB. --------- Co-authored-by: IDX GitHub Automation <>
This #8654 broke cargo builds and I need to add it back. Cargo build run: https://github.com/dfinity/ic/actions/runs/21635223254/job/62358667256?pr=8663 --------- Co-authored-by: IDX GitHub Automation <> Co-authored-by: Bas van Dijk <[email protected]>
This removes a set of packages that are either not needed at all or installed via bazel. They include:
Total container size reduction was ~400MB.