)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"25d105e4356be5ff004f837e02e463a09bff3b3a","unresolved":true,"context_lines":[{"line_number":7,"context_line":"nova: Use noVNC 1.3.0 when installing from source"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Additionally make the repo name lowercase to match the project name in"},{"line_number":10,"context_line":"our zuul config so that jobs can check the repo out."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic2d9c4fa837461bbc29e067a81912b5f72efd3ca"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ec939658_19b41a48","line":10,"updated":"2021-11-05 18:40:31.000000000","message":"Just a curiosity question, but does this mean it would have worked mixed case if the .zuul.yaml specified it as \"novnc/noVNC\"?","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"70d38c4b1d715decb6a8073772753fe5e41ed1d9","unresolved":true,"context_lines":[{"line_number":7,"context_line":"nova: Use noVNC 1.3.0 when installing from source"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Additionally make the repo name lowercase to match the project name in"},{"line_number":10,"context_line":"our zuul config so that jobs can check the repo out."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic2d9c4fa837461bbc29e067a81912b5f72efd3ca"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"44beab07_6e9e6f29","line":10,"in_reply_to":"b8d7e4f0_9b3d3178","updated":"2021-11-05 19:55:00.000000000","message":"Oh, got it, thanks for the pointer!","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"126b080bdb8d6790bb499e5de31573ccbd201480","unresolved":true,"context_lines":[{"line_number":7,"context_line":"nova: Use noVNC 1.3.0 when installing from source"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Additionally make the repo name lowercase to match the project name in"},{"line_number":10,"context_line":"our zuul config so that jobs can check the repo out."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic2d9c4fa837461bbc29e067a81912b5f72efd3ca"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b8d7e4f0_9b3d3178","line":10,"in_reply_to":"ec939658_19b41a48","updated":"2021-11-05 19:23:56.000000000","message":"I don\u0027t think that would then match the project as defined in the zuul config which is ultimately in lowercase:\n\nhttps://github.com/openstack/project-config/blob/388d2277047706b465d0fce5839f03d77e34fb45/zuul/main.yaml#L1469\n\nI think that\u0027s the reason the repo is initially cloned into a lowercase directory and the reason devstack then couldn\u0027t find it when looking for the camelcase noVNC version.","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"70d38c4b1d715decb6a8073772753fe5e41ed1d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c8bed478_e7b74686","updated":"2021-11-05 19:55:00.000000000","message":"This is shown to work to pull the novnc source code in the following nova change that Depends-On this:\n\nhttps://review.opendev.org/c/openstack/nova/+/816738\n\nLGTM","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"803ed07aee1846ae18a15691c4fd120e3939675f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a3e21256_3dee373e","updated":"2021-12-03 18:27:48.000000000","message":"lgtm.","commit_id":"31334f9a9b2edbdc4a7b45e2e04aaec827639d62"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"2e0f0c88399c578fd2bed5e2e04572196bfdc3fa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8a6b8924_0de2a570","updated":"2021-12-03 16:27:01.000000000","message":"recheck platform failures look unrelated","commit_id":"31334f9a9b2edbdc4a7b45e2e04aaec827639d62"}],"lib/nova":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"ed113df5de70979ca04d48c3254604fccf1db671","unresolved":true,"context_lines":[{"line_number":831,"context_line":"            NOVNC_WEB_DIR\u003d/usr/share/novnc"},{"line_number":832,"context_line":"            install_package novnc"},{"line_number":833,"context_line":"        else"},{"line_number":834,"context_line":"            NOVNC_WEB_DIR\u003d$DEST/novnc"},{"line_number":835,"context_line":"            git_clone $NOVNC_REPO $NOVNC_WEB_DIR $NOVNC_BRANCH"},{"line_number":836,"context_line":"        fi"},{"line_number":837,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"7218add1_927928f5","line":834,"range":{"start_line":834,"start_character":32,"end_line":834,"end_character":37},"updated":"2021-11-05 19:24:37.000000000","message":"^ sorry to be clear this is why the following call to git_clone actually tried to clone the repo.","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"70d38c4b1d715decb6a8073772753fe5e41ed1d9","unresolved":true,"context_lines":[{"line_number":831,"context_line":"            NOVNC_WEB_DIR\u003d/usr/share/novnc"},{"line_number":832,"context_line":"            install_package novnc"},{"line_number":833,"context_line":"        else"},{"line_number":834,"context_line":"            NOVNC_WEB_DIR\u003d$DEST/novnc"},{"line_number":835,"context_line":"            git_clone $NOVNC_REPO $NOVNC_WEB_DIR $NOVNC_BRANCH"},{"line_number":836,"context_line":"        fi"},{"line_number":837,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"9f7d77d6_6c87ee33","line":834,"range":{"start_line":834,"start_character":32,"end_line":834,"end_character":37},"in_reply_to":"7218add1_927928f5","updated":"2021-11-05 19:55:00.000000000","message":"++","commit_id":"c8477875914ddb4761da3986553ca5eafe41d62b"}]}