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 scripts to release from travis-ci #1582

Merged
merged 2 commits into from
Oct 26, 2017
Merged

Conversation

aledbf
Copy link
Member

@aledbf aledbf commented Oct 24, 2017

What this PR does / why we need it:

We need this PR to automate the release of images located in this repository using a third party service (travis-ci in this case)

To release an image we just require a personal travis-ci token with access to kubernetes/ingress-nginx and the name of the component to be released:

export TRAVIS_CI_TOKEN=xxxxxxx
export COMPONENT=nginx-slim

curl -sSL https://raw.githubusercontent.com/kubernetes/ingress-nginx/master/.travis/publish-from-travis.sh | bash

Edit: the change of registries can be reverted. This is just to show the release procedure

@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Oct 24, 2017
@nicksardo
Copy link
Contributor

/assign @bowei @ixdy

@coveralls
Copy link

Coverage Status

Coverage remained the same at 33.49% when pulling 47ab12a31c171acd30ce1492b02782431383b3aa on aledbf:publish-scripts into d965ee8 on kubernetes:master.

@aledbf aledbf changed the title Add scripts to release from travis-ci WIP: Add scripts to release from travis-ci Oct 24, 2017
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 24, 2017
@bowei
Copy link
Member

bowei commented Oct 24, 2017

@aledbf -- it would be great if we could move releasing of ingress-nginx off of gcr.io/google_containers to a k8s contributor controlled repo. That way releases would not be a manual process. @thockin

@aledbf
Copy link
Member Author

aledbf commented Oct 24, 2017

it would be great if we could move releasing of ingress-nginx off of gcr.io/google_containers to a k8s contributor controlled repo. That way releases would not be a manual process

No problem with that. Just tell me what registry I need to use.

@aledbf
Copy link
Member Author

aledbf commented Oct 24, 2017

@bowei besides that, it makes the change to publish the images?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 33.452% when pulling 42dc4108e29389b13cf6c86aee25310961d3745a on aledbf:publish-scripts into d965ee8 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 33.389% when pulling 7f85e5cc46295a6c2099ecaa87639f2fd08d80a1 on aledbf:publish-scripts into f8c2fb9 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.0e-05%) to 33.389% when pulling db04359 on aledbf:publish-scripts into f8c2fb9 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 33.352% when pulling febd7cf on aledbf:publish-scripts into cd37f96 on kubernetes:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 33.389% when pulling febd7cf on aledbf:publish-scripts into cd37f96 on kubernetes:master.

@aledbf aledbf changed the title WIP: Add scripts to release from travis-ci Add scripts to release from travis-ci Oct 26, 2017
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 26, 2017
@aledbf aledbf merged commit 65a7f07 into kubernetes:master Oct 26, 2017
@aledbf aledbf deleted the publish-scripts branch October 26, 2017 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. 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.

7 participants