Tuesday, 2017-07-18

*** itlinux has joined #openstack-nova00:01
*** brault has joined #openstack-nova00:02
*** brault has quit IRC00:07
*** mriedem has quit IRC00:11
*** gmann has quit IRC00:13
*** gmann has joined #openstack-nova00:13
openstackgerritmelanie witt proposed openstack/nova master: Request zero root disk for boot-from-volume instances  https://review.openstack.org/42848100:13
openstackgerritmelanie witt proposed openstack/nova master: Claim and report zero root disk for boot-from-volume instances  https://review.openstack.org/42850500:13
*** Apoorva_ has quit IRC00:13
*** jwcroppe has joined #openstack-nova00:14
*** yingjun has joined #openstack-nova00:19
*** gmann_ has joined #openstack-nova00:22
*** mriedem has joined #openstack-nova00:24
*** boris-42__ has quit IRC00:27
*** acormier has quit IRC00:27
*** acormier has joined #openstack-nova00:28
*** Apoorva has joined #openstack-nova00:28
*** Apoorva has quit IRC00:30
mriedemalex_xu: good morning. the server tags during create API change is ready now I think https://review.openstack.org/#/c/469800/ - the tags filters are supported in the build requests now too00:30
*** Apoorva has joined #openstack-nova00:30
mriedemKevin_Zheng: ^ you should probably get started on the novaclient support for that microversion too00:31
*** Apoorva has quit IRC00:31
*** jaypipes has quit IRC00:31
*** Apoorva has joined #openstack-nova00:31
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Amend the "list-show-all-server-migration-types" spec  https://review.openstack.org/46084700:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Use URIOpt  https://review.openstack.org/47921500:38
openstackgerritTakashi NATSUME proposed openstack/nova master: api-ref: Verify parameters in os-migrations.inc  https://review.openstack.org/47487900:38
openstackgerritTakashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_remote_consoles.py  https://review.openstack.org/47842300:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Stop using mox stubs in test_console_auth_tokens.py  https://review.openstack.org/47737400:39
openstackgerritTakashi NATSUME proposed openstack/nova master: Stop using mox stubs in cast_as_call.py  https://review.openstack.org/47773000:41
*** dave-mccowan has quit IRC00:42
*** mdnadeem has joined #openstack-nova00:42
*** vishwanathj has joined #openstack-nova00:43
*** lucasxu has joined #openstack-nova00:45
*** lucasxu has quit IRC00:45
*** dave-mccowan has joined #openstack-nova00:46
*** thorst has joined #openstack-nova00:46
*** Apoorva has quit IRC00:50
*** Apoorva has joined #openstack-nova00:51
*** thorst has quit IRC00:51
*** litao__ has joined #openstack-nova00:52
*** gcb has quit IRC00:53
*** Apoorva has quit IRC00:55
*** phuongnh has joined #openstack-nova00:58
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient master: Replace uuid.uuid4().hex with uuidutils.generate_uuid()  https://review.openstack.org/48424401:02
*** rajathagasthya has quit IRC01:03
*** esberglu has joined #openstack-nova01:06
*** kristian__ has joined #openstack-nova01:07
*** gjayavelu has quit IRC01:10
*** esberglu has quit IRC01:10
*** kristian__ has quit IRC01:11
*** trungnv has quit IRC01:12
*** sree_ has joined #openstack-nova01:12
*** sree_ is now known as Guest3302601:12
Kevin_Zhengmriedem: Thank you very very much on this01:13
*** acormier has quit IRC01:14
*** acormier has joined #openstack-nova01:15
*** crushil has quit IRC01:15
*** crushil has joined #openstack-nova01:15
*** Guest33026 has quit IRC01:17
mriedemsure yw, thanks for being prompt about responding to review comments01:17
*** kiennt_ has joined #openstack-nova01:19
*** acormier has quit IRC01:19
Kevin_Zhengbtw, could you tell me why did the test I wrote fail? it doesn't seems alot of difference to your patch?01:20
*** ckopper has joined #openstack-nova01:20
Kevin_Zhengoh, I got it01:20
*** gmann_ has quit IRC01:21
*** gmann_ has joined #openstack-nova01:22
*** armax has joined #openstack-nova01:23
alex_xumriedem: got it, I will go through the patch today01:25
sbezverkany knows an equivalent to nova service-list in openstack cli ?01:25
openstackgerritMerged openstack/nova master: fix test_volume_swap_server instability  https://review.openstack.org/48428801:26
*** mdnadeem has quit IRC01:28
*** yufei has joined #openstack-nova01:30
*** ckopper has quit IRC01:31
*** ckopper has joined #openstack-nova01:32
*** yufei has quit IRC01:32
*** Daisy_ has joined #openstack-nova01:33
*** harlowja has quit IRC01:34
*** yufei has joined #openstack-nova01:36
mriedemKevin_Zheng: to pay me back, you can take a look at this and see if there is something obvious that I'm doing wrong to cause this http://logs.openstack.org/70/484170/4/check/gate-nova-python27-ubuntu-xenial/3bafcd1/testr_results.html.gz01:38
mriedemit's totally random01:38
*** tylerderosagrund has joined #openstack-nova01:42
*** ckopper has quit IRC01:43
*** mdnadeem has joined #openstack-nova01:43
*** ckopper has joined #openstack-nova01:43
*** Daisy_ has quit IRC01:44
*** Daisy_ has joined #openstack-nova01:44
*** tylerderosagrund has quit IRC01:47
*** baoli has joined #openstack-nova01:48
*** Daisy_ has quit IRC01:49
*** acormier has joined #openstack-nova01:50
*** acormier has quit IRC01:51
*** acormier has joined #openstack-nova01:51
*** bkopilov has quit IRC01:52
*** esberglu has joined #openstack-nova01:57
*** esberglu has quit IRC01:57
mriedemmelwitt: replied to your replies in https://review.openstack.org/#/c/416521/02:02
mriedemmelwitt: you saw my -1 over the thing in conductor right?02:02
*** dave-mccowan has quit IRC02:02
*** vladikr has quit IRC02:03
*** zhurong has joined #openstack-nova02:05
*** litao___ has joined #openstack-nova02:06
*** nikhil_ has joined #openstack-nova02:06
*** samueldmq_ has joined #openstack-nova02:06
*** ameade_ has joined #openstack-nova02:06
*** carl_baldwin_ has joined #openstack-nova02:06
*** patrickeast_ has joined #openstack-nova02:06
*** nikhil_ is now known as Guest7866902:06
*** vladikr has joined #openstack-nova02:07
*** ildikov_ has joined #openstack-nova02:07
*** scottda_ has joined #openstack-nova02:07
*** kiennt_ has quit IRC02:10
*** kiennt_ has joined #openstack-nova02:10
*** Daisy_ has joined #openstack-nova02:11
*** sridharg has joined #openstack-nova02:12
*** _d34dh0r53_ has joined #openstack-nova02:12
*** hughsaunders_ has joined #openstack-nova02:13
*** toabctl_ has joined #openstack-nova02:13
*** litao__ has quit IRC02:13
*** scottda has quit IRC02:13
*** patrickeast has quit IRC02:13
*** ildikov has quit IRC02:13
*** toabctl has quit IRC02:13
*** larainema has quit IRC02:13
*** d34dh0r53 has quit IRC02:13
*** _pewp_ has quit IRC02:13
*** ameade has quit IRC02:13
*** samueldmq has quit IRC02:13
*** carl_baldwin has quit IRC02:13
*** hughsaunders has quit IRC02:13
*** Kimmo_ has quit IRC02:13
*** antonym has quit IRC02:13
*** nikhil has quit IRC02:13
*** hughsaunders_ is now known as hughsaunders02:13
*** litao___ is now known as litao__02:13
*** ildikov_ is now known as ildikov02:13
*** scottda_ is now known as scottda02:13
*** patrickeast_ is now known as patrickeast02:14
*** toabctl_ is now known as toabctl02:14
*** samueldmq_ is now known as samueldmq02:14
*** ameade_ is now known as ameade02:14
*** carl_baldwin_ is now known as carl_baldwin02:14
*** Guest78669 is now known as nikhil02:14
*** _pewp_ has joined #openstack-nova02:14
*** rhagarty has quit IRC02:16
*** jbernard has quit IRC02:17
*** jbernard has joined #openstack-nova02:18
*** antonym has joined #openstack-nova02:19
*** gcb has joined #openstack-nova02:19
*** rhagarty has joined #openstack-nova02:19
*** antonym is now known as Guest7384302:20
*** ckopper has quit IRC02:20
*** dave-mccowan has joined #openstack-nova02:26
*** sree_ has joined #openstack-nova02:31
*** trungnv has joined #openstack-nova02:31
*** ckopper has joined #openstack-nova02:31
*** sree_ is now known as Guest4761302:32
melwittmriedem: yeah, then I forgot to reply to it. will reply02:33
*** Guest47613 has quit IRC02:35
*** shuangyangqian has joined #openstack-nova02:39
*** reedip has quit IRC02:41
*** baoli has quit IRC02:41
*** raginbajin has quit IRC02:41
*** yamamoto has joined #openstack-nova02:41
*** ckopper has quit IRC02:42
*** rodrigods has quit IRC02:43
*** dave-mccowan has quit IRC02:44
*** shuangyangqian has quit IRC02:44
*** shuangyangqian has joined #openstack-nova02:44
*** edleafe has quit IRC02:47
*** thorst has joined #openstack-nova02:47
*** esberglu has joined #openstack-nova02:51
*** crushil has quit IRC02:51
*** thorst has quit IRC02:52
*** crushil has joined #openstack-nova02:52
*** rodrigods has joined #openstack-nova02:55
*** esberglu has quit IRC02:55
*** jogo has quit IRC02:55
*** yamahata has quit IRC02:56
*** vladikr has quit IRC02:57
openstackgerritFelipe Monteiro proposed openstack/nova master: Update policy description for 'instance_actions'  https://review.openstack.org/48460902:57
openstackgerritFelipe Monteiro proposed openstack/nova master: Update policy description for 'instance_actions'  https://review.openstack.org/48460902:58
*** edleafe has joined #openstack-nova02:59
*** hongbin has joined #openstack-nova02:59
*** raginbajin has joined #openstack-nova03:00
*** jogo has joined #openstack-nova03:00
TheJuliamriedem: good question, I'll inquire tomorroe. I do seem to remember some discussion of the callback in austin, but at that time it wasn't functional.  Of course, I may be crossing threads in my brain at the same time.03:09
TheJuliaTomorrow! :)03:09
*** Daisy_ has quit IRC03:11
*** Daisy_ has joined #openstack-nova03:12
*** Daisy_ has quit IRC03:16
*** acormier_ has joined #openstack-nova03:17
openstackgerritMerged openstack/nova master: Consistent policies  https://review.openstack.org/47977403:17
*** bkopilov has joined #openstack-nova03:20
*** acormier has quit IRC03:21
openstackgerritMerged openstack/nova master: Set wsgi.keep_alive=False globally for tests  https://review.openstack.org/48397203:25
openstackgerritMerged openstack/nova master: Update policy descriptions for base.  https://review.openstack.org/47262203:26
openstackgerritMerged openstack/nova master: scheduler: isolate _get_sorted_hosts()  https://review.openstack.org/48356403:26
openstackgerritMerged openstack/nova master: Use plain routes list for os-networks instead of stevedore  https://review.openstack.org/47446303:27
*** markvoelker has quit IRC03:29
*** sdague has quit IRC03:29
*** udesale has joined #openstack-nova03:30
*** harlowja has joined #openstack-nova03:40
*** ckopper has joined #openstack-nova03:40
*** acormier_ has quit IRC03:41
*** acormier has joined #openstack-nova03:42
*** shuangyangqian has quit IRC03:42
*** ckopper has quit IRC03:43
*** ckopper has joined #openstack-nova03:44
*** acormier has quit IRC03:46
*** tuanluong has joined #openstack-nova03:46
*** mtanino_ has quit IRC03:47
openstackgerritTakashi NATSUME proposed openstack/nova master: Add 'updated_at' field to InstancePayload in notifications  https://review.openstack.org/47527603:48
*** thorst has joined #openstack-nova03:48
*** marst has joined #openstack-nova03:49
*** trinaths has joined #openstack-nova03:50
*** shuangyangqian has joined #openstack-nova03:52
*** thorst has quit IRC03:53
*** links has joined #openstack-nova03:55
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot  https://review.openstack.org/48461703:56
*** yamahata has joined #openstack-nova03:58
*** sree has joined #openstack-nova03:59
*** shuangyangqian has quit IRC04:00
*** shuangyangqian has joined #openstack-nova04:01
*** crushil has quit IRC04:02
*** ckopper has quit IRC04:02
*** yufei has quit IRC04:03
*** sree has quit IRC04:03
*** Kimmo_ has joined #openstack-nova04:03
*** larainema has joined #openstack-nova04:03
*** ckopper has joined #openstack-nova04:06
*** hongbin has quit IRC04:11
*** fragatin_ has joined #openstack-nova04:19
*** adisky__ has joined #openstack-nova04:20
*** fragatin_ has quit IRC04:21
*** fragatin_ has joined #openstack-nova04:21
*** zhenq has joined #openstack-nova04:22
*** fragatina has quit IRC04:23
*** fragatin_ has quit IRC04:24
*** s1061123 has quit IRC04:25
*** shuangyangqian has quit IRC04:25
*** shuangyangqian has joined #openstack-nova04:26
openstackgerritMike Perez proposed openstack/nova master: conf: remove *_topic config opts  https://review.openstack.org/43392404:27
*** zhenq has quit IRC04:30
*** hshiina has joined #openstack-nova04:31
*** rajathagasthya has joined #openstack-nova04:31
*** shuangyangqian has quit IRC04:34
*** shuangyangqian has joined #openstack-nova04:34
*** shuangyangqian has quit IRC04:37
*** shuangyangqian has joined #openstack-nova04:37
*** kiennt_ is now known as kiennt04:38
*** gyee has quit IRC04:44
*** nicolasbock has joined #openstack-nova04:44
*** mamandle has joined #openstack-nova04:46
*** shuangyangqian has quit IRC04:49
*** shuangyangqian has joined #openstack-nova04:49
*** shuangyangqian has quit IRC04:50
*** shuangyangqian has joined #openstack-nova04:51
*** shuangyangqian has quit IRC04:52
*** shuangyangqian has joined #openstack-nova04:53
*** harlowja has quit IRC04:54
*** shuangyangqian has quit IRC04:54
*** shuangyangqian has joined #openstack-nova04:54
*** zzzeek has quit IRC04:55
*** zhurong has quit IRC05:00
*** psachin has joined #openstack-nova05:03
*** brault has joined #openstack-nova05:04
mamandlealex_xu: Can you please take a look at https://review.openstack.org/#/c/483911/, updated changes are out for review.05:05
*** diga has joined #openstack-nova05:07
*** prateek has joined #openstack-nova05:07
*** ratailor has joined #openstack-nova05:09
*** jjung has joined #openstack-nova05:09
*** armax has quit IRC05:11
*** armax has joined #openstack-nova05:12
*** isantosp_ has joined #openstack-nova05:12
*** josecastroleon1 has joined #openstack-nova05:12
*** armax has quit IRC05:12
*** armax has joined #openstack-nova05:13
*** armax has quit IRC05:13
*** bnemec has quit IRC05:13
*** isantosp has quit IRC05:15
*** josecastroleon has quit IRC05:15
*** fragatina has joined #openstack-nova05:15
*** tylerderosagrund has joined #openstack-nova05:19
*** Daisy_ has joined #openstack-nova05:19
*** fragatina has quit IRC05:19
*** nicolasbock has quit IRC05:19
*** nicolasbock has joined #openstack-nova05:21
*** Sara_ has quit IRC05:23
*** tylerderosagrund has quit IRC05:23
*** gouthamr has quit IRC05:25
*** rcernin has joined #openstack-nova05:29
*** markvoelker has joined #openstack-nova05:30
*** rajathagasthya has quit IRC05:31
*** yufei has joined #openstack-nova05:31
*** esberglu has joined #openstack-nova05:34
*** esberglu has quit IRC05:35
*** esberglu has joined #openstack-nova05:35
*** esberglu has quit IRC05:35
*** zzzeek has joined #openstack-nova05:36
*** fragatina has joined #openstack-nova05:37
*** zzzeek has quit IRC05:37
*** zzzeek has joined #openstack-nova05:38
*** kiennt has quit IRC05:43
*** ekuris has joined #openstack-nova05:45
*** thorst has joined #openstack-nova05:49
*** zhurong has joined #openstack-nova05:50
*** thorst has quit IRC05:53
*** jjung has quit IRC05:56
*** kiennt has joined #openstack-nova06:00
*** markvoelker has quit IRC06:04
*** Oku_OS-away is now known as Oku_OS06:05
*** andreas_s has joined #openstack-nova06:07
*** takashin has left #openstack-nova06:07
*** jjung has joined #openstack-nova06:09
*** shuangyangqian_ has joined #openstack-nova06:09
*** lpetrut_ has joined #openstack-nova06:11
*** shuangyangqian has quit IRC06:12
*** pdeore has joined #openstack-nova06:14
*** shuangyangqian_ has quit IRC06:16
*** shuangyangqian_ has joined #openstack-nova06:16
openstackgerritjichenjc proposed openstack/nova master: Adjust error msg for ImageNUMATopologyAsymmetric  https://review.openstack.org/48463406:19
*** yonglihe has joined #openstack-nova06:19
openstackgerritjichenjc proposed openstack/nova master: Adjust error msg for ImageNUMATopologyAsymmetric  https://review.openstack.org/48463406:24
*** tbachman has quit IRC06:25
*** fragatina has quit IRC06:26
*** tbachman has joined #openstack-nova06:27
*** fragatina has joined #openstack-nova06:27
*** trinaths has quit IRC06:30
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot  https://review.openstack.org/48461706:38
*** sridharg has quit IRC06:41
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: DNM: Regression test for confusing error log in test  https://review.openstack.org/48463906:46
*** sridharg has joined #openstack-nova06:48
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: Fix misleading error log when test fails  https://review.openstack.org/48464106:49
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: DNM: Regression test for confusing error log in test - 2  https://review.openstack.org/48464506:53
*** trinaths has joined #openstack-nova06:55
*** itlinux has quit IRC06:57
*** damien_r has joined #openstack-nova06:58
*** sshwarts has joined #openstack-nova06:59
*** liusheng has quit IRC06:59
*** liusheng has joined #openstack-nova06:59
*** markvoelker has joined #openstack-nova07:02
*** lpetrut_ has quit IRC07:02
openstackgerritGhanshyam Mann proposed openstack/nova master: Fix indentation in policy doc  https://review.openstack.org/48464607:15
*** lpetrut_ has joined #openstack-nova07:17
*** tesseract has joined #openstack-nova07:18
*** BobBall has quit IRC07:21
*** esberglu has joined #openstack-nova07:23
*** jwcroppe has quit IRC07:26
*** esberglu has quit IRC07:27
*** lpetrut_ has quit IRC07:29
*** yufei has quit IRC07:31
*** yufei has joined #openstack-nova07:32
*** markvoelker has quit IRC07:34
*** yufei has quit IRC07:35
*** yufei has joined #openstack-nova07:35
*** kfarr has quit IRC07:38
*** ralonsoh has joined #openstack-nova07:41
*** sridharg has quit IRC07:46
*** sridharg has joined #openstack-nova07:48
*** kristian__ has joined #openstack-nova07:49
*** thorst has joined #openstack-nova07:50
*** prateek has quit IRC07:51
*** moshele has joined #openstack-nova07:52
*** thorst has quit IRC07:54
*** jangutter has quit IRC08:03
*** jangutter has joined #openstack-nova08:03
*** trinaths has quit IRC08:03
openstackgerritjichenjc proposed openstack/nova master: Check root disk size when doing resize in API  https://review.openstack.org/48466408:08
*** gcb has quit IRC08:09
*** rmart04 has joined #openstack-nova08:10
*** sree_ has joined #openstack-nova08:10
*** sree_ is now known as Guest1163408:11
*** avolkov has joined #openstack-nova08:13
*** efoley has joined #openstack-nova08:14
openstackgerritjichenjc proposed openstack/nova master: Remove CannotResizeToSameFlavor exception  https://review.openstack.org/48466608:15
*** Guest11634 has quit IRC08:15
*** lucas-afk is now known as lucasagomes08:17
*** mriedem has quit IRC08:18
*** pdeore has quit IRC08:19
*** kristia__ has joined #openstack-nova08:19
openstackgerritHironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538508:21
openstackgerritGábor Antal proposed openstack/nova master: Add json style checking for sample notifications  https://review.openstack.org/44367708:21
*** moshele has quit IRC08:21
*** moshele has joined #openstack-nova08:22
*** kristian__ has quit IRC08:22
*** moshele has quit IRC08:23
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Handle concurrent registrations of the VC extension  https://review.openstack.org/48466708:23
*** mvk has quit IRC08:27
*** markvoelker has joined #openstack-nova08:32
*** yamamoto has quit IRC08:36
*** cdent has joined #openstack-nova08:37
openstackgerritRadoslav Gerganov proposed openstack/nova master: VMware: Handle missing volume vmdk during detach  https://review.openstack.org/48467508:37
*** alexchadin has joined #openstack-nova08:42
*** zhurong has quit IRC08:44
*** moshele has joined #openstack-nova08:46
*** kristian__ has joined #openstack-nova08:47
*** kristi___ has joined #openstack-nova08:50
*** kristia__ has quit IRC08:51
*** Daisy_ has quit IRC08:51
*** kristian__ has quit IRC08:52
*** Daisy_ has joined #openstack-nova08:52
*** cdent has quit IRC08:53
*** mvk has joined #openstack-nova08:55
*** Daisy_ has quit IRC08:56
rnoriegapkoniszewski, ping08:57
rnoriegaavolkov, ping08:57
*** derekh has joined #openstack-nova08:58
avolkovrnoriega: hi09:00
rnoriegaavolkov, hi!!09:00
rnoriegaavolkov, I have a doubt about this bug: https://bugs.launchpad.net/nova/+bug/141766709:00
openstackLaunchpad bug 1417667 in OpenStack Compute (nova) "migration/evacuation/rebuild/resize of instance with NUMA topology needs to recalculate NUMA topology" [Medium,In progress] - Assigned to Andrey Volkov (avolkov)09:00
rnoriegaavolkov, what's the status?? this bug covers many many things...09:01
*** yamamoto has joined #openstack-nova09:01
rnoriegaavolkov, cold migration with cpu pining is already fixed?? and by fixed I mean that if the scheduler is involved in the migration...09:01
*** yamamoto has quit IRC09:02
openstackgerritNgo Quoc Cuong proposed openstack/nova master: Remove translation of log messages  https://review.openstack.org/46663709:03
*** yamahata has quit IRC09:03
avolkovrnoriega: yeah, it's a big thing, but unfortunately I don't work on it now, AFAIK pkoniszewski also09:04
*** yamamoto has joined #openstack-nova09:04
*** markvoelker has quit IRC09:05
rnoriegaavolkov, ok...I think you are the asignee... hahahah09:05
*** udesale__ has joined #openstack-nova09:05
avolkovrnoriega: it's due I've made some rebase09:06
*** ociuhandu has quit IRC09:08
*** udesale has quit IRC09:08
*** Daisy_ has joined #openstack-nova09:08
*** Daisy_ has quit IRC09:09
*** Daisy_ has joined #openstack-nova09:09
*** diga has quit IRC09:10
*** udesale has joined #openstack-nova09:11
openstackgerritAlex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP  https://review.openstack.org/48037909:12
*** omkar_telee has joined #openstack-nova09:12
omkar_teleeHI, can I use mitaka controller with newton nova-compute ?09:12
*** udesale__ has quit IRC09:13
omkar_teleebecause when I am trying it, its giving error IncompatibleObjectVersion: Version 2.1 of InstanceList is not supported09:13
*** esberglu has joined #openstack-nova09:13
*** esberglu has quit IRC09:13
*** esberglu has joined #openstack-nova09:14
*** esberglu has quit IRC09:18
openstackgerritMerged openstack/nova master: Remove the test for the route '/resources.:(format)'  https://review.openstack.org/47890009:18
openstackgerritMerged openstack/nova master: Use plain routes list for images instead of stevedore  https://review.openstack.org/47849809:23
openstackgerritZhenyu Zheng proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot  https://review.openstack.org/48461709:25
*** udesale__ has joined #openstack-nova09:29
*** gmann_ has quit IRC09:30
*** udesale has quit IRC09:32
*** mvk has quit IRC09:33
openstackgerritMerged openstack/nova master: Use plain routes list for image-metadata instead of stevedore  https://review.openstack.org/47849909:34
*** udesale__ has quit IRC09:36
*** udesale__ has joined #openstack-nova09:36
openstackgerritMerged openstack/nova master: doc: Populate the 'cli' section  https://review.openstack.org/47847209:37
openstackgerrityuanyue proposed openstack/nova master: Destroy the ReqSpec object of deleted instance  https://review.openstack.org/48469409:37
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates  https://review.openstack.org/48111209:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits  https://review.openstack.org/47455009:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for traits  https://review.openstack.org/47418609:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations  https://review.openstack.org/47093309:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056309:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing  https://review.openstack.org/48092409:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010509:38
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055009:38
*** udesale has joined #openstack-nova09:40
*** Daisy_ has quit IRC09:40
*** Daisy_ has joined #openstack-nova09:41
*** udesale__ has quit IRC09:42
*** jichen has joined #openstack-nova09:44
*** yingjun has quit IRC09:45
*** Daisy_ has quit IRC09:45
openstackgerrityuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling  https://review.openstack.org/48134009:45
openstackgerritjichenjc proposed openstack/nova master: Remove CannotResizeToSameFlavor exception usage  https://review.openstack.org/48466609:45
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates  https://review.openstack.org/48111209:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP traits  https://review.openstack.org/47455009:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for traits  https://review.openstack.org/47418609:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations  https://review.openstack.org/47093309:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056309:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing  https://review.openstack.org/48092409:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010509:46
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055009:46
*** sambetts|afk is now known as sambetts09:46
*** mvk has joined #openstack-nova09:47
*** Dinesh_Bhor has quit IRC09:50
*** thorst has joined #openstack-nova09:51
*** bhagyashris has quit IRC09:53
*** hshiina has quit IRC09:53
*** mdnadeem has quit IRC09:54
*** thorst has quit IRC09:55
*** yufei has quit IRC09:57
*** omkar_telee has quit IRC09:57
*** markvoelker has joined #openstack-nova10:01
*** s1061123 has joined #openstack-nova10:01
openstackgerritArtem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support  https://review.openstack.org/48439110:04
*** moshele has quit IRC10:08
*** gcb has joined #openstack-nova10:11
*** jjung has quit IRC10:11
*** vladikr has joined #openstack-nova10:12
openstackgerrityuanyue proposed openstack/nova master: Destroy the ReqSpec object of deleted instance  https://review.openstack.org/48469410:12
*** thorst has joined #openstack-nova10:12
*** ralonsoh has quit IRC10:15
*** thorst has quit IRC10:16
openstackgerrityuanyue proposed openstack/nova master: Add log info in scheduler to mark start of scheduling  https://review.openstack.org/48134010:16
*** coreywright has quit IRC10:16
*** udesale has quit IRC10:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates  https://review.openstack.org/48111210:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocations  https://review.openstack.org/47093310:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages  https://review.openstack.org/48056310:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing  https://review.openstack.org/48092410:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages  https://review.openstack.org/45010510:16
openstackgerritAndrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations  https://review.openstack.org/45055010:16
*** yamamoto has quit IRC10:18
*** Dinesh_Bhor has joined #openstack-nova10:21
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance.resize.error notifications  https://review.openstack.org/41788210:22
*** lpetrut_ has joined #openstack-nova10:23
*** sdague has joined #openstack-nova10:23
*** mamandle_ has joined #openstack-nova10:23
*** mamandle has quit IRC10:25
*** mamandle_ is now known as mamandle10:25
*** yamamoto has joined #openstack-nova10:28
*** coreywright has joined #openstack-nova10:30
*** xinliang has quit IRC10:33
*** hshiina has joined #openstack-nova10:34
*** markvoelker has quit IRC10:34
*** shuangyangqian_ has quit IRC10:36
*** omkar_telee has joined #openstack-nova10:37
*** yamamoto has quit IRC10:38
*** alexchadin has quit IRC10:38
*** phuongnh has quit IRC10:39
*** jichen has quit IRC10:40
*** dtantsur|afk is now known as dtantsur10:41
*** xinliang has joined #openstack-nova10:46
*** bkopilov has quit IRC10:46
*** ociuhandu has joined #openstack-nova10:49
*** zhurong has joined #openstack-nova10:52
*** jjung has joined #openstack-nova10:53
*** bhagyashris has joined #openstack-nova10:53
*** thorst has joined #openstack-nova11:03
*** jistr is now known as jistr|tpb11:05
*** trinaths has joined #openstack-nova11:05
*** gszasz has joined #openstack-nova11:10
*** moshele has joined #openstack-nova11:11
*** ralonsoh has joined #openstack-nova11:14
*** kristi___ has quit IRC11:17
*** dave-mccowan has joined #openstack-nova11:17
*** kristian__ has joined #openstack-nova11:17
*** yufei has joined #openstack-nova11:17
*** sree has joined #openstack-nova11:18
*** acormier has joined #openstack-nova11:21
*** lucasagomes is now known as lucas-hungry11:21
*** sree has quit IRC11:23
*** dave-mccowan has quit IRC11:23
*** ralonsoh_ has joined #openstack-nova11:24
*** ralonsoh has quit IRC11:24
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair  https://review.openstack.org/48472611:25
*** jistr|tpb is now known as jistr11:28
*** sapcc-bot has joined #openstack-nova11:30
*** sapcc-bot has quit IRC11:30
*** sapcc-bot has joined #openstack-nova11:31
*** dgonzalez_ has joined #openstack-nova11:31
*** carthaca_ has joined #openstack-nova11:31
*** david_1 has joined #openstack-nova11:31
*** smatzek has joined #openstack-nova11:31
*** markvoelker has joined #openstack-nova11:32
*** trinaths has quit IRC11:32
*** dgonzalez_ has quit IRC11:33
*** carthaca_ has quit IRC11:33
*** david_1 has quit IRC11:33
*** yamamoto has joined #openstack-nova11:33
*** trinaths has joined #openstack-nova11:34
*** lpetrut__ has joined #openstack-nova11:35
*** lpetrut_ has quit IRC11:37
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair  https://review.openstack.org/48472611:37
*** acormier has quit IRC11:37
*** acormier has joined #openstack-nova11:38
*** fragatina has quit IRC11:42
openstackgerritMerged openstack/nova master: api-ref: mention disk size limitations in resize flavor  https://review.openstack.org/48417611:44
*** Daisy_ has joined #openstack-nova11:46
openstackgerritBalazs Gibizer proposed openstack/nova master: Transform instance.resize_revert notification  https://review.openstack.org/39681111:47
*** Daisy_ has quit IRC11:50
*** acormier has quit IRC11:54
*** tylerderosagrund has joined #openstack-nova11:55
*** acormier has joined #openstack-nova11:55
*** esberglu has joined #openstack-nova11:55
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/47709111:55
*** omkar_telee has quit IRC11:56
*** acormier has quit IRC11:59
*** esberglu has quit IRC11:59
bhagyashrissdague: Hi,12:01
*** yamamoto has quit IRC12:04
*** swamireddy has joined #openstack-nova12:04
*** jmlowe_ has quit IRC12:04
*** gmann_ has joined #openstack-nova12:05
openstackgerritArtem Tiumentcev proposed openstack/nova master: Add root and ephemeral disk quota  https://review.openstack.org/48439012:05
*** markvoelker has quit IRC12:05
*** litao__ has quit IRC12:06
*** clayton has quit IRC12:08
*** clayton has joined #openstack-nova12:09
*** kiennt has quit IRC12:14
*** tuanluong has quit IRC12:15
*** yamamoto has joined #openstack-nova12:19
*** sridharg has quit IRC12:24
*** tbachman has quit IRC12:25
*** ratailor has quit IRC12:25
*** lucas-hungry is now known as lucasagomes12:25
*** trinaths has quit IRC12:28
*** jjung has quit IRC12:29
*** edmondsw has joined #openstack-nova12:29
*** MVenesio has joined #openstack-nova12:29
openstackgerritDavanum Srinivas (dims) proposed openstack/nova master: Changes to Volume Encryption Key Manager Options  https://review.openstack.org/48403612:31
*** markvoelker has joined #openstack-nova12:33
openstackgerritAlex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications  https://review.openstack.org/46552612:33
*** sridharg has joined #openstack-nova12:34
*** jmlowe has joined #openstack-nova12:38
*** pchavva has joined #openstack-nova12:40
*** yufei has quit IRC12:42
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair  https://review.openstack.org/48472612:46
*** bkopilov has joined #openstack-nova12:46
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair  https://review.openstack.org/48472612:48
*** danpawlik has quit IRC12:49
*** catintheroof has joined #openstack-nova12:51
*** jaypipes has joined #openstack-nova12:51
*** esberglu has joined #openstack-nova12:52
*** esberglu has quit IRC12:52
*** ralonsoh_ is now known as ralonsoh12:52
*** jwcroppe has joined #openstack-nova12:52
*** danpawlik has joined #openstack-nova12:52
*** kylek3h has joined #openstack-nova12:54
jaypipesmorning supernovas.12:55
*** tbachman has joined #openstack-nova12:55
*** sree has joined #openstack-nova12:55
*** jamesdenton has joined #openstack-nova12:56
*** sree has quit IRC12:56
*** sree has joined #openstack-nova12:56
*** felipemonteiro_ has joined #openstack-nova12:57
*** catintheroof has quit IRC12:58
*** felipemonteiro__ has joined #openstack-nova12:58
*** omkar_telee has joined #openstack-nova12:58
efriedjaypipes Is that a greeting or a weather bulletin?13:00
jaypipesefried: :)13:01
*** felipemonteiro_ has quit IRC13:02
*** esberglu has joined #openstack-nova13:02
jaypipesgibi: looking into your placement troubles now..13:03
jaypipesgibi: MAGIC! The (Non)Gathering13:03
gibijaypipes: thanks for the help13:04
*** catintheroof has joined #openstack-nova13:06
*** eharney has joined #openstack-nova13:06
*** lyan has joined #openstack-nova13:08
*** udesale has joined #openstack-nova13:08
*** lyan has quit IRC13:09
*** lyan has joined #openstack-nova13:10
*** ekuris has quit IRC13:10
*** tbachman has quit IRC13:10
*** lyan has quit IRC13:10
*** catintheroof has quit IRC13:11
*** lyan has joined #openstack-nova13:11
*** hshiina has quit IRC13:11
*** kbaegis has joined #openstack-nova13:11
*** jjung has joined #openstack-nova13:12
*** randomhack has joined #openstack-nova13:14
*** baoli has joined #openstack-nova13:15
*** stvnoyes has left #openstack-nova13:16
*** tbachman has joined #openstack-nova13:17
*** sree has quit IRC13:20
*** tylerderosagrund has quit IRC13:21
*** tylerderosagrund has joined #openstack-nova13:21
*** mamandle has quit IRC13:23
openstackgerritAlex Szarka proposed openstack/nova master: Transform instance-evacuate notification  https://review.openstack.org/48214813:25
*** nicolasbock has quit IRC13:27
*** sree has joined #openstack-nova13:27
*** udesale has quit IRC13:27
*** zhurong has quit IRC13:28
*** udesale has joined #openstack-nova13:28
*** nicolasbock has joined #openstack-nova13:28
jaypipessean-k-mooney, moshele, jangutter: can you all do a google hangout now to discuss the "refctor hw_veb get config" patch? (yes, moshele, "refactor" is still misspelled in your commit summary :P )13:29
*** alexchadin has joined #openstack-nova13:29
jangutterjaypipes: I'm available.13:29
sean-k-mooneyjaypipes: sure i can join13:30
*** lucasxu has joined #openstack-nova13:31
*** jangutte_ has joined #openstack-nova13:31
*** nicolasbock has quit IRC13:32
*** sree_ has joined #openstack-nova13:32
*** sree has quit IRC13:32
*** jangutter has quit IRC13:32
*** jangutt__ has joined #openstack-nova13:32
*** sree_ is now known as Guest9284813:32
*** mamandle has joined #openstack-nova13:33
*** jangutt__ is now known as jangutter13:35
*** nicolasbock has joined #openstack-nova13:35
*** Daisy_ has joined #openstack-nova13:35
*** armax has joined #openstack-nova13:35
*** jangutte_ has quit IRC13:36
*** jangutter is now known as jangutter_13:37
*** links has quit IRC13:38
*** tbachman has quit IRC13:38
*** vishwanathj has quit IRC13:38
*** jjung has quit IRC13:38
*** mriedem has joined #openstack-nova13:39
*** alexchadin has quit IRC13:39
*** jangutter_ is now known as jangutter13:39
*** vishwanathj has joined #openstack-nova13:39
*** alexchadin has joined #openstack-nova13:39
*** Daisy_ has quit IRC13:40
*** smatzek has quit IRC13:40
*** danpawlik has quit IRC13:41
jaypipesgibi: good afternoon sir. wanted to introduce you to bhagyashris.13:41
*** awaugama has joined #openstack-nova13:42
jaypipesgibi: both of you are currently working on testing scenarios for the same thing (shared resource providers)13:42
bhagyashrisgibi: Hi,13:42
jaypipesgibi: I pointed bhagyashris to the ML thread we're currently talking about this.13:42
*** kristia__ has joined #openstack-nova13:42
*** tbachman has joined #openstack-nova13:42
*** udesale has quit IRC13:43
*** danpawlik has joined #openstack-nova13:44
*** esberglu has quit IRC13:45
gmann_mriedem: stephenfin alex_xu can you check this, this can be last patch for policy-doc BP - https://review.openstack.org/#/c/484646/13:46
gibibhagyashris: hi!13:46
*** ociuhandu has quit IRC13:46
*** kristian__ has quit IRC13:46
gibibhagyashris: do you have a patch/ML thread where your work is describe?13:46
*** hamzy has quit IRC13:47
mriedemgmann_: done13:48
*** crushil has joined #openstack-nova13:48
gmann_mriedem: thanks.13:49
bhagyashrisbhagyashris: no not right now,13:49
bhagyashrisgibi: no not right now.13:49
mriedemdansmith: melwitt: top o the morning - one thing i was thinking about at 3am - should we be including build requests in our instance count? we aren't currently, but i'd think that plays a part in checking quota in the API before casting to conductor13:49
jaypipesgibi: I pointed bhagyashris to your ML thread. he's reading it now.13:50
jaypipesas am I :)13:50
*** annegentle has joined #openstack-nova13:50
bhagyashrisgibi: Yeah,13:50
gibibhagyashris: OK. let me know if I can help you with something related to custom resource13:50
gibibhagyashris: I'm looking at that from user perspective, I'm not familiar with the implementation yet13:51
*** omkar_telee has quit IRC13:51
bhagyashrisgibi: Yeah thank you. In case of any doubts i will ping you.13:51
*** catintheroof has joined #openstack-nova13:51
stephenfingmann_: Sure13:52
gibibhagyashris, jaypipes: I will be away soon for an hour but I will be still back today13:52
jaypipesgibi: k. I'm pouring over these logs and script outputs.13:52
bhagyashrisgibi: same I am also not fully aware about the implementation yet but trying to get it.13:53
mamandlealex_xu: bauzas: can you please take a look at the updated changes for https://review.openstack.org/#/c/483911/ , thanks.13:53
dansmithmriedem: aren't we checking it at the point of instance creation?13:53
stephenfingmann_: Ah, mriedem got to it. We should fix that in oslo.policy upstream. It's silly that we have to wrap manually wrap things like that13:53
jaypipesgibi: haven't seen anything out of order yet13:53
*** Guest92848 has quit IRC13:53
gibijaypipes: logs looks clean to me, but somehow placement only like the allocation_candidate request if both providers has the traits13:53
stephenfinjaypipes, mriedem: Am I allowed to remove nova-network features so long as they don't break gate jobs?13:53
stephenfinReferring to jangutter's comments here https://review.openstack.org/#/c/483030/2/nova/network/linux_net.py13:54
mriedemif it's used by CONF.linuxnet_interface_driver and CONF.linuxnet_interface_driver isn't removed, then no13:54
dansmithmriedem: meaning I think we're still gating on actually creating the instance, which means we're fine.. if we included buildrequests we might be able to fail on quota before scheduling I guess, but we'd do more db traffic and be maybe too conservative on some of the checks I think13:55
jaypipesstephenfin: I'd say yes (especially considering all that linked code has been cleaned up and moved into os-vif), but it's mriedem's call.13:55
*** acormier has joined #openstack-nova13:55
mriedemdansmith: the instance is created in conductor, i'm thinking more about when we're still in the api13:55
mriedemin _provision_instances13:55
*** gmann_ has quit IRC13:56
bhagyashrisjaypipes, gibi: I working in the IST time zone so now my time is over. but yeah I will check it tomorrow. Thank you :)13:56
mriedemif we fail to create in conductor, the instances go to ERROR state and are put into cell013:56
jangutterstephenfin: wow, LinuxOVSInterfaceDriver dates from 6 years back.13:56
stephenfinjangutter: yuuup13:56
stephenfinand it's loaded using importlib so it's tough to even figure out where it's used, whether there's coverage etc.13:56
dansmithmriedem: so you're just saying if we check in api and count instances+buildrequests we can fail earlier if they're over quota yes?13:56
stephenfinI despise that pattern13:56
gibibhagyashris: have a good afternoon13:56
mriedemdansmith: i was just thinking about how the UX is before and after - before we'd create a reservation and fail early in API if we are over quota, but now we only check the instance count in the cells but that's ignoring build requests13:56
jaypipesbhagyashris: your watch has ended? :)13:56
gibibhagyashris: I'm in UTC+2 so still a bit of workday left13:57
*** jjung has joined #openstack-nova13:57
mriedemdansmith: yes i think that's what i'm saying13:57
*** mtanino__ has joined #openstack-nova13:57
mriedemso now, we might pass api and get to conductor and fail, and then you've got 20 ERROR instances to cleanup13:57
mriedemif you're bursting13:57
*** gouthamr has joined #openstack-nova13:57
bauzasmriedem: FWIW, I left a comment for https://bugs.launchpad.net/nova/+bug/170478813:57
openstackLaunchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,Confirmed]13:57
dansmithmriedem: because sometimes the instance and build request exist together for a second, you would end up racing to consume your last instance if you're doing a lot of builds, but maybe that's better I dunno13:58
mriedembauzas: where was "because that would mean we would go against the consensus we had in Ocata." documented?13:58
dansmithmriedem: my head is in something else atm, so let us chat with melwitt when she's around13:58
bhagyashrisjaypipes, gibi: yeah now it's 7:30 pm here13:58
jangutterstephenfin: somewhere, someone still has linuxnet_interface_driver = nova.network.linux_net.LinuxOVSInterfaceDriver set in a production config, and they are planning to use it in Queens.13:58
mriedemdansmith: sure, and i'm not liking the idea of adding more complexity to this pile13:58
mriedemjust got thinking about it though13:59
bauzasmriedem: I don't remember exactly when we discussed that, but AFAIR we said that having custom drivers was not something good13:59
jaypipesgibi: in your script, after line 157, can you call GET /allocation_candidates?resources=CUSTOM_MAGIC:512 and tell me what is returned please?13:59
*** esberglu has joined #openstack-nova14:00
mriedembauzas: regardless i think we scrooged the pooch, as the french would say, when it came to that choices restriction being put in since we didn't have a deprecation period on non-standard drivers14:00
dansmithmriedem: yeah, kindof a big thing to change at this point14:00
mriedem*screwed the pooch14:00
mriedemwow14:00
*** smatzek has joined #openstack-nova14:00
stephenfinjangutter: Possibly, but if they do then they're using nova-net and we won't/can't support them anymore because it's deprecated14:00
bauzasmriedem: possibly14:00
*** trinaths has joined #openstack-nova14:01
openstackgerritAlex Szarka proposed openstack/nova master: Refactor create_delete_server_with_instance_update  https://review.openstack.org/46629614:01
mriedemjangutter: stephenfin: to be clear, we don't have to add feature parity or support for any new features for nova-net at this point14:01
mriedemif that's a question14:01
stephenfinmriedem: Not quite. The question is can we remove nova-net stuff if it doesn't break the cells v1 jobs14:01
bauzasmriedem: so, in that case, should we just accept strings and asking to operators to modify setup.cfg ?14:02
stephenfinuntested nova-net stuff, I might add14:02
mriedemstephenfin: can't make that assumption - the cellsv1 job is just one config14:02
mriedembauzas: that's the way it worked prior to the choices restriction14:02
stephenfindrat14:02
stephenfinlooks like it's to stay, jangutter14:03
stephenfin*got to14:03
mriedembauzas: i'm not advocating supporting out of tree scheduler drivers14:03
jangutterstephenfin: here's to 6 more years!14:03
mriedembauzas: so what i'm proposing is we enable that ability, along with immediately deprecating it, and backport that to ocata14:03
mriedemand remove in queens14:03
bauzasmriedem: not really14:03
mriedemjangutter: stephenfin: tbc, i'm also not sure what the context is here14:03
bauzasmriedem: previously, we were not using strings14:03
bauzasmriedem: rather, we asked for the package14:03
mriedembauzas: the scheduler driver config option could be an entry point14:04
*** rmart04 has left #openstack-nova14:04
mriedemin setup.cfg14:04
mriedemand we'd load it with stevedore14:04
mriedemi know14:04
bauzasmriedem: correct14:04
mriedemi'm saying i think we have to go back to supporting that,14:04
mriedemand backport that to ocata14:04
mriedemas a bug fix14:04
mriedemand deprecate it at the same time14:04
mriedemsince the deprecation was never done properly before that was broken14:04
bauzasmriedem: that's why I'd say that if we accept custom drivers, instead of passing a path, you should just pass the name of the entrypoint14:04
mriedemi'm not advocating going back to loading from classpath14:05
mriedemseparate issues14:05
bauzasmriedem: and just changing the type of the option to not be a choice14:05
mriedemyes the choices kwarg would have to be removed14:05
bauzassec, verifying oslo.config14:05
*** baoli has quit IRC14:06
stephenfinmriedem: There's a number of interface drivers for nova-net. We're wondering if some of them can be removed because they're untested, possibly unused, and are the sole consumers of large chunks of code https://review.openstack.org/#/c/483030/2/nova/network/linux_net.py14:06
sean-k-mooneymriedem: out of tree scheduler dirver already work today though correct14:06
stephenfinmriedem: But we've no way of telling if someone is using them, so I guess they've just got to stay til cells v2 is ready14:06
*** baoli has joined #openstack-nova14:07
bauzasmriedem: okay, I'll write the patch14:07
mriedemsean-k-mooney: no14:07
bauzasmriedem: and I'll add a relnote14:07
mriedemsean-k-mooney: https://bugs.launchpad.net/nova/+bug/170478814:07
openstackLaunchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,Confirmed]14:07
sean-k-mooneymriedem: i taught we just had to register a nova.scheduler.driver stevador entry point14:07
mriedemthat was broken in ocata14:07
mriedemby restricting the scheduler driver config option to use the choices kwarg14:07
mriedemwhere your only choices are the known in-tree drivers14:07
mriedemhttps://review.openstack.org/#/c/349666/14:08
mriedemsean-k-mooney: https://review.openstack.org/#/c/349666/16/nova/conf/scheduler.py@6314:08
sean-k-mooneymriedem: ah well honestly i think we need to keep this plug point unless we move the scheduler selection into placement which i dont think is the right approch14:08
*** mmehan has joined #openstack-nova14:09
mriedemsean-k-mooney: why do we need to support out of tree scheduler drivers?14:09
mriedemwe want to make filter scheduler driver use placement14:09
mriedemwe want to deprecate caching scheduler driver14:09
mriedemthe other 2 in tree drivers are basically fakes for testing14:09
mriedemnote that the scheduler driver plugin != the scheduler filter plugin14:10
*** jdillaman has quit IRC14:10
*** alexchadin has quit IRC14:10
mriedemunless the argument is the same as why we allow out of tree virt drivers14:10
sean-k-mooneymriedem: several operators use costom scheduler and if we want to get to a point where we have a unifed scheduler that means introducing more filters or create a new scheduler14:10
mriedemsean-k-mooney: we don't want to get to a point of a unified scheduler14:11
mriedemgantt is dead14:11
mriedemthe nova scheduler is going to be biased toward compute things for nova14:11
mriedemplacement is the place to put generic things14:11
sean-k-mooneynova does not most compay i talk to in the mano space do14:11
mriedemconsumable by all other services14:11
*** Guest73843 is now known as antonym14:11
*** gbarros has quit IRC14:11
*** antonym has quit IRC14:11
*** antonym has joined #openstack-nova14:11
bauzasmriedem: that's why I asked for questions about why people want custom drivers14:12
bauzasmriedem: not filters14:12
*** jheroux has joined #openstack-nova14:12
bauzasmriedem: my main point would be that nova would only pass to the scheduler driver hosts that are supported by placement14:12
mriedemi can see it's going to take me awhile to get all of this shit off my shoe that i stepped into yesterday14:12
sean-k-mooneymriedem: i agree with jay that placement should not make scheduling decision but instead return a list of candiates that a scheduler then claims from. the filter schedueler is a good default but it may not be smart enough to cover all usecases14:13
bauzasmriedem: how then the scheduler driver would return a destination between all those supported hosts is possibly something custom14:13
mriedemsean-k-mooney: what does a custom driver buy you that custom filters can't?14:13
*** alexchadin has joined #openstack-nova14:14
*** fragatina has joined #openstack-nova14:14
*** tbachman has quit IRC14:14
*** coreywright has quit IRC14:14
sean-k-mooneymriedem: the abblity to intergrate with external inventory systems in addtion to placement to make a desission. i may not like the design of ONAP but in there model they own all inventory resources which is a conclift with how nova works today14:14
bauzasmriedem: I thought about something14:15
bauzasmriedem: what if we would pass a new choice named 'custom'14:15
*** fragatina has quit IRC14:15
bauzasmriedem: that would mean that if you choose 'custom', then you need to modify setup.cfg to add a 'custom' entrypoint14:15
mriedemsean-k-mooney: still, you can't do that in a filter?14:15
sean-k-mooneybauzas: i like the idea of tighing the contract for scheduler drivers for queens on to requried them to support placement.14:16
*** fragatina has joined #openstack-nova14:16
openstackgerritSean Dague proposed openstack/nova master: Ironic: Support boot from Cinder volume  https://review.openstack.org/21538514:16
bauzassean-k-mooney: you can call external inventory systems within a filter14:16
*** annegentle has quit IRC14:16
bauzassean-k-mooney: that said, it will call N times the 3rd party system, N being the number of hosts14:16
edleafemriedem: sean-k-mooney: like the way that TrustedFilter can call out?14:17
*** mgiles has joined #openstack-nova14:17
bauzasthat's the only limitation of that within a filter14:17
bauzasedleafe: shhhhhttttt14:17
mriedemedleafe: yes that or the attestation filter14:17
mriedemor did i just make that up14:17
bauzasedleafe: I *really* *hate* TrustedFilter14:17
bauzasbecause it's in tree14:17
edleafebauzas: me too, but it's "in-tree" :)14:17
sean-k-mooneyedleafe: yes that is one example where it needs to talk to an atestation server.14:17
bauzasmriedem: attestation server = silly TrustedFilter14:17
mriedemoh yes14:18
*** burt has joined #openstack-nova14:18
mriedemtrusted filter is gone in queens14:18
mriedemso don't complain too much14:18
sean-k-mooneymriedem: bauzas  i could do most of this with a costom filter + costum weigher to update the external inventory system with the final choice14:18
mriedemi'd argue the custom external inventory system should be pumping custom resource class information into placement14:18
mriedemwhich the filter scheduler can then use14:18
edleafemriedem: that would be best14:19
bauzashangout time ?14:19
edleafemriedem: but not always feasible14:19
sean-k-mooneybauzas: the trusted filter today dose not vaildate that the plathfrom trust is still present just that you labled it as such. with the atestation server it dose a mesuared boot and attest the trust is still valid14:19
bauzasI really wanted to explain my thoughts but my brain translates too slow14:19
bauzasso I hope my thoughts could be clearer if I would voice14:20
mriedemi can't spend all morning on this, so if there is further discussion we should probably move it to the ML14:20
bauzasto make it clear, thoughts are :14:20
mriedemi'll mark that bug as rc potential though since i think we need to handle it somehow14:20
bauzas1/ placement for all drivers14:20
mriedemnack to 114:20
mriedemyou can't enforce it14:20
bauzasI'd love to14:21
*** eharney has quit IRC14:21
mriedembauzas: just start a ML thread14:21
sean-k-mooneybauzas: mriedem  i can support my usecase with costome filter and weighers but a custom self contaitned driver just seemed cleaner so dont block on this cleanup on my account14:21
bauzasplacement would give to the driver which hosts to schedule14:21
mriedemand we'll bikeshed to exhaustion there14:21
sean-k-mooneydansmith: mriedem i just wanted to hilight that there may be usecase implications that would be limited14:21
sean-k-mooneybauzas: ^ not dansmith14:21
mriedemsean-k-mooney: honestly anyone that wants that level of custom is just going to carry a patch to revert the change to break the entrypoint anyway14:21
bauzashow the driver is choosing between hosts (either by looping over filters or doing at once a choice) is left to the internals of the driver14:21
bauzaslike, ChanceScheduler should just random.choice() between what's passed by placement, that's it14:22
gibijaypipes: asking for MAGIC only leads to HTTP 500 with a stack trace: http://paste.openstack.org/show/615732/14:22
bauzasbut having ChanceScheduler having a different list of hosts from FilterScheduler seems to me very wrong14:23
jaypipesgibi: aha! :) that's the source of the issue I think14:23
sean-k-mooneymriedem: ya probably but for interop reason it would be nice if they did not need too.14:23
jaypipesgibi: cdent fixed that bug.14:23
jaypipesgibi: one sec, grabbing link to patch for you to apply.14:23
gibijaypipes: thanks14:23
gibijaypipes: sounds good14:23
mriedemsean-k-mooney: if you have a custom out of tree scheduler driver, there is no interop guarantee14:24
bauzashow a scheduler driver works should be a blackbox, but inputs of that blackbox should necessarly be identical between all drivers14:24
mriedemsean-k-mooney: so it makes no difference imo14:24
sean-k-mooneymriedem: the other way to solve this is to allow claims against placement api resouces  to be passed as part of the nova boot and have a filter that will use those claimes for the placement.14:24
bauzasmriedem: no, you could interop if you decide between the same hosts14:24
jaypipesgibi: try this out: https://review.openstack.org/#/c/484162/14:24
sean-k-mooneymriedem: yes that is true14:24
gibijaypipes: OK, I will report back soon14:24
jaypipesbtw, mriedem and dansmith, https://review.openstack.org/#/c/484162/ is a good fix (and important one)14:24
bauzasmriedem: what I think people want is crazy foos of decision making based on weather or name of my pants14:24
mriedemjaypipes: starred14:25
dansmithbauzas: that was impressively unintelligible14:25
*** bnemec has joined #openstack-nova14:25
bauzasdansmith: isn't the intent ? :)14:25
dansmithbauzas: I dunno, I just saw that notification pop up and noticed not all those words belong together14:26
bauzaseveryone wants their blackbox for deciding which kid they have is the best14:26
dansmithif that was the intent, then.. bravo :)14:26
bauzasthe real problem is that the list of kids is depending on the blackbox14:26
sean-k-mooneybauzas: mriedem basically for the ONAP case when a host is selected and resouce allocated they need to relay that up into there own invetory system also. i think today they do that via connectin directly to the nova db.14:26
bauzassean-k-mooney: sure, but my point is that the input you should get should only hosts that placement agrees with14:27
sean-k-mooneybauzas: yes i agree with that too14:27
bauzassean-k-mooney: like, you should never choose between hosts that don't have resources for the asked flavor14:27
sean-k-mooneybauzas: yes14:28
bauzassean-k-mooney: that's how we guarantee interop14:28
bauzasa flavor is an interop thing14:28
*** sree has joined #openstack-nova14:28
bauzasso whatever the decision system is doing, you as a user will be sure that your instance will be on a host that can support X CPUs, Y RAM and Z disk14:28
bauzasthe rest is left to you14:29
sean-k-mooneybauzas: so as i said there are way to supprot them if they need it without a costom driver. i just dont know how many people have already used the plugable interface that would be effect by the bug.14:29
bauzasyou could decide to pick host A against B and C because you're Saggitarian if you wish14:29
dimsmriedem : i had a cleanup from looking at castellan, do we want to do that now or when queens opens up? https://review.openstack.org/#/c/484036/14:30
bauzasthat's why I turned my choice14:30
mriedemalex_xu: i replied in https://review.openstack.org/#/c/469800/14:31
mriedemdims: that conflicts with a change that kfarr did14:31
mriedemdid you see that?14:31
dimsoops checking14:31
*** hongbin has joined #openstack-nova14:31
mriedemdims https://review.openstack.org/#/c/439855/914:31
* jaypipes needs to buy sean-k-mooney an auto-spell-checker for his IRC client. :)14:31
*** hamzy has joined #openstack-nova14:32
dimsmriedem : kfarr's works! looking deeper14:32
sean-k-mooneyjaypipes: hehe i need to figure out how to integreate weechat with spell also a git hook for same14:32
jaypipessean-k-mooney: :)14:33
*** sree has quit IRC14:33
*** jamesdenton has quit IRC14:33
dimsmriedem : yep +1 on that review, we should get that in for Pike if possible14:34
openstackgerritMerged openstack/python-novaclient master: Updated from global requirements  https://review.openstack.org/48458214:34
*** coreywright has joined #openstack-nova14:34
jaypipessean-k-mooney: re: ONAP, if they have chosen to architect their system by subverting the public interfaces of Nova, it is not our job to cater to those architectural fauxpas. The best advice I would have to ONAP folks is what mriedem said and have their global inventory system work with the placement service via Placement's public REST APIs and let the filter scheduler do its job (with custom weighers as you mentioned)14:34
*** marst has quit IRC14:35
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-volume-attachments instead of stevedore  https://review.openstack.org/48482414:35
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for os-snapshots instead of stevedore  https://review.openstack.org/48482514:35
openstackgerritAlex Xu proposed openstack/nova master: Use plain routes list for extension_info instead of stevedore  https://review.openstack.org/48482614:35
sean-k-mooneyjaypipes: yep i agree and that is the direction i would be pushing them towords but i do not spend much time on ONAP to be honest so not sure how much they will listen.14:36
jaypipessean-k-mooney:  who are the driving forces in ONAP?14:36
jangutterjaypipes: Telcos, I believe.14:37
*** eharney has joined #openstack-nova14:37
sean-k-mooneyjaypipes: AT&T from the ecomp side and several asian telcos form the OpenO community. ONAP is a fusion of both comunitys under the linux foundation. intel is a member but we have not been invole in the achitecher of this part whic come mainly form ecomp i belive14:38
jaypipesjangutter: well, yeah :)14:38
openstackgerritSylvain Bauza proposed openstack/nova master: Add a scheduler custom driver possibility  https://review.openstack.org/48482814:38
sean-k-mooneyjaypipes: there is little european representation in ONAP current with most European telcos look more to OSM instead14:38
jaypipessean-k-mooney: wow, that was impressive :) comunitys, invole, achitecher, whic and belive. all new words in one sentence!14:39
bauzassean-k-mooney: mriedem: jaypipes: I need to disappear from a certain amount of time because I need to pick my daughter and do some stuff, but I'll respond to the ML thread14:39
jaypipessean-k-mooney: ok, I'm going to stop poking fun now :)14:39
bauzassean-k-mooney: mriedem: jaypipes: reviews welcome on https://review.openstack.org/48482814:39
jangutterjaypipes: there was _lots_ of representation from Chinese telcos on ONAP at OPNFV.14:40
sean-k-mooneyjaypipes: :)14:40
jaypipesjangutter: k. good to know. I'm more looking for specific people, though. any names or emails would be good to get so I can reach out and have a chat.14:41
jaypipesI suppose I can hit up alanmeadows from AT&T for that info14:41
*** marst has joined #openstack-nova14:42
jangutterjaypipes: I'll see if there's a nice mini-summit link for the presentations with contact details.14:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section  https://review.openstack.org/47847314:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'user' section  https://review.openstack.org/47847514:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme  https://review.openstack.org/47775114:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section  https://review.openstack.org/47847414:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext  https://review.openstack.org/48296114:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs  https://review.openstack.org/48007414:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext  https://review.openstack.org/47935814:42
openstackgerritStephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues  https://review.openstack.org/48051614:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section  https://review.openstack.org/47848414:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Rework index page per new sections  https://review.openstack.org/47848514:42
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove dead files  https://review.openstack.org/47847014:43
* jaypipes bashes stephenfin over the head.14:43
jaypipesjust kiddin, stephenfin :)14:43
jangutterjaypipes: http://events.linuxfoundation.org/events/opnfv-summit/extend-the-experience/onap14:43
jangutterjaypipes: just names though, you should be able to hit the presentations from the 2017 summary site.14:44
mriedemi'm sure i can ask around within huawei and someone might know14:44
stephenfinjaypipes: Sorry - merge conflicts, yet again 🙈14:44
* stephenfin goes back to reviewing https://etherpad.openstack.org/p/nova-pike-feature-freeze-status patches14:44
mriedemstephenfin: reviewing https://review.openstack.org/#/c/463987/ would be good14:47
sean-k-mooneybauzas: maybe add a release note but i think that review looks fine otherwise14:47
mriedemit's passed https://review.openstack.org/#/c/481290/14:47
mriedemsean-k-mooney: bauzas: i don't, but review comments incoming14:47
*** ckopper has quit IRC14:48
*** elod has joined #openstack-nova14:48
stephenfinmriedem: eek. ildikov has been imploring me to look at that, but I don't feel I know that code enough to really sign-off on it14:49
*** gbarros has joined #openstack-nova14:49
stephenfinI'll try, but don't blame me if everything explodes afterwards, heh14:49
mriedemit's ok to ask questions and not +214:50
mriedemi've been through it a couple of times14:50
*** annegentle has joined #openstack-nova14:50
sean-k-mooneymriedem: https://review.openstack.org/#/c/463987 is basically using the same workflow as the neutron portbinding then? eg createing a new binding/attachemt in this case on the destination and only removing the old one on succesfull migration?14:51
mriedemsean-k-mooney: similar yes14:51
sean-k-mooneymriedem: ok cool i have no ideay how the cinder integration works so proably wont review but that workflow makes sense to me. good to know14:51
mriedemdims: i'd like to get this in first https://review.openstack.org/#/c/484501/ so we can backport that to stable14:52
mriedembecause of the associated bug14:52
dimsmriedem : +1, though we should probably add an upgrade note to ask operators to make sure that option is set in everyone's nova.conf14:54
*** keerthi has joined #openstack-nova14:55
*** krtaylor has quit IRC14:55
mriedemdims: hmm, if we have to add an upgrade release note then i worry about backporting it14:57
mriedemdims: honestly i'm not really sure why the set_default there is overriding what the operator puts in the config,14:57
mriedembut that was reported yesterday14:57
mriedemdims: if you could figure that out it would be helpful14:58
*** kbaegis1 has joined #openstack-nova14:59
dimsmriedem : it should not be overriding what's in nova.conf, it just sets the default value if there's nothing in nova.conf14:59
dimsmriedem : http://git.openstack.org/cgit/openstack/oslo.config/tree/oslo_config/cfg.py#n273614:59
gibijaypipes: with cdent fix I still get the same KeyError as before, logs are here http://paste.openstack.org/show/615743/15:01
keerthischeduling always happen only 10 at a time in the nova ?15:01
*** mamandle has quit IRC15:01
keerthishall we change this value ?15:01
*** kbaegis has quit IRC15:03
keerthimriedem ?15:03
jaypipesgibi: k, thanks for that log output, that's helpful. I'm wondering if alex_xu's patch here might help: https://review.openstack.org/#/c/480379/15:04
jaypipesgibi: sorry to bother you on this, but do you think you can apply that patch too?15:04
gibijaypipes: I can test that as well15:04
jaypipesgibi: ty sir, much appreciated!15:04
jaypipesgibi: in the meantime, I am working on adding functional test scenarios for your specific case.15:04
gibijaypipes: put me in the review if any I will check that out15:05
jaypipesgibi: cheers15:05
alex_xumriedem: replied https://review.openstack.org/#/c/469800/, if we really think it is time to merge this patch first, I can change to +w15:05
jaypipesgibi: köszönöm!15:05
mriedemdims: yeah that's what i thought, but he was saying he had a value set in nova.conf and it would still use the default from code, and removing that line fixed it15:06
gibijaypipes: szivesen15:06
mriedemdims: i don't have a devstack handy to recreate15:06
mriedemkeerthi: scheduling what? server create?15:06
keerthiyes...15:06
dimsmriedem : barbican has a DSVM where this scenario works just fine. nova.conf has the BarbicanKeyManager and it gets picked up ok15:07
keerthiwhen ever i create boot request...scheduling happens only for 10 at a time15:07
*** ckopper has joined #openstack-nova15:07
keerthirest of the requests need to wait in the queue and it starts proceeding next..is there way we can change this behaviour in nova ?15:08
dimsmriedem : oops, castellan has a dsvm test  - see http://logs.openstack.org/19/476819/1/check/gate-castellan-dsvm-functional/c6e0d8c/logs/etc/nova/nova.conf.txt.gz15:08
keerthimriedem ?15:08
*** jjung has quit IRC15:09
*** kbaegis1 has quit IRC15:09
dimsmriedem : api log - http://logs.openstack.org/19/476819/1/check/gate-castellan-dsvm-functional/c6e0d8c/logs/screen-n-api.txt.gz#_Jun_23_08_05_22_68165215:09
mriedemkeerthi: nothing comes to mind15:09
*** kbaegis has joined #openstack-nova15:09
*** ociuhandu has joined #openstack-nova15:09
keerthijaypipes can you help me in solving my issue ?15:10
mriedemdims: hmm, yeah, some of the options stuff was also changed in i think pike, so i'm wondering if it was a latent bug in newton15:10
jaypipeskeerthi: please see /topic. this isn't a support channel. better to post a question to the openstack mailing list please.15:11
mriedemkeerthi: you probably have max_concurrent_builds defaulting to 1015:12
dimsmriedem : nothing has changed in set_default (oslo.config) or in nova keymgr in a while.... will search though15:12
mriedemwhich is used in the ompute15:12
mriedem*compute15:12
mriedemnot the scheduler15:12
mriedemjaypipes: ^15:12
*** gbarros has quit IRC15:12
keerthiThanks mriedem. i will look in to this15:12
*** Oku_OS is now known as Oku_OS-away15:13
*** kbaegis has quit IRC15:15
*** chyka has joined #openstack-nova15:15
*** yamamoto has quit IRC15:17
mriedemjaypipes: not-tags-any is tested in https://review.openstack.org/#/c/469800/15:19
mriedemhttps://review.openstack.org/#/c/469800/35/nova/tests/functional/wsgi/test_servers.py@23715:19
mriedemthe 4 filters are tested in the same functional test, there just need to be more wrinkles it sounds like15:19
mriedemand we can't do any of this in sql15:20
mriedemwhich sucks, but it is what it is15:20
*** yamamoto has joined #openstack-nova15:21
jaypipesmriedem: ok, fair enough. will remove my -1.15:21
*** omkar_telee has joined #openstack-nova15:21
gibijaypipes: with alex_xu's https://review.openstack.org/#/c/480379/ I still get the same stack trace. logs are here http://paste.openstack.org/show/615747/15:21
mriedemthanks15:22
gibijaypipes: I will try to combine cdent's and alex_xu's patch together15:22
jaypipesgibi: k15:22
*** omkar_telee has quit IRC15:22
*** Daisy_ has joined #openstack-nova15:24
*** gszasz has quit IRC15:25
*** tbachman has joined #openstack-nova15:25
*** mgiles has quit IRC15:28
*** krtaylor has joined #openstack-nova15:28
*** Daisy_ has quit IRC15:29
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: Test changes with multiple cells  https://review.openstack.org/46738315:29
jaypipesgibi: hmm, ok I have an idea...15:29
*** yamamoto has quit IRC15:29
jaypipesgibi: can you do me a favor?15:30
jaypipesgibi: these two lines: https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2393-L239415:30
gibijaypipes: sure15:30
jaypipesgibi: can you change them to have the usage fields on the *right* side of the condition?15:30
jaypipesgibi: in other words, they should be: inv.c.resource_provider_id == usage.c.resource_provider_id15:31
jaypipesand the same for the resource_class_id columns.15:31
openstackgerritGábor Antal proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485115:31
gibijaypipes: sure I can do that. Meanwhile the two combined patches resulted the same HTTP 50015:31
*** sshwarts has quit IRC15:31
gibijaypipes: do you need that change top of master or top of some bugfixes?15:31
gibijaypipes: or doesnt matter15:32
jaypipesgibi: I'm wondering if SQLAlchemy is constructing the LEFT JOIN verbatim with that order (which is an incorrect join order)15:32
jaypipesgibi: just make it locally on whatever code you have running.15:32
gibijaypipes: OK15:32
*** rcernin has quit IRC15:32
*** mgiles has joined #openstack-nova15:33
*** vishwana_ has joined #openstack-nova15:33
jaypipeszzzeek: hey Mike, does SA rewrite outerjoins to be the "correct" join condition column order if there's a mistake in the order of the join condition? :) see https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2393-L239415:33
mgilesmriedem ildikov I took a pass as the grenade tests that stvnoyes was going to work on.  See: https://review.openstack.org/#/c/484469/15:33
jaypipeszzzeek: it would be awesome if SA solved all my mistakes for me :P15:33
*** mvk has quit IRC15:34
ildikovmgiles: great, thank you!15:35
zzzeekjaypipes: column order....i dont think so?  you mean "A LEFT OUTER JOIN B" and not "B LEFT OUTER JOIN A" ?15:36
*** itlinux has joined #openstack-nova15:36
*** vishwanathj has quit IRC15:37
zzzeekjaypipes: like you want the two conditions to be in some order to satisfy an index or something ?15:37
jaypipeszzzeek: no, I was mostly joking with you :) I have the SQL correct in the code comment above there but have the column order wrong in the SQLalchemy join.15:39
*** annegentle has quit IRC15:39
zzzeekjaypipes: the SQL is rendering differently from what you specify ?15:40
ildikovstephenfin: happy to try to answer questions if you can take a look at the live_migrate patch :)15:40
jaypipeszzzeek: not sure, but I suspect it would be. I mean, I'm asking SQLalchemy to do a LEFT JOIN using the wrong order of columns in the join condition.15:41
*** kbaegis has joined #openstack-nova15:41
*** kbaegis1 has joined #openstack-nova15:41
jaypipeszzzeek: it's my mistake, not SA's :) I was just joking with you about having SA read my mind ;)15:41
zzzeekjaypipes: i still don't understand what "wrong order of columns" means15:42
*** trinaths has quit IRC15:42
jaypipeszzzeek: oh, I'm saying that I should be doing a LEFT JOIN b ON a.col = b.col, but I'm asking SA to do a LEFT JOIN b ON b.col = a.col15:42
zzzeekjaypipes: OK you mean in the condition around an operator15:43
jaypipesand maybe SA is doing b LEFT JOIN a ON b.col = a.col15:43
jaypipeszzzeek: ya15:43
zzzeekjaypipes: if you are doing "col <operator> col" it should preserve that order15:43
*** crushil has quit IRC15:43
jaypipeszzzeek: right, and that order is wrong :)15:43
zzzeekjaypipes: only if you have "<some literal python thing> <operator> col" might it switch things because the __eq__ operator is on the right15:43
jaypipeszzzeek: thus me saying it was my mistake ;)15:43
*** baoli has quit IRC15:44
*** gbarros has joined #openstack-nova15:44
gibijaypipes: I'm getting the same stacktrace after reodering the condition. git diff is on the top of the logs: http://paste.openstack.org/show/615749/15:44
zzzeekjaypipes: OK so, if you have "col <operator> othercol" that left/right is maintained15:44
*** andreas_s has quit IRC15:44
zzzeekjaypipes: also, it....shouldnt matter?  unless you're trying to hit an index on oracle15:44
zzzeekjaypipes: == operator is commutative...15:45
jaypipesgibi: :( ok, back to the drawing board. I really don't know why a KeyError is being raised there. the root provider ID should be in the summaries dict since _get_usages_by_rp_and_rc() should be returning a record for that rp15:45
*** kbaegis has quit IRC15:45
gibijaypipes: is there any log I can turn on to help?15:46
gibijaypipes: or if you provide a patch with extra LOGs then I can apply that15:46
jaypipeszzzeek: right, but I was thinking maybe SA saw the == operator column order and maybe made the expression b LEFT JOIN a instead of a LEFT JOIN b.15:46
jaypipeszzzeek: apparently not, though15:46
jaypipesgibi: I'll do the latter15:46
gibijaypipes: OK. I'm still around for an hour or so then I can continue tomorrow15:47
gibijaypipes: thank again for helping15:47
jaypipesgibi: I'll have a patch up in 5 mins.15:47
zzzeekjaypipes: ah.  no way :)15:47
gibijaypipes: I will test that!15:47
openstackgerritMatt Riedemann proposed openstack/nova master: DNM: test new style cinder attach with upgrades  https://review.openstack.org/48486015:48
mriedemmgiles: thanks, testing it here ^15:48
*** baoli has joined #openstack-nova15:49
mgilesmiredem: great! thanks15:49
mgilesmriedem ^15:49
melwittmriedem: I think we've glossed over that in the review and the thought was, we can't count those atomically together with instances because they're in the API DB ...15:51
*** stvnoyes has joined #openstack-nova15:51
*** baoli has quit IRC15:51
openstackgerritJay Pipes proposed openstack/nova master: TESTING - DO NOT MERGE  https://review.openstack.org/48486215:51
jaypipesgibi: ^^15:51
*** crushil has joined #openstack-nova15:52
*** baoli has joined #openstack-nova15:53
jaypipesdtantsur: looking at that devstack custom RCs patch now...15:53
*** annegentle has joined #openstack-nova15:53
mriedemmelwitt: do we count instances atomically across multiple cells?15:53
dtantsurthanks!15:53
gibijaypipes: looking...15:53
mriedemmelwitt: don't we have a separate session for each cell db?15:53
melwittmriedem: no, each cell is atomic15:53
dansmithmriedem: we can't count atomically across cells15:54
*** sree has joined #openstack-nova15:54
mriedemright,15:55
mriedemso my point is, saying we can't do it atomically b/c of the api db is kind of a cop out15:55
mriedemsince we can't do it atomically for multiple cells either15:55
dansmithwell,15:55
dansmithbut the cells don't overlap with each other15:55
mriedemtrue, yes15:55
dansmiththe api db and the cell dbs do overlap15:55
mriedemthe race window is a problem15:55
mriedembetween build requests and instances15:55
*** sree has quit IRC15:56
mriedemagain, i don't think i'm advocating trying to account for build requests in flight at the same time as counting the instances15:56
mriedemi just want to make sure we thought about it and are ok with it15:56
*** sree has joined #openstack-nova15:56
*** ckopper has quit IRC15:56
jaypipesdtantsur: answered.15:57
melwittmriedem: yeah, I mean, I was thinking ideally we should count them because they are parts of instances15:58
*** alexchadin has quit IRC15:58
dtantsurjaypipes: thanks! I wonder if we should enroll nodes after nova-compute is started then15:58
dtantsurjaypipes: to better emulate how things work in actual production15:58
dtantsurwdyt?15:59
mriedemthe "for x in 5; nova boot --min-count 20...." worries me15:59
*** damien_r has quit IRC15:59
jaypipesdtantsur: well, they automatically get enrolled when the nova-compute node starts, but it doesn't look like that has run by the time the test tries to boot an instance.15:59
dtantsurthis is suspicious.. okay, thanks for the hints. I'll take a deeper look tomorrow16:00
jaypipesdtantsur: looks to be just a simple ordering issue to me.16:00
jaypipesdtantsur: I will look further into it as well and leave comments on the patch.16:00
*** smatzek has quit IRC16:00
dtantsurthanks :)16:00
*** brad[] has quit IRC16:00
*** vishwana_ has quit IRC16:01
*** smatzek has joined #openstack-nova16:02
*** vishwanathj has joined #openstack-nova16:02
*** rcernin has joined #openstack-nova16:02
*** baoli has quit IRC16:02
*** tylerder_ has joined #openstack-nova16:03
*** gbarros has quit IRC16:04
*** lyan has quit IRC16:05
*** tylerderosagrund has quit IRC16:07
*** dtantsur is now known as dtantsur|afk16:07
*** sridharg has quit IRC16:08
*** armax has quit IRC16:11
*** baoli has joined #openstack-nova16:12
*** armax has joined #openstack-nova16:12
melwittmriedem: yeah. the bad thing about counting both is it could get too much usage if you say, count build requests and then count instances, some of those instances could have been build requests a split second ago, and then you get too much usage after you add them together16:14
*** Apoorva has joined #openstack-nova16:15
*** dtp has joined #openstack-nova16:17
gibijaypipes: here is the log http://paste.openstack.org/show/615753/16:17
*** rajathagasthya has joined #openstack-nova16:17
*** Daisy_ has joined #openstack-nova16:19
jaypipesgibi: excellent, that helps a lot, thank you!16:20
jaypipesgibi: the usage information isn't being returned for rp 1 for some reason.16:20
gibijaypipes: rp 1 is the compute provider?16:21
jaypipesgibi: yeah16:21
melwittmriedem, dansmith: apparently sqlalchemy supports two-phase commit. do you think that's something we could use to get atomic across databases? http://docs.sqlalchemy.org/en/latest/orm/session_transaction.html#enabling-two-phase-commit16:22
*** vishwanathj has quit IRC16:22
*** annegentle has quit IRC16:22
jaypipesmelwitt: I wouldn't go that route. It's a pain in the ass, frankly.16:22
dansmithnot worth it, IMHO16:23
*** Apoorva_ has joined #openstack-nova16:23
melwitthm, okay16:24
*** Daisy_ has quit IRC16:24
jaypipesgibi: what's the result of SELECT * FROM inventories WHERE resource_provider_id = 1?16:24
melwittdansmith: I guess the rechecking quota stuff takes care of the gap between build requests and instances ... in that after creating the instance objects we check quota again. so maybe that's how not counting build requests can be fine (if someone is configured to be strict about quotas)16:26
dansmithmelwitt: yep16:26
melwittmriedem: ^16:26
melwittwhew, good.16:26
*** randomhack has quit IRC16:27
*** mmehan has quit IRC16:27
openstackgerritMerged openstack/nova master: [placement] fix 500 error when allocating to bad class  https://review.openstack.org/48416216:27
*** Apoorva has quit IRC16:27
mriedemmelwitt: you mean the recheck performed in conductor?16:27
melwittmriedem: yeah16:27
gibijaypipes: http://paste.openstack.org/show/615755/16:28
mriedemyeah, my point earlier today was the difference in ux - before counting qoutas, you'd go overquota and fail the api with a 403 and no instances are created - after we if we catch it in conductor, the instances all get put into ERROR state and you have to clean them up16:28
mriedemmelwitt: not the end of the world, but it's different16:28
*** yamamoto has joined #openstack-nova16:29
dansmithnow I'm forgetting,16:29
dansmithbut the first check is done in api, and the extra if configured check is in conductor, right?16:29
melwittdansmith: yes16:29
mriedemyeah16:29
mriedemwe recheck for cells v1 in the api16:30
*** coreywright has quit IRC16:30
dansmithright, so the ux change is only across the boundary16:30
mriedemb/c that's where the instances are created16:30
mriedembut...cellsv116:30
dansmithwhich isn't that big of a deal, IMHO16:30
dansmithquota is so leaky right now,16:30
mriedemit's not the end of the world,16:30
mriedemi only ever worry about the NFV robots hammering things in unexpected ways16:30
jaypipesgibi: are you calling GET /allocation_candidates?resources=CUSTOM_MAGIC:512 or are you including VCPU and MEMORY_MB in the request as well?16:30
dansmithwe could have added another check to the current stuff where we decided to fail your instance because we healed the quota and the numbers no longer work out16:31
openstackgerritMerged openstack/nova master: Support tag instances when boot(4/4)  https://review.openstack.org/46980016:31
melwittevery quota.reserve() call heals before checking16:31
mriedemheal == refresh()?16:31
melwitter, it heals YOUR quota. but not others in your project. so you could still be screwed16:31
dansmithmelwitt: it doesn't heal negative things, right?16:32
*** jangutter has quit IRC16:32
dansmithlike when things get all out of whack and require admin and nova-manage intervention16:32
dansmiththat's what I meant16:32
*** jangutter has joined #openstack-nova16:32
gibijaypipes: I'm calling just MAGIC:512 first, that results in HTTP 500 then I call with both MAGIC and VCPU + MEMORY that goes through but results in an empty response16:32
melwittI don't think it stores negative usage16:32
*** rajathag_ has joined #openstack-nova16:33
jaypipesgibi: can you paste the logs from the second request for VCPU, MAGIC and MEMORY_MB?16:33
melwittyeah, you need nova-manage when not all users in the cloud are active in booting instances and their quota is out of whack and affecting those who are active16:33
dansmithmelwitt: that's all I mean.. when the current broke-ass shit gets broke-ass16:33
melwittyeah16:33
*** rajathagasthya has quit IRC16:33
mriedemcan i borrow $5 for some quota?16:34
dansmithmriedem: you're just going to spend it on booze, so no16:34
* dansmith has seen this scam before16:34
* mriedem itches himself16:34
mriedems/itches/scratches/16:35
mriedemsorry16:35
melwittheh. I was thinking about that. lots of people where I grew up did say you itch an itch. which never made sense to me16:35
gibijaypipes: that generate a lot less log: http://paste.openstack.org/show/615756/16:36
*** yamamoto has quit IRC16:37
jaypipesdansmith, mriedem: so, here's an interesting little wrinkle... if I request *only* resources that are provided by a sharing provider, and don't request any resources that say, a compute node, would have, there's no way to return an AllocationRequest that would include the compute node resource provider UUID.16:37
*** derekh has quit IRC16:37
dansmithwhy would you16:38
dansmith?16:38
dansmithif you don't request any resources from the compute node... you don't get a compute node16:38
*** lpetrut__ has quit IRC16:38
jaypipesdansmith: but the compute node is a target because the shared resource is shared with it.16:39
dansmithcan you give a concrete example?16:39
dansmithbecause shared disk should be allocate-able without a compute node if you don't care about CPU/RAM right?16:39
*** brad[] has joined #openstack-nova16:40
jaypipesdansmith: I actually can't think of a concrete use-case here.16:40
mriedemif you request things that don't result in compute nodes, you just get NoValidHost16:40
mriedemwhich is fine for nova scheduler right?16:40
dansmithmriedem: from scheduler, but not from placement16:41
mriedemwe can't build you a server on a ceph pool alone16:41
mriedemdansmith: sure16:41
jaypipesmriedem: well, technically you get a KeyError right now :) but, even then, I'm not sure.16:41
*** kristia__ has quit IRC16:41
dansmithright, but it should be fine for cinder to ask for an allocation against a storage provider with nothing else16:41
mriedemyeah so let's fix the KeyError in placement16:41
mriedemright16:41
*** kristian__ has joined #openstack-nova16:41
jaypipesmriedem: yeah, I'm working on fixing that :)16:42
*** coreywright has joined #openstack-nova16:43
*** tylerder_ has quit IRC16:46
*** tylerderosagrund has joined #openstack-nova16:47
*** yamahata has joined #openstack-nova16:48
*** ociuhandu has quit IRC16:48
melwittmriedem: something I was wondering, I noticed the func tests you wrote are in novaclient, so we wouldn't catch a problem unless we look at novaclient jobs16:49
jaypipeshttps://bugs.launchpad.net/nova/+bug/170507116:50
openstackLaunchpad bug 1705071 in OpenStack Compute (nova) "[placement] Attempting to find allocation candidates for shared-only resources results in KeyError" [High,Triaged]16:50
mriedemnotifications meeting in #openstack-meeting-4 in 10 minutes16:50
mriedemmelwitt: yup, i did that because they run single tenant in serial16:50
mriedemmelwitt: unlike the tempest dsvm jobs16:50
mriedembut idk maybe that doesn't make sense,16:51
mriedemwe could also do a functional test in tree16:51
melwittmriedem: okay, just wanted to make sure that was the intention. I was thinking of the overhead vs the fact that a failure there might go unnoticed for some time16:51
mriedemmy goal was the change that runs each test 15 times16:51
mriedemto check for leaks16:51
*** randomhack has joined #openstack-nova16:52
mriedembut we could also do that in tree tests i reckon16:52
melwittright. I was looking at them yesterday and wasn't sure whether to +W them once I noticed they're in novaclient. so I wanted to ask you first16:52
mriedemask and ye shall receive...a meh of an answer16:53
melwitt:)16:53
*** mgiles has quit IRC16:56
*** sambetts is now known as sambetts|afk16:57
*** randomhack has quit IRC16:57
*** sree has quit IRC16:58
*** psachin has quit IRC16:59
*** ralonsoh has quit IRC17:00
*** smatzek has quit IRC17:02
*** owalsh has quit IRC17:03
*** mgiles has joined #openstack-nova17:03
*** owalsh has joined #openstack-nova17:04
*** baoli has quit IRC17:06
*** baoli has joined #openstack-nova17:06
*** lyan has joined #openstack-nova17:06
*** jistr is now known as jistr|off17:06
*** yamamoto has joined #openstack-nova17:07
*** gbarros has joined #openstack-nova17:08
*** oorgeron has joined #openstack-nova17:10
*** keerthi has quit IRC17:10
*** randomhack has joined #openstack-nova17:10
moshelejaypipes: hi17:11
*** smatzek has joined #openstack-nova17:11
jaypipesmoshele: well hello there :)17:11
*** yamamoto has quit IRC17:12
moshelejaypipes: can we do goolge hangout ? I have some question regarding the designer/vnic_type/vif/plugins17:12
jaypipesmoshele: yes, OK with me. I had asked you, jangutter and sean-k-mooney earlier today to do one.17:13
moshelejaypipes:  around https://review.openstack.org/#/c/484197/ and also https://review.openstack.org/#/c/398265/17:13
*** Daisy_ has joined #openstack-nova17:13
moshelejaypipes: ha ok I guess I missed that17:13
moshelesean-k-mooney, jangutter : can we do a call tomorrow?17:15
*** smatzek has quit IRC17:15
sean-k-mooneymoshele: sure but we are cutting it rather close if we want to have support in pike for ovs with hardware offload17:15
*** smatzek has joined #openstack-nova17:15
moshelesean-k-mooney, jaypipes: what is blocking this than https://review.openstack.org/#/c/398265/17:16
moshelesean-k-mooney: we can do the hw_veb refactor (https://review.openstack.org/#/c/484197/) in queen17:17
sean-k-mooneymoshele: it need rodolfos feature based scheduling17:17
*** Daisy_ has quit IRC17:17
moshelesean-k-mooney: I thought we agreed that it will be just limitation with working with SR-IOV. and we will fix it in queens17:19
sean-k-mooneymoshele: jaypipes  to supprot https://review.openstack.org/#/c/398265 we basically need https://review.openstack.org/#/c/449257/ and https://review.openstack.org/#/c/451777/17:19
*** kristian__ has quit IRC17:20
sean-k-mooneymoshele: i dont think we should merge https://review.openstack.org/#/c/398265 without adressing the sriov work as the port creation will change. e.g. in queens you would need to set the feature request in teh port binings so it existing vms would be broken on upgrade17:20
*** kristian__ has joined #openstack-nova17:20
*** efoley has quit IRC17:20
jaypipesmoshele: I'm more concerned about os-vif patches that are dependencies.17:21
jaypipesmoshele: since those have a hard freeze date of this Thursday17:21
*** imacdonn has quit IRC17:21
jaypipesmoshele: do we have a list of those patches?17:21
*** imacdonn has joined #openstack-nova17:21
sean-k-mooneyjaypipes: you should have an email titled [openstack-dev][os-vif] 1.6.1 release for pike. in your inbox17:21
*** kristian__ has quit IRC17:22
*** baoli has quit IRC17:22
*** gbarros has quit IRC17:22
sean-k-mooneyjaypipes: i belive the should have section are the required os-vif patches17:22
*** kristian__ has joined #openstack-nova17:22
jaypipessean-k-mooney: yes, been going through that17:22
*** baoli has joined #openstack-nova17:22
jaypipessean-k-mooney: the representor ones, yeah?17:23
*** tbachman has quit IRC17:23
sean-k-mooneyyep these Improve OVS Representor Lookup  https://review.openstack.org/#/c/484051/17:23
sean-k-mooneyAdd support for VIFPortProfileOVSRepresentor https://review.openstack.org/#/c/483921/17:23
sean-k-mooneyunplug_vf_passthrough: don't try to delete representor netdev https://review.openstack.org/#/c/478820/17:23
jaypipesgotcha17:23
sean-k-mooneyfor melonox only https://review.openstack.org/#/c/478820/ is needed. for netronome the other two are also needed17:24
*** harlowja has joined #openstack-nova17:24
*** lucasagomes is now known as lucas-afk17:25
*** kristian__ has quit IRC17:26
*** kristian__ has joined #openstack-nova17:27
*** tbachman has joined #openstack-nova17:27
moshelesean-k-mooney: sorry I don't follow, how it will break upgrade. If the limitation will be that SR-IOV mechanism driver and OVS hardware offload can't coexist in pike17:28
*** tylerder_ has joined #openstack-nova17:29
*** tylerderosagrund has quit IRC17:29
sean-k-mooneyin queens the could and existing deployment could not be upgrades as vm that have neutron ports without the feature request could get scheduled to node following upgrade that had both deployed17:30
mriedemmelwitt: ok so i think we're hunky dorey on the current list of comments in https://review.openstack.org/#/c/416521/ - are you going to rebase that today?17:30
melwittmriedem: yeah, running unit tests over it now for sanity first17:30
mriedemawesome opposum17:30
melwittthanks :)17:30
mriedemi'm going to make coffee and rebase my service/hypervisor api uuid change for the next hour17:30
gibijaypipes: I'm leaving for today. I will read back tomorrow moring17:31
openstackgerritmelanie witt proposed openstack/nova master: Make security_group_rules use check_deltas() for quota  https://review.openstack.org/47770017:31
openstackgerritmelanie witt proposed openstack/nova master: Remove 'reserved' count from used limits  https://review.openstack.org/44624217:31
openstackgerritmelanie witt proposed openstack/nova master: Make key_pairs use check_deltas() for quota  https://review.openstack.org/47769917:31
openstackgerritmelanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage  https://review.openstack.org/44624317:31
openstackgerritmelanie witt proposed openstack/nova master: Count instances to check quota  https://review.openstack.org/41652117:31
openstackgerritmelanie witt proposed openstack/nova master: Make Quotas object favor the API database  https://review.openstack.org/41094517:31
*** kristian__ has quit IRC17:31
openstackgerritmelanie witt proposed openstack/nova master: Add online migration to move quotas to API database  https://review.openstack.org/41094617:31
sean-k-mooneymoshele: basicaly this break if you ever enable the sriovnicagent and ovs with offload in the same deployment and do a livemigrate unless you carfull segratate with availablity zone and make sure you dont have a mix off both backend in the same availablity zone17:31
*** alexchadin has joined #openstack-nova17:32
*** mgiles has quit IRC17:35
*** nicolasbock has quit IRC17:35
*** mgiles has joined #openstack-nova17:36
*** nicolasbock has joined #openstack-nova17:36
*** sjmc7 has joined #openstack-nova17:38
sjmc7hi gibi. sorry, meant to bring this up in the notifications meeting but i had to step away for a bit. we were having a discussion last week about the field that the API returns as ‘status’ - do the notifications have an equivalent?17:39
*** moshele has quit IRC17:41
openstackgerritFeodor Tersin proposed openstack/nova master: Implement ScaleIO image backend  https://review.openstack.org/40744017:42
*** gjayavelu has joined #openstack-nova17:44
*** mgiles has quit IRC17:47
*** mgiles has joined #openstack-nova17:47
*** mgiles has quit IRC17:49
*** mgiles has joined #openstack-nova17:50
mriedemdansmith: can you poke through this and see if i'm at least on the correct track? https://review.openstack.org/#/c/481748/ if so then i can go ahead with clarifying some of the wording17:51
*** mgiles has quit IRC17:54
*** mgiles has joined #openstack-nova17:54
*** dtp has quit IRC17:56
*** tesseract has quit IRC17:58
*** baoli has quit IRC18:00
*** baoli has joined #openstack-nova18:01
*** alexchadin has quit IRC18:01
*** mgiles has quit IRC18:03
*** mgiles has joined #openstack-nova18:04
*** deep-book-gk_ has joined #openstack-nova18:04
*** deep-book-gk_ has left #openstack-nova18:06
*** mgiles has quit IRC18:08
*** mgiles has joined #openstack-nova18:09
openstackgerritJay Pipes proposed openstack/nova master: placement: alloc candidates only shared resources  https://review.openstack.org/48490018:09
jaypipesgibi: ^^18:09
jaypipesmriedem, dansmith, edleafe, cdent: I'd like your opinion on above please.18:10
jaypipesbauzas: you too. :)18:10
*** annegentle has joined #openstack-nova18:13
*** mgiles has quit IRC18:15
*** mgiles has joined #openstack-nova18:16
edleafejaypipes: (after quick read) - so requesting some DISK_GB will return hosts that have it local, hosts that have it via shared resources, and the shared RP themselves?18:16
edleafe(*just* DIK_GB)18:17
edleafeDISK_GB18:17
jaypipesedleafe: it will return a single allocation_request (allocating against the shared storage provider) but will include both the shared storage provider as well as the shared-with providers (the compute nodes) in the provider_summaries part of the response.18:17
dansmithmriedem: does that help at all? it's a little rambly18:18
jaypipesedleafe: and yeah, this is for when someone requests allocation candidates and specifies requested resources and those resources are only shared.18:18
dansmithmriedem: edleafe: where is the set for the flavor overrides?18:18
dansmithI assume based on the questions in there that the healing of existing instances isn't in place18:18
edleafejaypipes: what about something like DISK_GB, where the resources are both local and shared?18:19
dansmithjaypipes: did you just say that if I ask for DISK_GB that I'll get back the shared provider (i.e. a netapp) and all the compute nodes as providers that share with it?18:20
jaypipesedleafe: that's handled already and does not produce the KeyError.18:20
edleafedansmith: I'm working on that code. When the ironic driver starts up, it will handle the inventory/allocation corrections18:20
dansmithedleafe: working on meaning it's not up in any form yet?18:21
edleafejaypipes: OK, fine. Like I said, I just did a quick read18:21
edleafedansmith: righty-o18:21
jaypipesdansmith: you will get back a single allocation_request that references the shared storage provider, and the provider_summaries section of the HTTP response will contain the UUIDs of the provider that are shared with.18:21
mriedemjaypipes: questions / comments inline18:21
edleafedansmith: I just got an ironic devstack working with custom RCs this morning18:21
dansmithedleafe: okay, but the flavor overrides stuff is up right? but not yet merged?18:21
jaypipesthx all18:21
jaypipesappreciated.18:21
dansmithjaypipes: okay, that's okay then, although kinda wasteful because we don't care about the shared-with providers if they don't have anything we asked for right?18:22
edleafedansmith: merged https://review.openstack.org/#/c/473627/18:22
mriedemdansmith: i commented on that in the change too18:23
mriedemwe all had the same question :)18:23
jaypipesdansmith: they *do* have what we asked for, though. :) it's just that a different provider is sharing those requested resources with them.18:23
dansmithedleafe: cool18:23
*** kristian__ has joined #openstack-nova18:23
dansmithjaypipes: if I ask for DISK_GB only, I do not need to know the uuids of the compute nodes associated with the disk provider placement returns to me18:24
jaypipesdansmith: no? I suppose so. I just figured it was best to return that information just in case. but I can easily be persuaded otherwise I suppose :)18:25
mriedemjaypipes: dansmith: so i asked the same in the change,18:25
dansmithjaypipes: definitely no.18:25
mriedembut concluded that for nova-scheduler, this doesn't matter as we wouldn't have this happen18:25
jaypipesheh18:25
dansmithjaypipes: it's not harmful, it's just noise18:25
dansmithright, but going forward,18:25
dansmithif something else is looking to just allocate some disk resources,18:25
dansmithit's weird to get back compute nodes18:26
dansmithanyway, as long as the allocation is right, we can worry about it later18:26
mriedemi agree it's sort of weird18:26
dansmithedleafe: and what about the change to start reporting only the custom inventory out of the ironic driver?18:27
mriedemhard to write an api w/o a consumer18:27
dansmithedleafe: like, is the healing of existing instances the only thing we need to finish here?18:27
mriedemjaypipes: dansmith: for now we could omit the shared providers with the thing you asked for (the compute nodes in this case),18:27
mriedemand later, if someone wanted that, we could make it a query parameter on the api18:28
mriedem?include_friends=True18:28
jaypipesmriedem: you mean omit the *shared-with* providers?18:28
mriedemreturn the provider that has the resource you're asking for,18:28
mriedemso the shared storage pool18:28
*** kristian__ has quit IRC18:28
mriedemand omit the compute nodes related to it18:28
mriedemadd that support in later with a query parameter, just an idea18:28
mriedemin other words, does it make sense to get back provider summaries that have 0 resource listed?18:29
dansmithmeaning zero resource allocated in the allocations returned right?18:29
mriedemhttps://review.openstack.org/#/c/484900/1/nova/objects/resource_provider.py@257718:29
mriedemthe compute nodes wouldn't come back in the allocations right?18:30
mriedemonly the shared storage provider18:30
*** eharney has quit IRC18:30
jaypipesdansmith, mriedem: well, here's the thing...18:30
mriedemhttps://review.openstack.org/#/c/484900/1/nova/tests/functional/db/test_resource_provider.py@270918:30
mriedemyeah18:30
dansmithto me, if I ask for DISK_GB allocations, and I get back a couple of allocation options against a DISK_GB provider, the provider summaries should only include info about those providers, not those providers and other random ones18:30
*** eharney has joined #openstack-nova18:30
jaypipesdansmith, mriedem: I'm kinda thinking ahead and envisioning the cinder scheduler calling placement and wanting to get back information about the compute nodes that have some DISK_GB resources shared *with* them. Information including, for instance, distances between the compute nodes and the shared storage provider (via aggregate distance links)18:31
mriedemjaypipes: but we can build that in later with a microversion and query parameter18:31
jaypipesdansmith, mriedem: in that case, we would want to return the compute nodes in the provider summaries but not in the allocation requests.18:31
jaypipesmriedem: yes, for sure.18:31
dansmithbut that's like, a long time off, and is an overlap between nova and cinder,18:31
dansmithwhich the cinder scheduler may or may not be doing, right?18:32
jaypipesdansmith: ack18:32
jaypipesjust trying to explain my thinking18:32
*** sjmc7 has left #openstack-nova18:32
dansmithspecifically if the cinder scheduler is asking placement for something, I expect it's for a non-nova user, and thus compute nodes are completely irrelevant18:32
mriedemwell,18:33
mriedemnothing is irrelevant between nova and cinder b/c we're incestual :)18:33
mriedeme.g. AZs!18:33
mriedemCONF.cinder.cross_az_attach ftw18:33
mriedemi would just prefer to keep this as basic as possible for now, and build out for other use cases later18:34
dansmithyes18:34
jaypipesmriedem: totally cool with me. like I said, I was just trying to explain my thoughts about the future.18:34
mriedemack18:34
mriedemon another note, i just got this text, "the only thing i need from you on my birthday is for you to say absolutely nothing in response to what my mom is sending to me for my birthday"18:35
mriedemto which my immediate reaction was, "oh f*"18:36
mriedemgod i hope it's a PUPPY!!!18:36
dansmithhah18:36
openstackgerritMichael Bayer proposed openstack/nova master: Generalize DB conf group copying  https://review.openstack.org/48490818:36
edleafemriedem: a restraining order?18:38
jaypipesedleafe: no, that was last year.18:38
*** kristian__ has joined #openstack-nova18:38
mriedemlaura's mom is sending her a restraining order to keep them apart?18:39
mriedemthat doesn't make sense18:39
mriedemit's probably a drone with an ipad controlled by a puppy18:39
mriedemand some kind of vacuum18:39
mriedemb/c that's all the kinds of crap laura's mom sends for every event18:39
dansmithno it has to be something better than an airborne puppy-controlled vacuum camera18:40
mriedembreaking news folks: it's yet another hardwood floor cleaner18:40
mriedemi hope it plays nice with the other 3 we already have18:40
* dansmith is disappointed18:40
melwittlol floor cleaner? that doesn't sound very fun18:40
mriedemlaura's mom (1) love gadgets (2) shopping on qvc and (3) cleaning18:41
mriedem*love's18:41
mriedemloves?18:41
mriedemso we have a pile of floor cleaners and tablets all over the house18:41
dansmithI was thinking more along the lines of a "how to get excited about having a second child" book or something18:41
mriedemthat ship sailed a couple of years ago18:42
melwittlol18:42
openstackgerritJay Pipes proposed openstack/nova master: placement: alloc candidates only shared resources  https://review.openstack.org/48490018:42
jaypipesmriedem, dansmith, edleafe: ok dokey ^^18:42
mriedemwhen amazon makes a life-like enough "my buddy" cyborg child, we'll probably have one18:42
mriedemand then i'll make it clean my floors18:42
*** baoli has quit IRC18:42
melwittmy buddy, that's a blast from the past18:43
mriedemdon't forget the hulk hogan my buddy18:43
melwittyikes18:43
*** baoli has joined #openstack-nova18:45
dansmithedleafe: do you have a game plan for the heal process?18:49
dansmithI was thinking we had to convert the instances from ram/cpu/disk, but really we just need to add the source class to them I think for pike18:50
dansmithedleafe: which we might be able to do in an online migration instead of in the driver, now that I think of it18:50
dansmithedleafe: but we also need the piece to make the resource tracker report allocations for the custom resources an instance has in its flavor, which I don't think is done yet18:50
dansmithjaypipes: right?18:51
mriedemcorrect18:51
jaypipescorrect18:51
mriedemthe data migration and posting allocations isn't done18:51
edleafedansmith: doing it in the driver's init_host has a lot of advantages18:52
dansmithah, the reason is because we don't easily know ironic vs. libirt instances in manage18:52
dansmiththat's why18:52
dansmithedleafe: we just need to be good about it (i.e. spawn a thread)18:53
edleafemostly that it runs once, and nova can't accidentally allocate against it until the healing is run18:53
dansmithedleafe: well, I don't think we can block compute startup entirely right?18:53
edleafedansmith: well, we can see about improving it once I get it working18:53
dansmithand we don't need to until we convert the requests to be custom-only, which is queens I think18:53
edleafedansmith: this has to be done in Pike so we can remove it in Queens and go to custom-only18:54
dansmithedleafe: the migration does yeah, so we can do it async18:54
openstackgerritBaodong (Robert) Li proposed openstack/nova-specs master: Expose vlan trunking in metadata/configdrive  https://review.openstack.org/47181518:54
dansmithso nobody is working on reporting allocations for custom resource type overrides in the flavors, right?18:55
edleafedansmith: the understanding was that operators would add the extra_specs stuff before starting Pike18:56
dansmithedleafe: to the base flavors?18:56
mriedemedleafe: they shouldn't do any of that,18:56
edleafedansmith: the nodes should already be populated with a resource_class in Ocata18:56
mriedemuntil this other stuff is done18:56
dansmithright18:57
dansmiththey can't18:57
dansmiththe spec says that in queens, afaik18:57
mriedemwe could likely build something into nova-status in queens18:57
edleafemriedem: well, yeah - that's why this is going into Pike18:57
dansmithif we report inventory in pike, and allocations for existing, then everything is good to cut over in queens18:57
mriedemyou could query all ironic compute nodes in all cells and get the instances from them and check to see if their flavors have been migrated18:57
mriedemanyway i think i put some words about the operator thing in the spec amendment18:58
dansmithso again,18:58
dansmithnobody is working on the reporting allocations part right?18:58
mriedemcorrectamundo18:59
dansmithokay I think that'll be easy so let me work on that a bit18:59
edleafedansmith: not sure what you mean by 'reporting allcations'18:59
*** adisky__ has quit IRC18:59
dansmithedleafe: healing the flavors on the instances to have the resource override is one thing,19:00
mriedemedleafe: let me find that from the spec amendment19:00
dansmithbut we have to actually report those values as consumed in placement when we're making/updating the instance allocations19:00
dansmithfrom resource tracker19:00
* edleafe is on a call right now19:00
mriedemedleafe: https://review.openstack.org/#/c/481748/1/specs/pike/approved/custom-resource-classes-in-flavors.rst@10819:00
edleafemriedem: ok, that19:01
edleafemriedem: I am adding similar in the flavor migration stuff19:01
*** Daisy_ has joined #openstack-nova19:01
edleafesetting the allocation for the custom ironic RC19:02
dansmithmigrating the instance flavor should make the allocation heal in the RT when that runs19:02
dansmithedleafe: don't make those calls from driver init19:02
dansmithotherwise we have it in two places19:02
*** mgiles has quit IRC19:04
*** Daisy_ has quit IRC19:06
*** Apoorva_ has quit IRC19:16
*** gyee has joined #openstack-nova19:17
*** lpetrut__ has joined #openstack-nova19:24
*** vishwanathj has joined #openstack-nova19:26
*** zzzeek has quit IRC19:29
*** crushil has quit IRC19:29
*** zzzeek has joined #openstack-nova19:30
*** jdillaman has joined #openstack-nova19:32
*** vishwanathj has quit IRC19:34
openstackgerritMerged openstack/os-vif master: unplug_vf_passthrough: don't try to delete representor netdev  https://review.openstack.org/47882019:35
*** vishwanathj has joined #openstack-nova19:35
*** crushil has joined #openstack-nova19:37
*** prometheanfire has left #openstack-nova19:37
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements  https://review.openstack.org/48421819:40
*** gjayavelu has quit IRC19:44
*** awaugama has quit IRC19:45
*** jangutter_ has joined #openstack-nova19:46
*** jangutter has quit IRC19:47
*** jangutter_ is now known as jangutter19:47
*** jangutter_ has joined #openstack-nova19:47
*** vishwanathj has quit IRC19:48
*** vishwanathj has joined #openstack-nova19:49
*** vishwanathj has quit IRC19:50
*** vishwanathj has joined #openstack-nova19:51
*** nicolasbock has quit IRC19:53
openstackgerritJackie Truong proposed openstack/nova master: [WIP] Add trusted certificates to InstanceExtras  https://review.openstack.org/45771119:55
openstackgerritJackie Truong proposed openstack/nova master: [WIP] Add trusted certificates to InstanceExtras  https://review.openstack.org/45771119:57
*** dave-mccowan has joined #openstack-nova19:58
oorgeronHi folks, I'm working on a patch to enable nova to use mysql cluster. In tests, I'm running into errors that seem to indicate that sqlite is being used in the sqlalchemy migrations. Michael Bayer said that this has happened before in the nova gate, anyone know how to fix that? https://review.openstack.org/#/c/446643/20:02
*** smatzek has quit IRC20:06
melwittoorgeron: the unit tests intentionally use sqlite20:06
melwittand functional tests20:06
*** randomhack has quit IRC20:08
melwittthat said, I don't know how you could get things working with your change. probably need to ask someone like sdague20:08
*** tylerder_ has quit IRC20:08
*** pchavva has quit IRC20:09
*** armax has quit IRC20:10
*** gjayavelu has joined #openstack-nova20:12
oorgeronmelwitt: I'll reach out to sdague.20:12
openstackgerritDan Smith proposed openstack/nova master: Consider instance flavor resource overrides in allocations  https://review.openstack.org/48493520:13
dansmithmriedem: edleafe ^.. I still need to figure out what gabbit is failing on one of the functional tests20:13
mriedemoorgeron: you can run mysql and postgresql opportunistic tests locally if you do some setup20:13
sdagueoorgeron: yeh, everything there looks like unit testable failures because of an assumption about an index20:14
mriedemthere should be a script in nova/tools to set this stuff up20:14
*** tylerderosagrund has joined #openstack-nova20:14
sdaguemriedem: it's not even that20:14
mriedemhttps://github.com/openstack/nova/blob/master/tools/test-setup.sh20:15
sdaguehttp://logs.openstack.org/43/446643/6/check/gate-nova-python27-ubuntu-xenial/6e79a39/console.html.gz#_2017-07-14_19_47_37_19708620:15
sdaguethese are just run of the mill, non mysql / pg tests failing20:15
oorgeronsdague: yeah the test is failing with a sqlite error with dropping an index. The scripts work fine with innodb and ndb as the database.20:16
sdagueoorgeron: right, they have to work with sqlite as well20:16
*** gjayavelu has quit IRC20:16
sdaguethat's our unit testing environment20:16
mriedemyeah...you can't just do stuff like this https://review.openstack.org/#/c/446643/6/nova/db/sqlalchemy/migrate_repo/versions/249_remove_duplicate_index.py20:16
*** jamesdenton has joined #openstack-nova20:17
*** Dinesh_Bhor has quit IRC20:17
*** dave-mccowan has quit IRC20:17
mriedemand this scares me https://review.openstack.org/#/c/446643/6/nova/db/sqlalchemy/models.py20:17
oorgeronmriedmen: A lot of the other services are just doing tests against mysql and maybe postgresql. I could use another method to drop and add  the foreignkey and not use the migrate method for doing that.20:18
mriedemoorgeron: everyone is using sqlite for unit tests20:18
mriedemwe test with mysql in the devstack jobs in the gate20:18
mriedemfor integration testing20:18
mriedemthe unit tests also run with the opportunistic mysql and postgresql backends in the gate20:19
oorgeronmriedem: I can drop foreign keys like this in other services and the tests pass.20:19
mriedembut locally you have to set that all up optionally20:19
*** jangutter has quit IRC20:19
mriedemoorgeron: that doesn't make it right20:19
mriedem"hey look i deleted a bunch of code and nothing blew up!"20:19
mriedemthat probably just means they aren't doing testing20:19
mriedemthe model changes here worry me that the schema will change for other backends, like postgresql20:20
oorgeronmriedem: first off, when you drop an index, you supposed to remove the foreign keys, if we're talking about the right order of sql steps. Innodb just happens to be very forgiving on that, doesn't make it right either.20:20
* mriedem summons jaypipes for a sql battle20:21
oorgeronmriedem: also, if you look at that, it adds the key back after the index is dropped, which is the right way to do this in sql20:21
* jaypipes stumbles in20:21
jaypipesmriedem: how may I be of assistance?20:21
oorgeronmriedem: But I'll try a dumber wait to drop and recreate the foreign key, instead of using the migrate method here.20:21
oorgeronway20:21
oorgeronmriedem: the changes to model will only apply when ndb is turned on. The ndb function calls there do the right thing automatically.20:23
mriedemso in https://review.openstack.org/#/c/446643/6/nova/db/sqlalchemy/migrate_repo/versions/249_remove_duplicate_index.py you'd have to handle the index not existing before trying to drop it20:24
*** dave-mccowan has joined #openstack-nova20:24
mriedemjaypipes: half joking20:25
mriedemi unsummon you20:25
mriedemand banish you back to placement land20:25
mriedem*placement island20:25
oorgeronmriedem: besides, 292_drop_nova_volumes_tables.py uses the same method and is not part of my patch :)20:26
*** jamesden_ has joined #openstack-nova20:26
*** dave-mccowan has quit IRC20:27
*** jamesdenton has quit IRC20:28
openstackgerritJan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor  https://review.openstack.org/48392120:28
* jaypipes vanishes in puff of smoke20:28
openstackgerritJackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils  https://review.openstack.org/47994920:30
*** esberglu has quit IRC20:31
*** vishwanathj has quit IRC20:32
*** vishwanathj has joined #openstack-nova20:33
*** kristian__ has quit IRC20:34
*** kristian__ has joined #openstack-nova20:34
*** lpetrut__ has quit IRC20:38
*** vishwanathj has quit IRC20:39
*** vishwanathj has joined #openstack-nova20:40
*** vishwanathj has quit IRC20:40
*** vishwanathj has joined #openstack-nova20:41
*** esberglu has joined #openstack-nova20:44
*** esberglu has quit IRC20:44
*** esberglu has joined #openstack-nova20:44
*** vishwanathj has quit IRC20:45
*** vishwanathj has joined #openstack-nova20:46
*** vishwanathj has quit IRC20:47
*** vishwanathj has joined #openstack-nova20:47
*** jamesden_ has quit IRC20:47
oorgeronsdague: is there a way to run the tests just against sqlite?20:49
*** jamesdenton has joined #openstack-nova20:49
*** armax has joined #openstack-nova20:49
*** Daisy_ has joined #openstack-nova20:50
openstackgerritJackie Truong proposed openstack/nova master: Add trusted certificates to InstanceExtras  https://review.openstack.org/45771120:54
*** crushil has quit IRC20:55
*** Daisy_ has quit IRC20:55
openstackgerritJan Gutter proposed openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor  https://review.openstack.org/48392120:57
*** rcernin has quit IRC20:57
*** tylerder_ has joined #openstack-nova20:59
*** tylerderosagrund has quit IRC21:02
*** lucasxu has quit IRC21:02
*** sdague has quit IRC21:02
*** tylerder_ has quit IRC21:03
openstackgerritEd Leafe proposed openstack/nova master: WIP - Migrate Ironic Flavors  https://review.openstack.org/48494921:06
edleafedansmith: ^^ Strawman code that has the inventory/allocation changes21:06
*** lucasxu has joined #openstack-nova21:06
edleafedansmith: I had already written that before you said not to :)21:06
mriedemdansmith: here it comes21:08
mriedemprepare21:08
openstackgerritMatt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API  https://review.openstack.org/47189921:08
openstackgerritMatt Riedemann proposed openstack/nova master: Use uuid for id in os-services API  https://review.openstack.org/46428021:08
*** MVenesio has quit IRC21:09
dansmithedleafe: remember how I asked a few times if anyone was working on that?21:14
*** kbaegis has joined #openstack-nova21:16
*** Apoorva has joined #openstack-nova21:16
*** kristian__ has quit IRC21:17
*** lucasxu has quit IRC21:17
edleafedansmith: remember how I said that I was adding that code to the ironic driver?21:17
*** kristian__ has joined #openstack-nova21:18
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix max_version for deprecated os-quota-class-sets parameters  https://review.openstack.org/48495221:18
mriedemalex_xu: gmann: ^ we got confused21:18
*** kbaegis1 has quit IRC21:19
*** annegentle has quit IRC21:22
*** kristian__ has quit IRC21:22
openstackgerritOctave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664321:23
mriedemedleafe: dansmith: now now now, you're BOTH wrong! :)21:24
mriedemedleafe: you shouldn't be doing the placement stuff in the init_host21:24
mriedem-1 for that21:24
mriedemdan's change builds on top of yours21:24
mriedemedleafe: just need you to update the existing instance flavors on init_host,21:24
mriedemupdate_available_resource will pick up on that (via dan's change) and adjust the allocations21:24
mriedemautomagically21:25
openstackgerritOctave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664321:25
*** rajathag_ has quit IRC21:26
*** jheroux has quit IRC21:28
openstackgerritMatt Riedemann proposed openstack/nova master: placement: alloc candidates only shared resources  https://review.openstack.org/48490021:29
mriedemjaypipes: +221:30
*** thorst has quit IRC21:30
jaypipesmriedem: danke21:30
mriedemkein Problem21:30
openstackgerritJackie Truong proposed openstack/nova master: Implement certificate_utils  https://review.openstack.org/47994921:31
*** baoli has quit IRC21:31
openstackgerritJackie Truong proposed openstack/nova master: Implement certificate_utils  https://review.openstack.org/47994921:32
edleafemriedem: like I said, I had most of that done before our little conversation earlier21:34
edleafeI just pushed it before I gut it :)21:35
*** annegentle has joined #openstack-nova21:39
openstackgerritEd Leafe proposed openstack/nova master: WIP - Migrate Ironic Flavors  https://review.openstack.org/48494921:39
openstackgerritMatt Riedemann proposed openstack/nova master: api-ref: fix description of service_disable_reason  https://review.openstack.org/48495521:39
edleafedansmith: mriedem: rebased it on dansmith's change and deleted the allocation stuff21:39
openstackgerritRick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API  https://review.openstack.org/44928821:39
edleafestill needs tests, though21:39
edleafebut they will have to wait until tomorrow21:39
mriedemedleafe: ok, that's the wrong order really, but ok21:39
*** eharney has quit IRC21:40
*** crushil has joined #openstack-nova21:40
edleafemriedem: no, because without his you could get errors21:40
edleafewith mine21:40
edleafewith his first nothing will happen until mine21:40
mriedemi don't understand how dan's change should impact yours21:41
mriedemthey could be done in parallel actually21:41
mriedembut dan's builds on yours21:41
edleafewell, maybe it only throws errors with the init_host updating inventory21:41
*** thorst has joined #openstack-nova21:42
edleafedidn't get to test that scenario21:42
*** Daisy_ has joined #openstack-nova21:42
melwittdansmith: I'm looking at the fleetify patch to figure out how to build upon it for per cell console proxies. it looks like it sets up only one cell? https://review.openstack.org/#/c/47755621:42
openstackgerritRick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API  https://review.openstack.org/44928821:43
mriedemmelwitt: you have to look at the multinode job21:43
mriedemhttp://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/21:43
melwittmriedem: I did, I only see super conductor and one conductor process and NOVA_NUM_CELLS is 1?21:44
*** thorst has quit IRC21:44
mriedemhmm http://logs.openstack.org/56/477556/5/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/aed9486/logs/devstack-gate-discover-hosts.txt.gz21:44
mriedemyeah i see 2 computes, 2 cell mappings, but really only 1 cell21:44
dansmithmelwitt: it treats cell0 as the main cell by pointing everything to it, so cell1 is the different cell,21:45
dansmithwhich is where the computes are21:45
melwittoh, okay21:45
melwittthanks21:45
dansmiththe plan was to make the second node create a second cell in some configurations to validate that cross-cell migrations don't happen and stuff,21:45
dansmithbut there is more devstack and d-g work to do there,21:45
*** jmlowe has quit IRC21:46
dansmithbut have had a lot of time waiting for this to get merged yaknow21:46
mriedemit did get merged once remember? :)21:46
melwittheh21:46
dansmitheven when it merged a couple weeks ago I figured we were a little late in the cycle to be splitting that out and dealing with big config changes21:46
dansmithheh yes for a hot minute21:46
*** Daisy_ has quit IRC21:47
dansmithwhen we do the two computes in two cells thing we also have to make sure we end up with a two-computes-one-cell job to keep testing migrations and stuff21:47
dansmithit's a whole thing21:47
dansmithbut, for your console thing I think the current config is going to test what you want21:47
dansmithbecause if you don't switch cells, you'll hit cell0 and find nothing21:47
dansmithmriedem: I don't think there's any strict ordering required between ed's patch and mine21:48
mriedemdansmith: there isn't a strict ordering no21:48
openstackgerritMatt Riedemann proposed openstack/nova master: Do not mention that tags are case sensitive in docs  https://review.openstack.org/48495721:48
mriedemcould be totally parallel21:48
dansmithmine is fine in isolation,21:48
dansmithmriedem: mine without his would work for all new instances21:48
edleafedansmith: when I tested the first version (with the allocation stuff in init_host) it would throw errors in the periodic update until I rebased on yours.21:50
*** MVenesio has joined #openstack-nova21:50
*** MVenesio has quit IRC21:50
edleafedansmith: now that I removed the allocations, it probably doesn't matter21:50
dansmith"when done properly, ordering doesn't matter"21:50
dansmithI should have sai21:50
dansmith*said21:50
edleafe"properly"21:51
edleafeheh21:51
*** jamesden_ has joined #openstack-nova21:51
*** jamesdenton has quit IRC21:52
*** gouthamr has quit IRC21:53
*** gbarros has joined #openstack-nova21:57
*** burt has quit IRC21:57
*** crushil has quit IRC21:59
*** tylerderosagrund has joined #openstack-nova22:00
*** tylerderosagrund has quit IRC22:05
melwittlost power, pasting what I wrote:22:06
melwittyeah, I'll have to think about it more. currently confused22:06
melwittconsole proxies currently run at the top, now we'll run them per cell. the initial request for a console will generate the auth in the cell based on the instance, then hand out the proxy url from the cell to the user22:06
melwittthen the user should connect directly to the proxy in the cell22:06
melwittI was trying to think what could possibly go wrong there or how to test it with the multicell devstack22:06
*** armax_ has joined #openstack-nova22:07
*** armax has quit IRC22:07
*** armax_ is now known as armax22:07
*** tbachman has quit IRC22:08
openstackgerritDan Smith proposed openstack/nova master: Consider instance flavor resource overrides in allocations  https://review.openstack.org/48493522:10
dansmithmelwitt: I'm not sure what you mean, but the proxy should start with the nova-cell1 config, which will point it at the real cell, where the compute is22:11
*** avolkov has quit IRC22:11
dansmithmelwitt: which will mean that the api will find it on that rpc bus, and not the api-level one22:12
*** jamesden_ has quit IRC22:12
melwittdansmith: I'm not sure what I mean either. I guess just trying to think what needs to be different. so currently the proxies are started with the api cell conf and have settings in the regular nova conf22:14
dansmithum22:14
dansmithwat22:14
dansmithyou mean they use the api db part of the used-to-be-unified config?22:14
melwittlike this? https://review.openstack.org/#/c/477556/5/lib/nova@88522:15
*** tbachman has joined #openstack-nova22:15
melwitt$api_cell_conf which is just $NOVA_CONF22:16
dansmithoh I see, that's the api conf if cellsv1 is in place right?22:16
*** crushil has joined #openstack-nova22:16
dansmithbut normally that's the same config as everything else, for non-cellsv122:16
melwittyeah, think that's why it's named that22:17
melwitt(as I keep looking at this)22:17
melwittyeah22:17
*** crushil has quit IRC22:17
*** crushil has joined #openstack-nova22:18
*** Apoorva_ has joined #openstack-nova22:19
*** gyee has quit IRC22:20
*** acormier has quit IRC22:20
*** acormier has joined #openstack-nova22:20
*** Apoorva has quit IRC22:21
*** kylek3h has quit IRC22:23
*** crushil has quit IRC22:24
*** gouthamr has joined #openstack-nova22:29
*** felipemonteiro__ has quit IRC22:31
*** felipemonteiro__ has joined #openstack-nova22:33
*** acormier has quit IRC22:34
*** kristian__ has joined #openstack-nova22:37
*** tbachman has quit IRC22:37
*** itlinux has quit IRC22:37
*** felipemonteiro__ has quit IRC22:38
*** gyee has joined #openstack-nova22:38
*** kristian__ has quit IRC22:41
*** tbachman has joined #openstack-nova22:43
*** edmondsw has quit IRC22:47
*** kbaegis has quit IRC22:52
openstackgerritMerged openstack/nova master: Use plain routes list for os-security-groups instead of stevedore  https://review.openstack.org/47890123:00
*** jwcroppe has quit IRC23:00
*** jwcroppe has joined #openstack-nova23:01
*** gcb has quit IRC23:02
*** jwcroppe_ has joined #openstack-nova23:03
openstackgerritMerged openstack/nova master: Use plain routes list for server-security-groups instead of stevedore  https://review.openstack.org/47890223:04
*** thorst has joined #openstack-nova23:04
*** jwcroppe has quit IRC23:05
*** lyan has quit IRC23:05
*** chyka has quit IRC23:06
*** thorst has quit IRC23:06
*** chyka has joined #openstack-nova23:06
*** marst has quit IRC23:08
*** annegentle has quit IRC23:10
*** chyka has quit IRC23:11
*** oorgeron has left #openstack-nova23:13
*** baoli has joined #openstack-nova23:20
*** baoli has quit IRC23:24
*** catintheroof has quit IRC23:25
*** kbaegis has joined #openstack-nova23:28
*** kbaegis has quit IRC23:28
*** dave-mccowan has joined #openstack-nova23:29
*** annegentle has joined #openstack-nova23:29
*** kbaegis has joined #openstack-nova23:29
*** sree_ has joined #openstack-nova23:33
*** sree_ is now known as Guest6153823:34
*** annegentle has quit IRC23:34
*** esberglu has quit IRC23:36
*** dave-mcc_ has joined #openstack-nova23:37
*** Guest61538 has quit IRC23:38
*** dave-mccowan has quit IRC23:38
*** hongbin has quit IRC23:39
*** jaypipes has quit IRC23:40
*** mtanino__ has quit IRC23:41
*** mtanino_ has joined #openstack-nova23:41
mriedemuh oh http://logs.openstack.org/99/471899/25/check/gate-grenade-dsvm-neutron-ubuntu-xenial/3f9a9e3/logs/new/screen-n-cond.txt.gz?level=TRACE23:42
mriedemsomeone better change the sheets23:44
*** dave-mcc_ has quit IRC23:45
*** jwcroppe_ has quit IRC23:45
mriedemhttps://bugs.launchpad.net/nova/+bug/170514123:46
openstackLaunchpad bug 1705141 in OpenStack Compute (nova) "select_destinations reults in "TypeError: 'NoneType' object is not iterable" randomly" [High,Confirmed]23:46
*** mtanino_ has quit IRC23:46
*** annegentle has joined #openstack-nova23:47
*** takashin has joined #openstack-nova23:48
openstackgerritOctave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova  https://review.openstack.org/44664323:49
*** itlinux has joined #openstack-nova23:54
*** lucasxu has joined #openstack-nova23:55
*** yingjun has joined #openstack-nova23:57

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