feat: replace debian_package_manager with apt #1542
Conversation
thesayyn
force-pushed
the
use_apt
branch
4 times, most recently
from
a13c3d0 to
a5c5e1d
Compare
|
🌳 🔄 Image Check |
| run: | | ||
| for i in $(seq 5); do | ||
| bazel --bazelrc=$GITHUB_WORKSPACE/.github/workflows/ci.bazelrc fetch //private/pkg/test/oci_image:test_sboms && break || sleep 20; | ||
| done |
| @@ -217,16 +223,15 @@ Distroless tracks the upstream Debian releases, using [Github actions to automat | |||
|
|
|||
| ### Debug Images | |||
|
|
|||
| Distroless images are minimal and lack shell access. The ```:debug``` image set for each language provides a busybox shell to enter. | |||
| Distroless images are minimal and lack shell access. The `:debug` image set for each language provides a busybox shell to enter. | |||
There was a problem hiding this comment.
@loosebazooka we should probably run pre-commit on the whole repo and rebase here. Can you do that?
| @@ -0,0 +1,166 @@ | |||
| #!/usr/bin/env bash | |||
There was a problem hiding this comment.
I think I would prefer these be separate scripts that had more descriptive names rather than knife?
There was a problem hiding this comment.
naming is tbd, i can split them as well.
Yes, i'll remove it from here. |
|
Oh, actually we can't as there's no replacement for what debian_package_manger did to upgrade snapshots etc. now those logic lives in this shell script. If i remove it from here, the CI will fail at midnight. |
This the effort for replacing debian package manager with GoogleContainerTools/rules_distroless#16
And a follow up to #1541