| *** http_GK1wmSU has left #openstack-nova | 00:02 | |
| *** chyka has joined #openstack-nova | 00:04 | |
| *** kbaegis has quit IRC | 00:07 | |
| *** kaisers2 has joined #openstack-nova | 00:09 | |
| *** chyka has quit IRC | 00:09 | |
| *** kaisers1 has quit IRC | 00:10 | |
| *** acormier has joined #openstack-nova | 00:12 | |
| *** dtp has quit IRC | 00:12 | |
| *** baoli has joined #openstack-nova | 00:16 | |
| *** gmann has quit IRC | 00:17 | |
| *** baoli has quit IRC | 00:17 | |
| *** gmann has joined #openstack-nova | 00:17 | |
| *** yangyapeng has joined #openstack-nova | 00:19 | |
| openstackgerrit | Merged openstack/nova master: Add redirect for api-microversion-history doc https://review.openstack.org/489641 | 00:21 |
|---|---|---|
| *** baoli has joined #openstack-nova | 00:22 | |
| *** ujjain has quit IRC | 00:23 | |
| *** yangyapeng has quit IRC | 00:24 | |
| openstackgerrit | Merged openstack/nova master: add top 404 redirect https://review.openstack.org/490181 | 00:24 |
| openstackgerrit | Merged openstack/nova master: sort redirectmatch lines https://review.openstack.org/490182 | 00:24 |
| openstackgerrit | Merged openstack/nova master: fix rpc broken rst comment https://review.openstack.org/490197 | 00:25 |
| openstackgerrit | Merged openstack/nova master: Update SSL cert used in testing https://review.openstack.org/489078 | 00:26 |
| *** mingyu has joined #openstack-nova | 00:26 | |
| *** mingyu has quit IRC | 00:31 | |
| *** ujjain has joined #openstack-nova | 00:31 | |
| *** ujjain has quit IRC | 00:31 | |
| *** ujjain has joined #openstack-nova | 00:31 | |
| *** kiennt has joined #openstack-nova | 00:36 | |
| *** gbarros has joined #openstack-nova | 00:38 | |
| *** Shunli has joined #openstack-nova | 00:40 | |
| *** zhurong has joined #openstack-nova | 00:45 | |
| *** rajathagasthya has quit IRC | 00:49 | |
| *** acormier has quit IRC | 00:50 | |
| *** tinwood has quit IRC | 00:56 | |
| *** litao__ has joined #openstack-nova | 00:56 | |
| *** tinwood has joined #openstack-nova | 00:57 | |
| *** crushil has quit IRC | 00:57 | |
| *** dixiaoli has joined #openstack-nova | 00:58 | |
| *** hongbin has joined #openstack-nova | 00:59 | |
| *** acormier has joined #openstack-nova | 01:00 | |
| *** Nel1x has joined #openstack-nova | 01:00 | |
| *** iceyao has joined #openstack-nova | 01:00 | |
| *** mdnadeem has joined #openstack-nova | 01:02 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** acormier has quit IRC | 01:04 | |
| *** acormier has joined #openstack-nova | 01:06 | |
| *** iceyao has quit IRC | 01:07 | |
| *** acormier has quit IRC | 01:07 | |
| *** acormier has joined #openstack-nova | 01:07 | |
| *** phuongnh has quit IRC | 01:08 | |
| *** lucasxu has joined #openstack-nova | 01:09 | |
| *** acormier has quit IRC | 01:10 | |
| *** lucasxu has quit IRC | 01:10 | |
| *** acormier has joined #openstack-nova | 01:10 | |
| *** Apoorva has quit IRC | 01:11 | |
| *** Apoorva has joined #openstack-nova | 01:12 | |
| *** jichen has joined #openstack-nova | 01:13 | |
| *** crushil has joined #openstack-nova | 01:15 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** iceyao has joined #openstack-nova | 01:17 | |
| *** phuongnh has joined #openstack-nova | 01:17 | |
| *** randomha1k has joined #openstack-nova | 01:18 | |
| *** yangyapeng has joined #openstack-nova | 01:18 | |
| *** yangyape_ has joined #openstack-nova | 01:20 | |
| *** yamahata has quit IRC | 01:20 | |
| *** randomha1k has quit IRC | 01:22 | |
| *** yangyapeng has quit IRC | 01:23 | |
| *** jamesdenton has joined #openstack-nova | 01:25 | |
| *** mingyu has joined #openstack-nova | 01:27 | |
| *** rajathagasthya has joined #openstack-nova | 01:28 | |
| *** gongysh has joined #openstack-nova | 01:30 | |
| *** mingyu has quit IRC | 01:32 | |
| *** armax has joined #openstack-nova | 01:34 | |
| *** johnhaan has joined #openstack-nova | 01:35 | |
| *** thorst_afk has quit IRC | 01:37 | |
| *** jdurgin has quit IRC | 01:37 | |
| *** tbachman has quit IRC | 01:37 | |
| *** gbarros has quit IRC | 01:38 | |
| *** jdurgin has joined #openstack-nova | 01:40 | |
| *** yamamoto has quit IRC | 01:45 | |
| *** yamamoto has joined #openstack-nova | 01:45 | |
| *** mriedem has quit IRC | 01:47 | |
| *** randomha1k has joined #openstack-nova | 01:50 | |
| *** randomha1k has quit IRC | 01:55 | |
| *** gbarros has joined #openstack-nova | 01:57 | |
| *** mtanino has quit IRC | 01:59 | |
| *** itlinux has joined #openstack-nova | 02:02 | |
| Kevin_Zheng | mriedem: flavor id is not included in the reply > 2.47 | 02:08 |
| *** acormier_ has joined #openstack-nova | 02:09 | |
| gmann | Kevin_Zheng, in server show? | 02:10 |
| Kevin_Zheng | gmann: I used list/details | 02:10 |
| Kevin_Zheng | "flavor": {"ephemeral": 0, "ram": 512, "original_name": "m1.tiny", "vcpus": 1, "extra_specs": {}, "swap": 0, "disk": 1} | 02:10 |
| Kevin_Zheng | I guess this is what horizon uses | 02:11 |
| gmann | Kevin_Zheng, yea, it is right as per microversion 2.47 | 02:12 |
| *** acormier has quit IRC | 02:12 | |
| Kevin_Zheng | do you know the flavor story matt mentioned last night? | 02:12 |
| Kevin_Zheng | Horizon-flavor | 02:13 |
| gmann | Kevin_Zheng, no, whats that. in 2,47 id not returned instead flavor data | 02:14 |
| Kevin_Zheng | horizon shows flavor name when display instances, right? | 02:14 |
| *** gcb has quit IRC | 02:15 | |
| Kevin_Zheng | it also provided a feature that can edit existing flavors, no matter it has been used or not | 02:15 |
| Kevin_Zheng | since nova don't provide this kind of action, horizon delete the old flavor and created a new one, the id of the flavor is changed | 02:16 |
| openstackgerrit | Merged openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | 02:16 |
| Kevin_Zheng | before we move flavor to api db | 02:16 |
| *** gcb has joined #openstack-nova | 02:16 | |
| gmann | yea | 02:16 |
| Kevin_Zheng | flavor is soft-deleted | 02:16 |
| Kevin_Zheng | when nova returned instance.flavor_id | 02:16 |
| Kevin_Zheng | horizon can still got the deleted flavor | 02:17 |
| Kevin_Zheng | but when it is in api db | 02:17 |
| Kevin_Zheng | delete is gone forever | 02:17 |
| Kevin_Zheng | so cannot display | 02:17 |
| gmann | i see | 02:18 |
| Kevin_Zheng | another thing is that even we can get the correct flavor name | 02:18 |
| Kevin_Zheng | if we modified the flavor details | 02:19 |
| Kevin_Zheng | the display of the flavor under this name will be incorrect | 02:19 |
| Kevin_Zheng | so I guess maybe showing id is better? | 02:19 |
| Kevin_Zheng | but we don't reply id > 2.47 | 02:20 |
| Kevin_Zheng | maybe we should add it to the current reply?? | 02:20 |
| gmann | Kevin_Zheng, so name should be same in horizon after modification right | 02:20 |
| Kevin_Zheng | it depends, you can also edit name | 02:20 |
| Kevin_Zheng | very complicated usecases | 02:21 |
| gmann | but with id it will be same issue that horizon will get deleted id | 02:21 |
| Kevin_Zheng | yeah, maybe people should come up with a solution, I've posted a maillist | 02:22 |
| *** ducnc has joined #openstack-nova | 02:22 | |
| *** mingyu has joined #openstack-nova | 02:23 | |
| gmann | Kevin_Zheng, so when horizon modified flavor(deleted old one). and if someone check instance it shows as flavor-id in instance detail even that is deleted | 02:26 |
| gmann | that is till 2,47 | 02:26 |
| gmann | right? | 02:26 |
| Kevin_Zheng | yeah, thats one option, or we could just display the details | 02:27 |
| Kevin_Zheng | because we already have it | 02:27 |
| gmann | i mean what was behavior till 2,47 | 02:27 |
| Kevin_Zheng | Not available will be displayed | 02:28 |
| gmann | instance use to show old flavor id which is deleted | 02:28 |
| *** trinaths has left #openstack-nova | 02:28 | |
| Kevin_Zheng | as we moved flavor to api db | 02:28 |
| gmann | yes | 02:28 |
| Kevin_Zheng | and delete is delete there | 02:28 |
| Kevin_Zheng | horizon could not get the flavor from db using id | 02:28 |
| gmann | yea but GET instance shows the flavor id (<2.47) | 02:30 |
| Kevin_Zheng | yeah, | 02:31 |
| gmann | so instance detail in horizon will have old id | 02:31 |
| Kevin_Zheng | correct | 02:31 |
| gmann | when horizon try to fetch the old flavor ? | 02:31 |
| Kevin_Zheng | yes | 02:32 |
| gmann | i mean why horizon try to get old flavor? | 02:33 |
| *** ducnc has quit IRC | 02:33 | |
| *** yushb has joined #openstack-nova | 02:33 | |
| Kevin_Zheng | because instance saved the old flavor id | 02:34 |
| Kevin_Zheng | and return it to horizon | 02:34 |
| Kevin_Zheng | horizon use it to show the flavor details | 02:34 |
| gmann | show flavor details in Instance detail tab? | 02:34 |
| Kevin_Zheng | yeah | 02:35 |
| Kevin_Zheng | also the instances list tab | 02:35 |
| gmann | humm | 02:35 |
| gmann | so with >2.47 horizon do not need to do that as instance can give flavor detail | 02:36 |
| Kevin_Zheng | maybe | 02:36 |
| gmann | so 2.47 actually solved the problem ? | 02:36 |
| *** baoli has quit IRC | 02:36 | |
| Kevin_Zheng | yeah it could | 02:36 |
| gmann | i mean just show the what instance return either it exit or deleted | 02:37 |
| Kevin_Zheng | yeah | 02:37 |
| *** randomha1k has joined #openstack-nova | 02:37 | |
| *** esberglu has quit IRC | 02:39 | |
| *** esberglu has joined #openstack-nova | 02:39 | |
| *** yamamoto has quit IRC | 02:41 | |
| *** randomha1k has quit IRC | 02:42 | |
| gmann | yea but that will be old stale thing that people would not find such flavor in system as horizon modified it | 02:44 |
| gmann | may be you are asking about disabling this feature is good idea | 02:44 |
| Kevin_Zheng | Yeah, I can think of one usecase of this | 02:44 |
| Kevin_Zheng | since flavor have alot of fields | 02:45 |
| gmann | yea | 02:45 |
| Kevin_Zheng | maybe people want to only change one of the field | 02:45 |
| Kevin_Zheng | so they don't want to create a new one and do all the input | 02:45 |
| *** acormier_ has quit IRC | 02:46 | |
| Kevin_Zheng | I was thinking, maybe instead of delete-and-create, horizon can provide a copy-and-modify | 02:46 |
| gmann | yea | 02:46 |
| Kevin_Zheng | leave the deleting to user | 02:46 |
| *** acormier has joined #openstack-nova | 02:46 | |
| gmann | because flavor id changes and thats problem for other instance operation like resize etc | 02:47 |
| *** esberglu has quit IRC | 02:47 | |
| *** tiendc has joined #openstack-nova | 02:47 | |
| Kevin_Zheng | yeah | 02:47 |
| Kevin_Zheng | and people don't know that editing is actually deleting and creating | 02:47 |
| gmann | true | 02:48 |
| Kevin_Zheng | replies on my maillist post is welcomed :) | 02:49 |
| *** itlinux has quit IRC | 02:50 | |
| *** acormier has quit IRC | 02:50 | |
| *** mingyu has quit IRC | 02:51 | |
| *** yamamoto has joined #openstack-nova | 02:51 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to instance_extra https://review.openstack.org/457711 | 02:52 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs to Instance object https://review.openstack.org/489408 | 02:53 |
| *** dikonoor has joined #openstack-nova | 02:54 | |
| openstackgerrit | zhangdaolong proposed openstack/nova master: Add debug log for fail to get console https://review.openstack.org/490302 | 02:58 |
| *** gouthamr has quit IRC | 03:03 | |
| *** mamandle has joined #openstack-nova | 03:05 | |
| *** esberglu has joined #openstack-nova | 03:05 | |
| *** mamandle has quit IRC | 03:10 | |
| openstackgerrit | zhangdaolong proposed openstack/nova master: Keep the code consistent https://review.openstack.org/490304 | 03:10 |
| *** rajathagasthya has quit IRC | 03:10 | |
| *** nicolasbock__ has joined #openstack-nova | 03:14 | |
| *** mingyu has joined #openstack-nova | 03:14 | |
| *** kbaegis has joined #openstack-nova | 03:15 | |
| *** masuberu has joined #openstack-nova | 03:18 | |
| *** masber has quit IRC | 03:22 | |
| *** dikonoo has joined #openstack-nova | 03:24 | |
| *** kbaegis has quit IRC | 03:27 | |
| *** dikonoor has quit IRC | 03:28 | |
| *** dixiaoli has quit IRC | 03:28 | |
| *** hongbin has quit IRC | 03:29 | |
| *** tuanluong has joined #openstack-nova | 03:30 | |
| *** yamamoto has quit IRC | 03:33 | |
| *** udesale has joined #openstack-nova | 03:34 | |
| *** mamandle has joined #openstack-nova | 03:36 | |
| *** thorst_afk has joined #openstack-nova | 03:38 | |
| openstackgerrit | Tony Breeds proposed openstack/nova master: Testing reno 2.5.0 https://review.openstack.org/490313 | 03:38 |
| *** yangyapeng has joined #openstack-nova | 03:39 | |
| *** yangyapeng has quit IRC | 03:40 | |
| *** yangyapeng has joined #openstack-nova | 03:41 | |
| *** Nel1x has quit IRC | 03:42 | |
| *** yamamoto has joined #openstack-nova | 03:42 | |
| *** yangyape_ has quit IRC | 03:43 | |
| *** yangyapeng has quit IRC | 03:46 | |
| *** yamamoto has quit IRC | 03:48 | |
| *** iceyao has quit IRC | 03:49 | |
| *** yamamoto has joined #openstack-nova | 03:49 | |
| *** thorst_afk has quit IRC | 03:52 | |
| *** yamamoto has quit IRC | 03:54 | |
| *** links has joined #openstack-nova | 03:54 | |
| *** gbarros has quit IRC | 03:54 | |
| *** kiennt has quit IRC | 03:56 | |
| openstackgerrit | Tony Breeds proposed openstack/nova master: Testing reno 2.5.0 https://review.openstack.org/490321 | 04:01 |
| *** kiennt has joined #openstack-nova | 04:01 | |
| *** adisky__ has joined #openstack-nova | 04:03 | |
| *** itlinux has joined #openstack-nova | 04:06 | |
| *** mingyu has quit IRC | 04:07 | |
| *** iceyao has joined #openstack-nova | 04:14 | |
| *** gongysh has quit IRC | 04:15 | |
| *** iceyao has quit IRC | 04:19 | |
| *** yushb has quit IRC | 04:19 | |
| *** yamahata has joined #openstack-nova | 04:32 | |
| *** iceyao has joined #openstack-nova | 04:32 | |
| *** iceyao has quit IRC | 04:37 | |
| *** yangyapeng has joined #openstack-nova | 04:43 | |
| *** yangyape_ has joined #openstack-nova | 04:48 | |
| *** iceyao has joined #openstack-nova | 04:48 | |
| *** mamandle has quit IRC | 04:49 | |
| *** wxy has quit IRC | 04:51 | |
| *** mamandle has joined #openstack-nova | 04:51 | |
| *** yangyapeng has quit IRC | 04:51 | |
| *** randomha1k has joined #openstack-nova | 04:53 | |
| openstackgerrit | Jonghan Park proposed openstack/nova master: Network should be distinguished by UUID instead of Name https://review.openstack.org/490332 | 04:55 |
| *** mamandle_ has joined #openstack-nova | 04:56 | |
| *** mamandle has quit IRC | 04:56 | |
| *** mamandle_ is now known as mamandle | 04:56 | |
| *** dikonoo has quit IRC | 04:57 | |
| *** randomha1k has quit IRC | 04:58 | |
| *** udesale__ has joined #openstack-nova | 05:05 | |
| *** udesale has quit IRC | 05:08 | |
| *** mingyu has joined #openstack-nova | 05:09 | |
| *** tbachman has joined #openstack-nova | 05:11 | |
| *** ratailor has joined #openstack-nova | 05:11 | |
| *** dikonoo has joined #openstack-nova | 05:12 | |
| *** armax has quit IRC | 05:13 | |
| *** mingyu has quit IRC | 05:13 | |
| *** armax has joined #openstack-nova | 05:13 | |
| *** armax has quit IRC | 05:13 | |
| *** lpetrut has joined #openstack-nova | 05:14 | |
| *** armax has joined #openstack-nova | 05:14 | |
| *** armax has quit IRC | 05:14 | |
| *** armax has joined #openstack-nova | 05:15 | |
| *** armax has quit IRC | 05:15 | |
| *** armax has joined #openstack-nova | 05:16 | |
| *** armax has quit IRC | 05:16 | |
| *** ekuris has joined #openstack-nova | 05:18 | |
| *** esberglu has quit IRC | 05:23 | |
| *** mingyu has joined #openstack-nova | 05:23 | |
| *** trinaths has joined #openstack-nova | 05:26 | |
| *** prateek has joined #openstack-nova | 05:31 | |
| *** mamandle has quit IRC | 05:36 | |
| *** itlinux has quit IRC | 05:37 | |
| *** sridharg has joined #openstack-nova | 05:38 | |
| *** mingyu has quit IRC | 05:39 | |
| *** robcresswell has quit IRC | 05:41 | |
| *** kiennt has quit IRC | 05:45 | |
| *** thorst_afk has joined #openstack-nova | 05:49 | |
| *** yushb has joined #openstack-nova | 05:50 | |
| *** mingyu has joined #openstack-nova | 05:50 | |
| *** moshele has joined #openstack-nova | 05:51 | |
| *** psachin has joined #openstack-nova | 05:51 | |
| *** dims has quit IRC | 05:52 | |
| *** thorst_afk has quit IRC | 05:53 | |
| *** dims has joined #openstack-nova | 05:54 | |
| *** tbachman has quit IRC | 05:56 | |
| *** larainema has quit IRC | 05:56 | |
| *** brault has joined #openstack-nova | 05:58 | |
| *** slaweq has joined #openstack-nova | 06:00 | |
| *** slaweq has quit IRC | 06:02 | |
| *** dikonoo has quit IRC | 06:02 | |
| *** mwhahaha has quit IRC | 06:02 | |
| *** slaweq has joined #openstack-nova | 06:03 | |
| *** kiennt has joined #openstack-nova | 06:03 | |
| trinaths | guys! I get this error, Instance failed to spawn: VolumeDeviceNotFound . Do I need cinder volumes mandatory for nova-compute to spawn VMS? | 06:07 |
| *** slaweq has quit IRC | 06:08 | |
| *** josecastroleon has joined #openstack-nova | 06:08 | |
| *** mamandle has joined #openstack-nova | 06:10 | |
| *** lpetrut has quit IRC | 06:11 | |
| *** hoonetorg has quit IRC | 06:12 | |
| *** dikonoor has joined #openstack-nova | 06:14 | |
| *** kiennt has quit IRC | 06:18 | |
| *** kiennt has joined #openstack-nova | 06:18 | |
| *** rcernin has joined #openstack-nova | 06:19 | |
| *** mingyu has quit IRC | 06:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix getting instance bdms in multiple cells https://review.openstack.org/490340 | 06:22 |
| *** Oku_OS-away is now known as Oku_OS | 06:23 | |
| *** alexchadin has joined #openstack-nova | 06:24 | |
| *** hoonetorg has joined #openstack-nova | 06:25 | |
| *** cfriesen has quit IRC | 06:26 | |
| *** mingyu has joined #openstack-nova | 06:26 | |
| *** andreas_s has joined #openstack-nova | 06:27 | |
| *** ralonsoh has joined #openstack-nova | 06:29 | |
| *** mingyu has quit IRC | 06:31 | |
| *** mingyu has joined #openstack-nova | 06:31 | |
| *** yushb has quit IRC | 06:39 | |
| *** yushb has joined #openstack-nova | 06:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 06:40 |
| *** pcaruana has joined #openstack-nova | 06:48 | |
| *** randomha1k has joined #openstack-nova | 06:54 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 06:55 |
| *** randomha1k has quit IRC | 06:59 | |
| *** mamandle has quit IRC | 06:59 | |
| *** mamandle has joined #openstack-nova | 07:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 07:03 |
| *** slaweq has joined #openstack-nova | 07:03 | |
| *** mamandle_ has joined #openstack-nova | 07:04 | |
| *** gcb has quit IRC | 07:05 | |
| *** sshwarts has joined #openstack-nova | 07:05 | |
| *** ducnc has joined #openstack-nova | 07:07 | |
| *** mamandle has quit IRC | 07:08 | |
| *** mamandle_ is now known as mamandle | 07:08 | |
| *** slaweq has quit IRC | 07:09 | |
| *** Dinesh_Bhor has quit IRC | 07:10 | |
| *** iceyao has quit IRC | 07:13 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:16 | |
| *** tesseract has joined #openstack-nova | 07:17 | |
| *** udesale__ has quit IRC | 07:18 | |
| *** udesale__ has joined #openstack-nova | 07:19 | |
| *** iceyao has joined #openstack-nova | 07:21 | |
| *** markus_z has joined #openstack-nova | 07:21 | |
| *** thorst_afk has joined #openstack-nova | 07:22 | |
| *** yushb has quit IRC | 07:25 | |
| *** zz9pzza has joined #openstack-nova | 07:25 | |
| *** thorst_afk has quit IRC | 07:27 | |
| *** ducnc has quit IRC | 07:28 | |
| *** abalutoiu has joined #openstack-nova | 07:34 | |
| *** aarefiev_afk is now known as aarefiev | 07:35 | |
| *** priteau has joined #openstack-nova | 07:39 | |
| *** jpena|off is now known as jpena | 07:39 | |
| *** robcresswell has joined #openstack-nova | 07:42 | |
| *** abalutoiu has quit IRC | 07:44 | |
| *** abalutoiu has joined #openstack-nova | 07:51 | |
| *** larainema has joined #openstack-nova | 07:54 | |
| *** abalutoiu_ has joined #openstack-nova | 07:55 | |
| *** abalutoiu has quit IRC | 07:56 | |
| *** abalutoiu has joined #openstack-nova | 08:00 | |
| *** abalutoiu_ has quit IRC | 08:00 | |
| *** hshiina has joined #openstack-nova | 08:01 | |
| *** efoley has joined #openstack-nova | 08:02 | |
| *** mwhahaha has joined #openstack-nova | 08:03 | |
| *** jangutter_ has joined #openstack-nova | 08:03 | |
| *** slaweq has joined #openstack-nova | 08:04 | |
| *** jangutter has quit IRC | 08:05 | |
| *** lpetrut has joined #openstack-nova | 08:07 | |
| *** slaweq has quit IRC | 08:09 | |
| *** sree_ has joined #openstack-nova | 08:13 | |
| *** sree_ is now known as Guest52077 | 08:14 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/nova master: ironic: Use internal API endpoint by default https://review.openstack.org/489537 | 08:17 |
| *** Guest52077 has quit IRC | 08:17 | |
| *** derekh has joined #openstack-nova | 08:18 | |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** udesale has joined #openstack-nova | 08:18 | |
| *** udesale__ has quit IRC | 08:20 | |
| *** abalutoiu_ has joined #openstack-nova | 08:21 | |
| *** mamandle has quit IRC | 08:21 | |
| *** mingyu has quit IRC | 08:22 | |
| *** mingyu has joined #openstack-nova | 08:22 | |
| *** chyka has joined #openstack-nova | 08:22 | |
| *** mingyu has quit IRC | 08:22 | |
| *** abalutoiu has quit IRC | 08:22 | |
| *** phuongnh has quit IRC | 08:24 | |
| *** yushb has joined #openstack-nova | 08:25 | |
| *** chyka has quit IRC | 08:27 | |
| *** phuongnh has joined #openstack-nova | 08:27 | |
| *** yangyapeng has joined #openstack-nova | 08:27 | |
| *** lucas-afk is now known as lucasagomes | 08:28 | |
| *** yangyap__ has joined #openstack-nova | 08:29 | |
| *** mamandle has joined #openstack-nova | 08:30 | |
| *** yangyape_ has quit IRC | 08:32 | |
| *** zenoway has joined #openstack-nova | 08:32 | |
| *** yangyapeng has quit IRC | 08:32 | |
| *** johnhaan has quit IRC | 08:34 | |
| *** mlakat has quit IRC | 08:34 | |
| *** abalutoiu__ has joined #openstack-nova | 08:34 | |
| *** openstackgerrit has joined #openstack-nova | 08:35 | |
| openstackgerrit | Jacek Tomasiak proposed openstack/nova master: ironic: Use internal API endpoint by default https://review.openstack.org/489537 | 08:35 |
| *** zenoway has quit IRC | 08:35 | |
| *** zenoway has joined #openstack-nova | 08:36 | |
| *** abalutoiu_ has quit IRC | 08:38 | |
| *** markvoelker has quit IRC | 08:39 | |
| *** mlakat has joined #openstack-nova | 08:44 | |
| *** jangutter has joined #openstack-nova | 08:46 | |
| *** jangutter_ has quit IRC | 08:50 | |
| *** wxy has joined #openstack-nova | 08:53 | |
| *** yushb has quit IRC | 08:59 | |
| *** abalutoiu__ has quit IRC | 09:00 | |
| *** kbaegis has joined #openstack-nova | 09:01 | |
| *** isq_ has joined #openstack-nova | 09:02 | |
| *** isq has quit IRC | 09:03 | |
| *** dims has quit IRC | 09:04 | |
| *** slaweq has joined #openstack-nova | 09:05 | |
| *** yushb has joined #openstack-nova | 09:06 | |
| *** dims has joined #openstack-nova | 09:08 | |
| *** slaweq has quit IRC | 09:10 | |
| openstackgerrit | zhangdaolong proposed openstack/nova master: Add debug log for fail to get console https://review.openstack.org/490302 | 09:13 |
| *** alexchadin has quit IRC | 09:15 | |
| *** mamandle has quit IRC | 09:19 | |
| *** mamandle has joined #openstack-nova | 09:20 | |
| *** edmondsw has joined #openstack-nova | 09:22 | |
| *** thorst_afk has joined #openstack-nova | 09:23 | |
| *** mingyu has joined #openstack-nova | 09:23 | |
| *** sdague has joined #openstack-nova | 09:25 | |
| *** dikonoor has quit IRC | 09:25 | |
| *** abalutoiu has joined #openstack-nova | 09:25 | |
| *** Shunli has quit IRC | 09:27 | |
| *** thorst_afk has quit IRC | 09:28 | |
| *** mingyu has quit IRC | 09:28 | |
| *** zenoway has quit IRC | 09:30 | |
| *** zenoway has joined #openstack-nova | 09:31 | |
| *** mamandle_ has joined #openstack-nova | 09:31 | |
| *** mamandle has quit IRC | 09:34 | |
| *** mamandle_ is now known as mamandle | 09:34 | |
| *** sambetts|afk is now known as sambetts | 09:38 | |
| *** takashin has left #openstack-nova | 09:39 | |
| *** yamamoto has joined #openstack-nova | 09:41 | |
| *** jichen has quit IRC | 09:42 | |
| *** zenoway has quit IRC | 09:44 | |
| mamandle | alex_xu: bauzas: can you please take a look at https://review.openstack.org/#/c/489848/1 which is a backport for Ocata? Thanks. | 09:48 |
| *** strigazi_OFF has quit IRC | 09:49 | |
| *** strigazi_OFF has joined #openstack-nova | 09:50 | |
| *** dikonoor has joined #openstack-nova | 09:52 | |
| *** yamahata has quit IRC | 09:55 | |
| *** yangyap__ has quit IRC | 09:56 | |
| *** yangyapeng has joined #openstack-nova | 09:57 | |
| *** yangyape_ has joined #openstack-nova | 09:59 | |
| *** yangyapeng has quit IRC | 10:01 | |
| *** kiennt has quit IRC | 10:01 | |
| *** ujjain has quit IRC | 10:03 | |
| *** slaweq has joined #openstack-nova | 10:07 | |
| *** aditipuntambekar has joined #openstack-nova | 10:07 | |
| *** zenoway has joined #openstack-nova | 10:09 | |
| *** dtantsur|afk is now known as dtantsur | 10:10 | |
| *** slaweq has quit IRC | 10:12 | |
| *** yangyapeng has joined #openstack-nova | 10:12 | |
| *** yangyape_ has quit IRC | 10:15 | |
| sambetts | edleafe: Hi, someone testing out Ironic+resource classes accidentally stubbled onto this error, we've put together a bug report for you: https://bugs.launchpad.net/nova/+bug/1708424 | 10:24 |
| openstack | Launchpad bug 1708424 in OpenStack Compute (nova) "When a flavor has resource extra_specs disabling all standard fields, nova tries to make a request to the placements API with no resources" [Undecided,New] | 10:24 |
| *** dikonoor has quit IRC | 10:25 | |
| *** yangyapeng has quit IRC | 10:25 | |
| kaisers2 | mriedem: Hi! FYI regarding your email (http://lists.openstack.org/pipermail/openstack-dev/2017-July/120120.html) i found an issue (https://bugs.launchpad.net/devstack/+bug/1700583), took me quite a while to find the connection, so sorry for beeing this late | 10:29 |
| openstack | Launchpad bug 1700583 in OpenStack Compute (nova) "No volume Block Device Mapping in assisted snapshot with Quobyte" [Undecided,New] | 10:29 |
| *** hrw has left #openstack-nova | 10:29 | |
| *** bkopilov_ has quit IRC | 10:35 | |
| mdbooth | lyarwood: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata+topic:bug/1686116 | 10:35 |
| mdbooth | The code isn't invasive | 10:36 |
| mdbooth | Just putting my tinfoil hat on, trying to think what this would change to an existing instance that was rebooted, for eg | 10:37 |
| *** zhurong has quit IRC | 10:38 | |
| mdbooth | disks would have a disk_unit where previously they didn't | 10:39 |
| *** markvoelker has joined #openstack-nova | 10:40 | |
| *** ujjain has joined #openstack-nova | 10:40 | |
| *** ujjain has quit IRC | 10:40 | |
| *** ujjain has joined #openstack-nova | 10:40 | |
| *** phuongnh has quit IRC | 10:40 | |
| *** bkopilov_ has joined #openstack-nova | 10:41 | |
| *** yushb has quit IRC | 10:42 | |
| mdbooth | lyarwood: https://review.openstack.org/#/c/481137/ | 10:43 |
| mdbooth | Disks will have a device address, but only if they've been previously parsed from the domain xml? | 10:43 |
| * lyarwood reads | 10:43 | |
| mdbooth | lyarwood: Anyway, the question was whether it's fit for stable. | 10:44 |
| mdbooth | lyarwood: Is there a specific driver to backport this to stable? | 10:44 |
| robcresswell | sdague: Followed the email chain; my thought on the Horizon "feature" is to gate it behind a setting, disable by default and warning the hell out of it. | 10:45 |
| lyarwood | mdbooth: yeah correct re the question | 10:45 |
| mdbooth | I suspect that it won't break anything. | 10:45 |
| mdbooth | However, if I was of a nervous disposition I wouldn't take it. | 10:45 |
| mdbooth | The code's fine. | 10:45 |
| lyarwood | right, I can't see anything in master that would handle a corner case for active instances | 10:46 |
| lyarwood | so having it in stable should be the same | 10:46 |
| robcresswell | sdague: Referring the edit flavor issue, if that wasnt apparent. I'm not a fan of UI's inventing features that APIs dont support, so I'm on board with getting rid of it. So I'll make the settings changes and deprecate it for Pike. | 10:46 |
| lyarwood | running instances wouldn't be touched until they reboot and the domain is recreated | 10:46 |
| mdbooth | My concern would exclusively be around potential for weirdness if an instance reboots and gets very slightly different domain xml. | 10:46 |
| *** mdnadeem has quit IRC | 10:47 | |
| *** ekuris has quit IRC | 10:47 | |
| mdbooth | It's probably fine, though. | 10:47 |
| sdague | robcresswell: works for me | 10:47 |
| mdbooth | It's just a question about how conservative we are with stable, and user's reasonable expectations around such changes with minor updates within stable. | 10:48 |
| robcresswell | sdague: Cool. Sorry for the trouble thats caused you. I wasn't really aware of the issue. | 10:48 |
| sdague | robcresswell: or at least a popup warning. I think the way it was presented to users they didn't realize that it wasn't a native feature and that it would cause issues | 10:48 |
| sdague | robcresswell: yeh, honestly, a lot of the nova team didn't realize it was a thing until atlanta ptg | 10:48 |
| *** moshele has quit IRC | 10:49 | |
| *** moshele has joined #openstack-nova | 10:50 | |
| *** udesale__ has joined #openstack-nova | 10:52 | |
| *** udesale has quit IRC | 10:53 | |
| *** nicolasbock__ has quit IRC | 10:54 | |
| *** cdent has joined #openstack-nova | 10:54 | |
| robcresswell | sdague: Yeah, fair enough. At least we can start solving it now. | 10:55 |
| *** dikonoor has joined #openstack-nova | 10:55 | |
| robcresswell | sdague: If anything else like this comes up, feel free to ping me about it. I do love to remove code. | 10:55 |
| sdague | will do | 10:56 |
| *** dixiaoli has joined #openstack-nova | 10:59 | |
| *** gszasz has joined #openstack-nova | 11:00 | |
| *** ekuris has joined #openstack-nova | 11:01 | |
| *** dixiaoli has quit IRC | 11:04 | |
| *** testuro3325 has joined #openstack-nova | 11:05 | |
| *** tetsuro has joined #openstack-nova | 11:05 | |
| *** moshele has quit IRC | 11:06 | |
| *** maestropandy has joined #openstack-nova | 11:06 | |
| *** slaweq has joined #openstack-nova | 11:07 | |
| maestropandy | Please vote (+3) for our presentation submitted for openstack sydney 2017. Today last day for voting, requesting all to vote. OpenStack cloud storage - Advanced performance tuning & operational best practices with Ceph https://www.openstack.org/summit/sydney-2017/vote-for-speakers/#/19056 ##### Docker Networking over OpenStack Cloud – Challenges, Solutions & Road-map >> https://www.openstack.org/summit/sydney-2017/vote-for-speakers/#/1 | 11:08 |
| *** udesale__ has quit IRC | 11:08 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Avoid error log on 405 response https://review.openstack.org/490021 | 11:08 |
| *** iceyao has quit IRC | 11:09 | |
| *** testuro3325 has quit IRC | 11:09 | |
| *** trinaths has quit IRC | 11:12 | |
| *** slaweq has quit IRC | 11:12 | |
| *** mingyu has joined #openstack-nova | 11:12 | |
| *** markvoelker has quit IRC | 11:13 | |
| *** maestropandy has left #openstack-nova | 11:14 | |
| *** kbaegis has quit IRC | 11:17 | |
| *** mingyu has quit IRC | 11:17 | |
| *** iceyao has joined #openstack-nova | 11:17 | |
| *** moshele has joined #openstack-nova | 11:23 | |
| *** lyan has joined #openstack-nova | 11:24 | |
| *** acormier has joined #openstack-nova | 11:27 | |
| *** maestropandy has joined #openstack-nova | 11:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091 | 11:30 |
| openstackgerrit | Sean Dague proposed openstack/nova master: Fix lists in process doc https://review.openstack.org/490431 | 11:32 |
| stephenfin | maestropandy: I would imagine this also applies to IRC http://lists.openstack.org/pipermail/openstack-dev/2017-July/120121.html Please don't do it | 11:33 |
| *** cdent has quit IRC | 11:34 | |
| *** smatzek has joined #openstack-nova | 11:37 | |
| *** iceyao has quit IRC | 11:41 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Fix whitespace in rest_api_version_history https://review.openstack.org/490435 | 11:42 |
| *** iceyao has joined #openstack-nova | 11:44 | |
| *** moshele has quit IRC | 11:44 | |
| *** yangyapeng has joined #openstack-nova | 11:45 | |
| *** maestropandy has quit IRC | 11:46 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Fix list rendering in code-review.rst https://review.openstack.org/490437 | 11:47 |
| *** maestropandy has joined #openstack-nova | 11:47 | |
| *** maestropandy has left #openstack-nova | 11:47 | |
| *** lucasagomes is now known as lucas-hungry | 11:48 | |
| *** yangyapeng has quit IRC | 11:48 | |
| *** iceyao has quit IRC | 11:48 | |
| *** yangyapeng has joined #openstack-nova | 11:49 | |
| *** mkucia has joined #openstack-nova | 11:49 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in rpc doc https://review.openstack.org/490442 | 11:51 |
| *** acormier has quit IRC | 11:51 | |
| *** dikonoor has quit IRC | 11:53 | |
| sdague | stephenfin: so I'm just going through the toc list and looking for the blockquotes popping out and fixing | 11:53 |
| *** eharney has joined #openstack-nova | 11:55 | |
| *** litao__ has quit IRC | 11:59 | |
| stephenfin | sdague: Done | 12:00 |
| openstackgerrit | Sean Dague proposed openstack/nova master: Fix list rendering in bdm doc. https://review.openstack.org/490446 | 12:00 |
| *** mkucia has quit IRC | 12:02 | |
| *** moshele has joined #openstack-nova | 12:02 | |
| *** thorst_afk has joined #openstack-nova | 12:03 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in aggregates https://review.openstack.org/490448 | 12:05 |
| stephenfin | sdague: That's a lot of patches | 12:05 |
| stephenfin | tbh, if they're all that simple, I think you could squash em | 12:06 |
| openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in cells https://review.openstack.org/490449 | 12:06 |
| sdague | stephenfin: if you want I can | 12:06 |
| sdague | some of them are big enough that one per file seemed fine | 12:06 |
| sdague | and less likely to merge conflict the world and redo them all | 12:06 |
| *** david-lyle has quit IRC | 12:07 | |
| sdague | the don't use much test resource | 12:07 |
| sdague | stephenfin: the space you point out here doesn't matter for html rendering - https://review.openstack.org/#/c/490431 | 12:08 |
| *** slaweq has joined #openstack-nova | 12:08 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in resize_instance https://review.openstack.org/487495 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Split Compute.errors_out_migration into a separate contextmanager https://review.openstack.org/485734 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Functional test for auto revert resize https://review.openstack.org/489654 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: fake_notifier: Refactor wait_for_versioned_notification https://review.openstack.org/489637 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Fix scope of errors_out_migration in finish_resize https://review.openstack.org/487515 | 12:09 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | 12:09 |
| *** markvoelker has joined #openstack-nova | 12:10 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: fix list rendering in policy-enforcement https://review.openstack.org/490451 | 12:10 |
| *** chyka has joined #openstack-nova | 12:11 | |
| sdague | stephenfin: that looks like all the top level ones at least | 12:12 |
| *** iceyao has joined #openstack-nova | 12:13 | |
| *** pchavva has joined #openstack-nova | 12:13 | |
| *** slaweq has quit IRC | 12:13 | |
| *** tuanluong has quit IRC | 12:14 | |
| *** chyka has quit IRC | 12:15 | |
| *** iceyao has quit IRC | 12:17 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update the active migrations DS to support migration result https://review.openstack.org/409260 | 12:17 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: ensure live-migration success https://review.openstack.org/409261 | 12:17 |
| *** mingyu has joined #openstack-nova | 12:17 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455 | 12:18 |
| openstackgerrit | Sean Dague proposed openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455 | 12:18 |
| *** tetsuro has quit IRC | 12:19 | |
| *** adisky__ has quit IRC | 12:21 | |
| moshele | sahid: hi can you review https://review.openstack.org/#/c/398265/ ? | 12:21 |
| *** mingyu has quit IRC | 12:22 | |
| *** sree has joined #openstack-nova | 12:28 | |
| stephenfin | sdague, asettle: Oh, did we change the docs job to only build docs now? | 12:29 |
| *** tinwood is now known as tinwood-afk | 12:30 | |
| asettle | Uhh I don't think so, no | 12:30 |
| asettle | I asked about it | 12:30 |
| *** baoli has joined #openstack-nova | 12:30 | |
| asettle | You can adjust it, but there was a recommendation not to keep it | 12:30 |
| stephenfin | asettle: I ask because I note that https://review.openstack.org/#/c/490448/ has returned Jenkins results for only two jobs | 12:31 |
| stephenfin | so something changed | 12:31 |
| asettle | Oh, weird... I didn't touch that (I know, I know, I didn't stick my nose in this one :P) | 12:31 |
| asettle | stephenfin: maybe check in infra | 12:31 |
| asettle | It is only a white space edit... hmmm | 12:32 |
| *** markvoelker has quit IRC | 12:33 | |
| *** markvoelker has joined #openstack-nova | 12:33 | |
| *** catintheroof has joined #openstack-nova | 12:36 | |
| *** dosaboy has quit IRC | 12:38 | |
| *** dosaboy has joined #openstack-nova | 12:38 | |
| sambetts | jaypipes, edleafe: I think we might need to do this to fix the ironic resource class logic: https://review.openstack.org/#/c/490459/ | 12:39 |
| *** esberglu has joined #openstack-nova | 12:40 | |
| *** gcb has joined #openstack-nova | 12:41 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: WIP: resize with custom resource https://review.openstack.org/490461 | 12:41 |
| *** iceyao has joined #openstack-nova | 12:42 | |
| *** maestropandy has joined #openstack-nova | 12:42 | |
| *** maestropandy has left #openstack-nova | 12:43 | |
| *** jpena is now known as jpena|lunch | 12:44 | |
| *** esberglu has quit IRC | 12:45 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: WIP: resize with custom resource https://review.openstack.org/490461 | 12:46 |
| *** zhurong has joined #openstack-nova | 12:48 | |
| *** kylek3h has joined #openstack-nova | 12:52 | |
| edleafe | sambetts: just getting caffeinated | 12:52 |
| edleafe | sambetts: looking at the bug report, it seems that the the flavor itself in wrong | 12:52 |
| *** udesale has joined #openstack-nova | 12:53 | |
| sambetts | edleafe: the bug report was for a different issue | 12:53 |
| edleafe | sambetts: it zeroes out the "standard" resources, but doesn't include the ironic resource class | 12:53 |
| sambetts | edleafe: the bug report is unrelated to using it with Ironic | 12:53 |
| *** takedakn has joined #openstack-nova | 12:53 | |
| stephenfin | sdague: They're all done. The only one I'm not gone on is the last. I can fix the issues I have if you want https://review.openstack.org/#/c/490455/2 | 12:53 |
| edleafe | sambetts: well, not really | 12:53 |
| *** liverpooler has joined #openstack-nova | 12:53 | |
| edleafe | as Ironic would be the only system using flavors like that | 12:54 |
| sambetts | but in a non-ironic deployment there is nothing stopping me creating a flavor like that and nova doesn't error nicely | 12:54 |
| sambetts | thats what the bug report is saying | 12:54 |
| *** lucas-hungry is now known as lucasagomes | 12:54 | |
| sambetts | thats why it doesn't mention ironic | 12:55 |
| sambetts | you can't say its not a bug because someone would just not do that in a normal case | 12:55 |
| edleafe | sambetts: Sure, but there are a zillion incorrect ways to do things. We can't protect against every wrong setup | 12:55 |
| *** fmccrthy has quit IRC | 12:55 | |
| edleafe | sambetts: it is returning a 400, as it should | 12:56 |
| *** mamandle has quit IRC | 12:56 | |
| *** stvnoyes has joined #openstack-nova | 12:57 | |
| *** gbarros has joined #openstack-nova | 12:58 | |
| *** fmccrthy has joined #openstack-nova | 12:58 | |
| *** markus_z has quit IRC | 12:59 | |
| sambetts | I just don't expect that nova should even make the request to the placement API, because the error message identifies a bad request from nova | 13:00 |
| *** cdent has joined #openstack-nova | 13:04 | |
| * cdent renames gibi to chaos-monkey (with admiration) | 13:05 | |
| *** yamamoto has quit IRC | 13:06 | |
| *** links has quit IRC | 13:06 | |
| * gibi takes the new name with pride | 13:06 | |
| *** gibi is now known as gibi_chaos-monke | 13:07 | |
| *** gibi_chaos-monke is now known as gibi_chaosmonkey | 13:07 | |
| cdent | \o/ | 13:07 |
| *** josecastroleon has quit IRC | 13:08 | |
| cdent | so no matt, jay or dan yet it seems, I guess I’ll see if can fix the test issues in 488510 | 13:08 |
| *** josecastroleon has joined #openstack-nova | 13:08 | |
| *** slaweq has joined #openstack-nova | 13:09 | |
| *** esberglu has joined #openstack-nova | 13:10 | |
| *** ratailor has quit IRC | 13:10 | |
| *** lucasxu has joined #openstack-nova | 13:11 | |
| *** psachin has quit IRC | 13:13 | |
| *** slaweq has quit IRC | 13:14 | |
| *** markus_z has joined #openstack-nova | 13:16 | |
| edleafe | cdent: there are some ironic resource class issues if you're bored :) | 13:18 |
| cdent | edleafe: heh, issues abound | 13:19 |
| cdent | i’m looking at fixing these test issues | 13:19 |
| *** aditipuntambekar has quit IRC | 13:19 | |
| cdent | they are mostly mechanical | 13:19 |
| cdent | but plentiful | 13:19 |
| edleafe | cool | 13:19 |
| *** gbarros has quit IRC | 13:19 | |
| *** udesale has quit IRC | 13:22 | |
| *** dikonoor has joined #openstack-nova | 13:22 | |
| *** udesale has joined #openstack-nova | 13:23 | |
| *** psachin has joined #openstack-nova | 13:24 | |
| *** mriedem has joined #openstack-nova | 13:24 | |
| *** randomha1k has joined #openstack-nova | 13:27 | |
| *** udesale has quit IRC | 13:28 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455 | 13:29 |
| *** takedakn has quit IRC | 13:29 | |
| *** mtanino has joined #openstack-nova | 13:29 | |
| *** takedakn has joined #openstack-nova | 13:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | 13:30 |
| *** mamandle has joined #openstack-nova | 13:33 | |
| *** dikonoor has quit IRC | 13:34 | |
| *** dikonoor has joined #openstack-nova | 13:35 | |
| *** takedakn has quit IRC | 13:36 | |
| openstackgerrit | Merged openstack/nova master: Fix lists in process doc https://review.openstack.org/490431 | 13:37 |
| *** yushb has joined #openstack-nova | 13:39 | |
| dansmith | mriedem: are you working on that set? | 13:40 |
| openstackgerrit | Merged openstack/nova master: Fix whitespace in rest_api_version_history https://review.openstack.org/490435 | 13:40 |
| dansmith | mriedem: I just commented on both the bottom two | 13:40 |
| *** tinwood-afk is now known as tinwood | 13:41 | |
| openstackgerrit | Merged openstack/nova master: Fix list rendering in code-review.rst https://review.openstack.org/490437 | 13:41 |
| *** smatzek has quit IRC | 13:41 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Make use of definition lists, literals https://review.openstack.org/490481 | 13:42 |
| cdent | dansmith, mriedem I was going to fix the failing unit tests on the top patch, but my lack of familiarity is meaning that teasing out the mocks is proving slow. If one of you two want to pick it up, that’s cool, otherwise I’ll carry on and learn something | 13:43 |
| stephenfin | mriedem: Is it too late to ask for FFE for this? https://review.openstack.org/#/c/453808/3 | 13:43 |
| *** yushb has quit IRC | 13:44 | |
| stephenfin | I can't work on moving nova-manage to cliff for another six months without it | 13:44 |
| dansmith | cdent: okay I'm surprised mriedem or jaypipes aren't already in the middle of that, so let's confirm before I go make another mess | 13:44 |
| *** awaugama has joined #openstack-nova | 13:44 | |
| *** cfriesen has joined #openstack-nova | 13:44 | |
| cdent | at least one of the failures is because we’ve now got code within methods making demands that unrelated “unit” tests are not satisfying | 13:46 |
| *** mmehan has joined #openstack-nova | 13:46 | |
| cdent | (just the first one, the others make some sense) | 13:46 |
| *** jpena|lunch is now known as jpena | 13:46 | |
| *** crushil has quit IRC | 13:48 | |
| *** felipemonteiro has joined #openstack-nova | 13:48 | |
| *** felipemonteiro_ has joined #openstack-nova | 13:49 | |
| *** sree has quit IRC | 13:49 | |
| *** jaypipes has joined #openstack-nova | 13:49 | |
| *** moshele has quit IRC | 13:50 | |
| cdent | dansmith: are you jumping, or shall I carry on with what I’m able to get for now? | 13:51 |
| dansmith | cdent: I'm handling other morning stuff waiting for mriedem to catch me up.. if you're fixing things, I'd just keep going with that | 13:52 |
| dansmith | and/or push fixes you already have working | 13:52 |
| cdent | roger that | 13:52 |
| cdent | check with me before you dive in | 13:52 |
| cdent | I’ll get what I can make sense of | 13:52 |
| dansmith | ack | 13:52 |
| *** felipemonteiro has quit IRC | 13:53 | |
| jaypipes | I could use a catch-up too. | 13:53 |
| *** moshele has joined #openstack-nova | 13:54 | |
| *** tiendc has quit IRC | 13:55 | |
| openstackgerrit | Merged openstack/nova master: fix list rendering in rpc doc https://review.openstack.org/490442 | 13:55 |
| openstackgerrit | Merged openstack/nova master: Fix list rendering in bdm doc. https://review.openstack.org/490446 | 13:56 |
| openstackgerrit | Merged openstack/nova master: fix list rendering in aggregates https://review.openstack.org/490448 | 13:57 |
| *** efried has joined #openstack-nova | 13:58 | |
| openstackgerrit | Merged openstack/nova master: fix list rendering in cells https://review.openstack.org/490449 | 13:58 |
| openstackgerrit | Merged openstack/nova master: fix list rendering in policy-enforcement https://review.openstack.org/490451 | 13:58 |
| *** crushil has joined #openstack-nova | 13:59 | |
| dansmith | jaypipes: I had just said I was waiting to see if mriedem had anything queued locally for that set, I assume that means you do not? | 14:00 |
| * cdent will have a small number of adjustments to push in a few minutes | 14:01 | |
| *** smatzek has joined #openstack-nova | 14:01 | |
| dansmith | jaypipes: per my comment a few minutes ago, if we don't address the flavor overrides in that un-doubling, we'll mess up accounting after a move with an overriding flavor | 14:02 |
| *** krtaylor has quit IRC | 14:02 | |
| dansmith | and edleafe has a flavor-to-resources method we can just use I think | 14:02 |
| *** alezil has quit IRC | 14:03 | |
| *** dikonoor has quit IRC | 14:03 | |
| *** xyang1 has joined #openstack-nova | 14:04 | |
| *** alezil has joined #openstack-nova | 14:05 | |
| *** smatzek has quit IRC | 14:05 | |
| *** hshiina has quit IRC | 14:05 | |
| *** hongbin has joined #openstack-nova | 14:06 | |
| *** yamamoto has joined #openstack-nova | 14:06 | |
| *** acormier has joined #openstack-nova | 14:07 | |
| *** tbachman has joined #openstack-nova | 14:08 | |
| *** annegentle has joined #openstack-nova | 14:09 | |
| mriedem | dansmith: cdent: just got home, | 14:10 |
| mriedem | i'm not currently working on anything | 14:10 |
| *** slaweq has joined #openstack-nova | 14:10 | |
| *** pchavva has quit IRC | 14:10 | |
| dansmith | mriedem: okay | 14:11 |
| dansmith | cdent: so I guess push your changes when you're ready and then I will see about fixing the resource override stuff | 14:11 |
| *** yamamoto has quit IRC | 14:11 | |
| *** sree has joined #openstack-nova | 14:12 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 14:12 |
| cdent | good timing ^ | 14:12 |
| cdent | that fixes the pep8 problems, and the calls where flavor was not included in a call to remove_provider_from … | 14:12 |
| cdent | but it does not fix an assertion with AssertionError: Expected 'delete_allocation_for_instance' to be called once. Called 0 times. | 14:13 |
| cdent | nor some mock problems in “unrelated” tests | 14:13 |
| cdent | dansmith, mriedem, jaypipes, I release the lock | 14:14 |
| *** smatzek has joined #openstack-nova | 14:14 | |
| *** josecastroleon has quit IRC | 14:15 | |
| *** slaweq has quit IRC | 14:15 | |
| dansmith | ah, mriedem's patch should already be kosher in this respect actually | 14:15 |
| dansmith | I was thinking with my uncaffeinated brain | 14:15 |
| dansmith | it's just the last one that needs it | 14:16 |
| cdent | just to highlight it, gibi pointed out yet another edge case on ps16: https://review.openstack.org/#/c/490461 | 14:16 |
| *** zhurong has quit IRC | 14:17 | |
| mriedem | dansmith: so no resource override in my change? | 14:17 |
| *** markus_z has quit IRC | 14:17 | |
| mriedem | b/c if so, you'll have to deal with it b/c resource override is lost on me | 14:17 |
| dansmith | mriedem: right yours should be good because the scheduler has already done that part | 14:17 |
| mriedem | ok | 14:17 |
| *** josecastroleon has joined #openstack-nova | 14:18 | |
| *** udesale has joined #openstack-nova | 14:19 | |
| *** mamandle_ has joined #openstack-nova | 14:19 | |
| *** mamandle has quit IRC | 14:20 | |
| *** mamandle_ is now known as mamandle | 14:20 | |
| *** felipemonteiro_ has quit IRC | 14:22 | |
| *** lucasxu has quit IRC | 14:22 | |
| jaypipes | dansmith: sorry, having some issues today :( | 14:22 |
| jaypipes | dansmith: last I checked in I was waiting on the bottom patch in that series to test positive and then I was going to appove it (it was the test resize same host patch) | 14:23 |
| jaypipes | dansmith: I think my status currently is I need to pull whatever the latest that mriedem pushed last night and cherry-pick the changes I was working on last night onto that series. | 14:24 |
| *** dfisher has joined #openstack-nova | 14:24 | |
| dansmith | okay that's what I was just asking, if you had pending changes | 14:24 |
| jaypipes | dansmith: those changes include fixing up the "cheating" section | 14:24 |
| dansmith | anyway, let me finish what I'm doing here, which is fixing the cheating section, but so it accounts for flavor overrides | 14:24 |
| dansmith | almost done with that | 14:24 |
| jaypipes | dansmith: oh, ok | 14:24 |
| jaypipes | dansmith: np | 14:24 |
| jaypipes | dansmith: and sorry, I'm having some medical issues today. trying to get in to see a doctor. | 14:25 |
| dansmith | ack, np | 14:25 |
| jaypipes | so it's not an ideal work day unfortunately :( | 14:25 |
| jaypipes | if cdent would send me a box of Crunchie bars and a new hip and lower back, that would be groovy. | 14:26 |
| * cdent sends jaypipes warm fuzzies and a box of crunchies | 14:26 | |
| jaypipes | thx cdent :) | 14:26 |
| cdent | they are a part of my official travel to openstack events habit, so if you can survive until september: hooked up | 14:26 |
| *** moshele has quit IRC | 14:26 | |
| * mriedem sends jaypipes a coritsone shot | 14:27 | |
| mriedem | *cortisone | 14:27 |
| mriedem | damn | 14:27 |
| cdent | I’m going home, brb | 14:27 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: imagebackend: cleanup constructor args to Rbd https://review.openstack.org/490499 | 14:27 |
| *** cdent has quit IRC | 14:27 | |
| jaypipes | mriedem: that cortisone come with morphine? | 14:27 |
| mdbooth | ^^^ is the most trivial trivial patch I've ever submitted, btw | 14:28 |
| * mdbooth waits for somebody to ask for a test | 14:28 | |
| *** moshele has joined #openstack-nova | 14:28 | |
| sdague | https://bugs.launchpad.net/nova/+bug/1708424 ... that's a weird bug | 14:28 |
| openstack | Launchpad bug 1708424 in OpenStack Compute (nova) "When a flavor has resource extra_specs disabling all standard fields, nova tries to make a request to the placements API with no resources" [Undecided,New] | 14:28 |
| mdbooth | Actually, I'm sure I've submitted a typo fix in a comment before | 14:28 |
| *** marst has joined #openstack-nova | 14:28 | |
| edleafe | sdague: IMO that one is operator error | 14:29 |
| sdague | yeh | 14:29 |
| edleafe | sdague: they go to the trouble of zeroing out all the standard resources, but never add the ironic resource class | 14:29 |
| *** moshele has quit IRC | 14:30 | |
| edleafe | sdague: so perhaps we could catch that in Nova, but if they really wanna shoot themselves in the foot... | 14:30 |
| sdague | yeh | 14:30 |
| *** markus_z has joined #openstack-nova | 14:30 | |
| sdague | so is that a hack to do something ironicy? | 14:30 |
| *** thorst_afk has quit IRC | 14:30 | |
| *** jangutter has quit IRC | 14:31 | |
| edleafe | It's part of the switch to ironic custom resource classes | 14:31 |
| *** jangutter has joined #openstack-nova | 14:31 | |
| *** jangutter has quit IRC | 14:32 | |
| jaypipes | mdbooth: +2 from me. | 14:32 |
| edleafe | In Queens, an ironic flavor will have the extra_specs contain the custom resource class, along with zeroes for the "standard" resources | 14:32 |
| *** jangutter has joined #openstack-nova | 14:32 | |
| *** thorst_afk has joined #openstack-nova | 14:32 | |
| sdague | bauzas: why did you mark https://bugs.launchpad.net/nova/+bug/1707160 as critical even though you didn't think it was a nova bug? | 14:32 |
| openstack | Launchpad bug 1707160 in neutron "test_create_port_in_allowed_allocation_pools test fails on ironic grenade" [Critical,Confirmed] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 14:32 |
| edleafe | In Pike, it should just have the ironic custom RC | 14:32 |
| *** zenoway has quit IRC | 14:32 | |
| *** zenoway has joined #openstack-nova | 14:33 | |
| bauzas | sdague: just for getting traction | 14:33 |
| bauzas | because it's a gate issue | 14:33 |
| bauzas | but anyway | 14:34 |
| mdbooth | jaypipes: Thanks, also for the attaboy ;) | 14:34 |
| jaypipes | mdbooth: heh :) | 14:34 |
| sdague | bauzas: ok, I thought we save critical for must fix rc bugs | 14:34 |
| bauzas | sdague: np, your modification is good to me | 14:35 |
| *** mingyu has joined #openstack-nova | 14:35 | |
| *** sree has quit IRC | 14:36 | |
| *** ekuris has quit IRC | 14:36 | |
| mriedem | melwitt: some suggestions in https://review.openstack.org/#/c/470578/ | 14:37 |
| *** thorst_afk has quit IRC | 14:37 | |
| melwitt | mriedem: cool, thanks | 14:38 |
| *** lpetrut has quit IRC | 14:39 | |
| *** gszasz has quit IRC | 14:39 | |
| sdague | I'm assuming this would need a spec - https://bugs.launchpad.net/nova/+bug/1708458 ? | 14:39 |
| openstack | Launchpad bug 1708458 in OpenStack Compute (nova) "Expose instance system_metadata in compute API" [Undecided,New] | 14:39 |
| *** mamandle has quit IRC | 14:40 | |
| *** gaoyan has joined #openstack-nova | 14:40 | |
| mriedem | sdague: jesus yes | 14:42 |
| mriedem | we shouldn't flat out expose system metadata | 14:42 |
| *** zenoway has quit IRC | 14:42 | |
| mriedem | "if you want to query the point in time properties that where inherited from an image during the launch." | 14:42 |
| mriedem | expose those as some other field then | 14:42 |
| *** thorst_afk has joined #openstack-nova | 14:42 | |
| mriedem | we don't need to expose all of the garbage we carry in system metadata | 14:43 |
| mriedem | like old_vm_state during a resize | 14:43 |
| mriedem | bauzas: i backported the placement fix to overwrite allocations https://review.openstack.org/#/c/490231/ | 14:44 |
| *** pchavva has joined #openstack-nova | 14:45 | |
| sdague | ok, I'm good with Won't Fix | 14:46 |
| *** MVenesio has joined #openstack-nova | 14:47 | |
| *** armax has joined #openstack-nova | 14:47 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | 14:47 |
| mriedem | well, alternatively the fix is a microversion to expose some specific part of system metadata and what that entails | 14:47 |
| *** yamamoto has joined #openstack-nova | 14:47 | |
| *** yamamoto has quit IRC | 14:47 | |
| mriedem | sdague: ever thought about indexing qemu instance logs in our ci runs? | 14:47 |
| mriedem | when live migration jobs fail, a lot of the time it's due to | 14:48 |
| mriedem | http://logs.openstack.org/10/490110/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/6c1da1c/logs/subnode-2/libvirt/qemu/instance-00000003.txt.gz | 14:48 |
| mriedem | /build/qemu-orucB6/qemu-2.8+dfsg/nbd/server.c:nbd_co_receive_request():L1135: reading from socket failed | 14:48 |
| mriedem | but ^ isn't exposed in anything we index | 14:48 |
| jaypipes | dansmith: did you see cdent just pushed a revision on the confirm resize patch? | 14:48 |
| *** jamesden_ has joined #openstack-nova | 14:48 | |
| *** slagle has quit IRC | 14:48 | |
| dansmith | jaypipes: a bit ago while we were talking, yeah. he said so and that's when I pulled to start working | 14:48 |
| jaypipes | gotcha. just making sure you noticed. carry on. | 14:49 |
| *** kbaegis has joined #openstack-nova | 14:49 | |
| mriedem | what i'd really love is if the libvirt / qemu job had some way to get those details from the guest | 14:49 |
| *** jamesdenton has quit IRC | 14:49 | |
| mriedem | kashyap: mdbooth: you know how during a live migration we're checking the domain job status to see when it completes, or if it fails? | 14:50 |
| mdbooth | mriedem: Yep | 14:50 |
| mriedem | is there any way to get the qemu guest logs when that fails, like http://logs.openstack.org/10/490110/2/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/6c1da1c/logs/subnode-2/libvirt/qemu/instance-00000003.txt.gz | 14:50 |
| *** slagle has joined #openstack-nova | 14:50 | |
| mriedem | i really want: /build/qemu-orucB6/qemu-2.8+dfsg/nbd/server.c:nbd_co_receive_request():L1135: reading from socket failed | 14:50 |
| *** gaoyan has quit IRC | 14:50 | |
| mriedem | when ^ happens, the only failure we get in the n-cpu logs is that on the destination when we're doing post-live migration at destination, the instance (guest domain) isn't found | 14:51 |
| mriedem | because it blew up on the source side | 14:51 |
| *** felipemonteiro has joined #openstack-nova | 14:52 | |
| mdbooth | mriedem: Is ^^^ from dest? | 14:52 |
| *** zenoway has joined #openstack-nova | 14:52 | |
| * mdbooth assumes so | 14:52 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:53 | |
| *** eharney has quit IRC | 14:53 | |
| mriedem | no that's source | 14:53 |
| mriedem | here is another one http://logs.openstack.org/66/483566/10/check/gate-grenade-dsvm-neutron-multinode-live-migration-nv/0437fbe/logs/subnode-2/libvirt/qemu/instance-00000011.txt.gz | 14:53 |
| mriedem | different error, but results in the same kind of thing in the dest n-cpu logs | 14:54 |
| *** gszasz has joined #openstack-nova | 14:54 | |
| mriedem | InstanceNotFound during post live migration at destination | 14:54 |
| mriedem | b/c it failed on the source | 14:54 |
| cfriesen | mriedem: what's the complication with getting that file from the dest? | 14:54 |
| *** slaweq has joined #openstack-nova | 14:55 | |
| mriedem | https://bugs.launchpad.net/nova/+bug/1706377 | 14:55 |
| openstack | Launchpad bug 1706377 in OpenStack Compute (nova) "(libvirt) live migration fails on source host due to "Assertion `!(bs->open_flags & BDRV_O_INACTIVE)' failed."" [Undecided,Confirmed] | 14:55 |
| mdbooth | mriedem: Why are we calling post if the migration failed? | 14:55 |
| mriedem | mdbooth: because libvirt told us the job was complete | 14:56 |
| mriedem | see my notes in https://bugs.launchpad.net/nova/+bug/1706377 | 14:56 |
| mdbooth | mriedem: *That's* the bug | 14:56 |
| mdbooth | mriedem: And we already kinda knew about that, right? | 14:56 |
| mdbooth | Didn't I leave a comment in there to that effect? | 14:56 |
| *** felipemonteiro has quit IRC | 14:56 | |
| mriedem | in where? | 14:57 |
| mdbooth | libvirt/drive | 14:57 |
| mdbooth | r | 14:57 |
| * mdbooth goes looking | 14:57 | |
| mdbooth | mriedem: Sorry, libvirt/guest.py | 14:57 |
| mdbooth | is_job_complete | 14:57 |
| dansmith | jaypipes: mriedem: okay I got the resource override stuff working in jaypipes' patch and some unified code between them for doubling/undoubling resources, so now I'm going to look at the peripheral test failures | 14:58 |
| mdbooth | mriedem: It's there in one of my trademark big blocks of comment | 14:58 |
| dansmith | I have about 30 minutes until my next meeting so I will push ahead of that regardless of my progress | 14:58 |
| *** trinaths1 has joined #openstack-nova | 14:58 | |
| *** zenoway has quit IRC | 14:58 | |
| mdbooth | # Secondly, with the current method we only know that 'no job' | 14:58 |
| mdbooth | # indicates completion. It does not necessarily indicate successful | 14:58 |
| mdbooth | # completion: the job could have failed, or been cancelled. When | 14:58 |
| mdbooth | # polling for block job info we have no way to detect this, so we | 14:58 |
| mdbooth | # assume success. | 14:58 |
| *** zenoway has joined #openstack-nova | 14:58 | |
| jaypipes | dansmith: k. I'm happy to take the baton on fixing periphery tests when you go to your meeting. | 14:58 |
| mriedem | ah ok, | 14:58 |
| mriedem | that was written around the time of the great swap volume rewrite | 14:59 |
| dansmith | jaypipes: ack | 14:59 |
| mriedem | cfriesen: i don't understand your question | 14:59 |
| *** yingjun has joined #openstack-nova | 14:59 | |
| mriedem | cfriesen: the migration completes but actually fails on the source, | 14:59 |
| *** krtaylor has joined #openstack-nova | 15:00 | |
| mriedem | but we don't know it fails, we just know the job is 'complete' so we tell dest to do post live migration stuff, and when it does, the guest never made it to dest (or it was deleted by libvirt when it found that the source failed) | 15:00 |
| *** Swami has joined #openstack-nova | 15:00 | |
| mriedem | so i'm trying to figure out a way to get the qemu instance logs into the n-cpu logs for debug | 15:00 |
| mdbooth | mriedem: I think we should rewrite that polling block to consume events instead. It's also less buggy. | 15:00 |
| cfriesen | mriedem: I was just thinking that we had all the info needed to get the file, so didn't see what the problem was....but it's not "can we get the file", but "can we determine there was a failure so we know to go get the file" | 15:00 |
| *** thorst_afk has quit IRC | 15:00 | |
| mdbooth | As in, it was designed for this in the first place. | 15:00 |
| mriedem | cfriesen: maybe, i don't know how configurable that path is | 15:01 |
| *** gbarros has joined #openstack-nova | 15:01 | |
| *** thorst_afk has joined #openstack-nova | 15:01 | |
| mriedem | seems pretty hacky though, i'd think you could get qemu guest logs from libvirt apis | 15:01 |
| mdbooth | mriedem: I don't think so, btw. | 15:02 |
| *** zenoway has quit IRC | 15:02 | |
| cfriesen | mriedem: ah, right, we don't control all the clouds this runs on. I think it is configurable where those logs go. | 15:02 |
| *** zenoway has joined #openstack-nova | 15:02 | |
| mriedem | right | 15:03 |
| mriedem | that's why i'd need an api | 15:03 |
| mdbooth | Basically we should switch to using libvirt events api. Extensive documentation here: http://libvirt.org/docs/libvirt-appdev-guide/en-US/html/Application_Development_Guide-Guest_Domains-Event_Not.html | 15:04 |
| cfriesen | with a big TBD on that page? | 15:04 |
| mriedem | mdbooth: yeah i suppose virConnectDomainEventJobCompletedCallback | 15:04 |
| mdbooth | cfriesen: You need more? Pshaw | 15:04 |
| *** sridharg has quit IRC | 15:05 | |
| *** Oku_OS is now known as Oku_OS-away | 15:05 | |
| mdbooth | cfriesen: It's a small TBD, anyway. Classier that way. | 15:05 |
| mdbooth | mriedem: I wonder if we could register a libvirt error handler, and dump errors into nova compute logs as a matter of course:http://libvirt.org/docs/libvirt-appdev-guide-python/en-US/html/libvirt_application_development_guide_using_python-Error_Handling-Registering_Error_Handler.html | 15:07 |
| mdbooth | That might achieve what you want in practise. | 15:07 |
| *** zenoway has quit IRC | 15:07 | |
| *** kbaegis has quit IRC | 15:07 | |
| *** cdent has joined #openstack-nova | 15:09 | |
| *** eharney has joined #openstack-nova | 15:09 | |
| *** priteau has quit IRC | 15:09 | |
| mriedem | where is the error array defined? | 15:11 |
| mdbooth | mriedem: rtfs | 15:11 |
| cfriesen | mriedem: mdbooth: is there a libvirt bug here? I mean the source is running _live_migration_monitor() and calling guest.get_job_info(). shouldn't libvirt detect a failure? | 15:11 |
| mriedem | ha, we already register an error handler | 15:11 |
| mriedem | def _libvirt_error_handler(context, err): | 15:11 |
| mriedem | # Just ignore instead of default outputting to stderr. | 15:11 |
| mriedem | pass | 15:11 |
| mdbooth | mriedem: hehe | 15:12 |
| *** gbarros has quit IRC | 15:12 | |
| cfriesen | and if it doesn't, are we going to get an error in the callback? | 15:12 |
| mdbooth | cfriesen: No | 15:12 |
| mriedem | "with error being a list of information about the error being raised. " | 15:12 |
| mriedem | i suppose it's similar to a libvirtError | 15:12 |
| mdbooth | cfriesen: I don't recall the detail now, but at the time kashyap and I went over the libvirt and libvirt python binding code very carefully | 15:13 |
| mdbooth | cfriesen: We're extracting everything from it which can be extracted | 15:13 |
| mriedem | ah yup it's just the libvirtError.err list | 15:14 |
| mdbooth | Hence my big comment explaining what we're not getting | 15:14 |
| mdbooth | It's not designed to be used this way | 15:15 |
| mdbooth | The intention was that you'd consume events instead. That api has been stable for much longer. | 15:15 |
| sdague | mriedem: you'd need another grok parser | 15:17 |
| *** iceyao has quit IRC | 15:19 | |
| cfriesen | mdbooth: okay, I think I got it. On another note, currently with block live migration if the guest is dirtying disk quickly the current logs don't show information about the initial block transfer. | 15:19 |
| mriedem | would be nice if the libvirtError python binding class just had a nice __repr__ | 15:20 |
| mriedem | maybe it does already... | 15:20 |
| openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 15:21 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 15:21 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | 15:21 |
| dansmith | jaypipes: I gotta start getting ready for my call, so I'm pushing.. the last patch is the only one that needs attention, AFAIK, a few fails in unit tests at least | 15:22 |
| jaypipes | dansmith: got it. will take the ball. | 15:22 |
| *** krtaylor has quit IRC | 15:22 | |
| *** annegentle has quit IRC | 15:23 | |
| dansmith | jaypipes: note the new patch in the middle that adds a couple of utils and generalizes something in mriedem's patch | 15:24 |
| *** annegentle has joined #openstack-nova | 15:24 | |
| jaypipes | dansmith: noted | 15:24 |
| jaypipes | dansmith: I presume that's not the patch with test failures, though, yes? the top is the one with failures? | 15:25 |
| dansmith | jaypipes: just your last confirm/revert one yeah | 15:25 |
| jaypipes | got it thx | 15:25 |
| *** gszasz_ has joined #openstack-nova | 15:27 | |
| *** gszasz has quit IRC | 15:27 | |
| mriedem | comments in the utils patch | 15:31 |
| *** zz9pzza has quit IRC | 15:32 | |
| *** baoli has quit IRC | 15:36 | |
| *** iceyao has joined #openstack-nova | 15:38 | |
| *** baoli has joined #openstack-nova | 15:38 | |
| *** iceyao has quit IRC | 15:42 | |
| *** dtp has joined #openstack-nova | 15:43 | |
| *** randomha1k has quit IRC | 15:44 | |
| *** sambetts is now known as sambetts|afk | 15:45 | |
| *** rebase has joined #openstack-nova | 15:46 | |
| *** rebase has left #openstack-nova | 15:47 | |
| *** yamamoto has joined #openstack-nova | 15:48 | |
| *** andreas_s has quit IRC | 15:48 | |
| *** penick has joined #openstack-nova | 15:49 | |
| *** gaoyan has joined #openstack-nova | 15:50 | |
| *** randomha1k has joined #openstack-nova | 15:51 | |
| *** aarefiev is now known as aarefiev_afk | 15:53 | |
| *** yamamoto has quit IRC | 15:53 | |
| *** penick has quit IRC | 15:53 | |
| *** sree has joined #openstack-nova | 15:55 | |
| *** gaoyan has quit IRC | 15:55 | |
| *** udesale has quit IRC | 15:55 | |
| mriedem | cdent: where would one find the current placement api-ref? | 15:58 |
| mriedem | or is that still just in builds that touch it? | 15:58 |
| cdent | mriedem: there’s no publishing job yet as we were waiting for the stack to merge so that we could then move it to the new location and _then_ publish it | 15:59 |
| *** psachin has quit IRC | 16:01 | |
| ildikov | mriedem: hey | 16:01 |
| *** mtanino has quit IRC | 16:01 | |
| ildikov | mriedem: coming to the meeting? | 16:01 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 16:03 |
| mriedem | cdent: ok i was going to see if we needed any changes to the api-ref for the allocations minItems:1 thing | 16:04 |
| *** gaoyan has joined #openstack-nova | 16:05 | |
| cdent | mriedem: i think this one is allocations: https://review.openstack.org/#/c/470933/ | 16:05 |
| cdent | so not merged yet | 16:05 |
| *** gaoyan_ has joined #openstack-nova | 16:06 | |
| *** gaoyan has quit IRC | 16:06 | |
| *** david-lyle has joined #openstack-nova | 16:07 | |
| *** gaoyan_ has quit IRC | 16:08 | |
| *** Apoorva has joined #openstack-nova | 16:08 | |
| *** thorst_afk has quit IRC | 16:08 | |
| cdent | mriedem: when you use the term “latent issue” what does that actually mean? | 16:08 |
| mriedem | not introduced in pike | 16:09 |
| mriedem | not a regression | 16:09 |
| cdent | thanks | 16:10 |
| *** sshwarts has quit IRC | 16:11 | |
| *** rcernin has quit IRC | 16:12 | |
| *** krtaylor has joined #openstack-nova | 16:12 | |
| *** dave-mccowan has joined #openstack-nova | 16:14 | |
| mriedem | cdent: ok questions in https://review.openstack.org/#/c/470933/ | 16:15 |
| *** thorst_afk has joined #openstack-nova | 16:15 | |
| cdent | mriedem: cool, I’ll hope andrey can look at those soon. If not, I can, I’m in the api-wg meeting now and then after that am gone (officially) until monday | 16:16 |
| *** mingyu has quit IRC | 16:16 | |
| *** mingyu has joined #openstack-nova | 16:16 | |
| *** jmlowe has quit IRC | 16:20 | |
| *** vishwana_ has joined #openstack-nova | 16:20 | |
| *** vishwanathj has quit IRC | 16:24 | |
| *** mingyu has quit IRC | 16:26 | |
| *** JoseMello has joined #openstack-nova | 16:29 | |
| mriedem | dtantsur: jlvillal: did ironic ever go ahead with raising minimum required microversions? | 16:31 |
| dtantsur | mriedem: nope, we haven't got to it | 16:31 |
| dtantsur | I think we have an api-wg guideline for that, resulting from our discussions | 16:32 |
| mriedem | ok | 16:33 |
| mriedem | edleafe: i think the unit test failures in https://review.openstack.org/#/c/487925/ are probably related | 16:33 |
| mriedem | it's super rare to have a random unit test failure with the ironic stuff in nova, and it's suspect when you're touching that driver | 16:33 |
| mriedem | need some cores to look at this https://review.openstack.org/#/c/489763/ - it's a regression in pike, pretty simple fix | 16:35 |
| *** burt has joined #openstack-nova | 16:35 | |
| edleafe | mriedem: those failures do seem to be more or less random. They don't happen locally | 16:35 |
| mriedem | edleafe: i've never seen either of those happen in our ci though | 16:37 |
| mriedem | http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22AssertionError%3A%20Expected%20call%3A%20call('node.list'%2C%20associated%3DTrue%2C%20limit%3D0)%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d | 16:39 |
| mriedem | that only shows up in that change | 16:39 |
| openstackgerrit | Merged openstack/nova master: reflow rpc doc to 80 columns https://review.openstack.org/490455 | 16:40 |
| *** Swami has quit IRC | 16:40 | |
| openstackgerrit | Merged openstack/nova master: doc: Make use of definition lists, literals https://review.openstack.org/490481 | 16:41 |
| *** oomichi has quit IRC | 16:41 | |
| *** jmlowe has joined #openstack-nova | 16:41 | |
| *** lucasagomes is now known as lucas-afk | 16:42 | |
| *** sree has quit IRC | 16:42 | |
| *** annegentle has quit IRC | 16:42 | |
| *** sree has joined #openstack-nova | 16:43 | |
| *** smatzek has quit IRC | 16:43 | |
| mriedem | bauzas: any idea why the ServerGroupAntiAffinityFilter would stop working if you aren't tracking instance changes between the compute and scheduler service? Like i know there is a race possibility, but is that filter completely dependent on those? shouldn't it fallback to check the db or something if it's not tracking instance updates? | 16:44 |
| *** oomichi has joined #openstack-nova | 16:44 | |
| mriedem | because with superconductor we don't have the upcalls from the computes to the scheduler for tracking instance chnages | 16:44 |
| mriedem | *changes | 16:44 |
| *** sergek_ has quit IRC | 16:47 | |
| *** sree has quit IRC | 16:48 | |
| *** sergek_ has joined #openstack-nova | 16:48 | |
| dansmith | jaypipes: I have emerged from my hole | 16:48 |
| *** efoley has quit IRC | 16:49 | |
| dansmith | jaypipes: any progress? | 16:50 |
| jaypipes | dansmith: finishing up test runs now | 16:51 |
| *** yufei has joined #openstack-nova | 16:51 | |
| jaypipes | dansmith: had to go into your sched utils patch and add support for boot-from-volume... :( | 16:51 |
| dansmith | my patch didn't regress that, right? | 16:54 |
| *** vks1 has joined #openstack-nova | 16:54 | |
| dansmith | I dunno what change is needed for that, but I'll check it out when you push | 16:54 |
| *** pcaruana has quit IRC | 16:54 | |
| jaypipes | dansmith: mriedem had pointed out that we were not handling bfv in the "cheating" section. | 16:55 |
| jaypipes | dansmith: thus my needing to put it into the resources_from_flavor() method | 16:56 |
| dansmith | I'm saying I don't know what needs doing is all | 16:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 16:56 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | 16:56 |
| jaypipes | dansmith: gotcha. it required passing the instance as well as flavor. pls see above. | 16:56 |
| *** lyan has quit IRC | 16:56 | |
| jaypipes | dansmith: I'm still working on the top patch (confirm resize one) | 16:56 |
| jaypipes | dansmith: but figured I'd push to show you what I changed. | 16:57 |
| dansmith | oh zeroing root, I see | 16:57 |
| *** dtantsur is now known as dtantsur|afk | 16:57 | |
| *** cdent has quit IRC | 16:57 | |
| jaypipes | dansmith: ya | 16:57 |
| jaypipes | dansmith: I added you as co-author on the confirm resize one. | 16:58 |
| jaypipes | dansmith: tag, you're it. :) | 16:58 |
| *** derekh has quit IRC | 16:58 | |
| dansmith | co-blamee | 16:58 |
| jaypipes | heh | 16:58 |
| dansmith | jaypipes: does that mean you want me fixing the tests now? | 16:58 |
| jaypipes | dansmith: nah, I might in a little bit, but I'm currently working on them. I need to head out at 12:15 your time, at which point I'll push what I've gotten done. | 16:59 |
| dansmith | ah | 16:59 |
| dansmith | er ack | 16:59 |
| *** gszasz_ has quit IRC | 16:59 | |
| *** smatzek has joined #openstack-nova | 17:01 | |
| *** efried has quit IRC | 17:01 | |
| *** rcernin has joined #openstack-nova | 17:02 | |
| *** armax_ has joined #openstack-nova | 17:03 | |
| *** armax has quit IRC | 17:03 | |
| *** armax_ is now known as armax | 17:03 | |
| dansmith | once again, BFV is a blemish on otherwise organized stuff | 17:04 |
| *** trinaths1 has left #openstack-nova | 17:04 | |
| *** rajathagasthya has joined #openstack-nova | 17:07 | |
| mriedem | didn't you see the ops thread? boeing wants all bfv all the time | 17:07 |
| dansmith | lots of people do | 17:07 |
| mriedem | by extension, the entire US military wants BFV | 17:07 |
| mriedem | 25% budget baby! | 17:08 |
| dansmith | and if we had the ephemeral driver, that would be totally fine | 17:08 |
| mriedem | sorry, 60% i think | 17:08 |
| *** iceyao has joined #openstack-nova | 17:10 | |
| *** baoli has quit IRC | 17:10 | |
| *** Swami has joined #openstack-nova | 17:13 | |
| *** diablo_rojo has joined #openstack-nova | 17:14 | |
| *** iceyao has quit IRC | 17:14 | |
| *** yufei has quit IRC | 17:16 | |
| *** markus_z has quit IRC | 17:17 | |
| *** lyan has joined #openstack-nova | 17:24 | |
| *** jpena is now known as jpena|off | 17:25 | |
| *** yamahata has joined #openstack-nova | 17:25 | |
| *** randomha1k has quit IRC | 17:26 | |
| *** randomha1k has joined #openstack-nova | 17:29 | |
| *** baoli has joined #openstack-nova | 17:29 | |
| mriedem | danicus maximus, are you ok with me adding the other limitations to the cells v2 doc like upcalls and such | 17:39 |
| mriedem | ? | 17:39 |
| dansmith | mriedem: sure, I was going to do that but then it merged | 17:40 |
| dansmith | so I can if you want, but feel free | 17:40 |
| mriedem | i'm going to poke through logs first and try to sort out just why the anti affinity group filter fails if we're not tracking instances | 17:43 |
| dansmith | okay I'll push up some things for the doc then | 17:44 |
| *** tesseract has quit IRC | 17:49 | |
| *** yingjun has quit IRC | 17:49 | |
| dansmith | mriedem: | 17:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 17:50 |
| *** vks1 has quit IRC | 17:51 | |
| *** gouthamr has joined #openstack-nova | 17:53 | |
| *** baoli has quit IRC | 17:58 | |
| *** annegentle has joined #openstack-nova | 17:59 | |
| mriedem | what the f | 18:02 |
| mriedem | sdague: this is totally wrong, right? https://github.com/openstack/nova/blob/master/nova/policies/server_groups.py#L34 | 18:02 |
| mriedem | BASE_POLICY_RULE == rule:os_compute_api:os-server-groups | 18:02 |
| mriedem | which should be something like, rule:admin_or_owner | 18:03 |
| jaypipes | dansmith: still working on fixes. down to the one func test failure (that ocata pike ironic one). unit tests all fixed up now. | 18:05 |
| sdague | mriedem: yeh, that doesn't look right | 18:05 |
| dansmith | jaypipes: ack | 18:05 |
| mriedem | sdague: goes back to ocata https://review.openstack.org/#/c/391113/ | 18:06 |
| mriedem | edmondsw: ^ you helped review this | 18:06 |
| mriedem | does this make any sense? | 18:06 |
| *** baoli has joined #openstack-nova | 18:07 | |
| *** lyan has quit IRC | 18:08 | |
| edmondsw | mriedem sdague yeah, there's some odd history there | 18:10 |
| mriedem | well i see that here https://review.openstack.org/#/c/391113/13/nova/api/openstack/compute/server_groups.py | 18:11 |
| mriedem | it was doing context.can(sg_policies.BASE_POLICY_NAME) | 18:11 |
| edmondsw | originally there was only rule:os_compute_api:os-server-groups and for backward compat we wanted whatever someone had for that to still be what they got if they used the new rules | 18:11 |
| mriedem | and the rule for that was | 18:11 |
| mriedem | policy.RuleDefault( | 18:11 |
| mriedem | name=BASE_POLICY_NAME, | 18:11 |
| mriedem | check_str=base.RULE_ADMIN_OR_OWNER), | 18:11 |
| mriedem | it was using rule:admin_or_owner before | 18:11 |
| mriedem | i don't see where rule:os_compute_api:os-server-groups came from | 18:11 |
| *** ralonsoh has quit IRC | 18:12 | |
| *** kbaegis has joined #openstack-nova | 18:13 | |
| *** kbaegis has quit IRC | 18:13 | |
| edmondsw | tat is rule:os_compute_api:os-server-groups that you just pasted | 18:13 |
| edmondsw | BASE_POLICY_NAME = os_compute_api:os-server-groups | 18:14 |
| edmondsw | so in order for the the new rules to default to whatever you have set for that old rule, we pointed them to the old rule... rule:os_compute_api:os-server-groups | 18:14 |
| edmondsw | mriedem make sense? | 18:14 |
| edmondsw | backward compat and policy are a nightmare... | 18:15 |
| edmondsw | one of the things we want to talk about fixing at the PTG | 18:15 |
| mriedem | so context.can(sg_policies.BASE_POLICY_NAME) turns that into rule:os_compute_api:os-server-groups ? | 18:18 |
| mriedem | in oslo.context? | 18:18 |
| mriedem | or oslo.policy | 18:18 |
| mriedem | isn't context.can just looking up the action? | 18:18 |
| mriedem | which was os_compute_api:os-server-groups | 18:18 |
| mriedem | and in the before times, that mapped to rule:admin_or_owner | 18:18 |
| mriedem | not rule:os_compute_api:os-server-groups | 18:18 |
| mriedem | dansmith: i left some random thoughts in your cells v2 docs patch | 18:19 |
| mriedem | fighting with over/under doc'ing | 18:19 |
| mriedem | edmondsw: yeah here https://docs.openstack.org/nova/latest/configuration/sample-policy.html | 18:20 |
| dansmith | mriedem: I was leaving the console thing to melwitt | 18:20 |
| mriedem | #"os_compute_api:os-server-groups": "rule:admin_or_owner" | 18:20 |
| mriedem | was the old thing | 18:20 |
| mriedem | and was changed to | 18:20 |
| mriedem | #"os_compute_api:os-server-groups:create": "rule:os_compute_api:os-server-groups" | 18:20 |
| mriedem | which is exactly backward incompatible | 18:20 |
| mriedem | edmondsw: also going back to mitaka before policy in code https://github.com/openstack/nova/blob/mitaka-eol/etc/nova/policy.json#L445 | 18:22 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle ironicclient failures in Ironic driver https://review.openstack.org/487925 | 18:25 |
| *** slaweq_ has joined #openstack-nova | 18:25 | |
| *** slaweq has quit IRC | 18:27 | |
| mriedem | edmondsw: https://bugs.launchpad.net/nova/+bug/1708508 | 18:27 |
| openstack | Launchpad bug 1708508 in OpenStack Compute (nova) "os-server-groups policy rules are wrong" [Undecided,New] | 18:27 |
| mriedem | lbragstad: what happens if you define a policy action to have a rule which is not actually defined? | 18:27 |
| mriedem | does it just default to the default rule? | 18:27 |
| lbragstad | mriedem: i would assume that to error | 18:28 |
| lbragstad | i can test though | 18:28 |
| mriedem | lbragstad: like say i define "os_compute_api:os-server-groups:create": "rule:doesnotexist", | 18:28 |
| mriedem | and there is no rule for doesnotexist | 18:28 |
| edmondsw | mriedem os_compute_api:os-server-groups is a rule... the value for that rule is by default "rule:admin_or_owner" because there's another rule called admin_or_owner, and then you look at the value for admin_or_owner... they chain | 18:28 |
| edmondsw | so what we did here was add another level to that chain | 18:28 |
| edmondsw | when you use a rule as a value, you need to prefix "rule:" | 18:29 |
| edmondsw | make sense? | 18:29 |
| mriedem | no | 18:29 |
| mriedem | looking at https://docs.openstack.org/nova/latest/configuration/sample-policy.html | 18:29 |
| mriedem | i see | 18:29 |
| mriedem | #"admin_or_owner": "is_admin:True or project_id:%(project_id)s" | 18:29 |
| mriedem | defining the policy for the admin_or_owner rule | 18:29 |
| edmondsw | take "os_compute_api:os-server-groups:create": "rule:os_compute_api:os-server-groups" for example | 18:29 |
| mriedem | oh shit | 18:29 |
| mriedem | i get it now | 18:29 |
| mriedem | #"os_compute_api:os-server-groups": "rule:admin_or_owner" | 18:29 |
| edmondsw | :) | 18:30 |
| mriedem | gdi | 18:30 |
| mriedem | #"os_compute_api:os-server-groups:create": "rule:os_compute_api:os-server-groups" | 18:30 |
| mriedem | points to | 18:30 |
| mriedem | #"os_compute_api:os-server-groups": "rule:admin_or_owner" | 18:30 |
| edmondsw | yep | 18:30 |
| mriedem | my god | 18:30 |
| mriedem | lbragstad: nvm | 18:30 |
| edmondsw | so when we remove os_compute_api:os-server-groups we have to also, at the same time, update the defaults for the other rules to go straight to admin_or_owner instead of via the removed rule | 18:31 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | 18:33 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | 18:33 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rework index page per new sections https://review.openstack.org/478485 | 18:33 |
| mriedem | yup | 18:33 |
| mriedem | and we can do that in queens i guess | 18:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 18:34 |
| mriedem | edmondsw: thanks for holding my hand | 18:35 |
| mriedem | policy is a scary neighborhood | 18:35 |
| *** acormier has quit IRC | 18:35 | |
| mriedem | like TMNT lane that dan lives on | 18:35 |
| *** acormier has joined #openstack-nova | 18:35 | |
| edmondsw | mriedem np... policy will drive you nuts | 18:35 |
| lbragstad | fact | 18:37 |
| dansmith | mriedem: heh | 18:40 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 18:40 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | 18:40 |
| jaypipes | dansmith: ok, so I'm still stumped on that ocata to pike ironic failure. everything else is passing. | 18:41 |
| * dansmith pulls | 18:41 | |
| jaypipes | dansmith: hoping you might put some fresh eyes on that, since I need to run soon. | 18:41 |
| mriedem | melwitt: you want to look at this additional cells v2 multi-cell doc goody? https://review.openstack.org/#/c/490612/ | 18:41 |
| jaypipes | dansmith: I suspect it might have something to do with the new resources extra specs processing thing, but honestly I'm not positive. | 18:41 |
| dansmith | jaypipes: yeah. should we slam in mriedem's single node patch at the bttom before you go? | 18:42 |
| jaypipes | dansmith: I hadn't seen that, but I trust you whatever you want sure | 18:42 |
| dansmith | oh wait | 18:42 |
| dansmith | er, cancel that wait | 18:42 |
| jaypipes | heh | 18:42 |
| dansmith | jaypipes: this guy: https://review.openstack.org/#/c/490085/7 | 18:42 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate '--version' parameters https://review.openstack.org/453808 | 18:43 |
| jaypipes | dansmith: ah, yeah, totes. I didn't touch that guy | 18:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate '--version' parameters https://review.openstack.org/453808 | 18:43 |
| dansmith | yeah | 18:43 |
| jaypipes | dansmith: +2 from me. | 18:43 |
| *** dtp has quit IRC | 18:43 | |
| dansmith | mriedem: you okay with me +2ing that even though I did the test bit? | 18:43 |
| mriedem | for my change? | 18:45 |
| mriedem | sure | 18:45 |
| dansmith | yeah | 18:45 |
| mriedem | it wasn't much - just removing TODOs | 18:45 |
| mriedem | not to make your contribution to the change seem small or otherwise unimportant, of course :) | 18:46 |
| * mriedem strokes the ego | 18:46 | |
| * dansmith keeps sobbing | 18:46 | |
| *** MVenesio has quit IRC | 18:47 | |
| *** prashkre has joined #openstack-nova | 18:47 | |
| dansmith | jaypipes: just unit tests I need to look at right? functional all work? | 18:47 |
| *** slaweq_ has quit IRC | 18:47 | |
| jaypipes | dansmith: nope, all units should work fine. it's one functional test in tests/functional/compute/test_resource_tracker.py. | 18:48 |
| dansmith | oh okay | 18:48 |
| jaypipes | dansmith: it's expecting instance_claim() to have written allocation records for an ironic instance but it's not writing those allocations for some reason. | 18:48 |
| dansmith | okay | 18:49 |
| jaypipes | dansmith: and if I had to guess, maybe something to do with how we're pulling resources now with extra specs? | 18:49 |
| dansmith | yeah found it | 18:49 |
| dansmith | the test I mean | 18:49 |
| prashkre | mriedem: Hi. Have you created a LP bug for issue with patch in https://review.openstack.org/#/c/391113/? | 18:50 |
| *** prateek has quit IRC | 18:51 | |
| mriedem | prashkre: i did but invalidated it once i understood how the policy rules were being linked | 18:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add track_instance_changes note in disable_group_policy_check_upcall https://review.openstack.org/490627 | 18:54 |
| prashkre | mriedem: I got confused with your query, now I got it why we had made it like that. due to compatablitiy with people who already started using that rule, it was made like that.. | 18:56 |
| *** masber has joined #openstack-nova | 18:57 | |
| mriedem | melwitt: L35 in here about cell caches https://etherpad.openstack.org/p/nova-pike-cells-v2-todos - i think we have that covered in the FAQs page | 18:58 |
| mriedem | but let me know if you were thinking of something else | 18:58 |
| *** masuberu has quit IRC | 18:59 | |
| openstackgerrit | Merged openstack/nova master: hardware offload support for openvswitch https://review.openstack.org/398265 | 19:08 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Handle addition of new nodes/instances in ironic flavor migration https://review.openstack.org/487954 | 19:10 |
| *** dtp has joined #openstack-nova | 19:12 | |
| *** Sukhdev has joined #openstack-nova | 19:13 | |
| mriedem | dansmith: ok i traced this ServerGroupAntiAffinityFilter test which is doing min_count=2 so multi-create, and the filter scheduler correctly puts the instances on different hosts | 19:15 |
| mriedem | because of this https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L229 | 19:16 |
| dansmith | mriedem: because one scheduling request for N instances maintains the state, but two parallel requests do not, right? | 19:16 |
| mriedem | the first host chosen for the first instance is put in the request spec hosts list | 19:16 |
| mriedem | yeah this is single request, | 19:16 |
| mriedem | parallel would require the upcall safety check | 19:16 |
| dansmith | right | 19:16 |
| mriedem | single request, 2 instances, seems ok | 19:16 |
| dansmith | I thought that's what I was tryin gto say | 19:17 |
| *** slaweq has joined #openstack-nova | 19:17 | |
| mriedem | ah ok | 19:17 |
| *** jianghuaw has quit IRC | 19:17 | |
| mriedem | so if you care about affinity/anti-affinity, the user has to do multi-create or stagger their create requests | 19:17 |
| mriedem | i'd think most are doing multi-create in this kind of use case anyway... | 19:18 |
| *** ykuo_ has joined #openstack-nova | 19:18 | |
| dansmith | well, if you are about it, presumably you want it to actually work, not just work in some circumstances | 19:18 |
| dansmith | I dunno, maybe | 19:18 |
| dansmith | if you're booting two nodes to be mirrors, then maybe | 19:18 |
| mriedem | yeah if you don't care that much, you create the server group using the soft affinity/anti-affinity policy | 19:18 |
| *** pchavva has quit IRC | 19:18 | |
| *** Sukhdev_ has joined #openstack-nova | 19:19 | |
| mriedem | if i'm creating 2 db VMs and want them on different hosts for protection, then i want it to work | 19:19 |
| dansmith | right, which might be a num_instances=2 thing | 19:19 |
| dansmith | but if you're bursting, | 19:20 |
| cfriesen | mriedem: what about heat? | 19:20 |
| dansmith | two things might be adding more webserver processes in parallel in response to load | 19:20 |
| cfriesen | mriedem: we've had a bunch of issues due to heat issuing requests in quick succession | 19:20 |
| dansmith | and those wouldn't be the single request | 19:20 |
| mriedem | dansmith: yeah | 19:20 |
| mriedem | sucks for the end user since the way this is going to behave is not up to them, it's dependent on the configuration of the cloude, | 19:21 |
| mriedem | *cloud, | 19:21 |
| mriedem | but in the beforetimes, | 19:21 |
| mriedem | this could fail due to load anyway | 19:21 |
| mriedem | or dropped messages | 19:21 |
| *** eharney has quit IRC | 19:22 | |
| dansmith | well, the late check will fail and error the instance if it can't verify | 19:22 |
| cfriesen | dansmith: weren't we dropping the late check with cellsv2? or did that get added back? | 19:22 |
| dansmith | cfriesen: that's what we're discussing | 19:22 |
| mriedem | sweet my test passed http://logs.openstack.org/54/489754/6/experimental/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/a4fbbb3/console.html#_2017-08-03_18_04_43_399430 | 19:23 |
| cfriesen | dansmith: for what it's worth, we're missing an equivalent late check in the live-migration path. we found a race there. | 19:23 |
| mriedem | we need an equivalent to dan's "HA and Pray" slogan for live migration | 19:23 |
| dansmith | live migration is always prayer-powered | 19:24 |
| *** jianghuaw has joined #openstack-nova | 19:25 | |
| * dfisher snorts | 19:25 | |
| dfisher | i shouldn't have laughed that hard | 19:25 |
| *** felipemonteiro_ has quit IRC | 19:27 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:27 | |
| *** mingyu has joined #openstack-nova | 19:27 | |
| *** baoli has quit IRC | 19:27 | |
| *** prashkre has quit IRC | 19:28 | |
| cfriesen | seems like to close these sched races we need something like "make the choice, store the choice in the DB, verify the choice was valid and retry if not". | 19:28 |
| dansmith | jaypipes: you're gone right? | 19:28 |
| *** baoli has joined #openstack-nova | 19:28 | |
| dansmith | cfriesen: that's placement | 19:28 |
| *** gyee has joined #openstack-nova | 19:29 | |
| *** felipemonteiro_ has quit IRC | 19:30 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:30 | |
| *** mingyu has quit IRC | 19:32 | |
| dansmith | mriedem: I think I've fixed jaypipes' thing | 19:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 19:32 |
| cfriesen | dansmith: so are we leaving races in there (due to parallel schedulers for example)? As I understand it we originally needed the re-verification because we didn't store the scheduler decision to the DB until after the request hit the compute node. | 19:33 |
| mriedem | sdague: these old images actually seem pretty useful https://review.openstack.org/#/c/478470/ | 19:34 |
| mriedem | i wonder why they were no longer used | 19:34 |
| dansmith | cfriesen: when we track affinity in placement we won't need any of this because we'll be making the writing to the db as part of making the decison | 19:34 |
| mriedem | although the might be inaccurate now | 19:34 |
| cfriesen | dansmith: ah, okay. | 19:35 |
| *** Sukhdev has quit IRC | 19:36 | |
| *** Sukhdev_ is now known as Sukhdev | 19:36 | |
| dansmith | gdi there's some gabbit failure now | 19:36 |
| dansmith | which I suck at | 19:36 |
| *** lyan has joined #openstack-nova | 19:38 | |
| *** artom_ has quit IRC | 19:38 | |
| *** artom_ has joined #openstack-nova | 19:38 | |
| sdague | mriedem yeh, sfinucan and I went back and forth on that patch. I pushed for wanting to keep them, but they aren't used today, and it is more important that we get things merged | 19:38 |
| *** esberglu has quit IRC | 19:38 | |
| *** ijw has joined #openstack-nova | 19:38 | |
| mriedem | sdague: we also have https://docs.openstack.org/nova/latest/reference/vm-states.html | 19:39 |
| mriedem | which is probably due for an audit also | 19:39 |
| sdague | mriedem: many things are due for an audit | 19:39 |
| mriedem | i'm trying to find the ironic doc that has all of their state machinery | 19:39 |
| sdague | while I was fixing up the all misrendered block quotes I got a survery through what's there | 19:40 |
| mriedem | why don't we have a search toolbar with the new docs theme? | 19:40 |
| sdague | we need a pretty concrete discussion on information architecture at the PTG | 19:40 |
| sdague | mriedem: I don't know | 19:40 |
| mriedem | oh i found it | 19:41 |
| mriedem | way at the bottom | 19:41 |
| sdague | that's not search in our docs though, what is that? | 19:42 |
| openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | 19:43 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514 | 19:43 |
| *** felipemonteiro_ has quit IRC | 19:47 | |
| *** efried has joined #openstack-nova | 19:48 | |
| *** esberglu has joined #openstack-nova | 19:53 | |
| mriedem | efried: what am i supposed to call powervm again? so as to not confuse it with powervm support via libvirt (phyp?) | 19:55 |
| efried | mriedem PowerVM. That other thing doesn't exist. thorst_afk ? | 19:56 |
| *** harlowja has quit IRC | 19:57 | |
| mriedem | PowerVMv2: the re-powering | 19:57 |
| *** annegentle has quit IRC | 19:57 | |
| *** randomha1k has quit IRC | 19:58 | |
| efried | mriedem Is there some context in which that other thing is actually present an causing confusion? | 19:58 |
| *** jianghuaw has quit IRC | 19:58 | |
| efried | and* | 19:58 |
| thorst_afk | It was my understanding that the libvirt code for powervm is not fully functional (I'll have to investigate a bit). I know there was some work on it several years ago, but I don't know that anyone has been maintaining or updating it for years. | 20:00 |
| mriedem | no i was just going to add it to the list here https://review.openstack.org/#/c/480074/7 | 20:00 |
| mriedem | and wasn't sure if there was a more specific name | 20:00 |
| mriedem | that wasn't NEO or something | 20:00 |
| mriedem | but we use PowerVM in the feature support matrix, so that makes sense to use here also | 20:00 |
| *** annegentle has joined #openstack-nova | 20:01 | |
| thorst_afk | PowerVM is what we want to go with | 20:01 |
| efried | In the words of Michael Bolton, "Why should I change? He's the one who sucks." | 20:01 |
| *** awaugama has quit IRC | 20:02 | |
| *** eharney has joined #openstack-nova | 20:02 | |
| *** jianghuaw has joined #openstack-nova | 20:04 | |
| mriedem | great, now i'm thinking about all of michael bolton's classic smokey throated wonder hits | 20:05 |
| dansmith | mriedem: powervm != powerkvm if that's what you mean | 20:06 |
| mriedem | dansmith: heh, i know | 20:06 |
| dansmith | I don't think powervm via libvirt works from nova | 20:06 |
| mriedem | powervm != powervc | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import installation guide https://review.openstack.org/477488 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Import administration guide https://review.openstack.org/477497 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Start using oslo_policy.sphinxext https://review.openstack.org/479358 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: policies: Fix Sphinx issues https://review.openstack.org/480516 | 20:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 20:06 |
| mriedem | the 3rd party ci says it's working | 20:06 |
| dansmith | link? | 20:06 |
| *** liverpooler has quit IRC | 20:08 | |
| mriedem | http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/65/398265/34/check/tempest-dsvm-full-xenial/8144094/ | 20:10 |
| mriedem | they even have a grenade job | 20:10 |
| mriedem | http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/nova/65/398265/34/check/grenade-dsvm-xenial/27db95e/ | 20:10 |
| dansmith | that's pkvm == powerkvm right? | 20:10 |
| mriedem | yeah | 20:10 |
| mriedem | oh you said powervm via libvirt | 20:10 |
| dansmith | yeah | 20:10 |
| mriedem | yeah nvm then | 20:10 |
| mmedvede | so, it is KVM on Power, not to be confused with PowerKVM OS | 20:10 |
| mmedvede | naming is unfortunate | 20:11 |
| mriedem | https://libvirt.org/drvphyp.html | 20:11 |
| mriedem | is http://www-03.ibm.com/systems/power/software/virtualization/index.html the old powervm driver? | 20:11 |
| mriedem | that ran ssh commands through the vios shell or whatever? | 20:11 |
| mriedem | it's been awhile | 20:12 |
| dansmith | mriedem: phyp via libvirt is the thing I doubt works with nova | 20:12 |
| dansmith | mriedem: which is different than the powervm driver that was in nova at first | 20:12 |
| dansmith | I think the phyp driver uses local hypercall access, IIRC | 20:12 |
| mriedem | hypercall? | 20:13 |
| mriedem | that sounds enterprise ready | 20:13 |
| dansmith | hypercal is like a syscall but to the hypervisor | 20:13 |
| dansmith | *hypercall | 20:13 |
| efried | http://www-03.ibm.com/systems/power/software/virtualization/index.html is the PowerVM that we're enabling via nova.virt.powervm | 20:14 |
| *** kbaegis has joined #openstack-nova | 20:15 | |
| *** slaweq has quit IRC | 20:18 | |
| *** slaweq has joined #openstack-nova | 20:18 | |
| *** slaweq has quit IRC | 20:20 | |
| *** slaweq has joined #openstack-nova | 20:20 | |
| *** baoli has quit IRC | 20:22 | |
| *** lpetrut has joined #openstack-nova | 20:23 | |
| *** mriedem has quit IRC | 20:25 | |
| *** smatzek has quit IRC | 20:31 | |
| *** kbaegis has quit IRC | 20:35 | |
| *** hamzy has quit IRC | 20:37 | |
| openstackgerrit | Sean Dague proposed openstack/nova master: Add Contributor Guide section page https://review.openstack.org/490644 | 20:40 |
| *** Sukhdev_ has joined #openstack-nova | 20:42 | |
| *** mriedem has joined #openstack-nova | 20:44 | |
| mriedem | ooo team meeting in 15 minutes | 20:45 |
| sdague | mriedem: https://review.openstack.org/#/c/477488 - follow on comment about why it's fine to have it where it is | 20:47 |
| *** felipemonteiro has joined #openstack-nova | 20:47 | |
| melwitt | mriedem, dansmith: question on the cellsv2 doc https://review.openstack.org/490612 | 20:48 |
| *** slaweq has quit IRC | 20:48 | |
| sdague | mriedem: basically to make the TOC not complete sidebar vomit, we're going to strip a lot out of it an the front page and build topic landing pages with deep links like - https://review.openstack.org/#/c/490644/ | 20:48 |
| *** slaweq has joined #openstack-nova | 20:49 | |
| sdague | that lets us tidy things up without another giant page shuffle that we'd be 404 chasing | 20:49 |
| mriedem | melwitt: replied | 20:49 |
| openstackgerrit | Merged openstack/nova master: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | 20:50 |
| dansmith | mriedem: do you remember the name of the affinity reporting conf item? I was going to add that in there for her, but I can never remember the name | 20:52 |
| melwitt | mriedem: okay, so I guess that means there's nothing wrong with compute node reporting, it's just that the lag between the two (report vs affinity check) could cause a problem with parallel requests | 20:52 |
| *** felipemonteiro_ has joined #openstack-nova | 20:52 | |
| dansmith | melwitt: no | 20:52 |
| dansmith | melwitt: there is a compute->scheduler reporting mechanism that can't happen either | 20:52 |
| dansmith | track_instance_changes | 20:53 |
| mriedem | yeah that one | 20:53 |
| dansmith | he had it in his previous comment | 20:53 |
| mriedem | i mentioned that eralier | 20:53 |
| mriedem | *earlier even | 20:53 |
| melwitt | okay, maybe I'm hard of reading :( | 20:53 |
| mriedem | there are about 10 pieces to any single affinity filter puzzle | 20:53 |
| *** slaweq has quit IRC | 20:53 | |
| mriedem | like, (1) server group (2) scheduler hint (3) filters (4) various config options | 20:54 |
| mriedem | and then, maybe, it might work | 20:54 |
| dansmith | (5) luck | 20:54 |
| *** takashin has joined #openstack-nova | 20:54 | |
| melwitt | yeah, it's just when I read "instance info reporting doesn't work" seemed sort of alarming. the second paragraph explains what the end point is, it's just after reading something like "compute node reporting doesn't work" seems sort of alarming, at least to me | 20:55 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 20:55 |
| cfriesen | dansmith: /s/luck/timing | 20:55 |
| *** felipemonteiro has quit IRC | 20:56 | |
| mriedem | melwitt: dansmith: i've thought about putting some details about the issue into the FAQs page, like "why does affinity/anti-affinity seem to suck now with multi-cell?" | 20:57 |
| dansmith | melwitt: better? | 20:57 |
| dansmith | mriedem: s/now.*// | 20:57 |
| mriedem | that's not as discoverable though | 20:57 |
| mriedem | heh | 20:57 |
| mriedem | especially for an end user | 20:57 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 20:57 |
| mriedem | the end user shouldn't have any idea if the reason things don't work is because of the multi-cell decision in the cloud | 20:58 |
| dansmith | well, | 20:58 |
| dansmith | nor should the user be surprised that the cloud op doesn't have that filter enabled | 20:58 |
| dansmith | but alas | 20:58 |
| mriedem | you've got a bug in there | 20:59 |
| mriedem | i'm intimately familiar with that option now | 20:59 |
| dansmith | I've got a bug in who now? | 20:59 |
| melwitt | dansmith: yes, much better, thanks. that makes it clear IMHO | 20:59 |
| mriedem | your change | 20:59 |
| mriedem | meeting starting | 21:00 |
| melwitt | what's wrong with it? | 21:00 |
| dansmith | oh god ugly | 21:00 |
| *** Apoorva_ has joined #openstack-nova | 21:00 | |
| mriedem | ha | 21:00 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 21:01 |
| *** krtaylor has quit IRC | 21:02 | |
| *** itlinux has joined #openstack-nova | 21:03 | |
| *** Sukhdev_ has quit IRC | 21:03 | |
| *** Apoorva has quit IRC | 21:04 | |
| *** ijw has quit IRC | 21:04 | |
| *** rtjure has joined #openstack-nova | 21:05 | |
| *** ijw has joined #openstack-nova | 21:05 | |
| *** imacdonn has quit IRC | 21:08 | |
| *** imacdonn has joined #openstack-nova | 21:09 | |
| *** alkhodos_ has joined #openstack-nova | 21:09 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 21:09 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 21:09 |
| *** slaweq has joined #openstack-nova | 21:10 | |
| *** crushil has quit IRC | 21:10 | |
| *** thorst_afk has quit IRC | 21:10 | |
| *** thorst_afk has joined #openstack-nova | 21:12 | |
| *** Sukhdev_ has joined #openstack-nova | 21:12 | |
| dansmith | edleafe: can you push a rev to that spec to address jaypipes' typo nits at least? | 21:12 |
| *** iceyao has joined #openstack-nova | 21:13 | |
| edleafe | dansmith: sure thing | 21:13 |
| dansmith | thanks | 21:13 |
| *** slaweq has quit IRC | 21:14 | |
| *** harlowja has joined #openstack-nova | 21:15 | |
| *** Apoorva_ has quit IRC | 21:16 | |
| *** Apoorva has joined #openstack-nova | 21:16 | |
| *** thorst_afk has quit IRC | 21:17 | |
| *** iceyao has quit IRC | 21:18 | |
| efried | jangutter Some colleagues and I are talking about revamping PCI in general and SR-IOV specifically. Please add me to reviews for blueprints and the like. | 21:18 |
| *** jmlowe has quit IRC | 21:18 | |
| jangutter | efried: will do, I should also be at the Queens PTG. | 21:19 |
| efried | jangutter See you there. We'll hopefully be on the agenda with a (draft, at least) blueprint. | 21:19 |
| *** efoley has joined #openstack-nova | 21:20 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 21:21 |
| *** yamamoto has joined #openstack-nova | 21:21 | |
| dansmith | mriedem: thanks for that.. was going too fast | 21:21 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 21:22 |
| *** yamamoto has quit IRC | 21:26 | |
| alkhodos_ | Hi everyone. I have a question about live migration. Is there an option to migrate a VM with storage? Any help or maybe a doc to look at is is much appreciated, thanks! | 21:26 |
| cfriesen | dansmith: how does the "track_instance_changes" thing work? I see the call to _init_instance_info(), is there a further "track ongoing changes" that isn't covered by a "if self.track_instance_changes" test? | 21:28 |
| mriedem | cfriesen: it's in the scheduler | 21:28 |
| dansmith | cfriesen: when you find out, let me know | 21:28 |
| mriedem | cfriesen: compute rpc casts things to the scheduler | 21:28 |
| mriedem | which then updates the list of tracked instances per HostState object, | 21:28 |
| mriedem | which is used by the affinity filters | 21:28 |
| *** slaweq has joined #openstack-nova | 21:28 | |
| cfriesen | mriedem: is that part unconditional? | 21:29 |
| mriedem | cfriesen: mind you, that's all computes in the deployment rpc casting to the scheduler (which is probably a single one to avoid collisions) once every minute by default | 21:29 |
| mriedem | cfriesen: which part? | 21:29 |
| mriedem | if you set CONF.filter_scheduler.track_instance_changes=False in nova.conf on the compute, it won't rpc cast to the scheduler | 21:29 |
| cfriesen | the ongoing update...because I only see self.track_instance_changes being tested in one place | 21:29 |
| mriedem | in the periodic or on tracked instance changes, like create or delete or move | 21:29 |
| *** thorst_afk has joined #openstack-nova | 21:30 | |
| openstackgerrit | Ed Leafe proposed openstack/nova-specs master: Amend spec for "Allow custom resource classes in flavor extra specs" https://review.openstack.org/481748 | 21:30 |
| edleafe | dansmith: ^^ | 21:30 |
| cfriesen | mriedem: ah, got it. I missed the self.send_instance_updates piece on the computes | 21:30 |
| mriedem | alkhodos_: there is some stuff in https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-configuring-migrations.rst and https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/compute-live-migration-usage.rst | 21:31 |
| mriedem | alkhodos_: google is probably your best bet here, | 21:31 |
| mriedem | alkhodos_: there are also summit videos that have tips and tricks and such | 21:31 |
| mriedem | alkhodos_: https://www.openstack.org/videos/search?search=live%20migration | 21:31 |
| *** shiyer has joined #openstack-nova | 21:31 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 21:31 |
| dansmith | edleafe: thanks | 21:32 |
| *** thorst_afk has quit IRC | 21:34 | |
| mriedem | so it looks like any functional test for shared storage providers is going to have to override the local_gb amount in the FakeDriver | 21:34 |
| mriedem | https://github.com/openstack/nova/blob/61159ed563463c096565fade370389014bde22dd/nova/virt/fake.py#L136 | 21:34 |
| alkhodos_ | mriedem: thanks a lot! | 21:35 |
| *** lpetrut has quit IRC | 21:37 | |
| *** gyee has quit IRC | 21:38 | |
| *** lyan has quit IRC | 21:43 | |
| *** annegentle has quit IRC | 21:45 | |
| *** thorst_afk has joined #openstack-nova | 21:50 | |
| *** lyan has joined #openstack-nova | 21:51 | |
| openstackgerrit | Merged openstack/nova master: doc: Remove dead files https://review.openstack.org/478470 | 21:54 |
| openstackgerrit | Merged openstack/nova master: doc: Rework README to reflect new doc URLs https://review.openstack.org/480074 | 21:54 |
| *** thorst_afk has quit IRC | 21:55 | |
| openstackgerrit | Merged openstack/nova master: doc: Start using oslo_config.sphinxext https://review.openstack.org/482961 | 21:55 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 21:56 |
| *** yamamoto has joined #openstack-nova | 21:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 21:58 |
| *** burt has quit IRC | 22:04 | |
| *** esberglu has quit IRC | 22:06 | |
| *** esberglu has joined #openstack-nova | 22:06 | |
| *** rajathagasthya has quit IRC | 22:07 | |
| *** Apoorva_ has joined #openstack-nova | 22:07 | |
| *** kylek3h has quit IRC | 22:08 | |
| *** annegentle has joined #openstack-nova | 22:09 | |
| *** esberglu has quit IRC | 22:10 | |
| *** Apoorva has quit IRC | 22:11 | |
| *** dfisher has quit IRC | 22:12 | |
| *** mmehan has quit IRC | 22:14 | |
| *** annegentle has quit IRC | 22:15 | |
| *** krtaylor has joined #openstack-nova | 22:17 | |
| openstackgerrit | Merged openstack/nova master: Add a caveat section about cellsv2 upcalls https://review.openstack.org/490612 | 22:19 |
| *** krtaylor has quit IRC | 22:23 | |
| *** edmondsw has quit IRC | 22:23 | |
| *** xyang1 has quit IRC | 22:25 | |
| *** acormier has quit IRC | 22:26 | |
| *** JoseMello has quit IRC | 22:26 | |
| *** edmondsw has joined #openstack-nova | 22:26 | |
| *** krtaylor has joined #openstack-nova | 22:27 | |
| *** krtaylor has quit IRC | 22:31 | |
| *** Apoorva_ has quit IRC | 22:31 | |
| *** edmondsw has quit IRC | 22:31 | |
| *** Apoorva has joined #openstack-nova | 22:31 | |
| *** Sukhdev_ has quit IRC | 22:34 | |
| *** felipemonteiro_ has quit IRC | 22:41 | |
| *** acormier has joined #openstack-nova | 22:41 | |
| *** Sukhdev_ has joined #openstack-nova | 22:43 | |
| *** Apoorva_ has joined #openstack-nova | 22:47 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_ksa_adapter() https://review.openstack.org/488137 | 22:47 |
| *** Apoorva has quit IRC | 22:47 | |
| efried | mordred Oboy, big changes, but I think you will like. ^^ | 22:47 |
| efried | Actually makes the code a lot smaller & simpler. | 22:48 |
| *** efried is now known as efried_zzz | 22:50 | |
| *** dave-mccowan has quit IRC | 22:51 | |
| *** slaweq has quit IRC | 22:51 | |
| *** acormier has quit IRC | 22:51 | |
| *** slaweq has joined #openstack-nova | 22:51 | |
| *** acormier has joined #openstack-nova | 22:51 | |
| *** acormier has quit IRC | 22:52 | |
| *** diablo_rojo has quit IRC | 22:54 | |
| *** slaweq has quit IRC | 22:56 | |
| *** sdague has quit IRC | 22:56 | |
| *** rcernin has quit IRC | 22:58 | |
| *** ijw has quit IRC | 23:04 | |
| *** ijw has joined #openstack-nova | 23:05 | |
| *** gbarros has joined #openstack-nova | 23:06 | |
| *** lyan has quit IRC | 23:08 | |
| *** ijw has quit IRC | 23:09 | |
| *** ijw has joined #openstack-nova | 23:14 | |
| *** ijw has quit IRC | 23:15 | |
| *** mriedem1 has joined #openstack-nova | 23:23 | |
| *** itlinux has quit IRC | 23:25 | |
| *** mriedem has quit IRC | 23:25 | |
| *** ijw has joined #openstack-nova | 23:25 | |
| *** thorst_afk has joined #openstack-nova | 23:26 | |
| *** jmlowe has joined #openstack-nova | 23:27 | |
| *** baoli has joined #openstack-nova | 23:27 | |
| *** baoli has quit IRC | 23:32 | |
| *** thorst_afk has quit IRC | 23:32 | |
| *** hamzy has joined #openstack-nova | 23:33 | |
| *** hongbin has quit IRC | 23:34 | |
| *** abalutoiu has quit IRC | 23:38 | |
| *** acormier has joined #openstack-nova | 23:43 | |
| *** tbachman has quit IRC | 23:47 | |
| *** acormier has quit IRC | 23:48 | |
| *** marst has quit IRC | 23:51 | |
| *** alkhodos_ has quit IRC | 23:53 | |
| *** edmondsw has joined #openstack-nova | 23:54 | |
| *** dtp has quit IRC | 23:56 | |
| *** edmondsw has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!