| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-02-12 | |||
| 16:56:33 | melwitt | thanks gibi, will look | |
| 16:56:43 | sean-k-mooney | is that not failing upstream becasue we dont have a downstrema backport | |
| 16:56:48 | sean-k-mooney | ? | |
| 16:57:17 | gibi | this is a race so it probably fails upstream 2023.1 too we just did not notice it | |
| 16:57:53 | sean-k-mooney | ack | |
| 16:58:05 | sean-k-mooney | oh its the greanpool stuff | |
| 16:58:28 | gibi | jepp it is | |
| 16:58:31 | sean-k-mooney | ok ill review properly after the call | |
| 17:00:46 | gibi | thanks | |
| 17:08:25 | noonedeadpunk | hey folks. I've starting seing some weird behaviour in one of regions running some "newer" 2023.1 version (supposedly from SHA 4f4f27be27cb202be6a8fb949ad21021a5cd735d) | |
| 17:08:52 | noonedeadpunk | and weird part, is that nova-compute just exits with code 0 when it catches some rabbitmq issue | |
| 17:09:07 | noonedeadpunk | ie https://paste.openstack.org/show/bTFcwAN8MTJlPtvSwXwA/ | |
| 17:10:21 | noonedeadpunk | like... I saw before how services were failing on rpc issues, but not exit for rc 0 | |
| 17:13:04 | noonedeadpunk | have you heard anything like that before? As I've never seen anything like that even in other regions of mine running 2023.1 | |
| 17:32:51 | opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | |
| 17:44:10 | sean-k-mooney | melwitt: im documenting my testing here https://hedgedoc.k8s.seanmooney.info/jPyZ_H2oS7mIkNmZtuK1ag?view= by the way. hi have not done much since i updated eariler today but ill make progress on teh diffent instance ations this evening | |
| 17:50:10 | melwitt | sean-k-mooney: sweet, that looks awesome | |
| 17:50:48 | sean-k-mooney | i normally use etherpad for this im playing iwht hedegeoc which is an opensouce hackmd fork | |
| 17:50:54 | sean-k-mooney | that leave you use markdown | |
| 17:50:57 | noonedeadpunk | oh, that is actually sweet ^ | |
| 17:51:19 | sean-k-mooney | if you press ctrl alt b or just clike on the slit view icon | |
| 17:51:36 | sean-k-mooney | you will be able to collabertivly edit it and see a realtime render on the right | |
| 17:51:47 | sean-k-mooney | so https://hedgedoc.k8s.seanmooney.info/jPyZ_H2oS7mIkNmZtuK1ag?both= instead of the view url | |
| 17:53:01 | melwitt | really cool | |
| 17:59:12 | sean-k-mooney | melwitt: i havent got there yet but what is the expecte behaivor for resize from encypeted to non encypted and viceversa | |
| 17:59:20 | sean-k-mooney | should that be blocked at the api? | |
| 17:59:43 | sean-k-mooney | or are we expect the disks to be decrypted or encypted | |
| 18:00:33 | melwitt | sean-k-mooney: it should be blocked at the api | |
| 18:00:40 | melwitt | sean-k-mooney: I did implement switching from encrypted to non encrypted for rebuild but that should be it | |
| 18:00:59 | sean-k-mooney | cool then ill check that | |
| 18:01:38 | sean-k-mooney | for rebuild i assuem that is only when the encyption is requested by the image correct | |
| 18:01:52 | melwitt | sean-k-mooney: right | |
| 18:02:00 | sean-k-mooney | cool ill note that too and test it | |
| 18:04:31 | sean-k-mooney | ah cool you noted the behvior in the commit message too | |
| 18:04:33 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/870939/30 | |
| 20:40:21 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 20:40:25 | opendevreview | ribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process https://review.opendev.org/c/openstack/nova/+/880075 | |
| 20:40:25 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 20:40:26 | opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | |
| 20:40:27 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 20:40:28 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 20:40:29 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 20:40:30 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 20:40:30 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 20:40:31 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 20:40:31 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 20:40:33 | opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | |
| 20:40:33 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 20:40:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | |
| 20:40:35 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | |
| 20:40:35 | opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | |
| 20:40:37 | opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | |
| 20:40:37 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 20:40:39 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 20:40:39 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | |
| 20:40:41 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 20:40:41 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 20:40:43 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 20:40:43 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 20:40:45 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 20:40:45 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 20:40:47 | opendevreview | ribaudr proposed openstack/nova master: Follow up patch to correct a minor documentation issue https://review.opendev.org/c/openstack/nova/+/908863 | |
| 20:40:47 | opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | |
| 20:40:49 | opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | |
| 20:56:31 | opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | |
| 21:03:15 | opendevreview | Merged openstack/nova stable/2023.1: Fix URLs in status check results https://review.opendev.org/c/openstack/nova/+/908693 | |
| 21:15:58 | sean-k-mooney | melwitt: im going to call it a day there but i have basically tested all the operators excpet evacuate, and requesting encyption via the image | |
| 21:16:12 | sean-k-mooney | i have updated https://hedgedoc.k8s.seanmooney.info/jPyZ_H2oS7mIkNmZtuK1ag?view= | |
| 21:16:41 | sean-k-mooney | i realise i should have been chekcign the libvirt/barican serets along the way but it seams to be working properly | |
| 21:16:52 | sean-k-mooney | im not seeing and libvirt secrets being leaked | |
| 21:17:06 | sean-k-mooney | and there were no errors of note in the compute agent logs | |
| 21:17:28 | sean-k-mooney | so at least with qcow guest images and raw backing files this looks like it working as inteded | |
| 21:19:00 | melwitt | sean-k-mooney: ok, nice, thank you! I'll check out the hedgedoc | |
| 21:20:05 | sean-k-mooney | i didn find any issue when test so if the code looks fine i think we can merge it up to at least the qcow point. ill do a end to end review tomorrow | |
| 21:20:25 | sean-k-mooney | i likely wont test this again with other backedn and just rely on your DNM tempest jobs | |
| 21:20:34 | opendevreview | Merged openstack/nova stable/wallaby: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/844753 | |
| 21:20:48 | sean-k-mooney | but if there is anything specifically you want me to test i can | |
| 21:22:10 | melwitt | sean-k-mooney: ok cool, thanks | |
| #openstack-nova - 2024-02-13 | |||
| 03:29:26 | tonyb | There's a glance_store constraints bump on stable/2303.1 that's failing | |
| 03:29:35 | tonyb | can someone take a look at: https://zuul.opendev.org/t/openstack/build/e485baf66457414fa8b79420e50441ad/log/job-output.txt#30523 | |
| 03:30:01 | tonyb | and give me a pointer on what's going on? There are lots of libvirt and db errors | |
| 05:44:54 | melwitt | sean-k-mooney: fyi I'm working on func tests for rescue atm ... and also handling to not go into ERROR state in the case of an encryption secret not being found in barbican | |
| 05:46:03 | sean-k-mooney[m] | ack, i work up early and since im awake i might as well review code :) | |
| 05:46:20 | sean-k-mooney[m] | its totally not because that will make me fall a asleep :P | |
| 05:46:49 | sean-k-mooney[m] | i havent got that far in the seriese but ill keep that in mind | |
| 05:47:46 | melwitt | sean-k-mooney[m]: haha :) ok, just wanted to mention that in case you did get that far and being like where's the func tests | |
| 07:07:25 | tkajinam | I'm just curious but wonder why rng device specs can be configured only by flavor extra specs and can't by image properties. | |
| 07:09:44 | Uggla | bauzas, gibi I have pushed the virtio serie yesterday evening. I still need to fix some comments and do extensive tests with devstack but it will be glad to have a review on the new asynchronous stuff. | |
| 07:36:49 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Stateless firmware support https://review.opendev.org/c/openstack/nova-specs/+/908297 | |
| 07:40:39 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 07:48:02 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: IUpdate spec reference links https://review.opendev.org/c/openstack/os-traits/+/908886 | |
| 07:48:56 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Update spec reference links https://review.opendev.org/c/openstack/os-traits/+/908886 | |
| 07:50:13 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 07:54:14 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908887 | |
| 07:55:41 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 08:13:03 | opendevreview | Takashi Kajinami proposed openstack/os-traits master: Add a new trait for stateless firmware support https://review.opendev.org/c/openstack/os-traits/+/908885 | |
| 08:14:22 | opendevreview | Takashi Kajinami proposed openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | |
| 08:16:14 | opendevreview | Takashi Kajinami proposed openstack/nova master: Report availability of stateless firmware support https://review.opendev.org/c/openstack/nova/+/908888 | |
| 08:57:48 | sean-k-mooney[m] | melwitt: i made it though the whole seriese. overall im +2 on most of it +1 on most of the rest and -1 on one patch | |
| 08:58:21 | sean-k-mooney[m] | in general i think we could proably merge the seriese as is safely but there is i think one bug in the resize api validation | |