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

move security repo to jenkins temporarily #5966

Closed

Conversation

BenTheElder
Copy link
Member

we need to sort out some more things before this can be on prow :(

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BenTheElder

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

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-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 Dec 14, 2017
@BenTheElder
Copy link
Member Author

BenTheElder commented Dec 14, 2017

/cc @cjwagner @tallclair
I'm only moving some jobs as a proof of concept we can canary test for a short term option. Many jobs that exist now never existed on Jenkins to begin with.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 14, 2017
@BenTheElder
Copy link
Member Author

@cjwagner and I have been discussing this.

The first fix necessary for Prow was made here:

The two remaining things Prow needs are:

@BenTheElder
Copy link
Member Author

See also: #5804 which will ensure that if they are moved to prow they are configured properly and point logs and artifacts to a private bucket. (already merged)

@tallclair
Copy link
Member

lgtm. Thanks!

@BenTheElder
Copy link
Member Author

/hold
it seems we're doing pretty well on the track to move things to prow safely, discussed with @cjwagner again today

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 21, 2017
@BenTheElder BenTheElder deleted the such-irony-much-sadness branch January 20, 2018 01:17
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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. 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