Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-15
03:22:49 sapd1 melwitt: Can I workaround to attach SR-IOV to exist instance? :D
03:26:04 melwitt sapd1: not that I know of. I haven't gotten around to asking artom about the bug https://bugs.launchpad.net/nova/+bug/1708433
03:26:04 openstack Launchpad bug 1708433 in OpenStack Compute (nova) "Attaching sriov nic VM fail with keyError pci_slot" [Undecided,New]
03:31:13 alex_xu mriedem: enjoy...Xi'an food :)
03:32:06 mriedem alex_xu: biang biang noodle!
03:32:11 mriedem chinese hamburger
03:32:27 alex_xu mriedem: hahaha
03:33:32 mriedem sapd1: you can't attach sriov ports to existing instances
03:33:43 mriedem beyond just the KeyError
03:34:00 mriedem we should likely just 400 those types of requestse
03:34:49 melwitt yeah, sean-k-mooney was saying he thought it used to 400 but that artom might know about patch(es) that might have inadvertently changed it?
03:34:56 sapd1 mriedem: so we don't have any way to attach sriov ports, do you?
03:36:21 mriedem sapd1: it's not currently supported no
03:36:32 mriedem see https://review.openstack.org/#/c/139910/
03:38:03 mriedem it would definitely be a microversion to start saying we support that type of attach
03:39:22 mriedem so i think we essentially need to check the vnic_type on the port and if it's not 'normal' we need to fail
03:41:57 mriedem sean-k-mooney: nova never pre-creates sriov ports right because you have to specify that in the binding:vnic_type when creating the port
03:42:23 mriedem therefore if a user tries to attach a port to a server by passing a network id rather than a port id and nova creates the port on the network, it will always be vnic_type 'normal' today
03:48:06 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Force refresh instance info_cache during heal https://review.openstack.org/591607
04:11:59 openstackgerrit Chen proposed openstack/nova master: Revisons on notifications doc https://review.openstack.org/591621
04:12:16 sapd1 mriedem: I saw in neutron-sriov-agent log, When launch instance It update profile of sr-iov port, But when attach port to exist instance It does not.
04:58:45 gmann alex_xu: do you have anything for today API office hour ? i am little busy on QA releases so if nothing much i would like to skip it.
05:04:14 mriedem sapd1: because attaching sriov interfaces to existing instances is not supported in nova
05:04:38 mriedem the api might say the interface is attached, but nova-compute doesn't actually allocate any pci device
05:31:02 alex_xu gmann: we can skip it today
05:40:01 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/591898
05:48:13 gmann ok
06:54:32 mriedem so quiet in here
06:54:36 mriedem is everyone on wechat?
07:09:06 sapd1 mriedem: I'm still here
07:10:22 openstackgerrit Merged openstack/nova master: Remove blacklisted py3 xen tests https://review.openstack.org/591419
07:10:34 brinzhang mriedem: If we use wechat, we can speak Chinese? :)
07:14:21 brinzhang anyone know a project of masakari? It has too few document.
08:07:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: consumer gen: move_allocations https://review.openstack.org/591810
08:07:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: consumer gen: more tests for delete allocation cases https://review.openstack.org/591811
08:07:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: consumer gen: support claim_resources https://review.openstack.org/583667
08:59:29 sean-k-mooney mriedem i noticed the same thing for the last week or so
09:00:33 sean-k-mooney basically since we started requireing registered nics that chat has been much more quite, and there have been less nics i have not seen before
09:04:47 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976
10:14:01 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976
10:39:26 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add change-before filter https://review.openstack.org/591976
11:03:12 pvc hi guys
11:03:12 pvc Volume size extension is not supported by the hypervisor.: ExtendVolumeNotSupported: Volume size extension is not supported by the hypervisor.
11:15:47 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872
11:16:30 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove the deprecated API extensions policies https://review.openstack.org/586872
11:22:12 pvc hi guys
11:22:12 pvc anyone
11:22:13 pvc here
11:49:33 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge extended availability zone response into server controller https://review.openstack.org/502859
12:03:14 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge config drive extension response into server controller https://review.openstack.org/584223
12:18:19 sean-k-mooney sorry for the delay, alot of people are on vacation in europe and most of the us folks are not online yet
12:18:24 sean-k-mooney pvc: ^
12:19:12 sean-k-mooney pvc: alot of eu people that are working today are also gone to lunch
12:19:51 sean-k-mooney pvc: you were getting an error with exetending voluems "ExtendVolumeNotSupported"
12:21:15 sean-k-mooney pvc: im not sure i can help but if you have a specifc question/issue i might at least be able to point you in the right direction
12:31:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge extended server attributes extension response https://review.openstack.org/584590
12:47:13 edmondsw pvc which virt driver are you using?
12:47:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge keypair extension response into server view builder https://review.openstack.org/584748
12:47:45 gibi btw I will take Thursday and Friday afternoon off and then Monday is public holiday here and I will take Tuesday out as well
13:00:17 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge server usage extension response into server view builder https://review.openstack.org/585262
13:14:03 openstackgerrit Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041
13:20:41 efried mriedem: I was approaching it from "I just happened to notice this while I was doing something else," not, "Hey, we need to clean up a big thing and this is one of the pieces." I'm sure it makes sense to go looking for the chunk of code that guy tests...
13:22:17 efried jaypipes: Did you run that locally? I'm wondering if you need to do list(traits.keys()) else you'd wind up with output like <Generator at 0x0a7e0a> etc.
13:23:02 efried whoops, wrong channel.
13:25:53 openstackgerrit Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041
13:29:13 openstackgerrit Jay Pipes proposed openstack/nova master: [placement] split gigantor SQL query, add logging https://review.openstack.org/590041
13:33:57 openstackgerrit Merged openstack/nova master: api-ref: fix GET /flavors?is_public description https://review.openstack.org/588092
13:36:28 efried mriedem: Why are we waiting until rocky rc2 to merge things into stein?
13:36:42 efried dansmith: maybe you know this answer?
13:36:57 dansmith efried: um we're not?
13:37:07 dansmith stable/rocky exists which means master is stein
13:37:10 efried okay. Matt said something weird earlier.
13:37:19 dansmith which was what?
13:37:33 efried http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-08-15.log.html#t2018-08-15T00:39:03
13:37:53 dansmith well, first off,
13:38:12 dansmith he said master was stein,
13:38:13 dansmith but his point was that if you do big removals or refactors,
13:38:14 dansmith and we then suddenly need to backport a fix,
13:38:15 dansmith it may be much uglier
13:38:21 dansmith when the big code removal shouldn't be any huge rush
13:38:25 efried okay, I figured if it was anything it was that.
13:38:25 dansmith I imagine that was his point
13:38:32 efried fair enuf
13:38:58 efried though I think that one probably could have been removed in rocky anyway.
13:39:15 efried since it was testing an ocata->pike thing.
13:39:43 mriedem slurm
13:43:06 openstackgerrit Gaudenz Steinlin proposed openstack/nova master: Ignore some updates from virt driver https://review.openstack.org/523006
13:45:38 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475
13:48:48 openstackgerrit Eric Fried proposed openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863
13:52:17 openstackgerrit Matthew Booth proposed openstack/nova master: Add regression test for bug 1550919 https://review.openstack.org/591733
13:52:17 openstack bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9)
13:53:21 lyarwood dansmith: https://review.openstack.org/#/c/591761/ is good to go right?
13:53:34 lyarwood dansmith: you didn't want to wait until your other backport was ready or anything?
13:54:01 openstackgerrit Matthew Booth proposed openstack/nova master: Py3 fix in fake image service https://review.openstack.org/592056
13:54:03 dansmith lyarwood: no, I don't think we should wait and yes I think it's good to go
13:54:08 lyarwood dansmith: ack thanks
13:58:18 mriedem dansmith: just replied in the other
13:58:31 mriedem will check it over in the morning
13:59:13 dansmith mriedem: coolt hanks
13:59:22 gmann mriedem: melwitt can either of you approve this BP. i have rebased the patches and added stein BP link - https://blueprints.launchpad.net/nova/+spec/api-extensions-merge-stein

Earlier   Later