Differential D1227

Initialize broker and notifications
ClosedPublic
Actions

Authored by dereckson on Nov 17 2017, 03:13.

Details

Reviewers

dereckson

Maniphest Tasks

T1309: Deploy Dæghrefn, Wearg and TC2 through Salt

Commits

rVIPER14e4bb33fe65: Initialize broker and notifications

Summary

After 90 seconds, enough to allow the egg to connect to Freenode
and join every channel, load first the broker, then the notifications
code (the notifications code depends of the broker).

This responsibility was previously for the configuration file.

It's more convenient to move this part to the application code,
especially as the bot-specific configuration file is now standardized.

Ref T1309

Test Plan

Restart Wearg, check it connects to broker

Diff Detail

Repository

rVIPER ViperServ scripts

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson created this revision.Nov 17 2017, 03:13

Harbormaster completed remote builds in B1949: Diff 3152.Nov 17 2017, 03:13

dereckson accepted this revision.Nov 17 2017, 03:13

This revision is now accepted and ready to land.Nov 17 2017, 03:13

dereckson planned changes to this revision.Nov 17 2017, 03:29

dereckson added inline comments.

Wearg/Time.tcl
8	03:28 <wearg> [03:28:50] Tcl error [dcc:source]: formal parameter "()" is an array element

Fix proc syntax error

This revision is now accepted and ready to land.Nov 17 2017, 03:30

03:29 <dereckson> .tcl mq connected
03:29 <wearg> Tcl error: invalid command name "mq"
03:29 <dereckson> .tcl onload
03:29 <wearg> Tcl: {* * * * ::notifications::channel_notify} {DockerHub * * * ::notifications::docker_build_summary}
03:29 <dereckson> .tcl mq connected
03:29 <wearg> Tcl: 1

Closed by commit rVIPER14e4bb33fe65: Initialize broker and notifications (authored by dereckson). · Explain WhyNov 17 2017, 03:31

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

Wearg/

Time.tcl

6 lines

Diff 3154

View Options

Initialize broker and notificationsClosedPublicActions