)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1484cb92101d513e62e52e08203dbf50952586fb","unresolved":true,"context_lines":[{"line_number":18,"context_line":"Periodic RDO jobs will need to be fixed"},{"line_number":19,"context_line":"in the same manner."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: Iecb1328f1fb3d54cfd91bb9c23d2d8ee40cf3b33"},{"line_number":22,"context_line":"Related-Bug: #1922784"},{"line_number":23,"context_line":"Change-Id: I4fbe340a110cb0b99e786844760af134672cda37"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"d65f5ca2_780bde6d","line":21,"updated":"2021-04-13 06:38:20.000000000","message":"this one merged ^^^ but per the discussion here then we should add depends-on https://review.rdoproject.org/r/c/rdoinfo/+/33233 so we don\u0027t break train?","commit_id":"8b994f9450f4e74a7318b784ce4da9474f72fce1"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ea972aa3cd330199c666b78b4033486eb0590162","unresolved":true,"context_lines":[{"line_number":18,"context_line":"Periodic RDO jobs will need to be fixed"},{"line_number":19,"context_line":"in the same manner."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Depends-On: Iecb1328f1fb3d54cfd91bb9c23d2d8ee40cf3b33"},{"line_number":22,"context_line":"Related-Bug: #1922784"},{"line_number":23,"context_line":"Change-Id: I4fbe340a110cb0b99e786844760af134672cda37"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"1659e599_5eb3bf8a","line":21,"in_reply_to":"d65f5ca2_780bde6d","updated":"2021-04-13 15:54:20.000000000","message":"I wasn\u0027t aware we could do depends-on with RDO gerrit :) Will do, thanks Marios!","commit_id":"8b994f9450f4e74a7318b784ce4da9474f72fce1"}],"zuul.d/standalone-jobs.yaml":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"28e165f603ab26555d37c8b17b68d5ba71a340d4","unresolved":true,"context_lines":[{"line_number":443,"context_line":"          \u0027share.multitenancy_enabled\u0027: \u0027False\u0027"},{"line_number":444,"context_line":"          \u0027share.enable_protocols\u0027: \u0027nfs\u0027"},{"line_number":445,"context_line":"          \u0027share.capability_snapshot_support\u0027: \u0027False\u0027"},{"line_number":446,"context_line":"          \u0027share.capability_create_share_from_snapshot_support\u0027: \u0027False\u0027"},{"line_number":447,"context_line":"        tempest_test_whitelist:"},{"line_number":448,"context_line":"          - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":449,"context_line":"          - \u0027manila_tempest_tests.tests.api.test_shares.SharesCephFSTest.test_create_get_delete_share\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"0724d721_53186313","side":"PARENT","line":446,"updated":"2021-04-09 05:54:34.000000000","message":"since this patch depends-on tempestconf, need to ensure that tempestconf with the patch get\u0027s updated in rdoinfo for train else train jobs can have issues, currently it\u0027s pinned to 3.1.0.","commit_id":"8055430854f56deaef8e94fac3a6abe60b647162"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"99b877d63213813fa09d24b2f444c47798ec0c7b","unresolved":true,"context_lines":[{"line_number":443,"context_line":"          \u0027share.multitenancy_enabled\u0027: \u0027False\u0027"},{"line_number":444,"context_line":"          \u0027share.enable_protocols\u0027: \u0027nfs\u0027"},{"line_number":445,"context_line":"          \u0027share.capability_snapshot_support\u0027: \u0027False\u0027"},{"line_number":446,"context_line":"          \u0027share.capability_create_share_from_snapshot_support\u0027: \u0027False\u0027"},{"line_number":447,"context_line":"        tempest_test_whitelist:"},{"line_number":448,"context_line":"          - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":449,"context_line":"          - \u0027manila_tempest_tests.tests.api.test_shares.SharesCephFSTest.test_create_get_delete_share\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"f1d2b55c_36ca7b77","side":"PARENT","line":446,"in_reply_to":"0724d721_53186313","updated":"2021-04-09 06:50:54.000000000","message":"trying to understand the implications; I guess with older tempestconf we\u0027d just default to false, as per manila-tempest-plugin [1]?\n\n1. https://opendev.org/openstack/manila-tempest-plugin/src/branch/master/manila_tempest_tests/config.py#L106","commit_id":"8055430854f56deaef8e94fac3a6abe60b647162"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"95ef0a4e2ecd2975507f0f15fe5f77781999ee81","unresolved":true,"context_lines":[{"line_number":443,"context_line":"          \u0027share.multitenancy_enabled\u0027: \u0027False\u0027"},{"line_number":444,"context_line":"          \u0027share.enable_protocols\u0027: \u0027nfs\u0027"},{"line_number":445,"context_line":"          \u0027share.capability_snapshot_support\u0027: \u0027False\u0027"},{"line_number":446,"context_line":"          \u0027share.capability_create_share_from_snapshot_support\u0027: \u0027False\u0027"},{"line_number":447,"context_line":"        tempest_test_whitelist:"},{"line_number":448,"context_line":"          - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":449,"context_line":"          - \u0027manila_tempest_tests.tests.api.test_shares.SharesCephFSTest.test_create_get_delete_share\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"9b6c0497_96c3d895","side":"PARENT","line":446,"in_reply_to":"6a962859_c717edaa","updated":"2021-04-09 19:48:33.000000000","message":"Good point. I see rdoinfo has tempestconf pinned to 2.4.0 for stein and 3.2.0 for train\nThis job is for stein. \n\nThe implication of not having these overrides is:\n\n \u0027share.multitenancy_enabled\u0027 is set to True (manila-tempest-plugin default)\n \u0027share.enable_protocols\u0027: is set to \u0027nfs,cifs\u0027 (manila-tempest-plugin default, incompatible with tripleo using \"nfs\")\n \u0027share.capability_snapshot_support\u0027: is set to True (manila-tempest-plugin default, incompatible with all releases prior to wallaby for the cephfs driver that tripleo uses in testing)\n \u0027share.capability_create_share_from_snapshot_support\u0027: is set to True (manila-tempest-plugin default, incompatible with all releases prior to wallaby for the cephfs driver that tripleo uses in testing)\n\n\nSo this will surely break jobs in stable/stein and stable/train. \nIs there a reason we\u0027re pinning this for older releases? If not: https://review.rdoproject.org/r/c/rdoinfo/+/33214","commit_id":"8055430854f56deaef8e94fac3a6abe60b647162"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"2acde0be17ca19130714e9a555d1e9999d099b53","unresolved":true,"context_lines":[{"line_number":443,"context_line":"          \u0027share.multitenancy_enabled\u0027: \u0027False\u0027"},{"line_number":444,"context_line":"          \u0027share.enable_protocols\u0027: \u0027nfs\u0027"},{"line_number":445,"context_line":"          \u0027share.capability_snapshot_support\u0027: \u0027False\u0027"},{"line_number":446,"context_line":"          \u0027share.capability_create_share_from_snapshot_support\u0027: \u0027False\u0027"},{"line_number":447,"context_line":"        tempest_test_whitelist:"},{"line_number":448,"context_line":"          - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":449,"context_line":"          - \u0027manila_tempest_tests.tests.api.test_shares.SharesCephFSTest.test_create_get_delete_share\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"4fb7ca74_7bd23e8a","side":"PARENT","line":446,"in_reply_to":"9b6c0497_96c3d895","updated":"2021-04-12 23:35:53.000000000","message":"Thanks for your comments on https://review.rdoproject.org/r/c/rdoinfo/+/33214\nI agree there\u0027s no need to remove the pin version all over the branches.\n\nMartin\u0027s helping with adopting a newer tag on stable/train: https://review.rdoproject.org/r/c/rdoinfo/+/33233","commit_id":"8055430854f56deaef8e94fac3a6abe60b647162"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"b2f17a82d18016ab944513f3afa014f5ebca88d2","unresolved":true,"context_lines":[{"line_number":443,"context_line":"          \u0027share.multitenancy_enabled\u0027: \u0027False\u0027"},{"line_number":444,"context_line":"          \u0027share.enable_protocols\u0027: \u0027nfs\u0027"},{"line_number":445,"context_line":"          \u0027share.capability_snapshot_support\u0027: \u0027False\u0027"},{"line_number":446,"context_line":"          \u0027share.capability_create_share_from_snapshot_support\u0027: \u0027False\u0027"},{"line_number":447,"context_line":"        tempest_test_whitelist:"},{"line_number":448,"context_line":"          - \u0027tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops\u0027"},{"line_number":449,"context_line":"          - \u0027manila_tempest_tests.tests.api.test_shares.SharesCephFSTest.test_create_get_delete_share\u0027"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"6a962859_c717edaa","side":"PARENT","line":446,"in_reply_to":"f1d2b55c_36ca7b77","updated":"2021-04-09 11:16:09.000000000","message":"I just recalled the issue from  past https://bugs.launchpad.net/tripleo/+bug/1914041 for which we had to do enable_protocols: nfs, if it\u0027s not valid now then in train too it shouldn\u0027t be an issue, will remove my vote.","commit_id":"8055430854f56deaef8e94fac3a6abe60b647162"}]}