| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-09-22 | |||
| 12:43:08 | sean-k-mooney | be a charcutier not a butcher of our lovely codebase :) | |
| 12:55:40 | sean-k-mooney | stephenfin: since your about can you look at https://review.opendev.org/c/openstack/nova/+/895655 | |
| 12:56:17 | sean-k-mooney | once thats merged i can propsoe the revert adn ping it to rodolfo and co so we can renable it once neutron says the test is fixed for ovn | |
| 12:56:32 | stephenfin | ack, done | |
| 12:56:56 | sean-k-mooney | that test is one of the thing that cased the hackign patch to fail | |
| 12:57:12 | sean-k-mooney | the other failure was volume detach related | |
| 13:05:01 | bauzas | fwiw, starting to look at the rest of sql14 series | |
| 13:09:04 | opendevreview | Felix Huettner proposed openstack/nova master: ignore errors when deleting volume attachments https://review.opendev.org/c/openstack/nova/+/896235 | |
| 13:18:05 | bauzas | okay, this is Friday, I'm not a better and decent position to start a debate https://luminousmen.com/post/my-unpopular-opinion-about-black-code-formatter | |
| 13:18:30 | bauzas | now* | |
| 13:18:57 | sean-k-mooney | for what its worht a number of opentack proejct have arelady adopted black | |
| 13:19:09 | bauzas | that doesn't make the point valid :) | |
| 13:19:12 | sean-k-mooney | we already have autopep8 which was a compomise | |
| 13:19:58 | bauzas | yup, and which doesn't force your code to change, right? | |
| 13:20:14 | bauzas | just because a ending bracket has to be on a separate line | |
| 13:20:17 | sean-k-mooney | it only fixes code if it not pep8 complient | |
| 13:20:33 | sean-k-mooney | and it undersands the things we disable in tox.ini | |
| 13:20:57 | bauzas | black is *very* noisy in terms of code change since it can't adapt to the existing code style | |
| 13:21:11 | sean-k-mooney | i.e. it reads https://github.com/openstack/nova/blob/master/tox.ini#L307 | |
| 13:21:21 | bauzas | and it forces us to do the way blacks thinks being the best | |
| 13:22:16 | sean-k-mooney | i think that is better then use disagreeing on what is the best style | |
| 13:22:30 | bauzas | btw. this twitter post which is replied by Guido is absolutly gold | |
| 13:23:16 | sean-k-mooney | bauzas: by the way i actully dont really care what the tool is provide we have one that does it | |
| 13:23:23 | bauzas | it shows the mindset of many developers : in order to follow *best* practices of coding, it says "install X and Y" | |
| 13:23:37 | bauzas | my head explodes when I read this | |
| 13:23:54 | bauzas | I thought for 2 decades that coding best practices were about code patterns and reusability | |
| 13:24:21 | sean-k-mooney | partly but we are not talking about best praticies | |
| 13:24:36 | sean-k-mooney | if you think we are then again your are missing the point | |
| 13:24:40 | bauzas | sean-k-mooney: read my link above, you'll see this tweet :) | |
| 13:24:47 | sean-k-mooney | i have seen those before | |
| 13:28:24 | bauzas | ouch https://black.readthedocs.io/en/stable/guides/introducing_black_to_your_project.html#avoiding-ruining-git-blame | |
| 13:47:39 | bauzas | sean-k-mooney: just in order to move on black and because I want to exactly understand the implications, do you know how the other projects manage the git blame mangling thing ? | |
| 13:48:24 | sean-k-mooney | i belvie you can annotate the commit to be ignored for blame | |
| 13:48:47 | sean-k-mooney | and it will then not be seein in github or locally by default | |
| 13:49:17 | sean-k-mooney | bauzas: which stephen did there https://review.opendev.org/c/openstack/nova/+/896231 | |
| 13:49:26 | bauzas | yeah, that's what black is saying on their docs | |
| 13:49:55 | bauzas | okay, then that's planned, gtk | |
| 13:56:32 | sean-k-mooney | yep its definetly a nice ux improvment ot have that | |
| 13:57:24 | sean-k-mooney | althoguh i woudl also take blame working in github at all on the libvirt driver... | |
| 17:34:41 | opendevreview | Merged openstack/nova master: Fix pep8 errors with new hacking https://review.opendev.org/c/openstack/nova/+/874517 | |
| #openstack-nova - 2023-09-23 | |||
| 08:21:53 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2024.1 Caracal https://review.opendev.org/c/openstack/nova/+/896305 | |
| 19:08:41 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: [stable-only] Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896019 | |
| 19:09:53 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896020 | |
| 19:59:25 | opendevreview | Merged openstack/nova stable/2023.2: [stable-only] Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896019 | |
| 20:02:03 | frickler | why is nova-tox-validate-backport non-voting in check, but voting in gate? seems inconsistent | |
| 20:06:13 | opendevreview | Merged openstack/nova stable/2023.2: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896020 | |
| #openstack-nova - 2023-09-24 | |||
| 14:56:33 | mo | Hello :) Nova newbie here, sorry to interrupt but I'm a bit lost trying to understand Nova's code, can you refer me to a good place to start? I've already read the developer docs (https://docs.openstack.org/nova/ocata). thanks ! | |
| #openstack-nova - 2023-09-25 | |||
| 04:35:11 | opendevreview | Yonghyun Kim proposed openstack/nova master: Fixed 500 response error for concurrent API requests. https://review.opendev.org/c/openstack/nova/+/896320 | |
| 06:07:36 | elodilles | frickler: mainly to avoid unnecessary rechecks | |
| 09:09:59 | lajoskatona | bauzas: Hi, quick question regarding libvirt/os support matrix: https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix | |
| 09:10:47 | lajoskatona | bauzas: is there any hanging or abandoned patch to update this doc for 2023.1 and .2? I can't find it... :-( | |
| 09:17:42 | frickler | lajoskatona: that's a wiki page, were you intending to refer to something in nova docs? wiki pages don't get updated by patches and this one looks pretty abandoned | |
| 09:19:10 | frickler | elodilles: I'm not convinced that trading reducing rechecks for increasing gate failures is the right direction, ymmv | |
| 09:20:29 | lajoskatona | frickler: sorry, there's a doc page also: https://docs.openstack.org/nova/latest/reference/libvirt-distro-support-matrix.html | |
| 09:21:01 | lajoskatona | frickler: it's one release fresher with xena included | |
| 09:21:41 | frickler | lajoskatona: that still looks pretty outdated, that's sad indeed | |
| 09:22:59 | bauzas | lajoskatona: yes, please use the above HTML | |
| 09:24:01 | frickler | bauzas: the question is why this isn't updated for Zed and beyond | |
| 09:24:21 | bauzas | I can ask kashyap to add the new versions for Zed and more | |
| 09:24:27 | lajoskatona | frickler, bauzas: exactly | |
| 09:25:03 | lajoskatona | bauzas: thanks that would be nice, it is quite hard to check the reno pages for related entries | |
| 09:25:51 | elodilles | frickler: well, in case of proper (not stable-only) patches we only +W patches that have merged versions in N+1 branch, so then the gate won't fail, but we spare some unnecessary rechecks. so that is quite rare where we accidentally accept patches that is failing on gate with this validation | |
| 09:29:39 | opendevreview | Felix Huettner proposed openstack/nova master: ignore errors when deleting volume attachments https://review.opendev.org/c/openstack/nova/+/896235 | |
| 09:33:10 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP:Adds device tagging functional tests https://review.opendev.org/c/openstack/nova/+/895162 | |
| 10:10:15 | opendevreview | Sylvain Bauza proposed openstack/nova master: docs: update libvirt support matrix by adding min versions https://review.opendev.org/c/openstack/nova/+/896372 | |
| 10:10:28 | bauzas | frickler: lajoskatona: ^ | |
| 10:12:01 | opendevreview | Stephen Finucane proposed openstack/placement master: Drop py38 based jobs and add py310 instead https://review.opendev.org/c/openstack/placement/+/881366 | |
| 10:50:04 | lajoskatona | bauzas: thanks, I check it | |
| 11:43:20 | opendevreview | Merged openstack/placement master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/placement/+/895493 | |
| 12:43:17 | opendevreview | Yonghyun Kim proposed openstack/nova master: Fixed 500 response error for concurrent API requests. https://review.opendev.org/c/openstack/nova/+/896320 | |
| 13:23:58 | bauzas | doh forgot to review https://review.opendev.org/c/openstack/nova/+/896021 | |
| 13:24:01 | bauzas | thanks gibi for it | |
| 13:24:25 | bauzas | -ETOOMANYREVIEWEMAILSINMYINBOX | |
| 13:25:53 | bauzas | gibi: sean-k-mooney: dansmith: that's way too much overdue https://review.opendev.org/c/openstack/nova/+/896372 | |
| 13:27:22 | kashyap | bauzas: Thanks for updating this! - https://review.opendev.org/c/openstack/nova/+/896372 | |
| 13:27:26 | bauzas | np | |
| 13:28:21 | kashyap | bauzas: So you want me to add a mapping of OSes and their corresponding libvirt/QEMU/libguestfs versions? | |
| 13:29:07 | bauzas | kashyap: the fact is that https://docs.openstack.org/nova/latest/reference/libvirt-distro-support-matrix.html#os-distribution-versions is a bit outdated :) | |
| 13:31:39 | opendevreview | Merged openstack/nova master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/nova/+/896021 | |
| 13:32:00 | kashyap | bauzas: Agree; it is a bit stale; I'll take care of it | |
| 14:21:19 | opendevreview | Merged openstack/nova master: docs: update libvirt support matrix by adding min versions https://review.opendev.org/c/openstack/nova/+/896372 | |
| 14:34:51 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add libvirt-dev-alias spec https://review.opendev.org/c/openstack/nova-specs/+/892806 | |
| 14:36:06 | dansmith | bauzas: I'm not sure if this ^ needs any discussion at the PTG or not | |
| 14:36:12 | dansmith | hopefully it's pretty straightforward | |
| 14:54:36 | bauzas | dansmith: okidoki, will look at it next week | |
| #openstack-nova - 2023-09-26 | |||
| 00:30:09 | opendevreview | Yonghyun Kim proposed openstack/nova master: Fixed 500 response error for concurrent API requests. https://review.opendev.org/c/openstack/nova/+/896320 | |
| 05:56:11 | opendevreview | Ryota, Ochiai proposed openstack/nova master: doc: Delete deprecated settings of the installation guide https://review.opendev.org/c/openstack/nova/+/896074 | |
| 07:11:47 | hamidlotfi_ | good morning | |
| 07:12:00 | hamidlotfi_ | I ran Rally tempest for the compute part and some tests failed with the message `API os-agents is not supported.` | |
| 07:12:00 | hamidlotfi_ | Does anyone know the solution to this problem? | |
| 07:29:53 | bauzas | hamidlotfi_: the os-agents API is no longer supported indeed https://docs.openstack.org/api-ref/compute/#guest-agents-os-agents | |
| 07:30:26 | bauzas | hamidlotfi_: so you should change Rally to not look at this API | |
| 07:53:46 | hamidlotfi_ | bauzas: Thanks. | |
| 08:55:42 | admin1 | hi all .. how do I stop a Shelving Image Pending Upload operation ? | |
| 10:16:57 | opendevreview | yatin proposed openstack/os-vif master: Revert "set default qos policy" https://review.opendev.org/c/openstack/os-vif/+/896440 | |
| 10:22:59 | opendevreview | yatin proposed openstack/os-vif master: Revert "set default qos policy" https://review.opendev.org/c/openstack/os-vif/+/896440 | |
| 12:07:41 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: fix AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | |
| 15:00:20 | bauzas | reminder : nova meeting in 1 hour | |
| 15:00:23 | bauzas | (here) | |
| 15:39:02 | bauzas | dansmith: interesting bugfix you may be aware https://review.opendev.org/c/openstack/nova/+/894288 | |