Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package kargo-cli for openSUSE:Factory checked in at 2026-02-02 14:56:13 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/kargo-cli (Old) and /work/SRC/openSUSE:Factory/.kargo-cli.new.1995 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "kargo-cli" Mon Feb 2 14:56:13 2026 rev:43 rq:1330277 version:1.9.1 Changes: -------- --- /work/SRC/openSUSE:Factory/kargo-cli/kargo-cli.changes 2026-01-30 18:26:49.844996798 +0100 +++ /work/SRC/openSUSE:Factory/.kargo-cli.new.1995/kargo-cli.changes 2026-02-02 14:56:34.706028465 +0100 @@ -1,0 +2,6 @@ +Sun Feb 01 19:35:45 UTC 2026 - Johannes Kastl <[email protected]> + +- Update to version 1.9.1: + no CLI-related changes or dependency updates + +------------------------------------------------------------------- Old: ---- kargo-cli-1.9.0.obscpio New: ---- kargo-cli-1.9.1.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ kargo-cli.spec ++++++ --- /var/tmp/diff_new_pack.hJBV2D/_old 2026-02-02 14:56:37.670153753 +0100 +++ /var/tmp/diff_new_pack.hJBV2D/_new 2026-02-02 14:56:37.670153753 +0100 @@ -19,7 +19,7 @@ %define executable_name kargo Name: kargo-cli -Version: 1.9.0 +Version: 1.9.1 Release: 0 Summary: CLI for the Kubernetes Application lifecycle orchestration License: Apache-2.0 ++++++ _service ++++++ --- /var/tmp/diff_new_pack.hJBV2D/_old 2026-02-02 14:56:37.926164574 +0100 +++ /var/tmp/diff_new_pack.hJBV2D/_new 2026-02-02 14:56:37.954165758 +0100 @@ -3,7 +3,7 @@ <param name="url">https://github.com/akuity/kargo</param> <param name="scm">git</param> <param name="exclude">.git</param> - <param name="revision">v1.9.0</param> + <param name="revision">v1.9.1</param> <param name="versionformat">@PARENT_TAG@</param> <param name="versionrewrite-pattern">v(.*)</param> <param name="changesgenerate">enable</param> ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.hJBV2D/_old 2026-02-02 14:56:37.998167617 +0100 +++ /var/tmp/diff_new_pack.hJBV2D/_new 2026-02-02 14:56:38.002167787 +0100 @@ -1,6 +1,6 @@ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/akuity/kargo</param> - <param name="changesrevision">d16522090b99645053862138dacd7191ba44d207</param></service></servicedata> + <param name="changesrevision">e594e1508d679aff1a64961cd59cd87a6fb1a6f9</param></service></servicedata> (No newline at EOF) ++++++ kargo-cli-1.9.0.obscpio -> kargo-cli-1.9.1.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/docs/docs/50-user-guide/60-reference-docs/30-promotion-steps/git-clone.md new/kargo-cli-1.9.1/docs/docs/50-user-guide/60-reference-docs/30-promotion-steps/git-clone.md --- old/kargo-cli-1.9.0/docs/docs/50-user-guide/60-reference-docs/30-promotion-steps/git-clone.md 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/docs/docs/50-user-guide/60-reference-docs/30-promotion-steps/git-clone.md 2026-01-31 03:11:53.000000000 +0100 @@ -37,7 +37,7 @@ | `checkout[].commit` | `string` | N | A specific commit to check out. Mutually exclusive with `branch` and `tag`. If none of these is specified, the default branch will be checked out. | | `checkout[].path` | `string` | Y | The path for a working tree that will be created from the checked out revision. This path is relative to the temporary workspace that Kargo provisions for use by the promotion process. | | `checkout[].tag` | `string` | N | A tag to check out. Mutually exclusive with `branch` and `commit`. If none of these is specified, the default branch will be checked out. | -| `checkout[].sparse` | `[]string` | N | Directory paths for sparse checkout. Only the specified directories (and their contents) will be checked out. Paths must be relative to the repository root (e.g., `src/app`, `configs/prod`). Glob patterns are not supported. When all checkouts use sparse patterns, a [blobless clone][] is performed automatically to reduce clone time and disk usage. | +| `checkout[].sparse` | `[]string` | N | Directory paths for sparse checkout. Only the specified directories (and their contents) will be checked out. Paths must be relative to the repository root (e.g., `src/app`, `configs/prod`). Glob patterns are not supported. | ## Output diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/component/map_based_registry.go new/kargo-cli-1.9.1/pkg/component/map_based_registry.go --- old/kargo-cli-1.9.0/pkg/component/map_based_registry.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/component/map_based_registry.go 2026-01-31 03:11:53.000000000 +0100 @@ -41,6 +41,9 @@ func (r *mapBasedRegistry[V, MD]) Register( reg NameBasedRegistration[V, MD], ) error { + if reg.Name == "" { + return fmt.Errorf("registration name cannot be empty") + } r.mu.Lock() defer r.mu.Unlock() if _, exists := r.registrations[reg.Name]; exists && !r.opts.AllowOverwriting { diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/component/map_based_registry_test.go new/kargo-cli-1.9.1/pkg/component/map_based_registry_test.go --- old/kargo-cli-1.9.0/pkg/component/map_based_registry_test.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/component/map_based_registry_test.go 2026-01-31 03:11:53.000000000 +0100 @@ -145,15 +145,28 @@ Metadata: "meta", } testCases := []struct { - name string - registry *namedRegistry - assertions func(*testing.T, *namedRegistry, error) + name string + registry *namedRegistry + newRegistration namedRegistration + assertions func(*testing.T, *namedRegistry, error) }{ { + name: "no name", + registry: &namedRegistry{ + registrations: map[string]namedRegistration{}, + }, + newRegistration: namedRegistration{}, + assertions: func(t *testing.T, registry *namedRegistry, err error) { + require.ErrorContains(t, err, "registration name cannot be empty") + require.Empty(t, registry.registrations) + }, + }, + { name: "basic registration", registry: &namedRegistry{ registrations: map[string]namedRegistration{}, }, + newRegistration: testReg, assertions: func(t *testing.T, registry *namedRegistry, err error) { require.NoError(t, err) require.Equal( @@ -169,6 +182,7 @@ opts: NameBasedRegistryOptions{AllowOverwriting: false}, registrations: map[string]namedRegistration{testReg.Name: testReg}, }, + newRegistration: testReg, assertions: func(t *testing.T, registry *namedRegistry, err error) { require.ErrorContains(t, err, "cannot overwrite registration") require.Equal( @@ -184,6 +198,7 @@ opts: NameBasedRegistryOptions{AllowOverwriting: false}, registrations: map[string]namedRegistration{testReg.Name: testReg}, }, + newRegistration: testReg, assertions: func(t *testing.T, registry *namedRegistry, err error) { require.ErrorContains(t, err, "cannot overwrite registration") require.Equal( @@ -199,7 +214,7 @@ testCase.assertions( t, testCase.registry, - testCase.registry.Register(testReg), + testCase.registry.Register(testCase.newRegistration), ) }) } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/controller/git/bare_repo.go new/kargo-cli-1.9.1/pkg/controller/git/bare_repo.go --- old/kargo-cli-1.9.0/pkg/controller/git/bare_repo.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/controller/git/bare_repo.go 2026-01-31 03:11:53.000000000 +0100 @@ -125,11 +125,14 @@ return b, nil } -func (b *bareRepo) clone(opts *BareCloneOptions) error { +func (b *bareRepo) clone(_ *BareCloneOptions) error { args := []string{"clone", "--bare"} - if opts.Filter != "" { - args = append(args, "--filter", opts.Filter) - } + // NOTE(hidde): Temporarily disabled until we figure out why this can result + // in "could not fetch <commit> from promisor remote" errors. + // + // if opts.Filter != "" { + // args = append(args, "--filter", opts.Filter) + // } args = append(args, b.accessURL, b.dir) cmd := b.buildGitCommand(args...) cmd.Dir = b.homeDir // Override the cmd.Dir that's set by r.buildGitCommand() diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/controller/git/repo.go new/kargo-cli-1.9.1/pkg/controller/git/repo.go --- old/kargo-cli-1.9.0/pkg/controller/git/repo.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/controller/git/repo.go 2026-01-31 03:11:53.000000000 +0100 @@ -128,9 +128,12 @@ if opts.Depth > 0 { args = append(args, "--depth", fmt.Sprint(opts.Depth)) } - if opts.Filter != "" { - args = append(args, "--filter", opts.Filter) - } + // NOTE(hidde): Temporarily disabled until we figure out why this can result + // in "could not fetch <commit> from promisor remote" errors. + // + // if opts.Filter != "" { + // args = append(args, "--filter", opts.Filter) + // } args = append(args, r.accessURL, r.dir) cmd := r.buildGitCommand(args...) cmd.Dir = r.homeDir // Override the cmd.Dir that's set by r.buildGitCommand() diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/http/http.go new/kargo-cli-1.9.1/pkg/http/http.go --- old/kargo-cli-1.9.0/pkg/http/http.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/http/http.go 2026-01-31 03:11:53.000000000 +0100 @@ -2,6 +2,7 @@ import ( "encoding/json" + "fmt" "net/http" "time" ) @@ -26,8 +27,8 @@ if w == nil { return } - w.Header().Set("Cache-Control", "public, max-age="+maxAge.String()) - w.Header().Set("Expires", time.Now().Add(timeUntilExpiry).Format(time.RFC1123)) + w.Header().Set("Cache-Control", fmt.Sprintf("public, max-age=%d", int(maxAge.Seconds()))) + w.Header().Set("Expires", time.Now().UTC().Add(timeUntilExpiry).Format(http.TimeFormat)) } func WriteResponseJSON(w http.ResponseWriter, code int, body any) { diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/http/http_test.go new/kargo-cli-1.9.1/pkg/http/http_test.go --- old/kargo-cli-1.9.0/pkg/http/http_test.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/http/http_test.go 2026-01-31 03:11:53.000000000 +0100 @@ -4,6 +4,7 @@ "net/http" "net/http/httptest" "testing" + "time" "github.com/stretchr/testify/require" ) @@ -36,3 +37,12 @@ }) } } + +func TestSetCacheHeaders(t *testing.T) { + w := httptest.NewRecorder() + SetCacheHeaders(w, 30*24*time.Hour, 7*24*time.Hour) + headers := w.Result().Header + require.Equal(t, "public, max-age=2592000", headers.Get("Cache-Control")) + expires := headers.Get("Expires") + require.Equal(t, expires[len(expires)-4:], " GMT", "Expires header must end with ' GMT', got '%s'.", expires) +} diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/kargo-cli-1.9.0/pkg/server/watch_warehouses_v1alpha1.go new/kargo-cli-1.9.1/pkg/server/watch_warehouses_v1alpha1.go --- old/kargo-cli-1.9.0/pkg/server/watch_warehouses_v1alpha1.go 2026-01-29 05:00:42.000000000 +0100 +++ new/kargo-cli-1.9.1/pkg/server/watch_warehouses_v1alpha1.go 2026-01-31 03:11:53.000000000 +0100 @@ -60,6 +60,12 @@ if !ok { return fmt.Errorf("unexpected object type %T", e.Object) } + // Necessary because serializing a Warehouse as part of a protobuf + // message does not apply custom marshaling. The call to this helper + // compensates for that. + if err := prepareOutboundWarehouse(warehouse); err != nil { + return fmt.Errorf("prepare outbound warehouse: %w", err) + } if err := stream.Send(&svcv1alpha1.WatchWarehousesResponse{ Warehouse: warehouse, Type: string(e.Type), ++++++ kargo-cli.obsinfo ++++++ --- /var/tmp/diff_new_pack.hJBV2D/_old 2026-02-02 14:56:43.686408048 +0100 +++ /var/tmp/diff_new_pack.hJBV2D/_new 2026-02-02 14:56:43.706408894 +0100 @@ -1,5 +1,5 @@ name: kargo-cli -version: 1.9.0 -mtime: 1769659242 -commit: d16522090b99645053862138dacd7191ba44d207 +version: 1.9.1 +mtime: 1769825513 +commit: e594e1508d679aff1a64961cd59cd87a6fb1a6f9 ++++++ vendor.tar.gz ++++++ /work/SRC/openSUSE:Factory/kargo-cli/vendor.tar.gz /work/SRC/openSUSE:Factory/.kargo-cli.new.1995/vendor.tar.gz differ: char 13, line 1
