jscotka
jscotka
I suggest to let packit.yaml in dowstream, I think this is good. But my idea is, to not sync it only as file, but also store there information about state....
I'm also not clear what could be better For me, it would be nice, in case of error revert to last working solution (as I think it is implemented inside...
@garrett I understand your PoV, missing info are hidden inside this bugzilla. So some example setup: - NFS is mounted, - root cause is missing direcory structure under this mount...
> Hi @garrett NFS is mounted properly, just directories there does not exists.  There is more solutions how I can imagine to fix it. - Fix...
@garrett your proposal screenskot seem very well and clean.
maybe one more - [ ] create issues based on ``TODO`` comments and link them to code or remove them from code if not necessary
for example methods like: * ``__repr__`` * ``__str__`` * ``inspect`` * ``list_all`` * ``cleanup`` for list all of them, list all ``TODOs`` comments in nspawn section with text like ``API``...
Yes, this is little bit tricky part, that's why it is easier, to use ``run`` as alias for ``run_via_binary`` and just in case some backend don't have binary interface and...
Okay, I understand this PoV. and I'm familiar to avoid similar solution. My example were just, an example that this solution could be very simple, but tricky and not clear...
@dhodovsk see this issue, and probably would be nice to improve it with today's comments about this ``run`` method