Transfer #483
Merged
Transfer #483
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
mandelsoft
force-pushed
the
transfer
branch
5 times, most recently
from
8f2d9d9 to
adbffb3
Compare
mandelsoft
force-pushed
the
transfer
branch
6 times, most recently
from
cba8c9d to
81fde89
Compare
mandelsoft
force-pushed
the
transfer
branch
8 times, most recently
from
4a346c4 to
59f07c5
Compare
mandelsoft
force-pushed
the
transfer
branch
2 times, most recently
from
0a76330 to
5594a70
Compare
mandelsoft
commented
Sep 19, 2023
mandelsoft
force-pushed
the
transfer
branch
from
9437f17 to
a6d11bc
Compare
|
Hi @fabianburth , changes from the past days pulled down an rebased to main. |
fabianburth
approved these changes
Sep 20, 2023
robertwol
pushed a commit
that referenced
this pull request
Sep 25, 2023
* simplify ocm repo implementation interface * equal tests * fixes + transfer tests * fix transfer tests * uploader tests * accmeth, digester, blobhandler tests * add resources tests * download tests + improved gomega matchers * hash test * command tests * builder with default modopts * signing tests * signing tests for legacy/regular flavors * transfer/signing combi tests * more combi and equivalent tests * first step towards label value merging * label merging * plugin support for merge handlers * plugin support for assignments * comparch modification test * generalize composition environment * HOME in file names + global key options * fix transport messages * fix modified state in value merge handlers * fix cmd get cv * fix equivalence * fix none normalization + align equivalence and normalization * fix transfer logic * rework resource adding * generate CLI docs
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 this PR does / why we need it:
Adding resources to a Component Version now already adds digests by default.
This enables delta transports of volatile data in the Component Descriptor.
This volatile data can now be merged: Local (volatile) changes in the target repository
of a transfer are merges with the new inbound version.
This is mainly used to merge non-signature-relevant label values. Therefore,
value merge algorithms are introduced, which can be declared in the component descriptor
or pre configured in the code. The standard library provides a set of basic algorithms
with appropriate configuration options, which can be extended by plugins.
This fatur can, for example , be used for implementing routing slips on top of the
Open Component Model in form of dedicated labels. The delta transport feature is then used
to re-import/transport new routing slip entries from the software provider.
To be compatible, the handling of resources without a digest is supported, also,but with
limited functionality.
A possible (not yet implemented) extension would be to introduce a new transfer option
to enable on-the-fly calculation of missing digests.
This will always enable to skip the value transfer of resource versions already available at the target,
if the overwrite option is chosen.
Which issue(s) this PR fixes:
Fixes #
Relates to #469
Special notes for your reviewer:
Release Notes: