Skip to content

Conversation

@mulkieran
Copy link
Member

@mulkieran mulkieran commented Nov 27, 2023

Related #1033

@mulkieran mulkieran self-assigned this Nov 27, 2023
@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch 2 times, most recently from f60cbfb to b166f92 Compare November 27, 2023 18:58
@mulkieran
Copy link
Member Author

mulkieran commented Nov 27, 2023

make_dyn_class should memo-ize the dynamically generated classes, and should take a list of keys.

@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch 5 times, most recently from 3cb269c to 69828b1 Compare November 28, 2023 14:25
@mulkieran
Copy link
Member Author

The invocations of make_dyn_class are too wordy. They should be super simple, just one arg which very closely resembles the class name. make_dyn_class should then interpret that one argument.

@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch from 69828b1 to 93c34b1 Compare November 29, 2023 19:50
@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch from 0f77dff to 20e4e26 Compare December 4, 2023 15:11
@mulkieran
Copy link
Member Author

Need to add into a test that invokes CreatePool.

Signed-off-by: mulhern <[email protected]>
@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch from 67fb6bf to 513ce91 Compare December 4, 2023 18:29
Signed-off-by: mulhern <[email protected]>
@mulkieran mulkieran force-pushed the issue_stratis-cli_1033 branch from 513ce91 to 044e083 Compare December 4, 2023 18:44
@mulkieran
Copy link
Member Author

This is not the correct way to do this. It should really be possible to make dynamic self-generating classes, and that would require working with dbus-python-client-gen.

@mulkieran
Copy link
Member Author

Obsoleted by the better idea of instantiating the whole class with only the methods needed for that invocation. Would still have to remember to add the absolute path check.

@mulkieran mulkieran closed this Jan 15, 2026
@mulkieran mulkieran deleted the issue_stratis-cli_1033 branch January 15, 2026 18:55
@mulkieran mulkieran removed this from tracking Jan 15, 2026
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