search

praiskup / resalloc

Allocator and manager for (expensive) resources
GNU General Public License v2.0
7 stars 7 forks source link

Add command for printing logs #77

Closed FrostyX closed 2 years ago

FrostyX commented 2 years ago

Fix #74

It allows to tail -f logs for a given resource, e.g.

resalloc-maint resource-logs 123
FrostyX commented 2 years ago

I haven't properly tested this yet.

praiskup commented 2 years ago

I think the manual page generator is broken, so RPM build fails.

FrostyX commented 2 years ago

I think the manual page generator is broken, so RPM build fails.

True, thank you for the hint. I fixed the original issue but it still fails for EPEL7 for some strange reason.

praiskup commented 2 years ago

I think argparse-manpage is broken for EL7 for the "function" mechanism. That's because the runpy-evaluated function sees all the imports from within the file as None objects (they are defined, but None) on EL7.

For this, I would probably prefer fixing the manpage generator first, or keep the parser top-level for now (as it is mostly a cosmetic thing). Or find some work-around.

FrostyX commented 2 years ago

or keep the parser top-level for now

I dropped my change and left it top-level while reporting https://github.com/praiskup/argparse-manpage/issues/45

The CI passes now.

praiskup commented 2 years ago

Thank you!