| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-16 | |||
| 17:22:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 17:22:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect port-resource-request-groups neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 17:23:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 17:23:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 17:23:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 17:23:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 17:27:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 17:30:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 17:30:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 17:33:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Parse extended resource request from the port data https://review.opendev.org/c/openstack/nova/+/800085 | |
| 17:33:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 17:35:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 17:35:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support boot with extended resource request https://review.opendev.org/c/openstack/nova/+/800086 | |
| 17:36:27 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support move ops with extended resource request https://review.opendev.org/c/openstack/nova/+/800087 | |
| 17:41:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor interface attach with qos https://review.opendev.org/c/openstack/nova/+/800088 | |
| 17:44:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 17:45:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:46:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| 18:05:06 | artom | :O | |
| 19:28:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support interface attach / detach with new resource request format https://review.opendev.org/c/openstack/nova/+/800089 | |
| 19:28:10 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 19:30:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: [nova-manage]support extended resource request https://review.opendev.org/c/openstack/nova/+/802060 | |
| #openstack-nova - 2021-08-17 | |||
| 02:26:09 | opendevreview | Yongli He proposed openstack/nova master: Accelerator smartnic SRIOV support https://review.opendev.org/c/openstack/nova/+/804320 | |
| 07:24:37 | gibi | my broadband acting up in the last 24 hours so I might not immediatly see pings | |
| 07:57:33 | aarents | Hi gibi, Thks for comment on https://review.opendev.org/c/openstack/nova/+/764435 I replied | |
| 09:38:23 | lyarwood | gibi: \o hey how was PTO? | |
| 09:50:30 | lyarwood | https://review.opendev.org/c/openstack/nova/+/804275 (and the various patches either side) could use reviews this week if anyone has time | |
| 09:51:04 | lyarwood | ops that should be https://review.opendev.org/c/openstack/nova/+/804230/ but that API change is also ready | |
| 09:51:22 | lyarwood | I was just going to add novaclient support for the microversion alongside | |
| 09:52:03 | lyarwood | another bugfix https://review.opendev.org/c/openstack/nova/+/802317 is also ready for review FWIW | |
| 10:22:52 | gibi___ | aarents: ack, I will get back to that | |
| 10:23:47 | gibi___ | lyarwood: o/ PTO was good, thanks. now I have broadband issues :/ but I will get to both of your reviews | |
| 10:26:12 | gibi___ | stephenfin: I respun the pps series yesterday and fixed your nits along the way. So if you have time for a quick re-review then that would be appreciated | |
| 10:29:24 | stephenfin | gibi___: will do | |
| 10:29:55 | gibi___ | stephenfin: thanks | |
| 10:30:28 | Gowthami__ | Hi All, Hope you are doing fine. https://review.opendev.org/c/openstack/nova/+/764482/ commit is made for the bug https://launchpad.net/bugs/1581977. The tempest introduced along with commit has been failing on "IBM PowerKVM CI" unable to ping the floating ip. The "guest-instance-1.domain.com" vm created in the https://review.opendev.org/c/openstack/tempest/+/795699 ( ServersTestFqdnHostnames.test_create_server_with_fq | |
| 10:32:42 | Gowthami__ | Hi All, Hope you are doing fine. https://review.opendev.org/c/openstack/nova/+/764482/ commit is made for the bug https://launchpad.net/bugs/1581977. The tempest introduced along with commit has been failing on "IBM PowerKVM CI" unable to ping the floating ip. The "guest-instance-1.domain.com" vm created in the ( ServersTestFqdnHostnames.test_create_server_with_fqdn_name) is active but couldn't ping the vm from its na | |
| 10:32:50 | Gowthami__ | The tempest is being executed on adevstack vm and please find the error link: https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/nova/82/764482/2/check/tempest-dsvm-full-focal-py3/de4be5a/job-output.txt openstack console log show doesn't have any error "Trying to load: from: /pci@800000020000000/scsi@3 ... Successfully loaded\" May I ask if you could suggest way forward for this ? | |
| 10:37:04 | lyarwood | Gowthami__: that's from within the guestOS itself | |
| 10:37:48 | lyarwood | Gowthami__: so whatever guest image you're using doesn't seem to boot fully in this example | |
| 10:38:04 | lyarwood | Gowthami__: I would highly doubt that is due to the test and/or fix you have referenced above | |
| 10:38:46 | lyarwood | Gowthami__: and it's likely more of an issue with your test env (lack of resources given to each test instance?) or guest image (a recent update maybe?) | |
| 10:40:36 | lyarwood | 2021-08-17 07:31:10.923262 | devstack-focal-newcloud | ++ stackrc:source:693 : IMAGE_URLS+=http://download.cirros-cloud.net/0.4.0/cirros-0.4.0-ppc64le-disk.img | |
| 10:40:36 | lyarwood | 2021-08-17 07:31:10.921287 | devstack-focal-newcloud | ++ stackrc:source:692 : DEFAULT_IMAGE_FILE_NAME=cirros-0.4.0-ppc64le-disk.img | |
| 10:40:36 | lyarwood | 2021-08-17 07:31:10.919117 | devstack-focal-newcloud | ++ stackrc:source:691 : DEFAULT_IMAGE_NAME=cirros-0.4.0-ppc64le-disk | |
| 10:41:03 | lyarwood | I'd recommend trying to use 0.5.2 tbh | |
| 10:42:58 | lyarwood | 2021-08-17 07:31:10.883720 | devstack-focal-newcloud | ++ stackrc:source:672 : CIRROS_VERSION=0.4.0 | |
| 10:43:15 | lyarwood | missed that this job is hardcoded to 0.4.0, | |
| 10:55:14 | gibi___ | stephenfin: I have a request in https://review.opendev.org/c/openstack/nova/+/799684/5/nova/tests/unit/db/api/test_migrations.py | |
| 11:00:44 | stephenfin | gibi / gibi___: replied | |
| 11:00:57 | stephenfin | (done in a follow-up https://review.opendev.org/c/openstack/nova/+/800484/4/tox.ini) | |
| 11:04:56 | songwenping__ | stephenfin: hi, have you already fixed the oslo.db for 8.5.0 version about the mysql conflict message changed? | |
| 11:06:25 | stephenfin | songwenping__: that's probably better asked on #openstack-oslo, but the answer is the patches have been merged but they have not been released yet | |
| 11:07:15 | songwenping__ | got it, thanks. | |
| 11:08:18 | stephenfin | songwenping__: https://review.opendev.org/c/openstack/releases/+/804844 | |
| 11:10:54 | opendevreview | Merged openstack/nova stable/wallaby: virt: Add destroy_secrets kwarg to destroy and cleanup https://review.opendev.org/c/openstack/nova/+/796257 | |
| 11:13:02 | gibi___ | stephenfin: thanks | |
| 11:14:06 | songwenping__ | stephenfin: thanks, wait for the release patch merged. | |
| 11:29:03 | gibi___ | stephenfin: I have comments in https://review.opendev.org/c/openstack/nova/+/800078 | |
| 11:33:55 | gibi___ | stephenfin, lyarwood : I'm done with the alembic series, I'm mostly +2. | |
| 11:33:59 | gibi___ | stephenfin: thanks for working on it | |
| 11:37:09 | lyarwood | ACK I think I still had some left to review in that series, I'll try to finish it today | |
| 11:38:21 | gibi___ | lyarwood: yepp, that is why I pinged you, as I saw you were doing active review previously on that series | |
| 11:38:50 | lyarwood | ah I see, thanks | |
| 11:50:36 | Gowthami__ | <lyarwood> Thank you . Will try with 0.5.2 and also increase the resources too. | |
| 11:51:16 | lyarwood | Gowthami__: yeah FWIW if you do try 0.5.2 you need to raise the resouces anyway https://github.com/cirros-dev/cirros/issues/53 | |
| 11:57:40 | gibi___ | aarents: I have still concerns about https://review.opendev.org/c/openstack/nova/+/764435/5/nova/virt/libvirt/driver.py#9970 | |
| 12:00:06 | sean-k-mooney | gibi___: my understainding is we are nver ment to attempt to rollback a live migration once we have actully started it in qemu | |
| 12:00:24 | sean-k-mooney | we can rollback if we call migrate on libvirt and it imideatly returns with an error | |
| 12:00:40 | sean-k-mooney | but once it start we dont rollback unless it times out | |
| 12:01:20 | sean-k-mooney | besided timeout to we have other cases where we rollback after the migration has started today? | |
| 12:03:06 | sean-k-mooney | im not sure that aarents patch will solve the issue they are trying to solve in this case either | |
| 12:03:31 | sean-k-mooney | but for different reasons, the migration may continue as they said and the vm can end up on the destionation | |
| 12:04:02 | sean-k-mooney | so reverting the db state may or may not be the correct thing to do | |
| 12:04:52 | sean-k-mooney | for example im concerned about what happens with post copy | |
| 12:06:00 | sean-k-mooney | the instance would be still migrating but running on the dest and we would have already executed part or all of post_live_migrateion assuimg we recived the post_copy_reume event before the monitor connection died | |
| 12:06:18 | sean-k-mooney | whihc shoudl mean the host is already updated. | |
| 12:07:08 | aarents | gibi___: Hum, but I think I call live_migration_abort() of libvirt driver not from manager and it only call libvirt.api but now I have doubt | |
| 12:07:58 | sean-k-mooney | in the libvirt dirver it just does https://review.opendev.org/plugins/gitiles/openstack/nova/+/refs/changes/35/764435/5/nova/virt/libvirt/driver.py#9434 | |
| 12:08:18 | aarents | sean-k-mooney: yes thanks for the link | |
| 12:12:28 | sean-k-mooney | so i think that is fine it will jsut call libvirt | |
| 12:12:44 | sean-k-mooney | althoguh if the monitor connect is down it may not be able to mange the vm | |
| 12:13:50 | aarents | sean-k-mooney: yes in that case it will not work | |
| 12:14:23 | sean-k-mooney | which is the case you are trying to fix right. in the even the monitor connection drops you want to about the migration job | |
| 12:14:34 | sean-k-mooney | you can tell libvirt to about the migration | |
| 12:14:44 | sean-k-mooney | but it may or may not be able ot comply | |
| 12:15:17 | sean-k-mooney | i assume the except Exception: is to catch the libvirt error that is raised when that happens | |
| 12:16:41 | gibi_ | aarents: oops sorry I jumped to the wrong driver.live_migration_abort call. | |
| 12:16:45 | aarents | this will work only with network,RPC,DB issues not for libvirt issue | |
| 12:17:11 | sean-k-mooney | aarents: so for those cases im not sure we want to abort the migration | |
| 12:17:24 | sean-k-mooney | aarents: unless you want to abort all other operation when that happens | |
| 12:17:28 | aarents | sean-k-mooney: or it may work if there is only one flap from libvirt | |
| 12:17:33 | sean-k-mooney | spawns, deletes ectra | |
| 12:20:00 | sean-k-mooney | its a larger chagne but to me what feels like a more robost change would be to suspend the green thread if the connection is closed and resume it when we reconnect and only try to send the rpc call then | |
| 12:20:55 | sean-k-mooney | realistically if the rpc bus is down there is notight on the compute we can do to update the db state | |
| 12:24:18 | aarents | sean-k-mooney: honestly, the change is just ensuring to kill job regardless if state in can or cannot update in DB, we loss some instances due to that as explain in bug | |
| 12:24:19 | gibi_ | hm, so assuming we have the RPC down. the patch aborts the libvirt job. then raises the exception as today. That exception expected to update the instance and migration states which will not happen while the RPC is down. the nova compute RPC call to update the DB will time out eventually I guess. | |
| 12:24:55 | aarents | gibi_: yes | |
| 12:24:56 | gibi_ | so the nova DB will still see the migration as runnig | |
| 12:25:02 | gibi_ | but the compute already aborted it | |