Thanks @jkroepke
Doing a test CI run so we understand scope of changes required.

Thanks @jkroepke Doing a test CI run so we understand scope of changes required.

I know, currently everything is broke and I have also compile error locally. Thats why the PR is in draft and not ready to review. However, I would like to share the info that someone is working on it.

The only exception here is

https://github.com/project-zot/zot/actions/runs/9847797795/job/27233306056?pr=2532

11% binary increase size. Thats nothing, what I can do here.

Doing a test CI run so we understand scope of changes required.

could you please do it again?

And I have no access to the license jobs.

What is the error? And what is the solution?

@jkroepke One more rebase pls.
The license issue is likely resolved.

@rchincha done

@rchincha Could you please trigger the CI again?

No idea about snyk

I need assistance here.

• https://github.com/project-zot/zot/actions/runs/9993299036/job/27620342470?pr=2532

Check binary size. I can do nothing here.

• https://github.com/project-zot/zot/actions/runs/9993299070/job/27620342130?pr=2532

It checks for warnings, but there is an warning which can be considered as expected:

{"level":"warn","config version":"1.1.0","supported version":"1.1.0+dev","goroutine":1,"caller":"zotregistry.dev/zot/pkg/cli/server/root.go:745","time":"2024-07-18T16:48:06.654323+02:00","message":"config dist-spec version differs from version actually used"}

The distribution project now using a way more modern S3 library from AWS which has some incompatibilities with minio.

Codecov Report

Attention: Patch coverage is 84.61538% with 4 lines in your changes missing coverage. Please review.

Project coverage is 92.72%. Comparing base (7729fef) to head (a2426db).

@@           Coverage Diff           @@
##             main    #2532   +/-   ##
=======================================
  Coverage   92.72%   92.72%           
=======================================
  Files         169      169           
  Lines       22463    22471    +8     
=======================================
+ Hits        20828    20836    +8     
  Misses       1018     1018           
  Partials      617      617           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rchincha I created 2 additional PR #2555 and #2556 which are dependencies for this PR.

In general, CI is looking fine, with both PRs are merged.

• Verify Example Config: No idea whats going one there.
• snyk is always red
• binary size: Reports an 11% bigger binary.

Otherwise, you can do an intermediate review, if you agree with the code changes.

this feels ready to review

@jkroepke can you rebase pls?

@jkroepke can you rebase pls?

To lax merge conflict, I cut the reformat of the go.mod into a dedicated PR: #2582

I merged the other PR. Opening a separate PR was a very good idea. I was about to suggest it.

@jkroepke the go code changes look ok, but there are CI failures.

The failure in Check binary size increase I think we can live with.
The Verify Example Config job is failing because the version in the example config files doesn't match the version in the imported library.
The License Compliance failures I reviewed and identified false positives which I marked as such in FOSSA.
The snyk results I need to review at some point
The GC tests I restarted (we need to find a way to stabilize those).
@jkroepke please look at the failures in the Running tests jobs.

everything is green now.

PR binary size: 39018817 Bytes
main branch binary size: 34980161 Bytes
zot minimal binary increased by 11.54% comparing with main