)]}'
{"releasenotes/notes/8_0_0-535e0fed9755d83c.yaml":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"cd9463353f62421a89848cdb042ac50728327055","unresolved":true,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"  - |"},{"line_number":21,"context_line":"    The agent processes inside storlet containers now accept SIGUSER1 signal"},{"line_number":22,"context_line":"    and shutdown gracefully."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  - |"},{"line_number":25,"context_line":"    The new ``max_containers_per_node`` option has been added to the docker"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"433227b5_e33c67da","line":22,"updated":"2021-09-14 02:30:19.000000000","message":"I may be missing this change, which commit are you meaning?","commit_id":"f4781709697af0c6cd0ca85f0a7430c8b680bdc6"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"256c9a3d56492a257aacd29b2c55379699c05987","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"  - |"},{"line_number":21,"context_line":"    The agent processes inside storlet containers now accept SIGUSER1 signal"},{"line_number":22,"context_line":"    and shutdown gracefully."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"  - |"},{"line_number":25,"context_line":"    The new ``max_containers_per_node`` option has been added to the docker"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"401eabbf_a0d19dc9","line":22,"in_reply_to":"433227b5_e33c67da","updated":"2021-09-14 02:50:40.000000000","message":"I intended https://review.opendev.org/c/openstack/storlets/+/671057 but I now noticed I mentioned the wrong singal(should be SIGHUP instead of SIGUSR1). I\u0027ve fixed this.","commit_id":"f4781709697af0c6cd0ca85f0a7430c8b680bdc6"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"cd9463353f62421a89848cdb042ac50728327055","unresolved":true,"context_lines":[{"line_number":25,"context_line":"    The new ``max_containers_per_node`` option has been added to the docker"},{"line_number":26,"context_line":"    gateway configuration file. This option limits the number of storlet"},{"line_number":27,"context_line":"    containers in a single node, and would be useful to distribute containers"},{"line_number":28,"context_line":"    among multiple nodes."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"upgrade:"},{"line_number":31,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"03af2dc3_0b115a38","line":28,"updated":"2021-09-14 02:30:19.000000000","message":"Perhaps, we may have note here about that this would affect to user experience when no enough nodes available for container launch.","commit_id":"f4781709697af0c6cd0ca85f0a7430c8b680bdc6"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"256c9a3d56492a257aacd29b2c55379699c05987","unresolved":false,"context_lines":[{"line_number":25,"context_line":"    The new ``max_containers_per_node`` option has been added to the docker"},{"line_number":26,"context_line":"    gateway configuration file. This option limits the number of storlet"},{"line_number":27,"context_line":"    containers in a single node, and would be useful to distribute containers"},{"line_number":28,"context_line":"    among multiple nodes."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"upgrade:"},{"line_number":31,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5197a8fd_9d8983cf","line":28,"in_reply_to":"03af2dc3_0b115a38","updated":"2021-09-14 02:50:40.000000000","message":"That\u0027s a good point. I\u0027ve added a note about the potential issue with \"too low limit\"","commit_id":"f4781709697af0c6cd0ca85f0a7430c8b680bdc6"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"cd9463353f62421a89848cdb042ac50728327055","unresolved":true,"context_lines":[{"line_number":32,"context_line":"    The ``restart_docker_container`` script has been removed, and now"},{"line_number":33,"context_line":"    the storlet_handler middleware directly communicates with the docker daemon"},{"line_number":34,"context_line":"    to manage docker containers. Because of this change, now the user to run"},{"line_number":35,"context_line":"    swift processes (which is usually the ``swift`` user) should belong to"},{"line_number":36,"context_line":"    the ``docker`` group."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"  - |"},{"line_number":39,"context_line":"    Now ``managed_by\u003dstorlets`` is added as a tag to all storlet containers,"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"822eadf2_f2a35818","line":36,"range":{"start_line":35,"start_character":0,"end_line":36,"end_character":25},"updated":"2021-09-14 02:30:19.000000000","message":"Yes. good docs. this is new constraint on storlets configuration.","commit_id":"f4781709697af0c6cd0ca85f0a7430c8b680bdc6"}]}