*** armax has quit IRC | 00:01 | |
*** gabor_antal has quit IRC | 00:04 | |
*** gabor_antal has joined #openstack-nova | 00:04 | |
*** hferenc has quit IRC | 00:05 | |
*** hferenc has joined #openstack-nova | 00:05 | |
*** moshele has joined #openstack-nova | 00:07 | |
*** rama_y has joined #openstack-nova | 00:09 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 00:12 |
---|---|---|
*** salv-orlando has joined #openstack-nova | 00:14 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console opt to console group https://review.openstack.org/389647 | 00:14 |
*** anush has joined #openstack-nova | 00:14 | |
*** gabor_antal_ has joined #openstack-nova | 00:14 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: move console opt to console group https://review.openstack.org/389647 | 00:15 |
*** gabor_antal has quit IRC | 00:18 | |
*** yuntongjin has joined #openstack-nova | 00:19 | |
*** yuntongjin has quit IRC | 00:19 | |
*** salv-orlando has quit IRC | 00:19 | |
*** ducttape_ has joined #openstack-nova | 00:41 | |
*** hongbin has quit IRC | 00:45 | |
*** tovin07 has joined #openstack-nova | 00:46 | |
*** ducttape_ has quit IRC | 00:48 | |
*** fragatin_ has quit IRC | 00:50 | |
*** mlavalle has quit IRC | 00:50 | |
*** fragatina has joined #openstack-nova | 00:51 | |
*** giri has quit IRC | 00:51 | |
*** artom__ is now known as artom | 00:53 | |
*** ihrachys has quit IRC | 00:55 | |
*** thorst_ has joined #openstack-nova | 01:00 | |
*** ducttape_ has joined #openstack-nova | 01:00 | |
*** thorst_ has quit IRC | 01:01 | |
*** thorst_ has joined #openstack-nova | 01:01 | |
*** thorst_ has quit IRC | 01:04 | |
*** adisky has quit IRC | 01:05 | |
*** ducnc has joined #openstack-nova | 01:12 | |
*** jamesdenton has joined #openstack-nova | 01:13 | |
*** jamesdenton has quit IRC | 01:13 | |
*** salv-orlando has joined #openstack-nova | 01:16 | |
*** hfu has joined #openstack-nova | 01:16 | |
*** salv-orlando has quit IRC | 01:20 | |
*** ducttape_ has quit IRC | 01:21 | |
*** dimtruck is now known as zz_dimtruck | 01:25 | |
*** zz_dimtruck is now known as dimtruck | 01:25 | |
*** ducttape_ has joined #openstack-nova | 01:28 | |
*** rama_y has quit IRC | 01:29 | |
*** ducttape_ has quit IRC | 01:29 | |
*** anush has quit IRC | 01:30 | |
*** anush has joined #openstack-nova | 01:30 | |
*** zhangjl has joined #openstack-nova | 01:32 | |
*** _ducttape_ has joined #openstack-nova | 01:34 | |
*** dimtruck is now known as zz_dimtruck | 01:34 | |
*** anush has quit IRC | 01:35 | |
*** zz_dimtruck is now known as dimtruck | 01:39 | |
*** shuquan has joined #openstack-nova | 01:39 | |
*** kaisers has joined #openstack-nova | 01:40 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: added notifications group https://review.openstack.org/392145 | 01:42 |
*** kaisers1 has quit IRC | 01:42 | |
*** dave-johnston has quit IRC | 01:43 | |
*** anush has joined #openstack-nova | 01:44 | |
*** yufei has joined #openstack-nova | 01:48 | |
*** _ducttape_ has quit IRC | 01:55 | |
*** dimtruck is now known as zz_dimtruck | 01:57 | |
*** zz_dimtruck is now known as dimtruck | 01:57 | |
openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional judgment https://review.openstack.org/402256 | 01:57 |
openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host https://review.openstack.org/357105 | 01:58 |
*** ljxiash has joined #openstack-nova | 02:00 | |
*** shuquan has quit IRC | 02:00 | |
*** ducttape_ has joined #openstack-nova | 02:01 | |
*** shuquan has joined #openstack-nova | 02:02 | |
openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional judgment https://review.openstack.org/402256 | 02:05 |
*** moshele has quit IRC | 02:05 | |
*** dimtruck is now known as zz_dimtruck | 02:06 | |
*** Jeffrey4l has joined #openstack-nova | 02:11 | |
*** salv-orlando has joined #openstack-nova | 02:17 | |
*** ducttape_ has quit IRC | 02:18 | |
openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional judgment https://review.openstack.org/402256 | 02:18 |
*** zz_dimtruck is now known as dimtruck | 02:20 | |
*** salv-orlando has quit IRC | 02:21 | |
*** ducttape_ has joined #openstack-nova | 02:27 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_keypairs to Python 3 https://review.openstack.org/398909 | 02:30 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port libvirt test_driver to Python 3 https://review.openstack.org/400697 | 02:30 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port virt vmwareapi tests to Python 3 https://review.openstack.org/401080 | 02:30 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port xenapi test_xenapi to Python 3 https://review.openstack.org/401065 | 02:30 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port ironic unit tests to Python 3 https://review.openstack.org/398948 | 02:30 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Port test_metadata to Python 3 https://review.openstack.org/398750 | 02:30 |
*** hongbin has joined #openstack-nova | 02:33 | |
*** victorhe has joined #openstack-nova | 02:34 | |
*** victorhe has quit IRC | 02:38 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache https://review.openstack.org/307155 | 02:39 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Remove tests from tests-py3.txt https://review.openstack.org/399571 | 02:43 |
*** alexpilotti has quit IRC | 02:44 | |
*** kaisers_ has joined #openstack-nova | 02:46 | |
*** gjayavelu has joined #openstack-nova | 02:46 | |
*** thorst_ has joined #openstack-nova | 02:47 | |
*** thorst_ has quit IRC | 02:47 | |
*** kaisers_ has quit IRC | 02:50 | |
*** cfriesen has joined #openstack-nova | 02:56 | |
*** ducttape_ has quit IRC | 02:58 | |
*** code-R has joined #openstack-nova | 03:02 | |
*** ducttape_ has joined #openstack-nova | 03:05 | |
openstackgerrit | Rong Han proposed openstack/nova: Make hypervisor-servers support one host has multi-compute-nodes. https://review.openstack.org/398661 | 03:12 |
*** salv-orlando has joined #openstack-nova | 03:18 | |
*** kaisers_ has joined #openstack-nova | 03:20 | |
*** salv-orlando has quit IRC | 03:22 | |
*** rama_y has joined #openstack-nova | 03:23 | |
*** kaisers_ has quit IRC | 03:25 | |
*** ducttape_ has quit IRC | 03:26 | |
*** victorhe has joined #openstack-nova | 03:33 | |
*** victorhe has quit IRC | 03:38 | |
*** alexpilotti has joined #openstack-nova | 03:44 | |
*** ducttape_ has joined #openstack-nova | 03:44 | |
*** code-R has quit IRC | 03:45 | |
*** anush has quit IRC | 03:45 | |
*** thorst_ has joined #openstack-nova | 03:46 | |
*** anush has joined #openstack-nova | 03:46 | |
*** ayoung has quit IRC | 03:47 | |
*** yufei has quit IRC | 03:48 | |
*** links has joined #openstack-nova | 03:48 | |
*** ducttape_ has quit IRC | 03:49 | |
*** alexpilotti has quit IRC | 03:49 | |
*** huanxie has joined #openstack-nova | 03:51 | |
*** thorst_ has quit IRC | 03:51 | |
*** tuanluong has joined #openstack-nova | 03:52 | |
*** code-R has joined #openstack-nova | 03:53 | |
*** ducttape_ has joined #openstack-nova | 03:56 | |
*** namnh has joined #openstack-nova | 03:59 | |
*** code-R has quit IRC | 04:00 | |
*** ducttape_ has quit IRC | 04:00 | |
*** psachin has joined #openstack-nova | 04:09 | |
*** alexpilotti has joined #openstack-nova | 04:15 | |
*** tovin07 has quit IRC | 04:17 | |
*** alexpilotti has quit IRC | 04:21 | |
*** I has joined #openstack-nova | 04:28 | |
*** sudipto_ has joined #openstack-nova | 04:28 | |
*** I is now known as Guest63071 | 04:28 | |
*** shuquan has quit IRC | 04:30 | |
*** mirrorbo1 has quit IRC | 04:34 | |
*** mirrorbo1 has joined #openstack-nova | 04:35 | |
*** anush has quit IRC | 04:35 | |
*** anush has joined #openstack-nova | 04:35 | |
*** anush has quit IRC | 04:36 | |
*** anush has joined #openstack-nova | 04:36 | |
*** anush has quit IRC | 04:36 | |
*** anush has joined #openstack-nova | 04:37 | |
*** hongbin has quit IRC | 04:37 | |
*** anush has quit IRC | 04:37 | |
*** code-R has joined #openstack-nova | 04:37 | |
*** anush has joined #openstack-nova | 04:37 | |
*** anush has quit IRC | 04:38 | |
*** anush has joined #openstack-nova | 04:38 | |
*** anush has quit IRC | 04:39 | |
*** anush has joined #openstack-nova | 04:39 | |
*** anush has quit IRC | 04:40 | |
*** anush has joined #openstack-nova | 04:40 | |
*** anush has quit IRC | 04:40 | |
*** code-R has quit IRC | 04:40 | |
*** anush has joined #openstack-nova | 04:41 | |
*** code-R has joined #openstack-nova | 04:41 | |
*** anush has quit IRC | 04:41 | |
*** anush has joined #openstack-nova | 04:41 | |
*** anush has quit IRC | 04:42 | |
*** huanxie has quit IRC | 04:43 | |
*** tovin07 has joined #openstack-nova | 04:45 | |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: conf: Remove deprecated service manager opts https://review.openstack.org/319532 | 04:45 |
*** code-R_ has joined #openstack-nova | 04:46 | |
*** code-R has quit IRC | 04:46 | |
*** preethipy has joined #openstack-nova | 04:49 | |
*** code-R_ has quit IRC | 04:52 | |
*** victorhe has joined #openstack-nova | 04:53 | |
*** code-R has joined #openstack-nova | 04:53 | |
*** yuntongjin has joined #openstack-nova | 04:53 | |
*** huanxie has joined #openstack-nova | 04:54 | |
*** ducttape_ has joined #openstack-nova | 04:57 | |
*** victorhe has quit IRC | 04:57 | |
*** ducnc has quit IRC | 04:58 | |
openstackgerrit | Abhishek Kekane proposed openstack/nova: Use is_valid_cidr, is_valid_ipv6_cidr from oslo_utils https://review.openstack.org/320375 | 05:01 |
*** gjayavelu has quit IRC | 05:01 | |
*** ducttape_ has quit IRC | 05:02 | |
*** sudipto_ has quit IRC | 05:04 | |
*** Guest63071 has quit IRC | 05:04 | |
*** gjayavelu has joined #openstack-nova | 05:07 | |
*** gjayavelu has quit IRC | 05:12 | |
*** mdnadeem has joined #openstack-nova | 05:12 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Move tag schema to parameter_types.py https://review.openstack.org/402295 | 05:13 |
*** prateek has joined #openstack-nova | 05:16 | |
*** alexpilotti has joined #openstack-nova | 05:18 | |
*** dimtruck is now known as zz_dimtruck | 05:21 | |
*** zz_dimtruck is now known as dimtruck | 05:21 | |
*** ratailor has joined #openstack-nova | 05:22 | |
*** ayogi has joined #openstack-nova | 05:23 | |
*** alexpilotti has quit IRC | 05:27 | |
*** gjayavelu has joined #openstack-nova | 05:28 | |
*** dimtruck is now known as zz_dimtruck | 05:31 | |
*** huanxie has quit IRC | 05:31 | |
*** yufei has joined #openstack-nova | 05:32 | |
*** gjayavelu has quit IRC | 05:32 | |
*** gmann_ has joined #openstack-nova | 05:34 | |
*** takashin has left #openstack-nova | 05:36 | |
*** kaisers_ has joined #openstack-nova | 05:36 | |
*** kaisers_ has quit IRC | 05:40 | |
*** preethipy has quit IRC | 05:40 | |
*** preethipy has joined #openstack-nova | 05:41 | |
*** yufei has quit IRC | 05:41 | |
*** shuquan has joined #openstack-nova | 05:42 | |
*** TravT has joined #openstack-nova | 05:44 | |
*** gjayavelu has joined #openstack-nova | 05:48 | |
*** yuntongjin has quit IRC | 05:50 | |
*** gjayavelu has quit IRC | 05:52 | |
*** rama_y has quit IRC | 05:52 | |
*** victorhe has joined #openstack-nova | 05:54 | |
*** victorhe has quit IRC | 05:58 | |
*** siva_krish has joined #openstack-nova | 06:01 | |
*** preethipy has quit IRC | 06:02 | |
*** mirrorbo1 has quit IRC | 06:04 | |
*** mirrorbo1 has joined #openstack-nova | 06:05 | |
*** gjayavelu has joined #openstack-nova | 06:09 | |
*** gjayavelu has quit IRC | 06:11 | |
*** gjayavelu has joined #openstack-nova | 06:11 | |
*** huanxie has joined #openstack-nova | 06:13 | |
*** nandal has joined #openstack-nova | 06:17 | |
*** mdnadeem has quit IRC | 06:18 | |
*** salv-orlando has joined #openstack-nova | 06:19 | |
*** hfu has quit IRC | 06:20 | |
*** shihanzhang has quit IRC | 06:20 | |
*** lpetrut has joined #openstack-nova | 06:21 | |
*** hfu has joined #openstack-nova | 06:21 | |
*** hfu has quit IRC | 06:21 | |
openstackgerrit | YuYang proposed openstack/nova: VMware:fix get vm'root deivce conditional judgment https://review.openstack.org/402256 | 06:22 |
*** tojuvone has quit IRC | 06:23 | |
*** TravT has quit IRC | 06:25 | |
*** salv-orlando has quit IRC | 06:25 | |
*** yufei has joined #openstack-nova | 06:25 | |
*** code-R has quit IRC | 06:26 | |
*** shuquan has quit IRC | 06:26 | |
*** yuntongjin has joined #openstack-nova | 06:28 | |
*** jaosorior has joined #openstack-nova | 06:28 | |
*** hfu has joined #openstack-nova | 06:33 | |
*** shuquan has joined #openstack-nova | 06:33 | |
*** shuquan_ has joined #openstack-nova | 06:39 | |
*** shuquan has quit IRC | 06:39 | |
*** cfriesen has quit IRC | 06:40 | |
*** salv-orlando has joined #openstack-nova | 06:43 | |
*** mdnadeem has joined #openstack-nova | 06:44 | |
*** sridharg has joined #openstack-nova | 06:46 | |
*** tovin07 has quit IRC | 06:49 | |
*** shuquan has joined #openstack-nova | 06:51 | |
*** shuquan_ has quit IRC | 06:51 | |
*** nkrinner_afk is now known as nkrinner | 06:52 | |
*** TravT has joined #openstack-nova | 06:52 | |
*** mjura has joined #openstack-nova | 06:54 | |
*** jaosorior has quit IRC | 06:58 | |
*** ducttape_ has joined #openstack-nova | 06:58 | |
*** jaosorior has joined #openstack-nova | 06:58 | |
*** mjura has quit IRC | 07:01 | |
*** mjura has joined #openstack-nova | 07:02 | |
*** ducttape_ has quit IRC | 07:03 | |
*** yuntongjin has quit IRC | 07:05 | |
*** rcernin has joined #openstack-nova | 07:05 | |
*** victorhe has joined #openstack-nova | 07:06 | |
openstackgerrit | Thomas Bechtold proposed openstack/nova: Fix root_device_name for Xen https://review.openstack.org/402318 | 07:07 |
*** yuntongjin has joined #openstack-nova | 07:08 | |
*** oanson has joined #openstack-nova | 07:08 | |
*** lpetrut has quit IRC | 07:09 | |
*** yufei has quit IRC | 07:10 | |
*** victorhe has quit IRC | 07:11 | |
*** shuquan has quit IRC | 07:11 | |
*** alexpilotti has joined #openstack-nova | 07:24 | |
*** yuntongjin has quit IRC | 07:24 | |
*** yuntongjin has joined #openstack-nova | 07:26 | |
*** rcernin has quit IRC | 07:26 | |
*** tovin07 has joined #openstack-nova | 07:28 | |
*** rcernin has joined #openstack-nova | 07:28 | |
*** oanson has quit IRC | 07:28 | |
*** TravT has quit IRC | 07:29 | |
*** alexpilotti has quit IRC | 07:29 | |
*** hfu has quit IRC | 07:29 | |
*** hfu has joined #openstack-nova | 07:30 | |
*** kaisers_ has joined #openstack-nova | 07:33 | |
*** code-R has joined #openstack-nova | 07:33 | |
*** moshele has joined #openstack-nova | 07:37 | |
*** namnh has quit IRC | 07:41 | |
openstackgerrit | Rajesh Tailor proposed openstack/nova: Allow force-delete irrespective of VM task_state https://review.openstack.org/397542 | 07:41 |
*** namnh has joined #openstack-nova | 07:41 | |
*** namnh has quit IRC | 07:42 | |
*** namnh has joined #openstack-nova | 07:43 | |
*** code-R has quit IRC | 07:43 | |
*** namnh_ has joined #openstack-nova | 07:43 | |
*** namnh_ has quit IRC | 07:43 | |
*** namnh has quit IRC | 07:43 | |
*** namnh has joined #openstack-nova | 07:43 | |
*** nherciu has joined #openstack-nova | 07:44 | |
*** rcernin has quit IRC | 07:51 | |
*** gszasz has joined #openstack-nova | 07:52 | |
*** rcernin has joined #openstack-nova | 07:54 | |
*** gmann_ has quit IRC | 07:56 | |
*** yufei has joined #openstack-nova | 07:56 | |
*** sahid has joined #openstack-nova | 07:57 | |
*** Jeffrey4l has quit IRC | 07:57 | |
*** salv-orlando has quit IRC | 07:59 | |
*** shuquan has joined #openstack-nova | 08:00 | |
*** huanxie has quit IRC | 08:03 | |
*** siva_krish has quit IRC | 08:03 | |
*** yuntongjin1 has joined #openstack-nova | 08:10 | |
*** amoralej|off is now known as amoralej | 08:11 | |
*** matrohon has joined #openstack-nova | 08:11 | |
*** markus_z has joined #openstack-nova | 08:12 | |
*** nandal has quit IRC | 08:12 | |
*** yuntongjin has quit IRC | 08:12 | |
*** huanxie has joined #openstack-nova | 08:16 | |
*** vladikr has quit IRC | 08:17 | |
*** ihrachys has joined #openstack-nova | 08:17 | |
*** pcaruana has joined #openstack-nova | 08:19 | |
*** zenoway has joined #openstack-nova | 08:19 | |
*** aloga has quit IRC | 08:21 | |
*** aloga has joined #openstack-nova | 08:22 | |
*** code-R has joined #openstack-nova | 08:26 | |
*** code-R_ has joined #openstack-nova | 08:31 | |
*** ketann has joined #openstack-nova | 08:31 | |
*** code-R has quit IRC | 08:33 | |
*** code-R_ has quit IRC | 08:35 | |
*** gjayavelu has quit IRC | 08:35 | |
*** ihrachys has quit IRC | 08:35 | |
*** code-R has joined #openstack-nova | 08:36 | |
*** gjayavelu has joined #openstack-nova | 08:41 | |
*** ducnc has joined #openstack-nova | 08:42 | |
*** gjayavelu has quit IRC | 08:45 | |
*** masuberu has joined #openstack-nova | 08:46 | |
masuberu | hi | 08:46 |
masuberu | was wondering if anyone has seen this error before? | 08:46 |
masuberu | https://ibin.co/33AvrictkWCD.png | 08:47 |
masuberu | I get this error almost every time I create or start an instance | 08:47 |
openstackgerrit | Rong Han proposed openstack/nova: Make hypervisor-servers support one host has multi-compute-nodes. https://review.openstack.org/398661 | 08:47 |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (2) https://review.openstack.org/335411 | 08:50 |
*** victorhe has joined #openstack-nova | 08:52 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: api-ref: Deprecate the server actions to get consoles in v2.5 https://review.openstack.org/402138 | 08:55 |
openstackgerrit | alex proposed openstack/nova: there are some data left over after resize https://review.openstack.org/402361 | 08:56 |
*** yuntongjin1 has quit IRC | 08:57 | |
*** ducttape_ has joined #openstack-nova | 08:59 | |
*** zzzeek has quit IRC | 09:00 | |
*** zzzeek has joined #openstack-nova | 09:00 | |
*** victorhe has quit IRC | 09:00 | |
openstackgerrit | Zhenyu Zheng proposed openstack/python-novaclient: Check destination_type when booting with bdm provided https://review.openstack.org/402364 | 09:02 |
*** sudipto_ has joined #openstack-nova | 09:03 | |
*** sudswas__ has joined #openstack-nova | 09:03 | |
*** ducttape_ has quit IRC | 09:04 | |
*** moshele has quit IRC | 09:04 | |
*** gjayavelu has joined #openstack-nova | 09:06 | |
*** ralonsoh has joined #openstack-nova | 09:06 | |
*** moshele has joined #openstack-nova | 09:07 | |
*** gjayavelu has quit IRC | 09:11 | |
*** lpetrut has joined #openstack-nova | 09:13 | |
*** ducnc1 has joined #openstack-nova | 09:17 | |
*** ducnc has quit IRC | 09:18 | |
*** ducnc1 is now known as ducnc | 09:18 | |
*** sandanar has joined #openstack-nova | 09:19 | |
*** gjayavelu has joined #openstack-nova | 09:22 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided https://review.openstack.org/402372 | 09:22 |
*** cnf has left #openstack-nova | 09:23 | |
*** alexpilotti has joined #openstack-nova | 09:23 | |
*** gjayavelu has quit IRC | 09:26 | |
*** code-R has quit IRC | 09:26 | |
*** nandal has joined #openstack-nova | 09:29 | |
*** tojuvone has joined #openstack-nova | 09:30 | |
*** lucas-afk is now known as lucasagomes | 09:30 | |
*** rmart04 has joined #openstack-nova | 09:30 | |
*** code-R has joined #openstack-nova | 09:31 | |
*** code-R_ has joined #openstack-nova | 09:33 | |
*** code-R has quit IRC | 09:36 | |
*** derekh has joined #openstack-nova | 09:38 | |
*** yassine has joined #openstack-nova | 09:39 | |
*** yassine is now known as Guest47999 | 09:40 | |
*** code-R_ has quit IRC | 09:42 | |
*** gjayavelu has joined #openstack-nova | 09:42 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 09:43 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP libvirt: Fetch LibvirtConfigGuestDisk by target device or source file https://review.openstack.org/402163 | 09:43 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Ensure to_xml always returns an ASCII string https://review.openstack.org/402382 | 09:43 |
*** code-R has joined #openstack-nova | 09:43 | |
openstackgerrit | Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3) https://review.openstack.org/335765 | 09:43 |
*** psachin has quit IRC | 09:43 | |
*** psachin has joined #openstack-nova | 09:43 | |
sfinucan | johnthetubaguy: Thanks for all the reviews yesterday. My in-progress queue is looking much healthier this morning :) | 09:45 |
*** dpawar has joined #openstack-nova | 09:46 | |
*** gjayavelu has quit IRC | 09:46 | |
markus_z | johnthetubaguy: Yeah, a big 'thank you' from my side too. | 09:47 |
*** kevinz has quit IRC | 09:49 | |
*** ihrachys has joined #openstack-nova | 09:50 | |
*** dpawar has quit IRC | 09:51 | |
*** nandal has quit IRC | 09:51 | |
*** dpawar has joined #openstack-nova | 09:51 | |
dpawar | hi | 09:51 |
*** code-R has quit IRC | 09:51 | |
dpawar | I am new to openstack, trying to setup CI .. can someone guide me ? | 09:52 |
dpawar | i mean Contineuos Integression test setup | 09:53 |
*** shuquan has quit IRC | 09:55 | |
*** shuquan_ has joined #openstack-nova | 09:55 | |
*** dpawar has quit IRC | 09:56 | |
*** victorhe has joined #openstack-nova | 10:01 | |
*** rmart04_ has joined #openstack-nova | 10:02 | |
*** gjayavelu has joined #openstack-nova | 10:02 | |
*** mlakat has quit IRC | 10:03 | |
*** rmart04 has quit IRC | 10:04 | |
*** rmart04_ is now known as rmart04 | 10:04 | |
*** victorhe has quit IRC | 10:05 | |
*** rmart04_ has joined #openstack-nova | 10:06 | |
*** gjayavelu has quit IRC | 10:07 | |
*** rmart04_ has quit IRC | 10:07 | |
*** shuquan_ has quit IRC | 10:08 | |
*** mlakat has joined #openstack-nova | 10:08 | |
*** rmart04 has quit IRC | 10:08 | |
*** tuanluong has quit IRC | 10:09 | |
*** shuquan has joined #openstack-nova | 10:09 | |
*** ralonsoh_ has joined #openstack-nova | 10:10 | |
*** code-R has joined #openstack-nova | 10:11 | |
*** rmart04 has joined #openstack-nova | 10:11 | |
*** ralonsoh has quit IRC | 10:12 | |
*** tovin07 has quit IRC | 10:13 | |
*** shuquan has quit IRC | 10:14 | |
*** hieulq has quit IRC | 10:15 | |
*** hieulq has joined #openstack-nova | 10:16 | |
*** code-R has quit IRC | 10:17 | |
*** namnh has quit IRC | 10:18 | |
*** hieulq has quit IRC | 10:19 | |
*** cdent has joined #openstack-nova | 10:21 | |
*** gjayavelu has joined #openstack-nova | 10:23 | |
*** paul-carlton has joined #openstack-nova | 10:24 | |
*** jaosorior has quit IRC | 10:24 | |
bauzas | rpodolyaka: hola | 10:24 |
rpodolyaka | hey! | 10:24 |
rpodolyaka | (in a meeting, so somewhat distracted right now - will be back in 20 minutes) | 10:25 |
bauzas | I'm still struggling to identify the right path for fixing https://bugs.launchpad.net/nova/+bug/1644513 | 10:25 |
openstack | Launchpad bug 1644513 in OpenStack Compute (nova) "(psycopg2.ProgrammingError) column "deleted" is of type integer but expression is of type boolean" [Critical,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 10:25 |
*** sayalilunkad has quit IRC | 10:25 | |
bauzas | rpodolyaka: np, I'll have to be afk in 20 mins | 10:25 |
bauzas | we could discuss that later | 10:26 |
*** lpetrut1 has joined #openstack-nova | 10:26 | |
openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.volume_attach notification https://review.openstack.org/401992 | 10:26 |
*** lpetrut has quit IRC | 10:26 | |
*** lpetrut1 is now known as lpetrut | 10:26 | |
*** ralonsoh__ has joined #openstack-nova | 10:27 | |
*** bks has joined #openstack-nova | 10:27 | |
*** gjayavelu has quit IRC | 10:27 | |
*** danpawlik has joined #openstack-nova | 10:29 | |
*** ralonsoh_ has quit IRC | 10:29 | |
danpawlik | Hello, if you can, please review https://review.openstack.org/#/c/389216/ | 10:29 |
*** sayalilunkad has joined #openstack-nova | 10:29 | |
*** zhangjl has quit IRC | 10:29 | |
*** fragatina has quit IRC | 10:31 | |
*** fragatina has joined #openstack-nova | 10:31 | |
*** yufei has quit IRC | 10:32 | |
*** tojuvone has quit IRC | 10:35 | |
*** gjayavelu has joined #openstack-nova | 10:43 | |
*** jaosorior has joined #openstack-nova | 10:47 | |
*** hfu has quit IRC | 10:47 | |
*** gjayavelu has quit IRC | 10:48 | |
markus_z | danpawlik: spec freeze was last week: https://releases.openstack.org/ocata/schedule.html#o-nova-spec-freeze I'm afraid you have to propose that for Pike again. | 10:54 |
*** claudiub|2 has joined #openstack-nova | 10:56 | |
markus_z | danpawlik: You might also want to switch to a "blueprint" as a means to track your effort instead of a bug report: https://blueprints.launchpad.net/nova | 10:57 |
markus_z | danpawlik: Here's a wiki page which explains this: https://wiki.openstack.org/wiki/Blueprints | 10:58 |
openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance.rebuild.error notification https://review.openstack.org/396621 | 11:00 |
markus_z | danpawlik: My last advice: if possible, have some code changes ready when the dev cycle for the next release starts, even if the spec is not yet approved. Showing some code helps to better grasp the idea, which helps to get your spec approved. | 11:00 |
*** ducttape_ has joined #openstack-nova | 11:00 | |
openstackgerrit | sahid proposed openstack/nova: objects: add new field cpu_emulator_threads_policy https://review.openstack.org/385022 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: objects: add new field cpus_overhead in NUMACell https://review.openstack.org/400874 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores https://review.openstack.org/400875 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus overhead https://review.openstack.org/400876 | 11:04 |
openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu overhead https://review.openstack.org/400877 | 11:04 |
*** gjayavelu has joined #openstack-nova | 11:04 | |
*** WillUK2010 has joined #openstack-nova | 11:05 | |
*** ducttape_ has quit IRC | 11:05 | |
*** ralonsoh__ is now known as ralonsoh | 11:05 | |
*** gjayavelu has quit IRC | 11:08 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Ensure to_xml always returns an ASCII string https://review.openstack.org/402382 | 11:09 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 11:09 |
openstackgerrit | Lee Yarwood proposed openstack/nova: WIP libvirt: Fetch LibvirtConfigGuestDisk by target device or source file https://review.openstack.org/402163 | 11:09 |
*** rpodolyaka is now known as rpodolyakahungry | 11:10 | |
*** owalsh has joined #openstack-nova | 11:10 | |
*** moshele has quit IRC | 11:16 | |
*** bks has quit IRC | 11:17 | |
*** tojuvone has joined #openstack-nova | 11:18 | |
*** zenoway has quit IRC | 11:19 | |
*** zenoway has joined #openstack-nova | 11:20 | |
openstackgerrit | John Garbutt proposed openstack/nova: config options: improve libvirt remotefs https://review.openstack.org/370843 | 11:24 |
openstackgerrit | John Garbutt proposed openstack/nova: Improve consistency in libvirt https://review.openstack.org/353679 | 11:24 |
openstackgerrit | John Garbutt proposed openstack/nova: config options: improve libvirt remotefs https://review.openstack.org/370843 | 11:24 |
*** zenoway has quit IRC | 11:24 | |
*** gjayavelu has joined #openstack-nova | 11:24 | |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Mock imagebackend template funcs in ImageBackendFixture https://review.openstack.org/400871 | 11:27 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend https://review.openstack.org/333244 | 11:27 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Refactor _create_image into storagepolicy https://review.openstack.org/356651 | 11:27 |
openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func https://review.openstack.org/333263 | 11:27 |
*** gjayavelu has quit IRC | 11:29 | |
*** zenoway has joined #openstack-nova | 11:33 | |
*** zenoway has quit IRC | 11:38 | |
openstackgerrit | John Garbutt proposed openstack/nova: Improve help text for interval_opts https://review.openstack.org/331761 | 11:44 |
*** sudswas__ has quit IRC | 11:44 | |
*** sudipto_ has quit IRC | 11:44 | |
*** dpawar1 has joined #openstack-nova | 11:44 | |
openstackgerrit | John Garbutt proposed openstack/nova: Remove deprecated compute_available_monitors option https://review.openstack.org/371062 | 11:45 |
*** gjayavelu has joined #openstack-nova | 11:45 | |
openstackgerrit | John Garbutt proposed openstack/nova: Remove config option snapshot_name_template https://review.openstack.org/371203 | 11:46 |
*** zenoway has joined #openstack-nova | 11:48 | |
kukacz | hi, I'm looking for some description of how `nova usage` calculates the consumption data | 11:48 |
*** thorst_ has joined #openstack-nova | 11:48 | |
*** gjayavelu has quit IRC | 11:49 | |
kukacz | ... thinking of using that for billing | 11:49 |
*** ketann has quit IRC | 11:54 | |
*** thorst_ has quit IRC | 11:56 | |
*** ratailor has quit IRC | 12:03 | |
*** zenoway has quit IRC | 12:04 | |
*** zenoway has joined #openstack-nova | 12:05 | |
*** lucasagomes is now known as lucas-hungry | 12:06 | |
*** oanson has joined #openstack-nova | 12:09 | |
*** nicolasbock has joined #openstack-nova | 12:09 | |
*** dpawar1 has quit IRC | 12:10 | |
*** code-R has joined #openstack-nova | 12:11 | |
*** omnipresent has joined #openstack-nova | 12:13 | |
*** oanson has quit IRC | 12:13 | |
*** zenoway has quit IRC | 12:13 | |
*** huanxie has quit IRC | 12:17 | |
*** omnipresent has quit IRC | 12:17 | |
*** raildo has joined #openstack-nova | 12:19 | |
*** oanson has joined #openstack-nova | 12:29 | |
*** zenoway has joined #openstack-nova | 12:31 | |
*** ayogi has quit IRC | 12:32 | |
*** oanson has quit IRC | 12:33 | |
*** mdnadeem has quit IRC | 12:41 | |
*** ljxiash has quit IRC | 12:42 | |
*** ljxiash has joined #openstack-nova | 12:42 | |
openstackgerrit | Chris Dent proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 12:43 |
openstackgerrit | Roman Podoliaka proposed openstack/nova: WIP: change deleted default from False to 0 https://review.openstack.org/402517 | 12:45 |
rpodolyakahungry | tdurakov: ^ | 12:46 |
*** ljxiash has quit IRC | 12:47 | |
openstackgerrit | jichenjc proposed openstack/nova: no extension for multiple create https://review.openstack.org/396622 | 12:53 |
openstackgerrit | Flavio Percoco proposed openstack/python-novaclient: Show team and repo badges on README https://review.openstack.org/402531 | 12:55 |
openstackgerrit | Flavio Percoco proposed openstack/nova-specs: Show team and repo badges on README https://review.openstack.org/402533 | 12:56 |
*** gcb has quit IRC | 12:56 | |
markus_z | ^ trivial fix | 12:57 |
openstackgerrit | Flavio Percoco proposed openstack/nova: Show team and repo badges on README https://review.openstack.org/402535 | 12:58 |
openstackgerrit | Flavio Percoco proposed openstack/os-vif: Show team and repo badges on README https://review.openstack.org/402536 | 12:59 |
*** ducttape_ has joined #openstack-nova | 13:01 | |
*** sridharg has quit IRC | 13:03 | |
*** ayogi has joined #openstack-nova | 13:03 | |
*** _ducttape_ has joined #openstack-nova | 13:05 | |
*** ducttape_ has quit IRC | 13:05 | |
*** hfu has joined #openstack-nova | 13:05 | |
*** sridharg has joined #openstack-nova | 13:06 | |
*** hfu has quit IRC | 13:06 | |
*** baoli has joined #openstack-nova | 13:15 | |
*** lucas-hungry is now known as lucasagomes | 13:16 | |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Don't reinvent __init__ https://review.openstack.org/402560 | 13:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Combine multiple NUMA-generation functions https://review.openstack.org/402561 | 13:17 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Remove duplicate NumaHostInfo https://review.openstack.org/402562 | 13:17 |
*** amoralej is now known as amoralej|lunch | 13:17 | |
*** claudiub has joined #openstack-nova | 13:18 | |
*** salv-orlando has joined #openstack-nova | 13:19 | |
*** oanson has joined #openstack-nova | 13:20 | |
*** claudiub|2 has quit IRC | 13:21 | |
*** jaosorior has quit IRC | 13:22 | |
*** flaper87 has quit IRC | 13:25 | |
*** vladikr has joined #openstack-nova | 13:27 | |
*** flaper87 has joined #openstack-nova | 13:32 | |
sfinucan | johnthetubaguy: Regarding https://review.openstack.org/#/c/399653/1/nova/tests/functional/libvirt/test_numa_servers.py - what kind of checks would you like to see? | 13:32 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 13:32 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 13:32 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 13:32 |
openstackgerrit | Chris Dent proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 13:32 |
sfinucan | (second comment) | 13:32 |
johnthetubaguy | sfinucan: thats a nit rather than a blocker | 13:33 |
johnthetubaguy | sfinucan: I was wondering if we could inspect the state in fake libvirt to see if the "right thing" has been done, although I guess it might not store that state? | 13:34 |
sfinucan | johnthetubaguy: Well I'm totally reworking that test and bulking it out, so if you've good ideas :) | 13:34 |
sfinucan | Of the host, or instance? | 13:34 |
johnthetubaguy | I guess I mean both, like making sure the instance is being pinned to the correct sockets | 13:34 |
*** ljxiash has joined #openstack-nova | 13:34 | |
johnthetubaguy | sfinucan: like if we were using real libvirt, we might want to check the instance XML, I guess, to make sure the right things got in there | 13:35 |
johnthetubaguy | I guess we could still do that | 13:36 |
sfinucan | johnthetubaguy: Sounds fair. I'll try work that in and see how I get on | 13:36 |
johnthetubaguy | sfinucan: honestly, probably worth looking at that as a more general follow up on those tests | 13:36 |
johnthetubaguy | sfinucan: made a comment on the one I approved, would be awesome if we didn't have to mock out that scheduler filter, but no idea how doable that is | 13:37 |
sfinucan | Yeah, they're a good start but could definitely do with some fleshing out. Did some minor cosmetic surgery on them already | 13:37 |
* johnthetubaguy nods | 13:37 | |
sfinucan | Will try get something into intel-nfv-ci too, if it's not already covered. Would be nice to test this stuff from an integration standpoint | 13:38 |
johnthetubaguy | yeah, that would be great too | 13:38 |
*** ljxiash has quit IRC | 13:39 | |
*** rcernin has quit IRC | 13:39 | |
*** ayogi has quit IRC | 13:45 | |
*** rcernin has joined #openstack-nova | 13:49 | |
*** _ducttape_ has quit IRC | 13:52 | |
*** kevinz has joined #openstack-nova | 13:52 | |
*** ljxiash has joined #openstack-nova | 13:53 | |
*** portdirect has joined #openstack-nova | 13:53 | |
*** prateek has quit IRC | 13:54 | |
*** hogepodge has quit IRC | 13:59 | |
*** baoli has quit IRC | 13:59 | |
*** hfu has joined #openstack-nova | 14:00 | |
openstackgerrit | Diana Clarke proposed openstack/nova: Simple tenant usage pagination https://review.openstack.org/386093 | 14:00 |
*** baoli has joined #openstack-nova | 14:00 | |
*** code-R has quit IRC | 14:02 | |
openstackgerrit | int32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space https://review.openstack.org/402632 | 14:03 |
*** hfu has quit IRC | 14:04 | |
*** anush has joined #openstack-nova | 14:05 | |
*** cfriesen has joined #openstack-nova | 14:05 | |
openstackgerrit | Chris Dent proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 14:06 |
*** rpodolyakahungry is now known as rpodolyaka | 14:07 | |
*** hfu has joined #openstack-nova | 14:07 | |
bauzas | johnthetubaguy: in case you don't know https://bugs.launchpad.net/nova/+bug/1644513 | 14:09 |
openstack | Launchpad bug 1644513 in OpenStack Compute (nova) "(psycopg2.ProgrammingError) column "deleted" is of type integer but expression is of type boolean" [Critical,In progress] - Assigned to Roman Podoliaka (rpodolyaka) | 14:09 |
rpodolyaka | will update the commit soon | 14:09 |
johnthetubaguy | honestly, its going to be a quick tricky to keep the compatiblity | 14:10 |
*** hfu_ has joined #openstack-nova | 14:10 | |
bauzas | rpodolyaka: I don't really like your patch | 14:11 |
rpodolyaka | bauzas: suggestions are welcome :) | 14:12 |
*** hfu has quit IRC | 14:12 | |
bauzas | rpodolyaka: yup, I'm thinking about that | 14:12 |
johnthetubaguy | s/quick/quite/ | 14:13 |
bauzas | rpodolyaka: tbh, I'm trying to create a postgre devstack | 14:13 |
johnthetubaguy | bauzas: should we just revert the patch that caused problems? | 14:13 |
bauzas | johnthetubaguy: it could be difficult | 14:14 |
johnthetubaguy | bauzas: do we have postgres on experimental for us, btw? | 14:14 |
johnthetubaguy | bauzas: as its a DB migration? | 14:14 |
bauzas | not sure about that | 14:14 |
*** hfu_ has quit IRC | 14:15 | |
*** baoli has quit IRC | 14:15 | |
diana_clarke | Here's an easy Friday api-ref review: https://review.openstack.org/#/c/400851/ | 14:15 |
diana_clarke | ^ The list server groups endpoint enforces osapi_max_limit, but doesn't document how to get the rest of the results (using limit & offset). Reported in the wild, by actual suck users. | 14:15 |
diana_clarke | s/suck/stuck ;) | 14:16 |
*** psachin has quit IRC | 14:17 | |
*** ducttape_ has joined #openstack-nova | 14:19 | |
kashyap | :D | 14:20 |
kashyap | diana_clarke: We know, it's not malicious intent, but only freudian slip | 14:21 |
*** dave-mccowan has joined #openstack-nova | 14:23 | |
*** hogepodge has joined #openstack-nova | 14:23 | |
diana_clarke | kashyap: That was one of the banned words in my house growing up & why I wasn't allowed to watch the simpsons. My subconscious must be feeling belligerent today ;) | 14:24 |
kashyap | Hehe | 14:24 |
*** gszasz has quit IRC | 14:26 | |
*** amoralej|lunch is now known as amoralej | 14:26 | |
*** hogepodge has quit IRC | 14:28 | |
*** nkrinner is now known as nkrinner_afk | 14:29 | |
* sfinucan just discovered that Python variable names can't start with a number. Who knew. | 14:30 | |
*** kevinz has quit IRC | 14:31 | |
*** links has quit IRC | 14:31 | |
*** jwcroppe has quit IRC | 14:32 | |
bauzas | sfinucan: >>> True = 'False' | 14:32 |
bauzas | >>> "False" is True | 14:32 |
bauzas | True | 14:32 |
bauzas | love it, heh ? | 14:32 |
mdbooth | >>> True = False | 14:33 |
mdbooth | >>> True | 14:33 |
mdbooth | False | 14:33 |
mdbooth | OMFG | 14:33 |
bauzas | that too :) | 14:34 |
bauzas | I remember some guido's paper about that | 14:34 |
mdbooth | What possible good can come from the ability to redefine builtin constants? | 14:34 |
bauzas | yeah that http://python-history.blogspot.fr/2013/11/story-of-none-true-false.html | 14:34 |
bauzas | but I think Py3 has a different behaviour | 14:36 |
*** hfu has joined #openstack-nova | 14:36 | |
bauzas | oh the followup gives more details http://python-history.blogspot.fr/2013/11/the-history-of-bool-true-and-false.html | 14:36 |
bauzas | anyway, /me goes back resurrecting a devstack | 14:37 |
*** ducttape_ has quit IRC | 14:37 | |
markus_z | muahahaha :D Can I sneak that somehow into a system of a co-worker? | 14:38 |
mdbooth | markus_z: That's the first place my mind went :) | 14:39 |
sfinucan | I favour messing with their autocorrect in Office | 14:39 |
markus_z | April 1st is coming and that's perfect | 14:40 |
sfinucan | there's only so many the-teh corrections one can take before taking a chair to the screen | 14:40 |
markus_z | rofl, that's awesome :D | 14:40 |
markus_z | regarding the python variable names, that's the same with module names. You cannot have "1_hello_world.py" and "2_hello_extended.py". | 14:41 |
bauzas | modules are just objects as well :) | 14:42 |
openstackgerrit | Ludovic Beliveau proposed openstack/nova: PCI: Release PCI devices on drop_move_claim() https://review.openstack.org/370374 | 14:42 |
markus_z | yep, that's how I learned it :) | 14:42 |
bauzas | so luckily, that's consistent :) | 14:42 |
bauzas | and oops, we're on BlackFriday, so.... | 14:43 |
*** bauzas is now known as _bowser | 14:43 | |
*** nherciu has quit IRC | 14:43 | |
_bowser | oh snap, all of the bowser names are registered | 14:44 |
*** jwcroppe has joined #openstack-nova | 14:45 | |
*** salv-orlando has quit IRC | 14:47 | |
*** salv-orlando has joined #openstack-nova | 14:47 | |
*** cdent_ has joined #openstack-nova | 14:49 | |
cfriesen | sfinucan: mdbooth: back in school we stuck a background daemon on a friend's machine that inserted "NN" at random intervals into the active window...was during peak essay-writing time. Not sure if he ever did find out who did it.. | 14:49 |
mdbooth | cfriesen: LOL | 14:50 |
markus_z | ^^ | 14:50 |
*** cdent has quit IRC | 14:50 | |
*** cdent_ is now known as cdent | 14:50 | |
mdbooth | Also, the japes which could be played before anybody used authentication in their X server | 14:50 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Validate NUMA topology + huge page combo https://review.openstack.org/399653 | 14:50 |
cfriesen | mdbooth: 100 instances of xeyes | 14:50 |
kashyap | Anyone with DevStack instance at hand (mine is still building) can tell me what do you see the value of 'state_path' in /etc/nova/nova.conf? | 14:52 |
markus_z | kashyap: state_path = /opt/stack/data/nova | 14:53 |
kashyap | markus_z: Thanks | 14:53 |
*** dave-mccowan has quit IRC | 14:54 | |
diana_clarke | kashyap: ditto (/opt/stack/data/nova) | 14:54 |
*** hfu has quit IRC | 14:54 | |
kashyap | diana_clarke: Thanks; I don't suppose you have a installer-based setup, too? | 14:54 |
*** hfu has joined #openstack-nova | 14:55 | |
diana_clarke | kashyap: nope | 14:55 |
*** nicolasbock has quit IRC | 14:56 | |
kashyap | Np | 14:56 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Ensure to_xml always returns an ASCII string https://review.openstack.org/402382 | 14:57 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 14:57 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | 14:57 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Wait for abort_job to complete when not resizing https://review.openstack.org/382449 | 14:57 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | 14:57 |
* lyarwood really needs to disable git-review's auto rebase bs | 14:58 | |
*** hfu has quit IRC | 15:00 | |
*** hogepodge has joined #openstack-nova | 15:00 | |
*** mjura has quit IRC | 15:01 | |
*** rama_y has joined #openstack-nova | 15:01 | |
mdbooth | >>> class Evil(object): | 15:03 |
mdbooth | ... def __nonzero__(self): | 15:03 |
mdbooth | ... if random.random() > 0.9: | 15:03 |
mdbooth | ... return 1 | 15:03 |
mdbooth | ... return 0 | 15:03 |
mdbooth | ... | 15:03 |
mdbooth | >>> True = Evil() | 15:03 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Validate NUMA topology + huge page combo https://review.openstack.org/399653 | 15:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Combine multiple NUMA-generation functions https://review.openstack.org/402561 | 15:04 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Remove duplicate NumaHostInfo https://review.openstack.org/402562 | 15:04 |
*** beagles is now known as beagles_afk | 15:04 | |
*** lpetrut has quit IRC | 15:05 | |
*** hogepodge has quit IRC | 15:05 | |
*** lpetrut has joined #openstack-nova | 15:05 | |
*** paw has joined #openstack-nova | 15:09 | |
markus_z | mdbooth: hahahaha :D | 15:12 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471 | 15:13 |
*** Jeffrey4l has joined #openstack-nova | 15:14 | |
*** ratailor has joined #openstack-nova | 15:14 | |
*** hfu has joined #openstack-nova | 15:15 | |
*** rcernin has quit IRC | 15:17 | |
*** cdent has quit IRC | 15:18 | |
*** openstackgerrit has quit IRC | 15:18 | |
*** openstackgerrit has joined #openstack-nova | 15:18 | |
*** kaisers_ has quit IRC | 15:20 | |
*** ducttape_ has joined #openstack-nova | 15:21 | |
*** cdent has joined #openstack-nova | 15:21 | |
*** ihrachys has quit IRC | 15:21 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: libvirt: virtlogd: use virtlogd for char devices https://review.openstack.org/323765 | 15:22 |
*** tbachman has quit IRC | 15:22 | |
*** rcernin has joined #openstack-nova | 15:22 | |
kashyap | markus_z: Timely, speaking of virtlogd and Nova | 15:23 |
kashyap | markus_z: Do you know why Nova keeps instances & logs in the same directory? | 15:23 |
*** kevinz has joined #openstack-nova | 15:24 | |
*** hongbin has joined #openstack-nova | 15:24 | |
kashyap | Because: Incidentally I'm triaging a downstream bug where on systems when virtlogd "suddenly appears" (due to update of libvirt) | 15:24 |
kashyap | Migrations fail | 15:24 |
kashyap | Because of SELinux context | 15:24 |
markus_z | kashyap: Hm, good question. I don't know why it's the same directory. I assume it was just easier back then? | 15:25 |
*** prateek has joined #openstack-nova | 15:25 | |
kashyap | Yep, "just easier back then" sounds like the perfect rationale :P | 15:26 |
markus_z | kashyap: There are chowns on that file because of permission issues. | 15:26 |
kashyap | markus_z: More seriously, not sure. But the way downstream intends to resolve is | 15:26 |
kashyap | To upgrade SELinux policy to give permissions to virtlogd | 15:26 |
markus_z | I have very little knowledge about SELinux, but my naive approach would be to do that, yes. | 15:26 |
markus_z | kashyap: Does it make sense to add this to a reno file in the virtlogd patch? Or can we do something in Nova code? | 15:27 |
*** kevinz has quit IRC | 15:28 | |
*** kevinz has joined #openstack-nova | 15:29 | |
*** rcernin has quit IRC | 15:30 | |
*** salv-orlando has quit IRC | 15:33 | |
macsz | johnthetubaguy: you there? wanted to catch up on your comment to config opts and your divine -2 :) | 15:34 |
kashyap | markus_z: Sorry, was distracted by a call. But a reno note definitely sounds good | 15:34 |
kashyap | Also: I'm testing a bit more | 15:34 |
diana_clarke | tpatzig_1: Do you still plan on working on "support flavor with no local disks"? | 15:35 |
kashyap | markus_z: I was asking about the 'state_path' because, apparently that issue is not seen _if_ 'state_path' in nova.conf is NOT the default /var/lib/nova | 15:35 |
* _bowser is about to tableflip his devstack | 15:36 | |
*** _bowser is now known as bauwser | 15:36 | |
*** kevinz has quit IRC | 15:37 | |
tpatzig_1 | @diana_clarke: yes. we definitely require that functionality. i just did not found enough time to write the missing tempest stuff. the feature itself is more or less ready and works. | 15:37 |
*** hferenc has quit IRC | 15:38 | |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | 15:38 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect when to pivot https://review.openstack.org/399735 | 15:38 |
openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | 15:38 |
diana_clarke | tpatzig_1: I wonder if it can still be worked on, given that we're past spec-freeze now... I don't think it was re-proposed for ocata, right? | 15:38 |
tpatzig_1 | diana_clarke: oh it was…let me check | 15:39 |
diana_clarke | tpatzig_1: (work has asked me to potentially help on this front) | 15:39 |
*** sridharg has quit IRC | 15:41 | |
kashyap | markus_z: Err s/not seen/seen/ (I.e. if the "state_path" is default, then things look kosher) | 15:41 |
tpatzig_1 | @diana_clarke: https://review.openstack.org/#/c/373054/ | 15:42 |
diana_clarke | tpatzig_1: ah, nice! | 15:42 |
*** ducttape_ has quit IRC | 15:43 | |
*** ratailor has quit IRC | 15:44 | |
johnthetubaguy | macsz: ah, yeah, good question | 15:49 |
johnthetubaguy | macsz: is that the one about needing a deprecation cycle, or the one about being used on compute nodes not on the console proxy service? | 15:49 |
bauwser | johnthetubaguy: rpodolyaka: I'm stuck by my 5Mbps connection bandwidth at home, so I can't really have a postgre devstack now | 15:49 |
*** kaisers_ has joined #openstack-nova | 15:49 | |
bauwser | johnthetubaguy: rpodolyaka: I think we should be discussing about how to modify that | 15:50 |
macsz | johnthetubaguy: the compute one | 15:50 |
johnthetubaguy | bauwser: I guess this is blocking other folks? | 15:50 |
johnthetubaguy | macsz: cools, yeah, I am thinking all the other options in the group are used by the console service, which is optional (I think?) | 15:51 |
bauwser | johnthetubaguy: rpodolyaka: I think the problem is with https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L225-L226 | 15:51 |
johnthetubaguy | macsz: the other one is used more widely, and is set on the compute node, so it feels unrelated | 15:51 |
johnthetubaguy | macsz: now granted they are all console related, but they are related to different console implementations, I believe | 15:51 |
bauwser | johnthetubaguy: rpodolyaka: and https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L85 | 15:52 |
johnthetubaguy | macsz: now my understand of the console stuff is very sketchy, so I am only 60% sure thats the right thing to do, does that make sense? | 15:52 |
*** ajo has quit IRC | 15:52 | |
rpodolyaka | bauwser: so I tried you suggestion from the comment on review and looks like it won't help - https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/fields.py#L435-L438 | 15:52 |
rpodolyaka | ^ the value is coerced here on assignment | 15:53 |
bauwser | rpodolyaka: right, but if you see the difference before and after the change, we try to set the Instance fields by using the BuildRequest.instance, right? | 15:53 |
pkholkin | flwang: hello, are you here? | 15:54 |
macsz | johnthetubaguy: i see your point, but having this in mind this patch is yes and no in the same time | 15:54 |
rpodolyaka | bauwser: my understanding of the problem is that mark all fields as dirty | 15:54 |
rpodolyaka | *we | 15:54 |
rpodolyaka | and as long as we do that we'll get the coerced default from the object - which is False | 15:55 |
macsz | johnthetubaguy: but i guess that this particular option was good where it was before i moded it | 15:55 |
macsz | moved* | 15:55 |
bauwser | rpodolyaka: well, that's not exactly the problem IMHO | 15:55 |
bauwser | rpodolyaka: the problem is that we're updating the deleted field instead of getting it from the DB | 15:56 |
bauwser | so given it's marked as dirty, we save it now | 15:56 |
rpodolyaka | it fails on INSERT, does not it? | 15:56 |
macsz | johnthetubaguy: so after all, it seems that -2 was not a part of a black friday give away, but quite reasonable one ;) | 15:56 |
rpodolyaka | there is nothing to get from the DB | 15:56 |
bauwser | so, yeah we have the problem because the field is dirty, but that's due to the fact we set it straight | 15:57 |
johnthetubaguy | macsz: FWIW, I had to go digging in the code for a while to try work out what that all does again | 15:57 |
*** bkopilov has joined #openstack-nova | 15:58 | |
johnthetubaguy | macsz: that stuff is quite murky | 15:58 |
bauwser | rpodolyaka: previously, we were instance.create() | 15:58 |
rpodolyaka | we do now as well | 15:58 |
*** ducttape_ has joined #openstack-nova | 15:58 | |
rpodolyaka | it's where it fails :) | 15:59 |
bauwser | in create_db_entry_for_new_instance() | 15:59 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: [scheduler][tests]: Fix incorrect aggr mock values https://review.openstack.org/402910 | 15:59 |
bauwser | right, but given we were doing instance.create(), deleted was marked as non-dirty | 15:59 |
rpodolyaka | correct | 16:00 |
*** ajo has joined #openstack-nova | 16:00 | |
bauwser | ie. not part of the updates | 16:00 |
bauwser | here, we're setting straight deleted = 0 | 16:00 |
bauwser | and boom | 16:00 |
rpodolyaka | bauwser: so what's your suggestion, we remove https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L85 ? | 16:02 |
bauwser | rpodolyaka: that's what I'd love to test | 16:02 |
bauwser | because I'm not sure we need to set it | 16:02 |
bauwser | but the mixin in oslo.db doesn't define a default value | 16:03 |
bauwser | oh, sorry it is | 16:03 |
bauwser | https://github.com/openstack/oslo.db/blob/master/oslo_db/sqlalchemy/models.py#L142 | 16:03 |
bauwser | \o/ | 16:03 |
bauwser | okay, I'm writing a patch | 16:03 |
bauwser | rpodolyaka: if you don't mind | 16:03 |
rpodolyaka | bauwser: feel free to ;) | 16:04 |
bauwser | rpodolyaka: I'll co-author you | 16:04 |
rpodolyaka | it's just that coercing of the value in the DB layer would allow to prevent this once and for all | 16:05 |
bauwser | rpodolyaka: I'd by far prefer to fix the oslo.versionedobject field | 16:05 |
rpodolyaka | I'll bring this up in oslo | 16:05 |
bauwser | sorry, the field type | 16:05 |
bauwser | a Boolean is incorrect | 16:05 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Validate NUMA topology + huge page combo https://review.openstack.org/399653 | 16:05 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Combine multiple NUMA-generation functions https://review.openstack.org/402561 | 16:05 |
openstackgerrit | Stephen Finucane proposed openstack/nova: tests: Remove duplicate NumaHostInfo https://review.openstack.org/402562 | 16:05 |
bauwser | rpodolyaka: yeah, file it against o.vo | 16:05 |
rpodolyaka | whether it makes sense to have such a check in oslo.db just in case | 16:05 |
bauwser | rpodolyaka: honestly, I don't like that, | 16:06 |
bauwser | rpodolyaka: because it surely fixes the problem, but adds another layer of salt on a wound | 16:06 |
rpodolyaka | well, it's not ovo which should be changed | 16:06 |
rpodolyaka | it's nova base object class | 16:06 |
rpodolyaka | isn't it? | 16:06 |
bauwser | sec | 16:06 |
rpodolyaka | the boolean default value is fine | 16:06 |
bauwser | rpodolyaka: you're right | 16:06 |
rpodolyaka | it's just that deleted must be a different type - integer | 16:07 |
*** cdent has quit IRC | 16:07 | |
bauwser | rpodolyaka: you're right, https://github.com/openstack/nova/blob/520ee65a55bf9894d5915d3cebc892439d703312/nova/objects/base.py#L219 must be something else but Boolean | 16:07 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: [scheduler][tests]: Fix incorrect aggr mock values https://review.openstack.org/402910 | 16:08 |
*** victorhe has joined #openstack-nova | 16:10 | |
*** derekh has quit IRC | 16:10 | |
bauwser | rpodolyaka: oh wait, unsetting deleted = 0 in L82 wouldn't probably fix the problem | 16:10 |
bauwser | rpodolyaka: because we reset the bits in the next call | 16:11 |
*** Oku_OS is now known as Oku_OS-away | 16:11 | |
bauwser | the real problem is that we take the value of deleted when we loop over the fields and set it dirty | 16:11 |
*** hfu has quit IRC | 16:12 | |
bauwser | so, I won't really remove L82, but I'll rather mark deleted as non-dirty | 16:12 |
bauwser | that will work but with the big constraint that we wouldn't persist a change in that field | 16:13 |
johnthetubaguy | bauwser: do we need to convert the boolean to an int in the object save logic, or something like that? | 16:13 |
*** hfu has joined #openstack-nova | 16:13 | |
bauwser | which is acceptable | 16:13 |
bauwser | johnthetubaguy: I want to fix something before the week-end | 16:13 |
bauwser | johnthetubaguy: my own favor would go to fix the right field type | 16:13 |
*** markus_z has quit IRC | 16:13 | |
rpodolyaka | bauwser: I quickly tried it on devstack and at least `nova boot ` succeeded - but the devstack is a mess at this point, so... | 16:13 |
bauwser | johnthetubaguy: but here, I want to leave a big TODO | 16:14 |
bauwser | rpodolyaka: you tested what ? | 16:14 |
bauwser | removing L82 or marking deleted as non-dirty? | 16:14 |
rpodolyaka | removed line https://github.com/openstack/nova/blob/master/nova/objects/build_request.py#L85 | 16:14 |
bauwser | mmmm | 16:14 |
johnthetubaguy | bauwser: you just need to implement a _save_deleted() method I guess? | 16:14 |
bauwser | rpodolyaka: okay, that works because the field isn't set | 16:15 |
bauwser | rpodolyaka: but by looking at L85 and why alaski added that, I guess we would break the API contract | 16:15 |
bauwser | if we remove it | 16:15 |
johnthetubaguy | bauwser: actually thats non-sense, ignore me | 16:15 |
johnthetubaguy | bauwser: is it worth just copying this pattern? https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L739 | 16:16 |
*** zenoway has quit IRC | 16:16 | |
bauwser | okay, here is the deal, lemme do the dirty>non-dirty hack in the new method and put a big TODO, that could work and we could go on week-end | 16:16 |
*** hfu has quit IRC | 16:16 | |
*** zenoway has joined #openstack-nova | 16:16 | |
*** hfu has joined #openstack-nova | 16:17 | |
johnthetubaguy | bauwser: OK, just looking at this code here: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L343 | 16:17 |
bauwser | johnthetubaguy: I was planning to obj_reset_changes(['deleted']) | 16:17 |
johnthetubaguy | bauwser: yeah, I think that works, cool | 16:18 |
bauwser | oh man | 16:18 |
bauwser | oh man oh man https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L347 | 16:18 |
bauwser | WTF? | 16:18 |
bauwser | so yeah, I get it | 16:18 |
johnthetubaguy | bauwser: it makes sense, just been reading through that now | 16:18 |
bauwser | I think I understand why we bool that in the object | 16:18 |
johnthetubaguy | bauwser: only db.instance_destroy should write to the deleted field, I believe | 16:18 |
johnthetubaguy | yeah, bool is what the user wants to see | 16:19 |
bauwser | johnthetubaguy: I tend to agree | 16:19 |
johnthetubaguy | not some random id | 16:19 |
bauwser | zactly | 16:19 |
bauwser | so, yeah, let's just obj_reset_changes() that | 16:19 |
bauwser | johnthetubaguy: I'd love if you could find some other +2er :) | 16:19 |
bauwser | most of the people I know are probably full of turkety | 16:20 |
bauwser | turkey | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: objects: Move 'arch' to 'fields.Architecture' https://review.openstack.org/356067 | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: objects: Move 'vm_mode' to 'fields.VMMode' https://review.openstack.org/356553 | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: objects: Move 'hv_type' to 'fields.HVType' https://review.openstack.org/356424 | 16:20 |
openstackgerrit | Stephen Finucane proposed openstack/nova: objects: Move 'cpumodel' to 'fields.CPU*' https://review.openstack.org/356398 | 16:20 |
johnthetubaguy | bauwser: good point, wonder if we can get someone else to write it... although that seems a little silly | 16:20 |
johnthetubaguy | was that sfinucan stepping up to volunteer? | 16:21 |
sfinucan | I didn't do it | 16:21 |
johnthetubaguy | sfinucan: hehe | 16:21 |
johnthetubaguy | sfinucan: we have a critical bug, and only looks like two cores are hanging around the place right now | 16:21 |
johnthetubaguy | sfinucan: BTW, did that stuff just need a rebase somewhere? what was the clash? | 16:22 |
sfinucan | johnthetubaguy: OK. Can I be of help? | 16:22 |
johnthetubaguy | I think bauwser can fill you in on the bug, basically we should really store deleted when you save an instance, it should only be written by destroy | 16:23 |
johnthetubaguy | we indirectly broke postgres | 16:23 |
bauwser | the patch is up locally | 16:23 |
bauwser | I just wonder what to write as tests | 16:24 |
sfinucan | johnthetubaguy: RE those changes, it was resolving a merge conflict due to markus_z's serial port rework. The only changes are in '_create_consoles' | 16:24 |
bauwser | because that's a no-op | 16:24 |
bauwser | sfinucan: we have a critical bug requiring some other help | 16:24 |
rpodolyaka | bauwser: I'm updating my change just in case, so we can choose which one to merge | 16:24 |
bauwser | rpodolyaka: well, you could then | 16:24 |
sfinucan | bauwser: sorry - he asked :) I'm following | 16:24 |
openstackgerrit | Roman Podoliaka proposed openstack/nova: db: coerce deleted param to be an integer value https://review.openstack.org/402090 | 16:25 |
*** zenoway has quit IRC | 16:25 | |
bauwser | rpodolyaka: http://paste.openstack.org/show/590494/ | 16:25 |
bauwser | rpodolyaka: I still don't like your proposal :) | 16:26 |
bauwser | rpodolyaka: you're asking to change a lot more than needed | 16:26 |
rpodolyaka | yeah, I get it :) | 16:26 |
*** zenoway has joined #openstack-nova | 16:26 | |
bauwser | rpodolyaka: see http://paste.openstack.org/show/590494/ I think it could do the trick | 16:26 |
bauwser | rpodolyaka: I can even write the test :) | 16:28 |
rpodolyaka | bauwser: ^ pass `nova boot` on my devstack + psql | 16:28 |
bauwser | \o/ | 16:28 |
*** zenoway has quit IRC | 16:30 | |
rpodolyaka | bauwser: tdurakov uploaded a depends-on change to ceilometer when reviewing my patch - https://review.openstack.org/#/c/402524/ | 16:30 |
rpodolyaka | we might try something like that for this change as well | 16:30 |
bauwser | rpodolyaka: yup, was planning to | 16:31 |
*** sahid has quit IRC | 16:31 | |
*** ducttape_ has quit IRC | 16:31 | |
cfriesen | so how did the postgres-breaking change pass the tests? if there's something we weren't testing before are we adding that test? | 16:32 |
bauwser | cfriesen: we don't gate on postgre | 16:33 |
johnthetubaguy | cfriesen: I don't think Nova or integrated gates on postgres and more, but some folks do check it | 16:33 |
cfriesen | okay, that's kind of scary | 16:33 |
johnthetubaguy | it wasn't being kept green in the past, I think thats why it was dropped, but I am not certain now | 16:34 |
bauwser | yup that | 16:35 |
sfinucan | bauwser: right, so now I'm up to speed | 16:36 |
* sfinucan didn't know that MySQL would coerce something in the first place. Two new things for the day | 16:38 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix instance creation problem with postgre https://review.openstack.org/403035 | 16:39 |
bauwser | sfinucan: johnthetubaguy: cfriesen: rpodolyaka | 16:40 |
bauwser | : ^ | 16:40 |
bauwser | the soft_delete call in the PersistentMixin object does the trick | 16:40 |
johnthetubaguy | bauwser: oh... so thats the change that broke things, that makes more sense to me now | 16:41 |
bauwser | johnthetubaguy: I know we're out of core reviewers, but we could maybe assume that due to the criticity of the bug and if people like sfinucan, rpodolyaka and cfriesen give them go, we could get enough confidence to merge that patch with a slighter rule ? | 16:41 |
bauwser | I can take responsibility of the +W if you will | 16:42 |
*** markvoelker has quit IRC | 16:42 | |
sfinucan | bauwser: Sure, I'll stack and try that now | 16:42 |
sfinucan | bauwser, rpodolyaka: Any chance either of you have a local.conf for postgres DevStack install? | 16:43 |
bauwser | sfinucan: I failed for that | 16:43 |
rpodolyaka | sfinucan: I basically installed postgresql on my devstack and create nova and nova_api dbs | 16:43 |
rpodolyaka | but it's pretty much broken at this point, so it should not be trusted 100% :P | 16:44 |
sfinucan | rpodolyaka: Sounds fair :) Time to build it from scratch so | 16:44 |
johnthetubaguy | bauwser: could we get a postgres test run on that patch somehow? | 16:46 |
bauwser | johnthetubaguy: a Depends-On will do the trick | 16:46 |
*** prateek has quit IRC | 16:46 | |
bauwser | on some Tempest or Ceilometer patch | 16:46 |
johnthetubaguy | bauwser: right, just need doing | 16:46 |
*** ayoung has joined #openstack-nova | 16:46 | |
bauwser | I'm on | 16:46 |
johnthetubaguy | cool | 16:46 |
bauwser | a Tempest one would go my favor | 16:46 |
johnthetubaguy | if we get a +1 there, I feel less bad about approving it | 16:46 |
bauwser | because that's where we got the most falires | 16:47 |
bauwser | failures | 16:47 |
bauwser | I can try to find danpb | 16:47 |
cfriesen | just added a review note | 16:49 |
bauwser | cfriesen: I saw | 16:49 |
bauwser | cfriesen: you familiar with how we persist changes in the object models ? | 16:49 |
bauwser | ie. the difference between a dirty and a non-dirty bit ? | 16:50 |
*** sandanar has quit IRC | 16:50 | |
cfriesen | enough, I think. I understand what you're trying to say, I just don't think the comment is clearly written. | 16:50 |
*** hfu has quit IRC | 16:50 | |
*** links has joined #openstack-nova | 16:51 | |
*** pilgrimstack has joined #openstack-nova | 16:51 | |
*** pilgrimstack1 has quit IRC | 16:52 | |
cfriesen | ie "We never want to persist the value of the deleted field but only when destroying the instance." is not a proper sentance. | 16:52 |
bauwser | cfriesen: I think you entered the French-to-English grey area full of badly written phrases | 16:53 |
cfriesen | maybe "We only ever want to persist the value of the deleted field when destroying the instance." ? | 16:53 |
cfriesen | I think it actually makes sense grammatically in French. :) | 16:53 |
bauwser | cfriesen: you does speak French ? oh bon sang. | 16:54 |
cfriesen | Jai apris le francais dans l'ecole primaire, et jai vecu dans un pays francophone. | 16:55 |
cfriesen | mais je ne le parle pas souvant | 16:55 |
bauwser | :) | 16:55 |
sfinucan | bauwser: For context, dirty bits refer to changes in the objects that haven't been propagated to the DB? | 16:56 |
bauwser | sfinucan: yup, that aren't in sync | 16:56 |
*** tbachman has joined #openstack-nova | 16:56 | |
cfriesen | sfinucan: no, he's talking about nsfw stuff | 16:56 |
*** tbachman has quit IRC | 16:56 | |
sfinucan | bauwser: Cool. Same as the Django ORM so | 16:56 |
sfinucan | lol | 16:56 |
bauwser | ie. you touch a field, it sets it as needing to be persisted | 16:56 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: [scheduler][tests]: Fix incorrect aggr mock values https://review.openstack.org/402910 | 16:56 |
bauwser | cfriesen: bits have a very clear nsfw meaning in French, I guess you know | 16:57 |
cfriesen | english too. :) | 16:57 |
bauwser | but I'm diverging | 16:57 |
bauwser | I'm just about to propose my first ceilo patch \o/ | 16:57 |
*** matrohon has quit IRC | 16:57 | |
*** cnf has joined #openstack-nova | 16:58 | |
*** TravT has joined #openstack-nova | 17:01 | |
bauwser | johnthetubaguy: https://review.openstack.org/#/c/403101/ is there for testing postgre | 17:01 |
*** TravT has quit IRC | 17:03 | |
*** ralonsoh has quit IRC | 17:04 | |
*** lucasagomes is now known as lucas-afk | 17:04 | |
johnthetubaguy | cfriesen: I found the email about postgresql http://lists.openstack.org/pipermail/openstack-dev/2016-August/101892.html | 17:04 |
*** WillUK2010 has quit IRC | 17:04 | |
*** TravT has joined #openstack-nova | 17:04 | |
johnthetubaguy | bauwser: cool :) | 17:04 |
bauwser | okay, that's confirmed, we don't really care of passing the "deleted" value | 17:07 |
bauwser | when destroying the instance, we call the soft_delete() method from oslo.db that does set the deleted field to the ID | 17:07 |
bauwser | johnthetubaguy: and when we get the instance from the DB, we just do the boolean check you showed to me | 17:08 |
bauwser | so, yeah, nothing really needing to be updated | 17:08 |
openstackgerrit | Pavel Kholkin proposed openstack/nova: Adds view builders for keypairs controller https://review.openstack.org/347289 | 17:09 |
*** mnestratov has quit IRC | 17:10 | |
bauwser | johnthetubaguy: and when we .save() the instance, we compare and swap so we don't really change the value for deleted | 17:12 |
*** Jeffrey4l has quit IRC | 17:12 | |
sfinucan | bauwser: Can you put *all* of that into the comment in https://review.openstack.org/#/c/403035 | 17:12 |
sfinucan | Might be overly prescriptive, but who cares | 17:12 |
bauwser | sfinucan: I can | 17:12 |
bauwser | in comments, you mean ? | 17:13 |
sfinucan | Yup | 17:13 |
bauwser | okay | 17:13 |
*** salv-orlando has joined #openstack-nova | 17:13 | |
johnthetubaguy | bauwser: feels like deleted should be a read_only=true field | 17:13 |
openstackgerrit | int32bit proposed openstack/nova: Add lock record to instance action list https://review.openstack.org/396510 | 17:14 |
*** links has quit IRC | 17:15 | |
bauwser | sfinucan: comments done | 17:18 |
*** zz_dimtruck is now known as dimtruck | 17:18 | |
bauwser | johnthetubaguy: yeah | 17:18 |
*** claudiub|2 has joined #openstack-nova | 17:18 | |
bauwser | do we have that argument in a field type ? | 17:18 |
*** rmart04 has quit IRC | 17:19 | |
bauwser | I wasn't knowing it | 17:19 |
bauwser | oh yes we have it | 17:19 |
johnthetubaguy | bauwser: I thought I saw something like that | 17:19 |
bauwser | fun, never noticed it | 17:19 |
bauwser | I'm looking at o.vo to see what's done with that | 17:19 |
*** claudiub has quit IRC | 17:21 | |
*** hogepodge has joined #openstack-nova | 17:24 | |
*** lpetrut has quit IRC | 17:26 | |
*** dimtruck is now known as zz_dimtruck | 17:28 | |
bauwser | johnthetubaguy: okay, I think we could constraint more the deleted field https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L76-L77 | 17:28 |
bauwser | without requiring some backwards check | 17:28 |
bauwser | but that can be deferred next week | 17:29 |
*** matrohon has joined #openstack-nova | 17:29 | |
*** pilgrimstack has quit IRC | 17:31 | |
*** pcaruana has quit IRC | 17:33 | |
*** TravT has quit IRC | 17:37 | |
*** cdent has joined #openstack-nova | 17:41 | |
bauwser | WTF... the ceilo gate isn't passing on my Depends-On... | 17:42 |
bauwser | wait, wait, it's calling an ec2_instance_create ?! | 17:44 |
*** shuquan has joined #openstack-nova | 17:45 | |
bauwser | johnthetubaguy: rpodolyaka: ^ | 17:46 |
bauwser | I totally missed that, but that's not calling at all the same DB SQLA API... | 17:47 |
*** victorhe has quit IRC | 17:47 | |
*** victorhe has joined #openstack-nova | 17:47 | |
bauwser | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L1809 | 17:49 |
*** zz_dimtruck is now known as dimtruck | 17:49 | |
*** shuquan has quit IRC | 17:49 | |
*** portdirect is now known as portdirect_away | 17:51 | |
*** paul-carlton has quit IRC | 17:54 | |
*** ducttape_ has joined #openstack-nova | 17:56 | |
*** yamahata has joined #openstack-nova | 17:57 | |
*** dimtruck is now known as zz_dimtruck | 17:59 | |
bauwser | rpodolyaka: johnthetubaguy: okay, I'm pretty sure we need to not set the deleted field... | 18:04 |
*** rama_y has quit IRC | 18:09 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Fix instance creation problem with postgre https://review.openstack.org/403035 | 18:09 |
*** victorhe has quit IRC | 18:09 | |
*** victorhe has joined #openstack-nova | 18:10 | |
bauwser | johnthetubaguy: rpodolyaka: cfriesen: sfinucan: ^ updated change | 18:10 |
*** lpetrut has joined #openstack-nova | 18:11 | |
sfinucan | bauwser: Still waiting on my VM to stack - it failed first time round :( (old, non-fernet'y DevStack) | 18:11 |
bauwser | yeah I had a problem too, I needed to update to 16.04.1 | 18:11 |
bauwser | but given I only have a 5Mbps b/w, ... | 18:11 |
bauwser | holy f*** | 18:13 |
sfinucan | bauwser: I've 120 MB, but DevStack is sloooowwww | 18:13 |
*** ljxiash has quit IRC | 18:13 | |
bauwser | I made a mistake with my Depends-On tag ! | 18:13 |
bauwser | GRAAAAAAAAAAAAH | 18:13 |
*** ljxiash has joined #openstack-nova | 18:14 | |
sfinucan | bauwser: That made me lol ever so slightly | 18:14 |
bauwser | so, I tested the ceilometer patch with a wrong dependson | 18:14 |
bauwser | so, I got the same error | 18:15 |
bauwser | graaaah | 18:15 |
bauwser | now I provided a new PS, that should be fine | 18:15 |
bauwser | I'm getting mad | 18:15 |
*** armax has joined #openstack-nova | 18:16 | |
diana_clarke | bauwser: Isn't it wine o'clock in France by now? | 18:17 |
bauwser | unfortunately yes, but the gate is borken | 18:17 |
bauwser | borked even | 18:17 |
*** ljxiash has quit IRC | 18:18 | |
*** raildo has quit IRC | 18:19 | |
bauwser | I'm waiting for the ceilometer canary patch to pass and then \o | 18:19 |
*** amoralej is now known as amoralej|off | 18:23 | |
kukacz | hi, I'm looking for some description of how `nova usage` calculates the consumption data | 18:24 |
kukacz | could someone point me at anything helpful, pls? | 18:25 |
diana_clarke | kukacz: I would start here: https://github.com/openstack/nova/blob/7188bd01ec0264f6efee1ee5014a9edf71a57e5b/nova/api/openstack/compute/simple_tenant_usage.py#L100 | 18:28 |
*** salv-orl_ has joined #openstack-nova | 18:30 | |
*** yamahata has quit IRC | 18:32 | |
*** salv-orlando has quit IRC | 18:32 | |
kukacz | diana_clarke: thanks, I was hoping for something higher level :-) | 18:35 |
*** claudiub has joined #openstack-nova | 18:35 | |
kukacz | this is beyond my skills | 18:35 |
kukacz | maybe if I just ask - do I understand correctly it's pure Nova, no dependency on Ceilometer? | 18:36 |
diana_clarke | kukacz: yup | 18:37 |
*** claudiub|2 has quit IRC | 18:38 | |
kukacz | diana_clarke: great. and does it take into account also deleted instances and not-running states during the reported period? | 18:38 |
*** zz_dimtruck is now known as dimtruck | 18:39 | |
sean-k-mooney | kukacz: it looks like this is being comuted form the db so deleted instances will still be there so i think it will but im reading the code for the first time now | 18:40 |
diana_clarke | kukacz: the stats are bases on an instances query with a WHERE clause like the following: WHERE (instances.terminated_at IS NULL OR instances.terminated_at > '2016-09-28 21:02:51') AND instances.launched_at < '2016-09-28 21:02:51'; | 18:40 |
*** ducttape_ has quit IRC | 18:40 | |
sfinucan | bauwser: Right, that's stacked. Is it any use to you or are you just waiting on ceilometer? | 18:40 |
*** ihrachys has joined #openstack-nova | 18:40 | |
sean-k-mooney | diana_clarke: so it get any instces that are still running or that were terminated in the window specifed in the query | 18:42 |
kukacz | diana_clarke: looks good. so it calculates the statistics on query execution, right? it's not based on some internal counter being increased over lifetime ... | 18:42 |
kukacz | lifetime - I meant instance runtime | 18:43 |
*** moshele has joined #openstack-nova | 18:43 | |
*** armax has quit IRC | 18:44 | |
diana_clarke | kukacz: no internal counters, just that arithmetic I pointed to earlier based on instances active in the time window | 18:45 |
kukacz | diana_clarke: ok. when I download the usage report from Horizon, it includes instances whose utilization in summary is less than what 'nova usage' gives | 18:47 |
kukacz | so I'm trying to find the cause. if perhaps deleted instances play a role or ... | 18:48 |
kukacz | now, when looking at it, it seems it's about those deleted instances | 18:50 |
diana_clarke | kukacz: Horizon consumes the python-novaclient usage API, traverses the data and filters it a bit I think. Here's that code: http://paste.openstack.org/show/590496/ | 18:50 |
*** ihrachys has quit IRC | 18:50 | |
kukacz | diana_clarke: greate source, thanks! I'll try to go through it to confirm | 18:51 |
robcresswell | There are a couple of people looking at that code in horizon at the moment, so if there are bugs its a good time to mention it to horizon folk too. | 18:51 |
* robcresswell goes back to lurking :) | 18:51 | |
kukacz | diana_clarke: btw. don't you know if there's something similar to `nova usage` in Cinder? like a volume usage report | 18:51 |
diana_clarke | kukacz: I don't know, sorry. | 18:51 |
*** lpetrut has quit IRC | 18:51 | |
kukacz | diana_clarke: nevermind. thank you a lot for this help! | 18:52 |
*** matrohon has quit IRC | 18:55 | |
*** ducttape_ has joined #openstack-nova | 19:02 | |
kukacz | diana_clarke: you saved my day! found in that Horizon code, that appending "show_terminated=TRUE" (switched to Kilo) in the URL will send the report including deleted instances | 19:02 |
kukacz | then the report is 1:1 precise with 'nova usage' | 19:03 |
diana_clarke | kukacz: nice! | 19:03 |
*** salv-orl_ has quit IRC | 19:05 | |
*** ducttape_ has quit IRC | 19:06 | |
*** ducttape_ has joined #openstack-nova | 19:07 | |
*** victorhe has quit IRC | 19:11 | |
*** ducttape_ has quit IRC | 19:11 | |
*** hogepodge has quit IRC | 19:11 | |
*** victorhe has joined #openstack-nova | 19:12 | |
*** moshele has quit IRC | 19:13 | |
*** lpetrut has joined #openstack-nova | 19:16 | |
*** cdent has quit IRC | 19:17 | |
*** cdent has joined #openstack-nova | 19:17 | |
*** umbSublime_ is now known as Lord_balancer | 19:21 | |
*** Lord_balancer is now known as HIDE_YO_PINGS_HI | 19:22 | |
*** HIDE_YO_PINGS_HI is now known as Lord_balancer | 19:22 | |
*** mgoddard_ has joined #openstack-nova | 19:23 | |
*** mgoddard has quit IRC | 19:27 | |
*** mnestratov has joined #openstack-nova | 19:27 | |
*** mgoddard_ has quit IRC | 19:29 | |
*** shuquan has joined #openstack-nova | 19:33 | |
*** gjayavelu has joined #openstack-nova | 19:33 | |
*** shuquan has quit IRC | 19:38 | |
*** giri has joined #openstack-nova | 19:40 | |
*** gjayavelu has quit IRC | 19:42 | |
kukacz | diana_clarke: if you don't mind I got one more - is it so that a shutdown instance is also counted into the usage? | 19:55 |
diana_clarke | kukacz: Sorry, I don't know all the details. Just that usage iterates over the instances in a start-end window according to the WHERE clause I pasted above. | 19:57 |
kukacz | diana_clarke: ok, thanks. I'm thinking of a billing usecase, so these states also play a role | 19:58 |
*** claudiub|2 has joined #openstack-nova | 20:02 | |
diana_clarke | kukacz: I would probably caution against using this endpoint for billing. It's probably called "simple" tenant usage as a warning. | 20:03 |
*** shuquan has joined #openstack-nova | 20:04 | |
*** giri has quit IRC | 20:04 | |
*** claudiub has quit IRC | 20:05 | |
*** ihrachys has joined #openstack-nova | 20:05 | |
*** beagles_afk is now known as beagles | 20:06 | |
kukacz | diana_clarke: probably yes. I just got hard time finding anything better | 20:07 |
*** shuquan has quit IRC | 20:08 | |
*** hogepodge has joined #openstack-nova | 20:08 | |
*** markvoelker has joined #openstack-nova | 20:20 | |
*** markvoelker has quit IRC | 20:25 | |
*** salv-orlando has joined #openstack-nova | 20:36 | |
bauwser | johnthetubaguy: still around ? | 20:36 |
bauwser | johnthetubaguy: https://review.openstack.org/#/c/403035/2 is passing for https://review.openstack.org/#/c/403101/ | 20:36 |
*** slaweq_ has quit IRC | 20:36 | |
*** slaweq_ has joined #openstack-nova | 20:37 | |
*** salv-orlando has quit IRC | 20:40 | |
*** slaweq_ has quit IRC | 20:42 | |
*** slaweq_ has joined #openstack-nova | 20:44 | |
*** ducttape_ has joined #openstack-nova | 20:44 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: config options: improve libvirt utils https://review.openstack.org/377740 | 20:45 |
*** TravT has joined #openstack-nova | 20:58 | |
*** clenimar has quit IRC | 20:58 | |
*** slaweq_ has quit IRC | 20:59 | |
*** slaweq_ has joined #openstack-nova | 21:01 | |
*** salv-orlando has joined #openstack-nova | 21:01 | |
*** ducttape_ has quit IRC | 21:02 | |
*** WillUK2010 has joined #openstack-nova | 21:10 | |
*** TravT has quit IRC | 21:19 | |
*** slaweq_ has quit IRC | 21:19 | |
*** slaweq_ has joined #openstack-nova | 21:19 | |
*** slaweq_ has quit IRC | 21:23 | |
*** ducttape_ has joined #openstack-nova | 21:24 | |
*** ducttape_ has quit IRC | 21:25 | |
*** dimtruck is now known as zz_dimtruck | 21:25 | |
*** fragatina has quit IRC | 21:31 | |
*** fragatina has joined #openstack-nova | 21:31 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: config options: improve libvirt lvm https://review.openstack.org/378303 | 21:32 |
*** mnestratov has quit IRC | 21:36 | |
*** cfriesen has quit IRC | 21:38 | |
*** oanson has quit IRC | 21:45 | |
*** gjayavelu has joined #openstack-nova | 21:49 | |
*** cdent has quit IRC | 21:58 | |
*** cdent has joined #openstack-nova | 21:59 | |
*** thorst_ has joined #openstack-nova | 22:02 | |
*** thorst_ has quit IRC | 22:05 | |
*** thorst_ has joined #openstack-nova | 22:06 | |
*** shuquan has joined #openstack-nova | 22:10 | |
*** shuquan has quit IRC | 22:15 | |
*** thorst_ has quit IRC | 22:16 | |
*** slaweq_ has joined #openstack-nova | 22:16 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 22:31 |
*** cdent has quit IRC | 22:31 | |
openstackgerrit | Maciej Szankin proposed openstack/nova: conf: Move api options to a group https://review.openstack.org/384490 | 22:32 |
*** catintheroof has joined #openstack-nova | 22:34 | |
*** hogepodge has quit IRC | 22:40 | |
*** vladikr has quit IRC | 22:41 | |
*** armax has joined #openstack-nova | 22:49 | |
*** hogepodge has joined #openstack-nova | 22:51 | |
*** dave-mccowan has joined #openstack-nova | 22:57 | |
*** armax has quit IRC | 22:57 | |
*** victorhe has quit IRC | 23:02 | |
*** slaweq_ has quit IRC | 23:04 | |
*** salv-orlando has quit IRC | 23:05 | |
*** slaweq_ has joined #openstack-nova | 23:05 | |
*** lpetrut has quit IRC | 23:13 | |
*** thorst_ has joined #openstack-nova | 23:14 | |
*** ducttape_ has joined #openstack-nova | 23:15 | |
*** slaweq_ has quit IRC | 23:16 | |
*** ducttape_ has quit IRC | 23:17 | |
*** ducttape_ has joined #openstack-nova | 23:17 | |
*** _ducttape_ has joined #openstack-nova | 23:20 | |
*** ducttape_ has quit IRC | 23:20 | |
*** thorst_ has quit IRC | 23:21 | |
*** _ducttape_ has quit IRC | 23:22 | |
*** ducttape_ has joined #openstack-nova | 23:23 | |
*** mlavalle has joined #openstack-nova | 23:23 | |
*** cnf has left #openstack-nova | 23:23 | |
*** ducttape_ has quit IRC | 23:27 | |
*** ihrachys has quit IRC | 23:36 | |
*** erlon-airlong has quit IRC | 23:42 | |
*** unicell1 has quit IRC | 23:48 | |
*** zz_dimtruck is now known as dimtruck | 23:49 | |
*** gongysh has joined #openstack-nova | 23:56 | |
*** rfolco has joined #openstack-nova | 23:58 | |
*** rfolco has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!