| *** yingjun has joined #openstack-nova | 00:01 | |
| *** thorst_afk has joined #openstack-nova | 00:02 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Make NovaException format errors fatal for tests https://review.openstack.org/411615 | 00:07 |
|---|---|---|
| openstackgerrit | melanie witt proposed openstack/nova master: Fix unit test exception KeyErrors https://review.openstack.org/462764 | 00:07 |
| *** Apoorva has quit IRC | 00:07 | |
| melwitt | mriedem: I revived my old attempt at the exception KeyErrors detection ^ I ordered the fixes after the detection patch for now to demonstrate that it works | 00:07 |
| *** Daviey_ has quit IRC | 00:11 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix unit test exception KeyErrors https://review.openstack.org/462764 | 00:13 |
| *** Yangyapeng has quit IRC | 00:14 | |
| jaypipes | mriedem: ty sir. | 00:15 |
| jaypipes | mriedem: to cut a release, do I just git tag -a it? | 00:15 |
| *** thorst_afk has quit IRC | 00:17 | |
| *** tjones has left #openstack-nova | 00:21 | |
| *** jerrygb has joined #openstack-nova | 00:22 | |
| *** slaweq has joined #openstack-nova | 00:23 | |
| *** jamesdenton has joined #openstack-nova | 00:24 | |
| *** jerrygb has quit IRC | 00:26 | |
| *** zz_dimtruck is now known as dimtruck | 00:27 | |
| *** slaweq has quit IRC | 00:28 | |
| *** nic has quit IRC | 00:29 | |
| *** mriedem1 has joined #openstack-nova | 00:31 | |
| mriedem1 | jaypipes: nope | 00:31 |
| mriedem1 | you post to openstack/releases | 00:31 |
| mriedem1 | jaypipes: like this https://github.com/openstack/releases/commit/33ebc95728b40ca6a06c764a51a79dc24e95b555 | 00:32 |
| *** hoonetorg has quit IRC | 00:32 | |
| mriedem1 | so you need a new - version entry | 00:32 |
| mriedem1 | with the latest git hash | 00:32 |
| mriedem1 | highlights are optional | 00:32 |
| *** mriedem has quit IRC | 00:33 | |
| mriedem1 | melwitt: ok | 00:33 |
| mriedem1 | nice | 00:33 |
| mriedem1 | melwitt: i'll have to think of some other crazy crap that needs fixing now | 00:33 |
| *** iceyao has joined #openstack-nova | 00:35 | |
| mriedem1 | melwitt: wait, so, | 00:39 |
| mriedem1 | how does the bottom change not fail if you didn't also fix the things in the top change at the same time? | 00:39 |
| *** mriedem1 is now known as mriedem | 00:39 | |
| *** iceyao has quit IRC | 00:40 | |
| *** thorst_afk has joined #openstack-nova | 00:45 | |
| *** hoonetorg has joined #openstack-nova | 00:46 | |
| *** thorst_afk has quit IRC | 00:46 | |
| *** iceyao has joined #openstack-nova | 00:49 | |
| *** winston-d_ has joined #openstack-nova | 00:50 | |
| *** yamamoto has joined #openstack-nova | 00:51 | |
| *** Shunli has joined #openstack-nova | 00:53 | |
| *** iceyao has quit IRC | 00:53 | |
| *** yamamoto has quit IRC | 00:54 | |
| *** yamahata has joined #openstack-nova | 00:55 | |
| *** salv-orlando has joined #openstack-nova | 00:55 | |
| mriedem | Kevin_Zheng: i've got a start on the forum session for listing instances using searchlight https://etherpad.openstack.org/p/BOS-forum-using-searchlight-to-list-instances | 00:58 |
| mriedem | in case you have things you'd like to see covered in that session; the etherpad is a bit basic at the moment | 00:58 |
| *** jamesdenton has quit IRC | 00:59 | |
| *** salv-orlando has quit IRC | 00:59 | |
| *** jamesdenton has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| *** tuanluong has joined #openstack-nova | 01:03 | |
| *** brault has joined #openstack-nova | 01:05 | |
| *** dixiaoli has joined #openstack-nova | 01:06 | |
| *** gjayavelu has quit IRC | 01:06 | |
| *** eandersson has quit IRC | 01:06 | |
| *** brault has quit IRC | 01:10 | |
| *** iceyao has joined #openstack-nova | 01:10 | |
| *** markvoelker has quit IRC | 01:12 | |
| *** markvoelker has joined #openstack-nova | 01:13 | |
| *** mingyu has joined #openstack-nova | 01:13 | |
| mriedem | jaypipes: and this one for you to gaze upon https://etherpad.openstack.org/p/BOS-forum-compute-instance-volume-affinity-hpc | 01:14 |
| *** lbragstad_alt has quit IRC | 01:14 | |
| *** Yangyapeng has joined #openstack-nova | 01:15 | |
| *** thorst_afk has joined #openstack-nova | 01:15 | |
| jaypipes | mriedem: cool, thx. | 01:15 |
| *** baoli has quit IRC | 01:16 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: add cache which keeps DB and os_traits in sync https://review.openstack.org/462769 | 01:16 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: integrate trait cache with TraitList and Trait https://review.openstack.org/462770 | 01:16 |
| *** kevinz has joined #openstack-nova | 01:16 | |
| *** jamesdenton has quit IRC | 01:16 | |
| jaypipes | mriedem, dansmith: ^^ depends on the as-yet-unreleased os-traits 0.3 release. | 01:16 |
| *** lbragstad has joined #openstack-nova | 01:17 | |
| *** gouthamr has joined #openstack-nova | 01:17 | |
| *** markvoelker has quit IRC | 01:17 | |
| *** mingyu has quit IRC | 01:18 | |
| *** lbragstad has quit IRC | 01:18 | |
| *** lbragstad has joined #openstack-nova | 01:18 | |
| mriedem | jaypipes: ping me when you get that releases patch up | 01:18 |
| mriedem | or have me push it | 01:18 |
| mriedem | whichever | 01:18 |
| mriedem | b/c i can push it real good | 01:18 |
| *** ssurana has joined #openstack-nova | 01:18 | |
| *** ssurana has quit IRC | 01:19 | |
| Kevin_Zheng | mriedem: I do have some information about this, could I update it in the bottom of the etherpad and maybe you can have some idea and update the whole ehterpad？ | 01:19 |
| *** thorst_afk has quit IRC | 01:19 | |
| mriedem | Kevin_Zheng: sure | 01:19 |
| *** lbragstad has quit IRC | 01:19 | |
| *** lbragstad has joined #openstack-nova | 01:20 | |
| Kevin_Zheng | mriedem: :) | 01:20 |
| *** crushil has joined #openstack-nova | 01:20 | |
| mriedem | Kevin_Zheng: i put an "other notes" section at the bottom https://etherpad.openstack.org/p/BOS-forum-using-searchlight-to-list-instances | 01:20 |
| Kevin_Zheng | OK | 01:20 |
| jaypipes | mriedem: https://review.openstack.org/462771 | 01:21 |
| *** dave-mccowan has joined #openstack-nova | 01:21 | |
| *** lbragstad has quit IRC | 01:21 | |
| *** lbragstad has joined #openstack-nova | 01:22 | |
| *** lbragstad has quit IRC | 01:23 | |
| *** lbragstad has joined #openstack-nova | 01:23 | |
| mriedem | jaypipes: beautiful | 01:25 |
| *** kaisers has joined #openstack-nova | 01:25 | |
| jaypipes | mriedem: thanks honey. | 01:25 |
| sean-k-mooney | jaypipes: does that new release have the network namespace out of interest? | 01:26 |
| jaypipes | sean-k-mooney: yup. | 01:26 |
| sean-k-mooney | cool | 01:26 |
| jaypipes | sean-k-mooney: also... what the heck are you doing up? :) | 01:26 |
| sean-k-mooney | i did not sleep yesterday so i had a 4 hour nap from 8pm to ~midnight my time so im awake again | 01:27 |
| sean-k-mooney | not ideal but oh well | 01:27 |
| jaypipes | ah :) | 01:27 |
| *** kaisers_ has quit IRC | 01:28 | |
| *** marst has quit IRC | 01:28 | |
| *** marst has joined #openstack-nova | 01:28 | |
| sean-k-mooney | jaypipes: do you know what timezone sdauge is in? | 01:29 |
| jaypipes | EDT | 01:30 |
| jaypipes | sean-k-mooney: ^ | 01:30 |
| sean-k-mooney | ah so he will be offline for the night. its pretty late for you too right | 01:31 |
| *** zhurong has joined #openstack-nova | 01:31 | |
| jaypipes | sean-k-mooney: 21:30 here, yeah | 01:32 |
| sean-k-mooney | i need to open a minor bug with the new devstack systemd code | 01:32 |
| sean-k-mooney | it almost works across reboots except for https://github.com/openstack-dev/devstack/blob/a292c5068ce8b285afc1ecfd473c91c8789922d3/lib/apache#L231-L232 | 01:33 |
| sean-k-mooney | just need to have systemd create the folder automatically and it will work | 01:33 |
| sean-k-mooney | currently the placement api and keystone are fialing to start because of that | 01:34 |
| *** baoli has joined #openstack-nova | 01:35 | |
| *** yolkfull__ has joined #openstack-nova | 01:36 | |
| melwitt | mriedem: the bottom change should fail, I did that intentionally (temporarily) to show that it will work with the text fixes on top of it | 01:44 |
| melwitt | *test fixes | 01:45 |
| *** jamesdenton has joined #openstack-nova | 01:45 | |
| melwitt | once people are good with the bottom change, I'll flip the order | 01:46 |
| melwitt | gotta run for now | 01:46 |
| *** gouthamr has quit IRC | 01:48 | |
| *** thorst_afk has joined #openstack-nova | 01:49 | |
| *** zhurong has quit IRC | 01:50 | |
| mriedem | ok | 01:50 |
| *** gouthamr has joined #openstack-nova | 01:51 | |
| *** jamesdenton has quit IRC | 01:51 | |
| *** jamesdenton has joined #openstack-nova | 01:51 | |
| *** lbragsta_ has joined #openstack-nova | 01:54 | |
| *** lbragsta_ has quit IRC | 01:55 | |
| *** jamielennox is now known as jamielennox|away | 01:55 | |
| *** salv-orlando has joined #openstack-nova | 01:56 | |
| *** thorst_afk has quit IRC | 01:56 | |
| *** thorst_afk has joined #openstack-nova | 01:57 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Use plain routes list for server-password endpoint instead of stevedore https://review.openstack.org/462049 | 01:58 |
| *** salv-orlando has quit IRC | 02:00 | |
| *** thorst_afk has quit IRC | 02:02 | |
| *** yingjun has quit IRC | 02:03 | |
| *** salv-orlando has joined #openstack-nova | 02:04 | |
| *** Guest73837 has quit IRC | 02:06 | |
| mriedem | jaypipes: dansmith: edleafe: bauzas: claims in the scheduler forum etherpad https://etherpad.openstack.org/p/BOS-forum-move-claims-from-compute-to-scheduler | 02:08 |
| jaypipes | thx | 02:08 |
| *** salv-orlando has quit IRC | 02:09 | |
| *** gongysh has joined #openstack-nova | 02:10 | |
| *** Guest73837 has joined #openstack-nova | 02:10 | |
| *** jerrygb has joined #openstack-nova | 02:11 | |
| *** mdnadeem has joined #openstack-nova | 02:11 | |
| *** armax has quit IRC | 02:12 | |
| *** dtp has quit IRC | 02:15 | |
| mriedem | and now it's time for some veep | 02:15 |
| *** mriedem has quit IRC | 02:15 | |
| *** hongbin has joined #openstack-nova | 02:17 | |
| *** zsli_ has joined #openstack-nova | 02:18 | |
| *** jamielennox|away is now known as jamielennox | 02:19 | |
| *** Shunli has quit IRC | 02:21 | |
| *** yonglihe has joined #openstack-nova | 02:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: [POC] Searchlight intergration https://review.openstack.org/442248 | 02:24 |
| *** slaweq has joined #openstack-nova | 02:24 | |
| *** mingyu has joined #openstack-nova | 02:27 | |
| *** mdnadeem has quit IRC | 02:28 | |
| *** arunman has joined #openstack-nova | 02:28 | |
| *** ssurana has joined #openstack-nova | 02:28 | |
| *** slaweq has quit IRC | 02:29 | |
| *** alex_xu has joined #openstack-nova | 02:32 | |
| *** thorst_afk has joined #openstack-nova | 02:33 | |
| *** gouthamr has quit IRC | 02:37 | |
| *** zhurong has joined #openstack-nova | 02:39 | |
| *** vladikr has quit IRC | 02:39 | |
| *** marst_ has joined #openstack-nova | 02:44 | |
| *** marst has quit IRC | 02:45 | |
| *** mingyu has quit IRC | 02:45 | |
| *** zsli__ has joined #openstack-nova | 02:46 | |
| *** zsli_ has quit IRC | 02:46 | |
| *** nicolasbock has quit IRC | 02:46 | |
| *** mingyu has joined #openstack-nova | 02:47 | |
| *** thorst_afk has quit IRC | 02:51 | |
| *** zhurong has quit IRC | 02:54 | |
| *** zhurong has joined #openstack-nova | 02:54 | |
| *** mingyu has quit IRC | 02:55 | |
| *** mingyu has joined #openstack-nova | 02:56 | |
| *** iceyao has quit IRC | 03:00 | |
| *** salv-orlando has joined #openstack-nova | 03:05 | |
| *** brault has joined #openstack-nova | 03:06 | |
| *** jerrygb has quit IRC | 03:07 | |
| *** iceyao has joined #openstack-nova | 03:08 | |
| *** salv-orlando has quit IRC | 03:10 | |
| *** brault has quit IRC | 03:11 | |
| *** markvoelker has joined #openstack-nova | 03:11 | |
| *** vks1 has joined #openstack-nova | 03:12 | |
| *** jamesdenton has quit IRC | 03:12 | |
| *** dave-mccowan has quit IRC | 03:16 | |
| *** gouthamr has joined #openstack-nova | 03:17 | |
| *** baoli has quit IRC | 03:19 | |
| *** yolkfull__ is now known as yolkfull | 03:20 | |
| *** ljxiash has joined #openstack-nova | 03:22 | |
| *** mingyu has quit IRC | 03:23 | |
| *** ljxiash has quit IRC | 03:23 | |
| *** ljxiash has joined #openstack-nova | 03:24 | |
| *** mingyu has joined #openstack-nova | 03:24 | |
| *** mingyu has quit IRC | 03:24 | |
| *** mingyu has joined #openstack-nova | 03:25 | |
| *** armax has joined #openstack-nova | 03:25 | |
| *** cNilesh has joined #openstack-nova | 03:27 | |
| *** ljxiash has quit IRC | 03:28 | |
| *** fragatin_ has joined #openstack-nova | 03:29 | |
| *** fragatina has quit IRC | 03:32 | |
| *** fragatin_ has quit IRC | 03:33 | |
| *** fragatina has joined #openstack-nova | 03:35 | |
| *** mingyu has quit IRC | 03:38 | |
| *** mingyu has joined #openstack-nova | 03:38 | |
| *** fragatina has quit IRC | 03:39 | |
| *** yamamoto has joined #openstack-nova | 03:42 | |
| *** scottda has quit IRC | 03:45 | |
| *** mingyu has quit IRC | 03:47 | |
| *** mingyu has joined #openstack-nova | 03:47 | |
| *** thorst_afk has joined #openstack-nova | 03:48 | |
| *** yamamoto has quit IRC | 03:50 | |
| *** dimtruck is now known as zz_dimtruck | 03:53 | |
| *** iceyao has quit IRC | 03:57 | |
| *** gouthamr has quit IRC | 03:58 | |
| *** mingyu has quit IRC | 03:59 | |
| *** zhurong has quit IRC | 04:00 | |
| *** armax has quit IRC | 04:00 | |
| *** fragatina has joined #openstack-nova | 04:00 | |
| *** mingyu has joined #openstack-nova | 04:01 | |
| *** fragatina has quit IRC | 04:02 | |
| *** fragatina has joined #openstack-nova | 04:02 | |
| *** fragatina has quit IRC | 04:04 | |
| *** fragatina has joined #openstack-nova | 04:04 | |
| *** salv-orlando has joined #openstack-nova | 04:06 | |
| *** thorst_afk has quit IRC | 04:07 | |
| *** salv-orlando has quit IRC | 04:11 | |
| *** slaweq has joined #openstack-nova | 04:12 | |
| *** rfolco has quit IRC | 04:15 | |
| *** slaweq has quit IRC | 04:16 | |
| *** gongysh has quit IRC | 04:20 | |
| *** marst_ has quit IRC | 04:20 | |
| *** cNilesh has quit IRC | 04:22 | |
| *** hongbin has quit IRC | 04:22 | |
| *** iceyao has joined #openstack-nova | 04:23 | |
| *** abhi89 has joined #openstack-nova | 04:23 | |
| *** cNilesh has joined #openstack-nova | 04:25 | |
| *** iceyao has quit IRC | 04:27 | |
| *** armax has joined #openstack-nova | 04:27 | |
| *** zhurong has joined #openstack-nova | 04:32 | |
| *** nmathew has joined #openstack-nova | 04:36 | |
| *** hongbin has joined #openstack-nova | 04:37 | |
| *** iceyao has joined #openstack-nova | 04:38 | |
| *** trinaths has joined #openstack-nova | 04:39 | |
| *** armax has quit IRC | 04:41 | |
| *** salv-orlando has joined #openstack-nova | 04:45 | |
| *** Yangyapeng has quit IRC | 04:47 | |
| *** sridharg has joined #openstack-nova | 04:48 | |
| *** Yangyapeng has joined #openstack-nova | 04:48 | |
| *** ayogi has joined #openstack-nova | 04:49 | |
| *** mdavidson has quit IRC | 04:49 | |
| *** sree has joined #openstack-nova | 04:50 | |
| *** trinaths has quit IRC | 04:51 | |
| *** zsli__ has quit IRC | 04:56 | |
| *** zsli__ has joined #openstack-nova | 04:57 | |
| *** trinaths has joined #openstack-nova | 05:00 | |
| *** huanxie has joined #openstack-nova | 05:02 | |
| *** mdavidson has joined #openstack-nova | 05:02 | |
| *** thorst_afk has joined #openstack-nova | 05:03 | |
| *** brault has joined #openstack-nova | 05:07 | |
| *** thorst_afk has quit IRC | 05:08 | |
| *** gaurangt_ has quit IRC | 05:10 | |
| *** mingyu has quit IRC | 05:11 | |
| *** brault has quit IRC | 05:11 | |
| *** links has joined #openstack-nova | 05:13 | |
| *** iceyao has quit IRC | 05:13 | |
| *** tongli has quit IRC | 05:14 | |
| *** adisky_ has joined #openstack-nova | 05:16 | |
| *** cNilesh has quit IRC | 05:17 | |
| *** cNilesh has joined #openstack-nova | 05:18 | |
| *** yamahata has quit IRC | 05:19 | |
| *** gongysh has joined #openstack-nova | 05:19 | |
| *** yamahata has joined #openstack-nova | 05:20 | |
| *** hongbin has quit IRC | 05:20 | |
| *** Jack_Iv has joined #openstack-nova | 05:26 | |
| *** cNilesh has quit IRC | 05:27 | |
| *** tovin07_ has joined #openstack-nova | 05:27 | |
| *** Jack_Iv has quit IRC | 05:30 | |
| *** mdnadeem has joined #openstack-nova | 05:37 | |
| *** prateek has joined #openstack-nova | 05:42 | |
| *** mingyu has joined #openstack-nova | 05:45 | |
| *** tuanluong has quit IRC | 05:46 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Totally freeze the extension_info API https://review.openstack.org/462825 | 05:53 |
| alex_xu | gmann: johnthetubaguy ^ let us freeze the extension API totally :) | 05:53 |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | 05:55 |
| *** Jack_Iv has joined #openstack-nova | 05:56 | |
| *** tuanluong has joined #openstack-nova | 05:59 | |
| *** lucky__ has joined #openstack-nova | 06:00 | |
| *** mlakat has joined #openstack-nova | 06:02 | |
| *** andreas_s has joined #openstack-nova | 06:02 | |
| *** mingyu has quit IRC | 06:03 | |
| *** links has quit IRC | 06:04 | |
| *** mingyu has joined #openstack-nova | 06:07 | |
| *** mingyu has quit IRC | 06:08 | |
| *** iceyao has joined #openstack-nova | 06:09 | |
| *** kiwi_rot has joined #openstack-nova | 06:10 | |
| *** salv-orlando has quit IRC | 06:10 | |
| *** pcaruana has joined #openstack-nova | 06:11 | |
| *** nmathew has quit IRC | 06:12 | |
| *** iceyao has quit IRC | 06:16 | |
| *** slaweq has joined #openstack-nova | 06:17 | |
| *** Oku_OS-away is now known as Oku_OS | 06:17 | |
| *** slaweq has quit IRC | 06:21 | |
| *** links has joined #openstack-nova | 06:22 | |
| *** voelzmo has joined #openstack-nova | 06:26 | |
| *** abhishek has joined #openstack-nova | 06:27 | |
| *** abhi89 has quit IRC | 06:27 | |
| *** yolkfull has quit IRC | 06:30 | |
| *** yolkfull has joined #openstack-nova | 06:31 | |
| *** zhurong has quit IRC | 06:33 | |
| *** edmondsw has joined #openstack-nova | 06:36 | |
| *** gongysh has quit IRC | 06:37 | |
| *** mingyu has joined #openstack-nova | 06:39 | |
| *** trinaths has left #openstack-nova | 06:41 | |
| *** edmondsw has quit IRC | 06:41 | |
| *** hao has joined #openstack-nova | 06:41 | |
| *** ltomasbo|away is now known as ltomasbo | 06:42 | |
| hao | please have a look at https://review.openstack.org/#/c/456865/ | 06:43 |
| *** hao_ has joined #openstack-nova | 06:43 | |
| *** hao has quit IRC | 06:44 | |
| *** zhurong has joined #openstack-nova | 06:48 | |
| *** brault has joined #openstack-nova | 06:49 | |
| *** hao_ has quit IRC | 06:50 | |
| *** gjayavelu has joined #openstack-nova | 06:51 | |
| *** brault has quit IRC | 06:53 | |
| *** mingyu_ has joined #openstack-nova | 06:55 | |
| *** links has quit IRC | 06:55 | |
| *** mingyu has quit IRC | 06:57 | |
| *** bkopilov_ has quit IRC | 06:57 | |
| *** ygl has joined #openstack-nova | 07:01 | |
| *** thorst_afk has joined #openstack-nova | 07:05 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: [placement] Add api-ref for /resource_classes/{name} https://review.openstack.org/462837 | 07:06 |
| *** salv-orlando has joined #openstack-nova | 07:07 | |
| *** tesseract has joined #openstack-nova | 07:07 | |
| *** thorst_afk has quit IRC | 07:09 | |
| *** Jack_Iv has quit IRC | 07:10 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: [placement] Add api-ref for /resource_classes/{name} https://review.openstack.org/462837 | 07:10 |
| openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add POST for resource provider classes https://review.openstack.org/462441 | 07:10 |
| openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add api-ref for GET /resource_classes https://review.openstack.org/451661 | 07:10 |
| *** mingyu_ has quit IRC | 07:11 | |
| *** mingyu has joined #openstack-nova | 07:14 | |
| *** phuongnh has quit IRC | 07:15 | |
| *** phuongnh has joined #openstack-nova | 07:15 | |
| *** ssurana has quit IRC | 07:15 | |
| *** damien_r has joined #openstack-nova | 07:16 | |
| *** gongysh has joined #openstack-nova | 07:18 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: Add ability to view quota details for project owner https://review.openstack.org/386008 | 07:21 |
| *** trinaths has joined #openstack-nova | 07:22 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhancement comments on CountableResource https://review.openstack.org/402051 | 07:25 |
| *** mingyu has quit IRC | 07:25 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: [placement] Fix placement-api-ref check tool https://review.openstack.org/456198 | 07:25 |
| *** Jack_Iv has joined #openstack-nova | 07:28 | |
| *** lihao has joined #openstack-nova | 07:28 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova master: Update pci device fields from deleted object https://review.openstack.org/426243 | 07:30 |
| *** markus_z has joined #openstack-nova | 07:30 | |
| *** gjayavelu has quit IRC | 07:30 | |
| *** lpetrut has joined #openstack-nova | 07:35 | |
| *** jpena|off is now known as jpena | 07:42 | |
| *** aarefiev_afk is now known as aarefiev | 07:43 | |
| *** ralonsoh has joined #openstack-nova | 07:44 | |
| *** mingyu has joined #openstack-nova | 07:45 | |
| *** mingyu has quit IRC | 07:49 | |
| *** gszasz has joined #openstack-nova | 07:52 | |
| *** zsli_ has joined #openstack-nova | 07:53 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove the rebuild extension help methods https://review.openstack.org/451346 | 07:55 |
| *** zsli__ has quit IRC | 07:55 | |
| *** liusheng has quit IRC | 07:56 | |
| *** dixiaoli has quit IRC | 07:56 | |
| *** liusheng has joined #openstack-nova | 07:56 | |
| *** markus_z has quit IRC | 07:56 | |
| *** dixiaoli has joined #openstack-nova | 07:57 | |
| *** links has joined #openstack-nova | 07:57 | |
| *** bkopilov_ has joined #openstack-nova | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Totally freeze the extension_info API https://review.openstack.org/462825 | 08:00 |
| *** markus_z has joined #openstack-nova | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** karimb has joined #openstack-nova | 08:01 | |
| *** mingyu has joined #openstack-nova | 08:03 | |
| *** mingyu has quit IRC | 08:05 | |
| *** mingyu has joined #openstack-nova | 08:06 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 08:06 |
| *** thorst_afk has joined #openstack-nova | 08:07 | |
| *** aloga_ has joined #openstack-nova | 08:07 | |
| *** aloga_ has quit IRC | 08:09 | |
| *** aloga_ has joined #openstack-nova | 08:09 | |
| *** fragatina has quit IRC | 08:11 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova master: service: use restart_method='mutate' for all services https://review.openstack.org/462600 | 08:17 |
| *** slaweq has joined #openstack-nova | 08:18 | |
| *** brault has joined #openstack-nova | 08:19 | |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Change the parameter of the exception error message https://review.openstack.org/461985 | 08:19 |
| *** slaweq has quit IRC | 08:23 | |
| *** thorst_afk has quit IRC | 08:25 | |
| ygl | Hi all | 08:26 |
| ygl | can some one explain to me why boot from volume is faster than boot from image ? | 08:26 |
| *** links has quit IRC | 08:27 | |
| *** derekh has joined #openstack-nova | 08:32 | |
| *** lihao has quit IRC | 08:32 | |
| *** markus_z has quit IRC | 08:33 | |
| *** markus_z has joined #openstack-nova | 08:35 | |
| *** lpetrut_ has joined #openstack-nova | 08:39 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-migrations endpoint instead of stevedore https://review.openstack.org/461628 | 08:40 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 08:40 |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 08:40 |
| openstackgerrit | sahid proposed openstack/nova master: network: add command to configure trusted mode for VFs https://review.openstack.org/458513 | 08:40 |
| *** lpetrut has quit IRC | 08:42 | |
| *** cdent has joined #openstack-nova | 08:49 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Use plain routes list for os-services endpoint instead of stevedore https://review.openstack.org/460818 | 08:54 |
| *** salv-orl_ has joined #openstack-nova | 08:56 | |
| *** salv-orlando has quit IRC | 08:59 | |
| *** mingyu has quit IRC | 09:01 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Pass instance to connect_volume and disconnect_volume https://review.openstack.org/450383 | 09:05 |
| mdbooth | johnthetubaguy: ^^^ was a rebase after a merge conflict. Would you mind a fast re-approve? | 09:06 |
| johnthetubaguy | sure | 09:09 |
| mdbooth | Thanks | 09:09 |
| *** sshnaidm is now known as sshnaidm|off | 09:09 | |
| *** mingyu has joined #openstack-nova | 09:10 | |
| *** nmathew has joined #openstack-nova | 09:10 | |
| mdbooth | Does anybody else use os-log-merger, btw? | 09:10 |
| *** ociuhandu has quit IRC | 09:11 | |
| *** lucky__ has quit IRC | 09:13 | |
| *** thorst_afk has joined #openstack-nova | 09:22 | |
| *** zsli_ has quit IRC | 09:23 | |
| *** mingyu has quit IRC | 09:23 | |
| cdent | mdbooth: a) first I've heard of it but b) probably would have if I had known about it | 09:26 |
| mdbooth | I've been using it for a bit, just about to start submitting patches | 09:26 |
| cdent | there's a lot of opinion about logs: separate is better! merged is better! stop it you grapes, use journald! | 09:26 |
| *** thorst_afk has quit IRC | 09:26 | |
| cdent | etc | 09:26 |
| mdbooth | I'd previously written my own which performed the same function | 09:27 |
| *** Oku_OS is now known as Oku_OS-away | 09:27 | |
| mdbooth | Merged is the only way to go in my experience :) | 09:27 |
| * mdbooth very much enjoys being able to follow a single request id across multiple merged services in a single log stream | 09:27 | |
| kashyap | mdbooth: Yeah, first ajo called it "oslogmerger" | 09:28 |
| *** gszasz has quit IRC | 09:28 | |
| kashyap | My brain used to read it as "Oslo GMerger" | 09:28 |
| mdbooth | Unfortunately, os-log-merger unfortunately uses stupid sorting which makes it slow and a memory hog | 09:28 |
| mdbooth | But it suffices and I don't have to maintain it myself | 09:29 |
| mdbooth | The one I wrote in bash and awk was a lot faster, if a little obtuse :) | 09:29 |
| * mdbooth might patch it to use a proper merge sort | 09:30 | |
| kashyap | Ah, os-log-merger is part of OpenStack upstream repos | 09:30 |
| mdbooth | Yep | 09:30 |
| kashyap | mdbooth: Your `bash` / `awk` equivalent, all lives in one file I suppose? | 09:31 |
| mdbooth | Yep | 09:31 |
| mdbooth | http://paste.openstack.org/show/608943/ | 09:32 |
| *** Oku_OS-away is now known as Oku_OS | 09:33 | |
| mdbooth | Basically it creates an awk process per input file which munges the inputs, then merge sorts the output of the multiple awk processes | 09:34 |
| mdbooth | Like I said, it's very fast but a little obtuse :) | 09:34 |
| mdbooth | Also not very flexible with multiple timestamp formats | 09:34 |
| * kashyap clicks | 09:35 | |
| kashyap | What's obtuse in that line! :P - | 09:36 |
| kashyap | if ($2 ~ /[0-9]{2}:[0-9]{2}:[0-9]{2}\.[0-9]{3}/) | 09:36 |
| openstackgerrit | NidhiMittalHada proposed openstack/python-novaclient master: Nova volume-attach doesn't care for device name https://review.openstack.org/454048 | 09:36 |
| *** Dinesh_Bhor has quit IRC | 09:36 | |
| mdbooth | kashyap: What? Nothing could be clearer! | 09:36 |
| * mdbooth has written a *lot* of perl :) | 09:36 | |
| kashyap | I recall you've done Tcl, too | 09:37 |
| mdbooth | Indeed | 09:37 |
| mdbooth | And Java, C, and OCaml. | 09:37 |
| mdbooth | Professionally, anyway. | 09:38 |
| * sfinucan wonders how mdbooth can say one bad thing about Python having worked with Perl ;) | 09:38 | |
| sfinucan | ...and Tcl, for that matter | 09:38 |
| sfinucan | everything's a string, my ass | 09:38 |
| kashyap | LOL | 09:39 |
| mdbooth | sfinucan: Perl doesn't have a bug up it's arse about being a proper language :P | 09:39 |
| mdbooth | It's right there in the name | 09:39 |
| mdbooth | It's a disgusting bunch of hacks for sysadmins to use to get stuff done quickly with | 09:39 |
| mdbooth | Same as Python, but python is delusional | 09:39 |
| *** ericyoung has joined #openstack-nova | 09:40 | |
| *** eric_young has joined #openstack-nova | 09:40 | |
| sfinucan | mdbooth: We had a test framework for OVS when I worked in Intel | 09:41 |
| sfinucan | Interacted with some (outrageously expensive) IXIA traffic generators to run nightly performance benchmarking tests | 09:41 |
| sfinucan | Bash. Thousands of lines of bash. | 09:42 |
| mdbooth | Haha | 09:42 |
| mdbooth | My rule of bash is that as soon as you need the manpage, you need to be planning a rewrite :) | 09:42 |
| sfinucan | with some Tcl thrown in because IXIA knew how to build traffic generators but had zero idea about writing usable APIs | 09:42 |
| sfinucan | :D very true | 09:43 |
| sfinucan | I insist of 'set -e' at the top of any script | 09:43 |
| *** ericyoung has quit IRC | 09:44 | |
| *** eric_young has quit IRC | 09:44 | |
| sfinucan | *set -u, rather | 09:44 |
| *** ericyoung has joined #openstack-nova | 09:44 | |
| * mdbooth would have to check the manpage to recall which flag that is | 09:44 | |
| kashyap | "Treat unset variables and parameters other than the special parameters ‘@’ or ‘*’ as an error when performing parameter expansion. An error message will be written to the standard error, and a non-interactive shell will exit." | 09:45 |
| sfinucan | -e is quit on any exit | 09:45 |
| sfinucan | -u is that ^ :) | 09:45 |
| kashyap | sfinucan: Yeah, was wondering about '-e', which is about exit rules involving pipelines | 09:45 |
| sfinucan | *uit on any error | 09:45 |
| *** dixiaoli has quit IRC | 09:45 | |
| * mdbooth knows -e and -x, anything else is voodoo | 09:45 | |
| *** hieulq_ has joined #openstack-nova | 09:46 | |
| sfinucan | Still better than what my college buddies who chose the hardware engineering track have to deal with | 09:46 |
| sfinucan | Hardware testers with the cruftiest APIs you've ever seen (many using VB, oddly), and none of the skills to write wrappers and make things usable :( poor guys | 09:47 |
| * sfinucan stops moaning about the state of testing equipment APIs and finds patches to review | 09:48 | |
| mdbooth | sfinucan: But it's Friday :) Also, I feel like I haven't moaned about Python in months. | 09:49 |
| sfinucan | mdbooth: It is! And the suns out and people are cutting their lawns. No better day to embrace the way of Python ;) | 09:50 |
| mdbooth | Hehe, from datetime: "An aware object is used to represent a specific moment in time that is not open to interpretation [1]." | 09:50 |
| mdbooth | "[1]If, that is, we ignore the effects of Relativity" | 09:50 |
| * mdbooth wonders who nerded the author to include that. | 09:51 | |
| *** mingyu has joined #openstack-nova | 09:52 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | 09:54 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: Don't delete allocation if instance being scheduled https://review.openstack.org/460177 | 09:54 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | 09:54 |
| *** salv-orl_ has quit IRC | 09:55 | |
| *** zhurong has quit IRC | 09:56 | |
| *** trungnv has quit IRC | 09:56 | |
| *** mingyu has quit IRC | 09:57 | |
| * cdent adds the verb "to nerd" to his lexicon | 09:57 | |
| *** tovin07_ has quit IRC | 09:58 | |
| mdbooth | cdent: I'd copyright it, but I expect it's not original | 09:59 |
| cdent | seems likely to be plenty of independent developments | 10:00 |
| *** hieulq_ has quit IRC | 10:01 | |
| johnthetubaguy | we may have just out nerded that author | 10:01 |
| mdbooth | lol | 10:02 |
| cdent | \o/ | 10:02 |
| sfinucan | 🙈 | 10:02 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257 | 10:02 |
| openstackgerrit | suzhengwei proposed openstack/nova master: compute_model problem https://review.openstack.org/462897 | 10:02 |
| *** markvoelker has quit IRC | 10:03 | |
| openstackgerrit | suzhengwei proposed openstack/nova master: compute_model problem https://review.openstack.org/462897 | 10:04 |
| *** sree_ has joined #openstack-nova | 10:05 | |
| *** sree_ is now known as Guest72934 | 10:06 | |
| *** nicolasbock has joined #openstack-nova | 10:08 | |
| *** ociuhandu has joined #openstack-nova | 10:08 | |
| *** sree has quit IRC | 10:08 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Delete existing allocations for move operations or retries https://review.openstack.org/460433 | 10:11 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Allocate instance by the conductor https://review.openstack.org/460455 | 10:11 |
| *** Yangyapeng has quit IRC | 10:12 | |
| *** mingyu has joined #openstack-nova | 10:13 | |
| *** slaweq has joined #openstack-nova | 10:19 | |
| *** Oku_OS is now known as Oku_OS-away | 10:20 | |
| *** abhi89 has joined #openstack-nova | 10:21 | |
| *** smatzek has joined #openstack-nova | 10:22 | |
| *** abhishek has quit IRC | 10:23 | |
| *** smatzek has quit IRC | 10:24 | |
| *** slaweq has quit IRC | 10:24 | |
| *** smatzek has joined #openstack-nova | 10:24 | |
| *** Jack_Iv has quit IRC | 10:25 | |
| *** aloga_ has quit IRC | 10:31 | |
| *** Oku_OS-away is now known as Oku_OS | 10:32 | |
| *** Jack_Iv has joined #openstack-nova | 10:32 | |
| *** mdnadeem has quit IRC | 10:33 | |
| *** mingyu has quit IRC | 10:34 | |
| *** phuongnh has quit IRC | 10:35 | |
| *** mingyu has joined #openstack-nova | 10:36 | |
| *** gszasz has joined #openstack-nova | 10:38 | |
| bauzas | cdent: good point on the allocations being deleted when adding new ones | 10:40 |
| cdent | bauzas: It's kind of non-obvious | 10:42 |
| bauzas | cdent: I think that's the main problem | 10:42 |
| bauzas | cdent: maybe we should be explicit ? | 10:42 |
| *** ayogi has quit IRC | 10:42 | |
| bauzas | cdent: also, the problem is that if the scheduler doesn't provide a new destination, then we should still have the wrong allocation | 10:43 |
| cdent | well it depends on how we want things to mean. The way in which it is possible to replace allocations is something that we added in the late rush at the end of newton | 10:43 |
| bauzas | cdent: so, probably something being discussed at the summit | 10:43 |
| cdent | I do agree that managing the information explicitly is usually a good idea | 10:43 |
| *** tbachman has quit IRC | 10:44 | |
| *** brault_ has joined #openstack-nova | 10:48 | |
| *** brault has quit IRC | 10:51 | |
| *** mingyu has quit IRC | 10:56 | |
| *** yamahata has quit IRC | 10:56 | |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | 10:58 |
| openstackgerrit | falseuser proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | 11:03 |
| *** tuanluong has quit IRC | 11:04 | |
| *** markvoelker has joined #openstack-nova | 11:04 | |
| *** yamamoto has joined #openstack-nova | 11:05 | |
| *** jaypipes has quit IRC | 11:05 | |
| *** gongysh has quit IRC | 11:08 | |
| *** markvoelker has quit IRC | 11:10 | |
| *** Guest72934 has quit IRC | 11:10 | |
| *** yamamoto has quit IRC | 11:13 | |
| *** yolkfull has quit IRC | 11:17 | |
| *** jerrygb has joined #openstack-nova | 11:18 | |
| *** brault_ has quit IRC | 11:20 | |
| *** yamamoto has joined #openstack-nova | 11:20 | |
| *** ayogi has joined #openstack-nova | 11:23 | |
| *** yamamoto has quit IRC | 11:23 | |
| *** sree has joined #openstack-nova | 11:23 | |
| *** sree has quit IRC | 11:28 | |
| *** Qiming has quit IRC | 11:31 | |
| *** yamahata has joined #openstack-nova | 11:31 | |
| *** vks1 has quit IRC | 11:32 | |
| *** Qiming has joined #openstack-nova | 11:33 | |
| *** sdague has joined #openstack-nova | 11:39 | |
| *** tesseract has quit IRC | 11:43 | |
| *** tesseract has joined #openstack-nova | 11:43 | |
| *** nmathew has quit IRC | 11:43 | |
| *** kevinz has quit IRC | 11:44 | |
| *** thorst_afk has joined #openstack-nova | 11:45 | |
| *** gjayavelu has joined #openstack-nova | 11:45 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: remove scality volume driver https://review.openstack.org/449364 | 11:45 |
| *** brault has joined #openstack-nova | 11:47 | |
| *** salv-orlando has joined #openstack-nova | 11:48 | |
| ygl | hi all | 11:49 |
| *** tbachman has joined #openstack-nova | 11:49 | |
| ygl | can anyone explain to me why boot from volume is faster than boot from image ? | 11:49 |
| *** lucasagomes is now known as lucas-hungry | 11:51 | |
| *** cdent has quit IRC | 11:52 | |
| *** zz_dimtruck is now known as dimtruck | 11:54 | |
| *** rfolco has joined #openstack-nova | 11:54 | |
| *** ayogi has quit IRC | 11:56 | |
| *** tbachman has quit IRC | 11:58 | |
| *** tbachman has joined #openstack-nova | 11:58 | |
| *** jpena is now known as jpena|lunch | 11:59 | |
| *** Qiming has quit IRC | 12:03 | |
| *** jaypipes has joined #openstack-nova | 12:04 | |
| *** karimb has quit IRC | 12:04 | |
| *** markvoelker has joined #openstack-nova | 12:07 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename qemu_enable_gluster_storage_driver https://review.openstack.org/447091 | 12:07 |
| *** prateek has quit IRC | 12:08 | |
| *** yamamoto has joined #openstack-nova | 12:08 | |
| *** yamamoto has quit IRC | 12:08 | |
| *** voelzmo has quit IRC | 12:09 | |
| *** markvoelker has quit IRC | 12:11 | |
| *** voelzmo has joined #openstack-nova | 12:12 | |
| *** pchavva has joined #openstack-nova | 12:13 | |
| *** karthiks has joined #openstack-nova | 12:17 | |
| *** gcb has quit IRC | 12:18 | |
| *** cdent has joined #openstack-nova | 12:18 | |
| *** jerrygb has quit IRC | 12:23 | |
| *** salv-orlando has quit IRC | 12:24 | |
| *** jerrygb has joined #openstack-nova | 12:25 | |
| *** yamahata has quit IRC | 12:27 | |
| *** Qiming has joined #openstack-nova | 12:29 | |
| *** gcb has joined #openstack-nova | 12:30 | |
| *** andreas_s has quit IRC | 12:31 | |
| *** liusheng has quit IRC | 12:32 | |
| *** liusheng has joined #openstack-nova | 12:34 | |
| *** edmondsw has joined #openstack-nova | 12:35 | |
| *** aloga_ has joined #openstack-nova | 12:35 | |
| *** Yangyapeng has joined #openstack-nova | 12:37 | |
| *** gcb has quit IRC | 12:40 | |
| *** liusheng has quit IRC | 12:40 | |
| *** liusheng has joined #openstack-nova | 12:40 | |
| *** liusheng has quit IRC | 12:41 | |
| *** vladikr has joined #openstack-nova | 12:41 | |
| *** liusheng has joined #openstack-nova | 12:42 | |
| *** winston-d_ has quit IRC | 12:43 | |
| *** nmathew has joined #openstack-nova | 12:45 | |
| *** nmathew has quit IRC | 12:45 | |
| *** yamamoto has joined #openstack-nova | 12:48 | |
| *** gjayavelu has quit IRC | 12:48 | |
| *** lucas-hungry is now known as lucasagomes | 12:50 | |
| *** liusheng has quit IRC | 12:51 | |
| *** liusheng has joined #openstack-nova | 12:51 | |
| *** cdent_ has joined #openstack-nova | 12:53 | |
| *** lpetrut__ has joined #openstack-nova | 12:54 | |
| *** damien_r1 has joined #openstack-nova | 12:54 | |
| *** gcb has joined #openstack-nova | 12:56 | |
| *** dave-mccowan has joined #openstack-nova | 12:57 | |
| *** jpena|lunch is now known as jpena | 12:58 | |
| *** edleafe is now known as figleaf | 13:00 | |
| *** gcb has quit IRC | 13:02 | |
| *** cdent has quit IRC | 13:02 | |
| *** tesseract has quit IRC | 13:02 | |
| *** smatzek has quit IRC | 13:02 | |
| *** lpetrut_ has quit IRC | 13:02 | |
| *** markus_z has quit IRC | 13:02 | |
| *** damien_r has quit IRC | 13:02 | |
| *** yonglihe has quit IRC | 13:02 | |
| *** Guest73837 has quit IRC | 13:02 | |
| *** crushil has quit IRC | 13:02 | |
| *** wxy has quit IRC | 13:02 | |
| *** aweeks has quit IRC | 13:02 | |
| *** coreywright has quit IRC | 13:02 | |
| *** cdent_ is now known as cdent | 13:02 | |
| *** crushil has joined #openstack-nova | 13:03 | |
| *** aloga_ has quit IRC | 13:06 | |
| *** markus_z has joined #openstack-nova | 13:09 | |
| *** coreywright has joined #openstack-nova | 13:10 | |
| *** trinaths has left #openstack-nova | 13:10 | |
| *** cfriesen has joined #openstack-nova | 13:12 | |
| *** mriedem has joined #openstack-nova | 13:12 | |
| *** aloga_ has joined #openstack-nova | 13:12 | |
| *** yamamoto has quit IRC | 13:12 | |
| *** catintheroof has joined #openstack-nova | 13:13 | |
| *** lbragstad has quit IRC | 13:13 | |
| *** tesseract has joined #openstack-nova | 13:14 | |
| *** smatzek has joined #openstack-nova | 13:14 | |
| *** yonglihe has joined #openstack-nova | 13:14 | |
| *** Guest73837 has joined #openstack-nova | 13:14 | |
| *** wxy has joined #openstack-nova | 13:14 | |
| *** aweeks has joined #openstack-nova | 13:14 | |
| *** kristian__ has joined #openstack-nova | 13:14 | |
| *** efried has quit IRC | 13:16 | |
| *** strigazi has joined #openstack-nova | 13:17 | |
| *** liverpooler has joined #openstack-nova | 13:19 | |
| *** jaypipes is now known as leakypipes | 13:19 | |
| *** aloga_ has quit IRC | 13:21 | |
| *** sree has joined #openstack-nova | 13:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: [BugFix] Release the memory quota for video ram when deleting an instance. https://review.openstack.org/456050 | 13:23 |
| *** lbragstad has joined #openstack-nova | 13:26 | |
| *** ygl has quit IRC | 13:26 | |
| *** tesseract has quit IRC | 13:26 | |
| *** smatzek has quit IRC | 13:26 | |
| *** yonglihe has quit IRC | 13:26 | |
| *** Guest73837 has quit IRC | 13:26 | |
| *** wxy has quit IRC | 13:26 | |
| *** aweeks has quit IRC | 13:26 | |
| *** efried has joined #openstack-nova | 13:26 | |
| *** Guest73837 has joined #openstack-nova | 13:26 | |
| *** yonglihe has joined #openstack-nova | 13:27 | |
| *** aweeks has joined #openstack-nova | 13:27 | |
| *** tesseract has joined #openstack-nova | 13:27 | |
| *** sree has quit IRC | 13:27 | |
| *** wxy has joined #openstack-nova | 13:28 | |
| *** ralonsoh has quit IRC | 13:31 | |
| *** sree has joined #openstack-nova | 13:31 | |
| *** brault has quit IRC | 13:33 | |
| *** brault has joined #openstack-nova | 13:33 | |
| *** prateek has joined #openstack-nova | 13:34 | |
| *** jamesdenton has joined #openstack-nova | 13:35 | |
| *** xinliang has quit IRC | 13:36 | |
| *** zul has quit IRC | 13:37 | |
| *** zul has joined #openstack-nova | 13:42 | |
| *** iceyao has joined #openstack-nova | 13:43 | |
| *** dpawlik has joined #openstack-nova | 13:44 | |
| *** sshnaidm|off has quit IRC | 13:45 | |
| *** abhi89 has quit IRC | 13:45 | |
| *** crushil has quit IRC | 13:46 | |
| *** dimtruck is now known as zz_dimtruck | 13:47 | |
| dansmith | mriedem: you opened this bug: https://bugs.launchpad.net/nova/+bug/1656056 | 13:47 |
| openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "duplicate for #1656056 Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 13:47 |
| *** ljxiash has joined #openstack-nova | 13:47 | |
| dansmith | mriedem: which was marked as a dupe of: https://bugs.launchpad.net/nova/+bug/1657774 | 13:48 |
| openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 13:48 |
| dansmith | which has a pending change: https://review.openstack.org/#/c/422696 | 13:48 |
| mriedem | and you have a customer that needs it fixed *today* | 13:48 |
| dansmith | I'm not sure I grok what is going on | 13:48 |
| dansmith | no | 13:48 |
| mriedem | i thought sahid "fixed" this awhile ago | 13:48 |
| dansmith | is the problem token expiry or something? | 13:48 |
| *** jamesdenton has quit IRC | 13:48 | |
| dansmith | yeah, I remember that | 13:49 |
| dansmith | that was before your bug was filed though | 13:49 |
| *** lucasxu has joined #openstack-nova | 13:49 | |
| dansmith | sahid's thing was token expiration during the operation I think | 13:49 |
| mriedem | well, | 13:49 |
| mriedem | there was also a general non-useful error if you configured the neutron creds in nova.conf incorrectly i think | 13:50 |
| mriedem | there was an old bug for that from a long time ago | 13:50 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1656056 was from something i saw in gate runs | 13:50 |
| openstack | Launchpad bug 1657774 in OpenStack Compute (nova) "duplicate for #1656056 Nova does not re-raise 401 Unauthorized received from Neutron for admin users" [Undecided,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 13:50 |
| mriedem | i did some investigation but didn't dupe it myself | 13:50 |
| mriedem | i couldn't figure out what was going on at the time when i was triaging it | 13:50 |
| *** xinliang has joined #openstack-nova | 13:50 | |
| *** burt has joined #openstack-nova | 13:50 | |
| dansmith | so the commit message makes it sound like something where we don't raise 401 if the token is an admin, even if we didn't get it from the nova.conf ? | 13:50 |
| lbragstad | if the token is being expired during the operation i can take a poke at it - we should be able to fix that from a service user perspective if that helps | 13:50 |
| *** iceyao has quit IRC | 13:51 | |
| rpodolyaka | dansmith: yeah, that's the case, IIRC | 13:51 |
| mriedem | well, this is why we have the service user token spec | 13:51 |
| *** iceyao has joined #openstack-nova | 13:51 | |
| rpodolyaka | I saw this on my env on requests done by ceilometer polling agent | 13:52 |
| mriedem | https://blueprints.launchpad.net/nova/+spec/use-service-tokens-pike is for using a service token for long-running operations | 13:52 |
| lbragstad | aha - sure | 13:52 |
| rpodolyaka | basically , ceilometer agent uses one token until it expires | 13:52 |
| *** jamesdenton has joined #openstack-nova | 13:52 | |
| rpodolyaka | and instead of getting a proper 401, it gets 500 | 13:52 |
| rpodolyaka | in this corner case | 13:52 |
| lbragstad | we also have an endpoint in keystone that allows users with the service role to validate expired tokens | 13:52 |
| *** ljxiash has quit IRC | 13:53 | |
| mriedem | rpodolyaka: so this sounds like a side effect of sahid's change? | 13:53 |
| dansmith | rpodolyaka: so we should merge this so we can backport even though the service user thing should fix it? | 13:53 |
| dansmith | well, if it's used properly I guess | 13:53 |
| *** baoli has joined #openstack-nova | 13:53 | |
| mriedem | where it's not distinguishing between is_admin on the user context and the admin creds in nova.conf? | 13:53 |
| rpodolyaka | mriedem: yep, this ^ | 13:53 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 13:54 |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 13:54 |
| *** smatzek has joined #openstack-nova | 13:55 | |
| rpodolyaka | dansmith: I haven't seen the service user spec :( this seems to be a simple fix - we use it in downstream mitaka | 13:55 |
| rpodolyaka | *easy to backport | 13:55 |
| * leakypipes leaves for Boston... | 13:55 | |
| dansmith | rpodolyaka: yeah, this seems like just a bad assumption made in the when-to-raise code | 13:55 |
| *** leakypipes has left #openstack-nova | 13:55 | |
| *** eharney has joined #openstack-nova | 13:55 | |
| *** iceyao has quit IRC | 13:56 | |
| rpodolyaka | yeah, there was two different conditions in two different places | 13:56 |
| rpodolyaka | hopefully `use_service_credentials` now covers it properly in https://review.openstack.org/#/c/422696/3/nova/network/neutronv2/api.py | 13:57 |
| *** mingyu has joined #openstack-nova | 13:57 | |
| *** awaugama has joined #openstack-nova | 13:58 | |
| dansmith | the implicit "admin if admin context but no token" thing kinda scares me | 13:58 |
| rpodolyaka | ++ | 13:59 |
| dansmith | actually, I guess that already happens in the code | 13:59 |
| rpodolyaka | in the long term we should probably make sure we always pass admin=True to get_client() explicitly | 13:59 |
| dansmith | I guess the second hunk there is just trying to clean up the logic? | 13:59 |
| dansmith | nothing is actually changed I think | 13:59 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Check if domain is persistent before detaching volume https://review.openstack.org/441204 | 14:00 |
| *** mingyu has quit IRC | 14:01 | |
| johnthetubaguy | dansmith: thats refresh network_info I think, admin but no user context | 14:02 |
| johnthetubaguy | (and a few other periodic tasks) | 14:02 |
| dansmith | johnthetubaguy: metadata fetch is the example he gave | 14:02 |
| mriedem | ok i was hazy on that, | 14:03 |
| dansmith | johnthetubaguy: so does that patch seem reasonable? | 14:03 |
| johnthetubaguy | dansmith: true, although I assumed we just used the version cached in the instance for that | 14:03 |
| mriedem | so basically context.get_admin_context() and pass that down | 14:03 |
| mriedem | so it doesn't have a token, | 14:03 |
| dansmith | mriedem: right | 14:03 |
| mriedem | but is_admin=True | 14:03 |
| mriedem | ok | 14:03 |
| johnthetubaguy | I would tempted to have a re-raise_401=True? | 14:04 |
| mriedem | so in the commit message where it says we get the 401 b/c the token is expired, | 14:04 |
| mriedem | if we didn't have a token to begin with, | 14:05 |
| mriedem | is that referring to the token that neutronclient creates? | 14:05 |
| johnthetubaguy | I think its the user token passed into the API, that happens to be admin, but expires by the time we talk to neutron | 14:05 |
| johnthetubaguy | ... now if you enable service token auth (the second token) that whole crazy goes away | 14:05 |
| dansmith | johnthetubaguy: so that callers of this have to pass it to get proper behavior you mean? | 14:06 |
| johnthetubaguy | so, its a configuration option right now, to enable the service token to get sent with any user token | 14:06 |
| johnthetubaguy | (line 159 I guess) | 14:07 |
| mriedem | yeah so no one is using that yet, | 14:07 |
| mriedem | so we can ignore that part, | 14:07 |
| *** xinliang has quit IRC | 14:07 | |
| *** xyang1 has joined #openstack-nova | 14:07 | |
| mriedem | i'm also a bit lost where you said "I think its the user token passed into the API, that happens to be admin, but expires by the time we talk to neutron" | 14:07 |
| mriedem | i thought if you did context.get_admin_context() there is no token | 14:08 |
| dansmith | mriedem: correct | 14:08 |
| mriedem | hence the "and not context.auth_token" check | 14:08 |
| johnthetubaguy | oh wait, I kinda hate what this is doing | 14:08 |
| *** xinliang has joined #openstack-nova | 14:08 | |
| johnthetubaguy | I don't like converting what the user passed in, into our own service token | 14:09 |
| johnthetubaguy | I missed that bit | 14:09 |
| johnthetubaguy | well, miss-read it | 14:09 |
| *** zz_dimtruck is now known as dimtruck | 14:09 | |
| dansmith | converting what? | 14:09 |
| dansmith | the admin flag? | 14:09 |
| johnthetubaguy | so if the user passes in a token, I think we should pass that through to the next service | 14:10 |
| johnthetubaguy | what I think is happening here is we ignore the user token, and just use a service token instead, which sounds bad | 14:11 |
| *** iceyao has joined #openstack-nova | 14:11 | |
| dansmith | no, I don't think so | 14:11 |
| mriedem | if the user passes in a token, the "and not context.auth_token" is False | 14:11 |
| johnthetubaguy | actually, my brain is fried, its not doing that, got my admin's mixed up here | 14:11 |
| johnthetubaguy | admin=True means use service admin token, rather than context.auth_token, which we do anyways (for port binding reasons) | 14:12 |
| dansmith | this unifies the logic for determining if we use admin_auth with the logic for the admin= flag on ClientWrapper.. so they match | 14:12 |
| *** brault has quit IRC | 14:13 | |
| dansmith | it doesn't do anything other than make sure the ClientWrapper gets the right admin flag so that it does the right raise behavior I think | 14:14 |
| dansmith | L108 only I think | 14:14 |
| *** xinliang has quit IRC | 14:14 | |
| johnthetubaguy | yeah, sorry, I keep going around in silly loops | 14:15 |
| *** prateek has quit IRC | 14:15 | |
| johnthetubaguy | I wonder is that admin flag should be supress_401_error=True? | 14:15 |
| *** iceyao has quit IRC | 14:15 | |
| dansmith | oh is that what you meant? | 14:15 |
| dansmith | if ClientWrapper isn't used anywhere else, then that's probably an okay change | 14:16 |
| mriedem | that's pretty leaky tight coupling | 14:16 |
| mriedem | i'd rather change the admin kwarg in ClientWrapper to "use_service_credentials" or "using_service_credentials" or something | 14:16 |
| dansmith | mriedem: but ClientWrapper is really only used from right there and for this purpose | 14:16 |
| dansmith | mriedem: but that isn't right | 14:17 |
| dansmith | mriedem: it doesn't change that behavior | 14:17 |
| *** vks1 has joined #openstack-nova | 14:18 | |
| johnthetubaguy | looking at the exception name I am tempted by is_token_from_config=True/False | 14:19 |
| johnthetubaguy | in the client wrapper | 14:19 |
| *** Qiming has quit IRC | 14:19 | |
| dansmith | yeah that's more correct | 14:19 |
| johnthetubaguy | is_admin=False when context.is_admin = True, just doesn't sit right | 14:19 |
| dansmith | is_from_an_api_request=True makes most sense maybe | 14:20 |
| johnthetubaguy | yeah, that would work | 14:20 |
| johnthetubaguy | although that implies you would just check "if not context.auth_token" | 14:21 |
| openstackgerrit | Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753 | 14:21 |
| dansmith | or is_internal_request | 14:21 |
| dansmith | or is_nova_admin | 14:21 |
| mriedem | so in the long ago we used to just get the 401 from neutronclient and we didn't handle it, and we got a 500 in the compute api, | 14:21 |
| mriedem | then we handled the 401 from neutron and converted that to a nova 401 to fix the 500 | 14:22 |
| johnthetubaguy | then we made the config error back into a 500? | 14:22 |
| dansmith | in the long ago, we had nova-network. period. | 14:22 |
| mriedem | and we get the 401 now if your token expired (came in through the user context or you used nova.conf?) | 14:22 |
| johnthetubaguy | so its not 401 if you got the token from nova.conf right? | 14:23 |
| johnthetubaguy | should be 500 I think | 14:23 |
| *** avolkov has joined #openstack-nova | 14:23 | |
| *** Qiming has joined #openstack-nova | 14:23 | |
| mriedem | NeutronAdminCredentialConfigurationInvalid turns it into a 400 | 14:23 |
| johnthetubaguy | oh... | 14:23 |
| mriedem | i think this came up in the original review on sahid's change, because sdague thought the 500 was correct | 14:23 |
| johnthetubaguy | that feels like it should be 500 | 14:24 |
| mriedem | i.e. if you don't have nova.conf [neutron] configured correctly | 14:24 |
| mriedem | would have to dig back through the git history of time to find out how they changed sean's mind | 14:24 |
| dansmith | so the place I'm seeing this is in instance delete, | 14:24 |
| dansmith | which is failing and breaking the instance state | 14:24 |
| johnthetubaguy | which instance delete path? | 14:25 |
| dansmith | from compute manager while deallocating network | 14:25 |
| johnthetubaguy | oh, it takes a while, so gives the token plenty time to have timed out? | 14:26 |
| mriedem | johnthetubaguy: looks like this is our fault https://review.openstack.org/#/c/312014/ | 14:26 |
| johnthetubaguy | mriedem: I probably should have expected that | 14:26 |
| mriedem | https://review.openstack.org/#/c/312014/14/nova/network/neutronv2/api.py | 14:27 |
| *** eharney has quit IRC | 14:27 | |
| * johnthetubaguy nods in shame | 14:27 | |
| dansmith | johnthetubaguy: I dunno why it fails, it's from heat so heat might've gotten its token a long time ago or something | 14:27 |
| johnthetubaguy | dansmith: might be the trust based token has a shorter lifetime | 14:28 |
| johnthetubaguy | dansmith: the more general fix for this has merged already of course, you configure a service token to stop that expiry happening | 14:29 |
| dansmith | yeah | 14:29 |
| dansmith | can't backport that though | 14:30 |
| johnthetubaguy | so us triggering a 401 just changes the error code from 400, rather than make the delete work? | 14:31 |
| johnthetubaguy | or is this something deeper? | 14:31 |
| johnthetubaguy | heat does a retry, which fixes things? | 14:31 |
| *** Qiming has quit IRC | 14:32 | |
| dansmith | no, the instance is state locked at that point for some reason | 14:33 |
| dansmith | like, you can't retry the delete | 14:33 |
| dansmith | have to get more details on why | 14:33 |
| dansmith | anyway, my point is, I don't think that fixing what is in that patch actually helps me out of the box | 14:33 |
| dansmith | I'm guessing it's still task_state=deleting or something | 14:34 |
| johnthetubaguy | ah, dang | 14:34 |
| johnthetubaguy | would have to rollback the task state, I guess the instance might go into the ERROR state (or should) | 14:35 |
| *** Qiming has joined #openstack-nova | 14:35 | |
| *** aarefiev is now known as aarefiev_afk | 14:38 | |
| *** sree has quit IRC | 14:39 | |
| mriedem | dansmith: the service token thing is in ocata | 14:39 |
| mriedem | for neutron | 14:39 |
| *** lpetrut__ has quit IRC | 14:40 | |
| mriedem | unless you're talking about backporting to like, kilo | 14:40 |
| dansmith | johnthetubaguy: looks like it goes into error, but maybe not resetting the task_state or something | 14:40 |
| dansmith | mriedem: yeah, before ocata I think | 14:40 |
| mriedem | failures in terminate_instance in the compute manager would hit @reverts_task_state | 14:41 |
| dansmith | should, yeah, which is why I'm asking for detail from the bug filer | 14:42 |
| dansmith | it might be that heat just doesn't handle the failure but the instance is actually delete-able, which is what I'm trying to find out | 14:43 |
| *** lucasxu has quit IRC | 14:43 | |
| mriedem | ha, reading jay's 2nd set of slides, disk_available_least, that's fun | 14:44 |
| mriedem | compute.free_disk_gb | 14:44 |
| mriedem | compute.disk_available_least | 14:44 |
| mriedem | duh | 14:44 |
| *** markvoelker has joined #openstack-nova | 14:45 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Fix unit test exception KeyErrors https://review.openstack.org/462764 | 14:45 |
| openstackgerrit | melanie witt proposed openstack/nova master: Make NovaException format errors fatal for tests https://review.openstack.org/411615 | 14:45 |
| *** Qiming has quit IRC | 14:50 | |
| *** iceyao has joined #openstack-nova | 14:52 | |
| *** salv-orlando has joined #openstack-nova | 14:53 | |
| *** Qiming has joined #openstack-nova | 14:53 | |
| bauzas | hah, jay isn't around and I have a question for him https://review.openstack.org/#/c/460798/6 | 14:56 |
| *** salv-orl_ has joined #openstack-nova | 14:56 | |
| *** iceyao has quit IRC | 14:57 | |
| cdent | bauzas: there's been discussion of that in the comments on the earlier patches | 14:58 |
| *** dpawlik has quit IRC | 14:58 | |
| cdent | the basic answer is that the code is going to evolve to one query over a series of patches | 14:58 |
| cdent | (including traits) | 14:59 |
| *** Qiming has quit IRC | 14:59 | |
| cdent | bauzas: in other words, it's important to read all the comments on all the patchsets in the entire stack to get a good idea of what's going on | 14:59 |
| *** annegentle has joined #openstack-nova | 14:59 | |
| cdent | that's a pain, but that's how it is | 14:59 |
| bauzas | cdent: I see PS2 comments | 14:59 |
| *** lucasxu has joined #openstack-nova | 15:00 | |
| *** vks1 has quit IRC | 15:00 | |
| cdent | yeah, look elsewhere in the stack too | 15:00 |
| *** salv-orlando has quit IRC | 15:00 | |
| cdent | I had the same concerns and got rather confused because I was expecting things to be more complete in one patch, but that's not the path being taken | 15:00 |
| bauzas | cdent: okay, I just think he missed a verification | 15:01 |
| bauzas | cdent: about returning RPs having all the RCs | 15:01 |
| cdent | yeah, that maybe, but some of the verification is intentionally being done later | 15:02 |
| cdent | (in the stack) | 15:02 |
| bauzas | that's the problem when you duplicate code | 15:02 |
| cdent | yes, my complaint exactly | 15:02 |
| bauzas | you reintroduce bugs | 15:02 |
| cdent | but jay's aware of it | 15:02 |
| *** gouthamr has joined #openstack-nova | 15:02 | |
| *** Qiming has joined #openstack-nova | 15:03 | |
| *** voelzmo has quit IRC | 15:05 | |
| *** mlakat has quit IRC | 15:05 | |
| *** armax has joined #openstack-nova | 15:05 | |
| *** lyan has joined #openstack-nova | 15:06 | |
| *** crushil has joined #openstack-nova | 15:07 | |
| *** Qiming has quit IRC | 15:08 | |
| *** sridharg has quit IRC | 15:08 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: De-complicate some of the instance delete path https://review.openstack.org/458636 | 15:09 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Move to proper target_cell calling convention https://review.openstack.org/458634 | 15:09 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make target_cell() yield a new context https://review.openstack.org/458635 | 15:09 |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 15:09 |
| *** armax_ has joined #openstack-nova | 15:09 | |
| *** Qiming has joined #openstack-nova | 15:10 | |
| *** armax has quit IRC | 15:10 | |
| *** armax_ is now known as armax | 15:10 | |
| bauzas | cdent: okay confirmed that we need the HAVING count() for RCs, because we could return RPs not having the asked RC in their inventories | 15:12 |
| cdent | mriedem, jroll: have we lost jroll on this https://blueprints.launchpad.net/nova/+spec/custom-resource-classes-in-flavors or is he staying in view? | 15:12 |
| *** acabot___ has quit IRC | 15:14 | |
| mriedem | cdent: jay owns it now | 15:14 |
| melwitt | mriedem, dansmith, johnthetubaguy: did anyone want to do a hangout to get an overview of the quotas patches to help make review easier? | 15:14 |
| cdent | thanks mriedem | 15:14 |
| *** Oku_OS is now known as Oku_OS-away | 15:14 | |
| *** lpetrut has joined #openstack-nova | 15:15 | |
| mriedem | melwitt: i probably could in awhile, but wanted to get started on this chinese business visa paperwork before leaving for a week | 15:15 |
| *** vks1 has joined #openstack-nova | 15:15 | |
| mriedem | b/c i imagine this is going to be worse than reviewing those quotas patches :) | 15:15 |
| melwitt | heh, okay | 15:16 |
| *** Qiming_ has joined #openstack-nova | 15:17 | |
| *** mlavalle has joined #openstack-nova | 15:18 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova master: neutron: re-raise 401 for a non-admin client https://review.openstack.org/422696 | 15:18 |
| *** Qiming has quit IRC | 15:20 | |
| *** brault has joined #openstack-nova | 15:22 | |
| bauzas | cdent: FWIW, I tried to explain the problem with the current method only accepting one RC | 15:22 |
| bauzas | of course, we can merge that change as it is, but I think it will probably need some modifications later | 15:22 |
| bauzas | dansmith: ^ | 15:23 |
| cdent | "needing some modification later" is what I was trying to explain above. That's expected. | 15:24 |
| dansmith | bauzas: tbh, I'm not sure I see a ton of value in complicating these methods, which are effectively wrappers around sql statements, with lots of conditional behavior | 15:24 |
| bauzas | dansmith: the problem is not with that method, the problem is that that method executes the code | 15:25 |
| *** ljxiash has joined #openstack-nova | 15:25 | |
| dansmith | um, what? | 15:25 |
| bauzas | dansmith: I tried to explain the problem in the last comment | 15:25 |
| dansmith | executing code is usually what I'm looking for when shopping for a method | 15:25 |
| dansmith | I've read your comments | 15:26 |
| bauzas | dansmith: say that I want to ask placement for two resource amounts, each of them being for a specific RC | 15:26 |
| bauzas | then I would need to verify both RCs separately, instead of using the DBRM execution plan to make my single query faster | 15:26 |
| bauzas | and we *already* do that | 15:27 |
| bauzas | the SQL statement is just below | 15:27 |
| bauzas | also, the provided test is frankly positive | 15:27 |
| bauzas | it's just expecting something correct | 15:27 |
| *** lucasxu has quit IRC | 15:28 | |
| bauzas | I'd rather be interested to verify negative things, like having a shared RP that doesn't have enough disk | 15:28 |
| bauzas | ideally, two shared RPs, one having allocations | 15:28 |
| cdent | bauzas: there are some tests like that later in the stack | 15:29 |
| cdent | but yes, having them not with the actual code change is a bit disconcerting | 15:29 |
| *** ljxiash has quit IRC | 15:29 | |
| bauzas | that's what we ask when reviewing | 15:29 |
| *** lucasxu has joined #openstack-nova | 15:29 | |
| figleaf | What's the point of testing behavior that you know is wrong/incomplete? | 15:30 |
| cdent | fair point | 15:30 |
| cdent | but what's the point of merging code that you know is wrong/incomplete? | 15:30 |
| cdent | I thought changes were supposed to be atomic? | 15:30 |
| cdent | (I don't agree with that position, but I thought that was what was expected) | 15:31 |
| bauzas | figleaf: you're kidding, hopefully ? | 15:31 |
| figleaf | cdent: well, I agree with that. I always thought that the imperative to keep changes small was taken too seriously | 15:31 |
| figleaf | bauzas: no | 15:31 |
| bauzas | I probably misunderstood your question then | 15:31 |
| figleaf | If you are coding something that you a) know is not correct and b) plan on changing in the dependent patch, the test should be with the code that makes it work | 15:32 |
| bauzas | figleaf: negative coverage can be small https://github.com/openstack/nova/blob/master/nova/tests/functional/db/test_resource_provider.py#L678-L694 | 15:32 |
| *** Jack_Iv has quit IRC | 15:33 | |
| figleaf | How big should a patch be? https://www.brainyquote.com/quotes/quotes/j/jdsaling184470.html | 15:33 |
| *** annegentle has quit IRC | 15:33 | |
| bauzas | figleaf: so, just to make it clear, when you code and you know you're introducing bugs, you prefer to leave as it is for the proposal, and fix it later ? | 15:34 |
| *** hongbin has joined #openstack-nova | 15:34 | |
| *** annegentle has joined #openstack-nova | 15:34 | |
| bauzas | anyway, it's late for me | 15:34 |
| mriedem | so write the test asserting you know it's going to fail somehow, and then the later change fixes it once the code is in place - we do that with functional regressions bug tests all the time | 15:34 |
| mriedem | write the test to show the bug, then the patch that fixes the bug also changes the test | 15:35 |
| bauzas | mriedem: that happens when we merged code | 15:35 |
| figleaf | bauzas: I prefer not to have patches that don't work. | 15:35 |
| bauzas | mriedem: not when it's in the review state | 15:35 |
| bauzas | unless it's super critical | 15:35 |
| mriedem | none of this is used yet right? | 15:35 |
| *** ljxiash has joined #openstack-nova | 15:35 | |
| *** chyka has joined #openstack-nova | 15:35 | |
| figleaf | Given that jay is saying this is strictly to reduce review size, I'm fine with the test being with the code that actually works | 15:35 |
| bauzas | I agree, but I'd by far clarify *how* it would be used | 15:35 |
| mriedem | if none of this is used, and it's all split up to ease the review burden, then yeah doing the tests at the end seems fine | 15:36 |
| mriedem | we write new apis that plumb in the lower levels and we do'nt know that all works until we write the api tests at the end | 15:36 |
| mriedem | e.g. tempest tests | 15:36 |
| bauzas | and then we fix | 15:36 |
| *** Jack_Iv has joined #openstack-nova | 15:36 | |
| bauzas | either way, I'm +1 | 15:36 |
| mriedem | so you want a single giant patch? | 15:36 |
| mriedem | granted, i haven't looked at any of the latest series | 15:37 |
| *** eharney has joined #openstack-nova | 15:37 | |
| bauzas | so, seems we're diverting | 15:37 |
| *** brault has quit IRC | 15:37 | |
| bauzas | I'm not advocating for large chunks of code | 15:37 |
| mriedem | if it's complicated, we're going to be fixing parts of it later regardless | 15:37 |
| bauzas | I'm just pointing out that this method is probably underperformant | 15:37 |
| mriedem | i'm sure there is some law about LOC to bugs ratio | 15:37 |
| bauzas | my original concern is not coverage | 15:38 |
| bauzas | my concern is just, we already have code that does that | 15:38 |
| bauzas | and we're adding a fresh new code | 15:38 |
| bauzas | that new code doesn't include the multi-RC thing | 15:38 |
| bauzas | so, fine by me with merging that one, but then I'm pretty sure we'll require to modify it later | 15:38 |
| dansmith | that goes for every line of code we merge | 15:39 |
| cdent | bauzas: everyone knows and accept that | 15:39 |
| dansmith | I don't understand the problem at all | 15:39 |
| cdent | this code is written with the intent of being modified later | 15:39 |
| figleaf | yeah, we had this discussion two days ago | 15:39 |
| bauzas | sorry, French card played here _o | 15:39 |
| bauzas | \o | 15:40 |
| *** felipemonteiro has joined #openstack-nova | 15:41 | |
| bauzas | dansmith: the plan is to integrate it with get_all_by_filters() right? | 15:41 |
| *** Apoorva has joined #openstack-nova | 15:41 | |
| bauzas | dansmith: so, it's possibly asking for two or more resource classes | 15:41 |
| *** salv-orl_ has quit IRC | 15:42 | |
| bauzas | dansmith: if so, we need to call _get_providers_with_shared_capacity() twice, correct? | 15:42 |
| bauzas | for each RC | 15:42 |
| *** carthaca_ has quit IRC | 15:43 | |
| dansmith | I don't know that that's the plan | 15:43 |
| dansmith | cdent and figleaf are the ones that said the knew of plans to change something down the road | 15:43 |
| *** carthaca_ has joined #openstack-nova | 15:43 | |
| bauzas | when looking at the series, I can't find where that method is called, so I'm guessing given the commit msg | 15:44 |
| cdent | bauzas: the end goal is to end up with optimized sql but to get there in steps so that more people understand what the queries are actually doing. if we did that in one step, it would be even more incomprehensible than it is now | 15:44 |
| dansmith | that ^ | 15:44 |
| dansmith | I mean, | 15:44 |
| dansmith | the end goal is to merge this EVER. | 15:44 |
| * dansmith is frustrated | 15:45 | |
| cdent | bauzas: which method do you mean when you say "that method". If you clear that up, I'll point you to where it will be used | 15:45 |
| *** kiwi_rot has quit IRC | 15:45 | |
| bauzas | then, I'm possibly biased by the fact I wrote the SQL-fu for get_all_by_filters() so I'm seasoned with the SQL statement (and the SQLA calls) he wrote | 15:45 |
| bauzas | cdent: sorry, I wanted to explain that I don't see where _get_providers_with_shared_capacity() is used by get_all_by_filters() | 15:47 |
| bauzas | cdent: in the series | 15:47 |
| cdent | line 909 https://review.openstack.org/#/c/461574/9/nova/objects/resource_provider.py | 15:48 |
| cdent | and yes, for now it is once per rc_id | 15:49 |
| bauzas | cdent: I'm actually confused by https://review.openstack.org/#/c/461574/9/nova/objects/resource_provider.py | 15:50 |
| *** lyan has quit IRC | 15:50 | |
| bauzas | anyway, seems you're feeling me as a nitpicker, and that's the end of the week | 15:51 |
| bauzas | so... okay, fair enough, let's merge that code | 15:52 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Rename qemu_enable_gluster_storage_driver https://review.openstack.org/447091 | 15:53 |
| *** sshnaidm|off has joined #openstack-nova | 15:53 | |
| *** damien_r1 has quit IRC | 15:54 | |
| *** nic has joined #openstack-nova | 15:54 | |
| *** damien_r has joined #openstack-nova | 15:54 | |
| *** salv-orlando has joined #openstack-nova | 15:54 | |
| *** kristia__ has joined #openstack-nova | 15:55 | |
| *** iceyao has joined #openstack-nova | 15:56 | |
| *** iceyao has quit IRC | 15:57 | |
| *** kristian__ has quit IRC | 15:58 | |
| *** markus_z has quit IRC | 15:58 | |
| *** kristia__ has quit IRC | 15:59 | |
| *** kristian__ has joined #openstack-nova | 16:00 | |
| *** gjayavelu has joined #openstack-nova | 16:01 | |
| *** crushil has quit IRC | 16:03 | |
| mdbooth | johnthetubaguy: Got a second? I'm trying to work out where in the live migration flow nova tells cinder that a volume is going to be attached from a new host | 16:03 |
| mdbooth | I can't find it, though. Is that because it doesn't exist, or because I'm blind? | 16:03 |
| *** ljxiash has quit IRC | 16:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: Totally freeze the extension_info API https://review.openstack.org/462825 | 16:04 |
| alex_xu | sfinucan: thanks for the review :) | 16:05 |
| *** abhi89 has joined #openstack-nova | 16:06 | |
| sfinucan | alex_xu: No problemo 👍 | 16:06 |
| mriedem | mdbooth: pre_live_migration in the virt driver | 16:07 |
| mriedem | mdbooth: that's run on the destination host | 16:07 |
| mriedem | source compute rpc casts to the destination compute | 16:08 |
| mdbooth | mriedem: So that calls vol_driver.connect_volume | 16:08 |
| mriedem | mdbooth: sorry rpc call | 16:08 |
| mriedem | mdbooth: yeah loops over the bdms and calls self._connect_volume(connection_info, disk_info) | 16:09 |
| mdbooth | But I thought (and here I expect to find I'm wrong) that cinder didn't allow connections from unexpected hosts | 16:09 |
| mdbooth | So I was expecting to see a cinder call in there | 16:09 |
| mdbooth | That's just attempting to connect directly to the storage, and expects to be allowed already, right? | 16:09 |
| *** damien_r has quit IRC | 16:11 | |
| *** gjayavelu has quit IRC | 16:11 | |
| mriedem | i'm not sure where it initializes the connection | 16:11 |
| mriedem | btw, i think hemna has a flow chart for all of this | 16:11 |
| mdbooth | I think I've got a good grasp of the nova flow, I just can't find this piece | 16:12 |
| mdbooth | But that may be because it doesn't exist, or need to | 16:12 |
| mdbooth | In which case I've misunderstood cinder | 16:12 |
| johnthetubaguy | mdbooth: sorry stuck in a call right now | 16:12 |
| hemna | yah I did | 16:12 |
| hemna | let me see if I can find the url | 16:12 |
| mriedem | mdbooth: looks like post_live_migration | 16:12 |
| mdbooth | mriedem: Oh, interesting | 16:13 |
| mdbooth | If the connection isn't initialised until post_live_migration, how can it expect to connect in pre_live_migration? | 16:13 |
| mdbooth | Incidentally, I'm specifically looking for this because I've got logs where the connect_volume in pre_live_migration fails | 16:14 |
| mdbooth | However, I'm starting from the assumption that this works, and I've just missed something | 16:14 |
| hemna | https://drive.google.com/file/d/0B1Kp6K43HLHyRkFMZHVHWDJ3amM/view | 16:15 |
| mriedem | oh post_live_migration is where it initializes the connection in cinder to get the connection_info, which it uses to disconnect the volume from the source host | 16:15 |
| mdbooth | And the logs aren't due to anything architectural | 16:15 |
| hemna | live migration flow: https://drive.google.com/file/d/0B1Kp6K43HLHydU1wWFVIN29tc2s/view | 16:15 |
| mriedem | it seems odd we have to call cinder again for this since we store the connection_info in the bdm | 16:15 |
| mdbooth | hemna: Wow, thanks | 16:16 |
| mriedem | unless that's now the connection_info for the dest host? | 16:16 |
| hemna | mdbooth, np. :) | 16:16 |
| *** dtp has joined #openstack-nova | 16:16 | |
| mdbooth | mriedem: I didn't mean to drag anybody else into my confusion :) Just wondering if anybody happened to know the answer already. | 16:16 |
| mriedem | mdbooth: the answer to any question about the live migration flow is no one konws | 16:17 |
| mdbooth | woooooooo, it's a mystery | 16:17 |
| *** Jack_Iv_ has joined #openstack-nova | 16:19 | |
| *** smatzek has quit IRC | 16:22 | |
| *** Jack_Iv has quit IRC | 16:23 | |
| *** lucasagomes is now known as lucas-afk | 16:23 | |
| *** lpetrut has quit IRC | 16:23 | |
| johnthetubaguy | mdbooth: sorry, looking back... which bit we focusing on? | 16:24 |
| mdbooth | johnthetubaguy: I have log files where the connect_volume in pre_live_migration is failing | 16:24 |
| *** yamahata has joined #openstack-nova | 16:24 | |
| mdbooth | I was trying to work my way back through the flow to the point where nova told cinder to allow connections from the dest compute | 16:25 |
| johnthetubaguy | mdbooth: so today it kinda doesn't and breaks with some backends, I believe | 16:25 |
| johnthetubaguy | future, we create new attachments, and connect to those | 16:25 |
| mdbooth | Ah... | 16:25 |
| *** kristian__ has quit IRC | 16:26 | |
| *** crushil has joined #openstack-nova | 16:26 | |
| mdbooth | Right, I wasn't entirely sure what form this would take using cinder v2 | 16:26 |
| cfriesen | I just opened bug 1688599. I think this dates back to us allowing evac from an ERROR state, and we forgot to update _instance_in_resize_state(). I should be able to post a proposed fix shortly. | 16:26 |
| openstack | bug 1688599 in OpenStack Compute (nova) "resource audit races against evacuating instance" [Undecided,New] https://launchpad.net/bugs/1688599 | 16:26 |
| mdbooth | johnthetubaguy: So it's literally just trusting that the backend doesn't care where the connections come from? | 16:26 |
| mdbooth | This could be a red herring, btw. | 16:27 |
| * mdbooth is expecting it to be related to https://bugs.launchpad.net/nova/+bug/1683972 because the user is reporting it under heavy load. | 16:27 | |
| openstack | Launchpad bug 1683972 in os-brick "Overlapping iSCSI volume detach/attach can leave behind broken SCSI devices and multipath maps." [Undecided,New] | 16:27 |
| *** kristian__ has joined #openstack-nova | 16:27 | |
| openstackgerrit | Sandeep Pawar proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951 | 16:27 |
| *** esberglu has quit IRC | 16:28 | |
| *** gyee has joined #openstack-nova | 16:29 | |
| *** esberglu has joined #openstack-nova | 16:29 | |
| *** coreywright has quit IRC | 16:32 | |
| *** esberglu has quit IRC | 16:33 | |
| *** ltomasbo is now known as ltomasbo|away | 16:34 | |
| johnthetubaguy | mdbooth: yeah, maybe its just the connector we update in the BDM, rather than calling initalize on the new host, I don't remember now... | 16:36 |
| johnthetubaguy | mdbooth: i remember some of this being why I wanted the connector in cinder and an attachment for each host | 16:36 |
| *** derekh has quit IRC | 16:37 | |
| *** yamahata has quit IRC | 16:39 | |
| johnthetubaguy | mdbooth: wow, yeah, its totally a blind connect right now: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6706 | 16:39 |
| mdbooth | johnthetubaguy: I'm assuming this works most setups, though. | 16:42 |
| *** kristian__ has quit IRC | 16:42 | |
| johnthetubaguy | mdbooth: I think ISCSI works, if the password is volume specific not host specific | 16:42 |
| johnthetubaguy | mdbooth: that probably means yes (shrugs) | 16:42 |
| mdbooth | Hehe. That's good, anyway. This is iscsi. | 16:42 |
| *** annegentle has quit IRC | 16:42 | |
| *** kristian__ has joined #openstack-nova | 16:43 | |
| mdbooth | Although could be a funky proprietary iscsi target. | 16:43 |
| *** annegentle has joined #openstack-nova | 16:44 | |
| *** pchavva has quit IRC | 16:44 | |
| *** coreywright has joined #openstack-nova | 16:45 | |
| * johnthetubaguy heads out to tuba concert... | 16:46 | |
| *** esberglu has joined #openstack-nova | 16:47 | |
| *** owalsh has quit IRC | 16:47 | |
| *** owalsh has joined #openstack-nova | 16:49 | |
| *** imacdonn has quit IRC | 16:50 | |
| *** imacdonn has joined #openstack-nova | 16:50 | |
| *** jpena is now known as jpena|off | 16:51 | |
| cdent | needs to remember to pack before leaving | 16:52 |
| cdent | anybody want anything from england? | 16:52 |
| *** vks1 has quit IRC | 16:55 | |
| *** cdent has quit IRC | 16:56 | |
| *** ericyoung has quit IRC | 16:56 | |
| *** felipemonteiro has quit IRC | 17:00 | |
| *** arunman has quit IRC | 17:02 | |
| openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: 2.46: Show flavor info in server details https://review.openstack.org/435141 | 17:03 |
| *** harlowja has quit IRC | 17:03 | |
| *** hongbin has quit IRC | 17:04 | |
| cfriesen | cdent: beer. though I'm not sure how the head will hold up on the flight over. | 17:04 |
| *** fragatina has joined #openstack-nova | 17:06 | |
| *** Jack_Iv_ has quit IRC | 17:06 | |
| *** pchavva has joined #openstack-nova | 17:07 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513 | 17:10 |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 17:10 |
| dansmith | melwitt: I fixed that vnc thing.. it was a thing I missed in my context usage conversion | 17:10 |
| dansmith | melwitt: now hitting that weird delete case again, but hoping I can repro with debug and not have it dropped by journal this time | 17:10 |
| *** kfarr has joined #openstack-nova | 17:11 | |
| *** winston-d_ has joined #openstack-nova | 17:13 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload https://review.openstack.org/419730 | 17:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add separate instance.create payload type https://review.openstack.org/463001 | 17:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload https://review.openstack.org/463002 | 17:13 |
| *** abhi89 has quit IRC | 17:14 | |
| *** brault has joined #openstack-nova | 17:14 | |
| *** mirrorbox has quit IRC | 17:14 | |
| *** markvoelker has quit IRC | 17:15 | |
| *** brault has quit IRC | 17:16 | |
| *** brault has joined #openstack-nova | 17:16 | |
| *** ociuhandu has quit IRC | 17:16 | |
| *** markvoelker has joined #openstack-nova | 17:16 | |
| *** brault_ has joined #openstack-nova | 17:17 | |
| *** mnestratov has quit IRC | 17:19 | |
| *** brault has quit IRC | 17:21 | |
| *** markvoelker has quit IRC | 17:21 | |
| *** Jack_Iv has joined #openstack-nova | 17:23 | |
| *** kristian__ has quit IRC | 17:29 | |
| *** Apoorva_ has joined #openstack-nova | 17:31 | |
| *** gjayavelu has joined #openstack-nova | 17:31 | |
| *** slaweq has joined #openstack-nova | 17:32 | |
| *** pewp has quit IRC | 17:32 | |
| *** pewp has joined #openstack-nova | 17:32 | |
| *** Apoorva has quit IRC | 17:34 | |
| *** slaweq has quit IRC | 17:36 | |
| *** pewp has quit IRC | 17:38 | |
| *** pewp has joined #openstack-nova | 17:39 | |
| *** pewp has quit IRC | 17:41 | |
| *** pewp has joined #openstack-nova | 17:42 | |
| *** burgerk has joined #openstack-nova | 17:42 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 17:43 |
| *** pewp has quit IRC | 17:43 | |
| *** pewp has joined #openstack-nova | 17:44 | |
| *** pewp has quit IRC | 17:45 | |
| *** pewp has joined #openstack-nova | 17:46 | |
| *** fragatina has quit IRC | 17:48 | |
| melwitt | dansmith: ah, okay. cool | 17:48 |
| *** fragatina has joined #openstack-nova | 17:49 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add BDM to InstancePayload https://review.openstack.org/448779 | 17:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add key_name field to InstancePayload https://review.openstack.org/463002 | 17:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add keypairs field to InstanceCreatePayload https://review.openstack.org/419730 | 17:50 |
| melwitt | I had been looking at the code on master | 17:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add separate instance.create payload type https://review.openstack.org/463001 | 17:50 |
| dansmith | yeah | 17:52 |
| mriedem | dansmith: armax: didn't this already come up awhile back https://blueprints.launchpad.net/nova/+spec/sriov-bond? | 17:56 |
| *** cdent has joined #openstack-nova | 17:56 | |
| mriedem | for trunk ports? | 17:56 |
| *** jamesden_ has joined #openstack-nova | 17:56 | |
| armax | I think so | 17:56 |
| armax | that doesn’t sound new | 17:56 |
| *** jamesdenton has quit IRC | 17:56 | |
| *** iceyao has joined #openstack-nova | 17:57 | |
| knikolla | hey guys. why does the 'deleted' column in block_device_mapping have a number which is not zero or one. which i would expect from a column showing state? we're having to do some database surgery and not sure what value to manually set it to. | 17:59 |
| mriedem | knikolla: that's a legacy bdm v1 thing | 18:01 |
| mriedem | just take the default | 18:01 |
| *** ociuhandu has joined #openstack-nova | 18:01 | |
| knikolla | mriedem: so any nonzero number means deleted? | 18:01 |
| *** iceyao has quit IRC | 18:01 | |
| knikolla | mriedem: anywhere else we need to clean up for a stuck volume in the nova side of things? | 18:02 |
| mriedem | knikolla: actually sorry i was confusing that with the no_device field | 18:03 |
| mriedem | knikolla: 'deleted' is whether or not it's "soft" deleted in the database, | 18:03 |
| cfriesen | I'm hitting gate-tempest-dsvm failures in https://review.openstack.org/#/c/434514/. Is this a known thing at the moment? | 18:03 |
| mriedem | so deleted=0 means it's not deleted | 18:03 |
| mriedem | knikolla: if it's 'deleted', then deleted=id | 18:03 |
| mriedem | where id is the block_device_mappings.id primary key | 18:03 |
| knikolla | mriedem: i see, thanks a lot! | 18:05 |
| *** gszasz has quit IRC | 18:08 | |
| *** felipemonteiro has joined #openstack-nova | 18:09 | |
| openstackgerrit | Sandeep Pawar proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951 | 18:09 |
| mriedem | armax: yeah this is what i was thinking of https://review.openstack.org/#/c/213644/ | 18:10 |
| *** felipemonteiro_ has joined #openstack-nova | 18:10 | |
| *** pcaruana has quit IRC | 18:10 | |
| armax | mriedem: no I have a diffeent one in mind | 18:10 |
| armax | hang on | 18:10 |
| armax | https://review.openstack.org/#/c/182242/ | 18:11 |
| mriedem | yeah i pointed that one out in the bp | 18:11 |
| armax | mriedem: you always a step ahead of me | 18:12 |
| *** pchavva has quit IRC | 18:14 | |
| *** Jack_Iv has quit IRC | 18:14 | |
| *** hongbin has joined #openstack-nova | 18:14 | |
| *** felipemonteiro has quit IRC | 18:14 | |
| *** Jack_Iv has joined #openstack-nova | 18:14 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** kaisers_ has joined #openstack-nova | 18:18 | |
| *** kaisers has quit IRC | 18:20 | |
| *** brault_ is now known as brault|away | 18:26 | |
| *** mnestratov has joined #openstack-nova | 18:26 | |
| *** Jack_Iv has quit IRC | 18:29 | |
| *** lucasxu has quit IRC | 18:29 | |
| *** tesseract has quit IRC | 18:30 | |
| *** lucasxu has joined #openstack-nova | 18:31 | |
| *** crushil has quit IRC | 18:35 | |
| *** pchavva has joined #openstack-nova | 18:35 | |
| *** slaweq has joined #openstack-nova | 18:36 | |
| *** Jack_Iv has joined #openstack-nova | 18:39 | |
| *** chyka has quit IRC | 18:39 | |
| *** chyka has joined #openstack-nova | 18:40 | |
| *** salv-orlando has quit IRC | 18:41 | |
| *** thorst_afk has quit IRC | 18:42 | |
| *** Jack_Iv has quit IRC | 18:43 | |
| *** lbragstad has quit IRC | 18:44 | |
| *** slaweq has quit IRC | 18:45 | |
| *** Jack_Iv has joined #openstack-nova | 18:46 | |
| *** lbragstad has joined #openstack-nova | 18:47 | |
| *** crushil has joined #openstack-nova | 18:47 | |
| *** edmondsw has quit IRC | 18:50 | |
| *** openstackgerrit has joined #openstack-nova | 18:51 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Debug race in delete-while-booting test https://review.openstack.org/461513 | 18:51 |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Canary test against multi-cell devstack https://review.openstack.org/458537 | 18:51 |
| *** ericyoung has joined #openstack-nova | 18:53 | |
| mriedem | hemna: you might need to clarify for these guys https://review.openstack.org/#/c/443951/10/etc/nova/rootwrap.d/compute.filters | 18:54 |
| *** edmondsw has joined #openstack-nova | 18:54 | |
| mriedem | hemna: can all of that root_helper stuff be removed from os-brick? | 18:56 |
| *** tbachman has quit IRC | 18:56 | |
| *** lucasxu has quit IRC | 18:57 | |
| *** jamesden_ has quit IRC | 18:59 | |
| *** thorst_afk has joined #openstack-nova | 18:59 | |
| *** edmondsw has quit IRC | 18:59 | |
| *** lucasxu has joined #openstack-nova | 19:00 | |
| *** mnestratov has quit IRC | 19:03 | |
| mriedem | melwitt: i think i can say with a high level of confidence, that me starting on the counting quotas series at this time of day on a friday before travel, is not going to happen | 19:10 |
| sdague | dansmith: so... I didn't fully forget about the notes on the boot path. I just did a thing, which might be useful for that (or might not, you tell me), which is annotating a log dump tracing the server build request - https://dague.net//testing/flow.html | 19:13 |
| mriedem | i smell onboarding material | 19:13 |
| dansmith | sdague: that's not unhelpful | 19:13 |
| sdague | mriedem: you have a url pattern you've been using for etherpads for the forum? | 19:15 |
| mriedem | sdague: yeah | 19:15 |
| mriedem | https://wiki.openstack.org/wiki/Forum/Boston2017 | 19:15 |
| sdague | dansmith: ok... well, is that enough of a bit of backing material to feel comfortable that we've got notes? | 19:15 |
| mriedem | BOS-forum-... | 19:15 |
| mriedem | BOS-forum-nova-project-onboarding | 19:15 |
| dansmith | sdague: that's probably way too much detail to talk through, but if you cut it down to your notes, and then remove/rename some, that'd be a good skeleton | 19:15 |
| mriedem | sdague: in fact https://etherpad.openstack.org/p/BOS-forum-nova-project-onboarding | 19:15 |
| dansmith | sdague: if we don't have something to draw on, we could project that and handwave through it | 19:16 |
| mriedem | sdague: maybe you just want to push all of that crap down to the way bottom and do your things at the top | 19:16 |
| sdague | dansmith: yeh, I don't think we'd actually write that up there, it's just backing material | 19:16 |
| dansmith | yeah | 19:16 |
| sdague | mriedem: did you discover if we have flipboard? | 19:17 |
| *** ildikov is now known as coffee_cat | 19:17 | |
| mriedem | i didn't get a definitive answer no | 19:17 |
| sdague | you all and your friday double negatives | 19:18 |
| mriedem | i didn't not get no answer | 19:18 |
| dansmith | I don't not know what you mean | 19:18 |
| mriedem | i believe i was told i'd get a $20 and could deal with it myself - as a joke, but maybe not totally | 19:19 |
| mriedem | i'm calling the foundations' HR dept as we speak | 19:19 |
| sdague | .... deep breaths.... | 19:19 |
| mriedem | was just told the request was made so they should be there | 19:19 |
| * mriedem cancels HR form | 19:19 | |
| dansmith | I will feel less bad about squandering the time if not provided with any tools to conduct something useful | 19:20 |
| *** djohnsto has quit IRC | 19:20 | |
| sdague | mriedem: ok, my inclination is to move that existing etherpad to -ideas, make official one include a Questions section at the bottom. Encourage people to enter there questions while we are talking, and we can use that as fodder as we go. | 19:21 |
| sdague | but I don't want to stomp toes here | 19:22 |
| mriedem | as far as i'm concerned you're running this onboarding show | 19:22 |
| mriedem | so go nuts | 19:22 |
| mriedem | like i said, i don't get there until noon | 19:22 |
| mriedem | looking at your flow page, we shouldn't be creating a port twice, | 19:23 |
| mriedem | but we do POST a port and then PUT (update it with the host binding) later | 19:23 |
| dansmith | sdague: is that rst? | 19:24 |
| *** felipemonteiro_ has quit IRC | 19:24 | |
| *** felipemonteiro_ has joined #openstack-nova | 19:24 | |
| sdague | dansmith: I have rst | 19:24 |
| dansmith | can I have rst? | 19:24 |
| sdague | https://dague.net//testing/flow.rst | 19:24 |
| *** adisky_ has quit IRC | 19:24 | |
| sdague | I can also stick this in a throw away git repo if you want | 19:24 |
| mriedem | sdague: i only see "POST /v2.0/ports.json" once | 19:24 |
| *** lbragstad has quit IRC | 19:25 | |
| *** voelzmo has joined #openstack-nova | 19:25 | |
| sdague | mriedem: yeh, my bad, I glossed over | 19:25 |
| dansmith | thanks, I'll try to trim it | 19:25 |
| openstackgerrit | Eric Fried proposed openstack/nova master: nova.utils.get_service_url() https://review.openstack.org/458257 | 19:25 |
| sdague | dansmith: how about I do an import into a git repo | 19:25 |
| dansmith | sdague: that seems entirely overkill to me, but as you wish | 19:25 |
| mriedem | i'm not sure what this is: May 05 14:12:54 os4 neutron-server[24158]: DEBUG neutron.api.v2.base [req-8b4ccea7-1462-43cc-9a21-f0013b797999 demo demo] Request body: {u'port': {u'network_id': u'8356212e | 19:25 |
| mriedem | "neutron-server starts building that network:" isn't actually building a network | 19:26 |
| mriedem | it's finding the available network for the project | 19:27 |
| mriedem | on which to create the port | 19:27 |
| mriedem | we'd only create a network if we use the auto-allocated-topology (get me a network) stuff | 19:27 |
| dansmith | sounds like mriedem wants to trim it | 19:27 |
| mriedem | i don't, | 19:27 |
| mriedem | i just want to nit pick it | 19:27 |
| mriedem | i'll go back to reading the thread about stable branch LTS and n-5 upgrade support | 19:28 |
| sdague | dansmith: what's your github id again? | 19:28 |
| dansmith | kk7ds | 19:28 |
| sdague | https://github.com/sdague/nova-boot-flow | 19:28 |
| sdague | you have commit | 19:28 |
| sdague | mriedem: if you want so as well | 19:29 |
| sdague | dansmith: I was trying to keep all the logs around, just build the summaries when appropriate, so if you go to trim, how about just getting rid of some summaries | 19:29 |
| *** baoli has quit IRC | 19:32 | |
| dansmith | sdague: yeah, I meant trim the summaries, not the log lines | 19:33 |
| dansmith | trim and edit the summaries I mean | 19:33 |
| *** crushil has quit IRC | 19:34 | |
| sdague | dansmith: have at, you should have full write on that repo | 19:35 |
| *** catintheroof has quit IRC | 19:36 | |
| dansmith | sdague: okay, I'm still scrambling with other stuff but I'll try | 19:36 |
| dansmith | I guess that would be easy to do on the plane | 19:36 |
| sdague | dansmith: yep, being git and all | 19:36 |
| *** felipemonteiro__ has joined #openstack-nova | 19:36 | |
| *** catintheroof has joined #openstack-nova | 19:37 | |
| *** baoli has joined #openstack-nova | 19:37 | |
| *** edmondsw has joined #openstack-nova | 19:38 | |
| *** felipemonteiro_ has quit IRC | 19:39 | |
| *** crushil has joined #openstack-nova | 19:39 | |
| cdent | mriedem: maybe you should throw some downgrade support hand grenades into that thread | 19:41 |
| *** catintheroof has quit IRC | 19:42 | |
| *** edmondsw has quit IRC | 19:43 | |
| *** awaugama has quit IRC | 19:43 | |
| mriedem | jesus | 19:44 |
| mriedem | the features vs stability thing drives me nuts | 19:44 |
| mriedem | no one mentioned how we talked about non-feature bug fix only releases once | 19:45 |
| *** damien_r has joined #openstack-nova | 19:45 | |
| mriedem | 'openstack doesn't / can't scale', ok we'll work on that, 'but you're working on it makes the transition hard and complex and that pisses me off' | 19:45 |
| mriedem | it's just easier to not say anything at all in these threads because they pop up every 3-6 months and die on the vine anyway | 19:46 |
| mriedem | i did see cells v2 and placement get run through the mud a few times though, that's fun | 19:46 |
| *** Apoorva_ has quit IRC | 19:47 | |
| *** ericyoung has quit IRC | 19:47 | |
| *** edmondsw has joined #openstack-nova | 19:48 | |
| *** harlowja has joined #openstack-nova | 19:48 | |
| *** slaweq has joined #openstack-nova | 19:49 | |
| *** artom has quit IRC | 19:50 | |
| *** artom has joined #openstack-nova | 19:51 | |
| mriedem | i guess on one replied to the request for input on the onboarding session in the ML | 19:51 |
| sdague | yeh, my entire point on that thread at this point is going to be "show me a gratuitous feature" | 19:51 |
| mriedem | oh well | 19:51 |
| mriedem | sdague: cells v2 and placement were called out as causing pain with no benefit | 19:51 |
| sdague | I guess I missed where placement was called out on that | 19:51 |
| mriedem | not directly replying ot you | 19:51 |
| sdague | I did just rejump in on the cellsv2 one | 19:51 |
| *** artom has quit IRC | 19:52 | |
| sdague | so, I will agree that there were some more bumps than would have been nice there, but compared to cells v1, which people kept deploying and getting unfixable behavior... it was a much better path | 19:52 |
| *** artom has joined #openstack-nova | 19:52 | |
| mriedem | cdent: an anecdote really http://lists.openstack.org/pipermail/openstack-dev/2017-May/116440.html | 19:54 |
| mriedem | "nova placement API caused a bit of a headache for deployment tools..." | 19:54 |
| *** annegentle has quit IRC | 19:55 | |
| cdent | mriedem: I think you mean sdague ? | 19:55 |
| mriedem | oh yeah | 19:55 |
| mriedem | sorry | 19:55 |
| cdent | s'alright | 19:55 |
| cdent | "but consider that "leadership" is always limiting for innovation." | 19:55 |
| * cdent shrugs | 19:56 | |
| *** harlowja has quit IRC | 19:56 | |
| *** salv-orlando has joined #openstack-nova | 19:57 | |
| cdent | I'm hopeful that the "everyone deploy with wsgi" will finally get people understanding that deploying with wsgi means "you can do whatever you want, easily, but you have to choose" instead of "that's different" | 19:57 |
| sdague | mriedem: ok, that's a pretty soft hit honestly | 19:58 |
| *** crushil has quit IRC | 19:58 | |
| mriedem | sdague: yeah i said it was an anecdote on re-read | 19:58 |
| sdague | and, I would agree with it, the docs lagged | 19:58 |
| mriedem | which docs/ | 19:58 |
| mriedem | ? | 19:58 |
| mriedem | we had pretty solid in-tree docs by rc1 | 19:58 |
| mriedem | the install guide was late | 19:59 |
| mriedem | for both placement and cells v2 | 19:59 |
| sdague | the install guide | 19:59 |
| sdague | because that's the bit that people on the deploy side do need | 19:59 |
| sdague | and, I think that's fine critique of "yep, we need to do that better next time" | 19:59 |
| mriedem | sure, but when is the last time someone from nova was writing the install guide? | 19:59 |
| sdague | mriedem: yep, sure, a bunch of things changed since the last major service changes like this showed up | 20:00 |
| mriedem | isn't this like the thing in newton when matt kasawara wanted the install guide in nova so we'd own it, and we said no | 20:00 |
| sdague | mriedem: no, I think this was some collective falling through the cracks | 20:00 |
| sdague | where there was a lot of assumptions about who all owned things, that seemed to fall appart, because the last time a new service was rolled out was conductor, and the landscape was a lot different | 20:01 |
| sdague | and ... we kind of happened to have 2 hit at the same time | 20:01 |
| *** nicolasbock has quit IRC | 20:01 | |
| sdague | so, stuff happens | 20:01 |
| mriedem | i agree that not having a rushed clusterfck of a release in ocata would have been better :) | 20:02 |
| mriedem | and i think we retrospected on that | 20:02 |
| sdague | mriedem: right, also that | 20:02 |
| sdague | anyway | 20:02 |
| mriedem | but i will say i'm happy with the focus put on the in-tree docs for placement and cells v2 that a bunch of people worked on | 20:02 |
| mriedem | in years past our bar for documentation of features in tree was non-existent | 20:03 |
| mriedem | just like our bar for requiring testing | 20:03 |
| sdague | my point is, there is totally legit criticism on some things that were less than ideal for service / major feature rollout. | 20:03 |
| sdague | but that's very different than saying those are gratuitous features | 20:03 |
| mriedem | sure, i don't disagree, and i think we/i have been as critical | 20:03 |
| *** ssurana has joined #openstack-nova | 20:04 | |
| sdague | I'm just sick of "features trump stability/usability" where features are code for "stuff I don't like or understand" and "stability/usability" is code for "my personal priority". And don't want to let that one slide this time. | 20:05 |
| *** crushil has joined #openstack-nova | 20:06 | |
| *** gomarivera has joined #openstack-nova | 20:11 | |
| *** eharney has quit IRC | 20:11 | |
| dansmith | I especially like "rearchitect the entire project ... in a branch" | 20:12 |
| sdague | yeh... I am sure the upgrade process would be very smooth there | 20:17 |
| *** lucasxu has quit IRC | 20:18 | |
| *** lbragstad has joined #openstack-nova | 20:18 | |
| *** lbragstad has quit IRC | 20:19 | |
| *** lbragstad has joined #openstack-nova | 20:20 | |
| *** lucasxu has joined #openstack-nova | 20:21 | |
| *** efried has quit IRC | 20:23 | |
| *** lucasxu has quit IRC | 20:27 | |
| cdent | Is "solid governance model" code for "not letting the devs decide"? | 20:28 |
| *** baoli has quit IRC | 20:29 | |
| *** tblakes has joined #openstack-nova | 20:33 | |
| *** lbragstad has quit IRC | 20:33 | |
| *** lbragstad has joined #openstack-nova | 20:34 | |
| *** baoli has joined #openstack-nova | 20:34 | |
| *** lbragstad has quit IRC | 20:34 | |
| sdague | probably | 20:34 |
| *** efried has joined #openstack-nova | 20:35 | |
| sdague | or, more likely "put my distguished architect in charge" | 20:35 |
| sdague | ok... I'm saving to draft a response on that last oracle one. Because that's going to spiral out of control if I don't | 20:35 |
| sdague | I should make sure mordred sees it | 20:36 |
| sdague | because the "we need 12 months to integrate our value add".... is kind of trigger words | 20:36 |
| *** tblakes has quit IRC | 20:37 | |
| *** hongbin has quit IRC | 20:37 | |
| cdent | heh | 20:38 |
| *** xyang1 has quit IRC | 20:38 | |
| cdent | I have a sort of point of order question, or something like that: | 20:38 |
| cdent | Why are people so worried about shit spiralling out of control? From a slightly different point of view that looks like people just avoiding the truth and putting more barriers in place to getting to the root causes of problems. | 20:39 |
| *** Apoorva has joined #openstack-nova | 20:39 | |
| cdent | I know, as mriedem so eloquently puts it, there's a lot of bitter feelings, but that's a symptom of a greater disease, not the problem itself | 20:40 |
| mriedem | is it bad to pick a fight with what is probably an oracle exec? | 20:40 |
| cdent | I suppose there's a chance that disagreement on this stuff is just the nature of the universe, so keeping it in a pot is best, but that feels a bit dystopic. | 20:41 |
| sdague | cdent: because not everyone has your patience level for email fights | 20:41 |
| mriedem | "There needs to be more customer and vendor driven involvement to solidifying a core set of features that everyone can rely on working consistently across releases and upgrades." | 20:41 |
| mriedem | ^ well no sh*t | 20:41 |
| mriedem | so pony up the leads here oracle | 20:41 |
| *** crushil has quit IRC | 20:41 | |
| mriedem | sdague: oh heh, i just replied to that one | 20:42 |
| mriedem | i couldn't resist | 20:42 |
| cdent | please please sdague email _discussions_ ;) | 20:42 |
| sdague | many people find the random flaming basically too emotionally sapping to want to participate in discussions at all, so what happens is all the thoughtful middle gives up | 20:42 |
| cdent | why should you resist mriedem | 20:42 |
| mriedem | i'm glad i wasn't the only one that interpreted "we need you to do LTS because we have a hard time keeping up with our forks" | 20:42 |
| dansmith | at least oracle funds so many people working on stuff in the community | 20:43 |
| mriedem | to date, there is one person from oracle helping with multiattach | 20:43 |
| mriedem | i will give credit where credit is due | 20:44 |
| cdent | sdague: I guess I'm weird: I'm so conscious of the under- or un-expressed emotion that that is the thing that I find so sapping. | 20:44 |
| cdent | incredibly so | 20:44 |
| sdague | cdent: sure, that might be you. Lots of people basically end up ignoring the whole of the mailing list for some number of weeks after this because they see the mailing list as a place where loud people yell about things. | 20:45 |
| cdent | do you consider the current conversation to be yelling already or not quite there yet? | 20:45 |
| sdague | I think it is just about there | 20:45 |
| mriedem | stvnoyes: btw, let me thank you personally for helping out with the multiattach stuff | 20:45 |
| sdague | cdent: which is why I "save draft" on my last email | 20:46 |
| mriedem | stvnoyes: also gave you a tip in https://review.openstack.org/#/c/456877/6/nova/compute/manager.py | 20:46 |
| cdent | wow. I guess I spent too much time on usenet in the 90s | 20:46 |
| sdague | cdent: yep | 20:46 |
| cdent | to me this feel like polite but earnest dinner conversation | 20:47 |
| sdague | I think if you polled the audience you would not get that reaction | 20:47 |
| *** karthiks has quit IRC | 20:49 | |
| * cdent huhs | 20:49 | |
| *** pchavva has quit IRC | 20:49 | |
| melwitt | mriedem: cool, I figured. since we're going to be at the summit anyway, we could chat briefly about it then | 20:50 |
| mriedem | melwitt: perhaps on thursday night | 20:50 |
| mriedem | which is the only night i'll get to enjoy the company of my fellow nova compatriots | 20:50 |
| *** voelzmo has quit IRC | 20:50 | |
| melwitt | mriedem: k. bummer we only get to hang with you one evening | 20:52 |
| mriedem | and sunday night | 20:52 |
| mriedem | and friday morning | 20:52 |
| *** lbragstad has joined #openstack-nova | 20:52 | |
| mriedem | and the 2 minutes in between sessions | 20:52 |
| efried | mriedem Would you mind pushing https://review.openstack.org/#/c/391288/ ? I'm getting merge conflicts up the chain, and would like to limit how much rebasing is needed. | 20:52 |
| mriedem | efried: don't you know i'm all riled up? | 20:53 |
| mriedem | :) | 20:53 |
| efried | mriedem Me too. The day I've had. | 20:53 |
| *** jerrygb has quit IRC | 20:53 | |
| mriedem | just blame it on thorst_afk | 20:53 |
| efried | Always. | 20:53 |
| thorst_afk | he's always afk | 20:54 |
| melwitt | hah | 20:54 |
| *** baoli has quit IRC | 20:55 | |
| efried | Honestly, I'm not sure how https://review.openstack.org/#/c/443189/ got a merge conflict but its predecessors didn't. | 20:56 |
| *** dtp has quit IRC | 20:56 | |
| efried | It didn't touch any non-PowerVM files. | 20:56 |
| *** salv-orl_ has joined #openstack-nova | 20:57 | |
| mriedem | melwitt: i'm suprised this doesn't result in a keyerror | 20:57 |
| mriedem | https://review.openstack.org/#/c/456877/7/nova/tests/unit/compute/test_compute.py@10260 | 20:57 |
| mriedem | since it's not using a kwarg | 20:57 |
| mriedem | oh i know why | 20:57 |
| mriedem | it's using attachment_id as the message | 20:58 |
| melwitt | mriedem: yeah, that might match because of python named argument matching? | 20:58 |
| mriedem | no i think it's because | 20:59 |
| mriedem | def __init__(self, message=None, **kwargs): | 20:59 |
| mriedem | if not message: | 20:59 |
| mriedem | if you overwrite the message, we don't try the msg % kwargs | 20:59 |
| mriedem | and get the KeyError | 20:59 |
| *** salv-orlando has quit IRC | 20:59 | |
| melwitt | oh, I see. | 20:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Cinder v3 detach to shutdown_instance https://review.openstack.org/456877 | 21:00 |
| *** iceyao has joined #openstack-nova | 21:01 | |
| melwitt | I was thinking of the named argument thing but that works in the opposite way. you can specify positional args as kwargs by name (and in wrong order) and it will still work | 21:02 |
| melwitt | s/positional/normal/ | 21:02 |
| *** alexpilotti has joined #openstack-nova | 21:03 | |
| *** alexpilotti has quit IRC | 21:03 | |
| efried | aHA! | 21:03 |
| efried | It's melwitt's fault! | 21:03 |
| *** burgerk has quit IRC | 21:03 | |
| efried | https://review.openstack.org/#/c/462764/3/nova/tests/unit/virt/powervm/test_driver.py | 21:04 |
| *** alexpilotti has joined #openstack-nova | 21:04 | |
| melwitt | lol, you meant literally | 21:04 |
| mriedem | heh, well, it's really your fault | 21:04 |
| mriedem | stvnoyes: btw, reverify is ignored | 21:05 |
| mriedem | use recheck | 21:05 |
| efried | So, hm, is there any way for me to fix my merge conflict without rebasing the predecessors? | 21:06 |
| *** iceyao has quit IRC | 21:06 | |
| *** damien_r has quit IRC | 21:07 | |
| *** bmace has quit IRC | 21:09 | |
| *** bmace has joined #openstack-nova | 21:10 | |
| *** jjung has joined #openstack-nova | 21:11 | |
| *** Jack_Iv has quit IRC | 21:13 | |
| *** Jack_Iv has joined #openstack-nova | 21:14 | |
| *** kfarr has quit IRC | 21:14 | |
| *** jjung has quit IRC | 21:15 | |
| *** jjung has joined #openstack-nova | 21:16 | |
| *** dimtruck is now known as zz_dimtruck | 21:16 | |
| *** thingee has joined #openstack-nova | 21:18 | |
| thingee | hey anyone oppose me proposing the deprecation of the trusted filter? | 21:18 |
| thingee | it has been experimental since it has existed in may 28 2012 | 21:18 |
| *** thorst_afk has quit IRC | 21:19 | |
| cdent | new things crawl out of the woodwork every day | 21:20 |
| thingee | aight, I'll send an email to ops list to see if anyone cares. | 21:20 |
| cdent | good plan | 21:21 |
| *** jjung has quit IRC | 21:21 | |
| *** jjung has joined #openstack-nova | 21:21 | |
| *** artom has quit IRC | 21:21 | |
| *** artom has joined #openstack-nova | 21:22 | |
| *** felipemonteiro__ has quit IRC | 21:22 | |
| *** jjung has quit IRC | 21:22 | |
| *** jjung has joined #openstack-nova | 21:22 | |
| thingee | http://lists.openstack.org/pipermail/openstack-operators/2017-May/013371.html boom | 21:22 |
| *** jjung has quit IRC | 21:24 | |
| *** baoli has joined #openstack-nova | 21:26 | |
| *** harlowja has joined #openstack-nova | 21:34 | |
| *** lbragstad has quit IRC | 21:35 | |
| *** lbragstad has joined #openstack-nova | 21:36 | |
| stvnoyes | mriedem: ok thanks. I had read somewhere that reverify was used to re-kick a gate test, but it must have been stale info | 21:39 |
| *** Jack_Iv has quit IRC | 21:43 | |
| *** liverpooler has quit IRC | 21:44 | |
| *** kiwi_rot has joined #openstack-nova | 21:46 | |
| *** gouthamr has quit IRC | 21:47 | |
| *** thorst_afk has joined #openstack-nova | 21:48 | |
| *** thorst_afk has quit IRC | 21:49 | |
| *** gjayavelu has quit IRC | 21:50 | |
| *** kiwi_rot has quit IRC | 21:51 | |
| *** sshnaidm|off has quit IRC | 21:57 | |
| *** artom has quit IRC | 22:01 | |
| *** artom has joined #openstack-nova | 22:02 | |
| *** cdent has quit IRC | 22:05 | |
| *** burt has quit IRC | 22:12 | |
| *** thorst_afk has joined #openstack-nova | 22:15 | |
| *** thorst_afk has quit IRC | 22:17 | |
| openstackgerrit | Mike Perez proposed openstack/nova master: Deprecate scheduler trusted filter https://review.openstack.org/463065 | 22:20 |
| *** gouthamr has joined #openstack-nova | 22:23 | |
| openstackgerrit | Mike Perez proposed openstack/nova master: Deprecate scheduler trusted filter https://review.openstack.org/463065 | 22:24 |
| *** salv-orl_ has quit IRC | 22:27 | |
| *** tbachman has joined #openstack-nova | 22:29 | |
| *** gomarivera has quit IRC | 22:36 | |
| *** krtaylor has quit IRC | 22:39 | |
| *** gjayavelu has joined #openstack-nova | 22:40 | |
| *** tbachman has quit IRC | 22:41 | |
| *** baoli has quit IRC | 22:42 | |
| mordred | sdague: I feel like maybe I'm skipping too much of that thread | 22:42 |
| *** sshnaidm|off has joined #openstack-nova | 22:49 | |
| mordred | sdague: oh - nope. reading more of it does not make my life any better | 22:51 |
| *** alexpilotti has quit IRC | 22:53 | |
| *** sdague has quit IRC | 23:06 | |
| mriedem | thingee: that reminds me https://review.openstack.org/#/c/457812/ | 23:11 |
| mriedem | stvnoyes: in the long ago reverify would do the same thing as recheck, but i think they stopped handling that a few months ago | 23:12 |
| thingee | mriedem: deprecate all the filters | 23:12 |
| mriedem | thingee: btw, what prompted this? | 23:13 |
| *** catinthe_ has joined #openstack-nova | 23:14 | |
| *** thorst_afk has joined #openstack-nova | 23:15 | |
| *** iceyao has joined #openstack-nova | 23:18 | |
| *** thorst_afk has quit IRC | 23:18 | |
| thingee | The board/tc meeting discussed a goal of simplification. Reduction of options and what not | 23:18 |
| thingee | mriedem: ^ | 23:18 |
| thingee | I was pointed to this when I was gathering ideas | 23:19 |
| thingee | mriedem: let me know if you have suggestion of things worth looking into | 23:20 |
| mriedem | thingee: holy crap we probably have a ton | 23:20 |
| mriedem | thingee: i know we deprecated a lot of things in ocata, | 23:20 |
| mriedem | i think you could just be looking for things that are deprecated for removal | 23:20 |
| mriedem | assuming someone doesn't already have a patch up to remove those things and we just haven't seen it yet | 23:21 |
| thingee | mriedem: yeah I removed something out of nova recently that was marked deprecated. not sure if I saw anything else, but I'll double check | 23:21 |
| *** iceyao has quit IRC | 23:22 | |
| *** jose-phillips has quit IRC | 23:22 | |
| mriedem | thingee: grep for deprecated_since='15.0.0', and you'll see a few things | 23:23 |
| mriedem | although, | 23:24 |
| mriedem | i'd avoid anything under nova/conf/network as those are probably for nova-network which isn't gone yet | 23:24 |
| mriedem | really anything that's deprecated related to nova-network is just signaling right now | 23:24 |
| mtreinish | mriedem: an even longer time ago reverify would just run the gate queue again and not rerun check | 23:25 |
| mriedem | anyway, got to run | 23:25 |
| mriedem | mtreinish: i know | 23:25 |
| mriedem | sheesh | 23:25 |
| mtreinish | thingee: also you can look at logstash we always emit a ton of deprecation warnings during gate jobs: http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22deprecated%5C%22%20AND%20loglevel:%5C%22WARNING%5C%22%20AND%20build_branch:%5C%22master%5C%22 | 23:25 |
| mriedem | mtreinish: i didn't want to expose my nerd-like internal knowledge about the slight differences between recheck and reverify and the check vs gate queues :) | 23:25 |
| mtreinish | thingee: I made a push to try and clean that up a while ago, but it never really fixed the problem: http://lists.openstack.org/pipermail/openstack-dev/2015-December/081984.html | 23:26 |
| mtreinish | mriedem: heh, fair enough | 23:26 |
| mriedem | mtreinish: btw, you still owe me that cloudpipe cli/api deprecation patch for novaclient | 23:26 |
| mtreinish | oh, I do don't I | 23:26 |
| mtreinish | I'll look at that in a couple min | 23:26 |
| mriedem | i'm gonna go gamble | 23:27 |
| mtreinish | enjoy | 23:27 |
| *** mriedem has quit IRC | 23:27 | |
| *** zz_dimtruck is now known as dimtruck | 23:35 | |
| *** iceyao has joined #openstack-nova | 23:38 | |
| *** chyka has quit IRC | 23:39 | |
| *** chyka has joined #openstack-nova | 23:39 | |
| *** iceyao has quit IRC | 23:43 | |
| *** enki_anunnaki has joined #openstack-nova | 23:43 | |
| *** slaweq has quit IRC | 23:44 | |
| *** chyka has quit IRC | 23:44 | |
| *** Jack_Iv has joined #openstack-nova | 23:44 | |
| *** slaweq has joined #openstack-nova | 23:45 | |
| *** gongysh has joined #openstack-nova | 23:47 | |
| *** Jack_Iv has quit IRC | 23:48 | |
| *** tbachman has joined #openstack-nova | 23:49 | |
| *** jose-phillips has joined #openstack-nova | 23:51 | |
| *** artom has quit IRC | 23:52 | |
| *** slaweq has quit IRC | 23:52 | |
| *** artom has joined #openstack-nova | 23:52 | |
| *** artom has quit IRC | 23:54 | |
| *** artom has joined #openstack-nova | 23:55 | |
| *** iceyao has joined #openstack-nova | 23:55 | |
| *** Yangyapeng has quit IRC | 23:58 | |
| *** iceyao has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!