-
meniv joined the channel
-
kwe has quit
-
meniv
-
this is not merged in fact
-
can we change that? 😀
-
oh there’s one comment
-
obinothere is a comment there ..
-
🙂
-
I know you like to ignore them ...
-
obino hides
-
menivobino: fixed
-
meniv doesn’t have the energy to argue
-
h-rockmeniv: you argue?
-
i just thought you demanded
-
and everything gets done
-
appscalelovesyou joined the channel
-
NOTICE: [appscale] obino closed pull request #1792: No cassandra heap dumps and unified log location. (master...no-heap-dump) https://github.com/AppScale/appscale/pull/1792
-
appscalelovesyou has left the channel
-
menivh-rock: that’s not how I see it 😀 constructive conversations take place couple times a year
-
obinoROFL
-
h-rockHAHHAHAHAHAH
-
i think obino agrees with me
-
hahahah
-
obinogood work on the java heap:merged!
-
menivobino: lol thanks
-
h-rockmeniv: whats your definition of "constructive conversations"?
-
obinooppenent's blood on the floor?
-
menivh-rock: 2week architecture discussions
-
obino: shameful, man
-
Onlyangel joined the channel
-
Onlyangel has quit
-
shaunj joined the channel
-
cdonati has quit
-
obino has quit
-
meniv joined the channel
-
shaunj has quit
-
obino joined the channel
-
shaunj joined the channel
-
Onlyangel joined the channel
-
kwe joined the channel
-
kwe has quit
-
shaunj has quit
-
shaunj joined the channel
-
appscalelovesyou joined the channel
-
NOTICE: [appscale] menivaitsi opened pull request #1797: Fixing inconsistency in cron headers. (master...cron-headers) https://github.com/AppScale/appscale/pull/1797
-
appscalelovesyou has left the channel
-
meniv has quit
-
shaunj has quit
-
unknown_had has quit
-
anix joined the channel
-
RedRat_ joined the channel
-
RedRat joined the channel
-
anix joined the channel
-
RedRat joined the channel
-
unknown_had joined the channel
-
RedRat_ joined the channel
-
RedRat_ joined the channel
-
RedRat_ joined the channel
-
meniv joined the channel
-
obino has quit
-
cdonati joined the channel
-
obino joined the channel
-
kwe joined the channel
-
kwe has quit
-
shaunj joined the channel
-
kwe joined the channel
-
obino: cdonati: https://ocd.appscale.com:8080/job/Daily%20Build... this is passing the build
-
here’s the correct link: https://github.com/AppScale/appscale/pull/1797
-
shaunj has quit
-
appscalelovesyou joined the channel
-
NOTICE: [appscale] cdonati opened pull request #1798: Allow tools branch to be specified with dev (master...allow-separate-tools-branch) https://github.com/AppScale/appscale/pull/1798
-
appscalelovesyou has left the channel
-
cdonatimeniv: Is this demo-able?
-
menivcdonati: I can cook sth up
-
something*
-
cdonati: the question is: do I want to? :p
-
cdonatiYeah, I don't know. I'm fine with merging it, but I'm not sure exactly what it does.
-
menivcdonati: so this one customer is checking those headers in their app
-
that’s why it was brought up
-
cdonatiSo google's sdk does it wrong? Or inconsistently with gae?
-
shaunj joined the channel
-
menivcdonati: good question
-
obinoshaunj is here, let's stop talking about him
-
shaunjobino: if only i wasn’t short on coffee, i might be able to come up with a witty response to that..
-
obinolol well we can fix the lack of coffee for sure
-
RedRat__ joined the channel
-
kwe has quit
-
kwe joined the channel
-
kwe has quit
-
kwe joined the channel
-
meniv joined the channel
-
meniv has quit
-
shaunj has quit
-
meniv joined the channel
-
shaunj joined the channel
-
appscalelovesyou joined the channel
-
NOTICE: [appscale] cdonati opened pull request #1799: If retry_on_except is false, raise exception (master...raise-exception-if-no-retry) https://github.com/AppScale/appscale/pull/1799
-
appscalelovesyou has left the channel
-
kwe has quit
-
RedRat__ joined the channel
-
kwe joined the channel
-
appscalelovesyou joined the channel
-
NOTICE: [appscale] obino pushed 2 new commits to master: https://github.com/AppScale/appscale/compare/dc0a2d324198...9f39dd0051af
-
NOTICE: appscale/master 66b33d1 Chris Donati: Allow tools branch to be specified with dev...
-
NOTICE: appscale/master 9f39dd0 graziano obertelli: Merge pull request #1798 from cdonati/allow-separate-tools-branch...
-
appscalelovesyou has left the channel
-
cdonatimeniv: Can I ask you a question about how this java error app is deployed?
-
menivcdonati: sure go ahead
-
cdonatiIn error_app.rb, it looks like the original tarball is deleted, but then it looks like it tries to extract it.
-
Am I reading it wrong?
-
(lines 56-59)
-
menivlink ?
-
cdonati
-
menivcdonati: no, that’s tar is creating the dir
-
the gz*
-
it recreates it
-
cdonatiAh, I missed the 'c'
-
Thanks.
-
kwe has left the channel
-
meniv: Okay, my next question is regarding this block: https://github.com/AppScale/appscale/blob/maste...
-
As far as I can tell, place_error_app just puts the tarball there, but it doesn't deploy it.
-
Should I then add a call to AppManager to deploy the error app?
-
menivcdonati: looking
-
cdonatiIt looks like it adds the error app files to the live app directory (leaving the old ones too, though). But I don't see where it starts or restarts the appserver process.
-
menivcdonati: I believe that the first code block you linked above is where we delete the app tarball
-
because we encountered an error
-
and we swap it with the error app tarball
-
sound good so far?
-
cdonatiyeah
-
menivfor the restart portion I am still not sure
-
I think that
-
for python if you swap the serving directory
-
it gets picked up right away
-
because we’re talking about /var/apps here
-
cdonatithat makes sense since the dev_appserver does that by default.
-
menivso I am not certain if it’s the same for the java dir
-
cdonatiEven for the python one, though, wouldn't we want to remove whatever was in the old directory, though?
-
It may not affect the app, but it makes it confusing when looking at the serving dir.
-
To have a combination of app and error app files, that is.
-
menivcdonati: yeah, what you’re saying makes sense
-
if you don’t see the rm in there
-
we should put it
-
shaunj has quit
-
shaunj joined the channel
-
shaunj has quit
-
obinocdonati: are you in for a little ruby debugging?
-
I know you like it ...
-
cdonatisure thing 🙂
-
obino subliminal message
Menu
