| *** jamielennox is now known as jamielennox|away | 00:00 | |
| *** mlavalle has quit IRC | 00:01 | |
| *** tongli has quit IRC | 00:02 | |
| *** davidlenwell has quit IRC | 00:03 | |
| *** jamielennox|away is now known as jamielennox | 00:14 | |
| *** ducttape_ has joined #openstack-nova | 00:14 | |
| openstackgerrit | dane-fichter proposed openstack/nova: Use cursive for signature verification https://review.openstack.org/351232 | 00:15 |
|---|---|---|
| *** rmcall has quit IRC | 00:15 | |
| *** rmcall has joined #openstack-nova | 00:16 | |
| *** davidlenwell has joined #openstack-nova | 00:16 | |
| *** BjoernT has quit IRC | 00:20 | |
| *** vishwanathj has quit IRC | 00:22 | |
| *** kaisers has joined #openstack-nova | 00:24 | |
| *** jaypipes has quit IRC | 00:24 | |
| *** kaisers3 has joined #openstack-nova | 00:24 | |
| *** kaisers2 has quit IRC | 00:25 | |
| *** kaisers1 has quit IRC | 00:26 | |
| *** bdeschenes has quit IRC | 00:32 | |
| *** Sukhdev has joined #openstack-nova | 00:35 | |
| *** bdeschenes has joined #openstack-nova | 00:36 | |
| *** mtanino has quit IRC | 00:37 | |
| *** tjones has joined #openstack-nova | 00:38 | |
| *** tjones has left #openstack-nova | 00:38 | |
| *** bdeschenes has quit IRC | 00:39 | |
| *** ducttape_ has quit IRC | 00:42 | |
| *** bdeschenes has joined #openstack-nova | 00:43 | |
| *** tonytan4ever has joined #openstack-nova | 00:48 | |
| *** hoangcx has joined #openstack-nova | 00:48 | |
| *** tonytan4ever has quit IRC | 00:52 | |
| *** psilvad has joined #openstack-nova | 00:54 | |
| *** mriedem has joined #openstack-nova | 00:58 | |
| *** cheneydc has joined #openstack-nova | 00:59 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| *** cheneydc has quit IRC | 01:04 | |
| *** pwnall1 has quit IRC | 01:08 | |
| *** tbachman has quit IRC | 01:08 | |
| openstackgerrit | Leehom Li proposed openstack/nova: Support detach interface with same MAC from instance https://review.openstack.org/372243 | 01:12 |
| *** bdeschenes has quit IRC | 01:13 | |
| *** tjones has joined #openstack-nova | 01:18 | |
| *** tjones has left #openstack-nova | 01:18 | |
| *** tjones has joined #openstack-nova | 01:19 | |
| *** sdake_ has joined #openstack-nova | 01:19 | |
| *** mtanino has joined #openstack-nova | 01:20 | |
| *** sdake has quit IRC | 01:21 | |
| *** ducttape_ has joined #openstack-nova | 01:23 | |
| *** amotoki has joined #openstack-nova | 01:26 | |
| *** unicell has quit IRC | 01:27 | |
| *** jwcroppe has joined #openstack-nova | 01:29 | |
| *** tjones has left #openstack-nova | 01:30 | |
| *** tjones has joined #openstack-nova | 01:32 | |
| *** tjones has left #openstack-nova | 01:32 | |
| *** kaisers_ has joined #openstack-nova | 01:33 | |
| *** Jeffrey4l has joined #openstack-nova | 01:37 | |
| *** kaisers_ has quit IRC | 01:38 | |
| *** salv-orl_ has joined #openstack-nova | 01:42 | |
| *** yuntongjin has joined #openstack-nova | 01:44 | |
| *** salv-orlando has quit IRC | 01:44 | |
| *** stewie925_ has quit IRC | 01:48 | |
| *** tonytan4ever has joined #openstack-nova | 01:49 | |
| *** avolkov has quit IRC | 01:50 | |
| *** avolkov` has joined #openstack-nova | 01:50 | |
| *** baoli has joined #openstack-nova | 01:51 | |
| *** tonytan4ever has quit IRC | 01:54 | |
| *** RuiChen has left #openstack-nova | 01:58 | |
| *** yuntongjin has quit IRC | 01:58 | |
| *** Apoorva_ has joined #openstack-nova | 02:01 | |
| *** psilvad has quit IRC | 02:01 | |
| *** ducttape_ has quit IRC | 02:03 | |
| *** Apoorva has quit IRC | 02:05 | |
| *** dimtruck is now known as zz_dimtruck | 02:05 | |
| *** Apoorva_ has quit IRC | 02:05 | |
| *** tuanla has joined #openstack-nova | 02:05 | |
| *** zz_dimtruck is now known as dimtruck | 02:15 | |
| *** ducttape_ has joined #openstack-nova | 02:18 | |
| *** mriedem has quit IRC | 02:19 | |
| *** hemanthm|afk is now known as hemanthm | 02:23 | |
| *** tonytan4ever has joined #openstack-nova | 02:23 | |
| *** tjones has joined #openstack-nova | 02:24 | |
| *** amotoki has quit IRC | 02:25 | |
| *** tjones has left #openstack-nova | 02:28 | |
| *** sfinucan has quit IRC | 02:29 | |
| *** rama_y has quit IRC | 02:31 | |
| *** ducttape_ has quit IRC | 02:31 | |
| *** sfinucan has joined #openstack-nova | 02:31 | |
| *** ducttape_ has joined #openstack-nova | 02:35 | |
| *** yuntongjin has joined #openstack-nova | 02:38 | |
| *** mdrabe has quit IRC | 02:40 | |
| *** mdrabe has joined #openstack-nova | 02:40 | |
| *** amotoki has joined #openstack-nova | 02:41 | |
| *** ducttape_ has quit IRC | 02:41 | |
| *** amotoki has quit IRC | 02:49 | |
| *** amotoki has joined #openstack-nova | 02:49 | |
| *** priteau has joined #openstack-nova | 02:53 | |
| *** amotoki has quit IRC | 02:54 | |
| *** tbachman has joined #openstack-nova | 02:56 | |
| *** amotoki has joined #openstack-nova | 02:58 | |
| *** priteau has quit IRC | 02:58 | |
| openstackgerrit | huangtianhua proposed openstack/nova: Set 'last_checked' flag if start to check scheduler file https://review.openstack.org/344144 | 03:01 |
| *** baoli has quit IRC | 03:01 | |
| *** xiaoyang has quit IRC | 03:02 | |
| *** david-lyle has quit IRC | 03:04 | |
| *** amotoki has quit IRC | 03:04 | |
| *** tbachman_ has joined #openstack-nova | 03:10 | |
| *** tbachman has quit IRC | 03:12 | |
| *** tbachman_ is now known as tbachman | 03:12 | |
| *** kaisers_ has joined #openstack-nova | 03:22 | |
| *** armax has quit IRC | 03:23 | |
| *** tbachman_ has joined #openstack-nova | 03:24 | |
| *** armax has joined #openstack-nova | 03:25 | |
| *** tbachman has quit IRC | 03:25 | |
| *** tbachman_ is now known as tbachman | 03:25 | |
| *** kaisers_ has quit IRC | 03:26 | |
| *** ducttape_ has joined #openstack-nova | 03:27 | |
| *** dikonoor has joined #openstack-nova | 03:33 | |
| *** ijw has quit IRC | 03:40 | |
| *** sdake_ has quit IRC | 03:42 | |
| *** ijw_ has joined #openstack-nova | 03:45 | |
| *** ducttape_ has quit IRC | 03:46 | |
| *** ducttape_ has joined #openstack-nova | 03:46 | |
| *** amotoki has joined #openstack-nova | 03:46 | |
| *** ijw_ has quit IRC | 03:49 | |
| *** unicell has joined #openstack-nova | 03:50 | |
| *** tbachman has quit IRC | 03:51 | |
| *** ducttape_ has quit IRC | 03:59 | |
| *** lamt has quit IRC | 04:01 | |
| *** kaisers_ has joined #openstack-nova | 04:02 | |
| *** _ducttape_ has joined #openstack-nova | 04:02 | |
| *** markvoelker has quit IRC | 04:03 | |
| *** yingjun_ has joined #openstack-nova | 04:04 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova: Use raw disk format as default when create image with glance v2 https://review.openstack.org/366825 | 04:05 |
| *** markvoelker has joined #openstack-nova | 04:10 | |
| *** bdeschenes has joined #openstack-nova | 04:11 | |
| *** ijw has joined #openstack-nova | 04:14 | |
| *** _ducttape_ has quit IRC | 04:14 | |
| *** dimtruck is now known as zz_dimtruck | 04:16 | |
| *** zz_dimtruck is now known as dimtruck | 04:16 | |
| *** _ducttape_ has joined #openstack-nova | 04:16 | |
| *** ababich has quit IRC | 04:16 | |
| *** _ducttape_ has quit IRC | 04:17 | |
| *** ababich has joined #openstack-nova | 04:17 | |
| *** ijw has quit IRC | 04:19 | |
| *** rama_y has joined #openstack-nova | 04:19 | |
| *** ababich has quit IRC | 04:21 | |
| *** ababich has joined #openstack-nova | 04:21 | |
| *** bdeschenes has quit IRC | 04:25 | |
| *** dimtruck is now known as zz_dimtruck | 04:25 | |
| openstackgerrit | Leehom Li proposed openstack/nova: Support detach interface with same MAC from instance https://review.openstack.org/372243 | 04:30 |
| *** mdrabe has quit IRC | 04:30 | |
| *** tuanla has quit IRC | 04:35 | |
| *** ababich has quit IRC | 04:35 | |
| *** tuanla has joined #openstack-nova | 04:36 | |
| *** ababich has joined #openstack-nova | 04:36 | |
| *** psachin has joined #openstack-nova | 04:38 | |
| *** psachin has quit IRC | 04:40 | |
| *** psachin has joined #openstack-nova | 04:41 | |
| *** hemna_ has joined #openstack-nova | 04:43 | |
| *** hemna_ has quit IRC | 04:44 | |
| *** psachin has quit IRC | 04:45 | |
| *** psachin has joined #openstack-nova | 04:47 | |
| *** jaosorior has joined #openstack-nova | 04:50 | |
| *** garyk1 has joined #openstack-nova | 04:52 | |
| *** ducttape_ has joined #openstack-nova | 04:56 | |
| *** code-R has joined #openstack-nova | 04:56 | |
| *** ducttape_ has quit IRC | 05:02 | |
| *** prateek has joined #openstack-nova | 05:07 | |
| *** ratailor has joined #openstack-nova | 05:07 | |
| *** sc68cal_ has joined #openstack-nova | 05:09 | |
| *** woodster_ has quit IRC | 05:10 | |
| *** jaosorior has quit IRC | 05:10 | |
| *** jaosorior has joined #openstack-nova | 05:11 | |
| *** sc68cal has quit IRC | 05:11 | |
| *** jwcroppe has quit IRC | 05:11 | |
| *** jwcroppe has joined #openstack-nova | 05:12 | |
| *** pbandark has joined #openstack-nova | 05:14 | |
| *** claudiub|2 has joined #openstack-nova | 05:18 | |
| *** moshele has joined #openstack-nova | 05:21 | |
| *** wuhg has joined #openstack-nova | 05:22 | |
| openstackgerrit | edan david proposed openstack/nova: Add missing slash to dir path https://review.openstack.org/372558 | 05:24 |
| *** lamt has joined #openstack-nova | 05:39 | |
| *** unicell has quit IRC | 05:39 | |
| *** esracelik has joined #openstack-nova | 05:40 | |
| *** esracelik is now known as ecelik | 05:40 | |
| *** unicell has joined #openstack-nova | 05:41 | |
| *** ociuhandu has quit IRC | 05:42 | |
| *** unicell1 has joined #openstack-nova | 05:42 | |
| *** unicell has quit IRC | 05:43 | |
| *** rama_y has quit IRC | 05:45 | |
| *** gongysh has joined #openstack-nova | 05:47 | |
| *** paul-carlton2 has joined #openstack-nova | 05:47 | |
| *** Jeffrey4l has quit IRC | 05:49 | |
| *** Jeffrey4l has joined #openstack-nova | 05:51 | |
| *** code-R has quit IRC | 05:51 | |
| *** lpetrut has joined #openstack-nova | 05:54 | |
| *** code-R has joined #openstack-nova | 05:58 | |
| *** ducttape_ has joined #openstack-nova | 05:59 | |
| *** oanson has joined #openstack-nova | 06:00 | |
| *** markvoelker has quit IRC | 06:01 | |
| *** markvoelker has joined #openstack-nova | 06:03 | |
| *** ducttape_ has quit IRC | 06:04 | |
| *** oanson has quit IRC | 06:05 | |
| *** rcernin has joined #openstack-nova | 06:07 | |
| *** code-R_ has joined #openstack-nova | 06:09 | |
| *** markvoelker has quit IRC | 06:11 | |
| *** mjura has joined #openstack-nova | 06:11 | |
| *** code-R has quit IRC | 06:12 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/373709 | 06:13 |
| *** bhagyashri has joined #openstack-nova | 06:13 | |
| *** tonytan4ever has quit IRC | 06:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/373715 | 06:15 |
| *** mjura has quit IRC | 06:17 | |
| *** mjura has joined #openstack-nova | 06:17 | |
| *** garyk1 has quit IRC | 06:21 | |
| *** adisky has quit IRC | 06:22 | |
| *** sudipto_ has joined #openstack-nova | 06:23 | |
| *** sudswas__ has joined #openstack-nova | 06:23 | |
| *** namnh has joined #openstack-nova | 06:25 | |
| *** ociuhandu has joined #openstack-nova | 06:26 | |
| *** ociuhandu has quit IRC | 06:31 | |
| *** andreas_s has joined #openstack-nova | 06:32 | |
| *** yingjun_ has quit IRC | 06:32 | |
| *** ihrachys has joined #openstack-nova | 06:33 | |
| *** karthiks has joined #openstack-nova | 06:37 | |
| *** ljxiash has joined #openstack-nova | 06:38 | |
| *** oanson has joined #openstack-nova | 06:39 | |
| *** ManishD has joined #openstack-nova | 06:42 | |
| *** code-R_ has quit IRC | 06:45 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/373709 | 06:46 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/373772 | 06:47 |
| *** sahid has joined #openstack-nova | 06:48 | |
| openstackgerrit | sonu proposed openstack/nova: Add hacking checks for xrange() https://review.openstack.org/205307 | 06:51 |
| *** lpetrut has quit IRC | 06:52 | |
| openstackgerrit | Hussain Chachuliya proposed openstack/nova: Replace uuid4() with generate_uuid() from oslo_utils https://review.openstack.org/373794 | 06:54 |
| ManishD | HI Nova: please can someone let me know where do we configure the frequency of running the "periodic tasks" and what may be preferred value | 06:54 |
| *** ababich has quit IRC | 06:55 | |
| *** ababich has joined #openstack-nova | 06:56 | |
| openstackgerrit | Claudiu Belu proposed openstack/os-vif: Adds NoopPlugin https://review.openstack.org/362876 | 06:58 |
| *** ducttape_ has joined #openstack-nova | 07:00 | |
| *** namnh has quit IRC | 07:02 | |
| *** ratailor has quit IRC | 07:03 | |
| *** namnh has joined #openstack-nova | 07:03 | |
| openstackgerrit | sonu proposed openstack/nova: Add hacking checks for xrange() https://review.openstack.org/205307 | 07:04 |
| *** ducttape_ has quit IRC | 07:05 | |
| *** zenoway has joined #openstack-nova | 07:05 | |
| *** Sukhdev has quit IRC | 07:06 | |
| avolkov` | 4rfvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvvv7\ | 07:08 |
| *** markvoelker has joined #openstack-nova | 07:11 | |
| *** ababich has quit IRC | 07:12 | |
| *** armax has quit IRC | 07:12 | |
| *** pcaruana has joined #openstack-nova | 07:13 | |
| *** ababich has joined #openstack-nova | 07:13 | |
| *** markvoelker has quit IRC | 07:16 | |
| *** ratailor has joined #openstack-nova | 07:17 | |
| *** claudiub|2 has quit IRC | 07:19 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix serial console not correctly defined after live-migration https://review.openstack.org/356335 | 07:19 |
| *** matrohon has joined #openstack-nova | 07:20 | |
| *** acabot has joined #openstack-nova | 07:23 | |
| *** ociuhandu has joined #openstack-nova | 07:23 | |
| *** jlanoux has joined #openstack-nova | 07:27 | |
| *** mtanino has quit IRC | 07:27 | |
| *** ociuhandu has quit IRC | 07:28 | |
| *** sridharg has joined #openstack-nova | 07:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/373709 | 07:31 |
| *** zenoway has quit IRC | 07:31 | |
| *** yingjun has joined #openstack-nova | 07:32 | |
| *** yingjun has quit IRC | 07:32 | |
| *** zenoway has joined #openstack-nova | 07:32 | |
| *** hshiina has joined #openstack-nova | 07:33 | |
| *** ababich has quit IRC | 07:33 | |
| *** ababich has joined #openstack-nova | 07:34 | |
| *** kaisers_ has quit IRC | 07:34 | |
| *** code-R has joined #openstack-nova | 07:37 | |
| *** zenoway has quit IRC | 07:38 | |
| *** null0 has joined #openstack-nova | 07:38 | |
| *** zenoway has joined #openstack-nova | 07:40 | |
| *** salv-orlando has joined #openstack-nova | 07:42 | |
| *** tonytan4ever has joined #openstack-nova | 07:44 | |
| *** salv-orl_ has quit IRC | 07:45 | |
| *** code-R has quit IRC | 07:46 | |
| *** pilgrimstack has joined #openstack-nova | 07:49 | |
| *** tonytan4ever has quit IRC | 07:50 | |
| *** roeyc has joined #openstack-nova | 07:54 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** code-R has joined #openstack-nova | 08:02 | |
| *** gongysh has quit IRC | 08:03 | |
| *** ababich has quit IRC | 08:05 | |
| *** ababich has joined #openstack-nova | 08:05 | |
| *** gongysh has joined #openstack-nova | 08:06 | |
| *** danpb has joined #openstack-nova | 08:09 | |
| *** yassine has joined #openstack-nova | 08:09 | |
| *** yassine is now known as Guest98774 | 08:10 | |
| *** code-R has quit IRC | 08:11 | |
| *** ababich has quit IRC | 08:12 | |
| *** ababich has joined #openstack-nova | 08:12 | |
| *** paul-carlton2 has quit IRC | 08:14 | |
| *** null0 has quit IRC | 08:16 | |
| *** ababich has quit IRC | 08:18 | |
| *** ababich has joined #openstack-nova | 08:18 | |
| *** ababich has quit IRC | 08:22 | |
| *** ababich has joined #openstack-nova | 08:22 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Should not be able to sort instances based on joined tables https://review.openstack.org/359134 | 08:25 |
| *** jaosorior is now known as jaosorior_brb | 08:28 | |
| *** claudiub has joined #openstack-nova | 08:28 | |
| *** paul-carlton2 has joined #openstack-nova | 08:29 | |
| *** liverpooler has joined #openstack-nova | 08:31 | |
| *** alexchadin has joined #openstack-nova | 08:36 | |
| *** ociuhandu has joined #openstack-nova | 08:38 | |
| *** diga has joined #openstack-nova | 08:42 | |
| *** ababich has quit IRC | 08:43 | |
| *** ociuhandu has quit IRC | 08:43 | |
| *** ababich has joined #openstack-nova | 08:43 | |
| *** lpetrut has joined #openstack-nova | 08:43 | |
| *** derekh has joined #openstack-nova | 08:44 | |
| *** ababich has quit IRC | 08:50 | |
| *** ababich has joined #openstack-nova | 08:51 | |
| *** paul-carlton2 has quit IRC | 08:53 | |
| *** paul-carlton2 has joined #openstack-nova | 08:55 | |
| *** hshiina has quit IRC | 08:58 | |
| BobBall | johnthetubaguy: Does Rackspace use anything under https://github.com/openstack/nova/tree/master/plugins/xenserver/networking/etc ? | 08:58 |
| danpb | do we have any etherpad where we maintain a list of specs that are re-reproposed from previous release, so reviewers can concentrate on those ? | 08:59 |
| *** mvk has quit IRC | 09:00 | |
| *** abalutoiu has quit IRC | 09:00 | |
| *** efoley_ has joined #openstack-nova | 09:04 | |
| *** cdent has joined #openstack-nova | 09:05 | |
| *** roeyc has quit IRC | 09:05 | |
| *** efoley__ has joined #openstack-nova | 09:06 | |
| *** ababich has quit IRC | 09:08 | |
| *** ababich has joined #openstack-nova | 09:08 | |
| *** ociuhandu has joined #openstack-nova | 09:08 | |
| *** efoley_ has quit IRC | 09:10 | |
| *** ociuhandu has quit IRC | 09:13 | |
| *** ababich has quit IRC | 09:13 | |
| sfinucan | mikal: Could you give a quick scoot around for patches -2d because of the newton merge window | 09:14 |
| sfinucan | ...like this one https://review.openstack.org/#/c/205307/ | 09:14 |
| *** ababich has joined #openstack-nova | 09:15 | |
| *** bks has joined #openstack-nova | 09:16 | |
| openstackgerrit | Sharat Sharma proposed openstack/nova: Replace MagicMock with Mock class https://review.openstack.org/373940 | 09:16 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: add event lifecycle details https://review.openstack.org/373943 | 09:17 |
| *** ociuhandu has joined #openstack-nova | 09:17 | |
| openstackgerrit | Sharat Sharma proposed openstack/python-novaclient: Replace MagicMock with Mock class https://review.openstack.org/373946 | 09:20 |
| *** kaisers_ has joined #openstack-nova | 09:23 | |
| *** kaisers_ has quit IRC | 09:27 | |
| *** diga has quit IRC | 09:29 | |
| *** code-R has joined #openstack-nova | 09:30 | |
| johnthetubaguy | BobBall: unsure, I suspect we don't use any of that now | 09:31 |
| johnthetubaguy | BobBall: I think we use the custom set of stuff from quark | 09:31 |
| *** mvk has joined #openstack-nova | 09:31 | |
| *** code-R_ has joined #openstack-nova | 09:31 | |
| johnthetubaguy | danpb: if folks have correctly put the Git commit tag in, I found a way to search for those in gerrit | 09:32 |
| johnthetubaguy | danpb: I will try dig that up | 09:32 |
| BobBall | johnthetubaguy: Could you confirm? A goodly chunk of it looks very legacy and someone has found it and started to use it when we don't think it's needed | 09:33 |
| *** zenoway has quit IRC | 09:33 | |
| johnthetubaguy | BobBall: I am 90% sure we use the quark scripts now, I don't think we use those in the devstack setup, so I agree, don't see why they would be needed | 09:33 |
| *** zenoway has joined #openstack-nova | 09:34 | |
| oomichi | mriedem: bauzas: yeah I heard this kind of hardship related to dev schedule yesterday in QA mid-cycle and they have posted pre-approved https://review.openstack.org/#/c/373054/ for Ocata cycle | 09:34 |
| *** code-R has quit IRC | 09:34 | |
| BobBall | ok, I suggest we remove them then | 09:36 |
| johnthetubaguy | danpb: probably lots of false positives, but this seems to work rather well: https://review.openstack.org/#/q/project:openstack/nova-specs+Previously-approved | 09:37 |
| johnthetubaguy | danpb: whoops, https://review.openstack.org/#/q/project:openstack/nova-specs+status:open+Previously-approved | 09:38 |
| *** zenoway has quit IRC | 09:38 | |
| *** zenoway has joined #openstack-nova | 09:41 | |
| *** jwcroppe has quit IRC | 09:42 | |
| *** jwcroppe has joined #openstack-nova | 09:42 | |
| *** alexchadin has quit IRC | 09:44 | |
| *** pilgrimstack has quit IRC | 09:46 | |
| *** jwcroppe has quit IRC | 09:46 | |
| *** alexchadin has joined #openstack-nova | 09:48 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hardware: Rework docstrings https://review.openstack.org/367468 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Update references for split up hardware module https://review.openstack.org/368968 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hardware: Standardize '_get_XXX_constraints' functions https://review.openstack.org/367469 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hardware: Flatten functions https://review.openstack.org/367470 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hardware: Split into modules https://review.openstack.org/367467 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Split 'test_hardware' into modules https://review.openstack.org/368967 | 09:49 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Extend nfv feature matrix with pinning/NUMA https://review.openstack.org/327126 | 09:50 |
| *** pilgrimstack has joined #openstack-nova | 09:51 | |
| *** roeyc has joined #openstack-nova | 09:55 | |
| openstackgerrit | javeme proposed openstack/nova: Failed to evacuate instances booted from volume https://review.openstack.org/372045 | 09:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: plugins/xenserver: Add '.py' extension https://review.openstack.org/289431 | 09:57 |
| *** abalutoiu has joined #openstack-nova | 09:59 | |
| *** paul-carlton2 has quit IRC | 10:00 | |
| *** raorn_ is now known as raorn | 10:00 | |
| *** rdo_ has quit IRC | 10:02 | |
| *** gszasz has joined #openstack-nova | 10:03 | |
| *** rdo_ has joined #openstack-nova | 10:04 | |
| *** yuntongjin has quit IRC | 10:11 | |
| *** alexchadin has quit IRC | 10:13 | |
| *** PaulMurray has joined #openstack-nova | 10:15 | |
| *** oanson has quit IRC | 10:19 | |
| sfinucan | johnthetubaguy: Is there any plan to remove configurable AMQP topics? | 10:22 |
| sfinucan | I've seen it come up in reviews time and time again, but the config options are yet to be deprecated | 10:22 |
| *** paul-carlton2 has joined #openstack-nova | 10:23 | |
| johnthetubaguy | sfinucan: yeah, I think we should, I can't remember who had the patch up for that now | 10:25 |
| *** liuyulong__ has joined #openstack-nova | 10:25 | |
| sfinucan | johnthetubaguy: Found it https://review.openstack.org/#/c/351608/ I might take that over so | 10:26 |
| sfinucan | unless you plan to work on it more, macsz? | 10:26 |
| *** bdeschenes has joined #openstack-nova | 10:27 | |
| *** liuyulong has quit IRC | 10:28 | |
| *** brian_price has quit IRC | 10:28 | |
| johnthetubaguy | I think macsz was just waiting for the -2 to get lifted | 10:29 |
| johnthetubaguy | but worth double checking | 10:29 |
| macsz | sfinucan johnthetubaguy: oh yes, i lost this patch and totally forgot about it, it's not listed as mine in review so i always skipped it and waited until its fixed | 10:31 |
| macsz | sfinucan johnthetubaguy: will address comments and fix this today | 10:31 |
| sfinucan | macsz: cool cool | 10:31 |
| johnthetubaguy | oh, right, I split an old patch in two, my bad | 10:31 |
| *** phuongnh has quit IRC | 10:35 | |
| *** hoangcx has quit IRC | 10:36 | |
| *** sdague has joined #openstack-nova | 10:37 | |
| *** tuanla has quit IRC | 10:41 | |
| *** smatzek has joined #openstack-nova | 10:43 | |
| *** tonytan4ever has joined #openstack-nova | 10:46 | |
| *** sahid has quit IRC | 10:46 | |
| *** Jeffrey4l has quit IRC | 10:50 | |
| *** Jeffrey4l has joined #openstack-nova | 10:50 | |
| *** tonytan4ever has quit IRC | 10:52 | |
| *** claudiub has quit IRC | 10:54 | |
| *** gongysh has quit IRC | 10:56 | |
| *** code-R_ has quit IRC | 10:56 | |
| *** code-R has joined #openstack-nova | 10:56 | |
| *** davidlenwell has quit IRC | 10:57 | |
| *** claudiub has joined #openstack-nova | 10:57 | |
| *** bdeschenes has quit IRC | 10:58 | |
| *** davidlenwell has joined #openstack-nova | 10:58 | |
| openstackgerrit | Shoham Peller proposed openstack/nova-specs: Cleanup running shelved-offloaded instances https://review.openstack.org/373371 | 11:01 |
| *** namnh has quit IRC | 11:07 | |
| *** ijw has joined #openstack-nova | 11:07 | |
| *** rtheis has joined #openstack-nova | 11:08 | |
| *** PaulMurray has quit IRC | 11:11 | |
| *** ijw has quit IRC | 11:12 | |
| *** kaisers_ has joined #openstack-nova | 11:12 | |
| *** ecelik has quit IRC | 11:14 | |
| openstackgerrit | Cao ShuFeng proposed openstack/nova-specs: Add support booting baremetals from Cinder volume https://review.openstack.org/311696 | 11:15 |
| *** baoli has joined #openstack-nova | 11:15 | |
| *** kaisers_ has quit IRC | 11:16 | |
| *** baoli_ has joined #openstack-nova | 11:18 | |
| *** lucas-afk is now known as lucasagomes | 11:18 | |
| sfinucan | danpb: I'm updating the libvirt-numa guide to reference Fedora 24. Any reason you install under $HOME instead of '/opt/stack'? | 11:19 |
| *** bvanhav has joined #openstack-nova | 11:19 | |
| danpb | sfinucan: that's an inappropriate location for personal development files | 11:20 |
| *** baoli has quit IRC | 11:21 | |
| sfinucan | danpb: Ok, but it does make the guide a little longer that need be. Would you be averse to me sticking to '/opt'? | 11:24 |
| *** psilvad has joined #openstack-nova | 11:25 | |
| *** jed56 has quit IRC | 11:25 | |
| *** rmart04 has joined #openstack-nova | 11:26 | |
| *** jaosorior_brb is now known as jaosorior | 11:28 | |
| danpb | sfinucan: imho that's not even noticable difference in length - a couple of lines in a doc that's almost 700 lines long | 11:29 |
| claudiub | danpb: hello. any chance you can drop your -2 on this patch? https://review.openstack.org/#/c/140045/ | 11:30 |
| *** liverpooler has quit IRC | 11:30 | |
| *** sudswas__ has quit IRC | 11:30 | |
| *** sudipto_ has quit IRC | 11:30 | |
| *** ociuhandu has quit IRC | 11:32 | |
| *** bdeschenes has joined #openstack-nova | 11:34 | |
| rmart04 | rmart04: Liberty:Nova / Could anyone shed any light on how to remove old migraiton entires? There have been previous migraition errors that have ended up in terminating instances. Now both compute nodes are trying to check to see if the instances are there, but failing. Annoying logs! | 11:36 |
| rmart04 | [12:28pm] rmart04: Thinking its likely to be a db hackfest? | 11:36 |
| *** mtanino has joined #openstack-nova | 11:37 | |
| *** ociuhandu has joined #openstack-nova | 11:38 | |
| *** daemontool has joined #openstack-nova | 11:39 | |
| *** woodster_ has joined #openstack-nova | 11:40 | |
| *** code-R_ has joined #openstack-nova | 11:42 | |
| *** code-R has quit IRC | 11:45 | |
| *** artom has joined #openstack-nova | 11:46 | |
| *** ljxiash has quit IRC | 11:47 | |
| *** sahid has joined #openstack-nova | 11:47 | |
| *** edmondsw has joined #openstack-nova | 11:49 | |
| sfinucan | sdague: johnthetubaguy: There are two patches that just got +W but are going to conflict as they do the same thing https://review.openstack.org/#/c/350085 https://review.openstack.org/#/c/373262 | 11:50 |
| sfinucan | seeing as there are far more dependencies on the latter, can one of you remove the +W on the former to stop it going through? | 11:51 |
| sfinucan | (I should have -1 the first one myself, but I forgot :() | 11:51 |
| openstackgerrit | jichenjc proposed openstack/nova: Add more description when service delete https://review.openstack.org/374061 | 11:51 |
| *** pilgrimstack has quit IRC | 11:52 | |
| sfinucan | johnthetubaguy: wrong one :) | 11:52 |
| *** mtanino has quit IRC | 11:53 | |
| johnthetubaguy | which one? | 11:53 |
| sfinucan | johnthetubaguy: actually, I was looking at the wrong patch #needsmoretea | 11:54 |
| sfinucan | this one https://review.openstack.org/#/c/350085/5 | 11:54 |
| openstackgerrit | Sean Dague proposed openstack/nova: Change disable_live_snapshot workaround https://review.openstack.org/373430 | 11:54 |
| sfinucan | (the one without the 'conf' prefix) | 11:54 |
| johnthetubaguy | oh, I was trying to find that after I +2ed the other one, and failed | 11:54 |
| johnthetubaguy | well, I did something, but I am not sure it helped | 11:55 |
| sfinucan | johnthetubaguy: np. Will just mean another reordering patch if that one gets in | 11:56 |
| sfinucan | spent *all day* yesterday on that silly file :( | 11:57 |
| *** bks has quit IRC | 11:57 | |
| sdague | danpb: I got a bit lost in the unit tests on https://review.openstack.org/373430 because the live path is very different than the cold path (and all the snapshot tests assume cold path) | 11:57 |
| sdague | so that just forces the tests to be on the old path that were there before | 11:57 |
| sdague | danpb: we can probably take the risk of merge and just keep an eye on the stats to see if the issue exposes again | 11:58 |
| *** lamt has quit IRC | 11:58 | |
| *** amotoki has quit IRC | 11:58 | |
| *** rodrigods has quit IRC | 11:59 | |
| *** rodrigods has joined #openstack-nova | 11:59 | |
| *** coolsvap_ is now known as coolsvap | 12:00 | |
| danpb | sdague: hmm, the original change turning off live snapshots by default didn't touch unit tests at all | 12:01 |
| openstackgerrit | javeme proposed openstack/nova: Failed to evacuate instances booted from volume https://review.openstack.org/372045 | 12:01 |
| *** wuhg has quit IRC | 12:01 | |
| sdague | danpb: well, you can remove the flags line and see | 12:01 |
| *** NightKhaos has quit IRC | 12:03 | |
| *** pilgrimstack has joined #openstack-nova | 12:04 | |
| *** efoley_ has joined #openstack-nova | 12:05 | |
| *** PaulMurray has joined #openstack-nova | 12:06 | |
| *** efoley__ has quit IRC | 12:07 | |
| *** NightKhaos has joined #openstack-nova | 12:09 | |
| openstackgerrit | Alex Xu proposed openstack/nova: api-ref: Complete all the verifications of remote consoles https://review.openstack.org/338051 | 12:10 |
| *** ratailor has quit IRC | 12:11 | |
| *** ljxiash has joined #openstack-nova | 12:11 | |
| *** ManishD has quit IRC | 12:15 | |
| *** ljxiash has quit IRC | 12:15 | |
| *** annegentle has joined #openstack-nova | 12:17 | |
| *** kylek3h has joined #openstack-nova | 12:17 | |
| *** ducttape_ has joined #openstack-nova | 12:17 | |
| *** maticue has joined #openstack-nova | 12:18 | |
| openstackgerrit | jichenjc proposed openstack/nova: Add more description when service delete https://review.openstack.org/374061 | 12:18 |
| *** wznoinsk has quit IRC | 12:21 | |
| *** kylek3h has quit IRC | 12:23 | |
| *** wznoinsk has joined #openstack-nova | 12:28 | |
| *** lamt has joined #openstack-nova | 12:29 | |
| *** mdrabe has joined #openstack-nova | 12:30 | |
| *** lamt has quit IRC | 12:31 | |
| *** lamt has joined #openstack-nova | 12:32 | |
| *** markvoelker has joined #openstack-nova | 12:33 | |
| *** gcb has quit IRC | 12:33 | |
| *** egarbade has quit IRC | 12:34 | |
| *** efoley_ has quit IRC | 12:36 | |
| *** egarbade has joined #openstack-nova | 12:36 | |
| *** ducttape_ has quit IRC | 12:36 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: fix serial console not correctly defined after live-migration https://review.openstack.org/356335 | 12:37 |
| raj_singh | johnthetubaguy: PaulMurray: I missed your ping yesterday from LM meeting. Sorry I was out to sort out my visa for the summit. Yes I will work with pkoniszewski to help out on LM job. | 12:39 |
| PaulMurray | raj_singh, thanks - no worries about the meeting. Just wanted to check with you | 12:40 |
| raj_singh | PaulMurray: In general I am interested in LM work, so I will start my contributions to it in this cycle. | 12:41 |
| *** dane-fichter has joined #openstack-nova | 12:43 | |
| PaulMurray | that's good - thanks raj_singh | 12:44 |
| dane-fichter | anyone know when mriedem is usually on? | 12:44 |
| *** zenoway has quit IRC | 12:44 | |
| PaulMurray | Do you want me to add you to the ping list for the LM IRC meeting | 12:44 |
| raj_singh | I already added myself. I have been listening to last few meetings to get the context. | 12:45 |
| *** zenoway has joined #openstack-nova | 12:45 | |
| PaulMurray | oh - you're already there - ignore | 12:45 |
| *** eharney has joined #openstack-nova | 12:46 | |
| prateek | dane-fichter, in an hour maybe | 12:47 |
| *** annegentle has quit IRC | 12:47 | |
| prateek | johnthetubaguy, can you please review https://review.openstack.org/#/c/294513/ ? | 12:47 |
| johnthetubaguy | sfinucan: I was thinking about all those topic configs, I recon we should not move them into groups for now, we should probably just kill them | 12:48 |
| johnthetubaguy | sfinucan: by kill, I mean deprecate | 12:48 |
| dane-fichter | prateek: thanks. is there anyone else who can approve spec-less blueprints in the mean time | 12:48 |
| sfinucan | johnthetubaguy: Yeah, I've that done. Just waiting for tests to run now and validate | 12:48 |
| sfinucan | Deprecated the json conf file too, and removed the (useless) code using it | 12:48 |
| johnthetubaguy | sfinucan: sweet, there was another patch, consoleauth I think where there was something similar | 12:49 |
| dane-fichter | prateek: nvm it's actually been approved already | 12:49 |
| sfinucan | johnthetubaguy: also my patch :) I'll move onto that next | 12:49 |
| *** zenoway has quit IRC | 12:49 | |
| alex_xu | nova-api subteam meeting is in 10 mins at #openstack-meeting-4 | 12:49 |
| dane-fichter | anyone want to review a spec for some security functionality? https://review.openstack.org/#/c/357151/ | 12:50 |
| *** PaulMurray has quit IRC | 12:50 | |
| prateek | sdague, regarding your comment on https://review.openstack.org/#/c/327706/, if I remove the new exception , that would mean just catching the excception and raising with an error message and 500 , is it ? | 12:51 |
| prateek | alex_xu, need your opinion on this API patch ? | 12:51 |
| prateek | can you spare some time ? | 12:52 |
| sdague | prateek: yes | 12:52 |
| prateek | sdague, ok | 12:52 |
| sdague | prateek: oh... there was a different alternative I was thinking about for this as well | 12:52 |
| sdague | which is return a 206, and leave the security groups blank | 12:52 |
| sdague | so indicate that only partial result is returned | 12:53 |
| prateek | sdague, that one sounds good | 12:53 |
| *** ldeptula has joined #openstack-nova | 12:53 | |
| sdague | that would be a thing we'd need an API bump for, because it's new semantics | 12:53 |
| sdague | but that could be used in a similar pattern with things like glance / cinder | 12:53 |
| *** mriedem has joined #openstack-nova | 12:54 | |
| prateek | sdague, i am not sure how to bump API's but i would figure that out and when the exception is catched somehow we would have to return empty columns | 12:55 |
| prateek | thanks for the suggestion sdague | 12:55 |
| sdague | prateek: if we are going down that path, we should build a spec for it | 12:55 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Improve consistency in scheduler opts https://review.openstack.org/349666 | 12:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Group scheduler options https://review.openstack.org/349959 | 12:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Remove support for deprecated driver import https://review.openstack.org/339760 | 12:56 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate 'scheduler_json_file_location' https://review.openstack.org/374121 | 12:56 |
| dane-fichter | danpb: this spec could really benefit from your input https://review.openstack.org/#/c/357151/ | 12:56 |
| prateek | mriedem, can you please take a look at https://review.openstack.org/#/c/294513/ | 12:56 |
| *** david-lyle has joined #openstack-nova | 12:56 | |
| prateek | sdague, ok, i would try building a spec for it then | 12:56 |
| *** PaulMurray has joined #openstack-nova | 12:56 | |
| prateek | or if you want to write that and maybe i can implement that | 12:57 |
| *** roeyc has quit IRC | 12:57 | |
| prateek | sdague, ^ | 12:57 |
| *** krtaylor has joined #openstack-nova | 12:57 | |
| sdague | prateek: ok, I'll put it on my list. Will try to draft something this week | 12:57 |
| sfinucan | johnthetubaguy: ...and we're done https://review.openstack.org/#/c/349666 (plus others in 'Related changes') | 12:57 |
| prateek | sdague, thanks ! | 12:58 |
| *** roeyc has joined #openstack-nova | 12:59 | |
| *** roeyc has quit IRC | 13:00 | |
| *** liuyulong has joined #openstack-nova | 13:00 | |
| alex_xu | nova api meeting is running | 13:00 |
| *** sudswas__ has joined #openstack-nova | 13:01 | |
| *** sudipto_ has joined #openstack-nova | 13:01 | |
| openstackgerrit | dane-fichter proposed openstack/nova: Use cursive for signature verification https://review.openstack.org/351232 | 13:01 |
| *** kaisers_ has joined #openstack-nova | 13:01 | |
| dane-fichter | mriedem: any chance you could remove the -2 from https://review.openstack.org/351232 ? | 13:02 |
| mriedem | dane-fichter: the bp isn't approved yet | 13:02 |
| *** liuyulong__ has quit IRC | 13:02 | |
| mriedem | is there a cross-project spec for that or something? | 13:02 |
| dane-fichter | mriedem : there are multiple specs | 13:02 |
| *** amotoki has joined #openstack-nova | 13:03 | |
| dane-fichter | here's the blueprint https://blueprints.launchpad.net/nova/+spec/signature-code-cleanup | 13:03 |
| *** zz_dimtruck is now known as dimtruck | 13:03 | |
| dane-fichter | here's the cross project spec: https://review.openstack.org/#/c/314718/ | 13:03 |
| dane-fichter | not for the code cleanup, just for signature verification in general | 13:04 |
| dane-fichter | here's the nova one for cert validation: https://review.openstack.org/#/c/357151/ | 13:04 |
| *** pilgrimstack has quit IRC | 13:04 | |
| *** pilgrimstack has joined #openstack-nova | 13:05 | |
| *** roeyc has joined #openstack-nova | 13:05 | |
| *** vishwanathj has joined #openstack-nova | 13:06 | |
| *** rlrossit has joined #openstack-nova | 13:06 | |
| dane-fichter | mriedem: please feel free to ask questions | 13:06 |
| *** kaisers_ has quit IRC | 13:06 | |
| mriedem | ok, in the nova-api meeting atm | 13:06 |
| *** PaulMurray has quit IRC | 13:06 | |
| mriedem | maybe put this one on the nova weekly meeting open agenda section for awareness? | 13:06 |
| mriedem | as a specless bp | 13:06 |
| *** PaulMurray has joined #openstack-nova | 13:07 | |
| dane-fichter | mriedem: the code cleanup one? | 13:07 |
| *** mtanino has joined #openstack-nova | 13:07 | |
| dane-fichter | mriedem: link to the agenda | 13:08 |
| dane-fichter | ?? | 13:08 |
| *** dimtruck is now known as zz_dimtruck | 13:08 | |
| *** zz_dimtruck is now known as dimtruck | 13:08 | |
| *** jaosorior has quit IRC | 13:10 | |
| dane-fichter | mriedem: nvm found it. | 13:10 |
| *** sudipto_ has quit IRC | 13:11 | |
| *** sudswas__ has quit IRC | 13:11 | |
| *** jaosorior has joined #openstack-nova | 13:11 | |
| *** sudswas__ has joined #openstack-nova | 13:11 | |
| *** sudipto_ has joined #openstack-nova | 13:11 | |
| *** tbachman has joined #openstack-nova | 13:11 | |
| mriedem | dane-fichter: do we have any integration testing in the gate, like in tempest, that tests any of this? | 13:11 |
| mriedem | or does barbican have tests that would test this indirectly once it's in nova? | 13:11 |
| dane-fichter | mriedem: integration tests for what specifically? image signature verification? | 13:12 |
| dane-fichter | mriedem: currently there's no tempest scenario for this. | 13:13 |
| *** liverpooler has joined #openstack-nova | 13:13 | |
| mriedem | correct | 13:13 |
| mriedem | can we build a tempest test for this? | 13:13 |
| mriedem | to test it before refactoring all of that code to use an external library> | 13:13 |
| mriedem | ? | 13:13 |
| dane-fichter | mriedem: the library contains an exact copy of what's in Nova/Glance currently (see signature_utils.py) | 13:14 |
| dane-fichter | mriedem: to be clear, I think a tempest test is definitely something that should be done | 13:15 |
| dane-fichter | mriedem: but I don't necessarily think this code cleanup should block on it | 13:16 |
| *** tbachman has quit IRC | 13:16 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Deprecate all topic conf options https://review.openstack.org/351608 | 13:16 |
| *** hshiina has joined #openstack-nova | 13:16 | |
| *** tbachman has joined #openstack-nova | 13:17 | |
| mriedem | dane-fichter: there is a subtle difference in where the testing happens | 13:18 |
| mriedem | once it's outside of nova in a library, nova isn't testing it anymore | 13:18 |
| *** dimtruck is now known as zz_dimtruck | 13:18 | |
| mriedem | so changes in the library could break nova | 13:18 |
| mriedem | because nova is just going to mock the api calls to the library | 13:18 |
| mriedem | so i'd like to see some progress on a test being added to tempest | 13:18 |
| prateek | mriedem, johnthetubaguy, very sorry for interrupting in the meeting, i was writing on IRC and didn't notice the group i was writing in :/ | 13:19 |
| dane-fichter | mriedem: ok, I will pass that up the chain. unsure if we have the bandwidth to take that on as well but we will try | 13:19 |
| *** tbachman has quit IRC | 13:20 | |
| johnthetubaguy | prateek: it happens, sometimes people type their password | 13:20 |
| prateek | johnthetubaguy, hahaha | 13:20 |
| dane-fichter | password1234 | 13:20 |
| johnthetubaguy | prateek: my code was just an example of what I was thinking, feel free to make fix it | 13:20 |
| *** tbachman_ has joined #openstack-nova | 13:20 | |
| prateek | johnthetubaguy, yeah, gave me nice pointers, i am building something on it | 13:21 |
| *** psachin has quit IRC | 13:21 | |
| *** gouthamr has joined #openstack-nova | 13:21 | |
| *** psachin has joined #openstack-nova | 13:23 | |
| openstackgerrit | Christopher Martin proposed openstack/nova: Fix for resize vm_state stopped rollback https://review.openstack.org/370861 | 13:24 |
| *** psilvad has quit IRC | 13:24 | |
| *** psilvad has joined #openstack-nova | 13:25 | |
| *** READ10 has joined #openstack-nova | 13:25 | |
| *** ducttape_ has joined #openstack-nova | 13:27 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova: Fix check_config_option_in_central_place https://review.openstack.org/355597 | 13:29 |
| openstackgerrit | Prateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted https://review.openstack.org/294513 | 13:29 |
| openstackgerrit | Prateek Arora proposed openstack/nova: Delete traces of in-progress snapshot on VM being deleted https://review.openstack.org/294513 | 13:32 |
| *** roeyc has quit IRC | 13:33 | |
| *** kaisers_ has joined #openstack-nova | 13:33 | |
| *** ljxiash has joined #openstack-nova | 13:33 | |
| openstackgerrit | edan david proposed openstack/nova: WIP: User-controlled SR-IOV ports allocation https://review.openstack.org/374151 | 13:33 |
| *** code-R_ has quit IRC | 13:34 | |
| *** kb has joined #openstack-nova | 13:35 | |
| *** code-R has joined #openstack-nova | 13:36 | |
| *** ijw has joined #openstack-nova | 13:38 | |
| *** prateek has quit IRC | 13:39 | |
| *** tblakes has joined #openstack-nova | 13:40 | |
| *** kb has quit IRC | 13:41 | |
| *** kberger has joined #openstack-nova | 13:41 | |
| *** salv-orl_ has joined #openstack-nova | 13:42 | |
| *** ijw has quit IRC | 13:43 | |
| *** salv-orlando has quit IRC | 13:45 | |
| *** tblakes has quit IRC | 13:46 | |
| *** lamt has quit IRC | 13:47 | |
| *** xyang1 has joined #openstack-nova | 13:48 | |
| *** baoli_ has quit IRC | 13:48 | |
| mriedem | ayoung: did you ever push a nova change for adding a readme on how to generate the default policy? | 13:49 |
| *** esracelik has joined #openstack-nova | 13:49 | |
| *** esracelik is now known as ecelik | 13:49 | |
| *** ducttape_ has quit IRC | 13:50 | |
| ayoung | mriedem, not yet | 13:51 |
| ayoung | mriedem, but I did write this | 13:51 |
| *** tri2sing has joined #openstack-nova | 13:51 | |
| ayoung | http://adam.younglogic.com/2016/09/distinct-rbac-policy-rules/ | 13:51 |
| *** tonytan4ever has joined #openstack-nova | 13:52 | |
| *** diga_ has joined #openstack-nova | 13:53 | |
| *** kaisers_ has quit IRC | 13:53 | |
| *** tongli has joined #openstack-nova | 13:54 | |
| *** akshai has joined #openstack-nova | 13:55 | |
| *** lifeless has quit IRC | 13:57 | |
| *** r-daneel has joined #openstack-nova | 13:58 | |
| *** rlrossit has quit IRC | 13:58 | |
| *** sdake has joined #openstack-nova | 13:59 | |
| *** lifeless has joined #openstack-nova | 13:59 | |
| *** jaypipes has joined #openstack-nova | 13:59 | |
| *** sdake has quit IRC | 13:59 | |
| *** sdake has joined #openstack-nova | 14:00 | |
| *** ecelik has quit IRC | 14:00 | |
| *** jwcroppe has joined #openstack-nova | 14:01 | |
| *** kaisers_ has joined #openstack-nova | 14:02 | |
| *** efried has quit IRC | 14:03 | |
| *** tanvirt has joined #openstack-nova | 14:03 | |
| *** paul-carlton2 has quit IRC | 14:04 | |
| *** paul-carlton2 has joined #openstack-nova | 14:05 | |
| *** andymaier has joined #openstack-nova | 14:06 | |
| *** burt has joined #openstack-nova | 14:07 | |
| *** tblakes has joined #openstack-nova | 14:08 | |
| *** Swami has joined #openstack-nova | 14:08 | |
| *** Swami_ has joined #openstack-nova | 14:08 | |
| *** Swami_ has quit IRC | 14:08 | |
| *** Swami has quit IRC | 14:08 | |
| *** Swami has joined #openstack-nova | 14:09 | |
| *** marst has quit IRC | 14:09 | |
| *** cmartin has joined #openstack-nova | 14:10 | |
| *** rama_y has joined #openstack-nova | 14:11 | |
| *** annegentle has joined #openstack-nova | 14:11 | |
| mriedem | gmann: do you plan on updating this? https://review.openstack.org/#/c/299830/ | 14:12 |
| mriedem | i just left some more comments | 14:12 |
| mriedem | i could also rebase it and try to move it forward | 14:13 |
| *** pbandark has quit IRC | 14:16 | |
| *** dikonoor has quit IRC | 14:18 | |
| alaski | ayoung: btw, Nova can be used with oslopolicy-sample-generator as well. They do slightly different things. sample-generator spits out the rules registered in code, policy-generator merges those rules with an existing policy.json|yaml and spits it out | 14:18 |
| alaski | it's all documented in oslo.policy devref | 14:19 |
| *** marst has joined #openstack-nova | 14:22 | |
| *** ducttape_ has joined #openstack-nova | 14:23 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Change disable_live_snapshot workaround https://review.openstack.org/373430 | 14:23 |
| sdague | the last patch there had a bad rebase in it | 14:24 |
| mriedem | alaski: oo http://docs.openstack.org/developer/oslo.policy/usage.html#registering-policy-defaults-in-code | 14:25 |
| mriedem | oslopolicy-sample-generator --namespace nova.api --namespace nova.compute.api --output-file policy-sample.yaml | 14:25 |
| mriedem | alaski: can the namespace just be 'nova'? | 14:25 |
| mriedem | to be simpler? | 14:25 |
| alaski | yeah. in real life it is. I should update that example | 14:26 |
| *** pradk has quit IRC | 14:26 | |
| alaski | http://git.openstack.org/cgit/openstack/nova/tree/setup.cfg#n42 | 14:26 |
| *** yamahata has joined #openstack-nova | 14:28 | |
| *** mfisch has quit IRC | 14:28 | |
| mriedem | ok cool | 14:30 |
| *** psachin has quit IRC | 14:30 | |
| *** gszasz has quit IRC | 14:31 | |
| *** kylek3h has joined #openstack-nova | 14:31 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove support for deprecated driver import https://review.openstack.org/339760 | 14:31 |
| *** jed56 has joined #openstack-nova | 14:32 | |
| *** tblakes has quit IRC | 14:32 | |
| *** mfisch has joined #openstack-nova | 14:32 | |
| *** mfisch has quit IRC | 14:32 | |
| *** mfisch has joined #openstack-nova | 14:32 | |
| *** jlanoux has quit IRC | 14:32 | |
| *** abalutoiu has quit IRC | 14:32 | |
| *** moshele has quit IRC | 14:33 | |
| *** zz_dimtruck is now known as dimtruck | 14:33 | |
| *** jheroux has joined #openstack-nova | 14:35 | |
| *** pilgrimstack has quit IRC | 14:36 | |
| ayoung | alaski, excellent. I can see that being really useful, especially if we come up with a set of common policy rules..like a definition for context_is_admin | 14:38 |
| *** tblakes has joined #openstack-nova | 14:38 | |
| edleafe | johnthetubaguy: ^^ Added the reno | 14:39 |
| johnthetubaguy | edleafe: great, thank you | 14:39 |
| *** akshai has quit IRC | 14:40 | |
| mriedem | edleafe: wrong section for that reno | 14:41 |
| mriedem | should be 'upgrades' | 14:41 |
| edleafe | mriedem: really? ok | 14:41 |
| mriedem | yeah, you're not deprecating the thing | 14:41 |
| mriedem | you're removing the deprecated thing | 14:41 |
| *** daemontool_ has joined #openstack-nova | 14:41 | |
| *** mlavalle has joined #openstack-nova | 14:42 | |
| dane-fichter | mriedem: can you elaborate more on what you mean by a "happy path" test? | 14:43 |
| *** akshai has joined #openstack-nova | 14:43 | |
| mriedem | dane-fichter: boot a server with a signed image? | 14:44 |
| mriedem | happy path as in basic scenario | 14:44 |
| *** daemontool has quit IRC | 14:44 | |
| dane-fichter | mriedem: Ok. Looking into that now. Unsure if Tempest can currently support a key manager which can store a cert | 14:44 |
| *** cleong has joined #openstack-nova | 14:45 | |
| *** gszasz has joined #openstack-nova | 14:46 | |
| *** rlrossit has joined #openstack-nova | 14:47 | |
| openstackgerrit | Sean Dague proposed openstack/nova: Fix invalid exception mock for InvalidNUMANodesNumber https://review.openstack.org/374213 | 14:47 |
| edleafe | mriedem: upgraded :) | 14:48 |
| *** burgerk has joined #openstack-nova | 14:48 | |
| *** brian_price has joined #openstack-nova | 14:48 | |
| *** akshai has quit IRC | 14:49 | |
| mriedem | dane-fichter: hmm, i'm assuming barbican has any tempest tests in a tempest plugin somewhere | 14:50 |
| mriedem | this is what sucks about barbican being the only key manager in openstack, but is totally optional | 14:50 |
| mriedem | and the integrated gate isn't going to install/configure barbican | 14:50 |
| dane-fichter | mriedem: agreed. is there any way to mock out something behind castellan in tempest to just to return a cert? | 14:50 |
| mriedem | don't know | 14:51 |
| *** ihrachys has quit IRC | 14:52 | |
| *** akshai has joined #openstack-nova | 14:52 | |
| *** EinstCrazy has joined #openstack-nova | 14:52 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova: Remove check_attach https://review.openstack.org/335358 | 14:52 |
| *** PaulMurray has quit IRC | 14:53 | |
| *** baoli has joined #openstack-nova | 14:54 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Remove logging during plugin initialization https://review.openstack.org/374225 | 14:55 |
| sdague | danpb: going through and looking at how os-vif was interacting with nova and tests, I realized the call order on os-vif means logging is getting called before it's setup | 14:55 |
| sdague | above is one approach to being more clear, the other is changing how os-vif is called from nova | 14:56 |
| *** EinstCrazy has quit IRC | 14:56 | |
| *** daemontool_ has quit IRC | 14:58 | |
| mriedem | sdague: target that to close bug 1615676 please | 14:58 |
| openstack | bug 1615676 in OpenStack Compute (nova) "os-vif log messages corrupt and pollute test output" [Wishlist,In progress] https://launchpad.net/bugs/1615676 - Assigned to Pushkar Umaranikar (pushkar-umaranikar) | 14:58 |
| *** openstack has joined #openstack-nova | 15:01 | |
| alaski | cool. I was hoping that was the case | 15:01 |
| mriedem | could also be good low hanging fruit for someone new | 15:01 |
| danpb | sdague:are you sure about that - i swear i've actually see that logging message appear in the logs before | 15:02 |
| *** ducttape_ has quit IRC | 15:03 | |
| danpb | sdague: we could just move the os_vif.initialize() call into the nova compute startup path instead | 15:04 |
| sdague | danpb: it's definitely in a very odd place | 15:04 |
| sdague | danpb: yeh, that would be better | 15:04 |
| sdague | I'm fine doing that patch instead | 15:04 |
| danpb | i think i'd prefer the nova fix - as it'll be important to see logs about which plugins are loaded when we start getting out of tree vendor plugins | 15:05 |
| sdague | danpb: things get really funny when you effectively depend on module load order that oslo.log will get loaded and configured before this | 15:05 |
| sdague | danpb: ok, I can do that | 15:05 |
| *** abalutoiu has joined #openstack-nova | 15:05 | |
| sdague | we should probably then tune this down to debug | 15:05 |
| danpb | yeah, i can see that | 15:05 |
| sdague | regardless | 15:05 |
| sdague | danpb: so just put this in init_host? | 15:06 |
| danpb | i tend to think it is useful at INFO level - its the kind of thing i'd want to know when debugging even if log level doesn't enable DEBUG | 15:07 |
| danpb | sdague: i was actually thinking about nova/cmd/compute.py on the basis that we'll use it in all virt drivers eventually | 15:07 |
| danpb | sdague: and AFAIK, it shouldn't cause harm to initialize it even for drivers that are not yet converted | 15:07 |
| *** diga_ has quit IRC | 15:07 | |
| *** rrecio has joined #openstack-nova | 15:08 | |
| sdague | INFO should definitely be a lot more end user friendly. Like a single line saying the following network plugins are loaded. | 15:10 |
| danpb | sdague: sure, i'd take a patch which moved the log call out of the loop, so it just reported all plugins at once | 15:12 |
| sdague | danpb: ok, I'll respin that other patch | 15:13 |
| sdague | https://review.openstack.org/374233 | 15:13 |
| *** andymaier has quit IRC | 15:14 | |
| sdague | danpb: the reason this works in the gate, is that we're dumping to stdout (and then scraping from there), which is kind of the default. If you had configured nova to go to a log file or syslog or some other location I think the messages would have been lost | 15:14 |
| danpb | ah, that makes sense | 15:14 |
| *** lamt has joined #openstack-nova | 15:15 | |
| mriedem | mtreinish: are you still around? | 15:15 |
| *** pwnall1 has joined #openstack-nova | 15:17 | |
| *** jichen has joined #openstack-nova | 15:18 | |
| jichen | mriedem: not sure whether it's you talked about in API meeting https://review.openstack.org/#/c/357884/ about v3 diag, just added you and Alex as reviewer, FYI | 15:19 |
| *** gszasz has quit IRC | 15:20 | |
| alex_xu | jichen: yes, that is what we talk about | 15:20 |
| mriedem | jichen: thanks | 15:21 |
| jichen | np | 15:21 |
| alex_xu | jichen: thanks too, will try to review it | 15:21 |
| *** rlrossit has quit IRC | 15:23 | |
| jichen | alex_xu: thought you already sleep :) | 15:23 |
| *** EinstCrazy has joined #openstack-nova | 15:24 | |
| alex_xu | jichen: yea, but soon :) | 15:24 |
| *** omnipresent has joined #openstack-nova | 15:24 | |
| mriedem | jroll: how possible would it be for the ironic virt driver to implement the diagnostics API? | 15:25 |
| *** hieulq_ has joined #openstack-nova | 15:25 | |
| mriedem | is there anything in the ironic api to provide stats for a given node? | 15:25 |
| *** EinstCrazy has quit IRC | 15:26 | |
| *** rlrossit has joined #openstack-nova | 15:26 | |
| *** EinstCrazy has joined #openstack-nova | 15:27 | |
| *** efried has joined #openstack-nova | 15:27 | |
| *** wznoinsk has quit IRC | 15:28 | |
| *** sridharg has quit IRC | 15:28 | |
| *** andreas_s has quit IRC | 15:28 | |
| *** Guest98774 has quit IRC | 15:28 | |
| *** jamesden_ has joined #openstack-nova | 15:28 | |
| *** liuyulong has quit IRC | 15:29 | |
| *** pwnall1 has quit IRC | 15:29 | |
| *** liuyulong has joined #openstack-nova | 15:29 | |
| *** mvk has quit IRC | 15:31 | |
| sfinucan | Could I get another core to look (and maybe +W) at this config option patch? https://review.openstack.org/#/c/353631 | 15:31 |
| sfinucan | It's rather trivial, but it touches a lot of files and will likely remain in a perpetual state of merge conflict otherwise | 15:31 |
| *** rama_y has quit IRC | 15:31 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Make plugin loading more consistent with logging guidelines. https://review.openstack.org/374225 | 15:31 |
| *** Guest98774 has joined #openstack-nova | 15:32 | |
| mriedem | snikitin: comments in https://review.openstack.org/#/c/357884/ but overall it looks good | 15:32 |
| sdague | danpb: ^^^ is the logging tweak, see what you think | 15:32 |
| *** gszasz has joined #openstack-nova | 15:33 | |
| *** _ducttape_ has quit IRC | 15:33 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Improve consistency in scheduler opts https://review.openstack.org/349666 | 15:33 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Group scheduler options https://review.openstack.org/349959 | 15:33 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Deprecate 'scheduler_json_file_location' https://review.openstack.org/374121 | 15:33 |
| *** ducttape_ has joined #openstack-nova | 15:33 | |
| mriedem | sfinucan: i'll take a look at that one - the cross reference with the commits isn't going to be fun though - but thanks for having that in there | 15:34 |
| *** lucasagomes is now known as lucas-hungry | 15:35 | |
| sfinucan | mriedem: Lovely, thank you | 15:35 |
| rlrossit | I'm not familiar with how service workers work, but in my change I'm playing around with adding scheduler workers (https://review.openstack.org/#/c/372622/), and I was wondering, if I have 2 scheduler workers, that means I have a parent and 2 workers, will I have 3 "schedulers" that will accept requests, or will only the workers handle requests? | 15:36 |
| sdague | rlrossit: only the workers | 15:36 |
| rlrossit | awesome, thanks sdague | 15:37 |
| sdague | the parent is just for process management | 15:37 |
| rlrossit | was hoping that was the case | 15:37 |
| rlrossit | and from what I could read out of oslo_service, that's what I thought was happening | 15:37 |
| *** jichen has quit IRC | 15:37 | |
| *** maticue has quit IRC | 15:37 | |
| *** maticue has joined #openstack-nova | 15:38 | |
| sdague | rlrossit: out of curiosity, why do that vs just spawning multiple schedulers on different nodes? | 15:38 |
| *** dane-fichter has quit IRC | 15:38 | |
| *** jaosorior has quit IRC | 15:39 | |
| *** tri2sing has quit IRC | 15:40 | |
| *** tri2sing has joined #openstack-nova | 15:40 | |
| *** hemna_ has joined #openstack-nova | 15:41 | |
| *** hemna_ has quit IRC | 15:41 | |
| jaypipes | alaski, alex_xu, sdague, mriedem, dansmith: should be an easy win: https://review.openstack.org/#/c/373505 | 15:41 |
| mriedem | i was leaving that for dan | 15:42 |
| jaypipes | alaski, alex_xu, sdague, mriedem, dansmith: next patch in this series add generic PCI devices to the inventory/allocation tracking | 15:42 |
| dansmith | I don't really see the point of that, but... okay :) | 15:43 |
| alex_xu | mriedem: dansmith oops, I just +w... | 15:43 |
| alex_xu | dansmith: I can remove the +w | 15:44 |
| dansmith | alex_xu: no need | 15:44 |
| alaski | alex_xu: I would've +W'ed | 15:44 |
| alex_xu | dansmith: alaski ok | 15:44 |
| jaypipes | dansmith: object methods that don't reference the object or relate to the purpose of the object shouldn't be object methods... | 15:44 |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Skip malformed cookies https://review.openstack.org/346090 | 15:44 |
| mriedem | gerrit is being slooooow | 15:44 |
| dansmith | jaypipes: yeah, I mean I know the rhetoric, but .. still :) | 15:45 |
| jaypipes | dansmith: sorry, suppose I'm a pedant. | 15:45 |
| *** bks has joined #openstack-nova | 15:45 | |
| *** rama_y has joined #openstack-nova | 15:47 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Add server admin actions to general purpose feature matrix https://review.openstack.org/345708 | 15:48 |
| *** amotoki has quit IRC | 15:49 | |
| rlrossit | sdague: does being a lazy bum count? I just want lots of schedulers (across 2 nodes), but don't feel like starting lots of processes on my own (even via scripting) | 15:49 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Expand feature matrix with server admin actions https://review.openstack.org/362408 | 15:49 |
| openstackgerrit | Chris Dent proposed openstack/nova: [WIP] [placement] Manage log and other output in gabbi fixure https://review.openstack.org/374256 | 15:51 |
| *** kumarb has joined #openstack-nova | 15:51 | |
| *** bks has quit IRC | 15:52 | |
| *** paul-carlton2 has quit IRC | 15:52 | |
| mriedem | this is a completely random question, but you know how we have the shared storage check in the virt drivers (at least libvirt) for migration? it creates the temp file and checks it exists and all that? couldn't we just perform that once and store the result in memory per nova-compute service? | 15:53 |
| mriedem | https://review.openstack.org/#/c/366857/ made me think of it | 15:54 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Renamed parameters name in config.py https://review.openstack.org/374262 | 15:54 |
| ftersin | mriedem: afaik shared storage is an attribute of src-dest hosts pair | 15:56 |
| alex_xu | mriedem: after we have resource-provider support shared storage pool, then we needn't that check anymore? | 15:56 |
| mriedem | jaypipes: did you cross check all of those deprecated_since entries? | 15:57 |
| mriedem | i was in the middle of going through those when you +w'ed | 15:57 |
| johnthetubaguy | mriedem: yeah, could catch it for a specific pair of hosts | 15:57 |
| johnthetubaguy | s/catch/cache/ | 15:58 |
| *** anush has joined #openstack-nova | 15:59 | |
| dansmith | mriedem: getting ready to do placeholder migrations for ocata | 15:59 |
| *** EinstCrazy has quit IRC | 15:59 | |
| mriedem | jaypipes: please drop the +W on this https://review.openstack.org/#/c/353631/10 | 15:59 |
| dansmith | mriedem: and then one for blocking on previous online migrations that haven't completed so we can drop that code too | 15:59 |
| dansmith | just FYI | 15:59 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Add schedule_and_build_instances conductor method https://review.openstack.org/367557 | 16:00 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Tests: improve assertJsonEqual diagnostic message https://review.openstack.org/372644 | 16:00 |
| *** matrohon has quit IRC | 16:00 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Return build_requests instead of instances https://review.openstack.org/374267 | 16:00 |
| jaypipes | mriedem: done | 16:03 |
| *** moshele has joined #openstack-nova | 16:03 | |
| *** rcernin has quit IRC | 16:03 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 16:03 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: WIP Make schedule_and_build_instances cell aware https://review.openstack.org/372645 | 16:03 |
| jaypipes | mriedem: sorry bout that. | 16:05 |
| *** mat128|mtg is now known as mat128 | 16:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add deprecated_since parameter https://review.openstack.org/353631 | 16:06 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add placeholder DB migrations for Ocata https://review.openstack.org/374270 | 16:07 |
| mriedem | all fixed up | 16:07 |
| mriedem | dansmith: ok | 16:07 |
| mriedem | jaypipes: np | 16:07 |
| mriedem | ffs, maybe we should consider dropping that option that's been deprecated since juno... | 16:08 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Reduce duplication and complexity in format_dom https://review.openstack.org/232145 | 16:08 |
| *** baoli has quit IRC | 16:08 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Virtual guest device role tagging https://review.openstack.org/373354 | 16:08 |
| *** baoli has joined #openstack-nova | 16:09 | |
| mriedem | artom: i'm confused about https://review.openstack.org/#/c/373354/ | 16:10 |
| mriedem | the blueprint for that was marked complete in newton | 16:10 |
| mriedem | so re-proposing it for ocata is odd | 16:10 |
| artom | mriedem, dunno why it's marked as complete | 16:11 |
| *** pilgrimstack has quit IRC | 16:11 | |
| artom | We still need to finish attach/detach of tagged interfaces/volumes | 16:11 |
| artom | dansmith was on my ass for that ;) | 16:11 |
| mriedem | we've marked blueprints complete for less, at least partially complete | 16:12 |
| mriedem | i think we'd at least want a new blueprint for ocata to finish the attach/detach work | 16:12 |
| mriedem | i don't think we need to re-propose the spec though | 16:12 |
| mriedem | maybe just call it virt-device-role-tagging-ocata | 16:12 |
| mriedem | or something | 16:12 |
| artom | mriedem, OK, I wasn't sure what the process is for this kind of case though | 16:12 |
| artom | There'll be a new microversion | 16:13 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] Manage log and other output in gabbi fixure https://review.openstack.org/374256 | 16:13 |
| mriedem | artom: oh right, new microversion | 16:13 |
| *** slaweq_ has joined #openstack-nova | 16:14 | |
| mriedem | ok so let's re-propose the spec but work against an ocata specific blueprint | 16:15 |
| mriedem | i know it's weird | 16:15 |
| *** pcaruana has quit IRC | 16:15 | |
| mriedem | that's what happens when things are partially complete | 16:15 |
| artom | mriedem, sounds good. I'll create a bp called virt-device-tagged-attach-detach or something, and update the spec to that topic | 16:16 |
| mriedem | ok | 16:16 |
| mriedem | thanks | 16:16 |
| mriedem | if you hand dan a small child he'll detach from your ass | 16:17 |
| mriedem | btw | 16:17 |
| *** pcaruana has joined #openstack-nova | 16:17 | |
| dansmith | mriedem: artom threatens me with small children quite often via video chat | 16:18 |
| dansmith | mriedem: my therapist and I are working through it | 16:18 |
| artom | mriedem, see that's the thing, if the child is tagged he can't detach yet, hence the ocata spec/bp | 16:18 |
| woodster_ | mriedem: dane-fichter I asked around about barbican integration testing. There is an integration test available on the puppet-openstack-integration side that demonstrates volume encryption via cinder...here's a CR that activates it: https://review.openstack.org/#/c/363142/ This might be an example to consider for gate testing. Also note that Tempest calls | 16:19 |
| woodster_ | to barbican to configure its own devstack tests, hence there are no barbican tests in the Tempest code base itself. | 16:19 |
| *** moshele has quit IRC | 16:19 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Virtual guest device role tagging https://review.openstack.org/373354 | 16:23 |
| *** devananda|dinner is now known as devananda | 16:23 | |
| artom | Blah, forgot to update the bp link at the top | 16:24 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova-specs: Virtual guest device role tagging https://review.openstack.org/373354 | 16:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: trivial: Rewrap guide at 79 characters https://review.openstack.org/374279 | 16:27 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: doc: Update libvirt-numa guide https://review.openstack.org/374280 | 16:27 |
| mriedem | artom: on https://review.openstack.org/#/c/369176/1/nova/compute/api.py is it possible that we could be lazy-loading the bdm.tag attribute and blow up if the server create request is using microversion < 2.32? | 16:28 |
| *** lucas-hungry is now known as lucasagomes | 16:29 | |
| *** lpetrut has quit IRC | 16:29 | |
| artom | mriedem, to be honest I can't tell you off the top of my head, but I know everywhere else we check 'tag' in bdm first | 16:29 |
| artom | So you're probably right :) | 16:29 |
| mriedem | except in the libvirt driver here: | 16:30 |
| mriedem | tagged_bdms = {_get_device_name(bdm): bdm for bdm in bdms if bdm.tag} | 16:30 |
| *** dane-fichter has joined #openstack-nova | 16:30 | |
| *** rama_y has quit IRC | 16:30 | |
| mriedem | but yeah for hyperv we were defensive | 16:30 |
| mriedem | if 'tag' in vif and vif.tag: | 16:30 |
| mriedem | if bdm_obj and 'tag' in bdm_obj and bdm_obj.tag: | 16:31 |
| *** krtaylor has quit IRC | 16:31 | |
| artom | mriedem, given the essentially nil cost of being defensive I don't see how we can do anything else :) | 16:32 |
| mriedem | unless i'm missing something | 16:32 |
| mriedem | if i'm right then we have a bug in the libvirt driver too | 16:32 |
| mriedem | maybe dansmith remembers | 16:32 |
| artom | I agree that it's not rigorous... | 16:33 |
| artom | But is also saves us the mental gymnastics of wondering about lazy loaded tags every time | 16:33 |
| *** ldeptula has quit IRC | 16:33 | |
| *** Swami has quit IRC | 16:34 | |
| dansmith | Yeah I don't remember, I'd have to go look | 16:34 |
| mriedem | seems we could just test this with a nova boot with the 2.31 microversion | 16:35 |
| mriedem | i'm going to lunch | 16:35 |
| *** mriedem is now known as mriedem_away | 16:35 | |
| *** yamahata has quit IRC | 16:37 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Add deprecated_since parameter https://review.openstack.org/353631 | 16:37 |
| sfinucan | jaypipes: johnthetubaguy: As promised - merge hell :) https://review.openstack.org/#/c/353631/ | 16:38 |
| *** hshiina has quit IRC | 16:38 | |
| *** nic has joined #openstack-nova | 16:40 | |
| *** Apoorva has joined #openstack-nova | 16:40 | |
| *** gszasz has quit IRC | 16:40 | |
| siva_krishnan | artom mridem will change the code check for bdm tag first and try checking it with earlier micorversion as well | 16:43 |
| *** lamt has quit IRC | 16:43 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Move PCI options to a PCI group https://review.openstack.org/356604 | 16:44 |
| artom | siva_krishnan, cool :) | 16:44 |
| sfinucan | lxsli: Could you abandon this change? https://review.openstack.org/#/c/350085/ | 16:45 |
| *** nic has quit IRC | 16:45 | |
| sfinucan | lxsli: It's superseded by this more expansive change https://review.openstack.org/#/c/373262 | 16:45 |
| *** slaweq_ has quit IRC | 16:46 | |
| *** rmart04 has quit IRC | 16:47 | |
| jroll | mriedem_away: I guess it depends what kind of stats the diagnostics api needs - we do ship some hardware health metrics sort of stuff to ceilometer, we could probably store things | 16:47 |
| dansmith | jroll: diagnostics are just an opaque blob, only meaningful per virt driver afaik | 16:48 |
| *** Swami has joined #openstack-nova | 16:48 | |
| *** nic has joined #openstack-nova | 16:49 | |
| *** ljxiash has quit IRC | 16:50 | |
| jroll | dansmith: so just like, whatever we want? | 16:50 |
| dansmith | jroll: yeah pretty sure | 16:50 |
| jroll | cool | 16:50 |
| * jroll throws out a {} | 16:50 | |
| jroll | dansmith: but yeah, seems doable | 16:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: conf: Move consoleauth options to a group https://review.openstack.org/356611 | 16:52 |
| *** Swami_ has joined #openstack-nova | 16:55 | |
| *** lamt has joined #openstack-nova | 16:57 | |
| *** derekh has quit IRC | 16:59 | |
| *** Swami has quit IRC | 16:59 | |
| *** slaweq_ has joined #openstack-nova | 17:00 | |
| *** kaisers_ has quit IRC | 17:03 | |
| *** krtaylor has joined #openstack-nova | 17:04 | |
| *** jichen has joined #openstack-nova | 17:04 | |
| *** rama_y has joined #openstack-nova | 17:06 | |
| *** kumarb has quit IRC | 17:09 | |
| *** code-R has quit IRC | 17:09 | |
| *** cdent has quit IRC | 17:10 | |
| *** ociuhandu has quit IRC | 17:10 | |
| *** unicell1 has quit IRC | 17:10 | |
| *** Swami_ has quit IRC | 17:11 | |
| *** yamahata has joined #openstack-nova | 17:12 | |
| *** sahid has quit IRC | 17:15 | |
| *** Yingxin has quit IRC | 17:19 | |
| *** Yingxin has joined #openstack-nova | 17:20 | |
| *** slaweq_ has quit IRC | 17:20 | |
| *** Guest98774 has quit IRC | 17:22 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add placeholder DB migrations for Ocata https://review.openstack.org/374270 | 17:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add blocker migration to ensure for newton online migrations https://review.openstack.org/374307 | 17:22 |
| *** Swami has joined #openstack-nova | 17:24 | |
| *** lpetrut has joined #openstack-nova | 17:25 | |
| *** cdent has joined #openstack-nova | 17:25 | |
| *** jichen has quit IRC | 17:26 | |
| *** ducttape_ has quit IRC | 17:26 | |
| *** danpb has quit IRC | 17:27 | |
| *** jichen has joined #openstack-nova | 17:28 | |
| *** cdelatte has joined #openstack-nova | 17:32 | |
| *** jichen_ has joined #openstack-nova | 17:32 | |
| *** jichen has quit IRC | 17:33 | |
| *** jichen_ is now known as jichen | 17:33 | |
| *** pradk has joined #openstack-nova | 17:35 | |
| *** jed56 has quit IRC | 17:35 | |
| *** unicell has joined #openstack-nova | 17:36 | |
| *** omnipresent has quit IRC | 17:39 | |
| *** zenoway has joined #openstack-nova | 17:39 | |
| *** mat128 is now known as mat128|mtg | 17:39 | |
| *** zenoway has quit IRC | 17:43 | |
| *** ijw has joined #openstack-nova | 17:45 | |
| *** haplo37__ has joined #openstack-nova | 17:45 | |
| *** Dave has quit IRC | 17:45 | |
| *** jamesden_ has quit IRC | 17:45 | |
| dane-fichter | mriedem: I'm not sure a tempest test of image signing is really feasible without requiring Barbican | 17:46 |
| *** anush has quit IRC | 17:46 | |
| *** jamesden_ has joined #openstack-nova | 17:49 | |
| woodster_ | dane-fichter: I don't think you were on earlier FWIW: I asked around about barbican integration testing. There is an integration test available on the puppet-openstack-integration side that demonstrates volume encryption via cinder...here's a CR that activates it: https://review.openstack.org/#/c/363142/ This might be an example to consider for gate | 17:51 |
| woodster_ | testing. | 17:51 |
| *** Dave____ has joined #openstack-nova | 17:51 | |
| dane-fichter | woodster_: Is this run in any gate? i.e. if this fails does it have any actual consequences for a patch? | 17:52 |
| *** zenoway has joined #openstack-nova | 17:52 | |
| *** ekuris has joined #openstack-nova | 17:52 | |
| dane-fichter | because there's currently a pretty big outstanding bug in Cinder rn | 17:52 |
| *** jamesden_ has quit IRC | 17:53 | |
| woodster_ | dane-fichter: I think this is the bug fix: https://review.openstack.org/#/c/366750/ | 17:55 |
| dane-fichter | woodster_: yup. kaitlin is my colleague actually | 17:55 |
| woodster_ | dane-fichter: puppet-OpenStack-integration is running a gate job | 17:55 |
| *** pwnall1 has joined #openstack-nova | 17:55 | |
| *** zenoway has quit IRC | 17:56 | |
| *** mriedem_away is now known as mriedem | 17:56 | |
| *** hemanthm is now known as hemanthm|lunch | 17:56 | |
| mriedem | dansmith: the diag stuff has been structured for a long time (juno) | 17:56 |
| *** zenoway has joined #openstack-nova | 17:56 | |
| mriedem | dansmith: we just don't expose it in the structured format out of the api yet | 17:56 |
| dansmith | mriedem: ...it has? | 17:56 |
| dansmith | oh | 17:56 |
| dansmith | okay | 17:56 |
| woodster_ | dane-fichter: a link to a test result is here: http://logs.openstack.org/42/363142/10/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/fe14df0/ | 17:57 |
| mriedem | jroll: https://review.openstack.org/#/c/357884/ | 17:57 |
| dane-fichter | woodster_: what is pupper-OpenStack integration? | 17:57 |
| *** ekuris has quit IRC | 17:58 | |
| dansmith | mriedem: they are going to have trouble with some of those items I'd imagine | 17:58 |
| sdague | dansmith: yeh, it got lost during the API v3 shutdown | 17:58 |
| dansmith | sdague: I see that now yeah | 17:58 |
| woodster_ | dane-fichter: sorry, typing fail: puppet-openstack-integration | 17:58 |
| mriedem | dansmith: maybe | 17:59 |
| mriedem | a minimal set might be better than nothing | 17:59 |
| dane-fichter | woodster_: still unclear on what that is | 17:59 |
| *** jamesden_ has joined #openstack-nova | 17:59 | |
| dansmith | mriedem: agree, just sayin | 17:59 |
| *** hieulq_ has quit IRC | 18:00 | |
| woodster_ | dane-fichter: Here's the repo: https://github.com/openstack/puppet-openstack-integration It's intended to integration test Puppet modules used for OpenStack components | 18:00 |
| *** zenoway has quit IRC | 18:01 | |
| *** ducttape_ has joined #openstack-nova | 18:01 | |
| woodster_ | dane-fichter: I can put you in touch with the contributor working on this if you are interested in doing a similar integration style test on the Nova side | 18:01 |
| dane-fichter | woodster_: I'm wondering to what extent these tests actually prevent bad code from merging. What projects use these scenarios for testing? | 18:03 |
| dane-fichter | woodster_ : as in, are there gates running these puppet jobs? | 18:04 |
| woodster_ | dane-fichter: I'm not so sure about that. My sense is that we don't use or maintain these cross project checks enough. | 18:04 |
| woodster_ | dane-fichter: I think only that puppet-openstack-integration repo uses such tests right now | 18:05 |
| *** jamesden_ has quit IRC | 18:05 | |
| dane-fichter | woodster_ , mriedem : I'm skeptical of the value of writing a tempest test that will in all likelihood never prevent bad code from merging since Barbican isn't required by tempest | 18:06 |
| woodster_ | dane-fichter: if you have a moment though, you might pick the contributor's brain on this a bit (Ade Lee, alee) in the #openstack-barbican channel | 18:06 |
| mriedem | dane-fichter: does barbican have a tempest plugin running tests that would hit any of this? | 18:07 |
| mriedem | i know that's not integrated gate | 18:07 |
| *** alee has joined #openstack-nova | 18:07 | |
| alee | woodster_, whats up? | 18:08 |
| woodster_ | alee: dane-fichter has questions regarding integration gate checks for barbican | 18:08 |
| alee | dane-fichter, hi dane - whats up? | 18:09 |
| woodster_ | alee: barbican itself has no integration tests in its gate jobs, correct? | 18:09 |
| dane-fichter | alee: not really FOR barbican | 18:09 |
| alee | woodster_, correct | 18:09 |
| woodster_ | dane-fichter: the integration gate question resurfaces periodically though :) | 18:09 |
| dane-fichter | mriedem has asked me to write a tempest scenario to test that image signature verification works | 18:10 |
| dane-fichter | and basically tempest would have to support barbican in order to write that scenario | 18:10 |
| alee | dane-fichter, so this involves glance and barbican? | 18:10 |
| dane-fichter | it involves glance, barbican, and nova | 18:10 |
| alee | ok | 18:11 |
| dane-fichter | you have to be able to store a cert in the key manager, and glance and nova both have to retrieve that cert | 18:11 |
| dane-fichter | btw this is all for the purpose of preventing "bad" code from making it into nova | 18:11 |
| alee | dane-fichter, sounds reasonable | 18:11 |
| dane-fichter | bad = breaks the feature | 18:11 |
| dane-fichter | I'm not sure where to even start on this | 18:12 |
| alee | dane-fichter, similar tests would have been valuable for volume encryption recently. | 18:12 |
| mriedem | so i know there is a barbican dsvm job | 18:12 |
| mriedem | which i'm assuming runs on at least changes to barbican right? | 18:12 |
| *** jamesden_ has joined #openstack-nova | 18:12 | |
| dane-fichter | alee: yeah that bug is pretty damn awful | 18:13 |
| mriedem | and i'm assuming there is a tempest plugin somewhere for barbican to test the barbican api | 18:13 |
| dane-fichter | alee: is that correct ^ ? | 18:13 |
| alee | mriedem, so there is are barbican functional tests which are written against tempest-lib | 18:14 |
| alee | but this has not yet been converted into a tempest plugin | 18:14 |
| *** tonytan4ever has quit IRC | 18:14 | |
| mriedem | alee: do those run in a dsvm env? | 18:14 |
| mriedem | with nova/glance/cinder running too? | 18:14 |
| mriedem | and do they only hit the barbican api or also do things like create a server with an encrypted volume, or create a server with a signed image | 18:15 |
| *** tonytan4ever has joined #openstack-nova | 18:15 | |
| *** rmart04 has joined #openstack-nova | 18:15 | |
| alee | mriedem, no - the functional tests test the retrieval, generation etc. of secreets outside of nova etc. only keystone is there | 18:15 |
| mriedem | gate-barbican-simple-crypto-devstack-dsvm is probably what i'm looking for | 18:16 |
| dane-fichter | mriedem: these just gates for barbican though right? there's no existing nova gate that runs tempest tests w Barbican? | 18:16 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Annotate online db migrations with cycle added https://review.openstack.org/374323 | 18:16 |
| mriedem | https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/barbican.yaml#L28 | 18:16 |
| mriedem | which runs the functional tests that are in barbican | 18:17 |
| *** READ10 has quit IRC | 18:17 | |
| alee | mriedem, dane-fichter -- so for encrypted volumes for instance, there are a set of cinder tests in tempest that use a default key manager which is not barbican | 18:17 |
| mriedem | alee: yeah i know about that one | 18:17 |
| *** diana_clarke has joined #openstack-nova | 18:17 | |
| mriedem | will that not work for signed images? | 18:17 |
| mriedem | i mean, a test for booting a server with a signed image? | 18:17 |
| alee | mriedem, in openstack-puppet-ci, I wrote a scenartio that tests this against barbican. | 18:17 |
| alee | the idea was to set up barbican and then run the same test against barbican | 18:18 |
| alee | but of course, upstream would be better .. | 18:18 |
| dane-fichter | are you referring to test_encrypted_cinder_volumes.py? | 18:18 |
| *** TravT has quit IRC | 18:18 | |
| alee | yup | 18:18 |
| dane-fichter | lol ok but that obviously doesnt work | 18:18 |
| dane-fichter | since cinder volume encryption is currently broken | 18:19 |
| *** baoli has quit IRC | 18:19 | |
| *** rmart04_ has joined #openstack-nova | 18:19 | |
| alee | dane-fichter, well - it worked ahgaianst the default key manager :) | 18:19 |
| alee | dane-fichter, but failed against barbican when configured to run with barbican | 18:19 |
| alee | dane-fichter, and we caught that in puppet-ci | 18:19 |
| dane-fichter | alee: any idea if the default key manager can be configured to store a cert? | 18:19 |
| *** baoli has joined #openstack-nova | 18:20 | |
| *** eharney has quit IRC | 18:20 | |
| alee | dane-fichter, the default key manager is basically just a file | 18:20 |
| alee | and yeah - it uses castellan - which is a generic interface | 18:20 |
| *** rmart04 has quit IRC | 18:21 | |
| *** rmart04_ is now known as rmart04 | 18:21 | |
| dane-fichter | alee: ok. so with a little bit of work we could probably get it to spit out the right cert | 18:21 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add blocker migration to ensure for newton online migrations https://review.openstack.org/374307 | 18:21 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Annotate online db migrations with cycle added https://review.openstack.org/374323 | 18:21 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add placeholder DB migrations for Ocata https://review.openstack.org/374270 | 18:21 |
| mriedem | bauzas: fyi https://review.openstack.org/#/c/374324/ | 18:21 |
| dane-fichter | alee: how do service config options work w.r.t tempest? like if I need to flip some nova configs for a test | 18:21 |
| mriedem | dane-fichter: that would be configured via devstack | 18:22 |
| alee | dane-fichter, with a little bit of work, you could write a tempest test for image signing using a default key manager , just like the volume encryption case | 18:22 |
| mriedem | and part of the job setup | 18:22 |
| bauzas | mriedem: ++ | 18:22 |
| alee | dane-fichter, and ideally, we could run that also against barbican itslef | 18:22 |
| dane-fichter | ok yes I could write that test | 18:22 |
| *** cdent has quit IRC | 18:23 | |
| dane-fichter | would it ever be run in any meaningful way? we'd have to have a gate JUST for signature verification since it relies on a nova config currently | 18:23 |
| *** dave-mccowan has joined #openstack-nova | 18:24 | |
| alee | dane-fichter, not sure I follow -- why not set it to run with that config by default? | 18:24 |
| alee | dane-fichter, similar to what goes on in encrypted volumes? | 18:24 |
| dane-fichter | alee: I don't understand the question | 18:24 |
| alee | dane-fichter, I guess I dont understand yours :) | 18:25 |
| dane-fichter | in order for nova to actually check image signature properties, the nova config option verify_glance_signatures needs to be True | 18:25 |
| dane-fichter | so the tempest test would never get run in a gate since no gate would have that enabled in their nova | 18:26 |
| mriedem | dane-fichter: we could just set that in devstack... | 18:26 |
| mriedem | to run the tests | 18:26 |
| *** burgerk has quit IRC | 18:26 | |
| mriedem | the test might need a feature toggle in tempest, not sure | 18:26 |
| mriedem | but basically you'd just set all of those bits via devstack in whatever job is going to test them | 18:27 |
| dane-fichter | mriedem: I understand that devstack would need the option set | 18:27 |
| *** lpetrut has quit IRC | 18:27 | |
| alee | dane-fichter, similarly, in the encrypted volume test, cinder and nova have to be configured to talk to a key manager in order to test encrypted volumes .. I would see what they do there. | 18:27 |
| mriedem | the volume type is created with a luks/cryptsetup provider | 18:27 |
| mriedem | and the volume is created from the volume type | 18:27 |
| dane-fichter | I'm not communicating my point effectively here | 18:28 |
| *** kaisers_ has joined #openstack-nova | 18:28 | |
| dane-fichter | the point is that I don't see how this test actually prevents bad code from being merged | 18:28 |
| alee | mriedem, right - but there must be some config in cinder and nova to make that work .. | 18:28 |
| dane-fichter | no existing gate will run this test because no one would turn verify_glance_signatures on in their devstack instance | 18:29 |
| mriedem | alee: i don't think so | 18:29 |
| dane-fichter | because tempest doesn't let you toggle a service's config options | 18:29 |
| mriedem | dane-fichter: why wouldn't we just set that in devstack by default? | 18:29 |
| dane-fichter | and turning that option on will break A TON of other tests | 18:29 |
| mriedem | why would it break other tests? | 18:29 |
| *** hemanthm|lunch is now known as hemanthm | 18:30 | |
| mriedem | b/c hte images aren't signed? | 18:30 |
| dane-fichter | mriedem: because you can't boot unsigned images | 18:30 |
| mriedem | well shitballs | 18:30 |
| dane-fichter | yes | 18:30 |
| dane-fichter | lol exactly | 18:30 |
| dane-fichter | that's what im trying to say | 18:30 |
| dane-fichter | shitballs | 18:30 |
| mriedem | hmm | 18:30 |
| *** rcernin has joined #openstack-nova | 18:30 | |
| mriedem | well, | 18:30 |
| dane-fichter | if we had some type of "security gate" it'd be a different story | 18:31 |
| mriedem | tempest tests with 2 images that are created in devstack, | 18:31 |
| *** gouthamr has quit IRC | 18:31 | |
| *** jichen has quit IRC | 18:31 | |
| mriedem | if we were setting this flag in devstack, wouldn't we just create those images that tempest uses as signed to begin with? | 18:31 |
| mriedem | yeah, i'm thinking like a separate job | 18:31 |
| mriedem | that job sets a flag in devstack to create signed images for tempest to use in ALL tests | 18:31 |
| mriedem | then you don't need to write a new test | 18:31 |
| alee | dane-fichter, well - if we had a security gate -- we could set it to run against barbican itself. and test image signing and encrypted volumes for instance | 18:31 |
| dane-fichter | alee: and we wouldn't have this mess with volume encryption currently | 18:32 |
| mriedem | sure, you just add barbican/barbicanclient to PROJECTS | 18:32 |
| mriedem | like in https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/barbican.yaml#L28 | 18:32 |
| mriedem | you'll have to do the work of adding a flag to devstack to set this stuff all up, meaning the nova.conf values and signed images that tempest uses | 18:33 |
| mriedem | and then the job would just set that flag | 18:33 |
| mriedem | could just be a job in the experimental queue | 18:33 |
| *** eharney has joined #openstack-nova | 18:34 | |
| *** sudipto_ has quit IRC | 18:34 | |
| *** sudswas__ has quit IRC | 18:34 | |
| *** ijw has quit IRC | 18:34 | |
| dane-fichter | mriedem: can you say more about how I would do all this? | 18:34 |
| mriedem | well you're going to need one devstack change that adds a flag | 18:35 |
| mriedem | SIGNED_IMAGES or something | 18:35 |
| mriedem | defaults to False | 18:35 |
| dane-fichter | it seems like I'd need to merge changes in devstack, project-config, and tempest at least | 18:35 |
| mriedem | yes for the first two, i'm not sure about tempest | 18:35 |
| dane-fichter | wait why no more tempest? | 18:36 |
| mriedem | because if simply having tempest run with a signed image and that's used to boot the server, that tests the signature_utils code right? | 18:36 |
| *** ijw has joined #openstack-nova | 18:36 | |
| mriedem | and the encrypted volumes tests would use the configured castellan/barbican key manager | 18:36 |
| mriedem | *configured in nova.conf via devstack when SIGNED_IMAGES=True or whatever | 18:37 |
| dane-fichter | yes, I think it'd be extremely unlikely that I successfully merge a change that signs an image for no apparent reason | 18:37 |
| mriedem | the reason would be the job in project-config that sets the flag | 18:37 |
| mriedem | to test with signed images | 18:37 |
| mriedem | so, doesn't the barbican devstack plugin already configure the key manager in nova.conf? | 18:38 |
| dane-fichter | no | 18:38 |
| dane-fichter | it just sets up barbican | 18:38 |
| sdague | dane-fichter: huh? that seems surprising | 18:38 |
| dane-fichter | you have to tell nova what class of key manager to tell castellan to use | 18:38 |
| sdague | dane-fichter: right, but the expectation would be that it would do that if nova was installed | 18:38 |
| mriedem | doesn't look like it https://github.com/openstack/barbican/blob/master/devstack/lib/barbican | 18:38 |
| *** sdake has quit IRC | 18:39 | |
| sdague | in the same way that neutron configures nova if nova is installed | 18:39 |
| dane-fichter | sdague: the switch to castellan is new, could just be because of that newness | 18:39 |
| mriedem | sdague: the barbican devstack plugin just sets up barbican, and the dsvm job only runs functional in tree tests from barbican that test their api | 18:39 |
| mriedem | it doesn't run tempest | 18:39 |
| sdague | mriedem: ah | 18:39 |
| mriedem | well, | 18:39 |
| mriedem | as far as i can tell | 18:39 |
| sdague | so... it's not tested in any even pseudo real environment | 18:39 |
| dane-fichter | no security functionality is tested in any pseudo real env with any blocking power on patches in the big tent projects | 18:40 |
| sdague | mriedem: because I'm sick of staring at these errors at the beginning of every test run, can you =W - https://review.openstack.org/#/c/374213/ | 18:40 |
| mriedem | anyway, i was just thinking this job could configure everything via the barbican devstack plugin, but that wouldn't do it all | 18:40 |
| dane-fichter | that's precisely why cinder volume encryption is all screwed up | 18:40 |
| sdague | dane-fichter: right, but, it would help if those tests were run on the barbican job | 18:41 |
| sdague | or anywhere | 18:41 |
| dane-fichter | sdague: I don't see why that falls to barbican, it was a change in cinder that messed up the feature | 18:41 |
| dane-fichter | and castellan supports multiple backends | 18:41 |
| sdague | dane-fichter: because I would think that the barbican team would like to demonstrate how the whole thing works and holds together | 18:42 |
| dane-fichter | so it's theoretically possible to have volume encryption w/o barbican | 18:42 |
| sdague | to demonstrate project maturity so that we could make it less optional | 18:42 |
| mriedem | sdague: is that what's causing all of this garbage? http://logs.openstack.org/31/371331/1/check/gate-nova-python27-db-ubuntu-xenial/ff4ffe6/console.html#_2016-09-21_02_01_20_355196 | 18:42 |
| dane-fichter | you could make the argument that it won't mature until it is less optional | 18:43 |
| sdague | mriedem: it's one of those things | 18:43 |
| dane-fichter | sdague: i have literally no idea what that is | 18:44 |
| sdague | dane-fichter: the error I'm showing mriedem is unrelated | 18:44 |
| dane-fichter | haha ok | 18:44 |
| sdague | mriedem: the scramble is this exception | 18:44 |
| jroll | mriedem: commented | 18:44 |
| sdague | and the os-vif logging | 18:44 |
| jroll | mriedem: we kinda could do that feature, but will come up short in what we can support | 18:45 |
| cfriesen | question about object versions over upgrades....suppose object A has version X in the DB. now we upgrade to release N, and nova-compute requests version X+1 of that object. how does this not cause problems in obj_to_primitive? | 18:45 |
| mriedem | dane-fichter: things are generally less optional only after they are showing to be stable for a certain period of time | 18:45 |
| mriedem | not the other way around | 18:45 |
| dane-fichter | mriedem: you'll never hear me argue on behalf of barbican's stability | 18:45 |
| dane-fichter | that's precisely the reason that castellan exists | 18:45 |
| dane-fichter | or at least one of the reasons | 18:45 |
| mriedem | so you can have a key manager api but w/o barbican as the backend, right? | 18:46 |
| dane-fichter | yeah. I think you can use PyKMIP behind castellan (?) | 18:46 |
| dane-fichter | at least that's the end goal of castellan | 18:46 |
| dane-fichter | to allow for different key management backends | 18:47 |
| mriedem | ok, well, i think having a job in the experimental queue that's using some kind of non-fake key manager is a decent goal | 18:47 |
| mriedem | gate-tempest-dsvm-pykmip if you want | 18:47 |
| dane-fichter | I do too, I just have no clue how to do that | 18:47 |
| mriedem | it starts with something telling devstack to configure the various services | 18:48 |
| *** Dave____ has quit IRC | 18:48 | |
| mriedem | which is a job | 18:48 |
| mriedem | i thought we could use the barbican devstack plugin for that | 18:48 |
| mriedem | but that wouldn't completely do it | 18:48 |
| dane-fichter | yeah there has to be some initial configuration | 18:48 |
| dane-fichter | i've already scripted most of that for my own purposes actually | 18:49 |
| sdague | mriedem: well, I think the barbican devstack plugin really should do that | 18:49 |
| mriedem | the devstack-plugin-ceph repo sets up and configures ceph and cinder/glance/nova | 18:49 |
| mriedem | so yeah ^ is really the model | 18:50 |
| *** Dave has joined #openstack-nova | 18:50 | |
| mriedem | so when you deploy with the devstack ceph plugin, everything is using ceph | 18:50 |
| dane-fichter | sdague, mriedem: I'll push a patch to devstack that sets the right key_manager / key_mgr opts if it's not already there | 18:50 |
| sdague | dane-fichter: awesome | 18:50 |
| sdague | that would be step one | 18:50 |
| dane-fichter | ok, what next | 18:51 |
| mriedem | dane-fichter: to avoid being blocked on passing through local.conf | 18:51 |
| sdague | then a job that's running the normal base services + barbican would mean barbican is tested at least secondarily | 18:51 |
| mriedem | i think that flag in devstack will also need to create signed images | 18:51 |
| *** gouthamr has joined #openstack-nova | 18:51 | |
| mriedem | dane-fichter: i can help you out with the project-config changes | 18:51 |
| mriedem | and pulling it together | 18:51 |
| *** ig0r_ has joined #openstack-nova | 18:51 | |
| mriedem | we can POC in devstack-gate first to see it actually runs, that's what i usually do | 18:52 |
| alee | dane-fichter, please keep me in the loop on your patches. I'll try to help out if needed. | 18:53 |
| dane-fichter | alee: will do | 18:56 |
| openstackgerrit | Sean Dague proposed openstack/nova: move os_vif.initialize() to nova-compute start https://review.openstack.org/374233 | 18:56 |
| *** sdake has joined #openstack-nova | 18:56 | |
| dane-fichter | mriedem: what would the project-config change entail? | 18:56 |
| dane-fichter | mriedem: just a definition of a job that sets some $SIGNED_IMAGES flag in devstack and then runs the normal tempest tests? | 18:57 |
| *** moshele has joined #openstack-nova | 18:59 | |
| mriedem | dane-fichter: basically | 19:02 |
| mriedem | it would also need to run the barbican devstack plugin if it's going to use barbican as the key manager | 19:02 |
| mriedem | dane-fichter: but i'd start with a DNM devstack-gate change that sets the same flags that the project-config job is going to do | 19:02 |
| mriedem | that way you know if it's working before you propose the new job | 19:03 |
| dane-fichter | mriedem: DNM? | 19:03 |
| dane-fichter | ohh WIP | 19:03 |
| *** moshele has quit IRC | 19:03 | |
| mriedem | sort of like this https://review.openstack.org/#/c/362441/ | 19:03 |
| mriedem | yeah do not merge | 19:04 |
| mriedem | sdague: have you seen this https://review.openstack.org/#/c/366091/ ? i saw it was on the meeting agenda for tomorrow | 19:05 |
| mriedem | artom: did we ever figure out if the bdm.tag was lazy-loaded or not? | 19:05 |
| *** mat128|mtg is now known as mat128 | 19:06 | |
| artom | mriedem, when you went for lunch I went for a run | 19:07 |
| artom | So, no | 19:07 |
| mriedem | ok i just booted a server using microversion 2.31 w/o any issues | 19:07 |
| artom | mriedem, oh, btw, the XenAPI folks will likely want to get https://review.openstack.org/#/c/333781/ merged in Ocata | 19:07 |
| artom | I suppose if there's somewhere in the code that assume tag isn't lazy loaded when in actuality is is - it would have been noticed as a bug by now | 19:08 |
| sdague | mriedem: I had seen a little bit around it, the linked git repo is empty so it's hard to evaluate | 19:09 |
| mriedem | artom: yeah BobBall is right in there, needs a new bp for the xenapi feature parity part | 19:09 |
| mriedem | sdague: heh, get it into project-config and the code will write itself i guess | 19:09 |
| artom | mriedem, aha, so that's how that's handled | 19:09 |
| mriedem | looks like some red hat / enovance QA thing | 19:09 |
| mriedem | to upstream | 19:09 |
| sdague | it would be really nice to actually have a better understanding of what they believe those tests are | 19:10 |
| *** pwnall1 has quit IRC | 19:10 | |
| artom | Hrmm, if my understanding of that patch is correct it's essentially Tempest but with host diddling | 19:10 |
| artom | So checking instance XML and the like | 19:10 |
| artom | For NUMA, etc | 19:10 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif: Updated from global requirements https://review.openstack.org/373772 | 19:11 |
| mriedem | artom: yeah, which isn't tempest | 19:11 |
| dansmith | sdague: they want to do tests where they can boot a guest and examine the libvirt xml and system properties | 19:12 |
| mriedem | they basically want a dsvm-integration job for nova | 19:12 |
| mriedem | which we want too | 19:12 |
| dansmith | sdague: I've told them I think functional-ds would be a better route | 19:12 |
| mriedem | dansmith: yup me too | 19:12 |
| *** catintheroof has joined #openstack-nova | 19:12 | |
| mriedem | probably a misunderstanding that you can actually do functional integratoin testing with dsvm w/o tempest | 19:12 |
| sdague | yeh. | 19:12 |
| artom | Isn't this sort of thing precisely what plugins are for? Using all the Tempest infrastructure to do things that aren't actually right for Tempest? | 19:13 |
| mriedem | not really | 19:13 |
| dansmith | plugins are for vendor-specific things that would otherwise be in scope, AFAIK | 19:13 |
| *** mdrabe has quit IRC | 19:14 | |
| artom | dansmith, aha, TIL | 19:14 |
| *** ijw has quit IRC | 19:14 | |
| *** ijw has joined #openstack-nova | 19:15 | |
| *** slaweq_ has joined #openstack-nova | 19:15 | |
| mriedem | sdague: fyi i've updated https://wiki.openstack.org/wiki/Meetings/Nova for tomorrow | 19:16 |
| *** TravT has joined #openstack-nova | 19:17 | |
| sdague | mriedem: great, thanks | 19:18 |
| *** akshai has quit IRC | 19:20 | |
| openstackgerrit | Sean Dague proposed openstack/os-vif: Make plugin loading more consistent with logging guidelines. https://review.openstack.org/374225 | 19:21 |
| sdague | mriedem: that should fix that indent issue | 19:21 |
| *** rmart04 has quit IRC | 19:21 | |
| mriedem | +1 | 19:22 |
| *** jvgrant__ has quit IRC | 19:24 | |
| *** ijw has quit IRC | 19:25 | |
| *** lpetrut has joined #openstack-nova | 19:25 | |
| *** akshai has joined #openstack-nova | 19:25 | |
| *** bvanhav has quit IRC | 19:29 | |
| *** mdrabe has joined #openstack-nova | 19:31 | |
| *** jvgrant has joined #openstack-nova | 19:35 | |
| *** alee is now known as alee_afk | 19:36 | |
| *** mfisch has quit IRC | 19:37 | |
| *** lpetrut has quit IRC | 19:39 | |
| *** ijw has joined #openstack-nova | 19:40 | |
| *** smatzek has quit IRC | 19:40 | |
| *** ijw has quit IRC | 19:41 | |
| *** ijw has joined #openstack-nova | 19:41 | |
| *** tonytan_brb has joined #openstack-nova | 19:41 | |
| sdague | mriedem: so the other 1/2 of the scrambled test results is this - https://review.openstack.org/#/c/374233/ | 19:42 |
| *** salv-orlando has joined #openstack-nova | 19:42 | |
| sdague | os_vif.initialize() was getting called at module load time | 19:43 |
| sdague | and the logger is not initialized yet | 19:43 |
| mriedem | target that bug? | 19:43 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1615676 | 19:44 |
| openstack | Launchpad bug 1615676 in OpenStack Compute (nova) "os-vif log messages corrupt and pollute test output" [Wishlist,In progress] - Assigned to Pushkar Umaranikar (pushkar-umaranikar) | 19:44 |
| *** tonytan4ever has quit IRC | 19:44 | |
| *** dane-fichter has quit IRC | 19:45 | |
| openstackgerrit | Sean Dague proposed openstack/nova: move os_vif.initialize() to nova-compute start https://review.openstack.org/374233 | 19:46 |
| *** salv-orl_ has quit IRC | 19:46 | |
| sdague | done | 19:46 |
| sdague | yeh, we might want to backport these 2 | 19:46 |
| *** panbalag has joined #openstack-nova | 19:46 | |
| *** cmartin has quit IRC | 19:47 | |
| mriedem | sdague: crap i just posted comments | 19:47 |
| sdague | mriedem: ok, let me go look | 19:47 |
| mriedem | it's weird that we were initializing in nova/network/* before, but we don't do that in nova/cmd/network.py | 19:47 |
| *** smatzek has joined #openstack-nova | 19:47 | |
| mriedem | probably just because nothing in nova-network uses os-vif | 19:47 |
| mriedem | seems like os_vif_util should have really lived in nova/virt/ but too late now | 19:48 |
| panbalag | sdague: Reg your comment on https://review.openstack.org/#/c/366091/5 , can you let me know which mailing list? | 19:48 |
| sdague | panbalag: openstack-dev | 19:49 |
| sdague | tag it with [nova] [qa] | 19:49 |
| *** tanvirt has quit IRC | 19:50 | |
| *** smatzek has quit IRC | 19:50 | |
| *** kaisers_ has quit IRC | 19:50 | |
| *** smatzek has joined #openstack-nova | 19:51 | |
| *** jamesden_ has quit IRC | 19:53 | |
| *** cdelatte has quit IRC | 19:54 | |
| *** TravT has quit IRC | 19:56 | |
| *** rama_y has quit IRC | 19:57 | |
| sdague | mriedem: yeh, mostly that, nothing in nova-network uses it | 20:01 |
| sdague | I did go and look | 20:01 |
| sdague | because it confused me as well | 20:01 |
| *** panbalag has quit IRC | 20:02 | |
| *** prateek has quit IRC | 20:03 | |
| mriedem | ha https://review.openstack.org/#/c/373940/1 | 20:03 |
| *** jamesden_ has joined #openstack-nova | 20:03 | |
| *** panbalag has joined #openstack-nova | 20:03 | |
| *** bdeschenes has quit IRC | 20:06 | |
| *** panbalag has left #openstack-nova | 20:06 | |
| *** panbalag has joined #openstack-nova | 20:07 | |
| *** catintheroof has quit IRC | 20:08 | |
| *** rama_y has joined #openstack-nova | 20:08 | |
| *** lucasagomes is now known as lucas-afk | 20:09 | |
| *** panbalag has quit IRC | 20:12 | |
| *** panbalag has joined #openstack-nova | 20:12 | |
| *** pwnall1 has joined #openstack-nova | 20:14 | |
| *** TravT has joined #openstack-nova | 20:17 | |
| panbalag | sdague: got disconnected. Did you get my earlier message about the mailing list? | 20:17 |
| *** ijw has quit IRC | 20:18 | |
| *** pwnall1 has quit IRC | 20:19 | |
| *** pwnall1 has joined #openstack-nova | 20:20 | |
| lxsli | sfinucan: done | 20:25 |
| *** ijw has joined #openstack-nova | 20:29 | |
| *** panbalag has quit IRC | 20:30 | |
| *** panbalag has joined #openstack-nova | 20:30 | |
| sdague | mriedem: commented | 20:30 |
| *** sinval has joined #openstack-nova | 20:32 | |
| *** baoli has quit IRC | 20:33 | |
| *** bdeschenes has joined #openstack-nova | 20:33 | |
| *** baoli has joined #openstack-nova | 20:33 | |
| *** Jeffrey4l has quit IRC | 20:35 | |
| mriedem | alaski: i know you want to +W this https://review.openstack.org/#/c/372710/ | 20:35 |
| alaski | interesting | 20:37 |
| *** Jeffrey4l has joined #openstack-nova | 20:37 | |
| *** abalutoiu has quit IRC | 20:38 | |
| mriedem | sdague: replied | 20:40 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: conf: Move PCI options to a PCI group https://review.openstack.org/356604 | 20:41 |
| *** jamesden_ has quit IRC | 20:42 | |
| *** coolsvap has quit IRC | 20:42 | |
| sdague | mriedem: ah, interesting, test run order differences | 20:42 |
| sdague | ok, let me fix that before I head out for the day | 20:42 |
| mriedem | danka | 20:43 |
| * mriedem needs to reboot | 20:43 | |
| *** mriedem has quit IRC | 20:43 | |
| sdague | also... we really need to split up that test class | 20:43 |
| *** wznoinsk has joined #openstack-nova | 20:45 | |
| *** rlrossit has quit IRC | 20:46 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: conf: Move PCI options to a PCI group https://review.openstack.org/356604 | 20:46 |
| *** mriedem has joined #openstack-nova | 20:48 | |
| *** ijw has quit IRC | 20:49 | |
| openstackgerrit | Sean Dague proposed openstack/nova: move os_vif.initialize() to nova-compute start https://review.openstack.org/374233 | 20:50 |
| sdague | mriedem: fixed | 20:50 |
| *** ijw has joined #openstack-nova | 20:50 | |
| mriedem | ok | 20:50 |
| openstackgerrit | Michael Turek proposed openstack/nova: Ironic: use portgroups https://review.openstack.org/206163 | 20:51 |
| *** yohoffman has joined #openstack-nova | 20:52 | |
| mriedem | dansmith: this is for rc2 https://review.openstack.org/#/c/374324/ | 20:53 |
| *** ijw has quit IRC | 20:53 | |
| dansmith | ack | 20:54 |
| *** coreywright has left #openstack-nova | 20:55 | |
| *** krtaylor has quit IRC | 20:56 | |
| *** coreywright has joined #openstack-nova | 20:56 | |
| lifeless | mriedem: you pinged me yesterday? | 20:57 |
| *** rama_y has quit IRC | 20:58 | |
| mriedem | sfinucan: are you letting your OCD get the best of you? https://review.openstack.org/#/c/353631/12/nova/conf/service.py | 20:59 |
| *** akshai has quit IRC | 20:59 | |
| mriedem | lifeless: yeah i had added you to https://review.openstack.org/#/c/339232/ | 21:00 |
| mriedem | in case there was some crazy trap waiting there with that approach | 21:00 |
| mriedem | we were basically looking for a faster way to autospec mocked library interfaces | 21:00 |
| *** tongli has quit IRC | 21:00 | |
| lifeless | mriedem: faster perf or faster dev ? | 21:01 |
| mriedem | perf | 21:01 |
| mriedem | earlier comment in there: "Sooo.... doing full autospecs is a terrible idea. It took me 4000+ seconds to run all 301 hyperv unit tests, which is bad." | 21:01 |
| lifeless | mriedem: in principle I think that could go upstream to CPython and mock | 21:01 |
| mriedem | lifeless: yeah agree | 21:01 |
| lifeless | I have the tab open, shall look more closely in a bit | 21:02 |
| mriedem | which is why i was saying yesterday in here, i'm surprised this isn't an option lower down the stack | 21:02 |
| *** baoli has quit IRC | 21:02 | |
| mriedem | lifeless: thanks | 21:02 |
| *** Guest9190 has joined #openstack-nova | 21:02 | |
| *** baoli has joined #openstack-nova | 21:02 | |
| Guest9190 | hello | 21:02 |
| mriedem | "Autospecing only the public methods of those utils still takes a lot of time (1000+ seconds)." | 21:02 |
| mriedem | so he just changed it to autospec only when a thing is accessed | 21:02 |
| *** rama_y has joined #openstack-nova | 21:03 | |
| *** cleong has quit IRC | 21:04 | |
| yohoffman | Question about nova quota. It looks like updating the fixed_ips and floating_ips quota for a project or for a project's user updates the quota for both the project and the user. I don't see this behavior with the other quotas. Is this by design or a bug? | 21:05 |
| *** cdelatte has joined #openstack-nova | 21:08 | |
| *** burt has quit IRC | 21:08 | |
| *** burt has joined #openstack-nova | 21:08 | |
| *** dpeschman has joined #openstack-nova | 21:09 | |
| *** cdent has joined #openstack-nova | 21:11 | |
| *** sinval has quit IRC | 21:11 | |
| *** cdent has quit IRC | 21:14 | |
| dpeschman | Hello. We use cells v1, and have an occasional problem where the child and API cell databases become out of sync for an instance. | 21:14 |
| *** ijw has joined #openstack-nova | 21:14 | |
| dpeschman | We found the sync_instances API that appears to be for this purpose - is that right? | 21:15 |
| dpeschman | The purpose of re-synchronizing the databases, I mean. | 21:16 |
| *** baoli_ has joined #openstack-nova | 21:19 | |
| *** baoli has quit IRC | 21:19 | |
| *** ijw has quit IRC | 21:19 | |
| *** Guest9190 has quit IRC | 21:19 | |
| *** baoli_ has quit IRC | 21:19 | |
| *** baoli has joined #openstack-nova | 21:20 | |
| melwitt | yohoffman: I think that's expected because fixed_ips and floating_ips quota are project-scoped. that is, you can't set particular quota per user, it's per project only. networks quota is also project-scoped | 21:23 |
| *** melwitt has left #openstack-nova | 21:24 | |
| *** melwitt has joined #openstack-nova | 21:24 | |
| *** rcernin has quit IRC | 21:24 | |
| *** wasmum has joined #openstack-nova | 21:24 | |
| yohoffman | melwitt: So shouldn't those quota options not be allowed when setting a user's quota? | 21:25 |
| *** salv-orlando has quit IRC | 21:26 | |
| *** salv-orlando has joined #openstack-nova | 21:27 | |
| *** baoli has quit IRC | 21:28 | |
| *** baoli has joined #openstack-nova | 21:29 | |
| *** ujjain has quit IRC | 21:29 | |
| *** ujjain has joined #openstack-nova | 21:30 | |
| *** ujjain has quit IRC | 21:30 | |
| *** ujjain has joined #openstack-nova | 21:30 | |
| *** lpetrut has joined #openstack-nova | 21:32 | |
| melwitt | yohoffman: the api is just "update quota" and tenant_id is required, user_id is optional. I guess it allows one to update all quotas at the same time, no matter how they're scoped | 21:33 |
| *** gouthamr has quit IRC | 21:34 | |
| *** rtheis has quit IRC | 21:39 | |
| *** krtaylor has joined #openstack-nova | 21:40 | |
| *** edmondsw has quit IRC | 21:40 | |
| melwitt | dpeschman: looks like it is. I didn't know about it until now though | 21:41 |
| *** melwitt has left #openstack-nova | 21:42 | |
| *** melwitt has joined #openstack-nova | 21:42 | |
| dpeschman | melwitt: Thank you. We're found while trying to use it that it does not work, as Instance.save() considers for update only fields that had changed since the object was loaded in memory. | 21:44 |
| *** ig0r_ has quit IRC | 21:45 | |
| dpeschman | Is there a way to force the database to update certain fields / override the built-in dirty checking? | 21:46 |
| melwitt | dpeschman: ah, okay. it must have been accidentally broken during the conversion to versioned objects | 21:46 |
| *** jheroux has quit IRC | 21:48 | |
| melwitt | dpeschman: I think the gist of what would have to be done is dirty all of the fields for the instance in _sync_instance in nova/cells/messaging.py in the case of an update (it wouldn't be needed for a delete) | 21:49 |
| melwitt | might be able to do it by something like "instance._changed_fields = set(self.fields.keys())". I'm not sure | 21:54 |
| *** haplo37__ has quit IRC | 21:55 | |
| dpeschman | melwitt: I see. Would it be too meddling of us to add a method to Instance to modify _changed_fields directly? | 21:55 |
| *** Swami has quit IRC | 21:58 | |
| *** burt has quit IRC | 21:58 | |
| wznoinsk | hi all | 21:59 |
| *** armax has quit IRC | 21:59 | |
| wznoinsk | is there anyone from Red Hat CI team around? | 21:59 |
| *** armax has joined #openstack-nova | 22:00 | |
| *** ijw has joined #openstack-nova | 22:02 | |
| melwitt | dpeschman: I don't see another way based on the current state of the code, versioned objects don't have a method that means "dirty all fields." to get by, I would probably just do as I mentioned earlier "instance._changed_fields = set(instance.obj_fields)" in _sync_instances in cells | 22:03 |
| melwitt | dpeschman: note that we've been working on cells v2 that should be done hopefully by the end of ocata and there won't be the ability for instances to go out of sync as they'll be stored only in the cell databases | 22:05 |
| *** lpetrut has quit IRC | 22:07 | |
| *** annegentle has quit IRC | 22:11 | |
| dpeschman | melwitt: Thank you. We look forward to v2 with great anticipation :) | 22:13 |
| *** hemanthm is now known as hemanthm|away | 22:17 | |
| *** marst has quit IRC | 22:17 | |
| *** lamt has quit IRC | 22:20 | |
| *** xyang1 has quit IRC | 22:22 | |
| *** claudiub has quit IRC | 22:23 | |
| *** xinliang has quit IRC | 22:27 | |
| *** ducttape_ has quit IRC | 22:28 | |
| *** ducttape_ has joined #openstack-nova | 22:28 | |
| *** tbachman_ has left #openstack-nova | 22:31 | |
| *** thorst has joined #openstack-nova | 22:31 | |
| *** ijw has quit IRC | 22:33 | |
| *** marst has joined #openstack-nova | 22:36 | |
| *** ducttape_ has quit IRC | 22:36 | |
| *** smatzek has quit IRC | 22:37 | |
| *** xinliang has joined #openstack-nova | 22:40 | |
| *** yamahata has quit IRC | 22:40 | |
| *** thorst has quit IRC | 22:41 | |
| *** thorst has joined #openstack-nova | 22:41 | |
| *** tbachman has joined #openstack-nova | 22:42 | |
| *** markvoelker has quit IRC | 22:43 | |
| *** Swami has joined #openstack-nova | 22:44 | |
| *** mlavalle has quit IRC | 22:45 | |
| *** thorst has quit IRC | 22:46 | |
| *** Swami has quit IRC | 22:46 | |
| *** Swami has joined #openstack-nova | 22:46 | |
| *** Swami_ has joined #openstack-nova | 22:52 | |
| *** thorst has joined #openstack-nova | 22:54 | |
| *** Swami has quit IRC | 22:54 | |
| *** ijw has joined #openstack-nova | 22:54 | |
| *** yamahata has joined #openstack-nova | 22:56 | |
| *** annegentle has joined #openstack-nova | 22:58 | |
| *** ijw has quit IRC | 22:59 | |
| *** rrecio has quit IRC | 22:59 | |
| *** thorst has quit IRC | 22:59 | |
| *** sdague has quit IRC | 23:00 | |
| *** mvk has joined #openstack-nova | 23:04 | |
| *** baoli has quit IRC | 23:04 | |
| *** jamielennox is now known as jamielennox|away | 23:04 | |
| *** mdrabe_ has joined #openstack-nova | 23:07 | |
| openstackgerrit | Clint 'SpamapS' Byrum proposed openstack/nova: Remove boto from requirements https://review.openstack.org/374433 | 23:08 |
| *** hoonetorg has quit IRC | 23:10 | |
| harlowja | melwitt dpeschman might be useful when we have this patch working just to throw it up there | 23:10 |
| harlowja | * i guess more targeted at dpeschman | 23:10 |
| harlowja | lol | 23:10 |
| *** mdrabe has quit IRC | 23:10 | |
| harlowja | cause the change i've seen is not really v1 specific is it? | 23:11 |
| harlowja | dpeschman right? | 23:11 |
| harlowja | (perhaps i am wrong, cause idk the v2 stuff really) | 23:11 |
| dpeschman | you're right, it's not really v1 specific, although we're using it to get around a v1-only problem | 23:12 |
| harlowja | right | 23:12 |
| *** tblakes has quit IRC | 23:12 | |
| harlowja | i'm assuming v2 still has some kind of sync_instances ? | 23:12 |
| harlowja | maybe it doesn't (if it doesn't that's sweet, ha) | 23:12 |
| harlowja | fixes the glitch :-P | 23:12 |
| *** annegentle has quit IRC | 23:15 | |
| dansmith | harlowja: v2 does not, that's the whole point | 23:15 |
| harlowja | woot | 23:15 |
| harlowja | dansmith thx for verifying | 23:28 |
| *** efried has quit IRC | 23:28 | |
| harlowja | guessing that it still would be useful to throw this patch up once we have it | 23:28 |
| harlowja | even if it doesn't merge or whatever :) | 23:28 |
| *** tri2sing has quit IRC | 23:28 | |
| *** stewie925 has quit IRC | 23:31 | |
| *** ijw has joined #openstack-nova | 23:32 | |
| *** stew925 has joined #openstack-nova | 23:34 | |
| *** lbeliveau has quit IRC | 23:36 | |
| *** avolkov` has quit IRC | 23:36 | |
| *** avolkov` has joined #openstack-nova | 23:36 | |
| *** ijw has quit IRC | 23:36 | |
| *** ijw has joined #openstack-nova | 23:41 | |
| *** rmk has quit IRC | 23:42 | |
| mriedem | SpamapS: just remove it | 23:43 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2016-September/104291.html | 23:43 |
| *** lamt has joined #openstack-nova | 23:43 | |
| *** jamielennox|away is now known as jamielennox | 23:43 | |
| *** rmk has joined #openstack-nova | 23:43 | |
| *** rmk has quit IRC | 23:43 | |
| *** rmk has joined #openstack-nova | 23:43 | |
| mriedem | the emc guys might speak up | 23:43 |
| *** markvoelker has joined #openstack-nova | 23:44 | |
| *** bdeschenes has quit IRC | 23:45 | |
| *** lamt has quit IRC | 23:47 | |
| mriedem | harlowja: you mean you contacted HR about cells v1 and they just....fixed the glitch? | 23:48 |
| mriedem | so it's going to take care of itself? | 23:49 |
| *** markvoelker has quit IRC | 23:49 | |
| *** avolkov` has quit IRC | 23:50 | |
| harlowja | mriedem mostly i think | 23:51 |
| harlowja | although i would and am wondering a little bit about the amount of cells v1 stuff that i've seen being patched up | 23:51 |
| harlowja | and wondering if there is even a crappy way to organize those patches into something useable (an add-on-patch-kit if u will) | 23:52 |
| harlowja | *sorta like a crappy booster pack | 23:52 |
| harlowja | lol | 23:52 |
| *** gouthamr has joined #openstack-nova | 23:52 | |
| harlowja | but ya, cells v2 will fix stuff | 23:52 |
| harlowja | just slightly concerns myself for folks that run v1 and those folks not really able to do much about the number of patches :-/ | 23:53 |
| harlowja | (not like super-concerned, just a little) | 23:53 |
| harlowja | like its not hundreds, but ~10 i think godaddy has | 23:53 |
| *** smatzek has joined #openstack-nova | 23:55 | |
| *** dimtruck is now known as zz_dimtruck | 23:58 | |
| *** mtanino has quit IRC | 23:58 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Device tagging for epehemeral disks https://review.openstack.org/369176 | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!