| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-10-09 | |||
| 15:57:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Default AZ for instance if cross_az_attach=False and checking from API https://review.opendev.org/469675 | |
| 15:57:19 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cache image GETs for multi-create/multi-BDM requests https://review.opendev.org/686835 | |
| 16:07:09 | openstackgerrit | Merged openstack/os-traits master: Privatize and reorganize base package methods https://review.opendev.org/687378 | |
| 16:11:23 | openstackgerrit | Son Pham proposed openstack/nova-specs master: [WIP]Update nova-spec for blueprint https://review.opendev.org/687608 | |
| 16:20:30 | openstackgerrit | Son Pham proposed openstack/nova-specs master: add snapshot blueprint https://review.opendev.org/686948 | |
| 16:30:15 | sean-k-mooney | so im not crazy but we all agree that test_live_block_migration failed the https://51b862355f8b2cc3358d-6f76ca65e5801c05e86d356ab1b2046b.ssl.cf2.rackcdn.com/685678/2/check/nova-live-migration/523b3ea/logs/testr_results.html.gz | |
| 16:30:50 | sean-k-mooney | because the nova live migartion job that is from succeded https://review.opendev.org/#/c/685678/ | |
| 16:32:05 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Add support for cloud-init on LXC instances https://review.opendev.org/667976 | |
| 16:41:30 | openstackgerrit | Son Pham proposed openstack/nova-specs master: [WIP]Add blueprint: snapshot-vms-with-state of-processes https://review.opendev.org/686948 | |
| 17:05:36 | eandersson | gibi, thanks | |
| 17:05:43 | eandersson | appriciate it | |
| 17:05:50 | eandersson | will test it out in the lab today | |
| 17:14:56 | openstackgerrit | Merged openstack/nova master: trivial: Remove unused API sample template https://review.opendev.org/686901 | |
| 17:15:03 | openstackgerrit | Merged openstack/nova master: tox: Use common 'command' definition for unit tests https://review.opendev.org/684774 | |
| 17:15:10 | openstackgerrit | Merged openstack/nova master: tox: Stop overriding the 'install_command' https://review.opendev.org/684775 | |
| 17:27:39 | donnyd | where does nova-scheduler get the config for db connections | |
| 17:38:46 | donnyd | reason I ask is the ip address for the database connection moved. I obviously updated the config and restart the services to reflect the new values.. everything works except the scheduler | |
| 17:39:56 | dansmith | donnyd: the api database has sql urls in it for the cells | |
| 17:40:20 | dansmith | nova-manage cell_v2 update_cell | |
| 17:40:21 | dansmith | or something | |
| 17:40:25 | donnyd | Ah... well then that makes sense | |
| 17:46:02 | donnyd | thanks dansmith that was exactly what I was looking for | |
| 17:46:17 | dansmith | donnyd: ack, you can mail the check to me directly | |
| 17:46:32 | donnyd | address??? | |
| 17:47:25 | dansmith | heh | |
| 17:47:27 | donnyd | :) | |
| 17:47:28 | donnyd | :) | |
| 17:47:30 | donnyd | :) | |
| 17:47:36 | donnyd | will three smiles do instead | |
| 17:47:47 | donnyd | better make it 4 :) | |
| 17:49:00 | donnyd | smiles are redeemed in the soul dansmith | |
| 17:49:22 | donnyd | ok, you got me... 5 :) | |
| 18:01:17 | openstackgerrit | Merged openstack/python-novaclient master: Remove cells v1 and extension commands and APIs https://review.opendev.org/686516 | |
| 18:16:48 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Helper to start computes with different HostInfos https://review.opendev.org/686832 | |
| 18:16:49 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | |
| 18:16:51 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 18:16:51 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 18:17:13 | artom | dansmith, mriedem_afk, ^^ | |
| 18:17:38 | dansmith | yeah I need to get back to that | |
| 18:17:41 | artom | I now Matt's not super keen on the massive main func test patch, so I at least added the new test for bug 1845146 in a different patch | |
| 18:18:34 | artom | But I left the chunker as is - I know it's a pain, I hope you'd be willing to review it bit by bit, test method by test method | |
| 18:20:45 | mriedem | i'm gonna take a pass through dans' cache image series first | |
| 18:21:22 | artom | mriedem, fair, I was just bring it to y'alls attention | |
| 18:21:26 | artom | Not that the WIP is gone and stuff | |
| 18:21:36 | artom | *Now | |
| 18:32:49 | SonPham | hi . how to boot to instance with this conf07:16 <disk type="network" device="disk">07:16i changed xml file type="raw" to type="qcow2"07:16Nova error: Error: Failed to perform requested operation on instance "4444", the instance has an error status: Please try again later [Error: CantStartEngineError]. | |
| 18:33:23 | openstackgerrit | Merged openstack/nova master: Remove @safe_connect from put_allocations https://review.opendev.org/686579 | |
| 18:34:27 | SonPham | amhi . how to boot to instance with this conf <disk type="network" device="disk">i changed xml file type="raw" to type="qcow2"Nova error: Error: Failed to perform requested operation on instance "4444", the instance has an error status: Please try again later [Error: CantStartEngineError]. | |
| 18:35:05 | SonPham | I change disk type to qcow2 | |
| 18:35:25 | SonPham | But can't boot to instance | |
| 18:37:23 | mriedem | dansmith: a few things on https://review.opendev.org/#/c/687137/ | |
| 18:37:35 | mriedem | not sure if mdbooth needs to ack my concern or not | |
| 18:39:09 | dansmith | mriedem: I think I figured out that I'm good on the lock, but it was before I wrote the code so I'll check | |
| 18:39:36 | mriedem | i know the imagebackend code checks to see if the file exists after acquiring the lock, | |
| 18:40:06 | mriedem | but i'm not sure if your cache method could clobber the imagebackend cache fetching the image? | |
| 18:42:56 | dansmith | yeah, no I think you're right | |
| 18:43:39 | dansmith | I went and chased the answer to that for artom and then didn't do it right | |
| 18:44:09 | mriedem | ok i'm dropping some stuff on the next patch in the series | |
| 18:44:39 | dansmith | sweet thanks | |
| 18:47:54 | SonPham | Hi. I change disk type from raw to qcow2 | |
| 18:48:16 | SonPham | But can't boot to image( libvirt error) | |
| 18:48:33 | SonPham | Can't start instance with qcow2 type? | |
| 18:52:46 | mriedem | dansmith: ok those are posted | |
| 18:52:56 | mriedem | nothing major | |
| 18:55:47 | dansmith | thanks | |
| 18:56:00 | dansmith | been screwing with something else all day, but I'll get back to this a bit later or tomorrow | |
| 19:02:14 | SonPham | Hi. I change disk type from raw to qcow2 But can't boot to image( libvirt error) | |
| 19:06:31 | mriedem | dansmith: ack, i'm wading into the conductor waters | |
| 19:54:54 | efried | testtools.matchers._impl.MismatchError: ['memberfoo', 'memberbar'] != ['memberbar', 'memberfoo'] | |
| 19:54:54 | efried | mriedem: have you seen this func test fail before? | |
| 19:55:06 | efried | from nova.tests.functional.db.test_instance_group.InstanceGroupObjectTestCase.test_add_members | |
| 19:55:24 | efried | seems like unpredictable ordering | |
| 19:55:41 | efried | really old test tho, not sure how this shows up now when it hasn't before. | |
| 19:56:16 | mriedem | doesn't ring a bell | |
| 19:56:58 | efried | weird | |
| 19:57:05 | artom | efried, I guess something changed in the way group members are ordered now? | |
| 19:57:18 | artom | But... the solution is to use assertItemsEquals | |
| 19:57:38 | artom | (without the s) | |
| 19:57:53 | efried | well, yes, if this turns out to be a real problem; but I can't imagine why we would be seeing this now for the first time on a test that's been in there for 3y | |
| 19:58:25 | efried | https://review.opendev.org/#/c/330995/ | |
| 19:58:28 | mriedem | logstash says that only shows up on 687416 | |
| 19:59:05 | efried | and only the once? | |
| 20:07:34 | artom | efried, FWIW I can't reproduce locally (even with the review mriedem pointed out), so... recheck and hope for the best? ;) | |
| 20:07:44 | artom | Blame it on feral packets ;) | |
| 20:07:47 | efried | no, I couldn't repro either | |
| 20:08:04 | efried | and that review shouldn't be getting anywhere near that test. | |
| 20:08:20 | efried | Yeah, I'm pushing a new rev, shrug. | |
| 20:16:18 | openstackgerrit | Eric Fried proposed openstack/nova master: Stop using NoAuthMiddleware in tests https://review.opendev.org/687416 | |
| 20:16:18 | openstackgerrit | Eric Fried proposed openstack/nova master: Deprecate [api]auth_strategy and noauth2 https://review.opendev.org/687373 | |
| 20:22:51 | mriedem | dansmith: ok marked up the api patch too | |
| 20:23:53 | zzzeek | uh, what is "vine" and did it suddenly go python 3 only ? | |
| 20:23:54 | mriedem | i'm going to move that into a runway slot | |
| 20:26:28 | dansmith | mriedem: okay it's really still pretty thrown together so I didn't want to take up a slot just yet, but there was also not anything else on the list last I looked either | |
| 20:26:55 | mriedem | as of yesterday there were 2 open slots so yeah i think you're fine | |
| 20:27:04 | mriedem | i can remove though if yo uwant | |
| 20:27:55 | dansmith | I'll address all those things in the morning, so whatever you want | |
| 20:28:25 | mriedem | zzzeek: pypi and upper-constraints say 1.3.0 is the latest and was released back in march | |
| 20:28:30 | mriedem | so not new | |
| 20:28:36 | mriedem | it must be a transitive dep of something nova uses | |
| 20:28:36 | zzzeek | mriedem: yeha im looking at this | |