Nahum Shalman

@arp242 Thank you for taking a look at this. My biggest concern at the moment is that under stress tests I've been able to trigger the panic in my Event...

> Merging is a little bit easier, and with "squash and merge" it shows up as one commit in the main branch and we can rewrite the commit message. I...

> Oh, and the tests fail; output with some comments added. > > The sub directory is not watched; we should not see changes to /sub/file2. > > ``` >...

> Also, should maybe look at setting up a Solaris runner too; I don't think Solaris and illumos have shared much (or any?) code since OpenSolaris got Oracle'd? > >...

> fallout from [tinkerbell/tink#584](https://github.com/tinkerbell/tink/pull/584) Caused by that PR, or resolved by it?

No action yet on https://github.com/bmc-toolbox/bmclib/issues/233

Rather than make this configurable, perhaps we should just be changing the defaults for Windows?? It feels odd for an API like fsnotify to have options that only get used...

Any updates on this?

@davepacheco if you have time to dig in to this again, I'd recommend testing something like 1.19.2. Also, it's very very very unlikely to be related, but just to eliminate...

> On using 1.19.2: is that on general principles of trying on latest, or is there a particular fix that you're thinking of that might be related? General principles; as...