| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-evacuate notification https://review.openstack.org/482148 | 00:00 |
|---|---|---|
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | 00:10 |
| *** esberglu has joined #openstack-nova | 00:10 | |
| *** gmann has quit IRC | 00:13 | |
| *** baoli has joined #openstack-nova | 00:13 | |
| *** gmann has joined #openstack-nova | 00:13 | |
| *** esberglu has quit IRC | 00:15 | |
| *** gcb has joined #openstack-nova | 00:15 | |
| mriedem | bauzas: ^ | 00:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | 00:16 |
| *** owalsh has quit IRC | 00:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | 00:17 |
| mriedem | huanxie: is the citrix xenserver CI broken? | 00:17 |
| mriedem | it's automatically failed all day | 00:18 |
| *** edmondsw has joined #openstack-nova | 00:19 | |
| *** tylerderosagrund has joined #openstack-nova | 00:22 | |
| *** lucasxu has quit IRC | 00:22 | |
| tbachman | mriedem: consistency is perfection | 00:22 |
| *** jmlowe has joined #openstack-nova | 00:23 | |
| *** edmondsw has quit IRC | 00:24 | |
| *** tylerderosagrund has quit IRC | 00:26 | |
| *** acormier has joined #openstack-nova | 00:30 | |
| *** acormier has quit IRC | 00:30 | |
| *** acormier has joined #openstack-nova | 00:30 | |
| *** lucasxu has joined #openstack-nova | 00:35 | |
| *** acormier has quit IRC | 00:36 | |
| *** acormier has joined #openstack-nova | 00:36 | |
| *** acormier has quit IRC | 00:41 | |
| *** acormier has joined #openstack-nova | 00:42 | |
| *** acormier has quit IRC | 00:44 | |
| *** tbachman has quit IRC | 00:44 | |
| *** acormier has joined #openstack-nova | 00:44 | |
| *** owalsh has joined #openstack-nova | 00:45 | |
| *** takashin has left #openstack-nova | 00:46 | |
| gmann | mriedem, ohh, thanks a lot for fixing quota set microversion | 00:48 |
| *** kiennt has joined #openstack-nova | 00:49 | |
| *** takashin has joined #openstack-nova | 00:49 | |
| *** thorst has joined #openstack-nova | 00:49 | |
| *** thorst has quit IRC | 00:50 | |
| *** lucasxu has quit IRC | 00:55 | |
| *** baoli has quit IRC | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:01 | |
| *** Apoorva_ has quit IRC | 01:04 | |
| *** baoli has joined #openstack-nova | 01:04 | |
| *** Apoorva has joined #openstack-nova | 01:05 | |
| *** dave-mccowan has joined #openstack-nova | 01:06 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209 | 01:09 |
| openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 01:11 |
| *** harlowja has quit IRC | 01:13 | |
| *** zhurong has joined #openstack-nova | 01:13 | |
| *** hongbin has joined #openstack-nova | 01:14 | |
| *** takashin has quit IRC | 01:20 | |
| *** Daisy_ has joined #openstack-nova | 01:21 | |
| *** mriedem1 has joined #openstack-nova | 01:22 | |
| *** Daisy_ has quit IRC | 01:23 | |
| *** Daisy_ has joined #openstack-nova | 01:24 | |
| *** mriedem has quit IRC | 01:24 | |
| *** gouthamr has quit IRC | 01:25 | |
| *** kbaegis has quit IRC | 01:27 | |
| *** kbaegis has joined #openstack-nova | 01:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 01:28 |
| *** mdnadeem has joined #openstack-nova | 01:30 | |
| *** tbachman has joined #openstack-nova | 01:32 | |
| *** yushb has joined #openstack-nova | 01:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 01:38 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 01:38 |
| *** Apoorva_ has joined #openstack-nova | 01:39 | |
| *** tuanluong has joined #openstack-nova | 01:40 | |
| *** Apoorva has quit IRC | 01:42 | |
| *** yamahata has quit IRC | 01:43 | |
| *** Apoorva_ has quit IRC | 01:44 | |
| *** Daisy_ has quit IRC | 01:44 | |
| *** Daisy_ has joined #openstack-nova | 01:44 | |
| *** Daisy_ has quit IRC | 01:48 | |
| *** Daisy_ has joined #openstack-nova | 01:48 | |
| *** winston-d_ has joined #openstack-nova | 01:49 | |
| *** acormier has quit IRC | 01:53 | |
| *** acormier has joined #openstack-nova | 01:54 | |
| mriedem1 | melwitt: i'm about to +2 the counting instances change, just one question about test coverage in https://review.openstack.org/#/c/416521/63/nova/conductor/manager.py@989 | 01:59 |
| mriedem1 | i don't really get it though since i think you have the test for it, unless that test is missing something | 01:59 |
| *** mriedem1 is now known as mriedem | 01:59 | |
| mriedem | nvm, i poked at it, i think the coverage report is busted, so +2 | 02:07 |
| mriedem | dansmith: ^ +2 on counting instances https://review.openstack.org/#/c/416521/ | 02:07 |
| *** edmondsw has joined #openstack-nova | 02:07 | |
| *** edmondsw has quit IRC | 02:12 | |
| *** gyee has quit IRC | 02:17 | |
| *** acormier_ has joined #openstack-nova | 02:17 | |
| *** yamamoto has joined #openstack-nova | 02:17 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/python-novaclient master: Replace uuid.uuid4() with uuidutils.generate_uuid() https://review.openstack.org/484244 | 02:18 |
| *** tyrefors has quit IRC | 02:19 | |
| *** acormier has quit IRC | 02:21 | |
| *** gouthamr has joined #openstack-nova | 02:21 | |
| mriedem | ok i'm +2 up through https://review.openstack.org/#/c/446243/ now | 02:21 |
| *** acormier_ has quit IRC | 02:22 | |
| *** acormier has joined #openstack-nova | 02:23 | |
| *** lucasxu has joined #openstack-nova | 02:30 | |
| *** Shunli has joined #openstack-nova | 02:30 | |
| *** acormier has quit IRC | 02:36 | |
| *** acormier has joined #openstack-nova | 02:37 | |
| *** yamamoto has quit IRC | 02:38 | |
| *** owalsh_ has joined #openstack-nova | 02:40 | |
| *** acormier has quit IRC | 02:42 | |
| *** owalsh has quit IRC | 02:43 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore https://review.openstack.org/485011 | 02:45 |
| *** tyrefors has joined #openstack-nova | 02:45 | |
| *** owalsh_ has quit IRC | 02:46 | |
| *** owalsh has joined #openstack-nova | 02:46 | |
| *** yamamoto has joined #openstack-nova | 02:48 | |
| *** tylerderosagrund has joined #openstack-nova | 02:48 | |
| *** thorst has joined #openstack-nova | 02:51 | |
| *** lucasxu has quit IRC | 02:53 | |
| *** tylerderosagrund has quit IRC | 02:53 | |
| mriedem | alex_xu: i finally got comments addressed and rebased the os-services/os-hypervisors to use uuids series https://review.openstack.org/#/c/464280/ - it's probably easiest at quick glance to just look at the diff from the last patch set | 02:58 |
| mriedem | to see the major changes | 02:58 |
| alex_xu | mriedem: yea, got it | 02:58 |
| mriedem | cool, thanks | 02:58 |
| alex_xu | np | 02:58 |
| *** thorst has quit IRC | 02:59 | |
| *** markvoelker has quit IRC | 02:59 | |
| *** moshele has joined #openstack-nova | 03:00 | |
| *** mriedem has quit IRC | 03:02 | |
| openstackgerrit | Chason Chan proposed openstack/nova master: doc: Switch to openstackdocstheme https://review.openstack.org/477751 | 03:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 03:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 03:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 03:07 |
| *** jichen has joined #openstack-nova | 03:09 | |
| *** Daisy_ has quit IRC | 03:09 | |
| *** Daisy_ has joined #openstack-nova | 03:10 | |
| *** sree_ has joined #openstack-nova | 03:11 | |
| *** kbaegis has quit IRC | 03:11 | |
| *** sree_ is now known as Guest92762 | 03:11 | |
| *** moshele has quit IRC | 03:12 | |
| *** Daisy_ has quit IRC | 03:14 | |
| *** Guest92762 has quit IRC | 03:15 | |
| *** takashin has joined #openstack-nova | 03:16 | |
| *** lucasxu has joined #openstack-nova | 03:17 | |
| *** kbaegis has joined #openstack-nova | 03:18 | |
| *** litao__ has joined #openstack-nova | 03:22 | |
| *** jwcroppe has joined #openstack-nova | 03:24 | |
| *** udesale has joined #openstack-nova | 03:33 | |
| *** hongbin has quit IRC | 03:36 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 03:38 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 03:38 |
| *** masber has quit IRC | 03:51 | |
| *** edmondsw has joined #openstack-nova | 03:55 | |
| *** hieulq has quit IRC | 03:57 | |
| *** hieulq has joined #openstack-nova | 03:57 | |
| *** yamamoto has quit IRC | 03:57 | |
| *** links has joined #openstack-nova | 03:58 | |
| *** yamamoto has joined #openstack-nova | 03:59 | |
| *** edmondsw has quit IRC | 04:00 | |
| *** dave-mccowan has quit IRC | 04:01 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 04:01 |
| *** kbaegis has quit IRC | 04:02 | |
| *** lucasxu has quit IRC | 04:02 | |
| *** kbaegis has joined #openstack-nova | 04:03 | |
| *** kbaegis has quit IRC | 04:03 | |
| *** yamamoto has quit IRC | 04:04 | |
| *** yamamoto has joined #openstack-nova | 04:05 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:11 | |
| *** jwcroppe has quit IRC | 04:12 | |
| *** jwcroppe has joined #openstack-nova | 04:12 | |
| *** gouthamr has quit IRC | 04:14 | |
| *** yamamoto has quit IRC | 04:14 | |
| *** Dinesh_Bhor has quit IRC | 04:14 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:15 | |
| *** yushb has quit IRC | 04:15 | |
| *** yamamoto has joined #openstack-nova | 04:15 | |
| *** yamamoto has quit IRC | 04:21 | |
| *** psachin has joined #openstack-nova | 04:21 | |
| *** yushb has joined #openstack-nova | 04:24 | |
| *** trinaths has joined #openstack-nova | 04:26 | |
| *** coreywright has quit IRC | 04:26 | |
| *** harlowja has joined #openstack-nova | 04:31 | |
| *** mamandle has joined #openstack-nova | 04:34 | |
| *** yushb has quit IRC | 04:36 | |
| *** prateek has joined #openstack-nova | 04:38 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 04:39 |
| *** Daisy_ has joined #openstack-nova | 04:41 | |
| *** sudhirag0987 has joined #openstack-nova | 04:42 | |
| *** gouthamr has joined #openstack-nova | 04:42 | |
| *** yamamoto has joined #openstack-nova | 04:44 | |
| *** yamahata has joined #openstack-nova | 04:45 | |
| *** Daisy_ has quit IRC | 04:46 | |
| *** harlowja has quit IRC | 04:47 | |
| *** lpetrut__ has joined #openstack-nova | 04:51 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209 | 04:53 |
| *** zhurong has quit IRC | 04:55 | |
| *** sree_ has joined #openstack-nova | 04:59 | |
| *** markvoelker has joined #openstack-nova | 05:00 | |
| *** sree_ is now known as Guest7201 | 05:00 | |
| *** Daisy_ has joined #openstack-nova | 05:00 | |
| *** Guest7201 has quit IRC | 05:04 | |
| *** gouthamr has quit IRC | 05:07 | |
| *** moshele has joined #openstack-nova | 05:08 | |
| *** nicolasbock has joined #openstack-nova | 05:11 | |
| *** sridharg has joined #openstack-nova | 05:13 | |
| *** ratailor has joined #openstack-nova | 05:14 | |
| *** armax has quit IRC | 05:14 | |
| *** armax has joined #openstack-nova | 05:15 | |
| *** armax has quit IRC | 05:15 | |
| *** armax has joined #openstack-nova | 05:16 | |
| *** armax has quit IRC | 05:16 | |
| *** armax has joined #openstack-nova | 05:17 | |
| *** armax has quit IRC | 05:17 | |
| *** Dinesh_Bhor has quit IRC | 05:17 | |
| *** armax has joined #openstack-nova | 05:17 | |
| *** sudhirag0987 has quit IRC | 05:18 | |
| *** armax has quit IRC | 05:18 | |
| *** armax has joined #openstack-nova | 05:18 | |
| *** armax has quit IRC | 05:18 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: WIP: support os_xenapi's VDI streaming https://review.openstack.org/485039 | 05:22 |
| *** frickler has quit IRC | 05:24 | |
| *** frickler has joined #openstack-nova | 05:27 | |
| *** sudhirag0987 has joined #openstack-nova | 05:28 | |
| *** tylerderosagrund has joined #openstack-nova | 05:30 | |
| *** frickler has quit IRC | 05:33 | |
| *** markvoelker has quit IRC | 05:34 | |
| *** tylerderosagrund has quit IRC | 05:34 | |
| *** frickler has joined #openstack-nova | 05:38 | |
| *** acormier has joined #openstack-nova | 05:38 | |
| *** kiennt has quit IRC | 05:42 | |
| *** acormier has quit IRC | 05:43 | |
| *** edmondsw has joined #openstack-nova | 05:43 | |
| openstackgerrit | Merged openstack/nova master: Consider instance flavor resource overrides in allocations https://review.openstack.org/484935 | 05:44 |
| *** harlowja has joined #openstack-nova | 05:46 | |
| *** edmondsw has quit IRC | 05:48 | |
| *** ekuris has joined #openstack-nova | 05:49 | |
| *** shuangyangqian has joined #openstack-nova | 05:50 | |
| *** zhurong has joined #openstack-nova | 05:50 | |
| *** yushb has joined #openstack-nova | 05:50 | |
| *** rcernin has joined #openstack-nova | 05:50 | |
| *** Daisy_ has quit IRC | 05:52 | |
| *** Daisy_ has joined #openstack-nova | 05:53 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 05:55 |
| *** thorst has joined #openstack-nova | 05:56 | |
| *** Daisy_ has quit IRC | 05:57 | |
| *** Daisy_ has joined #openstack-nova | 05:58 | |
| *** thorst has quit IRC | 06:01 | |
| openstackgerrit | Merged openstack/nova master: Use plain routes list for os-security-group-rules instead of stevedore https://review.openstack.org/478903 | 06:01 |
| *** kiennt has joined #openstack-nova | 06:02 | |
| *** lpetrut__ has quit IRC | 06:04 | |
| *** Oku_OS-away is now known as Oku_OS | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 06:11 |
| *** jichen has quit IRC | 06:12 | |
| *** jjung has joined #openstack-nova | 06:20 | |
| *** kiennt has quit IRC | 06:21 | |
| *** baoli has quit IRC | 06:22 | |
| *** nicolasbock has quit IRC | 06:23 | |
| *** nicolasbock has joined #openstack-nova | 06:25 | |
| *** yushb has quit IRC | 06:31 | |
| *** markvoelker has joined #openstack-nova | 06:31 | |
| *** jwcroppe has quit IRC | 06:32 | |
| *** andreas_s has joined #openstack-nova | 06:32 | |
| *** nicolasbock has quit IRC | 06:37 | |
| *** jjung has quit IRC | 06:37 | |
| *** nicolasbock has joined #openstack-nova | 06:39 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the unittest for plugin framework https://review.openstack.org/485060 | 06:41 |
| openstackgerrit | Alex Xu proposed openstack/nova master: [DNM] Using plain routes for the microversions test https://review.openstack.org/485061 | 06:41 |
| *** harlowja has quit IRC | 06:42 | |
| *** frickler has quit IRC | 06:42 | |
| *** frickler has joined #openstack-nova | 06:42 | |
| *** sshwarts has joined #openstack-nova | 06:48 | |
| *** Shunli has quit IRC | 06:49 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:50 | |
| *** Shunli has joined #openstack-nova | 06:50 | |
| *** Dinesh_Bhor has quit IRC | 06:50 | |
| *** Shunli has quit IRC | 06:56 | |
| *** Shunli has joined #openstack-nova | 06:57 | |
| *** jjung has joined #openstack-nova | 07:00 | |
| *** annegentle has quit IRC | 07:01 | |
| *** udesale has quit IRC | 07:01 | |
| *** udesale has joined #openstack-nova | 07:01 | |
| *** markvoelker has quit IRC | 07:04 | |
| *** yushb has joined #openstack-nova | 07:04 | |
| *** lpetrut has joined #openstack-nova | 07:06 | |
| *** itlinux has quit IRC | 07:11 | |
| *** tesseract has joined #openstack-nova | 07:17 | |
| *** damien_r has joined #openstack-nova | 07:19 | |
| *** adisky__ has joined #openstack-nova | 07:20 | |
| *** damien_r has quit IRC | 07:24 | |
| *** damien_r has joined #openstack-nova | 07:25 | |
| *** catintheroof has joined #openstack-nova | 07:31 | |
| *** edmondsw has joined #openstack-nova | 07:31 | |
| gibi | jaypipes: I tested your patch https://review.openstack.org/#/c/484900 I commented the results in the review | 07:33 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix an expand button in os-quota-sets https://review.openstack.org/485075 | 07:35 |
| *** edmondsw has quit IRC | 07:36 | |
| *** catintheroof has quit IRC | 07:36 | |
| *** kiennt has joined #openstack-nova | 07:40 | |
| *** tylerderosagrund has joined #openstack-nova | 07:40 | |
| *** tylerderosagrund has quit IRC | 07:45 | |
| *** ralonsoh has joined #openstack-nova | 07:46 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: WIP: remove physnet https://review.openstack.org/485078 | 07:47 |
| openstackgerrit | Moshe Levi proposed openstack/nova master: WIP: remove physnet https://review.openstack.org/485078 | 07:49 |
| *** kristian__ has joined #openstack-nova | 07:51 | |
| *** lpetrut has quit IRC | 07:55 | |
| *** jjung has quit IRC | 07:55 | |
| *** thorst has joined #openstack-nova | 07:57 | |
| *** ociuhandu_ has joined #openstack-nova | 08:01 | |
| *** thorst has quit IRC | 08:02 | |
| *** takashin has left #openstack-nova | 08:02 | |
| *** tylerderosagrund has joined #openstack-nova | 08:05 | |
| *** mlakat has quit IRC | 08:06 | |
| *** elod has quit IRC | 08:07 | |
| *** derekh has joined #openstack-nova | 08:08 | |
| *** dtantsur|afk is now known as dtantsur | 08:09 | |
| *** tylerderosagrund has quit IRC | 08:09 | |
| *** elod has joined #openstack-nova | 08:14 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 08:15 |
| *** jangutter_ is now known as jangutter | 08:18 | |
| *** lucas-afk is now known as lucasagomes | 08:18 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: [placement] test shared custom resource provider https://review.openstack.org/485088 | 08:23 |
| *** yushb has quit IRC | 08:24 | |
| *** jjung has joined #openstack-nova | 08:24 | |
| *** xinliang has quit IRC | 08:28 | |
| *** xinliang has joined #openstack-nova | 08:29 | |
| *** kristian__ has quit IRC | 08:29 | |
| *** ociuhandu_ has quit IRC | 08:29 | |
| *** kristian__ has joined #openstack-nova | 08:30 | |
| stephenfin | gmann: Could you take a look at https://review.openstack.org/#/c/478484/ again? | 08:30 |
| *** artom_ has quit IRC | 08:31 | |
| *** efoley has joined #openstack-nova | 08:31 | |
| *** artom_ has joined #openstack-nova | 08:32 | |
| *** efoley has quit IRC | 08:32 | |
| *** efoley has joined #openstack-nova | 08:32 | |
| *** ralonsoh has quit IRC | 08:33 | |
| *** ralonsoh_ has joined #openstack-nova | 08:33 | |
| *** kristian__ has quit IRC | 08:34 | |
| *** kristian__ has joined #openstack-nova | 08:38 | |
| *** jjung has quit IRC | 08:38 | |
| *** yushb has joined #openstack-nova | 08:39 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122 | 08:40 |
| *** dtantsur is now known as dtantsur|brb | 08:41 | |
| *** Daisy_ has quit IRC | 08:41 | |
| *** Daisy_ has joined #openstack-nova | 08:42 | |
| *** mlakat has joined #openstack-nova | 08:42 | |
| *** kristian__ has quit IRC | 08:43 | |
| openstackgerrit | Merged openstack/nova master: Use plain routes list for os-security-group-default-rules instead of stevedore https://review.openstack.org/478904 | 08:45 |
| *** Daisy__ has joined #openstack-nova | 08:45 | |
| *** Daisy_ has quit IRC | 08:46 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | 08:47 |
| stephenfin | bauzas: Some easy +2/+W's here https://review.openstack.org/#/q/topic:doc-migration+project:openstack/os-traits+status:open | 08:47 |
| bauzas | stephenfin: yup, I saw them | 08:47 |
| bauzas | stephenfin: I wanted to discuss with matt about if he wanted to pass a new version for os-traits | 08:48 |
| bauzas | we're very close to the library deadline, right? | 08:48 |
| bauzas | yeah https://releases.openstack.org/pike/schedule.html#p-final-clientlib | 08:48 |
| *** openstackgerrit has quit IRC | 08:49 | |
| stephenfin | bauzas: We are, yeah | 08:49 |
| stephenfin | Do docs affect that deadline? | 08:49 |
| bauzas | in theory nope | 08:49 |
| bauzas | that's not a "feature freeze" | 08:49 |
| bauzas | here we call of the last time to version the library | 08:50 |
| bauzas | for pike | 08:50 |
| bauzas | in order to have stable dependencies | 08:50 |
| bauzas | so, we can merge your docs changes, but the big question is whether we need to tag for it | 08:50 |
| *** openstackgerrit has joined #openstack-nova | 08:57 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor Lookup https://review.openstack.org/484051 | 08:57 |
| *** markvoelker has joined #openstack-nova | 09:02 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add memoize function using oslo.cache https://review.openstack.org/472773 | 09:04 |
| *** yushb has quit IRC | 09:05 | |
| *** yushb has joined #openstack-nova | 09:05 | |
| *** yamamoto has quit IRC | 09:06 | |
| *** avolkov has joined #openstack-nova | 09:07 | |
| *** tbachman has quit IRC | 09:10 | |
| *** yamamoto has joined #openstack-nova | 09:10 | |
| *** yamamoto has quit IRC | 09:10 | |
| *** tbachman has joined #openstack-nova | 09:13 | |
| *** yamamoto has joined #openstack-nova | 09:14 | |
| *** avolkov has quit IRC | 09:18 | |
| *** avolkov has joined #openstack-nova | 09:19 | |
| *** edmondsw has joined #openstack-nova | 09:19 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 09:21 |
| *** efoley_ has joined #openstack-nova | 09:21 | |
| *** efoley has quit IRC | 09:24 | |
| *** edmondsw has quit IRC | 09:24 | |
| *** jjung has joined #openstack-nova | 09:28 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 09:29 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP:add cert when send keystone request https://review.openstack.org/485121 | 09:29 |
| *** Shunli has quit IRC | 09:30 | |
| *** sudhirag0987 has quit IRC | 09:34 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Adjust error msg for ImageNUMATopologyAsymmetric https://review.openstack.org/484634 | 09:34 |
| *** yingjun has quit IRC | 09:34 | |
| *** markvoelker has quit IRC | 09:35 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 09:36 |
| *** yamamoto has quit IRC | 09:36 | |
| *** yamamoto has joined #openstack-nova | 09:36 | |
| *** sdague has joined #openstack-nova | 09:37 | |
| *** sambetts|afk is now known as sambetts | 09:42 | |
| *** Daisy__ has quit IRC | 09:45 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the unittest for plugin framework https://review.openstack.org/485060 | 09:48 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore https://review.openstack.org/485011 | 09:48 |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: ransform instance.resize_confirm notification https://review.openstack.org/482557 | 09:49 |
| *** yushb has quit IRC | 09:49 | |
| openstackgerrit | sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007 | 09:50 |
| *** lpetrut has joined #openstack-nova | 09:50 | |
| *** mvk has joined #openstack-nova | 09:50 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 09:51 |
| *** yushb has joined #openstack-nova | 09:54 | |
| *** thorst has joined #openstack-nova | 09:58 | |
| *** shuangyangqian_ has joined #openstack-nova | 09:59 | |
| *** shuangyangqian has quit IRC | 10:02 | |
| *** thorst has quit IRC | 10:03 | |
| *** zz9pzza has joined #openstack-nova | 10:04 | |
| gmann | stephenfin, +1, agree to separate that later | 10:05 |
| *** moshele has quit IRC | 10:09 | |
| *** yushb has quit IRC | 10:09 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: [placement] test shared custom resource provider https://review.openstack.org/485088 | 10:15 |
| ildikov | stephenfin: thanks for the review! | 10:21 |
| *** udesale has quit IRC | 10:24 | |
| *** Daisy_ has joined #openstack-nova | 10:24 | |
| *** moshele has joined #openstack-nova | 10:24 | |
| *** Dinesh_Bhor has joined #openstack-nova | 10:27 | |
| *** Daisy_ has quit IRC | 10:29 | |
| *** moshele has quit IRC | 10:30 | |
| *** yamahata has quit IRC | 10:30 | |
| *** moshele has joined #openstack-nova | 10:30 | |
| *** markvoelker has joined #openstack-nova | 10:32 | |
| *** moshele has quit IRC | 10:37 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support virtual GPU resources https://review.openstack.org/450122 | 10:39 |
| stephenfin | ildikov: No problem :) | 10:41 |
| *** priteau has joined #openstack-nova | 10:41 | |
| *** vvargaszte has quit IRC | 10:43 | |
| * stephenfin gently prods bauzas and gmann to inform them issues with https://review.openstack.org/479869 and https://review.openstack.org/480090 are now resolved | 10:43 | |
| *** kiennt has quit IRC | 10:45 | |
| *** kristian__ has joined #openstack-nova | 10:48 | |
| *** kristian__ has quit IRC | 10:52 | |
| *** kristian__ has joined #openstack-nova | 10:53 | |
| *** zhurong has quit IRC | 10:53 | |
| *** lpetrut_ has joined #openstack-nova | 10:54 | |
| *** yamamoto has quit IRC | 10:55 | |
| *** kristia__ has joined #openstack-nova | 10:55 | |
| *** gcb has quit IRC | 10:55 | |
| *** lpetrut has quit IRC | 10:56 | |
| *** gcb has joined #openstack-nova | 10:57 | |
| *** andreas_s has quit IRC | 10:57 | |
| *** kristian__ has quit IRC | 11:00 | |
| *** moshele has joined #openstack-nova | 11:01 | |
| *** elod has quit IRC | 11:03 | |
| *** moshele has quit IRC | 11:04 | |
| *** vvargaszte has joined #openstack-nova | 11:04 | |
| *** tuanluong has quit IRC | 11:05 | |
| *** markvoelker has quit IRC | 11:06 | |
| *** edmondsw has joined #openstack-nova | 11:07 | |
| *** mdnadeem has quit IRC | 11:08 | |
| *** alexchadin has joined #openstack-nova | 11:09 | |
| *** elod has joined #openstack-nova | 11:11 | |
| *** roeyc has joined #openstack-nova | 11:11 | |
| *** edmondsw has quit IRC | 11:12 | |
| *** jmlowe has quit IRC | 11:14 | |
| *** cdent has joined #openstack-nova | 11:18 | |
| *** mdnadeem has joined #openstack-nova | 11:20 | |
| jangutter | Newbie question here: "what is aglo"? | 11:21 |
| *** shuangyangqian has joined #openstack-nova | 11:22 | |
| *** ekuris has quit IRC | 11:23 | |
| *** kristia__ has quit IRC | 11:23 | |
| *** shuangyangqian_ has quit IRC | 11:24 | |
| *** kristian__ has joined #openstack-nova | 11:24 | |
| *** mamandle has quit IRC | 11:25 | |
| *** moshele has joined #openstack-nova | 11:26 | |
| *** dave-mccowan has joined #openstack-nova | 11:26 | |
| *** sean-k-mooney has quit IRC | 11:27 | |
| *** kristian__ has quit IRC | 11:29 | |
| *** edmondsw has joined #openstack-nova | 11:30 | |
| *** bertys has joined #openstack-nova | 11:30 | |
| sdague | stephenfin / bauzas - https://review.openstack.org/#/c/479027/ easy delete | 11:31 |
| *** smatzek has joined #openstack-nova | 11:38 | |
| *** kaisers has quit IRC | 11:39 | |
| *** yamamoto has joined #openstack-nova | 11:40 | |
| *** acormier_ has joined #openstack-nova | 11:41 | |
| *** acormier_ has quit IRC | 11:41 | |
| *** acormier has joined #openstack-nova | 11:41 | |
| *** acormier has quit IRC | 11:42 | |
| *** acormier has joined #openstack-nova | 11:42 | |
| openstackgerrit | Merged openstack/nova master: Use plain routes list for os-baremetal-nodes endpoint instead of stevedore https://review.openstack.org/461480 | 11:45 |
| *** jmlowe has joined #openstack-nova | 11:46 | |
| *** acormier has quit IRC | 11:46 | |
| *** kaisers has joined #openstack-nova | 11:47 | |
| *** jmlowe has quit IRC | 11:49 | |
| *** litao__ has quit IRC | 11:51 | |
| *** phuongnh has quit IRC | 11:52 | |
| *** shuangyangqian has quit IRC | 11:55 | |
| *** shuangyangqian has joined #openstack-nova | 11:57 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Allow wrapping of closures https://review.openstack.org/479801 | 11:57 |
| *** avolkov has quit IRC | 11:59 | |
| mdbooth | dansmith: Got a minute to talk about https://review.openstack.org/#/c/479802/ ? | 12:00 |
| *** thorst has joined #openstack-nova | 12:02 | |
| *** markvoelker has joined #openstack-nova | 12:03 | |
| *** yamamoto has quit IRC | 12:03 | |
| *** thorst_ has joined #openstack-nova | 12:04 | |
| *** lucasagomes is now known as lucas-hungry | 12:04 | |
| *** mkucia has joined #openstack-nova | 12:06 | |
| *** thorst has quit IRC | 12:06 | |
| *** hoonetorg has joined #openstack-nova | 12:09 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 12:11 |
| *** sean-k-mooney has joined #openstack-nova | 12:13 | |
| *** ociuhandu_ has joined #openstack-nova | 12:15 | |
| *** mgiles has joined #openstack-nova | 12:16 | |
| *** jmlowe has joined #openstack-nova | 12:16 | |
| *** mgiles has quit IRC | 12:20 | |
| *** mgiles has joined #openstack-nova | 12:21 | |
| *** yamamoto has joined #openstack-nova | 12:21 | |
| *** catintheroof has joined #openstack-nova | 12:21 | |
| *** mkucia has quit IRC | 12:22 | |
| *** gszasz has joined #openstack-nova | 12:22 | |
| *** gmann_ has joined #openstack-nova | 12:23 | |
| frickler | jangutter: I'd guess it is a typo for "algo", short for "algorithm" | 12:24 |
| *** mdnadeem has quit IRC | 12:25 | |
| *** markvoelker has quit IRC | 12:25 | |
| *** markvoelker has joined #openstack-nova | 12:26 | |
| *** pchavva has joined #openstack-nova | 12:26 | |
| jangutter | frickler: Assumed as much. Could also be "Aklo" which refers to the fictional language that's able to control the universe and melt brains at 10 paces. | 12:26 |
| *** eharney has joined #openstack-nova | 12:27 | |
| openstackgerrit | Merged openstack/os-vif master: Improve OVS Representor Lookup https://review.openstack.org/484051 | 12:28 |
| *** jmlowe has quit IRC | 12:29 | |
| *** elod has quit IRC | 12:30 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Migration from 'ip' commands to pyroute2 https://review.openstack.org/484386 | 12:30 |
| *** elod has joined #openstack-nova | 12:30 | |
| *** mriedem has joined #openstack-nova | 12:31 | |
| *** baoli has joined #openstack-nova | 12:31 | |
| *** ralonsoh_ has quit IRC | 12:32 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 12:32 |
| *** baoli_ has joined #openstack-nova | 12:33 | |
| *** baoli has quit IRC | 12:37 | |
| *** MVenesio has joined #openstack-nova | 12:37 | |
| *** jmlowe has joined #openstack-nova | 12:38 | |
| *** kristian__ has joined #openstack-nova | 12:39 | |
| *** dtantsur|brb is now known as dtantsur | 12:40 | |
| *** kristian__ has quit IRC | 12:40 | |
| *** kristian__ has joined #openstack-nova | 12:40 | |
| *** avolkov has joined #openstack-nova | 12:41 | |
| mriedem | alex_xu: thanks for going over that uuid api change again | 12:42 |
| alex_xu | mriedem: np | 12:43 |
| *** acormier has joined #openstack-nova | 12:43 | |
| stephenfin | sdague: done | 12:43 |
| alex_xu | nova api meeting is in 15 mins at #openstack-meeting-4 | 12:45 |
| *** kristian__ has quit IRC | 12:45 | |
| *** kristian__ has joined #openstack-nova | 12:45 | |
| *** mgiles has quit IRC | 12:47 | |
| *** mgiles has joined #openstack-nova | 12:48 | |
| *** acormier has quit IRC | 12:48 | |
| *** kylek3h has joined #openstack-nova | 12:51 | |
| *** gcb has quit IRC | 12:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 12:56 |
| openstackgerrit | Alex Xu proposed openstack/nova master: Use plain routes list for versions instead of stevedore https://review.openstack.org/485011 | 12:56 |
| *** lyan has joined #openstack-nova | 12:56 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Remove the unittest for plugin framework https://review.openstack.org/485060 | 12:56 |
| *** lyan has quit IRC | 12:57 | |
| *** gouthamr has joined #openstack-nova | 12:58 | |
| alex_xu | nova api meeting is running | 13:01 |
| *** jamesdenton has joined #openstack-nova | 13:01 | |
| *** udesale has joined #openstack-nova | 13:01 | |
| *** mgiles has quit IRC | 13:04 | |
| *** mgiles has joined #openstack-nova | 13:05 | |
| *** lucasxu has joined #openstack-nova | 13:06 | |
| *** crushil has joined #openstack-nova | 13:09 | |
| *** esberglu has joined #openstack-nova | 13:09 | |
| *** winston-d_ has quit IRC | 13:10 | |
| *** randomhack has joined #openstack-nova | 13:11 | |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 13:13 |
| bauzas | mriedem: when you're up and fully coffeenated, just ping me about https://review.openstack.org/#/c/484828/ | 13:13 |
| bauzas | caffeinated even | 13:13 |
| *** lyan has joined #openstack-nova | 13:15 | |
| mriedem | bauzas: in a meeting for the next 2 hours | 13:17 |
| *** mriedem is now known as mriedem_meeting | 13:17 | |
| bauzas | mriedem_meeting: heh | 13:17 |
| bauzas | no worries | 13:17 |
| *** ralonsoh has joined #openstack-nova | 13:19 | |
| *** ralonsoh has quit IRC | 13:20 | |
| *** ralonsoh has joined #openstack-nova | 13:20 | |
| *** mgiles has quit IRC | 13:23 | |
| *** efoley_ has quit IRC | 13:23 | |
| *** mgiles has joined #openstack-nova | 13:24 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 13:24 |
| cdent | gibi: I’ve been able to create a flapping testing on /allocation_candidates too | 13:25 |
| cdent | wrote it gabbi-style, will push it up in a moment for additional discussion | 13:26 |
| gibi | cdent: cool then it is not an environemnet issue | 13:26 |
| *** gmann_ has quit IRC | 13:26 | |
| *** mgiles has quit IRC | 13:30 | |
| *** mgiles has joined #openstack-nova | 13:31 | |
| gibi | cdent: I have the SQL SELECT statements both for the passing and failing case, I put them in the bugreport | 13:31 |
| *** ratailor has quit IRC | 13:31 | |
| cdent | ah excellent | 13:31 |
| cdent | weird | 13:33 |
| gibi | cdent: looks quite different | 13:33 |
| cdent | yes | 13:33 |
| gibi | but I'm far from a SQL expert | 13:33 |
| *** sree has joined #openstack-nova | 13:34 | |
| cdent | i’ll take a closer look at the sql building code after I push up these tests | 13:34 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: [placement] test shared custom resource provider https://review.openstack.org/485088 | 13:35 |
| gibi | cdent: thanks, I'm pretty much lost | 13:35 |
| cdent | gibi: the code has become quite dense around this stuff | 13:36 |
| cdent | what you’re trying to do is supposed to work | 13:36 |
| gibi | cdent: and it works if the shared resource is not a custom resource or if I add the sharing trait to the compute provider as well | 13:37 |
| cdent | I think that you initially had it working with both providers having the MISC_* trait was coincidence; the data that comes back is a bit wack in that situation | 13:37 |
| cdent | you get too many allocation_requests | 13:37 |
| gibi | cdent: could be, I haven't dig in to that output | 13:38 |
| *** shuangyangqian has quit IRC | 13:38 | |
| *** shuangyangqian has joined #openstack-nova | 13:39 | |
| *** Daisy_ has joined #openstack-nova | 13:40 | |
| sdague | mriedem_meeting: I went through the hypervisor and service changes | 13:41 |
| sdague | there are some follow ups that could be done, but I think they are probably good enough to merge, those are mostly test tweaks | 13:42 |
| openstackgerrit | Chris Dent proposed openstack/nova master: DNM: [placement] demonstrate flapping /allocation_candidates results https://review.openstack.org/485209 | 13:42 |
| sdague | mriedem_meeting: so if you release your -2 they'll merge | 13:42 |
| cdent | gibi: ^^ has the gabbi | 13:43 |
| gibi | cdent: thanks | 13:43 |
| * cdent looks at sql | 13:43 | |
| *** trinaths has left #openstack-nova | 13:45 | |
| *** Daisy_ has quit IRC | 13:45 | |
| *** acormier has joined #openstack-nova | 13:46 | |
| *** udesale has quit IRC | 13:47 | |
| *** acormier has quit IRC | 13:47 | |
| *** acormier has joined #openstack-nova | 13:48 | |
| mriedem_meeting | sdague: ok thanks | 13:48 |
| *** bertys has quit IRC | 13:48 | |
| gibi | cdent: your gabbi does what I wanted to do | 13:49 |
| *** burt has joined #openstack-nova | 13:49 | |
| *** hongbin has joined #openstack-nova | 13:49 | |
| cdent | gibi: I was hoping this fix might be related but it’s not: https://review.openstack.org/#/c/480379/ | 13:50 |
| *** tylerderosagrund has joined #openstack-nova | 13:51 | |
| *** kfarr has joined #openstack-nova | 13:52 | |
| gibi | cdent: we tried that patch yestarday and didn't help | 13:52 |
| *** shuangyangqian_ has joined #openstack-nova | 13:54 | |
| cdent | :) | 13:54 |
| *** jamesden_ has joined #openstack-nova | 13:56 | |
| *** shuangyangqian has quit IRC | 13:57 | |
| *** jamesdenton has quit IRC | 13:58 | |
| *** awaugama has joined #openstack-nova | 13:58 | |
| *** armax has joined #openstack-nova | 13:59 | |
| *** rmart04 has joined #openstack-nova | 14:00 | |
| *** mamandle has joined #openstack-nova | 14:01 | |
| *** sridharg has quit IRC | 14:01 | |
| *** alexchadin has quit IRC | 14:02 | |
| *** roeyc has quit IRC | 14:02 | |
| *** jwcroppe has joined #openstack-nova | 14:07 | |
| *** felipemonteiro__ has joined #openstack-nova | 14:08 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257 | 14:08 |
| *** swebster_ has joined #openstack-nova | 14:08 | |
| *** yamamoto has quit IRC | 14:09 | |
| *** swebster has quit IRC | 14:10 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:11 | |
| *** jaypipes has joined #openstack-nova | 14:12 | |
| efried | sdague ^^ ? | 14:12 |
| efried | sdague A rebase? | 14:14 |
| *** felipemonteiro__ has quit IRC | 14:14 | |
| sdague | efried: yeh, I was just looking at the priority etherpad and it said that needed rebasing | 14:16 |
| sdague | so I did | 14:16 |
| efried | sdague Thanks! | 14:16 |
| sdague | efried: you should check that I did it correctly :) | 14:16 |
| efried | sdague I'm gonna need to rework it as soon as ksa releases anyway. | 14:17 |
| efried | sdague The client consumption won't change, prolly just the get_service_url() method itself. | 14:17 |
| efried | But yeah, it'll get put through the ringer at that time. | 14:17 |
| *** links has quit IRC | 14:18 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493 | 14:18 |
| sdague | efried: ok cool | 14:18 |
| *** xyang1 has joined #openstack-nova | 14:19 | |
| *** mtanino_ has joined #openstack-nova | 14:21 | |
| *** mgiles has quit IRC | 14:24 | |
| *** mgiles has joined #openstack-nova | 14:24 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot https://review.openstack.org/484617 | 14:24 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair https://review.openstack.org/484726 | 14:29 |
| cdent | gibi: I added a bit more info https://bugs.launchpad.net/nova/+bug/1705231 | 14:31 |
| openstack | Launchpad bug 1705231 in OpenStack Compute (nova) "Placement returns no allocation candidate for request that needs both compute resources and custom shared resources" [High,Triaged] | 14:31 |
| *** roeyc has joined #openstack-nova | 14:32 | |
| cdent | jaypipes: that ^ probably needs your sql brain | 14:32 |
| *** mamandle_ has joined #openstack-nova | 14:32 | |
| gibi | cdent: thanks. I hope jaypipes can figure out the root cause | 14:33 |
| *** annegentle has joined #openstack-nova | 14:34 | |
| *** mamandle has quit IRC | 14:34 | |
| *** mamandle_ is now known as mamandle | 14:35 | |
| *** marst has joined #openstack-nova | 14:35 | |
| jaypipes | cdent, gibi: thanks. will look at it shortly, | 14:36 |
| cdent | thanks jaypipes | 14:36 |
| gibi | jaypipes: thanks | 14:36 |
| *** moshele has quit IRC | 14:36 | |
| *** smatzek has quit IRC | 14:37 | |
| *** Sukhdev has joined #openstack-nova | 14:39 | |
| *** gyee has joined #openstack-nova | 14:39 | |
| *** shuangyangqian_ has quit IRC | 14:40 | |
| *** moshele has joined #openstack-nova | 14:40 | |
| *** mgiles has quit IRC | 14:45 | |
| *** mgiles has joined #openstack-nova | 14:46 | |
| *** lucas-hungry is now known as lucasagomes | 14:47 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair https://review.openstack.org/484726 | 14:50 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Migrate Ironic Flavors https://review.openstack.org/484949 | 14:50 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair https://review.openstack.org/484726 | 14:54 |
| edleafe | dansmith: mriedem_meeting: cdent: ^^ Addressed the concerns | 14:54 |
| *** lyan has quit IRC | 14:55 | |
| * cdent looks | 14:57 | |
| *** mamandle has quit IRC | 14:57 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Open vSwitch patch port https://review.openstack.org/485228 | 14:57 |
| *** jjung has quit IRC | 14:57 | |
| sdague | sfinucan: confused on https://review.openstack.org/#/c/428241 - because I thought we were going the other direction | 15:02 |
| *** hoonetorg has quit IRC | 15:02 | |
| *** smatzek has joined #openstack-nova | 15:02 | |
| stephenfin | sdague: Oh, I wasn't aware of that | 15:03 |
| stephenfin | That changes things | 15:03 |
| sdague | see efried's work there | 15:03 |
| *** mriedem_meeting is now known as mriedem | 15:03 | |
| stephenfin | will do | 15:03 |
| *** jmlowe has quit IRC | 15:05 | |
| *** Oku_OS is now known as Oku_OS-away | 15:07 | |
| *** psachin has quit IRC | 15:08 | |
| *** yamamoto has joined #openstack-nova | 15:09 | |
| *** mgiles has quit IRC | 15:10 | |
| *** mgiles has joined #openstack-nova | 15:10 | |
| *** cfriesen has joined #openstack-nova | 15:11 | |
| mriedem | bauzas: ok https://review.openstack.org/#/c/484828/ | 15:12 |
| mriedem | aller! | 15:12 |
| bauzas | mriedem: so, I was thinking about accepting custom drivers if they use the placement API | 15:12 |
| bauzas | mriedem: I mean, if all our drivers are given a list of hosts by the placement API | 15:13 |
| bauzas | mriedem: the problem with the FilterScheduler driver is that it will call filters N times when you have N hosts | 15:13 |
| bauzas | mriedem: so I think some operators would like to just call once a 3rd-party system | 15:14 |
| bauzas | mriedem: if we do that, I think it would not be an interop problem given we'll be sure that any destination would be accepted by placement API | 15:14 |
| openstackgerrit | Feodor Tersin proposed openstack/nova master: libvirt: Straighten resize condition in Image.cache https://review.openstack.org/485236 | 15:14 |
| bauzas | of course, like I said, it means that all the drivers would be calling placement for both getting hosts to verify, and would also claim the allocation too | 15:15 |
| bauzas | thoughts about that? | 15:15 |
| mriedem | bauzas: i'm not sure what that has to do with the bug fix at hand | 15:15 |
| *** yamahata has joined #openstack-nova | 15:15 | |
| mriedem | which needs to go back to ocata where the custom scheduler driver loading was broken | 15:15 |
| bauzas | mriedem: so, about that | 15:15 |
| bauzas | mriedem: for the bug | 15:16 |
| bauzas | mriedem: just adding the new choice would be okay I think | 15:16 |
| *** kristian__ has quit IRC | 15:16 | |
| bauzas | mriedem: but not saying it would be deprecated by Queens IMHO | 15:16 |
| sean-k-mooney | bauzas: it would not be transparent on upgrade | 15:16 |
| bauzas | sean-k-mooney: not sure I understand why | 15:16 |
| *** kristian__ has joined #openstack-nova | 15:17 | |
| bauzas | I wouldn't deprecate a choice | 15:17 |
| bauzas | just adding a new one | 15:17 |
| sean-k-mooney | bauzas: there was nothing that requried them to call the custom_driver custom_driver in the setup.cfg entrypoint before | 15:17 |
| bauzas | the problem with upgrades is when you stop to use an opt | 15:17 |
| mriedem | bauzas: adding choices is not ok | 15:17 |
| mriedem | it's not backward compatible | 15:17 |
| mriedem | when upgrading *to* ocata | 15:17 |
| *** felipemonteiro_ has quit IRC | 15:18 | |
| mriedem | we have to drop 'choices', | 15:18 |
| bauzas | mriedem: not only then | 15:18 |
| *** lyan has joined #openstack-nova | 15:18 | |
| mriedem | and when loading up the entry point in the scheduler manager, if it's not in our whitelist of 'choices' that we support today, then we log a warning | 15:18 |
| bauzas | mriedem: because if you upgrade from Newton, then you need to modify your nova.conf because the value wouldn't be the same | 15:19 |
| bauzas | mriedem: in Newton, you needed to pass a python path | 15:19 |
| sean-k-mooney | bauzas: you could not use the alis form stevador? | 15:19 |
| bauzas | mriedem: in Ocata, you need to pass just a name for an existing entrypoint | 15:19 |
| *** yamamoto has quit IRC | 15:19 | |
| *** liangy has joined #openstack-nova | 15:21 | |
| sean-k-mooney | bauzas: i guess the point still stands if we keep the choices field we would need a config change on upgrade and a change to the python module that exports the driver to export it as custom_driver | 15:21 |
| mriedem | bauzas: i don't think that's right | 15:21 |
| mriedem | in mitaka we deprecated the ability to classload and removed that in ocata | 15:21 |
| mriedem | and replaced with stevedore entrypoint aliases | 15:22 |
| mriedem | but still allows you to add your own stevedore entrypoint to load from | 15:22 |
| mriedem | that was until we added that choices kwarg and broke everything | 15:22 |
| mriedem | i can do some git history diving | 15:22 |
| mriedem | https://github.com/openstack/nova/commit/fe3d6dba3d1db8f97dab4baffcc11dda56368096 | 15:23 |
| *** lyan has quit IRC | 15:23 | |
| sean-k-mooney | stevedore also allows you to skip using the alais and use the class path it points to transparently so if the code was not moved the old config options would have worked | 15:23 |
| bauzas | mmm | 15:23 |
| mriedem | that merged on sept 22 | 15:24 |
| mriedem | https://review.openstack.org/#/c/349666/ introduced the choices kwarg which broke it | 15:24 |
| mriedem | that merged on sept 30 | 15:24 |
| bauzas | mriedem: so, to make it clear, if you were newton | 15:24 |
| bauzas | mriedem: and you didn't use the default value | 15:24 |
| bauzas | mriedem: you were passing a python path for knowing which module to use, right? | 15:24 |
| mriedem | not necessarily | 15:25 |
| mriedem | you could have already been using a stevedore entry | 15:25 |
| mriedem | https://review.openstack.org/#/c/339760/8/nova/scheduler/manager.py | 15:25 |
| *** gbarros has quit IRC | 15:26 | |
| TheJulia | mriedem: apologies for the minor interrupt, but I and dtantsur replied to https://review.openstack.org/#/c/215385/ which hopefully will clear up the confusion. | 15:26 |
| mriedem | ^ before that change, it would attempt to load from stevedore and failing that, try to load from classpath | 15:26 |
| bauzas | mriedem: ah-ha, nevermind my comment because https://github.com/openstack/nova/blob/fe3d6dba3d1db8f97dab4baffcc11dda56368096/releasenotes/notes/sched_remove_classpath_import-5d0f48eb388e6948.yaml | 15:27 |
| bauzas | mriedem: which is Newton | 15:28 |
| bauzas | mriedem: so I was wrong and you were right | 15:28 |
| bauzas | mriedem: okay, I'll then just remove the choices kwarg | 15:28 |
| bauzas | mriedem: the big question being whether we want to deprecate the possibility to use a custom scheduler by queens | 15:28 |
| sdague | bauzas / cdent - there a race with setting up placement/scheduler fixture it seems - http://logs.openstack.org/27/479027/1/gate/gate-nova-tox-functional-py35-ubuntu-xenial/0a995a0/console.html#_2017-07-19_14_33_46_324724 | 15:28 |
| sdague | which gives us novalid host on some functional tests | 15:28 |
| bauzas | arf, I have a meeting to run | 15:29 |
| cdent | sdague: there’s been quite a bit of kerfuffle with the compute api and the placement api fixtures not playing well together in the same tests. for a while there was some thought that it was eventlet related, but I’m not sure where that landed. mriedem may have more info? | 15:30 |
| mriedem | bauzas: https://github.com/openstack/nova/blob/fe3d6dba3d1db8f97dab4baffcc11dda56368096/releasenotes/notes/sched_remove_classpath_import-5d0f48eb388e6948.yaml is not newton | 15:30 |
| mriedem | https://review.openstack.org/#/c/339760/ | 15:31 |
| *** coreywright has joined #openstack-nova | 15:31 | |
| mriedem | Branches master, stable/ocata Tags 15.0.0, 15.0.0.0b1, 15.0.0.0b2, 15.0.0.0b3, 15.0.0.0rc1, 15.0.0.0rc2, 15.0.1, 15.0.2, 15.0.3, 15.0.4, 15.0.5, 15.0.6, 16.0.0.0b1, 16.0.0.0b2 | 15:31 |
| sdague | cdent: for things like that we could probably make super fake placement that says "thumbs up", given that we're running 1 compute | 15:31 |
| *** jmlowe has joined #openstack-nova | 15:32 | |
| mriedem | yes i've seen the timeout exception holding the lock in the compute update_available_resource | 15:32 |
| cdent | yeah, that’s probably a good idea | 15:32 |
| bauzas | mriedem: it's ocata, so anyway you have to change that | 15:32 |
| *** gcb has joined #openstack-nova | 15:32 | |
| *** randomhack has quit IRC | 15:32 | |
| mriedem | the PlacementFixture was turned on relatively global for functional tests recently | 15:32 |
| mriedem | so it's running in a lot more functional tests, whether we need it to or not | 15:32 |
| mriedem | sdague: semi related we also did this globally https://review.openstack.org/#/c/483972/ | 15:32 |
| mriedem | we didn't have a fingerprint for it, but it seems to have helped a little bit from anecdotal evidence, meaning me not having to recheck on weird wsgi errors as much | 15:33 |
| sdague | mriedem: yeh, I think I'd stop doing placement globally, that seems like a lot of complexity when you don't need it | 15:33 |
| *** ftersin has joined #openstack-nova | 15:35 | |
| *** tbachman has quit IRC | 15:35 | |
| *** tbachman_ has joined #openstack-nova | 15:35 | |
| mriedem | TheJulia: does a 'vendor driver' in ironic mean an out of tree driver? | 15:35 |
| *** ftersin has quit IRC | 15:36 | |
| *** ftersin has joined #openstack-nova | 15:36 | |
| mriedem | if the limitation is on fibrechannel support, then why not just say, iscsi is currently the only supported volume type | 15:36 |
| ftersin | mdbooth: hi. could you please review that (https://review.openstack.org/#/c/407440/) again? i addressed your comments and wait for new ones. | 15:37 |
| *** Apoorva has joined #openstack-nova | 15:38 | |
| *** chyka has joined #openstack-nova | 15:38 | |
| *** Apoorva_ has joined #openstack-nova | 15:38 | |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 15:39 |
| *** Sukhdev_ has joined #openstack-nova | 15:39 | |
| *** gyee has quit IRC | 15:40 | |
| TheJulia | mriedem: anything produced by a vendor, so it could be in or out of tree, in the terms I wrote, I meant in-tree vendor drivers. | 15:40 |
| mriedem | ok, not sure why 'vendor' is even a distinction then | 15:41 |
| TheJulia | mriedem: I'd prefer to just drop the tag completely due to confusion at this point | 15:41 |
| mriedem | what's a non-vendor driver in ironic? | 15:41 |
| mriedem | the ironic coop driver :) | 15:41 |
| TheJulia | mriedem: agent_ipmitool, pxe_ipmitool, or the ipmi, redfish hardware types | 15:41 |
| mriedem | bm traded for hemp | 15:41 |
| mriedem | ok | 15:41 |
| TheJulia | lol | 15:41 |
| mriedem | ok, so if the iscsi volume type is the only one that's currently supported, that's what i'd clarify in the support matrix and release note | 15:42 |
| *** Apoorva has quit IRC | 15:42 | |
| jaypipes | jangutter: 483921 approved. | 15:42 |
| *** gyee has joined #openstack-nova | 15:42 | |
| TheJulia | mriedem: those are the drivers we're able to test in the gate since we can emulate the BMC of baremetal hardware in those cases. | 15:42 |
| *** rmart04 has quit IRC | 15:43 | |
| TheJulia | mriedem: okay, I just want to make sure or at least hopefully make sure nobody freaks out when we ask to amend the support matrix later without putting code in nova :) | 15:43 |
| jangutter | jaypipes, sean-k-mooney: the people in the office just looked at me really oddly. I don't think they expected that kind of sound emitting from human vocal cords. | 15:44 |
| *** _ix has joined #openstack-nova | 15:44 | |
| TheJulia | mriedem: that being, when we have that support in place in our reference drivers | 15:44 |
| jaypipes | jangutter: :) | 15:44 |
| _ix | Good morning folks. I'm having a heck of a time migrating a host after a hypervisor failure. | 15:44 |
| mriedem | TheJulia: meaning you can do this with other volume types | 15:44 |
| mriedem | like fibrechannel | 15:44 |
| mriedem | right? | 15:44 |
| *** randomhack has joined #openstack-nova | 15:44 | |
| *** crushil has quit IRC | 15:44 | |
| _ix | Waiting some 30 minutes for a single migration only to find an unsatisfying 401 error in the logs. | 15:44 |
| mriedem | TheJulia: i think when there is >1 supported type, we'll just drop the note from the support matrix | 15:45 |
| mriedem | _ix: token timeout? | 15:45 |
| mriedem | cold or live migration? | 15:45 |
| _ix | Cold. | 15:45 |
| mriedem | if it's a cold migration of a large image, then the token probably timed out | 15:45 |
| mriedem | which you should find in the nova-compute logs | 15:45 |
| TheJulia | mriedem: yes, I'll revise that patch then in the next little bit to pull out the fc support in the matrix | 15:46 |
| *** rajathagasthya has joined #openstack-nova | 15:46 | |
| mriedem | TheJulia: ok there were other issues in there i pointed out, like the versioning thing, but i assume you're on top of all that | 15:46 |
| jaypipes | TheJulia: don't forget to take the red pill. | 15:46 |
| mriedem | _ix: fyi https://specs.openstack.org/openstack/nova-specs/specs/pike/approved/use-service-tokens.html | 15:46 |
| _ix | mriedem: I'll take a look. Thanks for the direction! | 15:46 |
| TheJulia | mriedem: in the reno? | 15:46 |
| sean-k-mooney | jaypipes: jangutter i think https://review.openstack.org/#/c/485125 is the last patch that is needed before we can make the os-vif release | 15:46 |
| mriedem | _ix: in ocata, you could configure nova to provide a service token for long-running tasks that involved cinder or neutron, | 15:47 |
| jaypipes | sean-k-mooney: yup, got that up in gertty now. | 15:47 |
| mriedem | that was expanded to glance in pike | 15:47 |
| TheJulia | jaypipes: I took the red pill a long time ago. | 15:47 |
| jaypipes | :) | 15:47 |
| _ix | mriedem: This is... Juno. | 15:47 |
| mriedem | _ix: well then you're f'ed :) | 15:47 |
| _ix | Ha! | 15:47 |
| mriedem | _ix: increase the token timeout i guess? | 15:47 |
| mriedem | idk | 15:47 |
| _ix | Is that in the nova.conf of the losing host? | 15:47 |
| mriedem | yeah | 15:48 |
| mriedem | but that service token stuff relies on things in keystone too | 15:48 |
| mriedem | so you likely can't backport any of that to juno | 15:48 |
| mriedem | TheJulia: yeah, >= 8.0 is wrong | 15:48 |
| TheJulia | mriedem: any objection to just saying pike release? | 15:49 |
| *** damien_r has quit IRC | 15:49 | |
| mriedem | ironic doesn't have a "pike" release does it? i thought ironic was too cool for such naming and boundaries | 15:49 |
| mriedem | hence the release with intermediary tag | 15:49 |
| _ix | mriedem: Is an 80gb ephemeral considered large? | 15:50 |
| mriedem | _ix: 80gb is pretty large | 15:50 |
| _ix | I suspected as much. Seems like the 20gb ephemerals migrated without issue. | 15:50 |
| jangutter | sean-k-mooney, jaypipes: thanks. I think it's wise for the Mellanox CI to have the final say. | 15:51 |
| TheJulia | mriedem: well, we've not cut 9.0 yet, so saying 9.0.0 is technically invalid as well | 15:51 |
| mriedem | jaypipes: you have a handle on https://review.openstack.org/#/c/483565/ today? | 15:51 |
| mriedem | TheJulia: that's why i said, >8.0 | 15:51 |
| TheJulia | ahh, yeah, that actually works | 15:51 |
| mriedem | if there is 8.0.1 or 8.1 then idk | 15:52 |
| TheJulia | because there is not an 8.x | 15:52 |
| *** gyee has quit IRC | 15:52 | |
| mriedem | ok | 15:52 |
| * TheJulia goes and does --ff-only on her local copy of nova | 15:52 | |
| *** vishwana_ has joined #openstack-nova | 15:52 | |
| jaypipes | mriedem: yeah. | 15:53 |
| jaypipes | TheJulia: do you ever wish there was a --ffs-only option? :) | 15:53 |
| openstackgerrit | Merged openstack/os-vif master: Add support for VIFPortProfileOVSRepresentor https://review.openstack.org/483921 | 15:53 |
| TheJulia | jaypipes: many times | 15:54 |
| _ix | mriedem: I'm not seeing anything regarding timeout options for keystone_authtoken. Any other ideas? | 15:54 |
| *** gcb has quit IRC | 15:54 | |
| TheJulia | jaypipes: only reason I don't have an alias to bring the repo to the current state of master is because I may loose my current mental state of where things are at | 15:54 |
| ralonsoh | stephenfin: hi. Can you take a look at https://review.openstack.org/#/c/449257/ and https://review.openstack.org/#/c/451777/? It could be a pity not to have this for Pike | 15:54 |
| jaypipes | TheJulia: yup, same. | 15:55 |
| _ix | mriedem: It also has a large volume attached in addition. Perhaps I need to detach before attempting to migrate? | 15:55 |
| *** tjones has joined #openstack-nova | 15:55 | |
| *** gcb has joined #openstack-nova | 15:55 | |
| *** crushil has joined #openstack-nova | 15:55 | |
| *** crushil has quit IRC | 15:55 | |
| mriedem | _ix: the volume should be fine | 15:56 |
| *** crushil has joined #openstack-nova | 15:56 | |
| *** vishwanathj has quit IRC | 15:56 | |
| mriedem | nova will detach the volume on the source host and attach it on the target host | 15:56 |
| _ix | Maybe a red herring, but the 401 appears to becoming from cinder? | 15:56 |
| mriedem | _ix: at this point debugging a juno problem is beyond the realm of time i can spend on this | 15:57 |
| _ix | mriedem: I understand. I appreciate the time you did give me. | 15:57 |
| mriedem | what i can tell you is, | 15:57 |
| mriedem | if the call is coming from within the house, get out | 15:57 |
| *** tjones has quit IRC | 15:57 | |
| _ix | To be fair, we're preparing for an upgrade in the coming months! | 15:57 |
| mriedem | good luck | 15:57 |
| _ix | Thanks. | 15:58 |
| jaypipes | edleafe: hi! would you mind providing your opinion on ralonsoh's os-traits patch here? https://review.openstack.org/#/c/483451 | 15:58 |
| jaypipes | edleafe: I suppose I could go either way, but would appreciate another opinion. | 15:58 |
| jaypipes | alex_xu: you too ^^ | 15:58 |
| edleafe | jaypipes: ok, will look in a bit | 15:59 |
| jaypipes | edleafe: cheers | 15:59 |
| cdent | can I look too jaypipes or is it seeekrit? | 16:00 |
| jaypipes | cdent: of course! | 16:01 |
| cdent | ;) | 16:01 |
| mriedem | cdent: not sure if you saw my comment on https://review.openstack.org/#/c/484162/ | 16:03 |
| mriedem | i tried backporting to ocata and the test needs some changes there | 16:03 |
| mriedem | since we don't have 1.7 | 16:03 |
| *** dtantsur is now known as dtantsur|afk | 16:03 | |
| mriedem | i got past that but there was one more failure in the allocation request which i couldn't dig into | 16:04 |
| cdent | mriedem: i did yeah, but haven’t had a chance to react. if you have some wip code and want me to pick it up I can do that | 16:04 |
| *** aarefiev is now known as aarefiev_afk | 16:04 | |
| mriedem | i might, se | 16:04 |
| cdent | I also so your comments on the dansmith’s functional test now being a real failure. I’m not sure if I own that by default now or if dan wishes to return to that. either is fine with me. | 16:05 |
| cdent | s/so/saw/ | 16:05 |
| *** felipemonteiro has joined #openstack-nova | 16:05 | |
| *** Apoorva_ has quit IRC | 16:05 | |
| mriedem | let's just ignore it for now | 16:06 |
| mriedem | cdent: https://review.openstack.org/485263 | 16:06 |
| cdent | cool, will inspect, thanks | 16:06 |
| *** felipemonteiro_ has joined #openstack-nova | 16:07 | |
| *** trinaths has joined #openstack-nova | 16:07 | |
| mriedem | bauzas: it would be good if you could start going through this series https://review.openstack.org/#/c/408955/ - takashin has been really patient with rebases and that's been around a few releases now, but hasn't gotten review | 16:08 |
| *** rcernin has quit IRC | 16:09 | |
| bauzas | mriedem: I did it a couple of times but I can do other cycles, for sure | 16:10 |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/484218 | 16:10 |
| *** felipemonteiro has quit IRC | 16:11 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Accept any scheduler driver entrypoint https://review.openstack.org/484828 | 16:11 |
| bauzas | mriedem: sean-k-mooney: ^ | 16:11 |
| *** Apoorva has joined #openstack-nova | 16:11 | |
| bauzas | mriedem: sean-k-mooney english grammar nits welcome | 16:12 |
| *** sree has quit IRC | 16:12 | |
| *** liangy has quit IRC | 16:12 | |
| *** sree has joined #openstack-nova | 16:13 | |
| bauzas | jianghuaw: thanks for updating https://review.openstack.org/#/c/450122/20/specs/queens/approved/virt-add-support-for-vgpu.rst | 16:13 |
| bauzas | jianghuaw: jaypipes: I had thoughts on that whether it was requiring nested RPs | 16:14 |
| jaypipes | bauzas: yes, it does. | 16:14 |
| bauzas | jianghuaw: jaypipes: tbh, I don't think we really need to wait for nested RPs if we can use custom RCs | 16:14 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Migrate Ironic Flavors https://review.openstack.org/484949 | 16:14 |
| bauzas | lemme explain | 16:14 |
| edleafe | cdent: Fixed the warning ^^ | 16:15 |
| bauzas | jaypipes: if we say that drivers can provide their own resource classes | 16:15 |
| bauzas | jaypipes: and if operators provide custom RCs in flavors for defining how many "foos" we need to consume | 16:15 |
| bauzas | jaypipes: why should we then requiring a parent/child relationship ? | 16:16 |
| bauzas | for example, libvirt is not having a tight parent/child relationship | 16:16 |
| bauzas | it's just providing a list of mediated devices having types | 16:16 |
| jaypipes | bauzas: that's fine for a non-interoperable solution of course. | 16:17 |
| jaypipes | bauzas: as soon as you have custom anything, you throw interop out the window | 16:17 |
| bauzas | jaypipes: I don't disagree with your statement | 16:17 |
| bauzas | jaypipes: I'm just saying that we have all the tooling in place for the feature | 16:17 |
| bauzas | nested resource providers will give us more | 16:18 |
| sean-k-mooney | bauzas: haha the only grammer/spelling corrections that i usally give are wrong. but ill take a look | 16:18 |
| mriedem | bauzas: rather than rely on extra specs, | 16:18 |
| *** sree has quit IRC | 16:18 | |
| mriedem | we will need to focus priority on actually getting nested RPs done in quens | 16:18 |
| mriedem | for vgpus support | 16:18 |
| jaypipes | bauzas: jianghuaw and I were trying to get GPU traits standardized in os-traits. once we have standard traits, you need resource providers to associate those traits to. and since different vGPU providers (pGPU groups) can have different traits, you need a way of saying "this pGPU group has these traits and this pGPU group has those traits". without nested providers, you can't do that. | 16:18 |
| mriedem | and fpga and everything else | 16:18 |
| *** annegentle has quit IRC | 16:19 | |
| bauzas | jaypipes: pGPU group is only a xen thing | 16:19 |
| bauzas | mriedem: again, I don't disagree | 16:19 |
| mriedem | bauzas: then why even bring it up? | 16:19 |
| jaypipes | bauzas: or physical GPU... if you have >1 on a compute node. | 16:19 |
| openstackgerrit | Julia Kreger proposed openstack/nova master: Ironic: Support boot from Cinder volume https://review.openstack.org/215385 | 16:19 |
| bauzas | mriedem: I just feel that the vGPU spec doesn't require it | 16:19 |
| bauzas | jaypipes: no | 16:19 |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 16:19 |
| mriedem | bauzas: there are lots of things that can be implemented the wrong way with extra specs | 16:20 |
| mriedem | that doesn't mean we should do them | 16:20 |
| sean-k-mooney | bauzas: so that version is effectivly a straight revert of just the incidental change so that should work well for the backport. | 16:20 |
| bauzas | mriedem: the spec itself uses extra specs for traits, y'know :) | 16:21 |
| cdent | presumably if we do something the quick and dirty way now, we have to allow that way to be present until the end times (or thereabouts) and we’d rather just do it right | 16:21 |
| bauzas | mriedem: AFAIK, the only possibility we'll get for asking qualititative resource types will be by using extra specs, correct? | 16:22 |
| bauzas | mriedem: and the same goes for some quantitative resources, if we speak of custom resource classes, right? | 16:22 |
| bauzas | after all, we implemented the custom resource classes series because we knew that we needed some flexible model | 16:23 |
| *** rcernin has joined #openstack-nova | 16:23 | |
| *** lucasagomes is now known as lucas-afk | 16:24 | |
| *** moshele has quit IRC | 16:24 | |
| mriedem | we talked about putting something into the api for qualitative things at the newton midcycle | 16:25 |
| mriedem | that didn't involve extra specs | 16:25 |
| mriedem | but that would all have to be flushed out | 16:25 |
| mriedem | unrelated, -1 on takashi's bottom change https://review.openstack.org/#/c/408955/ since it doesn't account for not allowing forced cold migrations between cells | 16:26 |
| mriedem | but there is all sorts of request spec and filter properties goodies in there for you to look at | 16:26 |
| bauzas | mriedem: I remember about that API | 16:27 |
| bauzas | mriedem: after all, the spec from jianghuaw is just an other usecase of how to use placement, right? | 16:27 |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 16:28 |
| sean-k-mooney | mriedem: you can migrate between cells? i taught you were normally limited to mightions within the same availablity zone. can you disable that check with a force flage? | 16:28 |
| bauzas | mriedem: most of the spec is describing not some implementation proposal, just the workflow how it should work | 16:28 |
| bauzas | mriedem: jaypipes: the only bits requiring implementation is internal to the drivers, right? | 16:29 |
| bauzas | given we already have the virt driver interface for all of that | 16:29 |
| openstackgerrit | Merged openstack/nova master: api-ref: Verify parameters in os-migrations.inc https://review.openstack.org/474879 | 16:29 |
| *** tylerderosagrund has quit IRC | 16:29 | |
| jaypipes | bauzas: sorry, reading back, was on a call | 16:30 |
| openstackgerrit | Merged openstack/nova master: Stop using mox stubs in test_remote_consoles.py https://review.openstack.org/478423 | 16:30 |
| mriedem | sean-k-mooney: you can't migrate between cells | 16:30 |
| sean-k-mooney | mriedem: oh sorry missread your comment. https://review.openstack.org/#/c/408955 currently allows it when it should not | 16:32 |
| jaypipes | bauzas: so, the vgpu spec indeed mentions traits in extra specs, but it doesn't mention *custom* traits or *custom* resource classes :) jianghuaw has been good about pushing for standardized resource classes and traits. | 16:33 |
| mriedem | sean-k-mooney: correct | 16:34 |
| *** roeyc has quit IRC | 16:34 | |
| *** david-lyle has quit IRC | 16:34 | |
| mriedem | jaypipes: bauzas: let's leave the vgpus stuff for after FF please | 16:36 |
| mriedem | ffs | 16:36 |
| bauzas | mriedem: jaypipes: sure didn't really wanted to make it clearly for now | 16:36 |
| bauzas | just sharing thoughts | 16:37 |
| *** gszasz has quit IRC | 16:37 | |
| *** Apoorva has quit IRC | 16:37 | |
| cdent | mriedem: I think I fixed https://review.openstack.org/#/c/485263/ | 16:38 |
| sean-k-mooney | jangutter: can you click the rebase button on https://review.openstack.org/#/c/485125 so that the commit id wont change when it merges | 16:39 |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 16:40 |
| sean-k-mooney | jangutter: im going to create a patch to the release repo to tag that commit for the 1.6.1 release of os-vif | 16:40 |
| jangutter | sean-k-mooney: done | 16:40 |
| sean-k-mooney | jangutter: cool ill just check locally but now it should be able to merge without a merge commit being created | 16:41 |
| *** mgiles has quit IRC | 16:44 | |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 16:44 |
| *** mgiles has joined #openstack-nova | 16:44 | |
| ralonsoh | cdent, edleafe: thanks for your reviews in https://review.openstack.org/#/c/483451/. I posted my reply | 16:47 |
| *** crushil has quit IRC | 16:47 | |
| melwitt | counting instances is in the gate \o/ \o/ \o/ | 16:48 |
| mriedem | f yeah | 16:49 |
| mriedem | melwitt: are you going to cleanup the 'remove refresh-quotas cli' change todya? | 16:49 |
| mriedem | if not, i can whack at that | 16:49 |
| *** gcb has quit IRC | 16:50 | |
| melwitt | mriedem: yes, gonna hit that next | 16:50 |
| mriedem | awesome | 16:50 |
| *** sree has joined #openstack-nova | 16:50 | |
| *** mvk has quit IRC | 16:51 | |
| mriedem | sheet, cellsv2 meeting in 9 minutes | 16:51 |
| mriedem | dansmith: you want me to run it? | 16:51 |
| *** david-lyle has joined #openstack-nova | 16:52 | |
| dansmith | ugh | 16:52 |
| dansmith | mriedem: melwitt do we have things to talk about other than the obvious/ | 16:52 |
| dansmith | obvious being "get the quotas shit merged" | 16:52 |
| mriedem | i don't have anythign that's not in the etherpad | 16:52 |
| mriedem | https://etherpad.openstack.org/p/nova-pike-cells-v2-todos | 16:53 |
| melwitt | heh, yeah. I don't have anything else at the moment | 16:53 |
| mriedem | the uuids apis are +W now too | 16:53 |
| *** randomhack has quit IRC | 16:53 | |
| melwitt | today is the best day | 16:54 |
| *** sree has quit IRC | 16:54 | |
| dansmith | let's just punt then because I'm covered up in other stuff | 16:55 |
| dansmith | except not so much | 16:55 |
| * dansmith is running packstack for mitaka | 16:55 | |
| *** randomhack has joined #openstack-nova | 16:56 | |
| melwitt | sounds cool | 16:56 |
| dansmith | it's so not | 16:57 |
| melwitt | I mean punting sounds cool | 16:58 |
| melwitt | :) | 16:58 |
| mriedem | packstack on mitaka sounds f'ing awesome | 16:58 |
| mriedem | like crowbar on folsom | 16:58 |
| melwitt | I ran packstack for kilo a few days ago and ... it worked | 16:58 |
| *** efried has quit IRC | 16:59 | |
| *** crushil has joined #openstack-nova | 17:00 | |
| *** acormier_ has joined #openstack-nova | 17:01 | |
| *** derekh has quit IRC | 17:02 | |
| *** acormier has quit IRC | 17:04 | |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 17:06 |
| mriedem | sdague: bauzas: jaypipes: easy one to keep quotas series going https://review.openstack.org/#/c/446242/ | 17:08 |
| bauzas | I saw the big quotas patch in the gate | 17:08 |
| bauzas | cool | 17:08 |
| melwitt | yaaay | 17:10 |
| *** efried has joined #openstack-nova | 17:11 | |
| openstackgerrit | John Haan proposed openstack/nova-specs master: Support volume_type with BDM paramter https://review.openstack.org/466595 | 17:13 |
| bauzas | mriedem: since we didn't added a relnote for https://review.openstack.org/#/c/416521/63 do you think we should at least add a note saying that the 'reserved' QS is no longer accepted in Pike as per https://review.openstack.org/#/c/446242/41 ? | 17:14 |
| bauzas | dansmith: melwitt: ^ | 17:14 |
| mriedem | bauzas: it is accepted | 17:14 |
| mriedem | it's just ignored | 17:14 |
| mriedem | the end cloud user isn't going to read the release notes, they are going to read the api reference | 17:15 |
| mriedem | and the api doesn't change | 17:15 |
| bauzas | mriedem: sure, I meant telling people that if they use it, it won't no longer be something we check | 17:15 |
| *** Apoorva has joined #openstack-nova | 17:15 | |
| bauzas | meh, good point | 17:15 |
| mriedem | no, that's just noise imo | 17:15 |
| bauzas | it's more an API doc | 17:15 |
| dansmith | and we're not reserving so ignoring it is reasonable right? | 17:15 |
| mriedem | yeah this is all internal tomfoolery | 17:15 |
| mriedem | as i noted in there, i'll propose removing that query parameter with a microversion in queens | 17:16 |
| dansmith | it's noise, IMHO | 17:16 |
| dansmith | reserved amounts are transient | 17:16 |
| dansmith | if we improved boot performance, you'd see the same behavior | 17:16 |
| *** acormier_ has quit IRC | 17:18 | |
| *** acormier has joined #openstack-nova | 17:18 | |
| bauzas | mriedem: dansmith: can't find the API reference for that API endpoint FWIW | 17:19 |
| bauzas | am I blind ? | 17:19 |
| mriedem | limits | 17:19 |
| mriedem | another way to think of that is if the cloud used the noop quota driver, | 17:19 |
| melwitt | the used limits API is read only, it's just showing "reserved" usage | 17:19 |
| mriedem | there wouldn't be reservations | 17:20 |
| mriedem | https://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail | 17:20 |
| bauzas | https://developer.openstack.org/api-ref/compute/#show-rate-and-absolute-limits | 17:20 |
| mriedem | https://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail#limits-limits | 17:20 |
| bauzas | yeah | 17:20 |
| bauzas | so my question is, should we take the opportunity to remove 'reserved' from the API doc $ | 17:20 |
| bauzas | ^ | 17:20 |
| mriedem | no | 17:21 |
| *** imacdonn has quit IRC | 17:21 | |
| mriedem | because the api doc is true for liberty deployments | 17:21 |
| mriedem | which aren't using counting quotas | 17:21 |
| melwitt | yeah, like I thought that would require a microversion to just remove it from the API | 17:21 |
| bauzas | oh right | 17:21 |
| *** imacdonn has joined #openstack-nova | 17:21 | |
| bauzas | we would need to microversion | 17:21 |
| melwitt | since we're not reserving, "reserved" is 0 technically | 17:21 |
| mriedem | jesus yes that's what i said in the patch itself | 17:22 |
| melwitt | lol | 17:22 |
| jaypipes | mriedem: done. | 17:22 |
| mriedem | https://review.openstack.org/#/c/446242/41/nova/api/openstack/compute/used_limits.py@a65 | 17:22 |
| bauzas | argh, jaypipes beated me up | 17:23 |
| *** gbarros has joined #openstack-nova | 17:23 | |
| *** gyee has joined #openstack-nova | 17:23 | |
| bauzas | mriedem: melwitt: for my knowledge, when did we stopped using that flag? | 17:23 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 17:23 |
| mriedem | https://review.openstack.org/#/c/416521/ | 17:23 |
| mriedem | in there ^ | 17:23 |
| mriedem | the last reservable resource was instances, | 17:24 |
| mriedem | and that change changes instnaces from a reservable resource to a countable resource | 17:24 |
| melwitt | I don't understand how we're not using it, we're just correctly saying it's zero | 17:24 |
| mriedem | hence reserved is no longer used | 17:24 |
| mriedem | melwitt: i think he's asking which change made reserved vestigial | 17:24 |
| melwitt | instead of letting it potentially show old unused reservations | 17:24 |
| mriedem | it's this gd beast https://review.openstack.org/#/c/416521/63/nova/quota.py | 17:24 |
| openstackgerrit | Merged openstack/nova master: Correct the description of 'disable-log-reason' api-ref https://review.openstack.org/480209 | 17:24 |
| melwitt | oh | 17:24 |
| bauzas | yeah just saw https://review.openstack.org/#/c/416521/63/nova/quota.py@139 | 17:25 |
| *** harlowja has joined #openstack-nova | 17:25 | |
| bauzas | melwitt: yeah, what mriedem said | 17:25 |
| *** harlowja has quit IRC | 17:25 | |
| melwitt | I guess we could leave it there and it would still be 0 because in nova/quota.py reserved is hardcoded to 0 | 17:26 |
| bauzas | melwitt: I just wondered when we began ignoring that API flag | 17:26 |
| *** harlowja has joined #openstack-nova | 17:26 | |
| bauzas | I'm definitely not a quota expert :( | 17:26 |
| melwitt | with each resource change from ReservableResource -> CountableResource in nova/quota.py we return reserved=0 for that resource since reserving is no longer possible | 17:27 |
| bauzas | anyway, time to bail out \o | 17:28 |
| melwitt | bah, counting instances failed in the gate on yet another unrelated func test "Authentication error" | 17:31 |
| dansmith | well if someting in front of it fails that test it'll get another go | 17:31 |
| openstackgerrit | Merged openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757 | 17:31 |
| melwitt | oooh | 17:32 |
| openstackgerrit | Merged openstack/nova master: Do not mention that tags are case sensitive in docs https://review.openstack.org/484957 | 17:32 |
| *** lpetrut_ has quit IRC | 17:33 | |
| openstackgerrit | Merged openstack/nova master: XenAPI: Fix ValueError in test_slave_asks_master_to_add_slave_to_pool https://review.openstack.org/484201 | 17:33 |
| openstackgerrit | Merged openstack/nova master: Convert HostState.limits['numa_topology'] to primitive https://review.openstack.org/484439 | 17:34 |
| openstackgerrit | Merged openstack/nova master: api-ref: fix max_version for deprecated os-quota-class-sets parameters https://review.openstack.org/484952 | 17:35 |
| dansmith | okay not likely now :) | 17:35 |
| *** annegentle has joined #openstack-nova | 17:38 | |
| * melwitt plays sad trombone | 17:39 | |
| mriedem | at least your patch got into the gate, | 17:39 |
| mriedem | i can't get https://review.openstack.org/#/c/464280/ out of check queue purgatory | 17:39 |
| melwitt | true. | 17:40 |
| *** gyee has quit IRC | 17:42 | |
| *** itlinux has joined #openstack-nova | 17:42 | |
| *** ralonsoh has quit IRC | 17:42 | |
| *** deep-book-gk_ has joined #openstack-nova | 17:44 | |
| *** fragatina has quit IRC | 17:44 | |
| *** gyee has joined #openstack-nova | 17:44 | |
| *** gyee has quit IRC | 17:44 | |
| *** bbansal has joined #openstack-nova | 17:45 | |
| bbansal | Hello All, anyone here can help me : how can I disable genration of n-dhcp logs ? | 17:45 |
| *** deep-book-gk_ has quit IRC | 17:46 | |
| *** mvk has joined #openstack-nova | 17:46 | |
| *** tesseract has quit IRC | 17:46 | |
| *** ckopper has joined #openstack-nova | 17:47 | |
| efried | bbansal symlink it to /dev/null? | 17:49 |
| efried | bbansal Set the log level to CRITICAL? | 17:49 |
| bbansal | I have devstack running as a part of CI. It generates n-dhcp log and its a symlink. At the end of the process, when we are trying to copy it to our ftp server, it fails because there is no such file there | 17:50 |
| mriedem | you mean q-dhcp? | 17:51 |
| bbansal | i am looking for n-dhcp | 17:51 |
| bbansal | q-dhcp logs are fine | 17:51 |
| mriedem | there is no n-dhcp | 17:52 |
| bbansal | n-dhcp.log -> n-dhcp.log.2017-07-18-233804 | 17:52 |
| bbansal | it generates like this | 17:52 |
| bbansal | i can see these logs in my logs directory | 17:53 |
| *** randomhack has quit IRC | 17:53 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Migrate Ironic Flavors https://review.openstack.org/484949 | 17:53 |
| bbansal | mriedem: any insights> | 17:54 |
| bbansal | ? | 17:54 |
| *** yamahata has quit IRC | 17:54 | |
| mriedem | there is no n-dhcp in devstack | 17:55 |
| mriedem | on master | 17:55 |
| mriedem | oh hold on | 17:55 |
| bbansal | sure | 17:56 |
| *** trinaths has left #openstack-nova | 17:56 | |
| mriedem | i see it | 17:56 |
| mriedem | https://github.com/openstack-dev/devstack/blob/master/lib/nova#L544-L554 | 17:56 |
| mriedem | that should only be for running nova-network i think | 17:57 |
| mriedem | sdague: ^? | 17:57 |
| bbansal | yeah and we dont need these and currently they are failing our builds somehow even if all tempest cases are passing | 17:57 |
| *** sshwarts has quit IRC | 17:57 | |
| mriedem | yeah it's a bug in devstack, it shouldn't be running that if you're using neutron | 17:58 |
| mriedem | i'll push a change | 17:59 |
| *** kristia__ has joined #openstack-nova | 17:59 | |
| *** kristia__ has quit IRC | 17:59 | |
| *** kristia__ has joined #openstack-nova | 18:00 | |
| sdague | mriedem: looking | 18:00 |
| bbansal | ok thanks | 18:01 |
| *** kristian__ has quit IRC | 18:02 | |
| sdague | mriedem: yeh, probably. It doesn't hurt anything because it's a zero length file | 18:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Remove useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 18:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 18:02 |
| sdague | but, yeh, it shouldn't be there | 18:02 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 18:02 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | 18:02 |
| sdague | mriedem: on things like this change, is this sufficient release notes - https://review.openstack.org/#/c/386008/9/releasenotes/notes/quota-show-detail-access-d6f37282d288fa33.yaml ? | 18:03 |
| sdague | I wasn't sure what our standard model is for policy changes | 18:03 |
| mriedem | bbansal: sdague: https://review.openstack.org/485300 | 18:04 |
| mriedem | sdague: i guess without ^ the 0 length file kills their ftp server | 18:04 |
| *** kristia__ has quit IRC | 18:04 | |
| mriedem | sdague: its not sufficient imo | 18:05 |
| mriedem | - | 18:05 |
| mriedem | -1 | 18:05 |
| *** ociuhandu_ has quit IRC | 18:06 | |
| mriedem | sdague: we don't really have a standard model for that, but "we changed something, you figure it out" is probably not good enough :) | 18:06 |
| bbansal | thanks mriedem | 18:06 |
| bbansal | what will be the ETA for this to get merged? | 18:06 |
| sdague | bbansal: my guess, a couple of days, it needs test runs and a second reviewer | 18:07 |
| bbansal | ok thanks a lot. I can have a temp fix so that my build can pass and then once it gets merged, will give another test run | 18:07 |
| bbansal | clear | 18:07 |
| sdague | bbansal: how was it making your tests fail? | 18:08 |
| bbansal | Its not making the test fails but at the end as a part of our CI we are publishing reports and logs to our ftp server and since it was a blank file out jenkins build got unstable | 18:08 |
| bbansal | as it tried to copy this and there were no such file present | 18:09 |
| *** baoli_ has quit IRC | 18:11 | |
| sdague | bbansal: ah, gotcha | 18:12 |
| *** baoli has joined #openstack-nova | 18:15 | |
| *** boris-42___ has joined #openstack-nova | 18:17 | |
| *** fragatina has joined #openstack-nova | 18:17 | |
| *** u_nuSLASHkm8 has joined #openstack-nova | 18:18 | |
| *** u_nuSLASHkm8 has left #openstack-nova | 18:20 | |
| openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | 18:22 |
| *** efried is now known as efried_lunch | 18:23 | |
| *** mgiles has quit IRC | 18:27 | |
| *** mgiles has joined #openstack-nova | 18:27 | |
| *** baoli has quit IRC | 18:27 | |
| *** randomhack has joined #openstack-nova | 18:28 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot https://review.openstack.org/484617 | 18:29 |
| *** ociuhandu_ has joined #openstack-nova | 18:29 | |
| cdent | jaypipes: i gotta dash, but ought to be later if you wanna chat about that weirdness gibi uncovered | 18:29 |
| * cdent dines | 18:29 | |
| *** cdent has quit IRC | 18:29 | |
| mriedem | sdague: https://review.openstack.org/#/c/484617/ should be good to go | 18:30 |
| *** yamahata has joined #openstack-nova | 18:31 | |
| *** lucasxu has quit IRC | 18:32 | |
| *** randomhack has quit IRC | 18:33 | |
| *** lucasxu has joined #openstack-nova | 18:35 | |
| sean-k-mooney | jaypipes: stephenfin mriedem: o/ https://review.openstack.org/#/c/485304 is the review for the new os-vif release. we still need to merge https://review.openstack.org/#/c/485125 but it has been rebased on master so the commit id will not change. | 18:35 |
| sdague | mriedem: looking | 18:35 |
| jaypipes | sean-k-mooney: I'm going to make those little nit corrections I just dinged jangutter for since I think jangutter is afk | 18:36 |
| sean-k-mooney | jaypipes: did you forget to save the suggestions on https://review.openstack.org/#/c/485125/ | 18:36 |
| sdague | mriedem: that's a fun new 3.5 minute test in tempest I noticed | 18:36 |
| mriedem | sean-k-mooney: merge commit? | 18:36 |
| jaypipes | sean-k-mooney: no... | 18:36 |
| *** randomhack has joined #openstack-nova | 18:36 | |
| mriedem | sdague: what's fun | 18:36 |
| mriedem | ? | 18:36 |
| jaypipes | sean-k-mooney: just got delayed :) | 18:36 |
| sdague | mriedem: build 3 volumes and 2 networks and plug plug plug | 18:36 |
| jangutter | jaypipes, sean-k-mooney: jangutter is not afk, but operating at about 20% caffeine. | 18:36 |
| sdague | for 3+ minutes | 18:36 |
| mriedem | sdague: did i add it? | 18:36 |
| sean-k-mooney | jaypipes: i dont see them in gerrit i assume you were talking to him on irc | 18:37 |
| sdague | mriedem: I don't know, I was just trying to figure out why we were close to 2 hours on runs, and that one jumped out as long and new | 18:37 |
| sdague | I didn't look at the authorship | 18:37 |
| mriedem | ok, your context switch from server tags to that threw me off | 18:37 |
| mriedem | i killed a spider on my porch | 18:37 |
| mriedem | and it's babies | 18:37 |
| jaypipes | sean-k-mooney: guh, was on a prior revision. :( | 18:38 |
| jaypipes | sean-k-mooney: sec | 18:38 |
| sean-k-mooney | mriedem: i had asked jangutter to rebase on master so the that commit id would not change and i would not have to update the sha in the commit to the release repo. | 18:38 |
| sean-k-mooney | jaypipes: no worries | 18:38 |
| *** Sukhdev has quit IRC | 18:39 | |
| mriedem | sean-k-mooney: if it's got new features and stuff it should be a minor version bump | 18:39 |
| sean-k-mooney | mriedem: i could bump to 1.7 but the bump to 1.6 was mainly for the hardware offload feature | 18:39 |
| sean-k-mooney | 1.6.1 was just removing some vendor specific assumtions but more or less the same | 18:40 |
| mriedem | sean-k-mooney: there is a new feature in there, and there is a new min bump on some dependencies | 18:40 |
| mriedem | so it's 1.7 | 18:40 |
| mriedem | or bust | 18:40 |
| mriedem | +oslo.config!=4.3.0,!=4.4.0,>=4.0.0 # Apache-2.0 | 18:40 |
| sean-k-mooney | mriedem: ill have to respin the patch any when jaypipes or jangutter respins so cool ill make it 1.7.0 | 18:41 |
| *** lucasxu has quit IRC | 18:41 | |
| jaypipes | sean-k-mooney: one sec. pushing now. | 18:41 |
| openstackgerrit | Jay Pipes proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 18:41 |
| jaypipes | sean-k-mooney: ^ | 18:41 |
| *** lucasxu has joined #openstack-nova | 18:41 | |
| sean-k-mooney | jaypipes: cool thanks jay | 18:41 |
| jangutter | sean-k-mooney, jaypipes: thanks! | 18:42 |
| jangutter | jaypipes: amazingly, moving those regex comment lines up didn't break anything else. I regard regexes as "write-only-code", so I tend to leave my future self notes as to what they should do. | 18:44 |
| sean-k-mooney | mriedem: https://review.openstack.org/#/c/485304 should now be up to date | 18:44 |
| sean-k-mooney | jangutter: yes regexes to tend to be like perl, write once debug never | 18:45 |
| *** kfarr has quit IRC | 18:46 | |
| *** randomhack has quit IRC | 18:47 | |
| sean-k-mooney | moshele seams to be offline and the new melonox ci dose not seam to be commenting back on this patchset. | 18:49 |
| sean-k-mooney | do we want to leave moshele have the final approval/wait for the ci to comment on https://review.openstack.org/#/c/485125 | 18:50 |
| *** lyan has joined #openstack-nova | 18:50 | |
| mriedem | melwitt: some nits in https://review.openstack.org/#/c/446243/ | 18:50 |
| melwitt | mriedem: cool, thanks | 18:51 |
| jaypipes | sean-k-mooney: approved it. not a high risk thing. | 18:52 |
| jangutter | sean-k-mooney: if it fails on the Mellanox CI, then there's a missing test case... | 18:53 |
| sean-k-mooney | jaypipes: cool well hopefully we can do the release and bump the upper constriants and global requirement tomorrow once https://review.openstack.org/#/c/485304 is merged. | 18:55 |
| *** lucasxu has quit IRC | 18:55 | |
| sean-k-mooney | im going to log off for the night so ill chat to ye tomorrow | 18:55 |
| *** Swami has joined #openstack-nova | 18:57 | |
| *** mgiles has quit IRC | 18:57 | |
| *** mgiles has joined #openstack-nova | 18:57 | |
| *** adisky__ has quit IRC | 18:58 | |
| jangutter | ditto, thanks very much! | 18:58 |
| mriedem | i'll take care of that today | 18:59 |
| sean-k-mooney | mriedem: the requirement or was that in responce to something else | 18:59 |
| mriedem | to your stuff | 19:01 |
| mriedem | i will make all your dreams come true | 19:01 |
| mriedem | https://www.youtube.com/watch?v=Ps7tVvQHLyo | 19:01 |
| *** rajathagasthya has quit IRC | 19:03 | |
| mriedem | TheJulia: so for the bfv change, is there a passing ci for it in https://review.openstack.org/#/c/472740/ ? | 19:03 |
| mriedem | oh i see, | 19:03 |
| mriedem | answer is 'no' because the project-config changes for the experimental queue job aren't landed | 19:04 |
| jaypipes | cfriesen: yet another reason the resize workflow (and its manual confirm/revert step) is antithetical to cloudy automation :( | 19:04 |
| mriedem | cfriesen: you guys have some people doing perf testing against master? | 19:05 |
| mriedem | like at least rally testing? | 19:05 |
| *** fragatina has quit IRC | 19:06 | |
| mriedem | jlvillal: left a suggestion in https://review.openstack.org/#/c/470307/ | 19:06 |
| mriedem | jlvillal: there are ways to test things using devstack-gate before they land as a job in project-config | 19:07 |
| jlvillal | mriedem, Thanks. Yeah I know about the hack devstack-gate thing :) | 19:07 |
| jlvillal | mriedem, We actually have a different patch which I think is ready to merge | 19:08 |
| TheJulia | mriedem: your own investigation beat me to answering | 19:08 |
| mriedem | btw, what's the difference between https://review.openstack.org/#/c/480332/ and https://review.openstack.org/#/c/470307/ ? | 19:08 |
| mriedem | jlvillal: that's what i was wondering | 19:08 |
| jlvillal | mriedem, https://review.openstack.org/#/c/480332/ | 19:08 |
| mriedem | can the merge conflict one be dropped | 19:08 |
| jlvillal | mriedem, I think so. I'll leave a comment | 19:08 |
| mriedem | cool, because i'd like to see this series actually pass ci somewhere before we approve it in nova | 19:09 |
| TheJulia | mriedem: derekh was working on it and then had to disappear for a week or two so hshiina picked it up and made a new patch | 19:09 |
| mriedem | ok | 19:09 |
| TheJulia | 470307 likely just needs to be abandoned at this point | 19:10 |
| TheJulia | since it is just creating confusion | 19:10 |
| mriedem | sdague: can you abandon? https://review.openstack.org/#/c/470307/ | 19:10 |
| sdague | mriedem: done | 19:11 |
| TheJulia | thanks sdague | 19:12 |
| openstackgerrit | melanie witt proposed openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 19:14 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 19:14 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 19:14 |
| *** gyee has joined #openstack-nova | 19:18 | |
| *** tinyurl_comSLASH has joined #openstack-nova | 19:18 | |
| *** tinyurl_comSLASH has left #openstack-nova | 19:21 | |
| mriedem | melwitt: doc build failed on the man page https://review.openstack.org/#/c/446243/43/doc/source/cli/nova-manage.rst | 19:25 |
| mriedem | you need a blank line in there | 19:25 |
| mriedem | otherwise +2 | 19:25 |
| *** moshele has joined #openstack-nova | 19:25 | |
| melwitt | oh, dangit | 19:25 |
| melwitt | forgot about the tox docs target | 19:27 |
| openstackgerrit | Merged openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | 19:27 |
| openstackgerrit | Merged openstack/python-novaclient master: Add resize down test which also verifies quota changes https://review.openstack.org/484151 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | 19:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | 19:28 |
| jlvillal | mriedem, As an FYI, I have asked for reviews on https://review.openstack.org/#/c/480332/ , but I know infra is pretty busy | 19:28 |
| mriedem | ok | 19:28 |
| jaypipes | mriedem, dansmith, bauzas, edleafe: rebased the placement-claims stuff on top of mriedem's Handle None returned from get_allocation... patch and addressed all of your review comments. | 19:29 |
| *** fragatin_ has joined #openstack-nova | 19:29 | |
| openstackgerrit | Merged openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826 | 19:30 |
| *** tinyurl_comSLASH has joined #openstack-nova | 19:30 | |
| *** lucasxu has joined #openstack-nova | 19:30 | |
| *** eharney has quit IRC | 19:32 | |
| *** tinyurl_comSLASH has left #openstack-nova | 19:32 | |
| openstackgerrit | Merged openstack/nova master: conf: remove *_topic config opts https://review.openstack.org/433924 | 19:32 |
| melwitt | tox -edocs fails locally with "dot command 'dot' cannot be run (needed for graphviz output), check the graphviz_dot setting" just fyi | 19:33 |
| openstackgerrit | melanie witt proposed openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | 19:33 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | 19:33 |
| openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | 19:33 |
| melwitt | this is an interesting func test failure, timed out holding a lock during update_available_resource http://logs.openstack.org/01/484501/1/gate/gate-nova-tox-functional-ubuntu-xenial/080e1fb/testr_results.html.gz | 19:35 |
| melwitt | or waiting for lock | 19:35 |
| *** awaugama has quit IRC | 19:38 | |
| *** lucasxu has quit IRC | 19:38 | |
| *** lucasxu has joined #openstack-nova | 19:39 | |
| mriedem | melwitt: that's a known one | 19:39 |
| mriedem | don't have a fingerprint for it since we can't really fingerprint it very well | 19:39 |
| mriedem | we have this though http://status.openstack.org/elastic-recheck/#1704588 | 19:39 |
| mriedem | which is caused by the same thing | 19:39 |
| mriedem | rebuild fails b/c of novalidhost | 19:39 |
| mriedem | not really sure why rebuild should care if it's on the same host though... | 19:40 |
| mriedem | we shouldn't do a claim for rebuild if it's on the same host | 19:40 |
| mriedem | http://logs.openstack.org/82/482382/4/gate/gate-nova-tox-functional-py35-ubuntu-xenial/fdfd05a/console.html#_2017-07-15_12_30_11_220266 | 19:41 |
| melwitt | it seems weird that it's deadlocked trying to update resources | 19:42 |
| *** lucasxu has quit IRC | 19:43 | |
| mriedem | melwitt: i've wondered if we should remove this v | 19:43 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/tests/functional/integrated_helpers.py#L64 | 19:43 |
| mriedem | and start whittling that down into specific tests that require locking | 19:43 |
| mriedem | surely 90% of our functional tests shouldn't require a global lock | 19:44 |
| *** lucasxu has joined #openstack-nova | 19:44 | |
| melwitt | hm, yeah. I guess I haven't really known what REQUIRES_LOCKING does. looking at test.py now | 19:46 |
| *** rajathagasthya has joined #openstack-nova | 19:46 | |
| *** bbansal has quit IRC | 19:46 | |
| *** MasterOfBugs has joined #openstack-nova | 19:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Remove global lock from _IntegratedTestBase https://review.openstack.org/485335 | 19:46 |
| mriedem | let's see what explodes ^ | 19:46 |
| melwitt | global external locking, okay | 19:46 |
| *** pramodrj07 has joined #openstack-nova | 19:47 | |
| melwitt | unrelated but I'm having a weird thing with my devstack patch where it keeps failing saying "depends on change that failed to merge" and I tried all the things it depends on and none of them had merge conflicts locally | 19:47 |
| melwitt | it starts running in zuul and then it gets kicked out | 19:48 |
| mriedem | using depends-on? | 19:48 |
| melwitt | yeah | 19:48 |
| mriedem | if the dependencies get rebased, it will kick it out of zuul with that message | 19:48 |
| melwitt | ohh. okay, that's what it is then | 19:48 |
| melwitt | thank you | 19:49 |
| melwitt | it was when I kept updating the nova-manage thing. duh. | 19:49 |
| mriedem | http://i3.kym-cdn.com/photos/images/newsfeed/000/620/434/f9e.gif | 19:49 |
| melwitt | lol for real | 19:49 |
| *** jmlowe has quit IRC | 19:50 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Adjust test_resize_down_revert to account for counting quotas https://review.openstack.org/484152 | 19:50 |
| *** pchavva has quit IRC | 19:51 | |
| *** lpetrut_ has joined #openstack-nova | 19:52 | |
| *** lucasxu has quit IRC | 19:53 | |
| *** jmlowe has joined #openstack-nova | 19:53 | |
| *** lucasxu has joined #openstack-nova | 19:54 | |
| *** annegentle has quit IRC | 19:55 | |
| openstackgerrit | Rick Bartra proposed openstack/nova master: Add policy granularity to the Flavors API https://review.openstack.org/449288 | 19:57 |
| cfriesen | is anyone aware of sources of practical info on setting up and configuring/operating cells? caveats, gotchas, etc. | 19:59 |
| *** efried_lunch is now known as efried | 19:59 | |
| mriedem | cfriesen: yes | 20:01 |
| *** jdwidari has joined #openstack-nova | 20:02 | |
| cfriesen | mriedem: care to share? :) Also, according to http://www.dorm.org/blog/exploring-openstack-nova-cells/ there's no way to configure neutron so that port state changes go to the right cell....is that still true or was that fixed? | 20:02 |
| mriedem | cfriesen: are you talking about cells v1 or v2? | 20:02 |
| mriedem | anything mdorman is talking about wrt cells is probably v1 | 20:03 |
| mriedem | which is what godaddy is running | 20:03 |
| cfriesen | mriedem: this would be in the context of Pike, so I'm assuming v2 | 20:03 |
| mriedem | cells v1 is a dumpster fire you want to avoid | 20:03 |
| *** jdwidari has quit IRC | 20:03 | |
| melwitt | yeah, no one is running cells v2 multi cell yet. all of the blogs and stuff you'll find from CERN, GoDaddy, Nectar, that's all cells v1 | 20:04 |
| mriedem | https://docs.openstack.org/nova/latest/cells.html | 20:04 |
| *** Daisy_ has joined #openstack-nova | 20:04 | |
| mriedem | https://docs.openstack.org/ocata/install-guide-ubuntu/nova.html | 20:04 |
| mriedem | the install guide in ocata covers the things needed to deploy with cells v2 stuff | 20:04 |
| mriedem | the devref is more detail | 20:04 |
| cfriesen | oh, excellent. I had found a few of the linked things but that's a much nicer overview | 20:05 |
| mriedem | https://docs.openstack.org/ocata/config-reference/compute/cells.html is all cells v1 | 20:05 |
| mriedem | i had opened a docs bug at some point to make that clear | 20:05 |
| mriedem | but with the docs transition that's probably lost in the ether | 20:05 |
| mriedem | stephenfin should know | 20:05 |
| cfriesen | cool, that'll give me somewhere to start at least | 20:06 |
| *** nicolasbock has quit IRC | 20:06 | |
| *** ujjain has quit IRC | 20:06 | |
| *** moshele has quit IRC | 20:07 | |
| *** rajathagasthya has quit IRC | 20:07 | |
| *** thorst_ has quit IRC | 20:07 | |
| mriedem | https://bugs.launchpad.net/openstack-manuals/+bug/1673616 but that was for the ops guide | 20:08 |
| openstack | Launchpad bug 1673616 in openstack-manuals "Scaling in Operations Guide - cells section needs to be updated" [Medium,Confirmed] - Assigned to Zhenyu Zheng (zhengzhenyu) | 20:08 |
| mriedem | which is dead now i think | 20:08 |
| *** Daisy_ has quit IRC | 20:09 | |
| *** thorst has joined #openstack-nova | 20:10 | |
| *** thorst has quit IRC | 20:14 | |
| mriedem | cfriesen: we have some docs todos for cellsv2 multi-cell in pike https://etherpad.openstack.org/p/nova-pike-cells-v2-todos | 20:15 |
| openstackgerrit | Merged openstack/nova master: Avoid false positives of Jinja2 in Bandit scan https://review.openstack.org/479437 | 20:15 |
| openstackgerrit | Merged openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 20:16 |
| *** annegentle has joined #openstack-nova | 20:16 | |
| cfriesen | mriedem: excellent, thanks | 20:17 |
| *** Sukhdev_ has quit IRC | 20:19 | |
| *** eharney has joined #openstack-nova | 20:20 | |
| *** Sukhdev has joined #openstack-nova | 20:20 | |
| *** ujjain has joined #openstack-nova | 20:20 | |
| *** ujjain has quit IRC | 20:20 | |
| *** ujjain has joined #openstack-nova | 20:20 | |
| openstackgerrit | Merged openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | 20:22 |
| mriedem | jaypipes: is there anything else in here that we care about before releasing os-vif 1.7? https://review.openstack.org/#/q/project:openstack/os-vif+status:open+branch:master | 20:23 |
| *** lyan has quit IRC | 20:23 | |
| *** lyan has joined #openstack-nova | 20:24 | |
| *** smatzek has quit IRC | 20:25 | |
| *** annegentle has quit IRC | 20:26 | |
| *** rajathagasthya has joined #openstack-nova | 20:28 | |
| *** bbansal has joined #openstack-nova | 20:29 | |
| *** annegentle has joined #openstack-nova | 20:30 | |
| jaypipes | mriedem: I'd like to get https://review.openstack.org/#/c/474914/ in | 20:32 |
| jaypipes | mriedem: I might just go fix that up for ralonso since he's out | 20:32 |
| jaypipes | mriedem: gimme five mins | 20:32 |
| mriedem | jaypipes: that depends on a nova and neutron change | 20:33 |
| mriedem | which aren't closed to merged | 20:33 |
| mriedem | so what's the point? | 20:33 |
| jaypipes | ohhh... | 20:33 |
| mriedem | *close to | 20:33 |
| jaypipes | mriedem: sorry, I did not realize that. | 20:33 |
| mriedem | ok | 20:33 |
| jaypipes | mriedem: never mind then :) | 20:34 |
| *** annegentle has quit IRC | 20:35 | |
| *** MVenesio has quit IRC | 20:41 | |
| *** mgiles has quit IRC | 20:49 | |
| *** mgiles has joined #openstack-nova | 20:49 | |
| openstackgerrit | Nicolas Simonds proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 20:50 |
| *** jmlowe has quit IRC | 20:53 | |
| *** jmlowe has joined #openstack-nova | 20:54 | |
| *** mgiles has quit IRC | 20:56 | |
| *** lucasxu has quit IRC | 20:57 | |
| openstackgerrit | Merged openstack/python-novaclient master: doc: Create directory structure for docs migration https://review.openstack.org/480135 | 21:00 |
| openstackgerrit | Merged openstack/python-novaclient master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/477757 | 21:00 |
| *** kylek3h has quit IRC | 21:02 | |
| *** esberglu has quit IRC | 21:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/485358 | 21:03 |
| *** cdent has joined #openstack-nova | 21:08 | |
| *** sambetts is now known as sambetts|afk | 21:13 | |
| *** thorst has joined #openstack-nova | 21:14 | |
| *** esberglu has joined #openstack-nova | 21:14 | |
| openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.52 - Support tag when boot https://review.openstack.org/484617 | 21:15 |
| *** thorst has quit IRC | 21:19 | |
| *** jwcroppe has quit IRC | 21:22 | |
| *** jwcroppe has joined #openstack-nova | 21:23 | |
| *** thorst has joined #openstack-nova | 21:25 | |
| *** _ix has quit IRC | 21:26 | |
| *** jmlowe has quit IRC | 21:26 | |
| *** jwcroppe has quit IRC | 21:28 | |
| *** crushil has quit IRC | 21:29 | |
| *** jwcroppe has joined #openstack-nova | 21:29 | |
| *** swamireddy has quit IRC | 21:35 | |
| *** priteau has quit IRC | 21:35 | |
| openstackgerrit | Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | 21:36 |
| mriedem | jaypipes: sdague: another easy quota one https://review.openstack.org/#/c/446243/ | 21:39 |
| *** boris-42___ has quit IRC | 21:41 | |
| *** Sukhdev has quit IRC | 21:42 | |
| *** Sukhdev has joined #openstack-nova | 21:43 | |
| openstackgerrit | Merged openstack/nova master: Modify some comments for virt driver https://review.openstack.org/482022 | 21:43 |
| *** crushil has joined #openstack-nova | 21:43 | |
| *** randomhack has joined #openstack-nova | 21:44 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Migrate Ironic Flavors https://review.openstack.org/484949 | 21:46 |
| *** baoli has joined #openstack-nova | 21:47 | |
| *** acormier_ has joined #openstack-nova | 21:47 | |
| *** Daisy_ has joined #openstack-nova | 21:47 | |
| *** baoli has quit IRC | 21:48 | |
| *** marst_ has joined #openstack-nova | 21:48 | |
| *** gouthamr has quit IRC | 21:49 | |
| *** baoli has joined #openstack-nova | 21:50 | |
| *** acormier has quit IRC | 21:50 | |
| *** Daisy_ has quit IRC | 21:51 | |
| *** jwcroppe has quit IRC | 21:51 | |
| *** marst has quit IRC | 21:52 | |
| *** bbansal has left #openstack-nova | 21:54 | |
| *** marst_ has quit IRC | 21:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix and optimize external_events for multiple cells https://review.openstack.org/445142 | 22:00 |
| openstackgerrit | Merged openstack/nova master: Fix incorrect docstrings in neutron network API https://review.openstack.org/480167 | 22:03 |
| *** lpetrut_ has quit IRC | 22:04 | |
| *** jwcroppe has joined #openstack-nova | 22:08 | |
| jaypipes | mriedem: done | 22:09 |
| mriedem | yay | 22:09 |
| jaypipes | mriedem: for the record, I'm still trying to track down the issue that gibi's func test with shared resources that are custom has exposed. there's some sort of race condition I'm trying to uncover. | 22:09 |
| mriedem | happy hunting | 22:10 |
| mriedem | i'm watching sheets of rain pound my domicile | 22:11 |
| mriedem | wrong word there i guess | 22:11 |
| mriedem | *abode | 22:11 |
| jaypipes | mriedem: err, same. http://www.fox13news.com/weather/skytower-radar/manatee-sarasota | 22:12 |
| mriedem | is fox 13 your action news leader in the tri-state area? | 22:12 |
| mriedem | http://www.kttc.com/category/162320/kttc-interactive-radar | 22:12 |
| jaypipes | mriedem: there is no tri-state area, but sure :) | 22:12 |
| mriedem | where i grew up in south dakota, there was not only a tri-state area, but it was also called kelo land | 22:13 |
| mriedem | http://www.keloland.com/ | 22:13 |
| mriedem | you won't find it on any map | 22:13 |
| jaypipes | heh | 22:13 |
| mriedem | my favorite weather guy, randy brock, went into real estate and i still want to find him to make a joke about his forecast for a great deal on selling me a house | 22:14 |
| *** felipemonteiro_ has quit IRC | 22:15 | |
| cdent | jaypipes: gibi’s func test flapping is the result of differing sort order on the sharing_providers dict | 22:16 |
| cdent | I noted that on the bug: https://bugs.launchpad.net/nova/+bug/1705231 | 22:16 |
| openstack | Launchpad bug 1705231 in OpenStack Compute (nova) "Placement returns no allocation candidate for request that needs both compute resources and custom shared resources" [High,Triaged] | 22:17 |
| cdent | messes up the order of the joins which for reasons unknown to me messes up the query | 22:18 |
| *** gouthamr has joined #openstack-nova | 22:19 | |
| *** Apoorva has quit IRC | 22:21 | |
| *** Apoorva has joined #openstack-nova | 22:22 | |
| *** baoli has quit IRC | 22:26 | |
| *** randomhack has quit IRC | 22:28 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Add a new section: "Upgrade impact" to the template https://review.openstack.org/456756 | 22:28 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/485358 | 22:30 |
| *** coreywright has quit IRC | 22:30 | |
| *** sdague has quit IRC | 22:33 | |
| *** cdent has quit IRC | 22:34 | |
| *** coreywright has joined #openstack-nova | 22:35 | |
| *** baoli has joined #openstack-nova | 22:36 | |
| *** edmondsw has quit IRC | 22:40 | |
| mriedem | melwitt: so the counting quotas changes are all approved, | 22:42 |
| mriedem | once that last one is done i think we can close that bp unless there is something i'm missing, | 22:43 |
| mriedem | the last 2 changes in the series are the quotas in api db bp | 22:43 |
| melwitt | mriedem: yeah. the rest will be adding test coverage and the like | 22:44 |
| mriedem | speaking of | 22:44 |
| mriedem | http://logs.openstack.org/52/484152/6/check/gate-novaclient-dsvm-functional-neutron-ubuntu-xenial/2bee19d/testr_results.html.gz | 22:44 |
| mriedem | so what i'm seeing in this test is, | 22:44 |
| mriedem | we resize down, | 22:44 |
| mriedem | usage changes, as expected | 22:44 |
| mriedem | b/c no more reservations and the instance vpus/memory_mb is changed in the compute | 22:45 |
| mriedem | then i revert the resize | 22:45 |
| mriedem | usage changes back to the original flavor, | 22:45 |
| mriedem | which makes sense, but in this case, it's a revert back *up* | 22:45 |
| *** jwcroppe has quit IRC | 22:45 | |
| mriedem | at which point i could be over quota | 22:45 |
| *** jwcroppe has joined #openstack-nova | 22:46 | |
| mriedem | oh f yeah https://review.openstack.org/#/c/416521/63/nova/compute/api.py@3040 | 22:46 |
| mriedem | you check for that | 22:46 |
| melwitt | right. but the thing fails? | 22:46 |
| mriedem | i think that was one of the more recent changes in that patch when we talked about it this week | 22:46 |
| mriedem | the only thing that failed was my assertion that usage didn't change | 22:46 |
| mriedem | which is wrong | 22:46 |
| mriedem | i was just worried we weren't checking quota on revert up | 22:46 |
| mriedem | but we are | 22:47 |
| mriedem | so awesome | 22:47 |
| melwitt | oh, good | 22:47 |
| melwitt | phew | 22:47 |
| mriedem | ha | 22:47 |
| *** jmlowe has joined #openstack-nova | 22:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Adjust test_resize_down_revert to account for counting quotas https://review.openstack.org/484152 | 22:49 |
| *** jwcroppe has quit IRC | 22:51 | |
| *** jwcroppe has joined #openstack-nova | 22:54 | |
| *** thorst has quit IRC | 22:56 | |
| *** vishwana_ has quit IRC | 22:58 | |
| *** itlinux has quit IRC | 22:58 | |
| *** vishwanathj has joined #openstack-nova | 22:58 | |
| *** jamesden_ has quit IRC | 23:00 | |
| *** catintheroof has quit IRC | 23:03 | |
| *** yamamoto has joined #openstack-nova | 23:04 | |
| *** lyan has quit IRC | 23:04 | |
| *** gbarros has quit IRC | 23:05 | |
| mriedem | good lord i guess the global lock in the functional tests is there for a reason http://logs.openstack.org/35/485335/1/check/gate-nova-tox-functional-ubuntu-xenial/854268f/testr_results.html.gz | 23:10 |
| mriedem | oh it's something else | 23:10 |
| cfriesen | in the context of cells v2, where do nova-consoleauth/nova-cert/nova-novncproxy run? on the nova-api node(s)? | 23:14 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: [placement] test shared custom resource provider https://review.openstack.org/485088 | 23:15 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: alloc candidates only shared resources https://review.openstack.org/484900 | 23:15 |
| mriedem | cfriesen: that's a question for melwitt | 23:15 |
| *** sbezverk has quit IRC | 23:15 | |
| mriedem | https://review.openstack.org/#/q/topic:bp/convert-consoles-to-objects+status:open | 23:16 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: proper JOIN order for shared resources https://review.openstack.org/485088 | 23:16 |
| *** jaypipes has quit IRC | 23:16 | |
| *** yamamoto has quit IRC | 23:18 | |
| cfriesen | mriedem: I guess nova-cert is now gone, so it's just the other two | 23:18 |
| mriedem | riht | 23:21 |
| mriedem | *right | 23:21 |
| cfriesen | mriedem: you wouldn't happen to know if bug 1691545 hits when using just one cell in newton? | 23:23 |
| openstack | bug 1691545 in OpenStack Compute (nova) ocata "Significant increase in DB connections with cells" [High,Fix committed] https://launchpad.net/bugs/1691545 - Assigned to melanie witt (melwitt) | 23:23 |
| *** gbarros has joined #openstack-nova | 23:25 | |
| *** vladikr has quit IRC | 23:25 | |
| *** jwcroppe has quit IRC | 23:29 | |
| *** jwcroppe has joined #openstack-nova | 23:29 | |
| *** hongbin has quit IRC | 23:30 | |
| *** esberglu has quit IRC | 23:33 | |
| *** jwcroppe has quit IRC | 23:33 | |
| *** esberglu has joined #openstack-nova | 23:33 | |
| *** esberglu has quit IRC | 23:33 | |
| melwitt | cfriesen: nova-consoleauth should be run globally for the deployment. currently nova-novncproxy should run globally too but after this series, it should be run per cell https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/convert-consoles-to-objects | 23:34 |
| *** rcernin has quit IRC | 23:34 | |
| melwitt | cfriesen: that bug affects one cell yes. the fix was backported to newton though | 23:34 |
| cfriesen | melwitt: I saw that, we haven't picked up that change yet | 23:35 |
| *** acormier_ has quit IRC | 23:35 | |
| melwitt | ah, okay | 23:35 |
| *** acormier has joined #openstack-nova | 23:35 | |
| *** eharney has quit IRC | 23:35 | |
| *** chyka has quit IRC | 23:37 | |
| *** acormier has quit IRC | 23:40 | |
| *** Swami has quit IRC | 23:44 | |
| *** ckopper has quit IRC | 23:46 | |
| *** itlinux has joined #openstack-nova | 23:47 | |
| *** mtanino_ has quit IRC | 23:53 | |
| *** edmondsw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
