Refactor jobsub_drobpox_cleanup tool to keep config info as high level as possible
Best practice golang is to keep the config at the main package level, and simply pass structs of necessary info down. Makes it much easier to maintain code. Refactor the code (after doing the rest of the updates!!!) so that it follows this.
#1 Updated by Shreyas Bhat 8 months ago
Case in point: https://peter.bourgon.org/go-in-production/#configuration
Don't need verbose to stick around. Just set the log level in main.init based on the verbose flag. Then everything we would print as verbose becomes "log.Debug" and it sorts itself out.
Have to think about how I want to handle things like templated commands. They're in the config, and I really don't want to pass them down as parameters from main.
#5 Updated by Shreyas Bhat 3 days ago
- Status changed from New to Closed
Canceling this ticket. With the advent of the Rapid Code Distribution (via CVMFS) service, this will no longer be necessary. There's no point in upgrading the code for this tool to make it prettier/more readable if it's going to be deprecated.