phosphor-host-ipmid/test, branch master

Implemented close session cmd in host interface

2019-09-25T17:42:56+00:00

This command can close any session via host interface. Tested: Close the existing valid session by session id ipmitool raw 0x6 0x3c Response : 00 // success Close the existing valid session by session handle ipmitool raw 0x6 0x3c Response : 00 // success Close the session by zero session id ipmitool raw 0x6 0x3c Response : 0x87 // inavlid session id Close the session by zero session handle ipmitool raw 0x6 0x3c Response : 0x88 // inavlid session handle Close an inactive session. ipmitool raw 0x6 0x3c Response : 0xcc // invalid data field in request Close an inactive session. ipmitool raw 0x6 0x3c Response : 0xcc // invalid data field in request Signed-off-by: Rajashekar Gade Reddy Change-Id: I8af290001d8effbbcdbbe2dd93aabf1b015e7a88

drop entity YAML in favor of json provided file

2019-09-19T17:20:47+00:00

Step 5 of moving from entity map from YAML to JSON drops support for a built-in YAML mapping of the entity containers. Tested: Not tested. No platform upstream updates this YAML file in their builds. Signed-off-by: Patrick Venture Change-Id: Ic2918f568f5a6f4a9f9135990889b3bb84a0c81d

entitymap: move accessor to separate module

2019-09-17T01:44:03+00:00

Signed-off-by: Patrick Venture Change-Id: I8c80acb694d067043b77d81dbb9a7ba0057fef56

fix logic error for unpack vector of tuple

2019-09-16T20:12:11+00:00

Unpacking a vector of tuples is failing if the correct number of bytes does not match an integral number of bytes needed to fully unpack all the tuples. Unpacking a tuple should return an error if it does not fully unpack all the items. This will signal the vector unpack to bail and return however many items it has unpacked to that point. A vector unpack should always return success because no matter how many items it has unpacked, it is fine, because a vector can have any number of items. Tested: Unit tests updated to check for proper unpacking of vectors and tuples (and optionals) as well as new unit tests added for more targetted testing. Change-Id: I4b45198f8bc4a49913beb923d10079983179402a Signed-off-by: Vernon Mauery

entitymap: add json parsing support

2019-08-20T15:55:01+00:00

Add a json parsing method that given an entity map in the format: [ { "id" : 1, "containerEntityId" : 2, "containerEntityInstance" : 3, "isList" : false, "isLinked" : false, "entities" : [ {"id" : 1, "instance" : 2}, {"id" : 1, "instance" : 3}, {"id" : 1, "instance" : 4}, {"id" : 1, "instance" : 5} ] } ] is constructed into the entity map used by sensorhandler. This is meant as part of the transition from the entity map in YAML to JSON. This is step 2. Step 1 moved access to the object behind a method. This adds JSON validation and parsing. Step 3 will add a file path to check and parse. Step 4 will provide a call to parse that file if present and use its data if non-empty. Tested: The method added has not been tested beyond unit-test validation. Signed-off-by: Patrick Venture Change-Id: Ic29f022d3812fa9e3af775d542ad055629fd5a01

message: Support prepending payloads

2019-04-29T19:06:35+00:00

Needed solely so that we can support adding the necessary bits for oem and group commands. We shouldn't be using this anywhere else, which is why it is not generalized to non-payload types. Change-Id: I6573f981fbe68cebb89abcdfb3de5de5d139e1e0 Signed-off-by: William A. Kennington III

message/pack: Allow packing payloads

2019-04-29T19:06:35+00:00

Some IPMI handlers need the ability to support variable return types. The easiest way to do that is to be able to return payloads and pack them into the final payload. Change-Id: I5098a1ab0998ada712096929eae40a3c88a6dea0 Signed-off-by: William A. Kennington III

message/pack: Support packing string_views

2019-04-29T19:06:35+00:00

Now we don't need to make an intermediate copy of our data into an array before packing it into a payload. Change-Id: Iac79a79e0ae95835cb67d617a966a92ce8dcd5f8 Signed-off-by: William A. Kennington III

message/pack: Check for outstanding bits

2019-04-29T19:06:35+00:00

Currently, if you pack a non-byte aligned member into a message and then pack an array, it will do the wrong thing and treat the unaligned data as being appended to the end of the message. Allowing for this behavior is convoluted and probably not useful, so just return an error. Change-Id: I6f200dbea96c41f49a110ba7536ccfd37115d277 Signed-off-by: William A. Kennington III

message/payload: Ignore unchecked unpack warning during unwind

2019-04-29T19:06:35+00:00

If we are caught in an exception unwind we don't care that the payload is not be fully checked. Otherwise, this encourages the pattern of wrapping all handler code in try catch blocks that set trailingOk as it is not required to read the entire payload when throwing an exception. Change-Id: I35149eedd33bd9fd41968e89d5a8614df7436872 Signed-off-by: William A. Kennington III