DAT-15208 :: Changeset object added back to "Update command completed successfully" JSON

[MalloD12]

Update logDeploymentOutcomeMdc to add mdc values before logging update command execution result"

{
  "timestamp": "2023-07-27T19:58:58.897Z",
  "level": "INFO",
  "class": "liquibase.command",
  "thread": "1",
  "message": "Update command completed successfully.",
  "operationStart": "2023-07-27T16:58:58.367",
  "liquibaseCommandName": "update",
  "liquibaseInternalCommand": "update",
  "liquibaseTargetUrl": "jdbc:postgresql://localhost:5438/lbcat",
  "liquibaseCatalogName": "lbcat",
  "liquibaseSchemaName": "",
  "rollbackOnError": "true",
  "commandLabelFilter": "primaryTable",
  "commandContextFilter": "fltr1",
  "deploymentId": "0487938863",
  "changelogFile": "changelogWithDropTableNotExistChangeset.xml",
  "rowsAffected": "1",
  "deploymentOutcomeCount": "1",
  "changesetsUpdated": {
    "changeset": [
      {
        "changesetAuthor": "Liquibase Pro User",
        "changesetFilepath": "changelogWithDropTableNotExistChangeset.xml",
        "changesetId": "createPrimaryTable",
        "changesetOutcome": "EXECUTED",
        "deploymentId": "0487938863"
      }
    ]
    ,
    "changesetCount": 1
  },
  "deploymentOutcome": "success"
}

Impact

• [ x] Bug fix (non-breaking change which fixes expected existing functionality)
• Enhancement/New feature (adds functionality without impacting existing logic)
• Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

• None

Things to worry about

• None

Additional Context

• Nothing

[MalloD12]

Hi guys, I just would like to check with you whether you think we should unit test this. I talked to Oleksii and confirmed to me we are covering this case as part of the functional tests in the UpdateCountStructuredLoggingTests and UpdateWithStructuredLoggingTests test suites.

[wwillard7800]

There are also tests in the liqubase-pro repository that should cover this. They run with each build.

[MalloD12]

Guys, closing this one and opening #4581 in order to avoid issues with the branch name. Regarding the test I saw failing it was because of an environment variable value I was using.