#hypothes.is

aron_: I've had a look (ssh dokku@dokku.hypothes.is run syncbridge hypothesis init_db production.ini) but it seems dokku run is broken

$ ssh dokku@dokku.hypothes.is run syncbridge ls

time="2015-02-04T11:00:48Z" level="fatal" msg="cannot enable tty mode on non tty input"

so I'll need to dig a bit more

oh, :(

i think we could fix this with an environment variable tweak, though

at the moment production.ini disables model creation at startup

because production runs with multiple workers

but we could probably turn it back on by default

and just disable it in prod/stream/stage with an environment variable

so if I create a branch with that flag turned on, and push that to dokku it should go smoothly?

yep. give me 30s

aron_: so if you merge that, rebase your branch, and redeploy that dokku, all should be well

Seems legit

nickstenn, didn't seem to fix the issue... https://syncbridge.dokku.hypothes.is/login

looks like the tables still don't exist, I'll try destroying and redeploying

looking

aron_: hmm -- that's a bit weird

..?

i don't really understand why that hasn't worked

have you tried destroying it and redeploying yet?

no, shall I do that now?

deleting...

nickstenn: working now!

Odd that this dokku picks up annotations from a different one https://syncbridge.dokku.hypothes.is/stream

All dokkus use the same ES instance, that's why

Cool

aron_: i've gone ahead and created us a sentry account (https://getsentry.com) having noticed a bunch of exceptions in the production logs that we don't appear to be doing anything about

if you fancy having a look at https://github.com/hypothesis/h/pull/1906 at some point that would be great

then I think it might be an idea to do another production deployment so we can start capturing those errors

hypothesis/h#4634 (sentry - 6bf551c : Nick Stenning): The build failed.

Change view : https://github.com/hypothesis/h/commit/6bf551c8...

Build details : http://travis-ci.org/hypothesis/h/builds/49449507

hypothesis/h#4635 (sentry - 7e3fcec : Nick Stenning): The build failed.

Change view : https://github.com/hypothesis/h/compare/6bf551c...

Build details : http://travis-ci.org/hypothesis/h/builds/49449673

nickstenn, is sentry a separate deploy?

there's a hosted version

https://www.getsentry.com/welcome/ ?

yep

we can always host it ourselves later if we want

Looks good

apart from the bit where I broke the build

Got a couple of travis errors

yar

hoping that moving those imports into the function will shut pylint up

otherwise I guess I'm going to have to annotate them...

hypothesis/h#4637 (sentry - 9a5f123 : Nick Stenning): The build is still failing.

Change view : https://github.com/hypothesis/h/compare/7e3fcec...

Build details : http://travis-ci.org/hypothesis/h/builds/49450945

argh

hypothesis/h#4638 (sentry - f3a0678 : Nick Stenning): The build was fixed.

Change view : https://github.com/hypothesis/h/compare/9a5f123...

Build details : http://travis-ci.org/hypothesis/h/builds/49452025

aron_: do you fancy merging that and then pairing with me on a production deploy?

there's a tag search bug and a couple of other things waiting to go out

I'm deep in the middle of testing this syncbridge branch, I can do it later to day