*** artom_ has joined #openstack-nova | 00:07 | |
*** sapcc-bot has joined #openstack-nova | 00:09 | |
*** sapcc-bot has quit IRC | 00:09 | |
*** sapcc-bot has joined #openstack-nova | 00:10 | |
*** sapcc-bot has quit IRC | 00:10 | |
*** gmann has quit IRC | 00:12 | |
*** gmann has joined #openstack-nova | 00:12 | |
*** sapcc-bot has joined #openstack-nova | 00:15 | |
*** sapcc-bot has quit IRC | 00:15 | |
*** sapcc-bot has joined #openstack-nova | 00:16 | |
*** sapcc-bot has quit IRC | 00:16 | |
*** sapcc-bot has joined #openstack-nova | 00:21 | |
*** sapcc-bot has joined #openstack-nova | 00:27 | |
*** sapcc-bot has quit IRC | 00:27 | |
*** kiennt has joined #openstack-nova | 00:35 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 00:40 |
---|---|---|
*** sapcc-bot has joined #openstack-nova | 00:44 | |
*** sapcc-bot has quit IRC | 00:44 | |
*** sapcc-bot has joined #openstack-nova | 00:45 | |
*** sapcc-bot has quit IRC | 00:45 | |
*** mdnadeem has joined #openstack-nova | 00:45 | |
*** acormier has joined #openstack-nova | 00:51 | |
*** sapcc-bot has joined #openstack-nova | 00:56 | |
*** sapcc-bot has quit IRC | 00:56 | |
*** acormier has quit IRC | 00:56 | |
*** acormier has joined #openstack-nova | 00:57 | |
*** thorst has joined #openstack-nova | 00:59 | |
*** thorst has quit IRC | 01:04 | |
*** huanxie has joined #openstack-nova | 01:05 | |
*** sapcc-bot has joined #openstack-nova | 01:09 | |
*** sapcc-bot has quit IRC | 01:09 | |
*** yufei has joined #openstack-nova | 01:16 | |
*** zhurong has joined #openstack-nova | 01:20 | |
*** sapcc-bot has joined #openstack-nova | 01:20 | |
*** sapcc-bot has quit IRC | 01:20 | |
*** zhurong has quit IRC | 01:23 | |
*** liujiong has joined #openstack-nova | 01:24 | |
*** markvoelker has joined #openstack-nova | 01:29 | |
*** phuongnh has joined #openstack-nova | 01:30 | |
*** sapcc-bot has joined #openstack-nova | 01:34 | |
*** sapcc-bot has quit IRC | 01:34 | |
*** liujiong_lj has joined #openstack-nova | 01:36 | |
*** liujiong has quit IRC | 01:37 | |
*** sapcc-bot has joined #openstack-nova | 01:39 | |
*** sapcc-bot has quit IRC | 01:39 | |
*** liujiong_lj is now known as liujiong | 01:39 | |
*** sbezverk has quit IRC | 01:42 | |
*** sapcc-bot has joined #openstack-nova | 01:50 | |
*** sapcc-bot has quit IRC | 01:50 | |
*** sapcc-bot has joined #openstack-nova | 01:58 | |
*** sapcc-bot has quit IRC | 01:58 | |
*** yufei has quit IRC | 01:59 | |
*** itlinux_ has quit IRC | 02:00 | |
*** mdnadeem has quit IRC | 02:01 | |
*** markvoelker has quit IRC | 02:03 | |
*** yufei has joined #openstack-nova | 02:05 | |
*** kiennt has quit IRC | 02:07 | |
*** sapcc-bot has joined #openstack-nova | 02:08 | |
*** sapcc-bot has quit IRC | 02:08 | |
*** itlinux has joined #openstack-nova | 02:10 | |
*** gcb has joined #openstack-nova | 02:11 | |
*** mdnadeem has joined #openstack-nova | 02:14 | |
*** sapcc-bot has joined #openstack-nova | 02:15 | |
*** sapcc-bot has quit IRC | 02:15 | |
*** litao__ has joined #openstack-nova | 02:16 | |
*** phuongnh has quit IRC | 02:17 | |
*** phuongnh has joined #openstack-nova | 02:21 | |
*** sapcc-bot has joined #openstack-nova | 02:26 | |
*** sapcc-bot has quit IRC | 02:26 | |
*** zhurong has joined #openstack-nova | 02:31 | |
*** sapcc-bot has joined #openstack-nova | 02:31 | |
*** sapcc-bot has quit IRC | 02:31 | |
*** sapcc-bot has joined #openstack-nova | 02:37 | |
*** sapcc-bot has quit IRC | 02:37 | |
*** sapcc-bot has joined #openstack-nova | 02:43 | |
*** sapcc-bot has quit IRC | 02:43 | |
*** sapcc-bot has joined #openstack-nova | 02:43 | |
*** sapcc-bot has quit IRC | 02:43 | |
*** yamamoto has joined #openstack-nova | 02:47 | |
*** sapcc-bot has joined #openstack-nova | 02:50 | |
*** sapcc-bot has quit IRC | 02:50 | |
*** acormier has quit IRC | 02:52 | |
*** acormier has joined #openstack-nova | 02:52 | |
*** yufei has quit IRC | 02:54 | |
*** yufei has joined #openstack-nova | 02:55 | |
*** acormier has quit IRC | 02:57 | |
*** markvoelker has joined #openstack-nova | 03:00 | |
*** thorst has joined #openstack-nova | 03:01 | |
*** sapcc-bot has joined #openstack-nova | 03:01 | |
*** sapcc-bot has quit IRC | 03:01 | |
*** dave-mccowan has quit IRC | 03:04 | |
*** thorst has quit IRC | 03:05 | |
*** baoli has quit IRC | 03:05 | |
*** yamamoto has quit IRC | 03:09 | |
*** sapcc-bot has joined #openstack-nova | 03:12 | |
*** sapcc-bot has quit IRC | 03:12 | |
*** s1061123 has joined #openstack-nova | 03:13 | |
*** kristian__ has joined #openstack-nova | 03:16 | |
*** baoli has joined #openstack-nova | 03:16 | |
*** kiennt has joined #openstack-nova | 03:18 | |
*** sapcc-bot has joined #openstack-nova | 03:18 | |
*** sapcc-bot has quit IRC | 03:18 | |
*** Nel1x has quit IRC | 03:20 | |
*** Sukhdev has quit IRC | 03:20 | |
*** itlinux has quit IRC | 03:23 | |
*** sapcc-bot has joined #openstack-nova | 03:25 | |
*** sapcc-bot has quit IRC | 03:25 | |
*** itlinux has joined #openstack-nova | 03:30 | |
*** markvoelker has quit IRC | 03:34 | |
*** sapcc-bot has joined #openstack-nova | 03:36 | |
*** sapcc-bot has quit IRC | 03:36 | |
*** hongbin has joined #openstack-nova | 03:39 | |
*** udesale has joined #openstack-nova | 03:40 | |
*** baoli has quit IRC | 03:41 | |
*** mikal has quit IRC | 03:42 | |
*** sapcc-bot has joined #openstack-nova | 03:42 | |
*** sapcc-bot has quit IRC | 03:42 | |
*** gouthamr has quit IRC | 03:42 | |
*** links has joined #openstack-nova | 03:44 | |
*** rajathagasthya has joined #openstack-nova | 03:47 | |
openstackgerrit | Tuan Luong-Anh proposed openstack/python-novaclient master: Replace uuid.uuid4().hex with uuidutils.generate_uuid() https://review.openstack.org/484244 | 03:48 |
*** sapcc-bot has joined #openstack-nova | 03:48 | |
*** sapcc-bot has quit IRC | 03:48 | |
*** kiennt has quit IRC | 03:52 | |
*** kiennt has joined #openstack-nova | 03:59 | |
*** sapcc-bot has joined #openstack-nova | 04:00 | |
*** sapcc-bot has quit IRC | 04:00 | |
*** sapcc-bot has joined #openstack-nova | 04:01 | |
*** sapcc-bot has quit IRC | 04:01 | |
*** hoonetorg has quit IRC | 04:05 | |
*** sapcc-bot has joined #openstack-nova | 04:05 | |
*** sapcc-bot has quit IRC | 04:05 | |
*** yamamoto has joined #openstack-nova | 04:09 | |
*** sapcc-bot has joined #openstack-nova | 04:11 | |
*** sapcc-bot has quit IRC | 04:11 | |
*** hongbin has quit IRC | 04:13 | |
*** adisky__ has joined #openstack-nova | 04:13 | |
*** sapcc-bot has joined #openstack-nova | 04:13 | |
*** yamamoto has quit IRC | 04:14 | |
*** trungnv has quit IRC | 04:15 | |
*** trungnv has joined #openstack-nova | 04:15 | |
*** psachin has joined #openstack-nova | 04:18 | |
*** sapcc-bot has joined #openstack-nova | 04:23 | |
*** sapcc-bot has quit IRC | 04:23 | |
*** sapcc-bot has joined #openstack-nova | 04:24 | |
*** sapcc-bot has quit IRC | 04:24 | |
*** markvoelker has joined #openstack-nova | 04:30 | |
*** rajathagasthya has quit IRC | 04:38 | |
openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 04:40 |
*** kristian__ has quit IRC | 04:41 | |
*** sapcc-bot has joined #openstack-nova | 04:42 | |
*** sapcc-bot has quit IRC | 04:42 | |
*** Dinesh_Bhor has joined #openstack-nova | 04:46 | |
*** sapcc-bot has joined #openstack-nova | 04:47 | |
*** sapcc-bot has quit IRC | 04:47 | |
*** Sukhdev has joined #openstack-nova | 04:47 | |
*** kristian__ has joined #openstack-nova | 04:52 | |
*** sapcc-bot has joined #openstack-nova | 04:52 | |
*** sapcc-bot has quit IRC | 04:52 | |
*** sapcc-bot has joined #openstack-nova | 04:53 | |
*** sapcc-bot has quit IRC | 04:53 | |
*** kristian__ has quit IRC | 04:54 | |
*** kristian__ has joined #openstack-nova | 05:00 | |
*** kristian__ has quit IRC | 05:00 | |
*** sapcc-bot has joined #openstack-nova | 05:00 | |
*** sapcc-bot has quit IRC | 05:00 | |
*** kristian__ has joined #openstack-nova | 05:00 | |
*** thorst has joined #openstack-nova | 05:01 | |
*** zhurong has quit IRC | 05:04 | |
*** markvoelker has quit IRC | 05:04 | |
*** kristian__ has quit IRC | 05:05 | |
*** sapcc-bot has joined #openstack-nova | 05:05 | |
*** sapcc-bot has quit IRC | 05:05 | |
*** thorst has quit IRC | 05:06 | |
*** jjung has joined #openstack-nova | 05:13 | |
*** ekuris_ has joined #openstack-nova | 05:13 | |
*** ratailor has joined #openstack-nova | 05:16 | |
*** trinaths has joined #openstack-nova | 05:22 | |
*** trinaths1 has joined #openstack-nova | 05:22 | |
*** armax has joined #openstack-nova | 05:23 | |
*** trinaths has quit IRC | 05:26 | |
*** sapcc-bot has joined #openstack-nova | 05:27 | |
*** sapcc-bot has quit IRC | 05:28 | |
*** nicolasbock has joined #openstack-nova | 05:28 | |
*** zhurong has joined #openstack-nova | 05:29 | |
*** armax has quit IRC | 05:30 | |
*** armax has joined #openstack-nova | 05:30 | |
*** armax has quit IRC | 05:31 | |
*** armax has joined #openstack-nova | 05:31 | |
*** armax has quit IRC | 05:32 | |
*** armax has joined #openstack-nova | 05:32 | |
*** armax has quit IRC | 05:32 | |
*** armax has joined #openstack-nova | 05:33 | |
*** armax has quit IRC | 05:33 | |
*** sapcc-bot has joined #openstack-nova | 05:33 | |
*** sapcc-bot has quit IRC | 05:33 | |
*** armax has joined #openstack-nova | 05:34 | |
*** sbezverk has joined #openstack-nova | 05:34 | |
*** armax has quit IRC | 05:34 | |
*** armax has joined #openstack-nova | 05:35 | |
*** armax has quit IRC | 05:35 | |
*** avolkov has joined #openstack-nova | 05:39 | |
*** sapcc-bot has joined #openstack-nova | 05:40 | |
*** sapcc-bot has quit IRC | 05:40 | |
*** sridharg has joined #openstack-nova | 05:40 | |
*** kiennt has quit IRC | 05:45 | |
*** vvargaszte has joined #openstack-nova | 05:56 | |
*** sapcc-bot has joined #openstack-nova | 05:56 | |
*** sapcc-bot has quit IRC | 05:56 | |
*** Sara has quit IRC | 05:58 | |
*** rcernin_ has joined #openstack-nova | 06:00 | |
*** sapcc-bot has joined #openstack-nova | 06:03 | |
*** sapcc-bot has quit IRC | 06:03 | |
*** kiennt has joined #openstack-nova | 06:03 | |
*** prateek has joined #openstack-nova | 06:03 | |
*** Oku_OS-away is now known as Oku_OS | 06:06 | |
*** sapcc-bot has joined #openstack-nova | 06:08 | |
*** nicolasbock has quit IRC | 06:13 | |
*** nicolasbock has joined #openstack-nova | 06:15 | |
*** kristian__ has joined #openstack-nova | 06:16 | |
*** kiennt_ has joined #openstack-nova | 06:17 | |
*** kiennt has quit IRC | 06:17 | |
*** sapcc-bot has joined #openstack-nova | 06:20 | |
*** sapcc-bot has quit IRC | 06:20 | |
*** kristian__ has quit IRC | 06:20 | |
*** andreas_s has joined #openstack-nova | 06:23 | |
*** mikal has joined #openstack-nova | 06:24 | |
*** vvargaszte1 has joined #openstack-nova | 06:24 | |
*** sapcc-bot has joined #openstack-nova | 06:25 | |
*** sapcc-bot has quit IRC | 06:26 | |
*** vvargaszte has quit IRC | 06:26 | |
*** vvargaszte1 is now known as vvargaszte | 06:26 | |
*** prateek has quit IRC | 06:30 | |
*** sapcc-bot has joined #openstack-nova | 06:31 | |
*** sapcc-bot has quit IRC | 06:31 | |
*** sapcc-bot has joined #openstack-nova | 06:32 | |
*** sapcc-bot has quit IRC | 06:32 | |
*** sapcc-bot has joined #openstack-nova | 06:33 | |
*** sapcc-bot has quit IRC | 06:33 | |
*** edmondsw has joined #openstack-nova | 06:39 | |
*** sapcc-bot has joined #openstack-nova | 06:39 | |
*** sapcc-bot has quit IRC | 06:39 | |
*** kristian__ has joined #openstack-nova | 06:40 | |
*** kristian__ has quit IRC | 06:40 | |
*** kristian__ has joined #openstack-nova | 06:41 | |
*** kristian__ has quit IRC | 06:41 | |
*** kristian__ has joined #openstack-nova | 06:41 | |
*** kristian__ has quit IRC | 06:42 | |
*** kristian__ has joined #openstack-nova | 06:43 | |
*** edmondsw has quit IRC | 06:43 | |
*** rdopiera has joined #openstack-nova | 06:44 | |
*** kristia__ has joined #openstack-nova | 06:46 | |
*** kristian__ has quit IRC | 06:47 | |
*** rdopiera has left #openstack-nova | 06:47 | |
*** sapcc-bot has joined #openstack-nova | 06:49 | |
*** sapcc-bot has quit IRC | 06:49 | |
*** mlakat has joined #openstack-nova | 06:49 | |
*** Sukhdev has quit IRC | 06:50 | |
*** sapcc-bot has joined #openstack-nova | 06:56 | |
*** sapcc-bot has quit IRC | 06:56 | |
*** sshwarts has joined #openstack-nova | 06:58 | |
*** itlinux has quit IRC | 06:59 | |
*** sapcc-bot has joined #openstack-nova | 07:00 | |
*** itlinux has joined #openstack-nova | 07:00 | |
*** sapcc-bot has quit IRC | 07:00 | |
*** markvoelker has joined #openstack-nova | 07:01 | |
*** huanxie has quit IRC | 07:01 | |
*** thorst has joined #openstack-nova | 07:02 | |
*** thorst has quit IRC | 07:07 | |
*** itlinux has quit IRC | 07:08 | |
*** damien_r has joined #openstack-nova | 07:12 | |
alex_xu | sdague: if you are around this week, there are something need your help on review https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike, just need one more +2. Thanks in advance! | 07:14 |
*** karthiks_afk is now known as karthiks | 07:16 | |
*** sapcc-bot has joined #openstack-nova | 07:21 | |
*** sapcc-bot has quit IRC | 07:21 | |
*** damien_r has quit IRC | 07:22 | |
*** alexchadin has joined #openstack-nova | 07:22 | |
*** damien_r has joined #openstack-nova | 07:22 | |
*** sapcc-bot has joined #openstack-nova | 07:23 | |
*** sapcc-bot has quit IRC | 07:23 | |
*** alexchad_ has joined #openstack-nova | 07:28 | |
*** sapcc-bot has joined #openstack-nova | 07:29 | |
*** sapcc-bot has quit IRC | 07:29 | |
*** alexcha__ has joined #openstack-nova | 07:29 | |
*** alexchadin has quit IRC | 07:30 | |
*** sapcc-bot has joined #openstack-nova | 07:31 | |
*** sapcc-bot has quit IRC | 07:31 | |
*** tesseract has joined #openstack-nova | 07:32 | |
*** alexchad_ has quit IRC | 07:32 | |
*** alexchadin has joined #openstack-nova | 07:33 | |
*** alexcha__ has quit IRC | 07:34 | |
*** kuzko has quit IRC | 07:34 | |
*** markvoelker has quit IRC | 07:35 | |
*** alexchad_ has joined #openstack-nova | 07:36 | |
*** kuzko has joined #openstack-nova | 07:37 | |
*** alexcha__ has joined #openstack-nova | 07:37 | |
*** prateek has joined #openstack-nova | 07:37 | |
*** alexchadin has quit IRC | 07:38 | |
*** alexchad_ has quit IRC | 07:40 | |
*** ralonsoh has joined #openstack-nova | 07:42 | |
*** alexcha__ is now known as alexchadin | 07:42 | |
*** sapcc-bot has joined #openstack-nova | 07:47 | |
*** sapcc-bot has quit IRC | 07:47 | |
*** huanxie has joined #openstack-nova | 07:51 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 07:52 |
*** sapcc-bot has joined #openstack-nova | 07:53 | |
*** sapcc-bot has quit IRC | 07:53 | |
*** acormier has joined #openstack-nova | 07:53 | |
*** acormier has quit IRC | 07:57 | |
*** sapcc-bot has joined #openstack-nova | 07:59 | |
*** sapcc-bot has quit IRC | 07:59 | |
openstackgerrit | Manas Mandlekar proposed openstack/nova master: Instance remains in migrating state forever https://review.openstack.org/483911 | 08:09 |
*** aarefiev has joined #openstack-nova | 08:10 | |
*** sapcc-bot has joined #openstack-nova | 08:11 | |
*** sapcc-bot has quit IRC | 08:11 | |
*** gjayavelu has joined #openstack-nova | 08:15 | |
*** sapcc-bot has joined #openstack-nova | 08:22 | |
*** sapcc-bot has quit IRC | 08:22 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | 08:23 |
*** thorst has joined #openstack-nova | 08:27 | |
*** sridharg has quit IRC | 08:27 | |
*** sapcc-bot has joined #openstack-nova | 08:29 | |
*** jangutter has joined #openstack-nova | 08:29 | |
*** sapcc-bot has quit IRC | 08:29 | |
*** gjayavelu has quit IRC | 08:30 | |
*** markvoelker has joined #openstack-nova | 08:31 | |
*** thorst has quit IRC | 08:32 | |
*** lucas-afk is now known as lucasagomes | 08:32 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix test_volume_swap_server instability https://review.openstack.org/484288 | 08:38 |
*** sapcc-bot has joined #openstack-nova | 08:41 | |
*** sapcc-bot has quit IRC | 08:41 | |
*** tesseract has quit IRC | 08:50 | |
openstackgerrit | Merged openstack/nova master: conf: fix netconf, my_ip and host are unclear https://review.openstack.org/441210 | 08:51 |
*** sapcc-bot has joined #openstack-nova | 08:52 | |
*** sapcc-bot has quit IRC | 08:52 | |
*** tesseract has joined #openstack-nova | 08:53 | |
*** mamandle has joined #openstack-nova | 08:54 | |
*** cdent has joined #openstack-nova | 08:56 | |
*** phuongnh has quit IRC | 08:58 | |
*** sapcc-bot has joined #openstack-nova | 08:58 | |
*** sapcc-bot has quit IRC | 08:58 | |
mamandle | @alex_xu I responded to your comments on https://review.openstack.org/#/c/483911/2, can you please check? Thanks. | 09:03 |
*** sapcc-bot has joined #openstack-nova | 09:04 | |
*** sapcc-bot has quit IRC | 09:04 | |
*** markvoelker has quit IRC | 09:05 | |
*** dixiaoli has joined #openstack-nova | 09:07 | |
*** dixiaoli has quit IRC | 09:07 | |
*** yamamoto has joined #openstack-nova | 09:13 | |
*** yamamoto has quit IRC | 09:13 | |
*** yamamoto has joined #openstack-nova | 09:14 | |
*** yamamoto has quit IRC | 09:16 | |
*** yamamoto has joined #openstack-nova | 09:17 | |
openstackgerrit | Manas Mandlekar proposed openstack/nova master: Instance remains in migrating state forever https://review.openstack.org/483911 | 09:20 |
*** yamamoto has quit IRC | 09:21 | |
*** yamamoto has joined #openstack-nova | 09:21 | |
*** derekh has joined #openstack-nova | 09:22 | |
*** sambetts|afk is now known as sambetts | 09:25 | |
*** yamamoto has quit IRC | 09:26 | |
*** sapcc-bot has joined #openstack-nova | 09:28 | |
*** sapcc-bot has quit IRC | 09:28 | |
*** tuanluong has joined #openstack-nova | 09:28 | |
*** sapcc-bot has joined #openstack-nova | 09:29 | |
*** sapcc-bot has quit IRC | 09:29 | |
*** yamamoto has joined #openstack-nova | 09:30 | |
*** sapcc-bot has joined #openstack-nova | 09:34 | |
*** sapcc-bot has quit IRC | 09:34 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | 09:34 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 09:34 |
openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.50 - fix quota class sets resource usage https://review.openstack.org/483060 | 09:38 |
*** yamamoto has quit IRC | 09:38 | |
*** sapcc-bot has joined #openstack-nova | 09:40 | |
*** sapcc-bot has quit IRC | 09:40 | |
*** lpetrut has joined #openstack-nova | 09:40 | |
*** takedakn has joined #openstack-nova | 09:41 | |
*** yufei has quit IRC | 09:45 | |
*** mdnadeem has quit IRC | 09:47 | |
*** sapcc-bot has joined #openstack-nova | 09:52 | |
*** sapcc-bot has quit IRC | 09:52 | |
openstackgerrit | Merged openstack/python-novaclient master: Add support for the 2.51 microversion https://review.openstack.org/483149 | 09:57 |
*** sapcc-bot has joined #openstack-nova | 09:58 | |
*** sapcc-bot has quit IRC | 09:58 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 10:01 |
*** markvoelker has joined #openstack-nova | 10:02 | |
*** sapcc-bot has joined #openstack-nova | 10:05 | |
*** sapcc-bot has quit IRC | 10:05 | |
*** gcb has quit IRC | 10:06 | |
*** trungnv has quit IRC | 10:06 | |
*** liujiong has quit IRC | 10:06 | |
*** jistr is now known as jistr|afk | 10:09 | |
*** sapcc-bot has joined #openstack-nova | 10:10 | |
*** sapcc-bot has quit IRC | 10:10 | |
*** sapcc-bot has joined #openstack-nova | 10:10 | |
*** sapcc-bot has quit IRC | 10:10 | |
*** udesale__ has joined #openstack-nova | 10:12 | |
*** udesale has quit IRC | 10:15 | |
*** edmondsw has joined #openstack-nova | 10:15 | |
*** sapcc-bot has joined #openstack-nova | 10:16 | |
*** sapcc-bot has quit IRC | 10:16 | |
*** sapcc-bot has joined #openstack-nova | 10:16 | |
*** sapcc-bot has quit IRC | 10:16 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 10:18 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764 | 10:18 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 10:18 |
*** edmondsw has quit IRC | 10:19 | |
*** sapcc-bot has joined #openstack-nova | 10:21 | |
*** sapcc-bot has quit IRC | 10:21 | |
*** sapcc-bot has joined #openstack-nova | 10:22 | |
*** sapcc-bot has quit IRC | 10:22 | |
*** trungnv has joined #openstack-nova | 10:23 | |
*** ecelik has joined #openstack-nova | 10:24 | |
*** thorst has joined #openstack-nova | 10:28 | |
*** thorst has quit IRC | 10:32 | |
*** mvk has quit IRC | 10:33 | |
*** ecelik has left #openstack-nova | 10:33 | |
*** cdent has quit IRC | 10:33 | |
*** sapcc-bot has joined #openstack-nova | 10:34 | |
*** sapcc-bot has quit IRC | 10:34 | |
*** markvoelker has quit IRC | 10:35 | |
*** aloga has quit IRC | 10:36 | |
*** aloga has joined #openstack-nova | 10:36 | |
*** yamamoto has joined #openstack-nova | 10:37 | |
*** kiennt_ has quit IRC | 10:37 | |
*** tuanluong has quit IRC | 10:38 | |
*** udesale__ has quit IRC | 10:39 | |
*** udesale has joined #openstack-nova | 10:40 | |
*** owalsh has quit IRC | 10:40 | |
*** owalsh has joined #openstack-nova | 10:42 | |
*** tbachman has quit IRC | 10:46 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_force_complete notification https://review.openstack.org/480561 | 10:46 |
*** lucasagomes is now known as lucas-hungry | 10:49 | |
*** sridharg has joined #openstack-nova | 10:50 | |
*** tbachman has joined #openstack-nova | 10:50 | |
*** fragatina has joined #openstack-nova | 10:57 | |
*** lpetrut_ has joined #openstack-nova | 10:57 | |
*** sapcc-bot has joined #openstack-nova | 10:57 | |
*** sapcc-bot has quit IRC | 10:57 | |
*** fragatina has quit IRC | 10:57 | |
*** sapcc-bot has joined #openstack-nova | 10:58 | |
*** sapcc-bot has quit IRC | 10:58 | |
*** fragatina has joined #openstack-nova | 10:58 | |
*** lpetrut has quit IRC | 11:00 | |
*** mvk has joined #openstack-nova | 11:00 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (3) https://review.openstack.org/452820 | 11:01 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data https://review.openstack.org/452818 | 11:01 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Factor out duplicated notification sample data (2) https://review.openstack.org/452819 | 11:01 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor instance.power-off notification samples https://review.openstack.org/475860 | 11:01 |
*** alexchadin has quit IRC | 11:04 | |
*** Guest55861 has left #openstack-nova | 11:06 | |
*** thorst has joined #openstack-nova | 11:08 | |
*** thorst has quit IRC | 11:09 | |
*** sapcc-bot has joined #openstack-nova | 11:09 | |
*** sapcc-bot has quit IRC | 11:09 | |
*** cdent has joined #openstack-nova | 11:14 | |
*** sapcc-bot has joined #openstack-nova | 11:15 | |
*** sapcc-bot has quit IRC | 11:15 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | 11:15 |
*** sapcc-bot has joined #openstack-nova | 11:16 | |
*** thorst has joined #openstack-nova | 11:16 | |
*** mamandle has quit IRC | 11:23 | |
*** hoonetorg has joined #openstack-nova | 11:32 | |
*** markvoelker has joined #openstack-nova | 11:32 | |
*** trinaths1 has quit IRC | 11:33 | |
*** ircuser-1 has quit IRC | 11:35 | |
openstackgerrit | Merged openstack/nova master: trivial: Remove unnecessary function https://review.openstack.org/483433 | 11:35 |
*** gszasz has joined #openstack-nova | 11:37 | |
*** udesale has quit IRC | 11:37 | |
*** sapcc-bot has joined #openstack-nova | 11:39 | |
*** sapcc-bot has quit IRC | 11:39 | |
*** ociuhandu has quit IRC | 11:46 | |
*** jjung has quit IRC | 11:47 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 11:48 |
*** ratailor has quit IRC | 11:50 | |
*** sapcc-bot has joined #openstack-nova | 11:51 | |
*** sapcc-bot has quit IRC | 11:51 | |
*** litao__ has quit IRC | 11:52 | |
*** mkucia has joined #openstack-nova | 11:55 | |
cdent | edleafe: yeah, I got pretty confused on what the right thing to do throughout 500 fix. It made think: here be edges and confusion | 11:55 |
*** smatzek has joined #openstack-nova | 11:56 | |
*** markvoelker has quit IRC | 11:56 | |
*** markvoelker has joined #openstack-nova | 11:57 | |
*** sapcc-bot has joined #openstack-nova | 12:02 | |
*** sapcc-bot has quit IRC | 12:02 | |
*** takedakn has quit IRC | 12:03 | |
*** fragatina has quit IRC | 12:05 | |
*** vladikr has joined #openstack-nova | 12:06 | |
*** ociuhandu has joined #openstack-nova | 12:10 | |
*** tylerderosagrund has joined #openstack-nova | 12:12 | |
*** ircuser-1 has joined #openstack-nova | 12:15 | |
*** jistr|afk is now known as jistr | 12:17 | |
*** tylerderosagrund has quit IRC | 12:24 | |
*** tylerderosagrund has joined #openstack-nova | 12:24 | |
*** ekuris_ has quit IRC | 12:25 | |
*** ekuris has joined #openstack-nova | 12:26 | |
*** MVenesio has joined #openstack-nova | 12:28 | |
*** sapcc-bot has joined #openstack-nova | 12:30 | |
*** sapcc-bot has quit IRC | 12:30 | |
*** dtantsur has joined #openstack-nova | 12:33 | |
*** jaypipes has joined #openstack-nova | 12:33 | |
*** prateek has quit IRC | 12:33 | |
*** edmondsw has joined #openstack-nova | 12:34 | |
*** edmondsw has quit IRC | 12:34 | |
*** edmondsw has joined #openstack-nova | 12:35 | |
*** sapcc-bot has joined #openstack-nova | 12:36 | |
*** sapcc-bot has quit IRC | 12:36 | |
*** jjung has joined #openstack-nova | 12:37 | |
*** kbaegis has joined #openstack-nova | 12:39 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35 https://review.openstack.org/484356 | 12:39 |
cdent | edleafe, jaypipes ^ | 12:39 |
cdent | may be of interest | 12:39 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | 12:42 |
*** mamandle has joined #openstack-nova | 12:42 | |
*** jjung has quit IRC | 12:44 | |
openstackgerrit | Dmitri Plakhov proposed openstack/nova master: Added Oberbooking quota driver https://review.openstack.org/484359 | 12:44 |
*** lyan has joined #openstack-nova | 12:45 | |
*** sapcc-bot has joined #openstack-nova | 12:47 | |
*** sapcc-bot has quit IRC | 12:47 | |
*** jjung has joined #openstack-nova | 12:47 | |
*** BobBall has joined #openstack-nova | 12:47 | |
*** efoley has joined #openstack-nova | 12:48 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 12:49 |
*** catintheroof has joined #openstack-nova | 12:51 | |
*** udesale has joined #openstack-nova | 12:52 | |
*** Jack_Iv has joined #openstack-nova | 12:52 | |
*** prateek has joined #openstack-nova | 12:53 | |
*** pchavva has joined #openstack-nova | 12:53 | |
Jack_Iv | Hi! could anyone suggest something? https://bugs.launchpad.net/nova/+bug/1704788 | 12:53 |
openstack | Launchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,New] | 12:53 |
*** mtanino_ has joined #openstack-nova | 12:56 | |
*** alexchadin has joined #openstack-nova | 12:57 | |
*** Sara_ has joined #openstack-nova | 12:59 | |
Sara_ | Hello all... I have booted a VM using "nova boot".. After the Vm is in active state.. I have done the following: | 13:00 |
Sara_ | virsh destroy <instance> | 13:00 |
Sara_ | virsh undefine "<instance>" | 13:00 |
*** gouthamr has joined #openstack-nova | 13:01 | |
Sara_ | In nova list, I could see "VM in Shutoff" state | 13:01 |
Sara_ | The VM is not deleted from nova database and resources are not released.. | 13:02 |
Sara_ | Is this the expected behaviour? | 13:02 |
bauzas | Sara_: it's not accepted | 13:02 |
*** fried_rice is now known as efried | 13:02 | |
bauzas | Sara_: Nova doesn't support operators killing VMs directly | 13:03 |
bauzas | instead, you need to "nova delete" your instance | 13:03 |
*** edmondsw_ has joined #openstack-nova | 13:04 | |
*** sapcc-bot has joined #openstack-nova | 13:04 | |
*** sapcc-bot has quit IRC | 13:04 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35 https://review.openstack.org/484356 | 13:04 |
Sara_ | bauzas_: But what happens in the scenario where compute host is removed from the openstack setup and rejoined ? | 13:05 |
bauzas | Sara_: you can still delete the instances | 13:05 |
bauzas | Sara_: even if the compute is not there | 13:05 |
*** jwcroppe has quit IRC | 13:05 | |
Sara_ | Does libvirt update the "undefine" event to nova? | 13:06 |
bauzas | Sara_: when the compute will be back, it will call the nova DB to know which instances are deleted, and then will destroy the libvirt VMs | 13:06 |
*** gcb has joined #openstack-nova | 13:06 | |
efried | Anyone savvy on nova microversion support timeframes? In particular, is pre-2.27 still supported? | 13:07 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform HostAPI-set_enabled notifications https://review.openstack.org/482622 | 13:07 |
*** edmondsw has quit IRC | 13:07 | |
Sara_ | bauzas_: Does libvirt update the "undefine" event to nova? I want to get an update from libvirt to nova when the domain gets undefined..Is it supported in Mitaka? | 13:08 |
cdent | efried: the removal of nova-net kind of broke the microversion contract (which is effectively that they are around for-ever) | 13:09 |
*** lucas-hungry is now known as lucasagomes | 13:09 | |
*** baoli has joined #openstack-nova | 13:10 | |
efried | cdent Okay, so we actually need to handle https://review.openstack.org/#/c/482746/10/keystoneauth1/session.py@539 this? | 13:10 |
cdent | urgh. I don’t know, that’s a good question. Some combination of mordred, mriedem and sdague may have ideas | 13:11 |
*** acormier has joined #openstack-nova | 13:12 | |
openstackgerrit | Chris Dent proposed openstack/nova master: [placement] fix 500 error when allocating to bad class https://review.openstack.org/484162 | 13:13 |
sdague | efried: honestly, there probably should be special case code in there to handle the compute case | 13:13 |
sdague | if the point is to support what's out there | 13:13 |
efried | sdague And if I understand the docs right, we can just send both headers down and nova will pick the new one if it's post 2.27, yes? | 13:14 |
*** baoli has quit IRC | 13:14 | |
sdague | efried: yep | 13:15 |
efried | k | 13:15 |
*** imacdonn has quit IRC | 13:16 | |
*** sree has joined #openstack-nova | 13:16 | |
*** sapcc-bot has joined #openstack-nova | 13:16 | |
*** sapcc-bot has quit IRC | 13:16 | |
*** mmehan has joined #openstack-nova | 13:16 | |
openstackgerrit | Merged openstack/nova master: fix unshelve notification test instability https://review.openstack.org/483986 | 13:18 |
*** ekuris has quit IRC | 13:18 | |
*** jwcroppe has joined #openstack-nova | 13:20 | |
*** sapcc-bot has joined #openstack-nova | 13:21 | |
*** sapcc-bot has quit IRC | 13:21 | |
*** acormier has quit IRC | 13:21 | |
*** zhurong has quit IRC | 13:22 | |
*** scottda has joined #openstack-nova | 13:23 | |
bauzas | stephenfin: not sure you're there, or if someone can answer me this question, but where can I find upstream docs about PCI passthru and SR-IOV ? | 13:24 |
jaypipes | cdent: nice! +2 from me. :) | 13:25 |
cdent | coo, thanks | 13:25 |
*** lbragstad has quit IRC | 13:26 | |
*** baoli has joined #openstack-nova | 13:26 | |
*** udesale has quit IRC | 13:27 | |
mordred | cdent: reading | 13:28 |
*** gcb has quit IRC | 13:28 | |
cdent | mordred: I think sdague pretty much nailed it: compute needs a special case. Unclear about other services. | 13:28 |
efried | cdent mordred Comments in the review. | 13:29 |
mordred | yah. agree - I don't think any other services made it out the door with that form, did they? | 13:29 |
bauzas | Sara_: like I said earlier, if you undefine a libvirt VM, Nova can just say that the instance is shutdown, because it can't find it | 13:30 |
bauzas | Sara_: there is no 'callback' from a specific virt driver for updating some internal Nova status about instances | 13:31 |
sdague | mordred: the only other one that might have was ironic | 13:31 |
bauzas | Sara_: that's just nova which periodically monitors host states wrt instances | 13:31 |
sdague | I think we got standardization before anything else adopted it | 13:31 |
*** sapcc-bot has joined #openstack-nova | 13:32 | |
*** sapcc-bot has quit IRC | 13:32 | |
bauzas | cdent: is there a dependent change for updating requirements to gabbi>=1.35 ? | 13:33 |
*** superdan is now known as dansmith | 13:33 | |
cdent | bauzas: it’s already merged | 13:33 |
bauzas | cdent: talking of https://review.openstack.org/#/c/484356/2/test-requirements.txt | 13:33 |
*** markvoelker_ has joined #openstack-nova | 13:33 | |
cdent | https://review.openstack.org/#/c/481665/ | 13:33 |
*** mriedem has joined #openstack-nova | 13:33 | |
*** markvoelker has quit IRC | 13:33 | |
*** ekuris has joined #openstack-nova | 13:34 | |
bauzas | cdent: oh ok, so I guess gabbi-1.35 supports legacy behaviour | 13:34 |
*** edmondsw has joined #openstack-nova | 13:34 | |
cdent | bauzas: yes, it follows semantic versioning | 13:34 |
bauzas | cdent: WAT? impossible :p | 13:35 |
cdent | ikr! | 13:35 |
bauzas | hehe | 13:35 |
bauzas | cdent: anyway, the gate works quite good today, so I guess you did your job correctly :p | 13:36 |
*** edmondsw_ has quit IRC | 13:36 | |
*** ociuhandu has quit IRC | 13:36 | |
*** ociuhandu has joined #openstack-nova | 13:36 | |
cdent | gabbi tests itself on gnocchi’s use of gabbi, I could probably add the same for nova’s use of gabbi too | 13:37 |
* cdent makes a note of it | 13:37 | |
*** links has quit IRC | 13:37 | |
*** Kris has joined #openstack-nova | 13:38 | |
*** Kris has left #openstack-nova | 13:38 | |
*** sapcc-bot has joined #openstack-nova | 13:38 | |
*** sapcc-bot has quit IRC | 13:38 | |
*** markvoelker_ has quit IRC | 13:38 | |
*** markvoelker has joined #openstack-nova | 13:39 | |
edleafe | Scheduler subteam meeting in 20 minutes in #openstack-meeting-alt | 13:40 |
efried | mordred Much as I hate to suggest further dirtying up sta, perhaps it should have legacy-microversion-header and legacy-microversion-header-before-version fields. | 13:42 |
mriedem | gmann: alex_xu: was anyone going to post a change for fixing the docs issues in https://review.openstack.org/#/c/467999/ ? if not i could work on that today | 13:43 |
*** sapcc-bot has joined #openstack-nova | 13:43 | |
*** sapcc-bot has quit IRC | 13:43 | |
*** gcb has joined #openstack-nova | 13:44 | |
*** udesale has joined #openstack-nova | 13:45 | |
*** cdent_ has joined #openstack-nova | 13:45 | |
efried | sdague mordred I can't even find anything about ironic that suggests microversion support *other* than via X-OpenStack-Ironic-API-Version | 13:46 |
efried | Was looking for a statement like this one that nova has: https://developer.openstack.org/api-ref/compute/#api-versions | 13:47 |
*** cdent has quit IRC | 13:47 | |
*** cdent_ is now known as cdent | 13:47 | |
sdague | efried: oh, maybe they predated and never fixed it | 13:47 |
*** xyang1 has joined #openstack-nova | 13:47 | |
*** sapcc-bot has joined #openstack-nova | 13:48 | |
*** sapcc-bot has quit IRC | 13:48 | |
*** beekneemech is now known as bnemec | 13:48 | |
*** gcb has quit IRC | 13:48 | |
*** gcb has joined #openstack-nova | 13:49 | |
*** yamamoto has quit IRC | 13:49 | |
*** sree has quit IRC | 13:50 | |
*** esberglu has joined #openstack-nova | 13:50 | |
*** kylek3h has joined #openstack-nova | 13:50 | |
*** ekuris has quit IRC | 13:50 | |
*** felipemonteiro has joined #openstack-nova | 13:51 | |
*** sbezverk has quit IRC | 13:51 | |
alex_xu | mriedem: here is https://review.openstack.org/483170 | 13:51 |
alex_xu | sorry, I forget to tell you the link | 13:51 |
*** sbezverk has joined #openstack-nova | 13:51 | |
*** felipemonteiro__ has joined #openstack-nova | 13:53 | |
*** stvnoyes has quit IRC | 13:53 | |
*** sapcc-bot has joined #openstack-nova | 13:53 | |
*** sapcc-bot has quit IRC | 13:53 | |
*** crushil has joined #openstack-nova | 13:53 | |
mordred | efried: ugh | 13:53 |
efried | agree | 13:54 |
*** sapcc-bot has joined #openstack-nova | 13:54 | |
*** sapcc-bot has quit IRC | 13:54 | |
*** awaugama has joined #openstack-nova | 13:54 | |
*** stvnoyes has joined #openstack-nova | 13:54 | |
cdent | mordred, efried: sorry not sorry | 13:55 |
efried | cdent I think you're off the hook - it was cmurphy brought it up. | 13:56 |
openstackgerrit | Steven Webster proposed openstack/nova master: Fix instance evacuation with PCI devices https://review.openstack.org/484381 | 13:56 |
*** damien_r has left #openstack-nova | 13:56 | |
mordred | efried: another option is to not support legacy headers in the ksa microversion helper method, since sending this at all is a new ksa feature in the first place - and we can have python-novaclient and python-ironicclient also add the old headers if they want to | 13:56 |
cdent | efried: I was the person who proposed the new form | 13:56 |
*** felipemonteiro has quit IRC | 13:56 | |
efried | cdent Oh, the generic OpenStack-API-Version ? | 13:56 |
cdent | mordred: I’d like that, but is that fair? | 13:56 |
mordred | I'm not crazy about either option tbh - just bringing it up | 13:56 |
cdent | efried: yes\ | 13:56 |
mordred | well - my concern is that there's no way for the user to know then that get('/servers', microversion='2.13') isn't going to work | 13:57 |
efried | cdent Yeah, it's the better way of doing it. You just got there too late. | 13:57 |
jangutter | bauzas: do you still need docs about PCI passthrough and SR-IOV? | 13:57 |
cdent | story of my life | 13:57 |
efried | mordred Right, python-*client isn't necessarily the only API consumer, right? | 13:57 |
mordred | cdent: the original form is basically unworkable in general - so thank you very much for making it the change | 13:57 |
mordred | efried: exactly | 13:58 |
efried | mordred So we hardcode it in ksa... or make it part of sta. | 13:58 |
efried | Cause I really don't like that other option - making "consumers" know about this quirk. | 13:58 |
*** gbarros has quit IRC | 13:58 | |
*** zhurong has joined #openstack-nova | 13:59 | |
bauzas | jangutter: I'd appreciate, yes | 13:59 |
*** zhurong has quit IRC | 13:59 | |
bauzas | jangutter: I mean, I can find other docs, and I know most of that | 13:59 |
bauzas | jangutter: but we had some upstream docs that give me now 404s and I'd like to make sure we still have those somewhere | 14:00 |
jangutter | bauzas: I always refer back to some very old IOMMU/VFIO/PCI-ASSIGN presentation waay back when. | 14:00 |
bauzas | I know we had some admin-guide doc | 14:01 |
*** gbarros has joined #openstack-nova | 14:01 | |
bauzas | but those are giving me 404s because of the doc migration | 14:01 |
mordred | efried: yah. me either. although maybe the answer for today is to not add the helper method in this release since this is getting in to a kind-of ugly place and needs some thought/design | 14:01 |
jangutter | bauzas: aah, end-user stuff. | 14:01 |
*** sree has joined #openstack-nova | 14:01 | |
edleafe | cdent: bauzas: scheduler meeting now running | 14:01 |
bauzas | edleafe: snap, caught by the police :p | 14:02 |
mordred | efried: or - we make sure that the "add microversion helper parameter" bit to the end of the stack so that if we can't all agree on the approach we don't block anything else | 14:02 |
jangutter | bauzas: https://wiki.openstack.org/wiki/SR-IOV-Passthrough-For-Networking <--- very stale. | 14:02 |
efried | mordred Probably at least that last thing. | 14:03 |
bauzas | asettle: hola, unsure if you can answer my question | 14:03 |
efried | mordred Cause I imagine it may take a little while to figure out all the special cases, whatever approach we land on. | 14:04 |
jangutter | bauzas: and the one for Ocata: https://docs.openstack.org/ocata/networking-guide/config-sriov.html <--- something like this, or do you want more low-level? | 14:04 |
bauzas | asettle: but I can see a 404 on http://docs.openstack.org/admin-guide/compute-pci-passthrough.html | 14:04 |
bauzas | jangutter: yeah, that will work out | 14:04 |
mordred | efried: yes - and I think we need to get a ksa release cut that hasall the other things this week so that it makes the cutoff and you can use it in nova | 14:05 |
bauzas | jangutter: it's not really for my knowledge, it's rather more about providing guidance to others | 14:05 |
efried | mordred Yes, and I'm a tad worried about that. Working hard on getting https://review.openstack.org/#/c/483604/ polished off, cause that affects the initial APIs of the new methods. | 14:05 |
efried | mordred Also concerned about getting that rename (min/max_version => min/max_endpoint_version) in place for same reason. | 14:06 |
*** lucasxu has joined #openstack-nova | 14:06 | |
mordred | efried: ++ | 14:06 |
efried | mordred Prolly should move this conversation to -dev/-sdks | 14:06 |
*** sree_ has joined #openstack-nova | 14:07 | |
*** sree_ is now known as Guest35636 | 14:07 | |
*** yamamoto has joined #openstack-nova | 14:09 | |
*** yamamoto has quit IRC | 14:09 | |
*** rnoriega_ has joined #openstack-nova | 14:09 | |
*** sree has quit IRC | 14:10 | |
*** sapcc-bot has joined #openstack-nova | 14:10 | |
*** sapcc-bot has quit IRC | 14:10 | |
*** hongbin has joined #openstack-nova | 14:11 | |
*** alexchadin has quit IRC | 14:11 | |
openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | 14:12 |
openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | 14:12 |
*** adisky__ has quit IRC | 14:12 | |
jaypipes | edleafe: done ^ | 14:12 |
*** Jack_Iv has quit IRC | 14:12 | |
edleafe | jaypipes: gracias | 14:12 |
openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | 14:13 |
*** rnoriega_ has quit IRC | 14:16 | |
*** burt has joined #openstack-nova | 14:19 | |
*** acormier has joined #openstack-nova | 14:23 | |
*** udesale has quit IRC | 14:24 | |
*** jwcroppe has quit IRC | 14:28 | |
asettle | bauzas: uno momento, in a meeting :) will look shortly | 14:28 |
asettle | bauzas: admin guide has been migrated | 14:29 |
bauzas | k | 14:29 |
asettle | Y'all need to migrate the Comptue section of the admin guide into the nova repo | 14:29 |
*** jwcroppe has joined #openstack-nova | 14:29 | |
*** lbragstad has joined #openstack-nova | 14:29 | |
*** marst has quit IRC | 14:30 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2 https://review.openstack.org/484386 | 14:31 |
*** sapcc-bot has joined #openstack-nova | 14:33 | |
*** sapcc-bot has quit IRC | 14:33 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2 https://review.openstack.org/484386 | 14:34 |
ralonsoh | stephenfin: hi! A present: https://review.openstack.org/#/c/484386/ | 14:35 |
ralonsoh | stephenfin: BTW. I'm running out of time. Can you take a look at https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/enable-sriov-nic-features? | 14:36 |
mdrabe | dansmith: I think https://review.openstack.org/#/c/482620 should be bueno | 14:36 |
*** gyee has joined #openstack-nova | 14:37 | |
dansmith | mdrabe: indeed, thanks | 14:37 |
openstackgerrit | Artem Tiumentcev proposed openstack/nova master: Add root and ephemeral disk quota https://review.openstack.org/484390 | 14:39 |
*** esberglu has quit IRC | 14:40 | |
*** yamamoto has joined #openstack-nova | 14:40 | |
*** mtanino_ has quit IRC | 14:41 | |
openstackgerrit | Artem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support https://review.openstack.org/484391 | 14:43 |
*** yamamoto has quit IRC | 14:46 | |
*** mtanino_ has joined #openstack-nova | 14:46 | |
*** marst has joined #openstack-nova | 14:46 | |
*** psachin has quit IRC | 14:48 | |
*** chyka has joined #openstack-nova | 14:52 | |
*** armax has joined #openstack-nova | 14:53 | |
*** mamandle has quit IRC | 14:53 | |
*** esberglu has joined #openstack-nova | 14:55 | |
*** sapcc-bot has joined #openstack-nova | 14:56 | |
*** sapcc-bot has quit IRC | 14:56 | |
*** esberglu has quit IRC | 14:56 | |
*** esberglu has joined #openstack-nova | 14:57 | |
*** jwcroppe has quit IRC | 14:58 | |
*** jjung has quit IRC | 14:58 | |
*** armax has quit IRC | 14:58 | |
*** yamahata has joined #openstack-nova | 14:59 | |
*** Guest35636 has quit IRC | 14:59 | |
*** lucasxu has quit IRC | 15:00 | |
*** cdent has quit IRC | 15:02 | |
openstackgerrit | Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | 15:02 |
*** sapcc-bot has joined #openstack-nova | 15:03 | |
*** armax has joined #openstack-nova | 15:06 | |
*** gyee has quit IRC | 15:06 | |
*** gyee has joined #openstack-nova | 15:08 | |
mriedem | sdague: can you go through this new style cinder attachments swap volume change? https://review.openstack.org/#/c/456971/ - i'm good with it and jay is +2, | 15:09 |
mriedem | sdague: we really need another core to help review the new style cinder attachments stuff, so if you could help here this week it'd be really good | 15:09 |
*** andreas_s has quit IRC | 15:10 | |
mriedem | keep in mind, everything is keyed off the bdm.attachment_id field being set - which doesn't happen until the very last patch where we turn this all on - which only happens when (1) cinder is new enough to provide 3.27 and (2) all computes are upgraded | 15:10 |
*** Oku_OS is now known as Oku_OS-away | 15:11 | |
gibi | mriedem: hi! I have a fix up for the volume_swap notification test instability https://review.openstack.org/#/c/484288/ | 15:11 |
*** mamandle has joined #openstack-nova | 15:11 | |
mriedem | gibi: thanks, +2 | 15:13 |
*** ociuhandu has quit IRC | 15:14 | |
*** trinaths has joined #openstack-nova | 15:20 | |
gibi | mriedem: thanks | 15:21 |
openstackgerrit | Artem Tiumentcev proposed openstack/python-novaclient master: Add 'local_gb' disk quota support https://review.openstack.org/484391 | 15:23 |
mriedem | sfinucan: i'm debugging a fun issue with NUMATopologyLimits not getting serialized in the scheduler with a new oslo.serialization change | 15:25 |
mriedem | http://logs.openstack.org/87/482287/6/check/gate-cross-nova-func-ubuntu-xenial/a645ce8/console.html#_2017-07-16_14_02_02_649384 | 15:25 |
mriedem | that's due to this change in oslo.serialization https://review.openstack.org/#/c/327564/ | 15:25 |
mriedem | which is failing here: https://review.openstack.org/#/c/482287/ | 15:26 |
*** lucasxu has joined #openstack-nova | 15:27 | |
mriedem | i'm trying to sort out where this is getting serializaed | 15:27 |
mriedem | *serialized | 15:27 |
mriedem | obviously the select_destinations call | 15:27 |
*** stvnoyes has quit IRC | 15:28 | |
mriedem | oh i think i see, | 15:28 |
mriedem | the request spec is serialized, and contains numa_topology which comes from the HostState object: | 15:28 |
mriedem | host_state.limits['numa_topology'] = limits | 15:29 |
mriedem | set via the NUMATopologyFilter | 15:29 |
mriedem | still not sure why that would fail to serialize | 15:31 |
*** annegentle has joined #openstack-nova | 15:31 | |
jangutter | newbie question about the gate: is there a way to inform it that the unit tests require an updated os-vif? | 15:32 |
mriedem | jangutter: depends-on | 15:33 |
mriedem | buth, | 15:33 |
mriedem | *but, | 15:33 |
*** sree has joined #openstack-nova | 15:33 | |
mriedem | you have to get the changes merged and released in os-vif first | 15:33 |
mriedem | you can't depends-on a source change in a library w/o it being released first in most jobs | 15:33 |
jangutter | mriedem: yah, thought as much. | 15:33 |
*** ociuhandu has joined #openstack-nova | 15:35 | |
*** jdillaman has joined #openstack-nova | 15:35 | |
jangutter | mriedem: funnily enough, the non-test code seem to load the patched os-vif. | 15:36 |
*** annegentle has quit IRC | 15:36 | |
*** trinaths has quit IRC | 15:36 | |
jangutter | mriedem: I presume it's because tox creates a sandbox. | 15:36 |
*** jwcroppe has joined #openstack-nova | 15:37 | |
*** sree has quit IRC | 15:37 | |
*** annegentle has joined #openstack-nova | 15:38 | |
mriedem | i'm not really understanding what you're saying | 15:40 |
*** david-lyle has joined #openstack-nova | 15:40 | |
mriedem | it's all going to run in a tox venv with dependencies using the versions from upper-constraints | 15:40 |
mriedem | https://github.com/openstack/requirements/blob/master/upper-constraints.txt | 15:40 |
*** stvnoyes has joined #openstack-nova | 15:40 | |
mriedem | so until the change in os-vif is released and in upper-constraints, then the tox venv isn't using it | 15:41 |
mriedem | at least in the CI system | 15:41 |
mriedem | so either your local venv is hacked or it's not working the way you think | 15:41 |
*** MVenesio has quit IRC | 15:42 | |
jangutter | mriedem: Yeah, I hacked my local venv (by running install on os-vif), but only for the unit tests. | 15:42 |
*** ckopper has joined #openstack-nova | 15:42 | |
mriedem | sfinucan: this is what we get right before select_destinations blows up http://paste.openstack.org/show/615631/ | 15:43 |
sdague | mriedem: will look | 15:44 |
mriedem | sdague: thanks, poke me with questions | 15:44 |
*** aloga_ has joined #openstack-nova | 15:46 | |
bauzas | jaypipes: dansmith: mriedem: sorry, was off last friday, but just saw https://review.openstack.org/#/c/483564/2 | 15:46 |
bauzas | jaypipes: dansmith: mriedem: so we're now passing a full list of weighed hosts to _schedule() even if don't really use those | 15:47 |
bauzas | jaypipes: dansmith: mriedem: I get of course why we need to do such in later changes but I do wonder of the performance penalty | 15:47 |
mriedem | huh? | 15:48 |
bauzas | jaypipes: dansmith: mriedem: the performance penalty being for example a list of 1000 nodes | 15:48 |
mriedem | what's getting passed to _schedule that you are worried about? | 15:48 |
bauzas | given we don't pass an iterator but a full ist | 15:48 |
bauzas | list | 15:48 |
bauzas | mriedem: https://review.openstack.org/#/c/483564/2/nova/scheduler/filter_scheduler.py@150 | 15:48 |
bauzas | that's a list, not an iterator | 15:48 |
mriedem | you mean not what's passed to _schedule, you mean what's passed from _get_sorted_hosts and used in _schedule | 15:49 |
bauzas | so we literally build in memory an object that could be large just for the purpose of selecting only the first item | 15:49 |
bauzas | yup, that | 15:49 |
bauzas | I'd frankly be more happy if that was an iterator | 15:49 |
bauzas | but I could be picky | 15:50 |
bauzas | just raising my concern out loud | 15:50 |
bauzas | not worth a revert of course | 15:50 |
mriedem | so push a change to do that | 15:50 |
mriedem | or leave a comment in the patch | 15:50 |
bauzas | mriedem: I can workout a patch | 15:50 |
*** ociuhandu has quit IRC | 15:50 | |
*** ociuhandu has joined #openstack-nova | 15:52 | |
bauzas | jaypipes: mriedem: oh looks like https://review.openstack.org/#/c/483564/2 will not land because of a functional test issue | 15:54 |
mriedem | i see the "# Note: remember, we are using an iterator here." on L93 but which variable is the iterator? is it 'hosts' returned from _get_all_host_states ? | 15:54 |
mriedem | because i think that returns a tuple | 15:54 |
*** aloga_ has quit IRC | 15:54 | |
mriedem | HostManager.get_host_states_by_uuids calls _get_host_states which returns a tuple | 15:55 |
*** mkucia has quit IRC | 15:55 | |
mriedem | I think get_filtered_hosts is what converts to an iterator | 15:56 |
mriedem | hosts = six.itervalues(name_to_cls_map) | 15:56 |
*** aloga_ has joined #openstack-nova | 15:56 | |
bauzas | mriedem: the whole filters/weighers accepts an iterator | 15:56 |
bauzas | AFAIR | 15:56 |
mriedem | and passes that to filter_handler.get_filtered_objects | 15:56 |
bauzas | mriedem: yup | 15:56 |
mriedem | and ^ calls the filters | 15:57 |
bauzas | mriedem: but the thing is that we build a list once we get the weighed hosts | 15:57 |
*** hoonetorg has quit IRC | 15:57 | |
mriedem | doesn't get_weighed_hosts return a sorted list? | 15:58 |
*** rajathagasthya has joined #openstack-nova | 15:59 | |
mriedem | ew now i see that WeighedObject is a wrapper around a HostState - wonder why someone used a wrapper rather than subclass HostState with a weight field? | 15:59 |
mriedem | WeighedHostState(HostState) | 15:59 |
mriedem | that's why we have that ugly host.obj.foo all over | 15:59 |
*** fragatina has joined #openstack-nova | 16:00 | |
mriedem | anyway, i'm a bit lost in here wrt the iterator | 16:00 |
bauzas | yeah, seems like it's uglier than I thought | 16:01 |
mriedem | bauzas: the new _get_sorted_hosts method is still calling host_manager.get_filtered_hosts which still returns an iterator | 16:01 |
bauzas | mriedem: I was remembering we were using iterators, but looks like our filtering/weighing helpers work with lists... | 16:01 |
mriedem | and that iterator is still passed to host_manager.get_weighed_hosts | 16:01 |
mriedem | and weighed_hosts comes back as a sorted list | 16:01 |
mriedem | not an iterator | 16:02 |
mriedem | from what i can tell | 16:02 |
bauzas | yup | 16:02 |
mriedem | and by the time you get to chosen_host = hosts[0] it's really no different from before | 16:02 |
mriedem | so seems like something that doesn't matter to this change | 16:02 |
bauzas | I was just concerned by the new code taking more memory than previously, but it's just honestly not the case | 16:02 |
bauzas | yup | 16:02 |
bauzas | because we did work with iterators but once you enter filtering/weighting you're back to full lists of hosts anyway | 16:03 |
* bauzas shrugs | 16:03 | |
*** armax has quit IRC | 16:03 | |
bauzas | anyway, I'm now looking at why that patch failed | 16:03 |
bauzas | mmmmpf http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_769898 | 16:05 |
*** rajathagasthya has quit IRC | 16:06 | |
*** rajathagasthya has joined #openstack-nova | 16:07 | |
*** cdent has joined #openstack-nova | 16:07 | |
*** tylerderosagrund has quit IRC | 16:08 | |
*** tylerderosagrund has joined #openstack-nova | 16:08 | |
mriedem | shouldn't that also be running through other filters? | 16:09 |
jaypipes | mriedem, bauzas: that whole comment about it being an iterator is nonsense. As soon as it enters the filter host manager, it converts it all to a list: https://github.com/openstack/nova/blob/master/nova/filters.py#L69 | 16:09 |
mriedem | like ServerGroupAntiAffinityFilter | 16:09 |
mordred | mriedem, sdague: if a client sends the legacy microversion header to a new nova, that's not an error, right? and if it sends both, that's also acceptable? | 16:09 |
jaypipes | mriedem: it was always trying to be too clever for its own good, IMHO. | 16:09 |
bauzas | jaypipes: yup, just saw that, see above the IRC discussion with matt | 16:10 |
bauzas | jaypipes: honestly, I'm not surprised | 16:10 |
sdague | mordred: correct, either are accepted | 16:10 |
jaypipes | mriedem: btw, I was trying to get rid of the host.obj foo nonsense. | 16:10 |
sdague | as are both, I think when both we take the standard one as gospel | 16:10 |
mordred | sdague: awesome. I believe I have a general solution to our earlier problem | 16:10 |
bauzas | jaypipes: someone probably did care of passing iterators, but someone else probably messed it up by just using lists afterwards | 16:11 |
sdague | cdent: I think you wrote the acceptor code, right? | 16:11 |
mriedem | jaypipes: yeah i noted that in the commit message | 16:11 |
cdent | sdague: yeah, it’s happy with both, prefers the newer one | 16:11 |
mriedem | less turducken is good | 16:11 |
mordred | cdent: sweet | 16:11 |
bauzas | jaypipes: anyway, it's not super critical, but something is : your patch is getting -1 because http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_769898 | 16:11 |
*** lyan has quit IRC | 16:11 | |
cdent | it always sends both back in the response too | 16:11 |
jaypipes | bauzas: I can't reproduce that locally :( | 16:12 |
openstackgerrit | zhang zhenzhong proposed openstack/nova master: fix rp registration failed while placement endpoint created after compute start https://review.openstack.org/483460 | 16:12 |
bauzas | jaypipes: I was just about to run it :) | 16:13 |
mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22%2C%20in%20test_boot_servers_with_soft_anti_affinity_one_available_host%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | 16:13 |
bauzas | my child at home is killing my work because of netflix eating all the poor internet bandwidth FWIW | 16:13 |
bauzas | plex FTW | 16:14 |
bauzas | git pull is taking more than 1 min | 16:14 |
*** prateek has quit IRC | 16:14 | |
mriedem | fwiw, that test also failed in https://review.openstack.org/#/c/482288/ | 16:14 |
mriedem | which would be totally unrelated | 16:14 |
*** aloga_ has quit IRC | 16:15 | |
bauzas | jaypipes: did you run the whole servergroup functional test group, or just the one failing? | 16:15 |
jaypipes | bauzas: both | 16:15 |
bauzas | k | 16:15 |
bauzas | last time I had such problem, it was because of the fixture | 16:16 |
bauzas | but I was able to reproduce it | 16:16 |
*** cdent has quit IRC | 16:17 | |
*** Apoorva has joined #openstack-nova | 16:18 | |
*** rcernin_ has quit IRC | 16:19 | |
mriedem | so the test creates a soft-anti-affinity server group and kills the 2nd compute service so there should only be 1 | 16:19 |
mriedem | since the 'soft' means it will go to the same host if there is only 1 available | 16:19 |
sdague | mriedem: you have an opinion on this - https://review.openstack.org/#/c/215385/21/nova/virt/ironic/client_wrapper.py - I thought we wanted N-1 compat | 16:19 |
*** fragatina has quit IRC | 16:20 | |
mriedem | sdague: that's basically how it's always worked with nova + ironic as far as i know; yes it's not what we do with anything else | 16:21 |
sdague | mriedem: ok, well we definitely need upgrade stanza in the release notes then | 16:21 |
mriedem | i raised a similar concern in another nova ironic change, | 16:22 |
mriedem | https://review.openstack.org/#/c/419975/11/releasenotes/notes/add-interface-attach-detach-support-in-ironic-cb2bf11f3875350a.yaml@4 | 16:22 |
mriedem | basically they implemented something in a driver and then started claiming support for hot plugging interfaces in nova | 16:23 |
*** derekh has quit IRC | 16:23 | |
mriedem | my argument was, what if my ironic is from ocata and i upgrade nova, and was told, that doesn't happen because everyone knows to upgrade ironic first | 16:23 |
sdague | mriedem: so... I'm extremely suspicious of "everybody knows" | 16:23 |
sdague | I'm fine with it being policy, but we still need upgrade notes | 16:24 |
mriedem | yeah they added one later here https://review.openstack.org/#/c/419975/17/releasenotes/notes/add-interface-attach-detach-support-in-ironic-cb2bf11f3875350a.yaml | 16:24 |
sdague | I'd be more happy with an N-1 policy, but we do need to communicate it every time it would trip people up | 16:25 |
*** kfarr has joined #openstack-nova | 16:26 | |
mriedem | i don't disagree | 16:26 |
*** tylerder_ has joined #openstack-nova | 16:28 | |
mriedem | jaypipes: bauzas: i have an idea on that failure | 16:29 |
mriedem | the functional test failure that is | 16:29 |
jaypipes | yeah? | 16:30 |
bauzas | ? | 16:30 |
mriedem | the test setup creates 2 compute services/nodes/hosts | 16:30 |
mriedem | you can see the 2nd host CN created here http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_768000 | 16:30 |
mriedem | with uuid a945d18d-e6b6-4fd5-8b3b-e50c973a1ff9 | 16:30 |
mriedem | and it's posted into placement | 16:30 |
mriedem | now the purpose of the test is to restrict to 1 host | 16:30 |
mriedem | so | 16:30 |
mriedem | self.compute2.kill() | 16:30 |
mriedem | HOWEVER | 16:31 |
mriedem | filter scheduler is getting it's hosties from placement now | 16:31 |
mriedem | so that RP is still floating around in placement | 16:31 |
*** ralonsoh has quit IRC | 16:31 | |
mriedem | you can see the scheduler do it's thing here | 16:31 |
mriedem | GET /allocation_candidates?resources=DISK_GB%3A20%2CMEMORY_MB%3A2048%2CVCPU%3A1 HTTP/1.1" status: 200 len: 992 time: 0.0521650 | 16:31 |
mriedem | and then | 16:32 |
mriedem | No compute service record found for host host2 | 16:32 |
mriedem | because the test killed it | 16:32 |
mriedem | Filter RetryFilter returned 0 hosts | 16:32 |
*** tylerderosagrund has quit IRC | 16:32 | |
mriedem | what i don't understand is why it doesn't try the other compute host | 16:32 |
mriedem | so one easy hack is deleting the host2 RP from test_boot_servers_with_soft_anti_affinity_one_available_host - but that's pretty hacky, it seems odd that the filtering wouldn't check the other host | 16:34 |
jaypipes | mriedem: it's also odd that this test fails randomly. | 16:35 |
*** annegentle has quit IRC | 16:37 | |
*** gszasz has quit IRC | 16:38 | |
mriedem | yeah so we get this https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L674 | 16:39 |
mriedem | so _get_host_states should return 1 host | 16:39 |
*** lyan has joined #openstack-nova | 16:39 | |
*** swamireddy has quit IRC | 16:40 | |
*** knikolla has quit IRC | 16:40 | |
*** hoonetorg has joined #openstack-nova | 16:41 | |
*** sridharg has quit IRC | 16:41 | |
*** annegentle has joined #openstack-nova | 16:41 | |
*** spotz has quit IRC | 16:42 | |
*** rhagarty has quit IRC | 16:43 | |
*** spotz has joined #openstack-nova | 16:45 | |
mriedem | my only guess is for whatever reason the retry is disabled when the test runs | 16:45 |
mriedem | nvm that doesn't make sense | 16:46 |
jaypipes | just failed that test again in the isolate get_sorted_hosts() patch :( | 16:46 |
*** yamahata has quit IRC | 16:46 | |
mriedem | jaypipes: yeah that's the one bauzas was talking about | 16:46 |
mriedem | it failed in the gate | 16:46 |
*** rhagarty has joined #openstack-nova | 16:47 | |
*** lucasagomes is now known as lucas-afk | 16:47 | |
mriedem | man you really need debug output when this fails | 16:48 |
mriedem | and why wouldn't we see this in the logs? https://github.com/openstack/nova/blob/master/nova/scheduler/filters/retry_filter.py#L47 | 16:49 |
jaypipes | mriedem: anything in the scheduler needs DEBUG output because of the policy of only doing DEBUG level logging :( | 16:49 |
mriedem | jaypipes: heh yeah i know | 16:49 |
jaypipes | mriedem: because it was never passed any hosts. | 16:49 |
jaypipes | mriedem: if you look at the log failures, you see REtryFilter(0 hosts) | 16:50 |
mriedem | RetryFilter: (start: 0, end: 0) ? | 16:50 |
jaypipes | yeah | 16:50 |
*** vladikr has quit IRC | 16:50 | |
mriedem | ok, so the first compute node never came back from GET /allocation_candidates probably | 16:51 |
mriedem | http://logs.openstack.org/64/483564/2/gate/gate-nova-tox-functional-ubuntu-xenial/0f4534d/console.html#_2017-07-17_15_20_35_765791 | 16:53 |
mriedem | that's when the first compute host starts up | 16:53 |
*** ircuser-1 has quit IRC | 16:53 | |
mriedem | and then "Compute_service record created for compute:fake-mini" | 16:53 |
mriedem | Another thread already created a resource provider with the UUID 3694a10a-e435-4400-83aa-7d6ade1fc866. Grabbing that record from the placement API. | 16:54 |
mriedem | i don't know what 3694a10a-e435-4400-83aa-7d6ade1fc866 is | 16:54 |
mriedem | i thought the first compute RP should be e9e74898-4332-4e50-a4c6-1ad75dd295e5 | 16:54 |
*** hoonetorg has quit IRC | 16:54 | |
mriedem | hmm wtf GET /resource_providers/3694a10a-e435-4400-83aa-7d6ade1fc866" status: 404 | 16:55 |
mriedem | Unable to refresh my resource provider record | 16:55 |
*** fragatina has joined #openstack-nova | 16:56 | |
mriedem | so whatever happened, it never posted inventory for the first compute host, | 16:56 |
mriedem | which is why it didn't come back in GET /allocation_candidates | 16:56 |
*** ociuhandu has quit IRC | 16:56 | |
mriedem | seems like another case of the wsgi eventlet weirdness | 16:56 |
mriedem | crossing streams | 16:56 |
mriedem | on a side note, it's silly that we go through all the work of calling down to get_filtered_objects when the original list we got back from GET /allocation_candidates is empty to begin with | 16:58 |
mriedem | ah yes jay's todo: "Really, we should raise NoValidHosts here" | 16:59 |
jaypipes | tada. | 16:59 |
*** vladikr has joined #openstack-nova | 16:59 | |
mriedem | so i think the failure is totally unrelated | 16:59 |
*** mtanino_ has quit IRC | 17:00 | |
*** ircuser-1 has joined #openstack-nova | 17:02 | |
*** hoonetorg has joined #openstack-nova | 17:03 | |
mriedem | i rechecked and left a big ass summary of the failure | 17:03 |
*** fragatina has quit IRC | 17:04 | |
*** tbachman has quit IRC | 17:05 | |
*** mvk has quit IRC | 17:06 | |
*** tbachman has joined #openstack-nova | 17:09 | |
mriedem | aha 10 hours later i think i figured out the ValueError: Cannot convert NUMATopologyLimits(cpu_allocation_ratio=16.0,ram_allocation_ratio=1.5) to primitive | 17:10 |
mriedem | and is a real bug it looks like | 17:10 |
*** armax has joined #openstack-nova | 17:12 | |
*** yamahata has joined #openstack-nova | 17:12 | |
mriedem | oooo and one mr jay introduced :) | 17:13 |
mriedem | i guess our numa 3rd party ci isn't running? | 17:13 |
jaypipes | mriedem: in, like, how long? :) | 17:13 |
mriedem | https://review.openstack.org/#/c/478291/1/nova/scheduler/manager.py | 17:13 |
mriedem | L48 | 17:13 |
mriedem | that's not a primitive | 17:13 |
mriedem | so when passing to jsonutils.to_primitive | 17:14 |
mriedem | kablammo | 17:14 |
mriedem | in the latest oslo.serialization | 17:14 |
mriedem | http://logs.openstack.org/87/482287/6/check/gate-cross-nova-func-ubuntu-xenial/a645ce8/console.html#_2017-07-16_14_02_02_649384 | 17:14 |
mriedem | but not to worry, | 17:14 |
jaypipes | mmm | 17:14 |
mriedem | i've wrangled the oslo team to blacklist that version because it was breaking us, because of the bug :) | 17:14 |
mriedem | i'll push a fix | 17:14 |
jaypipes | k, thanks matt | 17:15 |
*** tylerder_ has quit IRC | 17:15 | |
*** tylerderosagrund has joined #openstack-nova | 17:16 | |
*** chyka has quit IRC | 17:16 | |
*** chyka has joined #openstack-nova | 17:17 | |
*** harlowja has joined #openstack-nova | 17:17 | |
mriedem | sean-k-mooney: do you know if anyone is running the intel NUMA 3rd party ci anymore? | 17:17 |
*** imacdonn has joined #openstack-nova | 17:21 | |
*** imacdonn_ has joined #openstack-nova | 17:21 | |
mriedem | fwiw this could have always been busted https://review.openstack.org/#/c/478291/1/nova/scheduler/filter_scheduler.py | 17:21 |
mriedem | i don't see any fancy conversion before | 17:21 |
*** imacdonn_ has quit IRC | 17:22 | |
*** fragatina has joined #openstack-nova | 17:23 | |
*** sshwarts has quit IRC | 17:26 | |
openstackgerrit | Merged openstack/nova master: Fix the releasenote and api-ref for quota-class API https://review.openstack.org/483170 | 17:28 |
*** tylerderosagrund has quit IRC | 17:30 | |
*** tbachman has quit IRC | 17:31 | |
jaypipes | alex_xu: https://review.openstack.org/#/c/480379/ reviewed. couple tiny nits. if you can fix those up and re-push, I'm +2. | 17:31 |
*** tbachman has joined #openstack-nova | 17:31 | |
*** yamahata has quit IRC | 17:33 | |
*** hoonetorg has quit IRC | 17:34 | |
*** yamahata has joined #openstack-nova | 17:34 | |
*** prateek has joined #openstack-nova | 17:34 | |
*** mvk has joined #openstack-nova | 17:37 | |
*** sambetts is now known as sambetts|afk | 17:38 | |
*** efoley has quit IRC | 17:38 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 17:39 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | 17:39 |
*** MVenesio has joined #openstack-nova | 17:39 | |
*** tylerderosagrund has joined #openstack-nova | 17:44 | |
*** kristia__ has quit IRC | 17:49 | |
*** kristian__ has joined #openstack-nova | 17:50 | |
*** tbachman has quit IRC | 17:51 | |
*** tbachman has joined #openstack-nova | 17:52 | |
*** hoonetorg has joined #openstack-nova | 17:54 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | 17:54 |
*** kristian__ has quit IRC | 17:54 | |
*** jangutter has quit IRC | 17:55 | |
*** jangutter has joined #openstack-nova | 17:55 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.create notification https://review.openstack.org/454023 | 17:59 |
*** hoonetorg has quit IRC | 17:59 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902 | 17:59 |
*** lpetrut_ has quit IRC | 18:00 | |
*** tesseract has quit IRC | 18:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert HostState.limits objcets to primitives https://review.openstack.org/484439 | 18:03 |
mriedem | dansmith: sfinucan: ^ i'm going to need someone more familiar with the numa limits stuff to look at that | 18:03 |
mriedem | like sahid or vladikr? | 18:03 |
mriedem | i don't know how this was actually working before | 18:03 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert HostState.limits objects to primitives https://review.openstack.org/484439 | 18:04 |
mriedem | was conductor serializing that response before for us so the NUMATopologyLimits was getting serialized by conductor? | 18:05 |
*** dtantsur is now known as dtantsur|afk | 18:07 | |
*** gbarros has quit IRC | 18:07 | |
*** felipemonteiro__ has quit IRC | 18:07 | |
oomichi | alex_xu: most patches are approved on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike | 18:07 |
oomichi | alex_xu: need to update the remaining for conflicts | 18:08 |
*** gbarros has joined #openstack-nova | 18:09 | |
*** chyka has quit IRC | 18:15 | |
*** chyka has joined #openstack-nova | 18:16 | |
*** Sukhdev has joined #openstack-nova | 18:24 | |
*** rcernin has joined #openstack-nova | 18:26 | |
*** fragatina has quit IRC | 18:28 | |
*** fragatina has joined #openstack-nova | 18:28 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Log compute node uuid when the record is created https://review.openstack.org/484447 | 18:31 |
*** kristian__ has joined #openstack-nova | 18:37 | |
*** rnoriega has quit IRC | 18:38 | |
*** vishwanathj has quit IRC | 18:38 | |
*** vishwanathj has joined #openstack-nova | 18:39 | |
*** ociuhandu has joined #openstack-nova | 18:39 | |
*** felipemonteiro has joined #openstack-nova | 18:39 | |
*** felipemonteiro_ has joined #openstack-nova | 18:40 | |
*** lpetrut_ has joined #openstack-nova | 18:41 | |
*** kristian__ has quit IRC | 18:42 | |
*** felipemonteiro has quit IRC | 18:44 | |
*** jbernard has quit IRC | 18:45 | |
*** jbernard has joined #openstack-nova | 18:45 | |
*** lpetrut_ has quit IRC | 18:47 | |
*** gjayavelu has joined #openstack-nova | 18:47 | |
*** ftersin has quit IRC | 18:49 | |
*** rnoriega has joined #openstack-nova | 18:49 | |
mriedem | dansmith: sfinucan: nvm i think i know how that worked before - jsonutils.to_primitive() wouldn't error out, but wouldn't convert the NUMATopologyLimits either, so it would go through NovaObjectSerializer to do the serialize/deserialize work on the object | 18:52 |
mriedem | the new change in to_primitive to raise ValueError is what blew it all up because it would try to serialize before hitting the NovaObjectSerializer | 18:53 |
dansmith | sorry I missed the above ping, | 18:53 |
dansmith | but I assume I can ignore now | 18:53 |
mriedem | validation of my understanding, and feelings, is always appreciated, but yeah you can ignore probably | 18:53 |
dansmith | jsonutils.to_primitive used to fail with "recursion error" or something like that when it hit a non-serializeable object | 18:54 |
dansmith | mriedem: what change introduced this? | 18:56 |
dansmith | oh I see, in oslo | 18:56 |
mriedem | yeah that's what the oslo.serialization change was trying to prevent, the infinite recursion | 18:58 |
mriedem | by just failing fast | 18:58 |
dansmith | mriedem: so the thing I'm concerned about is: are we returning something that is only using jsonutils for serialization? | 18:59 |
dansmith | our object serializer doesn't go hunting in an iterable for any instance of an object to serialize | 18:59 |
*** avolkov has quit IRC | 19:00 | |
dansmith | doesn't go arbitrarily deep, I mean | 19:00 |
mriedem | hmm yeah because that's what i was looking at with https://github.com/openstack/nova/blob/master/nova/objects/base.py#L244 | 19:01 |
mriedem | so it goes 1 level? | 19:01 |
mriedem | well, actually, | 19:01 |
mriedem | so this would be a like {'limits': {'numa_topology': object}} | 19:01 |
mriedem | so that should hit it ^ | 19:01 |
dansmith | where is this exactly? | 19:02 |
dansmith | is this in a returned value? | 19:02 |
mriedem | yeah | 19:02 |
mriedem | https://review.openstack.org/#/c/484439/2/nova/scheduler/manager.py@140 | 19:02 |
*** nicolasbock has quit IRC | 19:03 | |
mriedem | creates a list of these dicts, and each dict will have a 'limits' key that maps to a dict, and that thing has the numa_topology_limits=NUMATopologyLimits object | 19:03 |
mriedem | map | 19:03 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Optimize creating servers with tags https://review.openstack.org/484170 | 19:05 |
dansmith | I guess it will go pretty deep | 19:05 |
dansmith | had to re-read that a couple times | 19:05 |
mriedem | yeah so i think that's how it wasn't blowing up before | 19:05 |
mriedem | well, and was actually serializing the thing | 19:05 |
*** nicolasbock has joined #openstack-nova | 19:05 | |
dansmith | yeah | 19:06 |
mriedem | the fact jsonutils.to_primitive ignored it was to our advantage | 19:06 |
*** tylerder_ has joined #openstack-nova | 19:06 | |
dansmith | yeah | 19:06 |
mriedem | that change is also being reverted https://review.openstack.org/#/c/484396/ | 19:06 |
mriedem | b/c i pointed out that releasing that under a patch release versoin w/o any prior warning <2 weeks to FF is likely to break other projects | 19:07 |
dansmith | heh | 19:07 |
dansmith | yeah I guess so | 19:07 |
mriedem | we only had prior warning b/c the requirements changes gate on nova unit/functional tests | 19:07 |
*** tylerderosagrund has quit IRC | 19:09 | |
*** tylerder_ has quit IRC | 19:10 | |
*** lpetrut_ has joined #openstack-nova | 19:11 | |
*** mtanino_ has joined #openstack-nova | 19:13 | |
*** knikolla has joined #openstack-nova | 19:14 | |
*** kristian__ has joined #openstack-nova | 19:25 | |
mriedem | oomichi: i'm +2 again on https://review.openstack.org/#/c/479774/ after your comments | 19:25 |
*** kbaegis has quit IRC | 19:26 | |
*** harlowja has quit IRC | 19:32 | |
*** smatzek has quit IRC | 19:35 | |
*** Ctesias has joined #openstack-nova | 19:39 | |
Ctesias | hi. I'm digging around in nova trying to figure out why my nova_override.conf file isn't overriding nova defaults and I'm curious where `import nova.conf` and `CONF = nova.conf.CONF` are coming from because I don't see those being defined anywhere | 19:39 |
mriedem | Ctesias: for nova-compute the options are loaded up here https://github.com/openstack/nova/blob/master/nova/cmd/compute.py#L42 | 19:41 |
mriedem | similar pattern for the other service entry points | 19:41 |
mriedem | https://github.com/openstack/nova/blob/master/nova/config.py#L49 | 19:41 |
Ctesias | Thanks for the pointer. | 19:43 |
Ctesias | Is putting an override_nova.conf file in the config directory the right way to get overrides into the nova config? | 19:43 |
mriedem | i've never used nova_override.conf | 19:44 |
mriedem | so if it works, it's some magic i'm not aware of | 19:44 |
*** dave-mccowan has joined #openstack-nova | 19:44 | |
Ctesias | well... It doesn't work :( | 19:45 |
Ctesias | Any tips on how I would go about overriding defaults in nova.conf the right way? | 19:45 |
mriedem | can't you pass overrides on the --config-file cli option | 19:45 |
mriedem | i'm not sure though | 19:46 |
*** sree has joined #openstack-nova | 19:46 | |
mriedem | there could be something in https://docs.openstack.org/oslo.config/latest/ | 19:46 |
*** smatzek has joined #openstack-nova | 19:47 | |
sdague | Ctesias: can you explain what you are trying to do with overrides besides modifying the nova.conf directly? Maybe we're just not understanding the question | 19:47 |
*** boris-42__ has joined #openstack-nova | 19:48 | |
Ctesias | I was under the impression modifying nova.conf directly was no bueno. But I'm trying to enable a barbican with Nova, so I need to set the api_class setting in config file under [keymgr] | 19:49 |
Ctesias | I created nova_override.conf in the same directory where nova.conf was and put [keymgr] and api_class = <the barbican manager> in it | 19:50 |
Ctesias | restarted nova | 19:50 |
Ctesias | but nova had the defaults when it came back | 19:50 |
*** sree has quit IRC | 19:51 | |
mriedem | just modify nova.conf | 19:51 |
mriedem | nova.conf can be totally blank and it won't override any defaults - the defaults are all in code | 19:52 |
*** tbachman_ has joined #openstack-nova | 19:52 | |
mriedem | so if you want to override any defaults, it happens in nova.conf | 19:52 |
mriedem | nova_override.conf is probably some package-specific trick | 19:52 |
Ctesias | .....Oh. | 19:52 |
*** mgiles has joined #openstack-nova | 19:52 | |
Ctesias | Well awesome. Thanks for your help, really | 19:52 |
mriedem | yw | 19:52 |
Ctesias | I've been pulling my hair out for days | 19:52 |
*** tbachman has quit IRC | 19:54 | |
*** mgiles has quit IRC | 19:56 | |
*** mgiles has joined #openstack-nova | 19:57 | |
*** nicolasbock has quit IRC | 19:58 | |
*** tbachman_ has quit IRC | 19:59 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 20:00 |
mriedem | Kevin_Zheng: ^ fixed up the functional test failures; it was pretty simple but not obvious until i injected some debug code and ran the tests, and then also had to tweak the tests a bit | 20:00 |
*** brault has joined #openstack-nova | 20:01 | |
*** tbachman has joined #openstack-nova | 20:02 | |
mriedem | sdague: can you take a gander at ^ ? it's pretty straight-forward and well tested | 20:03 |
*** cdent has joined #openstack-nova | 20:03 | |
*** brault has quit IRC | 20:05 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Optimize creating servers with tags https://review.openstack.org/484170 | 20:05 |
sdague | mriedem: I'll add it to the morning list | 20:06 |
mriedem | thanks - alex_xu might get it overnight then, he was reviewing it before too | 20:06 |
*** annegentle has quit IRC | 20:07 | |
oomichi | mriedem: thanks, +2 | 20:08 |
*** lpetrut_ has quit IRC | 20:10 | |
Ctesias | Hi again. I'm getting the same behavior with the changes directly in my nova.conf file. I've tried pointing at the directory with --config-dir and directly at the nova.conf file with --config-file with no change in behavior | 20:10 |
Ctesias | the api_class value is still the default | 20:10 |
mriedem | Ctesias: which service? | 20:11 |
Ctesias | nova service | 20:11 |
mriedem | which of the nova services, there are several | 20:11 |
mriedem | nova-osapi_compute? | 20:11 |
*** owalsh has quit IRC | 20:11 | |
mriedem | nova-compute, etc | 20:11 |
Ctesias | nova-compute | 20:12 |
*** dklyle has joined #openstack-nova | 20:12 | |
*** david-lyle has quit IRC | 20:13 | |
mriedem | when you restart nova-compute, do you have debug logging enabled so you can see it dump which config files are being used to and it will dump the options | 20:13 |
*** pchavva has quit IRC | 20:16 | |
Ctesias | the file is pretty long, there something I can grep for to get the file it's using? | 20:16 |
*** dave-mccowan has quit IRC | 20:17 | |
mriedem | yeah | 20:17 |
mriedem | "command line args" | 20:17 |
mriedem | "config files" | 20:18 |
mriedem | like here http://logs.openstack.org/47/484447/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/b2fbd0c/logs/screen-n-cpu.txt.gz#_Jul_17_18_47_11_345788 | 20:18 |
mriedem | and then it dumps the options it's using | 20:18 |
mriedem | Ctesias: you're using the wrong group | 20:19 |
mriedem | it's not [keymgr] | 20:19 |
mriedem | it's [key_manager] | 20:19 |
mriedem | http://logs.openstack.org/47/484447/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/b2fbd0c/logs/screen-n-cpu.txt.gz#_Jul_17_18_47_11_670125 | 20:19 |
Ctesias | yeah, that was a typo earlier. I've got [key_manager] in the file | 20:19 |
Ctesias | If I put a breakpoint in nova/keymgr/conf_key_mgr.py I can go to the line after it grabs the nova configuration and see it's the same | 20:21 |
Ctesias | (Pdb) CONF['key_manager']['api_class'] 'nova.keymgr.conf_key_mgr.ConfKeyManager' | 20:21 |
*** zhenq has joined #openstack-nova | 20:21 | |
mriedem | and the debug log output when you restart nova-compute shows the same when it dumps the options? | 20:23 |
*** mamandle has quit IRC | 20:23 | |
Ctesias | yes | 20:23 |
Ctesias | 2017-07-17 16:20:13.483 DEBUG oslo_service.service [req-ece128f3-fb7c-4da3-af58-78c39fca910c None None] key_manager.api_class = nova.keymgr.conf_key_mgr.ConfKeyManager from (pid=14641) log_opt_values /opt/pf9/venv/lib/python2.7/site-packages/oslo_config/cfg.py:2626-07-17 16:20:13.485 DEBUG oslo_service.service [req-ece128f3-fb7c-4da3-af58-78c39fca910c None None] osapi_v21.project_id_regex = None from (pid=14641) | 20:24 |
mriedem | hmm, api_class comes from castellan | 20:25 |
mriedem | which version of that library do you have installed? | 20:25 |
Ctesias | 0.4.0 | 20:26 |
mriedem | kfarr: ^ have you heard of anything like this? | 20:26 |
*** david-lyle has joined #openstack-nova | 20:26 | |
mriedem | Ctesias: which version of nova? master (pike)? or ocata? | 20:26 |
*** randomhack has joined #openstack-nova | 20:26 | |
mriedem | because that version is old for master https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L227 | 20:26 |
Ctesias | Newton, actually | 20:27 |
mriedem | ew | 20:27 |
Ctesias | speed of business, man | 20:27 |
*** dklyle has quit IRC | 20:27 | |
mriedem | so things were different in newton https://github.com/openstack/nova/blob/stable/newton/nova/conf/key_manager.py | 20:28 |
mriedem | looks like it should still set those options | 20:28 |
*** kristian__ has quit IRC | 20:29 | |
*** kristian__ has joined #openstack-nova | 20:29 | |
mriedem | Ctesias: this might be your problem https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L29 | 20:30 |
mriedem | not totally sure | 20:30 |
mriedem | need to probably talk to kfarr | 20:31 |
Ctesias | I don't see api_class as an option in /nova/conf/key_manager.py | 20:32 |
mriedem | that's not the options registry, | 20:32 |
mriedem | that's the default impl | 20:32 |
mriedem | the options are registered here https://github.com/openstack/nova/blob/stable/newton/nova/conf/key_manager.py | 20:32 |
mriedem | and then there is some backward compat magic in https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py | 20:33 |
*** prateek has quit IRC | 20:33 | |
*** kristian__ has quit IRC | 20:34 | |
Ctesias | yeah commenting out the lines in __init__.py fixes it | 20:37 |
Ctesias | Is this behavior fixed upstream from Newton anywhere yet? | 20:38 |
*** mgiles has quit IRC | 20:40 | |
mriedem | i think that stuff in init was all removed in ocata | 20:41 |
*** crushil has quit IRC | 20:41 | |
mriedem | oh i guess not ocata https://github.com/openstack/nova/blob/stable/ocata/nova/keymgr/__init__.py | 20:41 |
mriedem | or master https://github.com/openstack/nova/blob/master/nova/keymgr/__init__.py | 20:41 |
mriedem | damn | 20:41 |
mriedem | so the fact it's setting that b/c of devstack is a total hack | 20:42 |
mriedem | let me see if that's at least set in devstack | 20:42 |
*** vishwana_ has joined #openstack-nova | 20:43 | |
mriedem | Ctesias: https://review.openstack.org/#/c/439855/ | 20:43 |
*** burt has quit IRC | 20:43 | |
Ctesias | cool. Thanks | 20:44 |
mriedem | Ctesias: so did you just remove this line? https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L29 ? | 20:45 |
mriedem | or all of the stuff in there | 20:45 |
*** vishwanathj has quit IRC | 20:46 | |
Ctesias | I only commented out the lines that re-set the api_class value | 20:46 |
*** damien_r has joined #openstack-nova | 20:47 | |
Ctesias | but if you're curious I can try to get rid of all of it and see what happens | 20:47 |
mriedem | i'm curious if just commenting out https://github.com/openstack/nova/blob/stable/newton/nova/keymgr/__init__.py#L29 resolves your issue | 20:47 |
mriedem | because that would be easy for me to change on master and backport through to newton | 20:47 |
Ctesias | Yes. | 20:47 |
mriedem | ok | 20:47 |
Ctesias | If you would be so kind as to post that change once it comes in so I can follow it back and get upgraded to it that would be excellent. | 20:48 |
mriedem | i can mess around with pushing that change later then - i can't say i totally understand *why* it would cause an issue since it should just set a default, not override your value | 20:48 |
mriedem | unless it thinks that your value was the default or something? | 20:48 |
Ctesias | beats the hell out of me | 20:48 |
mriedem | i'll just remove it on the grounds that we have the value set in devstack so removing the hack is good enough | 20:49 |
*** vishwana_ has quit IRC | 20:50 | |
*** crushil has joined #openstack-nova | 20:50 | |
mriedem | Ctesias: reporting a bug against nova would also help with the backports | 20:50 |
mriedem | https://bugs.launchpad.net/nova/+filebug | 20:50 |
Ctesias | will do | 20:52 |
*** harlowja has joined #openstack-nova | 20:56 | |
*** damien_r has quit IRC | 20:59 | |
*** baoli has quit IRC | 20:59 | |
*** lucasxu has quit IRC | 21:01 | |
*** Ctesias has quit IRC | 21:01 | |
*** thorst has quit IRC | 21:10 | |
*** randomhack has quit IRC | 21:11 | |
*** MVenesio has quit IRC | 21:12 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update policy descriptions for base. https://review.openstack.org/472622 | 21:13 |
*** smatzek has quit IRC | 21:13 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 21:13 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Update policy descriptions for base. https://review.openstack.org/472622 | 21:14 |
mriedem | oomichi: another easy one and the final patch for the policy docs bp https://review.openstack.org/#/c/472622/ | 21:15 |
*** awaugama has quit IRC | 21:19 | |
*** owalsh has joined #openstack-nova | 21:21 | |
*** Sukhdev has quit IRC | 21:26 | |
mriedem | who here feels like adding back in the ability to load up out of tree scheduler drivers? https://bugs.launchpad.net/nova/+bug/1704788 | 21:27 |
openstack | Launchpad bug 1704788 in OpenStack Compute (nova) "Hardcoded choices for nova scheduler driver" [Undecided,Confirmed] | 21:27 |
dansmith | heh | 21:30 |
mriedem | bauzas: since you were +W on that thing, how do you feel about ^ | 21:30 |
mriedem | answer tomorrow when you're awake | 21:30 |
mriedem | because we really probably should have made that (1) a deprecation cycle and (2) ffs at least reno it | 21:30 |
openstackgerrit | Merged openstack/nova master: Handle Cinder 3.27 style attachments in swap_volume https://review.openstack.org/456971 | 21:33 |
*** rcernin has quit IRC | 21:33 | |
openstackgerrit | Merged openstack/nova master: [placement] Update allocation-candidates.yaml for gabbi 1.35 https://review.openstack.org/484356 | 21:34 |
cdent | \o/ | 21:34 |
*** dave-mccowan has joined #openstack-nova | 21:34 | |
mriedem | cdent: btw, just so you know, | 21:34 |
mriedem | we went hiking, toured a historic village, and went swimming yesterday | 21:34 |
mriedem | so it wasn't all work | 21:35 |
cdent | ah well, I am so relieved | 21:35 |
*** sree_ has joined #openstack-nova | 21:36 | |
*** sree_ is now known as Guest50476 | 21:36 | |
melwitt | mriedem: replied to your latest comments. the things that seem up in the air are 1) whether to add vm_states.SOFT_DELETED + task_states.RESTORING instances to the count and recheck in restore | 21:36 |
melwitt | 2) whether to check quota on resize confirm | 21:36 |
mriedem | ok | 21:38 |
mriedem | oh dan https://review.openstack.org/#/c/472622/ <3 | 21:38 |
melwitt | mriedem: I'm not sure what I think other than, if I do both 1) and 2) I guess that eliminates the TODOs I had written bc checking quota at resize confirm sort of serves as a recheck for resize (as I described in the review comment) | 21:40 |
*** Guest50476 has quit IRC | 21:40 | |
*** annegentle has joined #openstack-nova | 21:40 | |
melwitt | so maybe that one should be guarded by "if CONF.quota.recheck_quota" | 21:40 |
melwitt | correction, the revert_resize TODO would remain | 21:44 |
dansmith | mriedem: we might want to get this in before the quota-ectomy so we can backport: https://review.openstack.org/#/c/462521/4/nova/compute/manager.py | 21:44 |
mriedem | dansmith: that's 3 changes in a series | 21:45 |
mriedem | the 2nd change has no tests | 21:45 |
dansmith | ah, yup | 21:46 |
dansmith | the bottom one doesn't either and it seems like they'd be trivial | 21:49 |
*** itlinux_ has joined #openstack-nova | 21:52 | |
*** cdent has quit IRC | 21:52 | |
*** annegentle has quit IRC | 21:53 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove key_manager.api_class hack https://review.openstack.org/484501 | 21:58 |
*** kylek3h has quit IRC | 22:02 | |
*** felipemonteiro_ has quit IRC | 22:03 | |
*** lpetrut_ has joined #openstack-nova | 22:04 | |
*** esberglu has quit IRC | 22:05 | |
*** edmondsw has quit IRC | 22:10 | |
*** thorst has joined #openstack-nova | 22:10 | |
*** mriedem has quit IRC | 22:10 | |
*** gouthamr has quit IRC | 22:12 | |
*** gouthamr has joined #openstack-nova | 22:14 | |
*** thorst has quit IRC | 22:15 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Log compute node uuid when the record is created https://review.openstack.org/484447 | 22:15 |
*** mriedem has joined #openstack-nova | 22:16 | |
*** mmehan has quit IRC | 22:17 | |
*** lpetrut_ has quit IRC | 22:19 | |
*** esberglu has joined #openstack-nova | 22:20 | |
*** esberglu has quit IRC | 22:25 | |
*** baoli has joined #openstack-nova | 22:25 | |
*** baoli has quit IRC | 22:26 | |
*** edmondsw has joined #openstack-nova | 22:26 | |
*** xyang1 has quit IRC | 22:29 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert HostState.limits['numa_topology'] to primitive https://review.openstack.org/484439 | 22:29 |
mriedem | dansmith: ^ is that what you were looking for wrt being specific? | 22:29 |
dansmith | mriedem: well kinda, although I don't see why you need to check for if it's an object | 22:30 |
dansmith | that's just too much ambiguity unless it's really needed | 22:30 |
dansmith | if it's set, it's that object, right? | 22:30 |
mriedem | honestly i'd rather not even do this specific check at all | 22:30 |
mriedem | and just handle it generically | 22:30 |
*** edmondsw has quit IRC | 22:31 | |
mriedem | i don't like the tight coupling between the filters and the scheduler manager here | 22:31 |
*** zhenq has quit IRC | 22:32 | |
dansmith | idk, I just think more magic object conversions hide where this is happening | 22:32 |
mriedem | true | 22:33 |
mriedem | i don't like any of it really so i don't care too much :) | 22:33 |
*** acormier has quit IRC | 22:33 | |
*** acormier has joined #openstack-nova | 22:34 | |
dansmith | they're reverting the jsonutils change, but they plan to put it back in I guess? | 22:35 |
mriedem | they plan to put it back in with a UserWarning that the thing passed in can't be turned into a primitive | 22:35 |
mriedem | and then drop that at some point with a major version bump i guess | 22:36 |
dansmith | okay | 22:36 |
mriedem | since it's an api change in the library | 22:36 |
mriedem | but 6 months from now who knows if anyone will remember any of this | 22:36 |
dansmith | well, I dunno, we shouldn't really be doing this in the first place for an unversioned return value of an rpc call, IMHO | 22:36 |
dansmith | but IMHO, it's best to be very explicit and targeted if we're going to convert it | 22:37 |
dansmith | i.e. "if there is a numa limit, it is a NumaWhateverObject and we shall always convert it to primitive" | 22:37 |
*** marst has quit IRC | 22:37 | |
mriedem | ok making that change | 22:38 |
*** acormier has quit IRC | 22:38 | |
*** thorst has joined #openstack-nova | 22:41 | |
*** itlinux_ has quit IRC | 22:42 | |
*** thorst has quit IRC | 22:43 | |
*** ckopper has quit IRC | 22:46 | |
*** armax has quit IRC | 22:47 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Convert HostState.limits['numa_topology'] to primitive https://review.openstack.org/484439 | 22:48 |
*** mriedem has quit IRC | 22:54 | |
*** Apoorva has quit IRC | 22:55 | |
*** Apoorva has joined #openstack-nova | 22:56 | |
*** edmondsw has joined #openstack-nova | 22:57 | |
*** baoli has joined #openstack-nova | 23:02 | |
*** baoli has quit IRC | 23:02 | |
*** edmondsw has quit IRC | 23:02 | |
*** Apoorva_ has joined #openstack-nova | 23:04 | |
*** Apoorva has quit IRC | 23:07 | |
*** catintheroof has quit IRC | 23:07 | |
*** mriedem has joined #openstack-nova | 23:08 | |
*** lyan has quit IRC | 23:10 | |
mriedem | efried: is https://review.openstack.org/#/c/458257/ the last thing for that bp? | 23:11 |
mriedem | i see the dependent change is in ksa 2.20 | 23:11 |
efried | mriedem No, that one had a couple of the clienty changes; plan was to follow up with others. | 23:12 |
efried | mriedem Did ksa 2.20 happen?? | 23:12 |
efried | mriedem Or maybe 2.20 isn't the one I thought I was trying to squeeze stuff into right now. | 23:12 |
mriedem | https://review.openstack.org/#/c/482742/ ? | 23:13 |
mriedem | which change in ksa specifically do we need? | 23:14 |
mriedem | or all over https://review.openstack.org/#/q/topic:version-discovery ? | 23:14 |
mriedem | *all ov | 23:14 |
mriedem | *of | 23:14 |
efried | mriedem That was the hope. Specifically I wanted to fix the APIs in discover.py - https://review.openstack.org/#/c/483604/ | 23:15 |
efried | ...so we didn't release them with unnecessary params and have to support those (and their [really complicated] implications) forever. | 23:15 |
mriedem | ok so realizing that ksa boat of changes isn't going to land by thursday probably, | 23:17 |
*** ckopper has joined #openstack-nova | 23:17 | |
mriedem | do we have something pragmatic to land in nova to complete this for pike and then improve it later? | 23:17 |
mriedem | sfinucan: you took over as author on this https://review.openstack.org/#/c/428241/ | 23:18 |
mriedem | i think that git review / rebase bug where it changes author is hitting more than just me | 23:19 |
efried | mriedem If you're assuming the boat won't land... I would have to work something up that's better than https://review.openstack.org/#/c/458257/ but not the whole ksa hog. | 23:19 |
mriedem | efried: i'm assuming it won't land by thursday, no | 23:19 |
efried | But I've been operating under the hope that it's not too late. mordred Pipe dream? | 23:19 |
mriedem | and if it's rushed, it'll likely be buggy | 23:19 |
mordred | I don't think it's a pipe-dream | 23:21 |
mordred | the stuff we're working on getting in are mostly cleanups, better docs and better tests - the hard logic is in | 23:21 |
mordred | mriedem: I'm currently still pushing towards getting it out this week - I don't think it's unachievable | 23:22 |
efried | mordred You and I should probably take a hard look at whether https://review.openstack.org/#/c/483604/ (or some variant thereof) is within that achievable umbrella. | 23:22 |
efried | Cause that's hard logic. | 23:23 |
efried | Good news is, the test cases are robust to a point where, if we can get 'em passing, I'll have confidence in the code. | 23:23 |
*** sree_ has joined #openstack-nova | 23:24 | |
*** sree_ is now known as Guest1203 | 23:24 | |
mriedem | mordred: ok the release freeze is thursday | 23:25 |
mordred | efried: yes. I agree with that | 23:25 |
mordred | mriedem: yup | 23:25 |
mriedem | </project management status update check> | 23:25 |
mordred | efried: (I agree we need to take a hard look at it - and I agree that the tests are robust and trustable) | 23:25 |
*** ckopper has quit IRC | 23:27 | |
*** hongbin has quit IRC | 23:28 | |
*** Guest1203 has quit IRC | 23:28 | |
*** acormier has joined #openstack-nova | 23:29 | |
*** ckopper has joined #openstack-nova | 23:30 | |
*** gouthamr has quit IRC | 23:32 | |
*** takashin has joined #openstack-nova | 23:32 | |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - List/Show all server migration types https://review.openstack.org/430839 | 23:33 |
openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.52 - Enable cold migration with target host https://review.openstack.org/406707 | 23:33 |
*** acormier has quit IRC | 23:33 | |
*** ckopper has quit IRC | 23:34 | |
mriedem | TheJulia: question in https://review.openstack.org/#/c/419975/ | 23:48 |
mriedem | not sure who on the ironic team has the setup to test this, | 23:48 |
mriedem | but when plugging the vif, neutron should eventually recognize a state change in the port and send an event back to nova to refresh the cache | 23:48 |
mriedem | then we wouldn't need all that gorpy refresh cache code within the driver | 23:48 |
*** jwcroppe has quit IRC | 23:49 | |
*** gouthamr has joined #openstack-nova | 23:50 | |
*** acormier has joined #openstack-nova | 23:52 | |
*** gouthamr has quit IRC | 23:53 | |
*** gouthamr has joined #openstack-nova | 23:54 | |
*** chyka has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!