Add an optional config file for the Review Bot worker.

Review Request #8773 — Created Feb. 21, 2017 and submitted

Information

ReviewBot
master
6e61175...

Reviewers

The worker currently runs without configuration, but some of the new features
we're looking at will require it. In particular, the immediate need is that
PMD, a tool which we'll be adding, does not have a standard installation
mechanism, and often is run via a script from the download directory. The path
to this script has to be configured for the tool to work.

While I was in here, I fixed up the task serialization/accept config to get rid
of a big warning about accepting pickle content (which is insecure and going
away in future versions), and corrected the method of setting the broker URL,
which I hadn't noticed was incorrect because it defaults to localhost.

Ran the worker with and without a config file in place. Saw the expected
output, and that the (future) PMD tool loaded correctly.

Description From Last Updated

One line?

brenniebrennie
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
  2. 
      
david
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        bot/setup.py
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        bot/setup.py
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
  2. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        bot/setup.py
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        bot/setup.py
        bot/reviewbot/celery.py
        extension/reviewbotext/extension.py
        bot/reviewbot/config.py
    
    
  2. 
      
brennie
  1. 
      
  2. bot/reviewbot/config.py (Diff revision 3)
     
     
     
    Show all issues

    One line?

    1. I add stuff to this in a later change, so not worth it.

  3. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (a7f1e3d)
Loading...