)]}'
{"deployment/nova/nova-compute-container-puppet.yaml":[{"author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"change_message_id":"56d5ba8b57389e7c25ff41dceba2b74590b2587c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    description: The container image to use for the nova_libvirt config_volume"},{"line_number":12,"context_line":"    type: string"},{"line_number":13,"context_line":"  DockerNovaComputeUlimit:"},{"line_number":14,"context_line":"    default: [\u0027nofile\u003d1024\u0027, \u0027nproc\u003d126960\u0027]"},{"line_number":15,"context_line":"    description: ulimit for Nova Compute Container"},{"line_number":16,"context_line":"    type: comma_delimited_list"},{"line_number":17,"context_line":"  NovaComputeLoggingSource:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_5a11585c","line":14,"range":{"start_line":14,"start_character":15,"end_line":14,"end_character":27},"updated":"2019-07-01 12:39:51.000000000","message":"sorry I meant the other way round. we need a higher number in the compute container and synced it with cinder-volume where we have 131072 [1]. Otherwise we have issues in case a large number of ceph devices is used on a compute.\n\n[1] https://review.opendev.org/#/c/652048/","commit_id":"f4d230b77929c20618aea48d519b4f55abd9c5bb"}],"deployment/nova/nova-libvirt-container-puppet.yaml":[{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"fe04c1fcbc87f729238777fc15ee14417d7b2a93","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    description: The container image to use for the nova_libvirt config_volume"},{"line_number":14,"context_line":"    type: string"},{"line_number":15,"context_line":"  ContainerNovaLibvirtUlimit:"},{"line_number":16,"context_line":"    default: [\u0027nofile\u003d1024\u0027, \u0027nproc\u003d126960\u0027]"},{"line_number":17,"context_line":"    description: ulimit for Nova Libvirt Container"},{"line_number":18,"context_line":"    type: comma_delimited_list"},{"line_number":19,"context_line":"  ServiceData:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_274c9140","line":16,"range":{"start_line":16,"start_character":22,"end_line":16,"end_character":26},"updated":"2019-07-01 10:50:10.000000000","message":"shall I as well adjust these for the DockerNovaCompute Ulimits?","commit_id":"f12e254ec11e7e6232962537a6d637b72e5e4172"},{"author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"change_message_id":"909f11a647cd2e2d0e59e2fc5536a4b74ffb4624","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    description: The container image to use for the nova_libvirt config_volume"},{"line_number":14,"context_line":"    type: string"},{"line_number":15,"context_line":"  ContainerNovaLibvirtUlimit:"},{"line_number":16,"context_line":"    default: [\u0027nofile\u003d1024\u0027, \u0027nproc\u003d126960\u0027]"},{"line_number":17,"context_line":"    description: ulimit for Nova Libvirt Container"},{"line_number":18,"context_line":"    type: comma_delimited_list"},{"line_number":19,"context_line":"  ServiceData:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_3a3824d1","line":16,"range":{"start_line":16,"start_character":22,"end_line":16,"end_character":26},"in_reply_to":"9fb8cfa7_274c9140","updated":"2019-07-01 11:47:17.000000000","message":"yes I think it would be good to keep the file limit in sync.\n\n[1] https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/nova/nova-compute-container-puppet.yaml#L14","commit_id":"f12e254ec11e7e6232962537a6d637b72e5e4172"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"0e982f4be4a66faa83ef6df6566bea9de3df6094","unresolved":false,"context_lines":[{"line_number":601,"context_line":"              preserve_properties: true"},{"line_number":602,"context_line":"      docker_config:"},{"line_number":603,"context_line":"        step_3:"},{"line_number":604,"context_line":"          nova_virtlogd:"},{"line_number":605,"context_line":"            start_order: 0"},{"line_number":606,"context_line":"            image: {get_param: ContainerNovaLibvirtImage}"},{"line_number":607,"context_line":"            net: host"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_7a5c5c7c","line":604,"updated":"2019-07-01 12:23:54.000000000","message":"Can the ulimit be applied to nova_virtlogd container too? All the libvirt containers having the same configuration.","commit_id":"bbc5fc204bac82208dc63060e3db7a9a3ac58821"}]}