| hogepodge | TheJulia: devananda: thanks for your help and patience, it booted the ironic images, now on to deploying. :-D | 00:01 |
|---|---|---|
| TheJulia | \o/ | 00:02 |
| TheJulia | Excellent, stepping away for a little bit | 00:02 |
| mgagne | so I tried to reproduce my power sync issue where response from Ironic API is empty. With 1000 greenpool workers and 1000 requests, I'm getting zero issue in an artificial load test I made. both in dev and prod (which has the issue from same source to same destination) :-/ | 00:03 |
| devananda | hogepodge: woot! yvw :) | 00:04 |
| *** devananda is now known as devananda|dinner | 00:04 | |
| *** rpioso has quit IRC | 00:07 | |
| mgagne | devananda|dinner: I think we didn't have the opportunity to sync up after ops meetup. I'm the guy that reported some performance issue with Nova scheduler + Ironic. We finally found the issue and the fix can be found here: https://review.openstack.org/#/c/363944/ And more details were posted on the ops mailinglist: http://lists.openstack.org/pipermail/openstack-operators/2016-August/011429.html | 00:08 |
| patchbot | patch 363944 - nova - Make scheduler less greedy when loading instances ... | 00:08 |
| *** vishwanathj is now known as vishwanathj_zzz | 00:08 | |
| *** vishwanathj_zzz is now known as vishwanathj_ | 00:08 | |
| *** rbrud has joined #openstack-ironic | 00:16 | |
| *** rbrud has quit IRC | 00:17 | |
| *** rbrud has joined #openstack-ironic | 00:17 | |
| *** rbrud has quit IRC | 00:17 | |
| devananda|dinner | mgagne: ah! I wish everyone had their IRC nic on. it was great to meet you in person | 00:17 |
| devananda|dinner | I've marked that to take a look at it later | 00:18 |
| mgagne | devananda|dinner: true about irc nick =) | 00:18 |
| mgagne | thanks! | 00:18 |
| devananda|dinner | thanks for all your feedback & patches like that | 00:18 |
| mgagne | =) trying hard to upgrade to latest we can talk about "real" issues | 00:18 |
| *** rama_y has quit IRC | 00:23 | |
| openstackgerrit | Merged openstack/ironic-python-agent: [Mitaka only] Fix incomplete root device hints validation https://review.openstack.org/366050 | 00:24 |
| *** wajdi has joined #openstack-ironic | 00:29 | |
| *** rloo has quit IRC | 00:42 | |
| *** hoangcx has joined #openstack-ironic | 00:42 | |
| *** rajinir has quit IRC | 00:55 | |
| *** mtreinish has quit IRC | 01:01 | |
| *** mtreinish has joined #openstack-ironic | 01:05 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: osc node power on & off commands https://review.openstack.org/364440 | 01:11 |
| *** aNupoisc has quit IRC | 01:26 | |
| *** aNupoisc has joined #openstack-ironic | 01:26 | |
| *** syed_ has quit IRC | 01:32 | |
| *** wajdi has quit IRC | 01:39 | |
| *** wajdi has joined #openstack-ironic | 01:39 | |
| *** wajdi_ has joined #openstack-ironic | 01:46 | |
| *** wajdi has quit IRC | 01:49 | |
| *** melwitt has joined #openstack-ironic | 01:50 | |
| *** aNupoisc has quit IRC | 01:55 | |
| *** thrash is now known as thrash|g0ne | 01:57 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: 'ironic create' handles file args https://review.openstack.org/367715 | 02:12 |
| *** wajdi_ has quit IRC | 02:24 | |
| *** wajdi has joined #openstack-ironic | 02:24 | |
| *** chlong has joined #openstack-ironic | 02:26 | |
| *** yibo has joined #openstack-ironic | 02:30 | |
| *** devananda|dinner has quit IRC | 02:32 | |
| *** phuongnh has joined #openstack-ironic | 02:43 | |
| *** amotoki has quit IRC | 02:49 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: __ne__() unit tests & have special methods use (self, other) https://review.openstack.org/367643 | 02:53 |
| *** rbudden has quit IRC | 02:56 | |
| *** devananda|dinner has joined #openstack-ironic | 02:59 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: __ne__() unit tests & have special methods use (self, other) https://review.openstack.org/367643 | 02:59 |
| *** joprovost has quit IRC | 03:02 | |
| *** fragatina has quit IRC | 03:03 | |
| *** wajdi_ has joined #openstack-ironic | 03:33 | |
| *** wajdi has quit IRC | 03:37 | |
| *** wajdi_ has quit IRC | 03:44 | |
| *** amotoki has joined #openstack-ironic | 03:49 | |
| *** tuanla_ has joined #openstack-ironic | 03:54 | |
| *** wajdi has joined #openstack-ironic | 03:55 | |
| *** absubram has joined #openstack-ironic | 04:16 | |
| *** amotoki has quit IRC | 04:16 | |
| *** absubram_ has joined #openstack-ironic | 04:16 | |
| *** hamzy has joined #openstack-ironic | 04:19 | |
| *** absubram has quit IRC | 04:20 | |
| *** absubram_ is now known as absubram | 04:20 | |
| *** harlowja_at_home has joined #openstack-ironic | 04:23 | |
| *** fragatina has joined #openstack-ironic | 04:26 | |
| *** fragatina has quit IRC | 04:30 | |
| *** rm_work has quit IRC | 04:31 | |
| *** rm_work has joined #openstack-ironic | 04:34 | |
| *** rm_work has quit IRC | 04:36 | |
| *** rama_y has joined #openstack-ironic | 04:38 | |
| *** rm_work has joined #openstack-ironic | 04:38 | |
| *** ChubYann has quit IRC | 05:06 | |
| *** jaosorior has joined #openstack-ironic | 05:21 | |
| *** rama_y has quit IRC | 05:24 | |
| *** moshele has joined #openstack-ironic | 05:27 | |
| *** vishwanathj_ is now known as vishwanathj_zzz | 05:28 | |
| *** chlong_ has joined #openstack-ironic | 05:31 | |
| *** harlowja_at_home has quit IRC | 05:33 | |
| *** amotoki has joined #openstack-ironic | 05:43 | |
| *** mjura has joined #openstack-ironic | 06:01 | |
| *** mjura has quit IRC | 06:09 | |
| *** mjura has joined #openstack-ironic | 06:09 | |
| *** ccamacho has joined #openstack-ironic | 06:13 | |
| *** amotoki has quit IRC | 06:19 | |
| *** moshele has quit IRC | 06:21 | |
| *** e0ne has joined #openstack-ironic | 06:23 | |
| *** pcaruana has joined #openstack-ironic | 06:23 | |
| *** amotoki has joined #openstack-ironic | 06:28 | |
| *** harlowja_at_home has joined #openstack-ironic | 06:29 | |
| *** phuongnh has quit IRC | 06:31 | |
| *** phuongnh has joined #openstack-ironic | 06:32 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Clean nodes stuck in CLEANING state when ir-cond restarts https://review.openstack.org/349971 | 06:37 |
| *** harlowja_at_home has quit IRC | 06:47 | |
| *** harlowja_at_home has joined #openstack-ironic | 06:48 | |
| *** amotoki has quit IRC | 06:48 | |
| *** Ukesh has joined #openstack-ironic | 06:48 | |
| Ukesh | join | 06:49 |
| patchbot | Error: You don't have the admin capability. If you think that you should have this capability, be sure that you are identified before trying again. The 'whoami' command can tell you if you're identified. | 06:49 |
| Ukesh | I am having one query. in openstackclient, I couldn't find the code related to baremetal option | 06:50 |
| Ukesh | can anyone help on this ? | 06:51 |
| *** amotoki has joined #openstack-ironic | 06:52 | |
| *** fxpester has joined #openstack-ironic | 06:56 | |
| *** harlowja_at_home has quit IRC | 06:57 | |
| Ukesh | https://bugs.launchpad.net/python-ironicclient/+bug/1619091 | 06:59 |
| openstack | Launchpad bug 1619091 in python-ironicclient "[RFE] Extend OSC "node list" command to fetch nodes without instance UUID" [Wishlist,Confirmed] - Assigned to Ukesh (ukeshkumar) | 06:59 |
| *** amotoki has quit IRC | 07:02 | |
| *** tesseract- has joined #openstack-ironic | 07:07 | |
| *** amotoki has joined #openstack-ironic | 07:14 | |
| *** chlong has quit IRC | 07:19 | |
| *** chlong_ has quit IRC | 07:19 | |
| *** mgoddard has joined #openstack-ironic | 07:23 | |
| *** tuanla_ has quit IRC | 07:27 | |
| *** tuanla_ has joined #openstack-ironic | 07:27 | |
| *** phuongnh has quit IRC | 07:28 | |
| *** phuongnh has joined #openstack-ironic | 07:29 | |
| *** jpich has joined #openstack-ironic | 07:29 | |
| openstackgerrit | Cao Xuan Hoang proposed openstack/python-ironic-inspector-client: TrivialFix: Remove logging import unused https://review.openstack.org/367788 | 07:34 |
| *** vishwana_ has joined #openstack-ironic | 07:42 | |
| *** vishwana_ has quit IRC | 07:42 | |
| *** vishwana_ has joined #openstack-ironic | 07:43 | |
| *** vishwanathj_zzz has quit IRC | 07:44 | |
| *** ohamada has joined #openstack-ironic | 07:47 | |
| *** mbound has joined #openstack-ironic | 07:54 | |
| *** vsaienk0 has joined #openstack-ironic | 07:55 | |
| *** causten has quit IRC | 07:58 | |
| *** mbound has quit IRC | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** e0ne has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:02 | |
| *** derekh has joined #openstack-ironic | 08:05 | |
| *** athomas has joined #openstack-ironic | 08:06 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 08:09 |
| *** mkoderer has quit IRC | 08:10 | |
| *** mkoderer has joined #openstack-ironic | 08:11 | |
| *** causten has joined #openstack-ironic | 08:11 | |
| pas-ha | morning Ironicers :) | 08:18 |
| *** snecklifter has joined #openstack-ironic | 08:19 | |
| *** hoangcx has quit IRC | 08:22 | |
| lucasagomes | morning all | 08:24 |
| openstackgerrit | Ankit Kumar proposed openstack/proliantutils: Fix Inspection failure in Gen7 https://review.openstack.org/357134 | 08:25 |
| *** vishwana_ has quit IRC | 08:25 | |
| *** vishwanathj_zzz has joined #openstack-ironic | 08:25 | |
| *** hoangcx has joined #openstack-ironic | 08:26 | |
| *** Romanenko_K has joined #openstack-ironic | 08:30 | |
| dtantsur | Morning Ironic | 08:40 |
| *** jaosorior is now known as jaosorior_lunch | 08:40 | |
| dtantsur | aarefiev, morning. so, we have a patch, but it needs more work. that's already good. I think for now I'll try to disable the failing test in our gate, so that we don't block the release. then we can continue testing a revert. wdyt? | 08:46 |
| dtantsur | also morning pas-ha, lucasagomes | 08:46 |
| *** ptrlv has quit IRC | 08:46 | |
| pas-ha | morning dtantsur lucasagomes | 08:47 |
| aarefiev | dtantsur: morning, +1 for skip test, looks like there are a lot of work here, a lot of workarounds for paramiko | 08:51 |
| aarefiev | btw I've catched deadlock in paramiko last night | 08:51 |
| aarefiev | I'm wondering why only inspector grenade fails so often | 08:53 |
| *** e0ne has joined #openstack-ironic | 08:55 | |
| *** vsaienk0 has left #openstack-ironic | 08:56 | |
| aarefiev | Also Matthew don't want to revert fix with receive_status and proposed fix also has problems ( | 08:56 |
| *** sdake has joined #openstack-ironic | 09:02 | |
| *** amotoki has quit IRC | 09:04 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Disable neutron tests in our grenade https://review.openstack.org/367841 | 09:06 |
| dtantsur | aarefiev, let's see if this works ^^^ | 09:06 |
| openstackgerrit | zhang.xiuhua proposed openstack/ironic: Change note format in installation Guide https://review.openstack.org/367843 | 09:08 |
| aarefiev | dtantsur: +1, not big deal if don't test neutron api :) | 09:11 |
| dtantsur | yeah, we can live with it for a while | 09:11 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 09:12 |
| *** mbound has joined #openstack-ironic | 09:14 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update configure_tenant_networks https://review.openstack.org/356460 | 09:15 |
| *** sdake_ has joined #openstack-ironic | 09:16 | |
| *** sdake has quit IRC | 09:18 | |
| *** moshele has joined #openstack-ironic | 09:27 | |
| *** mgould|afk is now known as mgould | 09:29 | |
| *** sambetts|afk is now known as sambetts | 09:30 | |
| *** causten_ has joined #openstack-ironic | 09:41 | |
| *** causten has quit IRC | 09:44 | |
| *** sborkows has joined #openstack-ironic | 09:48 | |
| *** sdake has joined #openstack-ironic | 09:57 | |
| *** hoangcx has quit IRC | 09:59 | |
| *** sdake_ has quit IRC | 10:00 | |
| *** phuongnh has quit IRC | 10:10 | |
| *** tuanla_ has quit IRC | 10:11 | |
| sborkows | morning all | 10:14 |
| dtantsur | morning sborkows | 10:16 |
| sborkows | dtantsur, aarefiev: Is there still a problem with grenade? | 10:16 |
| dtantsur | sborkows, yes; we're working on disabling the failed test, as we're still unable to track the problem :( we'll recheck your patch as soon as we're ready, no worries. | 10:16 |
| sborkows | ok thanks for effort | 10:16 |
| dtantsur | thanks for your patience | 10:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 10:18 |
| sborkows | :) | 10:18 |
| aarefiev | morning sborkows, we'll unblock gates asap, don't worry | 10:18 |
| mgould | morning dtantsur aarefiev pas-ha lucasagomes sborkows and everyone else! | 10:19 |
| aarefiev | morning mgould | 10:19 |
| lucasagomes | hi there | 10:20 |
| *** moshele has quit IRC | 10:23 | |
| *** thrash|g0ne is now known as thrash | 10:30 | |
| *** sdake_ has joined #openstack-ironic | 10:31 | |
| *** moshele has joined #openstack-ironic | 10:32 | |
| *** sdake has quit IRC | 10:33 | |
| dtantsur | aarefiev, https://review.openstack.org/#/c/367841/ passes, let's give it a try? | 10:33 |
| patchbot | patch 367841 - ironic-inspector - Disable neutron tests in our grenade | 10:33 |
| dtantsur | wait, wut? | 10:35 |
| dtantsur | I did run test_network_basic_ops though........ | 10:35 |
| aarefiev | dtantsur: hmm second tempest run contains test_network_basic | 10:35 |
| *** Guest90983 is now known as rook | 10:36 | |
| dtantsur | aarefiev, hmm, it was a noop | 10:36 |
| aarefiev | I don't see test on first run | 10:36 |
| aarefiev | tests | 10:36 |
| aarefiev | ahh missed I see | 10:37 |
| dtantsur | aarefiev, it hasn't set TEMPEST_CONFIG | 10:38 |
| dtantsur | here where it gets reset, I guess: http://logs.openstack.org/41/367841/1/check/gate-grenade-dsvm-ironic-inspector/738244b/logs/grenade.sh.txt.gz#_2016-09-09_09_28_15_086 | 10:38 |
| aarefiev | ahh | 10:40 |
| dtantsur | hmm, I guess we should use disable_service and kill neutron completely... but that will kill ironic essentially too | 10:41 |
| *** jaosorior_lunch is now known as jaosorior | 10:41 | |
| dtantsur | aarefiev, or we need to monkey-patch python source, which seems more and more reasonable to me... | 10:42 |
| aarefiev | could disable neutron on late stage somehow | 10:42 |
| aarefiev | in grenade plugin | 10:42 |
| dtantsur | aarefiev, or we can disable any of these: https://github.com/openstack/tempest/blob/master/tempest/scenario/test_network_basic_ops.py#L85-L93 | 10:45 |
| dtantsur | but again, it will probably get overriden by devstack... damn | 10:46 |
| aarefiev | yeah ( | 10:47 |
| *** athomas has quit IRC | 10:56 | |
| *** athomas has joined #openstack-ironic | 11:02 | |
| sturivnyi_ | Hi All. I have bunch of patches for review https://review.openstack.org/#/q/status:open+project:openstack/python-ironicclient+branch:master+topic:%22functional+CLI+tests%22. Would be grateful if you could review them. Thanks. | 11:07 |
| Romanenko_K | From those patches there is at least one patch already imho OK to merge https://review.openstack.org/#/c/293634/ | 11:09 |
| patchbot | patch 293634 - python-ironicclient - Tests for testing chassis-create command | 11:09 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on https://review.openstack.org/330003 | 11:12 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 11:14 |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Attach/Detach changes https://review.openstack.org/367948 | 11:14 |
| *** ohamada has quit IRC | 11:15 | |
| *** ohamada has joined #openstack-ironic | 11:15 | |
| *** absubram has quit IRC | 11:16 | |
| dtantsur | aarefiev, I don't understand anything.. sometimes it just passes.. | 11:23 |
| *** baoli has joined #openstack-ironic | 11:24 | |
| jroll | Ukesh: the code for the openstackclient baremetal commands is in python-ironicclient/ironicclient/osc/ | 11:24 |
| jroll | morning everyone | 11:24 |
| jroll | happy friday \o/ | 11:24 |
| dtantsur | morning jroll.. I've seen Fridays happier, to be honest. | 11:25 |
| jroll | :( | 11:26 |
| jroll | what's up, still same grenade issue? | 11:26 |
| dtantsur | jroll, yes... nobody has a tiniest clue what causes it | 11:26 |
| dtantsur | jroll, I would just disabled the damned neutron test, if grenade/tempest allowed that | 11:26 |
| jroll | hrm, don't they? | 11:26 |
| dtantsur | if you know how... | 11:27 |
| sambetts | regex? | 11:27 |
| jroll | yes, regex would work | 11:27 |
| dtantsur | sambetts, grenade does not pass any regexs to tempest | 11:27 |
| dtantsur | it just runs 'tempest run --smoke' or something | 11:27 |
| jroll | ah, right | 11:27 |
| *** baoli_ has joined #openstack-ironic | 11:27 | |
| sambetts | dtantsur: whats the conflict with Neutron? | 11:27 |
| lucasagomes | jroll, morning | 11:28 |
| dtantsur | sambetts, no clue. at all. nobody knows. it just does not work 90% of time in gate. | 11:28 |
| jroll | it's the paramiko thing, right? | 11:28 |
| dtantsur | jroll, presumably. unless it just hides something deeper | 11:28 |
| jroll | dtantsur: I'd maybe bump the qa thread on it and say "we need a way to skip this test for now, if we can't fix it, pls halp" | 11:29 |
| jroll | morning lucasagomes | 11:29 |
| dtantsur | jroll, qa folks were not so far able to come up with any SSH implementation that works for everyone | 11:29 |
| dtantsur | yeah, will bump probably | 11:29 |
| jroll | right | 11:29 |
| * jroll can't imagine what's special about ironic :( | 11:29 | |
| *** baoli has quit IRC | 11:30 | |
| jroll | er, about inspector | 11:31 |
| dtantsur | I can only think about the way we configure networking | 11:32 |
| dtantsur | it's the only difference between ironic devstack plugin and inspector devstack plugin... | 11:32 |
| dtantsur | I also have no clue why it worked until roughly a week ago | 11:32 |
| *** moshele has quit IRC | 11:33 | |
| jroll | well, that would make sense with ssh hanging | 11:33 |
| jroll | what's different about it? | 11:33 |
| dtantsur | but in 10% of cases is passes :) | 11:33 |
| dtantsur | lemme find a link | 11:33 |
| jroll | yeah, that's the weird part | 11:33 |
| dtantsur | jroll, essentially this bit https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L254-L258 and the fact that we have our own DHCP | 11:34 |
| jroll | dtantsur: so this one broke ironic... https://github.com/openstack-dev/devstack/commit/434035e2846dc8ebcdadf7349897f8b0b29c3b74 | 11:34 |
| *** baoli_ has quit IRC | 11:34 | |
| jroll | so you do additional things on top of what ironic does, yes? | 11:34 |
| dtantsur | yep | 11:35 |
| jroll | hm | 11:35 |
| jroll | dtantsur: this was our fix for ironic breakage https://github.com/openstack/ironic/commit/a53dae62afbeaf06e3b5bbbed133d9ad2aebcd76 | 11:36 |
| jroll | doesn't seem related, but could be? seems like you'd pull that in anyway | 11:37 |
| *** baoli has joined #openstack-ironic | 11:38 | |
| jroll | this doesn't make sense at all :( | 11:38 |
| jroll | dtantsur: is there a unique log I can look for to tell this happens? | 11:39 |
| jroll | or rather, has anyone filed an e-r check | 11:40 |
| jroll | and if not, what can I use to do so | 11:40 |
| jroll | (bug # might be helpful too :) | 11:40 |
| dtantsur | jroll, bug https://bugs.launchpad.net/ironic-inspector/+bug/1621791 | 11:42 |
| openstack | Launchpad bug 1621791 in Ironic Inspector "Inspector grenade fails in test_network_basic_ops" [Critical,In progress] - Assigned to Dmitry Tantsur (divius) | 11:42 |
| dtantsur | we more or less figured out that it only affects inspector | 11:43 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Update openstack baremetal driver commands https://review.openstack.org/362047 | 11:43 |
| *** baoli has quit IRC | 11:44 | |
| jroll | right, I want to search it in e-r | 11:44 |
| jroll | that's good enough, thanks | 11:44 |
| jroll | s/e-r/logstash | 11:44 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/python-ironicclient: Add command RAID logical disk properties https://review.openstack.org/362047 | 11:44 |
| TheJulia | Good morning folks | 11:45 |
| sambetts | Morning TheJulia | 11:45 |
| TheJulia | FYI I'm basically out for the next week and a few days. If you need me though, just ping me. :) | 11:46 |
| sambetts | dtantsur: Seems weird that it only affects inspector, are we doing something that no one else is doing/ | 11:46 |
| sambetts | ? | 11:46 |
| dtantsur | sambetts, networking configuration maybe, see above | 11:46 |
| dtantsur | morning TheJulia | 11:47 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Disable neutron tests in our grenade https://review.openstack.org/367841 | 11:49 |
| dtantsur | this is crazy of course ^^^ | 11:49 |
| *** wajdi has quit IRC | 11:50 | |
| *** wajdi has joined #openstack-ironic | 11:50 | |
| mat128 | morning ironicers | 11:50 |
| dtantsur | morning mat128 | 11:51 |
| *** wajdi_ has joined #openstack-ironic | 11:51 | |
| jroll | dtantsur: so you're right in that bug, the first hit was 2016-09-02T06:10:56.802-04:00 | 11:51 |
| *** mgoddard has quit IRC | 11:52 | |
| jroll | here's the e-r https://review.openstack.org/367969 | 11:54 |
| patchbot | patch 367969 - openstack-infra/elastic-recheck - Add bug #1621791 | 11:54 |
| openstack | bug 1621791 in Ironic Inspector "Inspector grenade fails in test_network_basic_ops" [Critical,In progress] https://launchpad.net/bugs/1621791 - Assigned to Dmitry Tantsur (divius) | 11:54 |
| *** lucasagomes is now known as lucas-hungry | 11:55 | |
| *** wajdi has quit IRC | 11:55 | |
| jroll | dtantsur: well, this merged just before it started happening: https://github.com/openstack/tempest/commit/bce51c53a0ab5c95d863955067e44bb309f5d1a1 | 11:57 |
| jroll | lovely | 11:57 |
| dtantsur | jroll, that's what aarefiev suspects. reverting it seems to help, but breaks other people... | 11:58 |
| jroll | who does it break? | 11:58 |
| dtantsur | jroll, not sure, see the bug linked to that patch | 11:58 |
| jroll | yeah looking | 11:58 |
| mgould | morning TheJulia sambetts jroll mat128 Romanenko_K | 11:59 |
| jroll | so it breaks something out of tree, out of openstack-ci, it seems | 11:59 |
| *** baoli has joined #openstack-ironic | 11:59 | |
| jroll | heya mgould | 11:59 |
| jroll | dtantsur: see also https://bugs.launchpad.net/tempest/+bug/1577632 | 12:00 |
| openstack | Launchpad bug 1577632 in tempest "SSH Client hangs on execute command" [Undecided,Fix committed] - Assigned to Franklin Naval (franknaval) | 12:00 |
| jroll | lots of "does this happen in CI?" | 12:00 |
| jroll | wish we'd just throw out tempest's ssh client | 12:00 |
| *** trown|outtypewww is now known as trown | 12:01 | |
| jroll | dtantsur: feels like we should revert that :/ | 12:02 |
| jroll | and find a different fix | 12:02 |
| jroll | maybe with http://docs.paramiko.org/en/2.0/api/channel.html#paramiko.channel.Channel.exit_status_ready | 12:02 |
| sambetts | ++ sounds like a better plan | 12:03 |
| dtantsur | jroll, +1, and we are having some interesting discussion on #openstack-neutron as well | 12:03 |
| jroll | oh, neat | 12:03 |
| * jroll looks | 12:03 | |
| mat128 | o/ mgould | 12:04 |
| aarefiev | jroll, dtantsur: reverting that fix will break folks, it's known bug http://docs.paramiko.org/en/2.0/api/channel.html#paramiko.channel.Channel.recv_exit_status | 12:13 |
| jroll | aarefiev: yes, it will break out-of-tree code in downstream CIs, I tend to think we should unblock inspector and those folks should find a different fix | 12:14 |
| aarefiev | here another fix https://review.openstack.org/#/c/367478/2 | 12:15 |
| patchbot | patch 367478 - tempest - WIP: Refactor ssh exec command | 12:15 |
| *** mhayden has left #openstack-ironic | 12:16 | |
| jroll | aarefiev: that doesn't seem like it works very well :P | 12:16 |
| dtantsur | lol | 12:16 |
| jroll | code looks okay though, so maybe just a trivial bug in there | 12:17 |
| aarefiev | but it looks like doesn't cause deadlocks | 12:17 |
| * jroll imagines all the mocking involved in fixing those unit tests and cries | 12:17 | |
| jroll | agree | 12:17 |
| mat128 | "will cause recv_exit_status to hang indefinitely if it is called prior to a sufficiently large read" | 12:17 |
| mat128 | just issue a big read() before | 12:17 |
| jroll | interesting http://logs.openstack.org/78/367478/2/check/gate-grenade-dsvm-multinode/98861dd/console.html#_2016-09-08_17_10_12_434777 | 12:18 |
| *** hoangcx has joined #openstack-ironic | 12:18 | |
| *** wajdi_ has quit IRC | 12:18 | |
| sambetts | jroll: rsync command? or am I missing something? | 12:19 |
| jroll | sambetts: all the junk +++++ stuff in the response | 12:19 |
| jroll | it is rsync though | 12:19 |
| jroll | oh never mind, this has nothing to do with the change | 12:19 |
| jroll | ignore me :) | 12:19 |
| sambetts | ah right :-P hehe, must be something to do with how rsync views the file system | 12:20 |
| * dtantsur is sad and tired | 12:20 | |
| * jroll too | 12:20 | |
| * dtantsur will get some food and will be a bit less sad and a bit more tired | 12:20 | |
| * sambetts joins the tired party | 12:20 | |
| mgould | there's a tired party? awesome | 12:21 |
| mgould | this is the kind of party where you all sit in the kitchen with a gin and tonic and have a natter, right? | 12:22 |
| mgould | ^^^ best kind of party | 12:22 |
| * jroll googles natter | 12:22 | |
| mat128 | lol | 12:22 |
| jroll | ah yes | 12:22 |
| mgould | heh, TIL that's a Briticism | 12:22 |
| jroll | I was thinking s/kitchen/couch/ and s/gin and tonic/bourbon/ | 12:22 |
| jroll | but yes +2 | 12:22 |
| mgould | the effect is much the same :-) | 12:22 |
| mgould | dtantsur, jroll, sambetts feel better soon! | 12:23 |
| mgould | apply bourbon and/or gin as appropriate | 12:23 |
| jroll | mgould: just a rough week, no worries :) | 12:23 |
| jroll | bourbon was applied last night | 12:23 |
| jroll | hence the tired :D | 12:23 |
| mgould | aaaah :-) | 12:23 |
| mgould | TGIF | 12:24 |
| jroll | ++ | 12:24 |
| mat128 | oh yes | 12:24 |
| * jroll steps away for a bit | 12:24 | |
| *** joprovost has joined #openstack-ironic | 12:24 | |
| *** hoangcx2 has joined #openstack-ironic | 12:29 | |
| *** hoangcx has quit IRC | 12:31 | |
| *** joprovost has quit IRC | 12:33 | |
| *** wznoinsk has quit IRC | 12:40 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Attach/Detach changes https://review.openstack.org/367948 | 12:41 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 12:41 |
| openstackgerrit | Mathieu Mitchell proposed openstack/ironic: Import install and configure into new install guide https://review.openstack.org/367988 | 12:43 |
| mat128 | one more core and we get to publish a (very basic) install guide! https://review.openstack.org/#/c/363294/ | 12:44 |
| patchbot | patch 363294 - ironic - Add a basic install guide | 12:44 |
| *** wznoinsk has joined #openstack-ironic | 12:47 | |
| *** sdake_ is now known as sdake | 12:47 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: DO NOT REVIEW https://review.openstack.org/296432 | 12:50 |
| *** lucas-hungry is now known as lucasagomes | 12:51 | |
| *** jaosorior has quit IRC | 12:52 | |
| *** leitan has joined #openstack-ironic | 12:52 | |
| *** jaosorior has joined #openstack-ironic | 12:52 | |
| *** Goneri has joined #openstack-ironic | 12:54 | |
| *** Ukesh has quit IRC | 12:55 | |
| *** vnogin has joined #openstack-ironic | 12:55 | |
| *** jcoufal has joined #openstack-ironic | 12:59 | |
| dtantsur | aarefiev, sambetts, ok, this is super ugly https://review.openstack.org/#/c/367841/ but it does disable neutron tests for us! | 12:59 |
| patchbot | patch 367841 - ironic-inspector - Disable neutron tests in our grenade | 12:59 |
| dtantsur | we can use it to unbreak the gate for now, and then figure out whatever we can do to fix it | 13:00 |
| dtantsur | jroll, ^^ | 13:00 |
| aarefiev | looking | 13:00 |
| dtantsur | it also make our tests 30 minutes shorter! I like, dunno if we're going to really revert it :D | 13:01 |
| aarefiev | :) | 13:02 |
| aarefiev | there are lot of api tests | 13:02 |
| aarefiev | neutron I mean | 13:02 |
| aarefiev | why do we need them? | 13:02 |
| dtantsur | well, they're tagged "smoke" and probably don't state neutron requirement properly... | 13:03 |
| dtantsur | anyway, they don't break us | 13:03 |
| aarefiev | lgtm | 13:03 |
| dtantsur | anyone wants to review it before I go ahead and approve? | 13:04 |
| *** sdake has quit IRC | 13:05 | |
| * mgould takes a quick look | 13:05 | |
| mgould | dtantsur: lgtm | 13:06 |
| mgould | is there any way we could break neutron? Alternatively, are the interactions tested elsewhere in our grenade job? | 13:07 |
| *** cdearborn has joined #openstack-ironic | 13:08 | |
| dtantsur | mgould, in theory we can break anything, but it's highly improbable. even with this test disabled, we still test nova's ability to launch instances, so I think we're all covered. | 13:09 |
| mgould | if the answers are "no, *" or "yes, yes" then we can drop those tests permanently speed up our gate and reduce our flakiness surface | 13:09 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient: Add negative tests for baremetal node commands https://review.openstack.org/337719 | 13:09 |
| mgould | sweet, burn it with fire | 13:09 |
| dtantsur | also, the way we setup network in devstack is pretty different from how real baremetal network works | 13:09 |
| mgould | awesome | 13:09 |
| mgould | death to pointless tests! | 13:09 |
| aarefiev | +1 | 13:09 |
| *** mjura has quit IRC | 13:13 | |
| *** vnogin has left #openstack-ironic | 13:13 | |
| sborkows | aarefiev dtantsur: I have got a message that in paramiko they have similar problem with grenade | 13:17 |
| sborkows | The same test is failing | 13:17 |
| sborkows | They suggest that this fix: https://review.openstack.org/#/c/358610/ is causing this problem | 13:18 |
| patchbot | patch 358610 - tempest - Fix exec_command to hang indefinitely (MERGED) | 13:18 |
| dtantsur | this is the suspect, but qa folks don't want to revert it, so we ended up disabling the test | 13:20 |
| dtantsur | sborkows, the gate should be back to norm after https://review.openstack.org/#/c/367841/ merges | 13:21 |
| patchbot | patch 367841 - ironic-inspector - Disable neutron tests in our grenade | 13:21 |
| sborkows | oops I just noticed, that aarefiev wrote that email :O | 13:21 |
| aarefiev | it was me :) | 13:23 |
| *** hoangcx has joined #openstack-ironic | 13:24 | |
| *** hoangcx2 has quit IRC | 13:25 | |
| jroll | dtantsur: yay for working gate | 13:26 |
| *** rbudden has joined #openstack-ironic | 13:26 | |
| *** joprovost has joined #openstack-ironic | 13:27 | |
| aarefiev | mtreinish: we are skipping neutron tests for unblock gates https://review.openstack.org/#/c/367841/2 | 13:28 |
| patchbot | patch 367841 - ironic-inspector - Disable neutron tests in our grenade | 13:28 |
| dtantsur | which does not cancel fixing tempest ssh code of course :) | 13:29 |
| aarefiev | for sure, just for notice :) | 13:29 |
| mtreinish | dtantsur, aarefiev: you should set that via your devstack/grenade plugin instead of in code | 13:30 |
| mtreinish | but that's fine | 13:30 |
| *** sdake has joined #openstack-ironic | 13:30 | |
| dtantsur | mtreinish, that does not quite work, these setting get overriden by lib/tempest which runs after settings | 13:30 |
| dtantsur | unless I misunderstood anything | 13:30 |
| dtantsur | anyway, this a quick hack | 13:30 |
| mtreinish | dtantsur: there is a devstack plugin phase for test setup which runs after the tempest config you could use | 13:31 |
| mtreinish | but if you found something that works that's fine, it just seems like that would be potentially fragile | 13:31 |
| dtantsur | mtreinish, if we end up leaving this long-term, I'll refactor it to something nicer, thanks for the hints! | 13:31 |
| aarefiev | I hope it would be not long-term | 13:32 |
| mtreinish | I don't think it'll take me much longer to sort out: https://review.openstack.org/#/c/367478/ | 13:32 |
| patchbot | patch 367478 - tempest - Refactor ssh exec command | 13:32 |
| dtantsur | we'll see :) thanks for working on it, anyway | 13:33 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on https://review.openstack.org/330003 | 13:33 |
| *** wajdi has joined #openstack-ironic | 13:41 | |
| *** dansmith is now known as superdan | 13:50 | |
| *** e0ne has quit IRC | 13:53 | |
| *** e0ne has joined #openstack-ironic | 13:53 | |
| *** alaski is now known as lascii | 13:54 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:54 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 13:55 | |
| *** hj-hpe has joined #openstack-ironic | 13:56 | |
| *** anush has joined #openstack-ironic | 14:00 | |
| *** fragatina has joined #openstack-ironic | 14:01 | |
| *** joprovost1 has joined #openstack-ironic | 14:03 | |
| *** joprovost has quit IRC | 14:04 | |
| *** joprovost1 is now known as joprovost | 14:04 | |
| *** fragatina has quit IRC | 14:06 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-inspector: Add translation marker to help of config opts https://review.openstack.org/365888 | 14:06 |
| *** cdearborn has quit IRC | 14:09 | |
| *** hoangcx has quit IRC | 14:13 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type https://review.openstack.org/363647 | 14:20 |
| *** syed_ has joined #openstack-ironic | 14:26 | |
| openstackgerrit | Merged openstack/ironic: Add a basic install guide https://review.openstack.org/363294 | 14:27 |
| *** hoangcx has joined #openstack-ironic | 14:28 | |
| *** leitan has quit IRC | 14:29 | |
| *** joprovost1 has joined #openstack-ironic | 14:31 | |
| mat128 | ^woohoo | 14:32 |
| mat128 | Thanks sambetts | 14:32 |
| *** joprovost has quit IRC | 14:32 | |
| *** joprovost1 is now known as joprovost | 14:32 | |
| openstackgerrit | Merged openstack/ironic-inspector: Disable neutron tests in our grenade https://review.openstack.org/367841 | 14:33 |
| dtantsur | recheck all the things ^^^ :) | 14:36 |
| aarefiev | merge party :) | 14:37 |
| openstackgerrit | Beth Elwell proposed openstack/ironic-ui: Add functionality for editing an existing node https://review.openstack.org/345362 | 14:37 |
| openstackgerrit | Beth Elwell proposed openstack/ironic-ui: Updated to ironic API v1.11 and added manageable state https://review.openstack.org/361098 | 14:37 |
| *** michauds has joined #openstack-ironic | 14:37 | |
| sborkows | weeee | 14:37 |
| *** rajinir has joined #openstack-ironic | 14:37 | |
| *** jheroux has joined #openstack-ironic | 14:39 | |
| *** hoangcx has quit IRC | 14:41 | |
| *** sabeen has joined #openstack-ironic | 14:41 | |
| *** mkam has joined #openstack-ironic | 14:41 | |
| *** mkam has quit IRC | 14:43 | |
| *** leitan has joined #openstack-ironic | 14:43 | |
| *** mkam has joined #openstack-ironic | 14:43 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic: IPMI command should depend on console type https://review.openstack.org/363647 | 14:47 |
| *** leitan has quit IRC | 14:52 | |
| *** mkam has quit IRC | 14:52 | |
| *** mkam has joined #openstack-ironic | 14:52 | |
| *** jcoufal has quit IRC | 14:53 | |
| *** jcoufal has joined #openstack-ironic | 14:56 | |
| *** sborkows has quit IRC | 14:57 | |
| *** fxpester has quit IRC | 14:58 | |
| *** jaosorior has quit IRC | 15:07 | |
| *** pcaruana has quit IRC | 15:08 | |
| *** leitan has joined #openstack-ironic | 15:08 | |
| *** rpioso has joined #openstack-ironic | 15:10 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add PCI devices plugin to inspector https://review.openstack.org/352295 | 15:22 |
| *** mbound has quit IRC | 15:22 | |
| *** jcoufal has quit IRC | 15:22 | |
| *** vishwanathj_zzz is now known as vishwanath | 15:27 | |
| *** vishwanath is now known as vishwanathj | 15:27 | |
| *** vnogin has joined #openstack-ironic | 15:27 | |
| *** vnogin has quit IRC | 15:27 | |
| *** amotoki has joined #openstack-ironic | 15:27 | |
| *** mkam has quit IRC | 15:35 | |
| *** jistr is now known as jistr|biab | 15:35 | |
| *** jcoufal has joined #openstack-ironic | 15:37 | |
| openstackgerrit | Sam Betts proposed openstack/ironic: DO NOT MERGE Testing Attach/Detach changes https://review.openstack.org/367948 | 15:39 |
| openstackgerrit | Sam Betts proposed openstack/ironic: Add Network Interface Attach and Detach APIs https://review.openstack.org/327046 | 15:39 |
| *** MattMan has quit IRC | 15:47 | |
| *** MattMan has joined #openstack-ironic | 15:47 | |
| *** mkam has joined #openstack-ironic | 15:47 | |
| *** hshiina has joined #openstack-ironic | 15:48 | |
| *** openstackgerrit has quit IRC | 15:49 | |
| *** tesseract- has quit IRC | 15:49 | |
| *** openstackgerrit has joined #openstack-ironic | 15:49 | |
| * sambetts -> home | 15:52 | |
| *** sambetts is now known as sambetts|afk | 15:52 | |
| -openstackstatus- NOTICE: New setuptools release appears to have a circular import which is breaking many jobs - check for ImportError: cannot import name monkey. | 15:53 | |
| *** ChanServ changes topic to "New setuptools release appears to have a circular import which is breaking many jobs - check for ImportError: cannot import name monkey." | 15:53 | |
| dtantsur | well, our CI has been working for a couple of hours.. | 15:53 |
| *** rbrud has joined #openstack-ironic | 15:54 | |
| lucasagomes | :-/ | 15:55 |
| *** e0ne has quit IRC | 15:55 | |
| lucasagomes | dtantsur, sounds like friday :D | 15:55 |
| *** jcoufal has quit IRC | 15:56 | |
| dtantsur | this probably means it's time to enjoy the weekend :) | 15:56 |
| dtantsur | have a nice weekend, ironicers | 15:56 |
| *** jcoufal has joined #openstack-ironic | 15:57 | |
| jlvillal | vdrok, If it is okay for you to change the presentation date/time can you also reply? Or if not, please reply too :) | 15:57 |
| lucasagomes | dtantsur, enjoy it! | 15:57 |
| * jlvillal tries to think of some joke about monkeys | 15:58 | |
| jroll | see ya dtantsur :) | 15:58 |
| dtantsur | o/ | 15:58 |
| jroll | chaos monkey at work, obviously | 15:58 |
| jlvillal | dtantsur, Good night | 15:58 |
| jlvillal | heh | 15:58 |
| dtantsur | also, #openstack-infra has a few good jokes about monkeys | 15:58 |
| dtantsur | <zigo> So, to sum-up: the monkey broke setuptools... :P | 15:59 |
| dtantsur | ok, ok, leaving for real now :) | 15:59 |
| *** dtantsur is now known as dtantsur|afk | 15:59 | |
| *** amotoki_ has joined #openstack-ironic | 16:02 | |
| *** amotoki_ has quit IRC | 16:03 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/368104 | 16:04 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/368105 | 16:04 |
| *** amotoki has quit IRC | 16:05 | |
| *** Romanenko_K has quit IRC | 16:06 | |
| *** mkoderer has quit IRC | 16:21 | |
| *** jcoufal has quit IRC | 16:22 | |
| TheJulia | betherly: ppiela: sorry for missing the call today. I'm trying to take the day off, I see you two had questions, I'll do my best to answer :) | 16:22 |
| *** mbound has joined #openstack-ironic | 16:22 | |
| ppiela | TheJulia: no problem. We can talk next week. | 16:23 |
| *** jcoufal has joined #openstack-ironic | 16:24 | |
| *** jistr|biab is now known as jistr | 16:25 | |
| *** rbrud has quit IRC | 16:28 | |
| *** mbound has quit IRC | 16:28 | |
| *** sdake has quit IRC | 16:28 | |
| *** hshiina has quit IRC | 16:31 | |
| *** fragatina has joined #openstack-ironic | 16:32 | |
| TheJulia | ppiela: I will join if I have a connection next week :) I put some comments on the etherpad. I may/likely be missing discussion context, but hopefully it makes things a little more clear. | 16:32 |
| TheJulia | ^^ betherly | 16:33 |
| *** snecklifter has quit IRC | 16:33 | |
| *** athomas has quit IRC | 16:34 | |
| *** aNupoisc has joined #openstack-ironic | 16:35 | |
| *** leitan has quit IRC | 16:37 | |
| lucasagomes | I'm calling it a day as well | 16:43 |
| lucasagomes | have a great weekend all! Talk to you all on monday | 16:43 |
| *** lucasagomes is now known as lucas-afk | 16:44 | |
| *** mkoderer has joined #openstack-ironic | 16:44 | |
| NobodyCam | good morning ironicers | 16:45 |
| NobodyCam | and TGIF | 16:45 |
| NobodyCam | night lucas-afk | 16:46 |
| TheJulia | good morning NobodyCam | 16:47 |
| NobodyCam | :) | 16:48 |
| *** ChubYann has joined #openstack-ironic | 16:50 | |
| ppiela | TheJulia: Thanks for lookign at our questions | 16:50 |
| *** jpich has quit IRC | 16:52 | |
| *** derekh has quit IRC | 16:54 | |
| jlvillal | Hey NobodyCam. I think your Friday nick should be IthinkICam :) | 16:54 |
| jlvillal | Or something like that | 16:54 |
| NobodyCam | lol | 16:54 |
| * jlvillal has not figured out anything clever with his own nick | 16:55 | |
| *** trown is now known as trown|lunch | 16:56 | |
| *** ChanServ changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 16:56 | |
| -openstackstatus- NOTICE: setuptools 27.1.2 addresses the circular import | 16:56 | |
| jlvillal | Woo hoo! | 16:56 |
| mat128 | it's recheck time? | 16:57 |
| jlvillal | recheck 'em if you got 'em ? | 16:57 |
| mat128 | yeah | 16:57 |
| NobodyCam | jlvillal: I use to use Blart Versenwald | 16:58 |
| NobodyCam | as my friday nick | 16:58 |
| jlvillal | NobodyCam, Well I would have had no idea who you were with that one :) | 16:58 |
| *** ohamada has quit IRC | 16:58 | |
| mat128 | jlvillal: that was quick: https://github.com/pypa/setuptools/commit/2a7a7179ca18b1c97e435726e62c3ab0227fb20e | 16:59 |
| NobodyCam | jlvillal: https://en.wikipedia.org/wiki/List_of_minor_The_Hitchhiker%27s_Guide_to_the_Galaxy_characters | 16:59 |
| jlvillal | NobodyCam, I don't even know the major characters! | 16:59 |
| NobodyCam | lol +++ | 17:00 |
| *** anush has quit IRC | 17:00 | |
| jlvillal | mat128, Where is the change? | 17:00 |
| mat128 | https://github.com/pypa/setuptools/pull/781 | 17:00 |
| jlvillal | mat128, https://github.com/pypa/setuptools/commit/b7b9cb23f217095e79c618c0e3196712d2d9a285 | 17:01 |
| jlvillal | I think that one. | 17:01 |
| mat128 | ooh yeah | 17:01 |
| mat128 | #781 is what broke us, I think | 17:01 |
| jlvillal | mat128, Ah | 17:02 |
| mat128 | or what led to that "from . import msvc" | 17:02 |
| jlvillal | mat128, Or maybe the first attempt to fix it | 17:02 |
| jlvillal | mat128, That's a lot of commits in a 3 hour period | 17:03 |
| mat128 | :) | 17:03 |
| mat128 | oh man | 17:04 |
| mat128 | you're right | 17:04 |
| mat128 | https://github.com/pypa/setuptools/pulse/daily | 17:04 |
| jlvillal | mat128, I guess I'm thinking for such a critical infrastructure type project. I'm surprised so much churn. But hopefully they add some gating tests to catch that type of issue in the future. | 17:05 |
| * mat128 glances at the test folder | 17:05 | |
| * mat128 cries :( | 17:05 | |
| mat128 | https://github.com/pypa/setuptools/tree/master/tests | 17:05 |
| mat128 | I guess users are their tests =) | 17:05 |
| jlvillal | Yikes! | 17:05 |
| mat128 | oh | 17:06 |
| mat128 | https://github.com/pypa/setuptools/tree/master/setuptools/tests | 17:06 |
| mat128 | more tests there | 17:06 |
| patchbot | Error: I haven't seen tests. | 17:06 |
| jlvillal | There entire unit test suite is a 98 line file | 17:06 |
| jlvillal | s/There/Their/ | 17:06 |
| mat128 | nah there's one under setuptools/ too | 17:06 |
| jlvillal | That's better | 17:06 |
| jlvillal | I get used to living in this world where unit tests are law (or close to it) | 17:07 |
| mat128 | :) | 17:07 |
| mat128 | https://8thlight.com/blog/uncle-bob/2012/01/11/Flipping-the-Bit.html | 17:07 |
| mat128 | now publishing draft guide: http://docs-draft.openstack.org/29/366329/2/check/gate-ironic-install-guide/24a20bd//install-guide/build/html/ | 17:08 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack https://review.openstack.org/366518 | 17:08 |
| mat128 | :D | 17:08 |
| jlvillal | mat128, I think I agree with that article with my quick scan. Unlike the one on boolean values to functions ;) | 17:08 |
| mat128 | "What is THE BIT? THE BIT is the boolean variable within your subconscious that represents your belief that unit tests take time. I want you to flip that bit so that your belief changes. I want you to believe that Test Driven Development saves time in every case and every situation without exception amen." | 17:09 |
| mat128 | thats the tl;dr | 17:09 |
| jlvillal | mat128, I'm pretty much on board with that | 17:09 |
| mat128 | :) | 17:11 |
| mgould | that's the right side to err on, definitely | 17:17 |
| * jlvillal wonders if mgould is replying about TDD | 17:17 | |
| mgould | jlvillal: yep | 17:17 |
| mgould | what was the bools-to-functions article? | 17:17 |
| jlvillal | Hey mgould. I hope things are well. I haven't chatted with you much recently. | 17:17 |
| NobodyCam | morning mgould :) | 17:18 |
| mgould | jlvillal: I just got back from a three-week trip to Kyrgyzstan, that's why :-) | 17:18 |
| mgould | NobodyCam: morning! | 17:18 |
| jlvillal | mgould, bools as function arguments | 17:18 |
| mat128 | mgould: same guy, but this is "flipping the bit" | 17:18 |
| persia | Depends on the quality/coverage of functional requirement testing. Units tests & TDD is *way* better than not, but I think maybe not as useful as excutable requirements validation & BDD, assuming one can cause the requirement drivers to participate. | 17:18 |
| NobodyCam | :) | 17:18 |
| jlvillal | mgould, Welcome back! | 17:18 |
| mgould | jlvillal: thanks! | 17:18 |
| mgould | https://twitter.com/andymilesexped << a few photos here, but we still need to go through most of them | 17:19 |
| *** e0ne has joined #openstack-ironic | 17:20 | |
| mgould | persia: I've yet to really grok BDD | 17:20 |
| mgould | requirements capture is a Hard Problem, of course | 17:20 |
| persia | mgould: Yep. And there are not a lot of good tooling, meaning TDD is the best most of us can do most of the time today. | 17:20 |
| jlvillal | mgould, It was buried in here: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-09-02.log.html#t2016-09-02T16:07:48 | 17:21 |
| mgould | jlvillal: boolean function arguments are often a pain to understand, but I'm not convinced that duplicating all your functions is the cure :-) | 17:21 |
| jlvillal | mgould, I'm with you on that. I didn't agree with the premise | 17:21 |
| mat128 | JayF: Getting there! http://docs-draft.openstack.org/88/367988/1/check/gate-ironic-install-guide/2a4d9d0/install-guide/build/html/install-ubuntu.html | 17:21 |
| jlvillal | mgould, There is the article: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2016-09-02.log.html#t2016-09-02T17:31:58 | 17:22 |
| jlvillal | mgould, But I think you already found it | 17:22 |
| mgould | yep, reading now | 17:22 |
| * jlvillal wonders if mgould rode a horse while the back of his jacket was on fire | 17:23 | |
| * mgould used to work with a guy who'd add obscurely-named boolean parameters to functions, and then pass them unchanged through six layers of function calls without explanation until they finally got to an if-statement somewhere that changed its behaviour *completely* based on the parameter | 17:24 | |
| mgould | jlvillal: sadly not, but I did climb some mountains :-) | 17:24 |
| mat128 | mgould: those always bite you | 17:24 |
| mat128 | I bet they also bit the guy who wrote htem | 17:24 |
| jlvillal | :) | 17:24 |
| mgould | yep | 17:24 |
| mgould | but they bit him less than they bit the rest of us, so he still looked productive :-/ | 17:25 |
| mat128 | haha | 17:25 |
| mat128 | he remembered them | 17:25 |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic: Change $LOGFILE path in configuration devstack https://review.openstack.org/366518 | 17:25 |
| mgould | jlvillal: https://twitter.com/AndyMilesExped/status/771006773847261188 | 17:26 |
| mgould | mat128: yep! Reverse-engineering his code was not fun :-( | 17:26 |
| jlvillal | mgould, Cool! | 17:26 |
| mat128 | mgould: click-click-click-click-click-AHHH! | 17:26 |
| mat128 | We have an inhouse system for old server delivery that is somewhat like that | 17:27 |
| mgould | https://twitter.com/AndyMilesExped/status/770942239623241728 is a nice shot too | 17:27 |
| mat128 | mgould: oh wow ^ | 17:27 |
| mat128 | Welcome back | 17:27 |
| mgould | see also https://twitter.com/AndyMilesExped/status/770942995340390400 :-) | 17:28 |
| *** bnemec is now known as beekneemech | 17:28 | |
| jlvillal | mgould, Is that the Red Beard guy from CoreOS with you? :) | 17:28 |
| mgould | mat128: thanks :-) | 17:28 |
| mgould | jlvillal: not to the best of my knowledge :-) | 17:28 |
| mgould | that's my friend Andy, he's a mechanical engineer | 17:29 |
| jlvillal | mgould, http://photos2.meetupstatic.com/photos/member/8/f/1/0/highres_10776624.jpeg | 17:29 |
| jlvillal | I saw him present in Vancouver | 17:29 |
| mgould | I definitely see the resemblance | 17:29 |
| * mgould wonders if they've ever been seen in the same room at the same time | 17:29 | |
| jlvillal | mgould, So will you take the short hop to Barcelona for the Summit? | 17:29 |
| *** jcoufal has quit IRC | 17:30 | |
| jlvillal | Actually an almost 3 hour plane ride is longer than I thought to get to Barcelona. | 17:31 |
| mgould | jlvillal: yes! Got my flights booked and everything | 17:31 |
| mgould | will you be there too? | 17:31 |
| jlvillal | mgould, Luckily for me, I will be. | 17:32 |
| mgould | awesome! | 17:32 |
| jlvillal | Got my AirBnb booked. Those suggested hotels seemed very expensive. | 17:32 |
| mgould | greatly looking forward to meeting you (and everyone else!) in person | 17:32 |
| *** e0ne has quit IRC | 17:32 | |
| mgould | yeah, I'm in an AirBnB too | 17:32 |
| *** devananda|dinner is now known as devananda | 17:32 | |
| jlvillal | I vaguely recall prices about 200 euros per night | 17:32 |
| mgould | saved a lot of money, and I'm only a few minutes further away from the hotel | 17:33 |
| mgould | s/hotel/conference center | 17:33 |
| jlvillal | I'm about 18 minutes away using the tram. | 17:33 |
| jlvillal | mgould, I was right. First hotel I checked for just four nights 844 euros. Ouch | 17:34 |
| mgould | yikes! | 17:35 |
| mgould | jlvillal: awesome | 17:36 |
| mgould | I think I'm just about within walking distance - party at mine :-) | 17:36 |
| jlvillal | :) | 17:37 |
| *** e0ne has joined #openstack-ironic | 17:38 | |
| *** fragatina has quit IRC | 17:39 | |
| *** fragatina has joined #openstack-ironic | 17:43 | |
| * mgould checks the listing | 17:44 | |
| mgould | "House rules: no parties or events" | 17:44 |
| mgould | bah | 17:44 |
| jlvillal | mgould, Same at mine. | 17:45 |
| mgould | "this isn't a party, this is merely a gathering of 30 of my close friends!" | 17:45 |
| mgould | "with booze" | 17:45 |
| mgould | anyway, we shall bring a bottle or two of good Scotch to share with a sub-party number of folks | 17:47 |
| *** fragatina has quit IRC | 17:48 | |
| *** ChubYann has quit IRC | 17:48 | |
| mgould | <<< not a drunk, honest | 17:48 |
| mgould | yeah, we're 16 mins walk from the conference centre along Avenue Diagonal | 17:49 |
| mgould | or Diagon Alley, as I shall henceforth be calling it | 17:49 |
| jroll | +1 | 17:50 |
| jroll | I'm in one of the conference hotels if we want a less interesting, less rule-breaking place to hang out | 17:50 |
| jroll | or surely there are many places that sell drinks and have chairs | 17:50 |
| mgould | this seems like a key bit of conference infrastructure, certainly | 17:51 |
| *** trown|lunch is now known as trown | 17:55 | |
| jroll | :D | 17:55 |
| jroll | totally is | 17:55 |
| jroll | how many times have we said "this discussion requires a bar setting" here? :) | 17:55 |
| TheJulia | uhh.... do we really want to answer that question | 17:58 |
| TheJulia | >.> | 17:58 |
| mgould | :-) | 17:58 |
| *** ChubYann has joined #openstack-ironic | 18:00 | |
| *** rbrud has joined #openstack-ironic | 18:10 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode job https://review.openstack.org/368173 | 18:13 |
| *** fragatina has joined #openstack-ironic | 18:34 | |
| *** jcoufal has joined #openstack-ironic | 18:34 | |
| *** fragatina has quit IRC | 18:34 | |
| *** fragatina has joined #openstack-ironic | 18:34 | |
| *** joprovost1 has joined #openstack-ironic | 18:36 | |
| *** joprovost has quit IRC | 18:36 | |
| *** joprovost1 is now known as joprovost | 18:36 | |
| *** mgould is now known as mgould|afk | 18:44 | |
| *** wajdi_ has joined #openstack-ironic | 18:44 | |
| *** Sukhdev has joined #openstack-ironic | 18:48 | |
| *** wajdi has quit IRC | 18:48 | |
| *** joprovost has quit IRC | 18:56 | |
| *** joprovost has joined #openstack-ironic | 18:58 | |
| *** leitan has joined #openstack-ironic | 19:00 | |
| *** hoangcx has joined #openstack-ironic | 19:07 | |
| openstackgerrit | Merged openstack/python-ironicclient: 'ironic create' handles file args https://review.openstack.org/367715 | 19:10 |
| *** e0ne has quit IRC | 19:11 | |
| *** hoangcx has quit IRC | 19:14 | |
| *** mkoderer has quit IRC | 19:23 | |
| *** hoangcx has joined #openstack-ironic | 19:23 | |
| *** jheroux has quit IRC | 19:26 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Testing multinode job https://review.openstack.org/368173 | 19:36 |
| *** hoangcx has quit IRC | 19:37 | |
| *** sdake has joined #openstack-ironic | 19:38 | |
| *** sdake has quit IRC | 19:42 | |
| mgagne | jroll: I see that all comment output are logged in DEBUG: https://github.com/openstack/ironic/blob/master/ironic/common/utils.py#L75 | 19:43 |
| *** mkam has quit IRC | 19:43 | |
| mgagne | jroll: for some command, this means a single line and a line return at the end. This means 2 lines in syslog. (therefore multiline logs) | 19:43 |
| mgagne | jroll: would it make sense to trim the line return? Or would it cause confusion when debugging? | 19:44 |
| *** sdake has joined #openstack-ironic | 19:44 | |
| *** mkam has joined #openstack-ironic | 19:48 | |
| *** ccamacho has quit IRC | 19:49 | |
| *** leitan has quit IRC | 19:56 | |
| *** mkam has quit IRC | 19:56 | |
| *** mkam has joined #openstack-ironic | 19:57 | |
| *** aNupoisc has quit IRC | 19:59 | |
| *** michauds has quit IRC | 20:08 | |
| *** aNupoisc has joined #openstack-ironic | 20:10 | |
| *** rpioso has quit IRC | 20:14 | |
| *** mbound has joined #openstack-ironic | 20:20 | |
| *** _vsaienko has joined #openstack-ironic | 20:27 | |
| *** superdan is now known as dansmith | 20:27 | |
| *** rama_y has joined #openstack-ironic | 20:32 | |
| _vsaienko | jlvillal: looks like alternative variant to use ip address from provisioning network as tftp and http works https://review.openstack.org/#/c/368173/ | 20:33 |
| patchbot | patch 368173 - ironic - Testing multinode job | 20:33 |
| *** baoli has quit IRC | 20:33 | |
| jlvillal | Awesome vsaienko! who appears to have messaged and dashed | 20:33 |
| *** baoli has joined #openstack-ironic | 20:33 | |
| *** rama_y has quit IRC | 20:36 | |
| *** mkoderer has joined #openstack-ironic | 20:38 | |
| *** jcoufal has quit IRC | 20:43 | |
| *** e0ne has joined #openstack-ironic | 20:54 | |
| *** openstackstatus has quit IRC | 20:57 | |
| *** openstackstatus has joined #openstack-ironic | 20:59 | |
| *** ChanServ sets mode: +v openstackstatus | 20:59 | |
| *** baoli has quit IRC | 21:02 | |
| *** baoli has joined #openstack-ironic | 21:02 | |
| *** lascii is now known as alaski | 21:04 | |
| *** joprovost has quit IRC | 21:04 | |
| *** rbudden has quit IRC | 21:05 | |
| *** baoli has quit IRC | 21:07 | |
| *** wajdi_ has quit IRC | 21:09 | |
| *** e0ne has quit IRC | 21:09 | |
| *** wajdi has joined #openstack-ironic | 21:09 | |
| *** mbound has quit IRC | 21:19 | |
| *** joprovost has joined #openstack-ironic | 21:19 | |
| *** trown is now known as trown|outtypewww | 21:21 | |
| *** joprovost1 has joined #openstack-ironic | 21:22 | |
| *** joprovost has quit IRC | 21:24 | |
| *** joprovost1 has quit IRC | 21:26 | |
| *** wajdi_ has joined #openstack-ironic | 21:29 | |
| *** wajdi has quit IRC | 21:33 | |
| *** wajdi_ has quit IRC | 21:34 | |
| *** rbrud has quit IRC | 21:47 | |
| *** mkam has quit IRC | 22:03 | |
| *** yee379 has joined #openstack-ironic | 22:14 | |
| *** [1]cdearborn has quit IRC | 22:15 | |
| *** sdake has quit IRC | 22:40 | |
| *** ppiela has quit IRC | 22:49 | |
| *** causten_ has quit IRC | 23:16 | |
| *** sdake has joined #openstack-ironic | 23:22 | |
| *** _vsaienko has quit IRC | 23:30 | |
| *** vishwanathj has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
