Skip to content

Add a generation error hierarchy - #19

Draft
mulkieran wants to merge 1 commit into
stratis-storage:masterfrom
mulkieran:master-generation-error
Draft

Add a generation error hierarchy#19
mulkieran wants to merge 1 commit into
stratis-storage:masterfrom
mulkieran:master-generation-error

Conversation

@mulkieran

@mulkieran mulkieran commented Nov 15, 2018

Copy link
Copy Markdown
Member

Related: #10.

@mulkieran

Copy link
Copy Markdown
Member Author

Looks like we should be catching pyparsing's ParseBaseException. RecursiveGrammarException just doesn't apply here.

Signed-off-by: mulhern <amulhern@redhat.com>
@mulkieran
mulkieran force-pushed the master-generation-error branch from 64ba4cb to c9891e2 Compare November 15, 2018 15:41
@agrover agrover changed the title Add a generation error heirarchy Add a generation error hierarchy Nov 15, 2018
@mulkieran

Copy link
Copy Markdown
Member Author

This PR is still partial, and the new exception isn't being used. It is only relevant if the D-Bus signature is ill-formed and can't be parsed. We can let this go until later.

@mulkieran
mulkieran marked this pull request as draft March 10, 2021 13:51
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.

1 participant