Project

General

Profile

Feature #20195

Take out viper config from subpackages for jobsub_dropbox_cleanup tool

Added by Shreyas Bhat about 1 year ago. Updated about 1 month ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
-
Target version:
Start date:
06/19/2018
Due date:
% Done:

0%

Estimated time:
Stakeholders:
Duration:

Description

This comes from the code review from the jobsub_dropbox_cleanup tool v1.0 Take out all references to viper config from subpackages - pass the info down as structs or direct arguments instead.

This would address the following comment too:

  • maybe pass query endpoints to queryWrapper() rather than configKey

Related issues

Has duplicate JobSub - Idea #20498: Refactor jobsub_drobpox_cleanup tool to keep config info as high level as possibleClosed2018-07-31

History

#1 Updated by Shreyas Bhat 9 months ago

  • Has duplicate Idea #20498: Refactor jobsub_drobpox_cleanup tool to keep config info as high level as possible added

#2 Updated by Shreyas Bhat 9 months ago

Will work in the dupe ticket #20498, as I have more notes there

#3 Updated by Shreyas Bhat 3 months ago

  • Priority changed from Normal to Low

#4 Updated by Dennis Box about 1 month ago

  • Target version set to v1.3.3

#5 Updated by Shreyas Bhat about 1 month 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.



Also available in: Atom PDF