-
Notifications
You must be signed in to change notification settings - Fork 450
Allow plugin download to use an insecure repository #866
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
Allow plugin download to use an insecure repository #866
Conversation
|
hi @adambkaplan , can you PTAL ? |
|
@akram I assume this is to address https://jira.coreos.com/browse/DEVEXP-367 and https://bugzilla.redhat.com/show_bug.cgi?id=1693533 ? |
|
@akram @adambkaplan I'm taking a look |
gabemontero
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@akram since this is a RH subscription customer support case calling for this we should expose this tuning option:
- update the README's env var section to explain it
- update the templates under https://github.com/openshift/jenkins/tree/master/openshift/templates
- update the docs under https://github.com/openshift/openshift-docs ... as those are still in a bit of flux, reach out to @bmcelvee to nail down the right branch/location
|
Also, to help @akram on the e2e-aws-jenkins failure, I took a peek for him. It failed in the client plugin test becase of an apparent expired certificate. From https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/openshift_jenkins/866/pull-ci-openshift-jenkins-master-e2e-aws-jenkins/288: On the surface that feels like a api.ci / openshift CICD flake. I saw similar ones in unrelated networking e2es in the e2e-aws run:
|
|
/retest |
Hi @adambkaplan |
Hi @gabemontero , thank you for reviewing. |
gabemontero
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a couple of additional req's @akram while waiting on the template update
34d24b9 to
b2a99df
Compare
akram
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like my editor did some clean up of whitespaces that I didn't notice when doing my git diff.
Documentation update is in this branch: I contacted @bmcelvee to know which branch to rely on to create the PR. I am waiting for her reply. |
|
PRs for documentation: |
Adds JENKINS_UC_INSECURE documentation in README.md Adds JENKINS_UC_INSECURE param and env to the openshift templates
5b121e4 to
d7d4f47
Compare
gabemontero
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes on the new README hits
|
/approve |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: akram, gabemontero The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
No description provided.