| *** shaner has quit IRC | 00:00 | |
| *** shaner has joined #openstack-nova | 00:02 | |
| *** felipemonteiro_ has quit IRC | 00:02 | |
| *** shaner has quit IRC | 00:07 | |
| *** shaner has joined #openstack-nova | 00:07 | |
| *** Qiming has quit IRC | 00:10 | |
| *** Qiming has joined #openstack-nova | 00:11 | |
| *** claudiub has quit IRC | 00:17 | |
| *** Nel1x has joined #openstack-nova | 00:19 | |
| *** Shunli has joined #openstack-nova | 00:30 | |
| *** tidwellr has quit IRC | 00:32 | |
| *** hieulq has joined #openstack-nova | 00:33 | |
| *** thorst_afk has joined #openstack-nova | 00:41 | |
| *** Qiming has quit IRC | 00:44 | |
| *** Qiming has joined #openstack-nova | 00:45 | |
| *** thorst_afk has quit IRC | 00:46 | |
| *** slaweq has joined #openstack-nova | 00:50 | |
| *** thorst_afk has joined #openstack-nova | 00:51 | |
| *** Qiming has quit IRC | 00:51 | |
| *** slaweq has quit IRC | 00:55 | |
| *** Qiming has joined #openstack-nova | 00:55 | |
| *** tyrefors has joined #openstack-nova | 00:56 | |
| *** yangyapeng has joined #openstack-nova | 00:57 | |
| *** superdan is now known as dansmith | 00:57 | |
| *** yangyapeng has quit IRC | 00:58 | |
| *** trungnv has joined #openstack-nova | 00:58 | |
| *** yangyape_ has joined #openstack-nova | 01:00 | |
| *** yangyapeng has joined #openstack-nova | 01:01 | |
| *** slaweq has joined #openstack-nova | 01:01 | |
| *** gmann has quit IRC | 01:02 | |
| *** gmann has joined #openstack-nova | 01:02 | |
| *** hongbin has joined #openstack-nova | 01:02 | |
| *** yangyape_ has quit IRC | 01:04 | |
| *** thorst_afk has quit IRC | 01:06 | |
| *** litao__ has joined #openstack-nova | 01:06 | |
| *** phuongnh has joined #openstack-nova | 01:06 | |
| *** thorst_afk has joined #openstack-nova | 01:07 | |
| *** slaweq has quit IRC | 01:07 | |
| *** thorst_afk has quit IRC | 01:11 | |
| *** edmondsw has joined #openstack-nova | 01:12 | |
| *** edmondsw has quit IRC | 01:15 | |
| *** edmondsw has joined #openstack-nova | 01:16 | |
| *** edmondsw has quit IRC | 01:16 | |
| *** Qiming has quit IRC | 01:21 | |
| *** Qiming has joined #openstack-nova | 01:21 | |
| *** gcb has joined #openstack-nova | 01:27 | |
| *** Qiming has quit IRC | 01:27 | |
| *** yingjun has joined #openstack-nova | 01:28 | |
| *** Qiming has joined #openstack-nova | 01:28 | |
| *** zhurong has quit IRC | 01:28 | |
| *** tetsuro has joined #openstack-nova | 01:30 | |
| *** Qiming has quit IRC | 01:37 | |
| *** Qiming has joined #openstack-nova | 01:39 | |
| *** Qiming has quit IRC | 01:44 | |
| *** Qiming has joined #openstack-nova | 01:46 | |
| *** tidwellr has joined #openstack-nova | 01:54 | |
| *** vikrant has joined #openstack-nova | 01:59 | |
| *** cali_boxer has joined #openstack-nova | 02:01 | |
| *** yangyapeng has quit IRC | 02:02 | |
| *** yangyapeng has joined #openstack-nova | 02:04 | |
| *** cali_boxer has quit IRC | 02:05 | |
| *** baoli has quit IRC | 02:07 | |
| *** baoli has joined #openstack-nova | 02:08 | |
| *** baoli has quit IRC | 02:13 | |
| *** Shunli has quit IRC | 02:14 | |
| *** felipemonteiro_ has joined #openstack-nova | 02:16 | |
| *** Shunli has joined #openstack-nova | 02:17 | |
| openstackgerrit | Sean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287 | 02:20 |
|---|---|---|
| *** trungnv has quit IRC | 02:21 | |
| *** felipemonteiro_ has quit IRC | 02:22 | |
| *** thorst_afk has joined #openstack-nova | 02:23 | |
| *** thorst_afk has quit IRC | 02:24 | |
| *** sree has joined #openstack-nova | 02:24 | |
| *** felipemonteiro has joined #openstack-nova | 02:27 | |
| *** felipemonteiro has quit IRC | 02:32 | |
| *** ljxiash has joined #openstack-nova | 02:35 | |
| *** baoli has joined #openstack-nova | 02:35 | |
| *** yamamoto has joined #openstack-nova | 02:38 | |
| *** yamamoto has quit IRC | 02:38 | |
| *** zhurong has joined #openstack-nova | 02:38 | |
| *** vikrant has quit IRC | 02:40 | |
| *** amotoki__away is now known as amotoki | 02:42 | |
| *** Qiming has quit IRC | 02:43 | |
| *** yamamoto has joined #openstack-nova | 02:43 | |
| *** hoonetorg has quit IRC | 02:43 | |
| *** Qiming has joined #openstack-nova | 02:45 | |
| *** itlinux has quit IRC | 02:45 | |
| *** Qiming has quit IRC | 02:49 | |
| *** trungnv has joined #openstack-nova | 02:52 | |
| *** Qiming has joined #openstack-nova | 02:53 | |
| *** kiennt has joined #openstack-nova | 02:54 | |
| *** trungnv has quit IRC | 02:58 | |
| *** Qiming has quit IRC | 02:58 | |
| *** Qiming has joined #openstack-nova | 02:59 | |
| *** hoonetorg has joined #openstack-nova | 03:00 | |
| *** trungnv has joined #openstack-nova | 03:09 | |
| *** trungnv_ has joined #openstack-nova | 03:09 | |
| *** Qiming has quit IRC | 03:10 | |
| *** trungnv_ has quit IRC | 03:11 | |
| *** trungnv has quit IRC | 03:11 | |
| *** trungnv has joined #openstack-nova | 03:12 | |
| *** Qiming has joined #openstack-nova | 03:12 | |
| *** Qiming has quit IRC | 03:17 | |
| *** trungnv has quit IRC | 03:21 | |
| *** thorst_afk has joined #openstack-nova | 03:25 | |
| *** thorst_afk has quit IRC | 03:30 | |
| *** Qiming has joined #openstack-nova | 03:31 | |
| *** mrhillsman has left #openstack-nova | 03:35 | |
| *** trungnv has joined #openstack-nova | 03:36 | |
| *** hongbin has quit IRC | 03:36 | |
| *** yangyapeng has quit IRC | 03:45 | |
| *** gbarros has quit IRC | 03:46 | |
| *** udesale has joined #openstack-nova | 03:49 | |
| *** baoli has quit IRC | 03:54 | |
| *** sridharg has joined #openstack-nova | 03:56 | |
| *** links has joined #openstack-nova | 04:00 | |
| *** trungnv has quit IRC | 04:15 | |
| *** claudiub has joined #openstack-nova | 04:18 | |
| *** ljxiash has quit IRC | 04:19 | |
| *** yangyapeng has joined #openstack-nova | 04:22 | |
| *** thorst_afk has joined #openstack-nova | 04:26 | |
| *** yangyapeng has quit IRC | 04:27 | |
| *** yangyapeng has joined #openstack-nova | 04:28 | |
| *** shan has joined #openstack-nova | 04:28 | |
| *** diga has joined #openstack-nova | 04:29 | |
| *** thorst_afk has quit IRC | 04:31 | |
| *** ljxiash has joined #openstack-nova | 04:33 | |
| *** shan has quit IRC | 04:33 | |
| *** ljxiash has quit IRC | 04:37 | |
| *** shan has joined #openstack-nova | 04:38 | |
| *** kiennt has quit IRC | 04:47 | |
| *** kiennt has joined #openstack-nova | 04:48 | |
| *** shan7 has joined #openstack-nova | 05:00 | |
| *** shan has quit IRC | 05:01 | |
| *** shan7 is now known as shan | 05:01 | |
| *** ljxiash has joined #openstack-nova | 05:02 | |
| *** ljxiash has quit IRC | 05:06 | |
| *** ljxiash has joined #openstack-nova | 05:08 | |
| *** Nel1x has quit IRC | 05:11 | |
| *** yangyapeng has quit IRC | 05:12 | |
| *** yangyapeng has joined #openstack-nova | 05:12 | |
| *** adisky__ has joined #openstack-nova | 05:13 | |
| *** jaosorior has joined #openstack-nova | 05:18 | |
| *** shan7 has joined #openstack-nova | 05:20 | |
| *** shan_afk has joined #openstack-nova | 05:21 | |
| *** shan has quit IRC | 05:22 | |
| *** shan_afk has quit IRC | 05:22 | |
| *** shan7 has quit IRC | 05:22 | |
| *** shan has joined #openstack-nova | 05:23 | |
| *** markvoelker_ has joined #openstack-nova | 05:25 | |
| *** thorst_afk has joined #openstack-nova | 05:27 | |
| *** markvoelker has joined #openstack-nova | 05:29 | |
| *** markvoelker_ has quit IRC | 05:29 | |
| *** rcernin has joined #openstack-nova | 05:30 | |
| *** thorst_afk has quit IRC | 05:32 | |
| *** markvoelker has quit IRC | 05:33 | |
| *** markvoelker has joined #openstack-nova | 05:34 | |
| *** udesale has quit IRC | 05:35 | |
| *** udesale__ has joined #openstack-nova | 05:35 | |
| *** bhagyashri_s has joined #openstack-nova | 05:36 | |
| *** ps_jadhav has quit IRC | 05:38 | |
| *** pooja_jadhav has quit IRC | 05:38 | |
| *** pooja_jadhav has joined #openstack-nova | 05:39 | |
| *** ps_jadhav has joined #openstack-nova | 05:39 | |
| *** bhagyashris_ has quit IRC | 05:39 | |
| *** mdnadeem has joined #openstack-nova | 05:39 | |
| *** mdnadeem has quit IRC | 05:40 | |
| *** mdnadeem has joined #openstack-nova | 05:40 | |
| *** ratailor has joined #openstack-nova | 05:46 | |
| openstackgerrit | jichenjc proposed openstack/nova master: use conf for keystone session creation https://review.openstack.org/485121 | 05:51 |
| *** trinaths has joined #openstack-nova | 05:52 | |
| *** lajoskatona has joined #openstack-nova | 05:54 | |
| *** ijw has joined #openstack-nova | 05:55 | |
| *** shan has quit IRC | 05:56 | |
| *** shan has joined #openstack-nova | 05:59 | |
| *** shan has quit IRC | 06:00 | |
| *** yangyapeng has quit IRC | 06:02 | |
| *** edmondsw has joined #openstack-nova | 06:02 | |
| *** yangyapeng has joined #openstack-nova | 06:05 | |
| *** shan has joined #openstack-nova | 06:07 | |
| *** slaweq has joined #openstack-nova | 06:08 | |
| *** yangyapeng has quit IRC | 06:10 | |
| *** yangyapeng has joined #openstack-nova | 06:11 | |
| *** slaweq has quit IRC | 06:13 | |
| *** karthiks has joined #openstack-nova | 06:16 | |
| *** yangyapeng has quit IRC | 06:22 | |
| *** yangyapeng has joined #openstack-nova | 06:23 | |
| *** mlakat has quit IRC | 06:28 | |
| *** thorst_afk has joined #openstack-nova | 06:28 | |
| *** bkopilov has joined #openstack-nova | 06:29 | |
| *** shan_afk has joined #openstack-nova | 06:30 | |
| *** shan7 has joined #openstack-nova | 06:31 | |
| *** shan_afk has quit IRC | 06:31 | |
| *** shan_sha has joined #openstack-nova | 06:31 | |
| *** thorst_afk has quit IRC | 06:32 | |
| *** shan_sha has quit IRC | 06:34 | |
| *** shan7 has quit IRC | 06:34 | |
| *** shan has quit IRC | 06:34 | |
| *** belmoreira has joined #openstack-nova | 06:34 | |
| *** shan has joined #openstack-nova | 06:36 | |
| *** pcaruana has joined #openstack-nova | 06:36 | |
| *** sree has quit IRC | 06:38 | |
| *** slaweq has joined #openstack-nova | 06:43 | |
| *** mlakat has joined #openstack-nova | 07:02 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 07:04 |
| *** gixxer1k has joined #openstack-nova | 07:05 | |
| *** andreas_s has joined #openstack-nova | 07:14 | |
| *** yingjun has quit IRC | 07:14 | |
| *** jpena|off is now known as jpena | 07:29 | |
| *** tesseract has joined #openstack-nova | 07:31 | |
| openstackgerrit | Rajesh Tailor proposed openstack/nova master: Host addition host-aggregate should be case-sensitive https://review.openstack.org/498334 | 07:32 |
| *** markus_z has joined #openstack-nova | 07:37 | |
| *** edmondsw has quit IRC | 07:40 | |
| gibi | good morning nova | 07:46 |
| *** ragiman has joined #openstack-nova | 07:54 | |
| *** dtantsur|sick is now known as dtantsur | 08:01 | |
| *** ragiman has quit IRC | 08:02 | |
| *** ragiman has joined #openstack-nova | 08:02 | |
| *** jaypipes has joined #openstack-nova | 08:08 | |
| *** slaweq_ has joined #openstack-nova | 08:10 | |
| *** josecastroleon has joined #openstack-nova | 08:14 | |
| *** slaweq_ has quit IRC | 08:15 | |
| *** bhagyashri_s has quit IRC | 08:16 | |
| *** pooja_jadhav has quit IRC | 08:16 | |
| *** ps_jadhav has quit IRC | 08:16 | |
| *** lucas-afk is now known as lucasagomes | 08:16 | |
| *** efoley has joined #openstack-nova | 08:16 | |
| *** efoley_ has joined #openstack-nova | 08:18 | |
| *** efoley has quit IRC | 08:18 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature https://review.openstack.org/476188 | 08:20 |
| *** pkoniszewski has joined #openstack-nova | 08:25 | |
| *** cali_boxer has joined #openstack-nova | 08:26 | |
| *** ralonsoh has joined #openstack-nova | 08:28 | |
| *** thorst_afk has joined #openstack-nova | 08:30 | |
| *** cali_boxer has quit IRC | 08:31 | |
| *** thorst_afk has quit IRC | 08:34 | |
| openstackgerrit | jichenjc proposed openstack/nova master: check query param for service's index function https://review.openstack.org/489492 | 08:35 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 08:35 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 08:36 |
| *** trinaths has quit IRC | 08:38 | |
| *** sree has joined #openstack-nova | 08:39 | |
| *** jaosorior has quit IRC | 08:39 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: slowly live-migration to ensure network is ready https://review.openstack.org/497457 | 08:41 |
| *** trinaths has joined #openstack-nova | 08:41 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: fix memory locking limit https://review.openstack.org/472633 | 08:41 |
| openstackgerrit | sahid proposed openstack/nova master: compute: reset instance events https://review.openstack.org/420026 | 08:42 |
| *** sree has quit IRC | 08:44 | |
| *** brault has joined #openstack-nova | 08:45 | |
| *** kiennt_ has joined #openstack-nova | 08:46 | |
| *** trungnv has joined #openstack-nova | 08:48 | |
| *** kiennt has quit IRC | 08:49 | |
| *** sree has joined #openstack-nova | 08:49 | |
| *** jaosorior has joined #openstack-nova | 08:50 | |
| *** damien_r has joined #openstack-nova | 08:52 | |
| *** damien_r has quit IRC | 08:53 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Allow users to unset 'keymap' options https://review.openstack.org/496605 | 08:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: De-duplicate some graphics tests https://review.openstack.org/497969 | 08:56 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 08:59 |
| *** ratailor is now known as ratailor|session | 09:02 | |
| *** lpetrut has joined #openstack-nova | 09:03 | |
| *** bhagyashris has joined #openstack-nova | 09:03 | |
| *** ps_jadhav has joined #openstack-nova | 09:03 | |
| *** pooja_jadhav has joined #openstack-nova | 09:03 | |
| *** claudiub|2 has joined #openstack-nova | 09:08 | |
| jaypipes | alex_xu: hey, would you be OK splitting the functional test here: https://review.openstack.org/#/c/480379/19/nova/tests/functional/db/test_resource_provider.py out into a separate patch (with commented-out asserts linking to a bug number)? | 09:08 |
| *** diga has quit IRC | 09:08 | |
| *** bhagyashris has quit IRC | 09:08 | |
| *** pooja_jadhav has quit IRC | 09:08 | |
| *** ps_jadhav has quit IRC | 09:08 | |
| jaypipes | alex_xu: it would probably help reviewers in seeing exactly where the bug is located if they could review the functional test with bad assertions (and comments saying "this is wrong because of bug XXXX") | 09:09 |
| *** claudiub has quit IRC | 09:10 | |
| *** slaweq_ has joined #openstack-nova | 09:11 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename two VNC options https://review.openstack.org/498387 | 09:11 |
| *** markvoelker has quit IRC | 09:15 | |
| *** slaweq_ has quit IRC | 09:16 | |
| *** trungnv has quit IRC | 09:19 | |
| *** pooja_jadhav has joined #openstack-nova | 09:22 | |
| *** bhagyashris has joined #openstack-nova | 09:22 | |
| *** ps_jadhav has joined #openstack-nova | 09:22 | |
| *** mnestratov has joined #openstack-nova | 09:24 | |
| alex_xu | jaypipes: got it, I will split out the functional test | 09:27 |
| *** gabor_antal has quit IRC | 09:28 | |
| jaypipes | alex_xu: ty sir! | 09:29 |
| *** thorst_afk has joined #openstack-nova | 09:30 | |
| *** Shunli has quit IRC | 09:31 | |
| *** thorst_afk has quit IRC | 09:35 | |
| *** edmondsw has joined #openstack-nova | 09:41 | |
| *** alexchadin has joined #openstack-nova | 09:42 | |
| alex_xu | jaypipes: np sir! | 09:44 |
| bhagyashris | jaypipes, cdent, melwitt: Hi, Just wanted to discuss about bug: [1]: https://bugs.launchpad.net/nova/+bug/1712816 bascially I have rported this issue specifically for stable/pike so IMO this is not duplicate. Can you please check my comment #3 on LP bug [1] | 09:45 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "duplicate for #1712816 instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 09:45 |
| *** edmondsw has quit IRC | 09:45 | |
| *** cdent has joined #openstack-nova | 09:46 | |
| jaypipes | bhagyashris: reporting an issue specifically for stable/pike still makes it a duplicate :) just add pike-backport-potential tag to the original bug. | 09:50 |
| bhagyashris | cdent: Hi, Just wanted to discuss about bug: [1]: https://bugs.launchpad.net/nova/+bug/1712816 bascially I have rported this issue specifically for stable/pike so IMO this is not duplicate. Can you please check my comment #3 on LP bug [1] | 09:50 |
| openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "duplicate for #1712816 instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to melanie witt (melwitt) | 09:50 |
| bhagyashris | jaypipes: ok | 09:50 |
| * cdent wonders what time zone jaypipes is in now | 09:52 | |
| *** sree has quit IRC | 09:53 | |
| bhagyashris | jaypipes: Thank you, for inputs :) | 09:53 |
| jaypipes | cdent: GMT+3 | 09:55 |
| jaypipes | bhagyashris: np | 09:55 |
| cdent | bhagyashris: given that mel’s patches are fixes for the other bug and they fix the problem you are reporting, I would say the newer bug does count as a duplicate. Why do you think not? | 09:55 |
| cdent | jaypipes: oh, I had assumed from your tweet about cnn that you had headed home | 09:56 |
| bhagyashris | cdent: yeah, but i was one doubt that As i have specifically reported for pike so I was thinking that that should not be duplicate for pike, but from jaypipes reply i clear my doubt. | 10:00 |
| cdent | bhagyashris: I suspect matt and I didn’t help by adding our comments there. that was in response to some discussion in one of the meetings last week | 10:02 |
| bhagyashris | cdent: thank you :) | 10:02 |
| bhagyashris | cdent: which comment?? | 10:05 |
| cdent | comments 1 and 2 from Matt and me. We should probably have commented on the older bug. | 10:05 |
| bhagyashris | cdent: ohh ok. | 10:06 |
| *** sridharg has quit IRC | 10:09 | |
| *** alexchadin has quit IRC | 10:09 | |
| bhagyashris | cdent: thank you :) | 10:09 |
| *** alexchadin has joined #openstack-nova | 10:09 | |
| cdent | you’re welcome | 10:09 |
| *** kiennt has joined #openstack-nova | 10:11 | |
| *** slaweq_ has joined #openstack-nova | 10:12 | |
| *** kiennt_ has quit IRC | 10:13 | |
| *** slaweq_ has quit IRC | 10:17 | |
| bhagyashris | cdent, jaypipes: just have one doubt in my mind as the stable/ocata has no issue in case of bfv but is issue in stable/pike so still this is consider as duplicate | 10:18 |
| *** yangyapeng has quit IRC | 10:19 | |
| bhagyashris | cdent, jaypipes: I mean i don't know about marking of bug duplication rules. | 10:19 |
| cdent | bhagyashris: The way to think of bugs is that there should only be one. If that bug is something that should be backported it will get tagged and targeted as necessary. | 10:21 |
| cdent | bhagyashris: the fact that the behavior appeared to be correct in ocata is only partially true | 10:22 |
| bhagyashris | cdent: ok. | 10:22 |
| *** baoli has joined #openstack-nova | 10:25 | |
| *** kiennt has quit IRC | 10:27 | |
| *** baoli has quit IRC | 10:30 | |
| *** gixxer1k has quit IRC | 10:31 | |
| *** thorst_afk has joined #openstack-nova | 10:31 | |
| *** thorst_afk has quit IRC | 10:36 | |
| *** baoli has joined #openstack-nova | 10:39 | |
| tetsuro | This patch of trivial typo fix is ready for review : https://review.openstack.org/#/c/495665/ Can anyone review it, please? | 10:43 |
| *** ljxiash has quit IRC | 10:44 | |
| *** ljxiash has joined #openstack-nova | 10:44 | |
| *** ljxiash has quit IRC | 10:49 | |
| *** gcb has quit IRC | 10:51 | |
| *** jamielennox has quit IRC | 10:51 | |
| *** imacdonn has quit IRC | 10:52 | |
| *** jamielennox has joined #openstack-nova | 10:52 | |
| *** lucasagomes is now known as lucas-hungry | 10:55 | |
| *** anthonyper has quit IRC | 11:09 | |
| *** anthonyper has joined #openstack-nova | 11:09 | |
| *** slaweq_ has joined #openstack-nova | 11:13 | |
| *** yamamoto has quit IRC | 11:15 | |
| *** markvoelker has joined #openstack-nova | 11:15 | |
| *** yamamoto has joined #openstack-nova | 11:16 | |
| *** yamamoto has quit IRC | 11:16 | |
| *** slaweq_ has quit IRC | 11:18 | |
| *** ratailor|session is now known as ratailor | 11:19 | |
| *** jpena is now known as jpena|lunch | 11:22 | |
| *** hferenc has quit IRC | 11:23 | |
| *** hferenc has joined #openstack-nova | 11:25 | |
| *** udesale has joined #openstack-nova | 11:31 | |
| *** gabor_antal has joined #openstack-nova | 11:31 | |
| *** yamamoto has joined #openstack-nova | 11:31 | |
| *** dave-mccowan has joined #openstack-nova | 11:32 | |
| *** szaher has quit IRC | 11:32 | |
| *** sree has joined #openstack-nova | 11:32 | |
| *** thorst_afk has joined #openstack-nova | 11:33 | |
| *** udesale__ has quit IRC | 11:33 | |
| *** szaher has joined #openstack-nova | 11:36 | |
| *** bkopilov has quit IRC | 11:39 | |
| *** thorst_afk has quit IRC | 11:40 | |
| *** tetsuro has quit IRC | 11:41 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 11:41 |
| *** edmondsw has joined #openstack-nova | 11:42 | |
| *** edmondsw has quit IRC | 11:46 | |
| cdent | vdrok: is this still alive: https://review.openstack.org/#/c/415512/ ? | 11:48 |
| vdrok | cdent: good afternoon :) kind of, trying to update it now and then | 11:49 |
| cdent | cool, just checking. It comes around on my to do list every now and again | 11:49 |
| *** markvoelker has quit IRC | 11:50 | |
| *** ratailor has quit IRC | 11:50 | |
| *** trinaths has left #openstack-nova | 11:50 | |
| * vdrok needs to spend some time and add a complete list of items needed in nova | 11:51 | |
| VAhl | Hi. I have some problem. I wonder if you could give me some hint what the problem could be. http://paste.openstack.org/show/619397/ There is resources available to create an VM. but it failes anyway. The backend is an ceph storage. In that storage there is three image, where Cirros is one of them | 11:53 |
| cdent | [t v2a] | 11:53 |
| purplerbot | <dtantsur> ENOTMUCHTIME is a common error code nowadays [2017-08-10 14:14:41.151800] [n v2a] | 11:53 |
| dtantsur | lol, morning cdent :) | 11:54 |
| cdent | :) | 11:54 |
| dtantsur | vdrok: do we want to discuss this owner-awareness at the PTG again? | 11:55 |
| *** litao__ has quit IRC | 11:55 | |
| cdent | VAhl: are you running pike, ocata, what? It looks like your placement service is not set up right and the logs for the placement-api would be needed to know what’s going on. But also see /topic | 11:56 |
| *** alexchadin has quit IRC | 11:57 | |
| *** alexchadin has joined #openstack-nova | 11:58 | |
| VAhl | cdent: Sorry Ocata | 11:58 |
| cdent | VAhl: at a guess I would say that it’s probably a database configuration problem. it’s rare for placement to have untrapped errors, but not being able to talk to the database might be one | 11:59 |
| vdrok | dtantsur: dunno. with the currently proposed way, there should be no changes to placement. what I need to check is what's the way to do exclusive filtering with traits, so RPs having certain traits would be chosen only if the corresponding trait is requested | 11:59 |
| *** alexchadin has quit IRC | 12:01 | |
| *** alexchadin has joined #openstack-nova | 12:01 | |
| dtantsur | no idea if it's possible, that's certainly not how capabilities work | 12:02 |
| vdrok | yup. I've seen some spec that was doing it for image properties iirc | 12:03 |
| *** udesale has quit IRC | 12:03 | |
| vdrok | dtantsur: ah ,maybe this one https://specs.openstack.org/openstack/nova-specs/specs/juno/approved/restrict-image-isolation-with-defined-keys.html | 12:04 |
| vdrok | but it still talks about aggregates :( | 12:05 |
| dtantsur | I see. I suspect with traits we may need both approaches.. | 12:05 |
| dtantsur | this does sound like something to talk on the PTG to me | 12:05 |
| vdrok | ok yeah, this is on the ironic etherpad already, do I need to mark it somehow? | 12:06 |
| *** phuongnh has quit IRC | 12:06 | |
| vdrok | like 'possible x-project' | 12:06 |
| dtantsur | vdrok: should we talk to nova folks? I've put some ironic stuff on nova's etherpad already, maybe append it there? | 12:06 |
| vdrok | ok will do | 12:07 |
| dtantsur | dear compute overlords, when do you plan to more or less finalize your PTG schedule? :) | 12:07 |
| *** lucas-hungry is now known as lucasagomes | 12:11 | |
| vdrok | dtantsur: added it beneath ironic problems | 12:13 |
| cdent | lucasagomes: are you in/near dublin? | 12:13 |
| dtantsur | thnx | 12:13 |
| dtantsur | cdent: he's in dub, yeah | 12:13 |
| * cdent has a spare afternoon in dublin on wednesday | 12:14 | |
| *** slaweq_ has joined #openstack-nova | 12:14 | |
| lucasagomes | cdent, hi there! I'm in dublin | 12:14 |
| lucasagomes | you around ? | 12:15 |
| cdent | will be on wednesday afternoon/evening | 12:15 |
| cdent | waiting on a plane | 12:15 |
| lucasagomes | cdent, great! We can catch up for some drinks | 12:15 |
| *** lifeless has quit IRC | 12:15 | |
| *** jpena|lunch is now known as jpena | 12:16 | |
| cdent | lucasagomes: wfm, I’ll ping you on wednesday (not yet sure when I’m actually getting in) and we can work out a plan? | 12:16 |
| lucasagomes | cdent, sounds good | 12:17 |
| vdrok | hey lucasagomes good afternoon to you too :) | 12:17 |
| lucasagomes | hey vdrok! You too, how are things ? | 12:17 |
| vdrok | is everyone slowly migrating to nova channel? :) | 12:17 |
| *** lifeless has joined #openstack-nova | 12:17 | |
| lucasagomes | hahah apparently so :-P | 12:17 |
| vdrok | pretty good :) | 12:17 |
| dtantsur | yes, it's going to end with ironic merging back into nova-baremetal :D | 12:18 |
| *** hrw has joined #openstack-nova | 12:18 | |
| hrw | hi | 12:18 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | 12:19 |
| *** slaweq_ has quit IRC | 12:20 | |
| hrw | sorry to bother but could someone take a look at https://review.openstack.org/#/c/489951/ and help us get aarch64 VMs booting out of the box without having to *always* set hw_firmware_type=uefi property? | 12:20 |
| *** gszasz has joined #openstack-nova | 12:23 | |
| *** zhurong has quit IRC | 12:23 | |
| *** gouthamr has quit IRC | 12:24 | |
| *** claudiub has joined #openstack-nova | 12:26 | |
| *** claudiub|2 has quit IRC | 12:29 | |
| *** sdague has joined #openstack-nova | 12:29 | |
| *** thorst_afk has joined #openstack-nova | 12:30 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Add method for verify multiple versioned notifications https://review.openstack.org/465526 | 12:32 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Refactor init_instance:resume_guests_state https://review.openstack.org/349582 | 12:32 |
| * efried appreciates Monday brain dump from jaypipes | 12:32 | |
| *** claudiub|2 has joined #openstack-nova | 12:33 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - manager.py https://review.openstack.org/359868 | 12:35 |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - linux_net.py https://review.openstack.org/359871 | 12:35 |
| *** claudiub has quit IRC | 12:36 | |
| *** catintheroof has joined #openstack-nova | 12:36 | |
| *** dave-mccowan has quit IRC | 12:38 | |
| *** sridharg has joined #openstack-nova | 12:40 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add functional resize tests using shared storage https://review.openstack.org/490733 | 12:42 |
| sdague | mikal: looks like your patch series needs some rebasing | 12:42 |
| *** slaweq has quit IRC | 12:42 | |
| *** gouthamr has joined #openstack-nova | 12:45 | |
| *** slaweq has joined #openstack-nova | 12:45 | |
| *** lyan has joined #openstack-nova | 12:46 | |
| *** gbarros has joined #openstack-nova | 12:46 | |
| efried | jaypipes I want to start putting these ideas into a blueprint to socialize over the next couple weeks before PTG. Work for you if I use https://blueprints.launchpad.net/nova/+spec/devices-as-resources or did you have another already started? | 12:48 |
| stephenfin | jaypipes, sdague: Could we get this in today? https://review.openstack.org/#/c/479869/ | 12:49 |
| *** dave-mccowan has joined #openstack-nova | 12:49 | |
| *** alexchadin has quit IRC | 12:50 | |
| sdague | stephenfin: +2 | 12:50 |
| *** alexchadin has joined #openstack-nova | 12:51 | |
| stephenfin | ta | 12:54 |
| hrw | efried: good point with setting hw_firmware_type=uefi only if unset | 12:55 |
| efried | hrw :) | 12:55 |
| hrw | efried: in case someone one day invent other way for aarch64 | 12:55 |
| efried | Right. | 12:55 |
| efried | I'm generally not a fan of implementing-for-future, but in this case it's just as easy and arguably more correct. | 12:55 |
| efried | hrw Also I suspect set-if-None will satisfy your other -1er. | 12:56 |
| hrw | efried: and for us whatever gets accepted as long as it will work ootb | 12:56 |
| efried | Suresure. | 12:56 |
| efried | hrw Is there a bug associated with this? | 12:57 |
| hrw | efried: would have to check as I have no idea. | 12:57 |
| hrw | efried: noticed that it still is in queue when looked at my list of patches to/in review | 12:57 |
| efried | I have no skin in the game; just seemed like it might. If you want to backport it to stable/pike, you'll need one anyway. | 12:57 |
| hrw | https://bugs.launchpad.net/nova will be right place? | 12:58 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove custom comparison methods https://review.openstack.org/472285 | 12:58 |
| efried | Yes. Where did this behavior come into existence? Sounds like some time after mitaka? | 12:58 |
| hrw | efried: yes, Mitaka added uefi support for aarch64 | 12:58 |
| hrw | before it was kernel+initrd only | 12:58 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: De-duplicate some graphics tests https://review.openstack.org/497969 | 13:00 |
| *** markvoelker has joined #openstack-nova | 13:01 | |
| *** lucasxu has joined #openstack-nova | 13:05 | |
| *** ljxiash has joined #openstack-nova | 13:06 | |
| *** esberglu has joined #openstack-nova | 13:06 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Account for shared storage in the report client https://review.openstack.org/491098 | 13:07 |
| *** hferenc has quit IRC | 13:08 | |
| efried | sdague Quick +W on https://review.openstack.org/#/c/475810/ and https://review.openstack.org/#/c/476159/ ? (https://review.openstack.org/#/c/476204/ already primed) | 13:12 |
| jaypipes | stephenfin: done | 13:14 |
| jaypipes | efried: sorry, been in 3 hour meeting... | 13:14 |
| jaypipes | efried: yeah, go for it. that's as good a name as any :) | 13:14 |
| hrw | ok, mail to Kevin sent | 13:14 |
| efried | jaypipes Roger wilco. | 13:14 |
| *** hferenc has joined #openstack-nova | 13:14 | |
| jaypipes | efried: and yeah, I appreciate you collating info on that | 13:14 |
| efried | But of course | 13:14 |
| jaypipes | efried: feel free to get an etherpad started too if you want | 13:14 |
| efried | jaypipes You mean split out from the main PTG one? | 13:15 |
| efried | Can do | 13:15 |
| *** slaweq_ has joined #openstack-nova | 13:15 | |
| jaypipes | efried: yup. I presume we'll have a session on that | 13:15 |
| jaypipes | efried: and a separate etherpad would be good to start on | 13:16 |
| openstackgerrit | Merged openstack/os-traits master: doc: Switch from oslosphinx to openstackdocstheme https://review.openstack.org/479869 | 13:18 |
| *** Yingxin has quit IRC | 13:19 | |
| openstackgerrit | Merged openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816 | 13:19 |
| *** Yingxin has joined #openstack-nova | 13:19 | |
| *** pchavva has joined #openstack-nova | 13:20 | |
| *** slaweq_ has quit IRC | 13:20 | |
| *** gixxer1k has joined #openstack-nova | 13:22 | |
| *** esberglu has quit IRC | 13:22 | |
| *** esberglu has joined #openstack-nova | 13:22 | |
| *** esberglu has quit IRC | 13:23 | |
| *** esberglu has joined #openstack-nova | 13:23 | |
| openstackgerrit | Merged openstack/os-traits master: doc: Remove cruft from conf.py https://review.openstack.org/480090 | 13:25 |
| sdague | efried: looking | 13:26 |
| *** hferenc has quit IRC | 13:27 | |
| sdague | efried: done | 13:27 |
| efried | sdague Thanks! | 13:27 |
| *** esberglu has quit IRC | 13:27 | |
| *** josecastroleon has quit IRC | 13:29 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Update PCI passthrough doc for moved options https://review.openstack.org/498461 | 13:30 |
| edleafe | Scheduler subteam meeting in 30 minutes in #openstack-meeting-alt | 13:30 |
| efried | stephenfin ^ split out from https://review.openstack.org/#/c/493701/ as requested, and filed against a bug (https://bugs.launchpad.net/nova/+bug/1713502) | 13:30 |
| openstack | Launchpad bug 1713502 in OpenStack Compute (nova) "PCI passthrough documentation needs updating since options moved to [pci] section" [Undecided,In progress] - Assigned to Eric Fried (efried) | 13:30 |
| efried | Not sure who can answer whether that should be backported to at least pike | 13:31 |
| efried | (Backport to ocata would be a non-cherry-pick, cause docs all moved) | 13:31 |
| *** burt has joined #openstack-nova | 13:31 | |
| *** josecastroleon has joined #openstack-nova | 13:31 | |
| stephenfin | efried: Yeah, iirc they didn't version the configuration guide so there's nothing to backport to for Ocata | 13:32 |
| efried | okay. The deprecated options still work, so it's not crucial. | 13:32 |
| stephenfin | I think doc patches should go back to Pike though. Maybe sdague would have some thoughts on the matter? | 13:33 |
| stephenfin | Oh, so it would actually | 13:33 |
| *** mriedem has joined #openstack-nova | 13:33 | |
| sdague | stephenfin: which doc patches? | 13:33 |
| stephenfin | sdague: https://review.openstack.org/#/c/498461/1 | 13:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: [Trivial] docstrings, typos, minor refactoring https://review.openstack.org/493701 | 13:34 |
| efried | stephenfin Rebased ^ on top | 13:34 |
| sdague | yeh, admin guide stuff should go back as it's all published per release | 13:35 |
| stephenfin | efried: Odd. I expected the +2 to disappear. Apparently not | 13:35 |
| efried | stephenfin When gerrit recognizes an auto rebase, it carries code reviews forward. | 13:36 |
| efried | Which is *almost* always okay :) | 13:36 |
| *** bkopilov has joined #openstack-nova | 13:37 | |
| *** takedakn has joined #openstack-nova | 13:37 | |
| *** sree_ has joined #openstack-nova | 13:38 | |
| *** esberglu has joined #openstack-nova | 13:38 | |
| *** sree_ is now known as Guest50673 | 13:38 | |
| *** gcb has joined #openstack-nova | 13:38 | |
| efried | stephenfin sdague https://review.openstack.org/#/c/498463/ is the pike cherry-pick. So should I go make the change in ocata too? | 13:38 |
| *** alexchadin has quit IRC | 13:40 | |
| sdague | efried: no, the admin guide is not in ocata | 13:40 |
| *** sree has quit IRC | 13:40 | |
| efried | sdague Okay, that's why I can't find it :) | 13:40 |
| sdague | yep | 13:40 |
| efried | That doc must have existed somewhere? | 13:40 |
| efried | Just in spec form mebbe? | 13:40 |
| sdague | on the openstack manuals side | 13:40 |
| *** markvoelker has quit IRC | 13:40 | |
| sdague | only in pike did the manuals go in tree for nova | 13:41 |
| *** lbragstad has joined #openstack-nova | 13:42 | |
| efried | sdague Right; so my question is whether I should go find and update that manual wherever it lived in ocata | 13:42 |
| *** edmondsw has joined #openstack-nova | 13:43 | |
| stephenfin | efried: I don't think they versioned the admin guide either, so there's nothing to backport to for Ocata | 13:44 |
| sdague | yeh, I'm not sure I would bother | 13:44 |
| efried | cool cool. | 13:44 |
| *** hferenc has joined #openstack-nova | 13:46 | |
| *** priteau has joined #openstack-nova | 13:47 | |
| *** edmondsw has quit IRC | 13:47 | |
| *** hferenc has quit IRC | 13:48 | |
| *** Guest50673 has quit IRC | 13:50 | |
| *** liverpooler has joined #openstack-nova | 13:50 | |
| *** sree has joined #openstack-nova | 13:51 | |
| *** felipemonteiro has joined #openstack-nova | 13:51 | |
| *** shan has quit IRC | 13:55 | |
| *** cfriesen_ has joined #openstack-nova | 13:55 | |
| *** sree has quit IRC | 13:55 | |
| *** awaugama has joined #openstack-nova | 13:56 | |
| *** felipemonteiro has quit IRC | 13:58 | |
| mriedem | gmann: i'm not sure https://review.openstack.org/#/c/490722/ warrants a microversion bump | 13:58 |
| mriedem | should probably be discussed in the nova-api meeting this week | 13:58 |
| *** cali_boxer has joined #openstack-nova | 13:59 | |
| cdent | mriedem: i rebased your shared providers functional test again. same deal as last time: leaving the failing tests with TODOs. I did make a couple of tweaks to report client in the patch above it to get a few more things passing | 14:00 |
| *** hongbin has joined #openstack-nova | 14:00 | |
| *** edmondsw has joined #openstack-nova | 14:00 | |
| *** efoley_ is now known as efoley | 14:01 | |
| mriedem | cdent: that was pretty incomplete from what i remember | 14:01 |
| mriedem | i know i need to abandon the wip for the fixes in the report client | 14:02 |
| cdent | It was but there were a couple of simple syntax thingies that move it forward a bit | 14:02 |
| *** gbarros has quit IRC | 14:02 | |
| *** ljxiash_ has joined #openstack-nova | 14:03 | |
| *** gbarros has joined #openstack-nova | 14:03 | |
| *** cali_boxer has quit IRC | 14:04 | |
| *** ljxiash has quit IRC | 14:06 | |
| *** david-lyle has quit IRC | 14:10 | |
| *** crushil has joined #openstack-nova | 14:11 | |
| *** alexchadin has joined #openstack-nova | 14:13 | |
| *** tidwellr has quit IRC | 14:14 | |
| *** armax has joined #openstack-nova | 14:16 | |
| *** slaweq_ has joined #openstack-nova | 14:16 | |
| *** jbernard has quit IRC | 14:20 | |
| *** slaweq_ has quit IRC | 14:21 | |
| *** jbernard has joined #openstack-nova | 14:21 | |
| sdague | mriedem: https://review.openstack.org/#/c/496515 is probably worthy of backport | 14:21 |
| *** jbernard has quit IRC | 14:22 | |
| *** links has quit IRC | 14:22 | |
| sdague | I push for pike, what do you think about ocata as well? | 14:22 |
| kashyap | sdague: Randomly chiming in, it sounds like backport-worthy, given the description | 14:23 |
| mriedem | seems pretty low probability | 14:24 |
| mriedem | wouldn't this be someone creating 2 instances with the same name on separate requests ~the same time? | 14:24 |
| sdague | mriedem: no | 14:24 |
| sdague | there is no name uniqueness requirement | 14:25 |
| sdague | or, it's behind a config option, off by default | 14:25 |
| sdague | so, it is 2 different requests, but they can be at any time | 14:25 |
| *** ljxiash_ has quit IRC | 14:26 | |
| mriedem | as long as you didn't delete the other first? | 14:26 |
| sdague | yes | 14:26 |
| sdague | but it could be made by a different tenant | 14:27 |
| mriedem | this is fine to backport. it's unfortunate that a 2 line code fix requires 100+ LOC change for tests | 14:27 |
| *** armax has quit IRC | 14:27 | |
| *** jbernard has joined #openstack-nova | 14:27 | |
| sdague | well, they tried to be comprehensive there | 14:27 |
| mriedem | i assume that's just because of how tightly coupled this all ways to the driver when it was moved out of the driver and into the host module | 14:27 |
| sdague | yeh | 14:28 |
| kashyap | Yeah, the tests seem to be trying to be a bit more comprehensive | 14:28 |
| sdague | mriedem: before I approve this change of yours, you want to put in newlines to the output for readability - https://review.openstack.org/#/c/486660 ? | 14:28 |
| *** xyang1 has joined #openstack-nova | 14:29 | |
| *** beekneemech is now known as bnemec | 14:31 | |
| *** tidwellr has joined #openstack-nova | 14:34 | |
| openstackgerrit | Alex Szarka proposed openstack/nova master: Reduce code complexity - instance.py https://review.openstack.org/359878 | 14:34 |
| *** edmondsw has quit IRC | 14:37 | |
| *** gbarros has quit IRC | 14:38 | |
| *** gbarros has joined #openstack-nova | 14:39 | |
| *** gbarros has quit IRC | 14:40 | |
| *** lajoskatona has quit IRC | 14:41 | |
| *** gbarros has joined #openstack-nova | 14:41 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: evacuate with no compute https://review.openstack.org/498482 | 14:41 |
| *** sree has joined #openstack-nova | 14:45 | |
| *** jdillaman has joined #openstack-nova | 14:45 | |
| *** hrw has quit IRC | 14:47 | |
| mriedem | sdague: it's going to be a bit | 14:47 |
| *** hrw has joined #openstack-nova | 14:48 | |
| *** sree has quit IRC | 14:49 | |
| mriedem | sdague: please redo those cherry picks using the -x option | 14:49 |
| *** gbarros has quit IRC | 14:54 | |
| *** chyka has joined #openstack-nova | 14:55 | |
| *** gyee has joined #openstack-nova | 14:56 | |
| *** artom has joined #openstack-nova | 14:57 | |
| *** ratailor has joined #openstack-nova | 14:57 | |
| *** belmoreira has quit IRC | 14:58 | |
| *** gbarros has joined #openstack-nova | 14:58 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: SPICE native client support https://review.openstack.org/442040 | 14:59 |
| *** ijw has quit IRC | 15:00 | |
| dansmith | cdent: it's still totes possible and should be on the radar for sure, but we've got a giant mess at the moment, and a bunch of unfinished bits that are way higher priority, IMHO | 15:00 |
| mriedem | cdent: i don't really see any reason why placement couldn't be split out, we're doing everything over the microversions onw | 15:00 |
| mriedem | *now | 15:00 |
| * jaypipes heads back to hotel... | 15:01 | |
| *** belmoreira has joined #openstack-nova | 15:01 | |
| mriedem | right, was going to say, the only thing/reason to not focus on the split right now is the existing amount of stuff we have to do | 15:01 |
| dansmith | especially since it's its own catalog entry, I don't think it's noticeably a nova thing to any other project other than where the code lives | 15:01 |
| dansmith | yeah | 15:01 |
| *** belmoreira has quit IRC | 15:02 | |
| edleafe | dansmith: one concern was the choice to shove everything in resource_providers.py | 15:02 |
| *** alexchadin has quit IRC | 15:02 | |
| edleafe | until the split | 15:02 |
| cdent | dansmith: yeah, your response points to a more detailed question: if those unfinished things are going to take a while and starting on moving things will disrupt them, there’s little point to start the extraction process soonish | 15:02 |
| mriedem | was that a conscious choice? | 15:02 |
| dansmith | I'm not sure what the single file has to do with anything, | 15:03 |
| *** ratailor has quit IRC | 15:03 | |
| dansmith | and I've complained along the way that that file is too fat already | 15:03 |
| mriedem | if you want to create nova/objects/alternative_hosts.py that's fine | 15:03 |
| dansmith | if splitting things out of there is helpful, then | 15:03 |
| dansmith | right | 15:03 |
| edleafe | mriedem: yes, because having placement in Nova was always supposed to be a temporary thing, and single file would make separation easier | 15:04 |
| openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280 | 15:04 |
| *** takedakn has quit IRC | 15:05 | |
| mriedem | i don't see how it makes it really any easier | 15:05 |
| mriedem | like, if we have 1 objects module or 10, that's not going to be the pain in the split | 15:06 |
| cdent | assuming it is contained (down collabortion without adding up collaboration) | 15:06 |
| edleafe | mriedem: I think the idea was it would keep us from scattering placement code throughout nova | 15:06 |
| mriedem | sure that's fair | 15:06 |
| cdent | jinxish | 15:06 |
| mriedem | but that's not going to be the painful part | 15:06 |
| dansmith | not at all | 15:06 |
| mriedem | i imagine the db will be the hardest | 15:06 |
| cdent | collabortion is my neologism of the day. I’ll have to figure out what it means. | 15:07 |
| *** josecastroleon has quit IRC | 15:07 | |
| edleafe | mriedem: sure, it's just not adding to that pain | 15:07 |
| cdent | db doesn’t have to be hardest. | 15:07 |
| cfriesen_ | it's when you have too many people working on something and they mess it up. :) | 15:07 |
| cdent | a) we have code for it already, b) we don’t need to use a different db | 15:07 |
| mriedem | cdent: you don't think at some point people will want to use a placement db without the nova_api db schema? | 15:07 |
| mriedem | like build_requests and instance_mappings? | 15:08 |
| cdent | cfriesen_: works, thanks! | 15:08 |
| mriedem | and have a placement user that can access that db rather than the nova user? | 15:08 |
| cdent | we have _some_ of that already | 15:08 |
| mriedem | i'd think you could do something during transition where reads start in the placement db and if not found, fallback to the nova_api db, and eventually you drop that transition code | 15:08 |
| mriedem | like when we move things from the nova db to the nova_api db | 15:09 |
| dansmith | or | 15:09 |
| cdent | but really why not just keep using the nova api db? | 15:09 |
| cdent | (that’s a question not a suggestion) | 15:09 |
| dansmith | make the first post-split db migration drop the non-placement tables and just require people to do a snapshot/move of their api db to get placement isolated | 15:09 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | 15:09 |
| *** sterdnotshaken has joined #openstack-nova | 15:10 | |
| mriedem | well, why not just keep placement in nova then? :) | 15:10 |
| *** jaypipes has quit IRC | 15:10 | |
| mriedem | if we're going to split it out, we should split it out | 15:10 |
| cdent | for sake of memory: https://review.openstack.org/#/c/362766/ https://github.com/cdent/placement | 15:10 |
| edleafe | cdent: sure it could, but if I wanted to just run placement, why do I need nova-api stuff? | 15:10 |
| cdent | edleafe: thank you. that leads to a useful answer: so I can run placement without nova having to exist. | 15:11 |
| mriedem | right, ironic and cinder will write to placement for k8s stuff (in some future world) | 15:11 |
| cdent | dansmith: a snapshot followed by a drop migration is a good idea | 15:12 |
| *** vikrant has joined #openstack-nova | 15:12 | |
| cdent | would it be easier to do that in tooling instead of in placement itself? | 15:13 |
| mriedem | we'd expect that in deployment tooling, not placement | 15:14 |
| mriedem | e.g. grenade | 15:14 |
| dansmith | the snapshot yeah | 15:15 |
| dansmith | but not the drop, IMHO | 15:15 |
| cdent | I meant the drop too | 15:15 |
| dansmith | but again, I think this is a distraction to even condsider | 15:15 |
| mriedem | the drop of instance_mappings, build_requests, etc is fine in placement itself | 15:15 |
| dansmith | (at the moment) | 15:15 |
| *** gcb has quit IRC | 15:15 | |
| mriedem | i'm ok if cdent wants to tease out the ideas if he wants to anyway, | 15:16 |
| mriedem | but i don't consider it a project priority atm | 15:16 |
| dansmith | no, I demand cdent not even think about it! | 15:16 |
| cdent | dansmith: yeah, that’s what I’m trying to find out. If everyone agrees with you, cool. mriedem: I’d personally prefer placement’s db handling to be clean slate, no migrations etc | 15:16 |
| *** david-lyle has joined #openstack-nova | 15:16 | |
| dansmith | of course, I just don't want to get into a long discussion about it at this stage, is all | 15:16 |
| cdent | dansmith: I’ll stop thinking if you promise to do my thinking for me henceforth. I could do with the break. | 15:17 |
| mriedem | i'm not sure what i *think* about that | 15:17 |
| *** slaweq_ has joined #openstack-nova | 15:17 | |
| mriedem | i'll be here all week | 15:17 |
| openstackgerrit | Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 15:18 |
| dansmith | mriedem: so for this migration spec, do you want the spec to be for "use uuid to hold allocations during a migration" or just "migrations should have uuids because uuids are great" ? | 15:18 |
| mriedem | the former | 15:18 |
| dansmith | yeah okay | 15:18 |
| mriedem | the problem statement is really related to the issues we found with move operations in pike with ocata computes overwriting n stuff | 15:19 |
| mriedem | also have to think about at what point is the allocation dropped from the source node, and the migration consumer allocation 'transferred' to the instance consumer | 15:20 |
| *** john5223 has quit IRC | 15:20 | |
| mriedem | maybe that's just a single PUT /resource_providers/uuid/allocations call? | 15:20 |
| *** ragiman has quit IRC | 15:21 | |
| *** rhagarty has quit IRC | 15:21 | |
| dansmith | um, say what? you mean on a revert? | 15:21 |
| *** gmann has quit IRC | 15:22 | |
| *** gmann has joined #openstack-nova | 15:22 | |
| *** slaweq_ has quit IRC | 15:22 | |
| sdague | mriedem: yeh, can do, I guess that is done automatically after it's a landed patch? | 15:23 |
| sdague | I was just using the gerrit ui | 15:23 |
| *** eharney has joined #openstack-nova | 15:24 | |
| *** jaypipes has joined #openstack-nova | 15:24 | |
| *** rhagarty has joined #openstack-nova | 15:24 | |
| *** john5223 has joined #openstack-nova | 15:25 | |
| *** armax has joined #openstack-nova | 15:25 | |
| mriedem | sdague: yeah for gerrit ui to do it the original has to be merged | 15:27 |
| sdague | gotcha, yeh, I'll redo after the master merge. My bad | 15:27 |
| mriedem | dansmith: doesn't the allocation on the dest node at some point have to transfer from the migration object to the instance object? | 15:27 |
| mriedem | especially if we ever count usage using placement for quota stuff | 15:28 |
| dansmith | mriedem: well I was going to do it the other way: | 15:28 |
| dansmith | mriedem: replace the source allocation with one for the migration, then allocate for the instance on the destination, assuming that's the expected end state | 15:29 |
| *** gbarros has quit IRC | 15:29 | |
| mriedem | and once the move is done, drop the migration allocation from the source node | 15:29 |
| mriedem | that works too | 15:29 |
| dansmith | right | 15:30 |
| dansmith | there is a gotcha in there in that we need space on the source to claim again for the size of the instance, but we could fix that with another operation against placement letting us do an atomic swap | 15:30 |
| *** jistr is now known as jistr|off|mtg | 15:33 | |
| *** jistr|off|mtg is now known as jistr | 15:33 | |
| *** links has joined #openstack-nova | 15:34 | |
| *** ijw has joined #openstack-nova | 15:36 | |
| *** ijw has quit IRC | 15:36 | |
| *** ijw has joined #openstack-nova | 15:36 | |
| *** gbarros has joined #openstack-nova | 15:39 | |
| *** coreywright has quit IRC | 15:40 | |
| *** markus_z has quit IRC | 15:41 | |
| openstackgerrit | Merged openstack/nova master: Don't warn on expected network-vif-unplugged events https://review.openstack.org/465794 | 15:44 |
| openstackgerrit | Merged openstack/nova master: Add device tag support info in support matrix https://review.openstack.org/481478 | 15:44 |
| openstackgerrit | Merged openstack/nova master: conf: Allow users to unset 'keymap' options https://review.openstack.org/496605 | 15:45 |
| openstackgerrit | Merged openstack/nova master: Deprecate CONF.monkey_patch https://review.openstack.org/498113 | 15:45 |
| *** mnestratov has quit IRC | 15:47 | |
| *** burt has quit IRC | 15:47 | |
| *** pcaruana has quit IRC | 15:47 | |
| *** ljxiash has joined #openstack-nova | 15:50 | |
| *** coreywright has joined #openstack-nova | 15:53 | |
| efried | mriedem Not sure if you're caught up on the ML yet, but ^^ vs. https://review.openstack.org/#/c/494305/ -- one of 'em needs to be reverted, nah? | 15:58 |
| *** lucasagomes is now known as lucas-afk | 16:00 | |
| openstackgerrit | Dan Smith proposed openstack/nova-specs master: WIP: Add migration-allocations spec https://review.openstack.org/498510 | 16:01 |
| dansmith | mriedem: ^ | 16:01 |
| mriedem | efried: thanks, commented on the puppet review | 16:03 |
| *** felipemonteiro has joined #openstack-nova | 16:05 | |
| *** cali_boxer has joined #openstack-nova | 16:06 | |
| *** andreas_s has quit IRC | 16:07 | |
| *** rcernin has quit IRC | 16:10 | |
| *** cali_box_ has joined #openstack-nova | 16:10 | |
| *** felipemonteiro has quit IRC | 16:10 | |
| *** cali_box_ has quit IRC | 16:11 | |
| *** cali_bo__ has joined #openstack-nova | 16:11 | |
| *** cali_boxer has quit IRC | 16:11 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optimize MiniDNS for fewer syscalls https://review.openstack.org/486829 | 16:12 |
| *** rajathagasthya has joined #openstack-nova | 16:13 | |
| *** mriedem1 has joined #openstack-nova | 16:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Provide hints when nova-manage db sync fails to sync cell0 https://review.openstack.org/486660 | 16:15 |
| *** mriedem has quit IRC | 16:16 | |
| *** slaweq_ has joined #openstack-nova | 16:18 | |
| *** eharney has quit IRC | 16:20 | |
| *** slaweq_ has quit IRC | 16:23 | |
| openstackgerrit | Sean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists https://review.openstack.org/498287 | 16:23 |
| *** annegentle has joined #openstack-nova | 16:24 | |
| *** links has quit IRC | 16:27 | |
| *** sterdnotshaken has quit IRC | 16:29 | |
| *** sterdnotshaken has joined #openstack-nova | 16:30 | |
| *** lpetrut has quit IRC | 16:30 | |
| *** baoli has quit IRC | 16:30 | |
| *** tesseract has quit IRC | 16:30 | |
| *** baoli has joined #openstack-nova | 16:37 | |
| *** jpena is now known as jpena|off | 16:39 | |
| openstackgerrit | Merged openstack/nova master: Remove useless error handling in prep_resize https://review.openstack.org/497976 | 16:40 |
| *** ociuhandu has joined #openstack-nova | 16:45 | |
| *** rcernin has joined #openstack-nova | 16:48 | |
| *** ociuhandu has quit IRC | 16:51 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Return alternate allocation requests to scheduler https://review.openstack.org/471927 | 16:51 |
| *** ssmith has joined #openstack-nova | 16:52 | |
| *** sterdnotshaken1 has joined #openstack-nova | 16:52 | |
| ssmith | Good morning. We're trying to prevent an instance from being deleted and have selected LOCK INSTANCE from the UI but people can still delete it. | 16:53 |
| mriedem1 | people == admins? | 16:53 |
| *** mriedem1 is now known as mriedem | 16:53 | |
| *** sterdnotshaken has quit IRC | 16:53 | |
| ssmith | mriedem1: Yes, they are admins | 16:54 |
| mriedem | ok admins can do stuff to instances even if they are locked | 16:56 |
| ssmith | mriedem: Yes, they are admins. I'm fine with them unlocking it and then deleting | 16:56 |
| ssmith | Is that in a policy somewhere that can be disabled? | 16:56 |
| mriedem | in this case it's hard-coded in code | 16:56 |
| mriedem | https://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/api.py#L2120 | 16:56 |
| mriedem | https://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/api.py#L198 | 16:56 |
| *** aarefiev_afk is now known as aarefiev | 16:57 | |
| mriedem | ssmith: https://review.openstack.org/#/c/492477/ is semi related | 16:57 |
| ssmith | So remove " and not context.is_admin" from the code? | 16:59 |
| mriedem | well, as i suggested in that patch, it could be a configurable policy check | 16:59 |
| ssmith | I agree with the need for the patch | 16:59 |
| mriedem | that patch is wrong in two ways: 1. it's not on the master branch and 2. it's config-driven API via nova.conf; configuring access to things in the API should be via policy rules, not nova.conf | 17:00 |
| mriedem | i wasn't -2 on the thing it's trying to address, just the way it's triyng to do it | 17:00 |
| mriedem | plus it's on stable/ocata which is wrong | 17:00 |
| ssmith | We're running Newton | 17:00 |
| mriedem | all changes start on master, so whatever the solution, it has to start on master | 17:01 |
| *** ralonsoh has quit IRC | 17:01 | |
| *** cdent has quit IRC | 17:10 | |
| *** ircuser-1 has joined #openstack-nova | 17:13 | |
| *** efoley has quit IRC | 17:14 | |
| *** vikrant has quit IRC | 17:18 | |
| *** baoli has quit IRC | 17:21 | |
| *** Swami has joined #openstack-nova | 17:25 | |
| *** yamahata has quit IRC | 17:26 | |
| *** shan has joined #openstack-nova | 17:27 | |
| *** lpetrut has joined #openstack-nova | 17:30 | |
| *** baoli has joined #openstack-nova | 17:32 | |
| *** sridharg has quit IRC | 17:41 | |
| *** sree has joined #openstack-nova | 17:50 | |
| *** ljxiash has quit IRC | 17:54 | |
| *** sree has quit IRC | 17:54 | |
| *** egonzalez has joined #openstack-nova | 17:54 | |
| *** yamahata has joined #openstack-nova | 17:55 | |
| *** carthaca_1 has joined #openstack-nova | 18:03 | |
| *** carthaca_ has quit IRC | 18:03 | |
| *** mnestratov has joined #openstack-nova | 18:04 | |
| *** mdnadeem has quit IRC | 18:05 | |
| *** sterdnotshaken has joined #openstack-nova | 18:05 | |
| *** burt has joined #openstack-nova | 18:07 | |
| *** sterdnotshaken1 has quit IRC | 18:08 | |
| *** dtantsur is now known as dtantsur|afk | 18:10 | |
| *** ijw has quit IRC | 18:20 | |
| *** slaweq_ has joined #openstack-nova | 18:21 | |
| *** itlinux has joined #openstack-nova | 18:23 | |
| *** slaweq_ has quit IRC | 18:26 | |
| *** cdent has joined #openstack-nova | 18:38 | |
| *** ijw has joined #openstack-nova | 18:41 | |
| *** rajathagasthya has quit IRC | 18:43 | |
| *** rajathagasthya has joined #openstack-nova | 18:44 | |
| *** ijw has quit IRC | 18:46 | |
| *** eharney has joined #openstack-nova | 18:48 | |
| openstackgerrit | Merged openstack/nova master: docs: Document the scheduler workflow https://review.openstack.org/475810 | 18:50 |
| openstackgerrit | Merged openstack/nova master: Monkey patch the blockdiag extension https://review.openstack.org/476159 | 18:50 |
| *** mwynne has quit IRC | 18:51 | |
| *** ijw has joined #openstack-nova | 18:51 | |
| openstackgerrit | Merged openstack/nova master: Add formatting to scheduling activity diagram https://review.openstack.org/476204 | 18:51 |
| openstackgerrit | Merged openstack/nova master: Update PCI passthrough doc for moved options https://review.openstack.org/498461 | 18:51 |
| openstackgerrit | Merged openstack/nova master: VMware: Do not check if folder already exists in vCenter https://review.openstack.org/376387 | 18:52 |
| *** mwynne has joined #openstack-nova | 18:52 | |
| *** armax has quit IRC | 18:53 | |
| *** sterdnotshaken has quit IRC | 18:54 | |
| *** ijw has quit IRC | 18:55 | |
| *** lpetrut has quit IRC | 18:56 | |
| *** mnestratov has quit IRC | 18:56 | |
| *** adisky__ has quit IRC | 19:02 | |
| *** ijw has joined #openstack-nova | 19:03 | |
| *** catintheroof has quit IRC | 19:04 | |
| *** rajathagasthya has quit IRC | 19:08 | |
| *** sterdnotshaken has joined #openstack-nova | 19:10 | |
| edleafe | OK, nova objects experts: what sort of Field would I use to store a dict like this: http://paste.openstack.org/show/619670/ | 19:10 |
| *** sterdnotshaken1 has joined #openstack-nova | 19:12 | |
| artom | edleafe, I think that depends on how you're going to be accessing it | 19:12 |
| *** annegentle has quit IRC | 19:12 | |
| edleafe | artom: probably just obj.foo to get the dict | 19:13 |
| cdent | edleafe: any of this help https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2338-L2359 | 19:14 |
| artom | edleafe, I would think it'd be just a Dict then | 19:14 |
| *** sterdnotshaken has quit IRC | 19:15 | |
| cdent | edleafe: of https://github.com/openstack/nova/blob/master/nova/objects/resource_provider.py#L2437-L2455 | 19:16 |
| cdent | basically reuse the objects use to create the data in the first place | 19:16 |
| cdent | If people want structured objects... | 19:16 |
| * cdent would rather native dicts | 19:16 | |
| edleafe | artom: but I got the feeling that fields.Dict was not to be used anymore | 19:16 |
| edleafe | artom: https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L72 | 19:16 |
| *** chyka_ has joined #openstack-nova | 19:17 | |
| openstackgerrit | Merged openstack/nova master: VMware: Handle missing volume vmdk during detach https://review.openstack.org/484675 | 19:17 |
| artom | edleafe, ah, true, didn't see the comment | 19:17 |
| artom | dansmith's the expert | 19:17 |
| edleafe | cdent: The object needs to have the hostname and the corresponding allocation | 19:17 |
| *** catintheroof has joined #openstack-nova | 19:17 | |
| edleafe | that's it | 19:17 |
| artom | cdent, I think the idea behind objects is a poor man's type system | 19:17 |
| openstackgerrit | Merged openstack/nova master: libvirt: Fix getting a wrong guest object https://review.openstack.org/496515 | 19:17 |
| artom | So that we don't have random unversioned unknown undiscoverable dicts flying around | 19:18 |
| openstackgerrit | Merged openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589 | 19:18 |
| cdent | artom: I know. Me and types have never felt all that friendly. | 19:18 |
| *** ijw has quit IRC | 19:18 | |
| *** marst has joined #openstack-nova | 19:18 | |
| artom | cdent, typist ;) | 19:18 |
| edleafe | cdent: I was thinking about using AllocationList, but that has DB stuff. This should just be a blob to send back | 19:20 |
| *** slaweq_ has joined #openstack-nova | 19:20 | |
| cdent | edleafe: the other thing to keep in mind is that we really don’t want to be using placement objects on the nova side of the equation if we can help it | 19:20 |
| *** chyka has quit IRC | 19:20 | |
| cdent | so a fresh object might be the way to go | 19:20 |
| edleafe | cdent: yeah, that too | 19:20 |
| mriedem | edleafe: https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L72 doesn't mean you can't use them, | 19:21 |
| mriedem | it means use the fields from ovo | 19:21 |
| mriedem | rather than nova | 19:21 |
| edleafe | but this allocations dict doesn't seem to fit any of the ovoo types | 19:21 |
| mriedem | as we're moving stuff from nova to ova | 19:21 |
| mriedem | *ovo | 19:21 |
| mriedem | so you have an Allocation object | 19:21 |
| mriedem | which has a resource_provider and resources field | 19:22 |
| cdent | edleafe: so you’re idea at this point is an object that is effectively; ‘target’: String, ‘allocation’: Dict | 19:22 |
| cdent | could be moultiple providers | 19:22 |
| edleafe | cdent: pretty much | 19:22 |
| cdent | is the idea to maintain the opacity of the allocation request bits? | 19:22 |
| *** pchavva has quit IRC | 19:23 | |
| edleafe | IMO, the allocation dict is already too much placement internals to be passed around in Nova | 19:23 |
| edleafe | But everyone else seems fine with it, so... | 19:23 |
| *** shan7 has joined #openstack-nova | 19:23 | |
| edleafe | cdent: and the example I pasted contains multiple providers. It will only have one hostname, though | 19:24 |
| edleafe | We're still in nova-land | 19:24 |
| cdent | I was responding to mriedem on the rp thing | 19:24 |
| cdent | I’m ambivalent about the allocation requests being passed around. sailed ship | 19:24 |
| mriedem | passing allocation requests dicts around within the scheduler, as we are today, seems fine | 19:25 |
| mriedem | if you want to pass something structured back over rpc to conductor to use for alternatives, you can define an object for that | 19:25 |
| mriedem | to avoid some list of 2-item tuples | 19:25 |
| edleafe | mriedem: that's what I'm trying to do | 19:25 |
| edleafe | mriedem: my question was what field type to use for the set of allocations | 19:25 |
| edleafe | We don't seem to have a matching type for that kind of nested dict structure | 19:26 |
| mriedem | well, you don't want to end up with a ListOfDictOfLists field | 19:26 |
| edleafe | DictOfListOfDicts? | 19:27 |
| *** shan has quit IRC | 19:27 | |
| mriedem | for http://paste.openstack.org/show/619670/ we're not going to just pass a top-level dict with a single "allocations" key are we? | 19:27 |
| mriedem | as the outermost object | 19:27 |
| mriedem | pass a list of allocation objects | 19:27 |
| edleafe | mriedem: yes. This is the body that is needed to claim/unclaim a host | 19:27 |
| artom | Remove the top-level 'allocation' and just use a ListOfObjectsField? | 19:27 |
| mriedem | artom: +1 | 19:28 |
| artom | I still think it depends on what you'll be doing with that | 19:28 |
| edleafe | So the object will have to have a method for creating the POST body | 19:28 |
| artom | As in, if the provider uuids are important at all, it could be a dict keyed on those | 19:28 |
| edleafe | artom: this should be opaque. It is the body needed to claim/unclaim a selected host and its resources | 19:28 |
| *** shan7 has quit IRC | 19:29 | |
| mriedem | creating an actual allocation request is pretty trivial https://review.openstack.org/#/c/496031/3/nova/conductor/tasks/live_migrate.py@122 | 19:29 |
| artom | Completely opaque? Just a string then? | 19:29 |
| mriedem | we're doing that when bypassing the scheduler and forcing a host for live migration ^ | 19:29 |
| cdent | edleafe: should we fix this first: https://bugs.launchpad.net/nova/+bug/1708204 | 19:29 |
| openstack | Launchpad bug 1708204 in OpenStack Compute (nova) "placement allocation representation asymetric on PUT and GET" [Wishlist,Confirmed] | 19:29 |
| edleafe | sure, we can chop this up any way that fits ovo, but then it has be able to put back together | 19:29 |
| edleafe | mriedem: that won't be as easy with shared/nested providers | 19:30 |
| *** shan7 has joined #openstack-nova | 19:30 | |
| mriedem | yeah i understand | 19:30 |
| mriedem | and per the todo in that patch, i'd like to still call the scheduler even if we're forcing the host during live migration | 19:31 |
| edleafe | the whole point of returning these allocation dicts was to handle those more complex situations | 19:31 |
| *** chyka_ has quit IRC | 19:31 | |
| mriedem | we already have an AllocationCandidate object today right? | 19:31 |
| mriedem | sorry, it's called AllocationRequest | 19:32 |
| *** burt has quit IRC | 19:32 | |
| mriedem | which has a list of AllocationRequestResource | 19:32 |
| *** shan has joined #openstack-nova | 19:33 | |
| edleafe | mriedem: I guess I'll try following that pattern | 19:33 |
| mriedem | why not just use the same exact objects? | 19:33 |
| mriedem | if that's exactly what you want for the structure? | 19:33 |
| edleafe | it's not | 19:33 |
| *** cdent has quit IRC | 19:34 | |
| mriedem | ok | 19:34 |
| edleafe | I need a thing that is a hostname, and the blob needed to claim/unclaim the resources for the particular request | 19:34 |
| mriedem | "(2:22:02 PM) cdent: edleafe: so you’re idea at this point is an object that is effectively; ‘target’: String, ‘allocation’: Dict" | 19:34 |
| edleafe | mriedem: yep | 19:35 |
| *** burt has joined #openstack-nova | 19:35 | |
| mriedem | and then return a list of those | 19:35 |
| mriedem | AlternativeHost, and AlternativeHostList | 19:35 |
| mriedem | or something | 19:35 |
| edleafe | we can bikeshed on the name :) | 19:35 |
| *** ijw has joined #openstack-nova | 19:35 | |
| mriedem | and you're wondering if you need to break down the 'allocation' field into a more specific type? | 19:35 |
| edleafe | what I was asking is: what ovo field type do I use for the allocation part of that? | 19:36 |
| edleafe | I don't want to break it down if at all possible | 19:36 |
| edleafe | It's a unit | 19:36 |
| edleafe | (for a particular claim) | 19:36 |
| artom | edleafe, so a single allocation (ie, resource_provider and resources) | 19:37 |
| artom | ? | 19:37 |
| edleafe | artom: well, it would be a set of allocations, but yes | 19:37 |
| mriedem | edleafe: then it's just Dict | 19:37 |
| mriedem | edleafe: and this is an allocation, yes? | 19:37 |
| mriedem | { "resource_provider": { "uuid": "9cf544dd-f0d7-4152-a9b8-02a65804df09" }, "resources": { "VCPU": 2, "MEMORY_MB": 8096 } } | 19:37 |
| edleafe | mriedem: even with dansmith's note at https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L72 ? | 19:37 |
| mriedem | edleafe: yes, as i said, that's about using the nova.fields vs ovo fields | 19:38 |
| mriedem | versioned objects started in nova, | 19:38 |
| mriedem | that note is saying, stuff is moving to ovo, so use the fields from there directly | 19:38 |
| mriedem | https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L657 | 19:38 |
| edleafe | mriedem: ok, then that note confused me | 19:38 |
| edleafe | everyone back to work! | 19:39 |
| mriedem | update the note with "Use these directly from oslo.versionedobjects for new code." or something if that clarifies it | 19:39 |
| *** armax has joined #openstack-nova | 19:40 | |
| edleafe | mriedem: will do | 19:41 |
| sdague | mriedem: https://review.openstack.org/#/c/498477 and https://review.openstack.org/#/c/498478 are redone with the git sha | 19:42 |
| mriedem | sdague: ok, but i'm going to hold off on stable/pike stuff until we release | 19:42 |
| sdague | mriedem: that's fine | 19:42 |
| sdague | it will still have real test results on it | 19:43 |
| openstackgerrit | Steve Noyes proposed openstack/nova master: update live migration to use v3 cinder api https://review.openstack.org/463987 | 19:43 |
| sdague | I was less concerned for the release, but just more for "these things should be in there sometime" | 19:43 |
| *** shan7 has quit IRC | 19:44 | |
| *** shan has quit IRC | 19:44 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Clarify the field usage guidelines https://review.openstack.org/498581 | 19:46 |
| edleafe | mriedem: quick note clarification ^^ | 19:46 |
| *** chyka has joined #openstack-nova | 19:51 | |
| *** edmondsw has joined #openstack-nova | 19:52 | |
| *** edmondsw has quit IRC | 19:54 | |
| *** chyka has quit IRC | 19:56 | |
| *** tidwellr has quit IRC | 19:57 | |
| *** tidwellr has joined #openstack-nova | 19:57 | |
| *** edmondsw has joined #openstack-nova | 20:00 | |
| *** liverpooler has quit IRC | 20:03 | |
| *** edmondsw has quit IRC | 20:04 | |
| *** cali_bo__ has quit IRC | 20:06 | |
| *** kfarr has joined #openstack-nova | 20:08 | |
| *** felipemonteiro has joined #openstack-nova | 20:19 | |
| *** gbarros has quit IRC | 20:21 | |
| *** cfriesen_ has quit IRC | 20:23 | |
| *** cfriesen_ has joined #openstack-nova | 20:23 | |
| *** annegentle has joined #openstack-nova | 20:23 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add uuid to migration object and migrate-on-load https://review.openstack.org/496934 | 20:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add placeholder migrations for Pike backports https://review.openstack.org/496932 | 20:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add uuid to migration table https://review.openstack.org/496933 | 20:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add uuid online migration for migrations https://review.openstack.org/498589 | 20:24 |
| dansmith | mriedem: that placeholder one failed out of the gate if you want to hit it again ^ | 20:25 |
| mnaser | (sorry, trying to get some help from another project that might have ran into this: i've discovered a performance issue in sahara and identified it to sqlalchemy lazy='joined' in the relationships which results in a giant set of joins) | 20:25 |
| mnaser | i noticed the nova code had none of that. was that a decision that was taken or just.. no one ever set that to anything | 20:25 |
| *** crushil has quit IRC | 20:26 | |
| mnaser | i figure nova is probably one of the longer codebases out here that might have ran into these perofmance regressions so i'd ask around | 20:26 |
| dansmith | mnaser: we generally join things explicitly when we want them | 20:26 |
| dansmith | intentionally. | 20:27 |
| mnaser | dansmith makes perfet sense. looking at the git blame.. looks like it was just magically changed with no explanation. ill do my checks | 20:28 |
| mnaser | thank you for that info | 20:28 |
| *** sree has joined #openstack-nova | 20:29 | |
| *** sree has quit IRC | 20:33 | |
| *** rajathagasthya has joined #openstack-nova | 20:34 | |
| *** kfarr has quit IRC | 20:38 | |
| *** chyka has joined #openstack-nova | 20:45 | |
| *** chyka has quit IRC | 20:46 | |
| *** lucasxu has quit IRC | 20:46 | |
| *** chyka has joined #openstack-nova | 20:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refactor ServerMovingTests for non-move tests https://review.openstack.org/498596 | 20:49 |
| *** priteau has quit IRC | 20:50 | |
| mriedem | dansmith: ^ as you asked for | 20:50 |
| dansmith | ah thanks | 20:50 |
| *** ijw has quit IRC | 20:51 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move lvm handling to privsep. https://review.openstack.org/495516 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move xend existence probes to privsep. https://review.openstack.org/495538 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move shred to privsep. https://review.openstack.org/495537 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls https://review.openstack.org/494423 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more. https://review.openstack.org/495542 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Replace dd with shred for zeroing lvm volumes. https://review.openstack.org/495532 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737 | 20:52 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438 | 20:52 |
| mikal | sdague: that's me rebasing that series, thanks for the review on the first one | 20:53 |
| mikal | sdague: https://review.openstack.org/#/c/495532/ is a similarly trivial head-of-series refactor if you're bored | 20:53 |
| mikal | sdague: well, "refactor" might be strong. "removal of historical silliness" might be more accurate. | 20:54 |
| *** ijw has joined #openstack-nova | 20:54 | |
| sdague | sure, it's about end of day here, so I can look tomorrow | 20:55 |
| edleafe | mikal: I prefer "hysterical silliness", myself | 20:55 |
| *** ijw has quit IRC | 20:56 | |
| *** ociuhandu has joined #openstack-nova | 20:57 | |
| *** ijw has joined #openstack-nova | 20:57 | |
| *** jdurgin has joined #openstack-nova | 20:57 | |
| *** felipemonteiro has quit IRC | 20:58 | |
| *** felipemonteiro has joined #openstack-nova | 20:58 | |
| mriedem | mikal: your test removal in the bottom change cuts a bit too deep | 21:00 |
| *** edmondsw has joined #openstack-nova | 21:03 | |
| *** burt has quit IRC | 21:03 | |
| mriedem | wow | 21:03 |
| mriedem | "Took 269.99 seconds to spawn the instance on the hypervisor." | 21:03 |
| *** tonyb has quit IRC | 21:04 | |
| mriedem | clarkb: ^ | 21:04 |
| cfriesen_ | is that running on a raspberry pi? | 21:04 |
| mriedem | ubuntu-xenial-vexxhost-ca-ymq-1-10660012 | 21:04 |
| mriedem | that was on http://logs.openstack.org/94/498094/3/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8e0a0d0/ | 21:05 |
| mikal | mriedem: as in on that shred change? | 21:06 |
| mriedem | mikal: yeah | 21:07 |
| mikal | mriedem: hmmm, I'll take a look today | 21:07 |
| mriedem | mikal: the slow node thing above is not that change | 21:07 |
| mriedem | just the test removal | 21:07 |
| *** edmondsw has quit IRC | 21:07 | |
| *** catintheroof has quit IRC | 21:09 | |
| *** yamamoto has quit IRC | 21:09 | |
| clarkb | mriedem: what image is being booted? that looks normal for ubuntu | 21:09 |
| mriedem | cirros | 21:10 |
| mriedem | cf0d1557-c4b3-4304-b4ec-5e6eaf4a7123 | 21:11 |
| *** tonyb has joined #openstack-nova | 21:11 | |
| mriedem | sorry | 21:11 |
| mriedem | http://logs.openstack.org/94/498094/3/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8e0a0d0/logs/devstacklog.txt.gz#_2017-08-28_14_49_41_357 | 21:11 |
| mriedem | cirros-0.3.5-x86_64-disk | 21:11 |
| mriedem | looking at most of these "seconds to spawn the instance on the hyperviso" messages, most are around 2 seconds | 21:14 |
| mriedem | but some just spike to like 1-2+ minutes | 21:14 |
| *** tidwellr has quit IRC | 21:14 | |
| *** tidwellr has joined #openstack-nova | 21:15 | |
| *** gixxer1k has quit IRC | 21:17 | |
| *** tidwellr has quit IRC | 21:19 | |
| openstackgerrit | Merged openstack/nova master: rename binary to source in versioned notifications https://review.openstack.org/487126 | 21:19 |
| openstackgerrit | Merged openstack/nova master: Use enum value instead of string service name https://review.openstack.org/476538 | 21:23 |
| *** gouthamr has quit IRC | 21:23 | |
| openstackgerrit | Merged openstack/nova master: Cleanup unnecessary logic in os-volume_attachments controller code https://review.openstack.org/485823 | 21:27 |
| *** FL1SK has quit IRC | 21:29 | |
| *** thorst_afk has quit IRC | 21:32 | |
| *** rcernin has quit IRC | 21:32 | |
| *** baoli has quit IRC | 21:34 | |
| *** baoli has joined #openstack-nova | 21:35 | |
| *** baoli has quit IRC | 21:35 | |
| *** thorst_afk has joined #openstack-nova | 21:35 | |
| *** baoli has joined #openstack-nova | 21:36 | |
| *** yamamoto_ has joined #openstack-nova | 21:38 | |
| *** edmondsw has joined #openstack-nova | 21:38 | |
| *** thorst_afk has quit IRC | 21:40 | |
| *** baoli has quit IRC | 21:41 | |
| *** edmondsw has quit IRC | 21:42 | |
| *** ssmith has quit IRC | 21:51 | |
| openstackgerrit | Merged openstack/nova-specs master: Return alternate allocation requests to scheduler https://review.openstack.org/471927 | 21:53 |
| *** xyang1 has quit IRC | 21:54 | |
| *** FL1SK has joined #openstack-nova | 21:59 | |
| *** crushil has joined #openstack-nova | 22:00 | |
| mriedem | stephenfin: efried: was the claims part of scheduling going to get built into this at some point? https://review.openstack.org/#/c/475810/ | 22:02 |
| *** egonzalez has quit IRC | 22:03 | |
| *** yamamoto_ has quit IRC | 22:04 | |
| *** chyka_ has joined #openstack-nova | 22:04 | |
| efried | mriedem I have to defer to stephenfin / edleafe / whoever. I just made the words fit in the boxes. | 22:05 |
| *** felipemonteiro_ has joined #openstack-nova | 22:06 | |
| mriedem | http://docs-draft.openstack.org/10/475810/11/check/gate-nova-docs-ubuntu-xenial/5b58371//doc/build/html/reference/scheduling.html | 22:06 |
| mriedem | they don't fit for me | 22:06 |
| *** chyka has quit IRC | 22:08 | |
| efried | mriedem Two changes built on top of that one make 'em fit. | 22:08 |
| *** felipemonteiro__ has joined #openstack-nova | 22:08 | |
| efried | And I think they merged. Where's the official link for that doc? | 22:08 |
| *** esberglu has quit IRC | 22:09 | |
| *** edmondsw has joined #openstack-nova | 22:09 | |
| *** felipemonteiro has quit IRC | 22:10 | |
| *** esberglu has joined #openstack-nova | 22:10 | |
| *** esberglu has quit IRC | 22:10 | |
| *** cfriesen__ has joined #openstack-nova | 22:10 | |
| *** cfriesen__ is now known as cfriesen | 22:10 | |
| *** ijw has quit IRC | 22:10 | |
| edleafe | efried: mriedem: yeah, that will have to be updated in the future. It's been a moving target. | 22:11 |
| mriedem | efried: https://docs.openstack.org/nova/latest/reference/index.html | 22:11 |
| mriedem | it's not posted yet | 22:11 |
| *** felipemonteiro_ has quit IRC | 22:11 | |
| *** cfriesen_ has quit IRC | 22:11 | |
| mriedem | the content is also wrong | 22:11 |
| *** ijw has joined #openstack-nova | 22:11 | |
| mriedem | so i'm not sure why this was merged as is | 22:11 |
| *** slaweq_ has quit IRC | 22:11 | |
| *** felipemonteiro_ has joined #openstack-nova | 22:12 | |
| efried | mriedem Well, for what it's worth, the wrong content is formatted properly via http://docs-draft.openstack.org/04/476204/5/check/gate-nova-docs-ubuntu-xenial/b74b92c//doc/build/html/reference/scheduling.html | 22:12 |
| *** slaweq_ has joined #openstack-nova | 22:12 | |
| *** awaugama has quit IRC | 22:12 | |
| mriedem | heh, ok, well at least there is that | 22:12 |
| mriedem | now i can read what the wrong words are | 22:13 |
| *** edmondsw has quit IRC | 22:13 | |
| *** cfriesen_ has joined #openstack-nova | 22:13 | |
| mriedem | the diagram looks like ocata, the text looks like pike | 22:14 |
| mriedem | and the alternatives stuff is queens | 22:14 |
| *** felipemonteiro has joined #openstack-nova | 22:14 | |
| *** ijw has quit IRC | 22:16 | |
| *** felipemonteiro__ has quit IRC | 22:16 | |
| *** cfriesen has quit IRC | 22:16 | |
| *** cfriesen__ has joined #openstack-nova | 22:16 | |
| *** cfriesen__ is now known as cfriesen | 22:16 | |
| cfriesen | my VPN is going nuts...grr | 22:16 |
| *** felipemonteiro_ has quit IRC | 22:16 | |
| *** slaweq_ has quit IRC | 22:17 | |
| *** edmondsw has joined #openstack-nova | 22:17 | |
| mriedem | i'll see if i can adjust this a bit | 22:19 |
| *** cfriesen_ has quit IRC | 22:19 | |
| *** edmondsw has quit IRC | 22:21 | |
| *** edmondsw has joined #openstack-nova | 22:27 | |
| *** esberglu has joined #openstack-nova | 22:28 | |
| *** itlinux has quit IRC | 22:28 | |
| *** marst has quit IRC | 22:31 | |
| *** esberglu has quit IRC | 22:33 | |
| *** ijw has joined #openstack-nova | 22:33 | |
| *** edmondsw has quit IRC | 22:34 | |
| *** yamamoto_ has joined #openstack-nova | 22:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Updates to scheduling workflow doc https://review.openstack.org/498613 | 22:35 |
| mriedem | here we are ^ | 22:35 |
| *** jdurgin has quit IRC | 22:41 | |
| *** markvoelker has joined #openstack-nova | 22:42 | |
| *** slaweq_ has joined #openstack-nova | 22:43 | |
| *** yamamoto_ has quit IRC | 22:43 | |
| *** mriedem has quit IRC | 22:45 | |
| *** slaweq_ has quit IRC | 22:47 | |
| *** esberglu has joined #openstack-nova | 22:49 | |
| *** yamamoto_ has joined #openstack-nova | 22:49 | |
| openstackgerrit | Jamie Lennox proposed openstack/nova master: Remove unneeded attributes from context https://review.openstack.org/498615 | 22:50 |
| *** felipemonteiro_ has joined #openstack-nova | 22:51 | |
| cfriesen | is it just me or is the response to a "GET /flavors/{flavor_id}/os-flavor-access" kind of odd? It looks like the flavor id will be returned once per access rule, which seems somewhat redundant. | 22:52 |
| *** felipemonteiro__ has joined #openstack-nova | 22:53 | |
| *** rajathagasthya has quit IRC | 22:53 | |
| *** lyan has quit IRC | 22:53 | |
| *** rajathagasthya has joined #openstack-nova | 22:54 | |
| *** felipemonteiro has quit IRC | 22:54 | |
| *** felipemonteiro_ has quit IRC | 22:56 | |
| *** ragasthya has joined #openstack-nova | 22:57 | |
| *** yamamoto_ has quit IRC | 22:59 | |
| *** yamamoto_ has joined #openstack-nova | 23:06 | |
| *** hongbin has quit IRC | 23:07 | |
| *** baoli has joined #openstack-nova | 23:08 | |
| *** mriedem has joined #openstack-nova | 23:09 | |
| *** baoli has quit IRC | 23:12 | |
| *** esberglu has quit IRC | 23:15 | |
| *** beagles has joined #openstack-nova | 23:20 | |
| beagles | are the devs working on os-vif in this channel, neutron, or somewhere else? | 23:21 |
| *** sdague has quit IRC | 23:21 | |
| beagles | sean-k-mooney, ^ | 23:21 |
| *** cfriesen has quit IRC | 23:23 | |
| *** cfriesen has joined #openstack-nova | 23:23 | |
| gmann | mriedem, i agree on bug fix but for POST aggregate API its changing 200 to 400 | 23:27 |
| mriedem | beagles: here mostly i think | 23:28 |
| *** itlinux has joined #openstack-nova | 23:28 | |
| mriedem | gmann: yeah but it was a wrong 200 which probably led to a later NoValidHost anyway | 23:28 |
| beagles | mriedem, k thanks | 23:28 |
| *** itlinux has quit IRC | 23:28 | |
| gmann | mriedem, humm | 23:29 |
| *** chyka_ has quit IRC | 23:34 | |
| *** chyka has joined #openstack-nova | 23:35 | |
| *** chyka has quit IRC | 23:39 | |
| *** ijw has quit IRC | 23:41 | |
| *** ijw has joined #openstack-nova | 23:42 | |
| *** jdurgin has joined #openstack-nova | 23:42 | |
| *** claudiub|2 has quit IRC | 23:43 | |
| *** slaweq_ has joined #openstack-nova | 23:44 | |
| *** ijw has quit IRC | 23:46 | |
| *** markvoelker has quit IRC | 23:46 | |
| *** yamamoto_ has quit IRC | 23:47 | |
| *** slaweq_ has quit IRC | 23:48 | |
| *** jdurgin has quit IRC | 23:49 | |
| *** jdurgin has joined #openstack-nova | 23:50 | |
| *** ijw has joined #openstack-nova | 23:52 | |
| *** edmondsw has joined #openstack-nova | 23:53 | |
| *** gouthamr has joined #openstack-nova | 23:54 | |
| *** baoli has joined #openstack-nova | 23:55 | |
| *** ijw has quit IRC | 23:56 | |
| *** edmondsw_ has joined #openstack-nova | 23:56 | |
| *** jdurgin has quit IRC | 23:57 | |
| *** edmondsw has quit IRC | 23:58 | |
| *** diga has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!