| *** d34dh0r53 has quit IRC | 00:16 | |
| *** pvradu has quit IRC | 00:18 | |
| *** d34dh0r53 has joined #openstack-nova | 00:25 | |
| *** d34dh0r53 has quit IRC | 00:25 | |
| *** mlavalle has quit IRC | 00:26 | |
| *** d34dh0r53 has joined #openstack-nova | 00:28 | |
| *** d34dh0r53 has quit IRC | 00:30 | |
| *** d34dh0r53 has joined #openstack-nova | 00:35 | |
| *** d34dh0r53 has quit IRC | 00:36 | |
| *** d34dh0r53 has joined #openstack-nova | 00:39 | |
| *** d34dh0r53 has quit IRC | 00:39 | |
| *** d34dh0r53 has joined #openstack-nova | 00:43 | |
| *** d34dh0r53 has quit IRC | 00:44 | |
| *** hoonetorg has quit IRC | 00:45 | |
| *** d34dh0r53 has joined #openstack-nova | 00:52 | |
| *** d34dh0r53 has quit IRC | 00:53 | |
| *** hoonetorg has joined #openstack-nova | 00:57 | |
| *** d34dh0r53 has joined #openstack-nova | 00:59 | |
| *** d34dh0r53 has quit IRC | 00:59 | |
| *** d34dh0r53 has joined #openstack-nova | 01:01 | |
| *** d34dh0r53 has quit IRC | 01:04 | |
| *** d34dh0r53 has joined #openstack-nova | 01:14 | |
| *** d34dh0r53 has quit IRC | 01:14 | |
| *** lvbin02 has joined #openstack-nova | 01:14 | |
| *** lvbin01 has quit IRC | 01:18 | |
| *** lvbin02 is now known as lvbin01 | 01:18 | |
| *** liuyulong has quit IRC | 01:23 | |
| *** d34dh0r53 has joined #openstack-nova | 01:28 | |
| *** d34dh0r53 has quit IRC | 01:28 | |
| openstackgerrit | Merged openstack/nova master: nova-net: Remove firewall support (pt. 1) https://review.opendev.org/696514 | 01:30 |
|---|---|---|
| *** d34dh0r53 has joined #openstack-nova | 01:33 | |
| *** d34dh0r53 has quit IRC | 01:34 | |
| *** d34dh0r53 has joined #openstack-nova | 01:37 | |
| *** d34dh0r53 has quit IRC | 01:38 | |
| *** d34dh0r53 has joined #openstack-nova | 01:41 | |
| *** d34dh0r53 has quit IRC | 01:42 | |
| *** d34dh0r53 has joined #openstack-nova | 01:52 | |
| *** d34dh0r53 has quit IRC | 01:52 | |
| openstackgerrit | Merged openstack/nova master: nova-net: Remove firewall support (pt. 2) https://review.opendev.org/700510 | 01:55 |
| *** d34dh0r53 has joined #openstack-nova | 01:56 | |
| *** d34dh0r53 has quit IRC | 01:57 | |
| *** d34dh0r53 has joined #openstack-nova | 02:02 | |
| *** d34dh0r53 has quit IRC | 02:03 | |
| *** d34dh0r53 has joined #openstack-nova | 02:05 | |
| *** d34dh0r53 has quit IRC | 02:05 | |
| *** d34dh0r53 has joined #openstack-nova | 02:09 | |
| *** d34dh0r53 has quit IRC | 02:10 | |
| *** pvradu has joined #openstack-nova | 02:16 | |
| *** d34dh0r53 has joined #openstack-nova | 02:19 | |
| *** d34dh0r53 has quit IRC | 02:19 | |
| *** d34dh0r53 has joined #openstack-nova | 02:21 | |
| *** d34dh0r53 has quit IRC | 02:22 | |
| *** hamzy has joined #openstack-nova | 02:22 | |
| *** d34dh0r53 has joined #openstack-nova | 02:38 | |
| *** d34dh0r53 has quit IRC | 02:39 | |
| *** mmethot has quit IRC | 02:43 | |
| *** yedongcan has joined #openstack-nova | 02:44 | |
| *** d34dh0r53 has joined #openstack-nova | 02:45 | |
| *** d34dh0r53 has quit IRC | 02:45 | |
| *** d34dh0r53 has joined #openstack-nova | 02:47 | |
| *** d34dh0r53 has quit IRC | 02:47 | |
| *** pvradu has quit IRC | 02:47 | |
| *** d34dh0r53 has joined #openstack-nova | 02:54 | |
| *** d34dh0r53 has quit IRC | 02:55 | |
| *** mkrai_ has joined #openstack-nova | 03:02 | |
| *** d34dh0r53 has joined #openstack-nova | 03:09 | |
| *** d34dh0r53 has quit IRC | 03:09 | |
| *** d34dh0r53 has joined #openstack-nova | 03:19 | |
| *** d34dh0r53 has quit IRC | 03:19 | |
| *** d34dh0r53 has joined #openstack-nova | 03:28 | |
| *** d34dh0r53 has quit IRC | 03:28 | |
| *** psachin has joined #openstack-nova | 03:33 | |
| *** mmethot has joined #openstack-nova | 03:34 | |
| *** d34dh0r53 has joined #openstack-nova | 03:41 | |
| *** d34dh0r53 has quit IRC | 03:41 | |
| *** d34dh0r53 has joined #openstack-nova | 03:48 | |
| *** ileixe has quit IRC | 03:59 | |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341 | 03:59 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | 03:59 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | 04:00 |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 04:00 |
| *** ileixe has joined #openstack-nova | 04:03 | |
| *** bhagyashris has joined #openstack-nova | 04:18 | |
| *** d34dh0r53 has quit IRC | 04:40 | |
| *** d34dh0r53 has joined #openstack-nova | 04:43 | |
| *** pvradu has joined #openstack-nova | 04:44 | |
| *** d34dh0r53 has joined #openstack-nova | 04:45 | |
| *** d34dh0r53 has quit IRC | 04:45 | |
| *** d34dh0r53 has joined #openstack-nova | 04:47 | |
| *** d34dh0r53 has quit IRC | 04:48 | |
| *** d34dh0r53 has joined #openstack-nova | 04:53 | |
| *** d34dh0r53 has quit IRC | 04:53 | |
| *** d34dh0r53 has joined #openstack-nova | 04:54 | |
| *** d34dh0r53 has quit IRC | 04:56 | |
| *** gyee has quit IRC | 04:57 | |
| *** d34dh0r53 has joined #openstack-nova | 05:01 | |
| *** udesale has joined #openstack-nova | 05:03 | |
| *** spsurya has joined #openstack-nova | 05:05 | |
| *** ileixe has quit IRC | 05:06 | |
| *** ileixe has joined #openstack-nova | 05:08 | |
| *** TxGirlGeek has quit IRC | 05:09 | |
| *** links has joined #openstack-nova | 05:15 | |
| *** pvradu has quit IRC | 05:17 | |
| *** mkrai_ has quit IRC | 05:20 | |
| *** mkrai_ has joined #openstack-nova | 05:24 | |
| *** bhagyashris has quit IRC | 05:28 | |
| *** mkrai_ has quit IRC | 05:29 | |
| *** bhagyashris has joined #openstack-nova | 05:30 | |
| *** mkrai has joined #openstack-nova | 05:30 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-nova | 05:33 | |
| *** dave-mccowan has quit IRC | 05:36 | |
| *** mkrai has quit IRC | 05:36 | |
| *** tetsuro has joined #openstack-nova | 05:41 | |
| *** mkrai has joined #openstack-nova | 05:44 | |
| openstackgerrit | Merged openstack/nova master: Fix the suppress of policy deprecation warnings https://review.opendev.org/700797 | 05:59 |
| *** yaawang has quit IRC | 06:12 | |
| *** yaawang has joined #openstack-nova | 06:12 | |
| openstackgerrit | Merged openstack/nova master: Remove 'nova-xvpvncproxy' https://review.opendev.org/687909 | 06:27 |
| *** jawad_axd has joined #openstack-nova | 06:49 | |
| *** ratailor has joined #openstack-nova | 06:50 | |
| *** bhagyashris has quit IRC | 06:51 | |
| *** bhagyashris has joined #openstack-nova | 06:52 | |
| *** dpawlik has joined #openstack-nova | 06:52 | |
| *** jawad_ax_ has joined #openstack-nova | 06:52 | |
| *** jawad_axd has quit IRC | 06:54 | |
| *** pvradu has joined #openstack-nova | 07:00 | |
| *** damien_r has joined #openstack-nova | 07:20 | |
| *** damien_r has quit IRC | 07:25 | |
| *** pcaruana has joined #openstack-nova | 07:38 | |
| *** ociuhandu has joined #openstack-nova | 07:46 | |
| *** bhagyashris has quit IRC | 07:48 | |
| *** ratailor has quit IRC | 07:50 | |
| *** zhanglong has joined #openstack-nova | 07:54 | |
| *** ociuhandu has quit IRC | 07:58 | |
| *** ratailor has joined #openstack-nova | 07:59 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:02 | |
| *** pvradu has quit IRC | 08:02 | |
| *** pvradu has joined #openstack-nova | 08:03 | |
| *** damien_r has joined #openstack-nova | 08:03 | |
| *** tosky has joined #openstack-nova | 08:05 | |
| *** tesseract has joined #openstack-nova | 08:06 | |
| *** bhagyashris has joined #openstack-nova | 08:06 | |
| *** zhanglong has quit IRC | 08:06 | |
| *** zhanglong has joined #openstack-nova | 08:08 | |
| *** huaqiang has quit IRC | 08:13 | |
| *** lbragstad has quit IRC | 08:28 | |
| *** tkajinam has quit IRC | 08:29 | |
| *** bhagyashris has quit IRC | 08:32 | |
| *** tosky_ has joined #openstack-nova | 08:33 | |
| *** tosky has quit IRC | 08:34 | |
| *** bhagyashris has joined #openstack-nova | 08:37 | |
| *** tosky_ is now known as tosky | 08:39 | |
| *** lbragstad has joined #openstack-nova | 08:40 | |
| *** _rajat_ has joined #openstack-nova | 08:40 | |
| *** pvradu has quit IRC | 08:41 | |
| *** brinzhang_ has quit IRC | 08:46 | |
| *** yedongcan has quit IRC | 08:47 | |
| *** yedongcan has joined #openstack-nova | 08:48 | |
| *** ralonsoh has joined #openstack-nova | 08:52 | |
| gibi | gmann: ack, thanks for the headsup abandoned my policy warning patch | 09:02 |
| *** jraju__ has joined #openstack-nova | 09:03 | |
| *** links has quit IRC | 09:04 | |
| *** xek has joined #openstack-nova | 09:05 | |
| *** diablo_rojo has joined #openstack-nova | 09:07 | |
| gibi | stephenfin: hi! could you check back to https://review.opendev.org/#/c/701364 it is green now | 09:08 |
| *** brinzhang has joined #openstack-nova | 09:12 | |
| *** brinzhang_ has joined #openstack-nova | 09:14 | |
| *** brinzhang_ has quit IRC | 09:16 | |
| *** brinzhang_ has joined #openstack-nova | 09:16 | |
| *** brinzhang has quit IRC | 09:18 | |
| *** rpittau|afk is now known as rpittau | 09:25 | |
| *** zhanglong has quit IRC | 09:29 | |
| *** brinzhang has joined #openstack-nova | 09:31 | |
| *** brinzhang_ has quit IRC | 09:34 | |
| *** brinzhang has quit IRC | 09:36 | |
| *** martinkennelly has joined #openstack-nova | 09:39 | |
| *** derekh has joined #openstack-nova | 09:42 | |
| *** bhagyashris has quit IRC | 09:43 | |
| *** ociuhandu has joined #openstack-nova | 10:02 | |
| openstackgerrit | Merged openstack/nova master: libvirt: avoid cpu check at s390x arch https://review.opendev.org/696228 | 10:03 |
| openstackgerrit | jichenjc proposed openstack/nova stable/train: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701525 | 10:06 |
| openstackgerrit | jichenjc proposed openstack/nova stable/stein: libvirt: avoid cpu check at s390x arch https://review.opendev.org/701526 | 10:06 |
| *** dpawlik has quit IRC | 10:07 | |
| *** dpawlik has joined #openstack-nova | 10:14 | |
| *** ociuhandu has quit IRC | 10:19 | |
| *** links has joined #openstack-nova | 10:22 | |
| *** jraju__ has quit IRC | 10:23 | |
| *** ivve has joined #openstack-nova | 10:31 | |
| *** ratailor_ has joined #openstack-nova | 10:50 | |
| *** dtantsur|afk is now known as dtantsur | 10:50 | |
| *** ratailor has quit IRC | 10:52 | |
| *** ociuhandu has joined #openstack-nova | 10:59 | |
| *** sapd1_x has joined #openstack-nova | 11:02 | |
| *** mkrai has quit IRC | 11:04 | |
| *** portdirect has quit IRC | 11:06 | |
| *** shilpasd has joined #openstack-nova | 11:08 | |
| *** ociuhandu has quit IRC | 11:18 | |
| *** ociuhandu has joined #openstack-nova | 11:29 | |
| *** ociuhandu has quit IRC | 11:34 | |
| *** yedongcan has left #openstack-nova | 11:38 | |
| *** diablo_rojo has quit IRC | 11:38 | |
| *** sapd1_x has quit IRC | 11:47 | |
| *** mkrai has joined #openstack-nova | 11:55 | |
| *** shilpasd has quit IRC | 12:07 | |
| *** dviroel has joined #openstack-nova | 12:07 | |
| stephenfin | gibi: Sorry, missed the ping. Will hit now | 12:15 |
| stephenfin | gibi: done | 12:22 |
| *** ratailor_ has quit IRC | 12:23 | |
| *** mkrai has quit IRC | 12:31 | |
| stephenfin | efried: question on https://review.opendev.org/#/c/686804/12/specs/ussuri/approved/add-emulated-virtual-tpm.rst@365 | 12:34 |
| stephenfin | wdym we can't do an API check? How's this different from the check sean-k-mooney added for the NUMA stuff? | 12:34 |
| stephenfin | basically if the old and new images have different vTPM metadata, fail? | 12:35 |
| *** jawad_ax_ has quit IRC | 12:38 | |
| sean-k-mooney | stephenfin: i was wondering that too | 12:48 |
| sean-k-mooney | i had acess to the old image which is embded in the instance, the new image and the flavor | 12:49 |
| sean-k-mooney | so that should be enough to do a check in the rebuild case | 12:50 |
| *** udesale has quit IRC | 12:52 | |
| *** mkrai has joined #openstack-nova | 12:53 | |
| *** jawad_axd has joined #openstack-nova | 12:55 | |
| *** liuyulong has joined #openstack-nova | 12:58 | |
| *** jawad_ax_ has joined #openstack-nova | 12:59 | |
| *** liuyulong has quit IRC | 12:59 | |
| *** jawad_axd has quit IRC | 12:59 | |
| openstackgerrit | Merged openstack/nova master: Fix excessive runtime of test test_migrate_within_cell https://review.opendev.org/701364 | 13:01 |
| *** jawad_ax_ has quit IRC | 13:03 | |
| *** jawad_axd has joined #openstack-nova | 13:04 | |
| *** liuyulong has joined #openstack-nova | 13:08 | |
| *** liuyulong_ has joined #openstack-nova | 13:08 | |
| *** jawad_axd has quit IRC | 13:09 | |
| *** ociuhandu has joined #openstack-nova | 13:09 | |
| gibi | stephenfin: thanks | 13:09 |
| *** jawad_axd has joined #openstack-nova | 13:09 | |
| *** _rajat_ has quit IRC | 13:10 | |
| *** jawad_ax_ has joined #openstack-nova | 13:12 | |
| *** jawad_axd has quit IRC | 13:14 | |
| *** jawad_axd has joined #openstack-nova | 13:16 | |
| *** jawad_ax_ has quit IRC | 13:16 | |
| *** ociuhandu has quit IRC | 13:17 | |
| *** jawad_axd has quit IRC | 13:21 | |
| *** sapd1 has joined #openstack-nova | 13:24 | |
| *** shilpasd has joined #openstack-nova | 13:31 | |
| *** dpawlik has quit IRC | 13:37 | |
| *** dpawlik has joined #openstack-nova | 13:37 | |
| *** ociuhandu has joined #openstack-nova | 13:38 | |
| *** ociuhandu has quit IRC | 13:42 | |
| *** nweinber__ has joined #openstack-nova | 13:44 | |
| *** jawad_axd has joined #openstack-nova | 13:45 | |
| *** mkrai has quit IRC | 13:49 | |
| *** jawad_axd has quit IRC | 13:49 | |
| *** Liang__ has joined #openstack-nova | 13:53 | |
| *** jawad_axd has joined #openstack-nova | 13:56 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove firewall support (pt. 3) https://review.opendev.org/700511 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Copy shared utils from nova-net module https://review.opendev.org/696515 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove final references to nova-network https://review.opendev.org/696516 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove 'MetadataManager' https://review.opendev.org/696748 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Add TODOs to remove security group-related objects https://review.opendev.org/696517 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove dependency on nova-net from fake cache https://review.opendev.org/696749 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Kill it https://review.opendev.org/696518 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused exceptions https://review.opendev.org/697149 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unnecessary 'neutronv2' prefixes https://review.opendev.org/696776 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Rename 'nova.network.neutronv2' -> 'nova.network' https://review.opendev.org/696745 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Make the security group API a module https://review.opendev.org/696746 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'FloatingIP' object https://review.opendev.org/697153 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'Network' object https://review.opendev.org/697154 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove use of legacy 'SecurityGroup' object https://review.opendev.org/697155 | 13:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | 13:57 |
| *** Liang__ is now known as LiangFang | 13:58 | |
| *** jawad_axd has quit IRC | 14:00 | |
| *** lvbin01 has quit IRC | 14:19 | |
| *** lvbin01 has joined #openstack-nova | 14:20 | |
| *** jawad_axd has joined #openstack-nova | 14:21 | |
| *** mriedem has joined #openstack-nova | 14:23 | |
| *** jawad_axd has quit IRC | 14:26 | |
| shilpasd | mriedem: thanks for review | 14:32 |
| *** links has quit IRC | 14:37 | |
| *** ociuhandu has joined #openstack-nova | 14:38 | |
| *** bhagyashris has joined #openstack-nova | 14:39 | |
| *** eharney has quit IRC | 14:42 | |
| *** ociuhandu has quit IRC | 14:43 | |
| *** ociuhandu has joined #openstack-nova | 14:44 | |
| stephenfin | gibi: when you've a chance, could you hit https://review.opendev.org/#/c/700511/ again? Just addressed two of efried's nits | 14:44 |
| gibi | stephenfin: looking... | 14:47 |
| *** ociuhandu has quit IRC | 14:47 | |
| gibi | stephenfin: done | 14:48 |
| gibi | stephenfin: I will try to look into the next set of patches in the series during today or tomorrow | 14:49 |
| *** mriedem has quit IRC | 14:54 | |
| *** mriedem has joined #openstack-nova | 14:57 | |
| bauzas | gibi: can I ask you a question ? | 14:57 |
| bauzas | gibi: I'm looking at your comments on https://review.opendev.org/#/c/670112/11/nova/tests/functional/test_nova_manage.py | 14:58 |
| bauzas | gibi: I honestly prefer to leave castascall rather than having more than one method call per contextmanager | 14:58 |
| gibi | bauzas: sure | 14:58 |
| gibi | bauzas: I mean sure, lets look at my comment | 14:58 |
| bauzas | just because I think it's more understanding why we need to mock | 14:59 |
| bauzas | gibi: okay, so you would still be OK if I don't modify it ? | 14:59 |
| *** LiangFang has quit IRC | 15:00 | |
| gibi | bauzas: unfortunately having only the post under the mock is a lie as such post starts an async task and the post returns before the function you are mocking really runs | 15:00 |
| *** rcernin has quit IRC | 15:01 | |
| gibi | in a unit test I'm ok with castascall but not in a functional test | 15:01 |
| gibi | sorry | 15:01 |
| efried | stephenfin: dahh, I guess we do have access to the old image meta, don't we? So yeah, sean-k-mooney is right, I can add a check for that.Do you want the spec updated now or fup'd? | 15:01 |
| gibi | in a functional test we want to express (and test) the async behavior of our API we are calling | 15:01 |
| sean-k-mooney | efried: i think we can just do it in the implemnation | 15:02 |
| sean-k-mooney | i dont think we need a spec update but a fup would be fine if you want | 15:02 |
| bauzas | gibi: mmmm not sure I understand you, sorry :( | 15:02 |
| *** tbachman has joined #openstack-nova | 15:02 | |
| stephenfin | efried: If you can get it today, I'm happy to fast approve and carry gibi's +2 | 15:02 |
| efried | really in that spot I was just trying to give examples of how things could go wrong. I got one of the wrong things wrong, but it's the *thought* that counts :P | 15:03 |
| efried | stephenfin: Roger that, will do it right now... | 15:03 |
| bauzas | gibi: ah-ha I think I understand your concern | 15:03 |
| gibi | bauzas: in my eyes the nova internal in functional test shoudl behave exactly as it behaves in real life (except virt driver as it is hard, and except external dependencies like neutron) so I would like to keep the nova API async behaviour in func test | 15:03 |
| bauzas | gibi: honestly given we call directly by castascall, it's no longer async | 15:03 |
| bauzas | (and that's also why it works | 15:04 |
| gibi | bauzas: yes, and I'm against such behavior change in func test | 15:04 |
| bauzas | gibi: okay, fair enough :-) | 15:04 |
| bauzas | gibi: no worries, I'll then fix this | 15:05 |
| gibi | bauzas: we tend to allow to use some mocks in func test to inject faults as that is hard without the API but other than that I'm also against mocks in func test too | 15:05 |
| gibi | s/without the API/ with only the API/ | 15:06 |
| *** bhagyashris has quit IRC | 15:06 | |
| gibi | bauzas: thanks | 15:06 |
| *** dave-mccowan has joined #openstack-nova | 15:06 | |
| *** priteau has joined #openstack-nova | 15:06 | |
| stephenfin | efried: Sweet. Ta. When you're done, I have done as commanded on https://review.opendev.org/700511 and it's just _itching_ to get through that gate | 15:12 |
| *** bhagyashris has joined #openstack-nova | 15:12 | |
| efried | stephenfin: firewall pt. 3 +A | 15:15 |
| efried | stephenfin: regarding the ImageMetaProps updates... I'm not sure. | 15:16 |
| *** jmlowe has quit IRC | 15:17 | |
| *** liuyulong has quit IRC | 15:18 | |
| *** liuyulong_ has quit IRC | 15:18 | |
| efried | I always thought of image meta props the same as flavor extra specs -- a bag of random key/values that we can "enhance" without microversion changes -- except for some reason (that I never understood, never asked about) we actually have OVOs for image meta props, which need to be updated. | 15:18 |
| *** ociuhandu has joined #openstack-nova | 15:18 | |
| efried | but like extra specs, I would have expected them to show up in a REST payload like the others | 15:19 |
| efried | and be settable like anything else | 15:20 |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Spec: Ussuri: Encrypted Emulated Virtual TPM https://review.opendev.org/686804 | 15:21 |
| sean-k-mooney | efried: yes the can be extended without microverion changes | 15:21 |
| efried | stephenfin, gibi, sean-k-mooney: with just that update about the API check --^ | 15:22 |
| sean-k-mooney | but the image metadata object is an ovo | 15:22 |
| sean-k-mooney | so it will alter the object version | 15:22 |
| efried | right; stephenfin is asking whether the new fields will be visible/settable via the API, and I don't see why not? | 15:22 |
| * efried caffeinates | 15:23 | |
| sean-k-mooney | we do not allow arbiary keys to be set in the image metadata | 15:23 |
| sean-k-mooney | but it should be visable yes | 15:23 |
| sean-k-mooney | we dont allow you to downgrade the ovo | 15:23 |
| sean-k-mooney | to strip out new keys and get an older version | 15:23 |
| *** ociuhandu has quit IRC | 15:23 | |
| efried | by "arbitrary" you mean keys that we haven't explicitly registered, which is why we register them. But anything we've registered, you can set | 15:24 |
| sean-k-mooney | yes | 15:24 |
| sean-k-mooney | although this is set in glance not nova obviously | 15:24 |
| *** artom has quit IRC | 15:26 | |
| efried | right, and then we pull in a copy of the meta and store it with the instance record, which is why this object exists in nova in the first place. | 15:28 |
| efried | not sure why we needed it for image meta but not flavor extra specs tho | 15:28 |
| *** eharney has joined #openstack-nova | 15:28 | |
| sean-k-mooney | yes more or less | 15:28 |
| efried | or rather, why we couldn't leave it out for image meta like we left it out for extra specs. | 15:28 |
| sean-k-mooney | we did not have it for the image metadata orgithily | 15:28 |
| sean-k-mooney | it was just a dict of strings | 15:29 |
| dansmith | efried: you can use flavor extra specs for other things though right? | 15:29 |
| dansmith | and schedule on them with things like the json filter? | 15:29 |
| sean-k-mooney | but we stadarised it wehn we made the nova image an objec t in juno i think | 15:29 |
| sean-k-mooney | dansmith: you used to be able to do the same with the image metadata | 15:29 |
| dansmith | I think the problem with image props came because people were using them as a covert conduit between the user (which can set them) and the virt driver and bypassing nova | 15:29 |
| efried | I'm not sure I understand a distinction dansmith. We've been using image meta props for scheduling decisions as long as I've been involved | 15:29 |
| efried | okay, right, I was going to ask whether it had something to do with admin/flavor vs user/image. | 15:30 |
| dansmith | efried: also, the problem is that image properties live a long time and we had a huge problem with virt drivers using different props for the same thing.. flavor extra specs are admin-only and we can force people to fix them during an upgrade | 15:30 |
| efried | Okay. If there's a reason, so be it. I don't super care to challenge the status quo, since it's easy to add properties. Really just curious why they were different. | 15:31 |
| efried | but back to stephenfin's question, which seems odd to me as worded | 15:32 |
| sean-k-mooney | well the topic of makeing the extra specs versioned like the image meta has come up before | 15:32 |
| dansmith | I think the "images live forever and are controlled by users" is the reason | 15:32 |
| sean-k-mooney | but then we dont have a place to do the current non standdard thngs | 15:32 |
| *** xek has quit IRC | 15:32 | |
| efried | Right, we've always punted on it because of the nonstandard things, things we don't know about etc. | 15:33 |
| efried | which is one of the challenges stephenfin is facing with the validation effort. | 15:33 |
| *** xek has joined #openstack-nova | 15:33 | |
| dansmith | yeah, it's also why I have a hard time getting excited about that | 15:33 |
| *** mkrai has joined #openstack-nova | 15:34 | |
| sean-k-mooney | ya so we could always add a seperate sandardsed metadata filed that was versioned and leve extra specs for the unversioned stuff but its never really been urgent enough | 15:34 |
| dansmith | I know we need to do it, | 15:34 |
| dansmith | but it's always been a big bag of stuff | 15:34 |
| *** TxGirlGeek has joined #openstack-nova | 15:34 | |
| *** amodi has quit IRC | 15:35 | |
| *** jmlowe has joined #openstack-nova | 15:38 | |
| *** udesale has joined #openstack-nova | 15:39 | |
| *** ociuhandu has joined #openstack-nova | 15:42 | |
| *** ociuhandu has quit IRC | 15:50 | |
| *** ociuhandu has joined #openstack-nova | 15:51 | |
| *** dtantsur is now known as dtantsur|bbl | 15:55 | |
| *** portdirect has joined #openstack-nova | 15:55 | |
| *** ociuhandu has quit IRC | 15:58 | |
| *** artom has joined #openstack-nova | 16:01 | |
| efried | gibi: Could I ask you to review this please https://review.opendev.org/#/c/699050/ ? It bumps us to using placement 1.35, sets a request_level_params next to requested_resources in the RequestSpec, and lets us unwind the request filter hacks where we were transiently modifying the flavor extra specs. | 16:04 |
| efried | ...and Dan is +2. | 16:04 |
| *** mlavalle has joined #openstack-nova | 16:05 | |
| *** hoonetorg has quit IRC | 16:07 | |
| sean-k-mooney | bauzas: https://review.opendev.org/#/c/700529/ | 16:11 |
| bauzas | sean-k-mooney: that's a different bug but okay | 16:11 |
| sean-k-mooney | yes its a similar cause the _copy_resources function does create a copy and instead modifyes the stats | 16:12 |
| openstackgerrit | Victor Morales proposed openstack/nova master: Update Testing NUMA documentation https://review.opendev.org/701301 | 16:12 |
| *** hoonetorg has joined #openstack-nova | 16:20 | |
| *** N3l1x has joined #openstack-nova | 16:23 | |
| *** N3l1x_ has joined #openstack-nova | 16:24 | |
| *** N3l1x_ has quit IRC | 16:24 | |
| *** N3l1x has quit IRC | 16:29 | |
| *** openstackgerrit has quit IRC | 16:30 | |
| *** ivve has quit IRC | 16:35 | |
| *** bhagyashris has quit IRC | 16:35 | |
| *** N3l1x has joined #openstack-nova | 16:37 | |
| stephenfin | I'm seeing a lot of failures in the nova-next job, all due to 'Multiple possible networks found, use a Network ID to be more specific.' | 16:43 |
| stephenfin | I feel we have a bug open against this. Anyone know of one? | 16:43 |
| artom | stephenfin, it does ring a bell... | 16:44 |
| *** gyee has joined #openstack-nova | 16:45 | |
| *** bhagyashris has joined #openstack-nova | 16:45 | |
| *** jmlowe has quit IRC | 16:50 | |
| *** jmlowe has joined #openstack-nova | 16:50 | |
| efried | stephenfin: Yeah, this cropped up again recently and was fixed again. I could dig, but probably asking in -neutron would be the fastest way to get to the bottom. | 16:50 |
| *** damien_r has quit IRC | 16:52 | |
| stephenfin | efried: Yeah, I'll do that | 16:53 |
| *** openstackgerrit has joined #openstack-nova | 16:56 | |
| openstackgerrit | Merged openstack/nova master: Update Testing NUMA documentation https://review.opendev.org/701301 | 16:56 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add a placement audit command https://review.opendev.org/670112 | 17:03 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Avoid PlacementFixture silently swallowing kwargs https://review.opendev.org/695180 | 17:03 |
| *** ociuhandu has joined #openstack-nova | 17:03 | |
| *** udesale has quit IRC | 17:05 | |
| *** artom has quit IRC | 17:06 | |
| *** bhagyashris has quit IRC | 17:06 | |
| *** ociuhandu has quit IRC | 17:08 | |
| *** tbachman has quit IRC | 17:12 | |
| *** tbachman_ has joined #openstack-nova | 17:12 | |
| *** psachin has quit IRC | 17:12 | |
| *** mkrai has quit IRC | 17:15 | |
| gibi | efried: added https://review.opendev.org/#/c/699050/ to my queue | 17:21 |
| gibi | efried: I will look at it tomorrow morning | 17:22 |
| efried | thanks gibi! | 17:22 |
| *** ociuhandu has joined #openstack-nova | 17:26 | |
| *** ociuhandu has quit IRC | 17:27 | |
| *** ociuhandu has joined #openstack-nova | 17:27 | |
| mriedem | efried: since i reviewed that before, i've +2ed the latest since it lgtm, but will let someone else approve | 17:28 |
| *** mkrai has joined #openstack-nova | 17:29 | |
| efried | mriedem: okay, cool, thank you. | 17:29 |
| *** tbarron_ is now known as tbarron | 17:30 | |
| *** ociuhandu has quit IRC | 17:31 | |
| efried | stephenfin: in case you missed it, vTPM spec is updated as discussed earlier. But I was curious about your comment about viewing/setting the image meta props. | 17:32 |
| *** evrardjp has quit IRC | 17:33 | |
| *** evrardjp has joined #openstack-nova | 17:34 | |
| *** ociuhandu has joined #openstack-nova | 17:34 | |
| *** mlavalle has quit IRC | 17:38 | |
| *** jawad_axd has joined #openstack-nova | 17:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: define boot from volume in the glossary https://review.opendev.org/699009 | 17:41 |
| *** ociuhandu has quit IRC | 17:41 | |
| *** mkrai has quit IRC | 17:42 | |
| *** mriedem has quit IRC | 17:44 | |
| *** jawad_axd has quit IRC | 17:44 | |
| *** mriedem has joined #openstack-nova | 17:44 | |
| *** ociuhandu has joined #openstack-nova | 17:44 | |
| *** ociuhandu has quit IRC | 17:49 | |
| *** rpittau is now known as rpittau|afk | 17:51 | |
| *** factor has joined #openstack-nova | 17:53 | |
| *** tosky has quit IRC | 17:54 | |
| *** ivve has joined #openstack-nova | 17:56 | |
| *** derekh has quit IRC | 17:57 | |
| *** artom has joined #openstack-nova | 17:58 | |
| *** dpawlik has quit IRC | 17:58 | |
| *** mlavalle has joined #openstack-nova | 17:59 | |
| *** jawad_axd has joined #openstack-nova | 18:00 | |
| *** priteau has quit IRC | 18:01 | |
| *** jawad_axd has quit IRC | 18:05 | |
| *** maciejjozefczyk has quit IRC | 18:11 | |
| *** ociuhandu has joined #openstack-nova | 18:13 | |
| *** dtantsur|bbl is now known as dtantsur | 18:13 | |
| *** tesseract has quit IRC | 18:15 | |
| *** ociuhandu has quit IRC | 18:17 | |
| *** mriedem has quit IRC | 18:22 | |
| *** mriedem has joined #openstack-nova | 18:23 | |
| openstackgerrit | Dustin Cowles proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 18:23 |
| *** mriedem has quit IRC | 18:26 | |
| *** mriedem has joined #openstack-nova | 18:26 | |
| *** pcaruana has quit IRC | 18:27 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078 | 18:29 |
| dansmith | efried: ^this is left over from a couple PTGs ago. You were +2 on it and I just tweaked a few things you and mriedem pointed out | 18:29 |
| *** dtantsur is now known as dtantsur|afk | 18:34 | |
| *** tbachman_ is now known as tbachman | 18:37 | |
| *** martinkennelly has quit IRC | 18:38 | |
| efried | ... | 18:38 |
| *** nweinber_ has joined #openstack-nova | 18:38 | |
| *** jmlowe has quit IRC | 18:39 | |
| *** nweinber__ has quit IRC | 18:41 | |
| efried | dansmith: pretty much ready to +2 that; if you could just respond to those couple of comments please. | 18:56 |
| efried | And would prefer at least the backtick change be made. I'm here to re-look quickly. | 18:56 |
| dansmith | efried: I'll fix the backticks, section naming and s/ceph/rbd. | 18:59 |
| efried | cool | 18:59 |
| *** jmlowe has joined #openstack-nova | 19:01 | |
| efried | dansmith: I was referring to the suggestion of adding compute_driver=libvirt as a related option. | 19:01 |
| dansmith | oh, missed that too I guess | 19:02 |
| dansmith | I had made a few of these changes right when the feedback came in like a year ago | 19:02 |
| *** jawad_axd has joined #openstack-nova | 19:02 | |
| efried | I don't consider that blocking, just .. yeah, wanted to make sure you had at least seen it | 19:02 |
| dansmith | and then went back to finish just now, so.. | 19:02 |
| efried | nbd | 19:02 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078 | 19:04 |
| dansmith | efried: ^ | 19:04 |
| *** maciejjozefczyk has joined #openstack-nova | 19:07 | |
| *** jawad_axd has quit IRC | 19:07 | |
| efried | dansmith: missed the rename in the reno; and while you're respinning, a quick change to the conf help. | 19:07 |
| dansmith | oh doh | 19:08 |
| dansmith | lol | 19:08 |
| dansmith | the backticks thing I looked a line below to see if the went in or out and then didn't do what I saw | 19:09 |
| efried | heh | 19:09 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078 | 19:09 |
| dansmith | how about this one? JEEZ | 19:09 |
| efried | this is why we recommend avoiding the hard liquor during work hours. | 19:10 |
| efried | Fortified wine max. | 19:10 |
| efried | +2 | 19:10 |
| efried | I really should have racked up some -1s for stackalytics there. | 19:10 |
| dansmith | heh | 19:11 |
| *** nweinber__ has joined #openstack-nova | 19:14 | |
| *** nweinber_ has quit IRC | 19:16 | |
| *** maciejjozefczyk has quit IRC | 19:20 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: move os-vif-ovs to be a non legacy job. https://review.opendev.org/701601 | 19:42 |
| *** bnemec has quit IRC | 19:47 | |
| *** bnemec has joined #openstack-nova | 19:52 | |
| openstackgerrit | sean mooney proposed openstack/os-vif master: optimise tempest testing https://review.opendev.org/701607 | 19:58 |
| *** eharney has quit IRC | 20:01 | |
| *** damien_r has joined #openstack-nova | 20:08 | |
| *** eharney has joined #openstack-nova | 20:15 | |
| mriedem | since i got a ping, i left comments on https://review.opendev.org/#/c/657078/ | 20:21 |
| openstackgerrit | Victor Coutellier proposed openstack/nova master: Non-admin user can filter their instances by AZs https://review.opendev.org/701609 | 20:29 |
| openstackgerrit | Victor Coutellier proposed openstack/nova master: Non-admin user can filter their instances by AZs https://review.opendev.org/701609 | 20:30 |
| *** openstackgerrit has quit IRC | 20:31 | |
| *** eharney has quit IRC | 20:38 | |
| *** openstackgerrit has joined #openstack-nova | 20:43 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078 | 20:43 |
| *** amodi has joined #openstack-nova | 20:53 | |
| *** nweinber__ has quit IRC | 20:55 | |
| *** mgariepy has quit IRC | 20:57 | |
| efried | dansmith: Remind me where it's written that microversion requires spec? I thought http://specs.openstack.org/openstack/nova-specs/readme.html but no | 20:57 |
| dansmith | uh, on the walls? I dunno | 20:58 |
| sean-k-mooney | efried: i think it was more a micorversion bump is an api change and all api changes need a spec | 21:05 |
| efried | sean-k-mooney: okay, but I thought we had that documented somewhere. | 21:06 |
| mriedem | it's in the template | 21:07 |
| mriedem | spec template | 21:07 |
| sean-k-mooney | ya i thought that was in the spec repo readme but its not | 21:07 |
| efried | aha, https://docs.openstack.org/nova/latest/contributor/process.html#when-do-i-need-a-blueprint-vs-a-spec | 21:07 |
| mriedem | and there is a doc on when new microversions are needed | 21:07 |
| sean-k-mooney | oh the template | 21:07 |
| efried | I like how it says you should ask the PTL. Like I would know. | 21:08 |
| mriedem | also https://docs.openstack.org/nova/latest/contributor/blueprints.html | 21:08 |
| mriedem | right at the top(ish) | 21:08 |
| efried | thanks | 21:09 |
| dansmith | efried: it's in the template | 21:10 |
| *** ociuhandu has joined #openstack-nova | 21:10 | |
| efried | cool | 21:10 |
| efried | fyi I was working on this https://review.opendev.org/#/c/701609/ | 21:12 |
| sean-k-mooney | ya filtering on az makes sense since the user can see what az the vm is in in a few ways | 21:13 |
| efried | dansmith: I marked your bug Wishlist. Should I make it Triaged, Confirmed, Won't Fix, ...? (<== mriedem) | 21:15 |
| dansmith | well, we will fix it by implementing a feature at some point, which is whyI don't really think it's a bug | 21:16 |
| dansmith | wishlist and confirmed is fine with me | 21:16 |
| dansmith | as is closed, rejected, etc :) | 21:16 |
| efried | ack, done | 21:16 |
| efried | I guess I would rather close it somehow so it doesn't clutter our bug numbers, but meh. | 21:17 |
| dansmith | well, this patch doesn't fix it, so don't close it because of that.. but yeah, that's why I didn't want to open in | 21:19 |
| efried | understood | 21:19 |
| dansmith | you could reject it as "not a bug" and it would still fulfill the purpose of providing a link in the workaround config | 21:19 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate base rules in favor of new rules https://review.opendev.org/701624 | 21:25 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682 | 21:29 |
| *** ralonsoh has quit IRC | 21:30 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095 | 21:30 |
| *** spsurya has quit IRC | 21:32 | |
| *** ociuhandu has quit IRC | 21:39 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_password policies https://review.opendev.org/701629 | 21:41 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_password policies https://review.opendev.org/701629 | 21:43 |
| *** tbachman has quit IRC | 21:49 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-admin-password https://review.opendev.org/701630 | 21:52 |
| *** damien_r has quit IRC | 22:03 | |
| *** tbachman has joined #openstack-nova | 22:04 | |
| *** jawad_axd has joined #openstack-nova | 22:06 | |
| *** eharney has joined #openstack-nova | 22:08 | |
| *** jawad_axd has quit IRC | 22:10 | |
| *** mriedem has quit IRC | 22:11 | |
| *** slaweq has quit IRC | 22:15 | |
| openstackgerrit | Victor Coutellier proposed openstack/nova master: Non-admin user can filter their instances by AZs https://review.opendev.org/701609 | 22:24 |
| *** rchurch has joined #openstack-nova | 22:27 | |
| *** N3l1x has quit IRC | 22:30 | |
| *** tbachman has quit IRC | 22:32 | |
| *** rchurch has quit IRC | 22:33 | |
| *** rchurch has joined #openstack-nova | 22:33 | |
| *** tkajinam has joined #openstack-nova | 22:56 | |
| *** tkajinam has quit IRC | 22:58 | |
| *** tkajinam has joined #openstack-nova | 23:00 | |
| openstackgerrit | melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478 | 23:05 |
| openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | 23:05 |
| *** jawad_axd has joined #openstack-nova | 23:08 | |
| *** slaweq has joined #openstack-nova | 23:11 | |
| *** jawad_axd has quit IRC | 23:12 | |
| *** dviroel has quit IRC | 23:12 | |
| *** rcernin has joined #openstack-nova | 23:13 | |
| *** mlavalle has quit IRC | 23:14 | |
| *** slaweq has quit IRC | 23:16 | |
| *** eharney has quit IRC | 23:19 | |
| *** jawad_axd has joined #openstack-nova | 23:29 | |
| *** shilpasd has quit IRC | 23:32 | |
| *** jawad_axd has quit IRC | 23:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-admin-password policies https://review.opendev.org/701639 | 23:49 |
| *** tetsuro_ has joined #openstack-nova | 23:50 | |
| *** tetsuro has quit IRC | 23:52 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!