)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"change_message_id":"b5cd8849ee6880414bbdd5e200d2c6a909a45d74","unresolved":false,"context_lines":[{"line_number":9,"context_line":"After running of glance role, the owner and group of NFS local folder"},{"line_number":10,"context_line":"is root instead of glance."},{"line_number":11,"context_line":"Moreover, the location that the image is going to be registered is"},{"line_number":12,"context_line":"under to \u0027v2/images\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The ownership of the local NFS folder is updated and the folder of"},{"line_number":15,"context_line":"the \u0027images\u0027 has been moved under folder \u0027v2\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f79a3b5_8f446678","line":12,"range":{"start_line":12,"start_character":10,"end_line":12,"end_character":20},"updated":"2018-10-30 19:56:55.000000000","message":"I think you are confusing the REST API url with the location where images should be stored. https://192.168.122.220:9292/v2/images/877b3ccc-4bf6-4946-8b28-f6efa7195cad/file has nothing to do with the filesystem where the image will be stored, this has to do with how it is passing variable to REST API","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"},{"author":{"_account_id":27749,"name":"Panagiotis Karalis","email":"pkaralis@intracom-telecom.com","username":"pkaralis"},"change_message_id":"d44015ce70ae7682789206cc52ec741e69efaf88","unresolved":false,"context_lines":[{"line_number":9,"context_line":"After running of glance role, the owner and group of NFS local folder"},{"line_number":10,"context_line":"is root instead of glance."},{"line_number":11,"context_line":"Moreover, the location that the image is going to be registered is"},{"line_number":12,"context_line":"under to \u0027v2/images\u0027."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"The ownership of the local NFS folder is updated and the folder of"},{"line_number":15,"context_line":"the \u0027images\u0027 has been moved under folder \u0027v2\u0027."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3f79a3b5_32c80375","line":12,"range":{"start_line":12,"start_character":10,"end_line":12,"end_character":20},"in_reply_to":"3f79a3b5_8f446678","updated":"2018-10-31 16:10:07.000000000","message":"The reason that I have pushed this patch is due to image registration. Specifically, When I was trying to register an image, the registration was failing (with a message that giving me the impression that the folder was not the proper one). Once, I manually created and used the \u0027v2/images\u0027 path instead of the \u0027images\u0027, the image had stored normally under the \u0027v2/images\u0027 path. I can collect logs about this behavior for sure. Maybe, my environment was not the updated , since I\u0027m working using my fork for releng-xci and maybe I\u0027ve lost some updates. I don\u0027t know. \nSo, this patch is actually an improvement even if the filesystem path should be updated in the future for a specific scenario or something. Otherwise, the default value is remaining unaffected. \nThat was the whole idea behind of this patch!","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"}],"defaults/main.yml":[{"author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"change_message_id":"b5cd8849ee6880414bbdd5e200d2c6a909a45d74","unresolved":false,"context_lines":[{"line_number":218,"context_line":"# Define nfs information to enable nfs shares as mounted directories for"},{"line_number":219,"context_line":"# glance. The ``glance_nfs_client`` value is a list of dictionaries that must"},{"line_number":220,"context_line":"# be filled out completely to enable the persistent NFS mounts."},{"line_number":221,"context_line":"glance_nfs_local_path: \"/var/lib/glance/images\""},{"line_number":222,"context_line":"glance_nfs_client: []"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"# Example of the expected dict structure:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_0faa961a","line":221,"range":{"start_line":221,"start_character":0,"end_line":221,"end_character":47},"updated":"2018-10-30 19:56:55.000000000","message":"We are already using a variable: https://github.com/openstack/openstack-ansible-os_glance/blob/master/templates/glance-api.conf.j2#L87","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"},{"author":{"_account_id":27749,"name":"Panagiotis Karalis","email":"pkaralis@intracom-telecom.com","username":"pkaralis"},"change_message_id":"d44015ce70ae7682789206cc52ec741e69efaf88","unresolved":false,"context_lines":[{"line_number":218,"context_line":"# Define nfs information to enable nfs shares as mounted directories for"},{"line_number":219,"context_line":"# glance. The ``glance_nfs_client`` value is a list of dictionaries that must"},{"line_number":220,"context_line":"# be filled out completely to enable the persistent NFS mounts."},{"line_number":221,"context_line":"glance_nfs_local_path: \"/var/lib/glance/images\""},{"line_number":222,"context_line":"glance_nfs_client: []"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"# Example of the expected dict structure:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_12d067f4","line":221,"range":{"start_line":221,"start_character":0,"end_line":221,"end_character":47},"in_reply_to":"3f79a3b5_0faa961a","updated":"2018-10-31 16:10:07.000000000","message":"I have put it there because of my previous reply. Just the backward compatibility to be kept. Giving also the flexibility to define a different path if this is needed using the \u0027glance_nfs_local_path\u0027 variable in the \u0027openstack_user_config.yml\u0027 file of respective role.","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"}],"tasks/glance_install.yml":[{"author":{"_account_id":21883,"name":"Manuel Buil","email":"mbuil@suse.com","username":"mbuil"},"change_message_id":"b5cd8849ee6880414bbdd5e200d2c6a909a45d74","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    - path: \"{{ glance_system_user_home }}/scrubber\""},{"line_number":77,"context_line":"    - path: \"{{ glance_system_user_home }}/images\""},{"line_number":78,"context_line":"      mode: \"0755\""},{"line_number":79,"context_line":"      owner: \"glance\""},{"line_number":80,"context_line":"      group: \"glance\""},{"line_number":81,"context_line":"    - path: \"{{ glance_system_user_home }}/v2/images\""},{"line_number":82,"context_line":"      mode: \"0755\""},{"line_number":83,"context_line":"      owner: \"glance\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_eff2ba69","line":80,"range":{"start_line":79,"start_character":0,"end_line":80,"end_character":21},"updated":"2018-10-30 19:56:55.000000000","message":"the owner and group is already glance, this is not needed. The reason why this does not work and becomes root is because the NFS mount is pointing to a wrong address. Check the previous paths (e.g. /var/lib/glance/cache/) and you will see that it has glance as owner and group. Deploy without HA and you will see that things work","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"},{"author":{"_account_id":27749,"name":"Panagiotis Karalis","email":"pkaralis@intracom-telecom.com","username":"pkaralis"},"change_message_id":"d44015ce70ae7682789206cc52ec741e69efaf88","unresolved":false,"context_lines":[{"line_number":76,"context_line":"    - path: \"{{ glance_system_user_home }}/scrubber\""},{"line_number":77,"context_line":"    - path: \"{{ glance_system_user_home }}/images\""},{"line_number":78,"context_line":"      mode: \"0755\""},{"line_number":79,"context_line":"      owner: \"glance\""},{"line_number":80,"context_line":"      group: \"glance\""},{"line_number":81,"context_line":"    - path: \"{{ glance_system_user_home }}/v2/images\""},{"line_number":82,"context_line":"      mode: \"0755\""},{"line_number":83,"context_line":"      owner: \"glance\""}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_38e9c5e3","line":80,"range":{"start_line":79,"start_character":0,"end_line":80,"end_character":21},"in_reply_to":"3f79a3b5_eff2ba69","updated":"2018-10-31 16:10:07.000000000","message":"This is very weird....\nAt the beginning I had the same impression. But, after some tests, I saw that the same behavior is happening also for environment with \u0027mini\u0027 flavor. \nThat\u0027s why I thought that finally could be a missing impact. Without be sure whether this missing impact was capable to have the same behavior also to other flavors. Meaning, maybe to other flavors have this behavior (e.g. root:root) , but the image registration was completed normally.\nLet\u0027s check the behavior after the NFS fixing...maybe indeed this is the root of the whole problem!:)","commit_id":"147ad78151c066f39653bcbb29675ecf4d2c416a"}]}