-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Closed
Description
#10163 which resolved #9359 added support for inotify watching of directories to catch symlink rotation. I think this might have a bug. Specifically, if we have a cert in /foo/bar/cert.pem and perform a symlink rotation on bar, we don't see any SDS filesystem action.
The reason is that the watch is added on bar, rather than foo, see
envoy/source/common/secret/sds_api.cc
Line 68 in f95f539
| watcher_->addWatch(absl::StrCat(result.directory_, "/"), |
@tsaarni can you confirm that this is an issue? I'm surprised it worked in your Contour example, but maybe some other feature was triggering the inotify, e.g. the file deletes?
I think the solution is to watch the grandparent, rather than the parent directory.
Reactions are currently unavailable