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.

Sign up for GitHub

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

Jump to bottom

OPSEXP-1353 Deprecate Alfresco Sync Service HPA template #874

Merged
merged 7 commits into from
Feb 3, 2023
Merged

OPSEXP-1353 Deprecate Alfresco Sync Service HPA template #874

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
be35c2d
OPSEXP-1353 Removed Alfresco Sync service HPA chart
upandey123 Feb 1, 2023
3ba5e0c
removed sync service enabled from values.yaml
upandey123 Feb 1, 2023
2fd6f04
fixed helm docs pre-commit
upandey123 Feb 1, 2023
73d1cf2
Removed sync service values HPA parameters
upandey123 Feb 2, 2023
be44681
added sync service enabled in main values file
upandey123 Feb 2, 2023
e6b7f31
Fixed Helm Pre-commit docs
upandey123 Feb 2, 2023
4a3cb2d
Revert syncservice: enabled in values file for deployment
upandey123 Feb 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 0 additions & 8 deletions helm/alfresco-content-services/charts/alfresco-sync-service/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,16 +59,8 @@ Alfresco Sync Service
| replicaCount | int | `1` | |
| repository.host | string | `"alfresco-cs-repository"` | |
| repository.port | int | `80` | |
| syncservice.enabled | bool | `true` | |
| syncservice.environment.EXTRA_JAVA_OPTS | string | `""` | |
| syncservice.environment.JAVA_OPTS | string | `"-Dsync.metrics.reporter.graphite.enabled=false -Dsync.metrics.reporter.graphite.address=127.0.0.1 -Dsync.metrics.reporter.graphite.port=2003 -XX:MinRAMPercentage=50 -XX:MaxRAMPercentage=80"` | |
| syncservice.horizontalPodAutoscaling.CPU.enabled | bool | `true` | |
| syncservice.horizontalPodAutoscaling.CPU.targetAverageUtilization | int | `80` | |
| syncservice.horizontalPodAutoscaling.enabled | bool | `true` | |
| syncservice.horizontalPodAutoscaling.maxReplicas | int | `3` | |
| syncservice.horizontalPodAutoscaling.memory.enabled | bool | `true` | |
| syncservice.horizontalPodAutoscaling.memory.targetAverageUtilization | int | `60` | For the memory a lower threshold(60) for the targetAverageUtilization is needed. We need to allow the resource metrics to be queried by the metrics-server, before the pod is killed # by Kubernetes due to reaching memory limits(the infamous message one might see in the pod events history "Terminated: OOMKilled"). The metrics are checked every 15 seconds by default, configured by the global cluster flag --horizontal-pod-autoscaler-sync-period |
| syncservice.horizontalPodAutoscaling.minReplicas | int | `1` | |
| syncservice.image.internalPort | int | `9090` | |
| syncservice.image.pullPolicy | string | `"IfNotPresent"` | |
| syncservice.image.repository | string | `"quay.io/alfresco/service-sync"` | |
Expand Down
33 changes: 0 additions & 33 deletions helm/alfresco-content-services/charts/alfresco-sync-service/templates/hpa-syncservice.yaml
View file
Open in desktop

This file was deleted.

18 changes: 0 additions & 18 deletions helm/alfresco-content-services/charts/alfresco-sync-service/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ podSecurityContext:
runAsNonRoot: true
fsGroup: 1000
syncservice:
enabled: true
upandey123 marked this conversation as resolved.
Show resolved Hide resolved
image:
repository: quay.io/alfresco/service-sync
tag: 4.0.0-M5
Expand Down Expand Up @@ -50,23 +49,6 @@ syncservice:
periodSeconds: 10
failureThreshold: 12
timeoutSeconds: 10
horizontalPodAutoscaling:
enabled: true
minReplicas: 1
maxReplicas: 3
CPU:
enabled: true
targetAverageUtilization: 80
memory:
enabled: true
# -- For the memory a lower threshold(60) for the targetAverageUtilization
# is needed. We need to allow the resource metrics to be queried by the
# metrics-server, before the pod is killed # by Kubernetes due to reaching
# memory limits(the infamous message one might see in the pod events
# history "Terminated: OOMKilled").
# The metrics are checked every 15 seconds by default, configured by the
# global cluster flag --horizontal-pod-autoscaler-sync-period
targetAverageUtilization: 60
repository:
host: alfresco-cs-repository
port: 80
Expand Down