| *** jaybeale has quit IRC | 00:00 | |
| *** piet has quit IRC | 00:01 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct release note syntax https://review.openstack.org/290193 | 00:03 |
|---|---|---|
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Update the reno index https://review.openstack.org/290194 | 00:03 |
| *** alex_xu has quit IRC | 00:05 | |
| JayF | TheJulia: if you have anything low-hanging in bifrost, natorious is trying to get his ATC discount for the summit | 00:06 |
| JayF | TheJulia: I looked and there's not a lot of actual low hanging fruit in IPA :( | 00:06 |
| *** alex_xu has joined #openstack-ironic | 00:07 | |
| TheJulia | natorious: I would totally <3 a doc update pointing to the various role readmes so it is easy for someone to find them :) | 00:08 |
| TheJulia | and would approve it in a heartbeat if it is just doc update | 00:08 |
| openstackgerrit | Merged openstack/ironic: Replace depricated tempest-lib with tempest.lib https://review.openstack.org/289984 | 00:10 |
| *** suro-patz has quit IRC | 00:30 | |
| *** smoriya_ has joined #openstack-ironic | 00:30 | |
| *** bapalm has quit IRC | 00:37 | |
| *** bapalm has joined #openstack-ironic | 00:54 | |
| *** harlowja has quit IRC | 00:55 | |
| TheJulia | natorious: also, in ironic-inspector, deva filed a bug yesterday with regard to IPA blowing up if no disks are found, and thus no data from inspection is getting pushed back to ironic. I've not looked at it myself, but it seems like it might be good low hanging fruit and would be in IPA code | 00:58 |
| *** mtanino has quit IRC | 01:02 | |
| *** phuongnh has joined #openstack-ironic | 01:03 | |
| *** trown|outtypewww is now known as trown | 01:06 | |
| *** priteau has joined #openstack-ironic | 01:10 | |
| *** rbrud has left #openstack-ironic | 01:13 | |
| *** rbrud has joined #openstack-ironic | 01:14 | |
| *** priteau has quit IRC | 01:15 | |
| *** rbrud has quit IRC | 01:17 | |
| *** izaakk has quit IRC | 01:17 | |
| *** rbrud has joined #openstack-ironic | 01:18 | |
| *** rloo has quit IRC | 01:32 | |
| openstackgerrit | Stephanie Miller proposed openstack/ironic: Fix error in cleaning docs https://review.openstack.org/290222 | 01:35 |
| *** daemontool has joined #openstack-ironic | 01:37 | |
| *** praneshp has quit IRC | 01:39 | |
| *** praneshp has joined #openstack-ironic | 01:39 | |
| *** praneshp has quit IRC | 01:42 | |
| *** mtanino has joined #openstack-ironic | 01:53 | |
| *** lazy_prince has joined #openstack-ironic | 01:54 | |
| *** kan_ has joined #openstack-ironic | 01:56 | |
| *** jaybeale has joined #openstack-ironic | 02:00 | |
| *** spandhe has quit IRC | 02:03 | |
| *** baoli has joined #openstack-ironic | 02:06 | |
| *** rbrud has quit IRC | 02:13 | |
| *** alexpilotti has joined #openstack-ironic | 02:17 | |
| *** alexpilotti has quit IRC | 02:21 | |
| *** rbrud has joined #openstack-ironic | 02:23 | |
| *** baoli has quit IRC | 02:32 | |
| *** baoli has joined #openstack-ironic | 02:34 | |
| *** lazy_prince has quit IRC | 02:36 | |
| *** harshs has quit IRC | 02:38 | |
| *** alexpilotti has joined #openstack-ironic | 02:38 | |
| *** shu-mutou-AFK is now known as shu-mutou | 02:40 | |
| *** alexpilotti has quit IRC | 02:42 | |
| *** yonglihe has quit IRC | 03:02 | |
| *** jaybeale has quit IRC | 03:04 | |
| *** Marga__ has joined #openstack-ironic | 03:06 | |
| *** Marga_ has quit IRC | 03:10 | |
| *** Marga__ has quit IRC | 03:20 | |
| *** Marga_ has joined #openstack-ironic | 03:20 | |
| *** dims has quit IRC | 03:25 | |
| *** suro-patz has joined #openstack-ironic | 03:38 | |
| *** Nisha has joined #openstack-ironic | 03:39 | |
| *** sivaramakrishna has joined #openstack-ironic | 03:43 | |
| sivaramakrishna | Good evening Ironicers!, | 03:43 |
| sivaramakrishna | jroll: any idea on the remaining patches of refactoring? | 03:44 |
| *** garthb has quit IRC | 03:52 | |
| *** suro-patz has quit IRC | 03:58 | |
| *** links has joined #openstack-ironic | 04:03 | |
| *** baoli has quit IRC | 04:04 | |
| *** Nisha_away has joined #openstack-ironic | 04:06 | |
| *** Nisha has quit IRC | 04:06 | |
| *** jaybeale has joined #openstack-ironic | 04:07 | |
| *** jaybeale has quit IRC | 04:07 | |
| *** vinm213 has joined #openstack-ironic | 04:08 | |
| *** Marga_ has quit IRC | 04:23 | |
| *** vinaym213 has joined #openstack-ironic | 04:33 | |
| *** Marga_ has joined #openstack-ironic | 04:35 | |
| *** vinm213 has quit IRC | 04:36 | |
| *** baoli has joined #openstack-ironic | 04:37 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 04:38 |
| *** Marga_ has quit IRC | 04:40 | |
| *** kalpase has joined #openstack-ironic | 04:40 | |
| *** mannidi has joined #openstack-ironic | 04:42 | |
| *** baoli has quit IRC | 04:43 | |
| *** baoli has joined #openstack-ironic | 04:47 | |
| *** Nisha_away has quit IRC | 04:49 | |
| *** Nisha has joined #openstack-ironic | 04:49 | |
| *** phuongnh has quit IRC | 04:51 | |
| *** Marga_ has joined #openstack-ironic | 04:53 | |
| *** Marga_ has quit IRC | 04:54 | |
| *** Marga_ has joined #openstack-ironic | 04:54 | |
| *** alexpilotti has joined #openstack-ironic | 04:54 | |
| *** Marga_ has quit IRC | 04:54 | |
| *** Marga_ has joined #openstack-ironic | 04:54 | |
| *** Marga_ has quit IRC | 04:55 | |
| *** Marga_ has joined #openstack-ironic | 04:55 | |
| *** alexpilotti has quit IRC | 04:59 | |
| *** baoli has quit IRC | 05:00 | |
| *** harshs has joined #openstack-ironic | 05:11 | |
| *** praneshp has joined #openstack-ironic | 05:25 | |
| *** harshs has quit IRC | 05:27 | |
| *** harlowja_at_home has joined #openstack-ironic | 05:29 | |
| *** baoli has joined #openstack-ironic | 05:36 | |
| *** baoli has quit IRC | 05:40 | |
| *** spandhe has joined #openstack-ironic | 05:41 | |
| *** awiddersheim has quit IRC | 05:43 | |
| *** rbrud has quit IRC | 05:43 | |
| *** hoangcx has joined #openstack-ironic | 05:45 | |
| *** vinaym213 has quit IRC | 05:52 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: [Devstack]Add ability to enable shellinabox SSL certificate https://review.openstack.org/289671 | 05:57 |
| *** garthb has joined #openstack-ironic | 05:59 | |
| *** garthb has quit IRC | 06:06 | |
| *** rcernin has joined #openstack-ironic | 06:09 | |
| *** bnemec has quit IRC | 06:11 | |
| *** amotoki has joined #openstack-ironic | 06:25 | |
| *** haomeng_ has joined #openstack-ironic | 06:34 | |
| haomeng_ | chenke, ping | 06:34 |
| chenke | haomeng_:yes | 06:36 |
| haomeng_ | I am here:) | 06:36 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Add support to create UEFI bootable iso https://review.openstack.org/289260 | 06:46 |
| *** vishwanathj has quit IRC | 06:49 | |
| *** vishwanathj has joined #openstack-ironic | 06:51 | |
| *** chlong has quit IRC | 06:54 | |
| *** harlowja_at_home has quit IRC | 07:03 | |
| *** alexpilotti has joined #openstack-ironic | 07:03 | |
| *** pas-ha has joined #openstack-ironic | 07:05 | |
| openstackgerrit | Aparna proposed openstack/ironic: Documentation update to enable HTTPS in Glance and Ironic communication https://review.openstack.org/280533 | 07:06 |
| *** suro-patz has joined #openstack-ironic | 07:07 | |
| *** chlong has joined #openstack-ironic | 07:07 | |
| *** alexpilotti has quit IRC | 07:08 | |
| *** spandhe has quit IRC | 07:10 | |
| *** vishwana_ has joined #openstack-ironic | 07:10 | |
| *** vishwanathj has quit IRC | 07:13 | |
| *** praneshp has quit IRC | 07:19 | |
| *** suro-patz has quit IRC | 07:24 | |
| *** KennethWilke has quit IRC | 07:25 | |
| *** eglute has quit IRC | 07:35 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add nodeinfo_list() to Node Object https://review.openstack.org/290313 | 07:36 |
| *** KennethWilke has joined #openstack-ironic | 07:38 | |
| *** alexpilotti has joined #openstack-ironic | 07:45 | |
| *** alexpilotti has quit IRC | 07:50 | |
| *** ohamada has joined #openstack-ironic | 07:56 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 07:56 |
| *** NobodyCa1 has joined #openstack-ironic | 08:00 | |
| *** NobodyCam has quit IRC | 08:03 | |
| *** haomeng_ has quit IRC | 08:03 | |
| *** ChubYann has quit IRC | 08:04 | |
| *** lazy_prince has joined #openstack-ironic | 08:07 | |
| *** d0ugal has joined #openstack-ironic | 08:09 | |
| *** e0ne has joined #openstack-ironic | 08:10 | |
| *** moshele has joined #openstack-ironic | 08:16 | |
| *** e0ne has quit IRC | 08:19 | |
| *** mkovacik has joined #openstack-ironic | 08:22 | |
| *** mgoddard has joined #openstack-ironic | 08:22 | |
| *** mgoddard has quit IRC | 08:29 | |
| *** smoriya_ has quit IRC | 08:29 | |
| *** pece has joined #openstack-ironic | 08:31 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Remove mock called of ironic-lib inner methods https://review.openstack.org/289008 | 08:33 |
| *** ndipanov has quit IRC | 08:35 | |
| *** athomas has joined #openstack-ironic | 08:36 | |
| *** daemontool_ has joined #openstack-ironic | 08:47 | |
| *** openstackgerrit has quit IRC | 08:47 | |
| *** openstackgerrit has joined #openstack-ironic | 08:48 | |
| *** daemontool has quit IRC | 08:49 | |
| *** mgoddard has joined #openstack-ironic | 08:51 | |
| *** ndipanov has joined #openstack-ironic | 08:55 | |
| *** links has quit IRC | 08:55 | |
| *** ifarkas has joined #openstack-ironic | 08:56 | |
| openstackgerrit | Jane.zhang proposed openstack/ironic-python-agent: Fix bug: Iscsi target fail to be created via lio with error messages: ironic-python-agent log: "Could not create NetworkPortal in configFS" syslog: "kernel_bind() failed: -98" Root cause is tgt service is listing on the same port on OS start, lio cloud https://review.openstack.org/290335 | 09:00 |
| *** ChubYann has joined #openstack-ironic | 09:02 | |
| pas-ha | morning Ironic | 09:04 |
| *** mbound has joined #openstack-ironic | 09:04 | |
| *** dtantsur|afk is now known as dtantsur | 09:10 | |
| *** amotoki has quit IRC | 09:10 | |
| dtantsur | Morning Ironic | 09:10 |
| *** bapalm has quit IRC | 09:14 | |
| *** _degorenko|afk is now known as degorenko | 09:15 | |
| *** lucas-dinner is now known as lucasagomes | 09:16 | |
| Haomeng | dtantsur: morning:) | 09:18 |
| *** jistr has joined #openstack-ironic | 09:19 | |
| Haomeng | dtantsur: I have addressed your comments for my bp - https://review.openstack.org/#/c/266688/, can you help to review if you have time? I add wildcard filter also to run with LIKE SQL operator | 09:19 |
| Nisha | morning ironic | 09:20 |
| dtantsur | sure, but when I clean up the current pre-release priorities | 09:20 |
| *** priteau has joined #openstack-ironic | 09:21 | |
| Haomeng | dtantsur: sure, thanks, nice day, I will go home:) | 09:22 |
| *** eglute has joined #openstack-ironic | 09:24 | |
| *** jcoufal has joined #openstack-ironic | 09:25 | |
| *** amotoki has joined #openstack-ironic | 09:25 | |
| *** bapalm has joined #openstack-ironic | 09:27 | |
| johnthetubaguy | mrda: the priorities etherpad is meant to be were you can argue the priority out and pick the important nova patches, long with the mitaka-rc-potential, so you can get above the shouting. | 09:31 |
| *** hoangcx has quit IRC | 09:33 | |
| *** amotoki has quit IRC | 09:34 | |
| *** mgould has joined #openstack-ironic | 09:34 | |
| *** electrofelix has joined #openstack-ironic | 09:34 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 09:35 |
| lucasagomes | morning all | 09:37 |
| dtantsur | o/ | 09:38 |
| ifarkas | morning! | 09:39 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: DO NOT MERGE, test the agent_ipmitool-nv job https://review.openstack.org/290356 | 09:39 |
| ifarkas | dtantsur, the bug I mentioned yesterday: https://bugs.launchpad.net/ironic/+bug/1554542 | 09:40 |
| openstack | Launchpad bug 1554542 in Ironic "If driver uses iscsi_deploy for deploy interface it always tries to prepare for inband cleaning" [Undecided,New] | 09:40 |
| dtantsur | yeah | 09:40 |
| *** amotoki has joined #openstack-ironic | 09:43 | |
| *** derekh has joined #openstack-ironic | 09:46 | |
| pas-ha | dtantsur, I'm seeing a lot of failures to sync power state in my devstack with virtualbmc, seems that futurist-driven task is being executed for too long https://bugs.launchpad.net/ironic/+bug/1548652 | 09:50 |
| openstack | Launchpad bug 1548652 in Ironic "tmp file for ipmi password might get reused which leads to failure" [Undecided,New] - Assigned to Ramamani Yeleswarapu (ramamani-yeleswarapu) | 09:50 |
| pas-ha | not that lock for executing the ipmitool command was held for 200+ seconds, which is suspiciously too long | 09:50 |
| pas-ha | s/not/note/ | 09:51 |
| *** amotoki has quit IRC | 09:51 | |
| *** e0ne has joined #openstack-ironic | 09:51 | |
| dtantsur | pas-ha, not sure how it's related to Futurist, I regularly see it on bare metal :) we should probably figure out why virtualbmc/libvirt may take so long.. | 09:52 |
| openstackgerrit | xiexs proposed openstack/python-ironicclient: Add a node validation for the osc command "baremetal set/unset" https://review.openstack.org/290365 | 09:53 |
| openstackgerrit | Merged openstack/ironic: Fix error in cleaning docs https://review.openstack.org/290222 | 09:53 |
| pas-ha | well, this may be just coincidence, but I started seeing this soon after futurist patches were merged (but I've started using vbmc around this time too...) | 09:53 |
| mgould | morning everyone! | 09:53 |
| *** alexpilotti has joined #openstack-ironic | 09:53 | |
| *** amotoki has joined #openstack-ironic | 09:55 | |
| *** amotoki has quit IRC | 09:56 | |
| *** ChubYann has quit IRC | 09:58 | |
| *** stendulker has joined #openstack-ironic | 10:00 | |
| *** e0ne has quit IRC | 10:01 | |
| *** ChubYann has joined #openstack-ironic | 10:02 | |
| *** e0ne has joined #openstack-ironic | 10:04 | |
| *** amotoki has joined #openstack-ironic | 10:06 | |
| vdrok | morning ironic and morning pas-ha dtantsur Haomeng Nisha lucasagomes and ifarkas | 10:11 |
| vdrok | and good morning mgould :) | 10:11 |
| Nisha | vdrok, o/ | 10:11 |
| *** ChubYann has quit IRC | 10:12 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic-specs: Add neutron-ironic port mapping function https://review.openstack.org/279148 | 10:18 |
| mgould | vdrok, pas-ha Haomeng Nisha morning! | 10:19 |
| *** nathan-h has joined #openstack-ironic | 10:19 | |
| *** daemontool_ is now known as daemontool | 10:19 | |
| lucasagomes | vdrok, o/ morning | 10:20 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic-specs: Add neutron-ironic port mapping function https://review.openstack.org/279148 | 10:21 |
| *** sambetts|afk is now known as sambetts | 10:28 | |
| sambetts | Morning all o/ | 10:28 |
| dtantsur | morning sambetts | 10:29 |
| mgould | morning sambetts | 10:29 |
| sambetts | o/ dtantsur | 10:29 |
| sambetts | o/ mgould | 10:30 |
| zigo | Hi over here! | 10:30 |
| zigo | Trying to build 4.3.0, I get: http://paste.openstack.org/show/489804/ | 10:30 |
| zigo | Is this known, and maybe already solved? | 10:31 |
| zigo | (this is the result of building in Debian experimental, FYI) | 10:31 |
| zigo | dtantsur: ^ | 10:32 |
| zigo | Ironic is the last package I get to upload to Debian before I can start the tempest validation process for Mitaka... :P | 10:32 |
| mkoderer | sambetts: ping | 10:33 |
| * zigo goes for lunch, hopefully someone will reply to that before I come back to finish Ironic packaging for Mitaka... | 10:33 | |
| sambetts | mkoderer: Hi | 10:33 |
| mkoderer | sambetts: one question to ml2 ironic.. do you have an neutron agent that set the binding to complete? | 10:34 |
| mkoderer | sambetts: or will it be just a mech driver that set's it? | 10:34 |
| lucasagomes | sambetts, o/ morning | 10:37 |
| *** mgoddard has quit IRC | 10:37 | |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 10:38 |
| sambetts | mkoderer: at least in our (Cisco's) case thats handled in the ml2 driver, its the mech driver doing all the required work so we complete the binding at that layer iirc | 10:39 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic-specs: Add neutron-ironic port mapping function https://review.openstack.org/279148 | 10:40 |
| mkoderer | sambetts: ok yeah an agent does not really work in that case | 10:40 |
| sambetts | mkoderer: yeah it doesn't really fit the BM model yet :) | 10:42 |
| dtantsur | zigo, I don't quite remember but I think we got broken by one of libraries in this fashion. it's fixed in master, lemme find a patch | 10:42 |
| *** sivaramakrishna has quit IRC | 10:44 | |
| mkoderer | sambetts: ok I see.. for multi-segment (like HPB) it's might needed to have a mech driver that terminates the binding process.. I wrote a short fake mech driver for manila like http://paste.openstack.org/show/489806/ | 10:44 |
| dtantsur | zigo, still not 100% sure, but looks familiar to https://bugs.launchpad.net/ironic/+bug/1466405 | 10:44 |
| openstack | Launchpad bug 1466405 in Ironic "broken ACL tests " [Critical,Fix released] - Assigned to Lucas Alvares Gomes (lucasagomes) | 10:44 |
| lucasagomes | zigo, I think the patch for that error was backported to liberty already | 10:45 |
| lucasagomes | 1 sec | 10:45 |
| lucasagomes | I mean, not the one in the bug. The recent error | 10:45 |
| lucasagomes | zigo, dtantsur https://review.openstack.org/#/c/284436/ | 10:45 |
| lucasagomes | I believe the long term fix for that problem is to Ironic start using the keystoneauth session, we have some patches in the queue for it already | 10:47 |
| lucasagomes | we probably need to escalate them | 10:47 |
| dtantsur | lucasagomes++ | 10:47 |
| mkoderer | sambetts: ah and btw this ML thread could be intressting also for you http://thread.gmane.org/gmane.comp.cloud.openstack.devel/77292 not sure if Ironic need's also the segmentation_id of the lowest bound segemnt | 10:48 |
| *** dtantsur is now known as dtantsur|brb | 10:49 | |
| *** mbound has quit IRC | 10:52 | |
| Nisha | dtantsur|brb, lucasagomes ....i was testing bios + localboot with agent drivers, even though IPA returns (prepare_image) the root uuid, the https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/image.py#L53 returns uuid as None | 10:55 |
| Nisha | any idea what can be issue | 10:55 |
| Nisha | ironic side changes work absolutely fine for uefi + localboot | 10:56 |
| Nisha | lucasagomes, ^^^^ | 10:56 |
| *** mgoddard has joined #openstack-ironic | 10:59 | |
| *** mbound has joined #openstack-ironic | 10:59 | |
| Nisha | lucasagomes, so i am doubting its something to do with _get_partition() call of the extensions/image.py in IPA | 10:59 |
| Nisha | lucasagomes, error logs can be seen at http://paste.openstack.org/show/489811/ | 11:01 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add documentation about the disk_label capability https://review.openstack.org/288520 | 11:05 |
| *** dims has joined #openstack-ironic | 11:07 | |
| *** stendulker has quit IRC | 11:07 | |
| lucasagomes | Nisha, odd... the only difference I can spot is that we use different tools on each side | 11:07 |
| lucasagomes | blkid for when partitioning the disk | 11:08 |
| lucasagomes | lsblk in _get_partition() | 11:08 |
| lucasagomes | Nisha, this is when BIOS is used? So can I assume the disk label is msdos (MBR) ? | 11:08 |
| Nisha | yes | 11:08 |
| Nisha | its in bios | 11:09 |
| Nisha | uefi works fine | 11:09 |
| Nisha | lucasagomes, ^^^ | 11:09 |
| Nisha | so shud i change the utility and try? | 11:10 |
| *** smoriya has quit IRC | 11:10 | |
| Nisha | blkid or lsblk | 11:11 |
| lucasagomes | Nisha, that would be one option, but I would be extremely surprised if that was the error | 11:12 |
| lucasagomes | cause these are very stable tools and should work with MBR no problem | 11:12 |
| lucasagomes | worth trying tho | 11:12 |
| Nisha | lucasagomes, true but i get the error from that line only if u check the logs as given in paste url | 11:13 |
| lucasagomes | Nisha, you sure it's not any method you may have that is not correctly parsing the result returned by the IPA method that partition the disk? | 11:16 |
| Nisha | lucasagomes, yes | 11:17 |
| Nisha | i added the pdb also to check that | 11:17 |
| Nisha | the root uuid is returned to the agent call "install_bootloader" | 11:17 |
| Nisha | in that call it calls _install_grub2 which calls get_partition | 11:18 |
| *** ChubYann has joined #openstack-ironic | 11:18 | |
| Nisha | from where i see the error | 11:18 |
| lucasagomes | Nisha, https://github.com/openstack/ironic-lib/blob/3bdfe793f40ba34e5f1e2cbe70dc813d95901b38/ironic_lib/disk_utils.py#L531 | 11:18 |
| *** smoriya_afk has joined #openstack-ironic | 11:19 | |
| lucasagomes | now see L555 of https://review.openstack.org/#/c/162008/10/ironic/drivers/modules/agent.py | 11:19 |
| lucasagomes | apparently ironic-lib returns "root uuid" (space between words) | 11:19 |
| *** smoriya_afk is now known as smoriya | 11:19 | |
| lucasagomes | the patch is looking for "root_uuid" (underscore between words) | 11:19 |
| lucasagomes | may that be the cause? | 11:19 |
| Nisha | lucasagomes, that works perfectly fine | 11:20 |
| lucasagomes | can you add a debug log there, make sure the _get_uuid_from_result is returning a uuid? | 11:20 |
| lucasagomes | Nisha, ok | 11:20 |
| Nisha | agent returns the root uuis | 11:20 |
| Nisha | uuid* | 11:20 |
| Nisha | that function gets the uuid from the log messages returned by IPA | 11:20 |
| Nisha | i have addedd that in IPA patch | 11:20 |
| lucasagomes | Nisha, right... yeah I would need an env to debug it | 11:21 |
| lucasagomes | looking at the code is harder to figure out] | 11:21 |
| Nisha | hmmm | 11:21 |
| lucasagomes | Nisha, isn't it kinda strange also that we are parsing the command_result as a string? Wouldn't that be a dictionary? | 11:22 |
| lucasagomes | the work_on_disk method from ironic-lib does return a dictionary | 11:23 |
| Nisha | yes that returns dictionary | 11:23 |
| lucasagomes | I need to get my head on it, but I don't see why we need string manipulation there? | 11:23 |
| Nisha | but its is returned to prepare_image() of agent | 11:23 |
| Nisha | work_on_disk is in ironic_lib | 11:24 |
| Nisha | while we need to return from IPA | 11:24 |
| lucasagomes | Nisha, so why _get_uuid_from_result() is calling split() ? | 11:24 |
| lucasagomes | right | 11:24 |
| lucasagomes | yeah, it's complex code that involved more than one project, I would need an env to dig into | 11:25 |
| Nisha | lucasagomes, hmmm | 11:25 |
| lucasagomes | by looking at the code in reviews from queue and github makes it hard to debug | 11:25 |
| Nisha | agent returns something like this for prepare_image | 11:25 |
| Nisha | Status of agent commands for node 3cdc35d5-0f75-4dd5-9e56-9fbcd09d1472: prepare_image: result "{u'result': u'prepare_image: ima | 11:25 |
| Nisha | ge (ubuntu-cloud-image.qcow2) written to device /dev/sdb root_uuid=51f17d61-e4df-4813-a768-a06a69bcf393 efi_system_partition_uuid=None'}" | 11:25 |
| lucasagomes | Nisha, try to add a bunch of debug logs, see what the tools is retuning and so on | 11:25 |
| Nisha | hmm | 11:25 |
| lucasagomes | Nisha, right... maybe in ur patch see L526 | 11:26 |
| lucasagomes | command = self._client.get_commands_status(node)[-1] | 11:26 |
| lucasagomes | is it possible that this is returning another command other than "prepare_image" ? | 11:27 |
| lucasagomes | since you are using an index to get the command (the last one) and not checking the command name? | 11:27 |
| Nisha | lucasagomes, my patch returns the uuid corectly as it works for other combinations | 11:27 |
| lucasagomes | ok | 11:27 |
| lucasagomes | well, you will have to debug it then in the env you have | 11:27 |
| lucasagomes | sorry, only looking at code changes is tough | 11:27 |
| Nisha | i understand | 11:28 |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for building ISO for deploy ramdisk https://review.openstack.org/264590 | 11:29 |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 11:29 |
| lucasagomes | Nisha, I'm kinda asking about that bits, because EFI partitions have a standardlized UUID | 11:30 |
| lucasagomes | so that wouldn't surprise me if EFI works even tho the uuid there is None | 11:30 |
| lucasagomes | EFI partitions UUID is always "C12A7328-F81F-11D2-BA4B-00A0C93EC93B" | 11:30 |
| lucasagomes | https://en.wikipedia.org/wiki/GUID_Partition_Table#Partition_type_GUIDs | 11:31 |
| lucasagomes | see the EFI system partition entry there | 11:31 |
| lucasagomes | this also may shed some light on the problem: https://www.freedesktop.org/wiki/Specifications/DiscoverablePartitionsSpec/ | 11:32 |
| *** ChubYann has quit IRC | 11:32 | |
| *** ohamada_ has joined #openstack-ironic | 11:36 | |
| *** ohamada has quit IRC | 11:36 | |
| *** priteau has quit IRC | 11:39 | |
| *** ChubYann has joined #openstack-ironic | 11:41 | |
| *** chlong has quit IRC | 11:42 | |
| *** links has joined #openstack-ironic | 11:48 | |
| *** sylwesterB has joined #openstack-ironic | 11:49 | |
| Nisha | lucasagomes, i think the issue is due to different commands | 11:50 |
| Nisha | just an example of what i did on my devstack server | 11:51 |
| Nisha | $ lsblk -PbioKNAME,UUID,TYPE /dev/loop0 | 11:51 |
| Nisha | KNAME="loop0" UUID="" TYPE="loop" | 11:51 |
| lucasagomes | Nisha, that's the L526 I've pointed out? | 11:51 |
| lucasagomes | oh | 11:51 |
| lucasagomes | ok | 11:51 |
| Nisha | $ sudo blkid -s UUID -o value /dev/loop0 | 11:51 |
| Nisha | 9ea64e98-e9d8-47fd-b9d6-35ed01e138a5 | 11:51 |
| lucasagomes | there you go... I'm very surprised | 11:51 |
| lucasagomes | Nisha, can you check the disk label ? | 11:51 |
| lucasagomes | is it really MBR? | 11:51 |
| *** sinval-afk is now known as sinval | 11:52 | |
| sinval | good morning Ironic | 11:52 |
| Nisha | i checked it is msdos while doing the partitioning | 11:52 |
| lucasagomes | Nisha, alright. Wow so lsblk seems broken in the OS you are using | 11:52 |
| lucasagomes | that's quite a surprise | 11:52 |
| Nisha | lucasagomes, ok so i am trying on ubuntu | 11:53 |
| lucasagomes | Nisha, change it to use blkid to get the UUID see if goes trough | 11:53 |
| lucasagomes | Nisha, I would fill a bug against the version of ubuntu you are using reporting it | 11:53 |
| Nisha | lucasagomes, while on fedora the localboot for uefi doesnt work as efibootmgr is broken and doesnt install the boot loader in the deploy ramdisk | 11:53 |
| lucasagomes | it's quite a finding | 11:53 |
| Nisha | :) | 11:53 |
| Nisha | lucasagomes, yes i will build the ramdisk and do it | 11:54 |
| lucasagomes | Nisha, ack | 11:54 |
| lucasagomes | Nisha, btw, you don't need to install boot loader in the deploy ramdisk | 11:58 |
| lucasagomes | Nisha, why would do that? | 11:58 |
| *** mannidi has quit IRC | 11:58 | |
| lucasagomes | Nisha, the bootloader should be installed in the image (as the EFI modules) | 11:59 |
| Nisha | lucasagomes, i did taht and after that localboot + uefi passed | 11:59 |
| Nisha | with ubuntu ramdisk | 11:59 |
| lucasagomes | o.Õ | 12:00 |
| lucasagomes | but we do chroot onto the user image to install the bootloader for local boot | 12:00 |
| Nisha | with fedora ramdisk even iscsi_ilo localboot + uefi or localboot + bios didnt pass | 12:00 |
| lucasagomes | https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/image.py#L118-L121 | 12:00 |
| Nisha | lucasagomes, i know but i guess the deploy ramdisk itself should have sane boot loader too | 12:01 |
| Nisha | lucasagomes, i just tried and it worked with ubuntu | 12:01 |
| lucasagomes | Nisha, we do [i]PXE | 12:01 |
| lucasagomes | that's the bootloader we use | 12:01 |
| lucasagomes | anyway... that's another discussion | 12:01 |
| Nisha | lucasagomes, yes, thats something how should be the ramdisk built | 12:01 |
| Nisha | that should be out of scope of this feature | 12:02 |
| lucasagomes | I'm just scratching my head because IMHO having a the bootloader in the deploy ramdisk makes little sense to me | 12:02 |
| lucasagomes | since it's not used | 12:02 |
| lucasagomes | yeah | 12:02 |
| lucasagomes | (little sense cause grub will use at least 30MB of binaries that will just increase the size of the ramdisk) | 12:03 |
| *** trown is now known as trown|commute | 12:03 | |
| Nisha | lucasagomes, anyway thanks, i will just add the blkid and try rebuilding the ramdisk and try deploy | 12:03 |
| lucasagomes | Nisha, cool | 12:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Extend the Conductor RPC object https://review.openstack.org/287914 | 12:04 |
| *** moshele has quit IRC | 12:07 | |
| *** thrash|g0ne is now known as thrash | 12:10 | |
| *** amotoki has quit IRC | 12:10 | |
| *** chlong has joined #openstack-ironic | 12:14 | |
| *** dtantsur|brb is now known as dtantsur | 12:18 | |
| *** lucasagomes is now known as lucas-hungry | 12:21 | |
| *** baoli has joined #openstack-ironic | 12:22 | |
| *** daemontool_ has joined #openstack-ironic | 12:23 | |
| *** amotoki has joined #openstack-ironic | 12:24 | |
| *** Marga__ has joined #openstack-ironic | 12:24 | |
| zigo | dtantsur: lucas-hungry: Thanks guys, I'm trying this patch. | 12:25 |
| *** baoli has quit IRC | 12:26 | |
| *** alineb has quit IRC | 12:27 | |
| *** greghaynes has quit IRC | 12:27 | |
| *** chlong has quit IRC | 12:27 | |
| *** Marga_ has quit IRC | 12:27 | |
| *** bapalm has quit IRC | 12:27 | |
| *** daemontool has quit IRC | 12:27 | |
| *** alaski has quit IRC | 12:27 | |
| *** bapalm has joined #openstack-ironic | 12:27 | |
| *** daemontool_ is now known as daemontool | 12:29 | |
| *** amotoki has quit IRC | 12:29 | |
| *** alaski has joined #openstack-ironic | 12:31 | |
| *** kalpase has quit IRC | 12:33 | |
| zigo | dtantsur: lucas-hungry: Great! It fixed the problem. | 12:34 |
| dtantsur | good :) | 12:34 |
| *** alineb has joined #openstack-ironic | 12:37 | |
| *** thiagop has joined #openstack-ironic | 12:39 | |
| zigo | Ironic 4.3.0 uploaded to Debian Experimental. Thanks for your help. | 12:39 |
| zigo | Do you guys plan on doing one more release for Mitaka, or what? | 12:39 |
| thiagop | Good morning Ironic | 12:40 |
| *** chlong has joined #openstack-ironic | 12:40 | |
| *** greghaynes has joined #openstack-ironic | 12:40 | |
| dtantsur | zigo, yes. maybe even two of them: this week and around RC. but better ask jroll or devananda | 12:41 |
| dtantsur | morning thiagop | 12:42 |
| zigo | Ok, thanks for the info. I'll stay tuned. | 12:42 |
| * zigo works on updating ironic-inspector now. | 12:42 | |
| *** baoli has joined #openstack-ironic | 12:46 | |
| *** baoli_ has joined #openstack-ironic | 12:53 | |
| *** mannidi has joined #openstack-ironic | 12:54 | |
| *** baoli has quit IRC | 12:56 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 13:00 |
| *** ChubYann has quit IRC | 13:01 | |
| zigo | ironic-inspector 3.0.0 uploaded to Debian experimental. No problem at all with that one. | 13:01 |
| *** ChubYann has joined #openstack-ironic | 13:02 | |
| *** krtaylor has quit IRC | 13:02 | |
| dtantsur | zigo, good to know, thanks :) | 13:03 |
| zigo | dtantsur: I can't find an oslo-config-generator file for Ironic by the way. | 13:04 |
| zigo | Should I use the sample? | 13:04 |
| zigo | That's not the OpenStack way anymore ... | 13:04 |
| zigo | (and no, I can't use tox when packaging...) | 13:04 |
| dtantsur | zigo, yeah.. ironic does not use it. I think the reason was because it required too big refactoring | 13:04 |
| dtantsur | lucas-hungry or vdrok , do you remember ^^ | 13:04 |
| zigo | Ok, thanks. | 13:04 |
| *** lazy_prince has quit IRC | 13:05 | |
| *** dims has quit IRC | 13:05 | |
| *** dims_ has joined #openstack-ironic | 13:05 | |
| vdrok | dtantsur, yep, I think you're right | 13:05 |
| *** dims_ has quit IRC | 13:10 | |
| *** dims has joined #openstack-ironic | 13:14 | |
| *** thiagop is now known as thiagop-afk | 13:14 | |
| *** dprince has joined #openstack-ironic | 13:14 | |
| openstackgerrit | William Stevenson proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 13:15 |
| *** trown|commute is now known as trown | 13:18 | |
| *** thiagop-afk is now known as thiagop | 13:24 | |
| marlinc | Test | 13:26 |
| thiagop | marlinc: pong | 13:28 |
| lucas-hungry | zigo, cool | 13:28 |
| *** lucas-hungry is now known as lucasagomes | 13:28 | |
| lucasagomes | zigo, dtantsur yeah the refactor was huge... but at the time that wasn't the main reason | 13:29 |
| lucasagomes | I think it was more about the design of the oslo.config? Because it requires you to put list every option group in a single file | 13:29 |
| lucasagomes | and keep maintaining it as options are added | 13:29 |
| lucasagomes | and the old script worked for us, so we didn't see the benefit in using it | 13:30 |
| lucasagomes | zigo, FWIW, next meeting (monday 17:00 UTC) we are going to talk about it | 13:30 |
| lucasagomes | I think there's some new stuff that came to oslo.config that we may need to address and perhaps switch ironic to use it | 13:30 |
| *** links has quit IRC | 13:35 | |
| TheJulia | good morning | 13:37 |
| dtantsur | morning TheJulia | 13:38 |
| mkovacik | aarefiev, hi, would you mind me fixing the discovery_exercise.sh script? | 13:38 |
| thiagop | morning TheJulia | 13:39 |
| *** afaranha has quit IRC | 13:46 | |
| pas-ha | zigo, I just tried to do it recently and yes, to big refactoring, and in one shot, as config options are defined all over the place (basically in every driver module) | 13:48 |
| *** afaranha has joined #openstack-ironic | 13:48 | |
| pas-ha | so I slacked on this idea for now. however if people decide we need it, I'd be happy to take it on | 13:49 |
| *** keedya has joined #openstack-ironic | 13:50 | |
| * pas-ha also kind of needs it for keystoneauth migration... | 13:50 | |
| *** trown is now known as trown|brb | 13:50 | |
| *** trown|brb is now known as trown | 13:52 | |
| pas-ha | lucasagomes, not in single file any more. just add list_opts function to every module defining config options, add oslo.config plug points to setup.cfg and use them as input for config-generator | 13:52 |
| *** mannidi_ has joined #openstack-ironic | 13:53 | |
| pas-ha | see my patch in inspector https://review.openstack.org/#/c/286510 | 13:53 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing node-create command https://review.openstack.org/262055 | 13:54 |
| *** mannidi has quit IRC | 13:54 | |
| *** amotoki has joined #openstack-ironic | 13:54 | |
| *** ohamada_ has quit IRC | 13:57 | |
| *** Goneri has joined #openstack-ironic | 14:00 | |
| *** Nisha has quit IRC | 14:00 | |
| *** daemontool_ has joined #openstack-ironic | 14:03 | |
| *** daemontool has quit IRC | 14:07 | |
| NobodyCa1 | Good Morning Ironicers :) | 14:14 |
| vdrok | morning NobodyCa1 :) | 14:15 |
| lucasagomes | pas-ha, thanks will take a look | 14:17 |
| *** mtanino has quit IRC | 14:19 | |
| thiagop | morning NobodyCa1 | 14:20 |
| lucasagomes | NobodyCa1, morning | 14:22 |
| sambetts | o/ NobodyCam's alt nick | 14:22 |
| *** NobodyCa1 is now known as NobodyCam | 14:23 | |
| NobodyCam | must of had a network burp | 14:23 |
| NobodyCam | :) | 14:23 |
| dtantsur | morning NobodyCam | 14:23 |
| NobodyCam | morning all :) | 14:23 |
| sambetts | did you get over your flu? | 14:23 |
| NobodyCam | mostly, still have a cough that I cant seem to shake, but am feeling much better | 14:26 |
| *** cdearborn has joined #openstack-ironic | 14:26 | |
| keedya | G morning!! | 14:27 |
| NobodyCam | mornign keedya :) | 14:27 |
| sambetts | NobodyCam: thats the same as a friend of mine has, and he can't shake the cough, his whole family caught it :( | 14:27 |
| NobodyCam | ugh :( | 14:27 |
| NobodyCam | sambetts: don't say that :) | 14:29 |
| NobodyCam | hehehe | 14:29 |
| sambetts | :-P | 14:29 |
| *** awiddersheim has joined #openstack-ironic | 14:31 | |
| sylwesterB | hi guys, do you have Ironic integration tests with ironic drivers which don’t require devstack to run? | 14:36 |
| lucasagomes | jlvillal, around ? | 14:36 |
| lucasagomes | folks if you have a second, mind taking a look at https://review.openstack.org/#/c/289290/ ? It has 2+2's already | 14:37 |
| openstackgerrit | Merged openstack/ironic-inspector: Deprecate root_device_hint name for raid_device plugin https://review.openstack.org/289400 | 14:39 |
| openstackgerrit | Merged openstack/ironic-inspector: Fix incorrect string formatting in the SSL code https://review.openstack.org/289329 | 14:39 |
| sylwesterB | guys, does anyone know the answer? | 14:39 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Only pause in VMs when performing multinode testing https://review.openstack.org/290539 | 14:41 |
| sambetts | sylwesterB: I'm not sure what you mean? Integration tests for standalone ironic without the rest of openstack? or Integration tests with OpenStack installed using a method other than devstack? | 14:41 |
| keedya | good morning dtantsur | 14:43 |
| dtantsur | keedya, o/ | 14:43 |
| *** mannidi_ has quit IRC | 14:43 | |
| keedya | Is there any established work on telemtry and sel logs in ironic? | 14:43 |
| dtantsur | keedya, http://specs.openstack.org/openstack/ironic-specs/specs/juno-implemented/send-data-to-ceilometer.html is probably the closest | 14:44 |
| sylwesterB | sambetts: I meant the no. 1 option. I want to introduce such testing in openstack/fuel-web and I'm looking for similar solutions ;) | 14:48 |
| *** igornsa_ has joined #openstack-ironic | 14:48 | |
| openstackgerrit | Imre Farkas proposed openstack/ironic: [WIP] Move cleaning related methods to boot interface https://review.openstack.org/290547 | 14:50 |
| sambetts | sylwesterB: They might have integration testing for bifrost, TheJulia may know more | 14:52 |
| sylwesterB | TheJulia: are you there? | 14:52 |
| sylwesterB | and thanks sambetts for the info ;) | 14:53 |
| *** bnemec has joined #openstack-ironic | 14:53 | |
| *** frontrunner has joined #openstack-ironic | 14:54 | |
| *** igornsa_ has quit IRC | 14:55 | |
| TheJulia | sylwesterB: I'm here, whats up? | 14:55 |
| sylwesterB | TheJulia: hi, do you use devstack to test bifrost? | 14:56 |
| TheJulia | NobodyCam: the bug I caught on by way from seattle to chicago in early december, i didn't loose that cough for at least a month | 14:56 |
| sylwesterB | TheJulia: or maybe more clearly: Do you use standalone Ironic for testing somewhere? | 14:57 |
| TheJulia | sylwesterB: no, bifrost is sets ironic up in stand-alone mode so no devstack required | 14:57 |
| NobodyCam | oh thats icky | 14:57 |
| *** sinval is now known as sinval-afk | 14:57 | |
| * NobodyCam notes to buy lots of cough drops | 14:57 | |
| sylwesterB | TheJulia: That is what i'm looking for ;) could you show be the script which is doing it? | 14:58 |
| TheJulia | sylwesterB: bifrost's CI jobs, we use a bare instance and set everything up | 14:58 |
| TheJulia | sylwesterB: scripts/test-bifrost.sh or scripts/test-bifrost-build-image.sh | 14:58 |
| TheJulia | and I guess there is now a dhcp inventory test script | 14:58 |
| TheJulia | yeah, that merged | 14:58 |
| TheJulia | sylwesterB: Are you attempting to setup some sort of downstream ci? | 14:59 |
| TheJulia | sylwesterB: or are you just looking to validate things basically still work | 14:59 |
| *** praneshp has joined #openstack-ironic | 15:02 | |
| *** praneshp has quit IRC | 15:02 | |
| sylwesterB | We just introduced 'extensions' in openstack/fuel-web project which can be installed from external repositories, and now I'm gathering requirements for testing them | 15:02 |
| sylwesterB | so for now i'm looking for the best practices used openstack-wide | 15:02 |
| *** Nisha has joined #openstack-ironic | 15:03 | |
| *** jaybeale has joined #openstack-ironic | 15:03 | |
| *** praneshp has joined #openstack-ironic | 15:04 | |
| keedya | thanks dtantsur | 15:04 |
| *** saibarspeis has joined #openstack-ironic | 15:05 | |
| *** xek_ is now known as xek | 15:05 | |
| TheJulia | sylwesterB: Okay, well if you have any questions regarding bifrost, just let me know :) | 15:06 |
| NobodyCam | off the wall Historical question.. anyone have the link to the Paris summit planing etherpad? | 15:06 |
| *** mtanino has joined #openstack-ironic | 15:06 | |
| jlvillal | lucasagomes, I am around now :) | 15:07 |
| lucasagomes | jlvillal, good morning | 15:07 |
| NobodyCam | morning jlvillal :) | 15:07 |
| jlvillal | lucasagomes, Good afternoon :) | 15:07 |
| lucasagomes | jlvillal, thanks for the comment on the disk label patch. I've put some thought about it and commented back | 15:07 |
| lucasagomes | jlvillal, lemme know what you think | 15:07 |
| TheJulia | NobodyCam: search the logs yet? | 15:08 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Only pause in VMs when performing multinode testing https://review.openstack.org/290539 | 15:08 |
| TheJulia | like... from the first day of the summit? | 15:08 |
| jlvillal | lucasagomes, Yeah. It is just whenever I hear 'disk label' I think of the labels used in like /etc/fstab. | 15:08 |
| lucasagomes | jlvillal, ps* you can do it after ur cup of joe, no rush | 15:08 |
| NobodyCam | oh I have not | 15:08 |
| NobodyCam | but great idea | 15:08 |
| lucasagomes | jlvillal, yeah I understand. I've tried to reason about why we use it in ironic | 15:09 |
| jlvillal | lucasagomes, But I understand from the comments that disk label has been used in Ironic for what you mean. | 15:09 |
| * jlvillal still doesn't like it much ;) | 15:09 | |
| jlvillal | lucasagomes, But I will remove my -1! :) | 15:09 |
| TheJulia | NobodyCam: ugh, monday/tuesday ar emissing | 15:09 |
| lucasagomes | jlvillal, yeah. Problem is, each tool may call it differently | 15:10 |
| vdrok | morning thiagop TheJulia and jlvillal ! | 15:10 |
| thiagop | hey vdrok | 15:10 |
| lucasagomes | jlvillal, and honestly I think using "type" is very overused cause there are so many types when comes to partitioning :-( | 15:10 |
| *** [1]cdearborn has joined #openstack-ironic | 15:10 | |
| lucasagomes | disk type, partition type, filesystem type, and other types such as (primary, logical or extended partitions) | 15:10 |
| sylwesterB | TheJulia: thanks! | 15:10 |
| lucasagomes | jlvillal, anyway. Lemme know if you still need we need to change it | 15:11 |
| jlvillal | lucasagomes, I did a +1 | 15:11 |
| jlvillal | lucasagomes, Thanks | 15:11 |
| lucasagomes | ty | 15:11 |
| TheJulia | NobodyCam: https://wiki.openstack.org/wiki/Design_Summit/Kilo/Etherpads#Ironic | 15:11 |
| NobodyCam | w00t Thank you :) | 15:11 |
| lucasagomes | s/need/think | 15:12 |
| vdrok | keedya, there is a spec about SEL - https://review.openstack.org/#/c/246965/ | 15:12 |
| *** kan_ has quit IRC | 15:16 | |
| keedya | vdrok, dtantsur: is there a plan to use plugins for this task as well? | 15:16 |
| keedya | RackHD already implemented this and have amqp channels for sel and sdr | 15:17 |
| *** harshs has joined #openstack-ironic | 15:20 | |
| keedya | vdrok, dtantsur: here is the discussion we had setup for shovel: https://blueprints.launchpad.net/shovel/+spec/amqp-event-handler | 15:21 |
| *** praneshp__ has joined #openstack-ironic | 15:23 | |
| yuriyz | hi keedya current SEL spec only for extend Ironic driver API, IMO this is preferred way in current architecture | 15:24 |
| *** cdearborn has quit IRC | 15:24 | |
| *** praneshp has quit IRC | 15:24 | |
| *** praneshp__ is now known as praneshp | 15:24 | |
| *** suro-patz has joined #openstack-ironic | 15:28 | |
| *** suro-patz has quit IRC | 15:32 | |
| keedya | oh ok! thanks yuriyz | 15:33 |
| *** mgoddard has quit IRC | 15:33 | |
| *** mgoddard has joined #openstack-ironic | 15:34 | |
| *** jaybeale has quit IRC | 15:35 | |
| *** krtaylor has joined #openstack-ironic | 15:37 | |
| keedya | yuriyz: the reason I asked is I see that we are trying to use external DB in inspector(CMDB). Is there a similar plan for sel and termal | 15:43 |
| keedya | In this case, Ironic doesn't need the ipmi credentials and can rely on an external service to parse sel | 15:44 |
| yuriyz | keedya AFAIK no plan for "generic" use but anyone can write a custom driver with management interface that will use any external source (if spec will be approved) | 15:46 |
| keedya | yuriyz: Thanks!!! | 15:49 |
| devananda | g'morning, all | 15:49 |
| NobodyCam | morning devananda :) | 15:49 |
| dtantsur | morning devananda | 15:49 |
| dtantsur | JayF, morning! when you're around, could you please take a look at https://review.openstack.org/289897 ? | 15:50 |
| dtantsur | a bit controversial, but I think we should do it.. | 15:50 |
| dtantsur | devananda, if you have some time ^^ | 15:50 |
| lucasagomes | devananda, morning | 15:53 |
| vdrok | morning devananda | 15:53 |
| * devananda takes a look | 15:54 | |
| devananda | dtantsur: oh, ouch | 15:55 |
| devananda | dtantsur: so that's an upgrade problem, IIUC | 15:56 |
| dtantsur | devananda, yep. both landing this patch and not landing it are upgrade problems | 15:56 |
| devananda | someone migrating from the bash ramdisk to IPA, and either a) issues a rebuild, b) initiates a new deploy without cleaning enabled, is going to have probems | 15:57 |
| devananda | but someone whos been using IPA will be broken by this change unless they're *also* using root-device-hints already | 15:57 |
| dtantsur | devananda, that's what people using tripleo are faced, as tripleo has cleaning disabled.. | 15:57 |
| dtantsur | yep! | 15:57 |
| devananda | shit :( | 15:57 |
| dtantsur | awesome, isn't it? | 15:57 |
| devananda | yah | 15:57 |
| devananda | good time to find it out though | 15:58 |
| devananda | we still have time before RC | 15:58 |
| dtantsur | yep | 15:58 |
| devananda | wdyt about a compatibility mode flag for IPA ? | 15:58 |
| devananda | and some "hey, if you're upgrading, set $this" | 15:58 |
| * dtantsur thinks | 15:59 | |
| *** Sukhdev has joined #openstack-ironic | 15:59 | |
| devananda | or put another way, a flag to IPA that tells it "default to first disk" or "default to smallest disk" | 16:00 |
| openstackgerrit | Merged openstack/ironic: DevStack: Enable VirtualBMC logs https://review.openstack.org/289290 | 16:00 |
| *** suro-patz has joined #openstack-ironic | 16:00 | |
| dtantsur | devananda, rather "default to smallest disk" (the default) or "use the 'disk' argument", right? | 16:00 |
| devananda | not quite | 16:01 |
| devananda | if a root device hint is passed to IPA, it should use that | 16:01 |
| devananda | the issue is when there is no hint passed | 16:01 |
| dtantsur | yeah, ignore root hints for now, they always have priority | 16:01 |
| dtantsur | we can't default to first, cause people might have changed https://github.com/openstack/ironic/blob/master/etc/ironic/ironic.conf.sample#L2012 | 16:02 |
| dtantsur | to e.g. always use "sdb" | 16:02 |
| devananda | IIUC, current IPA default is different than the bash ramdisk default. We could introduce a flag, passed as kernel boot param, that allows tripleo folks (and anyone else upgrading to IPA) to preserve old behavior | 16:02 |
| *** zzzeek has quit IRC | 16:03 | |
| dtantsur | devananda, well, the old behaviour was to respect the "disk" kernel argument, which comes from the "disk_devices" ironic option | 16:03 |
| devananda | ooh righ. wait. is the problem just that IPA isn't using that option // has no equivalent? | 16:03 |
| dtantsur | devananda, yep. it's passed as "disk" kernel argument to both old ramdisk and IPA | 16:03 |
| dtantsur | but IPA ignores it | 16:03 |
| devananda | oh | 16:03 |
| devananda | THAT's the problem | 16:03 |
| devananda | IPA shouldn't be ignorint | 16:04 |
| devananda | *ignoring it | 16:04 |
| *** zzzeek has joined #openstack-ironic | 16:04 | |
| *** harshs has quit IRC | 16:04 | |
| dtantsur | devananda, yeah, that's what my patch fixes. but it changes the default behavior for those who already use IPA.. | 16:04 |
| devananda | dtantsur: I see now. the bug report doesn't say "IPA completely ignores this parameter" | 16:04 |
| dtantsur | my bad | 16:05 |
| dtantsur | devananda, fixed the bug | 16:06 |
| devananda | dtantsur: np. it's early for me. still on my first cup of coffee too :) | 16:07 |
| devananda | dtantsur: much better read now, thanks | 16:07 |
| *** ohamada has joined #openstack-ironic | 16:07 | |
| devananda | dtantsur: if the kernel parameter is not passed in, can IPA behavior remain unchanged from what it is today? | 16:07 |
| devananda | dtantsur: that would solve the upgrade problem for existing IPA users | 16:08 |
| devananda | and allow for bash ramdisk users to upgrade to IPA and preserver their current behavior | 16:08 |
| dtantsur | devananda, 1. yes, 2. not really. With my patch if "disk" is empty, the old logic is used, so the agent_* drivers are not affected. Existing users of the iSCSI family will get affected though... | 16:08 |
| dtantsur | cause we do pass the "disk" parameter now for iSCSI drivers, no matter which ramdisk (obviously) | 16:09 |
| zigo | lucasagomes: pas-ha: If you don't want to waste your time to list every option group in a single file, and keep maintaining it, you may have a look how Cinder does it. They have automation to do that. | 16:09 |
| devananda | dtantsur: gotcha | 16:09 |
| devananda | I don't think I see a way around that :( | 16:10 |
| dtantsur | devananda, seems like there's no easy win, get the 2nd coffee first :) | 16:10 |
| devananda | hehe | 16:10 |
| lucasagomes | zigo, oh good to know | 16:10 |
| lucasagomes | zigo, I will take a look | 16:10 |
| lucasagomes | zigo, can't find it. See https://github.com/openstack/cinder/blob/master/cinder/opts.py | 16:11 |
| dtantsur | devananda, we got have the upgrade procedure the way around: change the behavior and ask the existing IPA users to reset 'disk_devices' to an empty value..... | 16:11 |
| lucasagomes | that's the one file | 16:11 |
| dtantsur | not an optimal solution too, I know... | 16:11 |
| dtantsur | devananda, finally we can have an option to respect "disk" option :) | 16:11 |
| zigo | lucasagomes: Make sure you make it in a packaging friendly way though. ie: remember we don't run tox when building package. What they did is generating opts.py with a script that does introspection, and they have a gate check to make sure that file is up-to-date in the Git. | 16:11 |
| sambetts | dtantsur: can't we add a flag that tells IPA to ignore the disk or not? | 16:12 |
| zigo | (I haven't checked the details, I'm merely and lamely repeating conversations on IRC here...) | 16:12 |
| dtantsur | sambetts, yeah, this is the last option I mention | 16:12 |
| lucasagomes | devananda, dtantsur yeah IPA do not use that option... In order to support IPA with the pxe_* drivers I've merged both pxe templates | 16:12 |
| lucasagomes | dtantsur, devananda so it will contain option used by the bash ramdisk but not by ipa and vice versa (ipa-* options are only used for IPA) | 16:13 |
| lucasagomes | and the reason to do that was to make the change of ramdisk just a drop-in replacement in Ironic | 16:13 |
| dtantsur | that's pretty expected and valid.. except that we forgot about "disk" | 16:13 |
| lucasagomes | dtantsur, yeah, that was required for bash ramdisk | 16:13 |
| lucasagomes | but not for IPA, that's what I'm saying | 16:14 |
| dtantsur | lucasagomes, it's just as required for any of them | 16:14 |
| dtantsur | it was just our disk-choosing logic.. we could go for a smallest drive in the bash disk too | 16:14 |
| lucasagomes | dtantsur, it's not... I understand the incosistency but it was not required | 16:14 |
| lucasagomes | cause IPA do find a disk based on size | 16:14 |
| dtantsur | lucasagomes, yeah, this is THE problem | 16:14 |
| lucasagomes | the bahs do not find any disk if "disk" is not passed | 16:14 |
| lucasagomes | dtantsur, I understand | 16:14 |
| lucasagomes | but wasn't "required" IPA worked that way before we supported it for other drivers | 16:15 |
| dtantsur | right, right | 16:15 |
| dtantsur | and that's why we kind of dropped "disk" on the floor.. | 16:15 |
| lucasagomes | yeah | 16:16 |
| lucasagomes | since we started supporting 2 ramdisks things got more complicated and yeah incosistencies were expected | 16:17 |
| lucasagomes | I'm glad we only use one now | 16:17 |
| *** absubram has joined #openstack-ironic | 16:17 | |
| dtantsur | true :) but this one is pretty bad | 16:17 |
| lucasagomes | but still, we have to iron out the rest of the incosistencies left | 16:17 |
| lucasagomes | yeah | 16:17 |
| dtantsur | so I'm thinking of ipa-root-fallback={smallest,compat} or something like that | 16:18 |
| dtantsur | the former is what we have now (the default), the latter means "respect 'disk' argument" | 16:19 |
| lucasagomes | dtantsur, yeah... that could be one way | 16:21 |
| lucasagomes | dtantsur, if we solve this via root device hint "name" would be another option? Tho it only supports the specific name :-/ | 16:21 |
| lucasagomes | I mean one specific name* | 16:21 |
| dtantsur | ok, lets also wait for devananda not get a required coffeine injection | 16:21 |
| dtantsur | lucasagomes, the problem is in the default behavior. I also think that *everyone* with >1 disks should be using root hints | 16:22 |
| lucasagomes | dtantsur, agreed | 16:22 |
| dtantsur | supporting several values (not only names) in the root device hints is a great idea fwiw | 16:23 |
| lucasagomes | dtantsur, yeah I'll start an RFE for improving root device hints | 16:23 |
| lucasagomes | I want to have operators for size as well | 16:23 |
| lucasagomes | dtantsur, main problem is: we use kernel cmdline to pass that info | 16:24 |
| lucasagomes | now that we only have IPA we may want to use the IPA API to do it | 16:24 |
| dtantsur | ++ | 16:24 |
| lucasagomes | then we can pass more complex structures like a dict | 16:24 |
| lucasagomes | but before, since we had to support the bahs ramdisk, the kernel cmdline was the only interface available :-( | 16:25 |
| lucasagomes | dtantsur, we may end up with a spec for it honestly | 16:25 |
| lucasagomes | there are some forklifiting needed to get it done | 16:25 |
| lucasagomes | fork lifting* | 16:25 |
| *** suro-patz has quit IRC | 16:27 | |
| *** suro-patz has joined #openstack-ironic | 16:28 | |
| *** mbound has quit IRC | 16:32 | |
| *** dims has quit IRC | 16:33 | |
| *** harlowja_at_home has joined #openstack-ironic | 16:34 | |
| *** jaybeale has joined #openstack-ironic | 16:35 | |
| lucasagomes | zigo, gotcha! That's a cool way of doing it | 16:40 |
| *** trown has quit IRC | 16:41 | |
| *** suro-patz has quit IRC | 16:41 | |
| *** jaybeale has joined #openstack-ironic | 16:42 | |
| *** dims has joined #openstack-ironic | 16:44 | |
| *** trown has joined #openstack-ironic | 16:46 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/290644 | 16:47 |
| *** vishwana_ is now known as vishwanathj | 16:48 | |
| *** e0ne has quit IRC | 16:50 | |
| *** trown is now known as trown|brb | 16:52 | |
| *** izaakk has joined #openstack-ironic | 16:52 | |
| *** trown has joined #openstack-ironic | 16:53 | |
| *** mgoddard_ has joined #openstack-ironic | 16:58 | |
| *** jxiaobin has quit IRC | 16:59 | |
| *** Nisha_away has joined #openstack-ironic | 17:01 | |
| *** Nisha has quit IRC | 17:01 | |
| *** mgoddard has quit IRC | 17:02 | |
| *** piet has joined #openstack-ironic | 17:04 | |
| *** trown|brb has quit IRC | 17:05 | |
| *** daemontool has joined #openstack-ironic | 17:05 | |
| *** jistr has quit IRC | 17:06 | |
| *** daemontool_ has quit IRC | 17:07 | |
| *** rcernin has quit IRC | 17:09 | |
| *** ohamada has quit IRC | 17:13 | |
| *** MattMan has quit IRC | 17:17 | |
| *** MattMan has joined #openstack-ironic | 17:17 | |
| gmmaha | TheJulia: thanks for the patch to fix ubuntu images and simple-init.. https://review.openstack.org/#/c/279818/ | 17:19 |
| gmmaha | will run a test and let you know if it works fine | 17:19 |
| *** praneshp has quit IRC | 17:21 | |
| *** Sukhdev has quit IRC | 17:21 | |
| *** Sukhdev has joined #openstack-ironic | 17:22 | |
| *** jcoufal has quit IRC | 17:22 | |
| *** daemontool_ has joined #openstack-ironic | 17:22 | |
| *** daemontool has quit IRC | 17:23 | |
| *** keedya has quit IRC | 17:25 | |
| *** ifarkas has quit IRC | 17:25 | |
| dtantsur | see you tomorrow | 17:25 |
| yhvh | I'm adding an argument to an api method, a put, and wsme.api is complaining about an unknown argument (the new argument), is wsme reading the api to figure out the valid arguments? Grateful for any pointers | 17:26 |
| yhvh | the argument has an underscore in the name if this is significant | 17:26 |
| lucasagomes | dtantsur, have a good night | 17:27 |
| thiagop | see ya dtantsur | 17:27 |
| *** dtantsur is now known as dtantsur|afk | 17:27 | |
| lucasagomes | yhvh, underscore prefixing the name? If so yeah wsme will ignore it AFAIK | 17:27 |
| * lucasagomes I have to check | 17:27 | |
| yhvh | not prefix | 17:28 |
| lucasagomes | yhvh, oh if not prefixed that's fine then | 17:29 |
| lucasagomes | yhvh, can you show me the code? | 17:29 |
| *** harshs has joined #openstack-ironic | 17:30 | |
| yhvh | lucasagomes: sure 2 seconds | 17:30 |
| yhvh | Search for ssh_key http://ix.io/qkD | 17:32 |
| yhvh | it's the put method in NodeConsoleController | 17:32 |
| *** mbound has joined #openstack-ironic | 17:33 | |
| [1]cdearborn | sambetts, thx! I may take you up on the offer of advice! | 17:34 |
| sambetts | :) | 17:34 |
| *** mjturek1 has joined #openstack-ironic | 17:35 | |
| maurosr | sambetts: mjturek1 so just resuming what I was saying in the meeting, there are some details to be merged, currently we apply them on our pre test hook, given our last build (http://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/47/290547/1/check-silent/gate-tempest-dsvm-ironic-pxe_ipmitool/c966bf4/ ) I would say we need this patches no matter which arch | 17:35 |
| maurosr | you're playing with : http://paste.openstack.org/show/489887/ which are not merged | 17:35 |
| mjturek1 | right | 17:36 |
| maurosr | on the link of our run you can also see details of our network creation using providers network | 17:36 |
| TheJulia | gmmaha: awesome, thank you. I've honestly just not had time to context switch back to that :( | 17:37 |
| gmmaha | TheJulia: not a problem.. any which way i can help.. | 17:38 |
| *** mbound has quit IRC | 17:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add documentation for the IPMITool driver https://review.openstack.org/290718 | 17:38 |
| * lucasagomes had this item for ages in his TODO list | 17:39 | |
| sambetts | maurosr: cool thanks, I'll look into those patches | 17:40 |
| lucasagomes | yhvh, looks good to me | 17:40 |
| mjturek1 | maurosr: what about https://review.openstack.org/#/c/263706 | 17:40 |
| lucasagomes | yhvh, how are you requesting that via the API? | 17:40 |
| yhvh | lucasagomes: I have matching changes in the client | 17:40 |
| *** degorenko is now known as _degorenko|afk | 17:42 | |
| lucasagomes | yhvh, ok, try to pass --debug to it so you can look at the request being done to the API, see if it's correct | 17:42 |
| yhvh | lucasagomes: yup looks correct - about to paste | 17:43 |
| lucasagomes | ok | 17:43 |
| *** brud has joined #openstack-ironic | 17:44 | |
| *** brud is now known as fenrir | 17:44 | |
| maurosr | mjturek1: that's not really necessary if we build the ramdisk or use IPA | 17:44 |
| *** fenrir is now known as rbrud | 17:44 | |
| mjturek1 | maurosr: oh right | 17:44 |
| *** mgould has quit IRC | 17:44 | |
| maurosr | we don't build so we don't need a faster guest as controller | 17:45 |
| *** spandhe has joined #openstack-ironic | 17:45 | |
| yhvh | lucasagomes: client debug and the api log http://ix.io/qkU | 17:46 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add documentation for the IPMITool driver https://review.openstack.org/290718 | 17:46 |
| mjturek1 | maurosr: got it | 17:46 |
| *** athomas has quit IRC | 17:47 | |
| *** rbrud has left #openstack-ironic | 17:47 | |
| *** nathan-h has quit IRC | 17:48 | |
| *** rbrud has joined #openstack-ironic | 17:48 | |
| *** Marga__ has quit IRC | 17:49 | |
| *** mbound has joined #openstack-ironic | 17:49 | |
| *** mbound has quit IRC | 17:50 | |
| lucasagomes | yhvh, strange... looks fine to me | 17:51 |
| lucasagomes | yhvh, one thing you can try is to use body= on the expose() decorator | 17:51 |
| lucasagomes | You may need to create an object for the "enabled" and "ssh_key" attributes | 17:52 |
| lucasagomes | and set it on the expose as body=ObjectName | 17:52 |
| * lucasagomes should rename WSME to WSMH (Web Services Made Hard) | 17:53 | |
| sambetts | hahaha | 17:53 |
| *** amotoki has quit IRC | 17:57 | |
| *** ChubYann has quit IRC | 17:58 | |
| *** mgould has joined #openstack-ironic | 17:59 | |
| *** ChubYann has joined #openstack-ironic | 17:59 | |
| lucasagomes | mgould, thanks a lot for the comments in the documentation. I will fix them soonish | 18:03 |
| mgould | np | 18:03 |
| JayF | lucasagomes: dtantsur|afk: At the open compute summit so not really on IRC much; but I put a -1 on that patch. I'm willing to reverse it if backwards compatibility (even if it's months late) trumps architecture, but it's something we should bring up at the meeting before we land, imo -- enumerating disks by device name is an antipattern | 18:06 |
| *** derekh is now known as derekh_afk | 18:06 | |
| lucasagomes | JayF, hi there. Fair enough yeah I understand there may be concerns about that patch | 18:06 |
| lucasagomes | JayF, me and dtantsur|afk briefly talked here on IRC today about it, I think we could improve the root device hints to make it more accurate/easy when choosing a disk | 18:07 |
| lucasagomes | so yeah we can talk about it when you are around | 18:07 |
| JayF | I mean, or just talk about it in the review | 18:07 |
| JayF | async comms active today, sync comms not so much :) | 18:08 |
| lucasagomes | JayF, works for me | 18:08 |
| lucasagomes | I will comment on it, probably tomorrow since my head is a bit tired now | 18:08 |
| * lucasagomes is calling a day very soon | 18:08 | |
| *** Marga_ has joined #openstack-ironic | 18:08 | |
| *** Marga_ has quit IRC | 18:09 | |
| *** daemontool__ has joined #openstack-ironic | 18:10 | |
| *** Marga_ has joined #openstack-ironic | 18:11 | |
| *** daemontool_ has quit IRC | 18:13 | |
| *** mgoddard_ has quit IRC | 18:15 | |
| *** daemontool__ has quit IRC | 18:16 | |
| *** Marga_ has quit IRC | 18:17 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add documentation for the IPMITool driver https://review.openstack.org/290718 | 18:18 |
| lucasagomes | I'm calling it a day, have a great evening all! | 18:18 |
| lucasagomes | talk soon (-: | 18:18 |
| *** lucasagomes is now known as lucas-dinner | 18:18 | |
| *** mkovacik has quit IRC | 18:19 | |
| *** Marga_ has joined #openstack-ironic | 18:22 | |
| *** keedya has joined #openstack-ironic | 18:26 | |
| openstackgerrit | Merged openstack/ironic: Don't use token for glance & check for some unset vars https://review.openstack.org/279270 | 18:28 |
| *** yolanda has quit IRC | 18:28 | |
| *** e0ne has joined #openstack-ironic | 18:29 | |
| *** yolanda has joined #openstack-ironic | 18:29 | |
| gmmaha | TheJulia: with the patch on glean, made a trusty image with simple-init and i didnt have the authorized_keys piped into the system or the hostname.. couldnt test IP as i have a DHCP server running.. | 18:31 |
| gmmaha | but i am guessing ssh keys should be something that should have worked | 18:31 |
| * mgould -> home, good night! | 18:32 | |
| *** mgould has quit IRC | 18:32 | |
| openstackgerrit | Merged openstack/ironic: Add new 'disk_label' capability https://review.openstack.org/283537 | 18:36 |
| *** Nisha_away has quit IRC | 18:37 | |
| *** electrofelix has quit IRC | 18:40 | |
| *** praneshp has joined #openstack-ironic | 18:40 | |
| *** pece has quit IRC | 18:42 | |
| *** dims has quit IRC | 18:44 | |
| *** dims has joined #openstack-ironic | 18:46 | |
| openstackgerrit | Merged openstack/ironic-python-agent: CoreOS: Disable unused services https://review.openstack.org/289419 | 18:50 |
| *** Marga_ has quit IRC | 18:53 | |
| *** Marga_ has joined #openstack-ironic | 18:54 | |
| *** mgoddard has joined #openstack-ironic | 18:58 | |
| vdrok | good night everyone! | 19:01 |
| *** Sukhdev has quit IRC | 19:02 | |
| *** garthb has joined #openstack-ironic | 19:05 | |
| *** trown is now known as trown|lunch | 19:05 | |
| TheJulia | gmmaha: yes, odd.. hmm | 19:09 |
| *** sambetts is now known as sambetts|afk | 19:13 | |
| *** boris-42 has quit IRC | 19:14 | |
| *** mkovacik has joined #openstack-ironic | 19:19 | |
| *** baoli_ has quit IRC | 19:27 | |
| *** cdearborn has joined #openstack-ironic | 19:32 | |
| *** jaybeale has quit IRC | 19:32 | |
| * gmmaha wonders if my way of testing glean patch is flawed... | 19:33 | |
| gmmaha | TheJulia: patched glean upstream and modified /opt/stack/diskimage-builder to use this version.. Verified in the builder logs also that the top patch is yours. | 19:33 |
| *** [1]cdearborn has quit IRC | 19:47 | |
| *** piet has quit IRC | 19:56 | |
| *** piet has joined #openstack-ironic | 19:58 | |
| *** piet has quit IRC | 19:59 | |
| *** piet has joined #openstack-ironic | 20:00 | |
| mrda | thanks johnthetubaguy will do | 20:00 |
| mrda | Morning Ironic | 20:00 |
| TheJulia | good morning mrda | 20:01 |
| openstackgerrit | Devananda van der Veen proposed openstack/bifrost: add scripts/README and tools/README https://review.openstack.org/290819 | 20:01 |
| *** Sukhdev has joined #openstack-ironic | 20:05 | |
| *** garthb has quit IRC | 20:09 | |
| *** mgoddard has quit IRC | 20:12 | |
| *** pas-ha has quit IRC | 20:13 | |
| *** daemontool has joined #openstack-ironic | 20:14 | |
| *** jcoufal has joined #openstack-ironic | 20:15 | |
| mrda | Hi TheJulia! | 20:21 |
| *** snecklifter has quit IRC | 20:26 | |
| *** trown|lunch is now known as trown | 20:27 | |
| *** Sukhdev has quit IRC | 20:29 | |
| *** e0ne has quit IRC | 20:53 | |
| *** thiagop has quit IRC | 21:02 | |
| *** dims has quit IRC | 21:05 | |
| *** alexpilotti has quit IRC | 21:13 | |
| *** alexpilotti has joined #openstack-ironic | 21:15 | |
| *** trown is now known as trown|outtypewww | 21:25 | |
| *** derekh_afk is now known as derekh | 21:29 | |
| *** rbudden has joined #openstack-ironic | 21:30 | |
| *** keedya has quit IRC | 21:32 | |
| *** jaybeale has joined #openstack-ironic | 21:35 | |
| *** baoli has joined #openstack-ironic | 21:37 | |
| *** dims has joined #openstack-ironic | 21:37 | |
| *** dims has quit IRC | 21:56 | |
| *** ndipanov has quit IRC | 21:57 | |
| *** garthb has joined #openstack-ironic | 21:58 | |
| *** dims has joined #openstack-ironic | 21:58 | |
| *** Sukhdev has joined #openstack-ironic | 21:59 | |
| openstackgerrit | Julian proposed openstack/ironic-python-agent: Add __str__ method to BaseCommandResult https://review.openstack.org/290876 | 22:01 |
| *** absubram has quit IRC | 22:06 | |
| *** Sukhdev has quit IRC | 22:14 | |
| *** derekh has quit IRC | 22:19 | |
| *** david-lyle has quit IRC | 22:19 | |
| *** david-lyle has joined #openstack-ironic | 22:20 | |
| *** krtaylor has quit IRC | 22:23 | |
| *** praneshp has quit IRC | 22:25 | |
| *** jcoufal has quit IRC | 22:28 | |
| *** dims has quit IRC | 22:29 | |
| *** krtaylor has joined #openstack-ironic | 22:36 | |
| *** piet has quit IRC | 22:36 | |
| *** piet has joined #openstack-ironic | 22:37 | |
| *** jaybeale has quit IRC | 22:42 | |
| *** jaybeale has joined #openstack-ironic | 22:43 | |
| *** Sukhdev has joined #openstack-ironic | 22:44 | |
| *** dims has joined #openstack-ironic | 22:44 | |
| *** Sukhdev has quit IRC | 22:46 | |
| *** Sukhdev has joined #openstack-ironic | 22:55 | |
| *** baoli has quit IRC | 22:57 | |
| *** saibarscai has joined #openstack-ironic | 22:58 | |
| *** Sukhdev has quit IRC | 23:01 | |
| *** praneshp has joined #openstack-ironic | 23:01 | |
| *** baoli has joined #openstack-ironic | 23:01 | |
| *** garthb_ has joined #openstack-ironic | 23:01 | |
| *** saibarspeis has quit IRC | 23:01 | |
| *** saibarscai is now known as saibarspeis | 23:01 | |
| *** garthb has quit IRC | 23:01 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!