)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"d288974fb3a4fb7c271ade4fa37f2fc285da43d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5af7c3a6_9bf0b315","updated":"2026-07-30 12:40:17.000000000","message":"reviewing the generated doc","commit_id":"73f4932fc7003683aa04cbcb959c7d7f836e45fd"}],"doc/source/index.rst":[{"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":"52746fc22072444a39e61de516505486245e88c3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"53c64aad_17006c3c","line":5,"updated":"2026-07-30 12:58:06.000000000","message":"The commit message states that the introduction page is inlined into index.rst, but the Background Story, Transition to Cyborg Project, and Definition Breakdown sections from introduction.rst are absent from the new index.rst. Only a single one-line summary remains.\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: Users lose access to valuable background and architectural context about why Cyborg exists and what accelerators it manages. This is a content regression that contradicts the stated intent of the commit.\n\n**Suggestion**:\nEither inline the introduction.rst content (Background Story, Definition Breakdown sections) into the \u0027What is Cyborg?\u0027 section of index.rst, or add a note acknowledging the intentional removal with rationale.","commit_id":"73f4932fc7003683aa04cbcb959c7d7f836e45fd"},{"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":"52746fc22072444a39e61de516505486245e88c3","unresolved":false,"context_lines":[{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"0b745dde_a478dafe","line":5,"updated":"2026-07-30 12:58:06.000000000","message":"The commit message states that the usage page is inlined, but the content \u0027To use cyborg in a project: import cyborg\u0027 from usage.rst does not appear anywhere in the new index.rst.\n\n**Severity**: SUGGESTION | **Confidence**: 0.9\n\n**Benefit**: Minor content loss. The dropped snippet is trivial, but the commit\u0027s stated intent of inlining is not fully met.\n\n**Recommendation**:\nIf the import example is still relevant, add it to the new index.rst. If it is intentionally omitted, update the commit message to reflect that usage.rst content was dropped rather than inlined.","commit_id":"73f4932fc7003683aa04cbcb959c7d7f836e45fd"},{"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":"21d45d33a14bdfc4660bcaf0c38207de3a9437c6","unresolved":false,"context_lines":[{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."},{"line_number":8,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"b167622c_7c652cf4","line":5,"updated":"2026-07-30 14:43:57.000000000","message":"The commit message states that the introduction page would be inlined into index.rst, but the full content of doc/source/user/introduction.rst (Background Story, Transition to Cyborg Project, Definition Breakdown sections) was dropped entirely. Only a one-sentence summary was retained.\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: Users lose access to the project background, historical context, and conceptual definition breakdown. This is a meaningful documentation content loss that contradicts the patch\u0027s stated purpose of inlining (not removing) content.\n\n**Suggestion**:\nEither inline the full introduction.rst content (Background Story, Transition, Definition Breakdown) under the \u0027What is Cyborg?\u0027 section, or update the commit message to clarify that the introduction content was intentionally summarized/removed rather than inlined.","commit_id":"081fd5314f1b00d71c84298e43946aee57c4844f"},{"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":"21d45d33a14bdfc4660bcaf0c38207de3a9437c6","unresolved":false,"context_lines":[{"line_number":296,"context_line":""},{"line_number":297,"context_line":"Empty list \u003d cleanup complete, device available for reuse."},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"Tools for using Cyborg"},{"line_number":300,"context_line":"~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"Information on the commands available through Cyborg\u0027s Command Line"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7bba375e_132271a6","line":299,"updated":"2026-07-30 14:43:57.000000000","message":"The commit message states that the \u0027usage\u0027 page would be inlined into index.rst, but the content of doc/source/user/usage.rst (\u0027To use cyborg in a project: import cyborg\u0027) was dropped entirely with no equivalent content added to index.rst.\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: Minor content loss — the library usage snippet was brief. However, it does represent a discrepancy between the commit message\u0027s stated intent and the actual change.\n\n**Recommendation**:\nIf the library usage snippet was intentionally dropped (e.g., deemed out of scope for user docs), update the commit message to note the exclusion. Otherwise, add the usage snippet to the appropriate section of index.rst.","commit_id":"081fd5314f1b00d71c84298e43946aee57c4844f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f9aba4c79a94c09a87dfe3e74a6a989076ba7207","unresolved":true,"context_lines":[{"line_number":1,"context_line":"OpenStack Accelerator (Cyborg)"},{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"304ea7dd_a8d8a451","line":3,"updated":"2026-07-31 14:50:53.000000000","message":"note there is a some overlap between this and \n\nhttps://docs.openstack.org/cyborg/latest/admin\n\nthe https://docs.openstack.org/cyborg/latest/admin/#cyborg-and-nova-interaction-workflow is too low level for an admin facing doc and soudl move to its own refence doc","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"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":"e46bd001d3aa0a7d1d3c25626233c9898de66ede","unresolved":false,"context_lines":[{"line_number":1,"context_line":"OpenStack Accelerator (Cyborg)"},{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."}],"source_content_type":"text/x-rst","patch_set":3,"id":"ef04e084_acf6a6e6","line":4,"updated":"2026-07-31 07:23:48.000000000","message":"The commit message states it inlines the introduction page into index.rst, but the substantive content from introduction.rst was dropped entirely. Only the one-line summary \u0027Cyborg is a general management framework for accelerators.\u0027 was preserved. The Background Story (ETSI NFV, OPNFV DPACC, Nom...\n\n**Severity**: WARNING | **Confidence**: 0.9\n\n**Impact**: New users visiting the Cyborg documentation landing page lose context about what accelerators Cyborg manages (the Definition Breakdown listed concrete types like FPGA, GPU, NVMe SSD, dpdk/spdk, pmem), the project\u0027s origins, and its scope. This reduces the onboarding value of the documentation ind...\n\n**Suggestion**:\nEither inline the Background Story, Transition, and Definition Breakdown content from the old introduction.rst into the \u0027What is Cyborg?\u0027 section of index.rst (as the commit message intends), or explicitly note in the commit message that introduction.rst content was intentionally dropped and explain why.","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7c1562a76f035903697b492c74376345ceffe8b3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"OpenStack Accelerator (Cyborg)"},{"line_number":2,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"What is Cyborg?"},{"line_number":5,"context_line":"---------------"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."}],"source_content_type":"text/x-rst","patch_set":3,"id":"22822664_26a7a0c8","line":4,"in_reply_to":"ef04e084_acf6a6e6","updated":"2026-07-31 14:36:40.000000000","message":"this was intentional since the orginal content was mostly invalid or outdated\n\na detailed glossery of terms belongs elsewhere.","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8bf455a2f08b00685c3839d675b93b411b1fad64","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cyborg is a general management framework for accelerators."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Cyborg architecture"},{"line_number":10,"context_line":"~~~~~~~~~~~~~~~~~~~"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Cyborg design can be described by following diagram:"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ccced00b_aa1ced93","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":19},"updated":"2026-08-03 13:07:56.000000000","message":"I checked the rendered doc here: https://4bb663d7d7a47257f19c-81bbea0519e6e960b38d765ef17f047e.ssl.cf1.rackcdn.com/openstack/92f66dd594d344d9b3cef40f78df2fb1/docs/index.html#cyborg-architecture\n\ncyborg architecture and cyborg services docs are coming side by side.\n\nDocumentation for Operators section is also overlapping with cyborg architecure. The alignment needs to be fixed.","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"8bf455a2f08b00685c3839d675b93b411b1fad64","unresolved":true,"context_lines":[{"line_number":14,"context_line":".. image:: figures/cyborg-architecture.png"},{"line_number":15,"context_line":"    :width: 700 px"},{"line_number":16,"context_line":"    :scale: 99 %"},{"line_number":17,"context_line":"    :align: left"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"**cyborg-api** - cyborg-api is a cyborg service that provides **REST API**"},{"line_number":20,"context_line":"interface for the Cyborg project. It supports POST/PUT/DELETE/GET operations"}],"source_content_type":"text/x-rst","patch_set":3,"id":"e6a8d843_7f197dbd","line":17,"range":{"start_line":17,"start_character":12,"end_line":17,"end_character":16},"updated":"2026-08-03 13:07:56.000000000","message":"```suggestion\n    :align: center\n```\nThe architecture diagram will now render centered with the component descriptions flowing cleanly below it.","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"aa9066482c3ec75b071b1ad00cf846e7fd825e7f","unresolved":true,"context_lines":[{"line_number":80,"context_line":"As an end user of Cyborg, you\u0027ll use Cyborg to create and"},{"line_number":81,"context_line":"manage accelerators with either tools or the API directly."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Using Cyborg with Your Instance"},{"line_number":84,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"This guide shows you how to create OpenStack instances with accelerators"}],"source_content_type":"text/x-rst","patch_set":3,"id":"73432ec6_7e602ab0","line":83,"updated":"2026-07-31 12:58:19.000000000","message":"+1 for someone new to the project this is a lot more useful that the previous version","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"f9aba4c79a94c09a87dfe3e74a6a989076ba7207","unresolved":true,"context_lines":[{"line_number":80,"context_line":"As an end user of Cyborg, you\u0027ll use Cyborg to create and"},{"line_number":81,"context_line":"manage accelerators with either tools or the API directly."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Using Cyborg with Your Instance"},{"line_number":84,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"This guide shows you how to create OpenStack instances with accelerators"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3233351d_09816285","line":83,"in_reply_to":"22743d75_33819c54","updated":"2026-07-31 14:50:53.000000000","message":"the more i think about this the more i think this should not be inlined\n\nwe have to keep difnet persons in mind whn creating the doc and the usage docs shoudl be admin/usr focusn and not refence the devstack contbutor setup\n\n\nbuyt we might want to incopreate https://docs.openstack.org/cyborg/latest/admin/#user-requests","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"b1e77e915a12dd91b81cf93f4d8fcc7f0b615af2","unresolved":true,"context_lines":[{"line_number":80,"context_line":"As an end user of Cyborg, you\u0027ll use Cyborg to create and"},{"line_number":81,"context_line":"manage accelerators with either tools or the API directly."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Using Cyborg with Your Instance"},{"line_number":84,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"This guide shows you how to create OpenStack instances with accelerators"}],"source_content_type":"text/x-rst","patch_set":3,"id":"556eb140_b898dc23","line":83,"in_reply_to":"3233351d_09816285","updated":"2026-08-03 06:30:54.000000000","message":"To confirm, we want:\nUsing Cyborg with Your Instance - in its own doc, not inlined here.\nWhat is added here then?\nhttps://docs.openstack.org/cyborg/latest/admin/#user-requests should be incorporated here or where?","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7c1562a76f035903697b492c74376345ceffe8b3","unresolved":true,"context_lines":[{"line_number":80,"context_line":"As an end user of Cyborg, you\u0027ll use Cyborg to create and"},{"line_number":81,"context_line":"manage accelerators with either tools or the API directly."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"Using Cyborg with Your Instance"},{"line_number":84,"context_line":"~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"This guide shows you how to create OpenStack instances with accelerators"}],"source_content_type":"text/x-rst","patch_set":3,"id":"22743d75_33819c54","line":83,"in_reply_to":"73432ec6_7e602ab0","updated":"2026-07-31 14:36:40.000000000","message":"i tened to agree, i was excpecting to keep this as its own document however but i guess its fine to inline this as well","commit_id":"1546f2e542d20ac85a3531f676eebf6489b152fb"}]}