CLOUDP-186802: Add connection string options

[lucian-tosa]

Example:

  apiVersion: mongodbcommunity.mongodb.com/v1
  kind: MongoDBCommunity
  metadata:
    name: example-mongodb
  spec:
    members: 3
    type: ReplicaSet
    version: "6.0.5"
    security:
      authentication:
        modes: ["SCRAM"]
    users:
      - name: my-user
        db: admin
        passwordSecretRef:
          name: my-user-password
        roles:
          - name: clusterAdmin
            db: admin
          - name: userAdminAnyDatabase
            db: admin
        scramCredentialsSecretName: my-scram
        additionalConnectionStringConfig:
          readPreference: secondary
    additionalMongodConfig:
      storage.wiredTiger.engineConfig.journalCompressor: zlib
    additionalConnectionStringConfig:
      readPreference: primary

• In this example, the connection string for my-user will have as a connection string option readPreference=secondary by overriding the resource option.
• Any options included will not be validated and using the connection strings with incorrect options will fail.

Closes #1134

All Submissions:

• Have you opened an Issue before filing this PR?
• Have you signed our CLA?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?
• Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

[slaskawi]

Added a few questions and a suggestion but overall - great work 💪 LGTM!

[slaskawi]

Perfect! Thanks for checking!

…

On Thu, Jul 6, 2023 at 12:07 PM lucian-tosa ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In api/v1/mongodbcommunity_types.go <#1324 (comment)> : > @@ -119,6 +119,69 @@ type MongoDBCommunitySpec struct { // Prometheus configurations. // +optional Prometheus *Prometheus `json:"prometheus,omitempty"` + + // Additional options to be appended to the connection string. These options apply to the entire resource and to each user. + // +kubebuilder:validation:Type=object + // +optional + // +kubebuilder:pruning:PreserveUnknownFields + // +nullable + AdditionalConnectionStringConfig MapWrapper `json:"additionalConnectionStringConfig,omitempty"` I did not try that actually. But according to this open issue: kubernetes-sigs/controller-tools#636 <kubernetes-sigs/controller-tools#636> and the comments that were already in the code, it should not work. — Reply to this email directly, view it on GitHub <#1324 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACAG7BRTC6ZJ62IF24PRWTLXO2E7BANCNFSM6AAAAAAZ53YVZ4> . You are receiving this because your review was requested.Message ID: ***@***.*** com>

[mircea-cosbuc]

LGTM! Nice work