| *** baoli has quit IRC | 00:00 | |
| *** jistr has quit IRC | 00:01 | |
| *** jistr has joined #openstack-nova | 00:02 | |
| *** slaweq has quit IRC | 00:02 | |
| mriedem | no i'm up in a viewing area and she's down refusing to flip | 00:02 |
|---|---|---|
| mriedem | i'm with the screaming siblings | 00:02 |
| *** tbachman has joined #openstack-nova | 00:02 | |
| mriedem | preparing for the soul-crushing conversation on the car ride home about, "why didn't you do the thing we talked about you doing" | 00:03 |
| artom | Who's asking that question of whom? | 00:04 |
| mriedem | ha | 00:05 |
| *** mriedem has quit IRC | 00:07 | |
| *** marst has quit IRC | 00:10 | |
| *** ckopper has joined #openstack-nova | 00:11 | |
| *** ckopper has quit IRC | 00:12 | |
| *** masber has joined #openstack-nova | 00:13 | |
| *** tbachman has quit IRC | 00:14 | |
| *** Apoorva has quit IRC | 00:15 | |
| *** hshiina has joined #openstack-nova | 00:19 | |
| *** Apoorva has joined #openstack-nova | 00:20 | |
| *** lucasxu has joined #openstack-nova | 00:22 | |
| *** mtanino has joined #openstack-nova | 00:28 | |
| *** jwcroppe has joined #openstack-nova | 00:32 | |
| *** wy has joined #openstack-nova | 00:34 | |
| *** Apoorva has quit IRC | 00:34 | |
| *** wy has quit IRC | 00:35 | |
| *** tuanluong has joined #openstack-nova | 00:35 | |
| *** thorst has joined #openstack-nova | 00:40 | |
| *** tbachman has joined #openstack-nova | 00:40 | |
| *** mdnadeem has joined #openstack-nova | 00:42 | |
| *** thorst has quit IRC | 00:44 | |
| *** lucasxu has quit IRC | 00:45 | |
| *** lucasxu has joined #openstack-nova | 00:46 | |
| *** sbezverk has quit IRC | 00:49 | |
| *** mriedem has joined #openstack-nova | 00:52 | |
| *** mtanino has quit IRC | 00:55 | |
| *** lucasxu has quit IRC | 00:58 | |
| *** lucasxu has joined #openstack-nova | 00:59 | |
| *** lucasxu has quit IRC | 01:00 | |
| *** lucasxu has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| mriedem | alex_xu: you should probably take a look at https://review.openstack.org/#/c/469578/ | 01:03 |
| *** gjayavelu has quit IRC | 01:05 | |
| *** thorst has joined #openstack-nova | 01:07 | |
| *** thorst has quit IRC | 01:07 | |
| *** tovin07_ has joined #openstack-nova | 01:07 | |
| *** jwcroppe has quit IRC | 01:08 | |
| *** catintheroof has quit IRC | 01:10 | |
| *** Apoorva has joined #openstack-nova | 01:13 | |
| *** yamahata has quit IRC | 01:14 | |
| *** gongysh has joined #openstack-nova | 01:16 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** jamielennox is now known as jamielennox|away | 01:17 | |
| *** Apoorva has joined #openstack-nova | 01:20 | |
| *** tbachman_ has joined #openstack-nova | 01:22 | |
| mriedem | jaypipes: i just reviewed the os-traits sync thing https://review.openstack.org/#/c/469578/ and there is a comment in there saying it's synced on start of the process, but i don't see where/how that happens | 01:23 |
| mriedem | do you know? | 01:23 |
| *** tbachman has quit IRC | 01:25 | |
| *** tbachman_ is now known as tbachman | 01:25 | |
| *** Apoorva has quit IRC | 01:25 | |
| *** liujiong has joined #openstack-nova | 01:29 | |
| *** jamielennox|away is now known as jamielennox | 01:34 | |
| *** lucasxu has quit IRC | 01:34 | |
| *** gomarivera has joined #openstack-nova | 01:38 | |
| *** litao__ has joined #openstack-nova | 01:41 | |
| *** thorst has joined #openstack-nova | 01:41 | |
| *** armax has quit IRC | 01:44 | |
| *** mingyu has joined #openstack-nova | 01:45 | |
| *** Vek has joined #openstack-nova | 01:46 | |
| *** tbachman has quit IRC | 01:58 | |
| *** coreywright has quit IRC | 01:58 | |
| *** huanxie has joined #openstack-nova | 01:58 | |
| *** slaweq has joined #openstack-nova | 01:59 | |
| *** david_ has quit IRC | 02:01 | |
| *** carthaca_ has quit IRC | 02:01 | |
| *** databus23_ has quit IRC | 02:01 | |
| *** sapcc-bot has quit IRC | 02:01 | |
| *** sapcc-bot has joined #openstack-nova | 02:01 | |
| *** david_ has joined #openstack-nova | 02:01 | |
| *** carthaca_ has joined #openstack-nova | 02:01 | |
| *** databus23_ has joined #openstack-nova | 02:01 | |
| *** trinaths has joined #openstack-nova | 02:01 | |
| *** slaweq has quit IRC | 02:04 | |
| alex_xu | mriedem: I think it is only triggerred by get_by_name or get_all | 02:05 |
| *** xinliang has quit IRC | 02:06 | |
| *** thorst has quit IRC | 02:06 | |
| alex_xu | mriedem: I guess cdent means that after "process started and get_by_name or get_all" | 02:07 |
| *** Shunli has joined #openstack-nova | 02:08 | |
| *** bkopilov has quit IRC | 02:14 | |
| *** mdrabe has quit IRC | 02:15 | |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Change the condition to delete a resizing instance https://review.openstack.org/463466 | 02:15 |
| *** mingyu has quit IRC | 02:15 | |
| *** lei-zh has joined #openstack-nova | 02:16 | |
| *** coreywright has joined #openstack-nova | 02:16 | |
| *** jichen has joined #openstack-nova | 02:17 | |
| *** mingyu has joined #openstack-nova | 02:18 | |
| *** xinliang has joined #openstack-nova | 02:19 | |
| *** liuqing has quit IRC | 02:20 | |
| *** liuqing has joined #openstack-nova | 02:20 | |
| *** READ10 has quit IRC | 02:22 | |
| *** READ10 has joined #openstack-nova | 02:22 | |
| mriedem | alex_xu: maybe, but it's kind of weird to say, "none of this does anything w/o the code actually running" :) | 02:25 |
| mriedem | that's kind of assumed | 02:25 |
| mriedem | i was thinking maybe it was something people talked about in earlier patch sets but didn't actually do something | 02:26 |
| *** hongbin has joined #openstack-nova | 02:26 | |
| *** thorst has joined #openstack-nova | 02:26 | |
| *** thorst has quit IRC | 02:26 | |
| *** links has joined #openstack-nova | 02:27 | |
| *** trinaths has quit IRC | 02:27 | |
| *** jwcroppe has joined #openstack-nova | 02:29 | |
| *** links has quit IRC | 02:30 | |
| *** Apoorva has joined #openstack-nova | 02:30 | |
| *** links has joined #openstack-nova | 02:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923 | 02:35 |
| *** Apoorva has quit IRC | 02:39 | |
| *** thorst has joined #openstack-nova | 02:42 | |
| *** thorst has quit IRC | 02:42 | |
| *** thorst has joined #openstack-nova | 02:43 | |
| *** READ10 has quit IRC | 02:44 | |
| *** tiendc has joined #openstack-nova | 02:47 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: Fix the race condition with novnc https://review.openstack.org/470798 | 02:47 |
| *** thorst has quit IRC | 02:47 | |
| openstackgerrit | Merged openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 02:52 |
| openstackgerrit | Vu Cong Tuan proposed openstack/nova master: Fix html_last_updated_fmt for Python3 https://review.openstack.org/470676 | 02:53 |
| *** ericyoung has quit IRC | 02:57 | |
| *** diga has joined #openstack-nova | 02:58 | |
| *** slaweq has joined #openstack-nova | 02:59 | |
| *** dimtruck is now known as zz_dimtruck | 03:01 | |
| *** slaweq has quit IRC | 03:04 | |
| *** zsli_ has joined #openstack-nova | 03:05 | |
| *** Shunli has quit IRC | 03:07 | |
| *** cNilesh has joined #openstack-nova | 03:09 | |
| *** Shunli has joined #openstack-nova | 03:13 | |
| *** thorst has joined #openstack-nova | 03:14 | |
| *** mriedem has quit IRC | 03:14 | |
| *** vks1 has joined #openstack-nova | 03:15 | |
| *** zsli_ has quit IRC | 03:16 | |
| *** klkumar has joined #openstack-nova | 03:18 | |
| *** gomarivera has quit IRC | 03:21 | |
| *** gomarivera has joined #openstack-nova | 03:22 | |
| *** gomarivera has quit IRC | 03:23 | |
| *** gomarivera has joined #openstack-nova | 03:23 | |
| *** yamamoto_ has joined #openstack-nova | 03:25 | |
| *** zhurong has joined #openstack-nova | 03:28 | |
| *** thorst has quit IRC | 03:32 | |
| *** gomarivera has quit IRC | 03:34 | |
| *** gomarivera has joined #openstack-nova | 03:34 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 03:38 |
| *** gomarivera has quit IRC | 03:38 | |
| *** gomarivera has joined #openstack-nova | 03:44 | |
| *** vladikr_ has quit IRC | 03:47 | |
| *** jwnx has joined #openstack-nova | 03:54 | |
| *** hshiina has quit IRC | 03:58 | |
| *** slaweq has joined #openstack-nova | 04:00 | |
| *** hshiina has joined #openstack-nova | 04:01 | |
| *** slaweq has quit IRC | 04:05 | |
| *** psachin has joined #openstack-nova | 04:06 | |
| *** hongbin has quit IRC | 04:16 | |
| *** mingyu has quit IRC | 04:16 | |
| *** salv-orlando has joined #openstack-nova | 04:26 | |
| *** zhurong has quit IRC | 04:27 | |
| *** gouthamr has quit IRC | 04:30 | |
| *** salv-orlando has quit IRC | 04:31 | |
| *** takashin has quit IRC | 04:32 | |
| *** hshiina has quit IRC | 04:32 | |
| *** mingyu has joined #openstack-nova | 04:39 | |
| *** diga has quit IRC | 04:39 | |
| *** lei-zh has quit IRC | 04:40 | |
| *** edmondsw has joined #openstack-nova | 04:41 | |
| *** armax has joined #openstack-nova | 04:44 | |
| *** edmondsw has quit IRC | 04:46 | |
| *** mingyu has quit IRC | 04:46 | |
| *** gongysh has quit IRC | 04:50 | |
| *** takashin has joined #openstack-nova | 04:52 | |
| *** efried has quit IRC | 04:52 | |
| *** zhurong has joined #openstack-nova | 04:54 | |
| *** salv-orlando has joined #openstack-nova | 05:00 | |
| *** sridharg has joined #openstack-nova | 05:00 | |
| *** slaweq has joined #openstack-nova | 05:01 | |
| *** efried has joined #openstack-nova | 05:04 | |
| *** slaweq has quit IRC | 05:06 | |
| *** dikonoor has joined #openstack-nova | 05:07 | |
| *** yamamoto_ has quit IRC | 05:07 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 05:08 |
| *** tuanluong has quit IRC | 05:08 | |
| *** ltomasbo|away is now known as ltomasbo | 05:10 | |
| *** jwnx has quit IRC | 05:12 | |
| *** gyee has quit IRC | 05:12 | |
| *** prateek has joined #openstack-nova | 05:12 | |
| *** sree has joined #openstack-nova | 05:16 | |
| *** mingyu has joined #openstack-nova | 05:26 | |
| *** ekuris has joined #openstack-nova | 05:27 | |
| *** lei-zh has joined #openstack-nova | 05:28 | |
| *** thorst has joined #openstack-nova | 05:29 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 05:30 |
| *** gongysh has joined #openstack-nova | 05:30 | |
| *** mamandle has joined #openstack-nova | 05:33 | |
| *** thorst has quit IRC | 05:34 | |
| *** mingyu has quit IRC | 05:34 | |
| *** mingyu has joined #openstack-nova | 05:35 | |
| *** mingyu has quit IRC | 05:40 | |
| *** mingyu has joined #openstack-nova | 05:41 | |
| *** udesale has joined #openstack-nova | 05:41 | |
| *** jwcroppe has quit IRC | 05:42 | |
| *** kwathore has quit IRC | 05:42 | |
| *** jaosorior_away is now known as jaosorior | 05:44 | |
| *** mingyu_ has joined #openstack-nova | 05:46 | |
| *** mingyu has quit IRC | 05:46 | |
| *** mingyu_ has quit IRC | 05:47 | |
| *** mingyu has joined #openstack-nova | 05:48 | |
| *** zsli_ has joined #openstack-nova | 05:48 | |
| *** zsli__ has joined #openstack-nova | 05:49 | |
| *** Shunli has quit IRC | 05:51 | |
| *** yamamoto has joined #openstack-nova | 05:51 | |
| *** zsli_ has quit IRC | 05:52 | |
| *** mingyu has quit IRC | 05:53 | |
| *** zsli_ has joined #openstack-nova | 05:55 | |
| *** slaweq has joined #openstack-nova | 05:55 | |
| *** slaweq has quit IRC | 05:56 | |
| *** zsli__ has quit IRC | 05:57 | |
| *** trinaths has joined #openstack-nova | 05:57 | |
| *** zsli__ has joined #openstack-nova | 05:58 | |
| *** trinaths has left #openstack-nova | 05:59 | |
| *** zsli_ has quit IRC | 06:00 | |
| *** slaweq has joined #openstack-nova | 06:00 | |
| *** slaweq has quit IRC | 06:00 | |
| *** gomarivera has quit IRC | 06:01 | |
| *** gomarivera has joined #openstack-nova | 06:01 | |
| *** tuanluong has joined #openstack-nova | 06:04 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 06:05 |
| *** gomarivera has quit IRC | 06:06 | |
| *** mingyu has joined #openstack-nova | 06:07 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 06:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 06:07 |
| *** dikonoor has quit IRC | 06:09 | |
| *** kwathore has joined #openstack-nova | 06:10 | |
| *** rcernin has joined #openstack-nova | 06:10 | |
| *** jwcroppe has joined #openstack-nova | 06:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 06:12 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 06:12 |
| *** salv-orlando has quit IRC | 06:13 | |
| *** salv-orlando has joined #openstack-nova | 06:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | 06:16 |
| *** salv-orlando has quit IRC | 06:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 06:17 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923 | 06:18 |
| *** Oku_OS-away is now known as Oku_OS | 06:19 | |
| *** bkopilov has joined #openstack-nova | 06:22 | |
| *** itlinux_ has joined #openstack-nova | 06:29 | |
| *** thorst has joined #openstack-nova | 06:30 | |
| *** zhurong has quit IRC | 06:32 | |
| *** thorst has quit IRC | 06:35 | |
| *** udesale has quit IRC | 06:37 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 06:39 |
| *** pcaruana has joined #openstack-nova | 06:39 | |
| *** zhurong has joined #openstack-nova | 06:39 | |
| *** jaosorior is now known as jaosorior_away | 06:41 | |
| *** udesale has joined #openstack-nova | 06:42 | |
| *** yamamoto has quit IRC | 06:46 | |
| *** yamamoto has joined #openstack-nova | 06:47 | |
| *** jwcroppe has quit IRC | 06:47 | |
| *** yamahata has joined #openstack-nova | 06:47 | |
| *** patriciadomin has quit IRC | 06:49 | |
| *** markus_z has joined #openstack-nova | 06:50 | |
| *** danpawlik has quit IRC | 06:50 | |
| *** dpawlik has joined #openstack-nova | 06:50 | |
| *** patriciadomin has joined #openstack-nova | 06:51 | |
| *** itlinux_ has quit IRC | 06:51 | |
| *** udesale__ has joined #openstack-nova | 06:54 | |
| *** udesale has quit IRC | 06:56 | |
| *** trinaths1 has joined #openstack-nova | 06:57 | |
| *** trinaths1 has left #openstack-nova | 06:57 | |
| *** salv-orlando has joined #openstack-nova | 07:01 | |
| *** aarefiev has joined #openstack-nova | 07:03 | |
| *** trinaths has joined #openstack-nova | 07:04 | |
| *** eliqiao has quit IRC | 07:09 | |
| *** eliqiao has joined #openstack-nova | 07:10 | |
| *** tesseract has joined #openstack-nova | 07:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 07:13 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 07:14 |
| *** mingyu has quit IRC | 07:17 | |
| *** CristinaPauna has joined #openstack-nova | 07:20 | |
| *** klkumar has quit IRC | 07:21 | |
| *** yamahata has quit IRC | 07:23 | |
| *** klkumar has joined #openstack-nova | 07:23 | |
| *** cfriesen_ has quit IRC | 07:26 | |
| *** salv-orl_ has joined #openstack-nova | 07:28 | |
| *** armax has quit IRC | 07:29 | |
| *** jwcroppe has joined #openstack-nova | 07:29 | |
| *** salv-orlando has quit IRC | 07:31 | |
| *** nicolasbock has joined #openstack-nova | 07:31 | |
| *** thorst has joined #openstack-nova | 07:31 | |
| *** jpena|off is now known as jpena | 07:32 | |
| *** fnordahl has quit IRC | 07:32 | |
| *** thorst has quit IRC | 07:35 | |
| *** mingyu has joined #openstack-nova | 07:38 | |
| *** fnordahl has joined #openstack-nova | 07:39 | |
| *** trinaths has left #openstack-nova | 07:40 | |
| *** oanson_ has joined #openstack-nova | 07:42 | |
| *** pjm6 has quit IRC | 07:45 | |
| *** pjm6 has joined #openstack-nova | 07:46 | |
| *** alexchadin has joined #openstack-nova | 07:48 | |
| *** oanson_ has quit IRC | 07:50 | |
| openstackgerrit | Merged openstack/python-novaclient master: client.logger.warning wrongly used in migrations https://review.openstack.org/463508 | 07:51 |
| *** pjm6 has quit IRC | 07:51 | |
| *** pjm6 has joined #openstack-nova | 07:53 | |
| *** Hosam has joined #openstack-nova | 07:54 | |
| *** Hosam_ has joined #openstack-nova | 07:54 | |
| *** roeyc has joined #openstack-nova | 07:56 | |
| *** markvoelker has quit IRC | 07:57 | |
| *** markvoelker has joined #openstack-nova | 07:58 | |
| *** Hosam has quit IRC | 07:58 | |
| *** roeyc1 has joined #openstack-nova | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** roeyc has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** links has quit IRC | 08:02 | |
| *** gomarivera has joined #openstack-nova | 08:02 | |
| *** markvoelker has quit IRC | 08:02 | |
| *** pjm6 has quit IRC | 08:02 | |
| *** pjm6 has joined #openstack-nova | 08:03 | |
| *** gomarivera has quit IRC | 08:07 | |
| bauzas | p!spy | 08:09 |
| *** vks1 has quit IRC | 08:13 | |
| *** edmondsw has joined #openstack-nova | 08:17 | |
| *** links has joined #openstack-nova | 08:19 | |
| *** slaweq has joined #openstack-nova | 08:19 | |
| *** slaweq has quit IRC | 08:20 | |
| *** edmondsw has quit IRC | 08:22 | |
| *** mvk has quit IRC | 08:22 | |
| *** lucas-afk is now known as lucasagomes | 08:26 | |
| *** thorst has joined #openstack-nova | 08:32 | |
| *** zhurong has quit IRC | 08:35 | |
| *** karimb has joined #openstack-nova | 08:38 | |
| *** zhurong has joined #openstack-nova | 08:41 | |
| *** derekh has joined #openstack-nova | 08:43 | |
| *** fragatina has joined #openstack-nova | 08:45 | |
| *** mingyu has quit IRC | 08:48 | |
| *** priteau has joined #openstack-nova | 08:49 | |
| *** kwathore has quit IRC | 08:50 | |
| *** thorst has quit IRC | 08:51 | |
| *** hoonetorg has quit IRC | 08:51 | |
| *** oanson_ has joined #openstack-nova | 08:51 | |
| *** mvk has joined #openstack-nova | 08:52 | |
| *** jaosorior_away is now known as jaosorior | 08:53 | |
| *** nicolasbock has quit IRC | 09:03 | |
| *** jaosorior has quit IRC | 09:04 | |
| *** hoonetorg has joined #openstack-nova | 09:09 | |
| *** huanxie has quit IRC | 09:11 | |
| *** mnestratov has joined #openstack-nova | 09:11 | |
| *** huanxie has joined #openstack-nova | 09:12 | |
| *** mingyu has joined #openstack-nova | 09:13 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove incorrect comments in multiple_create https://review.openstack.org/471273 | 09:13 |
| *** Guest55399 has quit IRC | 09:15 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Modify incorrect comment on return_reservation_id https://review.openstack.org/471278 | 09:16 |
| *** Hosam_ has quit IRC | 09:19 | |
| *** Hosam has joined #openstack-nova | 09:19 | |
| *** jichen has quit IRC | 09:21 | |
| *** kwathore has joined #openstack-nova | 09:23 | |
| *** sambetts|afk is now known as sambetts | 09:24 | |
| *** links has quit IRC | 09:25 | |
| *** zsli__ has quit IRC | 09:30 | |
| *** hshiina has joined #openstack-nova | 09:31 | |
| *** jwcroppe has quit IRC | 09:33 | |
| *** zhurong has quit IRC | 09:33 | |
| *** lyarwood_ is now known as lyarwood | 09:34 | |
| *** lei-zh has quit IRC | 09:35 | |
| *** lyarwood is now known as lyarwood_ | 09:35 | |
| *** takashin has left #openstack-nova | 09:40 | |
| *** links has joined #openstack-nova | 09:42 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add PCIWeigher https://review.openstack.org/379524 | 09:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Prefer non-PCI host nodes for non-PCI instances https://review.openstack.org/379625 | 09:44 |
| *** sdague has joined #openstack-nova | 09:44 | |
| sfinucan | bauzas: Can you take a look at https://review.openstack.org/#/c/438887/ today? | 09:46 |
| *** mdnadeem has quit IRC | 09:47 | |
| *** alexchadin has quit IRC | 09:48 | |
| *** thorst has joined #openstack-nova | 09:48 | |
| *** alexchadin has joined #openstack-nova | 09:48 | |
| *** slaweq has joined #openstack-nova | 09:49 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Revert "Cap sphinx<1.6.1" https://review.openstack.org/471298 | 09:52 |
| *** thorst has quit IRC | 09:52 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Revert "Cap sphinx<1.6.1" https://review.openstack.org/471298 | 09:52 |
| *** alexchadin has quit IRC | 09:53 | |
| *** udesale has joined #openstack-nova | 09:53 | |
| *** mvk has quit IRC | 09:53 | |
| *** udesale__ has quit IRC | 09:54 | |
| *** slaweq has quit IRC | 09:54 | |
| *** slaweq has joined #openstack-nova | 09:54 | |
| *** oanson_ has quit IRC | 09:55 | |
| *** cdent has joined #openstack-nova | 09:57 | |
| *** alexchadin has joined #openstack-nova | 09:57 | |
| *** mvk has joined #openstack-nova | 09:58 | |
| *** fragatina has quit IRC | 09:58 | |
| *** tbachman has joined #openstack-nova | 09:58 | |
| *** fragatina has joined #openstack-nova | 09:59 | |
| *** markvoelker has joined #openstack-nova | 09:59 | |
| *** slaweq has quit IRC | 09:59 | |
| *** lpetrut has joined #openstack-nova | 10:03 | |
| *** abalutoiu has joined #openstack-nova | 10:05 | |
| *** slaweq has joined #openstack-nova | 10:07 | |
| *** slaweq has quit IRC | 10:07 | |
| *** jamielennox is now known as jamielennox|away | 10:10 | |
| *** tovin07_ has quit IRC | 10:11 | |
| *** cNilesh has quit IRC | 10:12 | |
| *** cNilesh has joined #openstack-nova | 10:13 | |
| *** jamielennox|away is now known as jamielennox | 10:16 | |
| *** dpawlik has quit IRC | 10:20 | |
| *** liujiong has quit IRC | 10:23 | |
| *** cNilesh has quit IRC | 10:23 | |
| *** Hosam has quit IRC | 10:25 | |
| *** gongysh has quit IRC | 10:26 | |
| *** danpawlik has joined #openstack-nova | 10:29 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database table https://review.openstack.org/469578 | 10:31 |
| *** markvoelker has quit IRC | 10:32 | |
| *** tbachman has quit IRC | 10:33 | |
| *** mnestratov is now known as mnestratov|3 | 10:33 | |
| *** purplerbot has quit IRC | 10:33 | |
| *** purplerbot has joined #openstack-nova | 10:33 | |
| *** tbachman has joined #openstack-nova | 10:38 | |
| *** klkumar has quit IRC | 10:38 | |
| *** tbachman_ has joined #openstack-nova | 10:41 | |
| *** tbachman has quit IRC | 10:43 | |
| *** tbachman_ is now known as tbachman | 10:43 | |
| *** nicolasbock has joined #openstack-nova | 10:44 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 10:48 |
| *** kwathore_ has joined #openstack-nova | 10:49 | |
| *** kwathore has quit IRC | 10:51 | |
| *** yamamoto has quit IRC | 10:52 | |
| *** sapcc-bot has quit IRC | 10:52 | |
| *** david_ has quit IRC | 10:52 | |
| *** carthaca_ has quit IRC | 10:52 | |
| *** databus23_ has quit IRC | 10:52 | |
| *** dgonzalez_ has joined #openstack-nova | 10:52 | |
| *** sapcc-bot has joined #openstack-nova | 10:52 | |
| *** david_ has joined #openstack-nova | 10:52 | |
| *** dgonzalez_ has quit IRC | 10:54 | |
| *** david_ has quit IRC | 10:54 | |
| *** klkumar has joined #openstack-nova | 10:57 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | 10:58 |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385 | 10:58 |
| *** sree has quit IRC | 10:58 | |
| *** tbachman has quit IRC | 10:59 | |
| *** bkopilov has quit IRC | 11:01 | |
| *** phuongnh has quit IRC | 11:07 | |
| sdague | anyone else want to bring this one in - https://review.openstack.org/#/c/467242/ - and get us one step closer? | 11:07 |
| *** brault has joined #openstack-nova | 11:10 | |
| *** mingyu has quit IRC | 11:13 | |
| *** salv-orl_ has quit IRC | 11:16 | |
| *** mamandle has quit IRC | 11:16 | |
| *** lucasagomes is now known as lucas-hungry | 11:16 | |
| *** vladikr_ has joined #openstack-nova | 11:22 | |
| *** brault has quit IRC | 11:26 | |
| *** markvoelker has joined #openstack-nova | 11:29 | |
| *** zhurong has joined #openstack-nova | 11:31 | |
| *** jamesdenton has joined #openstack-nova | 11:32 | |
| *** udesale has quit IRC | 11:34 | |
| *** jamesdenton has quit IRC | 11:37 | |
| *** udesale has joined #openstack-nova | 11:37 | |
| *** gouthamr has joined #openstack-nova | 11:40 | |
| *** klkumar has quit IRC | 11:42 | |
| *** jamesdenton has joined #openstack-nova | 11:43 | |
| *** jpena is now known as jpena|lunch | 11:47 | |
| *** brault has joined #openstack-nova | 11:48 | |
| *** mingyu has joined #openstack-nova | 11:48 | |
| *** rcernin has quit IRC | 11:52 | |
| *** abalutoiu_ has joined #openstack-nova | 11:53 | |
| *** abalutoiu has quit IRC | 11:53 | |
| *** pcaruana has quit IRC | 11:53 | |
| *** thorst has joined #openstack-nova | 11:54 | |
| *** edmondsw has joined #openstack-nova | 11:55 | |
| *** jaosorior has joined #openstack-nova | 11:56 | |
| *** yamamoto has joined #openstack-nova | 11:56 | |
| *** mingyu has quit IRC | 11:56 | |
| *** litao__ has quit IRC | 11:58 | |
| *** jwcroppe has joined #openstack-nova | 11:59 | |
| *** tbachman has joined #openstack-nova | 12:01 | |
| *** markvoelker has quit IRC | 12:03 | |
| *** Hosam has joined #openstack-nova | 12:04 | |
| *** jamesdenton has quit IRC | 12:05 | |
| *** luisnho223 has joined #openstack-nova | 12:05 | |
| *** rcernin has joined #openstack-nova | 12:07 | |
| *** rfolco has joined #openstack-nova | 12:07 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Remove Babel from requirements.txt https://review.openstack.org/468350 | 12:07 |
| *** baoli has joined #openstack-nova | 12:08 | |
| luisnho223 | hey guys one quick question. I am doing a request to get stats from my tenant/project and everything works well but when i see get the hours for each instance/server or total GB/RAM/Hours from the tenants, the response give me numbers like 1,99999e^-9 | 12:08 |
| luisnho223 | btw, on Horizon and CLI everything is working fine | 12:09 |
| *** baoli_ has joined #openstack-nova | 12:09 | |
| *** pcaruana has joined #openstack-nova | 12:09 | |
| *** klkumar has joined #openstack-nova | 12:11 | |
| *** zhurong has quit IRC | 12:12 | |
| *** baoli has quit IRC | 12:13 | |
| *** udesale__ has joined #openstack-nova | 12:14 | |
| *** gszasz has joined #openstack-nova | 12:14 | |
| jaypipes | luisnho223: pls see /topic. best to ask usage questions on the openstack@ Mailing List or ask.openstack.org, for future reference... | 12:15 |
| *** dave-mccowan has joined #openstack-nova | 12:16 | |
| jaypipes | luisnho223: when you say "doing a request to get stats", you mean you're doing a curl/http request directly to the Nova API? | 12:16 |
| *** udesale has quit IRC | 12:16 | |
| *** lucas-hungry is now known as lucasagomes | 12:16 | |
| *** tuanluong has quit IRC | 12:17 | |
| jaypipes | lucasagomes: how was breakfast? :) | 12:17 |
| *** Hosam has quit IRC | 12:18 | |
| lucasagomes | jaypipes, lol hey there! It was lunch, pretty good actually :D | 12:18 |
| lucasagomes | jaypipes, how r you btw ? | 12:18 |
| jaypipes | lucasagomes: doing ok I guess :) | 12:18 |
| jaypipes | lucasagomes: shortly about to head out to California for a brief business meeting trip... | 12:18 |
| lucasagomes | jaypipes, oh nice one! Have a safe trip, enjoy it | 12:19 |
| jaypipes | lucasagomes: thx! | 12:19 |
| *** markvoelker has joined #openstack-nova | 12:20 | |
| *** luisnho223 has quit IRC | 12:20 | |
| *** luisnho223_ has joined #openstack-nova | 12:20 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: code comments incorrectness https://review.openstack.org/449416 | 12:21 |
| *** yamamoto has quit IRC | 12:21 | |
| *** jamesdenton has joined #openstack-nova | 12:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/469581 | 12:25 |
| *** peter-hamilton has joined #openstack-nova | 12:25 | |
| *** claudiub has joined #openstack-nova | 12:26 | |
| *** MVenesio has joined #openstack-nova | 12:27 | |
| *** pchavva has joined #openstack-nova | 12:28 | |
| *** hshiina has quit IRC | 12:33 | |
| *** yamamoto has joined #openstack-nova | 12:38 | |
| *** gouthamr has quit IRC | 12:39 | |
| *** liverpooler has joined #openstack-nova | 12:39 | |
| *** mingyu has joined #openstack-nova | 12:39 | |
| *** catintheroof has joined #openstack-nova | 12:42 | |
| sfinucan | sdague: This would need unit tests, no? https://review.openstack.org/#/c/410815/ | 12:46 |
| *** lyan has joined #openstack-nova | 12:48 | |
| *** gouthamr has joined #openstack-nova | 12:51 | |
| *** jpena|lunch is now known as jpena | 12:51 | |
| *** links has quit IRC | 12:56 | |
| *** sridharg has quit IRC | 12:57 | |
| *** jamesdenton has quit IRC | 12:57 | |
| *** efoley has joined #openstack-nova | 12:59 | |
| sdague | sfinucan: are you imaginging anything other than checking that exception translation? | 12:59 |
| sdague | sfinucan: I'm fine if a test is added there | 13:00 |
| sfinucan | sdague: Assert that FloatingIpNotFoundForAddress is correctly transformed to HTTPNotFound? | 13:00 |
| *** felipemonteiro has joined #openstack-nova | 13:00 | |
| *** efoley has quit IRC | 13:00 | |
| *** efoley has joined #openstack-nova | 13:00 | |
| sfinucan | we surely have similar tests already there | 13:00 |
| * sfinucan looks | 13:00 | |
| sdague | sfinucan: we might, I'm just always very questioning of the utility of such things. But that's fine | 13:01 |
| *** mdrabe has joined #openstack-nova | 13:02 | |
| *** felipemonteiro has quit IRC | 13:04 | |
| *** lucasxu has joined #openstack-nova | 13:05 | |
| *** mriedem has joined #openstack-nova | 13:07 | |
| *** sridharg has joined #openstack-nova | 13:08 | |
| *** yamamoto has quit IRC | 13:10 | |
| *** jamesdenton has joined #openstack-nova | 13:11 | |
| *** yamamoto has joined #openstack-nova | 13:11 | |
| *** ftersin has joined #openstack-nova | 13:13 | |
| *** jmlowe has joined #openstack-nova | 13:17 | |
| *** gomarivera has joined #openstack-nova | 13:17 | |
| *** jamesdenton has quit IRC | 13:20 | |
| *** yamamoto has quit IRC | 13:21 | |
| *** yamamoto has joined #openstack-nova | 13:23 | |
| *** sridharg has quit IRC | 13:25 | |
| *** smatzek has joined #openstack-nova | 13:25 | |
| *** mamandle has joined #openstack-nova | 13:28 | |
| *** gomarivera has quit IRC | 13:30 | |
| mriedem | melwitt: mdbooth: lyarwood_: we've seen this with volume detach right? http://logs.openstack.org/24/471024/2/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/6b98d38/logs/screen-n-cpu.txt.gz?level=TRACE#_Jun_06_02_16_02_855503 - i'm seeing it with vif detach now too | 13:30 |
| * mdbooth clicks | 13:30 | |
| *** baoli_ has quit IRC | 13:31 | |
| mriedem | maybe related to https://review.openstack.org/#/c/441204/ ? | 13:31 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999 | 13:31 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Fixed some nits for microversion 2.48 https://review.openstack.org/471348 | 13:32 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fixed some nits for microversion 2.48 https://review.openstack.org/471348 | 13:34 |
| *** pcaruana has quit IRC | 13:34 | |
| *** baoli has joined #openstack-nova | 13:36 | |
| *** rcernin has quit IRC | 13:36 | |
| *** udesale__ has quit IRC | 13:37 | |
| *** kfarr has joined #openstack-nova | 13:37 | |
| mdbooth | mriedem: Certainly looks potentially related. It's annoying we've lost the interesting part of the stack trace there. | 13:38 |
| *** cleong has joined #openstack-nova | 13:38 | |
| *** sridharg has joined #openstack-nova | 13:38 | |
| openstackgerrit | Felipe Monteiro proposed openstack/nova-specs master: Patrole (RBAC) Keystone Gating https://review.openstack.org/471145 | 13:40 |
| mriedem | ok i'll work on cleaning up lee's patch | 13:40 |
| *** prateek has quit IRC | 13:40 | |
| *** zz_dimtruck is now known as dimtruck | 13:41 | |
| *** psachin has quit IRC | 13:41 | |
| *** abalutoiu_ is now known as abalutoiu | 13:41 | |
| *** karimb has quit IRC | 13:42 | |
| *** awaugama has joined #openstack-nova | 13:43 | |
| *** karimb has joined #openstack-nova | 13:43 | |
| *** abalutoiu has quit IRC | 13:44 | |
| *** abalutoiu_ has joined #openstack-nova | 13:44 | |
| sdague | mriedem: I pushed up a release request for python-novaclient - https://review.openstack.org/#/c/471323/ | 13:45 |
| sdague | please review when you get a chance | 13:45 |
| mriedem | hmm, that might be a 9.0 | 13:46 |
| sdague | mriedem: because of the 1.1 removes? | 13:46 |
| sdague | I was wondering that | 13:46 |
| sdague | but that code has been effectively dead for so long | 13:46 |
| mriedem | was thinking more the deprecated option removals https://docs.openstack.org/releasenotes/python-novaclient/unreleased.html#upgrade-notes | 13:47 |
| *** abalutoiu_ has quit IRC | 13:47 | |
| *** thorst is now known as thorst_afk | 13:47 | |
| sdague | ok, I can go either way. You want me to make it 9.0? | 13:48 |
| mriedem | weird, the 2.32 reno is showing up in 8.0.0-33 as a new feature | 13:48 |
| *** abalutoiu has joined #openstack-nova | 13:48 | |
| *** abalutoiu has quit IRC | 13:48 | |
| mriedem | i'm more worried about the misleading renos than the version number | 13:48 |
| sdague | ok | 13:48 |
| *** mingyu has quit IRC | 13:48 | |
| *** abalutoiu has joined #openstack-nova | 13:48 | |
| *** rcernin has joined #openstack-nova | 13:49 | |
| *** jamesdenton has joined #openstack-nova | 13:51 | |
| *** pcaruana has joined #openstack-nova | 13:51 | |
| *** itlinux_ has joined #openstack-nova | 13:53 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 13:53 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Improve coverage of floating IP delete https://review.openstack.org/471360 | 13:53 |
| *** mingyu has joined #openstack-nova | 13:54 | |
| sfinucan | sdague: ^ if you don't feel the latter is actually useful, I'm happy to drop it | 13:54 |
| sdague | sfinucan: works for me | 13:55 |
| *** psachin has joined #openstack-nova | 13:55 | |
| *** tidwellr has joined #openstack-nova | 13:55 | |
| mriedem | sdague: sfinucan: no tests needed for https://review.openstack.org/#/c/410815/ ? | 13:56 |
| sfinucan | mriedem: See above | 13:57 |
| mriedem | sfinucan: that doesn't test the neutronv2.api change | 13:57 |
| sfinucan | I broke them out into a separate change because none of those conditions had coverage | 13:57 |
| openstackgerrit | Sean Dague proposed openstack/nova master: Fix deprecation warning around get_transport() usage https://review.openstack.org/471362 | 13:57 |
| sfinucan | mriedem: https://review.openstack.org/#/c/471360/1/nova/tests/unit/api/openstack/compute/test_floating_ips.py doesn't? | 13:57 |
| mriedem | sdague: i've already got a patch for that | 13:58 |
| sdague | mriedem: ok, where? | 13:58 |
| mriedem | https://review.openstack.org/#/c/469659/ | 13:58 |
| *** klkumar has quit IRC | 13:58 | |
| sfinucan | mriedem: I can squash the two before it merges if you'd prefer? | 13:58 |
| mriedem | sfinucan: it doesn't because it's mocking out the call to the network_api | 13:59 |
| sfinucan | oh, wait - I getcha you now | 13:59 |
| mriedem | i don't really need the big ddt rewrite in the bug fix, | 13:59 |
| mriedem | because if we backport the fix, ddt isn't used in earlier branches | 13:59 |
| sdague | mriedem: good point | 13:59 |
| mriedem | but that's exactly why we should have tests for a bug fix | 13:59 |
| mriedem | and refactor/cleanup later | 13:59 |
| *** pcaruana has quit IRC | 13:59 | |
| *** pcaruana has joined #openstack-nova | 14:00 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 14:00 |
| mriedem | ^ just pulled it out of the queue | 14:00 |
| *** vks1 has joined #openstack-nova | 14:01 | |
| sfinucan | that's fair. I'll fix that up now | 14:01 |
| mriedem | thanks | 14:01 |
| *** jamesdenton has quit IRC | 14:02 | |
| *** salv-orlando has joined #openstack-nova | 14:04 | |
| *** rcernin has quit IRC | 14:04 | |
| *** rcernin has joined #openstack-nova | 14:05 | |
| mriedem | sdague: on the novaclient reno thing, i've been bugging doug with reno stuff multiple times in the last week or so, so i'm hesitant to bug him about another one here | 14:05 |
| mriedem | but someone else might have some karma with doug to ask about it... :) | 14:06 |
| mriedem | that reno was added https://github.com/openstack/python-novaclient/commit/7fb0bd4f357a7f9e24c864584b398a25171580cf and tagged in 5.0.0 | 14:06 |
| mriedem | so it shouldn't be showing up again | 14:06 |
| openstackgerrit | Felipe Monteiro proposed openstack/nova-specs master: Patrole (RBAC) Keystone Gating https://review.openstack.org/471145 | 14:06 |
| mriedem | actually nvm | 14:06 |
| mriedem | i know exactly what it is | 14:06 |
| mriedem | https://github.com/openstack/python-novaclient/commit/aa7f35d76b5768ec3fe4d45ad010d53adff7cccd | 14:06 |
| mriedem | someone updated the reno | 14:06 |
| mriedem | i'm going to revert that | 14:07 |
| *** rcernin has quit IRC | 14:07 | |
| *** rcernin has joined #openstack-nova | 14:07 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Revert "Fix a typo" https://review.openstack.org/471365 | 14:08 |
| mriedem | sdague: ^ | 14:08 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Revert "Fix a typo" https://review.openstack.org/471365 | 14:08 |
| *** tbachman has quit IRC | 14:08 | |
| *** sdatko has joined #openstack-nova | 14:09 | |
| *** burt has joined #openstack-nova | 14:09 | |
| sdague | mriedem: +A | 14:09 |
| sdague | yeh, when the reno magic breaks, it's kind of sad | 14:09 |
| *** klkumar has joined #openstack-nova | 14:10 | |
| *** jaypipes has quit IRC | 14:10 | |
| * mriedem makes sure that actually does what i want | 14:11 | |
| *** READ10 has joined #openstack-nova | 14:11 | |
| *** eharney has joined #openstack-nova | 14:11 | |
| *** tuanla_fujitsu has joined #openstack-nova | 14:11 | |
| *** mdrabe has quit IRC | 14:11 | |
| mriedem | i've got something you might like, | 14:12 |
| sfinucan | mriedem: So you can't update a reno once it's in a major release? | 14:12 |
| *** mdrabe has joined #openstack-nova | 14:12 | |
| *** ekuris has quit IRC | 14:12 | |
| mriedem | while working on removing some deprecated nova-network things from devstack https://review.openstack.org/#/c/471004/ - i found that tempest is using nova's floating IP proxy API for all things floating IP for ssh validation https://review.openstack.org/#/c/471116/ | 14:12 |
| *** links has joined #openstack-nova | 14:12 | |
| mriedem | sfinucan: i guess not. honestly the rules around reno seem to change for me per case | 14:13 |
| *** tbachman has joined #openstack-nova | 14:13 | |
| mriedem | because i know we've amended release notes that are tagged for milestones in nova, | 14:13 |
| mriedem | but server side it's less of an issue probably, or noticeable | 14:13 |
| mriedem | on the client it's much more obvious and weird | 14:13 |
| *** r-daneel has joined #openstack-nova | 14:14 | |
| sfinucan | Hmm, that sounds like a reno bug 🤔 | 14:14 |
| sfinucan | or at least something that should be documented better | 14:14 |
| * sfinucan _has_ fixed typos in renos, but only after a "pre-release" tag | 14:15 | |
| bauzas | sfinucan: mriedem: you *can* fix notes after a major release | 14:15 |
| *** armax has joined #openstack-nova | 14:15 | |
| mriedem | can and should are different things | 14:16 |
| bauzas | sfinucan: mriedem: but the only problem is that you need to backport it to the stable branch then | 14:16 |
| mriedem | maybe that was the issue, | 14:16 |
| sfinucan | oh, that would make sense | 14:16 |
| mriedem | they should have fixed the typo in stable/newton instead of master | 14:16 |
| * sfinucan wasn't checking any branch but master in above case | 14:16 | |
| bauzas | the problem is not really with branching | 14:16 |
| bauzas | it's rather with tagging | 14:17 |
| *** tiendc has quit IRC | 14:17 | |
| bauzas | since we tag a release by a git commit, it's not possible to update relnotes for a specific release | 14:17 |
| *** links has quit IRC | 14:17 | |
| mriedem | hmm, still there http://docs-draft.openstack.org/65/471365/2/check/gate-python-novaclient-releasenotes/7a8a8ff//releasenotes/build/html/unreleased.html | 14:18 |
| bauzas | it will be seen in the branch for sure, but not for the tag | 14:18 |
| bauzas | mriedem: what is the exact problem ? | 14:18 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert additional floating IP tests to ddt https://review.openstack.org/471360 | 14:18 |
| bauzas | I was just joining the discussion | 14:18 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 14:18 |
| sfinucan | ^ mriedem, sdague | 14:18 |
| sfinucan | ddt is pretty nifty. No wonder mdbooth likes it so much | 14:19 |
| sdague | sfinucan: I thought the ddt stuff was not going to be put in there | 14:19 |
| sdague | because that prevents backport | 14:19 |
| sfinucan | Oh | 14:19 |
| sfinucan | I thought he meant the conversion to ddt, which I kept separate | 14:19 |
| sfinucan | ddt in general? | 14:19 |
| mriedem | we don't use ddt in newton | 14:19 |
| mriedem | if this is going to go back that far | 14:19 |
| sfinucan | gotcha. round 3 | 14:20 |
| mriedem | so reno still includes this note on master unreleased because the revert doesn't delete the file, so it still looks changed | 14:21 |
| mdbooth | sfinucan: It is pleasantly succinct and very readable though, isn't it? | 14:22 |
| *** slaweq has joined #openstack-nova | 14:22 | |
| bauzas | mriedem: if so, you just need to revert the file in the master tree | 14:23 |
| *** mtanino has joined #openstack-nova | 14:23 | |
| bauzas | and do a stable-only note | 14:23 |
| mriedem | bauzas: revert the file as in delete the file? | 14:23 |
| bauzas | FWIW, I'm just discovering new options with reno | 14:23 |
| bauzas | yup | 14:23 |
| mriedem | bauzas: yes deleting is an option | 14:23 |
| bauzas | in theory, notes are attached to a commit | 14:24 |
| bauzas | so, when a commit is changing a thing, you have the note | 14:24 |
| bauzas | the beauty of that is with backports | 14:24 |
| bauzas | because the note will follow the code | 14:24 |
| bauzas | but sometimes, you're doing cleanups in notes, and you're discovering you forgot something specific on a branch | 14:24 |
| bauzas | then a single commit only including a reno file is worth it for that stable branch | 14:25 |
| bauzas | merging on master necessarly means it appears on the unreleased page | 14:25 |
| *** alexchadin has quit IRC | 14:25 | |
| bauzas | since we scan all commits since the beginning | 14:25 |
| *** MVenesio has quit IRC | 14:25 | |
| bauzas | (as it's defined per the release management team rules) | 14:26 |
| *** hongbin has joined #openstack-nova | 14:28 | |
| *** tbachman_ has joined #openstack-nova | 14:28 | |
| *** tbachman has quit IRC | 14:29 | |
| *** tbachman has joined #openstack-nova | 14:30 | |
| *** seife_ has joined #openstack-nova | 14:32 | |
| *** tpatzig_ has joined #openstack-nova | 14:32 | |
| *** dgonzalez_ has joined #openstack-nova | 14:32 | |
| *** slaweq has quit IRC | 14:33 | |
| openstackgerrit | Felipe Monteiro proposed openstack/nova-specs master: Patrole (RBAC) Nova Gating https://review.openstack.org/471145 | 14:33 |
| *** eharney has quit IRC | 14:33 | |
| *** tbachman_ has quit IRC | 14:33 | |
| *** tpatzig_ has quit IRC | 14:34 | |
| *** seife_ has quit IRC | 14:34 | |
| *** dgonzalez_ has quit IRC | 14:34 | |
| *** pewp has quit IRC | 14:34 | |
| *** pewp has joined #openstack-nova | 14:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Prevent 2.32 release note from showing up in 9.0.0 release notes https://review.openstack.org/471383 | 14:36 |
| *** crushil has joined #openstack-nova | 14:36 | |
| mriedem | bauzas: sfinucan: sdague: ^ is where i'm at | 14:38 |
| *** READ10 has quit IRC | 14:38 | |
| *** baoli has quit IRC | 14:41 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Improve assertJsonEqual error reporting https://review.openstack.org/428199 | 14:43 |
| *** cfriesen_ has joined #openstack-nova | 14:44 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt https://review.openstack.org/471360 | 14:45 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 14:45 |
| sfinucan | mriedem, sdague: I _think_ that should be all good now | 14:46 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 14:47 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 14:47 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 14:47 |
| *** mdrabe has quit IRC | 14:47 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 14:47 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 14:47 |
| *** eharney has joined #openstack-nova | 14:48 | |
| mriedem | sfinucan: reviewed | 14:49 |
| *** itlinux_ has quit IRC | 14:50 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818 | 14:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: use plain routes list for os-virtual-interfaces https://review.openstack.org/462173 | 14:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Use plain routes list for hosts endpoint instead of stevedore https://review.openstack.org/459231 | 14:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: use plain routes list for hypervisor endpoint instead of stevedore https://review.openstack.org/459235 | 14:51 |
| openstackgerrit | Merged openstack/nova master: [placement] Add api-ref for resource classes https://review.openstack.org/467596 | 14:51 |
| openstackgerrit | Merged openstack/nova master: Remove mox from nova.tests.unit.objects.test_instance.py https://review.openstack.org/441784 | 14:52 |
| openstackgerrit | Merged openstack/nova master: Check instance existing before check in mapping https://review.openstack.org/433470 | 14:52 |
| openstackgerrit | Merged openstack/nova master: Remove mox from unit/cells/test_cells_messaging.py https://review.openstack.org/448021 | 14:53 |
| *** psachin has quit IRC | 14:54 | |
| openstackgerrit | Merged openstack/nova master: Stop using mox int unit/virt/xenapi/image/test_utils.py https://review.openstack.org/342626 | 14:54 |
| *** rcernin has quit IRC | 14:54 | |
| openstackgerrit | Merged openstack/nova master: Remove mox from cells/test_cells_rpc_driver.py https://review.openstack.org/447412 | 14:54 |
| *** yamamoto has quit IRC | 14:55 | |
| openstackgerrit | Merged openstack/nova master: Ensure image conversion flushes output data to disk https://review.openstack.org/443230 | 14:55 |
| *** tuanla_fujitsu has quit IRC | 14:55 | |
| *** marst has joined #openstack-nova | 14:55 | |
| openstackgerrit | Merged openstack/nova master: Remove mox from nova/tests/unit/consoleauth/test_consoleauth.py https://review.openstack.org/448386 | 14:56 |
| *** kwathore_ has quit IRC | 14:57 | |
| *** READ10 has joined #openstack-nova | 14:57 | |
| *** armax has quit IRC | 14:58 | |
| *** dimtruck is now known as zz_dimtruck | 14:58 | |
| *** salv-orlando has quit IRC | 14:59 | |
| *** salv-orlando has joined #openstack-nova | 15:00 | |
| *** armax has joined #openstack-nova | 15:00 | |
| *** salv-orlando has quit IRC | 15:04 | |
| *** gomarivera has joined #openstack-nova | 15:05 | |
| *** Oku_OS is now known as Oku_OS-away | 15:08 | |
| *** sree has joined #openstack-nova | 15:09 | |
| *** tbachman has quit IRC | 15:11 | |
| *** ericyoung has joined #openstack-nova | 15:12 | |
| *** mdrabe has joined #openstack-nova | 15:13 | |
| *** sree has quit IRC | 15:14 | |
| artom | mriedem, what's going on with https://review.openstack.org/#/c/471353? | 15:15 |
| artom | You appear to have backported some extra exception handling, and an unrelated fix for console log permissions? | 15:16 |
| mriedem | i backported it | 15:16 |
| mriedem | huh? | 15:16 |
| mriedem | oh i didn't see you had backported already | 15:16 |
| artom | Ah, no, sorry | 15:16 |
| mriedem | you were diffing right? | 15:16 |
| artom | Yeah | 15:17 |
| mriedem | i also rebased on top of lee's backport | 15:17 |
| mriedem | to resolve the exception conflict | 15:17 |
| mriedem | https://review.openstack.org/#/c/460296/ | 15:18 |
| artom | Doh, of course | 15:18 |
| artom | I didn't notice it was the top patch in a two-patch series | 15:18 |
| mriedem | will need to do similar if backporting to newton, which i assume we are | 15:19 |
| mriedem | since it was reported against rhosp mitaka | 15:19 |
| artom | Yeah, I posted the newton backport | 15:19 |
| mriedem | on top of lee's newton backport? | 15:19 |
| mriedem | because if not, you should redo it | 15:19 |
| artom | No | 15:19 |
| artom | https://review.openstack.org/#/c/471356/ | 15:19 |
| mriedem | let me -1 that quick for you | 15:20 |
| mriedem | :) | 15:20 |
| artom | You're welcome :) | 15:21 |
| *** penick has joined #openstack-nova | 15:22 | |
| *** gomarivera has quit IRC | 15:22 | |
| *** gomarivera has joined #openstack-nova | 15:22 | |
| *** sbezverk has joined #openstack-nova | 15:25 | |
| *** pksingh has joined #openstack-nova | 15:29 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999 | 15:30 |
| pksingh | alex_xu: Hello | 15:30 |
| artom | mriedem, done. | 15:32 |
| *** tidwellr has quit IRC | 15:33 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | 15:34 |
| *** tidwellr has joined #openstack-nova | 15:35 | |
| *** karimb has quit IRC | 15:36 | |
| pksingh | Hello Guys, i found nova has depricated this API, https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ips.py#L216 | 15:37 |
| pksingh | i.e. associate-floating-ip, then how floating IP is updated in database, by some periodic task? | 15:38 |
| artom | pksingh, I think you're supposed to use Neutron: https://developer.openstack.org/api-ref/networking/v2/index.html?expanded=update-floating-ip-detail#update-floating-ip | 15:39 |
| mriedem | pksingh: that api is not deprecated | 15:39 |
| mriedem | i'm not sure why it says https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/floating_ips.py#L205 | 15:39 |
| mriedem | hmm, api-ref says the same | 15:40 |
| *** roeyc has joined #openstack-nova | 15:40 | |
| *** roeyc has quit IRC | 15:40 | |
| pksingh | mriedem: i checked here https://developer.openstack.org/api-ref/compute/?expanded=add-associate-floating-ip-addfloatingip-action-deprecated-detail#add-associate-floating-ip-addfloatingip-action-deprecated | 15:40 |
| mriedem | yeah my mistake https://docs.openstack.org/developer/nova/api_microversion_history.html#id39 | 15:40 |
| pksingh | mriedem: so, now how you get Flating IPs in nova database? | 15:41 |
| mriedem | pksingh: the spec has the details https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/deprecate-multinic-proxy-api.html | 15:41 |
| mriedem | you use neutron apis | 15:41 |
| mriedem | neutron floatingip-create EXT_NET_ID VM_PORT_ID | 15:41 |
| *** roeyc1 has quit IRC | 15:42 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add PowerVM to nova support matrix https://review.openstack.org/470999 | 15:42 |
| pksingh | mriedem: thanks i got that point, to assign floating IP to instance , use neutron API | 15:43 |
| pksingh | mriedem: then how do you update this table https://developer.openstack.org/api-ref/compute/?expanded=add-associate-floating-ip-addfloatingip-action-deprecated-detail#add-associate-floating-ip-addfloatingip-action-deprecated | 15:43 |
| pksingh | mriedem: sorry wron link, https://github.com/openstack/nova/blob/065cd6a8d69c1ec862e5b402a3150131f35b2420/nova/db/sqlalchemy/models.py#L910 | 15:43 |
| mriedem | pksingh: you don't | 15:43 |
| mriedem | the nova db tables for network resources are only ever used for nova-network | 15:44 |
| mriedem | if you use neutron, regardless of microversion, those tables weren't used | 15:44 |
| mriedem | if you had some out of tree code that was relying on those tables getting updated, that out of tree code was incorrect | 15:44 |
| pksingh | mriedem: ok, great, so to get floating IPs, any how we need to use neutron APIs | 15:45 |
| mriedem | yes | 15:45 |
| mriedem | or use a microversion<2.44 | 15:45 |
| mriedem | but it's in your best long-term interest to use the networking API directly (neutron) | 15:45 |
| *** sbezverk has quit IRC | 15:46 | |
| mriedem | gibi: i'll apologize ahead of time for not making any progress on notification stuff while you were out for 2 weeks | 15:46 |
| *** jaosorior is now known as jaosorior_away | 15:47 | |
| *** markus_z has quit IRC | 15:47 | |
| *** sbezverk has joined #openstack-nova | 15:47 | |
| pksingh | mriedem: thanks a lot for your help, one more question | 15:49 |
| pksingh | mriedem: so nova will not show floating IP in server detail get request | 15:49 |
| mriedem | pksingh: floating ips will still show up in the server 'addresses' details | 15:50 |
| pksingh | mriedem: ok, from where do you get it? | 15:50 |
| mriedem | i'd like to know what you're doing with the nova database... | 15:50 |
| mriedem | but they come from the instance_info_cache table | 15:51 |
| mriedem | in the network_info column | 15:51 |
| pksingh | mriedem: i work in openstack container team | 15:51 |
| *** jamesdenton has joined #openstack-nova | 15:51 | |
| mriedem | why do you need to know the internal data model then? | 15:51 |
| pksingh | mriedem: we were also thinking to assign floating IP to containers | 15:51 |
| cdent | dansmith: I think you've said in the past that you'd prefer to have your attention drawn to mailing list posts instead of having to pay constant attention: ed started a thread on "complex allocations". your input would be most appreciated. I followed up a bit later with a lot of explicit questions. http://lists.openstack.org/pipermail/openstack-dev/2017-June/117913.html | 15:51 |
| mriedem | pksingh: you should probably not copy nova | 15:51 |
| pksingh | mriedem: but we came to know that nova has deprecated this api | 15:52 |
| *** ltomasbo is now known as ltomasbo|away | 15:52 | |
| * cdent takes a clarifying walk | 15:52 | |
| *** hamzy has quit IRC | 15:52 | |
| gibi | mriedem: hi! no problem, I made no progress either ;) | 15:52 |
| pksingh | mriedem: we are not copying the nova, just was curious how nova does that | 15:52 |
| mriedem | pksingh: as the spec says, the compute api is deprecated because (1) nova-network is deprecated and (2) it's a proxy to neutron | 15:52 |
| mriedem | pksingh: i don't mean "don't copy nova" because nova is great and shouldn't be copied, i'm saying don't copy it because of technical debt and cruft and just legacy stuff that you probably don't want to consider when building a new system | 15:53 |
| dansmith | cdent: yeah I saw, but haven't read it yet... | 15:53 |
| cdent | âś” | 15:54 |
| *** crushil has quit IRC | 15:54 | |
| pksingh | mriedem: yes, we decide not to develop any such api in zun, but we were thinking how to show Floating IPs assigned to containers | 15:54 |
| pksingh | mriedem: so we tried to understand the nova way, and decide if that suits us | 15:55 |
| mriedem | probably also don't want to copy server['addresses'] format from nvoa | 15:55 |
| mriedem | mordred would probably say the same | 15:55 |
| pksingh | mriedem: yes, we have different approach | 15:55 |
| *** penick has quit IRC | 15:56 | |
| *** rfolco has quit IRC | 15:56 | |
| mriedem | if there are users asking for the feature, i'd ask them how they'd like the information represented or what's most useful to them | 15:56 |
| mordred | yes please. don't copy the server['addresses'] | 15:56 |
| mordred | format | 15:56 |
| mriedem | i'd also caution against proxying too much to neutron to fulfill your rest api response | 15:56 |
| *** gyee has joined #openstack-nova | 15:58 | |
| *** mvk has quit IRC | 15:58 | |
| mordred | pksingh: also keep in mind that it's perfectly valid to use both floating ips and fixed ips ... "floating ip" is not an alias for "get me an ip that has external access" | 15:58 |
| *** lucasagomes is now known as lucas-afk | 15:58 | |
| mordred | pksingh: and that especially with ipv6, tying too much thinking around the specific construct of floating_ip when a NAT construct is not specifically desired makes things harder on users | 15:58 |
| pksingh | mordred: ok | 15:59 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert floating IP tests to ddt https://review.openstack.org/471360 | 16:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 16:00 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Convert additional disassociate tests to mock https://review.openstack.org/471408 | 16:00 |
| *** penick has joined #openstack-nova | 16:00 | |
| *** rfolco has joined #openstack-nova | 16:01 | |
| *** sridharg has quit IRC | 16:02 | |
| sfinucan | mriedem: ...and reworked ^ | 16:02 |
| * mriedem puts gloves on | 16:03 | |
| *** mamandle has quit IRC | 16:03 | |
| sdague | mriedem: https://review.openstack.org/#/c/468356/ you can un WIP this because the neutronclient changes are now merged | 16:04 |
| *** Apoorva has joined #openstack-nova | 16:06 | |
| mriedem | sdague: which neutronclient changes? | 16:07 |
| mriedem | for global request id? that wasn't the reason for the wip | 16:07 |
| sdague | mriedem: what was the reason? | 16:07 |
| mriedem | those went into neutronclient 6.3.0 which was blacklisted from g-r | 16:07 |
| sdague | it was unblacklisted 33 hours ago | 16:08 |
| mriedem | https://github.com/openstack/requirements/commit/df081c07c68ee2b529291c8fec6f5686ab97b002 | 16:08 |
| mriedem | i see | 16:08 |
| sdague | https://review.openstack.org/#/c/470047/ | 16:08 |
| mriedem | pretty sparse commit message | 16:08 |
| *** jamesdenton has quit IRC | 16:08 | |
| sdague | yeh, well, armax just responded on my status email and said they reverted it | 16:09 |
| mriedem | heh, change the api is the answer? | 16:09 |
| sdague | and if you look at the requirements in nova, we're now back at that minimum | 16:09 |
| armax | mriedem: sparse the commit message of the requirement revert? | 16:09 |
| mriedem | yeah, i see the depends-on | 16:10 |
| armax | mriedem: we like people to follow the crumbs | 16:10 |
| mriedem | so anyone with neutronclient 6.3.0 hitting a server that doesn't have that api workaround is still broken client-side | 16:10 |
| armax | like a good TV show that leaves you hanging | 16:10 |
| mriedem | which kind of sucks | 16:10 |
| armax | mriedem: the server change has been backported for all stable branches | 16:10 |
| armax | mriedem: your point was taken into account | 16:10 |
| mriedem | yeah i saw | 16:10 |
| armax | surely 6.3.0 might work prior to newton | 16:11 |
| armax | server-side I mean | 16:11 |
| sdague | mriedem: yeh, well, regardless, it's good enough for the nova patch to move forward | 16:11 |
| armax | but if you ain’t test it you ain’t say it for soor | 16:11 |
| armax | sure | 16:11 |
| *** dtantsur has joined #openstack-nova | 16:11 | |
| sdague | armax: this was mostly fighting against pecan magic? | 16:11 |
| mriedem | sdague: i know this isn't nova's problem | 16:12 |
| mriedem | i just feel bad for the puppies and kittens that now have to die | 16:12 |
| *** jamesdenton has joined #openstack-nova | 16:12 | |
| mriedem | more bad for the kittens because, you know, dogs... | 16:12 |
| *** dtp has joined #openstack-nova | 16:12 | |
| armax | sdague: I don’t believe so, the issue seemed to be latent and just surface because of the client change that dropped the .json suffix on the API URIs | 16:12 |
| *** yamahata has joined #openstack-nova | 16:15 | |
| *** CristinaPauna has quit IRC | 16:17 | |
| *** vks1 has quit IRC | 16:17 | |
| efried | mriedem FYI PowerVM support matrix adds are ready: https://review.openstack.org/#/c/470999/ | 16:20 |
| mriedem | oh i know | 16:20 |
| mriedem | it's starred | 16:20 |
| efried | mriedem On another topic, thinking it may be a good idea to blast the ML for the InstanceInfo change, since it would affect OOT drivers. | 16:20 |
| mriedem | go head | 16:21 |
| mriedem | *ahead | 16:21 |
| *** pksingh has quit IRC | 16:21 | |
| *** yamamoto has joined #openstack-nova | 16:26 | |
| *** jamielennox is now known as jamielennox|away | 16:26 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix and optimize external_events for multiple cells https://review.openstack.org/445142 | 16:26 |
| *** abalutoiu has quit IRC | 16:27 | |
| *** jamesdenton has quit IRC | 16:27 | |
| *** clenimar_ has joined #openstack-nova | 16:28 | |
| *** clenimar_ has quit IRC | 16:28 | |
| *** penick has quit IRC | 16:28 | |
| *** abalutoiu has joined #openstack-nova | 16:29 | |
| *** baoli has joined #openstack-nova | 16:29 | |
| *** yamamoto has quit IRC | 16:31 | |
| *** tesseract has quit IRC | 16:32 | |
| openstackgerrit | Merged openstack/nova master: Remove DeviceIsBusy exception https://review.openstack.org/465723 | 16:32 |
| *** gbarros has joined #openstack-nova | 16:32 | |
| *** mhenkel_ has joined #openstack-nova | 16:33 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Use config types to parse XML for root disk https://review.openstack.org/411941 | 16:33 |
| *** abalutoiu_ has joined #openstack-nova | 16:33 | |
| openstackgerrit | Merged openstack/nova master: Remove oslo.config deprecated parameter enforce_type https://review.openstack.org/462364 | 16:34 |
| openstackgerrit | Merged openstack/nova master: Modify incorrect comment on return_reservation_id https://review.openstack.org/471278 | 16:34 |
| *** pcaruana has quit IRC | 16:34 | |
| mhenkel_ | mriedem: would you mind to check https://blueprints.launchpad.net/openstack/?searchtext=opencontrail-nova-vif-plugin and approve? | 16:34 |
| openstackgerrit | Merged openstack/nova master: Remove incorrect comments in multiple_create https://review.openstack.org/471273 | 16:35 |
| *** armax has quit IRC | 16:35 | |
| *** dillaman has quit IRC | 16:36 | |
| *** lpetrut has quit IRC | 16:36 | |
| *** dillaman has joined #openstack-nova | 16:37 | |
| *** abalutoiu has quit IRC | 16:37 | |
| *** armax has joined #openstack-nova | 16:38 | |
| *** abalutoiu_ is now known as abalutoiu | 16:40 | |
| *** itlinux_ has joined #openstack-nova | 16:43 | |
| *** mhenkel_ has quit IRC | 16:49 | |
| *** jpena is now known as jpena|off | 16:49 | |
| *** efoley has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/nova master: Support tag instances when boot(2/4) https://review.openstack.org/459569 | 16:51 |
| *** eharney has quit IRC | 16:51 | |
| *** klkumar has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/nova master: [placement] adjust resource provider links by microversion https://review.openstack.org/468923 | 16:52 |
| openstackgerrit | Merged openstack/nova master: make route and controller in alpha sequence https://review.openstack.org/469380 | 16:52 |
| openstackgerrit | Merged openstack/nova master: ComputeDriver.get_info not limited to inst name https://review.openstack.org/471106 | 16:53 |
| openstackgerrit | Merged openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 16:54 |
| *** derekh has quit IRC | 16:54 | |
| openstackgerrit | Merged openstack/nova master: Remove usage of parameter enforce_type https://review.openstack.org/469745 | 16:54 |
| openstackgerrit | Merged openstack/nova master: Pass a list of instance UUIDs to scheduler https://review.openstack.org/465171 | 16:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Send out notifications when instance tags changed https://review.openstack.org/459923 | 16:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move notifications/objects/test_base.py https://review.openstack.org/471425 | 16:55 |
| *** baoli has quit IRC | 16:58 | |
| *** baoli has joined #openstack-nova | 16:59 | |
| *** felipemonteiro has joined #openstack-nova | 16:59 | |
| *** zz_dimtruck is now known as dimtruck | 17:02 | |
| *** nic has joined #openstack-nova | 17:03 | |
| *** jamielennox|away is now known as jamielennox | 17:05 | |
| *** tbachman has joined #openstack-nova | 17:05 | |
| *** dtantsur is now known as dtantsur|afk | 17:07 | |
| *** nic1 has joined #openstack-nova | 17:09 | |
| *** jlvillal is now known as jlvacation | 17:10 | |
| *** penick has joined #openstack-nova | 17:10 | |
| *** roeyc has joined #openstack-nova | 17:11 | |
| *** nic has quit IRC | 17:11 | |
| *** lucasxu has quit IRC | 17:11 | |
| *** mlavalle has joined #openstack-nova | 17:11 | |
| mlavalle | mriedem: so this https://bugs.launchpad.net/nova/+bug/1696125 | 17:11 |
| openstack | Launchpad bug 1696125 in OpenStack Compute (nova) "Detach interface failed - Unable to detach from guest transient domain (pike)" [High,Confirmed] | 17:11 |
| mlavalle | and https://bugs.launchpad.net/neutron/+bug/1696006 | 17:12 |
| openstack | Launchpad bug 1696006 in neutron "Libvirt fails to detach network interface with Linux bridge" [Critical,New] | 17:12 |
| mlavalle | are the same, right? | 17:12 |
| *** sambetts is now known as sambetts|afk | 17:16 | |
| mriedem | mlavalle: looks like it | 17:16 |
| *** mingyu has quit IRC | 17:16 | |
| mriedem | mlavalle: the e-r query is written against bug 1696125 | 17:17 |
| openstack | bug 1696125 in OpenStack Compute (nova) "Detach interface failed - Unable to detach from guest transient domain (pike)" [High,Confirmed] https://launchpad.net/bugs/1696125 | 17:17 |
| mlavalle | mriedem: ok, cool. thanks for the follow up. will let you deal with it | 17:17 |
| *** hamzy has joined #openstack-nova | 17:19 | |
| *** eharney has joined #openstack-nova | 17:19 | |
| *** mlavalle has quit IRC | 17:20 | |
| *** baoli has quit IRC | 17:21 | |
| *** baoli has joined #openstack-nova | 17:22 | |
| *** mingyu has joined #openstack-nova | 17:22 | |
| mriedem | dtp: remember the instance.exists periodic hammering thing mentioned at the summit? | 17:23 |
| mriedem | did anyone ever report a bug on that with details? | 17:23 |
| dtp | i'll check w/ kris | 17:23 |
| *** egonzalez has joined #openstack-nova | 17:24 | |
| *** mhenkel_ has joined #openstack-nova | 17:25 | |
| *** sree has joined #openstack-nova | 17:25 | |
| efried | What's instance.exists? | 17:27 |
| *** mingyu has quit IRC | 17:27 | |
| efried | Tell me there's a nova API where I can send a UUID and ask if the record exists, without it actually pulling the whole record and packaging it up and sending it over the wire... | 17:27 |
| *** mhenkel__ has joined #openstack-nova | 17:28 | |
| efried | I looked and couldn't find one. | 17:28 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6121 | 17:28 |
| dtp | mriedem - he never did | 17:28 |
| *** penick has quit IRC | 17:28 | |
| efried | okay, yeah, that's not it at all :) | 17:29 |
| *** sree has quit IRC | 17:29 | |
| *** mingyu has joined #openstack-nova | 17:30 | |
| mriedem | dtp: ok going to need some details on the issue | 17:30 |
| mriedem | i see that by default the audit period is a month | 17:30 |
| mriedem | so the task is going to run every minute by default, | 17:30 |
| mriedem | and if configured to actually do an audit, it gets the instances that were active on that host within the last month | 17:30 |
| mriedem | and generates this task log thing | 17:30 |
| *** mhenkel_ has quit IRC | 17:30 | |
| mriedem | and the instance.exists notification for each instance in that window | 17:31 |
| mriedem | would need to know what your audit period is | 17:31 |
| dtp | k sec | 17:31 |
| harlowja | dtp whats up | 17:32 |
| harlowja | kris is typically in #openstack-operators if u want to find him | 17:32 |
| harlowja | under klindgren | 17:32 |
| dtp | harlowja being a middleman, how you doin | 17:32 |
| harlowja | okies, trying not to be middleman | 17:32 |
| *** eharney has quit IRC | 17:32 | |
| harlowja | lol | 17:32 |
| harlowja | :-P | 17:32 |
| harlowja | proxyharlow | 17:33 |
| harlowja | lol | 17:33 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Trim the fat from InstanceInfo https://review.openstack.org/471146 | 17:33 |
| *** mhenkel__ has quit IRC | 17:33 | |
| gibi | mriedem: I feel that if there is already a task created for the given audit period then the periodic task just returns | 17:33 |
| gibi | mriedem: here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6128 | 17:34 |
| dtp | mriedem what's the config key for audit period? | 17:35 |
| gibi | dtp: instance_usage_audit_period | 17:35 |
| mriedem | instance_usage_audit_period | 17:35 |
| dtp | ok, we do not set that value | 17:36 |
| *** itlinux_ has quit IRC | 17:36 | |
| *** itlinux_ has joined #openstack-nova | 17:36 | |
| mriedem | ok so by default it's a month | 17:37 |
| gibi | nova logs an info trace at every audit https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6138 | 17:37 |
| dtp | mriedem wait i was wrong - instance_usage_audit_period=hour | 17:37 |
| mriedem | dtp: ok so do you guys see the instance.exists stuff run every minute, like is it sending notifications every minute, or every hour? | 17:38 |
| mriedem | the periodic task will run by default every minute, | 17:38 |
| mriedem | but as noted above, it should exit early if there is already a task log for the same audit period (the last hour) | 17:39 |
| mriedem | but you would expect to get one every hour | 17:39 |
| mriedem | at least | 17:39 |
| *** xyang1 has joined #openstack-nova | 17:39 | |
| gibi | yeah. If you happen to see 'Running instance usage audit for' log every minute then that is a bug. But if it is only every hour then that is the expected behavior | 17:40 |
| *** nicolasbock has quit IRC | 17:41 | |
| dtp | the anecdote i've heard is we're seeing an instance.exist for every VM on the hour | 17:41 |
| *** mingyu has quit IRC | 17:41 | |
| *** mingyu has joined #openstack-nova | 17:42 | |
| *** nicolasbock has joined #openstack-nova | 17:42 | |
| mriedem | thats the design | 17:43 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6157 | 17:43 |
| mriedem | dtp: if you configure all of your computes to do this every hour, | 17:44 |
| mriedem | then every hour, it's going to get the list of instances that were/are active on that host within the last hour and send an instance.exists notification for each of them | 17:44 |
| mriedem | for all computes | 17:44 |
| mriedem | how many computes and how many live instances do you guys have? | 17:45 |
| dtp | i think the issue was they're all sending them at the same time, clogging rabbit | 17:45 |
| dtp | checking those #s for you | 17:46 |
| *** mingyu has quit IRC | 17:46 | |
| gibi | there is a fuzzy delay config option 'periodic_fuzzy_delay' for periodic tasks to avoid starting them at the same time | 17:46 |
| dtp | 1k computes, 13k instance in our largest control plane | 17:47 |
| dtp | thanks gibi, passing that along | 17:48 |
| gibi | I have to leave for today. I will read the back tomorrow. | 17:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add get_minimum_version_all_cells() helper for service https://review.openstack.org/471431 | 17:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix device metadata service version check for multiple cells https://review.openstack.org/471432 | 17:50 |
| dansmith | mriedem: artom ^ | 17:50 |
| dansmith | mriedem: artom I was also thinking, can we remove that service version check now? or soon? I think that was ocata stuff, right? | 17:51 |
| artom | dansmith, newton | 17:52 |
| *** slaweq has joined #openstack-nova | 17:52 | |
| *** jobewan has joined #openstack-nova | 17:52 | |
| dansmith | oh okay | 17:52 |
| artom | Removing it would assume every deployment supports tagging, no? | 17:53 |
| artom | That seems... imprudent | 17:53 |
| dansmith | the check is to make sure that the compute nodes are upgraded to .. newton | 17:53 |
| dansmith | in ocata, you needn't worry about mitaka computes, and definitely not in pike | 17:54 |
| *** luisnho223_ has quit IRC | 17:55 | |
| artom | Ah, I see - compute are at most N-1 behind controller | 17:55 |
| mriedem | dtp: your wussy rabbit can't handle 13K notifications every hour :) | 17:56 |
| mriedem | on the hour | 17:56 |
| mriedem | dtp: well it at least sounds like you're not hitting it every minute or something | 17:56 |
| mriedem | on the hour is kind of by design based on how you've configured things | 17:56 |
| dtp | hehe | 17:57 |
| mriedem | we could stagger those notifications to be 1 instance.exists per periodic (every minute) until the queue is drained, and then get a fresh set, | 17:57 |
| dansmith | dtp: you gonna take that? | 17:57 |
| *** nic1 is now known as nic | 17:57 | |
| dansmith | dtp: I think a dance off might be in order | 17:57 |
| mriedem | however, waiting to dequeue might mess up the actual point of the audit period | 17:57 |
| mriedem | like if the audit starts at 1:59 or something, and you got 10 instances, and it takes 10 minutes to dequeue them all | 17:58 |
| mriedem | seems like numbers could get fudgy a bi | 17:58 |
| mriedem | *bit | 17:58 |
| *** baoli has quit IRC | 17:58 | |
| mriedem | or, | 17:58 |
| mriedem | >60 instances, | 17:58 |
| mriedem | with audit period = hour | 17:58 |
| dtp | dansmith twinkie eating contest? | 17:58 |
| *** crushil has joined #openstack-nova | 17:58 | |
| dansmith | nice | 17:58 |
| * mriedem moves outside | 17:58 | |
| dansmith | I will officiate | 17:59 |
| *** gbarros has quit IRC | 17:59 | |
| *** lucasxu has joined #openstack-nova | 17:59 | |
| dtp | ok. i've passed along that randomizer setting. we aren't even sure why we're running it every hour instead of the default, so checking on that too | 17:59 |
| dtp | thanks for the infos | 18:00 |
| *** mriedem1 has joined #openstack-nova | 18:00 | |
| mriedem1 | i'm not sure if this working outside in nature is a good idea | 18:00 |
| openstackgerrit | Merged openstack/nova master: XenAPI: Deprecate nicira-iface-id for XenServer VIF https://review.openstack.org/463246 | 18:02 |
| *** mriedem1 has quit IRC | 18:03 | |
| *** mriedem has quit IRC | 18:03 | |
| *** mriedem1 has joined #openstack-nova | 18:03 | |
| *** mriedem1 is now known as mriedem | 18:03 | |
| openstackgerrit | Merged openstack/nova master: Remove mox from nova/tests/unit/virt/xenapi/test_xenapi.py[1] https://review.openstack.org/438825 | 18:06 |
| oomichi | gmann: I could not understand why we need different change on https://review.openstack.org/#/c/413876/39/nova/tests/unit/compute/test_compute_mgr.py in this patch | 18:06 |
| openstackgerrit | Merged openstack/nova master: Update server create networks API reference description for tags https://review.openstack.org/470004 | 18:06 |
| dtp | gibi mriedem - we have some periodics we don't want to be too fuzzy. does the fuzzy setting define the range after the start time that it might run? | 18:07 |
| *** fragatina has quit IRC | 18:08 | |
| *** baoli has joined #openstack-nova | 18:08 | |
| *** slaweq has quit IRC | 18:11 | |
| mriedem | dtp: don't really know, it's not documented https://docs.openstack.org/developer/oslo.service/ | 18:11 |
| *** gjayavelu has joined #openstack-nova | 18:11 | |
| *** penick has joined #openstack-nova | 18:13 | |
| *** slaweq has joined #openstack-nova | 18:14 | |
| *** itlinux_ has quit IRC | 18:16 | |
| *** lyarwood_ is now known as lyarwood | 18:18 | |
| dtp | k. we're just gonna run it less frequently and see how that goes | 18:19 |
| *** slaweq has quit IRC | 18:19 | |
| *** itlinux has joined #openstack-nova | 18:22 | |
| *** CristinaPauna has joined #openstack-nova | 18:22 | |
| *** baoli has quit IRC | 18:23 | |
| *** roeyc has quit IRC | 18:24 | |
| *** baoli has joined #openstack-nova | 18:24 | |
| *** kaisers has quit IRC | 18:27 | |
| *** yamamoto has joined #openstack-nova | 18:29 | |
| *** clenimar has quit IRC | 18:29 | |
| *** cdent has quit IRC | 18:31 | |
| mriedem | dansmith: comments on your bottom change | 18:33 |
| mriedem | for the service min version check | 18:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Check if domain is persistent before detaching devices https://review.openstack.org/441204 | 18:33 |
| dansmith | mriedem: sorry the sun out here is too bright to read your comments | 18:34 |
| *** roeyc has joined #openstack-nova | 18:34 | |
| *** yamamoto has quit IRC | 18:34 | |
| *** gszasz has quit IRC | 18:36 | |
| mriedem | the sound of large grading equipment can also be distracting | 18:37 |
| sdague | mriedem: rebuild get a lot worse? | 18:37 |
| sdague | https://review.openstack.org/#/c/471082/ | 18:37 |
| mriedem | sdague: it failed in my novaclient change too | 18:37 |
| mriedem | timed out | 18:37 |
| mriedem | i don't know what's going on there | 18:37 |
| sdague | yeh, that's double fail on a reno change | 18:37 |
| mriedem | i don't see any recent changes to nova or novaclient related to rebuild | 18:40 |
| *** itlinux has quit IRC | 18:42 | |
| mriedem | it also doesn't appear to be new | 18:43 |
| mriedem | i think i have looked into that _poll_for_status method in novaclient before and it seems a bit wonky | 18:43 |
| sdague | http://logs.openstack.org/82/471082/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/68a19f4/logs/screen-n-cpu.txt.gz#_Jun_06_17_35_44_082400 | 18:45 |
| sdague | yeh, I don't know, shouldn't we get an instance started emit? | 18:45 |
| mriedem | dansmith: there is a child wailing now - this outdoors thing isn't working | 18:45 |
| dansmith | mriedem: don't they have guns in minnesota? | 18:46 |
| *** gbarros has joined #openstack-nova | 18:47 | |
| *** mordred has quit IRC | 18:47 | |
| mriedem | they do | 18:47 |
| mriedem | and hotdish! | 18:48 |
| *** mordred has joined #openstack-nova | 18:48 | |
| *** roeyc has quit IRC | 18:48 | |
| *** tbachman has quit IRC | 18:48 | |
| mriedem | sdague: so it's req-5b3770c1-d332-4875-8933-97de8a9890b4 right? | 18:49 |
| sdague | I actually think it's req-f0668b70-fac7-4dae-b3b3-afba6c36ec77 | 18:50 |
| sdague | one sec, I just closed all my tabs | 18:51 |
| sdague | yeh req-f0668b70-fac7-4dae-b3b3-afba6c36ec77 is the rebuild | 18:51 |
| sdague | http://logs.openstack.org/82/471082/1/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/68a19f4/logs/screen-n-api.txt.gz#_Jun_06_17_35_38_992164 | 18:51 |
| mriedem | sdague: http://logs.openstack.org/82/471082/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/796acb7/logs/screen-n-cpu.txt.gz#_Jun_06_13_14_02_210879 | 18:52 |
| *** sbezverk has quit IRC | 18:52 | |
| mriedem | i'm looking at the gate queue job | 18:52 |
| mriedem | you're looking at check | 18:52 |
| mriedem | :) | 18:52 |
| *** mnestratov|3 has quit IRC | 18:52 | |
| openstackgerrit | Merged openstack/nova master: Send request_id on glance calls https://review.openstack.org/467242 | 18:53 |
| sdague | mriedem: ah, sure :) | 18:53 |
| *** kaisers has joined #openstack-nova | 18:53 | |
| *** eharney has joined #openstack-nova | 18:54 | |
| *** thorst_afk has quit IRC | 18:54 | |
| openstackgerrit | Merged openstack/nova master: Sync os-traits to Traits database table https://review.openstack.org/469578 | 18:55 |
| mriedem | this is a weird message | 18:55 |
| mriedem | Jun 06 13:14:02.594292 ubuntu-xenial-infracloud-chocolate-9158824 nova-compute[20994]: DEBUG nova.virt.libvirt.driver [None req-5b3770c1-d332-4875-8933-97de8a9890b4 admin admin] [instance: 573258a4-9416-4e13-a765-7c90683f3526] Shutting down instance from state 1: NotImplementedError {{(pid=20994) _clean_shutdown /opt/stack/new/nova/nova/virt/libvirt/driver.py:2524}} | 18:55 |
| openstackgerrit | Merged openstack/nova master: Use VIR_DOMAIN_BLOCK_REBASE_COPY_DEV when rebasing https://review.openstack.org/465205 | 18:55 |
| *** dtp has quit IRC | 18:55 | |
| *** pcaruana has joined #openstack-nova | 18:56 | |
| *** thorst_afk has joined #openstack-nova | 18:56 | |
| artom | mriedem, looks like something doesn't implement get_power_states | 18:57 |
| mriedem | or it's something in a repr | 18:58 |
| artom | That... would make more sense | 18:58 |
| mriedem | it should be a InstancePowerState object | 18:58 |
| mriedem | weird, ": NotImplementedError" shows up a lot in the logs | 19:00 |
| mriedem | http://logs.openstack.org/82/471082/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/796acb7/logs/screen-n-cpu.txt.gz | 19:00 |
| *** thorst_afk has quit IRC | 19:00 | |
| mriedem | maybe it's something in oslo.log? | 19:01 |
| mriedem | sdague: wasn't there something done with oslo.log recently to put extra metadata in messages or something? | 19:01 |
| mriedem | this error summary stuff | 19:02 |
| mriedem | "If the log format string - does not include ``%(error_summary)s`` it will be appended to the end + does not include ``%(error_summary)s`` the error summary will be - of the line automatically, only if there is an error. + appended to the end of the line automatically, only if there is an | 19:03 |
| mriedem | + error." | 19:03 |
| *** salv-orlando has joined #openstack-nova | 19:03 | |
| sdague | yeh, the NotImplementedError stuff is bleeding through there, that's going to need to be a dhellman ask | 19:03 |
| sdague | that was only supposed to happen if we were in an active exception block | 19:04 |
| mriedem | i'm pinging him in -oslo | 19:04 |
| sdague | I think that's irrelevant and just bleed over because that was the last exception thrown | 19:04 |
| clarkb | not to interrupt but looks like the nova python35 job is failing a lot and reseting things in the gate? | 19:04 |
| sdague | clarkb: not really | 19:05 |
| clarkb | http://logs.openstack.org/71/465171/11/gate/gate-nova-python35/2fcc59a/console.html#_2017-06-06_14_57_13_022429 and http://logs.openstack.org/89/418489/10/gate/gate-nova-tox-functional-py35-ubuntu-xenial/51b5776/console.html#_2017-06-06_13_35_33_574055 appear to be two different classes of error | 19:05 |
| sdague | clarkb: there was one bad patch, and because the gate was taking a long time and other things were running a lot, it managed to run and fail a lot in the gate | 19:05 |
| clarkb | gotcha | 19:05 |
| sdague | this is the params one? | 19:05 |
| sdague | man, the log server seems slow today | 19:06 |
| mriedem | i blame systemd | 19:06 |
| mriedem | er journald :) | 19:06 |
| mriedem | somethingd | 19:06 |
| sdague | yeh, the wrong params to __init__ was just a patch with out of date tests | 19:06 |
| clarkb | sdague: mriedem its less that the log server is slow and that its a 12MB html file | 19:06 |
| clarkb | sdague: the second one is a different error fwiw | 19:06 |
| mriedem | i kid | 19:06 |
| * dansmith looks up the ki daemon | 19:07 | |
| *** tbachman has joined #openstack-nova | 19:07 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:07 | |
| sdague | clarkb: it's actually the same thing | 19:07 |
| sdague | https://review.openstack.org/#/c/418489/ needs work | 19:08 |
| sdague | stuff was probably behind it and failing | 19:08 |
| mriedem | here is the oslo.log bug https://bugs.launchpad.net/oslo.log/+bug/1696213 | 19:09 |
| openstack | Launchpad bug 1696213 in oslo.log "error summary formatter is putting ": NotImplementedError" in non-error log messages" [Undecided,New] | 19:09 |
| mriedem | back to this rebuild failure | 19:10 |
| *** felipemonteiro has quit IRC | 19:10 | |
| *** gouthamr has quit IRC | 19:12 | |
| *** Qiming has quit IRC | 19:13 | |
| *** thorst_afk has joined #openstack-nova | 19:14 | |
| mriedem | sdague: i see the problem | 19:14 |
| mriedem | http://logs.openstack.org/82/471082/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/796acb7/logs/screen-n-cpu.txt.gz#_Jun_06_13_19_17_300332 | 19:14 |
| mriedem | vif plugging callback timeout | 19:14 |
| mriedem | during rebuild | 19:14 |
| mriedem | so the instance never goes active | 19:14 |
| mriedem | which someone from heat reported a rebuild bug about vif plug timeouts recently | 19:15 |
| *** gbarros has quit IRC | 19:15 | |
| *** thorst_afk has quit IRC | 19:15 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1694371 | 19:16 |
| openstack | Launchpad bug 1694371 in OpenStack Compute (nova) "test_stack_snapshot_restore failing intermittently" [Undecided,New] | 19:16 |
| *** thorst_afk has joined #openstack-nova | 19:16 | |
| mriedem | kevinbenton: we probably need some help from the neutron side here https://bugs.launchpad.net/nova/+bug/1694371 | 19:17 |
| openstack | Launchpad bug 1694371 in OpenStack Compute (nova) "Timeout while waiting for network-vif-plugged event during server rebuild" [Undecided,New] | 19:17 |
| *** Qiming has joined #openstack-nova | 19:17 | |
| mriedem | kevinbenton: seeing rebuild tests timeout waiting for vif plugged events from neutron | 19:18 |
| mriedem | dansmith: does it make sense to do anything in nova.objects.Instance.mutated_migration_context if there is no self.migration_context? | 19:22 |
| mriedem | because we're lazy-loading 3 fields in there just to do nothing | 19:22 |
| dansmith | um | 19:23 |
| mriedem | i'm seeing this during a rebuild | 19:23 |
| mriedem | where we're not migrating | 19:23 |
| mriedem | but the rebuild code also handles evacuate, which is the migration | 19:23 |
| mriedem | but it loos like we end up lazy-loading numa_topology, pci_requests and pci_devices one after another just to ultimately do nothing | 19:23 |
| dansmith | yeah I guess not.. you see that debug log warning that there's no context too I guess? | 19:24 |
| mriedem | http://logs.openstack.org/82/471082/1/gate/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/796acb7/logs/screen-n-cpu.txt.gz#_Jun_06_13_14_02_547424 | 19:25 |
| mriedem | yup | 19:25 |
| cfriesen_ | mriedem: if you're not actually migrating I think it'd be safe to bypass the mutated_migration_context | 19:25 |
| mriedem | see the next 4 there | 19:25 |
| mriedem | it lazy-loads 3 fields, and then says "you're dumb" | 19:25 |
| mriedem | and i believe each lazy-load on those fields is going to be a full pull from the db via conductor... | 19:26 |
| * mriedem opens bug | 19:26 | |
| kevinbenton | mriedem: in a rebuild what does nova tell neutron? | 19:27 |
| kevinbenton | mriedem: because the host_id doesn't change, right? | 19:28 |
| cfriesen_ | mriedem: can a rebuild end up picking a new host? | 19:28 |
| cfriesen_ | (when we're not doing an evacuate, I mean) | 19:29 |
| *** harlowja has quit IRC | 19:29 | |
| mriedem | cfriesen_: kevinbenton: no, same host | 19:30 |
| cfriesen_ | mriedem:, right, we specify the host in that case | 19:30 |
| mriedem | kevinbenton: so my theory is neutron is not sending the event maybe because the host doesn't change | 19:30 |
| mriedem | but i'd think that would be 100% fail | 19:30 |
| mriedem | so i wonder if we're hitting a race where neutron is queueing notifications and thinks it already told nova something about a given port | 19:31 |
| *** yamamoto has joined #openstack-nova | 19:31 | |
| kevinbenton | mriedem: we did have a recent change on the neutron side that reduced some port status changes when they weren't needed | 19:32 |
| kevinbenton | mriedem: so it's possible that there is a race that was being covered by side effects from that before | 19:32 |
| *** eharney has quit IRC | 19:33 | |
| kevinbenton | mriedem: do you have a set of logs from one of these failures? | 19:33 |
| *** pcaruana has quit IRC | 19:33 | |
| mriedem | kevinbenton: yeah that bug linked above has a log dive from me | 19:35 |
| mriedem | showing where things happen on the nova side | 19:35 |
| *** yamamoto has quit IRC | 19:36 | |
| efried | sdague Can you please help me understand whether/how global_request_id affordance is needed in https://review.openstack.org/#/c/412634/28/nova/image/glance.py ? | 19:37 |
| efried | I'm kinda hoping it's implicitly part of the ksa session... | 19:38 |
| efried | ...or auth | 19:38 |
| efried | ...because of the context | 19:40 |
| *** mdrabe has quit IRC | 19:46 | |
| *** jwnx has joined #openstack-nova | 19:46 | |
| *** eharney has joined #openstack-nova | 19:47 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | 19:49 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | 19:49 |
| *** harlowja has joined #openstack-nova | 19:50 | |
| sdague | efried: it's not implicit | 19:51 |
| sdague | efried: it's a single parameter to pass | 19:52 |
| efried | sdague So I need to add the header explicitly? | 19:52 |
| sdague | one sec | 19:52 |
| sdague | efried: https://review.openstack.org/#/c/467242/6/nova/image/glance.py | 19:52 |
| efried | sdague Right, I'm rebasing against that. | 19:53 |
| efried | sdague But in the new version, we don't do any of that stuff. | 19:53 |
| sdague | efried: yeh, so just add global_request_id=context.global_id | 19:53 |
| sdague | to the constructor args for glanceclient.Client | 19:53 |
| efried | sdague Okay, will do. Thanks. | 19:54 |
| *** sbezverk has joined #openstack-nova | 19:54 | |
| sdague | like this - https://github.com/openstack/nova/blob/2f712b84ecf8d149833dea17d2fa4b78ae9442c9/nova/volume/cinder.py#L105 | 19:54 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: log vm and task state when vif plugging times out https://review.openstack.org/471459 | 19:54 |
| sdague | efried: cool | 19:54 |
| efried | sdague But makes a guy wonder why the client couldn't grab it from the auth, which is made from the context, which contains that request ID... | 19:55 |
| *** felipemonteiro has joined #openstack-nova | 19:55 | |
| *** gbarros has joined #openstack-nova | 19:56 | |
| sdague | it could be more magical | 19:56 |
| *** gbarros has quit IRC | 19:56 | |
| sdague | but, being explicit here seems fine | 19:56 |
| *** mdrabe has joined #openstack-nova | 19:56 | |
| efried | ight | 19:57 |
| sdague | given that it was a one line change for each service linkage, being explicit didn't seem so bad :) | 19:57 |
| efried | sdague But a zero-line change would have been cooler ;-) | 19:58 |
| *** cdent has joined #openstack-nova | 19:58 | |
| *** felipemonteiro_ has quit IRC | 19:59 | |
| mriedem | 0-line change means non-0 amount of debugging in 2 years time | 19:59 |
| mriedem | b/c of magic | 19:59 |
| *** tidwellr has quit IRC | 20:00 | |
| sdague | mriedem: ++ | 20:00 |
| efried | sdague How/where does global_id get set on a RequestContext? | 20:00 |
| sdague | it's a property which is either global_request_id or request_id | 20:00 |
| sdague | so it prefers global_request_id if that was sent in | 20:01 |
| sdague | otherwise it uses the service request id | 20:01 |
| sdague | request_id is generated as a uuid | 20:01 |
| sdague | ok, TC / Pg thing, so won't be active here | 20:01 |
| efried | sdague Thanks, I found it in the superclass (duh) | 20:03 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Adopts keystoneauth with glance client. https://review.openstack.org/412634 | 20:04 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Add service_token for nova-glance interaction https://review.openstack.org/419662 | 20:05 |
| *** lyan has quit IRC | 20:05 | |
| *** lyan has joined #openstack-nova | 20:06 | |
| *** salv-orl_ has joined #openstack-nova | 20:06 | |
| *** armax has quit IRC | 20:07 | |
| *** priteau has quit IRC | 20:07 | |
| *** tidwellr has joined #openstack-nova | 20:07 | |
| *** salv-orlando has quit IRC | 20:08 | |
| *** dtp has joined #openstack-nova | 20:09 | |
| melwitt | mriedem: forgot to mention last week I wrote a functional test for that local delete allocations thing. looks like they do indeed get cleaned up upon init_host, but they would stay around if the compute host never returned https://review.openstack.org/#/c/470578 | 20:11 |
| *** peter-hamilton has quit IRC | 20:12 | |
| *** fragatina has joined #openstack-nova | 20:15 | |
| *** fragatina has quit IRC | 20:20 | |
| *** fragatina has joined #openstack-nova | 20:22 | |
| openstackgerrit | melanie witt proposed openstack/nova master: placement project_id, user_id in PUT /allocations https://review.openstack.org/469634 | 20:23 |
| openstackgerrit | melanie witt proposed openstack/nova master: placement: Add GET /usages to placement API https://review.openstack.org/469635 | 20:23 |
| *** dtantsur|afk has quit IRC | 20:23 | |
| *** dtantsur has joined #openstack-nova | 20:26 | |
| bauzas | mriedem: looks like https://bugs.launchpad.net/nova/+bug/1696082 is duplicate of https://bugs.launchpad.net/nova/+bug/1657774 ? | 20:28 |
| openstack | Launchpad bug 1696082 in OpenStack Compute (nova) "'nova.exception.NeutronAdminCredentialConfigurationInvalid" [Undecided,New] | 20:28 |
| openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 20:28 |
| *** pchavva has quit IRC | 20:29 | |
| mriedem | shrug | 20:30 |
| mriedem | TC meeting | 20:30 |
| mriedem | melwitt: ok | 20:30 |
| mriedem | melwitt: how's that bottom change on the quotas series going? | 20:30 |
| *** shaner has quit IRC | 20:31 | |
| artom | Speaking of bottom changes | 20:31 |
| artom | mriedem, can I get you to put tagged attach on your review list again? | 20:31 |
| bauzas | mriedem: dang. ok | 20:31 |
| *** Apoorva_ has joined #openstack-nova | 20:32 | |
| melwitt | mriedem: it's next on my list today, I just updated the placement stuff, working on the quotas thing now. when we talked at the summit, dansmith was saying it would be good to gate the "recheck quota" logic on a config option, in case someone doesn't mind quota going over during races | 20:32 |
| *** shaner has joined #openstack-nova | 20:32 | |
| *** yamamoto has joined #openstack-nova | 20:32 | |
| melwitt | since a quota recheck is a re-count of resources, not a lightweight thing | 20:32 |
| melwitt | so I was thinking of adding a config option also in the bottom change | 20:33 |
| mriedem | artom: it already is, with 100 other things | 20:34 |
| artom | mriedem, fair enough, thanks :) | 20:34 |
| *** jamielennox has quit IRC | 20:34 | |
| *** Apoorva has quit IRC | 20:34 | |
| mriedem | bauzas: shrug == idk and don't have time to look, but maybe | 20:35 |
| bauzas | mriedem: no worries, I see you active on the PG thingy | 20:35 |
| artom | bauzas, don't think so - first is error 500, second one is error 400 | 20:35 |
| *** nicolasbock has quit IRC | 20:35 | |
| * bauzas would like to propose MongoDB to the TC | 20:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: Check if domain is persistent before detaching devices https://review.openstack.org/441204 | 20:36 |
| * bauzas caughes | 20:36 | |
| artom | bauzas, but the message definitely is the same... | 20:36 |
| artom | I feel like we should just close the first one with not enough info | 20:36 |
| dansmith | melwitt: I think that's a good plan yeah | 20:36 |
| bauzas | artom: yup, looks like both are related, just the fact that the API for nova list transforms it into 400 | 20:36 |
| artom | You can't just paste a stack trace into launchpad and call it a bug report | 20:37 |
| *** yamamoto has quit IRC | 20:37 | |
| mriedem | maybe just review https://review.openstack.org/#/c/422696/ | 20:37 |
| melwitt | cool, thanks dansmith | 20:38 |
| *** jamielennox has joined #openstack-nova | 20:38 | |
| mriedem | melwitt: i thought your bottom change was sabout instance groups quota? | 20:38 |
| mriedem | wouldn't a config patch be separate? | 20:38 |
| mriedem | oh wait, | 20:38 |
| mriedem | the recheck quota thing in that patch, now i get it | 20:39 |
| mriedem | and default that to what? | 20:39 |
| mriedem | recheck by default? | 20:39 |
| melwitt | yeah. I could do a separate patch for the config option but it would be by itself with nothing using it | 20:39 |
| mriedem | just mix them together | 20:39 |
| melwitt | yeah I was thinking recheck by default | 20:39 |
| melwitt | ack | 20:39 |
| mriedem | probably with a reno for the config option | 20:39 |
| melwitt | yeah. thanks for reminding bc I probably would have forgotten the reno | 20:40 |
| *** gouthamr has joined #openstack-nova | 20:40 | |
| *** ckopper has joined #openstack-nova | 20:40 | |
| ckopper | danpawlik: I am finally back from my conference and was curious if you (or anyone else) had thoughts on what to do about the `img_hide_hypervisor_id` issue ( https://review.openstack.org/459753 ). I also hope I did not miss any discussion about this | 20:44 |
| ckopper | (I can certainly wait to discuss this some other time more convenient for you) | 20:45 |
| *** smatzek has quit IRC | 20:46 | |
| *** crushil has quit IRC | 20:47 | |
| *** jmlowe has quit IRC | 20:47 | |
| *** armax has joined #openstack-nova | 20:48 | |
| *** tbachman has quit IRC | 20:48 | |
| *** tidwellr has quit IRC | 20:51 | |
| *** jwnx has quit IRC | 20:53 | |
| *** tidwellr has joined #openstack-nova | 20:55 | |
| *** egonzalez has quit IRC | 20:56 | |
| *** egonzalez has joined #openstack-nova | 20:56 | |
| *** cleong has quit IRC | 20:56 | |
| bauzas | mriedem: heh, fun someone is asking for something you began to write https://bugs.launchpad.net/nova/+bug/1696043 | 21:03 |
| openstack | Launchpad bug 1696043 in OpenStack Compute (nova) "os-hypervisors api can not list the uuid of the specified VMM" [Undecided,Invalid] - Assigned to mylq (leibnizlq) | 21:03 |
| mriedem | melwitt: so on https://review.openstack.org/#/c/470578 i think we both probably think that if you delete the instance, if the compute is down, the local delete flow in the api should cleanup the allocations so they don't count against your quota? | 21:03 |
| bauzas | mriedem: be glad to feel yourself filling a gap :p | 21:03 |
| mriedem | heh, if i'd only start writing that patch | 21:04 |
| melwitt | mriedem: yeah, that's what I was thinking. but that introduces the dependency of nova-api on placement as you mentioned | 21:04 |
| *** Apoorva_ has quit IRC | 21:04 | |
| mriedem | yup | 21:04 |
| mriedem | but, i don't know what else we'd do there | 21:05 |
| *** Apoorva has joined #openstack-nova | 21:05 | |
| mriedem | unless we ran a periodic in scheduler, but that seems wrong | 21:05 |
| *** tbachman has joined #openstack-nova | 21:05 | |
| melwitt | yeah, same. | 21:05 |
| melwitt | I thought about it, noted it in the commit message, and then punted the thought for later :P | 21:05 |
| *** Apoorva_ has joined #openstack-nova | 21:05 | |
| *** dave-mccowan has quit IRC | 21:06 | |
| bauzas | melwitt: mriedem: AFAIK we removed any periodic task in the scheduler service | 21:06 |
| cdent | melwitt: I added yet more commentary on the lifestyle of consumer uuids. Not clear what should be done, but it feels like something. Also: good spot on the dangerous thing. | 21:07 |
| bauzas | and given I'd like to see scheduler code be running within the conductor worker, I tend to pretend it should not exist | 21:07 |
| *** jmlowe has joined #openstack-nova | 21:09 | |
| *** Apoorva has quit IRC | 21:09 | |
| melwitt | cdent: cool, will check it out. yeah, agreed. I'm hoping once we all chime in there, it'll become more clear what we should do | 21:10 |
| bauzas | melwitt: dansmith: fun bug https://bugs.launchpad.net/nova/+bug/1696001 | 21:10 |
| openstack | Launchpad bug 1696001 in OpenStack Compute (nova) "nova-manage cell_v2 map_cell0 can create invalid connection URL" [Undecided,New] | 21:10 |
| *** lucasxu has quit IRC | 21:11 | |
| melwitt | dammit, the rstrip() thing bit us. I guess my suggestion to do the equivalent of a "rreplace()" was actually needed | 21:12 |
| bauzas | yup | 21:12 |
| bauzas | I can cook a patch up | 21:12 |
| mriedem | bauzas: we are certainly running periodic tasks in the scheduler | 21:15 |
| mriedem | in the host manager | 21:15 |
| mriedem | the discover hosts thing from ocata is in there | 21:15 |
| dansmith | is this bug melwitt's fault? | 21:15 |
| dansmith | because I really need her to start taking some of the blame | 21:15 |
| *** fragatina has quit IRC | 21:15 | |
| melwitt | it could be reasoned as my fault for not pushing my rreplace() agenda | 21:15 |
| *** vladikr_ has quit IRC | 21:16 | |
| mriedem | it's my fault | 21:16 |
| mriedem | at least we have the unit test to recreate it, just add mysql+pymysql://nova:password@db.example.com/nova?charset=utf8 to the ddt list | 21:16 |
| melwitt | I'm still not sure that's really the right thing to do either because we want to make a change to the url.database variable really. it just didn't work right because of encoding or something | 21:16 |
| *** tbachman has quit IRC | 21:17 | |
| mriedem | cool that's also in ocata and newton b/c i backported it | 21:17 |
| melwitt | yeah. fun times | 21:18 |
| bauzas | I just feel we need to replace at the index where we found the string | 21:18 |
| *** tidwellr has quit IRC | 21:18 | |
| mriedem | i was doing sometihng like that in an earlier version | 21:18 |
| mriedem | and then the sqla url thing was pointed out to simplify | 21:18 |
| melwitt | bauzas: yeah, my suggestion in the review was "return cell0_db_name.join(connection.rsplit(url.database, 1))" | 21:19 |
| melwitt | but really we want to just change url.database and then re-make the whole url and return it | 21:19 |
| *** thorst_afk has quit IRC | 21:19 | |
| melwitt | but when mriedem tried it, it blew up because of the str() call | 21:19 |
| bauzas | lemme try it | 21:19 |
| *** thorst_afk has joined #openstack-nova | 21:20 | |
| *** rloo has joined #openstack-nova | 21:22 | |
| mriedem | str() blows up because sqla tries to do some things on __str__ | 21:23 |
| mriedem | for unicode or repr type stuff | 21:23 |
| mriedem | https://github.com/zzzeek/sqlalchemy/blob/master/lib/sqlalchemy/engine/url.py#L90 | 21:24 |
| mriedem | https://github.com/zzzeek/sqlalchemy/blob/master/lib/sqlalchemy/engine/url.py#L69 | 21:24 |
| mriedem | yeah it encodes the parameters | 21:24 |
| mriedem | for URL encoding | 21:24 |
| *** thorst_afk has quit IRC | 21:24 | |
| *** sree has joined #openstack-nova | 21:25 | |
| bauzas | mriedem: something I don't understand is why we don't change url.connection directly and stringify url ? | 21:25 |
| *** haukebruno has quit IRC | 21:26 | |
| mriedem | bauzas: you can't str(url) because of ^ | 21:26 |
| melwitt | bauzas: that's what we're talking about | 21:27 |
| mriedem | bauzas: what time is it in le france? | 21:27 |
| zzzeek | mriedem / melwitt : the URL object has all the fields you need to make whatever string you need | 21:27 |
| mriedem | zzzeek: yeah we could build a url i suppose using the parts | 21:27 |
| mriedem | frankenurl | 21:27 |
| mriedem | (tm) | 21:27 |
| bauzas | mriedem: 23:28 | 21:28 |
| melwitt | I'm still back on not understanding why it can't re-stringify the url just because we changed url.database | 21:28 |
| bauzas | sec, uploading the change | 21:28 |
| *** haukebruno has joined #openstack-nova | 21:29 | |
| *** sree has quit IRC | 21:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Avoid unnecessary lazy-loads in mutated_migration_context https://review.openstack.org/471486 | 21:31 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Fix cell0 naming when QS params on the connection https://review.openstack.org/471487 | 21:31 |
| bauzas | mriedem: melwitt: zzzeek: just lemme know why I can't just str() the object | 21:31 |
| bauzas | ^ | 21:31 |
| melwitt | heh, that's the same thing mriedem tried | 21:31 |
| mriedem | bauzas: because https://github.com/zzzeek/sqlalchemy/blob/master/lib/sqlalchemy/engine/url.py#L69 | 21:32 |
| mriedem | you don't get the same thing back | 21:32 |
| mriedem | you'll get URL-encoded username/password | 21:32 |
| bauzas | okay, that's a small thing then | 21:32 |
| bauzas | I'll rev the patch | 21:32 |
| *** yamamoto has joined #openstack-nova | 21:33 | |
| *** slaweq has joined #openstack-nova | 21:37 | |
| *** yamamoto has quit IRC | 21:37 | |
| *** Apoorva_ has quit IRC | 21:38 | |
| *** Apoorva has joined #openstack-nova | 21:39 | |
| *** gouthamr has quit IRC | 21:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: see what else is doing useless migration context stuff https://review.openstack.org/471491 | 21:40 |
| *** baoli has quit IRC | 21:41 | |
| zzzeek | bauzas: you should be able to str() it ? | 21:41 |
| zzzeek | bauzas: repr() will hide the password but str() should be fine... | 21:41 |
| melwitt | zzzeek: we tried that and it didn't work because it was percent-encoding colon and forward slash that were in the database connection url | 21:44 |
| *** mdrabe has quit IRC | 21:44 | |
| *** cheran has joined #openstack-nova | 21:46 | |
| zzzeek | melwitt: can you show me this URL | 21:48 |
| *** itlinux has joined #openstack-nova | 21:48 | |
| bauzas | zzzeek: eg. 'mysql+pymysql://test:abcd0123%AB@controller/nova' | 21:48 |
| melwitt | zzzeek: it was in the review I linked but here ya go: reference = 'mysql+pymysql://nova:abcd0123@AB@controller/nova_cell0' actual = u'mysql+pymysql://nova:abcd0123%40AB@controller/nova_cell0' | 21:49 |
| melwitt | that's one example | 21:49 |
| melwitt | it changed the @ in the password to %40 | 21:49 |
| bauzas | FWIW, I'm tempted to just rindex() where I can find the DB and just join what's before and after | 21:51 |
| melwitt | bauzas: yeah, that's what I pasted earlier | 21:51 |
| zzzeek | bauzas / melwitt OK so that first URL does go back and forth between URL / string symetrically: http://paste.openstack.org/show/611562/ however, it's not as easy to put inside a file b.c. the character is not ASCII | 21:51 |
| zzzeek | a function that returns an ASCII escaped form of a SQLA URL is defintely a good function to have, in oslo.db to start and SQLA can have it too | 21:52 |
| zzzeek | ah ok %40, let's try that | 21:53 |
| bauzas | zzzeek: right, that's the problem we have | 21:53 |
| bauzas | or we could assume it's a non-problem ? | 21:53 |
| zzzeek | bauzas: well the @ sign has to be %40 because @ is syntactically part of the URL | 21:54 |
| zzzeek | >>> u = url.make_url('mysql+pymysql://test:abcd0123%40AB@controller/nova') | 21:54 |
| zzzeek | >>> str(u) | 21:54 |
| zzzeek | 'mysql+pymysql://test:abcd0123%40AB@controller/nova' | 21:54 |
| zzzeek | if we are talking about @ sign, that has to be %40 in all str representations | 21:54 |
| bauzas | right | 21:54 |
| zzzeek | >>> u.password | 21:54 |
| zzzeek | 'abcd0123@AB' | 21:54 |
| zzzeek | that works symettrically, you can put it in a conf file no problem | 21:55 |
| *** itlinux has quit IRC | 21:56 | |
| * zzzeek would potentially argue that a system where passwords are machine generated and need to be consumed and re-stringified in conf files, probably should be very long hex strings w/o special chars | 21:56 | |
| melwitt | hmm | 21:56 |
| openstackgerrit | Merged openstack/nova master: Remove Babel from requirements.txt https://review.openstack.org/468350 | 21:56 |
| bauzas | that's what I wondered when I stated this a potential non-problem | 21:57 |
| *** jwnx has joined #openstack-nova | 21:57 | |
| melwitt | yeah, so I guess to the actual database it will work fine but the user would see their password look different in the conf file after we muck with it | 21:57 |
| *** gomarivera has quit IRC | 21:57 | |
| bauzas | melwitt: we're not updating the file, right? | 21:57 |
| *** gomarivera has joined #openstack-nova | 21:58 | |
| bauzas | melwitt: it's just the connection string that is in the DB that would differ from what the operator provided | 21:58 |
| bauzas | which couldn't be a problem if the service is able to connect thru the given creds even if they are not ASCII-shown | 21:58 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert "Remove Babel from requirements.txt" https://review.openstack.org/471498 | 21:58 |
| mriedem | sdague: ^ | 21:58 |
| mriedem | babel isn't used in code, it's used in config | 21:58 |
| *** kfarr has quit IRC | 21:58 | |
| *** dimtruck is now known as zz_dimtruck | 21:59 | |
| mriedem | sfinucan: ^ also fyi | 21:59 |
| *** penick has quit IRC | 22:00 | |
| *** eharney has quit IRC | 22:02 | |
| *** gomarivera has quit IRC | 22:02 | |
| bauzas | zzzeek: so, just to clarify, the fact that the operator uses special chars like in passwords isn't messed up if we URL-encode those ? | 22:04 |
| zzzeek | bauzas: special chars in the password can all be represented as %XX when they are passed in | 22:04 |
| *** mnestratov|3 has joined #openstack-nova | 22:04 | |
| *** burt has quit IRC | 22:05 | |
| *** egonzalez has quit IRC | 22:11 | |
| *** Hosam has joined #openstack-nova | 22:12 | |
| bauzas | mriedem: still around ? | 22:14 |
| mriedem | depends | 22:14 |
| bauzas | saw the above? | 22:14 |
| mriedem | i didn't follow it no | 22:14 |
| bauzas | the fact that stringifying the URL object makes it RFC1738 compatible doesn't necessarly mean it messes up the connection string | 22:15 |
| *** sdague has quit IRC | 22:18 | |
| melwitt | it doesn't mess it up as far as, it should still work for connecting to the database, but it could store a url different than the user has in their conf file for the nova DB. that is, the password will be percent-encoded for cell0, and they could see the difference when listing cells, if listing cells is possible | 22:25 |
| mriedem | listing cells is possible | 22:26 |
| mriedem | nova-manage cell_v2 list_cells --verbose | 22:26 |
| mriedem | dumps the cells and --verbose dumps the db connection and transport url | 22:27 |
| bauzas | okay, it's becoming late to me | 22:28 |
| *** claudiub has quit IRC | 22:29 | |
| bauzas | if we really want to have the string as close as possible with the original opt value, then I only see something like rindex or rsplit helping us | 22:29 |
| *** lbragstad has quit IRC | 22:30 | |
| bauzas | or we could construct from the URL object by getting the original username and password | 22:31 |
| *** slaweq has quit IRC | 22:31 | |
| bauzas | melwitt: mriedem: ^ | 22:31 |
| *** lyan has quit IRC | 22:31 | |
| melwitt | yeah, that's what we were saying earlier. either the rreplace() equivalent or a frankenurl | 22:31 |
| mriedem | ahem, that'll be $.05 please | 22:32 |
| bauzas | frankenurl ? :) | 22:32 |
| melwitt | the frankenurl seems more correct, if there's not danger of leaving something out while we're constructing it | 22:33 |
| melwitt | collecting royalties in perpetuity | 22:33 |
| *** yamamoto_ has joined #openstack-nova | 22:34 | |
| mriedem | i'd think we could plug the bits into some urlparse method to generate a url and then stringify it | 22:36 |
| mriedem | but i haven't looked of course | 22:36 |
| *** mnestratov|3 has quit IRC | 22:37 | |
| mriedem | eff we have some functional test race issue now | 22:37 |
| mriedem | gdi | 22:37 |
| mriedem | http://logs.openstack.org/98/471498/1/check/gate-nova-tox-functional-ubuntu-xenial/c84dfef/console.html#_2017-06-06_22_23_07_594859 | 22:37 |
| mriedem | cdent: ^ while you're still burning the oil :) | 22:37 |
| *** artom has quit IRC | 22:38 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1696267 | 22:38 |
| openstack | Launchpad bug 1696267 in OpenStack Compute (nova) "Functional test test_sync_standard_traits intermittently fails" [Undecided,New] | 22:38 |
| cdent | mriedem: joy | 22:38 |
| *** artom has joined #openstack-nova | 22:39 | |
| cdent | I'll have a gander, but may not last. If you catch sight of jaypipes (who is either in or going to cali) he might have some ideas, it's basically based off his code. | 22:39 |
| * cdent starts looking | 22:39 | |
| *** yamamoto_ has quit IRC | 22:40 | |
| *** felipemonteiro has quit IRC | 22:40 | |
| *** jobewan has quit IRC | 22:41 | |
| *** bmace has joined #openstack-nova | 22:41 | |
| mriedem | my guess is another test has set _TRAITS_SYNCED to True | 22:41 |
| mriedem | and didn't cleanup | 22:41 |
| mriedem | or something | 22:41 |
| *** bmace has left #openstack-nova | 22:41 | |
| *** catintheroof has quit IRC | 22:41 | |
| *** gomarivera has joined #openstack-nova | 22:43 | |
| cdent | that's a good guess | 22:43 |
| cdent | yeah that's almost ceratinly it. we need a setup | 22:43 |
| mriedem | you pushing or me? | 22:45 |
| cdent | mriedem: i've got it | 22:46 |
| cdent | nearly done | 22:46 |
| cdent | mriedem: i'm doing it as both a setup and teardown | 22:47 |
| cdent | so safety™ | 22:47 |
| mriedem | you can do that with addCleanup in the setUp :) | 22:47 |
| mriedem | if you want to be fancy | 22:47 |
| *** gomarivera has quit IRC | 22:47 | |
| mriedem | tearDown is so 2003 | 22:47 |
| mriedem | well maybe you can't if it's not a function, idk | 22:48 |
| cdent | oh I supose that's the cool way, sure | 22:48 |
| cdent | it can be a function | 22:48 |
| mriedem | lambda?! | 22:48 |
| mriedem | let's make it was weird as possible | 22:48 |
| mriedem | put an eval in there | 22:48 |
| *** thorst_afk has joined #openstack-nova | 22:50 | |
| *** awaugama has quit IRC | 22:51 | |
| *** marst has quit IRC | 22:51 | |
| *** marst has joined #openstack-nova | 22:51 | |
| cdent | i'll create a container to run the function in | 22:52 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Reset the _TRAITS_SYNCED global in Traits tests https://review.openstack.org/471508 | 22:55 |
| cdent | mriedem: ^ there go, feel free to change it if I've left in one of my inevitable typos | 22:55 |
| *** lyan has joined #openstack-nova | 22:56 | |
| *** gouthamr has joined #openstack-nova | 22:56 | |
| *** marst has quit IRC | 22:57 | |
| mriedem | f yes containers ftw | 22:57 |
| *** hongbin has quit IRC | 22:59 | |
| *** liuqing has quit IRC | 22:59 | |
| *** liuqing has joined #openstack-nova | 23:00 | |
| *** ckopper has quit IRC | 23:02 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Reset the _TRAITS_SYNCED global in Traits tests https://review.openstack.org/471508 | 23:02 |
| *** slaweq has joined #openstack-nova | 23:02 | |
| cdent | so many blank lines because I haz a tired. it is tomorrow now, so I'll go. It's all yours now mriedem | 23:03 |
| cdent | goodnight all | 23:03 |
| *** cdent has quit IRC | 23:03 | |
| *** jwnx_ has joined #openstack-nova | 23:05 | |
| *** jwcroppe has quit IRC | 23:07 | |
| *** slaweq has quit IRC | 23:07 | |
| openstackgerrit | Merged openstack/nova master: Catch neutronclient.NotFound on floating deletion https://review.openstack.org/410815 | 23:08 |
| *** jwnx has quit IRC | 23:08 | |
| *** thorst_afk has quit IRC | 23:09 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FixedIPList.get_count_by_project() https://review.openstack.org/446246 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add FloatingIPList.get_count_by_project() https://review.openstack.org/446247 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add get_count_by_vm_state() to InstanceList object https://review.openstack.org/446244 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add SecurityGroupList.get_counts() https://review.openstack.org/446245 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove 'reserved' count from used limits https://review.openstack.org/446242 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count server groups to check quota https://review.openstack.org/446240 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count networks to check quota https://review.openstack.org/446241 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count instances to check quota https://review.openstack.org/416521 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add InstanceGroupList.get_counts() https://review.openstack.org/457857 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add InstanceGroup._remove_members_in_db https://review.openstack.org/457858 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count server group members to check quota https://review.openstack.org/457859 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count security groups to check quota https://review.openstack.org/457860 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count fixed ips to check quota https://review.openstack.org/457861 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 23:16 |
| openstackgerrit | melanie witt proposed openstack/nova master: Count floating ips to check quota https://review.openstack.org/457862 | 23:17 |
| *** xyang1 has quit IRC | 23:18 | |
| openstackgerrit | Merged openstack/nova master: Fixed some nits for microversion 2.48 https://review.openstack.org/471348 | 23:27 |
| openstackgerrit | Merged openstack/nova master: Remove legacy v2.0 code from test_flavor_access https://review.openstack.org/460719 | 23:28 |
| *** david-lyle has quit IRC | 23:28 | |
| *** Oku_OS-away has quit IRC | 23:30 | |
| *** sulo has quit IRC | 23:30 | |
| *** sulo has joined #openstack-nova | 23:31 | |
| *** Oku_OS-away has joined #openstack-nova | 23:32 | |
| *** david-lyle has joined #openstack-nova | 23:32 | |
| *** cfriesen_ has quit IRC | 23:33 | |
| *** cfriesen_ has joined #openstack-nova | 23:33 | |
| *** cfriesen_ has quit IRC | 23:34 | |
| *** cfriesen_ has joined #openstack-nova | 23:34 | |
| mriedem | melwitt: want to put https://review.openstack.org/#/c/471508/ in since it's causing some failures in ci? | 23:35 |
| *** jwnx_ has quit IRC | 23:36 | |
| *** yamamoto_ has joined #openstack-nova | 23:36 | |
| *** Hosam has quit IRC | 23:37 | |
| *** gouthamr has quit IRC | 23:41 | |
| *** yamamoto_ has quit IRC | 23:42 | |
| *** ckopper has joined #openstack-nova | 23:43 | |
| *** gouthamr has joined #openstack-nova | 23:44 | |
| *** sbezverk has quit IRC | 23:46 | |
| *** gouthamr has quit IRC | 23:51 | |
| openstackgerrit | Merged openstack/nova master: Target cell on local delete https://review.openstack.org/464059 | 23:51 |
| *** david-lyle has quit IRC | 23:58 | |
| *** zz_dimtruck is now known as dimtruck | 23:58 | |
| *** artom has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!