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-45349] - Test Client: Allow disabling file caching in HelloGetResource and HelloGetResources. #181

Merged

Conversation

Projects
None yet
3 participants
@oleg-nenashev
Copy link
Member

commented Jul 20, 2017

This change adds an optional parameter, which disables file caching. My plan is to reuse it in Remoting tests in order to suppress some issues in Windows tests.

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

@reviewbybees @jtnord

[JENKINS-45349] - Allow disabling file caching in HelloGetResource an…
…d HelloGetResources.

This change adds an optional parameter, which disables file caching. My plan is to reuse it in Remoting tests in order to suppress some issues in Windows tests.
@reviewbybees

This comment has been minimized.

Copy link

commented Jul 20, 2017

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@@ -39,6 +39,10 @@ THE SOFTWARE.
This module is separate because some of the tests
require code in a jar file not available in the default classpath.
</description>

<properties>
<java.level>7</java.level>

This comment has been minimized.

Copy link
@jtnord

This comment has been minimized.

Copy link
@oleg-nenashev

oleg-nenashev Jul 27, 2017

Author Member

7 for now, Remoting is on Java 7, will update later to 8

@jtnord

jtnord approved these changes Jul 27, 2017

@oleg-nenashev oleg-nenashev merged commit b38f211 into jenkinsci:test-client Jul 28, 2017

1 check passed

Jenkins This pull request looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.