)]}'
{".zuul.yaml":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"da50e4d718f9f4a4f39f9f787a26c3bd4d9f9d27","unresolved":false,"context_lines":[{"line_number":303,"context_line":"        label: ubuntu-noble"},{"line_number":304,"context_line":"      - name: compute2"},{"line_number":305,"context_line":"        label: ubuntu-noble"},{"line_number":306,"context_line":"    groups:"},{"line_number":307,"context_line":"      # Node where tests are executed and test results collected"},{"line_number":308,"context_line":"      - name: tempest"},{"line_number":309,"context_line":"        nodes:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e2f675db_c8672544","line":306,"range":{"start_line":306,"start_character":4,"end_line":306,"end_character":10},"updated":"2025-11-25 17:44:00.000000000","message":"as an aside im thinking of using yaml anchors to clean up the duplication between the different topologies.\n\nevery time we defien a new not set with just the labes change i am reminded ot look into that but that out of scope of this reivew","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"da50e4d718f9f4a4f39f9f787a26c3bd4d9f9d27","unresolved":true,"context_lines":[{"line_number":308,"context_line":"      - name: tempest"},{"line_number":309,"context_line":"        nodes:"},{"line_number":310,"context_line":"          - controller"},{"line_number":311,"context_line":"      # Nodes running the compute service"},{"line_number":312,"context_line":"      - name: compute"},{"line_number":313,"context_line":"        nodes:"},{"line_number":314,"context_line":"          - compute1"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"594fa63b_5c5874cb","line":311,"updated":"2025-11-25 17:44:00.000000000","message":"you shoudl include the controller in the compute group as well\n\nto be consistent\n\nhttps://review.opendev.org/c/openstack/devstack/+/967310/5/.zuul.yaml#348\n\nthen in the job where your planning to use this disable the n-cpu service in the vars section\nand explicitly enable it in the subnode section.","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"3844f5f638037e96b351d9e59a1eacfcaa3cfb50","unresolved":true,"context_lines":[{"line_number":308,"context_line":"      - name: tempest"},{"line_number":309,"context_line":"        nodes:"},{"line_number":310,"context_line":"          - controller"},{"line_number":311,"context_line":"      # Nodes running the compute service"},{"line_number":312,"context_line":"      - name: compute"},{"line_number":313,"context_line":"        nodes:"},{"line_number":314,"context_line":"          - compute1"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"73b06a4f_579384f2","line":311,"in_reply_to":"594fa63b_5c5874cb","updated":"2025-12-01 11:19:38.000000000","message":"ok, restores controller into compute group","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6a86c9ff6ab61c832cc37f5f39159ae421ffb858","unresolved":false,"context_lines":[{"line_number":308,"context_line":"      - name: tempest"},{"line_number":309,"context_line":"        nodes:"},{"line_number":310,"context_line":"          - controller"},{"line_number":311,"context_line":"      # Nodes running the compute service"},{"line_number":312,"context_line":"      - name: compute"},{"line_number":313,"context_line":"        nodes:"},{"line_number":314,"context_line":"          - compute1"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"d26527d1_7747344d","line":311,"in_reply_to":"73b06a4f_579384f2","updated":"2025-12-02 12:48:54.000000000","message":"Done","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"91aa4b3261993ad8316c75665446efd46278d886","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fbd793a4_8df0600e","updated":"2025-11-17 10:03:15.000000000","message":"recheck","commit_id":"de4c0b59d187299803380bbfc072a5e2c33a87ad"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"eb68ac64cf1d1f225392e604b579c70b03ce5292","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"079032fe_7a1f4a00","updated":"2025-11-17 14:44:29.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"95ca36af8d0c2bb150c4f248c04f1c4f20d81283","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"139d9f71_32006d0c","updated":"2025-11-24 09:54:54.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"ba57e785f77e2ac4f0e6c6e8be09e21e4bd2d741","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3bf43f8c_e35e38fc","updated":"2025-11-18 09:17:13.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"98450e1972e8cf283acb896de99a1f0888f75857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"87dcfd52_de066e8e","updated":"2025-11-19 10:08:57.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"9b574088b6d7fcbac99b4e7ce3be2c4e48176ee0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"cd4aabef_1cca5468","updated":"2025-11-17 11:22:32.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"fc1e88399193555d18bf0deb8b0c538a0f3f3c9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ceae32a8_592d54b3","updated":"2025-11-21 11:30:22.000000000","message":"recheck","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"da50e4d718f9f4a4f39f9f787a26c3bd4d9f9d27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4215d8d9_120f6ce4","updated":"2025-11-25 17:44:00.000000000","message":"you can ignor rdo it does not have the ubuntu lables aviable.","commit_id":"74ba312671afe7cbac4c905bd0e401047662d745"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"2e9de7b2332da3ce0f5bbe5d6d7173a1f69089fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f70faf24_8f62eeb5","updated":"2025-12-19 12:51:17.000000000","message":"+1 based on results in: https://review.opendev.org/c/openstack/watcher/+/967331\nnote that this job in Watcher is running under experimental pipeline only, not in check/gate","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"be5094f83fbc2301d9286dac07c49f842c30b5ed","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4f2547a7_15806f2f","updated":"2025-12-19 13:15:24.000000000","message":"I\u0027m not sure that we should continue hosting seldomly used nodeset definitions in devstack. I would suggest you place the nodeset definition into watcher instead.","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"76f8168c06914c8edc61b4b2b4085f034cf72100","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"3def3fb2_aea293c7","updated":"2025-12-05 09:59:23.000000000","message":"check experimental","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"33b9b3c04c601c6ac208ce4325341afc5f7470a1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"72760ecc_a4ed9d04","updated":"2025-12-17 16:18:28.000000000","message":"https://review.opendev.org/c/openstack/watcher/+/967331 is now passing so upgrading to +2","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6a86c9ff6ab61c832cc37f5f39159ae421ffb858","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e4ffd1ad_83cfd5ed","updated":"2025-12-02 12:48:54.000000000","message":"lets see this run end to end and move \nhttps://review.opendev.org/c/openstack/watcher/+/967331 form DNM to a real patchs\n\nonce we have a version of that that is close to merge im ok with movign forward with the new node set.","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":34620,"name":"Francisco Seruca Salgado","email":"fserucas@redhat.com","username":"fserucas"},"change_message_id":"f8227eee20a3e8bed7e233ba9e8b1a5af0276a50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2ff5f56f_0449de40","updated":"2025-12-18 10:59:47.000000000","message":"recheck","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":34620,"name":"Francisco Seruca Salgado","email":"fserucas@redhat.com","username":"fserucas"},"change_message_id":"2d74a38d3bbb83b0c5993cc0eff0bbe586be3709","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"42282167_875c2344","updated":"2025-12-18 15:53:01.000000000","message":"recheck","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":34620,"name":"Francisco Seruca Salgado","email":"fserucas@redhat.com","username":"fserucas"},"change_message_id":"8223cb523a7885d4d328d7755d57ebba6f021623","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a0f69bf9_7bc6b18f","updated":"2025-12-18 14:37:25.000000000","message":"recheck","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":34620,"name":"Francisco Seruca Salgado","email":"fserucas@redhat.com","username":"fserucas"},"change_message_id":"78abd387dbe39616ec156b44500eaf2c8356e91c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a4deca68_a3c9ff0a","updated":"2025-12-18 14:28:55.000000000","message":"recheck","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":34620,"name":"Francisco Seruca Salgado","email":"fserucas@redhat.com","username":"fserucas"},"change_message_id":"a5db347acbf31ad21bd266dac7e10d9dc94be736","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a6512beb_d0005344","updated":"2025-12-19 08:25:32.000000000","message":"recheck","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"578d08e12b39c21e279f6a114b7b83803227384a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"90ca595f_8efa28cc","in_reply_to":"2058aa8c_0332421c","updated":"2026-01-09 09:34:23.000000000","message":"I consider the old nodeset definitions deprecated legacy, we cannot delete them because jobs are referencing them, but for me that does not transfer to creating new nodesets","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"2a0fa2675653cc8ead830fec5df603e99c75c591","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d4df4ff1_ca247a6b","in_reply_to":"3def3fb2_aea293c7","updated":"2025-12-05 10:00:09.000000000","message":"wrong repo","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"c85aededd3bccbd150b85f15ff0b1cd7cfdbc610","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2058aa8c_0332421c","in_reply_to":"4f2547a7_15806f2f","updated":"2026-01-09 09:28:14.000000000","message":"on this file, they are already created nodeset for theee nodes, but using old ubuntu versions, this review is just to add the same nodeset that we already have, but with the last ubuntu version, ubuntu-noble","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"71e553c4f0ecda98f7279e40ccd4b7da97436517","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ac2b89a4_9f603e57","in_reply_to":"90ca595f_8efa28cc","updated":"2026-01-09 12:48:39.000000000","message":"if that your prefence then sure i have no issue with having it in watcher.\n@dsanzmor@redhat.com can you factor this into the patch that consumes it and abandon this.","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":28647,"name":"David","display_name":"morenod","email":"dsanzmor@redhat.com","username":"morenod"},"change_message_id":"4529a9d7b1152d352050c488015c5e32f825f5f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6a9d6bbe_0cad9708","in_reply_to":"e4ffd1ad_83cfd5ed","updated":"2025-12-10 10:37:26.000000000","message":"after 4 consecutive successful CI runs on https://review.opendev.org/c/openstack/watcher/+/967331, we can consider it valid","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"315e92263592e5e057c8ce1635e551141ee56284","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6e9dfaea_8c4f4c96","in_reply_to":"f70faf24_8f62eeb5","updated":"2026-01-09 12:49:54.000000000","message":"its also in the periodic weekly pipelien","commit_id":"b17bbff149c5bd32c55b9d21d7b40d9a8b1301bf"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7d8ed8eb59d5f55eb6e5efcb4389a8640ab3d5a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"59f550ff_b211ac15","updated":"2026-01-09 17:53:34.000000000","message":"I think it make sense to define it on watcher side as it is only needed there for now. If it is needed by other also then we can think of moving it in devstack.","commit_id":"ae70029ee765eb93cd3ca3750c768144dd31bb55"}]}