search

Aiven-Open / aiven-mysql-migrate

MySQL® migration tool
Apache License 2.0
10 stars 5 forks source link

cli: introduce new flags to write the validation output in json format #9

Closed ansrivas closed 3 years ago

ansrivas commented 3 years ago

Description:

cli: introduce new flags to write the validation output in json format

    # json output 
    mysql_migrate --validate-only --json
    {"method": "dump", "reason": "Replication method is not available due to missing TARGET_MASTER_SERVICE_URI, falling back to dump"}

Removed the --json-file after the review, didn't seem necessary after discussing the use-case: ~~mysql_migrate --validate-only --json-file=/tmp/outout.json --log-level=error ❯ cat /tmp/outout.json {"method": "dump", "status": "Replication method is not available due to missing TARGET_MASTER_SERVICE_URI, falling back to dump"}~~

EDIT: current state after addressing the review.

Signed-off-by: Ankur Srivastava ansrivas@aiven.io

alexole commented 3 years ago

Closing as this functionality won't be needed.