| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-03 | |||
| 16:06:48 | bauzas | #action bauzas to provide an email about it | |
| 16:07:09 | bauzas | if folks are ok, let's then end this meeting | |
| 16:08:03 | bauzas | looks so | |
| 16:08:07 | bauzas | thanks all | |
| 16:08:10 | bauzas | #endmeeting | |
| 16:08:10 | opendevmeet | Meeting ended Tue Jan 3 16:08:10 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:08:10 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-03-16.00.html | |
| 16:08:10 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-03-16.00.txt | |
| 16:08:10 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-03-16.00.log.html | |
| 16:08:43 | Uggla | we could have done the meeting in French this time. ;) | |
| 16:09:34 | bauzas | oui :) | |
| 16:24:48 | dansmith_ | is anyone else not seeing syntax highlighting in gerrit after the upgrade? | |
| 16:28:38 | artom | dansmith, oh yeah | |
| 16:28:44 | artom | Annoying :( | |
| 16:29:08 | dansmith | wanted to make sure it wasn't just me | |
| 17:39:48 | gmann | yeah, it has changed few things and annoying. | |
| 17:39:54 | gmann | sean-k-mooney[m]: dansmith: placement RBAC changes are ready to review https://review.opendev.org/c/openstack/placement/+/865618 | |
| 18:18:19 | dansmith | gmann: apologies for not remembering, but.. I think placement has no concept of tenancy, so could technically use system scope | |
| 18:18:26 | dansmith | did we discuss and conclude that it should not? | |
| 18:18:41 | dansmith | (I might also be completely wrong about the tenancy thing) | |
| 18:42:36 | gmann | dansmith: we have not discussed that but as we are allowing admin_or_service access in placement keeping it consistent with nova, neutron etc will be good so that same admin token can work for service APIs + placement | |
| 18:42:44 | gmann | otherwise they need to switch | |
| 18:43:12 | dansmith | same service account/token I assume (eventually) but yeah okay makes sense I guess | |
| 18:43:32 | gmann | yeah | |
| 18:55:23 | opendevreview | Merged openstack/nova master: doc: soft delete and shadow tables https://review.opendev.org/c/openstack/nova/+/867001 | |
| #openstack-nova - 2023-01-04 | |||
| 02:27:27 | opendevreview | Ghanshyam Mann proposed openstack/python-novaclient stable/zed: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/python-novaclient/+/868559 | |
| 06:23:28 | opendevreview | Ghanshyam Mann proposed openstack/placement master: DNM: testing rbac new defaults https://review.opendev.org/c/openstack/placement/+/869145 | |
| 10:02:52 | frickler | bauzas: others: could you please have a look at the placement related tox4 fixes? the one for osvif seems to need further work, too. https://review.opendev.org/q/topic:tox4+status:open | |
| 10:11:15 | bauzas | frickler: ok, I can try | |
| 11:36:25 | opendevreview | Merged openstack/osc-placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/osc-placement/+/868422 | |
| 14:10:41 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova master: Correctly reset instance task state in rebooting hard https://review.opendev.org/c/openstack/nova/+/867832 | |
| 14:38:00 | admin1 | is it possible to set a disk type in a flavor to ide ( to test boot solaris 11.4) | |
| 16:00:29 | zigo | I'm starting a new project. sean-k-mooney[m] Would you say it's ok to "cheat" code on placement for my bootstraping? Is it considered "state-of-the-art"? | |
| 16:07:11 | artom | zigo, sean-k-mooney[m]'s is still out. What do you mean "cheat"? As in, copy code from? | |
| 16:07:25 | artom | I'm not sure about state of the art, but it's probably good enough :) | |
| 16:08:35 | sean-k-mooney[m] | placement is not state of the art but its a functional implementation | |
| 16:09:15 | sean-k-mooney[m] | if we rewrote it today we might proably would not use an sql database | |
| 16:10:06 | sean-k-mooney[m] | maybe graphql or another non sql set based db but sql is what we new | |
| 16:10:56 | sean-k-mooney[m] | sql is fast but it prevented us from doing everything in sql and the perfromacne is limited by the parts we need to do in python | |
| 16:14:11 | sean-k-mooney[m] | but ya im back on thursday placement has a current know issue with scaling the number of resouce groups and rps in a singel tree | |
| 16:16:55 | sean-k-mooney[m] | https://etherpad.opendev.org/p/nova-antelope-ptg#L290 | |
| 16:17:21 | sean-k-mooney[m] | zigo if ^ edgecase is not a concern placement is proably something worth considering | |
| 16:20:52 | zigo | artom: Yeah, cut/past some code. | |
| 16:22:05 | zigo | sean-k-mooney[m]: I need to copy/code for stuff like oslo.db, alembic migrations, etc, which aren't documented anywhere. | |
| 16:22:25 | zigo | I can read code, but there's things I still don't understand and just copy/past. | |
| 16:23:01 | sean-k-mooney[m] | ah in that case its proably one of the simpler references for that | |
| 16:23:11 | zigo | Thanks. | |
| 16:23:19 | sean-k-mooney[m] | its one of the newer and cleaner code bases as a result | |
| 16:24:08 | sean-k-mooney[m] | placement also breaks one convention for the better | |
| 16:24:15 | sean-k-mooney[m] | it does not use a global config instnace | |
| 16:24:27 | sean-k-mooney[m] | it passes it down when a function need the config | |
| 16:24:32 | sean-k-mooney[m] | that makes testing simpler | |
| 16:35:18 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova master: Correctly reset instance task state in rebooting hard https://review.opendev.org/c/openstack/nova/+/867832 | |
| 17:51:52 | danzelius | Hey there, I have some old instances I wan't to live-migrate. They used flavors which had some metadata assigned like "highmem: True". The scheduler can't find a host because the AggregateInstanceExtraSpecs-Filter (we included recently) filters them out since the "highmem" key is unscoped. In the nova database I found the instance_extra table and modified the instance in question, removing the extra_specs dict contents....Still the filter tries t | |
| 17:52:20 | danzelius | highmem"-metadata-field. Are there some other places in the db I have forgotten about? I also modified the original-flavor, removing the metadata, still no luck. | |
| 18:15:28 | opendevreview | Ghanshyam Mann proposed openstack/nova stable/zed: DNM: tetsing tox<4 pin https://review.opendev.org/c/openstack/nova/+/869250 | |
| 18:16:25 | opendevreview | Ghanshyam Mann proposed openstack/nova stable/yoga: DNM: tetsing tox<4 pin https://review.opendev.org/c/openstack/nova/+/869184 | |
| 18:16:41 | opendevreview | Ghanshyam Mann proposed openstack/nova stable/xena: DNM: tetsing tox<4 pin https://review.opendev.org/c/openstack/nova/+/869185 | |
| 18:16:50 | opendevreview | Ghanshyam Mann proposed openstack/nova stable/wallaby: DNM: tetsing tox<4 pin https://review.opendev.org/c/openstack/nova/+/869186 | |
| 19:27:39 | opendevreview | Ghanshyam Mann proposed openstack/nova master: Enable new defaults and scope checks by default https://review.opendev.org/c/openstack/nova/+/866218 | |
| 22:29:43 | dansmith | when did we stop being able to -1 specs? | |
| 22:30:04 | dansmith | oh, heh, because this one is merged, lol | |
| 22:30:17 | dansmith | guess I should refresh my tabs from early december | |
| #openstack-nova - 2023-01-05 | |||
| 01:52:34 | opendevreview | melanie witt proposed openstack/nova master: Add mock to avoid loading guestfs in unit test https://review.opendev.org/c/openstack/nova/+/862769 | |
| 02:16:29 | opendevreview | melanie witt proposed openstack/placement stable/ussuri: placement-status: check only consumers in allocation table https://review.opendev.org/c/openstack/placement/+/840703 | |
| 09:05:40 | sahid | o/ a gentle reminder if it's possible to get eyes on https://review.opendev.org/c/openstack/nova/+/858383 ? | |
| 09:30:56 | gibi | good morning | |
| 10:00:19 | Uggla | oh gibi, good morning and happy new year. | |
| 10:00:38 | gibi | Uggla: same to you too | |
| 10:01:42 | opendevreview | Balazs Gibizer proposed openstack/placement master: Make tox.ini tox 4.0.0 compatible https://review.opendev.org/c/openstack/placement/+/868418 | |
| 10:04:37 | gibi | bauzas, gmann: the os-vif tox4 fix is blocked by test failure from master confirmed by sean-k-mooney[m] last year. | |
| 10:04:56 | gibi | I fixed up the placement tox4 patch based on stephenfin's comments now | |
| 10:05:00 | bauzas | k | |
| 10:05:17 | bauzas | working on some reproducer downstream atm | |
| 10:05:52 | gibi | ack | |
| 12:31:19 | opendevreview | Konrad Gube proposed openstack/nova-specs master: Use extend volume completion action https://review.opendev.org/c/openstack/nova-specs/+/855490 | |
| 14:03:22 | sean-k-mooney | gibi: i have reviewed up to https://review.opendev.org/c/openstack/nova/+/854924/9 and am starting on it now. stephenfin is -1 on it i think just the docs/release notes can you adress those quickly? | |
| 14:03:36 | gibi | sean-k-mooney: looking... | |
| 14:04:21 | sean-k-mooney | i have approved everything before that so if you do fix it please avoid rebasing the previous patches :) | |
| 14:08:06 | gibi | sure :) | |
| 14:40:04 | opendevreview | Merged openstack/nova master: Support cold migrate and resize with PCI tracking in placement https://review.opendev.org/c/openstack/nova/+/854247 | |
| 15:05:35 | sean-k-mooney | gibi: stephenfin ok so at this point stephen and i have completed the review fo the pci seriese. i have approve most of it including the followps so there are two bits left. 1 the docs/release note fixes noted above and the refactor in the last patch which is optional | |
| 15:05:57 | gibi | sean-k-mooney: thanks I will provide the fix for the followup toda | |
| 15:05:58 | sean-k-mooney | so i think we can wrap this up this week | |
| 15:07:30 | sean-k-mooney | gibi: https://review.opendev.org/c/openstack/nova/+/854929/8 my suggestion fro that is make it a mixin instead and only mix it into the filters that need it | |
| 15:07:47 | gibi | I will think about that | |
| 15:08:22 | sean-k-mooney | ack i dont think its pressing in any case. if we did not merge the last patch it would have no ill effect one way or anohter | |
| 15:08:46 | gibi | yepp, this was the reason we moved it to the top | |
| 15:08:47 | sean-k-mooney | im going to go afk for a few mins chat in a bit | |
| 15:08:49 | gibi | so it does not block us | |
| 15:08:52 | gibi | ack | |
| 16:31:04 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova master: Reproducer test of bug #1999674 https://review.opendev.org/c/openstack/nova/+/867807 | |
| 16:31:04 | opendevreview | Pierre-Samuel Le Stang proposed openstack/nova master: Correctly reset instance task state in rebooting hard https://review.opendev.org/c/openstack/nova/+/867832 | |
| 16:41:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI scheduling in Placement https://review.opendev.org/c/openstack/nova/+/854924 | |
| 16:41:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Follow up for the PCI in placement series https://review.opendev.org/c/openstack/nova/+/855654 | |
| 16:41:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename _to_device_spec_conf to _to_list_of_json_str https://review.opendev.org/c/openstack/nova/+/855648 | |
| 16:41:32 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce PCI pool filtering bug https://review.opendev.org/c/openstack/nova/+/855649 | |
| 16:41:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: Strictly follow placement allocation during PCI claim https://review.opendev.org/c/openstack/nova/+/855650 | |
| 16:41:33 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP for the scheduler part of PCI in placement https://review.opendev.org/c/openstack/nova/+/862876 | |
| 16:41:34 | opendevreview | Balazs Gibizer proposed openstack/nova master: Split ignored_tags in stats.py https://review.opendev.org/c/openstack/nova/+/867978 | |