You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2023/01/15 23:25:22 UTC

[GitHub] [couchdb-helm] yekibud commented on a diff in pull request #105: Support existing persistent volume

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 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 this 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: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org