Monday, 2018-07-02

« Sunday, 2018-07-01 Index Tuesday, 2018-07-03 »
*** tetsuro has joined #openstack-placement00:20
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6)  https://review.openstack.org/57133000:27
openstackgerritMerged openstack/nova master: manage: Remove dead code  https://review.openstack.org/57911800:43
*** edmondsw has joined #openstack-placement01:29
*** edmondsw has quit IRC01:34
*** openstackgerrit has quit IRC01:49
*** openstackgerrit has joined #openstack-placement02:16
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7)  https://review.openstack.org/57199202:16
openstackgerritChen proposed openstack/osc-placement master: Fix docstring for delete allocation method  https://review.openstack.org/57707502:35
openstackgerritChen proposed openstack/osc-placement master: Fix docstring for delete allocation method  https://review.openstack.org/57707502:39
*** tetsuro_ has joined #openstack-placement03:04
*** tetsuro has quit IRC03:04
openstackgerritDeepak Mourya proposed openstack/nova master: Resource_provider API handler does not return specific error codes  https://review.openstack.org/57942303:14
*** edmondsw has joined #openstack-placement03:17
*** edmondsw has quit IRC03:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8)  https://review.openstack.org/57199304:09
openstackgerritjichenjc proposed openstack/nova master: Adjust log style and remove ocata support  https://review.openstack.org/57646204:24
openstackgerritjichenjc proposed openstack/nova master: [doc] enhance admin/configuration/api.rst  https://review.openstack.org/56600804:25
openstackgerritjichenjc proposed openstack/nova master: remove ec2 object definitions  https://review.openstack.org/55715004:25
openstackgerritjichenjc proposed openstack/nova master: remove ec2 db functions  https://review.openstack.org/55757204:25
openstackgerritTakashi NATSUME proposed openstack/nova master: Transform aggregate.update_prop notification  https://review.openstack.org/46257604:39
openstackgerritBhagyashri Shewale proposed openstack/nova master: PCPU: Define numa dedicated CPU resource class  https://review.openstack.org/56177004:42
openstackgerritBhagyashri Shewale proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes  https://review.openstack.org/57602104:42
openstackgerritBhagyashri Shewale proposed openstack/nova master: Report PCPU to placement  https://review.openstack.org/57703804:42
*** edmondsw has joined #openstack-placement05:06
*** edmondsw has quit IRC05:11
*** e0ne has joined #openstack-placement05:25
*** e0ne has quit IRC05:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing versioned notification examples  https://review.openstack.org/57943605:44
*** tetsuro_ has quit IRC05:47
*** tetsuro has joined #openstack-placement05:53
*** e0ne has joined #openstack-placement05:54
*** peereb has joined #openstack-placement06:00
openstackgerritTakashi NATSUME proposed openstack/nova master: Update admin/flavors document  https://review.openstack.org/57306306:19
*** nicolasbock has joined #openstack-placement06:39
*** e0ne has quit IRC06:43
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/57801906:49
*** edmondsw has joined #openstack-placement06:55
*** edmondsw has quit IRC07:00
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add queued for delete to instance_mappings table.  https://review.openstack.org/56678807:14
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object  https://review.openstack.org/56679507:23
*** e0ne has joined #openstack-placement08:02
openstackgerritChason Chan proposed openstack/nova master: Use an alternative class 'MoxStubout'  https://review.openstack.org/57948208:08
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748608:28
*** edmondsw has joined #openstack-placement08:43
*** edmondsw has quit IRC08:47
*** avolkov has joined #openstack-placement08:56
*** e0ne has quit IRC08:58
*** e0ne has joined #openstack-placement09:04
*** giblet is now known as gibi09:28
gibiheads up I think we regressed some parenting rules of the RPs09:28
gibistack@ubuntu:~/nova$ git log --oneline | head -n109:28
gibiI mean https://bugs.launchpad.net/nova/+bug/177963509:29
openstackLaunchpad bug 1779635 in OpenStack Compute (nova) "placement allows RP parent loop" [Undecided,New]09:29
openstackgerritSylvain Bauza proposed openstack/nova stable/queens: libvirt: Fix the rescue race for vGPU instances  https://review.openstack.org/57950309:32
*** finucannot is now known as stephenfin09:34
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Refactor the policies to policy  https://review.openstack.org/57911309:50
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337509:50
openstackgerritBalazs Gibizer proposed openstack/nova master: Regression test for bug 1779635  https://review.openstack.org/57950810:16
openstackbug 1779635 in OpenStack Compute (nova) "placement allows RP parent loop in PUT resource_providers/{uuid}" [Undecided,New] https://launchpad.net/bugs/177963510:16
openstackgerritStephen Finucane proposed openstack/nova master: Convert 'placement_api_docs' into a Sphinx extension  https://review.openstack.org/57882610:22
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove unnecessary execute permissions of a file  https://review.openstack.org/57951110:42
openstackgerritMerged openstack/nova master: Use 'version2' when syncing placement db  https://review.openstack.org/57920011:04
openstackgerritMerged openstack/nova master: More config drive docs updates  https://review.openstack.org/57889011:04
openstackgerritMerged openstack/nova master: Clarify log in RT._update_usage_from_migration  https://review.openstack.org/55462311:05
*** takashin_ has joined #openstack-placement11:14
*** edmondsw has joined #openstack-placement11:14
*** takashin_ has left #openstack-placement11:14
*** takashin has quit IRC11:16
openstackgerritBrin Zhang proposed openstack/nova-specs master: Add support specify volume type when boot instance  https://review.openstack.org/57952011:22
*** takashin has joined #openstack-placement11:22
openstackgerritBrin Zhang proposed openstack/nova-specs master: Add support specify volume type when boot instance  https://review.openstack.org/57952011:33
*** jroll has quit IRC11:36
*** jroll has joined #openstack-placement11:40
*** tetsuro has quit IRC11:41
openstackgerritBence Romsics proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667511:45
openstackgerritBence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804311:45
openstackgerritBence Romsics proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832611:45
*** peereb has quit IRC11:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337512:02
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340112:13
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146512:13
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753412:13
*** cdent has joined #openstack-placement12:17
deepak_mouryacdent: Hi12:31
openstackgerritMerged openstack/nova master: Resource tracker: improve resource tracker periodic task  https://review.openstack.org/57605212:34
openstackgerritMerged openstack/nova master: Fix regression when listing build_requests with marker and ip filter  https://review.openstack.org/57616112:34
openstackgerritMerged openstack/nova master: Update admin/flavors document  https://review.openstack.org/57306312:34
*** jaypipes has joined #openstack-placement12:36
cdenthi deepak_mourya12:36
deepak_mouryai have pushed the initial patch for the error code ,https://review.openstack.org/#/c/579423/ you can review  the same12:37
cdentyeah, saw that, should be able to get to it a bit later in the day12:38
deepak_mouryaokay no problem12:39
jaypipesdeepak_mourya: reviewed. thank you sir!12:44
deepak_mouryajaypipes: ok thanks, let me modify12:48
deepak_mouryanaming things are very difficult.12:48
jaypipesdeepak_mourya: no worries. wait until cdent has a chance to weigh in on the naming.12:50
deepak_mouryaokay :)12:50
jaypipesdeepak_mourya: and yes, I agree and believe cdent would agree with you that naming things sucks :)12:52
jaypipesbut still very important.12:52
openstackgerritTakashi NATSUME proposed openstack/nova stable/queens: Update admin/flavors document  https://review.openstack.org/57954212:55
deepak_mouryajaypipes:  can you take a look at this nova bug if you get some time. https://bugs.launchpad.net/python-novaclient/+bug/177905712:57
openstackLaunchpad bug 1779057 in python-novaclient "'nova list --sort id' command gives unsorted list result" [Low,Confirmed] - Assigned to Deepak Mourya (mourya007)12:57
openstackgerritStephen Finucane proposed openstack/nova master: conf: Resolve Sphinx errors  https://review.openstack.org/57954513:01
jaypipesdeepak_mourya: takashin looks to have provided some good feedback on that one.13:05
cdentI would agree that naming is hard13:07
cdentWhat's my name again?13:07
cdent:w13:07
cdentoops13:07
jaypipescdent just saved his name.13:09
deepak_mouryajaypipes: ok sure13:09
openstackgerritMerged openstack/nova master: Simplify instance name generation  https://review.openstack.org/51657313:18
openstackgerritDeepak Mourya proposed openstack/nova master: Resource_provider API handler does not return specific error codes  https://review.openstack.org/57942313:27
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform missing delete notifications  https://review.openstack.org/41029713:31
*** mriedem has joined #openstack-placement13:35
openstackgerritMatt Riedemann proposed openstack/nova stable/queens: Fix regression when listing build_requests with marker and ip filter  https://review.openstack.org/57956113:39
*** sususuryashines is now known as tssurya13:48
openstackgerritBalazs Gibizer proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667513:52
openstackgerritBalazs Gibizer proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804313:52
openstackgerritBalazs Gibizer proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832613:52
gibimriedem, jaypipes: I'm +2 on the rest of the osc-placmement patch series ^^13:53
*** PapaOurs is now known as bauzas13:59
mriedemgibi: i was just commenting on the bottom one14:01
gibimriedem: sorry for pushing before asking. I've just added a missing reno14:02
mriedemok comments posted14:04
mriedemnp14:04
mriedemgibi: if you're ok with those comments, i can address and re-push14:04
gibimriedem: looking14:05
gibimriedem: reply in https://review.openstack.org/#/c/546675/34/osc_placement/tests/functional/test_resource_provider.py@22814:13
jaypipesgibi: thx, will look shortly.14:13
mriedemgibi: ah ok14:14
mriedemgibi: ok i'll update the rest and push in a bit14:14
gibimriedem: thanks14:14
mriedemtrying to figure out when parent_provider_uuid was added to GET /resource_providers/{uuid}14:14
mriedemshould be 1.14 so the api ref is just missing that14:15
mriedemfixing14:15
mriedemstephenfin: fyi, be mindful of re-authoring when you rebase https://review.openstack.org/#/c/384807/14:21
stephenfinmriedem: Damn. Sorry, jaypipes14:21
* cdent will be back later14:25
*** cdent has quit IRC14:25
jaypipesstephenfin: doesn't bother me in the slightest.14:32
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix min_version for parent_provider_uuid in responses  https://review.openstack.org/57957714:39
openstackgerritStephen Finucane proposed openstack/nova master: network: Always retrieve network information if available  https://review.openstack.org/56444414:39
openstackgerritStephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_physnet_info'  https://review.openstack.org/56444514:39
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_metadata  https://review.openstack.org/56443914:39
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks  https://review.openstack.org/57548614:39
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement  https://review.openstack.org/56444814:39
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.network_metadata  https://review.openstack.org/56444214:39
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.network_metadata  https://review.openstack.org/56445214:39
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444014:39
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_metadata field  https://review.openstack.org/56444114:39
mriedemmelwitt: can i get review on https://review.openstack.org/#/c/574488/ to close out blueprint https://blueprints.launchpad.net/nova/+spec/add-consumer-generation14:53
*** takashin has left #openstack-placement15:06
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667515:07
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804315:07
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832615:07
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14)  https://review.openstack.org/54667515:09
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16)  https://review.openstack.org/54804315:09
openstackgerritMatt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17)  https://review.openstack.org/54832615:09
mriedemimport json15:23
mriedemimport pipes15:23
mriedemSO CLOSE15:23
jaypipesmriedem: I cannot be imported, sorry. you must first export me, save to file, then import.15:25
mriedemi'm not sure you'd get back in with the travel ban15:26
mriedembeing british and all15:26
jaypipesmriedem: good point.15:27
mriedemnice: https://review.openstack.org/#/q/topic:bp/placement-osc-plugin-rocky+status:open15:32
gibimriedem: now we can start adding more top of ^^ :)15:35
mriedemheh yeah we're just caught up with queens now15:37
mriedem1.18 - 1.29 to go15:38
openstackgerritSurya Seetharaman proposed openstack/nova master: Add queued for delete to instance_mappings table.  https://review.openstack.org/56678815:40
openstackgerritSurya Seetharaman proposed openstack/nova master: [WIP] Add queued_for_delete field to InstanceMapping object  https://review.openstack.org/56679515:40
openstackgerritSurya Seetharaman proposed openstack/nova master: [POC] Graceful handling of nova-list when a cell is down  https://review.openstack.org/56778515:40
openstackgerritsahid proposed openstack/nova master: libvirt: add qemu version check when configuring mtu for network  https://review.openstack.org/57961115:48
openstackgerritMatt Riedemann proposed openstack/nova master: Use assertJsonEqual in test_pre_live_migration_volume_backed* tests  https://review.openstack.org/57961515:51
*** e0ne has quit IRC15:54
openstackgerritBalazs Gibizer proposed openstack/nova master: Prevent updating an RP's parent to form a loop  https://review.openstack.org/57961816:01
openstackgerritBalazs Gibizer proposed openstack/nova master: Prevent updating an RP's parent to form a loop  https://review.openstack.org/57961816:03
openstackgerritCorey Bryant proposed openstack/nova master: Enable connection_info refresh for new-style attachments  https://review.openstack.org/57900416:05
openstackgerritMatt Riedemann proposed openstack/nova master: Handle nested serialized json entries in assertJsonEqual  https://review.openstack.org/57961516:28
*** mriedem has quit IRC16:49
*** alex_xu has quit IRC16:50
openstackgerritMerged openstack/nova master: Fix missing versioned notification examples  https://review.openstack.org/57943616:53
*** alex_xu has joined #openstack-placement16:54
*** cdent has joined #openstack-placement17:11
*** peereb has joined #openstack-placement17:14
cdentjaypipes: did you already have a solution in mind for https://bugs.launchpad.net/nova/+bug/1779717 or it another in the "until we figure out the lifecycle and meaning of a consumer, this is weird" collection?17:15
openstackLaunchpad bug 1779717 in OpenStack Compute (nova) "No ability to update consumer's project and/or user external ID" [Medium,Triaged] - Assigned to Jay Pipes (jaypipes)17:15
jaypipescdent: coming shortly.17:19
jaypipescdent: do we have a separate bug for a consumer record being created when PUT /allocations/{consumer} fails?17:20
cdentjaypipes: I think not (mixed in with one of the others), but lemme confirm17:21
jaypipescdent: I'm going to create a separate one for tracking that, if you don't mind.17:21
cdentjaypipes: this one touches on it: https://bugs.launchpad.net/nova/+bug/177859117:22
openstackLaunchpad bug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,Triaged]17:22
cdenti do not mind17:22
jaypipescdent: yeah, I'm fixing that particular one now (patch pushing shortly) but wanted a sep bug for tracking the "delete the auto-created consumer" thing.17:29
openstackgerritSurya Seetharaman proposed openstack/nova-specs master: Handling a down cell  https://review.openstack.org/55736917:33
*** e0ne has joined #openstack-placement17:44
*** tssurya has quit IRC17:46
openstackgerritJay Pipes proposed openstack/nova master: placement: return consumer info even when no alloc  https://review.openstack.org/57965417:52
jaypipescdent: k, I did a new fix for bug 1778591 that didn't require a new microversion. I'll work on a new microversion that returns a 404 for non-existent consumers.17:52
openstackbug 1778591 in OpenStack Compute (nova) "GET /allocations/{uuid} on a consumer with no allocations provides no generation" [Medium,In progress] https://launchpad.net/bugs/1778591 - Assigned to Jay Pipes (jaypipes)17:52
efriedjaypipes: Have we settled on how consumers get deleted?  Forays into this space would be premature without that decision made IMO17:54
jaypipesefried: nope. that fix doesn't address that. see https://bugs.launchpad.net/nova/+bug/1779725 for a more appropriate place for that conversation.17:55
openstackLaunchpad bug 1779725 in OpenStack Compute (nova) "Auto-created consumer record not cleaned up after failed allocation" [Medium,Triaged] - Assigned to Jay Pipes (jaypipes)17:55
efriedjaypipes: I'm saying, 404 on nonexistent consumer should be considered alongside ^17:56
efriedOh, no, that bug isn't what I was talking about at all17:58
openstackgerritMerged openstack/nova master: Remove unnecessary execute permissions of a file  https://review.openstack.org/57951118:00
cdentjaypipes, efried: https://review.openstack.org/#/c/579654/ is legit if we've concluded that consumers will continue to exist after their allocations go away or never existed in the first place. Have we concluded that?18:01
efriednot afaiaw, swhat I was sayin18:02
* cdent nods18:03
*** peereb has quit IRC18:07
efriedthat should have been more like "afaiaa" ("...am aware").  /me can't spell18:08
*** e0ne has quit IRC18:09
* cdent leaves some notes on the review18:13
*** e0ne has joined #openstack-placement18:20
*** e0ne has quit IRC18:36
*** e0ne has joined #openstack-placement19:07
*** e0ne has quit IRC19:13
*** tssurya has joined #openstack-placement19:22
openstackgerritMerged openstack/nova master: Request only instance_uuid in ironic node list  https://review.openstack.org/53950920:04
openstackgerritMatt Rabe proposed openstack/nova master: Add destination MSP IP address to PowerVM migrate data  https://review.openstack.org/57967620:05
*** avolkov has quit IRC20:18
*** nicolasbock has quit IRC21:24
openstackgerritMatt Rabe proposed openstack/nova master: Add destination MSP IP address to PowerVM migrate data  https://review.openstack.org/57967621:27
openstackgerritMerged openstack/nova master: Regression test for bug 1779635  https://review.openstack.org/57950821:28
openstackbug 1779635 in OpenStack Compute (nova) "placement allows RP parent loop in PUT resource_providers/{uuid}" [Medium,In progress] https://launchpad.net/bugs/1779635 - Assigned to Balazs Gibizer (balazs-gibizer)21:28
jaypipescdent: what is your preferred method of giving you feedback on your article here: https://anticdent.org/some-opinions-on-openstack.html ?21:50
jaypipescdent: since, as you write in that article "This only way in which this post is actionable is as a way to have further discussion." :)21:50
cdentjaypipes: comments on it are fine, or if you dont like disqus (which is totally reasonable to not like) then a post you write that you tell me about (and I"ll link to) or something in that email thread is all fine with me21:52
cdentjaypipes: I hope I gave enough caveats to indicate that I'm waving my hands around vaguely21:53
jaypipescdent: hmm... why am I not seeing any disqus comment section on your blog...22:05
cdentjaypipes: you might be block cookies in some way, or third party javascript22:06
jaypipescdent: certainly I would have commented on disqus. I just don't see a disqus section.22:06
jaypipescdent: gah. my NoScript extension was blocking disquscdn.com.22:07
jaypipescdent: Fixed. :)22:07
jaypipescdent: sorry about that!22:07
cdentjaypipes: no worries22:07
*** cdent has quit IRC22:21
efriedjaypipes, edleafe: Is there any restriction on the UUID I can specify when creating a provider?  I mean, does it have to be a "type whatever", with certain bits set or unset, or can it be literally any 128 bits?22:40
*** tssurya has quit IRC23:05
*** edmondsw has quit IRC23:05
*** edmondsw has joined #openstack-placement23:06
openstackgerritMerged openstack/nova master: Remove mox in unit/network/test_neutronv2.py (1)  https://review.openstack.org/57201823:11
*** edmondsw has quit IRC23:11
jaypipesefried: it just needs to match a UUID signature. 8-4-4-4-12 a-f segments (IIRC)23:15
jaypipesefried: there are some giblets that, for example, hard-code things like 11111111-1111-1111-1111-111111111111 to make recognition easier if that's what you're looking for?23:16
jaypipesguh, s/giblets/gabbits :)23:16
« Sunday, 2018-07-01 Index Tuesday, 2018-07-03 »

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