| *** devkulkarni2 has quit IRC | 00:00 | |
| *** shaohe_feng has quit IRC | 00:05 | |
| *** TravT_ has joined #openstack-nova | 00:06 | |
| *** shaohe_feng has joined #openstack-nova | 00:06 | |
| *** gouthamr has quit IRC | 00:08 | |
| *** gouthamr has joined #openstack-nova | 00:09 | |
| *** TravT has quit IRC | 00:10 | |
| *** code-R has joined #openstack-nova | 00:10 | |
| *** gouthamr has quit IRC | 00:10 | |
| *** code-R_ has joined #openstack-nova | 00:12 | |
| *** Sukhdev has quit IRC | 00:13 | |
| *** Sukhdev has joined #openstack-nova | 00:13 | |
| sean-k-mooneyAFK | jaypipes: as an fyi on the internal receck the failing test has passed | 00:15 |
|---|---|---|
| *** code-R has quit IRC | 00:15 | |
| *** shaohe_feng has quit IRC | 00:16 | |
| *** shaohe_feng has joined #openstack-nova | 00:16 | |
| *** diogogmt has quit IRC | 00:16 | |
| *** sonofthor has joined #openstack-nova | 00:19 | |
| *** jamesdenton has quit IRC | 00:20 | |
| *** kylek3h has joined #openstack-nova | 00:23 | |
| *** TravT_ has quit IRC | 00:24 | |
| *** TravT has joined #openstack-nova | 00:25 | |
| *** shaohe_feng has quit IRC | 00:26 | |
| *** shaohe_feng has joined #openstack-nova | 00:27 | |
| *** gouthamr has joined #openstack-nova | 00:29 | |
| *** agireud has quit IRC | 00:30 | |
| *** ijw has joined #openstack-nova | 00:32 | |
| *** raunak has joined #openstack-nova | 00:33 | |
| openstackgerrit | Michael Still proposed openstack/nova: Documentation for the vendordata reboot. https://review.openstack.org/353793 | 00:35 |
| *** shaohe_feng has quit IRC | 00:36 | |
| *** agireud has joined #openstack-nova | 00:37 | |
| *** shaohe_feng has joined #openstack-nova | 00:37 | |
| *** tonytan4ever has quit IRC | 00:37 | |
| *** tuanla_ has joined #openstack-nova | 00:37 | |
| *** JoseMello has joined #openstack-nova | 00:38 | |
| *** ijw has quit IRC | 00:40 | |
| *** ijw has joined #openstack-nova | 00:40 | |
| *** shaohe_feng has quit IRC | 00:46 | |
| *** shaohe_feng has joined #openstack-nova | 00:47 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Adds view builders for hypervisors controller https://review.openstack.org/335282 | 00:54 |
| *** code-R_ has quit IRC | 00:57 | |
| *** shaohe_feng has quit IRC | 00:57 | |
| *** kevinz has joined #openstack-nova | 00:57 | |
| *** code-R has joined #openstack-nova | 00:57 | |
| *** sonofthor has quit IRC | 00:59 | |
| *** shaohe_feng has joined #openstack-nova | 01:00 | |
| *** oomichi has quit IRC | 01:00 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Pass nodename to internal RT methods https://review.openstack.org/353799 | 01:02 |
| *** mtanino has joined #openstack-nova | 01:02 | |
| *** diogogmt has joined #openstack-nova | 01:03 | |
| *** cheneydc has joined #openstack-nova | 01:05 | |
| *** cheneydc has quit IRC | 01:06 | |
| sean-k-mooneyAFK | jaypipes: actully looking at that failing test it passed on another build but failed on that patchest. im calling it a night but we will look into it again in the morning | 01:07 |
| *** shaohe_feng has quit IRC | 01:07 | |
| sean-k-mooneyAFK | night all o/ | 01:07 |
| *** penick has quit IRC | 01:07 | |
| *** shaohe_feng has joined #openstack-nova | 01:08 | |
| jaypipes | sean-k-mooneyAFK: night night. | 01:09 |
| *** hoangcx has joined #openstack-nova | 01:09 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Refactor unit test for trackable migrations https://review.openstack.org/352610 | 01:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Create multiple resize claim unit test https://review.openstack.org/352625 | 01:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: New unit test for rebuild_claim() https://review.openstack.org/352578 | 01:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: delete test_resource_tracker.py https://review.openstack.org/353800 | 01:13 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: rename test_tracker to test_resource_tracker https://review.openstack.org/353801 | 01:13 |
| openstackgerrit | STEW TY proposed openstack/nova: Body Verification of api-ref os-volume-attachments.inc https://review.openstack.org/336334 | 01:15 |
| *** stewie925 has quit IRC | 01:16 | |
| *** shaohe_feng has quit IRC | 01:17 | |
| *** shaohe_feng has joined #openstack-nova | 01:18 | |
| *** kylek3h has quit IRC | 01:19 | |
| *** raunak has quit IRC | 01:23 | |
| *** Wenzhi_ has quit IRC | 01:24 | |
| *** Wenzhi has joined #openstack-nova | 01:24 | |
| *** jwcroppe has quit IRC | 01:27 | |
| *** shaohe_feng has quit IRC | 01:27 | |
| *** gouthamr_ has joined #openstack-nova | 01:28 | |
| *** Sukhdev has quit IRC | 01:28 | |
| *** gjayavelu has quit IRC | 01:28 | |
| *** shaohe_feng has joined #openstack-nova | 01:29 | |
| *** Sukhdev has joined #openstack-nova | 01:30 | |
| *** jwcroppe has joined #openstack-nova | 01:31 | |
| *** gouthamr has quit IRC | 01:31 | |
| *** EinstCrazy has joined #openstack-nova | 01:33 | |
| *** jwcroppe has quit IRC | 01:33 | |
| *** jamesdenton has joined #openstack-nova | 01:33 | |
| *** jwcroppe has joined #openstack-nova | 01:35 | |
| *** EinstCrazy has quit IRC | 01:36 | |
| *** EinstCrazy has joined #openstack-nova | 01:36 | |
| *** EinstCrazy has quit IRC | 01:38 | |
| *** shaohe_feng has quit IRC | 01:38 | |
| *** tonytan4ever has joined #openstack-nova | 01:38 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (2) https://review.openstack.org/316550 | 01:38 |
| *** EinstCrazy has joined #openstack-nova | 01:38 | |
| *** shaohe_feng has joined #openstack-nova | 01:39 | |
| *** jwcroppe has quit IRC | 01:39 | |
| *** mtanino has quit IRC | 01:41 | |
| *** code-R has quit IRC | 01:41 | |
| *** rama_y has joined #openstack-nova | 01:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Deprecate network-* commands and clamp to microversion 2.35 https://review.openstack.org/347514 | 01:42 |
| *** tonytan4ever has quit IRC | 01:42 | |
| *** hieulq has quit IRC | 01:44 | |
| *** hieulq has joined #openstack-nova | 01:44 | |
| *** mlavalle has quit IRC | 01:45 | |
| *** jdurgin1 has quit IRC | 01:47 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient: Deprecate network-* commands and clamp to microversion 2.35 https://review.openstack.org/347514 | 01:48 |
| *** shaohe_feng has quit IRC | 01:48 | |
| *** mriedem has quit IRC | 01:49 | |
| *** shaohe_feng has joined #openstack-nova | 01:49 | |
| *** annp has joined #openstack-nova | 01:50 | |
| *** mtanino has joined #openstack-nova | 01:51 | |
| *** wangqun has joined #openstack-nova | 01:53 | |
| *** Sukhdev has quit IRC | 01:56 | |
| *** mtanino has quit IRC | 01:57 | |
| *** ivase has quit IRC | 01:57 | |
| *** jamesdenton has quit IRC | 01:58 | |
| *** ivase has joined #openstack-nova | 01:58 | |
| *** shaohe_feng has quit IRC | 01:58 | |
| *** shaohe_feng has joined #openstack-nova | 01:59 | |
| *** Apoorva_ has quit IRC | 02:00 | |
| *** Apoorva has joined #openstack-nova | 02:00 | |
| *** tonytan4ever has joined #openstack-nova | 02:01 | |
| *** tonytan_brb has joined #openstack-nova | 02:04 | |
| *** Apoorva has quit IRC | 02:05 | |
| *** ijw has quit IRC | 02:05 | |
| *** unicell has quit IRC | 02:06 | |
| *** tonytan4ever has quit IRC | 02:07 | |
| *** shaohe_feng has quit IRC | 02:08 | |
| *** shaohe_feng has joined #openstack-nova | 02:09 | |
| *** pkholkin has quit IRC | 02:11 | |
| *** pkholkin has joined #openstack-nova | 02:12 | |
| *** gongysh has joined #openstack-nova | 02:15 | |
| *** mingyu has joined #openstack-nova | 02:16 | |
| *** zerda2 has joined #openstack-nova | 02:18 | |
| *** ducttape_ has joined #openstack-nova | 02:18 | |
| *** shaohe_feng has quit IRC | 02:19 | |
| *** shaohe_feng has joined #openstack-nova | 02:19 | |
| *** mingyu has quit IRC | 02:20 | |
| openstackgerrit | xhzhf proposed openstack/nova: [vncproxy] log for closing web is misleading https://review.openstack.org/353807 | 02:23 |
| *** whenry has quit IRC | 02:28 | |
| *** shaohe_feng has quit IRC | 02:29 | |
| *** shaohe_feng has joined #openstack-nova | 02:29 | |
| *** john51 has quit IRC | 02:31 | |
| *** john51 has joined #openstack-nova | 02:33 | |
| *** penick has joined #openstack-nova | 02:35 | |
| *** jwcroppe has joined #openstack-nova | 02:36 | |
| *** harlowja_at_home has joined #openstack-nova | 02:39 | |
| *** shaohe_feng has quit IRC | 02:39 | |
| *** penick_ has joined #openstack-nova | 02:40 | |
| *** shaohe_feng has joined #openstack-nova | 02:40 | |
| *** penick has quit IRC | 02:41 | |
| *** penick_ is now known as penick | 02:41 | |
| *** code-R has joined #openstack-nova | 02:47 | |
| *** code-R has quit IRC | 02:48 | |
| *** gouthamr_ has quit IRC | 02:49 | |
| *** shaohe_feng has quit IRC | 02:49 | |
| *** shaohe_feng has joined #openstack-nova | 02:50 | |
| *** baoli has joined #openstack-nova | 02:52 | |
| *** jwcroppe has quit IRC | 02:53 | |
| *** harlowja_at_home has quit IRC | 02:54 | |
| *** yamahata has quit IRC | 02:58 | |
| *** shaohe_feng has quit IRC | 03:00 | |
| *** raunak has joined #openstack-nova | 03:01 | |
| *** shaohe_feng has joined #openstack-nova | 03:01 | |
| *** baoli has quit IRC | 03:03 | |
| *** ducttape_ has quit IRC | 03:06 | |
| *** ducttape_ has joined #openstack-nova | 03:06 | |
| *** pwnall has quit IRC | 03:08 | |
| *** baoli has joined #openstack-nova | 03:09 | |
| *** brian_price has quit IRC | 03:09 | |
| *** shaohe_feng has quit IRC | 03:10 | |
| *** tri2sing_ has joined #openstack-nova | 03:10 | |
| *** gouthamr has joined #openstack-nova | 03:10 | |
| *** krotscheck_ has joined #openstack-nova | 03:12 | |
| *** krotscheck has quit IRC | 03:13 | |
| *** rkmrhj has joined #openstack-nova | 03:13 | |
| *** shaohe_feng has joined #openstack-nova | 03:13 | |
| *** tri2sing has quit IRC | 03:13 | |
| *** baoli has quit IRC | 03:13 | |
| *** krotscheck_ is now known as krotscheck | 03:14 | |
| rkmrhj | macsz: Excuse me...I reported "https://bugs.launchpad.net/nova/+bug/1587285". And I correct reproduce steps because that was switched to "Incomplete". | 03:15 |
| openstack | Launchpad bug 1587285 in OpenStack Compute (nova) "Volume status will be changed to "available" in spite of still attached to VM instance" [Undecided,Incomplete] - Assigned to Rikimaru Honjo (honjo-rikimaru-c6) | 03:15 |
| rkmrhj | macsz: Will you recheck this report? | 03:15 |
| *** mingyu has joined #openstack-nova | 03:16 | |
| *** gouthamr has quit IRC | 03:19 | |
| *** shaohe_feng has quit IRC | 03:20 | |
| *** dimtruck is now known as zz_dimtruck | 03:22 | |
| *** shaohe_feng has joined #openstack-nova | 03:24 | |
| *** pwnall has joined #openstack-nova | 03:28 | |
| *** Jeffrey4l has quit IRC | 03:28 | |
| *** pwnall has quit IRC | 03:28 | |
| *** shaohe_feng has quit IRC | 03:30 | |
| *** shaohe_feng has joined #openstack-nova | 03:31 | |
| *** tonytan_brb has quit IRC | 03:34 | |
| *** armax has quit IRC | 03:36 | |
| *** links has joined #openstack-nova | 03:37 | |
| *** amotoki has quit IRC | 03:39 | |
| *** shaohe_feng has quit IRC | 03:41 | |
| *** shaohe_feng has joined #openstack-nova | 03:41 | |
| macsz | rkmrhj sure | 03:44 |
| *** mkoderer__ has quit IRC | 03:46 | |
| *** ljjjustin has quit IRC | 03:46 | |
| *** stew925 has quit IRC | 03:47 | |
| *** beagles has quit IRC | 03:47 | |
| *** rwmjones has quit IRC | 03:47 | |
| *** purplerbot has quit IRC | 03:47 | |
| *** markmcclain has quit IRC | 03:47 | |
| *** mfisch has quit IRC | 03:48 | |
| *** rama_y has quit IRC | 03:48 | |
| *** rwmjones has joined #openstack-nova | 03:48 | |
| *** ljjjustin has joined #openstack-nova | 03:49 | |
| *** mingyu has quit IRC | 03:49 | |
| *** raunak has quit IRC | 03:50 | |
| *** shaohe_feng has quit IRC | 03:51 | |
| *** coolsvap has joined #openstack-nova | 03:51 | |
| *** shaohe_feng has joined #openstack-nova | 03:51 | |
| *** liusheng has quit IRC | 03:52 | |
| *** sc68cal has quit IRC | 03:52 | |
| *** aorourke has quit IRC | 03:52 | |
| *** liusheng has joined #openstack-nova | 03:52 | |
| *** b3nt_pin has joined #openstack-nova | 03:52 | |
| *** sc68cal has joined #openstack-nova | 03:52 | |
| *** b3nt_pin is now known as Guest54563 | 03:52 | |
| *** mfisch has joined #openstack-nova | 03:52 | |
| *** aorourke has joined #openstack-nova | 03:52 | |
| *** mingyu has joined #openstack-nova | 03:52 | |
| *** mkoderer__ has joined #openstack-nova | 03:52 | |
| *** mfisch has quit IRC | 03:53 | |
| *** mfisch has joined #openstack-nova | 03:53 | |
| *** code-R has joined #openstack-nova | 03:53 | |
| *** markmcclain has joined #openstack-nova | 03:54 | |
| *** raunak has joined #openstack-nova | 03:54 | |
| *** unicell has joined #openstack-nova | 03:56 | |
| *** fragatina has quit IRC | 03:57 | |
| *** fragatina has joined #openstack-nova | 03:58 | |
| *** fragatina has quit IRC | 03:58 | |
| *** code-R has quit IRC | 04:00 | |
| *** shaohe_feng has quit IRC | 04:01 | |
| *** shaohe_feng has joined #openstack-nova | 04:02 | |
| macsz | rkmrhj: it is 11 PM here, i will go back to this tomorrow, have to put some more thought into it | 04:03 |
| *** Jeffrey4l has joined #openstack-nova | 04:06 | |
| *** JoseMello has quit IRC | 04:10 | |
| *** pwnall has joined #openstack-nova | 04:11 | |
| *** pwnall has quit IRC | 04:11 | |
| *** shaohe_feng has quit IRC | 04:11 | |
| *** shaohe_feng has joined #openstack-nova | 04:12 | |
| *** daidv has joined #openstack-nova | 04:17 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 04:18 |
| *** vishwanathj has quit IRC | 04:20 | |
| *** vishwanathj has joined #openstack-nova | 04:21 | |
| *** shaohe_feng has quit IRC | 04:22 | |
| *** shaohe_feng has joined #openstack-nova | 04:22 | |
| *** unicell has quit IRC | 04:25 | |
| *** ivase_ has joined #openstack-nova | 04:25 | |
| *** ivase__ has joined #openstack-nova | 04:26 | |
| *** ivase has quit IRC | 04:29 | |
| *** ivase_ has quit IRC | 04:30 | |
| *** shaohe_feng has quit IRC | 04:32 | |
| *** unicell has joined #openstack-nova | 04:32 | |
| *** shaohe_feng has joined #openstack-nova | 04:33 | |
| *** tonytan4ever has joined #openstack-nova | 04:34 | |
| *** rajul has joined #openstack-nova | 04:37 | |
| rajul | Guys I am facing an wrt VM suspension. As per OpenStack docs, when a VM is suspended, state of the VM is stored on the disk and the resources are released to free pool. I created multiple VM till all the resources got exhausted, then suspended a few and tried to create a new VM. But it throws an error that "No host available to fulfill this request". | 04:39 |
| *** tonytan4ever has quit IRC | 04:39 | |
| rajul | Previously, there is a bug (Bug #1402502) raised on this as well on launchpad but that's marked abandoned. So anyone having any idea if a fix came up OR will come up OR any way around this problem? Thanks | 04:39 |
| openstack | bug 1402502 in openstack-manuals "Resource usage will not be updated when suspending instance" [Medium,Fix released] https://launchpad.net/bugs/1402502 - Assigned to Liyingjun (liyingjun) | 04:39 |
| *** rkmrhj has quit IRC | 04:41 | |
| *** Jeffrey4l has quit IRC | 04:42 | |
| *** shaohe_feng has quit IRC | 04:42 | |
| *** shaohe_feng has joined #openstack-nova | 04:42 | |
| *** Jeffrey4l has joined #openstack-nova | 04:47 | |
| *** clayton has quit IRC | 04:48 | |
| *** clayton has joined #openstack-nova | 04:48 | |
| *** jwcroppe has joined #openstack-nova | 04:49 | |
| *** ivase has joined #openstack-nova | 04:49 | |
| *** NightKhaos has quit IRC | 04:50 | |
| *** NightKhaos has joined #openstack-nova | 04:50 | |
| *** ivase__ has quit IRC | 04:52 | |
| *** tri2sing_ has quit IRC | 04:52 | |
| *** shaohe_feng has quit IRC | 04:52 | |
| *** shaohe_feng has joined #openstack-nova | 04:53 | |
| *** ijw has joined #openstack-nova | 04:54 | |
| *** jwcroppe has quit IRC | 04:54 | |
| *** ijw has quit IRC | 04:59 | |
| *** prateek_ has joined #openstack-nova | 05:00 | |
| *** dikonoor has joined #openstack-nova | 05:00 | |
| *** shaohe_feng has quit IRC | 05:03 | |
| *** shaohe_feng has joined #openstack-nova | 05:03 | |
| *** chandankumar has joined #openstack-nova | 05:04 | |
| *** ratailor has joined #openstack-nova | 05:05 | |
| *** raunak has quit IRC | 05:07 | |
| *** ecelik has joined #openstack-nova | 05:09 | |
| *** gcb has joined #openstack-nova | 05:11 | |
| *** lpetrut has joined #openstack-nova | 05:11 | |
| *** mingyu has quit IRC | 05:13 | |
| *** shaohe_feng has quit IRC | 05:13 | |
| *** shaohe_feng has joined #openstack-nova | 05:14 | |
| *** claudiub has quit IRC | 05:19 | |
| *** sdake has quit IRC | 05:20 | |
| *** oanson has joined #openstack-nova | 05:21 | |
| *** pbandark has joined #openstack-nova | 05:21 | |
| *** ivase has quit IRC | 05:21 | |
| *** jaosorior has joined #openstack-nova | 05:23 | |
| *** shaohe_feng has quit IRC | 05:23 | |
| *** shaohe_feng has joined #openstack-nova | 05:23 | |
| *** mingyu has joined #openstack-nova | 05:24 | |
| *** jichen has joined #openstack-nova | 05:27 | |
| *** rcernin has joined #openstack-nova | 05:29 | |
| *** mingyu has quit IRC | 05:29 | |
| *** lpetrut has quit IRC | 05:31 | |
| *** shaohe_feng has quit IRC | 05:33 | |
| *** lpetrut has joined #openstack-nova | 05:34 | |
| *** shaohe_feng has joined #openstack-nova | 05:35 | |
| *** ekuris_ has joined #openstack-nova | 05:35 | |
| jichen | alex_xu: do you know whether nova client has compatible capability to older version? e.g latest client still can talk to liberty code or mitaka nova code ? I guess no but where can I find a wiki or doc about it ? | 05:38 |
| *** liverpooler has joined #openstack-nova | 05:42 | |
| *** david-lyle_ has joined #openstack-nova | 05:43 | |
| *** liverpooler is now known as yrabl|WFH | 05:43 | |
| *** jed56_ has joined #openstack-nova | 05:43 | |
| *** yrabl|WFH is now known as liverpooler | 05:43 | |
| *** liverpooler has quit IRC | 05:44 | |
| *** serverascode_ has joined #openstack-nova | 05:44 | |
| *** shaohe_feng has quit IRC | 05:44 | |
| *** shaohe_feng has joined #openstack-nova | 05:44 | |
| *** syfy323_ has joined #openstack-nova | 05:44 | |
| *** liverpooler has joined #openstack-nova | 05:44 | |
| *** beekneemech has joined #openstack-nova | 05:45 | |
| *** StevenK_ has joined #openstack-nova | 05:46 | |
| *** jed56 has quit IRC | 05:48 | |
| *** rha has quit IRC | 05:48 | |
| *** nicolasbock has quit IRC | 05:48 | |
| *** david-lyle has quit IRC | 05:48 | |
| *** syfy323 has quit IRC | 05:48 | |
| *** bapalm has quit IRC | 05:48 | |
| *** sfinucan has quit IRC | 05:48 | |
| *** mvk has quit IRC | 05:48 | |
| *** serverascode has quit IRC | 05:48 | |
| *** karthiks has quit IRC | 05:48 | |
| *** bnemec has quit IRC | 05:48 | |
| *** StevenK has quit IRC | 05:48 | |
| *** bapalm_ has joined #openstack-nova | 05:48 | |
| *** jed56_ is now known as jed56 | 05:49 | |
| *** Hosam has joined #openstack-nova | 05:49 | |
| *** mvk has joined #openstack-nova | 05:49 | |
| *** nicolasbock has joined #openstack-nova | 05:49 | |
| *** liverpoo1er has joined #openstack-nova | 05:49 | |
| *** karthiks has joined #openstack-nova | 05:50 | |
| *** liverpooler has quit IRC | 05:50 | |
| *** sfinucan has joined #openstack-nova | 05:50 | |
| *** rha has joined #openstack-nova | 05:50 | |
| *** liverpooler has joined #openstack-nova | 05:51 | |
| *** jwcroppe has joined #openstack-nova | 05:51 | |
| *** serverascode_ is now known as serverascode | 05:51 | |
| *** shaohe_feng has quit IRC | 05:54 | |
| *** shaohe_feng has joined #openstack-nova | 05:55 | |
| *** jwcroppe has quit IRC | 05:56 | |
| *** Hosam_ has joined #openstack-nova | 05:57 | |
| *** Hosam has quit IRC | 06:00 | |
| *** daidv_ has joined #openstack-nova | 06:01 | |
| *** daidv_ has quit IRC | 06:01 | |
| *** rkmrhj has joined #openstack-nova | 06:02 | |
| *** shaohe_feng has quit IRC | 06:04 | |
| *** mingyu has joined #openstack-nova | 06:04 | |
| *** shaohe_feng has joined #openstack-nova | 06:05 | |
| rkmrhj | macsz: Oh, sorry. I got it. Thanks. | 06:05 |
| rkmrhj | (I quit once.) | 06:07 |
| *** rkmrhj has quit IRC | 06:08 | |
| *** shaohe_feng has quit IRC | 06:14 | |
| *** vishwanathj has quit IRC | 06:15 | |
| *** sridharg has joined #openstack-nova | 06:15 | |
| *** vishwanathj has joined #openstack-nova | 06:15 | |
| *** irenab_ has joined #openstack-nova | 06:16 | |
| *** shaohe_feng has joined #openstack-nova | 06:18 | |
| *** irenab has quit IRC | 06:19 | |
| *** irenab_ is now known as irenab | 06:19 | |
| *** rajul has quit IRC | 06:19 | |
| *** code-R has joined #openstack-nova | 06:20 | |
| *** shaohe_feng has quit IRC | 06:25 | |
| *** shaohe_feng has joined #openstack-nova | 06:26 | |
| *** lpetrut has quit IRC | 06:30 | |
| *** shaohe_feng has quit IRC | 06:35 | |
| *** shaohe_feng has joined #openstack-nova | 06:35 | |
| *** shihanzhang has joined #openstack-nova | 06:38 | |
| *** shz has quit IRC | 06:41 | |
| *** nkrinner_afk is now known as nkrinner | 06:45 | |
| *** shaohe_feng has quit IRC | 06:45 | |
| *** pcaruana has joined #openstack-nova | 06:45 | |
| *** shaohe_feng has joined #openstack-nova | 06:46 | |
| *** moshele has joined #openstack-nova | 06:47 | |
| *** tesseract- has joined #openstack-nova | 06:53 | |
| *** Jeffrey4l has quit IRC | 06:55 | |
| *** Jeffrey4l has joined #openstack-nova | 06:55 | |
| *** shaohe_feng has quit IRC | 06:55 | |
| *** shaohe_feng has joined #openstack-nova | 06:56 | |
| *** liverpoo1er has quit IRC | 06:58 | |
| *** code-R has quit IRC | 06:59 | |
| *** daidv has quit IRC | 07:00 | |
| *** irenab_ has joined #openstack-nova | 07:00 | |
| *** irenab has quit IRC | 07:01 | |
| *** irenab_ is now known as irenab | 07:01 | |
| openstackgerrit | John Hua proposed openstack/nova: Use physical utilisation for cached images https://review.openstack.org/353862 | 07:02 |
| *** shaohe_feng has quit IRC | 07:06 | |
| *** claudiub has joined #openstack-nova | 07:07 | |
| kevinz | Hi | 07:08 |
| *** shaohe_feng has joined #openstack-nova | 07:09 | |
| *** ijw has joined #openstack-nova | 07:11 | |
| openstackgerrit | John Hua proposed openstack/nova: Use physical utilisation for cached images https://review.openstack.org/353862 | 07:14 |
| *** ijw has quit IRC | 07:15 | |
| *** shaohe_feng has quit IRC | 07:16 | |
| *** shaohe_feng has joined #openstack-nova | 07:16 | |
| *** kevinz has quit IRC | 07:18 | |
| *** matrohon has joined #openstack-nova | 07:19 | |
| *** kevinz has joined #openstack-nova | 07:25 | |
| *** claudiub has quit IRC | 07:26 | |
| *** shaohe_feng has quit IRC | 07:26 | |
| *** shaohe_feng has joined #openstack-nova | 07:27 | |
| *** ecelik has quit IRC | 07:33 | |
| *** ecelik has joined #openstack-nova | 07:35 | |
| *** shaohe_feng has quit IRC | 07:36 | |
| *** shaohe_feng has joined #openstack-nova | 07:37 | |
| *** zenoway has joined #openstack-nova | 07:39 | |
| *** zenoway has quit IRC | 07:39 | |
| *** zenoway has joined #openstack-nova | 07:40 | |
| *** markus_z has joined #openstack-nova | 07:42 | |
| *** cdent has joined #openstack-nova | 07:44 | |
| *** kevinz has quit IRC | 07:46 | |
| *** shaohe_feng has quit IRC | 07:47 | |
| *** kevinz has joined #openstack-nova | 07:47 | |
| *** purplerbot has joined #openstack-nova | 07:47 | |
| *** Oku_OS-away is now known as Oku_OS | 07:49 | |
| *** shaohe_feng has joined #openstack-nova | 07:49 | |
| *** ratailor is now known as ratailor|Lunch | 07:50 | |
| *** tonytan4ever has joined #openstack-nova | 07:51 | |
| *** jwcroppe has joined #openstack-nova | 07:53 | |
| *** jaosorior is now known as jaosorior_brb | 07:53 | |
| *** moshele has quit IRC | 07:55 | |
| *** ecelik has quit IRC | 07:56 | |
| *** tonytan4ever has quit IRC | 07:56 | |
| *** shaohe_feng has quit IRC | 07:57 | |
| *** jwcroppe has quit IRC | 07:57 | |
| tdurakov | johnthetubaguy: hi, are you around? | 07:58 |
| *** shaohe_feng has joined #openstack-nova | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** danpb has joined #openstack-nova | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** moshele has joined #openstack-nova | 08:07 | |
| *** shaohe_feng has quit IRC | 08:07 | |
| *** shaohe_feng has joined #openstack-nova | 08:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** rmart04 has joined #openstack-nova | 08:13 | |
| *** xenogear_ has quit IRC | 08:15 | |
| *** lucas-dinner is now known as lucasagomes | 08:16 | |
| *** shaohe_feng has quit IRC | 08:17 | |
| *** shaohe_feng has joined #openstack-nova | 08:18 | |
| *** moshele has quit IRC | 08:21 | |
| *** sdake has joined #openstack-nova | 08:23 | |
| *** moshele has joined #openstack-nova | 08:24 | |
| *** vladani has joined #openstack-nova | 08:27 | |
| *** shaohe_feng has quit IRC | 08:28 | |
| *** shaohe_feng has joined #openstack-nova | 08:28 | |
| *** abalutoiu has joined #openstack-nova | 08:31 | |
| *** ihrachys has joined #openstack-nova | 08:33 | |
| *** yassine has joined #openstack-nova | 08:33 | |
| *** yassine is now known as Guest94303 | 08:33 | |
| *** shaohe_feng has quit IRC | 08:38 | |
| johnthetubaguy | tdurakov: I am around now | 08:39 |
| *** shaohe_feng has joined #openstack-nova | 08:39 | |
| tdurakov | johnthetubaguy: could you please review this patch, https://review.openstack.org/#/c/353002/ we've discussed that on live-migration subteam meeting | 08:39 |
| tdurakov | so the idea is to exclude unstable test from live-migration job for now | 08:40 |
| johnthetubaguy | sdague: yeah, those listen addresses, we probably should move them back to string so we don't dtop hostnames working | 08:40 |
| tdurakov | johnthetubaguy: tests, that were executed http://logs.openstack.org/02/353002/1/check/gate-tempest-dsvm-multinode-live-migration/ca6e996/console.html#_2016-08-09_17_14_57_479491 | 08:41 |
| *** coolsvap is now known as _coolsvap_ | 08:41 | |
| johnthetubaguy | tdurakov: oh, good idea, is that the one where we think its a libvirt bug? | 08:42 |
| tdurakov | johnthetubaguy: yes, that one | 08:42 |
| tdurakov | I've added link to it in comment | 08:42 |
| johnthetubaguy | yeah, had a quick scan of it, was just making sure its the one I thought it was | 08:43 |
| *** jaosorior_brb is now known as jaosorior | 08:44 | |
| *** ratailor|Lunch is now known as ratailor | 08:47 | |
| *** shaohe_feng has quit IRC | 08:48 | |
| johnthetubaguy | tdurakov: yeah, looking at the logs, it seems to be doing the right thing | 08:48 |
| johnthetubaguy | tdurakov: there are two skips related to iscsi in there, but that seems correct | 08:49 |
| *** shaohe_feng has joined #openstack-nova | 08:49 | |
| tdurakov | johnthetubaguy: iscsi are configurable, so I've tried to exclude only one, thanks for review! | 08:49 |
| johnthetubaguy | tdurakov: yeah, totally the right way to do it | 08:50 |
| tdurakov | johnthetubaguy: btw, haven't seen that policy https://governance.openstack.org/reference/tags/assert_supports-upgrade.html until you had mentioned it in https://review.openstack.org/#/c/323313/ What do you think about this one: https://review.openstack.org/#/c/344254/ - while that also violates this policy, AFAIR it's impossbile to boot instances with | 08:53 |
| tdurakov | negative overcommit ratios | 08:53 |
| *** jwcroppe has joined #openstack-nova | 08:54 | |
| johnthetubaguy | tdurakov: I am OK with that | 08:54 |
| johnthetubaguy | tdurakov: its a bit like microversions and 500 errors I guess, it makes it easier for admins to not allow crazy stuff | 08:54 |
| johnthetubaguy | tdurakov: FWIW the only reasons that min is not already there, is the patch to add min in oslo hadn't been released at the time we went through that file | 08:55 |
| *** sudipto has joined #openstack-nova | 08:57 | |
| *** sudipto_ has joined #openstack-nova | 08:57 | |
| *** jwcroppe has quit IRC | 08:58 | |
| *** shaohe_feng has quit IRC | 08:58 | |
| tdurakov | johnthetubaguy: ok, thanks for explanation | 08:59 |
| *** shaohe_feng has joined #openstack-nova | 08:59 | |
| *** mingyu has quit IRC | 09:00 | |
| *** bkopilov has joined #openstack-nova | 09:00 | |
| *** bkopilov has quit IRC | 09:00 | |
| *** bkopilov has joined #openstack-nova | 09:01 | |
| *** phuongnh has joined #openstack-nova | 09:01 | |
| *** sameo_ has joined #openstack-nova | 09:01 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Add deprecated_since parameter https://review.openstack.org/353631 | 09:01 |
| *** permalac has quit IRC | 09:02 | |
| *** rkrum has quit IRC | 09:02 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 09:03 |
| openstackgerrit | John Garbutt proposed openstack/nova: Maintain backwards compat for listen opts https://review.openstack.org/353917 | 09:07 |
| *** chandankumar has quit IRC | 09:08 | |
| *** shaohe_feng has quit IRC | 09:09 | |
| *** shaohe_feng has joined #openstack-nova | 09:09 | |
| *** chandankumar has joined #openstack-nova | 09:12 | |
| openstackgerrit | John Garbutt proposed openstack/nova: Improve help text for service options. https://review.openstack.org/332861 | 09:12 |
| openstackgerrit | Andrey Shestakov proposed openstack/nova: Add support for detaching interfaces in ironic https://review.openstack.org/347410 | 09:14 |
| *** cdent has quit IRC | 09:15 | |
| *** Hosam has joined #openstack-nova | 09:16 | |
| *** Hosam__ has joined #openstack-nova | 09:17 | |
| *** Hosam_ has quit IRC | 09:17 | |
| *** shaohe_feng has quit IRC | 09:19 | |
| *** shaohe_feng has joined #openstack-nova | 09:20 | |
| *** Hosam has quit IRC | 09:21 | |
| *** roeyc has joined #openstack-nova | 09:22 | |
| *** mingyu has joined #openstack-nova | 09:23 | |
| *** sonofthor has joined #openstack-nova | 09:29 | |
| *** shaohe_feng has quit IRC | 09:29 | |
| *** shaohe_feng has joined #openstack-nova | 09:29 | |
| *** lpetrut has joined #openstack-nova | 09:30 | |
| *** sonofthor has quit IRC | 09:33 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Maintenance reason to server https://review.openstack.org/310510 | 09:37 |
| *** wangqun has quit IRC | 09:38 | |
| *** _coolsvap_ is now known as coolsvap | 09:38 | |
| *** shaohe_feng has quit IRC | 09:39 | |
| *** shaohe_feng has joined #openstack-nova | 09:40 | |
| *** roeyc has quit IRC | 09:43 | |
| openstackgerrit | John Hua proposed openstack/nova: Use physical utilisation for cached images https://review.openstack.org/353862 | 09:45 |
| *** cdent has joined #openstack-nova | 09:47 | |
| *** mingyu has quit IRC | 09:49 | |
| *** gcb has quit IRC | 09:49 | |
| *** shaohe_feng has quit IRC | 09:50 | |
| *** shaohe_feng has joined #openstack-nova | 09:50 | |
| *** mvk has quit IRC | 09:52 | |
| *** mvk has joined #openstack-nova | 09:52 | |
| *** matrohon has quit IRC | 09:54 | |
| *** mvk has quit IRC | 09:56 | |
| *** mvk has joined #openstack-nova | 09:56 | |
| *** shaohe_feng has quit IRC | 10:00 | |
| *** shaohe_feng has joined #openstack-nova | 10:01 | |
| * mvk is away: Auto-away after 5 mins idle (gone at 11th Aug, 12:01:25) | 10:01 | |
| *** mvk is now known as mvk`afk | 10:01 | |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Skipping test_volume_backed_live_migration for live_migration job https://review.openstack.org/353002 | 10:03 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for resource inventories to placement API https://review.openstack.org/329152 | 10:05 |
| *** mvk`afk is now known as mvk | 10:07 | |
| * mvk returns (Auto-away after 5 mins idle [11m 4s]) (total away time: 11m 4s) | 10:07 | |
| *** mvk has quit IRC | 10:07 | |
| *** matrohon has joined #openstack-nova | 10:08 | |
| *** mvk has joined #openstack-nova | 10:08 | |
| *** hoangcx has quit IRC | 10:10 | |
| *** shaohe_feng has quit IRC | 10:10 | |
| *** shaohe_feng has joined #openstack-nova | 10:11 | |
| *** zerda2 has quit IRC | 10:13 | |
| *** sdague has joined #openstack-nova | 10:16 | |
| *** lpetrut has quit IRC | 10:17 | |
| *** phuongnh has quit IRC | 10:17 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Set vnic_type correctly for ironic https://review.openstack.org/339143 | 10:19 |
| *** shaohe_feng has quit IRC | 10:20 | |
| *** shaohe_feng has joined #openstack-nova | 10:21 | |
| tdurakov | markus_z: hi, are you around? | 10:22 |
| *** mingyu has joined #openstack-nova | 10:27 | |
| *** wangqun has joined #openstack-nova | 10:28 | |
| *** EinstCrazy has quit IRC | 10:29 | |
| *** shaohe_feng has quit IRC | 10:31 | |
| *** fragatina has joined #openstack-nova | 10:31 | |
| *** shaohe_feng has joined #openstack-nova | 10:32 | |
| *** fragatina has quit IRC | 10:33 | |
| *** fragatina has joined #openstack-nova | 10:34 | |
| *** gszasz has joined #openstack-nova | 10:35 | |
| *** amotoki has joined #openstack-nova | 10:35 | |
| *** hieulq has quit IRC | 10:37 | |
| markus_z | tdurakov: yep, I'm here, right back from lunch | 10:38 |
| tdurakov | markus_z: read this thread test strategy for the serial console feature | 10:39 |
| tdurakov | markus_z: so, there will be not only live-migration job for this, right? | 10:40 |
| markus_z | tdurakov: right | 10:40 |
| tdurakov | markus_z: so, what job are you trying to patch for that? | 10:40 |
| *** amotoki has quit IRC | 10:40 | |
| markus_z | tdurakov: that feature was basically untested and I had bugs for use cases in a single-node setup and multi-node-setup. | 10:41 |
| tdurakov | markus_z: yeah, understood that:) | 10:41 |
| *** shaohe_feng has quit IRC | 10:41 | |
| markus_z | tdurakov: I thought of changing the "gate-nova-python27-db-ubuntu-xenial" | 10:41 |
| *** Guest54563 is now known as beagles | 10:41 | |
| *** rkrum has joined #openstack-nova | 10:42 | |
| tdurakov | markus_z: I thought it doesn't use tempest, am I wrong? | 10:42 |
| *** mvk is now known as mvk_afk | 10:42 | |
| *** chandankumar has quit IRC | 10:42 | |
| markus_z | tdurakov: oh crap, you're right | 10:42 |
| *** mvk_afk is now known as mvk | 10:43 | |
| markus_z | tdurakov: Hm, let me think of another one | 10:43 |
| *** shaohe_feng has joined #openstack-nova | 10:44 | |
| markus_z | tdurakov: I think the "gate-tempest-dsvm-postgres-full-ubuntu-xenial" would do as well. | 10:44 |
| *** Hosam has joined #openstack-nova | 10:44 | |
| markus_z | tdurakov: I don't want to lose the coverage for the "console log" test cases | 10:45 |
| tdurakov | markus_z: probably sdague will help picking good candidate for patching? I expect it would be in-place one, as you've done for live-migration | 10:45 |
| sdague | what's going on? | 10:45 |
| tdurakov | I mean patch nova and tempest *after* devstack was deployed | 10:45 |
| tdurakov | sdague: hi, could you take a look at [openstack-dev] [nova] test strategy for the serial console feature on openstack-dev? | 10:46 |
| markus_z | http://lists.openstack.org/pipermail/openstack-dev/2016-July/100029.html | 10:46 |
| *** hieulq has joined #openstack-nova | 10:47 | |
| tdurakov | sdague: I think markus_z wants to cover serial console scenarios for nova by doing in-place env patching to improve resources utilization, as we did for live-migration bits | 10:47 |
| *** Hosam__ has quit IRC | 10:47 | |
| markus_z | yes | 10:48 |
| markus_z | If it needs to be "in-place env patching", I'm not sure yet. | 10:48 |
| sdague | tdurakov: that' | 10:49 |
| sdague | 's not possible in normal jobs | 10:49 |
| tdurakov | sdague, so, it could be a separate one | 10:49 |
| tdurakov | or move hooks to nova | 10:49 |
| sdague | markus_z: what was the break | 10:49 |
| markus_z | sdague: https://bugs.launchpad.net/nova/+bug/1455252 | 10:50 |
| openstack | Launchpad bug 1455252 in OpenStack Compute (nova) "enabling serial console breaks live migration" [High,In progress] - Assigned to sahid (sahid-ferdjaoui) | 10:50 |
| *** fragatin_ has joined #openstack-nova | 10:50 | |
| markus_z | sdague: https://bugs.launchpad.net/nova/+bug/1595962 | 10:50 |
| openstack | Launchpad bug 1595962 in OpenStack Compute (nova) "live migration with disabled vnc/spice not possible" [Undecided,In progress] - Assigned to Markus Zoeller (markus_z) (mzoeller) | 10:50 |
| markus_z | These two ^, different use cases | 10:51 |
| sdague | markus_z: so it's not the serial console broke | 10:51 |
| markus_z | s/use cases/root cause | 10:51 |
| *** shaohe_feng has quit IRC | 10:51 | |
| sdague | it's that the serial console interfers with live migration | 10:51 |
| sdague | in that case, I'd actually look at putting this into the dedicated live migration job | 10:52 |
| *** shaohe_feng has joined #openstack-nova | 10:52 | |
| tdurakov | sdague: for live-migration https://review.openstack.org/#/c/347471/ but as I understood the plan was to add testing for other cases too | 10:52 |
| markus_z | The tempest change series I have is here: https://review.openstack.org/#/c/346815/3 | 10:54 |
| sdague | tdurakov: well, I'd argue that it's not broken other places, so we shouldn't waste the CI time there | 10:54 |
| *** rkrum has quit IRC | 10:54 | |
| *** fragatina has quit IRC | 10:54 | |
| markus_z | sdague: If a live-migration can be done, and the serial console is accessible after that, that should cover most error cases. | 10:55 |
| sdague | markus_z: right | 10:55 |
| *** jed56 has quit IRC | 10:55 | |
| sdague | none of which is tested in tempest | 10:55 |
| tdurakov | sdague, markus_z: so the plan is to cover live-migration bits, and not testing other scenarios in single node job, right? I'm fine wth that | 10:55 |
| *** jwcroppe has joined #openstack-nova | 10:55 | |
| sdague | tdurakov: I think that's probably fine | 10:55 |
| sdague | it's also just weird that we have a dedicated action for this and that as a user you have to know if the back end is serial or not | 10:56 |
| markus_z | sdague: It was done like VNC/RDP/SPICE I think | 10:57 |
| tdurakov | sdague, markus_z btw about live-migration job https://review.openstack.org/#/c/353002/ :) | 10:57 |
| sdague | markus_z: the other longer term thing is whether or not this makes sense as an image / flavor attribute, in which case we could actually create serial console guests during a run, and not reconfigure | 10:58 |
| sdague | it seems like an image attribute at least, because the image needs to be configured for this | 10:58 |
| sdague | markus_z: https://review.openstack.org/#/c/346815/3/tempest/api/compute/servers/test_server_actions.py - doesn't actually test the serial console is actually working | 10:58 |
| sdague | it only tests the metadata | 10:58 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: WIP: Improve Nova and Neutron interactions https://review.openstack.org/353982 | 10:59 |
| *** jwcroppe has quit IRC | 11:00 | |
| markus_z | sdague: agreed to "flavor/image attr" | 11:00 |
| markus_z | sdague: true, I can remove that change for "get_serial_console" | 11:01 |
| sdague | oh, you have a whole series there | 11:01 |
| *** gongysh has quit IRC | 11:01 | |
| markus_z | I did is as atomic as possible to discuss it more easily. | 11:01 |
| alex_xu | jichen: not sure anywhere we doc something for that | 11:01 |
| *** JoseMello has joined #openstack-nova | 11:01 | |
| *** shaohe_feng has quit IRC | 11:01 | |
| sdague | markus_z: right, so honestly, right now in the cycle, I don't think we'd change the configuration of any of our main jobs | 11:02 |
| *** shaohe_feng has joined #openstack-nova | 11:02 | |
| *** ijw has joined #openstack-nova | 11:02 | |
| sdague | I would focus on the live migration test for now, and to make this more testable in future, think about an image/flavor attribute during the next cycle | 11:03 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Remove block_migration argument from live migration operations https://review.openstack.org/295316 | 11:03 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Remove block_migration from LM post operations https://review.openstack.org/295315 | 11:03 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: Remove block_migration from LM rollback https://review.openstack.org/292826 | 11:03 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for inventories to placement API https://review.openstack.org/329152 | 11:03 |
| markus_z | sdague: agreed, I'll focus the tests on live-migration and its hooks | 11:03 |
| tdurakov | sdague, thanks for helping | 11:04 |
| markus_z | sdague: I'm going to put some thought into the flavor/image attr as well. | 11:04 |
| sdague | tdurakov: no prob | 11:05 |
| markus_z | yep, I see a path now, thanks sdague and tdurakov | 11:05 |
| markus_z | *forwards | 11:05 |
| markus_z | tdurakov: You want me to review https://review.openstack.org/#/c/353002/2 or was it just for my information? | 11:06 |
| *** ijw has quit IRC | 11:07 | |
| *** wangqun has quit IRC | 11:07 | |
| *** wangqun has joined #openstack-nova | 11:07 | |
| tdurakov | markus_z: please review:) | 11:10 |
| markus_z | tdurakov: yep, will do in ~2h | 11:11 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 11:11 |
| *** bkopilov has quit IRC | 11:11 | |
| *** chandankumar has joined #openstack-nova | 11:12 | |
| *** shaohe_feng has quit IRC | 11:12 | |
| sdague | markus_z: so... I honestly want to object to this whole hacking rule - https://review.openstack.org/#/c/352473 | 11:12 |
| *** shaohe_feng has joined #openstack-nova | 11:13 | |
| *** wangqun has quit IRC | 11:13 | |
| markus_z | sdague: The whole rule? I wouldn't mind honestly. | 11:14 |
| sdague | markus_z: yeh, the whole rule | 11:14 |
| markus_z | sdague: np, will remove it in the next ps | 11:15 |
| sdague | hacking rules checking english is just silly | 11:15 |
| sdague | we have 2 human code reviewers for a reason :) | 11:15 |
| *** mingyu has quit IRC | 11:16 | |
| *** sshnaidm has joined #openstack-nova | 11:18 | |
| *** sean-k-mooneyAFK is now known as sean-k-mooney | 11:21 | |
| sdague | cdent: why are your patches on top of the hacking rule change? | 11:21 |
| cdent | sdague: because markus_z 's patches to change some conf for database are on top of them and the placement api requires those database changes | 11:22 |
| *** shaohe_feng has quit IRC | 11:22 | |
| markus_z | yep ^ | 11:22 |
| sdague | ah, ok, so, we should spin this sooner rather than later | 11:22 |
| *** shaohe_feng has joined #openstack-nova | 11:23 | |
| *** moshele has quit IRC | 11:24 | |
| sdague | markus_z: you respinning those patches now, or you want me to dive in and help? | 11:25 |
| markus_z | sdague: I'm in the middle right now, a few more minutes | 11:25 |
| sdague | ok, cool, I'll get more coffee then review | 11:25 |
| cdent | concurrency is hard, let's go shopping (for coffee) | 11:26 |
| *** bvanhav has joined #openstack-nova | 11:28 | |
| *** jaosorior has quit IRC | 11:29 | |
| *** jaosorior has joined #openstack-nova | 11:30 | |
| *** moshele has joined #openstack-nova | 11:30 | |
| *** shaohe_feng has quit IRC | 11:32 | |
| *** liusheng has quit IRC | 11:33 | |
| *** shaohe_feng has joined #openstack-nova | 11:33 | |
| *** liusheng has joined #openstack-nova | 11:33 | |
| *** sshnaidm has quit IRC | 11:35 | |
| sdague | ok, these conf things are crazy | 11:39 |
| sdague | if we had more time, I'd really want us not to go down this path | 11:39 |
| markus_z | sdague: I would remove the hacking check for placement in "nova/conf/" too, is that ok? | 11:40 |
| markus_z | sdague: which conf things? | 11:40 |
| sdague | markus_z: yeh | 11:40 |
| sdague | this whole enhance_help_text function that's introspecing options in a library and modifying ours on the fly | 11:40 |
| markus_z | sdague: that was the best tradeoff I could find to avoid maintaining the same help texts multiple times. | 11:41 |
| sdague | it's fine to maintain text | 11:41 |
| sdague | it's super easy to understand | 11:42 |
| sdague | and super clear to fix bugs | 11:42 |
| sdague | this new thing means it's super hard to figure out how to fix bad wording in a config file | 11:42 |
| *** shaohe_feng has quit IRC | 11:42 | |
| sdague | but, anyway, I don't want to rat hole on this and block placement API going in | 11:43 |
| *** shaohe_feng has joined #openstack-nova | 11:43 | |
| sdague | so lets get those other things fixed, we'll merge it and move on | 11:43 |
| *** rtheis has joined #openstack-nova | 11:48 | |
| *** chandankumar has quit IRC | 11:50 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Extend nfv feature matrix with pinning/NUMA https://review.openstack.org/327126 | 11:51 |
| *** lucasagomes is now known as lucas-hungry | 11:52 | |
| *** tonytan4ever has joined #openstack-nova | 11:53 | |
| *** shaohe_feng has quit IRC | 11:53 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Extend nfv feature matrix with pinning/NUMA https://review.openstack.org/327126 | 11:53 |
| *** shaohe_feng has joined #openstack-nova | 11:54 | |
| *** sudipto has quit IRC | 11:55 | |
| *** sudipto_ has quit IRC | 11:55 | |
| markus_z | got it. The last tests are running on that change series, I will push it in a few. | 11:55 |
| *** tonytan4ever has quit IRC | 11:57 | |
| markus_z | sdague: The placement db doesn't necessary depend on my patches. It's still fine to do the copy/paste and find a better way later IMO. | 11:58 |
| markus_z | s/necessary/necessarily | 11:58 |
| sdague | markus_z: well, right now in gerrit they do, so lets just move forward with it | 11:58 |
| markus_z | sdague: OK, just read your comment and wanted to point out the possibility | 11:59 |
| *** gouthamr has joined #openstack-nova | 11:59 | |
| cdent | I agree that we should just move forward with things as they are for now. | 11:59 |
| markus_z | fine with me | 12:00 |
| cdent | It also seems like there is a longer term issue with how texts related to config are used. It is true that maintaining text is easy. But it is also true that needing to duplicate configs all over the place is lame. | 12:00 |
| cdent | more for barca? | 12:00 |
| sdague | cdent: sure, in which case, make it part of oslo.db (I put a suggestion in the comment) | 12:01 |
| sdague | keystoneauth works like that now | 12:01 |
| cdent | +1 | 12:01 |
| * cdent hasn't seen the comment yet | 12:01 | |
| cdent | (because of concurrent coffee acquisition) | 12:02 |
| markus_z | Ah, I didn't know about the way ksa does this | 12:02 |
| *** shaohe_feng has quit IRC | 12:03 | |
| sdague | markus_z: yeh, like this - https://github.com/openstack/nova/blob/609bdea996b5aa8de852ca340ca02d3b6f314f72/nova/conf/neutron.py#L183-L184 | 12:04 |
| *** shaohe_feng has joined #openstack-nova | 12:04 | |
| cdent | bbs | 12:04 |
| *** cdent has quit IRC | 12:04 | |
| sdague | anyway... Ocata | 12:04 |
| *** mingyu has joined #openstack-nova | 12:06 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: improve help text of database (related) options (1/2) https://review.openstack.org/301238 | 12:08 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Remove hacking checks [N342] + [N347] for config options. https://review.openstack.org/352473 | 12:08 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: improve help text of database (related) options (2/2) https://review.openstack.org/320043 | 12:08 |
| sdague | markus_z: why did you remove the centralized rule? | 12:09 |
| *** baoli has joined #openstack-nova | 12:10 | |
| *** gongysh has joined #openstack-nova | 12:10 | |
| *** mvk is now known as mvk_afk | 12:10 | |
| *** annp has quit IRC | 12:10 | |
| markus_z | sdague: that's what I asked you previously | 12:10 |
| *** mvk_afk is now known as mvk | 12:11 | |
| sdague | ok, sorry, I guess I didn't understand | 12:11 |
| markus_z | I can put it back if you want to | 12:11 |
| sdague | yeh, lets just remove the help text one, because that's not likely to need discussion | 12:11 |
| markus_z | ok, on my way | 12:12 |
| sdague | sorry, I read placement in that sentence as placement api | 12:12 |
| *** tuanla_ has quit IRC | 12:12 | |
| sdague | so my bad | 12:12 |
| *** shaohe_feng has quit IRC | 12:13 | |
| *** shaohe_feng has joined #openstack-nova | 12:14 | |
| jaypipes | sdague: should be a relatively quick and easy one: https://review.openstack.org/#/c/352473/ | 12:15 |
| *** pradk has joined #openstack-nova | 12:16 | |
| jaypipes | sdague: it's markus_z's removal of that hacking check. | 12:16 |
| *** hshiina has joined #openstack-nova | 12:16 | |
| sdague | jaypipes: yes, we've been talking about it here | 12:16 |
| jaypipes | ah, sorry.. | 12:16 |
| sdague | I'm all for deleting N347, there was a miscommunication on N342 when markus_z spun the last patch, because I expect other people want to keep that one | 12:17 |
| jaypipes | oh | 12:18 |
| jaypipes | yeah, sorry, I didn't catch that removal... | 12:18 |
| markus_z | I feel like I took the placement api change hostage, that wasn't my intention. | 12:20 |
| markus_z | I'm not even good enough with guns and police negotiations | 12:20 |
| *** Guest52040 has joined #openstack-nova | 12:20 | |
| *** Hosam_ has joined #openstack-nova | 12:21 | |
| sdague | heh | 12:21 |
| *** matrohon has quit IRC | 12:22 | |
| *** rfolco has joined #openstack-nova | 12:23 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: improve help text of database (related) options (1/2) https://review.openstack.org/301238 | 12:23 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Remove hacking check [N347] for config options. https://review.openstack.org/352473 | 12:23 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: config options: improve help text of database (related) options (2/2) https://review.openstack.org/320043 | 12:23 |
| *** shaohe_feng has quit IRC | 12:24 | |
| *** Hosam has quit IRC | 12:24 | |
| *** shaohe_feng has joined #openstack-nova | 12:24 | |
| markus_z | I'll get me some coffee, just ping me if you want changes there ^ | 12:26 |
| *** mvk is now known as mvk_afk | 12:27 | |
| *** gongysh has quit IRC | 12:28 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: rename test_tracker to test_resource_tracker https://review.openstack.org/353801 | 12:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: delete test_resource_tracker.py https://review.openstack.org/353800 | 12:28 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: Create multiple resize claim unit test https://review.openstack.org/352625 | 12:28 |
| *** edmondsw has joined #openstack-nova | 12:28 | |
| *** ivase has joined #openstack-nova | 12:29 | |
| sdague | markus_z: good with those now, thanks | 12:29 |
| *** ratailor has quit IRC | 12:31 | |
| openstackgerrit | Chen Fan proposed openstack/nova: virt/image: between two words without a space in output message https://review.openstack.org/354030 | 12:31 |
| jaypipes | tdurakov: see review 352625 above. addressed your excellent review comments. please re-review at your leisure, sir! | 12:32 |
| *** mdrabe has joined #openstack-nova | 12:32 | |
| tdurakov | jaypipes: sure, will take a look, thank you | 12:33 |
| *** prateek_ has quit IRC | 12:33 | |
| *** shaohe_feng has quit IRC | 12:34 | |
| *** zz_dimtruck is now known as dimtruck | 12:34 | |
| *** shaohe_feng has joined #openstack-nova | 12:34 | |
| *** gongysh has joined #openstack-nova | 12:35 | |
| *** matrohon has joined #openstack-nova | 12:35 | |
| *** matrohon has quit IRC | 12:36 | |
| *** lpetrut has joined #openstack-nova | 12:37 | |
| *** mvk_afk has quit IRC | 12:40 | |
| *** mvk has joined #openstack-nova | 12:40 | |
| *** mnestratov has joined #openstack-nova | 12:41 | |
| *** lpetrut has quit IRC | 12:41 | |
| *** gongysh has quit IRC | 12:41 | |
| *** cristicalin has joined #openstack-nova | 12:43 | |
| *** burgerk has joined #openstack-nova | 12:43 | |
| *** sdake has quit IRC | 12:43 | |
| *** rlrossit has joined #openstack-nova | 12:43 | |
| *** lpetrut has joined #openstack-nova | 12:43 | |
| *** shaohe_feng has quit IRC | 12:44 | |
| *** devkulkarni1 has joined #openstack-nova | 12:44 | |
| *** shaohe_feng has joined #openstack-nova | 12:45 | |
| *** cdent has joined #openstack-nova | 12:46 | |
| *** baoli_ has joined #openstack-nova | 12:47 | |
| *** takedakn has joined #openstack-nova | 12:48 | |
| *** oanson has quit IRC | 12:48 | |
| *** hamzy has quit IRC | 12:48 | |
| *** matrohon has joined #openstack-nova | 12:49 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: [WIP] Init allocated ports set https://review.openstack.org/347479 | 12:49 |
| *** baoli has quit IRC | 12:50 | |
| *** lpetrut has quit IRC | 12:50 | |
| *** lpetrut has joined #openstack-nova | 12:51 | |
| *** tonytan4ever has joined #openstack-nova | 12:54 | |
| *** shaohe_feng has quit IRC | 12:54 | |
| *** lucas-hungry is now known as lucasagomes | 12:55 | |
| *** shaohe_feng has joined #openstack-nova | 12:55 | |
| *** jwcroppe has joined #openstack-nova | 12:57 | |
| *** tonytan4ever has quit IRC | 12:57 | |
| *** tonytan4ever has joined #openstack-nova | 12:57 | |
| *** avolkov has quit IRC | 12:58 | |
| *** avolkov has joined #openstack-nova | 12:58 | |
| *** mvk has quit IRC | 12:59 | |
| *** jwcroppe has quit IRC | 13:01 | |
| *** jamesdenton has joined #openstack-nova | 13:01 | |
| *** shaohe_feng has quit IRC | 13:05 | |
| *** shaohe_feng has joined #openstack-nova | 13:05 | |
| lennyb | wznoinsk, around? | 13:06 |
| *** jwcroppe has joined #openstack-nova | 13:06 | |
| *** ecelik__ has joined #openstack-nova | 13:06 | |
| *** cristicalin has quit IRC | 13:06 | |
| *** xyang1 has joined #openstack-nova | 13:07 | |
| *** ecelik__ has quit IRC | 13:07 | |
| *** esracelik has joined #openstack-nova | 13:07 | |
| *** esracelik is now known as ecelik | 13:08 | |
| *** kylek3h has joined #openstack-nova | 13:10 | |
| *** rodrigods has quit IRC | 13:10 | |
| *** rodrigods has joined #openstack-nova | 13:10 | |
| *** mvk has joined #openstack-nova | 13:11 | |
| sdague | cdent: I'm confused about this - https://review.openstack.org/#/c/329151/59/nova/api/openstack/placement/util.py@30 | 13:15 |
| *** shaohe_feng has quit IRC | 13:15 | |
| * cdent looks | 13:15 | |
| cdent | sdague: confused how/why? | 13:15 |
| *** shaohe_feng has joined #openstack-nova | 13:15 | |
| sdague | there being an seemingly unused private function in this space | 13:16 |
| sdague | this is because this decorator, instead of modifying this function, registers this function into some other part of the code tree? | 13:16 |
| cdent | it registers a FormatChecker on jsonschema itself | 13:17 |
| openstackgerrit | devdatta-kulkarni proposed openstack/nova: [WIP - DO NOT MERGE] Testing LXC Gate https://review.openstack.org/354062 | 13:17 |
| cdent | so outside the tree | 13:17 |
| sdague | so much magic ... ok | 13:17 |
| openstackgerrit | jichenjc proposed openstack/nova: Replace fake_utils by using Fixture https://review.openstack.org/230954 | 13:18 |
| *** beagles is now known as beagles_brb | 13:18 | |
| wznoinsk | lennyb: yes | 13:19 |
| cdent | sdague: yeah, I didn't have uuid validation in place until a reviewer requested that I make it specific in the schemas being used. Doing it this way removes a coupling with the nova validators modules; intentional to ease later fun. | 13:19 |
| *** rkrum has joined #openstack-nova | 13:20 | |
| *** dimtruck is now known as zz_dimtruck | 13:21 | |
| lennyb | wznoinsk, are you running CI on stable/mitaka and liberty? | 13:23 |
| sdague | cdent: so... I think you are going to have to rebase that set on markus_z's because zuul won't given that the patches were explicitly stacked up | 13:23 |
| cdent | sdague: yeah, we'll get to that in just a moment | 13:23 |
| wznoinsk | lennyb: yes | 13:24 |
| *** shaohe_feng has quit IRC | 13:25 | |
| cdent | edleafe: just from a learning standpoint if you're around and get a chance I'm eager to read a response from you on the exact string stuff in those tests you commented on (I left a comment in response) | 13:25 |
| *** ashestakov has quit IRC | 13:26 | |
| *** Hosam_ has quit IRC | 13:26 | |
| *** beagles_brb is now known as beagles | 13:27 | |
| *** Hosam has joined #openstack-nova | 13:27 | |
| *** shaohe_feng has joined #openstack-nova | 13:28 | |
| lennyb | wznoinsk, great, thanks. I have some issues with urllib3 version. I will recheck it first. | 13:28 |
| *** jistr has quit IRC | 13:28 | |
| *** tonytan4ever has quit IRC | 13:29 | |
| *** eharney has joined #openstack-nova | 13:29 | |
| *** mvk has quit IRC | 13:29 | |
| *** moshele has quit IRC | 13:30 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Optional separate database for placement API https://review.openstack.org/349021 | 13:30 |
| wznoinsk | lennyb: found one in last days on stable/mitaka was urllib had to be in versions >=3.15 otherwise it doesn't support 'chunked' calls, we were using an old devstack commit to workaround a different issue which cause this time to cause urllib problem... which was appearing only when the tempest was about to run | 13:30 |
| *** jistr has joined #openstack-nova | 13:30 | |
| lennyb | wznoinsk, yeap. exactly. | 13:31 |
| wznoinsk | lennyb: the above was spotted on https://review.openstack.org/#/c/352884/, fixed on our CI side and rechecked | 13:31 |
| *** Hosam has quit IRC | 13:31 | |
| wznoinsk | lennyb: recheck pls, it's fixed now | 13:31 |
| *** ducttape_ has quit IRC | 13:31 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add placement API web utility methods https://review.openstack.org/352573 | 13:31 |
| *** matrohon has quit IRC | 13:32 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: Add support for resource_providers urls https://review.openstack.org/329151 | 13:32 |
| jaypipes | sdague: ^^ just rebases on top of markus_z's latest helptext things. I workflow'd them all. | 13:33 |
| *** diogogmt has quit IRC | 13:33 | |
| jaypipes | tonyb: now that you are Requirements PTL, are you going to run for Preferences PTL as well? | 13:34 |
| *** diogogmt has joined #openstack-nova | 13:34 | |
| tonyb | jaypipes: has that thing happened? | 13:35 |
| lennyb | wznoinsk, thanks. I will try that | 13:35 |
| jaypipes | tonyb: since you are Requirements PTL and requirements > preferences, I'm sure you can will that into being. ;P | 13:35 |
| markus_z | tdurakov: To run https://review.openstack.org/#/c/348836/4 I need a "depends-on" on the nova-live-migration-hook change, right? | 13:35 |
| lennyb | wznoinsk, what about stable/liberty? | 13:35 |
| *** shaohe_feng has quit IRC | 13:35 | |
| *** mriedem has joined #openstack-nova | 13:36 | |
| tdurakov | markus_z: yes | 13:36 |
| *** jaosorior is now known as jaosorior_mtg | 13:36 | |
| *** shaohe_feng has joined #openstack-nova | 13:36 | |
| markus_z | tdurakov: got it. I'm going to squash some of the tempest tests for the serial console and let the nova change depend on it. I have to rebase the nova change anyhow :) | 13:37 |
| openstackgerrit | jichenjc proposed openstack/nova: Verify os-aggregates.inc on sample files https://review.openstack.org/333114 | 13:38 |
| *** matrohon has joined #openstack-nova | 13:39 | |
| *** takedakn has quit IRC | 13:40 | |
| *** devkulkarni1 has quit IRC | 13:40 | |
| johnthetubaguy | jaypipes: you mean soft requirements right :p | 13:41 |
| *** prateek has joined #openstack-nova | 13:41 | |
| cdent | johnthetubaguy: just say no to both | 13:42 |
| johnthetubaguy | heh | 13:42 |
| *** ekuris_ has quit IRC | 13:42 | |
| *** mriedem has quit IRC | 13:43 | |
| *** daemontool has joined #openstack-nova | 13:43 | |
| *** mriedem has joined #openstack-nova | 13:43 | |
| *** mgoddard_ has joined #openstack-nova | 13:44 | |
| *** hemanthm|away is now known as hemanthm | 13:45 | |
| mriedem | danpb: woot https://review.openstack.org/#/c/350595/ | 13:45 |
| mriedem | #success nova is now using the os-vif library | 13:45 |
| danpb | mriedem: yeah, too alot of rechecks ! | 13:45 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: add filtering by attrs to resource_providers https://review.openstack.org/343046 | 13:45 |
| openstackstatus | mriedem: Added success to Success page | 13:45 |
| danpb | s/too/took/ | 13:45 |
| *** shaohe_feng has quit IRC | 13:46 | |
| jroll | mriedem: \o/ | 13:46 |
| * danpb wonders where this "Success page" is and how much it is trolled | 13:46 | |
| johnthetubaguy | awesome stuff! | 13:46 |
| *** prateek has quit IRC | 13:46 | |
| *** shaohe_feng has joined #openstack-nova | 13:46 | |
| jroll | danpb: https://wiki.openstack.org/wiki/Successes | 13:46 |
| *** prateek has joined #openstack-nova | 13:47 | |
| danpb | #success ate a tasty lunch | 13:47 |
| openstackstatus | danpb: Added success to Success page | 13:47 |
| *** sdake has joined #openstack-nova | 13:47 | |
| markus_z | :D | 13:47 |
| *** mgoddard has quit IRC | 13:47 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 13:48 |
| *** sonofthor has joined #openstack-nova | 13:48 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 13:48 |
| markus_z | well, you're in Britain, that's worth a success story I guess ;) | 13:48 |
| mriedem | ha | 13:48 |
| johnthetubaguy | heh | 13:48 |
| jroll | lol | 13:48 |
| wznoinsk | lennyb: should be fine too | 13:48 |
| *** hamzy has joined #openstack-nova | 13:49 | |
| *** pwnall has joined #openstack-nova | 13:49 | |
| cdent | sdague, jaypipes: https://review.openstack.org/#/c/343046/ and https://review.openstack.org/#/c/340601/ are the next parts of the placement stack and rebased and ready for final review | 13:49 |
| *** diogogmt has quit IRC | 13:50 | |
| ildikov | mriedem: johnthetubaguy: hi | 13:50 |
| *** ayoung has joined #openstack-nova | 13:51 | |
| ildikov | mriedem: johnthetubaguy: I fixed the comments in the remove check_attch patch: https://review.openstack.org/#/c/335358/ and I also added a bunch of mox to mock changes in two patches in a row before that change | 13:51 |
| ildikov | mriedem: johnthetubaguy: if you happen to have some time and check that would be great :) | 13:51 |
| jaypipes | cdent: rock on. will get to that in a bit. | 13:52 |
| jaypipes | currently working on reviewing mdbooth's unit test refactoring of imagebackend | 13:52 |
| *** mvk has joined #openstack-nova | 13:53 | |
| johnthetubaguy | ildikov: OK, I should take a look at that soon | 13:53 |
| *** hoangcx has joined #openstack-nova | 13:54 | |
| *** takashin has joined #openstack-nova | 13:54 | |
| *** rlrossit_ has joined #openstack-nova | 13:55 | |
| *** code-R has joined #openstack-nova | 13:55 | |
| *** zz_dimtruck is now known as dimtruck | 13:56 | |
| *** shaohe_feng has quit IRC | 13:56 | |
| prateek | cdent, jaypipes : can you please look into this when you have time https://review.openstack.org/#/c/327706/ ? | 13:56 |
| *** pwnall has quit IRC | 13:56 | |
| *** shaohe_feng has joined #openstack-nova | 13:56 | |
| *** Rodrigo_BR has joined #openstack-nova | 13:57 | |
| *** code-R_ has joined #openstack-nova | 13:57 | |
| *** pwnall has joined #openstack-nova | 13:57 | |
| *** ecelik has quit IRC | 13:57 | |
| *** ljxiash has joined #openstack-nova | 13:58 | |
| Rodrigo_BR | Hi guys, during the live migration my instance change the status to Shutdown, I cannot understand de cause. This is my log http://paste.openstack.org/show/554147/ | 13:58 |
| *** rlrossit has quit IRC | 13:58 | |
| prateek | cdent, jaypipes there were couple of comments and i have addressed them , please let me know if you want any further changes | 13:59 |
| *** mlavalle has joined #openstack-nova | 13:59 | |
| *** code-R has quit IRC | 14:00 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Improve help text for compute manager options https://review.openstack.org/329639 | 14:00 |
| mriedem | nova meeting now in #openstack-meeting | 14:00 |
| *** anush_ has joined #openstack-nova | 14:00 | |
| *** doffm_ is now known as doffm | 14:00 | |
| *** baoli_ has quit IRC | 14:02 | |
| mdbooth | jaypipes: I've tried to address your concerns about them being less than perfectly 'unity' in the commit message. In short, most of the test are already not very unity. Some of the others I've had to change the unit test boundary for the purposes of the refactor, because I'm changing the interfaces they previously stopped at. | 14:02 |
| *** tblakes has joined #openstack-nova | 14:02 | |
| mdbooth | There are also a lot of perfunctory tests in there, unfortunately. | 14:03 |
| pkoniszewski | Rodrigo_BR: it's weird, it looks like the same operations were performed multiple times | 14:03 |
| *** eharney has quit IRC | 14:04 | |
| jaypipes | mdbooth: yeah, slowly getting through it all... :) | 14:04 |
| pkoniszewski | Rodrigo_BR: e.g. post operations after live migration were triggered 3 times, which is not normal | 14:05 |
| *** tonytan4ever has joined #openstack-nova | 14:05 | |
| mdbooth | jaypipes: I hope that the fixture is, at least, a little easier to follow. I tried to simplify it as much as possible without pushing any complexity into the tests. | 14:06 |
| Rodrigo_BR | pkoniszewski: | 14:06 |
| dansmith | alaski: you and me here after the meeting right? | 14:06 |
| *** shaohe_feng has quit IRC | 14:06 | |
| jaypipes | mdbooth: it is indeed. | 14:06 |
| alaski | dansmith: yes sir | 14:06 |
| mdbooth | Cool | 14:06 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: live-migration test for serial console https://review.openstack.org/347471 | 14:06 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: Acquire TCP ports for console during live migration https://review.openstack.org/275801 | 14:06 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: fix live-migration with serial console check https://review.openstack.org/335132 | 14:06 |
| Rodrigo_BR | pkoniszewski: after the live migrate I start the instance manually with success | 14:06 |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: Add functional regression test for bug 1595962 https://review.openstack.org/338416 | 14:06 |
| openstack | bug 1595962 in OpenStack Compute (nova) "live migration with disabled vnc/spice not possible" [Undecided,In progress] https://launchpad.net/bugs/1595962 - Assigned to Markus Zoeller (markus_z) (mzoeller) | 14:06 |
| *** sdake_ has joined #openstack-nova | 14:07 | |
| *** ducttape_ has joined #openstack-nova | 14:07 | |
| *** octopuszhang has joined #openstack-nova | 14:07 | |
| *** tonytan4ever has quit IRC | 14:07 | |
| *** octopuszhang has left #openstack-nova | 14:07 | |
| *** ducttape_ has quit IRC | 14:07 | |
| prateek | Rodrigo_BR, the instance is being shut off by itself | 14:08 |
| *** ducttape_ has joined #openstack-nova | 14:08 | |
| prateek | and nova is just syncing the power state | 14:08 |
| *** tonytan4ever has joined #openstack-nova | 14:08 | |
| prateek | Rodrigo_BR, so the operation is happening from libvirt, can you please check if you are out of disk or RAM ? | 14:08 |
| *** shaohe_feng has joined #openstack-nova | 14:08 | |
| *** Hieu has joined #openstack-nova | 14:09 | |
| prateek | Rodrigo_BR, that is a possible reason for the VM being shut down by libvirt | 14:09 |
| *** sdake has quit IRC | 14:09 | |
| tdurakov | prateek: it doesn't explains 3 post_live_migration in logs | 14:09 |
| tdurakov | Rodrigo_BR: which openstack version are you using? | 14:11 |
| Rodrigo_BR | prateek: I have only 1 instance this is the first test after deployement with openstack-ansible | 14:11 |
| Rodrigo_BR | tdurakov: mitaka | 14:11 |
| tdurakov | Rodrigo_BR: could you please do: nova instance-action-list fe889ac9-0078-4441-a22d-94cdaa8a160d | 14:11 |
| tdurakov | and nova migration-list | 14:12 |
| prateek | tdurakov, +1 | 14:12 |
| *** thorst_ has joined #openstack-nova | 14:12 | |
| tdurakov | after you get all actions on your instnce, please do nova instance-action fe889ac9-0078-4441-a22d-94cdaa8a160d req-uuid | 14:12 |
| *** burgerk has quit IRC | 14:13 | |
| *** tonytan_brb has joined #openstack-nova | 14:14 | |
| Rodrigo_BR | tdurakov: here are http://paste.openstack.org/show/554175/ | 14:14 |
| *** tonytan4ever has quit IRC | 14:14 | |
| *** rkrum has quit IRC | 14:15 | |
| tdurakov | Rodrigo_BR: interesting | 14:16 |
| *** jistr is now known as jistr|call | 14:16 | |
| tdurakov | there are 6 attempts to do live-migration | 14:16 |
| *** shaohe_feng has quit IRC | 14:16 | |
| tdurakov | and according to this output everything works fine:-D | 14:16 |
| Rodrigo_BR | tdurakov: yes I try a lot of times | 14:17 |
| *** sonofthor has quit IRC | 14:17 | |
| tdurakov | which doesn't correlates to log output | 14:17 |
| *** shaohe_feng has joined #openstack-nova | 14:17 | |
| tdurakov | Rodrigo_BR: I wonder, do you have some message-brocker issues at that time? | 14:17 |
| Rodrigo_BR | tdurakovthis is the problem I cannot see any error in logfiles | 14:17 |
| tdurakov | Rodrigo_BR: no traces at all? | 14:18 |
| pkoniszewski | tdurakov: i think that stop action is unexpected there, is it? | 14:18 |
| *** Hieu has quit IRC | 14:18 | |
| *** eharney has joined #openstack-nova | 14:18 | |
| tdurakov | pkoniszewski: as I understood from Rodrigo_BR he do it manyally, no? | 14:18 |
| pkoniszewski | tdurakov: no, he does not stop VMs manually - "during the live migration my instance change the status to Shutdown" | 14:19 |
| pkoniszewski | Rodrigo_BR: ^, correct? | 14:20 |
| ihrachys | seeing this in neutron job: http://logs.openstack.org/64/353664/2/check/gate-tempest-dsvm-neutron-linuxbridge/591295c/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-08-11_12_48_06_630 | 14:20 |
| ihrachys | operation failed: failed to read XML | 14:20 |
| ihrachys | any ideas? | 14:20 |
| tdurakov | pkoniszewski: it still doesn't explains multiple post steps | 14:22 |
| *** unicell has quit IRC | 14:22 | |
| Rodrigo_BR | durakov: I didn't stop ... my instance is running when a try the live migration to another host it change state to shutdown | 14:22 |
| pkoniszewski | tdurakov: look at times, this is only in log | 14:23 |
| *** beekneemech is now known as bnemec | 14:23 | |
| tdurakov | Rodrigo_BR: well, this request comes from api, definetly | 14:23 |
| *** unicell has joined #openstack-nova | 14:24 | |
| tdurakov | Rodrigo_BR: any additional users to stop vm? | 14:24 |
| *** rrecio has joined #openstack-nova | 14:25 | |
| Rodrigo_BR | tdurakov: look this http://paste.openstack.org/show/554178/ | 14:26 |
| *** cmartin has joined #openstack-nova | 14:26 | |
| *** shaohe_feng has quit IRC | 14:27 | |
| *** tonytan_brb is now known as tonytan4ever | 14:27 | |
| *** shaohe_feng has joined #openstack-nova | 14:28 | |
| tdurakov | Rodrigo_BR: could you please enable debug logging on computes | 14:28 |
| openstackgerrit | vu tran proposed openstack/nova: Not allow overcommit ratios to be negative https://review.openstack.org/344254 | 14:28 |
| tdurakov | and trigger live-migration once again | 14:28 |
| tdurakov | then please send logs | 14:28 |
| pkoniszewski | tdurakov: it does not need to come from the API | 14:29 |
| tdurakov | pkoniszewski: there is request id | 14:29 |
| pkoniszewski | tdurakov: when nova syncs power state of a VM and it is different it call the stop API | 14:29 |
| pkoniszewski | tdurakov: I mean, it might not need to be another user, it might be nova | 14:30 |
| *** Guest52040 has quit IRC | 14:30 | |
| *** amotoki has joined #openstack-nova | 14:31 | |
| jichen | mriedem: do you know any place that we mentioned , novaclient's backward compatible capability , that is , latest nova client still works with previous (liberty mitaka nova ) or some restrictions? | 14:33 |
| jichen | mriedem: I consulted with Alex and he don't know about it .. | 14:33 |
| openstackgerrit | jichenjc proposed openstack/nova: Add comment about how status field changed https://review.openstack.org/354119 | 14:33 |
| mriedem | jichen: i don't think that's documented, | 14:34 |
| mriedem | we use microversions and semver to signal compat | 14:34 |
| mriedem | i.e. we release a major version when we drop something in a backward incompatible way | 14:34 |
| cdent | sdague: quick easy win in the placement api context, but not in the stack: https://review.openstack.org/#/c/352945/ | 14:34 |
| Rodrigo_BR | tdurakov: Yes I will do this ! | 14:35 |
| jichen | mriedem: ok, thanks, I will go back to release history and find more info, and seems 2.35 is latest ? https://launchpad.net/python-novaclient/+series | 14:35 |
| sdague | cdent: that's on a stack that starts with a merge conflict? | 14:36 |
| cdent | sdague: no it's based off master, it just has the same topic with a bunch of stuff that is currently in merge conflict | 14:36 |
| sdague | ah | 14:36 |
| mriedem | jichen: yes | 14:37 |
| *** shaohe_feng has quit IRC | 14:37 | |
| *** shaohe_feng has joined #openstack-nova | 14:37 | |
| jichen | mriedem: ok, thanks~ | 14:37 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs: WIP: Improve Nova and Neutron interactions https://review.openstack.org/353982 | 14:37 |
| sdague | cdent: done | 14:37 |
| cdent | thanks sdague | 14:37 |
| *** burgerk has joined #openstack-nova | 14:38 | |
| Rodrigo_BR | tdurakov: logs with debug http://paste.openstack.org/show/554182/ | 14:42 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for inventories to placement API https://review.openstack.org/329152 | 14:42 |
| *** oanson has joined #openstack-nova | 14:43 | |
| *** raj_singh_ has joined #openstack-nova | 14:43 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Improve 404 and 405 response tests https://review.openstack.org/331483 | 14:43 |
| *** matrohon has quit IRC | 14:43 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server lock action https://review.openstack.org/351100 | 14:43 |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server suspend action https://review.openstack.org/353344 | 14:44 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a Usage and UsageList object https://review.openstack.org/329575 | 14:44 |
| tdurakov | Rodrigo_BR: what about another compute? | 14:44 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for usages in the placement API https://review.openstack.org/329643 | 14:44 |
| Rodrigo_BR | tdurakov: Sorry I don't understand | 14:45 |
| ihrachys | danpb: neutron hit a gate breakage with linuxbridge job. we are looking at our side, but could someone from nova folks also check it? https://bugs.launchpad.net/nova/+bug/1612281 can it be related to os-vif switch? | 14:45 |
| jroll | mriedem: sdague: hey, apparently we accidentally made our job in nova voting when we refactored a bunch of stuff, this fixes it: https://review.openstack.org/#/c/354124/1 | 14:45 |
| openstack | Launchpad bug 1612281 in neutron "Neutron Linuxbridge jobs failing with 'operation failed: failed to read XML'" [Critical,New] | 14:45 |
| * jroll doesn't want to get yelled at when that resets the gate a day before feature freeze | 14:45 | |
| tdurakov | Rodrigo_BR: http://paste.openstack.org/show/554182/ - there are logs only for ==> compute1/nova-compute.log <==, what's about second compute? | 14:46 |
| tdurakov | also logs are being dupliated even for pre_live_migration step | 14:46 |
| *** devkulkarni1 has joined #openstack-nova | 14:46 | |
| *** sdake has joined #openstack-nova | 14:47 | |
| sdague | jroll: thanks | 14:47 |
| *** sdake_ has quit IRC | 14:47 | |
| jroll | np | 14:47 |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server rebuild https://review.openstack.org/352767 | 14:47 |
| tdurakov | I'd say i could see 3 very same messages for pre-live-migration | 14:47 |
| Rodrigo_BR | tdurakov: ok I will post again | 14:47 |
| *** shaohe_feng has quit IRC | 14:47 | |
| *** dikonoor has quit IRC | 14:47 | |
| cdent | jroll: we can get someone to yell at you for something else | 14:47 |
| jroll | cdent: totes, the gate rage is just a bit too much for me :) | 14:48 |
| *** shaohe_feng has joined #openstack-nova | 14:48 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server rescue action https://review.openstack.org/353252 | 14:49 |
| *** diogogmt has joined #openstack-nova | 14:51 | |
| danpb | ihrachys: hmm, you're saying in that comments though that this problem started before os-vif conversion merged | 14:51 |
| *** jdurgin1 has joined #openstack-nova | 14:51 | |
| danpb | ihrachys: os-vif merged at 4:45, but you said failures in neutron stsarted at 3:00 | 14:51 |
| *** Hosam has joined #openstack-nova | 14:51 | |
| *** matrohon has joined #openstack-nova | 14:51 | |
| *** Hosam_ has joined #openstack-nova | 14:51 | |
| ihrachys | danpb: it's hard to say, grafana is not very helpful when it comes to exact pinpoint of the first case. lemme play with logstash. | 14:52 |
| mriedem | dansmith: alaski: jaypipes: johnthetubaguy: https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1283 | 14:52 |
| openstackgerrit | Sean Dague proposed openstack/nova: Add tests for user_id policy enforcement on start/stop https://review.openstack.org/353335 | 14:53 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1309 | 14:53 |
| alaski | yeah, I was fairly certain a size must be passed in for the bdm | 14:53 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1316 | 14:53 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1327 | 14:53 |
| dansmith | don't we also already have another meaning for root_gb=0 or root_gb=-1? | 14:53 |
| mriedem | yup, | 14:53 |
| mriedem | so flavor is only used for size on ephemeral and swap | 14:54 |
| mriedem | else size must be provided for bfv | 14:54 |
| alaski | there's no -1. but root_gb=0 means use the image size | 14:54 |
| *** takashin has left #openstack-nova | 14:55 | |
| ihrachys | danpb: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22operation%20failed%3A%20failed%20to%20read%20XML%5C%22 | 14:55 |
| ihrachys | danpb: first hit at 5:30 | 14:55 |
| Rodrigo_BR | tdurakov: here are http://paste.openstack.org/show/554187/ and http://paste.openstack.org/show/554188/ | 14:55 |
| mriedem | alaski: also, remember https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1239 :) | 14:55 |
| *** Hosam has quit IRC | 14:55 | |
| mriedem | we don't create a bdm in the db if the volume_size is - | 14:56 |
| *** ljxiash has quit IRC | 14:56 | |
| mriedem | *0 | 14:56 |
| danpb | ihrachys: ok, so that's after then | 14:56 |
| alaski | oh man, that thing | 14:56 |
| dansmith | so I just fundamentally don't want to tweak the instance flavor in this obscure way, and I also don't want to make this change for six months when we have a real solution on the horizon, | 14:56 |
| alaski | mriedem: yeah, why is that done? | 14:56 |
| dansmith | creating instances with special meanings that will last forever | 14:56 |
| dansmith | when this has been around since the beginning of time | 14:56 |
| mriedem | mdbooth: any idea why we don't create bdms in the api if the volume_size is 0? https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1239 | 14:56 |
| johnthetubaguy | alaski: hmm, totally my bad, I guess its the ephemeral stuff I was thinking about | 14:57 |
| *** mingyu has quit IRC | 14:57 | |
| mriedem | johnthetubaguy: were you waiting to pounce on open discussion? | 14:57 |
| *** shaohe_feng has quit IRC | 14:57 | |
| mdbooth | mriedem: I looked at that the other day... | 14:57 |
| johnthetubaguy | mriedem: I was just going to link to this one: https://review.openstack.org/#/c/353982 working on that stuff to talk to folks at the neutron midcycle, just a heads up | 14:57 |
| * mdbooth tries to remember what it was about | 14:57 | |
| *** shaohe_feng has joined #openstack-nova | 14:58 | |
| johnthetubaguy | mriedem: only started that this morning, but has some details | 14:58 |
| mriedem | mdbooth: i'm pretty sure i knew like a month ago when i was tracing all of that code for something, but i totally forget | 14:58 |
| tdurakov | Rodrigo_BR: anything else? I mean there is no mentions about live-migration at all http://paste.openstack.org/show/554187/ | 14:58 |
| mdbooth | mriedem: I knew 2 days ago, but it fell out :) | 14:58 |
| mdbooth | Gimme a minute to try to jog my memory | 14:58 |
| alaski | dansmith: I don't disagree. I didn't feel as strongly about it because all current root_gbs for bfv instances are meaningless so using 0 for some doesn't make much difference. but if we have another solution that's great | 14:58 |
| *** abhishekk has joined #openstack-nova | 14:58 | |
| *** matrohon has quit IRC | 14:58 | |
| dansmith | alaski: r-p is the solution | 14:58 |
| ihrachys | danpb: hm, the os-vif patch landed at 5:45, and we see hits ~5am | 14:59 |
| alaski | dansmith: yeah, and fixing the request_spec. that has to be done even with the r-p work | 14:59 |
| * ihrachys wonders whether his timezone in gerrit reflects logstash.. | 14:59 | |
| mriedem | jroll: so gate-tempest-dsvm-ironic-pxe_ipa-nv changed to tinyipa? | 14:59 |
| mriedem | jroll: i mean swapped the jobs that are used? | 15:00 |
| dansmith | alaski: it will be a lot cleaner too because we'll be specifically making out a list of requests for resources to the placement api | 15:00 |
| dansmith | alaski: instead of "request 0 disk" | 15:00 |
| *** abalutoiu has quit IRC | 15:00 | |
| *** Hieu has joined #openstack-nova | 15:00 | |
| danpb | ihrachys: gerrit says os-vif landed at 4;45, not 5:45 | 15:00 |
| ihrachys | danpb: it says 5:45 for me :-x | 15:01 |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server pause action https://review.openstack.org/352221 | 15:03 |
| *** code-R_ has quit IRC | 15:03 | |
| *** code-R has joined #openstack-nova | 15:03 | |
| mdbooth | mriedem: I *believe* it is because BDMs with zero size never become real disks | 15:03 |
| sdague | johnthetubaguy: I retooled gmann's patches into one series - https://review.openstack.org/352221 - and did the targets explicitly | 15:04 |
| ihrachys | danpb: btw searching for 'message:"libvirtError: Cannot get interface MTU on"' in logstash gives exact same timing | 15:04 |
| alaski | danpb: yep, overall much better | 15:05 |
| Rodrigo_BR | tdurakov: now is correct .... http://paste.openstack.org/show/554192/ and http://paste.openstack.org/show/554191/ sorry | 15:05 |
| mdbooth | mriedem: It was added in commit e05b3b11e67f18a6ff4867dfbc75554fd78cad1b , which added ephemeral disks | 15:05 |
| danpb | ihrachys: git timestapm is | 15:05 |
| danpb | Date: Thu Aug 11 03:45:43 2016 +0000 | 15:05 |
| lbeliveau | johnthetubaguy: reminder to review https://review.openstack.org/#/c/349060 | 15:05 |
| johnthetubaguy | sdague: ah, I will go hit that now ish | 15:06 |
| sdague | johnthetubaguy: I went and +2ed the stack as well, as I think that was my only complaint with it otherwise | 15:07 |
| *** armax has joined #openstack-nova | 15:07 | |
| *** raj_singh_ has quit IRC | 15:07 | |
| *** shaohe_feng has quit IRC | 15:08 | |
| johnthetubaguy | sdague: honestly, thats were I was at I think | 15:08 |
| mriedem | mdbooth: huh, great commit message | 15:08 |
| mriedem | "compute/api: pass down ephemeral device info" | 15:08 |
| mdbooth | mriedem: Indeed | 15:08 |
| *** shaohe_feng has joined #openstack-nova | 15:09 | |
| danpb | ihrachys: oh, i think i see the bug in nova - we're using the wrong attribute | 15:09 |
| dansmith | alaski: I think bauzas_off brought this up nicely right before my original -1 | 15:09 |
| *** migi is now known as migi_afk | 15:09 | |
| dansmith | alaski: he also seems to want the reqspec to request the thing we want, and not just always store root_gb=0 | 15:09 |
| ihrachys | HenryG: see danpb's last comment ^ | 15:10 |
| tdurakov | Rodrigo_BR: so, I could see pre_live_migration calls http://paste.openstack.org/show/554192/ - I assume it's destination node, but there are no logs for live-migration http://paste.openstack.org/show/554191/ - is it source? | 15:10 |
| dansmith | alaski: anyway, we can discuss the grenade thing now if you want | 15:10 |
| ihrachys | danpb: great. you think this is it? | 15:11 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: network: fix attribute names in os-vif conversion https://review.openstack.org/354143 | 15:12 |
| danpb | ihrachys: get some Neutron change and add a 'Depends-on' against that nova change^^^^ | 15:12 |
| *** penick has quit IRC | 15:12 | |
| alaski | dansmith: cool, I didn't see his comment. but seems like we're all on the same page | 15:12 |
| danpb | ihrachys: and let me know if CI then succeeeds | 15:12 |
| ihrachys | danpb: cool, on it | 15:12 |
| Rodrigo_BR | tdurakov: the paste.openstack are trunct my file | 15:12 |
| alaski | dansmith: so grenade... | 15:12 |
| dansmith | alaski: well, not really, I want to punt this until we have the proper fix, but.. anyway, grenade | 15:12 |
| danpb | i'm pretty confident this will be it, as its an obvious code bug | 15:12 |
| jroll | mriedem: it's the same job, we refactored the builders and how they are named (so it's easier to tell what exactly is going on in there) | 15:12 |
| mriedem | ok | 15:13 |
| jroll | mriedem: the patch I linked just puts it back to non-voting on nova again | 15:13 |
| alaski | dansmith: : https://etherpad.openstack.org/p/nova-newton-midcycle lines 373 and 376 | 15:13 |
| danpb | i guess the problem is that Nova CI only tests nova-network + linux bridge, and neutron + ovs, but not neutron + linux-bridge | 15:13 |
| dansmith | alaski: I made an initial stab at hacking grenade to run the api in the old side | 15:13 |
| markus_z | rlrossit_: Are you still working on this https://bugs.launchpad.net/nova/+bug/1581203 ? | 15:13 |
| openstack | Launchpad bug 1581203 in OpenStack Compute (nova) "Default policy allows unrestricted CRUD on os-server-tags" [Medium,In progress] - Assigned to Ryan Rossiter (rlrossit) | 15:13 |
| danpb | so we missed proper testing of this 3rd codepath | 15:13 |
| mriedem | jroll: yup, thanks | 15:14 |
| rlrossit_ | markus_z: nope, that change got abandoned and I'm no longer working on it. That can be freed up | 15:14 |
| markus_z | rlrossit_: ok, will do | 15:14 |
| jroll | np | 15:14 |
| *** Hieu has quit IRC | 15:14 | |
| ihrachys | danpb: https://review.openstack.org/354149 also posted the link into the comments for the nova fix | 15:14 |
| mriedem | danpb: there might be a neutron+lb job in the experimental queue, but probably not | 15:14 |
| openstackgerrit | Sarah Ulmer proposed openstack/python-novaclient: Added smaller flavors for novaclient functional tests to use https://review.openstack.org/354150 | 15:14 |
| *** matrohon has joined #openstack-nova | 15:14 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Improve consistency in libvirt https://review.openstack.org/353679 | 15:15 |
| markus_z | syjulian: This is resolved with the two partial fixes, isn't it? https://bugs.launchpad.net/nova/+bug/1295889 | 15:15 |
| openstack | Launchpad bug 1295889 in OpenStack Compute (nova) "No unit tests for nova.virt.firewall.IptablesFirewallDriver" [Low,In progress] - Assigned to Julian Sy (syjulian) | 15:15 |
| ihrachys | danpb: should we add Closes-Bug? | 15:16 |
| danpb | yeah just done so | 15:16 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: network: fix attribute names in os-vif conversion https://review.openstack.org/354143 | 15:16 |
| Rodrigo_BR | tdurakov: try see here https://justpaste.it/x7qp and https://justpaste.it/x7qq | 15:16 |
| *** jaosorior_mtg is now known as jaosorior | 15:17 | |
| *** Hieu has joined #openstack-nova | 15:17 | |
| *** hshiina has quit IRC | 15:17 | |
| *** shaohe_feng has quit IRC | 15:18 | |
| *** shaohe_feng has joined #openstack-nova | 15:18 | |
| *** eharney has quit IRC | 15:19 | |
| markus_z | BobBall: https://bugs.launchpad.net/nova/+bug/1554491 Should that script really be in-tree? | 15:19 |
| openstack | Launchpad bug 1554491 in OpenStack Compute (nova) "xenapi: tools/populate_other_config is broken" [Undecided,In progress] - Assigned to Sulochan Acharya (sulochan-acharya) | 15:19 |
| markus_z | The "xenserver" directory makes an orphaned impression | 15:21 |
| sdague | mriedem: there should be a mitaka job now on python-novaclient | 15:21 |
| syjulian | markus_z: there will be more tests added to that. The last bit has 300 ish loc. Should I just push all of those at once or push two? | 15:21 |
| mriedem | sdague: yup i saw that | 15:21 |
| mriedem | haven't seen it in action yet though | 15:21 |
| *** jistr|call is now known as jistr | 15:21 | |
| markus_z | syjulian: better two. squashing them is easier if that will be necessary | 15:22 |
| syjulian | markus_z: this was the original patch I'm splitting in chunks https://review.openstack.org/#/c/311758/5/nova/tests/unit/virt/test_firewall.py | 15:22 |
| sdague | I ran recheck on your patch just to see if it is working correctly | 15:22 |
| syjulian | gotcha | 15:22 |
| *** tesseract- has quit IRC | 15:23 | |
| markus_z | syjulian: Ah, that change is abandoned, that's why my script didn't find it. | 15:23 |
| edleafe | cdent: sorry, just catching up on IRC backscroll (meeting hell this morning) | 15:24 |
| edleafe | cdent: I'll take a look at that shortly | 15:24 |
| cdent | edleafe: no worries, no rush | 15:24 |
| cdent | thanks | 15:24 |
| *** baoli has joined #openstack-nova | 15:25 | |
| markus_z | sfinucan: Can I free that up or do you wanna keep that? https://bugs.launchpad.net/nova/+bug/1522454 | 15:26 |
| *** pwnall has quit IRC | 15:26 | |
| openstack | Launchpad bug 1522454 in OpenStack Compute (nova) "Nova is leaking libvirt internal ids on some Instance Not Found errors" [Medium,In progress] - Assigned to Stephen Finucane (stephenfinucane) | 15:26 |
| *** catintheroof has joined #openstack-nova | 15:27 | |
| *** shaohe_feng has quit IRC | 15:28 | |
| *** shaohe_feng has joined #openstack-nova | 15:29 | |
| *** sridharg has quit IRC | 15:31 | |
| *** eharney has joined #openstack-nova | 15:32 | |
| *** pwnall has joined #openstack-nova | 15:35 | |
| *** code-R has quit IRC | 15:36 | |
| *** shaohe_feng has quit IRC | 15:38 | |
| *** haplo37__ has joined #openstack-nova | 15:40 | |
| edleafe | cdent: ok, commented | 15:41 |
| cdent | thanks | 15:41 |
| *** dimtruck is now known as zz_dimtruck | 15:41 | |
| *** shaohe_feng has joined #openstack-nova | 15:41 | |
| BobBall | markus_z: I think that moving it out of tree would be fine medium term. I'm hoping to set up a os-xapi repo to share code common to multiple projects, and it might be most sensible to move this sort of tool to that repo too. | 15:42 |
| markus_z | tdurakov: That's a thing which won't be solved easily without a bigger refactoring, right? https://bugs.launchpad.net/nova/+bug/1487477 | 15:42 |
| openstack | Launchpad bug 1487477 in OpenStack Compute (nova) "Mess in live-migration compute-manager and drivers code" [Low,In progress] - Assigned to Timofey Durakov (tdurakov) | 15:42 |
| *** zz_dimtruck is now known as dimtruck | 15:42 | |
| BobBall | Nothing in the tool requires Nova code - it's just using utilities really | 15:42 |
| markus_z | BobBall: yeah, sounds reasonable. There's also the ops-repo for tooling, maybe it would fit there too | 15:42 |
| *** links has quit IRC | 15:43 | |
| markus_z | BobBall: https://github.com/openstack/osops-tools-generic | 15:43 |
| BobBall | Might be better; move the common code (e.g. session stuff) from nova and then the tool can go anywhere as long as it includes os-xenapi repo | 15:43 |
| BobBall | I mean "Yes, that might be a better idea" :) | 15:44 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Maintain backwards compat for listen opts https://review.openstack.org/353917 | 15:44 |
| markus_z | :) | 15:44 |
| *** pbandark has quit IRC | 15:46 | |
| markus_z | BobBall: not sure how the osops repo handle dependency mgmt (I don't find anything in there) | 15:46 |
| alaski | mriedem: mdbooth was distracted earlier, but my still open question on not writing 0 size bdms to the db is why do we even have the objects at that point? can we not filter those out earlier, or just not create them? | 15:47 |
| *** code-R has joined #openstack-nova | 15:47 | |
| sdague | mriedem: the mitaka nova client job seems to be working correctly - http://logs.openstack.org/14/347514/6/check/gate-novaclient-dsvm-functional-mitaka-nv/34d1922/logs/devstack-gate-setup-workspace-new.txt.gz | 15:47 |
| mriedem | alaski: not sure, would need to test it manually to see what happens | 15:48 |
| * mdbooth confesses to not having studied the lifecycle of a BDM much before they get to compute manager. | 15:48 | |
| *** shaohe_feng has quit IRC | 15:49 | |
| *** sdake_ has joined #openstack-nova | 15:49 | |
| alaski | okay. the mystery continues... | 15:49 |
| *** shaohe_feng has joined #openstack-nova | 15:49 | |
| *** mgoddard_ has quit IRC | 15:49 | |
| mdbooth | alaski: Although... it doesn't look like they should actually hit the db in that case? | 15:49 |
| *** mgoddard has joined #openstack-nova | 15:50 | |
| alaski | mdbooth: they don't | 15:50 |
| *** oanson has quit IRC | 15:50 | |
| alaski | that's why I'm wondering why they exist to begin with | 15:50 |
| alaski | but I haven't had time to dig in yet | 15:50 |
| dansmith | alaski: hmm, thinking about this, doesn't devstack need to do the simple_cell_setup even on a clean system? | 15:50 |
| *** sdake has quit IRC | 15:51 | |
| tdurakov | markus_z: I think you could close that, because it's more like bp rather then bug, tbh | 15:51 |
| dansmith | alaski: like mabe this? https://review.openstack.org/#/c/322311/3/lib/nova | 15:51 |
| dansmith | heh | 15:51 |
| alaski | dansmith: was just about to link that | 15:51 |
| alaski | yeah | 15:51 |
| mdbooth | alaski: Looks like they could be safely dropped in _bdm_validate_set_size_and_instance | 15:51 |
| markus_z | tdurakov: yeah, think so too. I'll close it. I liked the title though :) | 15:51 |
| mriedem | alaski: mdbooth: a guess, but the DriverEphemeralBlockDevice in nova.virt.block_device wraps the bdm object | 15:51 |
| mriedem | so i think the bdm object probably needs to exist for that reason | 15:52 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 15:52 |
| mdbooth | mriedem: But in the volume_size=0 case, the bdm won't exist | 15:52 |
| mriedem | not in the db | 15:52 |
| mdbooth | So the DBD won't exist either | 15:52 |
| mdbooth | Surely? | 15:52 |
| mriedem | not surely | 15:52 |
| mriedem | it's bdm | 15:52 |
| mriedem | remember | 15:52 |
| mdbooth | orly? | 15:52 |
| mriedem | don't calle me shirly :) | 15:52 |
| *** woodster_ has joined #openstack-nova | 15:53 | |
| tdurakov | Rodrigo_BR: sorry was at meeting | 15:53 |
| tdurakov | so going back to that strange things, | 15:54 |
| mdbooth | hehe | 15:54 |
| tdurakov | could you please share libvirt logs either, to check what's finally kills vm, there is nothing critical in openstack logs you've shared | 15:54 |
| openstackgerrit | Sarah Ulmer proposed openstack/python-novaclient: Added smaller flavors for novaclient functional tests to use https://review.openstack.org/354150 | 15:55 |
| *** burt has joined #openstack-nova | 15:55 | |
| mdbooth | Ah... I remember this weirdness | 15:55 |
| mdbooth | compute.API sends the fake block_device_mapping list to conductor, which doesn't use it, just re-grabs it from the db | 15:55 |
| mdbooth | I actually wrote a patch to remove that once, but it failed because... cells | 15:56 |
| mdbooth | I didn't investigate | 15:56 |
| mdbooth | But it appears that something cellsy actually uses the block_device_mapping passed from compute_api, which will contain the zero-size bdms | 15:56 |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 15:57 |
| *** jwcroppe has quit IRC | 15:57 | |
| mriedem | i think that's the update_or_create_at_top? | 15:57 |
| mdbooth | Non-cells refetches it from the DB in conductor build_instances | 15:57 |
| mdbooth | As zero-size bdms weren't written to the db in the first place, non-cells shouldn't propagate them | 15:57 |
| *** jwcroppe has joined #openstack-nova | 15:58 | |
| mriedem | how are they used in the compute then? | 15:59 |
| cdent | oh dear, placement api stack merge conflict, something beat it in the gate race: I'll fix it soon | 15:59 |
| *** jwcroppe_ has joined #openstack-nova | 15:59 | |
| *** shaohe_feng has quit IRC | 15:59 | |
| mriedem | driver.spawn calls blockinfo.get_disk_info which does all of this processing to get the mappings | 15:59 |
| mdbooth | mriedem: Right, but it does it on either the list it got via rpc, or the list it got from the db | 16:00 |
| *** shaohe_feng has joined #openstack-nova | 16:00 | |
| mdbooth | Because those are the only 2 possible sources | 16:00 |
| mdbooth | And neither of those should contain zero-size bdms | 16:00 |
| mdbooth | Because they both ultimately came from the db | 16:00 |
| *** markus_z has quit IRC | 16:00 | |
| mdbooth | Except possibly cells | 16:01 |
| mriedem | so i can boot an instance with a flavor with no ephemeral_gb set (0), and a block_device of blank->local and guest_format=ephemeral, and nothing will happen, but the api wont fail either? | 16:02 |
| *** Hieu has quit IRC | 16:03 | |
| *** jwcroppe has quit IRC | 16:03 | |
| mriedem | sorry i have to go back to how the api determines if it's an ephemeral bdm | 16:03 |
| tdurakov | dansmith: hi, want to discuss one thing I've noticed, during upgrade from kilo to liberty live-migration do some writes to Migration table, but most of them are left in pre-migrating state for live-migration, so afair we end up with several unfinished live-migration in db, could it be potential issue, what do you think? | 16:03 |
| *** Hieu has joined #openstack-nova | 16:03 | |
| mriedem | oh i think i see | 16:03 |
| *** zenoway has quit IRC | 16:03 | |
| *** raunak has joined #openstack-nova | 16:04 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1313-L1317 | 16:04 |
| dansmith | tdurakov: what issue do you think that will cause? | 16:04 |
| *** bnemec has quit IRC | 16:04 | |
| mriedem | although, | 16:04 |
| tdurakov | dansmith: we could potentially pick up wrong migration object for instance | 16:04 |
| johnthetubaguy | sdague: I wasn't sure about this big: https://review.openstack.org/#/c/352767/4 curious about your thoughts on that? we could fix that later in a follow up | 16:05 |
| mriedem | mdbooth: with https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1313-L1317 if i didn't specify bdm size, and my flavor doesn't have ephemeral_gb, but i requested a blank->local bdm with guest_format != 'swap', it looks like it won't fail | 16:05 |
| dansmith | tdurakov: where? | 16:05 |
| dansmith | alaski: you wanna jump on -qa for a sec? | 16:05 |
| sdague | johnthetubaguy: yeh, we could tweak the test, but it doesn't hurt things as is | 16:06 |
| dansmith | alaski: ah lunch, I remember.. so, after that | 16:07 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for xenapi_opts https://review.openstack.org/343797 | 16:07 |
| openstackgerrit | Michael Glaser proposed openstack/nova: Check opt consistency for neutron.py https://review.openstack.org/349605 | 16:07 |
| *** zenoway has joined #openstack-nova | 16:08 | |
| mdbooth | mriedem: So I guess that would only work if you also don't specify the bdm size? It will effectively be ignored. | 16:08 |
| *** jdurgin1 has quit IRC | 16:08 | |
| mdbooth | It would raise an exception if you specified any size other than zero. | 16:08 |
| mdbooth | I guess that's weird, but it's a pretty weird thing to be doing anyway. | 16:09 |
| *** shaohe_feng has quit IRC | 16:09 | |
| tdurakov | dansmith: trying to find out potential failure... the reason I think it could be important, we have methods like https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4745-L4784 | 16:09 |
| mriedem | mdbooth: sure | 16:10 |
| *** shaohe_feng has joined #openstack-nova | 16:10 | |
| mriedem | weird, but UX would be better if we failed in the api | 16:10 |
| mriedem | anyway, i'll try to test it out and see what happens | 16:10 |
| mdbooth | mriedem: IIUC it currently won't fail at all, but it won't do anything either, right? | 16:10 |
| mriedem | given the matrix of possibilities with bdms, it's nice to be explicit in the api validation | 16:10 |
| mriedem | mdbooth: right | 16:10 |
| mriedem | but as a user you might have expected to get an ephemeral disk | 16:10 |
| mriedem | and you didn't b/c you forgot something, or accidentally picked the wrong flavor | 16:11 |
| mriedem | idk | 16:11 |
| mriedem | usability | 16:11 |
| mdbooth | mriedem: Are we saying we should explicitly fail if the user specifies a bdm which ultimately won't be created? | 16:11 |
| mriedem | idk yet | 16:11 |
| mriedem | i want to find out if that's what happens first | 16:11 |
| mdbooth | Problem is, that would be an api change | 16:11 |
| *** seldenr has joined #openstack-nova | 16:11 | |
| tdurakov | dansmith: so, maybe I will not find any now, but once someone will decide to filter migrations from db that way, nova could start to fail | 16:11 |
| dansmith | tdurakov: still, I'd need to see where that actually caused a problem, but since kilo and liberty nodes don't know each others versions, I'm not sure what we can do about it | 16:11 |
| mdbooth | You just know that somebody, somewhere relies on this weirdness | 16:11 |
| mriedem | mdbooth: i don't see how you could | 16:12 |
| dansmith | tdurakov: newer code that looks for pending migrations is looking for non-live migrations, IIRC | 16:12 |
| *** bnemec has joined #openstack-nova | 16:12 | |
| mriedem | that would be, dumb? | 16:12 |
| dansmith | tdurakov: but if you have some way to guarantee that migrations are stale, you could write a periodic to clean them up I guess, but I don't think we should backport that | 16:12 |
| johnthetubaguy | sdague: did we want to add the policy for start? https://review.openstack.org/#/c/353335/3/nova/api/openstack/compute/servers.py | 16:12 |
| *** zenoway has quit IRC | 16:13 | |
| mdbooth | mriedem: Here's a use case, maybe: | 16:13 |
| mdbooth | User wants ephemeral disk, if the flavor has one, to be created with a particular guest_format | 16:13 |
| mdbooth | Therefore, they blanket set the format for all creations | 16:13 |
| mdbooth | Flavors which don't have an ephemeral disk don't get one | 16:13 |
| sdague | johnthetubaguy: oh, good point, I was refreshing those patches | 16:14 |
| sdague | let me remove it | 16:14 |
| mdbooth | Flavors which do have an ephemeral disk get the user-specified format | 16:14 |
| mdbooth | That's not completely insane | 16:14 |
| tdurakov | dansmith: what if we at least start ordering these queries by updated_at field? | 16:14 |
| tdurakov | dansmith: I will spend more time to figure out whether it's real issue or potential one | 16:14 |
| mriedem | mdbooth: yeah | 16:14 |
| dansmith | tdurakov: oka | 16:14 |
| dansmith | tdurakov: if you order by created or updated, and there is a completed one after an in-progress one, then the in-progress one is clearly stale | 16:15 |
| *** rcernin has quit IRC | 16:15 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Improve help text for compute manager options https://review.openstack.org/329639 | 16:15 |
| tdurakov | dansmith: yes, it's not that straightforward, that's why will spend more time on auditing that first. Just wanted to get some feedback on that | 16:16 |
| dansmith | tdurakov: cool | 16:16 |
| *** baoli has quit IRC | 16:17 | |
| *** baoli has joined #openstack-nova | 16:18 | |
| mriedem | sdague: dansmith: ha, i just found something out | 16:19 |
| mriedem | with this novaclient change and 2.36 | 16:19 |
| mriedem | oh it sucks to be us | 16:19 |
| mriedem | the cli looks up the image you pass in for boot | 16:19 |
| mriedem | guess which api it uses? | 16:19 |
| *** shaohe_feng has quit IRC | 16:19 | |
| mriedem | :) | 16:19 |
| *** shaohe_feng has joined #openstack-nova | 16:20 | |
| *** Apoorva has joined #openstack-nova | 16:20 | |
| mriedem | so i guess novaclient is going to have to require glanceclient | 16:22 |
| *** daemontool has quit IRC | 16:23 | |
| *** Swami has joined #openstack-nova | 16:23 | |
| *** matrohon has quit IRC | 16:23 | |
| sdague | mriedem: why does it look it up? | 16:23 |
| mriedem | to see if it exists i guess | 16:23 |
| mriedem | which the api also does | 16:23 |
| *** devkulkarni1 has quit IRC | 16:23 | |
| mriedem | server create i mean | 16:24 |
| sdague | right, so that seems wasteful | 16:24 |
| mriedem | but this is why the functional tests are also failing | 16:24 |
| sdague | ok, so that's good to know | 16:24 |
| *** mingyu has joined #openstack-nova | 16:24 | |
| sdague | but I think the fix is to not do that check, and pass it all the way in, right? | 16:24 |
| mriedem | i can probably change the functional tests to use glance via tempestlib | 16:24 |
| mriedem | idk, we have similar things for networks and volumes | 16:24 |
| mriedem | i.e. pass in network name or volume name and we look it up and pass in the id to the api | 16:25 |
| *** rmart04 has quit IRC | 16:25 | |
| mriedem | as a convenience | 16:25 |
| sdague | oh, right, name traslation | 16:25 |
| mriedem | so... | 16:25 |
| *** jaosorior has quit IRC | 16:25 | |
| mriedem | before we can claim to support 2.36 in the client, | 16:25 |
| mriedem | we probably have to first update the client to not use the stuff we just deprecated in 2.36 :( | 16:25 |
| mriedem | stepping away for a bit | 16:26 |
| openstackgerrit | Sean Dague proposed openstack/nova: Allow authorization by user_id for server pause action https://review.openstack.org/352221 | 16:26 |
| openstackgerrit | Sean Dague proposed openstack/nova: Add tests for user_id policy enforcement on stop https://review.openstack.org/353335 | 16:26 |
| sdague | johnthetubaguy: fixed | 16:26 |
| *** yamahata has joined #openstack-nova | 16:27 | |
| *** Hieu has quit IRC | 16:27 | |
| johnthetubaguy | sdague: I think we need to drop the drop in the unit tests, just spotted that | 16:28 |
| sdague | johnthetubaguy: actually we don't | 16:28 |
| sdague | because it's also in the next test class | 16:28 |
| sdague | I was confused about that | 16:28 |
| johnthetubaguy | ah... | 16:28 |
| sdague | that all the tests passed | 16:28 |
| johnthetubaguy | oh dear, maybe we are not actually testing what we think we are? | 16:29 |
| *** ijw has joined #openstack-nova | 16:29 | |
| sdague | johnthetubaguy: no, it's that gmann took the policy related tests and put them in a new clas | 16:29 |
| sdague | new class | 16:29 |
| *** shaohe_feng has quit IRC | 16:30 | |
| mriedem | sdague: in the interest of time, an alternative is we could change the default in the CLI to be 2.35, effectively capping the CLI, until we get novaclient itself off of the deprecated proxy APIs | 16:30 |
| mriedem | not sure how much people would hate that idea | 16:30 |
| *** shaohe_feng has joined #openstack-nova | 16:30 | |
| sdague | mriedem: that means get me a network (assuming we land it) has no CLI support in newton, right? | 16:31 |
| mriedem | you'd have to opt in, which defeats the purpose of making it easy | 16:31 |
| mriedem | but other stuff would break yeah | 16:31 |
| mriedem | so not really an option | 16:32 |
| *** nic has joined #openstack-nova | 16:32 | |
| sdague | I do agree this is a bigger issue though, because nova boot won't work after this point | 16:33 |
| johnthetubaguy | sdague: oh.... we used to pass the whole instance for start and stop as the target | 16:34 |
| *** dikonoor has joined #openstack-nova | 16:34 | |
| sdague | johnthetubaguy: oh... right, that's what gmann was saying | 16:34 |
| sdague | gdi, so I should probably revert revert? | 16:35 |
| sdague | so that you aren't writing policy based on server name on start | 16:35 |
| sdague | I need to get away from the computer for a bit and find food | 16:36 |
| johnthetubaguy | sdague: so I think it just gives us better coverage, so maybe lets stick with it, and fix up the other actions in here later? | 16:36 |
| *** takedakn has joined #openstack-nova | 16:37 | |
| *** Oku_OS is now known as Oku_OS-away | 16:37 | |
| *** gszasz has quit IRC | 16:40 | |
| *** shaohe_feng has quit IRC | 16:40 | |
| *** shaohe_feng has joined #openstack-nova | 16:41 | |
| *** takedakn has quit IRC | 16:41 | |
| *** TravT has quit IRC | 16:42 | |
| mriedem | so i'll start working on making novaclient use glanceclient as a start to move off the proxy apis and see how bad it is | 16:47 |
| *** tblakes_ has joined #openstack-nova | 16:47 | |
| *** tonytan4ever has quit IRC | 16:48 | |
| *** unicell has quit IRC | 16:49 | |
| *** _ducttape_ has joined #openstack-nova | 16:49 | |
| sdague | mriedem: we only need name lookup right? | 16:50 |
| sdague | like litterally only that translation element | 16:50 |
| *** shaohe_feng has quit IRC | 16:50 | |
| sdague | I would be more inclined to just put that one API call into our code | 16:50 |
| mriedem | for image i don't think so | 16:50 |
| *** lucasagomes is now known as lucas-dinner | 16:50 | |
| mriedem | i think the name lookup is for network for sure | 16:50 |
| mriedem | not sure about volume | 16:50 |
| mriedem | looking right now | 16:50 |
| sdague | --image definitely can be a name | 16:50 |
| *** shaohe_feng has joined #openstack-nova | 16:51 | |
| mriedem | ok i always use the id | 16:51 |
| *** kevinz has quit IRC | 16:51 | |
| *** tblakes has quit IRC | 16:51 | |
| *** fragatina has joined #openstack-nova | 16:51 | |
| *** tblakes_ is now known as tblakes | 16:52 | |
| mriedem | Name or ID of image (see 'glance image-list') | 16:52 |
| mriedem | yup | 16:52 |
| *** kevinz has joined #openstack-nova | 16:52 | |
| *** ducttape_ has quit IRC | 16:53 | |
| ihrachys | danpb: bummer, we still have the job broken, but now with http://logs.openstack.org/49/354149/1/check/gate-tempest-dsvm-neutron-linuxbridge/f8473e5/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-08-11_16_16_17_946 | 16:53 |
| ihrachys | danpb: should we maybe revert the original patch so that neutron gate is unblocked? | 16:54 |
| alaski | dansmith: back. should I hop over to -qa? | 16:54 |
| *** fragatin_ has quit IRC | 16:54 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/nova: Revert "libvirt: convert over to use os-vif for Linux Bridge & OVS" https://review.openstack.org/354221 | 16:55 |
| sdague | mriedem: anyway, about to actually go out and get my food, but given the way osc pulls us in, I'm wondering if us pulling in glance/cinder/neutron client is going to be exxtra derpy | 16:55 |
| *** yamahata has quit IRC | 16:55 | |
| ihrachys | danpb: ^ revert in case you think it's the right direction now | 16:55 |
| *** armax has quit IRC | 16:55 | |
| sdague | and just doing the one name => id lookup call inline for each is simpler | 16:55 |
| *** yamahata has joined #openstack-nova | 16:55 | |
| danpb | no, its quicker to just fix the actual bug | 16:55 |
| *** rama_y has joined #openstack-nova | 16:56 | |
| ihrachys | danpb: as you seem fit; I would still leave revert up for now just in case another attempt does not get us a better result | 16:56 |
| *** sdake_ has quit IRC | 16:57 | |
| *** armax has joined #openstack-nova | 16:58 | |
| *** bwallis has quit IRC | 16:58 | |
| *** armax has quit IRC | 16:59 | |
| *** anush_ has quit IRC | 17:00 | |
| *** shaohe_feng has quit IRC | 17:00 | |
| *** tblakes has quit IRC | 17:00 | |
| *** shaohe_feng has joined #openstack-nova | 17:01 | |
| *** rcernin has joined #openstack-nova | 17:02 | |
| *** cdent has quit IRC | 17:03 | |
| *** rcernin has quit IRC | 17:05 | |
| *** rcernin has joined #openstack-nova | 17:05 | |
| *** fragatina has quit IRC | 17:06 | |
| *** tblakes has joined #openstack-nova | 17:06 | |
| *** fragatina has joined #openstack-nova | 17:07 | |
| openstackgerrit | Daniel Berrange proposed openstack/nova: network: fix handling of linux-bridge in os-vif conversion https://review.openstack.org/354143 | 17:07 |
| mriedem | sdague: yeah looks like we just need the GET method, there are other CLIs using some other stuff but i think we just cap those CLIs at 2.35 | 17:07 |
| *** jichen has quit IRC | 17:08 | |
| *** tblakes_ has joined #openstack-nova | 17:10 | |
| *** shaohe_feng has quit IRC | 17:11 | |
| *** tblakes has quit IRC | 17:11 | |
| *** lpetrut has quit IRC | 17:11 | |
| *** tblakes_ is now known as tblakes | 17:11 | |
| *** shaohe_feng has joined #openstack-nova | 17:11 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add cell_v2 verify_instance command https://review.openstack.org/354261 | 17:12 |
| *** fragatina has quit IRC | 17:12 | |
| dansmith | alaski: sure | 17:12 |
| *** ftersin has joined #openstack-nova | 17:12 | |
| *** baoli has quit IRC | 17:13 | |
| *** baoli has joined #openstack-nova | 17:13 | |
| *** ftersin has left #openstack-nova | 17:14 | |
| *** thorst_ has quit IRC | 17:14 | |
| *** prateek_ has joined #openstack-nova | 17:15 | |
| *** zhangjn has quit IRC | 17:15 | |
| *** tongli has joined #openstack-nova | 17:16 | |
| *** tongli has quit IRC | 17:16 | |
| *** zhangjn has joined #openstack-nova | 17:17 | |
| *** prateek has quit IRC | 17:19 | |
| *** unicell has joined #openstack-nova | 17:19 | |
| *** shaohe_feng has quit IRC | 17:21 | |
| *** Hosam_ has quit IRC | 17:21 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Ironic Driver: override get_serial_console() https://review.openstack.org/328157 | 17:22 |
| *** shaohe_feng has joined #openstack-nova | 17:22 | |
| *** TravT has joined #openstack-nova | 17:23 | |
| *** thorst_ has joined #openstack-nova | 17:25 | |
| *** fragatina has joined #openstack-nova | 17:27 | |
| *** pwnall has quit IRC | 17:28 | |
| *** thorst_ has quit IRC | 17:29 | |
| *** _ducttape_ has quit IRC | 17:29 | |
| *** sdake has joined #openstack-nova | 17:29 | |
| mriedem | sdague: so i guess from novaclient.v2.client.Client we have to lookup the image endpoint in the service catalog and then do version discovery to find out which API to use (prefer 2 over 1)? | 17:30 |
| *** catintheroof has quit IRC | 17:30 | |
| *** nkrinner is now known as nkrinner_afk | 17:30 | |
| mriedem | i figured we'd do something in there since we already have the credentials at that point | 17:30 |
| *** shaohe_feng has quit IRC | 17:31 | |
| *** shaohe_feng has joined #openstack-nova | 17:32 | |
| *** thorst_ has joined #openstack-nova | 17:33 | |
| dansmith | alaski: for that None case, just saying it's "not mapped" is still valid, right? | 17:34 |
| dansmith | or do you think we should say something specific? | 17:34 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: config options: Improve help for SPICE https://review.openstack.org/317046 | 17:35 |
| dansmith | alaski: like, I don't say "error" or "oh gawd" right now, so I think "not mapped" is valid for both cases | 17:35 |
| alaski | dansmith: I think it'll be useful to distinguish between an instance in a cell with no mapping, and an instance mapped to None | 17:36 |
| dansmith | alaski: so what would you say for the two strings? | 17:36 |
| alaski | I think just saying mapped to None would be right | 17:36 |
| *** bwallis has joined #openstack-nova | 17:36 | |
| *** ijw has quit IRC | 17:36 | |
| alaski | "Instance is not mapped" "Instance mapped to cell None" | 17:36 |
| *** devkulkarni1 has joined #openstack-nova | 17:37 | |
| alaski | maybe "Instance has no mapping" for the first one | 17:37 |
| dansmith | but the default is a cell _called_ None | 17:37 |
| *** ijw has joined #openstack-nova | 17:37 | |
| dansmith | there are three cases: | 17:37 |
| dansmith | 1. Unmapped because upgrade is not done | 17:37 |
| dansmith | 2. Unmapped because the instance is not scheduled | 17:37 |
| dansmith | 2. Mapped | 17:37 |
| dansmith | er, 3. Mapped :) | 17:37 |
| dansmith | In the #3 case, we already say "cell None" if you used the simple command which does not name the cell right? | 17:38 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for glance options https://review.openstack.org/324007 | 17:38 |
| dansmith | Instance 02e6fa46-5853-4f14-9231-9533ce54e7d8 is in cell: None (c6c3521f-e8e3-4bb1-ab64-951086e61d95) | 17:38 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: network: fix handling of linux-bridge in os-vif conversion https://review.openstack.org/354143 | 17:38 |
| alaski | yeah | 17:38 |
| *** Guest94303 has quit IRC | 17:38 | |
| alaski | how about "instance mapped", "instance has no mapping", "instance mapped to no cell" | 17:39 |
| lbeliveau | mriedem: got +2 from jaypipes, added you as 2nd core reviewer since you have reviewed related patches https://review.openstack.org/#/c/347444 | 17:39 |
| dansmith | alaski: the last one is confusing to me | 17:39 |
| alaski | yeah, the default cell name of None was a poor choice it seems | 17:39 |
| dansmith | alaski: I think if we distinguish them then it should be pretty clear the difference and the last two sound pretty much the same | 17:39 |
| dansmith | yeah | 17:40 |
| dansmith | so how about: | 17:40 |
| dansmith | Instance has no mapping | 17:40 |
| dansmith | and something like | 17:40 |
| sdague | mriedem: you were right about the 2.35 issue - http://logs.openstack.org/50/354150/2/check/gate-novaclient-dsvm-functional-mitaka-nv/e075530/console.html#_2016-08-11_17_09_07_240232 | 17:40 |
| dansmith | Instance not yet scheduled | 17:40 |
| *** dikonoor has quit IRC | 17:40 | |
| dansmith | or something to indicate why a lack of mapping is different from no mapping | 17:40 |
| dansmith | or whatever | 17:40 |
| *** shaohe_feng has quit IRC | 17:41 | |
| *** ijw has quit IRC | 17:41 | |
| alaski | that could be weird if the instance is scheduled and a messed up mapping says that it's not, but I like the distinctiveness | 17:41 |
| dansmith | you mean scheduled but unupgraded? | 17:42 |
| *** shaohe_feng has joined #openstack-nova | 17:42 | |
| dansmith | because we can distinguish between those cases | 17:42 |
| alaski | I mean if there's a bug later | 17:43 |
| dansmith | o | 17:43 |
| sdague | mriedem: right, this is where we enter the world of suck, given that catalog types aren't standardized (we even force configuration on our side) | 17:43 |
| alaski | I'm convincing myself it's fine though | 17:43 |
| sdague | mriedem: so, I think what we probably do is the following | 17:43 |
| openstackgerrit | Julian proposed openstack/nova: Add unit tests for nova.virt.firewall.IpTablesFirewallDriver (Part 3) https://review.openstack.org/354277 | 17:43 |
| alaski | if we position verify_instance as reporting the state of the instance according to the mappings | 17:43 |
| alaski | if that differs from reality there's a bug | 17:43 |
| *** tri2sing has joined #openstack-nova | 17:44 | |
| dansmith | alaski: we could also do this: | 17:44 |
| dansmith | 1. Instance has no mapping (upgrade is incomplete) | 17:44 |
| dansmith | 2. Instance is unmapped | 17:44 |
| sdague | assume things about the catalog and names, only support one api version per service, if that's not there, just return back "name lookup not available because X not in catalog" and error | 17:44 |
| dansmith | 3. Instance is in cell foo | 17:44 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Improve consistency in WSGI opts https://review.openstack.org/348417 | 17:44 |
| *** gjayavelu has joined #openstack-nova | 17:44 | |
| sdague | because, otherwise, we're litterally just duplicating OSC | 17:44 |
| dansmith | ignoring the scheduling part | 17:44 |
| sdague | which, we were trying to avoid | 17:45 |
| *** parora has joined #openstack-nova | 17:45 | |
| alaski | dansmith: that makes sense to me | 17:46 |
| dansmith | okay | 17:46 |
| mriedem | sdague: yeah i figured we'd use v1 | 17:47 |
| mriedem | since that should be available everywhere | 17:47 |
| sdague | mriedem: except rax | 17:48 |
| *** prateek_ has quit IRC | 17:48 | |
| *** cdent has joined #openstack-nova | 17:48 | |
| mriedem | what's their problem? | 17:48 |
| sdague | we're forcing you to have v2 and dropping v1 support in Ocata | 17:48 |
| sdague | I assume we'd do v2 name lookups for glance | 17:48 |
| *** tonytan4ever has joined #openstack-nova | 17:49 | |
| mriedem | true | 17:49 |
| *** lpetrut has joined #openstack-nova | 17:50 | |
| *** haplo37__ has quit IRC | 17:50 | |
| *** haplo37__ has joined #openstack-nova | 17:50 | |
| *** ijw has joined #openstack-nova | 17:51 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add cell_v2 verify_instance command https://review.openstack.org/354261 | 17:52 |
| *** shaohe_feng has quit IRC | 17:52 | |
| mriedem | so, just service='image' and v2 client | 17:52 |
| mriedem | if that's not available, oops | 17:52 |
| *** ijw has quit IRC | 17:52 | |
| *** haplo37- has joined #openstack-nova | 17:52 | |
| *** shaohe_feng has joined #openstack-nova | 17:52 | |
| *** ijw has joined #openstack-nova | 17:52 | |
| *** prateek_ has joined #openstack-nova | 17:53 | |
| *** tonytan4ever has quit IRC | 17:54 | |
| mriedem | redoing the auth stuff in here is going to kind of suck | 17:54 |
| *** ijw has quit IRC | 17:54 | |
| *** ijw has joined #openstack-nova | 17:55 | |
| sdague | can't we use the session we've got? | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for inventories to placement API https://review.openstack.org/329152 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: add filtering by attrs to resource_providers https://review.openstack.org/343046 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a Usage and UsageList object https://review.openstack.org/329575 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for resource_providers urls https://review.openstack.org/329151 | 17:56 |
| *** parora has quit IRC | 17:56 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for usages in the placement API https://review.openstack.org/329643 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Improve 404 and 405 response tests https://review.openstack.org/331483 | 17:56 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add placement API web utility methods https://review.openstack.org/352573 | 17:56 |
| mriedem | sdague: yeah maybe | 17:57 |
| *** fragatina has quit IRC | 17:57 | |
| cdent | sdague, jaypipes: that ^ clears up the merge conflicts happenings from various placement stuff merging. some of that stack already had +2s on it | 17:57 |
| *** ametts has joined #openstack-nova | 17:57 | |
| cdent | bottom is https://review.openstack.org/352573 | 17:57 |
| *** woodster_ has quit IRC | 17:59 | |
| tdurakov | mriedem: how much time do you think we need to before making live-migration job voting? | 17:59 |
| *** oomichi has joined #openstack-nova | 17:59 | |
| mriedem | tdurakov: probably a week | 17:59 |
| mriedem | to see how it stabilizes | 17:59 |
| tdurakov | mriedem: ok, will get back to that next week, thank you | 18:00 |
| danpb | ihrachys: jenkins seems to be unhappy with your test commit complaining about merge failure | 18:01 |
| sdague | mriedem: ok, do you want help on the novaclient work, or would you rather I go review get-me-a-network? | 18:02 |
| *** shaohe_feng has quit IRC | 18:02 | |
| ihrachys | danpb: have you uploaded a new nova patch? because then we need a recheck after that | 18:02 |
| mriedem | sdague: i'm gonna need help with the novaclient stuff | 18:02 |
| ihrachys | danpb: zuul does not respin the jobs automatically | 18:02 |
| danpb | ihrachys: mriedem did a recheck | 18:02 |
| danpb | but jenkins just came back with "This change depends on a change that failed to merge." | 18:02 |
| *** shaohe_feng has joined #openstack-nova | 18:02 | |
| ihrachys | hm | 18:02 |
| sdague | mriedem: ok, is there any partial code up? | 18:03 |
| *** ducttape_ has joined #openstack-nova | 18:03 | |
| mriedem | sdague: not for using glance in novaclient yet no | 18:03 |
| mriedem | danpb: ihrachys: i just rechecked again | 18:03 |
| mriedem | was probably a hiccup | 18:03 |
| mriedem | b/c danpb pushed a new ps | 18:03 |
| sdague | mriedem: ok, and the theory is that we'll do this before the 2.35 max change, right? | 18:04 |
| mriedem | yes | 18:04 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add ObjectListBase concat methods https://review.openstack.org/352587 | 18:04 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Fix migration list + MigrationList operation https://review.openstack.org/353694 | 18:04 |
| *** pwnall has joined #openstack-nova | 18:04 | |
| danpb | mriedem: why does jenkins complain about failed merge though - its a completely sepaarate git repo | 18:04 |
| mriedem | idk, it's a zuul thing | 18:05 |
| danpb | oh yeah, zuul really ought to call itself zuul instead of jenkins when commenting ;-P | 18:06 |
| danpb | 123456 | 18:06 |
| danpb | oops, you know my super secure test vm pasword now ;-P | 18:06 |
| mriedem | ha | 18:07 |
| *** lpetrut has quit IRC | 18:11 | |
| *** shaohe_feng has quit IRC | 18:12 | |
| *** lpetrut has joined #openstack-nova | 18:13 | |
| *** shaohe_feng has joined #openstack-nova | 18:13 | |
| *** amotoki has quit IRC | 18:15 | |
| *** ijw has quit IRC | 18:16 | |
| *** lpetrut has quit IRC | 18:17 | |
| sdague | mriedem: ok... this find_resource path is funky as all hell - https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/utils.py#L277-L333 | 18:19 |
| mriedem | yup... | 18:20 |
| mriedem | i think essentially we (1) do a get with the id and if that gives a 404, do a (2) get to list images and try to match on name | 18:20 |
| *** mvk has quit IRC | 18:20 | |
| mriedem | i thought we'd try to hide the glanceclient stuff inside novaclient.v2.client.Client somehow but not really sure what's the best way to do it | 18:21 |
| sdague | so... I think we need to figure out how to turn our session to get to glance | 18:21 |
| sdague | I wonder if mordred is about | 18:21 |
| *** amotoki has joined #openstack-nova | 18:22 | |
| sdague | at least the shell goes through this - https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/v2/shell.py#L2254-L2256 | 18:22 |
| *** shaohe_feng has quit IRC | 18:22 | |
| sdague | which means we can just inline that there, get out of the hyper generic find_resource which has more branches than users | 18:23 |
| mriedem | yeah, so i figured we could change that to be like a cs.find_image method | 18:23 |
| *** shaohe_feng has joined #openstack-nova | 18:23 | |
| mriedem | and hide the glance internals inside that cs object (the v2 Client class) | 18:23 |
| mriedem | the problem i was having is cs.client is either a keystoneauth session or an httpclient | 18:23 |
| *** ayoung has quit IRC | 18:24 | |
| mriedem | i thought we'd need to construct a cs.glanceclient that's like cs.client but different endpoint/service/version | 18:24 |
| sdague | why is that? | 18:24 |
| mriedem | if we can just re-use the token/session though then that would be better | 18:24 |
| mriedem | because i'm a noob when it comes to swizzling this | 18:25 |
| mriedem | :P | 18:25 |
| *** ihrachys has quit IRC | 18:25 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Fix wrong _is_booted_from_volume usages https://review.openstack.org/334609 | 18:26 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Remove duplicated logic in check_can_live_migrate_source https://review.openstack.org/335542 | 18:26 |
| openstackgerrit | Feodor Tersin proposed openstack/nova: libvirt: Simplify internal usage of get_instance_disk_info https://review.openstack.org/334901 | 18:26 |
| sdague | mriedem: dammit mordred - https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/client.py#L716-L717 :P | 18:28 |
| mriedem | yeah i saw that earlier | 18:30 |
| mriedem | so i got stuck with, do i need to know the type of cs.client | 18:30 |
| mriedem | to know how to re-use it | 18:30 |
| mriedem | which sucks | 18:30 |
| mriedem | so i figured i'd just call cs.glanceclient = _construct_http_client but with glance values | 18:31 |
| mriedem | for service_type | 18:31 |
| mriedem | one problem with that is the version in httpclient is hard-coded | 18:31 |
| mriedem | https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/client.py#L188 | 18:31 |
| *** eharney has quit IRC | 18:32 | |
| *** shaohe_feng has quit IRC | 18:33 | |
| mriedem | i guess that's reset here https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/client.py#L577 | 18:33 |
| mriedem | is that keystone api version? | 18:33 |
| *** shaohe_feng has joined #openstack-nova | 18:33 | |
| *** catintheroof has joined #openstack-nova | 18:34 | |
| *** jamesdenton has quit IRC | 18:35 | |
| *** fragatina has joined #openstack-nova | 18:35 | |
| sdague | yeh | 18:36 |
| mriedem | heh, ok, pretty damn old | 18:36 |
| *** devkulkarni1 has quit IRC | 18:37 | |
| sdague | so... you know what, we had this trick in here before | 18:38 |
| mriedem | for switching to the volume service | 18:38 |
| sdague | yeh | 18:38 |
| sdague | which is why volume_service_name is still being passed around | 18:38 |
| sdague | even though we deleted some of it | 18:39 |
| mriedem | yup, that was removed in https://review.openstack.org/#/c/300636/ | 18:39 |
| mriedem | https://review.openstack.org/#/c/300636/6/novaclient/base.py | 18:39 |
| mriedem | used in here https://review.openstack.org/#/c/300636/6/novaclient/v2/volume_types.py | 18:40 |
| mriedem | self.api there is self.cs from the shell | 18:42 |
| mriedem | so novaclient.v2.client.Client | 18:42 |
| sdague | with self.alternate_service_type( | 18:42 |
| mriedem | so self.api.client mutates the service_type before making the request | 18:42 |
| mriedem | yup | 18:42 |
| mriedem | and then resets it back | 18:43 |
| *** shaohe_feng has quit IRC | 18:43 | |
| *** gyee has joined #openstack-nova | 18:43 | |
| *** devkulkarni1 has joined #openstack-nova | 18:43 | |
| *** shaohe_feng has joined #openstack-nova | 18:43 | |
| *** ducttape_ has quit IRC | 18:44 | |
| *** ducttape_ has joined #openstack-nova | 18:44 | |
| sdague | ok, so if we bring that back, might work? | 18:45 |
| *** sudipto has joined #openstack-nova | 18:45 | |
| mriedem | but then do we need something like volume_service_name but for image_service_name in the service_catalog module? | 18:45 |
| *** sudipto_ has joined #openstack-nova | 18:45 | |
| sdague | yeh | 18:45 |
| sdague | I think so | 18:45 |
| sdague | we also need to undeprecate that bit on the cli | 18:46 |
| mriedem | which bit? | 18:46 |
| mriedem | i don't think anything is currently using volume_service_name | 18:46 |
| *** oanson has joined #openstack-nova | 18:47 | |
| *** eharney has joined #openstack-nova | 18:47 | |
| sdague | https://github.com/openstack/python-novaclient/blob/4d971af927256e7e32dcfb5a43d5c4487f9c4af9/novaclient/shell.py#L475-L482 | 18:47 |
| mriedem | yeah, but it's not used | 18:47 |
| sdague | I thought you said volumes were going to have the same name lookup issue | 18:47 |
| sdague | it's not used now | 18:48 |
| mriedem | let me check | 18:48 |
| sdague | but isn't it going to bork in a similar way | 18:48 |
| mriedem | nova volume-attach requires a volume id | 18:48 |
| mriedem | it doesn't do name translation | 18:48 |
| sdague | do we have a network name issue? | 18:49 |
| mriedem | we already removed the volume api code in novaclient, which would have been used to do a find/get if we needed it | 18:49 |
| mriedem | yeah | 18:49 |
| mriedem | if k in nic_info: | 18:49 |
| mriedem | # if user has given a net-name resolve it to network ID | 18:49 |
| mriedem | if k == 'net-name': | 18:49 |
| mriedem | k = 'net-id' | 18:49 |
| mriedem | v = _find_network_id(cs, v) | 18:49 |
| mriedem | wha wha | 18:49 |
| sdague | ok, well, I can start trying to hack this up on my devstack and see how terribad it gets | 18:49 |
| mordred | sdague, mriedem: on the phone- back in 10 minutes and can helpk if needed | 18:50 |
| *** shaohe_feng has quit IRC | 18:53 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Fix consistency in API conf https://review.openstack.org/354336 | 18:53 |
| *** burt has quit IRC | 18:54 | |
| *** shaohe_feng has joined #openstack-nova | 18:54 | |
| *** tblakes has quit IRC | 18:54 | |
| *** ijw has joined #openstack-nova | 18:55 | |
| *** devkulkarni1 has quit IRC | 18:58 | |
| *** rlrossit_ has quit IRC | 18:58 | |
| *** devkulkarni1 has joined #openstack-nova | 18:58 | |
| openstackgerrit | Andrey Pavlov proposed openstack/nova: revert quota's behaviour https://review.openstack.org/354340 | 18:58 |
| *** fragatin_ has joined #openstack-nova | 19:00 | |
| mordred | sdague, mriedem : okie - back - is the "dammit mordred" that I hadn't done that todo yet? | 19:02 |
| *** pwnall has quit IRC | 19:02 | |
| sdague | mordred: yeh | 19:02 |
| sdague | because it makes our stuff way more complicated | 19:02 |
| *** fragatina has quit IRC | 19:03 | |
| sdague | and trying to extract ourselves from the proxy game is going to be tricky when we stop having our own image proxy to use for name => id lookup | 19:03 |
| *** shaohe_feng has quit IRC | 19:03 | |
| mordred | nod | 19:03 |
| mordred | well, doing that todo actually shouldn't be terribly hard | 19:04 |
| mordred | you want me to take a quick stab at it? | 19:04 |
| *** shaohe_feng has joined #openstack-nova | 19:04 | |
| *** mvk has joined #openstack-nova | 19:04 | |
| sdague | that would be helpful | 19:04 |
| mordred | k. working on it now | 19:04 |
| *** fifieldt has quit IRC | 19:06 | |
| *** pwnall has joined #openstack-nova | 19:11 | |
| *** pwnall has quit IRC | 19:11 | |
| *** pwnall has joined #openstack-nova | 19:11 | |
| mordred | sdague: proxy_token and proxy_tenant_id ... | 19:11 |
| mordred | sdague: I'm not 100% sure what to do with them when constructing a session | 19:12 |
| *** pwnall1 has joined #openstack-nova | 19:12 | |
| mriedem | mordred: well, those predate gerrit | 19:13 |
| mriedem | https://github.com/openstack/python-novaclient/commit/2c3a865f6b408d85aaeaafafd9ff9cdcee5d8cb4 | 19:13 |
| mriedem | mordred: so i'd say f'em | 19:13 |
| *** shaohe_feng has quit IRC | 19:14 | |
| mordred | mriedem: awesome! | 19:14 |
| *** pwnall1 has quit IRC | 19:14 | |
| *** pwnall has quit IRC | 19:14 | |
| *** shaohe_feng has joined #openstack-nova | 19:14 | |
| mriedem | https://review.openstack.org/#/c/5018/2/novaclient/client.py | 19:15 |
| mriedem | does the keystone tokens api allow a belongsTo param? | 19:15 |
| mriedem | i'll be damned http://git.openstack.org/cgit/openstack/keystone/tree/keystone/token/controllers.py#n432 | 19:16 |
| mriedem | that's keystone v2 only though, which is deprecated | 19:16 |
| mriedem | so i'd say we just don't support proxy token for sessions | 19:17 |
| sdague | mriedem: ok, I got a thing that totally works | 19:18 |
| sdague | and it hackoliphic | 19:18 |
| *** fifieldt has joined #openstack-nova | 19:18 | |
| mriedem | cool | 19:18 |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Fix consistency in crypto conf https://review.openstack.org/354348 | 19:19 |
| sdague | it totally breaks if you pass it image... btw | 19:20 |
| sdague | sorry, an image id | 19:20 |
| sdague | but, at least it's a concept patch | 19:20 |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient: WIP: look up image names directly in glance https://review.openstack.org/354349 | 19:21 |
| sdague | check out that hackery | 19:21 |
| *** Sukhdev has joined #openstack-nova | 19:22 | |
| *** eharney has quit IRC | 19:23 | |
| openstackgerrit | Danil Akhmetov proposed openstack/nova: Handle user-defined ephemeral disk size if provided https://review.openstack.org/352522 | 19:23 |
| *** shaohe_feng has quit IRC | 19:24 | |
| *** shaohe_feng has joined #openstack-nova | 19:25 | |
| *** tblakes has joined #openstack-nova | 19:25 | |
| *** mingyu has quit IRC | 19:25 | |
| *** kragtest has joined #openstack-nova | 19:25 | |
| mriedem | sdague: not bad | 19:26 |
| mriedem | is the image id always a uuid? | 19:26 |
| *** danpb has quit IRC | 19:26 | |
| mriedem | if so we could just hack that check too | 19:26 |
| sdague | yes | 19:26 |
| sdague | yeh, that was my thought | 19:26 |
| *** sudipto has quit IRC | 19:27 | |
| *** sudipto_ has quit IRC | 19:27 | |
| sdague | I think the only thing is this builds an image object | 19:27 |
| sdague | which... I'm mostly sure we only ever care about the id field | 19:27 |
| sdague | I did notice an interesting thing when I nova booted with this | 19:27 |
| sdague | | image | Image not found (4275e93c-9d2a-41a8-97cd-3ba38ad0b39f) | | 19:28 |
| sdague | so, we're doing another lookup there somewhere | 19:28 |
| mriedem | yeah that's in _print_server | 19:28 |
| *** catintheroof has quit IRC | 19:28 | |
| mriedem | it looks up by id | 19:29 |
| mriedem | so once yo ufix the lookup by id that will be fixed | 19:29 |
| sdague | GET /v2/images?name=4275e93c-9d2a-41a8-97cd-3ba38ad0b39f | 19:30 |
| sdague | right, we'll just still have to lookup by id as well | 19:30 |
| sdague | also, this is why things are freaking slow :) | 19:30 |
| mriedem | doing something for network lookups is going to be trickier, | 19:31 |
| mriedem | since we might not be able to use neutronclient | 19:31 |
| sdague | do we have uuid utils in here somewhere? | 19:31 |
| mriedem | utils.find_resource has something in there for messing with uuids | 19:32 |
| mriedem | sdague: just use this https://github.com/openstack/oslo.utils/blob/master/oslo_utils/uuidutils.py#L41 | 19:33 |
| sdague | yeh, it's pretty gorpy | 19:33 |
| sdague | ok, we already require oslo.utils | 19:34 |
| sdague | so I can use it from there | 19:34 |
| *** shaohe_feng has quit IRC | 19:34 | |
| *** shaohe_feng has joined #openstack-nova | 19:35 | |
| *** tonytan4ever has joined #openstack-nova | 19:35 | |
| *** kragtest has quit IRC | 19:36 | |
| *** eharney has joined #openstack-nova | 19:36 | |
| openstackgerrit | Danil Akhmetov proposed openstack/nova: Handle user-defined ephemeral disk size if provided https://review.openstack.org/352522 | 19:36 |
| *** takedakn has joined #openstack-nova | 19:37 | |
| *** ayoung has joined #openstack-nova | 19:40 | |
| *** tonytan4ever has quit IRC | 19:40 | |
| *** prateek_ has quit IRC | 19:40 | |
| *** ijw has quit IRC | 19:41 | |
| *** takedakn has quit IRC | 19:42 | |
| *** bvanhav has quit IRC | 19:42 | |
| sdague | mriedem: that second lookup is going through a different path | 19:44 |
| openstackgerrit | Monty Taylor proposed openstack/python-novaclient: Change construct_http_client to always use Session https://review.openstack.org/354354 | 19:44 |
| mordred | sdague: that's likely still early - but I wanted to toss it up for sake of conversation | 19:44 |
| *** shaohe_feng has quit IRC | 19:44 | |
| mriedem | sdague: how so? | 19:45 |
| *** shaohe_feng has joined #openstack-nova | 19:45 | |
| openstackgerrit | Monty Taylor proposed openstack/python-novaclient: Change construct_http_client to always use Session https://review.openstack.org/354354 | 19:47 |
| mriedem | sdague: or is it bombing out on the .name getattr? _find_image(cs, image_id).name | 19:47 |
| sdague | mriedem: it's not calling _find_image again | 19:49 |
| mordred | sdague: fwiw, over in shade land, we just do a list and then check the if name_or_id = image.name or name_or_id = image.id for the places where we have no idea if the thing in question is a name or an id | 19:49 |
| mordred | neutron is a little easier since their rest api actually will take names or ids interchangably | 19:50 |
| mordred | but that might be a bit expensive for novaclient | 19:50 |
| sdague | mriedem: oh, nope, I guess it did come back down here | 19:52 |
| *** tonytan4ever has joined #openstack-nova | 19:52 | |
| *** shaohe_feng has quit IRC | 19:55 | |
| openstackgerrit | Sean Dague proposed openstack/python-novaclient: WIP: look up image names directly in glance https://review.openstack.org/354349 | 19:55 |
| sdague | mriedem: ok, that seems to work for reals | 19:55 |
| *** lpetrut has joined #openstack-nova | 19:55 | |
| sdague | I did have to hack base _get because glance doesn't follow the convention that nova has in which every resource has a wrapper | 19:55 |
| *** shaohe_feng has joined #openstack-nova | 19:56 | |
| sdague | mordred: ok, except I'm not sure that nova accepts names in some of those places | 19:57 |
| sdague | the issue is really novaclient cli | 19:57 |
| sdague | --image can be a name | 19:57 |
| *** pwnall has joined #openstack-nova | 19:57 | |
| sdague | which we translated before handing to the nova boot api | 19:57 |
| sdague | because we were trying to be nice | 19:57 |
| mordred | totally - I think it's the right thing to do | 19:58 |
| sdague | yeh | 19:58 |
| mordred | turns out uuids are not user friendly | 19:58 |
| sdague | except... when we removed our image proxy | 19:58 |
| *** pwnall1 has joined #openstack-nova | 19:58 | |
| mordred | right. now you have to query glance for a list | 19:58 |
| sdague | right, so I seem to have that | 19:59 |
| sdague | because, yay, service catalog pivot magic | 19:59 |
| mordred | yup. I like your patch | 19:59 |
| sdague | but... it's going to be a bit wonky if novaclient actually has to talk to every other service | 19:59 |
| *** catintheroof has joined #openstack-nova | 20:00 | |
| mordred | fwiw, I'm personally fine with a requirement that image parameters to novaclient are id and not name - it will not break any of my code | 20:01 |
| *** jamesdenton has joined #openstack-nova | 20:01 | |
| *** pwnall has quit IRC | 20:01 | |
| sdague | mordred: well, if you talk to the API part of the client, you have to do this | 20:02 |
| sdague | it's only translation for the shell | 20:02 |
| sdague | https://review.openstack.org/#/c/354349/2/novaclient/v2/shell.py | 20:02 |
| sdague | that's the only path in for translation | 20:02 |
| sdague | this is all just UX on the cli | 20:02 |
| *** jamesden_ has joined #openstack-nova | 20:02 | |
| mordred | nod | 20:03 |
| mordred | sdague: oh - andreykurilin__ already has patches up: https://review.openstack.org/#/c/304035/17 | 20:03 |
| mordred | I'm going to abandon mine - although I do think he should use plugin loader instead of the auth classes directly | 20:04 |
| mordred | but I'll leave that as sa review | 20:04 |
| *** shaohe_feng has quit IRC | 20:05 | |
| sdague | mordred: cool | 20:05 |
| andreykurilin__ | mordred: one of previous comments to my patch was about plugin loader. It was something like "plugin loader was design to be used from cli" | 20:05 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for inventories to placement API https://review.openstack.org/329152 | 20:05 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: add filtering by attrs to resource_providers https://review.openstack.org/343046 | 20:05 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a Usage and UsageList object https://review.openstack.org/329575 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for resource_providers urls https://review.openstack.org/329151 | 20:06 |
| *** shaohe_feng has joined #openstack-nova | 20:06 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for usages in the placement API https://review.openstack.org/329643 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 20:06 |
| openstackgerrit | Chris Dent proposed openstack/nova: Improve 404 and 405 response tests https://review.openstack.org/331483 | 20:06 |
| mordred | andreykurilin__: oh? I'll go read | 20:06 |
| *** jamesdenton has quit IRC | 20:06 | |
| mordred | andreykurilin__: I use plugin loader from os-client-config and shade becuase I was told it was 'the right way to do it' | 20:06 |
| mordred | :) | 20:06 |
| andreykurilin__ | mordred: from my experience- plugin loader take some time to discover needed plugin and direct import is quicker | 20:07 |
| mordred | andreykurilin__: ah - jamielennox|away said it's ok | 20:08 |
| andreykurilin__ | :) | 20:08 |
| mriedem | sdague: overall it looks good, i think we have to handle NoUniqueMatch differently though so we get a CommandError back, | 20:08 |
| *** jamesden_ is now known as jamesdenton | 20:08 | |
| mriedem | might need to handle that in novaclient.v2.shell._find_image | 20:08 |
| mriedem | like how utils.find_resource would do it | 20:09 |
| mordred | andreykurilin__: I changed to +1 :) | 20:09 |
| *** ijw has joined #openstack-nova | 20:09 | |
| andreykurilin__ | Heh | 20:09 |
| *** ijw has quit IRC | 20:09 | |
| andreykurilin__ | Thanks | 20:09 |
| sdague | mriedem: sure. I think my brain is done for the day. So if you want to leave review comments I'll look in the morning. | 20:09 |
| *** tyrola has joined #openstack-nova | 20:09 | |
| *** tyrola has quit IRC | 20:09 | |
| sdague | or if you want to run with the patch otherwise | 20:10 |
| mriedem | sdague: you want me to just hack on it? we're going to have to also stop doing the proxy for functional tests, and i think we can use tempest for that | 20:10 |
| mordred | andreykurilin__: your patches are much better than my crappy patch :) | 20:10 |
| *** ijw has joined #openstack-nova | 20:10 | |
| sdague | mriedem: yeh, have at it | 20:10 |
| mriedem | ok | 20:10 |
| andreykurilin__ | mordred: I had enough time for them:) | 20:10 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add cell_v2 verify_instance command https://review.openstack.org/354261 | 20:10 |
| dansmith | god I'm an idoot | 20:11 |
| mordred | dansmith: clearly, since you can't even spell idiot | 20:11 |
| dansmith | mordred: would you believe I seriously envisioned you (specifically) saying that exact thing? | 20:12 |
| dansmith | like, to the letter | 20:12 |
| mordred | hahahaha | 20:12 |
| andreykurilin__ | mriedem sdague: I had several patches in novaclient related to Keystone . All of them have +2 and bunch of +1 . I will really appreciated if you review them(and put good marks) | 20:12 |
| andreykurilin__ | *I have | 20:13 |
| openstackgerrit | sean mooney proposed openstack/nova: convert libvirt driver to use os-vif for vhost-user with ovs. https://review.openstack.org/334048 | 20:14 |
| *** fragatin_ has quit IRC | 20:15 | |
| *** shaohe_feng has quit IRC | 20:15 | |
| *** fragatina has joined #openstack-nova | 20:16 | |
| *** shaohe_feng has joined #openstack-nova | 20:16 | |
| dansmith | alaski: I have now arranged to actually *call* my new command, so the next run should be extra gooder | 20:18 |
| *** amotoki has quit IRC | 20:20 | |
| *** Apoorva has quit IRC | 20:20 | |
| mriedem | ah crap | 20:21 |
| mriedem | novaclient functional assumes nova-network backed devstack | 20:21 |
| *** oanson has quit IRC | 20:21 | |
| dansmith | mriedem: so far, | 20:21 |
| *** ijw has quit IRC | 20:21 | |
| dansmith | this deprecation thing is going swimmingly | 20:21 |
| mriedem | it's the gd worst thing ever | 20:22 |
| dansmith | heh | 20:22 |
| mriedem | let's just revert the 2.36 microversion | 20:22 |
| mriedem | clearly, CLEARLY, we didn't think this through | 20:22 |
| *** ijw has joined #openstack-nova | 20:22 | |
| dansmith | I blame sdague | 20:22 |
| mriedem | i'm going to blame jaypipes and vishy and mordred for splitting nova apart like 4+ years ago | 20:22 |
| mriedem | we wouldn't have proxies if there were no other services to proxy o | 20:23 |
| mriedem | *to | 20:23 |
| dansmith | fair point | 20:23 |
| dansmith | and we'd have a unified scheduler | 20:23 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Add ObjectListBase concat methods https://review.openstack.org/352587 | 20:23 |
| mriedem | ha that's right | 20:23 |
| dansmith | and live upgrades for those components | 20:23 |
| *** ijw has quit IRC | 20:23 | |
| mordred | I have no part in having split anything from anything | 20:23 |
| *** ijw has joined #openstack-nova | 20:23 | |
| dansmith | mordred: you were near them when they said it | 20:23 |
| mordred | the only thing you can blame me for is nova and swift not having been combined into a single repo when we first started | 20:24 |
| mordred | the pitch from on high was "make a repo called openstack, put nova into a subdir called "compute" and swift into one called "storage"" | 20:24 |
| mordred | and jay and I did argue that that would be a bad move | 20:24 |
| mordred | which may or may not have been smart | 20:25 |
| dansmith | I'm pretty sure I can blame you for whatever I want | 20:25 |
| dansmith | like, the heat wave passing through portland this weekend | 20:25 |
| sdague | mriedem: the only real difference for neutron back end is we need to pass a net name right? | 20:25 |
| *** shaohe_feng has quit IRC | 20:25 | |
| mordred | dansmith: that's a good point | 20:25 |
| mordred | dansmith: I'm enjoying the heat here in dallas, fwiw | 20:26 |
| *** shaohe_feng has joined #openstack-nova | 20:26 | |
| mriedem | sdague: but we don't which service to call right? | 20:26 |
| mriedem | compute for nova-net and neutron for neutron | 20:26 |
| mriedem | we have to see if network is in the service catalog? | 20:27 |
| mriedem | i think that's what osc does | 20:27 |
| *** kevinz has quit IRC | 20:27 | |
| sdague | well, from our functional test perspective, we should just make it config | 20:27 |
| *** xyang1 has quit IRC | 20:28 | |
| mriedem | i'm less worried about fixing functional tests right now, that shouldn't be too terrible | 20:28 |
| sdague | mriedem: ok, what are you worried about? | 20:28 |
| mriedem | i opened a bug for it though https://bugs.launchpad.net/python-novaclient/+bug/1612410 | 20:28 |
| openstack | Launchpad bug 1612410 in python-novaclient "functional tests rely on nova-network backed devstack" [Undecided,New] | 20:28 |
| mriedem | sdague: i'm worried about the network lookup by name for the CLI | 20:28 |
| mriedem | it won't be as simple as your change for the image lookup | 20:28 |
| *** ivase has quit IRC | 20:29 | |
| sdague | mriedem: right, but we'll just flag on 2.36, right? | 20:29 |
| sdague | before then, we hit the proxy, after that we talk to neutron | 20:29 |
| mriedem | yeah i suppose | 20:30 |
| sdague | until such point that we actually delete everything from nova client thats nova net, then we always talk to neutron | 20:30 |
| sdague | and, given it's just name -> id mapping... it shouldn't be terrible | 20:30 |
| *** eharney has quit IRC | 20:30 | |
| *** hamzy has quit IRC | 20:30 | |
| auggy | melwitt: do you mind if i update the assignee on that bug you're working on to you? just to alert the original assignee, but it's probably no biggie since they haven't touched it in months | 20:30 |
| sdague | ok, got to head out for some errands, I'll look to see how things went in the morning. | 20:31 |
| *** lpetrut has quit IRC | 20:31 | |
| auggy | sometimes in bug cleanup a person might get unassigned etc so it also lets the bug team know that someone is actively working on it | 20:32 |
| *** ivase has joined #openstack-nova | 20:32 | |
| *** Jeffrey4l has quit IRC | 20:35 | |
| *** eharney has joined #openstack-nova | 20:35 | |
| *** xyang1 has joined #openstack-nova | 20:35 | |
| *** Jeffrey4l has joined #openstack-nova | 20:35 | |
| *** shaohe_feng has quit IRC | 20:36 | |
| *** shaohe_feng has joined #openstack-nova | 20:36 | |
| *** ivase has quit IRC | 20:37 | |
| *** beagles is now known as beagles_biab | 20:38 | |
| *** hemanthm is now known as hemanthm|afk | 20:39 | |
| *** Apoorva has joined #openstack-nova | 20:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: network: fix handling of linux-bridge in os-vif conversion https://review.openstack.org/354143 | 20:40 |
| mriedem | ^ fixes the gate blocker for neutron so we should probably get it in | 20:41 |
| mriedem | the neutron linuxbridge job passed on it here http://logs.openstack.org/49/354149/1/check/gate-tempest-dsvm-neutron-linuxbridge/a4d85ac/ | 20:42 |
| dansmith | mriedem: these are just literally the wrong keys used in the original patch or something? | 20:42 |
| mriedem | dansmith: yup | 20:43 |
| mriedem | the os vif object had different attrs | 20:43 |
| mriedem | from the vif model | 20:43 |
| dansmith | mriedem: and it didn't pass originally because why? | 20:43 |
| dansmith | er, didn't fail | 20:43 |
| mriedem | we don't run the neutron lb job in nova check | 20:43 |
| mriedem | else it would have | 20:43 |
| mriedem | we run nova-net + lb and neutron + ovs | 20:43 |
| mriedem | i've added the neutron + lb job to nova's experimental queue | 20:43 |
| mordred | mriedem: neutron takes id/name interchangably though- you don't need to do a lookup | 20:44 |
| dansmith | so since I wouldn't want it to look like we don't play well with others, I figure I'll trust you and slam this in | 20:44 |
| mriedem | mordred: compute api doesn't, so we'll still have to do that lookup for microversion<2.36 | 20:44 |
| mriedem | in case you're using nova-net | 20:44 |
| mordred | ah. nod | 20:44 |
| *** devkulkarni2 has joined #openstack-nova | 20:45 | |
| *** yamahata has quit IRC | 20:45 | |
| * mordred welcomes our new 2.36 microversion overlords | 20:45 | |
| mriedem | we should probably actually have the neutron + lb job in the nova check queue if it's gating for neutron | 20:45 |
| mriedem | but...mo jobs mo problems | 20:45 |
| *** ametts has quit IRC | 20:45 | |
| *** devkulkarni1 has quit IRC | 20:46 | |
| *** tonytan4ever has quit IRC | 20:46 | |
| *** shaohe_feng has quit IRC | 20:46 | |
| mriedem | especially when the check queue is destroyed during US business hours now | 20:46 |
| dansmith | yeah | 20:46 |
| *** shaohe_feng has joined #openstack-nova | 20:47 | |
| dansmith | mriedem: is this a thing we could use the regex matching for, to get the extra tests if we touch something specific? | 20:47 |
| *** sdake has quit IRC | 20:48 | |
| *** jamesdenton has quit IRC | 20:49 | |
| *** armax has joined #openstack-nova | 20:49 | |
| mriedem | hmm, yeah probably | 20:50 |
| mriedem | e.g. https://github.com/openstack-infra/project-config/blob/master/zuul/layout.yaml#L1285 | 20:51 |
| *** rlrossit has joined #openstack-nova | 20:51 | |
| *** ametts has joined #openstack-nova | 20:51 | |
| mriedem | so skip gate-tempest-dsvm-neutron-linuxbridge if not hits in nova.network.* | 20:51 |
| mriedem | *no | 20:51 |
| *** devkulkarni2 has quit IRC | 20:51 | |
| dansmith | yeah something like that might be worthwhile | 20:52 |
| mriedem | i can put something up like that | 20:52 |
| *** kragtest has joined #openstack-nova | 20:53 | |
| dansmith | I can go do something else while you do that | 20:53 |
| *** eharney has quit IRC | 20:53 | |
| mriedem | you know what we also need, | 20:54 |
| mriedem | is a non-voting novaclient functional job that runs with neutron | 20:54 |
| *** pradk has quit IRC | 20:54 | |
| mriedem | so we can start making novaclient functional work with neutron devstack | 20:54 |
| *** mingyu has joined #openstack-nova | 20:55 | |
| *** lpetrut has joined #openstack-nova | 20:56 | |
| *** shaohe_feng has quit IRC | 20:56 | |
| *** shaohe_feng has joined #openstack-nova | 20:56 | |
| *** ametts has quit IRC | 20:56 | |
| *** jamesdenton has joined #openstack-nova | 20:59 | |
| *** bwallis has quit IRC | 20:59 | |
| *** tblakes has quit IRC | 21:00 | |
| *** mingyu has quit IRC | 21:02 | |
| *** kragtest has quit IRC | 21:03 | |
| *** devkulkarni1 has joined #openstack-nova | 21:04 | |
| *** xinliang has quit IRC | 21:04 | |
| *** shaohe_feng has quit IRC | 21:06 | |
| *** yamahata has joined #openstack-nova | 21:06 | |
| *** shaohe_feng has joined #openstack-nova | 21:07 | |
| *** rlrossit has quit IRC | 21:08 | |
| *** burgerk has quit IRC | 21:08 | |
| *** tblakes has joined #openstack-nova | 21:11 | |
| *** seldenr has quit IRC | 21:12 | |
| mriedem | dansmith: https://review.openstack.org/#/c/354402/ | 21:13 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for interval_opts https://review.openstack.org/331761 | 21:14 |
| *** jamesdenton has quit IRC | 21:15 | |
| mriedem | http://tinyurl.com/hu9axx9 | 21:16 |
| *** bwallis has joined #openstack-nova | 21:16 | |
| *** xinliang has joined #openstack-nova | 21:16 | |
| *** sdake has joined #openstack-nova | 21:16 | |
| *** shaohe_feng has quit IRC | 21:17 | |
| *** shaohe_feng has joined #openstack-nova | 21:17 | |
| dansmith | mriedem: do we need to run that on gate? | 21:18 |
| *** ayoung has quit IRC | 21:18 | |
| mriedem | it gates on neutron | 21:18 |
| mriedem | so i figured if it's voting in check, then might as well | 21:19 |
| dansmith | is it likely to pass check and then later fail gate? | 21:21 |
| dansmith | just seems like one fewer job that we run if we punt on gate | 21:21 |
| *** coolsvap has quit IRC | 21:21 | |
| dansmith | but either way, col | 21:21 |
| *** tonytan4ever has joined #openstack-nova | 21:21 | |
| dansmith | also, cool | 21:22 |
| mtreinish | dansmith: you probably want to keep it symmetric with neutron. Otherwise you risk threading the needle with some cross interaction | 21:23 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for xenapi_vm_utils_opts https://review.openstack.org/346251 | 21:23 |
| mtreinish | although, I'm not sure how likely that is in practice | 21:23 |
| mtreinish | things like that have happened in the past | 21:23 |
| dansmith | mtreinish: yeah, this just seems to be unlikely to have that.. it's not that tightly coupled | 21:23 |
| dansmith | just seems like we're going to have to start making some risk decisions the way the gate has been lately | 21:24 |
| mtreinish | yeah, that's fair | 21:24 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improve help text for XenServer Options https://review.openstack.org/344492 | 21:24 |
| *** baoli has quit IRC | 21:25 | |
| mriedem | i'm fine with being conservative for now and moving it to just the check queue | 21:25 |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Improving help text for xenapi_vmops_opts https://review.openstack.org/347085 | 21:25 |
| *** pradk has joined #openstack-nova | 21:26 | |
| *** shaohe_feng has quit IRC | 21:27 | |
| *** woodster_ has joined #openstack-nova | 21:27 | |
| *** shaohe_feng has joined #openstack-nova | 21:27 | |
| *** kragtest has joined #openstack-nova | 21:29 | |
| dansmith | mriedem: conservative now meaning, run in both? I think we could probably take some time later to go through and look for other optimizations we can make | 21:30 |
| dansmith | either things that don't need to run in both places, or some file filters like this | 21:31 |
| mriedem | i meant conservative as in just run in check | 21:31 |
| dansmith | okay | 21:31 |
| mriedem | that's what you're thinking to do? | 21:31 |
| mriedem | slowly wade into this pool?/ | 21:31 |
| dansmith | that's what I'd do now, yeah, I'm saying I think we might also have some other cases where we could review the risk and save on some resources | 21:32 |
| dansmith | review later I mean | 21:33 |
| *** sdake has quit IRC | 21:33 | |
| *** ijw has quit IRC | 21:34 | |
| mriedem | done | 21:34 |
| dansmith | I have placed my weighty +1 on there | 21:35 |
| *** sameo_ has quit IRC | 21:35 | |
| *** prometheanfire has joined #openstack-nova | 21:37 | |
| *** shaohe_feng has quit IRC | 21:37 | |
| prometheanfire | does nova know about the breakage coming from the neutronclient update? http://logs.openstack.org/43/353443/2/check/gate-cross-nova-python27-db-ubuntu-xenial/2b8780d/testr_results.html.gz | 21:37 |
| *** shaohe_feng has joined #openstack-nova | 21:38 | |
| *** ijw has joined #openstack-nova | 21:40 | |
| *** ijw has quit IRC | 21:40 | |
| *** ijw has joined #openstack-nova | 21:40 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for inventories to placement API https://review.openstack.org/329152 | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: placement: add filtering by attrs to resource_providers https://review.openstack.org/343046 | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a Usage and UsageList object https://review.openstack.org/329575 | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for resource_providers urls https://review.openstack.org/329151 | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add support for usages in the placement API https://review.openstack.org/329643 | 21:41 |
| mriedem | prometheanfire: i don't | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Check capacity and allocations when changing Inventory https://review.openstack.org/340601 | 21:41 |
| mriedem | prometheanfire: we f'ed them today though so maybe we deserve it | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Improve 404 and 405 response tests https://review.openstack.org/331483 | 21:41 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add placement API web utility methods https://review.openstack.org/352573 | 21:41 |
| mriedem | but i just fixed a neutronclient 5.0 thing like a week ago | 21:41 |
| prometheanfire | lol | 21:42 |
| prometheanfire | you're welcome | 21:42 |
| prometheanfire | get mikal to work on it, he should be up soon :P | 21:42 |
| mriedem | gdi | 21:43 |
| anteaya | project config cores review for voting jobs to be in check and gate | 21:43 |
| mriedem | well, i'd jump on that grenade but i'm trying to sort out sdague's novaclient fix | 21:43 |
| anteaya | if you just want it to be in check, then that is okay as long as the job is non-voting | 21:44 |
| prometheanfire | np | 21:44 |
| prometheanfire | just wanted to let you know | 21:44 |
| anteaya | if it is voting it is in both check and gate | 21:44 |
| prometheanfire | now that we have our cross project gate we are catching things like this | 21:44 |
| prometheanfire | feelsgoodman | 21:44 |
| mriedem | prometheanfire: high 5 | 21:44 |
| prometheanfire | anyway, afk for a bit | 21:45 |
| cdent | If I just rebased this stack ( https://review.openstack.org/#/c/343046/ ) off an updated master and it is doing the "This change depends on a change that failed to merge." thing, what is actually going on? | 21:45 |
| mriedem | cdent: does it do that after like the first 2 changes? | 21:46 |
| cdent | mriedem: yeah | 21:46 |
| mriedem | if so, it's probably zuul or whatever timing out on pulling down the rest after that | 21:46 |
| cdent | sigh | 21:46 |
| mriedem | you have to manually recheck the rest | 21:46 |
| *** jwcroppe_ has quit IRC | 21:46 | |
| *** jwcroppe has joined #openstack-nova | 21:47 | |
| mriedem | anteaya: ok feel free to drop a -1 on that change | 21:47 |
| mriedem | will probably discuss with sdague in the morning | 21:47 |
| *** shaohe_feng has quit IRC | 21:47 | |
| cdent | meh, the top two things had +2s that got wiped by some rebases trying to fix it. I'll just get those merged and then worry about the rest later. Thanks for making me more knowing mriedem | 21:48 |
| mriedem | np | 21:48 |
| anteaya | mriedem: sounds good | 21:48 |
| *** shaohe_feng has joined #openstack-nova | 21:48 | |
| *** burt has joined #openstack-nova | 21:48 | |
| anteaya | sdague: and I and Ajaeger have discussed this issue in the past at length | 21:48 |
| anteaya | mriedem: done | 21:49 |
| *** jwcroppe_ has joined #openstack-nova | 21:49 | |
| *** JoseMello has quit IRC | 21:50 | |
| *** jwcroppe has quit IRC | 21:51 | |
| *** sameo_ has joined #openstack-nova | 21:51 | |
| *** TravT has quit IRC | 21:54 | |
| *** devkulkarni1 has quit IRC | 21:54 | |
| *** jwcroppe_ has quit IRC | 21:54 | |
| *** haplo37__ has quit IRC | 21:54 | |
| *** rtheis has quit IRC | 21:55 | |
| *** shaohe_feng has quit IRC | 21:58 | |
| *** mriedem has quit IRC | 21:58 | |
| *** shaohe_feng has joined #openstack-nova | 21:58 | |
| *** gouthamr has quit IRC | 22:00 | |
| *** gouthamr has joined #openstack-nova | 22:01 | |
| prometheanfire | HenryG: I guess, so does nova need an update or do you? | 22:02 |
| *** burt has quit IRC | 22:02 | |
| *** edmondsw has quit IRC | 22:03 | |
| *** devkulkarni1 has joined #openstack-nova | 22:05 | |
| *** tblakes has quit IRC | 22:06 | |
| HenryG | prometheanfire: sorry, I lost context? | 22:06 |
| prometheanfire | HenryG: the neutronclient update | 22:07 |
| *** dimtruck is now known as zz_dimtruck | 22:07 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova: Return None in get_instance_id_by_floating_address https://review.openstack.org/351960 | 22:07 |
| HenryG | prometheanfire: I would defer to amotoki | 22:08 |
| *** gouthamr has quit IRC | 22:08 | |
| *** shaohe_feng has quit IRC | 22:08 | |
| *** shaohe_feng has joined #openstack-nova | 22:09 | |
| *** tonytan4ever has quit IRC | 22:09 | |
| *** rama_y has quit IRC | 22:10 | |
| *** xyang1 has quit IRC | 22:12 | |
| *** rama_y has joined #openstack-nova | 22:13 | |
| *** TravT has joined #openstack-nova | 22:13 | |
| *** lpetrut has quit IRC | 22:14 | |
| *** gjayavelu1 has joined #openstack-nova | 22:16 | |
| *** thorst_ has quit IRC | 22:17 | |
| *** gjayavelu has quit IRC | 22:18 | |
| *** shaohe_feng has quit IRC | 22:18 | |
| *** shaohe_feng has joined #openstack-nova | 22:19 | |
| *** mdrabe has quit IRC | 22:19 | |
| *** thorst_ has joined #openstack-nova | 22:20 | |
| cfriesen_ | does anyone know how nova on CentOS ends up picking up stuff from /usr/share/nova/nova-dist.conf ? | 22:24 |
| *** thorst_ has quit IRC | 22:25 | |
| *** shaohe_feng has quit IRC | 22:28 | |
| *** shaohe_feng has joined #openstack-nova | 22:29 | |
| *** oomichi has quit IRC | 22:31 | |
| *** kylek3h has quit IRC | 22:33 | |
| *** kragtest has quit IRC | 22:35 | |
| *** catintheroof has quit IRC | 22:35 | |
| *** ducttape_ has quit IRC | 22:37 | |
| *** takedakn has joined #openstack-nova | 22:38 | |
| *** shaohe_feng has quit IRC | 22:39 | |
| *** shaohe_feng has joined #openstack-nova | 22:39 | |
| *** takedakn has quit IRC | 22:42 | |
| *** rkrum has joined #openstack-nova | 22:44 | |
| dansmith | alaski: the devstack thing isn't running I think: http://logs.openstack.org/65/354265/4/check/gate-grenade-dsvm-ubuntu-trusty/6293478/logs/grenade.sh.txt.gz#_2016-08-11_22_29_35_498 | 22:44 |
| *** oomichi has joined #openstack-nova | 22:45 | |
| *** pwnall1 has quit IRC | 22:47 | |
| dansmith | uh oh | 22:49 |
| *** shaohe_feng has quit IRC | 22:49 | |
| dansmith | I guess grenade verify runs before the target devstack | 22:49 |
| *** shaohe_feng has joined #openstack-nova | 22:49 | |
| *** jwcroppe has joined #openstack-nova | 22:50 | |
| *** cdent has quit IRC | 22:51 | |
| *** mriedem has joined #openstack-nova | 22:53 | |
| *** shaohe_feng has quit IRC | 22:59 | |
| *** jwcroppe has quit IRC | 22:59 | |
| *** shaohe_feng has joined #openstack-nova | 23:02 | |
| *** hoangcx2 has joined #openstack-nova | 23:07 | |
| *** gjayavelu1 has quit IRC | 23:08 | |
| *** hoangcx has quit IRC | 23:08 | |
| *** shaohe_feng has quit IRC | 23:09 | |
| *** tonytan4ever has joined #openstack-nova | 23:10 | |
| *** shaohe_feng has joined #openstack-nova | 23:10 | |
| *** devkulkarni1 has quit IRC | 23:12 | |
| *** devkulkarni1 has joined #openstack-nova | 23:12 | |
| *** ijw has quit IRC | 23:13 | |
| *** code-R has quit IRC | 23:13 | |
| *** Sukhdev has quit IRC | 23:13 | |
| *** tonytan4ever has quit IRC | 23:14 | |
| *** shaohe_feng has quit IRC | 23:20 | |
| *** bwallis has quit IRC | 23:20 | |
| *** ducttape_ has joined #openstack-nova | 23:20 | |
| *** shaohe_feng has joined #openstack-nova | 23:20 | |
| *** kragniz1 has joined #openstack-nova | 23:21 | |
| alaski | yeah, I see some db syncs and online migrations but not the cells setup | 23:22 |
| *** nic has quit IRC | 23:22 | |
| *** tri2sing has quit IRC | 23:23 | |
| *** kragniz1 is now known as testing1234 | 23:27 | |
| *** shaohe_feng has quit IRC | 23:30 | |
| *** shaohe_feng has joined #openstack-nova | 23:30 | |
| *** ijw has joined #openstack-nova | 23:31 | |
| *** rrecio has quit IRC | 23:34 | |
| *** ijw has quit IRC | 23:35 | |
| mriedem | sdague: fun, the fake client stubs in novaclient unit tests have stubbed up a way for tests to pass things like --image 1 and return a valid image response, | 23:36 |
| mriedem | even though that 1 should be a uuid | 23:36 |
| mriedem | you can't actually do like a nova image-show 1 from the command line | 23:36 |
| *** StevenK_ is now known as StevenK | 23:38 | |
| *** shaohe_feng has quit IRC | 23:40 | |
| alex_xu | good morning/evening everyone | 23:41 |
| *** shaohe_feng has joined #openstack-nova | 23:43 | |
| gmann | alex_xu: good morning | 23:43 |
| *** ijw has joined #openstack-nova | 23:43 | |
| alex_xu | gmann: good morning :) | 23:44 |
| gmann | sdague: Thanks for pushing those patch series. | 23:44 |
| gmann | sdague: I will puch other remaining by today. | 23:44 |
| sdague | gmann: great, thanks. | 23:44 |
| gmann | sdague: Also remove the same from start server? | 23:45 |
| sdague | gmann: lets keep it in one long series so that it doesn't have any self conflicting patches | 23:45 |
| oomichi | alex_xu: good morning | 23:45 |
| *** ijw has quit IRC | 23:45 | |
| sdague | gmann: I can't remember where johnthetubaguy landed on that one, and I'm only checking in briefly here before heading out for the night | 23:45 |
| *** ijw has joined #openstack-nova | 23:45 | |
| gmann | sdague: sure | 23:46 |
| *** rama_y has quit IRC | 23:47 | |
| *** beagles_biab is now known as beagles | 23:49 | |
| dansmith | alaski: yeah, I may need to replicate the cell setup in the nova upgrade part, even though devstack would normally do it for us.. need to make sure that's okay | 23:50 |
| *** shaohe_feng has quit IRC | 23:50 | |
| *** shaohe_feng has joined #openstack-nova | 23:51 | |
| dansmith | alaski: looks like the simple_cell_setup command is not reentrant | 23:52 |
| dansmith | er, idempotent | 23:52 |
| *** sdague has quit IRC | 23:55 | |
| *** jwcroppe has joined #openstack-nova | 23:56 | |
| *** jvgrant_ has joined #openstack-nova | 23:57 | |
| *** bwallis has joined #openstack-nova | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!