| *** catintheroof has joined #openstack-nova | 00:03 | |
| *** dtp has quit IRC | 00:05 | |
| *** jmlowe has quit IRC | 00:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 00:16 |
|---|---|---|
| *** gmann has quit IRC | 00:16 | |
| *** gmann has joined #openstack-nova | 00:16 | |
| *** rajathagasthya has quit IRC | 00:20 | |
| *** itlinux has joined #openstack-nova | 00:22 | |
| *** jmlowe has joined #openstack-nova | 00:23 | |
| *** stewie925 has joined #openstack-nova | 00:24 | |
| stewie925 | hello nova gurus, quick question? is there any rules about special characters in, say, flavor name or tenant name, etc? | 00:24 |
| openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 00:29 |
| openstackgerrit | Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816 | 00:29 |
| *** slaweq has joined #openstack-nova | 00:30 | |
| *** baoli has joined #openstack-nova | 00:35 | |
| *** slaweq has quit IRC | 00:35 | |
| *** baoli has quit IRC | 00:35 | |
| *** baoli has joined #openstack-nova | 00:36 | |
| *** mdnadeem has joined #openstack-nova | 00:36 | |
| *** sree_ has joined #openstack-nova | 00:37 | |
| *** sree_ is now known as Guest40523 | 00:37 | |
| *** kiennt has joined #openstack-nova | 00:37 | |
| *** acormier has quit IRC | 00:38 | |
| *** acormier has joined #openstack-nova | 00:39 | |
| *** thorst_afk has joined #openstack-nova | 00:40 | |
| *** Nel1x has joined #openstack-nova | 00:40 | |
| *** stewie925 has quit IRC | 00:41 | |
| *** Guest40523 has quit IRC | 00:41 | |
| *** thorst_afk has quit IRC | 00:43 | |
| *** _pewp_ has quit IRC | 00:43 | |
| *** thorst_afk has joined #openstack-nova | 00:43 | |
| *** acormier has quit IRC | 00:43 | |
| *** rajathagasthya has joined #openstack-nova | 00:43 | |
| *** randomha1k has joined #openstack-nova | 00:44 | |
| *** catintheroof has quit IRC | 00:44 | |
| *** _pewp_ has joined #openstack-nova | 00:45 | |
| *** acormier has joined #openstack-nova | 00:46 | |
| *** thorst_afk has quit IRC | 00:47 | |
| *** randomha1k has quit IRC | 00:48 | |
| *** awaugama has quit IRC | 00:49 | |
| *** gbarros has joined #openstack-nova | 00:51 | |
| *** takashin has joined #openstack-nova | 00:52 | |
| *** ijw has quit IRC | 00:56 | |
| *** ijw has joined #openstack-nova | 00:57 | |
| *** jamesdenton has quit IRC | 00:57 | |
| *** litao__ has joined #openstack-nova | 00:57 | |
| openstackgerrit | Merged openstack/nova master: Add some more cellsv2 doc goodness https://review.openstack.org/487183 | 00:57 |
| *** jamesdenton has joined #openstack-nova | 00:58 | |
| openstackgerrit | Merged openstack/nova master: always show urls in list_cells https://review.openstack.org/487860 | 00:58 |
| *** iceyao has joined #openstack-nova | 00:58 | |
| *** gyee has quit IRC | 01:00 | |
| *** hongbin has joined #openstack-nova | 01:01 | |
| *** iceyao has quit IRC | 01:03 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 01:06 |
| *** yushb has joined #openstack-nova | 01:10 | |
| *** iceyao has joined #openstack-nova | 01:11 | |
| *** chyka has joined #openstack-nova | 01:12 | |
| *** chyka has quit IRC | 01:12 | |
| *** chyka has joined #openstack-nova | 01:12 | |
| *** Apoorva has joined #openstack-nova | 01:16 | |
| *** yamahata has quit IRC | 01:20 | |
| *** Apoorva_ has quit IRC | 01:20 | |
| *** acormier has quit IRC | 01:20 | |
| *** Apoorva has quit IRC | 01:21 | |
| *** acormier has joined #openstack-nova | 01:21 | |
| *** cdent has quit IRC | 01:21 | |
| *** jmlowe has quit IRC | 01:21 | |
| *** vladikr has joined #openstack-nova | 01:22 | |
| *** annegentle has joined #openstack-nova | 01:22 | |
| *** Shunli has joined #openstack-nova | 01:24 | |
| *** itlinux has quit IRC | 01:24 | |
| *** https_GK1wmSU has joined #openstack-nova | 01:24 | |
| *** jmlowe has joined #openstack-nova | 01:24 | |
| *** acormier has quit IRC | 01:26 | |
| *** https_GK1wmSU has left #openstack-nova | 01:27 | |
| *** mriedem has quit IRC | 01:30 | |
| *** sam_nowitzki has joined #openstack-nova | 01:31 | |
| *** ijw has quit IRC | 01:31 | |
| *** ijw has joined #openstack-nova | 01:32 | |
| *** tuanluong has joined #openstack-nova | 01:38 | |
| *** gouthamr has joined #openstack-nova | 01:42 | |
| *** thorst_afk has joined #openstack-nova | 01:44 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to Instance object https://review.openstack.org/489408 | 01:44 |
| openstackgerrit | Merged openstack/nova master: add redirects for existing broken docs urls https://review.openstack.org/489650 | 01:48 |
| openstackgerrit | Merged openstack/nova master: Deprecate cells v1 https://review.openstack.org/489632 | 01:49 |
| *** markvoelker has joined #openstack-nova | 01:54 | |
| *** rajathagasthya has quit IRC | 01:55 | |
| *** esberglu has quit IRC | 01:59 | |
| *** slagle has quit IRC | 01:59 | |
| *** thorst_afk has quit IRC | 02:00 | |
| *** takashin has quit IRC | 02:02 | |
| *** baoli has quit IRC | 02:02 | |
| *** baoli has joined #openstack-nova | 02:02 | |
| *** tbachman has quit IRC | 02:03 | |
| *** dixiaoli has joined #openstack-nova | 02:05 | |
| *** slaweq has joined #openstack-nova | 02:07 | |
| *** itlinux has joined #openstack-nova | 02:07 | |
| *** bkopilov_ has quit IRC | 02:11 | |
| *** slaweq has quit IRC | 02:11 | |
| *** slagle has joined #openstack-nova | 02:12 | |
| *** crushil has quit IRC | 02:18 | |
| *** crushil has joined #openstack-nova | 02:18 | |
| *** tbachman has joined #openstack-nova | 02:22 | |
| *** markvoelker has quit IRC | 02:23 | |
| *** yamamoto has quit IRC | 02:24 | |
| *** ijw has quit IRC | 02:25 | |
| *** slaweq has joined #openstack-nova | 02:25 | |
| *** tbachman has quit IRC | 02:27 | |
| *** yamamoto has joined #openstack-nova | 02:30 | |
| *** slaweq has quit IRC | 02:30 | |
| *** tbachman has joined #openstack-nova | 02:31 | |
| *** yamamoto has quit IRC | 02:35 | |
| *** yamamoto has joined #openstack-nova | 02:36 | |
| *** esberglu has joined #openstack-nova | 02:38 | |
| *** saphi has joined #openstack-nova | 02:39 | |
| *** dave-mcc_ has joined #openstack-nova | 02:40 | |
| *** yamamoto has quit IRC | 02:41 | |
| *** esberglu has quit IRC | 02:42 | |
| *** dave-mccowan has quit IRC | 02:42 | |
| *** baoli has quit IRC | 02:43 | |
| *** yamamoto has joined #openstack-nova | 02:45 | |
| *** dixiaoli has quit IRC | 02:47 | |
| *** dixiaoli has joined #openstack-nova | 02:51 | |
| *** nicolasbock has joined #openstack-nova | 02:57 | |
| *** markvoelker has joined #openstack-nova | 02:59 | |
| *** itlinux has quit IRC | 03:00 | |
| *** takashin has joined #openstack-nova | 03:05 | |
| *** tbachman has quit IRC | 03:11 | |
| *** dave-mccowan has joined #openstack-nova | 03:12 | |
| *** dave-mcc_ has quit IRC | 03:14 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 03:19 |
| openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 03:19 |
| openstackgerrit | Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816 | 03:19 |
| *** rajathagasthya has joined #openstack-nova | 03:22 | |
| *** thorst_afk has joined #openstack-nova | 03:23 | |
| *** thorst_afk has quit IRC | 03:27 | |
| *** hongbin has quit IRC | 03:30 | |
| *** mamandle has joined #openstack-nova | 03:32 | |
| *** yamamoto has quit IRC | 03:33 | |
| *** dave-mccowan has quit IRC | 03:33 | |
| *** udesale has joined #openstack-nova | 03:33 | |
| *** yangyapeng has joined #openstack-nova | 03:36 | |
| openstackgerrit | Spencer Yu proposed openstack/python-novaclient master: Nova client should retry with Retry-After value https://review.openstack.org/447766 | 03:37 |
| *** gbarros has quit IRC | 03:39 | |
| *** johnhaan has joined #openstack-nova | 03:39 | |
| *** bkopilov_ has joined #openstack-nova | 03:44 | |
| *** crushil has quit IRC | 03:45 | |
| *** crushil has joined #openstack-nova | 03:46 | |
| *** links has joined #openstack-nova | 03:48 | |
| *** yamamoto has joined #openstack-nova | 03:49 | |
| *** sree has joined #openstack-nova | 04:00 | |
| *** itlinux has joined #openstack-nova | 04:02 | |
| *** rajathagasthya has quit IRC | 04:03 | |
| *** slaweq has joined #openstack-nova | 04:03 | |
| *** yamamoto has quit IRC | 04:05 | |
| *** mamandle has quit IRC | 04:05 | |
| *** gouthamr has quit IRC | 04:06 | |
| *** gouthamr has joined #openstack-nova | 04:07 | |
| *** slaweq has quit IRC | 04:07 | |
| *** yamamoto has joined #openstack-nova | 04:10 | |
| *** mamandle has joined #openstack-nova | 04:13 | |
| *** gongysh has joined #openstack-nova | 04:13 | |
| *** iceyao has quit IRC | 04:14 | |
| *** mamandle has quit IRC | 04:16 | |
| *** mamandle has joined #openstack-nova | 04:18 | |
| *** yushb has quit IRC | 04:18 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:19 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 04:20 |
| *** peter-hamilton has quit IRC | 04:21 | |
| *** adisky__ has joined #openstack-nova | 04:24 | |
| *** sree has quit IRC | 04:28 | |
| *** udesale__ has joined #openstack-nova | 04:29 | |
| *** kornicameister has quit IRC | 04:31 | |
| *** takashin has left #openstack-nova | 04:31 | |
| *** udesale has quit IRC | 04:31 | |
| *** yushb has joined #openstack-nova | 04:32 | |
| *** kornicameister has joined #openstack-nova | 04:32 | |
| *** Nel1x has quit IRC | 04:33 | |
| *** kbaegis has joined #openstack-nova | 04:36 | |
| *** iceyao has joined #openstack-nova | 04:37 | |
| *** dixiaoli has quit IRC | 04:40 | |
| *** ijw has joined #openstack-nova | 04:40 | |
| *** iceyao has quit IRC | 04:42 | |
| *** randomha1k has joined #openstack-nova | 04:44 | |
| *** ijw has quit IRC | 04:45 | |
| *** randomha1k has quit IRC | 04:48 | |
| *** iceyao has joined #openstack-nova | 04:48 | |
| *** yamamoto has quit IRC | 04:50 | |
| *** sree has joined #openstack-nova | 04:51 | |
| *** ijw has joined #openstack-nova | 04:52 | |
| *** jichen has joined #openstack-nova | 04:56 | |
| *** jichen has quit IRC | 04:57 | |
| *** jichen has joined #openstack-nova | 04:59 | |
| *** ijw has quit IRC | 05:00 | |
| *** chyka has quit IRC | 05:06 | |
| *** vks1 has joined #openstack-nova | 05:06 | |
| *** chyka has joined #openstack-nova | 05:06 | |
| *** vks1 has quit IRC | 05:06 | |
| *** yamamoto has joined #openstack-nova | 05:07 | |
| *** vks1 has joined #openstack-nova | 05:09 | |
| *** chyka has quit IRC | 05:11 | |
| *** mamandle has quit IRC | 05:13 | |
| *** ratailor has joined #openstack-nova | 05:15 | |
| *** yangyapeng has quit IRC | 05:17 | |
| *** yangyapeng has joined #openstack-nova | 05:19 | |
| *** psachin has joined #openstack-nova | 05:20 | |
| *** iceyao has quit IRC | 05:22 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add redirect for api-microversion-history doc https://review.openstack.org/489641 | 05:22 |
| *** thorst_afk has joined #openstack-nova | 05:23 | |
| *** iceyao has joined #openstack-nova | 05:24 | |
| *** slaweq has joined #openstack-nova | 05:25 | |
| *** dixiaoli has joined #openstack-nova | 05:28 | |
| *** thorst_afk has quit IRC | 05:28 | |
| *** dixiaoli has quit IRC | 05:28 | |
| *** dixiaoli has joined #openstack-nova | 05:29 | |
| *** slaweq has quit IRC | 05:30 | |
| *** ekuris has joined #openstack-nova | 05:33 | |
| *** kiennt has quit IRC | 05:36 | |
| *** mamandle has joined #openstack-nova | 05:37 | |
| *** yushb has quit IRC | 05:39 | |
| *** kbaegis has quit IRC | 05:46 | |
| *** mdnadeem has quit IRC | 05:47 | |
| *** trinaths has joined #openstack-nova | 05:48 | |
| *** mdnadeem has joined #openstack-nova | 05:48 | |
| *** mdnadeem has quit IRC | 05:49 | |
| *** mdnadeem has joined #openstack-nova | 05:49 | |
| *** mdnadeem_ has joined #openstack-nova | 05:50 | |
| *** mdnadeem has quit IRC | 05:51 | |
| *** mdnadeem has joined #openstack-nova | 05:51 | |
| *** vks1 has quit IRC | 05:53 | |
| *** gouthamr has quit IRC | 05:53 | |
| *** yasemin has quit IRC | 05:54 | |
| *** mdnadeem_ has quit IRC | 05:55 | |
| *** kiennt has joined #openstack-nova | 05:56 | |
| *** moshele has joined #openstack-nova | 05:56 | |
| *** yushb has joined #openstack-nova | 06:00 | |
| *** hoonetorg has quit IRC | 06:02 | |
| *** cfriesen has quit IRC | 06:02 | |
| *** vks1 has joined #openstack-nova | 06:03 | |
| *** Oku_OS-away is now known as Oku_OS | 06:09 | |
| *** kiennt has quit IRC | 06:10 | |
| *** kiennt has joined #openstack-nova | 06:12 | |
| *** sridharg has joined #openstack-nova | 06:13 | |
| *** hoonetorg has joined #openstack-nova | 06:15 | |
| *** slaweq has joined #openstack-nova | 06:15 | |
| *** mdnadeem has quit IRC | 06:18 | |
| *** mpaolino has joined #openstack-nova | 06:19 | |
| *** mingyu has joined #openstack-nova | 06:22 | |
| *** mingyu has quit IRC | 06:25 | |
| *** jaosorior has quit IRC | 06:25 | |
| *** mingyu has joined #openstack-nova | 06:26 | |
| *** sshwarts has joined #openstack-nova | 06:26 | |
| *** jaosorior has joined #openstack-nova | 06:34 | |
| *** sam_nowitzki has quit IRC | 06:34 | |
| *** pcaruana has joined #openstack-nova | 06:34 | |
| *** sam_nowitzki has joined #openstack-nova | 06:34 | |
| *** markus_z has joined #openstack-nova | 06:36 | |
| *** johnhaan has quit IRC | 06:39 | |
| *** hshiina has joined #openstack-nova | 06:43 | |
| *** yangyape_ has joined #openstack-nova | 06:43 | |
| *** edmondsw has joined #openstack-nova | 06:43 | |
| *** mamandle has quit IRC | 06:45 | |
| *** hao has joined #openstack-nova | 06:45 | |
| *** yangyape_ has quit IRC | 06:46 | |
| *** yangyape_ has joined #openstack-nova | 06:46 | |
| *** yangyapeng has quit IRC | 06:46 | |
| *** rcernin has joined #openstack-nova | 06:46 | |
| *** mamandle has joined #openstack-nova | 06:47 | |
| *** esberglu has joined #openstack-nova | 06:48 | |
| *** edmondsw has quit IRC | 06:48 | |
| *** mdnadeem has joined #openstack-nova | 06:48 | |
| *** yangyapeng has joined #openstack-nova | 06:50 | |
| *** yangyape_ has quit IRC | 06:50 | |
| *** markvoelker has quit IRC | 06:51 | |
| *** sam_nowitzki has quit IRC | 06:52 | |
| *** esberglu has quit IRC | 06:52 | |
| *** yangyapeng has quit IRC | 06:54 | |
| *** andreas_s has joined #openstack-nova | 06:55 | |
| *** cjvolzka has quit IRC | 06:56 | |
| *** prateek has joined #openstack-nova | 06:59 | |
| openstackgerrit | Merged openstack/nova master: Additional assertions to resize tests https://review.openstack.org/489714 | 07:04 |
| *** slaweq_ has joined #openstack-nova | 07:05 | |
| *** aarefiev_afk is now known as aarefiev | 07:07 | |
| *** sapcc-bot has joined #openstack-nova | 07:10 | |
| *** mkoderer_ has joined #openstack-nova | 07:10 | |
| *** carthaca_ has joined #openstack-nova | 07:10 | |
| *** tpatzig_ has joined #openstack-nova | 07:10 | |
| *** slaweq_ has quit IRC | 07:10 | |
| *** tpatzig_ has quit IRC | 07:12 | |
| *** mkoderer_ has quit IRC | 07:12 | |
| *** carthaca_ has quit IRC | 07:12 | |
| *** ducnc has joined #openstack-nova | 07:13 | |
| *** tpatzig_ has joined #openstack-nova | 07:13 | |
| *** mkoderer_ has joined #openstack-nova | 07:13 | |
| *** carthaca_ has joined #openstack-nova | 07:13 | |
| *** sapcc-bot2 has quit IRC | 07:14 | |
| *** tpatzig_ has quit IRC | 07:15 | |
| *** carthaca_ has quit IRC | 07:15 | |
| *** mkoderer_ has quit IRC | 07:15 | |
| *** https_GK1wmSU has joined #openstack-nova | 07:16 | |
| *** https_GK1wmSU has left #openstack-nova | 07:19 | |
| *** trinaths is now known as trinaths|lunch | 07:19 | |
| *** zz9pzza has joined #openstack-nova | 07:24 | |
| *** thorst_afk has joined #openstack-nova | 07:24 | |
| *** takashin has joined #openstack-nova | 07:27 | |
| *** sree has quit IRC | 07:28 | |
| *** sree has joined #openstack-nova | 07:28 | |
| *** sree has quit IRC | 07:29 | |
| *** thorst_afk has quit IRC | 07:29 | |
| *** sree has joined #openstack-nova | 07:31 | |
| *** udesale has joined #openstack-nova | 07:33 | |
| *** annegentle has quit IRC | 07:34 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool https://review.openstack.org/489451 | 07:35 |
| *** udesale__ has quit IRC | 07:35 | |
| *** lpetrut has joined #openstack-nova | 07:37 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool https://review.openstack.org/489451 | 07:39 |
| *** randomha1k has joined #openstack-nova | 07:44 | |
| *** ralonsoh has joined #openstack-nova | 07:46 | |
| *** randomha1k has quit IRC | 07:48 | |
| *** gcb_ has quit IRC | 07:48 | |
| *** tesseract has joined #openstack-nova | 07:48 | |
| *** yangyapeng has joined #openstack-nova | 07:50 | |
| *** damien_r has joined #openstack-nova | 07:51 | |
| *** alexchadin has joined #openstack-nova | 07:53 | |
| *** trinaths|lunch is now known as trinaths | 07:59 | |
| *** mamandle has quit IRC | 07:59 | |
| *** josecastroleon has quit IRC | 08:00 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** yangyapeng has quit IRC | 08:05 | |
| *** yangyapeng has joined #openstack-nova | 08:06 | |
| *** mamandle has joined #openstack-nova | 08:06 | |
| *** ralonsoh has quit IRC | 08:12 | |
| *** ralonsoh has joined #openstack-nova | 08:13 | |
| *** derekh has joined #openstack-nova | 08:15 | |
| *** ralonsoh has quit IRC | 08:15 | |
| *** https_GK1wmSU has joined #openstack-nova | 08:16 | |
| *** saphi has quit IRC | 08:17 | |
| *** ralonsoh has joined #openstack-nova | 08:18 | |
| *** https_GK1wmSU has left #openstack-nova | 08:19 | |
| *** zenoway has joined #openstack-nova | 08:19 | |
| *** udesale__ has joined #openstack-nova | 08:20 | |
| *** udesale has quit IRC | 08:22 | |
| *** josecastroleon has joined #openstack-nova | 08:23 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Rehome OVO unit tests to tests.unit.test_object.py https://review.openstack.org/489922 | 08:24 |
| openstackgerrit | jichenjc proposed openstack/nova master: Raise MigrationPreCheckError if instance location is unavailable https://review.openstack.org/469779 | 08:26 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP:enhance error message for input field https://review.openstack.org/489519 | 08:26 |
| *** jpena has joined #openstack-nova | 08:26 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Rehome OVO unit tests to tests.unit.test_object.py https://review.openstack.org/489922 | 08:26 |
| *** efoley has joined #openstack-nova | 08:27 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: allow null to be input for key_name https://review.openstack.org/489505 | 08:27 |
| *** efoley has quit IRC | 08:28 | |
| *** efoley_ has joined #openstack-nova | 08:28 | |
| *** sree has quit IRC | 08:30 | |
| gibi | I don't see test patch up for the resize same host case so I start building that up top of master (https://bugs.launchpad.net/nova/+bug/1707252) | 08:30 |
| openstack | Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed] | 08:30 |
| openstackgerrit | jichenjc proposed openstack/nova master: Move common definition into common layer https://review.openstack.org/489491 | 08:31 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP:check query param for service's index function https://review.openstack.org/489492 | 08:31 |
| *** edmondsw has joined #openstack-nova | 08:31 | |
| *** lucas-afk is now known as lucasagomes | 08:32 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add description for tag support in attach https://review.openstack.org/481479 | 08:33 |
| openstackgerrit | jichenjc proposed openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478 | 08:33 |
| * stephenfin returns | 08:33 | |
| *** openstackgerrit has quit IRC | 08:33 | |
| stephenfin | ...and bursts into tears as he opens his email | 08:34 |
| *** edmondsw has quit IRC | 08:36 | |
| *** openstackgerrit has joined #openstack-nova | 08:37 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | 08:37 |
| *** yushb has quit IRC | 08:37 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | 08:38 |
| *** ralonsoh_ has joined #openstack-nova | 08:40 | |
| *** ralonsoh has quit IRC | 08:40 | |
| *** yangyapeng has quit IRC | 08:43 | |
| *** zenoway has quit IRC | 08:43 | |
| *** yangyapeng has joined #openstack-nova | 08:43 | |
| *** zz9pzza has quit IRC | 08:48 | |
| *** zenoway has joined #openstack-nova | 08:50 | |
| *** rmart04 has joined #openstack-nova | 08:53 | |
| *** slaweq_ has joined #openstack-nova | 08:59 | |
| *** markus_z has quit IRC | 09:00 | |
| *** hrw has joined #openstack-nova | 09:00 | |
| hrw | hello | 09:01 |
| *** zz9pzza has joined #openstack-nova | 09:02 | |
| *** zenoway has quit IRC | 09:04 | |
| *** zenoway has joined #openstack-nova | 09:04 | |
| hrw | does someone here uses non-x86 machines or q35 based x86-64 vm? PCI hotplug is different there than on typical x86-64 vm and I do not know (yet) does nova supports adding pcie conttrollers (pcie-root-port nodes) | 09:04 |
| *** slaweq_ has quit IRC | 09:05 | |
| *** yangyapeng has quit IRC | 09:08 | |
| *** hshiina has quit IRC | 09:09 | |
| *** abalutoiu has joined #openstack-nova | 09:11 | |
| *** yangyapeng has joined #openstack-nova | 09:14 | |
| *** markus_z has joined #openstack-nova | 09:15 | |
| *** zenoway has quit IRC | 09:16 | |
| *** yangyapeng has quit IRC | 09:16 | |
| *** yangyapeng has joined #openstack-nova | 09:17 | |
| *** dtantsur|afk is now known as dtantsur | 09:17 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | 09:21 |
| *** slaweq_ has joined #openstack-nova | 09:22 | |
| *** priteau has joined #openstack-nova | 09:24 | |
| *** thorst_afk has joined #openstack-nova | 09:25 | |
| *** slaweq_ has quit IRC | 09:27 | |
| *** gcb has joined #openstack-nova | 09:28 | |
| *** esberglu has joined #openstack-nova | 09:30 | |
| *** thorst_afk has quit IRC | 09:30 | |
| *** sdague has joined #openstack-nova | 09:32 | |
| *** esberglu has quit IRC | 09:34 | |
| *** Shunli has quit IRC | 09:35 | |
| *** sree has joined #openstack-nova | 09:39 | |
| *** mkucia has joined #openstack-nova | 09:39 | |
| *** yushb has joined #openstack-nova | 09:41 | |
| *** ducnc has quit IRC | 09:41 | |
| *** sree has quit IRC | 09:43 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | 09:44 |
| *** zz9pzza has quit IRC | 09:45 | |
| *** jichen has quit IRC | 09:49 | |
| openstackgerrit | Kevin Zhao proposed openstack/nova master: Make uefi as the default boot perproties for AArch64 https://review.openstack.org/489951 | 09:54 |
| *** yassine has quit IRC | 09:59 | |
| *** acormier has joined #openstack-nova | 09:59 | |
| openstackgerrit | Merged openstack/nova master: Add release note for PUT /os-services/* for non-compute services https://review.openstack.org/489621 | 10:00 |
| *** kiennt has quit IRC | 10:01 | |
| *** dixiaoli has quit IRC | 10:04 | |
| *** zz9pzza has joined #openstack-nova | 10:06 | |
| *** yushb has quit IRC | 10:06 | |
| *** tuanluong has quit IRC | 10:07 | |
| *** sambetts|afk is now known as sambetts | 10:09 | |
| *** jaosorior has quit IRC | 10:10 | |
| *** acormier has quit IRC | 10:13 | |
| *** iceyao has quit IRC | 10:17 | |
| *** edmondsw has joined #openstack-nova | 10:19 | |
| *** slaweq_ has joined #openstack-nova | 10:23 | |
| *** edmondsw has quit IRC | 10:24 | |
| *** slaweq_ has quit IRC | 10:27 | |
| *** mingyu has quit IRC | 10:28 | |
| *** iceyao has joined #openstack-nova | 10:32 | |
| *** iceyao has quit IRC | 10:36 | |
| bauzas | sdague: FWIW, looking at https://bugs.launchpad.net/nova/+bug/1707934 | 10:39 |
| openstack | Launchpad bug 1707934 in OpenStack Compute (nova) "RequestSpec not found error message during VM migration" [Undecided,New] | 10:39 |
| *** alexchadin has quit IRC | 10:43 | |
| *** zhurong has quit IRC | 10:43 | |
| *** hferenc has quit IRC | 10:47 | |
| *** bkopilov_ has quit IRC | 10:50 | |
| *** markvoelker has joined #openstack-nova | 10:52 | |
| *** iceyao has joined #openstack-nova | 10:54 | |
| *** gszasz has joined #openstack-nova | 10:54 | |
| *** markvoelker has quit IRC | 10:57 | |
| *** Shunli has joined #openstack-nova | 11:00 | |
| *** jaosorior has joined #openstack-nova | 11:02 | |
| *** mingyu has joined #openstack-nova | 11:04 | |
| sdague | bauzas: cool, yeh, I was going to poke you on that one | 11:04 |
| sdague | the new bug inbound queue is a lot less than it used to be, I remember regularly seeing 5 - 10 bugs a day | 11:05 |
| bauzas | yeah I'm surprised to only see 3 bugs | 11:05 |
| bauzas | (that's what we had when I did that this morning) | 11:05 |
| bauzas | so either people aren't yet there and just running old versions | 11:06 |
| bauzas | or we have less people testing those | 11:06 |
| sdague | yeh, it does change at different parts of the cycle, once we get RCs I expect more honestly | 11:06 |
| sdague | though, we also have less randomly misconfigured cloud Invalid bugs | 11:06 |
| sdague | so that's nice | 11:06 |
| bauzas | yup | 11:08 |
| *** udesale__ has quit IRC | 11:09 | |
| *** ababich has joined #openstack-nova | 11:12 | |
| *** smatzek_ has joined #openstack-nova | 11:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 11:13 |
| *** kaisers1 has quit IRC | 11:15 | |
| *** gongysh has quit IRC | 11:15 | |
| *** esberglu has joined #openstack-nova | 11:18 | |
| *** jaypipes has joined #openstack-nova | 11:18 | |
| *** kaisers1 has joined #openstack-nova | 11:21 | |
| *** thorst_afk has joined #openstack-nova | 11:21 | |
| *** hferenc has joined #openstack-nova | 11:21 | |
| *** Shunli has quit IRC | 11:22 | |
| *** jaypipes is now known as jaypipes-jurydut | 11:22 | |
| *** jaypipes-jurydut is now known as jaypipesjuryduty | 11:22 | |
| *** esberglu has quit IRC | 11:23 | |
| *** mingyu has quit IRC | 11:23 | |
| *** trinaths has left #openstack-nova | 11:26 | |
| *** moshele has quit IRC | 11:29 | |
| *** moshele has joined #openstack-nova | 11:29 | |
| *** edmondsw has joined #openstack-nova | 11:29 | |
| mamandle | bauzas: alex_xu: mriedem: can you please take a look at https://review.openstack.org/#/c/489848/, its cherrypick of https://review.openstack.org/#/c/483911/ to Ocata. Thanks. | 11:30 |
| *** edmondsw has quit IRC | 11:31 | |
| *** sapcc-bot has quit IRC | 11:32 | |
| *** sapcc-bot has joined #openstack-nova | 11:32 | |
| *** carthaca_ has joined #openstack-nova | 11:32 | |
| *** baoli has joined #openstack-nova | 11:32 | |
| *** baoli has quit IRC | 11:32 | |
| *** tpatzig_ has joined #openstack-nova | 11:32 | |
| *** mkoderer_ has joined #openstack-nova | 11:32 | |
| *** baoli has joined #openstack-nova | 11:32 | |
| *** baoli has quit IRC | 11:33 | |
| *** carthaca_ has quit IRC | 11:34 | |
| *** tpatzig_ has quit IRC | 11:34 | |
| *** mkoderer_ has quit IRC | 11:34 | |
| *** lucasagomes is now known as lucas-hungry | 11:37 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Always use application/json accept header in report client https://review.openstack.org/489772 | 11:37 |
| *** abalutoiu has quit IRC | 11:38 | |
| *** sree has joined #openstack-nova | 11:40 | |
| *** baoli has joined #openstack-nova | 11:40 | |
| *** nicolasbock_ has joined #openstack-nova | 11:41 | |
| *** nicolasbock__ has joined #openstack-nova | 11:42 | |
| *** nicolasbock has quit IRC | 11:44 | |
| *** sree has quit IRC | 11:45 | |
| *** nicolasbock has joined #openstack-nova | 11:45 | |
| *** baoli has quit IRC | 11:46 | |
| tonyb | bauzas: Any chance you can look at: https://review.openstack.org/489078 It's need to get the current glanceclient/glance_store into pike | 11:46 |
| *** nicolasbock_ has quit IRC | 11:46 | |
| *** nicolasbock__ has quit IRC | 11:47 | |
| tonyb | bauzas: I'll nag sdague and mriedem when I catch them tomorrow | 11:47 |
| *** kaisers1 has quit IRC | 11:48 | |
| *** mingyu has joined #openstack-nova | 11:48 | |
| *** gmann_ has joined #openstack-nova | 11:48 | |
| *** cdent has joined #openstack-nova | 11:50 | |
| *** vks1 has quit IRC | 11:52 | |
| *** markvoelker has joined #openstack-nova | 11:53 | |
| *** mingyu has quit IRC | 11:54 | |
| *** mingyu has joined #openstack-nova | 11:55 | |
| *** litao__ has quit IRC | 11:56 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | 12:00 |
| *** yassine has joined #openstack-nova | 12:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 12:02 |
| *** kaisers1 has joined #openstack-nova | 12:02 | |
| *** baoli has joined #openstack-nova | 12:02 | |
| *** mdnadeem has quit IRC | 12:03 | |
| *** chyka has joined #openstack-nova | 12:04 | |
| *** jpena is now known as jpena|lunch | 12:06 | |
| *** chyka has quit IRC | 12:09 | |
| *** alexchadin has joined #openstack-nova | 12:15 | |
| *** sean-k-mooney has quit IRC | 12:16 | |
| *** smatzek_ is now known as smatzek | 12:21 | |
| *** markvoelker has quit IRC | 12:22 | |
| *** markvoelker has joined #openstack-nova | 12:22 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: WIP: Backlog spec on preemptible servers https://review.openstack.org/438640 | 12:22 |
| *** kbaegis has joined #openstack-nova | 12:23 | |
| *** slaweq_ has joined #openstack-nova | 12:25 | |
| *** phuongnh has joined #openstack-nova | 12:29 | |
| *** lyan has joined #openstack-nova | 12:30 | |
| *** slaweq_ has quit IRC | 12:30 | |
| *** pchavva has joined #openstack-nova | 12:32 | |
| *** ratailor has quit IRC | 12:32 | |
| *** edmondsw has joined #openstack-nova | 12:33 | |
| *** iceyao has quit IRC | 12:35 | |
| *** baoli has quit IRC | 12:36 | |
| *** liverpooler has joined #openstack-nova | 12:36 | |
| gibi | whoa gerrit now has the new OpenStack logo | 12:38 |
| *** lucas-hungry is now known as lucasagomes | 12:39 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515 | 12:40 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | 12:40 |
| *** phuongnh has quit IRC | 12:40 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 12:41 |
| mdbooth | gibi: Reigned in my instincts with https://review.openstack.org/#/c/489637/ :) | 12:41 |
| gibi | mdbooth: looking.... | 12:42 |
| *** baoli has joined #openstack-nova | 12:43 | |
| *** sdague_ has joined #openstack-nova | 12:43 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | 12:44 |
| *** mamandle has quit IRC | 12:45 | |
| alex_xu | nova api meeting is in 10 mins at #openstack-meeeting-4 | 12:49 |
| *** abalutoiu has joined #openstack-nova | 12:49 | |
| *** baoli has quit IRC | 12:51 | |
| *** gmann__ has joined #openstack-nova | 12:55 | |
| *** gmann_ has quit IRC | 12:56 | |
| *** gouthamr has joined #openstack-nova | 12:56 | |
| *** crushil has quit IRC | 12:58 | |
| *** crushil has joined #openstack-nova | 12:59 | |
| alex_xu | nova api meeting is running | 13:00 |
| *** yamamoto has quit IRC | 13:02 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | 13:02 |
| *** iceyao has joined #openstack-nova | 13:03 | |
| *** sapcc-bot has quit IRC | 13:03 | |
| *** sapcc-bot has joined #openstack-nova | 13:03 | |
| *** xyang1 has joined #openstack-nova | 13:04 | |
| *** jpena|lunch is now known as jpena | 13:06 | |
| *** iceyao has quit IRC | 13:07 | |
| *** gmann__ has quit IRC | 13:11 | |
| *** esberglu has joined #openstack-nova | 13:11 | |
| *** lucasxu has joined #openstack-nova | 13:12 | |
| *** bkopilov_ has joined #openstack-nova | 13:13 | |
| *** awaugama has joined #openstack-nova | 13:15 | |
| *** yamamoto has joined #openstack-nova | 13:18 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | 13:22 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | 13:25 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637 | 13:25 |
| edleafe | Man, that new OpenStack logo in Gerrit looks huge | 13:26 |
| gibi | edleafe: first I didn't know why my review tabs looks soo strange | 13:27 |
| *** sree has joined #openstack-nova | 13:27 | |
| mdbooth | edleafe: And compression artifact-y | 13:27 |
| mdbooth | Is it scaled up, or something? | 13:28 |
| mdbooth | Interestingly, it's actually a gorgeous SVG | 13:28 |
| *** sree has quit IRC | 13:28 | |
| gibi | it is not artifacty for me | 13:28 |
| mdbooth | So it just that Chrome sucks at displaying it to me | 13:28 |
| *** prateek has quit IRC | 13:28 | |
| gibi | can be a browser issue | 13:29 |
| *** sree has joined #openstack-nova | 13:29 | |
| *** kylek3h has joined #openstack-nova | 13:29 | |
| *** acormier has joined #openstack-nova | 13:29 | |
| *** acormier has quit IRC | 13:31 | |
| *** acormier has joined #openstack-nova | 13:31 | |
| *** felipemonteiro has joined #openstack-nova | 13:31 | |
| *** baoli has joined #openstack-nova | 13:32 | |
| *** kbaegis has quit IRC | 13:32 | |
| *** jistr is now known as jistr|mtg | 13:32 | |
| *** xinliang has quit IRC | 13:32 | |
| *** cleong has joined #openstack-nova | 13:33 | |
| *** kbaegis has joined #openstack-nova | 13:34 | |
| *** kbaegis has quit IRC | 13:34 | |
| *** kbaegis has joined #openstack-nova | 13:34 | |
| *** abalutoiu has quit IRC | 13:36 | |
| *** abalutoiu has joined #openstack-nova | 13:36 | |
| cdent | edleafe: I responded to your comments on the aggregate map update thing. I feel like I’m not understanding you and you’re not understanding me. | 13:37 |
| *** felipemonteiro has quit IRC | 13:37 | |
| *** yangyapeng has quit IRC | 13:38 | |
| *** tbachman has joined #openstack-nova | 13:38 | |
| edleafe | cdent: sorry, still caffeinating | 13:41 |
| *** mriedem has joined #openstack-nova | 13:41 | |
| cdent | i know how that can be | 13:42 |
| edleafe | cdent: I was trying to say that another way of reducing the need to update the aggregate map would be to make those calls through the report client - at least the ones from Nova. This way the client "knows" when things change | 13:42 |
| cdent | “those calls”? | 13:42 |
| edleafe | cdent: when we set an aggregate in placement that nova cares about | 13:42 |
| cdent | oh | 13:42 |
| *** tbachman has quit IRC | 13:43 | |
| cdent | yeah, no, how would we know | 13:43 |
| *** sdague has quit IRC | 13:43 | |
| sdague_ | efried: you have a working devstack run on your change? | 13:44 |
| efried | sdague_ I put four links in the change set. | 13:44 |
| *** burt has joined #openstack-nova | 13:44 | |
| sdague_ | ah, there it is | 13:44 |
| *** xinliang has joined #openstack-nova | 13:45 | |
| *** randomha1k has joined #openstack-nova | 13:45 | |
| *** alexchadin has quit IRC | 13:45 | |
| efried | sdague_ One pair alters devstack to make glance use the new code. The other pair builds on that and proves ironic. | 13:46 |
| efried | sdague_ Each pair has one half using ksa creds to do real discovery; the other half using endpoint_override explicitly. | 13:46 |
| edleafe | cdent: just throwing out ideas to reduce the number of calls. We call so often precisely because we don't have a way of knowing otherwise | 13:47 |
| sdague_ | hmmm... ok, I guess I didn't see the change I was expecting, which was just pulling out the api_servers set | 13:47 |
| cdent | edleafe: i like thrown ideas | 13:50 |
| *** tbachman has joined #openstack-nova | 13:51 | |
| *** aarefiev is now known as aarefiev_afk | 13:51 | |
| sdague_ | efried: https://review.openstack.org/490031 is what I was thinking | 13:51 |
| sdague_ | which I actually want to merge | 13:51 |
| *** jistr|mtg is now known as jistr | 13:52 | |
| efried | sdague_ That's this one: https://review.openstack.org/#/c/488444/ | 13:54 |
| efried | sdague_ I don't think yours will work as-is, because you're not providing auth/session/adapter creds. | 13:54 |
| smatzek | mriedem, Recent changes to devstack setup related to cells have broke the affinity filters. This is breaking Trove's gate. My knowledge of cells, affinity filters, and the change are insufficient and could use pointers on how to get the filters working again. https://bugs.launchpad.net/devstack/+bug/1708171 | 13:55 |
| openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | 13:55 |
| *** yamamoto has quit IRC | 13:55 | |
| *** abalutoiu_ has joined #openstack-nova | 13:56 | |
| efried | sdague_ Note that https://review.openstack.org/#/c/488444/3/lib/nova@542 is where I get rid of api_servers by renaming it - I wanted to be able to see what the value would have been, which is why I didn't just remove the line. | 13:56 |
| sdague_ | efried: hmmm... how was glance working previously without those creds? | 13:56 |
| efried | sdague_ Because api_servers was setting explicit endpoint, so no discovery was done. | 13:56 |
| *** sdague has joined #openstack-nova | 13:56 | |
| sdague_ | ok, this seems weird | 13:57 |
| *** awaugama has quit IRC | 13:57 | |
| *** zhurong has joined #openstack-nova | 13:57 | |
| sdague_ | we need another set of creds to do discovery? | 13:57 |
| *** mamandle has joined #openstack-nova | 13:57 | |
| -openstackstatus- NOTICE: We have disable infracloud-vanilla due to the compute host running mirror.regionone.infracloud-vanilla.o.o being offline. Please recheck your failed jobs to schedule them to another cloud. | 13:57 | |
| sdague_ | that just feels wrong | 13:57 |
| sdague_ | I must be missing something | 13:57 |
| *** abalutoiu has quit IRC | 13:58 | |
| *** awaugama has joined #openstack-nova | 13:59 | |
| efried | sdague_ The service catalog ultimately lives in a ksa auth. | 13:59 |
| *** ekuris has quit IRC | 13:59 | |
| *** abalutoiu__ has joined #openstack-nova | 13:59 | |
| efried | sdague_ And the discovery process uses a Session to talk to the discovery endpoint to grab the version document. | 14:00 |
| sdague_ | ok, but that all happens in a user context right? | 14:01 |
| efried | sdague_ And the Adapter business is how we tell it what combo of service type, interface, etc. we're looking for. | 14:01 |
| sdague_ | can't we first order piggy back on that? | 14:01 |
| efried | mordred may be able to explain it better. | 14:01 |
| smatzek | dansmith, can you take a look at bug 1708171 and give me some pointers as you authored the transport_url change in devstack ^ ? | 14:01 |
| openstack | bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] https://launchpad.net/bugs/1708171 | 14:01 |
| mordred | efried, sdague_: reading scrollback | 14:02 |
| *** abalutoiu_ has quit IRC | 14:03 | |
| efried | sdague_ The user context may (or may not) have an appropriate auth in it. The util is set up to take that if it's present. | 14:03 |
| sdague_ | I'm all fine with a glance service user to get us past snapshot timeouts, but it seems weird to have the net change be here. "Hey, great feature, remove this one config line and add these 10, including passwords, to every system" feels less compelling | 14:03 |
| dansmith | smatzek: yep, known issue, you'll need to disable the multi-cell layout in your job if you need that | 14:03 |
| dansmith | mriedem: ^ | 14:03 |
| sdague_ | efried: doesn't have appropriate auth to find glance? | 14:03 |
| mordred | sdague_, efried: so - I think we can totally use the existing auth | 14:03 |
| efried | sdague_ I don't disagree the switchover feels burdensome for the glance case. | 14:04 |
| mordred | the thing we might need that's new is adapter params so an admin can *override* defaults | 14:04 |
| sdague_ | mordred: I'm fine with that | 14:04 |
| dansmith | smatzek: https://review.openstack.org/#/c/487478/ | 14:04 |
| mordred | (which I left some notes on as comments in the efried change) | 14:04 |
| *** nicolasbock_ has joined #openstack-nova | 14:04 | |
| sdague_ | I just want to be able to run without that | 14:04 |
| sdague_ | because, to the best of my knowledge every glance action happens within a user context in the current nova flows | 14:05 |
| mordred | yes. we should make sure that a) the admin can still provide api_servers and all works as before b) the admin can provide nothing and a proper setup will work c) the admin can provide override values for the adapter parameters and those will affect what discovery finds correctly | 14:05 |
| sdague_ | especially because screwing up 10 service auth config variables is actually *very* common | 14:05 |
| mordred | basically everyone gets them all wrong :) | 14:05 |
| *** awaugama has quit IRC | 14:05 | |
| sdague_ | we get a regular trickle of bugs in which are people having done that in neutron all the time | 14:05 |
| *** udesale__ has joined #openstack-nova | 14:06 | |
| sdague_ | i.e. the neutron part of the nova config | 14:06 |
| sdague_ | then 500s all over the place | 14:06 |
| mordred | efried: I'll take another pass through your patch with the above in mind - I may not have tracked the auth flow completely the last time | 14:06 |
| efried | It's worth pointing out that one of my motivations in this change set was to be as non-intrusive as possible to the code. E.g. in glance, isolate the change to where it finds the service URL, as opposed to doing a far-reaching rework to find a good auth to use, etc. | 14:06 |
| mordred | sdague_: incidentally - morgan had a question the other day which I don't think is terribly tractable at the moment... | 14:06 |
| larivee | /join #openstack-i18n | 14:06 |
| sdague_ | the reality is the only service I think Nova talks to outside of a user context is neutron | 14:07 |
| *** nicolasbock__ has joined #openstack-nova | 14:07 | |
| sdague_ | because it does background processing in a periodic to catch certain changes | 14:07 |
| *** nicolasbock has quit IRC | 14:07 | |
| mordred | sdague_: but currently the config sections are glance/cinder/ironic - but we've got this whole "use service-types" thing going on | 14:07 |
| sdague_ | mordred: sure | 14:07 |
| mordred | sdague_: maybe for the S cycle we should do a transition from glance to image - but certainly not for right now | 14:07 |
| sdague_ | I was just thrown with the idea that 10 lines of config needed to be added here | 14:07 |
| mordred | yah | 14:08 |
| sdague_ | mordred: yeh, that's low priority on the naming | 14:08 |
| sdague_ | just nice to have | 14:08 |
| mordred | I mean - there are a bunch of lines of potential config that need to be possible to set | 14:08 |
| mordred | but I agree, we need to make sure things work without them being set too | 14:08 |
| sdague_ | I just want to get us down to super minimal configs, because every line of config we can pull out of manditory setup docs makes it easier to get right | 14:08 |
| mordred | yup | 14:08 |
| smatzek | dansmith, thanks, so I can set singleconductor and it should work. The comment block says this option will be removed in the future. I assume the affinity filters will be fixed to not need it before that's removed? | 14:08 |
| mordred | 100% agree | 14:08 |
| mordred | and that's actually why we made some of the ksa changes - the ability to specify version ranges and lists of intefaces, for instance ... | 14:09 |
| *** sshwarts has quit IRC | 14:09 | |
| dansmith | smatzek: we have several things that we have to fix before we can remove that thing, yeah | 14:09 |
| *** andreas_s has quit IRC | 14:09 | |
| sdague_ | mordred: yep | 14:09 |
| efried | sdague_ So in this impl, a) api_servers is still supported, and takes precedence; b) if you set endpoint_override, you *shouldn't* need the other fields (I should edit https://review.openstack.org/#/c/489671/2/lib/nova to verify that) - that should be a straight swap. | 14:09 |
| mordred | is so that nova can say "I can handle v1-v2 of glance and prefer internal interface then public interface" | 14:09 |
| mordred | efried: we should supply defaults for adapter values so that it works ifyou don't set endpoint_override too | 14:10 |
| *** nicolasbock_ has quit IRC | 14:10 | |
| sdague_ | mordred: I'm all for this effort, just wanted to make sure I can land - https://review.openstack.org/#/c/490031/ | 14:10 |
| *** hongbin has joined #openstack-nova | 14:10 | |
| sdague_ | I legit want to land that as default devstack | 14:10 |
| mordred | sdague_: yes - what I'm saying is that we added a bunch of things to ksa so that you can | 14:10 |
| sdague_ | cool | 14:10 |
| mordred | sdague_: because otherwise it's not possible to actually express a good enough default value | 14:10 |
| sdague_ | ok, great | 14:11 |
| sdague_ | I might have misunderstood efried | 14:11 |
| efried | sdague_ That (getting glance auth from... somewhere else) was going to be a subsequent step in the process. | 14:11 |
| mordred | efried: we should be able to use what's there - it's already getting auth from somewhere else ... OH - I think I get whatyou're saying | 14:11 |
| efried | But it's the reason I coded the util to take an auth param. | 14:11 |
| *** crushil has quit IRC | 14:11 | |
| mordred | efried: the util function doesn't currently have 'pass auth in' plumbed in | 14:11 |
| efried | mordred Yeah, there's no auth in that method | 14:11 |
| *** awaugama has joined #openstack-nova | 14:12 | |
| *** crushil has joined #openstack-nova | 14:12 | |
| mordred | ok. I grok the whole end to end | 14:12 |
| efried | Exactly. And adding it in would have required some pretty far-reaching changes, which I wanted to put off for a followup. | 14:12 |
| efried | And focus this change on setting up and proving the viability of the util itself. | 14:12 |
| efried | Mm, I should clarify that with a TODO in the code. | 14:13 |
| *** slaweq has quit IRC | 14:14 | |
| *** larivee has left #openstack-nova | 14:14 | |
| *** mriedem1 has joined #openstack-nova | 14:14 | |
| mriedem1 | dansmith: smatzek: ack on https://bugs.launchpad.net/devstack/+bug/1708171 | 14:15 |
| openstack | Launchpad bug 1708171 in devstack "Nova Affinity filters no longer work" [Undecided,New] | 14:15 |
| *** slaweq has joined #openstack-nova | 14:15 | |
| mordred | efried: yes - I think a TODO will help - I have a couple of more comments - and also I see why it's complex to pass in the auth | 14:15 |
| mriedem1 | fwiw the affinity filter tests in tempest seem fine with the superconductor change | 14:15 |
| mriedem1 | maybe trove tests things differently | 14:15 |
| *** mriedem has quit IRC | 14:16 | |
| sdague_ | mriedem1: so you think that the wait for nova patch is needed back on ocata as well | 14:16 |
| mordred | sdague_: (for context, get_api_servers in nova/image/glance.py doesn't have the auth context atm since it's currently just dealing with config data - so efried is going to have to plumb that through in nova/image/glance.py | 14:16 |
| efried | mordred Roger, will respin today. Also ( sdague_ ) updated https://review.openstack.org/#/c/489671/ to see if *just* setting endpoint_override will work. I actually suspect it won't - I think we'll fail to build the Adapter. | 14:17 |
| smatzek | does nova have a negative test to ensure anti-affinity fails? (deploying 2 instances in 1 server group when we have 1 host) | 14:17 |
| mordred | efried: I agree with you - we will fail to do that - but I think that patch will be a good testcase of when the things have been plumbed all the way through | 14:17 |
| mriedem1 | smatzek: maybe in functional | 14:17 |
| mriedem1 | smatzek: the affinity filter test in tempest is here http://logs.openstack.org/32/489632/1/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/ae3cb6f/console.html#_2017-08-01_15_57_46_709238 | 14:17 |
| dansmith | smatzek: is that what you're doing and failing? | 14:18 |
| mriedem1 | i'm working on an anti-affinity multi-node test in tempest | 14:18 |
| smatzek | anti-affinity tests with 2 instances, 1 servergroup, 1 host, and verifying that the 2nd instance goes to error. The gate is failing because the second server goes to active. | 14:19 |
| mriedem1 | sdague: the wait for nova patch isn't working on all CIs | 14:19 |
| *** slaweq has quit IRC | 14:19 | |
| dansmith | smatzek: ack | 14:20 |
| *** prateek has joined #openstack-nova | 14:20 | |
| *** mriedem1 is now known as mriedem | 14:20 | |
| dansmith | mriedem: I really thought that a test like that requires the send-instance-info stuff in order to actually work | 14:20 |
| mriedem | smatzek: has anyone posted a patch for trove? | 14:20 |
| dansmith | I forget why all, something about when it refreshes the host list | 14:20 |
| *** mdnadeem has joined #openstack-nova | 14:20 | |
| sdague_ | mriedem: xenserver looks like the last one right? | 14:21 |
| sdague_ | I just pushed a patch to skip there | 14:21 |
| mriedem | dansmith: and i thought the scheduler / host manager would just pull the instance info if it wasn't getting updates | 14:21 |
| mriedem | sdague_: who knows how many others don't run on devstack changes which would be broken | 14:21 |
| smatzek | mriedem, no, I've been the only one digging into this gate failure and finally narrowed it down to that cells change yesterday late afternoon | 14:21 |
| dansmith | mriedem: but not fast enough or something | 14:21 |
| *** nicolasbock_ has joined #openstack-nova | 14:21 | |
| dansmith | smatzek: you have what you need now to set that conductor mode as a workaround though right? | 14:22 |
| mriedem | smatzek: is the trove job that's failing controlled through the trove repo? | 14:22 |
| mriedem | or something else? | 14:22 |
| *** eharney has joined #openstack-nova | 14:22 | |
| *** iceyao has joined #openstack-nova | 14:22 | |
| *** annegentle has joined #openstack-nova | 14:23 | |
| mriedem | nvm i see it | 14:23 |
| mriedem | i can push a patch in a minute | 14:23 |
| smatzek | I'm relatively new to Trove in the Trove channel have asked amrith the proper place to set the CELLSV2_SETUP=singleconductor env var. The trove job is controlled through the trove repo and its own devstack plugin.sh | 14:23 |
| mriedem | yeah i'm on it | 14:24 |
| smatzek | Unfortunately I'm going to be afk for the rest of the day starting in a bit. I'm going to take a stab and putting up a review that sets CELLSV2_SETUP where I think it needs to go and we'll see if the gate passes. | 14:24 |
| sdague_ | mriedem: it is possible, that's why I sent an email. | 14:24 |
| sdague_ | the xenserver folks popped up with their concerns | 14:25 |
| dansmith | smatzek: sounds like mriedem is doing that | 14:25 |
| *** belmoreira has joined #openstack-nova | 14:25 | |
| *** nicolasbock__ has quit IRC | 14:25 | |
| gibi | mriedem: just for your info I pushed functional test for resize to same host https://review.openstack.org/#/c/489973/ | 14:25 |
| smatzek | dansmith, mriedem thanks | 14:26 |
| cdent | gibi++ | 14:27 |
| dansmith | gibi: does that pass with the normal assertions on top of jaypipesjuryduty's current set? | 14:27 |
| dansmith | gibi: I found it super useful to make sure the other one passed on top, then rebase on master and comment out the assertions that failed | 14:27 |
| mriedem | gibi: yup i saw it, thanks | 14:27 |
| *** slaweq has joined #openstack-nova | 14:27 | |
| mriedem | jay's on jury duty?! | 14:27 |
| dansmith | again | 14:28 |
| dansmith | third time in four years or some such | 14:28 |
| mriedem | he is the youngest citizen in the county | 14:28 |
| dansmith | might be why they want him | 14:28 |
| cdent | dansmith: we don’t have a strategy for accounting for the allocations on same host yet, do we? | 14:29 |
| *** moshele has quit IRC | 14:29 | |
| gibi | dansmith: I haven't tried yet and I think it won't pass there | 14:29 |
| gibi | cdent: exactly | 14:29 |
| *** tikitavi has joined #openstack-nova | 14:29 | |
| gibi | so my commented out asserts there are only visionary things | 14:29 |
| gibi | feel free to comment it | 14:29 |
| dansmith | cdent: that's why I'm asking.. make sure it it passes on top of jay's current set, iterate on jay's set until it does, then flip the ordering | 14:29 |
| cdent | the current strategy is based on different resource provider ids | 14:30 |
| dansmith | I have a call now, biab | 14:30 |
| cdent | dansmith: yeah, but as far I know we haven’t got any ideas | 14:30 |
| gibi | I also have to go offline for 30 mins or so but I be back | 14:30 |
| cdent | me too, I haven’t eaten yet | 14:31 |
| mdbooth | mriedem: Incidentally, mind if I keep the functional test separate for backporting goodness: https://review.openstack.org/#/c/489654/ ? | 14:31 |
| *** slaweq has quit IRC | 14:32 | |
| *** cfriesen has joined #openstack-nova | 14:32 | |
| stephenfin | mriedem: Ahh, you're about. Anything super important that I missed in the last week? | 14:32 |
| mriedem | mdbooth: incidentally to what? | 14:33 |
| * stephenfin has cleared most of his mail except Gerrit stuff at this point | 14:33 | |
| mdbooth | mriedem: whatever you're doing right now? | 14:33 |
| *** cdent has quit IRC | 14:33 | |
| mdbooth | Your original comment. | 14:33 |
| mriedem | mdbooth: i just got online like 10 minutes ago | 14:33 |
| mriedem | so i'm not doing much | 14:33 |
| mriedem | stephenfin: the docs stuff | 14:33 |
| mdbooth | mriedem: I probably over use that word. | 14:34 |
| *** udesale has joined #openstack-nova | 14:34 | |
| mriedem | stephenfin: main issue is redirects were not done so when the pages moved things went to hell | 14:34 |
| mriedem | stephenfin: i'm tracking rc1 todos here https://etherpad.openstack.org/p/nova-pike-release-candidate-todo | 14:34 |
| mriedem | stephenfin: https://review.openstack.org/#/c/489650/ fixes a bunch of links, | 14:34 |
| mriedem | but some are still broken https://review.openstack.org/#/c/489650/ | 14:35 |
| stephenfin | mriedem: Yeah, looks like we got bogged down with those patches. So just that one change left? | 14:35 |
| mriedem | oops https://review.openstack.org/#/c/489641/ | 14:35 |
| *** udesale__ has quit IRC | 14:35 | |
| stephenfin | Oh, two so | 14:35 |
| stephenfin | I assume the docs patches still open aren't subject to the feature freeze? | 14:35 |
| mriedem | well, those are the 2 we know of | 14:35 |
| mriedem | no they aren't | 14:35 |
| stephenfin | ...but should now include additions to this htaccess file | 14:35 |
| mriedem | sdague and i are concerned about the massive one that changes the indexes | 14:35 |
| tikitavi | hi, I’m from ec2 team, we need some help | 14:36 |
| stephenfin | mriedem: index.rst ? | 14:36 |
| mriedem | this one https://review.openstack.org/#/c/478485/ | 14:36 |
| tikitavi | why extensions stopped working? in particular, extended_volumes.py stopped adding info about attached volumes to server list and server show? | 14:36 |
| mriedem | stephenfin: i also noted in https://review.openstack.org/#/c/477497/ that it's missing a bunch of nova-specific CLI guides that are now gone from the admin guide | 14:36 |
| mriedem | but are still in the stable/ocata branch for openstack-manuals | 14:36 |
| sdague_ | stephenfin: I'm more concerned about breaking up things like the policy doc, so that the only way to find information about nova policy is to already understand it, and know which subdoc to be in | 14:37 |
| stephenfin | mriedem: Yeah, I'm working with asettle on that this week | 14:37 |
| sdague_ | for stuff like that long pages that you can search in browser in are often more useful | 14:37 |
| mriedem | stephenfin: also, do we know if these got in from the user guide? https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-launch-instances.rst https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/user-guide/source/cli-delete-an-instance.rst ? | 14:37 |
| mriedem | stephenfin: maybe we push https://review.openstack.org/#/c/478485/ to the end and actually just work on getting the migration done for pike | 14:38 |
| stephenfin | sdague_: tbh, I'm not sure what's changed in that doc so I need to look at that. If you haven't left that comment on Gerrit yet, can you do so and I'll tackle it once I get there? :) | 14:38 |
| mriedem | and worry about completely re-arranging everything after that's done | 14:38 |
| sdague_ | stephenfin: I did | 14:38 |
| *** zhurong has quit IRC | 14:38 | |
| stephenfin | sdague_: Ta | 14:38 |
| stephenfin | mriedem: Yeah, a mass move and later refactor was my initial plan | 14:39 |
| mriedem | it also seems if we're going to re-arrange the home page for nova to be a certain form, that template should follow suit in the other projects, | 14:39 |
| *** kylek3h has quit IRC | 14:39 | |
| mriedem | and today it's not for cinder/glance/neutron either | 14:39 |
| mriedem | they are all different | 14:39 |
| *** ralonsoh_ has quit IRC | 14:39 | |
| mordred | efried: ok - I left a pile of additional comments | 14:40 |
| mordred | efried: don't hate me too much | 14:40 |
| efried | mordred Thanks! I will report back on hate-level once I've read 'em. | 14:40 |
| *** hrw has quit IRC | 14:41 | |
| stephenfin | OK, I'll work with asettle on that | 14:41 |
| mriedem | stephenfin: i can also summarize concerns in the ML quick | 14:41 |
| stephenfin | mriedem: As for user-guide etc., Chason Chan (no idea what his IRC nick is) has patches up to move those | 14:42 |
| * asettle puts on cape | 14:42 | |
| asettle | We will! | 14:42 |
| asettle | (what are we working on?) | 14:42 |
| mriedem | stephenfin: he has install guide and admin guide, not the user guide | 14:42 |
| stephenfin | that will also include the non-autogenerated cli stuff | 14:42 |
| mriedem | i'll post a quick brain dump to the ML | 14:42 |
| stephenfin | I'll do that one so | 14:42 |
| stephenfin | mriedem: Sounds good | 14:42 |
| stephenfin | asettle: The same thing we're always working on, asettle. Trying to take over the world | 14:43 |
| asettle | Of course, Pinky | 14:43 |
| stephenfin | Also, index.rst hijinks | 14:43 |
| asettle | That's less cool. | 14:44 |
| *** yangyapeng has joined #openstack-nova | 14:45 | |
| *** prateek has quit IRC | 14:45 | |
| *** dfisher has joined #openstack-nova | 14:45 | |
| *** hrw has joined #openstack-nova | 14:47 | |
| *** smatzek has quit IRC | 14:47 | |
| *** yangyapeng has quit IRC | 14:49 | |
| *** bauzas has quit IRC | 14:50 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 14:51 |
| *** yangyapeng has joined #openstack-nova | 14:53 | |
| *** armax_ has joined #openstack-nova | 14:53 | |
| *** armax has quit IRC | 14:53 | |
| *** armax_ is now known as armax | 14:53 | |
| mriedem | stephenfin: http://lists.openstack.org/pipermail/openstack-dev/2017-August/120482.html | 14:55 |
| *** yamamoto has joined #openstack-nova | 14:55 | |
| stephenfin | mriedem: Correct - *temporarily back | 14:56 |
| * stephenfin is off to Sziget next week 🙈 | 14:56 | |
| stephenfin | but let's skip over that | 14:56 |
| mriedem | my god | 14:56 |
| mriedem | how dare you enjoy being young and single | 14:57 |
| *** ratailor has joined #openstack-nova | 14:57 | |
| *** aditi_puntambeka has joined #openstack-nova | 14:57 | |
| *** mtanino has joined #openstack-nova | 14:58 | |
| *** kbaegis1 has joined #openstack-nova | 14:59 | |
| *** kbaegis has quit IRC | 14:59 | |
| *** felipemonteiro has joined #openstack-nova | 15:00 | |
| *** kbaegis1 has quit IRC | 15:00 | |
| *** yamamoto has quit IRC | 15:00 | |
| *** slaweq has joined #openstack-nova | 15:01 | |
| stephenfin | mriedem: Replied there. tl;dr: I agree with all point and will handle them this week | 15:01 |
| *** marst has joined #openstack-nova | 15:01 | |
| *** links has quit IRC | 15:01 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:04 | |
| ildikov | stephenfin: do the full experience with a tent, etc? | 15:04 |
| *** felipemonteiro has quit IRC | 15:04 | |
| stephenfin | ildikov: Yup, the full deal | 15:04 |
| *** felipemonteiro__ has joined #openstack-nova | 15:04 | |
| openstackgerrit | Gergely Csatari proposed openstack/nova master: Empty adminPass in Change Administrative Password https://review.openstack.org/490055 | 15:05 |
| ildikov | stephenfin: sounds cool! | 15:05 |
| stephenfin | Not looking forward to the 8am starts because the sun is up and you either get out of the tent and die from a hangover or slowly melt in the tent | 15:05 |
| ildikov | stephenfin: yeah, weather is a bit crazy over here | 15:05 |
| *** jistr is now known as jistr|mtg | 15:06 | |
| ildikov | stephenfin: the forecast says some storms for next week, but it's a whole week so they have no idea yet anyway :) | 15:06 |
| *** Oku_OS is now known as Oku_OS-away | 15:06 | |
| ildikov | stephenfin: but hopefully will cool down to manageable by next week regardless | 15:06 |
| *** felipemonteiro__ has quit IRC | 15:06 | |
| *** felipemonteiro__ has joined #openstack-nova | 15:07 | |
| *** aditi_puntambeka has quit IRC | 15:07 | |
| mordred | efried: ok - I've been nerd-sniped ... | 15:08 |
| *** mdnadeem has quit IRC | 15:08 | |
| stephenfin | ildikov: I hope so. No one wants rain but sub 30 temperatures wouldn't go amiss, heh | 15:08 |
| efried | mordred ? | 15:08 |
| *** felipemonteiro_ has quit IRC | 15:08 | |
| stephenfin | I'm sure I'll survive in any case though :) | 15:08 |
| ildikov | stephenfin: yeah, I didn't want to scare you with it, just wanted to be sure you're prepared :) | 15:08 |
| mordred | efried: I just wrote most of the auth followup patch as a result of reading through to make sure I understood where the issues were ;) | 15:09 |
| efried | mordred Figuring out how to get the auth into the glance client ... | 15:09 |
| mordred | yup. done. patch coming | 15:09 |
| sdague_ | mriedem: you misread my email about what's not being used in multinode | 15:09 |
| stephenfin | ildikov: Ha, cheers :) | 15:09 |
| efried | mordred Cool beans. | 15:09 |
| mordred | (this is the nerd-sniping that just hapepned) | 15:09 |
| mordred | efried: there is a test case that I didn't fix because it's a bit yuck and I don't know enough about the nova test cases | 15:09 |
| mriedem | sdague_: i realized after i sent that | 15:09 |
| ildikov | stephenfin: :) | 15:09 |
| mordred | (basically, there's a test case that doesn't have any auth context info already prepared) | 15:10 |
| *** coreywright has quit IRC | 15:10 | |
| mriedem | "it" with multiple nouns in the same sentence has been hurting me lately | 15:10 |
| sdague_ | heh, no problem | 15:10 |
| stephenfin | ildikov: I've learnt from my ill fated "Sure, you'll only need a light cotton sheet to sleep with" experience at Zurich last year. Turns out it gets colddd at night | 15:10 |
| sdague_ | mriedem: I also responded, email should hit soon | 15:10 |
| mriedem | i will prepare to respond to your response | 15:11 |
| mriedem | wow, has anyone realized that a single trove change runs 26 jobs? | 15:12 |
| *** iceyao has quit IRC | 15:13 | |
| *** thorst_afk has quit IRC | 15:14 | |
| ildikov | stephenfin: yeah, better to prepare as if you're lucky it gets down to 15 degrees Celsius at night the second half of next week | 15:15 |
| *** mamandle has quit IRC | 15:16 | |
| ildikov | stephenfin: which I like way better than 27 even if I have to sleep in a tent, but we all have different tastes :) | 15:16 |
| *** liverpooler has quit IRC | 15:16 | |
| *** liverpooler has joined #openstack-nova | 15:19 | |
| *** penick has joined #openstack-nova | 15:19 | |
| *** armax has quit IRC | 15:19 | |
| *** aditi_puntambeka has joined #openstack-nova | 15:20 | |
| *** nicolasbock__ has joined #openstack-nova | 15:21 | |
| *** mkucia has quit IRC | 15:21 | |
| openstackgerrit | Monty Taylor proposed openstack/nova master: WIP Use auth from context for glance api servers https://review.openstack.org/490057 | 15:22 |
| mordred | efried: ^^ | 15:22 |
| *** psachin has quit IRC | 15:22 | |
| efried | mordred Ack | 15:22 |
| *** coreywright has joined #openstack-nova | 15:22 | |
| mordred | efried: I didn't fix nova/tests/unit/api/openstack/compute/test_images.py because of the auth context thing | 15:23 |
| efried | mordred You gonna? | 15:23 |
| *** nicolasbock_ has quit IRC | 15:24 | |
| *** ratailor has quit IRC | 15:24 | |
| *** cdent has joined #openstack-nova | 15:25 | |
| *** iceyao has joined #openstack-nova | 15:25 | |
| *** cdent has quit IRC | 15:26 | |
| *** vks1 has joined #openstack-nova | 15:26 | |
| *** cdent has joined #openstack-nova | 15:27 | |
| mordred | efried: not this moment - I hit my nerd-snipe limit for the day - feel free to use/takeover/ignore that patch - or if I get stuck on other things later today and need a different mental task I may try poking again | 15:27 |
| *** annegentle has quit IRC | 15:28 | |
| efried | mordred Rgr. My comments may be for myself, then :) | 15:28 |
| *** cdent_ has joined #openstack-nova | 15:28 | |
| *** armax has joined #openstack-nova | 15:28 | |
| mordred | efried: woot! | 15:29 |
| *** cdent has quit IRC | 15:29 | |
| *** cdent_ is now known as cdent | 15:29 | |
| *** iceyao has quit IRC | 15:30 | |
| *** ralonsoh has joined #openstack-nova | 15:31 | |
| *** armax has quit IRC | 15:33 | |
| *** liverpooler has quit IRC | 15:34 | |
| *** damien_r1 has joined #openstack-nova | 15:34 | |
| *** damien_r has quit IRC | 15:34 | |
| *** udesale has quit IRC | 15:34 | |
| *** Sukhdev has joined #openstack-nova | 15:35 | |
| *** liverpooler has joined #openstack-nova | 15:36 | |
| *** tylerderosagrund has joined #openstack-nova | 15:36 | |
| *** annegentle has joined #openstack-nova | 15:36 | |
| *** abalutoiu__ has quit IRC | 15:38 | |
| mriedem | cdent: coming back to your question about the plan for fixing https://bugs.launchpad.net/nova/+bug/1707252 i don't know of one | 15:38 |
| openstack | Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed] | 15:38 |
| mriedem | but it's not part of https://review.openstack.org/#/c/488510/ | 15:39 |
| cdent | mriedem: yeah | 15:39 |
| cdent | mriedem: the ideas I’ve heard batted around include: | 15:40 |
| mriedem | the issue is here https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L196 | 15:40 |
| cdent | adding migration uuids and adding the old allocation using the migration uuid, setting the consumer_uuid to the new allocation | 15:41 |
| *** awaugama has quit IRC | 15:41 | |
| cdent | simply doubling in the move to same case and worrying about shared resource providers some other time (they complicate things rather a lot) | 15:41 |
| mriedem | the migration uuid thing can't happen in pike | 15:42 |
| cdent | I’m a bit confused on the real status of “shared providers”. Do you know? | 15:42 |
| cdent | we can returned shared providers, but I’m not sure we create them | 15:43 |
| *** gyee has joined #openstack-nova | 15:43 | |
| cdent | if we don’t create them, then the simple doubling thing _might_ be made to work | 15:43 |
| mriedem | we don't use them in any dsvm ci job | 15:43 |
| mriedem | we don't create them in nova no | 15:43 |
| mriedem | something like the ceph ci job would create a shared resource provider for disk | 15:43 |
| *** sdague_ has quit IRC | 15:43 | |
| mriedem | if we had a multinode ceph job, we could have 2 compute onde RPs and a single shared ceph disk RP | 15:43 |
| cdent | yes, but we’d first need a resource tracker that was properly shared provider aware | 15:44 |
| mriedem | that's on the etherpad for queens ptg to discuss | 15:44 |
| mriedem | right, which we know it's not | 15:44 |
| mriedem | which is https://bugs.launchpad.net/nova/+bug/1707256 | 15:44 |
| cdent | that suggests, then, that we can proceed with a non-sharing solution for now | 15:44 |
| openstack | Launchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client does not account for shared resource providers" [High,Confirmed] - Assigned to Jay Pipes (jaypipes) | 15:44 |
| edleafe | So in a migration, would a shared resource, like disk, also be doubled? | 15:45 |
| cdent | edleafe: long term, yes, but we don’t currently have them | 15:45 |
| mriedem | it oculd | 15:45 |
| mriedem | *could | 15:45 |
| *** itlinux has quit IRC | 15:45 | |
| mriedem | if i'm resizing my disk from 20GB to 40GB, i need to account for that new disk allocatoin | 15:45 |
| cdent | edleafe: and there’s also the cross case where a target host shares disk via two providers | 15:45 |
| edleafe | mriedem: right, but would we account for the old 20GB and the new 40GB? Or just the larger? | 15:46 |
| cdent | so you could migrate from: host a, disk x to host b, disk y OR host b, disk x | 15:46 |
| *** mtanino_ has joined #openstack-nova | 15:46 | |
| edleafe | cdent: yeah, where the compute moves, but the disk doesn't | 15:46 |
| cdent | edleafe: on the hangout earlier it was decided 60 | 15:46 |
| cdent | that is, potentially over allocate, do the simple math | 15:47 |
| edleafe | cdent: yeah, that's probably easier to implement | 15:47 |
| cdent | in some situations it would be wrong, but not wrong broken | 15:47 |
| edleafe | too many use cases overlapping | 15:47 |
| *** ralonsoh has quit IRC | 15:47 | |
| cdent | quite | 15:47 |
| *** ralonsoh has joined #openstack-nova | 15:47 | |
| *** damien_r1 has left #openstack-nova | 15:47 | |
| *** sridharg has quit IRC | 15:48 | |
| *** aditi_puntambeka has quit IRC | 15:49 | |
| cdent | mriedem, edleafe I could spike a doubling for resize to same host, but it wouldn’t be able to get started until about 4 hours from now | 15:49 |
| dansmith | cdent: mriedem right I think for pike we have to double the allocation and then just gracefully subtract our old_flavor from the allocation if we're the only provider | 15:49 |
| dansmith | and not worry about the shared stuff for the moment | 15:49 |
| *** chyka has joined #openstack-nova | 15:50 | |
| *** trinaths has joined #openstack-nova | 15:50 | |
| mriedem | there was a hangout earlier today? | 15:51 |
| cdent | no earlier in the week | 15:51 |
| mriedem | oh, there were a few :) | 15:51 |
| cdent | (at least that’s what I was referring to) | 15:51 |
| *** rmart04 has quit IRC | 15:53 | |
| mriedem | so when we resize to the same host, this code is ending up with new_rp_uuids as empty, right? https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L200 | 15:53 |
| mriedem | assuming no shared storage provider | 15:53 |
| mriedem | which makes this False | 15:54 |
| mriedem | if alloc['resource_provider']['uuid'] in new_rp_uuids: | 15:54 |
| mriedem | so the dest rp uuid, which is the same as the source rp uuid, not get added in | 15:54 |
| mriedem | for double the fun | 15:54 |
| cdent | yeah, that’s my recollection | 15:54 |
| mriedem | we could tell if this is a compute resource provider by checking for the VCPU resource class in the allocations | 15:55 |
| *** bauzas has joined #openstack-nova | 15:55 | |
| mriedem | if that helps narrow things a bit | 15:55 |
| *** belmoreira has quit IRC | 15:55 | |
| cdent | mriedem: that’s one of the places we ended up on that aforementioned hangout | 15:56 |
| cdent | but if we are discounting shared providers, for now, we don’t need to worry about that | 15:56 |
| *** chyka_ has joined #openstack-nova | 15:56 | |
| *** awaugama has joined #openstack-nova | 15:56 | |
| *** Apoorva has joined #openstack-nova | 15:56 | |
| mriedem | doesn't seem like it would be hard to keep accounting for shared providers | 15:56 |
| mriedem | if new_rp_uuids is empty, we know something is afoot | 15:56 |
| mriedem | i think i need to check out gibi | 15:57 |
| * gibi is here and reading | 15:57 | |
| mriedem | gibi's new test for resize to same host first | 15:57 |
| cdent | a) I’m thinkin in terms of trying to iterate, b) I think the code in jay’s stack is incorrect for shared providers anyway, based on some of the things dansmith said earlier in the week | 15:57 |
| mriedem | and then could start playing with a change that builds on that | 15:57 |
| dansmith | cdent: I assume so as well | 15:58 |
| gibi | mriedem: that test needs a bit of update based on the above discussion. I used assert(max(old, new), usage) type of asserts but you agreed about old+new as I see | 15:58 |
| *** links has joined #openstack-nova | 15:58 | |
| cdent | gibi: you’re near the end of your day, yeah? | 15:59 |
| *** chyka has quit IRC | 15:59 | |
| *** randomha1k has quit IRC | 15:59 | |
| *** catintheroof has joined #openstack-nova | 15:59 | |
| gibi | cdent: yeah, and on a train with spotty conenction | 15:59 |
| cdent | then I won’t say “maybe you should do the spike” :0 | 16:00 |
| gibi | at least not today | 16:00 |
| gibi | :) | 16:00 |
| *** josecastroleon has quit IRC | 16:00 | |
| *** catintheroof has quit IRC | 16:00 | |
| mriedem | so we know the scheduler report client doesn't know about shared storage providers, and will trample your shared storage provider allocations saying any disk consumed is local to the compute node provider | 16:00 |
| dansmith | mriedem: we could probably help by landing his two bottom patches at least | 16:00 |
| *** catintheroof has joined #openstack-nova | 16:00 | |
| dansmith | mriedem: I +2d the bottom one this morning and can look at the next one now | 16:01 |
| dansmith | mriedem: correct | 16:01 |
| *** randomha1k has joined #openstack-nova | 16:01 | |
| mriedem | so before we can say shared storage is supported, we have to fix that, and all of your computes have to be upgraded to the level that has that fix | 16:01 |
| *** mlavalle has joined #openstack-nova | 16:01 | |
| mriedem | meanwhile we're not doing a min service version check in the scheduler to account for that | 16:01 |
| bauzas | are folks discussing of https://bugs.launchpad.net/nova/+bug/1707256 ? | 16:02 |
| openstack | Launchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client does not account for shared resource providers" [High,Confirmed] - Assigned to Jay Pipes (jaypipes) | 16:02 |
| *** mlavalle has left #openstack-nova | 16:02 | |
| mriedem | we're discussing all things | 16:02 |
| bauzas | all things | 16:02 |
| bauzas | :) | 16:02 |
| mriedem | if we say, f it, shared storage isn't supported in pike, then we just fix the resize to same host thing by doubling allocations in the scheduler, right? | 16:03 |
| *** sdague has quit IRC | 16:03 | |
| *** gyee has quit IRC | 16:03 | |
| *** gaoyan has joined #openstack-nova | 16:03 | |
| dansmith | mriedem: yeah and ideally gracefully subtracting in the compute when done | 16:03 |
| dansmith | mriedem: we should be as graceful as possible though so we don't screw up queens nodes that may do it right | 16:04 |
| mriedem | on confirm resize? | 16:04 |
| *** isantosp_ has quit IRC | 16:04 | |
| dansmith | yeah | 16:04 |
| mriedem | hurts | 16:04 |
| mriedem | ma | 16:04 |
| mriedem | brain | 16:04 |
| * cdent passes out ibuprofen | 16:05 | |
| dansmith | this is oregon, | 16:06 |
| dansmith | we have better things at our disposal | 16:07 |
| cdent | my stash is cashed | 16:07 |
| *** lpetrut has quit IRC | 16:07 | |
| dansmith | mriedem: so I can start looking at making it do the right thing on the confirm if you want | 16:08 |
| *** rcernin has quit IRC | 16:08 | |
| dansmith | I wish his top patch didn't marry the two things he's fixing together | 16:09 |
| dansmith | the ocata compat and the resize_confirm fix | 16:09 |
| dansmith | I'll put mine on top at least | 16:09 |
| mriedem | i'll check out the bottom change that fixes PUT to overwrite all allocations - already did the other day and it made sense, seems simple, | 16:10 |
| bauzas | I just +Wd it | 16:11 |
| mriedem | also need to check out gibi's resize to same host tests, and then i was going to tinker with some of the code in https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L200 | 16:11 |
| *** zz9pzza has quit IRC | 16:12 | |
| cfriesen | mriedem: would we be looking at backporting any shared storage accounting fix back to pike? or only fixing in Q? (or does that sort of depend on what the fix looks like?) | 16:12 |
| mriedem | it depends, but also not sure since the scheduler isn't making any distinction based on the version of the compute serivce | 16:13 |
| mriedem | *Service | 16:13 |
| mriedem | so it kind of sucks to say, 'well make sure you have this fix and everything is upgraded first' | 16:13 |
| cdent | dansmith: are you doing just the undoubling, or also the doubling? | 16:13 |
| dansmith | cdent: we're already doubling right? | 16:13 |
| *** pcaruana has quit IRC | 16:14 | |
| mriedem | we only double if >1 provider | 16:14 |
| *** slaweq has quit IRC | 16:14 | |
| cdent | we don’t have info to double | 16:14 |
| cdent | yeah, what mriedem says | 16:14 |
| dansmith | ah okay | 16:14 |
| mriedem | we can figure out the resize to same host case | 16:14 |
| dansmith | then yeah I'll look at that too | 16:14 |
| mriedem | in the scheduler, that's what i was going to poke at | 16:14 |
| *** slaweq has joined #openstack-nova | 16:14 | |
| dansmith | cdent: can you look at my comment on the top one just now? | 16:17 |
| * cdent goes to look | 16:17 | |
| *** gaoyan has quit IRC | 16:17 | |
| *** gaoyan has joined #openstack-nova | 16:17 | |
| cdent | oh yeah that. every single time I read that chunk of code I get confused | 16:18 |
| cdent | they are different structures | 16:18 |
| *** armax has joined #openstack-nova | 16:18 | |
| cdent | i’m not sure how we ended up there | 16:18 |
| dansmith | cdent: they're supposed to be different you mean? | 16:19 |
| dansmith | GET vs PUT? | 16:19 |
| *** slaweq has quit IRC | 16:19 | |
| cdent | yeah | 16:19 |
| dansmith | how is that restful? | 16:19 |
| *** kbaegis has joined #openstack-nova | 16:19 | |
| cdent | it isn’t very | 16:19 |
| dansmith | okay, glad we agree on that :) | 16:19 |
| cdent | but there was a disagreement between you/me and jay at some point | 16:19 |
| *** lyan has quit IRC | 16:20 | |
| cdent | you and i wanted the GET to return a dict because it made processing the response easy | 16:20 |
| *** gyee has joined #openstack-nova | 16:20 | |
| cdent | (this was at the end of last summer or so) | 16:20 |
| *** acormier_ has joined #openstack-nova | 16:20 | |
| dansmith | not about that, that I know of, but maybe it ended up with a disparity as a side effect? | 16:20 |
| cdent | side effect of? | 16:20 |
| *** yassine has quit IRC | 16:20 | |
| dansmith | meaning, I would never argue for GET/PUT to be different structures, so I'm wondering if we just never made PUT match the changed GET or something and nobody realized? | 16:21 |
| cdent | oh, possibly? | 16:21 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 16:22 |
| dansmith | mriedem: you're working on the scheduler doubling or what? I kinda need to do it to test any changes I make for the un-doubling, so I might as well do it unless you've already started | 16:22 |
| cdent | but I also think there was some dislike (I don’t recall why) of the rp uuid being a key in the POST | 16:22 |
| dansmith | cdent: that's pretty disappointing | 16:22 |
| dansmith | regardless of how we ended up here | 16:22 |
| cdent | indeed | 16:22 |
| *** jistr|mtg is now known as jistr | 16:22 | |
| cdent | there are quite a lot of disappointments | 16:22 |
| mriedem | dansmith: was just starting with a unit test for the scheduler | 16:23 |
| *** mpaolino has quit IRC | 16:23 | |
| *** gaoyan has quit IRC | 16:23 | |
| dansmith | mriedem: okay I guess I'll hold off them | 16:23 |
| dansmith | *then | 16:23 |
| mriedem | i'll throw up the wip shortly | 16:23 |
| dansmith | I'm really kinda confused about this doubling anyway | 16:24 |
| *** acormier has quit IRC | 16:24 | |
| dansmith | I guess it's the attempt to account for shared storage that makes this complicated, | 16:25 |
| dansmith | and which avoids doubling for same-host | 16:25 |
| *** gaoyan has joined #openstack-nova | 16:25 | |
| cdent | dansmith: yes | 16:25 |
| mriedem | yup | 16:26 |
| mriedem | "Remove any allocations against resource providers that are | 16:26 |
| mriedem | # already allocated against on the source host (like shared storage | 16:26 |
| mriedem | # providers)" | 16:26 |
| mriedem | so i guess the intention was to specifically not double up shared storage | 16:26 |
| mriedem | on a mov | 16:26 |
| mriedem | *move | 16:26 |
| dansmith | which is wrong anyway | 16:26 |
| dansmith | for certain types of shared storage | 16:27 |
| mriedem | seemed like the right idea at the time?! | 16:27 |
| dansmith | it's not wrong for a volume, but is wrong for a compute node using ceph | 16:27 |
| mriedem | which was 72 hours ago? | 16:27 |
| *** markus_z has quit IRC | 16:27 | |
| dansmith | I know, looking at this, I swear I've never seen it before, but I'm pretty sure I +2d it not long ago | 16:27 |
| *** lucasagomes is now known as lucas-afk | 16:28 | |
| *** itlinux has joined #openstack-nova | 16:28 | |
| *** sree has quit IRC | 16:28 | |
| mriedem | ha yeah same here | 16:28 |
| mriedem | https://review.openstack.org/#/c/487589/ | 16:28 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 16:29 |
| *** hoonetorg has quit IRC | 16:30 | |
| mriedem | dansmith: melwitt: skip the cells v2 meeting yeah? | 16:30 |
| dansmith | oh heh, I meant to say I have a conflict today anyway | 16:31 |
| dansmith | so yeah | 16:31 |
| *** baoli has quit IRC | 16:32 | |
| *** baoli has joined #openstack-nova | 16:33 | |
| *** gaoyan has quit IRC | 16:37 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 16:39 |
| *** gaoyan has joined #openstack-nova | 16:39 | |
| *** sdague has joined #openstack-nova | 16:40 | |
| *** randomha1k has quit IRC | 16:40 | |
| *** damien_r has joined #openstack-nova | 16:42 | |
| *** damien_r has quit IRC | 16:42 | |
| *** sree has joined #openstack-nova | 16:42 | |
| cdent | dansmith: in honor of your pain, I created a bug, which we can decided to care about or not: https://bugs.launchpad.net/nova/+bug/1708205 | 16:42 |
| openstack | Launchpad bug 1708205 in OpenStack Compute (nova) "placement allocation representation asymetric on PUT and GET" [Low,Confirmed] | 16:42 |
| dansmith | oh I care | 16:42 |
| dansmith | I care bigly | 16:42 |
| *** hoonetorg has joined #openstack-nova | 16:42 | |
| cdent | tremendous | 16:42 |
| *** gaoyan has quit IRC | 16:44 | |
| *** gaoyan has joined #openstack-nova | 16:46 | |
| *** gaoyan has quit IRC | 16:46 | |
| mriedem | ok got the patch | 16:47 |
| mriedem | pushing soon | 16:47 |
| mriedem | prepare | 16:47 |
| * cdent summons the midwife | 16:49 | |
| *** phuongnh has joined #openstack-nova | 16:50 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 16:50 |
| mriedem | dansmith: cdent: ^ it's not the prettiest, and it doesn't account for a case that we'd have shared storage | 16:50 |
| * cdent looks | 16:51 | |
| jangutter | I've got a newbie question here regarding attaching/detaching SR-IOV ports in Nova. I presume it's not going to be "just something simple" and would need work in Queens at the very least? (ref: https://review.openstack.org/#/c/139910/ ) | 16:51 |
| *** lyan has joined #openstack-nova | 16:51 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | 16:51 |
| dansmith | ack I will build on it when I'm done with my thing | 16:52 |
| cdent | mriedem: If were on a non-shared some host resize, there will only be one allocation, so perhaps we can avoid some of the looping, which makes my brain explore? | 16:54 |
| cdent | s/some/same/ | 16:54 |
| mriedem | dansmith: ok, i have a todo question in the test - which is i'm not sure if we even need the compute to adjust things, e.g. if the new alloc for vcpu is smaller than the current alloc, wouldn't we just not sum those? in other words, shouldn't the new allocations when we're done be the max of the current + new? | 16:54 |
| jangutter | Has someone taken a look at attaching/detaching SR-IOV ports recently? If not, I can try to take a stab at it. | 16:54 |
| mriedem | jangutter: no | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 16:54 |
| cdent | mriedem: for the duration of the resize we need room for both vms, don’t we? | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rewrite index page https://review.openstack.org/490088 | 16:54 |
| dansmith | mriedem: I don't think we should be that discriminating | 16:55 |
| mriedem | cdent: even if it's on the same host? | 16:55 |
| dansmith | cdent: his point is that vcpus are somewhat ephemeral, not a strict quantity of things that go away | 16:55 |
| *** thorst_afk has joined #openstack-nova | 16:55 | |
| dansmith | except for pinning, but... | 16:55 |
| mriedem | cdent: yes i tried a few ways to do this w/o the looping and stuff and it hurt my brain | 16:55 |
| dansmith | I think the scheduler should stay out of deciding what it does and doesn't know about the hypervisor | 16:55 |
| mriedem | if i'm resizing from 1 cpu to 2 cpu on the same host, my total allocation on that host should be 2, not 3 | 16:55 |
| *** yangyapeng has quit IRC | 16:56 | |
| mriedem | yeah it's fair to say we don't want to bake all that logic into the scheduler | 16:56 |
| mriedem | just seems a bit wasteful | 16:56 |
| mriedem | until you confirm and we fix thigns | 16:56 |
| cdent | and for vmware and powervm and maybe others, the resize may not be on the same host, just the same n-cpu? | 16:56 |
| *** yangyapeng has joined #openstack-nova | 16:56 | |
| mriedem | powervm should be fine | 16:56 |
| dansmith | cdent: we're accounting based on node though | 16:56 |
| mriedem | vmware is the freak | 16:56 |
| dansmith | still, | 16:57 |
| mriedem | but yeah it's per node | 16:57 |
| *** randomha1k has joined #openstack-nova | 16:57 | |
| dansmith | for a hypervisor with very concrete resources, we don't know that some resources add and others don't | 16:57 |
| dansmith | and, | 16:57 |
| dansmith | we're not using double the disk in most cases, | 16:57 |
| openstackgerrit | Merged openstack/nova master: Accept any scheduler driver entrypoint https://review.openstack.org/484828 | 16:57 |
| dansmith | but it depends on the config of the node, | 16:57 |
| dansmith | so we should just grab double anyway for everything, IMHO | 16:58 |
| mriedem | bauzas: are you going to backport https://review.openstack.org/#/c/484828/ to ocata or should i? | 16:58 |
| *** rajathagasthya has joined #openstack-nova | 16:58 | |
| *** crushil has quit IRC | 16:58 | |
| bauzas | mriedem: I can | 16:58 |
| mriedem | dansmith: ok - my main concern was that ocata computes aren't going to fix the double allocation to remove the old flavor | 16:58 |
| dansmith | mriedem: they're just going to trample all over it anyway | 16:58 |
| mriedem | that's true | 16:59 |
| mriedem | gd ocata computes | 16:59 |
| mriedem | they're aholes | 16:59 |
| bauzas | mriedem: looks like I need to resolve a conflict but I'll do | 16:59 |
| dansmith | I know.. greedy punks | 16:59 |
| *** derekh has quit IRC | 16:59 | |
| mriedem | i will commence to lunching then | 16:59 |
| cdent | i first read that as octopus computes | 16:59 |
| * cdent doesn’t want one of those | 16:59 | |
| *** ralonsoh has quit IRC | 17:00 | |
| *** dtantsur is now known as dtantsur|afk | 17:00 | |
| *** yangyapeng has quit IRC | 17:00 | |
| * cdent dinners | 17:03 | |
| *** cdent has quit IRC | 17:03 | |
| *** aditi_puntambeka has joined #openstack-nova | 17:05 | |
| *** sree has quit IRC | 17:06 | |
| *** tesseract has quit IRC | 17:07 | |
| *** crushil has joined #openstack-nova | 17:09 | |
| melwitt | mriedem, dansmith: was on a call, ack to skipping cells meeting | 17:10 |
| *** dikonoor has joined #openstack-nova | 17:11 | |
| *** harlowja has joined #openstack-nova | 17:13 | |
| *** efoley_ has quit IRC | 17:15 | |
| openstackgerrit | Steve Noyes proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | 17:16 |
| *** aditi_puntambeka has quit IRC | 17:16 | |
| *** randomha1k has quit IRC | 17:18 | |
| openstackgerrit | Steve Noyes proposed openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | 17:21 |
| *** prateek has joined #openstack-nova | 17:22 | |
| *** tikitavi has quit IRC | 17:24 | |
| dansmith | mriedem: so that patch is not on top of jay's, nor under gibi's | 17:24 |
| dansmith | I'm thinking I will put this on top of gibi's | 17:26 |
| *** prateek has quit IRC | 17:28 | |
| *** rajathagasthya has quit IRC | 17:30 | |
| *** sambetts is now known as sambetts|afk | 17:30 | |
| openstackgerrit | Merged openstack/nova master: placement: remove existing allocs when set allocs https://review.openstack.org/489273 | 17:34 |
| dansmith | heh, found another problem | 17:36 |
| dansmith | at least with fake virt, we're reporting min_unit=1, max_unit=1, step_size=1, which means we can't claim 2 vcpus for the single instance | 17:37 |
| dansmith | which could be a problem in real life, | 17:37 |
| dansmith | if configured so that one instance can't take more than 25% of a host, but a resize would technically take 50% in a single allocation | 17:37 |
| dansmith | which would be solved by my migration uuid thing | 17:37 |
| *** mingyu has quit IRC | 17:39 | |
| *** baoli has quit IRC | 17:39 | |
| bauzas | dansmith: mriedem: just saw the above discussion about whether the scheduler should know the virt logic and the move ops, tbc MHO is * NOOOOOOO * | 17:40 |
| bauzas | because we have conductors for that | 17:40 |
| bauzas | not for virt stuff, but at least knowing whether it's a move or a boot | 17:40 |
| *** baoli has joined #openstack-nova | 17:40 | |
| bauzas | scheduler should just give you a destination, whether it's for a move or a boot, that's it | 17:41 |
| *** dtp has joined #openstack-nova | 17:41 | |
| bauzas | if we need more reasons why we need to do that, I don't want to hit your wounts, but that's why we first tried to claim by the conductors... | 17:42 |
| bauzas | anyway | 17:42 |
| * bauzas goes off | 17:42 | |
| dansmith | oye, the fake driver is only reporting one vcpu for some reason | 17:47 |
| *** jamesden_ has joined #openstack-nova | 17:47 | |
| *** jamesdenton has quit IRC | 17:47 | |
| *** rajathagasthya has joined #openstack-nova | 17:48 | |
| *** mmehan has joined #openstack-nova | 17:49 | |
| *** awaugama has quit IRC | 17:49 | |
| *** randomha1k has joined #openstack-nova | 17:49 | |
| *** mingyu has joined #openstack-nova | 17:49 | |
| *** awaugama has joined #openstack-nova | 17:50 | |
| dansmith | ohh | 17:50 |
| *** gszasz has quit IRC | 17:53 | |
| *** vks1 has quit IRC | 17:53 | |
| *** krtaylor has quit IRC | 17:53 | |
| mriedem | dansmith: yeah hadn't gotten to the resize functional tests yet | 17:56 |
| *** slaweq has joined #openstack-nova | 17:56 | |
| *** yangyapeng has joined #openstack-nova | 17:56 | |
| *** yamahata has joined #openstack-nova | 17:57 | |
| mriedem | dansmith: i can start rebasing my change on top of gibi's test, unless i need to hold off for something | 18:00 |
| *** yangyapeng has quit IRC | 18:01 | |
| dansmith | mriedem: already done | 18:02 |
| dansmith | I didn't realize we used SmallFakeDriver everywhere, which only has one vcpu | 18:02 |
| dansmith | so I got past that, | 18:02 |
| dansmith | but we will likely have obscure issues with that elsewhere | 18:02 |
| dansmith | like people won't be able to resize to the same host if they have to dip into overcommit for vcpu, which will make no sense to them | 18:03 |
| mriedem | dansmith: ok done locally or...? | 18:04 |
| mriedem | because i don't see those rebased | 18:04 |
| dansmith | yes | 18:04 |
| mriedem | ok | 18:04 |
| mriedem | i think i'll start building on https://review.openstack.org/#/c/490085/ with handling shared storage in resize to same host, see how terrible that looks | 18:04 |
| mriedem | won't push anything though | 18:04 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | 18:07 |
| *** corey_ has joined #openstack-nova | 18:07 | |
| *** corey_ is now known as Guest42667 | 18:07 | |
| *** cleong has quit IRC | 18:07 | |
| *** phuongnh has quit IRC | 18:10 | |
| *** jpena is now known as jpena|off | 18:10 | |
| *** mingyu has quit IRC | 18:11 | |
| *** baoli has quit IRC | 18:13 | |
| *** cdent has joined #openstack-nova | 18:18 | |
| *** kfox1111_ is now known as kfox1111 | 18:19 | |
| *** baoli has joined #openstack-nova | 18:20 | |
| *** dikonoor has quit IRC | 18:23 | |
| *** stvnoyes has quit IRC | 18:24 | |
| dansmith | mriedem: so I'm making your patch work on top of gibi's tests (or rather adjusting gibi's test for what you fix) | 18:27 |
| dansmith | and then jay's can go on top of that, with a fix for the compute node part | 18:27 |
| *** links has quit IRC | 18:31 | |
| mriedem | alright | 18:31 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 18:31 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 18:31 |
| dansmith | mriedem: ^ | 18:31 |
| *** acormier has joined #openstack-nova | 18:31 | |
| mgagne | We used to rely on a custom image props in our out-of-tree virt driver in Nova kilo. With the oslo.versionedobjects migration, I found (testing against Mitaka) that ImageMetaProps has a list of hardcoded supported properties and I can no longer inject or read our custom property. What's the best way for us to use our custom prop in Nova? | 18:32 |
| *** adisky__ has quit IRC | 18:32 | |
| *** acormier_ has quit IRC | 18:32 | |
| *** slaweq has quit IRC | 18:36 | |
| *** slaweq has joined #openstack-nova | 18:37 | |
| *** jaypipesjuryduty is now known as jaypipes | 18:37 | |
| mriedem | mgagne: upstream your image meta prop | 18:37 |
| jaypipes | dansmith, mriedem, cdent: hi folks. just back from jury duty. | 18:37 |
| mgagne | okay =( | 18:37 |
| cdent | jaypipes: did you manage to get excluded? | 18:38 |
| jaypipes | cdent: no. just sat there for 7 hours in the jury pool and the judge released everyone who hadn't been selected to a jury | 18:38 |
| cdent | fun! | 18:38 |
| mriedem | jaypipes: did you at least make some friends? | 18:39 |
| jaypipes | dansmith, mriedem, cdent: someone please fill me in on the latest? I was in the process of fixing up the remaining little test failures on the 1707071 bug patch. do I need to pull fresh? did anyone push any more changes on those patches? | 18:39 |
| dansmith | jaypipes: I'm working on it right now | 18:40 |
| jaypipes | dansmith: ok thanks Dan. I'll wait on a pull then. | 18:40 |
| jaypipes | dansmith: anything I should be aware of or be working on? | 18:40 |
| mriedem | gibi has a patch for resize to same host | 18:41 |
| mriedem | w/o shared storage | 18:41 |
| jaypipes | k | 18:41 |
| mriedem | i've got a wip for accounting for resize to same host w/o shared storage in the scheduler | 18:41 |
| mriedem | dan just rebased those to be lined up | 18:41 |
| *** slaweq has quit IRC | 18:41 | |
| mriedem | he's working on rebase your change on top of mine | 18:41 |
| mriedem | i'm working on handling shared storage with resize to same host | 18:41 |
| jaypipes | ok, will wait for further instructions. | 18:42 |
| dansmith | I guess jay's patch didn't even pass the gibi tests from yesterday? | 18:42 |
| dansmith | I should have fixed that first I guess, because now it fails everything | 18:42 |
| jaypipes | dansmith: yes, they were. | 18:43 |
| dansmith | https://review.openstack.org/#/c/488510/12 | 18:43 |
| dansmith | not according to that, afaict | 18:43 |
| dansmith | oh, nm, | 18:43 |
| jaypipes | dansmith: gibi's tests work. | 18:43 |
| dansmith | that's the ironic/ocata whatever | 18:43 |
| jaypipes | dansmith: there was an ocata ironic failure I was looking into | 18:43 |
| dansmith | hmm, well, not sure why they don't here then | 18:43 |
| *** krtaylor has joined #openstack-nova | 18:45 | |
| *** dfisher has left #openstack-nova | 18:46 | |
| cdent | jaypipes: I made a bug for that thing you just commented on, so we have it for future reference: https://bugs.launchpad.net/nova/+bug/1708205 | 18:47 |
| openstack | Launchpad bug 1708205 in OpenStack Compute (nova) "placement allocation representation asymetric on PUT and GET" [Low,Confirmed] | 18:47 |
| mriedem | problem in https://review.openstack.org/#/c/490085/2/nova/tests/functional/test_servers.py | 18:47 |
| jaypipes | cdent: cool. | 18:47 |
| *** lyan has quit IRC | 18:48 | |
| dansmith | jaypipes: there's another problem, btw | 18:48 |
| dansmith | jaypipes: let's say you have a compute node with 4 vcpus total, overcommit ratio of 16 like default | 18:48 |
| dansmith | jaypipes: and you have an instance there with three vcpus | 18:49 |
| dansmith | and want to do a same-host resize | 18:49 |
| dansmith | you'll fail to get a doubled allocation because the single-instance allocation on that host will be >max_unit | 18:49 |
| jaypipes | dansmith: ooh, yeah, certainly didn't think of that. nice catch... | 18:49 |
| dansmith | also solved by my migration claim idea, fwiw | 18:50 |
| jaypipes | dansmith: however, there would still be two separate allocation records, though, right? which would individually be <max_unit, yes? | 18:50 |
| dansmith | no | 18:50 |
| mriedem | no same host | 18:50 |
| dansmith | single instance | 18:50 |
| mriedem | same rp | 18:50 |
| dansmith | against the same rp | 18:51 |
| jaypipes | oh, yeah, doh. | 18:51 |
| dansmith | right | 18:51 |
| jaypipes | yup, sorry. | 18:51 |
| jaypipes | dansmith: yes, your migration UUID idea would indeed solve that. ++ | 18:51 |
| cdent | what’s the chances of leap know to a migration claim? is that completely off the table? | 18:51 |
| jaypipes | cdent: leap know? | 18:51 |
| dansmith | leap know what now? | 18:51 |
| cdent | sorry, homophones | 18:52 |
| jaypipes | who you calling a homophone? | 18:52 |
| cdent | leap now to use migration claims, instead of trying to work around all these constraints | 18:52 |
| dansmith | it's a lot of change | 18:52 |
| dansmith | to scheduling and the db schema, etc | 18:53 |
| jaypipes | cdent: yeah, what dansmith said. | 18:53 |
| dansmith | not that we're not making lots of change to fix these issues, but.. it's scary(er) | 18:53 |
| jaypipes | plus there's the issue of still needing to handle ocata migrations... | 18:53 |
| jaypipes | ocata to pike migrations, that is. | 18:53 |
| cdent | yeah, dansmith, I’m not entirely sure which is really scarier | 18:53 |
| jaypipes | dansmith, mriedem: do we still need a patch testing that when ocata computes are in the mix, that pike computes continue to behave badly? if so, I can begin work on that. | 18:54 |
| mriedem | i'll ask for the 7th time, couldn't we handle the ocata->pike issue by not claiming in the scheduler until everything in the compute is working the way we want and restrict the claim in the scheduler until the computes are all >=pike? | 18:54 |
| mriedem | feel free to just say no again :) | 18:54 |
| dansmith | mriedem: and again, it doesn't change anything if we don't claim first | 18:55 |
| dansmith | ocata will still stomp on everything | 18:55 |
| mriedem | ocata won't stomp if it's not ocata | 18:55 |
| mriedem | there is nothing to stomp if we don't put down the stompables until >=pike | 18:55 |
| jaypipes | dansmith: plus we've already released Pike software that always does claiming in the scheduler... | 18:55 |
| dansmith | mriedem: they'll stomp on pike things | 18:56 |
| jaypipes | dansmith: so it would be a pain to have to know whether the software installed does or does not do claims | 18:56 |
| dansmith | mriedem: but again, it doesn't solve anything to not do a thing that gets stomped on anyway | 18:56 |
| mriedem | dansmith: aren't we trying to fix the computes to not stomp on things the scheduler is doing? | 18:56 |
| *** annegentle has quit IRC | 18:56 | |
| mriedem | like, make the compute aware of shared storage allocations | 18:57 |
| dansmith | mriedem: not really | 18:57 |
| dansmith | not ocata computes | 18:57 |
| mriedem | right, pike computes | 18:57 |
| jaypipes | mriedem: no. we're trying to fix pikes to not stomp on things that other pike computes may have been doing. | 18:57 |
| dansmith | right | 18:57 |
| dansmith | while still being compatible with ocata computes | 18:57 |
| dansmith | and those are somewhat at odds | 18:57 |
| jaypipes | ya | 18:57 |
| *** annegentle has joined #openstack-nova | 18:58 | |
| mriedem | ok, so again, it's fixing latent bugs which we didn't care about until those latent bugs affected scheduling decisions, which they do now | 18:59 |
| dansmith | they affect ocata scheduler too | 18:59 |
| dansmith | claiming in the scheduler is after the existing decision gets made on the data we're stomping on | 19:00 |
| dansmith | and the stomping just makes placement think there is more room than there is, | 19:00 |
| dansmith | so not claiming doesn't make anything easier | 19:00 |
| dansmith | it makes it less likely to be right, but that's not really useful | 19:00 |
| mriedem | excluding shared storage providers, the computes are eventually consistent aren't they? | 19:01 |
| mriedem | i'll stop asking questions since these are things i've gone in circles on for 2+ weeks now | 19:01 |
| dansmith | I've about got all three patches working together | 19:04 |
| dansmith | which is scary, because I think I was supposed to modify the compute side code to make this work and I haven't done that yet | 19:04 |
| dansmith | (nor do I remember what that was anymore) | 19:05 |
| mriedem | are you fixing the issue in my patch? | 19:05 |
| dansmith | which issue? | 19:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 19:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Handle shared storage allocations when resize to same host https://review.openstack.org/490159 | 19:06 |
| mriedem | self.assertFlavorMatcheAllocation | 19:06 |
| mriedem | ^ fixed | 19:06 |
| dansmith | well, I'm just going to push over top of yours | 19:07 |
| dansmith | but yeah I had fixed that | 19:07 |
| dansmith | ooh, nice I'm getting a 500 from placement now | 19:09 |
| *** nicolasbock__ has quit IRC | 19:10 | |
| cdent | nice work dansmith | 19:10 |
| dansmith | I'll fix this undoubling thing and then push to let you placement peeps look at the 500 | 19:11 |
| cdent | yeah, I can look at that when there are some details | 19:11 |
| *** lyan has joined #openstack-nova | 19:12 | |
| *** mingyu has joined #openstack-nova | 19:12 | |
| mriedem | i'm really confused because https://review.openstack.org/490159 is passing the test i added w/o any code changes to handle it | 19:12 |
| *** edmondsw has quit IRC | 19:15 | |
| dansmith | mriedem: passing what? it failed gibi's same host tests when I put it on top | 19:15 |
| *** lyan has quit IRC | 19:15 | |
| mriedem | this is the shared storage + resize to same host one | 19:16 |
| mriedem | https://review.openstack.org/#/c/490159/ adds a unit test for that and i expected it to fail | 19:16 |
| *** mingyu has quit IRC | 19:16 | |
| *** edmondsw has joined #openstack-nova | 19:17 | |
| *** mingyu has joined #openstack-nova | 19:17 | |
| mriedem | oh no, i know why it's passing | 19:18 |
| mriedem | heh | 19:18 |
| mriedem | yup | 19:18 |
| mriedem | i'll just squash those changes together | 19:19 |
| *** dfisher has joined #openstack-nova | 19:19 | |
| *** lyan has joined #openstack-nova | 19:19 | |
| dansmith | mriedem: can you hold off? | 19:19 |
| mriedem | cdent: ^ is why i'm looping the allocations | 19:19 |
| mriedem | rather than assuming there is 1 | 19:19 |
| dansmith | I have a bunch of cuts against all three of these patches | 19:19 |
| mriedem | dansmith: like, deep cuts? | 19:19 |
| dansmith | gashes | 19:19 |
| dansmith | with rusty blades | 19:19 |
| mriedem | i meant like https://www.youtube.com/watch?v=KCdKBHdPz30 | 19:20 |
| mriedem | deep cuts | 19:20 |
| dansmith | heh | 19:20 |
| dansmith | nice one | 19:20 |
| mriedem | btw, fagen is forced to tour again https://qz.com/1041397/steely-dans-donald-fagen-is-back-on-tour-the-result-of-nobody-buying-music-albums-anymore/ | 19:21 |
| *** edmondsw has quit IRC | 19:21 | |
| mriedem | streaming music has broken him | 19:21 |
| mriedem | he should probably talk to a financial advisor about diversifying his portfolio | 19:21 |
| melwitt | lol | 19:22 |
| *** mingyu has quit IRC | 19:22 | |
| dfisher | is there a known issue with calling nova.context.get_admin_context() from within a virt driver? http://paste.openstack.org/show/617308/ | 19:22 |
| *** mingyu has joined #openstack-nova | 19:22 | |
| * cdent has clearly misunderstood mriedem’s age, all this time | 19:23 | |
| melwitt | did not expect that article to mention Ronnie James Dio | 19:23 |
| mriedem | hologram dio | 19:23 |
| mriedem | terrible | 19:23 |
| *** edmondsw has joined #openstack-nova | 19:23 | |
| melwitt | I know, like, seriously? | 19:23 |
| mriedem | i was at a sabbath reunion show once and dio scolded the audience and threatened to cut the show and leave if they didn't settle down | 19:23 |
| dansmith | cripes | 19:24 |
| dansmith | jaypipes: how do we get the old_flavor if we're here: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L460-L460 ? | 19:24 |
| mriedem | dansmith: isn't old_flavor stored on the instance? | 19:25 |
| dansmith | mriedem: well, I would assume not at this point since we go to great lengths to look it up on L443 | 19:25 |
| mriedem | maybe that predates the old_flavor being stored on the instance? | 19:26 |
| dansmith | no, | 19:26 |
| dansmith | we set it to None before we call this here: | 19:26 |
| dansmith | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L3501-L3521 | 19:26 |
| dansmith | son of a goat | 19:26 |
| mriedem | is it stashed on the migration record? | 19:26 |
| dansmith | no, only the id | 19:26 |
| mriedem | ha | 19:26 |
| mriedem | # NOTE(danms): delete stashed migration information | 19:26 |
| dansmith | wait | 19:27 |
| mriedem | BUT WHY?! | 19:27 |
| dansmith | we pass old_instance_type for instance_type, | 19:27 |
| dansmith | but clearly that can be None sometimes | 19:27 |
| dansmith | but I don't know when | 19:27 |
| *** edmondsw has quit IRC | 19:27 | |
| dansmith | maybe if we're doing a migrate? | 19:27 |
| mriedem | probably yeah | 19:27 |
| mriedem | no old/new if it's not a resize | 19:27 |
| dansmith | so maybe I can use old_instance_type or instance.flavor | 19:27 |
| mriedem | f yes you can | 19:27 |
| *** edmondsw has joined #openstack-nova | 19:29 | |
| mriedem | dfisher: if you're using a recent devstack, it's running in superconductor mode which means you can't do retries from the compute | 19:30 |
| mriedem | it can't upcall to the api db to get instance group info | 19:31 |
| mriedem | if you're doing some crazy crap in the oracle virt driver, then you're on your own | 19:31 |
| *** slaweq has joined #openstack-nova | 19:31 | |
| dfisher | i'm really not doing anything crazy … I don't think. | 19:32 |
| dansmith | dfisher: well, except for that one crazy thing | 19:32 |
| dfisher | like, I'm using nova.api.metadata.password.convert_password | 19:33 |
| dfisher | which takes a context | 19:33 |
| *** edmondsw has quit IRC | 19:33 | |
| *** edmondsw has joined #openstack-nova | 19:35 | |
| dfisher | dansmith: ssssh. don't tell them </loud whisper> | 19:35 |
| *** rajathagasthya has quit IRC | 19:35 | |
| mriedem | dfisher: if you're looking at trunk code, build_instances in the conductor is only hit if (1) youre' using cellsv1 which you shouldn't be or (2) if the compute is doing a retry of a build | 19:37 |
| dansmith | okay I think I'm doing the right thing, and now I'm failing because of that 500 from placement | 19:37 |
| dansmith | so I'll push this up and let someone else look at that in parallel | 19:37 |
| mriedem | dfisher: so assuming you're hitting (2), if you're using devstack in the default superconductor mode, the retry isn't going to work | 19:37 |
| dfisher | hmm. ok. | 19:38 |
| mriedem | dfisher: b/c you're hitting the cell conductor which doesn't have access to the scheduler | 19:38 |
| mriedem | or api db for that matter | 19:38 |
| *** rajathagasthya has joined #openstack-nova | 19:38 | |
| *** mingyu has quit IRC | 19:38 | |
| mriedem | if you're not using devstack, or you're using a devstack that's older than like 1 week, then *shrug* | 19:38 |
| dfisher | no, devstack is from yesterday or maybe Monday | 19:38 |
| melwitt | yeah. the error is saying the node handling the database read doesn't have the API database connection configured in nova.conf | 19:38 |
| openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 19:39 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 19:39 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 19:39 |
| *** awaugama has quit IRC | 19:39 | |
| mriedem | dfisher: if you need retries, then set CELLSV2_SETUP=singleconductor in your stackrc | 19:39 |
| dansmith | cdent: jaypipes^ check out 'tox -efunctional ServerMovingTest' in the top patch there ^ | 19:39 |
| mriedem | and restackaroo | 19:39 |
| *** edmondsw has quit IRC | 19:39 | |
| dfisher | very cool. thank you | 19:39 |
| dfisher | i really appreciate it | 19:39 |
| mriedem | dfisher: also https://docs.openstack.org/nova/latest/user/cellsv2_layout.html | 19:39 |
| mriedem | digest that | 19:39 |
| * dfisher belches | 19:39 | |
| *** edmondsw has joined #openstack-nova | 19:41 | |
| jaypipes | dansmith: gotcha. looking at it now. | 19:42 |
| dansmith | also note the FIXME about cheating | 19:42 |
| jaypipes | k | 19:42 |
| dansmith | that's clearly just a hack to do some math, but needs to be integrated in the right place | 19:42 |
| *** mingyu has joined #openstack-nova | 19:42 | |
| dansmith | specifically this here: consumer_id = allocs[0].consumer_id | 19:43 |
| dansmith | IndexError: list index out of range | 19:43 |
| dansmith | maybe that somehow means I'm passing an empty list of things to set and placement isn't having ti? | 19:43 |
| dansmith | *it | 19:43 |
| jaypipes | not sure, still looking :) | 19:44 |
| dansmith | oh, well, heh | 19:44 |
| dansmith | I'm definitely not adding it back in | 19:45 |
| dansmith | duh | 19:45 |
| *** edmondsw has quit IRC | 19:45 | |
| dansmith | heyyyyy | 19:46 |
| dansmith | woo | 19:46 |
| dansmith | fix comin' | 19:46 |
| openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 19:46 |
| dansmith | that passes & | 19:46 |
| *** edmondsw has joined #openstack-nova | 19:46 | |
| jaypipes | k, lemme pull that | 19:46 |
| dansmith | but, that means that placement will 500 if you pass it an empty set of allocations (/cc cdent) | 19:47 |
| *** edmondsw has quit IRC | 19:47 | |
| cdent | noted | 19:47 |
| *** edmondsw has joined #openstack-nova | 19:47 | |
| jaypipes | cdent: bug please if you would | 19:47 |
| jaypipes | that should be an easy fix. | 19:48 |
| cdent | yeah, just finishing something else first | 19:48 |
| *** mingyu has quit IRC | 19:48 | |
| *** pchavva has quit IRC | 19:49 | |
| *** edmondsw has quit IRC | 19:50 | |
| *** randomha1k has quit IRC | 19:51 | |
| *** acormier has quit IRC | 19:52 | |
| jaypipes | dansmith: am I kosher to work on fixing that ocata to pike func test failure or are you still working on that patch? | 19:52 |
| *** acormier has joined #openstack-nova | 19:52 | |
| jaypipes | dansmith: and yes, I confirm that latest patch is passing tox -efunctional ServerMovingTest | 19:52 |
| dansmith | jaypipes: yeah I hadn't looked at that yet, so I'll pass the baton back to you | 19:52 |
| dansmith | jaypipes: someone has to fix up the "cheating" part anyway | 19:52 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | 19:53 |
| jaypipes | dansmith: did I miss the cheating part? where is that? | 19:53 |
| jaypipes | oh, I see it now. | 19:53 |
| jaypipes | nm | 19:53 |
| dansmith | jaypipes: https://review.openstack.org/#/c/488510/14/nova/scheduler/client/report.py@1102 | 19:53 |
| jaypipes | yup, see it now. | 19:54 |
| dansmith | probably need to have RT pass in the allocationy thing to subtract in there | 19:54 |
| *** randomha1k has joined #openstack-nova | 19:54 | |
| dansmith | instead of the report client knowing about flavors | 19:54 |
| dansmith | and I figure there's some shared code to take a flavor and turn it into allocations, but I didn't go looking | 19:54 |
| dansmith | but anyway, cheating aside, that passes all the forwards and backwards single and multi host tests for pike-and-pike | 19:55 |
| jaypipes | dansmith: that something you want to fix up in this patch or do you want to tackle that in a followup? | 19:55 |
| dansmith | which is... good | 19:55 |
| *** annegentle has quit IRC | 19:55 | |
| dansmith | jaypipes: don't you think it's way too ugly to merge as is? | 19:55 |
| jaypipes | dansmith: ugly sure, but harmless :) | 19:55 |
| dansmith | jaypipes: i.e. it doesn't account for ephemeral_gb | 19:55 |
| dansmith | maybe that's an easy fix, adding ephemeral and swap in there | 19:55 |
| *** annegentle has joined #openstack-nova | 19:56 | |
| jaypipes | dansmith: sure, just remember swap is in MB :) | 19:56 |
| dansmith | but if we can make it work right, I'm okay leaving the ugly there | 19:56 |
| dansmith | jaypipes: you can just fix that bit while you're fixing the other test right? | 19:56 |
| jaypipes | dansmith: happy to. was just checking to make sure I wouldn't clobber anything you were working on. | 19:56 |
| dansmith | it's all yours | 19:57 |
| jaypipes | kk | 19:57 |
| *** yangyapeng has joined #openstack-nova | 19:57 | |
| jaypipes | dansmith: you want to keep those LOG.info() lines in remove_provider_from_instance_allocation()? | 19:58 |
| mriedem | this deals with the various disk things https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L154 | 19:58 |
| mriedem | so throw that into a utility method if we need | 19:58 |
| jaypipes | mriedem: yup, on it. | 19:58 |
| mriedem | dansmith: you added this back in https://review.openstack.org/#/c/490085/4/nova/tests/functional/test_servers.py@1300 | 19:59 |
| mriedem | can i work on cleaning up https://review.openstack.org/#/c/490085/ and squashing in https://review.openstack.org/#/c/490159/ ? | 20:00 |
| mriedem | this is quite a turducken we've gotten ourselves into | 20:00 |
| *** Guest42667 has quit IRC | 20:00 | |
| *** rajathagasthya has quit IRC | 20:00 | |
| *** edmondsw has joined #openstack-nova | 20:01 | |
| dansmith | mriedem: ah yeah I was swapping things around several times when I couldn't get his patch working on master, sorry about that | 20:01 |
| mriedem | so i can fix? | 20:01 |
| dansmith | mriedem: jay's working on top of it, but if he's willing to rebase | 20:01 |
| dansmith | your call | 20:02 |
| *** rajathagasthya has joined #openstack-nova | 20:02 | |
| dansmith | his call | 20:02 |
| *** yangyapeng has quit IRC | 20:02 | |
| mriedem | what i'm changing won't impact jay probably | 20:02 |
| dansmith | aye, just the rebase | 20:02 |
| mriedem | so i'm going for it | 20:02 |
| jaypipes | I can rebase no worries. | 20:02 |
| dansmith | that was not fun | 20:02 |
| dansmith | I need to do something more fun now | 20:02 |
| jaypipes | dansmith: I'm going to leave those LOG.info() lines in remove_provider_from_allocations() but just pep8 em up. | 20:02 |
| dansmith | like smash my finger in a door or something | 20:02 |
| mriedem | i have a fun story | 20:02 |
| mriedem | unrelated to this | 20:02 |
| dansmith | jaypipes: ack | 20:03 |
| mriedem | you know how horizon lets you edit a flavor? | 20:03 |
| openstackgerrit | Sean Dague proposed openstack/nova master: add top 404 redirect https://review.openstack.org/490181 | 20:03 |
| openstackgerrit | Sean Dague proposed openstack/nova master: sort redirectmatch lines https://review.openstack.org/490182 | 20:03 |
| dansmith | mriedem: "edit" yes | 20:03 |
| *** hoonetorg has quit IRC | 20:03 | |
| mriedem | we've got a customer that hit a fun scenario where they'd change the flavor on an instance in horizon, and it would show the old flavor even though it was deleted b/c of read_deleted='yes' from the nova db | 20:04 |
| mriedem | then they upgraded, or something, and same scenario, but this time flavor 404 b/c no soft delete in the api db, | 20:04 |
| mriedem | which we know about, | 20:04 |
| mriedem | but horizon.... | 20:04 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Update RT aggregate map less frequently https://review.openstack.org/489633 | 20:04 |
| mriedem | fixed in 2.47, | 20:05 |
| mriedem | but this is like mitaka -> pike | 20:05 |
| mriedem | good times | 20:05 |
| sdague | mriedem: right, that was the crux of the fight around whether the original flavor id was included in the embedded flavor structure | 20:05 |
| sdague | because of that feature in horizon | 20:05 |
| *** edmondsw has quit IRC | 20:05 | |
| mriedem | we should have added a cell0 for flavors :) | 20:05 |
| mriedem | nova_flavors db | 20:05 |
| sdague | which lets people dig themself a hole to fall in after they shoot themselves in the foot | 20:06 |
| mriedem | when you just can't get enough dbs | 20:06 |
| sdague | I'm telling you, db per project | 20:06 |
| mriedem | sounds like a fun game | 20:06 |
| *** edmondsw has joined #openstack-nova | 20:07 | |
| *** krtaylor has quit IRC | 20:08 | |
| *** edmondsw has quit IRC | 20:09 | |
| openstackgerrit | Rawan Herzallah proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 20:10 |
| *** edmondsw has joined #openstack-nova | 20:10 | |
| mriedem | so i think Kevin_Zheng is going to send something to the ML asking about changing that behavior in horizon | 20:12 |
| dansmith | meaning removing flavor editing? | 20:12 |
| cdent | dansmith, jaypipes, edleafe: https://bugs.launchpad.net/nova/+bug/1708260 you want to provide an opinion on whether the response should be a 400 or a lukewarm success (you asked me to do nothing, I have successfully done nothing) | 20:12 |
| openstack | Launchpad bug 1708260 in OpenStack Compute (nova) "Sending empty allocations list on a PUT /allocations/{consumer_uuid} results in 500" [Medium,Confirmed] - Assigned to Chris Dent (cdent) | 20:12 |
| mriedem | at least disabling the ability to do it on the instance record itself, | 20:12 |
| mriedem | like, there is a panel showing the instance and it's flavor and the ability to edit it right there on the instance record | 20:13 |
| mriedem | which we know doesn't actually resize the instance or anythign | 20:13 |
| dansmith | cdent: 200 vs 400 seems like something I'd have an uninformed gut opinion on, which you would immediately whip out some document to refute | 20:13 |
| dansmith | cdent: so... no, I don't care :) | 20:13 |
| dansmith | cdent: 500 seems wrong | 20:13 |
| cdent | dansmith: aw dan, I’m trying to be inclusive. | 20:13 |
| mriedem | 409 | 20:14 |
| mriedem | always 409 | 20:14 |
| * cdent finds a book to throw at mriedem | 20:14 | |
| dansmith | cdent: I'm being tongue-in-cheeky | 20:14 |
| cdent | me too | 20:14 |
| cdent | I’m going to call is a schema violation for now | 20:15 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to Instance object https://review.openstack.org/489408 | 20:15 |
| edleafe | If the allocations body is required, 409 seems like the correct response | 20:15 |
| *** liverpooler has quit IRC | 20:16 | |
| mriedem | ha | 20:16 |
| mriedem | wouldn't it be a 400? | 20:16 |
| * cdent frowns at edleafe | 20:16 | |
| cdent | that is, like, so wrong | 20:16 |
| cdent | it would be 400 | 20:16 |
| mriedem | 402? | 20:16 |
| mriedem | i'll take your nothing and charge you for it | 20:16 |
| * dansmith watches edleafe fall into the trap he just avoided | 20:17 | |
| mriedem | haha | 20:17 |
| edleafe | sorry, fat finger | 20:17 |
| edleafe | yes, 400 | 20:17 |
| dansmith | i'll hit the brakes and he'll fly right by | 20:17 |
| * edleafe vows to do his hand-stretching excercises more religiously | 20:18 | |
| sdague | 418 and call it a day :) | 20:18 |
| cdent | so anyway, edleafe: you’ve hit crux of my query: is it required? | 20:18 |
| *** slaweq has quit IRC | 20:18 | |
| cdent | one could argue that sending an empty allocations list is some kind of delete | 20:18 |
| cdent | but that just feels icky | 20:18 |
| cdent | so I’m going to change the schema | 20:18 |
| *** slaweq has joined #openstack-nova | 20:19 | |
| sdague | cdent: 400 on that and call it a schema violation seems right to me | 20:19 |
| edleafe | cdent: Having an allocation list is required. Populating it with allocations is not | 20:19 |
| sdague | the point of doing that is to help the user write a better application by erroring on them when they might have typoed a thing | 20:19 |
| edleafe | So yeah, an empty list == delete | 20:19 |
| dansmith | mriedem: come on I get no love for the 80s movie reference? | 20:19 |
| *** aditipuntambekar has joined #openstack-nova | 20:20 | |
| sdague | edleafe: a delete should be explicit, and not mistaken for a forgotten initialization | 20:20 |
| *** hoonetorg has joined #openstack-nova | 20:20 | |
| *** randomha1k has quit IRC | 20:20 | |
| edleafe | sdague: agree. So a schema change seems like the best solution | 20:21 |
| mriedem | dansmith: was coding | 20:21 |
| mriedem | oh top gun | 20:21 |
| mriedem | i see | 20:21 |
| mriedem | good one mav | 20:21 |
| cdent | edleafe, sdague: it is likely that before we added the consumer_id to the object, it operated as an unintentional delete | 20:22 |
| dansmith | by the way, I'll be taking up a collection to buy one of these for mriedem: https://chummytees.com/products/i-speak-fluent-movie-quotes-t-shirt-hoodie-tank-top | 20:23 |
| *** vishwana_ has joined #openstack-nova | 20:23 | |
| *** krtaylor has joined #openstack-nova | 20:23 | |
| * melwitt donates | 20:23 | |
| *** slaweq has quit IRC | 20:23 | |
| cfriesen | I was waiting for "she's real fine my 409" | 20:24 |
| melwitt | he can wear it when he gives summit presentations | 20:24 |
| *** slaweq has joined #openstack-nova | 20:25 | |
| mriedem | cfriesen: isn't that beach boys ala 60s? | 20:25 |
| mriedem | too old | 20:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Require at least one allocation when PUT https://review.openstack.org/490195 | 20:25 |
| mriedem | ha yup, 1962 | 20:26 |
| cdent | what’s your limit? | 20:26 |
| mriedem | well i got the reference | 20:26 |
| mriedem | so limit=01 | 20:26 |
| mriedem | *-1 | 20:26 |
| *** vishwanathj has quit IRC | 20:26 | |
| mriedem | the beach boys: playing at a state fair near you! | 20:26 |
| *** https_GK1wmSU has joined #openstack-nova | 20:27 | |
| cdent | is the doubling stack still in flux or is some of it ready for review? | 20:28 |
| mriedem | i'm doing it | 20:28 |
| cdent | is that yes or no? | 20:28 |
| mriedem | no | 20:29 |
| dansmith | not in flux or not reviewable? | 20:29 |
| dansmith | I think it's worth a review in its current state, personally | 20:29 |
| mriedem | i'm about to push, | 20:29 |
| mriedem | but should i rebase https://review.openstack.org/#/c/488510/ on top? i suppose i should | 20:29 |
| *** https_GK1wmSU has left #openstack-nova | 20:30 | |
| mriedem | here it comes | 20:31 |
| *** dikonoor has joined #openstack-nova | 20:31 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 20:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 20:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 20:31 |
| sdague | mriedem: has anyone attempted to figure out the toc on the sidebar thing? | 20:33 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | 20:34 |
| sdague | because otherwise I'll look at that tomorrow, because the current toc is driving me nuts | 20:34 |
| sdague | also, this is our top current 404 - https://review.openstack.org/#/c/490181/ | 20:34 |
| sdague | 56 fails on that in the last 2 days | 20:34 |
| mriedem | sdague: nope | 20:35 |
| *** aditipuntambekar has quit IRC | 20:35 | |
| mriedem | sdague: you should rebase on top of https://review.openstack.org/#/c/489641/ | 20:36 |
| sdague | mriedem: sure | 20:36 |
| openstackgerrit | Sean Dague proposed openstack/nova master: add top 404 redirect https://review.openstack.org/490181 | 20:37 |
| mriedem | you have to re-sort now too | 20:39 |
| openstackgerrit | Sean Dague proposed openstack/nova master: sort redirectmatch lines https://review.openstack.org/490182 | 20:39 |
| sdague | yep, done | 20:39 |
| mriedem | hot | 20:39 |
| mriedem | dog | 20:39 |
| *** acormier_ has joined #openstack-nova | 20:40 | |
| *** acormier_ has quit IRC | 20:40 | |
| mriedem | all approved | 20:40 |
| *** acormier_ has joined #openstack-nova | 20:41 | |
| sdague | \o/ | 20:41 |
| *** efoley has joined #openstack-nova | 20:42 | |
| *** acormier has quit IRC | 20:44 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: fix rpc broken rst comment https://review.openstack.org/490197 | 20:46 |
| sdague | mriedem: one more for you | 20:46 |
| sdague | smallest patch possible | 20:46 |
| mriedem | show it shows up | 20:46 |
| mriedem | -10 | 20:46 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: fix rpc broken rst comment https://review.openstack.org/490197 | 20:47 |
| *** gyee has quit IRC | 20:48 | |
| *** mingyu has joined #openstack-nova | 20:49 | |
| *** krtaylor has quit IRC | 20:50 | |
| *** mingyu has quit IRC | 20:54 | |
| mriedem | dansmith: there is a false sense of passing in the resize to same host tests https://review.openstack.org/#/c/490085/5/nova/tests/functional/test_servers.py@1300 | 20:55 |
| dansmith | mriedem: I made that change in response to a failure | 20:56 |
| dansmith | that was actually one of the first changes I made, in response to a fail | 20:58 |
| dansmith | lemme pull it out and see | 20:58 |
| dansmith | but if it's dead code, then it's not affecting the passingness right? | 20:59 |
| dansmith | yeah, removing it locally doesn't affect anything | 21:00 |
| *** acormier has joined #openstack-nova | 21:00 | |
| *** lucasxu has quit IRC | 21:00 | |
| mriedem | i'll fix | 21:00 |
| dansmith | maybe I misread the trace and modified it in the wrong place initially | 21:00 |
| dansmith | sorry | 21:00 |
| mriedem | i need to tweak the bottom test change | 21:00 |
| *** tylerderosagrund has quit IRC | 21:00 | |
| dansmith | can I blame the break-up of everything into non-unified test cases? | 21:01 |
| mriedem | sure | 21:01 |
| mriedem | blame it on the rain | 21:01 |
| *** acormier_ has quit IRC | 21:04 | |
| *** acormier has quit IRC | 21:05 | |
| *** acormier has joined #openstack-nova | 21:05 | |
| *** thorst_afk has quit IRC | 21:06 | |
| *** jamesden_ has quit IRC | 21:06 | |
| *** esberglu has quit IRC | 21:08 | |
| *** ijw has joined #openstack-nova | 21:08 | |
| *** imacdonn has quit IRC | 21:08 | |
| *** yamamoto_ has joined #openstack-nova | 21:08 | |
| *** imacdonn has joined #openstack-nova | 21:08 | |
| *** acormier has quit IRC | 21:09 | |
| *** eharney has quit IRC | 21:10 | |
| *** thorst_afk has joined #openstack-nova | 21:11 | |
| *** acormier has joined #openstack-nova | 21:11 | |
| *** slaweq has quit IRC | 21:13 | |
| jaypipes | ugh... rebase hell after pulling in mriedem's changes. | 21:13 |
| jaypipes | thought this was code that wasn't overlapping :( | 21:13 |
| mriedem | jaypipes: i'm fixing that stack now too | 21:13 |
| mriedem | should be up in a couple minutes | 21:13 |
| *** annegentle has quit IRC | 21:13 | |
| jaypipes | mriedem: I was working on the resize-same-host problem | 21:14 |
| *** annegentle has joined #openstack-nova | 21:14 | |
| *** thorst_afk has quit IRC | 21:15 | |
| *** yamamoto_ has quit IRC | 21:15 | |
| mriedem | in the compute? | 21:15 |
| mriedem | or scheduler? | 21:15 |
| *** smatzek has joined #openstack-nova | 21:15 | |
| jaypipes | mriedem: in the report client. fixing up and adding tests for the section dansmith had marked "this is cheating" | 21:16 |
| mriedem | ok i haven't looked at that yet | 21:16 |
| jaypipes | mriedem: fixing up the disk_gb calculation stuff. | 21:16 |
| cdent | I gotta crash, need to not repeat last night’s too lateness. If there’s unclear state to pick up in the morning, please send a status mail or something like that. | 21:16 |
| jaypipes | k | 21:17 |
| *** yamamoto_ has joined #openstack-nova | 21:17 | |
| cdent | Speaking of state: I’m not gonig to be around on friday, so if somebody else wants to do an rp/placement update, have at | 21:17 |
| cdent | good night and good luk | 21:17 |
| cdent | and luck even | 21:17 |
| *** cdent has quit IRC | 21:17 | |
| jaypipes | ciao | 21:17 |
| jaypipes | mriedem: I'll wait on you, then. cuz I pulled the latest update you pushed for the "remove provider allocs in confirm/revert resize" patch above and it's all fooey. | 21:19 |
| mriedem | i had just rebased | 21:19 |
| mriedem | and cherry picked that from gerrit | 21:19 |
| dansmith | maybe we should merge some of the turducken | 21:20 |
| dansmith | fewer plates in the air | 21:20 |
| mriedem | yeah, about there | 21:20 |
| mriedem | chris brought up something in my change i need to address | 21:20 |
| *** priteau has quit IRC | 21:21 | |
| *** lpetrut has joined #openstack-nova | 21:21 | |
| jaypipes | dansmith: if you're looking for something to get your mind off this resize stuff, your opinion on https://review.openstack.org/#/c/474914/7/os_vif/objects/vif.py would be useful. | 21:25 |
| *** baoli has quit IRC | 21:28 | |
| *** smatzek has quit IRC | 21:29 | |
| mriedem | ok coming up | 21:34 |
| mriedem | i assume someone is going to fix the pep8 failures in the last change | 21:35 |
| dansmith | jaypipes said he was | 21:35 |
| jaypipes | yes | 21:35 |
| dansmith | did we get jenkins runs? | 21:35 |
| dansmith | btw, | 21:36 |
| dansmith | jaypipes: reading your (now) top patch earlier, I was kinda thinking the ocata compat thing could/should be split from the fixing of resize_confirm | 21:36 |
| dansmith | are those tied together in some obscure way I'm not seeing? | 21:36 |
| *** esberglu has joined #openstack-nova | 21:37 | |
| jaypipes | dansmith: yeah, agree with you | 21:37 |
| jaypipes | dansmith: this all kinda snowballed on me. :( | 21:37 |
| * dansmith nods | 21:37 | |
| dansmith | cdent rightly noted that the patch is large and gangly at this point | 21:38 |
| jaypipes | yup | 21:38 |
| *** edmondsw has quit IRC | 21:39 | |
| *** edmondsw has joined #openstack-nova | 21:40 | |
| mriedem | f i got distracted and screwed up my rebase | 21:41 |
| *** dikonoor has quit IRC | 21:41 | |
| *** slaweq has joined #openstack-nova | 21:43 | |
| *** vishwana_ has quit IRC | 21:45 | |
| *** vishwanathj has joined #openstack-nova | 21:45 | |
| *** edmondsw has quit IRC | 21:46 | |
| *** thorst_afk has joined #openstack-nova | 21:46 | |
| *** thorst_afk has quit IRC | 21:50 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 21:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 21:51 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 21:51 |
| *** slaweq has quit IRC | 21:52 | |
| *** slaweq has joined #openstack-nova | 21:52 | |
| mriedem | ok i'm +2 on the bottom resize test change | 21:53 |
| mriedem | and my middle change should be ready | 21:53 |
| jaypipes | mriedem: lemme review the second then. | 21:53 |
| jaypipes | mriedem: would certainly be easier to merge the bottom two... | 21:53 |
| jaypipes | mriedem: will +2 the bottom one when I see the test runs | 21:54 |
| *** slaweq has quit IRC | 21:56 | |
| *** slaweq has joined #openstack-nova | 21:56 | |
| *** ijw has quit IRC | 21:57 | |
| *** yangyapeng has joined #openstack-nova | 21:58 | |
| *** efoley has quit IRC | 21:59 | |
| *** xyang1 has quit IRC | 21:59 | |
| *** annegentle has quit IRC | 22:00 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: Filter out stale migrations in resource audit https://review.openstack.org/339715 | 22:01 |
| *** takashin has joined #openstack-nova | 22:01 | |
| *** armax has quit IRC | 22:01 | |
| *** yangyapeng has quit IRC | 22:03 | |
| *** marst has quit IRC | 22:03 | |
| *** burt has quit IRC | 22:06 | |
| *** acormier has quit IRC | 22:07 | |
| *** ijw has joined #openstack-nova | 22:09 | |
| tonyb | mriedem: ANy chance we can get https://review.openstack.org/#/c/489078/ some love. It's needed to unblock a glanceclient update | 22:09 |
| *** felipemonteiro__ has quit IRC | 22:10 | |
| *** https_GK1wmSU has joined #openstack-nova | 22:11 | |
| mriedem | wtf what tests rely on those? | 22:12 |
| *** slaweq has quit IRC | 22:13 | |
| mriedem | oh nova.tests.unit.test_wsgi.TestWSGIServerWithSSL | 22:13 |
| *** slaweq has joined #openstack-nova | 22:13 | |
| mriedem | done | 22:14 |
| mriedem | tonyb: see if dansmith will love up on that now | 22:14 |
| *** https_GK1wmSU has left #openstack-nova | 22:14 | |
| *** lpetrut has quit IRC | 22:14 | |
| tonyb | mriedem: Thanks | 22:14 |
| mriedem | jaypipes: dansmith: fyi i also backported the PUT allocations overwrite change to ocata https://review.openstack.org/#/c/490231/ | 22:15 |
| * tonyb tries to work out what he can temp dansmith with | 22:15 | |
| mriedem | kangaroo sausage | 22:15 |
| dansmith | really really trying not to say something dirty here | 22:15 |
| dansmith | about "love up" on this patch | 22:15 |
| tonyb | dansmith: LOL | 22:15 |
| colby_ | down the newton version of nova support passing the libvirt cpu model from the flavor metadata hw:cpu_model ? Ive been having trouble finding that answer. Ive seen people say it worked in kilo but I have been unable to get it to work on newton | 22:16 |
| *** vishwanathj has quit IRC | 22:16 | |
| *** vishwanathj has joined #openstack-nova | 22:16 | |
| *** gouthamr has quit IRC | 22:16 | |
| mriedem | colby_: no, those are configured on the compute node, not something passed through flavor extra specs | 22:17 |
| mriedem | unless someone has hacked that into their cloud | 22:17 |
| *** slaweq has quit IRC | 22:17 | |
| colby_ | thanks! | 22:18 |
| mriedem | flavor extra specs are a complete wild west so someone could have hacked that in | 22:18 |
| *** thingee has quit IRC | 22:19 | |
| colby_ | yea was hoping it would support it. Was tyring to have flavors that only worked on newer hypervisors, through host aggregates and would use higher cpu model. We use ceph backend so we default to the lowest cpu model for live migration but would like to be able to create some tiers | 22:20 |
| *** ijw has quit IRC | 22:24 | |
| mriedem | colby_: cburgess might have some tips | 22:25 |
| *** acormier has joined #openstack-nova | 22:26 | |
| *** yamamoto_ has quit IRC | 22:29 | |
| *** lyan has quit IRC | 22:29 | |
| *** thingee has joined #openstack-nova | 22:32 | |
| *** yamamoto has joined #openstack-nova | 22:32 | |
| *** efried has quit IRC | 22:32 | |
| *** yamamoto has quit IRC | 22:36 | |
| cfriesen | colby_: mriedem: we hacked in setting the cpu model for a subset of Intel models in the flavor extra specs, and back around kilo we proposed upstreaming it but it turned out to be difficult to do generally (due to different clouds/hypervisors not treating cpu models the same) | 22:37 |
| *** mingyu has joined #openstack-nova | 22:38 | |
| cfriesen | colby_: could you use host aggregates, one per model that you want to expose? | 22:38 |
| *** Sukhdev has quit IRC | 22:40 | |
| *** yamamoto has joined #openstack-nova | 22:41 | |
| *** mingyu has quit IRC | 22:42 | |
| *** yamamoto has quit IRC | 22:44 | |
| *** catintheroof has quit IRC | 22:44 | |
| *** sdague has quit IRC | 22:50 | |
| *** yamamoto has joined #openstack-nova | 22:56 | |
| *** dfisher has quit IRC | 22:58 | |
| *** penick has quit IRC | 22:59 | |
| *** jaypipes has quit IRC | 23:00 | |
| *** yassine has joined #openstack-nova | 23:02 | |
| *** yangyapeng has joined #openstack-nova | 23:02 | |
| *** yangyapeng has quit IRC | 23:07 | |
| *** acormier has quit IRC | 23:08 | |
| *** yamamoto has quit IRC | 23:11 | |
| *** yamamoto has joined #openstack-nova | 23:14 | |
| *** itlinux has quit IRC | 23:20 | |
| *** hongbin has quit IRC | 23:21 | |
| *** http_GK1wmSU has joined #openstack-nova | 23:24 | |
| *** http_GK1wmSU has quit IRC | 23:26 | |
| *** mmehan has quit IRC | 23:27 | |
| *** coreywright has quit IRC | 23:27 | |
| *** rajathagasthya has quit IRC | 23:29 | |
| colby_ | cfriesen: we could. I was tyring to do it in a way where the older cpu model flavors could still run on the newer hypervisors to not restrict those to just the old hypervisors. I was hoping to be able to have instances with different cpu models on the same hypervisor. But that may not be an option. | 23:29 |
| cfriesen | colby_: that was what we hacked in...flavors could specify the cpu model they wanted and would be scheduled on that cpu model or "newer", but would be run with the cpu model they specified. | 23:31 |
| cfriesen | colby_: making it generic enough to apply upstream across different hypervisors and different providers turned out to be horribly complicated | 23:33 |
| *** gouthamr has joined #openstack-nova | 23:33 | |
| colby_ | yea that makes sense | 23:34 |
| *** mtanino has quit IRC | 23:39 | |
| *** mtanino_ is now known as mtanino | 23:39 | |
| *** chyka_ has quit IRC | 23:42 | |
| *** chyka has joined #openstack-nova | 23:43 | |
| *** krtaylor has joined #openstack-nova | 23:43 | |
| *** coreywright has joined #openstack-nova | 23:45 | |
| *** chyka has quit IRC | 23:48 | |
| *** rajathagasthya has joined #openstack-nova | 23:54 | |
| *** thorst_afk has joined #openstack-nova | 23:55 | |
| *** http_GK1wmSU has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!