Tuesday, 2020-01-28

*** threestrands has joined #openstack-nova00:04
dansmithefried: compute is the only thing allowed to be old00:13
efriedack00:13
dansmithefried: 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 it00:13
efriedright, but the point of this validation is that it's done at flavor creation time -- there's no compute in the picture.00:14
efriedsorry, you were answering my actual question, thanks.00:14
dansmiththere'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 request00:15
efriedright00:16
efriedwe 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 IRC00:17
dansmithnot sure what you mean exactly..00:17
dansmithyou 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 instance00:18
efriedI'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-mooneyam we proably would want a config option of some kind to to contol how deprecations are treated00:22
sean-k-mooneyso warn by default and make it an error if you opt in.00:22
sean-k-mooneyor something but i thnk we can cross that bridge when we have a example that requires it00:23
efriedWell, I definitely think we should have a granny switch that turns the whole validator on or off.00:24
efriedI don't know about per-key, if that's what you're suggesting.00:24
sean-k-mooneyi was more thinkg a strict vs compatiblity mode thing00:24
sean-k-mooneyrather then per key00:24
sean-k-mooneyif 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 releases00:26
sean-k-mooneythat said im not sure we need to do that00:26
sean-k-mooneyif the extra spec is not support by a compute anymore its just ignored00:27
dansmithefried: 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 something00:27
efriedyikes.00:28
dansmithyikes/00:29
dansmithI'm agreeing with you00:29
dansmithI mean, I think :D00:29
*** dviroel has quit IRC00:31
* efried bails for the day o/00:36
*** jhesketh has joined #openstack-nova00:41
*** damien_r has joined #openstack-nova01:00
*** mlavalle has quit IRC01:01
*** gyee has quit IRC01:17
*** KeithMnemonic1 has joined #openstack-nova01:25
*** KeithMnemonic2 has joined #openstack-nova01:26
*** KeithMnemonic has quit IRC01:29
*** KeithMnemonic1 has quit IRC01:30
*** TxGirlGeek has quit IRC01:40
*** factor has joined #openstack-nova01:46
*** damien_r has quit IRC02:20
*** Liang__ has joined #openstack-nova02:28
*** damien_r has joined #openstack-nova02:51
*** factor has quit IRC03:05
*** factor has joined #openstack-nova03:06
*** tbachman has quit IRC03:12
*** gentoora- has joined #openstack-nova03:26
*** gentoorax has quit IRC03:26
*** gentoora- is now known as gentoorax03:26
*** rcernin has quit IRC04:18
*** rcernin has joined #openstack-nova04:18
*** udesale has joined #openstack-nova05:25
*** links has joined #openstack-nova05:25
*** links has quit IRC05:25
*** tbachman has joined #openstack-nova06:01
*** toabctl has quit IRC06:05
*** factor has quit IRC06:05
*** threestrands has quit IRC06:08
*** tbachman has quit IRC06:09
*** tbachman has joined #openstack-nova06:10
*** toabctl has joined #openstack-nova06:12
*** tbachman has quit IRC06:12
*** ratailor has joined #openstack-nova06:16
*** lpetrut has joined #openstack-nova06:16
*** HagunKim has joined #openstack-nova06:24
HagunKimHello, Nova! I have a question. Sometimes my VMs are deleted unexpectedly. What conf or log should I check??06:29
HagunKimHere is nova-compute log when a vm is deleted. I removed date, req. id, instance id.06:29
HagunKimINFO nova.compute.manager [instance: intance id] Terminating instance06:29
HagunKimINFO nova.virt.libvirt.driver [-] [instance: intance id] Instance destroyed successfully.06:29
HagunKimINFO nova.compute.manager [-] [instance: intance id] VM Stopped (Lifecycle Event)06:29
HagunKimINFO nova.compute.manager [instance: intance id] During sync_power_state the instance has a pending task (deleting). Skip.06:29
HagunKimINFO nova.virt.libvirt.driver [instance: intance id] Deleting instance files /var/lib/nova/instances/instance_id_del06:29
HagunKimINFO nova.virt.libvirt.driver [instance: intance id] Deletion of /var/lib/nova/instances/instance_id_del complete06:29
HagunKimINFO nova.compute.manager [instance: intance id] Took 23.99 seconds to destroy the instance on the hypervisor.06:29
*** psachin has joined #openstack-nova06:39
*** ociuhandu has joined #openstack-nova07:30
*** dpawlik has joined #openstack-nova07:34
*** ociuhandu has quit IRC07:35
*** slaweq has joined #openstack-nova07:39
*** xek has joined #openstack-nova07:45
*** maciejjozefczyk has joined #openstack-nova07:57
*** rpittau|afk is now known as rpittau08:07
*** tosky has joined #openstack-nova08:13
*** tesseract has joined #openstack-nova08:16
*** iurygregory has joined #openstack-nova08:24
gibiHagunKim: with the request_id from your log I would grep the nova logs to see what other events lead to the deletion of the instance08:29
bauzasgood morning Nova08:29
*** maciejjozefczyk has quit IRC08:29
gibibauzas: good morning08:31
bauzasgibi: how are you ? I was a bit on and off last week08:31
*** maciejjozefczyk has joined #openstack-nova08:31
gibibauzas: 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 IRC08:34
bauzasgibi: this week, nope08:34
bauzasworking on specs and the cold migration issue08:34
bauzas(for GPUs)08:34
bauzasif you have things to be reviewed by me, tell me08:34
openstackgerritMerged openstack/nova-specs master: support live migration with virtual persistent memory  https://review.opendev.org/69586308:35
gibibauzas: 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 / respin08:36
gibibauzas: 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
bauzasgibi: sure, lemme know which test issue you found :)08:37
bauzasand /me clikcs on the qos live migration series :)08:37
*** ralonsoh has joined #openstack-nova08:38
gibibauzas: 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.html08:38
gibibauzas: thanks08:39
gibibauzas: you can also hit me with patches to review08:39
bauzasgibi: ah shit, I thought it was a functional test race08:39
bauzassince we had issues last week08:39
bauzasgibi: and sure, but my point is that I'd also like to review specs and changes08:40
bauzasgibi: so if you see other changes that you'd get second eyes...08:40
gibibauzas: sure I will let you know08:41
*** ccamacho has joined #openstack-nova08:57
*** tetsuro has joined #openstack-nova08:58
*** priteau has joined #openstack-nova09:00
*** martinkennelly has joined #openstack-nova09:01
*** martinkennelly has quit IRC09:09
*** maciejjozefczyk has quit IRC09:14
openstackgerritKobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found  https://review.opendev.org/67924709:34
*** maciejjozefczyk has joined #openstack-nova09:35
openstackgerritKobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found  https://review.opendev.org/67924709:44
openstackgerritKobi Samoray proposed openstack/nova master: Support large network queries towards neutron  https://review.opendev.org/70452009:48
*** _erlon_ has quit IRC10:03
*** _erlon_ has joined #openstack-nova10:03
*** ociuhandu has joined #openstack-nova10:08
*** ociuhandu has quit IRC10:21
*** abhishekk is now known as abhishekk|away10:27
*** martinkennelly has joined #openstack-nova10:47
*** ociuhandu has joined #openstack-nova10:55
*** dviroel has joined #openstack-nova10:56
*** dtantsur|afk is now known as dtantsur11:07
*** martinkennelly has quit IRC11:07
openstackgerritStephen Finucane proposed openstack/os-resource-classes master: setup.cfg: Configure 'python-requires'  https://review.opendev.org/70453311:12
openstackgerritStephen Finucane proposed openstack/os-traits master: setup.cfg: Configure 'python-requires'  https://review.opendev.org/70453411:12
openstackgerritStephen Finucane proposed openstack/os-traits master: setup.cfg: Remove unused 'build_sphinx' configuration  https://review.opendev.org/70453511:12
openstackgerritVladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent  https://review.opendev.org/70433011:17
*** rpittau is now known as rpittau|bbl11:18
*** tetsuro has quit IRC11:19
*** rm_work has quit IRC11:21
*** rm_work has joined #openstack-nova11:21
*** luksky has joined #openstack-nova11:24
openstackgerritKobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found  https://review.opendev.org/67924711:25
openstackgerritKobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found  https://review.opendev.org/67924711:27
openstackgerritKobi Samoray proposed openstack/nova master: Support large network queries towards neutron  https://review.opendev.org/70452011:27
*** martinkennelly has joined #openstack-nova11:36
*** bbowen has quit IRC11:45
*** lpetrut has quit IRC11:50
HagunKimgibi: Thanks for your reply. But I found nothing with request_id. I will change log level to debug and see it again.12:01
gibinot even in other nova services like nova api?12:05
*** mgariepy has joined #openstack-nova12:14
bauzasHagunKim: just use the os-instance-actions API12:15
bauzasHagunKim: https://docs.openstack.org/api-ref/compute/#servers-actions-servers-os-instance-actions12:16
HagunKimbauzas: 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
bauzasthen you'll get the req-ids12:20
bauzasso you can either ask again the same API for a specific req-id or just look at logs12:20
HagunKimOh I will check. Thank you!12:20
*** rcernin has quit IRC12:29
*** ociuhandu has quit IRC12:29
*** ociuhandu has joined #openstack-nova12:30
stephenfinyo, 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 land12:33
sean-k-mooneyi have in the irc discussions and yes12:34
*** ociuhandu has quit IRC12:35
sean-k-mooneyi 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 effectivly12:35
sean-k-mooneye.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:* syntax12:36
openstackgerritKobi Samoray proposed openstack/nova master: Avoid fetching metadata when no subnets found  https://review.opendev.org/67924712:36
stephenfinmy 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 about12:40
*** damien_r has quit IRC12:42
alex_xusean-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_xuI'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 problem12:45
sean-k-mooneyalex_xu: i guess the only way i can see moving to placement syntax workin is if we remove the group syntax form the flavor12:47
sean-k-mooneyso i for won dont ever see use moving to placemnt syntax unless that happens12:47
sean-k-mooneyif the resouces:* requests are coupled to the placemetn toplogy directly that is an upgrade issue12:48
sean-k-mooneythat said with an indirection laywer between the placment query and the flavor it would be managable12:49
alex_xuI see12:49
sean-k-mooneybut to your question i dont think doing hw:* is a blocker to doing resouces:* in the future12:49
alex_xubut 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-mooneywe can support that for cpu pinning today but i would continue to recommend using hw:cpu_policy=dedicated over resocues:PCPU=x12:50
alex_xuyea, that is what i'm thinking12:50
*** bbowen has joined #openstack-nova12:50
alex_xuhuaqiang: efried dansmith stephenfin ^12:51
*** nweinber has joined #openstack-nova12:51
sean-k-mooneyya we would have to deprecate both toghter but 1 vs 2 extra specs is not really a big differece12:51
alex_xuit is time pull everybody togetther again :)12:51
sean-k-mooneywe would still have to do an online data migration to update the embeded flavor or force the operator to resize every vm in either case12:52
sean-k-mooneythat is when we remove support for hw:cpu_policy if we ever do12:53
alex_xuah, resize, that is terrible12:53
*** bbowen_ has joined #openstack-nova12:53
sean-k-mooneyya its the only way to update the embeded flavor today12:53
sean-k-mooneywhich 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 migration12:54
*** munimeha1 has joined #openstack-nova12:54
alex_xuyea, i see12:55
sean-k-mooneygiven 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 change12:56
*** bbowen has quit IRC12:56
alex_xuand we don't know it will happen in next release12:58
sean-k-mooneyya we dont bauzas has reporposed it for this release https://review.opendev.org/#/c/552924/ but we have not really been reviewing it this cycle13:00
sean-k-mooneyso i think its unlikly to happen until next cycle at the earliest13:01
*** portdirect has quit IRC13:01
*** portdirect has joined #openstack-nova13:02
*** jistr has left #openstack-nova13:02
*** rpittau|bbl is now known as rpittau13:04
openstackgerritKobi Samoray proposed openstack/nova master: Support large network queries towards neutron  https://review.opendev.org/70452013:04
*** maciejjozefczyk has quit IRC13:04
*** psachin has quit IRC13:06
*** munimeha1 has quit IRC13:07
*** munimeha1 has joined #openstack-nova13:08
*** mgariepy has quit IRC13:09
*** ociuhandu has joined #openstack-nova13:10
*** damien_r has joined #openstack-nova13:11
openstackgerritKobi Samoray proposed openstack/nova master: Support large network queries towards neutron  https://review.opendev.org/70452013:13
openstackgerritKobi Samoray proposed openstack/nova master: Support large network queries towards neutron  https://review.opendev.org/70452013:13
*** eharney has joined #openstack-nova13:17
* alex_xu backs to the 'keep-stay-at-home' vacation13:19
*** Luzi has joined #openstack-nova13:20
*** dtantsur is now known as dtantsur|lunch13:20
*** ociuhandu has quit IRC13:20
*** udesale_ has joined #openstack-nova13:22
*** maciejjozefczyk has joined #openstack-nova13:22
*** udesale has quit IRC13:25
*** ratailor has quit IRC13:25
*** mgariepy has joined #openstack-nova13:27
openstackgerritVladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent  https://review.opendev.org/70433013:31
openstackgerritIvaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail  https://review.opendev.org/69917613:32
openstackgerritIvaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail  https://review.opendev.org/69917613:36
*** psachin has joined #openstack-nova13:39
bauzassean-k-mooney: alex_xu: comments are welcome on the spec13:40
bauzasand I can provide a new revision given gibi's comments13:40
openstackgerritIvaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail  https://review.opendev.org/69917613:41
*** mgariepy has quit IRC13:52
*** ociuhandu has joined #openstack-nova13:53
*** ociuhandu has quit IRC14:00
*** tkajinam has joined #openstack-nova14:00
*** nweinber has quit IRC14:07
sean-k-mooneybauzas: 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 supported14:08
sean-k-mooneyim somewhat flexible on the last point but im not a fan of moving memory_mb to the numa node as it stands14:08
*** dtantsur|lunch is now known as dtantsur14:12
*** gregwork has joined #openstack-nova14:13
bauzassean-k-mooney: yeah I need to think about the group_policy issue14:17
sean-k-mooneyits not just group_policy although that is a major issue i dont think the group syntax shoudl be used in the flavor at all14:23
sean-k-mooneyusing it between nova and placment im more ok with14:24
bauzassean-k-mooney: in the spec, I say that we should hide the placement queries14:24
*** pcaruana has quit IRC14:24
bauzasthat's the consensus said so far at least14:24
*** Luzi has quit IRC14:26
sean-k-mooneyyes that is a saving grace of the current proposal in that it is advocatign for doing translation form the existing extra specs14:26
sean-k-mooneythat said its in conflict with the mix pinned and unpind cpus in one instnace work14:27
*** jamesden_ has joined #openstack-nova14:28
*** hamzy__ is now known as hamzy14:28
*** jamesdenton has quit IRC14:29
*** luksky has quit IRC14:31
*** nweinber has joined #openstack-nova14:37
*** tbachman has joined #openstack-nova14:44
efriedstephenfin: whoops, we crossed in the mail, let me go back and respond to your comments before you respin please...14:55
*** Liang__ has quit IRC15:01
*** tkajinam has quit IRC15:02
*** jamesden_ is now known as jamesdenton15:07
*** psachin has quit IRC15:07
efriedstephenfin: done15:09
efriedI don't think anything changed :)15:09
*** luksky has joined #openstack-nova15:10
*** pcaruana has joined #openstack-nova15:11
*** mgariepy has joined #openstack-nova15:14
*** jamesden_ has joined #openstack-nova15:15
*** jamesdenton has quit IRC15:18
*** ociuhandu has joined #openstack-nova15:18
*** jamesden_ is now known as jamesdenton15:23
*** ociuhandu has quit IRC15:24
*** ociuhandu has joined #openstack-nova15:25
*** mriedem has joined #openstack-nova15:33
*** ociuhandu has quit IRC15:37
*** rchurch has quit IRC15:39
*** rchurch has joined #openstack-nova15:41
*** Sundar has joined #openstack-nova15:48
*** jmlowe has joined #openstack-nova15:51
*** TxGirlGeek has joined #openstack-nova15:53
*** Sundar has quit IRC15:56
*** Sundar has joined #openstack-nova16:03
*** ociuhandu has joined #openstack-nova16:08
*** martinkennelly has quit IRC16:12
*** ociuhandu has quit IRC16:12
*** psachin has joined #openstack-nova16:13
*** mlavalle has joined #openstack-nova16:13
SundarHello dansmith16:14
dansmithSundar: hi, I'm on a call right now16:15
*** sapd1_x has joined #openstack-nova16:17
*** maciejjozefczyk has quit IRC16:20
*** gyee has joined #openstack-nova16:20
*** udesale_ has quit IRC16:22
*** sapd1_x has quit IRC16:35
*** gregwork has quit IRC16:36
*** martinkennelly has joined #openstack-nova16:52
*** spatel has joined #openstack-nova16:58
*** tesseract has quit IRC17:01
*** iurygregory has quit IRC17:04
*** psachin has quit IRC17:15
*** artom has quit IRC17:22
*** artom has joined #openstack-nova17:22
*** artom has quit IRC17:27
*** martinkennelly has quit IRC17:28
*** artom has joined #openstack-nova17:29
*** mgariepy has quit IRC17:31
*** Sundar has quit IRC17:32
*** jmlowe has quit IRC17:36
*** jmlowe has joined #openstack-nova17:37
*** ociuhandu has joined #openstack-nova17:42
*** jrosser has quit IRC17:42
*** jrosser has joined #openstack-nova17:43
*** spatel has quit IRC17:45
*** igordc has joined #openstack-nova17:46
*** ociuhandu_ has joined #openstack-nova17:46
*** ociuhandu has quit IRC17:47
openstackgerritStephen Finucane proposed openstack/nova master: WIP: api: Add support for extra spec validation  https://review.opendev.org/70464317:49
stephenfinefried: ^17:49
efriedack. You removing the prototype stuff from the spec?17:50
stephenfinefried: 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
stephenfinand if it looks good, I'll remove the prototype stuff and make it more handwavey :)17:50
stephenfinwell, I'll remove that anyway but the handwavey stuff will reflect what I'm going to actually do17:51
efriedcool17:51
*** ociuhandu_ has quit IRC17:51
*** KeithMnemonic2 has quit IRC17:53
*** maciejjozefczyk has joined #openstack-nova17:55
*** dustinc has quit IRC17:58
*** dustinc has joined #openstack-nova17:58
*** derekh has quit IRC18:00
*** priteau has quit IRC18:01
*** jmlowe has quit IRC18:03
*** dtantsur is now known as dtantsur|afk18:07
*** jmlowe has joined #openstack-nova18:19
*** Li_Liu has quit IRC18:32
*** Li_Liu has joined #openstack-nova18:33
*** rpittau is now known as rpittau|afk18:40
*** jmlowe has quit IRC18:58
*** nweinber has quit IRC19:14
*** maciejjozefczyk has quit IRC19:20
*** martinkennelly has joined #openstack-nova19:41
*** ociuhandu has joined #openstack-nova19:50
*** ociuhandu has quit IRC19:54
*** spatel has joined #openstack-nova19:56
*** spatel has quit IRC20:02
*** damien_r has quit IRC20:12
*** jmlowe has joined #openstack-nova20:24
*** martinkennelly has quit IRC20:27
openstackgerritVladyslav Drok proposed openstack/nova master: Minor improvements to cell commands  https://review.opendev.org/69805320:33
*** damien_r has joined #openstack-nova20:38
*** damien_r has quit IRC20:38
*** bbowen_ has quit IRC20:44
*** nweinber has joined #openstack-nova20:51
*** ralonsoh has quit IRC20:57
*** damien_r has joined #openstack-nova21:08
*** spatel has joined #openstack-nova21:19
*** slaweq has quit IRC21:23
*** spatel has quit IRC21:25
*** slaweq has joined #openstack-nova21:29
*** spatel has joined #openstack-nova21:29
*** spatel has quit IRC21:29
*** spatel has joined #openstack-nova21:29
openstackgerritmelanie witt proposed openstack/nova master: Use COMPUTE_SAME_HOST_COLD_MIGRATE trait during migrate  https://review.opendev.org/69522021:30
*** slaweq has quit IRC21:33
*** spatel has quit IRC21:35
*** bbowen_ has joined #openstack-nova21:37
*** jmlowe has quit IRC21:37
*** jmlowe has joined #openstack-nova21:43
*** slaweq has joined #openstack-nova21:43
*** slaweq has quit IRC21:47
*** dpawlik has quit IRC21:55
*** rcernin has joined #openstack-nova22:00
*** nweinber has quit IRC22:04
*** mriedem has quit IRC22:07
*** jmlowe has quit IRC22:10
*** jmlowe has joined #openstack-nova22:26
*** jmlowe has quit IRC22:32
*** jmlowe has joined #openstack-nova22:32
efriedstephenfin: 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
efriedI 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
efriedIMO that's going to inform whether we set things up for strict validation out of the gate.22:34
*** jmlowe has quit IRC22:34
*** jmlowe has joined #openstack-nova22:38
*** jmlowe has quit IRC22:41
*** jmlowe has joined #openstack-nova22:42
*** jmlowe has quit IRC22:47
*** luksky has quit IRC22:48
*** jmlowe has joined #openstack-nova22:50
*** jmlowe has quit IRC22:52
*** jmlowe has joined #openstack-nova22:54
*** tkajinam has joined #openstack-nova23:06
*** slaweq has joined #openstack-nova23:11
*** jmlowe has quit IRC23:13
*** slaweq has quit IRC23:16
*** jmlowe has joined #openstack-nova23:18
*** dviroel has quit IRC23:23
*** jmlowe has quit IRC23:24
*** spatel has joined #openstack-nova23:35
*** panda has left #openstack-nova23:37
*** spatel has quit IRC23:40
*** jmlowe has joined #openstack-nova23:42
*** jmlowe has quit IRC23:54
*** damien_r has quit IRC23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!