Skip to content
This repository was archived by the owner on Sep 5, 2019. It is now read-only.

Use release.sh helper from prow-tests image#270

Merged
google-prow-robot merged 1 commit intoknative:masterfrom
adrcunha:better-release
Jul 22, 2018
Merged

Use release.sh helper from prow-tests image#270
google-prow-robot merged 1 commit intoknative:masterfrom
adrcunha:better-release

Conversation

@adrcunha
Copy link
Copy Markdown
Contributor

We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

release.sh helper is implemented by knative/test-infra#14

Bonus! Now build-crd release script has the following features, just like serving and eventing:

  • setting custom GCR/GCS buckets through environment variables
  • release tagging
  • test skipping
  • local build

Fixes #267

We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`release.sh` helper is implemented by knative/test-infra#14

**Bonus!** Now build-crd release script has the following features, just like serving and eventing:
* setting custom GCR/GCS buckets through environment variables
* release tagging
* test skipping
* local build

# Set default GCS/GCR
: ${BUILD_RELEASE_GCS:="build-crd"}
: ${BUILD_RELEASE_GCR:="gcr.io/build-crd"}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we move these to where serving and eventing put their releases? Which I believe is knative-releases?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We can and I'm happy to do that, but last time I talked to @mattmoor about this, he wanted to keep them in separate GCRs. Matt?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

knative-releases seems fine. This was just trickier when it was elafros.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done in #276

@jessiezcc
Copy link
Copy Markdown

/lgtm

@mattmoor
Copy link
Copy Markdown
Member

/approve

@google-prow-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adrcunha, mattmoor

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

The pull request process is described here

Details 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

@google-prow-robot google-prow-robot merged commit 3379241 into knative:master Jul 22, 2018
@adrcunha adrcunha deleted the better-release branch July 23, 2018 05:33
vdemeester pushed a commit to vdemeester/knative-build that referenced this pull request Apr 3, 2019
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`release.sh` helper is implemented by knative/test-infra#14

**Bonus!** Now build-crd release script has the following features, just like serving and eventing:
* setting custom GCR/GCS buckets through environment variables
* release tagging
* test skipping
* local build
vdemeester pushed a commit to vdemeester/knative-build that referenced this pull request Apr 3, 2019
We're consolidating the test infrastructure into a single place, so all repos get the same fixes, updates and new features.

`release.sh` helper is implemented by knative/test-infra#14

**Bonus!** Now build-crd release script has the following features, just like serving and eventing:
* setting custom GCR/GCS buckets through environment variables
* release tagging
* test skipping
* local build
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize release.sh with other knative repos' release processes

5 participants