@rshriram I noticed the following line is missing coverage in the fault filter. Do you mind adding a test for that case? It's pretty important to make sure the timer gets correctly disabled, etc.
https://github.com/lyft/envoy/blob/master/source/common/http/filter/fault_filter.cc#L121