Feature #14295
closed
Script to enable or disable debugging for all Foreman components
Description
I wrote a simple interactive (TUI) and CLI Ruby script (whiptail dependency, part of all distros, usually in package newt). It will be very similar to foreman-debug with loading of config file definitions from .d/ directory layout so plugins can drop their own components.
Updated by Lukas Zapletal over 8 years ago
- Category set to Packaging
Preliminary version at https://gist.github.com/lzap/9dd635ed9e572987fe55
Updated by Lukas Zapletal over 8 years ago
- Category changed from Packaging to foreman-debug
Not sure which category to pickup :-)
Updated by Dominic Cleal over 8 years ago
- Related to Bug #10788: provide unified tool for administrative tasks on a foreman/katello server added
Updated by Dominic Cleal over 8 years ago
Sounds very similar to #10788 which was refused in Foreman core - use the existing Hammer CLI framework please.
Updated by Lukas Zapletal over 8 years ago
I am not trying to do all-in-one script, it's s separate useful helper script.
Updated by Dominic Cleal over 8 years ago
The point was not that there should or shouldn't be all-encompassing scripts, but that the associated ticket wrote yet another CLI and extension framework so was refused.
Updated by Lukas Zapletal over 8 years ago
- Bugzilla link set to 1305085
I am not really sure what your concerns are, we are talking about three pages long separate helper script. I have no plans in merging into foreman-debug (the category might be misleading, could not find any better). I don't plan to use hammer CLI as well the script is quite simple - I started it in shell, but I needed some structures like arrays and hashes that made the code ugly in shell.
For the record, this feature was requested downstream and I want to make it as useful as possible in Foreman core, thus loading the external files (for katello and other plugins).
Updated by Lukas Zapletal over 8 years ago
On second thought tho, I can perhaps take a look on hammer and if this could be another attempt of starting one tool for everything, that might be possible.
Updated by Lukas Zapletal over 8 years ago
So this could be a new hammer cli "plugin" perhaps called "admin" with it's own subcommand "logging". I would like to provide both CLI and interactive (TUI) workflows, but that should work just fine. What's nice is hammer provides logging and i18n capabilities which are both useful.
On Smart Proxy, we can install just hammer_cli and the hammer_cli_admin packages and hopefully this works with just one plugin. I think that should work.
Updated by Dominic Cleal over 8 years ago
- Project changed from Foreman to Hammer CLI
- Category changed from foreman-debug to Foreman commands (obsolete)
Yep, seems you've got my points now, thanks for looking into it. Hammer provides a lot already, including the fact that we know how to package, distribute and extend it without extra dependencies.
It looks like development has moved to https://github.com/theforeman/hammer-cli-foreman-admin, so I'll move this ticket into the Hammer CLI project. You may want to do the same with the other, or just close it referencing that repository.
Updated by Lukas Zapletal over 8 years ago
- Related to Feature #14590: Allow array setting merging added
Updated by Lukas Zapletal over 8 years ago
- Related to deleted (Feature #14590: Allow array setting merging)
Updated by Lukas Zapletal over 8 years ago
- Related to Feature #14590: Allow array setting merging added
Updated by Lukas Zapletal over 8 years ago
- Status changed from New to Closed
- Pull request https://github.com/theforeman/hammer-cli-foreman-admin/pull/1 added
Merged. Packages build.