| *** zenoway has quit IRC | 00:00 | |
| *** pushkaru has quit IRC | 00:01 | |
| *** shaohe_feng has quit IRC | 00:03 | |
| *** jamielennox is now known as jamielennox|away | 00:04 | |
| *** shaohe_feng has joined #openstack-nova | 00:04 | |
| *** jamielennox|away is now known as jamielennox | 00:04 | |
| *** hieulq_ has joined #openstack-nova | 00:06 | |
| *** winston-d_ has joined #openstack-nova | 00:06 | |
| *** AmyLee7 has quit IRC | 00:07 | |
| *** harlowja_ has joined #openstack-nova | 00:11 | |
| *** sdake has quit IRC | 00:13 | |
| *** shaohe_feng has quit IRC | 00:13 | |
| *** markvoelker has quit IRC | 00:13 | |
| *** shaohe_feng has joined #openstack-nova | 00:14 | |
| *** harlowja has quit IRC | 00:15 | |
| *** diogogmt has quit IRC | 00:23 | |
| *** shaohe_feng has quit IRC | 00:23 | |
| *** shaohe_feng has joined #openstack-nova | 00:24 | |
| *** dimtruck is now known as zz_dimtruck | 00:26 | |
| *** diogogmt has joined #openstack-nova | 00:27 | |
| *** thorst_ has joined #openstack-nova | 00:27 | |
| *** rama_y has quit IRC | 00:29 | |
| *** rama_y has joined #openstack-nova | 00:29 | |
| *** shaohe_feng has quit IRC | 00:33 | |
| *** shaohe_feng has joined #openstack-nova | 00:34 | |
| *** rama_y has quit IRC | 00:35 | |
| *** mikal_ is now known as mikal | 00:35 | |
| *** diogogmt has quit IRC | 00:36 | |
| *** rfolco has quit IRC | 00:37 | |
| *** mtanino has quit IRC | 00:43 | |
| *** shaohe_feng has quit IRC | 00:44 | |
| *** ijw_ has joined #openstack-nova | 00:44 | |
| *** ijw has quit IRC | 00:46 | |
| *** hoangcx has joined #openstack-nova | 00:46 | |
| *** shaohe_feng has joined #openstack-nova | 00:47 | |
| *** stewie925 has quit IRC | 00:48 | |
| *** Kennan has quit IRC | 00:50 | |
| *** fragatina has quit IRC | 00:52 | |
| *** Apoorva has quit IRC | 00:54 | |
| *** shaohe_feng has quit IRC | 00:54 | |
| *** raunak has quit IRC | 00:54 | |
| *** shaohe_feng has joined #openstack-nova | 00:55 | |
| *** cheneydc has joined #openstack-nova | 00:59 | |
| *** nic has quit IRC | 01:00 | |
| *** Kennan has joined #openstack-nova | 01:01 | |
| lifeless | alaski: around ? | 01:02 |
|---|---|---|
| lifeless | alaski: there's something odd with test_patch_unboundmethod_with_staticmethod | 01:03 |
| *** Kevin_Zheng has joined #openstack-nova | 01:03 | |
| *** gjayavelu has quit IRC | 01:03 | |
| lifeless | alaski: if I use INST_C in that test, it returns 1, not tgtslf, 1 - e.g. the patch isn't applied ? :/ | 01:03 |
| *** shaohe_feng has quit IRC | 01:04 | |
| *** shaohe_feng has joined #openstack-nova | 01:05 | |
| *** iceyao has joined #openstack-nova | 01:05 | |
| *** moshele has joined #openstack-nova | 01:09 | |
| *** jinxing has joined #openstack-nova | 01:10 | |
| *** diogogmt has joined #openstack-nova | 01:14 | |
| *** shaohe_feng has quit IRC | 01:14 | |
| *** shaohe_feng has joined #openstack-nova | 01:17 | |
| *** lappy-rook has quit IRC | 01:19 | |
| *** mriedem has joined #openstack-nova | 01:19 | |
| *** ducttape_ has joined #openstack-nova | 01:20 | |
| *** Sukhdev has quit IRC | 01:22 | |
| *** doug-fish has joined #openstack-nova | 01:24 | |
| *** shaohe_feng has quit IRC | 01:25 | |
| *** shaohe_feng has joined #openstack-nova | 01:25 | |
| *** EinstCrazy has joined #openstack-nova | 01:26 | |
| *** raunak has joined #openstack-nova | 01:29 | |
| *** doug-fish has quit IRC | 01:29 | |
| *** Jeffrey4l has joined #openstack-nova | 01:30 | |
| *** raunak has quit IRC | 01:34 | |
| *** baoli has joined #openstack-nova | 01:35 | |
| *** shaohe_feng has quit IRC | 01:35 | |
| *** whenry has joined #openstack-nova | 01:36 | |
| *** skoude has quit IRC | 01:36 | |
| *** shaohe_feng has joined #openstack-nova | 01:36 | |
| *** moshele has quit IRC | 01:37 | |
| *** skoude has joined #openstack-nova | 01:37 | |
| *** Sukhdev has joined #openstack-nova | 01:39 | |
| openstackgerrit | kevin shen proposed openstack/nova: fix the limit of injected file content length is incorrect problem https://review.openstack.org/317865 | 01:40 |
| *** thorst_ has quit IRC | 01:42 | |
| *** Fdaisuke has joined #openstack-nova | 01:42 | |
| *** al_indigo has quit IRC | 01:42 | |
| *** thorst_ has joined #openstack-nova | 01:42 | |
| *** Sukhdev has quit IRC | 01:45 | |
| *** shaohe_feng has quit IRC | 01:45 | |
| *** shaohe_feng has joined #openstack-nova | 01:46 | |
| *** gjayavelu has joined #openstack-nova | 01:46 | |
| *** iceyao has quit IRC | 01:48 | |
| *** smatzek_ has joined #openstack-nova | 01:49 | |
| *** baoli has quit IRC | 01:51 | |
| *** baoli has joined #openstack-nova | 01:51 | |
| *** thorst_ has quit IRC | 01:51 | |
| *** ljxiash has joined #openstack-nova | 01:54 | |
| openstackgerrit | Siri Kim proposed openstack/nova: Title: bugfix https://review.openstack.org/318405 | 01:54 |
| *** yamahata has joined #openstack-nova | 01:55 | |
| *** shaohe_feng has quit IRC | 01:55 | |
| *** shaohe_feng has joined #openstack-nova | 01:56 | |
| *** ducttape_ has quit IRC | 01:56 | |
| *** baoli has quit IRC | 01:57 | |
| *** pradk has joined #openstack-nova | 01:59 | |
| *** whenry has quit IRC | 02:00 | |
| *** EinstCra_ has joined #openstack-nova | 02:00 | |
| *** EinstCrazy has quit IRC | 02:03 | |
| *** lixiaoy1 has joined #openstack-nova | 02:04 | |
| *** iceyao has joined #openstack-nova | 02:05 | |
| *** iceyao has quit IRC | 02:06 | |
| *** shaohe_feng has quit IRC | 02:06 | |
| *** iceyao_ has joined #openstack-nova | 02:08 | |
| *** vishwanathj has quit IRC | 02:09 | |
| *** shaohe_feng has joined #openstack-nova | 02:09 | |
| *** hieulq_ has quit IRC | 02:09 | |
| *** pradk has quit IRC | 02:10 | |
| *** shaohe_feng has quit IRC | 02:16 | |
| *** shaohe_feng has joined #openstack-nova | 02:17 | |
| *** yonglihe has joined #openstack-nova | 02:17 | |
| *** unicell has quit IRC | 02:20 | |
| *** unicell has joined #openstack-nova | 02:22 | |
| *** unicell has quit IRC | 02:22 | |
| *** sacharya_ has joined #openstack-nova | 02:26 | |
| *** shaohe_feng has quit IRC | 02:26 | |
| *** shaohe_feng has joined #openstack-nova | 02:27 | |
| *** ducttape_ has joined #openstack-nova | 02:28 | |
| *** sacharya has quit IRC | 02:29 | |
| *** jinxing has quit IRC | 02:30 | |
| *** sree has joined #openstack-nova | 02:30 | |
| *** houming has joined #openstack-nova | 02:30 | |
| *** BjoernT has joined #openstack-nova | 02:32 | |
| mriedem | did we ever store the image href (url) in the instances table? | 02:33 |
| mriedem | i hope not | 02:33 |
| mriedem | johnthetubaguy: i'm +2 on the glance v2 spec now https://review.openstack.org/#/c/301741/ | 02:33 |
| *** sree has quit IRC | 02:33 | |
| *** mriedem has quit IRC | 02:33 | |
| *** ducttape_ has quit IRC | 02:34 | |
| *** yamahata has quit IRC | 02:34 | |
| *** links has joined #openstack-nova | 02:35 | |
| *** baoli has joined #openstack-nova | 02:35 | |
| *** shaohe_feng has quit IRC | 02:36 | |
| *** jinxing has joined #openstack-nova | 02:37 | |
| *** shaohe_feng has joined #openstack-nova | 02:37 | |
| *** baoli has quit IRC | 02:39 | |
| *** BjoernT has quit IRC | 02:39 | |
| *** baoli has joined #openstack-nova | 02:39 | |
| *** smatzek_ has quit IRC | 02:40 | |
| *** sree has joined #openstack-nova | 02:40 | |
| *** mdrabe has joined #openstack-nova | 02:41 | |
| *** baoli has quit IRC | 02:42 | |
| *** baoli has joined #openstack-nova | 02:43 | |
| *** mdrabe has quit IRC | 02:43 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Clean up deprecated CLI options https://review.openstack.org/317795 | 02:43 |
| *** shaohe_feng has quit IRC | 02:47 | |
| *** shaohe_feng has joined #openstack-nova | 02:47 | |
| *** woodster_ has quit IRC | 02:48 | |
| *** thorst_ has joined #openstack-nova | 02:48 | |
| *** lakshmiS has quit IRC | 02:52 | |
| *** mdrabe has joined #openstack-nova | 02:54 | |
| *** thorst_ has quit IRC | 02:57 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Clean up deprecated CLI options https://review.openstack.org/317795 | 02:57 |
| *** shaohe_feng has quit IRC | 02:57 | |
| *** jinxing has quit IRC | 02:58 | |
| *** EinstCra_ has quit IRC | 02:58 | |
| *** iceyao_ has quit IRC | 02:58 | |
| *** shaohe_feng has joined #openstack-nova | 02:58 | |
| *** tangchen_ has joined #openstack-nova | 02:58 | |
| *** iceyao has joined #openstack-nova | 02:59 | |
| *** EinstCrazy has joined #openstack-nova | 02:59 | |
| *** jinxing has joined #openstack-nova | 02:59 | |
| *** ducttape_ has joined #openstack-nova | 03:01 | |
| *** raunak has joined #openstack-nova | 03:03 | |
| *** shaohe_feng has quit IRC | 03:07 | |
| *** shaohe_feng has joined #openstack-nova | 03:08 | |
| *** baoli has quit IRC | 03:09 | |
| *** zhangyufei has joined #openstack-nova | 03:10 | |
| *** ducttape_ has quit IRC | 03:12 | |
| *** ijw_ has quit IRC | 03:14 | |
| *** gjayavel_ has joined #openstack-nova | 03:15 | |
| *** rama_y has joined #openstack-nova | 03:16 | |
| *** AmyLee7 has joined #openstack-nova | 03:17 | |
| *** shaohe_feng has quit IRC | 03:17 | |
| *** ozialien10 has quit IRC | 03:18 | |
| *** ozialien10 has joined #openstack-nova | 03:19 | |
| *** gjayavelu has quit IRC | 03:19 | |
| *** shaohe_feng has joined #openstack-nova | 03:21 | |
| *** mdrabe has quit IRC | 03:21 | |
| *** gmann_ has joined #openstack-nova | 03:27 | |
| *** shaohe_feng has quit IRC | 03:28 | |
| *** shaohe_feng has joined #openstack-nova | 03:28 | |
| *** zz_dimtruck is now known as dimtruck | 03:31 | |
| *** chip__ has joined #openstack-nova | 03:38 | |
| *** shaohe_feng has quit IRC | 03:38 | |
| *** shaohe_feng has joined #openstack-nova | 03:38 | |
| *** ljxiash has quit IRC | 03:41 | |
| *** garyk1 has joined #openstack-nova | 03:47 | |
| *** shaohe_feng has quit IRC | 03:48 | |
| *** shaohe_feng has joined #openstack-nova | 03:49 | |
| *** ijw has joined #openstack-nova | 03:53 | |
| *** thorst_ has joined #openstack-nova | 03:54 | |
| *** ljxiash has joined #openstack-nova | 03:56 | |
| *** shaohe_feng has quit IRC | 03:58 | |
| *** shaohe_feng has joined #openstack-nova | 03:59 | |
| *** ljxiash has quit IRC | 04:00 | |
| *** links has quit IRC | 04:00 | |
| *** thorst_ has quit IRC | 04:01 | |
| *** prateek has joined #openstack-nova | 04:02 | |
| *** gmann_ has quit IRC | 04:04 | |
| *** flwang has quit IRC | 04:04 | |
| *** amotoki has joined #openstack-nova | 04:06 | |
| *** gmann_ has joined #openstack-nova | 04:07 | |
| *** shaohe_feng has quit IRC | 04:09 | |
| *** shaohe_feng has joined #openstack-nova | 04:10 | |
| *** rama_y has quit IRC | 04:10 | |
| *** prateek has quit IRC | 04:11 | |
| *** flwang has joined #openstack-nova | 04:11 | |
| *** gmann_ has quit IRC | 04:11 | |
| *** links has joined #openstack-nova | 04:16 | |
| *** Jeffrey4l has quit IRC | 04:16 | |
| *** zhangyufei has quit IRC | 04:18 | |
| *** dimtruck is now known as zz_dimtruck | 04:18 | |
| *** jinxing has quit IRC | 04:19 | |
| *** ducttape_ has joined #openstack-nova | 04:19 | |
| *** shaohe_feng has quit IRC | 04:19 | |
| *** shaohe_feng has joined #openstack-nova | 04:20 | |
| *** Sukhdev has joined #openstack-nova | 04:20 | |
| *** ducttape_ has quit IRC | 04:21 | |
| *** prateek has joined #openstack-nova | 04:21 | |
| *** can8dnSix has joined #openstack-nova | 04:21 | |
| *** Sukhdev has quit IRC | 04:24 | |
| *** Sukhdev has joined #openstack-nova | 04:24 | |
| *** shaohe_feng has quit IRC | 04:29 | |
| *** shaohe_feng has joined #openstack-nova | 04:30 | |
| *** AmyLee7 has quit IRC | 04:33 | |
| *** shaohe_feng has quit IRC | 04:39 | |
| *** shaohe_feng has joined #openstack-nova | 04:40 | |
| *** armax has quit IRC | 04:40 | |
| *** kengo_sakai has joined #openstack-nova | 04:41 | |
| *** zz_dimtruck is now known as dimtruck | 04:43 | |
| *** kengo_sakai has quit IRC | 04:46 | |
| *** chip__ has quit IRC | 04:47 | |
| *** chip__ has joined #openstack-nova | 04:48 | |
| *** shaohe_feng has quit IRC | 04:50 | |
| *** hemna has joined #openstack-nova | 04:52 | |
| *** shaohe_feng has joined #openstack-nova | 04:53 | |
| *** gongysh has joined #openstack-nova | 04:53 | |
| *** hemna has quit IRC | 04:53 | |
| *** ljxiash has joined #openstack-nova | 04:53 | |
| *** hemna has joined #openstack-nova | 04:54 | |
| *** armax has joined #openstack-nova | 04:54 | |
| *** mnestratov has quit IRC | 04:56 | |
| *** hemna has quit IRC | 04:56 | |
| *** ijw has quit IRC | 04:57 | |
| *** thorst_ has joined #openstack-nova | 04:59 | |
| *** shaohe_feng has quit IRC | 05:00 | |
| *** shaohe_feng has joined #openstack-nova | 05:01 | |
| *** armax has quit IRC | 05:01 | |
| *** kengo_sakai has joined #openstack-nova | 05:02 | |
| *** armax has joined #openstack-nova | 05:03 | |
| *** gongysh has quit IRC | 05:04 | |
| *** thorst_ has quit IRC | 05:06 | |
| *** sudipto has joined #openstack-nova | 05:06 | |
| *** cheneydc has quit IRC | 05:07 | |
| *** syfy323 has quit IRC | 05:09 | |
| *** cheneydc has joined #openstack-nova | 05:10 | |
| *** shaohe_feng has quit IRC | 05:10 | |
| *** shaohe_feng has joined #openstack-nova | 05:11 | |
| *** doug-fish has joined #openstack-nova | 05:11 | |
| *** can8dnSix has quit IRC | 05:12 | |
| *** klkumar_ has joined #openstack-nova | 05:13 | |
| *** doug-fish has quit IRC | 05:15 | |
| *** syfy323 has joined #openstack-nova | 05:17 | |
| *** shaohe_feng has quit IRC | 05:20 | |
| *** shaohe_feng has joined #openstack-nova | 05:21 | |
| *** chip__ has quit IRC | 05:22 | |
| *** chip__ has joined #openstack-nova | 05:24 | |
| *** irenab has joined #openstack-nova | 05:25 | |
| *** kengo_sakai has quit IRC | 05:27 | |
| *** markvoelker has joined #openstack-nova | 05:27 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Enable 'null' value for user_data in V2.1 API https://review.openstack.org/318447 | 05:29 |
| *** shaohe_feng has quit IRC | 05:31 | |
| *** shaohe_feng has joined #openstack-nova | 05:31 | |
| *** markvoelker has quit IRC | 05:34 | |
| *** syfy323 has quit IRC | 05:35 | |
| *** dimtruck is now known as zz_dimtruck | 05:35 | |
| *** syfy323 has joined #openstack-nova | 05:36 | |
| *** zhangyufei has joined #openstack-nova | 05:36 | |
| *** unicell has joined #openstack-nova | 05:36 | |
| *** jinxing has joined #openstack-nova | 05:37 | |
| *** shaohe_feng has quit IRC | 05:41 | |
| *** shaohe_feng has joined #openstack-nova | 05:42 | |
| *** yamahata has joined #openstack-nova | 05:44 | |
| *** zhangyufei has quit IRC | 05:51 | |
| *** shaohe_feng has quit IRC | 05:51 | |
| *** salv-orlando has joined #openstack-nova | 05:51 | |
| *** shaohe_feng has joined #openstack-nova | 05:52 | |
| *** zhangyufei has joined #openstack-nova | 05:55 | |
| *** fawadkhaliq has joined #openstack-nova | 05:57 | |
| *** Sukhdev has quit IRC | 05:57 | |
| *** apoorv has joined #openstack-nova | 05:57 | |
| *** doug-fish has joined #openstack-nova | 05:57 | |
| *** Jeffrey4l has joined #openstack-nova | 05:58 | |
| *** mjura has joined #openstack-nova | 06:00 | |
| *** armax has quit IRC | 06:00 | |
| *** doug-fish has quit IRC | 06:01 | |
| *** shaohe_feng has quit IRC | 06:01 | |
| *** malos has quit IRC | 06:02 | |
| *** shaohe_feng has joined #openstack-nova | 06:02 | |
| *** raunak has quit IRC | 06:02 | |
| *** thorst_ has joined #openstack-nova | 06:04 | |
| *** mnestratov has joined #openstack-nova | 06:06 | |
| *** mjura has quit IRC | 06:07 | |
| *** mjura has joined #openstack-nova | 06:07 | |
| *** moshele has joined #openstack-nova | 06:08 | |
| *** Oku_OS-away is now known as Oku_OS | 06:10 | |
| *** thorst_ has quit IRC | 06:10 | |
| *** shaohe_feng has quit IRC | 06:12 | |
| *** shaohe_feng has joined #openstack-nova | 06:12 | |
| *** rcernin has joined #openstack-nova | 06:13 | |
| *** Jeffrey4l has quit IRC | 06:18 | |
| *** jgriffith has quit IRC | 06:19 | |
| *** edand has joined #openstack-nova | 06:19 | |
| *** unicell has quit IRC | 06:20 | |
| *** unicell has joined #openstack-nova | 06:20 | |
| *** lpetrut has joined #openstack-nova | 06:22 | |
| *** shaohe_feng has quit IRC | 06:22 | |
| *** gmann_ has joined #openstack-nova | 06:23 | |
| *** sfinucan has joined #openstack-nova | 06:24 | |
| *** shaohe_feng has joined #openstack-nova | 06:25 | |
| *** jgriffith has joined #openstack-nova | 06:27 | |
| *** jgriffith is now known as Guest60864 | 06:27 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: Remove unused itype parameter from get migration context https://review.openstack.org/317977 | 06:27 |
| *** tbachman has quit IRC | 06:28 | |
| *** tbachman has joined #openstack-nova | 06:31 | |
| *** salv-orlando has quit IRC | 06:32 | |
| *** shaohe_feng has quit IRC | 06:32 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Fix the versions API for api-ref https://review.openstack.org/308720 | 06:33 |
| *** sudipto has quit IRC | 06:34 | |
| *** shaohe_feng has joined #openstack-nova | 06:35 | |
| *** sudipto has joined #openstack-nova | 06:36 | |
| *** shaohe_feng has quit IRC | 06:42 | |
| *** nkrinner_afk is now known as nkrinner | 06:43 | |
| *** belmoreira has joined #openstack-nova | 06:45 | |
| *** shaohe_feng has joined #openstack-nova | 06:45 | |
| *** malos has joined #openstack-nova | 06:46 | |
| *** gjayavel_ has quit IRC | 06:47 | |
| *** gjayavelu has joined #openstack-nova | 06:50 | |
| *** laszloh has joined #openstack-nova | 06:51 | |
| *** jinxing has quit IRC | 06:51 | |
| *** zhangyufei has quit IRC | 06:51 | |
| *** EinstCrazy has quit IRC | 06:51 | |
| *** iceyao has quit IRC | 06:51 | |
| *** liverpooler has joined #openstack-nova | 06:51 | |
| *** zhangyufei has joined #openstack-nova | 06:52 | |
| *** jinxing has joined #openstack-nova | 06:52 | |
| *** EinstCrazy has joined #openstack-nova | 06:52 | |
| *** iceyao has joined #openstack-nova | 06:52 | |
| *** shaohe_feng has quit IRC | 06:53 | |
| *** shaohe_feng has joined #openstack-nova | 06:53 | |
| *** jinxing has quit IRC | 06:54 | |
| *** EinstCrazy has quit IRC | 06:54 | |
| *** zhangyufei has quit IRC | 06:54 | |
| *** iceyao has quit IRC | 06:54 | |
| *** iceyao has joined #openstack-nova | 06:55 | |
| *** EinstCrazy has joined #openstack-nova | 06:56 | |
| *** zhangyufei has joined #openstack-nova | 06:56 | |
| *** jinxing has joined #openstack-nova | 06:57 | |
| *** iceyao has quit IRC | 06:57 | |
| *** zhangyufei has quit IRC | 06:57 | |
| *** EinstCrazy has quit IRC | 06:57 | |
| *** sacharya_ has quit IRC | 06:57 | |
| *** andreas_s has joined #openstack-nova | 06:58 | |
| *** EinstCrazy has joined #openstack-nova | 06:58 | |
| *** iceyao has joined #openstack-nova | 06:58 | |
| *** zhangyufei has joined #openstack-nova | 06:58 | |
| *** sridharg has joined #openstack-nova | 06:59 | |
| *** gjayavelu has quit IRC | 06:59 | |
| *** hoangcx has quit IRC | 06:59 | |
| *** jaosorior has quit IRC | 06:59 | |
| *** jaosorior has joined #openstack-nova | 07:00 | |
| *** EinstCrazy has quit IRC | 07:00 | |
| *** jinxing has quit IRC | 07:00 | |
| *** iceyao has quit IRC | 07:00 | |
| *** zhangyufei has quit IRC | 07:00 | |
| *** iceyao has joined #openstack-nova | 07:01 | |
| *** zhangyufei has joined #openstack-nova | 07:01 | |
| *** EinstCrazy has joined #openstack-nova | 07:01 | |
| *** jinxing has joined #openstack-nova | 07:02 | |
| *** shaohe_feng has quit IRC | 07:03 | |
| *** shaohe_feng has joined #openstack-nova | 07:03 | |
| *** cfriesen has quit IRC | 07:04 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Clean up deprecated CLI options https://review.openstack.org/317795 | 07:05 |
| *** gjayavelu has joined #openstack-nova | 07:06 | |
| *** roeyc has joined #openstack-nova | 07:07 | |
| *** roeyc has quit IRC | 07:08 | |
| *** thorst_ has joined #openstack-nova | 07:08 | |
| *** roeyc has joined #openstack-nova | 07:08 | |
| *** realitix has joined #openstack-nova | 07:09 | |
| *** jichen has joined #openstack-nova | 07:09 | |
| realitix | Hello everyone. Do you know why we can't rename a flavor ? Is there a good reason or is it just unimplemented ? | 07:11 |
| *** shaohe_feng has quit IRC | 07:13 | |
| *** shaohe_feng has joined #openstack-nova | 07:13 | |
| openstackgerrit | Sudipta Biswas proposed openstack/nova-specs: Complete Nova support for Glance's v2 https://review.openstack.org/301741 | 07:15 |
| *** thorst_ has quit IRC | 07:15 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Clean up deprecated CLI options https://review.openstack.org/317795 | 07:17 |
| *** gmann_ has quit IRC | 07:19 | |
| *** sfinucan has quit IRC | 07:22 | |
| *** shaohe_feng has quit IRC | 07:23 | |
| *** jlanoux has joined #openstack-nova | 07:24 | |
| *** mpavone has joined #openstack-nova | 07:24 | |
| *** shaohe_feng has joined #openstack-nova | 07:27 | |
| *** gjayavelu has quit IRC | 07:29 | |
| *** shaohe_feng has quit IRC | 07:34 | |
| *** lpetrut has quit IRC | 07:34 | |
| *** shaohe_feng has joined #openstack-nova | 07:35 | |
| *** dedery has joined #openstack-nova | 07:35 | |
| *** irenab_ has joined #openstack-nova | 07:36 | |
| *** ihrachys has joined #openstack-nova | 07:37 | |
| *** irenab has quit IRC | 07:38 | |
| *** irenab_ is now known as irenab | 07:38 | |
| *** fawadkhaliq has quit IRC | 07:39 | |
| *** fawadkhaliq has joined #openstack-nova | 07:39 | |
| *** matrohon has joined #openstack-nova | 07:40 | |
| *** ihrachys has quit IRC | 07:41 | |
| *** lpetrut has joined #openstack-nova | 07:43 | |
| *** shaohe_feng has quit IRC | 07:44 | |
| *** shaohe_feng has joined #openstack-nova | 07:45 | |
| *** salv-orl_ has joined #openstack-nova | 07:47 | |
| *** salv-orl_ has quit IRC | 07:47 | |
| *** danpawlik has joined #openstack-nova | 07:47 | |
| *** markmc``` is now known as markmc | 07:48 | |
| *** salv-orlando has joined #openstack-nova | 07:48 | |
| *** zhangyufei has quit IRC | 07:49 | |
| *** zenoway has joined #openstack-nova | 07:50 | |
| *** dmk0202 has joined #openstack-nova | 07:50 | |
| *** zhangyufei has joined #openstack-nova | 07:51 | |
| *** gmann_ has joined #openstack-nova | 07:52 | |
| openstackgerrit | Zhongcheng Lao proposed openstack/nova: VMware: Omit exceptions during snapshot cleanup https://review.openstack.org/216086 | 07:53 |
| *** haukebruno has joined #openstack-nova | 07:53 | |
| *** markus_z has joined #openstack-nova | 07:54 | |
| *** shaohe_feng has quit IRC | 07:54 | |
| *** shaohe_feng has joined #openstack-nova | 07:55 | |
| bauzas | good morning Nova | 07:56 |
| *** EinstCra_ has joined #openstack-nova | 07:57 | |
| *** sacharya has joined #openstack-nova | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** EinstCrazy has quit IRC | 08:00 | |
| *** iceyao_ has joined #openstack-nova | 08:02 | |
| *** zzzeek has joined #openstack-nova | 08:02 | |
| *** iceyao has quit IRC | 08:03 | |
| gibi | good morning bauzas | 08:03 |
| *** sacharya has quit IRC | 08:03 | |
| *** shaohe_feng has quit IRC | 08:04 | |
| *** jamielennox is now known as jamielennox|away | 08:05 | |
| *** shaohe_feng has joined #openstack-nova | 08:08 | |
| *** cheran has quit IRC | 08:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: [WIP] Testing latest u-c https://review.openstack.org/303345 | 08:10 |
| *** vnogin has joined #openstack-nova | 08:10 | |
| *** hoangcx has joined #openstack-nova | 08:10 | |
| *** lpetrut has quit IRC | 08:11 | |
| *** iceyao has joined #openstack-nova | 08:12 | |
| *** iceyao has quit IRC | 08:13 | |
| *** thorst_ has joined #openstack-nova | 08:14 | |
| *** iceyao has joined #openstack-nova | 08:14 | |
| *** danpb has joined #openstack-nova | 08:14 | |
| *** shaohe_feng has quit IRC | 08:15 | |
| *** markus_z has quit IRC | 08:15 | |
| *** shaohe_feng has joined #openstack-nova | 08:16 | |
| *** iceyao_ has quit IRC | 08:16 | |
| *** markus_z has joined #openstack-nova | 08:16 | |
| *** yamahata has quit IRC | 08:17 | |
| *** mpavone has quit IRC | 08:18 | |
| openstackgerrit | Hiroyuki Eguchi proposed openstack/nova-specs: Versioned notification for keypairs https://review.openstack.org/300942 | 08:19 |
| *** ihrachys has joined #openstack-nova | 08:20 | |
| *** daemontool has joined #openstack-nova | 08:21 | |
| *** tangchen_ has quit IRC | 08:21 | |
| *** thorst_ has quit IRC | 08:21 | |
| *** ljjjustin has quit IRC | 08:21 | |
| *** ljjjustin has joined #openstack-nova | 08:23 | |
| *** mpavone has joined #openstack-nova | 08:23 | |
| *** shaohe_feng has quit IRC | 08:25 | |
| *** lucas-dinner is now known as lucasagomes | 08:26 | |
| *** ozialien10 has quit IRC | 08:27 | |
| *** ozialien10 has joined #openstack-nova | 08:27 | |
| *** shaohe_feng has joined #openstack-nova | 08:28 | |
| *** aloga has quit IRC | 08:28 | |
| *** lpetrut has joined #openstack-nova | 08:29 | |
| *** hoangcx has quit IRC | 08:29 | |
| *** hoangcx has joined #openstack-nova | 08:29 | |
| *** aloga has joined #openstack-nova | 08:29 | |
| *** jistr has joined #openstack-nova | 08:30 | |
| *** liusheng has quit IRC | 08:34 | |
| *** liusheng has joined #openstack-nova | 08:35 | |
| *** shaohe_feng has quit IRC | 08:35 | |
| *** shaohe_feng has joined #openstack-nova | 08:36 | |
| lyarwood_ | mdbooth: morning, could I ask for a review of https://review.openstack.org/318266 if you have time? | 08:36 |
| *** EinstCrazy has joined #openstack-nova | 08:38 | |
| *** yassine has joined #openstack-nova | 08:40 | |
| *** yassine is now known as Guest86061 | 08:40 | |
| *** EinstCra_ has quit IRC | 08:40 | |
| *** lpetrut has quit IRC | 08:41 | |
| *** ljxiash has quit IRC | 08:42 | |
| lifeless | alaski: ahha, found it. | 08:42 |
| *** ljxiash has joined #openstack-nova | 08:42 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Clean up deprecated CLI options https://review.openstack.org/317795 | 08:42 |
| lifeless | alaski: we setattr to unpatch things, even if what we patched didn't actually exist on the object previously | 08:43 |
| lifeless | alaski: I'm going to have a stab at fixing that | 08:43 |
| *** iceyao has quit IRC | 08:45 | |
| *** shaohe_feng has quit IRC | 08:45 | |
| *** shaohe_feng has joined #openstack-nova | 08:46 | |
| *** ljxiash has quit IRC | 08:47 | |
| *** cdent has joined #openstack-nova | 08:48 | |
| *** iceyao has joined #openstack-nova | 08:51 | |
| *** PaulMurray has joined #openstack-nova | 08:51 | |
| *** al_indigo has joined #openstack-nova | 08:53 | |
| *** shaohe_feng has quit IRC | 08:56 | |
| *** shaohe_feng has joined #openstack-nova | 08:57 | |
| *** paul-carlton2 has quit IRC | 08:58 | |
| *** gmann_ has quit IRC | 08:58 | |
| *** derekh has joined #openstack-nova | 09:00 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: remove import_opts from cloudpipe section https://review.openstack.org/318517 | 09:02 |
| *** shaohe_feng has quit IRC | 09:06 | |
| *** al_indigo has quit IRC | 09:07 | |
| *** shaohe_feng has joined #openstack-nova | 09:09 | |
| mdbooth | lyarwood_: Sure | 09:11 |
| lyarwood_ | mdbooth: thanks | 09:12 |
| *** shaohe_feng has quit IRC | 09:16 | |
| *** shaohe_feng has joined #openstack-nova | 09:17 | |
| *** sambetts|afk is now known as sambetts | 09:17 | |
| *** lpetrut has joined #openstack-nova | 09:18 | |
| *** PaulMurray has quit IRC | 09:18 | |
| *** ptm_away has joined #openstack-nova | 09:19 | |
| *** thorst_ has joined #openstack-nova | 09:19 | |
| *** chenfan has joined #openstack-nova | 09:21 | |
| *** eliqiao has quit IRC | 09:21 | |
| *** chenfan has quit IRC | 09:23 | |
| *** ihrachys has quit IRC | 09:23 | |
| *** chenfan has joined #openstack-nova | 09:24 | |
| *** houming has quit IRC | 09:24 | |
| *** fawadkhaliq has quit IRC | 09:24 | |
| *** fawadkhaliq has joined #openstack-nova | 09:24 | |
| *** chenfan has quit IRC | 09:25 | |
| *** thorst_ has quit IRC | 09:26 | |
| *** shaohe_feng has quit IRC | 09:26 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform generic instance action notifications https://review.openstack.org/313654 | 09:27 |
| *** chenfan has joined #openstack-nova | 09:28 | |
| *** shaohe_feng has joined #openstack-nova | 09:28 | |
| *** chenfan has quit IRC | 09:29 | |
| *** ccard has joined #openstack-nova | 09:29 | |
| *** chenfan has joined #openstack-nova | 09:31 | |
| *** ccard_ has quit IRC | 09:31 | |
| *** EinstCrazy has quit IRC | 09:36 | |
| *** EinstCrazy has joined #openstack-nova | 09:36 | |
| *** shaohe_feng has quit IRC | 09:37 | |
| *** shaohe_feng has joined #openstack-nova | 09:38 | |
| *** iceyao has quit IRC | 09:40 | |
| *** iceyao has joined #openstack-nova | 09:41 | |
| *** iceyao_ has joined #openstack-nova | 09:42 | |
| *** iceyao__ has joined #openstack-nova | 09:43 | |
| *** iceyao has quit IRC | 09:46 | |
| *** ericksonsantos has quit IRC | 09:46 | |
| *** laszloh has quit IRC | 09:46 | |
| *** shaohe_feng has quit IRC | 09:47 | |
| *** iceyao_ has quit IRC | 09:47 | |
| *** shaohe_feng has joined #openstack-nova | 09:48 | |
| *** ericksonsantos has joined #openstack-nova | 09:49 | |
| *** eslamelhusseiny has joined #openstack-nova | 09:49 | |
| *** EinstCrazy has quit IRC | 09:51 | |
| openstackgerrit | Sharat Sharma proposed openstack/nova: Adding "security_group_default_rules" parameter https://review.openstack.org/315557 | 09:54 |
| *** rk4n has joined #openstack-nova | 09:55 | |
| *** hoangcx has quit IRC | 09:56 | |
| *** shaohe_feng has quit IRC | 09:57 | |
| *** shaohe_feng has joined #openstack-nova | 09:58 | |
| *** eslamelhusseiny has quit IRC | 09:59 | |
| *** cheneydc has quit IRC | 10:00 | |
| *** al_indigo has joined #openstack-nova | 10:01 | |
| *** porunov has joined #openstack-nova | 10:01 | |
| *** porunov has left #openstack-nova | 10:02 | |
| *** al_indigo has quit IRC | 10:03 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Enables Py34 tests for unit.api.openstack.compute.test_server_tags https://review.openstack.org/318543 | 10:03 |
| mdbooth | lyarwood_: That detach_volume patch requires a lot of context! | 10:03 |
| mdbooth | So, there's a comment in there just above the lines you patches (line 1286) | 10:04 |
| mdbooth | Which says that this is called post live migration | 10:04 |
| mdbooth | And in that case we *would* want to disconnect the volume, despite not being on the instance's host | 10:04 |
| *** matrohon has quit IRC | 10:04 | |
| mdbooth | However, afaict it is not actually called post live migration | 10:04 |
| *** hoangcx has joined #openstack-nova | 10:05 | |
| *** edand has quit IRC | 10:07 | |
| *** shaohe_feng has quit IRC | 10:07 | |
| *** zhangyufei has quit IRC | 10:08 | |
| *** shaohe_feng has joined #openstack-nova | 10:08 | |
| *** zhangyufei has joined #openstack-nova | 10:11 | |
| *** gjayavelu has joined #openstack-nova | 10:13 | |
| *** salv-orl_ has joined #openstack-nova | 10:14 | |
| *** rmart04 has joined #openstack-nova | 10:14 | |
| *** mvk_ has quit IRC | 10:15 | |
| lyarwood_ | mdbooth: hey sorry hilights aren't working in here for some reason | 10:17 |
| *** salv-orlando has quit IRC | 10:17 | |
| mdbooth | lyarwood_: Just thinking myself into another knot, btw. | 10:17 |
| *** gjayavelu has quit IRC | 10:17 | |
| lyarwood_ | mdbooth: and yeah sorry for the lack of context in the commit msg | 10:18 |
| mdbooth | So your test does: if connector.get('host') == instance.host | 10:18 |
| *** shaohe_feng has quit IRC | 10:18 | |
| *** claudiub has joined #openstack-nova | 10:18 | |
| mdbooth | However, aren't we interested in if it matches the current host? | 10:18 |
| lyarwood_ | mdbooth: right, and that connector is pulled from the current host | 10:18 |
| lyarwood_ | mdbooth: not from the connection_info | 10:18 |
| *** shaohe_feng has joined #openstack-nova | 10:18 | |
| mdbooth | Whoa | 10:19 |
| mdbooth | Can't we get that information more directly? | 10:19 |
| mdbooth | host=CONF.host) | 10:20 |
| mdbooth | (from get_volume_connector) | 10:20 |
| lyarwood_ | mdbooth: yeah we can just do that instead | 10:20 |
| mdbooth | Would if instance.host == CONF.host work there? | 10:20 |
| lyarwood_ | mdbooth: but your point re post live migration is a deal breaker for this | 10:21 |
| mdbooth | lyarwood_: I don't think it is, actually | 10:21 |
| mdbooth | I think that comment is a bug | 10:21 |
| mdbooth | I've commented on that in the review | 10:21 |
| lyarwood_ | ah and we don't call this? | 10:21 |
| *** cdent has quit IRC | 10:21 | |
| mdbooth | ...sec, let me post the interim | 10:21 |
| lyarwood_ | thanks | 10:21 |
| mdbooth | thar you go | 10:21 |
| mdbooth | lyarwood_: However, *please* double check that | 10:22 |
| mdbooth | Because it's not exactly straightforward | 10:22 |
| mdbooth | Also might be worth specifically testing it if you have the cycles | 10:22 |
| lyarwood_ | yeah I have a multi host devstack env ready to go to test this | 10:22 |
| lyarwood_ | I'll get on with that now, thanks again | 10:23 |
| mdbooth | lyarwood_: Can we not add that guard the level up? | 10:25 |
| mdbooth | It seems more intuitive to me to not call detach_volume() at all if you know it isn't required | 10:25 |
| mdbooth | And the code above doesn't look relevant in the remote rebuild case, either | 10:26 |
| mdbooth | I suspect it's just logging an error, in fact | 10:26 |
| lyarwood_ | mdbooth: in the compute layer? | 10:26 |
| mdbooth | lyarwood_: Just reminding myself where it's called from, but I guess so, yes | 10:26 |
| lyarwood_ | mdbooth: I really wanted to avoid breaking other virt drivers tbh | 10:27 |
| *** zhangyufei has quit IRC | 10:27 | |
| *** sahid has joined #openstack-nova | 10:27 | |
| mdbooth | lyarwood_: But would it? | 10:27 |
| mdbooth | If detach_volume has the same semantics across all drivers | 10:27 |
| mdbooth | i.e. cleanup the host | 10:27 |
| *** zhangyufei has joined #openstack-nova | 10:27 | |
| lyarwood_ | mdbooth: hmm I was worried about the single compute host drivers tbh | 10:27 |
| mdbooth | Then that should be ok, right? | 10:27 |
| sahid | danpb: can you re+W this serie when you have a moment https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/libvirt-clean-driver i had to rebase it | 10:28 |
| *** shaohe_feng has quit IRC | 10:28 | |
| *** shaohe_feng has joined #openstack-nova | 10:29 | |
| *** zhangyufei has quit IRC | 10:29 | |
| lyarwood_ | mdbooth: I'll move it up and run it past CI, hopefully it's valid. | 10:30 |
| *** zhangyufei has joined #openstack-nova | 10:30 | |
| *** zhangyufei has quit IRC | 10:30 | |
| danpb | sahid: when i get done wading through email backlog... | 10:32 |
| *** sfinucan has joined #openstack-nova | 10:35 | |
| *** rtheis has joined #openstack-nova | 10:35 | |
| *** sdague has joined #openstack-nova | 10:37 | |
| *** shaohe_feng has quit IRC | 10:38 | |
| *** shaohe_feng has joined #openstack-nova | 10:39 | |
| *** ansiwen_ is now known as ansiwen | 10:41 | |
| *** smatzek_ has joined #openstack-nova | 10:46 | |
| *** apoorv has quit IRC | 10:47 | |
| *** shaohe_feng has quit IRC | 10:48 | |
| *** yonglihe has quit IRC | 10:48 | |
| *** hieulq has quit IRC | 10:49 | |
| *** shaohe_feng has joined #openstack-nova | 10:49 | |
| *** mvk_ has joined #openstack-nova | 10:50 | |
| *** smatzek_ is now known as smatzek | 10:50 | |
| *** apoorv has joined #openstack-nova | 10:53 | |
| *** flwang1 has joined #openstack-nova | 10:54 | |
| openstackgerrit | sahid proposed openstack/nova: Introduce VNC Security Proxy Framework https://review.openstack.org/115483 | 10:55 |
| openstackgerrit | sahid proposed openstack/nova: Add VeNCrypt (TLS/x509) Security Proxy Driver https://review.openstack.org/115484 | 10:55 |
| *** cdent has joined #openstack-nova | 10:55 | |
| flwang1 | hi guys, can i put a customized filter to another place instead of nova/scheduler/filter? thanks | 10:55 |
| bauzas | flwang1: sure, easy thing | 10:55 |
| flwang1 | bauzas: how? ;) | 10:55 |
| bauzas | flwang1: http://docs.openstack.org/developer/nova/filter_scheduler.html#filtering | 10:56 |
| flwang1 | bauzas: just wondering how to make the puppet code easy | 10:56 |
| bauzas | flwang1: at the end of that section | 10:56 |
| bauzas | flwang1: you don't even need to have the same namespace for your own custom filter | 10:56 |
| flwang1 | oh, really? that's sound super cool | 10:57 |
| sfinucan | danpb: This has already been merged, but I would appreciate an eye over the NUMA/CPU pinning docs. Would be good to know I haven't said anything too outrageous http://docs.openstack.org/admin-guide/compute-numa-cpu-pinning.html | 10:57 |
| *** shaohe_feng has quit IRC | 10:59 | |
| bauzas | flwang1: tl;dr: what you need is to implement a new class inheriting from the BaseHostFilter and either return True or False | 10:59 |
| *** shaohe_feng has joined #openstack-nova | 10:59 | |
| bauzas | flwang1: then, it's just a configuration thing, by adding your class to the scheduler_available_filters opt like scheduler_available_filters += mynamespace.MyFilter | 11:00 |
| flwang1 | bauzas: but the place must be a package which can be accessible in python, right? | 11:00 |
| bauzas | flwang1: and then, use it by adding it to the scheduler_default_filters | 11:00 |
| bauzas | flwang1: of course, yes | 11:00 |
| bauzas | flwang1: you can also put your filters in the same namespace than the others, ie. nova.scheduler.filters | 11:01 |
| sdague | alex_xu: some comments back on the spec to make sure you are good with it | 11:01 |
| sdague | https://review.openstack.org/#/c/312209/3 | 11:01 |
| openstackgerrit | Sean Dague proposed openstack/nova-specs: Deprecate API Proxies https://review.openstack.org/312209 | 11:01 |
| bauzas | flwang1: in that case, you don't need to have your own namespace, nor modifying the scheduler_available_filters, because that module and class will be found by the all_filters module call | 11:01 |
| flwang1 | bauzas: i'm just thinking how to do that with lowest cost in puppet | 11:02 |
| bauzas | flwang1: well, it's just a File variable, right? | 11:02 |
| bauzas | then, you just need to depend your conf modification call from your File var | 11:02 |
| flwang1 | bauzas: so basically, just put the file under nova/scheduler/filter is the easiest way, right? | 11:02 |
| flwang1 | i'm not a puppet guy actually :) | 11:03 |
| bauzas | flwang1: the easiest but most confusing one, yes :) | 11:03 |
| *** jamielennox|away is now known as jamielennox | 11:03 | |
| bauzas | because operators could think your custom filter is in the nova tree | 11:04 |
| flwang1 | bauzas: from that view, yes, i can see your point | 11:04 |
| flwang1 | but i'm also wondering where to put the new filter them | 11:04 |
| bauzas | plus the fact that if we modify the scheduler filters location, then you're screwed | 11:04 |
| flwang1 | since we do need a namespace(python pkg) for it | 11:04 |
| bauzas | well, you just need a tree that can be found from the python path... | 11:05 |
| flwang1 | is there any best practice for this? | 11:06 |
| bauzas | it's a python thing | 11:06 |
| bauzas | not really needing Nova experts :) | 11:06 |
| *** raj_singh_ has quit IRC | 11:06 | |
| *** raj_singh_ has joined #openstack-nova | 11:07 | |
| flwang1 | ok, i see. thanks a lot | 11:07 |
| openstackgerrit | James Page proposed openstack/nova: Fix use of invalid assert calls https://review.openstack.org/318568 | 11:07 |
| *** ptm_away is now known as PaulMurray | 11:08 | |
| *** shaohe_feng has quit IRC | 11:09 | |
| *** shaohe_feng has joined #openstack-nova | 11:10 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform generic instance action notifications https://review.openstack.org/313654 | 11:15 |
| *** palexster has quit IRC | 11:17 | |
| *** palexster has joined #openstack-nova | 11:17 | |
| *** danpb has quit IRC | 11:18 | |
| *** rk4n has quit IRC | 11:18 | |
| *** jinxing has quit IRC | 11:19 | |
| *** shaohe_feng has quit IRC | 11:19 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform generic instance action notifications https://review.openstack.org/313654 | 11:19 |
| *** jinxing has joined #openstack-nova | 11:20 | |
| *** shaohe_feng has joined #openstack-nova | 11:20 | |
| *** rodrigods has quit IRC | 11:20 | |
| *** rodrigods has joined #openstack-nova | 11:20 | |
| *** jinxing has quit IRC | 11:21 | |
| *** sree has quit IRC | 11:23 | |
| Kevin_Zheng | johnthetubaguy: ping | 11:24 |
| *** thorst_ has joined #openstack-nova | 11:27 | |
| *** doug-fish has joined #openstack-nova | 11:28 | |
| *** doug-fish has quit IRC | 11:29 | |
| *** shaohe_feng has quit IRC | 11:29 | |
| *** shaohe_feng has joined #openstack-nova | 11:30 | |
| *** lucasagomes is now known as lucas-hungry | 11:30 | |
| *** fawadkhaliq has quit IRC | 11:30 | |
| *** mtanino has joined #openstack-nova | 11:31 | |
| *** iceyao__ has quit IRC | 11:32 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use tempes.lib instead of tempes_lib https://review.openstack.org/318585 | 11:32 |
| *** iceyao has joined #openstack-nova | 11:34 | |
| *** cdent has quit IRC | 11:38 | |
| *** shaohe_feng has quit IRC | 11:40 | |
| *** danpb has joined #openstack-nova | 11:40 | |
| *** shaohe_feng has joined #openstack-nova | 11:40 | |
| *** hoangcx has quit IRC | 11:41 | |
| *** shaohe_feng has quit IRC | 11:50 | |
| *** shaohe_feng has joined #openstack-nova | 11:51 | |
| openstackgerrit | Atsushi SAKAI proposed openstack/nova: api-ref: Add fault parameter details https://review.openstack.org/317955 | 11:52 |
| lbeliveau | moshele: hi | 11:56 |
| moshele | lbeliveau: hi :) I saw your comment | 11:57 |
| moshele | lbeliveau: I in a meeting will talk to you later | 11:57 |
| lbeliveau | moshele: ok | 11:57 |
| *** lucas-hungry is now known as lucasagomes | 11:57 | |
| lbeliveau | moshele: not sure about why there is a check there to make sure the flavor hasn't changeg | 11:57 |
| moshele | lbeliveau: but I think we hit different bug :) | 11:58 |
| moshele | lbeliveau: it is not always happens at least in my set | 11:58 |
| moshele | lbeliveau: my setup | 11:58 |
| lbeliveau | moshele: ok, let me know | 11:58 |
| *** ducttape_ has joined #openstack-nova | 11:59 | |
| *** sacharya has joined #openstack-nova | 12:00 | |
| *** shaohe_feng has quit IRC | 12:00 | |
| *** shaohe_feng has joined #openstack-nova | 12:01 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: Transform generic instance action notifications https://review.openstack.org/313654 | 12:02 |
| *** rk4n has joined #openstack-nova | 12:05 | |
| *** cdent has joined #openstack-nova | 12:05 | |
| *** sacharya has quit IRC | 12:06 | |
| *** penick has quit IRC | 12:06 | |
| *** jistr has quit IRC | 12:08 | |
| openstackgerrit | jichenjc proposed openstack/nova: verify api-ref metadata.inc https://review.openstack.org/318600 | 12:09 |
| *** shaohe_feng has quit IRC | 12:10 | |
| openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova: Add http_proxy_to_wsgi to api-paste https://review.openstack.org/318603 | 12:12 |
| openstackgerrit | Sean Dague proposed openstack/nova: api-ref: verify flavors.inc https://review.openstack.org/314133 | 12:13 |
| *** shaohe_feng has joined #openstack-nova | 12:13 | |
| *** al_indigo has joined #openstack-nova | 12:14 | |
| *** ljxiash has joined #openstack-nova | 12:15 | |
| *** raildo-afk is now known as raildo | 12:15 | |
| flip214 | I'd like to know how to proceed with https://review.openstack.org/#/c/256292/... spec was originally proposed as https://review.openstack.org/#/c/134153/ | 12:16 |
| flip214 | Thanks for any advice. | 12:16 |
| al_indigo | Hello guys. I have periodic random problems with nova-conductor and I don't know what to do with it. I get several times a day "ERROR oslo_service.periodic_task MessagingTimeout: Timed out waiting for a reply to message ID" errors and they make our cloud think that some compute nodes are lost or makes instances start in ERROR state. | 12:16 |
| *** markvoelker has joined #openstack-nova | 12:16 | |
| *** rk4n has quit IRC | 12:17 | |
| *** rk4n has joined #openstack-nova | 12:18 | |
| *** gszasz has joined #openstack-nova | 12:18 | |
| al_indigo | at the same time there is no any high load and I don't have ideas what causes such behaviour. The nova-conductor is resided at the same node with nova-api and neutron; controller is 32CPU/192GB RAM, network bandwidth is 2x10Gbit/s each to each. Database is postgres and there is no any high load on it. There are only 16 compute nodes in the cluster, nova version is stable/liberty 142323a73e379ab657af4dab6fe8b36c6710f1db | 12:19 |
| openstackgerrit | James Page proposed openstack/nova: Fix use of invalid assert calls https://review.openstack.org/318568 | 12:19 |
| *** jtp92 has joined #openstack-nova | 12:19 | |
| *** ljxiash has quit IRC | 12:19 | |
| jtp92 | Good morning everyone! Does anyone have experience with the VMware driver for Nova? | 12:20 |
| *** eantyshev has quit IRC | 12:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Avoid remote volume detach calls https://review.openstack.org/318266 | 12:20 |
| *** shaohe_feng has quit IRC | 12:21 | |
| al_indigo | config is the following: https://gist.github.com/al-indigo/fad6c84961ab51e00ebe803771c86af7 | 12:21 |
| *** shaohe_feng has joined #openstack-nova | 12:21 | |
| *** markvoelker has quit IRC | 12:21 | |
| al_indigo | Maybe I have something wrong in it? Could anyone look through it | 12:21 |
| al_indigo | *please | 12:21 |
| jtp92 | I have a controller node, a compute node (not sure if necessary), and a cluster of nodes available through vCenter Server. I found some documentation on what to add to the Nova config file, but no detailed write-up on how to actually deploy this type of environment. | 12:22 |
| jtp92 | Currently I configured the Controller as normal, and I also configured the compute node as normal but added the vmware config lines to the nova.conf file. Nova fails to start (can't find detailed reasons why) | 12:23 |
| alex_xu | sdague: i agree with attach vol/interface should be kept. but checking the implementation of addfixedip, it just call the neutron api to update the port to have all the ips from the subnets. if the thing which was done by that API, it is nothing about nova, then it should be pure proxy api. | 12:23 |
| *** syjulian has joined #openstack-nova | 12:23 | |
| *** markvoelker has joined #openstack-nova | 12:24 | |
| sdague | alex_xu: except it also triggers the nw_info_cache update | 12:25 |
| sdague | right? | 12:25 |
| *** apoorv has quit IRC | 12:25 | |
| *** gcb has quit IRC | 12:25 | |
| alex_xu | sdague: yeah, but if user do something at neutron side without proxy api, the nova should update nw_info_cache also | 12:25 |
| sdague | alex_xu: what is the call path to attach a fixed_ip if this is removed | 12:26 |
| *** penick has joined #openstack-nova | 12:27 | |
| *** jistr has joined #openstack-nova | 12:28 | |
| *** ducttape_ has quit IRC | 12:28 | |
| *** edmondsw has joined #openstack-nova | 12:28 | |
| alex_xu | sdague: just call the neutron api directly, and check the subnets of network which port belong to. Then update the port with neutron api. just as the code doing https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L975 | 12:28 |
| *** jistr is now known as jistr|bbl | 12:28 | |
| *** shaohe_feng has quit IRC | 12:31 | |
| *** shaohe_feng has joined #openstack-nova | 12:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use tempes.lib instead of tempes_lib https://review.openstack.org/318585 | 12:32 |
| *** sdague has quit IRC | 12:32 | |
| *** prateek has quit IRC | 12:33 | |
| *** mdrabe has joined #openstack-nova | 12:35 | |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 12:35 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Fix evaluation of host disk usage by volume-backed instances https://review.openstack.org/200870 | 12:35 |
| *** links has quit IRC | 12:36 | |
| *** gcb has joined #openstack-nova | 12:38 | |
| *** _prad_ has joined #openstack-nova | 12:40 | |
| *** kolesovdv has joined #openstack-nova | 12:40 | |
| *** shaohe_feng has quit IRC | 12:41 | |
| *** markvoelker has quit IRC | 12:41 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Save all instance extras in a single db call https://review.openstack.org/305437 | 12:42 |
| *** markvoelker has joined #openstack-nova | 12:42 | |
| *** shaohe_feng has joined #openstack-nova | 12:42 | |
| *** sdague has joined #openstack-nova | 12:43 | |
| *** kylek3h has joined #openstack-nova | 12:44 | |
| *** eharney has joined #openstack-nova | 12:45 | |
| *** efried has joined #openstack-nova | 12:45 | |
| *** markvoelker_ has joined #openstack-nova | 12:46 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make flavor-manage api call destroy with Flavor object https://review.openstack.org/244291 | 12:46 |
| *** prateek has joined #openstack-nova | 12:47 | |
| alex_xu | sdague: emm...looks like the virt drivers other than libvirt needs inject_network/reset_network, but i'm not very sure. At least inject_network/reset_network are admin api on nova side, if user call neutron api directly, and thing go wrong without inject_network/reset_network, that sounds bug also. | 12:48 |
| *** baoli has joined #openstack-nova | 12:48 | |
| *** markvoelker_ has quit IRC | 12:48 | |
| *** baoli_ has joined #openstack-nova | 12:49 | |
| *** penick has quit IRC | 12:49 | |
| *** markvoelker_ has joined #openstack-nova | 12:49 | |
| *** ivase has joined #openstack-nova | 12:50 | |
| *** markvoelker has quit IRC | 12:50 | |
| *** salv-orl_ has quit IRC | 12:50 | |
| *** shaohe_feng has quit IRC | 12:51 | |
| *** shaohe_feng has joined #openstack-nova | 12:52 | |
| *** baoli has quit IRC | 12:52 | |
| *** cleong has joined #openstack-nova | 12:53 | |
| *** markvoelker_ has quit IRC | 12:55 | |
| *** markvoelker has joined #openstack-nova | 12:55 | |
| *** winston-d_ has quit IRC | 12:55 | |
| *** akshai has joined #openstack-nova | 12:56 | |
| sdague | alex_xu: sorry, lost internet | 12:58 |
| sdague | anyway, I think we should leave the actions | 12:58 |
| alex_xu | sdague: np | 12:58 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Limit live migration progress warnings https://review.openstack.org/318644 | 12:58 |
| *** akshai has quit IRC | 12:58 | |
| alex_xu | sdague: ok, i guess there should be another evolution of api related to neutron interactive after nova-net removed...so we can leave that to that point | 12:59 |
| *** palexster has quit IRC | 13:00 | |
| *** AmyLee7 has joined #openstack-nova | 13:00 | |
| *** whenry has joined #openstack-nova | 13:00 | |
| alex_xu | sdague: if we keep it, should we stop the nova-net functional for that api, maybe some problem for securitygroup related api | 13:00 |
| *** roeyc has quit IRC | 13:01 | |
| *** garyk1 has quit IRC | 13:01 | |
| *** palexster has joined #openstack-nova | 13:01 | |
| *** garyk has joined #openstack-nova | 13:02 | |
| *** shaohe_feng has quit IRC | 13:02 | |
| *** roeyc has joined #openstack-nova | 13:02 | |
| *** shaohe_feng has joined #openstack-nova | 13:02 | |
| *** malos has quit IRC | 13:03 | |
| *** akshai has joined #openstack-nova | 13:03 | |
| *** palexster_ has joined #openstack-nova | 13:05 | |
| *** palexster_ has quit IRC | 13:05 | |
| *** ljxiash has joined #openstack-nova | 13:06 | |
| *** sdague has quit IRC | 13:07 | |
| *** palexster has quit IRC | 13:08 | |
| *** palexster has joined #openstack-nova | 13:09 | |
| *** palexster has quit IRC | 13:09 | |
| *** palexster has joined #openstack-nova | 13:09 | |
| *** sdague has joined #openstack-nova | 13:10 | |
| *** palexster has quit IRC | 13:10 | |
| alaski | lifeless: around now, though you're probably not :) I added one small comment on the test you tweaked, but otherwise the changes lgtm | 13:10 |
| *** MikeG451 has joined #openstack-nova | 13:11 | |
| *** sridharg has quit IRC | 13:11 | |
| * cdent writes a mock and shakes his tiny fist | 13:11 | |
| *** markvoelker_ has joined #openstack-nova | 13:12 | |
| *** markvoelker_ has quit IRC | 13:12 | |
| *** takedakn has joined #openstack-nova | 13:12 | |
| *** shaohe_feng has quit IRC | 13:12 | |
| *** rlrossit has joined #openstack-nova | 13:12 | |
| *** dave-mccowan has joined #openstack-nova | 13:12 | |
| *** markvoelker_ has joined #openstack-nova | 13:12 | |
| *** shaohe_feng has joined #openstack-nova | 13:13 | |
| gibi | sdague: hi!, It is just a friendly reminder to look at https://review.openstack.org/#/c/286675/ (versioned notification transformation spec) if you have time | 13:13 |
| realitix | Hello everyone. Do you know why we can't rename a flavor ? Is there a good reason or is it just unimplemented ? | 13:14 |
| *** markvoelker has quit IRC | 13:15 | |
| sdague | alex_xu: honestly, with the other interfaces gone, I think we can be sloppy about preventing it in the nova-net case | 13:16 |
| alex_xu | sdague: ok | 13:17 |
| *** palexster has joined #openstack-nova | 13:17 | |
| alex_xu | sdague: fyi, i found some comments about addfixedip/removefixedip at https://bugs.launchpad.net/nova/+bug/1430512/comments/1 and https://bugs.launchpad.net/nova/+bug/1430512/comments/3 | 13:17 |
| openstack | Launchpad bug 1430512 in OpenStack Compute (nova) "Adding a fixed_ip to an instance with neutron only updates the first port using the first subnet" [Low,In progress] - Assigned to Gary Kotton (garyk) | 13:17 |
| *** diogogmt has quit IRC | 13:17 | |
| *** jinxing has joined #openstack-nova | 13:17 | |
| *** lindycoder has joined #openstack-nova | 13:19 | |
| *** ctrath has joined #openstack-nova | 13:19 | |
| sdague | alex_xu: ok, cool, that's something we'll need to look into | 13:20 |
| openstackgerrit | Lee Yarwood proposed openstack/nova: compute: Avoid remote volume detach calls https://review.openstack.org/318266 | 13:20 |
| *** lindycod_ has joined #openstack-nova | 13:21 | |
| *** shaohe_feng has quit IRC | 13:22 | |
| *** sridharg has joined #openstack-nova | 13:23 | |
| *** lindycoder has quit IRC | 13:24 | |
| *** shaohe_feng has joined #openstack-nova | 13:25 | |
| *** sridharg has quit IRC | 13:25 | |
| *** sridharg has joined #openstack-nova | 13:25 | |
| *** rk4n_ has joined #openstack-nova | 13:25 | |
| *** rk4n has quit IRC | 13:26 | |
| *** irenab has quit IRC | 13:26 | |
| openstackgerrit | Alex Xu proposed openstack/nova: Enable 'null' value for user_data in V2.1 API https://review.openstack.org/318447 | 13:28 |
| openstackgerrit | Hironori Shiina proposed openstack/nova: Ironic: change flat network provider to 'flat' https://review.openstack.org/297895 | 13:29 |
| *** liverpooler has quit IRC | 13:30 | |
| *** AmyLee7 has quit IRC | 13:32 | |
| *** mriedem has joined #openstack-nova | 13:32 | |
| *** shaohe_feng has quit IRC | 13:32 | |
| *** mtanino has quit IRC | 13:33 | |
| *** shaohe_feng has joined #openstack-nova | 13:33 | |
| *** jichen has quit IRC | 13:33 | |
| *** penick has joined #openstack-nova | 13:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/316951 | 13:35 |
| *** AmyLee7 has joined #openstack-nova | 13:36 | |
| *** smatzek has quit IRC | 13:38 | |
| sdague | johnthetubaguy / mriedem you want to take a gander at the proxy deprecation spec again - https://review.openstack.org/#/c/312209/ ? | 13:39 |
| mriedem | sdague: maybe after the meeting | 13:40 |
| *** READ10 has joined #openstack-nova | 13:42 | |
| *** shaohe_feng has quit IRC | 13:43 | |
| *** garyk has quit IRC | 13:44 | |
| sdague | mriedem: cool | 13:44 |
| *** markvoelker_ has quit IRC | 13:44 | |
| sdague | mriedem: so, you head on vacation tomorrow, what's the run down I need for the n1 tagging. Is there anything specific we have to land by then? | 13:44 |
| diana_clarke | mriedem: Morning! Now that stable/liberty is security bugs only, I wondered if you have plans to cut another stable/liberty release soon-ish? | 13:45 |
| diana_clarke | mriedem: Asking b/c I'd like to package the last few bugs that made it in before the cutoff. A stable/mitaka release would be great too ;) | 13:45 |
| mriedem | diana_clarke: i'd have to look at the change log, i've been meaning to do that | 13:45 |
| *** shaohe_feng has joined #openstack-nova | 13:45 | |
| *** mgoddard_ has joined #openstack-nova | 13:45 | |
| mriedem | diana_clarke: we'll probably do stable/mitaka release around n-1 | 13:45 |
| diana_clarke | mriedem: Thanks! | 13:46 |
| mriedem | sdague: there isn't anything specific that i know of that has to be merged by n-1 | 13:46 |
| alex_xu | sdague: johnthetubaguy mriedem, fyi, for api high priority bug, https://review.openstack.org/318447, hope you can take a look at when you have time. | 13:46 |
| *** BobBall_AWOL is now known as BobBall | 13:47 | |
| *** takedakn has quit IRC | 13:47 | |
| *** mrkz has joined #openstack-nova | 13:48 | |
| *** mgoddard has quit IRC | 13:48 | |
| mriedem | alex_xu: thanks, comments inline. i was trying to get a functional regression test working for that bug last night but missed the extra validation stuff you had for that TypeError | 13:49 |
| mriedem | i couldn't figure out why i kept getting a 400 | 13:49 |
| mriedem | sdague: i woke up at 4am thinking we had something when getting an instance that put baremetal node properties in it and we proxied it to ironic, had a hard time getting back to sleep | 13:50 |
| mriedem | was a nightmare | 13:50 |
| *** akshai has quit IRC | 13:52 | |
| *** ametts has joined #openstack-nova | 13:52 | |
| *** penick has quit IRC | 13:52 | |
| *** gagehugo has joined #openstack-nova | 13:53 | |
| sdague | dude, you need this vacation | 13:53 |
| mriedem | i just need to review fewer api-ref patches | 13:53 |
| *** shaohe_feng has quit IRC | 13:53 | |
| sdague | on the upside - https://web.archive.org/web/20051016203842/http://diveintomark.org/archives/2004/08/16/specs is pretty funny, and relevant to spec review | 13:53 |
| sdague | mriedem: well, only 60% left... ;) | 13:53 |
| *** shaohe_feng has joined #openstack-nova | 13:54 | |
| *** belmoreira has quit IRC | 13:55 | |
| *** belmoreira has joined #openstack-nova | 13:55 | |
| mriedem | +2 on the api deprecation spec | 13:57 |
| mriedem | looks like i need to go back over the glance v2 spec after the meeting then https://review.openstack.org/#/c/301741/ | 13:58 |
| mriedem | was +2 on it before bed | 13:58 |
| mdbooth | diana_clarke: It's just Lvm and Flat which don't use the image cache for generated disks, btw. | 13:58 |
| mdbooth | The other 3 do. | 13:58 |
| mriedem | sdague: does this need to be re-kicked? https://review.openstack.org/#/c/316298/ | 13:59 |
| mriedem | dependencies are all merged | 13:59 |
| diana_clarke | mdbooth: thanks for checking | 13:59 |
| sdague | mriedem: looking | 13:59 |
| mriedem | nova meeting in #openstack-meeting | 14:00 |
| PaulMurray | mriedem, sdague I was looking at that just now - I think so | 14:00 |
| sdague | mriedem: yeh, project-config with depends on needs a A state flip to trigger | 14:00 |
| sdague | because ... details | 14:00 |
| *** jistr|bbl is now known as jistr | 14:00 | |
| *** akshai has joined #openstack-nova | 14:00 | |
| *** smatzek has joined #openstack-nova | 14:01 | |
| *** woodster_ has joined #openstack-nova | 14:02 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Use tempes.lib instead of tempes_lib https://review.openstack.org/318585 | 14:03 |
| *** shaohe_feng has quit IRC | 14:03 | |
| *** tbachman has quit IRC | 14:04 | |
| *** shaohe_feng has joined #openstack-nova | 14:04 | |
| *** ducttape_ has joined #openstack-nova | 14:05 | |
| *** ducttape_ has quit IRC | 14:07 | |
| *** salv-orlando has joined #openstack-nova | 14:07 | |
| *** ducttape_ has joined #openstack-nova | 14:07 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Make flavor-manage api call destroy with Flavor object https://review.openstack.org/244291 | 14:07 |
| *** ducttape_ has quit IRC | 14:08 | |
| *** ducttape_ has joined #openstack-nova | 14:08 | |
| *** raj_singh_ has quit IRC | 14:08 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Replace flavors.get_all_flavors_sorted_list() with object call https://review.openstack.org/268801 | 14:10 |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Refactor and objectify flavor fakes used in api tests https://review.openstack.org/268800 | 14:10 |
| *** mgoddard_ has quit IRC | 14:11 | |
| *** mgoddard has joined #openstack-nova | 14:11 | |
| *** salv-orlando has quit IRC | 14:11 | |
| *** shaohe_feng has quit IRC | 14:13 | |
| *** shaohe_feng has joined #openstack-nova | 14:14 | |
| *** marksei has joined #openstack-nova | 14:17 | |
| markus_z | sdague: Should the functional tests in Nova hit through to the underlying libvirt/kvm? Is there a general rule about the borders of these tests? | 14:18 |
| *** cfriesen has joined #openstack-nova | 14:18 | |
| mriedem | markus_z: the functional tests don't hit a real libvirt | 14:18 |
| markus_z | sdague: The background is, that we executed them on s390x and some failed | 14:18 |
| sdague | markus_z: they don't touch real libvirt | 14:19 |
| *** marksei has quit IRC | 14:20 | |
| *** vladikr has quit IRC | 14:20 | |
| *** marksei has joined #openstack-nova | 14:20 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Ploop: add create_from_image & create_from_func https://review.openstack.org/317689 | 14:20 |
| markus_z | mriedem: sdague: OK, then I assume that a mock of the arch is missing. Some code of s390x was executed. | 14:20 |
| *** marksei has quit IRC | 14:20 | |
| *** unicell1 has joined #openstack-nova | 14:20 | |
| *** unicell has quit IRC | 14:21 | |
| *** wangfeng has joined #openstack-nova | 14:21 | |
| *** xenogear has joined #openstack-nova | 14:21 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Rbd: add create_from_image & create_from_func https://review.openstack.org/317694 | 14:21 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Flat: add create_from_image & create_from_func https://review.openstack.org/317701 | 14:22 |
| markus_z | I dig deeper into it. The unit test "LibvirtConnTestCase.test_init_host" also executed s390x code paths. | 14:22 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Move is_volume_backed_instance to compute.utils https://review.openstack.org/270482 | 14:23 |
| openstackgerrit | Ankit Agrawal proposed openstack/nova: Fix evaluation of host disk usage by volume-backed instances https://review.openstack.org/200870 | 14:23 |
| *** shaohe_feng has quit IRC | 14:24 | |
| *** AmyLee7 has quit IRC | 14:24 | |
| *** syjulian has quit IRC | 14:24 | |
| *** shaohe_feng has joined #openstack-nova | 14:25 | |
| *** jinxing has quit IRC | 14:25 | |
| mriedem | moshele: you're not in the nova meeting | 14:26 |
| moshele | mriedem: is it now? | 14:26 |
| *** edand has joined #openstack-nova | 14:27 | |
| mriedem | yup | 14:27 |
| *** belmoreira has quit IRC | 14:27 | |
| alaski | sdague: the oslo.policy series is at https://review.openstack.org/#/c/313141/, seems to be in shape and is just waiting on more reviewers | 14:27 |
| *** sajeesh has joined #openstack-nova | 14:28 | |
| *** realitix has quit IRC | 14:28 | |
| *** ctrath has quit IRC | 14:28 | |
| *** jinxing has joined #openstack-nova | 14:29 | |
| *** fawadkhaliq has joined #openstack-nova | 14:29 | |
| *** edand_ has joined #openstack-nova | 14:30 | |
| *** mtanino has joined #openstack-nova | 14:31 | |
| *** edand has quit IRC | 14:31 | |
| openstackgerrit | Bob Ball proposed openstack/nova-specs: XenAPI: Support Nova services independently from the hypervisor https://review.openstack.org/277452 | 14:31 |
| *** diogogmt has joined #openstack-nova | 14:33 | |
| *** fawadkhaliq has quit IRC | 14:33 | |
| *** iceyao has joined #openstack-nova | 14:33 | |
| *** pushkaru has joined #openstack-nova | 14:34 | |
| *** shaohe_feng has quit IRC | 14:34 | |
| *** shaohe_feng has joined #openstack-nova | 14:34 | |
| *** penick has joined #openstack-nova | 14:37 | |
| *** raunak has joined #openstack-nova | 14:37 | |
| *** belmoreira has joined #openstack-nova | 14:38 | |
| *** raunak has quit IRC | 14:39 | |
| *** prateek has quit IRC | 14:39 | |
| *** penick has quit IRC | 14:39 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Implement nova rescue https://review.openstack.org/127159 | 14:40 |
| *** AmyLee7 has joined #openstack-nova | 14:40 | |
| *** AmyLee7 has quit IRC | 14:40 | |
| *** penick has joined #openstack-nova | 14:41 | |
| *** irenab has joined #openstack-nova | 14:41 | |
| *** yamahata has joined #openstack-nova | 14:43 | |
| *** ivase has quit IRC | 14:43 | |
| *** roeyc has quit IRC | 14:43 | |
| *** sdake_ has joined #openstack-nova | 14:44 | |
| *** shaohe_feng has quit IRC | 14:44 | |
| *** shaohe_feng has joined #openstack-nova | 14:45 | |
| *** unicell1 has quit IRC | 14:46 | |
| *** sridharg has quit IRC | 14:46 | |
| cdent | dansmith: I need your help with something when you have a moment. I'm trying to figure out how to deal with lazy relationships as in: https://review.openstack.org/#/c/275432/ | 14:46 |
| moshele | mriedem: hi | 14:46 |
| dansmith | cdent: okay, let me get this change pushed up first | 14:47 |
| cdent | dansmith: thanks | 14:47 |
| mriedem | moshele: hey | 14:47 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Ploop: add create_from_image & create_from_func https://review.openstack.org/317689 | 14:47 |
| moshele | mriedem: I found out the the resize is race it you do it fast enough it doesn't release the old resource | 14:47 |
| flip214 | mriedem: ping, about https://blueprints.launchpad.net/cinder/+spec/drbd-transport | 14:48 |
| moshele | mriedem: by fast I mean small image and resize-confirm it very fast | 14:48 |
| *** jlanoux has quit IRC | 14:48 | |
| *** ivase has joined #openstack-nova | 14:49 | |
| mriedem | moshele: this is resize in general you said? | 14:49 |
| moshele | mriedem: yes | 14:49 |
| *** salv-orlando has joined #openstack-nova | 14:49 | |
| *** tonytan4ever has joined #openstack-nova | 14:49 | |
| mriedem | and doesn't release which old resource? | 14:49 |
| mriedem | pci? | 14:49 |
| mriedem | or doesn't cleanup from the old host? | 14:49 |
| *** whenry has quit IRC | 14:50 | |
| mriedem | flip214: looks like the drbd connector went into os-brick version 0.7.0 | 14:51 |
| mriedem | flip214: given we require >=1.3.0 http://git.openstack.org/cgit/openstack/requirements/tree/global-requirements.txt#n118 we should be ok there | 14:51 |
| moshele | mriedem: the following section will not execute https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L315-L326 | 14:51 |
| *** penick has quit IRC | 14:51 | |
| flip214 | mriedem: yes, thanks. | 14:52 |
| *** dedery has quit IRC | 14:52 | |
| *** rk4n_ has quit IRC | 14:52 | |
| mriedem | flip214: want to create a nova bluerpint for the nova changes? https://blueprints.launchpad.net/nova/+spec/drbd-transport | 14:52 |
| flip214 | but the nova changes from https://review.openstack.org/#/c/256292/ were rejected.. | 14:52 |
| moshele | mriedem: so it won't remove the usage of the old guest | 14:52 |
| *** ivase has quit IRC | 14:53 | |
| *** fesp has joined #openstack-nova | 14:53 | |
| flip214 | mriedem: I had one: https://blueprints.launchpad.net/nova/+spec/drbd-driver but it never got looked at, AFAICT | 14:54 |
| *** jdurgin1 has joined #openstack-nova | 14:54 | |
| mriedem | flip214: ok, please update that with the details and link to the nova path | 14:54 |
| mriedem | *patch | 14:54 |
| flip214 | the spec that john asked about was https://review.openstack.org/#/c/134153/14 | 14:54 |
| mriedem | flip214: also, i left comments in your nova change | 14:54 |
| flip214 | thanks | 14:54 |
| mriedem | flip214: we don't need the spec | 14:54 |
| *** shaohe_feng has quit IRC | 14:54 | |
| cdent | dansmith: gonna switch locations, back in 15 min, thanks | 14:55 |
| *** cdent has quit IRC | 14:55 | |
| mriedem | flip214: the biggest issue you're going to have is that you can't add a new compute rootwrap filter to nova for this, you need to use oslo.privsep | 14:55 |
| *** edand_ has quit IRC | 14:55 | |
| *** shaohe_feng has joined #openstack-nova | 14:55 | |
| mriedem | i left a comment in the nova change about this | 14:55 |
| mriedem | os-brick is using privsep now | 14:55 |
| *** unicell has joined #openstack-nova | 14:55 | |
| mriedem | and at the summit we said nova wouldn't take any new compute rootwrap filters, anything new needs to be using privsep | 14:55 |
| *** ivase_ has joined #openstack-nova | 14:56 | |
| mdbooth | dansmith: So, interestingly I completely misread the locking code in imagebackend. It actually never locks an instance disk at all, only the image cache file. | 14:56 |
| danpb | mriedem: os-vif is using privsetp too ! | 14:56 |
| dansmith | mdbooth: okay | 14:56 |
| *** whenry has joined #openstack-nova | 14:56 | |
| mdbooth | dansmith: I'm still processing that :) | 14:56 |
| mriedem | danpb: yeah | 14:57 |
| dansmith | mdbooth: the terminology in there is definitely confusing as it's hard to tell whether we're talking about images, disks, etc at any point in there | 14:57 |
| mdbooth | So, compatibility-wise, I need to ensure that the locking in ImageCacheLocalDir is compatible with the locking in the various backends. | 14:57 |
| flip214 | mriedem: https://blueprints.launchpad.net/nova/+spec/drbd-driver updated. | 14:57 |
| dansmith | you know this of course | 14:57 |
| mriedem | flip214: https://etherpad.openstack.org/p/newton-privsep was from the austin summit, has some details on privsep | 14:57 |
| mdbooth | dansmith: Right. It also creates the same lock in 2 different ways. | 14:57 |
| flip214 | and yes, I'll switch to privsep, now that this has stabilized... is wasn't when I submitted this | 14:57 |
| flip214 | thanks for the pointer | 14:58 |
| mdbooth | I'd misread that to be disk and image cache. | 14:58 |
| sdague | alaski: who on the oslo side is needed to review this? | 14:58 |
| mriedem | danpb: if kashyap has already mentioned it, the live migration job on xenial nodes is failing to use the gate64 cpu model that gets setup by devstack | 14:58 |
| mdbooth | dansmith: I'm still trying to decide whether we *should* be locking the instance disk. | 14:58 |
| kashyap | mriedem: I didn't. Today by EOD I wanted to write current status to -devel list | 14:58 |
| mriedem | danpb: would be good to get that sorted out so we can use the live migration job again. we moved it to xenial nodes to use newer libvirt/qemu. | 14:59 |
| mdbooth | i.e. what concurrent operations might reasonably happen to it? | 14:59 |
| mriedem | kashyap: ok, thanks | 14:59 |
| mriedem | moshele: i have to run to a meeting, will read traceback when i get back | 14:59 |
| mdbooth | Well, reasonably or unreasonably, tbh. | 14:59 |
| *** mriedem is now known as mriedem_meeting | 14:59 | |
| kashyap | mriedem: Today was a yak-shave day upgrading systems to F24, and trying to fix some build deps for various components | 14:59 |
| *** tyrola has joined #openstack-nova | 14:59 | |
| *** jaosorior has quit IRC | 14:59 | |
| alaski | sdague: I don't actually know, because the oslo review team is one large group I don't know who looks at policy changes | 14:59 |
| sdague | https://review.openstack.org/#/admin/groups/556,members | 15:00 |
| moshele | mriedem: I will send a mail about in the ML | 15:00 |
| sdague | alaski: it's actually a lot of keystone core | 15:00 |
| danpb | mriedem_meeting: no, first i'm hearing of it | 15:00 |
| *** danpb has left #openstack-nova | 15:00 | |
| *** jaosorior has joined #openstack-nova | 15:00 | |
| *** danpb has joined #openstack-nova | 15:00 | |
| alaski | sdague: ahh, I didn't realize they split out like that as well | 15:00 |
| *** mgoddard_ has joined #openstack-nova | 15:00 | |
| kashyap | danpb: I'll write to upstream -devel list with current notes. | 15:00 |
| sdague | it looks like keystone core + oslo core is the review team | 15:00 |
| danpb | kashyap: what's the deal with cpu model ? | 15:00 |
| kashyap | danpb: Despite having 'gate64' CPU model in cpu_map.xml | 15:01 |
| openstackgerrit | Pavel Kholkin proposed openstack/nova: [WIP]: nova-api-hypervisor-cpu-info change https://review.openstack.org/304180 | 15:01 |
| kashyap | danpb: libvirt fails to detect it: | 15:01 |
| *** ctrath has joined #openstack-nova | 15:01 | |
| flip214 | mriedem_meeting: JFI, we edited https://blueprints.launchpad.net/nova/+spec/drbd-driver in parallel, my changes are lost ;[ | 15:01 |
| kashyap | [...] | 15:02 |
| kashyap | 2016-05-13 21:33:08.194 15003 ERROR nova.compute.manager [instance: 9bc566d2-c856-4f36-bc7c-48d82870aa30] libvirtError: internal error: early end of file from monitor, possible problem: 2016-05-13T21:33:07.143245Z qemu-system-x86_64: Unable to find CPU definition: gate64 | 15:02 |
| kashyap | 2016-05-13 21:33:43.625+0000: 3010: warning : x86Decode:1583 : Preferred CPU model gate64 not allowed by hypervisor; closest supported model will be used | 15:02 |
| kashyap | danpb: And the associated libvirt debug log (upcoming) doesn't record anything from the CPU driver. | 15:02 |
| kashyap | libvirt debug log: logs.openstack.org/03/315703/1/experimental/gate-tempest-dsvm-multinode-live-migration/1288a98/logs/libvirt/libvirtd.txt.gz | 15:03 |
| danpb | where does this 'gate64' cpu model come from ? | 15:03 |
| *** jaosorior has quit IRC | 15:03 | |
| kashyap | danpb: From here: https://review.openstack.org/#/c/168407/9/tools/cpu_map_update.py | 15:03 |
| *** dstepanenko__ has quit IRC | 15:03 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Correct the network config option help text https://review.openstack.org/318726 | 15:03 |
| edleafe | dansmith: ^^ | 15:03 |
| dansmith | edleafe: thanks | 15:03 |
| *** syjulian has joined #openstack-nova | 15:03 | |
| kashyap | danpb: It was added (by Infra folks) a year ago to make live migration work | 15:03 |
| *** jaosorior has joined #openstack-nova | 15:03 | |
| danpb | kashyap: soo must be a case that the xenial nodes are not exposing one of the cpu flags list there | 15:04 |
| *** mgoddard has quit IRC | 15:04 | |
| kashyap | danpb: Yeah, yesterday I went looking into Xenial's libvirt source to see if they're carrying any non-upstream patches: | 15:04 |
| kashyap | These are their patches list http://paste.openstack.org/show/497514/ | 15:05 |
| *** shaohe_feng has quit IRC | 15:05 | |
| kashyap | And I don't see anything CPU models related stuff. (That "dont-include-non-migrateable-features-in-host-model" patch is already upstream.) | 15:05 |
| danpb | well patches aren't neccessarily the problem - you've no way of knowing what cpu model they've configured their guest to expose | 15:05 |
| *** shaohe_feng has joined #openstack-nova | 15:05 | |
| gibi | rlrossit: regarding the ovo registration in https://review.openstack.org/#/c/309454/ | 15:05 |
| danpb | you'll need to look at the virsh capabilities output to see what nova is seeing | 15:06 |
| danpb | and identify the missing cpu flag(s) | 15:06 |
| kashyap | danpb: So, should we request -infra folks the missing CPU flags? | 15:06 |
| kashyap | s/the/to add the/ | 15:06 |
| gibi | rlrossit: my test result disagres with your comment that ovo base classes without register_if(False) doesn't work | 15:06 |
| danpb | no the CI needs to be configured to cope with whatever the nodes expose | 15:06 |
| *** jamesdenton has joined #openstack-nova | 15:07 | |
| gibi | rlrossit: can it be that it is enough to call register_if only on the concrete classes and that will take care of the initialization of the base classes too? | 15:07 |
| kashyap | danpb: Okay, I'll make it a point to check what `virsh capabilities` expose on the 'Xenial' release. | 15:08 |
| kashyap | Thanks! | 15:08 |
| dansmith | mriedem_meeting: when I add an if clause for an index because of db2, does that mean I have to leave it out of the model? otherwise the test complains about them not being in sync | 15:09 |
| *** jaosorior has quit IRC | 15:10 | |
| *** doug-fish has joined #openstack-nova | 15:10 | |
| *** claudiub has quit IRC | 15:10 | |
| *** adisky has quit IRC | 15:10 | |
| *** wangfeng has quit IRC | 15:10 | |
| *** links has joined #openstack-nova | 15:11 | |
| *** zz_dimtruck is now known as dimtruck | 15:12 | |
| *** doug-fish has quit IRC | 15:12 | |
| sdague | alaski: can you poke into #openstack-dev, talking with ayoung on oslo.policy reviews there | 15:12 |
| ayoung | sdague, thanks | 15:12 |
| openstackgerrit | Daniel Berrange proposed openstack/nova: os-vif: convert libvirt driver to use os-vif library https://review.openstack.org/269672 | 15:13 |
| openstackgerrit | alex proposed openstack/nova: Remove destroy instance which states is "accepted" after evacuate. https://review.openstack.org/318731 | 15:13 |
| *** heyongli has quit IRC | 15:13 | |
| *** alex_xu has quit IRC | 15:13 | |
| *** cdent has joined #openstack-nova | 15:15 | |
| *** shaohe_feng has quit IRC | 15:15 | |
| siva_krishnan | danpb I have few questions regarding libvirt. Can you please help me with that ? | 15:18 |
| *** doug-fish has joined #openstack-nova | 15:20 | |
| *** jlanoux has joined #openstack-nova | 15:20 | |
| *** yamahata has quit IRC | 15:20 | |
| *** DaiDV has joined #openstack-nova | 15:21 | |
| danpb | siva_krishnan: possibly what's the q | 15:21 |
| *** ijw has joined #openstack-nova | 15:22 | |
| siva_krishnan | danpb can you share me your mail id, so that I can forward the mail that I sent to openstack-dev mailing list | 15:23 |
| danpb | i'm on openstack-dev already | 15:23 |
| *** fawadkhaliq has joined #openstack-nova | 15:24 | |
| siva_krishnan | danpb can you pull the mail with following line as subject [openstack-dev][nova]Libvirt error code for failing volume in nova ? | 15:25 |
| *** whenry has quit IRC | 15:25 | |
| dansmith | mriedem_meeting: nevermind, I'm stupid | 15:25 |
| *** whenry has joined #openstack-nova | 15:25 | |
| dansmith | cdent: okay shoot | 15:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 15:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute nodes update their own inventory records https://review.openstack.org/317049 | 15:26 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 15:26 |
| *** links has quit IRC | 15:26 | |
| *** akshai has quit IRC | 15:27 | |
| cdent | dansmith: in https://review.openstack.org/#/c/275432/ the Inventory table as an orm.relationship for a resource_provider | 15:27 |
| *** kolesovdv has quit IRC | 15:27 | |
| cdent | the way it is currently written that will be lazily loaded when asked | 15:27 |
| cdent | it is easy, however, for that lazy loading to happen after the session has gone out of scope | 15:27 |
| cdent | throwing an error | 15:28 |
| dansmith | cdent: yeah, you should either load it explicitly on the query, or load it directly (not via sqla lazy load) later | 15:28 |
| *** belmoreira has quit IRC | 15:28 | |
| cdent | which is the fix for that: a) make it not be lazy b) engage the session when ever we want to pull on that lazy loaded attribute c) something else | 15:28 |
| *** claudiub has joined #openstack-nova | 15:28 | |
| *** andreas_s has quit IRC | 15:29 | |
| dansmith | cdent: your access to the model should be purely within the scope of an object operation, so you should know whether you're going to lazy-load it or not, right? | 15:29 |
| cdent | where this seems to be coming up with when use the results of getting an InventoryList | 15:29 |
| *** hemnafk is now known as hemna | 15:30 | |
| cdent | dansmith: I'm somewhat lost | 15:31 |
| dansmith | cdent: all who wander are not lost | 15:32 |
| dansmith | no, screwed that up | 15:32 |
| dansmith | not all who wander.. meh whatever :) | 15:32 |
| artom_ | dansmith, question about storing metadata in the database | 15:32 |
| cdent | this has come up because I was moving around some tests for Allocation and AllocationList | 15:32 |
| *** artom_ is now known as artom | 15:32 | |
| *** AmyLee7 has joined #openstack-nova | 15:32 | |
| dansmith | cdent: can you point me at a paste or something? | 15:32 |
| dansmith | artom: okay | 15:33 |
| cdent | yeah, let me see if I can construct something | 15:33 |
| *** dmk0202 has quit IRC | 15:33 | |
| artom | dansmith, if we put the JSON blob in instance_extra, this means the API will pull something from the DB that's been put there by the virt driver. I recall you not being keen on the idea of the API barfing up JSON directly from the virt driver | 15:33 |
| *** claudiub has quit IRC | 15:33 | |
| *** sudipto has quit IRC | 15:33 | |
| siva_krishnan | danpb were you able to get hold off my mail ? | 15:33 |
| dansmith | artom: if the virt driver is putting an abstract model that the api barfs up that's cool | 15:33 |
| *** akshai has joined #openstack-nova | 15:34 | |
| artom | dansmith, but can an abstract model be stored in a column of instance_extra? | 15:34 |
| artom | Wouldn't it need its own table? | 15:34 |
| dansmith | artom: what I mean by abstract model is something not driver-specific | 15:34 |
| dansmith | artom: of course, all of instance_extra is that :) | 15:34 |
| *** ljxiash has quit IRC | 15:35 | |
| artom | dansmith, right, but it's still a JSON blob that's served up to the user (with potentially some schema validation) | 15:35 |
| artom | dansmith, and thus something that the virt layer has generated becomes part of the API | 15:35 |
| dansmith | artom: no, when I say store a json blob I mean a serialized obejct | 15:35 |
| dansmith | artom: the object provides the generic model which would be used by all the virt drivers | 15:36 |
| dansmith | artom: that evokes another question.. surely we need to implement this interface for the other drivers so that they're not exposing "no devices" to the guest right? | 15:36 |
| *** kfarr has joined #openstack-nova | 15:37 | |
| artom | dansmith, aha, I see - so it would go virt layer -> metadata objects -> serialize -> DB -> unserialize -> metadata API -> construct user-facing jSON -> chuck it at user | 15:37 |
| dansmith | artom: i.e. it's not okay to provide a tag for a nic and have vmware or xenapi guests say "you have no tagged devices" to the guest, IMHO | 15:37 |
| artom | dansmith, yeah, we need a way of indicating that it's not implemented | 15:37 |
| dansmith | artom: yes | 15:37 |
| artom | dansmith, for that wouldn't it be better to make the device tags its own resource? We would then be able to use return codes | 15:38 |
| dansmith | artom: I don't think you get to make it not implemented by some drivers.. it's metadata and generic, so it should be implemented by them as well | 15:38 |
| artom | 500 not implemented, 404 if there are no tagged devices for example | 15:38 |
| artom | dansmith, well, should, yes, but until it is... | 15:38 |
| dansmith | does any of the rest of the metadata api behave that way? | 15:38 |
| *** vladikr has joined #openstack-nova | 15:38 | |
| artom | dansmith, utterly not :/ | 15:39 |
| dansmith | artom: well, the other option is to not expose it in the metadata api until you have it implemented everywhere | 15:39 |
| artom | dansmith, that could take while... where 'a while' may end up being infinity ;) | 15:40 |
| *** Swami has joined #openstack-nova | 15:40 | |
| danpb | i think blocking implementation in any driver until all implement it is rather overly extreme | 15:40 |
| dansmith | right but if you don't do it as part of this, it'll never happen and we have guest-facing api that differs (a lot) between drivers | 15:40 |
| danpb | if the virt driver could report whether they support this or not | 15:40 |
| dansmith | danpb: if this wasn't guest-facing api I'd agree | 15:40 |
| danpb | then we can reject it if the user specifies it when booting a guest | 15:40 |
| danpb | eg, nova/virt/driver.py could have a capability flag to indicate if it supports device tagging | 15:41 |
| dansmith | danpb: and for multi-driver clouds? | 15:41 |
| danpb | that way when we get a public API request to boot a guest, or hotplug a disk / nic we can report whether its supportable or not | 15:41 |
| dansmith | I think that's problematic because of the guest-facing aspect of this | 15:41 |
| *** mjura has quit IRC | 15:42 | |
| dansmith | I don't think it's okay to not have xenapi and vmware and hyperv implement this and thus have a hole in metadata | 15:42 |
| danpb | i don't see that as being a real problem | 15:42 |
| *** fungi has joined #openstack-nova | 15:42 | |
| *** clarkb has joined #openstack-nova | 15:42 | |
| clarkb | kashyap: hello | 15:42 |
| kashyap | clarkb: Hi | 15:42 |
| danpb | i mean it would be no different to the same guest image being booted on Newton vs Mitaka | 15:43 |
| kashyap | danpb: I see you're in a different conversatoin here, when you get a sec: clarkb from -infra tells us these are the CPU flags from `Xenial` -- http://paste.openstack.org/show/497747/ | 15:43 |
| dansmith | danpb: but once Ocata comes along, and you've got clearly-ocata things in metadata, but are missing things that were added in newton, then it becomes different | 15:43 |
| danpb | if the guest image needs this metadata to be provided, then perhaps we should tag the guest image itself as requiring it | 15:43 |
| danpb | eg img_dev_metadata_version=1.0 | 15:44 |
| dansmith | mordred: I bet you have a reality-based opinion here | 15:44 |
| danpb | to indicate that this image is only bootable if it can be given device metdata version 1.0 | 15:44 |
| mordred | uhoh | 15:44 |
| dansmith | mordred: let me catch you up: | 15:44 |
| danpb | if the hypervisor couldn't honour that then it can raise an error | 15:44 |
| kashyap | clarkb: On -infra, which file location has changed you say? | 15:44 |
| kashyap | (Or likely have changed.) | 15:45 |
| dansmith | mordred: we are working on a metadata chunk that will let you tag devices like nics and disks when you create them, and then the guest will be able to see what devices the hypevisor hooked them up to, by tag, in metadata | 15:45 |
| clarkb | kashyap: I dont know that it has changed, I just think that is more likely. The one which libvirt reads the cpu model defs from | 15:45 |
| dansmith | mordred: so you can see that the second nic is the one you tagged as "private" | 15:45 |
| dansmith | mordred: or that sdb is "fordatabasestuff" | 15:45 |
| kashyap | Yeah, I corrected myself with the hedge 'likely'. | 15:45 |
| dansmith | mordred: so I think it's very important that we keep that view consistent to the guests, and not have it be missing because nobody wanted to go implement this for, say, the xenapi driver | 15:46 |
| mordred | dansmith: yes please. this sounds fantastic | 15:46 |
| mordred | dansmith: agree | 15:46 |
| dansmith | mordred: I feel like our metadata api is our guest-facing ABI and should be _extremely_ stable and consistent | 15:46 |
| clarkb | kashyap: another possibility is that we are getting lobvirt to reread the config properly. On fedora service reload was sufficient, on truaty we had to stop start, maybe now we need something different | 15:46 |
| mordred | dansmith: I should not know you're running xen ever | 15:46 |
| mordred | dansmith: +10000 | 15:46 |
| kashyap | clarkb: It's cpu_maps.xml. You can see 'gate64' here: http://logs.openstack.org/03/315703/1/experimental/gate-tempest-dsvm-multinode-live-migration/1288a98/logs/libvirt/cpu_map.xml | 15:46 |
| danpb | dansmith: we've definitely making them consistent as we've defined a clear general object model for the format | 15:46 |
| danpb | i don't see why we need a big bang to implement it in every single driver at the same time | 15:46 |
| mordred | well.... | 15:47 |
| dansmith | danpb: right but just not having that present because we're on xen and nobody wanted to write that code is not okay to me.. either we support that driver or we don't, and this is not a hypervisor-doesn't-support-this-thing kind of deal | 15:47 |
| mordred | so as an end user - I'm still going to have to deal with clouds that have it and clouds that don't | 15:47 |
| dansmith | danpb: it won't happen if we land the metadata api and move on | 15:47 |
| dansmith | mordred: right, but after some point not having it but having newer things, because the provider is running xen is inconsistent | 15:48 |
| kashyap | clarkb: Let me write a quick email to the list, so people can chime in there -- it's a bit more cleaner | 15:48 |
| *** klkumar_ has quit IRC | 15:48 | |
| mordred | which is probably handled by the metadata api date version things already? | 15:48 |
| clarkb | kashyap: ok... I dont think we need any of this | 15:48 |
| clarkb | someone just needs to boot a xenial host, fix it, then patch devstack | 15:48 |
| dansmith | mordred: we don't bump that date for our own non-EC2 additions to that, AFAIK :( | 15:48 |
| mordred | oh | 15:48 |
| mordred | well nevermind then | 15:48 |
| clarkb | Ideally someone involved with live migrate as this is the only reason we have the workaround | 15:48 |
| dansmith | mordred: so all you have to go on is "if this thing is there, then I assume you're newer" and thus making that inconsistent across drivers is... uncool | 15:49 |
| danpb | the device role tag metadata document is itself versioned independantly of the metadata api though | 15:49 |
| kashyap | clarkb: I can try it on a 'Xenial' VM. | 15:49 |
| danpb | as it has an associated oslo.vo version | 15:49 |
| mordred | is the new thing a whole new document? | 15:49 |
| dansmith | danpb: viewable from the metadata api? and what version would you show for "not present" ? | 15:49 |
| mordred | like network_info.json ? | 15:49 |
| dansmith | mordred: does it matter if it's a separate document? | 15:50 |
| dansmith | mordred: meaning, aren't people going to look for something super new and assume that things older than that are present? | 15:50 |
| sdague | dansmith: isn't there already a mechanism for that in metadata service, because you get a new directory per version | 15:50 |
| cdent | dansmith: I seem to have more variables present than I thought, and I've got a meeting in a few minutes, so I'll cry in your general direction later if I must. | 15:51 |
| *** sacharya has joined #openstack-nova | 15:51 | |
| *** whenry has quit IRC | 15:51 | |
| dansmith | sdague: mechanism for what? versioning? | 15:51 |
| sdague | yeh | 15:51 |
| danpb | dansmith: why does a version matter if it isn't present | 15:51 |
| artom | dansmith, well, if it's a separate document return codes could be used to indicate status | 15:51 |
| dansmith | sdague: I think we don't do that for our own stuff do we? I've never known us to change it | 15:51 |
| danpb | its no different to just running on any existing nova version if the driver doesn't export it | 15:51 |
| mordred | danpb: nah - I think I'd look for the thing and only assume from having found it that that thing is present | 15:51 |
| mordred | gah | 15:51 |
| mordred | dansmith: ^^ | 15:51 |
| *** ctrath has quit IRC | 15:51 | |
| dansmith | danpb: I'm not saying it does | 15:52 |
| mordred | dansmith: like, I would not assume network_info.json is going to exist if I find the new devive mapping info in metadata.json | 15:52 |
| mordred | I'll be testing each thing independently | 15:52 |
| dansmith | mordred: okay | 15:52 |
| *** jaypipes has joined #openstack-nova | 15:52 | |
| danpb | BTW, I really *do* totally want the other virt drivers to implement this | 15:52 |
| dansmith | mordred: so because this hasn't existed since the beginning, it's okay to never add support for the other hypervisors for this? since you'll always have to deal with it being missing? | 15:52 |
| *** ctrath has joined #openstack-nova | 15:53 | |
| mordred | ya | 15:53 |
| dansmith | danpb: I know, but it won't happen if we don't arrange for it | 15:53 |
| mordred | since I have to deal with that anyway | 15:53 |
| mordred | becuase of old clouds | 15:53 |
| dansmith | mordred: well, hang on | 15:53 |
| danpb | i just think that if we block on having that, then we're going to end up with zero virt drivers having it for several more releases | 15:53 |
| mordred | dansmith: I think it's a bad idea to not implement it everywhere | 15:53 |
| dansmith | mordred: you have asked for it via the api and we said okay, but we put you on a node that won't ever generate it | 15:53 |
| mordred | oh... so this is now interesting | 15:53 |
| mordred | I may change my mind here | 15:53 |
| dansmith | danpb: yeah, I know, but we do things that take time to be consistent | 15:54 |
| artom | dansmith, well, hence danpb's idea of not booting it on a node with a virt driver that doesn't support it | 15:54 |
| mordred | this is a new user-face rest api thing I just opted in to isn't it? | 15:54 |
| dansmith | artom: yeah, adding yet another thing you have to specify manually to get correct behavior | 15:54 |
| mordred | dansmith: in that case, then no, I do not think it's ok for some drivers to not implement it - that woudl be massively suck from an end user pov | 15:54 |
| *** zenoway has quit IRC | 15:54 | |
| danpb | dansmith: yeah, that's why i think we could do with an image property to indicate that this disk image needs this feature | 15:54 |
| sdague | dansmith: I guess you are right, we only add things - https://github.com/openstack/nova/blob/cdfbb9a668fdcf289ffdfa5252d102e9d3e2ec35/nova/api/metadata/base.py#L325-L330 | 15:54 |
| dansmith | mordred: right | 15:54 |
| danpb | so we can intelligently pick the right node to put it on, if there's a choice of some nodes with & without the feature | 15:54 |
| dansmith | danpb: and I want fewer of those things | 15:54 |
| mordred | also - image properties are exceptionally opaque interface for end users, fwiw | 15:54 |
| dansmith | mordred: right, that | 15:55 |
| artom | dansmith, well, it would only devolve to that case if the cloud has 0 nodes that support tagging | 15:55 |
| sdague | mordred: that's mostly a documentation problem, right? | 15:55 |
| artom | Otherwise the scheduler should find a right one | 15:55 |
| mordred | sdague: api discoverability too | 15:55 |
| danpb | we could avoid the image property if we can look at the actuall VM boot request to see if the user has actually requested tags or not | 15:55 |
| dansmith | artom: no, you have to specify it in order to make it work | 15:55 |
| artom | dansmith, specify what? | 15:55 |
| mordred | sdague: "what image properties does this cloud think are meaningful" | 15:55 |
| sdague | mordred: ok, sure | 15:55 |
| dansmith | danpb: and fail after we make it to the compute node because you asked for a thing you were told was available but will never be available on this cloud? that sucks :) | 15:56 |
| dansmith | I want to avoid someone booting three hundred attempts, all failed, before calling support and them saying "oh we don't do that thing. we run xenapi" | 15:56 |
| sdague | dansmith: ++ | 15:56 |
| mordred | sdague: did you know that you need to put "vm_mode: hvm" on images in rackspace to have them work with performance flavors? as a for instance | 15:56 |
| *** zenoway has joined #openstack-nova | 15:56 | |
| dansmith | especially since implementing it in each driver is something we know we need to do, but nobody did it because we merged the api and moved on :) | 15:56 |
| sdague | mordred: I did not know that. Sounds awesome. | 15:57 |
| mordred | or "xenapi_use_agent: false" if you don't want the boot to wait for the xenapi agent to phone home | 15:57 |
| danpb | that's not a new problem though and we've not blocked stuff because of that | 15:57 |
| dansmith | that, to me, is why the project owning a driver in tree is expensive.. because we should maintain it and not just let it rot | 15:57 |
| danpb | we've got countless features which are only implemneted in one driver | 15:57 |
| dansmith | this matters more to me, because of the guest-facing api | 15:57 |
| dansmith | which we've not had before in terms of a virt-specific feature that affects metadata, that I know of | 15:58 |
| *** iceyao has quit IRC | 15:58 | |
| danpb | every aspect of virt machine configuration is guest facing ABI | 15:58 |
| danpb | whether its via metadata service or via the hardware platform, its all guest facing | 15:58 |
| dansmith | but we control the metadata api | 15:58 |
| dansmith | and it's the thing we're trying to abstract, not the hypervisor stuff we don't control | 15:58 |
| artom | Don't we already have the situation where you fail to boot an instance if you ask for a NUMA topology or some PCI device that doesn't exist in that cloud? | 15:58 |
| dansmith | anyway, that's MHO | 15:58 |
| *** armax has joined #openstack-nova | 15:59 | |
| artom | What's different if we refuse to boot an instance because there are no hosts with drivers that support tagging? | 15:59 |
| dansmith | artom: capacity aside, you should fail fast in the scheduler for that | 15:59 |
| artom | dansmith, and it's impossible to fail fast in the scheduler with tagging? | 15:59 |
| dansmith | artom: no, we could write a filter probably, but the filter would be "some drivers haven't been maintained" and not "this device is not available" | 16:00 |
| dansmith | which is kinda embarrassing no? :) | 16:00 |
| mriedem_meeting | flip214: ok, want to update the nova bp again? | 16:00 |
| *** mriedem_meeting is now known as mriedem | 16:00 | |
| *** Oku_OS is now known as Oku_OS-away | 16:00 | |
| auggy | good morning! | 16:00 |
| danpb | dansmith: i don't think it is saying that they've not been maintained | 16:01 |
| artom | dansmith, well, we could disguise it as "that driver can't do it" | 16:01 |
| *** sdake_ is now known as sdake | 16:01 | |
| artom | Or "strongly prefers not to do it, on principle" | 16:01 |
| artom | ;) | 16:01 |
| danpb | dansmith: it is just reflecting the reality that we don't have unlimited resources to implement features across every driver at once | 16:01 |
| sfinucan | dansmith: Aye - more like "this isn't a priority for anyone using this driver" | 16:01 |
| *** lakshmiS has joined #openstack-nova | 16:01 | |
| *** klkumar has joined #openstack-nova | 16:01 | |
| artom | Or a 202 accepted that becomes a 200 when the implementation is finished :D | 16:01 |
| danpb | and don't want to delay getting useful features to one driver that have 90% of openstack user survey share | 16:01 |
| *** stewie925 has joined #openstack-nova | 16:01 | |
| *** mpavone has quit IRC | 16:01 | |
| dansmith | sfinucan: not using, maintaining :) | 16:01 |
| *** ctrath has quit IRC | 16:02 | |
| danpb | while we implement it for another driver which has 5% user share and no immediate demand for the feature | 16:02 |
| *** mgoddard_ has quit IRC | 16:02 | |
| *** mgoddard_ has joined #openstack-nova | 16:02 | |
| dansmith | danpb: it's the users that have the demand, and they don't control what hypervisor was chosen | 16:02 |
| dansmith | that sets this apart from something like NUMA to me | 16:02 |
| *** penick has joined #openstack-nova | 16:02 | |
| sfinucan | dansmith: Sounds like what was discussed at the operator meetup | 16:03 |
| artom | Wait, who's demanding NUMA that are not users? | 16:03 |
| danpb | well the immediate near term demand for the feature is from NFV use case | 16:03 |
| sfinucan | operator: we need this, devs: provide contributions? | 16:03 |
| danpb | and those users certainly do control what hypervisor their imags are running on | 16:03 |
| *** marcusvrn_ has quit IRC | 16:03 | |
| danpb | sure, other non-NFV users will benefit from this feature too | 16:03 |
| dansmith | danpb: that is the worst argument ever :) | 16:03 |
| mriedem | flip214: the drbd bp is approved now, i've removed my -2 from the nova change, but still -1 for the comments inline | 16:04 |
| dansmith | everyone who has ever booted with more than one disk or network will use this :) | 16:04 |
| artom | dansmith, not if you tell them the cloud's chosen virt driver doesn't have it implemented yet ;) | 16:04 |
| *** rcernin has quit IRC | 16:04 | |
| vladikr | dansmith, not necessarily, most probably they won't care about the order | 16:04 |
| danpb | dansmith: it really isn't - we can give the feature to the people who really need it right now, and every one else gets it in 1 year time | 16:04 |
| danpb | dansmith: or we can give it to no one and make everyone wait a year | 16:05 |
| dansmith | vladikr: I completely disagree :) | 16:05 |
| danpb | its madness to artifically make everyone wait for it to be imeplemented across every driver | 16:05 |
| sdague | how much effort is it to get it added to hyperv? | 16:05 |
| sdague | that's sort of the #2 driver | 16:05 |
| dansmith | danpb: are you and artom going to work on the other drivers after you land the api? | 16:06 |
| artom | There *is* a chance people will just tag stuff "butts" because they can ;) | 16:06 |
| dansmith | sdague: my other concern is that these things get designed with no input from the others, and then there is a problem | 16:06 |
| *** jdurgin1 has quit IRC | 16:06 | |
| danpb | dansmith: no, because we're not domain experts in hyperv apis | 16:06 |
| *** igordcard has quit IRC | 16:06 | |
| dansmith | danpb: right, so it'll never get done for those other people | 16:06 |
| *** mriedem is now known as mriedem_afk | 16:06 | |
| artom | dansmith, I personally wouldn't mind, but like danbp has said, I have 0 clue about other drivers | 16:06 |
| danpb | dansmith: this has been throw spec review where other driver maintainers commented that the design was acceptrable | 16:06 |
| artom | So it would be a long and error fraught process | 16:06 |
| sdague | danpb: sure, but you could coordinate with the experts and try to get folks signed up for doing it this cycle as well | 16:06 |
| danpb | and even indicated they'd be willing to support it | 16:07 |
| sdague | this seems universal enough that having a consistent story would be really good | 16:07 |
| *** unicell has quit IRC | 16:07 | |
| dansmith | danpb: yeah and we've deviated from the spec's original design quite a bit remember, especially around when in the process we need to generate this data | 16:07 |
| danpb | sdague: different driver teams have different priorities for developing features | 16:07 |
| danpb | dansmith: my latest comments on the reviews were to actually go back to what the spec design said | 16:08 |
| *** unicell has joined #openstack-nova | 16:08 | |
| danpb | as the current patches up there are flawed | 16:08 |
| sdague | danpb: sure, but we should try to coordinate | 16:08 |
| dansmith | danpb: of making an rpc call from the metadata api? | 16:08 |
| danpb | dansmith: no, in terms of the way we're representing the data inside nova | 16:08 |
| *** ccard_ has joined #openstack-nova | 16:08 | |
| artom | sdague, I don't think anyone's disagreeing that we shouldn't coordinate, but the reality on the ground means that coordinating would probably delay the feature landing by a significant amount | 16:09 |
| danpb | all the versioned object model we defined in the spec & originally implemented got dropped in latest patches | 16:09 |
| dansmith | danpb: heh, okay | 16:09 |
| stewie925 | good morning all | 16:09 |
| artom | So, I think we have a plan to bo back to the original metadata objects representation while still using the database | 16:10 |
| danpb | anyway I think its a great disservice to our users if we artificially delay this until all drivers implement it | 16:10 |
| danpb | i don't really have much more to say | 16:10 |
| artom | The question we're all yelling about now is what to do with unequal driver support | 16:10 |
| *** ctrath has joined #openstack-nova | 16:10 | |
| artom | Either indicate it in the API somehow | 16:10 |
| artom | Or wait until all drivers have it | 16:10 |
| artom | Personally I prefer indicating | 16:11 |
| artom | I think danpb agrees? | 16:11 |
| *** sahid has quit IRC | 16:11 | |
| *** fawadkhaliq has quit IRC | 16:11 | |
| danpb | the approved spec already says that we'll do it in the libvirt driver for QEMU only | 16:11 |
| dansmith | artom: if we had any indication that there were implementations for the other drivers coming, then I'd be less concerned | 16:11 |
| danpb | unless someone wants to submit a patch to unapprove the spec | 16:11 |
| stewie925 | I just attempted to create an instance in cell environment and am getting error message ""No valid host was found. There are not enough hosts available." | 16:11 |
| danpb | "The initial implementation will be done for the Libvirt driver, however, other virt driver maintainers are encouraged to provide the same functionality." | 16:12 |
| stewie925 | but I have captured the n-api and n-sch logs and theyre in http://paste.openstack.org/show/497753/ | 16:12 |
| danpb | http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/virt-device-role-tagging.html | 16:12 |
| *** ccard has quit IRC | 16:12 | |
| *** nkrinner has quit IRC | 16:12 | |
| artom | dansmith, so how would be get said indications (or obtain evidence of lack thereof?) | 16:12 |
| danpb | so its not like this is a changein plan in any way | 16:12 |
| *** nkrinner has joined #openstack-nova | 16:12 | |
| *** nkrinner is now known as nkrinner_afk | 16:12 | |
| artom | Mailing list? | 16:12 |
| rlrossit | gibi: sorry, I was in meetings | 16:13 |
| dansmith | danpb: surely you appreciate that problems become clearer once you start implementing a thing, and that if you land a spec to do something crazy, it's okay to reconsider when other concerns are surfaced, right? | 16:13 |
| rlrossit | gibi: but it looks like https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L47-L51 says it'll take fields from all supers and apply them as setters/getters on itself, but it won't do it on the super classes | 16:13 |
| rlrossit | my guess is you aren't using the super classes at all, so you aren't affected by it | 16:13 |
| dansmith | danpb: so pointing to the letter of what was approved doesn't really add much to the argument, IMHO | 16:13 |
| dansmith | danpb: I'm just saying that this concerns me and that I think the reality will be that it never gets implemented elsewhere and that I think that's a problem.. | 16:14 |
| danpb | as i mentioned earlier though, this was discussed during the review of the spec and feedback from other virt driver maintainers explicitly solicited | 16:14 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Rbd: add create_from_image & create_from_func https://review.openstack.org/317694 | 16:15 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Flat: add create_from_image & create_from_func https://review.openstack.org/317701 | 16:15 |
| sdague | danpb: I think what's being asked is a bunch of the other virt driver authors said they liked it back in Nov 2015 | 16:15 |
| danpb | dansmith: if it never gets implemented in a particular driver imho that's more a reflection on the state of that driver's supportability than anything else | 16:15 |
| sdague | danpb: and that the ask is please *also* spend some time rounding up those folks and trying to get them to prioritize it | 16:16 |
| dansmith | danpb: I don't disagree | 16:16 |
| sdague | to make for a good community land of the feature in multiple drivers during this release | 16:16 |
| *** claudiub has joined #openstack-nova | 16:16 | |
| sdague | it's an organizing overhead for sure, but it makes all of nova better | 16:16 |
| dansmith | clarkb: ohai | 16:16 |
| artom | dansmith, there was some indications in the original Mitaka spec review from other drivers IIRC | 16:16 |
| dansmith | oops | 16:16 |
| dansmith | claudiub: ohai | 16:17 |
| danpb | i don't think we should be blocking features on other well maintained drivers because of possibility that other virt driver maintainers may drop the ball | 16:17 |
| artom | danpb, example from BobBall here: https://review.openstack.org/#/c/195662/8/specs/mitaka/approved/virt-device-role-tagging.rst@324 | 16:17 |
| sdague | artom: right, so please talk with those folks and try to get them on board. Is that an unreasonable ask? | 16:17 |
| claudiub | dansmith: hellou | 16:17 |
| dansmith | danpb: and I think that we're not just writing a proxy to libvirt here | 16:17 |
| * danpb leaves to get some useful work done now | 16:17 | |
| *** danpb has left #openstack-nova | 16:17 | |
| claudiub | am I in trouble? :) | 16:17 |
| dansmith | claudiub: wondering what your plans are for implementing the device tagging spec, if any? | 16:17 |
| dansmith | claudiub: I know you were interested in it during review, but I also kinda hope we can land more than just one driver's support this cycle | 16:18 |
| artom | sdague, totally reasonable :) | 16:18 |
| claudiub | dansmith: it's actually in my plans for this cycle. | 16:18 |
| dansmith | claudiub: sweet,t hat makes me feel better | 16:18 |
| artom | sdague, I would just argue that an all or nothing approach to landing this is too extreme, and if we adopt it we can essentially forget the whole thing entirely | 16:18 |
| sdague | artom: I think that just doing some leg work in talking to those folks and presenting what peoples plans are | 16:18 |
| sdague | would be great | 16:18 |
| sdague | artom: I honestly don't think all or nothing is what we're talking about. It's > 1 would be really good | 16:19 |
| claudiub | dansmith: just one question though, can the hyper-v implementation piggyback on another spec, or do I have to submit another one? | 16:19 |
| dansmith | sdague: artom: right..agree.. | 16:19 |
| sdague | and make everyone happier in getting the feature prioritized to land | 16:19 |
| dansmith | claudiub: I think the spec is landed and covers you | 16:19 |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref, os-fping.inc https://review.openstack.org/318764 | 16:19 |
| claudiub | yeay, music to my ears then. :) | 16:19 |
| artom | sdague, dansmith, ah, so if we have at least one other driver in process of implementing this... | 16:20 |
| dansmith | claudiub: we'd love to see even some prototype code up in parallel with the libvirt bits | 16:20 |
| BobBall | We'll implement in this cycle too, of course :) | 16:20 |
| BobBall | artom: ^^ | 16:20 |
| artom | Yeah, that's a reasonnable compromise | 16:20 |
| sdague | BobBall: great | 16:20 |
| dansmith | artom: all we're trying to do is get that ball rolling... and as you see, it's not that hard :) | 16:20 |
| artom | dansmith, oh for sure, and thanks for that :) | 16:20 |
| *** vishwanathj has joined #openstack-nova | 16:20 | |
| artom | I was afraid the argument was all or nothing | 16:21 |
| sdague | artom: yeh, I think the libvirt team is the team most ahead on this one, and if they took the extra responsibility of just generally touching base with other driver authors and keeping an eye on what's moving forward, that would be great | 16:21 |
| dansmith | BobBall: any idea when? it'd be cool to have some code up in parallel to the libvirt bits | 16:21 |
| artom | So, if we go back to the technical side of things, this means not all drivers will land this at the same time, yes? So we need a way to indicate/control tags and booting with tags | 16:22 |
| artom | Can we get the API side of that sorted out? | 16:22 |
| *** yamahata has joined #openstack-nova | 16:22 | |
| dansmith | artom: I don't want to have to add an image flag for it, and I think we're probably on a path to not need to | 16:23 |
| BobBall | dansmith: I've not been tracking this discussion very closely... When will the libvirt bits be up? :) Very happy to work for getting them up in parallel | 16:23 |
| dansmith | artom: but yeah, at least we should fail the boot if we have tags and no support | 16:23 |
| dansmith | BobBall: the bits are up, but have been redone multiple times | 16:23 |
| vladikr | https://review.openstack.org/#/q/topic:bp/virt-device-role-tagging | 16:24 |
| vladikr | BobBall, ^^ | 16:24 |
| dansmith | BobBall: I'd like the guys working on the libvirt side to coordinate with the other driver owners to synchronize exactly that.. when the model is up that you can implement | 16:24 |
| artom | BobBall, err, yeah, we need to re-jigger the bits, so wait for that please :) | 16:24 |
| BobBall | I was looking at the wrong tag :) | 16:24 |
| BobBall | Yeah - we're up for that dansmith. Want someone to talk to now artom, so they can get the lay of the land, etc? | 16:25 |
| dansmith | BobBall: I think the aforementioned rejiggering might should happen first to avoid confusion, but maybe try to do that sync next week if you'll be around? | 16:25 |
| BobBall | I will indeed | 16:26 |
| artom | BobBall, sure - I won't engage just yet, as we're in a re-write ourselves, but I definitely need a list of folks form various drivers to harass ;) | 16:26 |
| dansmith | I actually don't know who is working on vmware these days | 16:26 |
| dansmith | which is probably not good | 16:26 |
| artom | claudiub is on the hyperv side of things, yar? | 16:27 |
| BobBall | artom: My team is in PRC, so I'm an easy list of 1 for this sort of timezone (I'm in UK though - so won't be around much later than this) | 16:27 |
| dansmith | artom: yes | 16:27 |
| artom | BobBall, PRC as in... Taiwan? | 16:27 |
| claudiub | jaypipes: hellou, you around? | 16:27 |
| dansmith | artom: if xen and hyperv are up at least in prototype form I think we're in much less-concerned mode | 16:27 |
| sdague | well, honestly, if vmware is back to rotting in tree, that's what it is. Having hyperv, xen, libvirt would be pretty solid | 16:27 |
| dansmith | yeah | 16:28 |
| BobBall | artom: Peoples Republic of China :) | 16:28 |
| artom | No wait, the other "proper" China is PRC | 16:28 |
| dansmith | sdague: and if so, we probably need to be poking people | 16:28 |
| *** tonytan4ever has quit IRC | 16:28 | |
| artom | Damn communists ;) | 16:28 |
| stewie925 | hehehe - taiwan is ROC (Republic of China) | 16:29 |
| *** fesp has quit IRC | 16:29 | |
| artom | (Says the Soviet-born dude) | 16:29 |
| *** Apoorva has joined #openstack-nova | 16:29 | |
| clarkb | kashyap: fwiw sudo virsh cpu-models x86 does not work locally for me. 'error: failed to get CPU model names' this is whtout any modifications or devstack or anything | 16:30 |
| stewie925 | hello alaski and melwitt - only when you have a chance... my boot instance in cell environment failed with "no valid hosts" error and I posted the logs in http://paste.openstack.org/show/497753/ - much appreciated! | 16:32 |
| *** tonytan4ever has joined #openstack-nova | 16:33 | |
| clarkb | huh works for ppc64 so clearly pebkac | 16:34 |
| *** jinxing has quit IRC | 16:34 | |
| openstackgerrit | Sam Betts proposed openstack/nova: Eliminate resource tracker to host manager races https://review.openstack.org/308348 | 16:34 |
| *** raunak has joined #openstack-nova | 16:35 | |
| *** klkumar has quit IRC | 16:36 | |
| *** paul-carlton2 has joined #openstack-nova | 16:37 | |
| melwitt | stewie925: next you want to check n-cpu.log for errors. in n-sch it selected a host and after that something failed | 16:38 |
| stewie925 | oh thank you melwitt - checking on n-cpu logs now | 16:39 |
| melwitt | also check n-cond, n-cell-child, n-cell-region for errors. and maybe n-net too | 16:39 |
| *** akshai has quit IRC | 16:40 | |
| stewie925 | oh thanks - will make a note to check all n- logs as well | 16:40 |
| mgagne | melwitt: tyvm for your help with scheduler last week. it's now freaking fast :D The only other config we tweaked is max_concurrent_builds to allow more concurrent builds | 16:40 |
| kashyap | clarkb: Sorry, was AFK. Yeah, I could reproduce on x86 too -- "error: this function is not supported by the connection driver: virConnectGetCPUModelNames" | 16:41 |
| clarkb | kashyap: doesn't make it easy to see what is valid or not :) | 16:42 |
| melwitt | mgagne: awesome! so all you did was reorder the filters and increase max_concurrent_builds? or was there more you had to do? | 16:42 |
| *** PaulMurray has quit IRC | 16:42 | |
| mgagne | using caching_scheduler too to bypass the overhead of initializing all those host state objects | 16:42 |
| kashyap | clarkb: Let me check w/ virt folks on OFTC | 16:42 |
| melwitt | mgagne: ah, cool. that's right I forgot you had mentioned that last time | 16:43 |
| mgagne | baremetal takes a long time before cleaning anyway so we more or less care about them no being available for another minute. we also increased scheduler_max_attempts | 16:43 |
| mgagne | just to cover the case where scheduler allocates you a node that was claimed in the past minute. | 16:44 |
| sdague | clarkb: x86_64 | 16:44 |
| *** tyrola has quit IRC | 16:44 | |
| sdague | sudo virsh cpu-models x86_64 works fine on my xenial box | 16:44 |
| clarkb | sdague: aha | 16:44 |
| clarkb | sdague: so that doesn't match what is in the xml | 16:44 |
| clarkb | xml says x86 | 16:45 |
| *** nic has joined #openstack-nova | 16:45 | |
| melwitt | mgagne: makes sense | 16:45 |
| *** markus_z has quit IRC | 16:45 | |
| *** fawadkhaliq has joined #openstack-nova | 16:46 | |
| sdague | clarkb: huh, yep, guess it doesn't | 16:46 |
| *** zengine has joined #openstack-nova | 16:46 | |
| *** jamesdenton has quit IRC | 16:47 | |
| *** zengine has quit IRC | 16:47 | |
| clarkb | kashyap: ok I have locally confirmed the file path on xenial matches and that a sudo systemctl restart libvirt-bin is sufficient to make that a valid cpu | 16:47 |
| *** zengine has joined #openstack-nova | 16:47 | |
| kashyap | clarkb: You mean, the file path has not changed | 16:48 |
| clarkb | kashyap: correct | 16:48 |
| *** edand_ has joined #openstack-nova | 16:48 | |
| kashyap | clarkb: What test did you do locally w/ DevStack? | 16:48 |
| *** moshele has quit IRC | 16:48 | |
| *** earlephilhower has joined #openstack-nova | 16:48 | |
| clarkb | kashyap: I just ran sudo devstack/tools/cpu_map_update.py /usr/share/libvirt/cpu_map.xml && sudo systemctl restart libvirt-bin && virsh cpu-modles x86_64 | 16:49 |
| *** Jabadia_ has joined #openstack-nova | 16:50 | |
| *** sfinucan has quit IRC | 16:50 | |
| kashyap | clarkb: Does your `virsh uri` show "qemu:///system"? If not -- we see that error when you run `virsh cpu-models x86` (For me it was showing lxc:///) | 16:50 |
| *** doug-fish has quit IRC | 16:50 | |
| clarkb | kashyap: sdague pointed out I had to use x86_64 | 16:50 |
| *** doug-fish has joined #openstack-nova | 16:51 | |
| *** mvk_ has quit IRC | 16:51 | |
| clarkb | which is totally confusing because it doesn't match the config file but whatever it works | 16:51 |
| *** sambetts is now known as sambetts|afk | 16:52 | |
| kashyap | clarkb: Hmm. So, the issue is probably why isn't libvirt converting 'gate64' model into something it can understand... | 16:53 |
| *** mriedem has joined #openstack-nova | 16:53 | |
| *** penick has quit IRC | 16:54 | |
| *** mriedem_afk has quit IRC | 16:54 | |
| *** penick has joined #openstack-nova | 16:54 | |
| *** doug-fis_ has joined #openstack-nova | 16:54 | |
| clarkb | the next thing devstack does is a sudo service libvirt-bin stop && sudo service libvirt-bin start | 16:55 |
| *** doug-fish has quit IRC | 16:55 | |
| clarkb | when I do this locally I don't get a running libvirt-bin service | 16:55 |
| *** penick has quit IRC | 16:55 | |
| *** zenoway has quit IRC | 16:55 | |
| *** mriedem1 has joined #openstack-nova | 16:55 | |
| *** derekh has quit IRC | 16:55 | |
| clarkb | if I use systemctl I get a thing that works | 16:56 |
| *** zenoway has joined #openstack-nova | 16:56 | |
| clarkb | I wonder if this is the problem | 16:56 |
| clarkb | we might want to update devstack's service start and stop to detect if systemctl is present and use that if so else if ubuntu use service | 16:57 |
| kashyap | clarkb: Oh, it's the difference between upstart & systemd | 16:57 |
| sdague | clarkb: yeh, that's probably the crux of it | 16:57 |
| clarkb | kashyap: yes, I am beginning to suspect libvirt on xenial does not work properly if using hte old service management | 16:57 |
| sdague | clarkb: well, the issue is if sudo service libvirt-bin stop && sudo service libvirt-bin sta | 16:57 |
| sdague | and the first one is async submit to systemd | 16:58 |
| sdague | it's not really done when you do start | 16:58 |
| clarkb | ah | 16:58 |
| *** yamahata has quit IRC | 16:58 | |
| sdague | that is my guess | 16:58 |
| *** r-daneel has joined #openstack-nova | 16:58 | |
| dansmith | claudiub: hah, did jaypipes catch you with asking to see the rescue test run? https://review.openstack.org/#/c/127159/ | 16:58 |
| clarkb | sdague: would a systemctl stop && systemctl start be synchrnous? | 16:58 |
| *** yamahata has joined #openstack-nova | 16:58 | |
| sdague | clarkb: I don't know, but a restart is best | 16:58 |
| *** mtanino has quit IRC | 16:58 | |
| clarkb | sdague: restart does not work on trusty | 16:58 |
| sdague | because you've told systemd the intent | 16:58 |
| *** whenry has joined #openstack-nova | 16:59 | |
| *** doug-fis_ has quit IRC | 16:59 | |
| sdague | we should just make it a conditional for the backend, otherwise we're going to be chasing bugs one way or another | 16:59 |
| *** mriedem has quit IRC | 16:59 | |
| *** doug-fish has joined #openstack-nova | 16:59 | |
| sdague | clarkb: are we actually sure restart doesn't work on trusty? | 17:00 |
| sdague | or did it not work on precise | 17:00 |
| sdague | and it's a carry forward | 17:00 |
| dansmith | claudiub: meaning it looks like it failed on the first run :D | 17:00 |
| clarkb | sdague: I am 100% sure it did not work at the time I got this going a year or so ago | 17:00 |
| *** igordcard has joined #openstack-nova | 17:00 | |
| clarkb | sdague: it is possible that trusty has since fixed their broken libvirt service management | 17:00 |
| *** zenoway has quit IRC | 17:01 | |
| sdague | ok, well, systemd does work | 17:01 |
| clarkb | I spent about a day debugging why restart didi work | 17:01 |
| sdague | so lets do the working thing with systemd, and leave the trusty work around, conditional on trusty | 17:01 |
| *** penick has joined #openstack-nova | 17:01 | |
| kashyap | clarkb: Updating 'functions-common' to use systemctl work -- Maybe we can post a change to see what breaks? | 17:01 |
| claudiub | dansmith: well, the commit sat for so long, it was no longer valid. :) | 17:01 |
| clarkb | it was a huge waste of time | 17:01 |
| dansmith | claudiub: ah, okay, fair then :D | 17:01 |
| sdague | for all the griping people had about systemd, a ton of stuff is so much more reliable with it :) | 17:01 |
| claudiub | dansmith: image_meta changed since this code was implemented. it was dict-like before, now it isn't. I've updated that, ran tempest for the rescue tests in my env, worked. | 17:02 |
| kashyap | sdague: Fully agreed. It's just those who don't invest some time learning it complain | 17:02 |
| dansmith | claudiub: yep, well, I'll have a look when the results come back | 17:02 |
| claudiub | cool. :) | 17:03 |
| clarkb | sdague: kashyap if only unit file names didn't change the behavior of srevices... oh well (this is the root of at least one bug in centos7) | 17:03 |
| dansmith | claudiub: looks like also a unit test fail from jenkins? | 17:03 |
| claudiub | dansmith: indeed. something changed in imagecache since last jenkins ran on this patch. | 17:03 |
| sdague | ok, I'm going to get outside for a bit, clarkb / kashyap if you push up a patch for review, I'll hit it when I get back so we can unstick this | 17:03 |
| kashyap | clarkb: Oh, what way a file name change triggers _behavior_ of the service? | 17:03 |
| dansmith | claudiub: ah, yep | 17:03 |
| kashyap | sdague: No worries. Get some air! | 17:03 |
| clarkb | kashyap: @s mean run many of these, useful to inetd type replacement | 17:04 |
| *** doug-fish has quit IRC | 17:04 | |
| clarkb | kashyap: and people write their unit files assuming that behavior then don't get the filename right then centos7 git daemon doesn't work | 17:04 |
| clarkb | kashyap: are you going to write the devstack change? you can also depends-on that change on the live migrate change side to see if that makes it all work | 17:05 |
| kashyap | clarkb: I can. Got a URL to the live-migrate change? | 17:05 |
| clarkb | kashyap: I don't | 17:05 |
| kashyap | clarkb: No worries, I'll try find out, once I make some quick dinner | 17:06 |
| *** lucasagomes is now known as lucas-afk | 17:06 | |
| kashyap | clarkb: Where is systemctl located in Ubuntu? /bin I suppose? | 17:07 |
| clarkb | yup | 17:07 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Hyper-V: Implement nova rescue https://review.openstack.org/127159 | 17:09 |
| mriedem1 | sdague: is this your way of calling me an asshole for pointing out typos in your specs? :) https://web.archive.org/web/20051016203842/http://diveintomark.org/archives/2004/08/16/specs | 17:09 |
| *** mriedem1 is now known as mriedem | 17:09 | |
| *** doug-fish has joined #openstack-nova | 17:11 | |
| *** doug-fish has quit IRC | 17:11 | |
| *** doug-fish has joined #openstack-nova | 17:12 | |
| *** akshai has joined #openstack-nova | 17:13 | |
| *** paul-carlton2 has quit IRC | 17:13 | |
| *** paul-carlton2 has joined #openstack-nova | 17:14 | |
| *** fragatina has joined #openstack-nova | 17:15 | |
| *** jlanoux has quit IRC | 17:15 | |
| *** fragatina has quit IRC | 17:16 | |
| *** fragatina has joined #openstack-nova | 17:17 | |
| syjulian | jaypipes: hi jay, would you be able to review this patch fothe nova.virt.firewall.IpTablesFirewallDriver bug? https://review.openstack.org/#/c/311758/ | 17:18 |
| *** ivase_ has quit IRC | 17:18 | |
| syjulian | jaypipes: this is the bug https://bugs.launchpad.net/nova/+bug/1295889 | 17:18 |
| openstack | Launchpad bug 1295889 in OpenStack Compute (nova) "No unit tests for nova.virt.firewall.IptablesFirewallDriver" [Low,In progress] - Assigned to Julian Sy (syjulian) | 17:18 |
| mriedem | edleafe: is there a template somewhere for the config option help text? | 17:22 |
| mriedem | or just the spec? | 17:22 |
| *** BobBall is now known as BobBall_AWOL | 17:22 | |
| edleafe | mriedem: just the spec, but to be honest, I just copy/pasted markus_z's and edited from there | 17:24 |
| *** zzelle_ has joined #openstack-nova | 17:24 | |
| dansmith | artom: is it worth me commenting on this: https://review.openstack.org/#/c/264016/36/nova/virt/libvirt/driver.py ? | 17:24 |
| dansmith | artom: how much is going to change? | 17:25 |
| *** ivase has joined #openstack-nova | 17:25 | |
| artom | dansmith, that specific diff will change entirely... | 17:26 |
| dansmith | okay | 17:26 |
| dansmith | good, because it's fatally flawed :D | 17:26 |
| dansmith | oh wait, actually it's not | 17:26 |
| dansmith | glad I didn't comment | 17:26 |
| vladikr | dansmith, we will post a new draft in couple of mins | 17:26 |
| artom | dansmith, I'm giving you the Fry eyes right now | 17:27 |
| dansmith | vladikr: okie | 17:27 |
| artom | dansmith, can you quickly comment here? In case we're about to repeat the same mistake? | 17:27 |
| dansmith | artom: you didn't make a mistake, so no :) | 17:27 |
| dansmith | well, not the mistake I thought | 17:27 |
| dansmith | artom: vladikr: that hunk was nice and small for pointing other drivers at | 17:28 |
| *** zengine has quit IRC | 17:28 | |
| dansmith | a nice "do this like this" link | 17:28 |
| artom | Except... don't do it like this just yet :) | 17:28 |
| dansmith | well, right, I said "was" | 17:28 |
| dansmith | "could have been" | 17:28 |
| artom | So much lost potential | 17:28 |
| artom | Thanks for calling it a hunk though, it feels better about its body now | 17:29 |
| *** ivase has quit IRC | 17:29 | |
| flip214 | mriedem: thank you very much, I'll redo the rootwrap => privsep parts. | 17:30 |
| *** paul-carlton2 has quit IRC | 17:30 | |
| mriedem | dansmith: what hunks are nice for you now? | 17:32 |
| dansmith | mriedem: small hunks of concise code.. why? what hunks were _you_ thinking of? | 17:32 |
| mriedem | https://cgrzeszczak.files.wordpress.com/2012/11/untitled.png?w=341 | 17:33 |
| mriedem | that one | 17:33 |
| *** jwcroppe_ has quit IRC | 17:33 | |
| *** fawadkhaliq has quit IRC | 17:33 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Rbd: add create_from_image & create_from_func https://review.openstack.org/317694 | 17:34 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Flat: add create_from_image & create_from_func https://review.openstack.org/317701 | 17:34 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Ploop: add create_from_image & create_from_func https://review.openstack.org/317689 | 17:34 |
| *** ivase has joined #openstack-nova | 17:35 | |
| *** moshele has joined #openstack-nova | 17:35 | |
| flip214 | mriedem: grepping the nova code for privsep turns up nothing. would the DRBD transport be the first thing to use privsep in nova? https://review.openstack.org/#/q/privsep doesn't really help either... | 17:41 |
| *** lpetrut has quit IRC | 17:42 | |
| *** ctrath has quit IRC | 17:43 | |
| flip214 | hmmm, this part just calls os-brick anyway, and that's already converted... so there's nothing rootwrap or privsep specific left, is there? | 17:44 |
| *** r-daneel has quit IRC | 17:44 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Rbd: add create_from_image & create_from_func https://review.openstack.org/317694 | 17:45 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Flat: add create_from_image & create_from_func https://review.openstack.org/317701 | 17:45 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Ploop: add create_from_image & create_from_func https://review.openstack.org/317689 | 17:45 |
| *** tonytan4ever has quit IRC | 17:45 | |
| openstackgerrit | Philipp Marek proposed openstack/nova: Connecting Nova to DRBD storage nodes directly https://review.openstack.org/256292 | 17:45 |
| mriedem | flip214: there isn't anything in nova using privsep yet, no | 17:45 |
| flip214 | mriedem: thanks. | 17:45 |
| flip214 | new patch just pushed. | 17:45 |
| flip214 | thanks a lot for the *very* quick feedback! | 17:46 |
| mriedem | since the nova volume driver doesn't need to call drbdadm then we don't need the rootwrap filter, and os-brick is using privsep already, yeah, so we should be ok there | 17:46 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: Introduce a CONF flag to determine glance client version https://review.openstack.org/315963 | 17:47 |
| openstackgerrit | Mike Fedosin proposed openstack/nova: split the glance API path based on config https://review.openstack.org/316007 | 17:47 |
| mriedem | flip214: is there a third party ci for drbd in cinder/os-brick where i can see this passing? | 17:47 |
| *** akshai has quit IRC | 17:47 | |
| *** pushkaru has quit IRC | 17:47 | |
| *** daemontool has quit IRC | 17:47 | |
| flip214 | mriedem: uh, I don't know. | 17:47 |
| *** Guest86061 has quit IRC | 17:47 | |
| flip214 | the DRBD driver in Cinder is in openstack-infra, as it's pure Open Source... don't know about the os-brick and/or Nova parts, though. | 17:48 |
| flip214 | For single-machine CI it wouldn't be triggered anyway. | 17:48 |
| mriedem | flip214: ok, so is there any drbd ci for openstack, anywhere? | 17:48 |
| *** rlrossit has quit IRC | 17:48 | |
| flip214 | Multi-machine CI was an ongoing topic in -infra last time I asked, perhaps there's some example now that can be adapted. | 17:49 |
| mriedem | we have multinode jobs | 17:49 |
| *** haukebruno has quit IRC | 17:49 | |
| mriedem | but cinder isn't using any yet that i know of | 17:49 |
| mriedem | hemna: do you know anything about ci for drbd in cinder? | 17:49 |
| *** rama_y has joined #openstack-nova | 17:49 | |
| *** jwcroppe has joined #openstack-nova | 17:51 | |
| flip214 | mriedem: the jenkins job is called gate-tempest-dsvm-full-drbd-devstack-nv | 17:51 |
| flip214 | http://ci-watch.tintri.com/project?project=cinder&time=7+days has it in the overview, for example | 17:52 |
| hemna | hey | 17:52 |
| mriedem | flip214: ok, that's helpful | 17:52 |
| mriedem | i'll see what that runs on in project-config | 17:52 |
| hemna | oh looks like flip214 has it covered :) | 17:52 |
| flip214 | or https://review.openstack.org/#/c/318749/ has a recent run | 17:52 |
| flip214 | hemna: yeah, thanks. | 17:52 |
| flip214 | mriedem: right below the sheepdog gate check. | 17:53 |
| *** rlrossit_ has joined #openstack-nova | 17:53 | |
| flip214 | thanks for the attention, I'm /off now. | 17:54 |
| flip214 | have a nice vacation! | 17:54 |
| mriedem | thanks | 17:54 |
| *** akshai has joined #openstack-nova | 17:54 | |
| *** rmart04 has quit IRC | 17:55 | |
| *** tonytan4ever has joined #openstack-nova | 17:57 | |
| mriedem | hrm, tempest.api.compute.volumes.test_attach_volume.AttachVolumeTestJSON.test_attach_detach_volume is skipped in the drbd job | 17:59 |
| *** jichen has joined #openstack-nova | 17:59 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Completed migrations are not "in progress" https://review.openstack.org/318832 | 18:00 |
| *** Apoorva_ has joined #openstack-nova | 18:00 | |
| *** ducttape_ has quit IRC | 18:01 | |
| *** Jabadia_ has quit IRC | 18:02 | |
| mriedem | oh b/c it uses ssh | 18:03 |
| *** Apoorva has quit IRC | 18:04 | |
| *** cheran has joined #openstack-nova | 18:04 | |
| openstackgerrit | Karen Bradshaw proposed openstack/nova: api-ref: os-consoles.inc https://review.openstack.org/317536 | 18:06 |
| *** notmorgan has quit IRC | 18:11 | |
| *** ducttape_ has joined #openstack-nova | 18:11 | |
| *** edand_ has quit IRC | 18:12 | |
| *** moshele has quit IRC | 18:13 | |
| *** doug-fis_ has joined #openstack-nova | 18:16 | |
| *** eharney has quit IRC | 18:18 | |
| dansmith | alaski: so, sdague always comments that checking that columns were added during a migration isn't really the goal of the migration testing framework | 18:18 |
| dansmith | and I agree, but I never know what to put there | 18:18 |
| *** doug-fish has quit IRC | 18:19 | |
| alaski | dansmith: I'm not really concerned with the columns, just that each table exists | 18:19 |
| dansmith | alaski: I guess the conditional tests are worth it, although they're kinda silly because they're never going to pass | 18:19 |
| dansmith | alaski: well, the column check was pretty much for that, unless there is a table helper, but I can add one per table I gess | 18:19 |
| dansmith | er, guess | 18:19 |
| *** ctrath has joined #openstack-nova | 18:19 | |
| dansmith | alaski: so you're thinking one per table and the conditional things like the indexes? | 18:20 |
| alaski | yep | 18:20 |
| *** ctrath has quit IRC | 18:20 | |
| alaski | there doesn't seem to be a table helper | 18:20 |
| alaski | but db_utils.get_table not blowing up would be fine | 18:21 |
| alaski | and at some point we should nail down what the point of migration testing is, since I can never remember | 18:21 |
| dansmith | alaski: well, one column assert per table would be easy right? | 18:21 |
| dansmith | alaski: well, when we did data migrations, that was the point :) | 18:22 |
| dansmith | data migrations in schema migrations I mean | 18:22 |
| alaski | dansmith: yeah, a column test would be just as fine | 18:22 |
| dansmith | okay | 18:22 |
| alaski | oh right, data migrations | 18:22 |
| dansmith | I kinda wrote it off as "this doesn't matter I just need something" | 18:22 |
| dansmith | but I was also rather annoyed at the time | 18:22 |
| *** eharney has joined #openstack-nova | 18:23 | |
| *** ctrath has joined #openstack-nova | 18:24 | |
| alaski | if we're going to keep these tests around some cursory testing would be nice. but I really like it for the indexes because the way they're defined always concerns me since there's no .create() call or anything | 18:24 |
| dansmith | so the reason why I don't think it's worth it, | 18:24 |
| dansmith | is that we have the model sync test, | 18:25 |
| *** Vek has quit IRC | 18:25 | |
| dansmith | which will fail if we didn't actually create things exactly like they're specified in the model | 18:25 |
| dansmith | so if we failed to hit the "if not db2" thing, we'd fail that other test | 18:25 |
| *** whenry has quit IRC | 18:25 | |
| alaski | true | 18:25 |
| dansmith | and that's a lot stronger than hand-poking at things | 18:25 |
| alaski | so maybe we should scrap these tests | 18:25 |
| alaski | at least for the api db | 18:25 |
| *** READ10 has quit IRC | 18:26 | |
| *** notmorgan has joined #openstack-nova | 18:26 | |
| dansmith | maybe | 18:26 |
| *** cfriesen has quit IRC | 18:26 | |
| *** dillaman has quit IRC | 18:26 | |
| *** lennyb has quit IRC | 18:26 | |
| *** russellb has quit IRC | 18:26 | |
| *** dosaboy has quit IRC | 18:26 | |
| *** anthonyper has quit IRC | 18:26 | |
| *** wanghao has quit IRC | 18:26 | |
| *** dmellado has quit IRC | 18:26 | |
| *** bradm has quit IRC | 18:26 | |
| *** j_king has quit IRC | 18:26 | |
| *** crinkle has quit IRC | 18:26 | |
| *** lifeless has quit IRC | 18:26 | |
| *** mfisch has quit IRC | 18:26 | |
| *** klindgren has quit IRC | 18:26 | |
| *** edleafe has quit IRC | 18:26 | |
| *** rmk has quit IRC | 18:26 | |
| *** j_king has joined #openstack-nova | 18:26 | |
| *** bradm has joined #openstack-nova | 18:26 | |
| *** cfriesen has joined #openstack-nova | 18:26 | |
| *** dosaboy has joined #openstack-nova | 18:26 | |
| *** lifeless has joined #openstack-nova | 18:26 | |
| *** klindgren has joined #openstack-nova | 18:26 | |
| *** wanghao has joined #openstack-nova | 18:26 | |
| *** dillaman has joined #openstack-nova | 18:26 | |
| *** russellb has joined #openstack-nova | 18:26 | |
| *** lennyb has joined #openstack-nova | 18:27 | |
| *** rmk has joined #openstack-nova | 18:27 | |
| *** mfisch has joined #openstack-nova | 18:27 | |
| *** mfisch is now known as Guest76268 | 18:27 | |
| *** dmellado has joined #openstack-nova | 18:27 | |
| *** gagehugo has quit IRC | 18:27 | |
| *** crinkle has joined #openstack-nova | 18:27 | |
| *** anthonyper has joined #openstack-nova | 18:27 | |
| alaski | looking at the api db tests I don't see anything that wouldn't be caught by the model sync test | 18:28 |
| dansmith | yeah because these are all new | 18:28 |
| *** zhenguo_ has quit IRC | 18:30 | |
| *** edleafe has joined #openstack-nova | 18:31 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 18:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute nodes update their own inventory records https://review.openstack.org/317049 | 18:31 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 18:31 |
| alaski | I'd be happy to scrap them or use them | 18:31 |
| *** esnyder has quit IRC | 18:31 | |
| *** kgalanov has quit IRC | 18:31 | |
| *** andrewbogott has quit IRC | 18:31 | |
| *** zhiyan has quit IRC | 18:31 | |
| *** vdrok has quit IRC | 18:31 | |
| dansmith | alaski: we still want the base test because it walks them I think, which the rest don't do | 18:32 |
| dansmith | alaski: but just the required check phase is not so useful.. we could maybe just allow people putting pass in there in case we come up with a thing we need to test that way | 18:32 |
| *** kgalanov has joined #openstack-nova | 18:33 | |
| *** esnyder has joined #openstack-nova | 18:33 | |
| *** zhenguo_ has joined #openstack-nova | 18:34 | |
| *** andrewbogott has joined #openstack-nova | 18:34 | |
| *** whenry has joined #openstack-nova | 18:34 | |
| *** vdrok has joined #openstack-nova | 18:34 | |
| *** zhiyan has joined #openstack-nova | 18:34 | |
| mriedem | dansmith: alaski: on the last keypair migration patch, i'm basically in the same boat with alaski, looks like just wanting an abort on the main db delete if the api db create fails https://review.openstack.org/#/c/313664/ | 18:35 |
| mriedem | i think once that patch is done we can mark the bp complete right? | 18:35 |
| alaski | dansmith: sure. if it's pass or a good test I'm happy, just don't want a middle ground | 18:35 |
| dansmith | mriedem: so the problem with that is, I think because of enginefacade, I can't open a transaction against both at the same time | 18:36 |
| dansmith | mriedem: maybe if I create another context to use to fake it or something... | 18:36 |
| dansmith | alaski: yeah, the fake test busywork is the worst case | 18:36 |
| mriedem | can you just recreate in the main db from the same values that were in the deleted record? | 18:36 |
| mriedem | hokey but it's an option | 18:36 |
| *** moshele has joined #openstack-nova | 18:37 | |
| dansmith | mriedem: that's really not a solution right? | 18:37 |
| dansmith | I mean, if the power goes out, you've deleted one and can't recreate it | 18:37 |
| mriedem | yeah | 18:37 |
| *** efoley has joined #openstack-nova | 18:38 | |
| *** MikeG451 has quit IRC | 18:38 | |
| dansmith | it solves half of the problem I guess, but doesn't seem worth it | 18:38 |
| mriedem | wait, what constraint is keeping you from creating in the api db first? | 18:38 |
| dansmith | I can try another fake context to fool it | 18:38 |
| mriedem | the check in the keypair objecT? | 18:38 |
| dansmith | mriedem: the object constraint of checking that the thing doesn't exist in the old db | 18:38 |
| dansmith | mriedem: so I can do a different type of create if you'd prefer that, I've just been trying to keep these to using the mainline interfaces for such things | 18:39 |
| alaski | you could bypass create and use _create_in_db, but that's hacky too | 18:39 |
| mriedem | use a fake context if tha tworks | 18:39 |
| *** tri2sing has joined #openstack-nova | 18:39 | |
| dansmith | alaski: yeah, don't want to do that if we can help it | 18:39 |
| mriedem | let me just say, | 18:39 |
| mriedem | enterprise NFV disaster recovery | 18:39 |
| dansmith | let me try a fake context | 18:40 |
| dansmith | I bet this isn't going to work | 18:42 |
| dansmith | because the transaction won't be committed, so when create goes to check it'll still look present, right? | 18:42 |
| *** baoli_ has quit IRC | 18:42 | |
| dansmith | alaski: mriedem ^ | 18:42 |
| *** sean-k-m1 is now known as sean-k-mooney | 18:42 | |
| *** jistr has quit IRC | 18:43 | |
| alaski | I'm not sure, but I think so | 18:43 |
| *** ihrachys has joined #openstack-nova | 18:43 | |
| dansmith | I'll just add a non-remotable wrapper for the meat | 18:43 |
| mriedem | idk, why isn't it possible to stack up transaction writers? aren't they just decorators? | 18:43 |
| dansmith | mriedem: your session goes in your context and you can only have one | 18:44 |
| dansmith | mriedem: so you need another context for the second db, which will be a different session | 18:44 |
| alaski | I feel like the best thing here would be to have a mechanism for creating in the api even if it exists in the cell db, then catch and ignore a dupe write in the migration method | 18:44 |
| dansmith | alaski: yeah, I'm just going to do that, it's not that hard.. | 18:45 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/318865 | 18:46 |
| *** rcernin has joined #openstack-nova | 18:46 | |
| *** ctrath has quit IRC | 18:48 | |
| *** lpetrut has joined #openstack-nova | 18:48 | |
| *** tri2sing has quit IRC | 18:49 | |
| *** smatzek has quit IRC | 18:49 | |
| *** ctrath has joined #openstack-nova | 18:50 | |
| *** andrewbogott has quit IRC | 18:51 | |
| *** andrewbogott has joined #openstack-nova | 18:51 | |
| jaypipes | syjulian: will do my best to get to that today, yes. | 18:51 |
| *** dimtruck is now known as zz_dimtruck | 18:51 | |
| *** jtp92 has left #openstack-nova | 18:52 | |
| dansmith | alaski: yeah, that was easy, not sure why I talked myself out of doing that in the first place | 18:52 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add online migration to move keypairs from main to API database https://review.openstack.org/313664 | 18:52 |
| alaski | because you hadn't been sufficiently annoyed by reviewers yet | 18:53 |
| *** whenry has quit IRC | 18:53 | |
| *** gszasz has quit IRC | 18:53 | |
| *** renatoarmani has joined #openstack-nova | 18:53 | |
| *** renatoarmani has quit IRC | 18:54 | |
| *** whenry has joined #openstack-nova | 18:54 | |
| dansmith | cdent: see my pointer response to your comment and tell me if I'm blind or confused | 18:54 |
| *** Sukhdev has joined #openstack-nova | 18:55 | |
| *** diogogmt has quit IRC | 18:55 | |
| cdent | dansmith: my understanding, which could well be wrong, was the the special casing only mattered for single column uniq constraint/index pairs | 18:55 |
| cdent | that with multi column constraints the situation was different | 18:56 |
| cdent | but I dunno, thus invoking mriedem | 18:56 |
| dansmith | cdent: hmm, okay | 18:56 |
| cdent | I hope he will enlighten us | 18:57 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add online migration to move keypairs from main to API database https://review.openstack.org/313664 | 18:57 |
| dansmith | I'm a bit annoyed that we have this in there anyway, since apparently 0% of people run DB2 | 18:57 |
| *** doug-fis_ has quit IRC | 18:58 | |
| cdent | yeah, it all seems a bit bogus | 18:58 |
| mriedem | can someone point me to the thing i need to look at? | 18:58 |
| cdent | mriedem: https://review.openstack.org/#/c/315681/9/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/016_resource_providers.py@69 | 18:59 |
| *** dillaman has quit IRC | 19:00 | |
| *** salv-orlando has quit IRC | 19:01 | |
| *** claudiub has quit IRC | 19:04 | |
| *** pushkaru has joined #openstack-nova | 19:04 | |
| *** MikeG451 has joined #openstack-nova | 19:04 | |
| syjulian | jaypipes: thanks jay | 19:04 |
| *** moshele has quit IRC | 19:04 | |
| mriedem | jaypipes: doesn't mysql also create an index over the columns in a unique constraint, but mysql allows duplicate indexes? | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: New configuration classes to parse device address element https://review.openstack.org/304147 | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: libvirt: attach configdrive after instance XML https://review.openstack.org/303335 | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: adding an update method to virtual_interface https://review.openstack.org/310618 | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging metadata API support https://review.openstack.org/304511 | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Device tagging API support https://review.openstack.org/304510 | 19:05 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Compute manager device tagging support https://review.openstack.org/264017 | 19:06 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: Libvirt driver implementation of device tagging https://review.openstack.org/264016 | 19:06 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: db: add a device_metadata column to instance_extra https://review.openstack.org/318883 | 19:06 |
| openstackgerrit | Vladik Romanovsky proposed openstack/nova: objects: new InstanceDevicesMetadata object https://review.openstack.org/318884 | 19:06 |
| *** zengine has joined #openstack-nova | 19:07 | |
| *** sdague has quit IRC | 19:08 | |
| *** zenoway has joined #openstack-nova | 19:09 | |
| *** liverpooler has joined #openstack-nova | 19:09 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 19:09 |
| openstackgerrit | Chris Dent proposed openstack/nova: Extend ResourceProvider to have resources and aggregates https://review.openstack.org/284963 | 19:09 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 19:09 |
| openstackgerrit | Chris Dent proposed openstack/nova: Correct the Inventory object https://review.openstack.org/315288 | 19:09 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 19:09 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 19:09 |
| *** doug-fish has joined #openstack-nova | 19:09 | |
| *** sdague has joined #openstack-nova | 19:09 | |
| *** raunak has quit IRC | 19:10 | |
| mriedem | dansmith: cdent: comments inline | 19:10 |
| sdague | mriedem: :) | 19:10 |
| *** salv-orlando has joined #openstack-nova | 19:10 | |
| *** diogogmt has joined #openstack-nova | 19:12 | |
| *** zenoway has quit IRC | 19:13 | |
| dansmith | mriedem: so I get that I should do that thing cdent said, but I'm not sure if the earlier thing you quoted implies a change | 19:14 |
| *** diogogmt has quit IRC | 19:15 | |
| mriedem | dansmith: on Index('inventories_resource_provider_resource_class_idx', ? | 19:15 |
| cdent | the thing described (either no name, or a unique name) is the desired behavior, as I understand it. | 19:15 |
| mriedem | oh | 19:16 |
| mriedem | ah the first thing | 19:16 |
| *** ijw has quit IRC | 19:16 | |
| dansmith | cdent: no, not that | 19:16 |
| cdent | mriedem: dunno, I'm talking about the first thing, not sure if dansmith is, as there are two things | 19:16 |
| dansmith | mriedem: the "this is the problem index" one | 19:16 |
| mriedem | yeah, so the only reason i bring that up is, when you create a UC in db2, if any of the columns are nullable, then it doesn't create a UC, it creates a unique index | 19:16 |
| mriedem | which is really only a problem later if you try to have a fkey on that column | 19:16 |
| mriedem | which for name we won't | 19:16 |
| dansmith | sigh | 19:17 |
| mriedem | it's a problem for fkeys because fkey requires the UC or primary key, which we wouldn't have on name | 19:17 |
| dansmith | I just want that thing we created in the other db to be moved to this db | 19:17 |
| mriedem | clarified the comment inline | 19:18 |
| dansmith | mriedem: you don't care about db2 anymore? | 19:18 |
| mriedem | dansmith: i don't | 19:18 |
| dansmith | why (the f) are we still dancing around this? | 19:18 |
| mriedem | like i said, feel free to remove any if ibm_db_sa checks in there | 19:18 |
| mriedem | and let it break | 19:18 |
| * dansmith 's head explodes | 19:18 | |
| sdague | mriedem: so we can remove db2 with fire? | 19:19 |
| mriedem | sure | 19:19 |
| * cdent fetches the champagne | 19:19 | |
| sdague | dansmith: do you want to write the patch or approve it? | 19:20 |
| dansmith | sdague: lol.. tough choice. | 19:20 |
| dansmith | sdague: I feel like someone somewhere would use me writing the patch as a reason not to vote me onto the lollipop committee or something | 19:20 |
| sdague | dansmith: you never run for the lollipop committee | 19:20 |
| dansmith | hah | 19:21 |
| dansmith | funny you knew what I was saying there | 19:21 |
| sdague | :) | 19:21 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 19:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute nodes update their own inventory records https://review.openstack.org/317049 | 19:22 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 19:22 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Deprecate barbican options https://review.openstack.org/317684 | 19:22 |
| *** cdent has left #openstack-nova | 19:22 | |
| *** cdent has joined #openstack-nova | 19:23 | |
| *** vladikr has quit IRC | 19:24 | |
| sdague | dansmith: I'm happy to write the patch | 19:24 |
| dansmith | sdague: I'm happy to +2 the ever-loving snot out of it | 19:24 |
| *** jwcroppe has quit IRC | 19:28 | |
| *** cdent has quit IRC | 19:28 | |
| *** ducttape_ has quit IRC | 19:30 | |
| *** jwcroppe has joined #openstack-nova | 19:30 | |
| *** Guest76268 is now known as mfisch | 19:30 | |
| *** baoli has joined #openstack-nova | 19:30 | |
| *** mfisch has quit IRC | 19:30 | |
| *** mfisch has joined #openstack-nova | 19:30 | |
| *** ducttape_ has joined #openstack-nova | 19:33 | |
| *** ducttape_ has joined #openstack-nova | 19:33 | |
| *** rama_y has quit IRC | 19:36 | |
| *** diogogmt has joined #openstack-nova | 19:37 | |
| *** BjoernT has joined #openstack-nova | 19:40 | |
| openstackgerrit | Sean Dague proposed openstack/nova: remove db2 support from tree https://review.openstack.org/318906 | 19:40 |
| sdague | that rewrites migrations... so it should be looked at carefully to make sure I didn't screw up an if block | 19:40 |
| *** Apoorva_ has quit IRC | 19:43 | |
| stewie925 | would it be normal for an instance (flavor = micro) to spawn for > 5 minutes on a cell environment? | 19:43 |
| dansmith | stewie925: yes, wait at least 60 minutes | 19:44 |
| stewie925 | hey dan thanks! | 19:44 |
| stewie925 | whoops 60 minutes | 19:44 |
| dansmith | stewie925: and in case we haven't been introduced, I'm the one that says all the overly sarcastic things | 19:44 |
| stewie925 | lol I'll make note of that | 19:45 |
| stewie925 | hope it doesnt take that long lol | 19:46 |
| *** jamesdenton has joined #openstack-nova | 19:46 | |
| stewie925 | crap - it errored out ugh | 19:47 |
| stewie925 | "failed to allocate network, not rescheduling " :( | 19:48 |
| auggy | sdague: i think i'm a little confused about https://review.openstack.org/#/c/315216/ | 19:48 |
| auggy | i saw a code path for the additional limits parameters i documented, but maybe that's only for an older version? | 19:48 |
| sdague | auggy: got the reference? | 19:50 |
| auggy | sdague: yeah let me find it | 19:50 |
| auggy | sdague: also i had found tests that expected those params in the response | 19:51 |
| *** nic has quit IRC | 19:51 | |
| *** zenoway has joined #openstack-nova | 19:52 | |
| *** rama_y has joined #openstack-nova | 19:52 | |
| mriedem | dansmith: why do we have SERVICE_VERSION = 11 but SERVICE_VERSION_HISTORY is at 4.12? | 19:53 |
| mriedem | or that's unrelated? | 19:53 |
| mriedem | i guess 4.12 is compute rpc api version | 19:53 |
| dansmith | right | 19:53 |
| *** pushkaru has quit IRC | 19:53 | |
| mriedem | ok, so me checking in the api, i just need to update SERVICE_VERSION | 19:53 |
| mriedem | and don't care about SERVICE_VERSION | 19:53 |
| *** efoley has quit IRC | 19:53 | |
| mriedem | *SERVICE_VERSION_HISTORY | 19:53 |
| mriedem | for get me a network i mean | 19:54 |
| auggy | sdague: ok so i'll double double check that this is getting called - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/limits.py | 19:54 |
| *** ducttape_ has quit IRC | 19:54 | |
| dansmith | you need another entry in there, but it will be the same as the current verson | 19:54 |
| sdague | auggy: that doesn't have the rate limit keys in it | 19:54 |
| mriedem | dansmith: oh i see | 19:55 |
| mriedem | list of dicts, with duplicates | 19:55 |
| *** diogogmt has quit IRC | 19:55 | |
| sdague | auggy: oh, sorry, I see | 19:55 |
| auggy | sdague: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/limits.py#L68 | 19:55 |
| auggy | that might not actually be getting called by anything tho, so i should double check that | 19:56 |
| auggy | but i traced that and saw the unit test coverage and assumed it was accurate | 19:56 |
| auggy | "traced" = just followed a trail manually | 19:56 |
| *** ducttape_ has joined #openstack-nova | 19:57 | |
| sdague | auggy: https://github.com/openstack/nova/blob/d75fb4b0e457317054e20e1f493c2730aa172a2d/nova/api/openstack/compute/limits.py#L39 | 19:57 |
| auggy | if it's not actually getting called or being used then i can submit a change to remove it | 19:57 |
| auggy | sdague: right, that confused me a little | 19:57 |
| sdague | so, I think it's vestigial | 19:57 |
| auggy | gah | 19:57 |
| auggy | i thought maybe whatever created that nova.limits data was using this view | 19:58 |
| sdague | https://github.com/openstack/nova/blob/d75fb4b0e457317054e20e1f493c2730aa172a2d/nova/api/openstack/compute/legacy_v2/limits.py#L204 | 19:59 |
| sdague | it's only on the legacy_v2 stack | 19:59 |
| bauzas | mriedem: you'll be in vacation for the Newton-1 milestone, right? | 19:59 |
| sdague | which is getting deleted... soon | 19:59 |
| mriedem | bauzas: i'm back to work on 5/31 | 19:59 |
| mriedem | bauzas: n-1 is 6/2 | 19:59 |
| bauzas | mriedem: okay, cool then | 19:59 |
| mriedem | so i'll be back right before it | 19:59 |
| auggy | sdague: yarrgh ok yeah | 19:59 |
| auggy | man i can't believe i missed that | 19:59 |
| *** nic has joined #openstack-nova | 19:59 | |
| sdague | auggy: so, just say rate is an array, which will be empty | 19:59 |
| auggy | sdague: ok, i will also file doc bugs for places where i found other documentation | 20:00 |
| sdague | and the rest of those params after it we'll never speak of :) | 20:00 |
| bauzas | mriedem: no problem, was asking because of the release change for the milestone | 20:00 |
| auggy | sdague: haha wfm | 20:00 |
| mriedem | bauzas: sdague is covering it | 20:00 |
| auggy | sdague: should i file a bug or just start a change to remove that crud | 20:00 |
| sdague | well, the code is getting removed | 20:01 |
| auggy | sdague: or is that something that should just wait until the v2 effort is done? i just don't want to forget about it | 20:01 |
| auggy | ah ok | 20:01 |
| sdague | yeh, it should just go with the v2 effort | 20:01 |
| auggy | sdague: ok i'll see what is on the list and maybe see if i can help with that | 20:01 |
| auggy | sdague: with removing that bit since i was just in it | 20:01 |
| sdague | I guess we could patch the v2 stack to never even look at the env variable, let me do that quick | 20:01 |
| bauzas | mriedem: okay, I asked because I did the patch for Mitaka, but I'm fine with sdague doing it | 20:01 |
| bauzas | I'll review the reno notes btw. | 20:01 |
| *** gyee has joined #openstack-nova | 20:02 | |
| mriedem | bauzas: thanks | 20:03 |
| auggy | sdague: thanks, that definitely is what sent me down this path ;) | 20:03 |
| *** diogogmt has joined #openstack-nova | 20:04 | |
| *** Apoorva has joined #openstack-nova | 20:05 | |
| *** gjayavelu has joined #openstack-nova | 20:05 | |
| *** Sukhdev has quit IRC | 20:06 | |
| openstackgerrit | Sean Dague proposed openstack/nova: be more explicit that rate limits are gone in v2.1 https://review.openstack.org/318923 | 20:07 |
| sdague | auggy: how does that look? | 20:07 |
| *** sacharya_ has joined #openstack-nova | 20:08 | |
| auggy | sdague: a++ would +1 again | 20:08 |
| *** diogogmt has quit IRC | 20:08 | |
| auggy | although i'm just waiting for jenkins to do it's thang | 20:08 |
| *** Apoorva has quit IRC | 20:09 | |
| *** diogogmt has joined #openstack-nova | 20:09 | |
| auggy | sdague: i'm working on an update to my limits params patch and should have examples submitted shortly as well | 20:10 |
| openstackgerrit | jichenjc proposed openstack/nova: verify api-ref metadata.inc https://review.openstack.org/318600 | 20:10 |
| *** sachary__ has joined #openstack-nova | 20:11 | |
| *** sacharya has quit IRC | 20:11 | |
| *** pushkaru has joined #openstack-nova | 20:11 | |
| *** jamesdenton has quit IRC | 20:12 | |
| *** ducttape_ has quit IRC | 20:13 | |
| *** matrohon has joined #openstack-nova | 20:13 | |
| *** sacharya_ has quit IRC | 20:15 | |
| *** doug-fish has quit IRC | 20:15 | |
| *** markvoelker has joined #openstack-nova | 20:15 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: API change for verifying the scheduler when live migrating https://review.openstack.org/318334 | 20:16 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: API change for verifying the scheduler when evacuating https://review.openstack.org/315572 | 20:16 |
| bauzas | *cough cough* ladies and gentlemen, here comes the last changes for my non-priority blueprint | 20:16 |
| *** ducttape_ has joined #openstack-nova | 20:16 | |
| bauzas | jaypipes: dansmith: alaski: fasten your seatbelts, I'll now review your changes and nitpick them :p | 20:16 |
| dansmith | bauzas: your change yesterday put all mine in merge conflict, so don't expect kindness :P | 20:17 |
| bauzas | hehe | 20:17 |
| * alaski keeps his hands and feet inside the vehicle while it is in motion | 20:17 | |
| *** ijw has joined #openstack-nova | 20:18 | |
| bauzas | given now my non-prio BP is implemented before N-1, I can now hit the other changes with merge conflicts by implementing some priority specs :D (devil voice) | 20:18 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: api-ref: limits.inc validate parameters https://review.openstack.org/315216 | 20:18 |
| bauzas | dansmith: jaypipes: alaski: so, what should I review first ? the inventories changes, the API DB migrations or the cellsv2 scheduler changes ? | 20:19 |
| bauzas | I mean, for the sched prio | 20:20 |
| *** markvoelker has quit IRC | 20:20 | |
| alaski | for scheduler priority stuff don't worry about cells changes yet | 20:20 |
| openstackgerrit | Tom Patzig proposed openstack/nova-specs: Add "disable local disks" option to flavors https://review.openstack.org/267673 | 20:20 |
| bauzas | okay, I'll begin with biting dansmith's https://review.openstack.org/#/c/315681/ | 20:21 |
| *** raunak has joined #openstack-nova | 20:21 | |
| *** jichen has quit IRC | 20:21 | |
| dansmith | bauzas: I broke it just now, putting a fix up in a sec | 20:21 |
| bauzas | okay | 20:21 |
| bauzas | dansmith: which one did I wedged it ? | 20:22 |
| dansmith | _I_ broke it | 20:22 |
| bauzas | I mean, the merge conflict I made | 20:22 |
| dansmith | the db migration | 20:22 |
| bauzas | k | 20:22 |
| *** akshai has quit IRC | 20:22 | |
| *** Apoorva has joined #openstack-nova | 20:23 | |
| *** ayoung has quit IRC | 20:24 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config Options: Improve help text for Ipv6 options https://review.openstack.org/318291 | 20:24 |
| *** diogogmt has quit IRC | 20:25 | |
| openstackgerrit | Hans Lindgren proposed openstack/nova: Move some flavor fakes closer to where they are being used https://review.openstack.org/318934 | 20:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 20:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute nodes update their own inventory records https://review.openstack.org/317049 | 20:25 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 20:26 |
| *** flwang1 has quit IRC | 20:26 | |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: api-ref: limits.inc validate parameters https://review.openstack.org/315216 | 20:26 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config Options: Improve help text for debugger https://review.openstack.org/311822 | 20:27 |
| *** clenimar has joined #openstack-nova | 20:28 | |
| *** diogogmt has joined #openstack-nova | 20:29 | |
| mriedem | the keystone project id validation spec should be good to go if another spec reviewer wants to take a look https://review.openstack.org/#/c/294337/ | 20:31 |
| *** Sukhdev has joined #openstack-nova | 20:32 | |
| *** daemontool has joined #openstack-nova | 20:32 | |
| dansmith | I shall cast my eyes across it | 20:33 |
| dansmith | mriedem: there are two hyper-v patches that have +2 on them that have been waiting since before image meta was an object | 20:33 |
| *** cleong has quit IRC | 20:33 | |
| auggy | thanks mriedem | 20:33 |
| dansmith | mriedem: https://review.openstack.org/#/c/184039/28 | 20:33 |
| auggy | mriedem: i did see the formatting change but i was too much of a lazy ass to fix it | 20:34 |
| *** moshele has joined #openstack-nova | 20:34 | |
| dansmith | mriedem: the third needs a tweak since the imagemeta serial attribute was codified, but the first two can belanded | 20:34 |
| auggy | but if others feel it's a big enough deal i can fix it ;) | 20:34 |
| *** Vek has joined #openstack-nova | 20:34 | |
| mriedem | dansmith: there is probably no ci testing on the serial console one right? | 20:34 |
| mriedem | i don't think tempest does serial console testing | 20:34 |
| dansmith | mriedem: we don't have it on libvirt either.. it'd be a little rough expecting them to connect to a tcp socket and validate something I think | 20:35 |
| dansmith | certainly doable, but not something we do elsewhere that I know of | 20:35 |
| clarkb | mriedem: tempest does dump the nova console-log output on failures not sure if it does so as a proper test | 20:35 |
| dansmith | clarkb: we're talking about bidirectional serial console | 20:35 |
| clarkb | ah | 20:35 |
| dansmith | mriedem: either way, the two that are ready are more hyperv plumbing for the third one that needs a tweak, which actually exposes it | 20:36 |
| *** moshele has quit IRC | 20:36 | |
| *** mdrabe has quit IRC | 20:37 | |
| *** DaiDV has quit IRC | 20:37 | |
| *** doug-fish has joined #openstack-nova | 20:38 | |
| *** dmk0202 has joined #openstack-nova | 20:39 | |
| *** hemanthm is now known as hemanthm|afk | 20:40 | |
| *** rdopiera has quit IRC | 20:40 | |
| *** whenry has quit IRC | 20:41 | |
| *** rdopiera has joined #openstack-nova | 20:42 | |
| *** doug-fish has quit IRC | 20:43 | |
| *** fragatina has quit IRC | 20:43 | |
| *** doug-fish has joined #openstack-nova | 20:43 | |
| dansmith | waaaaaiit a sec | 20:43 |
| dansmith | can we really allow that spec to go in as authored by someone @cpan.org? | 20:44 |
| dansmith | just seems like bad form | 20:44 |
| auggy | dansmith: ha | 20:44 |
| * auggy is a little bit of a troll | 20:44 | |
| *** nic has quit IRC | 20:47 | |
| *** salv-orl_ has joined #openstack-nova | 20:48 | |
| *** zzelle_ has quit IRC | 20:49 | |
| *** baoli has quit IRC | 20:50 | |
| *** rdopiera has quit IRC | 20:50 | |
| *** rdopiera has joined #openstack-nova | 20:51 | |
| *** tonytan4ever has quit IRC | 20:51 | |
| *** salv-orlando has quit IRC | 20:51 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Skip network allocation if 'none' is requested https://review.openstack.org/309638 | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: nova-net: handle 'auto' network request in allocate_for_instance https://review.openstack.org/297906 | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Skip network validation if explicitly requesting no networks https://review.openstack.org/309602 | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: handle 'auto' network request in allocate_for_instance https://review.openstack.org/316275 | 20:52 |
| *** ivase has quit IRC | 20:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add helpers to NetworkRequest(List) objects for auto/none cases https://review.openstack.org/297871 | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Add REST API support for get me a network https://review.openstack.org/316398 | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: neutron: validate auto-allocate is available https://review.openstack.org/309574 | 20:52 |
| *** zz_dimtruck is now known as dimtruck | 20:57 | |
| *** ivase has joined #openstack-nova | 20:58 | |
| *** ivase_ has joined #openstack-nova | 20:59 | |
| *** AmyLee71 has joined #openstack-nova | 21:00 | |
| *** mmandel has joined #openstack-nova | 21:01 | |
| *** ivase__ has joined #openstack-nova | 21:01 | |
| *** AmyLee7 has quit IRC | 21:01 | |
| *** zengine has quit IRC | 21:02 | |
| *** ivase has quit IRC | 21:02 | |
| mriedem | dansmith: opinion on this - if you request network='none' (i don't want networking on my instance) and the computes aren't new enough, fail the request? or change to None and let the mitaka computes handle it as they do today, which is you get networking if you don't specify anything and a network is available to use, else you don't get networking. | 21:02 |
| mriedem | i'm thinking the former since you specifically request 'none' and we can't honor it | 21:03 |
| mriedem | at least with 'auto' we can try | 21:03 |
| bauzas | jaypipes: dansmith: I'm not an i18n expert, but is it cool to see an Unicode field in a latin-1 charset table? | 21:03 |
| dansmith | mriedem: I thought yesterday you said keep the default in both cases, which might be different for a while during a migration | 21:03 |
| *** ivase_ has quit IRC | 21:03 | |
| *** smatzek has joined #openstack-nova | 21:03 | |
| *** fragatina has joined #openstack-nova | 21:03 | |
| mriedem | keeping the default in both cases is easier to code, sure | 21:03 |
| mriedem | one less condition | 21:03 |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Stop using mox in test_firewall https://review.openstack.org/306536 | 21:04 |
| mriedem | but we can just say, nope, sorry, can't honor this | 21:04 |
| bauzas | dansmith: jaypipes: that looks possible (both are 8 bits) that said | 21:04 |
| dansmith | mriedem: I dunno, every time I try to think pedantically about this, someone reminds me this is a pretty silly defaults case, so I dunno | 21:04 |
| *** lindycod_ has quit IRC | 21:04 | |
| dansmith | bauzas: unicode is not 8 bits | 21:04 |
| mriedem | yeah, you'd have to request a lower microversion to get around it | 21:04 |
| dansmith | mriedem: so I dunno, I'm inclined to say as long as neither will be too surprising or problematic for the user, do the simpler/easier thing | 21:05 |
| *** ivase__ has quit IRC | 21:05 | |
| *** rtheis has quit IRC | 21:06 | |
| bauzas | dansmith: well, unicode is not an encoding, so yeah it depends | 21:06 |
| bauzas | but I'm probably nitpicking | 21:06 |
| *** ivase has joined #openstack-nova | 21:07 | |
| bauzas | because that's already in the cell DB | 21:07 |
| dansmith | bauzas: is this for my api patch? | 21:07 |
| bauzas | yup | 21:07 |
| dansmith | what table are you concerned about? | 21:07 |
| bauzas | ResourceProviders | 21:07 |
| dansmith | I know that cdent intentionally made some of those fields latin1 | 21:07 |
| bauzas | but like I said, that's already something done in the cell db | 21:07 |
| dansmith | well, don't avoid being picky for something like this, but I think he probably made certain decisions as that was all done recently, and I remember a charset thing | 21:08 |
| *** tpepper1 has joined #openstack-nova | 21:08 | |
| bauzas | just looking at the 318 DB migration | 21:09 |
| *** raildo is now known as raildo-afk | 21:10 | |
| dansmith | might look at the review that added that migration | 21:10 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/nova: WIP Don't check cinder volume states during attach https://review.openstack.org/315789 | 21:10 |
| bauzas | right | 21:10 |
| openstackgerrit | Augustina Ragwitz proposed openstack/nova: WIP: limits.inc example verification https://review.openstack.org/315714 | 21:11 |
| *** ivase has quit IRC | 21:11 | |
| *** yamahata has quit IRC | 21:11 | |
| jaypipes | bauzas: latin-1 is a subset of utf8, no? | 21:11 |
| clarkb | jaypipes: nope | 21:11 |
| dansmith | o.O | 21:11 |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs: Get Me a Network https://review.openstack.org/283206 | 21:11 |
| clarkb | ascii is though, but the non ascii bits of latin-1 don't overlap with utf8 iirc | 21:11 |
| *** smatzek has quit IRC | 21:11 | |
| clarkb | because they do differen things with that high bit | 21:11 |
| bauzas | utf8 is an encoding thing, AFAIK | 21:11 |
| bauzas | latin1 is a charset | 21:11 |
| jaypipes | clarkb: ah, duh, yeah, was mixing ascii and latin-1. thx | 21:11 |
| bauzas | which are 2 different things | 21:12 |
| clarkb | bauzas: latin-1 is an encoding | 21:12 |
| bauzas | AFAIR, utf8 is how we encode the list of chars | 21:12 |
| bauzas | graaah | 21:12 |
| jaypipes | bauzas: anyway, what is this in regards to? :) | 21:12 |
| dansmith | I have to run off for a bit, so if I have to change latin-1 to latin-2 or something, just leave a comment on the review | 21:13 |
| * dansmith hopes there is no latin-2, making that joke hilarious | 21:13 | |
| bauzas | jaypipes: I was a bit surprised to see an unicode mixed with latin-1, but like I said, it wasn't worth mentioning it yet in the review | 21:13 |
| bauzas | I need to load in mind a few bits before commenting | 21:14 |
| clarkb | 0xFF in latin-1 is ÿ but is invalid in utf8 iirc | 21:14 |
| bauzas | anyway, wikipedia seems perfectly fine with that https://en.wikipedia.org/wiki/Latin-1_Supplement_%28Unicode_block%29 | 21:15 |
| *** ihrachys has quit IRC | 21:16 | |
| openstackgerrit | Diana Clarke proposed openstack/nova: Image: add create_from_image & create_from_func https://review.openstack.org/318948 | 21:16 |
| clarkb | ya unicode and latin-1 do overlap I think. Just pointing out that latin-1 and utf8 do not | 21:16 |
| bauzas | I see | 21:17 |
| *** ihrachys has joined #openstack-nova | 21:17 | |
| bauzas | because unicode is a charset for noting all the chars we have, right? | 21:18 |
| bauzas | and then using an encoding that can be either latin-1 or utf-8, right? | 21:18 |
| clarkb | right | 21:19 |
| bauzas | okay | 21:19 |
| openstackgerrit | Diana Clarke proposed openstack/nova: Image: add create_from_image & create_from_func https://review.openstack.org/318948 | 21:19 |
| * bauzas loves OpenStack for that precise reason that I can learn things every day | 21:19 | |
| *** tpepper1 has quit IRC | 21:19 | |
| *** nic has joined #openstack-nova | 21:20 | |
| *** dmk0202 has quit IRC | 21:22 | |
| *** chip__ has quit IRC | 21:23 | |
| *** thorst_ has quit IRC | 21:23 | |
| jaypipes | bauzas: I offer you a 💩 | 21:23 |
| openstackgerrit | Clenimar Filemon proposed openstack/nova: Use Identity v3 params when connecting to Ironic https://review.openstack.org/300154 | 21:23 |
| bauzas | heh | 21:24 |
| melwitt | according to stackoverflow latin-1 is single-byte encoding that can represent the first 256 unicode characters http://stackoverflow.com/questions/7048745/what-is-the-difference-between-utf-8-and-iso-8859-1 | 21:24 |
| *** chip__ has joined #openstack-nova | 21:24 | |
| *** flwang1 has joined #openstack-nova | 21:24 | |
| *** matrohon has quit IRC | 21:25 | |
| stewie925 | hello melwitt - I have posted the error logs here - http://paste.openstack.org/show/497808/ | 21:25 |
| *** thorst_ has joined #openstack-nova | 21:25 | |
| stewie925 | had to re-create instance using public network nic | 21:26 |
| *** pushkaru has quit IRC | 21:27 | |
| stewie925 | but failing with "Failed to allocate the network(s), not rescheduling" | 21:27 |
| *** syjulian has quit IRC | 21:27 | |
| stewie925 | sorry I mean "had to create instance with PRIVATE nic" | 21:28 |
| *** thorst_ has quit IRC | 21:29 | |
| *** ctrath has quit IRC | 21:29 | |
| melwitt | stewie925: you have more than one network in 'nova network-list'? | 21:31 |
| stewie925 | yes I do | 21:31 |
| stewie925 | a public and a private - I think thats how it was created when I ran stack.sh | 21:31 |
| auggy | dansmith: there is latin-2 - http://www.ascii-codes.com/cp852.html | 21:32 |
| openstackgerrit | Tom Patzig proposed openstack/nova: Add ironic 'api_insecure' option to nova.conf https://review.openstack.org/311163 | 21:33 |
| *** eharney has quit IRC | 21:33 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova: Deprecate barbican options https://review.openstack.org/317684 | 21:34 |
| lifeless | alaski: cool | 21:34 |
| stewie925 | melwitt: I did a grep on the instance-id and there are only entries in n-api, n-cell-region, n-cell-child, and n-cpu (nothing in n-sch or n-cond) | 21:34 |
| *** imcsk8 has quit IRC | 21:34 | |
| *** ducttape_ has quit IRC | 21:35 | |
| melwitt | stewie925: hm. devstack only creates one network AFAIK, as seen in this log from a cells tempest job run http://logs.openstack.org/85/317785/1/check/gate-tempest-dsvm-cells/f8302ed/logs/devstacklog.txt.gz | 21:35 |
| *** baoli has joined #openstack-nova | 21:36 | |
| stewie925 | melwitt, maybe I can just delete one of the networks, if that may be the cause of the issue? | 21:36 |
| alaski | lifeless: if anything else comes up let me know | 21:36 |
| melwitt | stewie925: so it works when you pass the 'private' network with --nic? usually I don't specify the --nic option and it just picks by itself | 21:36 |
| stewie925 | melwitt: I initially did not specify the --nic, but nova errored and asked me to specify it - maybe because it finds there are two networks | 21:37 |
| *** imcsk8 has joined #openstack-nova | 21:37 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP: Add REST API support for get me a network https://review.openstack.org/316398 | 21:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Bump the service version for get-me-a-network support https://review.openstack.org/318953 | 21:37 |
| *** ctrath has joined #openstack-nova | 21:37 | |
| melwitt | stewie925: oh, okay. does instance boot work when you use the 'private' network? | 21:37 |
| *** doug-fish has quit IRC | 21:38 | |
| stewie925 | the error I have now is when I use the private ntetwork, actually | 21:38 |
| melwitt | stewie925: also in the error case, I would expect to find errors in n-net | 21:38 |
| *** ctrath has quit IRC | 21:38 | |
| stewie925 | ohhh - well i did not see a n-net in the list of services when I brought up devstacks screen | 21:39 |
| *** sdake has quit IRC | 21:39 | |
| lifeless | alaski: will do, I've poked another fixtures core for a review | 21:40 |
| * bauzas head explodes with charsets vs. encodings | 21:40 | |
| melwitt | stewie925: that's probably why it's failing then. what's the status of 'nova-network' when you do 'nova service-list' | 21:40 |
| stewie925 | checking - brb | 21:40 |
| melwitt | bauzas: it's pretty new to me. learned from reading up based on your question | 21:40 |
| stewie925 | melwitt: argh I see nove-compute is down | 21:41 |
| bauzas | melwitt: I'm coming from the Web world, where you usually do unicode with utf-8 and basta. | 21:41 |
| stewie925 | and nova-network is not listed | 21:41 |
| *** efried has quit IRC | 21:41 | |
| *** efried has joined #openstack-nova | 21:42 | |
| jaypipes | bauzas, melwitt, dansmith: https://review.openstack.org/#/c/316031/ should be ready to rock and roll. fixes an issue found in Ironic with new resource providers queries... | 21:42 |
| stewie925 | maybe I should re-stack, then after successful stack I will check 'nova service-list' ? | 21:42 |
| melwitt | stewie925: nova-network should be down too if it's not running in your screen session. oh, that's a bit unhelpful, not running but no entry to say it's 'down' | 21:42 |
| stewie925 | let me see the local.conf to see if enable_service n-net is present | 21:42 |
| bauzas | jaypipes: well, the patch uses the DB cell models, right? | 21:42 |
| melwitt | stewie925: you shouldn't have to name it explicitly, it should do it by default | 21:43 |
| bauzas | jaypipes: which is a bit orthogonal to dansmith's change | 21:43 |
| bauzas | jaypipes: I mean https://review.openstack.org/#/c/316031/5/nova/objects/resource_provider.py@205 | 21:43 |
| mriedem | alaski: dansmith: sdague: thread in the ops list about hitting the default 1000 limit for listing instances at scale, kind of interesting some of the views on that | 21:43 |
| mriedem | e.g. the one guy that just runs a direct query against the db | 21:44 |
| stewie925 | uh oh | 21:44 |
| *** doug-fish has joined #openstack-nova | 21:44 | |
| stewie925 | guess what melwitt - n-net was disabled | 21:44 |
| melwitt | bauzas: yeah. since utf-8 can represent the complete unicode set you need to use it if you want the thing to be multilingual. I'm not knowledgeable about when to choose latin-1 | 21:44 |
| stewie925 | I think that was how the local.conf was originally set up | 21:44 |
| stewie925 | let me remove and rerun stack | 21:44 |
| melwitt | stewie925: oh geez, that's not good | 21:45 |
| *** thorst_ has joined #openstack-nova | 21:45 | |
| stewie925 | yeah ... strange | 21:45 |
| bauzas | melwitt: well, I know that answer | 21:45 |
| stewie925 | right now I have two VMs - one with the regular devstack, and 2nd is this VM with devstack 'cell' | 21:45 |
| bauzas | melwitt: utf8 can be very greedy | 21:45 |
| *** lakshmiS has quit IRC | 21:45 | |
| bauzas | melwitt: in terms of bytes per char | 21:45 |
| stewie925 | brb, checking the original devstack (no cell) VM - I know I was able to create instance successfully there | 21:46 |
| melwitt | bauzas: yeah, so I guess if you know you can live with only the first 256, use the latin-1 to be efficient | 21:46 |
| bauzas | melwitt: that's why you usually prefer a certain other encoding if you don't care about supporting all languages | 21:46 |
| bauzas | melwitt: the problem is the last 127 | 21:46 |
| mriedem | alright, i'm out of here. dan and sean and probably others have my personal email if people need to get a hold of me while on vacation. otherwise see you all on the 31st. | 21:46 |
| bauzas | mriedem: \o | 21:46 |
| bauzas | mriedem: Europe, right? | 21:47 |
| mriedem | yes | 21:47 |
| melwitt | mriedem: o/ bon voyage | 21:47 |
| bauzas | k,enjoy your trip | 21:47 |
| bauzas | safe travels too | 21:47 |
| *** mdrabe has joined #openstack-nova | 21:47 | |
| mriedem | Děkuji! | 21:47 |
| *** mriedem has quit IRC | 21:47 | |
| bauzas | a-ha, Praha... one of the locations I'd love to visit | 21:48 |
| bauzas | well, Prague | 21:48 |
| *** doug-fish has quit IRC | 21:49 | |
| *** thorst_ has quit IRC | 21:49 | |
| *** diogogmt has quit IRC | 21:50 | |
| *** doug-fish has joined #openstack-nova | 21:50 | |
| *** kfarr has quit IRC | 21:52 | |
| *** Sukhdev has quit IRC | 21:53 | |
| *** MikeG451 has quit IRC | 21:54 | |
| *** AmyLee71 has quit IRC | 21:54 | |
| *** doug-fish has quit IRC | 21:54 | |
| *** Sukhdev has joined #openstack-nova | 21:56 | |
| *** rlrossit_ has quit IRC | 21:56 | |
| *** daemontool has quit IRC | 21:57 | |
| dansmith | auggy: blast! | 21:58 |
| *** Swami has quit IRC | 22:01 | |
| openstackgerrit | Robert Ellis proposed openstack/nova: Changed 202 for lock/unlock return code to 204. https://review.openstack.org/314267 | 22:01 |
| *** ducttape_ has joined #openstack-nova | 22:02 | |
| openstackgerrit | Carl Baldwin proposed openstack/nova-specs: Neutron Routed Networks https://review.openstack.org/263898 | 22:03 |
| stewie925 | hey melwitt - before i re-stack with local.conf's 'disable n-net' removed | 22:03 |
| stewie925 | should I just leave the nova-cells.conf and nova.conf alone | 22:04 |
| melwitt | stewie925: yeah | 22:04 |
| stewie925 | thank you | 22:05 |
| *** edmondsw has quit IRC | 22:06 | |
| *** AmyLee7 has joined #openstack-nova | 22:06 | |
| *** ayoung has joined #openstack-nova | 22:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/318865 | 22:13 |
| *** ducttape_ has quit IRC | 22:16 | |
| *** gjayavelu has quit IRC | 22:16 | |
| *** diogogmt has joined #openstack-nova | 22:19 | |
| *** nic has quit IRC | 22:21 | |
| *** rcernin has quit IRC | 22:23 | |
| *** tbachman has joined #openstack-nova | 22:24 | |
| *** mdrabe has quit IRC | 22:24 | |
| *** jaypipes has quit IRC | 22:25 | |
| *** zenoway has quit IRC | 22:25 | |
| *** zenoway has joined #openstack-nova | 22:26 | |
| *** markvoelker has joined #openstack-nova | 22:27 | |
| *** markvoelker has quit IRC | 22:29 | |
| *** markvoelker has joined #openstack-nova | 22:29 | |
| *** ducttape_ has joined #openstack-nova | 22:29 | |
| *** sdake has joined #openstack-nova | 22:30 | |
| *** zenoway has quit IRC | 22:30 | |
| *** kylek3h has quit IRC | 22:32 | |
| dansmith | bauzas: I'm not sure what you're asking me to do | 22:35 |
| dansmith | bauzas: are you saying I have to create the table and then apply that special ddl execution to change the definition? | 22:35 |
| *** mmandel has quit IRC | 22:35 | |
| *** lpetrut has quit IRC | 22:37 | |
| dansmith | surely there is some way to just specify the charset on the column in the table itself? | 22:38 |
| *** sdague has quit IRC | 22:39 | |
| *** ducttape_ has quit IRC | 22:40 | |
| openstackgerrit | bailin.zhang proposed openstack/nova: Create VM failed, unplug vifs https://review.openstack.org/306219 | 22:40 |
| *** tbachman_ has joined #openstack-nova | 22:41 | |
| *** ducttape_ has joined #openstack-nova | 22:42 | |
| *** tbachman has quit IRC | 22:42 | |
| *** tbachman_ is now known as tbachman | 22:42 | |
| *** tbachman has quit IRC | 22:44 | |
| *** doug-fish has joined #openstack-nova | 22:44 | |
| *** doug-fish has quit IRC | 22:44 | |
| *** sdake has quit IRC | 22:45 | |
| stewie925 | melwitt: very strange - a few nova services are down | 22:47 |
| *** tbachman has joined #openstack-nova | 22:47 | |
| stewie925 | nova-conductor, nova-network, and nova-compute | 22:47 |
| stewie925 | when I reran stack and it was successful (15 min ago), I checked and all nova services were up | 22:48 |
| *** fragatin_ has joined #openstack-nova | 22:49 | |
| stewie925 | ok.. all services are back up now except nova-compute | 22:50 |
| openstackgerrit | Dan Smith proposed openstack/nova: Add resource provider tables to the api database https://review.openstack.org/315681 | 22:51 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make compute nodes update their own inventory records https://review.openstack.org/317049 | 22:51 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make Inventory and ResourceProvider objects use the API DB instead https://review.openstack.org/315682 | 22:51 |
| melwitt | stewie925: when you're going to restack you should unstack.sh first and also make sure no screen session is still running, kill service processes if any are still running, etc before stack.sh again | 22:51 |
| melwitt | stewie925: there should be an error in n-cpu.log if it's not up | 22:51 |
| *** fragatina has quit IRC | 22:52 | |
| stewie925 | melwitt: I actually ran from a snapshot earlier - then I had to redo everything | 22:52 |
| melwitt | stewie925: or if it failed to start altogether, stack.sh should have shown an error and exited non-zero | 22:52 |
| stewie925 | I see that all nova services are now up... | 22:52 |
| melwitt | okay | 22:53 |
| stewie925 | thank you for these notes, melwitt | 22:53 |
| *** josecastroleon has quit IRC | 22:54 | |
| stewie925 | oh btw melwitt - right after I ran stack.sh I still seee a private and public network, is that okay? | 22:55 |
| *** diogogmt has quit IRC | 22:56 | |
| melwitt | stewie925: probably, I don't know why it's creating two networks for you, might be something in your local.conf. I don't have a current devstack handy to check | 22:57 |
| *** ducttape_ has quit IRC | 22:57 | |
| stewie925 | thank you melwitt - no problem - hope this env works now with n-net already up :) | 22:57 |
| *** markvoelker has quit IRC | 22:58 | |
| *** mrkz has quit IRC | 22:58 | |
| melwitt | the local.conf I use is very minimal for cells http://paste.openstack.org/show/497816/ | 22:59 |
| *** zenoway has joined #openstack-nova | 22:59 | |
| *** dimtruck is now known as zz_dimtruck | 23:01 | |
| *** ametts has quit IRC | 23:04 | |
| *** ihrachys has quit IRC | 23:04 | |
| *** zenoway has quit IRC | 23:04 | |
| *** sdake has joined #openstack-nova | 23:04 | |
| *** Sukhdev has quit IRC | 23:05 | |
| *** Sukhdev has joined #openstack-nova | 23:08 | |
| *** stewie925 has quit IRC | 23:08 | |
| *** salv-orl_ has quit IRC | 23:12 | |
| *** sdake has quit IRC | 23:14 | |
| *** sdake has joined #openstack-nova | 23:15 | |
| *** ducttape_ has joined #openstack-nova | 23:15 | |
| artom | dansmith, thanks for the reviews :) Don't spend *too* much time on it, as it's entirely possible tests are failing all over the place, because this is just the first patchset reorg to use the new "blob in database" approach | 23:16 |
| dansmith | artom: I know, I just want to make sure you have early feedback on structural things | 23:16 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova: Improve the help text for the rdp options https://review.openstack.org/318974 | 23:16 |
| artom | dansmith, http://www.goodpaper.com/images/21-010.jpg | 23:17 |
| dansmith | artom: feel free to ignore it until it's convenient, but if it saves a round-trip | 23:17 |
| artom | (Yes, I know this isn't reddit and we don't converse in image macros) | 23:17 |
| dansmith | then it makes sense | 23:17 |
| dansmith | heh | 23:17 |
| *** amotoki has quit IRC | 23:21 | |
| *** sdake has quit IRC | 23:21 | |
| *** amotoki has joined #openstack-nova | 23:21 | |
| *** ayoung has quit IRC | 23:22 | |
| *** Swami has joined #openstack-nova | 23:23 | |
| *** zenoway has joined #openstack-nova | 23:23 | |
| *** ducttape_ has quit IRC | 23:26 | |
| *** amotoki has quit IRC | 23:26 | |
| *** Sukhdev has quit IRC | 23:27 | |
| *** sorrison_laptop has joined #openstack-nova | 23:27 | |
| *** imcsk8 has quit IRC | 23:27 | |
| *** imcsk8 has joined #openstack-nova | 23:27 | |
| *** zenoway has quit IRC | 23:28 | |
| *** zz_dimtruck is now known as dimtruck | 23:31 | |
| *** stewie925 has joined #openstack-nova | 23:32 | |
| stewie925 | hi thank you Melwitt for sharing your local.conf - it sure is very minimal | 23:32 |
| stewie925 | mine has a lot of junk in it | 23:32 |
| *** BjoernT has quit IRC | 23:37 | |
| *** dimtruck is now known as zz_dimtruck | 23:41 | |
| *** baoli has quit IRC | 23:44 | |
| *** zenoway has joined #openstack-nova | 23:48 | |
| *** ayoung has joined #openstack-nova | 23:49 | |
| *** sorrison_laptop has quit IRC | 23:49 | |
| *** jamielennox is now known as jamielennox|away | 23:49 | |
| *** sorrison_laptop has joined #openstack-nova | 23:49 | |
| *** fragatin_ has quit IRC | 23:50 | |
| *** fragatina has joined #openstack-nova | 23:51 | |
| *** zz_dimtruck is now known as dimtruck | 23:51 | |
| *** sorrison_laptop has quit IRC | 23:52 | |
| *** sorriso__ has joined #openstack-nova | 23:52 | |
| *** sdake has joined #openstack-nova | 23:52 | |
| *** zenoway has quit IRC | 23:52 | |
| *** ducttape_ has joined #openstack-nova | 23:54 | |
| *** kylek3h has joined #openstack-nova | 23:56 | |
| *** sorrison_laptop has joined #openstack-nova | 23:56 | |
| *** sorriso__ has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
