| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-08-03 | |||
| 20:31:37 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/ussuri: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/744550 | |
| 20:47:23 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/744552 | |
| 20:56:57 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: Should not skip volume_size check for bdm.image_id == image_ref case https://review.opendev.org/744553 | |
| 20:56:57 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/744554 | |
| 23:55:04 | openstackgerrit | Merged openstack/nova stable/queens: zuul: remove legacy-tempest-dsvm-neutron-dvr-multinode-full https://review.opendev.org/742668 | |
| 23:55:11 | openstackgerrit | Merged openstack/nova stable/pike: Fix false ERROR message at compute restart https://review.opendev.org/699496 | |
| #openstack-nova - 2020-08-04 | |||
| 00:49:56 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 00:49:56 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 02:49:23 | tony_su | stephenfin: gibi: the last 3 patches for provider-config-file are ready for a new round of review with comments answered. Thanks. | |
| 02:50:50 | tony_su | I checked zuul failure and found no relation to us. | |
| 03:28:23 | openstackgerrit | renminmin proposed openstack/nova master: SRIOV: Fix instance with macvtap port live migration failed https://review.opendev.org/738432 | |
| 06:26:45 | devfaz_ | hi, just run into an issue with no longer existing server groups. I had to evacuate some instances, but they failed, because the server group no longer exists. Just googled a bit and found https://bugs.launchpad.net/nova/+bug/1298509. I understand, that it should be possible to delete a server group with instances, but there should be also a way to update / remove the entry from the instances - | |
| 06:26:45 | openstack | Launchpad bug 1298509 in OpenStack Compute (nova) ""nova server-group-delete" allows deleting server group with members" [Wishlist,Opinion] | |
| 06:26:50 | devfaz_ | else the makes the instances unbootable. | |
| 06:50:08 | gibi | devfaz: do you have a stack trace from the log with the failure? I've quickly tried with resize and that worked with deleted server group | |
| 06:51:20 | devfaz | gibi: one moment | |
| 06:55:04 | devfaz | gibi: no stack-trace, just Filtering removed all hosts for the request with instance ID 'adddf2c9-0252-4463-a97c-f1ec209d9f49'. Filter results: ['AvailabilityZoneFilter: (start: 2, end: 2)', 'ComputeFilter: (start: 2, end: 2)', 'ComputeCapabilitiesFilter: (start: 2, end: 2)', 'ImagePropertiesFilter: (start: 2, end: 2)', 'ServerGroupAntiAffinityFilter: (start: 2, end: 2)', | |
| 06:55:10 | devfaz | 'ServerGroupAffinityFilter: (start: 2, end: 0)'] | |
| 06:56:48 | gibi | devfaz: I think it worth a bug report. I think nova needs to ignore delete server groups during scheduling | |
| 06:56:52 | devfaz | and in the "server show", i get as fault-value. Instance group 4b9a2cc5-cc44-470e-9ad0-68dcc9f8c60e could not be found. | |
| 06:57:28 | devfaz | gibi: ok, i will create a new one | |
| 06:57:32 | devfaz | thx a lot! | |
| 07:05:14 | devfaz | https://bugs.launchpad.net/nova/+bug/1890244 | |
| 07:05:14 | openstack | Launchpad bug 1890244 in OpenStack Compute (nova) "nova scheduler should ignore removed groups" [Undecided,New] | |
| 07:05:18 | devfaz | gibi: fyi | |
| 07:10:49 | openstackgerrit | Bence Romsics proposed openstack/nova master: doc: Add IPv6 metadata address https://review.opendev.org/744492 | |
| 08:09:47 | openstackgerrit | Xinran WANG proposed openstack/os-resource-classes master: Add new resource class for QAT card. https://review.opendev.org/726314 | |
| 09:38:28 | lyarwood | gibi: https://review.opendev.org/#/c/744506/ - morning, when you have time this should fix the lower-constraints job on master at least for now | |
| 09:39:37 | lyarwood | https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-lower-constraints&project=openstack%2Fnova for context | |
| 10:03:14 | elod | lyarwood: it seems every job have some *constraint related issue now (but some are not visible as does not cause failure), see discussion from infra channel yesterday: http://eavesdrop.openstack.org/irclogs/%23openstack-infra/%23openstack-infra.2020-08-03.log.html#t2020-08-03T16:22:05 | |
| 10:03:54 | elod | lyarwood: the interesting thing is that I cannot reproduce that locally :/ | |
| 10:13:11 | lyarwood | elod: ack thanks | |
| 10:13:29 | lyarwood | elod: I was eventually able to reproduce the master issue yesterday on a 18.04 vm but yeah it wasn't easy at all | |
| 10:15:04 | elod | elod: oh, sounds good! \o/ | |
| 10:15:12 | elod | lyarwood: ^^^ :] | |
| 10:16:12 | elod | lyarwood: also there's a new release of pip, there's an interesting change: "Correctly find already-installed distributions with dot (.) in the name and uninstall them when needed. " ( https://pip.pypa.io/en/stable/news/#id1 ) | |
| 10:16:41 | elod | especially as the failure came when already installed packages were reinstalled (without constraints) | |
| 10:26:10 | lyarwood | elod: ah that is interesting | |
| 10:35:34 | elod | lyarwood: well, it seems something changed, but I still see the issue in zuul jobs :S | |
| 10:36:45 | elod | lyarwood: do you have somewhere the steps, how the failure can be reproduced locally? | |
| 10:40:20 | lyarwood | elod: I didn't write them down sorry, I just built an env using virt-builder and installed tox directly from pypy instead of packages | |
| 10:40:38 | lyarwood | elod: iirc I had to bump MarkupSafe to workaround another issue | |
| 10:49:07 | elod | lyarwood: ok, no problem. actually I did the same, installed latest pip and tox from pypi. worth to mention that once it worked for me as in zuul, but then I couldn't reproduce it again with the steps I thought it should be reproducable :S | |
| 11:03:43 | gibi | devfaz: thanks | |
| 11:05:55 | sean-k-mooney | gibi: stephenfin alex_xu could ye take a lok at https://review.opendev.org/#/c/739131/ im trying to move on to numa in placment work so if i can close that out it would make me happy | |
| 11:06:06 | gibi | lyarwood: thanks. I've approved https://review.opendev.org/#/c/744506/ | |
| 11:07:01 | sean-k-mooney | that said gibi i just booted my sriov servers so ill be deploying your sriov detach patches while i rebase some os-vif patches. ill let you know what i find in my testing | |
| 11:08:26 | gibi | sean-k-mooney: thanks a lot for those tests | |
| 11:08:34 | gibi | sean-k-mooney: I'm looking at https://review.opendev.org/#/c/739131/ now | |
| 11:32:32 | openstackgerrit | Harshavardhan Metla proposed openstack/nova stable/rocky: [stable-only] Moved the quoted section https://review.opendev.org/744681 | |
| 11:50:52 | gibi | sean-k-mooney: I have left comments in https://review.opendev.org/#/c/744506/ | |
| 11:53:33 | sean-k-mooney | gibi: did you? that is the lower constrits fix :) | |
| 11:54:02 | sean-k-mooney | you mean https://review.opendev.org/#/c/739131 | |
| 12:02:36 | gibi | I've mixed up the links | |
| 12:02:52 | gibi | I did left some review on lookup nic patch https://review.opendev.org/#/c/739131/ | |
| 12:03:11 | sean-k-mooney | yep reading them now | |
| 12:13:34 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Migrate to zuulv3 and break up the nova-live-migration job https://review.opendev.org/711604 | |
| 12:30:50 | sean-k-mooney | gibi: thanks i have responded to the comments ill quickly adress the comparison change since i just forgot to fully revert that and ill re layout the types traits | |
| 12:31:47 | sean-k-mooney | gibi: over all i did not see any functional changes requested so i think the patch is still fucntionally correct as is and the change are jsut style related for the most part right? | |
| 12:48:07 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Migrate to zuulv3 and break up the nova-live-migration job https://review.opendev.org/711604 | |
| 12:55:03 | openstackgerrit | Merged openstack/nova stable/stein: objects: Update keypairs when saving an instance https://review.opendev.org/743264 | |
| 12:56:28 | gibi | sean-k-mooney: I'm on a call I will get back to you in an hour | |
| 12:56:39 | sean-k-mooney | gibi: no rush | |
| 12:56:53 | sean-k-mooney | i should have your comments adress by then :) | |
| 12:58:03 | openstackgerrit | sean mooney proposed openstack/nova stable/train: Silence amqp heartbeat warning https://review.opendev.org/728057 | |
| 13:04:18 | lyarwood | does anyone know if there's a way of reproducing zuul jobs locally? | |
| 13:04:23 | lyarwood | ./me is pretty sure there isn't | |
| 13:08:08 | lyarwood | sean-k-mooney: ^ any ideas? | |
| 13:09:21 | artom | lyarwood, IIRC I asked in #infra a while ago - "no" was the answer | |
| 13:11:15 | lyarwood | artom: yup thanks, I'll keep spamming small changes then | |
| 13:11:38 | sean-k-mooney | lyarwood: not really you can technically just run the playbooks locally | |
| 13:11:59 | lyarwood | sean-k-mooney: yeah it's more the job definitions that I need to test at this point | |
| 13:12:03 | sean-k-mooney | lyarwood: but the issue is creating the inventory file and runing them is non trivial | |
| 13:12:15 | lyarwood | sean-k-mooney: just feels wrong spamming zuul | |
| 13:12:33 | artom | lyarwood, why small changes? | |
| 13:12:45 | sean-k-mooney | lyarwood: for what its worth its not that hard to deploy a zuul instance using docker-compose | |
| 13:12:57 | sean-k-mooney | but mimicing the upstream enve is still a little more inovled | |
| 13:13:35 | sean-k-mooney | lyarwood: i assume you are doint what i do and disablig all the other jobs when your deveolping one | |
| 13:13:44 | lyarwood | sean-k-mooney: yup | |
| 13:13:54 | sean-k-mooney | lyarwood: also if you want to make it go faster you can cheat and use the experimental pipeline | |
| 13:14:10 | sean-k-mooney | in which case you also need to disbale the experimental josb other then your own | |
| 13:14:55 | sean-k-mooney | lyarwood: zuul will also do basic syntax checking too so if youy have a typo in the yaml file it will somethime cacht that before runnign your job | |
| 13:17:17 | lyarwood | sean-k-mooney: yup cool | |
| 13:17:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: Migrate to zuulv3 and break up the nova-live-migration job https://review.opendev.org/711604 | |
| 13:17:48 | lyarwood | artom: well incremental changes then as I'm slowly working out how to covert the jobs | |
| 13:18:08 | lyarwood | something I'd normally do locally but have to do in the open thanks to zuul now :| | |
| 13:18:51 | artom | lyarwood, tbh, I don't think anyone will look, unless they understand it's a WIP and want to check in/help | |
| 13:18:55 | artom | Aka no shame | |
| 13:19:33 | sean-k-mooney | artom: i think lyarwood just did not want to waste gate resouces | |
| 13:20:02 | artom | sean-k-mooney, ah, yeah. Well, what are you gonna do | |
| 13:20:59 | sean-k-mooney | disable all jobs you are not updatinging and ask stephenfin to run unit test locally :P | |
| 13:21:36 | sean-k-mooney | you can replicate the gate as i said but its a lot of setup if you dont do this regularly | |
| 13:22:22 | sean-k-mooney | that is true | |
| 13:22:56 | openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Update references to image properties https://review.opendev.org/744198 | |
| 13:22:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop support for UML https://review.opendev.org/743230 | |
| 13:22:57 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Drop support for Xen https://review.opendev.org/743231 | |
| 13:22:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove 'hypervisor_version' from 'libvirt_info' https://review.opendev.org/744199 | |
| 13:23:53 | sean-k-mooney | stephenfin: https://www.sciencealert.com/images/How-Much-Are-We-Worth-V2-large.jpg | |
| 13:24:22 | sean-k-mooney | thats even before you take into account your value as a software engineer or a good person | |
| 13:24:31 | sean-k-mooney | machines are cheap in comparison :P | |