Wednesday, 2017-01-18

*** Swami has joined #openstack-nova00:01
*** dtp has quit IRC00:03
*** Swami_ has joined #openstack-nova00:05
*** harlowja has joined #openstack-nova00:06
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120900:06
*** Swami has quit IRC00:08
*** hongbin has quit IRC00:12
*** jdurgin has quit IRC00:14
*** takashin has joined #openstack-nova00:15
*** slaweq has joined #openstack-nova00:17
*** unicell has quit IRC00:17
*** raunak has quit IRC00:19
*** slaweq__ has quit IRC00:20
*** raunak has joined #openstack-nova00:21
*** jose-phillips has quit IRC00:28
*** jose-phillips has joined #openstack-nova00:30
*** mlavalle has quit IRC00:30
*** jdurgin has joined #openstack-nova00:31
*** jaosorior has quit IRC00:31
*** thorst has joined #openstack-nova00:33
openstackgerritJoshua Harlow proposed openstack/nova: Handle oslo.serialization type error and binascii error  https://review.openstack.org/42148200:34
*** thorst has quit IRC00:34
*** jwcroppe has quit IRC00:41
*** unicell has joined #openstack-nova00:44
*** hoangcx has joined #openstack-nova00:52
*** jwcroppe has joined #openstack-nova00:56
*** jwcroppe has quit IRC00:56
*** jwcroppe has joined #openstack-nova00:56
*** jose-phillips has quit IRC01:00
*** dimtruck is now known as zz_dimtruck01:02
*** tovin07 has joined #openstack-nova01:05
*** phuongnh has joined #openstack-nova01:06
*** tuanluong has joined #openstack-nova01:11
*** jerrygb has quit IRC01:11
*** jerrygb has joined #openstack-nova01:14
*** mtanino has quit IRC01:14
*** raunak has quit IRC01:15
*** esberglu has joined #openstack-nova01:16
*** huanxie has quit IRC01:16
*** huanxie has joined #openstack-nova01:17
*** raunak has joined #openstack-nova01:17
*** tovin07 has quit IRC01:19
*** kevinz has joined #openstack-nova01:20
*** esberglu has quit IRC01:21
*** salv-orlando has joined #openstack-nova01:22
*** bswartz has quit IRC01:24
*** ducttape_ has joined #openstack-nova01:25
*** dave-mccowan has quit IRC01:26
*** salv-orlando has quit IRC01:27
*** zayro has joined #openstack-nova01:27
*** huanxie has quit IRC01:27
*** tojuvone has quit IRC01:27
*** Dinesh_Bhor has quit IRC01:27
*** stevemar has quit IRC01:27
*** brault has quit IRC01:27
*** huanxie has joined #openstack-nova01:29
*** tojuvone has joined #openstack-nova01:29
*** Dinesh_Bhor has joined #openstack-nova01:29
*** stevemar has joined #openstack-nova01:29
*** brault has joined #openstack-nova01:29
*** gokhan has quit IRC01:31
*** sacharya has quit IRC01:35
*** thorst has joined #openstack-nova01:35
*** hfu has joined #openstack-nova01:35
*** jhesketh has quit IRC01:36
*** jhesketh has joined #openstack-nova01:36
openstackgerrithuangtianhua proposed openstack/python-novaclient: Make _console() public  https://review.openstack.org/41436401:37
*** mriedem1 has joined #openstack-nova01:39
*** thorst has quit IRC01:40
*** yuntongjin has joined #openstack-nova01:40
*** mriedem has quit IRC01:41
*** raunak has quit IRC01:41
*** zz_dimtruck is now known as dimtruck01:43
openstackgerritHuan Xie proposed openstack/nova: XenAPI: Fix vif plug problem during VM rescue/unrescue  https://review.openstack.org/41346901:48
*** Sukhdev_ has quit IRC01:48
*** rfolco has joined #openstack-nova01:49
*** rfolco has quit IRC01:49
*** rfolco has joined #openstack-nova01:50
*** thorst has joined #openstack-nova01:50
*** thorst has quit IRC01:50
*** rfolco has quit IRC01:50
*** browne1 has joined #openstack-nova01:52
*** yamahata has quit IRC01:52
*** sree has joined #openstack-nova01:54
*** diga has quit IRC01:54
*** yamahata has joined #openstack-nova01:55
*** rfolco has joined #openstack-nova01:55
*** rfolco has quit IRC01:55
mriedem1jroll: i think we have an ironic job in the nova experimental queue that's running neutron with some tenant networking glitz,01:55
mriedem1i was wondering if that would be able to run the attach/detach interfaces test in tempest for https://review.openstack.org/#/c/364413/22/nova/virt/ironic/driver.py01:55
*** mriedem1 is now known as mriedem01:55
mriedemi can poke into that later01:56
*** browne has quit IRC01:56
openstackgerritWang Qing wu proposed openstack/nova: Introduce base PowerVM driver  https://review.openstack.org/39128801:56
*** browne1 has quit IRC01:56
*** sree has quit IRC01:58
*** sree has joined #openstack-nova02:00
*** sree has quit IRC02:00
*** Apoorva has quit IRC02:04
*** sree has joined #openstack-nova02:09
*** unicell has quit IRC02:10
*** mingyu has joined #openstack-nova02:11
*** jamesdenton has joined #openstack-nova02:11
*** houming has joined #openstack-nova02:14
*** jamesdenton has quit IRC02:15
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120902:16
*** sree has quit IRC02:21
alex_xuoomichi: johnthetubaguy the whole series looks like close https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-whitelist-for-server-list-filter-sort-parameters  at least LGTM now02:21
*** jamesdenton has joined #openstack-nova02:22
*** ducttape_ has quit IRC02:24
*** nic has quit IRC02:27
*** ducttape_ has joined #openstack-nova02:29
*** thorst has joined #openstack-nova02:30
*** thorst has quit IRC02:30
*** browne has joined #openstack-nova02:30
*** sacharya has joined #openstack-nova02:36
*** harlowja has quit IRC02:36
*** raunak has joined #openstack-nova02:37
*** browne has quit IRC02:37
*** davidlenwell_ has joined #openstack-nova02:38
*** sacharya has quit IRC02:40
*** mingyu has quit IRC02:44
*** Apoorva has joined #openstack-nova02:44
openstackgerritZhenyu Zheng proposed openstack/nova: Strict pattern match query parameters  https://review.openstack.org/42049402:47
openstackgerritZhenyu Zheng proposed openstack/nova: Strict pattern match query parameters  https://review.openstack.org/42049402:50
*** mingyu has joined #openstack-nova02:50
openstackgerritMatt Riedemann proposed openstack/nova: DNM: Test interface attahc/detach for Ironic  https://review.openstack.org/42163802:52
mriedemjroll: fyi ^02:52
*** sacharya has joined #openstack-nova02:52
*** gcb has joined #openstack-nova02:54
*** hongbin has joined #openstack-nova02:54
*** sacharya has quit IRC02:56
openstackgerritHuan Xie proposed openstack/nova: XenAPI: Fix vif plug problem during VM rescue/unrescue  https://review.openstack.org/41346902:56
*** takashin has left #openstack-nova02:56
*** thorst has joined #openstack-nova02:56
*** thorst has quit IRC02:56
*** sacharya has joined #openstack-nova02:56
*** mingyu has quit IRC02:57
openstackgerritYoshihiko Atsumi proposed openstack/nova: Add revert resized server functional negative tests  https://review.openstack.org/42163902:57
huanxieHi jaypipes I've added a new unit test to cover all scenario in my VM rescue/unrescue patch https://review.openstack.org/413469, could you help review again, thanks a lot02:58
*** mingyu has joined #openstack-nova02:58
*** carthaca_ has quit IRC02:58
jaypipeshuanxie: you're up late :)02:58
jaypipeshuanxie: +2 from me.02:59
huanxiehaha :)02:59
*** Gorian has quit IRC02:59
*** carthaca_ has joined #openstack-nova02:59
huanxieThanks02:59
*** Gorian- has joined #openstack-nova02:59
jaypipesnp, thx for your patience!02:59
*** cfriesen_ has joined #openstack-nova03:00
*** cfriesen has quit IRC03:00
*** mingyu has quit IRC03:00
huanxiejaypipes: your comments are reasonable, I should fix it:)03:00
*** mingyu has joined #openstack-nova03:00
*** mingyu has quit IRC03:01
*** sacharya has quit IRC03:01
*** jschwarz has quit IRC03:01
*** esberglu has joined #openstack-nova03:04
*** davidlenwell_ has quit IRC03:06
*** jschwarz has joined #openstack-nova03:06
*** esberglu has quit IRC03:08
*** tommylikehu has joined #openstack-nova03:11
*** ducttape_ has quit IRC03:15
*** jamesdenton has quit IRC03:16
*** dave-mccowan has joined #openstack-nova03:18
*** Gorian- has quit IRC03:19
*** Gorian has joined #openstack-nova03:19
*** gongysh has joined #openstack-nova03:20
EmilienMmriedem: not 100% sure but I think I found another problem with placement api, where nova compute doesn't refresh the resource provider record correctly. It used to work a few days ago but not anymroe03:23
EmilienMhttp://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_45_52_92003:23
*** baoli has joined #openstack-nova03:23
*** dave-mccowan has quit IRC03:23
*** salv-orlando has joined #openstack-nova03:23
*** yamahata has quit IRC03:26
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable virt.test_virt_drivers.AbstractDriverTestCase on Python 3  https://review.openstack.org/41441903:27
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port compute.test_user_data.ServersControllerCreateTest to Python 3  https://review.openstack.org/40643503:27
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable more virt.xenapi tests on Python 3  https://review.openstack.org/41442103:27
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Enable virt.vmwareapi test cases on Python  https://review.openstack.org/41442003:27
*** thorst has joined #openstack-nova03:27
*** amotoki has quit IRC03:27
*** gouthamr has quit IRC03:28
*** salv-orlando has quit IRC03:28
*** baoli has quit IRC03:29
*** thorst has quit IRC03:33
*** tommylikehu1 has joined #openstack-nova03:33
*** sacharya has joined #openstack-nova03:33
*** sheel has joined #openstack-nova03:34
*** sree has joined #openstack-nova03:36
*** tommylikehu has quit IRC03:36
*** tommylikehu1 is now known as tommylikehu03:36
*** gcb has quit IRC03:38
*** gcb has joined #openstack-nova03:39
*** baoli has joined #openstack-nova03:39
*** yuntongjin has quit IRC03:40
*** amotoki has joined #openstack-nova03:42
*** sudipto_ has joined #openstack-nova03:50
*** sudipto has joined #openstack-nova03:50
*** Swami__ has joined #openstack-nova03:50
*** amotoki_ has joined #openstack-nova03:51
*** baoli has quit IRC03:52
*** Swami_ has quit IRC03:53
*** Swami has joined #openstack-nova03:53
*** amotoki has quit IRC03:54
*** Swami__ has quit IRC03:56
*** tovin07 has joined #openstack-nova03:57
*** esberglu has joined #openstack-nova03:59
*** mdnadeem has joined #openstack-nova04:01
*** links has joined #openstack-nova04:02
*** esberglu has quit IRC04:03
*** tovin07 has quit IRC04:04
*** xinliang has quit IRC04:09
*** xinliang has joined #openstack-nova04:09
*** amotoki_ has quit IRC04:11
*** psachin has joined #openstack-nova04:14
*** sudipto has quit IRC04:14
*** sudipto_ has quit IRC04:14
*** raunak has quit IRC04:15
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5)  https://review.openstack.org/33592104:15
*** nicolasbock has quit IRC04:17
*** raunak has joined #openstack-nova04:17
*** ijw has quit IRC04:19
*** pbandark has joined #openstack-nova04:23
*** amotoki has joined #openstack-nova04:25
*** tlian has quit IRC04:25
*** sree has quit IRC04:25
openstackgerritHieu LE proposed openstack/nova: Completely remove mox from objects/test_instance.py  https://review.openstack.org/34346504:28
*** yuntongjin has joined #openstack-nova04:29
*** yuntongjin has quit IRC04:32
*** Sukhdev has joined #openstack-nova04:34
*** yellow has quit IRC04:35
*** woodster_ has quit IRC04:35
*** sudipto_ has joined #openstack-nova04:35
*** sudipto has joined #openstack-nova04:35
*** udesale has joined #openstack-nova04:37
*** diga has joined #openstack-nova04:40
*** sacharya has quit IRC04:42
*** sacharya has joined #openstack-nova04:43
*** amotoki has quit IRC04:44
*** jwcroppe has quit IRC04:45
*** sacharya has quit IRC04:47
*** Apoorva has quit IRC04:51
*** ijw has joined #openstack-nova04:56
*** Swami has quit IRC04:59
*** prateek has joined #openstack-nova05:04
*** sudipto_ has quit IRC05:10
*** sudipto has quit IRC05:10
*** tanee_ has joined #openstack-nova05:10
*** sudipto has joined #openstack-nova05:12
*** sudipto_ has joined #openstack-nova05:13
*** Apoorva has joined #openstack-nova05:14
*** sree has joined #openstack-nova05:15
*** zayro has quit IRC05:15
gmannalex_xu:  filter values we cannot restrict/correct with schema without version bump right? on this - https://review.openstack.org/#/c/418880/05:16
alex_xugmann: yea, we can't do that now.05:17
alex_xugmann: this series looks like very close https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/add-whitelist-for-server-list-filter-sort-parameters05:17
gmannalex_xu: on that only :).05:18
*** dikonoor has joined #openstack-nova05:19
*** edmondsw has joined #openstack-nova05:20
*** edmondsw has quit IRC05:24
*** thorst has joined #openstack-nova05:29
*** thorst has quit IRC05:34
*** rama_y_ has quit IRC05:40
*** rama_y has quit IRC05:40
*** ratailor has joined #openstack-nova05:42
*** yamahata has joined #openstack-nova05:43
*** psachin has quit IRC05:44
*** jerrygb has quit IRC05:45
*** jerrygb has joined #openstack-nova05:46
*** esberglu has joined #openstack-nova05:46
*** hongbin has quit IRC05:48
*** psachin has joined #openstack-nova05:49
*** adisky_ has joined #openstack-nova05:50
*** esberglu has quit IRC05:50
*** jerrygb has quit IRC05:50
*** mingyu has joined #openstack-nova05:57
*** amotoki has joined #openstack-nova06:00
openstackgerritCao Xuan Hoang proposed openstack/nova: Add python 3 classifiers  https://review.openstack.org/42167706:00
*** mingyu has quit IRC06:05
openstackgerritHieu LE proposed openstack/nova: Completely remove mox from objects/test_instance.py  https://review.openstack.org/34346506:06
*** mingyu has joined #openstack-nova06:11
*** lpetrut has joined #openstack-nova06:12
*** lpetrut has quit IRC06:13
*** lpetrut has joined #openstack-nova06:14
*** sree has quit IRC06:17
*** nkrinner_afk is now known as nkrinner06:20
*** sree has joined #openstack-nova06:20
*** Apoorva has quit IRC06:21
*** tyrola has quit IRC06:23
*** mingyu has quit IRC06:23
*** mingyu has joined #openstack-nova06:24
*** satyar has joined #openstack-nova06:24
*** tyrola has joined #openstack-nova06:25
*** slaweq_ has joined #openstack-nova06:26
*** tovin07 has joined #openstack-nova06:29
*** tovin07_ has joined #openstack-nova06:29
*** sree has quit IRC06:30
*** sree has joined #openstack-nova06:30
*** qwebirc71030 has joined #openstack-nova06:31
*** tovin07 has quit IRC06:31
*** abalutoiu has quit IRC06:33
*** sree has quit IRC06:35
*** Sukhdev has quit IRC06:36
*** unicell has joined #openstack-nova06:37
*** mjura has joined #openstack-nova06:40
*** esberglu has joined #openstack-nova06:41
*** liuyulong has joined #openstack-nova06:43
*** qwebirc71030 has quit IRC06:44
*** dimtruck is now known as zz_dimtruck06:45
*** esberglu has quit IRC06:46
*** mingyu has quit IRC06:46
*** mingyu has joined #openstack-nova06:48
*** zhangjl has joined #openstack-nova06:51
*** zhangjl has left #openstack-nova06:52
*** sridharg has joined #openstack-nova06:58
*** houming has quit IRC06:58
*** gongysh has quit IRC07:00
*** phuongnh has quit IRC07:03
*** phuongnh has joined #openstack-nova07:04
*** sahid has joined #openstack-nova07:05
*** archies has joined #openstack-nova07:05
*** lpetrut has quit IRC07:06
archies@hi auggy07:07
*** salv-orlando has joined #openstack-nova07:12
*** bapalm has quit IRC07:16
*** salv-orlando has quit IRC07:17
*** Oku_OS-away is now known as Oku_OS07:17
*** raunak has quit IRC07:18
*** bapalm has joined #openstack-nova07:18
*** voelzmo has joined #openstack-nova07:21
*** mingyu has quit IRC07:22
*** tesseract has joined #openstack-nova07:25
*** tesseract has quit IRC07:25
*** andreas_s has joined #openstack-nova07:25
*** tesseract has joined #openstack-nova07:26
*** voelzmo has quit IRC07:27
*** sree has joined #openstack-nova07:28
*** sree has quit IRC07:29
*** sree has joined #openstack-nova07:29
*** sree has quit IRC07:29
*** thorst has joined #openstack-nova07:30
*** mingyu has joined #openstack-nova07:30
*** sree has joined #openstack-nova07:31
*** mingyu has quit IRC07:32
*** sahid has quit IRC07:32
*** voelzmo has joined #openstack-nova07:33
*** sahid has joined #openstack-nova07:33
*** sree_ has joined #openstack-nova07:34
*** sree has quit IRC07:34
*** sree_ is now known as Guest951307:34
*** thorst has quit IRC07:35
sahidjaypipes: hello, any chance you have a look at https://review.openstack.org/#/c/400875/ ?07:39
*** Guest9513 has quit IRC07:42
*** abalutoiu has joined #openstack-nova07:42
*** salv-orlando has joined #openstack-nova07:45
openstackgerritmelanie witt proposed openstack/nova: WIP Count resources to check quota for cells  https://review.openstack.org/41652107:48
openstackgerritmelanie witt proposed openstack/nova: Make Quotas object favor the API database  https://review.openstack.org/41094507:48
openstackgerritmelanie witt proposed openstack/nova: Add online migration to move quotas to API database  https://review.openstack.org/41094607:48
*** dgonzalez_ has joined #openstack-nova07:50
*** mkoderer_ has joined #openstack-nova07:50
*** tanee has quit IRC07:51
*** tanee_ is now known as tanee07:52
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Use mask_dict_password from oslo.utils  https://review.openstack.org/38834507:52
*** dgonzalez_ has quit IRC07:52
*** mkoderer_ has quit IRC07:52
*** salv-orlando has quit IRC07:53
*** phuongnh has quit IRC07:54
*** phuongnh has joined #openstack-nova07:55
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Add image_meta when unshelving volume backend vm  https://review.openstack.org/34706307:55
*** thorst has joined #openstack-nova08:00
*** thorst has quit IRC08:00
*** lpetrut has joined #openstack-nova08:00
openstackgerritTovin Seven proposed openstack/nova: Integrate OSProfiler and Nova  https://review.openstack.org/25470308:01
gmannalex_xu:  can you check my comments on this-  https://review.openstack.org/#/c/415142/2108:01
gmannalex_xu:  i did not get sort_dir filter things till i am not missing anything08:01
*** abalutoiu has quit IRC08:04
gmannalex_xu: API says "If you omit the sort direction in a pair, the API uses the natural sorting direction of the direction of the server sort_key attribute." it means we should prepare the sort_dir with 'desc' if not passed with sort_key but common.get_sort_params(req.params) does not seems to do ?08:05
Kevin_Zhenggmann08:06
Kevin_ZhengI already notice the problem about key and dir while coding08:06
Kevin_ZhengI think it is a bug08:07
gmannKevin_Zheng: oh, it mean it never working?08:07
*** lpetrut has quit IRC08:07
Kevin_ZhengI mean08:07
Kevin_Zhengif we don't strict the user to provide both key and dir08:08
gmannKevin_Zheng:  actually now in this BP it might be issue as we are removing the sort_dir and endup with removal of sor_dir of other sort_key08:08
Kevin_Zhengit will be a mess when we do the deleting08:08
Kevin_Zhengyes08:08
gmannKevin_Zheng: true but we says they can omit the sort_dir08:08
Kevin_ZhengI think it is already wrong when doing key=key1&key=key2&dir=dir2&key=key3&dir=dir308:10
*** markus_z has joined #openstack-nova08:10
Kevin_Zhengit will be [key1,key2,key3]and [dir2,dir3] in api08:10
Kevin_Zhengalready broken08:10
*** guchi_hiro_ has joined #openstack-nova08:10
Kevin_Zhengkey1 is already using dir208:10
Kevin_Zhengno matter we are deleting or not08:11
Kevin_Zhengit is already a bug IMO08:11
*** guchi_hiro_ has quit IRC08:11
Kevin_Zhengneutron strict the user to provide both key and dir08:11
Kevin_ZhengIf I remembered correctly08:12
gmannKevin_Zheng:  yea, thats true. did we force to use both ?08:12
Kevin_ZhengIn nova it is impossible to decide which dir belongs to which key08:12
Kevin_Zhengwell I think it needs further discussion and maybe a BP for next cycle08:13
*** tovin07 has joined #openstack-nova08:13
Kevin_Zhengbut the current implementation of whitelist bp is just kept current behaviour08:13
Kevin_Zhengsince it is already broken IMO08:14
*** gongysh has joined #openstack-nova08:14
alex_xugmann: the db layer done that https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L211108:15
Kevin_ZhengYeah, but as we discussed before, the key could be messed, because I was putting asc for key2 and key1 should be with defalt desc but the acutal response will be asc for key1 and defult desc for key208:17
Kevin_ZhengIMO it is a bug08:17
Kevin_ZhengStill, out of the scope of the BP, I thinks08:17
*** gszasz has joined #openstack-nova08:18
gmannalex_xu:  but it append at end and all sort_dir are being mixed if not provided for any key in between08:18
*** ralonsoh has joined #openstack-nova08:18
*** gokhan has joined #openstack-nova08:18
Kevin_ZhengI think we should strict user to provide both, just my opinion08:18
gmannhummm08:19
Kevin_Zhengor it should be handeled in a different way08:19
Kevin_Zhenglike key value pairs08:20
Kevin_Zhengtwo list is imposible to handle08:20
gmanni think we can do while fetching them from req08:20
gmannbut yea its out of scope of this BP, ll not block on this08:21
gmannsomething we can fix later08:22
Kevin_Zhengyeah08:22
gmannalex_xu:  Kevin_Zheng  about whitelist for sort_key, its not matching with spec one ?08:22
Kevin_ZhengI will check, too many fields and possible to miss one or two :)08:23
*** mingyu has joined #openstack-nova08:23
alex_xuyea, we didn'i miss anyone08:24
alex_xuoops, hope we didn't miss anyone08:25
gmannalex_xu:  Kevin_Zheng  i thin we missed in spec seems: created_at',  'updated_at',08:25
*** dikonoor has quit IRC08:26
gmannbut those seems very valid one so should be ok to add in code08:26
*** markvoelker has joined #openstack-nova08:26
*** dikonoor has joined #openstack-nova08:26
Kevin_Zhengyeah will do after writing the doc08:27
alex_xugmann: created_at and updated_at are in the line 271 and 27208:27
gmannalex_xu: yea in code its there but not in spec :)08:27
alex_xugmann: oops, I guess we miss that08:27
*** abalutoiu has joined #openstack-nova08:28
gmannKevin_Zheng: can just put those in spec but may be later no big deal as there will not be any objection on those08:28
alex_xugmann: +1 for update spec08:28
Kevin_Zhengyeah, might be a cleanup after code done08:29
*** esberglu has joined #openstack-nova08:29
openstackgerritVladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata  https://review.openstack.org/40595108:29
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata  https://review.openstack.org/40595408:29
gmann'updated_at' should be fine right ? i remember some discussion we had on this when this will be null at initial time08:30
gmanni hope we do not default it to created_at08:30
*** ijw has quit IRC08:31
alex_xugmann: updated_at and created_at looks like fine to me, that are in the API representation.08:32
gmannok08:32
*** amotoki_ has joined #openstack-nova08:33
*** esberglu has quit IRC08:34
*** mingyu has quit IRC08:34
*** rmart04 has joined #openstack-nova08:35
*** amoralej|off is now known as amoralej08:35
*** amotoki has quit IRC08:36
*** salv-orlando has joined #openstack-nova08:37
*** amotoki has joined #openstack-nova08:37
*** archies has quit IRC08:38
*** amotoki_ has quit IRC08:41
*** mingyu has joined #openstack-nova08:41
*** sree has joined #openstack-nova08:41
*** amotoki has quit IRC08:44
*** amotoki has joined #openstack-nova08:44
*** gongysh has quit IRC08:46
*** mingyu has quit IRC08:47
*** sahid has quit IRC08:47
*** sahid has joined #openstack-nova08:49
*** rmart04 has quit IRC08:50
gmannalex_xu: Kevin_Zheng just logged that as bug in case we forget - https://bugs.launchpad.net/nova/+bug/165736608:52
openstackLaunchpad bug 1657366 in OpenStack Compute (nova) "sort_key and sort_dir combination are not being honored in correct way" [Undecided,New] - Assigned to Ghanshyam Mann (ghanshyammann)08:52
alex_xugmann: cool08:52
Kevin_ZhengThanks08:52
*** hferenc has joined #openstack-nova08:55
*** rmart04 has joined #openstack-nova08:56
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** lpetrut has joined #openstack-nova09:01
*** mingyu has joined #openstack-nova09:02
*** mingyu_ has joined #openstack-nova09:03
*** mingyu has quit IRC09:03
*** mingyu_ has quit IRC09:05
*** sree has quit IRC09:08
*** ociuhandu has joined #openstack-nova09:10
*** huanxie2 has joined #openstack-nova09:11
openstackgerritZhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist  https://review.openstack.org/42176009:18
*** abalutoiu_ has joined #openstack-nova09:19
openstackgerritsahid proposed openstack/nova: libvirt: update the active migrations DS to support migration result  https://review.openstack.org/40926009:21
openstackgerritsahid proposed openstack/nova: libvirt: ensure live-migration success  https://review.openstack.org/40926109:21
*** salv-orlando has quit IRC09:21
*** gszasz has quit IRC09:22
*** jpena|off is now known as jpena09:22
*** abalutoiu has quit IRC09:22
*** derekh has joined #openstack-nova09:23
*** karimb has joined #openstack-nova09:25
*** jhesketh has quit IRC09:27
openstackgerritsahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology  https://review.openstack.org/40824209:27
openstackgerritsahid proposed openstack/nova: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/40824109:27
openstackgerritsahid proposed openstack/nova: objects: remove cpu_topology from __init__ of InstanceNUMATopology  https://review.openstack.org/40824009:27
*** amotoki_ has joined #openstack-nova09:29
*** tovin07 has quit IRC09:30
*** jhesketh has joined #openstack-nova09:30
*** amotoki has quit IRC09:31
*** ijw has joined #openstack-nova09:32
Kevin_Zhengalex_xu: should we strict sort_dir now?09:33
alex_xuKevin_Zheng: we shouldn't strict anything now09:33
Kevin_Zhengright, let me check what api_layer will do if bad dir provided09:34
*** jpena is now known as jpena|off09:35
Kevin_Zhengnothing, so I guess 500 will raise?09:36
*** yassine has joined #openstack-nova09:36
*** yassine is now known as Guest6256509:37
Kevin_Zhenghmm 400 so we are good09:37
*** ijw has quit IRC09:37
gmannKevin_Zheng:  alex_xu  we can strict for enum(desc, asc)09:39
*** sapcc-bot1 has quit IRC09:39
*** databus23_1 has quit IRC09:39
gmannKevin_Zheng: alex_xu  can we do date-time for created_at etc in https://review.openstack.org/#/c/420494/09:39
*** carthaca_ has quit IRC09:39
Kevin_ZhengI think the current bp is about to not limit those09:40
gmannKevin_Zheng:  in case you can run non date-time string for created_at and it 50009:40
Kevin_Zhengand we already response 400 for sort dir09:40
gmannKevin_Zheng: not limiting but i doubt it 500 currently same for created_at09:41
Kevin_Zhengdidnt dive too deep to find where that happened though09:41
gmannKevin_Zheng:  on sort_dir yes that why we can do on schema side, no need to go deep to get 40009:41
*** pcaruana has joined #openstack-nova09:41
*** abalutoiu__ has joined #openstack-nova09:42
*** jpena|off is now known as jpena09:42
gmannKevin_Zheng: alex_xu  here - https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L248909:45
gmannwe can just do on schema side09:45
*** lucas-afk is now known as lucasagomes09:45
*** mvk has quit IRC09:45
*** gongysh has joined #openstack-nova09:46
*** abalutoiu_ has quit IRC09:46
*** sapcc-bot has joined #openstack-nova09:46
*** mkoderer_ has joined #openstack-nova09:46
*** tpatzig_ has joined #openstack-nova09:46
*** dgonzalez_ has joined #openstack-nova09:46
Kevin_ZhengI tested created at09:47
Kevin_Zhengit will not return 50009:48
*** mkoderer_ has quit IRC09:48
*** dgonzalez_ has quit IRC09:48
*** tpatzig_ has quit IRC09:48
gmannthen we should make this also 400 like other strict validation in that patch, same for launched_at and terminated_at?09:50
Kevin_Zhengyeah09:50
Kevin_ZhengI think it will be ok with the current path09:51
Kevin_Zhengwill be a strict limit with microversion afterwards. I think thats the plan, if I rememered correctly09:51
*** tovin07_ has quit IRC09:51
Kevin_Zhengthe current is about to not affect current behavior09:51
*** abalutoiu__ is now known as abalutoiu09:54
*** tovin07 has joined #openstack-nova09:55
*** gszasz has joined #openstack-nova09:56
tovin07jaypipes, i rebase this patch (OSprofiler in Nova), could you review it? https://review.openstack.org/#/c/254703/ thanks!09:56
*** takashin has joined #openstack-nova09:56
tovin07because it conflict with recent merged09:57
*** tobasco has quit IRC09:57
*** tobasco has joined #openstack-nova09:57
*** owalsh_ has joined #openstack-nova09:59
*** owalsh has quit IRC09:59
openstackgerritVasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441309:59
*** thorst has joined #openstack-nova10:01
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc  https://review.openstack.org/41723310:02
openstackgerritVasyl Saienko proposed openstack/nova: Implement interface attach/detach in ironic virt driver  https://review.openstack.org/41997510:02
openstackgerritVasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups  https://review.openstack.org/38875610:03
*** hfu has quit IRC10:04
*** huanxie2 has quit IRC10:05
*** kevinz has quit IRC10:05
*** tuanluong_ has joined #openstack-nova10:06
*** larainema has quit IRC10:06
*** thorst has quit IRC10:06
*** yamahata has quit IRC10:06
*** slunkad has quit IRC10:06
*** hoangcx has quit IRC10:06
*** tuanluong has quit IRC10:07
*** slunkad has joined #openstack-nova10:10
*** tovin07 has quit IRC10:11
*** gongysh has quit IRC10:11
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514210:13
*** tuanluong_ has quit IRC10:14
gmannKevin_Zheng:  opps, i mis read about created_at return 500, if not 500 then it should not be modified as you said. thanks10:16
openstackgerritZhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist  https://review.openstack.org/42176010:16
*** esberglu has joined #openstack-nova10:17
*** guoyunxian has joined #openstack-nova10:17
openstackgerritzhangyujun proposed openstack/nova: Query parameter validation for hosts API  https://review.openstack.org/41888010:18
*** owalsh_ is now known as owalsh10:20
*** karimb has quit IRC10:20
*** mvk has joined #openstack-nova10:21
*** esberglu has quit IRC10:22
*** huanxie2 has joined #openstack-nova10:22
*** CristinaPauna has quit IRC10:26
*** huanxie2 has quit IRC10:26
*** CristinaPauna has joined #openstack-nova10:28
*** karimb has joined #openstack-nova10:31
*** ijw has joined #openstack-nova10:34
*** satyar has quit IRC10:35
openstackgerritBéla Vancsics proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622510:35
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough  https://review.openstack.org/42061410:37
*** pcaruana has quit IRC10:38
*** ijw has quit IRC10:39
*** edmondsw has joined #openstack-nova10:44
*** edmondsw has quit IRC10:48
openstackgerritClaudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps  https://review.openstack.org/42096010:49
*** phuongnh has quit IRC10:53
*** cdent has joined #openstack-nova10:58
openstackgerritAlvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances  https://review.openstack.org/32325011:02
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2)  https://review.openstack.org/40896411:04
*** gcb has quit IRC11:05
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004211:05
openstackgerritTakashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host  https://review.openstack.org/41492611:05
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host  https://review.openstack.org/40670711:05
*** huanxie has quit IRC11:06
alex_xuKevin_Zheng: yea, that is the plan11:10
*** esberglu has joined #openstack-nova11:12
*** udesale has quit IRC11:13
*** esberglu has quit IRC11:16
*** karimb has quit IRC11:19
*** sudipto_ has quit IRC11:25
*** sudipto has quit IRC11:25
*** karimb has joined #openstack-nova11:30
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters  https://review.openstack.org/42062611:31
openstackgerritSylvain Bauza proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method  https://review.openstack.org/41813411:32
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API  https://review.openstack.org/41796111:32
*** ijw has joined #openstack-nova11:35
tdurakovgibi: hi, are you around?11:38
*** aojea has quit IRC11:40
*** ijw has quit IRC11:40
*** acabot has quit IRC11:40
gibitdurakov: hi11:44
tdurakovgibi: smth related to notifications subteam, I believe: https://bugs.launchpad.net/nova/+bug/165742811:45
openstackLaunchpad bug 1657428 in OpenStack Compute (nova) "Nova sends notifications with different timestamp formats" [Undecided,New]11:45
gibitdurakov: you are right, this is notification problem11:46
gibitdurakov: I will look into this bug11:46
gibitdurakov: thanks for notifying11:46
tdurakovgibi, this one shouldn't be a big deal, do you need my help fixing this?11:48
*** salv-orlando has joined #openstack-nova11:49
*** aojea has joined #openstack-nova11:50
gibitdurakov: if you want to fix it then go ahead and just add me as a reviewer11:51
tdurakovgibi, yeah, why not)11:52
*** nicolasbock has joined #openstack-nova11:53
*** sree has joined #openstack-nova11:55
gibitdurakov: for me the interesting thing is that why created_at and launched_at has different format11:55
jrollmriedem: so, https://review.openstack.org/#/c/364413/ doesn't actually enable "nova port-attach" or whatever for ironic, it's changing stuff over to an ironic API to attach/detach vifs, so suspect you're testing the wrong thing there. also suspect we have a tempest regex for that job.11:55
gibitdurakov: those fields are copied from the instance object directly11:55
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850911:55
tdurakovgibi, yeah, that's strange11:55
*** pcaruana has joined #openstack-nova11:56
jrollmriedem: I think you want to test this, which 1) I don't expect to land this cycle, and 2) will need some devstack changes to give our VMs another nic. https://review.openstack.org/#/c/419975/511:56
jrollmriedem: that said, 364413 is tested in check queue, every boot will hit that new code path11:56
*** ratailor has quit IRC11:59
*** guchihiro has joined #openstack-nova12:00
*** amotoki_ has quit IRC12:01
*** thorst has joined #openstack-nova12:02
gibitdurakov: audit period values are generated here https://github.com/openstack/nova/blob/master/nova/notifications/base.py#L30812:05
* jroll notices that vasyl put the same comments in the patch12:06
*** thorst has quit IRC12:06
*** cdelatte has joined #openstack-nova12:15
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514212:17
openstackgerritZhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants  https://review.openstack.org/41533012:17
openstackgerritZhenyu Zheng proposed openstack/nova: Strict pattern match query parameters  https://review.openstack.org/42049412:17
*** acabot has joined #openstack-nova12:21
*** sapcc-bot has quit IRC12:23
*** dgonzalez_ has joined #openstack-nova12:23
*** tpatzig_ has joined #openstack-nova12:23
*** sapcc-bot1 has joined #openstack-nova12:23
cdentKevin_Zheng: if you agree with my last coment on https://review.openstack.org/#/c/410128/ can you abandon that?12:23
Kevin_Zhengcdent: ah yes, I was12:24
Kevin_ZhengGoing to but distracted by something else12:24
*** dgonzalez_ has quit IRC12:25
*** tpatzig_ has quit IRC12:25
*** sdague has joined #openstack-nova12:25
*** sree_ has joined #openstack-nova12:25
*** bvanhav_ has joined #openstack-nova12:26
*** sree has quit IRC12:27
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Call preallocate_for_instance before scheduler  https://review.openstack.org/41312412:30
*** abalutoiu_ has joined #openstack-nova12:31
*** smatzek has joined #openstack-nova12:33
*** sree has joined #openstack-nova12:33
*** dave-mccowan has joined #openstack-nova12:34
*** abalutoiu has quit IRC12:34
*** ijw has joined #openstack-nova12:36
*** guoyunxian has quit IRC12:38
*** nkrinner has quit IRC12:40
johnthetubaguyalex_xu: I was looking at: https://review.openstack.org/#/c/40857112:41
johnthetubaguyalex_xu: wondering where we raise 400 errors for _foo like properties12:41
openstackgerritAndreas Jaeger proposed openstack/python-novaclient: Fix help strings  https://review.openstack.org/42187912:41
*** ijw has quit IRC12:41
johnthetubaguydang it, I see now12:41
*** nkrinner has joined #openstack-nova12:43
takashinbauzas: alex_xu: Could you review https://review.openstack.org/#/c/408955/ again and check my comments?12:44
openstackgerritVladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata  https://review.openstack.org/40595112:44
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata  https://review.openstack.org/40595412:44
*** sudipto_ has joined #openstack-nova12:44
*** sudipto has joined #openstack-nova12:44
*** ducttape_ has joined #openstack-nova12:44
*** links has quit IRC12:46
*** amotoki has joined #openstack-nova12:46
openstackgerritJim Rollenhagen proposed openstack/nova: Move to tooz hash ring implementation  https://review.openstack.org/39902212:49
jaypipessahid: yeah, I'm still in the middle of a review on that. sorry, yesterday most of the day was having to do management stuff :(12:50
jaypipessahid: have 4 straight hours of meetings and performance reviews to do this morning. will try to finish up reviews after that12:50
wznoinskanyone with experience in nova-lxd around?12:51
*** thorst has joined #openstack-nova12:51
jaypipeswznoinsk: the folks with experience with that would more likely be in #openstack-snaps12:52
wznoinskjaypipes, oh ok, thanks12:52
jaypipeswznoinsk: and good morning :)12:53
wznoinskjaypipes, good morning to you too, must be v. early one for you?12:54
jaypipeswznoinsk: nah, not too bad. 7:50am12:55
wznoinskthat peace and quiet time when everybody else hasn't woken up to bother you ;-) anyways, have a good one, I'm going for lunch myself12:57
jaypipessahid: make that 5 straight hours of meetings and PRs...12:58
jaypipeswznoinsk: heh, remember I have 3 pugs that are *always* bothering me :)12:58
*** claudiub|2 is now known as claudiub12:58
sahidjaypipes: ok thanks jay, sorry to rush you on it but i'm little bit scrary with the ff comming12:58
*** jamielennox is now known as jamielennox|away12:59
jaypipessahid: understood. have you found more cores to look at the bottom two patches in that series? sfinucan maybe?12:59
sahidjaypipes: he is in holidays this week12:59
*** esberglu has joined #openstack-nova13:00
jaypipessahid: ah. perhaps you can reach out to another core? maybe mriedem or sdague?13:00
jaypipesor bauzas?13:00
*** tongli has joined #openstack-nova13:01
sahidjaypipes: yes sure i see dansmith commenting on one of the patch, then i remember johnthetubaguy interested on this feature and perhaps mriedem13:01
alex_xunova api meeting is running13:01
jaypipessahid: dansmith is out for a while (he's adventuring in Eastern Oregon in the mountains) :)13:01
*** sapcc-bot1 has quit IRC13:02
sahidoh i see13:02
*** sapcc-bot has joined #openstack-nova13:02
*** tpatzig_ has joined #openstack-nova13:02
*** mkoderer_ has joined #openstack-nova13:02
*** dgonzalez_ has joined #openstack-nova13:02
sahidjohnthetubaguy, mriedem one of you guy have a moment to review the feature emulator threads ? it's basically reserving a pCPU to make running emulator threads13:03
*** tpatzig_ has quit IRC13:04
*** mkoderer_ has quit IRC13:04
*** dgonzalez_ has quit IRC13:04
*** esberglu has quit IRC13:04
johnthetubaguystill recovering from a nasty bug, but starting to get through my review queue, I had hoped to get to that one13:04
*** Guest62565 has quit IRC13:06
*** udesale has joined #openstack-nova13:06
openstackgerritZhenyu Zheng proposed openstack/nova: Add release note and docs for filter/sort whitelist  https://review.openstack.org/42176013:07
*** amotoki has quit IRC13:08
*** Guest62565 has joined #openstack-nova13:09
*** ducttape_ has quit IRC13:09
*** esberglu has joined #openstack-nova13:11
*** esberglu has quit IRC13:11
*** esberglu has joined #openstack-nova13:11
*** tblakes has joined #openstack-nova13:11
*** rnoriega has quit IRC13:12
*** jschlueter has quit IRC13:12
*** jschlueter has joined #openstack-nova13:12
*** edmondsw has joined #openstack-nova13:12
*** rnoriega has joined #openstack-nova13:13
snikitinjohnthetubaguy: Hi John! I heard that you are good in Xen. Could you add this xen related patch into your review queue, please? :)  https://review.openstack.org/#/c/39961313:14
*** lucasagomes is now known as lucas-hungry13:15
*** esberglu has quit IRC13:15
*** kylek3h has quit IRC13:15
*** abalutoiu_ has quit IRC13:16
*** karimb has quit IRC13:17
*** mdnadeem has quit IRC13:18
*** jamielennox|away is now known as jamielennox13:19
*** beagles is now known as beagles_brb13:24
*** amotoki has joined #openstack-nova13:28
*** sean-k-mooney has quit IRC13:29
*** rfolco has joined #openstack-nova13:32
*** diga has quit IRC13:32
*** salv-orlando has quit IRC13:36
*** ijw has joined #openstack-nova13:38
*** esberglu has joined #openstack-nova13:39
*** pradk has joined #openstack-nova13:39
*** esberglu has quit IRC13:39
*** esberglu has joined #openstack-nova13:40
*** liverpooler has joined #openstack-nova13:41
*** pbandark has quit IRC13:43
*** ijw has quit IRC13:44
*** catintheroof has joined #openstack-nova13:45
*** esberglu has quit IRC13:46
*** jpena is now known as jpena|lunch13:47
*** kylek3h has joined #openstack-nova13:48
EmilienMowalsh: hey! so I have quick updates about Placement service13:49
owalshEmilienM: shoot13:49
EmilienMowalsh: the service was working fine in our CI (tripleo & puppet) a few days ago13:49
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [py35] Fixes to get more tempest tests working  https://review.openstack.org/42191113:49
EmilienMowalsh: and I've noticed yesterday that nova-compute fails to register the resource again13:49
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120913:49
EmilienMit tries in loop though, but it doesn't work13:50
EmilienMI tried to reproduce it locally but it worked, I'm a bit confused why it fails in CI13:50
EmilienMI can provide logs13:50
EmilienMhttp://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_45_52_92013:50
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120913:50
EmilienMcdent, bauzas ^ just fyi13:50
owalshEmilienM: hrmm, so it would suggest a timing issue, but it tries in a loop... for how long?13:51
EmilienMowalsh: I don't think it's a timing issue this time but let me check13:52
EmilienMowalsh: if you remember, the first time I reported this issue was because we didn't have this loop than dansmith did13:52
EmilienMbut now we have the loop and it still fails to register13:52
*** larainema has joined #openstack-nova13:53
markus_zmriedem: Did I clear up your doubts on https://review.openstack.org/#/c/347471/ ? The test hook behaves as expected IMO.13:53
*** ducttape_ has joined #openstack-nova13:53
*** xyang1 has joined #openstack-nova13:53
*** liverpooler has quit IRC13:54
*** liverpooler has joined #openstack-nova13:54
EmilienMowalsh: I would like to find in logs where it tries to register is and the trace of failure13:54
EmilienMowalsh: "Unable to refresh my resource provider record" is not enough13:54
*** voelzmo has quit IRC13:54
*** prateek has quit IRC13:55
EmilienMowalsh: I'm creating a launchpad bug13:56
*** jerrygb has joined #openstack-nova13:56
*** bvanhav__ has joined #openstack-nova13:59
*** amoralej is now known as amoralej|lunch13:59
*** jerrygb has quit IRC13:59
EmilienMowalsh: do you see something weird in the nova.conf? https://paste.fedoraproject.org/529703/14847479/14:00
*** bvanhav__ is now known as bvanhav14:00
*** bvanhav_ has quit IRC14:00
gmannjohnthetubaguy: alex_xu Kevin_Zheng  +1 on putting sort_key one last on series14:00
EmilienMowalsh: https://bugs.launchpad.net/nova/+bug/165746714:01
openstackLaunchpad bug 1657467 in OpenStack Compute (nova) "placement: unable to refresh compute resource provider record" [Undecided,New]14:01
*** esberglu has joined #openstack-nova14:01
owalshEmilienM: auth_urls is IPv6 loopback... does that work???14:01
*** mriedem has quit IRC14:01
*** mdrabe has joined #openstack-nova14:01
EmilienMowalsh: of course, why not?14:01
alex_xugmann: johnthetubaguy I guess for now, the policy one have more thing to do. maybe the policy one is last one14:02
*** burgerk has joined #openstack-nova14:02
EmilienMowalsh: it works for all services like tha.14:02
EmilienMthat*14:02
gmannalex_xu:  but, sort_key ignore list seems controversial which might take time14:03
johnthetubaguyalex_xu: yeah, what gmann said14:03
*** sacharya has joined #openstack-nova14:03
*** jwcroppe has joined #openstack-nova14:03
*** jerrygb has joined #openstack-nova14:03
*** jerrygb has quit IRC14:03
gmannalex_xu: johnthetubaguy  need to go to bed, ll check  logs and patches tomorrow.14:03
*** jerrygb has joined #openstack-nova14:04
alex_xugmann: johnthetubaguy ok, got it, that works for me14:04
*** eglynn has joined #openstack-nova14:04
alex_xugmann: see you tomorrow14:04
owalshEmilienM: just fishing for something that might differ from running locally14:04
*** amotoki has quit IRC14:04
*** liverpooler has quit IRC14:04
*** karimb has joined #openstack-nova14:05
*** liverpooler has joined #openstack-nova14:05
*** sacharya has quit IRC14:08
*** stvnoyes has quit IRC14:08
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_instances  https://review.openstack.org/42140414:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell_and_hosts  https://review.openstack.org/42142014:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 verify_instance  https://review.openstack.org/42143614:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 simple_cell_setup  https://review.openstack.org/42133314:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 map_cell0  https://review.openstack.org/42138414:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 delete_cell  https://review.openstack.org/42145914:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 discover_hosts  https://review.openstack.org/42144214:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 list_cells  https://review.openstack.org/42145814:09
openstackgerritDiana Clarke proposed openstack/nova: nova-manage docs: cell_v2 create_cell  https://review.openstack.org/42144414:09
*** stvnoyes has joined #openstack-nova14:09
eglynncdent: are the placement API logs normally found under /logs/nova ?14:09
*** cleong has joined #openstack-nova14:09
cdenteglynn: the logs only show up after it has actually had its first request14:10
cdentso if the resource-tracker can't see the service, it will never make a request14:10
cdentand never make a log entry14:10
*** burgerk has quit IRC14:10
*** owalsh is now known as owalsh-brb14:10
eglynna-ha, /me was wondering why none to be found under http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova14:10
cdentit's one of the side effects of being hosted under mod-wsgi14:11
*** tuan_luong has joined #openstack-nova14:11
jaypipeseglynn: heyo, how are you Eoghan? :)14:11
takashinalex_xu: Could you review https://review.openstack.org/#/c/408955/ again?14:11
eglynnjaypipes: all good :) ... and you?14:12
EmilienMeglynn, cdent: yes logs are in /var/log/nova14:12
cdenteglynn, EmilienM : the presenting error "The placement API endpoint not found." suggests either that the rt can't reach keystone, or that keystone is not yet up to date14:12
EmilienMthey always were at least14:12
jaypipeseglynn: busy as always, which is good :)14:12
*** tuanla_ has joined #openstack-nova14:12
EmilienMcdent: it worked 2 days ago :(14:12
eglynnindeed14:12
EmilienMcdent: I can show you logs, a sec14:12
*** pradk has quit IRC14:12
*** vladikr has joined #openstack-nova14:13
*** sneti has joined #openstack-nova14:13
*** gouthamr has joined #openstack-nova14:13
EmilienMcdent: here are logs from 5 days ago:14:13
EmilienMhttp://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/nova/14:13
EmilienMyou can see nova placement api log, and everything works14:13
*** sneti has quit IRC14:13
EmilienMwe haven't changed anything on this topic https://github.com/openstack/puppet-nova/commits/master14:14
*** tlian has joined #openstack-nova14:14
*** udesale has quit IRC14:15
cdentEmilienM: http://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/nova/nova-compute.txt.gz#_2017-01-13_21_20_29_24514:15
*** owalsh-brb has quit IRC14:15
EmilienMcdent: credentials are created by Puppet, we never changed that since December14:16
EmilienMlet me show the logs14:16
cdentI don't dispute that14:16
cdentall I'm saying is that the logs you pointed to me as being successful are saying they have bad credentials14:16
EmilienMhttp://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/puppet-20170113_212224.txt.gz#_2017-01-13_21_21_1414:16
*** owalsh-brb has joined #openstack-nova14:16
cdentI don't know if that means anything or not14:16
EmilienMthey are created at 21:21:1414:16
*** owalsh-brb is now known as owalsh14:16
*** yellow has joined #openstack-nova14:17
EmilienMand your log is at 21:2014:17
*** udesale has joined #openstack-nova14:17
EmilienMif you look later, it worked: http://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/nova/nova-compute.txt.gz#_2017-01-13_21_21_32_49014:17
* cdent nods14:17
EmilienMso like I said, it worked on January 13th14:17
EmilienMbut not now14:17
cdentI get it, I'm just trying to clarify the pieces14:17
*** amotoki has joined #openstack-nova14:18
EmilienMsomething changed in Nova between January 13th and at least 17th (I saw the thing yesterday)14:18
*** takashin has left #openstack-nova14:18
*** udesale has quit IRC14:18
*** beagles_brb is now known as beagles14:18
*** udesale has joined #openstack-nova14:19
EmilienMcdent: maybe https://github.com/openstack/nova/commit/c17772e1f202c9ffb651adc3299a1990c35766f3 ?14:19
openstackgerritZhenyu Zheng proposed openstack/nova: Strict pattern match query parameters  https://review.openstack.org/42049414:19
*** baoli has joined #openstack-nova14:20
cdentEmilienM no impact on the presence of the service. I think the problem is most likely on the keystone side, give me a chance to read through the logs more closely14:20
*** baoli has quit IRC14:20
EmilienMcdent: sure, and thanks for helping14:20
*** baoli has joined #openstack-nova14:21
*** udesale has quit IRC14:21
*** salv-orlando has joined #openstack-nova14:21
*** udesale has joined #openstack-nova14:21
eglynnthe timing of the credentials creation shouldn't be a show-stopper, right, as the report will be retried since https://review.openstack.org/#/c/418590/4/nova/scheduler/client/report.py landed on Jan 11th14:21
*** READ10 has joined #openstack-nova14:22
Kevin_Zhengjohnthetubaguy: https://review.openstack.org/420494 rebased to the fitler whitelist patch, so it will be the 2nd of the series, please have a check when you got a sec, hope we can merge the 1st and 2nd of the series, I will keep working on the policy one and sort one torrmorow :)14:22
*** lucas-hungry is now known as lucasagomes14:25
pumaranikarcdent, again checking https://review.openstack.org/#/c/395194/ json formatter for placements. I am wondering if we can use @wsgify.middleware decorator to alter webob response to check error codes.14:26
pumaranikarcdent, not sure about it. Any suggestions on it ?14:27
*** kaisers has joined #openstack-nova14:29
cdentpumaranikar: I'll have to get back to you about this a bit later, digging in some log files right now. In general however I'm not sure about middleware for this. It might work. Basically you would want to check for an error response and a body which is application/json, then modify the json14:29
pumaranikarcdent, sure thanks. I will try to find more on this. Will catch up later when you are free.14:31
cdentEmilienM: what is making these requests: http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/keystone/keystone.txt.gz#_2017-01-17_17_36_38_61414:32
*** voelzmo has joined #openstack-nova14:32
*** voelzmo has quit IRC14:33
EmilienMcdent: probably nova compute14:33
EmilienMcdent: but probably keystone resources haven't been created yet14:33
EmilienM"Could not find user: placement"14:33
*** mriedem has joined #openstack-nova14:34
* cdent nods, still trying to fill in the mental blanks14:34
eglynncdent, EmilienM: worth re-running the CI tests with debug logging? (not much detail to go on currently in the logs)14:35
cdentis there anything in the logs anywhere that is listing the service catalog, because from the resource tracker code side, that error is what it says: I can't find the endpoint I'm looking for14:36
EmilienMeglynn: I ran 20 CI jobs yesterday and 10 today14:36
EmilienMeglynn: it's all consistent, it fails all the time14:36
EmilienMcdent: not now but I can do it for debug14:37
*** karimb has quit IRC14:37
eglynnEmilienM: that would potentially be useful14:37
EmilienMcdent: but puppet does it http://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/puppet-20170113_212224.txt.gz#_2017-01-13_21_21_1914:37
EmilienM2017-01-13 21:21:19 +0000 /Stage[main]/Nova::Keystone::Auth_placement/Keystone::Resource::Service_identity[placement]/Keystone_service[placement::placement]/ensure (notice): created14:37
EmilienMwhich means we have a service called "placement"14:37
*** jose-phillips has joined #openstack-nova14:37
EmilienMlet me add some debug in the CI job14:38
*** smatzek has quit IRC14:39
*** abalutoiu_ has joined #openstack-nova14:39
*** tuanla_ has quit IRC14:39
*** tuan_luong has quit IRC14:39
*** ijw has joined #openstack-nova14:42
*** tuan_luong has joined #openstack-nova14:42
*** tuan_luong has quit IRC14:42
*** tuan_luong has joined #openstack-nova14:42
*** tuan_luong has quit IRC14:42
*** tuan_luong has joined #openstack-nova14:43
cdentEmilienM, eglynn: based on the logs, the resource tracker code is working as expected: it can't find the placement endpoint so it doesn't do the requested action. Later it tries again and still can't find the placement endpoint.14:43
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120914:43
EmilienMcdent: let me add some debug in puppet logs so we can see keystone resources14:44
cdentso something, I'm not sure what, is going wrong in the communication with keystone, unfortunately we're not actually logging much of that interaction and the error (that I can find)14:44
eglynnEmilienM: with debug logging, we should see the endpoint creation (o lack thereof) in the keystone logs?14:45
eglynn*or14:45
EmilienMeglynn: yes, let me look14:46
mriedemwell, why doesn't the RT try to connect again?14:46
mriedemfirst failure is here http://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/nova/nova-compute.txt.gz#_2017-01-13_21_19_30_09014:46
mriedembecause the placement endpoint isn't setup yet http://logs.openstack.org/66/420166/2/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/591bf96/logs/puppet-20170113_212224.txt.gz#_2017-01-13_21_21_1914:46
mriedemso that's expected14:46
*** ijw has quit IRC14:46
mriedembut later attempts should work14:46
cdentmriedem: and they don't14:46
EmilienMeglynn: well no I don't see it14:47
cdenthttp://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_43_51_91814:47
*** ekuris_ has joined #openstack-nova14:47
mriedemcdent: that's still before the endpoint is added in keystone14:47
mriedemso that's expected14:47
*** gcb has joined #openstack-nova14:47
cdentstill14:47
mriedemanything after 21:21:19 should be fine14:47
*** jamesdenton has joined #openstack-nova14:48
cdentI think I've got log confusion14:48
eglynnagreed, if the retry logic is working as intended14:48
cdentmriedem: the 01-13 logs are not in question14:48
cdentit's the 01-1714:48
EmilienMcdent: I'm adding more debug https://review.openstack.org/42198014:49
EmilienMoops typo14:49
* cdent needs to coffee up, brb14:49
mriedemso in the 1/17 logs the placement endpoint is registered here right? http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/puppet-20170117_173440.txt.gz#_2017-01-17_17_33_4214:49
EmilienMwhat is weird, is that it worked 5 days ago14:50
mriedemat 17:3314:50
*** jpena|lunch is now known as jpena14:50
openstackgerritNadya Shakhat proposed openstack/nova: Make all datastamps formats equal  https://review.openstack.org/42198114:50
mriedemand this would fail before that http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_31_39_86214:50
mriedemat 17:3114:50
openstackgerritEric Fried proposed openstack/nova: Introduce base PowerVM driver  https://review.openstack.org/39128814:50
EmilienMmriedem: yes exactly14:50
mriedembut it should work here http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_34_40_92014:51
*** sree has quit IRC14:51
EmilienMyes14:52
*** edand has joined #openstack-nova14:52
*** hongbin has joined #openstack-nova14:53
*** gongysh has joined #openstack-nova14:54
*** links has joined #openstack-nova14:56
*** links has quit IRC14:56
*** Jinxing has joined #openstack-nova14:57
*** Swami has joined #openstack-nova14:58
gcbjohnthetubaguy:  need +A  for the python 3 unit test https://review.openstack.org/#/c/406435/14:58
*** Swami_ has joined #openstack-nova14:58
*** sree has joined #openstack-nova14:58
*** Swami_ has quit IRC15:00
*** tongli has quit IRC15:00
*** Swami_ has joined #openstack-nova15:01
*** jheroux has joined #openstack-nova15:01
*** mtanino has joined #openstack-nova15:01
mriedemlyarwood: i -1ed https://review.openstack.org/#/c/421261/ just for the sheer confusion15:03
*** mingyu has joined #openstack-nova15:03
*** dikonoor has quit IRC15:04
*** smatzek has joined #openstack-nova15:04
*** jistr is now known as jistr|mtg15:05
*** sree has quit IRC15:05
*** jdandrea has joined #openstack-nova15:07
*** ducttape_ has quit IRC15:10
*** sree has joined #openstack-nova15:11
jdandreaIs there a time limit for nova-scheduler to do its job in scheduling a server, and where is that setting (even if it's hardcoded/fixed)?15:12
*** david-lyle has joined #openstack-nova15:13
*** amoralej|lunch is now known as amoralej15:14
*** ekuris_ has quit IRC15:14
*** eandersson__ has joined #openstack-nova15:15
*** openstack has joined #openstack-nova15:19
cdentthere's a retry limit, isn't there?15:19
*** hwoarang has joined #openstack-nova15:19
*** jbernard has joined #openstack-nova15:19
*** dulek has joined #openstack-nova15:19
*** mrhillsman has joined #openstack-nova15:19
*** JayF has joined #openstack-nova15:19
*** hieulq has joined #openstack-nova15:19
*** jlvillal_ is now known as jlvillal15:20
*** fungi has joined #openstack-nova15:20
markus_zadreznec: Got a minute? I'd like to setup the RTD job for nova-dpm and I'm curious what you did on the RTD side to enable that.15:20
*** jlvillal is now known as Guest8897715:20
*** ttx has joined #openstack-nova15:21
lyarwoodmriedem: thanks and apologies for missing that it hadn't landed on master etc15:21
* lyarwood isn't sure what he was thinking +2'ing that review :|15:21
*** NightKhaos has joined #openstack-nova15:21
jdandreamarkus_z *nod* that's good from an overall perspective, got it. I'm wondering if there's a timeout specific to the scheduler as well, in the process of trying conf.scheduler.max_attempts (3) times.15:22
*** McNinja has joined #openstack-nova15:22
*** Matias has joined #openstack-nova15:22
*** mgagne has joined #openstack-nova15:22
*** comstud has joined #openstack-nova15:22
*** Daviey has joined #openstack-nova15:22
*** amotoki has joined #openstack-nova15:22
*** esberglu has joined #openstack-nova15:22
*** sudipto has joined #openstack-nova15:22
*** sneti_ has joined #openstack-nova15:22
*** zz_dimtruck has joined #openstack-nova15:22
*** khappone_ has joined #openstack-nova15:22
*** leifz has joined #openstack-nova15:22
*** ejat has joined #openstack-nova15:22
*** yhvh has joined #openstack-nova15:22
*** Matias has quit IRC15:22
*** Matias has joined #openstack-nova15:22
markus_zcdent: I thought that's used for the RetryFilter. I assumed the scheduler *always* finds 0..* compute nodes. The question from above sounds like it never ends.15:23
*** mgagne has quit IRC15:23
*** mgagne has joined #openstack-nova15:23
*** test222 has quit IRC15:23
*** mgagne is now known as Guest5853115:23
*** BlackDex_ is now known as BlackDex15:23
mriedemalaski: when you get a minute, why do we need the cell_info stuff in https://review.openstack.org/#/c/369634/ ? - doesn't that defeat the purpose of having to run discover_hosts, i.e. we want auto-registration of hosts into a cell so that people/tools don't have to run discover_hosts manually. i'm thinking in at least the single-cell case auto-register is pretty simple.15:24
cdentjdandrea: do you want to stop sooner or less soon?15:25
jdandreamarkus_z I'm looking at https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L519 ... hmm, no timeout to speak of (that I can see).15:25
adreznecHey markus_z. It's been a while since I did that, but I'd be happy to help with anything I can remember15:25
jdandreacdent I don't want to stop sooner or less soon, neither. I am just trying to determine if there's a scheduling-specific timeout at all.15:25
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough  https://review.openstack.org/42061415:25
jdandreacdent Looking at this I suspect there ... isn't? (a scheduler-specific timeout) https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L51915:26
*** geekinutah has joined #openstack-nova15:26
*** mwhahaha has joined #openstack-nova15:27
*** DuncanT has joined #openstack-nova15:27
jdandreacdent That is, there's a 60 second overall timeout to boot a server. But is there anything that puts a time limit on nova-scheduler to come up with an answer to begin with?15:27
edleafejdandrea: nothing that I'm aware of15:28
jdandreaedleafe excellent, ty!15:28
jdandrea(and that matches up with what I'm finding as well)15:28
jdandreaI just wanted to be sure I wasn't missing something. ;)15:28
*** briancurtin has joined #openstack-nova15:29
*** fyxim has joined #openstack-nova15:29
markus_zadreznec: thanks in advance. What do you remember? :) I was wondering which account I have to use. My own? A new one?15:30
adreznecmarkus_z: So on the rtd side I went through the basic registration steps (set up my account, create project, etc) w/ sphinx docs and set the URL to point at the git.o.o for the repo.15:30
*** krtaylor has joined #openstack-nova15:31
cdentEmilienM: can you ping me please when that debug job comes through with info?15:31
*** pkholkin has joined #openstack-nova15:31
adreznecThen in the advanced settings I had to configure it to use a virtualenv for builds w/ the right requirements file15:31
jdandreamarkus_z, cdent ... unless there's an oslo *messaging* timeout? https://github.com/openstack/nova/blob/4e747092bcb015303efc2ab13da98ef5ce575ec8/nova/scheduler/rpcapi.py#L12715:32
adreznecI think that was about it actually, the infra hook took care of the rest (notifying for rebuild, etc)15:32
EmilienMcdent: yes15:32
EmilienMcdent: I will15:32
*** psachin has quit IRC15:33
*** bswartz has joined #openstack-nova15:33
markus_zadreznec: Did you use the email address of a "functional Notes ID" for the registration? Or simply your own?15:34
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: set vlan tag for macvtap on SRIOV VFs  https://review.openstack.org/42118615:34
*** baoli has quit IRC15:35
*** test222 has joined #openstack-nova15:35
*** david-lyle has quit IRC15:36
*** david-lyle_ has joined #openstack-nova15:36
adreznecmarkus_z: I originally just used my own, then later on added our CI email address as a maintainer to the docs projects15:36
*** yellow has quit IRC15:36
*** david-lyle_ has quit IRC15:37
*** david-lyle_ has joined #openstack-nova15:37
*** Jinxing has quit IRC15:37
*** sheel has quit IRC15:37
markus_zadreznec: Ah, ok, makes sense. I'll simply start so as well. Thanks!15:37
adreznecmarkus_z: Np, if you have any issues I can help with let me know15:38
*** jaosorior has joined #openstack-nova15:41
*** david-lyle_ has quit IRC15:41
cdentedleafe, bauzas, jaypipes: I'm going to be on PTO tomorrow and friday, so if an rp/placement update is desired if someone could do it that would be great. Otherwise I'll pick it back up next week.15:42
edleafecdent: roger that15:42
*** ijw has joined #openstack-nova15:42
*** zz_dimtruck is now known as dimtruck15:45
*** jvgrant__ has quit IRC15:45
*** kaisers has quit IRC15:46
*** Swami_ has quit IRC15:46
*** Swami_ has joined #openstack-nova15:47
bauzascdent: ack15:47
*** david-lyle has joined #openstack-nova15:47
*** burt has quit IRC15:48
openstackgerritMihail proposed openstack/nova: proposed fix for bug 1646896 - System hangs when using NFS storage backend with loopback mounts  https://review.openstack.org/42204115:48
openstackbug 1646896 in OpenStack Compute (nova) "System hangs when using NFS storage backend with loopback mounts" [Low,Confirmed] https://launchpad.net/bugs/164689615:48
EmilienMcdent: wait wait wait15:48
*** mvk has quit IRC15:48
EmilienMcdent: you're going on PTO? no please15:48
openstackgerritVladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata  https://review.openstack.org/40595115:48
cdentEmilienM: just for two days15:48
*** ducttape_ has joined #openstack-nova15:48
* EmilienM hopes a "no please" can makes the difference15:48
*** ijw has quit IRC15:48
cdentand I'm still here for another few hours today15:48
cdentfor you in particular15:49
cdentbecause you said please15:49
bauzasEmilienM, cdent: I missed today's discussion15:49
bauzasanything I need to look at ?15:49
EmilienMbauzas: https://bugs.launchpad.net/tripleo/+bug/165746715:49
openstackLaunchpad bug 1657467 in tripleo "placement: unable to refresh compute resource provider record" [High,Triaged]15:49
EmilienMcdent: ♥15:49
markus_zaww15:49
markus_z;)15:50
*** Swami has quit IRC15:50
bauzasEmilienM: not sure I understand correctly, do we have the placement service running yet in TripleO?15:50
EmilienMbauzas: yes15:51
kashyapmarkus_z: Hey there15:51
*** jistr|mtg is now known as jistr15:51
EmilienMbauzas: but placement api doesn't start because compute can't register the resource15:51
markus_zkashyap: hey ho15:51
EmilienMbauzas: it worked 5 days ago and now it's failing to register15:51
kashyapmarkus_z: So...Michele has pinged on email about this review (https://review.openstack.org/#/c/313568/ -- Plugin to setup libvirt/QEMU from tar releases)15:51
kashyapmarkus_z: I know you gave some feedback in December15:52
bauzasEmilienM: the line before litterally means something went wrong with keystone15:52
markus_zkashyap: uh oh spaghettio, another proof that I'm a lazy ass15:52
bauzasie. keystone giving you back a 40115:52
EmilienMbauzas: with cdent, we're trying to figure if keystone resources are correctly created, but I doubt this is the issue. Keystone resources for placement have been here for a while and they work15:52
kashyapmarkus_z: No, no, you're not!15:52
*** voelzmo has joined #openstack-nova15:52
markus_z;)15:52
kashyapmarkus_z: I'm actually burnt looking at it, as it's been dragged for over a year15:52
EmilienMbauzas: that's what log says, not sure it's true.15:53
*** voelzmo has quit IRC15:53
kashyap(It is spun off from a different review.)15:53
EmilienMbauzas: the keystone resources are ready *after* compute service starts15:53
*** tbachman is now known as tbachmanAfk15:53
*** kaisers has joined #openstack-nova15:53
kashyapmarkus_z: I think Michele should engage in IRC here if he really wants it in, and solicit feedback, and answer questions in real-time15:53
*** udesale has quit IRC15:53
EmilienMbauzas: dansmith fixed the issue by adding a loop, it worked but now it fails to register in the loop15:53
bauzasEmilienM: mmm, okay15:53
kashyapToo much lag via email I feel.15:53
bauzasEmilienM: yeah, exactly, we changed from a try-once into a recheck15:53
bauzasokay, so it would fail once but works after that15:54
*** voelzmo has joined #openstack-nova15:54
markus_zkashyap: Want me to have a look at it?15:54
*** eharney has joined #openstack-nova15:54
kashyapmarkus_z: If you have time, sure.  But I am willing to test it and see what falls through.  Just spinning up a new DevStack env with this patch on my slow test machine next to me15:55
EmilienMbauzas: yes, it should (and it did a few days ago) work when puppet creates keystone resources for placement (service, endpoint, etc)15:55
EmilienMbauzas: in a few min, I'll have output of keystone resources15:55
kashyapmarkus_z: But if you have other priority things, feel free to ignore it.  Because, if the submitter like to get this merged, they should engage more actively as well15:56
bauzasEmilienM: what looks weird to me is that we don't have "Updated inventory" logs in http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_32_39_09215:56
markus_zkashyap: I put it on my todo list for tomorrow.15:56
markus_zkashyap: TBH, having a test job for that is far too long on my todo list but I tabled it week after week. :-/15:56
kashyapSure.  markus_z - Just a heads-up: Tomorrow I'll only be working til 13 CET.  Then I'll be AFK, travelling the whole day15:56
bauzasEmilienM: just a brief explanation, we basically cache the list of RPs we create in a memory object to reduce the number of HTTP calls15:56
markus_zkashyap: cool, no biggie15:57
bauzasEmilienM: the log you mention is because we can't find the mapping in the cache for the RP15:57
kashyapmarkus_z: You're not alone in feeling that guilt, I know it all too well15:57
bauzasEmilienM: if the cache key corresponding to the compute is not found, that's probably because we failed updating the placement service, hence that log15:57
*** Sukhdev_ has joined #openstack-nova15:57
*** mdrabe has quit IRC15:58
bauzasEmilienM: if we would update that placement service, we put the result in the cach15:58
bauzascache15:58
*** marst_ has joined #openstack-nova15:58
*** _ducttape_ has joined #openstack-nova15:58
EmilienMbauzas: what could cause this mapping failing?15:59
bauzasEmilienM: excellent question16:00
mriedemmelwitt: so on this auto-register thing for cells, i just remembered that nova-compute actually does already make calls to nova-api https://github.com/openstack/nova/blob/045f08ab8a57b90b00e4777ca71a20b5603c296c/nova/compute/manager.py#L642116:00
bauzasEmilienM: that I'm trying to answer by looking at the logs16:00
mriedemnot even through rpc16:00
bauzasEmilienM: mmmm http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_31_39_86216:01
*** Jeffrey4l has quit IRC16:01
EmilienMbauzas: I think this one is because the keystone resources are not created yet16:01
*** ducttape_ has quit IRC16:01
*** Jeffrey4l_ has joined #openstack-nova16:01
EmilienMbauzas: let's see timestamps, gimme a sec16:02
EmilienMhttp://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_31_39_86216:02
*** Jeffrey4l_ is now known as Jeffrey4l16:02
EmilienMso puppet managed keystone resources at 17h33 but nova compute tries to reach them at 17h3116:02
melwittmriedem: looking ... compute/api will call the rpcapi next though16:02
EmilienMbauzas: nova compute should retry later no?16:03
bauzasEmilienM: I'm verifying that16:03
bauzasin theory, we auto-heal every 60 secs16:03
EmilienMeven if we could say "tell puppet to create keystone resource BEFORE starting nova compute", this is odd and nova-compute should retry later16:04
*** mingyu has quit IRC16:04
melwittmriedem: I don't think that's calling nova-api though, is it? self.compute_api is an interface to making that rpc call, which will I guess just call the same nova-compute16:04
*** yellow has joined #openstack-nova16:04
bauzasEmilienM: we do16:04
bauzasEmilienM: ie. the failing trace I showed you comes from a method that is running every 60 secs16:05
openstackgerritArtom Lifshitz proposed openstack/nova: Plumbing for tagged nic attachment  https://review.openstack.org/41562216:05
openstackgerritArtom Lifshitz proposed openstack/nova: Libvirt support for tagged volume attachment  https://review.openstack.org/41810016:05
openstackgerritArtom Lifshitz proposed openstack/nova: Plumbing for tagged volume attachment  https://review.openstack.org/39194116:05
openstackgerritArtom Lifshitz proposed openstack/nova: API support for tagged device attachment  https://review.openstack.org/37602116:05
openstackgerritArtom Lifshitz proposed openstack/nova: Libvirt support for tagged nic attachment  https://review.openstack.org/41552016:05
bauzasEmilienM: and that method is creating the RP in memory16:05
*** nkrinner is now known as nkrinner_afk16:05
bauzasEmilienM: but I'm trying to identify all the workflow16:05
*** igordcard has quit IRC16:05
bauzasso we could possibly have missed something16:06
bauzasin case keystone is back after a flip16:06
bauzass/flip/blip16:06
*** larainema has quit IRC16:06
*** burgerk has joined #openstack-nova16:07
*** mjura has quit IRC16:07
*** voelzmo has quit IRC16:07
*** igordcard has joined #openstack-nova16:07
*** mdrabe has joined #openstack-nova16:08
*** sacharya has joined #openstack-nova16:09
*** jvgrant has joined #openstack-nova16:10
openstackgerritVasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441316:10
openstackgerritVasyl Saienko proposed openstack/nova: Ensure we mark baremetal links as phy links  https://review.openstack.org/42206816:10
*** ijw has joined #openstack-nova16:10
*** markvoelker has quit IRC16:11
bauzasEmilienM: I got a shit ton of "placement API endpoint not found" even after keystone is started http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/nova/nova-compute.txt.gz#_2017-01-17_17_38_02_28216:12
markus_zadreznec: I think I have it up and running http://nova-dpm.readthedocs.io/en/latest/ Thanks again for your time!16:13
bauzasEmilienM: correct me if I'm wrong but you told me that keystone is ready by 17:35 ?16:13
bauzasoh, even 17:3316:13
bauzas"(17:02:45) EmilienM: so puppet managed keystone resources at 17h33 but nova compute tries to reach them at 17h31"16:13
bauzasthat's... weird16:13
adreznecmarkus_z: Awesome, looks good!16:14
bauzasEmilienM: because it seems we litterally fail to find the correct endpoint even after that16:14
openstackgerritEric Fried proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv)  https://review.openstack.org/40940116:14
bauzasEmilienM: ie. keystoneclient returns an EndpointNotFound exception16:14
EmilienMbauzas: 17h3316:14
EmilienMbauzas: let's see how my endpoint is structured16:15
EmilienMmaybe it's wrong16:15
*** zul has quit IRC16:15
*** zul has joined #openstack-nova16:15
*** rmart04 has quit IRC16:15
EmilienMbauzas: http://paste.openstack.org/show/H7pXYOKWuBjQROqkZjQm/16:16
*** jose-phillips has quit IRC16:16
* bauzas looking16:16
EmilienMcdent: ^16:16
* cdent looks16:17
bauzasEmilienM: maybe the placement service failed to start16:17
bauzasbecause the URL seems correct16:17
*** rama_y has joined #openstack-nova16:17
EmilienMbauzas: apache would have failed16:17
*** rama_y_ has joined #openstack-nova16:17
EmilienMlet's see apache logs16:17
cdentthe wsgi process shows up in ps.txt16:17
* bauzas shrugs16:17
*** yamahata has joined #openstack-nova16:18
EmilienMyeah16:18
bauzasI mean, something is wrong with that HTTP call16:18
cdentI'll need to review the wsgi config, but I'm surprised to see: http://127.0.0.1:8778/placement instead of either16:18
bauzasso, either it's the service catalog which is incorrect or the placement service wsgi stack not fully up16:18
cdenthttp://127.0.0.1/placement or http://127.0.0.1:8778/16:18
cdenthowever16:18
bauzascdent: you're correct, that's the reason16:18
*** markus_z has quit IRC16:18
bauzasEmilienM: I need to see your apache conf16:19
cdentexcept that the exception message is that the endpoint is never found16:19
bauzasbecause I guess the root cause is ^16:19
openstackgerritGábor Antal proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622516:19
bauzascdent: yeah, because it could fail16:19
cdentand the placement-api logs never start, which suggests the wsgi app is never contacted16:19
bauzasI mean getting the VirtualHost16:19
bauzascdent: exactly16:19
bauzasFWIW http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/apache_config/10-placement_wsgi.conf.txt.gz16:19
openstackgerritScott DAngelo proposed openstack/nova: Fix Nova to allow using cinder v3 endpoint  https://review.openstack.org/42207816:19
*** CristinaPauna has quit IRC16:20
bauzasI think this configuration is incorrect16:20
cdentbauzas: except that is the same as a previous working config, I _think_16:20
cdentbut that ought to be confirmed16:21
EmilienMso if I look at Puppet CI this morning it seems to work but not in TripleO16:21
EmilienMI don't see placement-api logs http://logs.openstack.org/98/417898/15/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/ee7ff1b/logs/overcloud-controller-0/var/log/nova/16:21
EmilienMbut I know the vhost has been created, see http://logs.openstack.org/98/417898/15/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/ee7ff1b/logs/overcloud-controller-0/var/log/httpd/16:21
*** weshay is now known as weshay_afk16:21
cdentthe critical point on the placement-api logs is that they will not exist until at least one request is made to the placement API16:22
cdentwhether or not it is up16:22
EmilienMok16:22
EmilienMeg: when you spawn a VM, I guess16:22
bauzasyup16:23
bauzashttps://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template is the devstack config16:23
bauzasthere is an alias that is out of the virtualhost16:23
EmilienMso in tripleo, we spawn overcloud nodes with nova/ironic and i don't see placement logs: http://logs.openstack.org/98/417898/15/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/ee7ff1b/logs/undercloud/var/log/nova/16:23
*** jheroux has quit IRC16:23
*** gongysh has quit IRC16:23
bauzashence why you get either :8778/ or :80/placement16:23
bauzaswe share the same wsgi process app, but that's two different directives16:24
*** salv-orl_ has joined #openstack-nova16:24
EmilienMbecause we use HAproxy and 80 is used for horizon16:24
bauzasEmilienM: yeah, got the problem16:24
bauzasEmilienM: in devstack, we alias on / for the vhost https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template#L816:24
*** tbachmanAfk is now known as tbachman16:24
bauzasEmilienM: in http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/apache_config/10-placement_wsgi.conf.txt.gz you alias on /placement16:25
openstackgerritGábor Antal proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622516:25
bauzasbut you don't provide the location directive16:25
EmilienMyep16:25
cdentif that's the case why did it work before?16:25
EmilienMyeah and I just noticed it works in some CI jobs this morning16:25
*** Guest88977 is now known as jlvillal16:25
EmilienMlet me grab the more recent one16:25
*** bvanhav has quit IRC16:26
EmilienMhttp://logs.openstack.org/80/421980/3/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/d4b019b/logs/nova/16:26
*** efoley has joined #openstack-nova16:26
cdentI wonder if the problem here is that we don't understand well enough how keystoneauth1 is working, and thus the pathways to that exception we are seeing may be more complex than we know?16:26
*** bvanhav has joined #openstack-nova16:27
*** salv-orlando has quit IRC16:27
*** sean-k-mooney has joined #openstack-nova16:28
_gryfseems similar to https://bugs.launchpad.net/nova/+bug/165607516:28
openstackLaunchpad bug 1656075 in OpenStack Compute (nova) "DiscoveryFailure when trying to get resource providers from the scheduler report client" [Low,Confirmed] - Assigned to Roman Dobosz (roman-dobosz)16:28
bauzasEmilienM: looks all good for that one http://logs.openstack.org/80/421980/3/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/d4b019b/logs/nova/nova-compute.txt.gz#_2017-01-18_15_44_36_29316:28
EmilienMok so good news: it works in Puppet OpenStack CI this morning. Now trying to figure why it doesn't work in TripleO16:28
EmilienMbauzas: I swear it didn't work yesterday16:29
bauzas:)16:29
bauzasI haven't touched anything !16:29
EmilienMif you look at https://bugs.launchpad.net/tripleo/+bug/165746716:29
openstackLaunchpad bug 1657467 in tripleo "placement: unable to refresh compute resource provider record" [High,Triaged]16:29
EmilienMlogs are from yesterday and it didn't work16:29
bauzasI'd look at the vhost config16:29
mriedemmelwitt: yeah good point, nova.compute.manager calling nova.compute.api to call compute rpcapi to cast the16:29
bauzasand compare both16:29
mriedem'stop' back to the same compute16:29
EmilienMbauzas: we didn't change it16:29
*** _ducttape_ has quit IRC16:29
*** ducttape_ has joined #openstack-nova16:30
mriedemmelwitt: i was thinking if that already was in place the compute manager could call the compute api to register with the cell mapping (in the single cell case) in the api db, but that's not really going to work16:30
mriedemunless the compute rpcapi did a cast to something else that has access to the api, like conductor16:31
cdentEmilienM, bauzas: I'll say again: [t 43Pk]16:31
purplerbot<cdent> I wonder if the problem here is that we don't understand well enough how keystoneauth1 is working, and thus the pathways to that exception we are seeing may be more complex than we know? [2017-01-18 16:26:39] [n 43Pk]16:31
bauzascdent: well, I agree with you on the point that EndpointNotFound is kinda obscure to me16:31
bauzascdent: I'm supposing that keystone failed finding the endpoint16:32
bauzasbut the endpoint can be /16:32
sdaguebauzas: it's not that keystone failed16:32
cdentit happens at two different places in the keystoneauth1 stack16:32
cdentonce when actually making a request and filtering the results16:32
melwittmriedem: I think if something in compute queried cell mapping through the object, it would go through conductor and be able to read from the api db16:32
cdentbut also when trying to construct a url16:32
bauzassdague: I'm all ears :)16:32
sdagueit's that the resolution of the endpoint from the token ended up with something unknown16:32
bauzasa-ha16:33
*** jheroux has joined #openstack-nova16:34
bauzassdague: I don't see any problem with http://paste.openstack.org/show/H7pXYOKWuBjQROqkZjQm/16:34
mriedemmelwitt: hmm, so like, list cell mappings, if one, get that one and register a host with it?16:35
mriedems/a host/my host/16:35
bauzassdague: except the fact that they call http://ip:8778/placement16:35
cdentsdague: by "resolution" do you mean looking at what's at the endpoint, for finding the endpoint in what's in the token?16:35
cdents/for finding/or finding/16:35
bauzasI thought the latter16:35
*** ducttape_ has quit IRC16:36
sdaguecdent: it's looking through the token16:36
sdaguethe end point information isn't an api call16:37
*** _ducttape_ has joined #openstack-nova16:37
*** JoseMello has joined #openstack-nova16:37
sdagueit's all embedded in the token16:37
melwittmriedem: it seems like that would work. technically it would be two cell mappings because of cell016:37
mriedemmelwitt: true, so list cell mappings, exclude cell0 by uuid, and if one left, register with it16:38
*** weshay_afk is now known as weshay_lunch16:39
*** dikonoor has joined #openstack-nova16:39
mriedemmelwitt: i'm not sure why we didn't think of just auto-registering that way, unless we wanted to auto-register in a more generic way for the multi-cell case16:39
melwittmriedem: I think so16:39
*** harlowja has joined #openstack-nova16:39
mriedembut it seems we could do the simple thing for now and hit most small/medium deployment scenarios that don't have any cells today16:40
mriedemidk16:40
mriedemjust thinking of options16:40
*** jose-phillips has joined #openstack-nova16:40
*** mlavalle has joined #openstack-nova16:40
mriedemmelwitt: not sure if you saw the discussion here https://review.openstack.org/#/c/420981/16:40
melwittmriedem: yeah, I dunno the history on the auto-register stuff. it's not straightforward for multi-cell because of the need for a "staging cell" and then a command to re-map something. for single-cell, I didn't know what the original plan was16:41
*** andreas_s has quit IRC16:41
melwittI saw you refer to alaski's patch from some weeks ago, but I hadn't looked at it before16:41
cdentEmilienM, bauzas: so, where are we?16:42
EmilienMI'm currently in the matrix reading tripleo logs16:42
bauzashonestly, I'm puzzled16:42
mriedemmelwitt: yeah i was looking at it this morning, n-cpu does a new rest api call to n-api to register it's host with a cell mapping16:42
bauzasI'm trying to compare the req-id in nova and match it in the keystone logs16:42
mriedembut there are some parts in there about storing the parent cell uuid in the main db for some reason16:42
sdaguemriedem: is there a reason the conductor can't just do that?16:42
mriedemsdague: that's what melwitt and i are talking about16:43
sdaguemriedem: I'm not sure I get why n-cpu needs to make an n-api call16:43
sdagueok16:43
melwittyeah, I thought it could just go through conductor via objects16:43
sdagueok, got to run to lunch, biab16:44
cdentmriedem, sdague, bauzas: if this merges, all the functional tests pass in python35: https://review.openstack.org/#/c/421529/16:45
bauzascdent: EmilienM: FWIW, I'll need to drop very soon, I have a meetup to go16:45
mriedemso in the short-term i probably need to just look at making devstack-gate call back into the primary node at the end of the setup (before tempest) to call discover_hosts instead of doing this https://review.openstack.org/#/c/420981/16:45
* cdent takes a short walk16:46
melwittah, I see16:47
mriedem+W16:47
cdentthanks bauzas and mriedem . dims we have arrived in the future.16:47
mriedemmelwitt: yeah so sean's main point is that my patch relies on the fact that it sets up the computes after the controller (i think)16:47
mriedemas in i think that's the main point16:48
*** sheel has joined #openstack-nova16:48
melwittright16:48
mriedemregardless of whether or not i can rely on the db connection in nova.conf, because DATABASE_HOST is still on the subnode when it's setup so i could build my own one-off nova.conf for running discover_hosts if i wanted,16:48
bauzascdent: EmilienM: humpf, nothing looks wrong when I see the keystone logs http://logs.openstack.org/66/421366/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/f138bcc/logs/apache/keystone_wsgi_main_access_ssl.txt.gz16:48
mriedembut i think that defeats the point he's making16:48
bauzascdent: EmilienM: but I can't introspect the token16:49
bauzasso, honestly, I'm really stuck16:49
*** liusheng has quit IRC16:49
*** liusheng has joined #openstack-nova16:50
melwittmriedem: oh, to get to the db regardless of whether the controller is up yet? yeah16:50
dimscdent : yay :)16:51
openstackgerritAnusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload  https://review.openstack.org/41973016:51
mriedemmelwitt: right. i mean i'm not sure how many deployments actually setup computes first.16:52
*** dtp has joined #openstack-nova16:53
melwittmriedem: yeah. we didn't, computes were last/parallel after controllers16:53
openstackgerritSujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload  https://review.openstack.org/41918516:54
*** tuanla_ has joined #openstack-nova16:55
*** tuan_luong has quit IRC16:56
*** _ducttape_ has quit IRC16:56
*** ducttape_ has joined #openstack-nova16:56
*** edand has quit IRC16:58
*** salv-orl_ has quit IRC16:58
openstackgerritMatt Riedemann proposed openstack/nova: DNM: Test interface attach/detach for Ironic  https://review.openstack.org/42163817:00
mriedemwhy god does this fail functional tests on newton but is no different from the change on master https://review.openstack.org/#/c/416760/17:03
mriedemwhy hast thou forsaken my backport17:03
_gryfbauzas, cdent, EmilienM, does any of you was able to reproduce/isolate that error on devstack? I was spent some time yesterday, but without any luck17:03
*** Swami has joined #openstack-nova17:04
bauzas_gryf: which error ?17:04
bauzasthe one we discussed above?17:04
alaskimriedem: the reason for the cell_info is that a compute does not know what cell it is in. But it is configured to use a specific db. So it queries the db it knows about for its cell, then calls to the API and says "hey, I'm a compute in cell 'foo', register me"17:04
_gryfbauzas, yes17:04
bauzas_gryf: nope17:04
_gryfbummer17:04
alaskimriedem: that avoids the api cell from needing to go through and discover the new host17:04
_gryfI'll try to investigate it further anyway17:05
*** Swami_ has quit IRC17:06
*** sudipto has quit IRC17:07
*** sudipto_ has quit IRC17:07
*** rmart04 has joined #openstack-nova17:09
mriedemalaski: so that's plumbed into discover_hosts here https://review.openstack.org/#/c/369634/1/nova/cmd/manage.py@1530 in the case that the compute_nodes list is empty?17:09
mriedemi.e. if compute_nodes then we create the host mappings for them in discover_hosts, if not we provide the info into the cell db to auto-register itself when it creates the compute node17:09
gabor_antalHey there, can I have a question? In kilo, there was a config flag name "allow_migrate_to_same_host". But in liberty (and since then) it is disappeared. Why it was removed? How can I get something like that nowadays for testing?17:10
mriedemgabor_antal: allow_resize_to_same_host17:10
alaskimriedem: it wasn't fully fleshed out, but it was written under the assumption that discover_hosts would be run once per cell to set it up and then after that auto-registration would happen.17:11
alaskimriedem: if auto-registration is the path forward finding a way to avoid the initial discover_hosts would be preferable17:11
gabor_antalmriedem: thanks, checking that out.17:11
mriedemalaski: yeah we were talking about a simple auto-register in the single-cell case, which most deploys will be at first in ocata17:12
mriedemalaski: since we know that besides cell0, if there is just a single cell, it's the one we want the host mapping in when creating a compute node record17:12
mriedemalternatively we could mark a specific cell as the 'default' cell or 'staging' cell or something like that17:13
mriedemwhich new computes show up in17:13
mriedemanyway with <2 weeks to FF auto-registration is probably not going to happen this release anyway so i should probably stop thinking about it17:14
melwittwhen you say that, it makes me think we'd need a list_cells_and_hosts that shows what hosts are mapped to what cells17:14
*** rmart04 has quit IRC17:14
cdentmriedem: I've got a few cycles to poke at https://review.openstack.org/#/c/416760/ if you like?17:14
alaskimriedem: yeah, there's a fair amount to consider to get this done17:14
mriedemmelwitt: i think we could just do like a nova-manage cell_v2 list_hosts or something that shows which cell each host is in, and vice-versa17:14
alaskimelwitt: that seems generally useful anyways17:14
mriedemcdent: go nuts17:15
cdentjfak;fjdasopvicu9v817:15
mriedemcdent: i suspect i'm just missing something before that backport17:15
*** dtp is now known as dtp-afk17:15
melwittmriedem: yeah17:15
mriedemcdent: but i can recreate this locally http://logs.openstack.org/60/416760/1/check/gate-nova-tox-db-functional-ubuntu-xenial/f192837/testr_results.html.gz17:15
cdentk17:15
*** jdandrea has quit IRC17:16
alaskimriedem: melwitt the thing to consider with registration/discovery is that a compute is tied to a cell by the db it's configured to use. Meaning you can only put a compute in a cell that uses the same db that it does. So registering with a default/staging cell is just configuring it for that db initially17:16
alaskiand then moving it requires changing the db it uses17:16
melwittalaski: excellent point. so really a compute should just list cell mappings and register with the one it matches db connection with. (you probably said that earlier)17:18
mriedemwell,17:18
mriedemhttps://review.openstack.org/#/c/421885/17:18
mriedemcase in point ^17:18
mriedemyou shouldn't need CONF.database.connection in nova.conf on the compute17:18
mriedemauto-registering with a default or staging cell would require being able to move those hosts into other cells later, yeah17:19
melwittoh, heh17:19
melwittyeah, that's true17:19
mriedemgod, i can already see this requirement: i want to be able to live migrate my instances AND hosts out of cell A into cell B17:19
alaskimelwitt: if not for that review mriedem just posted then yeah, when I was playing around with it I just added the necessary info to the db https://review.openstack.org/#/c/369634/17:20
alaskimriedem: use what the DARE program taught you, just say no17:20
*** tbachman is now known as tbachmanAfk17:21
mriedemoh i was doing drugs immediately so DARE failed17:21
alaskiwell, it did teach you all about them17:21
*** tesseract has quit IRC17:21
mriedemi blame video games and rock music17:22
mriedemand nancy reagan17:22
*** nic has joined #openstack-nova17:22
alaskimoving instances and hosts between cells would definitely be great to have, but they need some thought and should happen after cellsv2 is actually used by most/all deployers17:22
gabor_antalmriedem: can you help me? I keep getting "UnableToMigrateToSelf: Unable to migrate instance (1bad3469-0843-4ce4-82b1-321a648415e3) to current host (compute).17:22
gabor_antal" error, while "allow_resize_to_same_host" is set True. It's in a unit test btw.17:22
*** Nisha_Agarwal has joined #openstack-nova17:23
*** Swami has quit IRC17:23
mriedemgabor_antal: you have self.flags(allow_resize_to_same_host=True) in the unit test?17:23
gabor_antalmriedem: Ofc. Sorry, it's actually a notification-checker functional test.17:24
*** browne has joined #openstack-nova17:24
mriedemgabor_antal: the flag gets set for the test the same way17:24
melwittjust have to make sure it's set before the service that reads it, is started17:25
gabor_antalyeah, I've already set it. But I keep getting that annoying error17:25
*** unicell has quit IRC17:25
*** tuanla_ has quit IRC17:27
gabor_antalmriedem: I tried to set it right after this line, https://github.com/openstack/nova/blob/master/nova/tests/functional/notification_sample_tests/test_instance.py#L32, but keep getting that ugly error :(17:28
mriedemgabor_antal: do you have a stacktrace you can put into a paste and link that here?17:29
*** smatzek has quit IRC17:29
mriedemgabor_antal: btw,17:29
mriedemsee https://github.com/openstack/nova/blob/master/nova/tests/functional/notification_sample_tests/test_instance.py#L48017:29
*** yellow has quit IRC17:29
gabor_antalmriedem: I paste it in a second. I already checked those tests, but doing live migration tests still leads me to the same error, I think we have 3 patches which is stuck on this point17:30
*** lpetrut has quit IRC17:31
openstackgerritClaudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps  https://review.openstack.org/42096017:31
gabor_antalmriedem: http://paste.openstack.org/show/595439/17:32
*** jose-phillips has quit IRC17:32
*** tbachmanAfk has quit IRC17:32
*** harlowja has quit IRC17:33
siva_krishnantdurakov: This is regarding https://review.openstack.org/#/c/379032/2.  I just replied to your comments. Please let me know what I am planning to do looks acceptable17:33
*** Nisha_Agarwal has left #openstack-nova17:33
*** baoli has joined #openstack-nova17:33
mriedemjroll: i'm having a hard time understanding where this change https://review.openstack.org/#/c/364413/22/nova/virt/ironic/patcher.py is used in the rest of that patch17:34
*** a-pugachev has joined #openstack-nova17:34
*** sahid has quit IRC17:35
jrollmriedem: we moved the binding of the neutron port into ironic, since we do weird things with timing on that https://github.com/openstack/ironic/blob/e72e68eb38c4325bb868a504106be097251c3b5e/ironic/drivers/modules/network/flat.py#L6817:35
*** rama_y_ has quit IRC17:36
*** rama_y has quit IRC17:36
jrollmriedem: rather than sometimes passing back the host id to the compute manager (to bind it then), and sometimes passing None back (to bind it later in ironic)17:36
jrollnow we just always bind it in ironic17:37
jroll"binding it" meaning give it binding:host_id, which triggers ML2 stuff to actually connect it17:37
*** david-lyle is now known as bailing-wire17:37
mriedemok so now when creating a node you need to provide the instance.host17:37
mriedemand this patcher thing does that17:37
*** markvoelker has joined #openstack-nova17:38
jrollyeah, that patcher thing passes info we need from nova17:38
*** mvk has joined #openstack-nova17:38
mriedemok https://github.com/openstack/ironic/blob/e72e68eb38c4325bb868a504106be097251c3b5e/ironic/drivers/modules/network/flat.py#L68 clears it up17:38
mriedemthanks17:38
jrollI agree with vasyl we should clean that up later and just use the node uuid17:38
jrollnp17:38
*** tbachman has joined #openstack-nova17:40
mriedemalso, docs people want you guys to specifically say 'ironic' instead of 'Ironic'?17:40
mriedemthat seems weird17:40
mriedemi'd think that would be confusing to people new to openstack17:40
cdentmriedem: this https://review.openstack.org/#/c/416761/ needs to come before https://review.openstack.org/#/c/416760/ in that stack of fixes17:40
*** a-pugachev has quit IRC17:41
mriedemcdent: ah, well that's easy enough17:41
cdentmriedem: docs people want lower case project names, I've never seen the reason why, but I also though they wanted service names instead of project names in docs?17:41
mriedemyeah the idea is Compute service, Bare Metal service, Networking service, etc17:42
mriedemwhat happens when they get to Monitoring service :)17:42
cdenteveryone runs and hides17:42
mriedemImage Repository Service17:42
*** armax has joined #openstack-nova17:43
cdentmriedem: you want me to reorder that stuff, or you got it?17:43
*** catintheroof has quit IRC17:44
mriedemcdent: i'll get it, i have to rebase the last 2 in the series anyway now due to merge conflicts17:44
cdent✔17:44
* cdent thinks about dinner17:44
*** catintheroof has joined #openstack-nova17:44
*** markvoelker_ has joined #openstack-nova17:44
*** catintheroof has quit IRC17:44
*** Apoorva has joined #openstack-nova17:45
*** catintheroof has joined #openstack-nova17:45
*** dtp-afk is now known as dtp17:45
*** smatzek has joined #openstack-nova17:46
*** markvoelker has quit IRC17:46
*** bailing-wire has quit IRC17:46
mriedemjroll: ok so begins the elusive hunt for that 2nd core https://review.openstack.org/#/c/364413/17:46
*** mugsie has left #openstack-nova17:47
*** lucasagomes is now known as lucas-afk17:48
*** amoralej is now known as amoralej|off17:48
*** catintheroof has quit IRC17:50
*** karimb has quit IRC17:51
*** CristinaPauna has joined #openstack-nova17:51
*** gszasz has quit IRC17:51
*** beagles is now known as beagles-brb17:51
*** weshay_lunch is now known as weshay17:53
gabor_antalmriedem: Did you have any time to think about my problem or so? If it would be better, I can commit what I've done on the patchset, but it is not mine, just wanted to review.17:55
*** armax_ has joined #openstack-nova17:57
*** armax has quit IRC17:57
*** armax_ is now known as armax17:57
*** markvoelker has joined #openstack-nova17:58
*** shewless has joined #openstack-nova17:59
*** derekh has quit IRC18:01
*** markvoelker_ has quit IRC18:01
shewlesshello. does anyone know what user the  oslo_service.service runs as for nova?18:01
shewlessTrying to get SSL to work.. but apparently it can't the key_file.. I think it's because the key file is owned by root18:02
mriedemgabor_antal: you can't live migrate to the same host18:02
mriedemonly resize18:02
mriedemgabor_antal: so what i think you're trying to test isn't going to work this way18:03
shewlessI guessed nova but I don't think that's right18:03
*** owalsh is now known as owalsh-afk18:04
mriedemgabor_antal: so for testing you'd need 2 fake compute services, boot an instance on one and live migrate it to the other18:04
*** ralonsoh has quit IRC18:05
gabor_antalmriedem: I thought that way also, but in the kilo, there were a conf flag to use the same host to migrate (for testing), http://docs.openstack.org/kilo/config-reference/content/list-of-compute-config-options.html18:05
gabor_antalmriedem: do you know about any examples which works like you mentioned?18:05
shewlessif I open up the permissions of my "private" SSL cert directory so that "everyone" has read access then I get past that error.. so that's how I know it's a permission problem18:06
*** raunak has joined #openstack-nova18:07
mriedemgabor_antal: not without digging18:07
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs  https://review.openstack.org/42120918:08
*** owalsh-afk has quit IRC18:08
shewlessSo to summarize I found all the nova.conf settings to enable nova-api to be SSL but I don't know what user/permissions to use for my SSL keys.18:09
shewlessI have the users/permissions setup such that they work with apache but nova will have non of it18:09
*** owalsh-afk has joined #openstack-nova18:09
*** efoley has quit IRC18:09
*** owalsh-afk is now known as owalsh18:09
shewlesscan anyone point me in the right direction?18:10
*** Swami has joined #openstack-nova18:10
mriedemgabor_antal: btw this is the patch that removed the allow_migrate_to_same_host option in liberty https://review.openstack.org/#/c/118604/ - it was never for live migration18:10
gabor_antalmriedem: thanks for the info18:11
mriedemshewless: there is nothing in here http://docs.openstack.org/security-guide/compute.html ?18:12
gabor_antalmriedem: you mean something like this? https://github.com/openstack/nova/blob/master/nova/tests/functional/test_server_group.py18:12
mriedemgabor_antal: yeah something like this https://github.com/openstack/nova/blob/master/nova/tests/functional/test_server_group.py#L11418:12
*** bvanhav has quit IRC18:12
shewlessmriedem: no nothing that goes down that low.18:13
shewlessmriedem: the error I see is  ERROR oslo_service.service RuntimeError: Unable to find key_file : /etc/pki/tls/private/mykey.key18:13
shewlessmriedem: if I change the private directory to 777 just to test I get past that error and hit another silly permission error18:14
shewlessERROR oslo_service.service     self._context.load_cert_chain(certfile, keyfile) ERROR oslo_service.service IOError: [Errno 13] Permission denied18:15
shewlessI assume this oslo_service is either running as root or nova.. but that doesn't seem to be the case because I've set the "read" permissions on the files so that both nova and root can access them18:15
*** bvanhav has joined #openstack-nova18:17
*** gcb has quit IRC18:18
shewlesssigh. I can't get past this error even if I open up the permisions on my certs for everyone: ERROR oslo_service.service     self._context.load_cert_chain(certfile, keyfile) ERROR oslo_service.service IOError: [Errno 13] Permission denied18:22
mriedemshewless: clarkb got TLS working in the upstream CI environment, he might be able to help (maybe in #openstack-dev), else i'd probably ask in #openstack-operators or just #openstack18:23
*** jerrygb has quit IRC18:23
shewlessmriedem: thanks. SSL support for nova seems to be pretty dodgy..18:23
mriedemis this just nova or all of your services though?18:23
mriedemi'd expect this to be universal18:23
*** unicell has joined #openstack-nova18:23
shewlessmridem: nova so far18:23
shewlessI have keystone working perfectly18:24
shewlessmind you that's through apache18:24
shewlessit just works18:24
mriedembut glance/cinder/neutron too?18:24
shewlessnova has been a pain in my side18:24
*** CristinaPauna has quit IRC18:25
shewlessmy current goal is to get SSL to work for horizon (done), keystone (done), nova and spice. I haven't tried glance or neutron. But I'm dealing specifically with nova.conf.. not sure if neutron would have the same options18:25
mriedemwell,18:26
mriedemthe ssl options come from oslo.service I think18:26
mriedemso that's why i figured it'd be a universal problem,18:26
mriedemhaving said that, i think nova.conf has several different ssl-related options which gets confusing18:26
*** yamahata has quit IRC18:26
shewlessit sure does18:26
shewless(get confusing)18:27
*** sacharya has quit IRC18:27
gabor_antalmriedem: thanks, I try to do the job. I'll write you, if I have some problem18:27
mriedemshewless: sorry. if you find an answer it'd be great to get that fed back into the docs - not entirely sure which docs those are, since i'm finding several related to security in docs.o.o18:27
mriedemdane-fichter (not in channel atm) would be another good resource18:28
mriedemdane was working on getting a security-specific CI job configured for testing the various security-related features/configurations in openstack, like signed images, using barbican for key mgmt, etc18:28
mriedemso he knows infinitely more about this than i do18:29
shewlessokay thanks18:29
openstackgerritMatt Riedemann proposed openstack/nova: DNM: Test interface attach/detach for Ironic  https://review.openstack.org/42163818:29
shewlessah crap. restarting nova-api seemed to get rid of the error :( now to find out which change I made actually fixed the problem!18:31
mriedemha18:31
mriedemmy guess would be a config change?18:31
mriedemrestarting nova-api would pick up a config change18:31
mriedemcdent: got that series all cleaned up now, thanks for helping me out there18:32
cdentyou're very welcome18:32
shewlessquite possibly.. there are two sets of ssl options advertised in nova.conf.. I removed 1 of them.. it was either that or the file permission changes I made18:32
*** rama_y_ has joined #openstack-nova18:32
*** rama_y has joined #openstack-nova18:32
shewlessmriedem you know it seems to be the file permissions.. when I change them I seem to need to restart nova-api for that error to go away18:34
*** beagles-brb is now known as beagles18:34
shewlessanyways.. thanks.. I'm further on now18:35
*** krtaylor has quit IRC18:36
*** CristinaPauna has joined #openstack-nova18:36
*** jpena is now known as jpena|off18:39
*** karimb has joined #openstack-nova18:50
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: Raise general error message when volume attach fails  https://review.openstack.org/32016318:53
*** dharinic is now known as dharinic|lunch18:55
*** bvanhav has quit IRC18:58
*** Sukhdev_ has quit IRC18:59
mriedemhttps://review.openstack.org/#/c/398473/ needs another core19:00
mriedemwould get us down to 1 remaining change for ironic + resource classes19:00
mriedemjaypipes: john is +2 on the osprofiler change now, which you were +2 on before if you are still https://review.openstack.org/#/c/254703/19:01
*** woodster_ has joined #openstack-nova19:01
mriedemand when you get a break19:01
*** krtaylor has joined #openstack-nova19:03
_gryfEmilienM, bauzas, cdent, mriedem, doesn't placement API supposed to run on 80 port? if so, than this apache vhost is configured somehow wrong, doesn't it? http://logs.openstack.org/04/419604/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/5d95a8c/logs/apache_config/10-placement_wsgi.conf.txt.gz19:03
_gryfor maybe it is not relevant at all?19:03
EmilienMthis is not relevant, you can run an API how you like on the port you like19:04
cdent_gryf: how devstack configures things and how puppet configures things is not necessary going to be the same19:04
_gryfokay19:04
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850919:08
openstackgerritDiana Clarke proposed openstack/nova: Fix map_cell_and_hosts help  https://review.openstack.org/42217119:09
*** ociuhandu has quit IRC19:10
*** gszasz has joined #openstack-nova19:10
openstackgerritVasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups  https://review.openstack.org/38875619:10
*** gszasz has quit IRC19:10
*** gszasz has joined #openstack-nova19:11
openstackgerritDean Troyer proposed openstack/python-novaclient: DNM: another OSC+SDK check  https://review.openstack.org/42217419:11
*** cdent has quit IRC19:14
*** masayukig_ has joined #openstack-nova19:15
*** masayukig has quit IRC19:15
*** bvanhav has joined #openstack-nova19:16
*** masayukig_ is now known as masayukig19:16
*** masayukig is now known as masayukig_19:16
*** masayukig_ is now known as masayukig19:16
*** adisky_ has quit IRC19:19
*** burt has joined #openstack-nova19:19
*** Oku_OS is now known as Oku_OS-away19:19
jaypipesmriedem: done.19:21
*** ociuhandu has joined #openstack-nova19:21
*** jerrygb has joined #openstack-nova19:24
*** yamahata has joined #openstack-nova19:27
*** salv-orlando has joined #openstack-nova19:27
*** jerrygb has quit IRC19:29
*** eglynn has quit IRC19:31
*** dikonoor has quit IRC19:31
*** tjones has joined #openstack-nova19:32
*** sacharya has joined #openstack-nova19:33
mriedemjroll: can you have the same vif id in a list of ports and a list of portgroups?19:37
mriedemfor the same node?19:37
*** Sukhdev has joined #openstack-nova19:37
*** avolkov has quit IRC19:40
*** dharinic|lunch is now known as dharinic19:42
*** baoli has quit IRC19:42
*** jidar has left #openstack-nova19:45
*** voelzmo has joined #openstack-nova19:48
mriedemvdrok: i've got some questions in https://review.openstack.org/#/c/388756/19:53
jrollmriedem: no, a vif id can only be attached to one thing, AIUI19:53
mriedemjroll: question the 2nd: can you have a port in more than one portgroup for the same node?19:54
jrollmriedem: no19:55
*** CristinaPauna has quit IRC19:55
mriedem3rd: who is your daddy and what does he do?19:55
*** siva_krishnan has left #openstack-nova19:57
jroll:P19:58
vdrokmriedem: thanks, a bit late now, will post replies tomorrow. Short version - duplicates are not possible because of the way nova generates id ( was asked by rloo in previous PS ) and cloud init does not validate parameters :)19:59
mriedemok19:59
*** bailing-wire has joined #openstack-nova20:00
*** rfolco has quit IRC20:01
*** bailing-wire is now known as david-lyle20:02
*** markvoelker_ has joined #openstack-nova20:03
*** salv-orlando has quit IRC20:04
*** jerrygb has joined #openstack-nova20:05
*** markvoelker has quit IRC20:05
*** CristinaPauna has joined #openstack-nova20:09
*** krtaylor has quit IRC20:10
*** Guest62565 has quit IRC20:12
wasmuminstance build from an image - running fine.  needed a reboot.  The original image is no longer in the image store and thus instance erroring out.  Any way to resolve that?20:13
*** jerrygb has quit IRC20:13
mriedemwasmum: did you snapshot the instance?20:13
mriedemdo you have the image in the image cache on the compute host?20:13
wasmummriedem: image cache, tell me more?20:14
*** masayukig is now known as masayukig_20:15
mriedemif you're using the libvirt driver, it will store the base image on the compute host the first time it uses it so it doesn't have to re-download commonly used images that build on the same host20:16
*** Guest62565 has joined #openstack-nova20:16
mriedemi believe the image cache will clean after itself after there are no instances using a given image in the cache on that host though after some period of time20:16
wasmumwhere does it pull that image down too?20:17
wasmumfile location?  I'll check ?20:17
mriedemshould be under the instance state path, so i think /var/lib/instances somewhere20:17
openstackgerritSergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class  https://review.openstack.org/39448020:17
openstackgerritSergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API.  https://review.openstack.org/35554020:17
openstackgerritSergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen  https://review.openstack.org/39961320:17
*** Swami has quit IRC20:18
mriedemsee the image_cache_manager_interval config option20:18
mriedemand image_cache_subdirectory_name20:18
mriedemand cache_images20:18
*** Matias has quit IRC20:18
mriedemare you using local disk or shared?20:19
cfriesen__does the separation of some models out into api_models.py have any implications for upgrades?  or are they still the same DB tables, just moving the python code around?20:20
mriedemmdbooth: any idea if we have any decent documentation anywhere about the image cache manager?20:20
mriedemcfriesen__: such as?20:20
mriedemflavors?20:20
mriedemcfriesen__: api_models is the API DB schema20:20
mriedemsome tables are the same or similar, like flavors / instance_types20:21
mriedemcfriesen__: the models are not entirely the same though, i.e. the API DB doesn't use the soft delete mixin (the deleted and deleted_at columns are not in the API DB)20:21
mriedemcfriesen__: the nova-manage db online_data_migrations is meant to move the data from the cell db to the api db20:21
mriedems/cell/nova/20:21
openstackgerritSergey Nikitin proposed openstack/nova: DO NOT MERGE  https://review.openstack.org/42222120:22
cfriesen__mriedem: I was looking at InstanceGroup in particular20:22
*** READ10 has quit IRC20:23
cfriesen__mriedem: a cursory check didn't see any code to migrate InstanceGroup data from the old model to the new20:23
*** cfriesen__ is now known as cfriesen20:24
*** Matias has joined #openstack-nova20:24
*** krtaylor has joined #openstack-nova20:25
*** jaypipes has quit IRC20:26
mriedemcfriesen: https://github.com/openstack/nova/blob/master/nova/objects/instance_group.py#L52520:26
mriedemcfriesen: called from here https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L61220:27
mriedemwhich is initially called from, let's all say it now, https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L76320:27
cfriesenmriedem: much appreciated20:29
mriedemyw20:29
*** bvanhav has quit IRC20:30
*** CristinaPauna has quit IRC20:32
*** markvoelker_ has quit IRC20:32
*** karimb has quit IRC20:35
*** _ducttape_ has joined #openstack-nova20:36
wasmummriedem: looks like it was built from a snapshot, snapshot was deleted20:36
wasmumlearning lessons on why not to build snowflakes20:36
*** edmondsw has quit IRC20:36
mriedemwasmum: and the image isn't cached on the compute host?20:36
wasmumlooks like it was.  tried to "start" the instance using virsh and got the error of the missing backup file20:37
wasmumlooks like it was in the instances path in a "_base" directory and that must have expired?20:38
*** voelzmo has quit IRC20:38
mriedemnot sure20:38
mriedemthere is a related write up from kashyap here https://kashyapc.com/2013/03/03/openstack-nova-dealing-with-unused-base-images/20:38
*** ducttape_ has quit IRC20:39
*** CristinaPauna has joined #openstack-nova20:43
*** baoli has joined #openstack-nova20:44
openstackgerritSergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API.  https://review.openstack.org/35554020:45
openstackgerritSergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen  https://review.openstack.org/39961320:45
wasmummriedem: thanks a ton20:46
mriedemyw20:46
*** voelzmo has joined #openstack-nova20:46
openstackgerritSujitha proposed openstack/nova: Adding auto_disk_config field to InstancePayload  https://review.openstack.org/41918520:48
*** annegentle has joined #openstack-nova20:49
*** voelzmo has quit IRC20:50
mriedemedleafe: i know you're on PTO but are you satisfied https://review.openstack.org/#/c/418134/6/nova/tests/functional/db/test_compute_node.py@102 ?20:52
mriedemi guess you're looking for a slight wrinkle there20:53
edleafemriedem: I was only on PTO Monday :)20:53
mriedemwe have 2 computes, you want 120:53
mriedemmake sure you get 1 and not 220:53
edleafemriedem: I wanted a case where only one node was returned. The current are either all or none20:53
mriedemyeah20:53
mriedemok20:53
*** jerrygb has joined #openstack-nova20:53
mriedemi'll add that quick20:54
edleafemriedem: I was going to add it myself, but I didn't want to mess up bauzas's stuff20:55
*** haplo37_ has quit IRC20:55
mriedemi've got it20:55
*** haplo37_ has joined #openstack-nova20:58
openstackgerritMatt Riedemann proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method  https://review.openstack.org/41813420:59
openstackgerritMatt Riedemann proposed openstack/nova: WIP: Scheduler calling the Placement API  https://review.openstack.org/41796120:59
mriedem+2 on that mofo now21:00
*** aojea has quit IRC21:00
edleafemriedem: kewl21:00
*** ijw has quit IRC21:01
mriedemi'm trying to work on my list of 'these bp patches need another +2' by the end of hte day21:01
*** jdurgin has quit IRC21:03
oomichialex_xu: yeah, the first one is good fo me, thanks Kevin_Zheng and alex_xu21:04
*** sree has joined #openstack-nova21:04
*** jheroux has quit IRC21:05
oomichialex_xu: https://review.openstack.org/#/c/415330/ has johnthetubaguy's -121:05
openstackgerritRuby Loo proposed openstack/nova: Generate necessary network metadata for ironic port groups  https://review.openstack.org/38875621:06
*** CristinaPauna has quit IRC21:07
*** sree has quit IRC21:08
*** jheroux has joined #openstack-nova21:10
*** aojea has joined #openstack-nova21:12
mriedemmikal: so you love the metadata api,21:13
mriedemmikal: would changing the content of the network_data.json dict within the metadata api response constitute a bump in the API version, even though we don't filter the results to that level of granularity? i.e. https://review.openstack.org/#/c/388756/18/nova/virt/ironic/driver.py21:14
mriedemwell,21:14
mriedemthis would actually only be config drive since this is modified when the instance is created for the config drive and wouldn't be available via the metadata API, so nvm21:14
*** ijw has joined #openstack-nova21:16
*** voelzmo has joined #openstack-nova21:17
openstackgerritFrancesco Santoro proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser  https://review.openstack.org/38506121:21
*** voelzmo has quit IRC21:22
*** jdurgin has joined #openstack-nova21:22
*** JoseMello has quit IRC21:25
*** zul has quit IRC21:25
*** jamielennox is now known as jamielennox|away21:26
*** _ducttape_ has quit IRC21:28
*** zul has joined #openstack-nova21:28
*** ducttape_ has joined #openstack-nova21:28
*** kylek3h has quit IRC21:28
*** voelzmo has joined #openstack-nova21:29
*** krtaylor has quit IRC21:30
*** jerrygb_ has joined #openstack-nova21:30
*** Swami has joined #openstack-nova21:32
*** jerrygb has quit IRC21:33
*** gouthamr has quit IRC21:34
*** Jeffrey4l has quit IRC21:35
*** Jeffrey4l has joined #openstack-nova21:35
*** tyrola has quit IRC21:39
*** tyrola has joined #openstack-nova21:41
*** ijw has quit IRC21:42
*** voelzmo has quit IRC21:48
openstackgerritPushkar Umaranikar proposed openstack/nova: [Do not merge] Remove unused columns from BuildRequest table in nova_api db  https://review.openstack.org/42225921:49
*** gouthamr has joined #openstack-nova21:57
openstackgerritEric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv)  https://review.openstack.org/40940121:58
*** amotoki has quit IRC21:58
*** slaweq_ has quit IRC22:00
openstackgerritClaudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization  https://review.openstack.org/39850922:00
*** slaweq has joined #openstack-nova22:00
*** Guest76124 has quit IRC22:01
*** slaweq has quit IRC22:02
*** slaweq has joined #openstack-nova22:02
*** chyka has joined #openstack-nova22:02
*** thorst has quit IRC22:07
*** slaweq has quit IRC22:07
*** slaweq has joined #openstack-nova22:09
*** annegentle has quit IRC22:10
*** smatzek has quit IRC22:10
*** jamielennox|away is now known as jamielennox22:10
*** CristinaPauna has joined #openstack-nova22:13
*** dimtruck is now known as zz_dimtruck22:13
*** ijw has joined #openstack-nova22:13
*** jheroux has quit IRC22:16
*** haplo37 has quit IRC22:22
*** jamielennox is now known as jamielennox|away22:24
*** tblakes has quit IRC22:24
*** browne has quit IRC22:27
*** dave-mccowan has quit IRC22:28
*** jamesdenton has quit IRC22:28
*** sree has joined #openstack-nova22:28
*** slaweq has quit IRC22:29
*** Guest62565 has quit IRC22:29
*** slaweq has joined #openstack-nova22:30
*** haplo37 has joined #openstack-nova22:31
*** david-lyle has quit IRC22:32
mriedemclaudiub: should https://review.openstack.org/#/c/397722/ be abandoned?22:32
*** sree has quit IRC22:33
EmilienMmriedem: quick update on placement thing I reported this morning. We found a problem in HAproxy config in TripleO, we're working on it now. It's probably related. It's green for Puppet OpenStack CI.22:34
mriedemok22:34
EmilienMbauzas ^ for your morning22:34
*** mdrabe has quit IRC22:35
*** armax has quit IRC22:36
*** thorst has joined #openstack-nova22:39
*** sacharya has quit IRC22:39
*** david-lyle has joined #openstack-nova22:40
*** thorst has quit IRC22:41
*** annegentle has joined #openstack-nova22:41
*** esberglu has quit IRC22:44
*** CristinaPauna has quit IRC22:44
*** liverpoo1er has quit IRC22:49
*** burgerk has quit IRC22:49
*** sshen has joined #openstack-nova22:50
*** eharney has quit IRC22:54
*** CristinaPauna has joined #openstack-nova22:55
*** tjones has left #openstack-nova22:55
*** amotoki has joined #openstack-nova22:59
*** Swami has quit IRC23:04
*** salv-orlando has joined #openstack-nova23:05
*** claudiub|2 has joined #openstack-nova23:06
openstackgerritZhenyu Zheng proposed openstack/nova: Strict pattern match query parameters  https://review.openstack.org/42049423:06
mriedemanthonyper: you might be able to triage this https://bugs.launchpad.net/nova/+bug/165566223:07
openstackLaunchpad bug 1655662 in OpenStack Compute (nova) "libvirt xen hvm guests can't hot plug (attach detach) default block devices" [Undecided,New]23:07
*** claudiub has quit IRC23:09
*** salv-orlando has quit IRC23:09
*** gszasz has quit IRC23:11
*** mlavalle has quit IRC23:14
*** ducttape_ has quit IRC23:16
mriedemmelwitt: https://review.openstack.org/#/c/420451/ and down need your watchful eye23:16
melwittack23:17
*** sree has joined #openstack-nova23:18
*** sree has quit IRC23:22
*** claudiub|2 has quit IRC23:24
*** Jeffrey4l has quit IRC23:24
*** Matias has quit IRC23:24
*** yamahata has quit IRC23:24
*** dtp has quit IRC23:24
*** sheel has quit IRC23:24
*** test222 has quit IRC23:24
*** fungi has quit IRC23:24
*** sapcc-bot has quit IRC23:24
*** acabot has quit IRC23:24
*** tanee has quit IRC23:24
*** tojuvone has quit IRC23:24
*** Dinesh_Bhor has quit IRC23:24
*** stevemar has quit IRC23:24
*** brault has quit IRC23:24
*** claudiub|2 has joined #openstack-nova23:24
*** Jeffrey4l has joined #openstack-nova23:24
*** Matias has joined #openstack-nova23:24
*** yamahata has joined #openstack-nova23:24
*** dtp has joined #openstack-nova23:24
*** sheel has joined #openstack-nova23:24
*** fungi has joined #openstack-nova23:24
*** sapcc-bot has joined #openstack-nova23:24
*** acabot has joined #openstack-nova23:24
*** tanee has joined #openstack-nova23:24
*** tojuvone has joined #openstack-nova23:24
*** Dinesh_Bhor has joined #openstack-nova23:24
*** stevemar has joined #openstack-nova23:24
*** brault has joined #openstack-nova23:24
melwittmriedem: I had been thinking, what about deleting erroneously created cells that have e.g. broken urls in them? I guess the thinking is, those won't have hosts or instances mapped to them since their urls never worked?23:26
*** kaisers has quit IRC23:27
mriedemdeleting them how?23:27
mriedemlike, automatically?23:27
melwittI'm just imagining someone having a problem because they made a mistake creating some cell with a wrong url in it and they can't get rid of it from their list_cells view23:27
melwittno23:28
melwittoh, wait, I guess those would be empty23:28
mriedemyeah :)23:28
mriedemso, just use delete_cell ?23:28
*** jamesdenton has joined #openstack-nova23:28
*** test222 has joined #openstack-nova23:29
*** xyang1 has quit IRC23:29
*** Jeffrey4l has quit IRC23:29
melwittyeah. I was thinking if there's any problem where someone could get wedged. I think only in a case where they had a valid cell in the past and now want to get rid of it. but that's probably for later anyway, since it's related to idea of re-mapping cells too23:29
*** zz_dimtruck is now known as dimtruck23:30
*** jamesdenton has quit IRC23:31
*** annegentle has quit IRC23:32
*** larainema has joined #openstack-nova23:34
*** aignatov has quit IRC23:34
*** DinaBelova has quit IRC23:34
mriedemthis is also pretty easy https://review.openstack.org/#/c/420132/23:34
*** DinaBelova has joined #openstack-nova23:35
*** aignatov has joined #openstack-nova23:35
melwittk. I've been chasing a bug in my quota counting patch. that's my excuse for not reviewing these earlier23:36
mriedemtdurakov: if you have highlights from the live migration meeting this week please post them in https://wiki.openstack.org/wiki/Meetings/Nova#Weekly_Nova_team_meeting23:39
*** StevenK_ is now known as StevenK23:40
*** Jeffrey4l has joined #openstack-nova23:41
*** armax has joined #openstack-nova23:41
*** sacharya has joined #openstack-nova23:41
*** pradk has quit IRC23:42
*** jerrygb_ has quit IRC23:42
*** Jeffrey4l has quit IRC23:42
*** Jeffrey4l has joined #openstack-nova23:42
*** CristinaPauna has quit IRC23:43
*** timello has quit IRC23:44
*** Sukhdev has quit IRC23:45
*** jerrygb has joined #openstack-nova23:48
openstackgerritMatt Riedemann proposed openstack/nova: hyperv: make sure to plug OVS VIFs after resize/migrate  https://review.openstack.org/42231423:48
*** jaosorior has quit IRC23:49
*** browne has joined #openstack-nova23:49
*** DinaBelova has quit IRC23:57
*** aignatov has quit IRC23:57
*** timello has joined #openstack-nova23:57
*** DinaBelova has joined #openstack-nova23:57
*** aignatov has joined #openstack-nova23:57
*** cdelatte has quit IRC23:59
*** DinaBelova has quit IRC23:59
*** aignatov has quit IRC23:59

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