Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-26
12:49:23 kaisers_ artom: Go for it! :)
12:54:40 openstackgerrit Chen Hanxiao proposed openstack/nova master: libvirt: properly decode error message from qemu guest agent https://review.openstack.org/511459
12:55:59 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
12:57:05 jaypipes efried, gibi: ^ fixed unit test failrues
12:58:06 efried jaypipes +1 thanks
13:09:38 gibi jaypipes: thanks, I but back my vote
13:13:16 openstackgerrit Elod Illes proposed openstack/nova master: Transform keypair.import notification https://review.openstack.org/467514
13:13:22 mriedem claudiub|2: lpetrut: looks like hyperv ci might have issues on stable/pike https://review.openstack.org/#/c/515330/
13:13:58 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix AttributeError in BlockDeviceMapping.obj_load_attr https://review.openstack.org/515404
13:14:25 mriedem sdague: bauzas: simple fix backport for a regression in pike https://review.openstack.org/#/c/515404/
13:22:55 openstackgerrit Alex Xu proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id https://review.openstack.org/514649
13:22:55 openstackgerrit Alex Xu proposed openstack/nova master: Add _get_providers_with_any_resource method https://review.openstack.org/514197
13:22:56 openstackgerrit Alex Xu proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198
13:22:56 openstackgerrit Alex Xu proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184
13:22:57 openstackgerrit Alex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects https://review.openstack.org/514200
13:22:57 openstackgerrit Alex Xu proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList https://review.openstack.org/514199
13:22:58 openstackgerrit Alex Xu proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj https://review.openstack.org/514651
13:22:58 openstackgerrit Alex Xu proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object https://review.openstack.org/514650
13:22:59 openstackgerrit Alex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766
13:23:53 alex_xu efried: jaypipes, all the functional tests passed ^ in the last patch, but it still is WIP, more unittests, and better commit message, and so on...
13:24:31 efried alex_xu Ack. Anything in particular in there that I could/should work on today?
13:25:11 alex_xu efried: I added todo note in the commit message https://review.openstack.org/#/c/514197/
13:25:19 alex_xu I want to separate that single unittest to multiple one
13:25:29 cdent I like this todo in commit message handoff idea
13:25:48 efried alex_xu Okay, I'll see if I can tackle that today.
13:26:01 jaypipes alex_xu: yup, I'm getting to those reviews this morning. thanks for your patience.
13:26:35 alex_xu efried: and for other patch, few of them just missing the unittest, I didn't write the todo
13:26:43 alex_xu but you can find them easily
13:26:45 alex_xu jaypipes: thanks
13:27:09 efried alex_xu I think I mentioned this in a comment, but eventually I'd like to extend _get_providers_with_any_resource to be able to do either 'any' or 'all'.
13:27:21 efried alex_xu Because numbered groups are going to want *all*.
13:27:29 efried which is a much easier thing to write SQL for.
13:27:50 efried In fact, perhaps I can just write a separate method that does all resources *and* traits at the same time.
13:28:27 alex_xu efried: that what we doing currently, finally that will be huge sql, that is hard to maintain I think
13:29:08 efried alex_xu I think the "any" thing is what makes it really tricky. But I'll play with it and see if it winds up looking too heinous to do in one shot.
13:30:28 alex_xu efried: yea, it will be good you can take a look at the overall first, let us see which direction better
13:31:03 efried alex_xu Sure thing. I'll go over the updates today in any case.
13:35:23 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix the ocata config-reference URLs https://review.openstack.org/515410
13:35:41 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Import the config drive docs from openstack-manuals https://review.openstack.org/515053
13:35:51 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Import user-data page from openstack-manuals https://review.openstack.org/515054
13:36:00 lbragstad gmann_afk: as far as i know, the policy deprecation work in oslo policy is just waiting on reviews
13:42:48 mriedem stvnoyes: did you ever get a chance to test the new cinder attach patches and try to attach the same volume to the same instance multiple times even if the volume wasn't multiattach=true?
13:43:53 stvnoyes no, I've been off on something else for the past week. I'll give it a try today
13:43:57 edleafe alex_xu: I think adding a TODO in a commit message for a WIP patch is very smart
13:44:27 mriedem stvnoyes: ok, alternatively i can just add something to a tempest test, push that up and make it depend on the nova api change
13:44:45 stvnoyes having it in tempest would be good to have
13:44:57 mriedem ok i'll put something up this morning
13:46:05 openstackgerrit Eric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs https://review.openstack.org/515151
13:46:06 cdent edleafe, efried, alex_xu: are you familiar with the “park on a slope” concept, when experiencing task interruptions?
13:46:06 openstackgerrit Eric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec} https://review.openstack.org/515223
13:46:07 efried cdent I went ahead and made those couple of changes since I was editing the base patch anyway ^
13:46:26 efried cdent Not by that name. Googling...
13:46:44 edleafe cdent: sorry, you just interrupted my task
13:46:45 cdent efried: cool, given other commitments today it is unlikely I would have got back to it and/or there’s not enough coffee in the world to make my brain work
13:46:52 edleafe cdent: no, not familiar
13:47:31 efried Turns out that's kind of a tough thing to google.
13:48:28 cdent it’s aligned with this idea of leaving a TODO in the wip commit. park your car in a way that if the battery is flat when you get back to it, you can still rolling start it
13:48:44 cdent or in general: make it easy, by leaving info, to rolling start
13:48:48 cdent stigmergy
13:48:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Implement query param schema for flavor index https://review.openstack.org/515413
13:49:14 cdent efried: google stigmergy if you want something detailed to google
13:50:23 efried cdent Neat - the ants-following-scent-trails thing was a great way to crystallize the concept in my head.
13:50:53 edleafe efried: that's what we are, after all :)
13:51:15 efried cdent edleafe Not sure I would go so far as to say we "lack any memory, intelligence or even individual awareness of each other" though.
13:51:44 edleafe efried: so effin' literal!
13:52:09 cdent efried: that somehow reminded me of an old blog post: https://www.burningchrome.com/~cdent/mt/archives/000187.html
13:52:45 cdent efried: if we want to be an inclusive community that accepts all comers we should behave in a way that assumes there is no shared memory, that only external things can operate as memory
13:53:25 efried *mind blown*
13:55:41 edleafe efried: that's cdent's fancy way of saying "write it down!"
13:57:23 bauzas sahid: as you pointed out, there is a bug that prevents mediated devices to be shown by libvirt https://bugzilla.redhat.com/show_bug.cgi?id=1463285#c6
13:57:24 openstack bugzilla.redhat.com bug 1463285 in libvirt "mediated devices are not shown in nodedev-list --cap mdev output" [High,Post] - Assigned to eskultet
13:57:50 bauzas sahid: now, the problem is about which specific operator impact we should provide
13:58:11 stephenfin dansmith: Wanna direct your nova-network wrath on this? https://review.openstack.org/#/c/499168/
13:58:13 bauzas sahid: if we ask operators to pre-create the mdevs, it can work for the next instances to be created
13:58:54 dansmith stephenfin: I don't really have n-net wrath, but sure :)
13:58:58 bauzas sahid: but for existing instances, it can be an operator painpoint because you need to exactly recreate the mdev with the same existing UUID
13:59:05 dansmith anything to avoid working on this slide deck sitting in front of me
13:59:11 bauzas as reboots wipe out mediated devices
13:59:33 bauzas sahid: that's why I feel we should leave nova drive the creation, and not the operator
13:59:45 bauzas sahid: unless mediated devices become persistent upon reboot :)
14:00:08 sahid bauzas: what i could say is that, for SR-IOV we ask operator to allocate the VFs
14:00:16 openstackgerrit Alex Xu proposed openstack/nova master: Add _get_traits_by_rp_ids method https://review.openstack.org/511184
14:00:16 openstackgerrit Alex Xu proposed openstack/nova master: Enable _get_usages_by_provider_and_rc not filter on the resource class id https://review.openstack.org/514649
14:00:17 openstackgerrit Alex Xu proposed openstack/nova master: Add more caches for the ProviderSummary and ProviderSummaryList https://review.openstack.org/514199
14:00:17 openstackgerrit Alex Xu proposed openstack/nova master: Add ProviderSummaryList object https://review.openstack.org/514198
14:00:18 openstackgerrit Alex Xu proposed openstack/nova master: Add has_resources method to the AggregatedProviderSummary object https://review.openstack.org/514650
14:00:18 openstackgerrit Alex Xu proposed openstack/nova master: Add AggregatedProviderSummary and AggregatedProviderSummaryList objects https://review.openstack.org/514200
14:00:19 openstackgerrit Alex Xu proposed openstack/nova master: placement: rewrite AllocationCandidates.get_by_filters https://review.openstack.org/479766
14:00:19 openstackgerrit Alex Xu proposed openstack/nova master: Add as_allocation_request_obj method to AggregatedProvierSummary obj https://review.openstack.org/514651
14:00:20 openstackgerrit Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799
14:00:25 sahid if host is rebooting the operator can have the module to pre-allocate VFs anyway
14:00:37 sahid but without the insurance that the PCI addresses will be the same as previously
14:00:47 bauzas sahid: but the domain XML will be incorrect then, right?
14:00:51 dansmith stephenfin: can you fix that typo in the commit message and then I'll fast approve?
14:00:52 sahid right
14:01:00 bauzas sahid: if the VF is on the wrong PCI device
14:01:01 sahid so operator have to deal with that
14:01:09 bauzas sahid: how can they deal with that ?
14:01:09 alex_xu efried: ^ more todo, some of them is more interest than unittest,
14:01:21 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Move additional nova-net opts to 'network' https://review.openstack.org/499168

Earlier   Later