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

Remote Resolution of Tasks #97

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Remote Resolution of Tasks #97

merged 1 commit into from
Jan 10, 2024

Conversation

jerop
Copy link
Contributor

@jerop jerop commented Jan 10, 2024

Prior to this change, the project used Tasks applied directly to the cluster. In this change, we switch to remote resolution of Tasks in a Pipeline. This is aligned with the "config as code" recommendation, where build definitions are fetched from a version control system.

In addition, we make a few minor changes:

  • cleanup the documentation
  • remove unused result from the upload-model task

cc @mihaimaruseac @laurentsimon

Prior to this change, the project used Tasks applied directly to
the cluster. In this change, we switch to [remote resolution] of
Tasks in a Pipeline. This is aligned with the  "config as code"
recommendation, where build definitions are fetched from a version
control system.

In addition, we make a few minor changes:
- cleanup the documentation
- remove unused result from the upload-model task

[remote resolution]: https://tekton.dev/docs/pipelines/resolution/
@laurentsimon
Copy link
Collaborator

@mihaimaruseac can we merge with osx tests failing? I think so, but want to be sure

@mihaimaruseac
Copy link
Collaborator

I'm seeing a similar type of failure on Windows, I think it comes from sigstore bump. Let's merge and fix later.

@mihaimaruseac mihaimaruseac merged commit 052f29b into sigstore:main Jan 10, 2024
12 of 13 checks passed
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