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

Add nightly Pipeline + try to fix some upgrade test stuff #69

Merged
merged 2 commits into from
Sep 5, 2019

Conversation

bobcatfish
Copy link
Contributor

Changes

Unfortunately this PR is a mix of 2 different changes b/c the changes all had to go into the same file and I had to fix them at the same time to apply them. They are in 2 separate commits tho!

Change 1: Nightly release pipeline

Add nightly Pipelines release crescent_moon

This is the configuration to run the Pipeline + Tasks being added in
tektoncd/pipeline#1274 every night at midnight
PDT.

I've successfully run this by setting the cron to every 5 minutes. This
configuration is already applied to the repo.

Status can be seen at https://prow.tekton.dev/?type=periodic.

Part of tektoncd/pipeline#860

Change 2: Trying to get upgrade test running (WIP)

Trying to get upgrade test working construction

It looked like the indentation was off, and also some of the environment
variables (e.g. $PULL_REF) provided to presubmit jobs aren't provided
to presubmit jobs. I tried to copy example configuration in the
kubernetes test-infra repo but the prow "initupload" container is still
failing with an error about cloning refs.

(continuation of #56)

Needs some more work tho @houshengbo @vdemeester

I think the only significant change in the PR is github.com/$(REPO_OWNER)/$(REPO_NAME)=$(PULL_REFS)" being hardcoded now. I also changed some indentation to be consistent but I'm not sure that mattered.

It still isn't working (https://prow.tekton.dev/?type=periodic).

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.

This is the configuration to run the Pipeline + Tasks being added in
tektoncd/pipeline#1274 every night at midnight
PDT.

I've successfully run this by setting the cron to every 5 minutes. This
configuration is already applied to the repo.

Status can be seen at https://prow.tekton.dev/?type=periodic.

Part of tektoncd/pipeline#860
It looked like the indentation was off, and also some of the environment
variables (e.g. $PULL_REF) provided to presubmit jobs aren't provided
to presubmit jobs. I tried to copy example configuration in the
kubernetes test-infra repo but the prow "initupload" container is still
failing with an error about cloning refs.

(continuation of tektoncd#56)
@tekton-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bobcatfish

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 added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Sep 4, 2019
Copy link
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

/lgtm
@bobcatfish should be deploy to test it ?

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 5, 2019
@tekton-robot tekton-robot merged commit 0eb8f8b into tektoncd:master Sep 5, 2019
@vdemeester
Copy link
Member

lol I forgot it was already approved, Oops =)

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. lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants