Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-08-28
05:45:12 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs https://review.opendev.org/678448
05:45:13 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
05:45:13 openstackgerrit Luyao Zhong proposed openstack/nova master: Retrive the allocations early https://review.opendev.org/678450
05:45:14 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
05:45:14 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/678453
05:45:15 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
05:45:15 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
05:45:16 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
05:49:58 openstackgerrit Dustin Cowles proposed openstack/nova master: Public method to retrieve custom resource providers https://review.opendev.org/676029
05:49:58 openstackgerrit Dustin Cowles proposed openstack/nova master: Provider config file schema and loader https://review.opendev.org/673341
05:49:59 openstackgerrit Dustin Cowles proposed openstack/nova master: Load the custom resource providers to resource tracker https://review.opendev.org/676522
05:59:05 openstackgerrit zhufl proposed openstack/nova master: [Trivial]Remove unused helper get_allocated_disk_size https://review.opendev.org/678976
07:07:59 dr_gogeta86 hi
07:09:06 dr_gogeta86 nobody know a common pattern between nova-compute dying and rabbitmq connections falls to zero
07:09:19 dr_gogeta86 after a while from 10 to 30 minutes
07:09:28 dr_gogeta86 nova-compute kills all rabbitmq
07:09:33 dr_gogeta86 and going stuck
07:17:38 openstackgerrit ZhengMa proposed openstack/nova master: Check container_format on instance creation https://review.opendev.org/673407
07:32:11 bauzas good morning Nova
08:11:26 openstackgerrit Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/678451
08:33:28 openstackgerrit Balazs Gibizer proposed openstack/nova master: Func test for migrate re-schedule with bandwidth https://review.opendev.org/676972
08:33:29 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support migrating SRIOV port with bandwidth https://review.opendev.org/676980
08:35:56 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
08:38:57 kashyap aspiers: Thanks for the PS-6 changes here, just got back: https://review.opendev.org/#/c/673790/6/
08:39:29 kashyap aspiers: I _do_ owe a couple of drinks!
08:40:45 openstackgerrit Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827
09:00:22 aspiers kashyap: yw :)
09:01:06 kashyap aspiers: Investigating the "weird" thing you noticed: https://review.opendev.org/#/c/673790/6/nova/tests/unit/virt/libvirt/test_config.py@216
09:13:18 openstackgerrit Brin Zhang proposed openstack/nova master: Add user_id and project_id colume to Migration https://review.opendev.org/673990
09:22:22 openstackgerrit ya.wang proposed openstack/nova master: vCPU model selection https://review.opendev.org/670298
09:22:23 openstackgerrit ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300
09:22:23 openstackgerrit ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299
09:45:13 aspiers kashyap: replied
09:45:29 aspiers stupid question - where's the thing which does linting on our .rst files?
09:45:37 aspiers I can't see it in tox.ini
09:46:51 kashyap aspiers: Thanks; will look
09:47:56 kashyap aspiers: Hmm, I'm unclear why os.loader.features doesn't contain the given keys doesn't make sense.
09:57:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497
10:00:10 openstackgerrit Balazs Gibizer proposed openstack/nova master: Do not query allocations twice in finish_revert_resize https://review.opendev.org/678827
10:00:16 sean-k-mooney stephenfin: i noticed another helper function for nomalising tratis i can share between the prefilter and the driver code so im going to pull that out and then respin the first patch to adress your nits
10:00:30 stephenfin cool cool
10:00:47 stephenfin got some nits on the following one too if you want to wait 5 mins
10:01:11 sean-k-mooney also the verion in the prefilter has a bug
10:01:27 sean-k-mooney sure im just going to make coffee anyway
10:01:47 sean-k-mooney want me to wait untill you review all tree
10:01:57 sean-k-mooney then ill adress all your comments
10:02:36 openstackgerrit Balazs Gibizer proposed openstack/nova master: Allow resizing server with port resource request https://review.opendev.org/679019
10:11:30 stephenfin sean-k-mooney: done
10:12:48 sean-k-mooney thanks just looking at the scond one now
10:16:29 openstackgerrit Stephen Finucane proposed openstack/nova master: Add reshaper for PCPU https://review.opendev.org/674895
10:22:35 bauzas stephenfin: wow, chilling ^
10:23:33 sean-k-mooney bauzas: is that like something taht is almost cool
10:23:54 bauzas I should wear a t-shirt saying "I love reshapes"
10:24:16 sean-k-mooney if you did we would make you write all of them
10:24:33 bauzas \o/
10:24:37 bauzas anyway, lunch now
10:45:49 openstackgerrit Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452
10:45:50 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454
10:45:50 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/678453
10:45:51 openstackgerrit Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456
10:45:51 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455
10:45:52 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
11:03:12 openstackgerrit Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470
11:22:53 sean-k-mooney stephenfin: i hope that answers your question. https://review.opendev.org/#/c/665775/14/nova/scheduler/request_filter.py@176 im going to start reworkign the patches now but your comments make sense
11:28:46 openstackgerrit Brin Zhang proposed openstack/nova master: Add user_id and project_id colume to Migration https://review.opendev.org/673990
11:51:40 aspiers kashyap: I'm getting a parse error on your last sentence :)
12:06:41 aspiers kashyap: but what I don't understand is why a list of OS bootloader features would be subdivided into two sections called 'type' and 'readonly'
12:07:06 aspiers 'type' makes no sense as a category
12:20:16 kashyap aspiers: Err, sorry for the garbled sentence; I meant, why doesn't os.loader.features with type|readonly make sense.
12:20:51 kashyap aspiers: Okay, you don't think 'type' makes sense as as category...
12:24:59 kashyap aspiers: Anyway, I'll think a bit more on what I can do there.
12:26:28 aspiers kashyap: 'type' makes as much sense as a category as 'category' does
12:26:56 aspiers i.e. none
12:27:57 kashyap aspiers: Hmm. As you'll recall, it's the "type" ("category"? :-)) of loader. I'll see about reworking
12:41:25 kashyap aspiers: Also on your "not understanding" why a list of OS bootloader features would be subdivided into the two sections (enums) -- ask libvirt upstream :D
12:42:26 kashyap It _sounds_ like you temporarily forgot (but it's in the commit message, and you've seen it before :-)) the output spit out by getDomainCapabilities()
13:05:25 alex_xu aspiers: stephenfin in the end, I confused what the descision at here https://review.opendev.org/#/c/664420/42/nova/scheduler/request_filter.py@206
13:06:08 alex_xu in the end, we said use req filter or not...
13:24:08 stephenfin alex_xu: I thought we were going to copy the approach from https://review.opendev.org/#/c/671801/32/nova/scheduler/utils.py
13:24:38 stephenfin so that's why I dragged https://review.opendev.org/#/c/674894/ to the front of the series
13:25:25 stephenfin mriedem, dansmith: You have been summoned by efried https://review.opendev.org/#/c/675338/
13:26:45 stephenfin alex_xu: Yeah, that's what efried's comments are saying too. I guess aspiers is waiting to make sure that gets in first
13:27:09 mriedem stephenfin: why?
13:27:20 stephenfin Haven't a notion
13:27:52 mriedem the extra spec in those api samples could just as well be {'fart': 'sniffer'}
13:28:02 stephenfin Maybe because it's microversion related
13:28:11 stephenfin Yup, that's what I thought
13:28:27 mriedem it's not really,
13:28:36 mriedem but i see a good reason to -1 the shit out of your patch
13:28:47 mriedem oh drat https://review.opendev.org/#/c/675338/11/nova/tests/fixtures.py
13:28:48 mriedem you already got it
13:29:16 stephenfin #winning
13:30:15 mriedem +2
13:38:42 mriedem mdbooth: for https://review.opendev.org/#/c/677736/ i think we're just waiting for your customer to test it and ack it right?
13:38:44 aspiers stephenfin, alex_xu, efried: my understanding was that we were going to keep it in req filter for now and then change it later
13:39:24 aspiers kashyap: I don't have the slightest issue with features being split into two *meaningful* sections. But a section entitled "type" is totally meaningless.
13:39:42 mdbooth mriedem: So... they did test it but they didn't provide the same logs as last time. I've asked them for them, but they haven't been provided yet.
13:39:44 mriedem gibi: can i get a re-+W on https://review.opendev.org/#/c/637605/37 ?
13:40:00 mriedem mdbooth: did they notice the slow down?
13:40:18 aspiers kashyap: Let me take another tack. Please explain to me what this feature section entitled "type" is supposed to contain.

Earlier   Later