)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"5ec4b10f5772cb93e0d6df050e0d1f9d870ed6c7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Michal (inc0) Jastrzebski \u003cinc007@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-10-14 17:38:30 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix nova  upgrade permission issue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I61738f2d8ef89d6a8c5fb1187f18c5bb154bbe04"},{"line_number":10,"context_line":"Partial-Bug: #1631503"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa6399be_029e4eb8","line":7,"updated":"2016-10-16 14:22:42.000000000","message":"remove one space.","commit_id":"7b9133cc1db1e0f2e582162802c7dae85b1a7176"}],"ansible/roles/nova/templates/nova-compute.json.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"6e90adfc05e23425822810f49b9ad07dc31b2486","unresolved":false,"context_lines":[{"line_number":21,"context_line":"            \"recurse\": true"},{"line_number":22,"context_line":"        },"},{"line_number":23,"context_line":"        {"},{"line_number":24,"context_line":"            \"path\": \"/var/lib/nova\","},{"line_number":25,"context_line":"            \"owner\": \"nova:nova\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"fa6399be_022f2e3f","line":24,"updated":"2016-10-16 14:36:36.000000000","message":"files in /var/lib/nova should take care. not all files are owned by nova. like disk and console.log and libvirt.xml","commit_id":"7b9133cc1db1e0f2e582162802c7dae85b1a7176"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"422dfb167068f8b7bdd74d65fd31ea70373957b6","unresolved":false,"context_lines":[{"line_number":21,"context_line":"            \"recurse\": true"},{"line_number":22,"context_line":"        },"},{"line_number":23,"context_line":"        {"},{"line_number":24,"context_line":"            \"path\": \"/var/lib/nova\","},{"line_number":25,"context_line":"            \"owner\": \"nova:nova\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"fa6399be_4bf75a8c","line":24,"in_reply_to":"fa6399be_022f2e3f","updated":"2016-10-17 18:42:26.000000000","message":"however we run libvirt as root, so it will have access to nova-owned files regardless. I\u0027ll spawn vm and see once more how permissions looks like. Nova might require access to things like console-log for example.","commit_id":"7b9133cc1db1e0f2e582162802c7dae85b1a7176"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"38cfbf79cd8d4c240f94126bf6ee9df43f99b0ee","unresolved":false,"context_lines":[{"line_number":21,"context_line":"            \"recurse\": true"},{"line_number":22,"context_line":"        },"},{"line_number":23,"context_line":"        {"},{"line_number":24,"context_line":"            \"path\": \"/var/lib/nova\","},{"line_number":25,"context_line":"            \"owner\": \"nova:nova\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"fa6399be_265edb76","line":24,"in_reply_to":"fa6399be_4bf75a8c","updated":"2016-10-17 19:10:49.000000000","message":"drwxr-xr-x 2 nova nova  4096 Oct 17 19:01 .\ndrwxr-xr-x 5 nova nova  4096 Oct 17 19:01 ..\n-rw-r--r-- 1  108  109 19237 Oct 17 19:02 console.log\n-rw-r--r-- 1 nova nova  2948 Oct 17 19:01 libvirt.xml\n\nlibvirt.xml is owned by nova, console.log is owned by libvirt...this is going to be interesting","commit_id":"7b9133cc1db1e0f2e582162802c7dae85b1a7176"}],"ansible/roles/nova/templates/nova-ssh.json.j2":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"5ec4b10f5772cb93e0d6df050e0d1f9d870ed6c7","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            \"perm\": \"0600\""},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ],"},{"line_number":29,"context_line":"    \"permissions\": ["},{"line_number":30,"context_line":"        {"},{"line_number":31,"context_line":"            \"path\": \"/var/log/kolla/nova\","},{"line_number":32,"context_line":"            \"owner\": \"nova:nova\","}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"fa6399be_22a192f8","line":29,"updated":"2016-10-16 14:22:42.000000000","message":"remove this. nova-ssh ran by root user.","commit_id":"7b9133cc1db1e0f2e582162802c7dae85b1a7176"}]}