summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Holzinger <pholzing@redhat.com>2022-03-15 13:58:55 +0100
committerPaul Holzinger <pholzing@redhat.com>2022-03-15 15:06:42 +0100
commit0db184fb08d2cc41281e4d15d82aed8e5f9fb7ad (patch)
tree846318d381b7edada61886d4c7cb6845e471f702
parent68ce83fe919f2d37762b8b746a73495f45e550f3 (diff)
downloadpodman-0db184fb08d2cc41281e4d15d82aed8e5f9fb7ad.tar.gz
podman-0db184fb08d2cc41281e4d15d82aed8e5f9fb7ad.tar.bz2
podman-0db184fb08d2cc41281e4d15d82aed8e5f9fb7ad.zip
fix breaking change in pkg/bindings
pkg/bindings is considered stable. We cannot make changes that would break any users. If someone uses this field their code would fail to compile. Since the fix is obviously correct we will keep it but also add the old field back in to keep compatibility with old code. When both fields are set ImportArchive is preferred over ImportAchive. Fixes changes from commit 217197340c8f [NO NEW TESTS NEEDED] Signed-off-by: Paul Holzinger <pholzing@redhat.com>
-rw-r--r--pkg/bindings/containers/checkpoint.go9
-rw-r--r--pkg/bindings/containers/types.go23
-rw-r--r--pkg/bindings/containers/types_restore_options.go15
3 files changed, 38 insertions, 9 deletions
diff --git a/pkg/bindings/containers/checkpoint.go b/pkg/bindings/containers/checkpoint.go
index 84590d052..bcb944488 100644
--- a/pkg/bindings/containers/checkpoint.go
+++ b/pkg/bindings/containers/checkpoint.go
@@ -79,7 +79,14 @@ func Restore(ctx context.Context, nameOrID string, options *RestoreOptions) (*en
// Open the to-be-imported archive if needed.
var r io.Reader
- if i := options.GetImportArchive(); i != "" {
+ i := options.GetImportArchive()
+ if i == "" {
+ // backwards compat, ImportAchive is a typo but we still have to
+ // support this to avoid breaking users
+ // TODO: remove ImportAchive with 5.0
+ i = options.GetImportAchive()
+ }
+ if i != "" {
params.Set("import", "true")
r, err = os.Open(i)
if err != nil {
diff --git a/pkg/bindings/containers/types.go b/pkg/bindings/containers/types.go
index 3c8b1eefa..c87f82bf4 100644
--- a/pkg/bindings/containers/types.go
+++ b/pkg/bindings/containers/types.go
@@ -64,14 +64,21 @@ type RestoreOptions struct {
IgnoreVolumes *bool
IgnoreStaticIP *bool
IgnoreStaticMAC *bool
- ImportArchive *string
- Keep *bool
- Name *string
- TCPEstablished *bool
- Pod *string
- PrintStats *bool
- PublishPorts []string
- FileLocks *bool
+ // ImportAchive is the path to an archive which contains the checkpoint data.
+ //
+ // Deprecated: Use ImportArchive instead. This field name is a typo and
+ // will be removed in a future major release.
+ ImportAchive *string
+ // ImportArchive is the path to an archive which contains the checkpoint data.
+ // ImportArchive is preferred over ImportAchive when both are set.
+ ImportArchive *string
+ Keep *bool
+ Name *string
+ TCPEstablished *bool
+ Pod *string
+ PrintStats *bool
+ PublishPorts []string
+ FileLocks *bool
}
//go:generate go run ../generator/generator.go CreateOptions
diff --git a/pkg/bindings/containers/types_restore_options.go b/pkg/bindings/containers/types_restore_options.go
index e8a0e236c..b1b14a704 100644
--- a/pkg/bindings/containers/types_restore_options.go
+++ b/pkg/bindings/containers/types_restore_options.go
@@ -77,6 +77,21 @@ func (o *RestoreOptions) GetIgnoreStaticMAC() bool {
return *o.IgnoreStaticMAC
}
+// WithImportAchive set field ImportAchive to given value
+func (o *RestoreOptions) WithImportAchive(value string) *RestoreOptions {
+ o.ImportAchive = &value
+ return o
+}
+
+// GetImportAchive returns value of field ImportAchive
+func (o *RestoreOptions) GetImportAchive() string {
+ if o.ImportAchive == nil {
+ var z string
+ return z
+ }
+ return *o.ImportAchive
+}
+
// WithImportArchive set field ImportArchive to given value
func (o *RestoreOptions) WithImportArchive(value string) *RestoreOptions {
o.ImportArchive = &value