)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9adff1bf1945124e14c081774b63ec3d390415b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f21343b3_ce67ae31","updated":"2024-02-14 20:06:46.000000000","message":"I am not sure if this is right way to net compute node and I think doing it in devstack using service list API will be easy?","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"be6d6f595a3fe50e71f1db49a4384c718c82de63","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c34801fa_d2e27e75","in_reply_to":"318a112a_7c2299fc","updated":"2024-02-15 10:11:43.000000000","message":"we tried the devstack way here -  https://review.opendev.org/c/openstack/devstack/+/908331 - it found only one of the nodes, it could be because of many reasons like the scope that devstack code was executed in, the credentials etc it doesn\u0027t really matter though, the goal was to find out the hostnames and run the tests and that\u0027s what\u0027s done in this review .. so from my point of view the mission is accomplished.","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"944793f4e2448e0752aa514ea167b1a0b5a4b79e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"d384e061_b09fb533","in_reply_to":"51c277cd_bc64219e","updated":"2024-02-16 10:17:10.000000000","message":"do we have any statistics about devstack usage outside of opendev CI? I doubt that devstack is used by real openstack users - by real I mean users who run their production infrastructures on openstack. I see devstack as a lightweight openstack installer used in CI jobs and for deploying simple debug environments.","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"993f8f2863bdb4680ef646aa252bd84ea28be31c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"51c277cd_bc64219e","in_reply_to":"c34801fa_d2e27e75","updated":"2024-02-15 18:40:05.000000000","message":"Devstack should get that there is no reason why all node are not up when tempest is configured. Tempest is instaled/configure at the end when all services are up. If it does not then we might have seen issues in configuring the tempest. Also, doing it in playbook only help to run these in our CI and outside users cannot use it for example who just run devstack+tempest combination. \n\nAnyways, I am ok on how ever you would like to do, my main concern is that i can see test running in successful way and we can merge the test change.","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"75fa965dd28fd0f43c977b3213c1174bbcf556b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"318a112a_7c2299fc","in_reply_to":"f21343b3_ce67ae31","updated":"2024-02-15 10:08:57.000000000","message":"we tried doing it on devstack\u0027s side, didn\u0027t work, this does .. maybe it\u0027s just not easy for me .. anyway, as this works I don\u0027t want to spend any more time on finding a different solution.","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4a01049198fb30c9229dea3dad36265e05489cca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"488eb02b_f254f043","updated":"2024-02-28 19:22:30.000000000","message":"lgtm, migration hosts are set. thanks\n\n- https://zuul.opendev.org/t/openstack/build/eab824cdc926487aaed69054c0feb5ea/log/controller/logs/tempest_conf.txt#26-27\n\nand all new tests are running fine.\n\nhttps://zuul.opendev.org/t/openstack/build/eab824cdc926487aaed69054c0feb5ea/log/job-output.txt#34656-34668","commit_id":"450096d30ab2a2e0598c83aa8798f05f95d60169"}],"roles/set-src-dest-host/tasks/main.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9adff1bf1945124e14c081774b63ec3d390415b7","unresolved":true,"context_lines":[{"line_number":15,"context_line":"- name: Set migration_source_host in tempest.conf"},{"line_number":16,"context_line":"  become: true"},{"line_number":17,"context_line":"  community.general.ini_file:"},{"line_number":18,"context_line":"    path: \"{{ devstack_base_dir }}/tempest/etc/tempest.conf\""},{"line_number":19,"context_line":"    section: compute"},{"line_number":20,"context_line":"    option: migration_source_host"},{"line_number":21,"context_line":"    value: \"{{ devstack_hostnames[0] }}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Set migration_dest_host in tempest.conf"},{"line_number":24,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1c90be7e_de273113","line":21,"range":{"start_line":18,"start_character":0,"end_line":21,"end_character":40},"updated":"2024-02-14 20:06:46.000000000","message":"I am not sure if that will give the compute node always. is there any where it is fetched from nova or so?","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"75fa965dd28fd0f43c977b3213c1174bbcf556b6","unresolved":true,"context_lines":[{"line_number":15,"context_line":"- name: Set migration_source_host in tempest.conf"},{"line_number":16,"context_line":"  become: true"},{"line_number":17,"context_line":"  community.general.ini_file:"},{"line_number":18,"context_line":"    path: \"{{ devstack_base_dir }}/tempest/etc/tempest.conf\""},{"line_number":19,"context_line":"    section: compute"},{"line_number":20,"context_line":"    option: migration_source_host"},{"line_number":21,"context_line":"    value: \"{{ devstack_hostnames[0] }}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- name: Set migration_dest_host in tempest.conf"},{"line_number":24,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1cd2e84c_02a7af78","line":21,"range":{"start_line":18,"start_character":0,"end_line":21,"end_character":40},"in_reply_to":"1c90be7e_de273113","updated":"2024-02-15 10:08:57.000000000","message":"i got inspired by this role: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/emit-job-header/tasks/main.yaml\nAs this reads the inventory and doesn\u0027t ask openstack for nodes, I think it will be much more reliable - the inventory will always know about the nodes.","commit_id":"1c9dbb3942ff61a12761f9353c63e095c47778ff"}]}