Fix minor issues in metal

[Tao Li]

Brief Description
-----------------
There is some minor coding issues in metal, this bug report is tracing these issues fiexed.

Severity
--------
minor

Steps to Reproduce
------------------

Expected Behavior
------------------

Actual Behavior
----------------

Reproducibility
---------------

System Configuration
--------------------
STD system

Branch/Pull Time/Commit
-----------------------
master

[OpenStack Infra (hudson-openstack)]

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

[Ghada Khalil (gkhalil)]

Low priority / not gating any starlingx release -- code improvement; doesn't have any system impact

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/669097
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=c74d50c152446ce701727db01e80f5d63dce01ce
Submitter: Zuul
Branch: master

commit c74d50c152446ce701727db01e80f5d63dce01ce
Author: Tao Li <[e-mail]>
Date: Thu Jul 4 16:21:40 2019 +0800

    Fix the log issue in config handler

    The logged config file is not the parsed config file.

    Change-Id: I72257331f58811167d98a08b9032c51aa4b6681a
    Partial-Bug: #1835370
    Signed-off-by: Tao Li <[e-mail]>

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/669998
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=73146540e2516e3c1e1f4dec38c94b0171946791
Submitter: Zuul
Branch: master

commit 73146540e2516e3c1e1f4dec38c94b0171946791
Author: Tao Li <[e-mail]>
Date: Wed Jul 10 15:25:07 2019 +0800

    Add the mask check for the loaded config

    If some config is not loaded, we can't get the default value,
    so we must ensure all the configs are loaded.

    Change-Id: I198af4fac6f873d3e7785d2105d14898be084d48
    Partial-Bug: #1835370
    Signed-off-by: Tao Li <[e-mail]>

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/692222
Committed: https://git.openstack.org/cgit/starlingx/metal/commit/?id=e356019f954136f597d7fb00e4d972c2535ad504
Submitter: Zuul
Branch: master

commit e356019f954136f597d7fb00e4d972c2535ad504
Author: Don Penney <email address hidden>
Date: Wed Oct 30 16:40:56 2019 -0400

    Update host watchdog CONFIG_MASK

    The CONFIG_MASK in hostw.h includes CONFIG_START_DELAY, which
    correlates to an option that is not actually used in host watchdog. As
    a result, the recently added check that verifies all options in the
    CONFIG_MASK are set fails, and the host watchdog fails to launch.

    This update removes the unused CONFIG_START_DELAY bit from
    CONFIG_MASK.

    Change-Id: I330e15520bc0f01a6cbfd4f83a1953c1c737da2b
    Partial-Bug: 1835370
    Signed-off-by: Don Penney <email address hidden>

[OpenStack Infra (hudson-openstack)]

Change abandoned by Eric MacDonald (<email address hidden>) on branch: master
Review: https://review.opendev.org/669968
Reason: This update is stale. Abandoning.

[Ramaswamy Subramanian (rsubrama)]

No progress on this bug for more than 2 years. Candidate for closure.

If there is no update, this issue is targeted to be closed as 'Won't Fix' in 2 weeks.

[Ramaswamy Subramanian (rsubrama)]

Changing the status to 'Won't Fix' as there is no activity.