Page MenuHomePhabricator

move torrus behind misc-web
Closed, ResolvedPublic


torrus is on netmon1001 but not behind misc-web varnish.

this leads to cert errors, there is none for torrus.wm and it displays the one for librenms instead.

in the past torrus has not been moved to misc-web because we said we don't want monitoring tools behind it,but nowadays:

14:27 < bblack> anyways, if we're just doing aggregate power graphs, it can be through misc-web :P
14:27 < bblack> (or storage for that matter)

14:27 < mutante> it's not because we had a rule about monitoring tools

14:27 < bblack> yeah I think we need to narrow that scope a little
14:27 < bblack> I mean, it's fine that we don't want to create chicken-and-egg scenarios where we're blind if something breaks misc-web itself
14:28 < bblack> but aggregate power and storage graphs aren't a chicken-and-egg with misc-web functioning
14:28 < bblack> in the past there was varnish stuff in torrus too, but it's been completely broken for a very long time and nobody really cares

Event Timeline

Dzahn claimed this task.
Dzahn raised the priority of this task from to Needs Triage.
Dzahn updated the task description. (Show Details)
Dzahn added a project: SRE.
Dzahn added a subscriber: Dzahn.

@mark any concerns if we move torrus behind misc-web?

Change 255460 had a related patch set uploaded (by Dzahn):
torrus: move behind misc-web

Change 255463 had a related patch set uploaded (by Dzahn):
torrus: switch to misc-web

asked Mark if he has concerns. we can move both torrus and smokeping

Dzahn set Security to None.

Change 255460 merged by Dzahn:
varnish:misc: add torrus on netmon1001

Change 255463 merged by Dzahn:
torrus: switch to misc-web

done. but i'll also add the proto-redirect

Change 256969 had a related patch set uploaded (by Dzahn):
torrus: add protocol redirect

Change 256969 merged by Dzahn:
torrus: add protocol redirect

Dzahn removed a project: Patch-For-Review.

Change 257282 had a related patch set uploaded (by Filippo Giunchedi):
torrus: move icinga check to https

Change 257282 merged by Filippo Giunchedi:
torrus: move icinga check to https

I've moved the check to https but icinga still doesn't like it, acked there but I don't have the bandwidth to investigate @Dzahn

Change 259332 had a related patch set uploaded (by Dzahn):
torrus: fix http(s) monitoring

Change 259332 merged by Dzahn:
torrus: fix http(s) monitoring