yekibud commented on code in PR #105: URL: https://github.com/apache/couchdb-helm/pull/105#discussion_r1070711871
##########
couchdb/README.md:
##########
@@ -164,63 +164,67 @@ required options to set:
A variety of other parameters are also configurable. See the comments in the
`values.yaml` file for further details:
-| Parameter | Default
|
-|--------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------|
-| `adminUsername` | admin
|
-| `adminPassword` | auto-generated
|
-| `adminHash` |
|
-| `cookieAuthSecret` | auto-generated
|
-| `image.repository` | couchdb
|
-| `image.tag` | 3.2.1
|
-| `image.pullPolicy` | IfNotPresent
|
-| `searchImage.repository` | kocolosk/couchdb-search
|
-| `searchImage.tag` | 0.1.0
|
-| `searchImage.pullPolicy` | IfNotPresent
|
-| `initImage.repository` | busybox
|
-| `initImage.tag` | latest
|
-| `initImage.pullPolicy` | Always
|
-| `ingress.enabled` | false
|
-| `ingress.hosts` | chart-example.local
|
-| `ingress.annotations` |
|
-| `ingress.path` | /
|
-| `ingress.tls` |
|
-| `persistentVolume.accessModes` | ReadWriteOnce
|
-| `persistentVolume.storageClass` | Default for the Kube cluster
|
-| `persistentVolume.annotations` | {}
|
-| `podManagementPolicy` | Parallel
|
-| `affinity` |
|
-| `topologySpreadConstraints` |
|
-| `annotations` |
|
-| `tolerations` |
|
-| `resources` |
|
-| `service.annotations` |
|
-| `service.enabled` | true
|
-| `service.type` | ClusterIP
|
-| `service.externalPort` | 5984
|
-| `dns.clusterDomainSuffix` | cluster.local
|
-| `networkPolicy.enabled` | true
|
-| `serviceAccount.enabled` | true
|
-| `serviceAccount.create` | true
|
-| `serviceAccount.imagePullSecrets` |
|
-| `sidecars` | {}
|
-| `livenessProbe.enabled` | true
|
-| `livenessProbe.failureThreshold` | 3
|
-| `livenessProbe.initialDelaySeconds` | 0
|
-| `livenessProbe.periodSeconds` | 10
|
-| `livenessProbe.successThreshold` | 1
|
-| `livenessProbe.timeoutSeconds` | 1
|
-| `readinessProbe.enabled` | true
|
-| `readinessProbe.failureThreshold` | 3
|
-| `readinessProbe.initialDelaySeconds` | 0
|
-| `readinessProbe.periodSeconds` | 10
|
-| `readinessProbe.successThreshold` | 1
|
-| `readinessProbe.timeoutSeconds` | 1
|
-| `prometheusPort.enabled` | false
|
-| `prometheusPort.port` | 17896
|
-| `prometheusPort.bind_address` | 0.0.0.0
|
-| `placementConfig.enabled` | false
|
-| `placementConfig.image.repository` |
caligrafix/couchdb-autoscaler-placement-manager
|
-| `placementConfig.image.tag` | 0.1.0
|
+| Parameter | Default
|
+| -------------------------------------- |
------------------------------------------------ |
+| `adminUsername` | admin
|
+| `adminPassword` | auto-generated
|
+| `adminHash` |
|
+| `cookieAuthSecret` | auto-generated
|
+| `image.repository` | couchdb
|
+| `image.tag` | 3.2.1
|
+| `image.pullPolicy` | IfNotPresent
|
+| `searchImage.repository` | kocolosk/couchdb-search
|
+| `searchImage.tag` | 0.1.0
|
+| `searchImage.pullPolicy` | IfNotPresent
|
+| `initImage.repository` | busybox
|
+| `initImage.tag` | latest
|
+| `initImage.pullPolicy` | Always
|
+| `ingress.enabled` | false
|
+| `ingress.hosts` | chart-example.local
|
+| `ingress.annotations` |
|
+| `ingress.path` | /
|
+| `ingress.tls` |
|
+| `persistentVolume.accessModes` | ReadWriteOnce
|
+| `persistentVolume.storageClass` | Default for the Kube cluster
|
+| `persistentVolume.annotations` | {}
|
+| `persistentVolume.existing` | false
|
Review Comment:
Unfortunately [this recently merged
PR](https://github.com/apache/couchdb-helm/pull/106) added unnecessary and
excessive spacing to a table in the README, in addition to obscuring the diff.
The changes in the current PR are the additions of lines #190-193.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
