Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package minio-client for openSUSE:Factory checked in at 2024-02-02 15:46:57 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/minio-client (Old) and /work/SRC/openSUSE:Factory/.minio-client.new.1815 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "minio-client" Fri Feb 2 15:46:57 2024 rev:68 rq:1143482 version:20240131T085940Z Changes: -------- --- /work/SRC/openSUSE:Factory/minio-client/minio-client.changes 2024-01-30 18:25:23.557893485 +0100 +++ /work/SRC/openSUSE:Factory/.minio-client.new.1815/minio-client.changes 2024-02-02 15:47:11.160480624 +0100 @@ -1,0 +2,10 @@ +Thu Feb 01 18:34:36 UTC 2024 - opensuse_buildserv...@ojkastl.de + +- Update to version 20240131T085940Z: + * fix: mirror with local path (#4825) + * Fix syntax in examples (#4834) + * fix validation in replicate rm to allow stale target cleanup + (#4838) + * fix prometheus metrics response on success (#4835) + +------------------------------------------------------------------- Old: ---- mc-20240128T162314Z.obscpio New: ---- mc-20240131T085940Z.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ minio-client.spec ++++++ --- /var/tmp/diff_new_pack.dj1U56/_old 2024-02-02 15:47:17.344705067 +0100 +++ /var/tmp/diff_new_pack.dj1U56/_new 2024-02-02 15:47:17.344705067 +0100 @@ -22,7 +22,7 @@ %define binary_name minio-client Name: minio-client -Version: 20240128T162314Z +Version: 20240131T085940Z Release: 0 Summary: Client for MinIO License: AGPL-3.0-only ++++++ _service ++++++ --- /var/tmp/diff_new_pack.dj1U56/_old 2024-02-02 15:47:17.376706229 +0100 +++ /var/tmp/diff_new_pack.dj1U56/_new 2024-02-02 15:47:17.376706229 +0100 @@ -5,7 +5,7 @@ <param name="exclude">.git</param> <param name="changesgenerate">enable</param> <param name="versionformat">@PARENT_TAG@</param> - <param name="revision">RELEASE.2024-01-28T16-23-14Z</param> + <param name="revision">RELEASE.2024-01-31T08-59-40Z</param> <param name="match-tag">RELEASE.*</param> <param name="versionrewrite-pattern">RELEASE\.(.*)-(.*)-(.*)-(.*)-(.*)</param> <param name="versionrewrite-replacement">\1\2\3\4\5</param> @@ -19,7 +19,7 @@ <param name="compression">gz</param> </service> <service name="go_modules" mode="manual"> - <param name="archive">mc-20240128T162314Z.obscpio</param> + <param name="archive">mc-20240131T085940Z.obscpio</param> </service> </services> ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.dj1U56/_old 2024-02-02 15:47:17.396706954 +0100 +++ /var/tmp/diff_new_pack.dj1U56/_new 2024-02-02 15:47:17.400707100 +0100 @@ -1,6 +1,6 @@ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/minio/mc</param> - <param name="changesrevision">ba9fc6a3075df6d8a413cc7e423645706aaefae1</param></service></servicedata> + <param name="changesrevision">62b12a2770828f9d35ee7c5614d900b95ff6e6ab</param></service></servicedata> (No newline at EOF) ++++++ mc-20240128T162314Z.obscpio -> mc-20240131T085940Z.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/admin-prometheus-metrics.go new/mc-20240131T085940Z/cmd/admin-prometheus-metrics.go --- old/mc-20240128T162314Z/cmd/admin-prometheus-metrics.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/admin-prometheus-metrics.go 2024-01-31 09:59:40.000000000 +0100 @@ -118,6 +118,7 @@ if resp.StatusCode == http.StatusOK { printMsg(prometheusMetricsReader{Reader: resp.Body}) + return nil } return errors.New(resp.Status) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/anonymous-main.go new/mc-20240131T085940Z/cmd/anonymous-main.go --- old/mc-20240128T162314Z/cmd/anonymous-main.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/anonymous-main.go 2024-01-31 09:59:40.000000000 +0100 @@ -80,7 +80,7 @@ {{.Prompt}} {{.HelpName}} set public s3/public-commons/images 5. Set a custom prefix based bucket anonymous on Amazon S3 cloud storage using a JSON file. - {{.Prompt}} {{.HelpName}} set-json s3/public-commons/images /path/to/anonymous.json + {{.Prompt}} {{.HelpName}} set-json /path/to/anonymous.json s3/public-commons/images 6. Get bucket permissions. {{.Prompt}} {{.HelpName}} get s3/shared @@ -488,7 +488,7 @@ switch ctx.Args().First() { case "set", "set-json", "get", "get-json": // anonymous set [private|public|download|upload] alias/bucket/prefix - // anonymous set-json alias/bucket/prefix path-to-anonymous-json-file + // anonymous set-json path-to-anonymous-json-file alias/bucket/prefix // anonymous get alias/bucket/prefix // anonymous get-json alias/bucket/prefix runAnonymousCmd(ctx.Args()) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/difference_test.go new/mc-20240131T085940Z/cmd/difference_test.go --- old/mc-20240128T162314Z/cmd/difference_test.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/difference_test.go 2024-01-31 09:59:40.000000000 +0100 @@ -24,26 +24,40 @@ var testCases = []struct { pattern []string - object string + srcSuffix string match bool + + typ ClientURLType }{ - {nil, "testfile", false}, - {[]string{"test*"}, "testfile", true}, - {[]string{"file*"}, "file/abc/bcd/def", true}, - {[]string{"*"}, "file/abc/bcd/def", true}, - {[]string{""}, "file/abc/bcd/def", false}, - {[]string{"abc*"}, "file/abc/bcd/def", false}, - {[]string{"abc*", "*abc/*"}, "file/abc/bcd/def", true}, - {[]string{"*.txt"}, "file/abc/bcd/def.txt", true}, - {[]string{".*"}, ".sys", true}, - {[]string{"*."}, ".sys.", true}, + {nil, "testfile", false, objectStorage}, + {[]string{"test*"}, "testfile", true, objectStorage}, + {[]string{"file*"}, "file/abc/bcd/def", true, objectStorage}, + {[]string{"*"}, "file/abc/bcd/def", true, objectStorage}, + {[]string{""}, "file/abc/bcd/def", false, objectStorage}, + {[]string{"abc*"}, "file/abc/bcd/def", false, objectStorage}, + {[]string{"abc*", "*abc/*"}, "file/abc/bcd/def", true, objectStorage}, + {[]string{"*.txt"}, "file/abc/bcd/def.txt", true, objectStorage}, + {[]string{".*"}, ".sys", true, objectStorage}, + {[]string{"*."}, ".sys.", true, objectStorage}, + {nil, "testfile", false, fileSystem}, + {[]string{"test*"}, "testfile", true, fileSystem}, + {[]string{"file*"}, "file/abc/bcd/def", true, fileSystem}, + {[]string{"*"}, "file/abc/bcd/def", true, fileSystem}, + {[]string{""}, "file/abc/bcd/def", false, fileSystem}, + {[]string{"abc*"}, "file/abc/bcd/def", false, fileSystem}, + {[]string{"abc*", "*abc/*"}, "file/abc/bcd/def", true, fileSystem}, + {[]string{"abc*", "*abc/*"}, "/file/abc/bcd/def", true, fileSystem}, + {[]string{"*.txt"}, "file/abc/bcd/def.txt", true, fileSystem}, + {[]string{"*.txt"}, "/file/abc/bcd/def.txt", true, fileSystem}, + {[]string{".*"}, ".sys", true, fileSystem}, + {[]string{"*."}, ".sys.", true, fileSystem}, } func TestExcludeOptions(t *testing.T) { for _, test := range testCases { - if matchExcludeOptions(test.pattern, test.object) != test.match { - t.Fatalf("Unexpected result %t, with pattern %s and object %s \n", !test.match, test.pattern, test.object) + if matchExcludeOptions(test.pattern, test.srcSuffix, test.typ) != test.match { + t.Fatalf("Unexpected result %t, with pattern %s and srcSuffix %s \n", !test.match, test.pattern, test.srcSuffix) } } } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/mirror-main.go new/mc-20240131T085940Z/cmd/mirror-main.go --- old/mc-20240128T162314Z/cmd/mirror-main.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/mirror-main.go 2024-01-31 09:59:40.000000000 +0100 @@ -628,7 +628,7 @@ // joined to the targetURL. sourceSuffix := strings.TrimPrefix(eventPath, sourceURLFull) // Skip the object, if it matches the Exclude options provided - if matchExcludeOptions(mj.opts.excludeOptions, sourceSuffix) { + if matchExcludeOptions(mj.opts.excludeOptions, sourceSuffix, sourceURL.Type) { continue } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/mirror-url.go new/mc-20240131T085940Z/cmd/mirror-url.go --- old/mc-20240128T162314Z/cmd/mirror-url.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/mirror-url.go 2024-01-31 09:59:40.000000000 +0100 @@ -89,7 +89,15 @@ return } -func matchExcludeOptions(excludeOptions []string, srcSuffix string) bool { +func matchExcludeOptions(excludeOptions []string, srcSuffix string, typ ClientURLType) bool { + // if type is file system, remove leading slash + if typ == fileSystem { + if strings.HasPrefix(srcSuffix, "/") { + srcSuffix = srcSuffix[1:] + } else if runtime.GOOS == "windows" && strings.HasPrefix(srcSuffix, `\`) { + srcSuffix = srcSuffix[1:] + } + } for _, pattern := range excludeOptions { if wildcard.Match(pattern, srcSuffix) { return true @@ -146,13 +154,13 @@ srcSuffix := strings.TrimPrefix(diffMsg.FirstURL, sourceURL) // Skip the source object if it matches the Exclude options provided - if matchExcludeOptions(opts.excludeOptions, srcSuffix) { + if matchExcludeOptions(opts.excludeOptions, srcSuffix, newClientURL(sourceURL).Type) { continue } tgtSuffix := strings.TrimPrefix(diffMsg.SecondURL, targetURL) // Skip the target object if it matches the Exclude options provided - if matchExcludeOptions(opts.excludeOptions, tgtSuffix) { + if matchExcludeOptions(opts.excludeOptions, tgtSuffix, newClientURL(targetURL).Type) { continue } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/mc-20240128T162314Z/cmd/replicate-remove.go new/mc-20240131T085940Z/cmd/replicate-remove.go --- old/mc-20240128T162314Z/cmd/replicate-remove.go 2024-01-28 17:23:14.000000000 +0100 +++ new/mc-20240131T085940Z/cmd/replicate-remove.go 2024-01-31 09:59:40.000000000 +0100 @@ -19,7 +19,6 @@ import ( "context" - "errors" "github.com/fatih/color" "github.com/minio/cli" @@ -131,13 +130,18 @@ rcfg, err := client.GetReplication(ctx) fatalIf(err.Trace(args...), "Unable to get replication configuration") - if rcfg.Empty() { - fatalIf(probe.NewError(errors.New("replication configuration not set")).Trace(aliasedURL), - "Unable to remove replication configuration") - } rmAll := cliCtx.Bool("all") rmForce := cliCtx.Bool("force") ruleID := cliCtx.String("id") + + if rcfg.Empty() && !rmAll { + printMsg(replicateRemoveMessage{ + Op: cliCtx.Command.Name, + Status: "success", + URL: aliasedURL, + }) + return nil + } if rmAll && rmForce { fatalIf(client.RemoveReplication(ctx), "Unable to remove replication configuration") } else { ++++++ mc.obsinfo ++++++ --- /var/tmp/diff_new_pack.dj1U56/_old 2024-02-02 15:47:17.668716827 +0100 +++ /var/tmp/diff_new_pack.dj1U56/_new 2024-02-02 15:47:17.672716972 +0100 @@ -1,5 +1,5 @@ name: mc -version: 20240128T162314Z -mtime: 1706458994 -commit: ba9fc6a3075df6d8a413cc7e423645706aaefae1 +version: 20240131T085940Z +mtime: 1706691580 +commit: 62b12a2770828f9d35ee7c5614d900b95ff6e6ab ++++++ vendor.tar.gz ++++++ /work/SRC/openSUSE:Factory/minio-client/vendor.tar.gz /work/SRC/openSUSE:Factory/.minio-client.new.1815/vendor.tar.gz differ: char 5, line 1