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

[JENKINS-55960] Missing DeclarativePipeline podTemplate parameters #421

Merged
merged 2 commits into from
Feb 12, 2019

Conversation

cazorla19
Copy link
Contributor

@cazorla19 cazorla19 commented Feb 4, 2019

All issue details here:

https://issues.jenkins-ci.org/browse/JENKINS-55960

Changeset:

  1. Added slaveConnectTimeout and namespace to Declarative Pipeline class.

@cazorla19
Copy link
Contributor Author

@carlossg any update on this issue? It's blocking us as a plugin users, so I'd like to have any feedback

@carlossg carlossg merged commit fb5499f into jenkinsci:master Feb 12, 2019
@marshallford
Copy link
Contributor

marshallford commented Feb 26, 2019

How does the slaveConnectTimeout parameter relate to the system property org.csanchez.jenkins.plugins.kubernetes.PodTemplate.connectionTimeout and the agent connection timeout instructions in the readme?

@carlossg
Copy link
Contributor

org.csanchez.jenkins.plugins.kubernetes.PodTemplate.connectionTimeout is the default if you don't set slaveConnectTimeout

@marshallford
Copy link
Contributor

I am getting following error when setting slaveConnectTimeout to 6000 (100 minutes) after ~21 minutes. Is this parameter related to the error?

Cannot contact jenkins-worker-88405759-c90a-47a9-89b3-a1c368fd64d4-q6sfl-bss5r: hudson.remoting.ChannelClosedException: Channel "unknown": Remote call on JNLP4-connect connection from 10.8.35.7/10.8.35.7:57044 failed. The channel is closing down or has closed down

@carlossg
Copy link
Contributor

carlossg commented Mar 1, 2019

if you can ask in the jenkins-users mailing list with more details

@marshallford
Copy link
Contributor

@carlossg Any chance #442 is related to the issue I described above?

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