-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes to dbus api methods #76
Conversation
Signed-off-by: mulhern <amulhern@redhat.com>
It's a bit easier to debug, but I think it is also generally slightly better. Signed-off-by: mulhern <amulhern@redhat.com>
This is not ready yet, I will see if the similar can be done with the append calls. |
Also, be less verbose w/ signature information. Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
Signed-off-by: mulhern <amulhern@redhat.com>
@agrover review? It looks like the append*() methods, if used to their fullest extent, would be full of surprises, so I'm just going to rest on this. The last three commits are therefore just bug fixes and minor fixups. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 from me.
review/merge @trgill? just want to make sure you saw the API change, it's consistent with other methods etc.
Document align_to() method
Resolves: stratis-storage#76 Signed-off-by: mulhern <amulhern@redhat.com>
Uses the recommended approach for extracting data from message.
Distinguishes between missing args and invalid types of args.
If invalid type of arg, gives the index of the argument, not its actual value, which I think is just as good.
Also, swaps the order of devs and raidtype in create_pool.