)]}'
{"os_traits/compute/owner.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4436741558606276277ad623fb2f38723c20acfc","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TRAITS \u003d ["},{"line_number":16,"context_line":"    # The resource provider owner is Cyborg."},{"line_number":17,"context_line":"    \u0027OWNER_CYBORG\u0027,"},{"line_number":18,"context_line":"    # The resource provider owner is Nova."},{"line_number":19,"context_line":"    \u0027OWNER_NOVA\u0027"},{"line_number":20,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"d3c47d5a_5cfe0f96","line":17,"range":{"start_line":17,"start_character":5,"end_line":17,"end_character":17},"updated":"2021-01-13 11:28:53.000000000","message":"I\u0027m pretty sure this should just read \u0027CYBORG\u0027, since os-traits will use the module info in the name. At current, this looks like\n\n  COMPUTE_OWNER_OWNER_CYBORG\n\nOn that note, is this really the best place for this to live? Wouldn\u0027t a separate \u0027resource\u0027 module make more sense, e.g. \u0027RESOURCE_OWNER_CYBORG\u0027? I assume adding native support for this to placement isn\u0027t viable also?","commit_id":"73ff47b95e11357db29621217a0949744e6b5066"},{"author":{"_account_id":24872,"name":"YumengBao","email":"yumeng_bao@yahoo.com","username":"Yumeng_Bao"},"change_message_id":"2b381603c25e3db23b654c74d9eb6f104db956de","unresolved":true,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"TRAITS \u003d ["},{"line_number":16,"context_line":"    # The resource provider owner is Cyborg."},{"line_number":17,"context_line":"    \u0027OWNER_CYBORG\u0027,"},{"line_number":18,"context_line":"    # The resource provider owner is Nova."},{"line_number":19,"context_line":"    \u0027OWNER_NOVA\u0027"},{"line_number":20,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"185cb996_77638e4d","line":17,"range":{"start_line":17,"start_character":5,"end_line":17,"end_character":17},"in_reply_to":"d3c47d5a_5cfe0f96","updated":"2021-01-14 07:58:32.000000000","message":"Thanks stephenfin! As discussed with nova team in PTG[0](line 150), we want it like: OWNER_CYBORG. So according to the traits formatting rule, we need to put this owner.py in the root path,right? (but not sure if this will break other os-traits principles) Do you have any suggestions?\n \n[0]https://etherpad.opendev.org/p/nova-wallaby-ptg","commit_id":"73ff47b95e11357db29621217a0949744e6b5066"}]}