| *** Swami has quit IRC | 00:00 | |
| *** Swami_ has quit IRC | 00:00 | |
| *** Swami has joined #openstack-nova | 00:01 | |
| *** unicell has joined #openstack-nova | 00:10 | |
| *** unicell is now known as Guest98378 | 00:11 | |
| *** marst has quit IRC | 00:12 | |
| *** mtanino has quit IRC | 00:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 00:17 |
|---|---|---|
| *** Guest98378 has quit IRC | 00:17 | |
| *** ducttape_ has joined #openstack-nova | 00:20 | |
| *** tbachman has quit IRC | 00:20 | |
| *** hongbin has quit IRC | 00:22 | |
| *** tbachman has joined #openstack-nova | 00:24 | |
| *** dtp has quit IRC | 00:25 | |
| *** nic has quit IRC | 00:26 | |
| *** thorst_ has joined #openstack-nova | 00:28 | |
| *** dimtruck is now known as zz_dimtruck | 00:29 | |
| *** jwcroppe has quit IRC | 00:30 | |
| *** yingjun has joined #openstack-nova | 00:30 | |
| *** voelzmo has joined #openstack-nova | 00:30 | |
| *** ducttape_ has quit IRC | 00:31 | |
| *** salv-orl_ has quit IRC | 00:32 | |
| *** voelzmo has quit IRC | 00:35 | |
| *** mlavalle has quit IRC | 00:35 | |
| *** gongysh has joined #openstack-nova | 00:37 | |
| *** jose-phillips has quit IRC | 00:38 | |
| *** thorst_ has quit IRC | 00:45 | |
| *** tbachman has quit IRC | 00:45 | |
| *** thorst_ has joined #openstack-nova | 00:45 | |
| *** annegentle has quit IRC | 00:48 | |
| *** thorst_ has quit IRC | 00:50 | |
| *** tbachman has joined #openstack-nova | 00:52 | |
| *** gongysh has quit IRC | 00:52 | |
| *** hoangcx has joined #openstack-nova | 00:53 | |
| *** bradjones has quit IRC | 00:55 | |
| *** annegentle has joined #openstack-nova | 00:55 | |
| *** phuongnh has joined #openstack-nova | 00:55 | |
| *** tovin07 has joined #openstack-nova | 00:57 | |
| *** mriedem has joined #openstack-nova | 00:58 | |
| *** mvensky has quit IRC | 01:01 | |
| *** claudiub|2 has quit IRC | 01:04 | |
| *** knikolla has quit IRC | 01:07 | |
| *** knikolla has joined #openstack-nova | 01:08 | |
| *** knikolla has quit IRC | 01:08 | |
| *** knikolla has joined #openstack-nova | 01:08 | |
| *** knikolla has quit IRC | 01:08 | |
| *** knikolla has joined #openstack-nova | 01:09 | |
| *** knikolla has quit IRC | 01:09 | |
| *** knikolla has joined #openstack-nova | 01:09 | |
| *** knikolla has left #openstack-nova | 01:10 | |
| *** amotoki has joined #openstack-nova | 01:15 | |
| *** yingjun has quit IRC | 01:16 | |
| *** mvensky has joined #openstack-nova | 01:17 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 01:18 |
| *** kevinz has joined #openstack-nova | 01:18 | |
| *** thorst_ has joined #openstack-nova | 01:19 | |
| *** thorst_ has quit IRC | 01:19 | |
| *** annegentle has quit IRC | 01:20 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 01:20 |
| *** gyee has quit IRC | 01:23 | |
| *** mvensky has quit IRC | 01:25 | |
| *** mvensky has joined #openstack-nova | 01:26 | |
| *** mvensky has quit IRC | 01:29 | |
| *** mvensky has joined #openstack-nova | 01:30 | |
| *** jdurgin has quit IRC | 01:34 | |
| *** woodster_ has quit IRC | 01:45 | |
| *** mvensky has quit IRC | 01:46 | |
| *** Sukhdev has quit IRC | 01:49 | |
| *** tbachman has quit IRC | 01:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 01:51 |
| *** ducttape_ has joined #openstack-nova | 01:55 | |
| *** yuntongjin has joined #openstack-nova | 01:55 | |
| *** ducttape_ has quit IRC | 01:59 | |
| *** sofie has joined #openstack-nova | 01:59 | |
| *** tbachman has joined #openstack-nova | 02:00 | |
| openstackgerrit | Li Yingjun proposed openstack/nova: Code improvement for hypervisors controller https://review.openstack.org/335282 | 02:00 |
| *** zz_dimtruck is now known as dimtruck | 02:00 | |
| *** thorst_ has joined #openstack-nova | 02:01 | |
| *** thorst_ has quit IRC | 02:01 | |
| *** cr0wrx has quit IRC | 02:03 | |
| mriedem | harlowja: looks like we have a new nova py35 gate buster with new oslo.serialization release http://logs.openstack.org/37/410737/5/check/gate-nova-python35-db/4fec66c/console.html#_2017-01-20_01_56_21_021702 | 02:04 |
| harlowja | another one | 02:04 |
| mriedem | although isn't that was you fixed to get us to this point? | 02:05 |
| mriedem | might just need a rebase | 02:05 |
| harlowja | ya | 02:05 |
| harlowja | it looks like the one i changed, ha | 02:05 |
| mriedem | yeah just needs a rebase i think | 02:06 |
| mriedem | i'll do that | 02:06 |
| harlowja | k | 02:06 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | 02:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif-util: set vif_name for vhostuser ovs os-vif port https://review.openstack.org/410737 | 02:08 |
| *** gongysh has joined #openstack-nova | 02:11 | |
| *** ducttape_ has joined #openstack-nova | 02:13 | |
| *** raunak has quit IRC | 02:14 | |
| *** tbachman has quit IRC | 02:15 | |
| *** tbachman has joined #openstack-nova | 02:18 | |
| *** browne has quit IRC | 02:20 | |
| openstackgerrit | melanie witt proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 02:26 |
| mriedem | can we get this in to get some more details on this failure when it happens? https://review.openstack.org/#/c/421221/ | 02:27 |
| mriedem | just saw it hit another change | 02:27 |
| *** tiendc has joined #openstack-nova | 02:28 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745 | 02:28 |
| mriedem | melwitt: just a heads up since i'm not sure i'll have time to get to it before i leave for vacation, but on https://review.openstack.org/#/c/420981/ | 02:29 |
| mriedem | my plan was to make a call from devstack-gate to run discover_hosts on the primary node (or only node in a single-node setup) | 02:30 |
| mriedem | d-g uses ansible and we should be able to just write a task to ssh onto the host and run the discover_hosts command | 02:30 |
| mriedem | i'll probably try to write a handoff email to dansmith and you too about the cells stuff before i leave | 02:31 |
| melwitt | mriedem: okay, cool. thanks for the heads up | 02:31 |
| mriedem | i'll have my laptop with me, but probably only checking email in the morning | 02:31 |
| melwitt | k | 02:33 |
| *** Apoorva has quit IRC | 02:36 | |
| *** edmondsw_ has quit IRC | 02:36 | |
| *** baoli has joined #openstack-nova | 02:40 | |
| *** diga has joined #openstack-nova | 02:45 | |
| melwitt | mriedem: handoff email would be a nice to have, help us focus on specific things you had in mind | 02:45 |
| *** Oku_OS is now known as Oku_OS-away | 02:46 | |
| *** links has joined #openstack-nova | 02:53 | |
| *** ducttape_ has quit IRC | 02:54 | |
| *** edmondsw has joined #openstack-nova | 02:57 | |
| *** hongbin has joined #openstack-nova | 03:01 | |
| *** edmondsw has quit IRC | 03:02 | |
| mriedem | melwitt: actually hoping this might work https://review.openstack.org/423005 | 03:02 |
| mriedem | doesn't account for grenade yet | 03:02 |
| *** tovin07_ has joined #openstack-nova | 03:06 | |
| *** Swami__ has quit IRC | 03:07 | |
| *** Swami has quit IRC | 03:07 | |
| *** tbachman has quit IRC | 03:07 | |
| *** alexpilotti has quit IRC | 03:09 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 03:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 03:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Ensure we mark baremetal links as phy links https://review.openstack.org/422068 | 03:13 |
| *** larainema has quit IRC | 03:26 | |
| *** yamahata has quit IRC | 03:31 | |
| *** Apoorva has joined #openstack-nova | 03:32 | |
| *** yuntongjin has quit IRC | 03:32 | |
| *** tbachman has joined #openstack-nova | 03:34 | |
| *** thorst_ has joined #openstack-nova | 03:39 | |
| *** thorst_ has quit IRC | 03:40 | |
| *** armax has joined #openstack-nova | 03:40 | |
| *** nicolasbock has quit IRC | 03:41 | |
| *** raunak has joined #openstack-nova | 03:41 | |
| *** baoli has quit IRC | 03:42 | |
| *** baoli_ has joined #openstack-nova | 03:47 | |
| *** amotoki has quit IRC | 03:49 | |
| *** alexpilotti has joined #openstack-nova | 03:55 | |
| *** mdnadeem has joined #openstack-nova | 03:57 | |
| *** psachin has joined #openstack-nova | 03:59 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: placement: create aggregate map in report client https://review.openstack.org/407309 | 04:03 |
| mriedem | jaypipes: i went ahead and scoped the 1.1 microversion request to just the GET on the RP aggregates in ^ | 04:03 |
| mriedem | jaypipes: don't hate me | 04:03 |
| mriedem | i might be overthinking that one but i'm a bit touchy on the rollout of this stuff now | 04:03 |
| *** links has quit IRC | 04:16 | |
| *** links has joined #openstack-nova | 04:18 | |
| *** amotoki has joined #openstack-nova | 04:19 | |
| *** liverpooler has quit IRC | 04:23 | |
| *** pbandark has joined #openstack-nova | 04:24 | |
| *** amotoki_ has joined #openstack-nova | 04:25 | |
| *** udesale has joined #openstack-nova | 04:25 | |
| *** amotoki has quit IRC | 04:27 | |
| *** ratailor has joined #openstack-nova | 04:28 | |
| *** udesale has quit IRC | 04:30 | |
| *** guchihiro has joined #openstack-nova | 04:31 | |
| *** voelzmo has joined #openstack-nova | 04:32 | |
| *** Apoorva has quit IRC | 04:32 | |
| *** raunak_ has joined #openstack-nova | 04:33 | |
| *** raunak has quit IRC | 04:34 | |
| *** raunak_ is now known as raunak | 04:34 | |
| *** voelzmo has quit IRC | 04:37 | |
| *** hongbin_ has joined #openstack-nova | 04:39 | |
| *** hongbin has quit IRC | 04:40 | |
| *** yuntongjin has joined #openstack-nova | 04:42 | |
| *** gouthamr has quit IRC | 04:46 | |
| *** hongbin_ has quit IRC | 04:47 | |
| *** larainema has joined #openstack-nova | 04:48 | |
| *** ratailor has quit IRC | 04:50 | |
| *** catinthe_ has quit IRC | 04:54 | |
| *** krtaylor has joined #openstack-nova | 04:57 | |
| *** voelzmo has joined #openstack-nova | 05:00 | |
| *** voelzmo has quit IRC | 05:01 | |
| *** catintheroof has joined #openstack-nova | 05:02 | |
| *** dikonoor has joined #openstack-nova | 05:02 | |
| *** baoli_ has quit IRC | 05:03 | |
| *** dimtruck is now known as zz_dimtruck | 05:04 | |
| Kevin_Zheng | the nova intergration gate got stacked? | 05:04 |
| Kevin_Zheng | stucked | 05:04 |
| *** sheel has joined #openstack-nova | 05:04 | |
| *** baoli has joined #openstack-nova | 05:04 | |
| *** baoli has quit IRC | 05:05 | |
| *** jerrygb has quit IRC | 05:05 | |
| *** yuntongjin has quit IRC | 05:09 | |
| *** udesale has joined #openstack-nova | 05:09 | |
| *** sacharya_ has quit IRC | 05:10 | |
| *** udesale has quit IRC | 05:10 | |
| *** udesale has joined #openstack-nova | 05:11 | |
| *** sacharya has joined #openstack-nova | 05:11 | |
| *** tlian has quit IRC | 05:13 | |
| *** catintheroof has quit IRC | 05:14 | |
| *** prateek has joined #openstack-nova | 05:15 | |
| *** slaweq has quit IRC | 05:24 | |
| *** zz_dimtruck is now known as dimtruck | 05:24 | |
| *** slaweq has joined #openstack-nova | 05:26 | |
| *** archies has joined #openstack-nova | 05:29 | |
| *** eandersson_ has joined #openstack-nova | 05:30 | |
| archies | @swamireddy, Hi, wants to check your views on the https://bugs.launchpad.net/nova/+bug/1636489 | 05:31 |
| openstack | Launchpad bug 1636489 in OpenStack Compute (nova) "Volume attachment fails for all the available instances after running different volume operations for 1-2 hours or more." [Undecided,New] | 05:31 |
| *** mriedem has quit IRC | 05:32 | |
| archies | @swamireddy, appears from the bug which you are working on, seems similar to me | 05:32 |
| *** tbachman has quit IRC | 05:32 | |
| *** eandersson__ has quit IRC | 05:33 | |
| *** dimtruck is now known as zz_dimtruck | 05:36 | |
| *** cfriesen has quit IRC | 05:37 | |
| *** voelzmo has joined #openstack-nova | 05:37 | |
| *** cfriesen has joined #openstack-nova | 05:38 | |
| *** jerrygb has joined #openstack-nova | 05:39 | |
| *** thorst_ has joined #openstack-nova | 05:40 | |
| *** voelzmo has quit IRC | 05:41 | |
| *** catintheroof has joined #openstack-nova | 05:46 | |
| *** thorst_ has quit IRC | 05:48 | |
| *** ratailor has joined #openstack-nova | 05:50 | |
| gmann | Kevin_Zheng: yea with oslo release i think | 05:50 |
| *** ijw_ has quit IRC | 05:52 | |
| *** sandanar has joined #openstack-nova | 05:55 | |
| *** ecelik has joined #openstack-nova | 06:02 | |
| *** lpetrut has joined #openstack-nova | 06:03 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288 | 06:04 |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Ironic: Add soft power off support to Ironic driver. https://review.openstack.org/407977 | 06:06 |
| Kevin_Zheng | gamnn: so what can we do to solve this? | 06:08 |
| gmann | Kevin_Zheng: as mriedem said, let it merge and then rebase - https://review.openstack.org/#/c/422323/4 | 06:10 |
| gmann | alex_xu: oomichi johnthetubaguy in case you guys around ^^ | 06:10 |
| Kevin_Zheng | ah， good | 06:10 |
| gmann | we need to wait for that to merge | 06:12 |
| Kevin_Zheng | understood | 06:12 |
| *** ijw has joined #openstack-nova | 06:15 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Ironic: Add soft power off support to Ironic driver. https://review.openstack.org/407977 | 06:18 |
| gmann | alex_xu: thanks, | 06:19 |
| *** ijw has quit IRC | 06:19 | |
| *** armax has quit IRC | 06:19 | |
| alex_xu | gmann: thanks for your info also | 06:19 |
| *** armax has joined #openstack-nova | 06:20 | |
| alex_xu | gmann: Kevin_Zheng still thinking of whether we can merge this one first https://review.openstack.org/#/c/415142/23 | 06:20 |
| *** armax has quit IRC | 06:20 | |
| alex_xu | the policy one looks like still have something to figure out | 06:20 |
| *** dtp has joined #openstack-nova | 06:24 | |
| Kevin_Zheng | alex_xu: hmm, hope we can work it out soon | 06:24 |
| dtp | howdy. to propose a blueprint for pike, it sounds like i should drop a link in the nova meeting agenda? | 06:26 |
| *** zz_dimtruck is now known as dimtruck | 06:27 | |
| gmann | alex_xu: in meeting, ll check right after that | 06:27 |
| *** mriedem has joined #openstack-nova | 06:29 | |
| alex_xu | gmann: thanks | 06:29 |
| *** voelzmo has joined #openstack-nova | 06:30 | |
| *** phuongnh has quit IRC | 06:30 | |
| gokhan | hi team, I need help about nova -lxd . My openstaCk installation is newton on ubuntu 16.04 LTS. I created image with run 'glance image-create --name="trusty" --public --progress --container-format=bare --disk-format=raw --file xenial-server-cloudimg-amd64-root.tar.gz --property hypervisor_type=lxd' . but when I launched instance I am getting bad image format error. These are logs: http://paste.openstack.org/show/595724/ | 06:30 |
| *** phuongnh has joined #openstack-nova | 06:31 | |
| *** edmondsw has joined #openstack-nova | 06:34 | |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix typo in 216_havana.py https://review.openstack.org/423075 | 06:34 |
| *** voelzmo has quit IRC | 06:34 | |
| openstackgerrit | zhangyanxian proposed openstack/nova: Fix typo in 216_havana.py https://review.openstack.org/423075 | 06:35 |
| *** claudiub|2 has joined #openstack-nova | 06:35 | |
| *** edmondsw has quit IRC | 06:38 | |
| *** baoli has joined #openstack-nova | 06:40 | |
| *** dimtruck is now known as zz_dimtruck | 06:42 | |
| *** satyar has joined #openstack-nova | 06:44 | |
| *** baoli has quit IRC | 06:44 | |
| *** mjura has joined #openstack-nova | 06:46 | |
| *** ayogi has joined #openstack-nova | 06:47 | |
| *** nkrinner_afk is now known as nkrinner | 06:53 | |
| *** yuntongjin has joined #openstack-nova | 06:53 | |
| *** ecelik has left #openstack-nova | 06:56 | |
| *** tiendc has quit IRC | 06:57 | |
| *** lpetrut has quit IRC | 06:59 | |
| *** ijw has joined #openstack-nova | 07:04 | |
| openstackgerrit | Alex Xu proposed openstack/nova: [POC][placement] Report cpu features to placement Traits API https://review.openstack.org/417288 | 07:05 |
| *** ijw_ has joined #openstack-nova | 07:07 | |
| *** lpetrut has joined #openstack-nova | 07:07 | |
| *** salv-orlando has joined #openstack-nova | 07:09 | |
| *** Jeffrey4l_ has quit IRC | 07:10 | |
| *** ijw has quit IRC | 07:10 | |
| *** ijw__ has joined #openstack-nova | 07:11 | |
| *** ijw_ has quit IRC | 07:11 | |
| *** gabor_antal_ has joined #openstack-nova | 07:13 | |
| *** salv-orlando has quit IRC | 07:14 | |
| *** gabor_antal has quit IRC | 07:14 | |
| openstackgerrit | zhangyanxian proposed openstack/nova-specs: Fix typos in docs https://review.openstack.org/412338 | 07:14 |
| *** voelzmo has joined #openstack-nova | 07:17 | |
| *** Oku_OS-away is now known as Oku_OS | 07:17 | |
| openstackgerrit | Istvan Imre proposed openstack/python-novaclient: Pass relevant parameters to Token based authentication https://review.openstack.org/419441 | 07:17 |
| *** adisky_ has joined #openstack-nova | 07:17 | |
| *** tiendc has joined #openstack-nova | 07:17 | |
| *** phuongnh has quit IRC | 07:20 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: do not merge https://review.openstack.org/422841 | 07:20 |
| *** raunak has quit IRC | 07:22 | |
| *** raunak has joined #openstack-nova | 07:24 | |
| *** lpetrut has quit IRC | 07:24 | |
| *** dtp has quit IRC | 07:26 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 07:31 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 07:32 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 07:32 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 07:32 |
| *** zz_dimtruck is now known as dimtruck | 07:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 07:33 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 07:33 |
| *** markus_z has joined #openstack-nova | 07:35 | |
| *** yuntongjin has quit IRC | 07:35 | |
| *** sridharg has joined #openstack-nova | 07:37 | |
| *** gabor_antal_ has quit IRC | 07:37 | |
| *** gabor_antal has joined #openstack-nova | 07:37 | |
| *** sahid has joined #openstack-nova | 07:41 | |
| *** dimtruck is now known as zz_dimtruck | 07:43 | |
| *** pcaruana has joined #openstack-nova | 07:43 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 07:44 |
| *** John341__ has joined #openstack-nova | 07:46 | |
| *** dosaboy has joined #openstack-nova | 07:47 | |
| *** Hazelesque_ has joined #openstack-nova | 07:47 | |
| *** melwitt_ has joined #openstack-nova | 07:48 | |
| *** ijw__ has quit IRC | 07:50 | |
| *** ttx_ has joined #openstack-nova | 07:51 | |
| *** dgonzalez has quit IRC | 07:51 | |
| *** rmk has quit IRC | 07:51 | |
| *** flwang has quit IRC | 07:51 | |
| *** Hazelesque has quit IRC | 07:51 | |
| *** dosaboy_ has quit IRC | 07:51 | |
| *** ttx has quit IRC | 07:51 | |
| *** melwitt has quit IRC | 07:51 | |
| *** psachin has quit IRC | 07:51 | |
| *** John341_ has quit IRC | 07:51 | |
| *** dgonzalez has joined #openstack-nova | 07:51 | |
| *** esracelik has joined #openstack-nova | 07:51 | |
| *** esracelik has left #openstack-nova | 07:51 | |
| *** guchihiro has left #openstack-nova | 07:52 | |
| *** raunak has quit IRC | 07:52 | |
| *** ttx_ is now known as ttx | 07:53 | |
| *** flwang has joined #openstack-nova | 07:54 | |
| *** raunak has joined #openstack-nova | 07:55 | |
| *** huanxie has joined #openstack-nova | 07:56 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Extend tag length limit to 80 https://review.openstack.org/423112 | 07:57 |
| *** rmk has joined #openstack-nova | 07:57 | |
| *** rmk has quit IRC | 07:57 | |
| *** rmk has joined #openstack-nova | 07:57 | |
| *** psachin has joined #openstack-nova | 08:00 | |
| *** zz_dimtruck is now known as dimtruck | 08:00 | |
| tdurakov | gibi: hi, are you around? | 08:01 |
| *** psachin has quit IRC | 08:03 | |
| *** psachin has joined #openstack-nova | 08:03 | |
| *** tesseract has joined #openstack-nova | 08:05 | |
| *** eandersson_ has quit IRC | 08:06 | |
| *** eandersson_ has joined #openstack-nova | 08:06 | |
| *** salv-orlando has joined #openstack-nova | 08:11 | |
| *** dimtruck is now known as zz_dimtruck | 08:12 | |
| openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpus_reserved in NUMACell https://review.openstack.org/400874 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus reserved https://review.openstack.org/400876 | 08:12 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | 08:12 |
| *** phuongnh has joined #openstack-nova | 08:13 | |
| *** damien_r has joined #openstack-nova | 08:15 | |
| *** larainema has quit IRC | 08:16 | |
| *** yamahata has joined #openstack-nova | 08:22 | |
| *** vladikr has quit IRC | 08:23 | |
| *** ralonsoh has joined #openstack-nova | 08:23 | |
| *** vladikr has joined #openstack-nova | 08:23 | |
| *** tovin07 has quit IRC | 08:30 | |
| *** afazekas has quit IRC | 08:30 | |
| *** tovin07 has joined #openstack-nova | 08:30 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142 | 08:31 |
| *** liusheng has quit IRC | 08:31 | |
| *** liusheng has joined #openstack-nova | 08:31 | |
| *** artom has quit IRC | 08:31 | |
| *** bauzas has quit IRC | 08:32 | |
| *** artom has joined #openstack-nova | 08:32 | |
| *** afazekas has joined #openstack-nova | 08:35 | |
| *** bauzas has joined #openstack-nova | 08:35 | |
| gibi | tdurakov: hi | 08:36 |
| tdurakov | gibi, could you take a look, please https://review.openstack.org/#/c/421981/ | 08:36 |
| *** jpena|off is now known as jpena | 08:37 | |
| gibi | tdurakov: looking... | 08:38 |
| *** amoralej|off is now known as amoralej | 08:39 | |
| gibi | tdurakov: I'm wondering why this not cause any notification sample test to fail | 08:45 |
| tdurakov | gibi, good question, I think we need to change functional tests that verify these templates | 08:45 |
| tdurakov | I'd expect it compares 2 json objects, where timestamps are already converted to datetime.datetime objects | 08:46 |
| tdurakov | or smth. like that | 08:46 |
| *** wlhc has joined #openstack-nova | 08:47 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNodeList.get_all_by_uuids method https://review.openstack.org/418134 | 08:48 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 08:48 |
| tdurakov | gibi: so, what do you think about this change? | 08:49 |
| tdurakov | is it worth merging it, and then figure out what's wrong with tests | 08:49 |
| gibi | tdurakov: direction is good. I'm worried about the test and we might have to look in not just the instance.update code path but the instance.<action> pathes as well | 08:50 |
| *** jpena is now known as jpena|off | 08:51 | |
| tdurakov | gibi, could you please leave a comment with details? | 08:51 |
| gibi | tdurakov: will do | 08:51 |
| tdurakov | gibi: maybe you could explain me, why nova handles some fields explicilty? I'd expect it should be done by serializer | 08:53 |
| *** jpena|off is now known as jpena | 08:54 | |
| gibi | tdurakov: what do you mean by explicitly in this context? | 08:56 |
| *** shaner has quit IRC | 08:56 | |
| tdurakov | gibi: conversion of datetime objects to string | 08:56 |
| *** salv-orlando has quit IRC | 08:56 | |
| tdurakov | for example | 08:56 |
| *** Apoorva has joined #openstack-nova | 08:56 | |
| tdurakov | gibi: we briefly discussed this patch with mriedem yesterday, what do you think about version bump for this change | 08:57 |
| *** shaner has joined #openstack-nova | 08:57 | |
| *** Apoorva has quit IRC | 08:58 | |
| gibi | the explicit conversion was part of the legacy notification (for example in case of audit fields) and during the versioned transformation we did not changed that | 08:59 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| gibi | I personally didn't want to change the legacy code path of the instance.update | 09:00 |
| gibi | to avoid behavior change in the legacy | 09:00 |
| gibi | unfortunately I also piggybacked on that legacy implementation so the explicitness become the part of the versioned path as well | 09:01 |
| gibi | regarding the version bump | 09:01 |
| gibi | as far as I understand we didn't use to bump api versions for bugfixes | 09:01 |
| gibi | but here bumping a version is a lot cheaper | 09:02 |
| gibi | than in the apu | 09:02 |
| gibi | api | 09:02 |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** zz_dimtruck is now known as dimtruck | 09:03 | |
| gibi | btw you patch now changes the legacy instance.update as well | 09:03 |
| *** namnh has joined #openstack-nova | 09:05 | |
| tdurakov | gibi, should I worry about that? | 09:06 |
| tdurakov | about the legacy instance.update | 09:06 |
| diga | gibi: Hi | 09:08 |
| diga | I am writting spec for placement notification now | 09:08 |
| diga | gibi: have some questions on that | 09:08 |
| *** shaner has quit IRC | 09:09 | |
| *** openstackgerrit has joined #openstack-nova | 09:12 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver (console) https://review.openstack.org/409402 | 09:12 |
| *** dimtruck is now known as zz_dimtruck | 09:12 | |
| *** derekh has joined #openstack-nova | 09:15 | |
| diga | gibi: I need to bring all the resource provider field inside the versioned objects, correct ? | 09:15 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V PCI Passthrough https://review.openstack.org/420614 | 09:16 |
| *** johnthetubaguy has quit IRC | 09:16 | |
| *** shaner has joined #openstack-nova | 09:17 | |
| *** johnthetubaguy has joined #openstack-nova | 09:19 | |
| *** wlhc has quit IRC | 09:21 | |
| gmann | bauzas: just this merge will work on py35 failure - https://review.openstack.org/#/c/422323/ | 09:22 |
| gmann | no rebase i think | 09:22 |
| gibi | tdurakov: this might cause pain for the legacy consumers as the date is now in different format | 09:22 |
| gibi | tdurakov: but I think the legacy notification causes this kind of pain all the time so I think we can ignore it | 09:23 |
| tdurakov | gibi, honestly I'd expect current approach cause more pain | 09:23 |
| *** dgonzalez_ has joined #openstack-nova | 09:23 | |
| tdurakov | gibi: ++ | 09:23 |
| gibi | tdurakov: agree | 09:23 |
| gibi | diga: hi! good question | 09:24 |
| gibi | diga: I think we have to judge how usefull a field for the notification consumer | 09:24 |
| *** lpetrut has joined #openstack-nova | 09:24 | |
| diga | gibi: :) okay | 09:24 |
| gibi | diga: I suggest propose what you think is usefull and then in the spec review the placement guys can suggest improvements there | 09:25 |
| diga | gibi: Yep | 09:25 |
| diga | gibi: I will send one WIP spec today, just have a look at it | 09:25 |
| gibi | tdurakov: shall I look into the sample test problem or you will check that? | 09:25 |
| diga | gibi: that will be good approach | 09:25 |
| gibi | diga: good, I will try to check it | 09:25 |
| tdurakov | gibi, I'll check, but I think it worth to do it separately | 09:26 |
| diga | gibi: Thanks :) | 09:27 |
| *** dgonzalez has quit IRC | 09:27 | |
| *** dgonzalez_ is now known as dgonzalez | 09:27 | |
| gibi | tdurakov: OK let's do that separately | 09:27 |
| tdurakov | gibi, could you please +-1 the change with a fix | 09:29 |
| *** ratailor has quit IRC | 09:29 | |
| gibi | tdurakov: just did +1 it :) | 09:31 |
| gibi | with the comments we agreed | 09:31 |
| tdurakov | gibi, thank you! | 09:31 |
| gibi | tdurakov: thank you for fixing the bug! :) | 09:31 |
| *** aunnam has quit IRC | 09:32 | |
| *** sneti_ has quit IRC | 09:32 | |
| *** tommylikehu has quit IRC | 09:32 | |
| *** artom_ has joined #openstack-nova | 09:33 | |
| *** artom has quit IRC | 09:33 | |
| *** tommylikehu has joined #openstack-nova | 09:33 | |
| *** lucas-afk is now known as lucasagomes | 09:34 | |
| *** efoley has joined #openstack-nova | 09:35 | |
| *** tojuvone has quit IRC | 09:36 | |
| *** tojuvone has joined #openstack-nova | 09:36 | |
| *** kevinz has quit IRC | 09:38 | |
| *** yassine has joined #openstack-nova | 09:38 | |
| *** aunnam has joined #openstack-nova | 09:39 | |
| *** yassine is now known as Guest55718 | 09:39 | |
| *** sneti has joined #openstack-nova | 09:39 | |
| *** jerrygb_ has joined #openstack-nova | 09:41 | |
| *** lpetrut has quit IRC | 09:42 | |
| *** jerrygb has quit IRC | 09:43 | |
| *** jerrygb has joined #openstack-nova | 09:46 | |
| *** thorst_ has joined #openstack-nova | 09:46 | |
| *** mvk has quit IRC | 09:46 | |
| *** jerrygb_ has quit IRC | 09:48 | |
| *** thorst_ has quit IRC | 09:51 | |
| *** a-pugachev has joined #openstack-nova | 09:51 | |
| *** tiendc has quit IRC | 09:53 | |
| gibi | tdurakov: just checked the #2) comment and I can confirm that instance.<action> notifications are not affected | 09:53 |
| tdurakov | ok, cool, thank you | 09:54 |
| *** ratailor has joined #openstack-nova | 09:54 | |
| *** sacharya has quit IRC | 09:57 | |
| *** sacharya has joined #openstack-nova | 09:58 | |
| *** owalsh has quit IRC | 09:59 | |
| *** owalsh has joined #openstack-nova | 09:59 | |
| *** hfu has quit IRC | 10:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 10:01 |
| *** yamahata has quit IRC | 10:02 | |
| *** zz_dimtruck is now known as dimtruck | 10:04 | |
| *** karimb has joined #openstack-nova | 10:05 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: do not merge https://review.openstack.org/422841 | 10:06 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 10:06 |
| *** tiendc has joined #openstack-nova | 10:06 | |
| openstackgerrit | Duong Ha-Quang proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells/fakes.py https://review.openstack.org/342043 | 10:10 |
| *** edmondsw has joined #openstack-nova | 10:10 | |
| openstackgerrit | Duong Ha-Quang proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells/fakes.py https://review.openstack.org/342043 | 10:11 |
| *** jerrygb has quit IRC | 10:13 | |
| *** dimtruck is now known as zz_dimtruck | 10:13 | |
| *** sambetts|afk is now known as sambetts | 10:13 | |
| *** jerrygb has joined #openstack-nova | 10:14 | |
| *** gcb has joined #openstack-nova | 10:14 | |
| *** edmondsw has quit IRC | 10:15 | |
| *** mvk has joined #openstack-nova | 10:17 | |
| *** satyar has quit IRC | 10:19 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: PoC: add hierarchy quota driver https://review.openstack.org/391072 | 10:20 |
| *** lpetrut has joined #openstack-nova | 10:21 | |
| *** gongysh has quit IRC | 10:22 | |
| *** sacharya has quit IRC | 10:23 | |
| *** sacharya has joined #openstack-nova | 10:24 | |
| *** hoangcx has quit IRC | 10:25 | |
| *** aarefiev_afk is now known as aarefiev | 10:26 | |
| *** mdbooth has quit IRC | 10:26 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 10:26 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | 10:26 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 10:26 |
| *** mdbooth has joined #openstack-nova | 10:26 | |
| *** ansiwen_ has joined #openstack-nova | 10:27 | |
| *** abalutoiu has joined #openstack-nova | 10:27 | |
| *** ansiwen has quit IRC | 10:27 | |
| *** gongysh has joined #openstack-nova | 10:28 | |
| *** amotoki_ has quit IRC | 10:28 | |
| *** namnh has quit IRC | 10:29 | |
| *** owalsh has quit IRC | 10:30 | |
| *** gcb has quit IRC | 10:34 | |
| *** tiendc has quit IRC | 10:34 | |
| *** tovin07_ has quit IRC | 10:36 | |
| *** karimb has quit IRC | 10:36 | |
| *** gongysh has quit IRC | 10:37 | |
| *** owalsh has joined #openstack-nova | 10:38 | |
| *** amotoki has joined #openstack-nova | 10:38 | |
| *** phuongnh has quit IRC | 10:40 | |
| *** Jeffrey4l has joined #openstack-nova | 10:40 | |
| *** karimb has joined #openstack-nova | 10:42 | |
| *** avolkov has joined #openstack-nova | 10:47 | |
| *** sandanar_ has joined #openstack-nova | 10:47 | |
| bauzas | alex_xu: around ? | 10:49 |
| bauzas | alex_xu: I was working on the scheduler change when I saw that we haven't yet a fixture for calling the placement api | 10:50 |
| *** sandanar has quit IRC | 10:50 | |
| bauzas | so some of the functional tests are failing because we return NoValidHosts | 10:50 |
| bauzas | alex_xu: I was thinking of very quickly writing a fixture, but given we don't run the WSGI app for placement the same than OSAPI, then I'm a bit sad | 10:51 |
| bauzas | alex_xu: do you know if there is some way to hack that by having a WSGI server fixture able to deploy a WSGI app ? | 10:52 |
| bauzas | and sean is off, I can't ask advice from him | 10:52 |
| sahid | jaypipes: hello, i lost your +2 on https://review.openstack.org/#/c/400874/ because of a conflict if I can get it again that would be nice since next week (if i remember correclty) you will be out | 10:53 |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(4): Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | 10:54 |
| *** rfolco has joined #openstack-nova | 10:59 | |
| lyarwood | Morning all, if anyone has time I'd appreciate some additional reviews on the following findmnt bugfix - https://review.openstack.org/#/c/420146/ | 11:02 |
| *** tbachman has joined #openstack-nova | 11:04 | |
| *** zz_dimtruck is now known as dimtruck | 11:04 | |
| *** salv-orlando has joined #openstack-nova | 11:06 | |
| *** dimtruck is now known as zz_dimtruck | 11:14 | |
| *** huanxie has quit IRC | 11:23 | |
| *** salv-orlando has quit IRC | 11:24 | |
| *** sdague has joined #openstack-nova | 11:29 | |
| *** ducttape_ has joined #openstack-nova | 11:29 | |
| *** tbachman has quit IRC | 11:30 | |
| *** ducttape_ has quit IRC | 11:31 | |
| *** tbachman has joined #openstack-nova | 11:31 | |
| *** nicolasbock has joined #openstack-nova | 11:35 | |
| *** liuyulong has quit IRC | 11:35 | |
| *** gszasz has joined #openstack-nova | 11:35 | |
| *** tbachman has quit IRC | 11:41 | |
| bauzas | sdague: I think I come up with a potential problem for us wanting to get the placement upcall from the scheduler | 11:42 |
| bauzas | sdague: we haven't yet a fixture for running the placement API in our functional tests | 11:43 |
| bauzas | sdague: thoughts on the best approach ? | 11:43 |
| bauzas | sdague: I began to look at to embed the WSGI app into a fixture | 11:43 |
| bauzas | to look at how* | 11:43 |
| bauzas | since we can't reuse the OSAPIFixture, as it's a totally different structure AFAICS | 11:44 |
| *** amotoki has quit IRC | 11:44 | |
| bauzas | sdague: maybe using paste.fixture ? | 11:46 |
| *** thorst_ has joined #openstack-nova | 11:47 | |
| *** thorst_ has quit IRC | 11:52 | |
| sdague | bauzas: are, right, no oslo.service | 11:57 |
| sdague | so... my suggestion, spawn a thread and run the app inside https://docs.python.org/2/library/wsgiref.html#module-wsgiref.simple_server | 11:58 |
| bauzas | not sure I understood your first point ? | 11:58 |
| bauzas | "no oslo service " | 11:58 |
| bauzas | you mean that indeed the placement service is not oslo's | 11:59 |
| sdague | placement doesn't launch through oslo.service | 11:59 |
| bauzas | yeah | 11:59 |
| sdague | oslo.service gave us the ability to just run in a greenlet | 11:59 |
| bauzas | okay, so you'd recommend wsgiref instead of paste.fixture ? | 11:59 |
| *** pcaruana has quit IRC | 11:59 | |
| sdague | well, paste doesn't actually listen | 11:59 |
| bauzas | okay, disclaimer, I'm not a super expert on that | 12:00 |
| bauzas | so okay, using wsgiref | 12:00 |
| bauzas | given the short time we have, I'm rushing | 12:00 |
| bauzas | sdague: I'll just use a thread in the fixture and do like the above you mentioned ^ | 12:00 |
| bauzas | sdague: thanks for the very quick answer | 12:00 |
| sdague | bauzas: it's not the only thing you are going to need to do though, you potentially need to simulate the keystone catalog as well | 12:01 |
| sdague | because the placement api is only looked up by catalog | 12:01 |
| bauzas | sdague: I was thinking to noauth | 12:01 |
| bauzas | and mock the keystone calls | 12:01 |
| sdague | bauzas: yeh, that's possible | 12:01 |
| *** hfu has joined #openstack-nova | 12:01 | |
| sdague | it would be better long term to avoid mock there, but given the timing that should get you there fast | 12:02 |
| bauzas | dunno, the quickiest one given the short time | 12:02 |
| bauzas | yeah | 12:02 |
| sdague | I think this biggest trick of wsgiref is going to be picking ports | 12:02 |
| sdague | eventlet can bind an unused port and give it back to you | 12:02 |
| sdague | which is what we use | 12:02 |
| bauzas | but I have to be explicit for wsgiref? gotcha | 12:02 |
| sdague | yeh | 12:03 |
| bauzas | let's try to do something at least | 12:03 |
| bauzas | sdague: from a upper standpoint, our functional tests were working because we still update the DB | 12:04 |
| bauzas | sdague: which is unfortunate, I thought we were already mocking that placement upcall from the compute | 12:04 |
| bauzas | that's why we're at risk now | 12:04 |
| bauzas | I haven't identified it would require a new fixture | 12:05 |
| *** hfu has quit IRC | 12:05 | |
| *** zz_dimtruck is now known as dimtruck | 12:05 | |
| sdague | bauzas: actually, have you tried just launching the placement app in nova.wsgi.Server? | 12:05 |
| bauzas | sdague: I was tempted to | 12:05 |
| *** ociuhandu has quit IRC | 12:05 | |
| sdague | because, yes, that uses eventlet to do it | 12:06 |
| sdague | but ... it's not really that eventlet specific | 12:06 |
| *** pcaruana has joined #openstack-nova | 12:06 | |
| sdague | as I'm looking through the code | 12:06 |
| *** archies has quit IRC | 12:06 | |
| sdague | I think the only tricky part is the paste load is a little different | 12:07 |
| sdague | but, you might just be able to do that | 12:07 |
| bauzas | lemme write that | 12:07 |
| bauzas | my fixture is there | 12:07 |
| bauzas | I just need to update it | 12:07 |
| bauzas | and my change is a canary patch so I can see | 12:08 |
| *** cdelatte has joined #openstack-nova | 12:08 | |
| *** adisky_ has quit IRC | 12:09 | |
| *** edmondsw has joined #openstack-nova | 12:13 | |
| *** dimtruck is now known as zz_dimtruck | 12:15 | |
| *** karimb has quit IRC | 12:18 | |
| bauzas | sdague: that looks working | 12:19 |
| bauzas | using wsgi.Server() | 12:19 |
| bauzas | I now need to fake the keystone calls | 12:19 |
| bauzas | and verify a few tings | 12:19 |
| *** bvanhav_ has joined #openstack-nova | 12:20 | |
| *** moshele has joined #openstack-nova | 12:21 | |
| *** smatzek_ has joined #openstack-nova | 12:24 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make coherent logs when reboot success https://review.openstack.org/416994 | 12:25 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: rework _destroy method https://review.openstack.org/416995 | 12:25 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: rework hard reboot https://review.openstack.org/376350 | 12:25 |
| snikitin | BobBall: Hi, Bob! JFYI: I found the reason of diagnostics tempest tests fail. Sometimes (I don't know why) xen returns "NaN" instead of numbers. For example I saw such response in Xen CI job: {"last_update": "1484900660", "cpu0": "NaN", "memory": "NaN"}. | 12:30 |
| snikitin | My code can't convert float('NaN') into int and this is the reason of the fails. On my local Xen env everything is ok. I saw 'NaN' values only one time and I don't know how to reproduce it. But on CI I saw normal values only one time. All other times I see 'NaN'. | 12:30 |
| snikitin | I guess I need to add 'is NaN' checks into my code. Did you face with such strange behavior? | 12:30 |
| BobBall | I suspect this will be because we're getting the stats from the RRDs which are updated every 5 seconds. Perhaps a pause in the tempest test for 10 seconds(?) to ensure some stats are populated would fix it? | 12:32 |
| *** catinthe_ has joined #openstack-nova | 12:32 | |
| *** catintheroof has quit IRC | 12:33 | |
| snikitin | BobBall: Thank you for advice! I will check it. P.S. Even if some stats aren't populated I expect to see something like 'None' or 'null' rather than 'NaN'. strange behavior. | 12:35 |
| *** slagle_ is now known as slagle | 12:37 | |
| *** lucasagomes is now known as lucas-hungry | 12:37 | |
| BobBall | snikitin: The RRDs are typically based on min, max or averages over a period (as an instant reading doesn't make much sense for things like CPU usage). My guess is that they are providing a number based on the average over X-X seconds - which will be a number divided by 0 as we don't have sufficient time periods. I get what you're saying, but if a 10 second delay fixes the issue I would understand why it is returning "Not a Number" | 12:38 |
| *** ayogi has quit IRC | 12:39 | |
| bauzas | sdague: quick question, we deduce the endpoint from the token unless we use the endpoint_override here https://github.com/openstack/keystoneauth/blob/master/keystoneauth1/session.py#L509-L513 | 12:41 |
| *** ratailor has quit IRC | 12:42 | |
| bauzas | sdague: am I correct if I say that if I'm providing endpoint_override, I'd not lookup the SC ? | 12:42 |
| bauzas | and directly call the placement service | 12:42 |
| sdague | bauzas: looks like... however | 12:42 |
| sdague | if you are really using noauth | 12:43 |
| sdague | then probably you are actually going to want to build a fake client entirely, I think mocking out keystoneauth at all the right points is going to be harder | 12:44 |
| sean-k-mooney | dims: jaypipes looking at https://review.openstack.org/#/c/422838 do we still need to merge this and make a release to fix the python 3.5 support today? | 12:44 |
| bauzas | I don't want to mock out all the keystoneclient | 12:45 |
| bauzas | sdague: just recreating a new Session with an endpoint_override | 12:45 |
| snikitin | BobBall: Ok. I will check it, but I think you right. Anyway I think we should check 'NaN' values in nova, because user can get a 500 error because of it. | 12:45 |
| sdague | bauzas: ok | 12:45 |
| bauzas | anyway, let's try | 12:45 |
| bauzas | just taking a few mins for refueling myself at $lunch, and then back quickly | 12:46 |
| BobBall | snikitin: Sure - the xenapi drive could easily translate NaN to None if it makes more sense up the stack | 12:46 |
| *** jerrygb_ has joined #openstack-nova | 12:46 | |
| *** thorst_ has joined #openstack-nova | 12:47 | |
| *** sofie has quit IRC | 12:48 | |
| *** voelzmo has quit IRC | 12:48 | |
| *** tbachman has joined #openstack-nova | 12:49 | |
| *** jerrygb has quit IRC | 12:49 | |
| *** voelzmo has joined #openstack-nova | 12:50 | |
| *** hfu has joined #openstack-nova | 12:50 | |
| *** ociuhandu has joined #openstack-nova | 12:50 | |
| snikitin | BobBall: I checked. It works with sleep(10)! Thank you! | 12:51 |
| *** JoseMello has joined #openstack-nova | 12:52 | |
| dims | sean-k-mooney : nah. no need to rush it. py35 is just experiment anyway at the moment | 12:53 |
| *** hfu has quit IRC | 12:53 | |
| markus_z | sdague: We could make the testing gap for live-migration a little smaller with https://review.openstack.org/#/c/347471/ Would be nice if you could find the time for that. | 12:54 |
| *** satyar has joined #openstack-nova | 12:54 | |
| markus_z | mriedem: Maybe you too, I answered your questions there ^ | 12:54 |
| sean-k-mooney | dims: ok ill try and get a linux bridge devstack enviornment up and running next week and we can also look into adding a linux bridge job to os-vif also going forward | 12:55 |
| dims | ++ sean-k-mooney thansk! | 12:55 |
| *** links has quit IRC | 12:55 | |
| *** amotoki has joined #openstack-nova | 12:56 | |
| *** hfu has joined #openstack-nova | 12:56 | |
| *** hfu has quit IRC | 12:58 | |
| *** hfu has joined #openstack-nova | 12:59 | |
| *** Jeffrey4l_ has joined #openstack-nova | 13:00 | |
| *** hfu has quit IRC | 13:01 | |
| *** hfu has joined #openstack-nova | 13:03 | |
| *** Jeffrey4l has quit IRC | 13:04 | |
| *** hfu has quit IRC | 13:04 | |
| *** zz_dimtruck is now known as dimtruck | 13:06 | |
| jaypipes | dims: what's the latest status on the mox3 thing? has that revert merged? | 13:08 |
| *** liverpooler has joined #openstack-nova | 13:08 | |
| dims | jaypipes : y, should be green. please rebase if you still see an issue | 13:09 |
| jaypipes | dims: will do. cheers! | 13:09 |
| *** ducttape_ has joined #openstack-nova | 13:11 | |
| *** mfeoktistov has joined #openstack-nova | 13:12 | |
| *** tblakes has joined #openstack-nova | 13:13 | |
| *** dimtruck is now known as zz_dimtruck | 13:15 | |
| *** sandanar__ has joined #openstack-nova | 13:16 | |
| *** gcb has joined #openstack-nova | 13:19 | |
| *** jaypipes is now known as leakypipes | 13:19 | |
| *** sandanar_ has quit IRC | 13:20 | |
| *** hfu has joined #openstack-nova | 13:21 | |
| *** ducttape_ has quit IRC | 13:21 | |
| *** clenimar has quit IRC | 13:23 | |
| *** ducttape_ has joined #openstack-nova | 13:24 | |
| *** jheroux has joined #openstack-nova | 13:25 | |
| mriedem | getting this resource providers patch in, which is pretty simple, closes out the aggregates stuff for ocata https://review.openstack.org/#/c/407309/ | 13:25 |
| mriedem | needs another core ^ | 13:26 |
| leakypipes | mriedem: I'm on it, captain. | 13:26 |
| leakypipes | oh wait, that's mine. | 13:26 |
| mriedem | leakypipes: it's yours | 13:26 |
| leakypipes | never mind :) | 13:26 |
| mriedem | ha | 13:26 |
| *** mdnadeem has quit IRC | 13:27 | |
| mriedem | leakypipes: you could deal with the -1 on this though https://review.openstack.org/#/c/404472/ | 13:27 |
| leakypipes | mriedem: ya, will do. | 13:27 |
| leakypipes | mriedem: just trying to get sahid's reserved CPUs patches reviewed. | 13:28 |
| mriedem | melwitt_: sdague: i got the discover_hosts running in d-g http://logs.openstack.org/05/423005/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/2704d02/logs/discover-hosts.txt.gz | 13:28 |
| mriedem | needs to be fixed up for stable branches though | 13:28 |
| mriedem | and grenade | 13:28 |
| mriedem | http://logs.openstack.org/05/423005/2/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/d230b29/logs/discover-hosts.txt.gz | 13:29 |
| openstackgerrit | Timofey Durakov proposed openstack/nova: Make all timestamps formats equal https://review.openstack.org/421981 | 13:29 |
| * tbachman wonders if today calls for extra-special Friday NICs | 13:29 | |
| *** voelzmo has quit IRC | 13:30 | |
| tbachman | a little levity | 13:32 |
| tbachman | is | 13:32 |
| tbachman | needed | 13:32 |
| *** hfu has quit IRC | 13:32 | |
| leakypipes | tbachman: indeed :( | 13:33 |
| *** voelzmo has joined #openstack-nova | 13:33 | |
| *** ducttape_ has quit IRC | 13:34 | |
| mfeoktistov | Hello! Dear cores, can you please look at https://review.openstack.org/#/c/312488/ | 13:34 |
| *** diga has quit IRC | 13:34 | |
| *** tbachman is now known as IAMostrich | 13:36 | |
| * IAMostrich goes to his hole | 13:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 13:36 |
| sdague | mriedem: cool | 13:37 |
| *** jpena is now known as jpena|lunch | 13:37 | |
| *** bradjones has joined #openstack-nova | 13:38 | |
| *** bradjones has quit IRC | 13:38 | |
| *** bradjones has joined #openstack-nova | 13:38 | |
| *** kevinz has joined #openstack-nova | 13:39 | |
| sdague | mriedem: once I get some breakfast and this set of grenade results, I'll take a look | 13:39 |
| mriedem | the d-g change isn't really ready yet, i need to make it conditional on branches, or put a script in devstack to run discover_hosts and then d-g would only run it if that script exists | 13:40 |
| mriedem | https://review.openstack.org/#/c/423005/2/devstack-vm-gate.sh | 13:40 |
| mriedem | if you have input on what's the preferred approach there that would be good | 13:41 |
| *** IAMostrich is now known as tbachman | 13:42 | |
| leakypipes | tbachman: :) | 13:42 |
| *** voelzmo has quit IRC | 13:42 | |
| *** baoli has joined #openstack-nova | 13:42 | |
| *** baoli has quit IRC | 13:43 | |
| mdbooth | mriedem: NFS https://review.openstack.org/#/c/383859/ | 13:43 |
| *** baoli has joined #openstack-nova | 13:43 | |
| mriedem | blarg shared file systems | 13:43 |
| mdbooth | I've actually been hammering on this over the last couple of days because it fixes another bug too | 13:44 |
| *** prateek has quit IRC | 13:44 | |
| mdbooth | If you suspend an instance with an attached volume, the volume can be unmounted while it's suspended | 13:44 |
| mdbooth | So it fails when you resume it | 13:44 |
| mdbooth | Confirmed this patch fixes that bug | 13:44 |
| * mdbooth should ensure there's an upstream bug for that, actually... | 13:45 | |
| *** moshele has quit IRC | 13:45 | |
| mdbooth | https://review.openstack.org/#/c/383859/ | 13:45 |
| * mdbooth will add that to the commit message | 13:45 | |
| mdbooth | Except it's not a race, it's entirely deterministic in this case | 13:46 |
| mriedem | ok | 13:47 |
| mriedem | i'll be honest i'm going to have a very distracted day today, trying to wrap things up before flying out for vacation tomorrow | 13:47 |
| BobBall | snikitin: Glad to hear it | 13:48 |
| mdbooth | Ah, it's already in there | 13:48 |
| *** lucas-hungry is now known as lucasagomes | 13:48 | |
| *** kevinz has quit IRC | 13:48 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494 | 13:49 |
| *** kevinz has joined #openstack-nova | 13:49 | |
| mdbooth | Unrelated, can anybody check my memory for me: https://review.openstack.org/#/c/376350/ . The behaviour of reboot where is tries to fix potential local hypervisor problems is intentional, right? | 13:50 |
| *** kevinz has quit IRC | 13:51 | |
| mdbooth | i.e. recreating backing files and rewriting the domain xml. | 13:51 |
| mdbooth | Both of these behaviours are undesirable, imho, but iiuc they're relied upon by operators. Is that right? | 13:51 |
| mdbooth | sdague: ^^^ | 13:52 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 13:52 |
| bauzas | mriedem: FYI, the scheduler change is blocked because also another problem due to us not having yet a fixture for our functional tests | 13:52 |
| bauzas | mriedem: sdague: there is a Gabbi fixture in our gabbi branch, but not for the functional tests | 13:52 |
| * bauzas tableflips | 13:53 | |
| bauzas | sdague: any reason why when calling the placement API using urllib2 and noauth2 middleware, it still returns 401 ? | 13:53 |
| *** tbachman has quit IRC | 13:54 | |
| mdbooth | mriedem: I ran the latest patch version against the devstack nfs CI, btw. It passed. | 13:55 |
| johnthetubaguy | melwitt_: mriedem: I have been reading that quota stuff just now, any reason we can't change ProjectUserQuota to have user_id column as nullable, and just kill the Quota table? (or add a nullable user_id into the Quota table, and kill ProjectUserQuota)? | 13:55 |
| mriedem | johnthetubaguy: i don't know off the top of my head, i know the tables are used based on resource type | 13:56 |
| mriedem | i don't know how much of that is still valid after the move | 13:56 |
| *** tbachman has joined #openstack-nova | 13:57 | |
| *** salv-orlando has joined #openstack-nova | 13:57 | |
| johnthetubaguy | mriedem: no worries, I am going to keep reading | 13:57 |
| mriedem | i would assume things stay the same in those tables when we move to the api db, they might change their function or necessity after the counting quotas stuff is in, but i'd probably lean toward making any changes to the data model in a later release | 13:58 |
| *** edleafe is now known as figleaf | 14:00 | |
| *** bvanhav__ has joined #openstack-nova | 14:01 | |
| *** bvanhav_ has quit IRC | 14:01 | |
| *** dataman3 has joined #openstack-nova | 14:02 | |
| *** dataman3 has left #openstack-nova | 14:02 | |
| sahid | mdbooth: about https://review.openstack.org/#/c/376350/ - i do not understand your points | 14:03 |
| sahid | if there the domain XML can be caorrupted in someways to we probzbly have to fix the root cause | 14:03 |
| sahid | same for the disks of instance | 14:03 |
| *** hamzy has quit IRC | 14:04 | |
| *** Dinesh_Bhor has quit IRC | 14:04 | |
| *** hamzy has joined #openstack-nova | 14:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 14:05 |
| mdbooth | sahid: I've posted a hopefully clearer response on the review. | 14:05 |
| sahid | if you can explaining or at least report a bug so i could fix both of these issues but like that it's just you want something broken to stay broken | 14:05 |
| mdbooth | Specific bugs misses the point. | 14:06 |
| *** dgonzalez has quit IRC | 14:06 | |
| *** zz_dimtruck is now known as dimtruck | 14:06 | |
| *** hfu has joined #openstack-nova | 14:06 | |
| *** esberglu has joined #openstack-nova | 14:06 | |
| mdbooth | sahid: I was also unaware of this apparently undocumented design decision, which is why I'm hoping to get some additional input. | 14:07 |
| sahid | yes because that seems weird the hard-reboot should be helpfull when the guest operating system get stuck | 14:07 |
| *** dave-mccowan has joined #openstack-nova | 14:07 | |
| mdbooth | It's about the hypervisor rather than the guest OS, I think. | 14:08 |
| mdbooth | Well, I guess it's about either. | 14:08 |
| sahid | but i do not think it's how operators should repair an instance "broken" | 14:08 |
| mdbooth | It's like: steps to take before defenestrating the instance: | 14:09 |
| mdbooth | 1. reboot | 14:09 |
| mdbooth | 2. ... | 14:09 |
| *** stvnoyes has quit IRC | 14:09 | |
| sahid | but we need to be clear on what we are talking about, the qemu process or the guest operating system ? | 14:09 |
| mdbooth | s/qemu process/hypervisor/ | 14:09 |
| mdbooth | The aspect of reboot I'm talking about relates to the hypervisor | 14:10 |
| *** stvnoyes has joined #openstack-nova | 14:10 | |
| mdbooth | Not the guest OS | 14:10 |
| *** andymaier has joined #openstack-nova | 14:10 | |
| sahid | that seems really sad so... we really want to fix an issue like that | 14:10 |
| mdbooth | What issue? | 14:10 |
| sahid | the one you are talking about, no ? | 14:10 |
| mdbooth | I'm not talking about an issue | 14:10 |
| mdbooth | I'm talking about *any* issue | 14:11 |
| mdbooth | I'm talking about an imaginary, as yet undiscovered bug which has resulted in inconsistent state in the hypervisor | 14:11 |
| *** dgonzalez has joined #openstack-nova | 14:11 | |
| mdbooth | One of the goals of reboot is to try to get the instance going again anyway | 14:11 |
| mdbooth | It would be nice if it wasn't | 14:12 |
| mdbooth | But to the best of my knowledge, it is | 14:12 |
| *** voelzmo has joined #openstack-nova | 14:13 | |
| *** mdrabe has joined #openstack-nova | 14:13 | |
| sahid | ok so you don't want me to fix this because there is an imaginary/undiscovered bug | 14:15 |
| mdbooth | Incidentally, as I mentioned in one of my review comments, I think this is a very useful tool for operators to have. However, I also think it's problematic and ironically a potential source of bugs to do it in reboot. | 14:16 |
| sahid | interesting... i have 2 well real bugs to show you because of this code | 14:16 |
| mdbooth | sahid: No, I don't want you to do this because (IIUC) operators rely on this behaviour. | 14:16 |
| *** cleong has joined #openstack-nova | 14:16 | |
| mdbooth | Ideally we would separate it from reboot. | 14:16 |
| sahid | if they do, so we should explain to them they should not i guess | 14:17 |
| sahid | rscue and rebuild are better | 14:17 |
| mdbooth | However, if it's a deliberate design decision, and something that users expect to be there, we can't just take it away without thought or acknowledgement. | 14:17 |
| mdbooth | No, rescue doesn't help at all, and rebuild is entirely different. | 14:17 |
| sahid | because hard-reboot has been designed such issue ? | 14:17 |
| mdbooth | sahid: I suggest a bp. You might also post on the operators mailing list asking for input. | 14:18 |
| mdbooth | It would also need a release note, I guess. | 14:19 |
| sdague | bauzas: are you sure that placement is loading the noauth middleware? | 14:19 |
| bauzas | sdague: yup | 14:19 |
| bauzas | sdague: I just verified | 14:19 |
| sdague | bauzas: if you push a patch I can look | 14:19 |
| bauzas | sdague: okay, I'll do | 14:20 |
| bauzas | mmm, I think I found the issue | 14:20 |
| *** gouthamr has joined #openstack-nova | 14:22 | |
| *** sacharya has quit IRC | 14:23 | |
| *** weshay_afk is now known as weshay | 14:23 | |
| *** sacharya_ has joined #openstack-nova | 14:23 | |
| sahid | mdbooth: well if that could help i can provide a release note or a bp to fix this issue even if i'm not bringing any feature here | 14:23 |
| *** tlian has joined #openstack-nova | 14:23 | |
| mdbooth | sahid: I just want you to acknowledge and address a change in operator-expected behaviour. | 14:24 |
| sahid | mdbooth: you are saying that but i never hear about this point, can you share to me some pointer ? | 14:24 |
| sahid | a comment in lunchpad, a gerrit review or a thread in the ml ? | 14:24 |
| *** sacharya_ has quit IRC | 14:24 | |
| mdbooth | No, I really can't. I've never seen it documented. | 14:24 |
| sdague | mdbooth: I don't remember why the domain xml is rewriten, git log -S might find some things interesting to figure out if the relevant commit messages have clues | 14:25 |
| *** sacharya has joined #openstack-nova | 14:25 | |
| *** woodster_ has joined #openstack-nova | 14:27 | |
| *** sacharya has quit IRC | 14:30 | |
| *** hfu has quit IRC | 14:30 | |
| *** smatzek_ has quit IRC | 14:31 | |
| *** bvanhav__ is now known as bvanhav | 14:33 | |
| *** catintheroof has joined #openstack-nova | 14:33 | |
| *** nkrinner is now known as nkrinner_afk | 14:35 | |
| *** catinthe_ has quit IRC | 14:36 | |
| bauzas | sdague: okay, so found the problem | 14:37 |
| bauzas | sdague: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/auth.py#L43-L44 | 14:37 |
| bauzas | sdague: I just need to fake a token | 14:37 |
| bauzas | sdague: do you know if we have a fake token lib or helper ? :) | 14:38 |
| *** pbandark has quit IRC | 14:38 | |
| sdague | bauzas: https://github.com/openstack/nova/blob/9acfef2c90186644120942558437b3fcc90222de/nova/api/openstack/auth.py#L32-L47 | 14:39 |
| sdague | it looks like that bit wasn't copied over | 14:39 |
| bauzas | sdague: yeah, possibly because gabbi provides a fake token | 14:40 |
| sdague | bauzas: could be | 14:40 |
| bauzas | we miss cdent here | 14:40 |
| *** liusheng has quit IRC | 14:40 | |
| sdague | honestly, I have no idea about the gabbi magic | 14:40 |
| bauzas | me too | 14:40 |
| *** liusheng has joined #openstack-nova | 14:40 | |
| bauzas | but since cdent wrote that noauth middleware for the gabbi functests, I guess it's why | 14:40 |
| bauzas | I could just fake a token honestly and leave a big note for saying that we should rather have a better noauth middleware | 14:41 |
| *** amoralej is now known as amoralej|lunch | 14:41 | |
| leakypipes | sdague, bauzas: there's no magic in gabbi... at least, certainly no more magic than the existing functional API tests. | 14:42 |
| *** psachin has quit IRC | 14:42 | |
| bauzas | leakypipes: well the problem with gabbi is that we haven't yet a correct fixture | 14:42 |
| sdague | yeh, the noauthmiddleware is actually very gabbi specific | 14:43 |
| sdague | bauzas: you might be able to replumb the existing noauth bits in there instead | 14:43 |
| bauzas | and the current APIFixture we have in-tree is also a gabbi child | 14:43 |
| *** eharney has joined #openstack-nova | 14:43 | |
| bauzas | sdague: maybe | 14:43 |
| bauzas | sdague: I just don't want to have a long patch to be reviewed | 14:44 |
| sdague | bauzas: well, a long working patch is better than a non working one | 14:44 |
| *** dimtruck is now known as zz_dimtruck | 14:45 | |
| *** zz_dimtruck is now known as dimtruck | 14:45 | |
| bauzas | sdague: leakypipes: bingo https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/gabbits/aggregate.yaml#L7 | 14:45 |
| leakypipes | sdague, bauzas: guys... | 14:45 |
| *** mjura has quit IRC | 14:46 | |
| leakypipes | sdague, bauzas: what is it you are trying to do? | 14:46 |
| bauzas | leakypipes: because of that, we could be missing the placement upcall from the scheduler for Ocata... | 14:46 |
| bauzas | just because we wanted to have a very separate WSGI app than the existing one | 14:46 |
| bauzas | and using gabbi too | 14:46 |
| leakypipes | bauzas: I'm not following you. How does a functional test fixture that disables Keystone auth impact the "scheduler upcall"? | 14:46 |
| sdague | leakypipes: there is a need to get placement running inside the functional tests that aren't gabbi so you can have the whole working service collection | 14:46 |
| bauzas | leakypipes: see my scheduler change and the functional tests job | 14:47 |
| bauzas | we're getting NoValidHosts because the placement service isn't running | 14:47 |
| *** sheel has quit IRC | 14:47 | |
| bauzas | so I'm working since this morning on adding a fixture | 14:47 |
| leakypipes | bauzas: and how is that related to gabbi? | 14:47 |
| bauzas | leakypipes: because we have an existing fixture that is only gabbi'd | 14:48 |
| mriedem | so let's ignore gabbi :) | 14:48 |
| leakypipes | bauzas: I have no idea what that means. | 14:48 |
| mriedem | the point is, | 14:48 |
| bauzas | and when we added the placement service in Newton, we haven't verified that we could be adding a fixture for the existing tests | 14:48 |
| bauzas | anyway | 14:48 |
| mriedem | we need to run the placement service with a fixture, like we do for other services | 14:48 |
| mriedem | like n-cond, n-sch, n-cpu, etc in the functional tets | 14:48 |
| mriedem | *tests | 14:48 |
| bauzas | I don't want to diverge too moch | 14:48 |
| sdague | leakypipes: there exists no infrastructure in nova functional tests to start a running placement service outside of gabbi tests | 14:48 |
| mriedem | those rely on a noauth fixture | 14:48 |
| mriedem | since we don't have keystone | 14:48 |
| bauzas | given the very short deadline | 14:48 |
| bauzas | so folks, I'll be providing a WIP by the next 10 mins | 14:49 |
| mriedem | we have an api client fixture for n-api for the functional tests, seems we'd want something similar for placement, | 14:49 |
| *** jpena|lunch is now known as jpena | 14:49 | |
| mriedem | or simply have a PlacementFixture which isn't a real service | 14:49 |
| sdague | mriedem: it's a little different than that because of the way things glue together | 14:49 |
| sdague | but, yeh, bauzas is working through it | 14:50 |
| mriedem | because when placement is split out, we'll need to just stub it anyway | 14:50 |
| mriedem | like neutron | 14:50 |
| bauzas | sdague: mriedem: lemme upload it | 14:50 |
| leakypipes | bauzas: you're saying you need one of these for the placement API service, yes? https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L638 | 14:51 |
| *** clenimar has joined #openstack-nova | 14:51 | |
| bauzas | leakypipes: I'm just adding it now since this morning | 14:51 |
| bauzas | leakypipes: lemme upload it in the next 10 mins | 14:51 |
| leakypipes | bauzas: ok. I just get annoyed with the kvetching about gabbi and "a separate WSGI service" when really this isn't that hard of a problem to solve. | 14:52 |
| *** burt has joined #openstack-nova | 14:52 | |
| bauzas | leakypipes: it took me 1 hour for finding a solution because the WSGI placement service is not an oslo.service | 14:52 |
| leakypipes | bauzas: gabbi tests a REST API interface. it doesn't test integration between two services. | 14:52 |
| bauzas | fortunately, sdague gave me a good solution | 14:53 |
| *** ducttape_ has joined #openstack-nova | 14:53 | |
| sdague | leakypipes: which we all understand, it's that we also test more than just that, and it wasn't anticipated. And because this was plumbed differently it takes more effort to plumb something in. | 14:54 |
| *** dimtruck is now known as zz_dimtruck | 14:54 | |
| *** ratailor has joined #openstack-nova | 14:55 | |
| *** melwitt_ is now known as melwitt | 14:55 | |
| *** karimb has joined #openstack-nova | 14:56 | |
| *** smatzek has joined #openstack-nova | 14:58 | |
| *** dikonoor has quit IRC | 14:58 | |
| *** READ10 has joined #openstack-nova | 14:58 | |
| sdague | oh, man, so happy to have gotten the anchor issue fixed in os-api-ref | 14:58 |
| sdague | links should be a ton more useful in sharing now | 14:58 |
| sdague | http://developer.openstack.org/api-ref/compute/?expanded=pause-server-pause-action-detail#pause-server-pause-action - for instance | 14:59 |
| melwitt | johnthetubaguy: that's an excellent question. I should have asked you to review earlier :) what mriedem said is right, it was separated based on resource type (fixed_ips, networks, floating_ips in Quotas, everything else in ProjectUserQuota). I'll take a closer look and think about it | 14:59 |
| johnthetubaguy | melwitt: oh, I thought everything could be in both, depending on the resource type, its not as bad if we just get to kill a table later | 15:01 |
| johnthetubaguy | melwitt: but reading more, I am leaning towards the direct swap being simpler | 15:01 |
| melwitt | johnthetubaguy: I think you're probably right. just not totally sure off the top of my head. and yes, also true that at worst (if we don't change it now) that it would be a table kill later | 15:01 |
| *** ducttape_ has quit IRC | 15:02 | |
| mriedem | sdague: heh, for cyborgs such as yourself that actualy read the anchor :) | 15:03 |
| sdague | mriedem: well, the point is that when you click the link button on the side, it updates the address bar correctly now | 15:03 |
| sdague | so when people copy and paste links around, you can actually go to the right place in the page | 15:04 |
| johnthetubaguy | sdague: thats quite lovely | 15:04 |
| sdague | instead of having to search to find it | 15:04 |
| *** jwcroppe has joined #openstack-nova | 15:06 | |
| mriedem | i just got used to right-clicking to copy the link | 15:06 |
| mriedem | these are all creature comforts | 15:07 |
| sdague | mriedem: well, now it just works like it should :) | 15:07 |
| *** gcb has quit IRC | 15:09 | |
| *** mtanino has joined #openstack-nova | 15:10 | |
| openstackgerrit | Wang Qing wu proposed openstack/nova: WIP(5): Introduce base PowerVM driver (ovs vif) https://review.openstack.org/422512 | 15:10 |
| melwitt | johnthetubaguy: what did you mean by "direct swap?" changing it now or later? one thing I just thought of is, after we remove nova-network, we'll also need to remove the quota limits for networks, fixed_ips, and floating_ips and maybe that would be simpler if they're all in the same table | 15:12 |
| *** tbachman has quit IRC | 15:13 | |
| *** david-lyle has joined #openstack-nova | 15:13 | |
| *** amoralej|lunch is now known as amoralej | 15:13 | |
| johnthetubaguy | melwitt: direct swap: I meant the DB methods matching so closely, is really handy | 15:14 |
| johnthetubaguy | melwitt: yeah, if they are isolated in that way, its worth keeping | 15:15 |
| melwitt | oh, I see | 15:15 |
| johnthetubaguy | I totaly garbled that previous message, my bad | 15:15 |
| johnthetubaguy | melwitt: still reading through the series for proper context, it feels close on those bottom patches | 15:15 |
| *** kristian__ has joined #openstack-nova | 15:16 | |
| mriedem | melwitt: we're not dropping nova-network in ocata, so anything hinging on that should be deferred | 15:17 |
| *** zz_dimtruck is now known as dimtruck | 15:18 | |
| melwitt | johnthetubaguy: cool. on the online migration patch I realized I should add a catch of DBDuplicateEntry for the limit create in case of someone interrupting it after limits migrated to api db but before limits deleted from main db | 15:21 |
| *** udesale has quit IRC | 15:21 | |
| *** esberglu has quit IRC | 15:21 | |
| johnthetubaguy | melwitt: ah, cools, not really got through that one yet | 15:21 |
| *** udesale has joined #openstack-nova | 15:21 | |
| *** esberglu has joined #openstack-nova | 15:22 | |
| claudiub|2 | mriedem: hi. any chance for the hyper-v pci stuff? i've posted a comment with a sample instance + pci spawn and proof that it worked (+ logs, flavors, .conf files): https://review.openstack.org/#/c/420614/ | 15:22 |
| melwitt | limits are migrated in one transaction but the delete is a separate one, separate db. there's no dupe detection possible for the quota classes. I should probably do something like that manually maybe | 15:22 |
| mriedem | claudiub|2: would be good to get moshele, lbeliveau and sfinucan to take a look at that | 15:22 |
| *** ayogi has joined #openstack-nova | 15:24 | |
| *** udesale has quit IRC | 15:24 | |
| *** esberglu_ has joined #openstack-nova | 15:24 | |
| *** amotoki has quit IRC | 15:24 | |
| *** udesale has joined #openstack-nova | 15:25 | |
| *** david-lyle has quit IRC | 15:25 | |
| *** david-lyle has joined #openstack-nova | 15:25 | |
| *** esberglu has quit IRC | 15:26 | |
| *** slaweq__ has quit IRC | 15:27 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Change parent of diagnostics objects to NovaObject class https://review.openstack.org/394480 | 15:27 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 15:27 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 15:27 |
| *** ducttape_ has joined #openstack-nova | 15:29 | |
| *** baoli has quit IRC | 15:30 | |
| *** baoli has joined #openstack-nova | 15:33 | |
| *** bnemec is now known as beekneemech | 15:33 | |
| openstackgerrit | Ritesh proposed openstack/nova: CPU pinning to consider single set of vcpus https://review.openstack.org/423334 | 15:34 |
| *** annegentle has joined #openstack-nova | 15:34 | |
| *** sacharya has joined #openstack-nova | 15:35 | |
| *** pbandark has joined #openstack-nova | 15:38 | |
| *** jaosorior has joined #openstack-nova | 15:39 | |
| mdbooth | johnthetubaguy: We discussed this at the last summit: https://review.openstack.org/#/c/383859/ . It passes all the CI we've got, and I've been hammering on a version of it wrt a customer bug. Seems robust. Any chance you could take a look? | 15:42 |
| *** david-lyle has quit IRC | 15:42 | |
| *** salv-orlando has quit IRC | 15:44 | |
| mdbooth | It's a fix for this bug, btw: https://bugs.launchpad.net/nova/+bug/1421550 | 15:45 |
| openstack | Launchpad bug 1421550 in OpenStack Compute (nova) "Creating VM image fails under the race condition with detaching volume" [Medium,In progress] - Assigned to Matthew Booth (mbooth-9) | 15:45 |
| openstackgerrit | Ritesh proposed openstack/nova: CPU pinning to consider single set of vcpus https://review.openstack.org/423334 | 15:45 |
| melwitt | mriedem: I dunno if you saw I made a couple of tweaks to the instance creation patch and it passed tests. so I was hoping you could take a look and point out any issues with the current state of it, before you go on vacation | 15:46 |
| johnthetubaguy | mdbooth: its on my queue, but if I am honest, best to ping me just after feature freeze | 15:46 |
| mdbooth | johnthetubaguy: Bug fixes are good after FF? | 15:46 |
| *** tommylikehu1 has joined #openstack-nova | 15:47 | |
| johnthetubaguy | mdbooth: yes, assuming they are not features, and not going to risk hitting RC | 15:47 |
| melwitt | mriedem: I saw that you've rechecked it as you've updated the d-g change | 15:47 |
| *** jdurgin has joined #openstack-nova | 15:47 | |
| *** phealy has joined #openstack-nova | 15:48 | |
| mriedem | melwitt: yeah i saw it was passing except for the multinode job | 15:48 |
| mriedem | which i'm hoping my d-g change to run discover_hosts will fix | 15:48 |
| *** mvk has quit IRC | 15:48 | |
| melwitt | okay, cool. I'm keeping an eye on the test run | 15:49 |
| *** tommylikehu has quit IRC | 15:49 | |
| *** tommylikehu1 is now known as tommylikehu | 15:49 | |
| *** ratailor has quit IRC | 15:52 | |
| openstackgerrit | Feodor Tersin proposed openstack/nova-specs: Add ScaleIO ephemeral storage backend https://review.openstack.org/304715 | 15:56 |
| *** burgerk has joined #openstack-nova | 15:59 | |
| mriedem | melwitt: hmm for some reason in this run it didn't run discover_hosts.sh http://logs.openstack.org/05/423005/3/check/gate-neutron-dsvm-functional-ubuntu-xenial-nv/df5f610/logs/ | 16:00 |
| mriedem | my bash test for the existence of the file might be busted | 16:00 |
| melwitt | mriedem: I saw this on one of the grenade runs http://logs.openstack.org/79/319379/64/check/gate-grenade-dsvm-neutron-ubuntu-xenial/49c8367/console.html#_2017-01-20_15_53_00_248820 | 16:01 |
| mriedem | yeah that's old | 16:01 |
| mriedem | i fixed the permissions on the script | 16:01 |
| melwitt | oh | 16:01 |
| melwitt | hm, well it just did it just now | 16:01 |
| mriedem | http://logs.openstack.org/05/423005/3/check/gate-grenade-dsvm-neutron-ubuntu-xenial/e2dfc17/console.html#_2017-01-20_15_54_01_719388 | 16:02 |
| mriedem | hrm | 16:03 |
| *** marst has joined #openstack-nova | 16:03 | |
| *** pcaruana has quit IRC | 16:03 | |
| mriedem | do i need to run the test as the stack user? | 16:03 |
| mriedem | something is busted because it's also trying to run it on mitaka http://logs.openstack.org/05/423005/3/check/gate-tempest-dsvm-full-ubuntu-trusty-mitaka-nv/27e2070/logs/devstack-gate-discover-hosts.txt.gz | 16:03 |
| mriedem | where the discover_hosts.sh file doesn't exist | 16:03 |
| *** voelzmo has quit IRC | 16:05 | |
| *** markus_z has quit IRC | 16:05 | |
| melwitt | hm, I know little about d-g. is sdague around? | 16:05 |
| *** amotoki has joined #openstack-nova | 16:05 | |
| *** sridharg has quit IRC | 16:05 | |
| *** amotoki has quit IRC | 16:06 | |
| *** pbandark has quit IRC | 16:06 | |
| *** baoli_ has joined #openstack-nova | 16:07 | |
| *** baoli has quit IRC | 16:07 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 16:08 |
| *** tovin07 has quit IRC | 16:08 | |
| *** ijw has joined #openstack-nova | 16:08 | |
| mriedem | actually d-g already does this: | 16:08 |
| mriedem | "chown -R stack:stack '$BASE'" | 16:08 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 16:08 |
| *** hieulq has quit IRC | 16:09 | |
| melwitt | not related to the failure but is it supposed to trace like this when cell0 already mapped? http://logs.openstack.org/05/423005/3/check/gate-grenade-dsvm-neutron-ubuntu-xenial/e2dfc17/logs/grenade.sh.txt.gz#_2017-01-20_15_51_45_409 | 16:09 |
| mriedem | hmm, i think that's this https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/upgrade.sh#L78 | 16:11 |
| mriedem | blork https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1157 | 16:12 |
| mriedem | so yeah we don't handle the DBDuplicateEntry there | 16:12 |
| mriedem | melwitt: want to open a bug? | 16:12 |
| melwitt | mriedem: okay, sure | 16:12 |
| mriedem | melwitt: actually https://review.openstack.org/#/c/420132/4/nova/cmd/manage.py | 16:13 |
| *** dtp has joined #openstack-nova | 16:13 | |
| mriedem | just merge that bad boy | 16:13 |
| mriedem | dtp: to the rescue | 16:13 |
| *** tovin07 has joined #openstack-nova | 16:14 | |
| dtp | what's up my brother? | 16:14 |
| *** armax has joined #openstack-nova | 16:14 | |
| *** hongbin has joined #openstack-nova | 16:15 | |
| *** hieulq_ has joined #openstack-nova | 16:15 | |
| *** hieulq has joined #openstack-nova | 16:15 | |
| *** sandanar__ has quit IRC | 16:15 | |
| melwitt | mriedem: ah, right, I thought I had seen dupe handling somewhere. that's where | 16:16 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 16:16 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Add a PlacementFixture for functests https://review.openstack.org/423353 | 16:16 |
| *** raunak has quit IRC | 16:16 | |
| bauzas | sdague: your eyes could be nice for https://review.openstack.org/423353 | 16:17 |
| mriedem | dtp: nothing, just pointed melwitt to something you already fixed | 16:18 |
| *** Oku_OS is now known as Oku_OS-away | 16:18 | |
| *** satyar_ has joined #openstack-nova | 16:19 | |
| sdague | bauzas: that looks pretty sane | 16:19 |
| sdague | is there code exercising it somewhere? | 16:19 |
| bauzas | sdague: in the top patch | 16:19 |
| bauzas | sdague: I'm getting a 400 like devstack | 16:20 |
| bauzas | because we're not asking for the latest microversion | 16:20 |
| sdague | ok | 16:20 |
| *** satyar has quit IRC | 16:20 | |
| bauzas | but when tested locally, I'm able to see the REST resources correctly | 16:20 |
| bauzas | sdague: I haven't changed the noauth logic because I thought we could be discussing about the direction | 16:20 |
| bauzas | sdague: because nova doesn't create a fake token to pass up to the next middleware, it rather returns a 204 with that token so a later call could use it | 16:21 |
| bauzas | so it basically fakes keystone | 16:21 |
| bauzas | sdague: but in the scheduler report client, we don't have the same logic, and we just get the 204 instead | 16:22 |
| bauzas | so, the question would be : should we do like nova and have a better client, or should we just pass the fake token upcall ? | 16:22 |
| bauzas | I didn't want that question to be discussed while reviewing the fixture | 16:22 |
| *** bvanhav has quit IRC | 16:24 | |
| *** dave-mccowan has quit IRC | 16:29 | |
| dtp | mriedem to submit a blueprint for pike, it sounds like i just add a link to it to the next meeting agenda? | 16:30 |
| pumaranikar | sfinucan, https://review.openstack.org/#/c/345708/ feature classification matrix. Can you please check my comment ? | 16:30 |
| *** jwcroppe has quit IRC | 16:30 | |
| mriedem | dtp: specless blueprint? | 16:30 |
| pumaranikar | sfinucan, let me know if any changes required | 16:30 |
| dtp | yeah specless | 16:30 |
| *** Apoorva has joined #openstack-nova | 16:30 | |
| mriedem | dtp: just target it to pike for now | 16:30 |
| mriedem | we aren't reviewing pike bps yet | 16:31 |
| *** jwcroppe has joined #openstack-nova | 16:31 | |
| dtp | okey dokey, thanks! | 16:31 |
| *** salv-orlando has joined #openstack-nova | 16:31 | |
| *** jwcroppe_ has joined #openstack-nova | 16:32 | |
| *** sudipto has joined #openstack-nova | 16:32 | |
| *** sudipto_ has joined #openstack-nova | 16:32 | |
| *** hieulq_ has quit IRC | 16:32 | |
| mfeoktistov | mriedem: what do you think about https://review.openstack.org/#/c/312488/ also Virtuozzo CI ran and passed test_verify_created_server_ephemeral_disk | 16:33 |
| *** jwcroppe has quit IRC | 16:35 | |
| mriedem | mfeoktistov: i think i have to wrap up some internal stuff today before i can get back to reviewing things upstream | 16:35 |
| mriedem | mfeoktistov: but it was ok for the most part last i looked so i don't expect big issues | 16:36 |
| *** mlavalle has joined #openstack-nova | 16:37 | |
| mriedem | andreykurilin: one thing i wanted to remind you of, next thursday is the final release for novaclient for ocata | 16:38 |
| mriedem | andreykurilin: i think we have some stuff to get released, like at least one microversion, and maybe some bug fixes | 16:38 |
| andreykurilin | mriedem: thanks for reminding :) | 16:38 |
| *** ayogi has quit IRC | 16:39 | |
| andreykurilin | mriedem: we have one critical bug fix at review. I'll try to help author to finish it asap | 16:39 |
| mriedem | https://github.com/openstack/python-novaclient/compare/7.0.0...master | 16:39 |
| andreykurilin | mriedem: https://review.openstack.org/#/c/419441/ | 16:39 |
| openstackgerrit | Hironori Shiina proposed openstack/nova: ironic: Add trigger crash dump support to ironic driver https://review.openstack.org/376548 | 16:39 |
| *** bvanhav has joined #openstack-nova | 16:42 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 16:42 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add a PlacementFixture https://review.openstack.org/423353 | 16:42 |
| *** efoley has quit IRC | 16:42 | |
| bauzas | sdague: could you please reapply your +2 ? I just add UTs for validating the fixture | 16:42 |
| sdague | bauzas: done | 16:43 |
| bauzas | mriedem: leakypipes: in case you wanna quickly +W it ^ | 16:43 |
| bauzas | sdague: thanks | 16:43 |
| *** esberglu has joined #openstack-nova | 16:43 | |
| bauzas | now working on the microversion issue | 16:43 |
| bauzas | leakypipes: ^ | 16:43 |
| *** satyar has joined #openstack-nova | 16:43 | |
| *** yamahata has joined #openstack-nova | 16:45 | |
| *** satyar_ has quit IRC | 16:46 | |
| *** esberglu_ has quit IRC | 16:46 | |
| *** mdrabe has quit IRC | 16:46 | |
| *** mfeoktistov has quit IRC | 16:47 | |
| *** andymaier has quit IRC | 16:48 | |
| *** Apoorva has quit IRC | 16:48 | |
| mriedem | bauzas: sdague: why do we need to stub out the scheduler report client if we have a placement fixture? | 16:48 |
| mriedem | i.e. can't the scheduler report client just make calls to placement which is really going to be the placement fixture? | 16:48 |
| bauzas | mriedem: because we don't have yet a real placement client | 16:48 |
| *** jaosorior has quit IRC | 16:48 | |
| sdague | mriedem: because we'd need to emulate *all* the details of keystone | 16:49 |
| sdague | including the token contents | 16:49 |
| *** jaosorior has joined #openstack-nova | 16:49 | |
| sdague | we did the same thing on the osapi fixture | 16:49 |
| sdague | it just feels less janky because the test code is given that client back, and it makes the calls | 16:50 |
| bauzas | yeah I tried to use the same logic | 16:50 |
| sdague | but here we've got a keystone validated client in between services that aren't directly being talked to by the test | 16:50 |
| mriedem | so will this still use the resource providers in the nova_api db? | 16:50 |
| bauzas | yeah I would have loved to see something like a faked client but I wasn't able | 16:51 |
| mriedem | like we'll have functional tests for the scheduler that post some RPs to setup some data, and then the scheduler will run and GET them back? | 16:51 |
| bauzas | mriedem: yup, see the top patch | 16:51 |
| sdague | mriedem: yeh, all that linkage is there | 16:51 |
| *** jaosorior has quit IRC | 16:51 | |
| bauzas | eg. the server affinity ones | 16:51 |
| bauzas | I tested that with server group test class | 16:52 |
| mriedem | ok, it's not obvious from the server group test | 16:52 |
| bauzas | sec, finding you the logs | 16:52 |
| mriedem | i figured there would be a specific scheduler filter functional test that preloads (POSTs) some RPs into the placement fixture | 16:52 |
| mriedem | and then calls the scheduler filter to get some hosts back | 16:52 |
| mriedem | but whatever, i'm sure it was failing before you added this | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpuset_reserved in NUMACell https://review.openstack.org/400874 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus reserved https://review.openstack.org/400876 | 16:53 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | 16:53 |
| mriedem | and our fake computes in those tests are using the fake virt driver which is providing some fake resources so that the resource tracker puts that stuff into the placement service right? | 16:53 |
| bauzas | mriedem: lemme verify | 16:53 |
| mriedem | i wonder if we'll have a race there... | 16:54 |
| *** vladikr has quit IRC | 16:54 | |
| mriedem | because i'm not sure that starting the nova-compute service is blocking on doing update_available_resource | 16:54 |
| mriedem | in the RT | 16:54 |
| *** vladikr has joined #openstack-nova | 16:54 | |
| bauzas | yeah, sec | 16:55 |
| bauzas | pasting it | 16:55 |
| *** mdrabe has joined #openstack-nova | 16:55 | |
| bauzas | because the functional test job is not yet done | 16:55 |
| bauzas | mriedem: http://paste.openstack.org/show/595821/ | 16:55 |
| sdague | mriedem: that I don't know, I do know that conceptually this fixture means you can run the in process test cloud with placement so you can start exploring this space | 16:55 |
| bauzas | mriedem: L41 of that paste | 16:56 |
| bauzas | and L37 | 16:56 |
| robcresswell | o/ hey folks we've reached this cycles "horizon and novaclient are broken again" time. What are we usupposed to be using instead of absolute limits? | 16:56 |
| * robcresswell think he has a fun week of fixing ahead | 16:56 | |
| *** tbachman has joined #openstack-nova | 16:56 | |
| bauzas | we still get an error because of the microversion problem | 16:57 |
| *** sahid has quit IRC | 16:57 | |
| mriedem | bauzas: ok cool | 16:57 |
| bauzas | but the computes are correctly updating the Placement service AFAICS | 16:57 |
| bauzas | unless I missed something | 16:57 |
| *** tblakes has quit IRC | 16:57 | |
| mriedem | robcresswell: more context would be good | 16:57 |
| mriedem | what microversion are you breaking on? | 16:57 |
| bauzas | mriedem: the fact that the get method is not asking for 1.4 | 16:58 |
| robcresswell | mriedem: We don't use them, so 2.1 I'd assume. | 16:58 |
| mriedem | robcresswell: then i don't know why you'd be broken | 16:58 |
| robcresswell | Its just because of the 7.0.0 release I believe | 16:58 |
| bauzas | so when the scheduler asks for resources, the placement service says "meh, what are you asking?" | 16:58 |
| robcresswell | oo, interesting | 16:58 |
| mriedem | bauzas: i know | 16:59 |
| *** _ducttape_ has joined #openstack-nova | 16:59 | |
| bauzas | oh, sorry, I thought your "what microversion are you breaking on?" was related to me :) | 16:59 |
| bauzas | anyway, back to that problem | 16:59 |
| sdague | robcresswell: a well formed bug is probably important here, because at this point we're just all guessing :) | 16:59 |
| mriedem | nope | 16:59 |
| *** jose-phillips has joined #openstack-nova | 17:00 | |
| mriedem | i'm going to lunch and running errands, so be back in a couple of hours | 17:00 |
| *** derekh has quit IRC | 17:00 | |
| robcresswell | sdague: Yeah, I'll give it a go. | 17:00 |
| mriedem | when you guys will have figured and fixed everything | 17:00 |
| *** mriedem is now known as mriedem_afk | 17:00 | |
| openstackgerrit | Arnaud Morin proposed openstack/nova: Set default value to HTTP_X_FORWARDED_PROTO https://review.openstack.org/423375 | 17:01 |
| *** udesale has quit IRC | 17:02 | |
| *** ducttape_ has quit IRC | 17:02 | |
| *** damien_r has quit IRC | 17:03 | |
| *** salv-orlando has quit IRC | 17:04 | |
| *** ralonsoh has quit IRC | 17:05 | |
| dims | sdague : just 3 lines to review if you get a chance please - https://review.openstack.org/#/c/421911/ | 17:05 |
| dims | leakypipes : ^^ | 17:05 |
| sdague | dims: +2 | 17:05 |
| dims | thanks sdague ! | 17:06 |
| bauzas | Whipped | 17:06 |
| dims | thanks bauzas ! | 17:06 |
| leakypipes | dims: done. | 17:06 |
| dims | and leakypipes ! | 17:06 |
| leakypipes | fastest patch approval EVAH. | 17:06 |
| *** tblakes has joined #openstack-nova | 17:06 | |
| dims | i know!!!! | 17:06 |
| bauzas | leakypipes: https://review.openstack.org/#/c/423353/ in case you wanna beat my timings :) | 17:07 |
| leakypipes | bauzas: I'm already reviewing that one :) | 17:07 |
| bauzas | leakypipes: http://paste.openstack.org/show/595821/ for seeing the result | 17:07 |
| bauzas | leakypipes: FWIW, I'm planning to write a specific change for https://review.openstack.org/#/c/385693/27/nova/scheduler/client/report.py | 17:09 |
| bauzas | because it's a bug | 17:09 |
| leakypipes | bauzas: what's a bug? | 17:10 |
| bauzas | and we need that in order to merge the scheduler change | 17:10 |
| dims | update for all of you - sdague, leakypipes, bauzas, mriedem_afk : am down to one flaky issue in Nova and one persistent failure in neutron in that py35/tempest job | 17:10 |
| bauzas | leakypipes: the fact that we're stuck with 1.0 | 17:10 |
| sdague | dims: cool | 17:10 |
| sdague | dims what is the flakey issue? | 17:10 |
| robcresswell | sdague, mriedem_afk: https://bugs.launchpad.net/python-novaclient/+bug/1658148 Bug report so far, I'll continue to add more. | 17:10 |
| openstack | Launchpad bug 1658148 in python-novaclient "Horizon/Nova interactions broken " [Undecided,New] | 17:10 |
| robcresswell | Sorry for the vague title | 17:10 |
| leakypipes | bauzas: you're saying you are pushing a patch that only adds the OpenStack-APi-Version header to the reporting client. is that correct? | 17:11 |
| dims | sdague : tempest.api.compute.admin.test_servers.ServersAdminTestJSON.test_list_servers_by_admin_with_specified_tenant | 17:11 |
| leakypipes | bauzas: if so, cool by me. | 17:11 |
| bauzas | leakypipes: yup, zactly | 17:11 |
| leakypipes | bauzas: I have separate three patches waiting each with that same code block :) | 17:11 |
| bauzas | heh | 17:11 |
| bauzas | and mine too | 17:11 |
| bauzas | all my tempest jobs are red because of that | 17:12 |
| dims | the neutron failure is tempest.api.compute.admin.test_auto_allocate_network.AutoAllocateNetworkTest.test_server_multi_create_auto_allocate for the record | 17:12 |
| bauzas | FWIW, 7th coffee of the day | 17:12 |
| *** jose-phillips has quit IRC | 17:12 | |
| bauzas | I'm nearly close to beat my personal record | 17:12 |
| bauzas | still 1 | 17:13 |
| bauzas | but the day is not over | 17:13 |
| *** raunak has joined #openstack-nova | 17:13 | |
| johnthetubaguy | melwitt: I think I got my head more around the first two quota patches now | 17:15 |
| johnthetubaguy | melwitt: added some follow up comments | 17:16 |
| melwitt | johnthetubaguy: I saw, thanks, much appreciated. I replied to your one about duplicates a sec ago. I was thinking of the create_limit API when I say 'duplicates' | 17:16 |
| johnthetubaguy | melwitt: I think the outstanding question in my head is a nuts edge case, delete, if you delete in the new DB, but the old one wasn't deleted, it might come back to life | 17:16 |
| sdague | robcresswell: added comments to the bug, nova-api logs would be useful there for that request id | 17:16 |
| sdague | the 403 makes it seem like there is something going on differently on the auth side | 17:17 |
| johnthetubaguy | melwitt: I was thinking we only get duplicates from the migration | 17:17 |
| melwitt | johnthetubaguy: yeah, have to think about that one more. thanks for all the good points | 17:17 |
| johnthetubaguy | melwitt: no worries, this stuff really mashes your brain, somehow | 17:17 |
| *** nicolasbock has quit IRC | 17:18 | |
| *** nicolasbock has joined #openstack-nova | 17:18 | |
| melwitt | johnthetubaguy: yeah. exact duplicates most likely. but I was thinking of if someone already has a 'instances' limit in the main DB, if they create an 'instances' limit before migrations were run, they would have two, one in API DB and one in main DB | 17:19 |
| johnthetubaguy | melwitt: oh right, you get a "newer one" at the top | 17:19 |
| melwitt | yeah. again I don't see why anyone would do that but that could happen | 17:20 |
| johnthetubaguy | well, depends if you automate your user onboarding | 17:20 |
| johnthetubaguy | thats probably more likely that I would like | 17:20 |
| johnthetubaguy | s/that/than/ | 17:20 |
| johnthetubaguy | hmm, thats mostly nonsense probably, I need to look at the API | 17:22 |
| *** lucasagomes is now known as lucas-afk | 17:23 | |
| *** a-pugachev has quit IRC | 17:24 | |
| *** nic has joined #openstack-nova | 17:25 | |
| *** jdillaman has quit IRC | 17:28 | |
| *** jdillaman has joined #openstack-nova | 17:29 | |
| *** mvk has joined #openstack-nova | 17:29 | |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (cfg drv) https://review.openstack.org/409404 | 17:32 |
| *** jaosorior has joined #openstack-nova | 17:32 | |
| robcresswell | sdague: Updated the bug report with logs from nova-api. Seems to not recognise the project id. | 17:33 |
| *** _ducttape_ has quit IRC | 17:35 | |
| *** lpetrut has quit IRC | 17:36 | |
| sdague | robcresswell: hmmm | 17:36 |
| *** ducttape_ has joined #openstack-nova | 17:36 | |
| sdague | robcresswell: do you have a simple test case to reproduce it? That all seems very weird. | 17:36 |
| *** kristian__ has quit IRC | 17:38 | |
| *** tbachman has quit IRC | 17:39 | |
| robcresswell | sdague: Horizon master would be the simplest thing to do, if you have a devstack machine about | 17:40 |
| robcresswell | Had multiple other confirmations in the Horizon channel before heading over hear :) | 17:40 |
| robcresswell | here* | 17:40 |
| sdague | robcresswell: ok, yeh, I need to find some lunch first, but can probably look this afternoon | 17:42 |
| *** tbachman has joined #openstack-nova | 17:42 | |
| robcresswell | sdague: Cool, no problem. If you have any suggestions, please leave them in the bug report - its Friday evening here and has been a loooong week :) | 17:42 |
| sdague | robcresswell: yeh, no worries, enjoy the weekend | 17:43 |
| robcresswell | sdague: Thanks, you too | 17:43 |
| diana_clarke | About the horizon/python-novaclient thing: the tests all pass but it doesn't work in real life (aka devstack), correct? | 17:44 |
| *** baoli_ has quit IRC | 17:45 | |
| *** baoli has joined #openstack-nova | 17:46 | |
| *** gszasz has quit IRC | 17:46 | |
| *** gszasz has joined #openstack-nova | 17:46 | |
| robcresswell | diana_clarke: Seems that way, yep | 17:46 |
| robcresswell | diana_clarke: Haven't done much investigation, mainly just marched over here with my pitchfork. I'll be around again Monday if there are some suggestions, otherwise I'll just poke at Horizon until it works. | 17:47 |
| robcresswell | Primarily a time constraint thing though so far, will have more time to dedicate next week. PTl isn't around much right now, so I'm probably the easiest person to contact :) | 17:49 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove more imagecache info file code https://review.openstack.org/423392 | 17:49 |
| *** Apoorva has joined #openstack-nova | 17:50 | |
| diana_clarke | robcresswell: If I had to guess (without much digging), I suspect it has to do with the deprecations that were done here, specially if it's project id related: https://review.openstack.org/#/q/project:openstack/python-novaclient+branch:master+topic:session | 17:50 |
| diana_clarke | *especially | 17:51 |
| diana_clarke | (but I could be wrong) | 17:51 |
| diana_clarke | perhaps this, in particular: https://review.openstack.org/#/c/350106/14/releasenotes/notes/clarify-project-id-variable-5832698315000438.yaml | 17:52 |
| robcresswell | diana_clarke: Would you mind leaving suggestions on https://bugs.launchpad.net/python-novaclient/+bug/1658148 ? I'm happy to spend the time looking, but this is a busy channel so I'll likely lose track if suggestions are posted here. | 17:52 |
| openstack | Launchpad bug 1658148 in OpenStack Dashboard (Horizon) "Horizon/Nova interactions broken " [Critical,New] - Assigned to Rob Cresswell (robcresswell) | 17:52 |
| *** mnestratov has quit IRC | 17:52 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 17:53 |
| diana_clarke | robcresswell: I'm mostly just talking out loud at this point. If I find something legit I'll be sure to update that bug, I'm on PTO this afternoon though, so I probably won't look much until Monday either. | 17:53 |
| *** sacharya has quit IRC | 17:54 | |
| robcresswell | diana_clarke: Ah cool, not a problem. Even educated guesses are probably more valuable than my blundering around novaclient code though. Have a good weekend, and thanks for the help. | 17:54 |
| *** hieulq has quit IRC | 17:56 | |
| *** tovin07 has quit IRC | 17:56 | |
| *** karimb has quit IRC | 17:58 | |
| *** bvanhav has quit IRC | 18:02 | |
| *** bvanhav has joined #openstack-nova | 18:02 | |
| *** jwcroppe has joined #openstack-nova | 18:05 | |
| *** jwcroppe_ has quit IRC | 18:05 | |
| *** hieulq has joined #openstack-nova | 18:12 | |
| *** tovin07 has joined #openstack-nova | 18:12 | |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Fix races in glusterfs mount/umount https://review.openstack.org/423402 | 18:15 |
| mdbooth | johnthetubaguy: For context with the nfs volume driver, with that in place GlusterFS took me about 30 minutes ^^^ | 18:16 |
| *** claudiub has joined #openstack-nova | 18:18 | |
| *** jpena is now known as jpena|off | 18:19 | |
| *** claudiub|2 has quit IRC | 18:21 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 18:23 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Remove more imagecache info file code https://review.openstack.org/423392 | 18:30 |
| *** mvk has quit IRC | 18:30 | |
| *** ayogi has joined #openstack-nova | 18:32 | |
| *** sacharya has joined #openstack-nova | 18:34 | |
| *** jaosorior has quit IRC | 18:35 | |
| *** lpetrut has joined #openstack-nova | 18:36 | |
| *** annegentle has quit IRC | 18:36 | |
| *** ociuhandu has quit IRC | 18:40 | |
| *** esmiurium has quit IRC | 18:40 | |
| *** sudipto has quit IRC | 18:41 | |
| *** sudipto_ has quit IRC | 18:41 | |
| *** owalsh is now known as owalsh-brb | 18:42 | |
| *** gszasz has quit IRC | 18:43 | |
| *** Swami has joined #openstack-nova | 18:43 | |
| *** esmiurium has joined #openstack-nova | 18:47 | |
| *** browne has joined #openstack-nova | 18:47 | |
| sdague | the fail rate on one of the notification tests in the functional bucket is pretty problematic - http://logs.openstack.org/29/421529/1/gate/gate-nova-tox-db-functional-ubuntu-xenial/20c5e9c/console.html#_2017-01-20_18_13_50_140452 | 18:48 |
| *** owalsh-brb has quit IRC | 18:48 | |
| sdague | we've got 2 gate resets right now on that | 18:48 |
| sdague | http://logs.openstack.org/09/407309/16/gate/gate-nova-tox-db-functional-ubuntu-xenial/5680644/console.html#_2017-01-20_17_53_41_192031 as well | 18:49 |
| *** owalsh-brb has joined #openstack-nova | 18:49 | |
| *** ayogi has quit IRC | 18:51 | |
| *** rfolco has quit IRC | 18:51 | |
| *** JoseMello has quit IRC | 18:55 | |
| melwitt | sdague: mriedem_afk has this change up to add logging to help with investigating that https://review.openstack.org/#/c/421221/ | 18:56 |
| sdague | melwitt: +A | 18:59 |
| melwitt | sweet | 18:59 |
| *** ociuhandu has joined #openstack-nova | 19:00 | |
| *** mriedem_afk is now known as mriedem | 19:00 | |
| *** rfolco has joined #openstack-nova | 19:01 | |
| *** sacharya has quit IRC | 19:02 | |
| *** rfolco has quit IRC | 19:02 | |
| melwitt | mriedem: ohay, I just commented on your d-g change, I think maybe I found what's wrong | 19:05 |
| mriedem | melwitt: i just updated it for that thanks | 19:05 |
| melwitt | oh, cool | 19:05 |
| mriedem | that doesn't really explain why it's trying to run things on stable though | 19:05 |
| *** voelzmo has joined #openstack-nova | 19:05 | |
| mriedem | since: test -f discover_hosts.sh won't exist on stable | 19:05 |
| mriedem | so it shouldn't try the sudo part | 19:05 |
| melwitt | mriedem: oh, yeah sorry I found something else that I just commented on a sec ago, the permissions on the /logs dir | 19:06 |
| melwitt | I dunno about the stable thing though | 19:06 |
| mriedem | the logs permissions are changed before we run discover_host | 19:08 |
| mriedem | so i'm thinking that's not it | 19:08 |
| melwitt | I thought it's only doing that in the other code path | 19:09 |
| melwitt | in the "else" | 19:09 |
| mriedem | gdi | 19:12 |
| mriedem | melwitt: do you ever get tired of being right? | 19:12 |
| sdague | mriedem: is that the t-shirt slogan for Boston? | 19:13 |
| melwitt | haha, well, we'll see if that actually works. d-g looks like magic to me | 19:13 |
| oomichi | alex_xu: johnthetubaguy: gmann: Can you take a look at https://review.openstack.org/#/c/406377 ? | 19:14 |
| mriedem | sdague: i think the slogan is going to be "wait, what if i upgrade to ocata THIS way" | 19:15 |
| *** unicell has joined #openstack-nova | 19:15 | |
| melwitt | haha, yes that | 19:16 |
| *** unicell is now known as Guest59079 | 19:16 | |
| sdague | people didn't realize that ocata is actually a puzzle room | 19:16 |
| melwitt | lol | 19:16 |
| *** david-lyle has joined #openstack-nova | 19:17 | |
| mriedem | bauzas: ok +W on the placement fixture | 19:23 |
| *** bvanhav has quit IRC | 19:23 | |
| mriedem | sdague: so before laura gets home and explodes her travel anxiety on me, can you take a quick peek at https://review.openstack.org/#/c/423005/ and let me know if that's an OK track for this thing with cells v2 in d-g | 19:23 |
| mriedem | the point of putting discover_hosts into a script in devstack is to avoid needing ugly branch logic in d-g itself | 19:24 |
| *** david-lyle has quit IRC | 19:24 | |
| *** bvanhav has joined #openstack-nova | 19:25 | |
| *** baoli has quit IRC | 19:25 | |
| *** Guest55718 has quit IRC | 19:26 | |
| *** tesseract has quit IRC | 19:32 | |
| *** Guest59079 has quit IRC | 19:32 | |
| *** sshen has quit IRC | 19:34 | |
| *** tbachman has quit IRC | 19:34 | |
| sdague | mriedem: I think so... | 19:34 |
| *** kristian__ has joined #openstack-nova | 19:34 | |
| *** sambetts is now known as sambetts|afk | 19:36 | |
| openstackgerrit | Francesco Santoro proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | 19:36 |
| melwitt | something blowing up on the bottom change | 19:36 |
| openstackgerrit | STEW TY proposed openstack/nova: Transform instance.rescue notifications https://review.openstack.org/385644 | 19:38 |
| *** kristian__ has quit IRC | 19:39 | |
| melwitt | /opt/stack/new/devstack-gate/devstack-vm-gate.sh: line 288: localrc: Permission denied | 19:39 |
| melwitt | I wonder if those commands have to be after setup_localrc or something? | 19:40 |
| *** stvnoyes1 has joined #openstack-nova | 19:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/422323 | 19:43 |
| *** stvnoyes has quit IRC | 19:45 | |
| *** qiuyu has joined #openstack-nova | 19:45 | |
| *** yamahata has quit IRC | 19:45 | |
| *** owalsh-brb is now known as owalsh | 19:47 | |
| *** slaweq_ has joined #openstack-nova | 19:47 | |
| *** Sukhdev has joined #openstack-nova | 19:47 | |
| mriedem | maybe but i'm not sure why | 19:47 |
| *** qiuyu is now known as unicell_ | 19:48 | |
| *** jose-phillips has joined #openstack-nova | 19:48 | |
| *** voelzmo has quit IRC | 19:49 | |
| *** voelzmo has joined #openstack-nova | 19:50 | |
| mriedem | sdague: ^ do we need to run setup_localrc before we can change permissions on the files in the workspace? | 19:50 |
| *** catinthe_ has joined #openstack-nova | 19:50 | |
| sdague | is there a full log somewhere? | 19:51 |
| *** jaosorior has joined #openstack-nova | 19:52 | |
| melwitt | sdague: http://logs.openstack.org/17/423417/1/check/gate-grenade-dsvm-neutron-ubuntu-xenial/479e3c9/console.html#_2017-01-20_19_25_44_044049 I don't know how to see more detail than that | 19:52 |
| *** kristian__ has joined #openstack-nova | 19:52 | |
| *** catintheroof has quit IRC | 19:53 | |
| mriedem | yeah that's all i see | 19:55 |
| *** voelzmo has quit IRC | 19:55 | |
| melwitt | and this is the change https://review.openstack.org/#/c/423417/ | 19:55 |
| mriedem | it calls setup_localrc and things blow up | 19:55 |
| mriedem | oh i think i know | 19:55 |
| mriedem | because d-g can't write to the files because stack user owns then now? | 19:56 |
| *** unicell_ has quit IRC | 19:56 | |
| *** unicell_ has joined #openstack-nova | 19:56 | |
| mriedem | so, zuul user can't write to them in d-g | 19:56 |
| *** unicell_ is now known as unicell | 19:56 | |
| melwitt | ohh | 19:56 |
| *** unicell has quit IRC | 19:57 | |
| melwitt | so I guess normally it sets up things first, then changes things over to the stack user | 19:57 |
| *** unicell has joined #openstack-nova | 19:58 | |
| sdague | we do definitely drop privs completely for safety reasons, and remove files from sudoers | 20:00 |
| *** cleong has quit IRC | 20:00 | |
| *** tbachman has joined #openstack-nova | 20:00 | |
| *** jamesdenton has joined #openstack-nova | 20:01 | |
| mriedem | ok, so i'll redo this and just move that stuff into a common function | 20:02 |
| mriedem | so i can also call from the grenade branch | 20:02 |
| *** annegentle has joined #openstack-nova | 20:03 | |
| *** tbachman_ has joined #openstack-nova | 20:05 | |
| *** tbachman has quit IRC | 20:06 | |
| *** tbachman_ is now known as tbachman | 20:06 | |
| sdague | yeh, your analysis that you are changing permissions too early is right, sorry it took a minute for me to gather all the context | 20:06 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 20:07 |
| *** gouthamr has quit IRC | 20:07 | |
| *** moshele has joined #openstack-nova | 20:08 | |
| melwitt | thanks sdague | 20:08 |
| mriedem | pushed | 20:08 |
| mriedem | just in time for laura to walk in | 20:08 |
| * mriedem prepares | 20:08 | |
| sdague | the permission changing is done for the stack user right before the stack.sh hand off. It starts as being under the jenkins user. The stack user can itself sudo up until this point - https://github.com/openstack-infra/devstack-gate/blob/5883a8269a741d77c1853150c56d6d90fe01b83e/devstack-vm-gate.sh#L720-L724 | 20:08 |
| sdague | it's sudo privs are taken away before tempest runs, so that tempest doesn't cheat | 20:09 |
| dims | mriedem :) | 20:09 |
| dims | sdague : gate is 39 deep, something got busted? | 20:09 |
| sdague | dims: as far as I can, it's just a ton of paper cuts | 20:10 |
| sdague | nova notification tests are sometimes failing | 20:10 |
| dims | ouch ok. at least we have weekend to catch up on | 20:10 |
| sdague | I saw some db things failing on cinder | 20:10 |
| dims | will watch a bit as well | 20:10 |
| sdague | the keystone os-api-ref one, which I pushed a fix (and email on) | 20:10 |
| dims | ack y i saw that. thanks sdague | 20:11 |
| *** owalsh is now known as owalsh-afk | 20:12 | |
| dims | ugh. just spotted this in the gate queue - http://logs.openstack.org/11/423111/2/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/60c1d57/logs/screen-n-cpu.txt.gz?level=ERROR#_2017-01-20_19_36_46_607 | 20:14 |
| dims | libvirtError: internal error: client socket is closed | 20:15 |
| *** esmiurium has quit IRC | 20:15 | |
| *** READ10 has quit IRC | 20:15 | |
| mriedem | dims: that's known | 20:16 |
| mriedem | libvirt crashes intermittently on xenial | 20:16 |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1643911 | 20:17 |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1646779 | 20:17 |
| mriedem | http://status.openstack.org//elastic-recheck/index.html#1638982 | 20:17 |
| dims | ack mriedem | 20:18 |
| mriedem | jroll: vdrok: sambetts|afk: can i mark this as complete too? https://blueprints.launchpad.net/nova/+spec/ironic-plug-unplug-vifs-update | 20:21 |
| mriedem | the vifs plug/unplug patch is merged, | 20:22 |
| mriedem | the interface attach/detach thing isn't | 20:22 |
| jroll | mriedem: yes, that doesn't mention the latter, that should be a separate blueprint imo | 20:22 |
| mriedem | which is https://review.openstack.org/#/c/419975/ | 20:22 |
| mriedem | ok | 20:22 |
| mriedem | cool | 20:22 |
| mriedem | nice both of those bps are merged then (portgroups and vifs update) | 20:23 |
| *** yamahata has joined #openstack-nova | 20:24 | |
| *** amoralej is now known as amoralej|off | 20:27 | |
| bauzas | oh man, the gate... | 20:28 |
| *** xyang1 has joined #openstack-nova | 20:29 | |
| *** d9k has joined #openstack-nova | 20:31 | |
| *** esmiurium has joined #openstack-nova | 20:31 | |
| *** stvnoyes has joined #openstack-nova | 20:32 | |
| *** dave-mccowan has joined #openstack-nova | 20:34 | |
| *** eharney has quit IRC | 20:34 | |
| *** stvnoyes1 has quit IRC | 20:35 | |
| mriedem | ew we unconditionally run osprofiler on all unit test runs | 20:37 |
| mriedem | even if i just want to test a specific path | 20:37 |
| *** salv-orlando has joined #openstack-nova | 20:38 | |
| *** mvensky has joined #openstack-nova | 20:38 | |
| *** salv-orlando has quit IRC | 20:43 | |
| *** voelzmo has joined #openstack-nova | 20:44 | |
| *** bvanhav has quit IRC | 20:45 | |
| *** Apoorva has quit IRC | 20:46 | |
| *** david-lyle has joined #openstack-nova | 20:47 | |
| *** ijw has quit IRC | 20:48 | |
| openstackgerrit | Julian proposed openstack/nova: Create cmd/script that prints JSON Schema of Notification Payload Classes https://review.openstack.org/411836 | 20:50 |
| *** slaweq has quit IRC | 20:51 | |
| *** slaweq_ has quit IRC | 20:51 | |
| *** slaweq_ has joined #openstack-nova | 20:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | 20:52 |
| mriedem | sean-k-mooney: leakypipes: i'm happy with ^ now | 20:52 |
| *** slaweq has joined #openstack-nova | 20:53 | |
| mriedem | actually, hmm | 20:53 |
| leakypipes | mriedem: need to boot into Windows VM... biab. (yep, Webex calls) | 20:54 |
| *** leakypipes has quit IRC | 20:54 | |
| *** jaosorior has quit IRC | 20:57 | |
| *** dimtruck is now known as zz_dimtruck | 21:00 | |
| bauzas | mriedem: sdague: I'm currently writing a patch for adding the microversion header in the report client | 21:00 |
| bauzas | mriedem: sdague: do you think we should default to the min version or the max one ? | 21:01 |
| bauzas | I think the latter | 21:01 |
| bauzas | but maybe lemme upload the change | 21:01 |
| sdague | bauzas: I thought we always default to min unless specified | 21:01 |
| bauzas | sdague: I first thought about that | 21:02 |
| *** catinthe_ has quit IRC | 21:02 | |
| bauzas | sdague: okay, lemme try to do this | 21:02 |
| bauzas | and upload it so we can disucss | 21:02 |
| *** catintheroof has joined #openstack-nova | 21:03 | |
| *** dtp has quit IRC | 21:07 | |
| *** catintheroof has quit IRC | 21:07 | |
| mriedem | bauzas: i put up something like that last night | 21:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: os-vif: convert libvirt driver to use os-vif for fast path vhostuser https://review.openstack.org/385061 | 21:09 |
| mriedem | bauzas: in here https://review.openstack.org/#/c/407309/ | 21:09 |
| mriedem | that requests 1.1 specifically for the aggregates GET it needs | 21:09 |
| mriedem | which may fail, but then it's just that aggregates GET that fails and not everything else | 21:09 |
| bauzas | mriedem: :( | 21:09 |
| mriedem | the scheduler report client is currently not doing version discovery | 21:09 |
| bauzas | lemme upload it | 21:09 |
| *** raunak has quit IRC | 21:11 | |
| mriedem | sfinucan: +2 on https://review.openstack.org/#/c/385061/ | 21:11 |
| *** eharney has joined #openstack-nova | 21:13 | |
| *** gouthamr has joined #openstack-nova | 21:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Ironic: Add soft reboot support to ironic driver https://review.openstack.org/403745 | 21:15 |
| *** voelzmo has quit IRC | 21:17 | |
| bauzas | mriedem: KABOOOM | 21:17 |
| bauzas | mriedem: my patch is trampled by the above ^ | 21:18 |
| bauzas | mriedem: so I'll remove the get header | 21:18 |
| openstackgerrit | Eric Fried proposed openstack/nova: Introduce base PowerVM driver (disk drv) https://review.openstack.org/409401 | 21:19 |
| *** damien_r has joined #openstack-nova | 21:21 | |
| *** moshele has quit IRC | 21:21 | |
| *** shaner has quit IRC | 21:21 | |
| *** stewie925_ has joined #openstack-nova | 21:24 | |
| *** Apoorva has joined #openstack-nova | 21:24 | |
| *** karimb has joined #openstack-nova | 21:26 | |
| *** karimb has quit IRC | 21:26 | |
| *** tbachman has quit IRC | 21:26 | |
| *** shaner has joined #openstack-nova | 21:27 | |
| *** jamielennox is now known as jamielennox|away | 21:28 | |
| mriedem | https://review.openstack.org/#/c/403745/ also looks good to go | 21:29 |
| *** jdurgin has quit IRC | 21:30 | |
| slaweq_ | oomichi: hello | 21:30 |
| slaweq_ | oomichi: some time ago You reviewed my small patch https://review.openstack.org/#/c/420176/ | 21:30 |
| slaweq_ | oomichi: I answered to Your comment there, can You maybe have a look on it? | 21:31 |
| slaweq_ | oomichi: thx in advance | 21:31 |
| *** smatzek has quit IRC | 21:31 | |
| mriedem | claudiub: you're going to have to sync up with markus_z or someone from s390 on https://review.openstack.org/#/c/420960/ else we'll just have to move that forward without the s390 features | 21:32 |
| *** jdurgin has joined #openstack-nova | 21:34 | |
| *** satyar has quit IRC | 21:34 | |
| *** Jeffrey4l__ has joined #openstack-nova | 21:35 | |
| *** Jeffrey4l_ has quit IRC | 21:35 | |
| mriedem | melwitt: booya http://logs.openstack.org/05/423005/6/check/gate-grenade-dsvm-neutron-ubuntu-xenial/90ab2f2/logs/devstack-gate-discover-hosts.txt.gz | 21:37 |
| *** gouthamr has quit IRC | 21:37 | |
| mriedem | the stable jobs are still failing, looks like they are still trying to run discover_hosts, | 21:37 |
| mriedem | http://logs.openstack.org/05/423005/6/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial-newton-nv/39d0108/logs/devstack-gate-discover-hosts.txt.gz | 21:38 |
| mriedem | i wonder if the "test -f discover_hosts.sh" is making it exit with 1 | 21:38 |
| mriedem | i think it is | 21:39 |
| mriedem | user@xubuntu:~/git/devstack$ cd ~/git/devstack && test -f discover_hosts1.sh && sudo -H -u user stdbuf -oL -eL cat discover_hosts.sh | 21:39 |
| mriedem | user@xubuntu:~/git/devstack$ echo $? | 21:39 |
| mriedem | 1 | 21:39 |
| *** salv-orlando has joined #openstack-nova | 21:39 | |
| *** kristian__ has quit IRC | 21:39 | |
| *** kristian__ has joined #openstack-nova | 21:40 | |
| mriedem | which means i basically can't avoid branch logic in d-g for this | 21:40 |
| *** jerrygb_ has quit IRC | 21:41 | |
| *** ijw has joined #openstack-nova | 21:42 | |
| *** salv-orlando has quit IRC | 21:44 | |
| *** kristian__ has quit IRC | 21:44 | |
| mriedem | actually nvm | 21:44 |
| mriedem | || true to the rescue | 21:44 |
| *** david-lyle has quit IRC | 21:45 | |
| *** _ducttape_ has joined #openstack-nova | 21:48 | |
| *** _ducttape_ has quit IRC | 21:49 | |
| bauzas | I'm soooo f*** up | 21:50 |
| *** raunak has joined #openstack-nova | 21:50 | |
| bauzas | the gate is horribly late | 21:50 |
| *** _ducttape_ has joined #openstack-nova | 21:50 | |
| _gryf | mriedem, you might be interested in closing this bug https://review.openstack.org/#/c/422780/ | 21:50 |
| bauzas | and I can't upload new changes unless resetting my current ones in the gate | 21:50 |
| *** ducttape_ has quit IRC | 21:51 | |
| *** avolkov has quit IRC | 21:51 | |
| *** zz_dimtruck is now known as dimtruck | 21:52 | |
| mriedem | _gryf: commented | 21:53 |
| mriedem | bauzas: -R | 21:53 |
| mriedem | git review -R | 21:53 |
| bauzas | mriedem: yup I know | 21:53 |
| bauzas | mriedem: but I want to rebase based on the one you merged | 21:54 |
| bauzas | mriedem: you know, the one having the microversion in get() | 21:54 |
| mriedem | oh | 21:54 |
| mriedem | well, you can always work the weekend :) unless your govt has outlawed it | 21:55 |
| mriedem | just put on some loverboy | 21:55 |
| mriedem | https://www.youtube.com/watch?v=zL8G5pBZ5CI | 21:55 |
| *** jwcroppe has quit IRC | 21:55 | |
| _gryf | mriedem, oh shit. i'll cover that. | 21:56 |
| bauzas | mriedem: well, I'm not trumped by my govt :p | 21:56 |
| _gryf | i mean "ow snap" or smth like this | 21:57 |
| mriedem | i <3 trump | 21:57 |
| _gryf | ;P | 21:57 |
| bauzas | mriedem: I mean, I'm working now because I could see a wall next week between you and me | 21:57 |
| stewie925_ | hi all, I am fix a merge conflict for one of my nova notificaion commits. After rebasing I ran the functional test and it shows the following erro: No authentication information found for placement API. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading. | 21:59 |
| *** gouthamr has joined #openstack-nova | 22:01 | |
| *** thorst_ has quit IRC | 22:02 | |
| stewie925_ | oops - think I may have found something... brb | 22:02 |
| *** thorst_ has joined #openstack-nova | 22:03 | |
| melwitt | mriedem: a-ha, coolness on the d-g change | 22:05 |
| *** thorst_ has quit IRC | 22:07 | |
| mriedem | melwitt: i'm told in -infra that discover_hosts needs to be moved into post-stack for grenade | 22:08 |
| mriedem | but that can happen after we see if this d-g run is good | 22:08 |
| melwitt | okay | 22:08 |
| mriedem | and then either you or dansmith will have to do that | 22:08 |
| mriedem | bauzas: i'll have email. | 22:08 |
| mriedem | bauzas: and there are other cores around next week too | 22:08 |
| bauzas | mriedem: np, enjoy your time-off | 22:09 |
| bauzas | mriedem: I'm just done with trying to fix all the placement issues | 22:09 |
| mriedem | i'm not sure what that means | 22:09 |
| mriedem | i thought you just needed to pass in the 1.4 microversion | 22:09 |
| stewie925_ | thought I found something with the placement api error ... but I didnt :( | 22:10 |
| bauzas | mriedem: I *could* do that easily now with the direct get call, but I prefer to update the ReportClient by having a min and max versions | 22:12 |
| bauzas | I'd have actually by far preferred we hadn't gone that way with providing directly the header in the get method but meh. | 22:13 |
| melwitt | mriedem: so for non-grenade, it stays where it is, for grenade it moves right after the run of stack.sh | 22:15 |
| melwitt | or does this mean it moves out of d-g for grenade and into the grenade repo? | 22:16 |
| sdague | melwitt: so... it could be either honestly | 22:16 |
| melwitt | okay. just making sure I understand what to do if I have to do it | 22:16 |
| sdague | it needs to happen after the post-stack.sh run | 22:16 |
| sdague | that's where d-g puts the subnode orchestration commands | 22:17 |
| sdague | https://github.com/openstack-dev/grenade/blob/489202bd707866de9abc25a46a3f31c10e939076/grenade.sh#L244 | 22:18 |
| sdague | either way, a NOTE should be added about this being a work around for ocata until there is some auto registration for the single node case | 22:18 |
| sdague | sorry, single cell case | 22:18 |
| melwitt | I had just now found that, thanks. I think I understand now | 22:19 |
| sdague | cool, ok off to dinner, have a good weekend folks | 22:21 |
| mriedem | bauzas: yeah version discovery is obviously better | 22:22 |
| mriedem | things are just tight | 22:22 |
| *** tbachman has joined #openstack-nova | 22:22 | |
| *** slaweq has quit IRC | 22:22 | |
| mriedem | bauzas: so maybe work with sdague and jaypipes on the best way forward there on monday | 22:22 |
| mriedem | bauzas: we're pretty close though and we've already said you need placement for n-sch in ocata, | 22:22 |
| mriedem | so it's going to have to be there, | 22:22 |
| *** slaweq_ has quit IRC | 22:23 | |
| mriedem | so i think requesting 1.4 and handling the error if it's not at 1.4 (newton placement) with an ERROR and NoValidHost is expected behavior given the ML discussion | 22:23 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209 | 22:23 |
| bauzas | mriedem: fair point | 22:23 |
| bauzas | mriedem: given the gate, maybe just directly calling 1.4 is the quickiest one | 22:24 |
| mriedem | bauzas: which reminds me that we're going to need a docs update in addition to reno in your change, or immediately following it, | 22:24 |
| mriedem | i.e. we need to update the upgrade docs devref to make a note that placement is required for n-sch in ocata, and should be upgraded first or at the same time | 22:24 |
| bauzas | roger | 22:24 |
| mriedem | bauzas: yes i think we do 1.4 | 22:24 |
| *** burgerk has quit IRC | 22:25 | |
| *** esberglu has quit IRC | 22:26 | |
| *** armax has quit IRC | 22:26 | |
| *** esberglu has joined #openstack-nova | 22:26 | |
| *** tblakes has quit IRC | 22:27 | |
| *** esberglu has quit IRC | 22:30 | |
| melwitt | mriedem: is your latest comment saying we might not need discover_hosts at all for multi-node grenade? | 22:34 |
| *** jheroux has quit IRC | 22:34 | |
| *** jamesdenton has quit IRC | 22:35 | |
| aunnam | stewie925_, I looked at your patch, by looking at the logs I think the placement api is not the problem. Commented on your patch. | 22:36 |
| stewie925_ | aunnam: thanks I'll check it out | 22:36 |
| *** tbachman has quit IRC | 22:36 | |
| mriedem | melwitt: it says i'm not totally sure :) | 22:37 |
| mriedem | we'll have to see | 22:37 |
| melwitt | okay | 22:38 |
| mriedem | alright i'm signing off, ttyl | 22:39 |
| melwitt | k, have nice time off | 22:39 |
| *** mriedem has quit IRC | 22:39 | |
| *** salv-orlando has joined #openstack-nova | 22:40 | |
| *** jerrygb has joined #openstack-nova | 22:40 | |
| *** armax has joined #openstack-nova | 22:42 | |
| *** thorst_ has joined #openstack-nova | 22:43 | |
| *** salv-orlando has quit IRC | 22:44 | |
| *** dave-mccowan has quit IRC | 22:46 | |
| bauzas | any idea why the gate is consistently resetting ? | 22:46 |
| *** thorst_ has quit IRC | 22:47 | |
| *** burt has quit IRC | 22:52 | |
| *** mdrabe has quit IRC | 22:57 | |
| *** masber has quit IRC | 22:59 | |
| *** dtp has joined #openstack-nova | 22:59 | |
| *** masber has joined #openstack-nova | 23:00 | |
| *** damien_r has quit IRC | 23:04 | |
| *** xyang1 has quit IRC | 23:14 | |
| *** thorst_ has joined #openstack-nova | 23:15 | |
| *** thorst_ has quit IRC | 23:20 | |
| *** annegentle has quit IRC | 23:20 | |
| *** _ducttape_ has quit IRC | 23:22 | |
| *** figleaf is now known as edleafe | 23:31 | |
| *** salv-orlando has joined #openstack-nova | 23:41 | |
| *** armax has quit IRC | 23:41 | |
| *** lpetrut has quit IRC | 23:44 | |
| *** salv-orlando has quit IRC | 23:45 | |
| *** tbachman has joined #openstack-nova | 23:50 | |
| *** edmondsw has quit IRC | 23:53 | |
| *** edmondsw has joined #openstack-nova | 23:54 | |
| *** dimtruck is now known as zz_dimtruck | 23:58 | |
| *** sacharya has joined #openstack-nova | 23:58 | |
| *** edmondsw has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!