| *** dtp has quit IRC | 00:04 | |
| *** abalutoiu has quit IRC | 00:05 | |
| *** gmann has quit IRC | 00:11 | |
| *** gmann has joined #openstack-nova | 00:11 | |
| *** thorst has quit IRC | 00:17 | |
| *** rajathagasthya has quit IRC | 00:19 | |
| *** Swami has quit IRC | 00:22 | |
| *** sree has joined #openstack-nova | 00:23 | |
| mriedem | dansmith: video ram isn't tracked for placement :) it's checked on server create and decremented from ram quota on server delete, and checked in the libvirt driver, but not part of the resource claim tracking :) | 00:24 |
|---|---|---|
| mriedem | of course | 00:24 |
| openstackgerrit | Merged openstack/nova master: Pre-load instance.device_metadata in InstanceMetadata https://review.openstack.org/481235 | 00:26 |
| openstackgerrit | Merged openstack/nova master: hyperv: Fixes log message in livemigrationops https://review.openstack.org/479245 | 00:27 |
| *** sree has quit IRC | 00:27 | |
| *** vishwanathj has quit IRC | 00:30 | |
| *** vishwanathj has joined #openstack-nova | 00:30 | |
| *** xyang1 has quit IRC | 00:35 | |
| *** zhurong has joined #openstack-nova | 00:42 | |
| *** fragatin_ has joined #openstack-nova | 00:43 | |
| *** slaweq has joined #openstack-nova | 00:43 | |
| *** trungnv has quit IRC | 00:43 | |
| *** hieulq has quit IRC | 00:44 | |
| mriedem | alex_xu: i replied to your comments in https://review.openstack.org/#/c/464280/ - i probably won't be able to attend the api meeting this week, so i might post some thoughts in the ML for discussion, but i'm struggling with deciding if we should allow the same behavior as before regarding trying to disable an already disabled service and failing or ignoring it, same for enabling an enabled service since that's not a failure t | 00:45 |
| mriedem | but that's essentially how i wrote that change - it won't fail if you try to enable and enabled service and vice versa for disabled | 00:45 |
| mriedem | *an enabled service | 00:46 |
| *** fragatina has quit IRC | 00:46 | |
| *** fragatin_ has quit IRC | 00:47 | |
| *** Apoorva has quit IRC | 00:48 | |
| *** fragatina has joined #openstack-nova | 00:48 | |
| *** slaweq has quit IRC | 00:48 | |
| *** Shunli has joined #openstack-nova | 00:48 | |
| *** baoli has joined #openstack-nova | 00:49 | |
| *** vishwanathj has quit IRC | 00:53 | |
| *** vishwanathj has joined #openstack-nova | 00:54 | |
| *** liujiong has joined #openstack-nova | 00:56 | |
| *** gcb has quit IRC | 00:57 | |
| *** abalutoiu has joined #openstack-nova | 00:58 | |
| *** huanxie has joined #openstack-nova | 00:58 | |
| *** acormier has joined #openstack-nova | 00:58 | |
| *** trungnv has joined #openstack-nova | 00:58 | |
| *** hieulq has joined #openstack-nova | 00:59 | |
| *** Nel1x has quit IRC | 00:59 | |
| *** vladikr has quit IRC | 01:02 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| *** gouthamr has quit IRC | 01:06 | |
| *** dixiaoli has joined #openstack-nova | 01:09 | |
| *** yufei has joined #openstack-nova | 01:10 | |
| *** acormier has quit IRC | 01:12 | |
| *** acormier has joined #openstack-nova | 01:12 | |
| *** trungnv has quit IRC | 01:14 | |
| *** masuberu has joined #openstack-nova | 01:14 | |
| *** itlinux has joined #openstack-nova | 01:15 | |
| *** phuongnh has quit IRC | 01:16 | |
| *** hieulq has quit IRC | 01:16 | |
| *** vishwanathj has quit IRC | 01:16 | |
| *** vishwanathj has joined #openstack-nova | 01:17 | |
| *** vishwanathj has quit IRC | 01:18 | |
| *** masber has quit IRC | 01:18 | |
| *** vishwanathj has joined #openstack-nova | 01:18 | |
| *** vvargaszte has quit IRC | 01:20 | |
| *** acormier has quit IRC | 01:26 | |
| *** acormier has joined #openstack-nova | 01:26 | |
| *** mriedem has quit IRC | 01:33 | |
| *** awaugama has quit IRC | 01:38 | |
| *** slaweq has joined #openstack-nova | 01:45 | |
| *** slaweq has quit IRC | 01:49 | |
| *** masber has joined #openstack-nova | 01:57 | |
| *** yushb has joined #openstack-nova | 01:57 | |
| *** annegentle has joined #openstack-nova | 01:58 | |
| *** masuberu has quit IRC | 02:00 | |
| *** yamahata has quit IRC | 02:00 | |
| *** smatzek has joined #openstack-nova | 02:03 | |
| *** trungnv has joined #openstack-nova | 02:07 | |
| *** hieulq has joined #openstack-nova | 02:07 | |
| *** acormier has quit IRC | 02:08 | |
| *** acormier has joined #openstack-nova | 02:08 | |
| *** hongbin has joined #openstack-nova | 02:09 | |
| *** kiennt has joined #openstack-nova | 02:10 | |
| *** Vek has quit IRC | 02:10 | |
| *** huanxie has quit IRC | 02:18 | |
| *** fragatin_ has joined #openstack-nova | 02:18 | |
| *** coreywright has quit IRC | 02:20 | |
| *** litao__ has joined #openstack-nova | 02:22 | |
| *** trungnv has quit IRC | 02:22 | |
| *** fragatina has quit IRC | 02:22 | |
| *** gouthamr has joined #openstack-nova | 02:25 | |
| *** annegentle has quit IRC | 02:29 | |
| *** itlinux has quit IRC | 02:31 | |
| *** fragatin_ has quit IRC | 02:33 | |
| *** coreywright has joined #openstack-nova | 02:34 | |
| *** fragatina has joined #openstack-nova | 02:34 | |
| *** fragatina has quit IRC | 02:34 | |
| *** trungnv has joined #openstack-nova | 02:35 | |
| *** fragatina has joined #openstack-nova | 02:35 | |
| *** thorst has joined #openstack-nova | 02:39 | |
| *** hieulq has quit IRC | 02:39 | |
| *** kiennt has quit IRC | 02:40 | |
| *** trungnv has quit IRC | 02:40 | |
| *** thorst has quit IRC | 02:44 | |
| *** slaweq has joined #openstack-nova | 02:45 | |
| *** yamahata has joined #openstack-nova | 02:49 | |
| *** slaweq has quit IRC | 02:50 | |
| *** jmlowe has joined #openstack-nova | 02:50 | |
| *** zzzeek_ has quit IRC | 02:52 | |
| *** smatzek has quit IRC | 03:00 | |
| *** rajathagasthya has joined #openstack-nova | 03:05 | |
| *** rajathagasthya has quit IRC | 03:05 | |
| *** zhurong has quit IRC | 03:08 | |
| *** itlinux has joined #openstack-nova | 03:09 | |
| *** acormier has quit IRC | 03:10 | |
| *** acormier has joined #openstack-nova | 03:10 | |
| *** acormier has quit IRC | 03:15 | |
| *** trungnv has joined #openstack-nova | 03:16 | |
| *** brad[] has quit IRC | 03:16 | |
| *** hieulq has joined #openstack-nova | 03:16 | |
| *** tbachman has joined #openstack-nova | 03:17 | |
| *** dave-mccowan has joined #openstack-nova | 03:19 | |
| *** baoli has quit IRC | 03:22 | |
| *** itlinux has quit IRC | 03:23 | |
| openstackgerrit | zhangdaolong proposed openstack/nova master: Fix appropriate parameters name for unit test https://review.openstack.org/482782 | 03:23 |
| *** huanxie has joined #openstack-nova | 03:24 | |
| *** baoli has joined #openstack-nova | 03:26 | |
| *** gbarros has quit IRC | 03:28 | |
| *** baoli has quit IRC | 03:31 | |
| *** gcb has joined #openstack-nova | 03:36 | |
| *** yufei has quit IRC | 03:40 | |
| *** dave-mccowan has quit IRC | 03:40 | |
| *** udesale has joined #openstack-nova | 03:44 | |
| *** zhurong has joined #openstack-nova | 03:44 | |
| *** kiennt has joined #openstack-nova | 03:45 | |
| *** gouthamr has quit IRC | 03:46 | |
| *** slaweq has joined #openstack-nova | 03:46 | |
| *** slaweq has quit IRC | 03:51 | |
| *** huanxie has quit IRC | 03:54 | |
| *** itlinux has joined #openstack-nova | 03:54 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 03:55 |
| *** tonyb has quit IRC | 04:05 | |
| *** NostawRm has quit IRC | 04:08 | |
| *** Vek has joined #openstack-nova | 04:12 | |
| *** mdnadeem has joined #openstack-nova | 04:13 | |
| *** NostawRm has joined #openstack-nova | 04:13 | |
| *** phuongnh has joined #openstack-nova | 04:15 | |
| *** adisky__ has joined #openstack-nova | 04:21 | |
| *** phuongnh has quit IRC | 04:27 | |
| *** kiennt has quit IRC | 04:27 | |
| *** cNilesh has joined #openstack-nova | 04:27 | |
| *** tonyb has joined #openstack-nova | 04:28 | |
| *** swebster has joined #openstack-nova | 04:28 | |
| *** swebster_ has quit IRC | 04:29 | |
| *** trungnv has quit IRC | 04:29 | |
| *** hieulq has quit IRC | 04:30 | |
| *** psachin has joined #openstack-nova | 04:30 | |
| *** fragatin_ has joined #openstack-nova | 04:31 | |
| *** yushb has quit IRC | 04:32 | |
| *** yingjun has joined #openstack-nova | 04:32 | |
| *** yamamoto has joined #openstack-nova | 04:33 | |
| *** fragatina has quit IRC | 04:34 | |
| *** hongbin has quit IRC | 04:35 | |
| *** fragatin_ has quit IRC | 04:35 | |
| *** gongysh has joined #openstack-nova | 04:35 | |
| *** tbh__ has joined #openstack-nova | 04:38 | |
| *** tbh__ is now known as tbh_ | 04:38 | |
| *** thorst has joined #openstack-nova | 04:40 | |
| *** pcaruana has joined #openstack-nova | 04:41 | |
| *** fragatina has joined #openstack-nova | 04:44 | |
| *** trinaths has joined #openstack-nova | 04:44 | |
| *** thorst has quit IRC | 04:45 | |
| *** kiennt has joined #openstack-nova | 04:45 | |
| *** trungnv has joined #openstack-nova | 04:46 | |
| *** hieulq has joined #openstack-nova | 04:47 | |
| *** fragatina has quit IRC | 04:48 | |
| *** trinaths1 has joined #openstack-nova | 04:53 | |
| *** jmlowe has quit IRC | 04:56 | |
| *** trinaths has quit IRC | 04:57 | |
| *** pcaruana has quit IRC | 04:58 | |
| *** trinaths has joined #openstack-nova | 04:59 | |
| *** trinaths1 has quit IRC | 05:00 | |
| *** huanxie has joined #openstack-nova | 05:01 | |
| *** yufei has joined #openstack-nova | 05:04 | |
| *** slaweq has joined #openstack-nova | 05:05 | |
| *** sree has joined #openstack-nova | 05:05 | |
| *** slaweq has quit IRC | 05:05 | |
| *** fragatina has joined #openstack-nova | 05:14 | |
| *** Shunli has quit IRC | 05:14 | |
| *** armax has quit IRC | 05:14 | |
| *** armax has joined #openstack-nova | 05:14 | |
| *** armax has quit IRC | 05:15 | |
| *** Shunli has joined #openstack-nova | 05:15 | |
| *** armax has joined #openstack-nova | 05:15 | |
| *** armax has quit IRC | 05:16 | |
| *** NostawRm has quit IRC | 05:16 | |
| *** NostawRm has joined #openstack-nova | 05:16 | |
| *** armax has joined #openstack-nova | 05:16 | |
| *** armax has quit IRC | 05:16 | |
| *** tbachman has quit IRC | 05:17 | |
| *** armax has joined #openstack-nova | 05:17 | |
| *** armax has quit IRC | 05:17 | |
| *** krtaylor has quit IRC | 05:17 | |
| *** armax has joined #openstack-nova | 05:17 | |
| *** armax has quit IRC | 05:18 | |
| *** fragatina has quit IRC | 05:18 | |
| *** tbachman has joined #openstack-nova | 05:18 | |
| *** armax has joined #openstack-nova | 05:18 | |
| *** armax has quit IRC | 05:19 | |
| *** armax has joined #openstack-nova | 05:19 | |
| *** armax has quit IRC | 05:20 | |
| *** baoli has joined #openstack-nova | 05:22 | |
| *** baoli has quit IRC | 05:27 | |
| *** hieulq has quit IRC | 05:27 | |
| *** trungnv has quit IRC | 05:28 | |
| *** kiennt has quit IRC | 05:29 | |
| *** rcernin has joined #openstack-nova | 05:33 | |
| *** ratailor has joined #openstack-nova | 05:36 | |
| *** gcb has quit IRC | 05:37 | |
| *** nicolasbock has joined #openstack-nova | 05:38 | |
| *** diga has joined #openstack-nova | 05:38 | |
| *** moshele has joined #openstack-nova | 05:44 | |
| *** trungnv has joined #openstack-nova | 05:44 | |
| *** hieulq has joined #openstack-nova | 05:45 | |
| *** lpetrut has joined #openstack-nova | 05:46 | |
| *** itlinux has quit IRC | 05:52 | |
| *** gyee has quit IRC | 05:52 | |
| openstackgerrit | Merged openstack/nova master: Fix quota class set APIs https://review.openstack.org/467999 | 05:56 |
| *** ekuris has joined #openstack-nova | 05:57 | |
| *** prateek has joined #openstack-nova | 05:58 | |
| *** huanxie has quit IRC | 05:59 | |
| *** yushb has joined #openstack-nova | 06:00 | |
| *** yassine has quit IRC | 06:01 | |
| *** slaweq has joined #openstack-nova | 06:02 | |
| *** slaweq_ has joined #openstack-nova | 06:06 | |
| *** slaweq has quit IRC | 06:06 | |
| *** slaweq has joined #openstack-nova | 06:07 | |
| *** kiennt has joined #openstack-nova | 06:08 | |
| *** tjones has quit IRC | 06:09 | |
| *** slaweq_ has quit IRC | 06:12 | |
| *** crushil has quit IRC | 06:14 | |
| *** crushil has joined #openstack-nova | 06:15 | |
| *** ekuris has quit IRC | 06:15 | |
| *** liujiong has quit IRC | 06:19 | |
| *** Oku_OS-away is now known as Oku_OS | 06:19 | |
| *** liujiong has joined #openstack-nova | 06:19 | |
| *** Shunli has quit IRC | 06:20 | |
| *** Shunli has joined #openstack-nova | 06:21 | |
| *** kiennt_ has joined #openstack-nova | 06:21 | |
| *** kiennt has quit IRC | 06:22 | |
| *** phuongnh has joined #openstack-nova | 06:27 | |
| *** crushil_ has joined #openstack-nova | 06:28 | |
| *** Shunli has quit IRC | 06:30 | |
| *** ekuris has joined #openstack-nova | 06:31 | |
| *** sshwarts has joined #openstack-nova | 06:32 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Modify the value of instance's root_gb when flavor's root_gb is 0 https://review.openstack.org/398891 | 06:34 |
| *** gongysh has quit IRC | 06:34 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 06:36 |
| *** gjayavelu has joined #openstack-nova | 06:37 | |
| *** smcginnis has quit IRC | 06:39 | |
| *** lpetrut has quit IRC | 06:41 | |
| *** dixiaoli has quit IRC | 06:41 | |
| *** thorst has joined #openstack-nova | 06:41 | |
| *** phuongnh has quit IRC | 06:43 | |
| *** kiennt_ has quit IRC | 06:43 | |
| *** smcginnis has joined #openstack-nova | 06:43 | |
| *** andreas_s has joined #openstack-nova | 06:44 | |
| *** kiennt has joined #openstack-nova | 06:44 | |
| *** phuongnh has joined #openstack-nova | 06:45 | |
| *** thorst has quit IRC | 06:46 | |
| *** belmoreira has joined #openstack-nova | 06:47 | |
| *** openstackgerrit has quit IRC | 06:48 | |
| *** liujiong_lj has joined #openstack-nova | 06:50 | |
| -openstackstatus- NOTICE: FYI, ask.openstack.org is down, review.o.o is slow - please have patience until this is fixed | 06:52 | |
| *** phuongnh has quit IRC | 06:53 | |
| *** kiennt has quit IRC | 06:53 | |
| *** kman has joined #openstack-nova | 06:53 | |
| *** liujiong has quit IRC | 06:54 | |
| *** phuongnh has joined #openstack-nova | 06:54 | |
| *** kiennt has joined #openstack-nova | 06:55 | |
| *** aloga has quit IRC | 06:55 | |
| *** zhurong has quit IRC | 06:55 | |
| *** trinaths is now known as trinaths_lunch | 06:59 | |
| *** phuongnh has quit IRC | 07:03 | |
| *** udesale has quit IRC | 07:03 | |
| *** kiennt has quit IRC | 07:03 | |
| *** phuongnh has joined #openstack-nova | 07:05 | |
| *** kiennt has joined #openstack-nova | 07:05 | |
| *** slaweq_ has joined #openstack-nova | 07:07 | |
| *** damien_r has joined #openstack-nova | 07:10 | |
| *** gjayavelu has quit IRC | 07:11 | |
| *** tesseract has joined #openstack-nova | 07:11 | |
| *** slaweq_ has quit IRC | 07:12 | |
| *** aloga has joined #openstack-nova | 07:14 | |
| *** gjayavelu has joined #openstack-nova | 07:15 | |
| *** kman has quit IRC | 07:17 | |
| *** yassine has joined #openstack-nova | 07:18 | |
| *** hieulq has quit IRC | 07:19 | |
| *** trungnv has quit IRC | 07:20 | |
| *** kiennt has quit IRC | 07:22 | |
| *** phuongnh has quit IRC | 07:22 | |
| *** edmondsw has joined #openstack-nova | 07:24 | |
| *** phuongnh has joined #openstack-nova | 07:25 | |
| *** kiennt has joined #openstack-nova | 07:25 | |
| *** crushil_ has quit IRC | 07:27 | |
| *** edmondsw has quit IRC | 07:29 | |
| *** alexchadin has joined #openstack-nova | 07:30 | |
| *** pcaruana has joined #openstack-nova | 07:31 | |
| *** zhurong has joined #openstack-nova | 07:31 | |
| *** aarefiev_afk is now known as aarefiev | 07:32 | |
| *** trungnv has joined #openstack-nova | 07:34 | |
| *** hieulq has joined #openstack-nova | 07:35 | |
| *** tuanluong has joined #openstack-nova | 07:36 | |
| *** ygl has joined #openstack-nova | 07:36 | |
| *** ygl has left #openstack-nova | 07:37 | |
| *** roeyc has joined #openstack-nova | 07:38 | |
| *** openstackgerrit has joined #openstack-nova | 07:38 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013 | 07:38 |
| *** claudiub has joined #openstack-nova | 07:39 | |
| *** pcaruana has quit IRC | 07:41 | |
| *** ekuris has quit IRC | 07:41 | |
| danpawlik | Good morning | 07:42 |
| danpawlik | Does someone know, if its possible to change keyboard layout in novnc proxy without changing instnce xml file? | 07:43 |
| *** hferenc has quit IRC | 07:43 | |
| *** kristian__ has joined #openstack-nova | 07:44 | |
| *** amotoki_away is now known as amotoki | 07:45 | |
| *** kristian__ has quit IRC | 07:45 | |
| *** pcaruana has joined #openstack-nova | 07:45 | |
| *** hferenc has joined #openstack-nova | 07:45 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add quiesce and unquiesce in support matrix https://review.openstack.org/482390 | 07:45 |
| openstackgerrit | zhangdaolong proposed openstack/nova master: Fix wrong log parm. https://review.openstack.org/482832 | 07:47 |
| *** vvargaszte has joined #openstack-nova | 07:48 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted, due to low performance | 07:49 | |
| *** sree has quit IRC | 07:54 | |
| *** ekuris has joined #openstack-nova | 07:55 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 07:57 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482020 | 07:57 |
| *** sree_ has joined #openstack-nova | 07:58 | |
| *** sree_ is now known as Guest58976 | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** kristian__ has joined #openstack-nova | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** gouthamr has joined #openstack-nova | 08:02 | |
| *** liujiong_lj is now known as liujiong | 08:02 | |
| *** Guest58976 has quit IRC | 08:03 | |
| *** slaweq_ has joined #openstack-nova | 08:08 | |
| *** ekuris_ has joined #openstack-nova | 08:09 | |
| *** ekuris has quit IRC | 08:09 | |
| *** dr_gogeta86 has joined #openstack-nova | 08:13 | |
| *** slaweq_ has quit IRC | 08:13 | |
| *** trinaths_lunch is now known as trinaths | 08:13 | |
| *** dr_gogeta86 has quit IRC | 08:13 | |
| *** dr_gogeta86 has joined #openstack-nova | 08:13 | |
| *** ralonsoh has joined #openstack-nova | 08:17 | |
| *** efoley has joined #openstack-nova | 08:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 08:20 |
| *** gouthamr has quit IRC | 08:21 | |
| *** tojuvone has quit IRC | 08:26 | |
| *** kristian__ has quit IRC | 08:26 | |
| *** tojuvone has joined #openstack-nova | 08:26 | |
| *** yushb has quit IRC | 08:30 | |
| *** yushb has joined #openstack-nova | 08:30 | |
| *** lucas-afk is now known as lucasagomes | 08:32 | |
| *** tojuvone has quit IRC | 08:33 | |
| *** pcaruana has quit IRC | 08:33 | |
| *** tojuvone has joined #openstack-nova | 08:34 | |
| bauzas | p!spy | 08:36 |
| *** thorst has joined #openstack-nova | 08:42 | |
| *** dklyle has joined #openstack-nova | 08:45 | |
| *** david-lyle has quit IRC | 08:45 | |
| openstackgerrit | Merged openstack/os-vif master: Enable some off-by-default checks https://review.openstack.org/482384 | 08:46 |
| *** pcaruana has joined #openstack-nova | 08:46 | |
| *** karimb has joined #openstack-nova | 08:46 | |
| *** thorst has quit IRC | 08:47 | |
| *** vvargaszte1 has joined #openstack-nova | 08:47 | |
| *** liujiong has quit IRC | 08:47 | |
| *** liujiong has joined #openstack-nova | 08:47 | |
| *** tbh_ has quit IRC | 08:48 | |
| ildikov | bauzas: hi :) | 08:49 |
| bauzas | ildikov: hey | 08:49 |
| openstackgerrit | Merged openstack/os-vif master: set mtu on all code paths https://review.openstack.org/469951 | 08:49 |
| * bauzas in meeting | 08:49 | |
| ildikov | bauzas: I have a review with one +2 on it already: https://review.openstack.org/#/c/456971/ | 08:49 |
| ildikov | bauzas: was just wondering if you could take a look when you have a little time :) | 08:49 |
| ildikov | bauzas: pretty please :) | 08:49 |
| bauzas | ildikov: sure, I can | 08:49 |
| ildikov | bauzas: awesome, thank you! | 08:50 |
| *** vvargaszte has quit IRC | 08:50 | |
| *** vvargaszte1 is now known as vvargaszte | 08:50 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 08:50 |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 08:51 |
| *** karimb has quit IRC | 08:56 | |
| *** yushb has quit IRC | 08:58 | |
| *** mkucia has joined #openstack-nova | 08:59 | |
| *** udesale has joined #openstack-nova | 09:02 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Changed maximum code complexity to 30 https://review.openstack.org/351176 | 09:02 |
| openstackgerrit | iswarya vakati proposed openstack/nova master: Fixed wrap from taking negative values https://review.openstack.org/481465 | 09:04 |
| *** karimb has joined #openstack-nova | 09:07 | |
| *** ygl has joined #openstack-nova | 09:09 | |
| *** slaweq_ has joined #openstack-nova | 09:09 | |
| *** claudiub|2 has joined #openstack-nova | 09:11 | |
| *** edmondsw has joined #openstack-nova | 09:12 | |
| *** claudiub has quit IRC | 09:14 | |
| *** slaweq_ has quit IRC | 09:14 | |
| *** liujiong has quit IRC | 09:15 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore https://review.openstack.org/478901 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore https://review.openstack.org/478902 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore https://review.openstack.org/478499 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-volumes instead of stevedore https://review.openstack.org/479497 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore https://review.openstack.org/478498 | 09:15 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904 | 09:15 |
| *** liujiong has joined #openstack-nova | 09:15 | |
| *** claudiub has joined #openstack-nova | 09:15 | |
| *** edmondsw has quit IRC | 09:16 | |
| *** claudiub|2 has quit IRC | 09:18 | |
| *** yushb has joined #openstack-nova | 09:21 | |
| *** lpetrut has joined #openstack-nova | 09:27 | |
| *** huanxie has joined #openstack-nova | 09:31 | |
| *** pcaruana has quit IRC | 09:32 | |
| *** pcaruana has joined #openstack-nova | 09:32 | |
| *** yamahata has quit IRC | 09:34 | |
| *** sambetts|afk is now known as sambetts | 09:40 | |
| *** yingjun has quit IRC | 09:40 | |
| *** thorst has joined #openstack-nova | 09:43 | |
| *** mvk has joined #openstack-nova | 09:44 | |
| *** thorst has quit IRC | 09:47 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479 | 09:48 |
| openstackgerrit | jichenjc proposed openstack/nova master: Document device tags in support matrix https://review.openstack.org/481478 | 09:48 |
| *** kristian__ has joined #openstack-nova | 09:49 | |
| *** karimb has quit IRC | 09:50 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Recreate root volume for instance rebuild https://review.openstack.org/478330 | 09:51 |
| *** brad[] has joined #openstack-nova | 09:57 | |
| *** cdent has joined #openstack-nova | 09:57 | |
| *** karimb has joined #openstack-nova | 09:58 | |
| *** yufei has quit IRC | 09:58 | |
| *** cnf has joined #openstack-nova | 10:01 | |
| *** cnf has left #openstack-nova | 10:01 | |
| *** ygl has quit IRC | 10:02 | |
| *** gjayavelu has quit IRC | 10:08 | |
| *** slaweq_ has joined #openstack-nova | 10:10 | |
| *** kiennt has quit IRC | 10:12 | |
| *** phuongnh has quit IRC | 10:14 | |
| *** slaweq_ has quit IRC | 10:14 | |
| *** slaweq_ has joined #openstack-nova | 10:14 | |
| *** slaweq_ has quit IRC | 10:14 | |
| *** tuanluong has quit IRC | 10:14 | |
| *** hieulq has quit IRC | 10:16 | |
| *** trungnv has quit IRC | 10:18 | |
| *** sridharg has quit IRC | 10:21 | |
| *** liujiong has quit IRC | 10:23 | |
| *** udesale__ has joined #openstack-nova | 10:24 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Recreate root volume for instance rebuild https://review.openstack.org/478330 | 10:27 |
| *** udesale has quit IRC | 10:27 | |
| *** yushb has quit IRC | 10:27 | |
| *** dklyle has quit IRC | 10:28 | |
| *** david-lyle has joined #openstack-nova | 10:28 | |
| *** alexchadin has quit IRC | 10:28 | |
| *** sridharg has joined #openstack-nova | 10:30 | |
| *** udesale has joined #openstack-nova | 10:37 | |
| *** vvargaszte has quit IRC | 10:37 | |
| *** vvargaszte has joined #openstack-nova | 10:38 | |
| *** udesale__ has quit IRC | 10:38 | |
| *** vvargaszte has quit IRC | 10:38 | |
| *** vvargaszte has joined #openstack-nova | 10:39 | |
| *** zhurong has quit IRC | 10:41 | |
| *** vvargaszte has quit IRC | 10:41 | |
| *** vvargaszte has joined #openstack-nova | 10:41 | |
| *** tbachman_ has joined #openstack-nova | 10:42 | |
| *** vvargaszte has quit IRC | 10:42 | |
| *** vvargaszte has joined #openstack-nova | 10:42 | |
| *** tbachman has quit IRC | 10:44 | |
| *** tbachman_ is now known as tbachman | 10:44 | |
| *** udesale has quit IRC | 10:45 | |
| *** xinliang has quit IRC | 10:48 | |
| *** hferenc has quit IRC | 10:57 | |
| *** xinliang has joined #openstack-nova | 11:01 | |
| *** trungnv has joined #openstack-nova | 11:04 | |
| *** hieulq has joined #openstack-nova | 11:04 | |
| *** hieulq has quit IRC | 11:08 | |
| *** markvoelker has quit IRC | 11:09 | |
| *** lucasagomes is now known as lucas-hungry | 11:10 | |
| *** markvoelker has joined #openstack-nova | 11:10 | |
| *** markvoelker has quit IRC | 11:14 | |
| *** slaweq_ has joined #openstack-nova | 11:15 | |
| *** trungnv has quit IRC | 11:16 | |
| *** smatzek has joined #openstack-nova | 11:18 | |
| *** dave-mccowan has joined #openstack-nova | 11:19 | |
| *** slaweq_ has quit IRC | 11:20 | |
| *** gszasz has joined #openstack-nova | 11:21 | |
| *** ratailor has quit IRC | 11:27 | |
| *** acormier has joined #openstack-nova | 11:29 | |
| *** liverpooler has quit IRC | 11:31 | |
| *** lyan has joined #openstack-nova | 11:34 | |
| *** hferenc has joined #openstack-nova | 11:36 | |
| *** acormier has quit IRC | 11:37 | |
| *** acormier has joined #openstack-nova | 11:38 | |
| *** thorst has joined #openstack-nova | 11:38 | |
| *** huanxie has quit IRC | 11:38 | |
| *** acormier has quit IRC | 11:42 | |
| *** hieulq has joined #openstack-nova | 11:44 | |
| *** kristia__ has joined #openstack-nova | 11:45 | |
| *** trungnv has joined #openstack-nova | 11:45 | |
| *** ecelik has joined #openstack-nova | 11:47 | |
| *** ecelik has left #openstack-nova | 11:47 | |
| *** kristian__ has quit IRC | 11:47 | |
| *** smatzek has quit IRC | 11:50 | |
| *** karimb has quit IRC | 11:52 | |
| *** efried_zzz is now known as efried | 11:52 | |
| *** smatzek has joined #openstack-nova | 11:53 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for allocation_candidates https://review.openstack.org/481112 | 11:53 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for usages https://review.openstack.org/480563 | 11:53 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Make placement_api_docs.py failing https://review.openstack.org/480924 | 11:53 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP usages https://review.openstack.org/450105 | 11:53 |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Add api-ref for RP allocations https://review.openstack.org/450550 | 11:53 |
| *** markvoelker has joined #openstack-nova | 11:55 | |
| *** litao__ has quit IRC | 11:56 | |
| *** alexchadin has joined #openstack-nova | 12:00 | |
| *** READ10 has quit IRC | 12:01 | |
| *** mkucia has quit IRC | 12:03 | |
| *** sridharg has quit IRC | 12:07 | |
| *** lucas-hungry is now known as lucasagomes | 12:09 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Document device tags in support matrix https://review.openstack.org/481478 | 12:10 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Document device tags in support matrix https://review.openstack.org/481478 | 12:11 |
| *** pchavva has joined #openstack-nova | 12:13 | |
| *** slaweq_ has joined #openstack-nova | 12:16 | |
| *** udesale has joined #openstack-nova | 12:16 | |
| *** MVenesio has joined #openstack-nova | 12:18 | |
| *** lyan has quit IRC | 12:19 | |
| *** slaweq_ has quit IRC | 12:21 | |
| *** acormier has joined #openstack-nova | 12:21 | |
| *** jamesden_ has joined #openstack-nova | 12:21 | |
| *** acormier has quit IRC | 12:22 | |
| *** acormier has joined #openstack-nova | 12:22 | |
| *** krtaylor has joined #openstack-nova | 12:23 | |
| *** jamesdenton has quit IRC | 12:24 | |
| *** sridharg has joined #openstack-nova | 12:24 | |
| *** diga has quit IRC | 12:24 | |
| *** acormier has quit IRC | 12:27 | |
| *** crushil_ has joined #openstack-nova | 12:30 | |
| *** yushb has joined #openstack-nova | 12:32 | |
| *** zzzeek_ has joined #openstack-nova | 12:32 | |
| *** crushil is now known as rushil | 12:33 | |
| *** crushil_ is now known as crushil | 12:33 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-groups instead of stevedore https://review.openstack.org/478901 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the test for the route '/resources.:(format)' https://review.openstack.org/478900 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for server-security-groups instead of stevedore https://review.openstack.org/478902 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for image-metadata instead of stevedore https://review.openstack.org/478499 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-volumes instead of stevedore https://review.openstack.org/479497 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for images instead of stevedore https://review.openstack.org/478498 | 12:37 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904 | 12:37 |
| *** wasmum has quit IRC | 12:40 | |
| *** icey_ has joined #openstack-nova | 12:40 | |
| *** sballe__ has joined #openstack-nova | 12:40 | |
| *** zhenguo_ has joined #openstack-nova | 12:40 | |
| *** fmccrthy_ has joined #openstack-nova | 12:40 | |
| *** szaher has quit IRC | 12:40 | |
| *** cNilesh has quit IRC | 12:41 | |
| *** efried_ has joined #openstack-nova | 12:41 | |
| *** yamamoto has quit IRC | 12:41 | |
| *** fmccrthy has quit IRC | 12:41 | |
| *** ujjain- has quit IRC | 12:41 | |
| *** DinaBelova has quit IRC | 12:41 | |
| *** rfolco has quit IRC | 12:41 | |
| *** icey has quit IRC | 12:41 | |
| *** vipul has quit IRC | 12:41 | |
| *** zhenguo has quit IRC | 12:41 | |
| *** sballe_ has quit IRC | 12:41 | |
| *** efried has quit IRC | 12:41 | |
| *** melwitt has quit IRC | 12:41 | |
| *** rhagarty has quit IRC | 12:41 | |
| *** fmccrthy_ is now known as fmccrthy | 12:41 | |
| *** icey_ is now known as icey | 12:41 | |
| *** zhenguo_ is now known as zhenguo | 12:41 | |
| *** sballe__ is now known as sballe_ | 12:41 | |
| *** melwitt has joined #openstack-nova | 12:41 | |
| *** vipul has joined #openstack-nova | 12:41 | |
| *** cNilesh has joined #openstack-nova | 12:41 | |
| *** ujjain has joined #openstack-nova | 12:41 | |
| *** ujjain has quit IRC | 12:41 | |
| *** ujjain has joined #openstack-nova | 12:41 | |
| *** wasmum has joined #openstack-nova | 12:41 | |
| *** melwitt is now known as Guest86208 | 12:41 | |
| *** DinaBelova has joined #openstack-nova | 12:42 | |
| *** rfolco has joined #openstack-nova | 12:42 | |
| *** rhagarty has joined #openstack-nova | 12:42 | |
| *** yushb has quit IRC | 12:43 | |
| *** szaher has joined #openstack-nova | 12:44 | |
| *** dgonzalez has quit IRC | 12:44 | |
| alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | 12:45 |
| *** dgonzalez has joined #openstack-nova | 12:46 | |
| *** eharney has joined #openstack-nova | 12:47 | |
| *** liverpooler has joined #openstack-nova | 12:49 | |
| claudiub | moshele: hello. around? :) | 12:50 |
| *** edmondsw has joined #openstack-nova | 12:50 | |
| moshele | claudiub: hi | 12:51 |
| claudiub | moshele: great. :) If you have a couple of minutes, I have a couple of questions regarding sr-iov | 12:51 |
| moshele | claudiub: ok | 12:51 |
| claudiub | and more explicitly, some implementation details. | 12:51 |
| *** edmondsw_ has joined #openstack-nova | 12:51 | |
| *** edmondsw has quit IRC | 12:52 | |
| claudiub | moshele: so, whatever guides i've found online are kind of outdated, or do not contain all the information i need. :) | 12:52 |
| *** jaypipes has joined #openstack-nova | 12:53 | |
| claudiub | moshele: well, first of all: let's say I have a NIC with SR-IOV capabilities, and I have a couple of free VFs. I'm already reporting them to nova in the virt driver as PCI devices with the type type-VF. | 12:53 |
| moshele | claudiub: yes if they are in the whitelist | 12:54 |
| claudiub | and they're whitelisted in the compute node's conf and nova-api's conf | 12:54 |
| claudiub | up to this point, that should be fine IMO. | 12:54 |
| moshele | whitelisted in the compute conf not need in the nova api | 12:55 |
| claudiub | moshele: aliased, sorry. :) | 12:55 |
| claudiub | aliased in the api conf file | 12:55 |
| *** karimb has joined #openstack-nova | 12:55 | |
| moshele | the alise should be in nova-api and nova-cmpute | 12:55 |
| claudiub | oh, ok. i only knew about the alias in the nova-api. how does the alias help in the compute node? | 12:56 |
| *** xinliang has quit IRC | 12:56 | |
| claudiub | now... I was thinking that I could claim those VFs the same way I would claim other PCI devices: through flavor extra_specs pci_passthrough:alias="the_alias:1" | 12:56 |
| moshele | claudiub: for resize and migration | 12:56 |
| *** xinliang has joined #openstack-nova | 12:57 | |
| claudiub | moshele: hmm, meaning that the nova compute manager will check if the destination has those aliases as well? | 12:57 |
| moshele | claudiub: yes | 12:58 |
| claudiub | moshele: hm, interesting. i did not know that. :) good to know. | 12:58 |
| *** ralonsoh has quit IRC | 12:58 | |
| *** ralonsoh_ has joined #openstack-nova | 12:58 | |
| *** crushil has quit IRC | 12:59 | |
| *** sridharg has quit IRC | 12:59 | |
| *** gmann_ has joined #openstack-nova | 12:59 | |
| *** sridharg has joined #openstack-nova | 13:00 | |
| claudiub | anyways, continueing my idea. so, I would end up spawning an instance, and that instance would have in the instance.pci_requests.requests an VF. Now, I've been told before that all I have to do is just plug that VF into the VM and that's it | 13:00 |
| *** ralonsoh_ is now known as ralonsoh | 13:00 | |
| *** crushil has joined #openstack-nova | 13:01 | |
| claudiub | but that VF would not be tied to any neutron port, meaning that it there won't be normally any dhcp server serving it. I've been told that "the users would have to take care of that". is that ok? | 13:02 |
| claudiub | nor would it have security groups applied to it. | 13:02 |
| *** sridharg has quit IRC | 13:02 | |
| openstackgerrit | Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 13:07 |
| *** yamamoto has joined #openstack-nova | 13:08 | |
| *** mdrabe has joined #openstack-nova | 13:10 | |
| claudiub | moshele: also, the alias is quite limiting, IMO. for example, I have 2 identical SR-IOV NICs, which means same vendor id and same product id, but they're actually connected to different networks. the alias config option only allows the following fields: vendor_id, product_id, dev_type, name (alias). which means that I don't have any way to specify which network I actually want. :) | 13:10 |
| moshele | claudiub: see this https://docs.openstack.org/mitaka/networking-guide/config-sriov.html | 13:10 |
| *** zzzeek_ has quit IRC | 13:12 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473 | 13:12 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Enable pep8 on doc generation code https://review.openstack.org/480469 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead plugin https://review.openstack.org/478469 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove cruft from conf.py https://review.openstack.org/478468 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Use consistent author, section for man pages https://review.openstack.org/482960 | 13:13 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 13:13 |
| *** zzzeek_ has joined #openstack-nova | 13:13 | |
| *** yamamoto has quit IRC | 13:13 | |
| moshele | claudiub: there is SR-IOV for networking solution is work with the sriovnicswitch mechanism driver which is linux ip link based so maybe you need to add support there to windows | 13:13 |
| *** zhurong has joined #openstack-nova | 13:14 | |
| *** efried_ is now known as efried | 13:14 | |
| cdent | avolkov`++ | 13:14 |
| moshele | claudiub: what you talking is pci passthrough when neutron is not aware of the SR-IOV network interface | 13:14 |
| stephenfin | jaypipes: IF you get a chance to look through that doc-migration series again, I'd be mighty appreciative. Fixed the issue efried pointed out | 13:14 |
| stephenfin | Sorry - gibi | 13:15 |
| jaypipes | stephenfin: you mean the one you just rebased and pushed above? | 13:15 |
| *** cNilesh has quit IRC | 13:15 | |
| claudiub | moshele: are both cases viable? | 13:15 |
| *** kylek3h has joined #openstack-nova | 13:15 | |
| *** mdnadeem has quit IRC | 13:16 | |
| stephenfin | jaypipes: Yup https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova+status:open | 13:16 |
| jaypipes | stephenfin: sure thing :) | 13:16 |
| efried | stephenfin So that wasn't intended to be to my attn? | 13:17 |
| moshele | claudiub: yes both of them should work | 13:17 |
| stephenfin | efried: Nope, as always you can ignore me :) | 13:17 |
| *** slaweq_ has joined #openstack-nova | 13:17 | |
| claudiub | moshele: aham. interesting. well, i'll look into the neutron side as well then. :) thanks for the help. :) | 13:18 |
| moshele | claudiub: np :) | 13:18 |
| *** moshele has quit IRC | 13:21 | |
| bauzas | jaypipes: dansmith: sorry folks, was (again) on PTO yesterday | 13:21 |
| *** trinaths has left #openstack-nova | 13:21 | |
| avolkov` | stephenfin: placement-api-ref should also be moved to doc/source/api, I'll prepare patch when it's accomplished | 13:22 |
| *** slaweq_ has quit IRC | 13:22 | |
| stephenfin | avolkov`: I thought we weren't moving that yet? | 13:22 |
| stephenfin | Did plans change? | 13:22 |
| cdent | stephenfin: dhellman wants that to happen asap | 13:22 |
| cdent | so that we don't have to have a situation where there are publishing jobs that need to get changed | 13:22 |
| cdent | (there's a placement-api-ref publishing job pending) | 13:23 |
| stephenfin | cdent: Ah, hadn't heard that update | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'configuration' section https://review.openstack.org/478473 | 13:23 |
| stephenfin | Sounds fair | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'cli' section https://review.openstack.org/478472 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'user' section https://review.openstack.org/478475 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'contributor' section https://review.openstack.org/478474 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Enable pep8 on doc generation code https://review.openstack.org/480469 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Populate the 'reference' section https://review.openstack.org/478484 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | 13:23 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 13:23 |
| jaypipes | efried: stephenfin just wanted to wake you out of your slumber. | 13:23 |
| * stephenfin accidentally squashed two patches. Sorry, gate | 13:23 | |
| *** rushil has quit IRC | 13:24 | |
| jaypipes | ralonsoh: I'm really liking the direction of your native OVSDB os-vif work. | 13:24 |
| *** baoli has joined #openstack-nova | 13:28 | |
| stephenfin | bauzas: Could I ask you to look at the doc-migration series starting with https://review.openstack.org/#/c/478468/ too? | 13:29 |
| *** zzzeek_ has quit IRC | 13:29 | |
| bauzas | stephenfin: roger this. | 13:29 |
| stephenfin | Shouldn't take long, and it's easy re-arrange stuff later if we don't like the end result | 13:29 |
| stephenfin | bauzas: Ta :) | 13:29 |
| ralonsoh | jaypipes: thanks! Also I added functional tests. YOu only need to have a live OVS running in the system to execute them | 13:30 |
| jaypipes | ralonsoh: I saw those. ++ | 13:30 |
| *** fragatina has joined #openstack-nova | 13:31 | |
| *** frickler has quit IRC | 13:33 | |
| *** crushil has quit IRC | 13:34 | |
| *** zzzeek_ has joined #openstack-nova | 13:34 | |
| *** crushil has joined #openstack-nova | 13:35 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Always set MTU when plugging devices https://review.openstack.org/408781 | 13:36 |
| *** tbachman has quit IRC | 13:36 | |
| *** fragatina has quit IRC | 13:36 | |
| *** armax has joined #openstack-nova | 13:39 | |
| gibi | stephenfin: thanks for the rebase | 13:41 |
| *** frickler has joined #openstack-nova | 13:41 | |
| *** gmann_ has quit IRC | 13:42 | |
| openstackgerrit | Chason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme https://review.openstack.org/477757 | 13:46 |
| *** mriedem has joined #openstack-nova | 13:46 | |
| mriedem | o/ | 13:48 |
| jaypipes | cdent: thx for your comments on https://review.openstack.org/#/c/482383/1. Answered. Please have a gander at the latest revision. | 13:49 |
| *** yamamoto has joined #openstack-nova | 13:51 | |
| gibi | mriedem: good morning! I fixed the last outstanding issue with the BDM in notification patch https://review.openstack.org/#/c/448779/ | 13:51 |
| mriedem | jaypipes: should i just remove this sentence? https://review.openstack.org/#/c/454322/35/nova/api/openstack/compute/rest_api_version_history.rst@598 | 13:51 |
| mriedem | jaypipes: i think that would probably clear up the confusion in that paragraph | 13:51 |
| jaypipes | mriedem: yeah | 13:52 |
| jaypipes | mriedem: I'd appreciate that. sorry for not yet responding to you. | 13:52 |
| *** kristia__ has quit IRC | 13:52 | |
| mriedem | gibi: cool | 13:53 |
| mriedem | jaypipes: np | 13:53 |
| mriedem | gibi: i'll go through that today | 13:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 13:54 |
| gibi | mriedem: thanks! | 13:55 |
| *** xyang1 has joined #openstack-nova | 13:56 | |
| *** catintheroof has joined #openstack-nova | 13:57 | |
| *** frickler has quit IRC | 13:57 | |
| *** ZZelle has quit IRC | 13:57 | |
| *** frickler has joined #openstack-nova | 14:03 | |
| *** vladikr has joined #openstack-nova | 14:03 | |
| *** gcb has joined #openstack-nova | 14:04 | |
| *** kristian__ has joined #openstack-nova | 14:04 | |
| *** awaugama has joined #openstack-nova | 14:05 | |
| *** hongbin has joined #openstack-nova | 14:06 | |
| *** jangutter has quit IRC | 14:06 | |
| *** tylerderosagrund has joined #openstack-nova | 14:08 | |
| *** jmlowe has joined #openstack-nova | 14:09 | |
| *** kristian__ has quit IRC | 14:11 | |
| *** kristian__ has joined #openstack-nova | 14:12 | |
| *** ekuris_ has quit IRC | 14:13 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/477757 | 14:13 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: doc: Create directory structure for docs migration https://review.openstack.org/480135 | 14:13 |
| openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Remove custom autodoc implementation https://review.openstack.org/480262 | 14:13 |
| mriedem | alex_xu: i understand your comments on the os-services change now; fwiw i care less about the wasted db call - this api won't be used often and i'd rather keep the complexity in here to a minimum | 14:16 |
| *** coreywright has quit IRC | 14:16 | |
| *** acormier has joined #openstack-nova | 14:16 | |
| mriedem | but i can investigate it, and add validation for the other error cases you pointed out, | 14:17 |
| mriedem | like passing status=enabled and disabled_reason | 14:17 |
| *** tbachman has joined #openstack-nova | 14:18 | |
| *** slaweq_ has joined #openstack-nova | 14:18 | |
| *** catintheroof has quit IRC | 14:20 | |
| *** tjones has joined #openstack-nova | 14:20 | |
| *** slaweq_ has quit IRC | 14:23 | |
| *** zzzeek has quit IRC | 14:24 | |
| *** gbarros has joined #openstack-nova | 14:25 | |
| *** moshele has joined #openstack-nova | 14:27 | |
| *** tbachman has quit IRC | 14:27 | |
| *** tbachman has joined #openstack-nova | 14:31 | |
| *** jangutter has joined #openstack-nova | 14:34 | |
| *** gouthamr has joined #openstack-nova | 14:35 | |
| *** itlinux has joined #openstack-nova | 14:37 | |
| *** coreywright has joined #openstack-nova | 14:37 | |
| *** crushil_ has joined #openstack-nova | 14:38 | |
| *** udesale has quit IRC | 14:38 | |
| *** jdillaman has quit IRC | 14:38 | |
| mriedem | gibi: if you'll be around for 5 minutes i'll have something for you | 14:39 |
| mriedem | gibi: https://review.openstack.org/#/c/448779/ | 14:40 |
| mriedem | questions in there about why we aren't using the existing in scope BDMs when building the payload, like during instance.create | 14:40 |
| mriedem | seems that could be optimized to use the bdms we already have when we notify, rather than always look them up from the db - but maybe that's a later optimization | 14:40 |
| gibi | mriedem: looking... | 14:42 |
| *** mkucia has joined #openstack-nova | 14:43 | |
| gibi | mriedem: I think we can do some optimization on the loading of BDM. But I feel that it would be better to do that separately | 14:44 |
| mriedem | heh, of course you would :) | 14:45 |
| mriedem | also https://review.openstack.org/#/c/448779/38/nova/tests/fixtures.py | 14:45 |
| mriedem | gibi: it's not even 5pm yet! :) | 14:45 |
| stephenfin | sean-k-mooney, moshele, jaypipes: What on earth is IVS, in the context of network interfaces? | 14:45 |
| gibi | mriedem: true :) | 14:46 |
| jaypipes | stephenfin: it's Indigo (plumbgrid) stuff | 14:46 |
| jaypipes | plumgrid | 14:46 |
| mriedem | gibi: fwiw i'm ok with deferring the optimization part since loading bdms is disabled by default | 14:46 |
| mriedem | but i wanted to make sure we discussed it first | 14:46 |
| jaypipes | stephenfin: NIH OVS for NFV? :) | 14:47 |
| gibi | mriedem: let's see if I can make the optimization this week then we can squash the two patches but if I run out of time then it would be nice to merge at least what we have already | 14:47 |
| stephenfin | jaypipes: So not an "initialization vector"? :) It's rather un-Googleable | 14:47 |
| mriedem | gibi: how about you put a TODO in the code in the InstanceAction init where it calls BlockDevicePayload.from_instance to investigate optimizing that by passing in any existing in-scope BDMs, rather than pull from the db | 14:47 |
| jaypipes | stephenfin: no. I mean, there's the ivshmem thing but that's DPDK-related. | 14:47 |
| mriedem | gibi: i'll +2 with the todo | 14:48 |
| mriedem | check out my question in the cinder fixture also | 14:48 |
| jaypipes | stephenfin: depends on the context I guess. | 14:48 |
| gibi | mriedem: sure, I can do that til 5 pm :) | 14:48 |
| mriedem | \o/ | 14:48 |
| * stephenfin is back to figuring out where the nova os-vif boundary lies in nova/network/*.py files, and failing yet again | 14:48 | |
| stephenfin | jaypipes: Cool cool. TA | 14:48 |
| stephenfin | *Ta | 14:48 |
| *** itlinux has quit IRC | 14:49 | |
| *** nicolasbock has quit IRC | 14:50 | |
| jaypipes | mriedem, dansmith, edleafe, bauzas: so do you think we should be returning alternate *destination hosts* or do you think we should be returning alternate *allocation requests*? I'm questioning my thoughts here. :( | 14:50 |
| dansmith | jaypipes: I thought the whole point was to store alternate allocation requests | 14:50 |
| dansmith | jaypipes: otherwise cell conductor has to formulate such a thing, without the details the scheduler has, and I don't think that's going to work | 14:51 |
| dansmith | s/store/return/ | 14:51 |
| dansmith | so conductor can almost blindly throw the allocation request at placement, and if it sticks, send it along | 14:51 |
| cdent | I agree with dansmith. I seem to recall that's why we ended up down the allocation_candidates path in the first place? | 14:51 |
| dansmith | one of yeah | 14:51 |
| jaypipes | dansmith: yeah, that was my initial thought. I guess I've gotten bent out of shape on this latest patch series. :( | 14:52 |
| jaypipes | I think I need a braincation. | 14:52 |
| *** zhurong has quit IRC | 14:53 | |
| cdent | dansmith: since you're here, I was reading some cells docs and code: Is the main reason that discover_hosts doesn't do map_instances so that you can have granularity of timing and volume of change? Are there other reasons? | 14:53 |
| dansmith | cdent: we only needed map_instances to bridge between like mitaka and newton | 14:53 |
| dansmith | cdent: now instance maps are created when we create instances | 14:54 |
| *** psachin has quit IRC | 14:54 | |
| dansmith | cdent: and we would need it if we were grafting a cell from one deployment onto another or something like that, so I don't know that we should remove it necessarily | 14:54 |
| cdent | ah, right, I see. thanks. | 14:54 |
| cdent | yeah, keeping it around seems a handy thing | 14:55 |
| *** dtantsur is now known as dtantsur|afk | 14:55 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 14:56 |
| *** crushil has quit IRC | 14:56 | |
| *** vvargaszte has quit IRC | 14:57 | |
| *** crushil_ is now known as crushil | 14:57 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix api-ref for instance action events's attribute as optional https://review.openstack.org/482994 | 14:58 |
| *** david-lyle has quit IRC | 14:58 | |
| *** david-lyle has joined #openstack-nova | 14:58 | |
| moshele | jaypipes: hi can you send an invite to my gmaiil moshe010@gmail.com ? | 14:58 |
| ralonsoh | jaypipes: and rodolfo.ah@gmail.com | 14:58 |
| bauzas | jaypipes: allocation requests looks good to me | 14:59 |
| *** prateek has quit IRC | 14:59 | |
| bauzas | jaypipes: but I just think we need to make sure those allocation requests would be for RPs in the same cell, nope ? | 14:59 |
| jaypipes | moshele, ralonsoh: done | 14:59 |
| dansmith | bauzas: yes, and he noted that | 15:00 |
| jaypipes | bauzas: yes, I agree with that. but I need to rework these patches. I got off track returning alternate host states instead of a dict of allocation requests. | 15:00 |
| *** marst has joined #openstack-nova | 15:00 | |
| bauzas | jaypipes: btw. I just left a question in https://review.openstack.org/#/c/482382/3 | 15:00 |
| bauzas | jaypipes: about why we would not use the current existing method | 15:00 |
| jangutter | jaypipes: If my work hangouts doesn't work, please yell. I can also do personal. | 15:01 |
| bauzas | jaypipes: dansmith: btw. sorry about me not reviewing the patches previously, I was (again) in PTO | 15:01 |
| dansmith | bauzas: I just assume you're on PTO unless you say otherwise | 15:01 |
| *** rcernin has quit IRC | 15:01 | |
| bauzas | .... and Friday is an holiday day :( | 15:01 |
| bauzas | so terrible | 15:02 |
| bauzas | I should be done with all my PTOs for last year FWIW | 15:02 |
| *** zzzeek_ has quit IRC | 15:02 | |
| bauzas | I sincerely apologize for that | 15:02 |
| *** catintheroof has joined #openstack-nova | 15:03 | |
| *** zzzeek has joined #openstack-nova | 15:04 | |
| gibi | mriedem: added the TODO and fixed / answered the fixture comments in https://review.openstack.org/#/c/448779 | 15:05 |
| ftersin | mdbooth: hi, thank you for reviewing. If you have the opportunity, pls, look at my comments there, i just asked some details for the reworking | 15:06 |
| *** burt has joined #openstack-nova | 15:06 | |
| mriedem | gibi: is the indent intentional here? https://review.openstack.org/#/c/448779/39/nova/tests/fixtures.py@1270 | 15:07 |
| *** felipemonteiro has joined #openstack-nova | 15:07 | |
| gibi | mriedem: yes, I tried to explain the reason of that in my response to your comment | 15:08 |
| *** felipemonteiro_ has joined #openstack-nova | 15:08 | |
| gibi | mriedem: either need the explicity detach in the test or I break the swap volume tests. | 15:08 |
| mriedem | ok just checking | 15:09 |
| gibi | mriedem: will propose a clean up on this fixture at some point | 15:09 |
| *** nicolasbock has joined #openstack-nova | 15:10 | |
| mriedem | ok, +2 | 15:10 |
| mriedem | i left a comment for other reviewers too | 15:10 |
| gibi | mriedem: thanks | 15:10 |
| mriedem | a todo or note in there would also be good, | 15:10 |
| mriedem | but only if you respin | 15:10 |
| gibi | mriedem: OK | 15:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 15:11 |
| *** roeyc has quit IRC | 15:12 | |
| *** felipemonteiro has quit IRC | 15:12 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 15:12 |
| *** karimb has quit IRC | 15:12 | |
| bauzas | gibi: https://review.openstack.org/#/c/482497/1 is needed for what ? | 15:12 |
| bauzas | just a clean-up right? | 15:13 |
| gibi | bauzas: splited out from the BDM patch on top of it | 15:13 |
| gibi | bauzas: it basically optimize out some data generation if the notification will not be sent out at the end | 15:13 |
| bauzas | gibi: oh nevermind, it was just something missed from https://review.openstack.org/#/c/428260/ ? | 15:14 |
| gibi | bauzas: yes, from that and from other patches later on | 15:14 |
| bauzas | gibi: all good | 15:15 |
| gibi | bauzas: thanks for the review | 15:15 |
| *** yassine has quit IRC | 15:16 | |
| *** slaweq_ has joined #openstack-nova | 15:19 | |
| *** fragatina has joined #openstack-nova | 15:19 | |
| *** gbarros has quit IRC | 15:19 | |
| ildikov | gibi: that fixture is a mess :( | 15:20 |
| *** fragatina has quit IRC | 15:20 | |
| *** prateek has joined #openstack-nova | 15:20 | |
| *** fragatina has joined #openstack-nova | 15:20 | |
| *** gbarros has joined #openstack-nova | 15:21 | |
| *** tbachman has quit IRC | 15:21 | |
| gibi | ildikov: I have my sticker to clean up it, but first I will optimize out some db query related to the instance.create | 15:22 |
| *** chyka has joined #openstack-nova | 15:22 | |
| *** lyan has joined #openstack-nova | 15:22 | |
| gibi | ildikov: or do you have some time to work on it? ;] | 15:23 |
| ildikov | gibi: I have a patch touching it with a note to figure something out there :) | 15:23 |
| ildikov | gibi: well, let's see who gets there first ;) | 15:23 |
| gibi | ildikov: well, that will be a real fight for that juicy commit | 15:24 |
| *** slaweq_ has quit IRC | 15:24 | |
| ildikov | gibi: lol :) | 15:24 |
| *** alexchadin has quit IRC | 15:25 | |
| mriedem | gmann: i'm not sure i agree with this https://review.openstack.org/#/c/482994/ | 15:26 |
| mriedem | it gets really confusing which is why i left it out of my patch | 15:26 |
| *** chyka_ has joined #openstack-nova | 15:26 | |
| mriedem | to mark 'events' as optional | 15:26 |
| *** Oku_OS is now known as Oku_OS-away | 15:28 | |
| *** gszasz has quit IRC | 15:28 | |
| *** chyka has quit IRC | 15:29 | |
| *** tbachman has joined #openstack-nova | 15:29 | |
| bauzas | gibi: may I ask a silly question about https://review.openstack.org/#/c/448779/39 ? | 15:30 |
| *** belmoreira has quit IRC | 15:31 | |
| bauzas | gibi: why some notifications samples provide an empty list for bdm notifications objects while others provide it? | 15:31 |
| gibi | bauzas: sure | 15:31 |
| gibi | bauzas: in some cases the bdm is overridden during the test | 15:32 |
| gibi | bauzas: like test_volume_swap_server_with_error | 15:32 |
| *** READ10 has joined #openstack-nova | 15:32 | |
| gibi | bauzas: or test_create_delete_server_with_instance_update | 15:33 |
| *** mdrabe has quit IRC | 15:33 | |
| gibi | bauzas: in test_create_delete_server_with_instance_update the bdm appears only after the instance goes through the block device mapping step during the create | 15:33 |
| *** tylerderosagrund has quit IRC | 15:34 | |
| gibi | bauzas: so the early instance-updates has no bdm but at that point it starts appearing | 15:34 |
| gibi | bauzas: and I don't want to store different sample files for each steps | 15:34 |
| *** edmondsw_ is now known as edmondsw | 15:34 | |
| *** Sukhdev has joined #openstack-nova | 15:34 | |
| *** yamahata has joined #openstack-nova | 15:35 | |
| *** gszasz has joined #openstack-nova | 15:36 | |
| *** andreas_s has quit IRC | 15:38 | |
| *** mdrabe has joined #openstack-nova | 15:38 | |
| *** mvk has quit IRC | 15:40 | |
| *** cdent has quit IRC | 15:40 | |
| gibi | bauzas: specially in the instance.create case we only attach the volume after the boot not during it | 15:40 |
| *** kaisers_ has joined #openstack-nova | 15:41 | |
| *** liusheng has quit IRC | 15:42 | |
| mriedem | Vek: can you take a look at this? https://review.openstack.org/#/c/480392/ | 15:42 |
| mriedem | or andreykurilin ^ | 15:42 |
| gibi | bauzas: disregard my statement about test_create_delete_server_with_instance_update | 15:43 |
| gibi | bauzas: in that case the bdm is empty as there the volume also attached after the create | 15:43 |
| *** liusheng has joined #openstack-nova | 15:43 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 15:44 |
| andreykurilin | looking | 15:45 |
| *** fragatina has quit IRC | 15:47 | |
| *** crushil has quit IRC | 15:47 | |
| *** sshwarts has quit IRC | 15:47 | |
| *** yamamoto has quit IRC | 15:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: mark instance action events parameter as optional https://review.openstack.org/480792 | 15:49 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | 15:49 |
| *** yamamoto has joined #openstack-nova | 15:49 | |
| *** crushil has joined #openstack-nova | 15:49 | |
| *** yamamoto has quit IRC | 15:51 | |
| *** lucasagomes is now known as lucas-afk | 15:52 | |
| *** damien_r has quit IRC | 15:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | 15:53 |
| *** jmlowe has quit IRC | 15:54 | |
| *** itlinux has joined #openstack-nova | 15:54 | |
| *** amotoki is now known as amotoki_away | 15:56 | |
| *** rajathagasthya has joined #openstack-nova | 15:56 | |
| *** cdent has joined #openstack-nova | 15:57 | |
| *** amotoki_away is now known as amotoki | 15:59 | |
| *** tbachman has quit IRC | 15:59 | |
| *** amotoki is now known as amotoki_away | 15:59 | |
| *** jamesden_ has quit IRC | 15:59 | |
| bauzas | gibi: okay, that's reasonable, those notifications mutated ? | 16:00 |
| gibi | bauzas: the sample is mutated during the test | 16:01 |
| *** jamesdenton has joined #openstack-nova | 16:01 | |
| gibi | bauzas: but I'm not sure this is what you are after | 16:02 |
| mriedem | sean-k-mooney: ralonsoh: i wouldn't block on that out of band port update thingy, as i said, but i am not really an authority on that entire series of changes either as i haven't gone into it in detail, | 16:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 16:03 |
| mriedem | and my understanding of a lot of the pci stuff in nova is very limited | 16:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 16:03 |
| bauzas | gibi: it was just for my understanding | 16:03 |
| bauzas | gibi: I have all I need, I was just in a short meeting beforehand | 16:03 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 16:03 |
| ralonsoh | mriedem: thanks for your comments! | 16:04 |
| *** esberglu has quit IRC | 16:04 | |
| gibi | bauzas: OK. I will leave soon for today but just ping me tomorrow if you have questions | 16:04 |
| bauzas | gibi: that's not necessary :) | 16:04 |
| *** tbachman has joined #openstack-nova | 16:05 | |
| *** tesseract has quit IRC | 16:05 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 16:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 16:05 |
| gibi | bauzas: there are patches up to help keeping the style of the sample files consistent. I left the links under your comment | 16:07 |
| bauzas | ++ | 16:08 |
| bauzas | not really super important, but for review time, it helps | 16:08 |
| *** gcb has quit IRC | 16:09 | |
| gibi | bauzas: I agree | 16:10 |
| *** gszasz has quit IRC | 16:11 | |
| *** crushil has quit IRC | 16:11 | |
| *** dtp has joined #openstack-nova | 16:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 16:14 |
| *** annegentle has joined #openstack-nova | 16:14 | |
| *** roeyc has joined #openstack-nova | 16:15 | |
| *** esberglu has joined #openstack-nova | 16:18 | |
| *** damien_r has joined #openstack-nova | 16:18 | |
| *** damien_r1 has joined #openstack-nova | 16:19 | |
| *** jjung has joined #openstack-nova | 16:19 | |
| *** slaweq_ has joined #openstack-nova | 16:21 | |
| *** fragatina has joined #openstack-nova | 16:21 | |
| *** esberglu has quit IRC | 16:22 | |
| *** gszasz has joined #openstack-nova | 16:22 | |
| *** damien_r has quit IRC | 16:22 | |
| *** jjung has quit IRC | 16:24 | |
| *** slaweq_ has quit IRC | 16:25 | |
| *** esberglu has joined #openstack-nova | 16:27 | |
| mriedem | jaypipes: given sdague was +2 on https://review.openstack.org/#/c/454322/ and is out for the rest of this week, if you're cool with it, want to just fast approve that one? | 16:28 |
| jaypipes | mriedem: done | 16:28 |
| mriedem | thanks | 16:28 |
| *** kristian__ has quit IRC | 16:29 | |
| *** trinaths has joined #openstack-nova | 16:30 | |
| *** catintheroof has quit IRC | 16:30 | |
| *** catintheroof has joined #openstack-nova | 16:31 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 16:32 |
| *** ralonsoh has quit IRC | 16:32 | |
| *** annegentle has quit IRC | 16:33 | |
| *** annegentle has joined #openstack-nova | 16:34 | |
| *** damien_r1 has quit IRC | 16:34 | |
| *** damien_r has joined #openstack-nova | 16:35 | |
| *** lpetrut has quit IRC | 16:35 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove plug_ovs_hybrid, unplug_ovs_hybrid https://review.openstack.org/483030 | 16:36 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Remove useless test https://review.openstack.org/483031 | 16:36 |
| *** Guest86208 is now known as melwitt | 16:36 | |
| edleafe | 1/away | 16:38 |
| edleafe | geez | 16:38 |
| *** mdrabe has quit IRC | 16:39 | |
| *** mdrabe_ has joined #openstack-nova | 16:39 | |
| *** Sukhdev has quit IRC | 16:41 | |
| *** mvk has joined #openstack-nova | 16:42 | |
| bauzas | edleafe: 2/there | 16:43 |
| edleafe | bauzas: :) | 16:43 |
| * edleafe has fat fingers | 16:43 | |
| *** claudiub has quit IRC | 16:43 | |
| *** fragatina has quit IRC | 16:43 | |
| *** fragatina has joined #openstack-nova | 16:44 | |
| *** mdrabe_ has quit IRC | 16:46 | |
| *** mdrabe_ has joined #openstack-nova | 16:46 | |
| *** damien_r has quit IRC | 16:47 | |
| *** damien_r has joined #openstack-nova | 16:47 | |
| *** jamesden_ has joined #openstack-nova | 16:48 | |
| *** gjayavelu has joined #openstack-nova | 16:50 | |
| *** jamesdenton has quit IRC | 16:50 | |
| *** roeyc has quit IRC | 16:51 | |
| *** damien_r has quit IRC | 16:51 | |
| *** yamamoto has joined #openstack-nova | 16:51 | |
| *** pcaruana has quit IRC | 16:56 | |
| *** moshele has quit IRC | 16:56 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add claim_resources() to report client https://review.openstack.org/482382 | 16:57 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: claim resources in filter scheduler https://review.openstack.org/482383 | 16:57 |
| jaypipes | edleafe, mriedem, bauzas, dansmith: ^^ squashed the "alternate destinations" patch into the claim resources in filter scheduler patch and re-ordered the patches so the add claim_resources() to report client can be reviewed ahead of the claim in scheduler patch. | 16:58 |
| jaypipes | still need to handle the cells thing. | 16:58 |
| *** jmlowe has joined #openstack-nova | 16:59 | |
| *** kristian__ has joined #openstack-nova | 16:59 | |
| *** fragatin_ has joined #openstack-nova | 17:01 | |
| mriedem | ok | 17:01 |
| *** yamamoto has quit IRC | 17:02 | |
| *** fragatina has quit IRC | 17:04 | |
| *** gbarros has quit IRC | 17:08 | |
| *** eharney has quit IRC | 17:12 | |
| *** eharney has joined #openstack-nova | 17:13 | |
| mriedem | gmann: https://review.openstack.org/#/c/467999/ needs some work | 17:17 |
| mriedem | it's merged but the docs are lacking | 17:17 |
| *** weshay is now known as weshay|lunch | 17:20 | |
| *** weshay|lunch is now known as weshay | 17:20 | |
| edleafe | jaypipes: regarding filtering alternates by cell, feel free to take as much of this as works: https://review.openstack.org/#/c/465768/8/nova/scheduler/filter_scheduler.py@181 | 17:21 |
| *** slaweq_ has joined #openstack-nova | 17:21 | |
| *** tylerderosagrund has joined #openstack-nova | 17:21 | |
| *** claudiub has joined #openstack-nova | 17:22 | |
| *** lpetrut has joined #openstack-nova | 17:22 | |
| *** moshele has joined #openstack-nova | 17:23 | |
| *** Apoorva has joined #openstack-nova | 17:25 | |
| *** yamahata has quit IRC | 17:25 | |
| *** tylerderosagrund has quit IRC | 17:25 | |
| *** slaweq_ has quit IRC | 17:26 | |
| openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392 | 17:27 |
| *** jmlowe has quit IRC | 17:30 | |
| *** trinaths has left #openstack-nova | 17:31 | |
| *** kristian__ has quit IRC | 17:32 | |
| *** sambetts is now known as sambetts|afk | 17:32 | |
| *** tylerderosagrund has joined #openstack-nova | 17:43 | |
| *** rajathagasthya has quit IRC | 17:45 | |
| *** rajathagasthya has joined #openstack-nova | 17:46 | |
| *** gbarros has joined #openstack-nova | 17:46 | |
| *** fragatin_ has quit IRC | 17:47 | |
| *** jjung has joined #openstack-nova | 17:47 | |
| *** annegentle has quit IRC | 17:47 | |
| *** fragatina has joined #openstack-nova | 17:47 | |
| *** mkucia has quit IRC | 17:48 | |
| *** lpetrut has quit IRC | 17:49 | |
| *** gyee has joined #openstack-nova | 17:53 | |
| *** yamahata has joined #openstack-nova | 17:54 | |
| *** hoonetorg has quit IRC | 17:54 | |
| *** moshele has quit IRC | 17:56 | |
| *** jamesden_ has quit IRC | 18:00 | |
| *** jamesdenton has joined #openstack-nova | 18:01 | |
| *** yamamoto has joined #openstack-nova | 18:04 | |
| *** annegentle has joined #openstack-nova | 18:08 | |
| *** hoonetorg has joined #openstack-nova | 18:10 | |
| *** lpetrut has joined #openstack-nova | 18:10 | |
| *** READ10 has quit IRC | 18:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage https://review.openstack.org/483060 | 18:16 |
| *** yamamoto has quit IRC | 18:20 | |
| *** yamamoto has joined #openstack-nova | 18:21 | |
| *** gbarros has quit IRC | 18:22 | |
| *** slaweq_ has joined #openstack-nova | 18:22 | |
| *** gbarros has joined #openstack-nova | 18:22 | |
| *** yamamoto has quit IRC | 18:26 | |
| *** slaweq_ has quit IRC | 18:27 | |
| *** gbarros has quit IRC | 18:27 | |
| *** MVenesio has quit IRC | 18:32 | |
| cdent | edleafe: filtering rps by traits: https://review.openstack.org/#/c/474602/ still alive? | 18:36 |
| gheorghe_ | is there something wrong with migrations on ocata? my VMs are on the same host after migration...always... | 18:39 |
| mriedem | gheorghe_: are there errors in the logs? | 18:40 |
| mriedem | are you doing cold migration, live migration, or resize? | 18:40 |
| *** efoley has quit IRC | 18:40 | |
| mriedem | there are no known issues, upstream integration test for migrations is fine | 18:40 |
| *** lpetrut has quit IRC | 18:41 | |
| rm_work | Hey, I'm about to look into making keypairs be project-level scope instead of user+project | 18:43 |
| rm_work | Just wondering if there is some easy way to just enable this | 18:43 |
| rm_work | or if someone has already done this before | 18:43 |
| rm_work | (not necessarily to push upstream, we just need this in our cloud) | 18:43 |
| rm_work | ^^ before anyone freaks out | 18:44 |
| mriedem | rm_work: ask melwitt - quotas could be a problem | 18:44 |
| rm_work | we currently have quotas as -1 for keypairs | 18:44 |
| rm_work | so i'm not especially concerned | 18:44 |
| rm_work | but i'll take that note down | 18:44 |
| *** tylerderosagrund has quit IRC | 18:44 | |
| *** tylerderosagrund has joined #openstack-nova | 18:45 | |
| gheorghe_ | mriedem i am doing live migration, with block and without block (i also have some VMs on iscsi target) and in both cases the migration is not reporting any errors, but the VM stays on the same host | 18:45 |
| *** Sukhdev has joined #openstack-nova | 18:45 | |
| mriedem | gheorghe_: then something is failing | 18:45 |
| rm_work | melwitt: have you worked on keypairs per project? | 18:45 |
| mriedem | and it's either rolling back to the source host, or it's never even getting to the destination host | 18:45 |
| melwitt | rm_work: what do you mean by make it project scope instead of project+user? do you mean set a limit across a project and count allocated keypairs only across a project? | 18:46 |
| gheorghe_ | mriedem, i don't see anything in /var/log/libvirt/qemu# so i suspect something is failing on nova level | 18:46 |
| rm_work | melwitt: I mean, if user A on project X uploads a keypair, we want user B on project X to also see that keypair in that project | 18:47 |
| rm_work | and be able to use it | 18:47 |
| melwitt | oh, I see | 18:47 |
| rm_work | in most other clouds, this is the way key auth works | 18:48 |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| rm_work | and users are incredibly confused when they set up keypairs and then their service account or whatever goes to create a VM and errors out because the keypair isn't there (but THEY see it) | 18:48 |
| rm_work | and rather than try to educate/correct an ongoing set of users who have experience with other clouds or make an expectation based on how *everything else* works, we'd rather just ... enable the functionality they expect, because I can't see a good reason not to :P | 18:49 |
| rm_work | it's just public keys... | 18:49 |
| *** jmlowe has joined #openstack-nova | 18:49 | |
| rm_work | my public key is posted on my website... | 18:49 |
| mriedem | gheorghe_: sure, and that should be in the /var/log/nova logs | 18:50 |
| *** Swami has joined #openstack-nova | 18:50 | |
| rm_work | melwitt: have you worked on anything like this? | 18:50 |
| melwitt | rm_work: yeah, use case makes sense. I haven't heard of anyone doing it before though. I'm not sure what it would take, whether it would be hard. AFAIK it can't just be easily enabled | 18:50 |
| rm_work | kk | 18:50 |
| gheorghe_ | mriedem i get only info messages within /var/log/nova ... if you want, i can try i migration now and give you the nova logs. do you want to see the ones from controller or the ones from kvm? | 18:50 |
| *** rajathagasthya has quit IRC | 18:50 | |
| mriedem | gheorghe_: i don't want to debug your logs no | 18:51 |
| edleafe | cdent: yes, the filtering by traits is still alive. It can also be adapted for filtering the new allocation candidates method. | 18:51 |
| mriedem | gheorghe_: run it with debug=True in the nova.conf on all nodes, computes and controller | 18:51 |
| mriedem | and then i'd trace the request id through the nova-api, conductor and compute logs | 18:52 |
| gheorghe_ | request id returns only an info message | 18:52 |
| mriedem | the request id is logged with all messages for the given request context | 18:52 |
| mriedem | so if you have debug logging enabled, you should be able to see it there also | 18:52 |
| mriedem | e.g. http://logs.openstack.org/90/481290/3/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/fcc4b2f/logs/screen-n-cpu.txt.gz#_Jul_08_18_07_57_384617 | 18:53 |
| gheorghe_ | omg mariedem i actually found something | 18:53 |
| mriedem | req-df8aac96-7d4f-4652-9237-d6da65164ee5 is the request id | 18:53 |
| gheorghe_ | https://paste.gnome.org/pg7hkufiz | 18:54 |
| gheorghe_ | i will troubleshoot this, brb | 18:54 |
| *** raunak has joined #openstack-nova | 18:54 | |
| mriedem | yup that would make it fail | 18:54 |
| *** mdrabe_ has quit IRC | 18:54 | |
| *** gbarros has joined #openstack-nova | 18:55 | |
| *** mdrabe has joined #openstack-nova | 18:55 | |
| gheorghe_ | kvm-04 is not even listening on 16509 | 18:56 |
| *** annegentle has quit IRC | 18:57 | |
| rm_work | melwitt: ah, I was wrong about keypairs already being associated with a project | 18:59 |
| rm_work | they are purely user associated at the moment :( | 18:59 |
| gheorghe_ | mriedem does it work only with insecure conn? https://paste.gnome.org/pfdfgbkaj | 18:59 |
| rm_work | so yeah pretty big change I think T_T | 18:59 |
| *** baoli has quit IRC | 18:59 | |
| *** baoli has joined #openstack-nova | 19:00 | |
| *** baoli has quit IRC | 19:00 | |
| *** baoli has joined #openstack-nova | 19:01 | |
| *** annegentle has joined #openstack-nova | 19:02 | |
| *** tylerder_ has joined #openstack-nova | 19:03 | |
| melwitt | rm_work: *looks* yeah, I didn't know project_id isn't even part of the data model. so yeah, much bigger change than I was thinking too | 19:03 |
| mriedem | gheorghe_: there are lots of different ways to configure it i think, | 19:03 |
| mriedem | devstack uses ssh | 19:03 |
| mriedem | you could also run in tunneled mode but that slows it down | 19:04 |
| gheorghe_ | mriedem this is not devstack, this is the tutorial with 1 controller and 4 kvm hosts | 19:04 |
| mriedem | sure | 19:05 |
| mriedem | that's not my point though | 19:05 |
| mriedem | also, which tutorial? | 19:05 |
| mriedem | and https://docs.openstack.org/security-guide/instance-management/security-services-for-instances.html#instance-migrations | 19:05 |
| *** annegentle has quit IRC | 19:06 | |
| *** tylerderosagrund has quit IRC | 19:06 | |
| mriedem | also https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux_openstack_platform/7/html-single/migrating_instances/ | 19:06 |
| jangutter | jaypipes: so I got succesful plugging with an external os-vif + neutron plug-in. | 19:08 |
| jaypipes | jangutter: \0/ | 19:08 |
| jangutter | jaypipes: Yep. Nova changes were tiny. There are two hitches though. | 19:08 |
| jangutter | jaypipes: one is that stashed the PCI address inside the vhost-user socket path. | 19:09 |
| jangutter | jaypipes: and the second is that there's no filtering for the Agilio NICs. it's basically picking anything that says it's an ethernet VF. | 19:09 |
| jangutter | jaypipes: the second is the one we know about. | 19:09 |
| jaypipes | jangutter: put the Netronome vendor_id and product_id in the pci_passthrough_whitelist. | 19:09 |
| jangutter | jaypipes: yep, that I did. And it's working as expected. | 19:10 |
| jaypipes | jangutter: I don't understand... you said it's not filtering? | 19:10 |
| jangutter | Oh, I mean that if you have two sets of NICs in the same system, things will go... unexpected. | 19:11 |
| *** jmlowe has quit IRC | 19:11 | |
| jangutter | jaypipes: this is probably good enough (tm) for now in any case, if stashing the PCI address inside a socket path is acceptable. | 19:12 |
| jaypipes | jangutter: you mean if there's, say, a mlnx nic and a netronome nic, and you allow guests to use either of those, there's no way to select which NIC for a flavor, right? | 19:12 |
| *** raunak has quit IRC | 19:13 | |
| jangutter | jaypipes: correct. | 19:13 |
| jaypipes | jangutter: if both are connected to the same physical network, yes, unfortunately that's the case right now. until we land ralonsoh's patch for NIC capabilities. | 19:13 |
| jangutter | jaypipes: yep, so I'm happy to live with that for a first stab. | 19:14 |
| gheorghe_ | mriedem i am using ubuntu, not redhat | 19:14 |
| *** moshele has joined #openstack-nova | 19:15 | |
| jaypipes | jangutter: k, good with me. | 19:16 |
| jaypipes | jangutter: happy to hear you've made some good progress! | 19:16 |
| *** yamamoto has joined #openstack-nova | 19:16 | |
| *** nicolasbock has quit IRC | 19:16 | |
| *** harlowja has quit IRC | 19:17 | |
| gheorghe_ | mriedem i got libvirt to listen but none of the tutorials online helped :D i just had to dig through config files | 19:17 |
| *** david-lyle has quit IRC | 19:17 | |
| *** david-lyle has joined #openstack-nova | 19:17 | |
| *** abalutoiu_ has joined #openstack-nova | 19:18 | |
| jangutter | jaypipes: If I can get sean-k-mooney's way to set bridge name added, that means we only add the VIF_TYPE to nova.network.model, the associated _nova_to_osvif_vif_agilio_ovs() to os_vif_util | 19:18 |
| jangutter | jaypipes: (and moshel's _set_config_VIFHostDevice) of course. | 19:19 |
| *** kristian__ has joined #openstack-nova | 19:19 | |
| jaypipes | jangutter: right. the fewer changes, the better :) | 19:20 |
| *** adisky__ has quit IRC | 19:20 | |
| jangutter | jaypipes: anyway, I'll do my best to get the code uploaded ASAP. | 19:20 |
| jaypipes | jangutter: awesome, thanks! | 19:21 |
| *** abalutoiu has quit IRC | 19:22 | |
| *** slaweq_ has joined #openstack-nova | 19:23 | |
| *** slaweq_ has quit IRC | 19:26 | |
| *** slaweq_ has joined #openstack-nova | 19:26 | |
| *** tbachman has quit IRC | 19:27 | |
| *** crushil has joined #openstack-nova | 19:28 | |
| *** crushil has quit IRC | 19:28 | |
| *** annegentle has joined #openstack-nova | 19:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage https://review.openstack.org/483060 | 19:30 |
| gheorghe_ | mriedem it works now | 19:32 |
| *** crushil has joined #openstack-nova | 19:39 | |
| *** awaugama has quit IRC | 19:40 | |
| mriedem | gheorghe_: cool - if you have something to fix up the config options help text within nova, that would be appreciated | 19:43 |
| *** slaweq_ has quit IRC | 19:43 | |
| *** jjung has quit IRC | 19:43 | |
| mriedem | or was it libvirt config that you changed? | 19:43 |
| gheorghe_ | i will show you my config for deploying hosts | 19:43 |
| *** slaweq_ has joined #openstack-nova | 19:43 | |
| gheorghe_ | it configs nova AND libvirt for migration | 19:43 |
| gheorghe_ | wait a sec | 19:43 |
| gheorghe_ | check at the bottom: https://paste.gnome.org/p9vruj0zw ... as you will see here, this is the unsecure resolution. but it will enable you to migrate. also, it greps IP from a subnet deployed within my test environment | 19:44 |
| gheorghe_ | i just this on every kvm node and it enables nova-compute and migration for kvm | 19:45 |
| gheorghe_ | *i just RUN this | 19:45 |
| *** slaweq_ has quit IRC | 19:48 | |
| *** slaweq_ has joined #openstack-nova | 19:49 | |
| *** crushil has quit IRC | 19:51 | |
| *** lucasxu has joined #openstack-nova | 19:51 | |
| *** tblakes has joined #openstack-nova | 19:51 | |
| *** mdrabe_ has joined #openstack-nova | 19:53 | |
| *** gszasz has quit IRC | 19:53 | |
| *** mdrabe_ has left #openstack-nova | 19:53 | |
| *** moshele has quit IRC | 19:54 | |
| *** yamamoto has quit IRC | 19:54 | |
| *** crushil has joined #openstack-nova | 19:56 | |
| *** baoli has quit IRC | 19:56 | |
| *** tblakes has quit IRC | 19:59 | |
| *** FL1SK has joined #openstack-nova | 20:00 | |
| *** baoli has joined #openstack-nova | 20:00 | |
| *** baoli has quit IRC | 20:00 | |
| *** harlowja has joined #openstack-nova | 20:02 | |
| *** moshele has joined #openstack-nova | 20:04 | |
| *** baoli has joined #openstack-nova | 20:07 | |
| *** lucasxu has quit IRC | 20:08 | |
| *** liverpooler has quit IRC | 20:10 | |
| *** cdent has quit IRC | 20:10 | |
| *** jdandrea has joined #openstack-nova | 20:11 | |
| *** tbachman has joined #openstack-nova | 20:13 | |
| jdandrea | For "nova boot" how does one specify metadata with a dictionary value? | 20:14 |
| jdandrea | Trying --meta foo='{"bar": ["baz"]]}}' and it seems to arrive in the scheduler plugin's instance properties as a dict of the form {'foo': '{bar:[baz]}'} instead. | 20:14 |
| jdandrea | Which is not valid JSON nor YAML. Maybe I'm doing something wrong on the cmd line? | 20:14 |
| jdandrea | (I should say the value for foo isn't valid JSON nor YAML.) | 20:15 |
| openstackgerrit | Merged openstack/nova master: doc: Remove cruft from conf.py https://review.openstack.org/478468 | 20:15 |
| *** crushil has quit IRC | 20:16 | |
| *** rajathagasthya has joined #openstack-nova | 20:17 | |
| *** tylerder_ has quit IRC | 20:18 | |
| mriedem | jdandrea: i think you specify --meta multiple times | 20:19 |
| jdandrea | mriedem - how do you do that to reference a nested key though? | 20:19 |
| jdandrea | So I'd say --meta level_1_key=... something and then somehow refer to a level below level_1_key? | 20:20 |
| mriedem | oh, no | 20:20 |
| *** abalutoiu_ has quit IRC | 20:20 | |
| jdandrea | I want to provide a value that is itself a dict. So I figure I'd send in a JSON string enclosed in single quotes, but that doesn't quite work. | 20:20 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: WIP: Microversion 2.50 - fix quota class sets resource usage https://review.openstack.org/483060 | 20:20 |
| jdandrea | And I can't remove the single quotes without it going sideways. | 20:20 |
| jdandrea | My problem is what I get on the inside doesn't appear to be valid JSON or YAML, so I can't even string load it. | 20:21 |
| mriedem | the CLI isn't expect the value to be itself a dict, so it does a single split and assumes it's a string, which it is - and it's not going to unpack that | 20:23 |
| mriedem | return dict(v.split('=', 1) for v in metadata) | 20:23 |
| mriedem | is what the cli code is doing | 20:23 |
| jaypipes | jdandrea: who let you in here!? :P | 20:24 |
| jdandrea | jaypipes - STOP FOLLOWING ME. :p | 20:24 |
| jaypipes | hehe | 20:24 |
| jdandrea | lol | 20:24 |
| *** tylerderosagrund has joined #openstack-nova | 20:24 | |
| jdandrea | mriedem Right, that makes sense. So the trick is to encode it such that it lands in the plugin as something that can be converted sanely. | 20:25 |
| openstackgerrit | Merged openstack/nova master: doc: Use consistent author, section for man pages https://review.openstack.org/482960 | 20:25 |
| openstackgerrit | Merged openstack/nova master: doc: Remove dead plugin https://review.openstack.org/478469 | 20:25 |
| *** tylerderosagrund has quit IRC | 20:26 | |
| *** tylerderosagrund has joined #openstack-nova | 20:26 | |
| mriedem | jdandrea: hmm, do you need to encode it somehow? | 20:26 |
| mriedem | i'm not sure what's goofing it up when it gets over into the API | 20:26 |
| jdandrea | mriedem Yeah, I'm not either. Pondering. | 20:27 |
| mriedem | urlencode? | 20:27 |
| jdandrea | Mayyyyybe. | 20:27 |
| jdandrea | Plausible. | 20:27 |
| mriedem | i'm totally guessing | 20:27 |
| openstackgerrit | Merged openstack/nova master: api-ref: mark instance action events parameter as optional https://review.openstack.org/480792 | 20:27 |
| jdandrea | But messy, I admit. | 20:27 |
| jdandrea | Gonna hack on this a li'l bit ... | 20:27 |
| jdandrea | biab | 20:27 |
| * jdandrea scurries away again :) | 20:28 | |
| melwitt | guh, more random "OpenStackApiAuthenticationException: Authentication error" functional test fails | 20:29 |
| melwitt | gonna have to figure that out soon | 20:29 |
| melwitt | in the gate | 20:32 |
| mriedem | eventlet | 20:32 |
| * melwitt shakes fist | 20:32 | |
| mriedem | it's something with the compute api and placement api client helper things in the functional tests getting screwed up over eventlet, i think | 20:33 |
| mriedem | cdent had a patch at one point for a related issue where he put a yield in there | 20:33 |
| melwitt | oh, that sounds familiar | 20:33 |
| mriedem | i'm sure i can find it | 20:33 |
| mriedem | like a hog hunting for truffles | 20:33 |
| * mriedem makes hog sounds | 20:33 | |
| melwitt | lol | 20:33 |
| *** moshele has quit IRC | 20:36 | |
| *** raunak has joined #openstack-nova | 20:37 | |
| mriedem | https://review.openstack.org/#/c/459735/ is what we went with | 20:40 |
| mriedem | https://review.openstack.org/#/c/459733/ was cdents | 20:40 |
| melwitt | hm | 20:41 |
| mriedem | maybe we should set that globally, | 20:41 |
| mriedem | if it's failing on other tests that aren't using the integrated helpers stuf | 20:42 |
| mriedem | *stuff | 20:42 |
| melwitt | oh yeah, lemme see what this one was | 20:42 |
| mriedem | i had one earlier here http://logs.openstack.org/22/454322/37/check/gate-nova-tox-functional-ubuntu-xenial/76cbda6/console.html#_2017-07-12_17_44_14_514884 | 20:42 |
| melwitt | api_sample_tests, thought those were using integrated_helpers http://logs.openstack.org/74/467774/7/gate/gate-nova-tox-functional-ubuntu-xenial/4a00841/testr_results.html.gz | 20:42 |
| mriedem | api sample tests are yeah | 20:42 |
| *** crushil has joined #openstack-nova | 20:43 | |
| mriedem | mine was something else | 20:43 |
| mriedem | NoValidHost | 20:44 |
| mriedem | i think because the compute node never checked in | 20:44 |
| mriedem | because of some eventlet timeout | 20:44 |
| *** pchavva has quit IRC | 20:44 | |
| * jdandrea wanders in | 20:48 | |
| jaypipes | mriedem, dansmith, edleafe: did we want to return a list of alternates *for each instance in num_instances*? of a single list of alternate allocation requests for the entire request? | 20:48 |
| jaypipes | s/of a single/or a single/g | 20:48 |
| jdandrea | mriedem jaypipes So what I did was look at the value and if it is a string, I try to json.loads the thing. If that fails, I leave it as is. Mischief managed. | 20:48 |
| dansmith | jaypipes: well, if all of them can be rescheduled max_attempts time, then it needs to be num_instances * max_attempts | 20:49 |
| dansmith | jaypipes: but we could also take some percentage or something, and provide some number of bet-hedging alternates, but not enough for all of them to be rescheduled max times | 20:49 |
| *** smatzek has quit IRC | 20:49 | |
| jaypipes | dansmith: no, sorry, I meant should I return a single list of alternate allocation requests for the entire call to select_destinations() or should I return a list of alternate allocation requests for each host returned by select_destinations()? | 20:50 |
| dansmith | jaypipes: independent of the number? | 20:51 |
| dansmith | jaypipes: an alternate has to go with a given primary host, because it has to be the same cell, | 20:51 |
| dansmith | so a single list of num*max alternates won't help us once we lose context of which host is in which cell | 20:51 |
| jaypipes | right | 20:51 |
| jaypipes | ok, that answers that | 20:51 |
| melwitt | cells meeting in 8 minutes | 20:52 |
| dansmith | melwitt: correction | 20:52 |
| dansmith | melwitt: short-ass cells meetinglet in 8 minutes | 20:52 |
| *** penick has joined #openstack-nova | 20:54 | |
| melwitt | meetinglet, I like it | 20:54 |
| *** yamamoto has joined #openstack-nova | 20:54 | |
| edleafe | jaypipes: yeah, I think that a single list per instance is best, where each list is the selected (claimed) host followed by the alternates. | 20:56 |
| efried | In Portuguese, meetazinho. In Spanish, meetito. In French, meetette. In German, meetchen. | 20:56 |
| dansmith | ooh, meetito is great | 20:57 |
| dansmith | like dorito, but less yummy | 20:57 |
| efried | Nothing is less yummy than a cooler ranch dorito | 20:57 |
| jaypipes | edleafe: I'm going for a dict, keyed by instance_uuid, of two-tuples where the tuple is (WeighedHost(claimed_host), list(alternate allocation requests)) | 20:57 |
| jaypipes | edleafe, dansmith: ^^ make sense to you? | 20:57 |
| dansmith | sure | 20:58 |
| efried | jaypipes Is this yet another thing we're going to regret not being an object later? | 20:59 |
| melwitt | efried: phew, that was close. I misread at first that you were saying cooler ranch was the best dorito | 20:59 |
| efried | melwitt Rest easy. | 20:59 |
| edleafe | jaypipes: this is what scheduler is returning to the conductor? | 21:00 |
| jaypipes | edleafe: well, not *yet*... I'm currently referring to what filter scheduler _schedule() returns to filter scheduler's select_destinations() call. | 21:00 |
| jaypipes | edleafe: still need to figure out what's best for the RPC changes. | 21:00 |
| jaypipes | edleafe: it *may* be that structure... | 21:01 |
| jaypipes | haven't fully thought the RPC layer changes through yet | 21:01 |
| *** yamamoto has quit IRC | 21:01 | |
| edleafe | jaypipes: I'm still not clear why the selected host has to be in a separate structure than the alternates | 21:01 |
| jaypipes | edleafe: the alternates are meant to be opaque structures. the selected host is the host identifier. the allocation requests are alternate things that can be passed to placement to claim resources if the original destination host fails to fully launch the instance. | 21:03 |
| jaypipes | edleafe: or are you asking to just have a list where the first element of the list is a WeighedHost and the other elements of the list are allocation request blobs? | 21:04 |
| *** Sukhdev has quit IRC | 21:04 | |
| edleafe | jaypipes: the distinction seems unnecessary. Why not send a list of (host, alloc) for each instance, where the first item in the list is the selected host? This way, if the build fails, the cell conductor can easily unclaim the resources already claimed | 21:05 |
| jaypipes | dansmith: thoughts? ^ | 21:06 |
| dansmith | jaypipes: we're in the cells meeting right now | 21:06 |
| jaypipes | oh, sorry. | 21:06 |
| edleafe | jaypipes: passing a heterogenous structure seems like it will lead to a lot of conditional blocks to handle the different cases | 21:06 |
| jaypipes | edleafe: I don't disagree with you on that | 21:07 |
| *** dave-mccowan has quit IRC | 21:09 | |
| *** dgonzalez_ has joined #openstack-nova | 21:10 | |
| *** dgonzalez_ has quit IRC | 21:12 | |
| *** jmlowe has joined #openstack-nova | 21:12 | |
| *** edmondsw has quit IRC | 21:12 | |
| *** edmondsw has joined #openstack-nova | 21:13 | |
| *** edmondsw_ has joined #openstack-nova | 21:14 | |
| *** edmondsw has quit IRC | 21:14 | |
| *** tylerderosagrund has quit IRC | 21:14 | |
| *** edmondsw_ has quit IRC | 21:15 | |
| *** lennyb has quit IRC | 21:15 | |
| *** edmondsw has joined #openstack-nova | 21:15 | |
| *** kylek3h has quit IRC | 21:19 | |
| *** edmondsw has quit IRC | 21:20 | |
| dansmith | jaypipes: I would expect the compute to un-claim before passing the reschedule attempt to conductor, since it will un-claim always on failed boot, and then punt to conductor if it thinks it should be retried | 21:22 |
| mriedem | dansmith: dtp: melwitt: https://etherpad.openstack.org/p/nova-pike-cells-v2-todos | 21:23 |
| mriedem | i'll start dumping | 21:23 |
| melwitt | cool, thanks | 21:23 |
| dansmith | jaypipes: so I'm not sure that we need anything too complicated, and was kinda expecting the conductor to do the next claim in order, and then take the would-be-versioned _response_ from placement as an indication of which compute node to use | 21:23 |
| *** thorst has quit IRC | 21:23 | |
| dansmith | jaypipes: however, if passing (hostname, allocation) around is more convenient or symmetric then that's fine | 21:24 |
| dansmith | jaypipes: your select-destinations can't return a HostState outside of the scheduler though | 21:24 |
| jaypipes | dansmith: yeah, I think edleafe's suggestion is a good one. | 21:24 |
| dansmith | jaypipes: I'm not sure what he's suggesting, other than just making it a list where the implication is that we try the first one as the "selected host" | 21:25 |
| dansmith | I assumed your example of the weighedhost as the selected one was for efficiency of the next step in the process | 21:25 |
| *** efried has quit IRC | 21:25 | |
| edleafe | dansmith: yeah, that's pretty much it :) | 21:25 |
| jaypipes | dansmith: he's suggesting returning a list of (host, allocation_request) tuples for each instance_uuid in the request | 21:26 |
| dansmith | obviously that's the most you can return from the rpc call, if that's what you're talking about, but you said WeighedHost() above, so I figured you were talking internal still | 21:26 |
| *** lennyb has joined #openstack-nova | 21:27 | |
| dansmith | but yeah, I think I said before that just returning a list of destinations per instance, and processing them in order in each phase is the way I think of it | 21:27 |
| dansmith | like, I would expect each round to just pop the first one off the list, | 21:27 |
| dansmith | try it, and move on | 21:27 |
| dansmith | doing claim in conductor made that super symmetric | 21:27 |
| dansmith | doing it in scheduler means it's pre-claimed before we return, which is what I was arguing against in the first place :) | 21:28 |
| jaypipes | dansmith: but we can't claim in the conductor when we have num_instances > 0 unfortunately. see: instance groups, affinity, and similar issues. | 21:30 |
| *** lpetrut has quit IRC | 21:30 | |
| edleafe | jaypipes: so assuming a retry is necessary, how will the cell conductor deal with those things? | 21:31 |
| jaypipes | edleafe: unfortunately, the only way to deal with those things on retry is the last-minute check on the alternate target host :( | 21:32 |
| dansmith | jaypipes: I don't think I understand why that would be, but you guys steamrolled past that anyway, so it doesn't matter | 21:32 |
| jaypipes | dansmith: because of this line: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L127-L128 | 21:33 |
| jaypipes | dansmith: those little lines are the bane of my existence. | 21:33 |
| dansmith | I don't see how that affects claiming in the conductor, but whatever, not important | 21:34 |
| *** efried has joined #openstack-nova | 21:38 | |
| *** awaugama has joined #openstack-nova | 21:40 | |
| *** lyan has quit IRC | 21:40 | |
| *** zzzeek_ has joined #openstack-nova | 21:42 | |
| *** edmondsw has joined #openstack-nova | 21:46 | |
| *** catintheroof has quit IRC | 21:46 | |
| *** edmondsw has quit IRC | 21:50 | |
| *** armax has quit IRC | 21:51 | |
| *** felipemonteiro_ has quit IRC | 21:52 | |
| *** armax has joined #openstack-nova | 21:52 | |
| mriedem | melwitt: so i didn't get to the resize stuff yet https://review.openstack.org/#/c/416521/ but with fresh eyes i think i've found 2 issues already discussed about over quota on multiple create not cleaning up, and restore | 21:52 |
| *** eharney has quit IRC | 21:54 | |
| *** jmlowe has quit IRC | 21:54 | |
| *** armax has quit IRC | 21:56 | |
| *** tylerderosagrund has joined #openstack-nova | 21:57 | |
| *** Apoorva has quit IRC | 21:58 | |
| melwitt | mriedem: hm, okay. will look at that | 22:04 |
| *** Apoorva has joined #openstack-nova | 22:04 | |
| *** Apoorva has quit IRC | 22:04 | |
| *** burt has quit IRC | 22:08 | |
| *** Apoorva has joined #openstack-nova | 22:10 | |
| *** esberglu has quit IRC | 22:11 | |
| *** jamesdenton has quit IRC | 22:14 | |
| *** lyan has joined #openstack-nova | 22:14 | |
| *** jamesdenton has joined #openstack-nova | 22:14 | |
| *** zzzeek_ has quit IRC | 22:15 | |
| *** tylerderosagrund has quit IRC | 22:17 | |
| *** zzzeek_ has joined #openstack-nova | 22:17 | |
| *** zzzeek has quit IRC | 22:18 | |
| *** raunak has quit IRC | 22:19 | |
| *** jamesdenton has quit IRC | 22:19 | |
| *** penick has quit IRC | 22:19 | |
| melwitt | mriedem: replied about restore. reading through multi create | 22:21 |
| *** slaweq_ has quit IRC | 22:23 | |
| *** kristian__ has quit IRC | 22:26 | |
| *** kristian__ has joined #openstack-nova | 22:26 | |
| melwitt | mriedem: though now I see the change to state "soft-deleted" doesn't happen until after an rpc call to compute, so I think that's why I didn't bother with recheck_quota there | 22:28 |
| melwitt | maybe I should change it to set the state right away before the cast. not sure if that would be okay | 22:28 |
| *** tylerderosagrund has joined #openstack-nova | 22:30 | |
| *** sapcc-bot3 has joined #openstack-nova | 22:31 | |
| *** carthaca_1 has joined #openstack-nova | 22:31 | |
| *** d34dh0r53 has quit IRC | 22:32 | |
| *** liangy has joined #openstack-nova | 22:32 | |
| *** sapcc-bot3 has quit IRC | 22:32 | |
| *** carthaca_1 has quit IRC | 22:32 | |
| *** artom has quit IRC | 22:32 | |
| *** sapcc-bot has quit IRC | 22:32 | |
| *** carthaca_ has quit IRC | 22:32 | |
| *** kristian__ has quit IRC | 22:32 | |
| *** carthaca_2 has joined #openstack-nova | 22:32 | |
| *** sapcc-bot4 has joined #openstack-nova | 22:32 | |
| *** d34dh0r53 has joined #openstack-nova | 22:32 | |
| *** rm_work has quit IRC | 22:32 | |
| *** lyan has quit IRC | 22:32 | |
| *** rm_work has joined #openstack-nova | 22:32 | |
| *** artom has joined #openstack-nova | 22:33 | |
| *** anish has quit IRC | 22:34 | |
| *** rhagarty has quit IRC | 22:35 | |
| *** raginbajin has quit IRC | 22:35 | |
| *** csuttles has quit IRC | 22:35 | |
| *** anish has joined #openstack-nova | 22:35 | |
| *** tylerderosagrund has quit IRC | 22:35 | |
| *** slagle has quit IRC | 22:35 | |
| *** csuttles has joined #openstack-nova | 22:35 | |
| *** slagle has joined #openstack-nova | 22:35 | |
| *** raunak has joined #openstack-nova | 22:35 | |
| *** sambetts|afk has quit IRC | 22:36 | |
| *** szaher has quit IRC | 22:37 | |
| *** szaher has joined #openstack-nova | 22:37 | |
| mriedem | ok still going through some other parts of this | 22:37 |
| *** rhagarty has joined #openstack-nova | 22:37 | |
| *** sambetts_ has joined #openstack-nova | 22:37 | |
| openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: Extended PCI alias https://review.openstack.org/474218 | 22:38 |
| *** claudiub has quit IRC | 22:38 | |
| *** raginbajin has joined #openstack-nova | 22:42 | |
| mriedem | ok reading your replies now | 22:42 |
| *** thorst has joined #openstack-nova | 22:45 | |
| *** thorst has quit IRC | 22:45 | |
| mriedem | hmm, so when you soft delete an instance, | 22:45 |
| mriedem | the deleted_at field is set on the instance | 22:45 |
| mriedem | but when you restore it, i don't see the deleted_at field set to None | 22:46 |
| mriedem | so it's both ACTIVE and deleted_at | 22:46 |
| melwitt | *thumbs up* | 22:46 |
| mriedem | unless i'm missing something | 22:46 |
| melwitt | I doubt it | 22:46 |
| melwitt | either it was missed or someone thought it helpful to let it show that it had been deleted in the past | 22:47 |
| mriedem | i don't have a devstack handy | 22:47 |
| mriedem | yeah....but instance actions... | 22:47 |
| mriedem | if you want an audit record | 22:47 |
| melwitt | yeah | 22:47 |
| openstackgerrit | Vipin Balachandran proposed openstack/nova master: VMware:Use power_state const for power state check https://review.openstack.org/483122 | 22:49 |
| *** tylerderosagrund has joined #openstack-nova | 22:50 | |
| *** yamamoto has joined #openstack-nova | 22:52 | |
| mriedem | melwitt: ok, replies are all in - still haven't gotten t oresize | 22:53 |
| mriedem | *to resize | 22:53 |
| mriedem | to summarize, i think we're ok with restore, except maybe that we don't have a recheck deltas in there? | 22:53 |
| mriedem | and i also got a bit hung up on the fact we aren't accounting for the hw video ram when counting existing usage | 22:53 |
| mriedem | as dansmith pointed out | 22:53 |
| mriedem | for some reason i was ok with that last night, but now i'm not so sure | 22:53 |
| melwitt | mriedem: yeah, I think restore is fine. we might probably want to move the vm_state change to compute/api and do a recheck yeah | 22:54 |
| mriedem | oh i see, | 22:55 |
| melwitt | mriedem: yeah, for the majority of the time, I think we were counting video ram on allocate and not on deallocate. someone recently fixed a bug to add subtraction of video ram usage upon deallocate | 22:55 |
| mriedem | because today it's (1) check, (2) change state (api or compute), and rechecking at 1.5 wouldn't do anything since the state didn't change to affect the count, | 22:55 |
| mriedem | so to recheck, we'd have to (1) check, (2) change state, (3) recheck, (4) cast | 22:55 |
| melwitt | yes | 22:55 |
| mriedem | and older computes are setting the vm state to ACTIVE but that shouldn't cause a problem | 22:56 |
| mriedem | if the api did it i mean | 22:56 |
| mriedem | it's a bit of a lie though...for example, if we set the state to ACTIVE, counted, and then cast to compute, but the rpc message never got there | 22:56 |
| mriedem | we'd say the instance is active even though the guest is dead | 22:57 |
| melwitt | oh yeah. maybe it could be a problem if an old compute goes to set SOFT_DELETED and it's already SOFT_DELETED (if it expected a different vm_state) | 22:57 |
| mriedem | i think vm_state is ok, | 22:57 |
| mriedem | you can set that to ACTIVE twice and it's fine | 22:58 |
| mriedem | the problem is task_state | 22:58 |
| mriedem | instance.save(expected_task_state=[task_states.SOFT_DELETING]) | 22:58 |
| mriedem | and setting vm_state=ACTIVE while task_state is SOFT_DELETING is a problem, | 22:58 |
| melwitt | oh, guh sorry getting it backward. restore sets to active | 22:58 |
| mriedem | because when an instance is ACTIVE it's task_state is supposed to be None | 22:58 |
| *** armax has joined #openstack-nova | 22:58 | |
| mriedem | right | 22:58 |
| *** acormier has quit IRC | 22:58 | |
| mriedem | you would almost have to temporarily change the state to active for the recheck, and then set the vm_state back to SOFT_DELETED for the cast to the compute - which makes me barf a little | 22:59 |
| melwitt | yeah. that would suck to call it active if RPC message lost | 22:59 |
| *** acormier has joined #openstack-nova | 22:59 | |
| *** tylerder_ has joined #openstack-nova | 22:59 | |
| melwitt | lemme see | 22:59 |
| mriedem | melwitt: btw, let me hand you a power pack of good will for sticking with this, because this makes my brain bleed | 23:00 |
| melwitt | okay yeah there's no in between vm_state of RESTORING. that was a task_state | 23:00 |
| *** awaugama has quit IRC | 23:00 | |
| *** annegentle has quit IRC | 23:01 | |
| melwitt | mriedem: heh ... thanks. I felt like a burned out husk for quite a time. props to you and those who have braved reviewing it. I kind of cringed every time I asked someone to review it | 23:02 |
| *** tylerderosagrund has quit IRC | 23:02 | |
| *** jmlowe has joined #openstack-nova | 23:02 | |
| *** acormier has quit IRC | 23:03 | |
| *** crushil has quit IRC | 23:04 | |
| melwitt | mriedem: these state changes in the API change deleted_at to None. and I guess I had been thinking to just punt this and assume parallel restores are not really a thing in real life | 23:04 |
| melwitt | maybe I could add a filter to count RESTORING as usage too and do handle the recheck that way. I dunno | 23:06 |
| *** itlinux has quit IRC | 23:06 | |
| *** tylerderosagrund has joined #openstack-nova | 23:07 | |
| mriedem | ha | 23:07 |
| mriedem | instance.deleted_at = None | 23:07 |
| mriedem | right in front of my gd eyes | 23:07 |
| melwitt | mine too. and I still didn't see it until like the fourth time I looked at it | 23:08 |
| *** tylerder_ has quit IRC | 23:08 | |
| *** jmlowe has quit IRC | 23:09 | |
| *** tylerder_ has joined #openstack-nova | 23:12 | |
| *** tylerderosagrund has quit IRC | 23:14 | |
| *** tylerder_ has quit IRC | 23:14 | |
| *** tylerderosagrund has joined #openstack-nova | 23:14 | |
| *** tylerder_ has joined #openstack-nova | 23:15 | |
| *** tylerderosagrund has quit IRC | 23:15 | |
| mriedem | melwitt: so, | 23:15 |
| mriedem | re the hw video ram thing, | 23:15 |
| mriedem | we might want to just burn that turd out of the api before your series | 23:15 |
| dansmith | +10,000,000 | 23:16 |
| melwitt | lol | 23:16 |
| *** baoli_ has joined #openstack-nova | 23:16 | |
| mriedem | it was added back in icehouse https://review.openstack.org/#/q/topic:bp/libvirt-video-driver-selection | 23:16 |
| dansmith | going over that just now, that's nutsypants mcgee | 23:16 |
| mriedem | and is the only special virt-driver specific extra spec accounted against quota in the api | 23:16 |
| melwitt | well, that makes me feel bad for the person who has been trying to fix those video ram quota bugs | 23:16 |
| melwitt | but yeah, I was thinking to at least start off with no video ram | 23:16 |
| mriedem | well, they fixed something that was screwing up their quota | 23:16 |
| melwitt | yes | 23:17 |
| mriedem | if we don't count it against usage in the first place, that's not a problem | 23:17 |
| *** tylerder_ has quit IRC | 23:17 | |
| *** dtp has quit IRC | 23:17 | |
| melwitt | yeah, that's what I was thinking | 23:17 |
| melwitt | if anything they'll get more out of their quota if we don't count it :P | 23:17 |
| mriedem | well, it's not like there aren't other things we don't account for in quota, which are calculated as overheads in the virt driver | 23:18 |
| mriedem | this could potentially move there for the RT claim stuff | 23:18 |
| mriedem | or wherever | 23:18 |
| melwitt | yeah | 23:18 |
| *** baoli has quit IRC | 23:18 | |
| melwitt | fine by me. makes my life easier too | 23:18 |
| mriedem | i'll probably poke the ops list first to be safe | 23:19 |
| mriedem | dansmith: agree i assume? ^ | 23:19 |
| dansmith | mriedem: I'd rather you not but whatever | 23:19 |
| mriedem | so sneaky remove? | 23:19 |
| dansmith | bugfix, imho | 23:20 |
| melwitt | yeah. the bug where video ram usage wasn't ever decremented, that was a bug from whenever the feature was added to ... like last month | 23:20 |
| melwitt | so I'm guessing around 5 years at least | 23:20 |
| dansmith | dat bug yo | 23:20 |
| melwitt | I assume people can't have cared about it too much | 23:21 |
| dansmith | only people with video ram called out too | 23:21 |
| mriedem | https://review.openstack.org/#/c/456050/ | 23:22 |
| mriedem | was the recent fix | 23:23 |
| *** melwitt has left #openstack-nova | 23:23 | |
| *** melwitt has joined #openstack-nova | 23:23 | |
| dansmith | rage quit! | 23:23 |
| *** slaweq_ has joined #openstack-nova | 23:23 | |
| melwitt | lol | 23:24 |
| mriedem | ok so maybe i just remove it and mark it related to that bug, and bug 1704012 and the counting quotas series | 23:24 |
| openstack | bug 1704012 in OpenStack Compute (nova) "hw_video:ram_max_mb is tracked for quota but not compute node resource provider allocations" [Undecided,New] https://launchpad.net/bugs/1704012 | 23:24 |
| melwitt | I do that so often. open a link from the chat window and try to close the browser tab and close the channel tab | 23:24 |
| melwitt | (with ctrl w) | 23:24 |
| dansmith | melwitt: it wasn't even mentioned in the commit message, nor was the quota bit tested specifically, AFAICT | 23:25 |
| dansmith | er, mriedem ^ | 23:25 |
| *** baoli_ has quit IRC | 23:26 | |
| mriedem | so, | 23:27 |
| mriedem | can my commit message to remove it just say, "hey, what's that behind you!" | 23:27 |
| melwitt | lol | 23:27 |
| mriedem | melwitt: you want me to do it and you rebase on top or you just handle it? | 23:28 |
| melwitt | mriedem: yeah, you can do it | 23:28 |
| *** slaweq_ has quit IRC | 23:28 | |
| mriedem | ok | 23:29 |
| dansmith | mriedem: yes, yes it can | 23:29 |
| melwitt | you can have that glory | 23:29 |
| melwitt | I'm still trying to think about multi create | 23:30 |
| *** penick has joined #openstack-nova | 23:33 | |
| *** tbachman has quit IRC | 23:33 | |
| *** tbachman has joined #openstack-nova | 23:36 | |
| melwitt | mriedem: yeah, I think you're right about the multi create thing. in compute/api, we're in the place where we haven't yet returned anything to the user. so if we bail with OverQuota, there should be no instances left that were created as a result of the request | 23:39 |
| melwitt | it should be as if it never happend | 23:39 |
| mriedem | slide of hand | 23:39 |
| mriedem | *slight | 23:39 |
| melwitt | looks like would have to clean up request specs and build requests too. | 23:40 |
| mriedem | yeah | 23:40 |
| mriedem | similar issue in conductor | 23:40 |
| *** hongbin has quit IRC | 23:40 | |
| melwitt | yeah | 23:40 |
| *** chyka_ has quit IRC | 23:40 | |
| *** chyka has joined #openstack-nova | 23:41 | |
| *** marst has quit IRC | 23:41 | |
| *** chyka has quit IRC | 23:45 | |
| *** Swami has quit IRC | 23:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Stop counting hw_video:ram_max_mb against quota https://review.openstack.org/483133 | 23:56 |
| mriedem | dansmith: melwitt: ^ | 23:56 |
| *** moshele has joined #openstack-nova | 23:57 | |
| melwitt | mriedem: cool | 23:59 |
| melwitt | also, I think I finished replying to your other comments | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!