finnp: two main issues are: deployment is slow because we have a lot of `npm install` that needs to happen
finnp: so one hack to make it faster is to make a branch where we check in the node_modules to git, and use that as a sort of stable deploy branch
finnp: try the deploy out and you'll see what i mean in regards to the slowness
finnp: another issue is that they dont have a way to set env vars from the web, you have to do it via CLI
ryanj waves
ryanj: yo!
ryanj
hi!
ogd
finnp: a workaround for the env var issue is to turn dat into a cartridge
ryanj: finnp is our intern in berlin, ryanj works on openshift
finnp: cartridges are like heroku addons. and they can create their own random credentials that show up in the users admin page
finnp: so in that case dat would be like an external database service that gets added to the app
finnp: but that means we would have to do something like package only the dat-editor frontend as a standalone server and configure it to talk to the external dat
finnp
ogd: ok, interesting i am just creating an account
ryanj: hej!
ryanj
finnp: feel free to ping me if you have any openshift Qs. I'm interested in helping build a cart if its sounds useful
ogd
finnp: i had another idea which was to have smoething in the dat admin ui itself where you could set your admin pass on first run
finnp: but we'd have to like salt and hash the pw and store it on disk somewhere
substack: do you have a module for monkeypatching console.* methods for the purposes of intercepting to transmit during headless testing in remote browsers?
substack: also im writing a module for running tape tests in atom-shell, and need a good pun involving atoms and tape
sethvincent has quit
karissa
ogd: i think that having it in the dat admin (cli and ui) is a good idea.
ogd: and storing it on disk wouldn't be the worst idea.
ogd
karissa: yea as long as we get the crypto right around storing hashed pws
ryanj
on OpenShift those details usually end up in the system ENV
plain text
ogd
ryanj: yea thats how we do it now
finnp
ryanj: thanks
substack
ogd: I don't have a module for it but there's console-browserify
it also handles dom elements gracefully because dom elements otherwise have evil circular refs where the resulting reference is !== some previous values
so if you try to stringify them your whole program dies
floppy joined the channel
domanic has quit
ogd
substack: nice
finnp
ogd: mh deploying a simple hello world on openshift already needs 2.5 minutes, npm install adds like 1 minute to that
ogd
finnp: ah ok
finnp
ryanj: Is it normal that deploying on openshift needs at least 2 minutes for a simple node app?
shama joined the channel
xAt is now known as xat-
floppy has quit
yoshuawuyts joined the channel
ryanj
finnp: the initial app create is usually on the slow side. Waiting for DNS to propogate takes some time
yoshuawuyts has quit
Also, the free apps run on slower hardware
finnp
ryanj: okay. it would be cool if the web "Create Application" workflow would show some progress logs instead of a spinner
ryanj
finnp: totally agree
We're already rewriting the build system, so hopefully the UX will improve soon
ish
finnp
ryanj: okay awesome. is the web frontend open source as well?
ryanj
finnp: it sure is
it's written in ruby though
I was about to go though and attempt to create a PR for adding ENV vars
finnp
ryanj: that would be great!
ryanj: it could be cool if you could do this in such a way that you could set defaults through the url
ryanj
yeah, that was going to be my initial solution.
finnp
ryanj: great :)
ryanj
Maybe import all querystring params that start with "env_" or something?
Once that is working, I could add a form that shows which vars were imported, giving the user a chance to edit/review/approve
before creating the app
Maybe I should search for querystring params that are in all caps, instead of looking for an "env_" prefix to strip
finnp
ryanj: i think looking for "env[name]" query parameters would be nicer. caps sensitive url would be weird
ryanj
that works for me. They were already using [] in the url anyway
finnp
ryanj: yap.
ryanj: i think the ui should allow people to also add their own env vars at that point. so it would be like a 2-column table with input fields and a possibility to add new rows