stratis-cli icon indicating copy to clipboard operation
stratis-cli copied to clipboard

Clevis pin tidies

Open mulkieran opened this issue 1 year ago • 4 comments

mulkieran avatar Mar 18 '24 19:03 mulkieran

rebased

mulkieran avatar Mar 26 '24 17:03 mulkieran

Probably the solution to the problem of the incredibly messy Clevis configuration code, is to define a subclass of Action with a global object internally to store all the different Clevis stuff and then to define subclasses of that class for each possible option and then incrementally build that object as parser arguments are parsed. Even that is done, it will probably still be necessary to run a check in _main.py, just on the object produced by the parser rather than on individual namespace values.

mulkieran avatar Mar 27 '24 14:03 mulkieran

Should tidy up argument parsing first, probably.

mulkieran avatar Apr 22 '24 13:04 mulkieran

Blocked by https://github.com/stratis-storage/stratis-cli/pull/1073 .

mulkieran avatar Apr 26 '24 14:04 mulkieran