From ade216b378b18c8dc6b49d498d0611d2c56d4474 Mon Sep 17 00:00:00 2001 From: Aleksa Sarai Date: Wed, 16 Mar 2016 22:54:57 +1100 Subject: [PATCH] pkg: archive: don't fail Untar if xattrs are not supported Since certain filesystems don't support extended attributes, ignore errors produced (emitting a warning) when attempting to apply extended attributes to file. Signed-off-by: Aleksa Sarai --- archive/archive.go | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/archive/archive.go b/archive/archive.go index 47c5638..d1da240 100644 --- a/archive/archive.go +++ b/archive/archive.go @@ -425,10 +425,19 @@ func createTarFile(path, extractDir string, hdr *tar.Header, reader io.Reader, L } } + var errors []string for key, value := range hdr.Xattrs { if err := system.Lsetxattr(path, key, []byte(value), 0); err != nil { - return err + // We ignore errors here because not all graphdrivers support xattrs. + errors = append(errors, err.Error()) } + + } + + if len(errors) > 0 { + logrus.WithFields(logrus.Fields{ + "errors": errors, + }).Warn("ignored xattrs in archive: underlying filesystem doesn't support them") } // There is no LChmod, so ignore mode for symlink. Also, this