[tempestmail] --file option should not require log_url or LOG_PATH

[Rafael Folco]

Tempestmail allows you to specify --file <tempest_output.log.txt> file containing tempest test results:

tempest.test.x.y.z ... FAIL
tempest.test.x.y.w ... ok

But if you don't specify log_url / LOG_PATH, tempestmail doesn't even run correctly. It just loads config and quits.

LOG_PATH='99/526099/1/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/Z2b3cc321ef7e412f903ac4241b5720f6' python tempestmail.py -c /tmp/tempestmail_config.yaml --job gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master --file tempest_output.log.txt.gz --log-url \"http://logs.rdoproject.org\"

The request is to process --file <tempest_output.log.txt> regardless of LOG_PATH. This would allow local tests.

PS: unit tests work because LOG_PATH is mocked

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/694021

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/694021
Committed: https://git.openstack.org/cgit/openstack/tripleo-quickstart-extras/commit/?id=c0fbe50583cc940894f8929e94c8b554898a470b
Submitter: Zuul
Branch: master

commit c0fbe50583cc940894f8929e94c8b554898a470b
Author: Daniel Bengtsson <email address hidden>
Date: Wed Nov 13 10:28:01 2019 +0100

    The file option should not require environment variable.

    The file option should not require log_url or LOG_PATH. Actually the
    link variable can be an empty string.

    Change-Id: I33439eb4748ef892fcf132d7cab766bc03dff038
    Closes-Bug: #1742139