)]}'
{"id":"openstack%2Fdevstack~266537","triplet_id":"openstack%2Fdevstack~master~I801100134754454f902cfe9034021837b1a185dc","project":"openstack/devstack","branch":"master","hashtags":[],"change_id":"I801100134754454f902cfe9034021837b1a185dc","subject":"Fix ordering of check in yum_Install, so PIPESTATUS[0] is checked first","status":"ABANDONED","created":"2016-01-12 19:18:58.000000000","updated":"2016-01-12 20:40:22.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0505cb2ea7533f69f900a56384725b15817a2ec7","_number":266537,"virtual_id_number":266537,"owner":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"actions":{},"labels":{"Verified":{"all":[{"date":"2016-01-12 20:40:22.000000000","_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},{"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":{"all":[{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"date":"2016-01-12 19:20:30.000000000","_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},{"date":"2016-01-12 19:20:34.000000000","_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},{"date":"2016-01-12 19:29:36.000000000","_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":2},"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"}],"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":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-01-12 19:20:30.000000000","updated_by":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"reviewer":{"_account_id":17377,"name":"Federico Ressi","email":"federico.ressi@intel.com","username":"fressi"},"state":"REVIEWER"},{"updated":"2016-01-12 19:20:34.000000000","updated_by":{"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},"reviewer":{"_account_id":333,"name":"Isaku Yamahata","email":"isaku.yamahata@gmail.com","username":"yamahata"},"state":"REVIEWER"},{"updated":"2016-01-12 19:29:36.000000000","updated_by":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"reviewer":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"state":"REVIEWER"},{"updated":"2016-01-12 20:40:22.000000000","updated_by":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"reviewer":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"3d564c73f9a2e84f5e908ca1b15b020220a88ac2","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2016-01-12 19:18:58.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"63e563789d095a8cadf6083efe45d450236750aa","author":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"date":"2016-01-12 20:05:01.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1},{"id":"0505cb2ea7533f69f900a56384725b15817a2ec7","author":{"_account_id":10385,"name":"Citrix XenServer CI","username":"citrix_xenserver_ci","tags":["SERVICE_USER"]},"date":"2016-01-12 20:40:22.000000000","message":"Patch Set 1:\n\nTesting completed on Citrix Xenserver. For rechecking only on the Citrix Xenserver CI,add a review comment with \"xenserver: recheck\". Contact info: openstack@citrix.com.\nFor debugging suggestions, see https://wiki.openstack.org/wiki/Debugging_XenServer_CI_failures\n\n- check-citrix-xenserver http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/37/266537/1/22841/index.html : SUCCESS in 01 h 03 m 06 s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"85ed5aac9c81f7ea86a89e1ab691a448041f01eb","revisions":{"85ed5aac9c81f7ea86a89e1ab691a448041f01eb":{"kind":"REWORK","_number":1,"created":"2016-01-12 19:18:58.000000000","uploader":{"_account_id":11952,"name":"Flavio Fernandes","email":"flavio@flaviof.com","username":"ffernand"},"ref":"refs/changes/37/266537/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/37/266537/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/37/266537/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/37/266537/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/37/266537/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/37/266537/1"}}},"commit":{"parents":[{"commit":"2cb3db338a17bd26fef1f56fc155be5d01a6ef7e","subject":"Merge \"Don\u0027t die when yum fails.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/2cb3db338a17bd26fef1f56fc155be5d01a6ef7e"}]}],"author":{"name":"Flavio Fernandes","email":"ffernand@redhat.com","date":"2016-01-12 19:09:41.000000000","tz":-300},"committer":{"name":"Flavio Fernandes","email":"ffernand@redhat.com","date":"2016-01-12 19:15:48.000000000","tz":-300},"subject":"Fix ordering of check in yum_Install, so PIPESTATUS[0] is checked first","message":"Fix ordering of check in yum_Install, so PIPESTATUS[0] is checked first\n\nRe-ordered check for how yum_install went, so PIPESTATUS[0] is the value\nof the awk command. Failure in doing so renders the check to be after\nthe if statement, which gives it the value of 1.\n\nThis issue apperas to have been introduced in commit 2babf396\n\n${PIPESTATUS[0]} !\u003d 0\n\n...\nRunning transaction check\nRunning transaction test\nTransaction test succeeded\nRunning transaction\n  Installing : epel-release-7-5.noarch                                      1/1\n  Verifying  : epel-release-7-5.noarch                                      1/1\n\nInstalled:\n  epel-release.noarch 0:7-5\n\nComplete!\n+ result\u003d0\n+ \u0027[\u0027 0 \u0027!\u003d\u0027 0 \u0027]\u0027   \u003c\u003d\u003d result\n+ [[ 1 !\u003d 0 ]]       \u003c\u003d\u003d ${PIPESTATUS[0]}, used one statement too late\n+ echo 1344 \u0027yum install failure\u0027\n1344 yum install failure\n+ result\u003d1\n+ return 1\n+ die 286 \u0027Error installing EPEL repo, cannot continue\u0027\n+ local exitcode\u003d1\n\nReference-Change-Id: Ia33bb98f47017f6ec00eb3b7261ed42d72efc573\nChange-Id: I801100134754454f902cfe9034021837b1a185dc\nCloses-Bug: #1522590\nSigned-off-by: Flavio Fernandes \u003cffernand@redhat.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/85ed5aac9c81f7ea86a89e1ab691a448041f01eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/85ed5aac9c81f7ea86a89e1ab691a448041f01eb"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}