Skip to content
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

Allow usage of the *-oci-ta Task variants #26

Merged
merged 1 commit into from
May 20, 2024

Conversation

lcarva
Copy link
Collaborator

@lcarva lcarva commented May 17, 2024

As part of EC-20, we are adding Task variants that support Trusted Artifacts with OCI storage. These have the suffix of -oci-ta. These Tasks function similarly to their counter-parts. The main difference is that they share data between them via resources in an OCI registry instead of file in a shared PVC on-cluster.

This commit allows the -oci-ta Tasks to fullfill the required-tasks check.

Additionally, the summary Task is no longer marked as required. This was the outcome after some investigation on its actual value. More context is available on the comments of EC-643.

Finally, older entries that are no longer relevant have been removed from the list.

Ref: EC-644

As part of EC-20, we are adding Task variants that support Trusted
Artifacts with OCI storage. These have the suffix of `-oci-ta`. These
Tasks function similarly to their counter-parts. The main difference is
that they share data between them via resources in an OCI registry
instead of file in a shared PVC on-cluster.

This commit allows the `-oci-ta` Tasks to fullfill the required-tasks
check.

Additionally, the `summary` Task is no longer marked as required. This
was the outcome after some investigation on its actual value. More
context is available on the comments of EC-643.

Finally, older entries that are no longer relevant have been removed
from the list.

Ref: EC-644

Signed-off-by: Luiz Carvalho <[email protected]>
@lcarva
Copy link
Collaborator Author

lcarva commented May 17, 2024

@rhartman93, could you give a look here as well? I would like to have this merged soon as it's currently blocking the Trusted Artifacts work. 🙏

@rhartman93 rhartman93 merged commit 2b7eb0b into release-engineering:main May 20, 2024
1 check passed
@lcarva lcarva deleted the EC-644 branch May 20, 2024 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants