)]}'
{".zuul.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"554309285641f585618329498513a009db63449a","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_f98dd629","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"updated":"2018-10-03 10:15:27.000000000","message":"this can be removed as, tempest-full will only run the multiattach tests for all branches. \n\nFor pike it will skip the tests as microversion cap on devstack side [1] and other stable branches it will run. \n\n[1] http://logs.openstack.org/78/606978/2/check/tempest-full-pike/424b1bb/job-output.txt.gz#_2018-10-01_22_56_16_884286","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a1445b90568c4ca8b0e2b3dbd2eaff8ef3a7cf35","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_b4e8c313","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"in_reply_to":"3f79a3b5_0760760b","updated":"2018-12-10 14:22:24.000000000","message":"OK, I\u0027m just not sure how tempest-full-queens is passing with the UCA used in queens.","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"502723e6feb5c2cedb56759192a6e7f0c21b409e","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_707c97e8","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"in_reply_to":"3f79a3b5_30c61f43","updated":"2018-12-07 19:40:55.000000000","message":"Testing it with a nova queens change that depends on this tempest patch:\n\nhttps://review.openstack.org/#/c/623568/","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7a26597c95bfbc97e814f937a0623a2e00e8e290","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_30c61f43","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"in_reply_to":"3f79a3b5_30ee1ffe","updated":"2018-12-07 19:32:14.000000000","message":"Wouldn\u0027t this mean that tempest-full, running against stable/queens nova changes, would run with ENABLE_VOLUME_MULTIATTACH: true? If so, if we don\u0027t disable the UCA, I think that job will fail on those branches. You don\u0027t see that in here because tempest is not running this against stable/queens, as it has its own tempest-full-queens job which is *not* enabling multiattach. So I\u0027m not sure how we get around that?\n\nIs there a way to set variables conditionally based on branch?\n\nOtherwise would the tempest-full job usage in stable/queens need to override the variables?","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"548238e5af58b82ed6c0751968a66f24f5d6322d","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_0760760b","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"in_reply_to":"3f79a3b5_707c97e8","updated":"2018-12-10 07:17:16.000000000","message":"queens job defintion of nova-multiattach job do disable the ENABLE_UBUNTU_CLOUD_ARCHIVE [1] \n\nand tempest-full-queens/rocky will get ENABLE_VOLUME_MULTIATTACH\u003dTrue as those jobs are derived from tempest-full (@L184) so by enabling multiattach in tempest-full can test multiattach tests on stable branches job on tempest master gate. We can see multiattach test running in tempest0full-queens [2]\n\n[1] https://github.com/openstack/nova/blob/stable/queens/playbooks/legacy/nova-multiattach/run.yaml#L33\n\n[2] http://logs.openstack.org/78/606978/4/check/tempest-full-queens/bdefafd/job-output.txt.gz#_2018-12-07_20_21_40_354532","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a1616416725745cfe2d08bd656b78aea214f27cd","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      tox_envlist: full"},{"line_number":54,"context_line":"      devstack_localrc:"},{"line_number":55,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":56,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":57,"context_line":"        # Queens and Rocky."},{"line_number":58,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_30ee1ffe","line":57,"range":{"start_line":56,"start_character":0,"end_line":57,"end_character":27},"in_reply_to":"3f79a3b5_f98dd629","updated":"2018-12-07 19:26:28.000000000","message":"Yes but as the note mentions, we already test multiattach using the nova-multiattach job in stable/rocky and stable/queens. I wasn\u0027t planning on making changes to drop the job from those stable branches as well, since the ubuntu cloud archive might be different which impacts whether or not we can test multiattach - at least for queens I think.","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"554309285641f585618329498513a009db63449a","unresolved":false,"context_lines":[{"line_number":164,"context_line":"      tox_envlist: slow-serial"},{"line_number":165,"context_line":"      devstack_localrc:"},{"line_number":166,"context_line":"        CINDER_ENABLED_BACKENDS: lvm:lvmdriver-1,lvm:lvmdriver-2"},{"line_number":167,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":168,"context_line":"        # Queens and Rocky."},{"line_number":169,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":170,"context_line":"      tempest_concurrency: 2"},{"line_number":171,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_d996da3d","line":168,"range":{"start_line":167,"start_character":0,"end_line":168,"end_character":27},"updated":"2018-10-03 10:15:27.000000000","message":"ditto.","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a1616416725745cfe2d08bd656b78aea214f27cd","unresolved":false,"context_lines":[{"line_number":164,"context_line":"      tox_envlist: slow-serial"},{"line_number":165,"context_line":"      devstack_localrc:"},{"line_number":166,"context_line":"        CINDER_ENABLED_BACKENDS: lvm:lvmdriver-1,lvm:lvmdriver-2"},{"line_number":167,"context_line":"        # NOTE(mriedem): The nova-multiattach job tests multiattach in"},{"line_number":168,"context_line":"        # Queens and Rocky."},{"line_number":169,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":170,"context_line":"      tempest_concurrency: 2"},{"line_number":171,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_d0f6cbd6","line":168,"range":{"start_line":167,"start_character":0,"end_line":168,"end_character":27},"in_reply_to":"3f79a3b5_d996da3d","updated":"2018-12-07 19:26:28.000000000","message":"same","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"554309285641f585618329498513a009db63449a","unresolved":false,"context_lines":[{"line_number":308,"context_line":"              - ^playbooks/"},{"line_number":309,"context_line":"              - ^roles/"},{"line_number":310,"context_line":"              - ^.zuul.yaml$"},{"line_number":311,"context_line":"        - nova-multiattach:"},{"line_number":312,"context_line":"            # NOTE(mriedem): Starting in Stein, the tempest-full and"},{"line_number":313,"context_line":"            # tempest-slow jobs run the volume multiattach tests."},{"line_number":314,"context_line":"            branches: ^stable/(queens|rocky).*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_d9fd7a6d","line":311,"range":{"start_line":311,"start_character":10,"end_line":311,"end_character":26},"updated":"2018-10-03 10:15:27.000000000","message":"this job itself we can remove from nova if we can make and run tempest-slow job on stable branches like tempest-full.\n\nthen all multiattach tests can be covered without nova-multiattach job on nova gate s well as on tempest.","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"548238e5af58b82ed6c0751968a66f24f5d6322d","unresolved":false,"context_lines":[{"line_number":308,"context_line":"              - ^playbooks/"},{"line_number":309,"context_line":"              - ^roles/"},{"line_number":310,"context_line":"              - ^.zuul.yaml$"},{"line_number":311,"context_line":"        - nova-multiattach:"},{"line_number":312,"context_line":"            # NOTE(mriedem): Starting in Stein, the tempest-full and"},{"line_number":313,"context_line":"            # tempest-slow jobs run the volume multiattach tests."},{"line_number":314,"context_line":"            branches: ^stable/(queens|rocky).*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_f297369a","line":311,"range":{"start_line":311,"start_character":10,"end_line":311,"end_character":26},"in_reply_to":"3f79a3b5_d0bd6bd2","updated":"2018-12-10 07:17:16.000000000","message":"True. tempest-full is all ok herefrom point of running it on stable branch and it will cover multiattach tests also on stable branches too as you mentioned. \n\nBut tempest-slow (tempest-multinode-full) job does not run on nova stable branches (even nova .zuul.yaml has that job to run for stable branch) which we can say bug on Tempest side because tempest-slow job definition is to run only on master [1]. I am trying to enable that for all stable branches[2]. I got this issue while checking multiattach slow test are running on stable branch as part of tempest-slow or not. \n\nI am getting few failure on tempest-slow (tempest-multinode-full) for stable branches which might take time to fix and till then let\u0027s keep nova-multiattach on stable branches. \n\n[1] https://github.com/openstack/tempest/blob/a32467c4c515dff325e6b4b5ce7af24a0b7a9961/.zuul.yaml#L270\n\n[2] https://review.openstack.org/#/q/topic:tempest-multinode-slow-stable+(status:open+OR+status:merged)","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a1616416725745cfe2d08bd656b78aea214f27cd","unresolved":false,"context_lines":[{"line_number":308,"context_line":"              - ^playbooks/"},{"line_number":309,"context_line":"              - ^roles/"},{"line_number":310,"context_line":"              - ^.zuul.yaml$"},{"line_number":311,"context_line":"        - nova-multiattach:"},{"line_number":312,"context_line":"            # NOTE(mriedem): Starting in Stein, the tempest-full and"},{"line_number":313,"context_line":"            # tempest-slow jobs run the volume multiattach tests."},{"line_number":314,"context_line":"            branches: ^stable/(queens|rocky).*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_d0bd6bd2","line":311,"range":{"start_line":311,"start_character":10,"end_line":311,"end_character":26},"in_reply_to":"3f79a3b5_d9fd7a6d","updated":"2018-12-07 19:26:28.000000000","message":"I am dropping the nova-multiattach job on master:\n\nhttps://review.openstack.org/#/c/606981/\n\nbut good point about tempest being branchless below, I guess I was thinking about the tempest rocky/queens/pike jobs which run against changes to tempest to make sure they work on stable branch versions of devstack.\n\nI\u0027m not really sure what you\u0027re saying about tempest-slow on stable though, I guess you\u0027re saying remove nova-multiattach from *all* branches and replace it with tempest-full/slow? The problem with that, as noted above, is that stable/queens devstack uses a different UCA (Pike):\n\nhttps://github.com/openstack-dev/devstack/blob/stable/queens/tools/fixup_stuff.sh#L80\n\nWhich is why the nova-multiattach job on stable/queens has to set ENABLE_VOLUME_MULTIATTACH\u003dTrue and disable the UCA:\n\nhttps://github.com/openstack/nova/blob/stable/queens/playbooks/legacy/nova-multiattach/run.yaml#L32\n\nThat\u0027s not a problem in stable/rocky though since we use the Queens UCA:\n\nIcb971831c8d4fe5f940d9e7993d53f1c3765e30f\n\nI did check nova\u0027s .zuul.yaml in queens and rocky and it looks like we do already run tempest-full and tempest-slow there, so we\u0027re covered for that.","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"554309285641f585618329498513a009db63449a","unresolved":false,"context_lines":[{"line_number":308,"context_line":"              - ^playbooks/"},{"line_number":309,"context_line":"              - ^roles/"},{"line_number":310,"context_line":"              - ^.zuul.yaml$"},{"line_number":311,"context_line":"        - nova-multiattach:"},{"line_number":312,"context_line":"            # NOTE(mriedem): Starting in Stein, the tempest-full and"},{"line_number":313,"context_line":"            # tempest-slow jobs run the volume multiattach tests."},{"line_number":314,"context_line":"            branches: ^stable/(queens|rocky).*$"},{"line_number":315,"context_line":"            # Define list of irrelevant files to use everywhere else"},{"line_number":316,"context_line":"            irrelevant-files: \u0026tempest-irrelevant-files"},{"line_number":317,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_de3800ee","line":314,"range":{"start_line":311,"start_character":0,"end_line":314,"end_character":47},"updated":"2018-10-03 10:15:27.000000000","message":"as there is no stable branches for tempest, this can be removed completely. tempest-full will cover everything (except ocata which is ok as there is no multiattach in ocata. )","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"554309285641f585618329498513a009db63449a","unresolved":false,"context_lines":[{"line_number":373,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":374,"context_line":"    gate:"},{"line_number":375,"context_line":"      jobs:"},{"line_number":376,"context_line":"        - nova-multiattach:"},{"line_number":377,"context_line":"            # NOTE(mriedem): Starting in Stein, the tempest-full and"},{"line_number":378,"context_line":"            # tempest-slow jobs run the volume multiattach tests."},{"line_number":379,"context_line":"            branches: ^stable/(queens|rocky).*$"},{"line_number":380,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":381,"context_line":"        - tempest-slow:"},{"line_number":382,"context_line":"            irrelevant-files: *tempest-irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_5e8870c1","line":379,"range":{"start_line":376,"start_character":0,"end_line":379,"end_character":47},"updated":"2018-10-03 10:15:27.000000000","message":"ditto","commit_id":"b25df5b2424fd33317971e11e7f27dac51c7d2e9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f9cee5bd9bb767f884abb7c6a952551f601718ac","unresolved":false,"context_lines":[{"line_number":99,"context_line":"      tox_envlist: full"},{"line_number":100,"context_line":"      devstack_localrc:"},{"line_number":101,"context_line":"        ENABLE_FILE_INJECTION: true"},{"line_number":102,"context_line":"        ENABLE_VOLUME_MULTIATTACH: true"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"- job:"},{"line_number":105,"context_line":"    name: tempest-full-oslo-master"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"9fdfeff1_7e9d0609","line":102,"updated":"2019-02-05 16:40:27.000000000","message":"Need to do this for tempest-full-py3 as well.","commit_id":"53255df27c2b91f12af53f263b462df5ec6fe77d"}],"tempest/api/compute/admin/test_volume_swap.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"feff66971633b824c8226c0d6f239dc87bea7172","unresolved":false,"context_lines":[{"line_number":156,"context_line":"    @decorators.skip_because(bug\u003d\u00271807723\u0027,"},{"line_number":157,"context_line":"                             condition\u003dCONF.compute.min_compute_nodes \u003e 1)"},{"line_number":158,"context_line":"    @utils.services(\u0027volume\u0027)"},{"line_number":159,"context_line":"    def test_volume_swap_with_multiattach(self):"},{"line_number":160,"context_line":"        # Create two volumes."},{"line_number":161,"context_line":"        # NOTE(gmann): Volumes are created before server creation so that"},{"line_number":162,"context_line":"        # volumes cleanup can happen successfully irrespective of which volume"}],"source_content_type":"text/x-python","patch_set":8,"id":"9fdfeff1_45586a55","line":159,"range":{"start_line":159,"start_character":8,"end_line":159,"end_character":41},"updated":"2019-02-06 02:53:49.000000000","message":"so this is test skipped in all jobs now because this is \u0027slow\u0027 test which skip it in tempest-full and tempest-slow job is multinode so the proposed condition will skip it in tempest-slow job.","commit_id":"7581e99a057c3ec5390cd26bcda48a4785d8d476"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e80af78f49edbf79f3c559740846b082cb90bf7a","unresolved":false,"context_lines":[{"line_number":156,"context_line":"    @decorators.skip_because(bug\u003d\u00271807723\u0027,"},{"line_number":157,"context_line":"                             condition\u003dCONF.compute.min_compute_nodes \u003e 1)"},{"line_number":158,"context_line":"    @utils.services(\u0027volume\u0027)"},{"line_number":159,"context_line":"    def test_volume_swap_with_multiattach(self):"},{"line_number":160,"context_line":"        # Create two volumes."},{"line_number":161,"context_line":"        # NOTE(gmann): Volumes are created before server creation so that"},{"line_number":162,"context_line":"        # volumes cleanup can happen successfully irrespective of which volume"}],"source_content_type":"text/x-python","patch_set":8,"id":"9fdfeff1_2d6f1ffd","line":159,"range":{"start_line":159,"start_character":8,"end_line":159,"end_character":41},"in_reply_to":"9fdfeff1_19764240","updated":"2019-02-08 04:42:19.000000000","message":"ok, i saw your proposal for nova-next to include slow test which make this test run there so lgtm. thanks","commit_id":"7581e99a057c3ec5390cd26bcda48a4785d8d476"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"e7cb8e125f1988834f52982a0919034170055662","unresolved":false,"context_lines":[{"line_number":156,"context_line":"    @decorators.skip_because(bug\u003d\u00271807723\u0027,"},{"line_number":157,"context_line":"                             condition\u003dCONF.compute.min_compute_nodes \u003e 1)"},{"line_number":158,"context_line":"    @utils.services(\u0027volume\u0027)"},{"line_number":159,"context_line":"    def test_volume_swap_with_multiattach(self):"},{"line_number":160,"context_line":"        # Create two volumes."},{"line_number":161,"context_line":"        # NOTE(gmann): Volumes are created before server creation so that"},{"line_number":162,"context_line":"        # volumes cleanup can happen successfully irrespective of which volume"}],"source_content_type":"text/x-python","patch_set":8,"id":"9fdfeff1_33809946","line":159,"range":{"start_line":159,"start_character":8,"end_line":159,"end_character":41},"in_reply_to":"9fdfeff1_2d6f1ffd","updated":"2019-02-12 22:53:13.000000000","message":"\u003e ok, i saw your proposal for nova-next to include slow test which\n \u003e make this test run there so lgtm. thanks\n\nDone: https://review.openstack.org/#/c/636459/","commit_id":"7581e99a057c3ec5390cd26bcda48a4785d8d476"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"807ee41f770a2df299db2970b78fd7a6383758bc","unresolved":false,"context_lines":[{"line_number":156,"context_line":"    @decorators.skip_because(bug\u003d\u00271807723\u0027,"},{"line_number":157,"context_line":"                             condition\u003dCONF.compute.min_compute_nodes \u003e 1)"},{"line_number":158,"context_line":"    @utils.services(\u0027volume\u0027)"},{"line_number":159,"context_line":"    def test_volume_swap_with_multiattach(self):"},{"line_number":160,"context_line":"        # Create two volumes."},{"line_number":161,"context_line":"        # NOTE(gmann): Volumes are created before server creation so that"},{"line_number":162,"context_line":"        # volumes cleanup can happen successfully irrespective of which volume"}],"source_content_type":"text/x-python","patch_set":8,"id":"9fdfeff1_19764240","line":159,"range":{"start_line":159,"start_character":8,"end_line":159,"end_character":41},"in_reply_to":"9fdfeff1_45586a55","updated":"2019-02-06 15:11:47.000000000","message":"Unless there are single-node jobs that also run slow tests. I haven\u0027t done a full audit to find out if there are. Anyway, as noted in the commit message, I think it\u0027s a low priority test. If we change the nova-next job to just run compute.api and scenario tests, we could maybe make that also run slow tests.","commit_id":"7581e99a057c3ec5390cd26bcda48a4785d8d476"}]}