| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-02-26 | |||
| 00:38:21 | sean-k-mooney[m] | there si allso osops tools which has a nova specifc srcipt https://opendev.org/openstack/osops/src/branch/master/tools/generic/nova/openstack_db_archive.sh | |
| 00:39:26 | melwitt | yeah, I see what you mean now. I agree I think that would be ideal for there to be a unified thing that will work for all of the projects | |
| 00:39:59 | melwitt | when you said "external" I thought you meant external to openstack and was confused | |
| 00:41:22 | sean-k-mooney[m] | https://github.com/ovh/osarchiver | |
| 00:41:34 | sean-k-mooney[m] | well i was thinking of ^ | |
| 00:41:56 | sean-k-mooney[m] | thats the thing that basicaly does this for all proejcts in a consistent way | |
| 00:42:08 | melwitt | oh, interesting | |
| 00:42:59 | sean-k-mooney[m] | it basically does not use shadowtable at all | |
| 00:43:20 | sean-k-mooney[m] | it can arcive without deletign or it can archive and delete the rows in teh service dbs | |
| 00:43:27 | sean-k-mooney[m] | to get a similar effect | |
| 00:44:12 | sean-k-mooney[m] | given you can restore thing from the shadown tabels to the main tables i would actully say its more usefual because it mvoes the data to an entirly differnt db in teh same format | |
| 00:44:29 | sean-k-mooney[m] | which is you ever wanted to restore those rows back for any reason would be simpler | |
| 00:45:11 | sean-k-mooney[m] | its like doing incremental backups of only the rows marked as deleted | |
| 00:45:40 | sean-k-mooney[m] | the reason osarchiver is not hosted in opendev is because we didnt agree which repo to put it in | |
| 00:45:45 | sean-k-mooney[m] | which is a shame | |
| 00:45:50 | melwitt | yeah, I was just thinking that if people kept incremental backups there would be no use for shadow tables | |
| 00:46:14 | melwitt | hah, nice | |
| 00:46:53 | sean-k-mooney[m] | there was a maining list tread about it like 4 years ago i belvie and it just stalled out | |
| 00:47:29 | melwitt | reposhed | |
| 00:47:59 | sean-k-mooney[m] | melwitt: anyway i think that a better technial solution i just dont think we will ever get to that at least not unless something changes | |
| 00:48:29 | sean-k-mooney[m] | i wanted to use that in our new installer but…. it was too much effort to advocate for that and get by in | |
| 00:49:23 | melwitt | yeah. especially difficult if it's not in opendev | |
| 00:50:02 | sean-k-mooney[m] | it was more that pm didnt want use to focus on day two operations like automating backups of db in the operators | |
| 00:51:04 | sean-k-mooney[m] | so providing incremental backups and “archiving” of this in a centralised way was not a priority | |
| 00:51:16 | sean-k-mooney[m] | so we just use nova manage in a k8s cron job instead | |
| 00:51:39 | sean-k-mooney[m] | i.e. we keep is simpel using the upstream tooling | |
| 00:52:15 | sean-k-mooney[m] | im not saying it could never be in scope but it was not in the mvp list for v1 | |
| 00:52:32 | melwitt | yeah | |
| 00:54:00 | sean-k-mooney[m] | ill try and add that to my review list for tomorrow | |
| 00:54:29 | sean-k-mooney[m] | its not nessisally impacted by ff but while i have context i might as well | |
| 00:55:26 | melwitt | haha thanks | |
| 00:55:48 | sean-k-mooney[m] | if you feel like reviewing something small this and the repoducer under neath is waiting for a second +2 https://review.opendev.org/c/openstack/nova/+/932931/2 although i pinged sylain to hopefully take a look above | |
| 00:57:01 | sean-k-mooney[m] | i think its time i went to sleep chat to you tomorrow o/ | |
| 00:57:07 | melwitt | sylvain would be ideal for that one but if he doesn't get to it I can do it | |
| 00:57:13 | melwitt | seeya tomorrow o/ | |
| 01:24:46 | opendevreview | Merged openstack/nova master: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/909611 | |
| 01:39:28 | opendevreview | melanie witt proposed openstack/nova master: DNM test headers https://review.opendev.org/c/openstack/nova/+/942752 | |
| 06:38:33 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/942759 | |
| 06:45:22 | opendevreview | melanie witt proposed openstack/nova master: DNM test headers https://review.opendev.org/c/openstack/nova/+/942752 | |
| 08:50:33 | opendevreview | Merged openstack/nova master: Add repoducer test for bug 2074219 https://review.opendev.org/c/openstack/nova/+/932930 | |
| 09:11:53 | bauzas | gibi: sean-k-mooney[m] : I'd appreciate if you could get a swing for https://review.opendev.org/c/openstack/nova/+/845757/ | |
| 09:12:08 | bauzas | I'll also try to look at the functests failing | |
| 09:20:42 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.2: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/942764 | |
| 09:31:03 | opendevreview | Amit Uniyal proposed openstack/nova stable/2024.1: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/942765 | |
| 09:34:49 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 09:54:29 | opendevreview | Amit Uniyal proposed openstack/nova stable/2023.2: WIP: remove swap.disk from disk.info https://review.opendev.org/c/openstack/nova/+/942769 | |
| 10:17:21 | gibi | bauzas: is it something we need to land before FF or something we can land after FF? | |
| 10:17:53 | bauzas | gibi: well, this is not really a bugfix | |
| 10:18:17 | bauzas | but it fixes evecuates for VGPUs | |
| 10:38:37 | opendevreview | Merged openstack/nova master: Fix detaching devices by alias with mdevs https://review.opendev.org/c/openstack/nova/+/932931 | |
| 11:18:49 | opendevreview | Callum Dickinson proposed openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 11:25:09 | sean-k-mooney | bauzas: ack ill add it to my list, i have to do a pass on the nova-operator stuff and then melwitt's nova-manage command today but i think i can get to those as well | |
| 11:25:59 | sean-k-mooney | bauzas: i think it might fix evacuate for anything with nested resouce providers if i rememebr correctly | |
| 11:26:41 | sean-k-mooney | maybe not | |
| 11:27:06 | sean-k-mooney | that not the patch i was thinkign of | |
| 11:43:02 | opendevreview | Stefan Hoffmann proposed openstack/os-vif master: [ovs] Add method to get external_ids from Interface https://review.opendev.org/c/openstack/os-vif/+/942715 | |
| 12:04:40 | opendevreview | Stefan Hoffmann proposed openstack/nova master: optional wait for ovn plug and create tap https://review.opendev.org/c/openstack/nova/+/942786 | |
| 16:30:52 | frickler | Uggla: please check https://review.opendev.org/c/openstack/governance/+/942507/comment/029df38c_6df9f638/ . it would likely also be helpful if you could join the #openstack-tc channel while you are PTL | |
| 16:32:36 | Uggla | frickler, I'm currently in a meeting, but I'will have a look after. | |
| 16:34:04 | frickler | Uggla: thx, not urgent, just so you don't miss it | |
| 16:53:44 | sean-k-mooney | dviroel: left some minor feedback on https://review.opendev.org/c/openstack/nova/+/938604 but i think this is mostly good to go | |
| 16:55:56 | dviroel | sean-k-mooney: oh, thanks, indeed, going to update it | |
| 16:56:25 | sean-k-mooney | im looking at the tempest patch now so far looks good to me | |
| 17:05:21 | sean-k-mooney | dviroel: minor comment on https://review.opendev.org/c/openstack/tempest/+/939375 as well | |
| 17:05:40 | sean-k-mooney | over all it looks fine to me but i htink you need to update teh import | |
| 17:06:19 | sean-k-mooney | gmann: ^ would you be able to comment on the questions i have inline and confirm the schema inheritance is correct or not? | |
| 17:38:49 | opendevreview | Douglas Viroel proposed openstack/nova master: Add support for showing scheduler_hints in server details https://review.opendev.org/c/openstack/nova/+/938604 | |
| 17:40:42 | dviroel | sean-k-mooney: ^ updated. Lets see if functional is going to pass. I added a scheduler hint to the server creation req | |
| 17:41:00 | dviroel | in the api_sample_tests | |
| 17:45:34 | sean-k-mooney | assuming it green i think ill be +2 | |
| 17:45:41 | sean-k-mooney | ill check back in a bit once we get some results | |
| 17:46:40 | sean-k-mooney | if it does break thign we can revert that chagne and try and update it in a follow uponce the feature is merged | |
| 17:48:15 | sean-k-mooney | dviroel: it looks a little funky https://5671b50e1757693146fd-d2676880cd4ce07846d9d5d9e5ab28d1.ssl.cf1.rackcdn.com/938604/11/check/build-openstack-api-ref/6d89054/docs/#id30 | |
| 17:48:35 | sean-k-mooney | i have not looked at the code but did you really use a regex? | |
| 17:48:37 | sean-k-mooney | "scheduler_hints": { | |
| 17:48:39 | sean-k-mooney | "same_host": [ | |
| 17:48:41 | sean-k-mooney | "[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}" | |
| 17:48:43 | sean-k-mooney | ] | |
| 17:48:45 | sean-k-mooney | }, | |
| 17:51:40 | dviroel | i was reusing an example from a sever creation | |
| 17:51:45 | dviroel | v2.63 i think | |
| 17:52:08 | sean-k-mooney | ok well it net to be a uuid | |
| 17:52:19 | sean-k-mooney | as in the example respocne shoudl not be the validation critia | |
| 17:52:37 | sean-k-mooney | it shoudl be somethign like "dc877f7e-7d69-4c3c-956b-1978cd8ed754" | |
| 17:52:59 | dviroel | yeah, let me fix that | |
| 17:53:28 | sean-k-mooney | its apprently an arry of uuids | |
| 17:53:45 | sean-k-mooney | at least based on your tempets change | |
| 17:55:25 | sean-k-mooney | oh | |
| 17:55:36 | sean-k-mooney | it can be a single uuid as a stirng or an away or them | |
| 17:55:59 | sean-k-mooney | the 2.90 create exmaple has | |
| 17:56:04 | sean-k-mooney | "OS-SCH-HNT:scheduler_hints": { | |
| 17:56:06 | sean-k-mooney | "same_host": "48e6a9f6-30af-47e0-bc04-acaed113bb4e" | |
| 17:56:08 | sean-k-mooney | } | |
| 17:56:43 | sean-k-mooney | as an example in the request | |
| 17:56:58 | sean-k-mooney | so you can just do the same in the responce example | |
| 17:57:18 | sean-k-mooney | well without the OS-SCH-HNT prefix | |
| 17:57:52 | sean-k-mooney | everything else looked good excpti the example value | |
| 18:01:10 | dviroel | yeah, they are adding this in the request sample, but not in the template, but the test does not validate that | |
| 18:03:48 | melwitt | huh, yeah, this should not have been a regex 😝 https://github.com/openstack/nova/blob/master/doc/api_samples/servers/v2.63/server-create-req.json someone missed that in review | |
| 18:04:08 | sean-k-mooney | ya.. i noticed it was broken there | |