Skip to content

Use googles rich error model for grpc errors in Process Manager client#666

Merged
PawelPlesniak merged 2 commits intodevelopfrom
miruna/662-feature-use-googles-rich-error-model-for-grpc-errors-in-process-manager-client
Nov 10, 2025
Merged

Use googles rich error model for grpc errors in Process Manager client#666
PawelPlesniak merged 2 commits intodevelopfrom
miruna/662-feature-use-googles-rich-error-model-for-grpc-errors-in-process-manager-client

Conversation

@miruuna
Copy link
Contributor

@miruuna miruuna commented Nov 6, 2025

Description

Fixes #662

The implementation is identical to that of the Session Manager Client, making use of the extract_grpc_rich_error method.

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future.
    (Indicate issue here: # (issue))

Copy link
Collaborator

@PawelPlesniak PawelPlesniak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with MSQT, and manually by inserting a grpc error into the SSH PM. Errors much clearer.

@PawelPlesniak PawelPlesniak merged commit fe733a1 into develop Nov 10, 2025
4 checks passed
@PawelPlesniak PawelPlesniak deleted the miruna/662-feature-use-googles-rich-error-model-for-grpc-errors-in-process-manager-client branch November 10, 2025 16:34
Copy link
Contributor

@jamesturner246 jamesturner246 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I'm late. Thanks for this. Looks fine. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Use Google's rich error model for gRPC errors in Process Manager Client

4 participants