Skip to content

Commit ea1eeb5

Browse files
committed
remove github.com/crc-org/crc/v2 dependency
There is one user in the code and one in the tests. First the test is testing an external function that doesn't seem to make a lot of sense. Second when we pull the machine image and renmae doesn't work it would mean it is on a different disk. That should never happen. The oci pull is to ~/.local/share/containers/podman/machine/<provider>/cache/<sha>/... and then it gets moved to ~/.local/share/containers/podman/machine/qemu/cache/<sha>.<extension> These can under normal circumstances never be on different disk as it would mean the pulled sha is known ahead of time and then that the blob path is a mount point which is unrealistic and not something we have to support. It also removes three transitive dependencies. Signed-off-by: Paul Holzinger <[email protected]>
1 parent 49c42d0 commit ea1eeb5

39 files changed

+1
-3340
lines changed

go.mod

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ require (
2424
github.com/containers/storage v1.57.3-0.20250325222852-4d1ae4a7983d
2525
github.com/containers/winquit v1.1.0
2626
github.com/coreos/go-systemd/v22 v22.5.1-0.20231103132048-7d375ecc2b09
27-
github.com/crc-org/crc/v2 v2.45.0
2827
github.com/crc-org/vfkit v0.6.0
2928
github.com/cyphar/filepath-securejoin v0.4.1
3029
github.com/digitalocean/go-qemu v0.0.0-20250212194115-ee9b0668d242
@@ -150,8 +149,6 @@ require (
150149
github.com/lufia/plan9stats v0.0.0-20211012122336-39d0f177ccd0 // indirect
151150
github.com/mailru/easyjson v0.7.7 // indirect
152151
github.com/manifoldco/promptui v0.9.0 // indirect
153-
github.com/mattn/go-colorable v0.1.13 // indirect
154-
github.com/mattn/go-isatty v0.0.20 // indirect
155152
github.com/mattn/go-runewidth v0.0.16 // indirect
156153
github.com/mdlayher/socket v0.5.1 // indirect
157154
github.com/miekg/pkcs11 v1.1.1 // indirect
@@ -213,7 +210,6 @@ require (
213210
google.golang.org/genproto/googleapis/api v0.0.0-20241209162323-e6fa225c2576 // indirect
214211
google.golang.org/genproto/googleapis/rpc v0.0.0-20250115164207-1a7da9e5054f // indirect
215212
google.golang.org/grpc v1.70.0 // indirect
216-
gopkg.in/natefinch/lumberjack.v2 v2.2.1 // indirect
217213
gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7 // indirect
218214
tags.cncf.io/container-device-interface/specs-go v1.0.0 // indirect
219215
)

go.sum

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -99,8 +99,6 @@ github.com/coreos/go-systemd v0.0.0-20190719114852-fd7a80b32e1f/go.mod h1:F5haX7
9999
github.com/coreos/go-systemd/v22 v22.5.1-0.20231103132048-7d375ecc2b09 h1:OoRAFlvDGCUqDLampLQjk0yeeSGdF9zzst/3G9IkBbc=
100100
github.com/coreos/go-systemd/v22 v22.5.1-0.20231103132048-7d375ecc2b09/go.mod h1:m2r/smMKsKwgMSAoFKHaa68ImdCSNuKE1MxvQ64xuCQ=
101101
github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
102-
github.com/crc-org/crc/v2 v2.45.0 h1:7rScVQwFc3oy99SaiGSgQmTiIQf7UKKYIcZQ963NsdE=
103-
github.com/crc-org/crc/v2 v2.45.0/go.mod h1:ALUixzv85cMYNDEqone8jEn3m2ZKyu9oRzrQdEAjnzk=
104102
github.com/crc-org/vfkit v0.6.0 h1:gUasCX2QqY9pUPebFhYsuINB8XSS/iz0qy4v18CUyB4=
105103
github.com/crc-org/vfkit v0.6.0/go.mod h1:i+fGyDMg5MpuUYCFXc2VXw+5R7MBD6A/8xU9UxWv/9s=
106104
github.com/creack/pty v1.1.21 h1:1/QdRyBaHHJP61QkWMXlOIBfsgdDeeKfK8SYVUWJKf0=
@@ -306,7 +304,6 @@ github.com/manifoldco/promptui v0.9.0 h1:3V4HzJk1TtXW1MTZMP7mdlwbBpIinw3HztaIlYt
306304
github.com/manifoldco/promptui v0.9.0/go.mod h1:ka04sppxSGFAtxX0qhlYQjISsg9mR4GWtQEhdbn6Pgg=
307305
github.com/mattn/go-colorable v0.1.13 h1:fFA4WZxdEF4tXPZVKMLwD8oUnCTTo08duU7wxecdEvA=
308306
github.com/mattn/go-colorable v0.1.13/go.mod h1:7S9/ev0klgBDR4GtXTXX8a3vIGJpMovkB8vQcUbaXHg=
309-
github.com/mattn/go-isatty v0.0.16/go.mod h1:kYGgaQfpe5nmfYZH+SKPsOc2e4SrIfOl2e/yFXSvRLM=
310307
github.com/mattn/go-isatty v0.0.20 h1:xfD0iDuEKnDkl03q4limB+vH+GxLEtL/jb4xVJSWWEY=
311308
github.com/mattn/go-isatty v0.0.20/go.mod h1:W+V8PltTTMOvKvAeJH7IuucS94S2C6jfK/D7dTCTo3Y=
312309
github.com/mattn/go-runewidth v0.0.16 h1:E5ScNMtiwvlvB5paMFdw9p4kSQzbXFikJ5SQO6TULQc=
@@ -618,12 +615,10 @@ golang.org/x/sys v0.0.0-20220310020820-b874c991c1a5/go.mod h1:oPkhp1MJrh7nUepCBc
618615
golang.org/x/sys v0.0.0-20220520151302-bc2c85ada10a/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
619616
golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
620617
golang.org/x/sys v0.0.0-20220722155257-8c9f86f7a55f/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
621-
golang.org/x/sys v0.0.0-20220811171246-fbc7d0a398ab/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
622618
golang.org/x/sys v0.0.0-20220908164124-27713097b956/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
623619
golang.org/x/sys v0.1.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
624620
golang.org/x/sys v0.2.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
625621
golang.org/x/sys v0.5.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
626-
golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
627622
golang.org/x/sys v0.8.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
628623
golang.org/x/sys v0.10.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
629624
golang.org/x/sys v0.11.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
@@ -708,8 +703,6 @@ gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c h1:Hei/4ADfdWqJk1ZMxUNpqntN
708703
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c/go.mod h1:JHkPIbrfpd72SG/EVd6muEfDQjcINNoR0C8j2r3qZ4Q=
709704
gopkg.in/inf.v0 v0.9.1 h1:73M5CoZyi3ZLMOyDlQh031Cx6N9NDJ2Vvfl76EDAgDc=
710705
gopkg.in/inf.v0 v0.9.1/go.mod h1:cWUDdTG/fYaXco+Dcufb5Vnc6Gp2YChqWtbxRZE0mXw=
711-
gopkg.in/natefinch/lumberjack.v2 v2.2.1 h1:bBRl1b0OH9s/DuPhuXpNl+VtCaJXFZ5/uEFST95x9zc=
712-
gopkg.in/natefinch/lumberjack.v2 v2.2.1/go.mod h1:YD8tP3GAjkrDg1eZH7EGmyESg/lsYskCTPBJVb9jqSc=
713706
gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7 h1:uRGJdciOHaEIrze2W8Q3AKkepLTh2hOroT7a+7czfdQ=
714707
gopkg.in/tomb.v1 v1.0.0-20141024135613-dd632973f1e7/go.mod h1:dt/ZhP58zS4L8KSrWDmTeBkI65Dw0HsyUHuEVlX15mw=
715708
gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=

pkg/machine/compression/copy_test.go

Lines changed: 0 additions & 42 deletions
This file was deleted.

pkg/machine/ocipull/ociartifact.go

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import (
1818
"github.com/containers/podman/v5/pkg/machine/compression"
1919
"github.com/containers/podman/v5/pkg/machine/define"
2020
"github.com/containers/podman/v5/utils"
21-
crc "github.com/crc-org/crc/v2/pkg/os"
2221
"github.com/opencontainers/go-digest"
2322
specV1 "github.com/opencontainers/image-spec/specs-go/v1"
2423
"github.com/sirupsen/logrus"
@@ -293,13 +292,8 @@ func (o *OCIArtifactDisk) unpack(diskArtifactHash digest.Digest) error {
293292
diskBlobPath := filepath.Join(blobDir.GetPath(), "blobs", "sha256", blobInfo.Digest.Encoded())
294293

295294
// Rename and move the hashed blob file to the cache dir.
296-
// If the rename fails, we do a sparsecopy instead
297295
if err := os.Rename(diskBlobPath, cachedCompressedPath.GetPath()); err != nil {
298-
logrus.Errorf("renaming compressed image %q failed: %q", cachedCompressedPath.GetPath(), err)
299-
logrus.Error("trying again using copy")
300-
if err := crc.CopyFileSparse(diskBlobPath, cachedCompressedPath.GetPath()); err != nil {
301-
return err
302-
}
296+
return fmt.Errorf("failed to move downloaded blob to cache: %w", err)
303297
}
304298

305299
// Clean up the oci dir which is no longer needed

vendor/github.com/crc-org/crc/v2/LICENSE

Lines changed: 0 additions & 191 deletions
This file was deleted.

vendor/github.com/crc-org/crc/v2/pkg/crc/logging/inmemory.go

Lines changed: 0 additions & 44 deletions
This file was deleted.

0 commit comments

Comments
 (0)