| *** ociuhandu has joined #openstack-nova | 00:01 | |
| *** macz_ has quit IRC | 00:02 | |
| *** rcernin has quit IRC | 00:03 | |
| *** gyee has quit IRC | 00:03 | |
| *** dklyle has quit IRC | 00:03 | |
| *** artom has quit IRC | 00:03 | |
| *** tbachman has quit IRC | 00:03 | |
| *** alex_xu has quit IRC | 00:03 | |
| *** lemko has quit IRC | 00:03 | |
| *** yoctozepto has quit IRC | 00:03 | |
| *** hack-char has quit IRC | 00:03 | |
| *** benj_ has quit IRC | 00:03 | |
| *** bbowen has quit IRC | 00:03 | |
| *** openstackgerrit has quit IRC | 00:03 | |
| *** elod has quit IRC | 00:03 | |
| *** irclogbot_0 has quit IRC | 00:03 | |
| *** antonym has quit IRC | 00:03 | |
| *** spotz has quit IRC | 00:03 | |
| *** arne_wiebalck has quit IRC | 00:03 | |
| *** hamalq has quit IRC | 00:03 | |
| *** jmlowe has quit IRC | 00:03 | |
| *** tkajinam has quit IRC | 00:03 | |
| *** dviroel has quit IRC | 00:03 | |
| *** jkulik has quit IRC | 00:03 | |
| *** StevenK has quit IRC | 00:03 | |
| *** guilhermesp has quit IRC | 00:03 | |
| *** rm_work has quit IRC | 00:03 | |
| *** fyx has quit IRC | 00:03 | |
| *** donnyd has quit IRC | 00:03 | |
| *** cz3 has quit IRC | 00:03 | |
| *** andrewbogott has quit IRC | 00:03 | |
| *** coreycb has quit IRC | 00:03 | |
| *** gmann has quit IRC | 00:03 | |
| *** melwitt has quit IRC | 00:03 | |
| *** ttx has quit IRC | 00:03 | |
| *** hamalq has joined #openstack-nova | 00:05 | |
| *** andrewbogott has joined #openstack-nova | 00:05 | |
| *** jmlowe has joined #openstack-nova | 00:05 | |
| *** tkajinam has joined #openstack-nova | 00:05 | |
| *** dviroel has joined #openstack-nova | 00:05 | |
| *** jkulik has joined #openstack-nova | 00:05 | |
| *** StevenK has joined #openstack-nova | 00:05 | |
| *** guilhermesp has joined #openstack-nova | 00:05 | |
| *** rm_work has joined #openstack-nova | 00:05 | |
| *** fyx has joined #openstack-nova | 00:05 | |
| *** donnyd has joined #openstack-nova | 00:05 | |
| *** cz3 has joined #openstack-nova | 00:05 | |
| *** coreycb has joined #openstack-nova | 00:05 | |
| *** gmann has joined #openstack-nova | 00:05 | |
| *** melwitt has joined #openstack-nova | 00:05 | |
| *** ttx has joined #openstack-nova | 00:05 | |
| *** rcernin has joined #openstack-nova | 00:06 | |
| *** gyee has joined #openstack-nova | 00:06 | |
| *** dklyle has joined #openstack-nova | 00:06 | |
| *** artom has joined #openstack-nova | 00:06 | |
| *** bbowen has joined #openstack-nova | 00:06 | |
| *** alex_xu has joined #openstack-nova | 00:06 | |
| *** tbachman has joined #openstack-nova | 00:06 | |
| *** lemko has joined #openstack-nova | 00:06 | |
| *** elod has joined #openstack-nova | 00:06 | |
| *** spotz has joined #openstack-nova | 00:06 | |
| *** yoctozepto has joined #openstack-nova | 00:06 | |
| *** irclogbot_0 has joined #openstack-nova | 00:06 | |
| *** antonym has joined #openstack-nova | 00:06 | |
| *** arne_wiebalck has joined #openstack-nova | 00:06 | |
| *** benj_ has joined #openstack-nova | 00:06 | |
| *** ociuhandu has quit IRC | 00:07 | |
| *** masterpe has quit IRC | 00:07 | |
| *** cz3 has quit IRC | 00:07 | |
| *** artom has quit IRC | 00:07 | |
| *** tosky has quit IRC | 00:07 | |
| *** alex_xu has quit IRC | 00:08 | |
| *** artom has joined #openstack-nova | 00:08 | |
| *** hack-char has joined #openstack-nova | 00:08 | |
| *** bbowen_ has joined #openstack-nova | 00:09 | |
| *** guilhermesp has quit IRC | 00:09 | |
| *** fyx has quit IRC | 00:09 | |
| *** donnyd has quit IRC | 00:09 | |
| *** andrewbogott has quit IRC | 00:09 | |
| *** coreycb has quit IRC | 00:09 | |
| *** gmann has quit IRC | 00:09 | |
| *** bbowen has quit IRC | 00:10 | |
| *** andrewbogott has joined #openstack-nova | 00:10 | |
| *** donnyd has joined #openstack-nova | 00:10 | |
| *** coreycb has joined #openstack-nova | 00:10 | |
| *** gmann has joined #openstack-nova | 00:10 | |
| *** irclogbot_0 has quit IRC | 00:10 | |
| *** guilhermesp has joined #openstack-nova | 00:11 | |
| *** fyx has joined #openstack-nova | 00:11 | |
| *** cz3 has joined #openstack-nova | 00:11 | |
| *** irclogbot_3 has joined #openstack-nova | 00:12 | |
| *** spatel has joined #openstack-nova | 00:12 | |
| *** artom has quit IRC | 00:15 | |
| *** spatel has quit IRC | 00:17 | |
| *** hamalq has quit IRC | 00:40 | |
| *** mlavalle has quit IRC | 00:42 | |
| *** masterpe has joined #openstack-nova | 00:43 | |
| *** zenkuro has quit IRC | 00:44 | |
| *** masterpe has quit IRC | 00:52 | |
| *** masterpe has joined #openstack-nova | 01:01 | |
| *** songwenping_ has quit IRC | 01:50 | |
| *** spatel has joined #openstack-nova | 02:17 | |
| *** sapd1 has joined #openstack-nova | 02:28 | |
| *** brinzhang_ has joined #openstack-nova | 02:31 | |
| *** brinzhang0 has quit IRC | 02:34 | |
| *** ociuhandu has joined #openstack-nova | 02:43 | |
| *** ociuhandu has quit IRC | 02:47 | |
| *** sapd1 has quit IRC | 02:50 | |
| *** brinzhang_ has quit IRC | 03:00 | |
| *** brinzhang_ has joined #openstack-nova | 03:00 | |
| *** sapd1 has joined #openstack-nova | 03:06 | |
| *** rcernin has quit IRC | 03:11 | |
| *** rcernin has joined #openstack-nova | 03:12 | |
| *** openstackgerrit has joined #openstack-nova | 03:29 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/c/openstack/nova/+/729563 | 03:29 |
|---|---|---|
| *** bbowen has joined #openstack-nova | 03:29 | |
| *** bbowen_ has quit IRC | 03:30 | |
| *** mkrai has joined #openstack-nova | 03:33 | |
| *** psachin has joined #openstack-nova | 03:36 | |
| *** gyee has quit IRC | 04:12 | |
| *** dave-mccowan has quit IRC | 04:40 | |
| *** ratailor has joined #openstack-nova | 05:28 | |
| *** brinzhang0 has joined #openstack-nova | 05:36 | |
| *** brinzhang0 has quit IRC | 05:37 | |
| *** brinzhang0 has joined #openstack-nova | 05:37 | |
| *** brinzhang_ has quit IRC | 05:38 | |
| *** psachin has quit IRC | 05:40 | |
| *** psachin has joined #openstack-nova | 06:22 | |
| *** spatel has quit IRC | 07:03 | |
| *** rcernin has quit IRC | 07:05 | |
| *** rcernin has joined #openstack-nova | 07:43 | |
| *** rcernin has quit IRC | 07:48 | |
| *** rcernin has joined #openstack-nova | 07:49 | |
| *** rcernin has quit IRC | 07:54 | |
| *** dklyle has quit IRC | 07:55 | |
| *** ralonsoh has joined #openstack-nova | 07:55 | |
| gibi | dansmith: thanks for finding out reasone of the multistore failure an proposing a fix. nice catch! | 08:04 |
| *** rpittau|afk is now known as rpittau | 08:09 | |
| *** andrewbonney has joined #openstack-nova | 08:12 | |
| lyarwood | morning | 08:17 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION and NEXT_MIN_{LIBVIRT,QEMU}_VERSION https://review.opendev.org/c/openstack/nova/+/754700 | 08:18 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_BLOCKDEV https://review.opendev.org/c/openstack/nova/+/754701 | 08:18 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_S390X_CPU_COMPARE https://review.opendev.org/c/openstack/nova/+/754704 | 08:18 |
| *** tesseract has joined #openstack-nova | 08:21 | |
| *** xek has joined #openstack-nova | 08:25 | |
| *** rcernin has joined #openstack-nova | 08:26 | |
| *** rcernin has quit IRC | 08:31 | |
| *** tosky has joined #openstack-nova | 08:42 | |
| *** lpetrut has joined #openstack-nova | 08:50 | |
| *** swp20 has joined #openstack-nova | 08:59 | |
| *** swp20 has quit IRC | 09:00 | |
| *** swp20 has joined #openstack-nova | 09:02 | |
| *** songwenping_ has joined #openstack-nova | 09:11 | |
| openstackgerrit | wushiming proposed openstack/os-vif master: Drop lower-constraints job https://review.opendev.org/c/openstack/os-vif/+/770907 | 09:11 |
| *** swp20 has quit IRC | 09:14 | |
| *** spatel has joined #openstack-nova | 09:16 | |
| *** nightmare_unreal has joined #openstack-nova | 09:17 | |
| nightmare_unreal | Hello , is there a way to force a VM to boot from volume. For e.g. during VM creation i want to enforce that it should be from volume only | 09:17 |
| kashyap | lyarwood: Morning; for the NEXT_ version bump, we also traditionally give a heads-up on the openstack-discuss list -- wanna do that too? (E.g. one of my past emails: http://lists.openstack.org/pipermail/openstack-discuss/2019-November/010907.html) | 09:19 |
| *** spatel has quit IRC | 09:21 | |
| openstackgerrit | wushiming proposed openstack/python-novaclient master: Drop lower-constraints job https://review.opendev.org/c/openstack/python-novaclient/+/770912 | 09:22 |
| lyarwood | kashyap: yup can do | 09:24 |
| kashyap | Thank ya. | 09:25 |
| *** rcernin has joined #openstack-nova | 09:28 | |
| *** ociuhandu has joined #openstack-nova | 09:33 | |
| *** rcernin has quit IRC | 09:33 | |
| songwenping_ | sean-k-mooney: Hi, what do you means about the vgpu spec: https://review.opendev.org/c/openstack/nova-specs/+/750116/17#L97, should we use rpc.cast the cyborg-agent to create a new mdev? | 09:35 |
| *** derekh has joined #openstack-nova | 09:37 | |
| lyarwood | kashyap: sent | 09:37 |
| kashyap | lyarwood: Just read the mail in the archives; looks fine, thx. | 09:39 |
| lyarwood | cool thanks | 09:41 |
| *** ociuhandu has quit IRC | 09:44 | |
| *** mkrai has quit IRC | 09:50 | |
| *** mkrai_ has joined #openstack-nova | 09:50 | |
| *** ociuhandu has joined #openstack-nova | 09:50 | |
| *** sapd1 has quit IRC | 10:04 | |
| *** macz_ has joined #openstack-nova | 10:07 | |
| *** macz_ has quit IRC | 10:12 | |
| *** mkrai_ has quit IRC | 10:18 | |
| *** mkrai has joined #openstack-nova | 10:19 | |
| songwenping_ | gibi: hi, about your comment in line https://review.opendev.org/c/openstack/nova/+/622336/20/nova/api/openstack/compute/remote_consoles.py#186, `if svc.version < <the new service version>` shoudld we raise 400 or 404? | 10:20 |
| brinzhang0 | gibi: cyborg shelve/unshelve updated alex_xu's concern, pls re-check again, thanks https://review.opendev.org/c/openstack/nova/+/729563 | 10:21 |
| gibi | songwenping_: I would go with http 400 | 10:21 |
| gibi | brinzhang0: ack | 10:21 |
| songwenping_ | gibi: get | 10:22 |
| brinzhang0 | gibi: yes, the same as songwenping_'s concern , IMO 400 is more suitable | 10:23 |
| brinzhang0 | gibi: thanks for check the cyborg shelve/unshelve patch ^^ | 10:24 |
| *** dtantsur|afk is now known as dtantsur | 10:38 | |
| *** rcernin has joined #openstack-nova | 10:45 | |
| *** Oriz|2 has joined #openstack-nova | 10:46 | |
| *** ociuhandu has quit IRC | 10:46 | |
| *** sanderp has quit IRC | 10:55 | |
| *** psachin has quit IRC | 10:58 | |
| *** songwenping__ has joined #openstack-nova | 11:12 | |
| *** songwenping_ has quit IRC | 11:15 | |
| *** ociuhandu has joined #openstack-nova | 11:17 | |
| *** rcernin has quit IRC | 11:17 | |
| *** ociuhandu has quit IRC | 11:26 | |
| *** ociuhandu has joined #openstack-nova | 11:27 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Log os-resetState as an instance action https://review.opendev.org/c/openstack/nova/+/770926 | 11:29 |
| *** rcernin has joined #openstack-nova | 11:30 | |
| *** martinkennelly has joined #openstack-nova | 11:33 | |
| *** mkrai has quit IRC | 11:37 | |
| gibi | bauzas: are your concerns answered in https://review.opendev.org/c/openstack/nova/+/729563 ? I'm holding the +W until ack it | 11:43 |
| *** jangutter_ has joined #openstack-nova | 11:48 | |
| *** jangutter has quit IRC | 11:51 | |
| *** jangutter has joined #openstack-nova | 11:55 | |
| *** jangutter_ has quit IRC | 11:57 | |
| *** macz_ has joined #openstack-nova | 11:58 | |
| *** raildo has joined #openstack-nova | 12:00 | |
| *** macz_ has quit IRC | 12:03 | |
| *** jamesdenton has quit IRC | 12:07 | |
| *** jamesdenton has joined #openstack-nova | 12:07 | |
| *** mgariepy has quit IRC | 12:14 | |
| *** artom has joined #openstack-nova | 12:16 | |
| *** spatel has joined #openstack-nova | 12:17 | |
| *** spatel has quit IRC | 12:21 | |
| *** rcernin has quit IRC | 12:31 | |
| *** brinzhang_ has joined #openstack-nova | 12:32 | |
| sean-k-mooney | songwenping__: i was suggesting that gets shoudl not create teh mdev ever. the cyborg agent should create the mdevs for all bound arqs on start up. ideally it would set the binding state to "provisioning" on start up also and we would have anohter state "unknown" that the api would report if the cyborg agent misses its heart beat | 12:36 |
| *** brinzhang0 has quit IRC | 12:36 | |
| sean-k-mooney | songwenping__: so on start up nova would see 1 of 3 states, bound if the cyborg agent started first and hadn completed bininding, unknown if the cyborg agent has not heartbeat to the cyborg conductor yet | 12:37 |
| sean-k-mooney | or provisioning if the agent is in the process of creating the mdev | 12:37 |
| sean-k-mooney | if cyborg was to send the same bining complete event when it change teh status form provisioning to bound as it does during normal arq binding | 12:38 |
| sean-k-mooney | the process in nova awould be the same | 12:38 |
| sean-k-mooney | e.g. set up event handeler, check if its already bound if not wait for binding complete event, if it si bound cancel the even waiter and proceed with boot | 12:39 |
| *** ratailor has quit IRC | 12:40 | |
| sean-k-mooney | gibi: bauzas does that sound like a resonable approch to ye instead of have get arq sometimes do an rpc call to the cybrog agent to create mdevs | 12:40 |
| bauzas | gibi: will look at https://review.opendev.org/c/openstack/nova/+/729563 | 12:41 |
| gibi | bauzas: thanks | 12:42 |
| bauzas | and for the vGPU support in Cyborg, I'll also look at the new revision today | 12:42 |
| gibi | sean-k-mooney: do we have this event handling at nova compute startup for neutron ports too? | 12:43 |
| *** whoami-rajat__ has joined #openstack-nova | 12:43 | |
| sean-k-mooney | gibi: we do not need to rebinding them in the neutron case just replug them. | 12:44 |
| sean-k-mooney | in the neutorn case os-vif is the thing that attaches the ports to the network backend in general | 12:45 |
| *** rcernin has joined #openstack-nova | 12:45 | |
| sean-k-mooney | which si the equivalent of creating the mdev | 12:45 |
| gibi | sean-k-mooney: I see | 12:45 |
| sean-k-mooney | i just find it unsetaling that we would considre change a get form a simple db lookup into an rpc call | 12:47 |
| gibi | sean-k-mooney: so at compute startup nova gets the binding state from cyborg, if it is unknow or provisioning then keep the guest power state off but set up an event waiter. If the arq state is "OK" then nova would start the guest during compute startup. | 12:48 |
| sean-k-mooney | gibi: not quite | 12:48 |
| gibi | correct me please | 12:48 |
| sean-k-mooney | i was thinking we would always set up the waiter and early out if it was bound like we do for normal spwan | 12:49 |
| gibi | can nova we loose an event during the compute reboot? | 12:50 |
| gibi | if yes then that guest would be stuck waiting of the event that was sent by cyborg but lost in the comptue restart | 12:50 |
| gibi | if the event is never lost then I'm OK to wait for the event | 12:51 |
| sean-k-mooney | the event would hit the api and then be enqued to the compute node topic queue | 12:51 |
| sean-k-mooney | so i dont think it would be lost | 12:51 |
| gibi | sean-k-mooney: cool | 12:51 |
| gibi | that seem OK | 12:51 |
| gibi | hm | 12:51 |
| gibi | so in this case the sending the event is triggered by cyborg agent restart, in any other case sending the event is triggered by a nova bind request | 12:52 |
| sean-k-mooney | yes | 12:52 |
| gibi | so there are extra cases to handle. 1) a single cyborg agent restart will send events and if the compute service was not restarted then these events needs to be consumed but ignored | 12:53 |
| sean-k-mooney | we could just call bind if we wanted too and not require teh cyborg agent to auto create them but i think the auto create would be more efficent | 12:53 |
| sean-k-mooney | gibi: we have unexpeted event handeling in nova already | 12:54 |
| gibi | cool | 12:54 |
| sean-k-mooney | if we dont have a waiter when we deque it we just log it and discard | 12:54 |
| gibi | that seems OK too then | 12:54 |
| sean-k-mooney | which si ok because we will check the state when we get to that part of the code | 12:54 |
| sean-k-mooney | basically im just suggesting using the exact saem event system we use of inital sapwn after where we start the binidng in the conductor then wait for it with an early out in the compute | 12:55 |
| gibi | OK, I don't have a #2 actually :) | 12:56 |
| sean-k-mooney | but in this case the cyborg agent would start the bind on start up | 12:56 |
| gibi | sean-k-mooney: so far what you suggest feels OK to me | 12:56 |
| sean-k-mooney | ill find the time stamp for this and add it to the reveiw | 12:57 |
| gibi | cool | 12:57 |
| gibi | thanks | 12:57 |
| sean-k-mooney | bauzas: if you have time to read scool back and find anything concering with that please chime in | 12:57 |
| *** zzzeek has quit IRC | 13:00 | |
| *** rcernin has quit IRC | 13:00 | |
| *** zzzeek has joined #openstack-nova | 13:00 | |
| *** mgariepy has joined #openstack-nova | 13:02 | |
| *** zenkuro has joined #openstack-nova | 13:03 | |
| *** rcernin has joined #openstack-nova | 13:05 | |
| *** nightmare_unreal has quit IRC | 13:14 | |
| *** JamesBenson has joined #openstack-nova | 13:19 | |
| *** sapd1 has joined #openstack-nova | 13:26 | |
| bauzas | sean-k-mooney: looking | 13:26 |
| *** derekh has quit IRC | 13:29 | |
| *** derekh has joined #openstack-nova | 13:30 | |
| *** rcernin has quit IRC | 13:30 | |
| bauzas | sean-k-mooney: are you talking about creating the mdevs in sysfs or binding them to the instance by modifying the XML ? | 13:33 |
| sean-k-mooney | bauzas: sysfs | 13:35 |
| bauzas | ack | 13:36 |
| bauzas | if so, I agree, Cyborg should create them | 13:36 |
| sean-k-mooney | specifically cyborg creating them | 13:36 |
| bauzas | (the agent) | 13:36 |
| sean-k-mooney | right but it shoudl do it automatically rather then as a result of a GET ot ARQ show | 13:36 |
| sean-k-mooney | well GET /ARQ/<uuid> | 13:37 |
| *** spatel has joined #openstack-nova | 13:48 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: WIP/DNM libvirt: Start emitting DeviceRemovedEvent and DeviceRemovalFailedEvent events https://review.opendev.org/c/openstack/nova/+/749929 | 13:55 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: DNM try to replace retry with libvirt event in detach https://review.opendev.org/c/openstack/nova/+/770246 | 13:55 |
| *** mlavalle has joined #openstack-nova | 13:58 | |
| bauzas | sean-k-mooney: yeah, provisioning them directly | 13:59 |
| bauzas | once the operator modifies the config | 14:00 |
| sean-k-mooney | what config? | 14:00 |
| bauzas | their own config for telling which vgpu type for each pGPU | 14:00 |
| sean-k-mooney | we are talking about recreating them after a host reboot | 14:00 |
| sean-k-mooney | the cyborg spec currently say after a host reboot when we reboot the instance that we will do a arq show | 14:01 |
| bauzas | I haven't seen it | 14:01 |
| sean-k-mooney | and that show will do an rpc to the agent to create the mdev | 14:01 |
| bauzas | if so, -1 for me | 14:01 |
| sean-k-mooney | im suggesting that instead on start up the agent shoucl check what instance are on the current host and ensure there mdevs exist | 14:01 |
| sean-k-mooney | and that arqs should have 2 newe states | 14:02 |
| sean-k-mooney | unknon meaning the agent missed it heart beat | 14:02 |
| sean-k-mooney | and provisioning meaning its currenlty seting up the mdev | 14:02 |
| sean-k-mooney | so when we do the show arq binding call if its in provisioning or unknow we wait for the async event | 14:03 |
| sean-k-mooney | if its in bound we know cyborg is done and we proceed | 14:03 |
| *** songwenping__ has quit IRC | 14:10 | |
| *** songwenping__ has joined #openstack-nova | 14:10 | |
| bauzas | sean-k-mooney: then I agree with you | 14:10 |
| bauzas | have you provided those comments in the spec ? | 14:11 |
| sean-k-mooney | yes although not that cohently initally so i reference the irc logs above | 14:11 |
| sean-k-mooney | i had give that feedbac in patch set 7 or 9 too | 14:11 |
| bauzas | ++ | 14:22 |
| bauzas | I'll then add my comments then too | 14:22 |
| *** dave-mccowan has joined #openstack-nova | 14:28 | |
| *** ociuhandu_ has joined #openstack-nova | 14:28 | |
| *** ociuhandu has quit IRC | 14:31 | |
| *** nweinber has joined #openstack-nova | 14:35 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: trivial: Resolve (most) flake8 3.x issues https://review.opendev.org/c/openstack/nova/+/770943 | 14:39 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: WIP/DNM Switch to hacking 2.x https://review.opendev.org/c/openstack/nova/+/770944 | 14:39 |
| stephenfin | lyarwood: Does this need a blueprint, I wonder? It is kind of feature'ish https://review.opendev.org/c/openstack/nova/+/770926 | 14:47 |
| *** rpittau is now known as rpittau|afk | 14:48 | |
| lyarwood | stephenfin: likely, I was procrastinating this morning and wrote that without thinking | 15:03 |
| lyarwood | gibi: ^ re this, should I create a blueprint for this? | 15:07 |
| *** lpetrut has quit IRC | 15:07 | |
| gibi | lyarwood: yeah, a bp would be good for that it is adding a feature basically | 15:08 |
| gibi | or we can phrase the whole thing as a bug | 15:09 |
| gibi | we forget to log resetState into the instance actions | 15:09 |
| gibi | so meh, either a bp or a bug would be good | 15:09 |
| *** ociuhandu_ has quit IRC | 15:09 | |
| gibi | no structural API impact so definetly not a spec | 15:10 |
| lyarwood | gibi: ack let me spin this into a bug | 15:19 |
| gibi | works for me | 15:20 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: api-ref: Clarify 'all_tenants' command https://review.opendev.org/c/openstack/nova/+/770947 | 15:21 |
| stephenfin | That's the easiest "bugfix" anyone will see this week ^ | 15:21 |
| *** ociuhandu has joined #openstack-nova | 15:22 | |
| gibi | stephenfin: +@ | 15:24 |
| gibi | stephenfin: +2 | 15:24 |
| stephenfin | thanks :) | 15:24 |
| *** jmlowe has quit IRC | 15:27 | |
| dansmith | ahh, the rare but coveted +@ vote | 15:30 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Log os-resetState as an instance action https://review.opendev.org/c/openstack/nova/+/770926 | 15:30 |
| gibi | dansmith: :) | 15:39 |
| stephenfin | Is https://docs.openstack.org/ down for anyone else? | 15:41 |
| stephenfin | nvm, it's back | 15:42 |
| lyarwood | stephenfin: rate limited again? :D | 15:43 |
| *** dklyle has joined #openstack-nova | 15:50 | |
| * stephenfin glares at lyarwood | 15:51 | |
| *** mgariepy has quit IRC | 15:55 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Log os-resetState as an instance action https://review.opendev.org/c/openstack/nova/+/770926 | 16:06 |
| lyarwood | stephenfin: well you did ask me to write another bloody reno ;) ^ | 16:06 |
| dansmith | stephenfin: really nice job on the db compaction series... you made validating them as easy as I hoped, so I hope we'll get at least another core or two to do it :) | 16:09 |
| stephenfin | oh, thanks. Hopefully | 16:12 |
| *** slaweq has joined #openstack-nova | 16:19 | |
| stephenfin | melwitt: quota question: if I create a server in a server group, should I expect the server_group_members.in_use count seen in 'nova quota-show --detail' to increment? | 16:28 |
| melwitt | stephenfin: that might be one that stays stuck at zero. let me look and refresh my memory | 16:29 |
| melwitt | stephenfin: yeah, that is one of the two that stays at zero (legacy behavior that hasn't been changed) https://github.com/openstack/nova/blob/master/nova/quota.py#L178-L182 | 16:32 |
| melwitt | so you would expect it not to ever increment | 16:32 |
| stephenfin | okay, great. Thanks for the confirmation | 16:32 |
| * stephenfin closes https://bugs.launchpad.net/nova/+bug/1909269 | 16:33 | |
| openstack | Launchpad bug 1909269 in OpenStack Compute (nova) "I create a server_groups vm , but server_group_members doesn't add one." [Undecided,New] | 16:33 |
| *** dtantsur is now known as dtantsur|afk | 16:41 | |
| *** iurygregory has quit IRC | 16:41 | |
| *** mgariepy has joined #openstack-nova | 16:42 | |
| lyarwood | stephenfin: did you plan on wiring up openstack server migration show btw? | 16:43 |
| * lyarwood has lost the link to the pad tracking all of this if you have it to hand | 16:44 | |
| stephenfin | oh, fair point | 16:45 |
| stephenfin | I don't think there's a patch open for that. I can add one | 16:45 |
| stephenfin | https://etherpad.opendev.org/p/compute-api-microversion-gap-in-osc | 16:45 |
| * lyarwood bookmarks | 16:45 | |
| lyarwood | stephenfin: cool happy to also look at that post m2 | 16:46 |
| lyarwood | in the context of https://bugs.launchpad.net/nova/+bug/1910781 btw | 16:46 |
| openstack | Launchpad bug 1910781 in OpenStack Compute (nova) "Live migration to different host returns complete but nothing has happened" [Undecided,New] | 16:46 |
| lyarwood | looks like there's another bug there with --wait | 16:46 |
| * stephenfin looks | 16:47 | |
| *** ociuhandu_ has joined #openstack-nova | 16:53 | |
| *** ociuhandu has quit IRC | 16:56 | |
| *** ociuhandu_ has quit IRC | 16:58 | |
| *** sapd1 has quit IRC | 17:01 | |
| *** iurygregory has joined #openstack-nova | 17:06 | |
| openstackgerrit | Merged openstack/nova master: Fix typo in warning message https://review.opendev.org/c/openstack/nova/+/770067 | 17:06 |
| *** ociuhandu has joined #openstack-nova | 17:11 | |
| *** slaweq has quit IRC | 17:12 | |
| *** xek has quit IRC | 17:13 | |
| *** ociuhandu has quit IRC | 17:16 | |
| *** mlavalle has quit IRC | 17:31 | |
| *** mlavalle has joined #openstack-nova | 17:32 | |
| *** dave-mccowan has quit IRC | 17:38 | |
| *** tbachman_ has joined #openstack-nova | 17:38 | |
| *** tbachman has quit IRC | 17:39 | |
| *** tbachman_ is now known as tbachman | 17:39 | |
| *** ralonsoh has quit IRC | 17:56 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765772 | 17:57 |
| *** derekh has quit IRC | 18:03 | |
| *** gyee has joined #openstack-nova | 18:14 | |
| *** tesseract has quit IRC | 18:27 | |
| *** andrewbonney has quit IRC | 19:00 | |
| *** zenkuro has quit IRC | 19:03 | |
| *** spatel has quit IRC | 19:03 | |
| *** zenkuro has joined #openstack-nova | 19:04 | |
| *** Oriz|2 has quit IRC | 19:09 | |
| *** slaweq has joined #openstack-nova | 19:22 | |
| gmann | dansmith: melwitt if you are around can you check these two reviews for JSON->YAMl policy file migration - https://review.opendev.org/c/openstack/placement/+/764394 https://review.opendev.org/c/openstack/nova/+/763487 | 19:23 |
| *** jdillaman has quit IRC | 19:26 | |
| *** rcernin has joined #openstack-nova | 19:27 | |
| *** rcernin has quit IRC | 19:31 | |
| dansmith | gmann: you replace a function ref with a (function, dict) tuple.. do the upgrade checks support that for args? | 19:52 |
| dansmith | functools.partial or lambda would make that a little clearer, but I'm not sure what exactly parses this | 19:54 |
| *** andrewbogott has left #openstack-nova | 20:06 | |
| gmann | dansmith: I added the support for tuple in oslo.upgradecheck - https://github.com/openstack/oslo.upgradecheck/blob/a962943bdaa746ad238f003f56556eb02d1ea8e8/oslo_upgradecheck/upgradecheck.py#L104 | 20:10 |
| *** slaweq has quit IRC | 20:11 | |
| dansmith | gmann: okay | 20:11 |
| dansmith | gmann: left you another question | 20:11 |
| gmann | dansmith: this one? https://review.opendev.org/c/openstack/nova/+/763487/9/nova/policy.py#79 | 20:12 |
| melwitt | gmann: ack, will look | 20:12 |
| dansmith | gmann: yar | 20:12 |
| dansmith | oh sorry | 20:12 |
| * dansmith refreshes | 20:12 | |
| gmann | we have covered the test on oslo side, you want to keep existing nova test too | 20:12 |
| dansmith | no, | 20:13 |
| dansmith | I just want to make sure we know what is and isn't changing | 20:13 |
| dansmith | are you saying that oslo.policy controls CONF.policy_file, and is handling the CONF.find_file behavior? | 20:14 |
| gmann | ok, yeah be default this fallback is maintained in oslo side until oslo.policy remove the JSON support | 20:14 |
| dansmith | oh wait wait, I see | 20:14 |
| dansmith | because we're calling into policy.Enforcer... nevermind | 20:14 |
| dansmith | sorry | 20:14 |
| gmann | yeah https://github.com/openstack/oslo.policy/blob/0a228dea2ee96ec3eabed3361ca22502d0bbd4a1/oslo_policy/policy.py#L533 | 20:15 |
| openstackgerrit | Merged openstack/nova stable/stein: [stable-only] Fix lower-constraints job https://review.opendev.org/c/openstack/nova/+/769809 | 20:15 |
| openstackgerrit | Ghanshyam proposed openstack/placement master: [goal] Deprecate the JSON formatted policy file https://review.opendev.org/c/openstack/placement/+/764394 | 20:35 |
| gmann | dansmith: ^^ updated | 20:35 |
| melwitt | gmann: one comment on the nova change, looks like the code you need is in 1.2.0, did you want to go ahead and keep it at 1.3.0? | 20:38 |
| gmann | melwitt: i remember something extra i fixed in 1.3.0, let me check | 20:38 |
| melwitt | gmann: ok, completely up to you. just pointed it out in case it isn't what you intended | 20:39 |
| gmann | melwitt: CLI opt registration things added in 1.30 - https://github.com/openstack/oslo.upgradecheck/blob/1.3.0/oslo_upgradecheck/common_checks.py#L29 | 20:40 |
| gmann | replied in gerrit too | 20:40 |
| melwitt | gmann: ok, great. thanks for clarifying that | 20:41 |
| gmann | thanks dansmith melwitt for review, have a nice weekend | 20:53 |
| dansmith | gmann: you as well :) | 20:53 |
| gmann | thx :) | 20:54 |
| *** rcernin has joined #openstack-nova | 20:57 | |
| melwitt | gmann: have a nice weekend o/ | 20:58 |
| *** rcernin has quit IRC | 21:01 | |
| *** JamesBenson has quit IRC | 21:03 | |
| *** bbowen has quit IRC | 21:05 | |
| *** raildo has quit IRC | 21:08 | |
| *** nweinber has quit IRC | 21:24 | |
| *** whoami-rajat__ has quit IRC | 21:26 | |
| *** rcernin has joined #openstack-nova | 21:53 | |
| *** rcernin has quit IRC | 21:58 | |
| *** rcernin has joined #openstack-nova | 22:04 | |
| *** ociuhandu has joined #openstack-nova | 22:20 | |
| *** ociuhandu has quit IRC | 22:25 | |
| *** tbachman has quit IRC | 22:27 | |
| *** tbachman has joined #openstack-nova | 22:29 | |
| *** bbowen has joined #openstack-nova | 22:39 | |
| *** logan- has quit IRC | 22:40 | |
| openstackgerrit | Merged openstack/nova master: Reuse code from oslo lib for JSON policy migration https://review.opendev.org/c/openstack/nova/+/763487 | 22:44 |
| *** logan- has joined #openstack-nova | 22:53 | |
| *** yoctozepto5 has joined #openstack-nova | 22:54 | |
| *** yoctozepto has quit IRC | 22:55 | |
| *** yoctozepto5 is now known as yoctozepto | 22:55 | |
| *** vdrok has quit IRC | 22:57 | |
| *** vdrok has joined #openstack-nova | 22:58 | |
| *** flaviof has quit IRC | 22:58 | |
| *** TheJulia has quit IRC | 22:58 | |
| *** flaviof has joined #openstack-nova | 22:58 | |
| *** TheJulia has joined #openstack-nova | 22:58 | |
| *** nautik has quit IRC | 22:58 | |
| *** cap has quit IRC | 23:00 | |
| *** coreycb has quit IRC | 23:01 | |
| *** jrosser has quit IRC | 23:01 | |
| *** coreycb has joined #openstack-nova | 23:01 | |
| *** cap has joined #openstack-nova | 23:01 | |
| *** jrosser has joined #openstack-nova | 23:03 | |
| *** ftarasenko has quit IRC | 23:03 | |
| *** logan- has quit IRC | 23:04 | |
| *** songwenping_ has joined #openstack-nova | 23:08 | |
| *** songwenping__ has quit IRC | 23:11 | |
| *** yoctozepto6 has joined #openstack-nova | 23:16 | |
| *** yoctozepto has quit IRC | 23:17 | |
| *** yoctozepto6 is now known as yoctozepto | 23:17 | |
| *** logan- has joined #openstack-nova | 23:19 | |
| *** eandersson has quit IRC | 23:20 | |
| *** aarents has quit IRC | 23:20 | |
| *** johanssone has quit IRC | 23:20 | |
| *** aarents has joined #openstack-nova | 23:21 | |
| *** eandersson has joined #openstack-nova | 23:21 | |
| *** johanssone has joined #openstack-nova | 23:24 | |
| *** yoctozepto4 has joined #openstack-nova | 23:43 | |
| *** yoctozepto has quit IRC | 23:44 | |
| *** yoctozepto4 is now known as yoctozepto | 23:44 | |
| *** yoctozepto4 has joined #openstack-nova | 23:58 | |
| *** yoctozepto has quit IRC | 23:59 | |
| *** yoctozepto4 is now known as yoctozepto | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!