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

Issue 1. Fix prepending of erroneous /api/v1beta1 to test-connection HT... #1

Closed
wants to merge 2 commits into from
Closed

Conversation

ae6rt
Copy link
Contributor

@ae6rt ae6rt commented Mar 16, 2015

...TP GET

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

carlossg added a commit that referenced this pull request Apr 15, 2015
@carlossg
Copy link
Contributor

I have committed the doc and the fix. Abut the new tests, I'm not sure starting a new stub server is a good idea, I think there should be only 2 types of tests:

  • mock/stub classes, no http testing
  • starting a kubernetes master server for testing (if possible)

@carlossg
Copy link
Contributor

Thanks btw. I'm trying to cut a first release, but lacking permissions, hopefully soon will get that deployed to the update center

@ae6rt
Copy link
Contributor Author

ae6rt commented Apr 15, 2015

No problem on the embedded http server.

Also, you can probably cherry-pick this commit that gives the jelly templates help icons that describe the fields the plugin has control of:

https://github.com/xoom/kubernetes-plugin/commit/18f4a7df351074a7ef9621d763924781b7f6074d

Feel free to decline this PR now that it's work is done. The URL fix was sufficiently trivial that nothing will really be lost with the discarding of the embedded http server test. In retrospect, I should have done this work on a different branch than my fork's master. Live and learn.

@carlossg
Copy link
Contributor

Can you create a new PR for 18f4a7d so it gets tracked? Easier for me to merge too

@ae6rt
Copy link
Contributor Author

ae6rt commented Apr 15, 2015

Done.

On Wed, Apr 15, 2015 at 8:44 AM, Carlos Sanchez notifications@github.com
wrote:

Can you create a new PR for 18f4a7d
18f4a7d
so it gets tracked? Easier for me to merge too


Reply to this email directly or view it on GitHub
#1 (comment)
.

Mark

@carlossg carlossg closed this Apr 15, 2015
carlossg pushed a commit that referenced this pull request Nov 8, 2016
carlossg pushed a commit that referenced this pull request Jul 31, 2018
escoem pushed a commit to escoem/kubernetes-plugin that referenced this pull request Nov 22, 2018
jglick added a commit to jglick/kubernetes-plugin that referenced this pull request Nov 22, 2019
java.lang.AssertionError: Jenkins is still doing something after 1000ms: queue=[] building=[run Dir Context jenkinsci#1]
	at org.jvnet.hudson.test.JenkinsRule.waitUntilNoActivityUpTo(JenkinsRule.java:1877)
	at org.csanchez.jenkins.plugins.kubernetes.pipeline.KubernetesPipelineTest.allDead(KubernetesPipelineTest.java:96)
	at …
"Running CpsFlowExecution[Owner[run Dir Context/1:run Dir Context jenkinsci#1]]" Id=4356 Group=main RUNNABLE (in native)
	at sun.nio.ch.FileDispatcherImpl.force0(Native Method)
	at sun.nio.ch.FileDispatcherImpl.force(FileDispatcherImpl.java:80)
	at sun.nio.ch.FileChannelImpl.force(FileChannelImpl.java:388)
	at hudson.util.FileChannelWriter.close(FileChannelWriter.java:89)
	at hudson.util.AtomicFileWriter.close(AtomicFileWriter.java:174)
	at hudson.util.AtomicFileWriter.commit(AtomicFileWriter.java:188)
	at hudson.XmlFile.write(XmlFile.java:198)
	at org.jenkinsci.plugins.workflow.support.PipelineIOUtils.writeByXStream(PipelineIOUtils.java:30)
	at org.jenkinsci.plugins.workflow.job.WorkflowRun.save(WorkflowRun.java:1158)
	-  locked org.jenkinsci.plugins.workflow.job.WorkflowRun@2ad5551b
	at hudson.BulkChange.commit(BulkChange.java:98)
	at org.jenkinsci.plugins.workflow.cps.CpsFlowExecution.notifyListeners(CpsFlowExecution.java:1475)
	at org.jenkinsci.plugins.workflow.cps.CpsThreadGroup$3.run(CpsThreadGroup.java:458)
	at org.jenkinsci.plugins.workflow.cps.CpsVmExecutorService$1.run(CpsVmExecutorService.java:38)
	at …
Vlatombe referenced this pull request in Vlatombe/kubernetes-plugin Jan 13, 2020
Adjust after changes to kubernetes-credentials
Vlatombe pushed a commit that referenced this pull request Sep 11, 2020
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.

3 participants