)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ebe576c1c01b1a9e1e06b0cc3b0e3e566a383cd4","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit b59f458f2823bfcfd8a530003d34e912110c2f3a."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ia9273be6bf9fd55822dfe46f56503f70953c3ff3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_ba84295f","line":11,"updated":"2018-06-13 14:39:47.000000000","message":"This is why:\n\nhttps://github.com/openstack/nova/blob/dd87118acc2bf57f235cb287ed0ee12736263ecb/nova/compute/api.py#L1419","commit_id":"01471b0403deb96231277a6872a16fc0d8d42c26"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"49f5021def62421c1b4a0d4a0f95f8de4ecca62f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Furthermore, prior to Queens, the multi_instance_display_name_template"},{"line_number":15,"context_line":"config option could be used to customize the display name for"},{"line_number":16,"context_line":"multiple servers in a single request. Since Tempest is branchless,"},{"line_number":17,"context_line":"this means deployments older than Queens could now fail this test"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_0e2636cc","line":14,"range":{"start_line":14,"start_character":22,"end_line":14,"end_character":28},"updated":"2018-06-13 22:29:28.000000000","message":"This should be Rocky:\n\nhttps://review.openstack.org/#/c/499612/\n\nThe option was removed in Rocky, so anyone running tempest from master against ocata/pike/queens with this configured differently would fail this test. So this isn\u0027t really something tempest should be asserting until queens is in extended maintenance mode and tempest can baseline off rocky+.","commit_id":"fe002edfe3fa88ee4ca1917a7f4570b3fe719421"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"700073a65754be9f60e7cda1fde0b02cf0506e41","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Furthermore, prior to Queens, the multi_instance_display_name_template"},{"line_number":15,"context_line":"config option could be used to customize the display name for"},{"line_number":16,"context_line":"multiple servers in a single request. Since Tempest is branchless,"},{"line_number":17,"context_line":"this means deployments older than Queens could now fail this test"},{"line_number":18,"context_line":"if they are customing that option."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Really, the change to assert the server names shouldn\u0027t be possible"},{"line_number":21,"context_line":"until the nova pike series enters extended maintenance mode since that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_3a7254e4","line":18,"range":{"start_line":14,"start_character":0,"end_line":18,"end_character":34},"updated":"2018-06-13 18:44:49.000000000","message":"How about adding the corresponding option on Tempest side also and we will be able to the option after the EOL?","commit_id":"fe002edfe3fa88ee4ca1917a7f4570b3fe719421"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"59c9ea37ce4a997547ee8e5ba3d374d736c4447e","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Furthermore, prior to Queens, the multi_instance_display_name_template"},{"line_number":15,"context_line":"config option could be used to customize the display name for"},{"line_number":16,"context_line":"multiple servers in a single request. Since Tempest is branchless,"},{"line_number":17,"context_line":"this means deployments older than Queens could now fail this test"},{"line_number":18,"context_line":"if they are customing that option."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Really, the change to assert the server names shouldn\u0027t be possible"},{"line_number":21,"context_line":"until the nova pike series enters extended maintenance mode since that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_4ed8ae7d","line":18,"range":{"start_line":14,"start_character":0,"end_line":18,"end_character":34},"in_reply_to":"5f7c97a3_108979c8","updated":"2018-06-13 21:45:23.000000000","message":"s/customing/customizing","commit_id":"fe002edfe3fa88ee4ca1917a7f4570b3fe719421"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fb5b2cb3b74006e2af65585794d6fc5c7cb374b8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Furthermore, prior to Queens, the multi_instance_display_name_template"},{"line_number":15,"context_line":"config option could be used to customize the display name for"},{"line_number":16,"context_line":"multiple servers in a single request. Since Tempest is branchless,"},{"line_number":17,"context_line":"this means deployments older than Queens could now fail this test"},{"line_number":18,"context_line":"if they are customing that option."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Really, the change to assert the server names shouldn\u0027t be possible"},{"line_number":21,"context_line":"until the nova pike series enters extended maintenance mode since that"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5f7c97a3_108979c8","line":18,"range":{"start_line":14,"start_character":0,"end_line":18,"end_character":34},"in_reply_to":"5f7c97a3_3a7254e4","updated":"2018-06-13 20:28:31.000000000","message":"That seems excessive to me - is validation of multi-create server names blocking something else?\n\nAre you suggesting we add a compute feature option to tempest like \"default_multi_create_server_names\u003dTrue\" and then the nova-cells-v1 job has to set that to False?","commit_id":"fe002edfe3fa88ee4ca1917a7f4570b3fe719421"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"1d55d191e3220700f54b31c055a517d8e742da36","unresolved":false,"context_lines":[{"line_number":11,"context_line":"This breaks the nova-cells-v1 job since the cells v1 API"},{"line_number":12,"context_line":"does not honor the same multi-create instance name format."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Furthermore, prior to Rocky [1], the "},{"line_number":15,"context_line":"multi_instance_display_name_template config option could be used to"},{"line_number":16,"context_line":"customize the display name for multiple servers in a single request."},{"line_number":17,"context_line":"Since Tempest is branchless, this means deployments older than Rocky"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5f7c97a3_493c28be","line":14,"updated":"2018-06-13 22:36:04.000000000","message":"gah, trailing whitespace","commit_id":"5ab372d4df94acbaf190dd103c936eaa54ec339d"}]}