)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1004,"name":"Mohammed Naser","email":"mnaser@vexxhost.com","username":"mnaser"},"change_message_id":"0559396ab2a6663cb668a6ab2bd99a02df4b1372","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c11b8f98_cfb402b4","updated":"2022-07-06 18:20:58.000000000","message":"Oleks:\n\nhttps://opendev.org/openstack/openstack-helm/src/branch/master/nova/templates/job-ks-service.yaml#L15-L18\n\nshould we be doing something like that?","commit_id":"43145c73264d31214dc7498c96b85424371755b9"},{"author":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"change_message_id":"c017786ef712f940bdb2d4a8329813901cc41eb0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0774fdeb_78d7b600","in_reply_to":"c11b8f98_cfb402b4","updated":"2022-07-07 06:56:26.000000000","message":"i fixed indent.\nBtw, it is not same like that because it is rendering plain manifests instead of using htk","commit_id":"43145c73264d31214dc7498c96b85424371755b9"},{"author":{"_account_id":31746,"name":"Oleksandr Kozachenko","email":"okozachenko1203@gmail.com","username":"okozachenko"},"change_message_id":"43970fb2eaa0d42ddce97a35774772945a75c7f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4c5e3fff_c3330433","updated":"2022-07-07 07:09:43.000000000","message":"Well, yeah, got your point to combine with https://review.opendev.org/c/openstack/openstack-helm/+/848831 and it should be because the same kind of small changes.\nBut I made separate patches because you know adding hook in bootstrap job is more sensitive because nova, neutron and libvirt have cross dependency. It means nova will not be ready alone.\n\nIt means hooking in bootstrap will be failed with timeout because other dependent charts are not deployed yet.\nIn CI, bootstrap is configured to wait till compute service ready https://opendev.org/openstack/openstack-helm/src/branch/master/tools/deployment/component/compute-kit/compute-kit.sh#L45.\n\nSo this patch will require to change that deploy script or maybe be declined. From this point, i want to keep it alone.","commit_id":"424e1b17b167d8ae2197b6bdf6e2bc1416586962"}]}