Thursday, 2019-05-30

openstackgerritMerged openstack/placement master: Add RequestGroupSearchContext class  https://review.opendev.org/65877803:36
openstackgerritMerged openstack/placement master: Move search functions to the research context file  https://review.opendev.org/66004803:46
openstackgerritMerged openstack/placement master: Cache provider ids in requested aggregates  https://review.opendev.org/66004903:50
openstackgerritMerged openstack/placement master: Remove normalize trait map func  https://review.opendev.org/66005004:07
openstackgerritMerged openstack/placement master: Move seek providers with resource to context  https://review.opendev.org/66005104:10
openstackgerritMerged openstack/placement master: Reuse cache result for sharing providers capacity  https://review.opendev.org/66005204:12
*** takashin has quit IRC05:27
*** efried1 has joined #openstack-placement06:21
*** efried has quit IRC06:23
*** efried1 is now known as efried06:23
*** takashin has joined #openstack-placement06:39
*** helenafm has joined #openstack-placement07:10
*** cdent has joined #openstack-placement08:41
*** e0ne has joined #openstack-placement09:52
cdentefried: email and spec split in progress, gonna go dark while I work on that10:53
*** cdent has quit IRC10:55
openstackgerritChris Dent proposed openstack/placement master: Spec for nested magic 1  https://review.opendev.org/66219112:13
openstackgerritChris Dent proposed openstack/placement master: Spec for can_split part of nested magic  https://review.opendev.org/65851012:15
*** jaypipes has joined #openstack-placement12:54
*** cdent has joined #openstack-placement13:02
*** mriedem has joined #openstack-placement13:02
openstackgerritChris Dent proposed openstack/placement master: DNM: See what happens with 10000 resource providers  https://review.opendev.org/65742313:07
cdentthanks edleafe13:41
openstackgerritPawel Baclawski proposed openstack/osc-placement master: Add support for 1.22 microversion  https://review.opendev.org/65178313:46
*** artom has quit IRC13:46
edleafecdent: yw13:52
efriedcdent: You going to represent in the nova meeting?14:00
cdentcan do yeah, just lost track of time14:01
*** amodi has joined #openstack-placement14:24
*** helenafm has quit IRC14:34
cdentsean-k-mooney: it's mostly from the change to get rid of the null root provider protections14:55
mriedemso not backportable14:55
cdentthose could create a huge in_ sql a call, which turns out is costly14:55
cdentmriedem: correct14:55
cdentbut there were also a few other tweaks from both me and tetsuro14:56
cdentmriedem: the OVO removal makes lots of stuff not backportable (past stein)14:56
mriedemunrelated to placement but related to perf, a relatively simple change here https://review.opendev.org/#/c/659688/ yielded this comment from a user:14:56
mriedem"Thanks, Matt! I checked this patch on the my test environment on the stable/stein  branch. I have more than 1000 virtual servers (some real, some dummy).  Previously, in the stable/rocky branch, the time to build a cluster was  about 15-20 minutes, in the Stein branch there was a regression and the  time increased to 90 minutes.  After this patch, the build time is only 2 minutes."14:56
mriedemwhich is pretty amazing14:56
cdentwow14:57
mriedemfunny what happens when you audit someone else's code and how they are using the compute APIs14:57
cdentyes14:57
cdentI reckon the recommendation for people who are curious about achieving max placement performance, they get on master and protect themselves microversion-wise14:58
*** johnthetubaguy has joined #openstack-placement14:58
mriedemi think nova on in stein is still using stein placement microversions (or even rocky)14:59
mriedemoh but train nova is using the in_tree stuff14:59
mriedemthat doesn't really matter though, should be fine to run master placement and stein nova14:59
mriedemor rocky nova14:59
cdentyes14:59
mriedemas long as you did your data migrations14:59
sean-k-mooneymriedem: was this realted to watcher15:00
sean-k-mooneyi notice you were commenting on there use of the nova apis on the ML15:00
mriedemsean-k-mooney: my patch above is in watcher yes15:01
*** artom has joined #openstack-placement15:01
mriedemin no way related to placement, just trivial changes to boost perf15:01
mriedemand i wanted to share with *someone* since i can't talk to my coworkers in china :)15:02
cdentwe hear you15:03
*** takashin has left #openstack-placement15:04
*** e0ne has quit IRC15:23
openstackgerritChris Dent proposed openstack/placement master: WIP: Implement allocation candidate mappings  https://review.opendev.org/66224515:35
cdentefried, gibi : I sort of slipped and did that ^15:35
cdentBefore I get too much further in making it right, a sanity check would be handy15:36
cdentI was doing what amount to some warmup exploration and was so far along figure may as well...15:36
efriedcdent: what is still needed to get that spec merged (having not looked in a couple days)15:38
cdentprobably for you and me to vote on it. I think I last left a comment without voting15:38
cdentit isn't something that is in the needful path, but ... it was easy15:39
cdentwould be handy of edleafe and someone else besides you, me, gibi had a browse15:39
edleafecdent: will get to it in a bit15:40
cdentcool, thanks15:40
efriedcdent: in the storyboard world, is there a way to go from patch to spec more easily than15:41
efried-> click the story link in the commit message15:41
efried-> find the spec task15:41
efried-> click the link there15:41
efried?15:41
efriedI guess that's not super much harder than it was with lp15:42
efriedoh, you have them tagged against the same topic branch, that's nice15:42
cdentI'm not aware of anythign shorter than that15:42
cdentI tend to just click on openstack/placement anywhere in gerrit and vgrep, because usually everything shows up on the first page15:42
mriedemcdent: i think they have 12 step meetings for people that do that15:45
mriedemplease don't look at my open WIP reviews...15:46
cdentI get anxiety when the number of open reviews gets over 25. That's the major source of placement velocity15:46
mriedem      nova branch reviews                       Page refreshed at 2019-05-30 15:08:45 UTC                   469 active reviews15:48
cdentThat's the major reason for extracting placement ;)15:51
cdentedleafe, efried, gibi, mriedem : candidate logo from the foundation: https://tank-binaries.s3.amazonaws.com/1672bc630c034a00844da457f216e56c.jpe Any suggestions?16:01
cdentthat's an autstralian magpie16:02
cdentor australian even16:02
cdentit looks a little bit stern to me, but maybe that's okay16:02
edleafecdent: not sure what the relevance for Placement is16:02
edleafeand its poor chopped off feet!16:02
efriedcdent: Do its feet have to be cut off?16:02
efriedthat16:02
edleafecdent: Any other choices?16:03
cdenta) they take things and place them somewhere else, b) when has the animal mattered much16:03
mriedemhttps://en.wikipedia.org/wiki/Australian_magpie#Swooping16:04
cdentnot really on the other choices. I've requested input several times on the pupdate, didn't get it, so I've assumed it was up to me16:04
mriedemapparently they are assholes16:04
cdentwfm16:04
cdentit first got back in the running as far back as berlin16:04
mriedemthe mascot doesn't matter to me, i don't put stickers on my laptop16:04
edleafes/they/we16:05
cdenthttps://anticdent.org/placement-update-19-10.html16:05
efriedcdent: I'm on board. Feet would be nice. It seems a tad monochromatic; perhaps we can get a splash of color by making the eye red16:05
cdentefried: color and refinement comes a bit later, this is first draft:16:05
cdent"Once we agree on the mascot look, we can work on coloring and type treatment"16:05
mriedemnote the image is on aws s316:05
mriedemtsk tsk16:05
efriedcdent: Okay. Then I'll just say: I'm on board. Feet would be nice. It seems a tad monochromatic; perhaps we can get a splash of color by making the eye red16:05
cdentthat's where I put it, not them, they mailed it to me16:05
mriedemoh16:05
mriedemtsk tsk :)16:05
cdentmriedem: I know, it's horrible, right, but whereas s3 is available to me super super easily and has been for years, despite my history with openstack, I don't have access to _any_ openstack clouds16:06
edleafeI'm pretty meh on logos, so this is fine. But agree with efried: feet would be better than stilts16:06
cdenti'll report the feet and color comments16:07
cdentthanks for the input16:07
cdenti suspect if I hassled mnaser I could change that, but it sure seems like it should just be _normal_ that we all have stuff as a matter of course16:07
edleafecdent: regarding the mappings, it looks sane enough16:08
mriedemcdent: heh, just busting balls, i was using aws ebs for the first time a couple of weeks ago for an education thing and it was super slick16:10
mriedemdepressingly nice16:10
* cdent nods16:10
cdentI wasn't taking you seriously, just taking advantage of the opportunity to bitch16:11
cdent(that "tank" thing predates my involvement with openstack anyway, it's what I built to occupy my period of unemployment before it)16:11
cdentis sort of a wiki16:11
mnasersomeone say openstack cloud? ha :>16:16
* cdent waves16:45
*** cdent has quit IRC16:45
efriedcdent when you're back, and others who might have an opinion: Now that Placement is its own thing, IMO it should feature on https://www.openstack.org/assets/software/projectmap/openstack-map.pdf17:25
efriedI think it should be in the "Shared Services" box, in bold, probably between keystone and glance.17:25
mriedemefried: agree19:30
mriedemthat seems (using spock voice) most logical19:30
efriedthx19:31
efriedwe still seem to have no reliable path to getting that thing updated, but it'll be nice to have consensus on whatever we send into the void19:31
edleafeefried: Not sure where, but at some session in the PTG we discussed that drawing, and the agreement was that it should be in Shared Services. I don't think anyone took it upon themselves to act on it, though19:35
efriednope19:35
*** artom has quit IRC20:59
openstackgerritEric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity  https://review.opendev.org/65789821:45
*** takashin has joined #openstack-placement21:45
openstackgerritEric Fried proposed openstack/os-traits master: Create traits for NUMA & device subtree affinity  https://review.opendev.org/65789822:00
*** efried1 has joined #openstack-placement22:19
*** efried has quit IRC22:20
*** efried1 is now known as efried22:20
efriedmriedem: This ain't your mother's nova https://review.opendev.org/#/c/657898/3//COMMIT_MSG@1622:29
mriedemsuper22:30
mriedemhopefully that never bites anyone in the ass22:30
mriedemsince traits, like diamonds, are forever22:31
*** mriedem is now known as mriedem_away22:35
openstackgerritEric Fried proposed openstack/placement master: Bump openstackdocstheme to 1.30.0  https://review.opendev.org/66235623:08
*** artom has joined #openstack-placement23:48

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