Michelangelo Mori

Write an integration test covering the code path that goes from Minder receiving a webhook event to Minder opening a PR. The test should cover * webhook handler * transmission...

Here's a list of endpoints not covered by smoke tests, ordered by least-covered-first (in terms of unit tests). Note that coverage of routines used by these handlers might be different...

Here's a list of endpoints not covered by smoke tests, ordered by least-covered-first (in terms of unit tests). Note that coverage of routines used by these handlers might be different...

Add request id to logs.

4

# Summary This change adds a GRPC interceptor that fetches the header `Grpc-Metadata-Request-Id` and adds it to the logger. It also returns the request id to the client in the...

Handle GitHub throttling errors properly

1

GitHub occasionally throttles calls returning a wealth of information on how to correctly handle the issue ``` 403 API rate limit of 5000 still exceeded until 2024-10-02 09:30:01 +0000 UTC,...

We want both end users and our smoke tests to be able to report which requests failed, and then track down logs using a shared request id. Some work was...

There are spots in the codebase where we report information about the same error or action multiple times throughout the call stack. Wherever sensible * log only in top level...

We want the CLI to send request ids with all requests * create and send request ids on all RPCs * add support in Minder CLI for a `-v/--verbose` global...

Remove unnecessary logs from Minder codebase, the rule of thumb is * most `debug` messages are unnecessary or useful * if a `debug` message is truly necessary or useful, bump...

### Describe the issue Updating `minder-config` in Kubernetes does not trigger a restart of the service as it should. ### To Reproduce _No response_ ### What version are you using?...