)]}'
{"id":"openstack%2Fpython-cinderclient~802966","triplet_id":"openstack%2Fpython-cinderclient~master~I33399b4c094af2cc059da6e332f4c0a91e6ab57e","project":"openstack/python-cinderclient","branch":"master","topic":"change-timeout","hashtags":[],"change_id":"I33399b4c094af2cc059da6e332f4c0a91e6ab57e","subject":"Unset tempest.lib timeout in functional tests","status":"MERGED","created":"2021-07-29 22:13:46.000000000","updated":"2021-08-05 16:28:43.000000000","submitted":"2021-08-05 16:27:04.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"802966-change-timeout","meta_rev_id":"883eb06c41693c50a8f77b98c52b1fc1ad186f3d","_number":802966,"virtual_id_number":802966,"owner":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-08-05 16:27:03.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-08-04 14:41:05.000000000","_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},{"value":2,"date":"2021-08-05 14:42:46.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},{"value":1,"date":"2021-08-05 14:42:46.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"approved":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},{"value":0,"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":2,"date":"2021-08-02 12:07:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-07-29 23:26:02.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2021-08-04 14:38:03.000000000","updated_by":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"reviewer":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"state":"REVIEWER"},{"updated":"2021-08-05 14:42:46.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"}],"messages":[{"id":"6b384b5734e131f92822ddf398207e5641b3a5cc","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-07-29 22:13:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a60eada9f8bd119bd6004adc0a4fc301e3de4aa2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-29 23:26:02.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/5952adfe866046548086172344f489f5 : SUCCESS in 1h 10m 55s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1220226a94b641168ee9a51cd33df497 : SUCCESS in 5m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/2dbaa107eeaa4cbeb3b4e44c6f15d4df : SUCCESS in 4m 10s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/a2938134b6ef4c358dd790165a40e703 : SUCCESS in 4m 55s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/14a43312332f4f9abe12a380b9252ba0 : SUCCESS in 4m 23s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a09e1a5d273a4f16ae8846e384dbad8f : SUCCESS in 5m 39s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c2dc726a6df64264a413d62f22414795 : SUCCESS in 5m 03s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/f8c706472015417d90b994851a276fbd : SUCCESS in 8m 25s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/06c72d48e05044cea575f0e887667c18 : SUCCESS in 4m 11s\n- python-cinderclient-functional-py36 https://zuul.opendev.org/t/openstack/build/ac9064a543684121a6dd790c33f388fe : SUCCESS in 36m 02s\n- python-cinderclient-functional-py38 https://zuul.opendev.org/t/openstack/build/8f9a57e8137347c4850ff35584e8d0b0 : FAILURE in 27m 46s\n- openstack-tox-pylint https://zuul.opendev.org/t/openstack/build/3976253c429a4eba93dbfe77ad9375f3 : SUCCESS in 5m 21s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"8bc47e9ee6d10c3f700a6faa4e825e6cbcf9dbc5","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-07-30 02:01:27.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a9aec9fadddba8f620dfe6b38b5fea63ccdefcd6","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-07-30 02:04:18.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"6474d4da85fa1cc1c3eb6f7721c8f79628707fee","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 03:17:14.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/704b750788e149598d0d0e10f22a7374 : SUCCESS in 1h 11m 00s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b5b47c24aeee4ba5a51ab72874a517a5 : SUCCESS in 4m 12s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0fa8cb2151d94a69bf0a3b2cb35590bb : SUCCESS in 4m 39s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/4c5b12fddf7d47c1b37765118e8fc1cb : SUCCESS in 7m 22s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/cc35710d0c764d9c81505d111bdb65dd : SUCCESS in 3m 51s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/fe5a28a95ef9425783b6e5da3a5ce111 : SUCCESS in 8m 52s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/ff7712c574da4510957259bdf6b50f40 : SUCCESS in 4m 27s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/237400d71d0143398ee54c58feac2832 : SUCCESS in 8m 45s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/8b55897fd0fd4b51b8e8c70d78604c52 : SUCCESS in 7m 07s\n- python-cinderclient-functional-py36 https://zuul.opendev.org/t/openstack/build/979b63b38c4946b2b8e6d9080ed19866 : FAILURE in 32m 42s\n- python-cinderclient-functional-py38 https://zuul.opendev.org/t/openstack/build/2a3209f93d674e59962073146de0b95b : SUCCESS in 26m 17s\n- openstack-tox-pylint https://zuul.opendev.org/t/openstack/build/0bf6012fcb2948b394c456d0d577a199 : SUCCESS in 7m 45s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"a5145e0e4f0be9d560327744f6b49d55c361ad5c","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-07-30 10:44:31.000000000","message":"Patch Set 3:\n\nfailure in python-cinderclient-functional-py36, but it\u0027s not the timeout issue this patch addresses. \n This time it\u0027s in CinderVolumeExtendNegativeTests.test_volume_extend_with_incorrect_size VolumeLimitExceeded: Maximum number of volumes allowed (10) exceeded for quota \u0027volumes\u0027.  On this run, by the way, CinderBackupTests.test_backup_create_and_delete completed successfully in 45.485087 sec, which is under the \"old\" timeout limit of 60 sec.  We\u0027re running tests in 8 threads, though, so it\u0027s not too surprising that 10 might be too low a quota on volumes, especially if any tests are creating \u003e 1 volume.","accounts_in_message":[],"_revision_number":3},{"id":"6755ae96bdeacd4a316c8f7df494c9196d6d892f","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-07-30 10:45:11.000000000","message":"Patch Set 3:\n\nrecheck python-cinderclient-functional-py36","accounts_in_message":[],"_revision_number":3},{"id":"2c760c650f5bf309f38bfec3cd49aeda86d3d4fb","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-07-30 12:06:41.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/79bd1fcf28114ae48e53d6a2a5faa21c : SUCCESS in 1h 15m 37s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/a6e022542be14978817dacd46d713d40 : SUCCESS in 4m 18s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/efcf8bdf503541afa67dab12b38d5921 : SUCCESS in 4m 49s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ce88625c6d3d4bb48938c762fccc1278 : SUCCESS in 4m 27s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/c6c13fa8816e42baafa608cccb4d7bfb : SUCCESS in 4m 42s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/093918e7af464420973c19a73816d59e : SUCCESS in 5m 44s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/c4e3ac2fcc994ed68af9fbb2fde7a1cd : SUCCESS in 5m 00s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e43b48279b524953bf858be83d2b1cc0 : SUCCESS in 7m 28s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/99fa083ca9c14dcfbae47c0ac32ba635 : SUCCESS in 5m 34s\n- python-cinderclient-functional-py36 https://zuul.opendev.org/t/openstack/build/706992e6d42c42b583b1537d6726ea73 : SUCCESS in 29m 10s\n- python-cinderclient-functional-py38 https://zuul.opendev.org/t/openstack/build/61478ec02ebd4ecf81dadb7d6d371bce : SUCCESS in 26m 07s\n- openstack-tox-pylint https://zuul.opendev.org/t/openstack/build/665cea897a1f46f186a3dcf12e9b0b93 : SUCCESS in 5m 00s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"b393f97faacbcd776d6d48a7dc94874a5827f643","author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"date":"2021-08-02 12:07:55.000000000","message":"Patch Set 3: Review-Priority+2\n\nBy itself, this isn\u0027t a big deal, and the issue occurs intermittently (I think depending on what hardware the job lands on).  But each recheck also requires re-running tempest-full-py3, which is resource- and time-intensive.  I also want to point out that as far as the cinderclient test code is concerned, we have already accepted these timeout values (the shorter timeouts are caused by use of a non-obvious environment variable).\n\nLet\u0027s get this change merged and backported soon.","accounts_in_message":[],"_revision_number":3},{"id":"6544d1131d607d5ee38f1ea473d739b226ae3543","author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"date":"2021-08-04 14:41:05.000000000","message":"Patch Set 3: Code-Review+2\n\nXS patch looks good to me","accounts_in_message":[],"_revision_number":3},{"id":"81c21d004b48b7896caaffa695d092ce62fb66c9","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2021-08-05 14:42:46.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"0573d47302e7155133433e4c33a471562fa51619","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 14:43:15.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":3},{"id":"8885cb479611acd0816652ed1a67335809486b29","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 16:27:03.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- tempest-full-py3 https://zuul.opendev.org/t/openstack/build/301fbd1c91e04c959368cc3d1244b1d9 : SUCCESS in 1h 37m 37s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/80a1ad360d63439dab40a15028331ca3 : SUCCESS in 4m 41s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/86698c963641415aa88847e1ef92f3a6 : SUCCESS in 5m 29s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/7b262f34e69d47c9a806c359fcf421af : SUCCESS in 4m 58s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4229c686e69f45988806a2d9bbc03702 : SUCCESS in 6m 30s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/d97ca24c42624cfc96898384eaed9250 : SUCCESS in 4m 37s","accounts_in_message":[],"_revision_number":3},{"id":"c84991593e0fc2f7d0b603814c215b5712ca2104","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 16:27:04.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":3},{"id":"883eb06c41693c50a8f77b98c52b1fc1ad186f3d","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-08-05 16:28:43.000000000","message":"Patch Set 3:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/65ae9eb33f314c3bab41f86f57abf658 : SUCCESS in 1m 19s\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/51ea8cf0b5384e1bab993fc6e60dfcba : SUCCESS in 1m 06s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"d04ded6a6ff4cc9ba8377a1b025493e939047b2c","revisions":{"eb886c5e728f32a621626d4f1122d85c599fcd4c":{"kind":"REWORK","_number":1,"created":"2021-07-29 22:13:46.000000000","uploader":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"ref":"refs/changes/66/802966/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/66/802966/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/1"}}},"commit":{"parents":[{"commit":"e03111dea33d671661d01eadbb068d6ee6987cec","subject":"Merge \"Make instance_uuid optional in attachment create\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e03111dea33d671661d01eadbb068d6ee6987cec"}]}],"author":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-29 21:44:23.000000000","tz":-240},"committer":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-29 22:12:10.000000000","tz":-240},"subject":"Unset tempest.lib timeout in functional-py36","message":"Unset tempest.lib timeout in functional-py36\n\nThe test_cli.CinderBackupTests.test_backup_create_and_delete test\nis hitting timeout errors in the python-cinderclient-functional-py36\nzuul job.  This is happening because it\u0027s inheriting the\nOS_TEST_TIMEOUT value of 60 from the base testenv, and that value is\nbeing used by the tempest.lib class we inherit from as a timeout for\neach test.  This is a problem for test_backup_create_and_delete\nbecause it creates a volume, waits for available, creates a backup,\nwaits for available, deletes the volume, waits for deletion, deletes\nthe backup, waits for deletion.\n\nOur functional tests have their own timeout handling, so turn off\nthe tempest.lib timeout and use ours.  An alternative to turning it\noff is to set it at a value that respects our timeout for our longest\ntest, which would be:\n- time-to-available: 120 sec (x2)\n- time-to-deleted: 60 sec (x2)\nthat is, 360 sec.\n\nThis only seems to be happening for the functional-py36 job, so this\npatch only makes the change for that tox testenv.\n\nChange-Id: I33399b4c094af2cc059da6e332f4c0a91e6ab57e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eb886c5e728f32a621626d4f1122d85c599fcd4c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/eb886c5e728f32a621626d4f1122d85c599fcd4c"}]},"branch":"refs/heads/master"},"023f2cad453ca3f2bb9f0031a94aa8f74a40630d":{"kind":"REWORK","_number":2,"created":"2021-07-30 02:01:27.000000000","uploader":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"ref":"refs/changes/66/802966/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/66/802966/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/2"}}},"commit":{"parents":[{"commit":"e03111dea33d671661d01eadbb068d6ee6987cec","subject":"Merge \"Make instance_uuid optional in attachment create\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e03111dea33d671661d01eadbb068d6ee6987cec"}]}],"author":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-29 21:44:23.000000000","tz":-240},"committer":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-30 02:00:38.000000000","tz":-240},"subject":"Unset tempest.lib timeout in functional tests","message":"Unset tempest.lib timeout in functional tests\n\nThe test_cli.CinderBackupTests.test_backup_create_and_delete test\nis hitting timeout errors in the python-cinderclient-functional-py36\nzuul job.  This is happening because it\u0027s inheriting the\nOS_TEST_TIMEOUT value of 60 from the base testenv, and that value is\nbeing used by the tempest.lib class we inherit from as a timeout for\neach test.  This is a problem for test_backup_create_and_delete\nbecause it creates a volume, waits for available, creates a backup,\nwaits for available, deletes the volume, waits for deletion, deletes\nthe backup, waits for deletion.\n\nOur functional tests have their own timeout handling, so turn off\nthe tempest.lib timeout and use ours.  An alternative to turning it\noff is to set it at a value that respects our timeout for our longest\ntest, which would be:\n- time-to-available: 120 sec (x2)\n- time-to-deleted: 60 sec (x2)\nthat is, 360 sec.\n\nChange-Id: I33399b4c094af2cc059da6e332f4c0a91e6ab57e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/023f2cad453ca3f2bb9f0031a94aa8f74a40630d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/023f2cad453ca3f2bb9f0031a94aa8f74a40630d"}]},"branch":"refs/heads/master"},"d04ded6a6ff4cc9ba8377a1b025493e939047b2c":{"kind":"REWORK","_number":3,"created":"2021-07-30 02:04:18.000000000","uploader":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"ref":"refs/changes/66/802966/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-cinderclient","ref":"refs/changes/66/802966/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-cinderclient refs/changes/66/802966/3"}}},"commit":{"parents":[{"commit":"e03111dea33d671661d01eadbb068d6ee6987cec","subject":"Merge \"Make instance_uuid optional in attachment create\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/e03111dea33d671661d01eadbb068d6ee6987cec"}]}],"author":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-29 21:44:23.000000000","tz":-240},"committer":{"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","date":"2021-07-30 02:04:01.000000000","tz":-240},"subject":"Unset tempest.lib timeout in functional tests","message":"Unset tempest.lib timeout in functional tests\n\nThe test_cli.CinderBackupTests.test_backup_create_and_delete test\nis hitting timeout errors in the python-cinderclient-functional-py36\nzuul job.  This is happening because it\u0027s inheriting the\nOS_TEST_TIMEOUT value of 60 from the base testenv, and that value is\nbeing used by the tempest.lib class we inherit from as a timeout for\neach test.  This is a problem for test_backup_create_and_delete\nbecause it creates a volume, waits for available, creates a backup,\nwaits for available, deletes the volume, waits for deletion, deletes\nthe backup, waits for deletion.\n\nOur functional tests have their own timeout handling, so turn off\nthe tempest.lib timeout and use ours.  An alternative to turning it\noff is to set it at a value that respects our timeout for our longest\ntest, which would be:\n- time-to-available: 120 sec (x2)\n- time-to-deleted: 60 sec (x2)\nthat is, 360 sec.\n\nChange-Id: I33399b4c094af2cc059da6e332f4c0a91e6ab57e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/d04ded6a6ff4cc9ba8377a1b025493e939047b2c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-cinderclient/commit/d04ded6a6ff4cc9ba8377a1b025493e939047b2c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"}},{"label":"Review-Priority","status":"MAY","applied_by":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}