Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-04-27
16:54:34 sean-k-mooney sahid: yes but that is a libvirt dirive implentation detail. hyperv support numa tology too and at least initally it did not have the same implmentation
16:56:04 sean-k-mooney sahid: this is the most importat section of the spec in this regard http://git.openstack.org/cgit/openstack/nova-specs/tree/specs/juno/implemented/virt-driver-numa-placement.rst#n94
16:56:59 sahid this section is just saying that, you can ask for a specific host numa node
16:57:11 sahid can't
16:58:14 sean-k-mooney that part of it but that is not the only intent of that section.
17:00:16 sean-k-mooney there is also this wiki on the topic https://wiki.openstack.org/wiki/VirtDriverGuestCPUMemoryPlacement
17:01:18 sahid ok sean-k-mooney i will try to discuss with some people iternally to ensure that is not going to break anything
17:01:53 sean-k-mooney do you have a specifc change you are proposing ot make to numa topology
17:01:56 sahid but my point is to do not let user think it's guest run 2 numa nodes where it's not
17:02:31 sahid sean-k-mooney: i'm not sure i understand?
17:02:31 sean-k-mooney sahid: the guest has never had any guarteee that 2 virtual numa node ment 2 host numa nodes.
17:02:57 sahid sean-k-mooney: please it's the current behavior and how the spec has been defined and implemented
17:03:57 sean-k-mooney sahid: i helped write that spec and that how it was impemented in the libvirt driver. anyway we are not currently proposing changing it but future virtdirver are not bound to do the same thing as libvirt
17:04:46 sahid yes sean-k-mooney me as-well so there is nothing new for me
17:04:59 sahid it's late and i want to enjoy my long weekend :)
17:05:26 sean-k-mooney sure enjoy :) i still have to resping the mtu patch
17:05:47 sahid sean-k-mooney: cool thanks for that
17:20:51 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Implement granular policy rules for placement https://review.openstack.org/524425
17:20:52 openstackgerrit Matt Riedemann proposed openstack/nova master: Deduplicate config/policy reference docs from main index https://review.openstack.org/564846
17:41:59 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.62 - Add host/hostId to instance action event https://review.openstack.org/564667
18:33:28 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
18:58:13 jgwentworth has anyone else been noticing request-id changing across nova service calls lately? like, I'll see a request start off with one request-id in nova-api and then by the time it gets to nova-conductor, it's a new one
19:00:20 fried_rice jgwentworth: Don't quote me on this, but I think there's a *global* request ID that's supposed to stay the same. The original request ID was per service, so it did change across those boundaries, but that's why they implemented the global one.
19:00:48 fried_rice jgwentworth: Are you seeing the global request ID changing across service boundaries?
19:01:04 fried_rice jgwentworth: In any case, I think cdent and dhellmann were involved in that work - they would know more.
19:01:27 jgwentworth no, I mean the request-id staying within nova is changing. it used to stay consistent per request
19:02:05 jgwentworth like you used to be able to trace the same request id from nova-api -> nova-conductor -> nova-scheduler -> nova-compute
19:02:30 jgwentworth it's true that request-ids didn't use to span across nova to glance, for example, and that's what the global request-id work was about
19:14:56 openstackgerrit Dan Smith proposed openstack/nova master: Make service all-cells min version helper use scatter-gather https://review.openstack.org/539590
19:14:57 openstackgerrit Dan Smith proposed openstack/nova master: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/539617
20:06:40 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
20:06:41 openstackgerrit Brianna Poulos proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262
20:06:42 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
20:06:43 openstackgerrit Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269
20:11:26 jgwentworth bpoulos: hi, just wanted to confirm, is someone from your team going to propose a spec amendment to openstack/nova-specs as described on L80 here? https://etherpad.openstack.org/p/nova-runways-rocky
20:12:06 jgwentworth once we get that reviewed and merged, we can continue to review the remaining patches in the implementation
20:12:25 bpoulos jgwentworth: yes, we have a spec amendment internally, but it has to receive internal review approval before we can post the update. I hope to post the update soon!
20:12:45 jgwentworth bpoulos: perfect, thanks! just wanted to make sure we were on the same page
20:13:05 jgwentworth we'll be on the lookout for the update
20:13:12 bpoulos jgwentworth: no problem, thank you for checking
20:22:50 openstackgerrit Jay Pipes proposed openstack/nova master: add root parent NULL online migration https://review.openstack.org/564876
21:30:53 openstackgerrit sean mooney proposed openstack/nova master: add mtu to libvirt xml for ethernet and bridge types https://review.openstack.org/553072
21:42:22 openstackgerrit Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247
22:08:55 openstackgerrit Hongbin Lu proposed openstack/nova master: Use revision_number on adding/removing security group https://review.openstack.org/535510
22:24:32 openstackgerrit Eric Fried proposed openstack/nova master: Base test module/class for functional placement db https://review.openstack.org/564590
22:24:33 openstackgerrit Eric Fried proposed openstack/nova master: Use test_base symbols directly https://review.openstack.org/564592
22:24:34 openstackgerrit Eric Fried proposed openstack/nova master: Use helpers in test_resource_provider (func) https://review.openstack.org/564638
23:06:25 fried_rice ō/
#openstack-nova - 2018-04-28
00:05:39 openstackgerrit Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596
02:43:53 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
02:44:46 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
06:37:25 openstackgerrit jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897
07:20:50 openstackgerrit jichenjc proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.openstack.org/541152
07:22:08 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150
07:22:13 openstackgerrit jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572
07:34:13 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773
07:38:29 openstackgerrit jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773
10:03:21 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201
10:03:22 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475
10:03:23 openstackgerrit Jianghua Wang proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077
10:12:22 openstackgerrit sean mooney proposed openstack/os-vif master: convert os-vif docs to follow PTI https://review.openstack.org/559398
11:36:52 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
13:16:43 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
13:17:44 openstackgerrit sean mooney proposed openstack/os-vif master: move legacy-tempest-dsvm-nova-os-vif to repo https://review.openstack.org/559401
18:33:45 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043
18:33:46 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/565044
19:18:32 openstackgerrit Vladyslav Drok proposed openstack/nova master: Placement: allow to set reserved value equal to total for inventory https://review.openstack.org/564838
19:21:54 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
19:39:33 openstackgerrit Eric Fried proposed openstack/nova master: placement: Object changes for granular https://review.openstack.org/564351
19:39:34 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
20:30:54 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
20:32:41 openstackgerrit Lars Kellogg-Stedman proposed openstack/nova master: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/564964
#openstack-nova - 2018-04-29
00:04:55 openstackgerrit Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757
01:35:11 pvc hi
01:35:38 pvc may i know what can i do to make my instance performance good?
01:35:40 pvc like tuning
#openstack-nova - 2018-04-30
10:14:42 openstackgerrit fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181
10:20:44 openstackgerrit Radoslav Gerganov proposed openstack/nova master: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/565183
10:30:38 kashyap lyarwood: Et al, we also need this to be backported (all the way to Ocata) -- https://review.openstack.org/#/c/565043/
10:30:59 kashyap (The upstream change is not merged yet; but it's straightfoward)
10:36:20 openstackgerrit Merged openstack/nova master: Make service all-cells min version helper use scatter-gather https://review.openstack.org/539590
10:36:25 lyarwood kashyap: kk, why are you using the bp/ topic for these bug fixes and a bug/ topic for the original change?
10:36:28 openstackgerrit Merged openstack/nova master: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/539617
10:36:47 kashyap lyarwood: Right, I'll rename it; so that it's all under the same topic
10:36:57 kashyap lyarwood: Do you have a minute more?
10:37:10 kashyap lyarwood: I think I messed up something (unrelated to branch naming) :-(
10:37:18 kashyap See here: https://review.openstack.org/#/q/topic:bp/libvirt-cpu-model-extra-flags+(status:open+OR+status:merged)
10:37:20 lyarwood kashyap: https://review.openstack.org/#/c/563926/ & https://review.openstack.org/#/c/565044/ are duplicates right?
10:37:26 lyarwood kashyap: yeah
10:37:41 kashyap lyarwood: Allow me to explain how I arrived there:
10:38:31 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290
10:38:48 kashyap lyarwood: I made the casing patch, and on *top* of it, applied the "lift restrictions" patch.
10:38:58 kashyap lyarwood: Could I have done anything cleaner?
10:39:35 kashyap lyarwood: The first variant https://review.openstack.org/#/c/563926/ had 8 revisions. Now I will "lose" them (or I'll simply link to it from the other)
10:39:53 lyarwood kashyap: AFAICT you changed the change-id causing gerrit to create a new review
10:40:23 lyarwood kashyap: you can close out the new review and just rebase the case-insensitive change on top of the old one
10:40:31 kashyap lyarwood: Right
10:40:33 kashyap I'm doing to do that now
10:40:35 kashyap Thanks!
10:52:28 kashyap lyarwood: Pease remind me again, I think it's okay to include "Closes-Bug" tag on two Gerrit patches, right?

Earlier   Later