Citoid Syntaxerror on beta cluster
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	hashar
	Apr 9 2015, 8:48 PM

Description

I found in the /data/project/syslog/syslog file that the citoid service kept being respawned.

On deployment-sca01.eqiad.wmflabs in /var/log/upstart/citoid.log there is a bunch of:

etc/citoid/config.yaml:1
(function (exports, require, module, __filename, __dirname) { # Number of work
                                                              ^
SyntaxError: Unexpected token ILLEGAL
    at Module._compile (module.js:439:25)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/srv/deployment/citoid/deploy/src/server.js:27:20)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)

Details

	Subject	Repo	Branch	Lines +/-
	Citoid: Change Zotero's host for Beta	operations/puppet	production	+2 -0

Customize query in gerrit

Related Objects

Mentioned In: rOPUP39d97b29d6ef: Citoid: Change Zotero's host for Beta
T95652: On beta cluster citoid should self update and reload after change is merged
Mentioned Here: T95652: On beta cluster citoid should self update and reload after change is merged

Event Timeline

hashar created this task.Apr 9 2015, 8:48 PM

hashar raised the priority of this task from to High.

hashar updated the task description. (Show Details)

hashar added projects: Citoid, Beta-Cluster-Infrastructure.

hashar subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 9 2015, 8:48 PM

/etc/citoid/config.yaml is definitely a YAML file but somehow it is being loaded as a javascript file :/

Ah, yeah, my bad. We switched to a new config format but haven't updated the deployment-prep repo. Will do so tomorrow.

Merci beaucoup for noticing and letting us know @hashar !

greg removed a project: Beta-Cluster-Infrastructure.Apr 9 2015, 9:20 PM

greg set Security to None.

greg subscribed.

I have filled T95652: On beta cluster citoid should self update and reload after change is merged to get the code to auto update and the service to reload.

Citoid is now up, but Zotero refuses to start on deployment-sca01. Investigating ...

After conducting some pillow talk with trebuchet, everything is in its place now. The issue is not completely resolved in that zotero runs on deployment-zotero01 while citoid is on deployment-sca01. This patch addresses that problem.