* fix(monitoring): fix race condition in TestPopulateStorageMetrics Write images to filesystem before starting controller to prevent GC from reading index.json while it's being written. See failure: https://github.com/project-zot/zot/actions/runs/19633819097/job/56219606041 Signed-off-by: Andrei Aaron <andreifdaaron@gmail.com> * fix(sync): fix TestSignatures assertion on referrers order Search through all referrers to find the expected OCI ref digest instead of assuming it's at index.Manifests[1], as the order is not guaranteed. See https://github.com/project-zot/zot/actions/runs/19630382617/job/56208492397?pr=3579 Signed-off-by: Andrei Aaron <andreifdaaron@gmail.com> * fix(redis): fix error handling in GetMultipleRepoMeta The function was returning err at the end, which could be nil even when HGetAll should have failed, because err gets reassigned in the loop. Changed to return nil on success path, ensuring HGetAll errors are properly propagated. Fixes TestRedisUnreachable failure. See https://github.com/project-zot/zot/actions/runs/19634990856/job/56223414955?pr=3582 Signed-off-by: Andrei Aaron <andreifdaaron@gmail.com> * refactor(sync): simplify sync test structure and restore file permissions Remove unnecessary nested Convey blocks and integrate error tests into parent blocks. Add defer statements to restore file permissions after chmod operations to prevent side effects on subsequent tests. Fix potential root cause of https://github.com/project-zot/zot/actions/runs/19636284485/job/56227682319?pr=3579 Signed-off-by: Andrei Aaron <andreifdaaron@gmail.com> --------- Signed-off-by: Andrei Aaron <andreifdaaron@gmail.com>
Adding new extensions
As new requirements come and build time extensions need to be added, there are a few things that you have to make sure are present before commiting :
- files that should be included in the binary only with a specific extension must contain the following syntax at the beginning of the file :
//go:build sync will be added automatically by the linter, so only the second line is mandatory .
NOTE: the third line in the example should be blank, otherwise the build tag would be just another comment.
//go:build sync
// +build sync
package extensions
...................
-
when adding a new tag, specify the new order in which multiple tags should be used (bottom of this page)
-
for each and every new file that contains functions (functionalities) specific to an extension, one should create a corresponding file that must contain the exact same functions, but no functionalities included. This file must begin with an "anti-tag" (e.g. // +build !sync) which will include this file in binaries that don't include this extension ( in this example, the file won't be used in binaries that include sync extension ). See extension-sync-disabled.go for an example.
-
each extension is responsible with implementing authorization for newly added HTTP endpoints. zot will provide the necessary data, including user permissions, to the extension, but actual enforcement of these permissions is the responsibility of each extension. Each extension http.Handler has access to a context previously populated by BaseAuthzHandler with relevant user info. That info has the following structure:
type AccessControlContext struct { // read method action ReadGlobPatterns map[string]bool // detectManifestCollision behaviour action DmcGlobPatterns map[string]bool IsAdmin bool Username string Groups []string }This data can then be accessed from the request context so that every extension can apply its own authorization logic, if needed .
-
when a new extension comes out, the developer should also write some blackbox tests, where a binary that contains the new extension should be tested in a real usage scenario. See test/blackbox folder for multiple extensions examples.
-
newly added blackbox tests should have targets in Makefile. You should also add them as Github Workflows, in .github/workflows/ecosystem-tools.yaml
-
with every new extension, you should modify the EXTENSIONS variable in Makefile by adding the new extension. The EXTENSIONS variable represents all extensions and is used in Make targets that require them all (e.g make test).
-
the available extensions that can be used at the moment are: sync, search, scrub, metrics, lint, ui, mgmt, userprefs, imagetrust . NOTE: When multiple extensions are used, they should be listed in the above presented order.