Skip to content

fix: return the source database version and tuning params in any case

Artyom Kartasov requested to merge source-properties into master

Description

Always return the source database version and tuning parameters

Related issue

Examples

Success

{
  "status": "ok",
  "result": "ok",
  "message": "Database ready for dump and restore",
  "dbVersion": 14,
  "tuningParams": {}
}

Error

{
  "status": "error",
  "result": "connection_error",
  "message": "failed to connect to `host=172.17.0.2 user=postgres database=t`: server error (FATAL: database \"t\" does not exist (SQLSTATE 3D000))",
  "dbVersion": 14,
  "tuningParams": {}
}

Notice

{
  "status": "notice",
  "result": "unverified_database",
  "message": "Too many databases were requested to be checked. Only the following databases have been verified: postgres, protect, test, t1, t2",
  "dbVersion": 14,
  "tuningParams": {}
}

Checklist

  • MR description has been reviewed
  • MR changes are functionally tested
  • MR does NOT have API/CLI changes OR there are API/CLI changes and they have been reviewed & DOCS ARE ADJUSTED (reference doc, etc)
  • MR does NOT have UI changes OR there are UI changes and they have been reviewed & UX IS REVIEWED

Merge request reports