Timothy J. Raymond

Results 44 comments of Timothy J. Raymond

I've merged @invidian 's PR to my repro app FYI. That should be a good place to start investigating this.

Could you fill out the description for this PR to document the intentions of this change?

This looks fine to me, but I'll defer approval to @MikeZappa87 and / or @anubhabMajumdar , since they have been more involved with the ongoing work here.

@rbtr My controversial opinion is that we should just remove this linter entirely. Those lines I've seen it flag have generally not been appropriate and nitpicking trivialities in nature. The...

Is it possible to put only the things we like on an allowlist?

Just an update here, we think that after releasing v0.0.9, the minimum supported kernel version is now 5.4. Closing this, as it appears that the original issue is resolved.

@QxBytes is this still an issue?

@huntergregory is this still an issue? It seems like the offending log line has been changed to remove `zap.Any` entirely...

@wenhuwang Can you check to see if v0.0.9 incidentally fixes this issue? I've been attempting to repro this issue to review your associated PR, but I haven't been successful.