)]}'
{"id":"openstack%2Fnova~787175","triplet_id":"openstack%2Fnova~master~I441cff44346289eaf562cd495315a1143a7b0186","project":"openstack/nova","branch":"master","topic":"bug/1925144","hashtags":[],"change_id":"I441cff44346289eaf562cd495315a1143a7b0186","subject":"Force raising an timeout exception for rados connect","status":"ABANDONED","created":"2021-04-20 09:31:55.000000000","updated":"2021-04-20 10:13:34.000000000","total_comment_count":2,"unresolved_comment_count":2,"has_review_started":true,"meta_rev_id":"60002728fec4e63807b088201bc30a647c7efee8","_number":787175,"virtual_id_number":787175,"owner":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":{"disliked":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"all":[{"value":-1,"date":"2021-04-20 09:52:35.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-04-20 09:52:35.000000000","updated_by":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"reviewer":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"state":"REVIEWER"}],"messages":[{"id":"7bb5d3efea766bc2621d465d6c29ed1473f4bffa","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-04-20 09:31:55.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"04207a7e4b7de6c7692c27cf2833a93f5d6c613d","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-04-20 09:52:35.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)\n\nThanks for catching this, I was sure it was present back in I67f341bf895d6cc5d503da274c089d443295199e but https://github.com/ceph/ceph/commit/9997b32d0b8703e984a0b68c8dc828a442830832 is making me doubt if that was ever the case.\n\n-1 for some nits, unit tests and likely also needs a releasenote.\n\nOptionally we should deprecate the configurable I added in I67f341bf895d6cc5d503da274c089d443295199e but I can handle that in a FUP if you\u0027d like.","accounts_in_message":[],"_revision_number":1},{"id":"9e25de37f5588e89a214bad1da2105e7085aef8b","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-04-20 09:54:28.000000000","message":"Patch Set 1:\n\nIs this a duplicate of I441cff44346289eaf562cd495315a1143a7b0186 btw? Can you abandon one or the other.","accounts_in_message":[],"_revision_number":1},{"id":"0466ecb701c3aabd9627d6e8f40f914e57609dea","author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"date":"2021-04-20 09:54:41.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e Is this a duplicate of I441cff44346289eaf562cd495315a1143a7b0186 btw? Can you abandon one or the other.\n\nI2b1c7ee6e537cd71ad7ee5e6059792bfe06d7b1e sorry.","accounts_in_message":[],"_revision_number":1},{"id":"e10d38939ef4289314b1bb8c0dffbc6175b4dac9","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-04-20 10:03:54.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e \u003e Patch Set 1:\n\u003e \u003e \n\u003e \u003e Is this a duplicate of I441cff44346289eaf562cd495315a1143a7b0186 btw? Can you abandon one or the other.\n\u003e \n\u003e I2b1c7ee6e537cd71ad7ee5e6059792bfe06d7b1e sorry.\n\nerr... sorry for the duplicate commit, I made a mistake and delete change-id incorrectly....","accounts_in_message":[],"_revision_number":1},{"id":"ea038874008893d481325c1415f9185a8c95978e","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-04-20 10:12:53.000000000","message":"Patch Set 1:\n\nin fact, I personally like to have this timeout configuration, because\nwe have met up with all kinds of weird problems caused by incorrect network\nconfigurations, mtu and so on. so keep this configuration in my opinion will\nmake trouble shooting a little easier.\n\nbeside ceph command has a default timeout of 15 which seems implemented\nthough threading...","accounts_in_message":[],"_revision_number":1},{"id":"60002728fec4e63807b088201bc30a647c7efee8","tag":"autogenerated:gerrit:abandon","author":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"date":"2021-04-20 10:13:34.000000000","message":"Abandoned\n\nduplicate commit...","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"fe5c7fd803ec57d6f12ebe2f609105228ad3b46e","revisions":{"fe5c7fd803ec57d6f12ebe2f609105228ad3b46e":{"kind":"REWORK","_number":1,"created":"2021-04-20 09:31:55.000000000","uploader":{"_account_id":29071,"name":"norman shen","email":"yshxxsjt715@gmail.com","username":"ushen"},"ref":"refs/changes/75/787175/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/75/787175/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/787175/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/787175/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/75/787175/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/75/787175/1"}}},"commit":{"parents":[{"commit":"96e9124ad705df82fb4866c9241fac5e5e98b761","subject":"Merge \"Ignore PCI devices with 32bit domain\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/96e9124ad705df82fb4866c9241fac5e5e98b761"}]}],"author":{"name":"ushen","email":"yshxxsjt715@gmail.com","date":"2020-12-26 10:29:05.000000000","tz":480},"committer":{"name":"shenjiatong","email":"yshxxsjt715@gmail.com","date":"2021-04-20 09:22:26.000000000","tz":480},"subject":"Force raising an timeout exception for rados connect","message":"Force raising an timeout exception for rados connect\n\nrados::connect silently ignores timeout input which causes\nrados.connect hangs indefinitely and does not yield\ncontrol. This patch uses a tpool and raises timeout\nexception properly.\n\nChange-Id: I441cff44346289eaf562cd495315a1143a7b0186\nCloses-Bug: #1925144\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fe5c7fd803ec57d6f12ebe2f609105228ad3b46e"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/fe5c7fd803ec57d6f12ebe2f609105228ad3b46e"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
