| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-08-16 | |||
| 17:08:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect port-resource-request-groups neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 17:11:12 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 17:12:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 17:12:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 17:15:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 17:15:04 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 17:17:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor asserts in qos tests https://review.opendev.org/c/openstack/nova/+/798930 | |
| 17:17:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 17:21:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 17:21:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port resource request tests https://review.opendev.org/c/openstack/nova/+/801815 | |
| 17:22:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 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 | |