| *** tosky has quit IRC | 00:06 | |
| *** lemko4 has joined #openstack-nova | 00:20 | |
| *** lemko has quit IRC | 00:21 | |
| *** lemko4 is now known as lemko | 00:21 | |
| *** tbachman has quit IRC | 00:28 | |
| *** tbachman has joined #openstack-nova | 00:36 | |
| *** k_mouza has joined #openstack-nova | 00:45 | |
| *** k_mouza has quit IRC | 00:49 | |
| *** macz_ has quit IRC | 00:56 | |
| *** zzzeek has quit IRC | 01:06 | |
| *** zzzeek has joined #openstack-nova | 01:09 | |
| *** mlavalle has quit IRC | 01:10 | |
| *** iurygregory has quit IRC | 01:32 | |
| *** sapd1_x has joined #openstack-nova | 01:46 | |
| *** sapd1_x has quit IRC | 01:51 | |
| *** sapd1_x has joined #openstack-nova | 01:58 | |
| *** sapd1_x has quit IRC | 02:02 | |
| *** sapd1_x has joined #openstack-nova | 02:06 | |
| *** sapd1_x has quit IRC | 02:15 | |
| *** yingjisun has joined #openstack-nova | 02:24 | |
| *** sapd1_x has joined #openstack-nova | 02:37 | |
| *** sapd1_x has quit IRC | 02:41 | |
| *** rcernin has quit IRC | 02:44 | |
| *** artom has quit IRC | 02:55 | |
| *** xinranwang has joined #openstack-nova | 03:00 | |
| xinranwang | gibi: Hi gibi, could you please review the smartnic spec when you got time? ;) | 03:00 |
|---|---|---|
| *** rcernin has joined #openstack-nova | 03:06 | |
| *** rcernin has quit IRC | 03:13 | |
| *** sapd1_x has joined #openstack-nova | 03:14 | |
| *** sapd1_x has quit IRC | 03:14 | |
| *** mkrai has joined #openstack-nova | 03:16 | |
| *** sapd1 has joined #openstack-nova | 03:17 | |
| *** adrianc has quit IRC | 03:22 | |
| *** adrianc has joined #openstack-nova | 03:38 | |
| *** rcernin has joined #openstack-nova | 03:43 | |
| *** adrianc has quit IRC | 03:47 | |
| *** rcernin has quit IRC | 03:48 | |
| *** vishalmanchanda has joined #openstack-nova | 03:52 | |
| *** rcernin has joined #openstack-nova | 03:53 | |
| *** rcernin has quit IRC | 03:54 | |
| *** rcernin has joined #openstack-nova | 03:54 | |
| *** donnyd has quit IRC | 04:07 | |
| *** donnyd has joined #openstack-nova | 04:08 | |
| *** adrianc has joined #openstack-nova | 04:17 | |
| *** adrianc has quit IRC | 04:23 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | 04:26 |
| *** ratailor has joined #openstack-nova | 04:29 | |
| *** k_mouza has joined #openstack-nova | 04:46 | |
| *** johnsom has quit IRC | 04:46 | |
| *** johnsom has joined #openstack-nova | 04:49 | |
| *** k_mouza has quit IRC | 04:50 | |
| *** rcernin has quit IRC | 04:56 | |
| *** rcernin has joined #openstack-nova | 04:57 | |
| *** sapd1 has quit IRC | 05:06 | |
| *** fyx has quit IRC | 05:11 | |
| *** fyx has joined #openstack-nova | 05:14 | |
| *** johnsom has quit IRC | 05:18 | |
| *** johnsom has joined #openstack-nova | 05:19 | |
| *** zzzeek has quit IRC | 05:25 | |
| *** zzzeek has joined #openstack-nova | 05:27 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-nova | 05:33 | |
| *** zzzeek has quit IRC | 05:50 | |
| *** yingjisun has quit IRC | 05:51 | |
| *** zzzeek has joined #openstack-nova | 05:51 | |
| *** sapd1 has joined #openstack-nova | 05:52 | |
| *** JamesBenson has quit IRC | 05:53 | |
| *** zzzeek has quit IRC | 06:10 | |
| *** zzzeek has joined #openstack-nova | 06:11 | |
| *** flaviof has quit IRC | 06:11 | |
| *** flaviof has joined #openstack-nova | 06:12 | |
| *** johnsom has quit IRC | 06:27 | |
| *** johnsom has joined #openstack-nova | 06:27 | |
| *** yingjisun has joined #openstack-nova | 06:52 | |
| *** rcernin has quit IRC | 07:10 | |
| *** rcernin has joined #openstack-nova | 07:13 | |
| *** zzzeek has quit IRC | 07:14 | |
| *** adrianc has joined #openstack-nova | 07:15 | |
| *** zzzeek has joined #openstack-nova | 07:16 | |
| *** rm_work has quit IRC | 07:26 | |
| *** rm_work has joined #openstack-nova | 07:28 | |
| *** zzzeek has quit IRC | 07:37 | |
| *** zzzeek has joined #openstack-nova | 07:38 | |
| *** ociuhandu has joined #openstack-nova | 07:40 | |
| *** ociuhandu has quit IRC | 07:42 | |
| *** ociuhandu has joined #openstack-nova | 07:43 | |
| *** ociuhandu has quit IRC | 07:43 | |
| *** ralonsoh has joined #openstack-nova | 07:43 | |
| *** rcernin has quit IRC | 07:44 | |
| *** rcernin has joined #openstack-nova | 07:47 | |
| *** macz_ has joined #openstack-nova | 07:57 | |
| gibi | xinranwang: when I got time I will review :) | 07:58 |
| *** slaweq has joined #openstack-nova | 07:58 | |
| gibi | I mean I'm trying to get to it | 07:58 |
| *** sapd1 has quit IRC | 07:58 | |
| *** macz_ has quit IRC | 08:02 | |
| *** rpittau|afk is now known as rpittau | 08:05 | |
| *** dklyle has quit IRC | 08:06 | |
| *** sapd1 has joined #openstack-nova | 08:08 | |
| *** tesseract has joined #openstack-nova | 08:14 | |
| *** tobiash has quit IRC | 08:15 | |
| *** tobiash has joined #openstack-nova | 08:16 | |
| *** andrewbonney has joined #openstack-nova | 08:17 | |
| *** rcernin has quit IRC | 08:20 | |
| *** ociuhandu has joined #openstack-nova | 08:22 | |
| *** zzzeek has quit IRC | 08:26 | |
| xinranwang | gibi: thanks~ | 08:29 |
| *** zzzeek has joined #openstack-nova | 08:30 | |
| *** rcernin has joined #openstack-nova | 08:33 | |
| *** ociuhandu has quit IRC | 08:38 | |
| *** zzzeek has quit IRC | 08:38 | |
| *** zzzeek has joined #openstack-nova | 08:40 | |
| *** rcernin has quit IRC | 08:40 | |
| *** zzzeek has quit IRC | 08:48 | |
| *** zzzeek has joined #openstack-nova | 08:51 | |
| *** mgoddard has joined #openstack-nova | 08:52 | |
| *** ociuhandu has joined #openstack-nova | 08:52 | |
| *** iurygregory has joined #openstack-nova | 08:52 | |
| *** tosky has joined #openstack-nova | 08:56 | |
| *** k_mouza has joined #openstack-nova | 09:00 | |
| *** dtantsur|afk is now known as dtantsur | 09:04 | |
| *** k_mouza has quit IRC | 09:05 | |
| bauzas | good spec review day, Nova | 09:08 |
| *** links has joined #openstack-nova | 09:15 | |
| *** links has quit IRC | 09:22 | |
| *** martinkennelly has joined #openstack-nova | 09:25 | |
| *** sean-k-mooney has quit IRC | 09:27 | |
| *** sean-k-mooney has joined #openstack-nova | 09:27 | |
| *** mgoddard has quit IRC | 09:29 | |
| *** rcernin has joined #openstack-nova | 09:32 | |
| *** xek has joined #openstack-nova | 09:36 | |
| *** rcernin has quit IRC | 09:40 | |
| *** ociuhandu has quit IRC | 09:42 | |
| *** ociuhandu has joined #openstack-nova | 09:51 | |
| *** ociuhandu has quit IRC | 09:56 | |
| *** zzzeek has quit IRC | 09:57 | |
| *** ociuhandu has joined #openstack-nova | 09:58 | |
| bauzas | gibi: easy peasy to review for a spec review day https://review.opendev.org/#/c/756242/ | 10:00 |
| *** zzzeek has joined #openstack-nova | 10:02 | |
| *** rcernin has joined #openstack-nova | 10:04 | |
| *** rcernin has quit IRC | 10:09 | |
| *** jamesdenton has quit IRC | 10:10 | |
| *** jamesdenton has joined #openstack-nova | 10:10 | |
| *** whoami-rajat__ has joined #openstack-nova | 10:12 | |
| *** songwenping_ has joined #openstack-nova | 10:14 | |
| *** yingjisun has left #openstack-nova | 10:15 | |
| *** artom has joined #openstack-nova | 10:19 | |
| *** mgoddard has joined #openstack-nova | 10:19 | |
| *** zzzeek has quit IRC | 10:38 | |
| *** zzzeek has joined #openstack-nova | 10:40 | |
| gibi | bauzas: done | 10:43 |
| *** k_mouza has joined #openstack-nova | 10:51 | |
| stephenfin | gibi, bauzas: I know it's spec review day, but could take a real quick look at https://review.opendev.org/#/c/762898/ while it's fresh in my mind? | 10:54 |
| bauzas | gibi: cool, thanks | 10:54 |
| bauzas | stephenfin: ack, I clicked for it, I'm just looking at a spec now | 10:54 |
| stephenfin | all good, thanks :) | 10:55 |
| openstackgerrit | Merged openstack/nova-specs master: Re-proposes Routed Networks https://review.opendev.org/756242 | 10:58 |
| *** rcernin has joined #openstack-nova | 10:58 | |
| *** ociuhandu has quit IRC | 11:07 | |
| *** ociuhandu has joined #openstack-nova | 11:07 | |
| gibi | stephenfin: So the intention of this exception is to fail the startup of the compute service this why it is never caught? | 11:09 |
| stephenfin | gibi: Yes, exactly. It's presence indicates a misconfiguration meaning we should hard fail | 11:10 |
| stephenfin | tbc, we will only attempt to retrieve this information on the host if nova.conf tells us to | 11:13 |
| *** zzzeek has quit IRC | 11:17 | |
| *** zzzeek has joined #openstack-nova | 11:20 | |
| *** zzzeek has quit IRC | 11:27 | |
| *** zzzeek has joined #openstack-nova | 11:28 | |
| *** mkrai has quit IRC | 11:29 | |
| *** macz_ has joined #openstack-nova | 11:33 | |
| *** ociuhandu has quit IRC | 11:35 | |
| *** ociuhandu has joined #openstack-nova | 11:36 | |
| gibi | stephenfin: thanks. Then my only question is about underscores :) https://review.opendev.org/#/c/762898/1/nova/tests/unit/virt/libvirt/test_driver.py@27505 | 11:38 |
| sean-k-mooney | gibi: just reading the comment | 11:38 |
| stephenfin | oh, there shouldn't be three /o\ | 11:38 |
| sean-k-mooney | gibi: that is the style that stephenfin uses | 11:38 |
| *** macz_ has quit IRC | 11:38 | |
| gibi | sean-k-mooney: I know he likes underscores, but I think there is one extra now | 11:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add missing exception https://review.opendev.org/762898 | 11:39 |
| sean-k-mooney | yep | 11:39 |
| stephenfin | done; my mistake /o\ | 11:39 |
| sean-k-mooney | test___ instead of test__ | 11:39 |
| sean-k-mooney | im still fine with this | 11:40 |
| gibi | stephenfin: now you have one underscore, which is fine by me, but I guess your style would cause two underscores | 11:40 |
| stephenfin | it would, but I know not everyone likes that so I'm okay to stick with one | 11:40 |
| *** ociuhandu has quit IRC | 11:41 | |
| *** sapd1 has quit IRC | 11:41 | |
| gibi | ok | 11:41 |
| bauzas | sean-k-mooney: FWIW, I think I drafted a good nova-cyborg relationship in https://review.opendev.org/#/c/750116/9/specs/wallaby/approved/support-vGPU-nova-cyborg-interaction.rst@183 | 11:41 |
| bauzas | based on the open issues we have atm | 11:42 |
| bauzas | with the mdev persistence and the likes | 11:42 |
| bauzas | this would hugely benefit to Cyboth | 11:42 |
| bauzas | cyborg* | 11:42 |
| bauzas | sean-k-mooney: anyway, lunch | 11:42 |
| gibi | bauzas: does it have an impact on https://review.opendev.org/#/c/742785/6..9/specs/wallaby/approved/support-sriov-smartnic.rst too? | 11:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: api-ref: Move 'os-agents' API to obsolete section https://review.opendev.org/755729 | 11:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: virt: Remove 'change_instance_metadata' API https://review.opendev.org/749316 | 11:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: virt: Remove 'reset_network' API https://review.opendev.org/749315 | 11:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: virt: Remove 'get_all_bw_counters' API https://review.opendev.org/749312 | 11:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove 'BandwidthUsage', 'BandwidthUsageList' https://review.opendev.org/759114 | 11:42 |
| bauzas | gibi: I haven't reviewed this spec yet | 11:42 |
| gibi | bauzas: nvm, I will read your comment in the vGPU spec | 11:43 |
| bauzas | gibi: but tl;dr: I propose to leave cyborg manage the mdevs and just ask nova to bind them to the guest definition | 11:43 |
| bauzas | which is sometimes we already have | 11:43 |
| bauzas | s/sometimes/something | 11:43 |
| sean-k-mooney | bauzas: ill go read it shortly | 11:43 |
| bauzas | gibi: sean-k-mooney: the only difference would be the inventory reporting | 11:44 |
| bauzas | but this way, this would allow cyborg to keep a persisted state of mdevs, which is something I don't wanna managed in nova and which creates problems for us | 11:44 |
| *** rcernin has quit IRC | 11:45 | |
| bauzas | we haven't discussed this at the PTG, I reckon, but I feel we would all benefit of this solution | 11:45 |
| *** tbachman has quit IRC | 11:45 | |
| bauzas | like, my customers would have the choice to either manage the mdev fleet by themselves and just use nova, or play with cyborg | 11:45 |
| sean-k-mooney | bauzas im not sure they __need__ to precreate the mdev but i agree they could and it would ok to do so | 11:46 |
| bauzas | sean-k-mooney: precreating is way better for many reasons | 11:46 |
| bauzas | I mean, precreating after a config modification, that's it | 11:46 |
| sean-k-mooney | i just read you last comment i need to read the rest of the doc and your comments for context | 11:46 |
| bauzas | because once you're done with the config, your inventory won't change | 11:47 |
| sean-k-mooney | i think it was on v2 the last time i looked | 11:47 |
| bauzas | either way, I need to go lunching | 11:47 |
| bauzas | my kids need to go to school | 11:47 |
| sean-k-mooney | ya cyborg might want to be more dynmaic but i agree it has advantages | 11:47 |
| sean-k-mooney | simplcity being one of them | 11:47 |
| *** brinzhang0 has joined #openstack-nova | 11:50 | |
| *** ociuhandu has joined #openstack-nova | 11:53 | |
| *** brinzhang_ has quit IRC | 11:53 | |
| *** brinzhang_ has joined #openstack-nova | 11:55 | |
| *** songwenping__ has joined #openstack-nova | 11:55 | |
| *** spatel has joined #openstack-nova | 11:56 | |
| *** slaweq has quit IRC | 11:56 | |
| *** slaweq has joined #openstack-nova | 11:57 | |
| *** brinzhang0 has quit IRC | 11:58 | |
| *** rcernin has joined #openstack-nova | 11:58 | |
| *** songwenping_ has quit IRC | 11:59 | |
| *** spatel has quit IRC | 12:01 | |
| *** mgoddard has quit IRC | 12:01 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | 12:08 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | 12:08 |
| *** xinranwang has quit IRC | 12:09 | |
| *** JamesBenson has joined #openstack-nova | 12:12 | |
| *** raildo_ has joined #openstack-nova | 12:14 | |
| *** ratailor has quit IRC | 12:14 | |
| *** rcernin has quit IRC | 12:14 | |
| *** mgoddard has joined #openstack-nova | 12:15 | |
| *** raildo has quit IRC | 12:16 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | 12:20 |
| *** lpetrut has joined #openstack-nova | 12:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add modernize-os-hypervisors-api spec https://review.opendev.org/755109 | 12:28 |
| *** ociuhandu has quit IRC | 12:28 | |
| *** Yumeng has joined #openstack-nova | 12:28 | |
| Yumeng | sean-k-mooney, @gi | 12:29 |
| *** xinranwang has joined #openstack-nova | 12:30 | |
| Yumeng | good morning nova ^ | 12:31 |
| sean-k-mooney | o/ | 12:31 |
| Yumeng | hi sean, I wanna discuss some issues in the vGPU specs with you. https://review.opendev.org/#/c/750116/ | 12:33 |
| sean-k-mooney | im reviewing it currently | 12:36 |
| *** Luzi has joined #openstack-nova | 12:36 | |
| sean-k-mooney | but sure if you have topics bring them up | 12:36 |
| *** zzzeek has quit IRC | 12:39 | |
| Yumeng | sean-k-mooney: take your time. You can directly leave comments on the patch or ping me later here(if I am still here ^^). My question will be mainly on: 3. Who creates mdev device in the sys path?(proposed change part) which do you prefer | 12:41 |
| *** zzzeek has joined #openstack-nova | 12:41 | |
| sean-k-mooney | sorry had to step away for a phone call, im stell reading the options now, i think im leanign twords cyborg creating them but i need to read your content and sylvain's comments | 12:44 |
| sean-k-mooney | /stell/still/ | 12:44 |
| bauzas | Yumeng: fwiw, I already provided my thoughts | 12:46 |
| bauzas | and i prefer a solution with cyborg precreating the mdevs | 12:46 |
| bauzas | (and just ask nova to use an existing mdev) | 12:47 |
| gibi | bauzas, Yumeng: I've also just pushed my comments on the vGPU spec | 12:48 |
| Yumeng | bauzas: I am reviewing your comments now. yes, solution 2 in the spec is a cyborg precreating the mdevs. seems that's what you guys like. | 12:49 |
| Yumeng | bauzas: ok.thank you, I will take a look first | 12:50 |
| Yumeng | gibi: ok.thank you, I will take a look first | 12:50 |
| *** ociuhandu has joined #openstack-nova | 12:59 | |
| gibi | bauzas: if you want an easy spec: https://review.opendev.org/#/c/755477/ :) | 13:01 |
| *** rcernin has joined #openstack-nova | 13:02 | |
| *** rcernin has quit IRC | 13:02 | |
| *** ociuhandu has quit IRC | 13:04 | |
| *** ociuhandu has joined #openstack-nova | 13:04 | |
| bauzas | gibi: nit : s/Sylvian/Sylvain (Sylvian(e) is a woman name ;) ) | 13:11 |
| gibi | bauzas: sorry | 13:11 |
| bauzas | gibi: heh no worries ;) | 13:11 |
| * bauzas doesn't know how to say your first name ;) | 13:12 | |
| sean-k-mooney | gibi: bauzas i dont think we shoudl be treating mdevs and vf similarly unless you are suggesting we start tracking mdevs in teh pci tracker | 13:12 |
| sean-k-mooney | so that we persist them there and have nova precreate them | 13:12 |
| sean-k-mooney | to provide stable names | 13:13 |
| sean-k-mooney | and other things like numa affinity | 13:13 |
| sean-k-mooney | e.g. just add a type_mdev to the list | 13:13 |
| bauzas | why ? | 13:13 |
| bauzas | anyway, I need to look at other specs | 13:13 |
| gibi | sean-k-mooney: I don't suggest to plug mdevs into the pci tracker, especially that cyborgs would like to track them | 13:14 |
| sean-k-mooney | because mdev are not ment to work like vf in that they framework was desigined to have them dynmicaly created at runtime by higher level orcestors | 13:14 |
| sean-k-mooney | gibi: well my point is that without cycborg i dont think its valid to say they should be precreated | 13:14 |
| bauzas | gibi: me too | 13:15 |
| *** ociuhandu has quit IRC | 13:15 | |
| sean-k-mooney | that is just not how they were ment to be used | 13:15 |
| bauzas | gibi: sean-k-mooney: honestly, I would like to stop supporting to create mdevs in nova | 13:15 |
| bauzas | but first, we would need to support it by TripleO | 13:15 |
| gibi | sean-k-mooney: does libvirt handles mdevs differently than pci? | 13:15 |
| sean-k-mooney | right i dont think we shoudl do that not without providing a real alternitive | 13:15 |
| sean-k-mooney | gibi: yes it does | 13:15 |
| bauzas | gibi: mdevs are a bit different | 13:16 |
| bauzas | but, | 13:16 |
| sean-k-mooney | gibi: libvirt alows them to be allcoated dymically ti does not do the same for vfs | 13:16 |
| bauzas | I don't see why we couldn't just tell 'sorry, but please precreate them' | 13:16 |
| sean-k-mooney | bauzas: that would be a major regression in functionality for one | 13:16 |
| sean-k-mooney | and be an upgrade issue | 13:16 |
| bauzas | given we only support one type for GPU, when you tell which type you want to get for a pGPU, then we could just create them | 13:16 |
| sean-k-mooney | but the main reason is most device dont support one type | 13:17 |
| bauzas | anyway, I need to be off for 10 mins | 13:17 |
| bauzas | (someone is around my home) | 13:17 |
| sean-k-mooney | the whole we only support one type per PGPU is a limiatieon in nvidias solution that is not representivie of the mdev framework as a whole | 13:18 |
| *** artom has quit IRC | 13:18 | |
| sean-k-mooney | if we support generic mdevs in teh future and in the cyborg case in particalar it dose not hold true that all device will only support 1 mdev type | 13:18 |
| tacco | sean-k-mooney: fyi, still no luck. but all in all some more informations. I can see 248available cpus in the vm but everything above id63 are marked as offline. | 13:22 |
| gibi | hm, I more and more like the idea to hide this complexity from nova in cyborg. Then cyborg is free to pre-create or create on the fly the mdev at arq mind. Nova don't have to worry about it (except the recreation case at reboot, see my comment in the spec) | 13:22 |
| gibi | s/mind/bind/ | 13:23 |
| sean-k-mooney | tacco: ok so that is pointing to the guest kernel | 13:23 |
| sean-k-mooney | tacco: well most likely the kernel | 13:23 |
| sean-k-mooney | have you tried onlining them via /sys | 13:23 |
| sean-k-mooney | gibi: yep i was noting that in the spec | 13:24 |
| sean-k-mooney | solution 2 id fine if cyborg does the precreate when we bind at teh conductor | 13:24 |
| sean-k-mooney | its not ok if we need to do an api call on the compute | 13:24 |
| sean-k-mooney | they can also do option 3 and precreate them at agent start | 13:25 |
| gibi | sean-k-mooney: yeah, I assumed that cyborg provides the mdev to as in the same way as provides the pci for an fpga | 13:25 |
| gibi | s/to as/ to us/ | 13:25 |
| sean-k-mooney | yep in the arq when it completes the binding | 13:25 |
| tacco | sean-k-mooney: yes, but then i got a mem error. | 13:25 |
| sean-k-mooney | whcih we poll for on the compute | 13:25 |
| gibi | sean-k-mooney: exactly | 13:25 |
| sean-k-mooney | so its really just the uuid which we then stick in the xml | 13:25 |
| gibi | sean-k-mooney: and this uuid needs to be stable from nova perspective at reboot, or else nova needs to update the xml | 13:27 |
| sean-k-mooney | well we regenerate teh xml form scratch on reboot so | 13:27 |
| sean-k-mooney | it could change | 13:27 |
| sean-k-mooney | we wont notice | 13:27 |
| *** Yumeng has quit IRC | 13:28 | |
| openstackgerrit | Merged openstack/nova-specs master: Support interface attach with qos ports https://review.opendev.org/755477 | 13:28 |
| gibi | I mean hypervisor reboot not VM reboot | 13:28 |
| sean-k-mooney | oh same power-on for reasons calls hard reboot | 13:28 |
| gibi | hm, that could work then | 13:29 |
| sean-k-mooney | reasons being we required spwan and hard-reboot before we required stop start in the virt driver | 13:29 |
| sean-k-mooney | and we never changed libvirt because it just worked | 13:29 |
| gibi | do we rebind or requery the arq from cyborg at VM hard reboot? | 13:30 |
| gibi | we need that to pick up a the new uuid | 13:30 |
| sean-k-mooney | that was a question i was going to ask in the spec | 13:31 |
| sean-k-mooney | i cant rember if we get the arq or not | 13:31 |
| bauzas | (still on and off, but I see your convo, will be back in 20 mins-ish) | 13:31 |
| sean-k-mooney | we do not rebind ports on hard reboot | 13:31 |
| sean-k-mooney | for cinder block devices lyarwood would have to correct me but i think we cached the attachmemnt but maybe we recreate them | 13:32 |
| sean-k-mooney | gibi: point being that we had to solve this for fpgas already | 13:32 |
| sean-k-mooney | so i think what ever we did there should be suffienct | 13:32 |
| gibi | sean-k-mooney: yeah, I agree | 13:33 |
| gibi | I have to jump on a call back later | 13:33 |
| sean-k-mooney | cool | 13:33 |
| sean-k-mooney | im leaving comments on the spec but i think we agree cyborg should handel the mdev creation and just tell us | 13:34 |
| *** nweinber has joined #openstack-nova | 13:34 | |
| sean-k-mooney | where it dose that (agent start or bind) is up to them so long as it work for all lifecycle events | 13:34 |
| lyarwood | sean-k-mooney: we don't recreate volume attachments on hard reboot | 13:35 |
| gibi | sean-k-mooney: agre | 13:35 |
| sean-k-mooney | we cache the conection info in the db | 13:35 |
| sean-k-mooney | ? | 13:35 |
| lyarwood | sean-k-mooney: the libvirt driver disconnects volumes from the host and reconnects them but that's it | 13:35 |
| lyarwood | sean-k-mooney: yeah we cache the attachment id and associated connection_info | 13:35 |
| sean-k-mooney | ya | 13:36 |
| sean-k-mooney | so i tought we could hard reboot without external api calls | 13:36 |
| lyarwood | sean-k-mooney: the only way to refresh that at the moment is via a move operation, I've argued that we should do it during a hard reboot in the past | 13:36 |
| sean-k-mooney | but im not sure what the case is for cyborg | 13:36 |
| sean-k-mooney | lyarwood: yep that is why i was asking i knew you wanted to change it but didnt knwo if you wanted to add or remove the caching | 13:37 |
| lyarwood | sean-k-mooney: remove the connection_info caching | 13:37 |
| lyarwood | sean-k-mooney: but if we ever get there is another thing | 13:38 |
| sean-k-mooney | so we would then have a call to cinder on every hard reboot | 13:38 |
| lyarwood | sean-k-mooney: yeah | 13:38 |
| sean-k-mooney | i guess if the cinder service is down the storeage backend may also be down | 13:38 |
| sean-k-mooney | so even with the cache we are not guareteed to be able to connect | 13:39 |
| lyarwood | sean-k-mooney: really depends on the backend, most could be up still | 13:39 |
| lyarwood | sean-k-mooney: we could at least attempt to refresh tbh | 13:39 |
| *** k_mouza has quit IRC | 13:41 | |
| sean-k-mooney | ya ceph and other likely are not running on the same hosts as the cinder services | 13:41 |
| *** ociuhandu has joined #openstack-nova | 13:45 | |
| *** ociuhandu has quit IRC | 13:50 | |
| *** mlavalle has joined #openstack-nova | 13:56 | |
| *** ociuhandu has joined #openstack-nova | 13:57 | |
| tacco | hm.. struggling aroung with aggregates, strange behavior. i tought if i create a aggreagate with a key and attatch that key to a flavor only hosts with the specific flavors are on the aggregate hosts. | 14:01 |
| sean-k-mooney | tacco: only if you use the placement version | 14:02 |
| tacco | or is there anything else to do? For now only a couple of HVs already have the new filter. But i guess only the hosts in the aggregate group needs this filter, or to be aware of. | 14:02 |
| sean-k-mooney | if you use the fitler you need to add key=false to every other flavor | 14:02 |
| tacco | ah.. ok, thanks. that could help for the moment :) | 14:02 |
| sean-k-mooney | https://docs.openstack.org/nova/latest/reference/isolate-aggregates.html | 14:02 |
| tacco | i tought absence of this flavor also implies that it can not be on the hosts. | 14:03 |
| tacco | thanks for pointing me there. | 14:03 |
| *** songwenping__ has quit IRC | 14:03 | |
| sean-k-mooney | tacco: nope that what many assume but its not how it works | 14:03 |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP - Image and flavor defined ephemeral storage encryption https://review.opendev.org/752284 | 14:03 |
| sean-k-mooney | tacco: this is basically unmaintained and im not sure if it still works(it should) but https://opendev.org/x/nfv-filters provdes a filter that does what you want | 14:04 |
| *** songwenping__ has joined #openstack-nova | 14:04 | |
| sean-k-mooney | https://opendev.org/x/nfv-filters/src/branch/master/nfv_filters/nova/scheduler/filters/aggregate_instance_type_filter.py | 14:04 |
| *** songwenping__ has quit IRC | 14:04 | |
| sean-k-mooney | docs are here https://opendev.org/x/nfv-filters/src/branch/master/doc/source/scheduler_filters/aggregate-instance-type-filter.rst | 14:04 |
| sean-k-mooney | if you dont have placment avaiable with the prefilter then that shoudl bridge the gap | 14:05 |
| *** songwenping__ has joined #openstack-nova | 14:05 | |
| tacco | ok, will read first and decide then. :) but.. yeah.. found a bug that was already reported. yay | 14:05 |
| sean-k-mooney | its actully more powerful the what you can do with placemnt or the code in tree but its out of tree and not maintained by me really anymore | 14:06 |
| tacco | yes.. can understand that. I would like to stick as close as i can to upstream and maintained versions. | 14:08 |
| tacco | only if there is no other way out i would think about that.. but our first solution we had was way to full with cherry-picked stuff.. and deployment was only done by manual actions.. i hate it. Now we have a nice fully automated and functional setup i don't want to miss that anymore. :D | 14:09 |
| lyarwood | exit | 14:15 |
| lyarwood | argh | 14:15 |
| kashyap | "You can leave anytime, but you can never exit." | 14:16 |
| * kashyap ducks | 14:16 | |
| *** xek has quit IRC | 14:18 | |
| *** Luzi has quit IRC | 14:18 | |
| *** xek has joined #openstack-nova | 14:18 | |
| *** macz_ has joined #openstack-nova | 14:20 | |
| *** k_mouza has joined #openstack-nova | 14:24 | |
| *** k_mouza has quit IRC | 14:24 | |
| *** macz_ has quit IRC | 14:24 | |
| *** k_mouza has joined #openstack-nova | 14:24 | |
| *** brinzhang_ has quit IRC | 14:26 | |
| *** ociuhandu has quit IRC | 14:30 | |
| *** lbragstad__ has quit IRC | 14:37 | |
| *** sapd1 has joined #openstack-nova | 14:38 | |
| *** lbragstad has joined #openstack-nova | 14:40 | |
| *** tbachman has joined #openstack-nova | 14:42 | |
| tacco | sean-k-mooney: where do i have to set the scheduler.enable_isolated_aggregate_filtering nova.conf on the HVs? | 14:50 |
| sean-k-mooney | no nova.conf on the schduler | 14:50 |
| *** songwenping_ has joined #openstack-nova | 14:50 | |
| sean-k-mooney | so on the contollers | 14:51 |
| openstackgerrit | Merged openstack/nova master: Restore retrying the RPC connection to conductor https://review.opendev.org/762633 | 14:51 |
| openstackgerrit | Merged openstack/nova master: Fix the vGPU dynamic options race https://review.opendev.org/758470 | 14:51 |
| openstackgerrit | Merged openstack/nova master: api-ref: Move 'os-agents' API to obsolete section https://review.opendev.org/755729 | 14:51 |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP libvirt: Allow the default machine type to be changed https://review.opendev.org/762199 | 14:52 |
| lyarwood | kashyap: ^ btw you might be interested in this, I've used some text from a former spec of yours | 14:53 |
| tacco | ok i see. thansk will give it a try. :) | 14:53 |
| *** songwenping__ has quit IRC | 14:53 | |
| kashyap | lyarwood: Ack; will queue | 14:53 |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP libvirt: Allow the default machine type to be changed https://review.opendev.org/762199 | 14:58 |
| *** ociuhandu has joined #openstack-nova | 15:01 | |
| *** artom has joined #openstack-nova | 15:04 | |
| sean-k-mooney | melwitt:i saw your questions/comments on https://review.opendev.org/#/c/602432/ by the way ill adress them shortly and respin it. | 15:09 |
| sean-k-mooney | melwitt: tl;dr plugin a port for vif_type=ovs and hybrid-plug=false is currently a noop unless you are on windows where they pass crete-port to have os-vif create teh ovs port | 15:10 |
| *** ociuhandu has quit IRC | 15:10 | |
| *** ociuhandu has joined #openstack-nova | 15:10 | |
| sean-k-mooney | melwitt: it was a noop because libvirt was previously creating the ovs port which is racy and what im changing in my patch | 15:11 |
| sean-k-mooney | so we now need to pass create-port=true but ill likely remove that in a futre release of os-vif | 15:11 |
| sean-k-mooney | i orginally made it unconditional but it triggered a race in the ovs agent which has sicne been fixed | 15:12 |
| sean-k-mooney | in hignsight i should not have reverted the cahnge and added create-port but it was too late in the cyle to make it unconditonal again | 15:13 |
| sean-k-mooney | ill deprecate it this cycle and likely remove create-port in x | 15:13 |
| sean-k-mooney | since all knwon callers will now be passing true | 15:14 |
| *** lpetrut has quit IRC | 15:16 | |
| *** ociuhandu has quit IRC | 15:21 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP - Image and flavor defined ephemeral storage encryption https://review.opendev.org/752284 | 15:21 |
| *** ociuhandu has joined #openstack-nova | 15:27 | |
| tacco | whats the best way to figure out which hosts are given back by the API if i request a certain flavor? because try+error could be verry boring over the time :D | 15:32 |
| tacco | --debug to the cli command and see the api result? :) | 15:33 |
| *** dklyle has joined #openstack-nova | 15:34 | |
| sean-k-mooney | tacco: you cant really | 15:37 |
| sean-k-mooney | tacco: even if you were to ask placment that woudl not take into account features | 15:37 |
| sean-k-mooney | there is no dry run option | 15:37 |
| tacco | i see. | 15:37 |
| sean-k-mooney | or similar to get the list | 15:37 |
| *** ociuhandu has quit IRC | 15:38 | |
| hemanth_n | Hi appreciate any reviews on the following backports https://review.opendev.org/#/q/status:open+project:openstack/nova+topic:bug/1892361 thank you | 15:39 |
| stephenfin | sean-k-mooney: when did we talk about the project_admin behaviour /o\ https://review.opendev.org/#/c/755109/ | 15:49 |
| stephenfin | I agree it makes sense but I can't remember discussing it, heh | 15:49 |
| *** mkrai has joined #openstack-nova | 15:50 | |
| sean-k-mooney | in the post ptg review | 15:52 |
| sean-k-mooney | last week | 15:52 |
| sean-k-mooney | as an alternivie to gmann use system_reader+proejct_admin proposal | 15:53 |
| sean-k-mooney | also in the call with lance | 15:53 |
| sean-k-mooney | on the nova rbac stuff | 15:53 |
| sean-k-mooney | so not upstream unfortunetly but i suggested puting it into your spec on the call | 15:53 |
| sean-k-mooney | we did talk about alot of stuff on that call to be fair | 15:54 |
| *** artom has quit IRC | 15:54 | |
| stephenfin | sean-k-mooney: fair fair. Let me amend it real quick | 16:00 |
| sean-k-mooney | i was happy with the rest of the spec for what its worth | 16:01 |
| sean-k-mooney | i said it was fine but i ment it looked good | 16:01 |
| *** ociuhandu has joined #openstack-nova | 16:02 | |
| *** k_mouza has quit IRC | 16:07 | |
| tacco | sean-k-mooney: thanks that helped, now i have to see how to get the config part into os_nova for openstack-ansible to persist my change somewhere upstream. :D | 16:12 |
| sean-k-mooney | :) | 16:12 |
| *** bnemec has quit IRC | 16:16 | |
| *** hemna has quit IRC | 16:18 | |
| *** mkrai has quit IRC | 16:21 | |
| *** hemna has joined #openstack-nova | 16:23 | |
| *** k_mouza has joined #openstack-nova | 16:23 | |
| *** artom has joined #openstack-nova | 16:28 | |
| *** iurygregory has quit IRC | 16:29 | |
| *** xek has quit IRC | 16:32 | |
| gmann | stephenfin: sean-k-mooney replied on review. are we allowing hypervisor info for projects for use case of boot server on host? | 16:35 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Update modernize-os-hypervisors-api spec https://review.opendev.org/763043 | 16:35 |
| stephenfin | gmann, sean-k-mooney, bauzas, gibi: I've split that potential policy change out so we can debate the merits of it separately ^ | 16:36 |
| gibi | stephenfin: ack, thanks | 16:37 |
| gmann | stephenfin: +1, | 16:37 |
| sean-k-mooney | gmann: yes | 16:38 |
| sean-k-mooney | gmann: i dont think we shoudl be using system_read+proejct_admin | 16:38 |
| gmann | what we can do is keep GET /os-hypervisors for ['system' and 'projects'] and only return list of hypervisors for project what all they are access to via aggregate metadata otherwise emtpy | 16:38 |
| stephenfin | sean-k-mooney: You okay to discuss separately, yeah? Can you toggle your -1 if so? | 16:39 |
| gmann | sean-k-mooney: well if we make only project then system would not be able to list hypervisor at all | 16:39 |
| sean-k-mooney | stephenfin: yep if you file a second spec im happy to defer to there | 16:39 |
| gmann | and adding/removing the host are system level operation right | 16:39 |
| stephenfin | See https://review.opendev.org/763043 | 16:39 |
| sean-k-mooney | gmann: system would | 16:39 |
| sean-k-mooney | gmann: its an additive change | 16:39 |
| sean-k-mooney | fithe now yould admins can call /os-hyperviors | 16:40 |
| sean-k-mooney | .... | 16:40 |
| sean-k-mooney | right now only admins can call /os-hypervisors | 16:40 |
| gmann | but they need scope as project. so any system scope token would not be able to do 'project'-only scope | 16:40 |
| sean-k-mooney | right | 16:41 |
| sean-k-mooney | system scoped tokens should not be project only | 16:41 |
| gmann | yeah so how they get hyperviors ino | 16:41 |
| gmann | info | 16:41 |
| sean-k-mooney | im suggesting project_admin shoudl be able to list the host summaries for all hosts tehy can boot too | 16:41 |
| gmann | yeah that i agree. but keep scope_type as ['system', 'project'] sop that system can list all hyperviros and project can only list what they have access to | 16:42 |
| sean-k-mooney | with role admin | 16:42 |
| gmann | and default to SYSTEM_READER_OR_PROJECT_ADMIN | 16:43 |
| sean-k-mooney | we dont want project member or reader to have acess | 16:43 |
| gmann | yes | 16:43 |
| sean-k-mooney | ya | 16:43 |
| sean-k-mooney | SYSTEM_READER_OR_PROJECT_ADMIN woudl be correct | 16:43 |
| sean-k-mooney | if we do that we dont need to change the api to pass a proejct id | 16:43 |
| gmann | cool and in code we can check and return only the project accessible hypervisors but in that case we need to check the token's scope in nova code | 16:44 |
| gmann | sean-k-mooney: +1 on that. | 16:44 |
| sean-k-mooney | yes on just that one api where we are adding thei fucntionality | 16:44 |
| sean-k-mooney | if we need to expose something lese to project admins we can do that in a similar way | 16:44 |
| gmann | checking token's cope part is little different and new which we need to check how to do | 16:44 |
| sean-k-mooney | but we dont need full system_reader for project admin usecases | 16:45 |
| gmann | agree. | 16:45 |
| sean-k-mooney | i belive it should be in the context object | 16:45 |
| gmann | yes we can do from context only difference is, it is 'system_scope' in context and 'system' in oslo policy side. but not big deal | 16:46 |
| sean-k-mooney | stephenfin: updated https://review.opendev.org/#/c/755109/4 to a +1 if someone wants to +w | 16:47 |
| *** bnemec has joined #openstack-nova | 16:50 | |
| gmann | stephenfin: sean-k-mooney replied on this what we disucssed - https://review.opendev.org/#/c/763043/1 | 16:53 |
| stephenfin | Ack, will respin shortly. Thanks :) | 16:53 |
| *** ociuhandu has quit IRC | 16:55 | |
| *** ociuhandu has joined #openstack-nova | 16:55 | |
| *** hamalq has quit IRC | 16:56 | |
| *** hamalq has joined #openstack-nova | 16:57 | |
| *** ociuhandu has quit IRC | 17:00 | |
| lyarwood | kashyap / stephenfin ; thinking about the machine type enumeration problem a little more, do we care if an instance is using a versioned machine type? | 17:02 |
| sean-k-mooney | lyarwood: kashyap and i disagree | 17:03 |
| kashyap | lyarwood: Do you mean, should we record that or not? | 17:03 |
| sean-k-mooney | lyarwood: i prefer to use the unversioned ones | 17:03 |
| lyarwood | kashyap / stephenfin ; if we record the verioned machine type and always use it wouldn't that stop users getting security updates for free? | 17:03 |
| sean-k-mooney | there are reason to use the versioned ones | 17:03 |
| sean-k-mooney | livemigration/upgrades mainly | 17:03 |
| lyarwood | sean-k-mooney: right I think I'm with you if the ABI remains the same | 17:03 |
| sean-k-mooney | the machine type partly defines the abi | 17:04 |
| sean-k-mooney | so it wont nessisarly remian the same but its largly the same | 17:04 |
| sean-k-mooney | thing that can change are the max number of cores supported for example | 17:04 |
| sean-k-mooney | but it should be backward compatible | 17:04 |
| lyarwood | yeah | 17:04 |
| kashyap | lyarwood: Just so I get you: we're referring to CentOS/RHEL-based versioned machine types, yeah? (And not the upstream QEMU's per-release machine types) | 17:04 |
| sean-k-mooney | e.g. a q35-1 guest shoudl be upgradable to a q35-2 machine type | 17:05 |
| kashyap | sean-k-mooney: Well, wait. That sentence doesn't make sense | 17:05 |
| kashyap | To remind: when you migrate an instance with machine type q35-1, it *remains* q35-1 on the destination | 17:05 |
| sean-k-mooney | the abi between verions should change additivly | 17:06 |
| kashyap | ... until one explicitly changes it via nova.conf. | 17:06 |
| sean-k-mooney | kashyap: yes i never said migrate | 17:06 |
| *** sapd1 has quit IRC | 17:06 | |
| sean-k-mooney | i said upgrade i actully ment via a hard reboot | 17:06 |
| kashyap | sean-k-mooney: Err, my mind read the "upgradable" as "migratable". Silly me! | 17:06 |
| kashyap | sean-k-mooney: Even in the case of "upgrade", libvirt won't gratuitously update it, you have to explicitly request it. | 17:07 |
| sean-k-mooney | you can expect that a guest booed on an older vers should be able to boot in the newer verion of the same overall type | 17:07 |
| kashyap | lyarwood: Not sure if we're on track with your original question | 17:07 |
| lyarwood | kashyap: yeah this is still on the original question | 17:07 |
| sean-k-mooney | kashyap: well if you just use q35 in your nova.conf | 17:07 |
| openstackgerrit | Merged openstack/nova-specs master: Add modernize-os-hypervisors-api spec https://review.opendev.org/755109 | 17:07 |
| lyarwood | kashyap: so my spec as written will track the versioned machine type | 17:07 |
| sean-k-mooney | then since we destroy and recreate teh domain it will use the new one that the q35 alis now points too | 17:07 |
| kashyap | lyarwood: So ... on the security updates: they're not tied to machine types often; most fixes in machine types are typically garden-variety "bug fixes" | 17:08 |
| lyarwood | kashyap: but the issue with that is that you can never move from that specific version without rebuilding the ntire instance | 17:08 |
| lyarwood | entire* | 17:08 |
| sean-k-mooney | lyarwood: well really it should track the unversioned one | 17:08 |
| sean-k-mooney | unless its set in the nova.conf | 17:08 |
| lyarwood | sean-k-mooney: that's what I'm getting at | 17:08 |
| sean-k-mooney | in which case it should use that | 17:08 |
| lyarwood | sean-k-mooney: even then I think we should track the unversioned machine type | 17:08 |
| kashyap | sean-k-mooney: Lee's question is, what *if* the guest is explicitly using the versioned machine type | 17:08 |
| sean-k-mooney | if you had not set it the expecation is that it will change when you hard-reboot if you did a yum update and updated libvirt/qemu | 17:09 |
| sean-k-mooney | kashyap: if the guest is useing a verioned machinve type in the image you dont need to do anything | 17:09 |
| sean-k-mooney | if the guest is useing a version machine tyep from the config then use the config value | 17:09 |
| sean-k-mooney | the upgrade to a newer one was ment to be handeled by the new recreate api | 17:10 |
| sean-k-mooney | which we rejected so there is no upgrade path now | 17:10 |
| lyarwood | well just using the alias | 17:10 |
| lyarwood | that's the only way to provide an upgrade path to users | 17:10 |
| sean-k-mooney | right but on waht operation would it upgrade | 17:10 |
| kashyap | lyarwood: I think the clearest way to get understanding here is to lay out some example scenarios | 17:11 |
| lyarwood | move or hard reboot | 17:11 |
| lyarwood | assuming the underlying QEMU had been updated | 17:11 |
| kashyap | lyarwood: I've writte up some for RHOS QE for cold migration, live and evacuate scenarios | 17:11 |
| sean-k-mooney | i guess i could by that | 17:11 |
| *** ociuhandu has joined #openstack-nova | 17:11 | |
| kashyap | I can clean that up and provide a link, as lay out the impliciations. And that allows one to better reason about it, IMHO | 17:12 |
| sean-k-mooney | resize,hardreboot, unshelve, rebuild and cold migrate | 17:12 |
| lyarwood | does anyone actually care about running on a specific version of a machine type? | 17:12 |
| sean-k-mooney | lyarwood: do you have my hw:stable_abi extra spec in your spec | 17:12 |
| lyarwood | sean-k-mooney: no | 17:12 |
| sean-k-mooney | lyarwood: we do for ffu | 17:12 |
| sean-k-mooney | but beyond that im not sure | 17:13 |
| lyarwood | sean-k-mooney: why? Wouldn't that be covered by LM | 17:13 |
| sean-k-mooney | yes kind of | 17:13 |
| sean-k-mooney | it more we want to pin the new host to the old verion untill the rolling upgrade is complete | 17:13 |
| sean-k-mooney | i think | 17:14 |
| sean-k-mooney | so that we can technically migrate form new to old if we need too eventhough we say dont do that | 17:14 |
| sean-k-mooney | changing the machine_type is ment to be a post upgrade step | 17:14 |
| lyarwood | well you could still technically do that with FFU | 17:14 |
| lyarwood | actually no you couldn't ignore me | 17:15 |
| lyarwood | and agreed with post upgrade for changing default machine types | 17:15 |
| kashyap | A factor to consider: if you track the unversioned 'pc' or 'q35', it always aliases to the latest available versioned machine type on the box. | 17:16 |
| sean-k-mooney | from a donwstream point of view what we said was new installs should use the new default, existing installs should be upgrade with the old default then the default shoudl be changed as a post upgrade action | 17:16 |
| sean-k-mooney | kashyap: yes but if you have not express an opipion in the image and the config is either unset or set to the aliase i think that is the right thing to track anyway | 17:17 |
| kashyap | sean-k-mooney: Yep, that I agree with. | 17:17 |
| sean-k-mooney | the edge case is if you have a versioned machine type in the config | 17:17 |
| sean-k-mooney | i was suggesting recodign that | 17:18 |
| sean-k-mooney | but then we cant really allow it to change on hard reboot | 17:18 |
| lyarwood | it wouldn't | 17:18 |
| sean-k-mooney | it would have to be resize or rebuld only | 17:18 |
| sean-k-mooney | maintain the machine type on cold or live migate | 17:18 |
| lyarwood | if a versioned machine type is recorded the only way to change it would be with a rebuild | 17:18 |
| *** mlavalle has quit IRC | 17:18 | |
| sean-k-mooney | lyarwood: only rebuild or rebuild/resize | 17:19 |
| lyarwood | sean-k-mooney: rebuild/resize sorry | 17:19 |
| *** iurygregory has joined #openstack-nova | 17:19 | |
| sean-k-mooney | so the elephant in the room | 17:19 |
| sean-k-mooney | what about evacuate | 17:19 |
| lyarwood | sean-k-mooney: actually can you proivide the machine type via a flavor | 17:19 |
| sean-k-mooney | lyarwood: no | 17:19 |
| sean-k-mooney | but we could | 17:19 |
| lyarwood | sean-k-mooney: then just rebuild | 17:19 |
| sean-k-mooney | that does not work then | 17:20 |
| sean-k-mooney | rebuild is distructive | 17:20 |
| lyarwood | sean-k-mooney: and evacuate uses rebuild | 17:20 |
| sean-k-mooney | and for secuity reason we do need a way to change this | 17:20 |
| sean-k-mooney | which is one of the reason i had recreate | 17:20 |
| lyarwood | well that's no different to the situation we are in today | 17:20 |
| sean-k-mooney | no today you jsut change the config option and hardreboot | 17:21 |
| lyarwood | that a user has no access to | 17:21 |
| sean-k-mooney | or if its using the aliase just package update and reboot | 17:21 |
| lyarwood | that's basically the same as a db update | 17:21 |
| sean-k-mooney | it is yes | 17:21 |
| sean-k-mooney | we need a way for an admin to update it | 17:21 |
| sean-k-mooney | that is not a db update | 17:21 |
| lyarwood | if you're using an alias it's just a hard reboot with the new approach still | 17:21 |
| sean-k-mooney | we dont need a way for a user | 17:21 |
| sean-k-mooney | lyarwood: yep which is why i like aliasis amoung other things | 17:22 |
| sean-k-mooney | lyarwood: what about a nova manage command | 17:22 |
| lyarwood | sean-k-mooney: yup that could work | 17:22 |
| sean-k-mooney | to update the machine type in a suppotable way | 17:22 |
| sean-k-mooney | ideally support either the instance uuid/uuids or a host | 17:23 |
| lyarwood | but it still looks like we can't enumerate all of the possible machine types so I'm back to using a StringField | 17:23 |
| lyarwood | gah | 17:23 |
| sean-k-mooney | well ya its a sting filed | 17:23 |
| sean-k-mooney | they are different per disto | 17:23 |
| lyarwood | yup | 17:24 |
| sean-k-mooney | and you can add your own by just adding a file | 17:24 |
| lyarwood | I indeeed, right need to help with childcare and then I'll update the spec | 17:24 |
| sean-k-mooney | i know some people have done that to work around some default in the past realted to >2TB vms | 17:24 |
| lyarwood | thanks stephenfin, kashyap, sean-k-mooney! | 17:24 |
| *** tesseract has quit IRC | 17:25 | |
| *** rpittau is now known as rpittau|afk | 17:27 | |
| *** k_mouza has quit IRC | 17:30 | |
| *** mlavalle has joined #openstack-nova | 17:31 | |
| ralonsoh | sean-k-mooney, just a heads-up: https://review.opendev.org/740067 | 17:36 |
| sean-k-mooney | oh nice | 17:36 |
| sean-k-mooney | so now just the nova bit | 17:37 |
| sean-k-mooney | thanks | 17:37 |
| sean-k-mooney | ralonsoh: ill respin https://review.opendev.org/#/c/760047/ this week hopefully too | 17:37 |
| ralonsoh | sean-k-mooney, perfect! | 17:37 |
| * gibi leaves the keyboard | 17:42 | |
| gibi | o/ | 17:42 |
| *** ociuhandu_ has joined #openstack-nova | 17:43 | |
| *** ociuhandu has quit IRC | 17:46 | |
| *** ociuhandu_ has quit IRC | 17:47 | |
| *** k_mouza has joined #openstack-nova | 17:50 | |
| *** martinkennelly has quit IRC | 17:57 | |
| *** jangutter_ has quit IRC | 17:58 | |
| stephenfin | dansmith: Is there any way to identify what images have been cached for an aggregate? | 17:58 |
| stephenfin | Assuming such a request even makes sense? | 17:58 |
| *** jangutter has joined #openstack-nova | 17:58 | |
| dansmith | stephenfin: no, the caching thing is all point-in-time requests, akin to asking for boots that don't end up spawning actual instances | 17:59 |
| dansmith | so no accounting or recording | 17:59 |
| stephenfin | okay, so not displaying anything in response to an 'openstack aggregate cache image' call makes sense? | 18:00 |
| stephenfin | e.g. there's no point showing the aggregate since nothings has "changed" | 18:00 |
| dansmith | like a table of data or something? no, nothing to display. obviously if it returns 404 because there's no aggregate or something an error makes sense | 18:01 |
| dansmith | right, no point in showing the image or aggregate, IMHO | 18:01 |
| stephenfin | dansmith++ great, thanks :) | 18:01 |
| * stephenfin copy-pastes to https://review.opendev.org/#/c/762134/ | 18:01 | |
| dansmith | from context it sounds like you're ...cool | 18:02 |
| *** mgoddard has quit IRC | 18:02 | |
| dansmith | stephenfin: in case you hadn't seen it, I had good intentions: https://review.opendev.org/#/c/688960/ | 18:02 |
| dansmith | but never followed through because I suck | 18:02 |
| *** andrewbonney has quit IRC | 18:05 | |
| stephenfin | thankfully gtema is a machine 🤗 | 18:07 |
| *** k_mouza has quit IRC | 18:10 | |
| *** xek has joined #openstack-nova | 18:15 | |
| *** k_mouza has joined #openstack-nova | 18:19 | |
| *** xinranwang has quit IRC | 18:49 | |
| *** k_mouza has quit IRC | 18:56 | |
| *** gyee has joined #openstack-nova | 19:18 | |
| *** mgoddard has joined #openstack-nova | 19:21 | |
| *** jawad_axd has joined #openstack-nova | 19:36 | |
| *** xek has quit IRC | 19:44 | |
| *** xek has joined #openstack-nova | 19:44 | |
| *** k_mouza has joined #openstack-nova | 19:57 | |
| *** k_mouza has quit IRC | 20:01 | |
| *** gryf has quit IRC | 20:09 | |
| *** gryf has joined #openstack-nova | 20:09 | |
| *** k_mouza has joined #openstack-nova | 20:16 | |
| *** dtantsur is now known as dtantsur|afk | 20:17 | |
| *** k_mouza has quit IRC | 20:21 | |
| *** smcginnis has quit IRC | 20:24 | |
| *** mgoddard has quit IRC | 20:31 | |
| *** nweinber has quit IRC | 20:58 | |
| *** nweinber has joined #openstack-nova | 20:58 | |
| *** rcernin has joined #openstack-nova | 21:02 | |
| *** nweinber has quit IRC | 21:30 | |
| *** smcginnis has joined #openstack-nova | 21:59 | |
| *** ralonsoh has quit IRC | 22:03 | |
| *** xek has quit IRC | 22:22 | |
| *** songwenping__ has joined #openstack-nova | 22:50 | |
| *** songwenping_ has quit IRC | 22:53 | |
| rm_work | hey, what config does nova-manage pull the db connection stuff from by default? is it /etc/nova.conf? | 23:18 |
| rm_work | err, /etc/nova/nova.conf | 23:19 |
| rm_work | it seems to not be respecting all of the sql connection config | 23:19 |
| rm_work | we use x509 auth for our sql connection, which is in the connection string in nova.conf as &ssl_cert=...&ssl_key=... | 23:19 |
| rm_work | which works for the actual service, but the nova-manage db stuff doesn't seem to use it correctly | 23:20 |
| *** slaweq has quit IRC | 23:21 | |
| rm_work | yeah, verified even with the config explicitly set, it's not respecting the x509 args, so the db migration scripts fail on the sql connection | 23:23 |
| rm_work | hmmm, well that was my assumption, but maybe it's actually that it is missing some other sql permission? trying to dig in deeper, I just know I get this error when it tries to run `_get_marker_for_migrate_instances` | 23:28 |
| rm_work | `(pymysql.err.OperationalError) (1045, "Access denied for user 'nova'@'sqlhost' (using password: YES)")` | 23:28 |
| rm_work | but it does seem to be able to get SOME data, as it printed some numbers about the migration counts correctly | 23:29 |
| *** mloza has quit IRC | 23:46 | |
| *** tosky has quit IRC | 23:58 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!