| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-22 | |||
| 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 | |
| 00:12:35 | tonyb | It | |
| 00:12:41 | imacdonn | tonyb: fair enough - thanks! | |