| *** ralonsoh_ is now known as ralonsoh | 08:08 | |
| opendevreview | Vlad Gusev proposed openstack/nova stable/2023.1: Updates glance fixture for create image https://review.opendev.org/c/openstack/nova/+/908687 | 10:30 |
|---|---|---|
| opendevreview | Vlad Gusev proposed openstack/nova stable/2023.1: Fixes: bfv vm reboot ends up in an error state. https://review.opendev.org/c/openstack/nova/+/908688 | 10:32 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/2023.1: Fixes: bfv vm reboot ends up in an error state. https://review.opendev.org/c/openstack/nova/+/908688 | 10:33 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/zed: Remove outdated comment about allocation ratios https://review.opendev.org/c/openstack/nova/+/908692 | 10:59 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/2023.1: Fix URLs in status check results https://review.opendev.org/c/openstack/nova/+/908693 | 11:02 |
| opendevreview | Vlad Gusev proposed openstack/nova stable/zed: Unbind port when offloading a shelved instance https://review.opendev.org/c/openstack/nova/+/908694 | 11:03 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing requested az in output https://review.opendev.org/c/openstack/nova/+/904568 | 12:01 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing requested az in output https://review.opendev.org/c/openstack/nova/+/904568 | 13:09 |
| *** jph3 is now known as jph | 15:06 | |
| opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | 15:13 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | 15:13 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Removed explicit call to delete attachment https://review.opendev.org/c/openstack/nova/+/891289 | 15:13 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Separate OSError with ValueError https://review.opendev.org/c/openstack/nova/+/908825 | 15:13 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/908806 | 16:37 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: tests: Use GreenThreadPoolExecutor.shutdown(wait=True) https://review.opendev.org/c/openstack/nova/+/908806 | 16:43 |
| opendevreview | Balazs Gibizer proposed openstack/nova stable/2023.1: testing: Reset affinity support global variables https://review.opendev.org/c/openstack/nova/+/908839 | 16:43 |
| gibi | melwitt: sean-k-mooney: ^^ there are the missing backports that causes downstream functional failures | 16:46 |
| melwitt | thanks gibi, will look | 16:56 |
| sean-k-mooney | is that not failing upstream becasue we dont have a downstrema backport | 16:56 |
| sean-k-mooney | ? | 16:56 |
| gibi | this is a race so it probably fails upstream 2023.1 too we just did not notice it | 16:57 |
| sean-k-mooney | ack | 16:57 |
| sean-k-mooney | oh its the greanpool stuff | 16:58 |
| gibi | jepp it is | 16:58 |
| sean-k-mooney | ok ill review properly after the call | 16:58 |
| gibi | thanks | 17:00 |
| 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 |
| noonedeadpunk | and weird part, is that nova-compute just exits with code 0 when it catches some rabbitmq issue | 17:08 |
| noonedeadpunk | ie https://paste.openstack.org/show/bTFcwAN8MTJlPtvSwXwA/ | 17:09 |
| noonedeadpunk | like... I saw before how services were failing on rpc issues, but not exit for rc 0 | 17:10 |
| 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:13 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | 17:32 |
| 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:44 |
| melwitt | sean-k-mooney: sweet, that looks awesome | 17:50 |
| sean-k-mooney | i normally use etherpad for this im playing iwht hedegeoc which is an opensouce hackmd fork | 17:50 |
| sean-k-mooney | that leave you use markdown | 17:50 |
| noonedeadpunk | oh, that is actually sweet ^ | 17:50 |
| sean-k-mooney | if you press ctrl alt b or just clike on the slit view icon | 17:51 |
| sean-k-mooney | you will be able to collabertivly edit it and see a realtime render on the right | 17:51 |
| sean-k-mooney | so https://hedgedoc.k8s.seanmooney.info/jPyZ_H2oS7mIkNmZtuK1ag?both= instead of the view url | 17:51 |
| melwitt | really cool | 17:53 |
| 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 |
| sean-k-mooney | should that be blocked at the api? | 17:59 |
| sean-k-mooney | or are we expect the disks to be decrypted or encypted | 17:59 |
| melwitt | sean-k-mooney: it should be blocked at the api | 18:00 |
| melwitt | sean-k-mooney: I did implement switching from encrypted to non encrypted for rebuild but that should be it | 18:00 |
| sean-k-mooney | cool then ill check that | 18:00 |
| sean-k-mooney | for rebuild i assuem that is only when the encyption is requested by the image correct | 18:01 |
| melwitt | sean-k-mooney: right | 18:01 |
| sean-k-mooney | cool ill note that too and test it | 18:02 |
| sean-k-mooney | ah cool you noted the behvior in the commit message too | 18:04 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/870939/30 | 18:04 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | 20:40 |
| 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 |
| 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 |
| opendevreview | ribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion https://review.opendev.org/c/openstack/nova/+/881472 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | 20:40 |
| 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 |
| 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 |
| 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 |
| 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 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | 20:40 |
| 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 |
| opendevreview | ribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol https://review.opendev.org/c/openstack/nova/+/883862 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (compute manager) https://review.opendev.org/c/openstack/nova/+/885751 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add share lock/unlock and restrict visibility https://review.opendev.org/c/openstack/nova/+/890340 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (only API exception) https://review.opendev.org/c/openstack/nova/+/885752 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API) https://review.opendev.org/c/openstack/nova/+/836830 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Check shares support (API) https://review.opendev.org/c/openstack/nova/+/850499 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/885753 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | 20:40 |
| opendevreview | ribaudr proposed openstack/nova master: Docs about Manila shares API usage https://review.opendev.org/c/openstack/nova/+/871642 | 20:40 |
| 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 |
| opendevreview | ribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call https://review.opendev.org/c/openstack/nova/+/908864 | 20:40 |
| opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd https://review.opendev.org/c/openstack/nova/+/862416 | 20:56 |
| opendevreview | Merged openstack/nova stable/2023.1: Fix URLs in status check results https://review.opendev.org/c/openstack/nova/+/908693 | 21:03 |
| 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:15 |
| sean-k-mooney | i have updated https://hedgedoc.k8s.seanmooney.info/jPyZ_H2oS7mIkNmZtuK1ag?view= | 21:16 |
| 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 |
| sean-k-mooney | im not seeing and libvirt secrets being leaked | 21:16 |
| sean-k-mooney | and there were no errors of note in the compute agent logs | 21:17 |
| sean-k-mooney | so at least with qcow guest images and raw backing files this looks like it working as inteded | 21:17 |
| melwitt | sean-k-mooney: ok, nice, thank you! I'll check out the hedgedoc | 21:19 |
| 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 |
| sean-k-mooney | i likely wont test this again with other backedn and just rely on your DNM tempest jobs | 21:20 |
| 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 |
| sean-k-mooney | but if there is anything specifically you want me to test i can | 21:20 |
| melwitt | sean-k-mooney: ok cool, thanks | 21:22 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!