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.

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

DatabaseInstance replica configuration required for non-mysql instances #379

Closed
snelsondexcom opened this issue Sep 6, 2023 · 3 comments

Comments

@snelsondexcom
Copy link

What happened?

Per the docs, replicaConfiguration should only be applicable to MySQL instances:

Valid only for MySQL instances

I was able to create a read replica for a PostgreSQL CloudSQL instance using the DatabaseInstance resource but it failed to sync with the following error:

"error": "cannot run refresh: refresh failed: Missing required argument: \"replica_configuration.0.master_heartbeat_period\" ...

Workaround to get the refresh to succeed -- provide an "empty" replica config, e.g.

    replicaConfiguration:
    - clientCertificate: ""
      clientKey: ""
      connectRetryInterval: 0
      dumpFilePath: ""
      failoverTarget: false
      masterHeartbeatPeriod: 0
      sslCipher: ""
      username: ""
      verifyServerCertificate: false

How can we reproduce it?

Create a read replica for a PostgreSQL CloudSQL instance.

What environment did it happen in?

  • Crossplane Version: v1.10.2
  • Provider Version: v0.35.1
  • Kubernetes Version: v1.28.1
  • Kubernetes Distribution: GKE
@snelsondexcom snelsondexcom added bug Something isn't working needs:triage labels Sep 6, 2023
@turkenf
Copy link
Collaborator

turkenf commented Sep 6, 2023

Hi @snelsondexcom,

We have a new provider version, could you please try provider-gcp v0.36.0 and let us know?

@turkenf
Copy link
Collaborator

turkenf commented Oct 2, 2023

Could not reproduce this issue with the provided information in version 0.37.0, so I am closing for now.

If you still encounter the same issue, feel free to open a new issue by adding the replication steps and example MRs.

@snelsondexcom
Copy link
Author

New issue created: #413

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants