Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-06-29
13:29:56 bauzas so, maybe it's just me overthinking
13:29:58 bauzas anyway
13:30:47 openstackgerrit Merged openstack/python-novaclient master: Deprecate binary argument in nova service enable/disable/force-down CLIs https://review.openstack.org/477212
13:36:13 MR2000 Hi, Anybody there? I have a question about adding new subcommand to nova.
13:42:56 mriedem alex_xu: i'm +2 on the tagged attach api change now https://review.openstack.org/#/c/376021/ - you were +2 before
13:43:47 mriedem MR2000: it's ok to just ask
13:44:03 artom mriedem, is that our new coverage standard, "exxxxxxxtreme"? ;)
13:44:24 artom Aka - how tired are you right now?
13:44:59 mriedem artom: only 2xtreme
13:45:16 mriedem not 7x
13:45:45 artom Our Xs are like SLAs's 9s
13:51:16 sfinucan artom: mriedem likes his tests like he likes his pancake flipping
13:53:59 mriedem dansmith: why did we make instance.keypairs a list?
13:54:16 mriedem am i missing something or can you only associate a single keypair with an instance when it's created
13:54:43 dansmith mriedem: yeah, I think we just went future-looking
13:55:03 dansmith I vaguely remember a conversation
13:55:14 mriedem hmmm, ok because that's going to be codified in the versioned notifications https://review.openstack.org/#/c/419730/
13:55:30 dansmith is that a problem?
13:55:31 mriedem the future conan?
13:55:33 mriedem no
13:55:41 mriedem i was just comparing instance.key_name to keypairs
13:56:02 mriedem sometimes i forget why we do what we do
13:56:05 dansmith heh
13:57:06 mriedem sfinucan: i don't get the pancake flipping thing
13:57:10 mriedem you should only flip a pancake once
13:58:21 sfinucan mriedem: Pancake flipping was the most mundane thing I could think of at the time
13:58:43 sfinucan Besides, only flipping once doesn't let you demonstrate your pancake flipping prowess to you non-flipping buddies
14:00:22 bauzas FWIW, creps are larger than pancakes hence harder to flip :p
14:00:29 mriedem you are going to have to work on some new analogies
14:00:45 mriedem i like my tests like i like my mt dew
14:00:50 mriedem exxxxxxxxxxxxxxxxxxxxtreme!
14:00:52 mriedem there
14:00:54 mriedem that's an analogy
14:02:58 openstackgerrit Alex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)' https://review.openstack.org/478900
14:02:58 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore https://review.openstack.org/478901
14:02:59 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore https://review.openstack.org/478902
14:02:59 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903
14:03:00 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore https://review.openstack.org/478499
14:03:00 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463
14:03:01 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes instead of stevedore https://review.openstack.org/478905
14:03:01 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore https://review.openstack.org/478498
14:03:02 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904
14:03:02 openstackgerrit Alex Xu proposed openstack/nova master: Use plain routes list for volumes instead of stevedore https://review.openstack.org/478502
14:04:02 openstackgerrit Merged openstack/nova master: _schedule_instances() supporting a RequestSpec object https://review.openstack.org/469037
14:06:11 openstackgerrit Claudiu Belu proposed openstack/nova master: hyperv: Cleans up live migration Planned VM https://review.openstack.org/478943
14:23:05 sean-k-mooney sfinucan: o/ can you take a look at https://review.openstack.org/#/c/467173/8 our requirement conflict witht he upstrem ones
14:23:48 sean-k-mooney sfinucan: or rather we are missing some blacklisted package versions
14:23:52 sfinucan sean-k-mooney: Done and done
14:24:02 sean-k-mooney sfinucan: cool thanks
14:25:42 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: create functional test base class https://review.openstack.org/407055
14:25:42 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Validate huge pages https://review.openstack.org/399653
14:29:06 openstackgerrit Ed Leafe proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
14:30:51 openstackgerrit Merged openstack/os-vif master: Updated from global requirements https://review.openstack.org/467173
14:31:21 edleafe mriedem: ^^
14:32:46 mriedem oooo just in time
14:35:23 mriedem wow that was a nasty diff
14:35:51 edleafe mriedem: yeah, the section I changed was moved to another file
14:35:56 edleafe mriedem: always fun!
14:39:26 mriedem edleafe: dead code in the rebase
14:40:12 edleafe mriedem: yeah?
14:40:17 openstackgerrit Matt Riedemann proposed openstack/nova master: Extract custom resource classes from flavors https://review.openstack.org/473627
14:40:38 mriedem https://review.openstack.org/#/c/473627/10..11/nova/scheduler/utils.py
14:40:47 edleafe ugh
14:40:52 edleafe thanks for fixing that
14:41:05 edleafe odd that pep8 didn't catch that
14:41:37 mriedem +2
14:41:45 mriedem now it's time
14:42:02 edleafe sweet dreams
14:42:13 mriedem o7
14:47:32 sean-k-mooney jaypipes: bauzas if ye get a chance can ye take a look at https://review.openstack.org/#/q/topic:enable-sriov-nic-features
14:48:36 sean-k-mooney jaypipes: bauzas https://review.openstack.org/#/c/449257/28 in partcalar has been stuck for about a month without any really feedback but that feature is more or less complete if we can get those last few pathces over the line
14:49:10 bauzas sean-k-mooney: sure, no promise but I'll try to look :)
14:49:31 bauzas sean-k-mooney: I was horribly off for a few days before so got a shit ton of things to do now :p
14:50:14 openstackgerrit Merged openstack/nova master: api-ref: Fix missing parameters in API Versions https://review.openstack.org/475226
14:50:37 sean-k-mooney bauzas: cool understood. life as a core gets hecktic espcially if your unwell for a few days. hope your feeling better
14:50:56 openstackgerrit Merged openstack/nova master: Stop using mox stubs in test_keypairs.py https://review.openstack.org/477384
14:57:05 openstackgerrit Merged openstack/nova master: Fix redundant BDM lookups during rebuild https://review.openstack.org/476258
15:03:48 openstackgerrit Merged openstack/nova master: Cleanup the plethora of libvirt live migration options https://review.openstack.org/471834
15:04:33 openstackgerrit Merged openstack/nova master: Fix the note at the end of allocate_for_instance https://review.openstack.org/475578
15:09:33 mmehan Need a help from Numa experts, I have a unique sitatuation
15:10:02 mmehan the VM i want to spin is using two nics from different numa and has direct port
15:10:23 mmehan the VM when spinned gets half cpus from Numa0 and other half from Numa1
15:10:51 sfinucan mmehan: It already gets cores from two NUMA nodes, or you want it to get them that way?
15:10:54 sean-k-mooney mmehan: yes this is the default
15:11:12 mmehan what the VM vendor want is even if the nic ports are across numa they want to give cpus and memory from one numa
15:11:33 sfinucan mmehan: We can't do that...yet
15:12:21 sfinucan mmehan: You're going to need this https://review.openstack.org/#/c/361140/
15:12:33 sfinucan but it's not happening in Pike, so you'll need to wait for Queens at least
15:12:36 jaypipes sean-k-mooney: yup, will do today. today is review day while I sit on yet another 5 hour sales call.
15:12:51 sean-k-mooney mmehan: currently it is expcitly not supported because that intorduce alot of qpi overhead and reduce the performace of the plathform as a whole. the spec sfinucan linked make it configurable
15:13:05 sfinucan ^ what he said :)
15:14:00 mmehan is yes i understand QPI overhead, but thats needed to keep platform uniform as of now
15:14:28 mmehan thanks for spec, i will go over it
15:15:12 sfinucan mmehan: I'm pretty sure there are patches up there that you could test against too
15:17:06 dansmith jaypipes: edleafe are you both good with this now such that I should review? https://review.openstack.org/#/c/475448/
15:17:13 dansmith bauzas: are you actively reviewing that one?
15:17:27 jaypipes dansmith: I am, yes. obviously I can't vote +2 on it :)
15:17:36 dansmith jaypipes: yep
15:17:44 jaypipes thanks much to ed for fixing my style.
15:18:26 mmehan sfinucan: I will try to find the patches, if you have any insight please share

Earlier   Later