Tuesday, 2018-12-11

*** efried has quit IRC00:05
*** efried has joined #openstack-placement00:10
*** efried has quit IRC00:14
*** efried has joined #openstack-placement00:14
*** efried has quit IRC00:15
*** tetsuro has joined #openstack-placement00:25
*** lei-zh has joined #openstack-placement01:52
*** lei-zh1 has joined #openstack-placement02:20
*** lei-zh has quit IRC02:22
*** tetsuro has quit IRC02:26
*** tetsuro has joined #openstack-placement02:48
*** mriedem_away has quit IRC04:15
*** lei-zh1 has quit IRC04:19
*** tetsuro has quit IRC05:51
*** tetsuro has joined #openstack-placement05:53
*** e0ne has joined #openstack-placement07:10
*** e0ne has quit IRC07:12
*** e0ne has joined #openstack-placement07:13
*** e0ne has quit IRC07:14
*** e0ne has joined #openstack-placement07:15
*** e0ne has quit IRC07:17
*** evrardjp has joined #openstack-placement08:02
*** helenafm has joined #openstack-placement08:15
*** takashin has left #openstack-placement08:31
*** tssurya has joined #openstack-placement08:31
belmoreiraefried_cya_jan: unfortunately when I did the Queens upgrade I was not collecting that many metrics for placement to assess the impact. We ran for some time without disabling the resource_provider_association_refresh. This time was not possible, that's why I was assuming that the number of checks also increased08:36
belmoreiraefried_cya_jan cdent: I asked for feedback in the channel last Friday before publishing this week. It can always be updated. Let me know if you don't find it fair and you would like to change something.08:41
*** rubasov has quit IRC08:53
*** e0ne has joined #openstack-placement10:12
*** tetsuro has quit IRC10:33
*** cdent has joined #openstack-placement10:43
*** tetsuro has joined #openstack-placement10:59
*** dtantsur|afk is now known as dtantsur11:18
openstackgerritChris Dent proposed openstack/os-resource-classes master: Initial automated testing for os-resource-classes  https://review.openstack.org/62436611:38
*** rubasov has joined #openstack-placement11:39
openstackgerritChris Dent proposed openstack/os-resource-classes master: Initial automated testing for os-resource-classes  https://review.openstack.org/62436611:56
openstackgerritChris Dent proposed openstack/os-resource-classes master: Rename ORDERED_CLASSES to STANDARDS  https://review.openstack.org/62436911:56
openstackgerritChris Dent proposed openstack/os-resource-classes master: Sync STANDARDS with placement/nova rc_fields  https://review.openstack.org/62437011:56
openstackgerritChris Dent proposed openstack/placement master: WIP: Use os-resource-classes in placement  https://review.openstack.org/62355612:14
*** tetsuro has quit IRC12:33
*** tetsuro has joined #openstack-placement12:33
openstackgerritMerged openstack/placement master: Add alembic version stamp capability to the DB  https://review.openstack.org/62021612:45
*** tssurya has quit IRC12:46
*** e0ne has quit IRC12:47
openstackgerritChris Dent proposed openstack/os-resource-classes master: Tune up documentation to be more useful  https://review.openstack.org/62438412:50
*** tetsuro has quit IRC12:54
cdentgibi: if you're around, this stack of os-resource-classes changes may be of interest: https://review.openstack.org/#/c/624366/13:01
gibicdent: ack13:02
cdentthanks13:02
gibicdent: bit deep in sriov and pci handling of nova so I will get back to those later13:03
cdentgibi: no huge rush. i'm sorry for your luck.13:03
gibicdent: I knew when I signed up for the bandwidth feature that at some point I have to do a deep dive in sriov13:04
cdentit will be worth it in the long run13:04
*** tssurya has joined #openstack-placement13:21
openstackgerritChris Dent proposed openstack/placement master: WIP: Use os-resource-classes in placement  https://review.openstack.org/62355613:30
openstackgerritChris Dent proposed openstack/placement master: Add python3.7 jobs  https://review.openstack.org/61069314:02
*** mriedem has joined #openstack-placement14:06
*** e0ne has joined #openstack-placement14:45
cdentedleafe: just so you're aware: https://review.openstack.org/#/c/624387/14:55
cdentI think we should have caught that in the review, but meh14:55
cdentmriedem: in your copious free time if you could put some eyes on https://review.openstack.org/#/c/624366/ , which is a stack of changes to make os-resource-classes a real thing, that would be very helpful15:12
mriedemok15:15
edleafecdent: why shouldn't the acl follow the example of os-traits? That was my reasoning. They should both then change when placement has separate governance.15:36
cdentedleafe: two reasons15:36
edleafecdent: I'm fine changing it, but the we should also change os-traits15:36
cdentI want to be able to +2 in os-resource-classes sooner, not later15:37
cdentand while os-traits should also change, the two do not have to be in sync and the only reason os-traits is the way it is because it is much older and we didn't change it yet15:37
edleafeThen I would prefer we change them both15:38
edleafeIt's weird to have them different15:38
cdentokay, will update the patch15:38
cdentedleafe:  https://review.openstack.org/62438715:48
edleafecdent: +115:51
*** e0ne has quit IRC16:36
openstackgerritJack Ding proposed openstack/nova-specs master: Select cpu model from a list of cpu models  https://review.openstack.org/62095916:37
*** sean-k-mooney has quit IRC16:43
*** sean-k-mooney has joined #openstack-placement16:49
*** tssurya has quit IRC17:08
*** helenafm has quit IRC17:18
mriedemcdent: questions in https://review.openstack.org/#/c/624366/217:32
* cdent reads17:32
cdentthanks17:35
cdentmriedem: are you in the midst of looking at the rest of that stack, or shall I go ahead and push up the fixes?17:50
mriedempush away17:54
cdentaye aye17:54
mriedemi'm doing the old drive by tab review17:54
openstackgerritChris Dent proposed openstack/os-resource-classes master: Initial automated testing for os-resource-classes  https://review.openstack.org/62436617:54
openstackgerritChris Dent proposed openstack/os-resource-classes master: Rename ORDERED_CLASSES to STANDARDS  https://review.openstack.org/62436917:54
openstackgerritChris Dent proposed openstack/os-resource-classes master: Sync STANDARDS with placement/nova rc_fields  https://review.openstack.org/62437017:54
openstackgerritChris Dent proposed openstack/os-resource-classes master: Tune up documentation to be more useful  https://review.openstack.org/62438417:55
*** dtantsur is now known as dtantsur|afk18:07
*** e0ne has joined #openstack-placement18:10
openstackgerritChris Dent proposed openstack/placement master: Add python3.7 unit test job  https://review.openstack.org/61069318:17
cdentmriedem, dansmith : easy fix https://review.openstack.org/#/c/624143/18:18
cdent(its child less easy)18:18
openstackgerritChris Dent proposed openstack/placement master: Remove dead code in objects/resource_provider.py  https://review.openstack.org/62446018:39
openstackgerritChris Dent proposed openstack/placement master: Retry _ensure_aggregates a limited number of times  https://review.openstack.org/62414418:41
mriedemcdent: up to you if you want to fix the docstring in here https://review.openstack.org/#/c/624370/218:58
cdentsigh18:58
cdentI think I meant one or more18:59
cdentbut your way works too18:59
cdentI'll let it ride for a little while and see hwere it goes18:59
cdentor meh, I'll just fix it since it is the end of the stack19:01
cdentor not19:02
* cdent is tired19:02
* cdent will get it tomorrow19:04
cdent'night all19:04
*** cdent has quit IRC19:04
*** mriedem has quit IRC20:22
*** mriedem has joined #openstack-placement20:24
*** e0ne has quit IRC21:47

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