Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-22
17:17:36 openstackgerrit Dan Smith proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down https://review.openstack.org/584829
17:17:36 openstackgerrit Dan Smith proposed openstack/nova master: Return a minimal construct for nova show when a cell is down https://review.openstack.org/591658
17:56:12 dansmith mriedem: on that metadata thread from the ML, we shouldn't be doing the join thing that multiplies instances * metadata * sysmeta row counts
17:56:26 dansmith because we fetch the metadata bits separate and explicitly
17:56:34 AJaeger nova cores, I have two changes for your zuul config up - please review https://review.openstack.org/610271 and https://review.openstack.org/612230 . The first converts a job to new Zuul v3 tempest job, the second renames an ill-named job. If you have questions, I'm around to answer...
17:57:44 dansmith mriedem: because of this: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L2244
17:58:20 AJaeger nova cores, one more job change: https://review.openstack.org/610141
18:12:24 mriedem dansmith: that's when pulling an instance list though, yeah? the metadata api isn't listing instances.
18:12:51 dansmith hmm, is it not on get too?
18:13:48 dansmith okay I guess not
18:13:54 dansmith we use it from a bunch of places but not there
18:14:06 dansmith I guess because it was always that instances * meta * sysmeta made it big(ger)
18:14:47 dansmith so, makes sense I guess
18:15:21 mriedem as far as i can see from a quick search i just don't think we need to pre-load system_metadata in the meta api
18:15:33 dansmith we did at the time, because of flavors
18:16:19 mriedem hmm, ok, and that's in case of 2007-08-29 being requested i guess
18:16:22 mriedem to return the flavor name
18:16:26 mriedem well, anything after 2007-08-29
18:17:41 AJaeger mriedem: thanks for reviews!
18:18:58 dansmith mriedem: so I guess I'd just say we could/should just convert single-instance to do the same fill approach, as the real exposure is the user putting a ton of metadata items in (which they control) and making it get big
18:24:38 openstackgerrit sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384
18:32:32 dansmith mriedem: commented on this: https://review.openstack.org/#/c/602174/13
18:32:58 dansmith mriedem: ready to +2 either way, but seems like a quick removal of the related bug and a host assertion can be done easily if you're up for it
18:34:23 mriedem i can
18:35:15 mriedem note the coincidental fun with forcing the evacuate to bypass the ComputeFilter https://review.openstack.org/#/c/602174/13/gate/test_evacuate.sh@56
18:36:57 dansmith what about it?
18:37:14 mriedem related to what was being discussed in gibi's spec about deprecating the force flag,
18:37:32 mriedem and when people might actually use it (to bypass the ComputeFilter)
18:37:44 mriedem but in this case, it's just negatively trying to force us to the down host
18:37:44 dansmith oh
18:42:11 dansmith okay I'm not sure if I'm missing something,
18:42:20 markguz_ Hi nova peeps. Question about nova-compute on pike , on ppc64le on ubuntu 16.04LTS.
18:42:22 dansmith you're downing libvirt so the current host is marked as down,
18:42:34 dansmith and then making sure that we fail even when forcing it right?
18:42:41 markguz_ after running updates recently i noticed that i couldn't spin up vms any longer
18:42:46 markguz_ getting this error qemu-system-ppc64: failed to initialize KVM: Invalid argument
18:43:08 markguz_ any other ppc64le users out there having issues on ubuntu?
18:44:27 AJaeger thanks, melwitt !
18:44:52 dansmith markguz_: see topic
18:45:00 melwitt AJaeger: thanks for the patches
18:45:13 markguz_ dansmith: i see. sorry!
18:47:17 sean-k-mooney markguz_: #openstack may be more helpful but it sounds like you may have some packaging/depency issues. not many people use powerkvm as far as i know
18:49:26 markguz_ sean-k-mooney: could be, but i'm using the official ubuntu stuff, not making my own packages.
18:50:56 sean-k-mooney markguz_: well i was more thinking you kvm module paramters file my have somthing that was invalid. anyway this seams more like a distro issue then nova. sorry i can be of more help.
18:57:14 mmedvede markguz_: Hi, I can help. Not sure what is the best place to discuss, probably #openstack is as good place as any
18:58:05 markguz_ mmedvede: i'm logged into #openstack
18:58:30 mriedem dansmith: "and then making sure that we fail even when forcing it right?" yes
18:58:35 mriedem it's a negative scenario,
18:58:39 dansmith right
18:58:43 mriedem then bring the target back up and evacuate to it
19:03:34 mriedem is lee still out for awhile?
19:03:44 dansmith I think another week at least, IIRC
19:05:17 mriedem ok
19:07:03 dansmith back wednesday of next week it looks like
19:33:11 openstackgerrit Merged openstack/nova master: Move live_migration.pre.start to the start of the method https://review.openstack.org/610739
19:33:19 openstackgerrit Merged openstack/nova master: Ensure attachment cleanup on failure in driver.pre_live_migration https://review.openstack.org/587439
19:35:45 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/587071
19:35:57 mriedem stephenfin: addressed my nits in ^ and i'll +2
19:43:58 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/612485
19:43:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/612487
19:43:59 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add regression test for bug#1784353 https://review.openstack.org/612486
20:10:16 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/612494
20:10:17 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: Add regression test for bug#1784353 https://review.openstack.org/612495
20:10:19 openstackgerrit Matt Riedemann proposed openstack/nova stable/queens: conductor: Recreate volume attachments during a reschedule https://review.openstack.org/612496
20:16:31 openstackgerrit Eric Fried proposed openstack/nova-specs master: Spec: Provider config YAML file https://review.openstack.org/612497
20:27:19 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/612500
20:28:00 cfriesen if I get a -1 from Zuul in the "verified" section of a review, is there a way to see what it didn't like?
20:28:32 cfriesen maybe that's an infra question
20:29:50 melwitt what do you mean other than clicking the link for the failed job?
21:07:53 mriedem artom: are you going to start backports? https://review.openstack.org/#/q/topic:bug/1763051+(status:open+OR+status:merged)
21:08:21 mriedem melwitt: dansmith: one of you probably want this https://review.openstack.org/#/c/611093/
21:08:29 mriedem for your sweet sweet customers
21:08:43 melwitt yass
21:15:48 melwitt gah, gate, you are killing me
21:25:34 artom mriedem, yep
21:30:45 openstackgerrit Eric Fried proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/612500
21:34:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Add volume-backed evacuate test https://review.openstack.org/604397
21:34:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Add post-test hook for testing evacuate https://review.openstack.org/602174
21:34:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Refactor TestEvacuateDeleteServerRestartOriginalCompute https://review.openstack.org/608705
21:34:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix InstanceNotFound during _destroy_evacuated_instances https://review.openstack.org/606122
21:34:03 openstack bug 1794996 in OpenStack Compute (nova) "_destroy_evacuated_instances fails and kills n-cpu startup if lazy-loading flavor on a deleted instance" [High,In progress] https://launchpad.net/bugs/1794996 - Assigned to Matt Riedemann (mriedem)
21:34:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional regression test for bug 1794996 https://review.openstack.org/606106
21:34:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Run evacuate tests with local/lvm and shared/rbd storage https://review.openstack.org/604400
21:34:05 mriedem dansmith: added that assertion to the bottom change ^
21:46:57 dansmith mriedem: cool, I meant to work my way up that series but got distracted.. will try to hit the rest tomorrow
21:49:03 mriedem np
21:49:04 mriedem thanks
21:51:42 mriedem oh btw, i dawned on me on friday that with this detach/attach root volume spec https://review.openstack.org/#/c/600628/ that the underlying root image could change, which is maybe part of the point
21:51:48 mriedem that's also kind of a rebuild territory kind of thing
21:52:03 mriedem *it dawned
21:53:05 mriedem anyway, i'm sure nothing could go wrong there.
22:23:25 openstackgerrit Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779
22:53:31 openstackgerrit Merged openstack/nova master: fixtures: Track volume attachments within CinderFixtureNewAttachFlow https://review.openstack.org/587013
22:53:39 openstackgerrit Merged openstack/nova master: Add regression test for bug#1784353 https://review.openstack.org/587014
22:56:50 openstackgerrit Matt Riedemann proposed openstack/nova-specs master: Per-instance serial number (spec) https://review.openstack.org/612531
23:22:35 openstackgerrit sean mooney proposed openstack/os-vif master: add isolate_vif config option https://review.openstack.org/612534
#openstack-nova - 2018-10-23
00:10:49 imacdonn tonyb: around?
00:11:17 tonyb imacdonn: Kinda ... in meetings (yes plural) ;P
00:11:43 imacdonn tonyb: heh, ok .. whenever you get a chance, take a look at this backport .... https://review.openstack.org/611701
00:12:32 tonyb imacdonn: Will do. I'm thinking I might ask on the operators list before we merge

Earlier   Later