| *** mhen_ is now known as mhen | 02:41 | |
| opendevreview | Ivan Anfimov proposed openstack/freezer-specs master: Remove outdated comment in setup.py https://review.opendev.org/c/openstack/freezer-specs/+/974683 | 09:26 |
|---|---|---|
| opendevreview | Ivan Anfimov proposed openstack/freezer-specs master: Remove outdated comment in setup.py https://review.opendev.org/c/openstack/freezer-specs/+/974683 | 09:27 |
| opendevreview | Merged openstack/freezer master: docs: fix few mistakes https://review.opendev.org/c/openstack/freezer/+/974399 | 11:15 |
| opendevreview | Merged openstack/freezer-api master: Remove LegacyEngineFacade https://review.opendev.org/c/openstack/freezer-api/+/974447 | 13:36 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/freezer-api master: Ensure all_projects are defined for get_search_query https://review.opendev.org/c/openstack/freezer-api/+/973886 | 14:07 |
| opendevreview | Merged openstack/freezer-api master: Use SSL and authentication for elasticsearch https://review.opendev.org/c/openstack/freezer-api/+/974411 | 14:50 |
| noonedeadpunk | #startmeeting freezer | 15:00 |
| opendevmeet | Meeting started Mon Jan 26 15:00:18 2026 UTC and is due to finish in 60 minutes. The chair is noonedeadpunk. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'freezer' | 15:00 |
| noonedeadpunk | #topic rollcall | 15:00 |
| noonedeadpunk | o/ | 15:00 |
| noonedeadpunk | hey there! | 15:00 |
| mhen | o/ | 15:00 |
| noonedeadpunk | we have quite an agenda today btw | 15:01 |
| noonedeadpunk | #topic CI status | 15:03 |
| noonedeadpunk | So, CI for the project should be restored now, with https://review.opendev.org/c/openstack/freezer-api/+/974447 | 15:03 |
| mhen | good job! | 15:04 |
| noonedeadpunk | It was caused by oslo.db finally getting rid of legacy engine | 15:04 |
| noonedeadpunk | So now I'm re-checking things and will continue reviewing existing patches | 15:04 |
| noonedeadpunk | With that I kinda got stuck a little bit with elasticsearch engine on devstack | 15:05 |
| mhen | my patches pass tests again, yay \o/ | 15:05 |
| noonedeadpunk | (I hope I did not break anything more then it's broken with this change) | 15:05 |
| noonedeadpunk | #topic Elasticsearch DB | 15:05 |
| noonedeadpunk | I have spent quite some time into elasticsearch in devstack, once it's version was bumped | 15:06 |
| noonedeadpunk | while devstack does actually deploy now, it's completely borked | 15:06 |
| noonedeadpunk | Right now U-C has elactisearch library on 2.4.1: https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L332 | 15:07 |
| noonedeadpunk | With ELK 8/9 we need to update it to 9 basically, according to https://github.com/elastic/elasticsearch-py/tree/v9.2.1?tab=readme-ov-file#compatibility | 15:07 |
| noonedeadpunk | But then there are plenty of huge changes, so source should be refactored pretty much significantly as well | 15:08 |
| noonedeadpunk | Not impossible to do | 15:08 |
| noonedeadpunk | BUT. I wonder if we really wanna have that option... | 15:08 |
| mhen | what is the benefit for Freezer to use ES? | 15:08 |
| noonedeadpunk | I checked through the codesearch, and except kolla/helm (which have it for infra reasons), only freezer and venus having it as engine | 15:10 |
| noonedeadpunk | I think intention is to have faster alternative to relational DB | 15:11 |
| mhen | but is Freezer really under that heavy load with managing the clients and jobs to justify that? | 15:11 |
| mhen | I think some other OpenStack core components are much busier | 15:12 |
| noonedeadpunk | that is really a great question | 15:12 |
| noonedeadpunk | I'd say - it not | 15:13 |
| mhen | my suggestion would be to remove it as it's broken anyway | 15:13 |
| noonedeadpunk | I am inclining to that | 15:14 |
| noonedeadpunk | I think we should have some deprecation period, but right after 2026.1 we should be able to drop it | 15:14 |
| mhen | if the need arises because somebody identifies a bottleneck, it can be addressed by (re)introducing an alternative backend when the time comes | 15:14 |
| noonedeadpunk | and now we can just switch default devstack logic to use mariadb | 15:14 |
| mhen | +1 | 15:14 |
| noonedeadpunk | I think the most likely issue would be to drop historical records from DB | 15:15 |
| noonedeadpunk | but indeed - let's come to bridge before crossing itr | 15:15 |
| noonedeadpunk | #agreed deprecate elasticsearch engine and switch devstack defaults to use sqlalchemy | 15:16 |
| noonedeadpunk | #topic horizon plugin state | 15:16 |
| noonedeadpunk | I _think_ I have mostly fixed the horizon plugin | 15:17 |
| noonedeadpunk | with that I also was reviewing the patch when realized it's probably better to go slightly further | 15:17 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/freezer-web-ui/+/958584 | 15:17 |
| noonedeadpunk | and under further I mean getting rid of own dashboard and top-level menu, with getting group under Projects' | 15:18 |
| noonedeadpunk | Sorry, previous link was wrong, I meant https://review.opendev.org/c/openstack/freezer-web-ui/+/961314 | 15:18 |
| noonedeadpunk | So I came up with the following one | 15:19 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/freezer-web-ui/+/973949 | 15:19 |
| noonedeadpunk | It would be really nice to have some external "verification" of it | 15:19 |
| mhen | what do you mean with "external verification"? | 15:20 |
| noonedeadpunk | if it makes sense to somebody except of me, and if works :D | 15:21 |
| mhen | I could test it on my devstack and have a look at it if it helps | 15:21 |
| mhen | however, not before next week | 15:21 |
| * mhen is in a different context this week and cannot work on Freezer until next week | 15:21 | |
| noonedeadpunk | ok, I see | 15:22 |
| noonedeadpunk | I think there's no rush with this change | 15:22 |
| noonedeadpunk | and I'll also try to ask somebody from my org to test it in the meanwhile | 15:22 |
| noonedeadpunk | #topic container names passed to cinder-backup with invalid separators | 15:23 |
| noonedeadpunk | We've discussed that during previous week, and I see you've proposed a patch | 15:23 |
| noonedeadpunk | #link https://review.opendev.org/c/openstack/freezer/+/973627/12/freezer/openstack/backup.py#b91 | 15:23 |
| noonedeadpunk | I have not reviewed it yet, so will post an update in the patch, if it's fine with you | 15:24 |
| mhen | sure | 15:24 |
| mhen | it was a minor change so I added it to my Cinder fixup patchset | 15:24 |
| noonedeadpunk | ++ sure | 15:25 |
| noonedeadpunk | #topic restore-from-date behavior is different from documented one | 15:25 |
| noonedeadpunk | there's also a related bug report to this topic | 15:25 |
| noonedeadpunk | #link https://bugs.launchpad.net/freezer/+bug/2138883 | 15:25 |
| mhen | yea I thought keeping it as a bug is better to document the observed behavior | 15:26 |
| noonedeadpunk | yeah, I think I also saw some patch related to this topic? | 15:26 |
| mhen | you did? | 15:27 |
| noonedeadpunk | nah, maybe I am thinking about smth completely different now | 15:27 |
| noonedeadpunk | I can take this as an action point during this week | 15:27 |
| noonedeadpunk | #action noonedeadpunk to triage the bug report and propose a fix | 15:28 |
| mhen | we need to decide whether we want to have it behave like it says or if we simply adjust the documentation to reflect its actual behavior | 15:28 |
| noonedeadpunk | it could be also that we might want to rename the option to be more explicit about what it's doing... | 15:29 |
| mhen | from a user perspective I would lean towards the current documented behavior making a bit more sense and that the implementation is wrong | 15:30 |
| mhen | re: rename - yes that would also help | 15:30 |
| mhen | because currently it suggests that it will restore that exact timestamp down to the second when in reality it actually has a fuzziness to it | 15:31 |
| mhen | maybe we can discuss in the bug | 15:31 |
| noonedeadpunk | yes, sounds good | 15:31 |
| mhen | that would then also serve as some kind of decision record | 15:31 |
| noonedeadpunk | I am not sure so far what is most reasonable/expected behavior would be | 15:32 |
| * noonedeadpunk needs to sleep over with this | 15:32 | |
| noonedeadpunk | #topic consistent OpenStack resource naming | 15:33 |
| noonedeadpunk | there's a patch which suggest to align naming of created by Freezer resources | 15:34 |
| noonedeadpunk | #link https://bugs.launchpad.net/freezer/+bug/2138883 | 15:34 |
| mhen | #link https://review.opendev.org/c/openstack/freezer/+/974483/1/releasenotes/notes/consistent-openstack-resource-naming-fa8f3d72c24faefa.yaml | 15:34 |
| mhen | the release notes explain in detail how the change works | 15:34 |
| noonedeadpunk | oops I sent a wrong link again | 15:34 |
| noonedeadpunk | `~` - that is really a smart one | 15:35 |
| mhen | RFC 3986 said its safe | 15:35 |
| mhen | and it is less used by users and other components | 15:35 |
| noonedeadpunk | yeah, it must be. It did not come to my mind | 15:35 |
| noonedeadpunk | yup, really like it | 15:36 |
| mhen | I found the resulting user experience to be tremendously better when observing all the resources during backup/restore | 15:36 |
| noonedeadpunk | there seems to be some related pep8 issues | 15:37 |
| noonedeadpunk | #link https://zuul.opendev.org/t/openstack/build/ec4b099489ab48fda74b87eb8dc91ffa | 15:37 |
| mhen | I know | 15:37 |
| mhen | will address them | 15:37 |
| noonedeadpunk | otherwise it looks quite fair to me now | 15:38 |
| noonedeadpunk | this indeed should improve quality of life for users | 15:38 |
| opendevreview | Merged openstack/freezer master: Fix issues in Python code https://review.opendev.org/c/openstack/freezer/+/973620 | 15:38 |
| noonedeadpunk | ok, I guess this should be pretty much it from my side at least | 15:39 |
| noonedeadpunk | #topic open discussion | 15:39 |
| mhen | on another note I noticed that the backup timestamp of a cinder volume backup (via glance image) communicated by freezer-agent's output mismatches the resulting timestamp on disk in the backup directory | 15:39 |
| noonedeadpunk | it's off by couple of seconds? | 15:39 |
| mhen | #link https://github.com/openstack/freezer/blob/master/freezer/openstack/backup.py#L66 | 15:39 |
| noonedeadpunk | yeah, I think we should be taking timestamp from the resource.... | 15:40 |
| mhen | yes, because of this part in the code where a new timestamp is generated after the initial one from the job's start | 15:40 |
| noonedeadpunk | which is really confusing, agree | 15:41 |
| mhen | just a minor thing I noticed while I tested my timestamp appendix patch | 15:41 |
| noonedeadpunk | can take a look during the week | 15:41 |
| mhen | great | 15:41 |
| mhen | as stated before, I sadly cannot contribute this week | 15:42 |
| noonedeadpunk | this is fine, there're quite some outstanding patches from you to review/merge | 15:42 |
| noonedeadpunk | ok, cool then, thanks for joining! | 15:44 |
| noonedeadpunk | #endmeeting | 15:45 |
| opendevmeet | Meeting ended Mon Jan 26 15:45:40 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:45 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/freezer/2026/freezer.2026-01-26-15.00.html | 15:45 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/freezer/2026/freezer.2026-01-26-15.00.txt | 15:45 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/freezer/2026/freezer.2026-01-26-15.00.log.html | 15:45 |
| opendevreview | Merged openstack/freezer-api master: Ensure all_projects are defined for get_search_query https://review.opendev.org/c/openstack/freezer-api/+/973886 | 15:46 |
| opendevreview | Merged openstack/freezer master: Fix Glance integration https://review.opendev.org/c/openstack/freezer/+/973623 | 16:45 |
| opendevreview | Markus Hentsch proposed openstack/freezer master: Fix Nova integration and change clients to SDK https://review.opendev.org/c/openstack/freezer/+/973625 | 16:58 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/freezer-api master: Remove redundant wsgi file https://review.opendev.org/c/openstack/freezer-api/+/974754 | 18:03 |
| opendevreview | Merged openstack/freezer master: Fix Nova integration and change clients to SDK https://review.opendev.org/c/openstack/freezer/+/973625 | 18:11 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/freezer-api master: Update contributor guide https://review.opendev.org/c/openstack/freezer-api/+/974755 | 18:12 |
| opendevreview | Merged openstack/freezer master: Remove installation guide for openSUSE/SLES https://review.opendev.org/c/openstack/freezer/+/974659 | 18:19 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/freezer master: Add [os_brick]/lock_path config option https://review.opendev.org/c/openstack/freezer/+/974772 | 20:07 |
| opendevreview | Merged openstack/freezer master: Replace cinderclient with SDK, fix cinder engines https://review.opendev.org/c/openstack/freezer/+/973627 | 20:49 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!