| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-08-15 | |||
| 18:22:34 | sean-k-mooney | ill check tomorrow | |
| 21:19:32 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads on server create https://review.opendev.org/c/openstack/nova/+/891520 | |
| #openstack-nova - 2023-08-16 | |||
| 04:13:47 | opendevreview | Merged openstack/nova master: Decorate only Flavor.get_* methods that execute queries https://review.opendev.org/c/openstack/nova/+/888513 | |
| 05:11:42 | opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | |
| 05:11:42 | opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | |
| 05:11:43 | opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | |
| 07:13:34 | kevko | sean-k-mooney thank you, my colleguage viktor will reply today ..btw, several customers wants this feature (az-afiinity/anti-az-afinnity ) from us ...i was also discussing this with someone on summit ..i think julia kreger .. | |
| 07:29:36 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| 08:00:04 | gibi | dansmith: re: Cursor needed to be reset because of commit/rollback and can no longer be fetched from.: That seems fairly frequent in nova. At least there was 15 hint in the last 30 days | |
| 12:18:07 | sean-k-mooney | kevko: yep we have discussed it at the summit before. it could be done but not the way you suggested it requires addtional change to the api. just for context this has been disccused at at least 3 PTGs and has been rejected in the form ye preposed in the spec at least twice | |
| 12:18:24 | sean-k-mooney | i was not at the last pyshical summit. | |
| 12:18:53 | sean-k-mooney | effectivly we need a new api microversion to opt into dynamicaly selected AZ | |
| 12:19:15 | sean-k-mooney | we then need to ensure that when that is requested that we dont populate an az in the api | |
| 12:20:17 | sean-k-mooney | then we need a something likely prior to the placement query to determin what azs can be used althogh that could happen after a a filter/weigher | |
| 12:20:50 | supamatt | As an operator using openstack, I do concur with kevko.. this is a much needed feature when using multiple Nova AZs | |
| 12:21:42 | sean-k-mooney | supamatt: you reasise we declard doing this to be out of scope of nova to do the last time it was proposed as a spec | |
| 12:22:36 | supamatt | I saw your comments to what Ultimum was proposing | |
| 12:24:03 | sean-k-mooney | under the "No more Orchestration" https://docs.openstack.org/nova/latest/contributor/project-scope.html#no-more-orchestration | |
| 12:24:59 | sean-k-mooney | princiapl. i.e. we do not want to implement advanced orchestation supprot in nova itselsf. with that said there are less invaisve ways to enable this that might be in scope | |
| 12:26:37 | sean-k-mooney | supamatt: i understand why peope want this but for technial and practical reasons this is not something that could be enbled without using a new api microverion to have in my opipion | |
| 12:27:35 | sean-k-mooney | with the new api microversion however we could enable this feature. that microversion si opting out of the current defaulting that prevents the schduler for selecting AZs | |
| 12:27:50 | supamatt | I think that's a good idea. | |
| 12:29:40 | sean-k-mooney | i need to go to the pharmacy and pick up my blood presseur persciption so ill be afk for 30 mins or so. | |
| 12:30:02 | sean-k-mooney | but in general we can try evolving the spec that is propsoed to make it something that will actully work | |
| 12:30:48 | sean-k-mooney | that is why i did not -2 it. im not agaisnt the feature in genral jsut wanted to let kevko know that the acutal cahnge will need to be more complicated then sugested in the spec | |
| 13:30:43 | dansmith | gibi: ack, I hadn't seen it on any of mine in a long time | |
| 14:56:39 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads in resize https://review.opendev.org/c/openstack/nova/+/891605 | |
| 20:40:00 | opendevreview | Merged openstack/nova master: Add upgrade check for compute-object-ids linkage https://review.opendev.org/c/openstack/nova/+/889740 | |
| 20:43:58 | kevko | sean-k-mooney: my colleguage has working prototype with bumped microversion as you mentioned above ...and working .. wouldn't be best to share somehow this temporary work ? | |
| 20:44:22 | kevko | we know that bump microversion is needed | |
| 20:47:38 | kevko | sean-k-mooney thank you very much for discussion, I appreciate it very much, we (of course) know that the spec will be probably reworked and we are open to amend the spec and implement as we agree ... after all, there is a gerrit review for that so that we can get to the final patchset :) | |
| 21:16:50 | opendevreview | Merged openstack/nova master: Fix bindep for Debian bookworm https://review.opendev.org/c/openstack/nova/+/891256 | |
| #openstack-nova - 2023-08-17 | |||
| 04:44:47 | opendevreview | melanie witt proposed openstack/nova master: WIP nova-manage: Add 'limits migrate_to_unified_limits' https://review.opendev.org/c/openstack/nova/+/891646 | |
| 05:26:37 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/891648 | |
| 08:52:48 | opendevreview | Bence Romsics proposed openstack/nova master: Do not untrack resources of a server being unshelved https://review.opendev.org/c/openstack/nova/+/890406 | |
| 11:00:56 | stephenfin | could I get eyes on https://review.opendev.org/c/openstack/placement/+/888281 to close out the last sqlalchemy 2.x issue with placement? | |
| 14:58:53 | opendevreview | Elod Illes proposed openstack/osc-placement master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/osc-placement/+/891795 | |
| 15:03:33 | opendevreview | Elod Illes proposed openstack/os-vif master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/os-vif/+/891812 | |
| 15:09:30 | opendevreview | Elod Illes proposed openstack/python-novaclient master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/python-novaclient/+/891831 | |
| 22:30:21 | opendevreview | melanie witt proposed openstack/nova stable/2023.1: Decorate only Flavor.get_* methods that execute queries https://review.opendev.org/c/openstack/nova/+/891749 | |
| #openstack-nova - 2023-08-18 | |||
| 02:08:13 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| 05:42:24 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| 10:00:22 | opendevreview | melanie witt proposed openstack/nova master: Enable use of native threads in scatter_gather_cells https://review.opendev.org/c/openstack/nova/+/650172 | |
| 12:38:49 | opendevreview | Tobias Urdin proposed openstack/nova master: Fix wrong nova-manage command in upgrade check https://review.opendev.org/c/openstack/nova/+/880819 | |
| 12:42:45 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: set remaining to 0 when no disk to migrate https://review.opendev.org/c/openstack/nova/+/873846 | |
| 12:43:14 | opendevreview | Tobias Urdin proposed openstack/nova master: libvirt: set remaining to 0 when no disk to migrate https://review.opendev.org/c/openstack/nova/+/873846 | |
| 12:44:44 | andrewbonney | Hi. Let me know if this is best to wait until the weekly meeting, but I'm wondering if someone could take a look at https://bugs.launchpad.net/nova/+bug/2023035 ? | |
| 12:45:08 | andrewbonney | This is blocking us moving from Zed to 2023.1 in production unless we enable the old 'skip_cpu_compare_on_dest' workaround | |
| 22:49:35 | opendevreview | Spencer Harmon proposed openstack/nova master: libvirt tsc_frequency https://review.opendev.org/c/openstack/nova/+/891924 | |
| #openstack-nova - 2023-08-19 | |||
| 06:43:29 | opendevreview | Wenping Song proposed openstack/placement master: Fix missing oslo.versionedobjects library option https://review.opendev.org/c/openstack/placement/+/891925 | |
| #openstack-nova - 2023-08-21 | |||
| 08:49:53 | opendevreview | Balazs Gibizer proposed openstack/nova master: [functional] Don't leak running live migration https://review.opendev.org/c/openstack/nova/+/892126 | |
| 08:56:32 | gibi | dansmith, melwitt: ^^ that might help stabilizing other functional tests too | |
| 08:57:14 | gibi | as it is eventually leaked a greenlet after our cleanup code waiting for the greenlet to stop timed out | |
| 09:13:25 | auniyal | gibi, this change deletes the instance before the end of the test case to cancel the live migration .... | |
| 09:13:25 | auniyal | I am sure, I am missing someting, but migration is already finished right and its verified at here ? | |
| 09:13:25 | auniyal | https://review.opendev.org/c/openstack/nova/+/892126/1/nova/tests/functional/test_servers.py#1789 | |
| 09:21:41 | auniyal | okay, it's checking for project_2 but admin_api project is project_1, for which migration started , right ? | |
| 10:15:32 | admin1 | hi all .. i had a server added to a new cluster with right hostname but wrong fqdn . fqdn is corrected .. did openstack resource provider delete $uuid and openstack compute service delete $uuid of the affected compute node .. when i restart nova-compute , i still get No resource provider with $uuid found .. -- what is the correct way to | |
| 10:15:32 | admin1 | rectify such issues | |
| 10:33:07 | admin1 | it created resource provider .. host appears in resource provider list and in compute-service list, but not in nova hypervisor-list .. ..... restarting nova-compute does not give any error in journamctl | |
| 11:06:30 | gibi | auniyal: it is not finished it is just not visible from project_2 perspective | |
| 11:11:11 | auniyal | gibi ack, | |
| 11:11:40 | auniyal | just notice you replied here I had saved comment i patch | |
| 11:34:00 | sean-k-mooney | gibi: can you hit this quickly https://review.opendev.org/c/openstack/os-vif/+/885127 | |
| 11:38:14 | gibi | sean-k-mooney: done | |
| 11:39:25 | sean-k-mooney | thanks | |
| 11:39:46 | sean-k-mooney | i need to reboot my laptop after updates so ill be afk for a few muintes but be back soon | |
| 15:24:36 | opendevreview | Dan Smith proposed openstack/nova master: Log excessive lazy-loading behavior https://review.opendev.org/c/openstack/nova/+/891340 | |
| 15:24:37 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loading in resize and rebuild/evacuate https://review.opendev.org/c/openstack/nova/+/891336 | |
| 15:24:37 | opendevreview | Dan Smith proposed openstack/nova master: Remove a lazy load on every server show https://review.opendev.org/c/openstack/nova/+/891352 | |
| 15:24:38 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads on server create https://review.opendev.org/c/openstack/nova/+/891520 | |
| 15:24:38 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads in resize https://review.opendev.org/c/openstack/nova/+/891605 | |
| 15:25:22 | opendevreview | Dan Smith proposed openstack/nova master: Log excessive lazy-loading behavior https://review.opendev.org/c/openstack/nova/+/891340 | |
| 15:25:22 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loading in resize and rebuild/evacuate https://review.opendev.org/c/openstack/nova/+/891336 | |
| 15:25:23 | opendevreview | Dan Smith proposed openstack/nova master: Remove a lazy load on every server show https://review.opendev.org/c/openstack/nova/+/891352 | |
| 15:25:23 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads on server create https://review.opendev.org/c/openstack/nova/+/891520 | |
| 15:25:24 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads in resize https://review.opendev.org/c/openstack/nova/+/891605 | |
| 15:43:11 | admin1 | hi experts .. what does this error mean ? https://pastebin.com/raw/7JMScxZs | |
| 15:43:24 | admin1 | No such resource class MEMORY_MB. | |
| 15:53:28 | opendevreview | Dan Smith proposed openstack/nova master: Log excessive lazy-loading behavior https://review.opendev.org/c/openstack/nova/+/891340 | |
| 15:53:28 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loading in resize and rebuild/evacuate https://review.opendev.org/c/openstack/nova/+/891336 | |
| 15:53:29 | opendevreview | Dan Smith proposed openstack/nova master: Remove a lazy load on every server show https://review.opendev.org/c/openstack/nova/+/891352 | |
| 15:53:29 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads on server create https://review.opendev.org/c/openstack/nova/+/891520 | |
| 15:53:30 | opendevreview | Dan Smith proposed openstack/nova master: Avoid lazy-loads in resize https://review.opendev.org/c/openstack/nova/+/891605 | |
| 15:53:34 | dansmith | gibi: guh, sorry forgot to update the log assertion in the test ^ | |
| 16:03:22 | admin1 | how to populate placment resource-classes | |
| 16:04:35 | sean-k-mooney | admin1: its done automaticaly by placement when the placement api starts for all standard resouce classes | |
| 16:05:54 | admin1 | ok . i will try to reboot all placement containers ( openstack-ansible ) and see if it repopulates | |
| 16:07:16 | sean-k-mooney | https://github.com/openstack/placement/blob/master/placement/objects/resource_class.py#L209-L253 | |
| 16:07:49 | sean-k-mooney | that gets called here https://github.com/openstack/placement/blob/master/placement/deploy.py#L129-L158 | |
| 16:08:42 | sean-k-mooney | whihc is the entry point of the wsgi service here https://github.com/openstack/placement/blob/master/placement/wsgi.py#L147 | |
| 16:23:22 | sean-k-mooney | dansmith: one nit in https://review.opendev.org/c/openstack/nova/+/891340 but +2 | |
| 16:23:57 | sean-k-mooney | gibi: dan uploaded a revsions since you last reviewed so it lost your +2 | |
| 16:31:26 | opendevreview | Merged openstack/nova master: [functional] Don't leak running live migration https://review.opendev.org/c/openstack/nova/+/892126 | |
| 16:40:02 | admin1 | sean-k-mooney, that worked .. many thanks .. | |
| 16:40:29 | sean-k-mooney | for what its worth looking at the code i dont see any way that service can start without populating the db | |
| 16:40:49 | sean-k-mooney | so i assume there was a db connection string config issue or similar | |
| 16:41:04 | sean-k-mooney | perhaps the database was not creted when the contaienr was started or somethign like that | |
| 22:18:10 | opendevreview | Merged openstack/nova master: Remove n-v ceph live migration job from gate https://review.opendev.org/c/openstack/nova/+/889992 | |
| 23:21:13 | opendevreview | Spencer Harmon proposed openstack/nova master: libvirt tsc_frequency https://review.opendev.org/c/openstack/nova/+/891924 | |