| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-03-31 | |||
| 14:18:27 | artom | Warning: Permanently added '198.72.124.121' (ECDSA) to the list of known hosts. | |
| 14:18:27 | artom | rsync: link_stat "/var/lib/zuul/builds/e0e47deeb33b476db792fd8b83680fbb/work/ca-bundle.pem" failed: No such file or directory (2) | |
| 14:18:27 | artom | rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1183) [sender=3.1.1] | |
| 14:18:46 | artom | Asking here because... not really sure where Zuul job questions belong | |
| 14:19:05 | artom | I guess I'm doing something wrong in my .zuul.yml | |
| 14:22:11 | sean-k-mooney | no i dont think that is related to your job | |
| 14:22:31 | sean-k-mooney | i think its an issue win one of the other playbooks | |
| 14:39:38 | artom | sean-k-mooney, inorite? | |
| 14:39:49 | artom | Except it's consistently reproducible | |
| 14:51:12 | spatel | sean-k-mooney: is it possible that SR-IOV support security group? (i did google and didn't find any vendor who provide that card/nic) | |
| 14:54:12 | openstackgerrit | Merged openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227 | |
| 14:54:21 | openstackgerrit | Corey Bryant proposed openstack/nova stable/queens: hardware: fix memory check usage for small/large pages https://review.opendev.org/716328 | |
| 14:54:46 | sean-k-mooney | the nics dont in generally but you can sometimes use a heriachical port binding driver to implement security groups at the top of rack switch | |
| 14:55:45 | sean-k-mooney | if your using hardware offloade ovs which use ovs as the contol plane and sriov as the dataplane then it can technically do security groups | |
| 14:55:55 | sean-k-mooney | but normally if you use sriov then no | |
| 14:58:11 | spatel | sean-k-mooney: thanks for the explanation :) | |
| 15:12:54 | openstackgerrit | Merged openstack/python-novaclient master: Update to hacking 3.0 https://review.opendev.org/716228 | |
| 15:14:42 | gibi | sean-k-mooney: do you happen to know if we support booting with UEFI + PXE over IPv6? | |
| 15:14:57 | sean-k-mooney | in ironic? | |
| 15:15:10 | gibi | in nova + libvirt + qemu kvm | |
| 15:15:14 | sean-k-mooney | gibi: i dont think we support pxe boot in nova | |
| 15:15:41 | sean-k-mooney | gibi: i know we can enable the boot menu but i dont think we support pxe booting officaly | |
| 15:16:20 | gibi | thanks | |
| 15:38:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Add support for new cyborg extra specs https://review.opendev.org/716222 | |
| 15:38:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Add microversion 2.85, extra spec validation https://review.opendev.org/708436 | |
| 15:38:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037 | |
| 16:10:58 | dansmith | gibi: around? | |
| 16:11:14 | gibi | dansmith: yes | |
| 16:11:43 | dansmith | gibi: "doh" and "duh" on the rpc cast, thanks for that | |
| 16:12:17 | dansmith | gibi: what do you want to do about it? we can move the non-swap parameter updates before the swap | |
| 16:12:52 | dansmith | which means we'll update those values and then if the swap fails, we will have made part of the changes but not all, but given how swap works, there isn't much choice | |
| 16:14:27 | gibi | hm hm | |
| 16:15:00 | gibi | if changing the order does not mean that we update the d-o-t on the old volume attachment the swap that with a new attachement with default d-o-t then I'm OK with it | |
| 16:15:04 | dansmith | or go back to what I had, which was swap or d-o-t, but not both, although I definitely get the argument that you'd expect to be able to update all at once | |
| 16:15:31 | dansmith | gibi: I think because it looks up the BDM and updates it with the new volume that it should be fine | |
| 16:16:40 | gibi | OK, then I suggest to do the reordeing. If swap fails then we need to accept a partial PUT | |
| 16:16:47 | gibi | as swap is async | |
| 16:16:51 | dansmith | ack | |
| 16:16:59 | dansmith | gibi: by the way, on your comment about the notification test, | |
| 16:17:19 | dansmith | when enabling debug, I did not see the compute manager code logging the messages that should have come out | |
| 16:17:44 | dansmith | so I'm not sure the compute manager code is really running, even though that *seems* to be where the finish notification comes from | |
| 16:17:50 | gibi | with 2.85 you don't as the whole test failed before the compute had a chance to progress with the message | |
| 16:18:05 | dansmith | but I ran out of time trying to trace that down and the logs weren't coming.. | |
| 16:18:14 | dansmith | ahh, that makes sense | |
| 16:18:35 | gibi | yeah, it was not a trivial thing. I spent at least an hour figuring it out why it behaves differently | |
| 16:18:56 | dansmith | the notification stuff is too confusing in general so I didn't trust myself | |
| 16:19:13 | dansmith | and with no logs from compute manager... I tried putting castascall on there, butI got many more errors | |
| 16:19:50 | dansmith | gibi: can you check my reply here? https://review.opendev.org/#/c/693828/22/nova/api/openstack/compute/schemas/volumes.py | |
| 16:19:55 | gibi | looking | |
| 16:21:14 | gibi | dansmith: you copy the POST req schema, but that is different from the schema of the GET response | |
| 16:21:47 | gibi | if we want that the client GET the current attachment, change a field in it, then PUT it back, then PUT needs to accept the format of the GET response | |
| 16:22:02 | gibi | or did I missunderstood the intention here? | |
| 16:22:21 | dansmith | let me go look for an example of a get | |
| 16:23:09 | dansmith | are you saying because volumeAttachments[] vs volumeAttachment ? | |
| 16:23:59 | dansmith | 2.79 GET of a single attachment looks the same to me, no? | |
| 16:24:45 | gibi | it has id in it | |
| 16:24:49 | dansmith | https://docs.openstack.org/api-ref/compute/?expanded=update-server-detail,list-volume-attachments-for-an-instance-detail,show-a-detail-of-a-volume-attachment-detail#show-a-detail-of-a-volume-attachment | |
| 16:25:01 | tobias-urdin | doing upgrade testing after upgrade nova-compute throws a RuntimeError: maximum recursion depth exceeded while calling a Python object http://paste.openstack.org/show/791412/ | |
| 16:25:18 | gibi | crate only allows volumeId and device | |
| 16:25:21 | gibi | create | |
| 16:25:26 | dansmith | ah, okay gotcha | |
| 16:25:26 | tobias-urdin | based on traceback i've tried upgrading oslo.db 5.0.2 and oslo.concurrency 3.30.0 (and all other oslo for that matter) but it won't start | |
| 16:25:38 | gibi | you added tag and d-o-t | |
| 16:25:38 | dansmith | I thought you were saying it was structurally quite different or something | |
| 16:25:50 | gibi | I meant we have different amout of fields | |
| 16:26:18 | gibi | structure seems to be the same | |
| 16:26:32 | dansmith | okay I guess I missed serverId although I thought I had it in there | |
| 16:26:36 | gibi | just GET returns extra fields which PUT does not allow | |
| 16:26:47 | dansmith | yep, I see now | |
| 16:27:03 | gibi | serverId and id (which is the attachment id based on the doc) | |
| 16:31:51 | gmann | dansmith: gibi : should we allow tag update also in this - https://review.opendev.org/#/c/693828/21/nova/api/openstack/compute/volumes.py@475 | |
| 16:32:46 | dansmith | gmann: sorry I should have said something, but I looked into it and it requires an rpc call to update that, so I think we should punt | |
| 16:33:51 | gmann | dansmith: ohk. it's not our DB only things. got it. | |
| 16:33:56 | dansmith | yeah | |
| 16:35:50 | tobias-urdin | anybody has an idea what could be causing the traceback? :) http://paste.openstack.org/show/791412/ | |
| 17:07:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675 | |
| 17:07:22 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676 | |
| 17:26:39 | openstackgerrit | Merged openstack/os-vif master: Update hacking for Python3 https://review.opendev.org/715651 | |
| 17:44:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in lock server policy https://review.opendev.org/716114 | |
| 17:47:25 | sean-k-mooney | stephenfin: didnt efried already sumbit a patch for https://review.opendev.org/#/c/714676/ | |
| 17:47:38 | sean-k-mooney | did he abandon it? | |
| 17:49:11 | sean-k-mooney | stephenfin: https://review.opendev.org/#/c/708262/ | |
| 17:54:01 | sean-k-mooney | stephenfin: it looks like you have fixed import ordering and some other minor thinks so i guess your patch is more complete but you should probably cherry-pick the hacking change ontop of your patch | |
| 18:00:14 | openstackgerrit | Dan Smith proposed openstack/nova master: Allow PUT volume attachments API to modify delete_on_termination https://review.opendev.org/693828 | |
| 18:01:01 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in lock server policies https://review.opendev.org/716122 | |
| 18:33:18 | efried | stephenfin, sean-k-mooney: feel free to take over or abandon mine as needed. | |
| 18:33:44 | efried | but yeah, the hacking change should be included in the series. | |
| 19:01:11 | openstackgerrit | Corey Bryant proposed openstack/nova stable/queens: hardware: fix memory check usage for small/large pages https://review.opendev.org/716328 | |
| 19:06:22 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | |
| 19:10:00 | stephenfin | sean-k-mooney: Ah, I'd forgotten about that. Will combine | |
| 19:33:47 | openstackgerrit | Merged openstack/nova master: [Community goal] Update contributor documentation https://review.opendev.org/712420 | |
| 19:33:57 | openstackgerrit | Merged openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496 | |
| 19:34:04 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477 | |
| 19:34:11 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529 | |
| 19:34:18 | openstackgerrit | Merged openstack/nova master: Add new default roles in os-ips policies https://review.opendev.org/715545 | |
| 20:03:23 | openstackgerrit | Merged openstack/nova master: Pass the actual target in os-agents policy https://review.opendev.org/701649 | |
| 20:46:39 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in unlock override policy https://review.opendev.org/716428 | |
| 20:55:01 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761 | |
| 20:55:06 | dansmith | gmann: you around? | |
| 20:55:21 | dansmith | owalsh has a question for you :) | |