Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-20
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
15:40:20 mriedem dansmith: so those grenade results were just due to old setup?
15:40:42 bauzas mriedem: okay, fair enough
15:40:43 dansmith mriedem: yeah, from doing a pike->master upgrade
15:41:02 bauzas mriedem: the problem I see with LVM sparse LVs is that I guess nothing was checked
15:41:14 dansmith mriedem: the auto selection was choosing the right version, and master was rightly saying "I don't speak that version, yo"
15:41:37 mriedem hmm
15:41:42 mriedem with that 90s rap slang?
15:41:55 dansmith mriedem: obviously
15:42:07 mriedem ok; it's in a tab
15:42:08 dansmith mriedem: "this is how we dooooo, RPC versioning"
15:42:35 mriedem oh btw you might want to punch any red hat infra people about https://review.openstack.org/#/c/546183/
15:42:44 mriedem because it turns out we have http://grafana.openstack.org/dashboard/db/ceph-failure-rate
15:42:47 mriedem but it's been broken since october
15:43:18 dansmith mriedem: I was taught not to punch people
15:43:26 mriedem wet willy some red hat infra people
15:43:29 dansmith mriedem: unless they were wrong or said something I didn't like
15:43:31 dansmith hah, eww

Earlier   Later