Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-22
19:01:16 mnaser 2 seconds apart, ha
19:02:23 jaypipes efried: +2 from me on both. nice work.
19:02:29 efried Thanks jaypipes
19:03:04 jaypipes lol
19:03:25 jaypipes efried: dansmith is somewhere over the Atlantic I think.
19:04:00 jaypipes hehe
19:04:26 gryf don't be afraid. just increase the frequency of the shots ;)
19:49:21 mriedem cfriesen: we don't support live migration with sriov ports attached right?
19:52:27 mriedem i guess i'm thinking about https://review.openstack.org/#/c/286744/
19:57:37 mriedem jay-afk-2: this should be a simple spec re-approval https://review.openstack.org/#/c/543578/
20:24:43 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add code to bind a port against a dest host during live migration https://review.openstack.org/523604
20:24:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
20:34:45 dansmith mriedem: looking at the glance docs, I'm not sure I understand what you and mdbooth said earlier while I was landing.. min_disk should be the minimum size disk you can boot it on, which would be defined by the filesystem in the image
20:35:38 mriedem he left a comment in that patch if that helps
20:37:22 dansmith mriedem: the glance docs say --size is deprecated
20:37:32 dansmith https://docs.openstack.org/python-openstackclient/pike/cli/command-objects/image.html
20:37:39 dansmith "Image version 1 only."
20:38:17 mriedem ah
20:38:20 mriedem didn't realize that
20:38:32 dansmith what I don't understand from what he said, is that if you boot something that was min_disk=10 and even size=10,
20:38:41 dansmith then you inflate that into a 20gb flavor and snapshot it,
20:38:48 dansmith your min_disk is 20g now
20:38:56 dansmith unless there's some other meaning for that that I don't understand
20:38:58 mriedem wtf the glance v2 API says that size is in the response, and not deprecated https://developer.openstack.org/api-ref/image/v2/index.html#show-image-details
20:39:36 dansmith see, that says the size is the "size of the image data"
20:39:42 dansmith which may be smaller than the size of the disk you expect to have
20:39:47 dansmith if you have something like qcow2
20:40:36 mriedem ok i'm asking rosmaita if size is a thing in v2
20:40:37 mriedem or what
20:40:40 dansmith I mean, it's not overly verbose about it I guess, but..
20:47:21 openstackgerrit Merged openstack/nova master: Extending delete_cell --force to delete instance_mappings https://review.openstack.org/540073
20:47:43 openstackgerrit Merged openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482
20:58:26 openstackgerrit Merged openstack/nova master: Clarify `resources` query param for /r_p and /a_c https://review.openstack.org/547065
21:01:45 dansmith efried: what did you want to have a shootout over?
21:02:25 efried dansmith: Not a shootout. I was hearkening back to the chaps conversation. I was just looking for a review on placement API feature: https://review.openstack.org/#/c/546837/
21:02:54 dansmith efried: jebus, don't you think you have had your fair share of core time today?
21:03:24 efried dansmith: Since you *just* approved the spec earlier, I figured you would want to get the code reviewed before it all falls out of your old, old brain.
21:03:39 dansmith efried: oh, I see, so you're really doing me a favor, I see
21:03:54 efried dansmith: Exactly.
21:04:02 efried You're welcome.
21:04:03 dansmith so really I should be thanking you
21:04:09 efried :)
21:05:21 cfriesen mriedem: correct, no live migration with sriov ports attached.
21:06:05 cfriesen are we wanting to go anywhere with cold migration support for LVM?
21:11:37 mriedem efried: i think you should have alex_xu review that
21:11:45 mriedem given he wrote the traits support in the allocation candidates API
21:11:51 mriedem he should be back to the work grind now
21:12:16 efried mriedem: Sure, sounds good.
21:12:17 mriedem cfriesen: doesn't matter to me
21:13:25 dansmith mriedem: looks fairly straightforward to me
21:14:37 dansmith the biggest question I'd have is over the sql stuff, but jay specifically made his mark on that bit
21:17:51 mriedem nit in the release note, but could be cleaned up on top
21:18:19 mriedem +1
21:22:28 dansmith efried: if you wanna update that nit I'll throw my +2 on there and we can try to get alex_xu to look during his day
21:23:08 efried dansmith: Roger that. Picking nit...
21:23:21 dansmith but it looks pretty good to me
21:24:29 openstackgerrit Eric Fried proposed openstack/nova master: rp: GET /resource_providers?required= https://review.openstack.org/546837
21:24:35 efried dansmith, mriedem: Hecho ^
21:25:45 dansmith establadajo ^
21:26:43 efried dansmith: Thanks.
21:26:46 efried I mean, you're welcome.
21:26:50 dansmith heh
21:28:59 efried Spanish? That's Esperanto.
21:30:26 efried Oh, shit, it actually is
21:31:00 dansmith what? can't be, I just made it up
21:31:08 efried https://translate.google.com/#eo/en/establadajo
21:31:23 dansmith damn, I must be a sevant or something
21:31:32 efried Note that Esperanto was also "just made up" by people.
21:31:38 dansmith *savant
21:31:42 dansmith haha
21:31:52 efried It didn't come about like any other normal languages.
21:32:42 efried Was supposed to be a simple language that anyone could learn, and therefore become the universal lingua franca of the world.
21:32:51 efried But the creators effed it up
21:33:09 efried Like, the first thing you should do is make sure there aren't any goofy exceptions in spelling and grammar and stuff.
21:33:35 efried But they totally brought that kind of garbage in from their native languages, which I think were Portuguese and English.
21:33:48 efried For that reason and/or others, it never caught on.
21:33:53 efried That's my understanding anyway.
21:35:30 efried This article was clearly written by an advocate/evangelist, not an impartial researcher.
21:37:33 dansmith heh
21:56:30 efried Okay kids, I'm out til Dublin. See some of y'all there!
22:00:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Add vif info to migration object https://review.openstack.org/515423
22:00:11 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings https://review.openstack.org/522537
22:00:15 mriedem sean-k-mooney: finally got this series all lined up and rebased, and i think the conductor changes are mostly done ^
22:00:32 mriedem likely need to split that into virt drivers > compute manager changes > conductor at the end
22:00:36 mriedem since conductor turns it all on
22:00:37 mriedem for the new flow
22:03:12 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP stub out changes for multi port bindings https://review.openstack.org/522537
22:05:55 dansmith vif info on migration object huh?
22:06:43 mriedem migratedata
22:11:10 dansmith ah
22:11:16 dansmith subject says migration
22:15:20 mriedem dansmith: if you're looking for a good time https://review.openstack.org/#/q/topic:bug/1750666+status:open+branch:stable/pike
22:15:23 mriedem all need a final +2
22:15:36 dansmith gaahd
22:15:55 mriedem well sylvain ran off and never reviewed those
22:15:59 mriedem so tag you're it
22:25:34 dansmith hmm, cinder failure on the cinder-related patch...
22:25:42 dansmith you checked it to confirm it's unrelated?
22:29:12 openstackgerrit Merged openstack/nova-specs master: Re-propose use Neutron's new port binding API https://review.openstack.org/543578
22:33:56 mriedem dansmith: yeah it's a cinder api consistency group snapshot test which is nowhere related to nova http://logs.openstack.org/75/546275/4/check/tempest-full/a6efea1/job-output.txt.gz#_2018-02-22_20_19_14_919584

Earlier   Later