Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-20
14:57:45 tssurya dansmith: thank you
14:57:47 jaypipes efried: you could prolly uber into downtown Brooklyn for a nice little lunch.
14:57:55 jaypipes efried: and go shopping if you fancy that
14:58:39 mnaser my flight to dublin is packed
14:59:13 dansmith mine as well
14:59:19 jaypipes mnaser: from where?
14:59:33 mnaser jaypipes: connecting here and there but jfk => dublin
14:59:54 jaypipes mnaser: same flight as efried? :)
15:00:16 jaypipes mnaser: I'm taking the DUB -> JFK flight back. but taking the ATL -> AMS -> DUB flight over.
15:00:24 mnaser Delta 44 leaving at 9:50 pm and getting there at 9:20 am next day
15:00:34 efried so nope
15:00:49 mnaser 5am arrival
15:00:50 mnaser brave
15:00:52 jaypipes mnaser: yeah, I'm on the Delta flight 45 coming back :)
15:01:35 mnaser jaypipes: hah, i'm on that flight. leaving at 11am and arrivng at 1:50 on saturday
15:01:48 jaypipes mnaser: yup!
15:01:56 jaypipes mnaser: sweet. we can hack together.
15:02:12 mriedem jaypipes: smcginnis and i are on delta 45 on the way back too
15:02:14 mriedem it's going to be a party
15:02:18 jaypipes mriedem: nice. :)
15:02:30 jaypipes I smell a carpool from the Croke Park hotel.
15:02:31 mnaser lol, let's all get a row of seats, someone mirror a couple repos locally
15:02:32 mnaser :p
15:02:45 jaypipes I'm in 15G.
15:02:52 jaypipes come at me.
15:03:12 mriedem 19E
15:03:22 mriedem within shouting distance
15:03:25 mnaser 26A
15:03:26 mnaser lol
15:03:27 jaypipes damn it.
15:03:27 mriedem "hey jay! hey jay!"
15:03:53 mnaser 19D is free if someone wants to bother mriedem
15:03:54 mriedem we can synchronize watching the notebook at the same time
15:03:54 mnaser :P
15:04:03 jaypipes lmao
15:04:18 dansmith so glad I'm not on that flight
15:04:22 jaypipes heh
15:04:27 smcginnis 16F. I can throw things over the seat at jaypipes
15:04:35 jaypipes I'm glad I already had the flu...
15:04:43 jaypipes smcginnis: sweet :)
15:05:19 sean-k-mooney im not sure its a good think to have so many nova people on one plane
15:05:57 mnaser we'll plug it in and have devstack vms for everyone
15:06:09 dansmith so glad I'm not on that flight
15:06:24 mnaser <insert building clouds in the cloud jokes>
15:06:27 dansmith although maybe the screaming 1U fans will drown out the kid
15:07:42 cdent https://youtu.be/B3lsJmwNO40?t=40
15:08:13 dansmith tssurya: your second one needs to be rebased on the first right?
15:08:43 jaypipes mnaser: lol, that's awesome :)
15:08:57 jaypipes dansmith: yeah, my thoughts precisely.
15:09:42 tssurya dansmith: no I guess they are independent changes ?
15:09:45 mnaser this needs backporting eventually so i'd love some extra eyes on it, it should be ready for final review - https://review.openstack.org/#/c/340614/ :>
15:10:00 mriedem already fished for that one
15:10:13 tssurya dansmith: the new change I will have using --force depends on these
15:10:44 dansmith tssurya: they're independent, but they will conflict.. i.e. if the first one merges, the second one can't until you rebase,
15:10:48 dansmith so if you rebase now, then both can land
15:11:07 dansmith you don't have to, it'll just be better if you do.. your call :)
15:11:12 tssurya dansmith : oh okay, yes will rebase the second
15:13:24 openstackgerrit Surya Seetharaman proposed openstack/nova master: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/542964
15:17:09 bauzas heh, hoping that flights won't have problems, or nova would miss lots of people :p
15:19:52 jaypipes mnaser: damn. 2.5 year old patch.
15:20:12 openstackgerrit Chris Dent proposed openstack/nova master: WIP: experment with setting placement error codes https://review.openstack.org/546177
15:21:03 mnaser jaypipes: it bit us a few days ago and the work to make it functional and tested was done with help of melwitt and mriedem (you can look at some of the patches before which already merged)
15:25:09 openstackgerrit Chris Dent proposed openstack/nova master: WIP: experiment with setting placement error codes https://review.openstack.org/546177
15:26:08 mnaser bauzas: thank you for the reviews
15:26:32 bauzas np, here to help
15:27:49 bauzas damn, I found the perfect opportunity for bringing a Chartreuse bottle with me if I buy it at my local airport, but since I'll be connecting thru LHR, there are chances that customs would take it there
15:28:43 mnaser mriedem: can i update the commit message if you're busy to https://review.openstack.org/#/c/545132 so that we can merge it (server => volume)?
15:30:37 mriedem mnaser: i can update it quick
15:30:54 mriedem we'll lose mel's +2
15:31:39 jaypipes mriedem, dansmith, bauzas, melwitt: I'm good with efried's https://review.openstack.org/#/c/546009/. seems like a very simple API enhancement that will yield positive performance impacts.
15:31:40 mnaser mriedem: always bothered me that updating commit messages drops +2's but ehh.. if you think its fine and can convince someone to review and +A then even better :p
15:32:04 mnaser i just want to follow the patch through gates and get it backported (hopefully cleanly)
15:33:29 openstackgerrit Chris Dent proposed openstack/nova master: WIP: experiment with setting placement error codes https://review.openstack.org/546177
15:33:53 openstackgerrit Matt Riedemann proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614
15:33:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional recreate test of deleting a BFV server pre-scheduling https://review.openstack.org/545123
15:33:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Detach volumes when deleting a BFV server pre-scheduling https://review.openstack.org/545132
15:34:04 mriedem ah shit
15:34:11 mriedem bauzas: ^ can you re-approve those bottom 2?
15:34:31 efried mriedem: Dangit, where's that placement doc with the "creating a new microversion" checklist?
15:34:53 mriedem https://docs.openstack.org/nova/latest/search.html
15:35:10 bauzas mriedem: sure thing
15:35:34 cdent efried: this one? https://docs.openstack.org/nova/latest/contributor/placement.html
15:35:58 efried cdent, mriedem: yup, thank you.
15:36:00 openstackgerrit Surya Seetharaman proposed openstack/nova master: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/542964
15:36:35 bauzas mriedem: considering https://bugs.launchpad.net/nova/+bug/1740025
15:36:35 openstack Launchpad bug 1740025 in OpenStack Compute (nova) "LVM Sparse Volumes broken" [Undecided,New]
15:36:58 bauzas WDYT about deprecating the LVM sparse stuff ?
15:37:06 efried cdent: I wouldn't be offended if that linked off of either/both of https://developer.openstack.org/api-ref/placement/ https://docs.openstack.org/nova/latest/user/placement.html
15:37:26 efried certainly the latter
15:37:26 mriedem bauzas: don't know anything about it, but i think cfriesen does
15:38:01 mriedem efried: user docs don't really need to link to dev docs
15:38:23 efried Guess it depends on your definition of "user".
15:38:36 mriedem user != contributor writing code etc
15:38:38 mriedem imo
15:38:48 efried I would think there's enough crossover that such a link wouldn't hurt.
15:39:20 efried but whatevs. Now it's permanently open in a tab for me, so I don't care anymore.
15:39:24 cdent A "if you're curious about this stuff, seem dev notes" kind of link on the user might be useful for some (but could also confuse some) but having it on the api-ref: meh
15:39:51 dansmith mriedem: can you drop your -1 on this at least? https://review.openstack.org/#/c/543580/
15:39:58 mriedem or just https://docs.openstack.org/nova/latest/contributor/
15:40:12 mriedem dansmith: that's the least i can do

Earlier   Later