-
Notifications
You must be signed in to change notification settings - Fork 55
Switch Jenkins image tracking to registry tag due to Samples Operator… … deprecation #1151
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
apiVersion: template.openshift.io/v1 | ||
kind: Template | ||
metadata: | ||
name: jenkins-images-template | ||
objects: | ||
- apiVersion: image.openshift.io/v1 | ||
kind: ImageStream | ||
metadata: | ||
name: jenkins-agent-base | ||
spec: | ||
tags: | ||
- name: latest | ||
from: | ||
kind: DockerImage | ||
name: registry.redhat.io/ocp-tools-4/jenkins-agent-base-rhel9:v4.17.0 | ||
importPolicy: | ||
scheduled: true | ||
referencePolicy: | ||
type: Local | ||
|
||
- apiVersion: image.openshift.io/v1 | ||
kind: ImageStream | ||
metadata: | ||
name: jenkins | ||
spec: | ||
tags: | ||
- name: latest | ||
from: | ||
kind: DockerImage | ||
name: registry.redhat.io/ocp-tools-4/jenkins-rhel9:v4.17.0 | ||
importPolicy: | ||
scheduled: true | ||
referencePolicy: | ||
type: Local |
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
|
@@ -100,7 +100,7 @@ objects: | |||
-Dfile.encoding=UTF-8 | ||||
-Dorg.jenkinsci.plugins.durabletask.BourneShellScript.HEARTBEAT_CHECK_INTERVAL=900 | ||||
-Dorg.jenkinsci.plugins.durabletask.BourneShellScript.LAUNCH_DIAGNOSTICS=true | ||||
-Dorg.csanchez.jenkins.plugins.kubernetes.pipeline.PodTemplateStepExecution.defaultImage=image-registry.openshift-image-registry.svc:5000/${AGENT_NAMESPACE}/jenkins-agent-base:latest | ||||
ravanelli marked this conversation as resolved.
Show resolved
Hide resolved
|
||||
-Dorg.csanchez.jenkins.plugins.kubernetes.pipeline.PodTemplateStepExecution.defaultImage=image-registry.openshift-image-registry.svc:5000/${NAMESPACE}/jenkins-agent-base:${JENKINS_IMAGE_STREAM_TAG} | ||||
-Dorg.csanchez.jenkins.plugins.kubernetes.pipeline.PodTemplateStepExecution.defaultContainer.defaultCpuRequest=1 | ||||
-Dorg.csanchez.jenkins.plugins.kubernetes.pipeline.PodTemplateStepExecution.defaultContainer.defaultMemoryRequest=512Mi | ||||
-Dorg.csanchez.jenkins.plugins.kubernetes.pipeline.PodTemplateStepExecution.defaultContainer.defaultCpuLimit=1 | ||||
|
@@ -172,7 +172,7 @@ objects: | |||
- jenkins | ||||
from: | ||||
kind: ImageStreamTag | ||||
name: ${JENKINS_IMAGE_STREAM_TAG} | ||||
name: jenkins:2 | ||||
namespace: ${NAMESPACE} | ||||
lastTriggeredImage: "" | ||||
type: ImageChange | ||||
|
@@ -255,12 +255,8 @@ parameters: | |||
- description: The OpenShift Namespace where the Jenkins ImageStream resides. | ||||
displayName: Jenkins ImageStream Namespace | ||||
name: NAMESPACE | ||||
value: openshift | ||||
value: fedora-coreos-pipeline | ||||
Comment on lines
255
to
+258
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Probably not needed anymore since we'll be operating in our own namespace? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. which means we can also delete |
||||
# DELTA: add separate agent namespace parameter | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Delete
Suggested change
|
||||
- description: The OpenShift Namespace where the Jenkins Agent ImageStream resides. | ||||
displayName: Jenkins Agent ImageStream Namespace | ||||
name: AGENT_NAMESPACE | ||||
value: openshift | ||||
- description: Whether to perform memory intensive, possibly slow, synchronization | ||||
with the Jenkins Update Center on start. If true, the Jenkins core update monitor | ||||
and site warnings monitor are disabled. | ||||
|
@@ -270,7 +266,7 @@ parameters: | |||
- description: Name of the ImageStreamTag to be used for the Jenkins image. | ||||
displayName: Jenkins ImageStreamTag | ||||
name: JENKINS_IMAGE_STREAM_TAG | ||||
value: jenkins:2 | ||||
value: upstream | ||||
- description: When a fatal error occurs, an error log is created with information | ||||
and the state obtained at the time of the fatal error. | ||||
displayName: Fatal Error Log File | ||||
|
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.
Now we have two imagestreams named
jenkins
. The other one is on line 50 ofjenkins-s2i.yaml
. I think one of them has to go.