Closed Bug 1543683 Opened 7 years ago Closed 7 years ago

crontabber node needs webapp configuration

Categories

(Socorro :: Infra, task, P1)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

References

Details

Attachments

(1 file)

Previously, crontabber was a socorro configman app that ran using configman-style configuration. With bug #1493687, we're rewriting crontabber changing it from a socorro configman app into a Django command that runs other Django commands.

This inherently changes the context in which the code runs. Instead of running in a socorro configman app context, it needs to run in a webapp context. This came up with the updatesignatures rewrite which needs to use memcached which is only configured in the webapp.

We need a crontabber node. We're in an interim period where it needs crontabber AND webapp configuration. Soon it'll only need webapp configuration.

This bug covers figuring that out and making the required changes.

Making this a P1 since it definitely blocks updatesignatures and probably blocks rewriting other jobs.

Miles: What should we do here short term (today) vs. long term (soon)? Should we copy all the webapp configuration into the crontabber node? Then long term, we'll take out the configman-style configuration from the crontabber node and continue forward with that?

Blocks: 1542394
Flags: needinfo?(miles)
Priority: -- → P1
Summary: crontabber nodes need webapp configuration → crontabber node needs webapp configuration

Miles is at a conference. I talked to Brian.

We're probably going to scrap all this when we move to GCP, so we just need to solve the short term. I have fears that today we're hitting problem with memcached_host, and tomorrow we'll hit something different. Brian said we should just copy all the bits over so the crontabber and webapp instance types have the same configuration.

I'll work on that next.

Assignee: nobody → willkg
Status: NEW → ASSIGNED
Flags: needinfo?(miles)

The updatesignatures command kicked off and ran fine. I think we're good here. Marking as FIXED.

Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: