{"action":"created","issue":{"url":"https://api.github.com/repos/xcat2/xcat-core/issues/4779","repository_url":"https://api.github.com/repos/xcat2/xcat-core","labels_url":"https://api.github.com/repos/xcat2/xcat-core/issues/4779/labels{/name}","comments_url":"https://api.github.com/repos/xcat2/xcat-core/issues/4779/comments","events_url":"https://api.github.com/repos/xcat2/xcat-core/issues/4779/events","html_url":"https://github.com/xcat2/xcat-core/pull/4779","id":293685081,"number":4779,"title":"Handle the PowerSupplyRedundancy Sensor API issue where Value is lower case","user":{"login":"whowutwut","id":10049070,"avatar_url":"https://avatars0.githubusercontent.com/u/10049070?v=4","gravatar_id":"","url":"https://api.github.com/users/whowutwut","html_url":"https://github.com/whowutwut","followers_url":"https://api.github.com/users/whowutwut/followers","following_url":"https://api.github.com/users/whowutwut/following{/other_user}","gists_url":"https://api.github.com/users/whowutwut/gists{/gist_id}","starred_url":"https://api.github.com/users/whowutwut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whowutwut/subscriptions","organizations_url":"https://api.github.com/users/whowutwut/orgs","repos_url":"https://api.github.com/users/whowutwut/repos","events_url":"https://api.github.com/users/whowutwut/events{/privacy}","received_events_url":"https://api.github.com/users/whowutwut/received_events","type":"User","site_admin":false},"labels":[{"id":550107851,"url":"https://api.github.com/repos/xcat2/xcat-core/labels/component:openbmc","name":"component:openbmc","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":{"url":"https://api.github.com/repos/xcat2/xcat-core/milestones/34","html_url":"https://github.com/xcat2/xcat-core/milestone/34","labels_url":"https://api.github.com/repos/xcat2/xcat-core/milestones/34/labels","id":2631881,"number":34,"title":"2.13.11","description":"","creator":{"login":"whowutwut","id":10049070,"avatar_url":"https://avatars0.githubusercontent.com/u/10049070?v=4","gravatar_id":"","url":"https://api.github.com/users/whowutwut","html_url":"https://github.com/whowutwut","followers_url":"https://api.github.com/users/whowutwut/followers","following_url":"https://api.github.com/users/whowutwut/following{/other_user}","gists_url":"https://api.github.com/users/whowutwut/gists{/gist_id}","starred_url":"https://api.github.com/users/whowutwut/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/whowutwut/subscriptions","organizations_url":"https://api.github.com/users/whowutwut/orgs","repos_url":"https://api.github.com/users/whowutwut/repos","events_url":"https://api.github.com/users/whowutwut/events{/privacy}","received_events_url":"https://api.github.com/users/whowutwut/received_events","type":"User","site_admin":false},"open_issues":91,"closed_issues":14,"state":"open","created_at":"2017-07-10T14:31:58Z","updated_at":"2018-02-01T21:12:03Z","due_on":"2018-03-09T08:00:00Z","closed_at":null},"comments":0,"created_at":"2018-02-01T21:11:52Z","updated_at":"2018-02-01T21:15:21Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/xcat2/xcat-core/pulls/4779","html_url":"https://github.com/xcat2/xcat-core/pull/4779","diff_url":"https://github.com/xcat2/xcat-core/pull/4779.diff","patch_url":"https://github.com/xcat2/xcat-core/pull/4779.patch"},"body":"Just by chance, I noticed that there was no output for `rvitals` for the new field PowerSupplyRedundancy was blank\r\n\r\n```\r\n[root@briggs01 Sensors]# rvitals mid05tor12cn05 | grep PowerSupply\r\nmid05tor12cn05: PowerSupplyRedundancy:\r\n```\r\n\r\nInvestigating this, it looks like the API returns the following data: \r\n```\r\n{\r\n  \"data\": {\r\n    \"/xyz/openbmc_project/sensors/chassis/PowerSupplyRedundancy\": {\r\n      \"error\": 0,\r\n      \"units\": \"\",\r\n      \"value\": \"Enabled\"\r\n    },\r\n...\r\n    \"/xyz/openbmc_project/sensors/fan_tach/fan0_0\": {\r\n      \"Scale\": 0,\r\n      \"Target\": 3500,\r\n      \"Unit\": \"xyz.openbmc_project.Sensor.Value.Unit.RPMS\",\r\n      \"Value\": 3218\r\n    },\r\n...\r\n```\r\n\r\nAll other fields are Upper case \"Value\" and this one is lower case \"Value\" \r\n\r\n```\r\n[root@briggs01 Sensors]# grep Value output | wc -l\r\n194\r\n[root@briggs01 Sensors]# grep value output | wc -l\r\n1\r\n```\r\n\r\nTo quickly fix it, I am handling the problem in xCAT code but FW team said they probably will not fix it for now..  This PR patches the problem\r\n\r\nUnit Test: \r\n\r\nBefore: \r\n```\r\n[root@briggs01 Sensors]# rvitals mid05tor12cn05 | grep PowerSupply\r\nmid05tor12cn05: PowerSupplyRedundancy:\r\n```\r\n\r\nAfter:\r\n```\r\n[root@briggs01 Sensors]# rvitals mid05tor12cn05 | grep PowerSupply\r\nmid05tor12cn05: PowerSupplyRedundancy: Enabled\r\n```\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/xcat2/xcat-core/issues/comments/362403965","html_url":"https://github.com/xcat2/xcat-core/pull/4779#issuecomment-362403965","issue_url":"https://api.github.com/repos/xcat2/xcat-core/issues/4779","id":362403965,"user":{"login":"xcatbot","id":29909372,"avatar_url":"https://avatars2.githubusercontent.com/u/29909372?v=4","gravatar_id":"","url":"https://api.github.com/users/xcatbot","html_url":"https://github.com/xcatbot","followers_url":"https://api.github.com/users/xcatbot/followers","following_url":"https://api.github.com/users/xcatbot/following{/other_user}","gists_url":"https://api.github.com/users/xcatbot/gists{/gist_id}","starred_url":"https://api.github.com/users/xcatbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/xcatbot/subscriptions","organizations_url":"https://api.github.com/users/xcatbot/orgs","repos_url":"https://api.github.com/users/xcatbot/repos","events_url":"https://api.github.com/users/xcatbot/events{/privacy}","received_events_url":"https://api.github.com/users/xcatbot/received_events","type":"User","site_admin":false},"created_at":"2018-02-01T21:15:21Z","updated_at":"2018-02-01T21:15:21Z","author_association":"NONE","body":"``CI CHECK RESULT`` : > **PR FORMAT CORRECT**"}}
{"actor":{"display_login":"xcatbot"}}