| *** snizar_ has quit IRC | 00:02 | |
| *** snizar has quit IRC | 00:02 | |
| *** Sukhdev has quit IRC | 00:10 | |
| *** Sukhdev has joined #openstack-ironic | 00:12 | |
| *** wajdi has joined #openstack-ironic | 00:17 | |
| *** mtanino has quit IRC | 00:18 | |
| *** hoangcx has joined #openstack-ironic | 00:42 | |
| jroll | devananda: $next | 00:42 |
|---|---|---|
| *** ninguem has joined #openstack-ironic | 00:46 | |
| *** chlong has joined #openstack-ironic | 00:58 | |
| *** baoli has joined #openstack-ironic | 01:11 | |
| *** Sukhdev has quit IRC | 01:16 | |
| *** wajdi has quit IRC | 01:18 | |
| *** wajdi has joined #openstack-ironic | 01:18 | |
| *** wajdi_ has joined #openstack-ironic | 01:22 | |
| *** zhangjl has joined #openstack-ironic | 01:25 | |
| *** wajdi has quit IRC | 01:26 | |
| *** syed_ has quit IRC | 01:32 | |
| *** fragatina has quit IRC | 01:34 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move "server_profile_template_uri" to REQUIRED_ON_PROPERTIES https://review.openstack.org/347127 | 01:37 |
| openstackgerrit | Merged openstack/bifrost: Initialize 'auth' to empty dict on bifrost_configdrives_dynamic https://review.openstack.org/354620 | 01:54 |
| openstackgerrit | Tuan proposed openstack/ironic: Fix incorrect order of params of assertEqual() https://review.openstack.org/355708 | 01:56 |
| *** sabeen has joined #openstack-ironic | 02:10 | |
| *** baoli has quit IRC | 02:10 | |
| *** baoli has joined #openstack-ironic | 02:11 | |
| *** sdake has quit IRC | 02:13 | |
| *** sdake has joined #openstack-ironic | 02:13 | |
| openstackgerrit | Merged openstack/bifrost: Fix release notes formatting issue https://review.openstack.org/355668 | 02:14 |
| openstackgerrit | Merged openstack/bifrost: Introduce support for kvm acceleration https://review.openstack.org/355431 | 02:14 |
| openstackgerrit | Merged openstack/bifrost: Fix typo when querying the python version in scripts/env-setup.sh https://review.openstack.org/355485 | 02:14 |
| *** rama_y has quit IRC | 02:14 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power interface for soft reboot and soft power off https://review.openstack.org/216730 | 02:14 |
| *** aNupoisc has joined #openstack-ironic | 02:17 | |
| *** aNupoisc has quit IRC | 02:22 | |
| *** thrash is now known as thrash|g0ne | 02:29 | |
| *** baoli has quit IRC | 02:39 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Split v1 API root unit and functional tests https://review.openstack.org/348965 | 02:40 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move api driver tests to functional dir https://review.openstack.org/350365 | 02:40 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Prepare for functional tests https://review.openstack.org/349421 | 02:40 |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move root contriller tests to functional dir https://review.openstack.org/356188 | 02:40 |
| *** yuanying has quit IRC | 02:50 | |
| *** fragatina has joined #openstack-ironic | 03:00 | |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Use node uuid rather than id when acquiring node lock https://review.openstack.org/355732 | 03:04 |
| openstackgerrit | Cao ShuFeng proposed openstack/ironic: Use node uuid rather than id when acquiring node lock https://review.openstack.org/355732 | 03:04 |
| *** fragatina has quit IRC | 03:04 | |
| *** wajdi_ has quit IRC | 03:06 | |
| *** wajdi has joined #openstack-ironic | 03:06 | |
| *** wajdi_ has joined #openstack-ironic | 03:08 | |
| *** wajdi has quit IRC | 03:11 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix iLO drivers inconsistent boot mode default value https://review.openstack.org/345231 | 03:19 |
| *** vikrant has joined #openstack-ironic | 03:30 | |
| *** sdake has quit IRC | 03:34 | |
| *** baoli has joined #openstack-ironic | 03:41 | |
| *** wajdi_ has quit IRC | 03:43 | |
| *** baoli has quit IRC | 03:45 | |
| *** yuanying has joined #openstack-ironic | 03:49 | |
| openstackgerrit | Tuan proposed openstack/ironic: Fix incorrect order of params of assertEqual() https://review.openstack.org/355708 | 03:54 |
| *** links has joined #openstack-ironic | 03:54 | |
| *** yuanying has quit IRC | 03:55 | |
| *** vmud213 has joined #openstack-ironic | 03:56 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move root controller tests to functional dir https://review.openstack.org/356188 | 04:02 |
| *** sabeen has quit IRC | 04:12 | |
| *** yuanying has joined #openstack-ironic | 04:16 | |
| *** amotoki has joined #openstack-ironic | 04:38 | |
| *** davidlenwell has quit IRC | 04:42 | |
| *** rama_y has joined #openstack-ironic | 04:48 | |
| *** vmud213 has quit IRC | 04:54 | |
| *** vmud213 has joined #openstack-ironic | 04:54 | |
| *** ChubYann has quit IRC | 04:54 | |
| *** davidlenwell has joined #openstack-ironic | 04:55 | |
| *** tiendc has joined #openstack-ironic | 04:57 | |
| *** ChubYann has joined #openstack-ironic | 05:06 | |
| *** vmud213 has quit IRC | 05:06 | |
| *** vmud213 has joined #openstack-ironic | 05:07 | |
| *** vmud213 has quit IRC | 05:11 | |
| *** vmud213 has joined #openstack-ironic | 05:11 | |
| *** tiendc has quit IRC | 05:21 | |
| *** tiendc has joined #openstack-ironic | 05:22 | |
| *** yuanying_ has joined #openstack-ironic | 05:29 | |
| *** yuanying has quit IRC | 05:29 | |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Replace assertEqual(None, *) with assertIsNone in test_nodestates.py https://review.openstack.org/356231 | 05:31 |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Replace assertEqual(None, *) with assertIsNone in test_nodestates.py https://review.openstack.org/356231 | 05:32 |
| *** rama_y has quit IRC | 05:42 | |
| *** rcernin has joined #openstack-ironic | 05:46 | |
| *** tiendc has quit IRC | 06:00 | |
| *** rbartal has joined #openstack-ironic | 06:01 | |
| *** tiendc has joined #openstack-ironic | 06:03 | |
| *** moshele has joined #openstack-ironic | 06:04 | |
| *** vmud213 has quit IRC | 06:04 | |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Replace assertEqual(None, *) with assertIsNone in test_nodestates.py https://review.openstack.org/356231 | 06:06 |
| *** jhooker has quit IRC | 06:15 | |
| *** jhooker has joined #openstack-ironic | 06:15 | |
| *** Sukhdev has joined #openstack-ironic | 06:20 | |
| *** Lord_Phoenix has joined #openstack-ironic | 06:24 | |
| *** vmud213 has joined #openstack-ironic | 06:27 | |
| *** e0ne has joined #openstack-ironic | 06:30 | |
| *** milan has joined #openstack-ironic | 06:44 | |
| *** Sukhdev has quit IRC | 06:46 | |
| *** amotoki has quit IRC | 06:50 | |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Replace assertEqual(None, *) with assertIsNone in test_nodestates.py https://review.openstack.org/356231 | 06:55 |
| *** tesseract- has joined #openstack-ironic | 07:00 | |
| openstackgerrit | xiexs proposed openstack/pyghmi: Support NMI in bmc https://review.openstack.org/356264 | 07:02 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow to define vms with independent names https://review.openstack.org/354813 | 07:08 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Ipmitool power driver for soft reboot and soft power off https://review.openstack.org/216738 | 07:09 |
| *** chihhsin has quit IRC | 07:10 | |
| openstackgerrit | xiexs proposed openstack/virtualbmc: Adds power diag support to vbmc https://review.openstack.org/347368 | 07:14 |
| openstackgerrit | zhangyanxian proposed openstack/ironic: Replace assertEqual(None, *) with assertIsNone in test_nodestates.py https://review.openstack.org/356231 | 07:18 |
| *** Goneri has joined #openstack-ironic | 07:20 | |
| *** jpich has joined #openstack-ironic | 07:23 | |
| *** athomas has joined #openstack-ironic | 07:24 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: Refactoring code of generating temp_url https://review.openstack.org/352289 | 07:32 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Allow to define vms with independent names https://review.openstack.org/354813 | 07:36 |
| *** chihhsin has joined #openstack-ironic | 07:37 | |
| openstackgerrit | Tang Chen proposed openstack/ironic: Move port api tests to functional dir https://review.openstack.org/356287 | 07:41 |
| *** _vsaienko has joined #openstack-ironic | 07:45 | |
| *** dtantsur|afk is now known as dtantsur | 07:49 | |
| dtantsur | Morning Ironic | 07:50 |
| *** e0ne has quit IRC | 07:51 | |
| tangchen | Morning. :) | 07:55 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** amotoki has joined #openstack-ironic | 08:02 | |
| *** amotoki has quit IRC | 08:02 | |
| *** amotoki has joined #openstack-ironic | 08:03 | |
| *** xuao has joined #openstack-ironic | 08:04 | |
| *** hshiina has joined #openstack-ironic | 08:05 | |
| *** ccamacho has joined #openstack-ironic | 08:08 | |
| *** amotoki_ has joined #openstack-ironic | 08:08 | |
| *** amotoki has quit IRC | 08:08 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:10 |
| *** stendulker has joined #openstack-ironic | 08:13 | |
| *** _vsaienko has quit IRC | 08:15 | |
| *** lucas-dinner is now known as lucasagomes | 08:17 | |
| *** e0ne has joined #openstack-ironic | 08:19 | |
| *** ohamada has joined #openstack-ironic | 08:22 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Clean imports in code https://review.openstack.org/355704 | 08:22 |
| *** priteau has joined #openstack-ironic | 08:24 | |
| *** Jaison has joined #openstack-ironic | 08:35 | |
| *** hoangcx has quit IRC | 08:35 | |
| *** hoangcx has joined #openstack-ironic | 08:36 | |
| *** vmud213 has quit IRC | 08:36 | |
| *** vmud213 has joined #openstack-ironic | 08:37 | |
| *** links has quit IRC | 08:37 | |
| openstackgerrit | Andy Yan proposed openstack/ironic-specs: This blueprint proposes adding XClarity boot driver for Lenovo XClarity Administrator managed Systemx Flex/X6/M5 series servers. Servers, nodes are used interchangeably that denotes Flex/X6/M5 series servers. https://review.openstack.org/355389 | 08:53 |
| *** tiendc has quit IRC | 08:54 | |
| *** derekh has joined #openstack-ironic | 08:57 | |
| tangchen | lucasagomes, dtantsur: Hi guys, more functional test patches have been posted. And I also posted one to move the functional test config out of experimental section. https://review.openstack.org/#/c/356233/ | 08:58 |
| tangchen | lucasagomes, dtantsur: The comments you gave before have been fixed. Please help to review when you are free. :) | 08:59 |
| pas-ha | morning al | 08:59 |
| vmud213 | morning Ironic | 09:01 |
| *** tiendc has joined #openstack-ironic | 09:02 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add node provision state change notification https://review.openstack.org/348437 | 09:04 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 09:05 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 09:11 |
| milan | morning dtantsur lucasagomes pas-ha tangchen vmud213 ! | 09:13 |
| milan | morning Ironic :) | 09:13 |
| *** eset has quit IRC | 09:14 | |
| vmud213 | Morning milan :) | 09:14 |
| lucasagomes | hey all :-) | 09:17 |
| *** caoshufeng has joined #openstack-ironic | 09:19 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Clean imports in code https://review.openstack.org/355704 | 09:19 |
| vmud213 | morning lucasagomes, dtantsur | 09:21 |
| tangchen | morning milan | 09:21 |
| vmud213 | Hello all, I have a queston relateed to typical setup in the production environment w.r.t ironic | 09:22 |
| vmud213 | If i was to use IPMI based drivers i was assuming that the baremetal OOB NIC should be connected to the provider network for initial poweron/poweroff/setting boot modes etc operations. Is that mandatory? | 09:25 |
| *** sambetts|afk is now known as sambetts | 09:26 | |
| sambetts | Morning all | 09:26 |
| vmud213 | Morning sambetts | 09:26 |
| sambetts | vmud213: the OOB NIC just needs to be accessible by the Ironic conductor service, it doesn't need to be on the same network as the provisioning occurs | 09:27 |
| vmud213 | sambetts: aha..thanks | 09:28 |
| vmud213 | sambetts: So do u mean to say, in production it works the same way | 09:28 |
| vmud213 | i mean IPMI and provider networks are same or we use different network for IPMI communication | 09:29 |
| vdrok | morning dtantsur pas-ha milan vmud213 lucasagomes sambetts | 09:30 |
| dtantsur | morning milan, lucasagomes, vdrok, vmud213, sambetts | 09:30 |
| *** hshiina has quit IRC | 09:30 | |
| milan | morning sambetts :) | 09:31 |
| vmud213 | sambetts: So what i now understand, it can be part of same network or different network as long as they can be reachable....I am curious to know how they configure it in production. | 09:31 |
| vmud213 | Morning vdrok :) | 09:31 |
| sambetts | I normal production setup would have two networks, the control plane network where all the OpenStack services run, and a provider network where tenant traffic and ironic provisioning occurs. The OOB NIC should be accesible from the control plane network (whether that be directly in that network, or routable from that network), and the provisioning NIC should be connected to the provider network | 09:31 |
| sambetts | o/ milan lucasagomes dtantsur | 09:35 |
| vmud213 | sambetts: then what about the tenant network? My perception was that, we have a management network with which we reachout all the nodes for management operations, then provider network where all the services in openstack starts up and communicate and finally the tenant networks which come in picture once the instances are up | 09:38 |
| *** electrofelix has joined #openstack-ironic | 09:39 | |
| vmud213 | sambetts: I have searched through cpl of places in the web which only added to my confusion | 09:39 |
| sambetts | vmud213: so I think you have a misunderstanding on a couple of the terms, A provider network in OpenStack is not the control plane where the OpenStack services run, it just refers to a real world network provided to OpenStack for use in neutron | 09:41 |
| caoshufeng | lucasagomes, I hava a question about this: https://review.openstack.org/#/c/355732/4/ironic/conductor/manager.py | 09:43 |
| lucasagomes | caoshufeng, hi there | 09:43 |
| caoshufeng | lucasagomes, http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-07-13.log.html#t2016-07-13T03:02:24 | 09:44 |
| sambetts | vmud213: provider networks are for tenant traffic, these are used as either Neutron flat networks, or as external gateway networks for Neutron virtualise nerworks | 09:44 |
| caoshufeng | lucasagomes, TheJulia says we should log just the parameter. | 09:45 |
| sambetts | vmud213: up until Newton, Ironic has only supported flat neutron networks, and has required that tenant traffic and provisioning traffic travel over the same network | 09:45 |
| vmud213 | sambetts: now i get it. | 09:46 |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/ironic: Clean imports in code https://review.openstack.org/355704 | 09:46 |
| sambetts | vmud213: :) | 09:47 |
| vmud213 | sambetts: u are talking about the network separation patches which are recently merged in upstream? | 09:47 |
| vmud213 | sambetts: thanks | 09:47 |
| sambetts | vmud213: yes, those add tenant network support, which allows provisioning and tenant traffic to go across separate networks | 09:48 |
| sambetts | vmud213: however you need to have switch support | 09:48 |
| sambetts | vmud213: without switch support your still restricted to using a single flat network | 09:49 |
| lucasagomes | caoshufeng, hmm idk, maybe me and TheJulia have different opnions on that | 09:49 |
| lucasagomes | caoshufeng, in many methods we give it the node object as a paramater but just log the UUID | 09:49 |
| lucasagomes | seems that converting the ID to UUID in that function is very similar to that | 09:50 |
| lucasagomes | imo it's just more practical than converting it before calling that method | 09:51 |
| caoshufeng | lucasagomes, I personally agrees with you as you can see my view there. | 09:51 |
| vmud213 | sambetts: Do these patches all got merged upstream? | 09:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Modify the SQL's string from double quote to single quote. https://review.openstack.org/347795 | 09:52 |
| lucasagomes | caoshufeng, right, TheJulia will be online soon-ish we can ask her to comment on the patch | 09:52 |
| vmud213 | sambetts: will check it..no problem.. | 09:52 |
| caoshufeng | lucasagomes, Thanks for teaching me. | 09:52 |
| lucasagomes | caoshufeng, thank you for fixing it | 09:52 |
| lucasagomes | caoshufeng, I've added TheJulia as a reviwer on that patch | 09:53 |
| caoshufeng | lucasagomes, Thanks for your kindness. :) | 09:53 |
| lucasagomes | np, ty | 09:54 |
| sambetts | vmud213: For the base level support all the patches have merged :) | 09:54 |
| *** openstack has joined #openstack-ironic | 10:17 | |
| *** sdake_ has quit IRC | 10:18 | |
| *** e0ne has quit IRC | 10:18 | |
| *** sdake_ has joined #openstack-ironic | 10:19 | |
| *** sdake has quit IRC | 10:19 | |
| *** xuao has quit IRC | 10:24 | |
| *** tiendc has quit IRC | 10:29 | |
| *** milan has quit IRC | 10:32 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 10:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow changing lock purpose on lock upgrade https://review.openstack.org/356064 | 10:37 |
| openstackgerrit | Shivanand Tendulker proposed openstack/proliantutils: RIBCL calls fails with 'requests' library version 2.11.0 https://review.openstack.org/356369 | 10:38 |
| *** hoangcx has quit IRC | 10:58 | |
| *** caoshufeng has quit IRC | 11:03 | |
| openstackgerrit | Tuan proposed openstack/ironic-inspector: Fix order of arguments in assertEqual https://review.openstack.org/356391 | 11:11 |
| *** dtantsur is now known as dtantsur|bbl | 11:11 | |
| *** stendulker has quit IRC | 11:14 | |
| *** lucasagomes is now known as lucas-hungry | 11:23 | |
| *** sdake_ is now known as sdake | 11:24 | |
| *** hshiina has joined #openstack-ironic | 11:27 | |
| *** wajdi has joined #openstack-ironic | 11:27 | |
| *** wajdi_ has joined #openstack-ironic | 11:28 | |
| *** thrash|g0ne is now known as thrash | 11:29 | |
| *** wajdi has quit IRC | 11:31 | |
| *** narasimha_SV has joined #openstack-ironic | 11:32 | |
| narasimha_SV | after executing nova boot command to pxe boot a baremetal | 11:32 |
| narasimha_SV | i am getting this issue | 11:32 |
| narasimha_SV | http://paste.openstack.org/show/559000/ | 11:32 |
| narasimha_SV | is this exactly problem with permissions for ironic folder? | 11:32 |
| mat128 | good morning ironic | 11:44 |
| jroll | morning y'all | 11:45 |
| narasimha_SV | http://paste.openstack.org/show/559000/ | 11:45 |
| sambetts | Morning mat128, jroll | 11:46 |
| mat128 | hey sam | 11:49 |
| * mat128 has a laptop that only does IRC and paste.openstack.org this morning, no emails, no google search, no github, no terminal | 11:50 | |
| mat128 | if you guys are helpful enough I can get through my day | 11:50 |
| mat128 | ;) | 11:50 |
| sambetts | woah, those are some weird limitations :-P | 11:50 |
| mat128 | narasimha_SV: looks like the user running ironic-conductor has no permission on /var/lib/ironic | 11:50 |
| mat128 | sambetts: opened up my laptop to outlook saying it needs to rebuild my profile for some reason | 11:50 |
| mat128 | an iTerm2 update, which I did (it doesnt want to launch anymore) | 11:51 |
| mat128 | and 100% cpu for "ocspd" | 11:51 |
| mat128 | but no google to find out | 11:51 |
| mat128 | guess my mobile phone will be helpful | 11:51 |
| mat128 | or a reboot :) | 11:51 |
| * mat128 just found it it's related to spotlight index being rebuilt... | 11:52 | |
| mat128 | I guess I'm out of disk space and all caches got wiped :( | 11:53 |
| sambetts | oh dear :/ | 11:53 |
| mat128 | says 4.3GB free | 11:53 |
| mat128 | should be fine ;) | 11:53 |
| mat128 | about a DVD | 11:53 |
| mat128 | brb, rebooting | 11:53 |
| *** zhangjl has left #openstack-ironic | 11:54 | |
| *** xuao has joined #openstack-ironic | 11:56 | |
| mat128 | back :) | 11:59 |
| sambetts | \o/ | 11:59 |
| mat128 | found out why almost everything was blocked! | 11:59 |
| mat128 | cpspd is responsible for security / TLS / CRL checking | 11:59 |
| mat128 | so anything https was blocked | 11:59 |
| sambetts | oh man, thats crazy! | 12:00 |
| mat128 | and Cisco SSL VPN didnt want to connect | 12:00 |
| mat128 | ;) | 12:00 |
| * sambetts claims no responsiblity hehe | 12:01 | |
| mat128 | alright, all set! | 12:01 |
| *** sdake_ has joined #openstack-ironic | 12:01 | |
| *** xuao has quit IRC | 12:02 | |
| mat128 | sambetts: just chcked and we are now building tinyipa binaries | 12:02 |
| mat128 | congrats :) | 12:02 |
| sambetts | Yup :D and hopfully those jobs should be reporting success or failure into the health.openstack.org system | 12:03 |
| *** baoli has joined #openstack-ironic | 12:03 | |
| jroll | mat128: this is when you nuke everything and start over :D | 12:04 |
| sambetts | jroll, mat128: http://status.openstack.org/openstack-health/#/job/ironic-python-agent-buildimage-tinyipa !!!! Now we can see if its derping or not | 12:04 |
| jroll | \o/ | 12:05 |
| mat128 | oh sweet | 12:05 |
| mat128 | just found your commit fixing the single quote vs double | 12:05 |
| mat128 | so next build will appear correctly, right? | 12:05 |
| *** sdake has quit IRC | 12:05 | |
| *** sdake_ is now known as sdake | 12:05 | |
| *** mtanino has joined #openstack-ironic | 12:07 | |
| sambetts | mat128: yup, infra where/are working on getting zuul to report the success/failure into health.openstack.org too, so eventually we might not need the subunit data | 12:07 |
| sambetts | mat128: because it doesn't really make that much sense for this job | 12:08 |
| mat128 | yeah | 12:08 |
| *** baoli has quit IRC | 12:08 | |
| *** baoli has joined #openstack-ironic | 12:08 | |
| *** baoli_ has joined #openstack-ironic | 12:09 | |
| *** xuao has joined #openstack-ironic | 12:09 | |
| *** xuao has quit IRC | 12:10 | |
| *** amotoki_ has quit IRC | 12:13 | |
| *** baoli has quit IRC | 12:14 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on https://review.openstack.org/330003 | 12:16 |
| *** dprince has joined #openstack-ironic | 12:16 | |
| *** tiendc has joined #openstack-ironic | 12:16 | |
| *** vmud213 has quit IRC | 12:18 | |
| openstackgerrit | Merged openstack/pyghmi: Support NMI in bmc https://review.openstack.org/356264 | 12:18 |
| openstackgerrit | Merged openstack/pyghmi: Declare strings explicitly as binary https://review.openstack.org/354892 | 12:18 |
| *** tiendc has quit IRC | 12:18 | |
| jroll | JayF: re: rebuild discussion, seems you can't pass in user data on rebuild? http://docs.openstack.org/cli-reference/nova.html#nova-rebuild | 12:19 |
| *** amotoki has joined #openstack-ironic | 12:19 | |
| jroll | maybe the api though, I see personality http://developer.openstack.org/api-ref/compute/?expanded=rebuild-server-rebuild-action-detail#rebuild-server-rebuild-action | 12:20 |
| jroll | oh, that's different than user_data | 12:20 |
| jroll | booooooooooo | 12:20 |
| sambetts | jroll: I'd expect it to pass the same user data that was used to create the instance in the first place? (/me has never actually tried) | 12:22 |
| *** milan has joined #openstack-ironic | 12:24 | |
| *** MattMan has quit IRC | 12:24 | |
| *** MattMan has joined #openstack-ironic | 12:24 | |
| *** lucas-hungry is now known as lucasagomes | 12:26 | |
| lucasagomes | sambetts, didn't try that either | 12:27 |
| *** e0ne has joined #openstack-ironic | 12:28 | |
| *** wajdi_ has quit IRC | 12:29 | |
| *** wajdi has joined #openstack-ironic | 12:29 | |
| mat128 | sambetts, lucasagomes trying with a regular libvirt instance | 12:31 |
| lucasagomes | mat128, cool thanks | 12:31 |
| lucasagomes | mat128, and morning :D | 12:31 |
| sambetts | mat128: :D awesome thanks | 12:31 |
| lucasagomes | devananda, JayF isn't the policy work backward compat? | 12:34 |
| *** wajdi has quit IRC | 12:34 | |
| *** vmud213 has joined #openstack-ironic | 12:36 | |
| jroll | sambetts: we were looking at it yesterday and it seemed like it rebuilds the configdrive (and so we assumed you could pass user data again, since that isn't stored) | 12:36 |
| jroll | lucasagomes: it is, why'd you ask? | 12:36 |
| lucasagomes | jroll, http://paste.openstack.org/show/559012/ | 12:37 |
| openstackgerrit | Merged openstack/proliantutils: RIBCL calls fails with 'requests' library version 2.11.0 https://review.openstack.org/356369 | 12:38 |
| lucasagomes | jroll, it was possible before | 12:38 |
| lucasagomes | default devstack config | 12:38 |
| jroll | lucasagomes: using admin user? | 12:38 |
| jroll | try sourcing openrc again? (wondering if token is expired) | 12:39 |
| * lucasagomes checks | 12:39 | |
| lucasagomes | it may be it | 12:39 |
| jroll | I'm also not sure we made it backwards compat for devstack, we changed how the roles are set up there a bit | 12:39 |
| sambetts | jroll: oh? the user data isn't stored in the Nova DB so its rebuilt into new configdrive? That seems a little backwards if thats the case | 12:39 |
| lucasagomes | jroll, it's the demo admin | 12:39 |
| jroll | sambetts: I believe so but am not 100% sure, could be totally wrong | 12:40 |
| lucasagomes | but oddly enough I can use ironic node-list | 12:40 |
| lucasagomes | demo user* | 12:40 |
| sambetts | jroll: hopfully mat128's experiment well tell us :) | 12:40 |
| jroll | lucasagomes: oh yeah, so we made it so demo user is read only and admin user is read/write | 12:40 |
| jroll | in devstack | 12:40 |
| lucasagomes | jroll, gotcha | 12:40 |
| lucasagomes | I wasn't aware of it | 12:40 |
| lucasagomes | since node-list worked I assumed I was logged as admin | 12:41 |
| jroll | sambetts: heh. I honestly forgot a bunch of the conversation already so maybe don't listen to me :D | 12:41 |
| mat128 | really surprised by the results: http://paste.openstack.org/show/559013/ | 12:41 |
| mat128 | lucasagomes, sambetts, jroll ^ | 12:41 |
| mat128 | tl;dr nova rebuild kept my userdata | 12:42 |
| jroll | mat128: oh, neat | 12:42 |
| mat128 | nah, I expected the opposite | 12:42 |
| mat128 | having users re-submit user-data | 12:42 |
| mat128 | so that means | 12:42 |
| jroll | mat128: is this ironic, or virt? | 12:42 |
| jroll | because... | 12:42 |
| mat128 | nova must keep user-data, but we shuold regenerate configdrive | 12:42 |
| mat128 | virt | 12:42 |
| mat128 | from kilo or something | 12:42 |
| jroll | jay was looking yesterday and saying "we do rebuild wrong" | 12:42 |
| jroll | ah ok | 12:42 |
| jroll | huh. | 12:42 |
| lucasagomes | mat128, cool | 12:42 |
| lucasagomes | that's great | 12:42 |
| mat128 | old private cloud laying around for dev purposes | 12:43 |
| mat128 | rebuild() doesnt seem implemented in libvirt | 12:44 |
| * mat128 is utterly confused then :S | 12:44 | |
| vdrok | morning jroll and mat128 | 12:44 |
| vdrok | jroll: btw we have a bug in rebuild - https://review.openstack.org/#/c/306010/ | 12:45 |
| jroll | mat128: yeah, every other driver, rebuild is just "spawn() on a different host" | 12:45 |
| mat128 | ok, so thinking about it | 12:45 |
| jroll | vdrok: ah yeah, I remember this | 12:45 |
| mat128 | when you rebuild you can choose your own password, or have a new one generated | 12:45 |
| mat128 | so in any case we must regenerate the configdrive | 12:45 |
| mat128 | and I *think* (cant prove it yet) that nova will pass us everything that it kept for that instance (userdata, metadata, etc.) | 12:46 |
| *** rbudden has joined #openstack-ironic | 12:46 | |
| mat128 | vdrok: jlvillal was having this issue yesterday | 12:46 |
| jroll | mat128: yeah, I'd assume that's correct, userdata etc should be in the instance metadata object we use | 12:48 |
| sambetts | thats what I would expect | 12:49 |
| *** mtanino has quit IRC | 12:49 | |
| jroll | I think the only reason we have a rebuild() implementation is to do the preserve_ephemeral thing | 12:49 |
| jroll | which is silly :( | 12:49 |
| *** vmud213 has quit IRC | 12:50 | |
| sambetts | jroll: do we do rebuild wrong in that we redeploy it onto the same node? | 12:50 |
| mat128 | wouldn't we have to regenerate a configdrive? or not having a rebuild() means the nova core will call us with spawn() ? | 12:51 |
| mat128 | with vdrok's patch, that would mean ending up on the same node, which makes sense | 12:51 |
| jroll | sambetts: just that we re-implement it in the driver | 12:51 |
| sambetts | jroll: I was just thinking about what you said above which is that a rebuild for a virt instance moves it to a different host | 12:52 |
| sambetts | jroll: but in our case we just set provision state rebuild on a existing node | 12:53 |
| sambetts | jroll: we don't move to a new node and provision with the same information | 12:53 |
| jroll | sambetts: oh, I think I mis-spoke, I believe rebuild does it on the same host | 12:54 |
| jroll | and vdrok's bug iirc is because it puts it on a different node? (because rebuild only cares about host) | 12:54 |
| * jroll looks again | 12:54 | |
| jroll | and then migrate is "rebuild with different host" | 12:54 |
| sambetts | yup that makes more sense :) | 12:55 |
| vdrok | jroll: yup, the node is changed | 12:55 |
| jroll | vdrok: right, ok. pointed jaypipes at it, it may also be covered by his resource tracker refactor | 12:56 |
| vdrok | thanks | 12:56 |
| *** vikrant has quit IRC | 12:59 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add classes for Portgroups API enablement https://review.openstack.org/347549 | 13:00 |
| lucasagomes | dtantsur|bbl, :-( found a problem when updating the mac of an active instance | 13:04 |
| lucasagomes | https://bugs.launchpad.net/ironic/+bug/1614094 | 13:04 |
| openstack | Launchpad bug 1614094 in Ironic "Updating the port address for an active instance does not regenerate the PXE configuration" [High,New] - Assigned to Lucas Alvares Gomes (lucasagomes) | 13:04 |
| lucasagomes | only affects instance booting from the network tho, not local boot | 13:04 |
| sambetts | lucasagomes: oooo so if you do a reboot of a non-localboot node it fails! | 13:05 |
| lucasagomes | sambetts, yeah :-/ | 13:05 |
| lucasagomes | sambetts, should be a quick fix, just need to regenerate the pxe config | 13:05 |
| * lucasagomes code | 13:05 | |
| sambetts | lucasagomes: yeah, although I kinda think we should probably just call boot_interface.prepare_instance because it depends on the driver | 13:06 |
| lucasagomes | sambetts, good point | 13:07 |
| lucasagomes | will take a look at the code | 13:07 |
| *** tuanluong has joined #openstack-ironic | 13:07 | |
| EmilienM | what's up with ironic ci? | 13:13 |
| EmilienM | sounds like gate-grenade-dsvm-ironic is broken | 13:13 |
| sambetts | EmilienM: status of Ironic CI can be found here https://etherpad.openstack.org/p/IronicWhiteBoard | 13:14 |
| tuanluong | Hello sambetts | 13:15 |
| jroll | jebus, it's still down? | 13:16 |
| * jroll hasn't checked yet this morning | 13:16 | |
| jroll | how many gate bugs can we hit \o/ | 13:16 |
| sambetts | hi tuanluong | 13:16 |
| lucasagomes | jroll, the patch didn't merge yet :-/ | 13:17 |
| tuanluong | I was in many problem when I deploy | 13:17 |
| lucasagomes | gate is very slow | 13:17 |
| jroll | lucasagomes: indeed, third recheck | 13:17 |
| tuanluong | whole disk image. | 13:17 |
| *** sabeen has joined #openstack-ironic | 13:17 | |
| lucasagomes | it's being running for 8 hr 44 min | 13:17 |
| lucasagomes | heh | 13:17 |
| sambetts | jroll, lucasagomes: if you look at zuul the we're maxing out the avaible instances its crazy, I'm not sure whats going on | 13:17 |
| openstackgerrit | Michael Turek proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/332177 | 13:17 |
| *** sean-k-mooney has quit IRC | 13:18 | |
| tuanluong | I am just have been join to study and research in Ironic. | 13:18 |
| jroll | sambetts: yeah, a couple problems: low capacity, high rate of bugs resetting the gate :/ | 13:18 |
| sambetts | :( | 13:18 |
| vdrok | jroll: don't we need this one too? https://review.openstack.org/#/c/355602/ | 13:19 |
| vdrok | to fix grenade | 13:19 |
| *** baoli_ has quit IRC | 13:19 | |
| jroll | vdrok: no, so... | 13:20 |
| *** baoli has joined #openstack-ironic | 13:20 | |
| jroll | vdrok: that was put in and broke ALL of our jobs. it was reverted. that patch is an attempt to get it back in without breaking us | 13:20 |
| vdrok | aha, gotcha | 13:20 |
| jroll | vsaienk0: on your comment here, what's "ironic bootstrap"? https://review.openstack.org/#/c/354075/ | 13:22 |
| *** dtantsur|bbl is now known as dtantsur | 13:24 | |
| dtantsur | lucasagomes, sigh... | 13:24 |
| dtantsur | morning jroll, EmilienM | 13:24 |
| vsaienk0 | jroll: I meant 'IPA ramdisk' there | 13:25 |
| jroll | vsaienk0: ah yeah, going to add IPA upgrades as a follow-up | 13:26 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 13:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/356463 | 13:30 |
| *** narasimha_SV has quit IRC | 13:33 | |
| jroll | sambetts: apparently the major gate issue is corrected, at least, so it should flush out eventually | 13:36 |
| jroll | sdague | osic + trusty + ipv6 was a sad panda | 13:36 |
| sambetts | jroll: ah, well that good :) | 13:36 |
| sambetts | just might take a little while for the queues to push through | 13:37 |
| jroll | indeed, probably all day | 13:37 |
| *** mtanino has joined #openstack-ironic | 13:37 | |
| jroll | sambetts: is there a nova bp for interface attach/detach or just ironic spec? | 13:38 |
| *** jcoufal has joined #openstack-ironic | 13:39 | |
| sambetts | jroll: right now just an Ironic spec, I can write up a nova spec too if required | 13:39 |
| jroll | sambetts: I guess someone put up patches already? | 13:39 |
| sambetts | jroll: ? | 13:40 |
| *** ametts has joined #openstack-ironic | 13:40 | |
| jroll | in nova | 13:40 |
| *** syed_ has joined #openstack-ironic | 13:40 | |
| jroll | sambetts: https://review.openstack.org/#/c/350047/ | 13:41 |
| sambetts | jroll: errr, thats not got anything to do with my spec | 13:41 |
| sambetts | jroll: I've not seen an RFE/BP about this | 13:42 |
| jroll | sambetts: lol, yeah | 13:42 |
| jroll | thought so, just checking | 13:42 |
| *** wajdi has joined #openstack-ironic | 13:43 | |
| *** Jaison has quit IRC | 13:46 | |
| *** sdake has quit IRC | 13:50 | |
| *** wajdi has quit IRC | 13:54 | |
| *** tuanluong has quit IRC | 13:55 | |
| dtantsur | jroll, have your heard something about this almost-broken check-osc-plugins job? | 13:58 |
| dtantsur | I'm inclined to disable it if we don't get fix by eod | 13:58 |
| *** wajdi has joined #openstack-ironic | 13:58 | |
| jroll | dtantsur: I saw your email, haven't looked for responses | 13:59 |
| jroll | dtantsur: I know osic cloud was having lots of problems yesterday, I guess that's fixed, it seems related | 13:59 |
| dtantsur | it was still broken half an hour ago or so | 13:59 |
| dtantsur | ditto for grenade | 13:59 |
| jroll | yeah, grenade fix is in the long gate queue | 14:00 |
| *** sdake_ has joined #openstack-ironic | 14:01 | |
| jroll | dtantsur: they've turned off trusty in osic (which was the problem) so I think any new things that get in the queue should be okay | 14:02 |
| jroll | (assuming it's related, not 100% sure) | 14:02 |
| *** amotoki has quit IRC | 14:05 | |
| dtantsur | sambetts, jroll, do we have timeouts in IPA LLDP collection? | 14:09 |
| jroll | dtantsur: I believe so | 14:09 |
| sambetts | dtantsur: yup | 14:09 |
| dtantsur | people complain that sometimes introspection times out due to too long LLDP collection.. what are the timeouts? | 14:10 |
| sambetts | dtantsur: 30 seconds | 14:10 |
| sambetts | by default | 14:10 |
| dtantsur | hmm, interesting | 14:11 |
| sambetts | dtantsur: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/config.py#L90 | 14:11 |
| dtantsur | it it an overall timeouts for the whole LLDP collection? | 14:11 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introducing node introspection state management https://review.openstack.org/348943 | 14:11 |
| *** xuao has joined #openstack-ironic | 14:12 | |
| *** amotoki has joined #openstack-ironic | 14:12 | |
| dtantsur | unrelated: even though the client gate is half-down, I'd appreciate some reviews on https://review.openstack.org/#/c/328191/ | 14:12 |
| *** moshele has quit IRC | 14:20 | |
| openstackgerrit | Ricardo Carrillo Cruz proposed openstack/bifrost: Specify node_network_info is a dict https://review.openstack.org/356506 | 14:20 |
| sambetts | dtantsur: yup, its an overall timeout, we handle all the LLDP collection at once | 14:22 |
| dtantsur | sambetts, no, it's per interface: https://github.com/openstack/ironic-python-agent/blob/95e1e4e35a150c746de1771e27dfcd162e0be929/ironic_python_agent/netutils.py#L173 | 14:24 |
| sambetts | dtantsur: but they are all started at the same time | 14:24 |
| dtantsur | yeah, but you have: while <smth>: select.select(..., timeout) | 14:24 |
| sambetts | dtantsur: I'm not sure if you;ve run into a bug we posted a long time ago | 14:24 |
| sambetts | dtantsur: https://bugs.launchpad.net/ironic-python-agent/+bug/1533892 https://review.openstack.org/#/c/267537/3/ironic_python_agent/netutils.py | 14:25 |
| openstack | Launchpad bug 1533892 in ironic-python-agent "_get_lldp_info() loops indefinitely" [Medium,In progress] - Assigned to Baodong (Robert) Li (baoli) | 14:25 |
| dtantsur | ouch | 14:25 |
| sambetts | not sure if that might be causing your problem | 14:26 |
| jroll | is that still a bug? | 14:26 |
| dtantsur | jroll, the patch is not merged, sooo... | 14:26 |
| dtantsur | yeah, I suspect we might hit this. I was told something about 15 minutes of LLDP retries | 14:26 |
| dtantsur | wanna me resurrect this patch? | 14:26 |
| jroll | dtantsur: what patch? | 14:26 |
| jroll | oh I'm an idiot | 14:27 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Fix object save after refresh failure https://review.openstack.org/354744 | 14:27 |
| jroll | dtantsur: ++ | 14:27 |
| jroll | just needs a unit test, it seems | 14:27 |
| * dtantsur fixes | 14:27 | |
| * jroll puts the patch in the bug so he doesn't get confused again | 14:27 | |
| sambetts | dtantsur: awesome, that one slipped through the net, I forgot about it until you mentioned that happening | 14:28 |
| sambetts | dtantsur: I hope that solved your issue, otherwise there is a deep problem we need to debug | 14:28 |
| sambetts | deeper* | 14:28 |
| jroll | we can blame josh for that one btw :D | 14:28 |
| *** xuao has quit IRC | 14:32 | |
| *** rloo has joined #openstack-ironic | 14:35 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Build socket list right before select call https://review.openstack.org/267537 | 14:35 |
| dtantsur | sambetts, jroll ^^^ | 14:35 |
| dtantsur | also features a release note :) | 14:36 |
| sambetts | dtantsur: Nice! | 14:36 |
| jroll | woot | 14:36 |
| * dtantsur is glad that our internal folks do some extensive testing of git master | 14:38 | |
| dtantsur | jroll, does it ring bells: http://logs.openstack.org/35/354735/1/check/gate-ironic-python27-db-ubuntu-trusty/6b8e3d5/console.html#_2016-08-17_09_54_17_531880 ? this is mitaka | 14:40 |
| jroll | O_o | 14:40 |
| *** michauds has joined #openstack-ironic | 14:40 | |
| jroll | no, not at all | 14:40 |
| dtantsur | hmmm | 14:41 |
| * jroll looks at the patch | 14:41 | |
| wznoinsk | hi all | 14:42 |
| jroll | dtantsur: problem with the patch, I left a comment | 14:42 |
| dtantsur | fuuuhhh, that's good :) | 14:42 |
| jroll | :D | 14:42 |
| dtantsur | jroll, do you think it's appropriate to modify the backport like that? | 14:43 |
| jroll | dtantsur: yeah, it's just a refactoring, should be fine. maybe leave a note in commit message | 14:43 |
| jroll | the alternative is backporting the entire multitenant feature :) | 14:43 |
| dtantsur | though lucasagomes has found that this patch is incomplete, so we might want to wait for his fix as well | 14:44 |
| lucasagomes | I feel I'm uncovering a big ball of wax looking at the fix there | 14:44 |
| jroll | :/ | 14:44 |
| lucasagomes | tl;dr I'm not take_over works at all | 14:44 |
| lucasagomes | both methods are empty | 14:44 |
| lucasagomes | iscsi_deploy.py and agent.py | 14:45 |
| * lucasagomes was thinking about using take_over to recreate the pxe configuration etc... | 14:45 | |
| jroll | um | 14:46 |
| jroll | where did take_over go?! | 14:46 |
| TheJulia | Well, agent makes sense since whole disk images should have the bits locally to boot independently | 14:47 |
| jroll | it isn't implemented *anywhere*, though | 14:48 |
| TheJulia | Umm deploy.prepare generally has logic for if the instance is already running | 14:48 |
| dtantsur | nova patch has merged \o/ | 14:48 |
| dtantsur | recheck all the things | 14:48 |
| jroll | dtantsur: \o/ | 14:48 |
| lucasagomes | TheJulia, agent supports partition images now too | 14:48 |
| TheJulia | lucasagomes: partition images with bootloaders :) | 14:48 |
| lucasagomes | TheJulia, prepare calls prepare_instance() | 14:48 |
| jroll | lucasagomes: I guess it must all be handled in deploy.prepare now? | 14:48 |
| lucasagomes | which just switch the PXE deploy from "deploy" to "boot_<something> | 14:49 |
| lucasagomes | it does not recreate the PXE configuration | 14:49 |
| lucasagomes | jroll, that's a way, but I think prepare != take_over | 14:49 |
| lucasagomes | take over needs to regenerate the files | 14:49 |
| vdrok | lucasagomes: the PXE configs are not recreated, right, there is a bug on that and I think I saw the patch from Nisha some time ago | 14:50 |
| lucasagomes | vdrok, oh, lemme take a look | 14:50 |
| *** hshiina has quit IRC | 14:50 | |
| lucasagomes | TheJulia, btw, morning | 14:50 |
| vdrok | lucasagomes: https://review.openstack.org/#/c/246033/ this one I guess | 14:50 |
| lucasagomes | TheJulia, also can you take a look at my comment here https://review.openstack.org/#/c/355732/4/ironic/conductor/manager.py ? | 14:51 |
| vdrok | morning TheJulia | 14:51 |
| TheJulia | Good morning, says the very grumpy RV owner | 14:51 |
| jroll | lucasagomes: oh, we aren't writing things into /tftpboot, I see | 14:51 |
| jroll | wtf | 14:51 |
| lucasagomes | jroll, yeah, I think take over is broken atm | 14:51 |
| dtantsur | morning TheJulia | 14:51 |
| jroll | lucasagomes: yeah, lovely! | 14:51 |
| lucasagomes | jroll, I mean, the fact that it's not implemented anywhere ... | 14:51 |
| *** derekh has quit IRC | 14:52 | |
| jroll | right | 14:53 |
| TheJulia | lucasagomes: I feel like I need lots of coffee | 14:53 |
| TheJulia | :) | 14:53 |
| lucasagomes | TheJulia, heh just pointed it to you because I think you talked to the author yesterday about this patch | 14:54 |
| lucasagomes | TheJulia, not yesterday, he actually sent me the logs: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-07-13.log.html#t2016-07-13T03:02:24 | 14:54 |
| lucasagomes | TheJulia, anyway, go get your coffee | 14:54 |
| lucasagomes | chill and then you look at it | 14:54 |
| lucasagomes | (if you have time) | 14:55 |
| * TheJulia swaps context | 14:55 | |
| TheJulia | I have some time, I was expecting to be at the DMV longer.... got turned away | 14:55 |
| TheJulia | Ahh! context! | 14:56 |
| wznoinsk | would anyone have any suggestions how I can expose the ironic-node name (from ironic node-list) into inside the actual node? | 14:57 |
| lucasagomes | vdrok, I think that patch might do the trick for the take over yes | 14:58 |
| vdrok | yup, IIRC I even tested that and it worked | 14:58 |
| *** rama_y has joined #openstack-ironic | 14:58 | |
| lucasagomes | vdrok, I will update it to address the comments | 14:59 |
| lucasagomes | apparently it's been inactive | 14:59 |
| vdrok | thank you lucasagomes :) | 14:59 |
| sambetts | wznoinsk: there is no supported way, I think that would be a layer break | 14:59 |
| *** xuao has joined #openstack-ironic | 15:00 | |
| lucasagomes | wznoinsk, https://review.openstack.org/#/c/238593/ | 15:00 |
| lucasagomes | but nova will not accept it | 15:00 |
| lucasagomes | so... | 15:00 |
| sambetts | ewww XD | 15:01 |
| lucasagomes | :-) | 15:01 |
| JayF | jroll: so nova rebuild, just, it's impossible to have a rebuilt instance with new data? | 15:01 |
| wznoinsk | sambetts: yes, that was also my impression, I was wondering (if someone really wanted to expose these details) thru metadata service maybe? | 15:01 |
| JayF | jroll: I too saw "personality" and thought == user_data | 15:01 |
| jroll | JayF: yeah, it's different | 15:02 |
| sambetts | JayF: wouldn't that be a copy and modify not a rebuild | 15:02 |
| TheJulia | lucasagomes: yeah, move task_manager line 214 up to before the log? :) Thinking that would be best, if the object get supports the id or uuid since the doc string says both are supported | 15:02 |
| sambetts | JayF: e.g. snapshot, and deploy new instance with data | 15:02 |
| JayF | sambetts: huh? | 15:02 |
| mat128 | JayF: nova help rebuild | 15:02 |
| mat128 | JayF: http://paste.openstack.org/show/559013/ | 15:03 |
| JayF | jroll: ^ | 15:03 |
| mat128 | JayF: metadata is kept when rebuilding, but you can change admin password, which means we should regenerate the configdrive | 15:03 |
| jroll | JayF: right, it uses the user_data from the original build | 15:03 |
| JayF | mat128: where does nova store it? | 15:03 |
| *** openstackgerrit has quit IRC | 15:03 | |
| sambetts | JayF: in the DB | 15:04 |
| sambetts | JayF: instance metadata | 15:04 |
| JayF | ah | 15:04 |
| lucasagomes | TheJulia, yeah pretty much | 15:04 |
| JayF | well that makes what we're doing even more broken | 15:04 |
| mat128 | so nova takes care of it for us | 15:04 |
| TheJulia | lucasagomes: commenting on the review | 15:04 |
| *** vikrant has joined #openstack-ironic | 15:04 | |
| lucasagomes | TheJulia, thanks | 15:04 |
| mat128 | we just have to generate based on what is passed to us | 15:04 |
| *** openstackgerrit has joined #openstack-ironic | 15:04 | |
| *** vikrant has quit IRC | 15:04 | |
| JayF | So basically we should start deleting configdrive after a node goes active, and expect a new one to be provided before rebuild | 15:04 |
| mat128 | correct | 15:05 |
| vdrok | mat128: does admin_password work in case of ironic? | 15:05 |
| *** mari0jv has joined #openstack-ironic | 15:07 | |
| mat128 | havent tested it, but if we dont regenerate the configdrive then it's probably the old one | 15:07 |
| jroll | it gets in the configdrive | 15:07 |
| sambetts | wznoinsk: if your instance has access to your OpenStack API services, you could inject a script to get it from Ironic | 15:07 |
| jroll | so it's up to cloud-init or whatever | 15:07 |
| jroll | (not on rebuild, though, apparently) | 15:07 |
| mat128 | vdrok: but the user can pass a new admin_pass | 15:07 |
| mat128 | which is one more argument for rebuilding the configdrive | 15:07 |
| vdrok | yeah, it's in configdrive, but it did not work the last time I tried | 15:07 |
| mat128 | vdrok: it really should | 15:07 |
| JayF | vdrok: making it "work" is the job of a userspace tool in the image, such as cloud-init | 15:07 |
| jroll | vdrok: I don't think cloud-init supports it | 15:07 |
| mat128 | vdrok: same as virtual instances | 15:07 |
| jroll | (yet?) | 15:07 |
| mat128 | jroll: it doesnt? | 15:07 |
| mat128 | O_o | 15:07 |
| mat128 | lemme check | 15:07 |
| jroll | I can't remember | 15:07 |
| jroll | but I feel like not | 15:07 |
| sambetts | jroll: glean does read it either, it only handles ssh keys, hostname and network info | 15:08 |
| mat128 | jroll: a grep for "admin_pass" in cloud-init's codebase returned nothing (other than a method named admin_password) | 15:08 |
| * mat128 is unimpressed | 15:08 | |
| mat128 | what is this field for if nobody implements it? | 15:08 |
| sambetts | something something something future use? | 15:09 |
| sambetts | haha | 15:09 |
| mat128 | hah | 15:09 |
| * mat128 found why I thought it was implemented | 15:09 | |
| jroll | rackspace's nova-agent uses it :P | 15:09 |
| mat128 | we have local downstream patches to read it | 15:09 |
| mat128 | :P | 15:09 |
| *** amotoki has quit IRC | 15:10 | |
| mat128 | cloudbase-init reads it too | 15:10 |
| JayF | our homegrown-windows-cloudinit-thinger reads it | 15:11 |
| *** xuao has quit IRC | 15:11 | |
| JayF | (I don't know if we're using cloudbase-init for it0 | 15:11 |
| sambetts | mat128, jroll: nova doesn't generate a cloud-init compatible cloud-config that uses it does it? https://cloudinit.readthedocs.io/en/latest/topics/examples.html#including-users-and-groups | 15:11 |
| *** dprince has quit IRC | 15:11 | |
| *** xuao has joined #openstack-ironic | 15:11 | |
| JayF | sambetts: nova doesn't generate a cloud-config.yml | 15:11 |
| JayF | sambetts: nova just drops stuff in the various *_data.json files | 15:11 |
| mat128 | sambetts: nova provides meta_data.json | 15:11 |
| mat128 | which includes it | 15:11 |
| mat128 | sambetts: FYI http://paste.openstack.org/show/559056/ | 15:13 |
| mat128 | example meta_data.json | 15:13 |
| sambetts | JayF, mat128: looks like meta.js can use cloud-config keys and is read that way https://cloudinit.readthedocs.io/en/latest/topics/datasources.html?highlight=meta_data#keys-and-values | 15:13 |
| JayF | sambetts: that's only for v1 configdrive, which I haven't seen ... in a long, long, long time | 15:14 |
| sambetts | oooh :/ | 15:14 |
| mat128 | sambetts: a user can provide (via user-data) a cloud-config file to run commands via cloud-init | 15:14 |
| JayF | Either way, admin_password is still a bit of a contract with the instance | 15:15 |
| mat128 | sambetts: https://cloudinit.readthedocs.io/en/latest/topics/examples.html#yaml-examples | 15:15 |
| sambetts | mat128: yeah, thats why I wondered if it used that same mechanism, it seemed to make sense in my head that it would | 15:15 |
| JayF | even if the userspace tools suck | 15:15 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add create and delete notifications for ironic resources https://review.openstack.org/356541 | 15:15 |
| mat128 | JayF: correct, instances should use the provided admin_pass | 15:15 |
| JayF | mat128: I'd say it more like: the provided admin_pass should be in the configdrive | 15:16 |
| mat128 | ^removes tools from the equation | 15:16 |
| mat128 | very good | 15:16 |
| mat128 | https://bugs.launchpad.net/cloud-init/+bug/1236883 | 15:16 |
| openstack | Launchpad bug 1236883 in cloud-init "data source config drive doesn't use the Admin Pass given by OpenStack" [Low,Triaged] | 15:16 |
| mat128 | 2 years old :( | 15:16 |
| sambetts | mat128: :( | 15:17 |
| sambetts | I might submit a similar bug to glean | 15:17 |
| mat128 | small world, Anthony Woods commented on this bug and he was working for us a while ago | 15:17 |
| mat128 | please do | 15:19 |
| mat128 | smoser (cloud-init maintainer) said he's not against an opt-in "read password from metadata/configdrive" | 15:19 |
| mat128 | but he feels this is a security issue | 15:19 |
| mat128 | we should submit a patch | 15:19 |
| JayF | I mean, I think adminpass is kinda dumb too | 15:22 |
| JayF | but it exists, and should work if folks wanna go that way | 15:22 |
| *** ninguem has quit IRC | 15:28 | |
| openstackgerrit | Xu Ao proposed openstack/ironic: Fix the mistakes in Installation Guide https://review.openstack.org/353224 | 15:28 |
| *** thiagop has joined #openstack-ironic | 15:29 | |
| *** xuao has quit IRC | 15:29 | |
| *** hatTip has joined #openstack-ironic | 15:29 | |
| thiagop | Good morning, ironic | 15:29 |
| wznoinsk | sambetts: lucasagomes I think we'll actually set "ipmitool> mc setsysinfo system_name" on the physical node itself and read it in cloud-init to use as a hostname | 15:34 |
| *** Sukhdev has joined #openstack-ironic | 15:35 | |
| *** milan has quit IRC | 15:36 | |
| hatTip | \o good morning | 15:40 |
| *** ninguem has joined #openstack-ironic | 15:43 | |
| *** rama_y has quit IRC | 15:45 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Copy iPXE script over only when needed https://review.openstack.org/356562 | 15:48 |
| pas-ha | lucasagomes: do you plan to work on https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipxe-swift-tempurls.html ? If not I'd like to have a take on that | 15:55 |
| lucasagomes | pas-ha, hey there, please take it over yeah | 15:56 |
| pas-ha | ok, thanks | 15:56 |
| *** amotoki has joined #openstack-ironic | 15:56 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow using TempURLs for deploy images https://review.openstack.org/344160 | 15:58 |
| *** rcernin has quit IRC | 15:58 | |
| pas-ha | there it is :) no unit tests though for now :/ :) | 15:58 |
| *** bodepd has quit IRC | 15:59 | |
| *** dtantsur is now known as dtantsur|brb | 16:00 | |
| devananda | morning, all | 16:00 |
| *** sdake_ is now known as sdake | 16:00 | |
| *** sdake is now known as sdake_ | 16:00 | |
| *** syed_ has quit IRC | 16:02 | |
| *** snizar_ has joined #openstack-ironic | 16:03 | |
| *** snizar has joined #openstack-ironic | 16:03 | |
| vdrok | dtantsur|brb: regarding the last_error check in your --wait patch, it can be cleared after failure, eg the deploy failed and we've done node-set-power-state, maybe worth mentioning somewhere that we rely on last_error to determine that the operation failed? | 16:06 |
| vdrok | morning devananda | 16:07 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Conductor fails to do take_over for PXE drivers https://review.openstack.org/246033 | 16:10 |
| lucasagomes | vdrok, updated the patch ^ | 16:10 |
| vdrok | lucasagomes: thanks, will try to take a look today | 16:10 |
| lucasagomes | no rush, thanks | 16:10 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Mask instance secrets in API responses https://review.openstack.org/326768 | 16:16 |
| devananda | JayF: updated unit tests ^ | 16:16 |
| devananda | if anyone has a few minutes, here's an easy doc review for the policy changes that were released in 6.1.0: https://review.openstack.org/345556 | 16:18 |
| NobodyCam | Good Morning Ironicers | 16:24 |
| *** dprince has joined #openstack-ironic | 16:27 | |
| *** rama_y has joined #openstack-ironic | 16:31 | |
| TheJulia | Good morning NobodyCam, devananda | 16:33 |
| *** aNupoisc has joined #openstack-ironic | 16:33 | |
| NobodyCam | good morning TheJulia | 16:34 |
| NobodyCam | morning devananda :) | 16:34 |
| *** milan has joined #openstack-ironic | 16:34 | |
| *** jpich has quit IRC | 16:35 | |
| *** rbrud has joined #openstack-ironic | 16:44 | |
| devananda | JayF, rloo: on the topic of api ref and docs, we should have something like this: http://developer.openstack.org/api-guide/compute/microversions.html | 16:46 |
| devananda | i'd be tempted to just reference that, if it wasn't specific to nova | 16:46 |
| rloo | devananda: ++ | 16:46 |
| NobodyCam | morning rloo :) | 16:47 |
| rloo | devananda: if you aren't going to do it :), please open a bug so we don't forget. | 16:47 |
| devananda | we don't have a job publishing to /api-guide/ yet, do we? | 16:47 |
| rloo | devananda: oh. is that different from the api-ref stuff? | 16:48 |
| devananda | yep | 16:48 |
| devananda | different doc site | 16:48 |
| rloo | devananda: sigh. JayF ^^, you keeping track of this stuff? | 16:48 |
| *** dtantsur|brb is now known as dtantsur | 16:49 | |
| jroll | O_o | 16:49 |
| dtantsur | morning devananda, NobodyCam | 16:49 |
| NobodyCam | morning dtantsur :) | 16:49 |
| jroll | so this is api guide home, best I can tell http://developer.openstack.org/api-guide/quick-start/ | 16:49 |
| NobodyCam | morning jroll :) | 16:49 |
| jroll | and no clue how to get to 'compute' or whatever from there | 16:50 |
| dtantsur | vdrok, hmm, not sure where though | 16:50 |
| * JayF blinks | 16:50 | |
| JayF | api-guide, api-ref | 16:50 |
| jroll | yeah I'm not sure how relevant the api guide is | 16:50 |
| jroll | but it's used, apparently | 16:51 |
| sambetts | jroll: I thought this is it? http://developer.openstack.org/api-ref.html | 16:51 |
| jroll | sambetts: that's api-ref, which apparently is different than api-guide | 16:51 |
| vdrok | dtantsur: in the description maybe... Anyway I +2d it :) | 16:51 |
| sambetts | oh :/ | 16:51 |
| jroll | sambetts: e.g. https://github.com/openstack/nova/tree/master/api-guide/source vs https://github.com/openstack/nova/tree/master/api-ref/source | 16:52 |
| devananda | api-ref = technical reference; api-guide = more reader-friendly guide | 16:52 |
| devananda | eg, http://developer.openstack.org/api-guide/compute/index.html | 16:52 |
| jroll | devananda: right, I don't see a way to get to that URL from the api-guide start page | 16:52 |
| jroll | ( http://developer.openstack.org/api-guide/quick-start/ ) | 16:52 |
| devananda | navigating hte openstack doc pages is ... complicated | 16:53 |
| devananda | jroll: I'm inclined to just write a page on how we do microversions, post that in our dev docs and link from our api-ref/index.html | 16:53 |
| JayF | +1 | 16:53 |
| jroll | +1, I think that's a good start | 16:53 |
| * sambetts -> home | 16:53 | |
| devananda | jroll: and then work on an api-guide as we work on the api-v-next | 16:53 |
| jroll | sure | 16:54 |
| sambetts | jlvillal: I'm gunna miss the QA meeting, but I've not much to update on the QA meeting, main thing is that IPA post jobs are showing up on status.openstack.org/openstack-health now | 16:55 |
| jlvillal | sambetts: Thanks for the info. Have a good evening | 16:55 |
| sambetts | thanks cya later o/ | 16:55 |
| sambetts | night everyone | 16:56 |
| NobodyCam | night sambetts | 16:56 |
| *** sambetts is now known as sambetts|afk | 16:56 | |
| NobodyCam | morning jlvillal | 16:56 |
| jlvillal | NobodyCam: Good morning :) | 16:57 |
| NobodyCam | :) | 16:57 |
| lucasagomes | NobodyCam, devananda morning | 16:59 |
| NobodyCam | morning lucasagomes :) | 16:59 |
| lucasagomes | I'm calling it a day, have a great evening all :-) | 16:59 |
| *** cdearborn has joined #openstack-ironic | 16:59 | |
| jroll | oh, forgot to say good morning NobodyCam :) | 16:59 |
| jroll | night lucasagomes \o | 16:59 |
| NobodyCam | lol... have a good night lucasagomes | 16:59 |
| NobodyCam | :p | 16:59 |
| NobodyCam | hehehee | 17:00 |
| lucasagomes | :D | 17:00 |
| *** lucasagomes is now known as lucas-dinner | 17:00 | |
| *** tesseract- has quit IRC | 17:00 | |
| *** rpioso has joined #openstack-ironic | 17:00 | |
| *** wajdi_ has joined #openstack-ironic | 17:02 | |
| JayF | NobodyCam: https://review.openstack.org/#/c/345556/ if you have a sec, this could use another core review | 17:02 |
| *** wajdi__ has joined #openstack-ironic | 17:03 | |
| *** aarefiev_ has joined #openstack-ironic | 17:04 | |
| NobodyCam | morning JayF :) will do... prob take me a little bit to get to it | 17:04 |
| *** fragatina has joined #openstack-ironic | 17:05 | |
| *** wajdi has quit IRC | 17:05 | |
| *** e0ne has quit IRC | 17:05 | |
| *** wajdi_ has quit IRC | 17:06 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Update api-ref for v1.22 https://review.openstack.org/353117 | 17:09 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Allow using TempURLs for deploy images https://review.openstack.org/344160 | 17:13 |
| *** anush_ has joined #openstack-ironic | 17:16 | |
| *** Sukhdev has quit IRC | 17:17 | |
| *** amotoki has quit IRC | 17:24 | |
| *** Lord_Phoenix has quit IRC | 17:24 | |
| *** amotoki has joined #openstack-ironic | 17:25 | |
| *** rcernin has joined #openstack-ironic | 17:26 | |
| *** electrofelix has quit IRC | 17:27 | |
| *** amotoki has quit IRC | 17:28 | |
| *** fragatina has quit IRC | 17:30 | |
| *** fragatina has joined #openstack-ironic | 17:31 | |
| *** ninguem has quit IRC | 17:32 | |
| *** aarefiev_ has quit IRC | 17:32 | |
| devananda | jroll: wdyt of removing the autogenerated content from this page http://docs.openstack.org/developer/ironic/webapi/v1.html and replacing it a link tothe api-ref ? | 17:32 |
| devananda | and adding more information about _using_ the API there | 17:33 |
| jroll | devananda: I think you've asked me that before :) | 17:33 |
| jroll | +2 | 17:33 |
| devananda | oh | 17:33 |
| devananda | hah | 17:33 |
| * devananda does it | 17:33 | |
| JayF | +2A | 17:33 |
| jroll | no reason for duplicate info | 17:33 |
| devananda | right | 17:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Configure clean network to provision network https://review.openstack.org/356632 | 17:34 |
| *** ninguem has joined #openstack-ironic | 17:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Configure clean network to provision network https://review.openstack.org/356632 | 17:35 |
| *** priteau has quit IRC | 17:36 | |
| dtantsur | see you tomorrow! | 17:37 |
| *** dtantsur is now known as dtantsur|afk | 17:37 | |
| vdrok | night dtantsur|afk | 17:40 |
| *** rbartal has quit IRC | 17:45 | |
| *** ohamada has quit IRC | 17:46 | |
| jroll | stepping away for a while | 17:46 |
| rloo | jroll: when you get back and if you have a minute, how is ironic status being tracked? see https://review.openstack.org/#/c/355290/4/README.rst | 17:50 |
| *** wajdi__ has quit IRC | 17:50 | |
| *** wajdi has joined #openstack-ironic | 17:51 | |
| *** _vsaienko has joined #openstack-ironic | 17:56 | |
| *** jcoufal has quit IRC | 17:56 | |
| *** rbartal has joined #openstack-ironic | 17:58 | |
| *** jcoufal has joined #openstack-ironic | 17:59 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Update help shown for node-delete https://review.openstack.org/356650 | 18:08 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Configure clean network to provision network https://review.openstack.org/356632 | 18:09 |
| *** _vsaienko has quit IRC | 18:12 | |
| *** _vsaienko has joined #openstack-ironic | 18:12 | |
| *** mtanino has quit IRC | 18:13 | |
| *** ninguem has quit IRC | 18:21 | |
| *** ninguem has joined #openstack-ironic | 18:30 | |
| *** rbudden has quit IRC | 18:31 | |
| *** ametts has quit IRC | 18:45 | |
| * devananda gets carried away and updates a lot of other docs | 18:46 | |
| *** syed_ has joined #openstack-ironic | 18:48 | |
| *** hoonetorg has quit IRC | 18:53 | |
| *** ametts has joined #openstack-ironic | 18:56 | |
| *** _vsaienko has quit IRC | 18:59 | |
| *** athomas has quit IRC | 19:01 | |
| *** ElCoyote_ has joined #openstack-ironic | 19:03 | |
| *** rbudden has joined #openstack-ironic | 19:05 | |
| *** hoonetorg has joined #openstack-ironic | 19:06 | |
| *** Sukhdev has joined #openstack-ironic | 19:07 | |
| *** wznoinsk has quit IRC | 19:07 | |
| *** _vsaienko has joined #openstack-ironic | 19:11 | |
| *** snizar_ has quit IRC | 19:13 | |
| *** snizar has quit IRC | 19:13 | |
| *** hatTip has quit IRC | 19:14 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add 'openstack baremetal node adopt' command https://review.openstack.org/356671 | 19:18 |
| jroll | rloo: heh. I guess the most cohesive source is launchpad. for the rest, release notes / releases.openstack.org | 19:18 |
| rloo | jroll: except if you look at launchpad http://launchpad.net/ironic, under Series and Milestones, it sez 'mitaka seris is the current focus of development' | 19:20 |
| jroll | rloo: right, LP for status on a given feature/bug, releases.openstack.org for the rest | 19:20 |
| rloo | jroll: can you add something there, like after mitaka, look at ...? | 19:20 |
| jroll | or on all features/bugs I guess | 19:21 |
| jroll | maybe | 19:21 |
| *** _vsaienko has quit IRC | 19:21 | |
| rloo | jroll: sigh. even the 'Downloads' info is old. | 19:22 |
| jroll | rloo: yes, this was sufficiently complained about during the switchover, I don't think folks did anything about it though | 19:22 |
| jroll | but I also think the 'get everything off of launchpad' was supposed to happen much sooner | 19:22 |
| rloo | jroll: maybe add a note saying that is all outdated then. | 19:23 |
| jroll | rloo: wdyt about what neutron has: https://launchpad.net/neutron | 19:24 |
| jroll | besides the download bit | 19:24 |
| rloo | jroll: yeah, why do they have newton etc on their page? | 19:24 |
| jroll | rloo: because they're targeting blueprints to newton | 19:25 |
| jroll | I guess we could still register things? maybe? | 19:26 |
| rloo | jroll: so you're the PTL. Whatever is easiest for you to 'maintain'. | 19:26 |
| jroll | idk | 19:26 |
| *** wznoinsk has joined #openstack-ironic | 19:26 | |
| jroll | nothing on launchpad is easy | 19:26 |
| jroll | I hate it all | 19:26 |
| jroll | and now we're like half off it | 19:26 |
| rloo | jroll: ha ha. then put a note somewhere saying it is outdated. look at foo instead. and the only thing useful is ... ?? | 19:26 |
| jroll | rloo: my real questions: 1) is it difficult for a new contributor to figure out how to contribute, 2) is it difficult for a new deployer to figure out how to deploy, 3) does the front page of our launchpad confuse people further | 19:27 |
| jroll | (I'm not sure the answer to any of those) | 19:27 |
| rloo | jroll: if i were new, that page would be very confusing. | 19:27 |
| jroll | rloo: but! would you find that page first | 19:27 |
| jroll | ? | 19:27 |
| rloo | jroll: if i read the README or i dunno where else that link would be. | 19:27 |
| jroll | it isn't on the front page of the google results for "openstack ironic" | 19:28 |
| jroll | yeah | 19:28 |
| rloo | jroll: that page exists. how about we delete that link from README and i'll add something to the description on that page saying it is outdated. | 19:30 |
| rloo | jroll: *if* it lets me change it | 19:30 |
| jroll | rloo: yeah, or we can update LP to be accurate | 19:31 |
| rloo | jroll: *you* can update LP to be accurate. I don't want to learn anything more about it. | 19:31 |
| jroll | :/ | 19:31 |
| jroll | rloo: let me think on it | 19:31 |
| rloo | jroll: ok, i changed my mind. if you think it is worthwhile keeping it accurate, i'll volunteer to do it if i have perms and it is easier than walking over hot coals. | 19:32 |
| jroll | rloo: ok, it shouldn't be hard. you have access. click "change branding info" on https://launchpad.net/ironic | 19:33 |
| *** sabeen has quit IRC | 19:33 | |
| rloo | jroll: ok | 19:33 |
| rloo | jroll: someone could add pixie boot there... | 19:34 |
| jroll | hehe | 19:34 |
| jroll | we'll have a "redesigned" bear soon >.> | 19:34 |
| rloo | jroll: yeah, let's see how baby bear looks. | 19:35 |
| TheJulia | Sudden urge to save the SVG to my hard drive | 19:38 |
| *** mjturek1 has quit IRC | 19:39 | |
| rloo | zuul looks like it is on its deathbed: https://review.openstack.org/355290 | 19:40 |
| *** mjturek1 has joined #openstack-ironic | 19:40 | |
| TheJulia | ugh, 11 hours | 19:41 |
| jroll | they ran out of space on logs.o.o | 19:41 |
| jroll | it's being dealt with now | 19:42 |
| TheJulia | :( | 19:43 |
| *** mjturek1 has quit IRC | 19:44 | |
| -openstackstatus- NOTICE: The volume for logs.openstack.org filled up rather suddenly, causing a number of jobs to fail with a POST_FAILURE result and no logs; we're manually expiring some logs now to buy breathing room, but any changes which hit that in the past few minutes will need to be rechecked and/or approved again | 19:45 | |
| *** jcoufal has quit IRC | 19:48 | |
| * jlvillal noticed that error happen | 19:50 | |
| rajinir | sambetts, thisago, figured out the earlier issues. But now the tempest test fails, on BaremetalBasicOps. Failed to establish authenticated ssh connection to cirros@x.x.x.x. Any ideas on missing config? | 19:57 |
| rajinir | sambetts, thiago, figured out the earlier issues. But now the tempest test fails, on BaremetalBasicOps. Failed to establish authenticated ssh connection to cirros@x.x.x.x. Any ideas on missing config? | 19:58 |
| *** mjturek1 has joined #openstack-ironic | 20:06 | |
| *** Sukhdev has quit IRC | 20:06 | |
| rajinir | tempest test fails, on BaremetalBasicOps. Failed to establish authenticated ssh connection to cirros@x.x.x.x. Any ideas on missing config? | 20:11 |
| *** moshele has joined #openstack-ironic | 20:14 | |
| rloo | jroll: My contribution to the greater good. added milestones, don't know what to do about downloads: https://launchpad.net/ironic | 20:14 |
| jroll | rloo: thank you for that | 20:16 |
| jroll | rloo: maybe we point here for downloads? http://tarballs.openstack.org/ironic/ | 20:16 |
| rloo | jroll: yw | 20:16 |
| openstackgerrit | Merged openstack/ironic: Update readme file https://review.openstack.org/355290 | 20:16 |
| rloo | jroll: good idea | 20:17 |
| *** dprince has quit IRC | 20:19 | |
| rloo | jroll: done. it shows up in 'External downloads'. | 20:20 |
| *** e0ne has joined #openstack-ironic | 20:20 | |
| jroll | rloo: awesome, thanks! | 20:20 |
| rloo | jroll: :) Now, back to "real" work! | 20:21 |
| jroll | rloo: right? :P | 20:21 |
| *** _vsaienko has joined #openstack-ironic | 20:22 | |
| *** jvgrant_ has quit IRC | 20:23 | |
| rajinir | lucas-dinner: any idea about ssh timeout I see in BaremetalBasicOps> I see you have experience in https://bugs.launchpad.net/diskimage-builder/+bug/1571492 | 20:23 |
| openstack | Launchpad bug 1571492 in diskimage-builder "gate-tempest-dsvm-ironic-pxe_ssh-dib failing : test_baremetal_server_ops ssh timeout" [Critical,Confirmed] | 20:23 |
| jroll | rajinir: it's likely one of two things | 20:24 |
| jroll | 1) network connectivity between the machine being deployed and the machine running tempest | 20:24 |
| jroll | 2) networking not set up in the cirros OS correctly | 20:25 |
| jroll | or I guess, 3) timeout is not as long as the bootup | 20:25 |
| *** abashmak has joined #openstack-ironic | 20:27 | |
| rajinir | jroll: failed after 27 attenos in 26 secs, s there a parameter like ssh_timeout? | 20:27 |
| rajinir | 27 attempts | 20:27 |
| jroll | yeah, let me find the config | 20:27 |
| jroll | rajinir: maybe CONF.validation.ssh_timeout and CONF.validation.connect_timeout? | 20:29 |
| rajinir | <jroll>: Thanks a bunch. On a second thought , could it be sec group ? | 20:30 |
| jroll | rajinir: could be, maybe? | 20:30 |
| rajinir | jroll: thc | 20:31 |
| *** ninguem has quit IRC | 20:31 | |
| rajinir | thanks | 20:31 |
| jroll | you're welcome | 20:31 |
| abashmak | Good afternoon ironicers, is anybody else besides myself and Vasyl affected by this: https://bugs.launchpad.net/nova/+bug/1613622? | 20:33 |
| openstack | Launchpad bug 1613622 in Ironic "nova cellsv2 breaks ironic" [Undecided,Confirmed] | 20:33 |
| *** aNupoisc has quit IRC | 20:37 | |
| *** rcernin has quit IRC | 20:40 | |
| *** snizar has joined #openstack-ironic | 20:42 | |
| *** snizar_ has joined #openstack-ironic | 20:42 | |
| *** moshele has quit IRC | 20:43 | |
| jroll | abashmak: yes, that's affecting everyone, I believe that was fixed today, let me look | 20:45 |
| *** ninguem has joined #openstack-ironic | 20:46 | |
| jroll | abashmak: this is the fix: https://review.openstack.org/#/c/355659/ | 20:46 |
| * jroll closes the bug | 20:46 | |
| abashmak | jroll: great thank you | 20:47 |
| jroll | np | 20:47 |
| *** aNupoisc has joined #openstack-ironic | 20:51 | |
| *** rbrud has quit IRC | 20:54 | |
| *** e0ne has quit IRC | 21:00 | |
| *** sabeen has joined #openstack-ironic | 21:04 | |
| *** snizar_ has quit IRC | 21:06 | |
| *** snizar has quit IRC | 21:06 | |
| *** rbartal has quit IRC | 21:09 | |
| *** snizar has joined #openstack-ironic | 21:13 | |
| *** snizar_ has joined #openstack-ironic | 21:13 | |
| *** michauds has quit IRC | 21:15 | |
| *** thiagop has quit IRC | 21:15 | |
| *** rbrud has joined #openstack-ironic | 21:28 | |
| *** Sukhdev has joined #openstack-ironic | 21:33 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Add framework for doc building in ironic-lib https://review.openstack.org/356753 | 21:37 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Add framework for doc building in ironic-lib https://review.openstack.org/356753 | 21:38 |
| mrda | Morning | 21:42 |
| *** penick has joined #openstack-ironic | 21:43 | |
| *** jhooker has quit IRC | 21:47 | |
| *** jhooker has joined #openstack-ironic | 21:48 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Adds 'openstack baremetal node console' commands https://review.openstack.org/356759 | 21:53 |
| *** wajdi has quit IRC | 21:56 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Fix warnings and inaccuracies in docstrings https://review.openstack.org/356761 | 21:58 |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add 'openstack baremetal node adopt' command https://review.openstack.org/356671 | 21:58 |
| *** adu has joined #openstack-ironic | 22:02 | |
| *** ametts has quit IRC | 22:04 | |
| *** sdake_ has quit IRC | 22:06 | |
| *** adu has quit IRC | 22:07 | |
| *** adu has joined #openstack-ironic | 22:07 | |
| *** amotoki has joined #openstack-ironic | 22:09 | |
| *** amotoki_ has joined #openstack-ironic | 22:10 | |
| *** rpioso has quit IRC | 22:12 | |
| *** cdearborn has quit IRC | 22:14 | |
| *** amotoki has quit IRC | 22:14 | |
| *** sdake has joined #openstack-ironic | 22:21 | |
| *** lucas-dinner has quit IRC | 22:23 | |
| *** aNupoisc has quit IRC | 22:27 | |
| NobodyCam | morning mrda :) | 22:30 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Documentation update https://review.openstack.org/356788 | 22:30 |
| *** lucasagomes has joined #openstack-ironic | 22:30 | |
| devananda | major overhaul of our documentation landing page ^ JayF rloo | 22:31 |
| JayF | devananda: speaking of docs ^^^ I'm adding them to ironic-lib | 22:31 |
| devananda | nice | 22:31 |
| JayF | devananda: have a couple of simple starts above and am about to drop a giant block of metrics dev docs into that chain | 22:31 |
| devananda | JayF: I would eventually like to have all the subprojects' docs linked from our frontpage too | 22:32 |
| JayF | should be simple reviews though, and I'd like to land it so I can add the docs job to project-config | 22:32 |
| devananda | JayF: doubly awesome. I also just added a TODO in that change for the missing metrics page | 22:32 |
| devananda | I'm quite pleased with how the new front page looks (rendered locally) | 22:32 |
| JayF | I suspect the ironic docs for metrics will be basically a link-back to ironic-lib docs, with some informatino about how to configure metrics | 22:32 |
| devananda | JayF: I would prefer the ironic doc page describe how to make use of htem | 22:33 |
| JayF | I mean, here's my problem | 22:33 |
| devananda | also, having spent all day rooting around our docs, there are some really bad pages in there | 22:33 |
| JayF | I don't want to document the same thing both places | 22:33 |
| JayF | and it /should/ be in ironic-lib | 22:33 |
| JayF | I'll have a metrics page, just for "how to metric a thing" I'll have a link back. | 22:34 |
| devananda | a) how to metric a thing in your code | 22:34 |
| devananda | b) how to configure the ironic service to emit metrics into your thing | 22:34 |
| JayF | (a) will be in ironic-lib, (b) will be in ironic | 22:35 |
| devananda | c) what kind of metrics ironic is going to spew and some examples that you might find useful | 22:35 |
| JayF | but (a) will be linked from where (b) is | 22:35 |
| devananda | JayF: perfect | 22:35 |
| devananda | well | 22:35 |
| devananda | also | 22:35 |
| JayF | (c) is useful, but I'm not going to bite off all that today :) | 22:35 |
| JayF | I should have a + b done today at this rate | 22:35 |
| devananda | there should be a page in ironic dev docs for (a) -- which is mostly just a link to that in ironic-lib | 22:35 |
| devananda | and there should be a different page in ironic deploy docs for (b) | 22:35 |
| devananda | that ^ comment will make more sense after you see the newly rendered front page | 22:36 |
| JayF | that makes sense | 22:36 |
| devananda | vsaienk0: http://docs.openstack.org/developer/ironic/dev/ironic-multitenant-networking.html looks like it's incomplete -- are there more patches coming that will flesh out the page? | 22:36 |
| *** aNupoisc has joined #openstack-ironic | 22:37 | |
| JayF | devananda: https://review.openstack.org/#/c/228496/ | 22:37 |
| openstackgerrit | Merged openstack/ironic-python-agent: Use CoreOS 1068.9.0 https://review.openstack.org/354821 | 22:37 |
| JayF | devananda: is the patch you're looking for :) | 22:37 |
| JayF | (I think, at least) | 22:38 |
| *** sdake has quit IRC | 22:38 | |
| *** rbrud has quit IRC | 22:40 | |
| mrda | o/ | 22:40 |
| *** adu_ has joined #openstack-ironic | 22:40 | |
| *** adu has quit IRC | 22:41 | |
| *** adu_ is now known as adu | 22:41 | |
| JayF | o/ | 22:42 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Fix warnings and inaccuracies in docstrings https://review.openstack.org/356761 | 22:43 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-lib: Add developer documentation on metrics https://review.openstack.org/356796 | 22:43 |
| *** jvgrant has joined #openstack-ironic | 22:43 | |
| JayF | If someone wants a quick review https://review.openstack.org/#/c/356753 + https://review.openstack.org/356761 should be quick, easy reviews and will allow me to start the next step (running docs jobs on ironic-lib) | 22:44 |
| JayF | (https://review.openstack.org/356796 would be a bonus, but admittedly that's more substantial :D) | 22:44 |
| * devananda runs out the door to an appointment, will bbl | 22:45 | |
| *** amotoki_ has quit IRC | 22:50 | |
| *** jvgrant has quit IRC | 22:56 | |
| rloo | thx devananda! I'll look tomorrow. | 22:56 |
| *** penick has quit IRC | 23:02 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add create command to ironic client https://review.openstack.org/328955 | 23:04 |
| *** xavierr has quit IRC | 23:05 | |
| *** xavierr has joined #openstack-ironic | 23:06 | |
| *** amotoki has joined #openstack-ironic | 23:08 | |
| *** amotoki has quit IRC | 23:13 | |
| *** causten_ has quit IRC | 23:21 | |
| *** mari0jv has quit IRC | 23:21 | |
| *** baoli has quit IRC | 23:33 | |
| *** amotoki has joined #openstack-ironic | 23:35 | |
| *** amotoki has quit IRC | 23:41 | |
| *** adu has quit IRC | 23:50 | |
| *** adu has joined #openstack-ironic | 23:54 | |
| *** hjanssen-hpe has quit IRC | 23:58 | |
| *** hpe-hj has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!