-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cli server version #1709
Merged
andaaron
merged 1 commit into
project-zot:main
from
laurentiuNiculae:cli-server-version
Nov 22, 2023
Merged
Cli server version #1709
andaaron
merged 1 commit into
project-zot:main
from
laurentiuNiculae:cli-server-version
Nov 22, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
laurentiuNiculae
force-pushed
the
cli-server-version
branch
4 times, most recently
from
August 18, 2023 13:20
8ed525c
to
0e49a3a
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1709 +/- ##
==========================================
- Coverage 91.88% 91.85% -0.04%
==========================================
Files 164 165 +1
Lines 27992 28125 +133
==========================================
+ Hits 25721 25834 +113
- Misses 1676 1692 +16
- Partials 595 599 +4 ☔ View full report in Codecov by Sentry. |
laurentiuNiculae
force-pushed
the
cli-server-version
branch
5 times, most recently
from
August 25, 2023 08:05
3636575
to
4ab529d
Compare
laurentiuNiculae
requested review from
roxanaN,
eusebiu-constantin-petu-dbk,
shimish2,
adodon2go,
rchincha and
andaaron
as code owners
August 25, 2023 10:58
laurentiuNiculae
force-pushed
the
cli-server-version
branch
3 times, most recently
from
August 28, 2023 08:32
f563402
to
ed10fca
Compare
laurentiuNiculae
force-pushed
the
cli-server-version
branch
2 times, most recently
from
September 8, 2023 07:53
e08d137
to
44a6fc4
Compare
laurentiuNiculae
force-pushed
the
cli-server-version
branch
4 times, most recently
from
September 15, 2023 08:50
caa4de8
to
9c29625
Compare
laurentiuNiculae
force-pushed
the
cli-server-version
branch
from
October 25, 2023 14:56
9c29625
to
95d3d33
Compare
laurentiuNiculae
force-pushed
the
cli-server-version
branch
4 times, most recently
from
November 1, 2023 08:35
4441781
to
4d835db
Compare
laurentiuNiculae
force-pushed
the
cli-server-version
branch
from
November 9, 2023 11:12
4d835db
to
3b7b38e
Compare
andaaron
previously approved these changes
Nov 10, 2023
eusebiu-constantin-petu-dbk
previously approved these changes
Nov 10, 2023
laurentiuNiculae
force-pushed
the
cli-server-version
branch
2 times, most recently
from
November 13, 2023 16:27
a4d3094
to
67c7757
Compare
rchincha
reviewed
Nov 13, 2023
laurentiuNiculae
force-pushed
the
cli-server-version
branch
from
November 15, 2023 14:19
67c7757
to
28a50e4
Compare
laurentiuNiculae
dismissed stale reviews from eusebiu-constantin-petu-dbk and andaaron
via
November 17, 2023 12:31
8bc262a
laurentiuNiculae
force-pushed
the
cli-server-version
branch
2 times, most recently
from
November 17, 2023 15:07
8bc262a
to
39ee704
Compare
andaaron
reviewed
Nov 17, 2023
laurentiuNiculae
force-pushed
the
cli-server-version
branch
2 times, most recently
from
November 17, 2023 17:09
f52aef9
to
f9dc59f
Compare
Here are some examples of outputs $ zli-linux-amd64 status --url http://127.0.0.1:5000 --user laur:password
Server Status: online
Server Version: bsd-0.0.4
Dist Spec Version: 1.1.0-dev
Built with: imagetrust, lint, metrics, mgmt, profile, scrub, search, sync, ui, userprefs Bad Credentials: $ zli-linux-amd64 status --url http://127.0.0.1:5000 --user laur:bad-pass
Server Status: unknown
Error: unauthorised access, given credentials are invalid No Credentials: $ zli-linux-amd64 status --url http://127.0.0.1:5000
Server Status: unknown
Error: unauthorised access, endpoint requires valid user credentials (add the flag '--user [user]:[password]') MGMT is not enabled (minimal version): $ zli-linux-amd64 status --url http://127.0.0.1:5000
Server Status: online
Error: /v2/_zot/ext/mgmt endpoint is not available Server is offline: $ zli-linux-amd64 status --url http://127.0.0.1:5000
Server Status: offline
Error: Get "http://127.0.0.1:8080/v2/": dial tcp 127.0.0.1:8080: connect: connection refused```
Given server isn't a OCI Registry: Server is not Zot: $ zli-linux-amd64 status --url http://127.0.0.1:8000
Server Status: offline
Error: registry at the given address doesn't implement the correct API: request at http://127.0.0.1:8000/v2/ failed |
laurentiuNiculae
force-pushed
the
cli-server-version
branch
5 times, most recently
from
November 20, 2023 14:46
d4d35bc
to
c2fb0cf
Compare
…ails Signed-off-by: Laurentiu Niculae <[email protected]>
laurentiuNiculae
force-pushed
the
cli-server-version
branch
from
November 22, 2023 14:46
c2fb0cf
to
8569fdb
Compare
andaaron
approved these changes
Nov 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What type of PR is this?
Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:
Automation added to e2e:
Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.