Move GraceDb service URL from argument to global config value (!59) · Merge requests · emfollow / gwcelery

Leo P. Singer requested to merge leo-singer/gwcelery:no-service-arg into master May 22, 2018

!48 (merged) and !58 (merged) have revealed use cases where we do not have an LVAlert message from which to extract the GraceDb service URL. Rather than carry around the GraceDb API URL as an argument to tasks, it makes more sense to store it in the global Celery configuration dictionary.

@deep.chatterjee, @shaon.ghosh, is this OK with you?

Edited May 23, 2018 by Leo P. Singer

Move GraceDb service URL from argument to global config value

Merge request reports