openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP zuul: nova-evacuate https://review.opendev.org/744883 | 00:04 |
---|---|---|
*** hamalq has quit IRC | 00:18 | |
*** brinzhang0 has joined #openstack-nova | 00:29 | |
*** tetsuro has joined #openstack-nova | 00:32 | |
*** brinzhang_ has quit IRC | 00:32 | |
*** songwenping_ has joined #openstack-nova | 00:37 | |
*** songwenping__ has joined #openstack-nova | 00:40 | |
*** swp20 has quit IRC | 00:41 | |
*** Liang__ has joined #openstack-nova | 00:41 | |
*** songwenping_ has quit IRC | 00:43 | |
*** songwenping_ has joined #openstack-nova | 01:37 | |
*** songwenping__ has quit IRC | 01:40 | |
*** kaisers has quit IRC | 02:01 | |
*** yaawang has quit IRC | 02:05 | |
*** yaawang has joined #openstack-nova | 02:06 | |
*** yaawang has quit IRC | 02:10 | |
*** yaawang has joined #openstack-nova | 02:11 | |
*** spatel has joined #openstack-nova | 02:13 | |
*** yaawang has quit IRC | 02:15 | |
*** yaawang has joined #openstack-nova | 02:16 | |
*** markvoelker has quit IRC | 02:26 | |
*** Yumeng has joined #openstack-nova | 02:31 | |
*** Liang__ has quit IRC | 02:59 | |
*** songwenping_ has quit IRC | 03:02 | |
*** songwenping_ has joined #openstack-nova | 03:02 | |
*** spatel has quit IRC | 03:02 | |
*** songwenping_ has quit IRC | 03:03 | |
*** songwenping_ has joined #openstack-nova | 03:03 | |
*** songwenping_ has joined #openstack-nova | 03:04 | |
*** xinranwang__ has joined #openstack-nova | 03:06 | |
*** brinzhang_ has joined #openstack-nova | 03:08 | |
*** brinzhang0 has quit IRC | 03:10 | |
openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 03:20 |
*** psachin has joined #openstack-nova | 03:31 | |
*** markmcclain has quit IRC | 03:31 | |
*** mkrai has joined #openstack-nova | 03:31 | |
*** markmcclain has joined #openstack-nova | 03:35 | |
*** gyee has quit IRC | 03:50 | |
*** rcernin has quit IRC | 04:03 | |
*** mkrai has quit IRC | 04:23 | |
*** rcernin has joined #openstack-nova | 04:24 | |
*** mkrai has joined #openstack-nova | 04:26 | |
*** ratailor has joined #openstack-nova | 04:30 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-nova | 04:33 | |
*** links has joined #openstack-nova | 04:51 | |
*** efried has quit IRC | 05:05 | |
*** ociuhandu has joined #openstack-nova | 05:15 | |
*** ociuhandu has quit IRC | 05:20 | |
*** efried has joined #openstack-nova | 05:24 | |
*** links has quit IRC | 05:30 | |
*** links has joined #openstack-nova | 05:47 | |
*** mlycka has joined #openstack-nova | 05:48 | |
brinzhang_ | alex_xu: hi, do you know why this patch didnot run zuul +2? https://review.opendev.org/#/c/744280/ | 05:54 |
brinzhang_ | is it due to the tag of Signed-off-by? | 05:55 |
*** mkrai has quit IRC | 05:59 | |
*** mkrai_ has joined #openstack-nova | 05:59 | |
*** brinzhang0 has joined #openstack-nova | 06:07 | |
*** vishalmanchanda has joined #openstack-nova | 06:09 | |
*** brinzhang_ has quit IRC | 06:10 | |
*** udesale has joined #openstack-nova | 06:18 | |
*** dklyle has quit IRC | 06:39 | |
*** ociuhandu has joined #openstack-nova | 06:41 | |
*** yaawang has quit IRC | 06:44 | |
*** yaawang has joined #openstack-nova | 06:44 | |
*** xiaolin has quit IRC | 06:49 | |
*** slaweq has joined #openstack-nova | 06:53 | |
*** rcernin has quit IRC | 06:54 | |
*** tesseract has joined #openstack-nova | 07:03 | |
*** tetsuro has quit IRC | 07:09 | |
gibi | brinzhang0: I don't think that Signed-off stops zuul. | 07:18 |
*** rcernin has joined #openstack-nova | 07:20 | |
brinzhang0 | gibi: thanks. I dont know which stoped zuul to go :( | 07:20 |
*** maciejjozefczyk has joined #openstack-nova | 07:20 | |
brinzhang0 | gibi: Do you have time to check? | 07:21 |
*** Liang__ has joined #openstack-nova | 07:21 | |
*** mvorwerk has joined #openstack-nova | 07:23 | |
*** rcernin has quit IRC | 07:25 | |
*** xinranwang__ has quit IRC | 07:25 | |
*** brinzhang_ has joined #openstack-nova | 07:29 | |
*** nightmare_unreal has joined #openstack-nova | 07:29 | |
*** dtantsur|afk is now known as dtantsur | 07:30 | |
*** brinzhang0 has quit IRC | 07:32 | |
*** brinzhang has joined #openstack-nova | 07:32 | |
*** ociuhandu has quit IRC | 07:33 | |
gibi | brinzhang_: honestly I don't know where to look. I suggest to do a recheck, or even better fix the commit message according to stephenfin's comments and that will retrigger zuul | 07:34 |
*** brinzhang_ has quit IRC | 07:34 | |
*** avolkov has joined #openstack-nova | 07:39 | |
*** dougsz has joined #openstack-nova | 07:44 | |
*** tosky has joined #openstack-nova | 07:46 | |
*** ralonsoh has joined #openstack-nova | 07:47 | |
*** ralonsoh has quit IRC | 07:48 | |
*** udesale has quit IRC | 07:50 | |
*** udesale has joined #openstack-nova | 07:50 | |
*** jraju__ has joined #openstack-nova | 07:51 | |
*** links has quit IRC | 07:52 | |
*** ociuhandu has joined #openstack-nova | 07:54 | |
*** Yumeng has quit IRC | 07:57 | |
*** udesale has quit IRC | 08:00 | |
*** udesale has joined #openstack-nova | 08:00 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: [Trivial] Remove wrong format_message() conversion https://review.opendev.org/744280 | 08:01 |
*** takamatsu has quit IRC | 08:02 | |
brinzhang | gibi: thanks, updated, and removed the singned-off-by, can you +2 for it? | 08:02 |
gibi | sure | 08:03 |
brinzhang | gibi: thanks ^^ | 08:04 |
*** sapd__x has joined #openstack-nova | 08:04 | |
gibi | doen | 08:04 |
gibi | done | 08:05 |
gibi | I hope zuul will pick it up this time | 08:05 |
*** ralonsoh has joined #openstack-nova | 08:06 | |
brinzhang | gibi: cool, looks the zuu can be run | 08:06 |
brinzhang | before I rechecked it, and it also in zuul +1 status, I guess because of the signed-of-by, but Iam not sure | 08:07 |
*** jraju__ has quit IRC | 08:13 | |
*** Liang__ has quit IRC | 08:17 | |
*** Liang__ has joined #openstack-nova | 08:18 | |
*** links has joined #openstack-nova | 08:23 | |
*** belmoreira has joined #openstack-nova | 08:25 | |
*** links has quit IRC | 08:29 | |
*** priteau has joined #openstack-nova | 08:29 | |
*** derekh has joined #openstack-nova | 08:30 | |
*** osmanlicilegi has quit IRC | 08:32 | |
*** xek has joined #openstack-nova | 08:35 | |
*** links has joined #openstack-nova | 08:36 | |
*** jraju__ has joined #openstack-nova | 08:42 | |
*** arxcruz|off is now known as arxcruz | 08:42 | |
*** links has quit IRC | 08:42 | |
*** iurygregory has quit IRC | 08:51 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow up for Ibf8dca4bd57b3bddb39955b53cc03564506f5754 https://review.opendev.org/745095 | 08:54 |
*** takamatsu has joined #openstack-nova | 08:58 | |
*** links has joined #openstack-nova | 08:59 | |
*** jraju__ has quit IRC | 09:00 | |
*** iurygregory has joined #openstack-nova | 09:02 | |
*** jraju__ has joined #openstack-nova | 09:04 | |
*** links has quit IRC | 09:04 | |
*** takamatsu has quit IRC | 09:07 | |
*** takamatsu has joined #openstack-nova | 09:07 | |
*** k_mouza has joined #openstack-nova | 09:10 | |
*** osmanlicilegi has joined #openstack-nova | 09:10 | |
*** jangutter_ has joined #openstack-nova | 09:12 | |
*** xek has quit IRC | 09:12 | |
*** jangutter has quit IRC | 09:14 | |
*** gokhani has joined #openstack-nova | 09:21 | |
*** efried has quit IRC | 09:25 | |
*** efried has joined #openstack-nova | 09:25 | |
*** jangutter has joined #openstack-nova | 09:46 | |
*** jangutter_ has quit IRC | 09:46 | |
*** jangutter_ has joined #openstack-nova | 09:46 | |
*** jangutter has quit IRC | 09:50 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/744958 | 09:52 |
*** takamatsu has quit IRC | 09:53 | |
kashyap | stephenfin: I think you used the change ID in the commit summary because it is not yet merged? | 10:00 |
stephenfin | kashyap: No. I just wanted something to link back to the original patch for context | 10:01 |
kashyap | I personally find it strange to see such a large change ID as part of commit summary. How about "Follow-up for "Lookup nic feature by PCI address" patch | 10:01 |
stephenfin | I could, though the change ID allows you to click through to the original change | 10:02 |
kashyap | Yeah, I see the value; but recall: we also have to bear in mind the "guy working from the woods" w/ no internet connection :-) - all needed info present w/ no clicking required | 10:03 |
kashyap | I'd personally put it in the commit body. But I let you decide here. | 10:05 |
*** k_mouza has quit IRC | 10:06 | |
*** markvoelker has joined #openstack-nova | 10:11 | |
*** martinkennelly has joined #openstack-nova | 10:11 | |
*** k_mouza has joined #openstack-nova | 10:18 | |
sean-k-mooney | stephenfin: they are on the same topic so at least in gerrit its easy to find | 10:19 |
sean-k-mooney | oh you are folding back in the funcion | 10:20 |
sean-k-mooney | ya makes sense | 10:20 |
*** markvoelker has quit IRC | 10:21 | |
sean-k-mooney | stephenfin: by the way you realise that mypy support ducktyping | 10:27 |
stephenfin | I do. What's the context? | 10:27 |
sean-k-mooney | your responce to melwitt about not using mypy in tests | 10:28 |
sean-k-mooney | so ducktyping would not be an issue | 10:28 |
sean-k-mooney | mock proably would be | 10:28 |
stephenfin | It would if the code expects a Instance object and we're passing a dict or a string | 10:28 |
sean-k-mooney | but if we create a local class and implement the same interface it would pass | 10:28 |
stephenfin | which we do in placement | 10:28 |
stephenfin | *places | 10:28 |
stephenfin | Possibly, but that would be so much work | 10:29 |
sean-k-mooney | i know | 10:29 |
sean-k-mooney | i just mean if the thing we pass ducktypes to the real thing it will pass | 10:29 |
sean-k-mooney | if it does not we proably should not be passing it now :) | 10:29 |
sean-k-mooney | im hoping we ill graduly stop passing incorrect types in tests | 10:30 |
sean-k-mooney | im not going to hold my breath but i can hope | 10:30 |
openstackgerrit | Merged openstack/nova master: Lookup nic feature by PCI address https://review.opendev.org/739131 | 10:33 |
sean-k-mooney | :) | 10:33 |
*** Liang__ has quit IRC | 10:33 | |
*** tkajinam has quit IRC | 10:33 | |
mlycka | Hello, would anyone be kind enough to advise me on how one would go about getting a freeze exception for a spec? | 10:44 |
gibi | mlycka: what is the spec in question? | 10:46 |
gibi | mlycka: in general we will make decisions about spec freeze exception request on the today's nova meeting | 10:46 |
mlycka | gibi: https://review.opendev.org/#/c/702810/ | 10:46 |
mlycka | I was told by a Mr. Yarwood to go ahead and ask for an exception | 10:46 |
lyarwood | \o ./me waves | 10:48 |
sean-k-mooney | mlycka: johnthetubaguy still had some concerns about why we are now taking a different approch then was previously apporved | 10:48 |
gibi | mlycka, lyarwood: does the sync - async problem was resolved in the spec? | 10:48 |
mlycka | gibi: I think the idea was to treat is as sync for now and see where that gets us. The spec allows for a change later on during implementation. | 10:50 |
gibi | mlycka: I will add it to the meeting agenda. but we would need johnthetubaguy and Groka to check your answers to their questions as now we have -1s on the patch | 10:51 |
mlycka | gibi: Right right, thank you. Should I contact them directly or are they likely to be present in the meeting? | 10:52 |
lyarwood | mlycka: well the os-server-external-events API that you're using is async so I don't see how this could ever be sync tbh | 10:52 |
sean-k-mooney | that api is not for enduser to invoke | 10:52 |
sean-k-mooney | its for other services | 10:53 |
lyarwood | and that's also a valid point that I missed | 10:53 |
gibi | mlycka: given that it is summer I don't think they will get back to the spec today or tomorrow | 10:53 |
mlycka | Yeah, I think a separate endpoint is probably the way to go. | 10:53 |
sean-k-mooney | mlycka: quiesing a vm shoudl be a server action | 10:53 |
sean-k-mooney | as would undoing it | 10:53 |
sean-k-mooney | like pause and unpause | 10:54 |
mlycka | sean-k-mooney: What do you mean? Yeah, it would have to be done by the VM. | 10:54 |
sean-k-mooney | https://docs.openstack.org/api-ref/compute/#servers-run-an-action-servers-action | 10:54 |
sean-k-mooney | i mean you should be extending /servers/{server_id}/action | 10:55 |
mlycka | Okay, that doesn't sound unreasonable. Any objections? | 10:57 |
*** markvoelker has joined #openstack-nova | 10:57 | |
*** ratailor has quit IRC | 10:57 | |
sean-k-mooney | i object to you using the event api :) but other then that no :) | 10:57 |
mlycka | Sure, makes sense. I won't have the time change the spec today, but I should be able to get to it tomorrow | 10:58 |
lyarwood | mlycka: would you also be able to start posting code for this? | 10:59 |
*** ociuhandu has quit IRC | 11:00 | |
mlycka | lyarwood: I mean, not tomorrow but yeah, by next week I could start producing code. | 11:00 |
lyarwood | kk | 11:00 |
*** ociuhandu has joined #openstack-nova | 11:01 | |
sean-k-mooney | mlycka: by the way the fact you used the service only external events api is proably why peopel assume this is only for use with cinder | 11:03 |
mlycka | sean-k-mooney: Might be. I think I misunderstood the use of that endpoint when I first found out about it. | 11:04 |
sean-k-mooney | ya that is why we have the warning https://docs.openstack.org/api-ref/compute/?expanded=add-associate-floating-ip-addfloatingip-action-deprecated-detail#create-external-events-os-server-external-events | 11:04 |
mlycka | Yep yep, clearly a reading comprehension failure on my part. | 11:05 |
*** sapd__x has quit IRC | 11:06 | |
*** ociuhandu has quit IRC | 11:06 | |
*** takamatsu has joined #openstack-nova | 11:06 | |
*** xek has joined #openstack-nova | 11:07 | |
*** markvoelker has quit IRC | 11:09 | |
*** jangutter_ has quit IRC | 11:11 | |
*** jangutter has joined #openstack-nova | 11:12 | |
*** xek has quit IRC | 11:15 | |
openstackgerrit | sean mooney proposed openstack/os-vif master: support pyroute2 0.5.13 https://review.opendev.org/744809 | 11:21 |
openstackgerrit | sean mooney proposed openstack/nova stable/train: catch libvirt exception when nodedev not found. https://review.opendev.org/745116 | 11:24 |
*** mkrai_ has quit IRC | 11:25 | |
*** mkrai_ has joined #openstack-nova | 11:25 | |
*** brinzhang_ has joined #openstack-nova | 11:32 | |
*** brinzhang has quit IRC | 11:35 | |
*** ratailor has joined #openstack-nova | 11:37 | |
*** ociuhandu has joined #openstack-nova | 11:38 | |
*** artom has quit IRC | 11:42 | |
*** mkrai_ has quit IRC | 11:44 | |
*** ociuhandu has quit IRC | 11:44 | |
*** JamesBenson has quit IRC | 11:48 | |
*** k_mouza has quit IRC | 11:50 | |
*** martinkennelly has quit IRC | 12:00 | |
*** xek has joined #openstack-nova | 12:00 | |
*** raildo has joined #openstack-nova | 12:01 | |
*** artom has joined #openstack-nova | 12:02 | |
*** udesale_ has joined #openstack-nova | 12:03 | |
*** k_mouza has joined #openstack-nova | 12:03 | |
*** udesale has quit IRC | 12:05 | |
*** bbowen has quit IRC | 12:07 | |
*** rcernin has joined #openstack-nova | 12:08 | |
*** nweinber has joined #openstack-nova | 12:17 | |
*** ralonsoh has quit IRC | 12:26 | |
*** ralonsoh has joined #openstack-nova | 12:26 | |
*** ociuhandu has joined #openstack-nova | 12:28 | |
*** bbowen has joined #openstack-nova | 12:32 | |
artom | Dammit, we never updated the emulator thread policy flavor extra spec docs following the CPU resources patches: https://docs.openstack.org/nova/ussuri/user/flavors.html | 12:35 |
artom | (Scroll down to Emulator threads policy) | 12:36 |
*** dave-mccowan has joined #openstack-nova | 12:36 | |
sean-k-mooney | artom: well its still technically correct | 12:36 |
sean-k-mooney | artom: it just does not mention that the second half is only true | 12:36 |
artom | Only if cpu_dedicated_set isn't also set | 12:36 |
sean-k-mooney | if you use vcpu_pin_set | 12:36 |
sean-k-mooney | correct | 12:36 |
artom | And what's the point of documenting the old legacy behavior | 12:37 |
sean-k-mooney | vcpu_pin_set is deprecated not removed and we default to the legacy behavior | 12:37 |
artom | I suppose | 12:37 |
artom | Still, stuff is definitely missing | 12:37 |
sean-k-mooney | stephen can update it when he removes vcpu_pin_set in a few weeks | 12:37 |
sean-k-mooney | i guess you could update them to make it clear for train and ussuri before that | 12:38 |
sean-k-mooney | but its not going to be very useful on master since it wont matter when victoria is released | 12:39 |
artom | True | 12:43 |
sean-k-mooney | i guess you could look at it as motivation | 12:43 |
sean-k-mooney | if you want to impove it better do it quick before its gone | 12:43 |
*** efried has quit IRC | 12:45 | |
artom | I need waaaay stronger motivation than that :P | 12:46 |
artom | Improving something only for it to disappear is not motivating :) | 12:46 |
*** mkrai has joined #openstack-nova | 12:46 | |
artom | I realize that taken to the extreme I've just endorsed nihilism, but... | 12:46 |
*** efried has joined #openstack-nova | 12:49 | |
*** ratailor has quit IRC | 12:49 | |
openstackgerrit | sean mooney proposed openstack/nova master: remove deprecated schduler filters https://review.opendev.org/744800 | 12:50 |
*** links has joined #openstack-nova | 12:52 | |
*** jraju__ has quit IRC | 12:53 | |
lyarwood | can I just edit https://wiki.openstack.org/wiki/Meetings/Nova with agenda items for todays meeting? | 13:10 |
* lyarwood always forgets what the process is upstream | 13:10 | |
stephenfin | lyarwood: yup | 13:14 |
lyarwood | stephenfin: cool done thanks | 13:15 |
*** mkrai has quit IRC | 13:21 | |
*** ralonsoh has quit IRC | 13:22 | |
*** jangutter_ has joined #openstack-nova | 13:30 | |
*** jangutter has quit IRC | 13:34 | |
openstackgerrit | Merged openstack/nova master: compute: As cellsv1 is no more fold _detach_volume back into detach_volume https://review.opendev.org/731506 | 13:37 |
*** mlycka has quit IRC | 13:37 | |
*** Luzi has joined #openstack-nova | 13:40 | |
*** rcernin has quit IRC | 13:40 | |
artom | stephenfin, we're not proposing bauzas's routed networks scheduling spec for exception? | 13:43 |
sean-k-mooney | i would have expected bauzas to do that | 13:43 |
artom | PTO | 13:44 |
*** ralonsoh has joined #openstack-nova | 13:44 | |
sean-k-mooney | oh then yes we should | 13:44 |
sean-k-mooney | he was hoping that gibi would readd his +2 on monday | 13:44 |
sean-k-mooney | i think | 13:44 |
artom | I don't recall anyone except stephenfin being +2 | 13:45 |
kashyap | lyarwood: Hey, do you remember: when you cold-migrate an instance, will it in the end do a 'rebuild' on the destination? (/me goes to check the code) | 13:45 |
gibi | sean-k-mooney: ohh, I missed that | 13:45 |
sean-k-mooney | artom: gibi was +2 on v4 | 13:45 |
sean-k-mooney | gibi: we were ment to ping you to remind you, bauzas asked us too since he is on PTO | 13:46 |
sean-k-mooney | i mean we can still request an exception it technically needs one | 13:46 |
gibi | I will check it shortly and if I'm OK with it then I will ask for an exception on the todays meeting | 13:46 |
lyarwood | kashyap: resize | 13:55 |
*** jangutter has joined #openstack-nova | 13:55 | |
lyarwood | kashyap: https://github.com/openstack/nova/blob/9ecefeb836964c52a5a2969b15c82b11c51d32ab/nova/api/openstack/compute/migrate_server.py#L76-L77 | 13:55 |
artom | lyarwood, could you revisit https://review.opendev.org/#/c/729763/11 when you get a chance? I didn't actually address any of your feedback, but hopefully my reasons make sense to you | 13:57 |
artom | stephenfin, ^^ same request, though I did fix the nit | 13:58 |
*** jangutter_ has quit IRC | 13:58 | |
*** mkrai has joined #openstack-nova | 13:58 | |
lyarwood | artom: yup was waiting for my changes to actually land before I went back to it | 14:01 |
lyarwood | artom: I'll hit it this afternoon | 14:01 |
*** artom has quit IRC | 14:05 | |
*** artom has joined #openstack-nova | 14:05 | |
sean-k-mooney | anyone rememebr off the top of there head when placment became required? | 14:10 |
sean-k-mooney | was it pike? | 14:10 |
lyarwood | sean-k-mooney: ocata I think | 14:11 |
*** mkrai has quit IRC | 14:11 | |
sean-k-mooney | cool and prefilters were added in rocky right | 14:12 |
*** mkrai has joined #openstack-nova | 14:12 | |
*** brinzhang0 has joined #openstack-nova | 14:12 | |
*** brinzhang_ has quit IRC | 14:15 | |
sean-k-mooney | yes they were. | 14:16 |
*** xek has quit IRC | 14:17 | |
*** mvorwerk has quit IRC | 14:23 | |
*** mvorwerk has joined #openstack-nova | 14:25 | |
*** links has quit IRC | 14:27 | |
*** mvorwerk has quit IRC | 14:32 | |
*** links has joined #openstack-nova | 14:34 | |
*** links has quit IRC | 14:34 | |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: mypy: Add type annotations https://review.opendev.org/745162 | 14:40 |
openstackgerrit | Stephen Finucane proposed openstack/os-vif master: mypy: Extend type annotations to 'vif_plug_noop' https://review.opendev.org/745163 | 14:40 |
*** psachin has quit IRC | 14:43 | |
openstackgerrit | Merged openstack/os-vif master: support pyroute2 0.5.13 https://review.opendev.org/744809 | 14:45 |
*** dklyle has joined #openstack-nova | 14:51 | |
*** suryasingh has joined #openstack-nova | 14:54 | |
*** belmoreira has quit IRC | 15:06 | |
suryasingh | Hello Team | 15:12 |
suryasingh | Sorry to interrupt in middle of your work. | 15:12 |
suryasingh | I have a query about downsizing the nova instance. | 15:12 |
suryasingh | When I downsize with —poll status comes out to be Finished. But confirming the the same fails with below error. | 15:12 |
suryasingh | ERROR (Conflict): Cannot 'confirmResize' instance eaff75ba-1e16-4096-8882-33028d8b97fd while it is in vm_state active (HTTP 409) (Request-ID: req-71f53b9c-d01d-4bfe-a22c-b62401545f3c) | 15:12 |
suryasingh | So I just wanted to confirm, if it’s feasible operation from nova side and if not, is there any historical reason of it to not support downsize operation ? | 15:12 |
*** mlavalle has joined #openstack-nova | 15:19 | |
*** Luzi has quit IRC | 15:22 | |
suryasingh | melwitt: dansmith lyarwood gmann efried would be very thankful if any of you respond to my query. | 15:26 |
gibi | suryasingh: do you try to downsize the disk in the flavor? | 15:27 |
suryasingh | @gibi whole flavor including disk. | 15:27 |
lyarwood | yeah I didn't think we supported down sizing tbh | 15:28 |
lyarwood | we can't downsize the disk at least | 15:28 |
*** jangutter_ has joined #openstack-nova | 15:28 | |
gibi | the resize action fails if you try to decrease the disk size with 'Resize error: Unable to resize disk down.' | 15:28 |
gibi | the cpu and memory can be downsized | 15:28 |
sean-k-mooney | you can downsize the ram or cpus | 15:29 |
sean-k-mooney | but not the disk | 15:29 |
sean-k-mooney | as that would loose data potentially | 15:29 |
sean-k-mooney | oh gibi just said that | 15:29 |
* gibi even tried that! :) | 15:30 | |
suryasingh | gibi: sean-k-mooney lyarwood understood. Thanks for responding to query. | 15:30 |
*** jangutter has quit IRC | 15:31 | |
stephenfin | Does anyone care to sanity check this idea for me? I end up wanting the feature every time I have to use osc-placement (along with auto-translation of RP names to UUIDs, but that's more work) https://review.opendev.org/#/c/743976/ | 15:37 |
stephenfin | As an aside, there are a quite a few patches queued up for review at https://review.opendev.org/#/q/is:open+project:openstack/osc-placement | 15:38 |
*** mkrai has quit IRC | 15:44 | |
gibi | nova meeting starts in 10 minutes on #openstack-meeting-3 | 15:50 |
*** penick has joined #openstack-nova | 15:54 | |
gibi | stephenfin: https://review.opendev.org/#/c/743976/ work for me. Also I can pay (with some review time) for the RP name -> uuid translation in the client | 15:57 |
*** udesale_ has quit IRC | 16:01 | |
*** penick has quit IRC | 16:03 | |
*** penick has joined #openstack-nova | 16:04 | |
sean-k-mooney | gibi: i the neutron docs are confusing on the ip allocation policy. i think i saw one place refecne that neutorn would do it automaticlly but i know when it was first added it needed to be set manually | 16:09 |
gibi | sean-k-mooney: I don't have my mutlisegment test env working so I cannot check right now but as far as I remember I had to set it manually | 16:10 |
*** iurygregory has quit IRC | 16:10 | |
*** iurygregory has joined #openstack-nova | 16:11 | |
sean-k-mooney | ok nova does not actully support multisegment prodier netwroks correctly | 16:12 |
sean-k-mooney | if you try to use them it will fail in at least two cases | 16:12 |
sean-k-mooney | sriov | 16:12 |
sean-k-mooney | and if one host has 2 phsynets it wont work right | 16:13 |
sean-k-mooney | also numa_aware_vswitch or anything that depens on the phsynet wont work properly | 16:13 |
sean-k-mooney | but ya that as seperate problem | 16:13 |
gibi | I guess I never tried with two physnets then | 16:15 |
*** nightmare_unreal has quit IRC | 16:16 | |
*** hamalq has joined #openstack-nova | 16:17 | |
sean-k-mooney | gibi: we just grap the first physnet form the list in nova | 16:17 |
sean-k-mooney | so if a network has more then one phsynet then there is no guarenttee that it will be correct. | 16:18 |
sean-k-mooney | the current hack that existsi assuem that we will have at most 1 provider network and 1 tunnel network but not multipele provider networks | 16:19 |
*** hamalq has quit IRC | 16:19 | |
*** hamalq has joined #openstack-nova | 16:20 | |
gibi | would it make sense to have two provide segment in the same network? | 16:23 |
*** dougsz has quit IRC | 16:29 | |
sean-k-mooney | yes that is what the routed provider netwrok feature in neturon is for | 16:40 |
sean-k-mooney | it uses physnets to map hosts to segments | 16:40 |
sean-k-mooney | there is no other way to map host to segments that i have found | 16:40 |
*** k_mouza has quit IRC | 16:45 | |
*** gyee has joined #openstack-nova | 16:51 | |
*** bbowen has quit IRC | 16:54 | |
*** JamesBenson has joined #openstack-nova | 16:56 | |
openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [goal] migrate testing to ubuntu focal https://review.opendev.org/738130 | 16:58 |
gmann | sean-k-mooney: ^^ need re Approved | 16:59 |
sean-k-mooney | already done | 16:59 |
gmann | thanks | 16:59 |
sean-k-mooney | gmann: also nova core technically are os-vif cores although we bacsically just ask you to use your judgement. if you feel comfortably approving you can if not then you dont have too :) | 17:00 |
sean-k-mooney | os-vif review is totally optional | 17:00 |
*** derekh has quit IRC | 17:01 | |
gmann | sean-k-mooney: sure. as long as you are on irc (which is always :p) i am ok to bother you | 17:02 |
sean-k-mooney | yep happy to be pinged about os-vif | 17:03 |
*** dtantsur is now known as dtantsur|afk | 17:03 | |
sean-k-mooney | oh i got the github "Arctic Code Vault Contributor" badge :) that is cool now in 1000 year peopel will see my terrible code and be equally stumped as we are now to how that ever worked :) | 17:05 |
sean-k-mooney | https://archiveprogram.github.com/ | 17:05 |
sean-k-mooney | i wonder what version of nova the archived | 17:05 |
sean-k-mooney | oh it was master from 02/02/2020 | 17:07 |
sean-k-mooney | huh that a pallandrome im sure they did that intentionally | 17:08 |
gibi | ohh, that was my brithday too :) | 17:08 |
sean-k-mooney | nice | 17:11 |
*** penick has quit IRC | 17:19 | |
*** martinkennelly has joined #openstack-nova | 17:20 | |
*** k_mouza has joined #openstack-nova | 17:21 | |
*** k_mouza has quit IRC | 17:26 | |
*** priteau has quit IRC | 17:27 | |
* gibi ends for today | 17:48 | |
lyarwood | \o | 17:52 |
*** tesseract has quit IRC | 17:53 | |
openstackgerrit | Merged openstack/nova-specs master: Routed network scheduling spec https://review.opendev.org/733703 | 17:54 |
*** grandchild has joined #openstack-nova | 18:00 | |
*** ralonsoh has quit IRC | 18:06 | |
*** artom has quit IRC | 18:12 | |
*** artom has joined #openstack-nova | 18:13 | |
*** k_mouza has joined #openstack-nova | 18:19 | |
*** artom has quit IRC | 18:23 | |
*** k_mouza has quit IRC | 18:23 | |
*** k_mouza has joined #openstack-nova | 18:29 | |
*** gmann is now known as gmann_afk | 18:35 | |
*** artom has joined #openstack-nova | 18:44 | |
lyarwood | does anyone know what the file paths for file backed memory look like in Nova? | 18:49 |
lyarwood | is it specific to the instance? | 18:49 |
*** k_mouza has quit IRC | 18:51 | |
sean-k-mooney | lyarwood: yes they are | 18:55 |
sean-k-mooney | they are generated by libvirt | 18:55 |
lyarwood | wonderful | 18:55 |
sean-k-mooney | with one per numa node | 18:55 |
sean-k-mooney | the are in the instance directory | 18:56 |
sean-k-mooney | why do you ask | 18:56 |
sean-k-mooney | thats one per guest numa node by the way | 18:56 |
lyarwood | oh no reason | 18:56 |
sean-k-mooney | there si a libvirt config option memory_backing_dir that you can set to chooese where to put them | 18:57 |
*** vishalmanchanda has quit IRC | 18:57 | |
*** k_mouza has joined #openstack-nova | 18:58 | |
*** avolkov has quit IRC | 18:59 | |
*** k_mouza has quit IRC | 19:08 | |
*** artom has quit IRC | 19:11 | |
*** artom has joined #openstack-nova | 19:14 | |
artom | stephenfin, so you're saying you'll edit out the typing stuff when you backport https://review.opendev.org/#/c/744958/ ? | 19:14 |
artom | I mean, I'd have done it right away by doing 2 patches, but w/e works | 19:14 |
stephenfin | artom: Yup, though only because I have to for Train (it's not Python 3-only) | 19:16 |
stephenfin | If it was, I'd leave them since they're a noop until mypy is enabled for that file (via mypy-files.txt) | 19:16 |
artom | stephenfin, ack, I'll remove my -1 then. I still don't fully get the problem/fix though :P | 19:17 |
stephenfin | a second patch is also doable, but I find the type hints help understand what's getting passed around | 19:17 |
stephenfin | I'm replying with that now | 19:17 |
artom | Yeah, I can understand that (the typing hints helping) | 19:17 |
stephenfin | artom: comments left. Let me know if that makes sense | 19:19 |
*** tosky has quit IRC | 19:20 | |
artom | stephenfin, ah, I think I get it - we don't "delta" the usage, we rebuild it from scratch every time. So if a certain migration/instance doesn't get counted as "tracked", it'll effectively unpin its PCPUs | 19:20 |
stephenfin | Yup, exactly | 19:21 |
artom | OK, great, so that's 1 hour to understand the concept. | 19:21 |
artom | Now to review the actual code :P | 19:21 |
artom | Need to pick up food first though | 19:22 |
stephenfin | TBH, that whole periodic task feels racy and generally wrong to me, but it does state that it's intention is to blow everything away and start from scratch so... | 19:22 |
stephenfin | yet another thing to add my "tech debt I really need to get back to" list, I guess | 19:22 |
artom | It's definitely not the first race in there | 19:23 |
artom | See: https://review.opendev.org/#/c/684409/ | 19:23 |
stephenfin | For sure. I decided not to start listing the related bugs of the related bugs, because it started getting ridiculous | 19:23 |
stephenfin | concurrency is hard | 19:24 |
artom | Yeah, chill with that, you trying to one-up George RR Martin or something? | 19:24 |
stephenfin | Try to write an explanation for https://review.opendev.org/744021 (with other nova devs that don't have this specific libvirt CPU pinning knowledge in mind) and let me know how you get on :P | 19:26 |
* stephenfin goes back to soldering stuff | 19:29 | |
sean-k-mooney | stephenfin: speaking about racy periodics | 19:44 |
sean-k-mooney | its possible that we have a race between them starting in general on inital boot and the compute agent looking up it compute node uuid or maybe even creating the compute node record | 19:45 |
sean-k-mooney | i say its posible becuase im expecting a bug to be filed for http://pastebin.test.redhat.com/891237 | 19:46 |
sean-k-mooney | and that is my working theory on how we could be trying to update a compute node recored with a compute_node_id of none | 19:47 |
sean-k-mooney | oh but that one does not have have startup set.... | 19:49 |
sean-k-mooney | https://github.com/openstack/nova/blob/9ecefeb836964c52a5a2969b15c82b11c51d32ab/nova/compute/manager.py#L9723 | 19:49 |
*** bbowen has joined #openstack-nova | 19:49 | |
sean-k-mooney | and it gets the compute nodes at the start https://github.com/openstack/nova/blob/9ecefeb836964c52a5a2969b15c82b11c51d32ab/nova/compute/manager.py#L9697-L9700 | 19:49 |
sean-k-mooney | well that theroy did not last long | 19:50 |
sean-k-mooney | oh i did not read that write it was trying to insert a default consturced/empty pci deivcie object | 19:52 |
*** nweinber has quit IRC | 19:53 | |
sean-k-mooney | tommorows problem | 19:54 |
*** dave-mccowan has quit IRC | 20:19 | |
*** dave-mccowan has joined #openstack-nova | 20:20 | |
*** xek has joined #openstack-nova | 20:40 | |
*** k_mouza has joined #openstack-nova | 21:09 | |
*** k_mouza has quit IRC | 21:14 | |
*** rcernin has joined #openstack-nova | 21:21 | |
*** slaweq has quit IRC | 21:23 | |
*** rcernin has quit IRC | 21:27 | |
*** xek has quit IRC | 21:33 | |
*** rcernin has joined #openstack-nova | 21:34 | |
*** martinkennelly has quit IRC | 21:39 | |
*** rcernin has quit IRC | 21:42 | |
*** ociuhandu_ has joined #openstack-nova | 22:03 | |
*** dklyle has quit IRC | 22:04 | |
*** ociuhandu has quit IRC | 22:04 | |
*** dklyle has joined #openstack-nova | 22:09 | |
*** rcernin has joined #openstack-nova | 22:11 | |
*** hoonetorg has quit IRC | 22:15 | |
*** rcernin has quit IRC | 22:17 | |
*** gmann_afk is now known as gmann | 22:35 | |
*** rcernin has joined #openstack-nova | 22:47 | |
*** rcernin has quit IRC | 22:52 | |
*** tkajinam has joined #openstack-nova | 22:55 | |
*** mlavalle has quit IRC | 22:56 | |
*** JamesBenson has quit IRC | 23:01 | |
*** dklyle has quit IRC | 23:20 | |
*** rcernin has joined #openstack-nova | 23:20 | |
*** dklyle has joined #openstack-nova | 23:20 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!