)]}'
{".zuul.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a43e9a1ebffea04f071ee4280b3543b6c8e53e95","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        TEMPEST_RUN_VALIDATION: false"},{"line_number":25,"context_line":"      tox_envlist: all"},{"line_number":26,"context_line":"      tempest_test_regex: |"},{"line_number":27,"context_line":"              (^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume)|(^cinder_tempest_plugin))"},{"line_number":28,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/devstack-plugin-ceph\"].src_dir }}/tempest_blacklist.txt\u0027"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_65a07c91","line":27,"updated":"2019-06-26 21:41:10.000000000","message":"I think we could make this change and the one below separately from the nova fix since the configuration in tempest.conf should handle skipping these scenario tests. Then we only need to manage the setting in one place in the plugin.","commit_id":"0466d3cafd51d59c4d4f87e933366438b5fb25a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0711245868bfe9049c9deacbd4b3583845c9fec8","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        TEMPEST_RUN_VALIDATION: false"},{"line_number":25,"context_line":"      tox_envlist: all"},{"line_number":26,"context_line":"      tempest_test_regex: |"},{"line_number":27,"context_line":"              (^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume)|(^cinder_tempest_plugin))"},{"line_number":28,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/devstack-plugin-ceph\"].src_dir }}/tempest_blacklist.txt\u0027"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_e1d9a582","line":27,"in_reply_to":"9fb8cfa7_41867168","updated":"2019-06-28 01:41:47.000000000","message":"Actually this is wrong, I thought this was the blacklist but it\u0027s the whitelist, so it\u0027s an enabled test but then gets skipped because of the tempest config in plugin.sh.\n\nWe can abandon this part of the series so we still run the scenario test with shelve enabled on master.","commit_id":"0466d3cafd51d59c4d4f87e933366438b5fb25a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"55289b4690842656fd5e8fefea5ecb2d86658495","unresolved":false,"context_lines":[{"line_number":24,"context_line":"        TEMPEST_RUN_VALIDATION: false"},{"line_number":25,"context_line":"      tox_envlist: all"},{"line_number":26,"context_line":"      tempest_test_regex: |"},{"line_number":27,"context_line":"              (^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume)|(^cinder_tempest_plugin))"},{"line_number":28,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/devstack-plugin-ceph\"].src_dir }}/tempest_blacklist.txt\u0027"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_41867168","line":27,"in_reply_to":"9fb8cfa7_65a07c91","updated":"2019-06-28 01:37:59.000000000","message":"Done","commit_id":"0466d3cafd51d59c4d4f87e933366438b5fb25a9"}],"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"640ba0b2ec8b43303142eb7cd9084aaab07bb9d1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP/DNM tempest: re-enable shelve tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Depends-On: If3c9d1de3ce0fe394405bd1e1f0fa08ce2baeda8"},{"line_number":10,"context_line":"Change-Id: Ifbfc15033020c430a93574f2726b2279c9be130d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fb8cfa7_73291535","line":9,"updated":"2019-06-27 14:06:07.000000000","message":"Before I forget, since enabling the tests is going to be per-branch, I think we likely want to have the devstack-plugin-ceph changes lock-step with the nova changes so this would be a URL per branch that we backport the change to (so this would change to the gerrit review URL for the change on master).","commit_id":"0466d3cafd51d59c4d4f87e933366438b5fb25a9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"55289b4690842656fd5e8fefea5ecb2d86658495","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP/DNM tempest: re-enable shelve tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Depends-On: If3c9d1de3ce0fe394405bd1e1f0fa08ce2baeda8"},{"line_number":10,"context_line":"Change-Id: Ifbfc15033020c430a93574f2726b2279c9be130d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9fb8cfa7_a1b16d46","line":9,"in_reply_to":"9fb8cfa7_73291535","updated":"2019-06-28 01:37:59.000000000","message":"Done","commit_id":"0466d3cafd51d59c4d4f87e933366438b5fb25a9"}],"devstack/plugin.sh":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"a43e9a1ebffea04f071ee4280b3543b6c8e53e95","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        # \"After unshelve, shelved image is not deleted\""},{"line_number":103,"context_line":"        # failure.  Re-enable this feature when that test is fixed."},{"line_number":104,"context_line":"        # https://review.openstack.org/#/c/471352/"},{"line_number":105,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled shelve False"},{"line_number":106,"context_line":"    fi"},{"line_number":107,"context_line":"fi"},{"line_number":108,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"9fb8cfa7_c5b848c9","side":"PARENT","line":105,"updated":"2019-06-26 21:41:10.000000000","message":"We\u0027re going to have to make this branch specific until the nova fix is on stable branches (this plugin is branchless so if we land this it\u0027s going to break the job on whatever branch doesn\u0027t have the nova fix).","commit_id":"4a4481995303d97b5f646c6d3d008fbb3eab554e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"55289b4690842656fd5e8fefea5ecb2d86658495","unresolved":false,"context_lines":[{"line_number":102,"context_line":"        # \"After unshelve, shelved image is not deleted\""},{"line_number":103,"context_line":"        # failure.  Re-enable this feature when that test is fixed."},{"line_number":104,"context_line":"        # https://review.openstack.org/#/c/471352/"},{"line_number":105,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled shelve False"},{"line_number":106,"context_line":"    fi"},{"line_number":107,"context_line":"fi"},{"line_number":108,"context_line":""}],"source_content_type":"text/x-sh","patch_set":3,"id":"9fb8cfa7_61893573","side":"PARENT","line":105,"in_reply_to":"9fb8cfa7_c5b848c9","updated":"2019-06-28 01:37:59.000000000","message":"Done","commit_id":"4a4481995303d97b5f646c6d3d008fbb3eab554e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b99f63751f8f90e46508b4a568ca6864ac379b8d","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # Only enable shelve testing for branches which have the fix for"},{"line_number":101,"context_line":"        # nova bug 1653953."},{"line_number":102,"context_line":"        if [[ \"$TARGET_BRANCH\" \u003d\u003d master ]]; then"},{"line_number":103,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve True"},{"line_number":104,"context_line":"        else"},{"line_number":105,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve True"}],"source_content_type":"text/x-sh","patch_set":5,"id":"9fb8cfa7_01cab99f","line":102,"updated":"2019-06-28 01:43:54.000000000","message":"Would probably be better to do something like blacklist the branches *without* the fix, so if branch is stable/(ocata|pike|queens|rocky|stein) then disable, and then we can lay patches on top with branch-specific dependent patches to remove each branch.","commit_id":"b56d9f1c3a07a6adf2c21d199942d29e9532dd88"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"83159441630290d9f41dc984304794f749f8c05c","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # Only enable shelve testing for branches which have the fix for"},{"line_number":101,"context_line":"        # nova bug 1653953."},{"line_number":102,"context_line":"        if [[ \"$TARGET_BRANCH\" \u003d~ stable/(ocata|pike|queens|rocky|stein) ]]; then"},{"line_number":103,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve False"},{"line_number":104,"context_line":"        else"},{"line_number":105,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve True"}],"source_content_type":"text/x-sh","patch_set":6,"id":"7faddb67_85cc158a","line":102,"range":{"start_line":102,"start_character":48,"end_line":102,"end_character":52},"updated":"2019-07-06 19:41:31.000000000","message":"This doesn\u0027t seem to be working for pike (maybe others):\n\nhttp://logs.openstack.org/38/669538/1/check/devstack-plugin-ceph-tempest/a1dc2a3/controller/logs/devstacklog.txt.gz#_2019-07-05_22_04_10_952\n\n2019-07-05 22:04:10.952 | ++ /opt/stack/devstack-plugin-ceph/devstack/plugin.sh:source:102 :   [[ \u0027\u0027 \u003d~ stable/(ocata|pike|queens) ]]\n\nLooks like TARGET_BRANCH isn\u0027t set for some reason?\n\nMaybe that\u0027s something due to how devstack on stable/pike works? Or zuul?","commit_id":"fdb1efbe0ad053b2939452403cbe4b847af2e684"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"118ee965523d44c060db7e2586d82aa230c15a65","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"        # Only enable shelve testing for branches which have the fix for"},{"line_number":101,"context_line":"        # nova bug 1653953."},{"line_number":102,"context_line":"        if [[ \"$TARGET_BRANCH\" \u003d~ stable/(ocata|pike|queens|rocky|stein) ]]; then"},{"line_number":103,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve False"},{"line_number":104,"context_line":"        else"},{"line_number":105,"context_line":"            iniset $TEMPEST_CONFIG compute-feature-enabled shelve True"}],"source_content_type":"text/x-sh","patch_set":6,"id":"7faddb67_e5a069a8","line":102,"range":{"start_line":102,"start_character":48,"end_line":102,"end_character":52},"in_reply_to":"7faddb67_85cc158a","updated":"2019-07-06 19:44:49.000000000","message":"https://bugs.launchpad.net/devstack-plugin-ceph/+bug/1835627","commit_id":"fdb1efbe0ad053b2939452403cbe4b847af2e684"}]}