chore(deps): bump google.golang.org/grpc from v1.38.0 to v1.41.1

[blkperl]

Signed-off-by: William Van Hevelingen [email protected]

Don't bother creating a PR until you've done this:

• Run make pre-commit -B to fix codegen, lint, and commit message problems.

Create your PR as a draft.

• Your PR needs to pass the required checks before it can be approved. If the check is not required (e.g. E2E tests) it
  does not need to pass.
• Once required tests have passed, you can make it "Ready for review".
• Say how how you tested your changes. If you changed the UI, attach screenshots.

Tips:

• If changes were requested, and you've made them, then dismiss the review to get it looked at again.
• Add you organization to USERS.md if you like.
• You can ask for help!

[blkperl]

This should unblock dependabot from updating viper in #7645

[terrytangyuan]

LGTM. @alexec Please help double check as well.

[blkperl]

@terrytangyuan any idea on how to fix the failing e2e test or what could be causing it?

[terrytangyuan]

@terrytangyuan any idea on how to fix the failing e2e test?

Perhaps the new version changed the default content type? I’d check the release notes to see if anything else needs to be updated.

[alexec]

Look like it broke the test, that needs fixing.

[alexec]

I wonder if we should change dependant to only update the patch version, too many minor versions seem to be breaking.

[blkperl]

@terrytangyuan @alexec I checked the grpc release notes and nothing stood out to me. I'm not familiar with grpc so I'm not quite sure what to try next.

@alexec re: dependabot only updating patch version I think there will be edge cases like grpc here but most of the time it should work if they don't break backwards compatibility on a minor release. The viper pr only broke because it pulled in the new grpc version because it was an indirect dependency.

I'm also happy to submit prs to update the grpc minor version one at a time until we find which release causes the test failure.

[blkperl]

I asked upstream about 1.42.0 grpc/grpc-go#5175

[blkperl]

@alexec We can merge this in the meantime as it passes on grpc 1.41.1.

Tests pass

[terrytangyuan]

Thank you!