*** threestrands has joined #openstack-nova | 00:04 | |
dansmith | efried: compute is the only thing allowed to be old | 00:13 |
---|---|---|
efried | ack | 00:13 |
dansmith | efried: the controllers are always required to enforce the version rule, so if it's something that the compute might not be able to do, it needs to check the version and refuse, downlevel, or find a compute that *can* handle it | 00:13 |
efried | right, but the point of this validation is that it's done at flavor creation time -- there's no compute in the picture. | 00:14 |
efried | sorry, you were answering my actual question, thanks. | 00:14 |
dansmith | there's compute at boot time, so you can create a newer flavor, you just need to refuse to boot an instance from it when there are no computes able to handle the request | 00:15 |
efried | right | 00:16 |
efried | we just need to be careful not to enforce a new rule that strictifies something until we're sure all the computes are upgraded. Kind of thing. | 00:17 |
*** luksky has quit IRC | 00:17 | |
dansmith | not sure what you mean exactly.. | 00:17 |
dansmith | you want to ban creation of the flavor with something if some computes are still old? that's definitely okay if it's appropriate, just not required if it's a spec that just specifies something about the instance | 00:18 |
efried | I'm saying if we decide to, I don't know, deprecate and remove the 'shared' hw:cpu_policy, it would be reasonable to think you could change the validator accordingly to bounce new flavors with hw:cpu_policy=shared. But you can't, because old computes can (and should be allowed to) still handle 'shared'. | 00:20 |
efried | (I know that's not a good example, just trying to explain the issue.) | 00:20 |
sean-k-mooney | am we proably would want a config option of some kind to to contol how deprecations are treated | 00:22 |
sean-k-mooney | so warn by default and make it an error if you opt in. | 00:22 |
sean-k-mooney | or something but i thnk we can cross that bridge when we have a example that requires it | 00:23 |
efried | Well, I definitely think we should have a granny switch that turns the whole validator on or off. | 00:24 |
efried | I don't know about per-key, if that's what you're suggesting. | 00:24 |
sean-k-mooney | i was more thinkg a strict vs compatiblity mode thing | 00:24 |
sean-k-mooney | rather then per key | 00:24 |
sean-k-mooney | if we wanted to deprecate something then we would mark it as such in the validtor and consier it an error in stict mode but just warn on it in compatibliyt mode. then if we drop support entirely we can graduate to always an error becasue we know it will not be supprotin in n+X releases | 00:26 |
sean-k-mooney | that said im not sure we need to do that | 00:26 |
sean-k-mooney | if the extra spec is not support by a compute anymore its just ignored | 00:27 |
dansmith | efried: right so that's an example where controlling the flavor definition based on compute levels would be good.. where changing the flavor impacts the way computes behave, as opposed to just something that affects the instance like a new disk controller or something | 00:27 |
efried | yikes. | 00:28 |
dansmith | yikes/ | 00:29 |
dansmith | I'm agreeing with you | 00:29 |
dansmith | I mean, I think :D | 00:29 |
*** dviroel has quit IRC | 00:31 | |
* efried bails for the day o/ | 00:36 | |
*** jhesketh has joined #openstack-nova | 00:41 | |
*** damien_r has joined #openstack-nova | 01:00 | |
*** mlavalle has quit IRC | 01:01 | |
*** gyee has quit IRC | 01:17 | |
*** KeithMnemonic1 has joined #openstack-nova | 01:25 | |
*** KeithMnemonic2 has joined #openstack-nova | 01:26 | |
*** KeithMnemonic has quit IRC | 01:29 | |
*** KeithMnemonic1 has quit IRC | 01:30 | |
*** TxGirlGeek has quit IRC | 01:40 | |
*** factor has joined #openstack-nova | 01:46 | |
*** damien_r has quit IRC | 02:20 | |
*** Liang__ has joined #openstack-nova | 02:28 | |
*** damien_r has joined #openstack-nova | 02:51 | |
*** factor has quit IRC | 03:05 | |
*** factor has joined #openstack-nova | 03:06 | |
*** tbachman has quit IRC | 03:12 | |
*** gentoora- has joined #openstack-nova | 03:26 | |
*** gentoorax has quit IRC | 03:26 | |
*** gentoora- is now known as gentoorax | 03:26 | |
*** rcernin has quit IRC | 04:18 | |
*** rcernin has joined #openstack-nova | 04:18 | |
*** udesale has joined #openstack-nova | 05:25 | |
*** links has joined #openstack-nova | 05:25 | |
*** links has quit IRC | 05:25 | |
*** tbachman has joined #openstack-nova | 06:01 | |
*** toabctl has quit IRC | 06:05 | |
*** factor has quit IRC | 06:05 | |
*** threestrands has quit IRC | 06:08 | |
*** tbachman has quit IRC | 06:09 | |
*** tbachman has joined #openstack-nova | 06:10 | |
*** toabctl has joined #openstack-nova | 06:12 | |
*** tbachman has quit IRC | 06:12 | |
*** ratailor has joined #openstack-nova | 06:16 | |
*** lpetrut has joined #openstack-nova | 06:16 | |
*** HagunKim has joined #openstack-nova | 06:24 | |
HagunKim | Hello, Nova! I have a question. Sometimes my VMs are deleted unexpectedly. What conf or log should I check?? | 06:29 |
HagunKim | Here is nova-compute log when a vm is deleted. I removed date, req. id, instance id. | 06:29 |
HagunKim | INFO nova.compute.manager [instance: intance id] Terminating instance | 06:29 |
HagunKim | INFO nova.virt.libvirt.driver [-] [instance: intance id] Instance destroyed successfully. | 06:29 |
HagunKim | INFO nova.compute.manager [-] [instance: intance id] VM Stopped (Lifecycle Event) | 06:29 |
HagunKim | INFO nova.compute.manager [instance: intance id] During sync_power_state the instance has a pending task (deleting). Skip. | 06:29 |
HagunKim | INFO nova.virt.libvirt.driver [instance: intance id] Deleting instance files /var/lib/nova/instances/instance_id_del | 06:29 |
HagunKim | INFO nova.virt.libvirt.driver [instance: intance id] Deletion of /var/lib/nova/instances/instance_id_del complete | 06:29 |
HagunKim | INFO nova.compute.manager [instance: intance id] Took 23.99 seconds to destroy the instance on the hypervisor. | 06:29 |
*** psachin has joined #openstack-nova | 06:39 | |
*** ociuhandu has joined #openstack-nova | 07:30 | |
*** dpawlik has joined #openstack-nova | 07:34 | |
*** ociuhandu has quit IRC | 07:35 | |
*** slaweq has joined #openstack-nova | 07:39 | |
*** xek has joined #openstack-nova | 07:45 | |
*** maciejjozefczyk has joined #openstack-nova | 07:57 | |
*** rpittau|afk is now known as rpittau | 08:07 | |
*** tosky has joined #openstack-nova | 08:13 | |
*** tesseract has joined #openstack-nova | 08:16 | |
*** iurygregory has joined #openstack-nova | 08:24 | |
gibi | HagunKim: with the request_id from your log I would grep the nova logs to see what other events lead to the deletion of the instance | 08:29 |
bauzas | good morning Nova | 08:29 |
*** maciejjozefczyk has quit IRC | 08:29 | |
gibi | bauzas: good morning | 08:31 |
bauzas | gibi: how are you ? I was a bit on and off last week | 08:31 |
*** maciejjozefczyk has joined #openstack-nova | 08:31 | |
gibi | bauzas: thanks I'm pretty OK, spent half of my last week downstream with helping planning some internal REST APIs. Do you still need to spend a lot of time downstream? | 08:34 |
*** tkajinam has quit IRC | 08:34 | |
bauzas | gibi: this week, nope | 08:34 |
bauzas | working on specs and the cold migration issue | 08:34 |
bauzas | (for GPUs) | 08:34 |
bauzas | if you have things to be reviewed by me, tell me | 08:34 |
openstackgerrit | Merged openstack/nova-specs master: support live migration with virtual persistent memory https://review.opendev.org/695863 | 08:35 |
gibi | bauzas: I think there is a small test issue in your placement audit https://review.opendev.org/#/c/670112 but besides that it is pretty close. And the placement fixture enahncement top of that needs a squash / respin | 08:36 |
gibi | bauzas: on my side the qos live migration needs some review. The bootom patch was commented by dansmith so there we need a re-review from dansmith but you can be the second review if you wish https://review.opendev.org/#/c/702261/ | 08:37 |
bauzas | gibi: sure, lemme know which test issue you found :) | 08:37 |
bauzas | and /me clikcs on the qos live migration series :) | 08:37 |
*** ralonsoh has joined #openstack-nova | 08:38 | |
gibi | bauzas: there is two test failure in the functional test in the placement audit patch https://834f0f74916383443a4f-1960c95d24ebbbc2bc76dbab48507616.ssl.cf1.rackcdn.com/670112/13/check/nova-tox-functional-py36/61deb16/testr_results.html | 08:38 |
gibi | bauzas: thanks | 08:39 |
gibi | bauzas: you can also hit me with patches to review | 08:39 |
bauzas | gibi: ah shit, I thought it was a functional test race | 08:39 |
bauzas | since we had issues last week | 08:39 |
bauzas | gibi: and sure, but my point is that I'd also like to review specs and changes | 08:40 |
bauzas | gibi: so if you see other changes that you'd get second eyes... | 08:40 |
gibi | bauzas: sure I will let you know | 08:41 |
*** ccamacho has joined #openstack-nova | 08:57 | |
*** tetsuro has joined #openstack-nova | 08:58 | |
*** priteau has joined #openstack-nova | 09:00 | |
*** martinkennelly has joined #openstack-nova | 09:01 | |
*** martinkennelly has quit IRC | 09:09 | |
*** maciejjozefczyk has quit IRC | 09:14 | |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | 09:34 |
*** maciejjozefczyk has joined #openstack-nova | 09:35 | |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | 09:44 |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520 | 09:48 |
*** _erlon_ has quit IRC | 10:03 | |
*** _erlon_ has joined #openstack-nova | 10:03 | |
*** ociuhandu has joined #openstack-nova | 10:08 | |
*** ociuhandu has quit IRC | 10:21 | |
*** abhishekk is now known as abhishekk|away | 10:27 | |
*** martinkennelly has joined #openstack-nova | 10:47 | |
*** ociuhandu has joined #openstack-nova | 10:55 | |
*** dviroel has joined #openstack-nova | 10:56 | |
*** dtantsur|afk is now known as dtantsur | 11:07 | |
*** martinkennelly has quit IRC | 11:07 | |
openstackgerrit | Stephen Finucane proposed openstack/os-resource-classes master: setup.cfg: Configure 'python-requires' https://review.opendev.org/704533 | 11:12 |
openstackgerrit | Stephen Finucane proposed openstack/os-traits master: setup.cfg: Configure 'python-requires' https://review.opendev.org/704534 | 11:12 |
openstackgerrit | Stephen Finucane proposed openstack/os-traits master: setup.cfg: Remove unused 'build_sphinx' configuration https://review.opendev.org/704535 | 11:12 |
openstackgerrit | Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330 | 11:17 |
*** rpittau is now known as rpittau|bbl | 11:18 | |
*** tetsuro has quit IRC | 11:19 | |
*** rm_work has quit IRC | 11:21 | |
*** rm_work has joined #openstack-nova | 11:21 | |
*** luksky has joined #openstack-nova | 11:24 | |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | 11:25 |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | 11:27 |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520 | 11:27 |
*** martinkennelly has joined #openstack-nova | 11:36 | |
*** bbowen has quit IRC | 11:45 | |
*** lpetrut has quit IRC | 11:50 | |
HagunKim | gibi: Thanks for your reply. But I found nothing with request_id. I will change log level to debug and see it again. | 12:01 |
gibi | not even in other nova services like nova api? | 12:05 |
*** mgariepy has joined #openstack-nova | 12:14 | |
bauzas | HagunKim: just use the os-instance-actions API | 12:15 |
bauzas | HagunKim: https://docs.openstack.org/api-ref/compute/#servers-actions-servers-os-instance-actions | 12:16 |
HagunKim | bauzas: Can I use that api for removed instances? | 12:19 |
bauzas | "Action information of deleted instances can be returned for requests starting with microversion 2.21." | 12:19 |
bauzas | :) | 12:19 |
bauzas | then you'll get the req-ids | 12:20 |
bauzas | so you can either ask again the same API for a specific req-id or just look at logs | 12:20 |
HagunKim | Oh I will check. Thank you! | 12:20 |
*** rcernin has quit IRC | 12:29 | |
*** ociuhandu has quit IRC | 12:29 | |
*** ociuhandu has joined #openstack-nova | 12:30 | |
stephenfin | yo, sean-k-mooney, you probably want to weigh in on https://review.opendev.org/668656 since I _think_ you're of the same mindset as me RE: not forcing people to use placement constructs in nova land | 12:33 |
sean-k-mooney | i have in the irc discussions and yes | 12:34 |
*** ociuhandu has quit IRC | 12:35 | |
sean-k-mooney | i would be oke with the placement syntax but only if we remove support for placement groups syntax form the flavor and stop doing a direct passthough to the api query effectivly | 12:35 |
sean-k-mooney | e.g. resouce:* would be ok if and only if we have a layer that parses that and then generate a different query based on the other extraspecs but i prefer the hw:* syntax | 12:36 |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found https://review.opendev.org/679247 | 12:36 |
stephenfin | my arguments slightly different to that. I'm okay with the placement syntax but I don't want it mixed with the legacy stuff. It's too confusing to reason about | 12:40 |
*** damien_r has quit IRC | 12:42 | |
alex_xu | sean-k-mooney: stephenfin, if we are going to the vision about placement syntax in the future, will it be hard to deprecated those extra specs we added? | 12:43 |
alex_xu | I'm just thinking, it is vision, agreed by some people, but it is vision, we don't know when it happend, but people afarid we add more stuff which aren't existing in that vision. just thinking of if we have road to transfer to that vision, then we continue add extra specs, then it won't be a problem | 12:45 |
sean-k-mooney | alex_xu: i guess the only way i can see moving to placement syntax workin is if we remove the group syntax form the flavor | 12:47 |
sean-k-mooney | so i for won dont ever see use moving to placemnt syntax unless that happens | 12:47 |
sean-k-mooney | if the resouces:* requests are coupled to the placemetn toplogy directly that is an upgrade issue | 12:48 |
sean-k-mooney | that said with an indirection laywer between the placment query and the flavor it would be managable | 12:49 |
alex_xu | I see | 12:49 |
sean-k-mooney | but to your question i dont think doing hw:* is a blocker to doing resouces:* in the future | 12:49 |
alex_xu | but at that time, we need to deprecated some extra sepcs like 'hw:cpu_policy', any problem we deprecated one more more extra spec for 'hw:dedicated_cpu_set'(I forget the name)? | 12:50 |
sean-k-mooney | we can support that for cpu pinning today but i would continue to recommend using hw:cpu_policy=dedicated over resocues:PCPU=x | 12:50 |
alex_xu | yea, that is what i'm thinking | 12:50 |
*** bbowen has joined #openstack-nova | 12:50 | |
alex_xu | huaqiang: efried dansmith stephenfin ^ | 12:51 |
*** nweinber has joined #openstack-nova | 12:51 | |
sean-k-mooney | ya we would have to deprecate both toghter but 1 vs 2 extra specs is not really a big differece | 12:51 |
alex_xu | it is time pull everybody togetther again :) | 12:51 |
sean-k-mooney | we would still have to do an online data migration to update the embeded flavor or force the operator to resize every vm in either case | 12:52 |
sean-k-mooney | that is when we remove support for hw:cpu_policy if we ever do | 12:53 |
alex_xu | ah, resize, that is terrible | 12:53 |
*** bbowen_ has joined #openstack-nova | 12:53 | |
sean-k-mooney | ya its the only way to update the embeded flavor today | 12:53 |
sean-k-mooney | which is why i dont like the idea of useing the placement syntax direcly as it is copled to the placement toplogy and if that chagne we need to either do a resize or an online data migration | 12:54 |
*** munimeha1 has joined #openstack-nova | 12:54 | |
alex_xu | yea, i see | 12:55 |
sean-k-mooney | given we want to move pcpus under numa nodes in the next release or two i worry about relying on the placment syntax currently as that will be a toplogy change | 12:56 |
*** bbowen has quit IRC | 12:56 | |
alex_xu | and we don't know it will happen in next release | 12:58 |
sean-k-mooney | ya we dont bauzas has reporposed it for this release https://review.opendev.org/#/c/552924/ but we have not really been reviewing it this cycle | 13:00 |
sean-k-mooney | so i think its unlikly to happen until next cycle at the earliest | 13:01 |
*** portdirect has quit IRC | 13:01 | |
*** portdirect has joined #openstack-nova | 13:02 | |
*** jistr has left #openstack-nova | 13:02 | |
*** rpittau|bbl is now known as rpittau | 13:04 | |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520 | 13:04 |
*** maciejjozefczyk has quit IRC | 13:04 | |
*** psachin has quit IRC | 13:06 | |
*** munimeha1 has quit IRC | 13:07 | |
*** munimeha1 has joined #openstack-nova | 13:08 | |
*** mgariepy has quit IRC | 13:09 | |
*** ociuhandu has joined #openstack-nova | 13:10 | |
*** damien_r has joined #openstack-nova | 13:11 | |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520 | 13:13 |
openstackgerrit | Kobi Samoray proposed openstack/nova master: Support large network queries towards neutron https://review.opendev.org/704520 | 13:13 |
*** eharney has joined #openstack-nova | 13:17 | |
* alex_xu backs to the 'keep-stay-at-home' vacation | 13:19 | |
*** Luzi has joined #openstack-nova | 13:20 | |
*** dtantsur is now known as dtantsur|lunch | 13:20 | |
*** ociuhandu has quit IRC | 13:20 | |
*** udesale_ has joined #openstack-nova | 13:22 | |
*** maciejjozefczyk has joined #openstack-nova | 13:22 | |
*** udesale has quit IRC | 13:25 | |
*** ratailor has quit IRC | 13:25 | |
*** mgariepy has joined #openstack-nova | 13:27 | |
openstackgerrit | Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330 | 13:31 |
openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | 13:32 |
openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | 13:36 |
*** psachin has joined #openstack-nova | 13:39 | |
bauzas | sean-k-mooney: alex_xu: comments are welcome on the spec | 13:40 |
bauzas | and I can provide a new revision given gibi's comments | 13:40 |
openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | 13:41 |
*** mgariepy has quit IRC | 13:52 | |
*** ociuhandu has joined #openstack-nova | 13:53 | |
*** ociuhandu has quit IRC | 14:00 | |
*** tkajinam has joined #openstack-nova | 14:00 | |
*** nweinber has quit IRC | 14:07 | |
sean-k-mooney | bauzas: you spec mainly look at the queries sent to placement not how that is expressed in the flavor but i have conserns with how resouce request are represent in the flavor and i dont really think you should proceed untill hugepage/mempages are supported | 14:08 |
sean-k-mooney | im somewhat flexible on the last point but im not a fan of moving memory_mb to the numa node as it stands | 14:08 |
*** dtantsur|lunch is now known as dtantsur | 14:12 | |
*** gregwork has joined #openstack-nova | 14:13 | |
bauzas | sean-k-mooney: yeah I need to think about the group_policy issue | 14:17 |
sean-k-mooney | its not just group_policy although that is a major issue i dont think the group syntax shoudl be used in the flavor at all | 14:23 |
sean-k-mooney | using it between nova and placment im more ok with | 14:24 |
bauzas | sean-k-mooney: in the spec, I say that we should hide the placement queries | 14:24 |
*** pcaruana has quit IRC | 14:24 | |
bauzas | that's the consensus said so far at least | 14:24 |
*** Luzi has quit IRC | 14:26 | |
sean-k-mooney | yes that is a saving grace of the current proposal in that it is advocatign for doing translation form the existing extra specs | 14:26 |
sean-k-mooney | that said its in conflict with the mix pinned and unpind cpus in one instnace work | 14:27 |
*** jamesden_ has joined #openstack-nova | 14:28 | |
*** hamzy__ is now known as hamzy | 14:28 | |
*** jamesdenton has quit IRC | 14:29 | |
*** luksky has quit IRC | 14:31 | |
*** nweinber has joined #openstack-nova | 14:37 | |
*** tbachman has joined #openstack-nova | 14:44 | |
efried | stephenfin: whoops, we crossed in the mail, let me go back and respond to your comments before you respin please... | 14:55 |
*** Liang__ has quit IRC | 15:01 | |
*** tkajinam has quit IRC | 15:02 | |
*** jamesden_ is now known as jamesdenton | 15:07 | |
*** psachin has quit IRC | 15:07 | |
efried | stephenfin: done | 15:09 |
efried | I don't think anything changed :) | 15:09 |
*** luksky has joined #openstack-nova | 15:10 | |
*** pcaruana has joined #openstack-nova | 15:11 | |
*** mgariepy has joined #openstack-nova | 15:14 | |
*** jamesden_ has joined #openstack-nova | 15:15 | |
*** jamesdenton has quit IRC | 15:18 | |
*** ociuhandu has joined #openstack-nova | 15:18 | |
*** jamesden_ is now known as jamesdenton | 15:23 | |
*** ociuhandu has quit IRC | 15:24 | |
*** ociuhandu has joined #openstack-nova | 15:25 | |
*** mriedem has joined #openstack-nova | 15:33 | |
*** ociuhandu has quit IRC | 15:37 | |
*** rchurch has quit IRC | 15:39 | |
*** rchurch has joined #openstack-nova | 15:41 | |
*** Sundar has joined #openstack-nova | 15:48 | |
*** jmlowe has joined #openstack-nova | 15:51 | |
*** TxGirlGeek has joined #openstack-nova | 15:53 | |
*** Sundar has quit IRC | 15:56 | |
*** Sundar has joined #openstack-nova | 16:03 | |
*** ociuhandu has joined #openstack-nova | 16:08 | |
*** martinkennelly has quit IRC | 16:12 | |
*** ociuhandu has quit IRC | 16:12 | |
*** psachin has joined #openstack-nova | 16:13 | |
*** mlavalle has joined #openstack-nova | 16:13 | |
Sundar | Hello dansmith | 16:14 |
dansmith | Sundar: hi, I'm on a call right now | 16:15 |
*** sapd1_x has joined #openstack-nova | 16:17 | |
*** maciejjozefczyk has quit IRC | 16:20 | |
*** gyee has joined #openstack-nova | 16:20 | |
*** udesale_ has quit IRC | 16:22 | |
*** sapd1_x has quit IRC | 16:35 | |
*** gregwork has quit IRC | 16:36 | |
*** martinkennelly has joined #openstack-nova | 16:52 | |
*** spatel has joined #openstack-nova | 16:58 | |
*** tesseract has quit IRC | 17:01 | |
*** iurygregory has quit IRC | 17:04 | |
*** psachin has quit IRC | 17:15 | |
*** artom has quit IRC | 17:22 | |
*** artom has joined #openstack-nova | 17:22 | |
*** artom has quit IRC | 17:27 | |
*** martinkennelly has quit IRC | 17:28 | |
*** artom has joined #openstack-nova | 17:29 | |
*** mgariepy has quit IRC | 17:31 | |
*** Sundar has quit IRC | 17:32 | |
*** jmlowe has quit IRC | 17:36 | |
*** jmlowe has joined #openstack-nova | 17:37 | |
*** ociuhandu has joined #openstack-nova | 17:42 | |
*** jrosser has quit IRC | 17:42 | |
*** jrosser has joined #openstack-nova | 17:43 | |
*** spatel has quit IRC | 17:45 | |
*** igordc has joined #openstack-nova | 17:46 | |
*** ociuhandu_ has joined #openstack-nova | 17:46 | |
*** ociuhandu has quit IRC | 17:47 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: api: Add support for extra spec validation https://review.opendev.org/704643 | 17:49 |
stephenfin | efried: ^ | 17:49 |
efried | ack. You removing the prototype stuff from the spec? | 17:50 |
stephenfin | efried: Assuming you're not against the general idea I have in that spec, it might be worth skimming that patch to see if it addresses your concerns (note the TODOs to do microversion stuff) | 17:50 |
stephenfin | and if it looks good, I'll remove the prototype stuff and make it more handwavey :) | 17:50 |
stephenfin | well, I'll remove that anyway but the handwavey stuff will reflect what I'm going to actually do | 17:51 |
efried | cool | 17:51 |
*** ociuhandu_ has quit IRC | 17:51 | |
*** KeithMnemonic2 has quit IRC | 17:53 | |
*** maciejjozefczyk has joined #openstack-nova | 17:55 | |
*** dustinc has quit IRC | 17:58 | |
*** dustinc has joined #openstack-nova | 17:58 | |
*** derekh has quit IRC | 18:00 | |
*** priteau has quit IRC | 18:01 | |
*** jmlowe has quit IRC | 18:03 | |
*** dtantsur is now known as dtantsur|afk | 18:07 | |
*** jmlowe has joined #openstack-nova | 18:19 | |
*** Li_Liu has quit IRC | 18:32 | |
*** Li_Liu has joined #openstack-nova | 18:33 | |
*** rpittau is now known as rpittau|afk | 18:40 | |
*** jmlowe has quit IRC | 18:58 | |
*** nweinber has quit IRC | 19:14 | |
*** maciejjozefczyk has quit IRC | 19:20 | |
*** martinkennelly has joined #openstack-nova | 19:41 | |
*** ociuhandu has joined #openstack-nova | 19:50 | |
*** ociuhandu has quit IRC | 19:54 | |
*** spatel has joined #openstack-nova | 19:56 | |
*** spatel has quit IRC | 20:02 | |
*** damien_r has quit IRC | 20:12 | |
*** jmlowe has joined #openstack-nova | 20:24 | |
*** martinkennelly has quit IRC | 20:27 | |
openstackgerrit | Vladyslav Drok proposed openstack/nova master: Minor improvements to cell commands https://review.opendev.org/698053 | 20:33 |
*** damien_r has joined #openstack-nova | 20:38 | |
*** damien_r has quit IRC | 20:38 | |
*** bbowen_ has quit IRC | 20:44 | |
*** nweinber has joined #openstack-nova | 20:51 | |
*** ralonsoh has quit IRC | 20:57 | |
*** damien_r has joined #openstack-nova | 21:08 | |
*** spatel has joined #openstack-nova | 21:19 | |
*** slaweq has quit IRC | 21:23 | |
*** spatel has quit IRC | 21:25 | |
*** slaweq has joined #openstack-nova | 21:29 | |
*** spatel has joined #openstack-nova | 21:29 | |
*** spatel has quit IRC | 21:29 | |
*** spatel has joined #openstack-nova | 21:29 | |
openstackgerrit | melanie witt proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate https://review.opendev.org/695220 | 21:30 |
*** slaweq has quit IRC | 21:33 | |
*** spatel has quit IRC | 21:35 | |
*** bbowen_ has joined #openstack-nova | 21:37 | |
*** jmlowe has quit IRC | 21:37 | |
*** jmlowe has joined #openstack-nova | 21:43 | |
*** slaweq has joined #openstack-nova | 21:43 | |
*** slaweq has quit IRC | 21:47 | |
*** dpawlik has quit IRC | 21:55 | |
*** rcernin has joined #openstack-nova | 22:00 | |
*** nweinber has quit IRC | 22:04 | |
*** mriedem has quit IRC | 22:07 | |
*** jmlowe has quit IRC | 22:10 | |
*** jmlowe has joined #openstack-nova | 22:26 | |
*** jmlowe has quit IRC | 22:32 | |
*** jmlowe has joined #openstack-nova | 22:32 | |
efried | stephenfin: The prototype code looks fine to me philosophically (I didn't leave any comments yet, though there are several things I noticed) but it really only answers one of the more minor issues (the non-splitting-out of namespace vs extra spec validators). | 22:34 |
efried | I think we're in agreement on most of the other pieces, but I'd like to close on the discussion of whether there will be something beyond microversion to enable/disable the validation; and if so, whether that thing can be made granular to the point of selecting whether to reject unknown keys. | 22:34 |
efried | IMO that's going to inform whether we set things up for strict validation out of the gate. | 22:34 |
*** jmlowe has quit IRC | 22:34 | |
*** jmlowe has joined #openstack-nova | 22:38 | |
*** jmlowe has quit IRC | 22:41 | |
*** jmlowe has joined #openstack-nova | 22:42 | |
*** jmlowe has quit IRC | 22:47 | |
*** luksky has quit IRC | 22:48 | |
*** jmlowe has joined #openstack-nova | 22:50 | |
*** jmlowe has quit IRC | 22:52 | |
*** jmlowe has joined #openstack-nova | 22:54 | |
*** tkajinam has joined #openstack-nova | 23:06 | |
*** slaweq has joined #openstack-nova | 23:11 | |
*** jmlowe has quit IRC | 23:13 | |
*** slaweq has quit IRC | 23:16 | |
*** jmlowe has joined #openstack-nova | 23:18 | |
*** dviroel has quit IRC | 23:23 | |
*** jmlowe has quit IRC | 23:24 | |
*** spatel has joined #openstack-nova | 23:35 | |
*** panda has left #openstack-nova | 23:37 | |
*** spatel has quit IRC | 23:40 | |
*** jmlowe has joined #openstack-nova | 23:42 | |
*** jmlowe has quit IRC | 23:54 | |
*** damien_r has quit IRC | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!