Monday, 2017-01-23

*** thorst_ has joined #openstack-nova00:03
*** sacharya has joined #openstack-nova00:03
*** jamielennox|away is now known as jamielennox00:05
*** voelzmo has joined #openstack-nova00:06
*** ducttape_ has joined #openstack-nova00:06
*** thorst_ has quit IRC00:08
*** voelzmo has quit IRC00:11
*** diana_clarke has joined #openstack-nova00:15
*** tbachman has joined #openstack-nova00:19
*** jamesdenton has joined #openstack-nova00:22
*** gouthamr has quit IRC00:24
*** sacharya has quit IRC00:27
*** sacharya has joined #openstack-nova00:27
*** gouthamr has joined #openstack-nova00:33
*** ducttape_ has quit IRC00:37
*** ducttape_ has joined #openstack-nova00:37
*** ducttape_ has quit IRC00:37
*** ducttape_ has joined #openstack-nova00:38
*** hfu has joined #openstack-nova00:39
*** ducttape_ has joined #openstack-nova00:42
*** edmondsw has joined #openstack-nova00:43
*** mtanino has quit IRC00:43
*** edmondsw has quit IRC00:48
*** thorst_ has joined #openstack-nova00:50
openstackgerritDigambar proposed openstack/nova-specs: Add placement nofications using versionedObjects.  https://review.openstack.org/42387200:51
*** claudiub has quit IRC00:53
*** thorst_ has quit IRC00:53
*** hoangcx has joined #openstack-nova00:54
*** sacharya has quit IRC00:57
*** alexpilotti has joined #openstack-nova01:00
*** jamesdenton has quit IRC01:00
*** alexpilotti has quit IRC01:04
*** jamesdenton has joined #openstack-nova01:06
*** phuongnh has joined #openstack-nova01:07
*** ijw has joined #openstack-nova01:09
*** abalutoiu__ has joined #openstack-nova01:10
*** abalutoiu_ has quit IRC01:13
*** yuntongjin has joined #openstack-nova01:38
*** dikonoor has quit IRC01:39
*** jamesdenton has quit IRC01:45
openstackgerritzhangyujun proposed openstack/nova: move 'MEMPAGES_*' to fields 'fileds.MemoryPageSize'  https://review.openstack.org/41774201:47
*** mingyu has joined #openstack-nova01:49
*** thorst_ has joined #openstack-nova01:55
*** thorst_ has quit IRC01:56
*** ijw_ has joined #openstack-nova01:56
*** ijw has quit IRC01:59
*** baoli has quit IRC02:05
*** voelzmo has joined #openstack-nova02:07
*** voelzmo has quit IRC02:12
openstackgerritDigambar proposed openstack/nova-specs: Add placement nofications using versionedObjects  https://review.openstack.org/42387202:13
*** tiendc has joined #openstack-nova02:16
*** artom_ has quit IRC02:16
openstackgerritDigambar proposed openstack/nova-specs: Add placement nofications using versionedObjects  https://review.openstack.org/42387202:19
*** esmiurium has quit IRC02:20
*** esmiurium has joined #openstack-nova02:26
*** yuntongjin has quit IRC02:29
*** mingyu has quit IRC02:31
*** yuntongjin has joined #openstack-nova02:37
*** dave-mccowan has joined #openstack-nova02:38
*** sacharya has joined #openstack-nova02:43
*** jamesdenton has joined #openstack-nova02:45
*** mingyu has joined #openstack-nova02:46
*** thorst_ has joined #openstack-nova02:48
*** thorst_ has quit IRC02:48
openstackgerritDavanum Srinivas (dims) proposed openstack/os-vif: [py35] Switch filter to list comprehensions  https://review.openstack.org/42371802:53
*** jerrygb has joined #openstack-nova02:55
*** tbachman has quit IRC02:56
*** amotoki has quit IRC02:59
*** Apoorva has joined #openstack-nova03:00
*** Apoorva has quit IRC03:03
*** gongysh has joined #openstack-nova03:05
*** amotoki has joined #openstack-nova03:10
*** amotoki has quit IRC03:15
*** david-lyle has joined #openstack-nova03:15
*** kevinz has joined #openstack-nova03:16
*** amotoki has joined #openstack-nova03:20
*** david-lyle has quit IRC03:21
*** baoli has joined #openstack-nova03:26
*** ducttape_ has quit IRC03:28
*** ducttape_ has joined #openstack-nova03:28
*** ducttape_ has quit IRC03:29
*** ducttape_ has joined #openstack-nova03:29
*** baoli has quit IRC03:30
*** sudipto has joined #openstack-nova03:31
*** sudipto_ has joined #openstack-nova03:31
*** amotoki_ has joined #openstack-nova03:33
*** takashin has joined #openstack-nova03:35
*** amotoki has quit IRC03:35
gmannoomichi: looks nice just few query/comment - https://review.openstack.org/#/c/406377/303:36
*** yuntongjin has quit IRC03:36
*** zz_dimtruck is now known as dimtruck03:39
*** amotoki_ has quit IRC03:42
*** esmiurium has quit IRC03:42
*** jamesdenton has quit IRC03:45
*** esmiurium has joined #openstack-nova03:49
*** baoli has joined #openstack-nova03:53
*** dave-mccowan has quit IRC03:53
*** esmiurium has quit IRC03:54
*** dikonoor has joined #openstack-nova03:54
*** baoli has quit IRC04:00
*** jerrygb has quit IRC04:03
*** Sukhdev has joined #openstack-nova04:03
*** dave-mccowan has joined #openstack-nova04:04
*** abalutoiu__ has quit IRC04:05
Kevin_Zhenggmann: https://bugs.launchpad.net/nova/+bug/165857104:07
openstackLaunchpad bug 1658571 in OpenStack Compute (nova) "Microversion 2.37 break 2.32 usage" [Undecided,New] - Assigned to Zhenyu Zheng (zhengzhenyu)04:07
Kevin_Zhenglooks like I found a critical bug here04:08
*** sree has joined #openstack-nova04:09
*** sree has joined #openstack-nova04:09
*** tbachman has joined #openstack-nova04:09
*** ducttape_ has joined #openstack-nova04:10
*** ducttape_ has quit IRC04:11
*** ducttape_ has joined #openstack-nova04:12
*** ducttape_ has quit IRC04:12
*** ducttape_ has joined #openstack-nova04:12
*** Dinesh_Bhor has joined #openstack-nova04:13
*** ducttape_ has quit IRC04:13
*** ducttape_ has joined #openstack-nova04:13
gmannKevin_Zheng:  ohh, yea you are right. it overridden the tag field. :(04:14
*** ducttape_ has quit IRC04:14
*** nicolasbock has quit IRC04:14
gmannKevin_Zheng:  i agree we should backport that04:14
Kevin_Zhengwill try to fix this after launch04:15
gmannKevin_Zheng:  very nice catch04:15
Kevin_Zheng:)04:15
gmannKevin_Zheng: +104:15
*** amotoki has joined #openstack-nova04:16
*** jwcroppe has quit IRC04:17
*** mdnadeem has joined #openstack-nova04:22
*** tbachman has quit IRC04:23
*** psachin has joined #openstack-nova04:24
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases  https://review.openstack.org/39543604:25
*** dikonoor has quit IRC04:26
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2)  https://review.openstack.org/40895504:26
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2)  https://review.openstack.org/40896404:26
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host  https://review.openstack.org/40670704:27
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host  https://review.openstack.org/41492604:27
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc  https://review.openstack.org/41691104:27
*** sofie has joined #openstack-nova04:27
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters  https://review.openstack.org/41688204:27
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters  https://review.openstack.org/42062604:27
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004204:27
openstackgerritTakashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls  https://review.openstack.org/41387604:27
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases  https://review.openstack.org/39473804:27
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc  https://review.openstack.org/41644404:27
*** dave-mccowan has quit IRC04:35
*** gongysh has quit IRC04:36
*** mingyu has quit IRC04:43
*** esmiurium has joined #openstack-nova04:45
*** mdnadeem has quit IRC04:46
*** thorst_ has joined #openstack-nova04:49
*** thorst_ has quit IRC04:54
*** esmiurium has quit IRC05:01
*** udesale has joined #openstack-nova05:02
*** yuntongjin has joined #openstack-nova05:03
*** gouthamr has quit IRC05:07
*** esmiurium has joined #openstack-nova05:07
*** ayogi has joined #openstack-nova05:08
*** voelzmo has joined #openstack-nova05:09
*** voelzmo has quit IRC05:14
*** esmiurium has quit IRC05:14
*** diga has joined #openstack-nova05:15
*** prateek has joined #openstack-nova05:15
*** gongysh has joined #openstack-nova05:16
*** ducttape_ has joined #openstack-nova05:23
*** ratailor has joined #openstack-nova05:24
*** ecelik has joined #openstack-nova05:25
*** ecelik has left #openstack-nova05:25
*** hongbin has quit IRC05:28
*** ducttape_ has quit IRC05:29
*** esmiurium has joined #openstack-nova05:31
*** gongysh has quit IRC05:32
*** ijw_ has quit IRC05:32
*** guchihiro has joined #openstack-nova05:38
*** ratailor is now known as ratailor|afk05:47
openstackgerritTakashi NATSUME proposed openstack/nova: Use ListOfIntegersField in oslo.versionedobjects  https://review.openstack.org/42391205:53
*** ratailor|afk is now known as ratailor06:01
*** jerrygb has joined #openstack-nova06:04
*** jerrygb has quit IRC06:08
*** sacharya has quit IRC06:10
*** edmondsw has joined #openstack-nova06:13
*** lpetrut has joined #openstack-nova06:14
*** mdnadeem has joined #openstack-nova06:14
openstackgerritzhangyanxian proposed openstack/nova-specs: Delete the unnecessary word in pcs-support.rst  https://review.openstack.org/42392706:17
*** edmondsw has quit IRC06:17
openstackgerritzhangyanxian proposed openstack/nova-specs: Delete the unnecessary word in pcs-support.rst  https://review.openstack.org/42392706:17
*** bkopilov has joined #openstack-nova06:23
*** moshele has joined #openstack-nova06:26
openstackgerritHussain Chachuliya proposed openstack/nova: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751406:31
*** dikonoor has joined #openstack-nova06:31
*** ekuris_ has joined #openstack-nova06:34
*** claudiub has joined #openstack-nova06:38
*** claudiub|2 has joined #openstack-nova06:40
*** claudiub has quit IRC06:42
*** sridharg has joined #openstack-nova06:45
*** thorst_ has joined #openstack-nova06:50
*** ducttape_ has joined #openstack-nova06:54
*** thorst_ has quit IRC06:55
openstackgerritzhurong proposed openstack/python-novaclient: Fix client can not auth with token  https://review.openstack.org/42394606:56
*** ducttape_ has quit IRC06:59
*** markus_z has joined #openstack-nova06:59
*** avolkov has joined #openstack-nova07:04
*** ircuser-1 has quit IRC07:06
*** hfu has quit IRC07:07
*** aojea has quit IRC07:07
*** hfu has joined #openstack-nova07:07
openstackgerritZhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37  https://review.openstack.org/42395207:08
*** dikonoor has quit IRC07:09
*** sacharya has joined #openstack-nova07:11
*** dikonoor has joined #openstack-nova07:11
*** Oku_OS-away is now known as Oku_OS07:15
*** Sukhdev has quit IRC07:15
*** sacharya has quit IRC07:15
*** sahid has joined #openstack-nova07:17
*** voelzmo has joined #openstack-nova07:18
*** aojea has joined #openstack-nova07:20
*** ircuser-1 has joined #openstack-nova07:24
*** tesseract has joined #openstack-nova07:31
*** yuntongjin has quit IRC07:33
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514207:34
*** yonglihe has joined #openstack-nova07:45
*** hfu has quit IRC07:48
openstackgerritWang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251207:52
*** takashin has left #openstack-nova07:58
*** lpetrut has quit IRC08:00
*** nkrinner_afk is now known as nkrinner08:02
snikitinBobBall: Hi, Bob! Could you please look at Xen diagnostics again? https://review.openstack.org/#/c/399613/ Citrix XenServer CI is finally green! https://review.openstack.org/#/c/413606/08:02
*** brault has joined #openstack-nova08:07
*** lpetrut has joined #openstack-nova08:08
openstackgerritgengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache  https://review.openstack.org/42375408:10
openstackgerritPawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448908:12
openstackgerritPawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration  https://review.openstack.org/32899708:12
*** amoralej|off is now known as amoralej08:12
openstackgerritPawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML  https://review.openstack.org/28674408:12
*** swamireddy has quit IRC08:15
*** dikonoor has quit IRC08:17
*** lpetrut has quit IRC08:17
*** pcaruana has joined #openstack-nova08:17
*** dikonoor has joined #openstack-nova08:18
*** ducttape_ has joined #openstack-nova08:25
openstackgerritZhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37  https://review.openstack.org/42395208:28
*** ducttape_ has quit IRC08:30
*** swamireddy has joined #openstack-nova08:34
openstackgerritZhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37  https://review.openstack.org/42395208:34
openstackgerritsahid proposed openstack/nova: numa: add numa constraints for emulator threads policy  https://review.openstack.org/38502308:38
openstackgerritsahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested  https://review.openstack.org/38536408:38
openstackgerritsahid proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell  https://review.openstack.org/40087408:38
openstackgerritsahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores  https://review.openstack.org/40087508:38
openstackgerritsahid proposed openstack/nova: virt: update overhead to take into account vCPUs  https://review.openstack.org/38502408:38
openstackgerritsahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU  https://review.openstack.org/40087808:39
openstackgerritsahid proposed openstack/nova: numa: take into account cpus reserved  https://review.openstack.org/40087608:39
openstackgerritsahid proposed openstack/nova: numa: update numa usage incording the cpu reserved  https://review.openstack.org/40087708:39
*** jpena|off is now known as jpena08:41
*** a-pugachev has joined #openstack-nova08:42
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514208:44
*** satyar has joined #openstack-nova08:47
*** thorst_ has joined #openstack-nova08:51
*** pkoniszewski_ is now known as pkoniszewski08:52
*** ralonsoh has joined #openstack-nova08:54
*** guchihiro has quit IRC08:55
*** thorst_ has quit IRC08:56
openstackgerritIstvan Imre proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication  https://review.openstack.org/41944108:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:01
*** ekuris_ has quit IRC09:04
*** mjura has joined #openstack-nova09:05
*** mlakat has joined #openstack-nova09:11
*** acabot_ has joined #openstack-nova09:12
*** ekuris_ has joined #openstack-nova09:20
*** slaweq has quit IRC09:20
*** slaweq has joined #openstack-nova09:22
openstackgerritZhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37  https://review.openstack.org/42395209:26
*** yarkot has quit IRC09:29
*** efoley has joined #openstack-nova09:29
*** abalutoiu has joined #openstack-nova09:33
*** yassine has joined #openstack-nova09:37
*** yassine is now known as Guest1726809:37
*** lennyb has joined #openstack-nova09:37
*** mingyu has joined #openstack-nova09:37
openstackgerritgengchc2 proposed openstack/nova: Improved-the-judgment-of-the-service-status-in-memcache  https://review.openstack.org/42375409:42
*** amotoki has quit IRC09:44
openstackgerritDuong Ha-Quang proposed openstack/nova: Remove mox in nova/tests/unit/compute/test_shelve.py (3)  https://review.openstack.org/34264809:46
*** derekh has joined #openstack-nova09:47
*** iceyao has joined #openstack-nova09:49
sfinucanMorning, mdbooth. Any chance you could take a gawk at https://review.openstack.org/#/c/392643/ when you have a chance?09:49
openstackgerritRikimaru Honjo proposed openstack/nova: Consolidate unit tests for shelve API  https://review.openstack.org/42122009:49
*** edmondsw has joined #openstack-nova09:50
*** lpetrut has joined #openstack-nova09:54
*** edmondsw has quit IRC09:54
alex_xujohnthetubaguy: gmann Kevin_Zheng, I guess we need some discussion on https://review.openstack.org/#/c/415330/16/nova/policies/servers.py for making progress, are you guys around?09:55
pkoniszewskihello, can I have some eyes on this series https://review.openstack.org/#/q/topic:lm-downtime ? Freeze is near, those 3 are simple patches to increase live migration success rate and reduce configuration complexity09:55
Kevin_Zhengalex_xu: you can represent me :), I'm about to driving09:55
*** ducttape_ has joined #openstack-nova09:55
alex_xuKevin_Zheng: yea, just focus on driving :)09:56
*** ekuris_ has quit IRC09:56
*** iceyao_ has joined #openstack-nova09:57
mikalwere09:58
mikalDoh, sorry09:58
*** iceyao_ has quit IRC09:58
*** hachi has joined #openstack-nova09:59
*** iceyao has quit IRC10:00
*** ducttape_ has quit IRC10:00
snikitinBobBall: thank you for review!10:03
BobBallsnikitin: Thank you for your perseverence with the CI ;)10:03
gmannalex_xu: hi10:04
gmannalex_xu:  actually problem is we cannot not know whether rule is overridden or not right?10:04
bauzasjohnthetubaguy: hola, got a sec ?10:04
bauzasjohnthetubaguy: not sure you saw my ML thread about the placement API10:05
gmannalex_xu: it can be with same default value RULE_ADMIN_API and operator can claim this was our overridden value why you broke us10:05
bauzasjohnthetubaguy: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110661.html10:05
alex_xugmann: I thought we have some way, just like this https://github.com/openstack/oslo.policy/blob/master/oslo_policy/generator.py#L21610:05
bauzasjohnthetubaguy: but I'd love your operator insights, in particular wrt the CachingScheduler10:06
*** sambetts|afk is now known as sambetts10:06
*** sapcc-bot has quit IRC10:06
*** dgonzalez_ has joined #openstack-nova10:06
*** sapcc-bot has joined #openstack-nova10:06
gmannalex_xu:  yea but if they have overridden with RULE_ADMIN_API then they expect to work on same way which makes 1st and 2nd case same10:07
openstackgerritDuong Ha-Quang proposed openstack/nova: Remove mox in nova/tests/unit/compute/test_shelve.py (3)  https://review.openstack.org/34264810:08
*** dgonzalez_ has quit IRC10:08
gmannalex_xu: making old rule to RULE_ANY breaks the old operator as johnthetubaguy pointed10:09
*** ekuris_ has joined #openstack-nova10:09
alex_xugmann: so we expect 1st and 2nd cases work as old behaviour?10:11
lyarwoodreviews on the following findmnt removal change would be appreciated if anyone has time, currently hitting EL based distros pretty hard - https://review.openstack.org/#/c/420146/10:12
*** sapcc-bot has quit IRC10:12
gmannalex_xu: only 1st but it is hard to differentiate those cases10:12
*** sapcc-bot has joined #openstack-nova10:12
*** mvk has quit IRC10:12
*** dgonzalez_ has joined #openstack-nova10:12
alex_xugmann: so we expect 2nd case as new behaviour10:12
gmannalex_xu: yea10:13
alex_xugmann: if we can check whether there is override, then we can implement all three cases.10:14
*** dgonzalez_ has quit IRC10:14
gmannalex_xu: even override with default value we consider that as 1st case right ?10:15
alex_xugmann: yes, I think it is fine10:15
alex_xugmann: that is way for operator to enforce a old behaviour10:16
gmannalex_xu: so how to check if overridden ?10:16
alex_xugmann: i'm thinking this way https://github.com/openstack/oslo.policy/blob/master/oslo_policy/generator.py#L17610:16
*** slaweq has quit IRC10:17
alex_xugmann: if the rule in the enforcer.file_rules, then it should mean the rule in the policy.json10:17
*** kevinz has quit IRC10:17
*** gszasz has joined #openstack-nova10:17
*** CristinaPauna has quit IRC10:18
*** slaweq has joined #openstack-nova10:19
gmannalex_xu:  but i think we say, generate the policy with all default rules and then override if needed, it means all rules will be in policy.json ?(not sure how operator prepare policy.json)10:19
sahidjaypipes: could you ack https://review.openstack.org/#/c/400874 i repin it to change the variable name10:19
*** mariusv has quit IRC10:19
gmanni am just assuming how people use config file10:19
*** CristinaPauna has joined #openstack-nova10:20
*** phuongnh has quit IRC10:20
*** mariusv has joined #openstack-nova10:20
*** mariusv has quit IRC10:20
*** mariusv has joined #openstack-nova10:20
alex_xugmann: we expect the policy.json empty. the policy.json should only have override rule10:24
gmannalex_xu: like this - http://docs-draft.openstack.org/30/415330/16/check/gate-nova-docs-ubuntu-xenial/13f51ca//doc/build/html/sample_policy.html10:24
gmannalex_xu: and operator just override the interested one10:24
gmannalex_xu: but m not sure about that this is how they use or they have only overriden rules in policy.json10:24
*** ociuhandu has quit IRC10:26
alex_xugmann: if the user really have override rule, there will be a warning message :)10:27
*** Hazelesque_ is now known as Hazelesque10:27
*** hoangcx has quit IRC10:27
*** abalutoiu has quit IRC10:27
*** dikonoor has quit IRC10:28
gmannalex_xu: yea that case is fine as you said but if anyone assuming overridden rule even with default value only and expect to work same way10:28
gmannthey do not believe the nova policy default values and want to make sure by overriding the rule even with same default value10:29
claudiub|2sfinucan: moshele: hello. If you have a tiny bit of time, can you take a look at the hyper-v pci passthrough patch? it is quite small too: https://review.openstack.org/#/c/420614/10:30
*** satyar has quit IRC10:31
sfinucanclaudiub|2: Just back from vacation so I'm chowing down on (a crazy amount of) mails and other reviews. If you stick me on the review though, I'll aim to get around to it today or tomorrow10:32
claudiub|2sfinucan: sure, ty. :)10:32
*** claudiub|2 is now known as claudiub10:32
alex_xugmann: emmm....maybe johnthetubaguy and sdague can ensure that whether we expect a empty policy.json for new installation10:34
mdboothsfinucan: Looking now.10:35
mdboothsfinucan: Incidentally, this one needs to get into Ocata: https://review.openstack.org/#/c/423392/ . It's a continuation of a previous code removal. Needs to be phased for upgrades, hence need for Ocata.10:36
mdboothIt's very simple.10:36
sfinucanmdbooth: Sure thing. I'll chuck that on my review queue :)10:37
*** owalsh-afk has joined #openstack-nova10:38
*** owalsh-afk is now known as owalsh10:38
gmannalex_xu: yea, that will be nice to know for me too10:41
*** cdent has joined #openstack-nova10:42
*** purplerbot has quit IRC10:46
*** mvk has joined #openstack-nova10:46
mosheleclaudiub|2: sure I will take a look later today10:47
*** purplerbot has joined #openstack-nova10:48
*** abalutoiu has joined #openstack-nova10:49
*** dikonoor has joined #openstack-nova10:51
cdentbauzas, jaypipes : was just reading back through the log about something to do with noauthmiddleware and fixtures and placement, but can't really piece together the gist. Were you able to work things out?10:51
bauzascdent: I fixed the problem10:51
*** ratailor has quit IRC10:51
bauzascdent: it was merged on Friday by adding a PlacementFixture10:52
*** sahid has quit IRC10:52
*** thorst_ has joined #openstack-nova10:52
cdentbauzas: In any case there is a functional test of placement api that does not use gabbi: https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/test_report_client.py10:52
bauzascdent: now I have another question about having resource usage call being optionals given the existing filters10:52
bauzascdent: I know10:52
cdentbauzas: and the noauthmiddleware is _not_ something I wrote special, it is the nova one adapted10:53
cdentdo you have a link to the new fixture?10:53
cdent(I mean the review that made it)10:53
bauzascdent: the problem was with the existing other functional tests that call the scheduler10:53
bauzascdent: see nova.fixtures.PlacementFixture10:53
cdentYeah, I've seen the mail about the optionals, it's next on my list to look at more closely10:53
bauzascdent: I'll be pushing a new PS by today10:53
cdentcool10:54
bauzasthat would +1 the functional tests10:54
bauzasI had a shit ton of problems related to the functional tests, but that's done10:55
*** udesale has quit IRC10:55
*** thorst_ has quit IRC10:57
cdentbauzas: did you consider using wsgi-intercept (as in the pre-existing functional test) instead of doing a server.run()?11:02
bauzasyup11:02
cdentand that wouldn't work?11:02
bauzasbut we agreed with sdague on using the Server directly11:03
bauzascdent: honestly, that looks a nit11:03
cdentI'm not complaining, I'm just trying to understand the decision process11:03
bauzascdent: given the short deadline, I don't want to discuss again about it11:03
bauzasyeah, I can understand11:03
bauzasbut sometimes you need to find solutions that work easily11:03
*** tbachman has joined #openstack-nova11:06
hachihello11:06
bauzascdent: just one point11:06
bauzascdent: I'm not sure I like https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/test_report_client.py#L89-L9211:06
bauzascdent: because we are mocking an internal attribute here11:06
hachihow i can use the policy:11:06
hachi"owner" : "user_id:%(user_id)s"11:06
hachiin nova api 2.111:06
bauzascdent: in case keystone changes its interface, we wouldn't have mock complaning at all11:07
hachiwith newton release11:07
bauzascdent: but then we would have huge problems11:07
bauzascdent: I rather preferred to use the keystone public way of avoiding the SC11:07
*** efoley_ has joined #openstack-nova11:07
cdentkeystoneauth1 has made a commitment to not change the interface on its public methods without changing the entire name of the package?11:08
bauzasbut like I said previously, that looks a nit to me and we could discuss on that later on11:08
bauzascdent: sure, we can trust them11:08
bauzascdent: that's just riskier, that's it11:08
bauzasanyway11:08
cdentand even if it did break, we could just fix it when it broke: the tests will have done their job in that case11:09
bauzasI'm totally up discussing on that after FF :)11:09
bauzasprobably we could accept not to fake our scheduler client methods and rather mock the Service object11:09
bauzasbut I think we could still use the Server proxy to really run an wsgi server and not intercept11:10
bauzasbut that's my thoughts - just coming along while tests are running on my local desktop and that I have a bit of time :)11:10
*** efoley has quit IRC11:11
*** dikonoor has quit IRC11:11
cdentOut of curiosity why do you prefer the server proxy?11:11
*** dikonoor has joined #openstack-nova11:12
bauzascdent: because that's running our internals11:14
cdentWhat are "our internals"?11:15
bauzasinternal piece of code11:15
bauzasand we're not emulating a socket, we *run* it11:15
bauzaswhich makes possible to debug the app outside of the test11:16
*** pkoniszewski has quit IRC11:16
bauzasbut meh.11:16
bauzasanyway, back to work11:16
*** andreas_s has joined #openstack-nova11:17
cdentokay, but when the placement api runs for real, it doesn't use any of the nova Server() handling11:17
*** ducttape_ has joined #openstack-nova11:22
openstackgerritBob Ball proposed openstack/nova: Use physical utilisation for cached images  https://review.openstack.org/35386211:23
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850911:25
cdentbauzas: I've got a couple questions on the CoreFilter issue that are probably easier to track here than email. You got time to answer them now or should I just post a response to the email?11:26
bauzascdent: just throw me a bit of them11:27
cdentIn the world before Placement API existed, if I disabled the CoreFilter, that means that select_destinations could return some compute nodes that would reject a "placement" and cause a retry because it didn't have enough cores. Is that right?11:29
*** tbachman has quit IRC11:30
*** krtaylor has quit IRC11:31
openstackgerritAlex Xu proposed openstack/nova: Limit the min length of string for integer JSON-Schema  https://review.openstack.org/41510311:31
cdentbauzus: If it is right, then asking the placement api for vcpu resources is simply short circuiting that retry and we should do that?11:31
*** moshele has quit IRC11:31
johnthetubaguyalex_xu: I would expect no policy.json file in a new install, empty I guess is OK too.11:31
*** dikonoor has quit IRC11:31
alex_xujohnthetubaguy: got it, thanks11:32
johnthetubaguyalex_xu: in the future I was thinking a fully commented out yaml file, a bit like what we do for config11:32
alex_xujohnthetubaguy: ok11:32
*** krtaylor has joined #openstack-nova11:33
alex_xujohnthetubaguy: I probably figure out a way to implement the 3 cases you pointed out. so I will work with Kevin_Zheng to update the patch.11:33
johnthetubaguyalex_xu: Kevin_Zheng: let me know how that goes, for me the warning message is an optional extra, as we could leave the old rules ill we can get that into olso.policy11:35
*** rfolco has joined #openstack-nova11:36
*** tbachman has joined #openstack-nova11:38
bauzascdent: sorry, missed your question11:40
bauzascdent: in a world where an operator disables the CoreFilter, we don't verify VCPUs11:40
*** tbachman has quit IRC11:41
*** ducttape_ has quit IRC11:41
bauzascdent: most of them for example are disabiling the DiskFilter for the exact reason it doesn't count correctly11:41
*** ducttape_ has joined #openstack-nova11:41
cdentwe don't verify vcpus at the scheduler level but do at the compute node level, or we don't very vcpus _anywhere_?11:41
bauzascdent: if you don't run the CoreFilter, we don't return a limit11:41
*** ducttape_ has quit IRC11:42
alex_xuhachi: we didn't suggest to use user_id based policy check. That was added back to the code just for backward-compatible, it may be removed in the future11:42
bauzascdent: if so, that's coming to the compute, and then we verify the claim without verifying the limit11:42
*** ducttape_ has joined #openstack-nova11:42
bauzascdent: which means no allocation ratio11:42
cdentinteresting11:42
*** ducttape_ has quit IRC11:42
*** ducttape_ has joined #openstack-nova11:43
*** nicolasbock has joined #openstack-nova11:43
bauzascdent: so, I'm a bit afraid that a lot of operators have disabled the DiskFilter11:43
*** ducttape_ has quit IRC11:43
bauzascdent: if now we check the disk size directly without having the aggregates created automatically for the RPs, then they could have problmes11:43
cdentso your idea is to remove resource classes from the request to /resource_providers based on which filters are disabled11:44
*** ducttape_ has joined #openstack-nova11:44
bauzascdent: yup11:44
bauzascdent: lemme upload the change this afternoon and you'll see11:44
cdentand the change would solely be on the nova-scheduler side11:44
bauzasyup11:44
*** ducttape_ has quit IRC11:44
*** haplo37_ has quit IRC11:44
cdentcool. It makes sense. My main concern was that we should change something on the placement server-side based on nova's config11:45
bauzasI just have a problem with our FakeDriver creating the same hypervisor hostname for all our computes11:45
bauzasso, once I fix that, I'll upload11:45
*** dikonoor has joined #openstack-nova11:45
openstackgerritChris Dent proposed openstack/nova: Optional separate database for placement API  https://review.openstack.org/36276611:46
openstackgerritStephen Finucane proposed openstack/nova: Fix and add some notes to the cells v2 first time setup doc  https://review.openstack.org/42142111:49
*** erlon has joined #openstack-nova11:49
*** karimb has joined #openstack-nova11:49
*** kevinz has joined #openstack-nova11:52
*** tiendc has quit IRC11:52
*** efoley__ has joined #openstack-nova11:53
openstackgerritAlvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325011:55
*** haplo37_ has joined #openstack-nova11:56
*** sree has quit IRC11:56
*** efoley_ has quit IRC11:57
*** sree has joined #openstack-nova11:59
openstackgerritZhenyu Zheng proposed openstack/nova: Fix device taging after API microversion 2.37  https://review.openstack.org/42395212:00
*** thorst_ has joined #openstack-nova12:03
*** sree has quit IRC12:04
*** thorst_ has quit IRC12:04
*** dave-mccowan has joined #openstack-nova12:05
*** diga has quit IRC12:09
*** ducttape_ has joined #openstack-nova12:12
*** mingyu has quit IRC12:15
*** amotoki has joined #openstack-nova12:15
*** ayogi has quit IRC12:17
*** mdnadeem has quit IRC12:19
*** mingyu has joined #openstack-nova12:20
*** karimb has quit IRC12:22
*** sdague has joined #openstack-nova12:22
*** ociuhandu has joined #openstack-nova12:22
*** prateek_ has joined #openstack-nova12:23
*** dave-mccowan has quit IRC12:25
*** prateek has quit IRC12:26
*** Guest17268 has quit IRC12:27
openstackgerritWang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251212:28
*** bvanhav has joined #openstack-nova12:29
*** ducttape_ has quit IRC12:32
*** amotoki has quit IRC12:33
*** mdnadeem has joined #openstack-nova12:35
*** diga has joined #openstack-nova12:36
*** baoli has joined #openstack-nova12:38
*** mingyu has quit IRC12:39
openstackgerritWang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif)  https://review.openstack.org/42251212:39
*** baoli has quit IRC12:39
*** Guest17268 has joined #openstack-nova12:40
*** ansiwen_ is now known as ansiwen12:43
*** iceyao has joined #openstack-nova12:44
*** dave-mccowan has joined #openstack-nova12:45
*** catintheroof has joined #openstack-nova12:45
openstackgerritAlex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification  https://review.openstack.org/42045312:48
*** patriciadomin_ is now known as patriciadomin12:49
*** catintheroof has quit IRC12:50
*** pkoniszewski has joined #openstack-nova12:50
*** psachin has quit IRC12:50
lyarwoodsfinucan: thanks for the review on the findmnt change, https://review.openstack.org/#/c/418371/ is another straight forward one adding a missing releasenote if you have time today12:55
*** tbachman has joined #openstack-nova12:57
*** catintheroof has joined #openstack-nova12:58
*** tblakes has joined #openstack-nova12:58
snikitinjohnthetubaguy: Hi John! can you look at instance diagnostics implementation for Xen please? https://review.openstack.org/#/c/399613/13:02
*** markus_z has quit IRC13:03
*** tbachman_ has joined #openstack-nova13:04
*** tbachman has quit IRC13:04
*** tbachman_ is now known as tbachman13:04
*** kylek3h_away has quit IRC13:04
*** ducttape_ has joined #openstack-nova13:05
hachialex_xu thank you.13:06
hachibut i don't want my users within the same project do accidently delete the VM(s) of each other.13:06
hachihow i can do that without user_id policy ?13:07
bhagyashriscdent: Hi,13:09
alex_xuhachi: you can lock the instance13:11
*** xyang1 has joined #openstack-nova13:11
cdentbhagyashris: hi, what's up?13:14
dimssean-k-mooney : jaypipes : https://review.openstack.org/#/c/423718/ does the trick for getting os-vif's linuxbridge working13:14
*** smatzek has joined #openstack-nova13:15
hachialex_xu thank you for the tip. I will try that.13:15
amrithandreykurilin, ping. I'm having a little issue with nova's 7.0.0 client, would you have a couple of minutes to help out? thx13:15
amrithone part of this is a change you submitted, https://review.openstack.org/#/c/412497/13:16
amrithappears that something changed in 7.0.0 which I'm not quite grok'ing. pl ping me if you have a minute andreykurilin. thx13:17
*** markus_z has joined #openstack-nova13:17
openstackgerritZhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants  https://review.openstack.org/41533013:18
*** edmondsw has joined #openstack-nova13:19
openstackgerritJulian proposed openstack/nova: Create cmd/script that prints JSON Schema of Notification Payload Classes  https://review.openstack.org/41183613:19
alex_xuhachi: np13:21
*** jwcroppe has joined #openstack-nova13:25
*** amoralej is now known as amoralej|luch13:26
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514213:27
*** sofie has quit IRC13:28
*** owalsh is now known as owalsh-lunch13:29
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API  https://review.openstack.org/41796113:31
openstackgerritSylvain Bauza proposed openstack/nova: Fix the FakeDriver using same hypervisor names  https://review.openstack.org/42411013:31
bauzascdent: johnthetubaguy: dansmith: latest versions for the high-prio scheduler placement changes ^13:32
* cdent enqueues13:33
bauzascdent: you'll see now the conditional branch for checking the filters before calling the placement service13:33
*** mdrabe has joined #openstack-nova13:34
bhagyashriscdent: I have few questions in my mind regarding Scheduler calling the Placement API [1] https://review.openstack.org/#/c/417961. 1. As it updates allocation information at the placement db so is there a need of compute node table for resource tracking?13:34
cdentbhagyashris: the resource tracker and the nova-scheduler will carry on needing to know about compute nodes, yes. Is that what you're asking? The placement services itself will not need to know about the compute node table, just the uuids of those compute nodes, represented as resource providers in the placement api13:37
*** satyar has joined #openstack-nova13:37
openstackgerritMikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers  https://review.openstack.org/31248813:37
*** lpetrut has quit IRC13:38
*** iceyao has quit IRC13:38
*** mingyu has joined #openstack-nova13:39
*** iceyao has joined #openstack-nova13:39
*** iceyao has quit IRC13:39
*** ducttape_ has quit IRC13:39
*** karimb has joined #openstack-nova13:40
*** kylek3h_away has joined #openstack-nova13:41
*** jpena is now known as jpena|lunch13:41
*** cleong has joined #openstack-nova13:41
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Do not allow instances with pinned CPUs to spawn  https://review.openstack.org/42411613:42
*** mingyu has quit IRC13:44
*** nicolasbock has quit IRC13:46
*** mnestratov has joined #openstack-nova13:47
sfinucanlyarwood: Done and done https://review.openstack.org/#/c/418371/113:47
sfinucanjohnthetubaguy: ^ that looks like a straightforward +W - could you take a glance if you have a moment?13:47
*** mfeoktistov has joined #openstack-nova13:48
lyarwoodsfinucan: cheers13:49
*** jheroux has joined #openstack-nova13:50
bhagyashriscdent:  I know , i am asking lot of questions :) I just want to know if we are using the placement apis then will compute node table be needed in that case? If it will used then in which case it will be used and for what?13:50
*** nicolasbock has joined #openstack-nova13:51
digajaypipes: Hi13:51
cdentbhagyashris: it will carry on being used for keeping track of the compute nodes because nova still needs to know about them13:51
johnthetubaguysfinucan: ack, thanks13:51
*** lpetrut has joined #openstack-nova13:51
cdentdiga: jaypipes is on PTO, maybe somebody else can help?13:51
digacdent: Hi, sure13:52
digacdent: https://review.openstack.org/#/c/423872/ can you review this spec - placement notification13:52
cdentyeah, sure, will stick that on my list for today13:53
*** satyar has quit IRC13:53
*** kylek3h_away is now known as kylek3h13:54
digacdent: :) thank you!13:54
digacdent: we have meeting now right ?13:54
cdentyeah, in five minutes13:54
*** tlian has joined #openstack-nova13:54
digaokay13:55
edleafediga: in #openstack-meeting-alt13:55
digaedleafe: Sure, will join13:55
*** vladikr has joined #openstack-nova13:55
*** mfeoktistov|2 has joined #openstack-nova13:55
andreykurilinamrith: hi13:56
amrithhi andreykurilin13:56
amrithdid you see the email I just sent you13:56
andreykurilinyes13:56
johnthetubaguybauzas: you say only ironic needs the host field, isn't that what we use for the RPC call?13:56
amriththe issue I'm facing is that if I use the v6.0.0 client things work fine13:57
*** gouthamr has joined #openstack-nova13:57
amrithif I switch to the v7.0.0 client, it bombs out13:57
bauzasjohnthetubaguy: I meant the problem is that host != node for Ironic13:57
johnthetubaguybauzas: or is that the node vs host thing13:57
bauzasyup that13:57
amritham not sure why that would be the case13:57
johnthetubaguybauzas: ah, gotcha13:57
*** artom has joined #openstack-nova13:57
bauzasmaybe my comment is unclear, sorry if so13:57
andreykurilinamrith: I have just several minutes before meeting and will able to help you in ~30 minutes13:57
bauzasbut I can't say that I could just use the RP name13:57
bauzasgiven it's the nodename which can be different for ironic13:58
amrithandreykurilin, I'll be here.13:58
andreykurilinamrith: but now I have one question: What is user.tenant  https://github.com/openstack/trove/blob/master/trove/tests/util/__init__.py#L170 here? Is it tenant name or id?13:58
bauzaswhile the other drivers (except the Fake driver, but that's fixed in a bottom patch) are using host == node13:58
bauzasjohnthetubaguy: ^13:58
amrithlooking13:58
*** nicolasbock has quit IRC13:58
*** amoralej|luch is now known as amoralej13:58
johnthetubaguybauzas: the custom resource class stuff should fix that, I am assuming13:59
bauzasjohnthetubaguy: yeah, but it could be merged by Pike14:00
bauzasjohnthetubaguy: hence me using the new object method I created14:00
*** kevinz has quit IRC14:00
bauzaswe could just cut that once the above BP you mentioned is fully merged14:00
*** AAR has joined #openstack-nova14:00
bauzasbut AFAICS, that's still not done yet hence my big fat comment14:01
johnthetubaguybauzas: yeah, no biggie, just getting my head straight on that14:01
*** AAR has left #openstack-nova14:01
bauzasnp14:01
bauzasjohnthetubaguy: I have a bigger concern that I wrote in a ML thread this week-end14:01
edleafebauzas: scheduler meeting?14:02
bauzasedleafe: I can do, for 20 mins14:02
edleafecool14:02
amrithandreykurilin, one second, you are looking at the tests. I wasn't running test code14:02
bauzasedleafe: but I'm pretty down given all the work I'm doing atm14:02
andreykurilinok...14:02
amrithI was running a straight trove command which tried to hit nova14:02
amrithI will find out what that is14:02
andreykurilinI just look at first failure at gates:)14:02
andreykurilin*looked14:03
amrithbut for what I've been trying to do, the code is at https://github.com/openstack/trove/blob/master/trove/common/remote.py#L10114:03
*** bvanhav_ has joined #openstack-nova14:03
amrithah, sorry14:03
amrithI am running trove locally to debug14:03
*** bvanhav has quit IRC14:03
amrithhere, we're making a client with version 2.1214:03
amrithwhich comes from https://github.com/openstack/trove/blob/master/trove/common/cfg.py#L6714:03
amrithand we're passing in a bunch of parameters14:04
amrithmy specific issue (and like I said in the email, trove may be doing something wrong here)14:04
amrithwe're making the call like https://github.com/openstack/trove/blob/master/trove/common/remote.py#L10114:04
amrithand if I understand compatibility right, this should work unchanged with the v7.0.0 client from nova14:04
andreykurilinamrith: need to go to meeting, sorry. PS: it should help(with depends-on) ttps://review.openstack.org/41249714:04
amrithbut it isn't14:04
amrithnope, doesn't help14:05
amrithtried it14:05
amrithsomething else is broken, not sure where14:05
amriththe issue I'm facing is that nova returns a 403 (forbidden) on a simple thing like flavor list14:05
andreykurilinamrith: I added depends-on there14:05
bhagyashriscdent: Ok. Thank you. :)14:06
andreykurilinamrith: did you check it with pulling not-merged changed from novcliane?14:06
amrithoh, sorry14:06
amrithdifferent change14:06
amrithlet me look14:06
amrithNO, I did not check with pulling not merged change in novaclient14:06
amriththat may be it. let me check14:06
*** nicolasbock has joined #openstack-nova14:06
amrithcatch yo after your meeting14:06
amriththanks!14:06
alex_xujohnthetubaguy: sdague, hope you can take a look at this patch https://review.openstack.org/#/c/423952/5, we break some old microversion in newton release. cdent have a good point on it. I'm not sure we can fix it directly, maybe another microversion?14:07
johnthetubaguyalex_xu: yeah, sounds like it should be added back in a the next microversion14:08
*** thorst_ has joined #openstack-nova14:08
*** esberglu has joined #openstack-nova14:09
*** mfeoktistov has left #openstack-nova14:09
alex_xujohnthetubaguy: yea, we already bump few versions after break that.14:10
*** mfeoktistov|2 has left #openstack-nova14:10
*** cjvolzka has joined #openstack-nova14:10
*** amotoki has joined #openstack-nova14:11
*** mfeoktistov has joined #openstack-nova14:12
*** lpetrut has quit IRC14:13
*** jamesdenton has joined #openstack-nova14:13
*** moshele has joined #openstack-nova14:13
*** jerrygb has joined #openstack-nova14:14
sdaguealex_xu: so, before fixing it, is there something we can learn about how to not let something like that fall through review again?14:14
sdagueno one seemed to catch that over a lot of review iterations (64 revisions on that patch, that bug showed up in revision 44)14:15
*** ducttape_ has joined #openstack-nova14:15
sdaguehttps://review.openstack.org/#/c/304510/44/nova/api/openstack/compute/block_device_mapping.py14:15
*** jerrygb_ has joined #openstack-nova14:15
diana_clarkeamrith, andreykurilin: Horizon is reporting a similar issue with python-novaclient 7.0.0. https://bugs.launchpad.net/python-novaclient/+bug/165814814:15
openstackLaunchpad bug 1658148 in OpenStack Dashboard (Horizon) "Horizon/Nova interactions broken " [Critical,New] - Assigned to Rob Cresswell (robcresswell)14:15
alex_xusdague: i'm thinking is there any test can cover that. we probably only run the test on specific version, not in the future version.14:16
alex_xubut even the test can run in the future version, but the bug only can be found when we bump a new version14:17
*** ducttape_ has quit IRC14:17
*** jerrygb__ has joined #openstack-nova14:18
sdaguealex_xu: right, I was wonder if we could iterate through the schema and compare at various versions14:18
*** jerrygb has quit IRC14:19
sdagueincluding a fake new version after the one being committed14:19
sdagueand if we see a blip and back, that's probably wrong14:19
andreykurilinamrith: ok, I returned14:19
*** baoli has joined #openstack-nova14:19
bauzasjaypipes: I thought you were on PTO14:19
alex_xusdague: yea, that should works14:19
*** owalsh-lunch has quit IRC14:20
*** hongbin has joined #openstack-nova14:20
alex_xusdague: actually that patch fixes two bugs, another one break the old microversion in the later patch.14:20
*** jerrygb_ has quit IRC14:20
alex_xumaybe we should run the api sample tests on the specific version and the lastest version. it is fine to find bug before we bump next version14:21
sdaguealex_xu: yeh14:22
sdaguethat might be good enough as well, run everything at n+114:22
sdagueI don't know, just trying to figure out how to make it easier for folks14:23
sdaguehonestly, == is probably always the wrong answer for microversion comparison14:23
alex_xuyes, that is totally wrong.14:24
*** owalsh-lunch has joined #openstack-nova14:25
*** amotoki has quit IRC14:25
sdagueif version stops being a string, and is actually a version object when it gets to get_server_create_version then it would be harder to make it an equality14:26
*** lpetrut has joined #openstack-nova14:26
*** owalsh-lunch is now known as owalsh14:26
sdagueversion.between('2.32', None)14:26
*** ducttape_ has joined #openstack-nova14:27
sdagueif you really want an equal, then do between('2.32','2.32')14:27
sdaguethat will be a lot more obvious to reviewers that it is wrong14:27
sdaguethat might be more effective than complicated testing14:28
*** catinthe_ has joined #openstack-nova14:29
*** abalutoiu_ has joined #openstack-nova14:30
*** smatzek has quit IRC14:31
*** mdnadeem has quit IRC14:31
*** ducttape_ has quit IRC14:31
*** voelzmo has quit IRC14:32
*** catintheroof has quit IRC14:32
*** dikonoor has quit IRC14:32
*** abalutoiu has quit IRC14:33
amrithhi andreykurilin14:34
amrithlet's watch the change you proposed (with the dependency)14:34
amrithand go from there14:34
amrithI'm looking at the test you mentioned14:34
amrithwhich also failed14:34
amrithlast time around14:34
*** sridharg has quit IRC14:35
andreykurilinamrith: actually, it looks like I have a workaround (depends-on patch looks like not ready yet and require some additional work). I'll propose one more change14:35
amrithok, thx andreykurilin14:35
*** diga has quit IRC14:36
*** voelzmo has joined #openstack-nova14:36
*** moshele has quit IRC14:36
alex_xusdague: yea, we can pass the object instead of string. but the object support __eq__ :)14:36
*** satyar has joined #openstack-nova14:38
*** voelzmo has quit IRC14:38
*** hongbin has quit IRC14:39
*** hongbin has joined #openstack-nova14:40
alex_xujohnthetubaguy: sdague, so that bug need to be fixed by microversion, is that something we can done before release, or it will be a Pike thing. And have a reno 'issue' note in Ocata?14:40
*** cdent has quit IRC14:40
*** mingyu has joined #openstack-nova14:40
*** liverpooler has joined #openstack-nova14:41
sdaguealex_xu: I think we should make the object stop supporting eq natively14:41
johnthetubaguyI am with sdague on stopping the route cause here urgently14:42
alex_xusdague johnthetubaguy : ok, I will check that tomorrow14:42
sdaguealex_xu: so, what happens when 2.33 is used?14:42
andreykurilinamrith: what is context.token here https://github.com/openstack/trove/blob/master/trove/common/remote.py#L102 ?)14:43
*** slaweq_ has joined #openstack-nova14:44
*** mingyu has quit IRC14:45
alex_xusdague: emm...looks like there is no 'addtionalProperties: False', that means the tag still work in the 2.33, just lose validation, I need to check that14:45
*** liverpooler has quit IRC14:45
*** eharney has joined #openstack-nova14:45
*** xinliang has quit IRC14:45
*** liverpooler has joined #openstack-nova14:45
jrolledleafe: one more question, did we figure out what an ironic flavor using resource classes looks like yet?14:45
jrolljaypipes: ^^14:45
sdaguealex_xu: it would be extremely useful to me to get a mailing list post out about what went wrong, what this allows, and what the fall out from various fixing options would be14:45
sdaguebecause otherwise everyone needs to build that up in their own head from scratch, and it's hard to agree on path forward unless we all agree where we stand14:46
*** mingyu has joined #openstack-nova14:46
edleafejroll: good question14:46
alex_xusdague: i see now. I will investigate more, then send out an email14:47
jrolledleafe: I'd like to have that in my docs for the ironic install guide, I guess14:47
jrollthough, if it doesn't work until pike, idk14:47
edleafejroll: I'm still not clear on whether we need to add the resource class to the flavor14:47
jrolledleafe: how else do we map a flavor to said resource class?14:47
edleafejroll: or if there is some other mapping14:47
sdaguealex_xu: thank you14:47
edleafejroll: jinx14:47
alex_xusdague: np14:48
*** jpena|lunch is now known as jpena14:48
edleafejroll: my understanding is that the flavor will be: CUSTOM_IRONIC_CLASS: 114:48
* alex_xu also learned the way to resolve problem.14:48
edleafejroll: and that placement will find a provider for that class14:49
jrolledleafe: where, in the flavor? e.g. how do I create said flavor?14:49
edleafejroll: IOW, no disk/ram/cpu BS like now14:49
openstackgerritPawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448914:50
openstackgerritPawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration  https://review.openstack.org/32899714:50
openstackgerritPawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML  https://review.openstack.org/28674414:50
edleafejroll: well, like any other flavor, but not sure what changes the custom RC stuff will need14:50
jrolledleafe: sure, so just nova flavor-key my-flavor set CUSTOM_FOO_THING=1?14:50
jrollright14:50
edleafejroll: jaypipes had all this on his mental whiteboard14:50
jrolledleafe: so here's the thing, if we require this to be set up for a smooth upgrade to pike, then we need to be able to make a working flavor in ocata14:51
edleafejroll: agreed14:51
jrolledleafe: and we're three days from feature freeze and this is but a butterfly in jaypipes' brain, correct?14:51
edleafejroll: unless that butterfly laid some eggs in cdent's brain, yet14:52
*** eharney has quit IRC14:52
edleafes/yet/yes14:52
*** udesale has joined #openstack-nova14:52
edleafejroll: He may have written it up somewhere - I can dig for it14:52
jrolledleafe: I mean, if it isn't written down at this point, I don't believe it will happen, which means this can't be mandatory fora pike upgrade14:52
jrollokm,14:52
jrollok*14:52
jrollthanks14:53
jrollall I've seen is maybe an extra spec14:53
edleafejroll: I'll let you know what I find14:53
jrolledleafe: thank you14:53
*** dimtruck is now known as zz_dimtruck14:54
*** zz_dimtruck is now known as dimtruck14:54
*** ducttape_ has joined #openstack-nova14:55
*** tbachman has quit IRC14:55
*** jaosorior has joined #openstack-nova14:56
sfinucanvladikr: Comments left on https://review.openstack.org/#/c/421186/214:56
*** smatzek has joined #openstack-nova14:57
*** xinliang has joined #openstack-nova14:57
*** smatzek_ has joined #openstack-nova14:58
vladikrsfinucan, thanks14:58
*** _ducttape_ has joined #openstack-nova14:58
*** tbachman has joined #openstack-nova14:58
*** Tahvok has joined #openstack-nova14:59
amrithandreykurilin, the token there is something that the client (trove client) would have created and passed to the server. so the auth and token creation is done on the trove client side and the API service (or TM) on the trove control plane uses those credentials. That would be a keystone token.15:00
*** hongbin has quit IRC15:00
*** hongbin_ has joined #openstack-nova15:00
andreykurilinamrith: oh, sorry. I wrote the wrong question :( it should be : what is tenant there15:01
amrithproject_id15:01
dansmithsdague: sfinucan: johnthetubaguy: bauzas: The two patches at the head of this series have been +1 from jenkins for a while and have seen little review. They're not super complicated and I'd really like to see them get in while we work on the third, which is the major goal for ocata for cells: https://review.openstack.org/#/c/39677515:01
*** smatzek has quit IRC15:01
*** ducttape_ has quit IRC15:01
andreykurilinamrith: nice15:02
bauzasdansmith: okay, I can take a look now15:02
dansmiththanks15:02
sfinucandansmith: Likewise15:02
*** _ducttape_ has quit IRC15:03
*** dimtruck is now known as zz_dimtruck15:03
dansmithalso thanks :)15:04
*** eharney has joined #openstack-nova15:06
ildikovjohnthetubaguy Hi, Can you make it to the meeting today?15:06
*** hongbin_ has quit IRC15:08
openstackgerritStephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology  https://review.openstack.org/41947915:12
*** voelzmo has joined #openstack-nova15:13
sfinucandansmith, sahid: If ye could take another look at ^, I'd be mighty appreciative15:13
bauzasdansmith: comments in your bottom change15:13
dansmithbauzas: thanks15:14
dansmithsfinucan: yep15:14
bauzasdansmith: just wanted to make sure we have a limit but not a marker, right?15:15
dansmithbauzas: the marker is handled in that code15:16
bauzasdansmith: mmm? lemme look, I maybe missed it15:17
dansmithbauzas: we pass the marker down to all the sub-list commands and honor the limit at the top15:17
*** mtanino has joined #openstack-nova15:19
*** voelzmo has quit IRC15:19
*** knikolla has joined #openstack-nova15:20
*** knikolla has quit IRC15:20
bauzasdansmith: I'm seeing it now15:20
*** knikolla has joined #openstack-nova15:20
bauzasdansmith: but I actually wonder how it works with 2 cells not having the same number of instances in there15:20
bauzasdansmith: but lemme look at that15:20
dansmithbauzas: how what works?15:20
bauzasmy brain is just fried by the scheduler upcall so I could be misunderstanding15:21
bauzasdansmith: so, you construct a global limit and you pass it down to the cell-related instance.get_all() call15:21
bauzaswith leaving the marker, right?15:21
dansmiththe global limit is just to make sure that we don't keep querying cells for more instances15:22
dansmithso if you limit=100 and we return 90 from the first cell, then the second cell should only fetch 10, and the third cell 015:22
dansmithsfinucan: why no test on that filter change/15:23
bauzasdansmith: right, I understood that15:24
sfinucandansmith: Because I couldn't figure out where to slot it in šŸ™ˆ15:24
bauzasI'm thinking of the limit=50 and cell1 having 6015:24
*** zz_dimtruck is now known as dimtruck15:24
dansmithbauzas: then we'll limit=50 on the first cell, and then limit will be zero afterwards, an we will never  call the next one15:25
bauzasin that case, we get the first 50 elements and we set the marker to the 50th injstance15:25
dansmithbauzas: and return 50 instances15:25
bauzasright15:25
bauzasso, next call would get the cell1 marker to the 50th instance, so a new limit of 50 would return the 10 left + 40 from cell2 if we have enough, right?15:25
bauzasI think I see the logic15:26
bauzaswe consume all instances from cell to cell15:26
bauzasand the marker is just related to the cell currently half-checked15:26
dansmithright15:26
bauzasdansmith: because there is huge assumption that we get once the cells, and we don't get again15:28
dansmithbauzas: I'm not sure what you mean15:28
bauzasdansmith: you only set your global cells value once, right?15:28
dansmithbauzas: you mean we never re-query the list of cells currently15:28
*** mingyu has quit IRC15:29
dansmith?15:29
bauzasyeah, your TODO15:29
bauzasI mean your FIXME sorry15:29
jrolledleafe: line 70 here is the major reason I need to figure out what a flavor looks like https://review.openstack.org/#/c/345081/6/install-guide/source/include/configure-nova-flavors.rst15:29
*** mingyu has joined #openstack-nova15:29
dansmithbauzas: that's correct -- what does that have to do with the limit/marker thing?15:29
*** prateek_ has quit IRC15:29
bauzasdansmith: because we could imagine that we look over 3 cells15:29
*** baoli has quit IRC15:30
bauzashave the marker in the middle of the 2nd cell15:30
bauzasand then all instances newly created in cell1 would never appear15:30
bauzasin a further call15:30
edleafejroll: yeah, I see that15:30
dansmithsfinucan: test_numa_topology_filters.py ?15:30
edleafejroll: looking through the flavor docs, they are super-VM centric. So I think extra-specs will have to be used for now15:31
*** ducttape_ has joined #openstack-nova15:31
dansmithbauzas: that's the sorting problem we've already identified, right? either way, pagination is always lossy and if you are sorted on something like uuid you'd not see those new instances even in a single cell currently15:31
edleafejroll: the other resource classes are hard-coded into flavors15:31
jrolledleafe: bleh15:32
edleafejroll: which I wanted flavors to DIAFF long ago15:32
edleafes/which/which is15:32
bauzasdansmith: sure, I just wanted to make sure we are at the same page15:32
jrolledleafe: now I'm wondering how nova behaves if nothing is set but resource classes :P15:32
jrollI guess this is why we CI15:32
* jroll continues working on that15:32
dansmithbauzas: yes, we're on the same marker :)15:32
edleafejroll: cdent is away ATM, but I'll pick his brain when he gets back15:33
openstackgerritMaciej Szankin proposed openstack/nova: conf: Resolve TODOs in 'database'  https://review.openstack.org/39369515:33
jrolledleafe: yeah, fair, thanks15:33
*** ekuris_ has quit IRC15:33
*** beekneemech is now known as bnemec15:33
bauzasdansmith: since you're working on a new PS, maybe writing this scenario into a unittest could be nice to me15:34
bauzasdansmith: like, create 3 cells, iterate over 1 and then the 2nd, add a new instance to cell1 and make sure that it's never read15:34
bauzasdansmith: because I think that it could be something important for operators wanting to have multi-cells15:35
bauzasand they could come to us and say 'why can't I see my new instances?"15:35
dansmithbauzas: ...really?15:35
bauzasdansmith: well, at least documenting it then15:35
dansmithbauzas: but it's a marker/limit query and that situation exists today15:35
bauzassomething saying that we know our limitations15:35
dansmithbut it's exactly the way it behaves today :)15:36
sfinucanSo I'm clear, the marker is applied before the limit, right?15:37
bauzasdansmith: okay then nevermind15:37
*** cdent has joined #openstack-nova15:37
vladikrdansmith, I have reworked the "sriov-pf-passthrough-neutron-port-vlan" series according to your suggestion to get the vlan from the info_cache, could you please take a look when you have the time, before it hits the code freeze ? https://review.openstack.org/#/q/topic:bp/sriov-pf-passthrough-neutron-port-vlan15:37
sfinucanI'm curious what would happen if there was nothing matching the marker in the first cell but we hadn't indexed any others yet. I'm guessing '_get_instances_by_filters' will return nothing and limit wouldn't be deprecated, but just making sure :)15:37
*** udesale has quit IRC15:37
*** voelzmo has joined #openstack-nova15:37
*** efoley has joined #openstack-nova15:40
*** efoley__ has quit IRC15:41
cdentjroll, edleafe: re [t 22sr] no catepillars, just maggots15:41
purplerbot<edleafe> jroll: unless that butterfly laid some eggs in cdent's brain, yet [2017-01-23 14:52:19] [n 22sr]15:41
*** baoli has joined #openstack-nova15:41
jroll:|15:41
edleafecdent: so extra_specs is all we got?15:41
cdentI vaguely recall this being discussed in a review and that extra_specs was the way to go for now because we hadn't had a chance to kill flavors15:42
edleafecdent: and do we have the code in place to convert 'resource_class' in extra_specs to an actual ironic request?15:42
cdentbut I wouldn't want to be held to that version of the truth15:42
*** sdague has quit IRC15:43
edleafegod, extra_specs is some special kind of suck15:43
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622515:43
cdentit doesn't look like it, but my understanding was that for this cycle we weren't planning to. all we were caring about was getting the inventory right, not using it (in the ironic case)15:44
amrithandreykurilin, looks like https://review.openstack.org/#/c/423086/ is trying to fix something related to what you are addressing; I've set that one to depend on your change; let's see how all three together do15:44
edleafecdent: so what jroll needs is to be able to write the guidance for ironic operators for ocata so that they are in a good place when pike rolls out15:45
dansmithsfinucan: ah, we probably need to skip NotFound if we have a marker and limit!=015:45
cdentedleafe, jroll I'm sorry I don't know more. My devious plan to get people to correct me when I'm wrong or leave things out in the rp updates has been a fail15:45
*** jerrygb has joined #openstack-nova15:45
dansmithsfinucan: that's what the intent is, otherwise the marker would always have to be in cell115:45
dansmithwhich wouldn't be the case after we can have multiple15:45
jrollcdent: and my plan to make sure that's in a good place has been undermined by a million other things to do15:45
cdentplans suck, or something15:46
* jroll is so excited that there's only a couple of weeks until I can mostly just write code15:46
andreykurilinamrith: it should fix only tests...15:46
*** mingyu has quit IRC15:46
amrithandreykurilin, I don't even understand that test15:46
sfinucanjroll: And review all the code, like a good core :)15:46
jrollsfinucan: I'll think about it15:47
jroll:)15:47
amrithwhy it makes sense for a test to inspect some internal attributes of the nova client object I know not15:47
sfinucanheh :)15:47
andreykurilinamrith: heh15:47
amrithseems like that test would be more appropriate if it were in nova15:47
amrithbut this is a test in trove :)15:47
*** jerrygb__ has quit IRC15:47
amriththe fix seems to correct for the fact that the variable is no longer context.user but why should trove care?15:47
amrithmy 2c worth15:47
cdentjroll, edleafe: I reckon for the guideance the thing to do at this stage is establish a pattern of people setting the resource class on the ironic side so that the inventory setting, established in this patch: https://review.openstack.org/#/c/404472/ can happen15:48
amrithlet's see if it passes tests; I'll fix the test a different way if it does and dep on your change15:48
jrollcdent: I agree, I just don't think we can require that for a pike upgrade, if people can't transition their flavors in ocata15:48
cdentUnfortunately I can't really comment on that: I don't have enough of a picture of all the pieces15:49
*** bvanhav_ is now known as bvanhav15:49
edleafejroll: I can't imaging flavors being nuked any time soon - they're just too entrenched to kill without splattering their guts all over15:50
edleafejroll: so I would recommend going the extra_specs route for the foreseeable future15:50
edleafejroll: it sucks bigly, but that's all we have to work with15:50
jrolledleafe: if we're only tracking resources the new way (as we just said we wanted to in pike), then the existing flavors (which include cpu/ram/gb) won't work. which means we need to be able to make new flavors (with only resource classes) in ocata, and those need to work (I think?).15:51
*** mriedem has joined #openstack-nova15:52
*** mfeoktistov has quit IRC15:53
mriedembuenos dias15:53
*** yamahata has joined #openstack-nova15:54
*** jose-phillips has joined #openstack-nova15:55
edleafemriedem: that's not how you vacation!15:55
mriedemit's how *i* vacation15:55
ildikovmriedem: hi15:56
ildikovmriedem: are you on vacation today?15:56
*** hongbin has joined #openstack-nova15:56
mriedemildikov: yes15:56
*** a-pugachev has quit IRC15:56
openstackgerritMaciej Szankin proposed openstack/nova: conf: Resolve TODOs in 'database'  https://review.openstack.org/39369515:57
ildikovmriedem: oh nice, then I'll not harass you to come to the meeting :)15:57
ildikovmriedem: BTW, would any other day at the same time work for you in general?15:57
ildikovmriedem: there are a few folks with collisions so I'm trying to find another slot that can work15:57
*** _ducttape_ has joined #openstack-nova15:57
ildikovmriedem: no other questions before saying Happy Vacations! :)15:58
*** burgerk has joined #openstack-nova15:59
mriedemildikov: the time slot itself in general works for me16:00
ildikovmriedem: great, thanks!16:00
*** ducttape_ has quit IRC16:00
*** efoley_ has joined #openstack-nova16:01
*** nklenke has joined #openstack-nova16:02
*** ratailor has joined #openstack-nova16:02
*** efoley has quit IRC16:05
*** amotoki has joined #openstack-nova16:06
*** Jeffrey4l_ has quit IRC16:06
*** Jeffrey4l_ has joined #openstack-nova16:06
*** voelzmo has quit IRC16:07
cdentedleafe: since you committed to rebasing the nested resource providers, shall I do the customer resource classes? https://review.openstack.org/#/q/topic:bp/custom-resource-classes+status:open16:09
edleafecdent: I was gonna do them both, but if you're bored...16:09
cdentedleafe: go for it, definitely not bored16:10
*** gouthamr has quit IRC16:12
BobBallIs there a way to set per-driver defaults for config values?  For example, ram_allocation_ratio defaults to 1.5 - but this does not make sense for XenServer since the driver doesn't enable over-commit.16:12
*** gouthamr has joined #openstack-nova16:12
mriedemgibi: http://logs.openstack.org/45/403745/11/gate/gate-nova-tox-db-functional-ubuntu-xenial/76b1646/console.html#_2017-01-23_15_02_47_38225116:13
*** markus_z has quit IRC16:13
mriedemBobBall: nope16:13
sfinucanBobBall: Not that I'm aware off, I'm afraid. That would probably be something for your deployment tool16:13
sfinucan*of16:13
BobBallWell, poohsticks.16:13
mriedemBobBall: i'd expect different virt drivers to be configured on different compute hosts16:13
*** pcaruana has quit IRC16:13
BobBallOf course - admins can set ram_allocation_ratio in the config file, I was just trying to reduce their work by making the defaults make sense :)16:14
mriedemBobBall: well...you could possibly do something fancy on startup if you know which virt driver is used and if the user has explicitly set the config rather than just picking up the default,16:15
mriedemi.e. change the default in code based on which driver is used at runtime16:15
mriedembut i don't think there is anything in oslo.config to tell you if the option was loaded from config or defauls16:16
jrollmriedem: BobBall: couldn't you use set_default? http://docs.openstack.org/developer/oslo.config/configopts.html#oslo_config.cfg.ConfigOpts.set_default16:17
*** mvk has quit IRC16:18
*** jose-phillips has quit IRC16:18
mriedemjroll: yeah maybe when loading up the compute driver16:20
bauzasf****, we have a shit ton of tests that are having 'fake-mini' as hypervisor_hostname16:20
bauzas:(16:20
*** tbachman has quit IRC16:20
*** tbachman_ has joined #openstack-nova16:20
bauzascdent: mriedem: dansmith: I think we have a very important problem with our unique constraint about the RP name being unique :/16:21
*** moshele has joined #openstack-nova16:21
jrollmriedem: oh, that config is in the scheduler process, isn't it, maybe not then16:21
*** nklenke has quit IRC16:21
cdentbauzas: do tell?16:21
erlonandreykurilin: hey andrey, I have found out that service_catalog is not part of novaclient anymore16:22
erlonandreykurilin: how suppose a client to use it now?16:22
erlonandreykurilin: do you have any good example?16:22
bauzascdent: https://review.openstack.org/#/c/424110/ tries to avoid hitting -1 from functional tests because https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api_models.py#L294-L29516:22
erlonandreykurilin: this removal breaks Cinder migration and some other features16:23
BobBalljroll: I thought it had to be on the compute since the scheduler could be managing different computes with different ratios?16:23
*** moshele has quit IRC16:23
BobBalljroll: "This configuration specifies ratio for RamFilter which can be set16:23
BobBallper compute node"16:23
bauzascdent: when you run 2 'compute' services in a functional test, it then runs the FakeDriver but with the same hypervisor_hostname so we're trampled by the placement engine giving us a 40916:23
bauzaswhen creating the RP for the 2nd compute16:23
BobBall(sorry - thought I could delete the linebreak after paste)16:24
jrollBobBall: not sure, I'm not a nova+virt expert :)16:24
andreykurilinerlon: yes. service_catalog was a custom implementation for thing that should be done by keystoneclient. service_catalog is now available in keystone session object16:24
cdentbauzas: so the problem is not so much that the constraint exists, but that existing fixtures weren't prepared for it?16:24
bauzascdent: zactly16:24
erlonandreykurilin: hmm, ok, thanks16:24
bauzascdent: I'm not saying it's due to the UC, just that the problem is popping up now because all our computes have 'fake-mini'16:24
BobBalljroll: OK - my reading is that I think that setting the default per-compute will work as the scheduler reads from the host_state rather than using a default value from the config file.16:25
bauzascdent: I fixed that in https://review.openstack.org/#/c/424110/ but that blows up, probably because it makes a shit ton of assumptions16:25
jrollBobBall: ah, then yeah, conf.set_default in the driver's initialization would probably work out16:25
jrollI guess it's easily tested16:25
BobBallI'm already hacking it up to test :)16:25
cdentbauzas: your use of the phrase "very important" made it sound like it would be hard to fix and might impact actually using the system, but it sounds more like it is something we need to address (and may be hard to address) but once addressed, not a problem?16:25
bauzascdent: it's a show-stopper for merging the scheduler upcall16:26
bauzastbc16:26
edleafebauzas: thinking about real-world deployments: are we requiring operators to give every compute node a unique name?16:26
bauzasedleafe: well, I think we made this hypothesis a while ago because you can get_by_name a RP16:27
bauzasnow, it's part of the world16:27
bauzasand we can't easily go against that way16:27
*** markus_z has joined #openstack-nova16:27
edleafebauzas: ok, so how likely is it that a deployment with thousands of compute nodes has named them all uniquely?16:28
*** jdillaman has quit IRC16:28
bauzasedleafe: well, that's certainly a good point, but you're advocating of removing that UC, then ?16:28
bauzassounds a big deal to me16:28
edleafebauzas: not advocating - more like thinking aloud16:29
edleafebauzas: as someone who hasn't ever managed a large deployment16:29
bauzasedleafe: AFAIR, libvirt is taking the hostname as the hypervisor_hostname16:29
bauzasedleafe: and that goes the same for other drivers16:29
edleafebauzas: causing tests to fail is bad. Causing deployments to fail is... worse16:30
bauzasedleafe: given it's based on the service hostname, that looks not difficult to me to have different hostnames :)16:30
*** nkrinner is now known as nkrinner_afk16:30
sfinucanDinaBelova: Quick question - if I run something like 'tox -e py27 nova.tests.unit.scheduler.filters.test_numa_topology_filters', is it expected that the os_profiler test be run?16:31
openstackgerritEric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv)  https://review.openstack.org/40940116:32
edleafebauzas: ok, that's reassuring16:32
*** david-lyle has joined #openstack-nova16:32
cdentbauzas: so sounds like the first thing to do is fix the fixtures and the functional tests? Do you need/want help with that?16:33
bauzascdent: http://logs.openstack.org/10/424110/1/check/gate-nova-tox-db-functional-ubuntu-xenial/e2972f1/testr_results.html.gz16:33
mriedemso rp.name is optional16:33
bauzascdent: giving you a sense of what I call "a shit ton of assumptions"16:33
mriedemis it the resource tracker that's setting it to hypervisor_hostname?16:33
bauzasmriedem: it is indeed16:34
mriedemso how about we name it hypervisor_hostname + nodename?16:34
mriedemsurely those would be unique in the real world16:34
mriedemfor our fixtures and tests, we probably have those copied all over the place, but we could just fix that16:34
mriedemwith uuids or something16:34
bauzasmriedem: hypervisor_hostname == nodename but I get your idea16:34
bauzasmriedem: you mean service hostname + hypervisor_hostname16:35
bauzaslemme try this16:35
openstackgerritDan Smith proposed openstack/nova: Multicell support for instance listing  https://review.openstack.org/39677516:35
openstackgerritDan Smith proposed openstack/nova: Make api_samples tests use simple cell environment  https://review.openstack.org/41047316:35
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937916:35
dansmithsfinucan: thanks for catching that, fixed here ^16:35
BobBalljroll: What do you know - it looks like it works!  Many thanks16:36
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850916:36
openstackgerritClaudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps  https://review.openstack.org/42096016:36
*** nklenke has joined #openstack-nova16:36
cdentmriedem, bauzas: wait, are we saying that hypervisor hostname is not unique? is it not an FQDN?16:36
bauzascdent: not with the FakeDriver at least16:36
cdentbauzas: the fake driver is _fake_16:37
cdentand can be changed16:37
cdentso isn't really a true issue16:37
bauzascdent: sure, what I did16:37
cdentI'm trying to figure out if there is a real issue16:37
bauzascdent: I changed that to being defaulted to the service hostname16:37
bauzascdent: but kaboom16:37
cdenthypervisor hostname is unique, job done16:37
bauzasbecause in lots of places, we litterally say hypervisor_hostname == 'fake-mini'16:37
*** dtp has joined #openstack-nova16:37
cdentthat's because in a test context service hostname is not unique, right?16:37
*** efoley__ has joined #openstack-nova16:38
jrollBobBall: \o/16:38
cdentso we just fix that, right?16:38
*** bvanhav has quit IRC16:38
bauzascdent: that's because we stupidely get the value from the first CONF.host value which is set to 'fake-mini' (because of the ConfFixture)16:38
bauzasso we wrote around 500 tests saying "cool, let's just assume it's fake-mini whatever the compute node name is"16:39
*** sacharya has joined #openstack-nova16:39
bauzashonestly, mriedem's proposal seems the less boring approach16:39
cdentthat means changing the names of existing resource providers out there in a deployed world?16:40
mriedembauzas: i also agree with cdent,16:40
mriedemin that if the hostname is just fqdn, and we have a bunch of dumb tests, we just fix the tests that need fixing,16:40
mriedemi doubt it's 500 that we need to fix 'right now'16:41
*** efoley_ has quit IRC16:41
bauzasmriedem: sure, we can do that, I leave others fix that then16:41
bauzashttps://review.openstack.org/#/c/424110/1 is just red tomato16:41
*** hshiina has joined #openstack-nova16:42
cdentbauzas: If it helps, I can do that.16:42
cdentAs it has to be done, right?16:42
*** hoonetorg has joined #openstack-nova16:42
*** ratailor has quit IRC16:43
bauzascdent: if we agree on that being a necessary prereq for merging the scheduler change, I'm fine16:43
openstackgerritBob Ball proposed openstack/nova: XenAPI: fix memory usage miscalculation  https://review.openstack.org/39485716:43
bauzascdent: yup, lemme show you why it's important16:43
cdentbauzas, fine with me doing it, or fine you'll take care of it?16:43
cdentI know why it is important16:43
bauzascdent: http://paste.openstack.org/show/595821/ L6216:44
*** hoonetorg has quit IRC16:45
mriedembauzas: so let's just focus on fixing the tests that need fixing right now to move you along16:45
bauzasokay16:45
*** mingyu has joined #openstack-nova16:46
cdentbauzas: If there's some task or chunk of a task that you can split off to me, let me know.16:47
bauzasmriedem: cdent: so we all agree with the fact that the FakeDriver should stop reporting the same hypervisor hostname?16:47
mriedemsure16:47
mriedemi mean yeah probably16:47
mriedemit depends on context16:47
cdentAt least in the context of the scheduler related tests16:47
*** hoonetorg has joined #openstack-nova16:47
mriedemjinx16:47
cdentbuy me a coke16:47
bauzasmriedem: cdent: we could compare with other drivers16:48
cdentPerhaps a UniqueHostnameFakeDriver subclass could help?16:48
mriedemyeah that's not a bad idea16:48
bauzascdent: interesting idea16:48
bauzasthat would prevent us to fix the world16:48
mriedemthen just override the config with that in setUp for the tests that care16:48
bauzasmriedem: cdent: I like that idea, I can spin a patch easily then16:49
mriedemwell, you could TODO it in ocata,16:49
mriedemTODO(bauzas): this is temporary until we can fix all of the tests16:49
bauzasmriedem: agreed16:49
cdent\o/16:49
bauzasmriedem: I also need to verify the other drives16:49
cdentwe have just experienced successful collaboration16:49
gabor_antalI'd like to notice, that we also have this hostname problem when we want to test live migration notification (e.g. https://review.openstack.org/#/c/402124/)16:50
cdentachievement unlocked!16:50
bauzasI'm pretty sure for libvirt that the nodename is equal to the service name16:50
bauzasfor ironic, it's easy : it's the ironic node uuid16:50
bauzasbut I leave the question open for other drivers16:50
*** mingyu has quit IRC16:52
openstackgerritBob Ball proposed openstack/nova: XenAPI: fix memory usage miscalculation  https://review.openstack.org/39485716:53
*** david-lyle_ has joined #openstack-nova16:57
*** a-pugachev has joined #openstack-nova16:58
andreykurilinmriedem: it looks like "token" auth is compitely broken in novaclient. It doesn't work for CLI where keystoneauth creates session object itself(I do not know how many releases it was broken) and in python API (my patch broke it). I'll try to fix is asap and we need to cut new release16:58
*** efoley_ has joined #openstack-nova16:58
*** _ducttape_ has quit IRC16:58
*** ducttape_ has joined #openstack-nova17:01
*** baoli has quit IRC17:01
*** andreas_s has quit IRC17:01
*** efoley__ has quit IRC17:02
*** david-lyle_ has quit IRC17:03
mriedemandreykurilin: https://review.openstack.org/#/c/419441/ right?17:03
openstackgerritSTEW TY proposed openstack/nova: Transform instance.unrescue notifications  https://review.openstack.org/38827517:03
andreykurilinmriedem: 1) it contains a bug. It fixes only python api17:04
andreykurilinI'll re-submit it to pass gates17:04
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication  https://review.openstack.org/41944117:06
openstackgerritDan Smith proposed openstack/nova: Only warn about hostmappings during ocata upgrade  https://review.openstack.org/42424317:07
*** jose-phillips has joined #openstack-nova17:07
dimsmriedem : notice the gate-tempest-dsvm-py35-ubuntu-xenial-nv in check queue? (example https://review.openstack.org/#/c/323250/)17:08
*** karimb has quit IRC17:09
mdboothbauzas: When doing a migration with an explicit destination, would it make sense to do make a scheduler call anyway restricted to just that destination to see if it succeeds?17:09
*** bvanhav has joined #openstack-nova17:09
mdboothFor all I know we already do this somewhere, just thinking...17:10
bauzasmdbooth: for a live migration, we do that already17:10
bauzasmdbooth: for cold ones, there is a BP from takashi17:10
mdboothI'm looking here: https://review.openstack.org/#/c/244489/58/nova/conductor/tasks/live_migrate.py17:10
*** mlavalle has joined #openstack-nova17:10
mdboothexplicit destination doesn't call _find_destination() there17:10
mdboothBut perhaps it's called elsewhere.17:11
mdboothWell, not _find_destination, but some other scheduler call.17:11
mriedemdims: notice what?17:11
mriedemdims: if it's non-voting i'm not noticing it right now17:12
*** markus_z has quit IRC17:14
bauzasmriedem: https://github.com/openstack/nova/blob/c9eb9530314d047f5013941ebcfd5ef0192a9dc3/nova/compute/api.py#L377317:14
*** tesseract has quit IRC17:15
dimsmriedem : ack. it's green that's all :)17:16
mriedemdims: oh you were looking for some back patting, i see17:16
mriedem#success dims got gate-tempest-dsvm-py35-ubuntu-xenial-nv green17:16
openstackstatusmriedem: Added success to Success page17:16
dimsLOL, just wanted to be sure i won't get yelled at if you notice it later in the week (when it goes RED) :)17:17
mriedemi won't notice17:18
*** a-pugachev has quit IRC17:18
mriedemdims: i'm on vacation so only water temperature and some other key things are high priority for me right now17:18
dimsfair enough17:18
*** baoli has joined #openstack-nova17:18
*** nicolasbock has quit IRC17:19
mriedemi don't have to officially care about py35 until march17:19
mriedem:)17:19
dimsack mriedem : have fun!17:19
*** nic has joined #openstack-nova17:19
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937917:26
*** mvk has joined #openstack-nova17:26
*** stevelle has joined #openstack-nova17:27
*** nicolasbock has joined #openstack-nova17:27
jaosoriorgibi: you around?17:27
openstackgerritStephen Finucane proposed openstack/nova: scheduler: Don't modify RequestSpec.numa_topology  https://review.openstack.org/41947917:28
*** mat128 is now known as mat128|afk17:28
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: [CLI] Fix token auth type  https://review.openstack.org/42425617:30
*** sudipto has quit IRC17:30
*** sudipto_ has quit IRC17:30
*** sudipto_ has joined #openstack-nova17:31
*** sudipto has joined #openstack-nova17:31
*** sudipto has quit IRC17:31
*** sudipto_ has quit IRC17:31
andreykurilinmriedem: have a nice vacation! I'll try to ping some other cores to merge fixes when it will be ready and propose a release17:31
*** derekh has quit IRC17:33
mriedemack17:33
mriedemandreykurilin: thanks for being on top of it17:33
sfinucandansmith: Added tests to https://review.openstack.org/419479. Looking at the cells limit patch again now17:33
*** cjvolzka has quit IRC17:34
*** Swami has joined #openstack-nova17:37
*** sacharya_ has joined #openstack-nova17:37
*** tbachman_ is now known as tbachman17:37
*** sacharya has quit IRC17:38
*** nicolasbock has quit IRC17:40
*** Apoorva has joined #openstack-nova17:40
*** cjvolzka has joined #openstack-nova17:43
openstackgerritStephen Finucane proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell  https://review.openstack.org/40087417:45
mdboothCan anybody tell me what the 'limits' in the host object returned by scheduler_client.select_destinations() means?17:45
*** marst has joined #openstack-nova17:45
mdboothbauzas: ^^^ ?17:45
bauzasmdbooth: it's what the claim uses for verifying the limit17:45
mdboothWhat's its scope?17:46
bauzasmdbooth: for example, see the CoreFilter, it's telling what the max amount of the resouce is given the allocation ratio17:46
mdboothi.e. can it race with the clam?17:46
mdboothOk, so it's essentially a constant? It doesn't vary as resources are used.17:47
mdboothI'm looking at a patch where conductor fetches this limits value and passes it to compute, which then uses it to create a claim17:48
*** cjvolzka has quit IRC17:48
mdboothIIUC, what you describe means there's no race there, as the value can't change in the presence of other placements.17:49
mdboothI assumed that was the case because in claim: free = limit - used17:50
mdboothwhere used is dynamic17:50
dansmithsfinucan: re-reviewed, thanks17:50
andreykurilindims: I do not what had happened, but rally-nova-py3 jobs is ok now:)17:52
mlavallemriedem: is it still the plan in Ocata to have the scheduler use the data in resource providers and inventories for instances placement in host?17:53
andreykurilin*I do not know17:53
edleafemlavalle: yes. The scheduler will get a pared-down list of hosts from placement that match the basic resources in the request17:54
edleafemlavalle: it will then use its other filters to select, as it currently does17:54
*** browne has joined #openstack-nova17:54
*** nicolasbock has joined #openstack-nova17:55
mriedemmlavalle: yes17:55
openstackgerritJohn Garbutt proposed openstack/nova: WIP Support cinder v3 API  https://review.openstack.org/42020117:56
*** david-lyle_ has joined #openstack-nova17:56
*** david-lyle_ has quit IRC17:56
*** Jeffrey4l__ has joined #openstack-nova17:57
*** david-lyle_ has joined #openstack-nova17:57
mlavalleedleafe, mriedem: so, given that the patches that I wrote for resource providers and routed networks are merging, i can remove the first bullet here, right: http://docs.openstack.org/newton/networking-guide/config-routed-networks.html#limitations?17:58
*** eharney has quit IRC17:58
edleafemlavalle: no17:58
*** Jeffrey4l_ has quit IRC17:59
edleafemlavalle: I guess I should have specified what I meant by "basic" :)17:59
edleafemlavalle: just disk, RAM, and VCPU for Ocata17:59
mriedemmlavalle: what ed said. this is also going to be a pre-req for scheduling with routed networks https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/prep-for-network-aware-scheduling-ocata18:00
mriedemmlavalle: and it's looking like at this point that series won't be happening in ocata18:00
*** stevelle has left #openstack-nova18:01
*** ralonsoh has quit IRC18:01
mlavalleedleafe, mriedem: so IPV4_ADDRESS inventories are postponed to Pike?18:01
mlavalleas far as being used by the scheduler18:02
edleafemlavalle: They can be reported and tracked in Ocata. The scheduler won't be able to do anything with them, though18:02
mlavalleedleafe, mriedem: thanks for the update!18:02
edleafemlavalle: IOW, the placement service will be able to handle it. The scheduler won't be using that information, yet'18:03
mriedemmlavalle: yeah you definitely want them reporting in as soon as possible18:03
mriedemso that when the scheduler can handle it, the data is already there18:03
dimsandreykurilin : we must have fixed some stuff in nova while we got the dsvm+tempest working :)18:04
*** kfarr has joined #openstack-nova18:04
dimsandreykurilin : thanks for the good news :)18:04
mlavallemriedem, edleafe: ok, on our side the functionality is complete. the last patchset remaining has already 2 +2's, so it should merge today. I even update the documentation18:04
andreykurilindims: there is only one failed scenario. will look at it more close a bit later18:05
*** kbaegis has joined #openstack-nova18:05
dimsack andreykurilin : thanks18:05
mlavallemriedem, edleafe: I will focus next on creating a tempest test to exercise it in the gate, so we all get feedback18:05
*** david-lyle has quit IRC18:05
kbaegisHi all. I'm having some issues with devstack and nova. From horizon I'm getting a ton of 400s, but from nova/openstack cli everything works fine18:05
*** david-lyle_ is now known as david-lyle18:05
diana_clarkekbaegis: You might be hitting this. https://bugs.launchpad.net/python-novaclient/+bug/165814818:06
openstackLaunchpad bug 1654183 in python-novaclient "duplicate for #1658148 Token based authentication in Client class does not work" [High,In progress] - Assigned to Andrey Kurilin (andreykurilin)18:06
*** bvanhav has quit IRC18:07
*** cjvolzka has joined #openstack-nova18:09
*** cjvolzka has quit IRC18:09
kbaegisThanks guys18:10
*** armax has joined #openstack-nova18:10
*** owalsh is now known as owalsh-afk18:10
*** efoley__ has joined #openstack-nova18:11
*** lpetrut has quit IRC18:12
*** jbryce has joined #openstack-nova18:14
*** efoley_ has quit IRC18:14
*** eharney has joined #openstack-nova18:14
jbryceis there a practical upper limit to how many cells a nova deployment might have? this is for a nova cells related question in the user survey18:16
openstackgerritDan Smith proposed openstack/nova: Multicell support for instance listing  https://review.openstack.org/39677518:17
openstackgerritDan Smith proposed openstack/nova: Make api_samples tests use simple cell environment  https://review.openstack.org/41047318:17
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937918:17
dansmithjbryce: assume you're talking about cellsv1?18:17
*** amotoki has quit IRC18:18
*** kbaegis has left #openstack-nova18:18
jbrycedansmith: iā€™d be curious about both v1 and v2 as this round of updates will probably live into the time when we start getting v2 deployments cataloged18:19
dansmithjbryce: well, as of this moment cellsv2 can't have multiple cells18:19
dansmithjbryce: for cellsv1 you'd probably need to ask someone that actually runs it to see how many they have and/or if they've seen any such practical limits18:19
dansmithjbryce: there is no hard-coded limit or anything like that though18:20
jbrycedansmith: thanks18:21
*** owalsh-afk has quit IRC18:21
*** sambetts is now known as sambetts|afk18:21
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850918:22
openstackgerritClaudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps  https://review.openstack.org/42096018:22
*** mnestratov has quit IRC18:22
*** bvanhav has joined #openstack-nova18:25
*** cjvolzka has joined #openstack-nova18:27
*** jpena is now known as jpena|off18:31
*** dharinic is now known as dharinic|lunch18:35
*** mjura has quit IRC18:38
*** efoley__ has quit IRC18:43
*** tbachman has quit IRC18:45
*** catintheroof has joined #openstack-nova18:45
*** owalsh-afk has joined #openstack-nova18:45
*** catinthe_ has quit IRC18:47
*** beagles is now known as beagles-brb18:47
*** tbachman has joined #openstack-nova18:48
*** mingyu has joined #openstack-nova18:49
*** yamahata has quit IRC18:51
*** mingyu has quit IRC18:53
*** mriedem has quit IRC18:55
*** tbachman_ has joined #openstack-nova18:56
*** tbachman has quit IRC18:56
*** tbachman_ is now known as tbachman18:56
*** sree has joined #openstack-nova18:57
*** kukacz has quit IRC19:03
*** catinthe_ has joined #openstack-nova19:03
*** sree has quit IRC19:03
amrithhi andreykurilin ... any luck?19:05
*** catintheroof has quit IRC19:06
*** gszasz has quit IRC19:07
*** hshiina has quit IRC19:11
*** amoralej is now known as amoralej|off19:13
*** voelzmo has joined #openstack-nova19:15
*** Oku_OS is now known as Oku_OS-away19:16
*** amotoki has joined #openstack-nova19:18
*** gouthamr has quit IRC19:20
*** enki_anunnaki has joined #openstack-nova19:21
*** Sukhdev_ has joined #openstack-nova19:21
*** hshiina has joined #openstack-nova19:22
oomichislaweq_: thanks, now LGTM19:22
*** sdague has joined #openstack-nova19:22
*** knikolla has quit IRC19:23
*** beagles-brb is now known as beaglkes19:23
*** knikolla has joined #openstack-nova19:23
*** knikolla has quit IRC19:23
*** beaglkes is now known as beagles19:24
*** knikolla has joined #openstack-nova19:24
*** knikolla has quit IRC19:24
*** knikolla has joined #openstack-nova19:25
*** knikolla has quit IRC19:25
*** knikolla has joined #openstack-nova19:25
*** knikolla has quit IRC19:25
*** Guest17268 has quit IRC19:26
openstackgerritKen'ichi Ohmichi proposed openstack/nova: Deny invalid flavor extraspec for cpu_policy  https://review.openstack.org/40637719:27
oomichigmann: thanks, applied your comments19:27
*** yamahata has joined #openstack-nova19:29
openstackgerritChris Dent proposed openstack/nova: Avoid redundant call to update_resource_stats from RT  https://review.openstack.org/42430519:35
cdentbauzas, edleafe ^ is a potential minor performance fix in the resource tracker19:36
cdentnot sure how much it matters but seemed low hanging fruit in the bug list: https://bugs.launchpad.net/nova/+bug/165862919:37
openstackLaunchpad bug 1658629 in OpenStack Compute (nova) "Resource tracker causes update of compute_nodes table every minute" [Medium,In progress] - Assigned to Chris Dent (cdent)19:37
cdentand with that, good night19:37
*** cdent has quit IRC19:37
openstackgerritStephen Gordon proposed openstack/nova: libvirt: avoid generating script with empty path  https://review.openstack.org/41193619:42
*** jdurgin has joined #openstack-nova19:46
openstackgerritSTEW TY proposed openstack/nova: Transform instance.unrescue notifications  https://review.openstack.org/38827519:46
*** ociuhandu has quit IRC19:47
*** dharinic|lunch is now known as dharinic19:49
*** tbachman has quit IRC19:49
*** jaosorior has quit IRC19:50
*** voelzmo has quit IRC19:51
*** hshiina has quit IRC19:59
*** abalutoiu_ has quit IRC19:59
*** kaisers has joined #openstack-nova20:00
*** sdague has quit IRC20:00
*** raj_singh_ has joined #openstack-nova20:02
*** nic has quit IRC20:05
slaweqoomichi: thx for +2 :)20:06
*** jose-phillips has quit IRC20:10
*** satyar has quit IRC20:13
*** slaweq__ has joined #openstack-nova20:13
openstackgerritSTEW TY proposed openstack/nova: Transform instance.rescue notifications  https://review.openstack.org/38564420:15
*** mnestratov has joined #openstack-nova20:16
*** satyar has joined #openstack-nova20:19
*** tbachman has joined #openstack-nova20:20
*** liverpooler has quit IRC20:22
*** rfolco has quit IRC20:27
*** tbachman has quit IRC20:29
*** _ducttape_ has joined #openstack-nova20:30
*** pradk- has joined #openstack-nova20:31
*** pradk has quit IRC20:31
*** bvanhav has quit IRC20:32
*** stvnoyes has quit IRC20:33
*** ducttape_ has quit IRC20:33
*** stvnoyes has joined #openstack-nova20:33
*** raj_singh_ has quit IRC20:35
openstackgerritIldiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow  https://review.openstack.org/33028520:41
openstackgerritIldiko Vancsa proposed openstack/nova: Remove check_attach  https://review.openstack.org/33535820:41
openstackgerritIldiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow  https://review.openstack.org/33028520:42
openstackgerritIldiko Vancsa proposed openstack/nova: Remove check_attach  https://review.openstack.org/33535820:42
*** moshele has joined #openstack-nova20:43
*** abalutoiu_ has joined #openstack-nova20:48
*** nklenke has quit IRC20:50
*** mingyu has joined #openstack-nova20:50
amrithandreykurilin, please let me know if I can help.20:52
*** mingyu has quit IRC20:54
*** slaweq__ has quit IRC20:56
*** kaisers has quit IRC20:57
*** pradk- is now known as pradk20:58
*** hoangcx has joined #openstack-nova21:01
*** ociuhandu has joined #openstack-nova21:02
*** oanson has quit IRC21:10
*** dimtruck is now known as zz_dimtruck21:11
*** jerrygb has quit IRC21:12
*** satyar has quit IRC21:15
andreykurilinamrith: I need just several minutes for latest version of patches. they should help21:17
openstackgerritAndrey Kurilin proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication  https://review.openstack.org/41944121:18
*** _ducttape_ has quit IRC21:19
*** ducttape_ has joined #openstack-nova21:20
*** ducttape_ has joined #openstack-nova21:20
*** tbachman has joined #openstack-nova21:22
*** ijw has joined #openstack-nova21:23
moshelejaypipes: hi, can you review this simple fix https://review.openstack.org/#/c/423608/21:26
*** kaisers has joined #openstack-nova21:26
*** jose-phillips has joined #openstack-nova21:27
*** oanson has joined #openstack-nova21:30
*** smatzek_ has quit IRC21:30
*** jerrygb has joined #openstack-nova21:31
*** zz_dimtruck is now known as dimtruck21:32
*** jerrygb_ has joined #openstack-nova21:32
*** toure is now known as toure|biab21:32
*** Jeffrey4l_ has joined #openstack-nova21:34
*** thorst_ has quit IRC21:35
*** Jeffrey4l__ has quit IRC21:35
*** thorst_ has joined #openstack-nova21:35
*** jerrygb has quit IRC21:36
*** cleong has quit IRC21:39
*** jose-phillips has quit IRC21:39
*** slaweq has quit IRC21:39
*** slaweq has joined #openstack-nova21:40
*** thorst_ has quit IRC21:40
*** jose-phillips has joined #openstack-nova21:42
*** markvoelker has quit IRC21:47
*** moshele has quit IRC21:47
*** anush has quit IRC21:48
*** kfarr has quit IRC21:48
*** mnestratov has quit IRC21:49
openstackgerritEd Leafe proposed openstack/nova: placement: adds REST API for nested providers  https://review.openstack.org/38480721:51
openstackgerritEd Leafe proposed openstack/nova: placement: allow filter providers in tree  https://review.openstack.org/37721521:51
openstackgerritEd Leafe proposed openstack/nova: pci: add uuid field to PciDevice object  https://review.openstack.org/41476821:51
openstackgerritEd Leafe proposed openstack/nova: placement: integrate ProviderTree to report client  https://review.openstack.org/41592121:51
openstackgerritEd Leafe proposed openstack/nova: placement: SRIOV PF devices as child providers  https://review.openstack.org/41476921:51
openstackgerritEd Leafe proposed openstack/nova: placement: adds ProviderTree for nested resources  https://review.openstack.org/41592021:51
openstackgerritEd Leafe proposed openstack/nova: placement: update client to set parent provider  https://review.openstack.org/38569321:51
openstackgerritEd Leafe proposed openstack/nova: placement: add nested resource providers  https://review.openstack.org/37713821:51
*** mingyu has joined #openstack-nova21:51
*** slaweq has quit IRC21:52
*** slaweq has joined #openstack-nova21:54
*** mingyu has quit IRC21:56
*** ducttape_ has quit IRC21:56
*** oanson has quit IRC21:56
*** oanson has joined #openstack-nova21:57
*** smatzek_ has joined #openstack-nova21:58
*** thorst_ has joined #openstack-nova21:58
openstackgerritSTEW TY proposed openstack/nova: Transform instance.rescue notifications  https://review.openstack.org/38564421:59
*** kaisers has quit IRC22:02
*** thorst_ has quit IRC22:03
*** harlowja has quit IRC22:03
*** eharney has quit IRC22:03
*** baoli has quit IRC22:04
*** kukacz has joined #openstack-nova22:05
*** jdurgin has quit IRC22:05
*** hoangcx has quit IRC22:05
*** nic has joined #openstack-nova22:06
*** burgerk has quit IRC22:10
amriththx andreykurilin22:12
*** jheroux has quit IRC22:12
*** browne has quit IRC22:12
*** thorst_ has joined #openstack-nova22:13
amrithandreykurilin, this one (412497) isn't looking promising22:13
*** a-pugachev has joined #openstack-nova22:18
*** thorst_ has quit IRC22:22
*** jdurgin has joined #openstack-nova22:23
*** oanson has quit IRC22:24
*** oanson has joined #openstack-nova22:25
*** slaweq has quit IRC22:26
*** slaweq has joined #openstack-nova22:27
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514222:30
*** kaisers has joined #openstack-nova22:32
*** esberglu has quit IRC22:33
*** esberglu has joined #openstack-nova22:33
*** esberglu_ has joined #openstack-nova22:35
openstackgerritIldiko Vancsa proposed openstack/nova: Implement new attach/detach Cinder flow  https://review.openstack.org/33028522:37
*** esberglu has quit IRC22:37
*** esberglu_ has quit IRC22:39
*** smatzek_ has quit IRC22:41
openstackgerritEd Leafe proposed openstack/nova: placement: minor refactor _allocate_for_instance()  https://review.openstack.org/40570522:43
openstackgerritEd Leafe proposed openstack/nova: placement: RT now adds proper Ironic inventory  https://review.openstack.org/40447222:43
openstackgerritEd Leafe proposed openstack/nova: placement: report client handle InventoryInUse  https://review.openstack.org/40499022:43
*** owalsh-afk has quit IRC22:46
*** owalsh-afk has joined #openstack-nova22:46
*** browne has joined #openstack-nova22:48
*** mtanino has quit IRC22:49
*** marst has quit IRC22:51
*** tblakes has quit IRC22:51
*** mingyu has joined #openstack-nova22:52
*** Swami_ has joined #openstack-nova22:56
*** mingyu has quit IRC22:56
*** kylek3h has quit IRC22:58
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API  https://review.openstack.org/41796122:58
*** jamesdenton has quit IRC23:00
*** Swami has quit IRC23:00
*** Swami_ has quit IRC23:01
*** cjvolzka has quit IRC23:04
*** mdrabe has quit IRC23:05
*** esberglu has joined #openstack-nova23:12
*** esberglu has quit IRC23:12
andreykurilinamrith: based on novaclient's functional tests, it should work. I do not sure that depends on tag actually works for novaclient's library. Can you recheck it locally?23:18
*** alexpilotti has joined #openstack-nova23:19
*** catinthe_ has quit IRC23:23
*** xyang1 has quit IRC23:25
andreykurilinamrith: I try to play more with trove code tomorrow, but everything should be ok now23:30
*** edleafe- has joined #openstack-nova23:33
*** harlowja has joined #openstack-nova23:34
*** harlowja has quit IRC23:35
*** jose-phillips has quit IRC23:35
*** edleafe has quit IRC23:36
*** jose-phillips has joined #openstack-nova23:37
*** tbachman has quit IRC23:38
*** dave-mccowan has quit IRC23:38
openstackgerritAnusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload  https://review.openstack.org/41973023:39
*** mtanino has joined #openstack-nova23:40
*** mriedem has joined #openstack-nova23:40
*** Swami has joined #openstack-nova23:41
*** takashin has joined #openstack-nova23:42
mriedemdansmith: looks like grenade isn't happy with https://review.openstack.org/#/c/424252/23:42
dansmithmriedem: isn't that the dep thing?23:43
* dansmith checks23:43
dansmithmriedem: yeah, that's the dep thing.. I've been waiting all day to get a run of my patch with this23:44
dansmithit just merged and I rechecked mine -- just rechecked this one too23:44
*** dimtruck is now known as zz_dimtruck23:44
mriedemdansmith: i'm not sure what dep thing you're talking about23:45
mriedemyou changed the d-g patch to not call discover_hosts for grenade23:45
dansmithmriedem: https://review.openstack.org/#/c/424323/23:46
*** harlowja has joined #openstack-nova23:48
openstackgerritOpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements  https://review.openstack.org/39534723:50
mriedemdansmith: ok i wasn't aware of that23:51
*** tbachman has joined #openstack-nova23:53
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API  https://review.openstack.org/41796123:56
bauzasYippee ki-yay mother ^23:57
bauzashoping Jenkins to be green after that one23:57
bauzasnow, calling it a day23:57
*** edmondsw has quit IRC23:57
*** edmondsw has joined #openstack-nova23:59

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