)]}'
{"id":"openstack%2Fdevstack~51067","triplet_id":"openstack%2Fdevstack~master~If91092caf266a88168362ae350bd54c8c63f522c","project":"openstack/devstack","branch":"master","topic":"directory-acl","hashtags":[],"change_id":"If91092caf266a88168362ae350bd54c8c63f522c","subject":"make sure devstack change owner for all files in /etc/*/conf","status":"ABANDONED","created":"2013-10-10 22:13:58.000000000","updated":"2013-10-14 15:45:24.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"6f197e325a8a9848f83c63b5bade3c1e4ccc43d1","_number":51067,"virtual_id_number":51067,"owner":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":1,"date":"2013-10-10 23:08:51.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},{"_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":{"rejected":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"recommended":{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},"all":[{"value":0,"date":"2013-10-14 15:23:48.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":-2,"date":"2013-10-14 15:43:06.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":1,"date":"2013-10-13 12:28:05.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},{"value":-1,"date":"2013-10-14 14:50:17.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":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"date":"2013-10-14 00:59:38.000000000","_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"date":"2013-10-14 13:47:47.000000000","_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"date":"2013-10-14 13:47:25.000000000","_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"date":"2013-10-12 13:14:51.000000000","_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},{"date":"2013-10-14 14:50:17.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":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},{"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":3,"name":"Jenkins","username":"jenkins"},{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2013-10-10 23:08:51.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"},{"updated":"2013-10-12 13:14:51.000000000","updated_by":{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},"reviewer":{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},"state":"REVIEWER"},{"updated":"2013-10-14 00:59:38.000000000","updated_by":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"reviewer":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"state":"REVIEWER"},{"updated":"2013-10-14 13:47:25.000000000","updated_by":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"reviewer":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"state":"REVIEWER"},{"updated":"2013-10-14 13:47:47.000000000","updated_by":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"reviewer":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"state":"REVIEWER"},{"updated":"2013-10-14 14:50:17.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"}],"messages":[{"id":"0e13975cf39c63539e6b58daa3f2427101b754a7","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2013-10-10 23:08:51.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- check-tempest-devstack-vm-full http://logs.openstack.org/67/51067/1/check/check-tempest-devstack-vm-full/9965bfe : SUCCESS in 46m 11s\n- check-tempest-devstack-vm-postgres-full http://logs.openstack.org/67/51067/1/check/check-tempest-devstack-vm-postgres-full/88cfaee : SUCCESS in 47m 39s\n- check-tempest-devstack-vm-neutron http://logs.openstack.org/67/51067/1/check/check-tempest-devstack-vm-neutron/32337c8 : SUCCESS in 23m 12s\n- gate-tempest-devstack-vm-large-ops http://logs.openstack.org/67/51067/1/check/gate-tempest-devstack-vm-large-ops/d2be111 : SUCCESS in 24m 58s\n- check-grenade-devstack-vm http://logs.openstack.org/67/51067/1/check/check-grenade-devstack-vm/5521c2c : SUCCESS in 34m 42s\n- check-swift-devstack-vm-functional http://logs.openstack.org/67/51067/1/check/check-swift-devstack-vm-functional/d90e873 : SUCCESS in 22m 28s\n","accounts_in_message":[],"_revision_number":1},{"id":"30b8fe5365e67d863d0230ad8fb2d36e173cb22b","author":{"_account_id":7051,"name":"Swapnil Kulkarni","email":"skulkarn@redhat.com","username":"swapnilkulkarni2608"},"date":"2013-10-13 12:28:05.000000000","message":"Patch Set 1: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":1},{"id":"54eecf93446ce0070bb4bd6d870885194b4d98bc","author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"date":"2013-10-14 14:50:17.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nif the user changes the ownership on the files, I\u0027m not sure why we\u0027d go back and rechange them","accounts_in_message":[],"_revision_number":1},{"id":"b302211fc0bd8b7d5fbf1b3e3ccaf8d7107e4092","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-14 15:23:48.000000000","message":"Patch Set 1:\n\nSean, I address the problem for below scenario:\n- Update some configuration of the config file by vim, thus I change the file\u0027s ownership.\n- After several days, some services are down.\n- I don\u0027t want to bother to manual fix it, thus I re-run ./stack.sh\n\nThen the error complains.\n\nAs a conclusion, devstack need to update these configuration files, when re-run ./stack.sh","accounts_in_message":[],"_revision_number":1},{"id":"6d9e42cf33d92c3427f8a6ab40ad0d8bf8a6d78c","author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"date":"2013-10-14 15:43:06.000000000","message":"Patch Set 1: Do not merge\n\nThis change un-does the proper ownership on the rootwrap files.\n\nI totally understand that security and devstack are things that usually don\u0027t get said in the same breath, but we shouldn\u0027t be making that worse...","accounts_in_message":[],"_revision_number":1},{"id":"bdd897ab3f290b35c963d95b5497c099cff845d2","author":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"date":"2013-10-14 15:45:24.000000000","message":"Patch Set 1: Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"d9f9e04558ea8647b214322231fd689082162516","revisions":{"d9f9e04558ea8647b214322231fd689082162516":{"kind":"REWORK","_number":1,"created":"2013-10-10 22:13:58.000000000","uploader":{"_account_id":8641,"name":"DennyZhang","email":"filebat.mark@gmail.com","username":"dennyzhang"},"ref":"refs/changes/67/51067/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/devstack","ref":"refs/changes/67/51067/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/51067/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/51067/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/devstack refs/changes/67/51067/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/devstack refs/changes/67/51067/1"}}},"commit":{"parents":[{"commit":"67f7da2de299f4f9ab58718aafce336012035cd8","subject":"Merge \"Use the rdo havana repo with the RHEL family\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/67f7da2de299f4f9ab58718aafce336012035cd8"}]}],"author":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-10 22:13:49.000000000","tz":-300},"committer":{"name":"DennyZhang","email":"denny@unitedstack.com","date":"2013-10-10 22:13:57.000000000","tz":-300},"subject":"make sure devstack change owner for all files in /etc/*/conf","message":"make sure devstack change owner for all files in /etc/*/conf\n\nCurrently devstack only change the ownership of /etc/*/conf,\nwithout changing its children.\n\nThis will incur permission denied error, when re-run ./stack.sh.\n\nCheck bugs.launchpad.net for detail explanation\nCloses-Bug: #1238294\n\nChange-Id: If91092caf266a88168362ae350bd54c8c63f522c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d9f9e04558ea8647b214322231fd689082162516"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/devstack/commit/d9f9e04558ea8647b214322231fd689082162516"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}