This API is proven end-to-end in https://github.com/containers/image/pull/2876 and
https://github.com/containers/skopeo/pull/2645 , but it is not yet convenient to
use becahse the Rust dependency has to be compiled manually.
So, for now, add the API as a stub only; that allows building the CLIs
and tests on top, and they will light up once the backend is added.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
... to simplify, and to allow using recent packages.
We were using Debian because previously we were using Ubuntu,
and that was because Ubuntu was the native distribution of
Travis even before we were running CI in any kind of container;
so there seems to have been no specific intent to use a non-Fedora OS,
at least originally.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
We typically only use it for a single operation, so we
would leak connections + goroutines.
Matches Rekor 1.1.0.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Setup a local rekor server using podman to test against. With that we
can verify that our custom client code works against a real server.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This removes 4.728 MB from a macOS Skopeo binary and similar saving can
be seen in Podman and Buildah.
The costs are:
- A few dozen lines of new code
- A major loss of flexibility. All the removed layers had many options,
this code hard-codes the choices we were making. Reintroducing the
options could be pretty costly in extra code.
- We lose the ability to debug Rekor API accesses via environment
variables (something I had no idea exists; so there’s that.)
- The previous code was able to decode responses in many formats, e.g.
YAML, based on the Content-Type: header in the response. This one
asks for JSON and expects JSON.
A proper test against a real rekor server is added in the next commit.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
It was added for debian so they could build without the deps but this is
no longer needed. Get rid of it to simplify the codebase.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
It was added for debian so they could build without the deps but this is
no longer needed, the tag is broken and fails to compile so just get rid
of it to simplify the codebase.
Fixes: #2877
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
For CI logs it is extremely useful to get the full output to see what
tests were run and skipped so one can verifiy CI is working as expected
and runs all out tests.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This is a quick and dirty fix but should get the job done and ensures
the tests are actually run by CI. The idea is simple to not waste
resources we just look for the test names using
ensureTestCanCreateImages() to skip as rootless.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
For CI it would be useful if we can pass extra arguments to go test such
as -v or extra filters via -run.
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Miloslav Trmač