Gofumpt replaces both gofmt and goimports, and does better at both.

Afterwards, I run make format

note about these pr's

They're basically remote-diagnosis of the ci system, which is why there's many of them, they touch many files, and they're similar.

Fairly sure at this point that we've got an intermittent test failure that we can just step aside by tossing in an additional commit.

PR checklist

• Tests written/updated, or no tests needed
• CHANGELOG_PENDING.md updated, or no changelog entry needed
• Updated relevant documentation (docs/) and code comments, or no
  documentation updates needed