| *** gmann has quit IRC | 00:09 | |
| *** gmann has joined #openstack-nova | 00:09 | |
| *** hongbin has quit IRC | 00:15 | |
| *** hieulq_ has quit IRC | 00:20 | |
| *** trungnv has quit IRC | 00:20 | |
| *** edmondsw has joined #openstack-nova | 00:34 | |
| *** trungnv has joined #openstack-nova | 00:34 | |
| *** hieulq_ has joined #openstack-nova | 00:34 | |
| *** yingjun has joined #openstack-nova | 00:35 | |
| *** hieulq has joined #openstack-nova | 00:38 | |
| *** edmondsw has quit IRC | 00:38 | |
| *** yingjun has quit IRC | 00:40 | |
| *** thorst has joined #openstack-nova | 00:42 | |
| *** thorst has quit IRC | 00:42 | |
| *** vladikr has joined #openstack-nova | 00:44 | |
| *** zhurong has joined #openstack-nova | 00:48 | |
| *** zhurong has quit IRC | 00:49 | |
| *** baoli has joined #openstack-nova | 00:56 | |
| *** amotoki is now known as amotoki_away | 00:58 | |
| *** amotoki_away is now known as amotoki | 00:58 | |
| *** yingjun has joined #openstack-nova | 01:01 | |
| *** baoli has quit IRC | 01:01 | |
| *** iceyao has joined #openstack-nova | 01:03 | |
| *** Shunli has joined #openstack-nova | 01:04 | |
| *** phuongnh has joined #openstack-nova | 01:04 | |
| *** huanxie has joined #openstack-nova | 01:07 | |
| *** iceyao has quit IRC | 01:08 | |
| masber | good morning, I am trying to setup sr-iov on my openstack ocata environment | 01:09 |
|---|---|---|
| masber | this is the document I am following https://docs.openstack.org/ocata/networking-guide/config-sriov.html | 01:10 |
| *** vladikr has quit IRC | 01:10 | |
| *** acormier has joined #openstack-nova | 01:10 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle exception on adding secgroup https://review.openstack.org/465173 | 01:11 |
| masber | my question is, I need to whitelist the pci devices on nova.conf, I have a bonded network so can I do like this pci_passthrough_whitelist = { "devname": "ens1f*", "physical_network": "physnet1"}? | 01:11 |
| *** litao__ has joined #openstack-nova | 01:11 | |
| *** hongbin has joined #openstack-nova | 01:11 | |
| *** tuanluong has joined #openstack-nova | 01:15 | |
| *** jichen has joined #openstack-nova | 01:20 | |
| *** iceyao has joined #openstack-nova | 01:20 | |
| *** liujiong has joined #openstack-nova | 01:22 | |
| *** acormier has quit IRC | 01:27 | |
| *** acormier has joined #openstack-nova | 01:27 | |
| *** zhurong has joined #openstack-nova | 01:29 | |
| *** takashin has joined #openstack-nova | 01:29 | |
| *** zhurong has quit IRC | 01:33 | |
| *** gaoyuan has joined #openstack-nova | 01:33 | |
| *** gaoyuan has quit IRC | 01:36 | |
| *** gaoyuan has joined #openstack-nova | 01:36 | |
| *** zhurong has joined #openstack-nova | 01:37 | |
| *** clayton has quit IRC | 01:41 | |
| *** baoli has joined #openstack-nova | 01:42 | |
| *** zhurong has quit IRC | 01:47 | |
| *** sree has joined #openstack-nova | 01:51 | |
| *** clayton has joined #openstack-nova | 01:52 | |
| *** slaweq_ has joined #openstack-nova | 01:53 | |
| *** hieulq has quit IRC | 01:54 | |
| *** sean-k-m1 has joined #openstack-nova | 01:55 | |
| *** sean-k-mooney has quit IRC | 01:57 | |
| *** hieulq_ has quit IRC | 01:57 | |
| *** clayton has quit IRC | 01:57 | |
| *** trungnv has quit IRC | 01:58 | |
| *** slaweq_ has quit IRC | 01:58 | |
| *** phuongnh has quit IRC | 01:59 | |
| *** tuanluong has quit IRC | 01:59 | |
| *** acormier has quit IRC | 02:01 | |
| *** acormier has joined #openstack-nova | 02:02 | |
| *** catintheroof has joined #openstack-nova | 02:03 | |
| *** sree has quit IRC | 02:03 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593 | 02:05 |
| *** acormier has quit IRC | 02:06 | |
| *** gcb has joined #openstack-nova | 02:06 | |
| *** clayton has joined #openstack-nova | 02:07 | |
| *** catintheroof has quit IRC | 02:08 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use URIOpt https://review.openstack.org/479215 | 02:14 |
| *** vladikr has joined #openstack-nova | 02:18 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in server-security-groups https://review.openstack.org/476434 | 02:19 |
| *** thorst has joined #openstack-nova | 02:20 | |
| *** thorst has quit IRC | 02:20 | |
| *** hieulq has joined #openstack-nova | 02:21 | |
| *** edmondsw has joined #openstack-nova | 02:22 | |
| *** esberglu has joined #openstack-nova | 02:25 | |
| *** esberglu has quit IRC | 02:25 | |
| *** edmondsw has quit IRC | 02:26 | |
| *** thorst has joined #openstack-nova | 02:30 | |
| *** thorst has quit IRC | 02:35 | |
| *** armax has joined #openstack-nova | 02:46 | |
| *** yamamoto has joined #openstack-nova | 02:49 | |
| *** slaweq_ has joined #openstack-nova | 02:54 | |
| *** armax has quit IRC | 02:56 | |
| *** armax has joined #openstack-nova | 02:57 | |
| *** slaweq_ has quit IRC | 02:59 | |
| *** armax has quit IRC | 02:59 | |
| *** MasterOfBugs has quit IRC | 03:03 | |
| *** trungnv has joined #openstack-nova | 03:04 | |
| *** hieulq_ has joined #openstack-nova | 03:04 | |
| *** Kevin_Zheng has joined #openstack-nova | 03:14 | |
| *** esberglu has joined #openstack-nova | 03:19 | |
| *** itlinux has joined #openstack-nova | 03:20 | |
| *** baoli has quit IRC | 03:21 | |
| *** esberglu has quit IRC | 03:23 | |
| *** hongbin has quit IRC | 03:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 03:33 |
| *** hieulq has quit IRC | 03:37 | |
| *** iceyao has quit IRC | 03:45 | |
| *** udesale has joined #openstack-nova | 03:46 | |
| *** amotoki is now known as amotoki_away | 03:47 | |
| *** yamamoto has quit IRC | 03:50 | |
| *** yamamoto has joined #openstack-nova | 03:53 | |
| *** hongbin has joined #openstack-nova | 03:55 | |
| *** liujiong has quit IRC | 03:58 | |
| *** cNilesh has joined #openstack-nova | 03:59 | |
| *** iceyao has joined #openstack-nova | 04:03 | |
| *** iceyao has quit IRC | 04:07 | |
| *** hieulq has joined #openstack-nova | 04:08 | |
| *** kornica has joined #openstack-nova | 04:08 | |
| *** kornica has quit IRC | 04:09 | |
| *** edmondsw has joined #openstack-nova | 04:10 | |
| *** hieulq has quit IRC | 04:14 | |
| *** amotoki_away is now known as amotoki | 04:15 | |
| *** psachin has joined #openstack-nova | 04:15 | |
| *** edmondsw has quit IRC | 04:15 | |
| *** itlinux has quit IRC | 04:18 | |
| *** iceyao has joined #openstack-nova | 04:20 | |
| *** hongbin has quit IRC | 04:21 | |
| *** iceyao has quit IRC | 04:25 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 04:26 |
| *** Dinesh_Bhor has joined #openstack-nova | 04:26 | |
| *** cNilesh is now known as cNilesh|INTRV | 04:27 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:28 |
| *** thorst has joined #openstack-nova | 04:31 | |
| *** claudiub has joined #openstack-nova | 04:34 | |
| *** Nel1x has quit IRC | 04:35 | |
| *** thorst has quit IRC | 04:35 | |
| *** amotoki is now known as amotoki_away | 04:36 | |
| *** iceyao has joined #openstack-nova | 04:43 | |
| *** amotoki_away is now known as amotoki | 04:47 | |
| *** pewp has joined #openstack-nova | 04:48 | |
| *** huanxie has quit IRC | 04:48 | |
| *** pewp is now known as Guest6929 | 04:48 | |
| *** gcb has quit IRC | 04:48 | |
| *** gcb has joined #openstack-nova | 04:48 | |
| *** itlinux has joined #openstack-nova | 04:49 | |
| *** huanxie has joined #openstack-nova | 04:54 | |
| *** slaweq_ has joined #openstack-nova | 04:55 | |
| *** sapcc-bot has quit IRC | 04:56 | |
| *** sapcc-bot has joined #openstack-nova | 04:56 | |
| *** sree has joined #openstack-nova | 04:57 | |
| *** slaweq_ has quit IRC | 05:00 | |
| *** gcb has quit IRC | 05:06 | |
| *** gcb has joined #openstack-nova | 05:07 | |
| *** zsli_ has joined #openstack-nova | 05:12 | |
| *** itlinux has quit IRC | 05:13 | |
| *** Shunli has quit IRC | 05:14 | |
| *** hshiina has joined #openstack-nova | 05:19 | |
| *** itlinux has joined #openstack-nova | 05:20 | |
| *** adisky__ has joined #openstack-nova | 05:42 | |
| *** prateek has joined #openstack-nova | 05:44 | |
| *** ratailor has joined #openstack-nova | 05:47 | |
| *** moshele has joined #openstack-nova | 05:50 | |
| *** rcernin has joined #openstack-nova | 05:58 | |
| *** esberglu has joined #openstack-nova | 06:03 | |
| *** esberglu has quit IRC | 06:03 | |
| *** itlinux has quit IRC | 06:04 | |
| *** liujiong has joined #openstack-nova | 06:04 | |
| *** Guest6929 has quit IRC | 06:05 | |
| *** rcernin has quit IRC | 06:06 | |
| *** pewp has joined #openstack-nova | 06:08 | |
| *** pewp has quit IRC | 06:08 | |
| *** pewp has joined #openstack-nova | 06:09 | |
| *** pewp has quit IRC | 06:09 | |
| *** pewp has joined #openstack-nova | 06:10 | |
| *** pewp has quit IRC | 06:10 | |
| *** pewp has joined #openstack-nova | 06:10 | |
| *** pewp has quit IRC | 06:11 | |
| *** pewp has joined #openstack-nova | 06:11 | |
| *** pewp has quit IRC | 06:11 | |
| *** Oku_OS-away is now known as Oku_OS | 06:12 | |
| *** pewp has joined #openstack-nova | 06:12 | |
| *** pewp is now known as Guest51204 | 06:12 | |
| *** rcernin has joined #openstack-nova | 06:20 | |
| *** slaweq_ has joined #openstack-nova | 06:27 | |
| *** andreas_s has joined #openstack-nova | 06:29 | |
| *** alexchadin has joined #openstack-nova | 06:29 | |
| *** yingjun has quit IRC | 06:29 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | 06:30 |
| *** takashin has left #openstack-nova | 06:30 | |
| *** thorst has joined #openstack-nova | 06:32 | |
| *** slaweq_ has quit IRC | 06:32 | |
| *** zsli_ has quit IRC | 06:32 | |
| *** zsli_ has joined #openstack-nova | 06:33 | |
| *** thorst has quit IRC | 06:36 | |
| openstackgerrit | Andreas Scheuring proposed openstack/nova master: DO NOT MERGE! Dummy patch https://review.openstack.org/478186 | 06:37 |
| *** trinaths has joined #openstack-nova | 06:40 | |
| *** slaweq_ has joined #openstack-nova | 06:44 | |
| *** slaweq_ has quit IRC | 06:44 | |
| *** slaweq_ has joined #openstack-nova | 06:45 | |
| *** trinaths has quit IRC | 06:48 | |
| *** sshwarts has joined #openstack-nova | 06:48 | |
| *** markus_z has joined #openstack-nova | 06:51 | |
| *** huanxie has quit IRC | 06:53 | |
| *** trungnv has quit IRC | 06:54 | |
| *** iceyao has quit IRC | 06:55 | |
| *** iceyao has joined #openstack-nova | 06:57 | |
| *** pcaruana has joined #openstack-nova | 06:59 | |
| *** iceyao has quit IRC | 07:01 | |
| *** pcaruana has quit IRC | 07:03 | |
| *** pcaruana has joined #openstack-nova | 07:05 | |
| *** jpena|off is now known as jpena | 07:08 | |
| *** mlakat has quit IRC | 07:13 | |
| *** ralonsoh has joined #openstack-nova | 07:17 | |
| *** slaweq_ has quit IRC | 07:17 | |
| *** iceyao has joined #openstack-nova | 07:18 | |
| *** tesseract has joined #openstack-nova | 07:20 | |
| *** slaweq_ has joined #openstack-nova | 07:21 | |
| *** zhurong has joined #openstack-nova | 07:22 | |
| *** mlakat has joined #openstack-nova | 07:22 | |
| *** brault has joined #openstack-nova | 07:23 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741 | 07:24 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: update logic to configure device for scsi controller https://review.openstack.org/459740 | 07:24 |
| *** hferenc has joined #openstack-nova | 07:24 | |
| *** zsli_ has quit IRC | 07:24 | |
| *** zsli_ has joined #openstack-nova | 07:25 | |
| ralonsoh | sfinucan: hi!! An easy ball: https://review.openstack.org/#/c/466040/. Do you mind to take a look? Thanks! | 07:26 |
| ralonsoh | sfinucan: and if you feel generous (hehehe) this one: https://review.openstack.org/#/c/465861/ | 07:27 |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 07:27 |
| *** slaweq_ has quit IRC | 07:30 | |
| *** damien_r has joined #openstack-nova | 07:43 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 07:47 |
| *** openstackgerrit has quit IRC | 07:48 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** karthiks has joined #openstack-nova | 08:01 | |
| *** trungnv has joined #openstack-nova | 08:12 | |
| *** slaweq_ has joined #openstack-nova | 08:14 | |
| *** slaweq_ has quit IRC | 08:15 | |
| *** yassine has joined #openstack-nova | 08:16 | |
| *** slaweq_ has joined #openstack-nova | 08:17 | |
| *** huanxie has joined #openstack-nova | 08:19 | |
| ralonsoh | dansmith: hi. About https://review.openstack.org/#/c/449257/28/nova/objects/fields.py. If I replace this field with a new object, I'll need to create a new DB table to store this new object | 08:20 |
| ralonsoh | dansmith: I was trying to avoid it | 08:20 |
| *** bjolo has joined #openstack-nova | 08:23 | |
| *** lpetrut has joined #openstack-nova | 08:26 | |
| *** slaweq_ has quit IRC | 08:29 | |
| *** thorst has joined #openstack-nova | 08:33 | |
| *** phuongnh has joined #openstack-nova | 08:37 | |
| jichen | hi, folks, not attend PTG before, seems PTG didn't charge before, now it will charge 100$? | 08:37 |
| *** thorst has quit IRC | 08:38 | |
| *** karimb has joined #openstack-nova | 08:43 | |
| *** alexchadin has quit IRC | 08:46 | |
| sfinucan | jichen: That was the same last time | 08:48 |
| jichen | sfinucan: thanks, I didn't get budget last time so didn't pay attention :( | 08:48 |
| jichen | hopefully can make it this time... | 08:48 |
| *** avolkov has joined #openstack-nova | 08:50 | |
| *** openstackgerrit has joined #openstack-nova | 09:01 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Reduce (notification) test duplication https://review.openstack.org/391428 | 09:01 |
| sfinucan | ralonsoh: Both done. Cheers for the ping | 09:08 |
| *** udesale has quit IRC | 09:12 | |
| *** Guest51204 has quit IRC | 09:12 | |
| *** pewp has joined #openstack-nova | 09:15 | |
| *** pewp is now known as Guest21893 | 09:15 | |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** yamamoto has quit IRC | 09:20 | |
| *** yamamoto has joined #openstack-nova | 09:22 | |
| sfinucan | ralonsoh: If you want to fix this merge conflict I can resubmit https://review.openstack.org/466040 | 09:23 |
| ralonsoh | sfinucan: I'll resubmit this one in a minute | 09:24 |
| ralonsoh | sfinucan: thanks! | 09:24 |
| *** jichen has quit IRC | 09:30 | |
| *** zsli_ has quit IRC | 09:33 | |
| *** thorst has joined #openstack-nova | 09:34 | |
| *** amotoki is now known as amotoki_away | 09:36 | |
| *** thorst has quit IRC | 09:38 | |
| *** slaweq_ has joined #openstack-nova | 09:38 | |
| *** iceyao has quit IRC | 09:39 | |
| *** iceyao has joined #openstack-nova | 09:39 | |
| *** slaweq_ has quit IRC | 09:41 | |
| *** slaweq_ has joined #openstack-nova | 09:41 | |
| *** sambetts|afk is now known as sambetts | 09:41 | |
| *** esberglu has joined #openstack-nova | 09:41 | |
| *** esberglu has quit IRC | 09:41 | |
| *** mvk has quit IRC | 09:42 | |
| *** openstackgerrit has joined #openstack-nova | 09:43 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Set default for live_migration_scheme https://review.openstack.org/457716 | 09:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571 | 09:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | 09:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | 09:43 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: add traits support to get resource_providers with shared https://review.openstack.org/478464 | 09:44 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: return traits in the ProviderSummary object https://review.openstack.org/479766 | 09:44 |
| openstackgerrit | Béla Vancsics proposed openstack/nova master: Reduce (notification) test duplication https://review.openstack.org/391428 | 09:44 |
| openstackgerrit | NidhiMittalHada proposed openstack/nova master: Nova treats hostname as pattern https://review.openstack.org/474949 | 09:47 |
| *** iceyao has quit IRC | 09:52 | |
| *** slaweq_ has quit IRC | 10:01 | |
| *** yassine has quit IRC | 10:06 | |
| *** iceyao has joined #openstack-nova | 10:07 | |
| *** mvk has joined #openstack-nova | 10:10 | |
| *** iceyao has quit IRC | 10:12 | |
| *** dims has quit IRC | 10:14 | |
| *** dims has joined #openstack-nova | 10:15 | |
| *** liujiong has quit IRC | 10:15 | |
| *** sree has quit IRC | 10:19 | |
| *** sree has joined #openstack-nova | 10:19 | |
| *** gcb has quit IRC | 10:20 | |
| *** udesale has joined #openstack-nova | 10:21 | |
| *** mvk has quit IRC | 10:22 | |
| *** phuongnh has quit IRC | 10:23 | |
| *** sree has quit IRC | 10:24 | |
| *** trungnv has quit IRC | 10:25 | |
| *** hieulq_ has quit IRC | 10:25 | |
| *** tbachman has quit IRC | 10:25 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Use oslo.polcy DocumentedRuleDefault https://review.openstack.org/472634 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Update policy descriptions for base. https://review.openstack.org/472622 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'create_rule_default' https://review.openstack.org/479773 | 10:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 10:27 |
| *** fragatina has joined #openstack-nova | 10:28 | |
| *** fragatina has quit IRC | 10:28 | |
| *** fragatina has joined #openstack-nova | 10:29 | |
| *** zsli_ has joined #openstack-nova | 10:35 | |
| *** esberglu has joined #openstack-nova | 10:35 | |
| *** mvk has joined #openstack-nova | 10:35 | |
| *** gszasz has joined #openstack-nova | 10:38 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: return traits in the ProviderSummary object https://review.openstack.org/479766 | 10:38 |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: add traits to the 'GET /allocation_candidates' endpoint https://review.openstack.org/479776 | 10:38 |
| *** esberglu has quit IRC | 10:39 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 10:42 |
| *** yamamoto has quit IRC | 10:45 | |
| *** yamamoto has joined #openstack-nova | 10:46 | |
| *** yamamoto has quit IRC | 10:46 | |
| *** iceyao has joined #openstack-nova | 10:47 | |
| *** tbachman has joined #openstack-nova | 10:53 | |
| *** alexchadin has joined #openstack-nova | 10:59 | |
| *** aignatov has quit IRC | 11:03 | |
| *** fragatina has quit IRC | 11:04 | |
| *** lyarwood has quit IRC | 11:05 | |
| *** blaisebool has quit IRC | 11:05 | |
| *** tristanC has quit IRC | 11:05 | |
| *** lucasagomes is now known as lucas-hungry | 11:05 | |
| *** tristanC has joined #openstack-nova | 11:05 | |
| *** aignatov has joined #openstack-nova | 11:06 | |
| *** lyarwood has joined #openstack-nova | 11:06 | |
| *** kfarr has joined #openstack-nova | 11:12 | |
| *** psachin has quit IRC | 11:14 | |
| *** psachin has joined #openstack-nova | 11:15 | |
| *** trungnv has joined #openstack-nova | 11:15 | |
| *** hieulq has joined #openstack-nova | 11:16 | |
| *** slaweq_ has joined #openstack-nova | 11:17 | |
| *** slaweq_ has quit IRC | 11:18 | |
| *** Guest21893 has quit IRC | 11:18 | |
| *** yassine has joined #openstack-nova | 11:20 | |
| *** slaweq_ has joined #openstack-nova | 11:21 | |
| *** slaweq_ has quit IRC | 11:22 | |
| *** pewp has joined #openstack-nova | 11:22 | |
| *** pewp is now known as Guest74994 | 11:23 | |
| *** slaweq_ has joined #openstack-nova | 11:24 | |
| *** karimb has quit IRC | 11:25 | |
| *** slaweq_ has quit IRC | 11:25 | |
| *** jwcroppe has quit IRC | 11:27 | |
| *** sree has joined #openstack-nova | 11:29 | |
| *** lyan has joined #openstack-nova | 11:31 | |
| *** jwcroppe has joined #openstack-nova | 11:32 | |
| *** sree has quit IRC | 11:33 | |
| *** sree has joined #openstack-nova | 11:33 | |
| *** baoli has joined #openstack-nova | 11:33 | |
| *** thorst has joined #openstack-nova | 11:34 | |
| *** thorst has quit IRC | 11:39 | |
| *** slaweq_ has joined #openstack-nova | 11:40 | |
| *** slaweq_ has quit IRC | 11:40 | |
| *** yamamoto has joined #openstack-nova | 11:46 | |
| *** yamamoto has quit IRC | 11:50 | |
| *** yamamoto has joined #openstack-nova | 11:50 | |
| *** jaypipes has joined #openstack-nova | 11:53 | |
| *** yamamoto has quit IRC | 11:53 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 11:54 |
| *** slaweq_ has joined #openstack-nova | 11:56 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 11:56 |
| *** litao__ has quit IRC | 11:56 | |
| openstackgerrit | Luis Pigueiras proposed openstack/nova master: Handle keypair not found from metadata server using cells https://review.openstack.org/476122 | 12:00 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 12:01 |
| *** jpena is now known as jpena|lunch | 12:03 | |
| *** vvargaszte has quit IRC | 12:08 | |
| *** vvargaszte has joined #openstack-nova | 12:08 | |
| jaypipes | morning folks. | 12:09 |
| *** karimb has joined #openstack-nova | 12:10 | |
| *** yamamoto has joined #openstack-nova | 12:10 | |
| *** thorst has joined #openstack-nova | 12:11 | |
| *** slaweq_ has quit IRC | 12:11 | |
| *** slaweq_ has joined #openstack-nova | 12:12 | |
| *** lucas-hungry is now known as lucasagomes | 12:12 | |
| bauzas | jaypipes: morning | 12:14 |
| *** slaweq_ has quit IRC | 12:15 | |
| *** thorst has quit IRC | 12:16 | |
| *** bkopilov_ has quit IRC | 12:16 | |
| *** vvargaszte has quit IRC | 12:19 | |
| *** yamamoto has quit IRC | 12:19 | |
| *** markvoelker has joined #openstack-nova | 12:23 | |
| *** yamamoto has joined #openstack-nova | 12:23 | |
| bhagyashri_s | johnthetubaguy: Hi Could you please review patch : https://review.openstack.org/#/c/409644/ ? Addressed all review comments. | 12:25 |
| *** vvargaszte has joined #openstack-nova | 12:26 | |
| *** vvargaszte has quit IRC | 12:27 | |
| *** vvargaszte has joined #openstack-nova | 12:27 | |
| *** yamamoto has quit IRC | 12:28 | |
| *** tbachman has quit IRC | 12:31 | |
| *** catintheroof has joined #openstack-nova | 12:31 | |
| *** blaisebool has joined #openstack-nova | 12:34 | |
| *** blaisebool is now known as Guest46113 | 12:34 | |
| *** baoli has quit IRC | 12:34 | |
| *** ratailor has quit IRC | 12:35 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Automatically revert resize which fails on destination https://review.openstack.org/462521 | 12:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Allow wrapping of closures https://review.openstack.org/479801 | 12:35 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Ensure errors_out_migration errors out migration https://review.openstack.org/479802 | 12:35 |
| *** tbachman has joined #openstack-nova | 12:37 | |
| *** lpigueiras has quit IRC | 12:39 | |
| jaypipes | bauzas: https://review.openstack.org/#/q/topic:bug/1686116 | 12:40 |
| jaypipes | bauzas: that's a bug we've now run into a few times in our support team | 12:40 |
| bauzas | jaypipes: okay, looking it | 12:40 |
| jaypipes | bauzas: the code in general is tough to follow, but not because of sahid's stuff. just general ickiness with the BDM layer. | 12:40 |
| bauzas | FWIW, just had an internal mail server migration, now has 700 emails to review :( | 12:41 |
| jaypipes | bauzas: my support folks have applied it and said it fixes issues they've seen with virtio-scsi drives. | 12:41 |
| bauzas | jaypipes: okay, I trust you | 12:41 |
| jaypipes | bauzas: yeah, I hear ya :) good luck with your emails :) | 12:41 |
| *** udesale has quit IRC | 12:43 | |
| sfinucan | jaypipes: I know it's not the top priority for the cycle, but is there anything more I can do on nested-resource-providers stuff? | 12:46 |
| jaypipes | sfinucan: nope, not right now. reviews on https://review.openstack.org/#/q/topic:bug/1686116 would be great though | 12:47 |
| *** tbachman has quit IRC | 12:47 | |
| sfinucan | jaypipes: yup, sahid pinged me earlier. On the to-do list for today | 12:47 |
| *** tbachman has joined #openstack-nova | 12:49 | |
| jaypipes | sfinucan: cool, cheers :) | 12:52 |
| *** tbachman has quit IRC | 12:52 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Update policy descriptions for base. https://review.openstack.org/472622 | 12:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Consistent policies https://review.openstack.org/479774 | 12:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Remove 'create_rule_default' https://review.openstack.org/479773 | 12:54 |
| *** yamamoto has joined #openstack-nova | 12:59 | |
| *** iceyao has quit IRC | 12:59 | |
| *** baoli has joined #openstack-nova | 13:00 | |
| *** jpena|lunch is now known as jpena | 13:02 | |
| *** rcernin has quit IRC | 13:03 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | 13:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671 | 13:05 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | 13:05 |
| *** acormier has joined #openstack-nova | 13:06 | |
| *** rcernin has joined #openstack-nova | 13:07 | |
| *** mvk has quit IRC | 13:08 | |
| *** liverpooler has joined #openstack-nova | 13:13 | |
| openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | 13:14 |
| *** tbachman has joined #openstack-nova | 13:14 | |
| *** gcb has joined #openstack-nova | 13:15 | |
| *** esberglu has joined #openstack-nova | 13:20 | |
| *** esberglu has quit IRC | 13:20 | |
| *** esberglu has joined #openstack-nova | 13:20 | |
| *** zhurong has quit IRC | 13:21 | |
| *** ircuser-1 has joined #openstack-nova | 13:22 | |
| openstackgerrit | Kaitlin Farr proposed openstack/nova master: Remove deprecated keymgr code https://review.openstack.org/439855 | 13:22 |
| *** tbachman has quit IRC | 13:24 | |
| *** esberglu has quit IRC | 13:24 | |
| *** cNilesh has joined #openstack-nova | 13:25 | |
| *** tbachman has joined #openstack-nova | 13:26 | |
| *** acormier has quit IRC | 13:26 | |
| *** liverpooler has quit IRC | 13:27 | |
| *** liverpooler has joined #openstack-nova | 13:27 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: DO NOT MERGE: debug failed test https://review.openstack.org/479818 | 13:27 |
| *** cNilesh|INTRV has quit IRC | 13:28 | |
| *** yonglihe has quit IRC | 13:29 | |
| *** scottda has joined #openstack-nova | 13:30 | |
| *** iceyao has joined #openstack-nova | 13:31 | |
| sfinucan | jaypipes: Comments left on that but you might be able to answer of the top of your head: can you hotplug SCSI devices? | 13:31 |
| jaypipes | sfinucan: you mean virtual SCSI devices? | 13:32 |
| sfinucan | yup | 13:32 |
| sfinucan | maybe hotplug is the wrong term | 13:32 |
| jaypipes | sure, you can attach/detach scsi block devices. | 13:32 |
| sfinucan | OK, so my comment on that is whether always incrementing this unit parameter when attaching a scsi device is a good idea | 13:33 |
| sfinucan | I'd imagine if we attach/detach a few times on a long-running VM, we'd hit that 256 device limit? | 13:33 |
| sfinucan | ...but maybe it's a YAGNI kind of thing | 13:34 |
| jaypipes | sfinucan: I think the whole max(unit) logic prevents that, no? | 13:34 |
| *** mvk has joined #openstack-nova | 13:34 | |
| sfinucan | jaypipes: not really. If every device gets max(unit) + 1, then we could eventually hit 257, right? | 13:35 |
| *** iceyao has quit IRC | 13:36 | |
| sfinucan | once we've executed that many attach operations, that is | 13:36 |
| jaypipes | sfinucan: oh, I see what you mean... meh, not sure I'm too concerned about that. | 13:36 |
| *** zzzeek has quit IRC | 13:36 | |
| *** zzzeek has joined #openstack-nova | 13:36 | |
| sfinucan | jaypipes: Think it's a small enough edge case to ignore so? | 13:36 |
| jaypipes | sfinucan: yeah. considering rebooting the VM would "clear that out", right? | 13:37 |
| *** zsli_ has quit IRC | 13:37 | |
| sfinucan | jaypipes: I don't think so? The XML isn't rebuilt on a reboot, right? Only a resize/migrate? | 13:38 |
| *** cNilesh has quit IRC | 13:38 | |
| jaypipes | sfinucan: is it? I thought it was rebuilt on reboot... | 13:38 |
| * sfinucan goes to check | 13:38 | |
| sfinucan | jaypipes: Looks like the complete rebuild only happens if libvirt doesn't support 'reset' https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2422-L2424 | 13:42 |
| sfinucan | However, it is an edge case. I'll wait an see what sahid says and if he's OK with it, it can slide | 13:43 |
| jaypipes | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2428 | 13:44 |
| jaypipes | sfinucan: ^ | 13:44 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | 13:45 |
| sfinucan | jaypipes: The comments lie. Grr | 13:45 |
| *** iceyao has joined #openstack-nova | 13:47 | |
| sfinucan | That wouldn't be the case for soft_reboot though, I guess - I've yet to see any undefine calls yet https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2384-L2385 | 13:47 |
| jaypipes | sfinucan: no, I'm referring to a hard reboot. | 13:48 |
| sfinucan | Folks with +2 permissions: would appreciate reviews on this series today --> https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova | 13:53 |
| sfinucan | I want to get started on moving the guides from openstack-manuals in here this week | 13:53 |
| *** damien_r has left #openstack-nova | 13:53 | |
| *** hieulq_ has joined #openstack-nova | 13:57 | |
| *** alexchadin has quit IRC | 13:57 | |
| *** sree has quit IRC | 13:58 | |
| alex_xu | looks like we don't have scheduler meeting today | 14:02 |
| jaypipes | sfinucan: ++ | 14:02 |
| *** Guest74994 has quit IRC | 14:02 | |
| *** amotoki_away is now known as amotoki | 14:02 | |
| jaypipes | alex_xu: I'm always happy to chat about scheduler :) | 14:02 |
| jaypipes | alex_xu: but yeah, a number of US folks are out. | 14:03 |
| *** pewp has joined #openstack-nova | 14:03 | |
| alex_xu | jaypipes: ah, holiday? | 14:03 |
| *** efoley has joined #openstack-nova | 14:03 | |
| *** pewp is now known as Guest81918 | 14:03 | |
| *** fragatina has joined #openstack-nova | 14:03 | |
| jaypipes | alex_xu: July 4th tomorrow so lots of folks taking today off too. | 14:04 |
| alex_xu | jaypipes: cool, got it | 14:04 |
| *** kristian__ has joined #openstack-nova | 14:08 | |
| *** itlinux has joined #openstack-nova | 14:10 | |
| *** tidwellr has joined #openstack-nova | 14:13 | |
| *** fragatina has quit IRC | 14:15 | |
| *** thorst has joined #openstack-nova | 14:17 | |
| sfinucan | moshele: You about. I've a question about SR-IOV devices and PCI affinity | 14:18 |
| *** masber has quit IRC | 14:18 | |
| *** hieulq_ has quit IRC | 14:18 | |
| sfinucan | moshele: Namely, if I boot a VM with an SR-IOV device, then will that VM be forced to use CPUs from the same NUMA node as the actual PCI device. I thought this was the case | 14:19 |
| *** karimb has left #openstack-nova | 14:20 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 14:20 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | 14:20 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632 | 14:20 |
| openstackgerrit | dane-fichter proposed openstack/nova master: Mask password when logging connection_info during swap volume https://review.openstack.org/466533 | 14:21 |
| *** thorst has quit IRC | 14:21 | |
| moshele | sfinucan: this is what was implemented https://review.openstack.org/#/c/140290/ | 14:27 |
| *** hieulq_ has joined #openstack-nova | 14:27 | |
| *** mriedem has joined #openstack-nova | 14:29 | |
| mriedem | wubba wubba wubba | 14:29 |
| moshele | sfinucan: is that answer you question? | 14:30 |
| *** iceyao has quit IRC | 14:30 | |
| *** hieulq_ has quit IRC | 14:30 | |
| *** dave-mccowan has joined #openstack-nova | 14:30 | |
| mriedem | jaypipes: so you fixed the pep8 issue in https://review.openstack.org/#/c/475448/ but did you see the functional tests failing listing traits? | 14:31 |
| *** dave-mccowan has quit IRC | 14:31 | |
| mriedem | something going on with the lock or loading traits from the library it seems | 14:31 |
| mriedem | http://logs.openstack.org/48/475448/21/check/gate-nova-tox-functional-ubuntu-xenial/59c59c0/console.html#_2017-07-01_01_12_45_285206 | 14:31 |
| mriedem | i could recreate it locally, but couldn't figure out what was causing it | 14:31 |
| jaypipes | mriedem: I thought edleafe- had fixed that in a patch last week... lemme check. | 14:32 |
| *** rmart04 has joined #openstack-nova | 14:33 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 14:33 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | 14:33 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632 | 14:33 |
| jaypipes | mriedem: pulled in master ^^ | 14:33 |
| jaypipes | mriedem: I think that func test failure is fixed. | 14:33 |
| sfinucan | moshele: Kind of. That's for PCI devices, but would that include VF devices? | 14:33 |
| sfinucan | I guess | 14:33 |
| jaypipes | mriedem: will see in about an hour ;) | 14:33 |
| sfinucan | * I assume it would, but I'm not sure | 14:33 |
| moshele | sfinucan | 14:34 |
| moshele | moshele | 14:34 |
| mriedem | jaypipes: ok i can hunt for a recent change related to that | 14:34 |
| mriedem | unless edleafe- can point it out quick | 14:34 |
| moshele | sfinucan: yes it is the same code | 14:34 |
| jaypipes | mriedem: gimme a sec. | 14:34 |
| moshele | sfinucan: VF is eventually a pci device | 14:34 |
| jaypipes | mriedem: edl | 14:35 |
| jaypipes | mriedem: edl | 14:35 |
| jaypipes | oh ffs. | 14:35 |
| jaypipes | https://review.openstack.org/#/c/478564/ | 14:35 |
| jaypipes | mriedem: ^^ | 14:35 |
| jaypipes | `damn paste buffers. | 14:35 |
| sfinucan | moshele: Right, that's what I thought. However, I've an instance with an OVS-DPDK-managed PCI device on node 1, and it's booting on node 0 | 14:35 |
| sfinucan | Must be something to do with OVS-DPDK aspect of it, and how it's being attached to the instance | 14:35 |
| mriedem | jaypipes: oh nice | 14:35 |
| mriedem | whew | 14:35 |
| * sfinucan didn't even know ovs-dpdk could handle PCI devices now | 14:36 | |
| moshele | sfinucan: did you puth the numa filter ? and what libvirt verions are you using? | 14:36 |
| * jaypipes addresses hungry bear syndrome. | 14:36 | |
| jaypipes | back in a bit. | 14:36 |
| sfinucan | moshele: I'm not doing it personally, but I'd assume you'd need that filter for things to even boot. I will ask though | 14:37 |
| sfinucan | fwiw, all the logs are here. I'm guessing it's accessible outside RH? https://bugzilla.redhat.com/show_bug.cgi?id=1459543 | 14:38 |
| openstack | bugzilla.redhat.com bug 1459543 in openstack-nova "Virt NUMA placements on NUMA1 using flavor extra specs - places Virt guest on NUMA0" [Medium,Assigned] - Assigned to sfinucan | 14:38 |
| *** penick has joined #openstack-nova | 14:38 | |
| -openstackstatus- NOTICE: We have switched now all docs publishing jobs to new documentation builds. For details see dhellmann's email http://lists.openstack.org/pipermail/openstack-dev/2017-July/119221.html . For problems, join us on #openstack-doc | 14:38 | |
| mriedem | jaypipes: fix this up and i'll +W the change https://review.openstack.org/#/c/475448/23/nova/api/openstack/placement/rest_api_version_history.rst | 14:38 |
| sfinucan | mriedem: Per openstackstatus above, would appreciate reviews on this series to get those redirects working --> https://review.openstack.org/#/q/topic:doc-migration+project:openstack/nova | 14:40 |
| sfinucan | At least, I _think_ that means we need to do those now to fix those build | 14:40 |
| * sfinucan will be tackling os-vif and python-novaclient next | 14:40 | |
| mriedem | yeah probably | 14:41 |
| mriedem | sfinucan: sdague isn't around but you guys are going to need to sort out https://review.openstack.org/#/c/478468/ which is the bottom change in your series | 14:43 |
| sfinucan | mriedem: Damn. I think I can kick that to the end. The rest should be a-OK | 14:44 |
| moshele | mriedem: hi, now that https://review.openstack.org/#/c/478923/ is merge how do I update the requirement file? is it done automatically | 14:44 |
| *** hieulq_ has joined #openstack-nova | 14:44 | |
| sfinucan | moshele: Looks like they're using libvirt 2.0 but they don't have the NUMA filter on. I'll ping em. Thanks for the tip :) | 14:45 |
| sfinucan | AR me: Put note that NUMA affinity is supported in the SR-IOV admin guide | 14:45 |
| mriedem | moshele: you have to update the global-requirements file in the openstack/requirements repo | 14:45 |
| mriedem | moshele: this https://github.com/openstack/requirements/blob/master/global-requirements.txt#L161 | 14:46 |
| mriedem | and the related entry in upper-constraints in that same repo | 14:46 |
| mriedem | moshele: once it's merged in global-requirements it will get auto-synced to nova | 14:46 |
| *** iceyao has joined #openstack-nova | 14:47 | |
| moshele | sfinucan: np :) | 14:47 |
| moshele | mriedem: cool thanks | 14:48 |
| sfinucan | Yup, Googling for openstack-docs is not a thing atm :'( | 14:49 |
| *** iceyao has quit IRC | 14:51 | |
| *** hieulq_ has quit IRC | 14:52 | |
| *** sean-k-m1 is now known as sean-k-mooney | 14:53 | |
| *** tidwellr has quit IRC | 14:55 | |
| *** gyee has joined #openstack-nova | 14:58 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: scheduler uses allocation candidates https://review.openstack.org/476631 | 14:58 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | 14:58 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: WIP - placement: claim resources in scheduler https://review.openstack.org/476632 | 14:58 |
| jaypipes | mriedem: done. | 14:58 |
| *** thorst has joined #openstack-nova | 14:58 | |
| mriedem | thanks, +W | 15:00 |
| sean-k-mooney | jaypipes: o/ since your about just wanted to run something past you regrading the traits api. what would your opipion be on extending traits to allow tag that are key value pairs with stardadised keys? e.g. standardised HW_CPU_FREQUENCY key with integer or string value field | 15:02 |
| jaypipes | sean-k-mooney: I would be opposed to that. | 15:02 |
| sean-k-mooney | ok good to know :) | 15:02 |
| jaypipes | sean-k-mooney: what would the CPU frequency be used for w.r.t. scheduling? | 15:03 |
| mriedem | jaypipes: alex_xu: that reminds me, something that came up in shenzhen last week, and i'm sure there is a reason for it, but why don't we allow traits on resource provider aggregates? | 15:03 |
| mriedem | we have talked about a distance trait on an aggregate | 15:03 |
| mriedem | for affinity | 15:03 |
| jaypipes | mriedem: because the aggregates are simply groups of resource providers. they aren't things in and of themselves. | 15:03 |
| *** thorst has quit IRC | 15:04 | |
| mriedem | ok this question was coming from some guys doing baremetal, i'm not exactly sure what the use case was though, so i'd have to follow up | 15:04 |
| jaypipes | mriedem: you mean Mogan? :) | 15:04 |
| sean-k-mooney | jaypipes: for latence reasons i may not want to secheld to a node with a specific frequence below X but it more useful for adative power management. where i coudl declare teh maxium and minium frequence support by a specic cpu | 15:05 |
| jaypipes | sean-k-mooney: I think that kind of things is probably best left as a custom trait that you decorate a provider with. | 15:06 |
| jaypipes | sean-k-mooney: CUSTOM_HIGH_FREQUENCY_CPUS or similar. | 15:06 |
| *** rcernin has quit IRC | 15:07 | |
| sean-k-mooney | jaypipes: yep that is an option though i was just concerend that it might result in a lot of traits. that said they are just db row so not exeaclty the most expencive things in the world to create | 15:07 |
| *** tonyb has quit IRC | 15:08 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | 15:08 |
| sean-k-mooney | there are a few other resouces capablityes that i can think of such as disk iops that would be better mapped with a key and value then just a lable but lables can still be used | 15:09 |
| *** hongbin has joined #openstack-nova | 15:09 | |
| mriedem | jaypipes: yeah related | 15:11 |
| mriedem | jaypipes: but they had some other things for nova+ironic, which is what we run in the public cloud | 15:11 |
| mriedem | which leads me to other nova vm+bm questions for ops for johnthetubaguy | 15:11 |
| sean-k-mooney | mriedem: are the bm guys perhapss modeling multi node chassies as a aggreagate which might be why they are interested in traits on aggreages? if so nested resouce providers sould support that usecase | 15:13 |
| bauzas | jaypipes: could you please ACK'ing my comments here https://review.openstack.org/#/c/415920/15 so I could +W it ? | 15:13 |
| *** itlinux has quit IRC | 15:14 | |
| bauzas | FWIW, hectic day | 15:14 |
| * bauzas raises his fist to Gmail | 15:14 | |
| *** yassine has quit IRC | 15:15 | |
| *** hshiina has quit IRC | 15:18 | |
| -openstackstatus- NOTICE: "We're experiencing a few problems with the reorg on docs.openstack.org and are looking into these..." | 15:18 | |
| *** ChanServ changes topic to ""We're experiencing a few problems with the reorg on docs.openstack.org and are looking into these..."" | 15:18 | |
| *** slaweq has quit IRC | 15:19 | |
| *** slaweq has joined #openstack-nova | 15:20 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: DO NOT MERGE: debug failed test https://review.openstack.org/479818 | 15:20 |
| *** tbachman has quit IRC | 15:21 | |
| *** mkucia has joined #openstack-nova | 15:21 | |
| *** moshele has quit IRC | 15:22 | |
| *** slaweq has quit IRC | 15:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: doc: Create directory structure for docs migration https://review.openstack.org/479861 | 15:24 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479862 | 15:24 |
| sfinucan | jaypipes, sean-k-mooney, moshele: More docs fixes ^ | 15:25 |
| sfinucan | (for https://specs.openstack.org/openstack/docs-specs/specs/pike/os-manuals-migration.html) | 15:25 |
| sean-k-mooney | sfinucan: what is openstackdocstheme? | 15:26 |
| *** itlinux has joined #openstack-nova | 15:26 | |
| sfinucan | sean-k-mooney: the theme that docs.openstack.org uses | 15:26 |
| *** psachin has quit IRC | 15:27 | |
| sfinucan | We're supposed to use it for any official OpenStack projects now | 15:27 |
| sean-k-mooney | but is it a theme for sphinx that is maintined by the docs team for consitnecy across all offical docs? | 15:27 |
| sean-k-mooney | ok cool | 15:27 |
| mriedem | sean-k-mooney: i don't actually know what the use case is, | 15:28 |
| mriedem | but nested RPs might be an option, it did come up too for BM | 15:28 |
| mriedem | but not for multi-node chassis, | 15:28 |
| *** yamamoto has quit IRC | 15:28 | |
| mriedem | more for modeling NICs and Disks | 15:28 |
| mriedem | and putting traits on those like type and capabilities | 15:29 |
| sean-k-mooney | mriedem: i think blade servers are a good candiated for nested resouce providers as anti afinity at the chassis level may be needed to ha reasons | 15:29 |
| *** yamamoto has joined #openstack-nova | 15:31 | |
| mriedem | yes that was one example that came up too | 15:31 |
| mriedem | making sure you're not in the same rack as the other servers in the same group | 15:31 |
| sean-k-mooney | mriedem: that could be modeled with aggregates too but i guess it just depends on what you are trying to achive. | 15:32 |
| mriedem | we got talking about the distance stuff from the boston session which then goes down the road of do we standarize the distance enums? or do we leave that up to ops (which wouldn't be interoperable)? or do you define it at an SLA level, but is your SLA guaranteed within the same rack, row or DC? | 15:33 |
| *** chenying has quit IRC | 15:33 | |
| *** chenying has joined #openstack-nova | 15:34 | |
| *** Oku_OS is now known as Oku_OS-away | 15:34 | |
| sean-k-mooney | mriedem: ceph stiks a resonably good blance there. they provide standard values in the crush map for osd,host,rack... but you can just change the vaule if you want too. | 15:34 |
| sean-k-mooney | for example if you want to add a lay for chassis in between host and rack you can by editing the crushmap, honestly i dont know whay the best aproch would be but have a predifed set would be good for interop | 15:36 |
| *** yamamoto has quit IRC | 15:36 | |
| *** gcb has quit IRC | 15:36 | |
| *** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Pike_Release_Schedule" | 15:37 | |
| -openstackstatus- NOTICE: "docs.openstack.org is working fine again, due to move of new location, each repo needs to merge one change to appear on docs.o.o" | 15:37 | |
| mdbooth | sfinucan: Here's a fun patch if you've got time: https://review.openstack.org/#/c/462521/ . It has 2 dependencies which are stacked above it. | 15:42 |
| mdbooth | The idea is to automatically kick off a revert if it fails on the destination. It's a bit tricky basically because of instance state, migration state, and quotas. | 15:43 |
| mdbooth | It's obviously a user-friendly feature, but it's actually intended as a workaround for a bug we can't easily fix right now due to our datamodel. | 15:44 |
| mdbooth | Bug details are in the bug link, but basically if your resize fails on the destination node, your instance is using a deleted glance image, and you don't revert within 24 hours, it'll delete the only remaining copy of your image. At this point your instance is unrecoverable. | 15:45 |
| mdbooth | Ideally we would have the image cache not delete the image in this case, but our datamodel means it's not possible to identify an instance which is awaiting confirmation on the destination. | 15:46 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081 | 15:47 |
| *** crushil_ has joined #openstack-nova | 15:47 | |
| *** AJaeger has joined #openstack-nova | 15:49 | |
| AJaeger | mriedem, or any cores around? | 15:49 |
| AJaeger | Could you please abandon this change: https://review.openstack.org/#/c/479657/ - it will never build and is wrong | 15:49 |
| AJaeger | Hangs currently in zuul... | 15:49 |
| mriedem | AJaeger: i don't see the abandon button | 15:50 |
| AJaeger | ok, then I'll ask infra-root... | 15:50 |
| fungi | that's a fun one | 15:51 |
| fungi | looking now | 15:51 |
| AJaeger | fungi, you're everywhere ;) Ignore my ping on #openstack-infra then :-) - and thanks | 15:52 |
| *** andreas_s has quit IRC | 15:52 | |
| *** tbachman has joined #openstack-nova | 15:53 | |
| *** lucasagomes is now known as lucas-afk | 15:55 | |
| fungi | okay, abandoned | 15:55 |
| AJaeger | thanks | 15:56 |
| *** AJaeger has left #openstack-nova | 15:56 | |
| *** brad[] has quit IRC | 15:58 | |
| *** slaweq has joined #openstack-nova | 15:59 | |
| *** tuanla_fujitsu has joined #openstack-nova | 16:00 | |
| *** slaweq has quit IRC | 16:00 | |
| sfinucan | mdbooth: Ooh, that's a tricky one | 16:00 |
| sfinucan | Thanks for the context | 16:00 |
| *** slaweq has joined #openstack-nova | 16:00 | |
| *** tuanla__ has joined #openstack-nova | 16:01 | |
| *** crushil_ has quit IRC | 16:01 | |
| *** crushil_ has joined #openstack-nova | 16:01 | |
| mriedem | melwitt: do you know if james is also serving VMs in his cloud or just baremetal? | 16:04 |
| bauzas | mriedem: AFAIK, mel is on patriotic PTO today | 16:06 |
| *** crushil_ has quit IRC | 16:06 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: DO NOT MERGE: debug failed test https://review.openstack.org/479818 | 16:06 |
| jaypipes | bauzas: responded. | 16:08 |
| mriedem | bauzas: ok | 16:08 |
| jaypipes | bauzas: there's no hurry on the n-r-p series, though. | 16:08 |
| jaypipes | bauzas: allocation candidates is the more important one at this point. | 16:08 |
| bauzas | jaypipes: I know, but I think your change is fine by me | 16:08 |
| bauzas | jaypipes: sure | 16:08 |
| mriedem | bauzas: can you go over this one? https://review.openstack.org/#/c/476631/15 | 16:08 |
| mriedem | is dan out today too? | 16:08 |
| mriedem | i.e. is today a red hat holiday? | 16:09 |
| bauzas | mriedem: dan too | 16:09 |
| bauzas | mriedem: dunno | 16:09 |
| mriedem | most people make it a long weekend | 16:09 |
| bauzas | mriedem: pssst, I'm not an US folk :p | 16:09 |
| mriedem | really? | 16:09 |
| bauzas | even if we have elephants having trumps, any of them are our president :p | 16:10 |
| bauzas | anyway, looking at your change | 16:10 |
| mriedem | our child care provider is closed today so i'm hosting a play date and working | 16:11 |
| bauzas | heh | 16:12 |
| *** links has joined #openstack-nova | 16:13 | |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 16:15 |
| *** acormier has joined #openstack-nova | 16:16 | |
| *** amotoki is now known as amotoki_away | 16:17 | |
| *** hongbin_ has joined #openstack-nova | 16:19 | |
| *** sheel has joined #openstack-nova | 16:19 | |
| *** hongbin has quit IRC | 16:21 | |
| *** slaweq has quit IRC | 16:21 | |
| *** moshele has joined #openstack-nova | 16:22 | |
| *** itlinux has quit IRC | 16:22 | |
| *** junbo has quit IRC | 16:22 | |
| *** rtjure has quit IRC | 16:23 | |
| *** tesseract has quit IRC | 16:25 | |
| *** acormier has quit IRC | 16:26 | |
| *** rtjure has joined #openstack-nova | 16:27 | |
| *** junbo has joined #openstack-nova | 16:28 | |
| *** links has quit IRC | 16:29 | |
| *** rajathagasthya has joined #openstack-nova | 16:29 | |
| *** sshwarts has quit IRC | 16:31 | |
| sfinucan | mdbooth: IMA I'm pretty confused by https://review.openstack.org/#/c/479802/ | 16:31 |
| mdbooth | sfinucan: What specifically? | 16:32 |
| sfinucan | I left comments in the review, but tl;dr why's the scope change necessary? | 16:33 |
| sfinucan | The only functional change I can see is that you removed the status check | 16:33 |
| *** databus23_ has quit IRC | 16:33 | |
| *** sapcc-bot has quit IRC | 16:33 | |
| *** moshele has quit IRC | 16:33 | |
| *** yamamoto has joined #openstack-nova | 16:33 | |
| mdbooth | sfinucan: Actually in that one specifically the benefit is marginal | 16:34 |
| *** databus23_ has joined #openstack-nova | 16:34 | |
| *** sapcc-bot has joined #openstack-nova | 16:34 | |
| mdbooth | sfinucan: The change is that everything after the instance is 'done' is outside the scope of the context manager | 16:35 |
| *** pcaruana has quit IRC | 16:35 | |
| mdbooth | Although you could argue that quotas should be in... | 16:35 |
| mdbooth | Yeah, that one could probably reasonably go | 16:35 |
| *** jpena is now known as jpena|off | 16:35 | |
| mriedem | mgagne: let me know if you have any questions about the instance action thing i'm talking about in https://review.openstack.org/#/c/454322/ | 16:36 |
| mdbooth | However, resize_instance is where it's at | 16:36 |
| mdbooth | sfinucan: The issue in resize_instance is that you've cast the control flow to another host, but after that's happened you've still got this rogue context manager which might yet futz with your state | 16:36 |
| mdbooth | There are no circumstances where you want that to happen | 16:36 |
| sfinucan | mdbooth: wait, are you still referring to that patch or the next one in the series? | 16:37 |
| mdbooth | sfinucan: The only you just posted | 16:37 |
| mdbooth | errors_out_migration | 16:37 |
| mdbooth | And that's directly relevant to the following patch, and why it's a dep | 16:38 |
| sfinucan | ah, gotcha | 16:38 |
| sfinucan | so 'finish_revert_resize' is the marginal change | 16:38 |
| sfinucan | but 'resize_instance' is not | 16:38 |
| mdbooth | Yeah, I could potentially ditch that bit of the diff | 16:38 |
| sfinucan | yeah, you might drop that or at least split it. Just lets me reason about it a little easier | 16:39 |
| *** yamamoto has quit IRC | 16:39 | |
| mdbooth | The important one is resize_instance, because it does a cast | 16:39 |
| mdbooth | So allowing the context to continue is a race | 16:40 |
| sfinucan | mdbooth: You're referring to the call to finish_resize, yes? | 16:40 |
| mdbooth | sfinucan: Yes | 16:40 |
| sfinucan | OK, that clarifies things. Thanks for the explanation | 16:41 |
| sfinucan | If you can split those out/drop the less necessary diffs, I'm good to sign off on that | 16:41 |
| * sfinucan knocks off for the evening | 16:43 | |
| mdbooth | sfinucan: Thanks for looking at that. | 16:43 |
| sfinucan | no problem at al | 16:43 |
| sfinucan | *all | 16:43 |
| sfinucan | *atal atal | 16:43 |
| mdbooth | sfinucan: I've just quickly replied to 1 comment, btw. The scope change is actually related, even if it's not immediately obvious why. | 16:44 |
| * mdbooth might look harder at his commit message too | 16:44 | |
| openstackgerrit | Merged openstack/nova master: pci: add uuid field to PciDevice object https://review.openstack.org/469147 | 16:44 |
| *** edmondsw has joined #openstack-nova | 16:46 | |
| *** rmart04 has quit IRC | 16:47 | |
| *** hemna is now known as _pewp_ | 16:50 | |
| *** _pewp_ is now known as hemna | 16:50 | |
| *** edmondsw has quit IRC | 16:51 | |
| *** Guest81918 has quit IRC | 16:51 | |
| *** penick has quit IRC | 16:51 | |
| *** _pewp_ has joined #openstack-nova | 16:52 | |
| *** crushil_ has joined #openstack-nova | 16:53 | |
| *** crushil_ has quit IRC | 16:55 | |
| *** trinaths has joined #openstack-nova | 16:56 | |
| *** trinaths has quit IRC | 16:56 | |
| *** crushil_ has joined #openstack-nova | 16:56 | |
| *** esberglu has joined #openstack-nova | 16:57 | |
| *** esberglu has quit IRC | 16:58 | |
| *** esberglu has joined #openstack-nova | 16:58 | |
| *** esberglu has quit IRC | 16:58 | |
| *** thorst has joined #openstack-nova | 16:58 | |
| *** thorst has quit IRC | 16:58 | |
| *** _pewp_ has quit IRC | 16:59 | |
| *** hemna is now known as _pewp_ | 16:59 | |
| *** _pewp_ is now known as hemna | 16:59 | |
| *** markus_z has quit IRC | 17:00 | |
| *** _pewp_ has joined #openstack-nova | 17:00 | |
| *** tuanla_fujitsu has quit IRC | 17:02 | |
| *** tuanla__ has quit IRC | 17:02 | |
| *** tbachman has quit IRC | 17:02 | |
| *** ralonsoh has quit IRC | 17:03 | |
| claudiub | mriedem: hello. i have a question. so there is a patch that basically sanitizes the instance object in the InstanceMetadata ( https://review.openstack.org/#/c/478991/3/nova/api/metadata/base.py ), which will get rid of the instance's context, which will make unset attributes no longer loadable. Do we have an alternative solution to this, rather than touching the lazy-loading attributes? | 17:10 |
| claudiub | IMO, that can be very error-prone. | 17:10 |
| mriedem | claudiub: i wasn't aware of that change, but i remember that we hit some issues with metadata api with multiple cells so fail to lazy-load correctly anyway; the commit message says this isn't the long-term strategic solution, | 17:15 |
| mriedem | so i think long-term we want to whitelist what we store and cache | 17:16 |
| mriedem | because we've had other bugs where we can't cache the instance object using things like memcache because it uses pickle and things in there don't pickle correctly, like the sqlalchemy db connection stored in the context, as noted in the commit message | 17:16 |
| mriedem | yeah that's what the bug was, i remember this bug https://bugs.launchpad.net/nova/+bug/1694666 | 17:16 |
| openstack | Launchpad bug 1694666 in OpenStack Compute (nova) "metadata service PicklingError: Can't pickle <class 'sqlalchemy.orm.session.Session'> when using memcached" [Medium,Fix released] - Assigned to Dan Smith (danms) | 17:16 |
| claudiub | well, can't the db connection be recreated after the pickle? | 17:17 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: Hyper-V: fix live migration with CSVs https://review.openstack.org/380448 | 17:18 |
| mriedem | claudiub: maybe, probably depends on what that's needed for | 17:18 |
| mriedem | claudiub: i'm assuming you guys have extra fields on the instance object for hyperv and you need to lazy-load stuff for the metadata api? | 17:18 |
| *** crushil_ has quit IRC | 17:19 | |
| *** tonyb has joined #openstack-nova | 17:19 | |
| claudiub | yeah, we need the device_metadata field when we create the instance config drive, but that's not really a niche field. it's a basic field needed for the config drive | 17:19 |
| *** brad[] has joined #openstack-nova | 17:19 | |
| mriedem | hmm, have you talked to artom? | 17:20 |
| mriedem | because i'd think libvirt needs the same thing | 17:20 |
| *** lyan has quit IRC | 17:21 | |
| claudiub | mriedem: so, the reason why it worked for libvirt is because libvirt always touches that instance.device_metadata field, while we only set it whenever we have device tags to set in it. | 17:21 |
| mriedem | ah | 17:22 |
| mriedem | claudiub: have you reported a bug? | 17:22 |
| claudiub | while we can also always touch it, this isn't a driver specific issue, we shouldn't have to touch it | 17:22 |
| mriedem | sure, | 17:22 |
| mriedem | i think it's an oversight in dan's patch | 17:22 |
| mriedem | i left a comment in there | 17:22 |
| claudiub | well, i think we can simply reopen the same bug, and post a comment on launchpad about the new issue. | 17:23 |
| mriedem | claudiub: it should be a new bug | 17:23 |
| mriedem | otherwise LP tracking gets confusing | 17:23 |
| claudiub | i see. ok then. i'll open up a new bug | 17:24 |
| *** lyan has joined #openstack-nova | 17:24 | |
| mriedem | it's also an unfortunate change in another way, in that obj_load_attr for these will pull the entire instance for each entry and join on another table, it would be nicer to do those all at once with expected_attrs, | 17:24 |
| mriedem | but maybe don't mix those issues together | 17:25 |
| mriedem | it could be optimized for sure though | 17:25 |
| claudiub | yeah, that would be nicer. | 17:25 |
| claudiub | mriedem: also, regarding hyper-v generation 2 vms CI testing, it is in the plan, but the main issue is that we have to test both generation 1 and generation 2 VMs, as they are both currently supported. While more and more OSes are supported on generation 2, it's still not all. | 17:27 |
| *** sambetts is now known as sambetts|afk | 17:28 | |
| claudiub | and the way the VMs are built is quite different too, especially when it comes to attaching volumes / disks, setting boot order, nic hot plugging, and other new features. | 17:30 |
| *** rajathagasthya has quit IRC | 17:31 | |
| claudiub | anyways. planning to have some coverage in the ci for gen 2 by the end of pike. | 17:31 |
| mriedem | claudiub: if the majority of users are gen2, then i'd just run the ci on gen2 | 17:36 |
| mriedem | we could always put something in the docs that since pike the hyperv CI is running against gen2 and that's the recommended configuration | 17:36 |
| *** rajathagasthya has joined #openstack-nova | 17:45 | |
| *** dave-mccowan has joined #openstack-nova | 17:48 | |
| *** acormier has joined #openstack-nova | 17:50 | |
| *** tbachman has joined #openstack-nova | 17:50 | |
| *** esberglu has joined #openstack-nova | 17:51 | |
| *** esberglu has quit IRC | 17:55 | |
| *** slaweq has joined #openstack-nova | 17:58 | |
| openstackgerrit | Merged openstack/nova master: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | 18:03 |
| *** hongbin_ has quit IRC | 18:06 | |
| *** thorst has joined #openstack-nova | 18:10 | |
| *** jangutter has joined #openstack-nova | 18:12 | |
| openstackgerrit | Merged openstack/nova master: placement: support GET /allocation_candidates https://review.openstack.org/475448 | 18:15 |
| *** thorst has quit IRC | 18:15 | |
| *** gjayavelu has joined #openstack-nova | 18:19 | |
| *** thorst has joined #openstack-nova | 18:23 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: DO NOT MERGE: debug failed test https://review.openstack.org/479818 | 18:26 |
| *** lpetrut has quit IRC | 18:26 | |
| *** thorst has quit IRC | 18:26 | |
| *** kristian__ has quit IRC | 18:27 | |
| *** rajathagasthya has quit IRC | 18:28 | |
| *** sheel has quit IRC | 18:28 | |
| *** slaweq has quit IRC | 18:34 | |
| *** edmondsw has joined #openstack-nova | 18:35 | |
| *** slaweq has joined #openstack-nova | 18:36 | |
| *** kristian__ has joined #openstack-nova | 18:36 | |
| *** efoley has quit IRC | 18:37 | |
| *** acormier has quit IRC | 18:38 | |
| *** edmondsw has quit IRC | 18:39 | |
| *** slaweq has quit IRC | 18:40 | |
| *** kristian__ has quit IRC | 18:42 | |
| *** MrWatson is now known as NostawRm | 18:43 | |
| *** moshele has joined #openstack-nova | 18:43 | |
| openstackgerrit | Merged openstack/nova master: libvirt: update logic to configure device for scsi controller https://review.openstack.org/459740 | 18:45 |
| openstackgerrit | Merged openstack/nova master: libvirt: update the logic to configure volume with scsi controller https://review.openstack.org/459741 | 18:46 |
| mriedem | jaypipes: i went through https://review.openstack.org/#/c/476631/ | 18:47 |
| mriedem | jaypipes: i think we're going to have to adjust that a bit to not call placement if using the caching scheduler | 18:47 |
| jaypipes | mriedem: at some point we need to bite that bullet. | 18:48 |
| *** rajathagasthya has joined #openstack-nova | 18:49 | |
| *** moshele has quit IRC | 18:50 | |
| mriedem | jaypipes: sure, but it's easy to not do it in this change | 18:53 |
| mriedem | put a flag on the scheduler driver that the host manager can check | 18:53 |
| mriedem | *scheduler manager | 18:53 |
| jaypipes | mriedem: back shortly. need to go pick the pugs up from the groomers. | 18:54 |
| mriedem | oolala | 18:54 |
| *** gszasz has quit IRC | 18:55 | |
| *** wasmum has quit IRC | 19:03 | |
| *** efried_backJuly5 has quit IRC | 19:03 | |
| *** liverpooler has quit IRC | 19:04 | |
| *** slaweq has joined #openstack-nova | 19:05 | |
| *** slaweq has quit IRC | 19:05 | |
| *** slaweq has joined #openstack-nova | 19:07 | |
| *** slaweq has quit IRC | 19:12 | |
| *** efried_backJuly5 has joined #openstack-nova | 19:13 | |
| *** kristian__ has joined #openstack-nova | 19:14 | |
| *** MasterOfBugs has joined #openstack-nova | 19:18 | |
| *** MasterOfBugs has quit IRC | 19:18 | |
| *** kristian__ has quit IRC | 19:18 | |
| *** MasterOfBugs has joined #openstack-nova | 19:19 | |
| *** wasmum has joined #openstack-nova | 19:20 | |
| *** lpetrut has joined #openstack-nova | 19:21 | |
| *** efoley has joined #openstack-nova | 19:22 | |
| *** wasmum has quit IRC | 19:25 | |
| jaypipes | mriedem: nothin' better than the smell of a sweet-smellin' grumble of freshly-groomed pugs. | 19:32 |
| mriedem | dry or wet? | 19:33 |
| jaypipes | dry of course. :) | 19:34 |
| mriedem | good | 19:35 |
| *** kristian__ has joined #openstack-nova | 19:35 | |
| *** jbernard has quit IRC | 19:37 | |
| *** jbernard has joined #openstack-nova | 19:42 | |
| *** wasmum has joined #openstack-nova | 19:42 | |
| *** markvoelker has quit IRC | 19:45 | |
| *** fragatina has joined #openstack-nova | 19:45 | |
| jangutter | Due to some scary flu, external deadlines and various conjunctions I was out of OpenStack action for about 2 months... | 19:46 |
| jangutter | And boy, did the nova and os-vif bits that I'm interested change! | 19:46 |
| *** wasmum has quit IRC | 19:47 | |
| *** tidwellr has joined #openstack-nova | 19:47 | |
| *** markvoelker has joined #openstack-nova | 19:47 | |
| jaypipes | jangutter: glad you are feeling better! | 19:47 |
| jangutter | My question is to do with the new OVS_ACCELERATION vif that moshele contributed... I'm not seeing where the PCI devices get allocated. | 19:47 |
| *** zzzeek has quit IRC | 19:48 | |
| jangutter | Lemme grab the patch set. | 19:48 |
| *** zzzeek has joined #openstack-nova | 19:48 | |
| jaypipes | jangutter: PCI devices do not get allocated by os-vif. They are allocated by the resource tracker when a launch instance request is received that includes a PCI device request. | 19:48 |
| jangutter | Yep, that I understand. | 19:49 |
| *** slaweq has joined #openstack-nova | 19:49 | |
| jaypipes | jangutter: the resource tracker has a PCI device manager inside it, and it's that that does the actual "allocation" of a PCI device to a guest | 19:49 |
| jangutter | But does it just pick _any_ available PCI address? | 19:49 |
| *** kristian__ has quit IRC | 19:49 | |
| jangutter | ( https://review.openstack.org/#/q/topic:ovs_acc ) | 19:50 |
| jaypipes | jangutter: it picks any available PCI address that matches the PCI device request that is received from the conductor. | 19:50 |
| jaypipes | jangutter: are you wondering who creates that PCI device request? :) | 19:50 |
| *** kfarr has quit IRC | 19:51 | |
| jangutter | Yep, remember, I had the brain-dead idea of putting it into neutron-api bits. | 19:51 |
| jaypipes | jangutter: well, there's still quite a bit of Neutron API that is used to construct the PCI device request. | 19:51 |
| jaypipes | jangutter: and that existed before your idea ;) | 19:52 |
| jaypipes | jangutter: what moshele's patch did on the os-vif side was simply create a specialized VIF type class for the "new" OVS_ACCELERATED vif type. That vif type is something that still needs to be created in Neutron and supplied to the nova boot request by passing in a port_id value that corresponds to the pre-created Neutron port that has the port binding information indicating it's an "OVS_ACCELERATED" port. | 19:53 |
| jangutter | Yep, that I got. | 19:54 |
| *** thorst has joined #openstack-nova | 19:54 | |
| *** thorst has quit IRC | 19:54 | |
| jangutter | Very similar to what happened in our patch: but the OVS_ACCELERATED vif is new. | 19:54 |
| jaypipes | jangutter: new in Neutron, yeah? | 19:55 |
| jangutter | Oh, hang on. Does it auto-allocate because of VIFHostDevice ? | 19:57 |
| jaypipes | jangutter: what do you mean by "auto-allocate"? | 19:57 |
| jangutter | I'm looking for where the pci_request is made -> it's still at create_pci_requests_for_sriov_ports? | 19:58 |
| *** kristian__ has joined #openstack-nova | 20:00 | |
| *** acormier has joined #openstack-nova | 20:01 | |
| jaypipes | jangutter: yep | 20:01 |
| *** jamesden_ has joined #openstack-nova | 20:04 | |
| *** wasmum has joined #openstack-nova | 20:04 | |
| *** jamesdenton has quit IRC | 20:05 | |
| *** fragatina has quit IRC | 20:06 | |
| *** wasmum has quit IRC | 20:10 | |
| jangutter | jaypipes: OK, that's weird. It must be right before my eyes, but I can't see any way that https://review.openstack.org/#/c/398265/ triggers a pci_request. | 20:10 |
| *** hongbin has joined #openstack-nova | 20:11 | |
| *** vladikr has quit IRC | 20:13 | |
| jangutter | jaypipes: Aaah, sorry, my addled mind has finally caught up: if vnic_type in network_model.VNIC_TYPES_SRIOV: | 20:14 |
| jaypipes | correct | 20:15 |
| jangutter | OK, now that makes sense. It picks up the PCI device from phynet_name | 20:18 |
| *** cfriesen has joined #openstack-nova | 20:18 | |
| *** wasmum has joined #openstack-nova | 20:19 | |
| *** avolkov has quit IRC | 20:21 | |
| *** wasmum has quit IRC | 20:24 | |
| *** tidwellr has quit IRC | 20:27 | |
| *** _pewp_ has quit IRC | 20:27 | |
| *** tidwellr has joined #openstack-nova | 20:27 | |
| *** efoley has quit IRC | 20:28 | |
| *** _pewp_ has joined #openstack-nova | 20:30 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils https://review.openstack.org/479949 | 20:32 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: [WIP] Implement certificate_utils https://review.openstack.org/479949 | 20:34 |
| *** esberglu has joined #openstack-nova | 20:36 | |
| *** esberglu has quit IRC | 20:36 | |
| ildikov | mriedem: welcome back :) | 20:39 |
| mriedem | thanks - i never really left | 20:40 |
| ildikov | mriedem: I was wondering whether you could review the swap volume patch by any chance, so while you celebrate freedom I can be the slave of fixing review comments? :) | 20:40 |
| ildikov | mriedem: I know, but still :) | 20:41 |
| mriedem | wow | 20:42 |
| mriedem | maybe, i need to wrap something up first and then i'm out for a bit | 20:42 |
| ildikov | I can keep myself busy with attach that's true, but would rather fix swap hoping it'll be just a long and not endless list | 20:44 |
| ildikov | was just an idea :) | 20:44 |
| mriedem | ildikov: did anyone get started on the grenade test changes last week? | 20:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle uuids in os-hypervisors API https://review.openstack.org/471899 | 20:55 |
| mriedem | to attach on the old side and detach on the new side? | 20:55 |
| *** lyan has quit IRC | 20:56 | |
| ildikov | mriedem: stvnoyes1 is looking into Grenade, he had some issues with setting it up, but he was having progress last week | 20:56 |
| mriedem | ok | 20:56 |
| mriedem | i'm going to run out for a bit | 20:56 |
| *** yamahata has joined #openstack-nova | 21:06 | |
| *** dave-mccowan has quit IRC | 21:09 | |
| *** lpetrut has quit IRC | 21:25 | |
| *** esberglu has joined #openstack-nova | 21:30 | |
| *** esberglu has quit IRC | 21:34 | |
| andreykurilin | mriedem: hi | 21:36 |
| andreykurilin | mriedem: just note: version list is broken in novaclient due to recent changes in devstack. I suppose it is about https://github.com/openstack-dev/devstack/commit/b90bb1a46193b052cd98513213eed688e8e29d23 . I have POC(https://review.openstack.org/#/c/479818/) of fix, tomorrow will add tests and it will be ready. | 21:38 |
| *** yassine has joined #openstack-nova | 21:40 | |
| *** claudiub has quit IRC | 21:46 | |
| *** kristian__ has quit IRC | 21:52 | |
| *** thorst has joined #openstack-nova | 21:55 | |
| *** thorst has quit IRC | 22:00 | |
| *** yassine has quit IRC | 22:01 | |
| *** takashin has joined #openstack-nova | 22:03 | |
| *** takashin has quit IRC | 22:10 | |
| *** takashin has joined #openstack-nova | 22:11 | |
| *** edmondsw has joined #openstack-nova | 22:11 | |
| *** edmondsw has quit IRC | 22:15 | |
| *** bkopilov_ has joined #openstack-nova | 22:16 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: WIP/POC Implement new attach Cinder flow https://review.openstack.org/330285 | 22:21 |
| *** lyan has joined #openstack-nova | 22:27 | |
| *** hongbin has quit IRC | 22:27 | |
| *** hongbin has joined #openstack-nova | 22:28 | |
| *** r-daneel has joined #openstack-nova | 22:28 | |
| mriedem | andreykurilin: ok thanks for the heads up | 22:29 |
| *** slaweq has quit IRC | 22:48 | |
| *** slaweq has joined #openstack-nova | 22:49 | |
| *** slaweq has quit IRC | 22:53 | |
| *** hongbin has quit IRC | 23:17 | |
| mriedem | andreykurilin: track it with this bug https://bugs.launchpad.net/python-novaclient/+bug/1702194 | 23:23 |
| openstack | Launchpad bug 1702194 in python-novaclient "test_version_list fails with "(TypeError): 'NoneType' object has no attribute '__getitem__'"" [Undecided,New] | 23:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Mask password when logging connection_info during swap volume https://review.openstack.org/466533 | 23:28 |
| *** r-daneel has quit IRC | 23:29 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640 | 23:30 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add missing parameters in limits.inc https://review.openstack.org/475640 | 23:34 |
| *** hieulq has quit IRC | 23:35 | |
| *** trungnv has quit IRC | 23:35 | |
| *** hshiina has joined #openstack-nova | 23:39 | |
| *** thorst has joined #openstack-nova | 23:41 | |
| *** gyee has quit IRC | 23:45 | |
| *** thorst has quit IRC | 23:46 | |
| *** trungnv has joined #openstack-nova | 23:49 | |
| *** hieulq has joined #openstack-nova | 23:49 | |
| *** masber has joined #openstack-nova | 23:56 | |
| *** s1061123 has quit IRC | 23:57 | |
| *** s1061123 has joined #openstack-nova | 23:59 | |
| *** edmondsw has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!