| *** 16WAALNTP has quit IRC | 00:01 | |
| *** zz_dimtruck is now known as dimtruck | 00:01 | |
| *** Marga_ has joined #openstack-nova | 00:01 | |
| *** haomaiwang has joined #openstack-nova | 00:01 | |
| *** Marga_ has quit IRC | 00:02 | |
| *** Marga_ has joined #openstack-nova | 00:03 | |
| *** Marga_ has quit IRC | 00:03 | |
| *** Marga_ has joined #openstack-nova | 00:03 | |
| *** vilobhmm11 has quit IRC | 00:06 | |
| *** ZZelle_ has quit IRC | 00:06 | |
| *** vilobhmm11 has joined #openstack-nova | 00:06 | |
| mriedem | mikal: tonyb: mitaka and liberty backports for a regression that's been around since liberty https://review.openstack.org/#/q/I9ba58707d03d19018a025d7760f2a77f84d23aad,n,z | 00:08 |
|---|---|---|
| mriedem | clean backports, should be an easy +W | 00:08 |
| tonyb | mriedem: cool. I'll look at them in a few minutes | 00:09 |
| *** diogogmt has quit IRC | 00:14 | |
| *** bjornar has quit IRC | 00:22 | |
| *** mtanino has quit IRC | 00:25 | |
| *** claudiub|2 has quit IRC | 00:27 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Fix import statement order in nova/rpc.py https://review.openstack.org/301986 | 00:27 |
| *** sdake has joined #openstack-nova | 00:32 | |
| *** sdake_ has quit IRC | 00:34 | |
| *** Sukhdev has quit IRC | 00:35 | |
| *** takashin has joined #openstack-nova | 00:39 | |
| mriedem | dansmith: are we waiting on anything before approving https://review.openstack.org/#/c/300127/ ? | 00:39 |
| mriedem | besides the devstack dependency | 00:39 |
| dansmith | nope just devstack | 00:40 |
| mriedem | alright, +W coming up | 00:40 |
| dansmith | cool | 00:40 |
| *** dimtruck is now known as zz_dimtruck | 00:43 | |
| *** Jeffrey4l has joined #openstack-nova | 00:47 | |
| *** zz_dimtruck is now known as dimtruck | 00:49 | |
| *** vilobhmm11 has quit IRC | 00:49 | |
| *** dave-mcc_ has joined #openstack-nova | 00:50 | |
| *** sdake_ has joined #openstack-nova | 00:51 | |
| *** sdake has quit IRC | 00:51 | |
| *** dave-mccowan has quit IRC | 00:51 | |
| *** baoli has joined #openstack-nova | 00:52 | |
| *** knikolla has quit IRC | 00:56 | |
| *** Oku_OS-away has quit IRC | 00:59 | |
| *** Oku_OS-away has joined #openstack-nova | 00:59 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** haomaiwang has joined #openstack-nova | 01:01 | |
| *** mylu has quit IRC | 01:03 | |
| *** jwcroppe has joined #openstack-nova | 01:06 | |
| *** erlon has quit IRC | 01:07 | |
| *** diogogmt has joined #openstack-nova | 01:07 | |
| *** zenoway has joined #openstack-nova | 01:07 | |
| *** zzzeek_ has quit IRC | 01:10 | |
| *** zenoway has quit IRC | 01:12 | |
| *** jwcroppe has quit IRC | 01:12 | |
| *** zzzeek has joined #openstack-nova | 01:13 | |
| *** rmevans has joined #openstack-nova | 01:14 | |
| *** yamahata has joined #openstack-nova | 01:15 | |
| *** Swami has quit IRC | 01:17 | |
| *** mriedem has quit IRC | 01:20 | |
| *** akshai has joined #openstack-nova | 01:21 | |
| *** akshai_ has joined #openstack-nova | 01:22 | |
| *** thorst_ has quit IRC | 01:22 | |
| *** haomaiwang has quit IRC | 01:23 | |
| *** thorst has joined #openstack-nova | 01:23 | |
| *** unicell1 has quit IRC | 01:25 | |
| *** akshai has quit IRC | 01:25 | |
| *** thorst has quit IRC | 01:31 | |
| *** salv-orlando has joined #openstack-nova | 01:32 | |
| *** EinstCrazy has joined #openstack-nova | 01:34 | |
| *** Jeffrey4l has quit IRC | 01:34 | |
| *** apoorvad has quit IRC | 01:35 | |
| *** mylu has joined #openstack-nova | 01:38 | |
| *** ducttape_ has joined #openstack-nova | 01:38 | |
| *** ljxiash has joined #openstack-nova | 01:39 | |
| *** zenoway has joined #openstack-nova | 01:40 | |
| *** Jeffrey4l has joined #openstack-nova | 01:42 | |
| *** jamielennox is now known as jamielennox|away | 01:43 | |
| *** rmevans has quit IRC | 01:43 | |
| *** zenoway has quit IRC | 01:44 | |
| *** hoangcx has quit IRC | 01:46 | |
| *** hoangcx has joined #openstack-nova | 01:46 | |
| *** woodster_ has quit IRC | 01:47 | |
| *** salv-orlando has quit IRC | 01:47 | |
| *** yonglihe has joined #openstack-nova | 01:48 | |
| *** ducttape_ has quit IRC | 01:53 | |
| *** jamielennox|away is now known as jamielennox | 01:53 | |
| openstackgerrit | Wenzhi Yu proposed openstack/nova: Check if a exception has a code on it before read the code https://review.openstack.org/300751 | 01:53 |
| *** sdake has joined #openstack-nova | 01:55 | |
| *** busterswt has joined #openstack-nova | 01:55 | |
| *** _ducttape_ has joined #openstack-nova | 01:56 | |
| *** sdake_ has quit IRC | 01:56 | |
| *** bustersw_ has joined #openstack-nova | 01:57 | |
| *** houming has joined #openstack-nova | 01:59 | |
| *** EinstCra_ has joined #openstack-nova | 02:01 | |
| *** busterswt has quit IRC | 02:01 | |
| *** ljxiash has quit IRC | 02:03 | |
| *** ljxiash has joined #openstack-nova | 02:03 | |
| *** zenoway has joined #openstack-nova | 02:04 | |
| *** EinstCrazy has quit IRC | 02:04 | |
| *** ljxiash has quit IRC | 02:08 | |
| *** lixiaoy1 has quit IRC | 02:08 | |
| *** zenoway has quit IRC | 02:08 | |
| *** haomaiwang has joined #openstack-nova | 02:13 | |
| *** mylu has quit IRC | 02:16 | |
| *** mylu has joined #openstack-nova | 02:18 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Re-Propose Expose quiesce/unquiesce API https://review.openstack.org/295595 | 02:20 |
| *** zzzeek has quit IRC | 02:21 | |
| *** zzzeek has joined #openstack-nova | 02:23 | |
| *** Kevin_Zheng has joined #openstack-nova | 02:24 | |
| *** pwnall has quit IRC | 02:24 | |
| mikal | mriedem: I don't have +W on stable/mitaka | 02:28 |
| *** edmondsw has quit IRC | 02:29 | |
| *** dimtruck is now known as zz_dimtruck | 02:29 | |
| *** thorst has joined #openstack-nova | 02:29 | |
| mikal | tonyb: you should review https://review.openstack.org/#/q/I9ba58707d03d19018a025d7760f2a77f84d23aad,n,z | 02:29 |
| tonyb | mikal: Yessir! (FWIW: mriedem pointed me at them earlier) | 02:32 |
| mikal | Yes, but you didn't thing the do | 02:32 |
| mikal | And I'm not core, so I'm not very helpful | 02:32 |
| *** bustersw_ has quit IRC | 02:33 | |
| *** achanda has joined #openstack-nova | 02:34 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-novaclient: Fix typos in docstrings and comments https://review.openstack.org/301993 | 02:34 |
| *** jwcroppe has joined #openstack-nova | 02:35 | |
| *** thorst has quit IRC | 02:36 | |
| *** zenoway has joined #openstack-nova | 02:40 | |
| *** tobe has joined #openstack-nova | 02:42 | |
| *** tobe has quit IRC | 02:42 | |
| *** mylu has quit IRC | 02:44 | |
| *** zenoway has quit IRC | 02:44 | |
| *** lixiaoy1 has joined #openstack-nova | 02:45 | |
| *** unicell has joined #openstack-nova | 02:49 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Support "locked" filter when list servers https://review.openstack.org/292749 | 02:49 |
| *** mylu has joined #openstack-nova | 02:49 | |
| *** ljxiash has joined #openstack-nova | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-nova | 03:01 | |
| *** sdake_ has joined #openstack-nova | 03:01 | |
| *** salv-orlando has joined #openstack-nova | 03:03 | |
| *** sdake has quit IRC | 03:04 | |
| *** _ducttape_ has quit IRC | 03:05 | |
| *** mylu has quit IRC | 03:10 | |
| *** jwcroppe_ has joined #openstack-nova | 03:11 | |
| *** hoangcx has quit IRC | 03:11 | |
| *** hoangcx has joined #openstack-nova | 03:11 | |
| *** mylu has joined #openstack-nova | 03:13 | |
| *** salv-orlando has quit IRC | 03:13 | |
| *** mylu has quit IRC | 03:14 | |
| *** coreywright has quit IRC | 03:14 | |
| *** mylu has joined #openstack-nova | 03:15 | |
| *** jwcroppe has quit IRC | 03:15 | |
| *** zenoway has joined #openstack-nova | 03:15 | |
| *** zenoway has quit IRC | 03:20 | |
| *** markvoelker has quit IRC | 03:26 | |
| *** achanda has quit IRC | 03:27 | |
| *** hoangcx has quit IRC | 03:29 | |
| *** aghadge has joined #openstack-nova | 03:30 | |
| *** aghadge has quit IRC | 03:33 | |
| *** ljxiash has quit IRC | 03:34 | |
| *** thorst has joined #openstack-nova | 03:34 | |
| *** ljxiash has joined #openstack-nova | 03:34 | |
| *** coreywright has joined #openstack-nova | 03:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Replace assert statements in iptables and virt tests https://review.openstack.org/282992 | 03:37 |
| openstackgerrit | John Wood proposed openstack/nova: Ensure snapshot min_disk tracks instance root_gb https://review.openstack.org/301966 | 03:39 |
| *** ljxiash has quit IRC | 03:40 | |
| *** hoangcx has joined #openstack-nova | 03:40 | |
| *** mylu has quit IRC | 03:41 | |
| *** thorst has quit IRC | 03:41 | |
| *** mylu has joined #openstack-nova | 03:41 | |
| *** mylu has quit IRC | 03:42 | |
| openstackgerrit | Allen Gao proposed openstack/nova: config options: centralize baseproxy cli options https://review.openstack.org/300715 | 03:42 |
| *** mylu has joined #openstack-nova | 03:42 | |
| openstackgerrit | Allen Gao proposed openstack/nova: Use IPOpt and PortOpt in vnc options https://review.openstack.org/300716 | 03:42 |
| *** baoli has quit IRC | 03:43 | |
| openstackgerrit | Atul Agarwal proposed openstack/nova-specs: Add compute node on the go https://review.openstack.org/301552 | 03:43 |
| *** vilobhmm11 has joined #openstack-nova | 03:45 | |
| openstackgerrit | Allen Gao proposed openstack/nova: config options: centralize quota options https://review.openstack.org/300940 | 03:45 |
| *** coreywright has quit IRC | 03:48 | |
| *** anush_ has joined #openstack-nova | 03:49 | |
| *** zenoway has joined #openstack-nova | 03:51 | |
| *** links has joined #openstack-nova | 03:52 | |
| *** zenoway has quit IRC | 03:56 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** 6A4AAM2CW has joined #openstack-nova | 04:01 | |
| *** hoangcx has quit IRC | 04:02 | |
| *** hoangcx has joined #openstack-nova | 04:02 | |
| *** coreywright has joined #openstack-nova | 04:05 | |
| *** bogdano has quit IRC | 04:06 | |
| *** bogdano has joined #openstack-nova | 04:07 | |
| *** ljxiash has joined #openstack-nova | 04:13 | |
| *** jwcroppe_ has quit IRC | 04:13 | |
| *** jwcroppe has joined #openstack-nova | 04:14 | |
| *** jwcroppe_ has joined #openstack-nova | 04:17 | |
| *** jwcroppe has quit IRC | 04:19 | |
| *** hoangcx has quit IRC | 04:20 | |
| *** hoangcx has joined #openstack-nova | 04:21 | |
| *** jamielennox is now known as jamielennox|away | 04:21 | |
| *** markvoelker has joined #openstack-nova | 04:26 | |
| *** zenoway has joined #openstack-nova | 04:26 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/302016 | 04:29 |
| *** jamielennox|away is now known as jamielennox | 04:29 | |
| *** zenoway has quit IRC | 04:31 | |
| *** salv-orlando has joined #openstack-nova | 04:31 | |
| *** markvoelker has quit IRC | 04:32 | |
| *** diogogmt has quit IRC | 04:33 | |
| *** salv-orlando has quit IRC | 04:39 | |
| *** thorst has joined #openstack-nova | 04:39 | |
| *** edtubill has joined #openstack-nova | 04:41 | |
| *** woodster_ has joined #openstack-nova | 04:44 | |
| *** thorst has quit IRC | 04:47 | |
| *** browne has joined #openstack-nova | 04:51 | |
| *** mylu has quit IRC | 04:51 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize libvirt options https://review.openstack.org/302030 | 04:53 |
| *** jaosorior has joined #openstack-nova | 04:55 | |
| *** achanda has joined #openstack-nova | 05:00 | |
| *** 6A4AAM2CW has quit IRC | 05:01 | |
| *** armax has quit IRC | 05:01 | |
| *** anush_ has quit IRC | 05:01 | |
| *** jamielennox is now known as jamielennox|away | 05:01 | |
| *** haomaiwa_ has joined #openstack-nova | 05:01 | |
| *** Sukhdev has joined #openstack-nova | 05:03 | |
| *** browne has quit IRC | 05:07 | |
| *** mylu has joined #openstack-nova | 05:08 | |
| *** jamielennox|away is now known as jamielennox | 05:11 | |
| *** mylu has quit IRC | 05:12 | |
| *** browne has joined #openstack-nova | 05:14 | |
| *** prateek has joined #openstack-nova | 05:14 | |
| *** norman has joined #openstack-nova | 05:15 | |
| *** browne has quit IRC | 05:18 | |
| *** jamielennox is now known as jamielennox|away | 05:22 | |
| *** jwcroppe_ has quit IRC | 05:25 | |
| *** ljxiash has quit IRC | 05:26 | |
| *** ljxiash has joined #openstack-nova | 05:26 | |
| *** akshai_ has quit IRC | 05:28 | |
| *** krtaylor has quit IRC | 05:29 | |
| *** jamielennox|away is now known as jamielennox | 05:30 | |
| *** sekrit has quit IRC | 05:30 | |
| *** ljxiash has quit IRC | 05:31 | |
| *** krtaylor has joined #openstack-nova | 05:31 | |
| *** ljxiash has joined #openstack-nova | 05:32 | |
| *** ChrisBenson has joined #openstack-nova | 05:32 | |
| *** Marga_ has quit IRC | 05:34 | |
| *** nkrinner has joined #openstack-nova | 05:35 | |
| *** ekuris has joined #openstack-nova | 05:37 | |
| *** Marga_ has joined #openstack-nova | 05:37 | |
| *** Marga_ has quit IRC | 05:38 | |
| *** Marga_ has joined #openstack-nova | 05:38 | |
| *** sdake has joined #openstack-nova | 05:40 | |
| *** sdake_ has quit IRC | 05:42 | |
| *** Marga_ has quit IRC | 05:43 | |
| *** liverpooler has quit IRC | 05:44 | |
| *** thorst has joined #openstack-nova | 05:44 | |
| *** salv-orlando has joined #openstack-nova | 05:46 | |
| *** moshele has joined #openstack-nova | 05:46 | |
| *** sdake_ has joined #openstack-nova | 05:49 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text https://review.openstack.org/302034 | 05:49 |
| *** prateek has quit IRC | 05:49 | |
| *** prateek has joined #openstack-nova | 05:51 | |
| *** Marga_ has joined #openstack-nova | 05:52 | |
| *** prateek has quit IRC | 05:52 | |
| *** thorst has quit IRC | 05:52 | |
| *** sdake has quit IRC | 05:52 | |
| *** claudiub|2 has joined #openstack-nova | 05:53 | |
| *** prateek has joined #openstack-nova | 05:54 | |
| *** salv-orlando has quit IRC | 05:55 | |
| *** Marga_ has quit IRC | 05:56 | |
| *** dave-mcc_ has quit IRC | 05:56 | |
| *** qeelee has joined #openstack-nova | 05:57 | |
| *** rcernin has joined #openstack-nova | 05:59 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** Marga_ has joined #openstack-nova | 06:01 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (1) https://review.openstack.org/302034 | 06:01 |
| *** haomaiwang has joined #openstack-nova | 06:01 | |
| *** fawadkhaliq has joined #openstack-nova | 06:02 | |
| *** Sukhdev has quit IRC | 06:08 | |
| *** ChrisBenson has quit IRC | 06:11 | |
| *** aswadr_ has joined #openstack-nova | 06:14 | |
| *** scheuran has joined #openstack-nova | 06:20 | |
| *** fawadkhaliq has quit IRC | 06:20 | |
| *** josecastroleon has joined #openstack-nova | 06:21 | |
| *** mjura has joined #openstack-nova | 06:23 | |
| *** sekrit has joined #openstack-nova | 06:23 | |
| *** lpetrut has joined #openstack-nova | 06:24 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [POC] Allow user defined stop https://review.openstack.org/298221 | 06:24 |
| *** apoorv has joined #openstack-nova | 06:24 | |
| *** zenoway has joined #openstack-nova | 06:25 | |
| *** edtubill has quit IRC | 06:28 | |
| *** jwcroppe has joined #openstack-nova | 06:28 | |
| *** markvoelker has joined #openstack-nova | 06:29 | |
| *** rk4n has joined #openstack-nova | 06:29 | |
| *** belmoreira has joined #openstack-nova | 06:30 | |
| *** zenoway has quit IRC | 06:30 | |
| openstackgerrit | Tomi Juvonen proposed openstack/nova-specs: Get servers filtered by host status https://review.openstack.org/276671 | 06:32 |
| *** dave-mcc_ has joined #openstack-nova | 06:33 | |
| *** markvoelker has quit IRC | 06:35 | |
| *** jwcroppe has quit IRC | 06:35 | |
| openstackgerrit | esracelik proposed openstack/nova: Config options: centralize "configdrive" options https://review.openstack.org/254092 | 06:36 |
| *** claudiub has joined #openstack-nova | 06:37 | |
| *** jamielennox is now known as jamielennox|away | 06:38 | |
| openstackgerrit | esracelik proposed openstack/nova: Improve the help text for configdrive options https://review.openstack.org/266875 | 06:40 |
| *** claudiub|2 has quit IRC | 06:40 | |
| openstackgerrit | Jiajun Liu proposed openstack/nova: allow detach interface from a soft deleted instance https://review.openstack.org/302052 | 06:41 |
| *** sridharg has joined #openstack-nova | 06:43 | |
| *** woodster_ has quit IRC | 06:47 | |
| *** achanda has quit IRC | 06:47 | |
| *** thorst has joined #openstack-nova | 06:49 | |
| *** dave-mccowan has joined #openstack-nova | 06:49 | |
| *** esracelik has joined #openstack-nova | 06:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata https://review.openstack.org/301478 | 06:51 |
| *** achanda has joined #openstack-nova | 06:53 | |
| *** zenoway has joined #openstack-nova | 06:53 | |
| *** dave-mcc_ has quit IRC | 06:54 | |
| *** rook-lappy has quit IRC | 06:56 | |
| *** thorst has quit IRC | 06:57 | |
| *** rook-lappy has joined #openstack-nova | 06:57 | |
| *** garyk has joined #openstack-nova | 06:59 | |
| *** achanda has quit IRC | 07:00 | |
| *** jaosorior has quit IRC | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-nova | 07:01 | |
| *** openstackgerrit has quit IRC | 07:02 | |
| *** openstackgerrit has joined #openstack-nova | 07:03 | |
| *** claudiub|2 has joined #openstack-nova | 07:04 | |
| *** sheel has joined #openstack-nova | 07:05 | |
| *** Marga_ has quit IRC | 07:05 | |
| *** liverpooler has joined #openstack-nova | 07:05 | |
| *** pcaruana has joined #openstack-nova | 07:06 | |
| *** ihrachys has joined #openstack-nova | 07:06 | |
| *** jaosorior has joined #openstack-nova | 07:07 | |
| *** claudiub has quit IRC | 07:08 | |
| *** rcernin has quit IRC | 07:08 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: improve libvirt help text (2) https://review.openstack.org/302060 | 07:08 |
| *** rcernin has joined #openstack-nova | 07:09 | |
| *** vilobhmm11 has quit IRC | 07:10 | |
| *** houming has quit IRC | 07:11 | |
| *** paul-carlton2 has quit IRC | 07:11 | |
| openstackgerrit | esracelik proposed openstack/nova: Improve the help text for configdrive options https://review.openstack.org/266875 | 07:11 |
| *** houming has joined #openstack-nova | 07:12 | |
| *** salv-orlando has joined #openstack-nova | 07:12 | |
| *** rook-lappy has quit IRC | 07:14 | |
| *** paul-carlton has quit IRC | 07:16 | |
| *** dedery has joined #openstack-nova | 07:17 | |
| *** dedery has quit IRC | 07:17 | |
| *** dedery has joined #openstack-nova | 07:17 | |
| *** paul-carlton has joined #openstack-nova | 07:21 | |
| *** jichen has joined #openstack-nova | 07:22 | |
| *** jaosorior has quit IRC | 07:22 | |
| *** jaosorior has joined #openstack-nova | 07:22 | |
| *** mgoddard has joined #openstack-nova | 07:25 | |
| *** femnad has joined #openstack-nova | 07:26 | |
| *** lpetrut has quit IRC | 07:27 | |
| *** bjornar has joined #openstack-nova | 07:31 | |
| *** jlanoux has joined #openstack-nova | 07:32 | |
| bauzas | good morning Nova | 07:32 |
| *** paul-carlton2 has joined #openstack-nova | 07:34 | |
| *** femnad has quit IRC | 07:35 | |
| *** femnad has joined #openstack-nova | 07:36 | |
| *** Oku_OS-away is now known as Oku_OS | 07:38 | |
| *** vilobhmm11 has joined #openstack-nova | 07:39 | |
| *** gcb has quit IRC | 07:41 | |
| *** jaosorior has quit IRC | 07:47 | |
| *** jaosorior has joined #openstack-nova | 07:47 | |
| *** unicell has quit IRC | 07:47 | |
| *** rdo has quit IRC | 07:47 | |
| *** gcb has joined #openstack-nova | 07:53 | |
| *** thorst has joined #openstack-nova | 07:55 | |
| *** sfinucan has joined #openstack-nova | 07:58 | |
| *** mgoddard_ has joined #openstack-nova | 07:59 | |
| *** takashin has left #openstack-nova | 07:59 | |
| *** ndipanov has joined #openstack-nova | 07:59 | |
| *** achanda has joined #openstack-nova | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-nova | 08:01 | |
| *** thorst has quit IRC | 08:02 | |
| *** mgoddard has quit IRC | 08:02 | |
| *** prateek is now known as prateek|lunch | 08:04 | |
| *** daemontool has joined #openstack-nova | 08:04 | |
| jichen | alex_xu: do you plan to work on https://github.com/openstack/api-wg/blob/master/guidelines/errors-example.json? I also want to contribute to it , any bp submitted? | 08:05 |
| *** achanda has quit IRC | 08:05 | |
| *** apoorv has quit IRC | 08:05 | |
| alex_xu | jichen: I guess no, I say I would give a help if this is priority item in the newton, but finally the team decided this won't be a priority item | 08:06 |
| *** paul-carlton has quit IRC | 08:07 | |
| jichen | alex_xu: ok, I see, but not a priority work doesn't mean we can't submit a bp or try to do some work on it? | 08:07 |
| alex_xu | jichen: yes, I also think about that | 08:08 |
| jichen | alex_xu: ok, I will see whether I can write a bp ,thanks | 08:08 |
| *** jsheeren has joined #openstack-nova | 08:08 | |
| alex_xu | jichen: did you see a email from sdague about api prority items? i remember sdague also mention it depend on a api ref support, this is something you should notice | 08:09 |
| jsheeren | hey all, i have a weird issue with the openstack-client and the nova command | 08:09 |
| jsheeren | if i do "openstack hypervisor list" i get my compute nodes; if i do "nova hypervisor-list" i get: "ERROR (AttributeError): 'unicode' object has no attribute 'get'" | 08:09 |
| jsheeren | anyone seen this before? | 08:09 |
| jichen | alex_xu: ok, I see the mail but I might miss the dependency item, will take a look at that again, thx~ | 08:10 |
| alex_xu | jichen: np | 08:10 |
| openstackgerrit | Akira YOSHIYAMA proposed openstack/nova: WIP: ironic: add console-log capability https://review.openstack.org/302084 | 08:10 |
| openstackgerrit | Akira YOSHIYAMA proposed openstack/nova: WIP: ironic: Add serial console capability https://review.openstack.org/302085 | 08:10 |
| *** atuvenie has joined #openstack-nova | 08:10 | |
| jsheeren | i pulled the openstack-client from github; the nova command points to the nova in the venv of the openstack-client | 08:10 |
| *** paul-carlton has joined #openstack-nova | 08:11 | |
| *** artom has joined #openstack-nova | 08:14 | |
| *** rdo has joined #openstack-nova | 08:15 | |
| *** jistr has joined #openstack-nova | 08:22 | |
| *** speller has joined #openstack-nova | 08:23 | |
| bauzas | alex_xu: aloha | 08:26 |
| *** brad[] has quit IRC | 08:26 | |
| bauzas | alex_xu: I can search for docs, but maybe you can quickly reply me on that | 08:26 |
| alex_xu | bauzas: yea, anything I can help? | 08:26 |
| *** ndipanov has quit IRC | 08:26 | |
| *** brad[] has joined #openstack-nova | 08:27 | |
| bauzas | alex_xu: if a change modifies a HTTP error code, it needs a microversion, right? | 08:27 |
| bauzas | alex_xu: eg. changing the raised exception | 08:27 |
| alex_xu | bauzas: yes, I think so except 500 | 08:27 |
| bauzas | alex_xu: https://review.openstack.org/#/c/301907/1/nova/api/openstack/compute/legacy_v2/contrib/security_groups.py | 08:27 |
| speller | I'm getting this bug on Liberty - https://bugs.launchpad.net/nova/+bug/1418298 | 08:28 |
| openstack | Launchpad bug 1418298 in OpenStack Compute (nova) "After service deleted, the corresponding compute-node can't restart again" [High,Fix released] - Assigned to Alex Xu (xuhj) | 08:28 |
| speller | DBDuplicateEntry when trying to compute_node_create() | 08:28 |
| speller | anyone saw something like that in Liberty? | 08:28 |
| bauzas | alex_xu: so, I'll -1 for many points, but here for example, the reporter wants to modify the behavior by returing HTTP 413 instead of 409 | 08:28 |
| alex_xu | bauzas: yes, I think -1 it is right thing | 08:29 |
| *** ndipanov has joined #openstack-nova | 08:29 | |
| bauzas | alex_xu: plus the fact he wants to change the stable API, which is impossible | 08:30 |
| alex_xu | bauzas: yes, good point | 08:30 |
| bauzas | alex_xu: okay, I'll try to figure out the docs | 08:30 |
| *** markvoelker has joined #openstack-nova | 08:30 | |
| bauzas | alex_xu: for commenting my -1 | 08:30 |
| alex_xu | bauzas: :) | 08:31 |
| *** bharathk has joined #openstack-nova | 08:32 | |
| bauzas | alex_xu: can only find http://developer.openstack.org/api-guide/compute/microversions.html docs, but that's not very handy | 08:33 |
| bauzas | alex_xu: do we have a better pointer for this ? | 08:33 |
| *** jwcroppe has joined #openstack-nova | 08:33 | |
| bauzas | oh http://docs.openstack.org/developer/nova/stable_api.html | 08:33 |
| alex_xu | bauzas: no, that is the only one, or you can use this first https://review.openstack.org/301978 | 08:33 |
| *** yassine has joined #openstack-nova | 08:34 | |
| *** yassine is now known as Guest9848 | 08:34 | |
| bauzas | alex_xu: okay | 08:34 |
| *** Guest9848 is now known as yassou | 08:35 | |
| *** markvoelker has quit IRC | 08:35 | |
| *** e0ne has joined #openstack-nova | 08:36 | |
| *** jwcroppe has quit IRC | 08:38 | |
| *** norman has quit IRC | 08:38 | |
| *** haukebruno has joined #openstack-nova | 08:39 | |
| bauzas | alex_xu: thanks | 08:39 |
| openstackgerrit | Abhilash Goyal proposed openstack/nova: Case-sensitive neutron security groups https://review.openstack.org/302095 | 08:39 |
| alex_xu | bauzas: np | 08:39 |
| *** paul-carlton has quit IRC | 08:39 | |
| *** vilobhmm11 has quit IRC | 08:40 | |
| *** mfedosin has joined #openstack-nova | 08:42 | |
| *** dave-mccowan has quit IRC | 08:43 | |
| *** paul-carlton has joined #openstack-nova | 08:43 | |
| *** apoorv has joined #openstack-nova | 08:45 | |
| *** jsheeren has quit IRC | 08:45 | |
| *** salv-orlando has quit IRC | 08:45 | |
| *** derekh has joined #openstack-nova | 08:46 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: [POC] Allow user defined stop https://review.openstack.org/298221 | 08:48 |
| *** danpb has joined #openstack-nova | 08:49 | |
| *** PaulMurray has joined #openstack-nova | 08:49 | |
| *** apoorv has quit IRC | 08:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs: Add support for non-admin users to query their own deleted instances https://review.openstack.org/292769 | 08:53 |
| *** lixiaoy1 has quit IRC | 08:54 | |
| alex_xu | markus_z: you are right on https://bugs.launchpad.net/nova/+bug/1541691, it is fixed | 08:56 |
| openstack | Launchpad bug 1541691 in OpenStack Compute (nova) "server boot with leading and trailing white spaces in name, displays weird error message" [High,Fix released] - Assigned to Alex Xu (xuhj) | 08:56 |
| markus_z | alex_xu: cool, thanks for double-checking | 08:57 |
| alex_xu | markus_z: np | 08:57 |
| *** thorst has joined #openstack-nova | 09:00 | |
| *** e0ne has quit IRC | 09:00 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** lpetrut has joined #openstack-nova | 09:01 | |
| *** haomaiwang has joined #openstack-nova | 09:01 | |
| *** kashyap has joined #openstack-nova | 09:02 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova: Improve 'workarounds' conf options documentation https://review.openstack.org/301077 | 09:02 |
| sfinucan | johnthetubaguy: danpb: Could I get a rebase +W for this? https://review.openstack.org/#/c/301077/ | 09:04 |
| *** itamarl has joined #openstack-nova | 09:04 | |
| sfinucan | ...please and thank you | 09:04 |
| johnthetubaguy | sfinucan: sure thing | 09:04 |
| *** Marga_ has joined #openstack-nova | 09:05 | |
| *** thorst has quit IRC | 09:06 | |
| *** prateek|lunch is now known as prateek | 09:10 | |
| *** prateek is now known as parora | 09:10 | |
| *** Marga_ has quit IRC | 09:10 | |
| *** cdent has joined #openstack-nova | 09:13 | |
| openstackgerrit | Matthew Booth proposed openstack/nova-specs: Persist libvirt instance storage metadata https://review.openstack.org/302117 | 09:14 |
| *** salv-orlando has joined #openstack-nova | 09:15 | |
| *** ljxiash has quit IRC | 09:15 | |
| *** mjura has quit IRC | 09:17 | |
| *** ljxiash has joined #openstack-nova | 09:17 | |
| *** apoorv has joined #openstack-nova | 09:18 | |
| *** salv-orlando has quit IRC | 09:19 | |
| *** wuhg has joined #openstack-nova | 09:20 | |
| openstackgerrit | Matthew Booth proposed openstack/nova-specs: Persist libvirt instance storage metadata https://review.openstack.org/302117 | 09:20 |
| *** claudiub has joined #openstack-nova | 09:21 | |
| *** ljxiash has quit IRC | 09:22 | |
| *** claudiub|2 has quit IRC | 09:24 | |
| *** haomaiwang has quit IRC | 09:24 | |
| cdent | p!spy 1 | 09:26 |
| *** mjura has joined #openstack-nova | 09:30 | |
| *** mgoddard_ has quit IRC | 09:31 | |
| *** achanda has joined #openstack-nova | 09:33 | |
| *** mgoddard has joined #openstack-nova | 09:37 | |
| *** achanda has quit IRC | 09:38 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: docs: Remove warnings from "api-ref" https://review.openstack.org/302129 | 09:38 |
| *** mjura_ has joined #openstack-nova | 09:41 | |
| openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: docs: Remove warnings from "api-ref" https://review.openstack.org/302129 | 09:41 |
| *** mjura has quit IRC | 09:43 | |
| *** mjura_ has quit IRC | 09:45 | |
| *** mjura has joined #openstack-nova | 09:46 | |
| *** bjornar has quit IRC | 09:46 | |
| *** ujjain has quit IRC | 09:47 | |
| *** hieulq has joined #openstack-nova | 09:49 | |
| allen_gao | johnthetubaguy: markus_z , there is a high prio bug back port for mitaka, any tune to +w? thanks. https://review.openstack.org/#/c/301349/ | 09:54 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Doc: Add rough plan for cellsv2 upgrade scenarios https://review.openstack.org/267153 | 09:57 |
| *** BobBall has joined #openstack-nova | 09:57 | |
| *** ljxiash has joined #openstack-nova | 09:58 | |
| *** EinstCra_ is now known as EinstCrazy | 10:00 | |
| *** mirrorbox has quit IRC | 10:00 | |
| *** mirrorbox has joined #openstack-nova | 10:00 | |
| *** haomaiwa_ has joined #openstack-nova | 10:01 | |
| *** salv-orlando has joined #openstack-nova | 10:02 | |
| *** sdake_ has quit IRC | 10:02 | |
| *** salv-orlando has quit IRC | 10:03 | |
| *** salv-orlando has joined #openstack-nova | 10:03 | |
| *** thorst has joined #openstack-nova | 10:04 | |
| *** e0ne has joined #openstack-nova | 10:06 | |
| *** bjornar has joined #openstack-nova | 10:07 | |
| *** ekarlso- has quit IRC | 10:09 | |
| *** sdague has joined #openstack-nova | 10:09 | |
| *** hieulq has quit IRC | 10:10 | |
| *** thorst has quit IRC | 10:11 | |
| *** EinstCrazy has quit IRC | 10:12 | |
| *** artom has quit IRC | 10:14 | |
| *** salv-orlando has quit IRC | 10:18 | |
| *** claudiub|2 has joined #openstack-nova | 10:18 | |
| -openstackstatus- NOTICE: npm lint jobs are failing due to a problem with npm registry. The problem is under investigation, and we will update once the issue is solved. | 10:18 | |
| *** ChanServ changes topic to "npm lint jobs are failing due to a problem with npm registry. The problem is under investigation, and we will update once the issue is solved." | 10:18 | |
| *** hoangcx has quit IRC | 10:19 | |
| *** hoangcx has joined #openstack-nova | 10:19 | |
| * danpb wonders why openstackstatus feels the need change the nova channel topic | 10:21 | |
| *** claudiub has quit IRC | 10:21 | |
| *** ekarlso- has joined #openstack-nova | 10:21 | |
| *** ekarlso- has quit IRC | 10:22 | |
| *** ekarlso has joined #openstack-nova | 10:22 | |
| *** mvk_ has quit IRC | 10:23 | |
| *** salv-orlando has joined #openstack-nova | 10:26 | |
| openstackgerrit | Hieu LE proposed openstack/nova: Config options: centralize libvirt options https://review.openstack.org/302030 | 10:27 |
| *** ujjain has joined #openstack-nova | 10:29 | |
| *** ujjain has quit IRC | 10:29 | |
| *** ujjain has joined #openstack-nova | 10:29 | |
| *** salv-orlando has quit IRC | 10:30 | |
| *** cdent has quit IRC | 10:30 | |
| *** bjornar has quit IRC | 10:31 | |
| *** markvoelker has joined #openstack-nova | 10:31 | |
| *** achanda has joined #openstack-nova | 10:35 | |
| *** markvoelker has quit IRC | 10:36 | |
| *** jwcroppe has joined #openstack-nova | 10:36 | |
| *** gcb has quit IRC | 10:37 | |
| *** gszasz has joined #openstack-nova | 10:38 | |
| *** gcb has joined #openstack-nova | 10:38 | |
| *** ljxiash has quit IRC | 10:39 | |
| markus_z | allen_gao: AFAIK the release gets tagged tomorrow, maybe they want to omit this change. I'll ask mriedem later. | 10:40 |
| *** jwcroppe has quit IRC | 10:41 | |
| *** achanda has quit IRC | 10:41 | |
| markus_z | bauzas: You wanted to get a new ping for https://bugs.launchpad.net/nova/+bug/1477098 | 10:41 |
| openstack | Launchpad bug 1477098 in OpenStack Compute (nova) "When you unshelve an instance, AggregateImagePropertiesIsolation filter doesn't work." [Undecided,New] | 10:41 |
| *** gcb has quit IRC | 10:43 | |
| *** gcb has joined #openstack-nova | 10:44 | |
| *** salv-orlando has joined #openstack-nova | 10:47 | |
| *** salv-orlando has quit IRC | 10:48 | |
| *** ljxiash has joined #openstack-nova | 10:50 | |
| *** hieulq has joined #openstack-nova | 10:52 | |
| *** rodrigods has quit IRC | 10:52 | |
| *** rodrigods has joined #openstack-nova | 10:53 | |
| *** ljxiash has quit IRC | 10:54 | |
| markus_z | BobBall: Maybe interesting for you: https://bugs.launchpad.net/nova/+bug/1566622 Looks like a change from you has an upgrade impact. | 10:56 |
| openstack | Launchpad bug 1566622 in OpenStack Compute (nova) "live migration fails with xenapi virt driver and SRs with old-style naming convention" [Undecided,New] | 10:56 |
| *** salv-orlando has joined #openstack-nova | 10:57 | |
| *** salv-orlando has quit IRC | 10:57 | |
| *** salv-orlando has joined #openstack-nova | 10:57 | |
| *** salv-orlando has quit IRC | 10:57 | |
| *** salv-orlando has joined #openstack-nova | 10:58 | |
| *** salv-orlando has quit IRC | 10:58 | |
| *** itamarl has quit IRC | 10:58 | |
| *** itamarl has joined #openstack-nova | 10:58 | |
| *** cdent has joined #openstack-nova | 10:58 | |
| *** claudiub has joined #openstack-nova | 10:59 | |
| zioproto | hello I have this review https://review.openstack.org/#/c/295865/ that is failing the gate-grenade-dsvm. However replying with recheck: gate-grenade-dsvm is not triggering the recheck. Am I doing somewthing wrong ? | 10:59 |
| *** paul-carlton1 has joined #openstack-nova | 10:59 | |
| *** zioproto has quit IRC | 11:00 | |
| *** gongysh has quit IRC | 11:00 | |
| *** haomaiwa_ has quit IRC | 11:01 | |
| *** 18VAAKXIZ has joined #openstack-nova | 11:01 | |
| *** zioproto has joined #openstack-nova | 11:02 | |
| *** claudiub|2 has quit IRC | 11:02 | |
| zioproto | sorry I closed my client by mistake :) | 11:03 |
| markus_z | zioproto: I left a comment in your review | 11:03 |
| zioproto | markus_z: oh thanks, so just 'recheck' without the explicit info on which job, Thank you | 11:04 |
| markus_z | zioproto: yep, also depending on the CI. Xen uses "xen: recheck" for example. | 11:04 |
| allen_gao | markus_z: got it, thank you. | 11:05 |
| PaulMurray | eliqiao, I've taken over "convert consoles code to use objects framework" : https://review.openstack.org/#/c/301158/ | 11:05 |
| zioproto | markus_z: oh ok, so I guess you learn all this shorcuts over time. I used xen: recheck in the past, that is why I thought they all worked that way | 11:05 |
| PaulMurray | eliqiao, planning to resurrect a patch based on your abandoned https://review.openstack.org/#/c/177078/ | 11:06 |
| *** gongysh has joined #openstack-nova | 11:06 | |
| PaulMurray | eliqiao, let me know if you are interested | 11:06 |
| markus_z | zioproto: yeah, it's not 100% consistent over the different CIs. Usually there will be a comment from the CI in your review which also tells how to restart things. | 11:06 |
| BobBall | markus_z: Thanks - already on my radar. Just got a few other urgent things going on then I'll be having a look. | 11:07 |
| BobBall | markus_z: But FYI 'xenapi' was the right tag just to distinguish between libvirt+xen and the XenAPI driver :) | 11:07 |
| markus_z | BobBall: oh, OK, I didn't know that. What is "xenserver" then used for? | 11:08 |
| BobBall | markus_z: My bad - I just realised - most bugs are indeed tagged XenServer rather than XenAPI | 11:08 |
| BobBall | which is confusing, since other products (e.g Inspur's hypervisor) also use XenAPI and are not Citrix's "XenServer" ;) | 11:09 |
| *** thorst has joined #openstack-nova | 11:09 | |
| markus_z | BobBall: is it important for you to differ the bug reports with "xen" and "xenserver"/"xenapi"? If not, we could just plainly use "xen" and someone is watching this and then ping the right people for it. | 11:10 |
| BobBall | markus_z: Yeah - its different people. My team knows comparatively little about libvirt+xen | 11:11 |
| *** vtech has joined #openstack-nova | 11:13 | |
| johnthetubaguy | markus_z: BobBall: yeah, I think we need that distinction | 11:13 |
| johnthetubaguy | its very different code that gets hit | 11:13 |
| markus_z | Got it. "xenapi" == "xenserver". "xen" == libvirt + xen | 11:14 |
| BobBall | Spot on | 11:14 |
| *** houming has quit IRC | 11:15 | |
| *** sahid has joined #openstack-nova | 11:15 | |
| *** Marga_ has joined #openstack-nova | 11:15 | |
| markus_z | anthonyper: Are you then still the right contact for libvirt+xen issues? Your name is at https://wiki.openstack.org/wiki/Nova/BugTriage#Tags | 11:15 |
| *** hieulq_ has joined #openstack-nova | 11:15 | |
| *** coreywright has quit IRC | 11:16 | |
| *** Marga_ has quit IRC | 11:16 | |
| anthonyper | markus_z: Yes. | 11:16 |
| *** thorst has quit IRC | 11:16 | |
| *** Marga_ has joined #openstack-nova | 11:16 | |
| markus_z | ok, cool. I'll update the wiki to make that difference clear. | 11:17 |
| *** hieulq has quit IRC | 11:17 | |
| *** gmann_ has joined #openstack-nova | 11:19 | |
| *** claudiub|2 has joined #openstack-nova | 11:20 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Py3: fix serial console output https://review.openstack.org/302182 | 11:20 |
| *** thorst has joined #openstack-nova | 11:20 | |
| *** claudiub has quit IRC | 11:22 | |
| *** coreywright has joined #openstack-nova | 11:24 | |
| openstackgerrit | Matthew Booth proposed openstack/nova-specs: Persist libvirt instance storage metadata https://review.openstack.org/302117 | 11:24 |
| *** gongysh has quit IRC | 11:25 | |
| *** coreywright has quit IRC | 11:25 | |
| *** Goneri has quit IRC | 11:25 | |
| *** coreywright has joined #openstack-nova | 11:25 | |
| markus_z | anthonyper: That one should be closed as "invalid", shouldn't it? https://bugs.launchpad.net/nova/+bug/1443898 | 11:26 |
| openstack | Launchpad bug 1443898 in OpenStack Compute (nova) "TestVolumeBootPattern.test_volume_boot_pattern fail with libvirt-xen" [Low,Incomplete] | 11:26 |
| *** gongysh has joined #openstack-nova | 11:27 | |
| *** rtheis has joined #openstack-nova | 11:29 | |
| *** paul-carlton1 has quit IRC | 11:30 | |
| *** ccesario has quit IRC | 11:30 | |
| coreywright | markus_z: yeah, stupid me for tagging the nova bug with "nova" | 11:33 |
| markus_z | coreywright: no worries :) | 11:33 |
| *** ljxiash has joined #openstack-nova | 11:33 | |
| coreywright | BobBall: yeah, i didn't know whether to use "xen", "xenserver", or "xenapi" (as in the nova driver, not the dom0 interface), so i (unfortunately) used the most generic | 11:33 |
| coreywright | so, thanks all for straightening out my bug report tags | 11:33 |
| *** ljxiash has quit IRC | 11:37 | |
| bauzas | markus_z: aloha | 11:38 |
| *** gmann_ has quit IRC | 11:38 | |
| bauzas | markus_z: looking at https://bugs.launchpad.net/nova/+bug/1477098 | 11:38 |
| openstack | Launchpad bug 1477098 in OpenStack Compute (nova) "When you unshelve an instance, AggregateImagePropertiesIsolation filter doesn't work." [Undecided,New] | 11:38 |
| markus_z | bauzas: hey ho | 11:38 |
| *** mvk_ has joined #openstack-nova | 11:38 | |
| bauzas | markus_z: oh "environments: Icehouse" | 11:39 |
| anthonyper | markus_z: that may still be an issue. But I guess a wontfix would be more appropriate since it's about an older api (there is BDMv2 and this bug is about BDMv1). We do skip the test in the xenproject ci loop | 11:39 |
| markus_z | bauzas: right, but I thought I saw someone commenting that it happens also on a newer version | 11:40 |
| markus_z | anthonyper: So BDMv1 is not supported anymore? If yes, I agree to "won't fix". | 11:40 |
| bauzas | markus_z: reading the comments | 11:41 |
| *** speller has quit IRC | 11:41 | |
| markus_z | bauzas: Hm, checked again, didn't find it, I must have been mistaken. | 11:41 |
| anthonyper | markus_z: I can not remember if it can work with libvirt+xen | 11:41 |
| markus_z | bauzas: Just from a high-level-view, is the expectation of the bug report correct? That the instance should be on an aggregate with the same metadata? | 11:43 |
| bauzas | markus_z: I need to review the filter code | 11:43 |
| *** mastermind has joined #openstack-nova | 11:46 | |
| *** eheglsz has joined #openstack-nova | 11:46 | |
| *** gmann_ has joined #openstack-nova | 11:46 | |
| *** eheglsz has quit IRC | 11:47 | |
| *** zenoway has quit IRC | 11:47 | |
| *** eheglsz has joined #openstack-nova | 11:47 | |
| *** eheglsz has quit IRC | 11:47 | |
| markus_z | anthonyper: Is the tempest test case then omitted for libvirt-xen? | 11:47 |
| *** speller has joined #openstack-nova | 11:48 | |
| *** laszloh has joined #openstack-nova | 11:48 | |
| bauzas | markus_z: so, I think it's not a bug | 11:49 |
| markus_z | bauzas: why so? | 11:49 |
| bauzas | markus_z: if the host has no metadata attached to the aggregate it belongs to, the filter considers this destination as valid | 11:50 |
| *** doug-fish has joined #openstack-nova | 11:50 | |
| bauzas | we could change that behaviour, but that's then a behaviour change, not a bug | 11:50 |
| bauzas | markus_z: see https://github.com/openstack/nova/blob/master/nova/scheduler/filters/aggregate_image_properties_isolation.py#L44 | 11:51 |
| bauzas | if the metadata list is empty, then the filter returns True | 11:51 |
| markus_z | bauzas: Does this mean the hosts need the same metadata as the aggregate to achieve the result the bug reporter is asking for? | 11:52 |
| bauzas | but before telling that, I'd love to get more info on the target aggregate | 11:52 |
| bauzas | markus_z: the host doesn't have metadata, that's the aggregate | 11:52 |
| *** busterswt has joined #openstack-nova | 11:55 | |
| *** aysyd has joined #openstack-nova | 11:55 | |
| markus_z | bauzas: Please correct me if I understand the code wrong: | 11:57 |
| markus_z | bauzas: If a host doesn't have information attached that it belongs to an aggregate, it passes | 11:57 |
| markus_z | bauzas: If a host is attached to an aggregate which matches the image properties, it passes | 11:58 |
| markus_z | bauzas: Is that right? | 11:58 |
| bauzas | the latter, yup | 11:58 |
| bauzas | oh sec | 11:58 |
| markus_z | I though both | 11:58 |
| bauzas | lemme rephrase it | 11:58 |
| bauzas | if a host is attached to an aggregate that doesn't have metadata, or is not attached to an aggregate, the filter passes | 11:58 |
| markus_z | yep, OK, that's my understanding as well | 11:59 |
| *** takedakn has joined #openstack-nova | 11:59 | |
| *** busterswt has quit IRC | 11:59 | |
| markus_z | bauzas: The bug reporter said "two or more aggregates exist.one of which has same metadata as image's Property." <= that's the issue, right? It's possible that the second aggregate doesn't have metadata. | 12:00 |
| *** 18VAAKXIZ has quit IRC | 12:01 | |
| markus_z | He expects an enforcement, but the filters don't work that way | 12:01 |
| *** haomaiwang has joined #openstack-nova | 12:01 | |
| bauzas | if an host is attached to an aggregate having metadata, the filter verifies each metadata key if it uses a namespace, and if so, matches if the image property named by the keyname matches with one of the values in the value of the metadata key | 12:01 |
| bauzas | markus_z: yup, so I'm asking for more details | 12:02 |
| mdbooth | markus_z: Re image cache manager: http://lists.openstack.org/pipermail/openstack-dev/2016-April/091539.html | 12:02 |
| bauzas | markus_z: basically, the filter has a debug level that complains if a key can't be found | 12:02 |
| markus_z | bauzas: OK, cool, got it now. I agree that there could be a wrong expectation on the reporters side and we should ask for details. | 12:02 |
| *** garyk has quit IRC | 12:02 | |
| *** al_indig_ has joined #openstack-nova | 12:03 | |
| markus_z | mdbooth: Oh, right, I owe a ML discussion there. Let me read what you wrote. | 12:04 |
| *** hieulq_ has quit IRC | 12:05 | |
| *** rfolco has joined #openstack-nova | 12:05 | |
| *** al_indigo has quit IRC | 12:06 | |
| *** Marga_ has quit IRC | 12:07 | |
| markus_z | anthonyper: Would you leave a concluding comment in the bug report please? If a new bug report describes the same issue in the future, I'd like to have a reference with a reasoning. Otherwise the whole discussion starts again. | 12:07 |
| *** Jeffrey4l has quit IRC | 12:09 | |
| *** Marga_ has joined #openstack-nova | 12:09 | |
| *** salv-orlando has joined #openstack-nova | 12:10 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: HyperV: Nova serial console access support https://review.openstack.org/145004 | 12:13 |
| *** e0ne has quit IRC | 12:13 | |
| *** raildo-afk is now known as raildo | 12:14 | |
| markus_z | mdbooth: well written. Maybe it gets enough feedback to discuss it at the summit. | 12:14 |
| *** parora has quit IRC | 12:15 | |
| *** markvoelker has joined #openstack-nova | 12:15 | |
| *** baoli has joined #openstack-nova | 12:16 | |
| *** e0ne has joined #openstack-nova | 12:17 | |
| *** wuhg has quit IRC | 12:19 | |
| mnaser | morning! utilization aware scheduling is currently broken in liberty because a patch is missing, it's a clean backport and jenkins gave it the +1 .. would someone be able to have a look? https://review.openstack.org/#/c/300134/ | 12:19 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add Allocation and AllocationList objects https://review.openstack.org/282442 | 12:21 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add ResourcePool and ResourcePoolList objects https://review.openstack.org/284963 | 12:21 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add a get_by_uuid for aggregates https://review.openstack.org/284895 | 12:21 |
| openstackgerrit | Chris Dent proposed openstack/nova: Add name to ResourceProvider object https://review.openstack.org/281945 | 12:21 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 12:21 |
| *** baoli_ has joined #openstack-nova | 12:21 | |
| *** apoorv has quit IRC | 12:22 | |
| *** vladikr has quit IRC | 12:24 | |
| *** baoli has quit IRC | 12:24 | |
| *** vladikr has joined #openstack-nova | 12:25 | |
| *** pradk_ has joined #openstack-nova | 12:26 | |
| *** salv-orlando has quit IRC | 12:26 | |
| *** unicell has joined #openstack-nova | 12:27 | |
| *** andymaier has joined #openstack-nova | 12:30 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: make sure device relationships are kept in memory https://review.openstack.org/301859 | 12:32 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: objects: Add 2 relation fields to PciDevice https://review.openstack.org/301858 | 12:32 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: related updates are done without DB lookups https://review.openstack.org/301860 | 12:32 |
| *** ljxiash has joined #openstack-nova | 12:32 | |
| *** Marga_ has quit IRC | 12:36 | |
| openstackgerrit | Petrut Lucian proposed openstack/nova: Py3: fix serial console output https://review.openstack.org/302182 | 12:37 |
| *** achanda has joined #openstack-nova | 12:38 | |
| *** ociuhandu has quit IRC | 12:39 | |
| *** jwcroppe has joined #openstack-nova | 12:39 | |
| johnthetubaguy | sfinucan: I was just spotting that we haven't updated this test much yet: https://review.openstack.org/#/c/302030/2/nova/hacking/checks.py, did you fancy taking a quick look at that, based on how we are doing at the moment? | 12:43 |
| *** achanda has quit IRC | 12:43 | |
| johnthetubaguy | sfinucan: or maybe we just wait a little bit? | 12:43 |
| *** jwcroppe has quit IRC | 12:44 | |
| *** speller has quit IRC | 12:46 | |
| *** ccarmack has joined #openstack-nova | 12:47 | |
| openstackgerrit | Moshe Levi proposed openstack/nova: ironic driver: Adding InfiniBand support https://review.openstack.org/266540 | 12:48 |
| sfinucan | johnthetubaguy: Sure, I can do that | 12:48 |
| *** ljxiash has quit IRC | 12:48 | |
| *** tbachman_ has joined #openstack-nova | 12:50 | |
| *** tbachman has quit IRC | 12:51 | |
| *** tbachman_ is now known as tbachman | 12:51 | |
| * alex_xu reminders nova-api meeting in 10 mins | 12:51 | |
| *** jdillaman has joined #openstack-nova | 12:53 | |
| *** bharathk has quit IRC | 12:54 | |
| *** links has quit IRC | 12:56 | |
| *** speller has joined #openstack-nova | 12:57 | |
| *** ccesario has joined #openstack-nova | 12:58 | |
| * alex_xu reminders nova-api meeting is running | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** artom has joined #openstack-nova | 13:01 | |
| *** akshai has joined #openstack-nova | 13:01 | |
| *** haukebruno has quit IRC | 13:01 | |
| *** haomaiwang has joined #openstack-nova | 13:01 | |
| *** ociuhandu has joined #openstack-nova | 13:02 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/302016 | 13:02 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Enable to share PCI devices between NUMA nodes. https://review.openstack.org/302244 | 13:03 |
| *** ivase has joined #openstack-nova | 13:04 | |
| bauzas | alex_xu: where ? :) | 13:05 |
| bauzas | okay, found #openstack-meeting-4 | 13:06 |
| alex_xu | bauzas: yes | 13:06 |
| *** pwnall has joined #openstack-nova | 13:06 | |
| allen_gao | /join #openstack-meeting-4 | 13:06 |
| *** ivase_ has joined #openstack-nova | 13:07 | |
| *** rlrossit has joined #openstack-nova | 13:07 | |
| *** burgerk has joined #openstack-nova | 13:09 | |
| *** salv-orlando has joined #openstack-nova | 13:09 | |
| *** edmondsw has joined #openstack-nova | 13:09 | |
| *** ivase has quit IRC | 13:09 | |
| *** tbachman_ has joined #openstack-nova | 13:11 | |
| *** tbachman has quit IRC | 13:12 | |
| *** tbachman_ is now known as tbachman | 13:12 | |
| *** mrkz has joined #openstack-nova | 13:13 | |
| *** kgalanov has quit IRC | 13:13 | |
| *** carl_baldwin has quit IRC | 13:14 | |
| *** speller has quit IRC | 13:14 | |
| *** mdrabe has joined #openstack-nova | 13:14 | |
| *** sdake has joined #openstack-nova | 13:15 | |
| *** ctrath has joined #openstack-nova | 13:15 | |
| *** kgalanov has joined #openstack-nova | 13:15 | |
| *** carl_baldwin has joined #openstack-nova | 13:16 | |
| *** mriedem has joined #openstack-nova | 13:16 | |
| *** salv-orlando has quit IRC | 13:17 | |
| *** Goneri has joined #openstack-nova | 13:18 | |
| *** zz_dimtruck is now known as dimtruck | 13:18 | |
| *** gongysh has quit IRC | 13:18 | |
| *** sdake_ has joined #openstack-nova | 13:19 | |
| *** sdake has quit IRC | 13:19 | |
| *** speller has joined #openstack-nova | 13:20 | |
| *** gongysh has joined #openstack-nova | 13:20 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Check API versions intersects https://review.openstack.org/301278 | 13:20 |
| *** ccesario has quit IRC | 13:22 | |
| *** sdake_ has quit IRC | 13:24 | |
| *** artom has quit IRC | 13:24 | |
| johnthetubaguy | mriedem: that draft summit schedule is looking good | 13:25 |
| johnthetubaguy | mriedem: nice job | 13:25 |
| mriedem | \o/ | 13:25 |
| rlrossit | mriedem: you want more happiness? https://review.openstack.org/#/c/265951/ | 13:26 |
| rlrossit | all green baby | 13:26 |
| mriedem | don't call me baby | 13:26 |
| rlrossit | all green, hunksicle | 13:26 |
| rlrossit | that better? | 13:26 |
| mriedem | i don't think that's spelled correctly, but sure | 13:26 |
| mriedem | ah maybe it is, touche | 13:27 |
| rlrossit | http://www.urbandictionary.com/define.php?term=Hunksicle | 13:27 |
| *** sdake has joined #openstack-nova | 13:27 | |
| openstackgerrit | sahid proposed openstack/nova: network: make nova to handle port_security_enabled=False https://review.openstack.org/284095 | 13:28 |
| *** takedakn has quit IRC | 13:29 | |
| *** gongysh has quit IRC | 13:31 | |
| *** alex_xu has quit IRC | 13:33 | |
| *** gongysh has joined #openstack-nova | 13:34 | |
| *** jichen has quit IRC | 13:34 | |
| *** ametts has joined #openstack-nova | 13:35 | |
| *** alex_xu has joined #openstack-nova | 13:35 | |
| *** mtanino has joined #openstack-nova | 13:37 | |
| *** ljxiash has joined #openstack-nova | 13:37 | |
| *** qeelee has quit IRC | 13:38 | |
| *** matrohon has joined #openstack-nova | 13:38 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from InstanceAction https://review.openstack.org/301905 | 13:39 |
| *** READ10 has joined #openstack-nova | 13:39 | |
| *** amotoki has joined #openstack-nova | 13:39 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added db API layer to add instance tag-list filtering support https://review.openstack.org/248805 | 13:40 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags controller https://review.openstack.org/128940 | 13:40 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova: Added server tags support in nova-api https://review.openstack.org/268932 | 13:40 |
| *** gongysh has quit IRC | 13:41 | |
| *** eharney has quit IRC | 13:41 | |
| *** Jeffrey4l has joined #openstack-nova | 13:42 | |
| *** diogogmt has joined #openstack-nova | 13:42 | |
| *** atuvenie has quit IRC | 13:44 | |
| *** jaosorior has quit IRC | 13:44 | |
| dansmith | bauzas: can you comment on https://review.openstack.org/#/c/279313/13/nova/objects/compute_node.py line 386? | 13:45 |
| dansmith | bauzas: and disk on line 398 | 13:46 |
| *** ccesario has joined #openstack-nova | 13:46 | |
| bauzas | dansmith: okay, I'm just trying to resurrect my devstack instance | 13:46 |
| *** burt has joined #openstack-nova | 13:47 | |
| *** dimtruck is now known as zz_dimtruck | 13:47 | |
| *** gongysh has joined #openstack-nova | 13:48 | |
| dansmith | bauzas: thanks | 13:48 |
| *** pradk_ has quit IRC | 13:49 | |
| *** cleong has joined #openstack-nova | 13:50 | |
| *** matrohon has quit IRC | 13:53 | |
| *** edtubill has joined #openstack-nova | 13:53 | |
| *** matrohon has joined #openstack-nova | 13:53 | |
| *** eharney has joined #openstack-nova | 13:55 | |
| *** Oku_OS is now known as Oku_OS-away | 13:55 | |
| *** diogogmt has quit IRC | 13:55 | |
| *** diogogmt has joined #openstack-nova | 13:56 | |
| *** knikolla has joined #openstack-nova | 13:57 | |
| *** EinstCrazy has joined #openstack-nova | 13:59 | |
| *** EinstCrazy has quit IRC | 13:59 | |
| *** EinstCrazy has joined #openstack-nova | 13:59 | |
| *** breton has joined #openstack-nova | 14:00 | |
| *** mriedem is now known as mriedem_meeting | 14:00 | |
| *** ccesario has quit IRC | 14:00 | |
| *** jwcroppe has joined #openstack-nova | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** esracelik has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-nova | 14:01 | |
| *** cdent has quit IRC | 14:02 | |
| *** claudiub|2 has quit IRC | 14:02 | |
| *** laszloh has quit IRC | 14:02 | |
| *** gmann_ has quit IRC | 14:02 | |
| bauzas | sdague: not sure you followed but devstack is now bundling a base64 encoded pip for bootstrapping the correct version | 14:06 |
| bauzas | sdague: for some reason, it just tells me "f* off" when I'm using get_pip.py | 14:06 |
| bauzas | because of the -c flag | 14:06 |
| bauzas | sdague: any chance you know that problem ? | 14:06 |
| *** haomaiwang has quit IRC | 14:07 | |
| bauzas | the fun fact is that pip is bundled with a 1.5.6 version | 14:07 |
| *** mtanino has quit IRC | 14:08 | |
| *** busterswt has joined #openstack-nova | 14:08 | |
| sdague | bauzas: sorry, back up, can you point me at a thing? | 14:09 |
| *** EinstCrazy has quit IRC | 14:09 | |
| *** zz_dimtruck is now known as dimtruck | 14:09 | |
| *** takedakn has joined #openstack-nova | 14:10 | |
| *** liverpooler has quit IRC | 14:10 | |
| bauzas | sdague: nevermind, just saw that /opt/devstack/files/get_pip.py was bundling a pip base64 encoded but it seems the file was removed | 14:10 |
| bauzas | sdague: can't find it in https://github.com/openstack-dev/devstack/tree/master/files | 14:10 |
| sdague | files/ is not bundled | 14:10 |
| *** EinstCrazy has joined #openstack-nova | 14:10 | |
| sdague | mostly | 14:10 |
| sdague | they are downloaded | 14:10 |
| sdague | that is created by tools/install_pip.sh | 14:10 |
| bauzas | utterly strange | 14:10 |
| sdague | which comes from https://bootstrap.pypa.io/get-pip.py | 14:11 |
| *** eharney has quit IRC | 14:11 | |
| *** takedakn has quit IRC | 14:11 | |
| sdague | bauzas: what distro? | 14:12 |
| bauzas | hmmmm | 14:12 |
| bauzas | ubuntu 14.04 LTS | 14:12 |
| *** jaosorior has joined #openstack-nova | 14:12 | |
| bauzas | sdague: see http://paste.openstack.org/show/493164/ | 14:12 |
| *** busterswt has quit IRC | 14:13 | |
| *** ccesario has joined #openstack-nova | 14:13 | |
| sdague | bauzas: I don't know | 14:14 |
| *** busterswt has joined #openstack-nova | 14:15 | |
| sdague | bauzas: is this a system that's been around for a while? | 14:15 |
| bauzas | sdague: yeah, possibly some crap | 14:15 |
| sdague | I would do a ./clean.sh | 14:15 |
| *** PaulMurray has quit IRC | 14:15 | |
| sdague | then give it another shot | 14:16 |
| sdague | there is supposed to be some cache management on those files... but it may be broken | 14:16 |
| sdague | clean at least will get you back to a known base state | 14:16 |
| sdague | it should be nearly as good as a fresh machine | 14:16 |
| *** laszloh has joined #openstack-nova | 14:17 | |
| bauzas | I did :) | 14:19 |
| bauzas | I guess it's more a caching thing | 14:19 |
| dansmith | I find clean.sh often doesn't clean up enough | 14:20 |
| dansmith | I added the lvm thing a while back, which was my major problem, | 14:20 |
| dansmith | but I have run into things before where re-imaging was faster | 14:20 |
| *** cdent has joined #openstack-nova | 14:22 | |
| bauzas | I'm giving a couple of minutes to spend on that problem, and I'll start a fresh new VM if I'm not good | 14:23 |
| *** diogogmt has quit IRC | 14:23 | |
| openstackgerrit | Shiina, Hironori proposed openstack/nova: Remove mox from tests/unit/network/security_group/test_neutron_driver.py https://review.openstack.org/302283 | 14:24 |
| sarafraj | morning | 14:25 |
| sdague | alaski: hey, a thought on the policy in code | 14:25 |
| sdague | alaski: you about? | 14:26 |
| alaski | yep | 14:26 |
| *** achanda has joined #openstack-nova | 14:26 | |
| *** sheel has quit IRC | 14:27 | |
| sdague | so... in all cases where we do a policy check, we do it with a context, right? | 14:27 |
| sdague | because it's meaningless without one | 14:27 |
| *** diogogmt has joined #openstack-nova | 14:28 | |
| alaski | yeah, context is required | 14:28 |
| *** Bjoern_ has joined #openstack-nova | 14:28 | |
| *** claudiub has joined #openstack-nova | 14:28 | |
| alaski | well, by Nova. oslo.policy doesn't use it | 14:28 |
| alaski | we turn it into a credentials dict | 14:28 |
| sdague | if so, would it be more sensible to make the code flow context.can("os_compute_api:servers:create") | 14:28 |
| sdague | right, our context is a lot more than oslo.context | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/api directory https://review.openstack.org/258180 | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (4) https://review.openstack.org/297237 | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (2) https://review.openstack.org/260587 | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (1) https://review.openstack.org/260586 | 14:29 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (3) https://review.openstack.org/260588 | 14:29 |
| sdague | then the details of where it's doing that lookup go into our context, because our context is always required element | 14:29 |
| *** gongysh has quit IRC | 14:30 | |
| *** eharney has joined #openstack-nova | 14:30 | |
| sdague | and we don't have to drag that POLICY singleton into all the code | 14:31 |
| *** matrohon has quit IRC | 14:31 | |
| *** mriedem_meeting is now known as mriedem | 14:31 | |
| *** Swami has joined #openstack-nova | 14:31 | |
| *** Goneri has quit IRC | 14:32 | |
| alaski | seems reasonable to me, but the catch is that people did want a similar interface to config, meaning the ability to do *.os_compute_api.servers.create() | 14:32 |
| cdent | sdague++ | 14:32 |
| *** Swami_ has joined #openstack-nova | 14:32 | |
| *** Swami_ has quit IRC | 14:32 | |
| *** Swami has quit IRC | 14:32 | |
| sdague | alaski: on specification though | 14:32 |
| *** Oku_OS-away is now known as Oku_OS | 14:32 | |
| bauzas | sdague: oh fun, I think I catched my problem | 14:32 |
| *** ekuris has quit IRC | 14:32 | |
| *** Swami has joined #openstack-nova | 14:32 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize netconf options https://review.openstack.org/294312 | 14:32 |
| alaski | sdague: for the check itself | 14:33 |
| bauzas | sdague: correct me if I'm wrong, but localrc is now fully no longer supported ? | 14:33 |
| *** dedery_ has joined #openstack-nova | 14:33 | |
| alaski | sdague: the first implementation wasn't going to include it, but it's been asked about | 14:33 |
| sdague | bauzas: it should still work for legacy | 14:33 |
| sdague | alaski: can you provide an example of what that means? | 14:33 |
| sdague | I'm not sure I understand | 14:33 |
| bauzas | sdague: okay | 14:33 |
| *** nkrinner has quit IRC | 14:33 | |
| alaski | sdague: context.can("os_compute_api:servers:create") -> POLICY.os_compute_api.servers.create(context) | 14:34 |
| alaski | or could be context.os_compute_api.servers.create() | 14:34 |
| alaski | but that seems odd to me | 14:34 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hacking: Add placeholders for 'conf' check https://review.openstack.org/302288 | 14:34 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hacking: Add new files to centralized config check https://review.openstack.org/302289 | 14:34 |
| sfinucan | johnthetubaguy: markus_z: ^^^ | 14:34 |
| sdague | oh, because os_compute_api:servers would be a fallback if :create is not defined | 14:34 |
| *** zul has quit IRC | 14:35 | |
| sdague | I think in order to do aliasing we're going to need a string representation | 14:35 |
| alaski | at the moment that's not possible. but I think there's a desire to do it | 14:35 |
| sdague | so we should assume we'll always have some string representation | 14:35 |
| sdague | context.can("os_compute_api:servers:create") could be translated to whatever that check needs to be I think | 14:36 |
| markus_z | sfinucan: Ah, cool, I get the idea, I need a minute to think about it. | 14:36 |
| *** dedery has quit IRC | 14:37 | |
| alaski | sdague: yeah, behind the scenes either implementation can work the same | 14:37 |
| *** ccesario has quit IRC | 14:37 | |
| sdague | I'm mostly trying to think about making the code flow a bit better on the usage side | 14:38 |
| tdurakov | dansmith: hi, are you around? got question about versioned objects: why not to init fields with None in case them explicitly set? | 14:38 |
| markus_z | sfinucan: It's a thing I should have done at the beginning, I think. Let me check were we are right and if that's still necessary. | 14:38 |
| alaski | sdague: I have no resistance to embedding it in context, I just want to mention a desired calling convention I've been asked about. we may end up with two ways to call this if we go the context route | 14:39 |
| sdague | alaski: why? | 14:39 |
| *** eharney has quit IRC | 14:39 | |
| sdague | I guess that's the thing I don't get, it seems like within the code we're always going to be specifying some pretty specific permissions for things | 14:39 |
| sdague | even if only "os_compute_api:servers" was defined | 14:40 |
| *** eharney has joined #openstack-nova | 14:40 | |
| sfinucan | markus_z: Agreed. It should either be part of all commits or removed. | 14:40 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hacking: Add new files to centralized config check https://review.openstack.org/302289 | 14:40 |
| openstackgerrit | Stephen Finucane proposed openstack/nova: hacking: Add placeholders for 'conf' check https://review.openstack.org/302288 | 14:40 |
| dansmith | tdurakov: I'm around, but not sure what you're asking | 14:40 |
| markus_z | sfinucan: http://paste.openstack.org/show/493172/ It's looking good IMO | 14:40 |
| markus_z | sfinucan: The biggest chunk left is libvirt and someone is already working on that (I was too slow) | 14:41 |
| markus_z | sfinucan: As much as I appreciate the idea and effort, I think it's fine to wait until the rest is merged and then do the full check without any filtering. | 14:41 |
| *** sdake_ has joined #openstack-nova | 14:42 | |
| tdurakov | dansmith: if field is described in fields dict, but not explicitly passed to init method than there will be exception | 14:42 |
| *** sdake has quit IRC | 14:42 | |
| *** laszloh has quit IRC | 14:42 | |
| markus_z | sfinucan: In case you want to query this tree too: https://github.com/markuszoeller/openstack/blob/master/scripts/code-analysis/count_config_options_as_tree.py | 14:42 |
| *** sdake_ is now known as sdake | 14:43 | |
| dansmith | tdurakov: you mean if that field is subsequently used before it's set, right? | 14:43 |
| tdurakov | dansmith: yes http://xsnippet.org/361600/ | 14:44 |
| dansmith | tdurakov: right, so if we initialized used to None, and you called .save() on that object, it would try to null out the values in the database | 14:45 |
| dansmith | tdurakov: when really they are just not loaded | 14:45 |
| dansmith | tdurakov: so we can't assume some default value without requiring you to always have a full object | 14:45 |
| *** Goneri has joined #openstack-nova | 14:46 | |
| tdurakov | dansmith, thank you for details | 14:47 |
| dansmith | tdurakov: np :) | 14:47 |
| alaski | sdague: see Johns comment at https://review.openstack.org/#/c/290155/2/specs/newton/approved/policy-in-code.rst@96 . The desire seems to be that policy and config can be called similarly. CONF.group.option, POLICY.foo.bar.baz(context) | 14:47 |
| *** sdake_ has joined #openstack-nova | 14:48 | |
| *** ericksonsantos has quit IRC | 14:48 | |
| alaski | sdague: but I'm fine with throwing it on context, just bringing it up | 14:48 |
| sdague | I don't know, I feel like there are some anti patterns in oslo.config that I don't think fit here | 14:48 |
| sdague | especially if we are going to eventually try to make our policy definitions look more like our rest urls | 14:49 |
| *** raildo is now known as raildo-afk | 14:49 | |
| alaski | that's a good point. if we change the "foo:bar" format and don't have a natural separator the config style access no longer makes sense | 14:50 |
| *** ducttape_ has joined #openstack-nova | 14:50 | |
| sdague | because we really end up having to specify, at some level: method, url, action | 14:50 |
| sdague | and maybe more | 14:50 |
| *** sdake has quit IRC | 14:51 | |
| *** raildo-afk is now known as raildo | 14:51 | |
| kaisers | Hey Novas! I've a driver bugfix change open for some weeks (https://review.openstack.org/#/c/264752/), if a core find's the time to take a glance I'd be very happy. :) | 14:51 |
| *** salv-orlando has joined #openstack-nova | 14:51 | |
| *** sc68cal has joined #openstack-nova | 14:51 | |
| alaski | sdague: see my comment at https://review.openstack.org/#/c/289405/1/specs/newton/approved/discoverable-policy-api.rst@80 depending on what you mean by action we're in agreement on that | 14:52 |
| sdague | which is also the benefit of not exposing POLICY into most of the code, but going through context. We can defer changes with POLICY calling conventions until later, and they are somewhat isolated | 14:52 |
| *** achanda has quit IRC | 14:53 | |
| *** jaosorior has quit IRC | 14:54 | |
| sdague | alaski: yeh, action == details for the most part | 14:54 |
| sdague | method == http_action in your sentence | 14:54 |
| sdague | methods = (GET, POST, PUT, DELETE) | 14:54 |
| alaski | I have an argument against using dotted notation now so I'm comfortable proposing and defending that | 14:54 |
| *** mtanino has joined #openstack-nova | 14:54 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize notification options https://review.openstack.org/273761 | 14:55 |
| alaski | sdague: yeah, bad terminology on my part | 14:55 |
| sdague | I was thinking "action" in the sense of nova server /actions | 14:55 |
| sfinucan | markus_z: no problemo | 14:55 |
| sdague | though they probably are more general for things like the volume attach that you listed | 14:55 |
| alaski | right. it's the same problem, policy may differ based on payload not just method/url | 14:55 |
| sdague | alaski: yep | 14:56 |
| sdague | ok, so I put another comment in about the registration process, just to make it a little easier to catch issues in review I think | 14:57 |
| *** artom has joined #openstack-nova | 14:57 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: [WIP] Add console log offset https://review.openstack.org/292473 | 14:57 |
| sdague | conceptually I'm all good on the approach, just trying to figure out what details we should agree on before starting to implement | 14:57 |
| alaski | just saw that. that's what I originally had, and the argument against it is it makes adding more details later harder | 14:57 |
| alaski | i.e. deprecated, aliases | 14:58 |
| *** yamahata has quit IRC | 14:59 | |
| *** eharney has quit IRC | 14:59 | |
| *** Jeffrey4l has quit IRC | 15:01 | |
| *** zul has joined #openstack-nova | 15:01 | |
| *** jwcroppe has quit IRC | 15:01 | |
| openstackgerrit | Gleb Stepanov proposed openstack/nova: Check API versions intersects https://review.openstack.org/301278 | 15:01 |
| sdague | honestly, I think again we're hitting oslo.config anti pattern here. We would not expect a 3rd party library to modify this | 15:02 |
| sdague | this is going to be in one chunk of code more or less | 15:02 |
| *** pushkaru has joined #openstack-nova | 15:03 | |
| sdague | all those object wraps seem a little premature engineering, we could add it later if we really need it | 15:03 |
| sdague | but if we start with it, we're building that object wrap up forever | 15:03 |
| *** burgerk_ has joined #openstack-nova | 15:04 | |
| jroll | mriedem: are there plans for deprecating the nova CLI in favor of OSC any time soon? | 15:04 |
| jroll | (anyone can answer, really) | 15:04 |
| alaski | there's two things I think we want to add later. The ability to deprecate/alias policies, and the ability to group them under a capability/endpoint. Doing the object wrap makes it easily extensible. So it's a matter of do we start with something we might immediately change? | 15:05 |
| alaski | sdague: ^ | 15:05 |
| sdague | alaski: I think it ends up being a lot of boiler plate that causes people to miss things in review | 15:06 |
| *** burgerk has quit IRC | 15:06 | |
| *** mhayden has joined #openstack-nova | 15:06 | |
| sdague | because the important details are nested under a lot of parens / parameters | 15:07 |
| *** Jeffrey4l has joined #openstack-nova | 15:07 | |
| *** vladikr has quit IRC | 15:07 | |
| sdague | plus... merging the existing policy.json overrides seems simpler if we are fundamentally just processing that list | 15:07 |
| alaski | I get that. I just think we're going to want to immediately add more here and then have to go throught a transition of string rules to object rules | 15:08 |
| sdague | alaski: sure, but, that's fine | 15:08 |
| sdague | I would much rather do that transition from a thing we know is working | 15:08 |
| *** PaulMurray has joined #openstack-nova | 15:08 | |
| *** itamarl has quit IRC | 15:08 | |
| *** jogo has quit IRC | 15:09 | |
| bauzas | alaski: cells meeting at 2100UTC right? | 15:09 |
| sdague | because, again, this is basically going to be isolated in a couple py files right? | 15:09 |
| dansmith | bauzas: yeah | 15:09 |
| alaski | sdague: okay. I'll throw the object under alternatives and see if there's any feedback | 15:09 |
| bauzas | \o/ | 15:09 |
| sdague | so changing internal representations is fine | 15:09 |
| *** jogo has joined #openstack-nova | 15:10 | |
| alaski | sure. it's just tedious | 15:10 |
| alaski | but I understand why you want it | 15:11 |
| *** cdent has quit IRC | 15:12 | |
| *** Jeffrey4l has quit IRC | 15:12 | |
| *** mjura has quit IRC | 15:12 | |
| *** mdrabe has quit IRC | 15:13 | |
| *** mdrabe has joined #openstack-nova | 15:13 | |
| bauzas | interesting, I think I catched my bug | 15:13 |
| sdague | jroll: there is a ton of function in nova cli that is not exposed in osc | 15:13 |
| bauzas | install_pip doesn't download a new get_pip.py file | 15:13 |
| *** eharney has joined #openstack-nova | 15:14 | |
| dansmith | bauzas: maybe it shouldn't, but clean.sh should remove it? | 15:14 |
| bauzas | dansmith: apparently, it doesn't | 15:14 |
| sdague | bauzas: really? | 15:14 |
| kaisers | markus_z: Thanks for reviewing, i added answers. | 15:14 |
| dansmith | I'm saying maybe clean _should_ remove it | 15:14 |
| jroll | sdague: so certainly not soon, is that the long-term goal or meh? | 15:14 |
| dansmith | jroll: I hate change | 15:14 |
| jroll | s/meh/maybe one day someday far away/ | 15:15 |
| * jroll hands dansmith some new food he's never tried | 15:15 | |
| bauzas | sdague: yeah http://paste.openstack.org/show/493179/ | 15:15 |
| * dansmith runs away screaming | 15:15 | |
| *** dave-mccowan has joined #openstack-nova | 15:15 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the OSAPI options https://review.openstack.org/274879 | 15:15 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/api directory https://review.openstack.org/258180 | 15:15 |
| jroll | :D | 15:15 |
| sdague | jroll: I don't think there is any real game plan for a lot of the nova admin commands for osc | 15:15 |
| sdague | which is a sticking point | 15:15 |
| jroll | sdague: ah yeah, admin. fun. | 15:15 |
| sdague | and so, without that, it's kind of just tabled | 15:16 |
| jroll | alright, that answers my question (and partially mhayden's question by proxy) | 15:16 |
| sdague | and not really a priority to figure out | 15:16 |
| jroll | yep | 15:16 |
| jroll | fair enough | 15:16 |
| bauzas | dansmith: I'm not a devstack ninja but I can do this :) | 15:16 |
| jroll | sdague: for context, if you care: https://github.com/major/supernova/issues/108 | 15:16 |
| dansmith | bauzas: I believe you can :) | 15:16 |
| *** Marga_ has joined #openstack-nova | 15:16 | |
| *** belmoreira has quit IRC | 15:17 | |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: make sure device relationships are kept in memory https://review.openstack.org/301859 | 15:17 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: related updates are done without DB lookups https://review.openstack.org/301860 | 15:17 |
| sdague | jroll: that whole problem domain I think is solved by osc | 15:18 |
| jroll | sdague: sure, unless you want the nova cli :) | 15:18 |
| sdague | <dig> would have been nice if the supernova folks had helped osc get there quicker instead of writing their own tool | 15:18 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (4) https://review.openstack.org/297237 | 15:18 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (2) https://review.openstack.org/260587 | 15:18 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (1) https://review.openstack.org/260586 | 15:18 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the API options (3) https://review.openstack.org/260588 | 15:18 |
| jroll | sdague: for instance, I use supernova with one environment being an admin api | 15:18 |
| dansmith | sdague: you forgot to close your dig tag | 15:18 |
| sdague | jroll: right, you've seen clouds.yaml right | 15:19 |
| jroll | sdague: also, this is maintained by one person (mhayden) and was in working order before osc was a thing... | 15:19 |
| sdague | and the whole infrastructure to do all that in occ, which is in osc | 15:19 |
| jroll | sdague: yes, but osc doesn't have admin commands, like you said | 15:19 |
| bauzas | okay, I think I hit a fun problem | 15:19 |
| jroll | so it doesn't work for someone that needs admin commands | 15:19 |
| mhayden | thanks for bringing it up, jroll ;) | 15:19 |
| jroll | mhayden: not a problem | 15:20 |
| *** READ10 has quit IRC | 15:21 | |
| *** claudiub has quit IRC | 15:21 | |
| jroll | anyway, this is still a generally useful thing until OSC can replace novaclient | 15:21 |
| jroll | thanks for the info | 15:21 |
| johnthetubaguy | dansmith: alaski: do you remember why we had to cut and paste the config options for the API DB? I guess we couldn't just add "CONF.register_opts(oslo_db_options.database_opts, 'api_database')" https://git.openstack.org/cgit/openstack/nova/commit/?id=ba407e33d3fedd2d283ba8752ff592fb3e0e2d43 | 15:21 |
| sdague | jroll: well, it was only started 4 days before osc :) | 15:22 |
| jroll | heh | 15:22 |
| *** salv-orlando has quit IRC | 15:22 | |
| jroll | I didn't realize osc was that old | 15:22 |
| dansmith | johnthetubaguy: I dunno, are they identical? | 15:22 |
| *** vladikr has joined #openstack-nova | 15:22 | |
| dansmith | johnthetubaguy: maybe oslo.config is not cool with reusing those options? | 15:22 |
| dansmith | johnthetubaguy: the help strings aren't the same | 15:23 |
| johnthetubaguy | dansmith: yeah, thats what I was wondering I guess | 15:23 |
| sdague | https://github.com/major/supernova/commit/80cb777534f65bc3a3a74850af6c52b2acfd2a27 - https://github.com/openstack/python-openstackclient/commit/f4b5ef39f6f84e66af532583040c9be7556e9b69 | 15:23 |
| openstackgerrit | Zhenguo Niu proposed openstack/nova: cmd: add nova-shellinaboxproxy service https://review.openstack.org/302316 | 15:23 |
| alaski | I'm trying to recall now | 15:23 |
| *** bjornar has joined #openstack-nova | 15:23 | |
| bauzas | johnthetubaguy: I think that oslo.config doesn't like registering twice the same opt | 15:23 |
| sdague | bauzas: ++ that | 15:23 |
| bauzas | johnthetubaguy: even for two different groups | 15:23 |
| *** Jeffrey4l has joined #openstack-nova | 15:24 | |
| *** mylu has joined #openstack-nova | 15:24 | |
| sdague | yeh, it has a bunch of internal checks that all explode | 15:24 |
| mhayden | jroll: current usage -> http://pypi-ranking.info/module/supernova | 15:24 |
| johnthetubaguy | OK, that makes sense | 15:24 |
| johnthetubaguy | well, its something we need a different work around for I guess | 15:24 |
| alaski | yeah, I think that was it. would be easy enough to test | 15:24 |
| bauzas | I'm pretty it is | 15:24 |
| bauzas | pretty *sure even | 15:25 |
| *** rmevans has joined #openstack-nova | 15:25 | |
| johnthetubaguy | ack, I was being lazy and using the hive mind :) | 15:25 |
| johnthetubaguy | I saw the config move, and it confused me | 15:25 |
| bauzas | I had the problem a while ago on a separate project trying to define same opts as for nova (boooh the ugly hack) | 15:25 |
| sdague | mhayden: realistically, it's probably better to figure out if there is a subset of the admin commands that your users need and get those over into osc | 15:25 |
| alaski | johnthetubaguy: https://review.openstack.org/#/c/159266/14/nova/db/sqlalchemy/api.py@125 | 15:25 |
| alaski | it had to do with deprecated options | 15:26 |
| alaski | some of those were still available under the default group | 15:26 |
| alaski | and couldn't be registered twice | 15:26 |
| *** sheel has joined #openstack-nova | 15:26 | |
| *** armax has joined #openstack-nova | 15:27 | |
| *** zul_ has joined #openstack-nova | 15:27 | |
| diana_clarke | The unit tests are a lot slower again. Is someone already looking into why? | 15:27 |
| *** zul has quit IRC | 15:27 | |
| johnthetubaguy | alaski: cool, thank you, regained context now | 15:28 |
| dansmith | diana_clarke: might be my flavor migration that happens on each test execution | 15:28 |
| dansmith | diana_clarke: waiting for a dependent patch to merge before we can land the thing that undoes that | 15:28 |
| sdague | dansmith: which is the dependent patch? | 15:28 |
| dansmith | sdague: the devstack one | 15:28 |
| sdague | oh, let me poke dean | 15:28 |
| sdague | hmmm... why is that a thing? | 15:29 |
| dansmith | why is what? | 15:29 |
| sdague | I guess I'm not sure why that would impact that unit tests | 15:29 |
| *** sdake has joined #openstack-nova | 15:29 | |
| *** dave-mcc_ has joined #openstack-nova | 15:30 | |
| *** sridharg has quit IRC | 15:30 | |
| *** sdake_ has quit IRC | 15:30 | |
| dansmith | sdague: we have to migrate the flavors from the old db to the new db on every test setup right now | 15:30 |
| dansmith | sdague: my patch to remove them from the main db makes that go away | 15:30 |
| dansmith | but people didn't want to land that without the devstack thing first | 15:31 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the neutron API options https://review.openstack.org/274126 | 15:31 |
| dansmith | I don't know that this is what diana_clarke is seeing, I'm just guessing | 15:31 |
| sdague | ok, is there a reason we don't do that in the Database fixture though | 15:31 |
| sdague | so it's in the schema cache? | 15:31 |
| dansmith | I have to be on a call | 15:31 |
| dansmith | but it's one patch to get it gone now | 15:31 |
| *** EinstCrazy has quit IRC | 15:31 | |
| sdague | ok, well the devstack patch is about to be approved, so maybe it doesn't matter | 15:32 |
| dansmith | witha nice fixture we can narrow the scope of | 15:32 |
| dansmith | okay cool | 15:32 |
| * dansmith -> call | 15:32 | |
| *** moshele has quit IRC | 15:32 | |
| *** dave-mccowan has quit IRC | 15:32 | |
| sdague | devstack change approved | 15:32 |
| dansmith | woot | 15:33 |
| sdague | so that should be in in a couple of hours | 15:33 |
| *** moshele has joined #openstack-nova | 15:33 | |
| diana_clarke | dansmith: I have notes from Feb 5th saying tests took 14 minutes locally. Today they are taking 24 minutes locally. Same number of workers etc (but more tests obviously). | 15:34 |
| diana_clarke | dansmith: http://paste.openstack.org/show/493185/ | 15:34 |
| *** haplo37 has joined #openstack-nova | 15:34 | |
| *** rmevans has quit IRC | 15:34 | |
| *** moshele has quit IRC | 15:35 | |
| sdague | yeh, I see things roughly doubled here too. It's probably a good guess that's the issue | 15:35 |
| diana_clarke | Here are the current slow tests: http://paste.openstack.org/show/493182/ with the number of slow tests to show changed from 10 to 100 | 15:36 |
| *** ametts has quit IRC | 15:36 | |
| sdague | so lets see what happens after dansmith's change | 15:36 |
| diana_clarke | okie dokie | 15:37 |
| openstackgerrit | Sujitha proposed openstack/nova: Config options: Centralize console options https://review.openstack.org/294261 | 15:37 |
| mriedem | jroll: looks like you got your question answered (i was in a meeting); we do conveniently say the novaclient CLI is deprecated when we don't want to add new features, we tell people to add them in osc | 15:38 |
| mriedem | jroll: there was dev thread related to that recently | 15:38 |
| *** sdake_ has joined #openstack-nova | 15:40 | |
| *** Swami_ has joined #openstack-nova | 15:41 | |
| *** armax_ has joined #openstack-nova | 15:41 | |
| *** armax has quit IRC | 15:42 | |
| *** armax_ is now known as armax | 15:42 | |
| *** sdake has quit IRC | 15:43 | |
| sc68cal | found a bug while on "vacation" - https://bugs.launchpad.net/nova/+bug/1566942 | 15:43 |
| openstack | Launchpad bug 1566942 in OpenStack Compute (nova) "Nova live-migrations cannot be done with mixed versions of nova-api and nova-compute" [Undecided,New] | 15:43 |
| *** jdurgin1 has joined #openstack-nova | 15:44 | |
| *** jogo has quit IRC | 15:44 | |
| *** Swami has quit IRC | 15:45 | |
| mriedem | sc68cal: what is 'up to date' controller? master? | 15:46 |
| mriedem | or mitaka rc3? | 15:46 |
| *** dedery has joined #openstack-nova | 15:46 | |
| *** p01nt3r75 has joined #openstack-nova | 15:46 | |
| *** jogo has joined #openstack-nova | 15:46 | |
| sc68cal | I can give the exact SHA if that helps, but it was basically a devstack run, from master branch | 15:46 |
| sc68cal | about a week or so ago | 15:46 |
| mriedem | sc68cal: ok, i'm pretty sure we don't support newton conductor + liberty compute | 15:46 |
| mriedem | we support n-1 | 15:46 |
| sc68cal | I think it was still mitaka timeframe though | 15:46 |
| sc68cal | sha was b55d5bc | 15:47 |
| *** rmevans has joined #openstack-nova | 15:48 | |
| *** dedery_ has quit IRC | 15:49 | |
| *** vladikr has quit IRC | 15:49 | |
| *** yamahata has joined #openstack-nova | 15:49 | |
| bauzas | mriedem: I saw you providing https://etherpad.openstack.org/p/newton-nova-summit-ideas | 15:50 |
| *** andymaier has quit IRC | 15:50 | |
| bauzas | mriedem: I mean the proposed schedule | 15:50 |
| sdague | sc68cal: b55d5bc is master | 15:50 |
| *** dedery has quit IRC | 15:50 | |
| sdague | that same commit was backported to mitaka as well | 15:50 |
| sc68cal | sdague: yeah, sorry b55d5bc was a note from something else | 15:51 |
| bauzas | mriedem: do you think it could be nice to discuss on PCI/NUMA/FPGA/other stuff during a session ? | 15:51 |
| sdague | but there are other things in there that might be master and make it not work for the n-2 jump | 15:51 |
| *** dedery has joined #openstack-nova | 15:51 | |
| sc68cal | sdague: I think it was just n-1 jump though | 15:51 |
| *** Sukhdev_ has joined #openstack-nova | 15:51 | |
| bauzas | mriedem: in particular given your PCI thread and the FPGA one | 15:51 |
| sdague | sc68cal: ok, details would be good | 15:52 |
| *** claudiub has joined #openstack-nova | 15:52 | |
| mriedem | sc68cal: what is CONF.upgrade_levels.compute in nova.conf? | 15:52 |
| sc68cal | mriedem: whatever the default it | 15:52 |
| sc68cal | *is | 15:52 |
| sc68cal | via devstack | 15:52 |
| *** salv-orlando has joined #openstack-nova | 15:53 | |
| *** ccesario has joined #openstack-nova | 15:53 | |
| *** dedery_ has joined #openstack-nova | 15:54 | |
| mriedem | sc68cal: so given i don't have your specific devstack from when you stacked it, can you check? | 15:54 |
| mriedem | bauzas: a whole session on pci/numa/fpga? | 15:54 |
| mriedem | bauzas: personally i'm not interested in that as a priority, but you could make a case | 15:55 |
| bauzas | mriedem: I dunno, like a session about performant guests | 15:55 |
| bauzas | and how the resources can be provided | 15:55 |
| bauzas | performent* even | 15:55 |
| mriedem | bauzas: well, we have 2 sessions for scheduler | 15:55 |
| sc68cal | mriedem: ok grabbing it now | 15:55 |
| mriedem | if you're talking about resource providers | 15:55 |
| *** dedery has quit IRC | 15:55 | |
| bauzas | not really | 15:55 |
| bauzas | it's more than that - it's also related to the 3rd party CIs that we ask | 15:56 |
| *** salv-orlando has quit IRC | 15:56 | |
| sc68cal | mriedem: I don't see a section in my nova.conf with upgrade_levels | 15:56 |
| *** xyang1 has joined #openstack-nova | 15:57 | |
| mriedem | sc68cal: see the grenade multinode job nova.conf | 15:57 |
| mriedem | http://logs.openstack.org/18/295218/3/check/gate-grenade-dsvm-multinode/cb39c7b/logs/etc/nova/nova.conf.txt.gz | 15:57 |
| *** salv-orlando has joined #openstack-nova | 15:58 | |
| mriedem | dansmith: is there a reason that CONF.upgrade_levels.compute doesn't default to 'auto'? | 15:58 |
| * dansmith retargets that bug to the "sc68cal's config" project | 15:58 | |
| dansmith | mriedem: we only got auto in mitaka and wanted to let it sink in a bit | 15:58 |
| *** yamahata has quit IRC | 15:59 | |
| dansmith | mriedem: we set it to that in the gate so hopefully that means it's fairly good to go | 15:59 |
| *** yamahata has joined #openstack-nova | 15:59 | |
| sc68cal | I think I would argue that setting it at the gate, and not having it out of the box is a serious problem | 15:59 |
| sc68cal | I mean I guess that it's good that in mitaka+ it's set. | 16:00 |
| dansmith | sc68cal: well, it's a very new feature and we wanted to make sure that it worked | 16:00 |
| dansmith | sc68cal: it was not possible pre-mitaka | 16:00 |
| mriedem | sc68cal: it's not set by default | 16:00 |
| *** ducttape_ has quit IRC | 16:00 | |
| *** claudiub|2 has joined #openstack-nova | 16:00 | |
| dansmith | mriedem: I'm cool with defaulting it on in newton and can propose that change if you want | 16:00 |
| mriedem | dansmith: let's do it | 16:01 |
| mriedem | dansmith: you can use sc68cal's bug https://bugs.launchpad.net/nova/+bug/1566942 | 16:01 |
| openstack | Launchpad bug 1566942 in OpenStack Compute (nova) "Nova live-migrations cannot be done with mixed versions of nova-api and nova-compute" [Undecided,Invalid] | 16:01 |
| sc68cal | yay! :) | 16:01 |
| sc68cal | While we're on the subject... | 16:02 |
| sc68cal | I almost want to propose a new spec | 16:02 |
| sc68cal | "get me a migration" | 16:02 |
| sc68cal | because wow is live migration a rat's nest | 16:02 |
| *** ducttape_ has joined #openstack-nova | 16:02 | |
| *** claudiub has quit IRC | 16:02 | |
| *** cdent has joined #openstack-nova | 16:03 | |
| dansmith | mriedem: I'll related-bug it, but it doesn't close that because that bug is about non-newton code | 16:03 |
| openstackgerrit | Dan Smith proposed openstack/nova: Default compute upgrade_level to auto https://review.openstack.org/302341 | 16:04 |
| *** mrkz has quit IRC | 16:05 | |
| *** mrkz has joined #openstack-nova | 16:05 | |
| *** mdrabe has quit IRC | 16:05 | |
| sc68cal | dansmith: can we also backport that to mitaka? | 16:05 |
| *** apoorvad has joined #openstack-nova | 16:06 | |
| *** erlon has joined #openstack-nova | 16:07 | |
| mriedem | dansmith: yeah, i'm fine with related bug | 16:07 |
| mriedem | sc68cal: no | 16:08 |
| sc68cal | aww, why not? | 16:08 |
| mriedem | well, it changes the default value on a config option for one | 16:08 |
| mriedem | a new config option in a thing that hasn't been officially released yet | 16:08 |
| *** claudiub has joined #openstack-nova | 16:08 | |
| *** claudiub|2 has quit IRC | 16:08 | |
| mriedem | sc68cal: before 'auto' deployers had to be used to setting upgrade_levels before anyway | 16:09 |
| mriedem | auto is a new convenience | 16:09 |
| mriedem | and once we can feel safe that it's not ruining the world once mitaka is officially released and people start upgrading to it, then it makes more sense to make it a default | 16:09 |
| sahid | sdague: i had to rebase https://review.openstack.org/#/c/176242/, if you can re +2 it, this code is waiting for a while that can help to encourage contributor to do some reviews | 16:10 |
| *** jistr has quit IRC | 16:10 | |
| sc68cal | but if deployers are setting upgrade_levels already, that's fine | 16:10 |
| sc68cal | it's the people who are not, and are not aware, that are at issue | 16:11 |
| *** mastermind has quit IRC | 16:11 | |
| dansmith | sc68cal: upgrade_levels has been around since before icehouse and is part of a pretty standard set of things people do | 16:12 |
| *** zenoway has joined #openstack-nova | 16:12 | |
| dansmith | sc68cal: being able to set compute to auto is new goodness | 16:12 |
| dansmith | but otherwise I think it's a well-known thing | 16:12 |
| *** anush_ has joined #openstack-nova | 16:13 | |
| *** mylu has quit IRC | 16:14 | |
| sc68cal | I guess I disagree. To put the shoe on the other foot, if I said to you "oh everyone knows to set XYZ in Neutron, and unless you do stuff breaks" what would be your thought? | 16:15 |
| sc68cal | and just because deployment tools are setting it, doesn't mean people are always aware of it | 16:15 |
| dansmith | sc68cal: if you said "we have this fixed but it's not something we want to backport" I think I'd say "cool" :) | 16:16 |
| mriedem | sc68cal: i see the point, and sane defaults are good, and that's why we're changing it in newton | 16:16 |
| *** zenoway has quit IRC | 16:16 | |
| sc68cal | in fact, we did have this. The whole MTU business | 16:16 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: make sure device relationships are kept in memory https://review.openstack.org/301859 | 16:16 |
| openstackgerrit | Nikola Dipanov proposed openstack/nova: pci: related updates are done without DB lookups https://review.openstack.org/301860 | 16:16 |
| sc68cal | hmmm, and we didn't backport. | 16:16 |
| mriedem | sc68cal: but you're not backporting that default mtu value are you? | 16:17 |
| mriedem | right | 16:17 |
| sc68cal | welp, guess that's settled then. | 16:17 |
| *** anush_ has quit IRC | 16:17 | |
| sc68cal | thanks for the quick turnaround on the fix and being patient :) | 16:17 |
| *** jwcroppe has joined #openstack-nova | 16:17 | |
| *** jwcroppe has quit IRC | 16:18 | |
| *** jwcroppe has joined #openstack-nova | 16:18 | |
| dansmith | mriedem: I was going to wait for jay and bauzas to comment on those two remaining items on the inventory migration patch | 16:19 |
| dansmith | but I have the other stuff tweaked, | 16:19 |
| mriedem | i'm reading up on usb pointer models | 16:20 |
| *** mdrabe has joined #openstack-nova | 16:20 | |
| dansmith | so if I push it up will you mirror your concerns on the new patch series so that the things we want feedback on are clean and easy to read? | 16:20 |
| *** mylu has joined #openstack-nova | 16:20 | |
| dansmith | I guess I could do the mirroring too | 16:20 |
| openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 16:21 |
| *** vtech has quit IRC | 16:23 | |
| mriedem | NOOOO!!! | 16:24 |
| mriedem | j/k i haven't read the replies yet | 16:24 |
| mriedem | brief glance was mostly, 'no, you're wrong and this is why' | 16:25 |
| *** woodster_ has joined #openstack-nova | 16:25 | |
| dansmith | yep | 16:25 |
| dansmith | :) | 16:25 |
| dansmith | except for one thing from you and one from johnthetubaguy | 16:26 |
| mriedem | sahid: comments inline https://review.openstack.org/#/c/176242/ | 16:26 |
| mriedem | i need lunch | 16:27 |
| sahid | mriedem: cool, looking | 16:27 |
| *** daemontool has quit IRC | 16:28 | |
| *** jistr has joined #openstack-nova | 16:28 | |
| *** browne has joined #openstack-nova | 16:28 | |
| *** moshele has joined #openstack-nova | 16:29 | |
| *** ljxiash has quit IRC | 16:29 | |
| *** jwcroppe has quit IRC | 16:30 | |
| *** jlanoux has quit IRC | 16:30 | |
| *** thumpba has joined #openstack-nova | 16:31 | |
| *** claudiub has quit IRC | 16:31 | |
| openstackgerrit | Andrew Laski proposed openstack/nova-specs: Embed policy defaults in code https://review.openstack.org/290155 | 16:32 |
| *** pushkaru has quit IRC | 16:32 | |
| *** nic has joined #openstack-nova | 16:33 | |
| *** Swami_ has quit IRC | 16:34 | |
| *** jistr is now known as jistr|off | 16:34 | |
| *** Marga_ has quit IRC | 16:34 | |
| *** zul_ has quit IRC | 16:35 | |
| mriedem | moshele: add specless blueprint proposals to the nova meeting agenda please | 16:36 |
| mriedem | so people can see it ahead of time | 16:36 |
| moshele | mriedem: ok I will | 16:36 |
| *** Marga_ has joined #openstack-nova | 16:37 | |
| *** claudiub has joined #openstack-nova | 16:39 | |
| *** doug-fish has quit IRC | 16:40 | |
| *** doug-fish has joined #openstack-nova | 16:40 | |
| *** itamarl has joined #openstack-nova | 16:40 | |
| *** haukebruno has joined #openstack-nova | 16:42 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (4) https://review.openstack.org/275423 | 16:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (2) https://review.openstack.org/271489 | 16:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (3) https://review.openstack.org/272231 | 16:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Move config options from nova/network/linux_net.py https://review.openstack.org/270441 | 16:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the linuxnet options (1) https://review.openstack.org/270959 | 16:42 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve help text for the network object options https://review.openstack.org/269313 | 16:43 |
| *** scheuran has quit IRC | 16:43 | |
| *** Marga_ has quit IRC | 16:44 | |
| *** claudiub has quit IRC | 16:44 | |
| mriedem | dansmith: ok, replied, seems fine | 16:44 |
| mriedem | fine(ish) | 16:44 |
| *** Bjoern_ is now known as Bjoern_zZzZzZzZ | 16:45 | |
| *** doug-fish has quit IRC | 16:45 | |
| *** femnad has quit IRC | 16:46 | |
| *** vilobhmm11 has joined #openstack-nova | 16:47 | |
| *** vilobhmm11 has quit IRC | 16:47 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the legacy API options https://review.openstack.org/273266 | 16:47 |
| *** Swami has joined #openstack-nova | 16:47 | |
| *** zenoway has joined #openstack-nova | 16:47 | |
| *** amotoki has quit IRC | 16:48 | |
| *** claudiub has joined #openstack-nova | 16:50 | |
| *** derekh has quit IRC | 16:50 | |
| *** sahid has quit IRC | 16:50 | |
| *** Bjoern_zZzZzZzZ is now known as Bjoern_ | 16:50 | |
| *** zul has joined #openstack-nova | 16:51 | |
| *** zenoway has quit IRC | 16:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add api-ref/build/* to .gitignore https://review.openstack.org/302358 | 16:52 |
| *** ndipanov has quit IRC | 16:52 | |
| *** vtech has joined #openstack-nova | 16:53 | |
| *** achanda has joined #openstack-nova | 16:53 | |
| *** rmevans has quit IRC | 16:53 | |
| *** burgerk has joined #openstack-nova | 16:53 | |
| *** ctrath has quit IRC | 16:54 | |
| *** claudiub has quit IRC | 16:54 | |
| *** mylu has quit IRC | 16:54 | |
| *** sc68cal has quit IRC | 16:55 | |
| *** burgerk_ has quit IRC | 16:55 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix typo in compute node mega join comments https://review.openstack.org/302363 | 16:56 |
| *** scheuran has joined #openstack-nova | 16:56 | |
| *** scheuran has quit IRC | 16:56 | |
| *** ctrath has joined #openstack-nova | 16:56 | |
| *** Oku_OS is now known as Oku_OS-away | 16:57 | |
| *** vtech_ has joined #openstack-nova | 16:57 | |
| *** zhangjn has quit IRC | 16:57 | |
| Swami | mriedem: Can you take a look at this patch. It is waiting for your blessing for a while. https://review.openstack.org/#/c/275073/ | 16:57 |
| *** speller has quit IRC | 16:58 | |
| *** zhangjn has joined #openstack-nova | 16:58 | |
| *** vtech has quit IRC | 16:58 | |
| *** ayoung has quit IRC | 16:59 | |
| mriedem | Swami: but i've been sleeping so well at night :) | 16:59 |
| *** davideagnello has joined #openstack-nova | 16:59 | |
| *** bjornar has quit IRC | 16:59 | |
| Swami | mriedem: sorry if my patch disturbs you a lot | 16:59 |
| *** dedery has joined #openstack-nova | 17:01 | |
| *** mgoddard has quit IRC | 17:03 | |
| *** mgoddard has joined #openstack-nova | 17:03 | |
| *** dedery_ has quit IRC | 17:05 | |
| *** dedery has quit IRC | 17:06 | |
| *** rwmjones is now known as rwmjones_hols | 17:06 | |
| *** sfinucan has quit IRC | 17:07 | |
| *** moshele has quit IRC | 17:08 | |
| *** angdraug has joined #openstack-nova | 17:08 | |
| *** danpb has quit IRC | 17:09 | |
| mriedem | dansmith: some more random questoins in https://review.openstack.org/#/c/279313/14/nova/objects/compute_node.py | 17:12 |
| *** salv-orlando has quit IRC | 17:12 | |
| dansmith | mriedem: replied | 17:14 |
| *** ctrath has quit IRC | 17:15 | |
| doffm | So there is some code in the db api that enforces uniqueness of an aggregate name. https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L5888 | 17:15 |
| doffm | dansmith, alaski: Do you think that we should add a uniqueness constraint to the table when moving aggregates to the API db? | 17:16 |
| *** BobBall is now known as BobBall-AWOL | 17:16 | |
| dansmith | doffm: that's a laski queston | 17:16 |
| *** e0ne has quit IRC | 17:17 | |
| *** p01nt3r75 has quit IRC | 17:17 | |
| *** ctrath has joined #openstack-nova | 17:18 | |
| *** eil397 has joined #openstack-nova | 17:18 | |
| *** gongysh has joined #openstack-nova | 17:18 | |
| alaski | that is an odd bit of code | 17:19 |
| doffm | alaski: The same is present in the create function. Its enforcing a uniqueness constraint in the code. | 17:19 |
| alaski | based on that I don't see why we wouldn't have a uniqueness constraint in the db | 17:19 |
| alaski | doffm: yeah. maybe there's some reason it's done this way, but a UC seems appropriate instead | 17:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/302016 | 17:20 |
| mriedem | https://github.com/openstack/nova/commit/2b2decfd1b5b0d68813bc7e5b5e4349a1ab592aa | 17:20 |
| doffm | alaski: Ok. I'll add that modification to the spec. Hopefully if there is a reason, someone will shout. | 17:21 |
| *** irenab_ has joined #openstack-nova | 17:21 | |
| mriedem | doffm: see ^ | 17:21 |
| *** sdake_ is now known as sdake | 17:21 | |
| mriedem | doffm: check the review that added it for comments, or the original bug to understand the use case | 17:21 |
| *** ljxiash has joined #openstack-nova | 17:21 | |
| alaski | that looks like it added the check to update and mentions it was already on create | 17:22 |
| *** ccarmack has quit IRC | 17:22 | |
| *** irenab has quit IRC | 17:23 | |
| *** irenab_ is now known as irenab | 17:23 | |
| *** itamarl has quit IRC | 17:23 | |
| mriedem | alaski: yeah https://review.openstack.org/#/c/78749/5/nova/db/sqlalchemy/api.py@5011 | 17:24 |
| mriedem | doffm: alaski: based on that, it seems like we should have a unique constraint on name then | 17:24 |
| *** doug-fish has joined #openstack-nova | 17:24 | |
| alaski | https://review.openstack.org/#/c/3035/ | 17:24 |
| *** eil397 has left #openstack-nova | 17:25 | |
| doffm | The original create code goes all the way back to aggregate inception. | 17:25 |
| alaski | https://review.openstack.org/#/c/3035/8/nova/db/sqlalchemy/migrate_repo/versions/071_add_host_aggregate_tables.py@34 | 17:25 |
| *** jwcroppe has joined #openstack-nova | 17:25 | |
| alaski | looks like it was meant to have that constraint | 17:26 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (1) https://review.openstack.org/280415 | 17:26 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (3) https://review.openstack.org/281901 | 17:26 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (4) https://review.openstack.org/282039 | 17:26 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove unused config option 'fake_call' https://review.openstack.org/282419 | 17:26 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (2) https://review.openstack.org/280438 | 17:26 |
| *** achanda has quit IRC | 17:27 | |
| mriedem | alaski: yeah weird that was dropped at some point https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/216_havana.py#L194 | 17:27 |
| *** Marga_ has joined #openstack-nova | 17:28 | |
| *** ChrisBenson has joined #openstack-nova | 17:28 | |
| *** Sukhdev_ has quit IRC | 17:28 | |
| doffm | I'm very confused. Dropping it seems wierd. Unless someone looked at the code and decided that because of the code the constraint wasn't needed. | 17:29 |
| *** claudiub has joined #openstack-nova | 17:29 | |
| *** sdake_ has joined #openstack-nova | 17:29 | |
| *** sdake has quit IRC | 17:29 | |
| alaski | I'm guessing it was just missed in the migration squash | 17:30 |
| doffm | Then its going back. (At least for the api db) | 17:31 |
| *** vladikr has joined #openstack-nova | 17:31 | |
| alaski | +1 | 17:32 |
| * alaski goes for a late lunch | 17:32 | |
| mriedem | wow | 17:32 |
| mriedem | armax: remember doing this? https://review.openstack.org/#/c/3035/ | 17:32 |
| *** mfedosin has quit IRC | 17:32 | |
| *** mylu has joined #openstack-nova | 17:32 | |
| *** kapil has joined #openstack-nova | 17:33 | |
| *** ayoung has joined #openstack-nova | 17:33 | |
| kapil | hi, i want to know how does the nova-scheduler get the metrics sent my monitors on nova-compute as notification ? | 17:35 |
| johnthetubaguy | mriedem: armax: wow a blast from our Citrix past there | 17:36 |
| mriedem | doffm: looks like the 133_folsom migration lost it | 17:36 |
| kapil | so, from the code, i see that metric notifications are sent every 60s from the compute node | 17:37 |
| *** ccarmack has joined #openstack-nova | 17:38 | |
| kapil | and the filter scheduler reads the metrics, computes the weights, etc... | 17:38 |
| *** ayoung has quit IRC | 17:38 | |
| kapil | i want to know how does the controller receive the notifications and convert them into metrics ? | 17:38 |
| *** ayoung has joined #openstack-nova | 17:39 | |
| mriedem | doffm: alaski: this is what dropped the UC https://review.openstack.org/#/c/13308/ | 17:40 |
| doffm | Man patches got through with far fewer reviews back then. | 17:41 |
| *** sdake has joined #openstack-nova | 17:42 | |
| *** david-lyle has quit IRC | 17:44 | |
| mriedem | it still seems that adding a UC on the aggregates.name column makes sense. it seems like soft delete and lack of foreign keys messed up this code back in essex/folsom | 17:44 |
| *** sdake_ has quit IRC | 17:44 | |
| *** gszasz has quit IRC | 17:44 | |
| mriedem | yeah they would read a deleted aggregate by name and then recreate it | 17:44 |
| mriedem | which would have it's old metadata | 17:44 |
| mriedem | so with the API DB we won't have soft delete, so that's one helper | 17:45 |
| kapil | also, are the notifications are stored in a db on the controller ? i cannot use ceilometer because apparently, as i understand, nova-scheduler doesn't read from ceilometer or does it ? | 17:45 |
| doffm | Wouldn't be a problem even if we did. All the other tables have something like UC(name, id, deleted). | 17:45 |
| doffm | Sorry, UC(name, deleted) | 17:46 |
| *** burt has quit IRC | 17:46 | |
| *** ChrisBenson has quit IRC | 17:46 | |
| mriedem | kapil: the notifications are not stored in the nova db, they are just emitted over rpc | 17:48 |
| mriedem | kapil: the scheduler is getting compute node stats from the resource tracker, which updates every 60 seconds by default | 17:48 |
| *** lpetrut has quit IRC | 17:49 | |
| kapil | https://github.com/openstack/nova/blob/stable/kilo/nova/scheduler/host_manager.py#L183 | 17:49 |
| *** ihrachys has quit IRC | 17:49 | |
| kapil | i see this is the function where the metrics are updated | 17:49 |
| mriedem | yeah, and that compute is a compute node record in the db https://github.com/openstack/nova/blob/stable/kilo/nova/scheduler/host_manager.py#L535 | 17:50 |
| mriedem | see https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L6460 in the compute manager | 17:50 |
| kapil | but the function get_all_host_states https://github.com/openstack/nova/blob/stable/kilo/nova/scheduler/host_manager.py#L525, it is called by the scheduler, not every 60s | 17:52 |
| kapil | so are the notifications dropped if they are not used ? | 17:52 |
| *** ayoung has quit IRC | 17:52 | |
| *** PaulMurray has quit IRC | 17:53 | |
| *** vilobhmm11 has joined #openstack-nova | 17:55 | |
| *** rcernin has quit IRC | 17:56 | |
| *** mriedem has quit IRC | 17:56 | |
| *** xyang1 has quit IRC | 17:58 | |
| *** ducttape_ has quit IRC | 17:58 | |
| *** achanda has joined #openstack-nova | 17:59 | |
| *** haukebruno has quit IRC | 18:00 | |
| *** vilobhmm111 has joined #openstack-nova | 18:00 | |
| *** vilobhmm11 has quit IRC | 18:02 | |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (1) https://review.openstack.org/280415 | 18:02 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (3) https://review.openstack.org/281901 | 18:02 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (4) https://review.openstack.org/282039 | 18:02 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Remove unused config option 'fake_call' https://review.openstack.org/282419 | 18:02 |
| openstackgerrit | Ed Leafe proposed openstack/nova: Improve the help text for the network options (2) https://review.openstack.org/280438 | 18:02 |
| *** david-lyle has joined #openstack-nova | 18:02 | |
| *** e0ne has joined #openstack-nova | 18:04 | |
| *** Jeffrey4l has quit IRC | 18:05 | |
| *** jichen has joined #openstack-nova | 18:06 | |
| *** sshen has quit IRC | 18:06 | |
| *** sshen has joined #openstack-nova | 18:07 | |
| kapil | thanks mriedem, i think i get it now. https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L644 | 18:08 |
| *** doug-fis_ has joined #openstack-nova | 18:11 | |
| *** mriedem has joined #openstack-nova | 18:11 | |
| *** doug-fi__ has joined #openstack-nova | 18:12 | |
| *** p01nt3r75 has joined #openstack-nova | 18:12 | |
| *** kapil has quit IRC | 18:13 | |
| *** e0ne has quit IRC | 18:14 | |
| *** doug-fish has quit IRC | 18:14 | |
| *** doug-fish has joined #openstack-nova | 18:15 | |
| *** doug-fis_ has quit IRC | 18:15 | |
| *** ducttape_ has joined #openstack-nova | 18:16 | |
| *** haukebruno has joined #openstack-nova | 18:16 | |
| *** zenoway has joined #openstack-nova | 18:17 | |
| *** doug-fi__ has quit IRC | 18:17 | |
| *** Bjoern_ has quit IRC | 18:20 | |
| *** yamahata has quit IRC | 18:21 | |
| *** e0ne has joined #openstack-nova | 18:24 | |
| *** ociuhandu has quit IRC | 18:24 | |
| mriedem | dansmith: wow the remove default flavors thread has exploded | 18:26 |
| dansmith | mriedem: and the patch merged while it was going with nary a -1 | 18:26 |
| dansmith | and a pretty impressive number of +2s and +1s I might add | 18:27 |
| *** pwnall has quit IRC | 18:27 | |
| *** ljxiash has quit IRC | 18:29 | |
| openstackgerrit | Chuck Carmack proposed openstack/nova: DB API changes for the nova-manage quota_usage_refresh command https://review.openstack.org/299441 | 18:29 |
| *** ayoung has joined #openstack-nova | 18:29 | |
| *** thumpba has quit IRC | 18:29 | |
| sdague | dansmith: damn, I think I missed the chance to +2 that | 18:30 |
| mriedem | dansmith: sounds like tim bell is asking for the dynamic flavors API extension :) | 18:30 |
| sdague | mriedem: no, he doesn't actually want that | 18:30 |
| mriedem | let me go dig up the old ibm grizzly code here quick... | 18:30 |
| dansmith | mriedem: no he's asking for sdague's thing | 18:30 |
| mriedem | sdague: he does! | 18:30 |
| sdague | mriedem: no, he wants my selector | 18:30 |
| dansmith | right | 18:30 |
| mriedem | so, first, i'm joking | 18:30 |
| dansmith | or for heat to have that, which I find insane that it doesn't | 18:30 |
| mriedem | second, i haven't finished the thread | 18:30 |
| dansmith | mriedem: more poo? | 18:30 |
| mriedem | no, i set the trap this time | 18:31 |
| sdague | I thought heat did | 18:31 |
| dansmith | sdague: get infra to INSERT INTO it for you | 18:31 |
| *** thumpba has joined #openstack-nova | 18:31 | |
| sdague | dansmith: heh | 18:31 |
| sdague | oh, wow, heat really doesn't | 18:32 |
| *** jdurgin1 has quit IRC | 18:32 | |
| sdague | man, I swore they had that before | 18:32 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate metadata functions work with API db https://review.openstack.org/297350 | 18:32 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Add aggregates tables to the API db. https://review.openstack.org/294806 | 18:32 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate host operations work against API db https://review.openstack.org/296597 | 18:32 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make AggregateList.get_ return API & cell db items https://review.openstack.org/296062 | 18:32 |
| openstackgerrit | Mark Doffman proposed openstack/nova: Make Aggregate.get operation favor the API db https://review.openstack.org/296014 | 18:32 |
| dansmith | it really doesn't matter.. if you're hard coding flavor names you're doing it wrong, | 18:32 |
| *** pushkaru has joined #openstack-nova | 18:33 | |
| dansmith | and we opened that assumption by having these | 18:33 |
| armax | mriedem: I did recall that blueprint | 18:33 |
| sdague | dansmith: agreed | 18:33 |
| sdague | nodepool does it right - https://github.com/openstack-infra/project-config/blob/71ff7b2b5bcd79c4e44502f997d9582b4c125caf/nodepool/nodepool.yaml#L664 | 18:33 |
| armax | mriedem: is there anything code of mine left in the tree? I thought that had been overhauled by the v2 version of it | 18:33 |
| *** mvk_ has quit IRC | 18:35 | |
| dansmith | sdague: of course it does because it's ... real | 18:35 |
| mriedem | armax: in the api? | 18:35 |
| armax | mriedem: maybe not in the api | 18:36 |
| mriedem | armax: this db model stuff is still in there | 18:36 |
| sdague | mriedem: let me know if you want me to write up the flavors selector API as a spec, it should be a pretty easy one | 18:37 |
| mriedem | sdague: dansmith: the thread about deprecation and removal of the image CLIs/APIs in novaclient is also getting surprisingly long | 18:37 |
| mriedem | sdague: up to you | 18:37 |
| sdague | I guess we'll see where the crowd gets to by tomorrow and evaluate | 18:37 |
| mriedem | i'm not sure people understood the point was 'nix an entry point to the api that we don't want people using' | 18:37 |
| *** angdraug has quit IRC | 18:37 | |
| sdague | right, well I think there were mixed concerns in that thread, so it went a couple of different directions | 18:38 |
| sdague | speaking of which did you post the remove of the baremetal commands from the CLI yet? | 18:38 |
| *** jwcroppe has quit IRC | 18:39 | |
| alaski | doffm: one consideration on adding a unique constraint is ensuring migration still works. presumably the only 'name' dupes are with deleted aggregates but it's something to consider | 18:40 |
| oomichi | mriedem: hi, about the shedule of the summit | 18:42 |
| oomichi | mriedem: it is fine to schedule it on 9 and 9:50 thuesday as you said on today nova-api meeting | 18:43 |
| mriedem | oomichi: oh, good, i have it in cheddar for the nova schedule already | 18:43 |
| *** ChrisBenson has joined #openstack-nova | 18:43 | |
| mriedem | sdague: that already merged | 18:43 |
| mriedem | sdague: https://review.openstack.org/#/c/301931/ | 18:43 |
| oomichi | mriedem: cool | 18:44 |
| doffm | alaski: I'll make sure to catch any exceptions in the (forthcoming) migrations patch. I guess we emit a big warning and skip the aggregate if a duplicate is found. | 18:44 |
| *** mfedosin has joined #openstack-nova | 18:44 | |
| sdague | mriedem: oh, we should deprecate the rest of it as well | 18:45 |
| sdague | and remove with the image commands | 18:45 |
| alaski | doffm: yeah. and we'll want to skip deleted aggregates which should mostly avoid the issue | 18:45 |
| mriedem | sdague: like list and show | 18:45 |
| sdague | mriedem: yeh | 18:45 |
| mriedem | sdague: i can get behind that | 18:45 |
| mriedem | i need to get the patch up for deprecating the image commands first | 18:46 |
| sdague | mriedem: ok, once you do that I'll follow with this | 18:46 |
| *** yassou has quit IRC | 18:46 | |
| *** e0ne has quit IRC | 18:46 | |
| *** lpetrut has joined #openstack-nova | 18:47 | |
| mriedem | ops meeting in 12 minutes in #openstack-meeting-4 | 18:48 |
| mriedem | i'll bring up the flavor thing again but it shouldn't be news | 18:48 |
| *** rlrossit has quit IRC | 18:48 | |
| * dansmith dons flak jacket | 18:49 | |
| *** rlrossit has joined #openstack-nova | 18:53 | |
| *** angdraug has joined #openstack-nova | 18:53 | |
| *** ihrachys has joined #openstack-nova | 18:55 | |
| *** ociuhandu has joined #openstack-nova | 18:56 | |
| *** mylu has quit IRC | 18:59 | |
| *** mylu has joined #openstack-nova | 19:01 | |
| *** Swami has quit IRC | 19:04 | |
| *** Swami has joined #openstack-nova | 19:06 | |
| *** xyang1 has joined #openstack-nova | 19:06 | |
| *** Sukhdev has joined #openstack-nova | 19:08 | |
| rlrossit | mriedem: alaski: how willing are we/should we be to impose objects in the cells v1 rpc api? | 19:11 |
| *** sc68cal has joined #openstack-nova | 19:12 | |
| rlrossit | (for reference: https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L626 https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3756) | 19:12 |
| bauzas | woah, I just left for 2 hours and just saw the flavor thread... :p | 19:12 |
| mriedem | rlrossit: not very | 19:12 |
| rlrossit | mriedem: so then I can't get rid of DictCompat until we get rid of cells v1..? | 19:12 |
| alaski | yeah, we basically said whatever hasn't been converted at this point stays | 19:13 |
| alaski | rlrossit: where is DictCompat needed for cellsv1 | 19:13 |
| rlrossit | alaski: cellsv1 is passing a dict back to the api instead of an object, which is causing the cells job to blow up | 19:14 |
| * rlrossit digs up link | 19:14 | |
| rlrossit | https://review.openstack.org/#/c/301905/ | 19:14 |
| rlrossit | http://logs.openstack.org/05/301905/2/check/gate-tempest-dsvm-cells/5c4a1c2/logs/screen-n-api.txt.gz?level=ERROR | 19:14 |
| *** burt has joined #openstack-nova | 19:15 | |
| alaski | instance actions were always objects. I think dansmith gave me a gold star or something for being the first fully object compatible thing | 19:15 |
| rlrossit | so we need to maintain dict syntax on all InstanceActions things | 19:15 |
| * alaski looks | 19:15 | |
| *** ayoung has quit IRC | 19:16 | |
| *** ayoung has joined #openstack-nova | 19:16 | |
| *** sheel has quit IRC | 19:17 | |
| *** ducttape_ has quit IRC | 19:17 | |
| mriedem | so cells v1 turns it into a primitive? | 19:18 |
| mriedem | i don't get it | 19:18 |
| rlrossit | mriedem: cells never touches objects | 19:19 |
| rlrossit | it's calling the raw rpc client instead of handing the context to an object/objectified function | 19:19 |
| alaski | https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L756 | 19:20 |
| *** e0ne has joined #openstack-nova | 19:20 | |
| mriedem | ooo https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L754 | 19:21 |
| mriedem | yeah | 19:21 |
| alaski | rlrossit: I think it would be acceptable to convert the dict to an object in https://github.com/openstack/nova/blob/master/nova/cells/rpcapi.py#L362 | 19:21 |
| mriedem | so couldn't you just convert that dict to an object? | 19:21 |
| rlrossit | mriedem: can't I just do https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3759-L3761 | 19:22 |
| rlrossit | the same way the regular rpc api does? | 19:22 |
| alaski | the InstanceAction[List] object doesn't have logic for querying the correct cell, you'd be querying the top cell db | 19:23 |
| rlrossit | alaski: what are all of these different actions_get doing? | 19:24 |
| rlrossit | I get the one in the compute api, but there's one in cells rpcapi and cells messaging | 19:24 |
| mriedem | gather round children | 19:25 |
| alaski | heh | 19:25 |
| mriedem | it's time to hear the tale of cells v1 | 19:25 |
| alaski | so when cells is enabled compute/api doesn't run in nova-api compute/cells_api does | 19:25 |
| *** haukebruno has quit IRC | 19:25 | |
| alaski | which sends the call to cells rpcapi, which calls to the nova-cells service | 19:25 |
| rlrossit | I got that part | 19:25 |
| alaski | at that point cells/messaging runs which calls to nova-cells in the child cell | 19:26 |
| rlrossit | ah | 19:26 |
| alaski | which then runs compute/api or whatever | 19:26 |
| rlrossit | so the there is no messaging equivalent in the non-cells world because that is all run in nova-cells? | 19:26 |
| *** Sukhdev has quit IRC | 19:26 | |
| alaski | right | 19:26 |
| rlrossit | so I can't give the object function the message.ctxt like all of the other objects things in that class do? | 19:27 |
| rlrossit | because that's the wrong context? | 19:27 |
| rlrossit | (talkin' 'bout https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L756 here) | 19:27 |
| alaski | you could | 19:28 |
| alaski | if you queried using objects.InstanceMapping.get_* that would work fine | 19:28 |
| dansmith | alaski: are you thinking about shelve? I'm pretty sure instance actions predate objects | 19:28 |
| *** irenab_ has joined #openstack-nova | 19:28 | |
| alaski | dansmith: ahh right, that may be it | 19:28 |
| dansmith | alaski: that gold star will haunt me forever, like shelve itself | 19:29 |
| rlrossit | alaski: should I do that in a prequel patch to my DictCompat one? | 19:29 |
| alaski | rlrossit: the problem is that when that method returns the result goes over RPC, and we don't want to change that format for cellsv1 | 19:29 |
| rlrossit | why can't cellsv1 be cool | 19:30 |
| alaski | dansmith: heh. I am forever stained by that feature | 19:30 |
| rlrossit | it's so square | 19:30 |
| alaski | ikr | 19:30 |
| dansmith | alaski: heh, stained is particularly dirty sounding.. kudos | 19:30 |
| mriedem | rlrossit: you can't just start sending an object back w/o an rpc version change | 19:30 |
| mriedem | to handle the format difference | 19:30 |
| alaski | rlrossit: if you convert to an object at https://github.com/openstack/nova/blob/master/nova/cells/rpcapi.py#L362 with the result of the call you avoid changing any RPC | 19:30 |
| *** irenab has quit IRC | 19:31 | |
| *** irenab_ is now known as irenab | 19:31 | |
| *** ametts has joined #openstack-nova | 19:31 | |
| *** mvk_ has joined #openstack-nova | 19:31 | |
| mriedem | alaski: why there and not https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L633 ? | 19:31 |
| *** ducttape_ has joined #openstack-nova | 19:32 | |
| mriedem | because https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L633 is what the rest api is using isn't it? | 19:32 |
| alaski | no reason. that would work as well | 19:32 |
| rlrossit | wait I'm confused now | 19:32 |
| alaski | mriedem: yep | 19:32 |
| *** pwnall has joined #openstack-nova | 19:33 | |
| *** Goneri has quit IRC | 19:35 | |
| alaski | rlrossit: the call chain is API->compute/cells_api->compute/rpcapi->RPC. You can convert anytime between API and RPC without affecting upgrades | 19:35 |
| rlrossit | that ain't what mriedem just told me | 19:35 |
| mriedem | it isn't? | 19:36 |
| alaski | woops, I meant cells/rpcapi | 19:36 |
| *** mylu has quit IRC | 19:36 | |
| alaski | yeah, he recommend step two in the chain, I had said step three | 19:36 |
| mriedem | do we have an example of this in either already for precedent? | 19:37 |
| *** salv-orlando has joined #openstack-nova | 19:37 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/compute/cells_api.py#L516 | 19:37 |
| mriedem | service_get_all | 19:37 |
| *** Sukhdev has joined #openstack-nova | 19:38 | |
| rlrossit | but mriedem said I couldn't do it in cells/rpcapi because that's server side | 19:38 |
| *** pwnall has quit IRC | 19:38 | |
| rlrossit | which we don't want because that would require an rpc version bump | 19:39 |
| rlrossit | or maybe I'm just a fool that can't understand mouth words, only text words | 19:40 |
| alaski | I think he was referring to cells/messaging | 19:40 |
| bauzas | dansmith: stupid question but shouldn't we comment the new service version given by https://review.openstack.org/#/c/279313/14/nova/objects/service.py in nova.compute.rpcapi ? | 19:40 |
| dansmith | bauzas: what do you man? | 19:40 |
| dansmith | mean | 19:40 |
| bauzas | dansmith: I understand your version bump for signaling that the compute is new | 19:40 |
| bauzas | dansmith: I mean this https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L320 | 19:41 |
| bauzas | adding a new docstring line for 4.12 | 19:41 |
| rlrossit | dansmith: bauzas: shouldn't that number be 4.12? | 19:41 |
| dansmith | bauzas: this isn't 4.12 | 19:41 |
| dansmith | no | 19:41 |
| dansmith | this doesn't require a compute rpc version bump | 19:42 |
| *** rk4n_ has joined #openstack-nova | 19:42 | |
| bauzas | oh I see | 19:42 |
| *** Sukhdev has quit IRC | 19:42 | |
| *** pushkaru has quit IRC | 19:42 | |
| *** pumarani__ has joined #openstack-nova | 19:42 | |
| bauzas | that doesn't change the RPC contract but you still want to signal this is a new Compute version | 19:42 |
| bauzas | right? | 19:42 |
| dansmith | right, we don't even care about the computes | 19:43 |
| dansmith | we care about conductors and scheduler and api | 19:43 |
| dansmith | https://review.openstack.org/#/c/279313/14/nova/objects/compute_node.py | 19:43 |
| dansmith | L318 | 19:43 |
| *** rk4n has quit IRC | 19:44 | |
| bauzas | gotcha | 19:44 |
| dansmith | bauzas: the service version history has the compute rpc version in it, but it hasn't changed | 19:44 |
| *** salv-orlando has quit IRC | 19:44 | |
| dansmith | see v4.6 for example | 19:45 |
| dansmith | er. v3 and 4 | 19:45 |
| *** mylu has joined #openstack-nova | 19:45 | |
| bauzas | dansmith: yeah, my eyes made me wrong, I thought you were adding a new 4.12 | 19:45 |
| dansmith | okay | 19:45 |
| * bauzas doesn't recommend reviewing when the TV is on with Grey's Anatomy (for family purpose) | 19:46 | |
| *** thumpba has quit IRC | 19:47 | |
| *** aswadr_ has quit IRC | 19:50 | |
| *** e0ne has quit IRC | 19:50 | |
| *** salv-orlando has joined #openstack-nova | 19:54 | |
| *** salv-orlando has quit IRC | 19:54 | |
| *** pwnall has joined #openstack-nova | 19:54 | |
| *** salv-orlando has joined #openstack-nova | 19:55 | |
| *** john51_ has joined #openstack-nova | 19:55 | |
| *** eharney has quit IRC | 19:55 | |
| *** john51 has quit IRC | 19:59 | |
| *** mylu has quit IRC | 19:59 | |
| *** kapil has joined #openstack-nova | 19:59 | |
| *** rk4n has joined #openstack-nova | 20:02 | |
| auggy | bauzas: i was in the grocery store yesterday and saw something that made me think of you! https://goo.gl/photos/y9uAeSHAMSUfvewN6 | 20:02 |
| *** mkrcmari__ has joined #openstack-nova | 20:03 | |
| bauzas | heh | 20:03 |
| bauzas | "holland". Bah. | 20:03 |
| auggy | haha | 20:04 |
| *** mgagne_ is now known as mgagne | 20:04 | |
| mnaser | is python-novaclient incompatible with keystone v3? | 20:05 |
| mnaser | (it seems like it) | 20:05 |
| *** rk4n_ has quit IRC | 20:05 | |
| *** kmartin has quit IRC | 20:06 | |
| bauzas | mnaser: it should work, see http://docs.openstack.org/developer/python-novaclient/api.html | 20:06 |
| *** mvk_ has quit IRC | 20:06 | |
| *** kmartin has joined #openstack-nova | 20:07 | |
| mnaser | i cant get it to work and my novaclient/client.py has v2 references only | 20:07 |
| mnaser | https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L568-L597 | 20:07 |
| mnaser | i end up in the else: clause (self.version == 3), it doesnt try to do v3 auth anywhere there | 20:08 |
| *** mat128 has joined #openstack-nova | 20:08 | |
| *** mat128 has left #openstack-nova | 20:08 | |
| *** eharney has joined #openstack-nova | 20:09 | |
| *** rcernin has joined #openstack-nova | 20:09 | |
| *** jichen has quit IRC | 20:10 | |
| mriedem | mnaser: i don't think it does | 20:11 |
| bauzas | mnaser: if you see the doc, you'll see that it doesn't use the authenticate() method | 20:11 |
| bauzas | "Alternatively, you can create a client instance using the keystoneauth session API:" | 20:11 |
| mriedem | mnaser: see https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/functional/hooks/post_test_hook.sh#L33 | 20:11 |
| mnaser | oh so it relies on sessions to support v3 | 20:12 |
| mnaser | which im assuming are/were not implemented for the shell | 20:12 |
| bauzas | mnaser: yup | 20:12 |
| mnaser | grr | 20:12 |
| bauzas | that's my assumptions | 20:12 |
| mriedem | oh i guess https://github.com/openstack/python-novaclient/blob/master/novaclient/tests/functional/hooks/post_test_hook.sh#L33 predates the keystoneauth stuff added in mitaka | 20:12 |
| bauzas | but melwitt could comment that | 20:12 |
| mnaser | customer wants RBAC so v3 domains are the perfect solution | 20:12 |
| melwitt | this commit might make things work, it's only on master https://github.com/openstack/python-novaclient/commit/e458fab8efdaab327c8f07a0771e4d3571580a41 | 20:13 |
| *** yamahata has joined #openstack-nova | 20:13 | |
| mnaser | but this means that a lot of their stuff will break and they'll be forced to use the openstack client | 20:13 |
| bauzas | which is IMHO not a bad thing | 20:13 |
| *** nic has left #openstack-nova | 20:13 | |
| *** nic has joined #openstack-nova | 20:13 | |
| mnaser | bauzas: while i agree to an extent, i think the problem is just that it's a difficult transition | 20:14 |
| mnaser | it would have been nice for both to be available | 20:14 |
| *** pcaruana has quit IRC | 20:14 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 20:14 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Keep BuildRequest db entry around longer https://review.openstack.org/302425 | 20:14 |
| mriedem | the cli is supposed to be more or less deprecated in favor of openstackclient | 20:14 |
| *** matrohon has joined #openstack-nova | 20:14 | |
| mriedem | which is also the only cli for keystone now | 20:14 |
| mnaser | that's fine, but i think python-novaclient should still function as long as operations are still missing from openstackclient | 20:15 |
| mnaser | i may be out of the loop but there were some missing things from nova in there | 20:15 |
| mriedem | yes, that's why we haven't officially deprecated novaclient | 20:16 |
| bauzas | well | 20:16 |
| mriedem | like for microversions in the cli | 20:16 |
| bauzas | possibly adding v3 support in authenticate() doesn't sound bad | 20:16 |
| bauzas | that's not a CLI thing, just an other way to auth | 20:16 |
| mnaser | yep, but it sits in an awkward situation when a user has to maintain two sets of env variables to work one or the other | 20:16 |
| melwitt | mnaser: the commit I linked earlier might make novaclient do keystone v3 auth. I say might because I haven't tried it | 20:16 |
| mnaser | melwitt: yeah, i had a look and i think it'll trick it enough to make it work | 20:17 |
| mnaser | let me try to drop it in and see | 20:17 |
| bauzas | melwitt: sorry, missed your ping then, could you please give me the commit too ? | 20:17 |
| melwitt | it adds default values for domain related options so it's not really complete. and I wasn't sure if it would do enough | 20:17 |
| bauzas | for my knowledge | 20:17 |
| melwitt | bauzas: this one https://github.com/openstack/python-novaclient/commit/e458fab8efdaab327c8f07a0771e4d3571580a41 | 20:18 |
| bauzas | okay, so I guess https://github.com/openstack/python-novaclient/commit/e458fab8efdaab327c8f07a0771e4d3571580a41 is prereq for modifying https://github.com/openstack/python-novaclient/blob/master/novaclient/client.py#L568-L597 to add v3 | 20:19 |
| *** vtech_ has quit IRC | 20:19 | |
| mnaser | melwitt: it does a GET to http://../v3 (OS_AUTH_URL, then to http://.../v3/v2.0/tokens afterwards so, unless there's another patch | 20:19 |
| *** vtech has joined #openstack-nova | 20:19 | |
| bauzas | oh unrelated https://review.openstack.org/#/c/291453/ :( | 20:21 |
| *** e0ne has joined #openstack-nova | 20:22 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Create Instance from BuildRequest if not in a cell https://review.openstack.org/263927 | 20:23 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Put more into compute.api._populate_instance_for_create https://review.openstack.org/300580 | 20:23 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Keep BuildRequest db entry around longer https://review.openstack.org/302425 | 20:23 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Change BuildRequest to contain a serialized instance https://review.openstack.org/298455 | 20:23 |
| melwitt | bauzas: bah, I liked using rejoin-stack.sh to start all the screens again after I killed them | 20:26 |
| bauzas | anyway, I'm not knowing all the issues with that one, so my comment was rather because I'm a bit conservative :p | 20:27 |
| *** mylu has joined #openstack-nova | 20:27 | |
| melwitt | andreykurilin__: hey, was https://review.openstack.org/#/c/289369/ supposed to make keystone v3 auth work in novaclient or it's a partial work toward that? | 20:28 |
| *** e0ne has quit IRC | 20:28 | |
| andreykurilin__ | melwitt: hi! novaclient should work with keystone v3 from the shell even without my patch(but you will need to specify extra arguments) | 20:29 |
| openstackgerrit | Chris Dent proposed openstack/nova: WIP resource-pools/placement api https://review.openstack.org/293104 | 20:30 |
| *** yamahata has quit IRC | 20:30 | |
| andreykurilin__ | metwitt: when we are talking about shell, novaclient uses Session object from keystoneauth, so we don't need take care about v2/v3 compatibility | 20:30 |
| andreykurilin__ | melwitt^ | 20:30 |
| andreykurilin__ | melwitt: but when we talk about novaclient-as-a-lib, our custom HTTPClient takes care about authentication and I'm not sure that it works with keystone v3 | 20:32 |
| melwitt | andreykurilin__: okay, I'm trying to understand the part where the shell can call the keystone tokens api to get a token. that doesn't work with v3, right? | 20:33 |
| andreykurilin__ | melwitt: probably, we need to deprecate HTTPClient and support only session | 20:33 |
| andreykurilin__ | melwitt: it should work | 20:34 |
| *** ihrachys has quit IRC | 20:34 | |
| andreykurilin__ | melwitt: I checked novaclient with devstack (keystone v3 was chosen by default) | 20:36 |
| *** sdake_ has joined #openstack-nova | 20:37 | |
| *** matrohon has quit IRC | 20:37 | |
| andreykurilin__ | and it works | 20:37 |
| melwitt | hm, okay. mnaser tried out your patch and it ended up calling v2.0 tokens, which would mean something made it fall back to the HTTPClient | 20:37 |
| *** sdake has quit IRC | 20:39 | |
| *** vtech has quit IRC | 20:39 | |
| *** ljxiash has joined #openstack-nova | 20:41 | |
| *** sdake has joined #openstack-nova | 20:42 | |
| *** kapil has quit IRC | 20:42 | |
| *** sdake_ has quit IRC | 20:44 | |
| melwitt | mnaser: are you using an auth plugin other than keystone or using bypass url or os_cache or volume_service_name? those are the things that will make novaclient not use keystone session for the auth in the shell | 20:44 |
| *** ljxiash has quit IRC | 20:45 | |
| mnaser | melwitt: OS_CACHE=1 | 20:45 |
| mnaser | i presume that may have been causing false alerts? | 20:45 |
| *** vtech has joined #openstack-nova | 20:47 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Migrate compute node resource information to Inventory objects https://review.openstack.org/279313 | 20:47 |
| melwitt | mnaser: okay, that will make it fall back to the non-session based auth (I don't know the details on why token caching can't be used alongside keystone session) | 20:47 |
| *** mvk has joined #openstack-nova | 20:50 | |
| mriedem | alaski: cells meeting in 10 min? | 20:50 |
| alaski | mriedem: yes sir | 20:50 |
| mnaser | melwitt: it worked with OS_CACHE=0 | 20:50 |
| mriedem | nova/cinder meeting in #openstack-meeting-cp in 10 minutes too | 20:50 |
| alaski | mriedem: time to pick which of your children is your favorite | 20:51 |
| mriedem | alaski: i have to attend the cinder one | 20:51 |
| alaski | :( | 20:51 |
| melwitt | mnaser: nice | 20:51 |
| mriedem | i'm confident the cells one is in good hands :) | 20:51 |
| mnaser | is there a schedule for python-novaclient releases? | 20:51 |
| *** salv-orl_ has joined #openstack-nova | 20:51 | |
| mnaser | be nice if we can get this in a newer release soon | 20:52 |
| *** mkrcmari__ has quit IRC | 20:52 | |
| openstackgerrit | Ryan Rossiter proposed openstack/nova: Remove DictCompat from InstanceAction https://review.openstack.org/301905 | 20:52 |
| melwitt | andreykurilin__: it turned out to be OS_CACHE=1, that will prevent using the keystone session. it worked with OS_CACHE=0. thanks for all the guidance | 20:52 |
| alaski | mriedem: heh. I don't think there's anything in our meeting you can't catch up on later if you want | 20:53 |
| mriedem | mnaser: i plan on doing a novaclient 4.0 release next week | 20:54 |
| mnaser | oh, so i could have used it without the patch if i had explictly listed the domain | 20:54 |
| mikal | . | 20:54 |
| mriedem | .. | 20:54 |
| mikal | mriedem: I did those two reviews yesterday, but I don't have +2 on stable/mitaka | 20:54 |
| mnaser | mriedem: oh that's good. is novaclient still the go-to for being used as a library or should we migrate to something else if we're going to rework our stuff for keystonev3 | 20:54 |
| melwitt | mnaser: right, the os_project_domain_id and os_user_domain_id | 20:54 |
| andreykurilin__ | melwitt: nice | 20:55 |
| *** salv-orlando has quit IRC | 20:55 | |
| mriedem | mikal: only the release team has +2 on stable/mitaka until it's released | 20:55 |
| mikal | mriedem: ahhh, that makes sense | 20:55 |
| *** lpetrut has quit IRC | 20:55 | |
| *** lpetrut has joined #openstack-nova | 20:56 | |
| dansmith | alaski: all the more reason to keep it short | 20:56 |
| dansmith | alaski: this is my last thing planned for the day and I'm dying to get up | 20:56 |
| *** rk4n has quit IRC | 20:56 | |
| alaski | dansmith: +1 | 20:56 |
| melwitt | mnaser: the library part of novaclient will stay. it's just you have to get the keystone auth part elsewhere, either from keystoneclient lib itself or openstackclient as a lib, if that's a thing | 20:56 |
| dansmith | alaski: hmm, is that +1 for short meeting or +1 for "please get up and leave" ? | 20:56 |
| mriedem | mnaser: novaclient is still ok for the python api bindings, but it's really just meant for usage between services now | 20:57 |
| *** knikolla has quit IRC | 20:57 | |
| mriedem | mnaser: people should probably be moving to things like openstacksdk | 20:57 |
| alaski | dansmith: ...both? the former really | 20:57 |
| mnaser | melwitt: we use keystone sessions so we're good on that, good to know, so i think we should switch keystone sessions to v3 and get by | 20:57 |
| melwitt | mnaser: as long as you pass a keystone v3 token in when creating a novaclient object, it works fine as it just passes that along | 20:57 |
| dansmith | alaski: hrmph, well, I can leave.. but who will make snarky comments and start arguments on the list in my absence? | 20:57 |
| mnaser | yeah, anyways even with v2 right now we're just feeding the client a token (because we share the token with different services so we grab it first) | 20:58 |
| mriedem | i'll still be in both rooms | 20:58 |
| mriedem | i can snark it up | 20:58 |
| melwitt | mnaser: ah, okay | 20:58 |
| *** cleong has quit IRC | 20:58 | |
| *** busterswt has quit IRC | 20:59 | |
| alaski | dansmith: your inflammatory presence will be appreciated | 20:59 |
| mnaser | i'll do some more testing and if we successfully/smoothly change into v3 as a matter of changing the session provided to novaclient... great success! | 20:59 |
| dansmith | alaski: I'm ... glad? :) | 20:59 |
| alaski | hah | 20:59 |
| melwitt | alaski: +1 | 20:59 |
| melwitt | mnaser: cool, good luck! | 20:59 |
| bauzas | alaski: dansmith: I can nitpick if needed :p | 21:00 |
| *** busterswt has joined #openstack-nova | 21:00 | |
| mnaser | also if anyone on stable team doesnt mind checking this out for a few minutes, already got 3x +1s and tested, its a backport as utilization aware scheduler is broken in stable/liberty | 21:00 |
| *** burgerk has quit IRC | 21:01 | |
| *** Sukhdev has joined #openstack-nova | 21:01 | |
| *** raildo is now known as raildo-afk | 21:02 | |
| *** jistr|off has quit IRC | 21:04 | |
| *** mvk_ has joined #openstack-nova | 21:08 | |
| *** baoli_ has quit IRC | 21:09 | |
| *** mvk has quit IRC | 21:10 | |
| mnaser | well, it would help if i said which review it was.. https://review.openstack.org/#/c/300134/ heh | 21:11 |
| *** claudiub|2 has joined #openstack-nova | 21:11 | |
| *** knikolla has joined #openstack-nova | 21:13 | |
| *** doug-fish has quit IRC | 21:13 | |
| *** doug-fish has joined #openstack-nova | 21:13 | |
| *** doug-fish has quit IRC | 21:14 | |
| *** doug-fish has joined #openstack-nova | 21:14 | |
| *** cdent has quit IRC | 21:14 | |
| *** claudiub has quit IRC | 21:15 | |
| mriedem | already done?! | 21:15 |
| alaski | melwitt: for now we'll have the single command to go from non-cells to simple single cell, we could look at doing something like that for the cellsv1->v2 case though | 21:15 |
| *** Sukhdev has quit IRC | 21:16 | |
| melwitt | alaski: yeah, gotcha. the cells v1 to v2 case was what I was thinking of originally, when I had thought we would check for the cells table and use the info in it to create cell v2 cell mappings. but the thing missing there would be the db connections for getting the hosts | 21:16 |
| *** doug-fish has quit IRC | 21:18 | |
| dansmith | I'm cool with that being a script or a set of scripts | 21:19 |
| alaski | melwitt: yeah. I'm also not sure that anyone running v1 has a setup that they want to migrate all at once. I always figured they'd do it one cell at a time in a planned way | 21:19 |
| alaski | we could probably have a script to run once per cell, which is basically map_cell_and_hosts | 21:20 |
| melwitt | alaski: hm, yeah. I didn't consider that | 21:20 |
| *** doug-fish has joined #openstack-nova | 21:21 | |
| *** sdague has quit IRC | 21:24 | |
| *** mkrcmari__ has joined #openstack-nova | 21:25 | |
| *** doug-fish has quit IRC | 21:25 | |
| *** rcernin has quit IRC | 21:27 | |
| *** jwcroppe has joined #openstack-nova | 21:28 | |
| *** knikolla has quit IRC | 21:28 | |
| *** mvk_ has quit IRC | 21:28 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize vif & vol_utils options https://review.openstack.org/302446 | 21:29 |
| *** knikolla has joined #openstack-nova | 21:29 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova: Config options: Centralize vif & vol_utils options https://review.openstack.org/302446 | 21:29 |
| *** rtheis has quit IRC | 21:31 | |
| *** mfedosin has quit IRC | 21:31 | |
| *** vilobhmm111 has quit IRC | 21:31 | |
| *** vilobhmm11 has joined #openstack-nova | 21:32 | |
| diana_clarke | Does Nova have any mentoring programs? Something that pairs new contributors with experienced contributors, for example? | 21:32 |
| *** jwcroppe has quit IRC | 21:33 | |
| *** knikolla has quit IRC | 21:36 | |
| *** busterswt has quit IRC | 21:36 | |
| *** jwcroppe has joined #openstack-nova | 21:36 | |
| *** knikolla has joined #openstack-nova | 21:37 | |
| dansmith | diana_clarke: pick an experienced person, pair ;) | 21:39 |
| *** ayoung has quit IRC | 21:39 | |
| *** rfolco has quit IRC | 21:40 | |
| dansmith | I think this says that but in more words: http://docs.openstack.org/developer/nova/how_to_get_involved.html | 21:41 |
| *** thorst has quit IRC | 21:44 | |
| *** rlrossit has quit IRC | 21:44 | |
| *** thorst has joined #openstack-nova | 21:44 | |
| diana_clarke | dansmith: yup, I've read that. thanks | 21:46 |
| doffm | diana_clarke: I guess as part of openstack we participate in outreachy. https://wiki.openstack.org/wiki/Outreachy | 21:47 |
| doffm | I don't see any nova people on the mentors list though. | 21:47 |
| *** ChrisBenson has quit IRC | 21:49 | |
| *** Sukhdev has joined #openstack-nova | 21:49 | |
| *** zenoway has quit IRC | 21:49 | |
| doffm | There is also this from women of openstack. https://wiki.openstack.org/wiki/Mentors | 21:52 |
| mriedem | dansmith: looks like i replied in an old patch set https://review.openstack.org/#/c/279313/14/nova/objects/compute_node.py@376 | 21:52 |
| *** thorst has quit IRC | 21:53 | |
| diana_clarke | doffm: thanks! | 21:53 |
| mriedem | doesn't really need addressing | 21:53 |
| *** knikolla has quit IRC | 21:53 | |
| mriedem | dansmith is the unofficial welcome committee czar | 21:53 |
| *** Sukhdev has quit IRC | 21:53 | |
| *** sdake has quit IRC | 21:54 | |
| dansmith | mriedem: by stripping out the updates, I meant "not putting the desired values in the old columns" | 21:54 |
| dims | diana_clarke : i put my name for Nova/Oslo/Magnum - https://wiki.openstack.org/wiki/Outreachy | 21:55 |
| dims | diana_clarke : one candidate was interested in Magnum | 21:55 |
| mriedem | dansmith: yeah, and setting this more or less marker which is 0 | 21:55 |
| dansmith | mriedem: I'm still looking for feedback on those two issues from the other two people, but I'll try to update those comments for clarification | 21:55 |
| *** ChrisBenson has joined #openstack-nova | 21:56 | |
| mriedem | dansmith: yeah i think i'm good otherwise | 21:57 |
| *** stackdump has joined #openstack-nova | 21:57 | |
| *** stackdump has quit IRC | 21:57 | |
| mriedem | i haven't looked at tests much really, but i'm sure it's all top shelve :) | 21:57 |
| mriedem | *shelf | 21:57 |
| mriedem | damn | 21:57 |
| dansmith | mriedem: okay I just updated for that functional failure, but I'll try to poke jaypipes tomorrow if he decides to grace us | 21:58 |
| *** Sukhdev has joined #openstack-nova | 21:58 | |
| *** mdrabe has quit IRC | 21:58 | |
| dansmith | I'm guessing that the reserved memory thing comes out in the wash in how things are reported and once we have compute nodes updating inventories themselves they will be able to set their reserved to be nonzero and report the actual total | 22:00 |
| *** tbachman_ has joined #openstack-nova | 22:01 | |
| *** ChrisBenson has quit IRC | 22:01 | |
| *** zenoway has joined #openstack-nova | 22:01 | |
| openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Simple setup/migration command https://review.openstack.org/301273 | 22:01 |
| openstackgerrit | Andrew Laski proposed openstack/nova: Split out part of map_cell_and_hosts to return a uuid https://review.openstack.org/302451 | 22:01 |
| *** tbachman has quit IRC | 22:02 | |
| *** tbachman_ is now known as tbachman | 22:02 | |
| *** Sukhdev has quit IRC | 22:03 | |
| *** burt has quit IRC | 22:04 | |
| *** lpetrut has quit IRC | 22:04 | |
| *** pumarani__ has quit IRC | 22:05 | |
| *** mriedem has quit IRC | 22:05 | |
| *** zenoway has quit IRC | 22:06 | |
| *** gongysh has quit IRC | 22:08 | |
| *** ctrath has quit IRC | 22:09 | |
| *** ccarmack has quit IRC | 22:09 | |
| *** gongysh has joined #openstack-nova | 22:09 | |
| *** baoli has joined #openstack-nova | 22:10 | |
| *** claudiub has joined #openstack-nova | 22:13 | |
| *** gongysh has quit IRC | 22:14 | |
| *** gongysh has joined #openstack-nova | 22:15 | |
| *** claudiub|2 has quit IRC | 22:16 | |
| *** dimtruck is now known as zz_dimtruck | 22:19 | |
| *** baoli has quit IRC | 22:20 | |
| *** pushkaru has joined #openstack-nova | 22:20 | |
| *** baoli has joined #openstack-nova | 22:20 | |
| *** ametts has quit IRC | 22:21 | |
| *** haplo37 has quit IRC | 22:23 | |
| *** Sukhdev has joined #openstack-nova | 22:24 | |
| *** vilobhmm111 has joined #openstack-nova | 22:25 | |
| *** ducttape_ has quit IRC | 22:27 | |
| *** vilobhmm11 has quit IRC | 22:27 | |
| *** ccarmack has joined #openstack-nova | 22:29 | |
| *** edtubill has quit IRC | 22:29 | |
| *** achanda has quit IRC | 22:32 | |
| *** tbachman_ has joined #openstack-nova | 22:32 | |
| *** tbachman has quit IRC | 22:33 | |
| *** tbachman_ is now known as tbachman | 22:33 | |
| *** tull has joined #openstack-nova | 22:33 | |
| tull | I turned on H/T on a few compute nodes in my cloud - how do I get the scheduler/database to recognize this adjustment in vcpus? | 22:33 |
| *** ccarmack has quit IRC | 22:33 | |
| *** eharney has quit IRC | 22:35 | |
| *** ctrath has joined #openstack-nova | 22:38 | |
| *** achanda has joined #openstack-nova | 22:38 | |
| *** BobBall-1WOL has joined #openstack-nova | 22:41 | |
| *** ivase_ has quit IRC | 22:41 | |
| dansmith | tull: see topic please | 22:42 |
| *** ivase has joined #openstack-nova | 22:42 | |
| * tbachman notes that his topic seems to read “npm lint jobs are failing due to a problem with npm registry. The problem is under investigation, and we will update once the issue is solved.” atm | 22:43 | |
| dansmith | hah, damn openstack bot | 22:43 |
| tbachman | lol | 22:43 |
| dansmith | tull: this channel is for nova development discussion. #openstack is for usage issues | 22:44 |
| *** BobBall-AWOL has quit IRC | 22:44 | |
| tull | dansmith: ok thanks, sorry for the interrupt | 22:44 |
| dansmith | tull: np | 22:45 |
| *** tull has left #openstack-nova | 22:45 | |
| *** gcb has quit IRC | 22:45 | |
| *** mrkz has quit IRC | 22:49 | |
| *** zenoway has joined #openstack-nova | 22:49 | |
| *** gongysh has quit IRC | 22:49 | |
| *** gongysh has joined #openstack-nova | 22:49 | |
| *** gcb has joined #openstack-nova | 22:50 | |
| *** thorst has joined #openstack-nova | 22:50 | |
| *** pwnall has quit IRC | 22:52 | |
| *** zenoway has quit IRC | 22:53 | |
| *** angdraug has quit IRC | 22:56 | |
| *** thorst has quit IRC | 22:57 | |
| *** zz_dimtruck is now known as dimtruck | 22:58 | |
| *** vtech has quit IRC | 22:58 | |
| *** vtech has joined #openstack-nova | 22:58 | |
| *** ayoung has joined #openstack-nova | 23:09 | |
| *** mgoddard has quit IRC | 23:09 | |
| *** mgoddard has joined #openstack-nova | 23:10 | |
| *** gcb has quit IRC | 23:11 | |
| *** gcb has joined #openstack-nova | 23:11 | |
| *** zenoway has joined #openstack-nova | 23:13 | |
| *** pushkaru has quit IRC | 23:14 | |
| *** Sukhdev has quit IRC | 23:14 | |
| *** achanda has quit IRC | 23:15 | |
| *** zenoway has quit IRC | 23:17 | |
| *** ZZelle_ has joined #openstack-nova | 23:18 | |
| *** achanda has joined #openstack-nova | 23:22 | |
| *** claudiub has quit IRC | 23:25 | |
| *** gcb has quit IRC | 23:33 | |
| *** dimtruck is now known as zz_dimtruck | 23:34 | |
| *** jamielennox|away is now known as jamielennox | 23:34 | |
| *** gcb has joined #openstack-nova | 23:34 | |
| *** akshai has quit IRC | 23:34 | |
| *** aNupoisc has joined #openstack-nova | 23:34 | |
| *** Swami has quit IRC | 23:34 | |
| aNupoisc | Hi guys does any one knows where does oslo library gets installed with devstack | 23:34 |
| aNupoisc | i looked into lib/ and /opt/stack but not present there | 23:35 |
| aNupoisc | I am new to Openstack | 23:35 |
| *** mriedem has joined #openstack-nova | 23:36 | |
| *** salv-orl_ has quit IRC | 23:36 | |
| *** zenoway has joined #openstack-nova | 23:37 | |
| *** zenoway has quit IRC | 23:41 | |
| diana_clarke | aNupoisc: /usr/lib/python2.7/site-packages/ | 23:41 |
| *** baoli has quit IRC | 23:44 | |
| *** sdake has joined #openstack-nova | 23:44 | |
| aNupoisc | diana_clarke: I dont have oslo inside site-packages as well in dist-packages | 23:44 |
| *** ivase_ has joined #openstack-nova | 23:45 | |
| *** jwcroppe has quit IRC | 23:46 | |
| diana_clarke | aNupoisc: And you executed stack.sh already? | 23:46 |
| *** sdake_ has joined #openstack-nova | 23:46 | |
| *** ivase has quit IRC | 23:48 | |
| *** sdake has quit IRC | 23:49 | |
| *** ZZelle_ has quit IRC | 23:49 | |
| diana_clarke | aNupoisc: what OS? | 23:50 |
| *** Sukhdev has joined #openstack-nova | 23:50 | |
| *** sdake has joined #openstack-nova | 23:50 | |
| aNupoisc | diana_clarke: running devstack using vagrant with ubuntu/trusty64 | 23:51 |
| *** Sukhdev has quit IRC | 23:52 | |
| *** sdake_ has quit IRC | 23:52 | |
| *** hoangcx has quit IRC | 23:54 | |
| *** thorst has joined #openstack-nova | 23:55 | |
| aNupoisc | diana_clarke: yes i executed stack.sh | 23:56 |
| *** ljxiash has joined #openstack-nova | 23:56 | |
| aNupoisc | diana_clarke: I can see install_oslo() in stack.sh getting called which will install all but unable to find the location of files | 23:57 |
| diana_clarke | aNupoisc: what does sys.path say from your python interpreter? | 23:59 |
| *** pushkaru has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!