)]}'
{".zuul.yaml":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2aa18d4e45c49f382b562812cf12228e51470e44","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        ENABLE_NFS_CINDER: true"},{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"    irrelevant-files: *gate-irrelevant-files"},{"line_number":73,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky|stein|train)).*$"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"ebad8706_c3fcfd56","line":72,"updated":"2021-02-22 22:22:56.000000000","message":"Why not adding \u0027tempest_test_regex\u0027[1]?\n\n[1] https://opendev.org/openstack/devstack-plugin-ceph/src/branch/master/.zuul.yaml#L88","commit_id":"82afcc9e705ce453911ff5a7e99a5eb2bf1bc3af"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"7c78291a86054cca26be0771f2fe82061082af7c","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        ENABLE_NFS_CINDER: true"},{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"    irrelevant-files: *gate-irrelevant-files"},{"line_number":73,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky|stein|train)).*$"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"dc900e5d_f00bb06c","line":72,"in_reply_to":"5fe914db_20995d7c","updated":"2022-03-24 14:12:02.000000000","message":"Done","commit_id":"82afcc9e705ce453911ff5a7e99a5eb2bf1bc3af"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"b9c21de72ec0da3c0e9c6367a0379b5e26f4d4ad","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        ENABLE_NFS_CINDER: true"},{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"    irrelevant-files: *gate-irrelevant-files"},{"line_number":73,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky|stein|train)).*$"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5fe914db_20995d7c","line":72,"in_reply_to":"ebad8706_c3fcfd56","updated":"2021-02-23 09:36:19.000000000","message":"My understanding is that \"full\" and \"full-parallel\" mean \"all tests\":\nhttps://opendev.org/openstack/tempest/src/commit/76f63460b372dd4ed4d45d2261665d07dde910aa/tox.ini#L117","commit_id":"82afcc9e705ce453911ff5a7e99a5eb2bf1bc3af"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5e6d0a1c4c49ff4c29349f4a254e62a1da585d5d","unresolved":true,"context_lines":[{"line_number":59,"context_line":"        * legacy-tempest-dsvm-full-devstack-plugin-nfs"},{"line_number":60,"context_line":"    timeout: 10800"},{"line_number":61,"context_line":"    required-projects:"},{"line_number":62,"context_line":"      - openstack/devstack-plugin-nfs"},{"line_number":63,"context_line":"      - openstack/cinder-tempest-plugin"},{"line_number":64,"context_line":"    vars:"},{"line_number":65,"context_line":"      devstack_plugins:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"4ac3af38_a2591ab2","line":62,"updated":"2022-07-21 23:31:01.000000000","message":"please keep the full namespace, with opendev.org/","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c14d796f3d66ad8bd8ca89f9509ce507a7bcc725","unresolved":false,"context_lines":[{"line_number":59,"context_line":"        * legacy-tempest-dsvm-full-devstack-plugin-nfs"},{"line_number":60,"context_line":"    timeout: 10800"},{"line_number":61,"context_line":"    required-projects:"},{"line_number":62,"context_line":"      - openstack/devstack-plugin-nfs"},{"line_number":63,"context_line":"      - openstack/cinder-tempest-plugin"},{"line_number":64,"context_line":"    vars:"},{"line_number":65,"context_line":"      devstack_plugins:"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"a9f45d64_1f625128","line":62,"in_reply_to":"4ac3af38_a2591ab2","updated":"2022-07-22 19:32:21.000000000","message":"Done","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5e6d0a1c4c49ff4c29349f4a254e62a1da585d5d","unresolved":true,"context_lines":[{"line_number":69,"context_line":"        ENABLE_NFS_CINDER: true"},{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"        TEMPEST_PLUGINS: \u0027/opt/stack/cinder-tempest-plugin\u0027"},{"line_number":73,"context_line":"      devstack_services:"},{"line_number":74,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":75,"context_line":"        horizon: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"c1d9da41_76dc017d","line":72,"updated":"2022-07-21 23:31:01.000000000","message":"please use\nvars:\n  ....\n  tempest_plugins:\n    - cinder-tempest-plugin","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c14d796f3d66ad8bd8ca89f9509ce507a7bcc725","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        ENABLE_NFS_CINDER: true"},{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"        TEMPEST_PLUGINS: \u0027/opt/stack/cinder-tempest-plugin\u0027"},{"line_number":73,"context_line":"      devstack_services:"},{"line_number":74,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":75,"context_line":"        horizon: false"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"12b9afa1_f9840260","line":72,"in_reply_to":"c1d9da41_76dc017d","updated":"2022-07-22 19:32:21.000000000","message":"Done","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5e6d0a1c4c49ff4c29349f4a254e62a1da585d5d","unresolved":true,"context_lines":[{"line_number":73,"context_line":"      devstack_services:"},{"line_number":74,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":75,"context_line":"        horizon: false"},{"line_number":76,"context_line":"      tox_envlist: all"},{"line_number":77,"context_line":"      tempest_test_regex: |"},{"line_number":78,"context_line":"        (^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume|scenario\\.test_shelve_instance)|(^cinder_tempest_plugin))"},{"line_number":79,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/devstack-plugin-nfs\"].src_dir }}/tempest_skiplist.txt\u0027"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"285dbaec_e4341ec7","line":76,"updated":"2022-07-21 23:31:01.000000000","message":"that\u0027s the default, no need to set it","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c14d796f3d66ad8bd8ca89f9509ce507a7bcc725","unresolved":false,"context_lines":[{"line_number":73,"context_line":"      devstack_services:"},{"line_number":74,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":75,"context_line":"        horizon: false"},{"line_number":76,"context_line":"      tox_envlist: all"},{"line_number":77,"context_line":"      tempest_test_regex: |"},{"line_number":78,"context_line":"        (^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume|scenario\\.test_shelve_instance)|(^cinder_tempest_plugin))"},{"line_number":79,"context_line":"      tempest_test_blacklist: \u0027{{ ansible_user_dir }}/{{ zuul.projects[\"opendev.org/openstack/devstack-plugin-nfs\"].src_dir }}/tempest_skiplist.txt\u0027"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"687676eb_499cea66","line":76,"in_reply_to":"285dbaec_e4341ec7","updated":"2022-07-22 19:32:21.000000000","message":"Done","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"0687ac853615ceeaba074e7833e199e13808d1b1","unresolved":true,"context_lines":[{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"      tempest_plugins:"},{"line_number":73,"context_line":"        - cinder_tempest_plugin"},{"line_number":74,"context_line":"      devstack_services:"},{"line_number":75,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":76,"context_line":"        horizon: false"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"3dd47b2b_c9b7789b","line":73,"range":{"start_line":73,"start_character":10,"end_line":73,"end_character":31},"updated":"2022-07-25 17:13:28.000000000","message":"I think this needs to be cinder-tempest-plugin instead (with dashes)\n\nFrom the job:\n    Hint: It looks like a path. File \u0027/opt/stack/cinder_tempest_plugin\u0027 does not exist.","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c65780a8dd0af306843a8718d0edbacf58bf6246","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        # enabled by default in tempest-full:"},{"line_number":71,"context_line":"        ENABLE_VOLUME_MULTIATTACH: false"},{"line_number":72,"context_line":"      tempest_plugins:"},{"line_number":73,"context_line":"        - cinder_tempest_plugin"},{"line_number":74,"context_line":"      devstack_services:"},{"line_number":75,"context_line":"        # Disable horizon (inherited from the parent, not needed by the tests executed here)"},{"line_number":76,"context_line":"        horizon: false"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"8e5eb362_20652269","line":73,"range":{"start_line":73,"start_character":10,"end_line":73,"end_character":31},"in_reply_to":"3dd47b2b_c9b7789b","updated":"2022-07-27 17:57:51.000000000","message":"Done","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"}],"/COMMIT_MSG":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5e6d0a1c4c49ff4c29349f4a254e62a1da585d5d","unresolved":true,"context_lines":[{"line_number":11,"context_line":"future problems with all associated nfs drivers."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Normally, \u0027full-parallel\u0027 includes slow scenario tests. However,"},{"line_number":14,"context_line":"use \u0027all\u0027 because \u0027full-parallel\u0027 does not allow me to set"},{"line_number":15,"context_line":"\u0027tempest_test_regex\u0027 and I need it to run \u0027cinder_tempest_plugin\u0027."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1a916a49814b2f3e37b3455e6c4a318e1b2d7a6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"d0648a42_af32bfba","line":14,"updated":"2022-07-21 23:31:01.000000000","message":"I don\u0027t think this is correct. tempest_test_regex can be set with all environments. Check the content of roles/run-tempest/tasks/main.yaml","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"ee43818e301ea0bdd2532e4cc4dd2ad4e2b73f63","unresolved":true,"context_lines":[{"line_number":11,"context_line":"future problems with all associated nfs drivers."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Normally, \u0027full-parallel\u0027 includes slow scenario tests. However,"},{"line_number":14,"context_line":"use \u0027all\u0027 because \u0027full-parallel\u0027 does not allow me to set"},{"line_number":15,"context_line":"\u0027tempest_test_regex\u0027 and I need it to run \u0027cinder_tempest_plugin\u0027."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1a916a49814b2f3e37b3455e6c4a318e1b2d7a6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"85672943_a734d11c","line":14,"in_reply_to":"1efedd7c_22fa6008","updated":"2022-07-22 15:28:01.000000000","message":"https://paste.opendev.org/show/bci6mGxyHiO3jL3PlREV/","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c65780a8dd0af306843a8718d0edbacf58bf6246","unresolved":false,"context_lines":[{"line_number":11,"context_line":"future problems with all associated nfs drivers."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Normally, \u0027full-parallel\u0027 includes slow scenario tests. However,"},{"line_number":14,"context_line":"use \u0027all\u0027 because \u0027full-parallel\u0027 does not allow me to set"},{"line_number":15,"context_line":"\u0027tempest_test_regex\u0027 and I need it to run \u0027cinder_tempest_plugin\u0027."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1a916a49814b2f3e37b3455e6c4a318e1b2d7a6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"64b6c99c_1b3160a3","line":14,"in_reply_to":"85672943_a734d11c","updated":"2022-07-27 17:57:51.000000000","message":"Done","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"d54fcf36413bd1477670b32fb773e1604719dea4","unresolved":true,"context_lines":[{"line_number":11,"context_line":"future problems with all associated nfs drivers."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Normally, \u0027full-parallel\u0027 includes slow scenario tests. However,"},{"line_number":14,"context_line":"use \u0027all\u0027 because \u0027full-parallel\u0027 does not allow me to set"},{"line_number":15,"context_line":"\u0027tempest_test_regex\u0027 and I need it to run \u0027cinder_tempest_plugin\u0027."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I1a916a49814b2f3e37b3455e6c4a318e1b2d7a6a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":16,"id":"1efedd7c_22fa6008","line":14,"in_reply_to":"d0648a42_af32bfba","updated":"2022-07-22 15:03:07.000000000","message":"full-parallel runs by default (just run the command `tox -e full-parallel`):\n```\ntempest run --regex \u0027(^tempest\\.scenario.*)|(?!.*\\[.*\\bslow\\b.*\\])(^tempest\\.api)\u0027\n```\n\nAs you can see here, it has a default regex. The problem is that when I specify a `tempest_test_regex`, it tries to run both:\n\n```\ntempest run --regex \u0027(^tempest\\.scenario.*)|(?!.*\\[.*\\bslow\\b.*\\])(^tempest\\.api)\u0027 \u0027(^tempest\\.(api|scenario\\.test_encrypted_cinder_volumes|scenario\\.test_volume|scenario\\.test_shelve_instance)|(^cinder_tempest_plugin))\n\u0027 --blacklist-file\u003d/home/zuul/src/opendev.org/openstack/devstack-plugin-nfs/tempest_skiplist.txt --concurrency\u003d4\n```\n\nThis results in a syntax error.\n\nAs you mentioned, it is always possible to define the regex [1], so I am not sure why this is happening. As far as I know, I can definitely define the regex with the test cases for encryption and the scenario with \u0027all\u0027.\n\nI\u0027ll update the commit msg.","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"67de5696316ac9748dc10e30c36f666ff37d0a4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ee60b465_dfab942e","updated":"2022-07-05 01:26:13.000000000","message":"I think I should remove stable branches since encryption would only be available on master","commit_id":"fb1fc5788607e51cb54f8c65e02c40d0859277eb"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2f577c67d38822c6c5ed7718b5d440604fde3d3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"b5d80380_1db1ab29","updated":"2022-07-07 19:41:01.000000000","message":"The NFS CI is not running the cinder-tempest-plugin 😐","commit_id":"f9c9ea9f61716b63f7c274628684bac4670ae398"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"3ba4e2d155ed21c20c40591b92ef5afcf8983311","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"0c57b072_ed58dad4","updated":"2022-07-21 22:50:12.000000000","message":"devstack-plugin-nfs-tempest-full still doesn\u0027t run encrypted test\n```\n{0} setUpClass (cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes) ... SKIPPED: Encrypted volume attach is not supported\n```","commit_id":"f75bdc8c28638e531e5a88ed998f32b53cca3d56"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"5e6d0a1c4c49ff4c29349f4a254e62a1da585d5d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"c945f725_53929ef8","updated":"2022-07-21 23:31:01.000000000","message":"I think this is starting to be a bit too complex than what it should be","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c65780a8dd0af306843a8718d0edbacf58bf6246","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"fd3952d8_bfa8fa3b","in_reply_to":"046c5512_e102bc12","updated":"2022-07-27 17:57:51.000000000","message":"Done","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"d54fcf36413bd1477670b32fb773e1604719dea4","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"046c5512_e102bc12","in_reply_to":"c945f725_53929ef8","updated":"2022-07-22 15:03:07.000000000","message":"I know... :( Please let me know if you have any suggestions to improve this. \n\nOn the one hand, the generic NFS encryption only supports LUKS1. So we have to omit both the cryptsetup and LUKS2 tests. \n\nOn the other hand, the encryption is falling so this job should fail in master, but not in stable branches.","commit_id":"677369df897b8ae185d408a1413e4d447015cdf6"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"7d6465c09ca5144d5589d753fae3e2de65a1a63e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"668fb863_93da8b7d","updated":"2022-07-25 15:30:45.000000000","message":"recheck","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b9bac528c9554ab18ced185d51a59a654ff1aa54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"e93a3122_59e64d18","updated":"2022-08-24 12:50:11.000000000","message":"```\n{0} cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes.test_boot_cloned_encrypted_volume [291.765127s] ... FAILED\n```\n\nfails with:\n\nFailed to establish authenticated ssh connection to cirros@172.24.5.127 (timed out). Number attempts: 1. Retry after 2 seconds.\n\nLooks like this is not related to this patch. ","commit_id":"d7c4ecd2665fc91e066fdbfe1c3b581529bd9488"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2a12a35f848bb7cd6c4b5a07a1c853a0e0faf4ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"79da067e_a251f2eb","updated":"2022-08-25 13:46:04.000000000","message":"recheck devstack-plugin-nfs-tempest-full ","commit_id":"d7c4ecd2665fc91e066fdbfe1c3b581529bd9488"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"35c69f4b75dd5b23fce5d7cdc282bbd99fd10188","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"c40b3e6f_462098b8","updated":"2022-09-01 21:29:40.000000000","message":"recheck","commit_id":"c378fa40d61a3f17b1c0998477387487ede7dca2"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"4559388f4d69a47a75349222a8f17963318c490a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"cb57da47_6e2aec46","updated":"2022-12-19 16:39:13.000000000","message":"recheck","commit_id":"c378fa40d61a3f17b1c0998477387487ede7dca2"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c804af545e2ea689046e3f8c8cc7eda48e98b56a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":28,"id":"42d33975_516d530a","updated":"2023-01-23 18:30:43.000000000","message":"Looks like NFS driver needs some code for clone operations \n\n```\ncinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes.test_boot_cloned_encrypted_volume[compute,id-5bb622ab-5060-48a8-8840-d589a548b7e4,image,volume]\n\n{0} cinder_tempest_plugin.scenario.test_volume_encrypted.TestEncryptedCinderVolumes.test_attach_cloned_encrypted_volume [209.945348s] ... FAILED\n```","commit_id":"a3c016fb808c4a3f0a77fd8ca84737a46b51f600"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"1db1e82b546d163e47a3d1313939a59b39522527","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"c887226c_5d858348","updated":"2023-01-20 10:55:38.000000000","message":"recheck","commit_id":"a3c016fb808c4a3f0a77fd8ca84737a46b51f600"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0b02c9591910627e15086ccad10736584a17193b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"ca8eaa85_78b96cc2","updated":"2023-01-20 16:15:29.000000000","message":"recheck devstack-plugin-nfs-tempest-full ","commit_id":"a3c016fb808c4a3f0a77fd8ca84737a46b51f600"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"d92895fe5c2cc2041d704eef4b1d7b2d28309341","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":28,"id":"4de5b127_bf8379b3","in_reply_to":"42d33975_516d530a","updated":"2023-02-13 15:05:57.000000000","message":"Done","commit_id":"a3c016fb808c4a3f0a77fd8ca84737a46b51f600"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2243182b022d9e3b309171c873b00b5fe6bc8b6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"6f799671_de234d9f","updated":"2023-02-07 16:22:15.000000000","message":"recheck","commit_id":"2af1dc4298b069d4cd7f246d56eb1d2d3112092e"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"e5fda97e6c19549e7c5bb5e62e32bb920eaa8a50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"7e23d9f6_8ccb0ac9","updated":"2023-02-09 17:54:43.000000000","message":"recheck","commit_id":"2af1dc4298b069d4cd7f246d56eb1d2d3112092e"}],"devstack/plugin.sh":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f491d8f798708df88f18149ade08180909f394a9","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ master ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert False"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume False"},{"line_number":87,"context_line":"    else"},{"line_number":88,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":89,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"}],"source_content_type":"text/x-sh","patch_set":18,"id":"b0a3c86a_f7e18c03","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":14},"updated":"2022-07-25 17:21:37.000000000","message":"This check should be inside of something like\n\n    if [[ \"$TARGET_BRANCH\" \u003d~ stable/(ocata|pike) ]]; then\n        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume False\n    else\n        .... True\n    fi\n    \n\nThis will prevent the test from being disabled for stable/zed once we are on the next release.","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"954818b2d350c42c3f3fff7fe325831a6bc5228c","unresolved":false,"context_lines":[{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ master ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert False"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume False"},{"line_number":87,"context_line":"    else"},{"line_number":88,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":89,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"}],"source_content_type":"text/x-sh","patch_set":18,"id":"d38b51d9_921f4d53","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":14},"in_reply_to":"1150e62b_f20134b3","updated":"2022-07-27 17:56:51.000000000","message":"Done","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"67661c2701c7569ce67780d6aae142a67c5e5169","unresolved":true,"context_lines":[{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ master ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert False"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume False"},{"line_number":87,"context_line":"    else"},{"line_number":88,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":89,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"}],"source_content_type":"text/x-sh","patch_set":18,"id":"1150e62b_f20134b3","line":86,"range":{"start_line":86,"start_character":8,"end_line":86,"end_character":14},"in_reply_to":"b0a3c86a_f7e18c03","updated":"2022-07-26 13:52:07.000000000","message":"Yes, better use that condition (inverting the settings), and it may indirectly fix the syntax error.\nMaybe we need to keep two conditions here. We can\u0027t can enable volume_revert and attach_encrypted_volume for the same releases, or can we?","commit_id":"d2e53ce5a704539d945dae29ab5ebfe02694230d"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"ceea798ce49aa9b8ba9350e551e1ea60307be27e","unresolved":true,"context_lines":[{"line_number":81,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled backup False"},{"line_number":82,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled clone True"},{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ stable/(xena|wallaby|victoria|ussuri) ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"},{"line_number":87,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":21,"id":"8cddcc31_35fa2631","line":84,"updated":"2022-07-27 09:18:11.000000000","message":"Let\u0027s not change the condition for enabling volume_revert. The version sets where they can be enabled may diverge.\n\nif [[ \"$TARGET_BRANCH\" \u003d~ master ]]; then\n    iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert False\nelse\n    iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True\nfi\n\nif [[ \"$TARGET_BRANCH\" \u003d~ stable/(ocata|pike|queens|rocky|stein|train|ussuri|victoria|wallaby|xena|yoga) ]]; then\n    iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume False\nelse\n    iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True\nfi","commit_id":"3d3720114992538ac7f02d581c2df791113251f1"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"da9a420d93430b139d21b0d82adff4544824f4a2","unresolved":true,"context_lines":[{"line_number":81,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled backup False"},{"line_number":82,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled clone True"},{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ stable/(xena|wallaby|victoria|ussuri) ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"},{"line_number":87,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":21,"id":"4a573f28_7a48ca79","line":84,"in_reply_to":"13f2671f_dcdf780c","updated":"2022-07-27 18:00:05.000000000","message":"```\n{3} cinder_tempest_plugin.api.volume.test_volume_revert.VolumeRevertTests.test_volume_revert_to_snapshot [6.933982s] ... FAILED\n```","commit_id":"3d3720114992538ac7f02d581c2df791113251f1"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"95c9772058931041b7cd8c0ba51add5d023273a5","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled backup False"},{"line_number":82,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled clone True"},{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ stable/(xena|wallaby|victoria|ussuri) ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"},{"line_number":87,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":21,"id":"1bad7530_3b4f9ffb","line":84,"in_reply_to":"4a573f28_7a48ca79","updated":"2022-09-01 21:11:35.000000000","message":"Done","commit_id":"3d3720114992538ac7f02d581c2df791113251f1"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"954818b2d350c42c3f3fff7fe325831a6bc5228c","unresolved":true,"context_lines":[{"line_number":81,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled backup False"},{"line_number":82,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled clone True"},{"line_number":83,"context_line":"    iniset $TEMPEST_CONFIG volume-feature-enabled manage_snapshot False"},{"line_number":84,"context_line":"    if [[ \"$TARGET_BRANCH\" \u003d~ stable/(xena|wallaby|victoria|ussuri) ]]; then"},{"line_number":85,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled volume_revert True"},{"line_number":86,"context_line":"        iniset $TEMPEST_CONFIG compute-feature-enabled attach_encrypted_volume True"},{"line_number":87,"context_line":"    else"}],"source_content_type":"text/x-sh","patch_set":21,"id":"13f2671f_dcdf780c","line":84,"in_reply_to":"8cddcc31_35fa2631","updated":"2022-07-27 17:56:51.000000000","message":"It looks like [[ \"$TARGET_BRANCH\" \u003d~ master ]]; is not doing what I want, because devstack-plugin-nfs-tempest-full-xena is running the volume_revert tests.\n\n```\n{1} tempest.scenario.test_encrypted_cinder_volumes.TestEncryptedCinderVolumes.test_encrypted_cinder_volumes_luks [213.257984s] ... FAILED\n\n```","commit_id":"3d3720114992538ac7f02d581c2df791113251f1"}],"tempest_skiplist.txt":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"7356a1e924a6914aa51834edb9b577c885365e42","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# The NFS driver only supports LUKS encryption."},{"line_number":4,"context_line":"tempest.scenario.test_encrypted_cinder_volumes.TestEncryptedCinderVolumes.test_encrypted_cinder_volumes_cryptsetup"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# Skip network and identity API tests since they have no bearing on a Ceph backend."},{"line_number":7,"context_line":"tempest.api.network"},{"line_number":8,"context_line":"tempest.api.identity"}],"source_content_type":"text/plain","patch_set":23,"id":"c9713b3b_8ce1e950","line":6,"range":{"start_line":6,"start_character":70,"end_line":6,"end_character":74},"updated":"2022-07-27 15:33:30.000000000","message":"NFS","commit_id":"6338deca89c388df8ed06f0ce9d7a61886a5ce4b"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"c65780a8dd0af306843a8718d0edbacf58bf6246","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# The NFS driver only supports LUKS encryption."},{"line_number":4,"context_line":"tempest.scenario.test_encrypted_cinder_volumes.TestEncryptedCinderVolumes.test_encrypted_cinder_volumes_cryptsetup"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# Skip network and identity API tests since they have no bearing on a Ceph backend."},{"line_number":7,"context_line":"tempest.api.network"},{"line_number":8,"context_line":"tempest.api.identity"}],"source_content_type":"text/plain","patch_set":23,"id":"105f6adb_181e85f7","line":6,"range":{"start_line":6,"start_character":70,"end_line":6,"end_character":74},"in_reply_to":"c9713b3b_8ce1e950","updated":"2022-07-27 17:57:51.000000000","message":"Done","commit_id":"6338deca89c388df8ed06f0ce9d7a61886a5ce4b"}]}