Closed supalikashyap closed 5 years ago
@supalikashyap Could you address the static code analysis issues (Codacy, pep8) in this PR?
@bo-quan, as we discussed in the last meeting can we suppress the paramiko and subprocess warnings in codacy.
@raman-mann For now, let's ignore the paramiko and subprocess warnings in codacy.
What does this PR do?
Validates the config file
Do you have any concerns with this PR?
NA
How can the reviewer verify this PR?
changes can be verified by changing the config file parameters value and tag as well
Any background context you want to provide?
NA
Screenshots or logs (if appropriate)
NA
Questions:
Have you connected this PR to the issue it resolves? NA
Does the documentation need an update? document updated as few of unused parameters are removed
Does this add new Python dependencies? python paramiko module is added for ssh
Have you added unit or functional tests for this PR? NA
Does this patch update any configuration files? NA no