BS,PS: Streamline main

[lukedirtwalker]

• Make sure we call msger.CloseServer()
• PS: Start discorunner separately from other tasks
• Make kill on a nil periodic.Runner a no-op so that we don't have to have ifs in client code.

---

This change is 

[oncilla]

Reviewed 3 of 3 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @lukedirtwalker)

---

go/beacon_srv/main.go, line 501 at r1 (raw file):

	t.mtx.Lock()
	defer t.mtx.Unlock()
	if !t.running {

This will cause a panic because of double close if !t.running is that intended?
Also, what is t.running used for now? We can just drop it.

[lukedirtwalker]

Reviewable status: 1 of 3 files reviewed, 1 unresolved discussion (waiting on @oncilla)

---

go/beacon_srv/main.go, line 501 at r1 (raw file):

Previously, Oncilla wrote…

This will cause a panic because of double close if !t.running is that intended?
Also, what is t.running used for now? We can just drop it.

Good catch! I re-added the if and set t.running=true earlier so that we can kill a half started task set.

[oncilla]

Reviewed 2 of 2 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved

[oncilla]

Reviewable status: complete! all files reviewed, all discussions resolved