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

switch to generated password for jenkins service #10163

Merged
merged 1 commit into from Aug 3, 2016

Conversation

bparees
Copy link
Contributor

@bparees bparees commented Aug 2, 2016

No description provided.

@bparees
Copy link
Contributor Author

bparees commented Aug 2, 2016

@spadgett @jwforres fyi. I don't think this has a direct impact on the console except users won't know how to login anymore when you end them to the console....

@csrwng ptal

@bparees bparees force-pushed the jenkins_password branch 4 times, most recently from 72acf08 to ed43f4d Compare August 2, 2016 15:18
@jwforres
Copy link
Member

jwforres commented Aug 2, 2016

On Tue, Aug 2, 2016 at 11:11 AM, Ben Parees notifications@github.com
wrote:

@spadgett https://github.com/spadgett @jwforres
https://github.com/jwforres fyi. I don't think this has a direct impact
on the console except users won't know how to login anymore when you end
them to the console....

And what are we going to do about that...

@csrwng https://github.com/csrwng ptal


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

@@ -68,8 +68,10 @@ Steps
6. Create the sample application configuration

$ oc new-app application-template.json

Note the generated password (JENKINS_PASSWORD) reported in the new-app output, you will need it to login to Jenkins.
Copy link
Contributor

Choose a reason for hiding this comment

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

Should you include directions on how to get it if creating from the web console?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will update.

@csrwng
Copy link
Contributor

csrwng commented Aug 2, 2016

Not related to this pull, but I instantiated Jenkins from the template and I wanted to delete it ... I've gotten used to just doing 'oc delete all -l app=blah' to delete anything, but I didn't see a common app label in the stuff created by the template

@bparees
Copy link
Contributor Author

bparees commented Aug 2, 2016

And what are we going to do about that...

I have no idea :/

@bparees
Copy link
Contributor Author

bparees commented Aug 2, 2016

@csrwng it's got a common label of "template=jenkins-ephemeral-template", you don't see that?

however i'm going to change it to jenkins-pipelinetemplate.

}
],
"labels": {
"template": "jenkins-ephemeral-template"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

ahh, yes, sorry I was describing the dc and looking at the wrong field. Works for me.

"creationTimestamp": null
},
"spec": {
"host": "www.example.com",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@csrwng
Copy link
Contributor

csrwng commented Aug 2, 2016

LGTM

@bparees
Copy link
Contributor Author

bparees commented Aug 3, 2016

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 3, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7480/) (Image: devenv-rhel7_4747)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to a23e6c8

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to a23e6c8

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7447/)

@openshift-bot openshift-bot merged commit 3da4944 into openshift:master Aug 3, 2016
@bparees bparees deleted the jenkins_password branch August 4, 2016 17:36
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.

None yet

4 participants