Earlier  
Posted Nick Remark
#openstack-nova - 2021-02-09
12:15:21 sean-k-mooney kashyap: so did we :) but there was an opertunity for a joke to be made
12:16:23 lyarwood and as a new dad I need the practice
12:17:50 kashyap sean-k-mooney: Hehe, figured as much; don't want to rob you of a good tease
12:18:05 sean-k-mooney i dont know you were fairly on point. maybe it was too good and you need to recalibrate to dad joke levels
12:19:02 kashyap Heh
12:21:02 kashyap sean-k-mooney: Do you ever sleep, BTW? ... I almost see your scrollback at all hours. Unless my IRC client is tricking me
12:25:27 sean-k-mooney i didnt sleep much last night. went to sleep aroudn 2-3 work at about half 5 got up at 8 and started working around 8:30.
12:25:36 sean-k-mooney so ill be finishing early today
12:41:09 gibi kashyap: I left feedback in https://review.opendev.org/c/openstack/nova/+/774240/
12:45:37 kashyap sean-k-mooney: Ouch; get some good rest
12:45:43 kashyap gibi: Will look after lunch; thank you
13:16:22 kashyap gibi: In your review, which test job you quote that unit test failure from?
13:16:25 gibi bauzas: could you hit https://review.opendev.org/c/openstack/nova/+/756530 at some point? that is the last real change in the qos changes in Wallaby
13:16:34 gibi kashyap: tox -e py38
13:16:50 gibi kashyap: to get the debug log you need to define OS_DEBUG=1 in your env and make the test fail
13:20:09 kashyap gibi: Ah, thanks!
13:24:16 sean-k-mooney or just do "tox -e py3"
13:24:21 sean-k-mooney then you dont need to care
13:24:48 sean-k-mooney it will use your systems default python
13:25:24 sean-k-mooney kashyap: just be aware that OS_DEBUG at least for functional test will be very very verbose when it fails
13:25:46 kashyap sean-k-mooney: Noted :) Just unit tests
13:29:10 kashyap sean-k-mooney: Have you got a latest minimal DevStack (without Cinder)?
13:29:22 kashyap sean-k-mooney: I have this ... can this be cutdown any further? - https://paste.centos.org/view/c9c4ecdf
13:32:23 bauzas gibi: I was thinking about it, LOL.
13:32:30 bauzas it's my next todo
13:32:47 gibi bauzas: thanks
13:34:11 sean-k-mooney kashyap: its not a minimal config but this is the bases i work off https://paste.centos.org/view/c57cf6a4
13:35:30 sean-k-mooney kashyap: you should not need FORCE=yes or the neutron settings your adding
13:35:44 sean-k-mooney unless its conflicting with your home/work network
13:37:17 sean-k-mooney kashyap: you also done need the data dirs stuff but you are using a non default location so not minimal but i guess its what you need
13:37:23 bauzas gibi: no spec for the attach/detach case, right? https://review.opendev.org/c/openstack/nova/+/756530/21//COMMIT_MSG
13:37:43 bauzas I was thinking about the workflow, do we have other resources for attach/detach ?
13:38:01 gibi bauzas: there is spec for the qos series
13:38:05 kashyap sean-k-mooney: Reading ...
13:38:30 bauzas I know
13:38:31 bauzas https://review.opendev.org/c/openstack/nova-specs/+/755477
13:38:48 kashyap sean-k-mooney: The data dirs stuff -- that is intentional; as I want it to be in a non-default location
13:38:49 bauzas okay okay, will lookup
13:38:51 kashyap sean-k-mooney: Thanks!
13:38:55 gibi bauzas: then I don't get your question
13:39:11 bauzas gibi: I didn't remember whether we discussed about the workflow
13:39:14 bauzas but yeah we did
13:39:16 gibi bauzas: there is only neutron resource request and the PCI request it handled for interface attach
14:43:25 stephenfin lyarwood: You've done the next patch in the series but any chance you could hit this? https://review.opendev.org/c/openstack/nova/+/741663/
14:52:31 lyarwood ops my bad, sure
14:53:02 openstackgerrit Stephen Finucane proposed openstack/placement master: policy: Deprecate 'admin_api' rule https://review.opendev.org/c/openstack/placement/+/772334
14:53:03 openstackgerrit Stephen Finucane proposed openstack/placement master: policy: Add note about keystone's expansion of roles https://review.opendev.org/c/openstack/placement/+/772752
14:53:04 openstackgerrit Stephen Finucane proposed openstack/placement master: Move policy deprecation to base rules https://review.opendev.org/c/openstack/placement/+/772784
14:53:05 openstackgerrit Stephen Finucane proposed openstack/placement master: policy: Add releasenote for RBAC work https://review.opendev.org/c/openstack/placement/+/774654
14:53:35 stephenfin gmann: ^
14:53:44 bauzas gibi: had an open question with some nits too https://review.opendev.org/c/openstack/nova/+/756530
14:53:54 stephenfin release note fresh out of the oven
14:53:59 bauzas tl;dr: should we delete the QoS allocations on interface detach ?
14:54:13 gmann stephenfin: ack. mostly rebase right
14:54:22 stephenfin yeah, I just reapplied +W
14:54:31 stephenfin the releasenote is the only real change
14:56:06 gmann ok
15:05:29 gmann stephenfin: lgtm 1 comment on adding link to keystone doc https://review.opendev.org/c/openstack/placement/+/774654
15:07:13 lyarwood does anyone know if there's an easy way to list instances by cell via an existing cli?
15:07:41 lyarwood context being a nova-status check I've added lists cell uuids and not instance uuids which isn't that useful
15:08:10 lyarwood before I go and write something to list instances by cell uuid or instances that fail my check I wanted to ensure something didn't already exist
15:08:22 lyarwood I can't see anything obvious in osc
15:08:30 openstackgerrit Stephen Finucane proposed openstack/placement master: policy: Add releasenote for RBAC work https://review.opendev.org/c/openstack/placement/+/774654
15:08:31 stephenfin gmann: done ^
15:08:43 lyarwood or novaclient
15:09:51 stephenfin lyarwood: A quick search through api-ref suggest we don't expose anything about cells v2 to the REST API
15:10:11 lyarwood stephenfin: gah, well I guess I'm writing a nova-manage command then
15:10:24 lyarwood or rewriting this status check to list instance uuids
15:10:35 lyarwood but given the table output that's going to look awful
15:36:03 kashyap sean-k-mooney: That 'DEVSTACK_PARALLEL' thing in your config - is that merged, or were you testing something from Git?
15:36:19 sean-k-mooney oh no thats not merged yet
15:36:24 sean-k-mooney i was testing dans patch
15:37:00 sean-k-mooney kashyap: https://review.opendev.org/c/openstack/devstack/+/771505
15:37:07 sean-k-mooney it works pretty well
15:37:24 sean-k-mooney tested it on ubuntu and fedora
15:38:49 kashyap sean-k-mooney: Thanks; also, remind me again, 'g-reg' can be avoided right?
15:47:32 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Add support for microversion v2.88 https://review.opendev.org/c/openstack/python-novaclient/+/770573
15:53:36 sean-k-mooney kashyap: yes
15:53:42 sean-k-mooney do i still have it in min
15:53:52 sean-k-mooney you just need thet glance api now i think
16:03:36 kashyap sean-k-mooney: You don't, though. I just kept in as a old habit
16:03:38 kashyap Thx
16:03:48 kashyap lyarwood: Aside - have you tried DevStack on F33 yet?
16:06:15 openstackgerrit Stephen Finucane proposed openstack/os-vif master: Resolve dependency issues https://review.opendev.org/c/openstack/os-vif/+/774694
16:06:18 stephenfin sean-k-mooney: ^
16:06:27 stephenfin as promised
16:07:01 stephenfin it sounds like the general consensus has been to remove these lower-constraint jobs entirely though. They're gone in most of oslo land now as well as neutron
16:08:12 lyarwood kashyap: I have but couldn't get it to work
16:08:17 lyarwood kashyap: even when forcing py38
16:08:21 kashyap lyarwood: Okay ... thought so; F32?
16:08:32 lyarwood kashyap: iirc virtualenv tries to use py39 in places and that breaks
16:08:37 lyarwood kashyap: yeah f32 should work
16:09:05 kashyap Thx
16:10:17 kashyap sean-k-mooney: On your word, I'm trying the _PARALLEL thing ... hope it won't turn into a yak :D
16:10:36 kashyap I'll report back my result, if it works
16:10:55 sean-k-mooney stephenfin: lets try keeping it for this cycle
16:11:16 sean-k-mooney stephenfin: if it break agian before the end of the cycle we can drop it
16:12:39 sean-k-mooney stephenfin: our lower constratis job is a littel different since we actully run our unit tests and docs
16:14:13 stephenfin lyarwood: Is there any way to map a migration request (via '/servers/{server_id}/action (migrate)') to a migration record ('/servers/{server_id}/migrations')
16:14:31 stephenfin I can't see any. It's weird that the former doesn't return an ID in the success case

Earlier   Later