)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"7caca6ac17dd9a259a8820116ae481d9f723112f","unresolved":true,"context_lines":[{"line_number":14,"context_line":"permission to create the jobs_dir directory under the /var/lib/freezer"},{"line_number":15,"context_line":"directory."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[1] https://github.com/openstack/kolla/commit/2daf4331a648cc2df6982c1a6ec47a705e038255"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #2031953"},{"line_number":20,"context_line":"Change-Id: I360990096042e857813b4890bddd67dd4bed99e4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"d4e46a46_cb9a2e96","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":18},"updated":"2023-08-21 08:47:52.000000000","message":"Hi,\n\nthanks for your fix, but please use opendev urls, not urls from the external github mirror, thank you.\n\nthat would be https://review.opendev.org/c/openstack/kolla/+/722102\nor https://opendev.org/openstack/kolla/commit/2daf4331a648cc2df6982c1a6ec47a705e038255\n\nthanks!","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"16e82e60400453faefeeecd23078cc23f2a6ad94","unresolved":false,"context_lines":[{"line_number":14,"context_line":"permission to create the jobs_dir directory under the /var/lib/freezer"},{"line_number":15,"context_line":"directory."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"[1] https://github.com/openstack/kolla/commit/2daf4331a648cc2df6982c1a6ec47a705e038255"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Closes-Bug: #2031953"},{"line_number":20,"context_line":"Change-Id: I360990096042e857813b4890bddd67dd4bed99e4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"0820669d_7f1f6956","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":18},"in_reply_to":"d4e46a46_cb9a2e96","updated":"2023-08-22 00:55:49.000000000","message":"Done","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b996433b5a6b290cda04ecf8a26b0454b4f64ef1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"556ceb0d_c20eefa2","updated":"2023-08-19 11:06:10.000000000","message":"Reno missing","commit_id":"ac2a079a4a8ab4b5a4bcf7c80bfccc8367185342"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"1c91ff562ba8697486c9e0682983c26da8769bf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a58a84a5_a5ee7829","updated":"2023-08-21 06:10:33.000000000","message":"recheck","commit_id":"35a43d23fa8dd879c14969a892a2cc9bdbf80d2e"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"16e82e60400453faefeeecd23078cc23f2a6ad94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"50d71150_8967f5f8","updated":"2023-08-22 00:55:49.000000000","message":"Thank you very much for your advice.","commit_id":"2b24337f428a35610b9a3e1f3da8be4e9f991746"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"dd5e25d7ec2d45ef9aff6fdaf00cede27868cf44","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"dd350c5f_e0df1737","updated":"2023-08-24 07:43:03.000000000","message":"lgtm","commit_id":"88941fc65fd1f6d6f85c5a4e6c4ba990cc9f6b53"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"da0efacceca58e0ca75c07d058c5fa4aa7969973","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"3ea32496_fbb3e17c","updated":"2023-08-26 02:32:48.000000000","message":"recheck","commit_id":"88941fc65fd1f6d6f85c5a4e6c4ba990cc9f6b53"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"075b7026865b7d61e9878b671a95aeab2379e8c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d63a68ae_de583b4e","updated":"2023-08-26 06:02:11.000000000","message":"recheck","commit_id":"88941fc65fd1f6d6f85c5a4e6c4ba990cc9f6b53"}],"ansible/roles/freezer/templates/freezer-scheduler.json.j2":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"69b52aecacaa40fb948f8add1087234101e5a4dd","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"d5a1d42b_faa8c510","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"updated":"2023-08-23 07:48:59.000000000","message":"normally a configuration directory of a service is not directly owned by that service, but by root. this is done in order to prevent attacks when the service user is compromised.\n\nplease change this to a sub directory of `/etc/freezer/` which would be `/etc/freezer/scheduler` in this case.\n\nThanks","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"f6a05927189cc416263884c35f4a98f408a11dd9","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"a844f2df_05b70d2a","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"45154d1e_1e241b8a","updated":"2023-08-23 10:48:17.000000000","message":"hi, Maksim i add following config in config_file section in freezer-scheduler.json.j2:\n\n        {\n            \"source\": \"{{ container_config_directory }}/scheduler\",\n            \"dest\": \"/etc/freezer/scheduler\",\n            \"owner\": \"freezer\",\n            \"perm\": \"0755\"\n        }\nand i tested with optional \"true\" and optional \"false\" respectively. When optional is true, kolla_set_configs will not do anything. When optional is false, \"/var/lib/kolla/config_files/scheduler file is not found\" will appear mistake. I really don\u0027t know what to do next, Maksim, i will appreciate you very much if you can give me some valuable advice.","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"45c92e190073c731f710d7b4bbbfcce52be5b64b","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"792374a6_aeed9bbd","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"5a34c66a_1460b28a","updated":"2023-08-23 08:43:30.000000000","message":"Sven and Maksim, thank you very much for your review. I tested in the openstack environment, it doesn\u0027t work, because the /etc/freezer/scheduler directory is created during the startup of the freezer-scheduler process, and kolla_set_configs will not do nothing when /etc/freezer/scheduler directory does not exist. So,when the freezer-scheduler process starts, there will also be no permission to create the jobs_dir directory in the /etc/freezer directory.","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"00992bbdfafd855501eb59cbfb9e9d0faf8fd70c","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"45154d1e_1e241b8a","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"792374a6_aeed9bbd","updated":"2023-08-23 09:19:08.000000000","message":"sure, there is need config_files element too","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"88f47489ae3904b96c56ec017e6d08d1ceeaec95","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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"bb74ece4_2eda6e02","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"7badd6fd_6b125c6c","updated":"2023-08-24 07:26:16.000000000","message":"hi, maksim, i create a patch in the kolla repo, and made them related-bug, appriciate for your review again.","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"65f93f33b674997605021b785add513ea6549d8f","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"7badd6fd_6b125c6c","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"a844f2df_05b70d2a","updated":"2023-08-23 11:18:39.000000000","message":"sure. we need another dependent change in the kolla repo with an empty directory structure for this. don\u0027t forget to make them related-bug in the commit message as well.","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"cec3760f84842c4b6026ddbd1ed5929ef7f7bd13","unresolved":true,"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\": \"/etc/freezer\","},{"line_number":25,"context_line":"            \"owner\": \"freezer:freezer\","},{"line_number":26,"context_line":"            \"recurse\": true"},{"line_number":27,"context_line":"        }"},{"line_number":28,"context_line":"    ]"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"5a34c66a_1460b28a","line":25,"range":{"start_line":24,"start_character":12,"end_line":25,"end_character":39},"in_reply_to":"d5a1d42b_faa8c510","updated":"2023-08-23 08:02:15.000000000","message":"path: /etc/freezer/scheduler\nowner: freezer:freezer\nperm: 0770","commit_id":"ad58083b6bf84c3f9bb124decc79f012afb1d3df"}],"ansible/roles/freezer/templates/freezer.conf.j2":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"7caca6ac17dd9a259a8820116ae481d9f723112f","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"6fdb45d2_6e3a5b6e","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"updated":"2023-08-21 08:47:52.000000000","message":"I\u0027m not sure if it\u0027s correct to just use another directory here, but I have not much experience with \"freezer\" myself.\n\nWouldn\u0027t it be better if freezer has write access to the `/etc/freezer/scheduler/`\ndirectory?\n\nWhat is the content of the scheduler\u0027s conf.d directory, or the better question is, which user will write there?","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"0957fd528562e09002ca4e016f6664397146a14a","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"74d0a397_3051818a","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"3f713d15_f5d54e3c","updated":"2023-08-22 11:51:38.000000000","message":"no, you should change the ansible/roles/freezer/templates/freezer-scheduler.json.j2 file instead of ansible/roles/freezer/templates/freezer.conf.j2","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"746ba2ebee1b747d26290c4673196ffc9ba3d86c","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"3f713d15_f5d54e3c","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"5766779d_b2844950","updated":"2023-08-22 11:34:40.000000000","message":"Michal and Maksim, thank you very much for your review and suggestions. According to your suggestions, should I abandon this patch, and then patch the kolla project to add permissions to the /etc/freezer directory(like \"chown -R freezer:freezer /etc/freezer\") of the freezer-scheduler docker image, or modify the freezer code to elevate permissions when creating jobs_dir?","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"0b3c7fe5d9e4f00f7c3e3ff154dc812755d08233","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"5766779d_b2844950","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"57ed1874_69757eb3","updated":"2023-08-22 10:38:53.000000000","message":"sure, documented and here is the code:\nhttps://github.com/openstack/freezer/blob/master/freezer/scheduler/arguments.py#L29\nso imho we shouldn\u0027t change the default but create directory with correct permissions in the ansible/roles/freezer/templates/freezer-scheduler.json.j2 only.","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"410f721ef450fc8d04849dbd1fff77586d3ed238","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"d1debeac_efee97d4","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"66b216e0_9d440538","updated":"2023-08-22 09:07:31.000000000","message":"I would leave the default (no need to define it at all, as it\u0027s the freezer default) and allow freezer user to write in that directory?","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"e4cbbd4547aa688847dfe17be9369c8ca7ce3c32","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"66b216e0_9d440538","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"6fdb45d2_6e3a5b6e","updated":"2023-08-22 01:13:00.000000000","message":"In all honesty, I\u0027m not familiar with the freezer project either, and I discovered this issue when deploying freezer using kolla-ansible. From the freezer-scheduler logs, freezer-scheduler creates the scheduler/conf.d directory in the /etc/freezer directory using the freezer account at startup, this fails due to lack of permissions, and when we change the directory to /var/lib/ freezer, this does work. Also, should we add one of the core member of the freezer project to the attention set?","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":23303,"name":"howardlee","email":"lihongweibj@inspur.com","username":"howardlee"},"change_message_id":"22f4dd47079855ce10f4d23abb3c487b43c2bb31","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"11bd0d2f_84117657","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"74d0a397_3051818a","updated":"2023-08-23 01:52:35.000000000","message":"Maksim, thank you very much for your valuable comments. I have modified this patch and submitted it. I look forward to your review again, thanks again.","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"8d8c1666401bde89fda196d4b90175f9030d0091","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"57ed1874_69757eb3","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"9085e7d7_04df9a38","updated":"2023-08-22 10:19:06.000000000","message":"https://docs.openstack.org/freezer/latest/cli/freezer-scheduler.html#DEFAULT.jobs_dir - this is the documented default - isn\u0027t it?","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"91134d50b4d9d934fd72b871e6151ac112778c90","unresolved":true,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"{% if service_name \u003d\u003d \u0027freezer-scheduler\u0027 %}"},{"line_number":12,"context_line":"client_id \u003d {{ inventory_hostname }}"},{"line_number":13,"context_line":"jobs_dir \u003d /var/lib/freezer/scheduler/conf.d"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"os_username \u003d {{ openstack_auth.username }}"},{"line_number":16,"context_line":"os_password \u003d {{ openstack_auth.password }}"}],"source_content_type":"text/x-jinja2","patch_set":4,"id":"9085e7d7_04df9a38","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":11},"in_reply_to":"d1debeac_efee97d4","updated":"2023-08-22 09:32:58.000000000","message":"there is no defaults in the freezer for CONF.jobs_dir, so it used from the scheduler.conf by oslo. the recommended value is /etc/freezer/scheduler/conf.d but this doesn\u0027t correlates with Kolla architecture. IMHO the bugfix should change only ansible/roles/freezer/templates/freezer-scheduler.json.j2 and create the directory with correct permissions only.","commit_id":"e53d39156b310559e20d4c2311fafc5ab2cb9b50"}]}