| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-07-30 | |||
| 11:28:07 | sean-k-mooney[m] | well that bug does affect vgpu | |
| 11:28:13 | bauzas | songwenping__: look at the two different tests we have for vgpus https://review.opendev.org/c/openstack/nova/+/723858/3/nova/tests/functional/libvirt/test_vgpu.py#237 | |
| 11:28:29 | bauzas | sean-k-mooney[m]: depending on your cloud capacity | |
| 11:28:56 | songwenping__ | tbe exception in my env is: mediated device /sys/bus/mdev/devices/58c9a61a-874f-414c-a8e8-91892a9fc6dd is in use by driver QEMU, domain instance-00000008\n' | |
| 11:29:19 | sean-k-mooney[m] | in any case we don’t officially support multi create with vgpu today | |
| 11:30:21 | sean-k-mooney[m] | that looks like there is a uuid collision on the host or similar reuse of the same mdev | |
| 11:30:23 | bauzas | songwenping__: oh, that's another nvidia issue https://bugs.launchpad.net/nova/+bug/1758086 | |
| 11:30:37 | bauzas | because of IOMMU | |
| 11:31:20 | bauzas | you can't bind 2 vGPUs of the same pGPU on the same guest | |
| 11:31:26 | bauzas | 2 or more vGPUs tbc | |
| 11:31:56 | sean-k-mooney[m] | oh right ya. that is noted in our docs | |
| 11:32:06 | bauzas | you can bind 2 or more vGPUs to the same guest if and only if each of those vGPU is actually provided by a different pGPU | |
| 11:32:33 | bauzas | songwenping__: oh, this exception | |
| 11:32:38 | bauzas | sorry, I misread | |
| 11:32:50 | sean-k-mooney[m] | i’m not sure that limitation applies to intel guys by the way. | |
| 11:33:00 | bauzas | that means we tried to allocate a mdev which was already assigned to another instance | |
| 11:33:06 | bauzas | sean-k-mooney[m]: nope | |
| 11:33:25 | bauzas | anyway, I'm starving | |
| 11:33:32 | sean-k-mooney[m] | right that is what i meant by collision | |
| 11:34:04 | sean-k-mooney[m] | hehe and i’m off today but i get notification on my ipad. was looking a coffee machine reviews :) | |
| 11:34:22 | sean-k-mooney[m] | ill be back on tuesday | |
| 12:21:40 | stephenfin | artom: what's going on with the commit message here? https://review.opendev.org/c/openstack/nova/+/803001 | |
| 13:10:30 | artom | stephenfin, no idea | |
| 13:11:01 | opendevreview | Artom Lifshitz proposed openstack/nova master: I2f9ad3df25306e070c8c3538bfed1212d6d8682f fup: add log https://review.opendev.org/c/openstack/nova/+/803001 | |
| 13:25:54 | opendevreview | Merged openstack/nova master: Fix request path to query a resource provider by uuid https://review.opendev.org/c/openstack/nova/+/800855 | |
| 14:48:47 | bauzas | argh, forgot this ^ | |
| 14:49:04 | bauzas | a Friday 5pm outage ? man, does it mean I should stop then ? :) | |
| 14:50:16 | bauzas | melwitt: sorry I had no time reviewing your series but I haven't forgotten, will do it hopefully on Monday morning so you'd get feedback when you're back from time off | |
| 14:50:44 | bauzas | I'll tho try to start now unti Gerrit shutdowns | |
| 14:52:31 | bauzas | stephenfin: around ? how can I verify the revisions uuids for https://review.opendev.org/c/openstack/placement/+/669170/19/placement/db/sqlalchemy/alembic/versions/422ece571366_add_consumer_types_table.py#1 ? | |
| 14:52:45 | bauzas | L25-26 | |
| 14:52:50 | stephenfin | looking | |
| 14:53:16 | stephenfin | bauzas: the revision UUID should be unique | |
| 14:53:32 | stephenfin | so if you grep for that in the migrations directory, it shouldn't find anything | |
| 14:53:37 | bauzas | ok | |
| 14:53:44 | bauzas | do we have a test running it ? | |
| 14:53:45 | stephenfin | the down_revision on the other hand should be the most recent migration | |
| 14:53:53 | stephenfin | I suspect there are migrations tests | |
| 14:53:59 | bauzas | ok | |
| 14:54:04 | stephenfin | probably in placement.tests.db.test_migrations or similar | |
| 14:54:14 | bauzas | lemme look | |
| 14:55:16 | bauzas | stephenfin: do we need to run some command in some venv for creating a new alembic migration ? | |
| 14:55:20 | bauzas | like reno, I mean | |
| 14:55:55 | stephenfin | yes, 'alembic revision' | |
| 14:56:11 | stephenfin | I've documented it for nova https://review.opendev.org/c/openstack/nova/+/800078/3 | |
| 15:13:18 | bauzas | stephenfin: I see, thanks ! | |
| 16:40:08 | opendevreview | Takashi Kajinami proposed openstack/nova stable/wallaby: Fix request path to query a resource provider by uuid https://review.opendev.org/c/openstack/nova/+/803070 | |
| 16:40:45 | opendevreview | Sylvain Bauza proposed openstack/nova master: Rename vgpu options to mdev https://review.opendev.org/c/openstack/nova/+/801607 | |
| 16:40:46 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM (yet) : Expose the mdev class https://review.opendev.org/c/openstack/nova/+/801743 | |
| 16:40:46 | opendevreview | Sylvain Bauza proposed openstack/nova master: Provide the mdev class for every PCI device https://review.opendev.org/c/openstack/nova/+/802918 | |
| 16:54:52 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/c/openstack/nova/+/708783 | |
| 16:54:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/c/openstack/nova/+/708785 | |
| 16:54:53 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/c/openstack/nova/+/708784 | |
| 16:54:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/c/openstack/nova/+/708787 | |
| 16:54:54 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/c/openstack/nova/+/708786 | |
| 16:54:55 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: nova-audit: Use cliff instead of homegrown argparse bleh https://review.opendev.org/c/openstack/nova/+/803059 | |
| 16:54:55 | opendevreview | Stephen Finucane proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/c/openstack/nova/+/708788 | |
| 17:04:34 | opendevreview | Stephen Finucane proposed openstack/placement master: placement-status: check only consumers in allocation table https://review.opendev.org/c/openstack/placement/+/762987 | |
| 17:11:25 | melwitt | bauzas: ok no worry, thanks! | |
| 17:57:49 | opendevreview | Merged openstack/nova master: db: Use module-level imports for sqlalchemy (for real) https://review.opendev.org/c/openstack/nova/+/796519 | |
| 17:57:56 | opendevreview | Merged openstack/nova master: db: Move db.sqalchemy.migration to db.migration https://review.opendev.org/c/openstack/nova/+/799518 | |
| 19:05:34 | opendevreview | Merged openstack/placement master: Remove unused test helper https://review.opendev.org/c/openstack/placement/+/763958 | |
| 19:30:18 | opendevreview | Samuel proposed openstack/nova-specs master: Migrate Instance Between Projects https://review.opendev.org/c/openstack/nova-specs/+/802034 | |
| 19:48:51 | opendevreview | Merged openstack/placement master: placement-status: check only consumers in allocation table https://review.opendev.org/c/openstack/placement/+/762987 | |
| 20:33:37 | opendevreview | melanie witt proposed openstack/nova stable/stein: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/802935 | |
| 21:15:27 | opendevreview | Elod Illes proposed openstack/nova stable/rocky: [stable-only] Fix lower-constraints job https://review.opendev.org/c/openstack/nova/+/769910 | |
| #openstack-nova - 2021-07-31 | |||
| 00:38:54 | opendevreview | melanie witt proposed openstack/nova master: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803091 | |
| 00:57:44 | opendevreview | melanie witt proposed openstack/nova stable/wallaby: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803092 | |
| 00:58:41 | opendevreview | melanie witt proposed openstack/nova stable/victoria: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803093 | |
| 01:00:30 | opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reduce mocking in test_reject_open_redirect for compat https://review.opendev.org/c/openstack/nova/+/803094 | |
| 01:16:25 | opendevreview | melanie witt proposed openstack/nova stable/train: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/791807 | |
| 01:18:08 | opendevreview | melanie witt proposed openstack/nova stable/stein: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/802935 | |
| 01:20:42 | opendevreview | Merged openstack/osc-placement master: Mark microversion 1.37 supported https://review.opendev.org/c/openstack/osc-placement/+/784023 | |
| 16:08:29 | opendevreview | melanie witt proposed openstack/nova master: Restore original time override in test_archive_task_logs https://review.opendev.org/c/openstack/nova/+/803106 | |
| 16:13:41 | jgwentworth | the nova-tox-functional-* jobs are currently failing 100%, please see http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023962.html cc gibi_pto bauzas gmann lyarwood stephenfin | |
| #openstack-nova - 2021-08-01 | |||
| 11:56:00 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| 11:56:53 | opendevreview | Takashi Kajinami proposed openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | |
| #openstack-nova - 2021-08-02 | |||
| 08:15:42 | bauzas | gibi: good morning and welcome back | |
| 08:17:05 | bauzas | hope you had a good rejuvenated week | |
| 08:18:39 | gibi | bauzas: thanks it was good, thanks | |
| 08:19:02 | gibi | bauzas: anything major I should now about from last week? | |
| 08:20:07 | bauzas | gibi: honestly, no | |
| 08:20:22 | bauzas | I just saw melwitt's thread about the CI issue in this weekend | |
| 08:20:43 | bauzas | but we didn't have any problems before | |
| 08:23:05 | gibi | bauzas: yepp that is time sensitive, it hits on every last day of a month | |
| 08:23:19 | gibi | so it only failed on 31th | |
| 08:23:45 | gibi | melwitt: has a fix proposed but I'd like to test is in a VM where the clock is set back to 31th | |
| 08:23:55 | gibi | s/test is/ test it/ | |
| 10:52:29 | lyarwood | gibi / sean-k-mooney / stephenfin ; https://github.com/openstack/nova/blob/5a56ac42ad24d74a2bc150769fd2d2fdcbfa3fd6/nova/compute/manager.py#L8873-L8875 - stupid question, is this actually a no-op for Neutron? If so any objections if I remove it? | |
| 10:53:12 | lyarwood | unless it's already called | |
| 10:53:44 | lyarwood | oh right it's already called during pre_live_migration | |
| 10:58:27 | opendevreview | Lee Yarwood proposed openstack/nova master: WIP compute: Remove no-op setup_networks_on_host calls https://review.opendev.org/c/openstack/nova/+/803191 | |
| 11:20:32 | gibi | lyarwood: yepp | |
| 11:25:23 | lyarwood | kk I'll see what ^ borks in terms of tests and go from there | |
| 11:25:42 | dbulira | filters or Placement. Is there a way of setting up separate quotas for various vCPUs (cores) for a single project? Similarly what is done in Cinder when you have general quota for gigabytes and per-storage-type specific one. Any hint where to look at is highly welcomed. | |
| 11:25:42 | dbulira | Hi! Sorry for dropping the question here, however didn't get attention in the general channel. I have a general question regarding quotas in Nova. Let's assume I have two tiers of vCPUs lowfreq and highfreq or any other division (AMD/Intel, etc.). They are of course kept in separate host-aggregates and instance-host filtering is done via scheduler | |
| 13:39:10 | opendevreview | Jacek Tomasiak proposed openstack/nova stable/rocky: Reject open redirection in the console proxy https://review.opendev.org/c/openstack/nova/+/803182 | |
| 14:43:13 | opendevreview | Merged openstack/nova master: Restore original time override in test_archive_task_logs https://review.opendev.org/c/openstack/nova/+/803106 | |
| 15:24:59 | gibi | stephenfin: do you mean missing unit test coverage here? https://review.opendev.org/c/openstack/nova/+/793620/13#message-51af15791583c674361ebd329cab0759e2297d4d | |
| 15:25:31 | gibi | stephenfin: as there is a set of functional tests covering the modified APIs there | |