)]}'
{"cyborg/accelerator/drivers/pci/pci/sysinfo.py":[{"robot_id":"zuul","robot_run_id":"7090da07818b4ddeae42eff4bdc4e2f3","url":"https://zuul.teim.app/t/main/buildset/7090da07818b4ddeae42eff4bdc4e2f3","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":"7f0d38ad26bf037a92913776ae39c4175b315d3e","patch_set":1,"id":"478062bd_4581cca0","line":57,"updated":"2026-04-28 14:00:23.000000000","message":"Consider defining OWNER_CYBORG as a named constant in cyborg.common.constants rather than using a bare string literal.\n\n**Severity**: SUGGESTION | **Confidence**: 0.8\n\n**Benefit**: A named constant prevents typos and makes it easier to grep for all usages across the codebase. The NVIDIA GPU driver uses the same string literal.\n\n**Recommendation**:\nAdd OWNER_CYBORG \u003d \u0027OWNER_CYBORG\u0027 to cyborg/common/constants.py and import it in both pci/sysinfo.py and gpu/nvidia/sysinfo.py. This is a follow-up improvement, not a blocker for this patch.","commit_id":"5ea046df246188690d9e4ec9066ac5fb5a3ada4f"},{"robot_id":"zuul","robot_run_id":"cb48fd6118a44a00866504f8fe11aa0e","url":"https://zuul.teim.app/t/main/buildset/cb48fd6118a44a00866504f8fe11aa0e","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":"96d558893f1c88248ee59c88a15b5ad1dd428417","patch_set":2,"id":"e1b87a3b_874291e2","line":55,"updated":"2026-05-05 11:06:33.000000000","message":"The docstring examples in pci/sysinfo.py and nvidia/sysinfo.py still show the literal string \"OWNER_CYBORG\" rather than referencing the named constant, which is slightly misleading now that a constant exists.\n\n**Severity**: SUGGESTION | **Confidence**: 0.7\n\n**Benefit**: Keeping docstring examples consistent with the constant usage improves documentation accuracy and reduces confusion for future contributors.\n\n**Recommendation**:\nConsider updating the docstring examples in _get_traits() to reference the constant name, e.g. \u0027{traits:[constants.OWNER_CYBORG, \"CUSTOM_PCI_1EB8\"]}\u0027. This is a minor cosmetic point and not blocking.","commit_id":"4d8316e8e877ef0c59b049504f3a2196c30aeafa"}]}
