| *** baoli has joined #openstack-nova | 00:00 | |
| *** jdg has quit IRC | 00:02 | |
| *** jaypipes has quit IRC | 00:02 | |
| *** gyee has quit IRC | 00:04 | |
| *** jdg has joined #openstack-nova | 00:06 | |
| *** acormier has joined #openstack-nova | 00:06 | |
| *** brault has joined #openstack-nova | 00:08 | |
| *** jdg has quit IRC | 00:11 | |
| *** jdg has joined #openstack-nova | 00:12 | |
| *** brault has quit IRC | 00:13 | |
| *** crushil has quit IRC | 00:18 | |
| *** slaweq_ has quit IRC | 00:20 | |
| *** acormier has quit IRC | 00:20 | |
| *** slaweq_ has joined #openstack-nova | 00:25 | |
| *** crushil has joined #openstack-nova | 00:25 | |
| *** ijw has joined #openstack-nova | 00:26 | |
| *** gjayavelu has quit IRC | 00:31 | |
| *** gbarros has joined #openstack-nova | 00:32 | |
| *** flanders_ has quit IRC | 00:34 | |
| *** felipemonteiro__ has quit IRC | 00:38 | |
| *** Swami has quit IRC | 00:40 | |
| *** mriedem has quit IRC | 00:51 | |
| *** sree has joined #openstack-nova | 00:58 | |
| openstackgerrit | Merged openstack/nova master: Set group_members when converting to legacy request spec https://review.openstack.org/507938 | 00:58 |
|---|---|---|
| *** slaweq_ has quit IRC | 00:58 | |
| *** jdg has quit IRC | 00:59 | |
| *** tetsuro has joined #openstack-nova | 01:00 | |
| *** slaweq_ has joined #openstack-nova | 01:00 | |
| *** mingyu has quit IRC | 01:02 | |
| *** mingyu has joined #openstack-nova | 01:03 | |
| *** dillaman has quit IRC | 01:08 | |
| *** acormier has joined #openstack-nova | 01:10 | |
| *** sree has quit IRC | 01:10 | |
| *** sree has joined #openstack-nova | 01:11 | |
| *** Swami has joined #openstack-nova | 01:14 | |
| *** sree has quit IRC | 01:15 | |
| *** crushil has quit IRC | 01:15 | |
| *** hshiina has joined #openstack-nova | 01:18 | |
| *** jgriffith_ has joined #openstack-nova | 01:19 | |
| *** hongbin has joined #openstack-nova | 01:20 | |
| *** acormier has quit IRC | 01:20 | |
| *** acormier has joined #openstack-nova | 01:21 | |
| *** ijw has quit IRC | 01:21 | |
| *** erlon has quit IRC | 01:22 | |
| *** jgriffith_ has quit IRC | 01:23 | |
| *** jgriffith_ has joined #openstack-nova | 01:23 | |
| *** bnemec-backlater has quit IRC | 01:24 | |
| *** bnemec has joined #openstack-nova | 01:24 | |
| *** acormier_ has joined #openstack-nova | 01:26 | |
| *** jgriffith_ has quit IRC | 01:26 | |
| *** jgriffith_ has joined #openstack-nova | 01:27 | |
| *** acormier has quit IRC | 01:29 | |
| *** slaweq_ has quit IRC | 01:32 | |
| *** jgriffith_ has quit IRC | 01:34 | |
| *** acormier_ has quit IRC | 01:36 | |
| *** slaweq_ has joined #openstack-nova | 01:39 | |
| *** gbarros has quit IRC | 01:40 | |
| *** yamahata has quit IRC | 01:44 | |
| *** gbarros has joined #openstack-nova | 01:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - List/Show all server migration types https://review.openstack.org/430839 | 01:45 |
| *** phuongnh has joined #openstack-nova | 01:46 | |
| *** gbarros has quit IRC | 01:49 | |
| *** jhesketh has quit IRC | 01:51 | |
| *** jhesketh has joined #openstack-nova | 01:51 | |
| *** diga has quit IRC | 01:55 | |
| *** dave-mcc_ has quit IRC | 01:57 | |
| *** adreznec has quit IRC | 02:00 | |
| *** MikeG451 has quit IRC | 02:01 | |
| *** hoonetorg has quit IRC | 02:02 | |
| *** tojuvone has quit IRC | 02:02 | |
| *** afazekas has quit IRC | 02:02 | |
| *** tojuvone has joined #openstack-nova | 02:02 | |
| *** afazekas has joined #openstack-nova | 02:02 | |
| *** adreznec has joined #openstack-nova | 02:02 | |
| *** hoonetorg has joined #openstack-nova | 02:03 | |
| *** MikeG451 has joined #openstack-nova | 02:06 | |
| *** tpatil has joined #openstack-nova | 02:07 | |
| openstackgerrit | Merged openstack/nova master: Fix CellDatabases fixture swallowing exceptions https://review.openstack.org/506312 | 02:09 |
| *** slaweq_ has quit IRC | 02:09 | |
| *** hemna_ has joined #openstack-nova | 02:11 | |
| *** slaweq_ has joined #openstack-nova | 02:18 | |
| *** brault has joined #openstack-nova | 02:24 | |
| *** owalsh_ has joined #openstack-nova | 02:28 | |
| *** brault has quit IRC | 02:28 | |
| *** owalsh has quit IRC | 02:31 | |
| *** hemna_ has quit IRC | 02:39 | |
| *** vladikr has quit IRC | 02:41 | |
| *** vladikr has joined #openstack-nova | 02:42 | |
| *** jgriffith_ has joined #openstack-nova | 02:42 | |
| *** nicolasbock has quit IRC | 02:43 | |
| *** vishwanathj has quit IRC | 02:44 | |
| *** vladikr has quit IRC | 02:47 | |
| *** vladikr has joined #openstack-nova | 02:47 | |
| *** yamamoto has joined #openstack-nova | 02:48 | |
| *** mdnadeem has joined #openstack-nova | 02:49 | |
| *** slaweq_ has quit IRC | 02:51 | |
| *** ijw has joined #openstack-nova | 02:53 | |
| *** slaweq_ has joined #openstack-nova | 02:55 | |
| *** ijw has quit IRC | 02:57 | |
| *** ijw has joined #openstack-nova | 03:02 | |
| *** baoli has quit IRC | 03:02 | |
| *** tpatil has quit IRC | 03:04 | |
| *** claudiub has quit IRC | 03:07 | |
| *** ijw has quit IRC | 03:08 | |
| *** jgriffith_ has quit IRC | 03:08 | |
| *** ijw has joined #openstack-nova | 03:11 | |
| *** lbragstad has joined #openstack-nova | 03:12 | |
| *** ijw has quit IRC | 03:16 | |
| *** baoli has joined #openstack-nova | 03:17 | |
| *** gjayavelu has joined #openstack-nova | 03:21 | |
| *** larainema has joined #openstack-nova | 03:25 | |
| *** baoli has quit IRC | 03:26 | |
| *** vladikr has quit IRC | 03:26 | |
| *** vladikr has joined #openstack-nova | 03:26 | |
| *** slaweq_ has quit IRC | 03:28 | |
| *** sree has joined #openstack-nova | 03:30 | |
| *** udesale has joined #openstack-nova | 03:31 | |
| *** gjayavelu has quit IRC | 03:33 | |
| *** slaweq_ has joined #openstack-nova | 03:34 | |
| *** markvoelker has quit IRC | 03:36 | |
| *** acormier has joined #openstack-nova | 03:36 | |
| *** hshiina has quit IRC | 03:37 | |
| *** manasm has joined #openstack-nova | 03:40 | |
| *** acormier has quit IRC | 03:41 | |
| *** tonyb has quit IRC | 03:42 | |
| *** hongbin has quit IRC | 03:42 | |
| *** zzzeek has quit IRC | 03:46 | |
| *** links has joined #openstack-nova | 03:48 | |
| *** lbragstad has quit IRC | 03:50 | |
| *** vladikr has quit IRC | 03:51 | |
| *** vladikr has joined #openstack-nova | 03:52 | |
| *** tonyb has joined #openstack-nova | 03:56 | |
| *** hshiina has joined #openstack-nova | 03:57 | |
| *** abhishekk has joined #openstack-nova | 04:06 | |
| *** jgriffith has quit IRC | 04:11 | |
| *** jdwidari has quit IRC | 04:11 | |
| *** jgriffith has joined #openstack-nova | 04:11 | |
| *** armax has quit IRC | 04:12 | |
| *** armax has joined #openstack-nova | 04:13 | |
| *** armax has quit IRC | 04:13 | |
| *** armax has joined #openstack-nova | 04:13 | |
| *** armax has quit IRC | 04:14 | |
| *** jamielennox has quit IRC | 04:14 | |
| *** armax has joined #openstack-nova | 04:14 | |
| *** armax has quit IRC | 04:14 | |
| *** armax has joined #openstack-nova | 04:15 | |
| *** armax has quit IRC | 04:15 | |
| *** armax has joined #openstack-nova | 04:16 | |
| *** armax has quit IRC | 04:16 | |
| *** cburgess has quit IRC | 04:16 | |
| *** jamielennox has joined #openstack-nova | 04:18 | |
| *** cburgess has joined #openstack-nova | 04:19 | |
| *** gyee has joined #openstack-nova | 04:33 | |
| *** mingyu has quit IRC | 04:34 | |
| *** mingyu has joined #openstack-nova | 04:35 | |
| *** zzzeek has joined #openstack-nova | 04:38 | |
| *** mingyu has quit IRC | 04:39 | |
| *** slaweq_ has quit IRC | 04:41 | |
| *** mingyu has joined #openstack-nova | 04:44 | |
| *** ratailor has joined #openstack-nova | 04:44 | |
| *** slaweq_ has joined #openstack-nova | 04:49 | |
| *** gmann is now known as gmann_lunch | 04:58 | |
| *** psachin has joined #openstack-nova | 05:01 | |
| *** chyka has joined #openstack-nova | 05:04 | |
| *** gyee has quit IRC | 05:05 | |
| *** gouthamr has quit IRC | 05:06 | |
| *** chyka has quit IRC | 05:08 | |
| *** slaweq_ has quit IRC | 05:22 | |
| *** slaweq_ has joined #openstack-nova | 05:26 | |
| *** lajoskatona has joined #openstack-nova | 05:27 | |
| *** lpetrut has joined #openstack-nova | 05:30 | |
| *** chyka has joined #openstack-nova | 05:35 | |
| *** markvoelker has joined #openstack-nova | 05:37 | |
| *** chyka has quit IRC | 05:39 | |
| *** claudiub has joined #openstack-nova | 05:40 | |
| ratailor | Can anyone else reproduce this https://bugs.launchpad.net/nova/+bug/1720062 | 05:40 |
| openstack | Launchpad bug 1720062 in OpenStack Compute (nova) "Nova unit tests are failing " [Low,Incomplete] | 05:40 |
| *** ratailor is now known as rtailor | 05:41 | |
| *** udesale__ has joined #openstack-nova | 05:42 | |
| *** gmann_lunch is now known as gmann | 05:43 | |
| *** udesale has quit IRC | 05:45 | |
| *** vladikr has quit IRC | 05:49 | |
| *** vladikr has joined #openstack-nova | 05:50 | |
| *** vishwanathj has joined #openstack-nova | 05:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 05:55 |
| *** vladikr has quit IRC | 05:55 | |
| *** vladikr has joined #openstack-nova | 05:56 | |
| *** slaweq_ has quit IRC | 05:59 | |
| *** takashin has left #openstack-nova | 06:00 | |
| *** vladikr has quit IRC | 06:02 | |
| *** vladikr has joined #openstack-nova | 06:02 | |
| *** slaweq_ has joined #openstack-nova | 06:05 | |
| *** vladikr has quit IRC | 06:08 | |
| *** vladikr has joined #openstack-nova | 06:08 | |
| *** Oku_OS-away is now known as Oku_OS | 06:08 | |
| *** lpetrut has quit IRC | 06:10 | |
| *** markvoelker has quit IRC | 06:11 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova-specs master: Ironic: Resize and cold migration support https://review.openstack.org/449155 | 06:13 |
| *** andreas_s has joined #openstack-nova | 06:15 | |
| *** trinaths has joined #openstack-nova | 06:16 | |
| *** vishwana_ has joined #openstack-nova | 06:19 | |
| *** vishwanathj has quit IRC | 06:21 | |
| *** karthiks_afk is now known as karthiks | 06:22 | |
| *** vladikr has quit IRC | 06:23 | |
| *** rcernin has joined #openstack-nova | 06:24 | |
| *** vladikr has joined #openstack-nova | 06:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/pike: Updated from global requirements https://review.openstack.org/493146 | 06:26 |
| *** bhagyashri_s has joined #openstack-nova | 06:29 | |
| *** pooja-jadhav has joined #openstack-nova | 06:30 | |
| *** NehaAlhat has joined #openstack-nova | 06:30 | |
| *** abhishek_k has joined #openstack-nova | 06:30 | |
| *** abhishek_k has quit IRC | 06:31 | |
| *** abhishek_k has joined #openstack-nova | 06:31 | |
| *** pooja_jadhav has quit IRC | 06:32 | |
| *** neha_alhat has quit IRC | 06:32 | |
| *** bhagyashris has quit IRC | 06:32 | |
| *** abhishekk has quit IRC | 06:33 | |
| *** spectr has quit IRC | 06:34 | |
| *** spectr has joined #openstack-nova | 06:34 | |
| *** slaweq_ has quit IRC | 06:38 | |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Change live_migrate tests to use fakedriver https://review.openstack.org/505202 | 06:40 |
| *** Swami has quit IRC | 06:43 | |
| *** slaweq_ has joined #openstack-nova | 06:44 | |
| *** brault has joined #openstack-nova | 06:44 | |
| *** udesale has joined #openstack-nova | 06:46 | |
| *** udesale__ has quit IRC | 06:48 | |
| *** vishwana_ has quit IRC | 06:48 | |
| *** pooja-jadhav is now known as pooja_jadhav | 06:53 | |
| *** david_1 has joined #openstack-nova | 06:56 | |
| *** mkoderer_ has joined #openstack-nova | 06:56 | |
| *** dgonzalez_ has joined #openstack-nova | 06:56 | |
| *** seife_ has joined #openstack-nova | 06:56 | |
| *** seife_ has quit IRC | 06:58 | |
| *** dgonzalez_ has quit IRC | 06:58 | |
| *** mkoderer_ has quit IRC | 06:58 | |
| *** david_1 has quit IRC | 06:58 | |
| *** vishwanathj has joined #openstack-nova | 07:04 | |
| *** tesseract has joined #openstack-nova | 07:07 | |
| *** markvoelker has joined #openstack-nova | 07:08 | |
| *** pcaruana has joined #openstack-nova | 07:08 | |
| *** tetsuro has quit IRC | 07:10 | |
| *** slaweq_ has quit IRC | 07:17 | |
| *** cfriesen has quit IRC | 07:20 | |
| *** slaweq_ has joined #openstack-nova | 07:21 | |
| *** chyka has joined #openstack-nova | 07:30 | |
| *** chyka has quit IRC | 07:35 | |
| *** alexchadin has joined #openstack-nova | 07:37 | |
| *** markvoelker has quit IRC | 07:42 | |
| *** lucas-afk is now known as lucasagomes | 07:46 | |
| *** jpena|off is now known as jpena | 07:52 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Move last_bytes into the path module https://review.openstack.org/509417 | 07:53 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | 07:53 |
| *** ociuhandu has joined #openstack-nova | 07:53 | |
| *** vipul has quit IRC | 07:55 | |
| *** trinaths has quit IRC | 08:06 | |
| *** spectr has quit IRC | 08:09 | |
| *** spectr has joined #openstack-nova | 08:09 | |
| *** spectr has quit IRC | 08:17 | |
| *** spectr has joined #openstack-nova | 08:18 | |
| *** hieulq has quit IRC | 08:26 | |
| *** hieulq has joined #openstack-nova | 08:27 | |
| *** slaweq_ has quit IRC | 08:27 | |
| *** ralonsoh has joined #openstack-nova | 08:28 | |
| *** trungnv has quit IRC | 08:29 | |
| *** trungnv has joined #openstack-nova | 08:30 | |
| *** slaweq_ has joined #openstack-nova | 08:31 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove dead code of api.fault notification sending https://review.openstack.org/505164 | 08:37 |
| *** markvoelker has joined #openstack-nova | 08:38 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/504895 | 08:42 |
| *** owalsh_ is now known as owalsh | 08:50 | |
| *** sree_ has joined #openstack-nova | 09:02 | |
| *** sree_ is now known as Guest68095 | 09:02 | |
| *** slaweq_ has quit IRC | 09:03 | |
| *** sree has quit IRC | 09:05 | |
| *** markvoelker has quit IRC | 09:12 | |
| *** sahid has joined #openstack-nova | 09:13 | |
| *** alexchadin has quit IRC | 09:16 | |
| *** diga has joined #openstack-nova | 09:16 | |
| *** yamamoto has quit IRC | 09:16 | |
| *** alexchadin has joined #openstack-nova | 09:16 | |
| *** yamamoto has joined #openstack-nova | 09:20 | |
| *** trinaths has joined #openstack-nova | 09:21 | |
| *** vipul has joined #openstack-nova | 09:24 | |
| *** mvk has quit IRC | 09:28 | |
| *** slaweq_ has joined #openstack-nova | 09:28 | |
| *** alexchadin has quit IRC | 09:30 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | 09:32 |
| *** gszasz has joined #openstack-nova | 09:34 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | 09:35 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Support traits in the Ironic driver https://review.openstack.org/507052 | 09:37 |
| *** trinaths has left #openstack-nova | 09:38 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: rework _get_vpcu_total to return online pCPUs https://review.openstack.org/509730 | 09:39 |
| *** manasm has quit IRC | 09:39 | |
| bhagyashri_s | johnthetubaguy: Hi | 09:40 |
| johnthetubaguy | bhagyashri_s: hi | 09:40 |
| bhagyashri_s | johnthetubaguy: I have one doubt regarding the patch https://review.openstack.org/#/c/409644/9, As from the microversion 2.45 refer: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/create_backup.py#L82 the image[id] is return in the response irrespective of rotation parameter value and we are skipping the creation of image if rotation parameter value is 0 then in that case the image[id] will give the KeyError. | 09:46 |
| *** derekh has joined #openstack-nova | 09:47 | |
| *** mvk has joined #openstack-nova | 09:47 | |
| *** trungnv has quit IRC | 09:47 | |
| johnthetubaguy | bhagyashri_s: that does sound like an API change, which would need a new microversion and a new spec. | 09:48 |
| bhagyashri_s | johnthetubaguy: so should i add the condition rotation should be greater than 0 at api microversion 2.45 or should bump the microversion separately | 09:48 |
| johnthetubaguy | bhagyashri_s: maybe lets just go for returning null in the case of there being no image_id to return? | 09:49 |
| johnthetubaguy | and forgot the microversion in this case | 09:50 |
| johnthetubaguy | it looks like a bug in the new microversion really | 09:50 |
| johnthetubaguy | lets see how that looks in code, I don't know the right way forward there | 09:51 |
| bhagyashri_s | johnthetubaguy: ok. | 09:52 |
| bhagyashri_s | johnthetubaguy: let me try | 09:53 |
| *** chyka has joined #openstack-nova | 09:54 | |
| *** chyka has quit IRC | 09:59 | |
| *** sdague has joined #openstack-nova | 10:00 | |
| *** Guest68095 has quit IRC | 10:00 | |
| *** sree has joined #openstack-nova | 10:01 | |
| *** slaweq_ has quit IRC | 10:01 | |
| bhagyashri_s | johnthetubaguy: i have checked by returning null but in that case at nova side that means through curl command it's retuning 202 accepted and using the command line it's returning "ERROR (CommandError): No Image matching None. (HTTP 404)" 404 not found and this inconsistency | 10:02 |
| *** yamamoto has quit IRC | 10:02 | |
| openstackgerrit | Merged openstack/nova master: doc: Add documentation for cpu_realtime, cpu_realtime_mask https://review.openstack.org/502056 | 10:03 |
| *** yassine has joined #openstack-nova | 10:05 | |
| *** sree has quit IRC | 10:05 | |
| *** sree has joined #openstack-nova | 10:05 | |
| *** yamamoto has joined #openstack-nova | 10:06 | |
| *** markvoelker has joined #openstack-nova | 10:09 | |
| *** sree has quit IRC | 10:10 | |
| *** slaweq_ has joined #openstack-nova | 10:11 | |
| *** yamamoto has quit IRC | 10:12 | |
| *** alexchadin has joined #openstack-nova | 10:12 | |
| bhagyashri_s | johnthetubaguy: just need your opinion about my check as mentioned above | 10:12 |
| johnthetubaguy | bhagyashri_s: did you return {'image_id': None} ? | 10:13 |
| johnthetubaguy | oh, I see, the command line is looking up the image | 10:14 |
| johnthetubaguy | I guess this will need a microversion | 10:14 |
| bhagyashri_s | johnthetubaguy: return {'image_id': image.get('id', None)} like this | 10:14 |
| bhagyashri_s | johnthetubaguy at https://github.com/openstack/nova/blame/master/nova/api/openstack/compute/create_backup.py#L83 | 10:14 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: stabilize test_resize_server_error_and_reschedule_was_failed https://review.openstack.org/509750 | 10:15 |
| johnthetubaguy | bhagyashri_s: seems like we need a microversion, due to the new behaviour | 10:15 |
| bhagyashri_s | johnthetubaguy: ok. | 10:16 |
| bhagyashri_s | johnthetubaguy: and spec as well | 10:16 |
| johnthetubaguy | bhagyashri_s: which means we need a nova-spec, sadly, because of that 2.45 change | 10:16 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: stabilize test_resize_server_error_and_reschedule_was_failed https://review.openstack.org/509750 | 10:17 |
| bhagyashri_s | johnthetubaguy: ok. I will work on it. Thank you for your opinion. | 10:17 |
| bhagyashri_s | johnthetubaguy: :) | 10:18 |
| gibi | stephenfin, bauzas: Would you like to support my embarrassment in https://review.openstack.org/#/c/509750/ ? :) | 10:19 |
| *** yamamoto has joined #openstack-nova | 10:19 | |
| stephenfin | gibi: :D Gladly | 10:19 |
| stephenfin | gibi: thanks for yet another review on https://review.openstack.org/502056, btw | 10:19 |
| bauzas | gibi: expenses reporting paperwork + fun with children | 10:20 |
| bauzas | so maybe in the next 2 hours ? | 10:20 |
| bauzas | :p | 10:20 |
| gibi | bauzas: no hurries :) | 10:20 |
| *** manasm_ has joined #openstack-nova | 10:21 | |
| *** manasm_ is now known as manasm | 10:21 | |
| gibi | stephenfin: almost every added documentation is better than having no documentation at all. | 10:22 |
| *** alexchadin has quit IRC | 10:35 | |
| *** udesale has quit IRC | 10:38 | |
| *** cdent has joined #openstack-nova | 10:41 | |
| *** yamamoto has quit IRC | 10:41 | |
| *** markvoelker has quit IRC | 10:42 | |
| *** dtantsur|afk is now known as dtantsur | 10:44 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Move last_bytes into the path module https://review.openstack.org/509417 | 10:45 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | 10:45 |
| *** chyka has joined #openstack-nova | 10:45 | |
| rgerganov | does anyone know why Gerrit says "patch in merge conflict" even when the patch is cleanly rebased on master? https://review.openstack.org/#/c/509417/ | 10:47 |
| *** chyka has quit IRC | 10:50 | |
| *** slaweq_ has quit IRC | 10:50 | |
| *** alexchadin has joined #openstack-nova | 10:54 | |
| *** spectr has quit IRC | 10:54 | |
| *** andreas_s_ has joined #openstack-nova | 10:56 | |
| *** andreas_s has quit IRC | 10:58 | |
| *** jpena is now known as jpena|lunch | 10:58 | |
| *** nicolasbock has joined #openstack-nova | 10:59 | |
| *** yamamoto has joined #openstack-nova | 11:01 | |
| *** dave-mccowan has joined #openstack-nova | 11:04 | |
| *** aloga has quit IRC | 11:04 | |
| *** aloga has joined #openstack-nova | 11:04 | |
| *** nicolasbock has quit IRC | 11:05 | |
| *** nicolasbock has joined #openstack-nova | 11:05 | |
| *** priteau has joined #openstack-nova | 11:05 | |
| *** yamamoto has quit IRC | 11:06 | |
| *** avolkov has quit IRC | 11:09 | |
| *** avolkov has joined #openstack-nova | 11:10 | |
| *** abhishek_k is now known as abhishekk | 11:11 | |
| *** alexchadin has quit IRC | 11:14 | |
| *** alexchadin has joined #openstack-nova | 11:15 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Intel Fortville Dynamic Device Personalization (DDP) https://review.openstack.org/503001 | 11:15 |
| *** ociuhandu has quit IRC | 11:15 | |
| *** slaweq_ has joined #openstack-nova | 11:15 | |
| *** hemna_ has joined #openstack-nova | 11:17 | |
| *** lucasagomes is now known as lucas-hungry | 11:17 | |
| *** jdwidari has joined #openstack-nova | 11:19 | |
| *** alexchadin has quit IRC | 11:27 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Move 'ips' field from Subnet object to VIF object https://review.openstack.org/508498 | 11:28 |
| *** rtailor has quit IRC | 11:29 | |
| openstackgerrit | Merged openstack/nova master: Add instance.interface_attach notification https://review.openstack.org/503089 | 11:29 |
| openstackgerrit | Merged openstack/nova master: Add instance.interface_detach notification https://review.openstack.org/506284 | 11:30 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Change Subnet.dhcp_server to contain multiple IPs https://review.openstack.org/509107 | 11:31 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 11:35 |
| *** alexchadin has joined #openstack-nova | 11:35 | |
| *** smatzek has joined #openstack-nova | 11:37 | |
| *** dikonoor has joined #openstack-nova | 11:39 | |
| *** markvoelker has joined #openstack-nova | 11:39 | |
| *** chyka has joined #openstack-nova | 11:41 | |
| *** vladikr has quit IRC | 11:41 | |
| *** jdwidari has quit IRC | 11:43 | |
| *** chyka has quit IRC | 11:45 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | 11:46 |
| *** phuongnh has quit IRC | 11:48 | |
| *** slaweq_ has quit IRC | 11:51 | |
| *** priteau has quit IRC | 11:53 | |
| *** priteau has joined #openstack-nova | 11:53 | |
| *** priteau has quit IRC | 11:54 | |
| *** vladikr has joined #openstack-nova | 11:54 | |
| *** priteau has joined #openstack-nova | 11:54 | |
| *** abhishekk has quit IRC | 11:57 | |
| *** yamamoto has joined #openstack-nova | 12:00 | |
| *** tikitavi has joined #openstack-nova | 12:02 | |
| *** jangutter_ has joined #openstack-nova | 12:03 | |
| *** jangutter has quit IRC | 12:06 | |
| *** manasm has quit IRC | 12:07 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova master: TrivialFix: Fix the incorrect test case https://review.openstack.org/509758 | 12:09 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix unstable shelve offload functional tests https://review.openstack.org/509759 | 12:10 |
| *** alexchadin has quit IRC | 12:10 | |
| *** markvoelker has quit IRC | 12:12 | |
| *** jpena|lunch is now known as jpena | 12:13 | |
| *** edmondsw has joined #openstack-nova | 12:13 | |
| *** edmondsw_ has joined #openstack-nova | 12:16 | |
| *** edmondsw has quit IRC | 12:17 | |
| *** lucas-hungry is now known as lucasagomes | 12:17 | |
| *** slaweq_ has joined #openstack-nova | 12:18 | |
| *** pchavva has joined #openstack-nova | 12:20 | |
| *** jdwidari has joined #openstack-nova | 12:22 | |
| *** slaweq_ has quit IRC | 12:22 | |
| *** edmondsw_ has quit IRC | 12:22 | |
| *** lpetrut has joined #openstack-nova | 12:23 | |
| *** acormier has joined #openstack-nova | 12:23 | |
| *** acormier has joined #openstack-nova | 12:23 | |
| tikitavi | Hi, can anybody help? We have problem with running instances sometimes. The error is ERROR oslo_messaging.rpc.server IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/six-1.10.0.dist-info/METADATA' | 12:25 |
| tikitavi | http://logs.openstack.org/91/500091/7/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/b05f013/logs/screen-n-cond.txt.gz?level=ERROR | 12:26 |
| tikitavi | the problem is in stable/ocata and stable/pike | 12:26 |
| *** alexchadin has joined #openstack-nova | 12:27 | |
| *** markvoelker has joined #openstack-nova | 12:28 | |
| *** edmondsw has joined #openstack-nova | 12:29 | |
| *** bhagyashri_s has quit IRC | 12:30 | |
| *** bhagyashri_s has joined #openstack-nova | 12:31 | |
| *** slaweq_ has joined #openstack-nova | 12:33 | |
| *** edmondsw has quit IRC | 12:33 | |
| *** edmondsw has joined #openstack-nova | 12:36 | |
| *** yamamoto has quit IRC | 12:39 | |
| tikitavi | sorry in stable/ocata only | 12:39 |
| *** yamamoto has joined #openstack-nova | 12:41 | |
| *** Tom___ has joined #openstack-nova | 12:41 | |
| tikitavi | although sorry, in stable/pike too, the logs just moved to super-conductor | 12:41 |
| *** Tom_____ has quit IRC | 12:42 | |
| *** namnh has joined #openstack-nova | 12:42 | |
| *** namnh has quit IRC | 12:43 | |
| *** namnh has joined #openstack-nova | 12:43 | |
| *** links has quit IRC | 12:44 | |
| *** yamamoto has quit IRC | 12:46 | |
| *** acormier has quit IRC | 12:48 | |
| *** jdwidari is now known as plan908 | 12:48 | |
| *** plan908 is now known as jdwidari | 12:49 | |
| *** catintheroof has joined #openstack-nova | 12:50 | |
| *** hshiina has quit IRC | 12:50 | |
| *** catintheroof has quit IRC | 12:51 | |
| *** yamamoto has joined #openstack-nova | 12:51 | |
| *** catintheroof has joined #openstack-nova | 12:51 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Set group_members when converting to legacy request spec https://review.openstack.org/509766 | 12:51 |
| *** lyan has joined #openstack-nova | 12:54 | |
| *** vladikr has quit IRC | 12:54 | |
| *** kylek3h has joined #openstack-nova | 12:55 | |
| *** udesale has joined #openstack-nova | 12:58 | |
| *** mriedem has joined #openstack-nova | 12:59 | |
| mriedem | gmann: i'm working on a patch yeah | 12:59 |
| mriedem | johnthetubaguy: i'm +2 on stvnoyes' new style volume attachment live migration patch https://review.openstack.org/#/c/463987/ | 12:59 |
| *** dillaman has joined #openstack-nova | 12:59 | |
| johnthetubaguy | mriedem: yeah I saw that, its on my TODO queue | 13:00 |
| johnthetubaguy | production issues for a customer is slowing me down on reaching that quickly | 13:00 |
| andreykurilin | hi folks! Did you merged recently related to listing servers? | 13:01 |
| *** spectr has joined #openstack-nova | 13:02 | |
| andreykurilin | It looks like there is an issue with detailed listing servers. This action takes > 1 hour in our gates :( | 13:04 |
| *** yamamoto has quit IRC | 13:06 | |
| *** slaweq_ has quit IRC | 13:06 | |
| *** lbragstad has joined #openstack-nova | 13:06 | |
| *** coreywright has quit IRC | 13:09 | |
| *** esberglu has joined #openstack-nova | 13:09 | |
| *** eharney has joined #openstack-nova | 13:12 | |
| *** namnh has quit IRC | 13:15 | |
| *** namnh has joined #openstack-nova | 13:16 | |
| *** gbarros has joined #openstack-nova | 13:16 | |
| *** namnh has quit IRC | 13:20 | |
| *** slaweq_ has joined #openstack-nova | 13:20 | |
| *** baoli has joined #openstack-nova | 13:21 | |
| *** smatzek has quit IRC | 13:21 | |
| *** smatzek has joined #openstack-nova | 13:22 | |
| andreykurilin | my fault, detailed flag is not used. so the regular listing servers stucks | 13:22 |
| *** coreywright has joined #openstack-nova | 13:23 | |
| *** smatzek_ has joined #openstack-nova | 13:25 | |
| *** takashin has joined #openstack-nova | 13:25 | |
| *** smatzek_ has quit IRC | 13:25 | |
| tikitavi | I really need help, for more than a month the review https://review.openstack.org/#/c/500091/ didn't pass because one or more instances cannot start because of the errors in n-cond | 13:25 |
| *** smatzek_ has joined #openstack-nova | 13:25 | |
| *** gouthamr has joined #openstack-nova | 13:26 | |
| openstackgerrit | Kevin Lefevre proposed openstack/nova stable/pike: Support qemu >= 2.10 https://review.openstack.org/509774 | 13:27 |
| *** awaugama has joined #openstack-nova | 13:27 | |
| *** erlon has joined #openstack-nova | 13:29 | |
| *** smatzek has quit IRC | 13:29 | |
| *** smatzek_ has quit IRC | 13:30 | |
| *** ArchiFleKs has joined #openstack-nova | 13:31 | |
| *** manasm has joined #openstack-nova | 13:35 | |
| *** jaosorior has quit IRC | 13:35 | |
| *** alexchadin has quit IRC | 13:38 | |
| *** alexchadin has joined #openstack-nova | 13:39 | |
| *** spectr has quit IRC | 13:40 | |
| ArchiFleKs | Hi, do you know if this (https://review.openstack.org/#/c/505673/) can be backported to pike on other releases that run on Xenial, I have the issue on Pike with OpenStack Ansible, were the compute node have been update to qemu 2.10 | 13:41 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | 13:42 |
| *** archit has joined #openstack-nova | 13:43 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 13:43 |
| *** mriedem is now known as ronlund | 13:44 | |
| *** vladikr has joined #openstack-nova | 13:44 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 13:45 |
| *** alexchad_ has joined #openstack-nova | 13:46 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 13:47 |
| *** alexchadin has quit IRC | 13:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 13:48 |
| *** jaypipes has joined #openstack-nova | 13:52 | |
| *** slaweq_ has quit IRC | 13:54 | |
| *** lajoskatona has quit IRC | 13:57 | |
| ronlund | nova meeting in 3 minutes | 13:57 |
| *** slaweq_ has joined #openstack-nova | 13:57 | |
| *** spectr has joined #openstack-nova | 13:57 | |
| *** cfriesen has joined #openstack-nova | 13:58 | |
| *** crushil has joined #openstack-nova | 13:58 | |
| *** ronlund is now known as mriedem | 13:58 | |
| *** hongbin has joined #openstack-nova | 13:59 | |
| *** smatzek has joined #openstack-nova | 13:59 | |
| *** alexchad_ has quit IRC | 14:00 | |
| *** smatzek has quit IRC | 14:00 | |
| *** liangy has joined #openstack-nova | 14:00 | |
| *** lyan has quit IRC | 14:01 | |
| *** smatzek has joined #openstack-nova | 14:01 | |
| *** zzzeek has quit IRC | 14:03 | |
| *** yamamoto has joined #openstack-nova | 14:06 | |
| *** zzzeek has joined #openstack-nova | 14:07 | |
| *** spectr has quit IRC | 14:07 | |
| *** spectr has joined #openstack-nova | 14:08 | |
| *** jaosorior has joined #openstack-nova | 14:10 | |
| *** alexchadin has joined #openstack-nova | 14:11 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 14:11 |
| *** yamamoto has quit IRC | 14:13 | |
| *** namnh has joined #openstack-nova | 14:16 | |
| *** jheroux has joined #openstack-nova | 14:16 | |
| jaypipes | cdent: re: moving all the module-level DB access functions to the bottom of the resource_provider.py file, I'd prefer to do that all in a single commit later on. | 14:18 |
| cdent | that’s cool, just wanted to get it on the record | 14:18 |
| dansmith | jaypipes: yeah so I can -1 that one | 14:18 |
| dansmith | cdent: I dunno how you got that we organize files per my preference | 14:18 |
| dansmith | I mean, it is my preference that we order them definition first then use, but that wasn't my doing | 14:19 |
| cdent | I wasn’t saying you are the enforcer, I just happen to know your preference | 14:19 |
| cdent | so wanted to mention it so that you knew I wasn’t blithely ignoring it | 14:19 |
| cdent | I’m graciously taking it into account and choosing to consider it wrong ;) | 14:19 |
| dansmith | mkay, made it sound like all our files are like this because if not I'd throw a hissy | 14:19 |
| cdent | no sir | 14:20 |
| jaypipes | dansmith: that's fine. :) | 14:20 |
| *** cshen has quit IRC | 14:20 | |
| jaypipes | dansmith, cdent: is it just me or is the ubuntu xenial full tempest job just hosed? | 14:21 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Improve the performance of filtering instances by IP. https://review.openstack.org/509326 | 14:21 |
| dansmith | just you? | 14:21 |
| dansmith | we've been merging things | 14:22 |
| * jaypipes can't even get to logs.openstack.org URLs in any of the job output index htmls... | 14:22 | |
| jaypipes | maybe just needs a recheck.,.. | 14:22 |
| *** nikhil_k has joined #openstack-nova | 14:22 | |
| dansmith | neither I just now, so maybe something is up | 14:22 |
| cdent | logs.o.o has been behaving poorly for me of late | 14:22 |
| *** nikhil_k is now known as nikhil | 14:23 | |
| *** acormier has joined #openstack-nova | 14:25 | |
| *** lpetrut_ has joined #openstack-nova | 14:25 | |
| *** mdnadeem has quit IRC | 14:25 | |
| * efried noticed it was slow, but responded eventually. | 14:26 | |
| efried | cdent I rechecked yours, looked like the same thing as last time. | 14:26 |
| cdent | thanks | 14:27 |
| *** lpetrut has quit IRC | 14:27 | |
| efried | jaypipes One qq before I start slinging that spec: How are generic resources specified in the flavor? And what stage of reality is that stuff in (spec, proposed, merged, etc.)? | 14:29 |
| *** slaweq_ has quit IRC | 14:30 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405 | 14:33 |
| efried | edleafe Where's that agenda? https://wiki.openstack.org/wiki/Meetings/Scheduler just says "See mailing-list" | 14:35 |
| mriedem | andreas_s_: done https://review.openstack.org/#/admin/groups/511,members | 14:35 |
| *** gmann is now known as gmann_sleep | 14:35 | |
| edleafe | https://wiki.openstack.org/wiki/Meetings/NovaScheduler | 14:35 |
| edleafe | efried: ^^ | 14:36 |
| efried | edleafe Oh. Is that other one dead? | 14:36 |
| edleafe | efried: didn' | 14:37 |
| edleafe | didn't even know there was another one | 14:37 |
| edleafe | so yes | 14:37 |
| jaypipes | efried: by "generic resources" are you referring to things like vcpu, memory_mb, etc? | 14:37 |
| *** felipemonteiro_ has joined #openstack-nova | 14:37 | |
| *** alexchadin has quit IRC | 14:37 | |
| *** slaweq_ has joined #openstack-nova | 14:38 | |
| efried | jaypipes Ultimately those would be included, yes, but I understand they're treated specially today because they're (mandatory) separate fields in a flavor. | 14:38 |
| efried | jaypipes But for purposes of the immediate discussion, let's say it's something totally generic that isn't handled by RT at all. | 14:38 |
| *** felipemonteiro__ has joined #openstack-nova | 14:38 | |
| efried | For example, where does an Ironic flavor specify the ironic node inventory it wants? | 14:39 |
| efried | (Or is that another special case?) | 14:39 |
| andreas_s_ | mriedem, Thanks! | 14:39 |
| *** manasm has quit IRC | 14:41 | |
| cdent | efried: that’s kind of a special case and was sort of the genesis of custom resource classes | 14:41 |
| *** armax has joined #openstack-nova | 14:41 | |
| cdent | because the idea was that an ironic node is a single consumable thing | 14:41 |
| efried | edleafe I updated https://wiki.openstack.org/wiki/Meetings/Scheduler to point to https://wiki.openstack.org/wiki/Meetings/NovaScheduler | 14:42 |
| *** felipemonteiro_ has quit IRC | 14:42 | |
| efried | cdent jaypipes So does that mean we actually don't have a way to specify generic resources in flavors yet? | 14:42 |
| jaypipes | efried: in the flavor's extra_specs. there's a special key pattern called "resources:$RESOURCE_CLASS" and the value is the amount of that resource class the flavor wants. | 14:43 |
| efried | jaypipes Ah, perfect. And that's been implemented already? | 14:43 |
| jaypipes | efried: those special extra_specs resources: things are called resource overrides. | 14:43 |
| jaypipes | efried: yes. | 14:44 |
| jaypipes | efried: edleafe and alex_xu implemented that IIRC | 14:44 |
| efried | jaypipes Cool; and (as the name "overrides" implies) I can specify "standard" resources there too, like VCPU, DISK_GB, etc., and their corresponding hardcoded fields will be ignored? | 14:44 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.rebuild_scheduled notification https://review.openstack.org/473929 | 14:47 |
| efried | jaypipes So in a nutshell, this new thing will take resources$X:$RESOURCE_CLASS=$N and traits$X:$TRAIT=required and translate them into GET /allocation_candidates?resources$X=$N...&required$X=$TRAIT | 14:47 |
| efried | jaypipes Does that align with your thinking? | 14:47 |
| jaypipes | efried: precisely. | 14:48 |
| efried | beaut | 14:48 |
| *** READ10 has joined #openstack-nova | 14:48 | |
| mriedem | gibi: thanks for https://review.openstack.org/#/c/509759/ i was just looking at fixing that | 14:48 |
| mriedem | gibi: comment within | 14:48 |
| *** yamamoto has joined #openstack-nova | 14:48 | |
| *** yamamoto has quit IRC | 14:48 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix minor input items from previous patches https://review.openstack.org/506416 | 14:48 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595 | 14:48 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 14:48 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 14:48 |
| dansmith | mriedem: rebased now that your metadata cleanup has merged ^ | 14:49 |
| mriedem | ok | 14:49 |
| *** tssurya has quit IRC | 14:49 | |
| mriedem | says the bottom change is in merge conflict... ? | 14:49 |
| dansmith | um | 14:50 |
| mriedem | but it's queued up so idk | 14:50 |
| dansmith | Iliterally just pushed | 14:50 |
| mriedem | maybe a zuulv3 thing | 14:50 |
| *** gyee has joined #openstack-nova | 14:51 | |
| gibi | mriedem: I will push a new ps soon with your suggestion implemented | 14:52 |
| mriedem | dansmith: i see we're running singleconductor mode in grenade between pike->queens, couldn't we be running superconductor? | 14:52 |
| mriedem | maybe that would require making grenade aware of that kind of mode? | 14:53 |
| *** dillaman has quit IRC | 14:53 | |
| dansmith | yeah I think we just punted on any sort of telling grenade about that since we couldn't upgrade from one layout to the other | 14:53 |
| dansmith | I dunno how much it really needs to know about, but I haven't looked | 14:53 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Improve the performance of filtering instances by IP. https://review.openstack.org/509326 | 14:55 |
| mriedem | https://github.com/openstack-dev/grenade/blob/03de9e0fc7f4fc50a00db5d547413e26cf0780dd/grenade.sh#L222 | 14:55 |
| mriedem | might just be worth removing that from master grenade and see how it goes | 14:55 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576 | 14:55 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 14:56 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 14:56 |
| dansmith | jaypipes: cdent: I didn't see that failure in ^ locally, so this is a rebase and we'll see how it goes this time | 14:56 |
| jaypipes | kk | 14:57 |
| *** Swami has joined #openstack-nova | 14:58 | |
| *** karthiks has quit IRC | 14:59 | |
| *** takashin has left #openstack-nova | 15:00 | |
| jaypipes | dansmith, mriedem: I'm ready on this one after a number of back and forths with johnthetubaguy. https://review.openstack.org/#/c/507052/ | 15:00 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214 | 15:01 |
| johnthetubaguy | jaypipes: thanks for your time on that, glad to move that forward | 15:01 |
| *** penick has joined #openstack-nova | 15:02 | |
| dansmith | edleafe: is this fail related to your change? http://logs.openstack.org/15/486215/14/check/gate-nova-tox-functional-py35-ubuntu-xenial/f7d2653/testr_results.html.gz | 15:02 |
| *** gouthamr_ has joined #openstack-nova | 15:03 | |
| mriedem | jaypipes: i'll put it in the tab qeuue | 15:04 |
| mriedem | *queue | 15:04 |
| *** gouthamr has quit IRC | 15:04 | |
| *** gouthamr_ is now known as gouthamr | 15:04 | |
| mriedem | dansmith: i'll go through https://review.openstack.org/498950 again | 15:04 |
| *** tuanla_fujitsu has joined #openstack-nova | 15:05 | |
| *** Tom___ has quit IRC | 15:05 | |
| *** tuanla__ has joined #openstack-nova | 15:05 | |
| dansmith | mriedem: thanks, I don't think you had any feedback on that did you? | 15:05 |
| mriedem | yes... | 15:05 |
| *** Oku_OS is now known as Oku_OS-away | 15:05 | |
| *** Tom has joined #openstack-nova | 15:05 | |
| dansmith | oh the fail cleanup, right | 15:05 |
| *** jaosorior has quit IRC | 15:07 | |
| tikitavi | look to the nova-conductor logs, please (http://logs.openstack.org/91/500091/8/check/gate-functional-neutron-dsvm-ec2api-ubuntu-xenial/206b664/). We have problem with starting instances in stable/ocata and stable/pike. | 15:09 |
| *** Tom has quit IRC | 15:10 | |
| dansmith | tikitavi: you mean the traceback? | 15:10 |
| tikitavi | yes! | 15:10 |
| dansmith | 2017-10-05 10:48:16.014 30844 ERROR oslo_messaging.rpc.server IOError: [Errno 2] No such file or directory: '/usr/local/lib/python2.7/dist-packages/six-1.10.0.dist-info/METADATA' | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: de-ORM ResourceProvider.get_by_uuid() https://review.openstack.org/509025 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: Move RP._get|set_aggregates() to module scope https://review.openstack.org/509026 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: Remove RP.get_traits() method https://review.openstack.org/509027 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: move RP._set_traits() to module scope https://review.openstack.org/509028 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove CRUD operations on Inventory class https://review.openstack.org/509029 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033 | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035 | 15:10 |
| dansmith | tikitavi: seems clearly like some installation problem | 15:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: remove _HasAResourceProvider mixin https://review.openstack.org/509036 | 15:10 |
| *** ssurana has joined #openstack-nova | 15:11 | |
| *** slaweq_ has quit IRC | 15:11 | |
| tikitavi | dansmith: it is repeatable. we can't pass this review for more than a month | 15:11 |
| dansmith | tikitavi: I understand, but still, it's a package problem, nothing to do with any nova ode | 15:12 |
| dansmith | *code | 15:12 |
| edleafe | dansmith: I ran those tests locally w/o a fail, so I don't think so. | 15:13 |
| dansmith | edleafe: yea, I rechecked and noted that the next patch didn't hit the same failure | 15:13 |
| *** slaweq_ has joined #openstack-nova | 15:14 | |
| *** ssurana has left #openstack-nova | 15:15 | |
| mriedem | sdague: did your "wait for host mapping" change in devstack get backported to stable/pike? i'm seeing http://logs.openstack.org/56/509656/1/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/6851396/logs/old/screen-n-cond.txt.gz#_2017-10-04_23_25_13_456 in stable/pike grenade runs only | 15:16 |
| mriedem | oh hm https://review.openstack.org/#/c/490635 | 15:17 |
| sdague | right, that will need to land there. I can push it in | 15:17 |
| mriedem | +1 fwiw | 15:18 |
| mriedem | :) | 15:18 |
| *** andreas_s_ has quit IRC | 15:21 | |
| dansmith | johnthetubaguy: a spec from you with no discernible typos.. jaypipes must have been up all night. :P | 15:21 |
| jaypipes | lol | 15:21 |
| johnthetubaguy | dansmith: him and efried got most of them the previous time :) | 15:21 |
| dansmith | johnthetubaguy: heroes, those guys. | 15:22 |
| mriedem | ooo -1 | 15:22 |
| mriedem | i'll fix after it's merged | 15:22 |
| * jaypipes goes to add johnthetubaguy's spec to his article about placement stuff in queens... | 15:22 | |
| *** MVenesio has joined #openstack-nova | 15:23 | |
| *** shvepsy has joined #openstack-nova | 15:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Rename ironic-driver-traits spec https://review.openstack.org/509812 | 15:25 |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Use neutron's new port binding API https://review.openstack.org/375580 | 15:25 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: fix unstable shelve offload functional tests https://review.openstack.org/509759 | 15:26 |
| gibi | mriedem: fixed your comment in the shelve offload test bugfix ^^ | 15:26 |
| shvepsy | Hi all! Can you help me with my issue. | 15:27 |
| shvepsy | How I can call quiesce instance (freeze fs) from python API? | 15:27 |
| mdbooth | dansmith: I vaguely recall some error from the dim and distants depths of time which meant that an error in a periodic task would prevent that task from continuing to run. Does that ring any bells with you? | 15:27 |
| mriedem | johnthetubaguy: the thing on L37 of your spec confuses me | 15:27 |
| mriedem | aren't you just describing the flavor-access API? | 15:27 |
| dansmith | mdbooth: no, not that I know of | 15:27 |
| johnthetubaguy | mriedem: I guess | 15:28 |
| johnthetubaguy | mriedem: its just a public flavor and a private flavor, mapping to different groups of hosts | 15:28 |
| mriedem | johnthetubaguy: so flavor-access with host aggregates? | 15:28 |
| dansmith | mriedem: I'm not sure what your point is? that you can accomplish the same thing with aggregates and a private flavor? | 15:30 |
| mriedem | yeah | 15:30 |
| dansmith | but you can't right? because aggregates are around services not compute nodes | 15:30 |
| dansmith | and | 15:30 |
| johnthetubaguy | mriedem: well, ironic and host aggregates doesn't really do it | 15:30 |
| dansmith | that's just one use of the general traits stuff being described here, | 15:30 |
| dansmith | which also helps with external services using the same ironic | 15:30 |
| dansmith | like some dedicated to nova and some for direct use | 15:31 |
| dansmith | and nova aggregates wouldn't help with that | 15:31 |
| johnthetubaguy | ++ | 15:31 |
| mriedem | ok i figured the host aggregates thing screwed it up | 15:32 |
| mriedem | btw, at some point, can someone that understands the issues with host aggregates and ironic put some words about that in a section in https://docs.openstack.org/nova/pike/user/aggregates.html ? | 15:33 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwitdh resource provider https://review.openstack.org/502306 | 15:33 |
| mriedem | because i think it's purely tribal knowledge | 15:33 |
| *** MVenesio has quit IRC | 15:34 | |
| *** MVenesio has joined #openstack-nova | 15:36 | |
| jaypipes | ralonsoh: I'm going to have sean-k-mooney reach over and slap you if you don't fix that "bandwitdh" in your commit message :P | 15:38 |
| ralonsoh | jaypipes: sorry!!! | 15:38 |
| ralonsoh | one sec... | 15:38 |
| jaypipes | ralonsoh: lol, it's ok :) | 15:38 |
| *** jogo has quit IRC | 15:39 | |
| *** hemna__ has joined #openstack-nova | 15:39 | |
| *** pcaruana has quit IRC | 15:39 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 15:40 |
| jaypipes | \o/ | 15:40 |
| * edleafe needs more coffee | 15:41 | |
| *** jmlowe_ has joined #openstack-nova | 15:41 | |
| *** hemna_ has quit IRC | 15:42 | |
| *** jmlowe has quit IRC | 15:43 | |
| *** hemna_ has joined #openstack-nova | 15:43 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/504895 | 15:43 |
| mriedem | easy functional test race fix if someone wants to +W https://review.openstack.org/#/c/509759/ | 15:43 |
| *** hemna__ has quit IRC | 15:44 | |
| dansmith | oh this is the one I was just asking edleafe about | 15:44 |
| mriedem | shelve tests? | 15:44 |
| efried | mriedem Would a rename followup to https://review.openstack.org/#/c/507052/ resolve the mismatched bp/spec name issue, or does the old name stick around somehow? | 15:44 |
| mriedem | efried: see the patch on top | 15:44 |
| efried | mriedem Nyak. | 15:44 |
| *** hemna__ has joined #openstack-nova | 15:46 | |
| *** slaweq_ has quit IRC | 15:46 | |
| *** chyka has joined #openstack-nova | 15:47 | |
| *** hemna_ has quit IRC | 15:48 | |
| *** tikitavi has quit IRC | 15:49 | |
| *** yamamoto has joined #openstack-nova | 15:49 | |
| *** slaweq_ has joined #openstack-nova | 15:51 | |
| *** spectr has quit IRC | 15:51 | |
| *** ragiman has quit IRC | 15:51 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: fix unstable shelve offload functional tests https://review.openstack.org/509818 | 15:51 |
| dansmith | cdent: fwiw, I agree with you on this: https://review.openstack.org/#/c/495854/4/nova/scheduler/utils.py | 15:51 |
| dansmith | I would have made it a classmethod | 15:51 |
| dansmith | not a big deal of course, but that's what I would have done | 15:51 |
| *** namnh has quit IRC | 15:52 | |
| cdent | <british>cheers | 15:52 |
| *** namnh has joined #openstack-nova | 15:53 | |
| *** acormier has quit IRC | 15:53 | |
| edleafe | dansmith: heh, I had a 'from_host_state()' method on the orginal version. Should have left it there. :) | 15:54 |
| dansmith | just my preference.. doesn't matter | 15:55 |
| dansmith | edleafe: out of curiosity, why is this three patches instead of two with the object definition first? | 15:56 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove metadata/system_metadata filter handling from get_all https://review.openstack.org/509819 | 15:56 |
| dansmith | looking now, it seems like if you did the object patch first, you could have squashed the first and third together, which would be less back and forth | 15:56 |
| *** nicolasbock_ has joined #openstack-nova | 15:56 | |
| *** namnh has quit IRC | 15:57 | |
| *** yamamoto has quit IRC | 15:58 | |
| *** baoli has quit IRC | 15:59 | |
| *** baoli has joined #openstack-nova | 16:00 | |
| *** jpena is now known as jpena|off | 16:05 | |
| *** jpena|off is now known as jpena | 16:12 | |
| *** kylek3h has quit IRC | 16:12 | |
| *** kylek3h has joined #openstack-nova | 16:12 | |
| *** rcernin has quit IRC | 16:13 | |
| *** rcernin has joined #openstack-nova | 16:13 | |
| *** psachin has quit IRC | 16:14 | |
| *** rcernin has quit IRC | 16:14 | |
| *** manasm has joined #openstack-nova | 16:15 | |
| *** Apoorva has joined #openstack-nova | 16:19 | |
| *** namnh has joined #openstack-nova | 16:22 | |
| *** slaweq_ has quit IRC | 16:24 | |
| *** slaweq_ has joined #openstack-nova | 16:26 | |
| *** ctrath has joined #openstack-nova | 16:27 | |
| *** tesseract has quit IRC | 16:29 | |
| *** ralonsoh has quit IRC | 16:31 | |
| *** gouthamr_ has joined #openstack-nova | 16:32 | |
| *** mvk has quit IRC | 16:33 | |
| *** gouthamr has quit IRC | 16:35 | |
| *** Tom_ has joined #openstack-nova | 16:36 | |
| *** Swami has quit IRC | 16:37 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: Add validation for cpu_realtime_mask https://review.openstack.org/468203 | 16:39 |
| *** baoli has quit IRC | 16:39 | |
| *** Tom_ has quit IRC | 16:40 | |
| *** baoli has joined #openstack-nova | 16:44 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 16:44 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 16:44 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 16:44 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 16:44 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 16:44 |
| *** zhenq has joined #openstack-nova | 16:49 | |
| *** udesale has quit IRC | 16:49 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: fix unstable shelve offload functional tests https://review.openstack.org/509818 | 16:50 |
| *** tesseract has joined #openstack-nova | 16:50 | |
| *** derekh has quit IRC | 16:51 | |
| openstackgerrit | Merged openstack/nova master: Add attachment_get call to volume/cinder_api https://review.openstack.org/506805 | 16:54 |
| *** vks1 has joined #openstack-nova | 16:55 | |
| *** slaweq_ has quit IRC | 16:58 | |
| mriedem | melwitt: you should probably look at this one https://review.openstack.org/#/c/506822/ | 17:01 |
| mriedem | i think it's ok, but don't want to mess it up | 17:01 |
| dansmith | mriedem: https://review.openstack.org/#/c/498950/16/nova/tests/unit/conductor/tasks/test_migrate.py line 119, I asserted that it chose the migration type correctly | 17:04 |
| dansmith | mriedem: did you mean I didn't check that it detects resizes? | 17:04 |
| *** tuanla_fujitsu has quit IRC | 17:05 | |
| *** tuanla__ has quit IRC | 17:05 | |
| *** slaweq_ has joined #openstack-nova | 17:07 | |
| *** ijw has joined #openstack-nova | 17:07 | |
| mriedem | dansmith: yeah | 17:09 |
| dansmith | okay | 17:09 |
| mriedem | but a follow up is fine | 17:09 |
| *** Swami has joined #openstack-nova | 17:09 | |
| *** gbarros has quit IRC | 17:10 | |
| *** lucasagomes is now known as lucas-afk | 17:11 | |
| *** yamamoto has joined #openstack-nova | 17:11 | |
| *** gbarros has joined #openstack-nova | 17:13 | |
| *** Tom____ has joined #openstack-nova | 17:16 | |
| *** gjayavelu has joined #openstack-nova | 17:18 | |
| *** Tom____ has quit IRC | 17:21 | |
| *** tesseract has quit IRC | 17:22 | |
| *** markmcclain has quit IRC | 17:26 | |
| *** markmcclain has joined #openstack-nova | 17:29 | |
| *** vks1 has quit IRC | 17:30 | |
| *** dtantsur is now known as dtantsur|afk | 17:32 | |
| *** jpena is now known as jpena|off | 17:32 | |
| *** mvk has joined #openstack-nova | 17:32 | |
| *** penick has quit IRC | 17:32 | |
| *** gjayavelu has quit IRC | 17:34 | |
| *** mingyu has quit IRC | 17:37 | |
| *** lbragstad has quit IRC | 17:37 | |
| *** namnh has quit IRC | 17:38 | |
| *** penick has joined #openstack-nova | 17:39 | |
| *** slaweq_ has quit IRC | 17:40 | |
| *** mingyu has joined #openstack-nova | 17:41 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add test for resize migration type detection https://review.openstack.org/509852 | 17:41 |
| *** lpetrut_ has quit IRC | 17:41 | |
| *** slaweq_ has joined #openstack-nova | 17:43 | |
| *** yamahata has joined #openstack-nova | 17:43 | |
| *** xyang1 has joined #openstack-nova | 17:45 | |
| *** mingyu has quit IRC | 17:46 | |
| *** manasm has quit IRC | 17:46 | |
| *** xyang1 has left #openstack-nova | 17:47 | |
| *** gszasz has quit IRC | 17:51 | |
| *** hamzy has quit IRC | 17:57 | |
| *** claudiub has quit IRC | 17:57 | |
| *** tbachman has joined #openstack-nova | 18:01 | |
| *** dikonoor has quit IRC | 18:05 | |
| *** Tom___ has joined #openstack-nova | 18:07 | |
| *** lbragstad has joined #openstack-nova | 18:10 | |
| *** Tom___ has quit IRC | 18:11 | |
| openstackgerrit | Merged openstack/nova master: Update RT aggregate map less frequently https://review.openstack.org/489633 | 18:13 |
| *** dillaman has joined #openstack-nova | 18:13 | |
| openstackgerrit | Merged openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | 18:14 |
| dansmith | man, having only 80% of nodepool available has a bigger impact on throughput than I would have expected | 18:14 |
| *** slaweq_ has quit IRC | 18:16 | |
| *** ijw has quit IRC | 18:16 | |
| mriedem | hey, they can surely flip the switch back to 0% throughput again | 18:16 |
| mriedem | if you don't pipe down | 18:16 |
| mriedem | smith | 18:16 |
| dansmith | heh | 18:16 |
| melwitt | mriedem: sure, will do | 18:17 |
| *** ctrath has left #openstack-nova | 18:19 | |
| *** slaweq_ has joined #openstack-nova | 18:20 | |
| *** yamamoto has quit IRC | 18:24 | |
| *** avolkov has quit IRC | 18:26 | |
| *** yamamoto has joined #openstack-nova | 18:27 | |
| *** lpetrut_ has joined #openstack-nova | 18:29 | |
| *** nikhil has quit IRC | 18:32 | |
| *** yamamoto has quit IRC | 18:32 | |
| *** sambetts|afk has quit IRC | 18:33 | |
| *** sambetts_ has joined #openstack-nova | 18:36 | |
| *** ijw has joined #openstack-nova | 18:39 | |
| cdent | mriedem: is this back in business? https://review.openstack.org/#/c/267587/ | 18:40 |
| *** ijw has quit IRC | 18:41 | |
| *** hamzy has joined #openstack-nova | 18:41 | |
| *** gbarros has quit IRC | 18:45 | |
| mriedem | cdent: the spec isn't approved yet | 18:46 |
| mriedem | cdent: the multiattach stuff is all dependent on the new style volume attachment stuff | 18:46 |
| cdent | ah. I got so excited by seeing it laying around. | 18:47 |
| *** tbachman has quit IRC | 18:47 | |
| cdent | the feature that failed to live but won’t die | 18:48 |
| dansmith | too fast to live, too young to die | 18:49 |
| dansmith | james dean. | 18:49 |
| *** slaweq_ has quit IRC | 18:53 | |
| *** slaweq_ has joined #openstack-nova | 18:56 | |
| *** Tom___ has joined #openstack-nova | 18:57 | |
| *** Tom___ has quit IRC | 19:02 | |
| *** baoli has quit IRC | 19:02 | |
| *** baoli has joined #openstack-nova | 19:02 | |
| dansmith | mriedem: I was just looking at my live migration patch wondering why it wasn't failing a test because it doesn't clean up by migration uuid on rollback | 19:02 |
| dansmith | and then ran tests again to check, which it failed | 19:03 |
| dansmith | confused because I thought this was passing | 19:03 |
| dansmith | found the test that tests this, which I fail | 19:03 |
| dansmith | dug through the path to figure out where we're cleaning up now | 19:03 |
| *** tbachman has joined #openstack-nova | 19:03 | |
| dansmith | didn't find it, almost filed a bug | 19:03 |
| dansmith | re-read test, found bug reference, found patch that addeded | 19:03 |
| dansmith | remembered I reviewed and approved that | 19:03 |
| dansmith | and that you have a fix on top of it waiting for +W | 19:04 |
| dansmith | so, altogether a pretty productive hour.. total work product output: one +W vote | 19:04 |
| *** MasterOfBugs has joined #openstack-nova | 19:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif stable/newton: Updated from global requirements https://review.openstack.org/373293 | 19:12 |
| openstackgerrit | Merged openstack/nova master: fix unstable shelve offload functional tests https://review.openstack.org/509759 | 19:13 |
| *** elod has quit IRC | 19:13 | |
| dansmith | jaypipes: this is needed for the migration uuid stuff, if you wanna slap yer +W on it: https://review.openstack.org/#/c/508699/ | 19:15 |
| *** markvoelker_ has joined #openstack-nova | 19:15 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 19:15 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 19:15 |
| jaypipes | dansmith: on it. | 19:15 |
| *** markvoelker has quit IRC | 19:15 | |
| jaypipes | dansmith: the whole series I presume? | 19:16 |
| dansmith | jaypipes: just that linked one is what I want right now | 19:16 |
| dansmith | jaypipes: I was just pushing the series in parallel to begging for your +W | 19:16 |
| dansmith | jaypipes: i.e. https://review.openstack.org/#/c/508699/ | 19:16 |
| mriedem | dansmith: heh, well, i did beg for reviews a few times on that one | 19:18 |
| dansmith | mriedem: I know, I had a lot of stuff in my head at the time | 19:18 |
| mriedem | oh wait not that one | 19:18 |
| mriedem | was thinking this one https://review.openstack.org/#/c/507687/ | 19:19 |
| mriedem | is that what you meant? | 19:19 |
| mriedem | yeah it is, nvm | 19:19 |
| mriedem | just about done with this epic internal email on scheduler configuration | 19:20 |
| dansmith | mriedem: I meant the one I linked | 19:20 |
| dansmith | oh | 19:20 |
| dansmith | you mean my earlier story, yes, I was talkin gabout 687 | 19:20 |
| *** tbachman has quit IRC | 19:22 | |
| mriedem | 36 minutes until a call about instance users | 19:22 |
| mriedem | yay! | 19:22 |
| *** slaweq_ has quit IRC | 19:22 | |
| *** slaweq_ has joined #openstack-nova | 19:23 | |
| mriedem | i need a trombone as mine is in the shop | 19:23 |
| dansmith | mriedem: btw, since I re-pushed that whole migration set, I squashed that extra test patch into the bottom | 19:23 |
| melwitt | I can't remember what "instance users" was about. maybe I should be grateful | 19:23 |
| sdague | anyone seen any of the john hopkins folks recently on irc? I was going through their image signing spec and code and just had a few quick questions | 19:24 |
| mriedem | sdague: nope | 19:25 |
| sdague | neither dane nor kaitlin are here unless I suck at tab completing. But I'm not sure if others have irc nicks. | 19:26 |
| mriedem | that's who i was looking for | 19:28 |
| mriedem | cdent: btw this is the goal https://review.openstack.org/#/c/330285/ | 19:29 |
| mriedem | for the new style volume attach stuff, | 19:29 |
| mriedem | jgriffith is working on a new microversion on the cinder side, patch is up for that now but needs some work, | 19:29 |
| mriedem | but then the nova change will require that new cinder microversion and we can work on getting the nova patch in, | 19:29 |
| mriedem | and then it's multiattach time | 19:29 |
| * cdent considers holding his breath | 19:30 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Make setenv consistent for unit, func, and api-samples https://review.openstack.org/507976 | 19:30 |
| mriedem | cdent: have your corporate overlords expressed an interest in multiattach? | 19:30 |
| cdent | not that I’m aware of, but my corporate overlords interests via me are pretty focused | 19:31 |
| cdent | I would assume they probably are interested | 19:31 |
| cdent | my interest is mostly sparked by observing it for enough ptgs, summits, midcycles to think of it is a somewhat annoying friend | 19:32 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make setenv consistent for unit, func, and api-samples https://review.openstack.org/507976 | 19:33 |
| mriedem | melwitt: i believe, to summarize, an instance user was the ability for a guest to get a token to do things | 19:34 |
| melwitt | ohhh | 19:34 |
| melwitt | that was the new vendordata thing right? | 19:35 |
| mriedem | but i was never really involved (by my own choosing) in that discussion, so i'm blissfully ignorant | 19:35 |
| jaypipes | dansmith: k, done | 19:35 |
| melwitt | or that discussion resulted in the new vendordata I thought | 19:35 |
| mriedem | melwitt: kind of yeah | 19:35 |
| dansmith | well, only for certain types of things | 19:35 |
| dansmith | vendordata could be used for some of that, | 19:35 |
| mriedem | vendordata was also to get rid of hooks | 19:35 |
| mriedem | which, now that i think about it, we haven't removed yet | 19:36 |
| dansmith | but I think the real use case requires more integration than that | 19:36 |
| dansmith | the real instance_users case I mean | 19:36 |
| melwitt | okay | 19:36 |
| cdent | dansmith: your thing about microversions on Selection objects, there’s some discussion about it on https://review.openstack.org/#/c/498830/ (patchset 7) where I expressed a lot of confusion that ed and jay tried to clear it up. Eventually I capitulated | 19:36 |
| dansmith | cdent: okay I'm fairly concerned about this, but I shall go read | 19:37 |
| *** Tom___ has joined #openstack-nova | 19:38 | |
| cdent | It may not illuminate, but it may | 19:38 |
| mriedem | uh oh, lar bear is home | 19:38 |
| *** pchavva has quit IRC | 19:39 | |
| dansmith | cdent: uh, I certainly did not agree to that which was agreed to in denver | 19:40 |
| dansmith | cdent: that makes the microversion thing way too fluid, IMHO | 19:40 |
| dansmith | cdent: you remove a field in 1.5, add it again in 1.50 with a different meaning or format, and boom | 19:40 |
| jaypipes | dansmith: welcome to hear other ideas here, but please do read the one long comment I gave to cdent on PS7 on that patch | 19:41 |
| *** Tom___ has quit IRC | 19:42 | |
| dansmith | jaypipes: yeah I completely understand the situation | 19:42 |
| cdent | “microversion thing way too fluid” was my concern too, but, like I said, I decided to capitulate | 19:42 |
| dansmith | jaypipes: but making placement just be "meh" about the mismatch between versions and formats is totally not the right plan, IMHO | 19:43 |
| cdent | my capitulation is a somewhat more complex version of the plan that ed original presented | 19:43 |
| dansmith | jaypipes: all we have to do is pass the microversion that matches the blob, and have the report client use that version | 19:43 |
| dansmith | (in terms of "other ideas") | 19:44 |
| edleafe | dansmith: when will an allocation not match the version placement understands? | 19:46 |
| dansmith | edleafe: it's not about that | 19:47 |
| * cdent no longer thinks it makes sense to be completionist in the friday placement updates | 19:47 | |
| cdent | do people have preferences on what stays or goes? | 19:47 |
| dansmith | edleafe: it's about the version sent by the client not matching the payload | 19:47 |
| dansmith | edleafe: and the server side having to just "be flexible" about what it thinks the client wants | 19:47 |
| edleafe | dansmith: yes, I understand. But even an old client will have gotten (and be returning) a current-version allocation | 19:48 |
| dansmith | the client has said "this is a 1.5 thing" and the server says "well, kinda looks like a 1.10 thing, so...I'll just assume 1.10" | 19:48 |
| dansmith | edleafe: the allocation, sure, like Isaid initially: | 19:48 |
| edleafe | the client isn't saying that. It's saying: "here is the thing you just sent me" | 19:48 |
| dansmith | edleafe: we throw the a-r at placement with the version from which it came at the scheduler level, and then it can fetch the resulting allocation at the version it understands | 19:48 |
| *** baoli has quit IRC | 19:48 | |
| dansmith | edleafe: it has a version in the headers, so... it's saying the version it thinks the thing is | 19:49 |
| edleafe | an allocation_request is not affected by the scheduler | 19:49 |
| edleafe | it is an opaque blob that placement just returned to scheduler | 19:49 |
| *** baoli has joined #openstack-nova | 19:49 | |
| dansmith | at the version that the scheduler understands | 19:49 |
| dansmith | because the scheduler has said "I understand 1.10" | 19:50 |
| edleafe | wait - so now we're handling old *schedulers*? | 19:50 |
| melwitt | sdague, mriedem: the devstack "pip_install -U --force PasteDeploy" hack worked, FYI. so I'm onto the next problem now | 19:50 |
| edleafe | I thought the only situation of version mismatch was a claim from an old cell conductor | 19:50 |
| dansmith | edleafe: old scheduler? | 19:51 |
| dansmith | placement is/will be a separate thing, upgraded at different times from the scheduler, conductors, etc | 19:51 |
| edleafe | dansmith: -> because the scheduler has said "I understand 1.10" | 19:51 |
| dansmith | if cdent had his way, it would already be (/nudge) | 19:51 |
| dansmith | this is why we version shit | 19:52 |
| sdague | melwitt: \o/ | 19:52 |
| * cdent tries to catch up | 19:52 | |
| edleafe | allocation_requests are short-lived. They are not persisted | 19:52 |
| melwitt | progress :) | 19:52 |
| dansmith | edleafe: that has nothing to do with anything | 19:52 |
| dansmith | edleafe: services are long-lived | 19:52 |
| *** vvargaszte has joined #openstack-nova | 19:53 | |
| dansmith | edleafe: I upgrade placement to rocky a month before I upgrade my nova | 19:53 |
| edleafe | They are obtained from placement, and returned unchanged | 19:53 |
| dansmith | edleafe: the scheduler has to tell placement what it understands | 19:53 |
| edleafe | but a change to the format of an allocation_request does not affect the scheduler | 19:53 |
| edleafe | it's an opaque blob | 19:53 |
| dansmith | edleafe: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L270 | 19:53 |
| dansmith | that stops working when placement is kicked out | 19:54 |
| dansmith | it's not even really right currently, but we have't done it | 19:54 |
| dansmith | if placement were not lockstep with scheduler, | 19:54 |
| dansmith | then we'd break if we don't tell placement what we understand | 19:54 |
| dansmith | this is the whole point of microversions | 19:54 |
| edleafe | dansmith: for everything else between the scheduler and placement I agree | 19:54 |
| edleafe | but an a-r is an opaque blob, and will *always* be in the format that placement wants | 19:55 |
| *** vvargaszte has quit IRC | 19:57 | |
| *** READ10 has quit IRC | 19:59 | |
| sdague | mriedem: you want to land this backport - https://review.openstack.org/#/c/509774/ ? | 19:59 |
| *** smatzek has quit IRC | 19:59 | |
| *** smatzek has joined #openstack-nova | 20:00 | |
| *** smatzek has quit IRC | 20:00 | |
| *** xyang1 has joined #openstack-nova | 20:00 | |
| dansmith | edleafe: a-r cannot be an opaque blob to the scheduler because it has to interpret the results to weigh things | 20:00 |
| *** ianw|pto is now known as ianw | 20:00 | |
| dansmith | edleafe: that means that scheduler and placement have to agree on a format between them in order for the scheduler to reliably do its thing | 20:01 |
| dansmith | if the scheduler is behind placement and received an older formatted thing, | 20:01 |
| cdent | it uses the providers half of the tuple to weigh, not the a-r? | 20:01 |
| edleafe | cdent: correct | 20:02 |
| dansmith | but compute throws that at placement either at its version or "no version assume latest" it may be wrong | 20:02 |
| cdent | nm, I guess it has to scan the a-r | 20:02 |
| *** smatzek has joined #openstack-nova | 20:02 | |
| edleafe | why? | 20:02 |
| dansmith | cdent: doesn't matter if it did.. surely we're not suggesting having a versioned document where one part of it is "may be newer, don't look behind this curtain" | 20:02 |
| cdent | we do have a section that we’re claiming is “don’t look behind this curtain” | 20:02 |
| dansmith | but that's crazy | 20:03 |
| mriedem | penick: have you seen this thread? http://lists.openstack.org/pipermail/openstack-dev/2017-September/122904.html | 20:03 |
| dansmith | that's not how people work with APIs | 20:03 |
| mriedem | penick: rybridges: aren't you guys doing something similar? | 20:03 |
| cdent | edleafe: I may be wrong. I was thinking that during the process of choosing which of the a-rs to use, you have to know the rp ids | 20:03 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/509892 | 20:03 |
| cdent | dansmith: I agree with you, for the most part, I’m just reporting on “things we say" | 20:03 |
| penick | mriedem indeed we are, I think he asked about it in channel last week. I've been meaning to reply to the thread | 20:04 |
| cdent | “opaque" | 20:04 |
| dansmith | cdent: yep, understand | 20:04 |
| mriedem | penick: ah cool, on a call with him now | 20:04 |
| mriedem | this is over my head | 20:04 |
| dansmith | cdent: the only opaqueness I think we need is just between scheduler and the things downstream of it which need to throw it back at placement | 20:04 |
| cdent | (under it all I find the allocation_candidates thing way overly-specific and not very api-like, but it is is what we’ve reached as a workable solution when many other things would not, so… hard to keep my guns) | 20:04 |
| dansmith | having a big chunk of data that looks useful being exposed to the client and told that there be dragons within is not a good plan, IMHO | 20:05 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | 20:05 |
| edleafe | dansmith: having a big chunk of data being passed around is not a good plan IMO either, but this is what we are working with | 20:05 |
| *** tbachman has joined #openstack-nova | 20:05 | |
| *** smatzek has quit IRC | 20:06 | |
| edleafe | cdent: we do key on rp_uuid from the a-r, which was another design compromise | 20:06 |
| dansmith | how does the jsonschema validation work if you have a change in the a-r? does it just ignore a subtree of something and validate it separately or something? | 20:06 |
| penick | mriedem I'll reply to the ML today or tonight.. he notes that vendordata doesn't allow you to pass parameters, but I think that's something that can be addressed. Or he can write a vendordata driver | 20:06 |
| cdent | edleafe: so it isn’t opaque | 20:06 |
| edleafe | cdent: which is why for a given host there may be several a-rs, and we just take the first one for claiming | 20:07 |
| mriedem | penick: nova passes some stuff to the vendordata service | 20:07 |
| *** eharney has quit IRC | 20:07 | |
| cdent | “passes some stuff” is the new api guideline | 20:07 |
| *** edmondsw has quit IRC | 20:07 | |
| cdent | what should my api do? “pass some stuff” | 20:07 |
| edleafe | cdent: {$rp_uuid: <opaque>} | 20:07 |
| mriedem | penick: https://github.com/openstack/nova/blob/master/nova/api/metadata/vendordata_dynamic.py#L77 | 20:08 |
| *** edmondsw has joined #openstack-nova | 20:08 | |
| *** smatzek has joined #openstack-nova | 20:08 | |
| cdent | edleafe: that’s not what an a-r looks like now: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation_candidate.py#L55-L68 | 20:09 |
| jaypipes | cdent: the whole "I told you guys that this all sucks but don't have a better idea about how to solve this problem" attitude gets really old. | 20:09 |
| cdent | and we’re planning to change it with the new format | 20:09 |
| penick | Is he right about plaintext http only? That seems an easy fix | 20:09 |
| cdent | jaypipes: it gets especially old when you think that’s what we are doing and we aren’t actually | 20:09 |
| *** edmondsw_ has joined #openstack-nova | 20:09 | |
| jaypipes | cdent: sure seems like it. | 20:09 |
| *** artom has quit IRC | 20:09 | |
| penick | er, mriedem: ahah, thanks. Is he right about plaintext only? that'd be an easy fix. I'll reply to him on the ML | 20:09 |
| *** artom has joined #openstack-nova | 20:10 | |
| cdent | jaypipes: my apologies then, but I think you’ll find that this started because I pointed dan at some clarifying info about questions he reaised on a review | 20:10 |
| cdent | since then we’ve been talking, that’s all | 20:10 |
| penick | mriedem nm I see the ssl bit in the code | 20:10 |
| mriedem | penick: yeah we ust send some shit over json in the request and we hope the vendordata service finds it useful | 20:10 |
| edleafe | jaypipes: having to have this conversation about opaqueness gets pretty old, too | 20:11 |
| mriedem | and yeah we do the normal ksa stuff | 20:11 |
| mriedem | for the service user | 20:11 |
| jaypipes | cdent: ""microversion thing way too fluid” was my concern too, but, like I said, I decided to capitulate" <-- attitude. | 20:11 |
| cdent | jaypipes: because at the ptg you declared, with dan, that we should argue less, so I did: I capitulated, as requested | 20:11 |
| cdent | you perceive so much, without confirm it, and place me in this position of being a bad guy. It. Is. Not. Me. | 20:12 |
| jaypipes | cdent: like I said... attitude. | 20:12 |
| cdent | you read into that statement some kind of smug bullshit that is not there | 20:12 |
| cdent | it’s just me saying "okay" | 20:12 |
| dansmith | okay, let's pause the personal stuff for a minute | 20:12 |
| *** edmondsw has quit IRC | 20:12 | |
| cdent | I’m so sick of this. | 20:12 |
| jaypipes | fine with me. | 20:12 |
| dansmith | edleafe: your argument against a version is just that it's not needed, right? | 20:13 |
| * cdent happy to pause | 20:13 | |
| dansmith | edleafe: doesn't hurt anything, just isn't strictly required, right? | 20:13 |
| edleafe | for allocation_requests, yes, it's not needed | 20:13 |
| edleafe | no, it over-engineers things, so it does hurt | 20:13 |
| dansmith | edleafe: okay, so I know I don't have lots of karma to burn with you, but we could just put it in there, call it the dan_is_dumb field, and move forward without costing much else, right? | 20:13 |
| *** edmondsw_ has quit IRC | 20:14 | |
| dansmith | a single field over-engineers? | 20:14 |
| edleafe | dansmith: you do realize that it's a lot more than just adding a field, right? | 20:14 |
| edleafe | it's now having to do version checking in placement for a thing that will always be the current version | 20:15 |
| dansmith | we damn sure better be doing that version checking in placement anyway | 20:15 |
| edleafe | it's changing the placement API to add the version | 20:15 |
| dansmith | eh? | 20:15 |
| dansmith | this is protocol stuff. the envelope.. it's built into every call we make to placement if we pass version=something in report client, no? | 20:16 |
| edleafe | where is that a-r version coming from? | 20:16 |
| jaypipes | edleafe: it's currently hard-coded in the report client's claim_resources() method. That would need to be updated to pass an optional kwarg for the microversion override if received in the select_destinations() returned objects. | 20:17 |
| jaypipes | i.e. claim_resources(..., version=$what_i_got_from_scheduler) | 20:17 |
| dansmith | r = self.put(url, payload, version='1.10') | 20:18 |
| dansmith | exactly | 20:18 |
| jaypipes | we would want to have the scheduler package up the user and project ID into the allocation request blob it sends to. | 20:18 |
| jaypipes | to the caller of the select_destinations() method | 20:18 |
| edleafe | jaypipes: So we have an old scheduler, old conductor, and new placement. Are you saying that placement should modify the structure of the a-r part of the response based on the old scheduler/conductor? | 20:19 |
| jaypipes | also, instead of adding a separate version attribute to the Selection object, we could return a SelectionList object that had two fields, version and selections so we don't have to repeat the version field over and over again. | 20:19 |
| dansmith | edleafe: that's what versioning an API is all about | 20:19 |
| dansmith | jaypipes: I said no list object because there was no need. this would be a need, so that's fine with me, if that's desired | 20:20 |
| dansmith | when I argued against it, there was no such extra property | 20:20 |
| *** sahid has quit IRC | 20:20 | |
| edleafe | dansmith: I know what versioning an API is about. | 20:21 |
| edleafe | dansmith: the issue is whether the a-r should be dependent on placement, or on the things that call placement | 20:22 |
| dansmith | anything returned to the client is done so at the maximum version they both support | 20:22 |
| cdent | are we talking rpc api or http api here, because http api, it is the version the client asked for | 20:23 |
| cdent | which we have been generally controlling per request for nova->placement | 20:24 |
| *** smatzek has quit IRC | 20:24 | |
| jaypipes | cdent: two different clients here. | 20:24 |
| edleafe | the clients support receiving a-rs. That's all the versioning we need. If they can get an a-r blob, that's sufficient | 20:24 |
| edleafe | http | 20:24 |
| jaypipes | cdent: the placement client embedded in the scheduler that called GET /allocation_candidates is one client. The scheduler client embdedded in the cell conductor that needs to call PUT /allocations/ is a different client. | 20:24 |
| cdent | yes, I know | 20:24 |
| dansmith | cdent: really not much about rpc apis going on here | 20:25 |
| cdent | I was attempting to clarify dan’s statement about “maximum version they supprot” < that’s not true for the report client, it asks for a specific version, not a max | 20:25 |
| *** tbachman has quit IRC | 20:25 | |
| dansmith | cdent: we're sending a blob to the rpc consumer, which has an inbuilt version that it should use to communicate with an external thing.. it's opaque to _that_ service | 20:25 |
| dansmith | cdent: it's an rpc client, it's tightly controlled by us, versioned interface, and we have more strict versions that are "compatible" | 20:26 |
| dansmith | cdent: it's the maximum semantically, the server doesn't know anything other than what the client said is it's version (i.e. the max for this request) | 20:26 |
| cdent | yes, per request | 20:27 |
| *** tbachman has joined #openstack-nova | 20:28 | |
| mriedem | i'll say i remember saying we should include the microversion that we used to build the allocation request in the scheduler and pass that down so the conductor makes the same request at the same microversion later | 20:29 |
| mriedem | rather than 'latest' | 20:30 |
| dansmith | mriedem: it's actually always making 1.10 right now, not even latest | 20:30 |
| *** lbragstad has quit IRC | 20:30 | |
| dansmith | which makes it even worse, IMHO | 20:30 |
| mriedem | but i'd have to lookup where i said this, which might just have been irc... | 20:30 |
| jaypipes | this is the conversation that edleafe and I were having at a whiteboard in the corner. | 20:33 |
| dansmith | yeah, I wasn't over there, sorry about that | 20:33 |
| dansmith | so, let me just summarize where I think we stand | 20:33 |
| dansmith | call me out if I'm being biased | 20:34 |
| jaypipes | and I said it would be ok to have the placement API service handle seamlessly understanding old formats of allocation request body. | 20:34 |
| mriedem | i don't think the placement api should be trying to retrofit the request | 20:34 |
| mriedem | that's weird | 20:34 |
| dansmith | me either | 20:34 |
| jaypipes | I had previously asked edleafe to include the microversion the scheduler created the a-r for in the returned object from select_destinations() | 20:34 |
| cdent | mikal: I assume you’re not with us at the moment, but when you join does this make sense to you: https://review.openstack.org/#/c/509417/ | 20:35 |
| dansmith | so I was going to summarize... | 20:36 |
| dansmith | edleafe is of the opinion that this should definitely not be versioned | 20:36 |
| dansmith | I feel it should be | 20:36 |
| *** tbachman has quit IRC | 20:36 | |
| dansmith | cdent leans towards versioning | 20:37 |
| dansmith | I think jaypipes is saying he does too (is that right?) | 20:37 |
| cdent | I lean toward not doing the microversino fakery failover server side | 20:37 |
| cdent | I’m agnostic about what happens on the client side | 20:37 |
| dansmith | and mriedem seems to think strict microversion style versioning as well | 20:37 |
| jaypipes | dansmith: I lean towards placing a version field in the object returned from select_destinations(), yes. | 20:38 |
| dansmith | cdent: okay not sure how to avoid the fakery without the client being complicit.. your point is no fakery, whatever that means for the client behavior right? | 20:38 |
| cdent | yes | 20:38 |
| *** Tom has joined #openstack-nova | 20:38 | |
| *** edmondsw has joined #openstack-nova | 20:40 | |
| *** lbragstad has joined #openstack-nova | 20:40 | |
| *** archit has quit IRC | 20:41 | |
| cfriesen | if we don't version it, doesn't that heavily restrict what changes can be made in the future? | 20:42 |
| *** Tom has quit IRC | 20:43 | |
| dansmith | cfriesen: if we don't version it, then anything in the future becomes a archaeology expedition to decide what we can and can't do, yeah | 20:43 |
| cfriesen | like you could add/remove fields but not change the meaning of an existing field | 20:43 |
| dansmith | a version doesn't fix that completely, but it certainly helps draw a box around what we can do and how | 20:43 |
| dansmith | cfriesen: not necessarily even that | 20:44 |
| *** gouthamr_ has quit IRC | 20:44 | |
| dansmith | I feel like we're at this point, | 20:44 |
| dansmith | that we said we'd call out, recognize, and avoid rat-holing | 20:45 |
| jaypipes | edleafe: are you still with us? | 20:45 |
| edleafe | I had to walk away for a while. I was getting way too frustrated that this thing that we have said all along was an opaque blob now has versioned information | 20:46 |
| edleafe | No one seems to consider that an a-r is a placement artifact, not a scheduler/conductor thing | 20:47 |
| *** SamYaple has joined #openstack-nova | 20:47 | |
| edleafe | it is created by placement, and is used only by placment, and is never persisted | 20:47 |
| edleafe | it is always "the latest" | 20:47 |
| edleafe | But we want to treat it like it is a versioned data structure that scheduler "can know about" | 20:48 |
| cdent | (what ed just said about “always latest” is part of why I’m agnostic on the client side. It would be fundamentally correct for the report client to send the header as ‘latest’ because the placement service is always its own latest) | 20:49 |
| *** priteau has quit IRC | 20:49 | |
| dansmith | cdent: but the client has to construct the actual request | 20:50 |
| dansmith | right now it puts the user/project in there, and may have to do other things later | 20:50 |
| dansmith | so it can't say latest for the request, it has to say a version | 20:50 |
| cdent | yes, that’s where the notion of opaque blob falls apart | 20:50 |
| *** priteau has joined #openstack-nova | 20:50 | |
| edleafe | dansmith: right now it just passes back the a-r to claim. It doesn't build anything | 20:50 |
| dansmith | the other thing is, | 20:50 |
| edleafe | the user/project is in the a-r | 20:51 |
| dansmith | the scheduler cannot communicate with placement at version latest | 20:51 |
| jaypipes | edleafe: not currently it isn't, no. | 20:51 |
| dansmith | and it does have to look at the results of those calls | 20:51 |
| jaypipes | edleafe: the scheduler client's claim_resources() method adds user id and project id to the HTTP request payload. | 20:51 |
| dansmith | so the compute or conductor saying latest _cannot_ be right forever | 20:51 |
| edleafe | jaypipes: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L77 | 20:51 |
| *** hamzy has quit IRC | 20:52 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Pre-create migration object https://review.openstack.org/498950 | 20:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | 20:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | 20:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | 20:52 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 20:52 |
| jaypipes | edleafe: that's the request payload for PUT /allocations, not the format of the allocation_request object that is returned in the GET /allocation_candidates HTTP response. | 20:52 |
| jaypipes | edleafe: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation_candidate.py#L57-L66 | 20:53 |
| *** smatzek has joined #openstack-nova | 20:53 | |
| jaypipes | edleafe: we don't currently add the user and project ID into the allocation_request object in the return from GET /allocation_candidates | 20:53 |
| jaypipes | edleafe: unfortunately. was an oversight on my part. | 20:54 |
| jaypipes | edleafe: I'm sure dansmith at some point told me to put it in there and I just forgot. | 20:54 |
| dansmith | it doesn't matter, because 'latest' is not the version used by the scheduler (in the future when we're doing things correctly and placement is external) | 20:54 |
| *** lpetrut_ has quit IRC | 20:55 | |
| cdent | so if scheduler and placement are out of sync, grind | 20:56 |
| *** catintheroof has quit IRC | 20:56 | |
| dansmith | when placement is external, we must tolerate them being out of sync | 20:56 |
| *** MVenesio has quit IRC | 20:57 | |
| *** smatzek has quit IRC | 20:58 | |
| edleafe | oh, geez, I give up. I missed this: https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L154 | 20:59 |
| *** priteau has quit IRC | 20:59 | |
| edleafe | Forget everything I said about a-rs being opaque | 20:59 |
| mriedem | yes i remember pointing out late in pike that we needed to update nova-status' check for the required minimum placement microversion to be 1.10 because that's what the scheduler was requesting during claim_resources | 21:00 |
| edleafe | that ship has sailed. | 21:00 |
| mriedem | we really only needed 1.8 for the user_id/project_id thing (i think?) | 21:00 |
| *** slaweq_ has quit IRC | 21:00 | |
| *** slaweq_ has joined #openstack-nova | 21:00 | |
| mriedem | and we had to put something in the release notes saying you have to make sure to upgrade placement before scheduler since scheduler requires this new higher microversoin in placement that wasn't available in ocata | 21:00 |
| edleafe | I'm going to finish the stuff I've been trying to work on and then I'll rethink how to change the series to add a versioned allocation_request to the Selection object | 21:01 |
| mriedem | if the pike scheduler was requesting 'latest' to an ocata placement, the request might pass at whatever 'latest' is for placement in ocata, but not what the pike scheduler client actually needs | 21:01 |
| dansmith | edleafe: okay and you caught the bit I said about the selectionlist object potentially being okay if we're going to use it for holding a version right? | 21:02 |
| edleafe | dansmith: yeah, but that's minor | 21:03 |
| dansmith | edleafe: yep, just saying, if you wan to go back to doing it that way, I'm cool with it | 21:03 |
| cdent | edleafe’s link raises another wart doesn’t it? If _move_operation_alloc_request is working in the guts of alloc request, it has to know the version | 21:03 |
| cdent | is that called from only the scheduler, or also in the cells? | 21:04 |
| cdent | (and presumably there are others like it?) | 21:04 |
| jaypipes | cdent: we're trying to get rid of that entirely. | 21:04 |
| jaypipes | cdent: and do the migration owns allocation thing. | 21:04 |
| edleafe | cdent: it will be called from within the cells too | 21:04 |
| mriedem | cdent: it's called from the scheduler and, for the time being, superconductor | 21:04 |
| cdent | yes, but will still inspect don’t we? | 21:04 |
| mriedem | during force live migrate and force evacuate | 21:04 |
| mriedem | where the scheduler is skipped | 21:05 |
| mriedem | edleafe: not within the cells | 21:05 |
| cdent | and in any case that code is pike | 21:05 |
| mriedem | edleafe: oh you mean with alternate hosts yeah | 21:05 |
| edleafe | mriedem: the cell conductor will have to claim | 21:05 |
| mriedem | right right | 21:05 |
| dansmith | but we don't need too look inside the a-r in the claim during reschedule | 21:05 |
| mriedem | we just proxy it through | 21:05 |
| dansmith | this move claim thing is a good example of the scheduler needing to examine it | 21:05 |
| * cdent needs a stiff drink | 21:05 | |
| dansmith | mriedem: right | 21:05 |
| mriedem | "here is a request the scheduler told me to send at this version k?!" | 21:06 |
| mriedem | "<3 cell conductor" | 21:06 |
| edleafe | dansmith: and I was thinking that this move claim thing is a bad example | 21:06 |
| *** crushil has quit IRC | 21:06 | |
| dansmith | edleafe: it's a bad example in the cosmic sense of things that suck... yes :) | 21:06 |
| efried | mriedem Please cast your critical eye upon https://review.openstack.org/#/c/488137/ when you get time. | 21:11 |
| *** rajinir is now known as rajinir-afk | 21:11 | |
| *** jheroux has quit IRC | 21:11 | |
| efried | IIRC the goal was to get the whole series in by the first milestone thingy. | 21:11 |
| mriedem | efried: oh efried | 21:11 |
| mriedem | did i actually say that was a goal? | 21:11 |
| mriedem | i mentioned it as being doable during a meeting a few weeks back | 21:12 |
| mriedem | and you've been cruising my house at 1am ever since | 21:12 |
| efried | Oh, please let me find that. Stand by. | 21:12 |
| efried | mriedem http://eavesdrop.openstack.org/meetings/nova/2017/nova.2017-09-21-14.00.log.txt @14:03:09 | 21:13 |
| efried | It's entirely possible I've been misinterpreting "...should ... have ... merged by then" | 21:14 |
| mriedem | that can be interpreted so many different ways | 21:14 |
| mriedem | would never hold up in a court | 21:14 |
| jaypipes | can we finalize on a decision on this then? | 21:14 |
| efried | Fair enough. But if it ever gets interpreted as "this should have merged by then," I don't want it to be because I didn't pester people for reviews :) | 21:14 |
| mriedem | efried: don't worry i know you've asked several times | 21:16 |
| mriedem | jaypipes: pass the version down | 21:16 |
| dansmith | yep | 21:17 |
| edleafe | jaypipes: [t 4Bt] | 21:17 |
| purplerbot | <edleafe> I'm going to finish the stuff I've been trying to work on and then I'll rethink how to change the series to add a versioned allocation_request to the Selection object [2017-10-05 21:01:08.583741] [n 4Bt] | 21:17 |
| jaypipes | ok, thanks edleafe | 21:17 |
| *** crushil has joined #openstack-nova | 21:19 | |
| *** baoli has quit IRC | 21:19 | |
| *** rcernin has joined #openstack-nova | 21:22 | |
| *** crushil has quit IRC | 21:24 | |
| mriedem | ha, | 21:26 |
| mriedem | good news folks | 21:26 |
| mriedem | get ready for 100% nova gate failure | 21:26 |
| *** dillaman has quit IRC | 21:28 | |
| dansmith | wat | 21:28 |
| dansmith | I see the top of our gate is failing | 21:28 |
| mriedem | yeah i know what it is | 21:28 |
| mriedem | but i'll never tell | 21:28 |
| dansmith | just fix, I don't care if you tell | 21:28 |
| mtreinish | mriedem: oh it's your new test | 21:29 |
| mriedem | SHHHHHHHHHHHHHHHH | 21:29 |
| mriedem | TREINISH! | 21:29 |
| *** vladikr has quit IRC | 21:29 | |
| mriedem | who were the ad wizards that merged that one | 21:29 |
| *** Tom_ has joined #openstack-nova | 21:29 | |
| mtreinish | mriedem: oomichi_afk gave it the +W | 21:29 |
| dansmith | the shelve offload one? | 21:29 |
| mriedem | no | 21:29 |
| mriedem | i'm fixing | 21:29 |
| melwitt | lol | 21:30 |
| mtreinish | dansmith: https://review.openstack.org/#/c/480746/ | 21:30 |
| * dansmith shakes his head at mriedem | 21:31 | |
| mriedem | hey, you're welcome ^ | 21:31 |
| mriedem | oops | 21:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Blacklist test_extend_attached_volume from cells v1 job https://review.openstack.org/509907 | 21:32 |
| dansmith | well, melissaml +1d it so I'm surprised it was buggy | 21:32 |
| mriedem | usually pretty reliable | 21:32 |
| dansmith | melwitt: jaypipes ^ | 21:32 |
| *** Tom_ has quit IRC | 21:33 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: rp: break functions out of _set_traits() https://review.openstack.org/509908 | 21:34 |
| mtreinish | melwitt: hmm, on https://review.openstack.org/#/c/507976/ stestr said the blacklist didn't match anything | 21:35 |
| mtreinish | you might have found a bug in it | 21:35 |
| *** esberglu has quit IRC | 21:36 | |
| melwitt | okay :) | 21:36 |
| melwitt | mtreinish: do you know wassup with this? http://logs.openstack.org/76/507976/5/check/gate-nova-python35/859471a/console.html#_2017-10-05_21_00_25_162394 | 21:37 |
| mtreinish | melwitt: yeah the post-processing on results is going a bit crazy because the test runner bailed before generating any artifacts | 21:37 |
| mtreinish | so all the things are trying to operate on testrepository.subunit are blowing up because that was never created | 21:38 |
| mtreinish | that specific du check was there for testr because it would just pass silently if no tests were ever run | 21:39 |
| melwitt | oh | 21:39 |
| mtreinish | so the run tox script that zuul runs does a du to check there is actual subunit data generated | 21:39 |
| melwitt | do not matching anything made it bail? | 21:39 |
| mtreinish | it's not really necessary on stestr though because it fails if nothign is run | 21:39 |
| melwitt | *did | 21:40 |
| melwitt | I guess that didn't really make my sentence better | 21:40 |
| mtreinish | melwitt: yep, it exited with an error because it didn't match anything | 21:40 |
| mriedem | jaypipes: i have a sql question | 21:40 |
| mtreinish | that error message needs to be fixed though, it predates other non-regex selection mechanisms | 21:40 |
| melwitt | k | 21:40 |
| mriedem | i want to add an index on instance_actions_events for the action_id and event_id fields, | 21:40 |
| *** edmondsw has quit IRC | 21:41 | |
| mriedem | jaypipes: when we query those, we also include the deleted column so i'm assuming we want that in the index too right? | 21:41 |
| *** edmondsw has joined #openstack-nova | 21:41 | |
| jaypipes | mriedem: you don't *have* to do that, no... especially if the deleted column has very low cardinality | 21:42 |
| jaypipes | mriedem: i.e. deleted column has very few distinct values | 21:42 |
| mriedem | i've just noticed that we have deleted in most of our other indexes | 21:42 |
| mtreinish | melwitt: the thing I'm confused by is the selection code in stestr I basically just copy and pasted from ostestr, so I'm surprised it's behaving differently | 21:42 |
| melwitt | ah, yeah. I was wondering that | 21:42 |
| jaypipes | mriedem: yeah, I know we have deleted in a lot of the indexes... | 21:43 |
| *** gouthamr has joined #openstack-nova | 21:43 | |
| mriedem | jaypipes: well, the values are 0 or positive int | 21:43 |
| jaypipes | mriedem: right, but most are 0. | 21:43 |
| mriedem | sure | 21:43 |
| mriedem | maybe i should just run it both ways and see | 21:43 |
| jaypipes | mriedem: that's like have a phone book with all dan smiths in it. | 21:43 |
| mriedem | currently building a devstack with 500 of these | 21:44 |
| gregwork | has anyone seen where all the nova services (conductor/scheduler/api) all start barfing with: ValueError: too many values to unpack. just rolling logs of this kind of error: https://paste.fedoraproject.org/paste/Vn7-6s6INFVJa5FqWb7huQ | 21:44 |
| mriedem | there are 2 matt riedemanns in the dan smith phone book | 21:44 |
| gregwork | nova-manage db sync also returns that ValueError | 21:44 |
| jaypipes | gregwork: you have bad sql_connection config option values. | 21:45 |
| mriedem | gregwork: looks like a config issue | 21:45 |
| mriedem | yup | 21:45 |
| mriedem | check your [database]/connection value | 21:45 |
| *** edmondsw has quit IRC | 21:46 | |
| *** MasterOfBugs has quit IRC | 21:46 | |
| *** MasterOfBugs has joined #openstack-nova | 21:46 | |
| gregwork | in /etc/nova/nova.conf ? | 21:46 |
| *** felipemonteiro__ has quit IRC | 21:46 | |
| mriedem | yes | 21:47 |
| mtreinish | melwitt: found the issue, left a review comment | 21:48 |
| melwitt | looking | 21:49 |
| mtreinish | I was confused there for a second, because the flag worked fine locally for me | 21:49 |
| melwitt | ah, thanks | 21:49 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make setenv consistent for unit, func, and api-samples https://review.openstack.org/507976 | 21:50 |
| *** tbachman has joined #openstack-nova | 21:54 | |
| *** slaweq_ has quit IRC | 21:55 | |
| gregwork | yep that was exactly it | 22:02 |
| gregwork | was trying to deploy browbeat and it modified nova.conf to add +connmon to the connection= | 22:02 |
| gregwork | but the install failed however the conf's had been updated | 22:02 |
| gregwork | thanks! | 22:03 |
| *** Nel1x has joined #openstack-nova | 22:03 | |
| *** pramodrj07 has joined #openstack-nova | 22:04 | |
| *** rcernin has quit IRC | 22:06 | |
| *** MasterOfBugs has quit IRC | 22:07 | |
| *** burt has quit IRC | 22:07 | |
| *** baoli has joined #openstack-nova | 22:08 | |
| *** Tom_ has joined #openstack-nova | 22:10 | |
| *** baoli has quit IRC | 22:12 | |
| *** baoli has joined #openstack-nova | 22:12 | |
| *** Tom_ has quit IRC | 22:14 | |
| *** yamamoto has joined #openstack-nova | 22:14 | |
| mriedem | omg here it comes | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add live.migration.force.complete to the legacy notification whitelist https://review.openstack.org/508877 | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add _wait_for_action_fail_completion to InstanceHelperMixin https://review.openstack.org/509922 | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add recreate test for live migrate rollback not cleaning up dest allocs https://review.openstack.org/509923 | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add functional migrate force_complete test https://review.openstack.org/509924 | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add functional for live migrate delete https://review.openstack.org/509925 | 22:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Remove dest node allocations during live migration rollback https://review.openstack.org/509926 | 22:21 |
| *** tbachman has quit IRC | 22:23 | |
| *** awaugama has quit IRC | 22:25 | |
| *** lbragstad has quit IRC | 22:38 | |
| *** liangy has quit IRC | 22:41 | |
| *** jdwidari has quit IRC | 22:44 | |
| *** xyang1 has quit IRC | 22:45 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Note TrustedFilter deprecation in docs https://review.openstack.org/509931 | 22:48 |
| *** penick has quit IRC | 22:54 | |
| *** gmann_sleep is now known as gmann | 23:01 | |
| *** erlon has quit IRC | 23:01 | |
| *** hongbin has quit IRC | 23:07 | |
| *** Tom_ has joined #openstack-nova | 23:10 | |
| mriedem | well this is weird, i stop 1 instance and it generates 3 events http://paste.openstack.org/show/622795/ | 23:10 |
| mtreinish | mriedem: all the request ids are different? | 23:13 |
| mriedem | and 2 of the events don't have project ids | 23:14 |
| mriedem | i bet i know what this is | 23:14 |
| mriedem | sync_instance_power_state periodic | 23:14 |
| mriedem | the last 2 are 10 minutes apart | 23:14 |
| mriedem | which is the default on that periodic | 23:14 |
| *** Tom_ has quit IRC | 23:15 | |
| mriedem | ha, yeah, because the fake driver that i'm using doesn't change the state on the fake guest when you power it off | 23:16 |
| mriedem | so the db says it's stopped but the "hypervisor" doesn't | 23:16 |
| mriedem | so nova stops it | 23:16 |
| mriedem | sev1 fake driver bug | 23:16 |
| *** s1061123 has quit IRC | 23:17 | |
| *** s1061123 has joined #openstack-nova | 23:17 | |
| mtreinish | mriedem: hah, nice | 23:18 |
| *** sdague has quit IRC | 23:18 | |
| *** claudiub has joined #openstack-nova | 23:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement power_off/power_on for the FakeDriver https://review.openstack.org/509935 | 23:21 |
| *** baoli has quit IRC | 23:24 | |
| *** tbachman has joined #openstack-nova | 23:26 | |
| *** edmondsw has joined #openstack-nova | 23:28 | |
| *** namnh has joined #openstack-nova | 23:28 | |
| *** edmondsw has quit IRC | 23:32 | |
| *** takashin has joined #openstack-nova | 23:36 | |
| *** chyka has quit IRC | 23:36 | |
| *** chyka has joined #openstack-nova | 23:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add live.migration.force.complete to the legacy notification whitelist https://review.openstack.org/508877 | 23:38 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add functional migrate force_complete test https://review.openstack.org/509924 | 23:38 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add functional for live migrate delete https://review.openstack.org/509925 | 23:38 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Remove dest node allocations during live migration rollback https://review.openstack.org/509926 | 23:38 |
| *** chyka has quit IRC | 23:41 | |
| *** hamzy has joined #openstack-nova | 23:43 | |
| *** markvoelker_ has quit IRC | 23:51 | |
| *** pramodrj07 has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!