| *** ndipanov has quit IRC | 00:00 | |
| *** linuxgeek has joined #openstack-ironic | 00:03 | |
| *** linuxgeek has quit IRC | 00:03 | |
| *** linuxgeek has joined #openstack-ironic | 00:03 | |
| jlvillal | JayF: See you know much more than I do ;) | 00:04 |
|---|---|---|
| *** dims has joined #openstack-ironic | 00:08 | |
| *** thrash is now known as thrash|g0ne | 00:12 | |
| *** garthb has quit IRC | 00:13 | |
| *** suro-patz has quit IRC | 00:16 | |
| *** openstack has joined #openstack-ironic | 00:25 | |
| *** baoli has joined #openstack-ironic | 00:27 | |
| *** mtanino has quit IRC | 00:28 | |
| *** linuxgeek has quit IRC | 00:42 | |
| *** baoli has quit IRC | 00:42 | |
| *** dims has joined #openstack-ironic | 00:45 | |
| *** smoriya_ has joined #openstack-ironic | 00:48 | |
| *** hoangcx has joined #openstack-ironic | 00:52 | |
| *** phuongnh has joined #openstack-ironic | 00:52 | |
| *** ppiela has quit IRC | 00:56 | |
| *** linuxgeek has joined #openstack-ironic | 00:57 | |
| *** ppiela has joined #openstack-ironic | 00:58 | |
| *** david-lyle has joined #openstack-ironic | 01:04 | |
| *** dims has quit IRC | 01:07 | |
| *** ppiela has quit IRC | 01:08 | |
| *** lintan__ is now known as lintan | 01:09 | |
| *** aNupoisc has quit IRC | 01:10 | |
| *** dims has joined #openstack-ironic | 01:12 | |
| *** izaakk has quit IRC | 01:17 | |
| *** ppiela has joined #openstack-ironic | 01:18 | |
| *** tiendc has joined #openstack-ironic | 01:23 | |
| *** boris-42 has quit IRC | 01:24 | |
| *** suro-patz has joined #openstack-ironic | 01:25 | |
| *** dims has quit IRC | 01:29 | |
| *** dims has joined #openstack-ironic | 01:29 | |
| *** vishwanathj has joined #openstack-ironic | 01:37 | |
| *** harshs has left #openstack-ironic | 01:46 | |
| *** dims has quit IRC | 01:52 | |
| *** mgoddard has quit IRC | 01:54 | |
| *** baoli has joined #openstack-ironic | 01:55 | |
| *** dims has joined #openstack-ironic | 01:56 | |
| *** kan_ has joined #openstack-ironic | 01:56 | |
| *** ppiela has quit IRC | 01:57 | |
| *** ppiela has joined #openstack-ironic | 01:58 | |
| *** mannidi_ has joined #openstack-ironic | 01:59 | |
| *** dims has quit IRC | 02:03 | |
| *** suro-patz has quit IRC | 02:03 | |
| *** dims has joined #openstack-ironic | 02:05 | |
| *** Haomeng has quit IRC | 02:11 | |
| *** baoli_ has joined #openstack-ironic | 02:11 | |
| *** JieLi has joined #openstack-ironic | 02:13 | |
| *** baoli has quit IRC | 02:14 | |
| *** baoli_ has quit IRC | 02:23 | |
| *** baoli has joined #openstack-ironic | 02:23 | |
| *** baoli has quit IRC | 02:29 | |
| *** baoli has joined #openstack-ironic | 02:35 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add ensure_thread_contain_context() to task_manager https://review.openstack.org/284520 | 02:36 |
| openstackgerrit | bin proposed openstack/ironic: Fix VirtualBox cannot set boot device when powered on https://review.openstack.org/290951 | 02:42 |
| *** ppiela has quit IRC | 02:50 | |
| *** ppiela has joined #openstack-ironic | 02:54 | |
| *** krtaylor has quit IRC | 02:58 | |
| *** jaybeale has joined #openstack-ironic | 02:59 | |
| *** spandhe_ has joined #openstack-ironic | 03:04 | |
| *** spandhe has quit IRC | 03:05 | |
| *** spandhe_ is now known as spandhe | 03:05 | |
| *** kan__ has joined #openstack-ironic | 03:08 | |
| *** JieLi has quit IRC | 03:09 | |
| *** krtaylor has joined #openstack-ironic | 03:10 | |
| *** kan_ has quit IRC | 03:11 | |
| openstackgerrit | bin proposed openstack/ironic: Fix VirtualBox cannot set boot device when powered on https://review.openstack.org/290951 | 03:15 |
| *** yuanying has quit IRC | 03:20 | |
| *** kan_ has joined #openstack-ironic | 03:23 | |
| *** kan__ has quit IRC | 03:25 | |
| *** jaybeale has quit IRC | 03:35 | |
| *** dims has quit IRC | 03:37 | |
| *** praneshp_ has joined #openstack-ironic | 03:38 | |
| *** baoli has quit IRC | 03:39 | |
| *** ppiela has quit IRC | 03:44 | |
| *** ppiela has joined #openstack-ironic | 03:45 | |
| *** aswadr_ has joined #openstack-ironic | 03:47 | |
| *** vishwanathj has quit IRC | 04:05 | |
| *** yuanying has joined #openstack-ironic | 04:07 | |
| *** links has joined #openstack-ironic | 04:10 | |
| *** vinm213 has joined #openstack-ironic | 04:11 | |
| openstackgerrit | Merged openstack/ironic: Pass region_name to SwiftAPI https://review.openstack.org/291063 | 04:17 |
| openstackgerrit | Merged openstack/ironic: DevStack: Allow configuring the authentication strategy https://review.openstack.org/292287 | 04:19 |
| *** ppiela has quit IRC | 04:33 | |
| *** ppiela has joined #openstack-ironic | 04:34 | |
| *** mannidi_ has quit IRC | 04:38 | |
| *** sivaramakrishna has joined #openstack-ironic | 04:39 | |
| *** vinm213 has quit IRC | 04:46 | |
| *** ppiela has quit IRC | 04:46 | |
| *** vinm213 has joined #openstack-ironic | 04:50 | |
| *** ChrisAusten has joined #openstack-ironic | 04:53 | |
| *** ppiela has joined #openstack-ironic | 04:54 | |
| *** causten_ has joined #openstack-ironic | 04:57 | |
| *** ChrisAusten has quit IRC | 04:57 | |
| *** ppiela has quit IRC | 05:01 | |
| *** jaybeale has joined #openstack-ironic | 05:03 | |
| *** jaybeale has quit IRC | 05:03 | |
| *** praneshp_ has quit IRC | 05:06 | |
| *** ppiela has joined #openstack-ironic | 05:08 | |
| *** praneshp_ has joined #openstack-ironic | 05:10 | |
| *** ppiela has quit IRC | 05:12 | |
| *** boris-42 has joined #openstack-ironic | 05:13 | |
| *** alex_xu has quit IRC | 05:25 | |
| *** Goneri has quit IRC | 05:25 | |
| *** getvasanth has joined #openstack-ironic | 05:28 | |
| *** vishwanathj has joined #openstack-ironic | 05:31 | |
| *** alex_xu has joined #openstack-ironic | 05:31 | |
| *** vishwanathj has quit IRC | 05:31 | |
| *** david-lyle has quit IRC | 05:45 | |
| *** david-lyle has joined #openstack-ironic | 05:45 | |
| *** vishwanathj has joined #openstack-ironic | 05:48 | |
| openstackgerrit | xiexs proposed openstack/ironic: Use six.text_type() instead of unicode() https://review.openstack.org/293256 | 05:49 |
| *** vishwanathj has quit IRC | 05:51 | |
| *** Nisha has joined #openstack-ironic | 05:54 | |
| *** moshele has joined #openstack-ironic | 05:56 | |
| *** suro-patz has joined #openstack-ironic | 05:59 | |
| *** praneshp_ has quit IRC | 06:01 | |
| *** klindgren has joined #openstack-ironic | 06:06 | |
| *** praneshp_ has joined #openstack-ironic | 06:07 | |
| *** Nisha has quit IRC | 06:08 | |
| *** Nisha_away has joined #openstack-ironic | 06:08 | |
| *** moshele has quit IRC | 06:32 | |
| *** vishwanathj has joined #openstack-ironic | 06:34 | |
| *** vishwanathj is now known as vishwanathj_zzz | 06:35 | |
| *** jtomasek has joined #openstack-ironic | 06:36 | |
| *** praneshp_ has quit IRC | 06:37 | |
| *** harlowja_at_home has quit IRC | 06:41 | |
| *** ChubYann has quit IRC | 06:45 | |
| *** chenke has joined #openstack-ironic | 06:49 | |
| *** ishant has joined #openstack-ironic | 06:50 | |
| *** hoangcx has quit IRC | 06:59 | |
| *** causten_ has quit IRC | 07:00 | |
| *** hoangcx has joined #openstack-ironic | 07:04 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 07:08 |
| *** suro-patz has quit IRC | 07:11 | |
| *** Nisha has joined #openstack-ironic | 07:14 | |
| *** Nisha_away has quit IRC | 07:14 | |
| *** tesseract has joined #openstack-ironic | 07:16 | |
| *** tesseract is now known as Guest64967 | 07:17 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Generic power I/F for soft power off and inject nmi https://review.openstack.org/216730 | 07:20 |
| *** Guest64967 has quit IRC | 07:21 | |
| *** spandhe has quit IRC | 07:24 | |
| *** Marga_ has quit IRC | 07:25 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 07:30 |
| *** user01_ has joined #openstack-ironic | 07:35 | |
| *** tesseract- has joined #openstack-ironic | 07:36 | |
| *** moshele has joined #openstack-ironic | 07:36 | |
| *** Marga_ has joined #openstack-ironic | 07:38 | |
| *** Marga_ has quit IRC | 07:41 | |
| *** Marga_ has joined #openstack-ironic | 07:41 | |
| *** Marga_ has quit IRC | 07:42 | |
| *** chlong has quit IRC | 07:42 | |
| *** Marga_ has joined #openstack-ironic | 07:43 | |
| *** amotoki has joined #openstack-ironic | 07:46 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Overrides get_supported_power_states() for WOL Power https://review.openstack.org/293293 | 07:54 |
| *** mkovacik has joined #openstack-ironic | 07:57 | |
| mkovacik | morning Ironic! | 07:59 |
| *** ppiela has joined #openstack-ironic | 08:12 | |
| *** hoangcx has quit IRC | 08:14 | |
| pas-ha | morning mkovacik and all :) | 08:17 |
| *** hoangcx has joined #openstack-ironic | 08:19 | |
| *** permalac has joined #openstack-ironic | 08:20 | |
| *** ppiela has quit IRC | 08:21 | |
| *** kbyrne has joined #openstack-ironic | 08:22 | |
| Nisha | morning all | 08:23 |
| *** ppiela has joined #openstack-ironic | 08:23 | |
| *** vinm213 has quit IRC | 08:29 | |
| *** ig0r_ has quit IRC | 08:39 | |
| vsaienko | morning Nisha and all | 08:39 |
| Nisha | vsaienko, o/ | 08:39 |
| mkovacik | pas-ha, o/ | 08:40 |
| mkovacik | morning Nisha, vsaienko | 08:41 |
| *** vinm213 has joined #openstack-ironic | 08:42 | |
| *** hparekh_ has quit IRC | 08:42 | |
| Nisha | mkovacik, o/ morning | 08:43 |
| *** hparekh has joined #openstack-ironic | 08:44 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 08:44 |
| *** kan_ has quit IRC | 08:48 | |
| *** ohamada has joined #openstack-ironic | 08:58 | |
| *** ifarkas has joined #openstack-ironic | 09:07 | |
| *** mbound has joined #openstack-ironic | 09:10 | |
| *** mgoddard has joined #openstack-ironic | 09:12 | |
| *** getvasanth has quit IRC | 09:15 | |
| *** getvasanth has joined #openstack-ironic | 09:22 | |
| *** ndipanov has joined #openstack-ironic | 09:22 | |
| *** jistr has joined #openstack-ironic | 09:24 | |
| mkovacik | ifarkas, hi, would you mind reviewing this: https://review.openstack.org/#/c/276107/ bash discovery exercise; already has dtantsur|afk 's +2 | 09:25 |
| ifarkas | mkovacik, o/ sure, will take look | 09:27 |
| mkovacik | ifarkas, thanks! | 09:27 |
| alineb | morning all! | 09:32 |
| *** derekh has joined #openstack-ironic | 09:34 | |
| *** trown has joined #openstack-ironic | 09:38 | |
| *** lucas-dinner is now known as lucasagomes | 09:38 | |
| lucasagomes | morning all | 09:39 |
| *** dtantsur|afk is now known as dtantsur | 09:42 | |
| dtantsur | Morning Ironic, says man drinking coldrex | 09:43 |
| *** amotoki has quit IRC | 09:43 | |
| *** mannidi has joined #openstack-ironic | 09:46 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 09:47 |
| lucasagomes | dtantsur, morning, hope you get better soon | 09:47 |
| dtantsur | me too :D | 09:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 09:48 |
| dtantsur | ifarkas, sambetts|afk, I'd love to fix https://bugs.launchpad.net/ironic-inspector/+bug/1557979 before the release. Objections? | 09:49 |
| openstack | Launchpad bug 1557979 in Ironic Inspector "Prevent DHCP'ing when no introspection is going on and discovery is disabled" [High,Triaged] - Assigned to Dmitry Tantsur (divius) | 09:50 |
| ifarkas | dtantsur, +1 | 09:50 |
| *** dims has joined #openstack-ironic | 09:51 | |
| *** openstackgerrit has quit IRC | 09:53 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:53 | |
| *** openstackgerrit has quit IRC | 09:53 | |
| *** openstackgerrit_ has joined #openstack-ironic | 09:54 | |
| *** openstackgerrit_ is now known as openstackgerrit | 09:54 | |
| *** openstackgerrit_ has joined #openstack-ironic | 09:55 | |
| *** openstackgerrit has quit IRC | 09:55 | |
| *** openstackgerrit has joined #openstack-ironic | 09:56 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Agent: Out-of-band power off on deploy https://review.openstack.org/291829 | 09:58 |
| yuriyz | morning Ironic | 10:07 |
| Nisha | morning yuriyz dtantsur lucasagomes ifarkas alineb | 10:08 |
| *** hoangcx has quit IRC | 10:13 | |
| *** Nisha has quit IRC | 10:13 | |
| *** Nisha_away has joined #openstack-ironic | 10:13 | |
| yuriyz | dtantsur ifarkas if you have some time please look at https://review.openstack.org/#/c/283499/ (staging-drivers). Already has +2 from lucasagomes | 10:13 |
| *** electrofelix has joined #openstack-ironic | 10:15 | |
| aarefiev | morning all! | 10:15 |
| aarefiev | dtantsur: get well soon | 10:16 |
| dtantsur | morning aarefiev, thanks | 10:16 |
| dtantsur | morning yuriyz, Nisha_away | 10:16 |
| *** Nisha_away is now known as Nisha | 10:17 | |
| Nisha | o/ | 10:17 |
| ifarkas | yuriyz, morning! I will try to review the patch today | 10:20 |
| yuriyz | ifarkas thanks | 10:21 |
| sambetts|afk | Morning all o/ | 10:22 |
| *** sambetts|afk is now known as sambetts | 10:22 | |
| sambetts | dtantsur: How do you intend to solve it? | 10:25 |
| sambetts | dtantsur: I think we do need to be more in control of who PXEs on our DHCP server, something I've been considering is whether we need a periodic task to pull every port from Ironic and black list them all | 10:27 |
| sambetts | dtantsur: so that machines that have never been inspected get blacklisted | 10:27 |
| *** daemontool has joined #openstack-ironic | 10:28 | |
| dtantsur | sambetts, we actually know when we 1. don't have any nodes on introspection, 2. don't have a node_not_found_hook set | 10:29 |
| dtantsur | sambetts, in this case we can REJECT everything on the port | 10:29 |
| sambetts | dtantsur: that wouldn't solve for senarios where there is on going introspection for one node and deployment on going for another though | 10:30 |
| *** ohamada has quit IRC | 10:31 | |
| *** ohamada_ has joined #openstack-ironic | 10:31 | |
| dtantsur | sambetts, yep, it won't. I don't think we can solve this case | 10:31 |
| dtantsur | sambetts, I'd say it's pretty unusual though | 10:31 |
| dtantsur | anyway, I mostly want to prevent running instances from getting DHCP | 10:32 |
| sambetts | dtantsur: we need to periodically blacklist all the ports stored in Ironic them too | 10:32 |
| dtantsur | lemme put up a quick patch and we can discuss | 10:32 |
| sambetts | dtantsur: sure :) | 10:32 |
| mkovacik | dtantsur, I think this workaround is valid only for deployers that won't use the enroll the rest will still suffer; should we encourage a partial fix? | 10:34 |
| sambetts | I think the case of having introspection and deployment going on in parallel wouldn't be that unusual in a large environment where you might be adding new hardware | 10:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Disable DHCP completely when no nodes are on introspection https://review.openstack.org/293362 | 10:34 |
| dtantsur | sambetts, well, in a large environment you need to be careful splitting networks. and not allow e.g. 2 NIC's on the provisioning network, which is something that happens in the original bugzilla report | 10:35 |
| dtantsur | mkovacik, you mean, it won't work for people using discovery? right. still, our primary use case is introspection | 10:36 |
| dtantsur | sambetts, mkovacik, also a WIP patch is up ^^ | 10:36 |
| mkovacik | dtantsur, you mean: discovery people: bad luck if you hit this! ;) | 10:37 |
| sambetts | dtantsur: having 2 nics plugged into the provisioning network is something that is potentially unavoidable if you are running a bonded pair \ | 10:37 |
| * mkovacik checks dtantsur's patch | 10:38 | |
| dtantsur | sambetts, not sure how it works though.. will the problem also appear in this case? | 10:38 |
| sambetts | dtantsur: PXE doesn't run bonded so it would appear as 2 nics at that stage AFAIK its only once its loaded into the OS that it can actually bond them into a single interface, but from the switch side you have to program both ports to be identical | 10:39 |
| *** mgould has joined #openstack-ironic | 10:40 | |
| dtantsur | I still think that reducing amount of time we can the DHCP server open is valuable | 10:40 |
| dtantsur | sambetts, aha, got it. interesting, why does people use bonding on provisioning NIC? | 10:40 |
| dtantsur | * do | 10:40 |
| sambetts | dtantsur: the provisioning NIC are the same as what will become the tenant NIC | 10:41 |
| *** phuongnh has quit IRC | 10:42 | |
| dtantsur | sambetts, hmm, interesting... | 10:42 |
| sambetts | dtantsur: Ironic is adding a field pxe_enabled to all ports, which should mean the people can register all their network interfaces into Ironics port table and then select only specific ones for pxe_enabledness, so I think we should have inspector pull the list of ports from Ironic and blanket blacklist every mac we know about, even if a node has never gone through inspection | 10:44 |
| *** mannidi_ has joined #openstack-ironic | 10:44 | |
| dtantsur | sambetts, that's what we do now, no? modulo we blacklist all MAC's | 10:45 |
| dtantsur | sambetts, the problem right now is: you can't enroll all MAC's in Ironic. actually you MUST enroll only one PXE-booting MAC | 10:45 |
| dtantsur | there is no way for inspector to learn the remaining | 10:45 |
| sambetts | dtantsur: I thought we only blacklisted nodes that have inspected ? | 10:45 |
| dtantsur | sambetts, we blacklist all nodes except those inspected | 10:46 |
| sambetts | dtantsur: if I have a mixed env of manally added nodes and inspected nodes do the manually nodes nodes get blacklisted even though they never went through inspection? | 10:46 |
| dtantsur | sambetts, yes | 10:47 |
| sambetts | dtantsur: ah cool | 10:47 |
| dtantsur | sambetts, https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/firewall.py#L131-L132 | 10:47 |
| sambetts | dtantsur: +1 I missed that, I thought we only updated filters for those in our node cache | 10:48 |
| *** mannidi has quit IRC | 10:48 | |
| sambetts | dtantsur: I think that the pxe_enabled field will be our saviour then, because then all NICs will be added to Ironic for all nodes | 10:49 |
| mgould | morning everyone! | 10:49 |
| *** Waltz has joined #openstack-ironic | 10:49 | |
| dtantsur | sambetts, yeah, but it's not happening in mitaka (putting aside liberty) | 10:50 |
| sambetts | dtantsur: and they use the pxe_enabled flag to indicate the which is used for pxe booting, therefore we can know and blacklist all macs | 10:50 |
| dtantsur | morning mgould | 10:50 |
| dtantsur | sambetts, also I'd prefer we avoid providing DHCP to non-ironic nodes too :D | 10:50 |
| sambetts | dtantsur: I think that bit might have merged already | 10:50 |
| dtantsur | sambetts, possibly, but it's not used anywhere essentially, so we can't rely on it for now | 10:51 |
| mgould | dtantsur, sambetts morning! | 10:52 |
| *** dims has quit IRC | 10:53 | |
| sambetts | dtantsur: the providing DHCP to non-ironic nodes is the reason why I would like to make inspection 2 boots. First boot is done as blanket DHCP, literally boots up sends its mac address to inspector so that inspector can blacklist it, then shutdowns. Second boot is done using DHCP options assigned to a specifc mac address, and loads the full inspection image and processing. | 10:54 |
| dtantsur | sambetts, well, we'd still have to load IPA there (that's our only ramdisk).. so not sure what we get by it | 10:55 |
| sambetts | it means that any non-ironic node thats acidentally PXE booted off inspector gets blacklisted and then will never boot from it again unless you poke inspector and say please inspect a node with this mac address | 10:57 |
| dtantsur | interesting idea | 10:57 |
| dtantsur | sambetts, unfortunately it does not solve this case, when e.g. the node gets wrong DHCP from us. It will never try to boot. | 10:58 |
| dtantsur | so I'd prefer we do both eventually | 10:58 |
| dtantsur | (with this fix being the easiest to land) | 10:58 |
| *** pece has joined #openstack-ironic | 10:59 | |
| *** dims has joined #openstack-ironic | 10:59 | |
| sambetts | dtantsur: you could make it reboot after the first boot is done, so the machine then would act like it had never been discovered by inspector | 10:59 |
| dtantsur | sambetts, there are 2 cases: 1. the machine boots our ramdisk instead of the deployment one; 2. the user instance gets a wrong DHCP | 11:00 |
| dtantsur | the latter is not solved by your solution. the former is solved, but only after one retry | 11:00 |
| *** chenke has quit IRC | 11:00 | |
| dtantsur | my solution does not contradict yours: it tries to make probability of the wrong DHCP much lower | 11:00 |
| *** chenke has joined #openstack-ironic | 11:00 | |
| dtantsur | bigger deployments will benefit more from your one, while smaller will probably benefit from my | 11:01 |
| mkovacik | dtantsur, you might want to call it optimisation rather than fix ;) might convince me to +1 it | 11:03 |
| dtantsur | mkovacik, that's fair. I'll explicitly mention in the commit message that this is not an ultimate fix | 11:03 |
| dtantsur | well, if sambetts doesn't -2 it before :D | 11:04 |
| sambetts | dtantsur: when someone does inspection it returns all mac addresses, do we store them in the node cache and blacklist them all or do we only blacklist the ones we store in Ironic? | 11:04 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update ironic_ssh_check method https://review.openstack.org/293373 | 11:04 |
| sambetts | dtantsur: it doesn't cover manaully added nodes, but black listing all returned mac addresses would fix it for inspected nodes right? | 11:04 |
| dtantsur | sambetts, we don't store *introspected* MAC's in node cache | 11:05 |
| dtantsur | we do store all MAC's in Swift FWIW | 11:05 |
| mkovacik | also mgould, aarefiev morning! :) | 11:05 |
| sambetts | dtantsur: ah :/ maybe we should then we could black list all macs we know about from inspection or from Ironic | 11:05 |
| dtantsur | sambetts, to be fair, I'd do everything that we discuss here :) | 11:05 |
| aarefiev | mkovacik: morning | 11:06 |
| dtantsur | i.e. disabling DHCP covers manually enrolled nodes, blacklisting based on introspection data fully covers nodes after inspection | 11:06 |
| dtantsur | sambetts, my only problem with your idea is that we'll have to fetch swift data for all known nodes on every firewall update | 11:06 |
| openstackgerrit | Merged openstack/ironic-inspector: Add discover nodes exercise https://review.openstack.org/276107 | 11:07 |
| dtantsur | so we might need to create one more table to store them | 11:07 |
| sambetts | dtantsur: I would just add another table | 11:07 |
| sambetts | dtantsur: port cache | 11:07 |
| dtantsur | sambetts, +1. but it's probably out of scope for mitaka and liberty... | 11:07 |
| sambetts | dtantsur: I agree | 11:07 |
| dtantsur | as I said, I'd like to do both things | 11:07 |
| mkovacik | sambetts, dtantsur +1 let's do both the things; port cache +1 | 11:07 |
| * dtantsur will change Closes-Bug to Partial-Bug then | 11:08 | |
| sambetts | We should add this a discussion topic for the summit called something like "Optimising and Improving Inspector DHCP and Firewall" | 11:10 |
| *** sivaramakrishna has quit IRC | 11:10 | |
| dtantsur | sambetts, I still dream of handing this job to neutron... | 11:10 |
| dtantsur | but yeah, +1 | 11:10 |
| aarefiev | sambetts: +1, btw any news on neutron integration | 11:11 |
| *** trown is now known as trown|relocate | 11:12 | |
| sambetts | aarefiev: the specs are still in discussion, and the code has been blocked until Newton | 11:13 |
| dtantsur | aarefiev, would be also good to hear from you guys what you need re nailgun replacement | 11:13 |
| aarefiev | sambetts: hope we get it in newton we are interesting | 11:14 |
| aarefiev | dtantsur: actually, I don't know anything about nailgun :) | 11:15 |
| dtantsur | ok, got it :) | 11:15 |
| aarefiev | waiting for nailgun folks answers | 11:15 |
| sambetts | aarefiev: yeah I think it would be good, the functionality is split into 2 parts, subnet/network level DHCP options, and then serving IPs to Unknown mac addresses | 11:16 |
| sambetts | aarefiev: the first part I think is something that is likely to go in, they aren't so happy with my design for the second part, I think I may need to rework it | 11:17 |
| *** openstackgerrit has quit IRC | 11:18 | |
| aarefiev | sambetts: if you need help I think I can ask somebody from neutron | 11:18 |
| *** openstackgerrit has joined #openstack-ironic | 11:18 | |
| aarefiev | also I need some time to dig in it by myself | 11:19 |
| *** vaishnavi has joined #openstack-ironic | 11:19 | |
| sambetts | aarefiev: :) its had quite a bit of interest so I just need to sit down and really focus on the specs | 11:19 |
| *** kalpase has joined #openstack-ironic | 11:19 | |
| sambetts | aarefiev: I've had a bunch of other stuff to focus on recently with all the third party CI and neutron intergration work | 11:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Disable DHCP completely when no nodes are on introspection https://review.openstack.org/293362 | 11:20 |
| dtantsur | sambetts, mkovacik ^^ | 11:20 |
| TheJulia | good morning | 11:20 |
| dtantsur | morning TheJulia | 11:20 |
| *** deray has joined #openstack-ironic | 11:20 | |
| aarefiev | sambetts: ok :) | 11:20 |
| aarefiev | morning TheJulia | 11:20 |
| *** deray_ has joined #openstack-ironic | 11:20 | |
| *** deray_ has quit IRC | 11:20 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update ironic_ssh_check method https://review.openstack.org/293373 | 11:22 |
| *** deray_ has joined #openstack-ironic | 11:24 | |
| dtantsur | aarefiev, do you think we can close https://bugs.launchpad.net/ironic-inspector/+bug/1524753 ? | 11:24 |
| openstack | Launchpad bug 1524753 in Ironic Inspector "[RFE] In-band discover ironic nodes" [Wishlist,In progress] - Assigned to Anton Arefiev (aarefiev) | 11:24 |
| *** deray has quit IRC | 11:25 | |
| mkovacik | dtantsur, I've added some comments; will test it locally in a bit; lunch time now ;) | 11:25 |
| aarefiev | dtantsur: why not, all patches landed | 11:26 |
| *** deray_ is now known as deray | 11:26 | |
| dtantsur | mkovacik, thanks. I'm relying on the gate to test it this time | 11:26 |
| mkovacik | dtantsur, np; btw https://review.openstack.org/#/c/253675/13/specs/HA_inspector.rst has an update ;) | 11:27 |
| dtantsur | :) | 11:28 |
| mkovacik | aarefiev, ^^ not sure you've read through it already ;) | 11:28 |
| mkovacik | mind the locks are gone | 11:28 |
| * dtantsur will check soon | 11:28 | |
| mkovacik | dtantsur, I'd focus on pushing Tooz parts (queue/task/job whatever) if we get this approved | 11:29 |
| aarefiev | sure, I'll take look, interesting topic for discuss on summit | 11:29 |
| dtantsur | we'll definitely discuss it on the summit | 11:29 |
| mkovacik | aarefiev, dtantsur right, although it would be nice to discuss about something we've seen already ;) | 11:30 |
| dtantsur | ifarkas, please also check https://review.openstack.org/293362 when you have some time.. I'd love it to get in mitaka (actually also liberty) | 11:30 |
| * mkovacik not being pushy at all ;) | 11:30 | |
| ifarkas | dtantsur, ok | 11:31 |
| *** dtantsur is now known as dtantsur|brb | 11:34 | |
| *** vaishnavi has quit IRC | 11:40 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Adds doc - firmware update(iLO) manual clean step https://review.openstack.org/276765 | 11:42 |
| *** moshele has quit IRC | 11:43 | |
| Nisha | lucasagomes, dtantsur|brb https://review.openstack.org/160224 IPA patch for partition image | 11:43 |
| *** ishant has quit IRC | 11:43 | |
| deray | hello g'morning all Ironicers! | 11:49 |
| *** moshele has joined #openstack-ironic | 11:49 | |
| deray | lucasagomes, hi | 11:51 |
| deray | lucasagomes, regarding the Retry-After comment, I don't have the full context and knowledge behind that decision.. please see: https://specs.openstack.org/openstack/ironic-specs/specs/approved/manual-cleaning.html#get-nodes-node-ident-cleaning-steps | 11:53 |
| deray | "the driver MAY raise an error, which should be translated by the API service into: | 11:54 |
| deray | an HTTP 202" | 11:54 |
| lucasagomes | deray, hi there, oh right. But retry_after != Retry-Request-After right? | 11:55 |
| lucasagomes | this Retry-Request-After seems to be something custom in the API | 11:55 |
| lucasagomes | where Retry-After is a known header used for 503 | 11:55 |
| lucasagomes | I don't think we should use that for 202 | 11:55 |
| lucasagomes | it's part of the rfc https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html | 11:56 |
| lucasagomes | the 202 for error also doesn't make much sense, cause that's not an error code | 11:56 |
| lucasagomes | deray, I believe we should update the spec. | 11:56 |
| lucasagomes | jroll, ^ wdyt? | 11:57 |
| deray | lucasagomes, oh.. I thought it was misjudged to be Retry-Req-After instead of Retyr-After | 11:57 |
| deray | lucasagomes, its shudn't be an error, tho, I feel | 11:58 |
| lucasagomes | deray, right. So the problem is that when you do a GET on the clean_steps they may not be there at the moment correct? | 11:59 |
| deray | lucasagomes, ya | 11:59 |
| deray | right | 11:59 |
| lucasagomes | so they clean steps were not found. I feel like this should be a 404 no? | 11:59 |
| lucasagomes | I mean the clean_steps are not created yet or something like that | 11:59 |
| deray | lucasagomes, Not-Found | 11:59 |
| *** tiendc has quit IRC | 12:00 | |
| deray | lucasagomes, lemme chk what 3** codes refer to in http protocols | 12:00 |
| *** raildo-afk is now known as raildo | 12:00 | |
| lucasagomes | deray, to redirect the request | 12:00 |
| *** pece has quit IRC | 12:01 | |
| lucasagomes | for me 404 fits very well in that case | 12:01 |
| lucasagomes | 404 Not Found | 12:01 |
| lucasagomes | The requested resource could not be found but may be available again in the future. Subsequent requests by the client are permissible. | 12:01 |
| *** dtantsur|brb is now known as dtantsur | 12:01 | |
| dtantsur | lucasagomes, deray, 404 should be "node not found" in this case, not "result temporary unavailable" | 12:03 |
| lucasagomes | dtantsur, why node not found? I mean the node do exist | 12:03 |
| dtantsur | lucasagomes, then you should not return 404 | 12:04 |
| lucasagomes | it can be the case of the node not existing, then yes it's not not found | 12:04 |
| lucasagomes | dtantsur, why? | 12:04 |
| lucasagomes | dtantsur, it's trying to access a subresource that do not exist | 12:04 |
| dtantsur | lucasagomes, 404 means "node does not exist", if I remember this right | 12:04 |
| dtantsur | * the spec | 12:04 |
| deray | lucasagomes, dtantsur dont know y but 'm still not very convinced with 404 | 12:04 |
| lucasagomes | dtantsur, 404 == resource not found | 12:04 |
| lucasagomes | it can be port, chassis etc | 12:04 |
| lucasagomes | it's not node | 12:04 |
| lucasagomes | I mena, it's not specific for node | 12:04 |
| dtantsur | of course. but note the clean steps temporary unavailable | 12:04 |
| dtantsur | hmm, maybe I'm not too clear today | 12:05 |
| lucasagomes | correct, so at the moment of the request (it's a GET, so synchrouous) the resource is not there | 12:05 |
| *** _degorenko|afk is now known as degorenko | 12:05 | |
| lucasagomes | but as the description of 404 says: "but may be available again in the future. Subsequent requests by the client are permissible." | 12:05 |
| dtantsur | "In the absence of such data, the driver MAY raise an error, which should be translated by the API service into" <-- -2 to using 404 here | 12:05 |
| dtantsur | that's my point, ignore everything else :) | 12:05 |
| lucasagomes | dtantsur, oh right, the spec wording | 12:06 |
| dtantsur | the reason is that it won't be distinguishable from node not found | 12:06 |
| deray | lucasagomes, this specifically mentions that 4xx error codes are meant primarily for Client Error: https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html | 12:06 |
| dtantsur | meaning that people will report bugs on us saying "get clean step" suddenly returned "node not found" (meaning 404) | 12:06 |
| dtantsur | deray++ | 12:07 |
| lucasagomes | dtantsur, right yeah we need to sort out that message | 12:07 |
| dtantsur | also not sure what RFC says, but people never actually retry error 404 | 12:07 |
| *** baoli has joined #openstack-ironic | 12:07 | |
| dtantsur | we have enough experience with too broad error 409. lets not repeat the same mistake | 12:08 |
| deray | dtantsur, ^^ thats also a valid point | 12:08 |
| lucasagomes | deray, right, 4xx for client errors 5xxx for server errors, by trying to access a resource that is not there is a client error | 12:08 |
| dtantsur | lucasagomes, it is there | 12:08 |
| dtantsur | we just can't display it | 12:08 |
| dtantsur | if you mean clean steps of course | 12:08 |
| *** pece has joined #openstack-ironic | 12:09 | |
| lucasagomes | dtantsur, I mean the clean steps, it's being sync'ed so it's not there at the moment that's what I mean | 12:09 |
| lucasagomes | GET == sync operation | 12:09 |
| dtantsur | lucasagomes, "at the moment" is a critical thing here | 12:09 |
| dtantsur | "the resource is there, but we can't access it right now" | 12:09 |
| dtantsur | it's not 404 | 12:09 |
| *** pece has quit IRC | 12:10 | |
| lucasagomes | dtantsur, the same applies for all 404 in that logic | 12:10 |
| lucasagomes | the resource is not there but well cna be created in the future... | 12:10 |
| dtantsur | lucasagomes, no. node is not found, means there is no node. not matter how you try | 12:10 |
| deray | lucasagomes, dtantsur https://review.openstack.org/#/c/278566/3/ironic/api/controllers/v1/node.py@957 | 12:10 |
| dtantsur | we don't "create" clean steps. they do exist | 12:11 |
| lucasagomes | dtantsur, I think we either do a 404 or we return an 200 with an empty list | 12:11 |
| lucasagomes | I don't see how a GET can return 202 | 12:11 |
| dtantsur | even 200 with an empty list is better... | 12:11 |
| dtantsur | or not. well, both 200 and 404 will result in a big confusion | 12:11 |
| lucasagomes | + Retry-After which should be used for 503 errors | 12:11 |
| deray | lucasagomes, dtantsur it says "If the steps are not yet available (e.g. because the agent isn't on the node yet), an HTTP 202 is returned with Retry-After | 12:12 |
| deray | header. This indicates how long, in seconds, the client should | 12:12 |
| deray | wait before retrying. -1 is unknown amount of time." | 12:12 |
| dtantsur | to be fair, returning 503 would not be the worst idea.. but I've heard proxies tend to cache it for the whole server | 12:12 |
| dtantsur | I kind of agree that 202 is not commonly used with GET | 12:12 |
| *** pece has joined #openstack-ironic | 12:13 | |
| dtantsur | lucasagomes, deray, the more I think about it, the more I think we should not create such API at all | 12:13 |
| dtantsur | rather just provide the field on node which populates as soon as we get the information.... | 12:13 |
| deray | dtantsur, but from the operator's perspectiev she has to know whcih clean steps (manual) are available | 12:14 |
| deray | dtantsur, okay.. | 12:15 |
| dtantsur | deray, documentation to the rescue? Otherwise an API that returns error 404 half of the time does not sound useful | 12:15 |
| *** miles has joined #openstack-ironic | 12:15 | |
| deray | dtantsur, u mean driver_internal_info | 12:15 |
| lucasagomes | yeah, it's always hard to indicate async operations in apis :-/ | 12:15 |
| lucasagomes | deray, when do the steps gets populated? | 12:16 |
| dtantsur | lucasagomes, async POST is fine.. async GET is weird | 12:17 |
| deray | dtantsur, hmm.. thats also true. but for an operator with multiple vendor drivers need to collate the steps from different docs :) | 12:17 |
| lucasagomes | dtantsur, right | 12:17 |
| lucasagomes | dtantsur, but even the post in our API, since we don't really have a good way to track progress | 12:18 |
| *** mgould has quit IRC | 12:18 | |
| lucasagomes | for some stuff we do, like the target_* states are for that | 12:18 |
| dtantsur | lucasagomes, at least we have HTTP code 202... yeah, I agree. to be honest, HTTP is not an ideal fit for API | 12:18 |
| lucasagomes | yeah | 12:18 |
| lucasagomes | deray, do we have an API call to trigger Ironic to populate that clean_steps? | 12:19 |
| dtantsur | lucasagomes, deray, we could change the meaning of the API. I.e. from "get the steps" to "please update the steps" | 12:19 |
| * lucasagomes re-read the spec | 12:19 | |
| dtantsur | the latter would be a POST with 202, and will be always asynchronous | 12:19 |
| deray | lucasagomes, when do the steps gets populated? after soem tiem from conductor statup .. when the agents are all up | 12:19 |
| dtantsur | it's result will be stored on node or in driver_internal_info | 12:19 |
| deray | lucasagomes, do we have an API call to trigger Ironic to populate that clean_steps? No | 12:19 |
| deray | it gets automatically populated when the conductor starts up .. | 12:20 |
| lucasagomes | deray, right, I tend to think that returning the empty list (for no discovered steps) or a partial list (for steps that can be discovered synchronously) is fine | 12:25 |
| lucasagomes | because that endpoint will never be the source of truth right? If you say update the ramdisk afterwards the steps that are cached may not reflect all the steps available | 12:26 |
| lucasagomes | so it sounds like those are the "steps discovered at the moment the request arrived" | 12:26 |
| dtantsur | sambetts, https://review.openstack.org/#/c/293362/ works in gate and in my testing, please check | 12:26 |
| *** thrash|g0ne is now known as thrash | 12:27 | |
| *** ppiela has quit IRC | 12:27 | |
| *** pcaruana has quit IRC | 12:28 | |
| *** boris-42 has quit IRC | 12:34 | |
| *** thiagop has joined #openstack-ironic | 12:34 | |
| *** trown|relocate is now known as trown | 12:35 | |
| *** lucasagomes is now known as lucas-hungry | 12:35 | |
| * lucas-hungry brb lunch | 12:35 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic-specs: Adding InfiniBand Support to ironic https://review.openstack.org/265635 | 12:38 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: this is a test to check dhcp new job https://review.openstack.org/293402 | 12:38 |
| *** thrash has quit IRC | 12:40 | |
| *** ppiela has joined #openstack-ironic | 12:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Update links to existing documentation https://review.openstack.org/293406 | 12:42 |
| *** thrash has joined #openstack-ironic | 12:44 | |
| *** thrash has quit IRC | 12:44 | |
| *** thrash has joined #openstack-ironic | 12:44 | |
| * deray looks into code to understand the complete flow of getting clean steps | 12:44 | |
| *** ramishra has joined #openstack-ironic | 12:50 | |
| *** ppiela has quit IRC | 12:50 | |
| *** alexpilotti has joined #openstack-ironic | 12:51 | |
| *** jlvillal has quit IRC | 12:52 | |
| *** jlvillal has joined #openstack-ironic | 12:52 | |
| *** Goneri has joined #openstack-ironic | 12:53 | |
| deray | lucas-hungry, dtantsur this is the point when the agent scouts for the clean steps: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L450 | 12:53 |
| *** ppiela has joined #openstack-ironic | 12:54 | |
| deray | lucas-hungry, dtantsur And the clean steps are cached to enable the ``get_clean_steps`` call synchronous: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L223-L226 | 12:54 |
| *** vinm213 has quit IRC | 12:58 | |
| mkovacik | dtantsur, should we change the discovery gw to voting? and how to do that? ;) | 12:59 |
| *** Goneri has quit IRC | 12:59 | |
| *** smoriya_ has quit IRC | 12:59 | |
| *** Goneri has joined #openstack-ironic | 13:00 | |
| *** dprince has joined #openstack-ironic | 13:02 | |
| dtantsur | mkovacik, I want to it to be non-voting for a while, just in case. Then it's an update to project-config. I can do it, if you're not familiar with our infra. | 13:02 |
| mkovacik | dtantsur, ok, cool | 13:03 |
| *** rcernin has joined #openstack-ironic | 13:03 | |
| *** e0ne has joined #openstack-ironic | 13:10 | |
| *** Nisha_away has joined #openstack-ironic | 13:14 | |
| *** Nisha has quit IRC | 13:14 | |
| *** kro_sinus_hell is now known as krot_sinus_purga | 13:15 | |
| *** krot_sinus_purga is now known as kro_sniffles | 13:15 | |
| *** mannidi_ has quit IRC | 13:18 | |
| *** ig0r_ has joined #openstack-ironic | 13:24 | |
| *** lucas-hungry is now known as lucasagomes | 13:26 | |
| *** e0ne has quit IRC | 13:27 | |
| *** cdearborn has joined #openstack-ironic | 13:31 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: this is a test to check dhcp new job https://review.openstack.org/293402 | 13:33 |
| openstackgerrit | Merged openstack/ironic-inspector: Disable DHCP completely when no nodes are on introspection https://review.openstack.org/293362 | 13:34 |
| dtantsur | good. now I need to backport this to liberty. will be a fun exercise... | 13:34 |
| *** d0ugal has quit IRC | 13:35 | |
| *** d0ugal has joined #openstack-ironic | 13:35 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Add scenario for testing ironic without glance https://review.openstack.org/293434 | 13:35 |
| thiagop | Good morning Ironicville | 13:36 |
| *** ishant has joined #openstack-ironic | 13:37 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update ironic_ssh_check method https://review.openstack.org/293373 | 13:38 |
| NobodyCam | good morning Ironicers :) | 13:41 |
| lucasagomes | thiagop, NobodyCam morning | 13:41 |
| thiagop | NobodyCam: good morning | 13:41 |
| sambetts | o/ NobodyCam | 13:42 |
| dtantsur | morning thiagop, NobodyCam | 13:43 |
| NobodyCam | morning lucasagomes thiagop sambetts dtantsur jroll devananda rloo jlvillal gabriel-bezerra and all others not listed here :) | 13:43 |
| thiagop | dtantsur: o/ | 13:43 |
| *** spandhe has joined #openstack-ironic | 13:46 | |
| *** athomas has quit IRC | 13:51 | |
| jroll | mornin. | 13:52 |
| sambetts | Hey jroll | 13:52 |
| jroll | dtantsur: lucasagomes: deray: I do think we need the API, honestly, I'm not sure a 202 on a GET is too much of an api abuse | 13:53 |
| jroll | maybe we bug some api wg people or something to see if they have thoughts? | 13:53 |
| dtantsur | morning jroll | 13:53 |
| * jroll wonders what other APIs might have async GETs | 13:53 | |
| dtantsur | ++ for pinging API folks | 13:53 |
| lucasagomes | jroll, yeah that sounds like a good idea | 13:53 |
| deray | jroll, g'morning | 13:53 |
| yolanda | hi TheJulia, i'm looking at first pass on https://jenkins04.openstack.org/job/gate-bifrost-integration-dhcp-nv/1/console .. it doesn't look promising , taking a long time waiting for the VMS. But i think i discovered one problem, dnsmasq needs to be restarted to pick configuration of bifrost.dhcp-hosts, otherwise is not offering the leases | 13:53 |
| thiagop | morning jroll | 13:54 |
| *** mtanino has joined #openstack-ironic | 13:54 | |
| deray | NobodyCam, thiagop sambetts morning | 13:55 |
| thiagop | deray: o/ | 13:56 |
| *** athomas has joined #openstack-ironic | 13:56 | |
| NobodyCam | deray: morning :) | 13:56 |
| deray | jroll, lucasagomes dtantsur but is the API an async call? it just sees if the agent has the cached result and returns it, if not then .. our thoughts pondering | 13:57 |
| jroll | if not then asyncronously fetch them :) | 13:58 |
| deray | thiagop, NobodyCam :) | 13:58 |
| lucasagomes | jroll, deray yeah that's why maybe returning an empty list or partial list should be fine | 13:58 |
| lucasagomes | jroll, deray because that endpoint is not the source of truth right? | 13:58 |
| lucasagomes | in case one updates the deploy ramdisk, it may introduce new cleaning steps that are not listed there | 13:59 |
| cinerama | hi yolanda if you want to propose a patch to add that restart that'd be great | 13:59 |
| jroll | lucasagomes: I'm in #openstack-dev with sean btw | 13:59 |
| lucasagomes | so maybe we should consider the endpoint as "get me the discovered clean steps discovered at the moment" | 13:59 |
| jroll | if you're curious | 13:59 |
| yolanda | cinerama, it actually restars the service, but i'm thinking about some races | 14:00 |
| * lucasagomes joins | 14:00 | |
| deray | in case one updates the deploy ramdisk then wont the operator wait till the agent heart beat to trigger that API call | 14:00 |
| yolanda | test ended in failure for timeout | 14:00 |
| jroll | lucasagomes: to catch you up, http://eavesdrop.openstack.org/irclogs/%23openstack-dev/%23openstack-dev.2016-03-16.log.html | 14:00 |
| jroll | see the bottom | 14:00 |
| lucasagomes | thanks, looking | 14:00 |
| *** mgoddard_ has joined #openstack-ironic | 14:01 | |
| Goneri | hi, can someone take a look on this review (to make use of wipefs): https://review.openstack.org/#/c/286283/ | 14:02 |
| *** mgoddard has quit IRC | 14:05 | |
| *** mgagne has quit IRC | 14:07 | |
| *** mgagne has joined #openstack-ironic | 14:07 | |
| yolanda | cinerama, TheJulia, i holded the node that failed to check... | 14:08 |
| yolanda | i see an entry /etc/dnsmasq.d-available/libvirt-bin with | 14:08 |
| yolanda | cat /etc/dnsmasq.d-available/libvirt-bin | 14:08 |
| yolanda | bind-interfaces | 14:08 |
| yolanda | except-interface=virbr0 | 14:08 |
| *** kalpase has left #openstack-ironic | 14:08 | |
| yolanda | and looking at logs, this seems to be the interface used on our testing right? | 14:08 |
| TheJulia | yolanda: yes | 14:09 |
| *** absubram has joined #openstack-ironic | 14:11 | |
| *** Nisha_away is now known as Nisha | 14:11 | |
| *** absubram_ has joined #openstack-ironic | 14:12 | |
| yolanda | TheJulia, but that's not provided by bifrost right? | 14:12 |
| yolanda | i mean that configuration about ignoring virbr | 14:12 |
| TheJulia | no, that doesn't come from bifrost. :\ | 14:13 |
| TheJulia | afaik, and the other job seemingly worked :\ | 14:13 |
| yolanda | which job? | 14:14 |
| *** chenke has quit IRC | 14:14 | |
| TheJulia | the gate-bifrost-integration-coreosipa-cirros | 14:14 |
| *** chenke has joined #openstack-ironic | 14:14 | |
| yolanda | ah well, but is normal that mine doesn't work, because i'm tweaking dhcp there | 14:15 |
| yolanda | so that will behave differently i guess? | 14:15 |
| *** absubram has quit IRC | 14:15 | |
| *** absubram_ is now known as absubram | 14:15 | |
| TheJulia | so in the main config file we bind explicitly to virbr0 | 14:16 |
| TheJulia | thats just a libvirt default if memory serves | 14:16 |
| TheJulia | and I have the same config in place on my test machine | 14:16 |
| yolanda | mm, it may not be the problem then | 14:16 |
| yolanda | i also have the same | 14:16 |
| yolanda | fact is that in that test, dnsmasq doesn't seem to be providing the leases | 14:17 |
| yolanda | i have a node holded, do you want to look? | 14:17 |
| yolanda | last entries in ironic conductor are timeouts waiting for callback | 14:17 |
| TheJulia | i would love to take a look | 14:19 |
| TheJulia | because looking thte collected logs, it does appear that packets were received on the interface that matched dhcp :\ | 14:20 |
| *** Lydia_K has joined #openstack-ironic | 14:22 | |
| *** Lydia_K has left #openstack-ironic | 14:24 | |
| *** aswadr_ has quit IRC | 14:30 | |
| jroll | lucasagomes: so, all that said, I almost think this should wait for newton, since we won't have the client done for mitaka anyway | 14:31 |
| lucasagomes | jroll, yeah I don't think that this endpoint will invalidate the feature at all | 14:33 |
| lucasagomes | jroll, I'm good waiting | 14:33 |
| jroll | yeah +1 | 14:33 |
| lucasagomes | jroll, we can then talk about how the trigger will work etc, perhaps a spin-off spec from the main spec for the API bits | 14:33 |
| lucasagomes | and we can track the discussion there | 14:33 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 14:34 |
| *** rpioso has joined #openstack-ironic | 14:34 | |
| klindgren | Hello, I waa wondering. I was using ironic using the pxe_ipmitool configured with ipxe. I wanted to switch to using agent_ based so that the config drive would be created. However, It seems like ipxe deployment is not supported using agent_ipmitool on under pxe_ipmitool? | 14:34 |
| TheJulia | klindgren: agent + ipxe works really well from my experience | 14:35 |
| TheJulia | what exactly are you seeing? | 14:35 |
| klindgren | I am using kilo right now, when I attempted it - ironic never configured the next target correctl;y. so ipxe would start up but wouldn't be able to download the initrd and kernel for the ipa image | 14:36 |
| deray | jroll, lucasagomes eha .. followed the discussion in #openstack-dev | 14:39 |
| lucasagomes | deray, cool, feel free to jump in as well if you feel so | 14:39 |
| klindgren | I tried updating the template used from the agent_config.template to ipxe_config.template but then I ran into it was unable to find the the initrd because the path was configured wrong as ironic didn't do the work under the httpboot dir but instead the tftpboot dir | 14:40 |
| *** absubram has quit IRC | 14:40 | |
| klindgren | eg: agent.py is missing: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L251-L285 | 14:42 |
| *** absubram has joined #openstack-ironic | 14:43 | |
| *** mgoddard_ has quit IRC | 14:47 | |
| *** mgoddard has joined #openstack-ironic | 14:47 | |
| lucasagomes | klindgren, I don't really remember the state of things on Kilo :-( | 14:52 |
| klindgren | so TheJulia unless I am missing something it seems like its jsut a missing feature of agent_ deployments? which is sad because it takes pretty long time to tftp the ipa image/kernel compared to ipxe. At least a minute or more longer. | 14:52 |
| lucasagomes | tho I remember that the work to have pxe_* drivers using IPA rmadisk was done there | 14:52 |
| lucasagomes | and ipxe was introduced in Juno | 14:52 |
| *** deray has quit IRC | 14:52 | |
| klindgren | lucasagomes, well thats from master - I still dont see anything that would make ipxe work under agent_ based stuff as its missing the code to set any of the pxe things. However, I am also not seeing the pxe options being set under the agent.py either - so I have a feeling I am missing something | 14:53 |
| lucasagomes | klindgren, right, yeah it may be missing bits and pieces in kilo indeed | 14:53 |
| lucasagomes | it's a long time so I don't know off the top of my head what is missing | 14:54 |
| klindgren | (I am looking at master?) | 14:54 |
| lucasagomes | klindgren, that github link is from master yes | 14:55 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Remove generated default.leases file for libvirt https://review.openstack.org/293402 | 14:55 |
| TheJulia | I remember bifrost was boot loading ipxe when I first got it going, but we were also using a custom dnsmasq config to do it :\ | 14:56 |
| TheJulia | yolanda: crossing my fingers hoping it works :) | 14:56 |
| yolanda | let's pray to the dnsmasq gods :) | 14:56 |
| TheJulia | what if we offer them cookies instead? | 14:56 |
| lucasagomes | TheJulia, yeah, we had to introduce a patch in neutron creating the "ipxe" label and all | 14:56 |
| TheJulia | lucasagomes: that makes sense | 14:57 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Agent: Out-of-band power off on deploy https://review.openstack.org/291829 | 15:02 |
| cinerama | are brownies acceptable? i have brownies | 15:02 |
| thiagop | cinerama: send a box to me, please | 15:02 |
| * thiagop is the Brownie Monster | 15:02 | |
| lucasagomes | klindgren, btw you said configdrive. Are you deploying whole disk images? | 15:05 |
| *** garthb has joined #openstack-ironic | 15:05 | |
| klindgren | lucasagomes, TheJulia it looks like it was a change in Liberty where they refactored the agent.py to use the task.driver.boot.prepare_ramdisk which loads from the pxe driver | 15:05 |
| klindgren | lucasagomes, yes we are deploying whole disk images | 15:06 |
| lucasagomes | klindgren, right :-( | 15:06 |
| klindgren | I will look to see if I can pull in: https://github.com/openstack/ironic/commit/72b20fec757094031e770ee04ca9b023a5a6a44e and some of the other need patches - thanks! | 15:08 |
| lucasagomes | klindgren, cool | 15:10 |
| lucasagomes | yw | 15:10 |
| BadCub | mornign Ironic | 15:10 |
| NobodyCam | good morning BadCub :) | 15:11 |
| *** klindgren has quit IRC | 15:11 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers https://review.openstack.org/162008 | 15:12 |
| Nisha | NobodyCam, TheJulia lucasagomes jroll dtantsur need reviews on https://review.openstack.org/160224 and https://review.openstack.org/162008 (partition image patches) | 15:13 |
| yolanda | TheJulia, doesn't look promising https://jenkins03.openstack.org/job/gate-bifrost-integration-dhcp-nv/1/consoleFull | 15:13 |
| yolanda | i have to run for some errands now but i'll check later | 15:14 |
| *** david-lyle has quit IRC | 15:15 | |
| *** david-lyle has joined #openstack-ironic | 15:15 | |
| *** mbound has quit IRC | 15:22 | |
| thiagop | morning BadCub | 15:25 |
| *** mjturek1 has joined #openstack-ironic | 15:27 | |
| *** moshele has quit IRC | 15:29 | |
| mjturek1 | hey I deleted a nova instance that went to ERROR (missing some parameters in the ironic node) that was already associated with an ironic node. After successfully deleting it from nova, it stayed associated with the ironic node. Now I can't delete the ironic node, is there any way to force an instance disassociation from a node? | 15:30 |
| sambetts | mjturek1: if you do ironic node-set-provision-state <node uuid> active | 15:30 |
| sambetts | then ironic node-set-provision-state <node uuid> deleted | 15:31 |
| sambetts | it should clean up | 15:31 |
| jroll | eh... I'd go for ironic node-update <node> remove instance_uuid | 15:31 |
| jroll | maybe instance_info too | 15:31 |
| mjturek1 | awesome, I'll give it a go! thanks sambetts and jroll | 15:31 |
| jroll | if the goal is to delete the node altogether only instance_uuid is necessary | 15:32 |
| mjturek1 | jroll: right I'm just deleting the node so I can recreate it, so sambetts idea would work for me as well | 15:32 |
| jlvillal | Good morning: NobodyCam mjturek1 jroll sambetts thiagop yolanda BadCub Nisha lucasagomes TheJulia dtantsur miles cinerama and everyone else :) | 15:33 |
| mjturek1 | gm! | 15:33 |
| thiagop | morning jlvillal | 15:33 |
| jroll | mjturek1: well, sambetts' suggestion will try to do a deploy on it | 15:33 |
| lucasagomes | jlvillal, good morning | 15:33 |
| NobodyCam | morning jlvillal | 15:33 |
| mjturek1 | jroll: ahhhh understood | 15:33 |
| jroll | mjturek1: actually, you can do set-provision-state deleted from deployfail, if that's where it's at | 15:33 |
| jroll | which will tear everything down | 15:33 |
| mjturek1 | jroll: it's at available for some reason :-\ | 15:33 |
| dtantsur | morning jlvillal, BadCub | 15:33 |
| jroll | mjturek1: oh right, yeah just node-udpate then | 15:34 |
| jroll | mjturek1: which version of ironic? | 15:34 |
| mjturek1 | jroll: upstream | 15:34 |
| jroll | master? | 15:34 |
| mjturek1 | yep yep | 15:34 |
| NobodyCam | morning dtantsur :) | 15:34 |
| sambetts | jroll: I've seen this happen a couple of times on master recently too | 15:34 |
| *** david-lyle_ has joined #openstack-ironic | 15:34 | |
| Nisha | jlvillal, morning | 15:34 |
| mjturek1 | want me to open a bug report or anything? | 15:34 |
| jroll | hm, tear down that nova issued should have cleaned up instance uuid | 15:34 |
| jroll | maybe? | 15:34 |
| mjturek1 | not sure if it's reproducable | 15:34 |
| *** david-lyle has quit IRC | 15:34 | |
| jroll | yeah, that's the tough part | 15:35 |
| jroll | https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L640 | 15:35 |
| jroll | is the code | 15:35 |
| jroll | maybe look for the logs above that line and make sure tear_down ran? | 15:35 |
| mjturek1 | yeah I'll look at the logs now | 15:35 |
| *** getvasanth has quit IRC | 15:37 | |
| TheJulia | yolanda: I'll spin a fresh machine lcoally and run later today. Seems like something is wrong on a totally fresh CI machine. | 15:37 |
| *** vishwanathj_zzz is now known as vishwanathj | 15:38 | |
| TheJulia | :( | 15:38 |
| miles | morning jlvillal | 15:38 |
| JayF | jlvillal: the other piece of that earlier comment re: ONIE, is I have no idea if anything small/good for lab supports it :) | 15:39 |
| NobodyCam | morning jlvillal :) | 15:39 |
| jlvillal | JayF: Ah, thanks. I want to learn more about all this SDN stuff :) | 15:40 |
| JayF | jlvillal: ONIE is Open Network Install Environment; basically the open compute answer to switch provisioning API (that maybe, one day, years down the road, Ironic can work with) | 15:40 |
| mjturek1 | fyi jroll do_node_teardown seemingly didn't run (grep | 15:41 |
| mjturek1 | 'd the ir-cond los for the method) | 15:41 |
| *** jroll has quit IRC | 15:41 | |
| mjturek1 | logs* | 15:41 |
| *** jroll has joined #openstack-ironic | 15:41 | |
| *** jroll has quit IRC | 15:41 | |
| *** jroll has joined #openstack-ironic | 15:41 | |
| jlvillal | JayF: Ah yes. I think I have read about that. You can buy these whitebox switches and install the OS on them. They can be used instead of some of the big name switch vendor products. | 15:42 |
| *** jaybeale has joined #openstack-ironic | 15:43 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add test to verify ironic multitenancy https://review.openstack.org/269157 | 15:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 15:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 15:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 15:47 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: WIP: add network drivers https://review.openstack.org/285852 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 15:48 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to use NETWORK_DRIVERS in devstack https://review.openstack.org/293520 | 15:48 |
| dtantsur | much code many patches wow | 15:55 |
| TheJulia | wow | 15:55 |
| lucasagomes | jroll, sambetts I will give https://bugs.launchpad.net/ironic/+bug/1557497 a stab | 15:56 |
| openstack | Launchpad bug 1557497 in Ironic "Can not update node after aborting cleaning ("clean wait" state)" [High,Confirmed] - Assigned to Lucas Alvares Gomes (lucasagomes) | 15:56 |
| lucasagomes | jroll, sambetts can we agree in an assumption here. If the node state was moved to a stable state the target_provision_state should always be None ? | 15:56 |
| sambetts | lucasagomes: I think that makes sense | 15:56 |
| lucasagomes | because all stable states requires APIs requests to transit to another state, so I think it makes sense | 15:57 |
| lucasagomes | sambetts, ack I will see how I can fit that in the code | 15:57 |
| sambetts | lucasagomes: I think that might be a bandaid to a more complex problem though :/ not sure | 15:58 |
| jroll | lucasagomes: I would think that moving it to manageable would clear the target | 15:59 |
| jroll | I think that's the bug we want to target here | 15:59 |
| jroll | without re-doing the state machine all over again | 15:59 |
| sambetts | +1 | 15:59 |
| lucasagomes | jroll, right | 16:00 |
| lucasagomes | currently: ENROLL, MANAGEABLE, AVAILABLE, ACTIVE and ERROR are stable states | 16:00 |
| lucasagomes | they both requires API calls to move to another state, so I was thinking on something a bit more generic | 16:01 |
| *** e0ne has joined #openstack-ironic | 16:01 | |
| lucasagomes | == if the node is in one of these states the target_provision_state has to be cleared | 16:01 |
| lucasagomes | well lemme try, if it doesn't work we can do something more specific | 16:01 |
| *** KennethWilke has quit IRC | 16:03 | |
| jroll | yeah, I agree with that | 16:03 |
| jroll | tbh I thought we had code for that already | 16:03 |
| lucasagomes | yeah, apparently not | 16:03 |
| lucasagomes | should be easy fixable if I find the right place to place the code | 16:04 |
| sambetts | lucasagomes: 2 or 3 lines in _post_process_event should solve it then right? | 16:04 |
| lucasagomes | sambetts, looking | 16:05 |
| lucasagomes | sambetts, we have to consider the callback as well | 16:05 |
| sambetts | lucasagomes: thats where we currently reset the target state if when we reach it, so maybe just an elif state == stable self._target_state = None | 16:06 |
| sambetts | or even just an OR on the existing if | 16:06 |
| *** harlowja_at_home has joined #openstack-ironic | 16:07 | |
| jroll | lucasagomes: did you want to comment on the clean steps api thing, or want me to? | 16:08 |
| lucasagomes | sambetts, yeah, looking at it ty | 16:08 |
| lucasagomes | jroll, hmm if you have it handy do it. Otherwise I will do in a min | 16:09 |
| jroll | yeah I can | 16:09 |
| lucasagomes | thanks much | 16:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/python-ironicclient: Updates supporting ironic-neutron integration https://review.openstack.org/206144 | 16:12 |
| *** Nisha has quit IRC | 16:13 | |
| *** Nisha has joined #openstack-ironic | 16:13 | |
| *** harlowja_at_home has quit IRC | 16:15 | |
| * dtantsur sent a email about the root disk problem | 16:18 | |
| *** david-lyle_ is now known as david-lyle | 16:20 | |
| devananda | morning, all | 16:22 |
| *** harlowja_at_home has joined #openstack-ironic | 16:22 | |
| lucasagomes | devananda, morning | 16:22 |
| NobodyCam | good morning devananda :) | 16:23 |
| sambetts | o/ devananda | 16:23 |
| thiagop | morning devananda | 16:24 |
| *** ppiela has quit IRC | 16:26 | |
| *** [1]rpioso has joined #openstack-ironic | 16:27 | |
| *** rpioso has quit IRC | 16:28 | |
| *** mgoddard has quit IRC | 16:28 | |
| *** mgoddard has joined #openstack-ironic | 16:28 | |
| dtantsur | morning devananda | 16:29 |
| *** ramishra has quit IRC | 16:30 | |
| *** ramishra has joined #openstack-ironic | 16:32 | |
| *** KennethWilke has joined #openstack-ironic | 16:35 | |
| *** trown is now known as trown|lunch | 16:36 | |
| *** links has quit IRC | 16:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make sure target state is cleared on stable states https://review.openstack.org/293554 | 16:41 |
| lucasagomes | sambetts, jroll ^ (/me testing now) | 16:41 |
| *** clenimar has joined #openstack-ironic | 16:43 | |
| *** e0ne has quit IRC | 16:47 | |
| *** e0ne_ has joined #openstack-ironic | 16:47 | |
| *** athomas has quit IRC | 16:47 | |
| *** aNupoisc has joined #openstack-ironic | 16:49 | |
| *** e0ne has joined #openstack-ironic | 16:50 | |
| *** e0ne_ has quit IRC | 16:50 | |
| *** athomas has joined #openstack-ironic | 16:53 | |
| *** e0ne has quit IRC | 16:53 | |
| JayF | lucasagomes: can you tag the bug in the commit re: 292828 then :) | 16:55 |
| lucasagomes | JayF, it's there | 16:55 |
| lucasagomes | JayF, but I think the release note would be a good thing anyway, so I'm adding one too | 16:56 |
| JayF | not for replace sysrq commands, it's not | 16:56 |
| JayF | wait what | 16:56 |
| JayF | TIL commit messages can scroll in gerrit | 16:56 |
| JayF | lol | 16:56 |
| lucasagomes | JayF, hah it's https://review.openstack.org/#/c/292828/3//COMMIT_MSG | 16:56 |
| lucasagomes | L25 | 16:56 |
| JayF | Well I see it now, I usually just look at the commit on the first page, and apparently they scroll | 16:56 |
| * JayF wonders now how many erroneous "add a bug" comments he's made | 16:57 | |
| lucasagomes | oh right ;-( | 16:57 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 16:58 |
| Nisha | lucasagomes, NobodyCam jroll JayF ^^^ | 16:59 |
| *** raildo is now known as raildo-afk | 17:00 | |
| *** e0ne has joined #openstack-ironic | 17:00 | |
| openstackgerrit | Merged openstack/ironic: [devstack] Do not die if neutron is disabled https://review.openstack.org/293088 | 17:01 |
| *** piet has joined #openstack-ironic | 17:01 | |
| thiagop | JayF: that's new when infra changed the gerrit version some weeks ago | 17:01 |
| openstackgerrit | Merged openstack/ironic: Updating driver docs with DL hardwares requirements https://review.openstack.org/292865 | 17:02 |
| *** ifarkas has quit IRC | 17:03 | |
| *** KennethWilke has quit IRC | 17:04 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Replace SYSRQ commands https://review.openstack.org/292828 | 17:04 |
| lucasagomes | JayF, added a reno ^ | 17:04 |
| JayF | lucasagomes: I'm not sure how many folks we should have look at this before landing it | 17:04 |
| JayF | me + you + jroll might be enough but just trying to think if there's anyone else who might have hardware-based concerns with this | 17:05 |
| lucasagomes | JayF, yeah it's tricky... we can always revert if something goes bad too | 17:05 |
| lucasagomes | JayF, or perhaps make it configurable | 17:05 |
| JayF | I mean, in that case ... yeah, config | 17:06 |
| JayF | but I don't think it's needed now | 17:06 |
| JayF | I think this should cover the 99.99% case | 17:06 |
| JayF | Has someone tested that in DIB/TinyIPA? | 17:06 |
| JayF | I think that'd reduce the number of unknowns | 17:06 |
| *** aNupoisc has quit IRC | 17:06 | |
| lucasagomes | JayF, I haven't give tinyipa a go with it | 17:06 |
| lucasagomes | DIB is fine | 17:06 |
| lucasagomes | I will add to my TODO list to test tinyIPA | 17:07 |
| lucasagomes | it's getting late here and tomorrow is national holiday (paddy's day). So I think I will be able to test it on friday | 17:07 |
| lucasagomes | JayF, wants me to leave it as WIP until I get it tested? | 17:08 |
| JayF | Not at all | 17:08 |
| JayF | just thinking out loud, not adding requirements | 17:08 |
| lucasagomes | aight | 17:08 |
| JayF | who has been working on tinyipa? maybne at least have them review? | 17:08 |
| lucasagomes | sambetts, ^ | 17:08 |
| *** jistr has quit IRC | 17:12 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Make sure target state is cleared on stable states https://review.openstack.org/293554 | 17:15 |
| *** ohamada_ has quit IRC | 17:17 | |
| sambetts | lucasagomes, JayF: Thats a good point, I don't think tinycore linux has the shutdown application | 17:17 |
| jlvillal | miles: I pushed the update to https://github.com/JohnVillalovos/devstack-gate-test if you have any free time :) | 17:17 |
| *** praneshp_ has joined #openstack-ironic | 17:18 | |
| sambetts | lucasagomes: it might be possible to create a wrapper around its version of the same thing | 17:18 |
| jroll | well, tinyipa checks passed | 17:18 |
| *** MattMan has quit IRC | 17:18 | |
| *** Sukhdev has joined #openstack-ironic | 17:18 | |
| *** MattMan has joined #openstack-ironic | 17:19 | |
| lucasagomes | jroll, oh right we have a gate test for it now | 17:19 |
| jroll | oh heh | 17:19 |
| jroll | 2016-03-15 18:50:42.917 954 ERROR root SystemRebootError: Error rebooting system: Reboot script failed with exit code 127. stdout: . stderr: /usr/local/lib/python2.7/site-packages/ironic_python_agent/extensions/../shell/shutdown.sh: line 23: shutdown: command not found | 17:19 |
| jroll | http://logs.openstack.org/28/292828/3/check/gate-tempest-dsvm-ironic-pxe_ipa-tinyipa-nv/1ed9a96/logs/ironic-bm-logs/baremetalbrbm_0_console_15-03-2016-18:51:18.txt.gz | 17:19 |
| lucasagomes | oh ;-( | 17:19 |
| jroll | but ironic will use oob if that doesn't work | 17:19 |
| jroll | which is why it passed | 17:19 |
| sambetts | jroll: ironic will just hard shut it down if that fails right? | 17:19 |
| jroll | yep | 17:20 |
| jroll | but I'd rather fix that before landing this | 17:20 |
| dtantsur | c u tomorrow | 17:20 |
| *** dtantsur is now known as dtantsur|afk | 17:20 | |
| JayF | Maybe just check for existance of shutdown binary, if it doesn't exist, use sysrq? | 17:21 |
| sambetts | lucasagomes: tinyipa has a script called exitcheck.sh in /usr/bin, which can be called like exitcheck.sh shutdown, or exitcheck.sh reboot, so it should be possible to make a little script that translates from shutdown --reboot to exitcheck.sh reboot | 17:22 |
| *** piet has quit IRC | 17:22 | |
| lucasagomes | JayF, sambetts yeah... or as JayF said we can make SYSRQ a fallback | 17:24 |
| *** spandhe_ has joined #openstack-ironic | 17:24 | |
| sambetts | lucasagomes: just thinking about making sure we get a clean filesystem shutdown | 17:25 |
| *** trown|lunch is now known as trown | 17:25 | |
| JayF | I like the idea of using a sysrq fallback, honestly | 17:25 |
| JayF | sambetts: we always doa sync beforehand, so it's always safe | 17:25 |
| *** spandhe has quit IRC | 17:25 | |
| *** spandhe_ is now known as spandhe | 17:25 | |
| lucasagomes | sambetts, exitcheck.sh is the only way in tiny ipa? | 17:25 |
| lucasagomes | commands like halt or reboot is not present? | 17:26 |
| lucasagomes | yeah, the sysrq at least will work for pretty much every single linux os | 17:26 |
| sambetts | lucasagomes: there is reboot and poweroff in /sbin | 17:26 |
| sambetts | lucasagomes: halt is there too | 17:26 |
| sambetts | lucasagomes: tinyipa is based on busybox so anything from there | 17:27 |
| lucasagomes | sambetts, right, yeah those can be an alternative too | 17:27 |
| * lucasagomes don't want to make the script too complex tho | 17:27 | |
| sambetts | lucasagomes: sure :) | 17:27 |
| *** suro-patz has joined #openstack-ironic | 17:28 | |
| lucasagomes | sambetts, JayF right, so I will change it on friday | 17:28 |
| sambetts | lucasagomes: supporting busybox makes sense its the base for a lot of ramdisks / small distros | 17:28 |
| lucasagomes | sambetts, totally ++ | 17:28 |
| lucasagomes | I thought it would have a shutdown command as well | 17:28 |
| lucasagomes | but apparently not (-: | 17:28 |
| sambetts | lucasagomes: I'm pretty suprised that it doesn't | 17:29 |
| lucasagomes | if you guys reach any conclusion on what to use for the power action, mind comment on the patch please? | 17:29 |
| JayF | I still like sysrq because it's basically guaranteed to work | 17:29 |
| lucasagomes | JayF, yeah and doesn't require any utility | 17:30 |
| lucasagomes | I think that's the way I will solve it | 17:30 |
| *** e0ne has quit IRC | 17:30 | |
| lucasagomes | thanks a lot for the input folks | 17:30 |
| sambetts | :d | 17:30 |
| sambetts | :D | 17:30 |
| lucasagomes | I'm going to call it a day, gotta get the train back home yet | 17:30 |
| sambetts | o/ lucasagomes | 17:30 |
| lucasagomes | happy st patricks day for you all | 17:30 |
| lucasagomes | I talk to you on Friday | 17:31 |
| miles | wait, I'm confused: I thought we couldn't use sysrq because it was too brutal | 17:31 |
| miles | lucasagomes, enjoy Paddy's Day! | 17:31 |
| lucasagomes | miles, as a fallback | 17:31 |
| miles | ah, gotcha | 17:31 |
| lucasagomes | well let's put some thought on it tho | 17:31 |
| miles | yep | 17:31 |
| lucasagomes | miles, thanks, enjoy it too! I'm sure you guys are going to do something in Edinburgh right? | 17:31 |
| *** praneshp_ has quit IRC | 17:32 | |
| miles | I don't think there's a big official celebration, but there will be plenty of people out celebrating :-) | 17:32 |
| lucasagomes | heh fair enough | 17:33 |
| lucasagomes | aight then, see y'all | 17:33 |
| *** lucasagomes is now known as lucas-afk | 17:33 | |
| jlvillal | lucasagomes: miles Celebrations in the US too. Mostly seems to be an excuse to drink from what I recall :) | 17:33 |
| lucas-afk | jlvillal, well same here | 17:33 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: generator.py: Add comments & remove unused param https://review.openstack.org/283954 | 17:33 |
| lucas-afk | jlvillal, honestly I don't really go to the big parade or anything like that | 17:33 |
| lucas-afk | jlvillal, I just go to a pub and chill | 17:34 |
| lucas-afk | too many feral drunken youths on the streets | 17:34 |
| miles | :-) | 17:34 |
| jlvillal | lucas-afk: I bet! | 17:34 |
| *** miles is now known as mgould | 17:34 | |
| sambetts | lucas-afk is old man :-P | 17:34 |
| jlvillal | Maybe they aren't feral, but druken I can believe | 17:34 |
| jlvillal | s/druken/drunken/ | 17:34 |
| lucas-afk | jlvillal, you would be surprise :-P | 17:34 |
| lucas-afk | sambetts, pretty much | 17:35 |
| sambetts | lucas-afk: haha, I get exactly where your coming from though, I'm not even in Ireland and I'll be avoiding town :-P | 17:36 |
| *** ndipanov has quit IRC | 17:36 | |
| *** praneshp_ has joined #openstack-ironic | 17:38 | |
| *** sturivnyi has joined #openstack-ironic | 17:38 | |
| *** johnthetubaguy_ has joined #openstack-ironic | 17:39 | |
| sturivnyi | dtantsur|afk, :Hi=) Would be grateful if you could review my patch https://review.openstack.org/#/c/240944/ | 17:41 |
| *** johnthetubaguy has quit IRC | 17:41 | |
| *** johnthetubaguy_ is now known as johnthetubaguy | 17:41 | |
| *** vishwanathj is now known as vishwanathj_chow | 17:45 | |
| *** derekh has quit IRC | 17:50 | |
| NobodyCam | oh thats a new one: http://logs.openstack.org/24/160224/17/check/gate-tempest-dsvm-ironic-agent_ssh-src/48d4a10/logs/devstacklog.txt.gz#_2016-03-16_17_24_02_794 | 17:50 |
| *** athomas has quit IRC | 17:52 | |
| *** Sukhdev has quit IRC | 17:52 | |
| jlvillal | NobodyCam: I haven't noticed that before | 17:52 |
| *** rcernin has quit IRC | 17:53 | |
| NobodyCam | jlvillal: ya, Thou I'm sure that is the real error: http://logs.openstack.org/24/160224/17/check/gate-tempest-dsvm-ironic-agent_ssh-src/48d4a10/logs/devstacklog.txt.gz#_2016-03-16_17_24_02_647 | 17:54 |
| * jlvillal waits for giant logfile to load :) | 17:54 | |
| NobodyCam | ... thou I'm *NOT* sure.... | 17:54 |
| jlvillal | NobodyCam: I don't think so... | 17:55 |
| jlvillal | NobodyCam: Well the first line, I'm not sure about | 17:55 |
| jlvillal | NobodyCam: The 2nd error message is very very common with python-requests | 17:55 |
| NobodyCam | ya | 17:55 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Add unit tests for 'generator.py' https://review.openstack.org/283937 | 17:56 |
| *** alexpilotti has quit IRC | 17:58 | |
| jroll | NobodyCam: jlvillal: that looks like docker not running | 17:59 |
| *** mgoddard_ has joined #openstack-ironic | 17:59 | |
| jroll | http://logs.openstack.org/24/160224/17/check/gate-tempest-dsvm-ironic-agent_ssh-src/48d4a10/logs/devstacklog.txt.gz#_2016-03-16_17_24_01_466 | 17:59 |
| jroll | should be though. | 17:59 |
| NobodyCam | ya, Nisha kicked off a recheck | 18:00 |
| jlvillal | Ouch. Git security issue out there. Possible code execution on both client and server :( | 18:01 |
| NobodyCam | oh ? | 18:01 |
| jlvillal | Issues (CVE-2016-2315 and CVE-2016-2324) allow arbitrary code execution | 18:01 |
| jlvillal | Fixes: | 18:01 |
| jlvillal | •https://github.com/git/git/commit/34fa79a6cde56d6d428ab0d3160cb094ebad3305 | 18:01 |
| jlvillal | •https://github.com/git/git/commit/9831e92bfa833ee9c0ce464bbc2f941ae6c2698d | 18:01 |
| jlvillal | Info | 18:02 |
| jlvillal | •http://pastebin.com/UX2P2jjg | 18:02 |
| jlvillal | •http://www.openwall.com/lists/oss-security/2016/03/15/5 | 18:02 |
| jlvillal | •http://www.openwall.com/lists/oss-security/2016/03/16/8 | 18:02 |
| *** mgoddard has quit IRC | 18:02 | |
| *** baoli has quit IRC | 18:04 | |
| *** KennethWilke has joined #openstack-ironic | 18:05 | |
| * jroll on 2.7.3 already \o/ | 18:08 | |
| *** Nisha has quit IRC | 18:08 | |
| *** rolfb_ has joined #openstack-ironic | 18:10 | |
| *** rolfb_ has quit IRC | 18:11 | |
| *** rbrud has joined #openstack-ironic | 18:11 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Tests for testing chassis-create command https://review.openstack.org/293634 | 18:18 |
| *** tesseract- has quit IRC | 18:19 | |
| *** rpioso has joined #openstack-ironic | 18:19 | |
| * mgould -> climbing -> home; good night! | 18:20 | |
| NobodyCam | night mgould :) | 18:20 |
| mgould | night NobodyCam! | 18:20 |
| *** mgould has quit IRC | 18:20 | |
| *** degorenko is now known as _degorenko|afk | 18:23 | |
| *** ishant has quit IRC | 18:25 | |
| *** [1]rpioso has quit IRC | 18:26 | |
| *** spandhe_ has joined #openstack-ironic | 18:29 | |
| *** spandhe has quit IRC | 18:31 | |
| *** spandhe has joined #openstack-ironic | 18:32 | |
| *** spandhe_ has quit IRC | 18:34 | |
| *** rpioso has quit IRC | 18:35 | |
| *** Marga_ has quit IRC | 18:40 | |
| *** rpioso has joined #openstack-ironic | 18:43 | |
| NobodyCam | TheJulia: just fyi: I added the note requested by cinerama here: https://review.openstack.org/#/c/290819 | 18:44 |
| TheJulia | NobodyCam: and well, I've gone to great lengths to make sure test-bifrost.sh will work outside of CI as well :) | 18:47 |
| *** electrofelix has quit IRC | 18:49 | |
| *** baoli has joined #openstack-ironic | 18:49 | |
| openstackgerrit | Michael Turek proposed openstack/ironic: Allow user to specify cleaning network https://review.openstack.org/291427 | 18:49 |
| *** ppiela has joined #openstack-ironic | 18:50 | |
| sambetts | night all | 18:52 |
| *** sambetts is now known as sambetts|afk | 18:52 | |
| *** [1]rpioso has joined #openstack-ironic | 18:53 | |
| thiagop | night sambetts|afk | 18:54 |
| *** mgoddard_ has quit IRC | 18:54 | |
| *** mgoddard has joined #openstack-ironic | 18:54 | |
| *** amit2131 has quit IRC | 18:55 | |
| *** moshele has joined #openstack-ironic | 18:55 | |
| *** amit213 has joined #openstack-ironic | 18:55 | |
| *** pece has quit IRC | 18:56 | |
| *** vishwanathj_chow is now known as vishwanathj | 19:01 | |
| *** harlowja has joined #openstack-ironic | 19:04 | |
| *** rpioso has quit IRC | 19:07 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Ensure uuid-runtime is presesent on Debian based systems https://review.openstack.org/293662 | 19:09 |
| *** moshele has quit IRC | 19:12 | |
| *** moshele has joined #openstack-ironic | 19:12 | |
| cinerama | NobodyCam: oh thanks, i meant to get to that | 19:13 |
| NobodyCam | :) | 19:15 |
| *** Sukhdev has joined #openstack-ironic | 19:16 | |
| jlvillal | jroll: Ubuntu 14.04 is git version 1.9.1 :( | 19:20 |
| * jlvillal wonders what OS jroll runs... | 19:20 | |
| *** pece has joined #openstack-ironic | 19:25 | |
| *** kro_sniffles is now known as kro_mnckn_doc | 19:27 | |
| *** piet has joined #openstack-ironic | 19:28 | |
| *** daemontool has quit IRC | 19:28 | |
| *** ChubYann has joined #openstack-ironic | 19:36 | |
| *** Marga_ has joined #openstack-ironic | 19:36 | |
| *** moshele has quit IRC | 19:39 | |
| *** mjturek1 has left #openstack-ironic | 19:40 | |
| jlvillal | NobodyCam, or anyone else. Is there a good tool to view the baremetal console log? Currently I just do a cat, and mostly works. But I was wondering if there was something that could slow it down so I could see the various messages that might get erased during the process. | 19:40 |
| jlvillal | So this is the log that captures what the BIOS bootup is doing. | 19:41 |
| * jlvillal imagines he could write a python script to output bytes slowly... | 19:41 | |
| NobodyCam | jlvillal: I have been known to use less | 19:42 |
| jlvillal | NobodyCam: Oh does that handle the escape codes? I was trying vim and very ugly... | 19:42 |
| *** KennethWilke has quit IRC | 19:44 | |
| NobodyCam | no well but at least as good as cat | 19:44 |
| NobodyCam | s/no/not/ | 19:45 |
| *** KennethWilke has joined #openstack-ironic | 19:46 | |
| *** e0ne has joined #openstack-ironic | 19:46 | |
| *** harlowja has quit IRC | 19:46 | |
| jlvillal | NobodyCam: Hacky, hacky but I did this :) http://paste.openstack.org/show/490783/ | 19:48 |
| jlvillal | At least I could see that I'm not missing anything important. | 19:48 |
| NobodyCam | :) | 19:49 |
| *** ig0r_ has quit IRC | 19:55 | |
| *** Goneri has quit IRC | 19:56 | |
| *** jaybeale has quit IRC | 19:57 | |
| *** moshele has joined #openstack-ironic | 20:01 | |
| jroll | jlvillal: main machine is arch | 20:01 |
| jlvillal | Woo hoo! /me figured out that the DHCP address was not being handed out and found the error message in /var/log/syslog from dnsmasq | 20:02 |
| * jlvillal thinks that took way way way too long.... | 20:02 | |
| jlvillal | jroll: Ah. I haven't tried that yet. Ubuntu and Fedora are what I have running. | 20:02 |
| * jlvillal goes to buy a chocolate bar as a self-reward... | 20:03 | |
| *** jcoufal has quit IRC | 20:03 | |
| * jlvillal tries not to think about figuring out why dnsmasq says: DHCPDISCOVER(tapc0716d7e-64) 52:54:00:51:81:f8 no address available | 20:04 | |
| *** moshele has quit IRC | 20:10 | |
| *** mjturek1 has joined #openstack-ironic | 20:16 | |
| *** Sukhdev has quit IRC | 20:16 | |
| *** penick has joined #openstack-ironic | 20:18 | |
| *** sukhdev has joined #openstack-ironic | 20:26 | |
| *** penick has quit IRC | 20:29 | |
| *** baoli has quit IRC | 20:32 | |
| *** penick has joined #openstack-ironic | 20:33 | |
| *** jaybeale has joined #openstack-ironic | 20:33 | |
| *** praneshp_ has quit IRC | 20:34 | |
| *** praneshp_ has joined #openstack-ironic | 20:36 | |
| openstackgerrit | Lilia Sampaio proposed openstack/python-oneviewclient: Adding dynamic allocation operations https://review.openstack.org/258699 | 20:40 |
| *** harshs has joined #openstack-ironic | 20:50 | |
| *** KennethWilke has quit IRC | 20:53 | |
| *** harshs has left #openstack-ironic | 20:54 | |
| *** harshs has quit IRC | 20:54 | |
| *** harshs has joined #openstack-ironic | 20:55 | |
| *** e0ne has quit IRC | 20:58 | |
| *** praneshp_ has quit IRC | 20:59 | |
| *** Goneri has joined #openstack-ironic | 21:01 | |
| *** achanda has joined #openstack-ironic | 21:05 | |
| *** penick has quit IRC | 21:05 | |
| mrda | Morning Ironic | 21:17 |
| *** Sukhdev_ has joined #openstack-ironic | 21:17 | |
| *** kro_mnckn_doc has quit IRC | 21:19 | |
| *** anteaya has quit IRC | 21:19 | |
| *** achanda has quit IRC | 21:24 | |
| *** Sukhdev_ has quit IRC | 21:26 | |
| BadCub | howdy mrda | 21:26 |
| *** achanda has joined #openstack-ironic | 21:28 | |
| NobodyCam | morning mrda :) | 21:31 |
| *** dtantsur|afk has quit IRC | 21:31 | |
| mrda | hey BadCub and NobodyCam! | 21:32 |
| *** Sukhdev_ has joined #openstack-ironic | 21:33 | |
| *** dprince has quit IRC | 21:34 | |
| *** achanda has quit IRC | 21:36 | |
| *** Sukhdev_ has quit IRC | 21:36 | |
| *** Sukhdev_ has joined #openstack-ironic | 21:36 | |
| *** e0ne has joined #openstack-ironic | 21:37 | |
| *** dtantsur has joined #openstack-ironic | 21:37 | |
| *** trown is now known as trown|outtypewww | 21:39 | |
| *** e0ne has quit IRC | 21:40 | |
| cinerama | hi mrda | 21:40 |
| mrda | hey cinerama! | 21:40 |
| mrda | cinerama: are you in my timezone again? | 21:43 |
| *** achanda has joined #openstack-ironic | 21:47 | |
| *** absubram has quit IRC | 21:49 | |
| *** permalac has quit IRC | 21:50 | |
| *** absubram has joined #openstack-ironic | 21:50 | |
| *** piet has quit IRC | 21:50 | |
| cinerama | mrda: sadly no | 21:53 |
| mrda | cinerama: rats, because I am in SYD this week and next. We could have caught up :) | 21:53 |
| cinerama | although it's fined up nicely here so it could be worse | 21:53 |
| mrda | :) | 21:53 |
| *** alex_xu has quit IRC | 21:56 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Allow mysql_username and mysql_password to be passed via env var. https://review.openstack.org/293733 | 21:56 |
| *** saripurigopi has joined #openstack-ironic | 21:58 | |
| openstackgerrit | Chris Krelle proposed openstack/bifrost: Allow mysql_username and mysql_password to be passed via env var. https://review.openstack.org/293733 | 21:58 |
| *** alex_xu has joined #openstack-ironic | 21:58 | |
| *** KennethWilke has joined #openstack-ironic | 21:58 | |
| saripurigopi | Hello Ironic | 21:58 |
| TheJulia | greetings | 21:59 |
| *** sambetts|afk has quit IRC | 21:59 | |
| mrda | hi saripurigopi and TheJulia :) | 22:00 |
| saripurigopi | Hello mrda | 22:01 |
| *** sambetts has joined #openstack-ironic | 22:02 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 22:02 |
| *** alex_xu has quit IRC | 22:06 | |
| *** piet has joined #openstack-ironic | 22:07 | |
| *** davideagnello has quit IRC | 22:08 | |
| *** davideagnello has joined #openstack-ironic | 22:09 | |
| *** cdearborn has quit IRC | 22:09 | |
| *** Nisha has joined #openstack-ironic | 22:09 | |
| *** alex_xu has joined #openstack-ironic | 22:09 | |
| *** saripurigopi1 has joined #openstack-ironic | 22:16 | |
| *** dims_ has joined #openstack-ironic | 22:16 | |
| *** dims has quit IRC | 22:18 | |
| *** saripurigopi has quit IRC | 22:19 | |
| *** praneshp_ has joined #openstack-ironic | 22:19 | |
| *** praneshp__ has joined #openstack-ironic | 22:20 | |
| *** praneshp_ has quit IRC | 22:23 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: DHCP test improvements https://review.openstack.org/293743 | 22:27 |
| TheJulia | yolanda: ^^^ | 22:27 |
| TheJulia | I've not run it locally, but it is worth a shot. I kind of wonder if the VM memory footprint is just so small in the test VMs, that they just never really complete booting. | 22:28 |
| *** chlong has joined #openstack-ironic | 22:32 | |
| *** penick has joined #openstack-ironic | 22:34 | |
| *** Sukhdev_ has quit IRC | 22:34 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:34 | |
| *** Sukhdev_ has quit IRC | 22:36 | |
| *** penick has quit IRC | 22:36 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:36 | |
| *** Sukhdev__ has joined #openstack-ironic | 22:38 | |
| *** Sukhdev_ has quit IRC | 22:38 | |
| *** Sukhdev__ has quit IRC | 22:39 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:39 | |
| *** penick has joined #openstack-ironic | 22:40 | |
| *** Waltz has quit IRC | 22:40 | |
| *** Sukhdev_ has quit IRC | 22:41 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:41 | |
| *** Sukhdev__ has joined #openstack-ironic | 22:44 | |
| *** Sukhdev_ has quit IRC | 22:45 | |
| *** Sukhdev__ has quit IRC | 22:45 | |
| *** Sukhdev_ has joined #openstack-ironic | 22:56 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix and improve test job data collection https://review.openstack.org/293752 | 22:57 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: DHCP test improvements https://review.openstack.org/293743 | 22:59 |
| *** davideagnello has quit IRC | 23:02 | |
| *** davideagnello has joined #openstack-ironic | 23:04 | |
| *** vishwanathj has quit IRC | 23:06 | |
| *** ChrisAusten has joined #openstack-ironic | 23:06 | |
| *** krotscheck has joined #openstack-ironic | 23:11 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: DHCP test improvements https://review.openstack.org/293743 | 23:12 |
| *** suro-patz has quit IRC | 23:13 | |
| *** suro-patz has joined #openstack-ironic | 23:13 | |
| *** Sukhdev_ has quit IRC | 23:15 | |
| *** Sukhdev_ has joined #openstack-ironic | 23:18 | |
| *** piet has quit IRC | 23:18 | |
| *** suro-patz has quit IRC | 23:20 | |
| *** anteaya has joined #openstack-ironic | 23:21 | |
| *** sukhdev has quit IRC | 23:22 | |
| *** Nisha has quit IRC | 23:25 | |
| openstackgerrit | Merged openstack/bifrost: add scripts/README and tools/README https://review.openstack.org/290819 | 23:26 |
| *** achanda has quit IRC | 23:36 | |
| *** ChrisAusten has quit IRC | 23:39 | |
| *** [1]rpioso has quit IRC | 23:40 | |
| *** rpioso has joined #openstack-ironic | 23:40 | |
| *** rpioso has quit IRC | 23:40 | |
| *** chlong has quit IRC | 23:41 | |
| *** pece has quit IRC | 23:44 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Fixed NamedTemporaryFile() OSError Exception https://review.openstack.org/293762 | 23:44 |
| *** piet has joined #openstack-ironic | 23:53 | |
| *** penick has quit IRC | 23:57 | |
| *** Sukhdev_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!