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

re-enable jenkins autoprovisioning #11065

Merged
merged 1 commit into from
Oct 24, 2016

Conversation

bparees
Copy link
Contributor

@bparees bparees commented Sep 23, 2016

No description provided.

@bparees bparees changed the title re-enable jenkins autoprovisioning [DO_NOT_MERGE] re-enable jenkins autoprovisioning Sep 23, 2016
@bparees
Copy link
Contributor Author

bparees commented Sep 23, 2016

@smarterclayton do we really want to do this before oauth is enabled? it means we need to update a bunch of doc(this tutorial, official docs) telling people how to find out their jenkins password when the jenkins server gets provisioned for them.

@gabemontero

@bparees
Copy link
Contributor Author

bparees commented Sep 26, 2016

@smarterclayton bump

@smarterclayton
Copy link
Contributor

I guess not - since we're getting close to oauth and routes.

On Mon, Sep 26, 2016 at 3:39 PM, Ben Parees notifications@github.com
wrote:

@smarterclayton https://github.com/smarterclayton bump


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11065 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABG_p8UaAQIxfsogtjt1_uJbGTdSHju5ks5quB9WgaJpZM4KEt5I
.

@bparees
Copy link
Contributor Author

bparees commented Sep 26, 2016

I will hold this until the oauth stuff goes in and then revisit this PR.

@bparees bparees assigned bparees and gabemontero and unassigned bparees Oct 11, 2016
@bparees bparees changed the title [DO_NOT_MERGE] re-enable jenkins autoprovisioning re-enable jenkins autoprovisioning Oct 22, 2016
@bparees
Copy link
Contributor Author

bparees commented Oct 22, 2016

oauth has landed in the jenkins centos image and should be in the rhel image we publish for 3.4.
[merge]

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@bparees bparees force-pushed the enable_autoprovision branch 2 times, most recently from 354b6c8 to f21c7b7 Compare October 23, 2016 06:47
@bparees
Copy link
Contributor Author

bparees commented Oct 23, 2016

swagger regen'd
[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 1cbf66e

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10501/) (Base Commit: 2047a73)

@bparees
Copy link
Contributor Author

bparees commented Oct 23, 2016

flake #11114
[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 1cbf66e

@openshift-bot
Copy link
Contributor

openshift-bot commented Oct 24, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10501/) (Base Commit: 35b539d) (Image: devenv-rhel7_5234)

@openshift-bot openshift-bot merged commit 93209d9 into openshift:master Oct 24, 2016
@bparees bparees deleted the enable_autoprovision branch October 24, 2016 18:12
@smarterclayton
Copy link
Contributor

Update swagger

On Oct 22, 2016, at 12:45 AM, OpenShift Bot notifications@github.com
wrote:

continuous-integration/openshift-jenkins/test FAILURE (
https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10458/) (Base
Commit: b5156c2
b5156c2
)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11065 (comment),
or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABG_p379UM-vi3SBwwV1tSh0IFYehd_xks5q2ZTogaJpZM4KEt5I
.

@bparees
Copy link
Contributor Author

bparees commented Oct 25, 2016

@smarterclayton did you mean to comment on this PR? it's already merged.

@bparees
Copy link
Contributor Author

bparees commented Oct 25, 2016

(and yes, swagger was updated as part of it)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants