mirror of
https://github.com/project-zot/zot.git
synced 2026-06-17 21:17:58 +08:00
bc5fd1a357
* feat: add events config Signed-off-by: Piaras Hoban <phoban01@gmail.com> * feat: implement event support with log sink Signed-off-by: Piaras Hoban <phoban01@gmail.com> * feat: integrate events and update tests Signed-off-by: Piaras Hoban <phoban01@gmail.com> * refactor: update event config Signed-off-by: Piaras Hoban <phoban01@gmail.com> * feat: implement http and nats sinks. remove log sink Signed-off-by: Piaras Hoban <phoban01@gmail.com> * refactor: events extension setup Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: cleanup tests to use nil event recorder Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: update events config example and add more logging Signed-off-by: Piaras Hoban <phoban01@gmail.com> * refactor: better use of build tags for minimal binary Signed-off-by: Piaras Hoban <phoban01@gmail.com> * fix: missing store param in evelated privileges tests Signed-off-by: Piaras Hoban <phoban01@gmail.com> * fix: regression in config decoding Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: update check logs script to enable cross-platform usage via GREP_BIN_PATH envvar Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: fix log lint issue for events Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: fix failing events disabled test Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: add blackbox tests for events Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: specify architecture when downloading binaries in Makefile Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: improve failure handling when no valid sinks are provided Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: fix data race in events test Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: cleanup event decoding Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: fix logging tests Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: make nats server test more reliable Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: go mod cleanup Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: add sleep when setting up nats client Signed-off-by: Piaras Hoban <phoban01@gmail.com> * fix: ensure event sink errors do not propogate Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: increase coverage for events Signed-off-by: Piaras Hoban <phoban01@gmail.com> * feat(events): Refactor events to be non-blocking from caller. Signed-off-by: Asgeir Nilsen <asgeir.nilsen@bouvet.no> Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: remove harded-coded linux Co-authored-by: Andrei Aaron <andreifdaaron@gmail.com> Signed-off-by: Piaras Hoban <phoban01@gmail.com> * feat(events): fail to start if incorrect event sink is configured Signed-off-by: Piaras Hoban <phoban01@gmail.com> * test: allow cli tests to return errors instead of panic Signed-off-by: Piaras Hoban <phoban01@gmail.com> * chore: bump nats server to v2.11.3 Signed-off-by: Piaras Hoban <phoban01@gmail.com> --------- Signed-off-by: Piaras Hoban <phoban01@gmail.com> Signed-off-by: Asgeir Nilsen <asgeir.nilsen@bouvet.no> Co-authored-by: Asgeir Nilsen <asgeir.nilsen@bouvet.no> Co-authored-by: Andrei Aaron <andreifdaaron@gmail.com>
161 lines
3.8 KiB
Go
161 lines
3.8 KiB
Go
package config_test
|
|
|
|
import (
|
|
"testing"
|
|
"time"
|
|
|
|
. "github.com/smartystreets/goconvey/convey"
|
|
|
|
"zotregistry.dev/zot/pkg/api/config"
|
|
extconf "zotregistry.dev/zot/pkg/extensions/config"
|
|
"zotregistry.dev/zot/pkg/extensions/config/events"
|
|
)
|
|
|
|
func TestConfig(t *testing.T) {
|
|
Convey("Test config utils", t, func() {
|
|
firstStorageConfig := config.StorageConfig{
|
|
GC: true, Dedupe: true,
|
|
GCDelay: 1 * time.Minute, GCInterval: 1 * time.Hour,
|
|
}
|
|
secondStorageConfig := config.StorageConfig{
|
|
GC: true, Dedupe: true,
|
|
GCDelay: 1 * time.Minute, GCInterval: 1 * time.Hour,
|
|
}
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeTrue)
|
|
|
|
firstStorageConfig.GC = false
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeFalse)
|
|
|
|
firstStorageConfig.GC = true
|
|
firstStorageConfig.Dedupe = false
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeFalse)
|
|
|
|
firstStorageConfig.Dedupe = true
|
|
firstStorageConfig.GCDelay = 2 * time.Minute
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeFalse)
|
|
|
|
firstStorageConfig.GCDelay = 1 * time.Minute
|
|
firstStorageConfig.GCInterval = 2 * time.Hour
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeFalse)
|
|
|
|
firstStorageConfig.GCInterval = 1 * time.Hour
|
|
|
|
So(firstStorageConfig.ParamsEqual(secondStorageConfig), ShouldBeTrue)
|
|
|
|
isSame, err := config.SameFile("test-config", "test")
|
|
So(err, ShouldNotBeNil)
|
|
So(isSame, ShouldBeFalse)
|
|
|
|
dir1 := t.TempDir()
|
|
|
|
isSame, err = config.SameFile(dir1, "test")
|
|
So(err, ShouldNotBeNil)
|
|
So(isSame, ShouldBeFalse)
|
|
|
|
dir2 := t.TempDir()
|
|
|
|
isSame, err = config.SameFile(dir1, dir2)
|
|
So(err, ShouldBeNil)
|
|
So(isSame, ShouldBeFalse)
|
|
|
|
isSame, err = config.SameFile(dir1, dir1)
|
|
So(err, ShouldBeNil)
|
|
So(isSame, ShouldBeTrue)
|
|
})
|
|
|
|
Convey("Test DeepCopy() & Sanitize()", t, func() {
|
|
conf := config.New()
|
|
So(conf, ShouldNotBeNil)
|
|
|
|
authConfig := &config.AuthConfig{LDAP: (&config.LDAPConfig{}).SetBindPassword("oina")}
|
|
conf.HTTP.Auth = authConfig
|
|
|
|
So(func() { conf.Sanitize() }, ShouldNotPanic)
|
|
|
|
conf = conf.Sanitize()
|
|
So(conf.HTTP.Auth.LDAP.BindPassword(), ShouldEqual, "******")
|
|
|
|
// negative
|
|
obj := make(chan int)
|
|
err := config.DeepCopy(conf, obj)
|
|
So(err, ShouldNotBeNil)
|
|
err = config.DeepCopy(obj, conf)
|
|
So(err, ShouldNotBeNil)
|
|
})
|
|
|
|
Convey("Test IsRetentionEnabled()", t, func() {
|
|
conf := config.New()
|
|
So(conf.IsRetentionEnabled(), ShouldBeFalse)
|
|
|
|
conf.Storage.Retention.Policies = []config.RetentionPolicy{
|
|
{
|
|
Repositories: []string{"repo"},
|
|
},
|
|
}
|
|
|
|
So(conf.IsRetentionEnabled(), ShouldBeFalse)
|
|
|
|
policies := []config.RetentionPolicy{
|
|
{
|
|
Repositories: []string{"repo"},
|
|
KeepTags: []config.KeepTagsPolicy{
|
|
{
|
|
Patterns: []string{"tag"},
|
|
MostRecentlyPulledCount: 2,
|
|
},
|
|
},
|
|
},
|
|
}
|
|
|
|
conf.Storage.Retention = config.ImageRetention{
|
|
Policies: policies,
|
|
}
|
|
|
|
So(conf.IsRetentionEnabled(), ShouldBeTrue)
|
|
|
|
subPaths := make(map[string]config.StorageConfig)
|
|
|
|
subPaths["/a"] = config.StorageConfig{
|
|
GC: true,
|
|
Retention: config.ImageRetention{
|
|
Policies: policies,
|
|
},
|
|
}
|
|
|
|
conf.Storage.SubPaths = subPaths
|
|
|
|
So(conf.IsRetentionEnabled(), ShouldBeTrue)
|
|
})
|
|
|
|
Convey("Test IsEventRecorderEnabled()", t, func() {
|
|
conf := config.New()
|
|
So(conf.IsEventRecorderEnabled(), ShouldBeFalse)
|
|
|
|
// Enable the event recorder
|
|
enable := true
|
|
conf.Extensions = &extconf.ExtensionConfig{}
|
|
conf.Extensions.Events = &events.Config{
|
|
Enable: &enable,
|
|
}
|
|
|
|
So(conf.IsEventRecorderEnabled(), ShouldBeTrue)
|
|
|
|
// Disabled scenario
|
|
disable := false
|
|
conf.Extensions.Events.Enable = &disable
|
|
So(conf.IsEventRecorderEnabled(), ShouldBeFalse)
|
|
|
|
// nil pointers
|
|
conf.Extensions.Events = nil
|
|
So(conf.IsEventRecorderEnabled(), ShouldBeFalse)
|
|
|
|
conf.Extensions = nil
|
|
So(conf.IsEventRecorderEnabled(), ShouldBeFalse)
|
|
})
|
|
}
|