Thursday, 2024-02-08

gmanndansmith: sean-k-mooney: updated hyperV driver series, please check in your morning https://review.opendev.org/q/topic:%22bp/remove-hyperv-driver%22 04:28
opendevreviewmelanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093210:21
opendevreviewmelanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093310:21
opendevreviewmelanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/90551210:21
opendevreviewmelanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093910:21
opendevreviewmelanie witt proposed openstack/nova master: Support rescue with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87367510:21
opendevreviewmelanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase  https://review.opendev.org/c/openstack/nova/+/87093610:21
opendevreviewmelanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093710:21
opendevreviewmelanie witt proposed openstack/nova master: Add backing_encryption_secret_uuid to BlockDeviceMapping  https://review.opendev.org/c/openstack/nova/+/90796010:21
opendevreviewmelanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2  https://review.opendev.org/c/openstack/nova/+/90796110:21
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS  https://review.opendev.org/c/openstack/nova/+/88431310:21
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS  https://review.opendev.org/c/openstack/nova/+/88991210:21
opendevreviewSylvain Bauza proposed openstack/nova master: Reserve mdevs to return to the source  https://review.opendev.org/c/openstack/nova/+/90420914:52
opendevreviewSylvain Bauza proposed openstack/nova master: Modify the mdevs in the migrate XML  https://review.opendev.org/c/openstack/nova/+/90425814:52
opendevreviewTakashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt  https://review.opendev.org/c/openstack/nova/+/90850816:12
opendevreviewMerged openstack/nova master: Remove the Hyper-V driver  https://review.opendev.org/c/openstack/nova/+/89446616:14
opendevreviewTakashi Kajinami proposed openstack/nova master: libvirt: Detect vtpm support by libvirt  https://review.opendev.org/c/openstack/nova/+/90850816:16
chris218Hi I was wondering if it's possible to run nova standalone, for testing and development purposes or you need neutron, glance, keystone?18:12
sean-k-mooneytechiially you can run without neuton and keystone by using the noauth plugin and only booting instance without networking18:16
sean-k-mooney but realistically no18:17
sean-k-mooneyi would not recomemnd even trying to do that18:17
sean-k-mooneythe minium set of service you should use are keystone,neutron,glance,placement and nova18:17
chris218So devstack is the way?18:18
sean-k-mooneynova has extenstive unit and functional tests18:18
sean-k-mooneybut if you want more then that you shoudl use devstack or some other tool18:18
chris218I was recently playing with the idea of writing my own driver and was wondering what would be the best way to test it18:18
sean-k-mooneywe offically do not supprot out of tree drivers and its unlikely we will add a new intree dirver in the medium to long term18:19
sean-k-mooneywhat hypervior were you hoping to integrate with18:19
sean-k-mooneythere is not sable external api for writing a virt driver for nova18:20
chris218It's mostly me trying to practice working with python in a larger codebase, but either cloud hypervisor or qemu through qmp instead of libvirt18:20
sean-k-mooney:) i have breifly looked at cloud hypervisor18:21
sean-k-mooneyif you wanted it to produced functional vms then devstack is the best approch18:21
chris218Awesome thanks for your help18:21
sean-k-mooneyon comment i would make is if you wanted ot gain experiance with workign on a large project and its not for work18:22
sean-k-mooneythere is alwasy a large bug backlog if you felt so inclidned18:23
sean-k-mooneychris218: but if you wanted a refecne fo what it might look like  this is what an out of tree driver would look like https://github.com/cdent/etcd-compute18:23
sean-k-mooneyactuly thats not a good example18:24
sean-k-mooneythat and example of a driver that woudl remove the rpc bus and use etcd instead18:24
chris218Oh awesome thanks, I was wondering why Kolla has etcd option18:25
sean-k-mooneyits not related to this18:25
chris218Oh :(18:25
sean-k-mooneythat was just a tought experiment that never went anywhere18:25
sean-k-mooneyectd in kolla is for use as a cache backend or distributed lock manager18:26
sean-k-mooneycinder via tooz can use etcd and i belive that is what koolla supprot it for18:26
sean-k-mooneyoslo.cache i belive also supprots etcd18:26
sean-k-mooneychris218: if your familar with kolla18:27
sean-k-mooneykolla has a dev mode18:27
sean-k-mooneywhere in bindmoutns local git repos into the conatiner18:27
chris218Btw while we're talking I was wondering for a long time, why do openstack projects still use pbr and setup.cfg? Don't setuptools with pyproject.toml have everything that pbr wanted to add?18:27
chris218Ooh that sounds nice18:27
sean-k-mooneyshort version no18:28
sean-k-mooneythere are several feature gaps18:28
sean-k-mooneypytproject.toml also changes how you manage depencies and we dont have a good solution for upper-constraits.txt18:28
sean-k-mooneyhttps://github.com/openstack/requirements/blob/master/upper-constraints.txt18:28
sean-k-mooneyso we dont have a good solution for how to maintain co installablity between all the repos18:29
sean-k-mooneywith that said it does not mean we cant add pyproject.toml18:29
chris218Oh I didn't even know that existed 18:29
sean-k-mooneyi actully started doing that in some places18:29
sean-k-mooneybut we cant just drop pbr right now18:29
sean-k-mooneychris218: https://review.opendev.org/q/topic:%22pip-23.1-support%2218:30
sean-k-mooneychris218: one of the things we depend on pbr for18:30
chris218But I think it should be possible to drop it from runtime deps and use importlib to check version, no?18:32
sean-k-mooneyis we make sue of a custome setuptools entry point that generage wsgi scripts for the rest apis18:35
sean-k-mooneywehn we enabel pyproject.toml even with pbr that breaks if you do an editiable install18:35
sean-k-mooneyso to progress with removing pbr we need to complete https://review.opendev.org/q/topic:%22remove-wsgi_scripts%2218:35
sean-k-mooneybauzas: speaking of https://review.opendev.org/q/topic:%22remove-wsgi_scripts%22+repo:openstack/nova can you review that18:35
sean-k-mooneyit would be good to land the first 2 patchs18:35
sean-k-mooneychris218: am we dont really have a runtim dep on setuptools18:37
sean-k-mooneyat least not in most of the code18:37
sean-k-mooneybut we do use stevador18:37
sean-k-mooneyhttps://github.com/openstack/stevedore18:37
sean-k-mooneyor rather https://opendev.org/openstack/stevedore18:37
chris218Hmm so openstack has everything home grown 😅18:41
sean-k-mooneynot entirly openstack predates a lot of the modern python tooling18:42
sean-k-mooneypbr is just a wrapper around setup tools18:42
chris218Yea like having own api-ref system instead of OpenAPI since it's the "new" hot thing18:43
sean-k-mooneyits not about it being the new hot thing18:43
sean-k-mooneywe actully discused supproting openAPI a few years ago 18:43
sean-k-mooneybut we did nto have contibutors that coudl work on it18:44
artomNobody here is hot or new :D18:44
chris218Haha18:44
chris218I'm new hello 18:44
sean-k-mooneythe 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 schemas18:45
sean-k-mooneywe have json schemas for exmaple in our apis already but we never had the time to rewirte themn to and openapi compaitble format18:46
chris218Well you can also generate docs from it, I've seen one project of person from sdk team working on it privately 18:46
sean-k-mooneyright i knwo there has been some discussion about it in the context of the opsntack sdk18:47
sean-k-mooneywe really dont wnat the sdk to create there own scemas 18:47
sean-k-mooneythat woudl be a very bad thing18:47
sean-k-mooneystephenfin has been trying to push the converstaion back to adding openapi supprot to the phyton projects instead18:48
sean-k-mooneyi.e. adding openapi supprot to nova/neutron ectra18:48
chris218Ok last question and I'll stop nagging you, what's OpenStack stance on using from __future__ import annotations?18:49
sean-k-mooneywe have to main python compatibality with 3.8 currrently 18:49
sean-k-mooneythat will raise to 3.9 soon18:49
sean-k-mooneyi would gnerally say from __future__ import <anything> shoudl not be used18:50
sean-k-mooneyunless thte feature has been fully merged in a released python18:50
sean-k-mooneywe defien our testing runtimes here https://github.com/openstack/governance/blob/master/reference/runtimes/2024.1.rst18:51
sean-k-mooneyone file per release18:51
sean-k-mooneychris218: im not exactly show what from __future__ import annotations is provideing 18:53
sean-k-mooneywe do use typing and mypy to a limited degree18:53
chris218It turns all type hints into strings so you don't need to import their dependencies18:53
sean-k-mooneyright but you dont need that anymore18:54
sean-k-mooneywe have a mix of string and non string import18:54
sean-k-mooney*type hints18:54
chris218True I guess 🤔18:54
sean-k-mooneythis was part of https://peps.python.org/pep-0563/ right18:54
sean-k-mooneyin general i prefer not to use the string version but somethiems i will to avoid the import18:55
sean-k-mooneythe string version is more error prone18:55
sean-k-mooneyhttps://codesearch.opendev.org/?q=from+__future__+import+annotations&i=nope&literal=nope&files=&excludeFiles=&repos=18:55
sean-k-mooneyit is used but i think it was mostly for the tanstion period when we had to supprot python 2.718:56
chris218Good to know thanks for the clarification. Also I didn't know codesearch existed that looks awesome 18:57
sean-k-mooneyits an instance of hound18:57
sean-k-mooneyhttps://github.com/hound-search/hound18:57
opendevreviewmelanie witt proposed openstack/nova master: WIP Support encrypted backing files for qcow2  https://review.opendev.org/c/openstack/nova/+/90796118:59
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS  https://review.opendev.org/c/openstack/nova/+/88431318:59
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS  https://review.opendev.org/c/openstack/nova/+/88991218:59
opendevreviewVO 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/+/90851919:03
opendevreviewVO 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/+/90851919:06
opendevreviewVO 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/+/90851919:20

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!