openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix the order of target host checks https://review.openstack.org/526225 | 00:02 |
---|---|---|
*** salv-orlando has quit IRC | 00:06 | |
openstackgerrit | Merged openstack/nova master: Add PowerVM to compute_driver options https://review.openstack.org/523978 | 00:06 |
*** salv-orlando has joined #openstack-nova | 00:06 | |
*** liverpooler has quit IRC | 00:06 | |
*** salv-orlando has quit IRC | 00:11 | |
*** david-lyle has quit IRC | 00:13 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Deduplicate functional test code https://review.openstack.org/526227 | 00:20 |
*** yangyapeng has quit IRC | 00:21 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231 | 00:27 |
*** chyka has quit IRC | 00:29 | |
*** Tom-Tom has quit IRC | 00:32 | |
*** yamamoto has joined #openstack-nova | 00:34 | |
*** sree has joined #openstack-nova | 00:36 | |
*** FoXMaN has quit IRC | 00:36 | |
*** FoXMaN has joined #openstack-nova | 00:36 | |
*** itlinux has joined #openstack-nova | 00:39 | |
*** yamamoto has quit IRC | 00:39 | |
*** sree has quit IRC | 00:41 | |
*** linkmark has quit IRC | 00:45 | |
*** liverpooler has joined #openstack-nova | 00:54 | |
*** jichen has joined #openstack-nova | 00:54 | |
*** phuongnh has joined #openstack-nova | 01:01 | |
*** huanxie has joined #openstack-nova | 01:09 | |
*** Kevin_Zheng has joined #openstack-nova | 01:09 | |
*** gyee has quit IRC | 01:12 | |
*** yangyapeng has joined #openstack-nova | 01:12 | |
*** edmondsw has quit IRC | 01:12 | |
*** hongbin has quit IRC | 01:15 | |
*** zhurong has joined #openstack-nova | 01:19 | |
*** itlinux has quit IRC | 01:19 | |
*** takedakn has joined #openstack-nova | 01:20 | |
*** sree has joined #openstack-nova | 01:20 | |
*** sree has quit IRC | 01:25 | |
*** salv-orlando has joined #openstack-nova | 01:27 | |
*** psachin has joined #openstack-nova | 01:30 | |
*** Sandy619 has joined #openstack-nova | 01:34 | |
*** Sandy619 has quit IRC | 01:34 | |
*** salv-orlando has quit IRC | 01:34 | |
*** salv-orlando has joined #openstack-nova | 01:35 | |
*** yamamoto has joined #openstack-nova | 01:35 | |
*** salv-orlando has quit IRC | 01:39 | |
*** yamahata has joined #openstack-nova | 01:40 | |
*** yamamoto has quit IRC | 01:41 | |
*** itlinux has joined #openstack-nova | 01:45 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 01:46 |
*** david-lyle has joined #openstack-nova | 01:48 | |
*** Dinesh_Bhor has joined #openstack-nova | 01:50 | |
*** yamamoto has joined #openstack-nova | 01:58 | |
*** david-lyle has quit IRC | 02:00 | |
*** salv-orlando has joined #openstack-nova | 02:08 | |
*** rcernin has quit IRC | 02:12 | |
*** trungnv has quit IRC | 02:12 | |
openstackgerrit | suzhengwei proposed openstack/nova master: send a regular notification when locked flag changing https://review.openstack.org/526251 | 02:13 |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 02:16 |
*** salv-orlando has quit IRC | 02:18 | |
*** salv-orlando has joined #openstack-nova | 02:19 | |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 02:20 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | 02:20 |
openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | 02:20 |
*** rcernin has joined #openstack-nova | 02:20 | |
*** harlowja has quit IRC | 02:22 | |
*** salv-orlando has quit IRC | 02:23 | |
*** Apoorva_ has joined #openstack-nova | 02:24 | |
*** Apoorva has quit IRC | 02:27 | |
*** Apoorva_ has quit IRC | 02:28 | |
openstackgerrit | suzhengwei proposed openstack/nova master: send a regular notification when locked flag changing https://review.openstack.org/526251 | 02:32 |
*** edmondsw has joined #openstack-nova | 02:33 | |
*** annp has joined #openstack-nova | 02:34 | |
*** edmondsw has quit IRC | 02:37 | |
*** yamahata has quit IRC | 02:41 | |
*** fragatina has quit IRC | 02:42 | |
*** fragatina has joined #openstack-nova | 02:43 | |
*** fragatina has quit IRC | 02:47 | |
*** fragatina has joined #openstack-nova | 02:48 | |
*** fragatina has quit IRC | 02:50 | |
*** tbachman has quit IRC | 02:52 | |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 02:56 |
openstackgerrit | suzhengwei proposed openstack/nova master: send a regular notification when locked flag changing https://review.openstack.org/526251 | 02:57 |
*** Yingxin has quit IRC | 02:57 | |
*** fragatina has joined #openstack-nova | 03:00 | |
*** fragatina has quit IRC | 03:00 | |
*** hongbin has joined #openstack-nova | 03:01 | |
*** Yingxin has joined #openstack-nova | 03:01 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate API extensions policies https://review.openstack.org/526258 | 03:01 |
*** takedakn has quit IRC | 03:08 | |
*** itlinux has quit IRC | 03:10 | |
*** armax has joined #openstack-nova | 03:11 | |
*** hemna_ has quit IRC | 03:13 | |
*** tbachman has joined #openstack-nova | 03:15 | |
*** tetsuro has joined #openstack-nova | 03:16 | |
alex_xu | jaypipes: efried have you guys discussed the 'tree=in:[uuid]' in the rest api? | 03:19 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate API extensions policies https://review.openstack.org/526258 | 03:27 |
*** abhishekk has joined #openstack-nova | 03:31 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 03:39 |
*** qsyqian has joined #openstack-nova | 03:39 | |
*** itlinux has joined #openstack-nova | 03:40 | |
*** sree has joined #openstack-nova | 03:42 | |
*** mdnadeem has joined #openstack-nova | 03:42 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | 03:44 |
*** yamamoto has quit IRC | 03:46 | |
*** tbachman has quit IRC | 03:50 | |
*** yamamoto has joined #openstack-nova | 03:52 | |
*** threestrands has joined #openstack-nova | 03:52 | |
*** threestrands has quit IRC | 03:52 | |
*** threestrands has joined #openstack-nova | 03:52 | |
*** sree_ has joined #openstack-nova | 03:53 | |
*** mdnadeem has quit IRC | 03:53 | |
*** sree_ is now known as Guest31 | 03:53 | |
*** sree has quit IRC | 03:54 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | 03:54 |
*** udesale has joined #openstack-nova | 04:01 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | 04:03 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate API extensions policies https://review.openstack.org/526258 | 04:06 |
*** sree has joined #openstack-nova | 04:08 | |
*** Tom-Tom has joined #openstack-nova | 04:09 | |
*** Guest31 has quit IRC | 04:10 | |
*** tetsuro has quit IRC | 04:10 | |
*** Apoorva has joined #openstack-nova | 04:15 | |
*** dave-mccowan has quit IRC | 04:17 | |
*** adisky_ has joined #openstack-nova | 04:21 | |
*** zhurong has quit IRC | 04:22 | |
*** links has joined #openstack-nova | 04:25 | |
*** andreas_s has joined #openstack-nova | 04:26 | |
*** moshele has joined #openstack-nova | 04:26 | |
*** andreas_s has quit IRC | 04:31 | |
*** vishwanathj has quit IRC | 04:31 | |
*** diga has joined #openstack-nova | 04:34 | |
*** gbarros has quit IRC | 04:36 | |
*** harlowja has joined #openstack-nova | 04:39 | |
openstackgerrit | melanie witt proposed openstack/nova master: Follow up on removing old-style quotas code https://review.openstack.org/524234 | 04:44 |
openstackgerrit | melanie witt proposed openstack/nova master: Add API and nova-manage tests that use the NoopQuotaDriver https://review.openstack.org/526270 | 04:44 |
*** hongbin has quit IRC | 04:47 | |
*** jaosorior has joined #openstack-nova | 04:49 | |
*** moshele has quit IRC | 04:50 | |
*** Tom-Tom has quit IRC | 04:51 | |
*** tbachman has joined #openstack-nova | 04:55 | |
*** Dinesh_Bhor has quit IRC | 05:00 | |
*** diga has quit IRC | 05:06 | |
*** armax has quit IRC | 05:06 | |
*** itlinux has quit IRC | 05:10 | |
*** janki has joined #openstack-nova | 05:14 | |
*** moshele has joined #openstack-nova | 05:20 | |
*** moshele has quit IRC | 05:21 | |
*** salv-orlando has joined #openstack-nova | 05:23 | |
*** gouthamr has quit IRC | 05:24 | |
*** diga has joined #openstack-nova | 05:26 | |
*** sridharg has joined #openstack-nova | 05:29 | |
*** yamamoto has quit IRC | 05:29 | |
*** harlowja has quit IRC | 05:30 | |
*** zhurong has joined #openstack-nova | 05:30 | |
*** salv-orlando has quit IRC | 05:31 | |
*** fragatina has joined #openstack-nova | 05:33 | |
*** hoangcx has quit IRC | 05:42 | |
*** hoangcx has joined #openstack-nova | 05:42 | |
*** tbachman has quit IRC | 05:43 | |
*** hieulq has quit IRC | 05:45 | |
*** hieulq has joined #openstack-nova | 05:45 | |
*** tetsuro has joined #openstack-nova | 05:46 | |
*** tbachman has joined #openstack-nova | 05:47 | |
*** yamamoto has joined #openstack-nova | 05:47 | |
*** Apoorva has quit IRC | 05:53 | |
*** Tom-Tom has joined #openstack-nova | 05:53 | |
*** Apoorva has joined #openstack-nova | 05:54 | |
*** Teja has joined #openstack-nova | 05:55 | |
*** sree has quit IRC | 05:56 | |
*** yamamoto has quit IRC | 05:56 | |
*** sree has joined #openstack-nova | 05:56 | |
*** Teja has left #openstack-nova | 05:56 | |
*** takedakn has joined #openstack-nova | 05:58 | |
*** Apoorva has quit IRC | 05:58 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: libvirt: validate SCSI controller model https://review.openstack.org/525055 | 06:00 |
*** sree_ has joined #openstack-nova | 06:02 | |
*** sree_ is now known as Guest38617 | 06:02 | |
*** sree has quit IRC | 06:02 | |
*** takedakn has quit IRC | 06:02 | |
*** moshele has joined #openstack-nova | 06:03 | |
*** fragatina has quit IRC | 06:06 | |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 06:06 |
openstackgerrit | Chen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command https://review.openstack.org/524588 | 06:06 |
*** gcb has quit IRC | 06:08 | |
*** gcb has joined #openstack-nova | 06:10 | |
*** trungnv has joined #openstack-nova | 06:12 | |
*** 07EAALF06 has joined #openstack-nova | 06:12 | |
*** 07EAALF06 has quit IRC | 06:13 | |
*** fragatina has joined #openstack-nova | 06:13 | |
*** AlexeyAbashkin has joined #openstack-nova | 06:15 | |
*** Guest38617 has quit IRC | 06:18 | |
*** Tom-Tom has quit IRC | 06:22 | |
*** sree has joined #openstack-nova | 06:23 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: update the picture in Xen hypervisor document https://review.openstack.org/523360 | 06:25 |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | 06:26 |
*** AlexeyAbashkin has quit IRC | 06:26 | |
*** salv-orlando has joined #openstack-nova | 06:28 | |
*** yamamoto has joined #openstack-nova | 06:28 | |
*** threestrands has quit IRC | 06:32 | |
*** sapd_ has joined #openstack-nova | 06:32 | |
*** sapd has quit IRC | 06:32 | |
*** yamamoto has quit IRC | 06:33 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795 | 06:34 |
*** fragatina has quit IRC | 06:37 | |
openstackgerrit | Merged openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 06:39 |
*** karthiks has quit IRC | 06:41 | |
*** sree_ has joined #openstack-nova | 06:43 | |
*** sree_ is now known as Guest84834 | 06:43 | |
*** yamamoto has joined #openstack-nova | 06:44 | |
openstackgerrit | Merged openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 06:45 |
*** diga has quit IRC | 06:45 | |
*** sree has quit IRC | 06:46 | |
*** sree has joined #openstack-nova | 06:47 | |
*** tetsuro has quit IRC | 06:47 | |
*** yamamoto has quit IRC | 06:48 | |
*** Guest84834 has quit IRC | 06:49 | |
*** salv-orlando has quit IRC | 06:49 | |
*** andreas_s has joined #openstack-nova | 06:53 | |
*** karthiks has joined #openstack-nova | 06:53 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | 06:54 |
openstackgerrit | wanghongxu proposed openstack/nova master: Update cinder info when attach volume that attached https://review.openstack.org/525540 | 06:54 |
*** brault has quit IRC | 06:54 | |
*** andreas_s has quit IRC | 06:54 | |
*** andreas_s has joined #openstack-nova | 06:55 | |
*** yamamoto has joined #openstack-nova | 06:58 | |
*** karthiks has quit IRC | 07:04 | |
*** rcernin has quit IRC | 07:11 | |
*** sree has quit IRC | 07:12 | |
*** sree has joined #openstack-nova | 07:13 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate configurable Hide Server Address Feature https://review.openstack.org/526297 | 07:15 |
*** zhurong has quit IRC | 07:16 | |
*** karthiks has joined #openstack-nova | 07:17 | |
*** sree has quit IRC | 07:21 | |
*** edand has joined #openstack-nova | 07:25 | |
*** sree has joined #openstack-nova | 07:29 | |
*** takashin has left #openstack-nova | 07:30 | |
*** sahid has joined #openstack-nova | 07:30 | |
*** nore_rabel has joined #openstack-nova | 07:35 | |
*** sree has quit IRC | 07:38 | |
*** sree has joined #openstack-nova | 07:39 | |
*** sree_ has joined #openstack-nova | 07:43 | |
*** sree_ is now known as Guest46029 | 07:44 | |
*** slaweq_ has joined #openstack-nova | 07:44 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready https://review.openstack.org/497457 | 07:45 |
*** sree has quit IRC | 07:47 | |
*** gaowen has joined #openstack-nova | 07:49 | |
openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 07:51 |
*** alexchadin has joined #openstack-nova | 07:51 | |
*** yamamoto has quit IRC | 07:51 | |
*** sshwarts has joined #openstack-nova | 07:53 | |
*** yamamoto has joined #openstack-nova | 07:58 | |
*** karthiks has quit IRC | 07:59 | |
*** salv-orlando has joined #openstack-nova | 08:00 | |
*** gaowen has quit IRC | 08:01 | |
*** yamamoto has quit IRC | 08:03 | |
*** salv-orlando has quit IRC | 08:04 | |
*** jaianshu has joined #openstack-nova | 08:05 | |
*** AlexeyAbashkin has joined #openstack-nova | 08:07 | |
*** sahid has quit IRC | 08:07 | |
gmann | jichen: its different policy for keypair not same - https://review.openstack.org/#/c/526258/ | 08:08 |
*** sahid has joined #openstack-nova | 08:08 | |
*** karthiks has joined #openstack-nova | 08:12 | |
*** yamamoto has joined #openstack-nova | 08:14 | |
*** tesseract has joined #openstack-nova | 08:15 | |
*** yamamoto has quit IRC | 08:18 | |
*** yamamoto has joined #openstack-nova | 08:18 | |
*** yamamoto has quit IRC | 08:18 | |
*** rcernin has joined #openstack-nova | 08:19 | |
*** damien_r has joined #openstack-nova | 08:21 | |
*** diga has joined #openstack-nova | 08:21 | |
*** mdnadeem has joined #openstack-nova | 08:22 | |
*** yamamoto has joined #openstack-nova | 08:22 | |
*** huanxie has quit IRC | 08:23 | |
*** yamamoto has quit IRC | 08:24 | |
*** yamamoto has joined #openstack-nova | 08:24 | |
*** yamamoto has quit IRC | 08:24 | |
*** sapd_ has quit IRC | 08:24 | |
*** phuongnh has quit IRC | 08:25 | |
*** phuongnh has joined #openstack-nova | 08:25 | |
*** sapd_ has joined #openstack-nova | 08:26 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | 08:31 |
*** jpena|off is now known as jpena | 08:32 | |
gibi | ildikov: debugging (6) can be easy with tools like http://mgvez.github.io/jsorrery/ | 08:37 |
*** imacdonn has quit IRC | 08:39 | |
*** imacdonn has joined #openstack-nova | 08:40 | |
*** yamamoto has joined #openstack-nova | 08:41 | |
*** ragiman has joined #openstack-nova | 08:44 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate configurable Hide Server Address Feature https://review.openstack.org/526297 | 08:45 |
gmann | alex_xu: can you check these 2 patches - https://review.openstack.org/#/c/526258/ https://review.openstack.org/#/c/526297/ | 08:46 |
*** sahid has quit IRC | 08:47 | |
*** sahid has joined #openstack-nova | 08:48 | |
alex_xu | gmann: yea, add them to my review list, will reach them asap | 08:55 |
gmann | alex_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 approved | 08:56 |
openstackgerrit | Merged openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | 08:56 |
alex_xu | gmann: ok, got it | 08:56 |
*** yamamoto has quit IRC | 08:56 | |
openstackgerrit | Merged openstack/nova master: [placement] re-use existing conf with auth token middleware https://review.openstack.org/523403 | 08:57 |
*** yamamoto has joined #openstack-nova | 08:58 | |
*** salv-orlando has joined #openstack-nova | 09:01 | |
*** yamamoto has quit IRC | 09:01 | |
*** Guest46029 has quit IRC | 09:03 | |
abhishekk | hi, 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 IRC | 09:06 | |
openstackgerrit | Deepak Mourya proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | 09:07 |
*** andreas_s has quit IRC | 09:07 | |
*** andreas_s has joined #openstack-nova | 09:08 | |
*** Eran_Kuris has quit IRC | 09:08 | |
*** andreas_s has quit IRC | 09:12 | |
openstackgerrit | Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395 | 09:13 |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 09:14 |
*** Eran_Kuris has joined #openstack-nova | 09:18 | |
*** jaianshu has quit IRC | 09:19 | |
*** linkmark has joined #openstack-nova | 09:23 | |
*** moshele has quit IRC | 09:29 | |
*** lucas-afk is now known as lucasagomes | 09:35 | |
*** alexchadin has quit IRC | 09:35 | |
*** derekh has joined #openstack-nova | 09:36 | |
openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: provide VGPU_DISPLAY_HEAD inventory in compute node https://review.openstack.org/523342 | 09:37 |
*** TuanLA has joined #openstack-nova | 09:37 | |
*** pooja-jadhav has joined #openstack-nova | 09:38 | |
*** alexchad_ has joined #openstack-nova | 09:38 | |
*** bhagyashri_s has joined #openstack-nova | 09:39 | |
*** Tahvok has quit IRC | 09:40 | |
*** niraj_singh has joined #openstack-nova | 09:40 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: De-dupe image verification https://review.openstack.org/526328 | 09:40 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | 09:40 |
*** diga has quit IRC | 09:40 | |
*** pooja_jadhav has quit IRC | 09:41 | |
*** nsingh has quit IRC | 09:41 | |
*** bhagyashris has quit IRC | 09:41 | |
stephenfin | mdbooth: Not _exactly_ your area of expertise, but mind having a look at ^ | 09:41 |
stephenfin | ? | 09:41 |
mdbooth | stephenfin: Yep | 09:42 |
stephenfin | Ta :) | 09:42 |
*** andreaf has quit IRC | 09:42 | |
*** elod has quit IRC | 09:42 | |
*** elod has joined #openstack-nova | 09:42 | |
*** andreaf has joined #openstack-nova | 09:42 | |
*** Tahvok has joined #openstack-nova | 09:43 | |
mdbooth | Haven't looked at that code since I was working on the vmware driver, I think | 09:45 |
mdbooth | stephenfin: We're reading the whole glance image into ram? | 09:48 |
mdbooth | At one time | 09:49 |
stephenfin | It sure looks like it | 09:49 |
stephenfin | that surprised me too | 09: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 | |
mdbooth | WOW | 09:52 |
mdbooth | Uhh... | 09:52 |
mdbooth | Wow | 09:52 |
* mdbooth wonders how he could turn that into a DoS attack | 09:52 | |
* mdbooth doesn't have to wonder for long | 09:52 | |
* stephenfin wonders what he's stumbled into here | 09:53 | |
mdbooth | Actually this is ringing a bell | 09:53 |
* mdbooth recalls hating this before | 09:54 | |
*** annp has quit IRC | 09:55 | |
*** jichen has quit IRC | 09:55 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 09:56 |
*** tetsuro has joined #openstack-nova | 09:57 | |
* mdbooth was +1 on the patch which added that :/ | 09:58 | |
mdbooth | I don't think I realised at the time that it's also used for download | 09:58 |
*** moshele has joined #openstack-nova | 10:00 | |
*** salv-orlando has joined #openstack-nova | 10:02 | |
*** yamamoto has joined #openstack-nova | 10:02 | |
*** gszasz has joined #openstack-nova | 10:04 | |
*** gmann is now known as gmann_afk | 10:05 | |
*** salv-orlando has quit IRC | 10:06 | |
*** chyka has joined #openstack-nova | 10:07 | |
*** yamamoto has quit IRC | 10:08 | |
*** chyka has quit IRC | 10:12 | |
openstackgerrit | Theodoros Tsioutsias proposed openstack/python-novaclient master: CommandError is raised for invalid server fields https://review.openstack.org/525110 | 10:13 |
bauzas | jianghuaw_: around ? | 10:14 |
bauzas | I'm thinking of a problem about allocations with multiple possible types | 10:14 |
bauzas | for 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 type | 10:15 |
bauzas | now, say we have more than one type | 10:15 |
bauzas | we should then try to understand which type is related to the allocation | 10:15 |
*** TuanLA has quit IRC | 10:18 | |
*** rcernin has quit IRC | 10:21 | |
*** Yingxin_ has joined #openstack-nova | 10:22 | |
*** shaohe_feng_ has joined #openstack-nova | 10:22 | |
*** Yingxin has quit IRC | 10:24 | |
*** Yingxin_ is now known as Yingxin | 10:24 | |
*** alex_xu has quit IRC | 10:24 | |
*** alex_xu has joined #openstack-nova | 10:25 | |
*** shaohe_feng has quit IRC | 10:25 | |
*** shaohe_feng_ is now known as shaohe_feng | 10:25 | |
*** alexchad_ has quit IRC | 10:27 | |
*** alexchadin has joined #openstack-nova | 10:28 | |
*** gcb has quit IRC | 10:29 | |
sean-k-mooney2 | stephenfin: wait what we should never load the glance image into ram in one go. where? | 10:32 |
*** dtantsur|afk is now known as dtantsur | 10:32 | |
stephenfin | sean-k-mooney2: The 'download' function modified here https://review.openstack.org/526328 | 10:32 |
*** kalyan has joined #openstack-nova | 10:33 | |
*** qsyqian has quit IRC | 10:34 | |
sean-k-mooney2 | on line 318? | 10:35 |
stephenfin | Actually, higher https://review.openstack.org/#/c/526328/1/nova/image/glance.py@179 | 10:36 |
sean-k-mooney2 | stephenfin: 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 painful | 10:39 |
*** alexchadin has quit IRC | 10:39 | |
*** yamamoto has joined #openstack-nova | 10:39 | |
*** alexchadin has joined #openstack-nova | 10:40 | |
*** yamamoto has quit IRC | 10:41 | |
sean-k-mooney2 | stephenfin: 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 one | 10:42 |
tetsuro | Hi 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 |
mdbooth | stephenfin: https://review.openstack.org/#/c/526328/1 | 10:43 |
sean-k-mooney2 | tetsuro: sure ill take a look now | 10:43 |
tetsuro | thanks in advance | 10:44 |
*** abhishekk has quit IRC | 10:44 | |
stephenfin | mdbooth: Fair. I'm reporting that as a bug now | 10:44 |
*** Yingxin_ has joined #openstack-nova | 10:48 | |
*** phuongnh has quit IRC | 10:48 | |
*** hoangcx has quit IRC | 10:48 | |
*** phuongnh has joined #openstack-nova | 10:49 | |
*** hoangcx has joined #openstack-nova | 10:49 | |
*** shaohe_feng has quit IRC | 10:50 | |
*** hieulq has quit IRC | 10:50 | |
*** trungnv has quit IRC | 10:50 | |
*** trungnv has joined #openstack-nova | 10:50 | |
*** Yingxin has quit IRC | 10:50 | |
*** Yingxin_ is now known as Yingxin | 10:50 | |
*** hieulq has joined #openstack-nova | 10:50 | |
*** alex_xu has quit IRC | 10:50 | |
*** qsyqian has joined #openstack-nova | 10:51 | |
*** alex_xu has joined #openstack-nova | 10:51 | |
*** shaohe_feng has joined #openstack-nova | 10:51 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | 10:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Handle images with no data https://review.openstack.org/526329 | 10:54 |
*** sdague has joined #openstack-nova | 10:56 | |
*** sdague has quit IRC | 10:57 | |
*** sambetts|afk is now known as sambetts | 11:02 | |
*** salv-orlando has joined #openstack-nova | 11:02 | |
*** andreas_s has joined #openstack-nova | 11:03 | |
*** salv-orlando has quit IRC | 11:07 | |
*** qsyqian has quit IRC | 11:11 | |
sean-k-mooney2 | bauzas: if your device support multiple mdev types you need to have multiple resouce provider pools. | 11:15 |
sean-k-mooney2 | bauzas: the tricky bit is doing the accounting correctly | 11:16 |
sean-k-mooney2 | when you claim the mdev you will have to revailidate the available_instances for all other mdev types on the same Physical device | 11:17 |
*** sdague has joined #openstack-nova | 11:22 | |
*** qsyqian has joined #openstack-nova | 11:23 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | 11:24 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add an online migration for BDM.uuid https://review.openstack.org/525599 | 11:24 |
openstackgerrit | Matthew Booth proposed openstack/nova master: DriverBlockDevice: make subclasses inherit _proxy_as_attr https://review.openstack.org/524167 | 11:24 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | 11:24 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add DriverLocalImageBlockDevice https://review.openstack.org/526347 | 11:24 |
*** liverpooler has quit IRC | 11:25 | |
*** qsyqian has quit IRC | 11:28 | |
openstackgerrit | Chen Hanxiao proposed openstack/nova master: InstanceInfo: store more runtime information of instance https://review.openstack.org/526348 | 11:28 |
*** udesale has quit IRC | 11:28 | |
*** yamamoto has joined #openstack-nova | 11:31 | |
*** Eran_Kuris has quit IRC | 11:36 | |
*** janki has quit IRC | 11:40 | |
*** janki has joined #openstack-nova | 11:40 | |
*** qsyqian has joined #openstack-nova | 11:41 | |
*** alexchadin has quit IRC | 11:43 | |
*** alexchadin has joined #openstack-nova | 11:43 | |
*** Eran_Kuris has joined #openstack-nova | 11:48 | |
*** smatzek has joined #openstack-nova | 11:57 | |
*** slaweq_ has quit IRC | 11:57 | |
*** qsyqian has quit IRC | 12:01 | |
*** salv-orlando has joined #openstack-nova | 12:03 | |
openstackgerrit | Chris Dent proposed openstack/nova stable/pike: [placement] re-use existing conf with auth token middleware https://review.openstack.org/526351 | 12:04 |
*** slaweq has joined #openstack-nova | 12:06 | |
*** salv-orlando has quit IRC | 12:07 | |
*** tetsuro has quit IRC | 12:08 | |
*** Eran_Kuris has quit IRC | 12:09 | |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 12:10 |
*** andreas_s has quit IRC | 12:12 | |
openstackgerrit | Wangpan proposed openstack/nova master: Handle glance exception during rotating instance backup https://review.openstack.org/523395 | 12:13 |
*** andreas_s has joined #openstack-nova | 12:13 | |
*** andreas_s has quit IRC | 12:22 | |
*** andreas_s has joined #openstack-nova | 12:22 | |
openstackgerrit | garyk proposed openstack/nova master: Conductor: ensure build cleanup works with cell https://review.openstack.org/526356 | 12:33 |
*** pchavva has joined #openstack-nova | 12:35 | |
jaypipes | sean-k-mooney2: I want that original pug. | 12:36 |
*** hui has quit IRC | 12:36 | |
sean-k-mooney2 | jaypipes: did you see my latest responce | 12:37 |
jaypipes | sean-k-mooney2: ya, reading it now :) | 12:37 |
*** slaweq has quit IRC | 12:38 | |
*** jpena is now known as jpena|lunch | 12:40 | |
sean-k-mooney2 | jaypipes: 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 |
jaypipes | sean-k-mooney2: no idea... we should ask clarkb or fungi | 12:41 |
*** andreas_s has quit IRC | 12:42 | |
sean-k-mooney2 | jaypipes: 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 be | 12:42 |
jaypipes | sean-k-mooney2: sounds good, thanks man | 12:42 |
*** andreas_s has joined #openstack-nova | 12:43 | |
sean-k-mooney2 | for 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-mooney2 | s/path/patch | 12:46 |
*** lucasagomes is now known as lucas-hungry | 12:47 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | 12:50 |
openstack | bug 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-nova | 12:52 | |
*** andreas_s has quit IRC | 12:52 | |
openstackgerrit | garyk proposed openstack/nova master: Conductor: ensure build cleanup works with cell https://review.openstack.org/526356 | 12:56 |
*** sahid has quit IRC | 12:58 | |
*** psachin has quit IRC | 12:59 | |
jaypipes | sean-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-nova | 12:59 | |
efried | alex_xu We took a vote and settled on in_tree=<uuid> | 13:01 |
sean-k-mooney2 | jaypipes: 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 that | 13:02 |
*** phuongnh has quit IRC | 13:04 | |
*** salv-orlando has joined #openstack-nova | 13:04 | |
jaypipes | efried: mornin. | 13:04 |
efried | jaypipes Howdy! | 13:04 |
*** liverpooler has joined #openstack-nova | 13:05 | |
jaypipes | efried: that's right... I forgot you're in Texas. :) | 13:05 |
jaypipes | pistols at dawn and all that. :P | 13:05 |
jaypipes | efried: though frankly, Florida might have just as many guns as Texas. | 13:06 |
efried | I 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 |
jaypipes | efried: my favorite thing about Sarasota is the ammo/gun store directly across from the courthouse called "The Bullet Hole" | 13:06 |
efried | Heh | 13:06 |
jaypipes | efried: touche | 13:06 |
jaypipes | efried: 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-nova | 13:07 | |
* efried clicks... | 13:08 | |
efried | wow | 13:08 |
*** erlon has joined #openstack-nova | 13:09 | |
*** salv-orl_ has joined #openstack-nova | 13:09 | |
*** slaweq has quit IRC | 13:09 | |
*** slaweq has joined #openstack-nova | 13:10 | |
*** salv-orlando has quit IRC | 13:11 | |
jaypipes | efried: you can thank dansmith for that :) there goes an hour of work time. | 13:11 |
*** Eran_Kuris has joined #openstack-nova | 13:11 | |
efried | jaypipes It's better than The Onion. | 13:11 |
jaypipes | efried: 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 |
efried | Hah, it's like that scene from one of the old Superman movies. | 13:12 |
jaypipes | yep. only "Florida style" | 13:12 |
jaypipes | efried: ok, so back to work stuff... how we doing on the n-r-p series. I see only the bottom patch merged. | 13:14 |
efried | Yup | 13:15 |
efried | I rebased (and slightly reworked) the pile of patches on top of that. | 13:15 |
efried | And also resurrected (and corrected) this series: https://review.openstack.org/#/c/521605/ | 13:15 |
*** janki has quit IRC | 13:15 | |
efried | ...which puts caching of aggs & traits into ProviderTree and out of the report client. | 13:16 |
jaypipes | ack | 13:16 |
jaypipes | OK, let me review that as soon as I comment on sean-k-mooney2's virt_type patch | 13:16 |
efried | Not 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 |
jaypipes | right | 13:16 |
efried | But if we can get 'em merged fairly quick... | 13:16 |
*** yangyapeng has quit IRC | 13:16 | |
*** andreas_s has quit IRC | 13:16 | |
*** yangyapeng has joined #openstack-nova | 13:17 | |
*** dave-mccowan has joined #openstack-nova | 13:18 | |
*** andreas_s has joined #openstack-nova | 13:18 | |
efried | jaypipes 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 |
jaypipes | efried: ack, that would be great. | 13:18 |
efried | jaypipes 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 |
jaypipes | efried: yes | 13:19 |
efried | k | 13:19 |
*** yangyapeng has quit IRC | 13:21 | |
*** alexchadin has quit IRC | 13:21 | |
*** andreas_s has quit IRC | 13:27 | |
*** lucas-hungry is now known as lucasagomes | 13:29 | |
*** sahid has joined #openstack-nova | 13:31 | |
*** andreas_s has joined #openstack-nova | 13:32 | |
*** salv-orl_ has quit IRC | 13:33 | |
*** salv-orlando has joined #openstack-nova | 13:33 | |
*** janki has joined #openstack-nova | 13:34 | |
*** slaweq has quit IRC | 13:35 | |
*** brault has joined #openstack-nova | 13:36 | |
*** slaweq has joined #openstack-nova | 13:37 | |
*** brault_ has joined #openstack-nova | 13:38 | |
*** salv-orlando has quit IRC | 13:39 | |
*** brault_ has quit IRC | 13:39 | |
*** brault_ has joined #openstack-nova | 13:39 | |
*** brault has quit IRC | 13:40 | |
*** andreas_s has quit IRC | 13:41 | |
*** andreas_s has joined #openstack-nova | 13:43 | |
*** tetsuro has joined #openstack-nova | 13:44 | |
*** yangyapeng has joined #openstack-nova | 13:45 | |
*** jpena|lunch is now known as jpena | 13:46 | |
*** andreas_s has quit IRC | 13:47 | |
*** andreas_s has joined #openstack-nova | 13:48 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Reformat _get_all_with_shared https://review.openstack.org/512799 | 13:51 |
*** peter-hamilton has joined #openstack-nova | 13:52 | |
*** alexchadin has joined #openstack-nova | 13:52 | |
*** abhishekk has joined #openstack-nova | 13:54 | |
*** alexchadin has quit IRC | 13:56 | |
*** mriedem has joined #openstack-nova | 13:57 | |
*** takedakn has joined #openstack-nova | 13:58 | |
*** salv-orlando has joined #openstack-nova | 13:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/523194 | 13:59 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | 13:59 |
mriedem | alex_xu: removed that file finally, thanks for pointing that out again ^ | 14:00 |
mriedem | sdague: can we get https://review.openstack.org/523194 in so i can get it backported to newton before eol | 14:01 |
*** links has quit IRC | 14:01 | |
*** lyan has joined #openstack-nova | 14:03 | |
*** tetsuro has quit IRC | 14:07 | |
sdague | mriedem: +2 from me | 14:07 |
alex_xu | mriedem: np, +w | 14:10 |
mriedem | thanks | 14:10 |
mriedem | the functional py35 logs are unreadable due to deprecation warnigns | 14:12 |
*** AlexeyAbashkin has quit IRC | 14:12 | |
mriedem | *warnings | 14:12 |
mriedem | from oslo.context and oslo.policy | 14:12 |
openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 14: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 |
fungi | jaypipes: 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 |
jaypipes | fungi: :) | 14:14 |
mriedem | bauzas: 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 tracker | 14:15 |
jianghuaw_ | bauzas, it should work if we can mapping back from RP uuid to the PGPU/gpu group. | 14:15 |
mriedem | when i compared logs, it went from logging nearly 400 times to 35 | 14: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-nova | 14:16 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: update_cell allows more than once cell to have the same db/transport url https://review.openstack.org/518556 | 14:19 |
jianghuaw_ | bauzas, see this patch: https://review.openstack.org/#/c/521717 | 14:19 |
bauzas | mriedem: agreed | 14:19 |
bauzas | mriedem: planning to backport ? or want me to do such thing ? | 14:20 |
bauzas | jianghuaw_: I think we should discuss maybe later (I'm a but rushing this afternoon tbh) about how to have multiple supported types | 14:20 |
bauzas | for the moment, let's merge only one per compute in Queens | 14:20 |
jianghuaw_ | bauzas, sure. | 14:21 |
bauzas | we also need to have flavor traits for that, remember | 14: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 |
efried | jaypipes 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 |
jaypipes | efried: k | 14:23 |
* efried gets his git wand out | 14:23 | |
*** Eran_Kuris has quit IRC | 14:23 | |
mriedem | bauzas: i'm planning on backporting | 14:25 |
bauzas | mriedem: k, ping me when you're done | 14:27 |
*** salv-orlando has quit IRC | 14:27 | |
*** salv-orlando has joined #openstack-nova | 14:28 | |
*** smatzek has quit IRC | 14:28 | |
*** edmondsw has joined #openstack-nova | 14:30 | |
*** awaugama has joined #openstack-nova | 14:30 | |
bauzas | jianghuaw_: wait, just saw https://review.openstack.org/#/c/516899/30/nova/tests/unit/virt/xenapi/test_driver.py@400 | 14:31 |
bauzas | snap, missed https://review.openstack.org/#/c/516217/24/nova/virt/xenapi/driver.py@457 | 14:31 |
dansmith | jaypipes: that wasn't me dude.. I appreciate the undue credit and all, but ... :P | 14:32 |
jianghuaw_ | bauzas, yes. That's because XenServer can only support one VGPU per VM. | 14:33 |
bauzas | but in libvirt, max_unit = max(vgpus) | 14:33 |
*** salv-orlando has quit IRC | 14:33 | |
bauzas | jianghuaw_: I think there is a confusion | 14:33 |
bauzas | ah ? | 14:33 |
jianghuaw_ | I saw that difference and I asked if libvirt can support multiple VGPUs. | 14:33 |
bauzas | in theory, one can create multiple mdevs in libvirt | 14:34 |
jianghuaw_ | There is no way to create multiple VGPU in XenServer at the moment. | 14:34 |
bauzas | jianghuaw_: that said, I haven't tested it yet | 14:34 |
bauzas | multiple 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 |
bauzas | jianghuaw_: given it's experimental for libvirt, I probably need to test that once I'm done with implementing it | 14:37 |
bauzas | but you're fortunate, you're at least 3 people working on the xen side, compared to one on libvirt :p | 14:37 |
bauzas | one being overcommitted :p | 14:37 |
jianghuaw_ | bauzas, That's true. We had 3 people. | 14:38 |
jianghuaw_ | :-) | 14:38 |
bauzas | you *had* ? | 14:38 |
*** takedakn has quit IRC | 14: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 |
bauzas | k | 14:42 |
*** ttx has quit IRC | 14: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 IRC | 14:48 | |
*** BryanS68 has quit IRC | 14:48 | |
*** Yingxin has quit IRC | 14:48 | |
*** niraj_singh has quit IRC | 14:48 | |
*** bhagyashri_s has quit IRC | 14:48 | |
*** pooja-jadhav has quit IRC | 14:48 | |
*** edand has quit IRC | 14:48 | |
*** openstackgerrit has quit IRC | 14:48 | |
*** _pewp_ has quit IRC | 14:48 | |
*** tlbr has quit IRC | 14:48 | |
*** lyarwood has quit IRC | 14:48 | |
*** gryf has quit IRC | 14:48 | |
*** ericyoung has quit IRC | 14:48 | |
*** lifeless has quit IRC | 14:48 | |
*** mfisch has quit IRC | 14:48 | |
*** afazekas has quit IRC | 14:48 | |
*** diegows has quit IRC | 14:48 | |
*** antonym has quit IRC | 14:48 | |
*** yamamoto has quit IRC | 14:48 | |
*** nore_rabel has quit IRC | 14:48 | |
*** adisky_ has quit IRC | 14:48 | |
*** sean-k-mooney2 has quit IRC | 14:48 | |
*** Kvisle has quit IRC | 14:48 | |
*** clayton has quit IRC | 14:48 | |
*** tinwood has quit IRC | 14:48 | |
*** amotoki has quit IRC | 14:48 | |
*** McNinja has quit IRC | 14:48 | |
*** betherly has quit IRC | 14:48 | |
*** mwhahaha has quit IRC | 14:48 | |
*** fungi has quit IRC | 14:48 | |
*** johnsom has quit IRC | 14:48 | |
*** thingee has quit IRC | 14:48 | |
*** ameade has quit IRC | 14:48 | |
*** TheJulia has quit IRC | 14:48 | |
*** wxy has quit IRC | 14:48 | |
*** karlamrhein has quit IRC | 14:48 | |
*** geekinutah has quit IRC | 14:48 | |
*** robcresswell has quit IRC | 14:48 | |
*** logan- has quit IRC | 14:48 | |
*** jogo has quit IRC | 14:48 | |
*** DuncanT has quit IRC | 14:48 | |
*** toan has quit IRC | 14:48 | |
*** johnthetubaguy has quit IRC | 14:48 | |
*** peter-hamilton has quit IRC | 14:48 | |
*** kalyan has quit IRC | 14:48 | |
*** ChanServ has quit IRC | 14:48 | |
*** pchavva has quit IRC | 14:48 | |
*** Guest99196 has quit IRC | 14:48 | |
*** yikun has quit IRC | 14:48 | |
*** mvk has quit IRC | 14:48 | |
*** slunkad_ has quit IRC | 14:48 | |
*** StevenK has quit IRC | 14:48 | |
*** alaski has quit IRC | 14:48 | |
*** ltomasbo has quit IRC | 14:48 | |
*** cargonza has quit IRC | 14:48 | |
*** d34dh0r53 has quit IRC | 14:48 | |
*** moshele has quit IRC | 14:48 | |
*** FoXMaN has quit IRC | 14:48 | |
*** jangutter has quit IRC | 14:48 | |
*** toabctl has quit IRC | 14:48 | |
*** Dave has quit IRC | 14:48 | |
*** szaher has quit IRC | 14:48 | |
*** larsks has quit IRC | 14:48 | |
*** BlackDex has quit IRC | 14:48 | |
*** naichuans_ has quit IRC | 14:48 | |
*** spotz has quit IRC | 14:48 | |
*** jpena has quit IRC | 14:48 | |
*** sdague has quit IRC | 14:48 | |
*** Tahvok has quit IRC | 14:48 | |
*** imacdonn has quit IRC | 14:48 | |
*** jose-phillips has quit IRC | 14:48 | |
*** masber has quit IRC | 14:48 | |
*** anthonyper has quit IRC | 14:48 | |
*** hoonetorg has quit IRC | 14:48 | |
*** egarbade has quit IRC | 14:48 | |
*** ianw has quit IRC | 14:48 | |
*** zigo has quit IRC | 14:48 | |
*** Nil_ has quit IRC | 14:48 | |
*** swamireddy has quit IRC | 14:48 | |
*** NightKhaos has quit IRC | 14:48 | |
*** jhesketh has quit IRC | 14:48 | |
*** andreas_s has quit IRC | 14:48 | |
*** brault_ has quit IRC | 14:48 | |
*** erlon has quit IRC | 14:48 | |
*** gszasz has quit IRC | 14:48 | |
*** sapd_ has quit IRC | 14:48 | |
*** sridharg has quit IRC | 14:48 | |
*** mikal has quit IRC | 14:48 | |
*** hamzy has quit IRC | 14:48 | |
*** gmann_afk has quit IRC | 14:48 | |
*** knikolla has quit IRC | 14:48 | |
*** oanson has quit IRC | 14:48 | |
*** auggy has quit IRC | 14:48 | |
*** tristanC has quit IRC | 14:48 | |
*** flwang has quit IRC | 14:48 | |
*** efried has quit IRC | 14:48 | |
*** vipul has quit IRC | 14:48 | |
*** rwellum has quit IRC | 14:48 | |
*** tssurya has quit IRC | 14:48 | |
*** jaypipes has quit IRC | 14:48 | |
*** calebb has quit IRC | 14:48 | |
*** dave-mccowan has quit IRC | 14:48 | |
*** liusheng has quit IRC | 14:48 | |
*** serverascode has quit IRC | 14:48 | |
*** melwitt has quit IRC | 14:48 | |
*** kencjohnston has quit IRC | 14:48 | |
*** oomichi_afk has quit IRC | 14:48 | |
*** tobasco has quit IRC | 14:48 | |
*** keekz has quit IRC | 14:48 | |
*** ioni has quit IRC | 14:48 | |
*** mriedem has quit IRC | 14:48 | |
*** alex_xu has quit IRC | 14:48 | |
*** sshwarts has quit IRC | 14:48 | |
*** jaosorior has quit IRC | 14:48 | |
*** syjulian has quit IRC | 14:48 | |
*** mdrabe has quit IRC | 14:48 | |
*** kashyap has quit IRC | 14:48 | |
*** ejat has quit IRC | 14:48 | |
*** shaner has quit IRC | 14:48 | |
*** sambetts has quit IRC | 14:48 | |
*** sileht has quit IRC | 14:48 | |
*** masayukig has quit IRC | 14:48 | |
*** s1061123 has quit IRC | 14:48 | |
*** jbernard has quit IRC | 14:48 | |
*** hemna has quit IRC | 14:48 | |
*** Vek has quit IRC | 14:48 | |
*** lennyb has quit IRC | 14:48 | |
*** janki has quit IRC | 14:48 | |
*** hoangcx has quit IRC | 14:48 | |
*** tbachman has quit IRC | 14:48 | |
*** coreywright has quit IRC | 14:48 | |
*** cfriesen_ has quit IRC | 14:48 | |
*** aloga has quit IRC | 14:48 | |
*** EmilienM has quit IRC | 14:48 | |
*** migi has quit IRC | 14:48 | |
*** jdillaman has quit IRC | 14:48 | |
*** sapcc-bot has quit IRC | 14:48 | |
*** tonyb has quit IRC | 14:48 | |
*** mgariepy has quit IRC | 14:48 | |
*** rmk has quit IRC | 14:48 | |
*** markmc has quit IRC | 14:48 | |
*** bfife has quit IRC | 14:48 | |
*** mdbooth has quit IRC | 14:48 | |
*** ansiwen has quit IRC | 14:48 | |
*** brad[] has quit IRC | 14:48 | |
*** stephenfin has quit IRC | 14:48 | |
*** rybridges has quit IRC | 14:48 | |
*** andreykurilin has quit IRC | 14:48 | |
*** DinaBelova has quit IRC | 14:48 | |
*** aignatov has quit IRC | 14:48 | |
*** Swami has joined #openstack-nova | 14:49 | |
*** yikun_ has joined #openstack-nova | 14:51 | |
*** lei-zh has joined #openstack-nova | 14:52 | |
*** yamamoto has joined #openstack-nova | 14:54 | |
*** adisky_ has joined #openstack-nova | 14:54 | |
*** sean-k-mooney2 has joined #openstack-nova | 14:54 | |
*** Kvisle has joined #openstack-nova | 14:54 | |
*** clayton has joined #openstack-nova | 14:54 | |
*** tinwood has joined #openstack-nova | 14:54 | |
*** amotoki has joined #openstack-nova | 14:54 | |
*** McNinja has joined #openstack-nova | 14:54 | |
*** betherly has joined #openstack-nova | 14:54 | |
*** mwhahaha has joined #openstack-nova | 14:54 | |
*** fungi has joined #openstack-nova | 14:54 | |
*** johnsom has joined #openstack-nova | 14:54 | |
*** thingee has joined #openstack-nova | 14:54 | |
*** ameade has joined #openstack-nova | 14:54 | |
*** TheJulia has joined #openstack-nova | 14:54 | |
*** wxy has joined #openstack-nova | 14:54 | |
*** karlamrhein has joined #openstack-nova | 14:54 | |
*** geekinutah has joined #openstack-nova | 14:54 | |
*** robcresswell has joined #openstack-nova | 14:54 | |
*** logan- has joined #openstack-nova | 14:54 | |
*** jogo has joined #openstack-nova | 14:54 | |
*** DuncanT has joined #openstack-nova | 14:54 | |
*** toan has joined #openstack-nova | 14:54 | |
*** johnthetubaguy has joined #openstack-nova | 14:54 | |
*** pchavva has joined #openstack-nova | 14:54 | |
*** Guest99196 has joined #openstack-nova | 14:54 | |
*** yikun has joined #openstack-nova | 14:54 | |
*** mvk has joined #openstack-nova | 14:54 | |
*** slunkad_ has joined #openstack-nova | 14:54 | |
*** StevenK has joined #openstack-nova | 14:54 | |
*** alaski has joined #openstack-nova | 14:54 | |
*** ltomasbo has joined #openstack-nova | 14:54 | |
*** cargonza has joined #openstack-nova | 14:54 | |
*** d34dh0r53 has joined #openstack-nova | 14:54 | |
*** ttx has joined #openstack-nova | 14:54 | |
*** antonym has joined #openstack-nova | 14:54 | |
*** diegows has joined #openstack-nova | 14:54 | |
*** afazekas has joined #openstack-nova | 14:54 | |
*** mfisch has joined #openstack-nova | 14:54 | |
*** lifeless has joined #openstack-nova | 14:54 | |
*** ericyoung has joined #openstack-nova | 14:54 | |
*** gryf has joined #openstack-nova | 14:54 | |
*** lyarwood has joined #openstack-nova | 14:54 | |
*** tlbr has joined #openstack-nova | 14:54 | |
*** _pewp_ has joined #openstack-nova | 14:54 | |
*** openstackgerrit has joined #openstack-nova | 14:54 | |
*** edand has joined #openstack-nova | 14:54 | |
*** pooja-jadhav has joined #openstack-nova | 14:54 | |
*** bhagyashri_s has joined #openstack-nova | 14:54 | |
*** niraj_singh has joined #openstack-nova | 14:54 | |
*** Yingxin has joined #openstack-nova | 14:54 | |
*** BryanS68 has joined #openstack-nova | 14:54 | |
*** yangyapeng has joined #openstack-nova | 14:54 | |
*** nore_rabel has joined #openstack-nova | 14:54 | |
*** amodi has joined #openstack-nova | 14:54 | |
*** gouthamr has joined #openstack-nova | 14:54 | |
*** mriedem has joined #openstack-nova | 14:54 | |
*** andreas_s has joined #openstack-nova | 14:54 | |
*** brault_ has joined #openstack-nova | 14:54 | |
*** janki has joined #openstack-nova | 14:54 | |
*** dave-mccowan has joined #openstack-nova | 14:54 | |
*** erlon has joined #openstack-nova | 14:54 | |
*** sdague has joined #openstack-nova | 14:54 | |
*** alex_xu has joined #openstack-nova | 14:54 | |
*** hoangcx has joined #openstack-nova | 14:54 | |
*** gszasz has joined #openstack-nova | 14:54 | |
*** moshele has joined #openstack-nova | 14:54 | |
*** Tahvok has joined #openstack-nova | 14:54 | |
*** imacdonn has joined #openstack-nova | 14:54 | |
*** sapd_ has joined #openstack-nova | 14:54 | |
*** sshwarts has joined #openstack-nova | 14:54 | |
*** tbachman has joined #openstack-nova | 14:54 | |
*** sridharg has joined #openstack-nova | 14:54 | |
*** jaosorior has joined #openstack-nova | 14:54 | |
*** FoXMaN has joined #openstack-nova | 14:54 | |
*** liusheng has joined #openstack-nova | 14:54 | |
*** mikal has joined #openstack-nova | 14:54 | |
*** syjulian has joined #openstack-nova | 14:54 | |
*** hamzy has joined #openstack-nova | 14:54 | |
*** jangutter has joined #openstack-nova | 14:54 | |
*** coreywright has joined #openstack-nova | 14:54 | |
*** gmann_afk has joined #openstack-nova | 14:54 | |
*** knikolla has joined #openstack-nova | 14:54 | |
*** cfriesen_ has joined #openstack-nova | 14:54 | |
*** oanson has joined #openstack-nova | 14:54 | |
*** auggy has joined #openstack-nova | 14:54 | |
*** tristanC has joined #openstack-nova | 14:54 | |
*** flwang has joined #openstack-nova | 14:54 | |
*** efried has joined #openstack-nova | 14:54 | |
*** jose-phillips has joined #openstack-nova | 14:54 | |
*** vipul has joined #openstack-nova | 14:54 | |
*** masber has joined #openstack-nova | 14:54 | |
*** rwellum has joined #openstack-nova | 14:54 | |
*** tssurya has joined #openstack-nova | 14:54 | |
*** jaypipes has joined #openstack-nova | 14:54 | |
*** aloga has joined #openstack-nova | 14:54 | |
*** anthonyper has joined #openstack-nova | 14:54 | |
*** calebb has joined #openstack-nova | 14:54 | |
*** hoonetorg has joined #openstack-nova | 14:54 | |
*** mdrabe has joined #openstack-nova | 14:54 | |
*** egarbade has joined #openstack-nova | 14:54 | |
*** ianw has joined #openstack-nova | 14:54 | |
*** toabctl has joined #openstack-nova | 14:54 | |
*** Nil_ has joined #openstack-nova | 14:54 | |
*** zigo has joined #openstack-nova | 14:54 | |
*** EmilienM has joined #openstack-nova | 14:54 | |
*** kashyap has joined #openstack-nova | 14:54 | |
*** Dave has joined #openstack-nova | 14:54 | |
*** stephenfin has joined #openstack-nova | 14:54 | |
*** migi has joined #openstack-nova | 14:54 | |
*** ejat has joined #openstack-nova | 14:54 | |
*** shaner has joined #openstack-nova | 14:54 | |
*** jdillaman has joined #openstack-nova | 14:54 | |
*** sapcc-bot has joined #openstack-nova | 14:54 | |
*** szaher has joined #openstack-nova | 14:54 | |
*** sambetts has joined #openstack-nova | 14:54 | |
*** larsks has joined #openstack-nova | 14:54 | |
*** sileht has joined #openstack-nova | 14:54 | |
*** swamireddy has joined #openstack-nova | 14:54 | |
*** tonyb has joined #openstack-nova | 14:54 | |
*** mgariepy has joined #openstack-nova | 14:54 | |
*** BlackDex has joined #openstack-nova | 14:54 | |
*** NightKhaos has joined #openstack-nova | 14:54 | |
*** jhesketh has joined #openstack-nova | 14:54 | |
*** masayukig has joined #openstack-nova | 14:54 | |
*** s1061123 has joined #openstack-nova | 14:54 | |
*** naichuans_ has joined #openstack-nova | 14:54 | |
*** spotz has joined #openstack-nova | 14:54 | |
*** jpena has joined #openstack-nova | 14:54 | |
*** jbernard has joined #openstack-nova | 14:54 | |
*** hemna has joined #openstack-nova | 14:54 | |
*** Vek has joined #openstack-nova | 14:54 | |
*** rmk has joined #openstack-nova | 14:54 | |
*** lennyb has joined #openstack-nova | 14:54 | |
*** markmc has joined #openstack-nova | 14:54 | |
*** serverascode has joined #openstack-nova | 14:54 | |
*** bfife has joined #openstack-nova | 14:54 | |
*** mdbooth has joined #openstack-nova | 14:54 | |
*** ansiwen has joined #openstack-nova | 14:54 | |
*** brad[] has joined #openstack-nova | 14:54 | |
*** rybridges has joined #openstack-nova | 14:54 | |
*** melwitt has joined #openstack-nova | 14:54 | |
*** kencjohnston has joined #openstack-nova | 14:54 | |
*** andreykurilin has joined #openstack-nova | 14:54 | |
*** tobasco has joined #openstack-nova | 14:54 | |
*** oomichi_afk has joined #openstack-nova | 14:54 | |
*** DinaBelova has joined #openstack-nova | 14:54 | |
*** aignatov has joined #openstack-nova | 14:54 | |
*** keekz has joined #openstack-nova | 14:54 | |
*** ioni has joined #openstack-nova | 14:54 | |
*** yikun_ has quit IRC | 14:55 | |
*** yikun_ has joined #openstack-nova | 14:55 | |
*** mtreinish has quit IRC | 14:55 | |
*** salv-orlando has joined #openstack-nova | 14:57 | |
*** mlavalle has joined #openstack-nova | 14:57 | |
*** cdent has joined #openstack-nova | 14:58 | |
*** yamahata has joined #openstack-nova | 14:58 | |
*** mtreinish has joined #openstack-nova | 14:58 | |
*** abhishekk has quit IRC | 15:00 | |
*** artom has quit IRC | 15:02 | |
mriedem | gibi: ildikov: replied in https://review.openstack.org/#/c/525787/ | 15:03 |
mriedem | gibi: thanks for the thorough review as always | 15:03 |
ildikov | mriedem: Tnx, will check in a bit | 15:04 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/526426 | 15:04 |
mriedem | sdague: 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:open | 15:06 |
dansmith | ya | 15:06 |
dansmith | mriedem: what is the pike only workaround? that hash in the commit message links to this | 15:07 |
mriedem | lyarwood: 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/pike | 15:07 |
dansmith | oh wait | 15:07 |
mriedem | dansmith: this https://review.openstack.org/#/c/523213/4/nova/tests/functional/test_servers.py | 15:07 |
mriedem | the setUp | 15:07 |
mriedem | i should have said, "in the functional test" or something | 15:08 |
dansmith | yeah, okay I was expecting something functional | 15:08 |
dansmith | yeah | 15:08 |
dansmith | also it just links back to this series which was confusing me about where the thing was added | 15:08 |
dansmith | outside of gerrit that will link properly though so that's fine | 15:08 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/526427 | 15:09 |
lyarwood | mriedem: yup will try, still handing over all of my tripleo work atm, hopefully back to nova full time next week btw | 15:09 |
mriedem | i anticipate your glorious return | 15:09 |
*** alexchadin has joined #openstack-nova | 15:11 | |
mriedem | sdague: 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 also | 15:12 |
gibi | mriedem: thanks for the reply in the attach flow patch. Now the BFV case is clear for me | 15:13 |
edleafe | dansmith: I have a question about the changes to the filter_scheduler._claim_resources() method | 15:13 |
mriedem | gibi: yeah that took me some time to figure out why those tests weren't working as i'd expected | 15:13 |
edleafe | dansmith: the docstring about the params doesn't match the params to the method | 15:13 |
*** alexchadin has quit IRC | 15:13 | |
edleafe | dansmith: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L272-L293 | 15:14 |
mriedem | jgriffith: do you remember why you added this? https://review.openstack.org/#/c/525787/3/nova/virt/block_device.py@448 | 15:14 |
mriedem | jgriffith: doesn't seem necessary so i plan on removing it unless there was a need for this | 15:14 |
dansmith | edleafe: is that a change I made? | 15:14 |
edleafe | dansmith: I think so. Were there any other changes to _claim_resources? | 15:15 |
edleafe | dansmith: just the rebuild check, right? | 15:15 |
dansmith | edleafe: I dunno, let me dig up history.. what's your question though? | 15:15 |
*** sahid has quit IRC | 15:16 | |
*** peter-hamilton has joined #openstack-nova | 15:16 | |
*** kalyan has joined #openstack-nova | 15:16 | |
*** ChanServ has joined #openstack-nova | 15:16 | |
*** barjavel.freenode.net sets mode: +o ChanServ | 15:16 | |
*** damien_r has quit IRC | 15:16 | |
edleafe | dansmith: rebasing my series. The docstring says that it is getting and instance UUID, a HostState object, and allocation_requests | 15:16 |
*** damien_r has joined #openstack-nova | 15:16 | |
mriedem | gibi: 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 |
edleafe | but it's still just getting a spec_obj, instance uuid and allocation_requests | 15:17 |
dansmith | edleafe: This is the change I didn't change the params: https://review.openstack.org/#/c/521662/10/nova/scheduler/filter_scheduler.py | 15:17 |
mriedem | gibi: maybe i should just remove the key from the *_driver_bdm variables since they are set in setUp anyway | 15:17 |
mriedem | to avoid confusion about why they are there | 15:17 |
edleafe | but it's still just getting a spec_obj, instance uuid and allocation_requests | 15:17 |
edleafe | oops | 15:17 |
edleafe | ok, then I'll just change the docstring to be correct | 15:18 |
dansmith | edleafe: yeah, I'm not sure where that docstring diverged from the signature, but it's clearly wrong | 15:18 |
edleafe | I was wondering if there were other changes being merged that affected it | 15:18 |
edleafe | dansmith: got it | 15:18 |
*** moshele has quit IRC | 15:18 | |
gibi | mriedem: just remove that is anyhow set in the setUp | 15:18 |
gibi | mriedem: but I that is the 4 change in top of the test_block_device.py :) | 15:19 |
edleafe | dansmith: 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 docstring | 15:21 |
sdague | mriedem: yeh, +2 on that still | 15:21 |
*** janki has quit IRC | 15:21 | |
*** smatzek has joined #openstack-nova | 15:21 | |
mriedem | sdague: thanks | 15:21 |
dansmith | edleafe: that sounds like him | 15:21 |
edleafe | dansmith: :) | 15:21 |
*** gbarros has joined #openstack-nova | 15:21 | |
openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree https://review.openstack.org/520663 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates https://review.openstack.org/521097 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: Traits ops on ProviderTree https://review.openstack.org/521605 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: Aggregate ops on ProviderTree https://review.openstack.org/526216 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree https://review.openstack.org/521685 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: Track provider traits in report client https://review.openstack.org/521686 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 15:23 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 15:23 |
efried | jaypipes cdent As threatened ^ | 15:23 |
jaypipes | edleafe: I blame efried. | 15:23 |
jaypipes | let the blame game commence. | 15:23 |
*** hongbin has joined #openstack-nova | 15:23 | |
*** itlinux has joined #openstack-nova | 15:23 | |
efried | wait, what am I getting blamed for? | 15:23 |
mriedem | i know some intrepid core wants to +W this https://review.openstack.org/#/c/521947/ | 15:23 |
jaypipes | efried: nothing :) was just kiddin. | 15:23 |
efried | jaypipes I figured, but still want to know. | 15:23 |
jaypipes | efried: apparently I goofed a patchset of edleafe's a while ago. | 15:24 |
cdent | since 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 handlers | 15:24 |
cdent | sorry takashi. typing is hard | 15:24 |
jaypipes | mriedem: looking at the bug now.. | 15:24 |
mriedem | cdent: is it covered by gabbits? | 15:25 |
*** links has joined #openstack-nova | 15:25 | |
*** edand has quit IRC | 15:25 | |
mriedem | oh it's a format thing https://review.openstack.org/#/c/521640/7/nova/tests/functional/api/openstack/placement/gabbits/allocations.yaml | 15:25 |
mriedem | not utcnow vs updated_at | 15:26 |
jaypipes | cdent: 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 test | 15:26 |
mriedem | hmm | 15:27 |
cdent | mriedem: it is covered, but the not the specific conditional that is being pointed at | 15:27 |
mriedem | so the last-modified can be created_at, updated_at, the max in a collection, or utcnow | 15:27 |
mriedem | and that is per-handler | 15:27 |
cdent | there are tests for the utility that is used | 15:27 |
cdent | but not for its use in the handlers | 15:27 |
*** hemna_ has joined #openstack-nova | 15:27 | |
mriedem | yeah i see that | 15:27 |
cdent | (except in the fact that the gabbits test that a last-modified is returned) | 15:27 |
*** alexchadin has joined #openstack-nova | 15:28 | |
mriedem | and it's a datetime format | 15:28 |
mriedem | idk, asserting that the handler is doing the one right thing it should be doing would only be there for regression | 15:28 |
mriedem | so i can see value in it, but i'm not sure how much extra work that is | 15:28 |
cdent | it’s a fair chunk because of the way the handlers do their own serializing | 15:29 |
cdent | but not super duper onerous | 15:29 |
mriedem | so....maybe a follow up? | 15:30 |
mriedem | as a compromise | 15:30 |
cdent | fine with me, if thats the consensus | 15:31 |
mriedem | commented inline | 15:32 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add new style volume attachment support to block_device.py https://review.openstack.org/525787 | 15:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 15:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | 15:33 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047 | 15:33 |
mriedem | gibi: ^ fixed | 15:33 |
*** awaugama has quit IRC | 15:33 | |
gibi | mriedem: looking... | 15:33 |
*** yamamoto has quit IRC | 15:33 | |
mriedem | i looked at the functional py35 failure in https://review.openstack.org/#/c/330285/ | 15:33 |
mriedem | it was really hard to see from the actual output what the issue was, and i'm assuming it's intermittent, but i can dig again | 15:34 |
mriedem | the py27 functional didn't fail | 15:34 |
mriedem | http://logs.openstack.org/85/330285/175/check/openstack-tox-functional-py35/9a23bfd/testr_results.html.gz | 15:34 |
gibi | mriedem: could be some timing issue. py35 thend to uncover those | 15:34 |
*** nikhil has joined #openstack-nova | 15:35 | |
mriedem | AssertionError: The migration table left empty. | 15:35 |
*** yamahata has quit IRC | 15:35 | |
mriedem | hmm | 15:36 |
mriedem | 2017-12-07 10:11:01,711 WARNING [nova.virt.block_device] Detaching volume from unknown instance | 15:36 |
mriedem | TestInstanceNotificationSampleWithMultipleComputeOldAttachFlow would suggest this is a latent issue | 15:36 |
ildikov | I don't recall seeing this before | 15:37 |
mriedem | http://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=7d | 15:37 |
mriedem | gibi: there are other unrelated patches hitting the same thing | 15:37 |
mriedem | so it's intermittent and latent | 15:37 |
gibi | mriedem: OK, your logstash link is convincing | 15:38 |
mriedem | i'll open a bug | 15:38 |
gibi | mriedem: I will put back my +2 | 15:38 |
ildikov | mriedem: BTW, I didn't fully get that part of the commit message of the block_device patch that gibi asked about | 15: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 |
mriedem | https://bugs.launchpad.net/nova/+bug/1736976 | 15:41 |
openstack | Launchpad 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 IRC | 15:41 | |
mriedem | ildikov: about the boot from volume thing? | 15:41 |
ildikov | mriedem: I mean even if the volume is created by Nova having or not having an attachment_id has nothing to do with that | 15:41 |
*** awaugama has joined #openstack-nova | 15:41 | |
*** sahid has joined #openstack-nova | 15:42 | |
ildikov | mriedem: the multiattach flag will indeed not be set by Nova | 15:42 |
ildikov | mriedem: yeah, BFV | 15:42 |
mriedem | ildikov: doesn't have anything to do with multiattach | 15:42 |
stephenfin | jianghuaw_: +2. Thanks for the follow-up | 15:42 |
mriedem | ildikov: i added todos like this https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@589 | 15:42 |
mriedem | does that make it clear? | 15:42 |
*** yangyapeng has quit IRC | 15:42 | |
jianghuaw_ | stephenfin, thanks. | 15:42 |
*** sahid has quit IRC | 15:42 | |
ildikov | mriedem: no :) | 15:43 |
edleafe | mriedem: I got the "AssertionError: The migration table left empty." error in one of my series too | 15:43 |
mriedem | stephenfin: are you still ok with this https://review.openstack.org/#/c/507473/ ? | 15:43 |
*** yangyapeng has joined #openstack-nova | 15:43 | |
edleafe | mriedem: passed locally | 15:43 |
ildikov | mriedem: we reserve the volume now for BFV | 15:43 |
*** sahid has joined #openstack-nova | 15:43 | |
ildikov | mriedem: or do we not do that still for the case when Nova creates the volume? | 15:43 |
stephenfin | mriedem: Eek, seems like forever since I reviewed that. I'll take another look shortly | 15:44 |
mriedem | ildikov: that ^ | 15:44 |
mriedem | ildikov: 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-nova | 15:44 | |
mriedem | when nova creates the volume, we don't create an attachment (on the compute) | 15:44 |
mriedem | so we attach using the old flow | 15:44 |
mriedem | because self['attachment_id'] isn't set in the DriverBlockDevice | 15:44 |
mriedem | hence the todo i left in that code | 15:44 |
ildikov | mriedem: isn't that refresh used for live_migrate? | 15:45 |
stephenfin | jaypipes, efried: Am I correct in saying libvirt doesn't currently report traits to placement? | 15:45 |
mriedem | ildikov: refresh? | 15:45 |
jaypipes | stephenfin: correct. | 15:45 |
stephenfin | jaypipes: Phew. And that's something not targeted for Queens | 15: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 |
ildikov | mriedem: sorry, looked at the wrong line, but you added a note into that one as well | 15:46 |
ildikov | mriedem: which I didn't really get | 15:46 |
jaypipes | stephenfin: no, not targeted | 15:46 |
mriedem | ildikov: link to the line please | 15:46 |
stephenfin | Ideal. Thanks, jaypipes | 15:46 |
ildikov | mriedem: what you pasted above: https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@589 | 15:47 |
mriedem | ildikov: what don't you get about ^ ? | 15:47 |
mriedem | ildikov: in that code, nova is creating the volume | 15:47 |
mriedem | agree? | 15:47 |
ildikov | mriedem: that why we need to create a new attachment here too: refresh_connection_info | 15:47 |
*** yangyapeng has quit IRC | 15:48 | |
mriedem | ildikov: do you mean https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@473 ? | 15:48 |
jaypipes | mdbooth: you working on fixing those unit test failures on the BDM uuid patch? | 15:48 |
*** yamamoto has joined #openstack-nova | 15:48 | |
*** yamamoto has quit IRC | 15:48 | |
ildikov | yeah, 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 IRC | 15:49 | |
ildikov | mriedem: no, I meant line 589, I don't get why we need to add that one | 15:49 |
*** itlinux has joined #openstack-nova | 15:49 | |
mriedem | ildikov: if we create the volume but don't create the attachment, | 15:50 |
mriedem | when we call super(...).attach() | 15:50 |
mriedem | https://review.openstack.org/#/c/525787/4/nova/virt/block_device.py@511 will be True | 15:50 |
mriedem | and we will go down _legacy_volume_attach | 15:51 |
ildikov | sorry, my bad, mixed it up | 15:51 |
ildikov | forget that one | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the to_dict() method to the Selection object https://review.openstack.org/523492 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 15:51 |
openstackgerrit | Ed Leafe proposed openstack/nova master: Change compute RPC to use alternates for resize https://review.openstack.org/526436 | 15:51 |
edleafe | mriedem: jaypipes: ^^ rebased | 15:51 |
edleafe | mriedem: also added the alternates for resize patch | 15:51 |
mriedem | ildikov: so we're all good now? | 15:52 |
mriedem | edleafe: ok reapproved the bottom 4 | 15:52 |
ildikov | I 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 removal | 15:53 |
edleafe | mriedem: thx | 15:53 |
ildikov | my bad, this code is super confusing... :/ | 15:53 |
mriedem | ildikov: well, calling check_attach in the old flow on a volume that we *just* created was always kind of pointless | 15:53 |
mriedem | something would have had to slip in between the microseconds that the volume went to 'available' and we started using it | 15:54 |
ildikov | mriedem: 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 head | 15:54 |
mriedem | i think we only ever reserved from the api | 15:54 |
*** itlinux has quit IRC | 15:54 | |
mriedem | yeah https://github.com/openstack/nova/blob/stable/ocata/nova/virt/block_device.py#L250 | 15:55 |
ildikov | yeah, I wasn't sure where exactly we create that volume I guess | 15:55 |
mriedem | check_attach wouldn't reserve | 15:55 |
mriedem | yup, for blank/image/snapshot it's created on the compute | 15:55 |
mriedem | right before we attach it | 15:55 |
mriedem | we should create the volumes way earlier, like in conductor, but that's not happening today | 15:56 |
ildikov | ok, so I'm just partially stupid, that's something :) | 15:56 |
mriedem | this is the attach code in ocata https://github.com/openstack/nova/blob/stable/ocata/nova/compute/api.py#L3666 | 15:56 |
mriedem | https://github.com/openstack/nova/blob/stable/ocata/nova/compute/api.py#L3653 | 15:57 |
mriedem | so that's the only time (aside from bfv with an existing volume) that we'd reserve | 15:57 |
ildikov | ok | 15:57 |
mriedem | i believe i have a todo sitting here to document how boot from volume actually works | 15:58 |
mriedem | because i explained it once in the mailing list | 15:58 |
mdbooth | jaypipes: Yeah. Working on something further down the stack atm. | 15:58 |
mdbooth | jaypipes: Wait, are they at the beginning of the stack, too? | 15:59 |
*** links has quit IRC | 15:59 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add regression test for bug 1735407 https://review.openstack.org/526095 | 15:59 |
openstack | bug 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 |
ildikov | mriedem: if you have a link to that I would be happy to read it :) | 15:59 |
jaypipes | mdbooth: no, second patch in series | 15:59 |
*** itlinux has joined #openstack-nova | 15:59 | |
mdbooth | Hmm, that was careless. | 15:59 |
mdbooth | jaypipes: fixing it now. | 15:59 |
* mdbooth knew about some other ones, but they were way after that. | 16:00 | |
ildikov | mriedem: we can have a quick meeting and see what's next | 16:00 |
mdbooth | Ah, fixed that locally, didn't realise I'd pushed the old one, sorry. | 16:01 |
*** itlinux has quit IRC | 16:04 | |
*** itlinux has joined #openstack-nova | 16:06 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | 16:07 |
maciejjozefczyk | Hello 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 IRC | 16:09 | |
*** nore_rabel has quit IRC | 16:13 | |
*** coreywright has quit IRC | 16:13 | |
*** tasker has joined #openstack-nova | 16:15 | |
tasker | morning! 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 IRC | 16:18 | |
*** slaweq has quit IRC | 16:18 | |
*** sahid has quit IRC | 16:19 | |
*** slaweq has joined #openstack-nova | 16:23 | |
mriedem | sdague: efried: i'm surprised our docs don't say anything about configuring [neutron] with a user that has the admin role | 16:23 |
mriedem | because we definitely require that | 16:24 |
mriedem | for doing thigns with the port binding profile | 16:24 |
mriedem | maybe the install guide does, but the config reference doesn't | 16:24 |
*** Apoorva has joined #openstack-nova | 16:24 | |
*** AlexeyAbashkin has joined #openstack-nova | 16:24 | |
*** READ10 has joined #openstack-nova | 16:24 | |
*** coreywright has joined #openstack-nova | 16:26 | |
mriedem | hmm, not here https://docs.openstack.org/nova/latest/install/compute-install-ubuntu.html | 16:26 |
efried | mriedem This coming from https://review.openstack.org/#/c/509892/ ? | 16:27 |
*** slaweq has quit IRC | 16:28 | |
mriedem | no | 16:28 |
mriedem | https://bugs.launchpad.net/cinder/+bug/1736773 | 16:28 |
openstack | Launchpad 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 |
mriedem | https://docs.openstack.org/neutron/latest/install/compute-install-ubuntu.html#configure-the-compute-service-to-use-the-networking-service | 16:31 |
mriedem | the networking install guide mentions it, but not sure if that 'neutron' user is an admin | 16:31 |
*** AlexeyAbashkin has quit IRC | 16:31 | |
mriedem | ah here it is https://docs.openstack.org/neutron/latest/install/controller-install-ubuntu.html#prerequisites | 16:33 |
mriedem | "Add the admin role to the neutron user:" | 16:33 |
mriedem | so if you've done everything correctly, things might work | 16:33 |
mriedem | but 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 something | 16:33 |
*** dmsimard is now known as dmsimard|afk | 16:34 | |
mriedem | assuming that's fine because the nova config section for [neutron] doesn't mention it | 16:34 |
*** ragiman has quit IRC | 16:36 | |
mriedem | https://docs.openstack.org/cinder/latest/install/cinder-controller-install-ubuntu.html#configure-compute-to-use-block-storage | 16:36 |
openstackgerrit | Merged openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | 16:38 |
*** amodi has quit IRC | 16:38 | |
efried | wohoo! ^ | 16:42 |
cdent | amazing | 16:42 |
* cdent enjoys the sublime pain of needing to rebase two different new microversions | 16:43 | |
efried | What, you haven't written a script for that yet? | 16:43 |
*** yangyapeng has joined #openstack-nova | 16:43 | |
cdent | that would ruin the enjoyment | 16:44 |
*** isq_ has joined #openstack-nova | 16:45 | |
gibi | getting n-rp REST merged... it is almost like Christmas | 16:45 |
*** slaweq has joined #openstack-nova | 16:45 | |
*** yamahata__ has quit IRC | 16:46 | |
mgagne | so 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 |
mriedem | jaypipes: done https://review.openstack.org/#/c/465160/9 | 16:48 |
*** yangyapeng has quit IRC | 16:48 | |
*** yamamoto has joined #openstack-nova | 16:49 | |
*** slaweq has quit IRC | 16:49 | |
jaypipes | mriedem: cheers | 16:50 |
*** itlinux has quit IRC | 16:50 | |
*** moshele has joined #openstack-nova | 16:51 | |
*** damien_r has quit IRC | 16:51 | |
openstackgerrit | Merged openstack/nova master: libvirt: Re-initialise volumes, encryptors, and vifs on hard reboot https://review.openstack.org/400384 | 16:52 |
*** itlinux has joined #openstack-nova | 16:53 | |
*** chyka has joined #openstack-nova | 16:53 | |
*** salv-orlando has quit IRC | 16:53 | |
mriedem | ildikov: http://lists.openstack.org/pipermail/openstack-dev/2017-April/115093.html was the ML thread i was thinking of | 16:54 |
*** fragatina has joined #openstack-nova | 16:54 | |
*** yamamoto has quit IRC | 16:55 | |
bauzas | holy fsck, the libvirt driver doesn't know a shit about the compute node | 16:55 |
ildikov | mriedem: cool, thanks | 16:55 |
bauzas | like, it's not knowing the compute UUID | 16:55 |
mriedem | bauzas: why should it? | 16:55 |
*** yangyapeng has joined #openstack-nova | 16:56 | |
bauzas | mriedem: since we're now passing the allocations, there will be a list of RP UUIDs with resources in them | 16:56 |
bauzas | mriedem: for the moment, it's not a problem | 16:56 |
*** fragatina has quit IRC | 16:56 | |
bauzas | mriedem: given we don't support nested RPs | 16:56 |
bauzas | mriedem: but once we'll have more than one RP for a compute, then we'll pass all the related allocations to the drivers | 16:57 |
*** fragatina has joined #openstack-nova | 16:57 | |
bauzas | then, how to know which one is for the root RP for example ? | 16:57 |
mriedem | sure, but the fact we don't have something that we've never had a need for before shouldn't be shocking | 16:57 |
mriedem | "wait, nova DOESN'T create ice cream?!" | 16:57 |
*** Alex_Staf has joined #openstack-nova | 16:57 | |
*** yamahata has joined #openstack-nova | 16:57 | |
bauzas | mriedem: sure, I understand that, but I thought we would at least know more than just the hostname :) | 16:58 |
bauzas | anyway | 16:58 |
bauzas | it's not a problem for me, but meh | 16:58 |
mriedem | i'm assuming we'd have to pass the cn uuid down through spawn() yes? | 16:58 |
*** james_li has joined #openstack-nova | 16:59 | |
mriedem | or 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 directly | 16:59 |
mriedem | ala the resource tracker | 16:59 |
bauzas | yeah | 16:59 |
stephenfin | mriedem: 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 |
mriedem | stephenfin: we are, alwaysh have been | 17:00 |
mriedem | the InstanceAction object extends the NovaPersistentObject class | 17:00 |
mriedem | which has the timstamp fields | 17:00 |
mriedem | *timestamp | 17:00 |
*** yangyapeng has quit IRC | 17:00 | |
stephenfin | Damn - I skimmed down to NovaTimestampObject and went no further | 17:01 |
stephenfin | Fairs. That's done now | 17:01 |
mriedem | thanks | 17:02 |
*** Apoorva has quit IRC | 17:02 | |
*** felipemonteiro_ has joined #openstack-nova | 17:06 | |
*** yamahata has quit IRC | 17:06 | |
stephenfin | mriedem: 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.html | 17:07 |
*** harlowja has joined #openstack-nova | 17:08 | |
efried | mriedem 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 |
efried | bauzas ^ | 17:09 |
*** lyan has quit IRC | 17:09 | |
efried | mriedem 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 |
mriedem | stephenfin: i haven't done it yet | 17:10 |
*** karthiks has quit IRC | 17:11 | |
bauzas | efried: ok, gtk | 17:11 |
bauzas | efried: so the driver will know the tree, all good with me | 17:11 |
efried | bauzas 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 |
bauzas | k | 17:12 |
efried | bauzas 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 |
bauzas | yeah I remember now that convo | 17:14 |
*** Swami has quit IRC | 17:15 | |
*** salv-orlando has joined #openstack-nova | 17:15 | |
*** dtantsur is now known as dtantsur|afk | 17:15 | |
*** sshwarts has quit IRC | 17:15 | |
efried | jaypipes cdent Here's a wrinkle: Can sharing RPs be trees? | 17:18 |
efried | Hint: it'd be a lot cooler if they couldn't. | 17:19 |
*** andreas_s has quit IRC | 17:19 | |
efried | I mean, it's clear that nothing is preventing you from doing that today. | 17:19 |
efried | I'm saying we should consider whether that's a good idea. | 17:19 |
*** andreas_s has joined #openstack-nova | 17:19 | |
*** james_li has quit IRC | 17:21 | |
*** slaweq has joined #openstack-nova | 17:21 | |
*** itlinux has quit IRC | 17:23 | |
mriedem | cdent: do you think https://review.openstack.org/#/c/518223/ is something we need to backport to pike? | 17:23 |
* cdent looking | 17:23 | |
*** salv-orlando has quit IRC | 17:24 | |
cdent | mriedem: is a nice to have but not a need to have | 17:24 |
*** moshele has quit IRC | 17:24 | |
cdent | efried: 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 meaningful | 17:25 |
efried | cdent Okay, yeah, I think I'm discovering that. | 17:25 |
*** slaweq has quit IRC | 17:25 | |
efried | cdent 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 |
cdent | what tree-ness would mean in that context is a path to get to | 17:26 |
efried | I 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 |
mriedem | cdent: ok | 17:27 |
efried | So 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 |
efried | The answer I'm coming to is: No, I just pretend he's a root. | 17:27 |
cdent | yes | 17:27 |
*** nore_rabel has joined #openstack-nova | 17:27 | |
cdent | at 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 |
efried | So 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 |
efried | cdent I agree with that in principle, but I have to code *something* here. | 17:28 |
*** andreas_s has quit IRC | 17:28 | |
cdent | efried: 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-nova | 17:29 | |
*** andreas_s has quit IRC | 17:29 | |
*** andreas_s has joined #openstack-nova | 17:29 | |
efried | cdent One of them, which I updated - see if it satisfies: https://review.openstack.org/#/c/521097/ | 17:30 |
cdent | efried: is there a spec or something where the reasons for the reason you gave (ComputeDriver.update_provider_tree) are explained? | 17:31 |
efried | jaypipes ^? | 17:31 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: [placement] Fix an error message in API validation https://review.openstack.org/526465 | 17:32 |
cdent | efried: 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 picture | 17:33 |
efried | cdent I agree it would be neat to have a spec that calls out ComputeDriver.update_provider_tree. | 17:33 |
efried | cdent Does the WIP patch that introduces that method help at all? https://review.openstack.org/#/c/521187/8/nova/virt/driver.py | 17:34 |
*** andreas_s has quit IRC | 17:34 | |
efried | not much, I guess. | 17:34 |
cdent | The “supersedes get_inventory()” is probably the most meaningful bit | 17:36 |
cdent | and 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-nova | 17:37 | |
efried | cdent Yeah, I think PowerVM will get to skip get_inventory() | 17:37 |
efried | but it'll be the exception. | 17:37 |
efried | Meh, actually I don't know. | 17:38 |
efried | We 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-nova | 17:38 | |
*** Apoorva has joined #openstack-nova | 17:39 | |
*** brault_ has quit IRC | 17:39 | |
jaypipes | efried: haven't thought about that, frankly | 17:39 |
efried | jaypipes I think cdent and I talked ourselves into relative sanity on the subject. | 17:40 |
*** brault has joined #openstack-nova | 17:40 | |
mriedem | melwitt: https://review.openstack.org/#/c/526182/ and the one above it have +2s from gibi | 17:41 |
mriedem | for the new volume attach stuff | 17:41 |
*** salv-orlando has joined #openstack-nova | 17:42 | |
*** Alex_Staf has quit IRC | 17:44 | |
*** brault has quit IRC | 17:45 | |
*** jpena is now known as jpena|off | 17:48 | |
*** penick has joined #openstack-nova | 17:50 | |
*** moshele has joined #openstack-nova | 17:51 | |
*** yamamoto has joined #openstack-nova | 17:51 | |
*** lucasagomes is now known as lucas-afk | 17:52 | |
*** salv-orlando has quit IRC | 17:53 | |
*** moshele has quit IRC | 17:53 | |
*** salv-orlando has joined #openstack-nova | 17:53 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix NoneType error when [service_user] is misconfigured https://review.openstack.org/526468 | 17:53 |
*** derekh has quit IRC | 17:54 | |
*** penick has quit IRC | 17:54 | |
*** yamamoto has quit IRC | 17:56 | |
*** penick has joined #openstack-nova | 17:57 | |
*** salv-orlando has quit IRC | 17:57 | |
*** penick has quit IRC | 17:59 | |
*** slaweq has joined #openstack-nova | 17:59 | |
*** salv-orlando has joined #openstack-nova | 18:00 | |
*** penick has joined #openstack-nova | 18:02 | |
*** damien_r has joined #openstack-nova | 18:04 | |
*** salv-orlando has quit IRC | 18:04 | |
*** slaweq has quit IRC | 18:04 | |
*** salv-orlando has joined #openstack-nova | 18:04 | |
*** yangyapeng has joined #openstack-nova | 18:05 | |
*** penick has quit IRC | 18:05 | |
*** andreas_s has joined #openstack-nova | 18:05 | |
*** mdnadeem has quit IRC | 18:08 | |
*** salv-orlando has quit IRC | 18:09 | |
*** sambetts is now known as sambetts|afk | 18:09 | |
*** andreas_s has quit IRC | 18:09 | |
*** yangyapeng has quit IRC | 18:09 | |
*** salv-orlando has joined #openstack-nova | 18:10 | |
*** mvk has quit IRC | 18:11 | |
*** tesseract has quit IRC | 18:13 | |
*** salv-orlando has quit IRC | 18:16 | |
*** salv-orlando has joined #openstack-nova | 18:16 | |
*** salv-orlando has quit IRC | 18:19 | |
*** nore_rabel has quit IRC | 18:20 | |
*** salv-orlando has joined #openstack-nova | 18:20 | |
*** felipemonteiro_ has quit IRC | 18:22 | |
*** vivsoni__ has joined #openstack-nova | 18:24 | |
*** vivsoni_ has quit IRC | 18:24 | |
*** salv-orlando has quit IRC | 18:24 | |
*** esberglu has quit IRC | 18:24 | |
*** esberglu has joined #openstack-nova | 18:25 | |
*** esberglu_ has joined #openstack-nova | 18:29 | |
*** esberglu has quit IRC | 18:30 | |
*** nikhil has quit IRC | 18:34 | |
*** ragiman has joined #openstack-nova | 18:34 | |
*** slaweq has joined #openstack-nova | 18:34 | |
*** damien_r has quit IRC | 18:35 | |
*** penick has joined #openstack-nova | 18:36 | |
*** penick has quit IRC | 18:37 | |
*** fragatina has quit IRC | 18:38 | |
*** nore_rabel has joined #openstack-nova | 18:38 | |
*** andreas_s has joined #openstack-nova | 18:39 | |
*** slaweq has quit IRC | 18:40 | |
*** moshele has joined #openstack-nova | 18:40 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Enable limiting GET /allocation_candidates https://review.openstack.org/513526 | 18:43 |
cdent | one down, one to go | 18:43 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Only log not correcting allocation once per period https://review.openstack.org/526477 | 18:43 |
*** andreas_s has quit IRC | 18:44 | |
*** artom has joined #openstack-nova | 18:48 | |
*** yamamoto has joined #openstack-nova | 18:52 | |
*** itlinux has joined #openstack-nova | 18:53 | |
*** yamamoto has quit IRC | 18:57 | |
efried | jaypipes 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 |
dansmith | is there something we need to be catching that we can't without raising there? | 18:59 |
cdent | my preference would be for many: makes any booms a bit more obvious | 18:59 |
dansmith | but, I'm sure multiple patches per call would be way easier | 18:59 |
dansmith | also what cdent said | 18:59 |
*** edmondsw has quit IRC | 19:00 | |
*** phuongnh has joined #openstack-nova | 19:00 | |
edleafe | efried: yeah, smaller chunks are better | 19:01 |
efried | dansmith Well, we caught a grenade bug (it went "boom") with the last one. | 19:02 |
efried | dansmith 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 |
dansmith | well, in some cases that's intentional, where we get allocations and if none, then do a thing | 19:04 |
dansmith | just saying all those places will need to be changed. I'll reserve judgment until I see them I guess | 19:04 |
*** phuongnh has quit IRC | 19:04 | |
*** nore_rabel has quit IRC | 19:06 | |
efried | dansmith 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-nova | 19:06 | |
*** moshele has quit IRC | 19:07 | |
*** baoli has joined #openstack-nova | 19:07 | |
* dansmith nods | 19:09 | |
dansmith | I just hadn't thought much about it so seeing them all will help | 19:09 |
*** felipemonteiro_ has joined #openstack-nova | 19:10 | |
*** adisky_ has quit IRC | 19:10 | |
efried | dansmith With that discussion, it's actually going to be pretty limited. So thanks for that. | 19:11 |
*** felipemonteiro__ has joined #openstack-nova | 19:11 | |
*** penick has joined #openstack-nova | 19:11 | |
*** slaweq has joined #openstack-nova | 19:12 | |
*** itlinux has quit IRC | 19:14 | |
*** felipemonteiro_ has quit IRC | 19:15 | |
*** slaweq has quit IRC | 19:17 | |
*** itlinux has joined #openstack-nova | 19:18 | |
*** andreas_s has joined #openstack-nova | 19:20 | |
*** harlowja has quit IRC | 19:21 | |
*** andreas_s has quit IRC | 19:25 | |
*** tasker has quit IRC | 19:26 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers https://review.openstack.org/521639 | 19:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640 | 19:29 |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add info about last-modified to contrib docs https://review.openstack.org/526084 | 19:29 |
*** esberglu_ is now known as esberglu | 19:33 | |
*** edmondsw has joined #openstack-nova | 19:35 | |
jaypipes | dansmith, 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 is | 19:39 | |
mriedem | it's placement | 19:39 |
mriedem | i'll tell you now | 19:39 |
*** edmondsw has quit IRC | 19:39 | |
jaypipes | :( | 19:40 |
mriedem | jaypipes: i'll trade you that for https://review.openstack.org/#/c/525787/ | 19:40 |
dansmith | I dunno what jaypipes' is, but it's not an even trade | 19:40 |
jaypipes | mriedem: sure, deal. | 19:40 |
*** edmondsw has joined #openstack-nova | 19:41 | |
mriedem | dansmith: is that a slight against mine or his? | 19:41 |
dansmith | mriedem: yours :) | 19:41 |
mriedem | ha | 19:42 |
mriedem | everyone loves some good block device'n | 19:42 |
efried | dansmith Yeah, don't take the trade. jaypipes' is easy. | 19:42 |
dansmith | I clicked on mriedem's link to see if it was one of my reviews he just just finished and was looking for a +W | 19:42 |
dansmith | but alas, no | 19:42 |
jaypipes | efried: not *just* BDM. BFV + BDM, otherwise known as BDSM. | 19:43 |
*** andreas_s has joined #openstack-nova | 19:43 | |
dansmith | haha | 19:43 |
mriedem | dansmith: you can get this easy refactor to complete the set https://review.openstack.org/#/c/526182/ | 19:43 |
*** slaweq has joined #openstack-nova | 19:43 | |
dansmith | mriedem: I'll trade you for revisiting my live migration allocations one | 19:43 |
dansmith | that will even out your karma for trading jay something easy for something hard | 19:44 |
mriedem | fwiw, https://review.openstack.org/#/c/385693/ has a problem in the commit message | 19:44 |
*** edmondsw has quit IRC | 19:45 | |
efried | mriedem Will y'all fast-approve if I make that edit? Hate to lose gibi's +2 | 19:46 |
mriedem | i'm still reviewing | 19:46 |
mriedem | there are other....concerns | 19:46 |
efried | ight | 19:46 |
mriedem | the ... is for intended dramatic effect | 19:46 |
efried | jaypipes FYI I have this series locally, with lots of deltas, so *please* don't re-publish. | 19:46 |
jaypipes | efried: roger | 19:46 |
*** andreas_s has quit IRC | 19:47 | |
*** slaweq has quit IRC | 19:47 | |
mriedem | efried: jaypipes: replied https://review.openstack.org/#/c/385693/ | 19:50 |
mriedem | pick your poison | 19:50 |
mriedem | it's probably premature to say in a release note what our minimum required version of placement is at this point | 19:51 |
efried | mriedem Nice. But FYI, I believe that ship already sailed. | 19:51 |
mriedem | since that's likely to change | 19:51 |
efried | That's what we discovered with that grenade bug. | 19:51 |
mriedem | so we currently say we require 1.10 | 19:51 |
mriedem | what do we actually require? | 19:51 |
mriedem | or were we just using 1.10? | 19:52 |
efried | Yuh, that's a lie at this point. I believe it's 1.14. | 19:52 |
mriedem | yes, ^ requires 1.14 | 19:52 |
mriedem | my point is, | 19:52 |
mriedem | was anything in nova before this change using something higher than 1.10? | 19:52 |
mriedem | because if not, grenade was doing it's job | 19:52 |
efried | Oh, I thought it was the patch before that one. Never mind, you're right. | 19:52 |
*** edmondsw has joined #openstack-nova | 19:52 | |
efried | Does that mean cdent's change was incorrect? | 19:52 |
mriedem | we 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, period | 19:53 |
mriedem | not necessarily, | 19:53 |
mriedem | we'll need grenade upgrading things for us to have sane CI | 19:53 |
mriedem | e.g. queens nova doesn't test against pike cinder | 19:53 |
*** yamamoto has joined #openstack-nova | 19:53 | |
efried | mriedem 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 |
mriedem | i think my point is just we aren't doing a good job about being careful with versions | 19:54 |
mriedem | unlike we do with other external services | 19:54 |
mriedem | regarding a release note, i said i think that's premature right now | 19:54 |
mriedem | since it's likely to bump again before we release queens | 19:55 |
mriedem | so https://github.com/openstack/nova/blob/master/nova/cmd/status.py#L202 needs to change in a follow up | 19:55 |
*** mikal_ has joined #openstack-nova | 19:55 | |
mriedem | and we should start working on Queens notes for https://docs.openstack.org/nova/latest/user/placement.html#upgrade-notes, in a follow up | 19:55 |
mriedem | agreed? | 19:55 |
jaypipes | works for me. | 19:55 |
cdent | that sounds right | 19:55 |
*** moshele has joined #openstack-nova | 19:55 | |
mriedem | ok, +W the | 19:56 |
mriedem | *then | 19:56 |
cdent | i agree we’ve not been careful about versions, but I’m not sure it is a huge problem in reality | 19:56 |
melwitt | mriedem: FYI I added some test coverage here and stacked the remove old quotas code follow up on top https://review.openstack.org/#/c/526270 | 19:56 |
*** edmondsw has quit IRC | 19:56 | |
*** mikal has quit IRC | 19:57 | |
*** yamamoto has quit IRC | 19:58 | |
*** edmondsw has joined #openstack-nova | 19:58 | |
*** edmondsw has quit IRC | 19:58 | |
*** edmondsw has joined #openstack-nova | 19:58 | |
*** brault has joined #openstack-nova | 19:58 | |
*** mikal has joined #openstack-nova | 20:00 | |
mriedem | melwitt: comments inline | 20:00 |
*** andreas_s has joined #openstack-nova | 20:01 | |
*** andreas_s has quit IRC | 20:01 | |
*** andreas_s has joined #openstack-nova | 20:01 | |
*** mikal_ has quit IRC | 20:01 | |
jaypipes | mriedem: reviewed https://review.openstack.org/#/c/525787/. +Wd | 20:02 |
*** edmondsw has quit IRC | 20:02 | |
melwitt | mriedem: I didn't have access to self.flags in fixtures.py, I assumed it's on the base TestCase class. but let me check | 20:03 |
mriedem | melwitt: you likely have to pass the test's self into the fixture | 20:04 |
mriedem | like we do in some other fixtures | 20:04 |
mriedem | i care less about the self.flags thing | 20:04 |
mriedem | and more about doing the cleanup after the thing you change, and removing the unused CONF in the sample test | 20:04 |
melwitt | oh, got it. I can do that then | 20:04 |
mriedem | jaypipes: thanks, replied about the setup thing | 20:05 |
melwitt | k | 20:05 |
*** andreas_s has quit IRC | 20:05 | |
*** BryanS68 has quit IRC | 20:07 | |
*** edmondsw has joined #openstack-nova | 20:07 | |
mriedem | jaypipes: btw, i haven't dealt with that much mox in a long time... | 20:08 |
mriedem | the strictness with which mox makes sure you hit bdm.save() cost me about 2 hours | 20:08 |
jaypipes | mriedem: I know, right? :) | 20:08 |
* cdent expires | 20:09 | |
cdent | good night | 20:09 |
*** cdent has quit IRC | 20:09 | |
*** edmondsw has quit IRC | 20:12 | |
mriedem | jaypipes: efried: want me to do the doc and nova-status stuff from https://review.openstack.org/#/c/385693/ ? | 20:12 |
efried | mriedem I would welcome that. Were you planning to do it in isolation or add it somewhere in this series? | 20:13 |
mriedem | isolation | 20:13 |
mriedem | i'm not touching that mess | 20:13 |
efried | Perfect | 20:13 |
efried | Yeah, you shouldn't. | 20:13 |
*** slaweq has joined #openstack-nova | 20:13 | |
*** moshele has quit IRC | 20:16 | |
melwitt | I don't know why but using self.flags causes the concurrent test fail for the NoopQuotaDriver | 20:19 |
mriedem | huh, maybe not global enough? | 20:19 |
*** slaweq_ has joined #openstack-nova | 20:20 | |
*** slaweq has quit IRC | 20:21 | |
*** james_li_ has joined #openstack-nova | 20:22 | |
melwitt | oh because self.flags doesn't clear the conf override. weird, I thought it would have to so it works with multiple tests | 20:23 |
*** melwitt has left #openstack-nova | 20:24 | |
*** melwitt has joined #openstack-nova | 20:24 | |
efried | melwitt I have a bug for that. | 20:24 |
efried | Stand 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 |
mriedem | james_li_: depends on the policy configuration, but not by default | 20:25 |
melwitt | gah, what other channel did I accidentally leave by pressing ctrl-w in the wrong window | 20:25 |
efried | melwitt Sorry, different bug, probably not related: https://bugs.launchpad.net/oslo.config/+bug/1709728 | 20:25 |
openstack | Launchpad bug 1709728 in oslo.config "CONF.set_override doesn't alias deprecated opts" [Undecided,Confirmed] | 20:25 |
*** gszasz has quit IRC | 20:26 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094 | 20:26 |
james_li_ | mriedem: cool so its possible if policy enables, e.g. admin user? | 20:26 |
mriedem | james_li_: the default rule is admin_or_owner, | 20:27 |
mriedem | if that were changed to @ | 20:27 |
mriedem | then anything goes | 20:27 |
mriedem | https://docs.openstack.org/nova/latest/configuration/sample-policy.html | 20:27 |
openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094 | 20: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 |
mriedem | james_li_: shouldn't require code changes | 20:28 |
mriedem | but policy is a funny thing so you'd have to test it | 20:28 |
james_li_ | :) | 20:28 |
*** salv-orlando has joined #openstack-nova | 20:31 | |
melwitt | api samples tests run super ugly under py3 | 20:31 |
melwitt | lots of warnings emitted to the screen | 20:32 |
*** baoli has quit IRC | 20:32 | |
melwitt | and clearing the override didn't seem to help. gdi | 20:32 |
efried | jaypipes 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 |
jaypipes | efried: nope. | 20:34 |
mriedem | melwitt: just use set_override, it's fine | 20:34 |
efried | jaypipes So theoretically I could create RP traits first, then create the RP they're associated with? | 20:34 |
mriedem | and yes on the py35 warnings - i was seeing that in the functional tests earlier today, oslo.contet and oslo.policy warnings | 20:34 |
efried | Or create an agg with orphan RP UUIDs, then create the RPs associated with those UUIDs? | 20:34 |
mriedem | we should consider ignoring those | 20:35 |
jaypipes | efried: yes, manually executing SQL statements. of course, the object layer won't allow you to do that, though. | 20:35 |
jaypipes | efried: since a TraitNotFound or ResourceProviderNotFound would be raised when attempting to do that via ResourceProvider.set_traits() | 20:36 |
*** mchiappero has joined #openstack-nova | 20:36 | |
*** sean-k-m1 has joined #openstack-nova | 20:36 | |
*** sean-k-m2 has joined #openstack-nova | 20:36 | |
jaypipes | efried: because there's a lookup of trait ID to names supplied in set_traits() | 20:36 |
efried | jaypipes oh, okay, phew. So the answer from the perspective of a REST API consumer is that we're strict about that stuff. | 20:36 |
jaypipes | efried: yes. | 20:36 |
efried | Good. | 20:36 |
jaypipes | efried: except for aggregates, which are just UUIDs and we have nothing to "check" against. | 20:36 |
efried | jaypipes Right, but you can't *associate* a random anonymous UUID with an aggregate? | 20:37 |
*** BryanS68 has joined #openstack-nova | 20:37 | |
jaypipes | efried: no. but you *can* associated a random UUID to a known resource provider. | 20:37 |
jaypipes | efried: it's the aggregate UUID we have no way of checking. | 20:37 |
*** mchiappe1o has quit IRC | 20:37 | |
*** sean-k-mooney has quit IRC | 20:37 | |
*** sean-k-mooney2 has quit IRC | 20:37 | |
efried | jaypipes Got it, cool, thanks. | 20:37 |
jaypipes | pas de probleme | 20:37 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update nova-status and docs for nova-compute requiring placement 1.14 https://review.openstack.org/526505 | 20:38 |
efried | jaypipes 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_map | 20:38 |
mriedem | efried: jaypipes: ^ | 20:38 |
jaypipes | efried: ack. | 20:38 |
jaypipes | mriedem: thank you sir. | 20:39 |
efried | mriedem Thanks | 20:39 |
*** salv-orlando has quit IRC | 20:39 | |
*** salv-orlando has joined #openstack-nova | 20:39 | |
*** itlinux has quit IRC | 20:40 | |
mriedem | imacdonn: 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 now | 20:40 |
mriedem | anthonyper is your closest bet | 20:40 |
imacdonn | yeah, 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 fail | 20:42 |
imacdonn | via guru meditation, I can see that the resource_tracker thread is stuck trying to call libvirt's getLibVersion() | 20:42 |
mriedem | which is then a call to the hypervisor | 20:42 |
mriedem | so you're likely deadlocking on something in the hypervisor | 20:42 |
imacdonn | that call is made through eventlet's thread pooling proxy thingy, which is now holding a lock | 20:42 |
mriedem | yeah that could also be screwing you | 20:43 |
mriedem | i'd enable debug logging for libvirt and see if something shows up in there | 20:43 |
mriedem | or, check to see if things changed around that code since icehouse and see if you need to backport a fix | 20:43 |
imacdonn | problem 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 wait | 20:43 |
imacdonn | I've looked around, but not found anything that looks like an obvious related fix | 20:44 |
*** salv-orlando has quit IRC | 20:44 | |
*** moshele has joined #openstack-nova | 20:44 | |
imacdonn | I can't tell for sure if it's libvirt hanging on the call, or eventlet getting hung up somehow and not even trying the call | 20:45 |
*** pchavva has quit IRC | 20:45 | |
*** slaweq has joined #openstack-nova | 20:45 | |
*** tssurya_ has joined #openstack-nova | 20:47 | |
*** salv-orlando has joined #openstack-nova | 20:48 | |
mriedem | imacdonn: well, i see this in kilo https://review.openstack.org/#/c/104930/ | 20:48 |
mriedem | https://review.openstack.org/#/c/104930/12/nova/virt/libvirt/host.py@192 | 20:49 |
mriedem | so, | 20:49 |
imacdonn | yeah, that made it "fun" to try to compare bits of of the code to see what might have changed | 20:49 |
mriedem | keep in mind that anything running in those threads that logs anything could lock you up | 20:49 |
imacdonn | yes, saw some stuff about that | 20:49 |
mriedem | so if you have a GMR when things are locked, i'd look for any libvirt driver/host methods in the thread dump, | 20:49 |
mriedem | and see if those do loging | 20:49 |
mriedem | *logging | 20:49 |
imacdonn | GMR is at https://pastebin.com/1jfgdurJ | 20:50 |
mriedem | because i'm sure we don't do a good job of auditing stuff like that | 20:50 |
imacdonn | getLibVersion() call at line 659 | 20:50 |
*** slaweq has quit IRC | 20:50 | |
imacdonn | victim thread waiting to acquire() lock around line 777 | 20:51 |
*** tssurya_ has quit IRC | 20:53 | |
*** tssurya_ has joined #openstack-nova | 20:53 | |
*** salv-orlando has quit IRC | 20:53 | |
imacdonn | seems 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-nova | 20:56 | |
mriedem | well, it's making a connection to libvirt i believe | 20:56 |
mriedem | so it's not like the libvirt-python package version or something | 20:56 |
mriedem | nova meeting in 3 minutes | 20:57 |
imacdonn | it'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 |
mriedem | true yeah | 20:58 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (usage) https://review.openstack.org/520603 | 20:59 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (inventory) https://review.openstack.org/520613 | 20:59 |
*** itlinux has joined #openstack-nova | 21:00 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (resource_class) https://review.openstack.org/520611 | 21:00 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (aggregate) https://review.openstack.org/520608 | 21:00 |
mriedem | melwitt: might want to check out v | 21:00 |
mriedem | https://bugs.launchpad.net/nova/+bug/1737011 | 21:00 |
openstack | Launchpad bug 1737011 in OpenStack Compute (nova) "ServerActionsTestJSON.test_reboot_server_hard failed to ssh into instance" [Undecided,New] | 21:00 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Separate API schemas (trait) https://review.openstack.org/520605 | 21:00 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | 21:00 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | 21:00 |
*** pchavva has joined #openstack-nova | 21:01 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606 | 21:01 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506 | 21:01 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094 | 21:02 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400 | 21:03 |
openstackgerrit | melanie witt proposed openstack/nova master: Add API and nova-manage tests that use the NoopQuotaDriver https://review.openstack.org/526270 | 21:03 |
openstackgerrit | melanie witt proposed openstack/nova master: Follow up on removing old-style quotas code https://review.openstack.org/524234 | 21:03 |
*** slaweq_ has quit IRC | 21:04 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 21:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 21:14 |
*** hemna_ has quit IRC | 21:15 | |
melwitt | mriedem: is that different than our usual sporadic ssh timeouts? | 21:16 |
*** linkmark has quit IRC | 21:18 | |
*** pchavva has quit IRC | 21:19 | |
mriedem | melwitt: i don't think ssh timeouts are all that sporadic anymore | 21:19 |
mriedem | andreaf: ^? | 21:19 |
*** gyee has quit IRC | 21:20 | |
melwitt | okay, 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 compare | 21:20 |
mriedem | i don't see cloud-init run at all | 21:21 |
*** moshele has quit IRC | 21:23 | |
mriedem | force_config_drive = True | 21:24 |
mriedem | so we force a config drive to inject keys and such | 21:24 |
*** slaweq has joined #openstack-nova | 21:24 | |
mriedem | we don't even get to the point of rebooting the serer | 21:27 |
mriedem | *server | 21:27 |
mriedem | it's trying to ssh into the guest to check uptime before that | 21:27 |
mriedem | and that's what fails | 21:27 |
mriedem | http://logs.openstack.org/83/526183/1/gate/legacy-tempest-dsvm-py35/166f0c9/job-output.txt.gz#_2017-12-07_13_37_58_430147 | 21:27 |
*** tssurya_ has quit IRC | 21:27 | |
melwitt | good eye | 21:27 |
*** tssurya_ has joined #openstack-nova | 21:27 | |
mriedem | the console output is all from tempest trying to gather information before the test pukes | 21:28 |
jaypipes | dansmith: 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 |
mriedem | jaypipes: i think you'd have to | 21:28 |
*** sambetts|afk has quit IRC | 21:29 | |
mriedem | the host mapping is the compute_node.host, not compute_nodes.hypervisor_hostname which is the node name | 21:29 |
*** slaweq has quit IRC | 21:29 | |
jaypipes | mriedem: right | 21:29 |
*** dmsimard|afk is now known as dmsimard | 21:30 | |
*** lyan has joined #openstack-nova | 21:30 | |
*** slaweq has joined #openstack-nova | 21:31 | |
*** tidwellr has quit IRC | 21:31 | |
*** penick has quit IRC | 21:31 | |
*** sambetts_ has joined #openstack-nova | 21:32 | |
melwitt | mriedem: 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_827380 | 21:33 |
mriedem | melwitt: http://logs.openstack.org/22/518022/8/check/legacy-tempest-dsvm-neutron-full/81721fe/job-output.txt.gz#_2017-11-30_21_57_15_838147 | 21:34 |
melwitt | oh heh | 21:35 |
*** tidwellr has joined #openstack-nova | 21:35 | |
melwitt | so not the same | 21:35 |
*** tssurya_ has quit IRC | 21:36 | |
*** slaweq has quit IRC | 21:36 | |
*** peter-hamilton has quit IRC | 21:36 | |
mriedem | melwitt: 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=7d | 21:37 |
mriedem | nothing super obvious there, not like a single node provider | 21:38 |
mriedem | but it's all master branch, so i wonder if we're using a new cirros image in queens | 21:38 |
melwitt | how do we check that? | 21:39 |
mriedem | it's in devstack | 21:39 |
mriedem | https://github.com/openstack-dev/devstack/blob/master/stackrc#L671 | 21:40 |
mriedem | https://github.com/openstack-dev/devstack/commit/9f2dcd333103553626db1924a019e151e3e7252e | 21:40 |
melwitt | cool thanks | 21:40 |
mriedem | that's not new so... | 21:40 |
mriedem | heh read the commit message there | 21:40 |
*** salv-orlando has joined #openstack-nova | 21:40 | |
melwitt | just ran logstash for 30d and it looks like this is pretty recent stuff. nothing before 11/27 | 21:40 |
*** moshele has joined #openstack-nova | 21:41 | |
openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094 | 21:41 |
melwitt | haha, huh. | 21:41 |
mriedem | melwitt: logstash only goes back 10 days | 21:41 |
melwitt | oh :( | 21:42 |
melwitt | yeah I see what you mean, all providers hitting the kernel panic though pretty rarely | 21:42 |
*** salv-orlando has quit IRC | 21:43 | |
*** sambetts_ has quit IRC | 21:43 | |
*** BryanS68 has quit IRC | 21:43 | |
*** ragiman has quit IRC | 21:43 | |
*** esberglu has quit IRC | 21:43 | |
*** Guest99196 has quit IRC | 21:43 | |
*** yikun has quit IRC | 21:43 | |
*** slunkad_ has quit IRC | 21:43 | |
*** StevenK has quit IRC | 21:43 | |
*** alaski has quit IRC | 21:43 | |
*** ltomasbo has quit IRC | 21:43 | |
*** cargonza has quit IRC | 21:43 | |
*** d34dh0r53 has quit IRC | 21:43 | |
*** Jeffrey4l has quit IRC | 21:45 | |
*** slaweq_ has joined #openstack-nova | 21:47 | |
*** salv-orlando has joined #openstack-nova | 21:48 | |
*** sambetts_ has joined #openstack-nova | 21:48 | |
*** BryanS68 has joined #openstack-nova | 21:48 | |
*** ragiman has joined #openstack-nova | 21:48 | |
*** esberglu has joined #openstack-nova | 21:48 | |
*** Guest99196 has joined #openstack-nova | 21:48 | |
*** yikun has joined #openstack-nova | 21:48 | |
*** slunkad_ has joined #openstack-nova | 21:48 | |
*** StevenK has joined #openstack-nova | 21:48 | |
*** alaski has joined #openstack-nova | 21:48 | |
*** ltomasbo has joined #openstack-nova | 21:48 | |
*** cargonza has joined #openstack-nova | 21:48 | |
*** d34dh0r53 has joined #openstack-nova | 21:48 | |
*** Jeffrey4l has joined #openstack-nova | 21:48 | |
*** openstackgerrit has quit IRC | 21:48 | |
*** salv-orlando has quit IRC | 21:49 | |
*** salv-orlando has joined #openstack-nova | 21:49 | |
mriedem | melwitt: i assume i'm pulling threads here https://review.openstack.org/#/c/524234/ | 21:51 |
melwitt | haha, that's okay | 21:52 |
mriedem | also, might be better to move that into a utility in QUOTAS | 21:53 |
mriedem | so the callers don't have to know about NoopQuotaDriver being a thing | 21:53 |
mriedem | QUOTAS.is_noop? | 21:53 |
melwitt | after fighting with the test coverage I wasn't thinking too clearly. that's my lame excuse | 21:53 |
*** armax has joined #openstack-nova | 21:53 | |
mriedem | ok. i don't know what stephenfin's excuse was then. :) | 21:53 |
*** salv-orlando has quit IRC | 21:54 | |
mriedem | thinking about bangers and mash during the football game at the pub | 21:54 |
melwitt | how is is_noop not knowing about the driver? | 21:55 |
melwitt | honestly the way it was before (the values coming from inside the driver) is the way the caller knows the least about it :P | 21:55 |
mriedem | i'm saying hide that check within QUOTAS | 21:55 |
mriedem | true | 21:55 |
mriedem | so, QUOTAS.get_reserved()? | 21:56 |
melwitt | so I'm like ¯\_(ツ)_/¯ | 21:56 |
mriedem | umm | 21:56 |
mriedem | ^ is reserved for our api response when a cell is down and you list instances | 21:56 |
melwitt | wwhhaaaaat | 21:56 |
*** slaweq has joined #openstack-nova | 21:56 | |
mriedem | all we have is the uuid in the instance mapping, | 21:57 |
mriedem | so we're going to use "¯\_(ツ)_/¯" for all other fields | 21:57 |
melwitt | oh right | 21:57 |
*** felipemonteiro__ has quit IRC | 21:57 | |
melwitt | that would be fun | 21:57 |
*** baoli has joined #openstack-nova | 21:57 | |
mriedem | reserve that patch for april 1 | 21:57 |
*** yangyapeng has joined #openstack-nova | 21:57 | |
*** penick has joined #openstack-nova | 21:58 | |
*** slaweq has quit IRC | 22:01 | |
*** baoli has quit IRC | 22:02 | |
*** yangyapeng has quit IRC | 22:02 | |
*** threestrands has joined #openstack-nova | 22:04 | |
*** baoli has joined #openstack-nova | 22:04 | |
*** penick has quit IRC | 22:06 | |
*** hemna_ has joined #openstack-nova | 22:06 | |
*** rcernin has joined #openstack-nova | 22:09 | |
*** rcernin has quit IRC | 22:09 | |
*** rcernin has joined #openstack-nova | 22:09 | |
mriedem | melwitt: dansmith: remind me, in the cellsv1 job, is n-api the top-level API or is that the cell API? | 22:11 |
melwitt | top-level API | 22:12 |
melwitt | I 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 |
mriedem | and 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-nova | 22:12 | |
melwitt | oh boy, memory cobwebs | 22:13 |
melwitt | that sounds right though | 22:13 |
*** openstackgerrit has joined #openstack-nova | 22:14 | |
openstackgerrit | melanie witt proposed openstack/nova master: Follow up on removing old-style quotas code https://review.openstack.org/524234 | 22:14 |
*** penick has joined #openstack-nova | 22:15 | |
*** READ10 has quit IRC | 22:17 | |
mriedem | hmm, 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_579991 | 22:18 |
mriedem | the patch creates an instance action in the api | 22:19 |
mriedem | then when the compute is trying to look it up, it can't find it and blows up | 22:19 |
mriedem | so i'm guessing, | 22:19 |
mriedem | the top level api database has the action record, but not the cell | 22:19 |
mriedem | cell db i mean | 22:19 |
mriedem | but then i don't know how we have action records working for anything in cellsv1 | 22:20 |
melwitt | yeah, 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 race | 22:20 |
melwitt | like compute might try to read before the sync makes it to the cell | 22:20 |
mriedem | this isn't instance.save() | 22:20 |
melwitt | okay, then it won't sync it | 22:20 |
mriedem | objects.InstanceAction.action_start | 22:20 |
melwitt | hm, so yeah does compute rely on reading instance actions in any other case? | 22:21 |
melwitt | because if so, like you said those shouldn't work either | 22:21 |
mriedem | oh i think i see something | 22:21 |
mriedem | so stop_instance is in this list https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L51 | 22:22 |
mriedem | and so is backup_instance | 22:22 |
mriedem | https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L1714 | 22:22 |
mriedem | stop_instance has that ^ | 22:22 |
mriedem | backup_instance doesn't | 22:22 |
melwitt | and you're seeing a fail with backup? | 22:23 |
*** weshay is now known as weshay|ruck | 22:23 | |
mriedem | yeah https://review.openstack.org/#/c/523676/11 | 22:24 |
*** penick has quit IRC | 22:24 | |
melwitt | so returning the response from the instance action vs not | 22:24 |
melwitt | I didn't realize instance action had a rpc response in any case | 22:25 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Deduplicate functional test code https://review.openstack.org/526227 | 22:25 |
*** penick has joined #openstack-nova | 22:26 | |
*** ragiman has quit IRC | 22:27 | |
melwitt | yeah it looks like for some reason stop_instance can return something but backup_instance can't in nova/compute/rpcapi.py | 22:27 |
melwitt | I don't see stop_instance ever returning anything in compute/manager though | 22:28 |
*** penick has quit IRC | 22:29 | |
*** slaweq has joined #openstack-nova | 22:29 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete https://review.openstack.org/525231 | 22:29 |
mriedem | melwitt: i think the do_cast thing is something else | 22:29 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (2) https://review.openstack.org/526263 | 22:29 |
mriedem | red herring | 22:29 |
melwitt | yeah, it's just saying wait for response vs not | 22:29 |
mriedem | so i'm not sure what ever creates instance action records in the top level vs cell db | 22:29 |
melwitt | and looks correct based on backup vs stop | 22:30 |
melwitt | well, 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 db | 22:30 |
melwitt | I think only instance records are synced back and forth in cells v1 | 22:30 |
melwitt | I'm checking to see if anything in backup_instance in compute does a write of instance action records | 22:31 |
mriedem | but if that's the case, then how does our @wrap_instance_event in the compute not blow up on everything? | 22:31 |
melwitt | I guess it can't be, else they'd be found | 22:31 |
mriedem | the action record stuff for backup_instance is new to this patch | 22:31 |
mriedem | following the same pattern as everywhere else | 22:31 |
melwitt | okay, I'll look at that then | 22:31 |
melwitt | yeah, that's just writing it in n-api so it'll go in the top level db only | 22:32 |
melwitt | I'm not sure how the other actions are currently working | 22:32 |
*** mvk has joined #openstack-nova | 22:33 | |
*** slaweq has quit IRC | 22:34 | |
*** rcernin has quit IRC | 22:36 | |
*** hemna_ has quit IRC | 22:36 | |
*** sambetts_ has quit IRC | 22:36 | |
*** BryanS68 has quit IRC | 22:36 | |
*** esberglu has quit IRC | 22:36 | |
*** Guest99196 has quit IRC | 22:36 | |
*** yikun has quit IRC | 22:36 | |
*** slunkad_ has quit IRC | 22:36 | |
*** StevenK has quit IRC | 22:36 | |
*** alaski has quit IRC | 22:36 | |
*** ltomasbo has quit IRC | 22:36 | |
*** cargonza has quit IRC | 22:36 | |
*** d34dh0r53 has quit IRC | 22:36 | |
*** penick has joined #openstack-nova | 22:39 | |
*** tssurya_ has quit IRC | 22:39 | |
*** penick has quit IRC | 22:39 | |
*** rcernin has joined #openstack-nova | 22:42 | |
*** hemna_ has joined #openstack-nova | 22:42 | |
*** sambetts_ has joined #openstack-nova | 22:42 | |
*** BryanS68 has joined #openstack-nova | 22:42 | |
*** esberglu has joined #openstack-nova | 22:42 | |
*** Guest99196 has joined #openstack-nova | 22:42 | |
*** yikun has joined #openstack-nova | 22:42 | |
*** slunkad_ has joined #openstack-nova | 22:42 | |
*** StevenK has joined #openstack-nova | 22:42 | |
*** alaski has joined #openstack-nova | 22:42 | |
*** ltomasbo has joined #openstack-nova | 22:42 | |
*** cargonza has joined #openstack-nova | 22:42 | |
*** d34dh0r53 has joined #openstack-nova | 22:42 | |
*** yamahata has joined #openstack-nova | 22:44 | |
mriedem | alaski: 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 IRC | 22:46 | |
*** jaypipes has quit IRC | 22:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add instance action record for backup instances https://review.openstack.org/523676 | 22:47 |
*** esberglu has quit IRC | 22:50 | |
*** tidwellr has quit IRC | 22:50 | |
*** salv-orlando has joined #openstack-nova | 22:50 | |
*** gyee has joined #openstack-nova | 22:50 | |
*** slaweq_ has quit IRC | 22:50 | |
*** BryanS68 has quit IRC | 22:52 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: [WIP] POC to use neutron port_list when filtering instance by ip https://review.openstack.org/525505 | 22:53 |
*** moshele has quit IRC | 22:54 | |
*** penick has joined #openstack-nova | 22:54 | |
*** itlinux has quit IRC | 22:56 | |
*** salv-orlando has quit IRC | 22:57 | |
*** felipemonteiro_ has joined #openstack-nova | 22:57 | |
*** gouthamr has quit IRC | 22:59 | |
*** james_li_ has quit IRC | 23:02 | |
*** markvoelker has quit IRC | 23:03 | |
*** lyan has quit IRC | 23:03 | |
*** tasker has joined #openstack-nova | 23:04 | |
*** markvoelker has joined #openstack-nova | 23:04 | |
tasker | I'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 |
tasker | I 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-nova | 23:06 | |
tasker | is something translating it to GET? | 23:06 |
*** harlowja has joined #openstack-nova | 23:07 | |
*** lyan has joined #openstack-nova | 23:07 | |
*** yangyapeng has joined #openstack-nova | 23:07 | |
mriedem | tempest does a POST successfully from the guest to change the password | 23:08 |
mriedem | i remember because of this bug https://bugs.launchpad.net/nova/+bug/1696848 | 23:08 |
openstack | Launchpad 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 |
mriedem | http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/test_server_basic_ops.py#n90 | 23:08 |
mriedem | tasker: are you passing a body? | 23:09 |
tasker | yeah. though now that I think about it, it's probably malformed | 23:09 |
tasker | curl -D - -X POST -d 'cloudnull' http://169.254.169.254/openstack/latest/password | 23:10 |
*** yangyapeng has quit IRC | 23:11 | |
*** slaweq has quit IRC | 23:11 | |
*** imacdonn has quit IRC | 23:11 | |
tasker | before 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 |
mriedem | metadata api doesn't really ever change | 23:12 |
mriedem | mikal might have some ideas | 23:12 |
mriedem | he did give the talk about it in sydney | 23:12 |
tasker | I 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 |
tasker | I'll keep hacking and tracking seeing if I can figure out what's going on. | 23:13 |
mriedem | are you using service_metadata_proxy in nova.conf? | 23:13 |
tasker | yes. set to True | 23:14 |
*** imacdonn has joined #openstack-nova | 23:14 | |
*** salv-orlando has joined #openstack-nova | 23:16 | |
tasker | or | 23:17 |
tasker | oh | 23:17 |
tasker | and .. that explains a lot. | 23:17 |
mriedem | not set? | 23:17 |
tasker | it's set to True. | 23:17 |
*** amodi has quit IRC | 23:17 | |
tasker | and I don't use the `neutron_agents` nor its metadata service | 23:18 |
mriedem | ah | 23:18 |
tasker | thanks, Matt. that shed a lot of light on what's going on. | 23:19 |
tasker | back to work! | 23:19 |
mriedem | o/ | 23:19 |
tasker | to me. not you. | 23:19 |
tasker | you do whatever it was you were doing. .) | 23:19 |
mriedem | working | 23:19 |
*** felipemonteiro_ has quit IRC | 23:22 | |
*** salv-orlando has quit IRC | 23:26 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Traits ops on ProviderTree https://review.openstack.org/521605 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Aggregate ops on ProviderTree https://review.openstack.org/526216 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Move aggregates from report client to ProviderTree https://review.openstack.org/521685 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Track provider traits in report client https://review.openstack.org/521686 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 23:26 |
efried | I broke the patchbot. | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | 23:26 |
openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | 23:26 |
*** salv-orlando has joined #openstack-nova | 23:26 | |
efried | ah, there it goes. | 23:26 |
efried | I got `related changes` to scroll :) | 23:27 |
*** erlon has quit IRC | 23:28 | |
*** yamahata has quit IRC | 23:29 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Add regression test for rebuilding a volume-backed server https://review.openstack.org/526542 | 23:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Only query BDMs once in API during rebuild https://review.openstack.org/526543 | 23:30 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/newton: Get original image_id from volume for volume-backed instance rebuild https://review.openstack.org/526544 | 23:30 |
*** salv-orlando has quit IRC | 23:30 | |
mriedem | tonyb: here we go https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton | 23:31 |
mriedem | i think that's going to be the end for newton | 23:31 |
mriedem | for real this time | 23:31 |
mriedem | i know i say this every week | 23:31 |
tonyb | ;P | 23:31 |
tonyb | I'll look at them today | 23:31 |
mriedem | https://www.youtube.com/watch?v=gfkfXcme5DA | 23:31 |
openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: skip authentication on root URI https://review.openstack.org/522002 | 23:32 |
mriedem | mtreinish: where are you when i make simpsons references? | 23:33 |
*** lyan has quit IRC | 23:37 | |
*** liverpooler has quit IRC | 23:37 | |
openstackgerrit | Hongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2 https://review.openstack.org/524755 | 23:39 |
*** penick has quit IRC | 23:40 | |
*** gouthamr has joined #openstack-nova | 23:42 | |
*** slaweq has joined #openstack-nova | 23:44 | |
*** yangyapeng has joined #openstack-nova | 23:47 | |
*** slaweq has quit IRC | 23:49 | |
*** yangyapeng has quit IRC | 23:51 | |
*** threestrands has quit IRC | 23:55 | |
*** yamamoto has joined #openstack-nova | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!