)]}'
{"id":"openstack%2Fdevstack-gate~129730","triplet_id":"openstack%2Fdevstack-gate~master~Ia068c2356ad6fa27052ef4b38d1a95701f7cf6ac","project":"openstack/devstack-gate","branch":"master","topic":"rx-permissions","hashtags":[],"change_id":"Ia068c2356ad6fa27052ef4b38d1a95701f7cf6ac","subject":"Ensure dir permissions are 755 for cleanup to work","status":"ABANDONED","created":"2014-10-20 20:57:20.000000000","updated":"2014-11-18 22:45:09.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"c4fd7c5b0e6392da85adffd626bf59e39d1210d0","_number":129730,"virtual_id_number":129730,"owner":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":1,"date":"2014-10-22 00:09:55.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},{"date":"2014-10-21 15:47:37.000000000","_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_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":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2014-10-27 03:49:36.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},{"value":1,"date":"2014-10-20 21:07:27.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},{"value":-1,"date":"2014-11-18 20:05:59.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"value":2,"date":"2014-11-13 17:38:55.000000000","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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_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}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-10-20 21:07:27.000000000","updated_by":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"reviewer":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"state":"REVIEWER"},{"updated":"2014-10-22 00:09:55.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2014-10-27 03:49:36.000000000","updated_by":{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},"reviewer":{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},"state":"REVIEWER"},{"updated":"2014-11-13 17:38:55.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":"2014-11-18 20:05:59.000000000","updated_by":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"reviewer":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"state":"REVIEWER"}],"messages":[{"id":"cdd61f702ec816f6f4995448303c26b21b0a88fc","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2014-10-20 20:57:20.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c774f71518b26be027e1147050f99128d497926a","author":{"_account_id":6554,"name":"Spencer Krum","email":"nibz@spencerkrum.com","username":"krum-spencer"},"date":"2014-10-20 21:07:27.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"16c927c03e60cb0544fe026918e67dca7e150e99","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-10-21 15:21:47.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n- check-tempest-dsvm-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-full/0302231 : SUCCESS in 53m 06s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-postgres-full/befa0cc : SUCCESS in 46m 17s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-full/a2627d4 : FAILURE in 1h 07m 35s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-heat-slow/beef59d : SUCCESS in 24m 19s\n- check-grenade-dsvm http://logs.openstack.org/30/129730/1/check/check-grenade-dsvm/8dc2873 : SUCCESS in 49m 34s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/30/129730/1/check/gate-tempest-dsvm-large-ops/a7b542b : SUCCESS in 22m 17s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/30/129730/1/check/gate-tempest-dsvm-neutron-large-ops/b80dfb3 : SUCCESS in 22m 48s\n- gate-devstack-gate-unittests http://logs.openstack.org/30/129730/1/check/gate-devstack-gate-unittests/6340f3d : SUCCESS in 8s\n- check-devstack-dsvm-cells http://logs.openstack.org/30/129730/1/check/check-devstack-dsvm-cells/c8a13aa : SUCCESS in 13m 14s\n- check-dg-tempest-dsvm-full http://logs.openstack.org/30/129730/1/check/check-dg-tempest-dsvm-full/26de933 : SUCCESS in 44m 58s\n- check-dg-tempest-dsvm-full-reexec http://logs.openstack.org/30/129730/1/check/check-dg-tempest-dsvm-full-reexec/17ba78c : SUCCESS in 57m 21s\n- check-dg-hooks-dsvm http://logs.openstack.org/30/129730/1/check/check-dg-hooks-dsvm/4b85f9d : SUCCESS in 5m 10s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/30/129730/1/check/check-grenade-dsvm-partial-ncpu/087939d : SUCCESS in 52m 01s\n- check-swift-dsvm-functional http://logs.openstack.org/30/129730/1/check/check-swift-dsvm-functional/459e9bc : SUCCESS in 21m 00s\n- check-tempest-dsvm-full-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-full-icehouse/78aca35 : SUCCESS in 53m 05s\n- check-tempest-dsvm-neutron-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-icehouse/d2284ab : SUCCESS in 57m 20s\n- check-tempest-dsvm-postgres-full-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-postgres-full-icehouse/ec74942 : SUCCESS in 49m 00s\n- check-devstack-dsvm-cells-icehouse http://logs.openstack.org/30/129730/1/check/check-devstack-dsvm-cells-icehouse/0915d4e : SUCCESS in 13m 20s\n- check-tempest-dsvm-ironic-pxe_ssh-nv http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-ironic-pxe_ssh-nv/3e1a55f : SUCCESS in 32m 03s\n- check-tempest-dsvm-ironic-pxe_ssh-postgres-nv http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-ironic-pxe_ssh-postgres-nv/8ea2aa1 : SUCCESS in 32m 34s\n- check-tempest-dsvm-f20 http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-f20/fbf6438 : SUCCESS in 48m 08s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"009a418b11846bb3b44c99a90f496899db852bb0","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2014-10-21 15:47:37.000000000","message":"Patch Set 1:\n\nrecheck nobug","accounts_in_message":[],"_revision_number":1},{"id":"0da7b712bc91b96c34f56b1aa53b376b9bdcbbb4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-10-22 00:09:55.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- check-tempest-dsvm-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-full/4998c41 : SUCCESS in 53m 54s\n- check-tempest-dsvm-postgres-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-postgres-full/cdbe99d : SUCCESS in 54m 02s\n- check-tempest-dsvm-neutron-full http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-full/37c48aa : SUCCESS in 1h 09m 33s\n- check-tempest-dsvm-neutron-heat-slow http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-heat-slow/7bedb3d : SUCCESS in 23m 16s\n- check-grenade-dsvm http://logs.openstack.org/30/129730/1/check/check-grenade-dsvm/43ad041 : SUCCESS in 43m 55s\n- gate-tempest-dsvm-large-ops http://logs.openstack.org/30/129730/1/check/gate-tempest-dsvm-large-ops/47519e6 : SUCCESS in 20m 23s\n- gate-tempest-dsvm-neutron-large-ops http://logs.openstack.org/30/129730/1/check/gate-tempest-dsvm-neutron-large-ops/3444eab : SUCCESS in 27m 01s\n- gate-devstack-gate-unittests http://logs.openstack.org/30/129730/1/check/gate-devstack-gate-unittests/3d5c987 : SUCCESS in 6s\n- check-devstack-dsvm-cells http://logs.openstack.org/30/129730/1/check/check-devstack-dsvm-cells/efcca63 : SUCCESS in 13m 11s\n- check-dg-tempest-dsvm-full http://logs.openstack.org/30/129730/1/check/check-dg-tempest-dsvm-full/79d2eb0 : SUCCESS in 1h 00m 32s\n- check-dg-tempest-dsvm-full-reexec http://logs.openstack.org/30/129730/1/check/check-dg-tempest-dsvm-full-reexec/56b9f51 : SUCCESS in 58m 11s\n- check-dg-hooks-dsvm http://logs.openstack.org/30/129730/1/check/check-dg-hooks-dsvm/0d38bb8 : SUCCESS in 5m 39s\n- check-grenade-dsvm-partial-ncpu http://logs.openstack.org/30/129730/1/check/check-grenade-dsvm-partial-ncpu/7a645fd : SUCCESS in 42m 12s\n- check-swift-dsvm-functional http://logs.openstack.org/30/129730/1/check/check-swift-dsvm-functional/11432aa : SUCCESS in 18m 00s\n- check-tempest-dsvm-full-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-full-icehouse/b97fe62 : SUCCESS in 50m 26s\n- check-tempest-dsvm-neutron-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-neutron-icehouse/3d86abe : SUCCESS in 58m 46s\n- check-tempest-dsvm-postgres-full-icehouse http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-postgres-full-icehouse/159c0f9 : SUCCESS in 49m 33s\n- check-devstack-dsvm-cells-icehouse http://logs.openstack.org/30/129730/1/check/check-devstack-dsvm-cells-icehouse/3d019e8 : SUCCESS in 13m 57s\n- check-tempest-dsvm-ironic-pxe_ssh-nv http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-ironic-pxe_ssh-nv/c86a6d6 : SUCCESS in 27m 58s\n- check-tempest-dsvm-ironic-pxe_ssh-postgres-nv http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-ironic-pxe_ssh-postgres-nv/9d7c3f2 : SUCCESS in 30m 21s\n- check-tempest-dsvm-f20 http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-f20/6e2f2ab : SUCCESS in 44m 02s (non-voting)\n- check-tempest-dsvm-centos7 http://logs.openstack.org/30/129730/1/check/check-tempest-dsvm-centos7/bc121ab : FAILURE in 5m 58s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"b73e3673b4121a5111c1c563c16f54168c9e6a00","author":{"_account_id":12182,"name":"azher ullah khan","email":"azher.khan@oracle.com","username":"AzherKhan"},"date":"2014-10-27 03:49:36.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"23f37ecde3d3ec9eabe3ae41797e3131029f2725","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2014-11-13 17:38:55.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"168318f6d0af7279a80331c2ad4e4371defcf52e","author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"date":"2014-11-18 20:05:59.000000000","message":"Patch Set 1: Code-Review-1\n\nSorry, I missed this when I proposed I3b949c7660efe1b5158a72b6b8ccbd0d7b9a0b38 (i thought the title referred to something else)\n\nI think it\u0027s probably easier to just cp -r the directory and skip this extra step","accounts_in_message":[],"_revision_number":1},{"id":"c3a28e6d3670762501846e720df38bb2ccb92fe5","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2014-11-18 22:00:41.000000000","message":"Patch Set 1:\n\nIan, I do not mind abandoning this patch in favor of yours. Would you please include libvirtd logs fix there as well?","accounts_in_message":[],"_revision_number":1},{"id":"8d39b21b34af13c5ed2bc1afe8ad36c88a82a1ea","author":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"date":"2014-11-18 22:45:09.000000000","message":"Abandoned\n\nI think I3b949c7660efe1b5158a72b6b8ccbd0d7b9a0b38 is a better solution","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"5f5db507e5af73877a6a7500bb042af27b567a8b","revisions":{"5f5db507e5af73877a6a7500bb042af27b567a8b":{"kind":"REWORK","_number":1,"created":"2014-10-20 20:57:20.000000000","uploader":{"_account_id":7979,"name":"Mikhail S Medvedev","email":"mihailmed@gmail.com","username":"msmedved"},"ref":"refs/changes/30/129730/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack-gate","ref":"refs/changes/30/129730/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/30/129730/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/30/129730/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack-gate refs/changes/30/129730/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack-gate refs/changes/30/129730/1"}}},"commit":{"parents":[{"commit":"fb1c754d501d673fafc6b8017c2b2d337b5071a5","subject":"Merge \"Use 1024 MB RAM for Ironic agent VMs\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/fb1c754d501d673fafc6b8017c2b2d337b5071a5"}]}],"author":{"name":"Mikhail S Medvedev","email":"mmedvede@us.ibm.com","date":"2014-10-20 20:21:46.000000000","tz":-300},"committer":{"name":"Mikhail S Medvedev","email":"mmedvede@us.ibm.com","date":"2014-10-20 20:21:46.000000000","tz":-300},"subject":"Ensure dir permissions are 755 for cleanup to work","message":"Ensure dir permissions are 755 for cleanup to work\n\ncleanup_host uses wildcards to copy several files. For example,\n\n  sudo cp /etc/sudoers.d/* $BASE/logs/sudoers.d/\n\nThis wildcard expansion would not work if the sudoers.d does not\nhave read permission for others. The lack of permission would result\nin \"cp: cannot stat ...\" error when a wildcard is used. This error\ncan currently be seen for sudoers.d in devstack-gate-cleanup-host.txt\n\nWith this patch we fix the permissions so that the expansion works for\n/etc/sudoers.d and for /var/log/libvirtd/.\n\nThe latter is not needed on Ubuntu, but fixes similar issue on\nFedora 20. We do not use is_fedora check because it would just add\nclutter, and the permission fix does not have any side effects as we\nare cleaning up.\n\nFurther permission fixes could be necessary and this patch is in no\nway covers all the cases.\n\nChange-Id: Ia068c2356ad6fa27052ef4b38d1a95701f7cf6ac\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/5f5db507e5af73877a6a7500bb042af27b567a8b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack-gate/commit/5f5db507e5af73877a6a7500bb042af27b567a8b"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}