Bug 1515972 – Review Jenkins JNLP slave setup script

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1515972 - Review Jenkins JNLP slave setup script

Summary:	Review Jenkins JNLP slave setup script

Status:	CLOSED ERRATA

Aliases:	None

Product:	OpenShift Container Platform
Classification:	Red Hat
Component:	Build (Show other bugs)
Sub Component:
Version:	unspecified
Hardware:	Unspecified Unspecified

Priority	unspecified Severity unspecified
Target Milestone:	---
Target Release:	3.9.0
Assigned To:	Jim Minter
QA Contact:	Wenjing Zheng
Docs Contact:

URL:
Whiteboard:
Keywords:

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2017-11-21 12:26 EST by Jim Minter
Modified:	2018-03-28 10:13 EDT (History)
CC List:	8 users (show)

See Also:
Fixed In Version:
Doc Type:	If docs needed, set a value
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2018-03-28 10:13:03 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHBA-2018:0489	None	None	None	2018-03-28 10:13 EDT

Groups: None (edit)

Description Jim Minter 2017-11-21 12:26:08 EST

- Look for opportunities to simplify the run-jnlp-client code
- Confirm why are we setting -Xms, what effect it has and if it is needed
- Allow -Xmx=<amount> to be specified as a specific amount, not just a % of container size
- Consider capping % of container size calculation for large containers (i.e. min(% * container size, 256MB)
- Consider master switch for disabling all smarts
- Consider not using JAVA_OPTS as an environment variable name as it might conflict with spawned Java processes
- Investigate how to tune based on container *request*, not container limits, e.g. using downward API
- Investigate if setting additional environment variable presets would make sense for subsequent workload (JAVA_OPTS, MAVEN_OPTS, GRADLE_OPTS, JAVA_TOOL_OPTIONS, _JAVA_OPTIONS?)

Comment 1 Gabe Montero 2017-11-27 09:54:45 EST

some bugzilla maintenance per request from Paul Weil wrt bugs with no target and 3.7 closing up ... setting this untargetted bug to post 3.7

Comment 2 Jim Minter 2017-12-18 14:20:40 EST

https://github.com/openshift/jenkins/pull/442

Comment 5 Jim Minter 2018-01-08 14:12:50 EST

Trello card: https://trello.com/c/SvSBSkOy/1425-8-jenkins-memory-cleanup-evg

Comment 7 Dongbo Yan 2018-01-10 00:29:53 EST

Test with
openshift3/jenkins-2-rhel7:v3.9(4e3dc986043c)
openshift3/jenkins-slave-base-rhel7:v3.9(3b1e124d2bcd)
openshift3/jenkins-slave-maven-rhel7:v3.9(aa5c4ce0e097)

could move to verified

Comment 13 errata-xmlrpc 2018-03-28 10:13:03 EDT

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0489

Note You need to log in before you can comment on or make changes to this bug.