Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: xenit-eu/helm-alfresco
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.8.1
Choose a base ref
...
head repository: xenit-eu/helm-alfresco
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
  • 9 commits
  • 6 files changed
  • 3 contributors

Commits on Nov 6, 2024

  1. [NYS2AWS-71] allow node anti-affinity for Solr pods

    pvriel committed Nov 6, 2024
    Copy the full SHA
    c4e9660 View commit details

Commits on Nov 7, 2024

  1. [NYS2AWS-71] replace hostname with zone affinity

    pvriel committed Nov 7, 2024
    Copy the full SHA
    d84a70d View commit details
  2. [NYS2AWS-71] update README with info from testing results

    pvriel committed Nov 7, 2024
    Copy the full SHA
    e8d56c1 View commit details
  3. Merge pull request #166 from xenit-eu/NYS2AWS-71-hostname-anti-affinity

    [NYS2AWS-71] allow node anti-affinity for Solr pods
    pvriel authored Nov 7, 2024

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    6e92fec View commit details
  4. [NYS2AWS-74] enforce quotes for ingress annotations

    pvriel committed Nov 7, 2024
    Copy the full SHA
    077bd27 View commit details
  5. Merge pull request #167 from xenit-eu/NYS2AWS-74-fix-ingress-annotati…

    …ons-quote-bug
    
    [NYS2AWS-74] enforce quotes for ingress annotations
    pvriel authored Nov 7, 2024

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    4f8f9b2 View commit details

Commits on Nov 26, 2024

  1. [XENOPS-1229] disable pvc creation for shared-file-system in case of …

    …community edition (#168)
    
    * [XENOPS-1229] remove shared-file-store pvc
    
    * [XENOPS-1229] remove shared-file-store pvc
    
    * [XENOPS-1229] remove shared-file-store pvc
    gert-glassee authored Nov 26, 2024

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    f049aae View commit details

Commits on Dec 19, 2024

  1. KANSUP74-24 Update README.md

    RVanhuysseXenit authored Dec 19, 2024

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    54a2792 View commit details

Commits on Dec 20, 2024

  1. Merge pull request #169 from xenit-eu/KANSUP74-24_extend_documentation

    KANSUP74-24 Update README.md
    RVanhuysseXenit authored Dec 20, 2024

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    b9f2b73 View commit details
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -6,6 +6,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
chronology things are added/fixed/changed and - where possible - links to the PRs involved.

### Changes
[v0.8.4]
* do not create a pvc for shared-file-store when enterprise=false

[v0.8.3]
* fixed a bug that caused the ingress annotations to miss necessary quotes

[v0.8.2]
* added solr.enforceZoneAntiAffinity

[v0.8.0]

21 changes: 20 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
@@ -308,6 +308,15 @@ nginx rules to redirect the normal pages to a 503 maintenance page.
* Default: nginx-default-service
* Description: the default service name that ingress will point to

**Note: root redirect on nginx**: Nginx can be configured to redirect from root to a specific app through annotations.
```yaml
ingress:
host: alf1.xenit.eu
protocol: https
ingressAnnotations:
nginx.ingress.kubernetes.io/app-root: /share
```
#### `ingress.defaultPath.port`

* Required: true
@@ -1330,6 +1339,16 @@ nginx rules to redirect the normal pages to a 503 maintenance page.
```
* Description: With this list of parameters you can add 1 or multiple annotations to the Solr service

#### `solr.enforceZoneAntiAffinity`

* Required: false
* Default: false
* Description: If true, this option enforces a pod anti-affinity on the Solr pods based on the zones.
I.e. if true, each Solr pod will require a unique zone. Starting more pods than zones will put the
remainder pods in a `Pending` state.
<br>
<b>Please note that switching off the anti-affinity does not alter the anti-affinity of the already running pods!</b>

#### `solr.serviceAccount`

* Required: false
@@ -2326,4 +2345,4 @@ additional settings can be added through additionalEnvironmentVariables.
storage: 2
efs:
volumeHandle: "efs-identifier"
```
```
2 changes: 1 addition & 1 deletion xenit-alfresco/templates/ingress/alfresco-ingress.yaml
Original file line number Diff line number Diff line change
@@ -14,7 +14,7 @@ metadata:
*/ -}}
{{- range $key, $value := .Values.ingress.ingressAnnotations }}
{{- if not ( and ($.Values.ingress.ingressClass) (eq $key "kubernetes.io/ingress.class")) }}
{{ $key }}: {{ $value }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
{{- end }}
12 changes: 12 additions & 0 deletions xenit-alfresco/templates/solr/solr-stateful-set.yaml
Original file line number Diff line number Diff line change
@@ -30,6 +30,18 @@ spec:
{{ toYaml .Values.solr.podAnnotations | nindent 8 }}
{{- end }}
spec:
{{- if .Values.solr.enforceZoneAntiAffinity }}
affinity:
podAntiAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
- topologyKey: "topology.kubernetes.io/zone"
labelSelector:
matchExpressions:
- key: "app"
operator: "In"
values:
- solr
{{- end }}
{{- if .Values.solr.serviceAccount }}
serviceAccountName: {{ .Values.solr.serviceAccount }}
{{- end }}
Original file line number Diff line number Diff line change
@@ -1,10 +1,12 @@
{{- $namespace := .Release.Namespace -}}
{{- if .Values.general.enterprise }}
{{- with .Values.persistentStorage.sharedFileStore }}
{{- if .enabled}}
{{- if .enabled }}
{{- $name := "shared-file-store" -}}
{{- $storageClassName := .storageClassName -}}
{{- $storage := .storage -}}
{{- $efsVolumeHandle := .efs.volumeHandle -}}
{{- include "hepers.volumeHelper" (list $namespace $name $storageClassName $storage $efsVolumeHandle) }}
{{- end }}
{{- end }}
{{- end }}
1 change: 1 addition & 0 deletions xenit-alfresco/values.yaml
Original file line number Diff line number Diff line change
@@ -151,6 +151,7 @@ postgresql:
solr:
enabled: true
replicas: 2
enforceZoneAntiAffinity: false
podManagementPolicy: Parallel
image:
registry: 'docker.io'