)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"b05e918028f14d93d2d0f96661cfa0faaf7660f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5bd42663_dd281bd0","updated":"2023-02-01 02:42:06.000000000","message":"Add a release note as well.","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"},{"author":{"_account_id":31737,"name":"Hirotaka Wakabayashi","email":"hiwkby@yahoo.com","username":"hiwkby"},"change_message_id":"6814ddcada46469ccd08c570e7c74f493e5da2ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"23ee4806_b68ef1c0","updated":"2023-01-31 07:50:25.000000000","message":"Hello Wu, Thank you for your reply:)","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"},{"author":{"_account_id":31737,"name":"Hirotaka Wakabayashi","email":"hiwkby@yahoo.com","username":"hiwkby"},"change_message_id":"f1dd0360400e8e1152ab7b7c5b3265d6d055244c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"358551ca_a46efe59","updated":"2023-01-27 08:24:32.000000000","message":"Hi Wu! Chould you please check this PR?\n\nTemporary files exist in the `/tmp` directory where only file owner can update them.　Here are easy steps to check this file permission problem:\n\n#1. Let\u0027s login to a Guest VM, and creates a temporary file that `ubuntu(uid is 1000)` owns.\n```\necho \"SELECT 1;\" \u003e /tmp/test.sql\nchmod 777 /tmp/test.sql\n```\n\n#2. Let\u0027s fork a mysql docker container process by `database(uidis 1001` user, mounting `/tmp` directory from Docker host.\n```\ndocker run --rm --name database_test --network bridge --mount \u0027type\u003dbind,src\u003d/tmp,dst\u003d/tmp\u0027 --user \u00271001:1001\u0027 -ti 10.0.0.22:5000/testuser/mysql:5.7.40 /bin/bash\n```\n\n#3. Let\u0027s try updating the temporary file, then you will get `Permission denied`.\n```\nbash-4.2$ echo test | tee -a /tmp/test.sql\ntee: /tmp/test.sql: Permission denied\ntest\nbash-4.2$ ls -l /tmp/test.sql\n-rwxrwxrwx 1 1000 1000 56 Jan 26 16:33 /tmp/test.sql\n```\n","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"13bc8cf9e841eb3087fc04355aa3352c01b0167e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"68e1e91d_ceb47801","in_reply_to":"23ee4806_b68ef1c0","updated":"2023-01-31 09:27:27.000000000","message":"I would like to write a tempest test case for this scenario, but I am really not familiar with tempest.😊😊😊","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"4c242bcd0a15a2637d8f2d3d8457c042bf9c6578","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8517ce8e_90b6161f","in_reply_to":"358551ca_a46efe59","updated":"2023-01-28 06:15:00.000000000","message":"ok, i will review this soon.","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"}],"trove/guestagent/datastore/mysql_common/manager.py":[{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"9c91b190cacad8cf0e757d0300af9364b81af47a","unresolved":true,"context_lines":[{"line_number":222,"context_line":"            }"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"            # Allow database service user to access the temporary files."},{"line_number":225,"context_line":"            for file in [init_file.name, err_file.name]:"},{"line_number":226,"context_line":"                operating_system.chmod(file,"},{"line_number":227,"context_line":"                                       operating_system.FileMode.SET_ALL_RWX(),"},{"line_number":228,"context_line":"                                       force\u003dTrue, as_root\u003dTrue)"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            try:"},{"line_number":231,"context_line":"                self.app.start_db(ds_version\u003dds_version, command\u003dcommand,"}],"source_content_type":"text/x-python","patch_set":2,"id":"651e5ff0_9e1fe8c2","side":"PARENT","line":228,"range":{"start_line":225,"start_character":0,"end_line":228,"end_character":64},"updated":"2023-02-01 02:41:36.000000000","message":"I want to know why these codes don\u0027t work? theoretically all users should have permitting to write these files.","commit_id":"7a62e826a402de29b6a929569dab27adb3259975"},{"author":{"_account_id":26285,"name":"wu.chunyang","email":"wchy1001@gmail.com","username":"wu.chunyang"},"change_message_id":"9c91b190cacad8cf0e757d0300af9364b81af47a","unresolved":true,"context_lines":[{"line_number":263,"context_line":"                )"},{"line_number":264,"context_line":"                docker_util.remove_container(self.app.docker_client)"},{"line_number":265,"context_line":"            except Exception as err:"},{"line_number":266,"context_line":"                LOG.error(\u0027Failed to remove container. error: %s\u0027,"},{"line_number":267,"context_line":"                          str(err))"},{"line_number":268,"context_line":"                pass"},{"line_number":269,"context_line":"            for file in [init_file.name, err_file.name]:"},{"line_number":270,"context_line":"                LOG.debug(\u0027Reverting the %s owner to %s \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"ead6a955_6892d7be","line":267,"range":{"start_line":266,"start_character":0,"end_line":267,"end_character":35},"updated":"2023-02-01 02:41:36.000000000","message":"Actually, we can\u0027t know the reason caused by removing container failed or fetching log failed.","commit_id":"553c578923d30d0eb15ad5dbd0a2c583cae274a4"}]}