Earlier  
Posted Nick Remark
#openstack-nova - 2020-03-31
14:18:18 artom Anyone seens https://zuul.opendev.org/t/openstack/build/e0e47deeb33b476db792fd8b83680fbb before?
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: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 Warning: Permanently added '198.72.124.121' (ECDSA) to the list of known hosts.
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 microversion 2.85, extra spec validation https://review.opendev.org/708436
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: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 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:26 dansmith ah, okay gotcha
16:25:38 dansmith I thought you were saying it was structurally quite different or something
16:25:38 gibi you added tag and d-o-t
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: Use unittest.mock instead of third party mock https://review.opendev.org/714676
17:07:22 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675
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?

Earlier   Later