)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7bdbd584c3311e5b230e0c261528519bb7a493eb","unresolved":false,"context_lines":[{"line_number":20,"context_line":"*-resp.json naming convention, and update the api-ref .inc files"},{"line_number":21,"context_line":"to use consistent headings (\"Example request:\" instead of \"Example"},{"line_number":22,"context_line":"post curl\") and add :language: javascript where it was missing."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"Closes-Bug: #2158147"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Assisted-by: Claude claude-opus-4-6 claude-code-2.1.190"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dccf1537_b36f8a5d","line":23,"updated":"2026-07-07 10:50:51.000000000","message":"yep this all looks good  to me thanks","commit_id":"e1db68f194ce9c9a47a99fe53ea7cecc4e240a30"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"7e5488804f0680d0c491dc8db5849618e6d9504d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5ea72153_5ccf9244","updated":"2026-06-29 12:25:29.000000000","message":"Looks good,\n\nIt is mow rendering json instead of curl https://e245814ca916ce959aec-2577efbf0e31bd3563af6b8ac46e8b56.ssl.cf2.rackcdn.com/openstack/1d4bc4cee5be484e8e9249ece2fb9fc6/docs/index.html?expanded\u003dcreate-accelerator-requests-detail#create-accelerator-requests\n\nbased on the fix.\n\nFrom my master checkout\n```\ngit grep \"curl.json\"\napi-ref/source/arqs.inc:.. literalinclude:: ../../doc/api_samples/accelerator_requests/accelerator_requests-post-curl.json\napi-ref/source/arqs.inc:.. literalinclude:: ../../doc/api_samples/accelerator_requests/accelerator_requests-patch-curl.json\napi-ref/source/attributes.inc:.. literalinclude:: ../../doc/api_samples/attributes/attributes-post-curl.json\napi-ref/source/device_profile.inc:.. literalinclude:: ../../doc/api_samples/device_profiles/device_profiles-post-curl.json\n```\nAll the above reference are updated with json file instead of curl.json.","commit_id":"e1db68f194ce9c9a47a99fe53ea7cecc4e240a30"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"675182322ef31f26d5256b6ba335008bafcd7d21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"66555dba_78c330dd","updated":"2026-07-06 14:24:58.000000000","message":"lgtm, thanks for fixing!","commit_id":"e1db68f194ce9c9a47a99fe53ea7cecc4e240a30"}],"doc/api_samples/attributes/attributes-post-req.json":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"ec1e0395d190fe2fc1934e40a1ffa472787a32b8","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"deployable_id\": \"1\","},{"line_number":3,"context_line":"    \"key\": \"rc\","},{"line_number":4,"context_line":"    \"value\": \"FPGA\""},{"line_number":5,"context_line":"}"}],"source_content_type":"application/json","patch_set":2,"id":"996594eb_30b27317","line":2,"updated":"2026-06-24 20:01:06.000000000","message":"deployable_id is a string \"1\" but the API model declares wtypes.IntegerType(). The corresponding response sample (attributes-create-resp.json) uses integer 1 for the same field. The request sample should match the declared type.\n\n**Severity**: WARNING | **Confidence**: 0.8\n\n**Impact**: Users copying this sample as a template may send a string where an integer is expected. While wsme may coerce the value at runtime, the documentation sample should demonstrate the correct type to avoid confusion and set a proper example for API consumers.\n\n**Suggestion**:\nChange \"deployable_id\": \"1\" to \"deployable_id\": 1 to match the wtypes.IntegerType() declaration in cyborg/api/controllers/v2/attributes.py:47 and the integer value used in the create response sample.","commit_id":"e1db68f194ce9c9a47a99fe53ea7cecc4e240a30"}]}
