Skip to content

Commit

Permalink
tune remote-resolvers again for memory issues seen on prod-rh01 (#4170)
Browse files Browse the repository at this point in the history
rh-pre-commit.version: 2.3.0
rh-pre-commit.check-secrets: ENABLED
  • Loading branch information
gabemontero authored Jul 24, 2024
1 parent 49281e4 commit 9965484
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -36,15 +36,15 @@
# replicas - default pipeline-service setting is 1
# memory limit - default upstream setting is 4Gi
value:
replicas: 2
replicas: 1
template:
spec:
containers:
- name: controller
resources:
limits:
cpu: "1"
memory: 8Gi
memory: 16Gi
requests:
cpu: 100m
memory: 100Mi
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2072,15 +2072,15 @@ spec:
memory: 100Mi
tekton-pipelines-remote-resolvers:
spec:
replicas: 2
replicas: 1
template:
spec:
containers:
- name: controller
resources:
limits:
cpu: "1"
memory: 8Gi
memory: 16Gi
requests:
cpu: 100m
memory: 100Mi
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2072,15 +2072,15 @@ spec:
memory: 100Mi
tekton-pipelines-remote-resolvers:
spec:
replicas: 2
replicas: 1
template:
spec:
containers:
- name: controller
resources:
limits:
cpu: "1"
memory: 8Gi
memory: 16Gi
requests:
cpu: 100m
memory: 100Mi
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2072,15 +2072,15 @@ spec:
memory: 100Mi
tekton-pipelines-remote-resolvers:
spec:
replicas: 2
replicas: 1
template:
spec:
containers:
- name: controller
resources:
limits:
cpu: "1"
memory: 8Gi
memory: 16Gi
requests:
cpu: 100m
memory: 100Mi
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2072,15 +2072,15 @@ spec:
memory: 100Mi
tekton-pipelines-remote-resolvers:
spec:
replicas: 2
replicas: 1
template:
spec:
containers:
- name: controller
resources:
limits:
cpu: "1"
memory: 8Gi
memory: 16Gi
requests:
cpu: 100m
memory: 100Mi
Expand Down

1 comment on commit 9965484

@ralphbean
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gabemontero we're still chasing remote resolver performance issues - most recently, seeking to tune the number of threads: tektoncd/pipeline#8344

Do you remember why you tuned the number of replicas down here?

Please sign in to comment.