)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"37b667a37ce2c976da5768e739e945dfc7915670","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a1a7f344_b978b378","updated":"2022-10-20 02:02:24.000000000","message":"I agree we better make the mount path owned by root, but that can be implemented in a much simpler way. Try https://review.opendev.org/c/openstack/puppet-swift/+/861931 .\n\nRegarding the logic to skip audited device, I\u0027m not sure whether that is really within a scope of this module. Even if we support it, the behavior should be optional, otherwise operators can\u0027t use puppet to \"restore\" the device, which is still a valid use case. Also, a smilar logic should be implemented in swift::storage::disk, so that the class do not attempt to format the broken device.","commit_id":"4691cde8364e8d573898cc14d566bdb93a7dee41"}]}