| *** crushil has joined #openstack-nova | 00:00 | |
| *** crushil has quit IRC | 00:03 | |
| *** crushil has joined #openstack-nova | 00:03 | |
| *** crushil has quit IRC | 00:06 | |
| *** crushil has joined #openstack-nova | 00:07 | |
| *** crushil has quit IRC | 00:10 | |
| *** crushil has joined #openstack-nova | 00:10 | |
| *** crushil has quit IRC | 00:13 | |
| *** dave-mccowan has joined #openstack-nova | 00:13 | |
| *** crushil has joined #openstack-nova | 00:13 | |
| *** wally has joined #openstack-nova | 00:15 | |
| wally | hi all, I have question regarding setup Placement api for Compute Node on Ocata in Ubuntu 1604, can anyone help? | 00:16 |
|---|---|---|
| *** crushil has quit IRC | 00:17 | |
| *** crushil has joined #openstack-nova | 00:17 | |
| wally | i have install the placement api endpoint and configure it, but in my compute node, in the nova-compute.log said The placement API endpoint not found. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading | 00:17 |
| *** baoli has quit IRC | 00:18 | |
| wally | can anyone help please | 00:18 |
| *** catintheroof has joined #openstack-nova | 00:20 | |
| *** crushil has quit IRC | 00:20 | |
| *** crushil has joined #openstack-nova | 00:20 | |
| *** baoli has joined #openstack-nova | 00:22 | |
| *** crushil has quit IRC | 00:23 | |
| *** crushil has joined #openstack-nova | 00:24 | |
| *** catintheroof has quit IRC | 00:26 | |
| *** crushil has quit IRC | 00:27 | |
| *** crushil has joined #openstack-nova | 00:27 | |
| *** claudiub has quit IRC | 00:30 | |
| *** crushil has quit IRC | 00:30 | |
| *** crushil has joined #openstack-nova | 00:31 | |
| *** crushil has quit IRC | 00:34 | |
| *** crushil has joined #openstack-nova | 00:34 | |
| *** crushil has quit IRC | 00:37 | |
| *** crushil has joined #openstack-nova | 00:38 | |
| *** jamielennox is now known as jamielennox|away | 00:38 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Drop deprecated aggregate-update positional args https://review.openstack.org/447332 | 00:40 |
| mriedem | wally: you have to configure the [placement] section of nova.conf on the compute node | 00:40 |
| *** yamamoto has joined #openstack-nova | 00:41 | |
| mriedem | with credentials to access the placement endpoint | 00:41 |
| *** crushil has quit IRC | 00:41 | |
| mriedem | wally: https://docs.openstack.org/developer/nova/placement.html#upgrade-notes | 00:41 |
| *** crushil has joined #openstack-nova | 00:41 | |
| *** crushil has quit IRC | 00:44 | |
| *** crushil has joined #openstack-nova | 00:44 | |
| *** yamamoto has quit IRC | 00:45 | |
| *** crushil has quit IRC | 00:51 | |
| *** gcb has joined #openstack-nova | 00:52 | |
| *** tovin07_ has joined #openstack-nova | 00:55 | |
| *** wally has quit IRC | 00:59 | |
| *** kevinz has joined #openstack-nova | 01:01 | |
| *** phuongnh has joined #openstack-nova | 01:02 | |
| *** baoli has quit IRC | 01:04 | |
| *** gcb has quit IRC | 01:07 | |
| *** gcb has joined #openstack-nova | 01:08 | |
| *** mriedem has quit IRC | 01:08 | |
| *** ducnc has joined #openstack-nova | 01:09 | |
| *** yamamoto has joined #openstack-nova | 01:11 | |
| *** yamamoto has quit IRC | 01:15 | |
| *** yassine has joined #openstack-nova | 01:16 | |
| *** yassine is now known as Guest68764 | 01:16 | |
| *** gongysh has joined #openstack-nova | 01:16 | |
| *** rfolco has joined #openstack-nova | 01:23 | |
| *** rfolco has quit IRC | 01:23 | |
| *** yamamoto has joined #openstack-nova | 01:28 | |
| *** yamamoto has quit IRC | 01:28 | |
| *** baoli has joined #openstack-nova | 01:30 | |
| *** yamamoto has joined #openstack-nova | 01:36 | |
| *** kaisers1 has quit IRC | 01:36 | |
| *** kaisers2 has joined #openstack-nova | 01:36 | |
| *** rfolco has joined #openstack-nova | 01:39 | |
| *** rfolco has quit IRC | 01:39 | |
| *** yamamoto has quit IRC | 01:41 | |
| *** markvoelker has joined #openstack-nova | 01:45 | |
| *** markvoelker has quit IRC | 01:49 | |
| *** wangqun has joined #openstack-nova | 01:51 | |
| *** rfolco has joined #openstack-nova | 01:54 | |
| *** rfolco has quit IRC | 01:54 | |
| *** jamesdenton has joined #openstack-nova | 01:58 | |
| *** jamesdenton has quit IRC | 01:58 | |
| *** rfolco has joined #openstack-nova | 02:00 | |
| *** rfolco has quit IRC | 02:00 | |
| *** kaisers_ has joined #openstack-nova | 02:02 | |
| *** kaisers has quit IRC | 02:06 | |
| *** yamamoto has joined #openstack-nova | 02:19 | |
| *** yamamoto has quit IRC | 02:19 | |
| *** tanee_away is now known as tanee | 02:25 | |
| *** fragatina has joined #openstack-nova | 02:26 | |
| *** yamamoto has joined #openstack-nova | 02:31 | |
| *** fragatina has quit IRC | 02:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 02:37 |
| *** dimtruck is now known as zz_dimtruck | 02:52 | |
| *** jichen has joined #openstack-nova | 03:11 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 03:14 |
| *** baoli has quit IRC | 03:14 | |
| *** dave-mccowan has quit IRC | 03:20 | |
| *** jamielennox|away is now known as jamielennox | 03:22 | |
| *** zhurong has joined #openstack-nova | 03:25 | |
| *** sree has joined #openstack-nova | 03:33 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Reset instance status on invalid host https://review.openstack.org/401009 | 03:43 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Return 400 when compute host is not found https://review.openstack.org/447355 | 03:43 |
| *** yamamoto has quit IRC | 03:50 | |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 03:56 |
| openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: API changes for live migration of rescued instance https://review.openstack.org/328280 | 03:56 |
| *** yamamoto has joined #openstack-nova | 03:57 | |
| *** yamamoto has quit IRC | 03:59 | |
| *** yamamoto has joined #openstack-nova | 04:00 | |
| *** yamamoto has quit IRC | 04:00 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:08 | |
| *** psachin has joined #openstack-nova | 04:09 | |
| *** Matias has quit IRC | 04:15 | |
| *** udesale has joined #openstack-nova | 04:16 | |
| *** zhurong has quit IRC | 04:16 | |
| *** gongysh has quit IRC | 04:18 | |
| *** links has joined #openstack-nova | 04:20 | |
| *** ratailor has joined #openstack-nova | 04:30 | |
| *** Matias has joined #openstack-nova | 04:31 | |
| *** kaisers_ has quit IRC | 04:41 | |
| *** diga has joined #openstack-nova | 04:42 | |
| *** unicell1 has joined #openstack-nova | 04:53 | |
| *** unicell has quit IRC | 04:54 | |
| *** yamamoto has joined #openstack-nova | 04:55 | |
| *** diga has quit IRC | 04:58 | |
| *** zhurong has joined #openstack-nova | 04:59 | |
| *** yamamoto has quit IRC | 05:00 | |
| *** gongysh has joined #openstack-nova | 05:04 | |
| *** kornicameister has quit IRC | 05:06 | |
| *** ayogi has joined #openstack-nova | 05:10 | |
| *** kornicameister has joined #openstack-nova | 05:13 | |
| *** zhurong has quit IRC | 05:14 | |
| *** vks1 has joined #openstack-nova | 05:16 | |
| *** adisky_ has joined #openstack-nova | 05:23 | |
| *** ygl has joined #openstack-nova | 05:27 | |
| *** nmathew has joined #openstack-nova | 05:32 | |
| *** ijw has quit IRC | 05:39 | |
| openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | 05:39 |
| *** vks1 has quit IRC | 05:39 | |
| *** ijw has joined #openstack-nova | 05:40 | |
| *** kornicameister has quit IRC | 05:41 | |
| *** kaisers has joined #openstack-nova | 05:41 | |
| *** kornicameister has joined #openstack-nova | 05:43 | |
| *** claudiub has joined #openstack-nova | 05:46 | |
| *** kaisers has quit IRC | 05:47 | |
| *** sridharg has joined #openstack-nova | 05:48 | |
| *** Jack_Iv has joined #openstack-nova | 05:50 | |
| *** sridharg has quit IRC | 05:54 | |
| *** vks1 has joined #openstack-nova | 05:55 | |
| *** gongysh has quit IRC | 05:58 | |
| *** jaosorior has joined #openstack-nova | 06:04 | |
| *** prateek has joined #openstack-nova | 06:05 | |
| *** Sukhdev has joined #openstack-nova | 06:07 | |
| *** gongysh has joined #openstack-nova | 06:10 | |
| *** kaisers has joined #openstack-nova | 06:11 | |
| *** yamamoto has joined #openstack-nova | 06:11 | |
| *** kaisers has quit IRC | 06:15 | |
| *** yamamoto has quit IRC | 06:18 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | 06:24 |
| *** yamahata has joined #openstack-nova | 06:29 | |
| *** moshele has joined #openstack-nova | 06:30 | |
| *** moshele has quit IRC | 06:36 | |
| *** sridharg has joined #openstack-nova | 06:37 | |
| openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | 06:39 |
| *** kornicameister has quit IRC | 06:41 | |
| *** kornicameister has joined #openstack-nova | 06:43 | |
| *** NikhilS has joined #openstack-nova | 06:44 | |
| *** lpetrut has joined #openstack-nova | 06:46 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Don't create instance backup image if rotation is 0 https://review.openstack.org/409644 | 06:47 |
| *** Sukhdev has quit IRC | 06:48 | |
| *** jaosorior has quit IRC | 06:53 | |
| *** sree has quit IRC | 06:54 | |
| *** jaosorior has joined #openstack-nova | 06:55 | |
| *** mnestratov has joined #openstack-nova | 06:57 | |
| *** karthiks_afk has joined #openstack-nova | 06:57 | |
| *** sree has joined #openstack-nova | 07:00 | |
| *** nkrinner_afk is now known as nkrinner | 07:03 | |
| *** sree_ has joined #openstack-nova | 07:04 | |
| *** sree_ is now known as Guest84360 | 07:05 | |
| *** sree has quit IRC | 07:05 | |
| *** Oku_OS-away is now known as Oku_OS | 07:08 | |
| *** Guest84360 has quit IRC | 07:08 | |
| *** kaisers has joined #openstack-nova | 07:11 | |
| *** jaosorior has quit IRC | 07:14 | |
| *** jaosorior has joined #openstack-nova | 07:15 | |
| *** jaosorior has quit IRC | 07:16 | |
| *** kaisers has quit IRC | 07:17 | |
| *** slaweq has joined #openstack-nova | 07:17 | |
| *** jaosorior has joined #openstack-nova | 07:19 | |
| *** zerda3 has joined #openstack-nova | 07:20 | |
| *** carthaca_ has quit IRC | 07:20 | |
| *** sapcc-bot has quit IRC | 07:20 | |
| *** satyar has joined #openstack-nova | 07:20 | |
| *** andreas_s has joined #openstack-nova | 07:24 | |
| *** moshele has joined #openstack-nova | 07:29 | |
| *** Jeffrey4l__ has quit IRC | 07:33 | |
| *** ygl has quit IRC | 07:33 | |
| *** Jack_Iv has quit IRC | 07:36 | |
| *** Jeffrey4l__ has joined #openstack-nova | 07:36 | |
| *** Jack_Iv has joined #openstack-nova | 07:36 | |
| *** ircuser-1 has joined #openstack-nova | 07:37 | |
| *** Jack_Iv has quit IRC | 07:37 | |
| *** Jack_Iv has joined #openstack-nova | 07:37 | |
| *** yonglihe has quit IRC | 07:39 | |
| *** voelzmo has joined #openstack-nova | 07:41 | |
| *** karthiks_afk is now known as karthiks | 07:43 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova master: Use HostAddressOpt for opts that accept IP and hostnames https://review.openstack.org/445782 | 07:43 |
| *** moshele has quit IRC | 07:45 | |
| *** markus_z has joined #openstack-nova | 07:49 | |
| *** youzp has joined #openstack-nova | 07:50 | |
| *** ralonsoh has joined #openstack-nova | 07:51 | |
| *** lpetrut has quit IRC | 07:52 | |
| *** youzp has quit IRC | 07:55 | |
| *** claudiub|2 has joined #openstack-nova | 07:57 | |
| *** tesseract has joined #openstack-nova | 07:58 | |
| *** youzp has joined #openstack-nova | 07:59 | |
| *** claudiub has quit IRC | 08:00 | |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add os-traits reference https://review.openstack.org/447401 | 08:00 |
| *** zzzeek has joined #openstack-nova | 08:00 | |
| *** brault has joined #openstack-nova | 08:01 | |
| *** lpetrut has joined #openstack-nova | 08:04 | |
| *** kaisers2 has quit IRC | 08:04 | |
| *** armax has quit IRC | 08:05 | |
| *** ociuhandu has joined #openstack-nova | 08:06 | |
| *** ratailor is now known as ratailor|Lunch | 08:08 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add os-traits reference https://review.openstack.org/447401 | 08:12 |
| *** pcaruana has joined #openstack-nova | 08:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add os-traits reference https://review.openstack.org/447401 | 08:15 |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 08:17 |
| *** lpetrut has quit IRC | 08:17 | |
| *** Jack_Iv has quit IRC | 08:17 | |
| *** kaisers has joined #openstack-nova | 08:18 | |
| *** Jack_Iv has joined #openstack-nova | 08:18 | |
| *** youzp has quit IRC | 08:21 | |
| *** yamamoto has joined #openstack-nova | 08:22 | |
| *** rdo has quit IRC | 08:24 | |
| *** youzp has joined #openstack-nova | 08:25 | |
| *** rdo has joined #openstack-nova | 08:26 | |
| *** fengzhr has joined #openstack-nova | 08:27 | |
| *** youzp has quit IRC | 08:32 | |
| *** kaisers has quit IRC | 08:34 | |
| *** fengzhr has quit IRC | 08:38 | |
| *** fengzhr has joined #openstack-nova | 08:38 | |
| bauzas | good morning Nova | 08:41 |
| openstackgerrit | Shunli Zhou proposed openstack/nova master: Remove mox from cells/test_cells_rpc_driver.py https://review.openstack.org/447412 | 08:43 |
| *** jahsis has joined #openstack-nova | 08:44 | |
| *** mlakat has joined #openstack-nova | 08:44 | |
| *** kaisers has joined #openstack-nova | 08:49 | |
| *** rmart04 has joined #openstack-nova | 08:54 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Add os-traits reference https://review.openstack.org/447401 | 08:54 |
| *** ayogi has quit IRC | 09:01 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** yamamoto has quit IRC | 09:04 | |
| *** openstackgerrit has joined #openstack-nova | 09:07 | |
| openstackgerrit | Michael Henkel proposed openstack/nova master: contrail: add vrouter VIF plugin type support https://review.openstack.org/441183 | 09:07 |
| *** kaisers_ has joined #openstack-nova | 09:14 | |
| *** kaisers_ has quit IRC | 09:19 | |
| *** claudiub|2 is now known as claudiub | 09:20 | |
| *** sridharg has quit IRC | 09:21 | |
| *** ayogi has joined #openstack-nova | 09:24 | |
| *** ayogi has quit IRC | 09:24 | |
| *** sridharg has joined #openstack-nova | 09:24 | |
| *** lpetrut has joined #openstack-nova | 09:25 | |
| *** ayogi has joined #openstack-nova | 09:26 | |
| *** lucas-afk is now known as lucasagomes | 09:28 | |
| lyarwood | mdbooth: can you take another look at https://review.openstack.org/#/c/439520/ this morning? | 09:29 |
| *** owalsh_ is now known as owalsh | 09:33 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 09:38 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 09:39 |
| *** arne_r has joined #openstack-nova | 09:39 | |
| *** derekh has joined #openstack-nova | 09:45 | |
| *** yamahata has quit IRC | 09:48 | |
| *** abalutoiu has joined #openstack-nova | 10:01 | |
| *** yamamoto has joined #openstack-nova | 10:03 | |
| *** kornica has joined #openstack-nova | 10:03 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Re-add templates https://review.openstack.org/446069 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Fix formatting errors and warnings https://review.openstack.org/407364 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: tox: Build docs with Python 2.7 https://review.openstack.org/445906 | 10:05 |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Treat doc build warnings as errors https://review.openstack.org/439883 | 10:05 |
| *** wangqun has quit IRC | 10:06 | |
| *** yamamoto has quit IRC | 10:07 | |
| *** Jack_Iv has quit IRC | 10:09 | |
| *** kornicameister has quit IRC | 10:09 | |
| *** nicolasbock has joined #openstack-nova | 10:12 | |
| *** Jack_Iv has joined #openstack-nova | 10:12 | |
| *** vks1 has quit IRC | 10:14 | |
| *** kaisers_ has joined #openstack-nova | 10:15 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 10:15 |
| *** tovin07_ has quit IRC | 10:16 | |
| *** sree has joined #openstack-nova | 10:17 | |
| *** kaisers_ has quit IRC | 10:19 | |
| *** yamamoto has joined #openstack-nova | 10:21 | |
| *** yamamoto has quit IRC | 10:21 | |
| *** sree has quit IRC | 10:21 | |
| *** gcb has quit IRC | 10:23 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | 10:24 |
| *** ijw has quit IRC | 10:25 | |
| *** ijw has joined #openstack-nova | 10:27 | |
| *** kevinz has quit IRC | 10:28 | |
| *** sridharg has quit IRC | 10:28 | |
| *** phuongnh has quit IRC | 10:30 | |
| *** satyar has quit IRC | 10:30 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add policy-remove-scope-checks spec https://review.openstack.org/433037 | 10:33 |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872 | 10:33 |
| *** vks1 has joined #openstack-nova | 10:34 | |
| *** dikonoor has joined #openstack-nova | 10:37 | |
| *** sridharg has joined #openstack-nova | 10:37 | |
| *** ratailor|Lunch has quit IRC | 10:42 | |
| *** dikonoor has quit IRC | 10:51 | |
| *** udesale__ has joined #openstack-nova | 10:56 | |
| *** udesale has quit IRC | 10:57 | |
| johnthetubaguy | lyarwood: I am not sure about stashing things for disconnect_volume | 10:58 |
| johnthetubaguy | lyarwood: it seems like we could easily screw things up if we do that, if thats lots of load on Cinder, we can optimise that later. | 10:59 |
| *** udesale has joined #openstack-nova | 10:59 | |
| johnthetubaguy | lyarwood: what do you think? | 10:59 |
| openstackgerrit | Roman Podoliaka proposed openstack/nova master: nova-status: don't coerce version numbers to floats for comparison https://review.openstack.org/447449 | 10:59 |
| lyarwood | johnthetubaguy: well we currently do that right, so not stashing it would be something new and untested | 11:00 |
| *** mnestratov has quit IRC | 11:00 | |
| johnthetubaguy | lyarwood: its not currently something cinder doesn't have though | 11:00 |
| lyarwood | johnthetubaguy: I'm just worried about the backend c-vol config changing and getting back connection_info for the disconnect that results on os-bick failing | 11:00 |
| johnthetubaguy | oops, I mean its something cinder couldn't give us right now | 11:00 |
| johnthetubaguy | lyarwood: honestly, I am worrying about the reverse | 11:00 |
| *** udesale__ has quit IRC | 11:00 | |
| johnthetubaguy | lyarwood: stuff gets updated, and we don't find out, and disconnect fails | 11:01 |
| lyarwood | johnthetubaguy: yeah that's something that could happen today and tbh I can't think of an example | 11:01 |
| lyarwood | johnthetubaguy: where we've seen it in the wild that is | 11:01 |
| johnthetubaguy | lyarwood: it just feels like we should keep the API clean here, and we loop back and fix up things after that | 11:02 |
| lyarwood | johnthetubaguy: yeah indeed, we are not dropping connection_info from the db / object models right? | 11:03 |
| lyarwood | johnthetubaguy: so we can always start stashing this again I guess | 11:03 |
| *** Jack_Iv has quit IRC | 11:03 | |
| *** cdent has joined #openstack-nova | 11:03 | |
| johnthetubaguy | lyarwood: well, we can't drop that will all existing volumes are transitioned, which is a big TODO right now | 11:03 |
| *** Jack_Iv has joined #openstack-nova | 11:04 | |
| *** Jack_Iv has quit IRC | 11:04 | |
| johnthetubaguy | lyarwood: if we start saving that, I just feel we need an open conversation with os-brick and cinder folks on how it should fit in the flow, maybe immutable stuff on their side per attachment is a better way forward | 11:04 |
| *** Jack_Iv has joined #openstack-nova | 11:04 | |
| *** psachin has quit IRC | 11:05 | |
| *** nmathew has quit IRC | 11:05 | |
| lyarwood | johnthetubaguy: yeah that's a fair point, they can and maybe should look after this for us | 11:05 |
| johnthetubaguy | lyarwood: this probably speaks to that improved swap volume work actually, this is a zero copy swap volume use case | 11:05 |
| johnthetubaguy | (ish) | 11:06 |
| lyarwood | johnthetubaguy: unless the instance is down I can't think of a situation where nova would leave swap volume up to cinder entirely, IMHO there would always be a need for nova to rebase and ensure everything is in sync. | 11:09 |
| *** aarefiev_afk is now known as aarefiev | 11:15 | |
| openstackgerrit | Pooja Jadhav proposed openstack/nova master: Deny volume swapping with non-bootable volume https://review.openstack.org/407346 | 11:15 |
| *** kaisers_ has joined #openstack-nova | 11:16 | |
| *** smatzek has joined #openstack-nova | 11:17 | |
| *** zhurong has joined #openstack-nova | 11:17 | |
| *** udesale has quit IRC | 11:19 | |
| *** udesale has joined #openstack-nova | 11:19 | |
| *** kaisers_ has quit IRC | 11:20 | |
| *** yamamoto has joined #openstack-nova | 11:22 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add spec to use cinder's new attachment API https://review.openstack.org/373203 | 11:22 |
| johnthetubaguy | lyarwood: I was thinking about cinder wanting to rotate a connection password, +1 it always has to involve Nova | 11:24 |
| *** dikonoor has joined #openstack-nova | 11:24 | |
| *** Jack_Iv has quit IRC | 11:27 | |
| *** yamamoto has quit IRC | 11:30 | |
| *** hferenc has quit IRC | 11:30 | |
| *** Jeffrey4l_ has joined #openstack-nova | 11:34 | |
| *** Jack_Iv has joined #openstack-nova | 11:35 | |
| mhenkel | sean-k-mooney: would you have some time helping me to figure out why this failed: http://logs.openstack.org/83/441183/10/check/gate-tempest-dsvm-py35-ubuntu-xenial/e42be9f/ ? | 11:35 |
| mhenkel | I walked through all logs but couldn't spot any error related to the commit | 11:35 |
| *** Jeffrey4l__ has quit IRC | 11:36 | |
| *** zhurong has quit IRC | 11:37 | |
| openstackgerrit | Roman Podoliaka proposed openstack/nova master: neutron: re-raise 401 for a non-admin client https://review.openstack.org/422696 | 11:40 |
| *** yamamoto has joined #openstack-nova | 11:41 | |
| *** psachin has joined #openstack-nova | 11:42 | |
| *** ociuhandu has quit IRC | 11:45 | |
| *** udesale has quit IRC | 11:47 | |
| *** tbachman has quit IRC | 11:48 | |
| *** dikonoor has quit IRC | 11:48 | |
| *** kylek3h has quit IRC | 11:55 | |
| *** dikonoor has joined #openstack-nova | 11:58 | |
| *** tbachman has joined #openstack-nova | 11:59 | |
| *** aysyd has joined #openstack-nova | 12:00 | |
| *** kevinz has joined #openstack-nova | 12:03 | |
| *** gongysh has quit IRC | 12:06 | |
| *** vks1 has quit IRC | 12:07 | |
| *** kevinz has quit IRC | 12:08 | |
| *** dave-mccowan has joined #openstack-nova | 12:08 | |
| *** kevinz has joined #openstack-nova | 12:08 | |
| *** diana_clarke has joined #openstack-nova | 12:10 | |
| *** diana_clarke has left #openstack-nova | 12:10 | |
| *** rfolco has joined #openstack-nova | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:14 | |
| *** cdent has quit IRC | 12:16 | |
| *** kaisers_ has joined #openstack-nova | 12:16 | |
| *** Chason has joined #openstack-nova | 12:18 | |
| *** sridharg has quit IRC | 12:20 | |
| *** sridharg has joined #openstack-nova | 12:21 | |
| Jeffrey4l_ | is there any consumer for nova versioned notifications? https://docs.openstack.org/developer/nova/notifications.html#versioned-notifications | 12:21 |
| *** kaisers_ has quit IRC | 12:21 | |
| *** mnestratov has joined #openstack-nova | 12:21 | |
| Jeffrey4l_ | seems ceilometer can not handle this. should i disable this when deploying? | 12:21 |
| openstackgerrit | Sean Dague proposed openstack/nova master: remove log translation from nova.api.metadata https://review.openstack.org/447466 | 12:26 |
| *** zul has joined #openstack-nova | 12:29 | |
| *** ociuhandu has joined #openstack-nova | 12:31 | |
| *** ralonsoh_ has joined #openstack-nova | 12:34 | |
| *** kaisers has quit IRC | 12:36 | |
| *** kylek3h has joined #openstack-nova | 12:37 | |
| *** liangy has joined #openstack-nova | 12:42 | |
| *** gcb has joined #openstack-nova | 12:49 | |
| *** kaisers has joined #openstack-nova | 12:49 | |
| *** mdrabe has joined #openstack-nova | 12:50 | |
| *** cleong has joined #openstack-nova | 12:51 | |
| Chason | Hi guys, since openstack server create does not support the options --swap and --ephemeral, are there some other options alternatives? | 12:53 |
| *** fengzhr has quit IRC | 12:53 | |
| *** satyar has joined #openstack-nova | 12:54 | |
| *** fengzhr has joined #openstack-nova | 12:54 | |
| *** yamamoto has quit IRC | 12:55 | |
| *** mriedem has joined #openstack-nova | 12:56 | |
| *** catintheroof has joined #openstack-nova | 12:57 | |
| *** gongysh has joined #openstack-nova | 12:58 | |
| *** awaugama has joined #openstack-nova | 13:00 | |
| *** markvoelker has joined #openstack-nova | 13:01 | |
| *** liverpooler has joined #openstack-nova | 13:02 | |
| *** cdent has joined #openstack-nova | 13:03 | |
| *** vladikr has joined #openstack-nova | 13:03 | |
| *** Jack_Iv has quit IRC | 13:07 | |
| *** gouthamr has joined #openstack-nova | 13:07 | |
| *** jaypipes has joined #openstack-nova | 13:08 | |
| *** ayogi has quit IRC | 13:11 | |
| *** links has quit IRC | 13:11 | |
| *** kaisers has quit IRC | 13:12 | |
| *** kaisers has joined #openstack-nova | 13:17 | |
| *** tblakes has joined #openstack-nova | 13:18 | |
| *** tblakes has quit IRC | 13:19 | |
| *** tblakes has joined #openstack-nova | 13:19 | |
| *** gabor_antal_km is now known as gabor_antal | 13:20 | |
| *** cdent has quit IRC | 13:20 | |
| *** Jack_Iv has joined #openstack-nova | 13:20 | |
| *** youzp has joined #openstack-nova | 13:21 | |
| *** kaisers has quit IRC | 13:22 | |
| *** gcb has quit IRC | 13:22 | |
| *** gcb has joined #openstack-nova | 13:23 | |
| *** Sukhdev has joined #openstack-nova | 13:25 | |
| *** youzp has quit IRC | 13:26 | |
| efried | Chason You specify those things via the flavor | 13:27 |
| *** youzp has joined #openstack-nova | 13:29 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/446674 | 13:29 |
| efried | Chason Take a look at `openstack flavor create` | 13:29 |
| *** kaisers has joined #openstack-nova | 13:30 | |
| Chason | efried, you mean I should create flavor with these options, then to create the instance? | 13:31 |
| efried | Chason Correct. | 13:31 |
| Chason | OK, thanks! | 13:31 |
| efried | The --flavor argument to `openstack server create` is mandatory anyway, I believe. | 13:31 |
| *** jamesdenton has joined #openstack-nova | 13:31 | |
| efried | YW, good luck. | 13:31 |
| johnthetubaguy | Chason efried is correct its normally set by the flavor, but do take a look at block_device_mapping_v2 in here: https://developer.openstack.org/api-ref/compute/?expanded=create-server-detail#create-server | 13:31 |
| Chason | Yeah, I see. | 13:31 |
| *** smatzek has quit IRC | 13:32 | |
| *** erhudy has joined #openstack-nova | 13:34 | |
| *** cdent has joined #openstack-nova | 13:35 | |
| *** cdent has quit IRC | 13:35 | |
| openstackgerrit | Pushkar Umaranikar proposed openstack/nova master: Placement api: add tox check for webob wsgify decorator https://review.openstack.org/436671 | 13:35 |
| openstackgerrit | Maciej Szankin proposed openstack/nova master: Stop using mox stubs in nova/tests/unit/cells/fakes.py https://review.openstack.org/342043 | 13:37 |
| *** baoli has joined #openstack-nova | 13:37 | |
| *** Sukhdev has quit IRC | 13:39 | |
| *** burt has joined #openstack-nova | 13:40 | |
| *** Sukhdev has joined #openstack-nova | 13:40 | |
| *** ralonsoh_ has quit IRC | 13:40 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - add some functional tests for placement https://review.openstack.org/446123 | 13:42 |
| *** eharney has joined #openstack-nova | 13:42 | |
| *** clenimar has quit IRC | 13:43 | |
| *** youzp has quit IRC | 13:43 | |
| edleafe- | Scheduler subteam meeting in 15 minutes in #openstack-meeting-alt | 13:45 |
| *** edleafe- is now known as edleafe | 13:45 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505 | 13:45 |
| *** hongbin has joined #openstack-nova | 13:47 | |
| EmilienM | jaypipes: hey, good morning. We're currently investigating a TripleO CI issue where fail to deploy the overcloud (nodes managed by ironic) since March 16th, so we're looking at commits in Nova and we found https://review.openstack.org/#/c/441544/ that might be related. If you want to see bug report: https://bugs.launchpad.net/tripleo/+bug/1674236 - thanks for looking | 13:47 |
| openstack | Launchpad bug 1674236 in tripleo "CI / promotion: Nova isn't aware of the nodes that were registered with Ironic" [Critical,Triaged] | 13:47 |
| *** READ10 has joined #openstack-nova | 13:48 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505 | 13:50 |
| jaypipes | EmilienM: looking... | 13:52 |
| EmilienM | some logs: http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-conductor.txt.gz#_2017-03-19_07_21_12_709 | 13:52 |
| *** crushil has joined #openstack-nova | 13:52 | |
| EmilienM | and http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-scheduler.txt.gz#_2017-03-19_07_19_34_158 | 13:53 |
| sdague | jaypipes: I'm confused by your reviews on https://review.openstack.org/#/c/427901/ - as it's got a -1 with a change requested, then a +2 | 13:53 |
| *** armax has joined #openstack-nova | 13:53 | |
| *** lucasagomes is now known as lucas-hungry | 13:53 | |
| mriedem | sdague: because they talked about it in irc and didn't update the comments in the review | 13:54 |
| jaypipes | sdague: yes, what mriedem said. | 13:54 |
| *** tommy_ has joined #openstack-nova | 13:54 | |
| *** sapcc-bot has joined #openstack-nova | 13:54 | |
| *** tpatzig_ has joined #openstack-nova | 13:54 | |
| *** databus23_ has joined #openstack-nova | 13:54 | |
| *** carthaca_ has joined #openstack-nova | 13:54 | |
| *** david_1 has joined #openstack-nova | 13:54 | |
| *** tommy_ is now known as Guest3670 | 13:55 | |
| mriedem | EmilienM: the compute logs are probably what is most interesting since that's where the resource tracker logs are | 13:55 |
| mriedem | which feeds the scheduler | 13:55 |
| *** yamamoto has joined #openstack-nova | 13:55 | |
| dansmith | compute is feeding placement correctly | 13:55 |
| *** mlavalle has joined #openstack-nova | 13:55 | |
| dansmith | scheduler just never finds any compute nodes, AFAICT | 13:55 |
| jroll | O_o | 13:56 |
| *** vks1 has joined #openstack-nova | 13:56 | |
| *** Guest3670 has quit IRC | 13:56 | |
| *** david_1 has quit IRC | 13:56 | |
| *** databus23_ has quit IRC | 13:56 | |
| *** tpatzig_ has quit IRC | 13:56 | |
| *** carthaca_ has quit IRC | 13:56 | |
| *** tblakes has quit IRC | 13:57 | |
| *** gcb has quit IRC | 13:57 | |
| jroll | oh, we don't take the old path if get_inventory() is implemented: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L754 | 13:58 |
| *** gcb has joined #openstack-nova | 13:58 | |
| mriedem | correct | 13:58 |
| jroll | I bet we aren't registering the "capabilities" | 13:58 |
| mriedem | i see this over and over in the compute logs | 13:58 |
| mriedem | 2017-03-19 07:01:23.652 16870 ERROR nova.compute.manager [req-cf2991f8-9fe2-45db-95e2-8cfc8cee2edd - - - - -] No compute node record for host tripleo-centos-7-tripleo-test-cloud-rh1-7948745 | 13:58 |
| dansmith | ah | 13:58 |
| jroll | it wouldn't explain starting with 0 nodes, though | 13:59 |
| dansmith | jroll: if we never create a computenode record though... | 13:59 |
| dansmith | shouldn't this have broken our ironic job too? | 13:59 |
| jroll | dansmith: right, I thought we did that either way | 13:59 |
| mriedem | i thought the compute node record was created before this | 13:59 |
| *** smatzek has joined #openstack-nova | 14:00 | |
| dansmith | ah yeah, you're right | 14:00 |
| dansmith | L745 | 14:00 |
| jroll | yeah | 14:00 |
| *** esberglu has joined #openstack-nova | 14:00 | |
| dansmith | we've already got a CN there | 14:00 |
| mriedem | yeah create it here if it doesn't exist https://github.com/openstack/nova/blob/66c3f0860b6d96500539b5e94856f9594d363da7/nova/compute/resource_tracker.py#L593 | 14:00 |
| jaypipes | mriedem: it looks like it's taking approx 30 minutes for the Ironic nodes to register with Nova.. | 14:00 |
| *** gcb_ has joined #openstack-nova | 14:01 | |
| mriedem | update it here using get_inventory https://github.com/openstack/nova/blob/66c3f0860b6d96500539b5e94856f9594d363da7/nova/compute/resource_tracker.py#L642 | 14:01 |
| *** irenab_ has joined #openstack-nova | 14:01 | |
| EmilienM | CN = compute node? | 14:02 |
| mriedem | yes | 14:02 |
| *** gcb has quit IRC | 14:02 | |
| edleafe | Scheduler meeting now running in #openstack-meeting-alt | 14:02 |
| EmilienM | iiuc, you have one job that test Ironic: gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial-nv - am I right? | 14:03 |
| jroll | +1 to what jaypipes said, I don't see the resource tracker even try to run until 07:10, that's odd | 14:03 |
| jroll | EmilienM: yes, and then the entire ironic gate. we'd know by now if it broke ironic :) | 14:03 |
| *** yamamoto has quit IRC | 14:04 | |
| EmilienM | right, and I've looked at commits in Ironic since March 16th and I haven't found anything suspicious | 14:04 |
| mriedem | one of the compute node records is created here http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-03-19_07_10_44_256 | 14:04 |
| jroll | EmilienM: I must ask, this is repeatable, right? | 14:04 |
| EmilienM | jroll: yes, we have it for 3 days | 14:04 |
| EmilienM | but nobody looked at it this weekend | 14:05 |
| jroll | cool, the timing thing seemed weird | 14:05 |
| EmilienM | (I was offline) | 14:05 |
| jroll | I guess you're trying to boot an instance after it's created anyway | 14:05 |
| jaypipes | mriedem: yes, and looking at the placement log, I see 3 PUTs for creating inventory successfully. | 14:05 |
| jroll | EmilienM: what does your flavor look like? | 14:05 |
| EmilienM | I can try a revert and Depends-On tripleo patch | 14:05 |
| EmilienM | so I can tell if this patch broke us or not | 14:05 |
| EmilienM | jroll: let me show you that | 14:06 |
| EmilienM | jroll: gimme 2 min | 14:06 |
| jroll | that would be good data, yeah | 14:06 |
| jroll | EmilienM: thanks | 14:06 |
| openstackgerrit | Emilien Macchi proposed openstack/nova master: DNM - Revert "virt: implement get_inventory() for Ironic" https://review.openstack.org/447512 | 14:07 |
| mriedem | Using cache for node eca19329-b67f-46b5-8509-586491a15f17, age: 7.65818786621 _node_from_cache /usr/lib/python2.7/site-packages/nova/virt/ironic/driver.py:700 | 14:07 |
| mriedem | is there a timer on the node cache in the ironic driver? | 14:08 |
| jroll | it refreshes every resource tracker run | 14:08 |
| *** felipemonteiro_ has joined #openstack-nova | 14:08 | |
| mriedem | jroll: are you sure about that? | 14:09 |
| mriedem | i don't see _node_from_cache refreshing | 14:09 |
| mriedem | and get_available_resource only does if it doesn't exist | 14:09 |
| jroll | mriedem: get_available_nodes | 14:10 |
| * jroll double checks | 14:10 | |
| jroll | ya https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L604 | 14:10 |
| EmilienM | jroll: I think the default flavor is 4096, 1, 40 | 14:11 |
| mriedem | and get_available_nodes is called for every update_available_resource in the compute manager, which is on startup and every 60s | 14:11 |
| jaypipes | mriedem: get_available_resource() is always called before get_inventory(), and get_available_resource() always refreshes the node cache: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L688 | 14:11 |
| EmilienM | jroll: details here: http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-03-19_06_38_20_764 | 14:12 |
| mriedem | jaypipes: get_available_resource does not always refresh the cache | 14:12 |
| openstackgerrit | Rafael Folco proposed openstack/nova master: DELETE all inventory for a resource provider https://review.openstack.org/416669 | 14:12 |
| mriedem | see the conditional block it's in | 14:12 |
| mriedem | if not self.node_cache: | 14:12 |
| mriedem | self._refresh_cache() | 14:12 |
| *** udesale has joined #openstack-nova | 14:12 | |
| jroll | EmilienM: okay, and we have Final resource view: name=eca19329-b67f-46b5-8509-586491a15f17 phys_ram=8192MB used_ram=0MB phys_disk=40GB used_disk=0GB total_vcpus=4 used_vcpus=0 pci_stats=[] | 14:12 |
| jaypipes | mriedem: well, I mean it will always trigger a full pull of the node list if the node cache is empty. | 14:12 |
| mriedem | anyway, looks like jroll is right and we refresh in get_available_nodes, which is called in each periodic in the manager | 14:13 |
| * jroll wonders if placement looks for exact matches here | 14:13 | |
| dansmith | jroll: no, but what has changed? | 14:13 |
| jaypipes | jroll: placement will return nodes that have enough RAM/VCPU/DISK | 14:14 |
| jroll | dansmith: we now have ironic inventory in the placement engine | 14:14 |
| jroll | right, that's what I thought, so this flavor should fit | 14:14 |
| jaypipes | jroll: scheduler isn't yet requesting custom resource class | 14:14 |
| dansmith | jroll: sure but we're not asking for it | 14:14 |
| jroll | the scheduler isn't asking placement for resource providers that match the flavor's cpu/ram/disk? | 14:15 |
| jroll | oh wait, that was already there | 14:15 |
| jroll | blah | 14:15 |
| dansmith | right | 14:15 |
| *** tblakes has joined #openstack-nova | 14:15 | |
| mriedem | Hypervisor: VCPU information unavailable _report_hypervisor_resource_view /usr/lib/python2.7/site-packages/nova/compute/resource_tracker.py:676 | 14:15 |
| mriedem | indicates the node is not available, right ^ ? | 14:15 |
| jroll | mriedem: yeah, but later it figures things out: http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-03-19_07_14_43_029 | 14:16 |
| EmilienM | (in the meantime, we're testing the revert here: https://review.openstack.org/#/c/447514) | 14:16 |
| jroll | log level filters here would be super useful, btw :P | 14:17 |
| dansmith | jroll: yeah :( | 14:17 |
| EmilienM | jroll: it's in our (long) todo | 14:17 |
| dansmith | jaypipes: a scheduling decision looks like /resource_providers?stuff=stuff right? | 14:17 |
| EmilienM | but yeah, I agree | 14:17 |
| dansmith | jaypipes: in placement | 14:17 |
| jaypipes | dansmith: yep. | 14:17 |
| dansmith | jaypipes: I don't see a single one of those in placement's log | 14:18 |
| jroll | I do | 14:18 |
| jroll | GET /placement/resource_providers?resources=DISK_GB%3A40%2CMEMORY_MB%3A4096%2CVCPU%3A1 | 14:18 |
| dansmith | I think we should be asking placement, then using that list of uuids to filter the compute_nodes list right? | 14:18 |
| *** kaisers_ has joined #openstack-nova | 14:18 | |
| dansmith | jroll: ah, yeah, okay | 14:18 |
| jaypipes | EmilienM: jroll is referring to better logging in the scheduler/placement service. | 14:18 |
| dansmith | must have fat-fingered my search | 14:18 |
| jroll | jaypipes: no, I'm referring to being able to filter compute logs by log level in tripleo logs :) | 14:19 |
| mriedem | jaypipes: i think it's per-job | 14:19 |
| mriedem | right | 14:19 |
| mriedem | we have filtering on the dsvm jbos | 14:19 |
| EmilienM | jaypipes: ah ok. I was talking about the capacity of filtering logs like devstack does | 14:19 |
| mriedem | *jobs | 14:19 |
| jaypipes | dansmith: right, but placement is returning 0 providers. | 14:19 |
| dansmith | jaypipes: is it? | 14:19 |
| jroll | looks like it to me | 14:19 |
| jroll | "Starting with 0 host(s) get_filtered_objects" | 14:19 |
| dansmith | okay that's different than no compute node records | 14:19 |
| dansmith | jroll: no, that's too late to know | 14:20 |
| jroll | is it? | 14:20 |
| jroll | lol | 14:20 |
| jaypipes | dansmith: "Starting with 0 hosts" | 14:20 |
| dansmith | jaypipes: that's after we've correlated the compute nodes and the placement result | 14:20 |
| jroll | ah, yeah | 14:21 |
| *** NikhilS has quit IRC | 14:21 | |
| dansmith | I wanna know if placement is returning zero, or we have no CNs in the DB | 14:21 |
| dansmith | I have been assuming the latter | 14:21 |
| dansmith | since we see us reporting from the compute node to placemetn | 14:21 |
| jroll | ya, needs more logging | 14:21 |
| *** cdent has joined #openstack-nova | 14:22 | |
| *** annegentle has joined #openstack-nova | 14:22 | |
| EmilienM | is the log level of tripleo jobs is enough? let me know if I can help | 14:22 |
| jroll | EmilienM: it's a failure to log things in nova code, not the log level | 14:23 |
| jaypipes | EmilienM: yeah, it is I think. it's just we need more debug logging in nova-scheduler. | 14:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Log placement result in scheduler https://review.openstack.org/447520 | 14:23 |
| dansmith | this ^ | 14:23 |
| jaypipes | thx dansmith | 14:23 |
| *** kaisers_ has quit IRC | 14:23 | |
| jroll | ++ | 14:23 |
| *** jichen has quit IRC | 14:24 | |
| dansmith | EmilienM: can you make something run on top of that log patch? | 14:24 |
| mriedem | that's going to blow up | 14:24 |
| mriedem | that's just a debug change right? are we planning on merging that? | 14:24 |
| dansmith | mriedem: no, just for debug | 14:24 |
| dansmith | mriedem: the real one need not log rps in the warning case I think | 14:25 |
| dansmith | I just wanted to know if it's empty or none | 14:25 |
| mriedem | please DNM prefix the commit message or something, or -2 | 14:25 |
| jroll | can we merge it just so mriedem gets slammed with logs? | 14:25 |
| *** ijw has quit IRC | 14:25 | |
| mriedem | dansmith: agree, but i could see someone mistakenly approving this | 14:25 |
| EmilienM | dansmith: yes | 14:25 |
| EmilienM | my revert | 14:26 |
| EmilienM | oh wait | 14:26 |
| *** NikhilS has joined #openstack-nova | 14:26 | |
| EmilienM | no | 14:26 |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Log placement result in scheduler https://review.openstack.org/447520 | 14:26 |
| EmilienM | gimme a sec | 14:26 |
| EmilienM | dansmith: here: https://review.openstack.org/#/c/447522/ | 14:28 |
| *** ijw has joined #openstack-nova | 14:28 | |
| *** NikhilS has quit IRC | 14:28 | |
| *** artom has joined #openstack-nova | 14:29 | |
| * jroll is still wondering what's different about the tripleo CI that this would fail | 14:30 | |
| *** zz_dimtruck is now known as dimtruck | 14:30 | |
| dansmith | jroll: yeah, | 14:31 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: encryptors: Switch to os-brick encryptor classes https://review.openstack.org/391597 | 14:31 |
| efried | So it turns out that assert_called_once and assert_not_called aren't actually methods on a mock. Without speccing, they'll succeed happily just like any nonexistent method would. There's 80ish instances of these in the nova tree. May I open a bug and fix this, or is there something I don't know? | 14:31 |
| dansmith | I don't even see some debug logs that we should be seeing in scheduler | 14:31 |
| jroll | the only thing I've noticed so far is that the flavor isn't an exact match to the node | 14:31 |
| jroll | oh, hm | 14:32 |
| dansmith | like we should see this: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L590-L591 | 14:32 |
| dansmith | but unless I just suck at searching I can't find it | 14:32 |
| jroll | agree | 14:32 |
| mriedem | efried: are you looking at std lib or the mock library? | 14:33 |
| efried | mriedem How do I know which is which? | 14:33 |
| mriedem | efried: b/c they are in the lib https://github.com/testing-cabal/mock/blob/2.0.0/mock/mock.py#L890 | 14:33 |
| mriedem | and we use the lib | 14:33 |
| mriedem | in the py27 job | 14:33 |
| *** CristinaPauna has quit IRC | 14:33 | |
| mriedem | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L149 | 14:34 |
| efried | What I do know is that the tests I found this out on are using 'import mock'. | 14:34 |
| jroll | dansmith: I wonder how relevant this is compute_manager=ironic.nova.compute.manager.ClusteredComputeManager | 14:34 |
| jroll | that should not be there | 14:34 |
| dansmith | jroll: w..t..f | 14:34 |
| efried | So I guess that behaves differently depending on whether the mock library is installed or not? | 14:34 |
| jroll | dansmith: wait, I think that's not even in ironic tree, wat | 14:34 |
| efried | ...the import, that is? | 14:34 |
| dansmith | jroll: that would be majorly problematic for a lot of things we've just done | 14:35 |
| jroll | dansmith: indeed | 14:35 |
| *** CristinaPauna has joined #openstack-nova | 14:35 | |
| jroll | dansmith: confirmed, that module doesn't exist in ironic master | 14:35 |
| dansmith | ummm.... | 14:35 |
| mriedem | you can't load up out of tree compute managers anymore | 14:35 |
| mriedem | we dropped the ability to do that awhile ago, like newton i think | 14:35 |
| jroll | mriedem: so that config is a noop yeah? | 14:35 |
| mriedem | most likely | 14:35 |
| jroll | ok | 14:36 |
| dansmith | oh, compute manager, I was thinking host manager | 14:36 |
| jroll | efried: we always use the third party lib, which is api-compatible with the 3.6 version of stdlib | 14:36 |
| mriedem | http://codesearch.openstack.org/?q=ironic.nova.compute.manager.ClusteredComputeManager&i=nope&files=&repos= | 14:36 |
| efried | jroll Okay, must be something in my local env that's using the stdlib instead, then. | 14:36 |
| openstackgerrit | Sean Dague proposed openstack/nova master: update i18n guide for nova https://review.openstack.org/447528 | 14:36 |
| dansmith | jroll: is that the manager that used to always report the same hostname to allow for "HA" ? | 14:37 |
| jroll | dansmith: yeah | 14:37 |
| dansmith | EmilienM: so that should be removed from tripleo, especially since it'll be removed at some point | 14:38 |
| *** kfarr has joined #openstack-nova | 14:38 | |
| *** efried has quit IRC | 14:39 | |
| jroll | dansmith: it's lone gone already | 14:39 |
| *** udesale has quit IRC | 14:39 | |
| dansmith | jroll: they're still setting it though yes? | 14:39 |
| jroll | yeah, it should be removed, but clearly not breaking things | 14:40 |
| jroll | "clearly" ehh | 14:40 |
| mriedem | https://github.com/openstack/nova/commit/d9a7c16c305cd7016195cada883867411d3a5eb9 | 14:40 |
| jroll | you get what I mean | 14:40 |
| *** isq has quit IRC | 14:40 | |
| dansmith | ah, yeah, I figured the CONF was still there to avoid breaking people with it set | 14:40 |
| mriedem | i guess we only removed the compute manager config in ocata | 14:40 |
| mriedem | we don't fail on busted config | 14:40 |
| dansmith | apparently not | 14:40 |
| mriedem | i think there is a flag to set if you have busted config, but we don't run with it | 14:40 |
| dansmith | okay, I know I've seen it before | 14:41 |
| jroll | thanks for confirmation mriedem | 14:41 |
| dansmith | well, I dunno why we're not seeing some of the other messages that should be in scheduler | 14:41 |
| dansmith | EmilienM: how can I see the exact git hash of nova that is being tested here? | 14:42 |
| mriedem | is the IronicHostManager overriding the methods that have the logging you expect to ese? | 14:42 |
| mriedem | *see | 14:42 |
| dansmith | I don't think so, but let me look | 14:42 |
| jroll | shouldn't be | 14:42 |
| dansmith | yeah, no | 14:42 |
| dansmith | I think the nova hash is 47d90a3 | 14:43 |
| *** diga has joined #openstack-nova | 14:44 | |
| dansmith | in this run http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/rpm-qa.txt.gz | 14:44 |
| * EmilienM reads backlog | 14:44 | |
| EmilienM | dansmith: yes that | 14:44 |
| dansmith | so actually, if we get nothing back from placement, we'd not hit the messages I'm looking for, I think | 14:45 |
| dansmith | so it must be that | 14:45 |
| dansmith | but I have no idea why | 14:45 |
| *** dikonoor has quit IRC | 14:45 | |
| EmilienM | ok so I need to remove ironic.nova.compute.manager.ClusteredComputeManager | 14:45 |
| dansmith | EmilienM: yeah, that's just a dead config opt | 14:45 |
| EmilienM | replaced by what? | 14:45 |
| dansmith | not replaced by anything | 14:45 |
| *** irenab_ has quit IRC | 14:45 | |
| mriedem | EmilienM: https://github.com/openstack/nova/commit/d9a7c16c305cd7016195cada883867411d3a5eb9 | 14:45 |
| EmilienM | ok | 14:46 |
| jroll | replaced by AWESOME | 14:46 |
| EmilienM | do you think removing the option would help tripleo issue? | 14:47 |
| jroll | EmilienM: the need for that was replaced by https://github.com/openstack/nova/commit/6047d790a32ef5a65d4d6b029f673ce53c3d4141 | 14:47 |
| jroll | no | 14:47 |
| EmilienM | or just cleanup | 14:47 |
| jroll | just cleanup | 14:47 |
| EmilienM | k | 14:47 |
| *** yamahata has joined #openstack-nova | 14:47 | |
| EmilienM | "There's no configuration to enable this mode; it's always running." | 14:47 |
| EmilienM | jroll: ❤ | 14:48 |
| jroll | :) | 14:48 |
| *** marst has quit IRC | 14:48 | |
| mriedem | seems wasteful for FilterScheduler._schedule to even call HostManager.get_filtered_hosts if we already know we don't have any hosts (resource providers from placement) | 14:48 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 14:49 |
| mriedem | but maybe the spaghetti makes sense someohw | 14:49 |
| dansmith | mriedem: right and we don't | 14:49 |
| *** sapcc-bot has quit IRC | 14:49 | |
| dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L175-L176 | 14:50 |
| EmilienM | jroll: /me taking care of this cleanup https://bugs.launchpad.net/puppet-nova/+bug/1674375 | 14:50 |
| openstack | Launchpad bug 1674375 in puppet-nova "compute_manager is deprecated, we should stop using it" [High,Triaged] - Assigned to Emilien Macchi (emilienm) | 14:50 |
| *** david_1 has joined #openstack-nova | 14:50 | |
| *** sapcc-bot has joined #openstack-nova | 14:50 | |
| *** tommy_ has joined #openstack-nova | 14:50 | |
| *** carthaca_ has joined #openstack-nova | 14:50 | |
| *** seife_ has joined #openstack-nova | 14:50 | |
| *** tpatzig_ has joined #openstack-nova | 14:50 | |
| *** databus23_ has joined #openstack-nova | 14:50 | |
| mriedem | dansmith: i'm talking about this https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L108 | 14:50 |
| jroll | EmilienM: nice, thanks! s/deprecated/removed/ btw :) | 14:50 |
| mriedem | hosts is [] as you just poined out | 14:50 |
| mriedem | *pointed | 14:50 |
| *** tommy_ is now known as Guest72847 | 14:50 | |
| mriedem | but we still go through self.host_manager.get_filtered_hosts and all of it's checks on an empty list | 14:50 |
| mriedem | which seems like a waste of cycles | 14:50 |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 14:50 |
| dansmith | mriedem: ah, well, it's empty, but yeah I see what you mean | 14:50 |
| mriedem | unrelated to this problem | 14:50 |
| dansmith | not very impactful | 14:50 |
| dansmith | yeah | 14:50 |
| mriedem | but i thought we were always paranoid about scheduler perf | 14:51 |
| mriedem | like, no logging | 14:51 |
| dansmith | that will fail after the first filter, though, | 14:51 |
| dansmith | and the first one is retryfilter, | 14:51 |
| dansmith | which does basically nothing | 14:51 |
| dansmith | so it's really small I think | 14:51 |
| dansmith | still wasteful, but.. minor I beleive | 14:51 |
| mriedem | first default filter | 14:51 |
| mriedem | sure | 14:52 |
| *** carthaca_ has quit IRC | 14:52 | |
| *** Guest72847 has quit IRC | 14:52 | |
| *** seife_ has quit IRC | 14:52 | |
| *** databus23_ has quit IRC | 14:52 | |
| *** tpatzig_ has quit IRC | 14:52 | |
| *** david_1 has quit IRC | 14:52 | |
| dansmith | sure, first default | 14:52 |
| EmilienM | jroll: done, thx | 14:52 |
| mriedem | the first filter in my custom deployment actually calls out to my backend inventory datastore and gathers all of the latest goodies, just to filter them on an empty list :) | 14:52 |
| bauzas | dansmith: EmilienM: just seeing your discussions, any questions I could help you ? | 14:52 |
| *** armax has quit IRC | 14:53 | |
| dansmith | bauzas: we don't need any more questions.. we need ANSWERS! | 14:53 |
| dansmith | mriedem: that's quite unfortunate :) | 14:53 |
| bauzas | so, I could help for answers :p | 14:53 |
| dansmith | mriedem: especially for that to be before retryfilter :D | 14:53 |
| mriedem | it also ssh's into all of my computes and gathers data | 14:53 |
| mriedem | it's not very smart | 14:54 |
| mriedem | but it's enterprise ready | 14:54 |
| dansmith | lol | 14:54 |
| *** carthaca_ has joined #openstack-nova | 14:54 | |
| *** tpatzig_ has joined #openstack-nova | 14:54 | |
| *** databus23_ has joined #openstack-nova | 14:55 | |
| *** david_1 has joined #openstack-nova | 14:55 | |
| *** seife_ has joined #openstack-nova | 14:55 | |
| *** lucas-hungry is now known as lucasagomes | 14:56 | |
| *** marst has joined #openstack-nova | 14:56 | |
| *** marst has quit IRC | 14:56 | |
| *** marst has joined #openstack-nova | 14:56 | |
| *** carthaca_ has quit IRC | 14:56 | |
| *** tpatzig_ has quit IRC | 14:56 | |
| *** databus23_ has quit IRC | 14:56 | |
| *** david_1 has quit IRC | 14:56 | |
| *** seife_ has quit IRC | 14:56 | |
| *** Chason has quit IRC | 14:57 | |
| *** Chason has joined #openstack-nova | 14:58 | |
| *** sapcc-bot1 has joined #openstack-nova | 14:58 | |
| *** sapcc-bot has quit IRC | 14:58 | |
| *** databus23_ has joined #openstack-nova | 14:58 | |
| *** tpatzig_ has joined #openstack-nova | 14:58 | |
| *** seife_ has joined #openstack-nova | 14:58 | |
| *** carthaca_ has joined #openstack-nova | 14:58 | |
| *** david_1 has joined #openstack-nova | 14:58 | |
| *** seife_ has quit IRC | 15:00 | |
| *** tpatzig_ has quit IRC | 15:00 | |
| *** databus23_ has quit IRC | 15:00 | |
| *** david_1 has quit IRC | 15:00 | |
| *** annegentle has quit IRC | 15:00 | |
| *** carthaca_ has quit IRC | 15:01 | |
| *** sapcc-bot1 has quit IRC | 15:01 | |
| *** annegentle has joined #openstack-nova | 15:01 | |
| *** sapcc-bot has joined #openstack-nova | 15:01 | |
| *** david_1 has joined #openstack-nova | 15:02 | |
| *** tpatzig_ has joined #openstack-nova | 15:02 | |
| *** carthaca_ has joined #openstack-nova | 15:02 | |
| *** seife_ has joined #openstack-nova | 15:02 | |
| *** databus23_ has joined #openstack-nova | 15:02 | |
| *** sapcc-bot has quit IRC | 15:02 | |
| *** tpatzig_ has quit IRC | 15:02 | |
| *** carthaca_ has quit IRC | 15:02 | |
| *** seife_ has quit IRC | 15:02 | |
| *** databus23_ has quit IRC | 15:02 | |
| *** david_1 has quit IRC | 15:02 | |
| *** david_1 has joined #openstack-nova | 15:03 | |
| *** sapcc-bot has joined #openstack-nova | 15:03 | |
| *** tpatzig_ has joined #openstack-nova | 15:03 | |
| *** carthaca_ has joined #openstack-nova | 15:03 | |
| *** seife_ has joined #openstack-nova | 15:03 | |
| *** databus23_ has joined #openstack-nova | 15:03 | |
| *** databus23_ has quit IRC | 15:04 | |
| *** seife_ has quit IRC | 15:04 | |
| *** tpatzig_ has quit IRC | 15:04 | |
| *** david_1 has quit IRC | 15:04 | |
| mriedem | how do we know which flavor was actually used in the server create request? | 15:05 |
| *** sridharg has quit IRC | 15:05 | |
| mriedem | EmilienM: you pointed at http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-03-19_06_38_20_764 but there are more than that flavor | 15:05 |
| dansmith | mriedem: you can look at the placement request | 15:05 |
| mriedem | ok | 15:05 |
| dansmith | mriedem: http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-placement-api.txt.gz#_2017-03-19_07_19_34_113 | 15:05 |
| mriedem | we don't correlate the request ids between the nova and placement services yet do we? | 15:05 |
| dansmith | mriedem: bauzas was just telling me that we do | 15:05 |
| mriedem | cdent: you were the last to look at that i thought | 15:06 |
| dansmith | mriedem: looks like the request I linked matches the 40,4g,1 flavor | 15:06 |
| EmilienM | jroll: https://review.openstack.org/#/q/topic:bug/1674375 | 15:06 |
| bauzas | dansmith: mriedem: enf of Ocata-3, we were not yet passing req-id | 15:06 |
| bauzas | dansmith: mriedem: but I thought cdent threw a patch for that | 15:07 |
| mriedem | i'm looking for req-702e96af-318d-4611-af19-d6a1f3321c6a | 15:07 |
| mriedem | http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-api.txt.gz#_2017-03-19_07_19_33_201 | 15:07 |
| mriedem | that's the ceph storage instance | 15:07 |
| *** kevinz has quit IRC | 15:07 | |
| mriedem | i know we log the placement request id from the response in the scheduler report client | 15:07 |
| mriedem | but we don't pass and use a request id from n-sch to placement and log with that req id in the placement logs | 15:07 |
| mriedem | which is what we need | 15:07 |
| jroll | we don't do that between n-cpu and ironic either :( | 15:08 |
| dansmith | mriedem: they all look the same to me | 15:08 |
| openstackgerrit | Sean Dague proposed openstack/nova master: update i18n guide for nova https://review.openstack.org/447528 | 15:08 |
| mriedem | but yeah otherwise GET /placement/resource_providers?resources=DISK_GB%3A40%2CMEMORY_MB%3A4096%2CVCPU%3A1 | 15:08 |
| *** adisky_ has quit IRC | 15:09 | |
| bauzas | mriedem: last time I debugged that, I was just looking at the time instead :( | 15:09 |
| mriedem | in the n-cpu logs, i see the 3 nodes reporting capacity just before that | 15:10 |
| mriedem | so those are all good | 15:10 |
| mriedem | at least i n_report_hypervisor_resource_view | 15:10 |
| cdent | bauzas, mriedem: not fully contextualized, but I improved some of the logging, but there was no super handy way to make the request id used on the nova side also because used on the placement id, it would require an additional header, or changing the middleware | 15:10 |
| mriedem | i figured we could pass a header | 15:10 |
| mriedem | and if the header is there, use it, else generate a new request id | 15:11 |
| cdent | yeah, that would require changing the oslo middleware, so I stalled out | 15:11 |
| dansmith | don't we already have a way to do that between services? | 15:11 |
| dansmith | I thought we did between nova and glance, for example | 15:11 |
| cdent | I think I left that as a comment on the bug, but it may have been lost to the sands of time | 15:12 |
| mriedem | i think it's something that keeps coming up with operators | 15:12 |
| mriedem | sdague might know | 15:12 |
| bauzas | dansmith: mriedem: cdent: yeah I think we're using same req-id between projects, nope ? | 15:12 |
| cdent | it does seem like a thing worth doing | 15:12 |
| EmilienM | jroll: next review on puppet-nova and you're core | 15:12 |
| cdent | if there's something we can borrow from nova glance interactions, great | 15:13 |
| *** diga has quit IRC | 15:13 | |
| jroll | EmilienM: nice, I'll go find some typo fixes | 15:13 |
| mriedem | i don't see anything between nova and glance | 15:13 |
| sdague | mriedem: it does keep coming up, there is no generic solution | 15:13 |
| dansmith | okay I thought that was sorted already | 15:13 |
| cdent | is it safe for service to accent an incoming request id? | 15:14 |
| sdague | there was going to be some logging of both at some point, but it remains unsolved | 15:14 |
| cdent | wow, my typing | 15:14 |
| cdent | sorry | 15:14 |
| cdent | accept! | 15:14 |
| sdague | cdent: that is the sticking point | 15:14 |
| sdague | some people get really concerned about that | 15:14 |
| sdague | it would all be super easy if we could get past that | 15:14 |
| mriedem | what's the concern? the request id is read-only isn't it? | 15:14 |
| sdague | mriedem: the concern is accepting request id from the user | 15:15 |
| jroll | so weird to me, today it's always wrong, if we accept one then worst case it's wrong | 15:15 |
| dansmith | mriedem: attackers confusing the ops' logs I imagine | 15:15 |
| jroll | reject anything that isn't 'req-' + uuid_regex | 15:15 |
| dansmith | accepting it but only using it if config says to seems reasonable | 15:15 |
| cdent | jroll: then send all the same uuid to fudge up the logs | 15:15 |
| dansmith | for ops that prefer correlation with some risk | 15:15 |
| mriedem | can we tell if it's a service user request rather than a user user? | 15:16 |
| sdague | yeh, so honestly the service user / service token bits might make this better | 15:16 |
| mriedem | and if so, use the provided request id | 15:16 |
| cdent | mriedem: then we get into strange issues with ordering of middleware | 15:16 |
| sdague | mriedem: we can now sometimes, that's really new though | 15:16 |
| mriedem | right, we're already doing the re-auth thing | 15:16 |
| jroll | cdent: I guess, if I was an operator I'd just ignore support requests and bugs for that request id :P | 15:16 |
| *** carthaca_ has quit IRC | 15:16 | |
| *** sapcc-bot has quit IRC | 15:16 | |
| *** moshele has joined #openstack-nova | 15:16 | |
| *** jamielennox has quit IRC | 15:17 | |
| sdague | mriedem: I think there is probably a path forward that wouldn't freak everyone out based on service token | 15:17 |
| mriedem | anywho, total tangent | 15:17 |
| sdague | yep | 15:17 |
| mriedem | sounds like something for lbragstad and the vm+baremetal+totally+not+exclusive+club+platform team | 15:17 |
| cdent | i've always assumed we want the thing which logs begin and end of request to the first and last pieces of middleware that get touched, thus before any token analysis | 15:17 |
| mriedem | so i'm trying to go back to what could possibly change with https://review.openstack.org/#/c/441544/ | 15:18 |
| mriedem | and how self.scheduler_client.update_compute_node(compute_node) works if get_inventory isn't there | 15:19 |
| dansmith | mriedem: yeah, I don't know either, but if we're getting back zero from placement, I'm not sure what else to blame | 15:19 |
| mriedem | if the node is disabled, they both do the same thing, they return {} inventory which tells the report client to delete inventory for that provider | 15:19 |
| dansmith | mriedem: so the log result plus the revert test of that should be interesting | 15:19 |
| *** kaisers_ has joined #openstack-nova | 15:20 | |
| *** crushil has quit IRC | 15:21 | |
| mriedem | so, does this seem weird to anyone else? | 15:22 |
| mriedem | if inv_data == curr.get('inventories', {}): | 15:22 |
| *** ijw has quit IRC | 15:23 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L495 | 15:23 |
| dansmith | if the thing we're going to report is equal to what we have, then don't report? | 15:23 |
| bauzas | mriedem: that just means "heh look, we already have that" | 15:23 |
| dansmith | and if what is being reported is nothing and nothing is already stored, then.. don't do anything either | 15:24 |
| bauzas | dansmith: mriedem: FWIW, we already have that in place in the RT | 15:24 |
| mriedem | is the dict compare always good in py27 even with random key order and whatnot? | 15:24 |
| *** ijw has joined #openstack-nova | 15:24 | |
| bauzas | old-world-ish of course | 15:24 |
| *** siva_krishnan has joined #openstack-nova | 15:24 | |
| *** shaner has quit IRC | 15:24 | |
| dansmith | sure | 15:24 |
| *** kaisers_ has quit IRC | 15:25 | |
| lbragstad | mriedem sdague just finished parsing the scroll back - kinda sounds similar to what jamielennox was doing with service users and validating expired tokens | 15:25 |
| mriedem | bauzas: if you're referring to this https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L730 it's a bit different | 15:25 |
| jroll | >>> {'a': 'b', 'c': 'd'} == {'c': 'd', 'a': 'b'} | 15:25 |
| jroll | True | 15:25 |
| jroll | mriedem: ^ | 15:25 |
| bauzas | mriedem: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L730 | 15:25 |
| bauzas | yeah | 15:25 |
| openstackgerrit | Sujitha proposed openstack/nova master: Add helper method to add additional data about policy rule. https://review.openstack.org/434842 | 15:26 |
| mriedem | ok nvm then, that just surprised me given how many times assertEqual on dicts has failed b/c of random hash seeds | 15:26 |
| jroll | well, assertEqual isn't so smart iirc | 15:27 |
| siva_krishnan | mriedem: can you remove -2 for this patch https://review.openstack.org/#/c/308198/ ? | 15:27 |
| dansmith | I think it's more assertEqual where we've pulled things out of a dict in random order anyway, right? | 15:27 |
| *** baoli has quit IRC | 15:28 | |
| *** tpatzig_ has joined #openstack-nova | 15:28 | |
| *** david_1 has joined #openstack-nova | 15:28 | |
| *** sapcc-bot has joined #openstack-nova | 15:28 | |
| *** mkoderer_ has joined #openstack-nova | 15:28 | |
| *** databus23_ has joined #openstack-nova | 15:28 | |
| *** seife_ has joined #openstack-nova | 15:28 | |
| *** prateek has quit IRC | 15:28 | |
| mriedem | siva_krishnan: the bp isn't approved | 15:29 |
| *** david_1 has quit IRC | 15:30 | |
| *** baoli has joined #openstack-nova | 15:30 | |
| *** mkoderer_ has quit IRC | 15:30 | |
| *** databus23_ has quit IRC | 15:30 | |
| *** shaner has joined #openstack-nova | 15:30 | |
| *** seife_ has quit IRC | 15:30 | |
| sdague | dansmith: right, it's not actually that 2 dicts aren't equal, it's that items() happens in different orders | 15:31 |
| dansmith | yeah | 15:31 |
| *** beekneemech is now known as bnemec | 15:32 | |
| *** armax has joined #openstack-nova | 15:33 | |
| mriedem | ok here are the calls for one resource provider, it's created here: | 15:34 |
| mriedem | http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-03-19_07_10_44_256 | 15:34 |
| siva_krishnan | mriedem: makes sense. it's kind of an POC, I thought reviewers can suggest any changes to spec or code if they feel so. | 15:34 |
| mriedem | 4 minutes later we finally get the real resource view from ironic | 15:34 |
| mriedem | http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-03-19_07_14_42_794 | 15:34 |
| mriedem | and right after that, same second, we update inventory for that provider | 15:35 |
| mriedem | http://logs.openstack.org/periodic/periodic-tripleo-ci-centos-7-ovb-nonha/12d034e/logs/undercloud/var/log/nova/nova-compute.txt.gz#_2017-03-19_07_14_42_928 | 15:35 |
| *** efried has joined #openstack-nova | 15:35 | |
| *** baoli has quit IRC | 15:36 | |
| mriedem | we never update inventory for that provider after that, because it never changes | 15:36 |
| jaypipes | mriedem: yep. | 15:37 |
| mriedem | the old way, we would have called driver.get_available_resource and updated the compute node record with the values (if they changed) | 15:37 |
| openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_actions. https://review.openstack.org/440829 | 15:37 |
| mriedem | and the compute node would be passed to self.scheduler_client.update_compute_node(compute_node) to update the rp inventory | 15:37 |
| mriedem | if it changed | 15:38 |
| *** baoli has joined #openstack-nova | 15:38 | |
| mriedem | new style we call get_inventory which in this case returns a dict with 3 keys, for vcpu/disk_gb/memory_mb | 15:38 |
| jaypipes | mriedem: well, I moved compute_node.save() into the RT's _update() method. | 15:38 |
| jaypipes | mriedem: so regardless of whether it's get_inventory() or update_compute_node() that is called, we still update the compute node DB record. | 15:39 |
| *** kenperkins has quit IRC | 15:39 | |
| mriedem | i know | 15:39 |
| *** kenperkins has joined #openstack-nova | 15:39 | |
| jaypipes | ok, sorry | 15:39 |
| mriedem | the save() doesn't really matter | 15:39 |
| mriedem | update_compute_node pulls the values off the ComputeNode object | 15:40 |
| jaypipes | sorry, I thought you were suggesting that the compute_nodes table wasn't being updated. | 15:40 |
| mriedem | which is updated in _init_compute_node | 15:40 |
| efried | mriedem jroll Okay, here's an update. I _am_ using the mock library; and mock _does_ have those methods (assert_not_called, assert_called_once) UNLESS I use mock.patch AND the target is a function (as opposed to a class). | 15:43 |
| sneti | alex_xu, can you please review this patch when you have time: https://review.openstack.org/#/c/396420/ | 15:43 |
| efried | Is that a bug in the mock lib? | 15:43 |
| mriedem | efried: no idea, i'd ask lifeless | 15:43 |
| efried | Here, or a different channel? | 15:44 |
| mriedem | he's here | 15:44 |
| mriedem | but new zealand | 15:44 |
| mriedem | so not here | 15:44 |
| efried | Ight. I guess I can open a bug, and it'll be bounced if it's invalid. Thanks. | 15:45 |
| *** jaosorior has quit IRC | 15:45 | |
| mriedem | efried: bug where? the mock lib/ | 15:45 |
| mriedem | ? | 15:45 |
| efried | Yeah. | 15:45 |
| mriedem | EmilienM: which jobs are we looking for in https://review.openstack.org/#/c/447522/ ? | 15:45 |
| mriedem | because some passed and one failed | 15:45 |
| EmilienM | mriedem: the ovb jobs | 15:50 |
| EmilienM | mriedem: ovb-nonha or ovb-ha | 15:50 |
| *** gongysh has quit IRC | 15:50 | |
| *** links has joined #openstack-nova | 15:50 | |
| mriedem | i see the other queue now | 15:50 |
| EmilienM | mriedem: gate-tripleo-ci-centos-7-ovb-nonha or gate-tripleo-ci-centos-7-ovb-ha or gate-tripleo-ci-centos-7-ovb-updates | 15:51 |
| EmilienM | they are the one who actually test Ironic / Nova | 15:51 |
| *** psachin has quit IRC | 15:51 | |
| *** links has quit IRC | 15:55 | |
| *** links has joined #openstack-nova | 15:56 | |
| *** voelzmo has quit IRC | 15:56 | |
| *** annegentle has quit IRC | 15:58 | |
| *** jaosorior has joined #openstack-nova | 15:59 | |
| *** annegentle has joined #openstack-nova | 16:00 | |
| sfinucan | mriedem: I have those sphinx issues all fixed in this series https://review.openstack.org/#/c/445906/ | 16:00 |
| *** markus_z has quit IRC | 16:01 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova master: Fix wrong unit test about config option enabled_apis https://review.openstack.org/446898 | 16:01 |
| *** Chason has quit IRC | 16:01 | |
| sfinucan | Just fyi | 16:01 |
| mriedem | ok cool | 16:01 |
| *** iceyao has joined #openstack-nova | 16:02 | |
| *** snecklifter has joined #openstack-nova | 16:03 | |
| *** voelzmo has joined #openstack-nova | 16:03 | |
| snecklifter | Hello #openstack-nova | 16:04 |
| snecklifter | Would it be possible to get a review on this: https://review.openstack.org/#/c/393489/ | 16:04 |
| snecklifter | CI is green except for a couple of non-voters | 16:04 |
| *** vks1 has quit IRC | 16:05 | |
| *** armax has quit IRC | 16:05 | |
| *** links has quit IRC | 16:05 | |
| *** liangy has quit IRC | 16:08 | |
| *** jahsis has quit IRC | 16:09 | |
| *** rmart04 has quit IRC | 16:09 | |
| *** crushil has joined #openstack-nova | 16:10 | |
| *** iceyao has quit IRC | 16:12 | |
| mriedem | snecklifter: you're missing a version bump on the ImageMetaProps object | 16:12 |
| mriedem | that test which checks the hash indicates that the version needs a bump | 16:12 |
| openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 16:13 |
| *** tesseract has quit IRC | 16:15 | |
| *** dtp has joined #openstack-nova | 16:16 | |
| *** Sukhdev has quit IRC | 16:16 | |
| *** Sukhdev_ has joined #openstack-nova | 16:16 | |
| *** Jack_Iv has quit IRC | 16:17 | |
| *** slaweq_ has joined #openstack-nova | 16:18 | |
| *** jamielennox has joined #openstack-nova | 16:19 | |
| *** gszasz has joined #openstack-nova | 16:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove unused VIFModel._get_legacy method https://review.openstack.org/447577 | 16:20 |
| mriedem | zzzeek: is there no way to add a column to an existing table and make it nullable=False by providing a default value? | 16:21 |
| mriedem | zzzeek: i.e. https://review.openstack.org/#/c/427901/3/nova/db/sqlalchemy/migrate_repo/versions/358_add_compute_node_mapped.py | 16:21 |
| *** jamielennox is now known as jamielennox|away | 16:21 | |
| zzzeek | mriedem: yes if you give it a server side default it will add. however, that's one of the "slow" migrations for a big table | 16:21 |
| zzzeek | mriedem: the code example is incorrect, it needs to be server_default="0" | 16:22 |
| mriedem | because "A FetchedValue instance, str, Unicode or text() construct representing the DDL DEFAULT value for the column." | 16:23 |
| mriedem | ? | 16:23 |
| zzzeek | "0" is a str | 16:23 |
| snecklifter | mriedem: thanks | 16:23 |
| mriedem | zzzeek: yeah so i'm not sure what code example is incorrect | 16:23 |
| zzzeek | mriedem: this one: https://review.openstack.org/#/c/427901/3/nova/db/sqlalchemy/migrate_repo/versions/358_add_compute_node_mapped.py | 16:24 |
| zzzeek | "server_default" vs. "default" | 16:24 |
| snecklifter | mriedem: so change 1.16 to 1.17? | 16:24 |
| *** tbachman has quit IRC | 16:24 | |
| dansmith | zzzeek: ah, jay said it wasn't slow | 16:24 |
| zzzeek | dansmith: well if you have 1M rows it has to popualte 1M rows w/ a value and probably do some locking too. depends on your table | 16:25 |
| dansmith | zzzeek: I tried server_default at jaypipes' suggestion anyway, and it causes other problems for us in the oslo.db migration checker | 16:25 |
| dansmith | zzzeek: ah, definitely can't have that then | 16:25 |
| mriedem | CERN has what, 10K+ compute nodes? | 16:25 |
| zzzeek | dansmith: i'd agree it's not "slow" but it is def. a variable-length operation, i would think | 16:25 |
| zzzeek | dansmith: problems in oslo.db should be fixed in oslo.db , feel free to post a launchpad | 16:26 |
| dansmith | zzzeek: okay, what I meant was, jaypipes said it wouldn't require a rewrite | 16:26 |
| dansmith | it really doesn't matter though, | 16:26 |
| dansmith | we can handle this on read just as easily and not have to walk the table | 16:26 |
| jaypipes | zzzeek: I don't think anyone's suggesting there's something wrong with oslo.db. | 16:26 |
| zzzeek | dansmith: ok jaypipes knows mysql internals more than I do but in my experience, when you have server deafult in a migration w/ nullable=false, it has to write data to every row | 16:27 |
| *** nic has joined #openstack-nova | 16:27 | |
| zzzeek | jaypipes: dansmith just referred to some problem with oslo.db's migration checker | 16:27 |
| *** nkrinner is now known as nkrinner_afk | 16:27 | |
| *** pcaruana has quit IRC | 16:27 | |
| mriedem | snecklifter: yes | 16:27 |
| jaypipes | zzzeek: yes, that's correct. | 16:27 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505 | 16:27 |
| *** arne_r has quit IRC | 16:27 | |
| jaypipes | zzzeek: ah, the migration checker thing... sorry, yeah | 16:27 |
| claudiub | efried: mriedem: ^ in case you're curious about the autospec stuff. :) | 16:29 |
| *** Sukhdev has joined #openstack-nova | 16:29 | |
| claudiub | still experimental, and it's just the 1st step. | 16:29 |
| mriedem | dansmith: why even set mapped=0 in fake compute node records in https://review.openstack.org/#/c/427901/ ? | 16:31 |
| *** yamahata has quit IRC | 16:31 | |
| EmilienM | mriedem: I have updates | 16:32 |
| EmilienM | 1. the patch that dansmith sent to have more logging, is still running so I'll give update later on logs | 16:32 |
| dansmith | mriedem: because we create from that and we don't have it optional on create | 16:32 |
| EmilienM | 2. the patch that revert jaypipes's patch is fixing our problem, I see it in the logs | 16:32 |
| dansmith | mriedem: go run without that and I think you'll see it tries to lazy-load on create | 16:32 |
| EmilienM | so I confirm the issue is related to this patch: https://review.openstack.org/#/c/441544/ | 16:33 |
| *** slaweq_ has quit IRC | 16:33 | |
| openstackgerrit | Christopher Brown proposed openstack/nova master: Add lan9118 as valid nic for hw_vif_model property for qemu https://review.openstack.org/393489 | 16:33 |
| mriedem | dansmith: create of the object or the db record? | 16:33 |
| EmilienM | [overcloud.AllNodesDeploySteps.CephStorageDeployment_Step3]: CREATE_COMPLETE state changed ---- It means that overcloud nodes were deployed successfuly (and now deployment is moving forward) | 16:33 |
| dansmith | mriedem: ComputeNode.create() | 16:34 |
| dansmith | mriedem: even still, we're adding something as nullable purely for speed, we don't want to treat it as optional going forward | 16:34 |
| mriedem | i don't see how create() will fail when _from_db_object has a check for null: value = 0 if value is None else value | 16:35 |
| *** xyang1 has joined #openstack-nova | 16:37 | |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Replace diagnostics objects with Nova diagnostics objects https://review.openstack.org/394480 | 16:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Added nova objects for intance diagnostics https://review.openstack.org/434745 | 16:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Standardization of VM diagnostics info API. https://review.openstack.org/355540 | 16:38 |
| openstackgerrit | Sergey Nikitin proposed openstack/nova master: Completed implementation of instance diagnostics for Xen https://review.openstack.org/399613 | 16:38 |
| dansmith | mriedem: it still pulls it out of the db object on L175, and only interprets the value and tolerates None on L207 | 16:40 |
| mriedem | dansmith: idk, but see my update in the change, i pulled it down, mangled that check and nothing failed | 16:41 |
| mriedem | so we're not testing the None case | 16:42 |
| dansmith | mriedem: you ran all of py27 and functional? | 16:42 |
| dansmith | mriedem: but yeah, might not be covering the None case | 16:42 |
| mriedem | no, nova.tests.unit.objects.test_compute_node and nova.tests.unit.compute | 16:42 |
| mriedem | i shouldn't probably need to dig into running all of functional for a simple logic check test | 16:42 |
| dansmith | if you're asking about covering none, then sure, | 16:43 |
| dansmith | if you're asking about changing that fake, I think lots of other places that use that fake | 16:43 |
| mriedem | i'm asking about testing none: https://review.openstack.org/#/c/427901/3/nova/objects/compute_node.py@208 | 16:43 |
| dansmith | yeah, willing to believe I'm missing that | 16:44 |
| mriedem | but i thought the whole point of a nullable=True with a default is you don't need to provide actual values | 16:44 |
| dansmith | mriedem: if you're really loading from the DB sure, | 16:44 |
| dansmith | if you're loading from a fake, then it matters | 16:44 |
| mriedem | true | 16:44 |
| mriedem | then you need the key | 16:45 |
| mriedem | but not the value | 16:45 |
| *** kaisers_ has joined #openstack-nova | 16:45 | |
| mriedem | i.e. your fake could just be mapped=NOne | 16:45 |
| mriedem | but anyway | 16:45 |
| dansmith | eventually I'd like to make this non-nullable, | 16:45 |
| openstackgerrit | Christopher Brown proposed openstack/nova master: Add lan9118 as valid nic for hw_vif_model property for qemu https://review.openstack.org/393489 | 16:45 |
| dansmith | so I'd rather a legit value in there, | 16:45 |
| dansmith | and then cover the specific load of null case as you pointed out | 16:45 |
| dansmith | but whatever | 16:45 |
| mriedem | sure, that's fine, let's just test the none case and move on with this enjoyable and productive day :) | 16:45 |
| mriedem | i'm in trouble for not putting the asparagus in the fridge | 16:46 |
| * cdent smh | 16:46 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Validate huge pages https://review.openstack.org/399653 | 16:46 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: create functional test base class https://review.openstack.org/407055 | 16:46 |
| mriedem | i mean, it grows in the ground right? | 16:46 |
| mriedem | which is unrefrigerated | 16:46 |
| cfriesen | This is maybe an odd question, but is there a primer somewhere on how to deal with running out of DB connections? Are there ways to optimize the python code to allow quicker connection re-use? | 16:47 |
| *** voelzmo has quit IRC | 16:47 | |
| *** voelzmo has joined #openstack-nova | 16:47 | |
| *** bmace has quit IRC | 16:48 | |
| *** andreas_s has quit IRC | 16:49 | |
| *** bmace has joined #openstack-nova | 16:49 | |
| *** jaosorior has quit IRC | 16:50 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Make 'get_domain' private https://review.openstack.org/417378 | 16:51 |
| *** hoangcx has joined #openstack-nova | 16:52 | |
| *** jaosorior has joined #openstack-nova | 16:52 | |
| *** Apoorva has joined #openstack-nova | 16:52 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Fix indentation in conf/netconf https://review.openstack.org/430717 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: netutils: Ignore 'use_ipv6' for network templates https://review.openstack.org/430910 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'use_ipv6' https://review.openstack.org/430911 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'use_ipv6' for port filters https://review.openstack.org/430909 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'allow_same_net_traffic' https://review.openstack.org/430913 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Final cleanups in conf/network https://review.openstack.org/431023 | 16:54 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Ignore 'allow_same_net_traffic' for port filters https://review.openstack.org/430912 | 16:54 |
| *** tbachman has joined #openstack-nova | 16:58 | |
| *** liangy has joined #openstack-nova | 16:59 | |
| *** unicell1 has quit IRC | 17:00 | |
| *** Sukhdev has quit IRC | 17:01 | |
| *** Sukhdev_ is now known as Sukhdev | 17:01 | |
| *** voelzmo has quit IRC | 17:01 | |
| *** liangy has quit IRC | 17:01 | |
| *** Jack_Iv has joined #openstack-nova | 17:01 | |
| *** kaisers_ has quit IRC | 17:05 | |
| *** fragatina has joined #openstack-nova | 17:05 | |
| *** fragatin_ has joined #openstack-nova | 17:06 | |
| *** fragatina has quit IRC | 17:10 | |
| *** Swami has quit IRC | 17:11 | |
| *** annegentle has quit IRC | 17:12 | |
| *** Jack_Iv has quit IRC | 17:15 | |
| *** Jack_Iv has joined #openstack-nova | 17:15 | |
| *** sc68cal has left #openstack-nova | 17:16 | |
| *** yamahata has joined #openstack-nova | 17:17 | |
| mriedem | EmilienM: yeah http://logs.openstack.org/22/447522/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-ha/2eecbc6/logs/undercloud/var/log/nova/nova-scheduler.txt.gz#_2017-03-20_16_44_02_099 | 17:17 |
| mriedem | logging in the scheduler shows nothing coming back from placement | 17:18 |
| mriedem | i'm going to add more logging to dan's patch for the actual resource reporting | 17:18 |
| EmilienM | ok, you'll need to recheck the tripleo-ci patch to actually see it in our logs | 17:18 |
| mriedem | yup | 17:18 |
| EmilienM | or even change something in the commit message to actually kick off jobs (jobs are still running now) | 17:19 |
| mriedem | when i push the dep, zuul will kill those running jobs | 17:20 |
| EmilienM | ah indeed | 17:21 |
| *** isq has joined #openstack-nova | 17:21 | |
| edmondsw | johnthetubaguy I replied to your comments in https://review.openstack.org/#/c/389314/ | 17:22 |
| johnthetubaguy | edmondsw: so I think we need a deprecated default on oslo.policy for all this kind of stuff | 17:24 |
| johnthetubaguy | s/on/in/ | 17:24 |
| edmondsw | johnthetubaguy for most things I agree, but I don't think you can in this case | 17:24 |
| *** Jack_Iv has quit IRC | 17:25 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Remove unused policy rule in admin_actions.py https://review.openstack.org/447611 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Log placement result in scheduler https://review.openstack.org/447520 | 17:25 |
| *** liangy has joined #openstack-nova | 17:25 | |
| *** moshele has quit IRC | 17:26 | |
| edmondsw | johnthetubaguy I gave 4 reasons that doesn't really make sense here in my reply to one of your comments | 17:26 |
| edmondsw | patch set 5 | 17:26 |
| johnthetubaguy | edmondsw: the problem is how do we keep our upgrade promise, that the old policy file works perfectly with the new release | 17:27 |
| *** unicell has joined #openstack-nova | 17:27 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: Log placement result in scheduler https://review.openstack.org/447520 | 17:28 |
| edmondsw | johnthetubaguy you shouldn't be promising that... 99.999% of the time, sure, but there will always be a 0.001% case where... | 17:28 |
| edmondsw | johnthetubaguy with this change, there is a 0.000001% chance that someone would break on upgrade... or something extremely slim like that | 17:28 |
| johnthetubaguy | edmondsw: there are always exceptions, but that is our promise in here: https://governance.openstack.org/tc/reference/tags/assert_supports-upgrade.html#requirements | 17:29 |
| asettle | mriedem cdent - we finally *might* have an answer on the doc patch: https://review.openstack.org/#/c/438328/ | 17:29 |
| *** Jack_Iv has joined #openstack-nova | 17:29 | |
| asettle | Petr has been looking into it. | 17:29 |
| johnthetubaguy | edmondsw: well the additional bit of info I guess is I could policy as configuration | 17:29 |
| asettle | We probably need someone from the nova team to take a look at this too. | 17:29 |
| edmondsw | johnthetubaguy this patch fits fine with that tag | 17:30 |
| cdent | asettle: the Directory thing? hasn't that been mentioned a few times before? | 17:30 |
| *** fragatin_ has quit IRC | 17:30 | |
| edmondsw | it has an upgrade note for operators to consider in the very unlikely event that they need to consider this | 17:30 |
| cdent | asettle: or maybe it was only in related email | 17:30 |
| asettle | cdent: I didn't see it in the comments. Perhaps I am forgetting - if so, sorry! Maybe on email. | 17:30 |
| johnthetubaguy | edmondsw: it violates bullet point 2, now that might be best thing to do, I just want to make sure thats the case | 17:30 |
| asettle | cdent: did we document it? | 17:31 |
| cdent | asettle: sorry, not fully contextualized at the moment, but I remember discussing it in a few different places, including asserting that one person's solution to grant all on '/' was too broad | 17:31 |
| edmondsw | johnthetubaguy that depends on how you read point 2 | 17:32 |
| asettle | cdent: interesting. I've mentioned it to Lana and Brian, since it's nearing the end of my day rapidly. I'll ensure I check in with them again in the morning. If by chance you see it again, please let us know :) | 17:32 |
| cdent | asettle: yeah, will look around some more once I'm done with the current thing | 17:33 |
| asettle | Thank you cdent :) | 17:33 |
| edmondsw | you have to consider policy as config (which I do, but not everyone does) and you have to assume that it means even really unlikely edge cases that nobody would do must still work... even if you've covered that under bullet 5 | 17:33 |
| johnthetubaguy | edmondsw: very true. We had deployers complain they had to tweak config and policy files in sync with upgrades, so we promised to avoid that, hence that was added in there and the grenade tests kinda built around that assumption. | 17:33 |
| edmondsw | sure, I'm fully on board with that | 17:34 |
| *** Jack_Iv has quit IRC | 17:34 | |
| edmondsw | the main point here is that this is not going to break anyone. We're fixing a bug here. The chances of someone relying on this bug for things to work are miniscule | 17:35 |
| *** jaosorior has quit IRC | 17:35 | |
| edmondsw | furthermore, it's not possible to fix this bug without any possible chance of an upgrade impact | 17:35 |
| *** annegentle has joined #openstack-nova | 17:36 | |
| edmondsw | johnthetubaguy so do we not fix the bug? That would be much worse for operators | 17:36 |
| *** iceyao has joined #openstack-nova | 17:36 | |
| edmondsw | johnthetubaguy we can't leave a fully-supported API not only doing its own policy check but also relying on the policy check of another API that is deprecated... non-deprecated cannot rely on deprecated | 17:38 |
| johnthetubaguy | edmondsw: as I said above, I am just trying to make sure we are doing the best thing here, you probably are | 17:38 |
| edmondsw | as an operator, I want to restrict policy for the deprecated API more than for the non-deprecated API, and I can't do that unless we fix this | 17:38 |
| edmondsw | cool | 17:38 |
| *** baoli has quit IRC | 17:40 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Read NIC features in libvirt https://review.openstack.org/444415 | 17:41 |
| *** sapcc-bot1 has joined #openstack-nova | 17:42 | |
| *** sapcc-bot has quit IRC | 17:42 | |
| *** baoli has joined #openstack-nova | 17:42 | |
| *** baoli has quit IRC | 17:42 | |
| johnthetubaguy | edmondsw: I should nudge sneti and aunnam about this change, as it will sure help the policy descriptions to have this one merged | 17:43 |
| edmondsw | sneti aunnam talking about https://review.openstack.org/#/c/389314/ | 17:43 |
| *** masber has quit IRC | 17:43 | |
| *** Jack_Iv has joined #openstack-nova | 17:43 | |
| johnthetubaguy | edmondsw: got a nit on the reno, what do you think is best there? | 17:44 |
| *** annegentle has quit IRC | 17:45 | |
| sneti | edmondsw, checking that | 17:45 |
| edmondsw | johnthetubaguy let me see what I can do there | 17:46 |
| *** baoli has joined #openstack-nova | 17:46 | |
| *** masber has joined #openstack-nova | 17:46 | |
| edmondsw | johnthetubaguy fyi, I removed _items to satisfy your earlier comment :) | 17:47 |
| johnthetubaguy | edmondsw: sorry, I was unclear, it was the policy check moving that worried me | 17:47 |
| edmondsw | I could go back to patch set 5 if you prefer | 17:47 |
| johnthetubaguy | edmondsw: but that moves the polcy check into the _details method, which is the bit that I really don't like | 17:48 |
| *** Jack_Iv has quit IRC | 17:48 | |
| *** annegentle has joined #openstack-nova | 17:48 | |
| johnthetubaguy | edmondsw: just wanting to see policy checks at the top of all the API methods, where possible | 17:48 |
| edmondsw | johnthetubaguy what about _details? there is no _details here | 17:49 |
| johnthetubaguy | edmondsw: oops, I mean _items | 17:49 |
| edmondsw | I don't think it makes sense to create the context var in the API method so we can use it for the policy check, then either recreate it in _items or update _items to let it be passed in, when _items isn't really needed anyway | 17:49 |
| edmondsw | easier to just remove it | 17:49 |
| edmondsw | but that's just my opinion... I can do it differently if need be | 17:49 |
| *** hoangcx has quit IRC | 17:51 | |
| johnthetubaguy | edmondsw: agreed with you, its just there are two changes in a single patch there, which isn't ideal | 17:51 |
| edmondsw | sure | 17:51 |
| johnthetubaguy | not blocking on that though, because its simple enough, and I like that change, its just a nit | 17:52 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | 17:56 |
| cdent | alex_xu: assume you are gone now, but for when you return ^ | 17:56 |
| mriedem | EmilienM: looks like the revert patch failed in the ovb-nonha job, timeout in infra https://review.openstack.org/#/c/447514/ | 17:56 |
| edmondsw | johnthetubaguy how does this sound for the reno? The os-volume_attachments APIs no longer check ``os_compute_api:os-volumes`` policy. They do still check ``os_compute_api:os-volumes-attachments`` policy rules. Deployers who have customized policy should confirm that their settings for os-volume_attachments policy checks are sufficient. | 17:58 |
| johnthetubaguy | edmondsw: thats cool, that matches what we do for config then. | 17:58 |
| edmondsw | cool, I'll throw it up | 17:58 |
| *** lpetrut has quit IRC | 17:59 | |
| *** hoangcx has joined #openstack-nova | 18:02 | |
| *** derekh has quit IRC | 18:02 | |
| *** lucasagomes is now known as lucas-afk | 18:02 | |
| *** ralonsoh has quit IRC | 18:02 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: fix os-volume_attachments policy checks https://review.openstack.org/389314 | 18:04 |
| sneti | edmondsw, that's a simple change. so two changes in single patch looks good. may be you can add a comment on why it is removed | 18:04 |
| *** masber has quit IRC | 18:05 | |
| *** cdent has quit IRC | 18:05 | |
| edmondsw | sneti, sure, I'll edit the commit msg | 18:06 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505 | 18:06 |
| *** aarefiev is now known as aarefiev_pto | 18:06 | |
| sneti | edmondsw, thanks! | 18:07 |
| EmilienM | mriedem: right, though it passed our functional tests (called pingtest) | 18:07 |
| *** cdent has joined #openstack-nova | 18:07 | |
| *** slaweq_ has joined #openstack-nova | 18:07 | |
| *** catintheroof has quit IRC | 18:08 | |
| mriedem | claudiub: if you're still around, can you also hit these in newton and mitaka? https://review.openstack.org/#/q/I1d217620e52d45595a3e0e49ed57b4ab33cd1688,n,z | 18:08 |
| *** catintheroof has joined #openstack-nova | 18:08 | |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: fix os-volume_attachments policy checks https://review.openstack.org/389314 | 18:08 |
| edmondsw | johnthetubaguy sneti ^ | 18:09 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | 18:09 |
| *** shaner has quit IRC | 18:09 | |
| *** baoli has quit IRC | 18:10 | |
| *** mtanino has joined #openstack-nova | 18:11 | |
| *** kaisers_ has joined #openstack-nova | 18:11 | |
| *** baoli has joined #openstack-nova | 18:11 | |
| sdague | hmmm... bad log message somewhere - http://logs.openstack.org/90/417590/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/c831c5d/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-03-20_17_10_19_178 ? | 18:11 |
| *** shaner has joined #openstack-nova | 18:15 | |
| *** baoli has quit IRC | 18:16 | |
| *** adrian_otto has joined #openstack-nova | 18:17 | |
| mriedem | i think i know where | 18:17 |
| *** annegentle has quit IRC | 18:17 | |
| *** armax has joined #openstack-nova | 18:18 | |
| mriedem | ha it's actually super obvious | 18:19 |
| mriedem | if thing: | 18:19 |
| mriedem | else: LOG.(something %s, thing.attribute) | 18:20 |
| *** READ10 has quit IRC | 18:21 | |
| mriedem | https://review.openstack.org/#/c/381888/1/nova/compute/manager.py@6641 | 18:21 |
| *** browne has joined #openstack-nova | 18:21 | |
| *** fragatina has joined #openstack-nova | 18:23 | |
| mriedem | sdague: you want to fix it? https://bugs.launchpad.net/nova/+bug/1674449 | 18:23 |
| openstack | Launchpad bug 1674449 in OpenStack Compute (nova) "NoneType logging warning with "Received unexpected event network-vif-unplugged" message" [Medium,Triaged] | 18:23 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Log a warning if there is only one cell when listing instances https://review.openstack.org/440652 | 18:23 |
| sdague | mriedem: sure | 18:23 |
| *** jamielennox|away is now known as jamielennox | 18:24 | |
| sdague | also... http://logs.openstack.org/90/417590/9/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/c831c5d/logs/screen-n-sch.txt.gz#_2017-03-20_17_07_00_573 | 18:24 |
| sdague | why would the compute node be auto disabled in the middle of a tempest run? | 18:24 |
| mriedem | b/c libvirt crashes | 18:24 |
| mriedem | that's an old gate bug for xenial | 18:24 |
| mriedem | the libvirt driver auto-disables the service when the host connection drops | 18:24 |
| mriedem | http://status.openstack.org/elastic-recheck/#1643911 | 18:25 |
| mriedem | http://status.openstack.org/elastic-recheck/#1646779 | 18:25 |
| mriedem | http://status.openstack.org/elastic-recheck/#1638982 | 18:25 |
| *** sneti_ has joined #openstack-nova | 18:25 | |
| *** crushil has quit IRC | 18:26 | |
| *** dharinic is now known as dharinic|lunch | 18:26 | |
| sdague | mriedem: ok, well interesting, the libvirt fail is on a different test where the guests are irrelevant | 18:27 |
| mriedem | but if libvirt fails and disables the only compute service in a single node run... | 18:28 |
| mriedem | it's going to break any test that needs to create a vm right? | 18:28 |
| *** baoli has joined #openstack-nova | 18:28 | |
| sdague | it recovers | 18:28 |
| mriedem | yeah eventually | 18:28 |
| sdague | well, actually pretty quick | 18:28 |
| sdague | I think we're building new guests within 12 seconds | 18:29 |
| sdague | only a single tempest test failed in this case | 18:29 |
| *** mnestratov has quit IRC | 18:32 | |
| mriedem | johnthetubaguy: should we really not expect hostnames on compute services to be unique across cells? | 18:32 |
| *** sneti_ has quit IRC | 18:32 | |
| *** baoli has quit IRC | 18:32 | |
| *** crushil has joined #openstack-nova | 18:34 | |
| johnthetubaguy | mriedem: seems like a bad assumption, given its not the FQDN, its just the hostname, but I might just be paranoid | 18:34 |
| *** sree has joined #openstack-nova | 18:34 | |
| *** Jack_Iv has joined #openstack-nova | 18:35 | |
| johnthetubaguy | mriedem: given we have a rabbit bus per cell, there is no restriction on that, so far, I might be overthinking that | 18:36 |
| *** kaisers_ has quit IRC | 18:36 | |
| johnthetubaguy | mriedem: I just figured if we started over, all those APIs would just use uuids | 18:36 |
| mriedem | i figured the hostname was fqdn | 18:36 |
| *** kaisers_ has joined #openstack-nova | 18:37 | |
| johnthetubaguy | I thought it was conf.host, which isn't usually fqdn | 18:37 |
| mriedem | johnthetubaguy: yeah i don't disagree with starting over, see my latest reply https://review.openstack.org/#/c/447149/1/specs/pike/approved/service-hyper-uuid-in-api.rst@148 | 18:37 |
| *** kaisers_ has quit IRC | 18:37 | |
| mriedem | it is CONF.host | 18:37 |
| *** kaisers_ has joined #openstack-nova | 18:37 | |
| *** baoli has joined #openstack-nova | 18:38 | |
| mriedem | Hostname, FQDN or IP address of this host. Must be valid within AMQP key. | 18:38 |
| mriedem | so i guess it doesn't have to be fqdn | 18:38 |
| *** ociuhandu has quit IRC | 18:38 | |
| mriedem | sdague: you might have input on ^ | 18:39 |
| mriedem | i don't know why the services actions apis are different from the server actions apis in this regard | 18:39 |
| *** sree has quit IRC | 18:39 | |
| mriedem | and why they are PUTs rather than POSTs | 18:39 |
| johnthetubaguy | mriedem: yeah, it does all seem a bit crufty in there | 18:39 |
| *** Jack_Iv has quit IRC | 18:40 | |
| johnthetubaguy | mriedem: I quite like the idea of adding the uuid in the URL in the new micro-version, to make it more normal | 18:40 |
| *** gus has quit IRC | 18:41 | |
| *** kaisers_ has quit IRC | 18:42 | |
| mriedem | i can throw it in the weekly api subteam meeting for discussions | 18:44 |
| *** baoli has quit IRC | 18:44 | |
| *** gus has joined #openstack-nova | 18:45 | |
| *** iceyao has quit IRC | 18:46 | |
| sdague | mriedem: I'm confused, you pointed at https://github.com/openstack/nova/blob/6f66119153a36ab2ee7d63c98418af3a56fd1f1e/nova/compute/manager.py#L6812 for that logging issue | 18:46 |
| sdague | except, that line is printed fine in the logs | 18:47 |
| mriedem | sdague: it's blowing up trying to process the sub vars for the message | 18:47 |
| mriedem | i'm assuming because event is None | 18:47 |
| sdague | well, are you sure? | 18:48 |
| mriedem | although it shouldn't be | 18:48 |
| sdague | Received unexpected event network-vif-unplugged-8ed92137-a81a-4afd-a391-854848cd2916 for instance | 18:48 |
| mriedem | because if it was https://github.com/openstack/nova/blob/6f66119153a36ab2ee7d63c98418af3a56fd1f1e/nova/compute/manager.py#L6867 | 18:48 |
| mriedem | ah yeah | 18:48 |
| mriedem | my mistake | 18:48 |
| mriedem | when i originally looked at it, i thought it was using _event | 18:48 |
| mriedem | https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py#L355 ? | 18:50 |
| sdague | and there are tons of these other places | 18:50 |
| *** crushil has quit IRC | 18:50 | |
| mriedem | https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py#L385 | 18:50 |
| sdague | you think that's blowing up? | 18:50 |
| *** Jack_Iv has joined #openstack-nova | 18:50 | |
| mriedem | "Logged from file processutils.py, line 385" | 18:50 |
| jaypipes | sean-k-mooney, sfinucan, alex_xu, edleafe, Yingxin, moshele: https://review.openstack.org/447643 | 18:51 |
| *** david-lyle_ has joined #openstack-nova | 18:52 | |
| *** david-lyle has quit IRC | 18:52 | |
| mriedem | but yeah they are everywhere https://goo.gl/sPpPN0 | 18:53 |
| *** david-lyle_ is now known as david-lyle | 18:53 | |
| *** eharney has quit IRC | 18:55 | |
| sdague | mriedem: right | 18:55 |
| mriedem | https://github.com/openstack/oslo.log/blob/master/oslo_log/formatters.py#L273 | 18:56 |
| mriedem | so what sets the log record.args? | 18:57 |
| sdague | most of these stack traces include a daemon.py reference | 18:57 |
| sdague | I wonder if it's this change - https://review.openstack.org/#/c/360252/ | 18:57 |
| mriedem | data['args'] = None | 18:58 |
| *** masber has joined #openstack-nova | 18:58 | |
| sdague | so... yes, it would be that change then | 18:59 |
| sdague | it's one that angus seems to have written over the summer, and dims just merged | 18:59 |
| mriedem | started using that version 2 weeks ago https://review.openstack.org/#/c/442067/ | 19:00 |
| sdague | I'm going to see what a revert does there in the run | 19:00 |
| sdague | yeh | 19:00 |
| dims | sdague : https://review.openstack.org/#/c/447334/ | 19:00 |
| dims | does that help? | 19:00 |
| *** armax has quit IRC | 19:01 | |
| sdague | dims: I think so | 19:01 |
| *** lpetrut has joined #openstack-nova | 19:01 | |
| mriedem | message:"TypeError: 'NoneType' object is not iterable" AND message:"Logged from file" AND build_change:"447334" | 19:01 |
| sdague | mriedem: yeh, though mostly at debug, so it wouldn't always show there | 19:02 |
| sdague | but, http://logs.openstack.org/34/447334/1/check/gate-tempest-dsvm-neutron-src-oslo.privsep-ubuntu-xenial/c147be6/logs/screen-n-cpu.txt.gz looks clean | 19:03 |
| *** iceyao has joined #openstack-nova | 19:04 | |
| *** crushil has joined #openstack-nova | 19:05 | |
| openstackgerrit | Eric Brown proposed openstack/python-novaclient master: Remove py34 tox env and pypi classifier https://review.openstack.org/442847 | 19:06 |
| *** Jack_Iv has quit IRC | 19:06 | |
| *** baoli has joined #openstack-nova | 19:07 | |
| *** eharney has joined #openstack-nova | 19:11 | |
| *** sneti_ has joined #openstack-nova | 19:12 | |
| *** wally has joined #openstack-nova | 19:13 | |
| wally | Hello, all, i got some question about the Openstack Ocata nova placement api,. | 19:13 |
| *** dharinic|lunch is now known as dharinic | 19:13 | |
| wally | I tried installed ocata on ubuntu 1604, and also configure the nova placement api, but still not wokring, | 19:14 |
| wally | can anyone help? | 19:14 |
| *** baoli has quit IRC | 19:14 | |
| wally | there is nothing about this on the installation doc | 19:14 |
| jaypipes | wally: please see /topic. It may be best to send an email to the openstack@ mailing list. | 19:15 |
| jaypipes | wally: also "but still not working" doesn't exactly provide us much information :) | 19:15 |
| *** Matias has quit IRC | 19:15 | |
| wally | on the controller node, i install the apt install nova-placement-api | 19:16 |
| wally | and create the placement endpoint, but when i run the nova-status upgrade check | 19:16 |
| *** Swami has joined #openstack-nova | 19:16 | |
| wally | Check: Placement API | Result: Failure | Details: Placement API endpoint not found. | 19:16 |
| jaypipes | wally: did you add the placement endpoint to the Keystone service catalog? | 19:17 |
| wally | i followed this link https://ask.openstack.org/en/question/102256/how-to-configure-placement-service-for-compute-node-on-ocata/ | 19:17 |
| jaypipes | wally: openstack service create --name placement --description "OpenStack Placement" placement | 19:17 |
| wally | yeah i create this as well | 19:17 |
| *** masber has quit IRC | 19:17 | |
| wally | openstack service create --name placement --description "OpenStack Placement" placement | 19:17 |
| wally | dont know what is wrong? | 19:18 |
| jaypipes | wally: and you did the endpoint create steps as well? | 19:18 |
| wally | openstack service create --name placement --description "OpenStack Placement" placement openstack endpoint create --region RegionOne placement public http://controller:8778 openstack endpoint create --region RegionOne placement admin http://controller:8778 openstack endpoint create --region RegionOne placement internal http://controller:8778 | 19:19 |
| wally | is the enpoint good?>> | 19:19 |
| jaypipes | wally: what does `openstack endpoint list` show? | 19:19 |
| wally | one sec | 19:19 |
| jaypipes | wally: also, please read /topic :) | 19:19 |
| wally | how to paste entire log | 19:19 |
| jaypipes | do not paste entire log. | 19:20 |
| jaypipes | use paste.openstack.org | 19:20 |
| *** gyee has joined #openstack-nova | 19:20 | |
| wally | here is it | 19:21 |
| wally | http://paste.openstack.org/show/603483/ | 19:21 |
| wally | @jaypipes, could you please help | 19:21 |
| wally | i followed this doc to do https://ask.openstack.org/en/question/102256/how-to-configure-placement-service-for-compute-node-on-ocata/ | 19:21 |
| mriedem | jroll: what do people use for ironic cli? osc or ironicclient? | 19:22 |
| jroll | mriedem: yes | 19:22 |
| jroll | mriedem: pip install python-ironicclient gets you ironic cli and the osc plugin | 19:23 |
| jroll | we're deprecating the former soon(TM) | 19:23 |
| jaypipes | wally: can you hit http://10.10.10.11:8778 from your compute node? | 19:23 |
| wally | one sec | 19:23 |
| mriedem | jroll: ok | 19:23 |
| wally | what do u mean hit http://10.10.10.11:8778 | 19:24 |
| wally | i can ping 10.10.10.11 | 19:24 |
| *** baoli has joined #openstack-nova | 19:25 | |
| wally | in the compute node, the nova-compute.log show The placement API endpoint not found. Placement is optional in Newton, but required in Ocata. Please enable the placement service before upgrading. | 19:25 |
| wally | and could you please check this http://paste.openstack.org/show/603484/ | 19:26 |
| mriedem | wally: hit == curl request | 19:26 |
| wally | one sec | 19:26 |
| mriedem | if nova.conf is cofigured properly on the compute node, but it's not trying the endpoint again, try restarting n-cpu | 19:27 |
| mriedem | the service | 19:27 |
| mriedem | although it should try everytime, and only log once every 10 minutes if it can't | 19:27 |
| *** Matias has joined #openstack-nova | 19:28 | |
| mriedem | also https://review.openstack.org/#/c/438328/ | 19:28 |
| wally | root@compute1:~# curl http://10.10.10.11:8778 curl: (7) Failed to connect to 10.10.10.11 port 8778: Connection refused | 19:28 |
| wally | not working | 19:28 |
| mriedem | see Petr's last comment in https://review.openstack.org/#/c/438328/ | 19:28 |
| *** tbachman has quit IRC | 19:29 | |
| mriedem | https://bugzilla.redhat.com/show_bug.cgi?id=1430540#c0 | 19:29 |
| openstack | bugzilla.redhat.com bug 1430540 in openstack-nova "[Ocata] openstack-nova-placement-api-15.0.0-1.el7.noarch missing <Directory> parameter" [Unspecified,New] - Assigned to nlevinki | 19:29 |
| wally | so this is a bug? | 19:30 |
| wally | but i use Ubuntu, is different with centos | 19:30 |
| mriedem | i don't know if there are problems with the ubuntu packages | 19:30 |
| mriedem | you could try what's specified in that bug and see if it fixes your issue | 19:31 |
| mriedem | and if so, then yeah it's probably a packaging bug in ubuntu if you're getting the placement code from them | 19:31 |
| wally | but anyone install the Ocata with ubuntu successfully before? | 19:31 |
| mriedem | i'd check the comments backward from the end of that docs patch | 19:32 |
| mriedem | where people are testing out the install guide | 19:32 |
| wally | ...so ... | 19:33 |
| wally | not working on Ubuntu? | 19:33 |
| mriedem | "(2:30:44 PM) mriedem: i don't know if there are problems with the ubuntu packages" | 19:33 |
| openstackgerrit | Roman Dobosz proposed openstack/nova-specs master: FPGA resource use cases https://review.openstack.org/318047 | 19:33 |
| mriedem | this isn't the nova ubuntu packaging channel | 19:33 |
| *** adrian_otto has quit IRC | 19:34 | |
| mriedem | coreycb: do you do ubuntu packaging for nova stuff? | 19:34 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated baremetal CLIs and APIs https://review.openstack.org/447657 | 19:35 |
| mriedem | jroll: just for you ^ | 19:35 |
| wally | yesh i use apt install nova-placement-api | 19:35 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/447505 | 19:35 |
| coreycb | mriedem, hi, yes | 19:36 |
| mriedem | EmilienM: package mirror issue here? http://logs.openstack.org/22/447522/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/b8990d9/console.html | 19:36 |
| mriedem | coreycb: are you getting reports of packaging issues with the placement API in ocata? | 19:36 |
| coreycb | mriedem, no, not yet | 19:36 |
| mriedem | like https://bugzilla.redhat.com/show_bug.cgi?id=1430540#c0 | 19:36 |
| openstack | bugzilla.redhat.com bug 1430540 in openstack-nova "[Ocata] openstack-nova-placement-api-15.0.0-1.el7.noarch missing <Directory> parameter" [Unspecified,New] - Assigned to nlevinki | 19:36 |
| *** kaisers_ has joined #openstack-nova | 19:36 | |
| *** crushil has quit IRC | 19:37 | |
| mriedem | wally: what does your httpd conf look like for the placement service? | 19:37 |
| wally | one set | 19:37 |
| wally | please see here | 19:38 |
| wally | http://paste.openstack.org/show/603485/ | 19:38 |
| wally | root@controller:/etc/apache2/sites-enabled# vim nova-placement-api.conf | 19:38 |
| wally | this is my apache nova-placement-api.conf located here | 19:39 |
| mriedem | coreycb: is that what you'd expect to see? | 19:39 |
| wally | one question, i cannot do curl http://10.10.10.11:8778 on compute node, not working, but it work for other endpoint | 19:40 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies to availability_zone https://review.openstack.org/440836 | 19:40 |
| wally | curl http://10.10.10.11:8778 curl: (7) Failed to connect to 10.10.10.11 port 8778: Connection refused | 19:40 |
| coreycb | wally, mriedem: that looks a little different from the site config we ship with the package | 19:40 |
| *** crushil has joined #openstack-nova | 19:41 | |
| coreycb | wally, mriedem: here's what the package defines, https://git.launchpad.net/~ubuntu-server-dev/ubuntu/+source/nova/tree/debian/nova-placement-api.conf?h=stable/ocata | 19:41 |
| *** iceyao has quit IRC | 19:41 | |
| wally | so i should use this one?> | 19:41 |
| wally | the config, i modified a little bit myself, | 19:41 |
| coreycb | wally, well it's the default shipped by the package so you can use that, or most deployment projects will likely modify that (ie. ansible, puppet, juju etc) | 19:42 |
| wally | ok, let me try to use this default one and try again | 19:42 |
| coreycb | wally, ok | 19:43 |
| cdent | coreycb: as a somewhat related aside: I think the wsgi script should not be in /usr/bin and the Directory statement should not grant on /usr/bin, but whatever the wsgi script dir is. It is pbr that is in the habit of installing the wsgi script in /usr/bin or /usr/local/bin and that's probably bad. | 19:43 |
| *** krtaylor has quit IRC | 19:43 | |
| cdent | wally: make sure you restart your apache2, of course | 19:43 |
| wally | this config is locate root@controller:/etc/apache2/sites-enabled# ls | 19:43 |
| wally | right | 19:43 |
| cdent | brb | 19:44 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to volumes_attachments.py https://review.openstack.org/445252 | 19:44 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in volumes.py https://review.openstack.org/445637 | 19:45 |
| EmilienM | mriedem: not mirror but building the rpm http://logs.openstack.org/22/447522/1/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/b8990d9/console.html#_2017-03-20_19_03_26_926347 | 19:45 |
| EmilienM | mriedem: I'm not sure about that one :/ | 19:45 |
| EmilienM | mriedem: i'm looking into logs now | 19:46 |
| *** smatzek has quit IRC | 19:47 | |
| wally | hi cdent , i have update the confi file could you please see here, | 19:47 |
| wally | http://paste.openstack.org/show/603486/ | 19:47 |
| wally | now i can get some error message when i run root@compute1:~# curl http://controller:8778 from compute node | 19:47 |
| EmilienM | mriedem: http://paste.openstack.org/show/603487/ - I don't know why | 19:48 |
| wally | maybe still the apache config file problem | 19:48 |
| EmilienM | mriedem: i'm poking packagers | 19:48 |
| mriedem | wally: check the placement service logs | 19:48 |
| *** diana_clarke has joined #openstack-nova | 19:48 | |
| wally | i check the nova_placement_error.log | 19:50 |
| wally | please see herer | 19:50 |
| wally | http://paste.openstack.org/show/603488/ | 19:50 |
| *** masber has joined #openstack-nova | 19:50 | |
| *** voelzmo has joined #openstack-nova | 19:51 | |
| wally | thsi is my config file http://paste.openstack.org/show/603489/ | 19:52 |
| mriedem | from signing_dir from nova.conf | 19:52 |
| mriedem | you have signing_dir=/var/cache/nova but signing_dir is deprecated | 19:52 |
| mriedem | and you don't have access to /var/cache/nova anyway | 19:53 |
| mriedem | s/you/the user running the placement service/ | 19:53 |
| * cdent gets dinner | 19:53 | |
| wally | is this igning_dir=/var/cache/nova in the nova.conf | 19:53 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 19:54 |
| *** kfarr has quit IRC | 19:55 | |
| *** tbachman has joined #openstack-nova | 19:55 | |
| *** voelzmo has quit IRC | 19:56 | |
| coreycb | cdent, that's a good point, thanks. we'll take a look at changing that. | 19:56 |
| *** kfarr has joined #openstack-nova | 19:56 | |
| *** gjayavelu has joined #openstack-nova | 19:58 | |
| rfolco | cdent, can you please see if all good for another +2 on https://review.openstack.org/#/c/416669/ please ? | 19:59 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add descripiton to policies in virtual_interfaces.py https://review.openstack.org/445652 | 19:59 |
| *** baoli has quit IRC | 20:01 | |
| *** baoli has joined #openstack-nova | 20:01 | |
| EmilienM | mriedem: nova Ron broke in RDO because a new rep related to tooz, we are working on it | 20:01 |
| wally | now i got this error root@compute1:~# curl http://controller:8778 {"error": {"message": "The request you have made requires authentication.", "code": 401, "title": "Unauthorized"}}root@compute1:~# | 20:01 |
| cfriesen | Hi cores...https://review.openstack.org/#/c/265282/ (spec for flavor information in the instance details) has one +2...anyone up for giving it another? I'm happy to fix johnthetubaguy's nit if that pushes it over. :) | 20:02 |
| EmilienM | Sorry, nova I'm RDO ( on my phone) | 20:02 |
| wally | 2017-03-20 13:01:37.065 25869 WARNING keystonemiddleware.auth_token [-] AuthToken middleware is set with keystone_authtoken.service_token_roles_required set to False. This is backwards compatible but deprecated behaviour. Please set this to True. | 20:02 |
| *** baoli has quit IRC | 20:02 | |
| wally | where to set this | 20:02 |
| *** Jack_Iv has joined #openstack-nova | 20:03 | |
| *** satyar has quit IRC | 20:04 | |
| efried | claudiub If you're still in a fixing-holes-in-mock mood: https://github.com/testing-cabal/mock/issues/398 | 20:04 |
| mriedem | rfolco: it's on my list | 20:04 |
| mriedem | cfriesen: on my list too | 20:05 |
| rfolco | mriedem, thanks | 20:05 |
| *** Jack_Iv has quit IRC | 20:07 | |
| wally | hi mriedem i getting this erro now | 20:09 |
| wally | root@compute1:~# curl http://controller:8778 {"error": {"message": "The request you have made requires authentication.", "code": 401, "title": "Unauthorized"}} | 20:09 |
| wally | any idea? | 20:09 |
| *** kenperkins has quit IRC | 20:11 | |
| *** lpetrut has quit IRC | 20:12 | |
| *** baoli has joined #openstack-nova | 20:12 | |
| *** masber has quit IRC | 20:13 | |
| *** dave-mcc_ has joined #openstack-nova | 20:14 | |
| *** kencjohnston_ has joined #openstack-nova | 20:15 | |
| *** abramley_ has joined #openstack-nova | 20:15 | |
| mriedem | wally: pass a token | 20:16 |
| *** inara has joined #openstack-nova | 20:16 | |
| wally | how | 20:16 |
| *** cdent_ has joined #openstack-nova | 20:16 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 20:16 |
| wally | from compute node? | 20:16 |
| mriedem | wally: https://developer.openstack.org/api-guide/quick-start/api-quick-start.html#openstack-api-quick-guide | 20:16 |
| *** adrian_otto has joined #openstack-nova | 20:17 | |
| *** Sukhdev_ has joined #openstack-nova | 20:17 | |
| mriedem | well, the nova-compute service will be using keystone, and use the credentials for the placement service user setup in the [placement] section of nova.conf | 20:17 |
| mriedem | you can test manually though with curl | 20:17 |
| *** purplerbot_ has joined #openstack-nova | 20:17 | |
| *** purplerbot_ is now known as purplerbot__ | 20:17 | |
| mriedem | OS_TOKEN=$(openstack token issue -f value -c id) | 20:17 |
| *** purplerbot__ is now known as purplerbot___ | 20:17 | |
| *** purplerbot___ is now known as purplerbot____ | 20:17 | |
| *** purplerbot____ is now known as purplerbot_____ | 20:17 | |
| *** purplerbot_____ is now known as purplerbot______ | 20:17 | |
| *** zhenguo_ has joined #openstack-nova | 20:18 | |
| mriedem | curl -s -H "X-Auth-Token: $OS_TOKEN" http://controller:8778/ | 20:18 |
| *** winston-2 has joined #openstack-nova | 20:18 | |
| *** mdrabe_ has joined #openstack-nova | 20:18 | |
| *** lucas-afk has quit IRC | 20:18 | |
| *** ijw has quit IRC | 20:18 | |
| *** Yingxin_ has joined #openstack-nova | 20:20 | |
| *** esmiurium has joined #openstack-nova | 20:20 | |
| wally | what can i get the $OS_TOKEN? | 20:20 |
| wally | how can i get the $OS_TOKEN | 20:20 |
| *** jamesdenton has quit IRC | 20:20 | |
| *** vsaienk01 has joined #openstack-nova | 20:20 | |
| *** McNinja_ has joined #openstack-nova | 20:21 | |
| *** aloga_ has joined #openstack-nova | 20:21 | |
| claudiub | efried: hm, that's rather surprising. the mock.patch stuff with autospec is kinda' broken, which is probably why it doesn't have the assert_called_with / assert_not_called methods. it creates NonCallableMagicMock objects, which are not callable, so it would make sense it doesn't have those assert methods. | 20:21 |
| *** lucasagomes has joined #openstack-nova | 20:21 | |
| wally | [placement] auth_uri = http://controller:5000 auth_url = http://controller:35357 memcached_servers = controller:11211 auth_type = password project_domain_name = default user_domain_name = default project_name = service username = placement password = conalab os_region_name = RegionOne | 20:21 |
| *** krtaylor has joined #openstack-nova | 20:21 | |
| *** BobBall_ has joined #openstack-nova | 20:21 | |
| wally | this is my [placement] in nova.conf | 20:21 |
| *** mirrorbo1 has joined #openstack-nova | 20:22 | |
| wally | is this correct | 20:22 |
| *** toabctl_ has joined #openstack-nova | 20:22 | |
| *** DuncanT_ has joined #openstack-nova | 20:22 | |
| claudiub | efried: i've addressed the mock.patch issue here: https://github.com/testing-cabal/mock/pull/397 . it might also solve the issue you linked. | 20:22 |
| efried | claudiub Looking.. | 20:22 |
| *** Tahvok_ has joined #openstack-nova | 20:23 | |
| *** amrith has quit IRC | 20:23 | |
| *** MasterofJOKers_ has joined #openstack-nova | 20:23 | |
| *** amrith has joined #openstack-nova | 20:23 | |
| *** baoli has quit IRC | 20:23 | |
| *** Gorian- has joined #openstack-nova | 20:23 | |
| *** lpetrut has joined #openstack-nova | 20:23 | |
| mriedem | wally: project_domain_name = default user_domain_name = default | 20:24 |
| mriedem | i think default is supposed to be Default | 20:24 |
| mriedem | https://review.openstack.org/#/c/438328/13/doc/install-guide/source/nova-compute-install.rst | 20:24 |
| *** lunarlamp has joined #openstack-nova | 20:24 | |
| mriedem | auth_url = http://controller:35357/v3 | 20:24 |
| mriedem | you don't have the v3 suffix | 20:24 |
| wally | ok let me try | 20:24 |
| mriedem | you will have to restart the nova-compute process after making those changes | 20:25 |
| wally | yeah | 20:25 |
| efried | claudiub That one might entail a serious behavior change | 20:25 |
| *** Sukhdev__ has joined #openstack-nova | 20:25 | |
| claudiub | true, but only if autospec=True, which never really worked anyways. | 20:25 |
| claudiub | or from what i've noticed at least. | 20:26 |
| *** markd_ has joined #openstack-nova | 20:26 | |
| *** armax has joined #openstack-nova | 20:26 | |
| *** s1061123 has joined #openstack-nova | 20:27 | |
| *** jgriffith_ has joined #openstack-nova | 20:27 | |
| wally | in the controller node, nova.conf. do i need to do the same thing | 20:27 |
| mriedem | yes | 20:27 |
| *** purplerbot______ has quit IRC | 20:28 | |
| *** Jack_Iv has joined #openstack-nova | 20:29 | |
| *** cdent has quit IRC | 20:29 | |
| *** Sukhdev has quit IRC | 20:29 | |
| *** tblakes has quit IRC | 20:29 | |
| *** mdrabe has quit IRC | 20:29 | |
| *** zul has quit IRC | 20:29 | |
| *** dave-mccowan has quit IRC | 20:29 | |
| *** mvk has quit IRC | 20:29 | |
| *** macsz_and_cheese has quit IRC | 20:29 | |
| *** mirrorbox has quit IRC | 20:29 | |
| *** Yingxin has quit IRC | 20:29 | |
| *** sdague has quit IRC | 20:29 | |
| *** vsaienk0 has quit IRC | 20:29 | |
| *** DuncanT has quit IRC | 20:29 | |
| *** MasterofJOKers has quit IRC | 20:29 | |
| *** purplerbot has quit IRC | 20:29 | |
| *** BobBall has quit IRC | 20:29 | |
| *** aloga has quit IRC | 20:29 | |
| *** Gorian has quit IRC | 20:29 | |
| *** zhenguo has quit IRC | 20:29 | |
| *** mdavidson has quit IRC | 20:29 | |
| *** McNinja has quit IRC | 20:29 | |
| *** tohayash has quit IRC | 20:29 | |
| *** inara` has quit IRC | 20:29 | |
| *** jgriffith has quit IRC | 20:29 | |
| *** toabctl has quit IRC | 20:29 | |
| *** esmiurium_ has quit IRC | 20:29 | |
| *** kencjohnston has quit IRC | 20:29 | |
| *** abramley has quit IRC | 20:29 | |
| *** mariusv has quit IRC | 20:29 | |
| *** Tahvok has quit IRC | 20:29 | |
| *** winston-d has quit IRC | 20:29 | |
| *** Tahvok_ is now known as Tahvok | 20:29 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 20:29 | |
| *** McNinja_ is now known as McNinja | 20:29 | |
| *** Yingxin_ is now known as Yingxin | 20:29 | |
| *** abramley_ is now known as abramley | 20:29 | |
| *** lunarlamp is now known as mariusv | 20:29 | |
| *** cdent_ is now known as cdent | 20:29 | |
| *** toabctl_ is now known as toabctl | 20:29 | |
| *** macsz_and_cheese has joined #openstack-nova | 20:29 | |
| *** Sukhdev__ is now known as Sukhdev | 20:29 | |
| *** mvk has joined #openstack-nova | 20:29 | |
| *** purplerbot has joined #openstack-nova | 20:29 | |
| *** zhenguo_ is now known as zhenguo | 20:30 | |
| *** jgriffith_ is now known as jgriffith | 20:30 | |
| *** zul has joined #openstack-nova | 20:31 | |
| *** DuncanT_ is now known as DuncanT | 20:32 | |
| *** dave-mcc_ is now known as dave-mccowan | 20:33 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119 | 20:34 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | 20:34 |
| efried | adreznec esberglu ^^ ready for in-house re-+1 | 20:35 |
| *** felipemonteiro_ has quit IRC | 20:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated tenant network APIs https://review.openstack.org/447678 | 20:36 |
| *** sdague has joined #openstack-nova | 20:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated tenant network APIs https://review.openstack.org/447678 | 20:37 |
| *** sdague has quit IRC | 20:37 | |
| *** baoli has joined #openstack-nova | 20:37 | |
| *** Gorian- is now known as Gorian | 20:41 | |
| *** kaisers_ has quit IRC | 20:43 | |
| *** tblakes has joined #openstack-nova | 20:43 | |
| *** sdague has joined #openstack-nova | 20:45 | |
| *** ZZelle has joined #openstack-nova | 20:46 | |
| *** kaisers_ has joined #openstack-nova | 20:47 | |
| *** kaisers_ has quit IRC | 20:48 | |
| *** owalsh has quit IRC | 20:50 | |
| *** owalsh has joined #openstack-nova | 20:50 | |
| *** unicell1 has joined #openstack-nova | 20:51 | |
| *** sdague has quit IRC | 20:51 | |
| wally | hi mriedem, i still got the error "message": "The request you have made requires authentication.", "code": 401, "title": "Unauthorized"}} | 20:51 |
| wally | after i change the nova.conf. | 20:51 |
| *** cleong has quit IRC | 20:51 | |
| wally | i used ubunt 1604 desktop, should I use Ubuntu 1604 server? | 20:51 |
| wally | is there any different? | 20:51 |
| mriedem | what roles does your placement user have? | 20:51 |
| *** alineb has joined #openstack-nova | 20:52 | |
| wally | openstack role add --project service --user placement admin | 20:52 |
| cdent | wally: are you sending an x-auth-token header? | 20:52 |
| *** sdague__ has joined #openstack-nova | 20:52 | |
| wally | not sure, how to check sending an x-auth-token header? | 20:52 |
| *** unicell has quit IRC | 20:52 | |
| cdent | is that 401 error happening when you do curl? or is it showing up somewhere else? | 20:53 |
| wally | when i do the curl on the compute node | 20:53 |
| wally | only for this endpoint, all the other endpoint are good | 20:54 |
| openstackgerrit | Diana Clarke proposed openstack/nova master: Fix rest_api_version_history (2.40) https://review.openstack.org/447688 | 20:54 |
| mriedem | wally: what does your curl request look like? | 20:54 |
| mriedem | if you aren't specifying the x-auth-token header, then you're not sending a token | 20:54 |
| cdent | does that curl statement include the phrase "x-auth-token", a bit like `curl -H "x-auth-token: <token value>" http://<the rest> | 20:54 |
| wally | root@compute1:~# curl http://controller:8778 {"error": {"message": "The request you have made requires authentication.", "code": 401, "title": "Unauthorized"}}root@compute1:~# root@compute1:~# | 20:54 |
| mriedem | OS_TOKEN=$(openstack token issue -f value -c id) | 20:54 |
| cdent | yeah, what mriedem just said | 20:55 |
| mriedem | curl -s -H "X-Auth-Token: $OS_TOKEN" http://controller:8778/ | 20:55 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | 20:55 |
| efried | adreznec esberglu ^ | 20:55 |
| mriedem | wally: you aren't sending a token | 20:55 |
| *** ijw has joined #openstack-nova | 20:55 | |
| esberglu | efried: ack | 20:55 |
| mriedem | wally: create an rc file using the same credentials that you have in the [placement] section of nova.conf, source that rc file, then run "OS_TOKEN=$(openstack token issue -f value -c id)" | 20:55 |
| mriedem | to get the token | 20:55 |
| mriedem | and then: curl -s -H "X-Auth-Token: $OS_TOKEN" http://controller:8778/ | 20:55 |
| wally | ok let me try again | 20:56 |
| *** ijw has quit IRC | 20:56 | |
| *** ijw has joined #openstack-nova | 20:56 | |
| wally | root@controller:/var/log/apache2# curl -s -H "X-Auth-Token: $OS_TOKEN" http://controller:8778/ {"versions": [{"min_version": "1.0", "max_version": "1.4", "id": "v1.0"}]} | 20:56 |
| wally | yeah, now is that correct? | 20:56 |
| wally | is the correct respone | 20:56 |
| mriedem | yes | 20:57 |
| mriedem | f yes | 20:57 |
| wally | so mean my placement api is working? | 20:57 |
| mriedem | yes | 20:57 |
| cdent | nice teamwork wally and mriedem | 20:58 |
| wally | but then why i ran this command root@controller:/var/log/apache2# nova-status upgrade check | 20:58 |
| wally | still show this http://paste.openstack.org/show/603499/ | 20:59 |
| wally | could you please check | 20:59 |
| mriedem | that's reading from nova.conf and using keystoneauth | 20:59 |
| wally | must have somehing not config right | 20:59 |
| wally | OH | 20:59 |
| wally | so how to fix this keystoneauth | 21:00 |
| wally | in nova.conf | 21:00 |
| *** kfarr has quit IRC | 21:00 | |
| wally | [keystone_authtoken] auth_uri = http://controller:5000 auth_url = http://controller:35357 memcached_servers = controller:11211 auth_type = password project_domain_name = default user_domain_name = default project_name = service username = nova password = conalab | 21:00 |
| wally | anything wrong? | 21:00 |
| mriedem | well, it's just the credentials from the [placement] section in nova.conf | 21:00 |
| wally | [placement] auth_uri = http://controller:5000 auth_url = http://controller:35357/v3 memcached_servers = controller:11211 auth_type = password project_domain_name = Default user_domain_name = Default project_name = service username = placement password = conalab os_region_name = RegionOne | 21:00 |
| wally | so there much be something wrong with my [placement] | 21:01 |
| wally | do i need to add : $OS_TOKEN" | 21:01 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated floating IP DNS domain/entry APIs https://review.openstack.org/447690 | 21:01 |
| wally | this is my controller node nova.conf | 21:01 |
| mriedem | wally: you have to pass a token with the curl command | 21:02 |
| mriedem | oops nvm | 21:02 |
| mriedem | was reading something old | 21:02 |
| wally | in the curl, it working, but when i run nova-status upgrade check, not working | 21:02 |
| mriedem | right, what's your nova.conf look like? | 21:03 |
| wally | mean something woring with nova.conf right? | 21:03 |
| mriedem | yes | 21:03 |
| *** READ10 has joined #openstack-nova | 21:03 | |
| wally | in the [placement] right | 21:03 |
| wally | can u please check this http://paste.openstack.org/show/603505/ | 21:04 |
| wally | how can i pass the token in the nova.conf | 21:04 |
| mriedem | the request goes through the keystoneauth library, | 21:04 |
| mriedem | provided the credentials in nova.conf, | 21:05 |
| mriedem | ksa generates the token | 21:05 |
| mriedem | compare with nova.conf from a CI run | 21:05 |
| mriedem | http://logs.openstack.org/77/447577/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f6ccfe1/logs/etc/nova/nova.conf.txt.gz | 21:05 |
| wally | u maen company my nova.conf with your nova.conf | 21:06 |
| wally | compare my nova.conf with your nova.conf | 21:06 |
| *** dimtruck is now known as zz_dimtruck | 21:07 | |
| mriedem | wally: what's the OS_AUTH_URL value in your localrc? | 21:07 |
| *** felipemonteiro_ has joined #openstack-nova | 21:08 | |
| wally | you mean the admin.rc | 21:08 |
| wally | export OS_PROJECT_DOMAIN_NAME=default export OS_USER_DOMAIN_NAME=default export OS_PROJECT_NAME=admin export OS_USERNAME=admin export OS_PASSWORD=conalab export OS_AUTH_URL=http://controller:35357/v3 export OS_IDENTITY_API_VERSION=3 export OS_IMAGE_API_VERSION=2 | 21:08 |
| wally | this is my admin-openrc.sh | 21:08 |
| *** baoli has quit IRC | 21:09 | |
| mriedem | i think you can remove the auth_uri option, that's deprecated | 21:09 |
| *** Swami has quit IRC | 21:09 | |
| wally | u mean in the nova.conf [placement]? | 21:10 |
| mriedem | right | 21:10 |
| *** baoli has joined #openstack-nova | 21:11 | |
| *** baoli has quit IRC | 21:11 | |
| mriedem | it looks like auth_uri is only set in the [keystone_authtoken] section | 21:11 |
| mriedem | what does your [keystone_authtoken] section look like? | 21:11 |
| *** baoli has joined #openstack-nova | 21:11 | |
| wally | [keystone_authtoken] auth_uri = http://controller:5000 auth_url = http://controller:35357 memcached_servers = controller:11211 auth_type = password project_domain_name = default user_domain_name = default project_name = service username = nova password = conalab | 21:11 |
| wally | this is the keysotne-auth | 21:11 |
| mriedem | are you using keystone v3? | 21:12 |
| *** eharney has quit IRC | 21:12 | |
| wally | i think so | 21:12 |
| mriedem | devstack sets this up like | 21:12 |
| mriedem | auth_uri = https://15.184.65.44/identity | 21:12 |
| mriedem | auth_url = https://15.184.65.44/identity_admin | 21:12 |
| *** crushil has quit IRC | 21:12 | |
| wally | in where? | 21:13 |
| wally | [placement]? | 21:13 |
| mriedem | in [keystone_authtoken] | 21:13 |
| mriedem | see http://logs.openstack.org/77/447577/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f6ccfe1/logs/etc/nova/nova.conf.txt.gz | 21:13 |
| wally | i see, but i am not sure devstack, | 21:14 |
| *** mdrabe_ is now known as mdrabe | 21:14 | |
| wally | not use devstack | 21:14 |
| wally | so i should change to this? | 21:14 |
| mriedem | i realize, | 21:15 |
| mriedem | i'm trying to show an example of a config that is known to work | 21:15 |
| mriedem | so you can compare to yours | 21:15 |
| wally | please | 21:15 |
| mriedem | which is ^ | 21:15 |
| wally | oh | 21:15 |
| wally | i see | 21:15 |
| wally | let me try yours | 21:15 |
| mriedem | it's not "mine", it's from a devstack CI test run | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated floating_ips_bulk API https://review.openstack.org/447695 | 21:17 |
| openstackgerrit | Christopher Brown proposed openstack/nova master: Add lan9118 as valid nic for hw_vif_model property for qemu https://review.openstack.org/393489 | 21:18 |
| openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in attach_interfaces.py https://review.openstack.org/447696 | 21:18 |
| *** Jack_Iv has quit IRC | 21:19 | |
| wally | -------------------------------------------------------------------+ | Check: Placement API | | Result: Success | | Details: None | +----------------- | 21:19 |
| wally | damn | 21:19 |
| wally | works | 21:19 |
| mriedem | ha | 21:19 |
| mriedem | so it was your keystone_authtoken auth_url/auth_uri? | 21:19 |
| wally | i dont know,I have to check | 21:19 |
| wally | ok and now what is this probnlem | 21:20 |
| wally | | Check: Resource Providers | | Result: Warning | | Details: There are no compute resource providers in the Placement | | service but there are 1 compute nodes in the deployment. | | This means no compute nodes are reporting into the | | Placement service and need to be upgraded and/or fixed. | | See | 21:20 |
| wally | resource providers, | 21:20 |
| wally | this is a problem? | 21:20 |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 21:20 |
| mriedem | wally: that is saying your compute service isn't reporting it's data into the placement service yet, | 21:21 |
| mriedem | which is not surprising if the nova.conf on your compute node can't talk to placement yet | 21:21 |
| wally | so ,, that is ok? | 21:21 |
| mriedem | well, you'll want to fix it | 21:21 |
| mriedem | you need to fix the nova.conf on your compute nodes | 21:21 |
| mriedem | and restart the nova-compute process | 21:21 |
| wally | ok i will need to make nova.conf the same as i just did in my controller nova.conf | 21:21 |
| wally | right | 21:21 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | 21:22 |
| *** tblakes has quit IRC | 21:22 | |
| mriedem | correct | 21:22 |
| efried | adreznec esberglu --^ | 21:23 |
| *** amotoki has joined #openstack-nova | 21:23 | |
| *** baoli has quit IRC | 21:24 | |
| *** diana_clarke has quit IRC | 21:27 | |
| openstackgerrit | Sujitha proposed openstack/nova master: Refactor: remove _items() in nova/api/openstack/compute/attach_interfaces.py https://review.openstack.org/447698 | 21:28 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated floating_ip_pools API https://review.openstack.org/447699 | 21:29 |
| *** aysyd has quit IRC | 21:29 | |
| *** baoli has joined #openstack-nova | 21:32 | |
| *** baoli has quit IRC | 21:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove the deprecated fping API https://review.openstack.org/447702 | 21:34 |
| *** Jeffrey4l_ has quit IRC | 21:35 | |
| *** ijw has quit IRC | 21:35 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #2: functional https://review.openstack.org/438598 | 21:36 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: power_on/off and reboot https://review.openstack.org/427380 | 21:37 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #3: TaskFlow https://review.openstack.org/438729 | 21:37 |
| *** Jeffrey4l_ has joined #openstack-nova | 21:38 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated security_group_default_rules APIs https://review.openstack.org/447705 | 21:38 |
| *** zz_dimtruck is now known as dimtruck | 21:42 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated security_group_rules APIs https://review.openstack.org/447707 | 21:44 |
| *** felipemonteiro_ has quit IRC | 21:47 | |
| *** felipemonteiro_ has joined #openstack-nova | 21:49 | |
| *** gouthamr has quit IRC | 21:50 | |
| openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 21:51 |
| mriedem | sdague__: looks like when we deprecated proxy APIs for network resources, we didn't deprecate the /servers/{server_id}/os-security-groups API even though that's a proxy to neutron | 21:51 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 21:51 |
| mriedem | sdague__: i just realized that on https://review.openstack.org/#/c/441730/1 | 21:51 |
| mriedem | am i missing anything? | 21:51 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 21:51 |
| mriedem | if we deprecate that from nova, then to get security groups for an instance, you have to find all ports for the instance by port.device_id, and then from the ports build the list of security group uuids | 21:53 |
| mriedem | alex_xu was looking to do similar for the multinic api | 21:53 |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in used_limits https://review.openstack.org/447710 | 21:54 |
| *** kylek3h has quit IRC | 21:58 | |
| *** baoli has joined #openstack-nova | 22:01 | |
| *** xyang1 has quit IRC | 22:01 | |
| *** esberglu has quit IRC | 22:05 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Remove the Allocation.create() method. https://review.openstack.org/416751 | 22:05 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add check for invalid inventory amounts https://review.openstack.org/428800 | 22:05 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add check for invalid allocation amounts https://review.openstack.org/416752 | 22:05 |
| *** esberglu has joined #openstack-nova | 22:06 | |
| EmilienM | mriedem: do we have any update on the ironic thing? | 22:06 |
| *** baoli has quit IRC | 22:06 | |
| *** ijw has joined #openstack-nova | 22:06 | |
| *** lyarwood has quit IRC | 22:07 | |
| *** kaisers has quit IRC | 22:08 | |
| *** lyarwood has joined #openstack-nova | 22:09 | |
| *** esberglu has quit IRC | 22:10 | |
| *** rfolco has quit IRC | 22:10 | |
| *** dave-mccowan has quit IRC | 22:12 | |
| *** mdrabe has quit IRC | 22:17 | |
| *** kaisers has joined #openstack-nova | 22:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 22:23 |
| openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/destroy #4: full flavor https://review.openstack.org/391288 | 22:23 |
| *** efried has quit IRC | 22:24 | |
| *** edmondsw has quit IRC | 22:26 | |
| *** lpetrut has quit IRC | 22:26 | |
| *** ijw has quit IRC | 22:26 | |
| *** mriedem1 has joined #openstack-nova | 22:36 | |
| mriedem1 | EmilienM: no because the jobs were failing, | 22:36 |
| mriedem1 | so i didn't get the latest logging chnges | 22:36 |
| mriedem1 | *changes | 22:36 |
| *** mriedem has quit IRC | 22:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/446674 | 22:39 |
| EmilienM | mriedem1: ok | 22:39 |
| *** amotoki has quit IRC | 22:41 | |
| *** mriedem1 is now known as mriedem | 22:41 | |
| mriedem | EmilienM: did you figure out the rpm thing? | 22:41 |
| mriedem | can we recheck? | 22:41 |
| EmilienM | mriedem: not yet, see https://review.rdoproject.org/r/5882 | 22:42 |
| EmilienM | it's in our gate | 22:43 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated security_groups APIs https://review.openstack.org/447716 | 22:45 |
| *** krtaylor has quit IRC | 22:45 | |
| *** tbachman has quit IRC | 22:45 | |
| mriedem | oh right | 22:45 |
| mriedem | tooz | 22:45 |
| *** masber has joined #openstack-nova | 22:45 | |
| *** tbachman has joined #openstack-nova | 22:48 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixed_ips APIs https://review.openstack.org/447718 | 22:49 |
| *** mtanino has quit IRC | 22:53 | |
| *** kenperkins has joined #openstack-nova | 22:56 | |
| *** mlavalle has quit IRC | 22:56 | |
| *** esberglu has joined #openstack-nova | 22:56 | |
| *** kaisers has quit IRC | 22:57 | |
| *** marst has quit IRC | 22:58 | |
| *** esberglu has quit IRC | 23:01 | |
| openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in suspend_server.py https://review.openstack.org/447722 | 23:06 |
| *** marst has joined #openstack-nova | 23:10 | |
| *** kaisers has joined #openstack-nova | 23:12 | |
| *** catintheroof has quit IRC | 23:13 | |
| *** sneti_ has quit IRC | 23:17 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated floating_ips APIs https://review.openstack.org/447724 | 23:19 |
| *** hoangcx has quit IRC | 23:25 | |
| *** Sukhdev has quit IRC | 23:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated floating_ips APIs https://review.openstack.org/447724 | 23:27 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated security_groups APIs https://review.openstack.org/447716 | 23:27 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated fixed_ips APIs https://review.openstack.org/447718 | 23:27 |
| *** masber has quit IRC | 23:32 | |
| *** jamielennox is now known as jamielennox|away | 23:36 | |
| *** cdent has quit IRC | 23:37 | |
| *** jamielennox|away is now known as jamielennox | 23:40 | |
| *** gyee_ has joined #openstack-nova | 23:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Remove deprecated network APIs https://review.openstack.org/447728 | 23:41 |
| *** felipemonteiro_ has quit IRC | 23:42 | |
| *** gyee has quit IRC | 23:43 | |
| *** dave-mccowan has joined #openstack-nova | 23:44 | |
| *** diga has joined #openstack-nova | 23:49 | |
| *** dtp has quit IRC | 23:49 | |
| *** gongysh has joined #openstack-nova | 23:55 | |
| *** liangy has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!