| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-08 | |||
| 18:41:23 | chris218 | Hmm so openstack has everything home grown 😅 | |
| 18:42:12 | sean-k-mooney | not entirly openstack predates a lot of the modern python tooling | |
| 18:42:25 | sean-k-mooney | pbr is just a wrapper around setup tools | |
| 18:43:08 | chris218 | Yea like having own api-ref system instead of OpenAPI since it's the "new" hot thing | |
| 18:43:35 | sean-k-mooney | its not about it being the new hot thing | |
| 18:43:48 | sean-k-mooney | we actully discused supproting openAPI a few years ago | |
| 18:44:03 | sean-k-mooney | but we did nto have contibutors that coudl work on it | |
| 18:44:25 | artom | Nobody here is hot or new :D | |
| 18:44:33 | chris218 | Haha | |
| 18:44:41 | chris218 | I'm new hello | |
| 18:45:05 | sean-k-mooney | the api-ref is part of our docs by the way where as openapi is a specification for defining api and automatic client generation form those schemas | |
| 18:46:02 | sean-k-mooney | we have json schemas for exmaple in our apis already but we never had the time to rewirte themn to and openapi compaitble format | |
| 18:46:25 | chris218 | Well you can also generate docs from it, I've seen one project of person from sdk team working on it privately | |
| 18:47:04 | sean-k-mooney | right i knwo there has been some discussion about it in the context of the opsntack sdk | |
| 18:47:19 | sean-k-mooney | we really dont wnat the sdk to create there own scemas | |
| 18:47:26 | sean-k-mooney | that woudl be a very bad thing | |
| 18:48:13 | sean-k-mooney | stephenfin has been trying to push the converstaion back to adding openapi supprot to the phyton projects instead | |
| 18:48:33 | sean-k-mooney | i.e. adding openapi supprot to nova/neutron ectra | |
| 18:49:14 | chris218 | Ok last question and I'll stop nagging you, what's OpenStack stance on using from __future__ import annotations? | |
| 18:49:39 | sean-k-mooney | we have to main python compatibality with 3.8 currrently | |
| 18:49:46 | sean-k-mooney | that will raise to 3.9 soon | |
| 18:50:07 | sean-k-mooney | i would gnerally say from __future__ import <anything> shoudl not be used | |
| 18:50:24 | sean-k-mooney | unless thte feature has been fully merged in a released python | |
| 18:51:19 | sean-k-mooney | we defien our testing runtimes here https://github.com/openstack/governance/blob/master/reference/runtimes/2024.1.rst | |
| 18:51:27 | sean-k-mooney | one file per release | |
| 18:53:02 | sean-k-mooney | chris218: im not exactly show what from __future__ import annotations is provideing | |
| 18:53:33 | sean-k-mooney | we do use typing and mypy to a limited degree | |
| 18:53:38 | chris218 | It turns all type hints into strings so you don't need to import their dependencies | |
| 18:54:03 | sean-k-mooney | right but you dont need that anymore | |
| 18:54:18 | sean-k-mooney | we have a mix of string and non string import | |
| 18:54:24 | sean-k-mooney | *type hints | |
| 18:54:35 | chris218 | True I guess 🤔 | |
| 18:54:50 | sean-k-mooney | this was part of https://peps.python.org/pep-0563/ right | |
| 18:55:13 | sean-k-mooney | in general i prefer not to use the string version but somethiems i will to avoid the import | |
| 18:55:24 | sean-k-mooney | the string version is more error prone | |
| 18:55:56 | sean-k-mooney | https://codesearch.opendev.org/?q=from+__future__+import+annotations&i=nope&literal=nope&files=&excludeFiles=&repos= | |
| 18:56:24 | sean-k-mooney | it is used but i think it was mostly for the tanstion period when we had to supprot python 2.7 | |
| 18:57:08 | chris218 | Good to know thanks for the clarification. Also I didn't know codesearch existed that looks awesome | |
| 18:57:31 | sean-k-mooney | its an instance of hound | |
| 18:57:45 | sean-k-mooney | https://github.com/hound-search/hound | |
| 18:59:54 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 18:59:54 | opendevreview | melanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| 18:59:55 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 19:03:48 | opendevreview | VO LE HUY proposed openstack/nova stable/2023.2: Force death and notice that the system clock is unstable https://review.opendev.org/c/openstack/nova/+/908519 | |
| 19:06:26 | opendevreview | VO LE HUY proposed openstack/nova stable/2023.2: Force down and logging that the system clock is unstable so administrator need to time sync by ntpd or chronyc, ... https://review.opendev.org/c/openstack/nova/+/908519 | |
| 19:20:53 | opendevreview | VO LE HUY proposed openstack/nova stable/2023.2: Do not use the abs(elapsed) function when elapsed is a negative number. Wrong operating principle but nova service still shows up status. https://review.opendev.org/c/openstack/nova/+/908519 | |
| #openstack-nova - 2024-02-09 | |||
| 00:35:29 | opendevreview | VO LE HUY proposed openstack/nova master: Shouldn't use the abs(elapsed) function when elapsed is a negative number. Wrong operating principle but nova service still shows up status. https://review.opendev.org/c/openstack/nova/+/908536 | |
| 02:03:28 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 02:46:04 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 03:01:40 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure both swtpm and swtpm_setup exist for vTPM support https://review.opendev.org/c/openstack/nova/+/908541 | |
| 03:02:18 | opendevreview | VO LE HUY proposed openstack/nova master: pep8: E501 line too long (80 > 79 characters) https://review.opendev.org/c/openstack/nova/+/908542 | |
| 03:06:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure both swtpm and swtpm_setup exist for vTPM support https://review.opendev.org/c/openstack/nova/+/908541 | |
| 03:18:39 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure swtpm_ioctl is available for vTPM support https://review.opendev.org/c/openstack/nova/+/908546 | |
| 03:19:15 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure swtpm_ioctl is available for vTPM support https://review.opendev.org/c/openstack/nova/+/908546 | |
| 03:44:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 04:05:56 | opendevreview | VO LE HUY proposed openstack/nova master: Add unit tests https://review.opendev.org/c/openstack/nova/+/908548 | |
| 04:18:16 | opendevreview | VO LE HUY proposed openstack/nova master: It may hide time sync problems causing backword clocks. https://review.opendev.org/c/openstack/nova/+/908536 | |
| 04:18:17 | opendevreview | VO LE HUY proposed openstack/nova master: Add unit test and fix pep8: E501 line too long https://review.opendev.org/c/openstack/nova/+/908553 | |
| 04:18:47 | opendevreview | VO LE HUY proposed openstack/nova master: Add unit test and fix pep8: E501 line too long https://review.opendev.org/c/openstack/nova/+/908553 | |
| 04:27:13 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestap as error https://review.opendev.org/c/openstack/nova/+/908536 | |
| 04:34:53 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestap as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 04:38:05 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 04:38:05 | opendevreview | melanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| 04:38:06 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 06:07:28 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestap as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 07:13:57 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 08:14:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 08:21:22 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 08:23:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure both swtpm and swtpm_setup exist for vTPM support https://review.opendev.org/c/openstack/nova/+/908541 | |
| 08:23:41 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ensure swtpm_ioctl is available for vTPM support https://review.opendev.org/c/openstack/nova/+/908546 | |
| 08:23:48 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt https://review.opendev.org/c/openstack/nova/+/908508 | |
| 09:05:31 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestap as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 09:07:30 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestap as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 09:07:55 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestamp as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 09:10:59 | opendevreview | VO LE HUY proposed openstack/nova master: Treat future service timestamp as error https://review.opendev.org/c/openstack/nova/+/908554 | |
| 09:20:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove old warning log about UEFI support https://review.opendev.org/c/openstack/nova/+/908563 | |
| 09:21:00 | tkajinam | I wonder if UEFI support is still experimental really or we can just remove that warning ^^^ | |
| 09:38:34 | noonedeadpunk | hey folks, do you know smth about centos9stream quite recent failures? | |
| 09:39:11 | noonedeadpunk | in terms that we've started getting smth like that for nova-compute recently https://zuul.opendev.org/t/openstack/build/4cacd5600c0e43b0be4e443ad33a9169/log/logs/host/nova-compute.service.journal-22-34-38.log.txt#6333 | |
| 09:39:25 | noonedeadpunk | this does not affect rocky linux nor ubuntu or debian | |
| 09:59:42 | tkajinam | noonedeadpunk, maybe https://issues.redhat.com/browse/RHEL-20609 ? | |
| 09:59:47 | tkajinam | the page is still loading for me, though | |
| 10:07:36 | noonedeadpunk | tkajinam: ok, yes, that's likely it:) | |
| 10:08:20 | tkajinam | yeah I see repeated unicode error in nova-compute logs | |
| 10:08:38 | tkajinam | we excluded that known bad version in puppet CI to workaround the issue | |
| 10:24:07 | noonedeadpunk | We mostly switched jobs to rocky linux, as it tends not to go on this bleeding edge as centos does. | |
| 10:24:13 | noonedeadpunk | So rocky is unaffected... | |
| 10:24:49 | noonedeadpunk | but trying to keep an eye on centos as well, so that issue won't propogate to rocky | |
| 10:28:43 | opendevreview | Amit Uniyal proposed openstack/nova master: Allow swap resize from non-zero to zero https://review.opendev.org/c/openstack/nova/+/857339 | |
| 10:38:11 | tkajinam | noonedeadpunk, Is that in OSA right ? I wonder if you are using RDO with Rocky or a different install source ? | |
| 10:39:09 | gibi | tkajinam: noonedeadpunk: yes libvirt 9.10 has a regression | |
| 10:39:45 | gibi | we needed to pin to a lower version https://github.com/openstack-k8s-operators/edpm-ansible/pull/560 | |
| 10:47:45 | noonedeadpunk | doesn't libvirt 10.0.0 giving only new modular version? | |
| 10:48:04 | noonedeadpunk | ie https://fedoraproject.org/wiki/Changes/LibvirtModularDaemons | |
| 10:49:03 | tkajinam | noonedeadpunk, afaik it still supports modular one | |
| 10:49:21 | noonedeadpunk | or well, integrated one?:) | |
| 10:49:31 | noonedeadpunk | old one... | |
| 10:49:42 | tkajinam | single libvirtd instead of virt(qemu|node|...)d | |
| 10:49:46 | noonedeadpunk | yeah | |
| 10:49:51 | noonedeadpunk | ok, good then | |