Commit 6ae1fade authored by Iustin Pop's avatar Iustin Pop
Browse files

Fix OS queries for API v20 w/parameters



OS parameters is a list of tuples, so we can't pass it directly to
utils.NiceSort, hence we use a sort key.

This was not detected in QA since QA only tests API v10 :(
Signed-off-by: default avatarIustin Pop <iustin@google.com>
Reviewed-by: default avatarMichael Hanselmann <hansmi@google.com>
parent 9f039737
......@@ -1955,7 +1955,9 @@ def _BuildOsFields():
None, 0, _ConvWrap(sorted, _GetItemAttr("api_versions"))),
(_MakeField("parameters", "Parameters", QFT_OTHER,
"Operating system parameters"),
None, 0, _ConvWrap(utils.NiceSort, _GetItemAttr("parameters"))),
None, 0, _ConvWrap(compat.partial(utils.NiceSort,
key=operator.itemgetter(0)),
_GetItemAttr("parameters"))),
(_MakeField("node_status", "NodeStatus", QFT_OTHER,
"Status from node"),
None, 0, _GetItemAttr("node_status")),
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment