Thursday, 2017-12-07

openstackgerritTakashi NATSUME proposed openstack/nova master: Fix the order of target host checks  https://review.openstack.org/52622500:02
*** salv-orlando has quit IRC00:06
openstackgerritMerged openstack/nova master: Add PowerVM to compute_driver options  https://review.openstack.org/52397800:06
*** salv-orlando has joined #openstack-nova00:06
*** liverpooler has quit IRC00:06
*** salv-orlando has quit IRC00:11
*** david-lyle has quit IRC00:13
openstackgerritTakashi NATSUME proposed openstack/nova master: Deduplicate functional test code  https://review.openstack.org/52622700:20
*** yangyapeng has quit IRC00:21
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete  https://review.openstack.org/52523100:27
*** chyka has quit IRC00:29
*** Tom-Tom has quit IRC00:32
*** yamamoto has joined #openstack-nova00:34
*** sree has joined #openstack-nova00:36
*** FoXMaN has quit IRC00:36
*** FoXMaN has joined #openstack-nova00:36
*** itlinux has joined #openstack-nova00:39
*** yamamoto has quit IRC00:39
*** sree has quit IRC00:41
*** linkmark has quit IRC00:45
*** liverpooler has joined #openstack-nova00:54
*** jichen has joined #openstack-nova00:54
*** phuongnh has joined #openstack-nova01:01
*** huanxie has joined #openstack-nova01:09
*** Kevin_Zheng has joined #openstack-nova01:09
*** gyee has quit IRC01:12
*** yangyapeng has joined #openstack-nova01:12
*** edmondsw has quit IRC01:12
*** hongbin has quit IRC01:15
*** zhurong has joined #openstack-nova01:19
*** itlinux has quit IRC01:19
*** takedakn has joined #openstack-nova01:20
*** sree has joined #openstack-nova01:20
*** sree has quit IRC01:25
*** salv-orlando has joined #openstack-nova01:27
*** psachin has joined #openstack-nova01:30
*** Sandy619 has joined #openstack-nova01:34
*** Sandy619 has quit IRC01:34
*** salv-orlando has quit IRC01:34
*** salv-orlando has joined #openstack-nova01:35
*** yamamoto has joined #openstack-nova01:35
*** salv-orlando has quit IRC01:39
*** yamahata has joined #openstack-nova01:40
*** yamamoto has quit IRC01:41
*** itlinux has joined #openstack-nova01:45
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748601:46
*** david-lyle has joined #openstack-nova01:48
*** Dinesh_Bhor has joined #openstack-nova01:50
*** yamamoto has joined #openstack-nova01:58
*** david-lyle has quit IRC02:00
*** salv-orlando has joined #openstack-nova02:08
*** rcernin has quit IRC02:12
*** trungnv has quit IRC02:12
openstackgerritsuzhengwei proposed openstack/nova master: send a regular notification when locked flag changing  https://review.openstack.org/52625102:13
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352602:16
*** salv-orlando has quit IRC02:18
*** salv-orlando has joined #openstack-nova02:19
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028502:20
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758702:20
openstackgerritIldiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104702:20
*** rcernin has joined #openstack-nova02:20
*** harlowja has quit IRC02:22
*** salv-orlando has quit IRC02:23
*** Apoorva_ has joined #openstack-nova02:24
*** Apoorva has quit IRC02:27
*** Apoorva_ has quit IRC02:28
openstackgerritsuzhengwei proposed openstack/nova master: send a regular notification when locked flag changing  https://review.openstack.org/52625102:32
*** edmondsw has joined #openstack-nova02:33
*** annp has joined #openstack-nova02:34
*** edmondsw has quit IRC02:37
*** yamahata has quit IRC02:41
*** fragatina has quit IRC02:42
*** fragatina has joined #openstack-nova02:43
*** fragatina has quit IRC02:47
*** fragatina has joined #openstack-nova02:48
*** fragatina has quit IRC02:50
*** tbachman has quit IRC02:52
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352602:56
openstackgerritsuzhengwei proposed openstack/nova master: send a regular notification when locked flag changing  https://review.openstack.org/52625102:57
*** Yingxin has quit IRC02:57
*** fragatina has joined #openstack-nova03:00
*** fragatina has quit IRC03:00
*** hongbin has joined #openstack-nova03:01
*** Yingxin has joined #openstack-nova03:01
openstackgerritGhanshyam Mann proposed openstack/nova master: Deprecate API extensions policies  https://review.openstack.org/52625803:01
*** takedakn has quit IRC03:08
*** itlinux has quit IRC03:10
*** armax has joined #openstack-nova03:11
*** hemna_ has quit IRC03:13
*** tbachman has joined #openstack-nova03:15
*** tetsuro has joined #openstack-nova03:16
alex_xujaypipes: efried have you guys discussed the 'tree=in:[uuid]' in the rest api?03:19
openstackgerritGhanshyam Mann proposed openstack/nova master: Deprecate API extensions policies  https://review.openstack.org/52625803:27
*** abhishekk has joined #openstack-nova03:31
openstackgerritZhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware  https://review.openstack.org/50748603:39
*** qsyqian has joined #openstack-nova03:39
*** itlinux has joined #openstack-nova03:40
*** sree has joined #openstack-nova03:42
*** mdnadeem has joined #openstack-nova03:42
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367603:44
*** yamamoto has quit IRC03:46
*** tbachman has quit IRC03:50
*** yamamoto has joined #openstack-nova03:52
*** threestrands has joined #openstack-nova03:52
*** threestrands has quit IRC03:52
*** threestrands has joined #openstack-nova03:52
*** sree_ has joined #openstack-nova03:53
*** mdnadeem has quit IRC03:53
*** sree_ is now known as Guest3103:53
*** sree has quit IRC03:54
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626303:54
*** udesale has joined #openstack-nova04:01
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626304:03
openstackgerritGhanshyam Mann proposed openstack/nova master: Deprecate API extensions policies  https://review.openstack.org/52625804:06
*** sree has joined #openstack-nova04:08
*** Tom-Tom has joined #openstack-nova04:09
*** Guest31 has quit IRC04:10
*** tetsuro has quit IRC04:10
*** Apoorva has joined #openstack-nova04:15
*** dave-mccowan has quit IRC04:17
*** adisky_ has joined #openstack-nova04:21
*** zhurong has quit IRC04:22
*** links has joined #openstack-nova04:25
*** andreas_s has joined #openstack-nova04:26
*** moshele has joined #openstack-nova04:26
*** andreas_s has quit IRC04:31
*** vishwanathj has quit IRC04:31
*** diga has joined #openstack-nova04:34
*** gbarros has quit IRC04:36
*** harlowja has joined #openstack-nova04:39
openstackgerritmelanie witt proposed openstack/nova master: Follow up on removing old-style quotas code  https://review.openstack.org/52423404:44
openstackgerritmelanie witt proposed openstack/nova master: Add API and nova-manage tests that use the NoopQuotaDriver  https://review.openstack.org/52627004:44
*** hongbin has quit IRC04:47
*** jaosorior has joined #openstack-nova04:49
*** moshele has quit IRC04:50
*** Tom-Tom has quit IRC04:51
*** tbachman has joined #openstack-nova04:55
*** Dinesh_Bhor has quit IRC05:00
*** diga has quit IRC05:06
*** armax has quit IRC05:06
*** itlinux has quit IRC05:10
*** janki has joined #openstack-nova05:14
*** moshele has joined #openstack-nova05:20
*** moshele has quit IRC05:21
*** salv-orlando has joined #openstack-nova05:23
*** gouthamr has quit IRC05:24
*** diga has joined #openstack-nova05:26
*** sridharg has joined #openstack-nova05:29
*** yamamoto has quit IRC05:29
*** harlowja has quit IRC05:30
*** zhurong has joined #openstack-nova05:30
*** salv-orlando has quit IRC05:31
*** fragatina has joined #openstack-nova05:33
*** hoangcx has quit IRC05:42
*** hoangcx has joined #openstack-nova05:42
*** tbachman has quit IRC05:43
*** hieulq has quit IRC05:45
*** hieulq has joined #openstack-nova05:45
*** tetsuro has joined #openstack-nova05:46
*** tbachman has joined #openstack-nova05:47
*** yamamoto has joined #openstack-nova05:47
*** Apoorva has quit IRC05:53
*** Tom-Tom has joined #openstack-nova05:53
*** Apoorva has joined #openstack-nova05:54
*** Teja has joined #openstack-nova05:55
*** sree has quit IRC05:56
*** yamamoto has quit IRC05:56
*** sree has joined #openstack-nova05:56
*** Teja has left #openstack-nova05:56
*** takedakn has joined #openstack-nova05:58
*** Apoorva has quit IRC05:58
openstackgerritChen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model  https://review.openstack.org/52505506:00
*** sree_ has joined #openstack-nova06:02
*** sree_ is now known as Guest3861706:02
*** sree has quit IRC06:02
*** takedakn has quit IRC06:02
*** moshele has joined #openstack-nova06:03
*** fragatina has quit IRC06:06
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352606:06
openstackgerritChen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command  https://review.openstack.org/52458806:06
*** gcb has quit IRC06:08
*** gcb has joined #openstack-nova06:10
*** trungnv has joined #openstack-nova06:12
*** 07EAALF06 has joined #openstack-nova06:12
*** 07EAALF06 has quit IRC06:13
*** fragatina has joined #openstack-nova06:13
*** AlexeyAbashkin has joined #openstack-nova06:15
*** Guest38617 has quit IRC06:18
*** Tom-Tom has quit IRC06:22
*** sree has joined #openstack-nova06:23
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: update the picture in Xen hypervisor document  https://review.openstack.org/52336006:25
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367606:26
*** AlexeyAbashkin has quit IRC06:26
*** salv-orlando has joined #openstack-nova06:28
*** yamamoto has joined #openstack-nova06:28
*** threestrands has quit IRC06:32
*** sapd_ has joined #openstack-nova06:32
*** sapd has quit IRC06:32
*** yamamoto has quit IRC06:33
openstackgerritOpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata  https://review.openstack.org/52479506:34
*** fragatina has quit IRC06:37
openstackgerritMerged openstack/nova master: XenAPI: provide vGPU inventory in compute node  https://review.openstack.org/51621706:39
*** karthiks has quit IRC06:41
*** sree_ has joined #openstack-nova06:43
*** sree_ is now known as Guest8483406:43
*** yamamoto has joined #openstack-nova06:44
openstackgerritMerged openstack/nova master: Add Selection objects  https://review.openstack.org/49923906:45
*** diga has quit IRC06:45
*** sree has quit IRC06:46
*** sree has joined #openstack-nova06:47
*** tetsuro has quit IRC06:47
*** yamamoto has quit IRC06:48
*** Guest84834 has quit IRC06:49
*** salv-orlando has quit IRC06:49
*** andreas_s has joined #openstack-nova06:53
*** karthiks has joined #openstack-nova06:53
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626306:54
openstackgerritwanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached  https://review.openstack.org/52554006:54
*** brault has quit IRC06:54
*** andreas_s has quit IRC06:54
*** andreas_s has joined #openstack-nova06:55
*** yamamoto has joined #openstack-nova06:58
*** karthiks has quit IRC07:04
*** rcernin has quit IRC07:11
*** sree has quit IRC07:12
*** sree has joined #openstack-nova07:13
openstackgerritGhanshyam Mann proposed openstack/nova master: Deprecate configurable Hide Server Address Feature  https://review.openstack.org/52629707:15
*** zhurong has quit IRC07:16
*** karthiks has joined #openstack-nova07:17
*** sree has quit IRC07:21
*** edand has joined #openstack-nova07:25
*** sree has joined #openstack-nova07:29
*** takashin has left #openstack-nova07:30
*** sahid has joined #openstack-nova07:30
*** nore_rabel has joined #openstack-nova07:35
*** sree has quit IRC07:38
*** sree has joined #openstack-nova07:39
*** sree_ has joined #openstack-nova07:43
*** sree_ is now known as Guest4602907:44
*** slaweq_ has joined #openstack-nova07:44
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745707:45
*** sree has quit IRC07:47
*** gaowen has joined #openstack-nova07:49
openstackgerritzhangyanxian proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352607:51
*** alexchadin has joined #openstack-nova07:51
*** yamamoto has quit IRC07:51
*** sshwarts has joined #openstack-nova07:53
*** yamamoto has joined #openstack-nova07:58
*** karthiks has quit IRC07:59
*** salv-orlando has joined #openstack-nova08:00
*** gaowen has quit IRC08:01
*** yamamoto has quit IRC08:03
*** salv-orlando has quit IRC08:04
*** jaianshu has joined #openstack-nova08:05
*** AlexeyAbashkin has joined #openstack-nova08:07
*** sahid has quit IRC08:07
gmannjichen: its different policy for keypair not same - https://review.openstack.org/#/c/526258/08:08
*** sahid has joined #openstack-nova08:08
*** karthiks has joined #openstack-nova08:12
*** yamamoto has joined #openstack-nova08:14
*** tesseract has joined #openstack-nova08:15
*** yamamoto has quit IRC08:18
*** yamamoto has joined #openstack-nova08:18
*** yamamoto has quit IRC08:18
*** rcernin has joined #openstack-nova08:19
*** damien_r has joined #openstack-nova08:21
*** diga has joined #openstack-nova08:21
*** mdnadeem has joined #openstack-nova08:22
*** yamamoto has joined #openstack-nova08:22
*** huanxie has quit IRC08:23
*** yamamoto has quit IRC08:24
*** yamamoto has joined #openstack-nova08:24
*** yamamoto has quit IRC08:24
*** sapd_ has quit IRC08:24
*** phuongnh has quit IRC08:25
*** phuongnh has joined #openstack-nova08:25
*** sapd_ has joined #openstack-nova08:26
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367608:31
*** jpena|off is now known as jpena08:32
gibiildikov: debugging (6) can be easy with tools like http://mgvez.github.io/jsorrery/08:37
*** imacdonn has quit IRC08:39
*** imacdonn has joined #openstack-nova08:40
*** yamamoto has joined #openstack-nova08:41
*** ragiman has joined #openstack-nova08:44
openstackgerritGhanshyam Mann proposed openstack/nova master: Deprecate configurable Hide Server Address Feature  https://review.openstack.org/52629708:45
gmannalex_xu: can you check these 2 patches  - https://review.openstack.org/#/c/526258/  https://review.openstack.org/#/c/526297/08:46
*** sahid has quit IRC08:47
*** sahid has joined #openstack-nova08:48
alex_xugmann: yea, add them to my review list, will reach them asap08:55
gmannalex_xu: thanks. they are Depends-On on oslo.policy 1.30.0 which i added on g-r and should be available soon in nova as g-r patch is approved08:56
openstackgerritMerged openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721508:56
alex_xugmann: ok, got it08:56
*** yamamoto has quit IRC08:56
openstackgerritMerged openstack/nova master: [placement] re-use existing conf with auth token middleware  https://review.openstack.org/52340308:57
*** yamamoto has joined #openstack-nova08:58
*** salv-orlando has joined #openstack-nova09:01
*** yamamoto has quit IRC09:01
*** Guest46029 has quit IRC09:03
abhishekkhi, if 'verify_glance_signatures' is true in nova and if instance is created using signed image then is it advisable to take snapshot of instance?09:06
*** salv-orlando has quit IRC09:06
openstackgerritDeepak Mourya proposed openstack/nova-specs master: Sriov-bond  https://review.openstack.org/46352609:07
*** andreas_s has quit IRC09:07
*** andreas_s has joined #openstack-nova09:08
*** Eran_Kuris has quit IRC09:08
*** andreas_s has quit IRC09:12
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339509:13
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance  https://review.openstack.org/51689909:14
*** Eran_Kuris has joined #openstack-nova09:18
*** jaianshu has quit IRC09:19
*** linkmark has joined #openstack-nova09:23
*** moshele has quit IRC09:29
*** lucas-afk is now known as lucasagomes09:35
*** alexchadin has quit IRC09:35
*** derekh has joined #openstack-nova09:36
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node  https://review.openstack.org/52334209:37
*** TuanLA has joined #openstack-nova09:37
*** pooja-jadhav has joined #openstack-nova09:38
*** alexchad_ has joined #openstack-nova09:38
*** bhagyashri_s has joined #openstack-nova09:39
*** Tahvok has quit IRC09:40
*** niraj_singh has joined #openstack-nova09:40
openstackgerritStephen Finucane proposed openstack/nova master: De-dupe image verification  https://review.openstack.org/52632809:40
openstackgerritStephen Finucane proposed openstack/nova master: Handle images with no data  https://review.openstack.org/52632909:40
*** diga has quit IRC09:40
*** pooja_jadhav has quit IRC09:41
*** nsingh has quit IRC09:41
*** bhagyashris has quit IRC09:41
stephenfinmdbooth: Not _exactly_ your area of expertise, but mind having a look at ^09:41
stephenfin?09:41
mdboothstephenfin: Yep09:42
stephenfinTa :)09:42
*** andreaf has quit IRC09:42
*** elod has quit IRC09:42
*** elod has joined #openstack-nova09:42
*** andreaf has joined #openstack-nova09:42
*** Tahvok has joined #openstack-nova09:43
mdboothHaven't looked at that code since I was working on the vmware driver, I think09:45
mdboothstephenfin: We're reading the whole glance image into ram?09:48
mdboothAt one time09:49
stephenfinIt sure looks like it09:49
stephenfinthat surprised me too09:49
* mdbooth sent a long message: mdbooth_2017-12-07_09:52:16.txt <https://matrix.devlab.redhat.com/_matrix/media/v1/download/redhat.com/fLvjBupFGTmcUusSnTLdZUqD>09:52
mdboothWOW09:52
mdboothUhh...09:52
mdboothWow09:52
* mdbooth wonders how he could turn that into a DoS attack09:52
* mdbooth doesn't have to wonder for long09:52
* stephenfin wonders what he's stumbled into here09:53
mdboothActually this is ringing a bell09:53
* mdbooth recalls hating this before09:54
*** annp has quit IRC09:55
*** jichen has quit IRC09:55
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632609:56
*** tetsuro has joined #openstack-nova09:57
* mdbooth was +1 on the patch which added that :/09:58
mdboothI don't think I realised at the time that it's also used for download09:58
*** moshele has joined #openstack-nova10:00
*** salv-orlando has joined #openstack-nova10:02
*** yamamoto has joined #openstack-nova10:02
*** gszasz has joined #openstack-nova10:04
*** gmann is now known as gmann_afk10:05
*** salv-orlando has quit IRC10:06
*** chyka has joined #openstack-nova10:07
*** yamamoto has quit IRC10:08
*** chyka has quit IRC10:12
openstackgerritTheodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields  https://review.openstack.org/52511010:13
bauzasjianghuaw_: around ?10:14
bauzasI'm thinking of a problem about allocations with multiple possible types10:14
bauzasfor the moment, we're sure that the compute node only support one type, so when creating the VGPU (ie. the mdev for libvirt) I can ask for that type10:15
bauzasnow, say we have more than one type10:15
bauzaswe should then try to understand which type is related to the allocation10:15
*** TuanLA has quit IRC10:18
*** rcernin has quit IRC10:21
*** Yingxin_ has joined #openstack-nova10:22
*** shaohe_feng_ has joined #openstack-nova10:22
*** Yingxin has quit IRC10:24
*** Yingxin_ is now known as Yingxin10:24
*** alex_xu has quit IRC10:24
*** alex_xu has joined #openstack-nova10:25
*** shaohe_feng has quit IRC10:25
*** shaohe_feng_ is now known as shaohe_feng10:25
*** alexchad_ has quit IRC10:27
*** alexchadin has joined #openstack-nova10:28
*** gcb has quit IRC10:29
sean-k-mooney2stephenfin: wait what we should never load the glance image into ram in one go. where?10:32
*** dtantsur|afk is now known as dtantsur10:32
stephenfinsean-k-mooney2: The 'download' function modified here  https://review.openstack.org/52632810:32
*** kalyan has joined #openstack-nova10:33
*** qsyqian has quit IRC10:34
sean-k-mooney2on line 318?10:35
stephenfinActually, higher https://review.openstack.org/#/c/526328/1/nova/image/glance.py@17910:36
sean-k-mooney2stephenfin: i was just going to say i really hope self._client.call(context, 2, 'data', image_id) is retruning an asyc stream to the data form the glance api but that call to list is painful10:39
*** alexchadin has quit IRC10:39
*** yamamoto has joined #openstack-nova10:39
*** alexchadin has joined #openstack-nova10:40
*** yamamoto has quit IRC10:41
sean-k-mooney2stephenfin: i take it we dont have an fuctional test that i dont know validate a 1TB qcow image of all 0 bytes do we because we shoudl remove if on line 176 and add one10:42
tetsuroHi sean, sorry for cutting in. Jay is asking to you about this patch, so could you response? https://review.openstack.org/#/c/465160/  ,10:43
mdboothstephenfin: https://review.openstack.org/#/c/526328/110:43
sean-k-mooney2tetsuro: sure ill take a look now10:43
tetsurothanks in advance10:44
*** abhishekk has quit IRC10:44
stephenfinmdbooth: Fair. I'm reporting that as a bug now10:44
*** Yingxin_ has joined #openstack-nova10:48
*** phuongnh has quit IRC10:48
*** hoangcx has quit IRC10:48
*** phuongnh has joined #openstack-nova10:49
*** hoangcx has joined #openstack-nova10:49
*** shaohe_feng has quit IRC10:50
*** hieulq has quit IRC10:50
*** trungnv has quit IRC10:50
*** trungnv has joined #openstack-nova10:50
*** Yingxin has quit IRC10:50
*** Yingxin_ is now known as Yingxin10:50
*** hieulq has joined #openstack-nova10:50
*** alex_xu has quit IRC10:50
*** qsyqian has joined #openstack-nova10:51
*** alex_xu has joined #openstack-nova10:51
*** shaohe_feng has joined #openstack-nova10:51
openstackgerritStephen Finucane proposed openstack/nova master: Handle images with no data  https://review.openstack.org/52632910:52
openstackgerritStephen Finucane proposed openstack/nova master: Handle images with no data  https://review.openstack.org/52632910:54
*** sdague has joined #openstack-nova10:56
*** sdague has quit IRC10:57
*** sambetts|afk is now known as sambetts11:02
*** salv-orlando has joined #openstack-nova11:02
*** andreas_s has joined #openstack-nova11:03
*** salv-orlando has quit IRC11:07
*** qsyqian has quit IRC11:11
sean-k-mooney2bauzas: if your device support multiple mdev types you need to have multiple resouce provider pools.11:15
sean-k-mooney2bauzas: the tricky bit is doing the accounting correctly11:16
sean-k-mooney2when you claim the mdev you will have to revailidate the available_instances for all other mdev types on the same Physical device11:17
*** sdague has joined #openstack-nova11:22
*** qsyqian has joined #openstack-nova11:23
openstackgerritMatthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260311:24
openstackgerritMatthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid  https://review.openstack.org/52559911:24
openstackgerritMatthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr  https://review.openstack.org/52416711:24
openstackgerritMatthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix  https://review.openstack.org/52634611:24
openstackgerritMatthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice  https://review.openstack.org/52634711:24
*** liverpooler has quit IRC11:25
*** qsyqian has quit IRC11:28
openstackgerritChen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance  https://review.openstack.org/52634811:28
*** udesale has quit IRC11:28
*** yamamoto has joined #openstack-nova11:31
*** Eran_Kuris has quit IRC11:36
*** janki has quit IRC11:40
*** janki has joined #openstack-nova11:40
*** qsyqian has joined #openstack-nova11:41
*** alexchadin has quit IRC11:43
*** alexchadin has joined #openstack-nova11:43
*** Eran_Kuris has joined #openstack-nova11:48
*** smatzek has joined #openstack-nova11:57
*** slaweq_ has quit IRC11:57
*** qsyqian has quit IRC12:01
*** salv-orlando has joined #openstack-nova12:03
openstackgerritChris Dent proposed openstack/nova stable/pike: [placement] re-use existing conf with auth token middleware  https://review.openstack.org/52635112:04
*** slaweq has joined #openstack-nova12:06
*** salv-orlando has quit IRC12:07
*** tetsuro has quit IRC12:08
*** Eran_Kuris has quit IRC12:09
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632612:10
*** andreas_s has quit IRC12:12
openstackgerritWangpan proposed openstack/nova master: Handle glance exception during rotating instance backup  https://review.openstack.org/52339512:13
*** andreas_s has joined #openstack-nova12:13
*** andreas_s has quit IRC12:22
*** andreas_s has joined #openstack-nova12:22
openstackgerritgaryk proposed openstack/nova master: Conductor: ensure build cleanup works with cell  https://review.openstack.org/52635612:33
*** pchavva has joined #openstack-nova12:35
jaypipessean-k-mooney2: I want that original pug.12:36
*** hui has quit IRC12:36
sean-k-mooney2jaypipes: did you see my latest responce12:37
jaypipessean-k-mooney2: ya, reading it now :)12:37
*** slaweq has quit IRC12:38
*** jpena is now known as jpena|lunch12:40
sean-k-mooney2jaypipes: for rodolfos path by the way im happy to enable the fuctional gate jobs but do you know if infra allow enableing new legacy jobs or do i need to swap to zuul v3 in repo job config first?12:40
jaypipessean-k-mooney2: no idea... we should ask clarkb or fungi12:41
*** andreas_s has quit IRC12:42
sean-k-mooney2jaypipes: i have been meaning to get linuxbridge gate jobs running of osvif for a while too so ill reach out to them and see how much work the v3 migration would be12:42
jaypipessean-k-mooney2: sounds good, thanks man12:42
*** andreas_s has joined #openstack-nova12:43
sean-k-mooney2for tetsuro path since the intel nfv ci is still offline(ment to be fixed in january) we do not have any gate jobs for any cpu pinning, hugepage, emulator pin or numa feautres bar unit test which kind of sucks right now.12:45
sean-k-mooney2s/path/patch12:46
*** lucasagomes is now known as lucas-hungry12:47
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609512:50
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)12:50
*** slaweq has joined #openstack-nova12:52
*** andreas_s has quit IRC12:52
openstackgerritgaryk proposed openstack/nova master: Conductor: ensure build cleanup works with cell  https://review.openstack.org/52635612:56
*** sahid has quit IRC12:58
*** psachin has quit IRC12:59
jaypipessean-k-mooney2: right, and I'm not really willing to approve any but the easiest/simplest patches that touch NUMA/CPU pinning/huge pages until that CI is green.12:59
*** BryanS68 has joined #openstack-nova12:59
efriedalex_xu We took a vote and settled on in_tree=<uuid>13:01
sean-k-mooney2jaypipes: the reason i created that patch  originally was to allow hugepages, numa topology and emulator pinning to be testing in the upstream gate but the current version does not enable that13:02
*** phuongnh has quit IRC13:04
*** salv-orlando has joined #openstack-nova13:04
jaypipesefried: mornin.13:04
efriedjaypipes Howdy!13:04
*** liverpooler has joined #openstack-nova13:05
jaypipesefried: that's right... I forgot you're in Texas. :)13:05
jaypipespistols at dawn and all that. :P13:05
jaypipesefried: though frankly, Florida might have just as many guns as Texas.13:06
efriedI haven't taken my pistol out of the safe in months.  You, on the other hand, are living the Florida stereotype to a "tee".13:06
jaypipesefried: my favorite thing about Sarasota is the ammo/gun store directly across from the courthouse called "The Bullet Hole"13:06
efriedHeh13:06
jaypipesefried: touche13:06
jaypipesefried: dansmith introduced me to https://www.reddit.com/r/FloridaMan/ about a year ago, much to my delight.13:07
*** andreas_s has joined #openstack-nova13:07
* efried clicks...13:08
efriedwow13:08
*** erlon has joined #openstack-nova13:09
*** salv-orl_ has joined #openstack-nova13:09
*** slaweq has quit IRC13:09
*** slaweq has joined #openstack-nova13:10
*** salv-orlando has quit IRC13:11
jaypipesefried: you can thank dansmith for that :) there goes an hour of work time.13:11
*** Eran_Kuris has joined #openstack-nova13:11
efriedjaypipes It's better than The Onion.13:11
jaypipesefried: one of my favorites from this past year: https://nypost.com/2017/05/11/man-drowns-in-freak-car-crash-with-fire-hydrant/13:12
efriedHah, it's like that scene from one of the old Superman movies.13:12
jaypipesyep. only "Florida style"13:12
jaypipesefried: ok, so back to work stuff... how we doing on the n-r-p series. I see only the bottom patch merged.13:14
efriedYup13:15
efriedI rebased (and slightly reworked) the pile of patches on top of that.13:15
efriedAnd also resurrected (and corrected) this series: https://review.openstack.org/#/c/521605/13:15
*** janki has quit IRC13:15
efried...which puts caching of aggs & traits into ProviderTree and out of the report client.13:16
jaypipesack13:16
jaypipesOK, let me review that as soon as I comment on sean-k-mooney2's virt_type patch13:16
efriedNot sure if I should fold those guys back into the main series or not.  We're going to need them all before those top three WIPs will work.13:16
jaypipesright13:16
efriedBut if we can get 'em merged fairly quick...13:16
*** yangyapeng has quit IRC13:16
*** andreas_s has quit IRC13:16
*** yangyapeng has joined #openstack-nova13:17
*** dave-mccowan has joined #openstack-nova13:18
*** andreas_s has joined #openstack-nova13:18
efriedjaypipes I'm working today and first half of tomorrow, then out til Jan.  My goal is to have at least some of those WIPs un-WIP'd and close enough for someone (cdent volunteered) to shepherd them while I'm gone.13:18
jaypipesefried: ack, that would be great.13:18
efriedjaypipes Though realistically, I'm not sure how much we can expect to merge the last two weeks of December.  Do lots of people disappear?13:19
jaypipesefried: yes13:19
efriedk13:19
*** yangyapeng has quit IRC13:21
*** alexchadin has quit IRC13:21
*** andreas_s has quit IRC13:27
*** lucas-hungry is now known as lucasagomes13:29
*** sahid has joined #openstack-nova13:31
*** andreas_s has joined #openstack-nova13:32
*** salv-orl_ has quit IRC13:33
*** salv-orlando has joined #openstack-nova13:33
*** janki has joined #openstack-nova13:34
*** slaweq has quit IRC13:35
*** brault has joined #openstack-nova13:36
*** slaweq has joined #openstack-nova13:37
*** brault_ has joined #openstack-nova13:38
*** salv-orlando has quit IRC13:39
*** brault_ has quit IRC13:39
*** brault_ has joined #openstack-nova13:39
*** brault has quit IRC13:40
*** andreas_s has quit IRC13:41
*** andreas_s has joined #openstack-nova13:43
*** tetsuro has joined #openstack-nova13:44
*** yangyapeng has joined #openstack-nova13:45
*** jpena|lunch is now known as jpena13:46
*** andreas_s has quit IRC13:47
*** andreas_s has joined #openstack-nova13:48
openstackgerritEric Fried proposed openstack/nova master: Reformat _get_all_with_shared  https://review.openstack.org/51279913:51
*** peter-hamilton has joined #openstack-nova13:52
*** alexchadin has joined #openstack-nova13:52
*** abhishekk has joined #openstack-nova13:54
*** alexchadin has quit IRC13:56
*** mriedem has joined #openstack-nova13:57
*** takedakn has joined #openstack-nova13:58
*** salv-orlando has joined #openstack-nova13:59
openstackgerritMatt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52319413:59
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate file injection  https://review.openstack.org/52202713:59
mriedemalex_xu: removed that file finally, thanks for pointing that out again ^14:00
mriedemsdague: can we get https://review.openstack.org/523194 in so i can get it backported to newton before eol14:01
*** links has quit IRC14:01
*** lyan has joined #openstack-nova14:03
*** tetsuro has quit IRC14:07
sdaguemriedem: +2 from me14:07
alex_xumriedem: np, +w14:10
mriedemthanks14:10
mriedemthe functional py35 logs are unreadable due to deprecation warnigns14:12
*** AlexeyAbashkin has quit IRC14:12
mriedem*warnings14:12
mriedemfrom oslo.context and oslo.policy14:12
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions  https://review.openstack.org/32632614:13
jianghuaw_bauzas, I think we can only enable one type for each RP (pgpu for libvirt; gpu group for xenapi). Then we can mapping the allocation to the single enabled type in that resource provider.14:13
fungijaypipes: wow, you almost make north carolina sound tame by comparison ;)14:13
fungi(though in full disclosure, i was born in florida so can't really judge)14:13
jaypipesfungi: :)14:14
mriedembauzas: https://review.openstack.org/#/c/508262/ - i think we should get that in and backported to avoid the tons of excessive logging about what state the computes are in when we're doing anything in the resource tracker14:15
jianghuaw_bauzas, it should work if we can mapping back from RP uuid to the PGPU/gpu group.14:15
mriedemwhen i compared logs, it went from logging nearly 400 times to 3514:15
jianghuaw_bauzas, for XenAPI we plan to use the gpu group's uuid as the VGPU RP's uuid. That should work.14:16
*** esberglu has joined #openstack-nova14:16
openstackgerritSurya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url  https://review.openstack.org/51855614:19
jianghuaw_bauzas, see this patch: https://review.openstack.org/#/c/52171714:19
bauzasmriedem: agreed14:19
bauzasmriedem: planning to backport ? or want me to do such thing ?14:20
bauzasjianghuaw_: I think we should discuss maybe later (I'm a but rushing this afternoon tbh) about how to have multiple supported types14:20
bauzasfor the moment, let's merge only one per compute in Queens14:20
jianghuaw_bauzas, sure.14:21
bauzaswe also need to have flavor traits for that, remember14:21
jianghuaw_bauzas, yes. that's not ready for go.14:21
jianghuaw_bauzas, Thanks for the comment on https://review.openstack.org/#/c/516899/14:22
jianghuaw_I uploaded a new ps.14:22
efriedjaypipes I don't see a way around it - I'm going to inject series https://review.openstack.org/#/c/521605/ into the main nrp stack.14:22
jaypipesefried: k14:23
* efried gets his git wand out14:23
*** Eran_Kuris has quit IRC14:23
mriedembauzas: i'm planning on backporting14:25
bauzasmriedem: k, ping me when you're done14:27
*** salv-orlando has quit IRC14:27
*** salv-orlando has joined #openstack-nova14:28
*** smatzek has quit IRC14:28
*** edmondsw has joined #openstack-nova14:30
*** awaugama has joined #openstack-nova14:30
bauzasjianghuaw_: wait, just saw https://review.openstack.org/#/c/516899/30/nova/tests/unit/virt/xenapi/test_driver.py@40014:31
bauzassnap, missed https://review.openstack.org/#/c/516217/24/nova/virt/xenapi/driver.py@45714:31
dansmithjaypipes: that wasn't me dude.. I appreciate the undue credit and all, but ... :P14:32
jianghuaw_bauzas, yes. That's because XenServer can only support one VGPU per VM.14:33
bauzasbut in libvirt, max_unit = max(vgpus)14:33
*** salv-orlando has quit IRC14:33
bauzasjianghuaw_: I think there is a confusion14:33
bauzasah ?14:33
jianghuaw_I saw that difference and I asked if libvirt can support multiple VGPUs.14:33
bauzasin theory, one can create multiple mdevs in libvirt14:34
jianghuaw_There is no way to create multiple VGPU in XenServer at the moment.14:34
bauzasjianghuaw_: that said, I haven't tested it yet14:34
bauzasmultiple VGPUs per instance, right?14:34
jianghuaw_bauzas, seems yes:-)14:36
jianghuaw_I asked XenServer developer. They said that restriction is in the tool stack - XAPI so it may work for libvirt.14:36
bauzasjianghuaw_: given it's experimental for libvirt, I probably need to test that once I'm done with implementing it14:37
bauzasbut you're fortunate, you're at least 3 people working on the xen side, compared to one on libvirt :p14:37
bauzasone being overcommitted :p14:37
jianghuaw_bauzas, That's true. We had 3 people.14:38
jianghuaw_:-)14:38
bauzasyou *had* ?14:38
*** takedakn has quit IRC14:39
jianghuaw_I hope we can go further to support multiple vGPU types and traits. But it still depends on n-rp and traits works which are not ready yet. So moving some effort to other feature.14:40
jianghuaw_that's why I said *had*14:40
bauzask14:42
*** ttx has quit IRC14:42
jianghuaw_bauzas, but I do wish very very much to get the patches on single vGPU can be merged soon. Particularly the long vacation is approaching:-)14:44
jianghuaw_bauzas, but feel free to let me know if further change needed for https://review.openstack.org/#/c/516899/14:46
jianghuaw_I'm happy to rework it.14:46
*** yangyapeng has quit IRC14:48
*** BryanS68 has quit IRC14:48
*** Yingxin has quit IRC14:48
*** niraj_singh has quit IRC14:48
*** bhagyashri_s has quit IRC14:48
*** pooja-jadhav has quit IRC14:48
*** edand has quit IRC14:48
*** openstackgerrit has quit IRC14:48
*** _pewp_ has quit IRC14:48
*** tlbr has quit IRC14:48
*** lyarwood has quit IRC14:48
*** gryf has quit IRC14:48
*** ericyoung has quit IRC14:48
*** lifeless has quit IRC14:48
*** mfisch has quit IRC14:48
*** afazekas has quit IRC14:48
*** diegows has quit IRC14:48
*** antonym has quit IRC14:48
*** yamamoto has quit IRC14:48
*** nore_rabel has quit IRC14:48
*** adisky_ has quit IRC14:48
*** sean-k-mooney2 has quit IRC14:48
*** Kvisle has quit IRC14:48
*** clayton has quit IRC14:48
*** tinwood has quit IRC14:48
*** amotoki has quit IRC14:48
*** McNinja has quit IRC14:48
*** betherly has quit IRC14:48
*** mwhahaha has quit IRC14:48
*** fungi has quit IRC14:48
*** johnsom has quit IRC14:48
*** thingee has quit IRC14:48
*** ameade has quit IRC14:48
*** TheJulia has quit IRC14:48
*** wxy has quit IRC14:48
*** karlamrhein has quit IRC14:48
*** geekinutah has quit IRC14:48
*** robcresswell has quit IRC14:48
*** logan- has quit IRC14:48
*** jogo has quit IRC14:48
*** DuncanT has quit IRC14:48
*** toan has quit IRC14:48
*** johnthetubaguy has quit IRC14:48
*** peter-hamilton has quit IRC14:48
*** kalyan has quit IRC14:48
*** ChanServ has quit IRC14:48
*** pchavva has quit IRC14:48
*** Guest99196 has quit IRC14:48
*** yikun has quit IRC14:48
*** mvk has quit IRC14:48
*** slunkad_ has quit IRC14:48
*** StevenK has quit IRC14:48
*** alaski has quit IRC14:48
*** ltomasbo has quit IRC14:48
*** cargonza has quit IRC14:48
*** d34dh0r53 has quit IRC14:48
*** moshele has quit IRC14:48
*** FoXMaN has quit IRC14:48
*** jangutter has quit IRC14:48
*** toabctl has quit IRC14:48
*** Dave has quit IRC14:48
*** szaher has quit IRC14:48
*** larsks has quit IRC14:48
*** BlackDex has quit IRC14:48
*** naichuans_ has quit IRC14:48
*** spotz has quit IRC14:48
*** jpena has quit IRC14:48
*** sdague has quit IRC14:48
*** Tahvok has quit IRC14:48
*** imacdonn has quit IRC14:48
*** jose-phillips has quit IRC14:48
*** masber has quit IRC14:48
*** anthonyper has quit IRC14:48
*** hoonetorg has quit IRC14:48
*** egarbade has quit IRC14:48
*** ianw has quit IRC14:48
*** zigo has quit IRC14:48
*** Nil_ has quit IRC14:48
*** swamireddy has quit IRC14:48
*** NightKhaos has quit IRC14:48
*** jhesketh has quit IRC14:48
*** andreas_s has quit IRC14:48
*** brault_ has quit IRC14:48
*** erlon has quit IRC14:48
*** gszasz has quit IRC14:48
*** sapd_ has quit IRC14:48
*** sridharg has quit IRC14:48
*** mikal has quit IRC14:48
*** hamzy has quit IRC14:48
*** gmann_afk has quit IRC14:48
*** knikolla has quit IRC14:48
*** oanson has quit IRC14:48
*** auggy has quit IRC14:48
*** tristanC has quit IRC14:48
*** flwang has quit IRC14:48
*** efried has quit IRC14:48
*** vipul has quit IRC14:48
*** rwellum has quit IRC14:48
*** tssurya has quit IRC14:48
*** jaypipes has quit IRC14:48
*** calebb has quit IRC14:48
*** dave-mccowan has quit IRC14:48
*** liusheng has quit IRC14:48
*** serverascode has quit IRC14:48
*** melwitt has quit IRC14:48
*** kencjohnston has quit IRC14:48
*** oomichi_afk has quit IRC14:48
*** tobasco has quit IRC14:48
*** keekz has quit IRC14:48
*** ioni has quit IRC14:48
*** mriedem has quit IRC14:48
*** alex_xu has quit IRC14:48
*** sshwarts has quit IRC14:48
*** jaosorior has quit IRC14:48
*** syjulian has quit IRC14:48
*** mdrabe has quit IRC14:48
*** kashyap has quit IRC14:48
*** ejat has quit IRC14:48
*** shaner has quit IRC14:48
*** sambetts has quit IRC14:48
*** sileht has quit IRC14:48
*** masayukig has quit IRC14:48
*** s1061123 has quit IRC14:48
*** jbernard has quit IRC14:48
*** hemna has quit IRC14:48
*** Vek has quit IRC14:48
*** lennyb has quit IRC14:48
*** janki has quit IRC14:48
*** hoangcx has quit IRC14:48
*** tbachman has quit IRC14:48
*** coreywright has quit IRC14:48
*** cfriesen_ has quit IRC14:48
*** aloga has quit IRC14:48
*** EmilienM has quit IRC14:48
*** migi has quit IRC14:48
*** jdillaman has quit IRC14:48
*** sapcc-bot has quit IRC14:48
*** tonyb has quit IRC14:48
*** mgariepy has quit IRC14:48
*** rmk has quit IRC14:48
*** markmc has quit IRC14:48
*** bfife has quit IRC14:48
*** mdbooth has quit IRC14:48
*** ansiwen has quit IRC14:48
*** brad[] has quit IRC14:48
*** stephenfin has quit IRC14:48
*** rybridges has quit IRC14:48
*** andreykurilin has quit IRC14:48
*** DinaBelova has quit IRC14:48
*** aignatov has quit IRC14:48
*** Swami has joined #openstack-nova14:49
*** yikun_ has joined #openstack-nova14:51
*** lei-zh has joined #openstack-nova14:52
*** yamamoto has joined #openstack-nova14:54
*** adisky_ has joined #openstack-nova14:54
*** sean-k-mooney2 has joined #openstack-nova14:54
*** Kvisle has joined #openstack-nova14:54
*** clayton has joined #openstack-nova14:54
*** tinwood has joined #openstack-nova14:54
*** amotoki has joined #openstack-nova14:54
*** McNinja has joined #openstack-nova14:54
*** betherly has joined #openstack-nova14:54
*** mwhahaha has joined #openstack-nova14:54
*** fungi has joined #openstack-nova14:54
*** johnsom has joined #openstack-nova14:54
*** thingee has joined #openstack-nova14:54
*** ameade has joined #openstack-nova14:54
*** TheJulia has joined #openstack-nova14:54
*** wxy has joined #openstack-nova14:54
*** karlamrhein has joined #openstack-nova14:54
*** geekinutah has joined #openstack-nova14:54
*** robcresswell has joined #openstack-nova14:54
*** logan- has joined #openstack-nova14:54
*** jogo has joined #openstack-nova14:54
*** DuncanT has joined #openstack-nova14:54
*** toan has joined #openstack-nova14:54
*** johnthetubaguy has joined #openstack-nova14:54
*** pchavva has joined #openstack-nova14:54
*** Guest99196 has joined #openstack-nova14:54
*** yikun has joined #openstack-nova14:54
*** mvk has joined #openstack-nova14:54
*** slunkad_ has joined #openstack-nova14:54
*** StevenK has joined #openstack-nova14:54
*** alaski has joined #openstack-nova14:54
*** ltomasbo has joined #openstack-nova14:54
*** cargonza has joined #openstack-nova14:54
*** d34dh0r53 has joined #openstack-nova14:54
*** ttx has joined #openstack-nova14:54
*** antonym has joined #openstack-nova14:54
*** diegows has joined #openstack-nova14:54
*** afazekas has joined #openstack-nova14:54
*** mfisch has joined #openstack-nova14:54
*** lifeless has joined #openstack-nova14:54
*** ericyoung has joined #openstack-nova14:54
*** gryf has joined #openstack-nova14:54
*** lyarwood has joined #openstack-nova14:54
*** tlbr has joined #openstack-nova14:54
*** _pewp_ has joined #openstack-nova14:54
*** openstackgerrit has joined #openstack-nova14:54
*** edand has joined #openstack-nova14:54
*** pooja-jadhav has joined #openstack-nova14:54
*** bhagyashri_s has joined #openstack-nova14:54
*** niraj_singh has joined #openstack-nova14:54
*** Yingxin has joined #openstack-nova14:54
*** BryanS68 has joined #openstack-nova14:54
*** yangyapeng has joined #openstack-nova14:54
*** nore_rabel has joined #openstack-nova14:54
*** amodi has joined #openstack-nova14:54
*** gouthamr has joined #openstack-nova14:54
*** mriedem has joined #openstack-nova14:54
*** andreas_s has joined #openstack-nova14:54
*** brault_ has joined #openstack-nova14:54
*** janki has joined #openstack-nova14:54
*** dave-mccowan has joined #openstack-nova14:54
*** erlon has joined #openstack-nova14:54
*** sdague has joined #openstack-nova14:54
*** alex_xu has joined #openstack-nova14:54
*** hoangcx has joined #openstack-nova14:54
*** gszasz has joined #openstack-nova14:54
*** moshele has joined #openstack-nova14:54
*** Tahvok has joined #openstack-nova14:54
*** imacdonn has joined #openstack-nova14:54
*** sapd_ has joined #openstack-nova14:54
*** sshwarts has joined #openstack-nova14:54
*** tbachman has joined #openstack-nova14:54
*** sridharg has joined #openstack-nova14:54
*** jaosorior has joined #openstack-nova14:54
*** FoXMaN has joined #openstack-nova14:54
*** liusheng has joined #openstack-nova14:54
*** mikal has joined #openstack-nova14:54
*** syjulian has joined #openstack-nova14:54
*** hamzy has joined #openstack-nova14:54
*** jangutter has joined #openstack-nova14:54
*** coreywright has joined #openstack-nova14:54
*** gmann_afk has joined #openstack-nova14:54
*** knikolla has joined #openstack-nova14:54
*** cfriesen_ has joined #openstack-nova14:54
*** oanson has joined #openstack-nova14:54
*** auggy has joined #openstack-nova14:54
*** tristanC has joined #openstack-nova14:54
*** flwang has joined #openstack-nova14:54
*** efried has joined #openstack-nova14:54
*** jose-phillips has joined #openstack-nova14:54
*** vipul has joined #openstack-nova14:54
*** masber has joined #openstack-nova14:54
*** rwellum has joined #openstack-nova14:54
*** tssurya has joined #openstack-nova14:54
*** jaypipes has joined #openstack-nova14:54
*** aloga has joined #openstack-nova14:54
*** anthonyper has joined #openstack-nova14:54
*** calebb has joined #openstack-nova14:54
*** hoonetorg has joined #openstack-nova14:54
*** mdrabe has joined #openstack-nova14:54
*** egarbade has joined #openstack-nova14:54
*** ianw has joined #openstack-nova14:54
*** toabctl has joined #openstack-nova14:54
*** Nil_ has joined #openstack-nova14:54
*** zigo has joined #openstack-nova14:54
*** EmilienM has joined #openstack-nova14:54
*** kashyap has joined #openstack-nova14:54
*** Dave has joined #openstack-nova14:54
*** stephenfin has joined #openstack-nova14:54
*** migi has joined #openstack-nova14:54
*** ejat has joined #openstack-nova14:54
*** shaner has joined #openstack-nova14:54
*** jdillaman has joined #openstack-nova14:54
*** sapcc-bot has joined #openstack-nova14:54
*** szaher has joined #openstack-nova14:54
*** sambetts has joined #openstack-nova14:54
*** larsks has joined #openstack-nova14:54
*** sileht has joined #openstack-nova14:54
*** swamireddy has joined #openstack-nova14:54
*** tonyb has joined #openstack-nova14:54
*** mgariepy has joined #openstack-nova14:54
*** BlackDex has joined #openstack-nova14:54
*** NightKhaos has joined #openstack-nova14:54
*** jhesketh has joined #openstack-nova14:54
*** masayukig has joined #openstack-nova14:54
*** s1061123 has joined #openstack-nova14:54
*** naichuans_ has joined #openstack-nova14:54
*** spotz has joined #openstack-nova14:54
*** jpena has joined #openstack-nova14:54
*** jbernard has joined #openstack-nova14:54
*** hemna has joined #openstack-nova14:54
*** Vek has joined #openstack-nova14:54
*** rmk has joined #openstack-nova14:54
*** lennyb has joined #openstack-nova14:54
*** markmc has joined #openstack-nova14:54
*** serverascode has joined #openstack-nova14:54
*** bfife has joined #openstack-nova14:54
*** mdbooth has joined #openstack-nova14:54
*** ansiwen has joined #openstack-nova14:54
*** brad[] has joined #openstack-nova14:54
*** rybridges has joined #openstack-nova14:54
*** melwitt has joined #openstack-nova14:54
*** kencjohnston has joined #openstack-nova14:54
*** andreykurilin has joined #openstack-nova14:54
*** tobasco has joined #openstack-nova14:54
*** oomichi_afk has joined #openstack-nova14:54
*** DinaBelova has joined #openstack-nova14:54
*** aignatov has joined #openstack-nova14:54
*** keekz has joined #openstack-nova14:54
*** ioni has joined #openstack-nova14:54
*** yikun_ has quit IRC14:55
*** yikun_ has joined #openstack-nova14:55
*** mtreinish has quit IRC14:55
*** salv-orlando has joined #openstack-nova14:57
*** mlavalle has joined #openstack-nova14:57
*** cdent has joined #openstack-nova14:58
*** yamahata has joined #openstack-nova14:58
*** mtreinish has joined #openstack-nova14:58
*** abhishekk has quit IRC15:00
*** artom has quit IRC15:02
mriedemgibi: ildikov: replied in https://review.openstack.org/#/c/525787/15:03
mriedemgibi: thanks for the thorough review as always15:03
ildikovmriedem: Tnx, will check in a bit15:04
openstackgerritMatt Riedemann proposed openstack/nova stable/ocata: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52642615:04
mriedemsdague: dansmith: bauzas: can we get these patches in for the recent cve fix (pike) and then i'll start prepping for a release https://review.openstack.org/#/q/topic:alternate/1664931+status:open15:06
dansmithya15:06
dansmithmriedem: what is the pike only workaround? that hash in the commit message links to this15:07
mriedemlyarwood: there are several patches with +2s on pike, can you go through those? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/pike15:07
dansmithoh wait15:07
mriedemdansmith: this https://review.openstack.org/#/c/523213/4/nova/tests/functional/test_servers.py15:07
mriedemthe setUp15:07
mriedemi should have said, "in the functional test" or something15:08
dansmithyeah, okay I was expecting something functional15:08
dansmithyeah15:08
dansmithalso it just links back to this series which was confusing me about where the thing was added15:08
dansmithoutside of gerrit that will link properly though so that's fine15:08
openstackgerritMatt Riedemann proposed openstack/nova stable/newton: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52642715:09
lyarwoodmriedem: yup will try, still handing over all of my tripleo work atm, hopefully back to nova full time next week btw15:09
mriedemi anticipate your glorious return15:09
*** alexchadin has joined #openstack-nova15:11
mriedemsdague: you were +2 on this before i robustified the tests (copying efried's work) https://review.openstack.org/#/c/521947/ - that is something we'll need to get backported also15:12
gibimriedem: thanks for the reply in the attach flow patch. Now the BFV case is clear for me15:13
edleafedansmith: I have a question about the changes to the filter_scheduler._claim_resources() method15:13
mriedemgibi: yeah that took me some time to figure out why those tests weren't working as i'd expected15:13
edleafedansmith: the docstring about the params doesn't match the params to the method15:13
*** alexchadin has quit IRC15:13
edleafedansmith: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L272-L29315:14
mriedemjgriffith: do you remember why you added this? https://review.openstack.org/#/c/525787/3/nova/virt/block_device.py@44815:14
mriedemjgriffith: doesn't seem necessary so i plan on removing it unless there was a need for this15:14
dansmithedleafe: is that a change I made?15:14
edleafedansmith: I think so. Were there any other changes to _claim_resources?15:15
edleafedansmith: just the rebuild check, right?15:15
dansmithedleafe: I dunno, let me dig up history.. what's your question though?15:15
*** sahid has quit IRC15:16
*** peter-hamilton has joined #openstack-nova15:16
*** kalyan has joined #openstack-nova15:16
*** ChanServ has joined #openstack-nova15:16
*** barjavel.freenode.net sets mode: +o ChanServ15:16
*** damien_r has quit IRC15:16
edleafedansmith: rebasing my series. The docstring says that it is getting and instance UUID, a HostState object, and allocation_requests15:16
*** damien_r has joined #openstack-nova15:16
mriedemgibi: do you want me to remove the  4 changes at the top of https://review.openstack.org/#/c/525787/3/nova/tests/unit/virt/test_block_device.py where i'm setting the attachment_id in those *_driver_bdm class dicts?15:17
edleafebut it's still just getting a spec_obj, instance uuid and allocation_requests15:17
dansmithedleafe: This is the change I didn't change the params: https://review.openstack.org/#/c/521662/10/nova/scheduler/filter_scheduler.py15:17
mriedemgibi: maybe i should just remove the key from the *_driver_bdm variables since they are set in setUp anyway15:17
mriedemto avoid confusion about why they are there15:17
edleafebut it's still just getting a spec_obj, instance uuid and allocation_requests15:17
edleafeoops15:17
edleafeok, then I'll just change the docstring to be correct15:18
dansmithedleafe: yeah, I'm not sure where that docstring diverged from the signature, but it's clearly wrong15:18
edleafeI was wondering if there were other changes being merged that affected it15:18
edleafedansmith: got it15:18
*** moshele has quit IRC15:18
gibimriedem: just remove that is anyhow set in the setUp15:18
gibimriedem: but I that is the 4 change in top of the test_block_device.py :)15:19
edleafedansmith: huh, it was in jaypipes's change for adding claiming back in July. I guess I had corrected it in my series so long ago, that I thought the merge conflict from your change also changed the docstring15:21
sdaguemriedem: yeh, +2 on that still15:21
*** janki has quit IRC15:21
*** smatzek has joined #openstack-nova15:21
mriedemsdague: thanks15:21
dansmithedleafe: that sounds like him15:21
edleafedansmith: :)15:21
*** gbarros has joined #openstack-nova15:21
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree  https://review.openstack.org/52066315:23
openstackgerritEric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates  https://review.openstack.org/52109715:23
openstackgerritEric Fried proposed openstack/nova master: Traits ops on ProviderTree  https://review.openstack.org/52160515:23
openstackgerritEric Fried proposed openstack/nova master: Aggregate ops on ProviderTree  https://review.openstack.org/52621615:23
openstackgerritEric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168515:23
openstackgerritEric Fried proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168615:23
openstackgerritEric Fried proposed openstack/nova master: WIP: ProviderTree.populate_from_iterable  https://review.openstack.org/52075615:23
openstackgerritEric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109815:23
openstackgerritEric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118715:23
openstackgerritEric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024615:23
efriedjaypipes cdent As threatened ^15:23
jaypipesedleafe: I blame efried.15:23
jaypipeslet the blame game commence.15:23
*** hongbin has joined #openstack-nova15:23
*** itlinux has joined #openstack-nova15:23
efriedwait, what am I getting blamed for?15:23
mriedemi know some intrepid core wants to +W this https://review.openstack.org/#/c/521947/15:23
jaypipesefried: nothing :) was just kiddin.15:23
efriedjaypipes I figured, but still want to know.15:23
jaypipesefried: apparently I goofed a patchset of edleafe's a while ago.15:24
cdentsince multiple placement people are gathered at the mo, does everyone agree with takeshi’s assertion that some unit tests ought to happen on https://review.openstack.org/#/c/521640/ We haven’t historically been super aggressive about unit tests in the handlers15:24
cdentsorry takashi. typing is hard15:24
jaypipesmriedem: looking at the bug now..15:24
mriedemcdent: is it covered by gabbits?15:25
*** links has joined #openstack-nova15:25
*** edand has quit IRC15:25
mriedemoh it's a format thing https://review.openstack.org/#/c/521640/7/nova/tests/functional/api/openstack/placement/gabbits/allocations.yaml15:25
mriedemnot utcnow vs updated_at15:26
jaypipescdent: the only things that I think need to go in unit tests are those situations (edge cases) that can't be easily reproduced in a func test15:26
mriedemhmm15:27
cdentmriedem: it is covered, but the not the specific conditional that is being pointed at15:27
mriedemso the last-modified can be created_at, updated_at, the max in a collection, or utcnow15:27
mriedemand that is per-handler15:27
cdentthere are tests for the utility that is used15:27
cdentbut not for its use in the handlers15:27
*** hemna_ has joined #openstack-nova15:27
mriedemyeah i see that15:27
cdent(except in the fact that the gabbits test that a last-modified is returned)15:27
*** alexchadin has joined #openstack-nova15:28
mriedemand it's a datetime format15:28
mriedemidk, asserting that the handler is doing the one right thing it should be doing would only be there for regression15:28
mriedemso i can see value in it, but i'm not sure how much extra work that is15:28
cdentit’s a fair chunk because of the way the handlers do their own serializing15:29
cdentbut not super duper onerous15:29
mriedemso....maybe a follow up?15:30
mriedemas a compromise15:30
cdentfine with me, if thats the consensus15:31
mriedemcommented inline15:32
openstackgerritMatt Riedemann proposed openstack/nova master: Add new style volume attachment support to block_device.py  https://review.openstack.org/52578715:33
openstackgerritMatt Riedemann proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028515:33
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758715:33
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Allow multi-attach in compute api  https://review.openstack.org/27104715:33
mriedemgibi: ^ fixed15:33
*** awaugama has quit IRC15:33
gibimriedem: looking...15:33
*** yamamoto has quit IRC15:33
mriedemi looked at the functional py35 failure in https://review.openstack.org/#/c/330285/15:33
mriedemit was really hard to see from the actual output what the issue was, and i'm assuming it's intermittent, but i can dig again15:34
mriedemthe py27 functional didn't fail15:34
mriedemhttp://logs.openstack.org/85/330285/175/check/openstack-tox-functional-py35/9a23bfd/testr_results.html.gz15:34
gibimriedem: could be some timing issue. py35 thend to uncover those15:34
*** nikhil has joined #openstack-nova15:35
mriedemAssertionError: The migration table left empty.15:35
*** yamahata has quit IRC15:35
mriedemhmm15:36
mriedem2017-12-07 10:11:01,711 WARNING [nova.virt.block_device] Detaching volume from unknown instance15:36
mriedemTestInstanceNotificationSampleWithMultipleComputeOldAttachFlow would suggest this is a latent issue15:36
ildikovI don't recall seeing this before15:37
mriedemhttp://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22AssertionError%3A%20The%20migration%20table%20left%20empty.%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d15:37
mriedemgibi: there are other unrelated patches hitting the same thing15:37
mriedemso it's intermittent and latent15:37
gibimriedem: OK, your logstash link is convincing15:38
mriedemi'll open a bug15:38
gibimriedem: I will put back my +215:38
ildikovmriedem: BTW, I didn't fully get that part of the commit message of the block_device patch that gibi asked about15:40
jianghuaw_stephenfin, Changed the picture back to having transparent background for this patch: https://review.openstack.org/#/c/523360/15:41
jianghuaw_stephenfin, thanks for the comment.15:41
mriedemhttps://bugs.launchpad.net/nova/+bug/173697615:41
openstackLaunchpad bug 1736976 in OpenStack Compute (nova) "test_live_migration_actions functional test randomly fails with "AssertionError: The migration table left empty."" [Undecided,New]15:41
*** itlinux has quit IRC15:41
mriedemildikov: about the boot from volume thing?15:41
ildikovmriedem: I mean even if the volume is created by Nova having or not having an attachment_id has nothing to do with that15:41
*** awaugama has joined #openstack-nova15:41
*** sahid has joined #openstack-nova15:42
ildikovmriedem: the multiattach flag will indeed not be set by Nova15:42
ildikovmriedem: yeah, BFV15:42
mriedemildikov: doesn't have anything to do with multiattach15:42
stephenfinjianghuaw_: +2. Thanks for the follow-up15:42
mriedemildikov: i added todos like this https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@58915:42
mriedemdoes that make it clear?15:42
*** yangyapeng has quit IRC15:42
jianghuaw_stephenfin, thanks.15:42
*** sahid has quit IRC15:42
ildikovmriedem: no :)15:43
edleafemriedem: I got the "AssertionError: The migration table left empty." error in one of my series too15:43
mriedemstephenfin: are you still ok with this https://review.openstack.org/#/c/507473/ ?15:43
*** yangyapeng has joined #openstack-nova15:43
edleafemriedem: passed locally15:43
ildikovmriedem: we reserve the volume now for BFV15:43
*** sahid has joined #openstack-nova15:43
ildikovmriedem: or do we not do that still for the case when Nova creates the volume?15:43
stephenfinmriedem: Eek, seems like forever since I reviewed that. I'll take another look shortly15:44
mriedemildikov: that ^15:44
mriedemildikov: when you pass a volume to bfv, we reserve it in the api by creating an attachment (with the new patch)15:44
*** tidwellr has joined #openstack-nova15:44
mriedemwhen nova creates the volume, we don't create an attachment (on the compute)15:44
mriedemso we attach using the old flow15:44
mriedembecause self['attachment_id'] isn't set in the DriverBlockDevice15:44
mriedemhence the todo i left in that code15:44
ildikovmriedem: isn't that refresh used for live_migrate?15:45
stephenfinjaypipes, efried: Am I correct in saying libvirt doesn't currently report traits to placement?15:45
mriedemildikov: refresh?15:45
jaypipesstephenfin: correct.15:45
stephenfinjaypipes: Phew. And that's something not targeted for Queens15:45
stephenfin?15:46
jianghuaw_mriedem, are you happy to have a quick look on https://review.openstack.org/#/c/523360/ which patch you ever +2; now only changed the picture's background back to transparent.15:46
jianghuaw_so hope a simple one:-)15:46
ildikovmriedem: sorry, looked at the wrong line, but you added a note into that one as well15:46
ildikovmriedem: which I didn't really get15:46
jaypipesstephenfin: no, not targeted15:46
mriedemildikov: link to the line please15:46
stephenfinIdeal. Thanks, jaypipes15:46
ildikovmriedem: what you pasted above: https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@58915:47
mriedemildikov: what don't you get about ^ ?15:47
mriedemildikov: in that code, nova is creating the volume15:47
mriedemagree?15:47
ildikovmriedem: that why we need to create a new attachment here too: refresh_connection_info15:47
*** yangyapeng has quit IRC15:48
mriedemildikov: do you mean https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@473 ?15:48
jaypipesmdbooth: you working on fixing those unit test failures on the BDM uuid patch?15:48
*** yamamoto has joined #openstack-nova15:48
*** yamamoto has quit IRC15:48
ildikovyeah, I agree with that, I just thought we covered everything for that case of BFV as well as we removed check attach, so things needed to be ok on the Cinder side as well regarding volume state, etc.15:48
*** lei-zh has quit IRC15:49
ildikovmriedem: no, I meant line 589, I don't get why we need to add that one15:49
*** itlinux has joined #openstack-nova15:49
mriedemildikov: if we create the volume but don't create the attachment,15:50
mriedemwhen we call super(...).attach()15:50
mriedemhttps://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@511 will be True15:50
mriedemand we will go down _legacy_volume_attach15:51
ildikovsorry, my bad, mixed it up15:51
ildikovforget that one15:51
openstackgerritEd Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts  https://review.openstack.org/52024215:51
openstackgerritEd Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver  https://review.openstack.org/49585415:51
openstackgerritEd Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object  https://review.openstack.org/52349215:51
openstackgerritEd Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts  https://review.openstack.org/51015915:51
openstackgerritEd Leafe proposed openstack/nova master: Change RPC for select_destinations()  https://review.openstack.org/51670715:51
openstackgerritEd Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils  https://review.openstack.org/51135715:51
openstackgerritEd Leafe proposed openstack/nova master: Make conductor pass and use host_lists  https://review.openstack.org/51135815:51
openstackgerritEd Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize  https://review.openstack.org/52643615:51
edleafemriedem: jaypipes: ^^ rebased15:51
edleafemriedem: also added the alternates for resize patch15:51
mriedemildikov: so we're all good now?15:52
mriedemedleafe: ok reapproved the bottom 415:52
ildikovI guess, I need to understand that part of the code as I thought we managed to cover everything regarding reserve back at the time of the old flow and check_attach removal15:53
edleafemriedem: thx15:53
ildikovmy bad, this code is super confusing... :/15:53
mriedemildikov: well, calling check_attach in the old flow on a volume that we *just* created was always kind of pointless15:53
mriedemsomething would have had to slip in between the microseconds that the volume went to 'available' and we started using it15:54
ildikovmriedem: I've never questioned that, I just said that we added extra an extra reserve call to have everything covered, but then it existed only in my head15:54
mriedemi think we only ever reserved from the api15:54
*** itlinux has quit IRC15:54
mriedemyeah https://github.com/openstack/nova/blob/stable/ocata/nova/virt/block_device.py#L25015:55
ildikovyeah, I wasn't sure where exactly we create that volume I guess15:55
mriedemcheck_attach wouldn't reserve15:55
mriedemyup, for blank/image/snapshot it's created on the compute15:55
mriedemright before we attach it15:55
mriedemwe should create the volumes way earlier, like in conductor, but that's not happening today15:56
ildikovok, so I'm just partially stupid, that's something :)15:56
mriedemthis is the attach code in ocata https://github.com/openstack/nova/blob/stable/ocata/nova/compute/api.py#L366615:56
mriedemhttps://github.com/openstack/nova/blob/stable/ocata/nova/compute/api.py#L365315:57
mriedemso that's the only time (aside from bfv with an existing volume) that we'd reserve15:57
ildikovok15:57
mriedemi believe i have a todo sitting here to document how boot from volume actually works15:58
mriedembecause i explained it once in the mailing list15:58
mdboothjaypipes: Yeah. Working on something further down the stack atm.15:58
mdboothjaypipes: Wait, are they at the beginning of the stack, too?15:59
*** links has quit IRC15:59
openstackgerritBalazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407  https://review.openstack.org/52609515:59
openstackbug 1735407 in OpenStack Compute (nova) "[Nova] Evacuation doesn't respect anti-affinity rules" [Medium,In progress] https://launchpad.net/bugs/1735407 - Assigned to Balazs Gibizer (balazs-gibizer)15:59
ildikovmriedem: if you have a link to that I would be happy to read it :)15:59
jaypipesmdbooth: no, second patch in series15:59
*** itlinux has joined #openstack-nova15:59
mdboothHmm, that was careless.15:59
mdboothjaypipes: fixing it now.15:59
* mdbooth knew about some other ones, but they were way after that.16:00
ildikovmriedem: we can have a quick meeting and see what's next16:00
mdboothAh, fixed that locally, didn't realise I'd pushed the old one, sorry.16:01
*** itlinux has quit IRC16:04
*** itlinux has joined #openstack-nova16:06
openstackgerritMatthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid  https://review.openstack.org/24260316:07
maciejjozefczykHello Guys, im trying to debug why I've faced a problem with scheduling. My instances stucks in 'scheduling' state. After enabling debug on nova-scheduler I've found that a host was selected. On nova-compute side there were 0 info about building an instance. It could be possible that the failure point is nova-conductor?16:09
*** alexchadin has quit IRC16:09
*** nore_rabel has quit IRC16:13
*** coreywright has quit IRC16:13
*** tasker has joined #openstack-nova16:15
taskermorning! couple of questions about nova metadata service: why ( and what for ) are there different dates in the URI? does "latest" coalesce the various other date entries and get all of the latest date from them? does "latest" count as a different date than the others? where is the metadata physically stored ( on disk or in a database )?16:16
*** smatzek has quit IRC16:18
*** slaweq has quit IRC16:18
*** sahid has quit IRC16:19
*** slaweq has joined #openstack-nova16:23
mriedemsdague: efried: i'm surprised our docs don't say anything about configuring [neutron] with a user that has the admin role16:23
mriedembecause we definitely require that16:24
mriedemfor doing thigns with the port binding profile16:24
mriedemmaybe the install guide does, but the config reference doesn't16:24
*** Apoorva has joined #openstack-nova16:24
*** AlexeyAbashkin has joined #openstack-nova16:24
*** READ10 has joined #openstack-nova16:24
*** coreywright has joined #openstack-nova16:26
mriedemhmm, not here https://docs.openstack.org/nova/latest/install/compute-install-ubuntu.html16:26
efriedmriedem This coming from https://review.openstack.org/#/c/509892/ ?16:27
*** slaweq has quit IRC16:28
mriedemno16:28
mriedemhttps://bugs.launchpad.net/cinder/+bug/173677316:28
openstackLaunchpad bug 1736773 in Cinder "attachment-show is including `connection_info` for non-admin callers, it shouldn't" [High,Triaged] - Assigned to John Griffith (john-griffith)16:28
mriedemhttps://docs.openstack.org/neutron/latest/install/compute-install-ubuntu.html#configure-the-compute-service-to-use-the-networking-service16:31
mriedemthe networking install guide mentions it, but not sure if that 'neutron' user is an admin16:31
*** AlexeyAbashkin has quit IRC16:31
mriedemah here it is https://docs.openstack.org/neutron/latest/install/controller-install-ubuntu.html#prerequisites16:33
mriedem"Add the admin role to the neutron user:"16:33
mriedemso if you've done everything correctly, things might work16:33
mriedembut i could see people creating a special networking user for nova to talk to neutron which is a service role rather than admin role or something16:33
*** dmsimard is now known as dmsimard|afk16:34
mriedemassuming that's fine because the nova config section for [neutron] doesn't mention it16:34
*** ragiman has quit IRC16:36
mriedemhttps://docs.openstack.org/cinder/latest/install/cinder-controller-install-ubuntu.html#configure-compute-to-use-block-storage16:36
openstackgerritMerged openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480716:38
*** amodi has quit IRC16:38
efriedwohoo!  ^16:42
cdentamazing16:42
* cdent enjoys the sublime pain of needing to rebase two different new microversions16:43
efriedWhat, you haven't written a script for that yet?16:43
*** yangyapeng has joined #openstack-nova16:43
cdentthat would ruin the enjoyment16:44
*** isq_ has joined #openstack-nova16:45
gibigetting n-rp REST merged... it is almost like Christmas16:45
*** slaweq has joined #openstack-nova16:45
*** yamahata__ has quit IRC16:46
mgagneso I installed osc-placement, how can I use the CLI command? I tried with --os-beta-command and resource commands are not available.16:48
mriedemjaypipes: done https://review.openstack.org/#/c/465160/916:48
*** yangyapeng has quit IRC16:48
*** yamamoto has joined #openstack-nova16:49
*** slaweq has quit IRC16:49
jaypipesmriedem: cheers16:50
*** itlinux has quit IRC16:50
*** moshele has joined #openstack-nova16:51
*** damien_r has quit IRC16:51
openstackgerritMerged openstack/nova master: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot  https://review.openstack.org/40038416:52
*** itlinux has joined #openstack-nova16:53
*** chyka has joined #openstack-nova16:53
*** salv-orlando has quit IRC16:53
mriedemildikov: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115093.html was the ML thread i was thinking of16:54
*** fragatina has joined #openstack-nova16:54
*** yamamoto has quit IRC16:55
bauzasholy fsck, the libvirt driver doesn't know a shit about the compute node16:55
ildikovmriedem: cool, thanks16:55
bauzaslike, it's not knowing the compute UUID16:55
mriedembauzas: why should it?16:55
*** yangyapeng has joined #openstack-nova16:56
bauzasmriedem: since we're now passing the allocations, there will be a list of RP UUIDs with resources in them16:56
bauzasmriedem: for the moment, it's not a problem16:56
*** fragatina has quit IRC16:56
bauzasmriedem: given we don't support nested RPs16:56
bauzasmriedem: but once we'll have more than one RP for a compute, then we'll pass all the related allocations to the drivers16:57
*** fragatina has joined #openstack-nova16:57
bauzasthen, how to know which one is for the root RP for example ?16:57
mriedemsure, but the fact we don't have something that we've never had a need for before shouldn't be shocking16:57
mriedem"wait, nova DOESN'T create ice cream?!"16:57
*** Alex_Staf has joined #openstack-nova16:57
*** yamahata has joined #openstack-nova16:57
bauzasmriedem: sure, I understand that, but I thought we would at least know more than just the hostname :)16:58
bauzasanyway16:58
bauzasit's not a problem for me, but meh16:58
mriedemi'm assuming we'd have to pass the cn uuid down through spawn() yes?16:58
*** james_li has joined #openstack-nova16:59
mriedemor have a hook into the driver that the compute manager can call to do whatever things it needs to do so the driver doesn't need to deal with placement directly16:59
mriedemala the resource tracker16:59
bauzasyeah16:59
stephenfinmriedem: Question on https://review.openstack.org/#/c/507473/ - we're storing in the database but not exposing via the object. That's expected, right?16:59
mriedemstephenfin: we are, alwaysh have been17:00
mriedemthe InstanceAction object extends the NovaPersistentObject class17:00
mriedemwhich has the timstamp fields17:00
mriedem*timestamp17:00
*** yangyapeng has quit IRC17:00
stephenfinDamn - I skimmed down to NovaTimestampObject and went no further17:01
stephenfinFairs. That's done now17:01
mriedemthanks17:02
*** Apoorva has quit IRC17:02
*** felipemonteiro_ has joined #openstack-nova17:06
*** yamahata has quit IRC17:06
stephenfinmriedem: Side note - do we have a patch to populate this? Looking for review stuff to avoid more bug fixing https://specs.openstack.org/openstack/nova-specs/priorities/queens-priorities.html17:07
*** harlowja has joined #openstack-nova17:08
efriedmriedem The plan is to have ComputeDriver.update_provider_tree, which we pass in a ProviderTree object.  The driver will likely have to know its own compute node UUID so it can find it in that tree.  But when it's done, the report client will take the updated ProviderTree and flush changes back to placement.17:09
efriedbauzas ^17:09
*** lyan has quit IRC17:09
efriedmriedem bauzas There's some rough WIP at the top of the NRP series that gets a start on that.  I'm noodling with it some more now.17:10
mriedemstephenfin: i haven't done it yet17:10
*** karthiks has quit IRC17:11
bauzasefried: ok, gtk17:11
bauzasefried: so the driver will know the tree, all good with me17:11
efriedbauzas Well, yes, the driver will mostly be responsible for *creating* the tree.  What it receives initially will probably just be the compute node provider (and maybe its traits).17:12
bauzask17:12
efriedbauzas At that point it'll be responsible for creating nested RPs (e.g. numa nodes, SR-IOV PFs, etc.) and possibly even sharing RPs; and setting up traits & aggregate associations.17:13
bauzasyeah I remember now that convo17:14
*** Swami has quit IRC17:15
*** salv-orlando has joined #openstack-nova17:15
*** dtantsur is now known as dtantsur|afk17:15
*** sshwarts has quit IRC17:15
efriedjaypipes cdent Here's a wrinkle: Can sharing RPs be trees?17:18
efriedHint: it'd be a lot cooler if they couldn't.17:19
*** andreas_s has quit IRC17:19
efriedI mean, it's clear that nothing is preventing you from doing that today.17:19
efriedI'm saying we should consider whether that's a good idea.17:19
*** andreas_s has joined #openstack-nova17:19
*** james_li has quit IRC17:21
*** slaweq has joined #openstack-nova17:21
*** itlinux has quit IRC17:23
mriedemcdent: do you think https://review.openstack.org/#/c/518223/ is something we need to backport to pike?17:23
* cdent looking17:23
*** salv-orlando has quit IRC17:24
cdentmriedem: is a nice to have but not a need to have17:24
*** moshele has quit IRC17:24
cdentefried: I think you and I have different mental models of rp trees. I think it should be theoretically fine for an rp which _happens_ to be part of a tree to be shareable but that it is in a tree shouldn’t be particularly meaningful17:25
efriedcdent Okay, yeah, I think I'm discovering that.17:25
*** slaweq has quit IRC17:25
efriedcdent This is coming from me trying to make sure the ProviderTree is fully populated before we hand it off to the compute driver.17:26
cdentwhat tree-ness would mean in that context is a path to get to17:26
efriedI have to spider out from the compute RP's tree, grab all the RPs associated via aggregate, and populate the ProviderTree with those guys.17:26
mriedemcdent: ok17:27
efriedSo I was wondering what happens if one of those sharing RPs happens to be a child - do I *also* have to grab his whole tree and populate the ProviderTree with *those* guys.17:27
efriedThe answer I'm coming to is: No, I just pretend he's a root.17:27
cdentyes17:27
*** nore_rabel has joined #openstack-nova17:27
cdentat least at a first pass. it feels like you are working a bit ahead of yourself. solve the problem you have in front of your face, not all the problems you can imagine?17:28
efriedSo I may end up with multiple agg-associated sharing RPs that happen to be in the same tree, but they'll appear flattened as roots in the ProviderTree.17:28
efriedcdent I agree with that in principle, but I have to code *something* here.17:28
*** andreas_s has quit IRC17:28
cdentefried: I assume you saw my comments on your commit messages needing to say more about why you are coding these *somethings*?17:29
*** andreas_s has joined #openstack-nova17:29
*** andreas_s has quit IRC17:29
*** andreas_s has joined #openstack-nova17:29
efriedcdent One of them, which I updated - see if it satisfies: https://review.openstack.org/#/c/521097/17:30
cdentefried: is there a spec or something where the reasons for the reason you gave (ComputeDriver.update_provider_tree) are explained?17:31
efriedjaypipes ^?17:31
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: [placement] Fix an error message in API validation  https://review.openstack.org/52646517:32
cdentefried: I mean, I’m willing to take it as good simply because you guys are committed to it, but it easier to revie when I have a bigger picture17:33
efriedcdent I agree it would be neat to have a spec that calls out ComputeDriver.update_provider_tree.17:33
efriedcdent Does the WIP patch that introduces that method help at all?  https://review.openstack.org/#/c/521187/8/nova/virt/driver.py17:34
*** andreas_s has quit IRC17:34
efriednot much, I guess.17:34
cdentThe “supersedes get_inventory()” is probably the most meaningful bit17:36
cdentand interesting in the sense that for most of the virt drivers we’ll introduce and then supersede get_inventory in the same cycle (if lucky, subsequent otherwise)17:36
*** gyee has joined #openstack-nova17:37
efriedcdent Yeah, I think PowerVM will get to skip get_inventory()17:37
efriedbut it'll be the exception.17:37
efriedMeh, actually I don't know.17:38
efriedWe should have update_provider_tree ready to go this cycle; any virt drivers that don't already have get_inventory() wouldn't *need* to implement it.  Though I suppose they may choose to.17:38
*** amodi has joined #openstack-nova17:38
*** Apoorva has joined #openstack-nova17:39
*** brault_ has quit IRC17:39
jaypipesefried: haven't thought about that, frankly17:39
efriedjaypipes I think cdent and I talked ourselves into relative sanity on the subject.17:40
*** brault has joined #openstack-nova17:40
mriedemmelwitt: https://review.openstack.org/#/c/526182/ and the one above it have +2s from gibi17:41
mriedemfor the new volume attach stuff17:41
*** salv-orlando has joined #openstack-nova17:42
*** Alex_Staf has quit IRC17:44
*** brault has quit IRC17:45
*** jpena is now known as jpena|off17:48
*** penick has joined #openstack-nova17:50
*** moshele has joined #openstack-nova17:51
*** yamamoto has joined #openstack-nova17:51
*** lucasagomes is now known as lucas-afk17:52
*** salv-orlando has quit IRC17:53
*** moshele has quit IRC17:53
*** salv-orlando has joined #openstack-nova17:53
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix NoneType error when [service_user] is misconfigured  https://review.openstack.org/52646817:53
*** derekh has quit IRC17:54
*** penick has quit IRC17:54
*** yamamoto has quit IRC17:56
*** penick has joined #openstack-nova17:57
*** salv-orlando has quit IRC17:57
*** penick has quit IRC17:59
*** slaweq has joined #openstack-nova17:59
*** salv-orlando has joined #openstack-nova18:00
*** penick has joined #openstack-nova18:02
*** damien_r has joined #openstack-nova18:04
*** salv-orlando has quit IRC18:04
*** slaweq has quit IRC18:04
*** salv-orlando has joined #openstack-nova18:04
*** yangyapeng has joined #openstack-nova18:05
*** penick has quit IRC18:05
*** andreas_s has joined #openstack-nova18:05
*** mdnadeem has quit IRC18:08
*** salv-orlando has quit IRC18:09
*** sambetts is now known as sambetts|afk18:09
*** andreas_s has quit IRC18:09
*** yangyapeng has quit IRC18:09
*** salv-orlando has joined #openstack-nova18:10
*** mvk has quit IRC18:11
*** tesseract has quit IRC18:13
*** salv-orlando has quit IRC18:16
*** salv-orlando has joined #openstack-nova18:16
*** salv-orlando has quit IRC18:19
*** nore_rabel has quit IRC18:20
*** salv-orlando has joined #openstack-nova18:20
*** felipemonteiro_ has quit IRC18:22
*** vivsoni__ has joined #openstack-nova18:24
*** vivsoni_ has quit IRC18:24
*** salv-orlando has quit IRC18:24
*** esberglu has quit IRC18:24
*** esberglu has joined #openstack-nova18:25
*** esberglu_ has joined #openstack-nova18:29
*** esberglu has quit IRC18:30
*** nikhil has quit IRC18:34
*** ragiman has joined #openstack-nova18:34
*** slaweq has joined #openstack-nova18:34
*** damien_r has quit IRC18:35
*** penick has joined #openstack-nova18:36
*** penick has quit IRC18:37
*** fragatina has quit IRC18:38
*** nore_rabel has joined #openstack-nova18:38
*** andreas_s has joined #openstack-nova18:39
*** slaweq has quit IRC18:40
*** moshele has joined #openstack-nova18:40
openstackgerritChris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates  https://review.openstack.org/51352618:43
cdentone down, one to go18:43
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Only log not correcting allocation once per period  https://review.openstack.org/52647718:43
*** andreas_s has quit IRC18:44
*** artom has joined #openstack-nova18:48
*** yamamoto has joined #openstack-nova18:52
*** itlinux has joined #openstack-nova18:53
*** yamamoto has quit IRC18:57
efriedjaypipes cdent edleafe dansmith I'm about to go through SchedulerReportClient and make all the methods raise execptions when appropriate instead of returning None.  Would you prefer I did that all in one patch, or split it into many (e.g. one per method or group of related methods)?18:58
dansmithis there something we need to be catching that we can't without raising there?18:59
cdentmy preference would be for many: makes any booms a bit more obvious18:59
dansmithbut, I'm sure multiple patches per call would be way easier18:59
dansmithalso what cdent said18:59
*** edmondsw has quit IRC19:00
*** phuongnh has joined #openstack-nova19:00
edleafeefried: yeah, smaller chunks are better19:01
efrieddansmith Well, we caught a grenade bug (it went "boom") with the last one.19:02
efrieddansmith It's not that we *couldn't* catch the errors without raising; it's that we *don't*.  We're not paying attention to the returns from these methods in many cases, so the errors slide on by without anyone noticing.19:03
dansmithwell, in some cases that's intentional, where we get allocations and if none, then do a thing19:04
dansmithjust saying all those places will need to be changed. I'll reserve judgment until I see them I guess19:04
*** phuongnh has quit IRC19:04
*** nore_rabel has quit IRC19:06
efrieddansmith Roger wilco.  I think the important ones are where the caller doesn't necessarily have to do anything with the return value from the method - just invoking for the side effect.  Methods like get_allocation_candidates don't fit that, so I guess it makes sense to leave 'em alone.19:06
*** fragatina has joined #openstack-nova19:06
*** moshele has quit IRC19:07
*** baoli has joined #openstack-nova19:07
* dansmith nods19:09
dansmithI just hadn't thought much about it so seeing them all will help19:09
*** felipemonteiro_ has joined #openstack-nova19:10
*** adisky_ has quit IRC19:10
efrieddansmith With that discussion, it's actually going to be pretty limited.  So thanks for that.19:11
*** felipemonteiro__ has joined #openstack-nova19:11
*** penick has joined #openstack-nova19:11
*** slaweq has joined #openstack-nova19:12
*** itlinux has quit IRC19:14
*** felipemonteiro_ has quit IRC19:15
*** slaweq has quit IRC19:17
*** itlinux has joined #openstack-nova19:18
*** andreas_s has joined #openstack-nova19:20
*** harlowja has quit IRC19:21
*** andreas_s has quit IRC19:25
*** tasker has quit IRC19:26
openstackgerritChris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers  https://review.openstack.org/52163919:29
openstackgerritChris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests  https://review.openstack.org/52164019:29
openstackgerritChris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs  https://review.openstack.org/52608419:29
*** esberglu_ is now known as esberglu19:33
*** edmondsw has joined #openstack-nova19:35
jaypipesdansmith, mriedem: https://review.openstack.org/#/c/385693/ should be ready to go.19:38
* dansmith tries to muster enough fucks to even click on the link to see what it is19:39
mriedemit's placement19:39
mriedemi'll tell you now19:39
*** edmondsw has quit IRC19:39
jaypipes:(19:40
mriedemjaypipes: i'll trade you that for https://review.openstack.org/#/c/525787/19:40
dansmithI dunno what jaypipes' is, but it's not an even trade19:40
jaypipesmriedem: sure, deal.19:40
*** edmondsw has joined #openstack-nova19:41
mriedemdansmith: is that a slight against mine or his?19:41
dansmithmriedem: yours :)19:41
mriedemha19:42
mriedemeveryone loves some good block device'n19:42
efrieddansmith Yeah, don't take the trade.  jaypipes' is easy.19:42
dansmithI clicked on mriedem's link to see if it was one of my reviews he just just finished and was looking for a +W19:42
dansmithbut alas, no19:42
jaypipesefried: not *just* BDM. BFV + BDM, otherwise known as BDSM.19:43
*** andreas_s has joined #openstack-nova19:43
dansmithhaha19:43
mriedemdansmith: you can get this easy refactor to complete the set https://review.openstack.org/#/c/526182/19:43
*** slaweq has joined #openstack-nova19:43
dansmithmriedem: I'll trade you for revisiting my live migration allocations one19:43
dansmiththat will even out your karma for trading jay something easy for something hard19:44
mriedemfwiw, https://review.openstack.org/#/c/385693/ has a problem in the commit message19:44
*** edmondsw has quit IRC19:45
efriedmriedem Will y'all fast-approve if I make that edit?  Hate to lose gibi's +219:46
mriedemi'm still reviewing19:46
mriedemthere are other....concerns19:46
efriedight19:46
mriedemthe ... is for intended dramatic effect19:46
efriedjaypipes FYI I have this series locally, with lots of deltas, so *please* don't re-publish.19:46
jaypipesefried: roger19:46
*** andreas_s has quit IRC19:47
*** slaweq has quit IRC19:47
mriedemefried: jaypipes: replied https://review.openstack.org/#/c/385693/19:50
mriedempick your poison19:50
mriedemit's probably premature to say in a release note what our minimum required version of placement is at this point19:51
efriedmriedem Nice.  But FYI, I believe that ship already sailed.19:51
mriedemsince that's likely to change19:51
efriedThat's what we discovered with that grenade bug.19:51
mriedemso we currently say we require 1.1019:51
mriedemwhat do we actually require?19:51
mriedemor were we just using 1.10?19:52
efriedYuh, that's a lie at this point.  I believe it's 1.14.19:52
mriedemyes, ^ requires 1.1419:52
mriedemmy point is,19:52
mriedemwas anything in nova before this change using something higher than 1.10?19:52
mriedembecause if not, grenade was doing it's job19:52
efriedOh, I thought it was the patch before that one.  Never mind, you're right.19:52
*** edmondsw has joined #openstack-nova19:52
efriedDoes that mean cdent's change was incorrect?19:52
mriedemwe are essentially side stepping any form of version discovery still with placement and doing the ironic thing and saying you just have to have external services upgraded first, period19:53
mriedemnot necessarily,19:53
mriedemwe'll need grenade upgrading things for us to have sane CI19:53
mriedeme.g. queens nova doesn't test against pike cinder19:53
*** yamamoto has joined #openstack-nova19:53
efriedmriedem So what are our actual options here, since we don't yet know what the minimum microversion will be?  We create the reno with 1.14 and just remember to bump it with each patch that uses something higher?19:54
mriedemi think my point is just we aren't doing a good job about being careful with versions19:54
mriedemunlike we do with other external services19:54
mriedemregarding a release note, i said i think that's premature right now19:54
mriedemsince it's likely to bump again before we release queens19:55
mriedemso https://github.com/openstack/nova/blob/master/nova/cmd/status.py#L202 needs to change in a follow up19:55
*** mikal_ has joined #openstack-nova19:55
mriedemand we should start working on Queens notes for https://docs.openstack.org/nova/latest/user/placement.html#upgrade-notes, in a follow up19:55
mriedemagreed?19:55
jaypipesworks for me.19:55
cdentthat sounds right19:55
*** moshele has joined #openstack-nova19:55
mriedemok, +W the19:56
mriedem*then19:56
cdenti agree we’ve not been careful about versions, but I’m not sure it is a huge problem in reality19:56
melwittmriedem: FYI I added some test coverage here and stacked the remove old quotas code follow up on top https://review.openstack.org/#/c/52627019:56
*** edmondsw has quit IRC19:56
*** mikal has quit IRC19:57
*** yamamoto has quit IRC19:58
*** edmondsw has joined #openstack-nova19:58
*** edmondsw has quit IRC19:58
*** edmondsw has joined #openstack-nova19:58
*** brault has joined #openstack-nova19:58
*** mikal has joined #openstack-nova20:00
mriedemmelwitt: comments inline20:00
*** andreas_s has joined #openstack-nova20:01
*** andreas_s has quit IRC20:01
*** andreas_s has joined #openstack-nova20:01
*** mikal_ has quit IRC20:01
jaypipesmriedem: reviewed https://review.openstack.org/#/c/525787/. +Wd20:02
*** edmondsw has quit IRC20:02
melwittmriedem: I didn't have access to self.flags in fixtures.py, I assumed it's on the base TestCase class. but let me check20:03
mriedemmelwitt: you likely have to pass the test's self into the fixture20:04
mriedemlike we do in some other fixtures20:04
mriedemi care less about the self.flags thing20:04
mriedemand more about doing the cleanup after the thing you change, and removing the unused CONF in the sample test20:04
melwittoh, got it. I can do that then20:04
mriedemjaypipes: thanks, replied about the setup thing20:05
melwittk20:05
*** andreas_s has quit IRC20:05
*** BryanS68 has quit IRC20:07
*** edmondsw has joined #openstack-nova20:07
mriedemjaypipes: btw, i haven't dealt with that much mox in a long time...20:08
mriedemthe strictness with which mox makes sure you hit bdm.save() cost me about 2 hours20:08
jaypipesmriedem: I know, right? :)20:08
* cdent expires20:09
cdentgood night20:09
*** cdent has quit IRC20:09
*** edmondsw has quit IRC20:12
mriedemjaypipes: efried: want me to do the doc and nova-status stuff from https://review.openstack.org/#/c/385693/ ?20:12
efriedmriedem I would welcome that.  Were you planning to do it in isolation or add it somewhere in this series?20:13
mriedemisolation20:13
mriedemi'm not touching that mess20:13
efriedPerfect20:13
efriedYeah, you shouldn't.20:13
*** slaweq has joined #openstack-nova20:13
*** moshele has quit IRC20:16
melwittI don't know why but using self.flags causes the concurrent test fail for the NoopQuotaDriver20:19
mriedemhuh, maybe not global enough?20:19
*** slaweq_ has joined #openstack-nova20:20
*** slaweq has quit IRC20:21
*** james_li_ has joined #openstack-nova20:22
melwittoh because self.flags doesn't clear the conf override. weird, I thought it would have to so it works with multiple tests20:23
*** melwitt has left #openstack-nova20:24
*** melwitt has joined #openstack-nova20:24
efriedmelwitt I have a bug for that.20:24
efriedStand by...20:24
james_li_Hi nova devs, a quick question: is it possible to attach a volume of another tenant to my server?20:24
mriedemjames_li_: depends on the policy configuration, but not by default20:25
melwittgah, what other channel did I accidentally leave by pressing ctrl-w in the wrong window20:25
efriedmelwitt Sorry, different bug, probably not related: https://bugs.launchpad.net/oslo.config/+bug/170972820:25
openstackLaunchpad bug 1709728 in oslo.config "CONF.set_override doesn't alias deprecated opts" [Undecided,Confirmed]20:25
*** gszasz has quit IRC20:26
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609420:26
james_li_mriedem: cool so its possible if policy enables, e.g. admin user?20:26
mriedemjames_li_: the default rule is admin_or_owner,20:27
mriedemif that were changed to @20:27
mriedemthen anything goes20:27
mriedemhttps://docs.openstack.org/nova/latest/configuration/sample-policy.html20:27
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609420:28
james_li_mriedem: thanks, just wanted to make sure if any code changes are needed for that.20:28
mriedem#"os_compute_api:os-volumes-attachments:create": "rule:admin_or_owner"20:28
mriedemjames_li_: shouldn't require code changes20:28
mriedembut policy is a funny thing so you'd have to test it20:28
james_li_:)20:28
*** salv-orlando has joined #openstack-nova20:31
melwittapi samples tests run super ugly under py320:31
melwittlots of warnings emitted to the screen20:32
*** baoli has quit IRC20:32
melwittand clearing the override didn't seem to help. gdi20:32
efriedjaypipes Do we have db constraints or other checks that prevent traits, aggs, and children from existing against a provider UUID that doesn't exist?20:33
jaypipesefried: nope.20:34
mriedemmelwitt: just use set_override, it's fine20:34
efriedjaypipes So theoretically I could create RP traits first, then create the RP they're associated with?20:34
mriedemand yes on the py35 warnings - i was seeing that in the functional tests earlier today, oslo.contet and oslo.policy warnings20:34
efriedOr create an agg with orphan RP UUIDs, then create the RPs associated with those UUIDs?20:34
mriedemwe should consider ignoring those20:35
jaypipesefried: yes, manually executing SQL statements. of course, the object layer won't allow you to do that, though.20:35
jaypipesefried: since a TraitNotFound or ResourceProviderNotFound would be raised when attempting to do that via ResourceProvider.set_traits()20:36
*** mchiappero has joined #openstack-nova20:36
*** sean-k-m1 has joined #openstack-nova20:36
*** sean-k-m2 has joined #openstack-nova20:36
jaypipesefried: because there's a lookup of trait ID to names supplied in set_traits()20:36
efriedjaypipes oh, okay, phew.  So the answer from the perspective of a REST API consumer is that we're strict about that stuff.20:36
jaypipesefried: yes.20:36
efriedGood.20:36
jaypipesefried: except for aggregates, which are just UUIDs and we have nothing to "check" against.20:36
efriedjaypipes Right, but you can't *associate* a random anonymous UUID with an aggregate?20:37
*** BryanS68 has joined #openstack-nova20:37
jaypipesefried: no. but you *can* associated a random UUID to a known resource provider.20:37
jaypipesefried: it's the aggregate UUID we have no way of checking.20:37
*** mchiappe1o has quit IRC20:37
*** sean-k-mooney has quit IRC20:37
*** sean-k-mooney2 has quit IRC20:37
efriedjaypipes Got it, cool, thanks.20:37
jaypipespas de probleme20:37
openstackgerritMatt Riedemann proposed openstack/nova master: Update nova-status and docs for nova-compute requiring placement 1.14  https://review.openstack.org/52650520:38
efriedjaypipes FYI this is coming from a place where I'm refactoring _ensure_resource_provider: In the _create_resource_provider path I can actually skip refresh_aggregate_map20:38
mriedemefried: jaypipes: ^20:38
jaypipesefried: ack.20:38
jaypipesmriedem: thank you sir.20:39
efriedmriedem Thanks20:39
*** salv-orlando has quit IRC20:39
*** salv-orlando has joined #openstack-nova20:39
*** itlinux has quit IRC20:40
mriedemimacdonn: you can ask but you're going to be hard pressed to find anyone that knows much about libvirt+xen in icehouse in channel right now20:40
mriedemanthonyper is your closest bet20:40
imacdonnyeah, I know ... OK .. so the issue is that nova-compute occassionally gets stuck seemingly in trying to talk to libvirt .. the symptoms are that the resource_tracker no longer reports every minute, and any VM operations that need libvirt fail20:42
imacdonnvia guru meditation, I can see that the resource_tracker thread is stuck trying to call libvirt's getLibVersion()20:42
mriedemwhich is then a call to the hypervisor20:42
mriedemso you're likely deadlocking on something in the hypervisor20:42
imacdonnthat call is made through eventlet's thread pooling proxy thingy, which is now holding a lock20:42
mriedemyeah that could also be screwing you20:43
mriedemi'd enable debug logging for libvirt and see if something shows up in there20:43
mriedemor, check to see if things changed around that code since icehouse and see if you need to backport a fix20:43
imacdonnproblem is it happens once in a while, and I have like 2k compute nodes ... don't really want to to turn debug on on all of them and wait20:43
imacdonnI've looked around, but not found anything that looks like an obvious related fix20:44
*** salv-orlando has quit IRC20:44
*** moshele has joined #openstack-nova20:44
imacdonnI can't tell for sure if it's libvirt hanging on the call, or eventlet getting hung up somehow and not even trying the call20:45
*** pchavva has quit IRC20:45
*** slaweq has joined #openstack-nova20:45
*** tssurya_ has joined #openstack-nova20:47
*** salv-orlando has joined #openstack-nova20:48
mriedemimacdonn: well, i see this in kilo https://review.openstack.org/#/c/104930/20:48
mriedemhttps://review.openstack.org/#/c/104930/12/nova/virt/libvirt/host.py@19220:49
mriedemso,20:49
imacdonnyeah, that made it "fun" to try to compare bits of of the code to see what might have changed20:49
mriedemkeep in mind that anything running in those threads that logs anything could lock you up20:49
imacdonnyes, saw some stuff about that20:49
mriedemso if you have a GMR when things are locked, i'd look for any libvirt driver/host methods in the thread dump,20:49
mriedemand see if those do loging20:49
mriedem*logging20:49
imacdonnGMR is at https://pastebin.com/1jfgdurJ20:50
mriedembecause i'm sure we don't do a good job of auditing stuff like that20:50
imacdonngetLibVersion() call at line 65920:50
*** slaweq has quit IRC20:50
imacdonnvictim thread waiting to acquire() lock around line 77720:51
*** tssurya_ has quit IRC20:53
*** tssurya_ has joined #openstack-nova20:53
*** salv-orlando has quit IRC20:53
imacdonnseems like getLibVersion() isn't asking much of libvirt ... don't think it'd even have to talk to the hypervisor ...20:55
*** takashin has joined #openstack-nova20:56
mriedemwell, it's making a connection to libvirt i believe20:56
mriedemso it's not like the libvirt-python package version or something20:56
mriedemnova meeting in 3 minutes20:57
imacdonnit'd make a native library call at least .. dunno if it'd have to connect to libvirtd .. OK, I'll shut up for now ;)20:57
mriedemtrue yeah20:58
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (usage)  https://review.openstack.org/52060320:59
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (inventory)  https://review.openstack.org/52061320:59
*** itlinux has joined #openstack-nova21:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_class)  https://review.openstack.org/52061121:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (aggregate)  https://review.openstack.org/52060821:00
mriedemmelwitt: might want to check out v21:00
mriedemhttps://bugs.launchpad.net/nova/+bug/173701121:00
openstackLaunchpad bug 1737011 in OpenStack Compute (nova) "ServerActionsTestJSON.test_reboot_server_hard failed to ssh into instance" [Undecided,New]21:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (trait)  https://review.openstack.org/52060521:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref  https://review.openstack.org/52300721:00
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref  https://review.openstack.org/52154121:00
*** pchavva has joined #openstack-nova21:01
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID  https://review.openstack.org/52360621:01
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API  https://review.openstack.org/52450621:01
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API  https://review.openstack.org/52409421:02
openstackgerritTakashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells  https://review.openstack.org/52140021:03
openstackgerritmelanie witt proposed openstack/nova master: Add API and nova-manage tests that use the NoopQuotaDriver  https://review.openstack.org/52627021:03
openstackgerritmelanie witt proposed openstack/nova master: Follow up on removing old-style quotas code  https://review.openstack.org/52423421:03
*** slaweq_ has quit IRC21:04
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2)  https://review.openstack.org/43060821:13
openstackgerritTakashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2)  https://review.openstack.org/45948321:14
*** hemna_ has quit IRC21:15
melwittmriedem: is that different than our usual sporadic ssh timeouts?21:16
*** linkmark has quit IRC21:18
*** pchavva has quit IRC21:19
mriedemmelwitt: i don't think ssh timeouts are all that sporadic anymore21:19
mriedemandreaf: ^?21:19
*** gyee has quit IRC21:20
melwittokay, I saw one recently that I had to recheck so I thought they were still going on. I guess I should look at that one and compare21:20
mriedemi don't see cloud-init run at all21:21
*** moshele has quit IRC21:23
mriedemforce_config_drive = True21:24
mriedemso we force a config drive to inject keys and such21:24
*** slaweq has joined #openstack-nova21:24
mriedemwe don't even get to the point of rebooting the serer21:27
mriedem*server21:27
mriedemit's trying to ssh into the guest to check uptime before that21:27
mriedemand that's what fails21:27
mriedemhttp://logs.openstack.org/83/526183/1/gate/legacy-tempest-dsvm-py35/166f0c9/job-output.txt.gz#_2017-12-07_13_37_58_43014721:27
*** tssurya_ has quit IRC21:27
melwittgood eye21:27
*** tssurya_ has joined #openstack-nova21:27
mriedemthe console output is all from tempest trying to gather information before the test pukes21:28
jaypipesdansmith: from the API layer, if I want to find which cell a compute node (note: not the service host, but the Ironic baremetal node) was in, how would I do that? do I loop through cells doing a query?21:28
mriedemjaypipes: i think you'd have to21:28
*** sambetts|afk has quit IRC21:29
mriedemthe host mapping is the compute_node.host, not compute_nodes.hypervisor_hostname which is the node name21:29
*** slaweq has quit IRC21:29
jaypipesmriedem: right21:29
*** dmsimard|afk is now known as dmsimard21:30
*** lyan has joined #openstack-nova21:30
*** slaweq has joined #openstack-nova21:31
*** tidwellr has quit IRC21:31
*** penick has quit IRC21:31
*** sambetts_ has joined #openstack-nova21:32
melwittmriedem: this is the change I was thinking of where I had to recheck it about a week ago. looks like the same deal http://logs.openstack.org/22/518022/8/check/legacy-tempest-dsvm-neutron-full/81721fe/job-output.txt.gz#_2017-11-30_21_57_15_82738021:33
mriedemmelwitt: http://logs.openstack.org/22/518022/8/check/legacy-tempest-dsvm-neutron-full/81721fe/job-output.txt.gz#_2017-11-30_21_57_15_83814721:34
melwittoh heh21:35
*** tidwellr has joined #openstack-nova21:35
melwittso not the same21:35
*** tssurya_ has quit IRC21:36
*** slaweq has quit IRC21:36
*** peter-hamilton has quit IRC21:36
mriedemmelwitt: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22Kernel%20panic%20-%20not%20syncing%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d21:37
mriedemnothing super obvious there, not like a single node provider21:38
mriedembut it's all master branch, so i wonder if we're using a new cirros image in queens21:38
melwitthow do we check that?21:39
mriedemit's in devstack21:39
mriedemhttps://github.com/openstack-dev/devstack/blob/master/stackrc#L67121:40
mriedemhttps://github.com/openstack-dev/devstack/commit/9f2dcd333103553626db1924a019e151e3e7252e21:40
melwittcool thanks21:40
mriedemthat's not new so...21:40
mriedemheh read the commit message there21:40
*** salv-orlando has joined #openstack-nova21:40
melwittjust ran logstash for 30d and it looks like this is pretty recent stuff. nothing before 11/2721:40
*** moshele has joined #openstack-nova21:41
openstackgerritEric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI  https://review.openstack.org/52609421:41
melwitthaha, huh.21:41
mriedemmelwitt: logstash only goes back 10 days21:41
melwittoh :(21:42
melwittyeah I see what you mean, all providers hitting the kernel panic though pretty rarely21:42
*** salv-orlando has quit IRC21:43
*** sambetts_ has quit IRC21:43
*** BryanS68 has quit IRC21:43
*** ragiman has quit IRC21:43
*** esberglu has quit IRC21:43
*** Guest99196 has quit IRC21:43
*** yikun has quit IRC21:43
*** slunkad_ has quit IRC21:43
*** StevenK has quit IRC21:43
*** alaski has quit IRC21:43
*** ltomasbo has quit IRC21:43
*** cargonza has quit IRC21:43
*** d34dh0r53 has quit IRC21:43
*** Jeffrey4l has quit IRC21:45
*** slaweq_ has joined #openstack-nova21:47
*** salv-orlando has joined #openstack-nova21:48
*** sambetts_ has joined #openstack-nova21:48
*** BryanS68 has joined #openstack-nova21:48
*** ragiman has joined #openstack-nova21:48
*** esberglu has joined #openstack-nova21:48
*** Guest99196 has joined #openstack-nova21:48
*** yikun has joined #openstack-nova21:48
*** slunkad_ has joined #openstack-nova21:48
*** StevenK has joined #openstack-nova21:48
*** alaski has joined #openstack-nova21:48
*** ltomasbo has joined #openstack-nova21:48
*** cargonza has joined #openstack-nova21:48
*** d34dh0r53 has joined #openstack-nova21:48
*** Jeffrey4l has joined #openstack-nova21:48
*** openstackgerrit has quit IRC21:48
*** salv-orlando has quit IRC21:49
*** salv-orlando has joined #openstack-nova21:49
mriedemmelwitt: i assume i'm pulling threads here https://review.openstack.org/#/c/524234/21:51
melwitthaha, that's okay21:52
mriedemalso, might be better to move that into a utility in QUOTAS21:53
mriedemso the callers don't have to know about NoopQuotaDriver being a thing21:53
mriedemQUOTAS.is_noop?21:53
melwittafter fighting with the test coverage I wasn't thinking too clearly. that's my lame excuse21:53
*** armax has joined #openstack-nova21:53
mriedemok. i don't know what stephenfin's excuse was then. :)21:53
*** salv-orlando has quit IRC21:54
mriedemthinking about bangers and mash during the football game at the pub21:54
melwitthow is is_noop not knowing about the driver?21:55
melwitthonestly the way it was before (the values coming from inside the driver) is the way the caller knows the least about it :P21:55
mriedemi'm saying hide that check within QUOTAS21:55
mriedemtrue21:55
mriedemso, QUOTAS.get_reserved()?21:56
melwittso I'm like  ¯\_(ツ)_/¯21:56
mriedemumm21:56
mriedem^ is reserved for our api response when a cell is down and you list instances21:56
melwittwwhhaaaaat21:56
*** slaweq has joined #openstack-nova21:56
mriedemall we have is the uuid in the instance mapping,21:57
mriedemso we're going to use "¯\_(ツ)_/¯" for all other fields21:57
melwittoh right21:57
*** felipemonteiro__ has quit IRC21:57
melwittthat would be fun21:57
*** baoli has joined #openstack-nova21:57
mriedemreserve that patch for april 121:57
*** yangyapeng has joined #openstack-nova21:57
*** penick has joined #openstack-nova21:58
*** slaweq has quit IRC22:01
*** baoli has quit IRC22:02
*** yangyapeng has quit IRC22:02
*** threestrands has joined #openstack-nova22:04
*** baoli has joined #openstack-nova22:04
*** penick has quit IRC22:06
*** hemna_ has joined #openstack-nova22:06
*** rcernin has joined #openstack-nova22:09
*** rcernin has quit IRC22:09
*** rcernin has joined #openstack-nova22:09
mriedemmelwitt: dansmith: remind me, in the cellsv1 job, is n-api the top-level API or is that the cell API?22:11
melwitttop-level API22:12
melwittI don't think we run n-api per cell though it is possible to do that (if you want to test a cell in isolation)22:12
mriedemand cell-region is the nova-cells service at the top and cell-child is nova-cells in the cell?22:12
*** tssurya_ has joined #openstack-nova22:12
melwittoh boy, memory cobwebs22:13
melwittthat sounds right though22:13
*** openstackgerrit has joined #openstack-nova22:14
openstackgerritmelanie witt proposed openstack/nova master: Follow up on removing old-style quotas code  https://review.openstack.org/52423422:14
*** penick has joined #openstack-nova22:15
*** READ10 has quit IRC22:17
mriedemhmm, ok, i'm trying to figure out http://logs.openstack.org/76/523676/11/check/legacy-tempest-dsvm-cells/1d155c6/logs/screen-n-cpu.txt.gz#_Dec_07_09_02_09_57999122:18
mriedemthe patch creates an instance action in the api22:19
mriedemthen when the compute is trying to look it up, it can't find it and blows up22:19
mriedemso i'm guessing,22:19
mriedemthe top level api database has the action record, but not the cell22:19
mriedemcell db i mean22:19
mriedembut then i don't know how we have action records working for anything in cellsv122:20
melwittyeah, so you'll want to check if the instance.save() at the top does a sync down to the cell. it probably does, but it can race22:20
melwittlike compute might try to read before the sync makes it to the cell22:20
mriedemthis isn't instance.save()22:20
melwittokay, then it won't sync it22:20
mriedemobjects.InstanceAction.action_start22:20
melwitthm, so yeah does compute rely on reading instance actions in any other case?22:21
melwittbecause if so, like you said those shouldn't work either22:21
mriedemoh i think i see something22:21
mriedemso stop_instance is in this list https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L5122:22
mriedemand so is backup_instance22:22
mriedemhttps://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L171422:22
mriedemstop_instance has that ^22:22
mriedembackup_instance doesn't22:22
melwittand you're seeing a fail with backup?22:23
*** weshay is now known as weshay|ruck22:23
mriedemyeah https://review.openstack.org/#/c/523676/1122:24
*** penick has quit IRC22:24
melwittso returning the response from the instance action vs not22:24
melwittI didn't realize instance action had a rpc response in any case22:25
openstackgerritTakashi NATSUME proposed openstack/nova master: Deduplicate functional test code  https://review.openstack.org/52622722:25
*** penick has joined #openstack-nova22:26
*** ragiman has quit IRC22:27
melwittyeah it looks like for some reason stop_instance can return something but backup_instance can't in nova/compute/rpcapi.py22:27
melwittI don't see stop_instance ever returning anything in compute/manager though22:28
*** penick has quit IRC22:29
*** slaweq has joined #openstack-nova22:29
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete  https://review.openstack.org/52523122:29
mriedemmelwitt: i think the do_cast thing is something else22:29
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2)  https://review.openstack.org/52626322:29
mriedemred herring22:29
melwittyeah, it's just saying wait for response vs not22:29
mriedemso i'm not sure what ever creates instance action records in the top level vs cell db22:29
melwittand looks correct based on backup vs stop22:30
melwittwell, it would be whichever service does the write. if n-api writes it, it will be in the top level db, if n-cpu writes it, it will be in the cell db22:30
melwittI think only instance records are synced back and forth in cells v122:30
melwittI'm checking to see if anything in backup_instance in compute does a write of instance action records22:31
mriedembut if that's the case, then how does our @wrap_instance_event in the compute not blow up on everything?22:31
melwittI guess it can't be, else they'd be found22:31
mriedemthe action record stuff for backup_instance is new to this patch22:31
mriedemfollowing the same pattern as everywhere else22:31
melwittokay, I'll look at that then22:31
melwittyeah, that's just writing it in n-api so it'll go in the top level db only22:32
melwittI'm not sure how the other actions are currently working22:32
*** mvk has joined #openstack-nova22:33
*** slaweq has quit IRC22:34
*** rcernin has quit IRC22:36
*** hemna_ has quit IRC22:36
*** sambetts_ has quit IRC22:36
*** BryanS68 has quit IRC22:36
*** esberglu has quit IRC22:36
*** Guest99196 has quit IRC22:36
*** yikun has quit IRC22:36
*** slunkad_ has quit IRC22:36
*** StevenK has quit IRC22:36
*** alaski has quit IRC22:36
*** ltomasbo has quit IRC22:36
*** cargonza has quit IRC22:36
*** d34dh0r53 has quit IRC22:36
*** penick has joined #openstack-nova22:39
*** tssurya_ has quit IRC22:39
*** penick has quit IRC22:39
*** rcernin has joined #openstack-nova22:42
*** hemna_ has joined #openstack-nova22:42
*** sambetts_ has joined #openstack-nova22:42
*** BryanS68 has joined #openstack-nova22:42
*** esberglu has joined #openstack-nova22:42
*** Guest99196 has joined #openstack-nova22:42
*** yikun has joined #openstack-nova22:42
*** slunkad_ has joined #openstack-nova22:42
*** StevenK has joined #openstack-nova22:42
*** alaski has joined #openstack-nova22:42
*** ltomasbo has joined #openstack-nova22:42
*** cargonza has joined #openstack-nova22:42
*** d34dh0r53 has joined #openstack-nova22:42
*** yamahata has joined #openstack-nova22:44
mriedemalaski: you remember anything special about how the hell instance action records get synced between the api cell and the child cells in v1?22:44
*** awaugama has quit IRC22:46
*** jaypipes has quit IRC22:47
openstackgerritMatt Riedemann proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367622:47
*** esberglu has quit IRC22:50
*** tidwellr has quit IRC22:50
*** salv-orlando has joined #openstack-nova22:50
*** gyee has joined #openstack-nova22:50
*** slaweq_ has quit IRC22:50
*** BryanS68 has quit IRC22:52
openstackgerritMatt Riedemann proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip  https://review.openstack.org/52550522:53
*** moshele has quit IRC22:54
*** penick has joined #openstack-nova22:54
*** itlinux has quit IRC22:56
*** salv-orlando has quit IRC22:57
*** felipemonteiro_ has joined #openstack-nova22:57
*** gouthamr has quit IRC22:59
*** james_li_ has quit IRC23:02
*** markvoelker has quit IRC23:03
*** lyan has quit IRC23:03
*** tasker has joined #openstack-nova23:04
*** markvoelker has joined #openstack-nova23:04
taskerI'm playing around with the nova-metadata service and every PUT or POST request I send to "openstack/latest/password" is seen by the service as a GET. is this a security feature of the metadata service or is there some configuration issue that I haven't uncovered yet?23:05
taskerI tracked it through the code from api/metadata/password.py:handle_password up through MetadataRequestHandler and it's a GET the whole time.23:06
*** slaweq has joined #openstack-nova23:06
taskeris something translating it to GET?23:06
*** harlowja has joined #openstack-nova23:07
*** lyan has joined #openstack-nova23:07
*** yangyapeng has joined #openstack-nova23:07
mriedemtempest does a POST successfully from the guest to change the password23:08
mriedemi remember because of this bug https://bugs.launchpad.net/nova/+bug/169684823:08
openstackLaunchpad bug 1696848 in OpenStack Compute (nova) pike "POST /openstack/2013-10-17/password returns 500 if instance is not found" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)23:08
mriedemhttp://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_server_basic_ops.py#n9023:08
mriedemtasker: are you passing a body?23:09
taskeryeah. though now that I think about it, it's probably malformed23:09
taskercurl -D - -X POST -d 'cloudnull' http://169.254.169.254/openstack/latest/password23:10
*** yangyapeng has quit IRC23:11
*** slaweq has quit IRC23:11
*** imacdonn has quit IRC23:11
taskerbefore we go much farther .. this Mitaka at best. I haven't been able to find anything that says it won't work there though.23:12
mriedemmetadata api doesn't really ever change23:12
mriedemmikal might have some ideas23:12
mriedemhe did give the talk about it in sydney23:12
taskerI can't seem to find anything else on the internet about it not working. as if it not working wasn't even a thing.23:13
taskerI'll keep hacking and tracking seeing if I can figure out what's going on.23:13
mriedemare you using service_metadata_proxy in nova.conf?23:13
taskeryes. set to True23:14
*** imacdonn has joined #openstack-nova23:14
*** salv-orlando has joined #openstack-nova23:16
taskeror23:17
taskeroh23:17
taskerand .. that explains a lot.23:17
mriedemnot set?23:17
taskerit's set to True.23:17
*** amodi has quit IRC23:17
taskerand I don't use the `neutron_agents` nor its metadata service23:18
mriedemah23:18
taskerthanks, Matt. that shed a lot of light on what's going on.23:19
taskerback to work!23:19
mriedemo/23:19
taskerto me. not you.23:19
taskeryou do whatever it was you were doing.  .)23:19
mriedemworking23:19
*** felipemonteiro_ has quit IRC23:22
*** salv-orlando has quit IRC23:26
openstackgerritEric Fried proposed openstack/nova master: Traits ops on ProviderTree  https://review.openstack.org/52160523:26
openstackgerritEric Fried proposed openstack/nova master: Aggregate ops on ProviderTree  https://review.openstack.org/52621623:26
openstackgerritEric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree  https://review.openstack.org/52168523:26
openstackgerritEric Fried proposed openstack/nova master: Track provider traits in report client  https://review.openstack.org/52168623:26
openstackgerritEric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable  https://review.openstack.org/52075623:26
efriedI broke the patchbot.23:26
openstackgerritEric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree  https://review.openstack.org/52109823:26
openstackgerritEric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree()  https://review.openstack.org/52118723:26
openstackgerritEric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker  https://review.openstack.org/52024623:26
openstackgerritEric Fried proposed openstack/nova master: Track associated sharing RPs in report client  https://review.openstack.org/52653923:26
openstackgerritEric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits  https://review.openstack.org/52654023:26
openstackgerritEric Fried proposed openstack/nova master: Track tree-associated providers in report client  https://review.openstack.org/52654123:26
*** salv-orlando has joined #openstack-nova23:26
efriedah, there it goes.23:26
efriedI got `related changes` to scroll :)23:27
*** erlon has quit IRC23:28
*** yamahata has quit IRC23:29
openstackgerritMatt Riedemann proposed openstack/nova stable/newton: Add regression test for rebuilding a volume-backed server  https://review.openstack.org/52654223:30
openstackgerritMatt Riedemann proposed openstack/nova stable/newton: Only query BDMs once in API during rebuild  https://review.openstack.org/52654323:30
openstackgerritMatt Riedemann proposed openstack/nova stable/newton: Get original image_id from volume for volume-backed instance rebuild  https://review.openstack.org/52654423:30
*** salv-orlando has quit IRC23:30
mriedemtonyb: here we go https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton23:31
mriedemi think that's going to be the end for newton23:31
mriedemfor real this time23:31
mriedemi know i say this every week23:31
tonyb;P23:31
tonybI'll look at them today23:31
mriedemhttps://www.youtube.com/watch?v=gfkfXcme5DA23:31
openstackgerritHongbin Lu proposed openstack/nova master: placement: skip authentication on root URI  https://review.openstack.org/52200223:32
mriedemmtreinish: where are you when i make simpsons references?23:33
*** lyan has quit IRC23:37
*** liverpooler has quit IRC23:37
openstackgerritHongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2  https://review.openstack.org/52475523:39
*** penick has quit IRC23:40
*** gouthamr has joined #openstack-nova23:42
*** slaweq has joined #openstack-nova23:44
*** yangyapeng has joined #openstack-nova23:47
*** slaweq has quit IRC23:49
*** yangyapeng has quit IRC23:51
*** threestrands has quit IRC23:55
*** yamamoto has joined #openstack-nova23:58

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