Use argparse only if name is main, call main with args dict

Description

Working on the tests, mildly annoyed by way the webservice uses argparse and how it's started so I changed it a bit:

• Adjusted some of the command line argument names and variable names so that they match
• Argparse setup is only done if webservice.py is called by python (if __name__ == "__main__")
• Changed the main function to take in keyword arguments
• Splat argparse args dict into main if called by python

NOTE: --logging has changed to --log-level

How Has This Been Tested?

Not at all

Called it manually with some print stuff:

❯ python3 ./webservice/webservice.py --log-level DEBUG
{'config_file': './webservice.yaml', 'mode': 'sim', 'log_file': './web.log', 'log_level': 'DEBUG'}
./webservice.yaml sim ./web.log DEBUG

Seems to work.

Relevant Documents (optional)

N/A

Types of changes

• Style (formatting changes only, no code changes)
• Refactor (refactoring code with no functionality changes)

Checklist:

• My code follows the code style of this project.

Reviewers

@danilevc @schmidtp @kluyvert

added Waiting for review label

LGTM. We discussed that you did test it by launching the webservice manually.

changed the description

LGTM

added 2 commits

• 6c5aba1f - Use argv argument
• 4fa15f65 - Sort imports

Compare with previous version

added 2 commits

• 1e70acde - Sort imports
• fd5dc6d8 - Add optional argv to main function

Compare with previous version

LGTM

added 1 commit

• 25194051 - Remove vars call

Compare with previous version

added 1 commit

• 25194051 - Remove vars call

Compare with previous version

added 1 commit

• cae262f0 - Use yaml safe load

Compare with previous version

added 1 commit

• e26bc835 - Use yaml safe load

Compare with previous version

resolved all discussions

enabled an automatic merge when the pipeline for e26bc835 succeeds

merged

mentioned in commit 2f9b4860