| *** davideagnello has joined #openstack-ironic | 00:02 | |
| *** Marga_ has joined #openstack-ironic | 00:03 | |
| *** Marga_ has quit IRC | 00:05 | |
| *** Marga_ has joined #openstack-ironic | 00:05 | |
| *** davideagnello has quit IRC | 00:07 | |
| *** meghal has quit IRC | 00:09 | |
| *** Marga_ has quit IRC | 00:10 | |
| *** ijw has joined #openstack-ironic | 00:29 | |
| *** ijw has quit IRC | 00:35 | |
| *** kejiahu has quit IRC | 00:37 | |
| *** kejiahu has joined #openstack-ironic | 00:37 | |
| *** naohirot has joined #openstack-ironic | 00:47 | |
| *** achanda has joined #openstack-ironic | 00:59 | |
| *** Marga_ has joined #openstack-ironic | 01:06 | |
| *** Marga_ has quit IRC | 01:11 | |
| *** meghal has joined #openstack-ironic | 01:20 | |
| *** meghal has quit IRC | 01:28 | |
| *** ijw has joined #openstack-ironic | 01:29 | |
| *** achanda has quit IRC | 01:33 | |
| *** ijw has quit IRC | 01:35 | |
| *** achanda has joined #openstack-ironic | 01:38 | |
| *** Marga_ has joined #openstack-ironic | 02:06 | |
| *** Marga_ has quit IRC | 02:11 | |
| *** ijw has joined #openstack-ironic | 02:29 | |
| *** Marga_ has joined #openstack-ironic | 02:33 | |
| *** Marga_ has quit IRC | 02:34 | |
| *** Marga_ has joined #openstack-ironic | 02:35 | |
| *** ijw has quit IRC | 02:35 | |
| *** ramineni has joined #openstack-ironic | 02:37 | |
| *** chlong has joined #openstack-ironic | 03:00 | |
| *** meghal has joined #openstack-ironic | 03:03 | |
| *** coolsvap|afk is now known as coolsvap | 03:29 | |
| *** ijw has joined #openstack-ironic | 03:29 | |
| *** naohirot has quit IRC | 03:31 | |
| *** achanda has quit IRC | 03:33 | |
| *** ijw has quit IRC | 03:35 | |
| openstackgerrit | IWAMOTO Toshihiro proposed openstack/ironic-python-agent: Add logs REST resource https://review.openstack.org/163737 | 03:40 |
|---|---|---|
| *** sandhya has joined #openstack-ironic | 04:03 | |
| *** naohirot has joined #openstack-ironic | 04:03 | |
| sandhya | Hi everyone.. When does kilo feature freeze get over? | 04:05 |
| *** achanda has joined #openstack-ironic | 04:24 | |
| ramineni | sandhya: kilo feature freeze is already done. | 04:25 |
| *** meghal has quit IRC | 04:26 | |
| *** ijw has joined #openstack-ironic | 04:29 | |
| *** ijw has quit IRC | 04:35 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 04:37 |
| sandhya | ramineni: Thanks! | 04:37 |
| *** meghal has joined #openstack-ironic | 04:43 | |
| *** Marga_ has quit IRC | 04:46 | |
| *** kalpase has joined #openstack-ironic | 05:10 | |
| *** ukalifon has joined #openstack-ironic | 05:18 | |
| *** davideagnello has joined #openstack-ironic | 05:19 | |
| *** ijw has joined #openstack-ironic | 05:29 | |
| *** coolsvap is now known as coolsvap|afk | 05:33 | |
| *** Nisha has joined #openstack-ironic | 05:34 | |
| *** ijw_ has joined #openstack-ironic | 05:35 | |
| *** ijw has quit IRC | 05:35 | |
| *** ijw_ has quit IRC | 05:40 | |
| *** meghal has quit IRC | 05:43 | |
| *** davideag_ has joined #openstack-ironic | 05:44 | |
| *** davideagnello has quit IRC | 05:45 | |
| *** coolsvap has joined #openstack-ironic | 05:47 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Update ilo drivers documentation for inspection https://review.openstack.org/170065 | 05:47 |
| *** penick has joined #openstack-ironic | 06:06 | |
| *** davideag_ has quit IRC | 06:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Imported Translations from Transifex https://review.openstack.org/169184 | 06:12 |
| *** kalpase has quit IRC | 06:17 | |
| *** kalpase has joined #openstack-ironic | 06:19 | |
| *** coolsvap has quit IRC | 06:25 | |
| *** ijw has joined #openstack-ironic | 06:29 | |
| *** ijw has quit IRC | 06:35 | |
| *** coolsvap has joined #openstack-ironic | 06:38 | |
| *** jamielennox|away is now known as jamielennox | 06:48 | |
| *** subscope has joined #openstack-ironic | 06:50 | |
| openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials https://review.openstack.org/170782 | 06:52 |
| *** rameshg87 has joined #openstack-ironic | 07:11 | |
| rameshg87 | good afternoon ironic | 07:11 |
| yuriyz | hi rameshg87 Ironic | 07:13 |
| rameshg87 | hello yuriyz | 07:16 |
| *** andreykurilin_ has joined #openstack-ironic | 07:20 | |
| *** kalpase has quit IRC | 07:21 | |
| *** pas-ha has joined #openstack-ironic | 07:22 | |
| *** kalpase has joined #openstack-ironic | 07:23 | |
| *** andreykurilin_ has quit IRC | 07:24 | |
| *** andreykurilin_ has joined #openstack-ironic | 07:24 | |
| pas-ha | morning all | 07:25 |
| *** achanda has quit IRC | 07:28 | |
| *** ijw has joined #openstack-ironic | 07:29 | |
| *** achanda has joined #openstack-ironic | 07:39 | |
| pas-ha | guys, https://bugs.launchpad.net/heat/+bug/1440653 is effectively blocking our functional gate | 07:43 |
| openstack | Launchpad bug 1440653 in heat "heat_integrationtests.functional.test_autoscaling.AutoScalingSignalTest.test_signal_during_suspend fails with "AtrributError: outputs"" [High,New] | 07:43 |
| pas-ha | I can't repro it on my devstack, test passes :( | 07:44 |
| pas-ha | oh, I can indeed, second and third passes failed. heisen-bug... | 07:47 |
| *** yog__ has joined #openstack-ironic | 07:48 | |
| GheRivero | morning all | 07:52 |
| *** davideagnello has joined #openstack-ironic | 07:58 | |
| viktors | Haomeng,Wang - ping | 08:00 |
| viktors | does anybody know Haomeng's IRC nickname? | 08:01 |
| *** davideagnello has quit IRC | 08:03 | |
| *** andreykurilin_ has quit IRC | 08:16 | |
| *** pradipta has joined #openstack-ironic | 08:22 | |
| *** achanda has quit IRC | 08:27 | |
| pas-ha | viktors, https://launchpad.net/~whaom - is that him? | 08:31 |
| viktors | pas-ha: perhaps, thank you. | 08:31 |
| *** ijw has quit IRC | 08:32 | |
| rameshg87 | viktors: it is Haomeng | 08:34 |
| rameshg87 | viktors: i think he is not there right now | 08:34 |
| viktors | rameshg87: do you know, when can I catch him online? | 08:35 |
| rameshg87 | viktors: he should be around this time, may be he is not there today | 08:36 |
| viktors | rameshg87: got it, thanks! | 08:36 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: iLO driver documentation for node cleaning https://review.openstack.org/169162 | 08:40 |
| openstackgerrit | Merged openstack/ironic: Add missing test for DB migration 2fb93ffd2af1 https://review.openstack.org/170406 | 08:52 |
| *** chlong_ has joined #openstack-ironic | 08:54 | |
| *** coolsvap has quit IRC | 08:55 | |
| *** coolsvap has joined #openstack-ironic | 08:57 | |
| *** chlong has quit IRC | 08:57 | |
| *** kalpase has quit IRC | 09:01 | |
| *** ijw has joined #openstack-ironic | 09:03 | |
| *** kalpase has joined #openstack-ironic | 09:03 | |
| *** ijw has quit IRC | 09:08 | |
| *** pradipta has quit IRC | 09:24 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Update ilo drivers documentation for inspection https://review.openstack.org/170065 | 09:25 |
| *** pradipta has joined #openstack-ironic | 09:27 | |
| *** ijw has joined #openstack-ironic | 09:29 | |
| *** coolsvap has quit IRC | 09:30 | |
| *** ijw has quit IRC | 09:35 | |
| *** jmank has joined #openstack-ironic | 09:42 | |
| *** jmankov has quit IRC | 09:44 | |
| *** coolsvap has joined #openstack-ironic | 09:45 | |
| *** yuanying has quit IRC | 09:46 | |
| *** naohirot has quit IRC | 09:47 | |
| *** coolsvap has quit IRC | 09:55 | |
| *** yuanying has joined #openstack-ironic | 09:55 | |
| *** coolsvap has joined #openstack-ironic | 10:07 | |
| *** edcragg has joined #openstack-ironic | 10:16 | |
| *** ijw has joined #openstack-ironic | 10:29 | |
| *** ijw has quit IRC | 10:35 | |
| *** romcheg has joined #openstack-ironic | 10:35 | |
| *** sandhya has quit IRC | 10:40 | |
| *** chlong_ has quit IRC | 10:41 | |
| *** yuanying has quit IRC | 10:49 | |
| *** edcragg has quit IRC | 10:54 | |
| *** edcragg has joined #openstack-ironic | 10:56 | |
| *** chlong_ has joined #openstack-ironic | 10:58 | |
| *** edcragg has quit IRC | 11:05 | |
| *** chlong_ has quit IRC | 11:05 | |
| *** ramineni has quit IRC | 11:09 | |
| *** edcragg has joined #openstack-ironic | 11:12 | |
| *** Nisha has quit IRC | 11:14 | |
| *** chlong has joined #openstack-ironic | 11:24 | |
| *** ijw has joined #openstack-ironic | 11:29 | |
| *** pradipta has quit IRC | 11:31 | |
| *** ijw has quit IRC | 11:35 | |
| -openstackstatus- NOTICE: gerrit has been restarted to restore event streaming. any change events missed by zuul (between 10:56 and 11:37 utc) will need to be rechecked or have new approval votes set | 11:39 | |
| * rameshg87 goes home | 11:40 | |
| *** rameshg87 has quit IRC | 11:40 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 11:45 |
| *** jamielennox is now known as jamielennox|away | 11:49 | |
| *** kalpase has quit IRC | 12:04 | |
| *** trown|outttypeww is now known as trown | 12:11 | |
| *** bradjones_ has joined #openstack-ironic | 12:22 | |
| *** bradjones has quit IRC | 12:24 | |
| *** bradjones_ is now known as bradjones | 12:24 | |
| *** bradjones has joined #openstack-ironic | 12:24 | |
| *** coolsvap has quit IRC | 12:25 | |
| *** dprince has joined #openstack-ironic | 12:25 | |
| *** ijw has joined #openstack-ironic | 12:29 | |
| *** ijw has quit IRC | 12:35 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add maintenance check before call do_node_deploy https://review.openstack.org/170149 | 12:57 |
| *** jjohnson2 has joined #openstack-ironic | 13:10 | |
| *** kkoski has joined #openstack-ironic | 13:15 | |
| *** davideagnello has joined #openstack-ironic | 13:25 | |
| *** davideagnello has quit IRC | 13:29 | |
| *** ijw has joined #openstack-ironic | 13:29 | |
| *** ijw has quit IRC | 13:35 | |
| *** subscope has quit IRC | 13:35 | |
| *** penick has left #openstack-ironic | 13:37 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: boot_mode is overwritten in node properties https://review.openstack.org/167952 | 13:44 |
| NobodyCam | Good morning Ironic | 13:50 |
| *** bradjones has quit IRC | 13:53 | |
| *** kkoski has quit IRC | 13:53 | |
| *** dprince has quit IRC | 13:53 | |
| *** early has quit IRC | 13:53 | |
| *** david-lyle has quit IRC | 13:53 | |
| *** lsmola_ has quit IRC | 13:53 | |
| *** mdbooth has quit IRC | 13:53 | |
| *** gilliard has quit IRC | 13:53 | |
| *** JoshNang has quit IRC | 13:53 | |
| *** logan2_ has quit IRC | 13:53 | |
| *** toabctl has quit IRC | 13:53 | |
| *** lintan_ has quit IRC | 13:53 | |
| *** raginbajin has quit IRC | 13:53 | |
| *** d0ugal has quit IRC | 13:53 | |
| *** romcheg has quit IRC | 13:53 | |
| *** yog__ has quit IRC | 13:53 | |
| *** kejiahu has quit IRC | 13:53 | |
| *** krtaylor has quit IRC | 13:53 | |
| *** mitz has quit IRC | 13:53 | |
| *** dtantsur|afk has quit IRC | 13:53 | |
| *** mordred has quit IRC | 13:53 | |
| *** mgagne has quit IRC | 13:53 | |
| *** adam_g_out has quit IRC | 13:53 | |
| *** comstud has quit IRC | 13:53 | |
| *** rainya has quit IRC | 13:53 | |
| *** maurosr has quit IRC | 13:53 | |
| *** ekarlso has quit IRC | 13:53 | |
| *** mikal has quit IRC | 13:53 | |
| *** zer0c00l has quit IRC | 13:53 | |
| *** coolsvap|afk has quit IRC | 13:53 | |
| *** Isotopp has quit IRC | 13:53 | |
| *** jroll has quit IRC | 13:53 | |
| *** eglute has quit IRC | 13:53 | |
| *** pleia2 has quit IRC | 13:53 | |
| *** aignatov has quit IRC | 13:53 | |
| *** jjohnson2 has quit IRC | 13:53 | |
| *** TheJulia has quit IRC | 13:53 | |
| *** rsFF has quit IRC | 13:53 | |
| *** davidlenwell has quit IRC | 13:53 | |
| *** gabriel-bezerra has quit IRC | 13:53 | |
| *** purp has quit IRC | 13:53 | |
| *** SpamapS has quit IRC | 13:53 | |
| *** trown has quit IRC | 13:53 | |
| *** sirushti has quit IRC | 13:53 | |
| *** vipul has quit IRC | 13:53 | |
| *** GheRivero has quit IRC | 13:53 | |
| *** enikanorov has quit IRC | 13:53 | |
| *** harlowja_away has quit IRC | 13:53 | |
| *** jrist has quit IRC | 13:53 | |
| *** viktors has quit IRC | 13:53 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Check status of bootloader installation for DIB ramdisk https://review.openstack.org/169419 | 13:54 |
| *** romcheg has joined #openstack-ironic | 13:55 | |
| *** yog__ has joined #openstack-ironic | 13:55 | |
| *** kejiahu has joined #openstack-ironic | 13:55 | |
| *** krtaylor has joined #openstack-ironic | 13:55 | |
| *** mitz has joined #openstack-ironic | 13:55 | |
| *** dtantsur|afk has joined #openstack-ironic | 13:55 | |
| *** mordred has joined #openstack-ironic | 13:55 | |
| *** mgagne has joined #openstack-ironic | 13:55 | |
| *** adam_g_out has joined #openstack-ironic | 13:55 | |
| *** rainya has joined #openstack-ironic | 13:55 | |
| *** comstud has joined #openstack-ironic | 13:55 | |
| *** maurosr has joined #openstack-ironic | 13:55 | |
| *** ekarlso has joined #openstack-ironic | 13:55 | |
| *** mikal has joined #openstack-ironic | 13:55 | |
| *** zer0c00l has joined #openstack-ironic | 13:55 | |
| *** coolsvap|afk has joined #openstack-ironic | 13:55 | |
| *** Isotopp has joined #openstack-ironic | 13:55 | |
| *** jroll has joined #openstack-ironic | 13:55 | |
| *** eglute has joined #openstack-ironic | 13:55 | |
| *** pleia2 has joined #openstack-ironic | 13:55 | |
| *** aignatov has joined #openstack-ironic | 13:55 | |
| *** jjohnson2 has joined #openstack-ironic | 13:56 | |
| *** TheJulia has joined #openstack-ironic | 13:56 | |
| *** rsFF has joined #openstack-ironic | 13:56 | |
| *** davidlenwell has joined #openstack-ironic | 13:56 | |
| *** gabriel-bezerra has joined #openstack-ironic | 13:56 | |
| *** purp has joined #openstack-ironic | 13:56 | |
| *** SpamapS has joined #openstack-ironic | 13:56 | |
| *** trown has joined #openstack-ironic | 13:56 | |
| *** sirushti has joined #openstack-ironic | 13:56 | |
| *** vipul has joined #openstack-ironic | 13:56 | |
| *** GheRivero has joined #openstack-ironic | 13:56 | |
| *** kkoski has joined #openstack-ironic | 13:57 | |
| *** dprince has joined #openstack-ironic | 13:57 | |
| *** early has joined #openstack-ironic | 13:57 | |
| *** david-lyle has joined #openstack-ironic | 13:58 | |
| *** lsmola_ has joined #openstack-ironic | 13:58 | |
| *** mdbooth has joined #openstack-ironic | 13:58 | |
| *** gilliard has joined #openstack-ironic | 13:58 | |
| *** JoshNang has joined #openstack-ironic | 13:58 | |
| *** logan2_ has joined #openstack-ironic | 13:58 | |
| *** toabctl has joined #openstack-ironic | 13:58 | |
| *** lintan_ has joined #openstack-ironic | 13:58 | |
| *** raginbajin has joined #openstack-ironic | 13:58 | |
| *** d0ugal has joined #openstack-ironic | 13:58 | |
| *** bradjones has joined #openstack-ironic | 13:58 | |
| *** enikanorov has joined #openstack-ironic | 13:58 | |
| *** harlowja_away has joined #openstack-ironic | 13:58 | |
| *** jrist has joined #openstack-ironic | 13:58 | |
| *** viktors has joined #openstack-ironic | 13:58 | |
| *** kkoski has quit IRC | 14:00 | |
| *** dprince has quit IRC | 14:00 | |
| *** early has quit IRC | 14:00 | |
| *** kkoski has joined #openstack-ironic | 14:01 | |
| *** dprince has joined #openstack-ironic | 14:01 | |
| *** early has joined #openstack-ironic | 14:01 | |
| *** Marga_ has joined #openstack-ironic | 14:02 | |
| *** Marga__ has joined #openstack-ironic | 14:06 | |
| *** Marga_ has quit IRC | 14:09 | |
| *** jmanko has joined #openstack-ironic | 14:12 | |
| *** jmank has quit IRC | 14:15 | |
| *** mtanino has joined #openstack-ironic | 14:23 | |
| *** mtanino has quit IRC | 14:27 | |
| *** mtanino has joined #openstack-ironic | 14:27 | |
| *** ijw has joined #openstack-ironic | 14:29 | |
| *** ijw has quit IRC | 14:35 | |
| *** achanda has joined #openstack-ironic | 14:35 | |
| *** edcragg has quit IRC | 14:37 | |
| *** edcragg has joined #openstack-ironic | 14:38 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: localboot deploy fails for iscsi_ilo driver https://review.openstack.org/168666 | 14:39 |
| *** achanda has quit IRC | 14:43 | |
| *** toure has joined #openstack-ironic | 14:44 | |
| devananda | morning, all | 14:46 |
| NobodyCam | good mornign devananda | 14:47 |
| * devananda pushes a client release of 0.5.0 | 14:51 | |
| * devananda notices dtantsur|afk 's email that ironicclient master breaks discoverd | 14:56 | |
| *** BadCub01 has joined #openstack-ironic | 14:57 | |
| BadCub01 | Morning folks | 14:57 |
| NobodyCam | good morning BadCub01 | 14:58 |
| BadCub01 | IRCCloud has my nic hostage this morning :( | 15:00 |
| *** chlong has quit IRC | 15:00 | |
| NobodyCam | :( their twitter seems to indacate they aer having issues :( | 15:00 |
| BadCub01 | yeah | 15:01 |
| *** BadCub01 has quit IRC | 15:03 | |
| *** GrumpyBear has joined #openstack-ironic | 15:03 | |
| *** GrumpyBear has quit IRC | 15:03 | |
| *** BadCub01 has joined #openstack-ironic | 15:04 | |
| * BadCub01 and IRC are not being getting along this morning *frowns* | 15:06 | |
| NobodyCam | :( | 15:06 |
| * NobodyCam wounders if we'll have enought folks for the meeting later this morning | 15:06 | |
| jroll | morning devananda BadCub01 NobodyCam and whoever else :) | 15:07 |
| NobodyCam | mornign jroll :) | 15:08 |
| BadCub01 | heya jroll:) | 15:08 |
| NobodyCam | its a slow day in channel this morning | 15:08 |
| BadCub01 | maybe everyone is using IRCCLoud lol | 15:09 |
| NobodyCam | EU folk are still on holiday | 15:09 |
| * BadCub01 is thinking that may be a good plan too | 15:10 | |
| NobodyCam | while I have a small group here. wanted to get thoughts on https://review.openstack.org/#/c/169918 adding a Lib to support tftp. I when this route for two main reasons. One I didn't want to addd a exteranl dependicy, also I felt the tftp protcol was simply and stable enough to not be a maintenance burden for us. But looking for all the input I can get | 15:13 |
| *** davideagnello has joined #openstack-ironic | 15:13 | |
| jlvillal | Good morning NobodyCam, BadCub01, devananda, and anybody else who is here :) | 15:15 |
| NobodyCam | morning jlvillal :) | 15:16 |
| BadCub01 | heya jlvillal :) | 15:16 |
| * NobodyCam pops out for quick smoke...brb | 15:16 | |
| *** dprince has quit IRC | 15:23 | |
| *** BadCub01 has quit IRC | 15:24 | |
| BadCub | yeah.. IRCCloud lives!!! | 15:24 |
| NobodyCam | w00 h00 | 15:25 |
| * BadCub still thinks taking a holiday day is a good idea | 15:25 | |
| *** toure is now known as toure_mtg | 15:29 | |
| *** ijw has joined #openstack-ironic | 15:29 | |
| *** ijw has quit IRC | 15:35 | |
| *** absubram has quit IRC | 15:35 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add PYTHONHASHSEED=0 to venv tox env https://review.openstack.org/170897 | 15:35 |
| *** rwsu has joined #openstack-ironic | 15:37 | |
| *** dprince has joined #openstack-ironic | 15:43 | |
| jroll | JayF: why does the gate use testenv:venv and not gendocs or something? | 15:45 |
| jroll | (and can that be changed in the gate rather than here) | 15:45 |
| JayF | jroll: I don't know; these are literally the exact templates that run for every service | 15:46 |
| jroll | :| | 15:46 |
| JayF | jroll: and I'm not going to have a special flower docs job for IPA that's different than the one for dozens of other services | 15:46 |
| jroll | well sure | 15:46 |
| JayF | It's the same for Ironic, fwiw | 15:46 |
| jroll | but that's pretty lame | 15:46 |
| jroll | k | 15:46 |
| NobodyCam | mornign JayF :) | 15:46 |
| devananda | so there should be a common gendocs entrypoint for the test framework | 15:46 |
| JayF | morning | 15:46 |
| JayF | devananda: in the gate; all docs jobs activate the venv and run `python setup.py build_sphinx` | 15:46 |
| devananda | see mordred's prior rants / posts to the TC about that. | 15:47 |
| JayF | devananda: rather than calling the docs env in tox explicitly | 15:47 |
| jroll | JayF: +A fwiw | 15:47 |
| JayF | jroll: before tests passed? lol | 15:47 |
| devananda | erm... I know it used to be that way, but thought there was work being done to change it | 15:47 |
| JayF | devananda: it | 15:47 |
| jroll | JayF: it doesn't merge if tests fail, so whatever | 15:47 |
| JayF | devananda: it's 100% true as of today; I had to put in https://review.openstack.org/#/c/170897/1 to fix IPA's new doc job as a result | 15:47 |
| devananda | JayF: also, ironic's docs throw warnings and stuff. would be awesome to gate on not doing that once it is not doing that | 15:48 |
| devananda | JayF: so I fully support IPA gating on a clean doc build job from the start | 15:48 |
| JayF | devananda: I agree generally; I'm configuring IPA to be that way | 15:48 |
| JayF | https://review.openstack.org/#/c/170259/ although obviously can't merge until the docs job is fixed in general | 15:48 |
| devananda | oh. meeting soon, eh? | 15:50 |
| jroll | in an hour no? | 15:50 |
| NobodyCam | devananda: its at 10 ost today no | 15:50 |
| NobodyCam | pst even | 15:50 |
| *** yog__ has quit IRC | 15:51 | |
| jroll | it's at 1700, currently is 1550 | 15:51 |
| devananda | ah yes | 15:51 |
| * devananda kicks his calendar | 15:51 | |
| NobodyCam | brb ... bbt | 15:51 |
| JayF | docs job already passed on my IPA patch \o/ (although the tempest jobs are still queued) | 15:52 |
| *** davideagnello has quit IRC | 15:53 | |
| *** dencaval has joined #openstack-ironic | 15:53 | |
| jroll | if that breaks tempest we should all just go home | 15:54 |
| BadCub | ++ on going home LOL | 15:54 |
| *** ukalifon has quit IRC | 15:54 | |
| devananda | BadCub: want to do a quick review of LP bug status with me? | 15:55 |
| BadCub | sure | 15:55 |
| * devananda looks at the whitebboard | 15:57 | |
| devananda | er, reviewdayboard | 15:57 |
| BadCub | lol | 15:58 |
| devananda | looks like https://review.openstack.org/#/c/169643/5 is almost ready, pending a question about pecan, so i've pinged ryan about that | 15:59 |
| BadCub | yeah, I still have that one listed as needing reviews | 16:00 |
| *** rameshg87 has joined #openstack-ironic | 16:03 | |
| BadCub | brb | 16:03 |
| BadCub | back | 16:06 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: iLO driver documentation for UEFI secure boot https://review.openstack.org/168844 | 16:06 |
| *** adam_g_out is now known as adam_g | 16:12 | |
| * devananda fixes up 169643 | 16:13 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 16:13 |
| *** pas-ha has quit IRC | 16:15 | |
| * BadCub updates several things on pad | 16:15 | |
| jroll | when is rc1? | 16:16 |
| BadCub | jroll: the 9th I beleive | 16:16 |
| NobodyCam | thursday I think | 16:16 |
| jroll | wow | 16:17 |
| *** meghal has joined #openstack-ironic | 16:19 | |
| *** meghal has quit IRC | 16:22 | |
| devananda | yah | 16:23 |
| jroll | I guess I should review some thing today :) | 16:23 |
| *** alexpilotti has joined #openstack-ironic | 16:23 | |
| BadCub | jroll: https://review.openstack.org/169643 could use some love LOL | 16:24 |
| *** edcragg has quit IRC | 16:25 | |
| * jroll looks | 16:26 | |
| *** trown is now known as trown|lunch | 16:27 | |
| BadCub | ty :) seems most of the others we had up for review have changed in status :( | 16:27 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 16:28 |
| devananda | mrda-away: I had a chat with ryanpetrello. he convinced me your original idea was better than my idea to monkeypatch, so I did it ^ | 16:28 |
| NobodyCam | devananda: something strange on that rev | 16:29 |
| devananda | ? | 16:29 |
| devananda | oh, i see | 16:29 |
| NobodyCam | Patch in Merge Conflict | 16:29 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: tests/db: Add autospec=True to mocks https://review.openstack.org/170911 | 16:29 |
| *** ijw has joined #openstack-ironic | 16:29 | |
| jroll | devananda: ughhhhh, the pain | 16:30 |
| jroll | (api method signature changes are rebase hell for us) :P | 16:30 |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 16:31 |
| devananda | jroll: oh? :( | 16:31 |
| jroll | @METRICS.instrument(...) | 16:32 |
| jroll | (hopefully we'll resolve this difference next cycle) | 16:32 |
| devananda | jroll: so the monkeypatch is in rev6, has no api signature changes | 16:32 |
| * rameshg87 will be back for the meeting | 16:32 | |
| *** rameshg87 is now known as rameshg87-away | 16:32 | |
| * devananda gets more coffee | 16:32 | |
| jroll | devananda: yeah, don't do that just for us though | 16:32 |
| *** meghal has joined #openstack-ironic | 16:35 | |
| *** ijw has quit IRC | 16:35 | |
| *** ukalifon1 has joined #openstack-ironic | 16:38 | |
| devananda | TheJulia: ping re: https://review.openstack.org/#/c/168120/ -- anything I can do to help? | 16:41 |
| *** degorenko has joined #openstack-ironic | 16:41 | |
| NobodyCam | brb quick smoke before meeting | 16:44 |
| *** killer_prince has joined #openstack-ironic | 16:45 | |
| *** killer_prince is now known as lazy_prince | 16:45 | |
| *** spandhe has joined #openstack-ironic | 16:47 | |
| *** davideagnello has joined #openstack-ironic | 16:52 | |
| *** rameshg87-away is now known as rameshg87 | 16:54 | |
| NobodyCam | morning rameshg87 :) WB | 16:54 |
| rameshg87 | folks, someone available to look at an rc1-bug: https://review.openstack.org/#/c/169419/ | 16:54 |
| rameshg87 | NobodyCam: morning :) | 16:54 |
| TheJulia | devananda: thoughts, on time would be nice | 16:54 |
| BadCub | heya rameshg87 | 16:54 |
| rameshg87 | hello BadCub | 16:54 |
| TheJulia | rameshg87: re, review 168120, I think trying to mock time is rather problematic and showing it's self to be unreliable to me | 16:55 |
| *** meghal has quit IRC | 16:55 | |
| TheJulia | rameshg87: any thoughts would be wonderful | 16:56 |
| devananda | rameshg87: that's a huge change to land 3 days before RC1 | 16:56 |
| rameshg87 | TheJulia: sure, let me take a look | 16:56 |
| rameshg87 | devananda: but that bug has been marked as critical :( | 16:56 |
| devananda | rameshg87: it's also adding a completely new step to the deployment process, and a new wait() for callback | 16:56 |
| *** meghal has joined #openstack-ironic | 16:56 | |
| rameshg87 | devananda: yes. another call-back for reporting the status | 16:57 |
| rameshg87 | devananda: would you prefer not landing it in kilo, rather ? | 16:57 |
| * NobodyCam heads over to meeting channel | 16:59 | |
| jroll | shall we have a meeting? | 16:59 |
| rameshg87 | devananda: but there is one issue, that change has to land now OR revert back the dependent DIB patch which got submitted https://review.openstack.org/#/c/169354/ | 16:59 |
| rameshg87 | devananda: i will bring this up in open discussion | 16:59 |
| devananda | mmm, meeting | 16:59 |
| * BadCub wanders off to meeting | 17:00 | |
| jroll | if any cores want a quick button to push during the meeting, I forgot this was in my pile https://review.openstack.org/#/c/169756/ | 17:02 |
| NobodyCam | jroll: +a'd | 17:04 |
| jroll | thanks! | 17:05 |
| *** victorfeitosa has joined #openstack-ironic | 17:07 | |
| *** Marga__ has quit IRC | 17:07 | |
| BadCub | NobodyCam: Ty :) | 17:08 |
| *** ukalifon1 has quit IRC | 17:11 | |
| *** Marga_ has joined #openstack-ironic | 17:12 | |
| *** ijw has joined #openstack-ironic | 17:12 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Add PYTHONHASHSEED=0 to venv tox env https://review.openstack.org/170897 | 17:13 |
| *** achanda has joined #openstack-ironic | 17:15 | |
| *** Marga_ has quit IRC | 17:19 | |
| *** toure_mtg is now known as toure | 17:25 | |
| openstackgerrit | Merged openstack/ironic-lib: Copy .gitignore from Ironic https://review.openstack.org/169756 | 17:33 |
| *** Nisha_away has joined #openstack-ironic | 17:36 | |
| *** jjohnson2 has quit IRC | 17:36 | |
| *** trown|lunch is now known as trown | 17:50 | |
| *** Marga_ has joined #openstack-ironic | 17:50 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add retry logic to _exec_ipmitool https://review.openstack.org/168120 | 17:51 |
| NobodyCam | is that with out the one test? | 17:52 |
| TheJulia | It is | 17:52 |
| NobodyCam | :) | 17:52 |
| TheJulia | I added an additional comment | 17:52 |
| TheJulia | also | 17:52 |
| TheJulia | w/r/t the string that it searches, looks like that is actually coming from the hardware | 17:52 |
| *** ijw has quit IRC | 17:57 | |
| * rameshg87 waits to push power button of laptop and run to sleep | 17:57 | |
| NobodyCam | :) two minutes rameshg87 | 17:58 |
| rameshg87 | :) | 17:58 |
| * rameshg87 goes to sleep | 18:00 | |
| rameshg87 | good night all | 18:00 |
| *** rameshg87 has quit IRC | 18:00 | |
| devananda | 18:00:49 < wanyen> The idea was to explore this proposal with upstream and see whether it's a reasonable approach | 18:01 |
| NobodyCam | night rameshg87 | 18:01 |
| *** wanyen has joined #openstack-ironic | 18:02 | |
| *** meghal has quit IRC | 18:03 | |
| NobodyCam | wanyen: if this is just a PoC or patch to explore an Idea I would really like to see that inthe commit message | 18:03 |
| *** meghal has joined #openstack-ironic | 18:03 | |
| *** Marga_ has quit IRC | 18:04 | |
| wanyen | NobodyCam, Shiv came up with this idea after Kilo3 discussion of secure boot so he put togther the code. The intetn was to bringthis up inthe ironic meeting and have folks take a look at it and see if it's a reasonable apporach and whehter he canget a FFE to pursue it. | 18:04 |
| *** Marga_ has joined #openstack-ironic | 18:04 | |
| wanyen | Based on my understanding, he brought this in one of the ironic meeting but the decision was to put it off | 18:06 |
| NobodyCam | wanyen: yes it was on the meeting agenda. | 18:06 |
| NobodyCam | i seems to recall chatting about it | 18:06 |
| NobodyCam | but IMBW | 18:07 |
| devananda | wanyen: so, there was a FFE granted for the uefi-secure-boot work. That was granted for patch https://review.openstack.org/165907 which was landed on 3/19 and the work considered complete at that time | 18:07 |
| wanyen | devananda, this is secure boot for pxe-ilo driver | 18:07 |
| devananda | wanyen: then this issue was raised and a new patch was proposed: https://review.openstack.org/166192 | 18:07 |
| devananda | wanyen: but that was not part of the FFE | 18:07 |
| BadCub | https://review.openstack.org/#/c/154808/ was also bumped to L for this same BP | 18:08 |
| devananda | that wasn't even available for reviewers to look at until after we thought uefi-secure-boot was complete :( | 18:08 |
| wanyen | no, that was not part of the ffe granted by ironic | 18:08 |
| devananda | BadCub: oh, i see. this was refactored to put the config option changes in 166192 instead of 154808 | 18:09 |
| rwsu | hi, where can I find the list of events ironic emits to the amqp bus? | 18:10 |
| BadCub | brb | 18:11 |
| *** harlowja_away is now known as harlowja | 18:11 | |
| wanyen | devananda, shiv discussed this at one of the irc meeting, folks in that meeting think that migration of elilio to grub2 shouldbe address in Liberty so he did not pursue it further | 18:11 |
| devananda | gotcha. i'm piecing it back together | 18:11 |
| devananda | https://review.openstack.org/#/c/154808/49 was the original, full proposal, which I blocked because it's changing much too much this late in the cycle | 18:12 |
| devananda | https://review.openstack.org/#/c/165907/2 has the common things for ilo drivers, and landed becaues it was well isolated to the ilo driver | 18:12 |
| devananda | https://review.openstack.org/#/c/166192/13 contains the pxe template changes to support grub2 | 18:12 |
| devananda | https://review.openstack.org/#/c/154808/53 contains some more of that, and is the remnants of the full proposal that I blocked previously | 18:13 |
| devananda | but because of the way shivanand refactored it, my -2 -- and ALL the discussion history -- got separated from the config changes | 18:14 |
| *** degorenko has quit IRC | 18:14 | |
| wanyen | devananda, yes. The initial pxe-ilo secure boot changed teh boot loader to grub2. revieweres have concerns about elilo migration to grub2 may not work for condtuctor take over so pxe-ilo seucre boot was blocked by secure boot for iscsi-ilo and agent-ilo were merged to Kilo. | 18:15 |
| devananda | wanyen: since the grub2 support in https://review.openstack.org/#/c/166192/13 has not, and almost definitely will not, land in Kilo, does this mean uefi-secure-boot is incomplete / does not work yet? | 18:15 |
| devananda | wanyen: or does it work, but only with some drivers? | 18:15 |
| wanyen | devananda, secure boot for iscsi-ilo and agent-ilo work. | 18:15 |
| *** Marga_ has quit IRC | 18:16 | |
| wanyen | seucre boot for pxe-ilo is bumpted to Liberty | 18:16 |
| devananda | cool | 18:16 |
| *** Marga_ has joined #openstack-ironic | 18:16 | |
| * devananda updates and notes this on https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot | 18:18 | |
| * devananda retargets uefi-secure-boot to Liberty | 18:19 | |
| BadCub | devananda: this is also listed on the Pad for L | 18:20 |
| wanyen | devananda, youmeant retarget secure boot for pxe-ilo to Liberty. seucre boot for agent-ilo and iscsi-ilo is for Kilo. right. | 18:21 |
| devananda | wanyen: well, there's only one BP .... | 18:21 |
| devananda | wanyen: while the code for agent-ilo and iscsi-ilo has landed, the feature isn't complete | 18:22 |
| wanyen | devananda, just want to make sure agent-ilo and iscsi-ilo will be part of the kilo. | 18:22 |
| wanyen | I meant secure boot for agent-ilo and iscsi-ilo will be part of kilo. | 18:23 |
| devananda | wanyen: yes, but it's awkward that it only works for those two drivers, and not for pxe_ilo | 18:24 |
| * NobodyCam eats a waffle | 18:25 | |
| wanyen | The users who use pxe-less drivers (agent-ilo and iscsi-ilo) they can use secure boot. We will submit a ndew bp for secure boot for pxe-ilo driver for liberty | 18:25 |
| wanyen | s/sdew/new | 18:26 |
| devananda | wanyen: ty | 18:28 |
| wanyen | devananda, thanks! | 18:28 |
| NobodyCam | https://review.openstack.org/#/c/167952 looks like it would be impacted by stringFreeze | 18:31 |
| wanyen | NobodyCam, so does it imply? | 18:34 |
| wanyen | NobodayCam, so what doe sit imply? | 18:34 |
| wanyen | I just cannot type | 18:34 |
| wanyen | what does it imply? | 18:34 |
| *** ijw has joined #openstack-ironic | 18:35 | |
| *** victorfeitosa has quit IRC | 18:38 | |
| NobodyCam | wanyen: string freeze is like the other freezes | 18:40 |
| NobodyCam | wanyen: https://wiki.openstack.org/wiki/StringFreeze | 18:40 |
| wanyen | Nobodycan, so can 167952 fix be included in Kilo? Is there anything else we need to do to merge this fix? | 18:42 |
| wanyen | s/Nobodycan/NobodyCam | 18:43 |
| BadCub | brb | 18:44 |
| devananda | NobodyCam: what string changes are there in that patch? | 18:50 |
| devananda | cause i dont see any ... | 18:51 |
| NobodyCam | hummm | 18:52 |
| devananda | "Deploy boot mode is %(boot_mode)s for %(node)s." is moved from one file to another, but the string isn't changed. And a debug message of "Secure boot deploy requested for node %s" is removed. | 18:54 |
| NobodyCam | ya I think I confused soemthing | 18:54 |
| NobodyCam | yep sure did | 18:55 |
| NobodyCam | 169419 was the patch set I was actually looking at | 18:55 |
| NobodyCam | :( | 18:55 |
| * NobodyCam looks at now empty coffee cup and wounder if he's had enough | 18:55 | |
| *** Marga_ has quit IRC | 18:56 | |
| NobodyCam | https://review.openstack.org/#/c/169419/12/ironic/drivers/modules/iscsi_deploy.py | 18:56 |
| devananda | oh | 18:56 |
| NobodyCam | I swapped the reviews with another opentab of mine | 18:56 |
| *** ijw has quit IRC | 18:57 | |
| devananda | yea, that patch scares me | 18:57 |
| devananda | we need to figure out WTF we're doing here | 18:57 |
| meghal | hello…had a quick question on how conductor affinity is set for ironic node when it is created for the first time ? I see that there is logic to update conductor affinity periodically for nodes which are actively deployed. But, but I could not figure out how does node get its initial conductor affinity. | 18:59 |
| meghal | could somebody explain or point me to related logic ? | 18:59 |
| *** ijw has joined #openstack-ironic | 18:59 | |
| BadCub | devananda: that one is listed on the pad already | 18:59 |
| NobodyCam | meghal: you looked at the hash_ring code already? | 19:00 |
| devananda | meghal: hash ring. nodes are distributed across the set of available conductors which can service that node's driver | 19:00 |
| devananda | i'm sure we have some documentation on that somewher e... | 19:01 |
| meghal | devananda: yep…I looked at that, but seems like periodic sync of local state is run only for node whose provision state = active | 19:01 |
| meghal | devananda: here is the link - https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1125 | 19:02 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Error doc builds on WARN or ERROR https://review.openstack.org/170259 | 19:03 |
| TheJulia | NobodyCam: There must always be coffee.... in accordance with the prophecy. | 19:03 |
| *** ijw_ has joined #openstack-ironic | 19:04 | |
| NobodyCam | :) | 19:05 |
| *** Nisha_away has quit IRC | 19:05 | |
| devananda | meghal: there probably shouldn't be any local state if provision-state != active (in-progress operations notwithstanding) | 19:05 |
| devananda | meghal: the affinity is based on the mapping. the periodic task to manage local state is separate from that affinity, but uses the same function. the affinity isn't actually /stored/ anywhere | 19:06 |
| *** ijw has quit IRC | 19:07 | |
| NobodyCam | brb | 19:10 |
| meghal | devananda: than during periodic power sync from each conductor process, conductor does power sync for nodes which are mapped to itself…and it picks up this mapping based on hash ring | 19:16 |
| meghal | so my question was if periodic hash ring refresh (sync_local_state) is done only for nodes whose provision state == active, then how would a new ironic node show up in hash ring ? | 19:17 |
| *** uschreiber_ has joined #openstack-ironic | 19:18 | |
| *** ijw_ has quit IRC | 19:23 | |
| *** uschreiber_ has quit IRC | 19:23 | |
| *** Marga_ has joined #openstack-ironic | 19:29 | |
| Shrews | devananda: we do store affinity in nodes table | 19:30 |
| *** trown is now known as trown|brb | 19:31 | |
| NobodyCam | everyone already take the OpenStack User Survey? | 19:36 |
| meghal | devananda, Shrews: yep…I do see it here - https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1120 | 19:36 |
| Shrews | meghal: a new node will not show up in the ring until it is provisioned, if i understand your question correctly. i just quickly glanced at the scrollback | 19:37 |
| meghal | Shrews: yes that is what my first question was | 19:37 |
| meghal | Shrews: follow-up question - conductor periodically does a power sync of nodes which are mapped to itself and this mapping is obtained from hash-ring | 19:38 |
| *** trown|brb is now known as trown | 19:38 | |
| meghal | Shrews: so does it mean that conductor does a periodic power sync of node's which are provisioned ? | 19:39 |
| meghal | s/node's/nodes/ | 19:39 |
| Shrews | meghal: i believe that is correct | 19:43 |
| *** Marga_ has quit IRC | 19:48 | |
| *** alexpilotti has quit IRC | 19:48 | |
| meghal | ok…great…that brings more clarity to my understanding | 19:50 |
| meghal | thanks Shrews and devananda | 19:51 |
| *** dencaval has quit IRC | 19:52 | |
| jroll | NobodyCam: I haven't, got a link or something? | 19:52 |
| jroll | Shrews: meghal: the hash ring manages all nodes, all nodes get the sync power state loop :/ | 19:53 |
| Shrews | jroll: i thought conductor affinity was only set when deployed? | 19:53 |
| jroll | Shrews: hash ring != conductor affinity | 19:54 |
| meghal | jroll: mapping is obtained from hash ring and hash ring has nodes which are provisioned…is that correct ? | 19:55 |
| jroll | meghal: hash ring has all nodes | 19:55 |
| jroll | but not all nodes have a conductor affinity | 19:55 |
| jroll | (because that's only needed for ACTIVE nodes) | 19:55 |
| NobodyCam | jroll: ??? | 19:57 |
| jroll | ??? what | 19:57 |
| NobodyCam | jroll: link for? | 19:57 |
| jroll | NobodyCam: oh, ha, the user survey | 19:57 |
| *** ijw has joined #openstack-ironic | 19:57 | |
| meghal | jroll: so when node is added to ironic, which periodic task will add the new node in hash ring? is it sync_local_state ? | 19:59 |
| NobodyCam | oh : https://www.openstack.org/user-survey | 19:59 |
| jroll | NobodyCam: thanks | 19:59 |
| jroll | meghal: I'm not sure off hand, I'll need to look | 19:59 |
| *** kkoski has quit IRC | 20:00 | |
| jroll | meghal: oh man, I think you're right. seems like a bug. | 20:00 |
| jroll | (the only-update-active-nodes thing there) | 20:00 |
| *** kkoski has joined #openstack-ironic | 20:01 | |
| *** kkoski has quit IRC | 20:02 | |
| *** kkoski has joined #openstack-ironic | 20:02 | |
| jroll | meghal: actually, no. see self.ring_manager.reset() there | 20:03 |
| jroll | that will reset the hash ring, which will force an update | 20:03 |
| meghal | jroll: yeah - https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1134 | 20:04 |
| meghal | jroll: so reset is being called as part of sync_local_state and internally hash rings are all updated | 20:04 |
| *** alexpilotti has joined #openstack-ironic | 20:04 | |
| jroll | meghal: correct | 20:04 |
| meghal | jroll: then sync_local_state goes on to update affinity and handle take over of active nodes | 20:04 |
| jroll | correct | 20:05 |
| meghal | jroll: periodic power sync uses same hash ring built during sync_local_state to obtain current conductor's mapping and report only power state for nodes which belong to current conductor in the mapping…right ? | 20:06 |
| *** zer0c00l has quit IRC | 20:10 | |
| jroll | meghal: correct | 20:10 |
| *** zer0c00l has joined #openstack-ironic | 20:10 | |
| JayF | The IPA docs build/upload worked -- the first time! http://docs.openstack.org/developer/ironic-python-agent/ | 20:11 |
| meghal | ok…perfect…that makes it clear | 20:14 |
| meghal | jroll: thanks | 20:15 |
| *** kkoski has quit IRC | 20:16 | |
| *** alexpilotti has quit IRC | 20:16 | |
| jroll | np meghal :) | 20:16 |
| JayF | JoshNang: jroll: one of you mind landing this? https://review.openstack.org/#/c/170259/ | 20:17 |
| *** Marga_ has joined #openstack-ironic | 20:19 | |
| jroll | JayF: I won the race | 20:19 |
| *** dprince has quit IRC | 20:19 | |
| JoshNang | lol | 20:20 |
| JayF | everyone wins | 20:20 |
| JayF | when patches get merged | 20:20 |
| NobodyCam | :) | 20:20 |
| JoshNang | jroll: heh you're +A wasn't there when I clicked review. double +A | 20:20 |
| JayF | s/'re/r/ | 20:21 |
| JoshNang | gah! | 20:21 |
| JayF | I'm really happy + surprised the docs build worked on the first try | 20:22 |
| JayF | although we still have lots of docs to write that's a great start, and it'll be easier to get contributions since they actually have a place to go | 20:22 |
| NobodyCam | JayF: +++++ | 20:22 |
| JoshNang | jroll: did you put up a fix for validate() in the agent requiring deploy_kernel/ramdisk? | 20:23 |
| jroll | JoshNang: nope, haven't touched it | 20:23 |
| JoshNang | jroll: kk, i'll roll it in with the other none dhcp provider fixes | 20:23 |
| jroll | sick, thanks | 20:24 |
| *** Marga_ has quit IRC | 20:24 | |
| *** ijw_ has joined #openstack-ironic | 20:24 | |
| *** ijw has quit IRC | 20:27 | |
| *** krtaylor has quit IRC | 20:28 | |
| *** andreykurilin_ has joined #openstack-ironic | 20:28 | |
| NobodyCam | TheJulia: just checking on 168120 that the string you are searching for there is not translated if the operator is out side the us? | 20:30 |
| *** ijw_ has quit IRC | 20:30 | |
| *** ijw has joined #openstack-ironic | 20:31 | |
| *** krtaylor has joined #openstack-ironic | 20:33 | |
| *** kkoski has joined #openstack-ironic | 20:35 | |
| NobodyCam | 169643 +a'd | 20:42 |
| *** trown is now known as trown|outttypeww | 20:42 | |
| TheJulia | NobodyCam: as far as I can tell, it is a string returned directly from the firmware, so honestly I think it would be dependent upon the chassis | 20:45 |
| *** firl has quit IRC | 20:46 | |
| TheJulia | figures my grep failed when I go back and look again | 20:54 |
| NobodyCam | ofc | 20:54 |
| NobodyCam | :-p | 20:54 |
| TheJulia | NobodyCam: its a hardcoded string in ipmitool | 20:55 |
| *** Marga_ has joined #openstack-ironic | 20:58 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic: Add retry logic to _exec_ipmitool https://review.openstack.org/168120 | 21:00 |
| openstackgerrit | Merged openstack/ironic-python-agent: Error doc builds on WARN or ERROR https://review.openstack.org/170259 | 21:01 |
| NobodyCam | Ty TheJulia forthe Note | 21:02 |
| JayF | TheJulia: https://review.openstack.org/#/c/168120/8 had a single issue I found, you might wanna fix it before it gets +2s on it | 21:06 |
| *** kkoski1 has joined #openstack-ironic | 21:09 | |
| *** kkoski has quit IRC | 21:09 | |
| *** meghal has quit IRC | 21:10 | |
| TheJulia | JayF: they are fundimentally different messages though, I kind of wonder if doing so might lead to partial translations | 21:12 |
| JayF | TheJulia: I thought they were identical messages? | 21:12 |
| JayF | TheJulia: OOOH, I see | 21:13 |
| JayF | TheJulia: you're right, I'm wrong | 21:13 |
| TheJulia | :) | 21:15 |
| NobodyCam | brb | 21:19 |
| *** kkoski1 has quit IRC | 21:20 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: None DHCP Provider does not work with agent driver https://review.openstack.org/170984 | 21:21 |
| JayF | JoshNang: ^ not trying to be that guy but... this should have a bug, and it should be labelled in the commit | 21:22 |
| JayF | JoshNang: too big of a bug to not have it documented in lp imo | 21:22 |
| *** meghal has joined #openstack-ironic | 21:23 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: None DHCP Provider does not work with agent driver https://review.openstack.org/170984 | 21:24 |
| JoshNang | JayF: heh i was already on it | 21:24 |
| JoshNang | as soon as i hit review i said 'dang it, need a bug' | 21:25 |
| TheJulia | I have a feeling that this change will break bifrost | 21:33 |
| NobodyCam | TheJulia: which change? | 21:33 |
| TheJulia | 170984\ | 21:33 |
| NobodyCam | humm | 21:34 |
| JoshNang | :( | 21:36 |
| JoshNang | anyway to avoid that? it was definitely breaking when i tried to deploy mostly stock ironic to our preprod env | 21:36 |
| TheJulia | How did it break? | 21:37 |
| TheJulia | See alternative suggestion in my review :) | 21:38 |
| JayF | TheJulia: for instance; we don't have or need deploy_kernel or deploy_ramdisk on nodes | 21:38 |
| JayF | TheJulia: because our external DHCP implementation includes tftp | 21:38 |
| TheJulia | Yup, that makes sense | 21:38 |
| JayF | I don't think having ironic run tftp while not running dhcp can make sense either | 21:38 |
| JayF | because you can't know which conductor a node is assigned to in the hash ring | 21:39 |
| JayF | so you can't know which conductor is running the tftp server for that node ahead of time | 21:39 |
| JayF | to configure your None external DHCP provider | 21:39 |
| TheJulia | In a small scale it does make sense though | 21:39 |
| JayF | I mean, with 1 conductor | 21:39 |
| JayF | which is a scale I don't think we should support at all, tbh -- you, at that point, are a SPOF and basically can't scale like a normal Ironic deployment | 21:40 |
| TheJulia | If your purely using it to orchustrate and stand-up machines that will continue to live, does it really matter to have a scaled/redundant ironic deployment? | 21:40 |
| JayF | It matters because regardless of what your use case is, if that option/feature exists, and we can't scale it up, then we (Ironic) will be to blame | 21:42 |
| JayF | See our current bug about people using Fake drivers in production because they didn't know better | 21:42 |
| jroll | JayF: TheJulia: so the thing is, 'dhcp_provider=none' isn't enough to handle all the use cases | 21:44 |
| TheJulia | jroll: Absoloutely agree | 21:45 |
| jroll | having those checks in 170984 is also a pretty harsh layer violation :/ | 21:45 |
| JayF | jroll: I'm trying to think of a use case that works for >1 conductor that involves Ironic not managing dhcp, but managing tftp/pxe configs | 21:45 |
| JayF | jroll: I can't think of any | 21:45 |
| JayF | I'm curious where else that setting is ... leaky | 21:45 |
| devananda | JayF: active-passive conductors with a floating IP where both build all the configs | 21:46 |
| jroll | JayF: I don't see why this doesn't work for many conductors | 21:46 |
| JayF | devananda: Is that an Ironic-supported config? | 21:46 |
| devananda | (never put those words together before, it's probably a horrible idea) | 21:46 |
| devananda | JayF: i have no idea | 21:46 |
| JayF | jroll: where do you tell your DHCP server to send the node for next-server? | 21:46 |
| jroll | devananda: I don't think that would work in ironic today | 21:46 |
| JayF | jroll: that's the reason it doesn't work | 21:46 |
| TheJulia | JayF: I do agree, I'm just thinking that if we focus only on massively scalable, then we loose sight of the 50 node cluster that would drive adoption | 21:46 |
| jroll | JayF: can't there be many next-servers? | 21:47 |
| jroll | where it tries in order or whatever | 21:47 |
| jroll | I feel like I've seen this but imbw | 21:47 |
| *** andreykurilin__ has joined #openstack-ironic | 21:47 | |
| JayF | jroll: With how bad Ironic can be at cleaning up PXE configs, even that seems like a dubious idea | 21:47 |
| *** andreykurilin_ has quit IRC | 21:47 | |
| jroll | sure | 21:47 |
| JayF | TheJulia: I agree, but I want to think of something that'll work for you + still be allowed to scale | 21:47 |
| JayF | TheJulia: hell, even dhcp_provider=PxeOnly, maybe | 21:48 |
| JayF | or however you'd format that | 21:48 |
| jroll | thing is, pxe configs / tftp stuff / etc isn't done in the dhcp provider today | 21:48 |
| TheJulia | JayF: to that, I think we're in agreement :) | 21:48 |
| jroll | hence the giant hack here | 21:48 |
| JoshNang | letting the dhcp drivers do most of this work seems like a good way to handle this | 21:48 |
| JoshNang | err yeah, what jroll said | 21:48 |
| devananda | jroll: yes, it may be a list -- https://tools.ietf.org/html/draft-ietf-dhc-nextserver-02 | 21:48 |
| devananda | 6.1. Next Server IP address option | 21:49 |
| devananda | The Next Server IP address Option specifies a list of IP | 21:49 |
| devananda | addresses for secondary servers | 21:49 |
| jroll | so for kilo, I think one of these two cases needs to win | 21:49 |
| jroll | I don't tend to care too much which, this patch isn't painful for us to carry downstream | 21:49 |
| jroll | but otoh then this isn't really a usable config for others, at >1 conductor | 21:49 |
| JayF | jroll: yeah, I agree with you | 21:49 |
| JayF | that being said ... the likely only two sets of stakeholders are here | 21:50 |
| JayF | heh | 21:50 |
| JayF | roshambo for supremecy? heh | 21:50 |
| jroll | JayF: well, anyone that deploys ironic in the next six months is a possible stakeholder | 21:50 |
| jroll | because release cycles etc | 21:50 |
| devananda | this code's going to stick around longer than that | 21:50 |
| JayF | jroll: ugh, I forget people go 6 months or longer without upgrading :( | 21:50 |
| jroll | devananda: long term it must be made better, short term is what counts 3 days from release | 21:51 |
| JayF | How hard/reasonable is it to put that code in the DHCP provider before K-release? | 21:51 |
| JayF | because that's the real/best fix, right? That code goes into DHCP provider, add a DHCP driver that does pxe-only | 21:51 |
| JoshNang | JayF: there's a fair bit of code | 21:51 |
| jroll | seems like a large refactor, I probably wouldn't +2 | 21:51 |
| jroll | I'd rather think about the cases better | 21:51 |
| JayF | that's what I thought, but just making sure | 21:51 |
| jroll | this is probably a great candidate for the whole BootInterface thing | 21:51 |
| TheJulia | It seems like dhcp management and pxe management just been to be split apart in the discussion | 21:52 |
| devananda | jroll: ++ | 21:52 |
| devananda | the whole deploy vs. boot interface split should make this stuff much clearer | 21:52 |
| devananda | also, what's actually broken right now? | 21:52 |
| devananda | JayF: I get the concern that "if its there, someone will (mis)use it" | 21:52 |
| jroll | devananda: none dhcp provider without putting deploy_kernel/ramdisk is broken | 21:52 |
| devananda | but what's -actually- broken that warrants a rush fix before RC1? | 21:52 |
| JayF | devananda: also more like "Do we *ever* want to encourage people to run a single conductor?" -- my answer to that is very no | 21:53 |
| devananda | JayF: sorry, I'm going to do it ALL the time | 21:53 |
| jroll | devananda: e.g. ironic not caring about the deploy ramdisk is broken. as in, how we run our environment | 21:53 |
| devananda | JayF: that's how we test in devstack. that's how I will run my home lab. | 21:53 |
| jroll | (it's been broken, we just hacked around it) | 21:53 |
| jroll | devananda: I think JayF is saying "don't ship code that breaks with >1 conductor" | 21:54 |
| JayF | devananda: sure; but that's to test for production environments. Which have >1 conductor, and which would completly dissapoint you as a deployer if something worked in devstack + home lab and then later broke when deploying it to >1 conductor in production | 21:54 |
| devananda | jroll: that's quite a different statement, and one I would definitely agree with | 21:54 |
| JayF | If anything; the fact most people run with 1 conductor makes me even more scared of adding a configuration which would generally break with a single conductor | 21:54 |
| JayF | because it means we're setting up a user for success in a lab but later production failures | 21:55 |
| mrda-away | Morning Ironic | 21:55 |
| *** mrda-away is now known as mrda | 21:55 | |
| devananda | "do we want 1 coductor to work?" -- yes. "should we sacrifice any multi-conductor functionality to do that?" definitely not. | 21:55 |
| JayF | devananda: here's the question to you: Should Ironic support features that only work in single-conductor environments? | 21:55 |
| jroll | JayF: (just thought of another way this works, put /tftpboot on a shared filesystem and yes that's horrible please don't hurt me) | 21:56 |
| JayF | jroll: I mean, you haven't sold me on that being that horrible yet | 21:56 |
| devananda | jroll: so we've got a proposal for exactly that. go look at the irmc driver's virtual media code | 21:56 |
| devananda | s/exactly/basically/ | 21:56 |
| jroll | JayF: assuming we have locks aroudn that stuff, it's probably fine, but afaik conductors could stomp on each other in that config today | 21:57 |
| jroll | devananda: right, I don't love that either :) | 21:57 |
| JayF | jroll: what I was thinking would be awesome is with iPXE + generated configs, you could have a static DHCP setup give out ipxe configs + an API vip to point to get the ipxe configs, then generate them on the fly | 21:57 |
| JayF | jroll: that's really the grade A solution to this kind of problem | 21:57 |
| jroll | JayF: lucas is working on that :) | 21:57 |
| JayF | jroll: have the static configs bootstrap you into a higher protocol, then use that to avoid the problem entirely | 21:57 |
| jroll | yep | 21:57 |
| JayF | Hell, we'd probably switch to that | 21:57 |
| devananda | JayF: that would be great. and yea, lucas has been working on that. i'm sure he'd love help :) | 21:57 |
| jroll | at any rate, this is far from the immediate problem | 21:57 |
| devananda | indeed | 21:57 |
| jroll | but I need to run to a meeting and I know the rest of j* does too :/ | 21:58 |
| jroll | but really one of these two models is going to break | 21:59 |
| devananda | :-/ | 22:00 |
| JayF | TheJulia: can we get an official pronunciation for "bifrost" 'bye-frost'? | 22:00 |
| jroll | I saw a movie the other day where it was pronounced bye-frost and that's how I've always said it | 22:01 |
| jroll | so wfm | 22:01 |
| *** firl has joined #openstack-ironic | 22:02 | |
| TheJulia | JayF: Sure, but first I need to go on vacation, go to the top of a mountain, and record what me saying it with my deepest most sinister voice, and then translate that into a pronouciation. :) | 22:02 |
| TheJulia | s/me/I'm/ | 22:02 |
| JoshNang | LOL! | 22:03 |
| devananda | TheJulia: +100 | 22:03 |
| TheJulia | JayF: hopefully I'll remember to pack the boom bike so I can get some good audio with my camera... and hopefully I'm not shivering at the time | 22:04 |
| devananda | TheJulia: you'll be warm and toasty from the hike to the top of te mountain | 22:05 |
| JayF | TheJulia: is it prounouced bye-doooooooooooooooooooom-frost? | 22:05 |
| * mrda has never heard of a boom bike | 22:05 | |
| *** wanyen has quit IRC | 22:07 | |
| TheJulia | JayF: bye-frost of doooooooom maybe | 22:07 |
| TheJulia | well, more ooooos are needed in that, but yeah :) | 22:07 |
| JayF | TheJulia: bifrost brings DOOOM TO SERVERS? | 22:07 |
| * TheJulia thinks bifrost will need a light red logo now | 22:07 | |
| JayF | TheJulia: ^ there's your winner, lmk if you need a professsional voiceover, haha | 22:07 |
| JayF | TheJulia: extra light red | 22:07 |
| TheJulia | lol | 22:07 |
| *** achanda has quit IRC | 22:08 | |
| TheJulia | Oh the memories... | 22:08 |
| TheJulia | JayF: and inside jokes :) | 22:09 |
| TheJulia | devananda: I'm planning on doing sunrise photography, so I'll be getting to the top of the mountain like an hour before sunrise.... I will be cold :( | 22:12 |
| mrda | TheJulia: Sounds lovely! (except the cold bit) | 22:14 |
| *** achanda has joined #openstack-ironic | 22:14 | |
| *** andreykurilin__ has quit IRC | 22:14 | |
| devananda | TheJulia: pre-sunrise is a great time to be shouting doom from the mountaintops :) | 22:15 |
| devananda | *dooooooom | 22:15 |
| devananda | *doooooooooooooooom | 22:15 |
| TheJulia | hehe | 22:16 |
| *** romcheg has quit IRC | 22:16 | |
| * mrda wonders whether devananda is talking about RC1 :) | 22:16 | |
| mrda | devananda: thanks btw for pushing through review 169643. I think it's great we got this one in. | 22:18 |
| NobodyCam | morning mrda | 22:19 |
| * NobodyCam liked Doom ][ better | 22:19 | |
| mrda | NobodyCam: hey | 22:20 |
| openstackgerrit | Merged openstack/ironic: Disable XML now that we have WSME/Pecan support https://review.openstack.org/169643 | 22:31 |
| devananda | mrda: welcome. thx for starting it | 22:34 |
| mrda | devananda: np | 22:36 |
| openstackgerrit | Merged openstack/ironic: Refactor test code to reduce duplication https://review.openstack.org/169981 | 22:37 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Shorten time for unittest test_create_cleaning_ports_fail https://review.openstack.org/170999 | 22:38 |
| * NobodyCam brb | 22:41 | |
| *** ijw has quit IRC | 22:42 | |
| *** ijw has joined #openstack-ironic | 22:43 | |
| *** jrist has quit IRC | 22:47 | |
| *** jrist has joined #openstack-ironic | 22:49 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Shorten time for unittest test_download_with_retries https://review.openstack.org/171002 | 22:54 |
| * jlvillal revels in the glory of shortening the time of unit tests :D | 22:55 | |
| jlvillal | TheJulia: The reason I don't do landscape photography. Too much work :) Waking up super early, hiking out in the cold, and then hoping when the golden hour arrives the sky looks good... | 22:57 |
| JoshNang | jlvillal: ahh thanks for fixing the cleaning port fail test. its been on my list...>1s tests bug me too | 22:57 |
| jlvillal | JoshNang: You're welcome | 22:58 |
| mrda | jlvillal: but there is something grand about capturing the shot at just the right moment... | 22:59 |
| *** devlaps has joined #openstack-ironic | 22:59 | |
| *** igordcard_ has joined #openstack-ironic | 22:59 | |
| jlvillal | JoshNang: It is giving me the chance to wander around in the code. Though I will be honest I have no idea where the time.sleep(0) call is coming from. When I wandered around in the code I couldn't figure out how it was called. But mock was telling me it is called. | 22:59 |
| jlvillal | mrda: Totally true! I just don't have the motivation to do. I do admire the people who have the motivation to do it though. | 23:00 |
| TheJulia | jlvillal: but the photos can be amazing! :) | 23:00 |
| jlvillal | TheJulia: Yes they can! :) | 23:01 |
| * mrda just bought a 7DMkII to replace his, ahem, older camera | 23:01 | |
| jlvillal | JoshNang: I'm thinking it is this: openstack/common/periodic_task.py: time.sleep(0) but I wasn't sure how that was getting called. But mocking time.sleep got rid of the long test time. | 23:01 |
| TheJulia | I <3 my 5DMkII | 23:01 |
| jlvillal | mrda: TheJulia: Good cameras both :) I'm a Nikon guy, but I think Canon, Nikon, and Sony all make good stuff. | 23:02 |
| TheJulia | Although, I've been tempted to pickup a film camera | 23:02 |
| mrda | TheJulia: \o/ I strongly considered buying tat when it came out, but I didn't want to go full-frame | 23:02 |
| jlvillal | TheJulia: Me too, but then I think about the actual effort involved in getting stuff developed. Used film cameras can be pretty cheap these days. | 23:02 |
| TheJulia | jlvillal: they make kits to do processing at home, I know a few people locally that develop their own film | 23:03 |
| TheJulia | but yeah, time | 23:03 |
| mrda | jlvillal, TheJulia: It's more the time for me. I don't have enogh time to shoot, let alone process at home | 23:03 |
| TheJulia | yeah | 23:04 |
| mrda | snap | 23:04 |
| TheJulia | my camera has barely exited it's bag in the last year | 23:04 |
| TheJulia | :( | 23:04 |
| mrda | TheJulia: Well, go climb a mountain then :) | 23:04 |
| jlvillal | mrda, TheJulia: Sound like me :( I got a lot of gear but haven't been doing much lately. I need to get back into it. Do some meetups... | 23:05 |
| *** jamielennox|away is now known as jamielennox | 23:07 | |
| TheJulia | Which reminds me, I took my lighting and backdrop kits to my partner's house, I should like... take them out of the entryway and stuff | 23:09 |
| *** chlong has joined #openstack-ironic | 23:10 | |
| jroll | jlvillal: in general, time.sleep(0) is used to signal eventlet to yield to another thread in the pool (not sure where that call in particular comes from) | 23:20 |
| jlvillal | jroll: I wandered the code for a bit looking but ran out of steam trying to track it down. The unittest fix worked and it did show that time.sleep was being called. | 23:21 |
| jroll | heh | 23:21 |
| jlvillal | jroll: So I said, "good enough!" | 23:22 |
| jroll | jlvillal: I see time.sleep(1) called in ironic/common/glance_service/base_image_service.py | 23:23 |
| jroll | I assume that's what you're looking for? | 23:23 |
| *** yuanying has joined #openstack-ironic | 23:25 | |
| * mrda always gets concerned about time.sleep() calls in production code | 23:27 | |
| jroll | for retry? :P | 23:27 |
| mrda | Well, sometimes they just hide races | 23:28 |
| mrda | or scare heisenbugs to move elsewhere | 23:28 |
| jroll | yeah, true | 23:31 |
| devananda | BadCub: how do you feel about the title "facilitator"? | 23:35 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow IPA to request reboot after clean steps https://review.openstack.org/169504 | 23:36 |
| jlvillal | jroll: Yes that time.sleep I saw. It was the cleaning one I was confused about | 23:39 |
| jroll | jlvillal: I'm looking at the wrong patch then :) | 23:40 |
| jlvillal | JoshNang: Do you want to fix the cleaning ports? Or should I just update my patch to try mocking the rollback function? | 23:40 |
| JoshNang | jlvillal: i can update that patch | 23:40 |
| jlvillal | JoshNang: Thanks :) | 23:40 |
| jlvillal | jroll: It is: https://review.openstack.org/170999 | 23:40 |
| JayF | JoshNang: https://review.openstack.org/#/c/169504 -> I just put a comment on, you missed one thing Dmitry pointed out | 23:41 |
| jroll | jlvillal: sorry, still not seeing the time.sleep(0) or why it matters | 23:42 |
| jlvillal | jroll: I couldn't find it either. But mock says time.sleep is called. | 23:43 |
| jroll | jlvillal: I guess I don't get why it matters | 23:44 |
| jlvillal | JoshNang: I'm also unsure why it decreased the time, but with my patch it no longer takes 2+ seconds. I don't see it listed after I do the patch. | 23:44 |
| jroll | if you can't find it, don't assert that it's called, right? | 23:44 |
| jlvillal | jroll: I put the assert in there to sort of prove that yes time.sleep really is being called. | 23:44 |
| jroll | jlvillal: but does it matter? | 23:45 |
| jlvillal | jroll: But JoshNang is going to mock a different function. So the time.sleep part should go away. | 23:45 |
| JoshNang | so the problem was a neutron delete call wasn't getting mocked. i'm guessing neutron uses time.sleep for its retries? | 23:45 |
| jroll | jlvillal: in other words, that test shouldn't be testing that time.sleep is called, because that's an implementation detail outside of the method being tested | 23:45 |
| jroll | ](imho) | 23:45 |
| jroll | s/]// | 23:45 |
| jlvillal | JoshNang: Sorry about that. Sometimes I'm unsure about how much people want to allow their unittests to run other functions. | 23:46 |
| mrda | +1 | 23:46 |
| JoshNang | jlvillal: don't be sorry, i'm the one that missed the mock :P | 23:46 |
| jlvillal | JoshNang: I was assuming that it was desired to call that delete call in the unittest. | 23:46 |
| *** Marga_ has quit IRC | 23:47 | |
| jlvillal | jroll: Makes sense on not checking that time.sleep is called. | 23:47 |
| *** stendulker has joined #openstack-ironic | 23:48 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Shorten time for unittest test_create_cleaning_ports_fail https://review.openstack.org/170999 | 23:48 |
| jlvillal | JoshNang: Do you want to update the commit message? | 23:48 |
| JoshNang | sure | 23:49 |
| jlvillal | JoshNang: Thanks! | 23:49 |
| *** achanda has quit IRC | 23:51 | |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Add missing mock for test_create_cleaning_ports_fail https://review.openstack.org/170999 | 23:52 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic: Allow IPA to request reboot after clean steps https://review.openstack.org/169504 | 23:55 |
| jlvillal | jroll: mock_clean instead of mock_sleep was a brain fart :( | 23:55 |
| jroll | jlvillal: ha, I figured | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!