[crontabber] daily_url crontabber app needs good config

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
6 years ago
2 years ago

People

(Reporter: peterbe, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
Our current "standard" is that crontabber and its jobs use the real default config variable (instead of `default="insert/something/here"`) so that the crontabber.ini can be minimal. 

However, the settings that daily_url.py are not set. E.g. 

    required_config.add_option(
        'public_output_path',
        default='.',
        doc="file system location to put the 'external/public' "
            "output csv file"
    )

All the defaults need to match Socorro production so that we don't need to add anything to /etc/socorro/crontabber.ini
(Reporter)

Comment 1

6 years ago
Note-to-self: This is how to override settings in crontabber.ini for this app:

  [crontabber]
      ...
      [[class-DailyURLCronApp]]
          public_output_path=/some/where
Assignee: nobody → peterbe
(Reporter)

Comment 2

6 years ago
Throwing this over to you Rob as per our discussion on Friday last week. 

What I did note was that the directory related settings were copied as they were before but what we need is the scp/ssh related stuff.
Assignee: peterbe → rhelmer
Not actively working on this, can help out though if needed.
Assignee: rhelmer → nobody
(Reporter)

Comment 4

2 years ago
We've gone back and forth so much about what the default config should be and never reach a standard. 
Also, the daily url functionality is slowly going out in favor of doing the same with Telemetry data. 
Let's not bother.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.