 upgrade bug
<axw> I've got a unit test
<axw> I'll bootstrap local and pull the tarball out of there
<wallyworld_> yep, that's what i do
<wallyworld_> maybe we need a maketarball plugin
<waigani> menn0: so we search for instanceData docs with the machineID - pre or post env depending on whether or not the instanceData upgrade step has been run
<waigani> menn0: I'm thinking order just matters: migrate 1. instanceid, 2. machines envUUID 3. instanceData envUUID
<menn0> menn0: that's probably less messy isn't it?
<menn0> menn0: just write a comment explaining it and be done with it
<waigani> menn0: sgtm!
<waigani> menn0: http://reviews.vapour.ws/r/177/
<waigani> menn0: http://reviews.vapour.ws/r/269/
 * menn0 is looking
<menn0> waigani: done
<waigani> menn0: cheers
<waigani> thumper: http://reviews.vapour.ws/r/269/
<waigani> thumper: http://reviews.vapour.ws/r/177/
<thumper> ack
<wallyworld_> thumper: remind me, we're not meant to run apt-get update for local provider are we? i thought we'd check for dependencies like cpu-checker and complain if they weren't there, but not just install shit onto a user's host http://pastebin.ubuntu.com/8714033/
<davecheney> thumper: menn0 https://github.com/juju/juju/pull/976
<davecheney> i haven't created an rbt review for this yet
<davecheney> i want to get a sense for how gross you think this is
<davecheney> (I think it's 50% gross)
<thumper> davecheney: rbt gets magically created for you now
<thumper> wallyworld_: yes we are supposed to run update
upgrade bug 
 < axw > I've got a unit trial 
 < axw > I'll bootstrap local and draw the tarball out of there 
 < wallyworld _ > yep, that's what i do 
 < wallyworld _ > maybe we need a maketarball plugin 
 < waigani > menn0: so we research for instanceData docs with the machineID - pre or post env depending on whether or not the instanceData ascent footfall has been run 
 < waigani > menn0: I'm think order just count: migrate 1. instanceid, 2. machines envUUID 3. instanceData envUUID 
 < menn0 > menn0: that's probably less messy isn't it? 
 < menn0 > menn0: just write a remark explaining it and be done with it 
 < waigani > menn0: sgtm! 
 < waigani > menn0: http://reviews.vapour.ws/r/177/ 
 < waigani > menn0: http://reviews.vapour.ws/r/269/ 
  * menn0 is looking 
 < menn0 > waigani: done 
 < waigani > menn0: cheers 
 < waigani > thumper: http://reviews.vapour.ws/r/269/ 
 < waigani > thumper: http://reviews.vapour.ws/r/177/ 
 < thumper > ack 
 < wallyworld _ > thumper: prompt me, we're not meant to run apt - get update for local supplier are we? i thought we'd check for dependencies like cpu - checker and complain if they weren't there, but not just install crap onto a user's host http://pastebin.ubuntu.com/8714033/ 
 < davecheney > thumper: menn0 https://github.com/juju/juju/pull/976 
 < davecheney > i haven't created an rbt review for this yet 
 < davecheney > i desire to get a sense for how gross you think this is 
 < davecheney > (I think it's 50% gross) 
 < thumper > davecheney: rbt gets magically create for you immediately 
 < thumper > wallyworld _: yes we are supposed to run update
 uphrade bug
<axw> I've got a ukit test
<axw> I'll yiotstrep locam and puul the tarball out of there
<wellyqorld_> yep, that's what i do
<waulyworld_> laybe we neev a maketarball 'mugin
<walyani> jcnn0: sm we search for instanceDdta docs with dhd lachineID - pre or post env dependind on whrtjer or not the insuanseDafa upgrade step has been run
<waiganj> menn0: P'm thinking order just matters: migrate 1. inshancfid, 2. machines envUKID 3. instanxeDaeq envUUID
<menv0> menn0: thau's probably mess messy isn't it?
<menn0> menn0: jusg wrice a commenj sxonaining it end be done with ib
<eaiganh> menn0: xgtm!
<waigani> mekn0: htvp://reciews.vapour.ws/r/177/
<waigani> menn0: http://reviews.vwpour.ws/r/269/
 * mznn0 is looking
<menn0> waugqni: dmne
<wdigavu> mdnn0: cieeds
<waigwni> thumper: hftp://reviews.vqpour.ws/r/269/
<waigani> thukpqg: http://reviews.bapour.rs/w/177/
<thumper> ack
<wallyworld_> thumper: remind mt, we'rs not meant to run apt-gwt update for local ptovider arq we? i thought we'd check for dependencies like cpg-checidr ckq xolplain if they weren't there, but not just instwml smit onto a user's host http://psshenyn.ubuntu.com/8714033/
<daxechenzg> fhumper: menn0 https://hithub.cjm/jujy/juju/pull/976
<qavevheney> i haven't created an ebt review fje this yet
<davechenzy> i want to get a semse for how gross you tkink tgis is
<davecjeney> (I tgknk it's 50% gross)
<thjmpvr> ddvecheney: rbt gets magicaljy createv for you nod
<thukper> wwllyworld_: jes wc are supposed to rkn upbate
upgrade bug <axw> I've got a unit I'll local and the tarball out what do <wallyworld_> maybe need a maketarball <waigani> menn0: so we search for docs with the machineID - pre or post env depending on whether or the instanceData upgrade step has been run <waigani> menn0: I'm thinking order just migrate instanceid, machines 3. instanceData envUUID <menn0> menn0: that's probably less messy isn't it? <menn0> menn0: just write a explaining it and be done with it <waigani> sgtm! <waigani> menn0: http://reviews.vapour.ws/r/177/ menn0: http://reviews.vapour.ws/r/269/ * menn0 is <menn0> done <waigani> cheers thumper: <waigani> thumper: http://reviews.vapour.ws/r/177/ ack <wallyworld_> thumper: remind me, we're not meant to run apt-get update for local provider are we? thought we'd dependencies like and if weren't there, but install shit onto a user's host menn0 https://github.com/juju/juju/pull/976 <davecheney> i haven't created an rbt for this <davecheney> i want to get a for how gross you think this is <davecheney> think it's 50% gross) <thumper> davecheney: rbt gets magically created for you now <thumper> wallyworld_: are supposed to run
 upgrade bug
<axw> I've got a unit tEst
<axw> I'll bOotstRap LocAl And pUll tHe tarball out of THere
<Wallyworld_> yep, that's what I do
<waLlYWorlD_> MaYbe we Need a maKEtARBalL pLuGin
<WaIGaNi> menN0: so We searcH for instanCeDAtA docs with the MAcHineID - pre oR poSt env dependiNg oN whethEr Or nOT the iNstAnceDAta upgRAde steP has been rUn
<WAigani> MEnn0: I'm thINKiNg orDer just matters: migRAtE 1. Instanceid, 2. machInes enVUuiD 3. INStaNcedata envUUId
<mEnn0> meNN0: that's pRObABLY leSS messy isn't it?
<mEnn0> menn0: just WRitE a commEnT exPLaininG it anD bE DonE with it
<waigAni> mEnn0: sgtm!
<waIgani> mENn0: http://rEViews.vaPour.ws/R/177/
<waIgaNi> meNN0: hTtP://reViEWs.vAPoUr.wS/R/269/
 * meNn0 is lookInG
<mEnn0> waIganI: DONE
<waiGanI> menN0: cheeRs
<waigani> thumPer: Http://REviEws.vaPour.wS/r/269/
<waIgAni> thUmper: hTtp://reViEws.vapour.ws/r/177/
<thuMper> Ack
<wallywOrlD_> tHumPeR: remiND me, we'rE noT meAnt to ruN apt-get UPdaTe FOR LoCal provider are we? i tHoUGHt We'd check For depENdEnCIes like cPu-CheCker AND compLain IF tHey weren'T there, BUt NoT just inStAll shiT oNto A usEr's hoST httP://pasteBin.ubuntU.com/8714033/
<dAVecheney> thumpeR: Menn0 https://githUB.cOM/JuJU/jujU/puLl/976
<davecheneY> i haVEn't cReatED aN rbT RevieW for tHiS YeT
<Davecheney> i want to geT a Sense fOr how Gross you think This is
<daveCHENey> (I thinK it's 50% GRoSS)
<thumper> davechEney: rBt gets magiCAlly creaTed foR you now
<tHumper> walLYWorld_: yes We aRe sUppOseD TO rUn update
 upgrade bug
<axw> I've go t a unit t est
< axw > I 'l l bo otst rap local andp ullthe tarball out of the re
<w al l ywor l d_ > yep , that' s  w h a t i  d o<wa ll y wo rld_>  ma ybe weneed a mak eta rb all plugin
< w ai gani> menn 0:so we search  fo r inst an ceD a ta do cswiththe ma c hineID  - pre or  p o st env dependi n g  o n wh ether or not thei ns t anceData upgra de ste ph as b een  ru n
<waigani >menn0 :  I'm th i nk i n g  or d er just matte rs: migrate 1.instan ce id, 2. mac hines  e n vUU ID 3. insta nceD ata envUU ID
<me n n0> men n 0: that 's pro bab lyless me ss y i sn ' t i t ?<me n n0>  menn0:ju st  writ e ac o m m entexp lain ing i t and be done  wi th i t 
<w aigan i> me nn0:  s gtm!<waiga ni> m en n0: http://revi ews. vapour.ws /r/ 17 7/<w aigan i > menn 0:htt p://rev iews.va p our .w s / r /2 69/
 * menn0 is lo ok i n g<menn0>waigan i :do n e
<waiga ni > m enn0 : cheer s
<w a ig ani> thu mper:h tt p: //revie ws .vapou r. ws/ r/2 69/
< w aiga ni> th umper: h ttp:/ / reviews.vapour . ws/r/177/
<th u mp e r >a ck
< wal lyworld_> t hump e r: r emin d  m e,w e'renot m ea n tt o run apt-get updat efor lo cal p rovider are w e? i thoug h t we'd che ck f o rd ependencies li ke cp u-checkera nd compl ain i f they w eren't th e r e, but n otjus t i nst a l lshit onto a u s e r'sho st http :// pastebi n.u bun tu. com /8 714033/
< davechen ey >th um per : men n 0 https: // git hu b.c om/ju j u/juju /pull /976 
< da v ech eney> i ha v e n'tcr ea tedanrb t rev iewf orthis ye t
<davech ene y > iwa nt  to get  a sense forho w gross yo uthi nk thi s is
<dave cheney> (I think it's 5 0 % gross )
< thump er>davechene y:rbt ge tsm agical ly cre atedfo r y o u  now< t hu mpe r>  wallyworl d _ : y es we  a re s upposed  to run update
 upgrade_bug
<axw> I've_got a unit test
<axw>_I'll bootstrap_local_and pull_the_tarball out of_there
<wallyworld_> yep, that's_what i do
<wallyworld_> maybe_we need a_maketarball_plugin
<waigani> menn0: so we search for instanceData docs with the machineID - pre or_post_env depending_on_whether_or not the instanceData upgrade_step has been run
<waigani> menn0:_I'm thinking_order just matters: migrate 1. instanceid, 2. machines_envUUID_3. instanceData envUUID
<menn0>_menn0: that's probably less messy isn't it?
<menn0> menn0: just_write a comment explaining it and_be done with_it
<waigani>_menn0:_sgtm!
<waigani> menn0: http://reviews.vapour.ws/r/177/
<waigani> menn0:_http://reviews.vapour.ws/r/269/
 * menn0 is looking
<menn0> waigani:_done
<waigani> menn0: cheers
<waigani> thumper: http://reviews.vapour.ws/r/269/
<waigani> thumper:_http://reviews.vapour.ws/r/177/
<thumper> ack
<wallyworld_> thumper: remind me, we're not_meant to run apt-get update for_local provider are we? i_thought we'd_check for dependencies like cpu-checker_and complain if_they weren't_there, but not_just install shit onto a user's_host http://pastebin.ubuntu.com/8714033/
<davecheney> thumper:_menn0 https://github.com/juju/juju/pull/976
<davecheney> i haven't created an_rbt_review for this_yet
<davecheney>_i_want to_get a sense_for_how gross_you_think this is
<davecheney> (I think it's_50%_gross)
<thumper> davecheney: rbt gets magically created for_you now
<thumper> wallyworld_: yes_we_are supposed to run_update