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

Fix release pipeline to handle #1122 #1327

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

afrittoli
Copy link
Member

Since #1122, we do not treat outputs that were inputs as well in any
special way, meaning that the output folder will be empty unless we
copy the input to the output.

Fix the release pipeline to handle that.

Fixes: #1325

Changes

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

See the contribution guide for more details.

Double check this list of stuff that's easy to miss:

Reviewer Notes

If API changes
are included, additive changes
must be approved by at least two OWNERS
and backwards incompatible changes
must be approved by more than 50% of the OWNERS,
and they must first be added
in a backwards compatible way.

Release Notes

Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

Since tektoncd#1122, we do not treat outputs that were inputs as well in any
special way, meaning that the output folder will be empty unless we
copy the input to the output.

Fix the release pipeline to handle that.

Fixes: tektoncd#1325
@googlebot googlebot added the cla: yes Trying to make the CLA bot happy with ppl from different companies work on one commit label Sep 17, 2019
@tekton-robot tekton-robot requested review from dibyom and a user September 17, 2019 22:41
@tekton-robot tekton-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 17, 2019
Copy link
Member

@dibyom dibyom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 18, 2019
@tekton-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: afrittoli, dibyom

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot merged commit 05c4164 into tektoncd:master Sep 18, 2019
@bobcatfish
Copy link
Collaborator

Just a quick note that the release pipeline (afaik - maybe this is changed now?) should be running against v0.3.1 of Tekton (https://github.com/tektoncd/pipeline/tree/master/tekton#install-tekton) - tho it's good to be ready for this change later!

@dibyom
Copy link
Member

dibyom commented Sep 18, 2019

Do the nightly and release jobs run on the same cluster?

@afrittoli
Copy link
Member Author

Do the nightly and release jobs run on the same cluster?

I'm not sure if they do, but at least I could verify that Tekton in the prow cluster in tekton-releases runs on version 0.3.1. Which means I should not update the release pipeline before running the release.

We could perhaps have a parameter added to the publish Task to conditionally handle the change introduced #1122, so that the pipeline may work both on pre and post v0.7.0.

BTW the only way I cound find out the version of Tekton was to compare the image sha from the controller pod in the cluster with the sha from the 0.3.1 release file on github. Is there any better way to find out about the version of Tekton that runs in the cluster?

@dibyom
Copy link
Member

dibyom commented Sep 19, 2019

+1 I like the parameter idea. Though it would have been cool to have a conditional that checks the version of Tekton and then runs the right task :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes Trying to make the CLA bot happy with ppl from different companies work on one commit lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The release pipeline does not store artifacts on the GCS bucket
5 participants