Tuesday, 2017-10-31

*** ijw has quit IRC00:01
*** ijw has joined #openstack-nova00:01
*** trinaths has joined #openstack-nova00:02
*** trinaths has left #openstack-nova00:03
*** gouthamr has quit IRC00:05
*** namnh has joined #openstack-nova00:07
*** markvoelker_ has quit IRC00:11
*** namnh has quit IRC00:11
*** ijw has quit IRC00:14
*** ijw has joined #openstack-nova00:14
*** ijw has quit IRC00:19
*** yangyapeng has quit IRC00:23
*** yangyapeng has joined #openstack-nova00:23
*** markvoelker has joined #openstack-nova00:25
*** trinaths1 has joined #openstack-nova00:26
*** ircuser-1 has joined #openstack-nova00:26
openstackgerritMerged openstack/nova master: Transform keypair.import notification  https://review.openstack.org/46751400:26
*** penick has joined #openstack-nova00:27
*** yangyapeng has quit IRC00:28
*** AlexeyAbashkin has joined #openstack-nova00:29
*** AlexeyAbashkin has quit IRC00:33
*** thorst has quit IRC00:34
*** ijw has joined #openstack-nova00:35
*** sgordon has quit IRC00:36
*** owalsh_pto has quit IRC00:36
*** hongbin has joined #openstack-nova00:39
*** penick_ has joined #openstack-nova00:40
*** penick has quit IRC00:41
*** sgordon has joined #openstack-nova00:45
*** liverpooler has quit IRC00:47
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103500:48
*** owalsh_ has joined #openstack-nova00:49
*** owalsh has joined #openstack-nova00:51
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103500:51
*** owalsh- has joined #openstack-nova00:53
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103500:54
*** zhurong has joined #openstack-nova00:54
*** owalsh_ has quit IRC00:54
*** owalsh_ has joined #openstack-nova00:55
*** owalsh has quit IRC00:56
*** owalsh has joined #openstack-nova00:58
*** owalsh- has quit IRC00:59
*** namnh has joined #openstack-nova01:00
*** owalsh- has joined #openstack-nova01:01
*** owalsh_ has quit IRC01:01
melwittmriedem: looks like everything's all good for ceph ephemeral on that patch too01:01
*** phuongnh has joined #openstack-nova01:02
*** ijw has quit IRC01:03
*** ijw has joined #openstack-nova01:03
*** ijw has quit IRC01:03
*** ijw has joined #openstack-nova01:04
*** yamahata has quit IRC01:04
*** owalsh_ has joined #openstack-nova01:04
*** owalsh has quit IRC01:04
*** baoli has joined #openstack-nova01:05
*** vladikr has quit IRC01:06
*** owalsh has joined #openstack-nova01:07
*** owalsh- has quit IRC01:08
*** ijw has quit IRC01:08
*** itlinux has quit IRC01:09
*** owalsh- has joined #openstack-nova01:10
*** owalsh_ has quit IRC01:11
*** smatzek has joined #openstack-nova01:14
*** owalsh has quit IRC01:14
*** sgordon has quit IRC01:15
*** Apoorva_ has joined #openstack-nova01:19
*** vladikr has joined #openstack-nova01:19
*** yangyapeng has joined #openstack-nova01:19
*** yangyape_ has joined #openstack-nova01:20
*** yangyapeng has quit IRC01:20
*** sgordon has joined #openstack-nova01:20
*** psachin has joined #openstack-nova01:21
*** psachin has quit IRC01:22
*** Apoorva has quit IRC01:22
*** psachin has joined #openstack-nova01:22
*** Apoorva_ has quit IRC01:23
*** larainema has joined #openstack-nova01:24
*** penick_ has quit IRC01:26
mriedemmikal: not really, but maybe01:26
*** AlexeyAbashkin has joined #openstack-nova01:29
masbergood morning, I wanted to ask about how nova manages numa nodes. My problem is that nova says my instance does not fit in the numa node http://paste.openstack.org/raw/625030/01:31
*** litao__ has joined #openstack-nova01:31
*** smatzek has quit IRC01:32
*** AlexeyAbashkin has quit IRC01:33
*** amodi has quit IRC01:33
masberapologise, forgot to mention my image properties --> --property hw_cpu_policy=dedicated --property hw_cpu_thread_policy=isolate01:35
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix joins in instance_get_all_by_host  https://review.openstack.org/51168201:36
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix test_instance_get_all_by_host  https://review.openstack.org/51648601:36
*** mriedem has quit IRC01:42
*** Sukhdev has quit IRC01:44
*** annp has joined #openstack-nova01:44
*** edmondsw has quit IRC01:45
masberany idea? I really have no clue of what is wrong... my compute servers have 2 sockets with 14 cpus each which means 28cpu with hyperthreading enabled and memory is 512GB (256GB per numa node), i am asking or 20 cpus and 80GB of RAM so shouldn't be a problem to get those resources? plus I have no other vms taking space so all resources are free01:46
*** dikonoor has joined #openstack-nova01:56
*** gmann_afk is now known as gmann01:58
*** fragatina has quit IRC02:01
*** dikonoor has quit IRC02:03
*** coreywright has quit IRC02:06
*** trinaths1 has left #openstack-nova02:09
*** vladikr has quit IRC02:22
*** coreywright has joined #openstack-nova02:23
*** vladikr has joined #openstack-nova02:23
*** catintheroof has joined #openstack-nova02:24
*** Laurene has joined #openstack-nova02:29
*** dikonoor has joined #openstack-nova02:32
*** thorst has joined #openstack-nova02:35
*** salv-orl_ has joined #openstack-nova02:39
*** thorst has quit IRC02:39
*** vladikr has quit IRC02:40
*** vladikr has joined #openstack-nova02:41
*** salv-orlando has quit IRC02:42
*** catintheroof has quit IRC02:43
*** dikonoor has quit IRC02:49
*** vladikr has quit IRC02:52
*** vladikr has joined #openstack-nova02:53
*** Shunli has joined #openstack-nova02:53
*** dave-mccowan has quit IRC02:56
*** hongbin has quit IRC02:56
*** hongbin_ has joined #openstack-nova02:56
*** hongbin_ has quit IRC02:59
*** hongbin has joined #openstack-nova03:00
*** bkopilov has quit IRC03:12
*** Sukhdev has joined #openstack-nova03:15
*** vladikr has quit IRC03:20
*** vladikr has joined #openstack-nova03:21
*** hongbin has quit IRC03:22
*** esberglu_ has quit IRC03:22
*** esberglu has joined #openstack-nova03:22
*** edmondsw has joined #openstack-nova03:24
*** vladikr has quit IRC03:26
*** vladikr has joined #openstack-nova03:27
*** edmondsw has quit IRC03:29
openstackgerritYikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040603:32
*** sridharg has joined #openstack-nova03:34
*** vladikr has quit IRC03:34
*** gouthamr has joined #openstack-nova03:34
*** vladikr has joined #openstack-nova03:35
*** armax has quit IRC03:35
*** armax has joined #openstack-nova03:36
*** dikonoor has joined #openstack-nova03:43
*** fragatina has joined #openstack-nova03:49
*** fragatina has quit IRC03:49
*** fragatina has joined #openstack-nova03:50
*** fragatina has quit IRC03:50
*** fragatina has joined #openstack-nova03:51
*** markvoelker has quit IRC03:55
*** Shunli has quit IRC03:59
*** ijw has joined #openstack-nova04:04
*** udesale has joined #openstack-nova04:06
*** ijw has quit IRC04:09
*** tetsuro has joined #openstack-nova04:14
*** diga has joined #openstack-nova04:16
*** armax_ has joined #openstack-nova04:21
*** john5223 has quit IRC04:22
*** fragatina has quit IRC04:22
*** armax has quit IRC04:22
*** armax_ is now known as armax04:22
*** jpena|off has quit IRC04:23
*** john5223 has joined #openstack-nova04:23
*** jpena|off has joined #openstack-nova04:25
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge flavor extensions controller code  https://review.openstack.org/51610404:26
*** fragatina has joined #openstack-nova04:27
gmannalex_xu: ^^ updated04:28
*** thorst has joined #openstack-nova04:34
*** gouthamr has quit IRC04:34
*** bkopilov has joined #openstack-nova04:38
*** thorst has quit IRC04:39
*** ratailor has joined #openstack-nova04:47
*** armax has quit IRC04:50
*** zhurong has quit IRC04:55
*** Sukhdev has quit IRC04:55
*** markvoelker has joined #openstack-nova04:56
*** tetsuro has quit IRC05:02
*** janki has joined #openstack-nova05:06
*** liusheng has quit IRC05:07
*** liusheng has joined #openstack-nova05:07
*** edmondsw has joined #openstack-nova05:13
*** edmondsw has quit IRC05:17
*** sree has joined #openstack-nova05:19
*** BenderRodriguez has quit IRC05:22
*** janki has quit IRC05:27
*** janki has joined #openstack-nova05:27
*** markvoelker has quit IRC05:30
*** yamahata has joined #openstack-nova05:35
*** Laurene has quit IRC05:36
*** zhurong has joined #openstack-nova05:45
*** fragatina has quit IRC05:47
*** lajoskatona has joined #openstack-nova05:53
*** lpetrut has joined #openstack-nova06:04
*** ijw has joined #openstack-nova06:05
*** namnh_ has joined #openstack-nova06:07
*** ijw has quit IRC06:10
*** namnh_ has quit IRC06:11
openstackgerritYikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040606:18
*** diga has quit IRC06:20
*** esberglu has quit IRC06:24
*** markvoelker has joined #openstack-nova06:27
*** gongysh has joined #openstack-nova06:28
*** nikhil has quit IRC06:30
openstackgerritYikun Jiang proposed openstack/python-novaclient master: Microversion 2.54 - Migrations list pagination  https://review.openstack.org/51616806:34
*** thorst has joined #openstack-nova06:35
*** hoonetorg has quit IRC06:37
*** ijw has joined #openstack-nova06:38
*** thorst has quit IRC06:39
*** vladikr has quit IRC06:40
*** ijw has quit IRC06:42
*** hemna_ has quit IRC06:45
*** hemna_ has joined #openstack-nova06:46
*** moshele has joined #openstack-nova06:49
*** hoonetorg has joined #openstack-nova06:53
*** vladikr has joined #openstack-nova06:53
*** ijw has joined #openstack-nova06:54
*** sahid has joined #openstack-nova06:58
*** ijw has quit IRC06:58
*** rcernin has quit IRC06:59
*** markvoelker has quit IRC07:00
*** sahid_ has joined #openstack-nova07:00
*** brault has joined #openstack-nova07:02
*** sahid_ has quit IRC07:06
*** sahid has quit IRC07:06
*** sahid has joined #openstack-nova07:07
*** spectr has joined #openstack-nova07:08
*** Oku_OS-away is now known as Oku_OS07:08
*** yushb has joined #openstack-nova07:10
*** salv-orl_ has quit IRC07:11
*** salv-orlando has joined #openstack-nova07:11
*** vks1 has joined #openstack-nova07:14
*** yamahata has quit IRC07:14
*** esberglu has joined #openstack-nova07:15
*** hshiina has joined #openstack-nova07:16
*** salv-orlando has quit IRC07:16
*** pcaruana has joined #openstack-nova07:17
*** Oku_OS is now known as Oku_OS-away07:19
*** esberglu has quit IRC07:19
*** Oku_OS-away is now known as Oku_OS07:19
*** yushb has quit IRC07:21
*** salv-orlando has joined #openstack-nova07:21
*** trinaths has joined #openstack-nova07:25
*** lpetrut has quit IRC07:27
*** hferenc has quit IRC07:36
*** Lirt has quit IRC07:42
*** rcernin has joined #openstack-nova07:44
*** sshwarts has joined #openstack-nova07:49
*** lpetrut has joined #openstack-nova07:54
moshelestephenfin: hi can you review  https://review.openstack.org/#/c/505613/ ?07:56
*** markvoelker has joined #openstack-nova07:58
*** AlexeyAbashkin has joined #openstack-nova07:58
*** lpetrut has quit IRC08:01
*** ragiman has joined #openstack-nova08:01
*** AlexeyAbashkin has quit IRC08:01
*** AlexeyAbashkin has joined #openstack-nova08:01
*** lei-zh has joined #openstack-nova08:04
alex_xuefried: what is the relationship between different request group, or between non-numbered group and numbered group?08:04
*** diga has joined #openstack-nova08:09
openstackgerritjichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement  https://review.openstack.org/51221508:10
openstackgerritjichenjc proposed openstack/nova master: [placement]Enhance doc for placement allocation list  https://review.openstack.org/51171508:13
*** ralonsoh has joined #openstack-nova08:15
*** tesseract has joined #openstack-nova08:16
*** trinaths has quit IRC08:16
*** sree has quit IRC08:17
openstackgerritjichenjc proposed openstack/nova master: remove glance usage inside compute  https://review.openstack.org/51139208:17
openstackgerritjichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url  https://review.openstack.org/51139708:17
*** AlexeyAbashkin has quit IRC08:21
*** priteau has joined #openstack-nova08:22
*** AlexeyAbashkin has joined #openstack-nova08:23
*** yamamoto has quit IRC08:23
*** josecastroleon has joined #openstack-nova08:24
*** lei-zh has quit IRC08:24
*** lei-zh has joined #openstack-nova08:24
*** salv-orlando has quit IRC08:27
*** salv-orlando has joined #openstack-nova08:28
*** markvoelker has quit IRC08:30
*** alexchadin has joined #openstack-nova08:31
*** gcb has joined #openstack-nova08:32
*** salv-orlando has quit IRC08:32
*** jpena|off is now known as jpena08:33
*** takashin has joined #openstack-nova08:34
*** Bo_Jin has joined #openstack-nova08:36
*** Bo_Jin has quit IRC08:37
*** josecastroleon has quit IRC08:37
*** BoJin has joined #openstack-nova08:38
*** sridharg has quit IRC08:39
*** ociuhandu has joined #openstack-nova08:42
*** aloga has quit IRC08:47
*** diga has quit IRC08:48
*** edmondsw has joined #openstack-nova08:49
*** salv-orlando has joined #openstack-nova08:50
*** edmondsw has quit IRC08:53
*** sdague has joined #openstack-nova08:53
*** baoli has quit IRC08:58
*** baoli has joined #openstack-nova08:58
*** lei-zh has quit IRC08:59
*** BoJin has quit IRC08:59
*** gmann is now known as gmann_afk09:02
*** gcb_ has joined #openstack-nova09:03
*** baoli has quit IRC09:03
*** zhurong has quit IRC09:03
*** gcb has quit IRC09:04
*** annp has quit IRC09:04
*** BoJin has joined #openstack-nova09:10
openstackgerritYikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040609:13
*** lei-zh has joined #openstack-nova09:15
*** Kevin_Zheng has joined #openstack-nova09:15
openstackgerritYikun Jiang proposed openstack/nova master: [WIP]Add pagination and Changes-since filter support for os-migrations.  https://review.openstack.org/33040609:16
*** lucas-afk is now known as lucasagomes09:19
*** lei-zh has quit IRC09:23
*** yamamoto has joined #openstack-nova09:24
*** Oku_OS is now known as Oku_OS-away09:25
*** mvk has quit IRC09:26
*** markvoelker has joined #openstack-nova09:27
*** jistr|mtgs is now known as jistr09:29
*** Oku_OS-away is now known as Oku_OS09:31
*** vks1 has quit IRC09:32
openstackgerritjichenjc proposed openstack/nova master: Remove unnecessary checks for root_gb = 0  https://review.openstack.org/51656909:33
*** dikonoor has quit IRC09:33
*** dikonoor has joined #openstack-nova09:33
*** yamamoto has quit IRC09:34
*** vks1 has joined #openstack-nova09:36
gibialex_xu: jaypipes and efried talked about numbered and unnumbered groups during the night (in my timezone) http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2017-10-30.log.html#t2017-10-30T18:59:4809:38
openstackgerritZhenyu Zheng proposed openstack/nova master: Don't update RT in _allocate_network  https://review.openstack.org/51347309:39
openstackgerritjichenjc proposed openstack/nova master: Add description for resize down support  https://review.openstack.org/51657209:46
*** owalsh- is now known as owalsh09:47
*** aloga has joined #openstack-nova09:48
*** slaweq has joined #openstack-nova09:49
openstackgerritStephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt  https://review.openstack.org/49961209:50
openstackgerritStephen Finucane proposed openstack/nova master: Simplify instance name generation  https://review.openstack.org/51657309:50
*** Oku_OS is now known as Oku_OS-away09:50
*** slaweq_ has quit IRC09:51
*** NightKhaos has quit IRC09:52
*** sambetts|afk is now known as sambetts09:55
*** NightKhaos has joined #openstack-nova09:56
*** jdurgin has quit IRC09:56
*** mvk has joined #openstack-nova09:56
*** brault has quit IRC09:56
*** Oku_OS-away is now known as Oku_OS09:57
openstackgerritjichenjc proposed openstack/nova master: Remove unnecessary checks for root_gb = 0  https://review.openstack.org/51656909:59
*** namnh has quit IRC10:00
*** jdurgin has joined #openstack-nova10:00
*** jistr_ has joined #openstack-nova10:00
*** hemna- has joined #openstack-nova10:00
*** markvoelker has quit IRC10:00
*** jistr has quit IRC10:01
*** hemna has quit IRC10:01
*** coreywright has quit IRC10:01
*** phuongnh has quit IRC10:01
*** krtaylor has quit IRC10:01
*** migi has quit IRC10:01
*** isq_ has joined #openstack-nova10:01
*** coreywright has joined #openstack-nova10:02
*** phuongnh has joined #openstack-nova10:02
*** krtaylor has joined #openstack-nova10:02
*** migi has joined #openstack-nova10:02
*** isq has quit IRC10:03
*** Jeffrey4l has quit IRC10:04
*** taneee has quit IRC10:04
*** cdent has joined #openstack-nova10:06
*** tanee has joined #openstack-nova10:07
*** Jeffrey4l has joined #openstack-nova10:07
*** namnh has joined #openstack-nova10:08
*** brault has joined #openstack-nova10:09
*** diga has joined #openstack-nova10:09
*** ratailor has quit IRC10:11
*** yangyape_ has quit IRC10:12
*** namnh has quit IRC10:12
*** Oku_OS is now known as Oku_OS-away10:17
*** lpetrut has joined #openstack-nova10:17
openstackgerritAlvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325010:20
*** brault has quit IRC10:24
*** udesale has quit IRC10:26
openstackgerritTakashi NATSUME proposed openstack/nova master: Fix missing versioned notification sample  https://review.openstack.org/51658210:28
*** phuongnh has quit IRC10:28
*** sree has joined #openstack-nova10:29
*** mvk has quit IRC10:30
*** sree has quit IRC10:33
*** trinaths1 has joined #openstack-nova10:34
*** sree has joined #openstack-nova10:35
*** hemna_ has quit IRC10:36
*** thorst has joined #openstack-nova10:36
*** edmondsw has joined #openstack-nova10:37
*** brault has joined #openstack-nova10:38
openstackgerritsahid proposed openstack/nova master: libvirt: disconnect volume from host during detach  https://review.openstack.org/51500810:38
ildikovjohnthetubaguy: hi10:38
*** Laurene has joined #openstack-nova10:38
johnthetubaguyildikov: hi10:39
ildikovjohnthetubaguy: we now have an etherpad for the Forum session on multiattach next week: https://etherpad.openstack.org/p/sydney-cinder-nova-cp-multi-attach10:39
ildikovjohnthetubaguy: please take a look at it if you have a few minutes and add comments or questions/topics for discussion10:39
*** yamamoto has joined #openstack-nova10:39
*** sree has quit IRC10:39
johnthetubaguyildikov: will take a look in a few mins10:40
ildikovjohnthetubaguy: thank you!10:40
*** edmondsw has quit IRC10:41
*** ociuhandu has quit IRC10:42
*** mvk has joined #openstack-nova10:43
*** brault has quit IRC10:43
*** thorst has quit IRC10:43
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103510:43
*** ociuhandu has joined #openstack-nova10:46
*** Oku_OS-away is now known as Oku_OS10:46
*** gongysh has quit IRC10:51
*** tbachman has quit IRC10:54
*** ijw has joined #openstack-nova10:55
*** priya has joined #openstack-nova10:58
*** markvoelker has joined #openstack-nova10:58
priyagibi: hi, Could you please review https://review.openstack.org/#/c/508533/10:58
*** ijw has quit IRC10:59
*** huanxie has quit IRC10:59
*** priteau has quit IRC11:09
*** priteau has joined #openstack-nova11:10
*** priteau has quit IRC11:11
*** Laurene has quit IRC11:11
*** dillaman has quit IRC11:13
*** vladikr has quit IRC11:13
*** zhouyaguo has joined #openstack-nova11:18
*** rcernin has quit IRC11:18
*** bjolo has quit IRC11:23
efriedalex_xu The "unnumbered" group (which we're calling the "sharing" group) is what you're coding up now: it can get its resources from any RP in a nested tree or sharing RP in the same aggregate.11:25
efriedalex_xu The "numbered" ("non-sharing") group can only get its resources from a single RP.11:26
*** ratailor has joined #openstack-nova11:27
efriedalex_xu When several groups are in a single request, all the resources as a whole must come from the same nested tree + sharing RPs in the same aggregate.11:27
openstackgerritMerged openstack/nova master: api-ref: document caveats with scheduler hints  https://review.openstack.org/51644211:27
efriedalex_xu It's all in the spec, which of course you memorized :)  https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/granular-resource-requests.html11:28
efriedalex_xu Specifically: https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/granular-resource-requests.html#semantics11:28
*** hemna has joined #openstack-nova11:29
*** markvoelker has quit IRC11:31
*** sridharg has joined #openstack-nova11:33
*** ociuhandu has quit IRC11:36
cdentefried: one thing I’m a bit curious about is how much type 2 fun a flavor author is going to have trying to figure out how to properly use this syntax11:42
efried"type 2 fun"?11:42
cdentone moment please11:42
efriedno, I got it.11:43
cdentk11:43
cdentit might even be type 311:43
*** awaugama has joined #openstack-nova11:44
cdentI mean, I guess once you’ve got it right you can forget about it11:44
*** smatzek has joined #openstack-nova11:44
efriedcdent That.  And a guy could just use all numbered groups and forgedaboutit11:45
*** vks1 has quit IRC11:45
efriedPutting a single RC into each of several numbered groups will have the same effect as putting them all into the unnumbered group.11:46
efriedThen you just cluster together resources that really need to be clustered to gether.11:46
efrieds/ //11:46
efriedGiven that we need to be able to do this, I'm not sure if there's a simpler syntax.11:47
efriedI would say I'm open to suggestions, but spec freeze and all that.11:47
cdentI suppose these are power tools for power users, so it’s okay. I’m glad it’s not a common case for daily end users.11:47
cdentOh sorry, I’m not disagreeing with the syntax, at all11:47
cdentIt solves the problem admirably11:47
cdentI’m disagreeing with the problem :)11:47
*** esberglu has joined #openstack-nova11:48
*** salv-orlando has quit IRC11:48
*** brault has joined #openstack-nova11:49
*** pchavva has joined #openstack-nova11:49
*** ratailor has quit IRC11:50
efriedcdent As in, you think we're solving a non-problem?  Or it just sucks that it's a thing that has to be solved?11:51
*** kylek3h has joined #openstack-nova11:51
*** liverpooler has joined #openstack-nova11:52
*** esberglu has quit IRC11:52
*** amarao1 has joined #openstack-nova11:52
*** kylek3h_ has joined #openstack-nova11:53
*** brault has quit IRC11:53
amarao1Which nova component produces the least amount of RPC messages? (I want to test timeout settings for idle connection and I need some nova service which do not send messages but only wait for them).11:53
cdentthe latter11:53
*** udesale has joined #openstack-nova11:54
*** diga has quit IRC11:54
cdentefried: it’s necessary complexity for a certain set of complex things but I’d prefer a world that was somehow simpler. Or something?11:54
efriedcdent I can get behind a desire for a simpler world.11:55
*** salv-orlando has joined #openstack-nova11:55
*** jaypipes has joined #openstack-nova11:55
*** kylek3h has quit IRC11:56
openstackgerritChris Dent proposed openstack/nova master: Optional separate database for placement API  https://review.openstack.org/36276611:57
*** thorst has joined #openstack-nova11:58
* cdent cuts the grass11:58
jaypipesalex_xu: hey bud, you up still?12:06
*** gszasz has joined #openstack-nova12:06
*** hemna has quit IRC12:07
jaypipesefried: you around yet?12:08
efriedjaypipes Yup12:08
*** alexchadin has quit IRC12:09
jaypipesefried: so I was thinking I might try and do a couple patches to clean up/standardize the AllocationCandidates._get_by_filters() mega-method12:09
jaypipesefried: I know alex_xu already has a series where the *end* of the series refactors that method12:09
jaypipesefried: but I'm thinking it might be more useful if I broke that method out into separate funcs before his series.12:09
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: implement get_inventory() driver method  https://review.openstack.org/50617512:10
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: fix memory stats  https://review.openstack.org/51663412:10
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: expose max vCPUs and max memory per ESX host  https://review.openstack.org/51663512:10
*** tbachman has joined #openstack-nova12:10
jaypipesefried: you've been reviewing alex's series (https://review.openstack.org/#/c/479766/ is the refactor megamethod patch). which is why I'm wanting to get your opinion.12:10
*** yamamoto has quit IRC12:11
*** yamamoto has joined #openstack-nova12:11
efriedjaypipes That was the intent behind what alex_xu and I have been working on in this series, for sure.  Are you talking about doing some refactoring that leads into that series, or a parallel effort that approaches it a different way?12:12
*** Oku_OS is now known as Oku_OS-away12:12
*** Oku_OS-away is now known as Oku_OS12:12
jaypipesefried: that leads *in* to that series.12:15
efriedjaypipes Sounds good to me.  What do you have in mind?12:16
jaypipesefried: I'm not a fan of the objectified approach in that series and would like to break that mega-method out into separate module-level functions that are appropriately named.12:16
efriedjaypipes As in, get rid of e.g. the ProviderSummary[List] and AggregatedProviderSummary[List] etc.?12:17
jaypipesyes, exactly.12:17
jaypipesI don't believe those objects are healthy.12:17
jaypipesor necessary.12:18
efriedjaypipes Okay, so essentially supplanting the series with a different approach.12:18
jaypipesefried: meh, in a way. more just laying a different foundation for the series to build from...12:18
efriedI'm less enthused about that, but not opposed to it.  We need to be able to move forward here, and it's clear that the current monolith is untenable.  But it's also clear we're having trouble constructing SQL that handles all the cases.12:19
*** hemna has joined #openstack-nova12:19
efriedSo on the one hand, having some jaypipes join-fu help would be great; but on the other hand, having something only jaypipes can understand... less than ideal.12:20
jaypipesefried: and I've said in reviews on this series (and others before) that we don't need to try and do all the pieces in one giant SQL statement.12:20
efried++12:20
efriedI'm not well versed enough to understand where we introduce huge inefficiencies by moving join-y stuff from the database to python.12:20
efriedBut in principle, I love that idea.12:21
jaypipescdent: for the life of me, I cannot understand why I'm getting the failure (only on the gabbit in py35) in this particular patch: https://review.openstack.org/#/c/385693/. If you might take a looksie, that would be most appreciated...12:21
cdentjaypipes: shore. grass still too wet to cut12:21
*** markvoelker has joined #openstack-nova12:21
efriedjaypipes So like, would it be appropriate to have one method that pulls just inventory/allocation info for a single RP+RC?  Or is that too small?12:22
jaypipesefried: it's not that complicated actually. the basic idea is don't complicate the SQL unless there's a reason to do so. All lengthy and complex SQL statements, as you know, can be broken out into composite sub-statements and executed separately. Sometimes, actually, doing multiple queries that each are very efficient can be faster than trying to do a single SQL complex statement.12:23
jaypipesefried: for the use_single_provider request groups, that is EXACTLY the right way to do things.12:23
*** kylek3h_ has quit IRC12:23
*** kylek3h has joined #openstack-nova12:24
efriedjaypipes Yeah, I've been thinking the SQL for the use_single_provider groups would be way simpler than what's happening now.12:24
*** salv-orlando has quit IRC12:24
efriedjaypipes Okay, so I'm behind the idea, but alex_xu and I ought to know what we should be doing with the existing series while you work that up.  Probably best to quit messing with it, cause it'll either change drastically or be abandoned altogether?12:25
jaypipesefried: think in terms of sets of records. there are multiple sets of records we want to retrieve based on the problem we're attempting to solve. in the case of shared providers, that's one set of records. the set of providers that provider ALL parts of a request of resources, that's another set. The set of providers that provides *some* but not all resources, that's yet another set of records. the trick is make efficient SQL statements to12:25
jaypipesproduce each of those sets in a way that can be reasoned about easily afterwards.12:25
*** edmondsw_ has joined #openstack-nova12:25
efriedjaypipes And presumably to make sure the amount of data we're pulling from the database and then paring down from python isn't way too big.12:26
jaypipesefried: right.12:27
priyagibi:Could you please review https://review.openstack.org/#/c/508533/12:27
efriedjaypipes Dunno if you remember in Austin the example I called out: they had a shell script that did "SELECT * FROM really_huge_database" | wc -l12:28
jaypipesefried: who is "they"?12:28
*** thorst_ has joined #openstack-nova12:29
jaypipespriya: I *think* gibi may be on public holiday today... not entirely sure though.12:29
efriedjaypipes I don't remember.  Could have been a cinder thing?  To count up soft-deleted entries or something?  And it wasn't production code, just a testing util.  But it still grated.12:29
jaypipesyeah...12:29
cdentjaypipes: the gabbi problem is a combination of case sensitivity and dict ordering unpredictability. you didn’t introduce it, it just didn’t fail before in commit a9105b4904b that added the failing test. I’ll note the fix on the review, which we might want to do separately to yours and merge nowish12:30
jaypipescdent: hmm, intersting. is this a py35 vs py27 thing? or just a freak occaaion?12:30
jaypipesoccasion..12:31
*** thorst has quit IRC12:31
cdentI suspect that py35’s dict ordering is a factor12:31
efriedjaypipes So let me know if I can help out here.  Sounds to me like the intent is to write a bunch of separate independent functions and eventually refactor _get_by_filters to use 'em.  So in that first stage, if you want to dole out "write me a function that does X" we can work in parallel.12:31
gibijaypipes, priya: I will be on public holiday tomorrow :) so I will look at it soon12:31
*** catintheroof has joined #openstack-nova12:31
cdentbut there’s also freakery involved otherwise it would have failed on efried’s patch12:31
jaypipesefried: zactly.12:31
*** catintheroof has quit IRC12:32
*** catintheroof has joined #openstack-nova12:32
jaypipesefried: just like the current code currently first calls out to a separate function to get "root providers" and then gets shared providers, etc...12:32
jaypipesefried: just needs to be cleaned up a bit.12:32
jaypipesefried: and better doc'd.12:32
*** rmart04 has joined #openstack-nova12:33
*** takashin has left #openstack-nova12:33
*** dave-mccowan has joined #openstack-nova12:36
*** jmlowe_ has joined #openstack-nova12:38
*** salv-orlando has joined #openstack-nova12:39
openstackgerritDavid Rabel proposed openstack/nova master: Fix format in live-migration-usage.rst  https://review.openstack.org/51641612:40
*** jmlowe has quit IRC12:40
cdentjaypipes: i’ll go ahead and fix that thing separately https://bugs.launchpad.net/nova/+bug/172893412:45
openstackLaunchpad bug 1728934 in OpenStack Compute (nova) "placement gabbi tests that manipulate microversion can intermittently fail" [Medium,Triaged] - Assigned to Chris Dent (cdent)12:45
*** lucasagomes is now known as lucas-hungry12:47
*** Dinesh_Bhor has quit IRC12:47
*** yangyapeng has joined #openstack-nova12:48
*** udesale has quit IRC12:49
*** udesale has joined #openstack-nova12:49
*** zhurong has joined #openstack-nova12:49
*** jpena is now known as jpena|lunch12:51
*** esberglu has joined #openstack-nova12:54
*** udesale has quit IRC12:56
*** yamamoto has quit IRC12:56
*** udesale has joined #openstack-nova12:56
*** bkopilov has quit IRC12:57
openstackgerritChris Dent proposed openstack/nova master: [placement] avoid case issues microversions in gabbits  https://review.openstack.org/51666212:57
efriedjaypipes Was about to respin the flavor-side granular series (https://review.openstack.org/#/c/515151/) - you got more to say there at the moment?12:57
cdentjaypipes, efried, gibi ^^ gate flakiness bug fix12:57
efriedcdent Would that be officially remedied by making gabbi use case insensitive dicts (like requests does) for headers?12:58
*** janki has quit IRC12:58
cdentefried: read the commit message for the answer to that12:58
cdentefried: the idea is that gabbi needs to support the behaviors of http, not whatever clients happen to be available12:59
gibicdent: looking...12:59
efriedcdent I thought the http standard was for headers to be case-insensitive.  But I'm no expert.12:59
efriedcdent Did you deliberately say "vagrancies" rather than "vagaries"?13:00
cdentand because http header names are case insensitive it is possible to accidentally set the same header twice in some implementations13:00
cdentno, but i’ll take it :)13:00
efriedI think even with case insensitivity you can set the same header multiple times.13:01
cdentyes, that’s the “implementation” part of the commit message.13:01
cdentit’s a yaml dict13:01
cdentso _if_ we want to send a header twice, we have to use different case13:02
cdentsloppy, but…13:03
efriedcdent Because yaml, or because gabbi?13:03
*** felipemonteiro has joined #openstack-nova13:03
cdentbecause of gabbi’s choice to use a dict for request headers13:03
*** edmondsw_ is now known as edmondsw13:04
cdentefried: there’s a gabbi bug about this from a few months ago: https://github.com/cdent/gabbi/issues/21313:04
cdentefried: note that in that case the bug is talking about all the dicts in defaults, not just request headers13:05
cdentthe resulting conclusion was “don’t normalize as then what you see is not what you get”13:06
gibicdent: is there a reason choosing the non capitalized form of the header name? It seems that placement returns a capitalized from in the response here https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/gabbits/microversion.yaml#L1613:07
*** BoJin has quit IRC13:07
cdentgibi: a) because nearly everything else you type into left hand side of gabbi is lowercase, b) uppercase is a pain to type. Yes, the response header is cased, and I noted that while making the changes and decided to still go with lower case.13:08
*** kylek3h has quit IRC13:09
gibicdent: if it is intentional then I'm cool with it :)13:09
*** kylek3h has joined #openstack-nova13:09
cdentyeah, it’s intentional. I hate the shift key :)13:10
* cdent tries again on the grass13:10
*** AlexeyAbashkin has quit IRC13:10
*** yamamoto has joined #openstack-nova13:10
*** lyan has joined #openstack-nova13:10
*** AlexeyAbashkin has joined #openstack-nova13:11
alex_xujaypipes: efried, hey I'm here13:14
*** yamamoto has quit IRC13:15
efriedalex_xu Check out the backscroll between me & jaypipes and speak up if you have objections/questions/comments on what jaypipes is planning to do wrt refactoring stuff for GET /allocation_candidates13:15
*** namnh has joined #openstack-nova13:17
alex_xuefried: yea, I read those, the current series can be rewrite as a set of sub-method instead of the objects also. but yea, I'm not sure what is the plan of jaypipes13:17
*** mriedem has joined #openstack-nova13:18
*** zhouyaguo has quit IRC13:18
*** namnh has quit IRC13:18
*** namnh has joined #openstack-nova13:18
alex_xujaypipes: efried, i'm not sure what I can do now, sounds like just abandoned the patch, or jaypipes do you want to share your idea on etherpad or anywhere. Then I and efried can help on that, since we already work on that series, and found few cases which easy to miss, that may help you?13:19
*** salv-orlando has quit IRC13:19
*** hemna has quit IRC13:20
*** salv-orlando has joined #openstack-nova13:20
*** Guest13420 is now known as yassine13:24
*** yassine is now known as yassinee13:24
*** salv-orlando has quit IRC13:24
jaypipesalex_xu: no, please don't abandon. lemme push some sample code and then let me know what you think about using it as the basis for your series.13:25
alex_xujaypipes: ok, got it, if you get the code up very soon, reading the code is more easy something13:26
jaypipesefried: no, feel free to respin. I think you got most of my feedback on IRC on that one, yeah?13:26
alex_xus/something/sometime/13:26
jaypipesalex_xu: working it right now :)13:26
efriedjaypipes Hopefully.  Respinning now...13:26
*** smatzek has quit IRC13:26
*** sahid has quit IRC13:26
*** smatzek has joined #openstack-nova13:27
* alex_xu is going to sleep, and expect some beautiful code show up when wake up13:28
jaypipesalex_xu: nighty night! :)(13:28
* efried waves to alex_xu 13:28
jaypipescdent: +W'd13:29
*** smatzek has quit IRC13:31
*** nikhil has joined #openstack-nova13:32
*** jaosorior has quit IRC13:33
*** baoli has joined #openstack-nova13:35
*** Dinesh_Bhor has joined #openstack-nova13:36
*** lbragstad has joined #openstack-nova13:37
efriedjaypipes You said the order of subclauses in an AND isn't important... so the database doesn't short-circuit the clause on the first false condition?13:39
jaypipesefried: no, it doesn't work that way. RDBMs will always prioritize the field(s) that are in indexes chosen by the query optimizer.13:40
efriedhm, neat.13:40
*** hshiina has quit IRC13:41
*** lucas-hungry is now known as lucasagomes13:41
*** eharney has joined #openstack-nova13:41
*** yamamoto has joined #openstack-nova13:41
jaypipesefried: so in that case, it makes no difference since the resource class and resource provider fields will always be evaluated before anything with total, max_unit, step_size, or min_unit13:41
*** zhurong has quit IRC13:42
efriedjaypipes But it's (slightly) cheaper to do the single-comparator math than the multiple-add-compare stuff.13:42
*** mlavalle has joined #openstack-nova13:42
efriedmultiply-add-compare, that is.13:42
efriedI'm sure it's negligible.13:42
jaypipesefried: I don't think you'd see the difference there, no.13:42
jaypipesefried: compared to the cost of loading all of the fields into memory after the join conditions are satisfied, the comparison of the data in those loaded fields is miniscule.13:43
efriednoted13:43
*** hongbin has joined #openstack-nova13:43
*** dinesh_ has joined #openstack-nova13:44
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103513:44
*** ratailor has joined #openstack-nova13:45
*** yamamoto has quit IRC13:45
*** felipemonteiro_ has joined #openstack-nova13:46
*** dinesh__ has joined #openstack-nova13:47
*** Dinesh_Bhor has quit IRC13:47
*** BoJin has joined #openstack-nova13:47
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements  https://review.openstack.org/51103513:48
*** felipemonteiro has quit IRC13:49
*** dinesh_ has quit IRC13:49
*** ratailor is now known as rtailor13:49
*** vks1 has joined #openstack-nova13:50
*** dinesh_ has joined #openstack-nova13:51
*** brault has joined #openstack-nova13:51
*** jpena|lunch is now known as jpena13:52
*** esberglu has quit IRC13:53
*** dinesh__ has quit IRC13:54
*** dinesh__ has joined #openstack-nova13:57
*** oanson has quit IRC13:57
*** oanson has joined #openstack-nova13:57
*** hemna has joined #openstack-nova13:58
*** Dinesh_Bhor has joined #openstack-nova13:59
*** smatzek has joined #openstack-nova13:59
openstackgerritEric Fried proposed openstack/nova master: Parse granular resources/traits from extra_specs  https://review.openstack.org/51515113:59
openstackgerritEric Fried proposed openstack/nova master: Granularize resources_from_{flavor|request_spec}  https://review.openstack.org/51522313:59
openstackgerritEric Fried proposed openstack/nova master: ResourceRequest.to_querystring()  https://review.openstack.org/51581113:59
efriedgibi jaypipes alex_xu ^13:59
efriedand cdent too ^13:59
*** dinesh_ has quit IRC14:00
*** gcb_ has quit IRC14:00
*** gcb_ has joined #openstack-nova14:00
*** dinesh__ has quit IRC14:01
*** janki has joined #openstack-nova14:01
*** smatzek has quit IRC14:03
cdent14:03
*** smatzek has joined #openstack-nova14:05
*** edand has joined #openstack-nova14:07
*** dinesh_ has joined #openstack-nova14:08
*** esberglu has joined #openstack-nova14:09
*** amodi has joined #openstack-nova14:09
*** marst has joined #openstack-nova14:09
*** dinesh__ has joined #openstack-nova14:10
*** gszasz has quit IRC14:11
*** Dinesh_Bhor has quit IRC14:11
*** catintheroof has quit IRC14:12
*** dinesh_ has quit IRC14:13
*** dinesh_ has joined #openstack-nova14:13
*** gszasz has joined #openstack-nova14:13
*** catintheroof has joined #openstack-nova14:14
*** armax has joined #openstack-nova14:17
*** dinesh__ has quit IRC14:17
*** dinesh_ has quit IRC14:18
*** jaosorior has joined #openstack-nova14:20
*** armax has quit IRC14:23
gibiefried: I'm +2 on the whole series, thanks for working on it14:27
*** armax has joined #openstack-nova14:28
*** catintheroof has quit IRC14:30
*** eharney has quit IRC14:31
*** reedip_ has joined #openstack-nova14:31
reedip_Hi guys , how much time does it take for an instance to boot up in Nova ? We have a 32 GB RAM for Compute node, and booting up a VM using Ubuntu 14.0414:32
reedip_but the VM doesnt boot up in 6 minutes....14:32
reedip_does that normally occur ?14:33
reedip_I tried it atleast 8-10 times , booting a set of 10 VMs.14:33
reedip_each VM is an Ubuntu Xenial CLoud image14:33
reedip_but of the 10 VMs, 4-6 boot up succesfully... failure occurs after that14:34
reedip_Flavor given is 2 GB RAM and 3 GB HDD14:34
jaypipesreedip_: 14.72917672 seconds. every time.14:35
*** catintheroof has joined #openstack-nova14:35
reedip_jaypipes : its that accurate ????14:35
jaypipesreedip_: no, of course not, I was only joking... :) there's no way to say. totally depends on the hardware, the virt driver, the number of nodes, etc etc14:36
reedip_jaypipes : so if a VM doesnt boot up ( BTW I am using Mitaka with Ubuntu 14.04 ), where can I find the logs ?14:36
reedip_jaypipes : I thought so :)14:36
jaypipesreedip_: look in the nova-scheduler logs. you will likely find an error about resources being over capacity...14:37
reedip_jaypipes : okay, let me check ...14:37
*** salv-orlando has joined #openstack-nova14:37
*** spectr has quit IRC14:37
jaypipesreedip_: for a small setup like that, if the VM doesn't boot in under 15 seconds, that's unusual...14:38
reedip_jaypipes : wait, the complete setup is below :14:39
*** spectr has joined #openstack-nova14:39
jaypipesfor tiny images like that, I'd expect something more like 6 seconds, especially if the image has already been launched/cached on the comput ehost.14:39
reedip_1 Controller: 4 GB RAM, 12 GB HDD14:39
*** vladikr has joined #openstack-nova14:40
*** tssurya has joined #openstack-nova14:40
*** salv-orl_ has joined #openstack-nova14:40
*** nicolasbock has joined #openstack-nova14:41
jaypipesreedip_: how many compute nodes?14:41
jaypipesreedip_: also, your instances should end up in an ERROR state if they aren't booting in six minutes. You should be able to check the error by looking at the instance faults returned in the GET /servers/{uuid}/detail call.14:42
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940414:42
*** vladikr has quit IRC14:43
*** vladikr has joined #openstack-nova14:43
*** reedip_ has quit IRC14:44
*** salv-orlando has quit IRC14:44
*** pkoniszewski has quit IRC14:44
*** amotoki has quit IRC14:44
*** jbernard has quit IRC14:44
*** jbernard has joined #openstack-nova14:45
*** eharney has joined #openstack-nova14:46
*** pkoniszewski has joined #openstack-nova14:46
*** trinaths1 has quit IRC14:47
*** udesale has quit IRC14:47
*** amotoki has joined #openstack-nova14:47
*** trinaths has joined #openstack-nova14:47
*** psachin has quit IRC14:48
*** reedip_ has joined #openstack-nova14:48
reedip_jaypipes : hi , did you get my earlier messages ?14:49
jaypipesreedip_: err, no?14:49
*** david-lyle has quit IRC14:50
reedip_jaypipes : sorry then, my network issue ...14:50
reedip_jaypipes : the complete setup is below :14:51
reedip_jaypipes :1 Controller: 4 GB RAM, 12 GB HDD,  3 Compute : 2 COmpute with 32 GB RAM and 100 GB HDD , 1 with 8 GB RAM and 20 GB HDD, 1 Glance server ,The request goes to the 100 GB COmpute machine, and we are using Qemu14:51
reedip_jaypipes : would Nova-Scheduler report it as an Error ?? I think it should but reconfirming14:52
jaypipesreedip_: what are your allocation ratios? have you changed any of the default configuration values for cpu_allocation_ratio, ram_allocation_ratio or disk_alllocation_ratio?14:52
reedip_jaypipes : in case resources are less14:52
reedip_jaypipes : no , not changed any default configurations14:53
jaypipesreedip_: yes. you would see a NoValidHost pop up.14:53
jaypipesreedip_: k14:53
jaypipesreedip_: look in the nova-scheduler and nova-compute log files. nova-scheduler on controller. nova-compute on each of the compute nodes. look for any ERROR log line.14:53
reedip_jaypipes : yeah, couldnt find that ... I know the NoValidHost error , happened to me amny a times now .. :)14:53
jaypipesreedip_: you see no errors at all?14:55
* dansmith glances across the channel topic14:56
reedip_jaypipes : no , I couldnt see the NoValidHost error .. there are some rabbitmq errors but that was because rabbitmq was failed ... this happened quite a long time after we noticed the VM failure, so I dont think that would be an issue14:57
*** dtantsur|afk is now known as dtantsur14:58
*** salv-orl_ has quit IRC14:58
jaypipesreedip_: rabbit errors would definitely be an issue...14:58
*** salv-orlando has joined #openstack-nova14:58
*** gcb_ has quit IRC14:58
openstackgerritEd Leafe proposed openstack/nova master: Add Selection objects  https://review.openstack.org/49923914:59
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585414:59
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015914:59
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135714:59
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135814:59
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670714:59
reedip_we resolved them , and retried the tests, but still got the failure, thats why I am thinking if it is or not14:59
reedip_jaypipes: ^^14:59
edleafemriedem: dansmith: ^^ refactored with additional RPC param14:59
*** gcb_ has joined #openstack-nova15:00
jaypipesreedip_: on a call for an hour or so..15:01
reedip_jaypipes : I may need to leave soon, but I will see the logs and accordingly report it on the launchpad. Maybe we can proceed from there ... is that ok ?15:01
openstackgerritDan Smith proposed openstack/nova master: Make put_allocations() retry on concurrent update  https://review.openstack.org/51670815:01
openstackgerritDan Smith proposed openstack/nova master: Refactor claim_resources() to use retries decorator  https://review.openstack.org/51670915:01
dansmithmriedem: ^15:01
reedip_jaypipes : then I think my plan is good enough .. thanks for your time15:01
*** yamamoto has joined #openstack-nova15:04
*** Dinesh_Bhor has joined #openstack-nova15:05
*** lbragstad has quit IRC15:05
*** dinesh_ has joined #openstack-nova15:06
*** salv-orlando has quit IRC15:06
*** salv-orlando has joined #openstack-nova15:07
*** dinesh__ has joined #openstack-nova15:07
*** lbragstad has joined #openstack-nova15:08
*** sree has joined #openstack-nova15:08
*** Dinesh_Bhor has quit IRC15:09
*** dinesh_ has quit IRC15:11
*** salv-orlando has quit IRC15:11
*** sree has quit IRC15:12
*** dinesh__ has quit IRC15:13
mriedemedleafe: dansmith: ack15:13
mriedemtrying to un-f ocata atm15:14
*** yamamoto has quit IRC15:14
*** dtantsur is now known as dtantsur|afk15:15
*** stvnoyes has joined #openstack-nova15:18
*** AlexeyAbashkin has quit IRC15:23
*** AlexeyAbashkin has joined #openstack-nova15:23
*** Tahvok has quit IRC15:23
*** Tahvok has joined #openstack-nova15:28
*** spectr has quit IRC15:29
*** catintheroof has quit IRC15:34
*** gyee has joined #openstack-nova15:34
*** ttsiouts has joined #openstack-nova15:34
*** edand has quit IRC15:37
*** brault has quit IRC15:38
*** dikonoor has quit IRC15:38
*** brault has joined #openstack-nova15:38
*** itlinux has joined #openstack-nova15:39
*** catintheroof has joined #openstack-nova15:40
*** gcb_ has quit IRC15:40
*** brault has quit IRC15:41
*** spectr has joined #openstack-nova15:41
*** catintheroof has quit IRC15:42
*** gcb_ has joined #openstack-nova15:43
*** moshele has quit IRC15:43
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data  https://review.openstack.org/45281815:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2)  https://review.openstack.org/45281915:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Deduplicated instance.(un)pause notification samples  https://review.openstack.org/45282015:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Deduplicate instance.power-off notification samples  https://review.openstack.org/47586015:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Deduplicate instance.interface_attach samples  https://review.openstack.org/51574015:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Deduplicate instance.live_migration notification samples  https://review.openstack.org/51574515:44
openstackgerritBalazs Gibizer proposed openstack/nova master: Deduplicate instance.reboot notification samples  https://review.openstack.org/51575115:44
*** lyan has quit IRC15:45
*** lyan has joined #openstack-nova15:46
gibijaypipes: thanks for the review on the notification sample refactoring patches. I fixed your comments15:47
jaypipesgibi: sweet. will re-review shortly15:47
gibijaypipes: thanks15:48
*** AlexeyAbashkin has quit IRC15:48
jaypipesefried: love the stickers on your wall, Eric. very haut couture.15:50
*** bkopilov has joined #openstack-nova15:50
*** gongysh has joined #openstack-nova15:51
jaypipesefried: you and mriedem's velvet dobermann should compete :)15:51
*** gongysh has quit IRC15:51
efriedjaypipes It's my youngest son's room.15:55
*** rtailor has quit IRC15:55
*** Apoorva has joined #openstack-nova15:56
*** Apoorva has quit IRC15:56
*** brault has joined #openstack-nova15:56
*** Apoorva has joined #openstack-nova15:56
*** smatzek has quit IRC15:57
jaypipesefried: it's your office, just admit it Eric. :P15:57
*** Apoorva has quit IRC15:57
*** gongysh has joined #openstack-nova15:57
*** gongysh has quit IRC15:57
*** slaweq has quit IRC15:58
efriedjaypipes You got me.15:59
*** smatzek has joined #openstack-nova15:59
*** janki has quit IRC16:00
*** bnemec has quit IRC16:00
*** smatzek_ has joined #openstack-nova16:01
*** yamamoto has joined #openstack-nova16:02
*** yamamoto has quit IRC16:02
*** liuyulong has joined #openstack-nova16:03
*** Oku_OS is now known as Oku_OS-away16:03
*** smatzek has quit IRC16:03
*** jaosorior has quit IRC16:03
*** BoJin has quit IRC16:03
mriedemdansmith: commented16:06
*** mvk has quit IRC16:07
liuyulongEnable reset keypair while rebuilding instance https://review.openstack.org/#/c/379128/16:07
*** namnh has quit IRC16:08
liuyulong^ needs nova cores review : )16:08
*** david-lyle has joined #openstack-nova16:09
*** esberglu has quit IRC16:09
dansmithmriedem: ack, will look in about an hour16:09
*** lajoskatona has quit IRC16:09
*** penick has joined #openstack-nova16:10
mriedemedleafe: https://review.openstack.org/#/c/495854/ is failing16:10
mriedemTypeErrors16:10
*** smatzek_ has quit IRC16:12
*** edmondsw has quit IRC16:12
*** smatzek has joined #openstack-nova16:12
*** bnemec has joined #openstack-nova16:15
*** fragatina has joined #openstack-nova16:20
*** fragatina has quit IRC16:21
*** smatzek has quit IRC16:21
*** smatzek has joined #openstack-nova16:21
*** fragatina has joined #openstack-nova16:22
*** rmart04 has quit IRC16:22
openstackgerritBalazs Gibizer proposed openstack/nova master: Moving more utils to ServerResourceAllocationTestBase  https://review.openstack.org/49953916:22
openstackgerritBalazs Gibizer proposed openstack/nova master: factor out compute service start in ServerMovingTest  https://review.openstack.org/50303716:22
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete  https://review.openstack.org/49515916:22
edleafemriedem: looking...16:23
*** brault has quit IRC16:26
openstackgerritBalazs Gibizer proposed openstack/nova master: Document the real behavior of notify_on_state_change  https://review.openstack.org/51626416:26
*** brault has joined #openstack-nova16:26
*** smatzek has quit IRC16:27
*** reedip_ has quit IRC16:27
*** smatzek has joined #openstack-nova16:27
*** brault has quit IRC16:27
*** brault has joined #openstack-nova16:28
*** esberglu has joined #openstack-nova16:29
*** smatzek has quit IRC16:32
*** felipemonteiro_ has quit IRC16:32
*** salv-orlando has joined #openstack-nova16:32
*** salv-orlando has quit IRC16:33
*** ijw has joined #openstack-nova16:33
*** smatzek has joined #openstack-nova16:33
*** priya has quit IRC16:35
*** salv-orlando has joined #openstack-nova16:37
*** smatzek has quit IRC16:38
*** ragiman has quit IRC16:38
*** catintheroof has joined #openstack-nova16:40
*** reedip_ has joined #openstack-nova16:41
*** tbachman has quit IRC16:41
*** sree has joined #openstack-nova16:42
*** Apoorva has joined #openstack-nova16:44
*** catintheroof has quit IRC16:45
*** sree has quit IRC16:47
*** yamahata has joined #openstack-nova16:47
*** reedip_ has quit IRC16:48
*** smatzek has joined #openstack-nova16:53
*** smatzek has quit IRC16:53
*** smatzek has joined #openstack-nova16:54
*** dikonoor has joined #openstack-nova16:55
*** bnemec has quit IRC16:57
*** moshele has joined #openstack-nova16:58
*** baoli has quit IRC17:00
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585417:00
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015917:00
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670717:00
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135717:00
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135817:00
*** baoli has joined #openstack-nova17:01
*** yamamoto has joined #openstack-nova17:03
*** trinaths has quit IRC17:03
*** trinaths has joined #openstack-nova17:05
*** mvk has joined #openstack-nova17:06
*** moshele has quit IRC17:06
*** baoli has quit IRC17:08
*** baoli has joined #openstack-nova17:09
*** edmondsw has joined #openstack-nova17:10
*** abalutoiu has joined #openstack-nova17:10
*** yamamoto has quit IRC17:11
*** lucasagomes is now known as lucas-afk17:12
*** catintheroof has joined #openstack-nova17:12
*** bnemec has joined #openstack-nova17:12
*** tbachman has joined #openstack-nova17:15
*** gouthamr has joined #openstack-nova17:17
*** markvoelker_ has joined #openstack-nova17:17
*** tbachman_ has joined #openstack-nova17:19
*** markvoelker has quit IRC17:19
*** tbachman has quit IRC17:19
*** tbachman_ is now known as tbachman17:19
*** AlexeyAbashkin has joined #openstack-nova17:20
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940417:20
openstackgerritEric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif  https://review.openstack.org/42251217:20
*** Swami has joined #openstack-nova17:21
*** markvoelker has joined #openstack-nova17:22
*** markvoelker_ has quit IRC17:23
*** salv-orlando has quit IRC17:24
*** AlexeyAbashkin has quit IRC17:24
*** fragatina has quit IRC17:24
*** felipemonteiro has joined #openstack-nova17:24
*** salv-orlando has joined #openstack-nova17:25
*** fragatina has joined #openstack-nova17:25
*** felipemonteiro_ has joined #openstack-nova17:25
*** lpetrut has quit IRC17:28
*** salv-orlando has quit IRC17:29
*** felipemonteiro has quit IRC17:29
openstackgerritDan Smith proposed openstack/nova master: Make put_allocations() retry on concurrent update  https://review.openstack.org/51670817:29
openstackgerritDan Smith proposed openstack/nova master: Refactor claim_resources() to use retries decorator  https://review.openstack.org/51670917:29
*** pcaruana has quit IRC17:29
*** fragatina has quit IRC17:29
*** sree has joined #openstack-nova17:29
*** ijw has quit IRC17:33
*** ijw has joined #openstack-nova17:33
*** ijw has quit IRC17:34
*** sree has quit IRC17:34
*** ijw has joined #openstack-nova17:34
*** liuyulong has quit IRC17:34
*** felipemonteiro_ has quit IRC17:36
*** tesseract has quit IRC17:36
*** felipemonteiro_ has joined #openstack-nova17:36
*** salv-orlando has joined #openstack-nova17:42
mriedemdansmith: so i +2ed the bottom change but i think there is a thing in the top change (or two) that we want to do in the bottom17:43
dansmithoh yeah sorry17:44
dansmithwas trying to fight through the call brain17:44
mriedemnp, i was pushing infra patches during a call this morning, so once those patches land and everything breaks we'll know why17:44
mriedemthose patches = my infra patches17:45
dansmithnoted17:45
openstackgerritDan Smith proposed openstack/nova master: Make put_allocations() retry on concurrent update  https://review.openstack.org/51670817:46
openstackgerritDan Smith proposed openstack/nova master: Refactor claim_resources() to use retries decorator  https://review.openstack.org/51670917:46
mriedemjaypipes: easy peasy ^17:48
jaypipesmriedem: cool. just wrapping up something locally and will look17:48
*** fragatina has joined #openstack-nova17:50
*** penick has quit IRC17:52
*** tomsou_ has joined #openstack-nova17:53
*** tbachman has quit IRC17:54
*** felipemonteiro__ has joined #openstack-nova17:54
*** dikonoor has quit IRC17:56
*** itlinux has quit IRC17:56
*** penick has joined #openstack-nova17:57
*** felipemonteiro_ has quit IRC17:57
*** vks1 has quit IRC18:04
*** itlinux has joined #openstack-nova18:06
*** zzzeek has quit IRC18:10
*** jpena is now known as jpena|off18:10
*** zzzeek has joined #openstack-nova18:12
*** moshele has joined #openstack-nova18:15
*** sree has joined #openstack-nova18:16
*** zzzeek has quit IRC18:17
*** zzzeek has joined #openstack-nova18:18
openstackgerritMatt Riedemann proposed openstack/nova master: libvirt: do not remove inst_base when volume-backed during resize  https://review.openstack.org/51639518:19
mriedemoomichi: ^ and https://review.openstack.org/516396 in tempest are linked18:19
mriedemverified the failure via the tempest test in the ceph and nfs jobs, now the tempest change depends on the nova fix which will have to go back to pike and ocata18:19
*** zzzeek has quit IRC18:20
*** hemna has quit IRC18:21
*** sree has quit IRC18:21
*** pchavva has quit IRC18:21
*** yassinee has quit IRC18:22
*** zzzeek has joined #openstack-nova18:25
*** sshwarts has quit IRC18:26
*** zzzeek_ has joined #openstack-nova18:27
openstackgerritMatt Riedemann proposed openstack/nova master: Add Flavor.description attribute  https://review.openstack.org/51437118:27
openstackgerritMatt Riedemann proposed openstack/nova master: Add FlavorPayload.description for versioned notifications  https://review.openstack.org/51478218:27
openstackgerritMatt Riedemann proposed openstack/nova master: Add microversion to allow setting flavor description  https://review.openstack.org/51521318:27
*** AlexeyAbashkin has joined #openstack-nova18:29
*** zzzeek has quit IRC18:30
*** ralonsoh has quit IRC18:31
*** zzzeek_ has quit IRC18:31
*** AlexeyAbashkin has quit IRC18:33
*** sambetts is now known as sambetts|afk18:34
dansmithman the gate is some kinda jammed up18:35
jaypipesyeah..18:37
*** bnemec has quit IRC18:38
*** zzzeek has joined #openstack-nova18:39
*** zzzeek has quit IRC18:42
*** abalutoiu has quit IRC18:44
*** hemna has joined #openstack-nova18:44
*** zzzeek has joined #openstack-nova18:44
mriedemhttps://www.youtube.com/watch?v=FcArnepkhv0 ?18:46
*** edand has joined #openstack-nova18:46
*** tbachman has joined #openstack-nova18:51
*** edand has quit IRC18:52
openstackgerritMatt Riedemann proposed openstack/nova master: Implement power_off/power_on for the FakeDriver  https://review.openstack.org/50993518:57
*** yamahata has quit IRC18:59
efriedmriedem Bringing FakeDriver to parity with the PowerVM in-tree driver, eh?19:03
efriedBout time you caught up.19:03
*** sree has joined #openstack-nova19:03
mriedem:)19:04
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940419:05
*** yamahata has joined #openstack-nova19:07
*** sree has quit IRC19:07
openstackgerritEric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif  https://review.openstack.org/42251219:08
openstackgerritEric Berglund proposed openstack/nova master: PowerVM Driver: config drive  https://review.openstack.org/40940419:09
mriedemedleafe: cdent: i think i've found a chink in this armor https://review.openstack.org/#/c/508164/19:10
mriedemand precisely why we wanted to include the microversion in the Selection object that the scheduler used to get allocation_candidates19:10
*** lpetrut has joined #openstack-nova19:11
openstackgerritEric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif  https://review.openstack.org/42251219:11
cdentmriedem: so the assumption I was making there is that that spec (and the related POST one) makes no demands on what the nova-side is doing, and nova can continue doing its thing at whatever microversion it likes, and we can migration the report client to the newer microversion across the board as required19:12
cdentso while we can change the output of allocation_candidates we don’t have to19:12
mriedemi think they should be consistent19:13
cdentbut if we do, we should do it soon19:13
cdents/should/could/19:13
mriedemthe point of GET /allocation_candidates was to return a thing that the caller can re-use for PUT /allocations/{consumer_id},19:13
mriedemso to make those inconsistent with the new PUT /allocations/{consumer_id} would be confusing19:13
cdentright, which it still can, if you use the right microversion19:13
mriedemespecially since nova (on the client side) doesn't do any microversoin negotiation19:14
*** catintheroof has quit IRC19:14
cdentwhere “right” is the same microversion it is currently using19:14
*** yamahata has quit IRC19:14
mriedemwe tend to hard-code to the latest in the client side nova code19:14
mriedemwhich is bad, i realize19:14
cdentit’s _good_ that it doesn’t do negotiation, instead its explicit, right?19:14
cdentno, it’s good19:14
mriedemwell, in this case,19:14
cdentit’s always good19:14
cdentit’s a thing we’ve realized during the many microversion discussions19:15
*** pcaruana has joined #openstack-nova19:15
mriedemthe client (scheduler/cell conductor) would need to know, do i PUT at version 1.12 or 1.8?19:15
cdents/discussions/therepy sessions/19:15
edleafemriedem: how would the client ever get anything other than "latest"?19:15
cdentit would carry on doing whatever it is doing now, until we change it19:15
cdentedleafe: because it always asks for a specific version19:15
*** moshele has quit IRC19:15
cdentmriedem: note, I’m not saying we should fix allocation_candidates (we probably should), just saying we’re not forced19:16
mriedemok so scheduler does a GET /allocation_candidates at 1.10 and stores the allocation_request in the Selection object which goes to the cell conductor,19:16
mriedemthe cell conductor makes a PUT /allocations/{consumer_id} request at 1.10, things should be OK19:16
mriedembecause it's old format19:17
cdentright19:17
mriedemnow let's say in the future, we add something to GET /allocation_candidates that the scheduler needs, so it starts requesting 1.1419:17
mriedemwe shove 1.14 in the Selection object,19:17
mriedemthe cell conductor blindly does a PUT /allocations/{consumer_id} at 1.14 which is now the wrong format19:17
*** yamahata has joined #openstack-nova19:17
mriedemand you get a 400 back19:17
*** moshele has joined #openstack-nova19:17
cdentexcept it would potentially die before then anyway19:18
edleafewait - why is 1.14 the wrong format for the PUT?19:18
mriedemedleafe: because PUT /allocations/{consumer_id} format is going to change19:18
mriedemper https://review.openstack.org/#/c/508164/519:18
cdentas it currently has code which modifies the allocation data structures19:18
edleafeSo requesting microversion 1.14 will return an alloc_cand that can't be PUT with 1.14?19:19
cdentuntil we change the report client code that cooks allocations client side, we can’t up the microversion at which allocation_candidates is requested19:19
mriedemcdent: die because the scheduler is assuming a certain format you mean right?19:19
cdentyes19:19
mriedemedleafe: correct19:19
edleafethat's completely messed up19:19
edleafethe whole point of allocation_candidates is defeated19:19
cdentboth of your are a bit wrong19:19
mriedemcdent: you have a fair point19:19
mriedemso i can concede that on the scheduler side,19:19
cdentif we bump the microverson on allocation_candidates we _have_ to change the format of the allocations19:20
mriedembut in general i think we need allocation_requests in GET /allocation_candidates to match PUT /allocations at the same microversion19:20
cdentby which I mean “at which allocation_candidates is requested"19:20
cdentyes, I agree, but it wasn’t specificied in that spec because I was trying to be limited19:20
mriedemyeah i understand, i think it will save us some headache later though19:20
mriedemto just do it now19:20
edleafethe only reason allocation_candidates exist is to be immediately PUT back in a claim19:21
mriedemedleafe: yes, that's my point in why we should be consistent19:21
edleafethose processses should always be in sync19:21
mriedemok so i think we're all basically agreeing19:22
cdentyes19:22
cdentone missing point19:22
mriedemcool19:22
cdentdo you think we need to update the client side too, or can that wait (that is can we pin report client to not go past 1.10 when get /allocation_cand for now)?19:23
cdentwe probably do19:23
mriedemthat can wait for now as it's opt-in19:23
cdentsince we want to get rid of the migration uuid races19:23
cdentalthough, again, we don’t _have_ to19:23
mriedemwell,19:23
*** ijw has quit IRC19:23
mriedemconductor is going to be calling POST /allocations for the swap,19:24
mriedemscheduler will still call GET /allocation_candidates at 1.10 for now19:24
*** ijw has joined #openstack-nova19:24
cdentright, but when calling POST /allocations for the swap, we are using allocations that are dict-ish19:24
cdentand the current cooking code is list ish19:24
jaypipesefried: ok, almost ready to push my "refactor AllocationCandidates._get_by_filters() mega-method" series. I think you're gonna dig it.19:25
cdentmany cascades19:25
mriedemwhen the conductor code that starts calling POST /allocations does that thing, it will have to know the microversoin and request format to use for that thing,19:25
mriedembut i don't think that's tied to GET /allocation_candidates,19:25
cdentyes19:25
mriedemit's maybe for GET /allocations19:26
mriedembut still, gonna have to know what we're requesting when we do POST /allocations19:26
efriedjaypipes Looking forward to it.  For my part, I'm about to push a new rev of the placement side of the granular parsing.  Much improved, with y'all's suggestions.19:26
mriedemi think the conductor code will be building that request itself, not taking it from GET /allocation_candidates19:26
cdentif that’s the case then cool19:26
mriedemcdent: yeah i think we'll be ok with the migration allocation swap stuff19:27
mriedemit's the scheduler -> cell via reschedule stuff that is hairy19:27
mriedemcdent: ok so you're going to tweak the spec and such? i've got to run for the elementary school halloween parade of joyfulness19:27
cdentmriedem: not tonight, but should be able to squeeze something out before I catch a plane tomorrow. if I get lost and can’t decide what to do, I’ll respond to your comments with a “huh?”19:28
*** eharney has quit IRC19:29
mriedemcdent: sure, wfm19:30
mriedemthanks19:30
*** eharney has joined #openstack-nova19:30
cdentu r welcome19:30
openstackgerritEric Fried proposed openstack/nova master: placement: Parse granular resources & traits  https://review.openstack.org/51409119:36
openstackgerritEric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates  https://review.openstack.org/51409219:36
efriedjaypipes cdent alex_xu ^19:36
efriedand gibi too ^19:37
edleafemriedem: bouncing between meeting and IRC. What did you mean by "i think the conductor code will be building that request itself, not taking it from GET /allocation_candidates"19:37
cdentedleafe: that was when managing the doubling of allocations for migrations etc19:43
edleafecdent: ah, that bit19:45
*** moshele has quit IRC19:46
*** pcaruana has quit IRC19:49
*** sree has joined #openstack-nova19:50
*** salv-orlando has quit IRC19:51
*** catintheroof has joined #openstack-nova19:55
*** sree has quit IRC19:55
efriedWhat be wrong with the zuul?20:03
* efried joins -infra...20:03
*** bhagyashris has quit IRC20:05
*** gszasz has quit IRC20:06
openstackgerritJay Pipes proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters()  https://review.openstack.org/51677820:07
openstackgerritJay Pipes proposed openstack/nova master: build ProviderSummary objects in sep function  https://review.openstack.org/51677920:07
openstackgerritJay Pipes proposed openstack/nova master: build alloc request resources for shared resources  https://review.openstack.org/51678020:07
openstackgerritJay Pipes proposed openstack/nova master: create allocation request for single provider  https://review.openstack.org/51678120:07
openstackgerritJay Pipes proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters()  https://review.openstack.org/51678220:07
jaypipesefried, alex_xu: ^^20:07
efriedjaypipes Sweet.  Trade ya.20:07
jaypipesefried: on it.20:07
openstackgerritEric Fried proposed openstack/nova master: placement: Contributor doc microversion checklist  https://review.openstack.org/51678320:13
efriedmriedem Updates we talked about the other day ^20:13
efriedmriedem Is a similar update needed at https://docs.openstack.org/nova/pike/contributor/microversions.html#other-necessary-changes ?20:15
*** cdent has quit IRC20:20
*** Swami has quit IRC20:22
*** ijw has quit IRC20:24
*** dave-mcc_ has joined #openstack-nova20:26
*** smatzek has quit IRC20:26
*** AlexeyAbashkin has joined #openstack-nova20:27
*** smatzek has joined #openstack-nova20:27
*** dave-mccowan has quit IRC20:27
*** erlon has quit IRC20:29
*** jdurgin has quit IRC20:31
*** smatzek has quit IRC20:31
*** AlexeyAbashkin has quit IRC20:31
*** jdurgin has joined #openstack-nova20:33
efriedjaypipes Is this series going to have any additional UT?20:34
jaypipesefried: no sir.20:34
efriedor functional, more to the point.20:34
jaypipesefried: none of this is unit-tested.20:35
jaypipesefried: nope. true refactoring. no functional changes at all.20:35
efriedjaypipes I haven't gone through the whole thing, but if that first thing is truly finding only RPs where *all* resources are satisfied, it's not going to be used yet (not until use_same_provider=True groups come in)20:36
efriedunless I'm totally missing something.20:36
*** sree has joined #openstack-nova20:37
jaypipesefried: yes, you're missing something :) that first one is an optimized code path for when the *deployment has no sharing providers*, not for when the *request is for same provider*. :)20:38
efriedjaypipes So it'll need to be reworked for nested.20:38
jaypipesefried: it's basically "hey, do we have any sharing providers? no? great, let's not fuck around with non-shared, shared mixology and just do this."20:38
jaypipesefried: well, nested and sharing providers are different complexities, but yes.20:39
efriedjaypipes Cause with nested in play, but without shared, the "unnumbered" group ought to be able to get its resources from anywhere in the non-sharing tree.20:39
*** edleafe has quit IRC20:39
jaypipesefried: sure, but this patch doesn't touch any of that.20:39
efriedjaypipes Cool, that's what I needed to understand.  So we're currently assuming no nested, single compute node RP; and this method also assumes no shared.20:40
jaypipesefried: correcto.20:40
*** edleafe has joined #openstack-nova20:41
efriedjaypipes And actually, when that stuff does come into play, we should leave this method alone, precisely for the use_same_provider=True case, and create different methods that handle nested and/or shared.20:41
*** sree has quit IRC20:42
jaypipesefried: k, Parse granular resources & traits patch reviewed. great work on that. much improved.20:42
mriedemefried: yes probably also need to say something in https://docs.openstack.org/nova/pike/contributor/microversions.html#other-necessary-changes about api-ref20:42
jaypipesefried: you hit the nail on the head :)20:42
efriedjaypipes Rockin.20:42
jaypipesefried: hit the nail on the head w.r.t. your description of the "leave this method alone" above.20:43
jaypipesefried: and hit the naail on the head with your patch, too... but still needs a couple minor fixups.20:43
efriedjaypipes Onnit.20:43
*** thorst_ has quit IRC20:44
*** priteau has joined #openstack-nova20:45
*** thorst has joined #openstack-nova20:47
*** tomsou_ has quit IRC20:50
*** felipemonteiro__ has quit IRC20:51
*** thorst has quit IRC20:51
*** smatzek has joined #openstack-nova20:56
*** hemna has quit IRC20:58
*** fanfi has joined #openstack-nova20:59
*** flanders_ has joined #openstack-nova21:00
*** stvnoyes has quit IRC21:00
*** smatzek has quit IRC21:01
*** stvnoyes has joined #openstack-nova21:02
*** salv-orlando has joined #openstack-nova21:05
*** penick_ has joined #openstack-nova21:05
openstackgerritEric Fried proposed openstack/nova master: placement: Parse granular resources & traits  https://review.openstack.org/51409121:07
openstackgerritEric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates  https://review.openstack.org/51409221:07
efriedjaypipes Updated ^21:07
efried(Left the WIP one alone for now)21:07
*** eharney has quit IRC21:07
*** penick has quit IRC21:08
efriedjaypipes The other side still on your radar?  https://review.openstack.org/#/c/515151/21:08
*** catintheroof has quit IRC21:09
*** ijw has joined #openstack-nova21:18
*** priteau has quit IRC21:22
*** priteau has joined #openstack-nova21:23
*** sree has joined #openstack-nova21:24
*** ijw has quit IRC21:25
*** fanfi has quit IRC21:26
*** priteau_ has joined #openstack-nova21:26
*** AlexeyAbashkin has joined #openstack-nova21:27
*** priteau has quit IRC21:27
*** sree has quit IRC21:29
*** AlexeyAbashkin has quit IRC21:31
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585421:31
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015921:31
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670721:32
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135721:32
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135821:32
*** rcernin has joined #openstack-nova21:32
*** armax_ has joined #openstack-nova21:36
*** stvnoyes has quit IRC21:37
*** stvnoyes has joined #openstack-nova21:37
*** armax has quit IRC21:38
*** armax_ is now known as armax21:38
*** lpetrut has quit IRC21:40
*** edmondsw has quit IRC21:40
efriedjaypipes Would you be *offended* if I added a functional test for _get_provider_ids_matching_all ?21:51
jaypipesefried: nope21:51
jaypipesefried: looking at your "other side" patch now.21:51
efriedjaypipes Can I chuck it in the same change set?21:51
jaypipesefried: of course!21:52
efried(which entails rebasing the series)21:52
*** amodi has quit IRC21:53
jaypipesefried: yup, go for it.21:54
openstackgerritEric Fried proposed openstack/nova master: begin refactor AllocCandidates._get_by_filters()  https://review.openstack.org/51677821:55
openstackgerritEric Fried proposed openstack/nova master: build ProviderSummary objects in sep function  https://review.openstack.org/51677921:55
openstackgerritEric Fried proposed openstack/nova master: build alloc request resources for shared resources  https://review.openstack.org/51678021:55
openstackgerritEric Fried proposed openstack/nova master: create allocation request for single provider  https://review.openstack.org/51678121:55
openstackgerritEric Fried proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters()  https://review.openstack.org/51678221:55
efriedjaypipes ^  -- It works.  I don't understand it yet, but it works.21:55
jaypipesheh21:55
*** bnemec has joined #openstack-nova21:55
*** vladikr has quit IRC21:58
*** ijw has joined #openstack-nova22:00
*** kuzko has quit IRC22:01
*** lyan has quit IRC22:04
*** kuzko has joined #openstack-nova22:06
*** marst has quit IRC22:07
*** awaugama has quit IRC22:07
*** kuzko has quit IRC22:11
*** slaweq has joined #openstack-nova22:12
openstackgerritC Leavett-Brown proposed openstack/nova master: Closes-Bug: 1728732  https://review.openstack.org/51679622:13
openstackbug 1728732 in OpenStack Compute (nova) "OpenStack nova service responds with an erroneous httpd redirect to a "GET,version_controller,show" request." [Undecided,In progress] https://launchpad.net/bugs/1728732 - Assigned to Colin Leavett-Brown (crlb-f)22:13
*** ijw has quit IRC22:13
*** slaweq has quit IRC22:15
*** moshele has joined #openstack-nova22:17
edleafeefried: guess that's better than understanding it, but it doesn't work. :)22:18
*** baoli has quit IRC22:18
efriededleafe I dunno.  If more people understand it, but it doesn't work, more than one person could conceivably fix it.22:18
*** ijw has joined #openstack-nova22:19
efriededleafe This is assuming, of course, that there are more people on the not-understanding side than on the understanding side.22:19
efriededleafe It may just be that this stuff is hard and I need to study it until it clicks.  I'd be curious to see how SQL afficionados such as yourself perceive it.22:20
efriededleafe Talking specifically about the new method in https://review.openstack.org/#/c/516778/22:20
*** ijw has quit IRC22:22
*** ijw has joined #openstack-nova22:22
*** ijw has quit IRC22:33
*** slaweq has joined #openstack-nova22:33
*** lbragstad has quit IRC22:36
*** edmondsw has joined #openstack-nova22:38
*** penick_ has quit IRC22:40
*** itlinux has quit IRC22:41
*** moshele has quit IRC22:42
*** cali_boxer has joined #openstack-nova22:42
*** edmondsw has quit IRC22:43
*** priteau_ has quit IRC22:47
*** mriedem has quit IRC22:50
*** stvnoyes has quit IRC22:53
*** ijw has joined #openstack-nova22:53
*** stvnoyes has joined #openstack-nova22:54
*** hongbin has quit IRC22:55
*** slaweq has quit IRC22:59
*** chyka has joined #openstack-nova23:00
*** gyee has quit IRC23:10
*** brault_ has joined #openstack-nova23:18
*** brault has quit IRC23:18
*** flanders_ has quit IRC23:19
*** yamahata has quit IRC23:21
*** yamahata has joined #openstack-nova23:21
*** moshele has joined #openstack-nova23:23
*** thorst has joined #openstack-nova23:24
*** gmann_afk is now known as gmann23:27
*** slaweq has joined #openstack-nova23:29
*** thorst has quit IRC23:31
*** chyka has quit IRC23:31
*** chyka has joined #openstack-nova23:31
openstackgerritEric Fried proposed openstack/nova master: placement: Parse granular resources & traits  https://review.openstack.org/51409123:33
openstackgerritEric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates  https://review.openstack.org/51409223:33
*** hieulq has quit IRC23:35
*** hieulq has joined #openstack-nova23:36
*** chyka has quit IRC23:36
*** nicolasbock has quit IRC23:39
*** moshele has quit IRC23:40
*** moshele has joined #openstack-nova23:40
*** baoli has joined #openstack-nova23:50
*** moshele has quit IRC23:50
*** markvoelker has quit IRC23:51
*** adreznec has quit IRC23:52
*** moshele has joined #openstack-nova23:54
*** takashin has joined #openstack-nova23:54
*** sdague has quit IRC23:56
*** trinaths has quit IRC23:57
*** trinaths has joined #openstack-nova23:57

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