)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"a28f7d00c93159aa4aaa276c9ded254eae8c541d","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a4591991_25d6f89a","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"updated":"2026-04-07 11:52:55.000000000","message":"This isn\u0027t true though, right? Selenium provides very similar functionality. I understand Playwright has an easier learning curve and is faster, meaning there are reasons to prefer it over Selenium, but we should at least note that here","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2d60f98075395b1071d831261e471792c91f9652","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"b8acb170_0f62deaf","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"3eeaf364_71b34ce6","updated":"2026-04-13 12:50:36.000000000","message":"maybe some discussion with the horizon team and other dashboard plugin maintainers would be good? adding this dependency only for a single plugin doesn\u0027t sound too convincing to me, if you could get others to join the boat and in the end possibly replace selenium completely, that would be quite a different story","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"79121d26f36e46a54181bfc1130961987a0585b9","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a4adcb1a_320c51e3","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"a4591991_25d6f89a","updated":"2026-04-07 12:06:43.000000000","message":"while it does its less reliable and has quite a lot of technical debt in comparison.\n\nmentioning that Selenium cna provide some of the same functionality is valid however. we evalcuated both last cycle and the developer expericne of actully wrinting the tests in playwrite vastly exceed any value we could have form contiuing to us selenium both in terms of reviewign the code and writing/debuging them form the POCs we did.\n\nhttps://review.opendev.org/c/openstack/watcher-dashboard/+/970353\nhttps://review.opendev.org/c/openstack/watcher-dashboard/+/976594\nvs \nhttps://review.opendev.org/c/openstack/watcher-dashboard/+/970573\n\nthe playwright test are still in early review but its clear its a net win on understandablity.","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6247a672fef0fce3b9bd44f655e59cd6aa2caaea","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"dc4a3dff_08c8bbc8","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"a4adcb1a_320c51e3","updated":"2026-04-07 15:07:22.000000000","message":"Yup, that\u0027s all fair. My point is simply that we should state that here, rather than fibbing and saying there\u0027s no alternative","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"800dd9d516c3c02e94140d24008bc8e9efe518a0","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"f8a131dc_a1dc2194","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"b8acb170_0f62deaf","updated":"2026-04-13 13:32:58.000000000","message":"i would love if we could replace selenium across the board but i didnt want to block watcher on converting horizon to playwright.\n\nform my perspective i would prefer not to have e2e test at all if we had to use selenium for it that just too much tech debt to me to maintain.\n\ni was proposing we use ti for grian-ui evenutlly.\n\nwe (mainly victoria) has started looking at grian-ui again to provide a\ntelemerty dashbaord on top of ateos/prometheus.\n\ni had hoped that could reuse playwright too.","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c0c2af00a42df0908c2366c06161a21dc386f4d9","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3eeaf364_71b34ce6","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"dc4a3dff_08c8bbc8","updated":"2026-04-08 12:38:14.000000000","message":"Thank you Sean and Stephen for point that out. I have included selenium here.","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"3b7ccb209a1b081d722e366b5fedcef4011cdb0e","unresolved":true,"context_lines":[{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    No."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* Is the library required for OpenStack project or related dev or"},{"line_number":36,"context_line":"  infrastructure setup? (Answer to this should be Yes, of course) Which?"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ac22e08c_582a4619","line":34,"range":{"start_line":33,"start_character":0,"end_line":34,"end_character":1},"in_reply_to":"f8a131dc_a1dc2194","updated":"2026-04-15 06:14:14.000000000","message":"Thank you Jens for review and comment and Thanks Sean for planning to adopt Playwright in grian-ui! \n\nI share your perspective that avoiding Selenium is preferable given the tech debt concerns, and Playwright offers a better path forward.                                   \nBased on the Watcher team\u0027s POC, Playwright appears easier for non-UI developers to automatically generate test code compared to Selenium. It helps developers to easily rewrite the code.\n\nI\u0027ve added Horizon reviewers for visibility, In case they want to reuse it in future. I hope we can build momentum for Playwright adoption starting with Watcher dashboard and grian-ui over time. \n\nAdding Playwright to requirements will allow watcher-dashboard to move forward. Thank you!","commit_id":"65686a3e402ae0eef6e6c701da30f71d6e3a5006"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"bdff7b10b6da3065819f2425c16171837b1c3ff5","unresolved":true,"context_lines":[{"line_number":25,"context_line":"* Is the library already packaged in the distros we target?"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Yes:"},{"line_number":28,"context_line":"        https://packages.debian.org/sid/python/python3-playwright"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"0030a561_03b0a8db","line":28,"updated":"2026-04-13 13:17:07.000000000","message":"I don\u0027t see it in the CentOS Stream package list: https://mirror.stream.centos.org/10-stream/BaseOS/x86_64/os/Packages/","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"800dd9d516c3c02e94140d24008bc8e9efe518a0","unresolved":true,"context_lines":[{"line_number":25,"context_line":"* Is the library already packaged in the distros we target?"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Yes:"},{"line_number":28,"context_line":"        https://packages.debian.org/sid/python/python3-playwright"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ad93a941_f6c8bc8a","line":28,"in_reply_to":"0030a561_03b0a8db","updated":"2026-04-13 13:32:58.000000000","message":"the intent was not to use it form disto packages for what its worth. just form pypi","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"80190ce64884ec0d669539d453cf7d54edfdea49","unresolved":true,"context_lines":[{"line_number":25,"context_line":"* Is the library already packaged in the distros we target?"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"    Yes:"},{"line_number":28,"context_line":"        https://packages.debian.org/sid/python/python3-playwright"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"* Is the function of this library already covered by other libraries"},{"line_number":31,"context_line":"  in global-requirements.txt?"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b7a9f664_54fcfa01","line":28,"in_reply_to":"ad93a941_f6c8bc8a","updated":"2026-04-14 12:07:58.000000000","message":"Yes, Add a note around that.","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"acb90167b985a1fa1dc6908f4cb38cc9ac25fe32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d0aea048_215714e6","updated":"2026-04-09 05:44:22.000000000","message":"recheck internal error in devstack functional job.","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e864024ba70c53838adc0140da44207edc4d5379","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"04b221ea_1d7f1e40","updated":"2026-04-17 09:11:32.000000000","message":"thx, voting +1 for now to indicate that I\u0027m not opposed to this, but I\u0027d prefer to see more feedback from others before approving","commit_id":"9f87a413ed30f96fa8d3c01e67472adddeea99cf"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"e3dc93e1779a54ecbf388656500998e19e09eedb","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":5,"id":"19a024c1_b700da3e","updated":"2026-04-27 15:21:08.000000000","message":"It\u0027s in merge conflict, but it seems playwright looks like a smaller learning curve for teams and if somebody starts fresh it\u0027s probably a better solution","commit_id":"b529f3204ce5ede5cf863a0e00af1c6f039d8f6a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"9a29cb559ad45c3625a0a67c89b812eacc4fc3e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"29d581fa_14b3e403","updated":"2026-04-27 15:27:05.000000000","message":"rebased now!","commit_id":"cffd7d85e3c9169b79ed5aaac2c9fe92745835e9"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"9795e3b86f3d5e3c476ebffa828e65c9d338178e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"589ce216_7c8635cb","updated":"2026-05-07 14:02:43.000000000","message":"As for today, Playwright seems to be a better tool for developers that want to write ui tests, and Watcher plans is to drop all Selemium related code to adopt Playwright within this cycle. I have the impression that more projects could benefit from Playwright as a starting point for ui testing. We would like to have the chance to be the first one.","commit_id":"2a92a998f9b65b43ee3f66c61d6094db69825bcf"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9602b89fe6affcd0e2ed88ab17688db86ea93de7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"3bff2168_06aabb93","updated":"2026-05-08 12:07:27.000000000","message":"This has a merge conflict that needs to be resolved.","commit_id":"2a92a998f9b65b43ee3f66c61d6094db69825bcf"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"afda19c6e6e51c0e75b597c5c3b74ec969f3ddc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"34b6b1d4_88a1de7e","in_reply_to":"3bff2168_06aabb93","updated":"2026-05-08 12:10:21.000000000","message":"Done, thank you!","commit_id":"2a92a998f9b65b43ee3f66c61d6094db69825bcf"}],"global-requirements.txt":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2d60f98075395b1071d831261e471792c91f9652","unresolved":true,"context_lines":[{"line_number":218,"context_line":"zstd  # BSD License (2 clause)"},{"line_number":219,"context_line":"zVMCloudConnector;sys_platform!\u003d\u0027win32\u0027  # Apache 2.0 License"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"## section:testing"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"bashate  # Apache-2.0"},{"line_number":224,"context_line":"coverage!\u003d4.4  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":3,"id":"d98b719f_0248f3a6","line":221,"updated":"2026-04-13 12:50:36.000000000","message":"iiuc the library would belong to this section, which is also where selenium is located?","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"80190ce64884ec0d669539d453cf7d54edfdea49","unresolved":false,"context_lines":[{"line_number":218,"context_line":"zstd  # BSD License (2 clause)"},{"line_number":219,"context_line":"zVMCloudConnector;sys_platform!\u003d\u0027win32\u0027  # Apache 2.0 License"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"## section:testing"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"bashate  # Apache-2.0"},{"line_number":224,"context_line":"coverage!\u003d4.4  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":3,"id":"88904a08_2f2abd24","line":221,"in_reply_to":"0e888118_c186fc03","updated":"2026-04-14 12:07:58.000000000","message":"Done","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"800dd9d516c3c02e94140d24008bc8e9efe518a0","unresolved":true,"context_lines":[{"line_number":218,"context_line":"zstd  # BSD License (2 clause)"},{"line_number":219,"context_line":"zVMCloudConnector;sys_platform!\u003d\u0027win32\u0027  # Apache 2.0 License"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"## section:testing"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"bashate  # Apache-2.0"},{"line_number":224,"context_line":"coverage!\u003d4.4  # Apache-2.0"}],"source_content_type":"text/plain","patch_set":3,"id":"0e888118_c186fc03","line":221,"in_reply_to":"d98b719f_0248f3a6","updated":"2026-04-13 13:32:58.000000000","message":"ya this sound more correct to me as well.","commit_id":"ea019940ab3222c41ca33a8e2f250d7b3c469831"}]}