Tuesday, 2018-09-18

*** e0ne has joined #openstack-placement00:20
*** e0ne has quit IRC01:20
*** e0ne has joined #openstack-placement01:27
*** mriedem_away is now known as mriedem01:55
*** tetsuro has joined #openstack-placement02:26
*** mriedem has quit IRC02:38
*** tetsuro has quit IRC02:38
*** tetsuro has joined #openstack-placement02:38
*** e0ne has quit IRC02:40
*** e0ne has joined #openstack-placement02:48
*** e0ne has quit IRC03:05
*** tetsuro has quit IRC04:01
*** tetsuro has joined #openstack-placement04:20
*** belmoreira has joined #openstack-placement05:30
*** tetsuro has quit IRC05:49
*** tetsuro has joined #openstack-placement05:50
*** tetsuro has quit IRC06:03
*** tetsuro has joined #openstack-placement06:04
*** rubasov has quit IRC07:16
*** helenafm has joined #openstack-placement07:29
*** rubasov has joined #openstack-placement07:30
*** takashin has left #openstack-placement08:06
*** belmoreira has quit IRC08:15
*** tetsuro has quit IRC08:41
*** cdent has joined #openstack-placement08:46
*** belmoreira has joined #openstack-placement09:05
*** belmoreira has quit IRC09:10
*** belmoreira has joined #openstack-placement09:11
*** tetsuro has joined #openstack-placement09:30
openstackgerritWenran Xiao proposed openstack/nova-specs master: Move juno approved specs  https://review.openstack.org/60334409:34
*** e0ne has joined #openstack-placement10:04
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate  https://review.openstack.org/60335210:21
*** s10 has joined #openstack-placement10:48
openstackgerritTetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate  https://review.openstack.org/60335210:59
*** tetsuro has quit IRC11:00
*** tssurya has joined #openstack-placement11:26
*** e0ne has quit IRC11:51
openstackgerritMerged openstack/os-traits master: import zuul job settings from project-config  https://review.openstack.org/60137712:24
openstackgerritMerged openstack/os-traits master: switch documentation job to new PTI  https://review.openstack.org/60137812:33
openstackgerritMerged openstack/os-traits master: add python 3.6 unit test job  https://review.openstack.org/60137912:33
*** efried_off is now known as efried13:05
efriedō/13:05
*** mriedem has joined #openstack-placement13:07
gibiefried: o/13:13
*** efried is now known as efried_afak13:32
*** efried_afak is now known as efried_afk13:32
cdentmiagra13:39
*** efried_afk is now known as efried13:47
efriedIs that a headache/ED medicine?13:48
cdentefried: https://www.youtube.com/watch?v=2WZLJpMOxS413:50
efriedor the third tone of the diatonic scale as sung in Uttar Pradesh?13:51
cdentcould be13:54
openstackgerritEric Fried proposed openstack/placement master: Reduce max-complexity to 15  https://review.openstack.org/60277714:47
*** s10 has quit IRC15:04
*** belmoreira has quit IRC15:15
openstackgerritDan Smith proposed openstack/placement master: Add nova database migration script  https://review.openstack.org/60323415:16
openstackgerritDan Smith proposed openstack/placement master: Add nova database migration script  https://review.openstack.org/60323415:16
openstackgerritStephen Finucane proposed openstack/nova-specs master: Add 'disallow-isolcpus-with-overcommit-vcpus' spec  https://review.openstack.org/60341715:20
openstackgerritStephen Finucane proposed openstack/nova-specs master: Specify doctree directory  https://review.openstack.org/60342115:32
*** helenafm has quit IRC16:02
openstackgerritEric Fried proposed openstack/placement master: Fix link from root doc to contributor guide  https://review.openstack.org/60343116:05
openstackgerritMerged openstack/nova-specs master: import zuul job settings from project-config  https://review.openstack.org/60137316:12
*** sean-k-mooney has quit IRC16:41
cdentefried: could you stick a procedural -2 on https://review.openstack.org/#/c/603372/ . it's both in the wrong place and the wrong thing (read the comments for more info)16:42
*** sean-k-mooney has joined #openstack-placement16:43
efried...16:43
stephenfinI've asked this in the cpu-resources spec, but why do resource groups have to map to a resource provider?16:47
stephenfinAs opposed to a resource class (correct me if I'm fudging my terminology. I need to write a cheatsheet for this stuff)16:48
cdentstephenfin: let's turn the question around the other way: why do want to group by resource class?16:50
stephenfincdent: I want to avoid the ickyness that is this https://review.openstack.org/#/c/555081/18/specs/stein/approved/cpu-resources.rst@40016:52
cdentbut the short answer is "resource providers are things"16:52
* cdent looks16:52
cdentwhy's that icky? that says: i need my pcpus to be on different resource providers16:53
stephenfinIt seems verbose16:54
efriedcdent: done16:54
cdentthanks efried16:54
cdentstephenfin: true, but presumably something(?) is automating that verbosity16:54
stephenfinI suggested 'resources1:PCPU=4:policy=none' as "give me 4 VCPUs, I don't care where they come from"16:54
stephenfinBut that doesn't make sense where resource groups are mapped to resource classes, naturally16:55
stephenfincdent: True16:55
jaypipesstephenfin: for the record, I absolutely despise the whole resources1=PCPU:1,resources2=PCPU:1,resources3=PCPU:1 thing as well. That's unfortunately how granular request groups work with same-kind resources, though. :(16:56
stephenfinjaypipes: Aye, it is unfortunate16:57
jaypipesstephenfin: for the record, I had proposed using a different syntax than that -- something like "allow_split=PCPU" IIRC. Was rejected.16:57
stephenfincdent: Is there a spec/doc/collection of either I could look at to explore the "long reason"?16:57
jaypipesstephenfin: yeah, one minute. lemme dig it up.16:57
jaypipesstephenfin: it was a granular request spec amendment patch.16:57
cdentstephenfin: efried probably has some context too. my mind went blank across a lot of nested stuff16:58
* cdent is just here for the HTTP16:58
efriedwha, what16:58
efriedoh, we want to know why we landed on group_policy={isolate|none} versus allow_split etc?16:59
jaypipesstephenfin: https://review.openstack.org/#/c/540179/3/specs/rocky/approved/granular-resource-requests.rst16:59
efriedWasn't rejected, just wasn't done right now.16:59
efriedIt was because we were at an impasse, so we went with a compromise that got us moving.17:00
jaypipesefried: no, we rejected my proposed interface of "split_resources=PCPU" in favor of the multiple request group with a single quantityt of the same resource class.17:00
stephenfinjaypipes: Some comments left (in non-sean-k-mooney quantities) on that spec https://review.openstack.org/#/c/55508117:01
stephenfinAnd on that note, run time. Thanks for the pointers, jaypipes/cdent17:02
cdento/17:02
jaypipesstephenfin: rejected/abandoned proposal I was referring to: https://review.openstack.org/#/c/560974/17:06
cdentefried, edleafe, mriedem, jaypipes, gibi: dansmith, justifiably, would like us to decide if we care about postgresql, sooner than later, with regard to https://review.openstack.org/#/c/603234/ . Opinions?17:24
efriedcdent: I don't have the background to render an opinion there.17:24
cdentack17:25
edleafeDidn't we (Nova) state that we are no longer supporting PostgreSQL because of the lack of testing?17:25
jaypipescdent: I've added my opinion on there. I don't care if postgreSQL is "officially" supported in the migrate script.17:25
cdentyeah, noted jaypipes, just didn't want to leave you out of any ensuing chatter here17:26
cdentI suspect it is probably legit to say "do something like this, but we ain't making it"17:26
cdentbut on the other hand we've also got this in progress: https://review.openstack.org/#/c/602124/17:27
cdentlike by me and mriedem. me because I tend to experiment with postgres (because it finds sql weirdnesses) and mriedem I would guess because of huawei17:27
mriedemedleafe: no17:27
mriedemthere is no official statement17:28
mriedemwe could very easily run neutron's pg job in the nova experimental queue17:28
mriedemcdent: regarding dan's patch, i think it's fine to have a mysql-specific script17:28
mriedemthose that care about pg can add a pg-specific script17:28
cdentwfm17:29
mriedemas you might have noticed in my grenade-postgresql change, grenade is very much mysql specific17:29
mriedemand that has to be fixed as well17:29
mriedemi plan on hacking that up after doing stable reviews17:29
cdent17:29
cdentbbl18:35
*** s10 has joined #openstack-placement20:07
* cdent waves goodnight20:39
*** cdent has quit IRC20:39
*** efried has quit IRC20:44
*** efried has joined #openstack-placement20:44
*** tssurya has quit IRC21:10
openstackgerritDan Smith proposed openstack/placement master: Add nova database migration script  https://review.openstack.org/60323421:22
openstackgerritDan Smith proposed openstack/placement master: Add nova database migration script  https://review.openstack.org/60323421:25
mriedemgrenade-postgresql is passing http://logs.openstack.org/24/602124/4/experimental/grenade-postgresql/c27d517/22:42
mriedemi neutered it like a mangy dog, but hey22:42
*** tetsuro has joined #openstack-placement22:45
*** mriedem has quit IRC22:59
*** s10 has quit IRC23:36

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