Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PreTag openbmc/skeleton#43 - Set Less Than Minimum PowerCAP #97

Closed
gkeishin opened this issue May 31, 2016 · 2 comments
Closed

PreTag openbmc/skeleton#43 - Set Less Than Minimum PowerCAP #97

gkeishin opened this issue May 31, 2016 · 2 comments

Comments

@gkeishin
Copy link
Member

Please refer to mkumatag/openbmc-automation#61
for detail description...

Our concern was if this is some sort of REST server code bug returning Error even when that PowerCap property existing on the system..

If by design ( #43 ) if itis not going to return HTTP OK ... then, why is this return HTTP OK here on Palmetto and throwing error on Barrelleye when PowerCap Property exist on both the systems..

Example:

On Palmetto :
openbmc-automation$ curl -c cjar -b cjar -k -H "Content-Type: application/json" -X POST -d "{"data": [ 1900 ]}" http://9.3.164.177/org/openbmc/sensors/host/PowerCap/action/setValue
{
"data": null,
"message": "200 OK",
"status": "ok"
}

On Barrelleye:
openbmc-automation$ curl -c cjar -b cjar -k -H "Content-Type: application/json" -X POST-d "{"data": [ 1900 ]}" https://9.3.23.24/org/openbmc/sensors/host/PowerCap/action/setValue
{
"data": {
"description": "org.openbmc.objectmapper.Error.NotFound: path or object not found: /org/openbmc/sensors/host/PowerCap"
},
"message": "404 Not Found",
"status": "error"
}

@gkeishin
Copy link
Member Author

gkeishin commented Sep 8, 2016

We are seeing the powercap setting errors now frequently.
Tests . Test Generic Conf
7 Test 4 pass 3 3 failed

out of 7 , 3 are failing

/org/openbmc/settings/host0, power_cap

  1. Set the power with string of characters (abcdefg)
  2. Set the power with greater then MAX_POWER_VALUE (1010)

/org/openbmc/settings/host0, boot_flags

  1. Set the boot flags with string ( 3ab56f )

We would like to know what are the valid policies and what are not from those attributes..

@williamspatrick
Copy link
Member

Issue moved to openbmc/openbmc #557 via ZenHub

JasonLuo701 pushed a commit to JasonLuo701/skeleton that referenced this issue May 19, 2017
…erty()

free the error and message of sd bus for set_dbus_property()

Change-Id: I974e41ceaf9464374c1b93fd1b9d4432e66b049e
Signed-off-by: Doyle Huang <doyle.sy.huang@mail.foxconn.com>

Related work items: openbmc#97
JasonLuo701 pushed a commit to JasonLuo701/skeleton that referenced this issue Jul 17, 2017
feature 97: BMC Memory utilization of BMC Health sensor

Change-Id: Ie37269ebb391aafb12204b27e75c2d65ed24508c
Signed-off-by: Alex Wu <alex.jd.wu@mail.foxconn.com>

Related work items: openbmc#97
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants