| *** matsuhashi has joined #openstack-ironic | 00:01 | |
| *** shakamunyi has joined #openstack-ironic | 00:31 | |
| *** shakamunyi has quit IRC | 00:37 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Add a @sync_command() decorator https://review.openstack.org/87144 | 00:39 |
|---|---|---|
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Decorated commands no longer take command_name https://review.openstack.org/87147 | 00:50 |
| *** derekh has quit IRC | 01:15 | |
| *** nosnos has joined #openstack-ironic | 01:41 | |
| *** Haomeng has joined #openstack-ironic | 02:27 | |
| *** coolsvap|afk is now known as coolsvap | 02:38 | |
| *** vkozhukalov has joined #openstack-ironic | 03:25 | |
| *** matsuhashi has quit IRC | 03:31 | |
| *** vkozhukalov has quit IRC | 03:41 | |
| *** nosnos has quit IRC | 03:49 | |
| *** eghobo has joined #openstack-ironic | 03:49 | |
| *** vkozhukalov has joined #openstack-ironic | 04:11 | |
| *** eghobo has quit IRC | 04:37 | |
| *** matsuhashi has joined #openstack-ironic | 04:38 | |
| *** eghobo has joined #openstack-ironic | 04:38 | |
| *** eguz has joined #openstack-ironic | 04:40 | |
| *** eghobo has quit IRC | 04:44 | |
| *** nosnos has joined #openstack-ironic | 04:48 | |
| *** rameshg87 has joined #openstack-ironic | 05:11 | |
| *** rameshg87 has left #openstack-ironic | 05:35 | |
| *** nosnos_ has joined #openstack-ironic | 05:36 | |
| *** nosnos has quit IRC | 05:36 | |
| *** radsy has quit IRC | 05:43 | |
| *** rameshg87 has joined #openstack-ironic | 05:44 | |
| *** zul has quit IRC | 05:46 | |
| *** rameshg87 has left #openstack-ironic | 05:49 | |
| *** matsuhas_ has joined #openstack-ironic | 06:00 | |
| *** zul has joined #openstack-ironic | 06:00 | |
| *** matsuhashi has quit IRC | 06:03 | |
| openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/83956 | 06:07 |
| *** Mikhail_D_ltp has joined #openstack-ironic | 06:14 | |
| *** max_lobur has joined #openstack-ironic | 06:26 | |
| *** nosnos has joined #openstack-ironic | 06:32 | |
| *** nosnos_ has quit IRC | 06:32 | |
| *** matsuhas_ has quit IRC | 06:37 | |
| *** matsuhashi has joined #openstack-ironic | 06:37 | |
| *** matsuhashi has quit IRC | 06:37 | |
| *** matsuhashi has joined #openstack-ironic | 06:38 | |
| *** Mikhail_D_ltp1 has joined #openstack-ironic | 06:45 | |
| *** vkozhukalov has quit IRC | 06:47 | |
| *** Mikhail_D_ltp has quit IRC | 06:49 | |
| *** romcheg has joined #openstack-ironic | 06:54 | |
| *** sabah has joined #openstack-ironic | 07:01 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Add Ironic User Guide https://review.openstack.org/83343 | 07:03 |
| dtantsur | Morning Ironic | 07:04 |
| *** Mikhail_D_ltp1 has quit IRC | 07:07 | |
| Haomeng | dtantsur: morning:) | 07:08 |
| *** engeepanoff has joined #openstack-ironic | 07:29 | |
| *** engeepanoff has quit IRC | 07:30 | |
| *** ndipanov has joined #openstack-ironic | 07:32 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Add Ironic User Guide https://review.openstack.org/83343 | 07:36 |
| *** romcheg has joined #openstack-ironic | 07:36 | |
| *** ifarkas has joined #openstack-ironic | 07:44 | |
| *** dshulyak has joined #openstack-ironic | 07:46 | |
| *** max_lobur has quit IRC | 07:48 | |
| *** eguz has quit IRC | 07:50 | |
| *** jistr has joined #openstack-ironic | 08:02 | |
| Mikhail_D_wk | Good morning folks! :) | 08:04 |
| Haomeng | Mikhail_D_wk: morning:) | 08:05 |
| *** martyntaylor has joined #openstack-ironic | 08:14 | |
| *** mrda is now known as mrda_away | 08:14 | |
| *** Mikhail_D_wk has quit IRC | 08:16 | |
| *** athomas has joined #openstack-ironic | 08:17 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 08:17 | |
| *** vkozhukalov has joined #openstack-ironic | 08:27 | |
| *** lucasagomes has joined #openstack-ironic | 08:28 | |
| *** ifarkas has quit IRC | 08:35 | |
| *** ifarkas has joined #openstack-ironic | 08:39 | |
| *** yuriyz has joined #openstack-ironic | 08:45 | |
| *** max_lobur has joined #openstack-ironic | 08:47 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Documentation for contributors https://review.openstack.org/85096 | 08:53 |
| openstackgerrit | Andrey Kurilin proposed a change to openstack/python-ironicclient: Sync latest code and reuse exceptions from oslo https://review.openstack.org/71500 | 08:54 |
| *** vkozhukalov1 has joined #openstack-ironic | 08:59 | |
| *** vkozhukalov has quit IRC | 09:03 | |
| *** Mikhail_D_wk has quit IRC | 09:10 | |
| *** andreykurilin has joined #openstack-ironic | 09:16 | |
| *** sabah has quit IRC | 09:17 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 09:21 | |
| *** derekh has joined #openstack-ironic | 09:21 | |
| *** romcheg has quit IRC | 09:44 | |
| *** JoshNang has quit IRC | 09:46 | |
| *** sabah has joined #openstack-ironic | 09:46 | |
| *** soren has quit IRC | 09:46 | |
| *** soren has joined #openstack-ironic | 09:47 | |
| *** matty_dubs|gone has quit IRC | 09:47 | |
| *** matty_dubs|gone has joined #openstack-ironic | 09:47 | |
| *** JoshNang has joined #openstack-ironic | 09:48 | |
| *** Mikhail_D_wk has quit IRC | 10:07 | |
| *** Mikhail_D_wk has joined #openstack-ironic | 10:07 | |
| openstackgerrit | A change was merged to openstack/ironic: Fix missed stopping of conductor service https://review.openstack.org/87075 | 10:12 |
| *** romcheg has joined #openstack-ironic | 10:14 | |
| *** matsuhashi has quit IRC | 10:40 | |
| *** matsuhashi has joined #openstack-ironic | 10:41 | |
| openstackgerrit | Sandhya Balakrishnan proposed a change to openstack/ironic: Add Ironic User Guide https://review.openstack.org/83343 | 10:43 |
| *** matsuhashi has quit IRC | 10:50 | |
| *** matsuhas_ has joined #openstack-ironic | 10:52 | |
| *** coolsvap is now known as coolsvap|afk | 10:52 | |
| *** matsuhas_ has quit IRC | 11:00 | |
| *** matsuhas_ has joined #openstack-ironic | 11:04 | |
| *** matsuhas_ has quit IRC | 11:15 | |
| *** matsuhashi has joined #openstack-ironic | 11:16 | |
| *** Shrews has joined #openstack-ironic | 11:27 | |
| *** matsuhashi has quit IRC | 11:46 | |
| *** lucasagomes is now known as lucas-hungry | 11:51 | |
| *** matsuhashi has joined #openstack-ironic | 11:51 | |
| *** sabah has quit IRC | 11:52 | |
| *** sabah has joined #openstack-ironic | 12:00 | |
| *** nosnos has quit IRC | 12:01 | |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add self to validator method https://review.openstack.org/87254 | 12:05 |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add FlowExtension https://review.openstack.org/85412 | 12:05 |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Resolve circular import, introduce ExecuteCommandMixin https://review.openstack.org/86341 | 12:05 |
| *** sabah has quit IRC | 12:10 | |
| *** matsuhashi has quit IRC | 12:29 | |
| *** matsuhashi has joined #openstack-ironic | 12:30 | |
| *** matsuhashi has quit IRC | 12:34 | |
| *** vkozhukalov has joined #openstack-ironic | 12:35 | |
| *** rloo has joined #openstack-ironic | 12:36 | |
| *** jdob has joined #openstack-ironic | 12:38 | |
| *** vkozhukalov1 has quit IRC | 12:38 | |
| *** matsuhashi has joined #openstack-ironic | 12:42 | |
| *** romcheg has quit IRC | 12:43 | |
| *** mdickson has joined #openstack-ironic | 12:44 | |
| *** rloo has quit IRC | 12:44 | |
| *** rloo has joined #openstack-ironic | 12:45 | |
| *** zul has quit IRC | 12:48 | |
| *** zul has joined #openstack-ironic | 12:51 | |
| *** lucas-hungry is now known as lucasagomes | 12:56 | |
| *** romcheg has joined #openstack-ironic | 12:59 | |
| NobodyCam | good morning Ironic | 13:04 |
| agordeev | good morning | 13:05 |
| agordeev | NobodyCam: morning! | 13:05 |
| yuriyz | morning agordeev, NobodyCam | 13:06 |
| lucasagomes | morning NobodyCam agordeev yuriyz | 13:06 |
| rloo | Morning NobodyCam, afternoon yuriyz, agordeev, lucasagomes | 13:06 |
| lucasagomes | morning rloo :) | 13:06 |
| NobodyCam | morning agordeev yuriyz and lucasagomes :) | 13:06 |
| NobodyCam | oh and rloo | 13:06 |
| agordeev | yuriyz, lucasagomes, rloo morning! | 13:06 |
| *** matsuhashi has quit IRC | 13:06 | |
| *** blamar has joined #openstack-ironic | 13:07 | |
| NobodyCam | texas Has strange weather | 13:07 |
| NobodyCam | and it seems I need to go walkies in it ... brb | 13:07 |
| lucasagomes | :) | 13:09 |
| lucasagomes | NobodyCam, heh I ireland has a strange weather... I thought texas was like hmmm warm | 13:10 |
| lucasagomes | I think* | 13:10 |
| *** matsuhas_ has joined #openstack-ironic | 13:10 | |
| NobodyCam | lol.. it strange rain. | 13:11 |
| NobodyCam | s/i/it's/ | 13:11 |
| lucasagomes | heh I c | 13:11 |
| *** florentflament has joined #openstack-ironic | 13:12 | |
| *** ndipanov has quit IRC | 13:14 | |
| NobodyCam | oh happy happy joy joy .. seems I get to battle tmobile again... | 13:17 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Add discrete states for event log status sensors https://review.openstack.org/87023 | 13:17 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Gracefully flag unrecognized discrete sensor states https://review.openstack.org/87027 | 13:19 |
| *** ndipanov has joined #openstack-ironic | 13:20 | |
| *** jbjohnso_ has joined #openstack-ironic | 13:21 | |
| *** lazy_prince has joined #openstack-ironic | 13:27 | |
| *** jgrimm has quit IRC | 13:30 | |
| *** matty_dubs|gone is now known as matty_dubs | 13:34 | |
| agordeev | russell_h: morning. are you around? | 13:40 |
| *** matsuhas_ has quit IRC | 13:42 | |
| *** yuriyz has quit IRC | 13:43 | |
| *** linggao has joined #openstack-ironic | 13:52 | |
| *** matsuhashi has joined #openstack-ironic | 13:52 | |
| *** matsuhashi has quit IRC | 13:54 | |
| NobodyCam | brb | 13:55 |
| *** yuriyz has joined #openstack-ironic | 13:55 | |
| *** jgrimm has joined #openstack-ironic | 14:06 | |
| dtantsur | Morning again :) Guys, have you ever encountered something like this: http://paste.openstack.org/show/75674/ ? It happens on invoking `ironic node-list`, does NOT happen for e.g. `nova list`. Instack, Fedora 20, Ironic RC1 | 14:07 |
| dtantsur | I tried to disable SSL for keystone, but nothing changed. Maybe I was doing it wrong... | 14:07 |
| NobodyCam | I have not seen that one | 14:08 |
| *** rwsu has joined #openstack-ironic | 14:09 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add DriverFieldsFactory https://review.openstack.org/86971 | 14:09 |
| lucasagomes | dtantsur, hmm this one is new for me too | 14:10 |
| dtantsur | and it does not happen if I install Ironic from Git manually. Quite likely to be something configuration-specific, but even have no clues how to debug :( | 14:11 |
| NobodyCam | just taking a quick look it seems like a keystone config issure | 14:12 |
| dtantsur | lucasagomes, NobodyCam: at least maybe you know, what is the proper way to disable everything SSL-related at all? | 14:12 |
| dtantsur | hopefully afterwards it would be easier to debug | 14:12 |
| NobodyCam | dtantsur: I would start by adding bebug=True to the ironic conf file | 14:14 |
| dtantsur | already done | 14:14 |
| *** dkehnx has quit IRC | 14:14 | |
| dtantsur | this is keystone.conf: http://paste.openstack.org/show/75676/ I guess, nothing suspicious | 14:14 |
| NobodyCam | are you using devtest or devstack? | 14:14 |
| lucasagomes | dtantsur, try to set auth_strategy=noauth in the ironic.conf | 14:19 |
| dtantsur | NobodyCam, it's instack, which is closer to devtest essentially | 14:19 |
| dtantsur | lucasagomes, will try, thanks | 14:19 |
| dtantsur | lucasagomes, ok, noauth works :) | 14:23 |
| lucasagomes | dtantsur, :) | 14:24 |
| lucasagomes | dtantsur, so the error might be coming from ks | 14:24 |
| dtantsur | lucasagomes, well... If I install Ironic manually, it works with Keystone | 14:25 |
| dtantsur | I still suspect something wrong with ironic rpms... | 14:25 |
| NobodyCam | seems like the ironic service is not getting registered with KS or something like that | 14:25 |
| NobodyCam | Invalid user token - rejecting request | 14:26 |
| lucasagomes | dtantsur, ah yeah | 14:26 |
| dtantsur | NobodyCam, how can I register it? What I did was service-create + endpoint-create (just like manually) | 14:26 |
| lucasagomes | somehting with the sertup | 14:26 |
| lucasagomes | setup* | 14:26 |
| NobodyCam | dt what do get with keystone service-list | 14:27 |
| dtantsur | | cfca1e6678c4480c81711984bce85985 | ironic | baremetal | <-- seems ok | 14:28 |
| NobodyCam | dtantsur: in the past you sent there are two different tokens are you sure one is correct? just checking.. | 14:30 |
| *** ndipanov has quit IRC | 14:30 | |
| dtantsur | NobodyCam, sorry for confusion, this is a copy-paste from two unrelated attempts | 14:31 |
| dtantsur | this thing is I got a bit different results for them | 14:32 |
| dtantsur | that's why I included both | 14:32 |
| NobodyCam | yaa | 14:33 |
| NobodyCam | *ahh | 14:33 |
| NobodyCam | :-p | 14:33 |
| Shrews | NobodyCam: Yeehaw, Texarcana! | 14:38 |
| *** stephenpearson has joined #openstack-ironic | 14:39 | |
| romcheg | Morning folks! | 14:40 |
| dtantsur | morning romcheg | 14:41 |
| romcheg | I'd like to give you an update: I'm currently working on some Rally benchmarks for Ironic | 14:41 |
| NobodyCam | mornig Shrews :) | 14:41 |
| NobodyCam | morning romcheg | 14:41 |
| romcheg | On the last meeting devananda told that it would be nice to see our API's performance | 14:41 |
| agordeev | Shrews, romcheg: morning | 14:41 |
| romcheg | Also there's no basic plumbing for Ironic in Rally so it needs to be done | 14:42 |
| Shrews | agordeev: morning :) | 14:42 |
| * romcheg feels like working with tempest and CI again :) | 14:42 | |
| *** ndipanov has joined #openstack-ironic | 14:44 | |
| rloo | thx romcheg. Any numbers? | 14:48 |
| romcheg | rloo: Not yet. | 14:48 |
| rloo | romcheg: was it as "simple" as the Rally person said it would be? | 14:49 |
| romcheg | rloo: It seems to be quite easy for basic benchmarking | 14:49 |
| romcheg | But some plumbing has to be done before implementing that | 14:50 |
| rloo | romcheg: there's always a but ;) | 14:50 |
| romcheg | I will try to show something today | 14:50 |
| romcheg | rloo: exectly :) | 14:50 |
| openstackgerrit | Pablo Fernando Cargnelutti proposed a change to openstack/ironic: Overwriting node_is_available in IronicDriver https://review.openstack.org/84214 | 14:52 |
| *** ifarkas has quit IRC | 14:54 | |
| agordeev | https://review.openstack.org/#/c/73223/4/ironic/tests/drivers/test_ipmitool.py L274, Gotcha in mock library? | 14:54 |
| agordeev | Mikhail_D_wk: ^ | 14:54 |
| Mikhail_D_wk | agordeev: ah, thanks! :) | 15:02 |
| *** lazy_prince has quit IRC | 15:05 | |
| *** yuriyz has quit IRC | 15:07 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Sync cliutils from oslo https://review.openstack.org/72417 | 15:08 |
| *** dwalleck has joined #openstack-ironic | 15:10 | |
| russell_h | agordeev: hey, I am now | 15:11 |
| russell_h | agordeev: ah, I | 15:14 |
| russell_h | er, I'm guessing I see what you wanted to talk about | 15:14 |
| russell_h | clever, I like it | 15:15 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 15:17 |
| agordeev | russell_h: hey, yup. Mostly about this -> https://review.openstack.org/#/c/87254/ | 15:18 |
| lucasagomes | dtantsur, heh yeah that factory might be over complicated | 15:18 |
| lucasagomes | dtantsur, what you think about having a static factory() method in the DriverFieldsFactory class which would return the correct class for a given node? | 15:19 |
| russell_h | agordeev: added a comment on one minor thing | 15:20 |
| dtantsur | lucasagomes, I mostly prefer just functions to staticmethods, otherwise it makes sense | 15:20 |
| lucasagomes | dtantsur, ack, could be a factory function as well | 15:21 |
| lucasagomes | dtantsur, will update the patch | 15:21 |
| openstackgerrit | Pablo Fernando Cargnelutti proposed a change to openstack/ironic: Overwriting node_is_available in IronicDriver https://review.openstack.org/84214 | 15:21 |
| russell_h | agordeev: also +2ed https://review.openstack.org/#/c/86341/ | 15:22 |
| russell_h | agordeev: all 3 look basically good to me excepting that I'd rather not call that param "self" | 15:23 |
| russell_h | I could be convinced otherwise | 15:24 |
| russell_h | anyway, going to be AFK for a bit | 15:24 |
| agordeev | russell_h: reasonable, thanks! | 15:24 |
| *** zul has quit IRC | 15:24 | |
| Shrews | So apparently, while working from a cafe this morning, a fire has started behind my house and I have no power at home. Neighbor says it's under control now, but I may not make the IRC meeting today. Heading home now to check it out. | 15:25 |
| *** coolsvap|afk is now known as coolsvap | 15:25 | |
| *** zul has joined #openstack-ironic | 15:25 | |
| openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Refactor the driver fields mapping https://review.openstack.org/86971 | 15:26 |
| *** dkehn has joined #openstack-ironic | 15:29 | |
| *** chuck_ has joined #openstack-ironic | 15:31 | |
| *** zul has quit IRC | 15:31 | |
| *** chuck_ is now known as zul | 15:32 | |
| matty_dubs | Oh geez, hope everything is alright, Shrews | 15:33 |
| NobodyCam | ieek, hope all is well Shrews | 15:34 |
| *** dkehn is now known as dkehnx | 15:35 | |
| *** BadCub01 has joined #openstack-ironic | 15:38 | |
| *** BadCub01 has left #openstack-ironic | 15:42 | |
| *** BadCub01 has joined #openstack-ironic | 15:42 | |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add FlowExtension https://review.openstack.org/85412 | 15:51 |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add self to validator method https://review.openstack.org/87254 | 15:51 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 2 https://review.openstack.org/73256 | 15:52 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 3 https://review.openstack.org/86536 | 15:52 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Cleanup mock patch without `with` part 1 https://review.openstack.org/73223 | 15:52 |
| openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid of the newline "\" https://review.openstack.org/66793 | 15:52 |
| *** rloo has quit IRC | 15:55 | |
| *** rloo has joined #openstack-ironic | 15:56 | |
| *** rloo has quit IRC | 15:56 | |
| *** rloo has joined #openstack-ironic | 15:57 | |
| NobodyCam | brb ... walkies time | 15:58 |
| openstackgerrit | Aleksandr Gordeev proposed a change to openstack/ironic-python-agent: Add FlowExtension https://review.openstack.org/85412 | 16:02 |
| *** coolsvap1 has joined #openstack-ironic | 16:02 | |
| *** zul has quit IRC | 16:02 | |
| *** eghobo has joined #openstack-ironic | 16:03 | |
| *** coolsvap1 has quit IRC | 16:03 | |
| *** dtantsur is now known as dtantsur|bbl | 16:04 | |
| *** zul has joined #openstack-ironic | 16:04 | |
| *** coolsvap1 has joined #openstack-ironic | 16:04 | |
| *** zul has quit IRC | 16:04 | |
| *** dwalleck has quit IRC | 16:07 | |
| *** coolsvap1 has quit IRC | 16:08 | |
| rloo | Hi Mikhail_D_wk: is this you? https://review.openstack.org/#/c/60025/27 | 16:08 |
| *** zul has joined #openstack-ironic | 16:08 | |
| *** newell_ has joined #openstack-ironic | 16:10 | |
| *** matty_dubs is now known as matty_dubs|lunch | 16:14 | |
| *** martyntaylor has left #openstack-ironic | 16:19 | |
| *** zdiN0bot has joined #openstack-ironic | 16:19 | |
| openstackgerrit | A change was merged to openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/86774 | 16:24 |
| *** dwalleck has joined #openstack-ironic | 16:24 | |
| devananda | g'morning, all | 16:27 |
| NobodyCam | good morning devananda :) | 16:27 |
| rloo | morning devananda. | 16:28 |
| lucasagomes | morning devananda | 16:33 |
| *** vkozhukalov has quit IRC | 16:34 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Cherry pick oslo rpc HA fixes https://review.openstack.org/87314 | 16:40 |
| devananda | i forgot to apply that fix to master when I did the backport to milestone-proposed ... this release is a learning process for us ... | 16:41 |
| jroll | morning all :) | 16:42 |
| devananda | though i should just pull the whole update from oslo, really | 16:42 |
| NobodyCam | morning jroll | 16:43 |
| jroll | \o | 16:43 |
| *** dwalleck_ has joined #openstack-ironic | 16:48 | |
| *** dwalleck has quit IRC | 16:51 | |
| *** harlowja_away is now known as harlowja | 16:54 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add version 1 of agent payload https://review.openstack.org/86173 | 16:54 |
| *** jistr has quit IRC | 16:55 | |
| adam_g | dwalleck_, ping | 16:56 |
| dwalleck_ | adam_g: Here now. What's up? | 16:57 |
| *** romcheg1 has joined #openstack-ironic | 16:57 | |
| *** romcheg has quit IRC | 16:57 | |
| adam_g | dwalleck_, hey--so the ironic scenario tests are running in the gate but do not seem to be passing. i spent friday trying to get to the bottom of it but was unable to reproduce on my own instances. ive narrowed it to TFTP issues. wondering if you ran into anything similar in your own testing that i may be overlooking | 16:58 |
| adam_g | nodes power on, get DHCP but never hit the TFTP server. | 16:58 |
| dwalleck_ | Hmm, I don't recall ever having that issue. Are you talking about the scenario you created, or are you having issues with the other nova scenarios? | 16:59 |
| *** romcheg has joined #openstack-ironic | 17:00 | |
| adam_g | dwalleck_, the baremetal scenario specifically. | 17:00 |
| dwalleck_ | Hmm, weird then. Once I was able to deploy ironic correctly with devstack it was working pretty consistently. Let me check again with what I have setup now | 17:01 |
| *** romcheg1 has quit IRC | 17:01 | |
| adam_g | dwalleck_, yeah, same for me. | 17:02 |
| dwalleck_ | Was there a sharp cutoff point where they stopped working? Or did it never work when run as the gate job? | 17:02 |
| dwalleck_ | The only thing that would be different for the gate job is how/where it's deployed | 17:03 |
| *** derekh has quit IRC | 17:03 | |
| adam_g | dwalleck_, only seem to be failing in the gate job | 17:04 |
| adam_g | i've tried to reproduce the environment as close to devstack-gate as possible. will continue to try that today | 17:05 |
| dwalleck_ | Well, I guess that it's good/bad that it can be narrowed down to that. | 17:05 |
| *** zdiN0bot has quit IRC | 17:06 | |
| *** epim has joined #openstack-ironic | 17:07 | |
| *** zdiN0bot has joined #openstack-ironic | 17:07 | |
| NobodyCam | Mikhail_D_wk: still around? | 17:08 |
| devananda | adam_g: what sort of TFTP issue? | 17:09 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding swift temp url support https://review.openstack.org/81391 | 17:09 |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Adding a reference driver for the agent https://review.openstack.org/84795 | 17:09 |
| adam_g | devananda, no idea. it just mysteriously doesn't work. looking at logs, i see the dnsmasq config written out with port info, the server restarted, node comes up and gets an IP but no request ever makes it to TFTP | 17:09 |
| dwalleck_ | adam_g: Can you point me to a failing run? The OpenStack Jenkins is melting my browser trying to load everything | 17:10 |
| adam_g | i think i've ruled out https://bugs.launchpad.net/ironic/+bug/1300589/comments/1 | 17:10 |
| adam_g | dwalleck_, http://logs.openstack.org/67/87067/1/experimental/check-tempest-dsvm-virtual-ironic-nv/afd80cc/ | 17:10 |
| *** lucasagomes is now known as lucas-afk | 17:10 | |
| devananda | adam_g: are you seeting the DHCP offer completion? | 17:10 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Bring agent deploy driver up to date with virt https://review.openstack.org/86490 | 17:11 |
| adam_g | devananda, looks like it | 17:11 |
| adam_g | http://logs.openstack.org/67/87067/1/experimental/check-tempest-dsvm-virtual-ironic-nv/afd80cc/logs/syslog.txt.gz | 17:11 |
| adam_g | Apr 12 03:50:48 | 17:11 |
| *** eghobo has quit IRC | 17:12 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Add lookup method to agent vendor mappings https://review.openstack.org/86402 | 17:12 |
| *** eguz has joined #openstack-ironic | 17:12 | |
| *** eguz has quit IRC | 17:12 | |
| *** eghobo has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Bring agent deploy driver up to date with virt https://review.openstack.org/86490 | 17:13 |
| NobodyCam | any seen this from neutron before? http://paste.openstack.org/show/cUi0tHNzh69GyNa0EnQE/ | 17:14 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Kill the build_agent function https://review.openstack.org/87327 | 17:15 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync RPC from oslo-incubator https://review.openstack.org/87328 | 17:16 |
| dwalleck_ | adam_g: Not sure if this helps, but working backwards from Compute, it seems to barf when attaching the port: HTTPBadRequest: Couldn't apply patch '[{'path': '/extra/vif_port_id', 'op': 'remove'}]'. Reason: u'vif_port_id' | 17:17 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: sync RPC from oslo-incubator https://review.openstack.org/87328 | 17:17 |
| adam_g | dwalleck_, oh, interesting | 17:18 |
| *** stephenpearson has quit IRC | 17:21 | |
| *** matty_dubs|lunch is now known as matty_dubs | 17:23 | |
| NobodyCam | we should prob do better at keeping in sync with oslo | 17:25 |
| * NobodyCam hopes GheRivero_ will keep us on our toes | 17:25 | |
| *** max_lobur has quit IRC | 17:26 | |
| linggao | Hi matty_dubs, thanks for reviewing the console patch. I'll fix the them after you do the testing on real hardware because the problems are minor. Is it okay with you? | 17:27 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Add version change sensor information https://review.openstack.org/87337 | 17:35 |
| *** zdiN0bot has quit IRC | 17:36 | |
| *** zdiN0bot has joined #openstack-ironic | 17:36 | |
| * Shrews back... for as long as his laptop battery holds out | 17:42 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Drivers may expose a top-level passthru API https://review.openstack.org/81919 | 17:42 |
| JoshNang | dwalleck_: got a sec? i've got some failing tempest tests and i'm not sure how to proceed | 17:46 |
| dwalleck_ | JoshNang: Sure, whatcha got? | 17:46 |
| JoshNang | i'm wrapping some exceptions, so I'm guessing a test somewhere broke. https://review.openstack.org/#/c/73121/ | 17:47 |
| jroll | JoshNang: did you and devananda ever talk about if IPA should be translated? | 17:47 |
| russell_h | jroll: JoshNang: JayF: https://review.openstack.org/#/c/86341/ | 17:48 |
| jroll | JoshNang: I'm hesitant to +2 this before that talk happens: https://review.openstack.org/#/c/87047/ | 17:48 |
| JoshNang | jroll: yup we did. leaning towards no | 17:48 |
| dwalleck_ | Okay, taking a look. Hopefully something easy | 17:48 |
| devananda | jroll: we did, briefly. | 17:48 |
| JoshNang | dwalleck_: hopefully. where are the tempest tests stored? | 17:48 |
| devananda | jroll: my view is, user-facing strings should be translated, but IPA doesn't really have those | 17:49 |
| jroll | devananda: I tend to agree - how about IPA errors that get bubbled back to the user? (not sure if those exist) | 17:49 |
| dwalleck_ | The actual tests? http://github.com/openstack/tempest | 17:49 |
| devananda | jroll: operator/admin-facing string translation is quiestionable. it might make it easier for some operators, but OTOH, it is more difficult to google search for a translated LOG.error message | 17:49 |
| devananda | jroll: i'm not sure they exist either. BUT. even if they did, IPA won't -- at least not right now -- know what language to translate them to | 17:50 |
| dwalleck_ | So I'm not sure any tests failed, but there were errors in the logs that it couldn't match to failures | 17:50 |
| JoshNang | dwalleck_: oooooo i didn't look deep enough in that repo. sweet. | 17:50 |
| devananda | jroll: the user lang is only exposed to python within the API service | 17:50 |
| devananda | jroll: that info does not, afaik, make it down to the conductor, and so it certainly doesn't make it to IPA | 17:50 |
| dwalleck_ | http://logs.openstack.org/21/73121/5/check/check-tempest-dsvm-ironic/11afea4/console.html , look all the way at the bottom to after the test summary finishes | 17:50 |
| devananda | that == http "Expect-language" header | 17:50 |
| jroll | devananda: true. thanks. | 17:51 |
| jroll | I'm going to just go ahead and block that then | 17:51 |
| JoshNang | dwalleck_: gotcha. the ir-api ones are definitely from me changing the outputs/wrapping exceptions. not sure on swiftclient | 17:52 |
| dwalleck_ | JoshNang: So the good news is that you didn't break a test. The bad news is that you may have broken something not tested =P | 17:52 |
| JoshNang | woooo! | 17:52 |
| JoshNang | ok, i can dig into this then. thanks for the help! | 17:53 |
| dwalleck_ | no problem! | 17:53 |
| *** coolsvap is now known as coolsvap|afk | 17:54 | |
| NobodyCam | grrr keep dropping | 17:55 |
| *** vkozhukalov has joined #openstack-ironic | 17:56 | |
| NobodyCam | grrr | 18:03 |
| *** dwalleck_ has quit IRC | 18:04 | |
| openstackgerrit | A change was merged to openstack/ironic: Cherry pick oslo rpc HA fixes https://review.openstack.org/87314 | 18:05 |
| *** jdob_ has joined #openstack-ironic | 18:06 | |
| *** dwalleck has joined #openstack-ironic | 18:07 | |
| russell_h | JayF: JoshNang: https://review.openstack.org/#/c/86201/ | 18:07 |
| JayF | looking | 18:08 |
| jroll | ^ those two should just take a look at all current IPA patches :P | 18:08 |
| JayF | I think I have a vote on most of them already | 18:08 |
| JayF | I'll doublecheck | 18:08 |
| jroll | nope :P | 18:08 |
| *** dwalleck has quit IRC | 18:09 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Resolve circular import, introduce ExecuteCommandMixin https://review.openstack.org/86341 | 18:09 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync oslo/config/generator https://review.openstack.org/87344 | 18:10 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync oslo/common/excutils https://review.openstack.org/87345 | 18:10 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Sync oslo logging https://review.openstack.org/87346 | 18:10 |
| *** stevehuang has joined #openstack-ironic | 18:12 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Add self to validator method https://review.openstack.org/87254 | 18:12 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Kill the build_agent function https://review.openstack.org/87327 | 18:12 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Fix conductor.manager test assertion order https://review.openstack.org/87035 | 18:13 |
| JayF | russell_h: JoshNang: this needs a look too | 18:17 |
| JayF | ^ https://review.openstack.org/#/c/86723/ | 18:17 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Add FlowExtension https://review.openstack.org/85412 | 18:17 |
| russell_h | JayF: jroll: what happens if you dd a file into a partition too small for it? | 18:19 |
| russell_h | JayF: jroll: put differently, do we need to validate the size of this file? | 18:19 |
| JayF | I asked jroll about this | 18:19 |
| JayF | he said nova had a cap of 64MB | 18:19 |
| JayF | and that's the size we're making the configdrive partition now | 18:20 |
| JayF | so it should be OK | 18:20 |
| jroll | that | 18:20 |
| jroll | but on that note | 18:20 |
| jroll | if nova's cap changes | 18:20 |
| russell_h | right | 18:20 |
| jroll | then yeahhhh | 18:20 |
| russell_h | or someone isn't using nova | 18:20 |
| JayF | also MiB vs MB | 18:20 |
| jroll | on that note, this partition is at the end of the drive | 18:22 |
| jroll | so I'd be shocked if dd did something weird | 18:22 |
| jroll | idk | 18:22 |
| jroll | we could validate it I suppose | 18:22 |
| JayF | Validating it seems smart | 18:23 |
| openstackgerrit | A change was merged to stackforge/pyghmi: Add version change sensor information https://review.openstack.org/87337 | 18:23 |
| jroll | JayF, russell_h, can one of you -1 that patch then? | 18:24 |
| russell_h | jroll: done | 18:24 |
| jroll | thanks | 18:24 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic-python-agent: Use configurable driver name in lookup URL https://review.openstack.org/86201 | 18:32 |
| *** jdob has quit IRC | 18:33 | |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Add more missing sensor specific event information https://review.openstack.org/87351 | 18:34 |
| *** pcargnel has joined #openstack-ironic | 18:38 | |
| comstud | sigh | 18:41 |
| NobodyCam | comstud: ? | 18:41 |
| comstud | Is there a hard rule about using decorators for mock patch? | 18:41 |
| *** zdiN0bot has quit IRC | 18:42 | |
| comstud | To test properly, I want to patch specific methods on an object | 18:42 |
| NobodyCam | thats the way things are moving | 18:42 |
| comstud | not a class or class method | 18:42 |
| NobodyCam | but I don't think we have a HARD rule. | 18:42 |
| NobodyCam | comment as to why you did or didn't inthe commit message | 18:43 |
| comstud | yeah | 18:43 |
| *** zdiN0bot has joined #openstack-ironic | 18:43 | |
| NobodyCam | maybe also a NOTE: in the code | 18:43 |
| NobodyCam | :) | 18:43 |
| comstud | I think there's a tricky way I can do this.. but just checking first | 18:43 |
| rloo | comstud: +1 (in code) | 18:44 |
| *** zdiN0bot has quit IRC | 18:47 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Use configurable driver name in lookup URL https://review.openstack.org/86201 | 18:48 |
| *** zdiN0bot has joined #openstack-ironic | 18:49 | |
| *** agordeev2 has joined #openstack-ironic | 18:50 | |
| *** eguz has joined #openstack-ironic | 18:51 | |
| devananda | NobodyCam: spinning up a tripleo env on my desktop today, and just got: No valid host was found. Reason: No conductor service registered which supports driver null. | 18:54 |
| devananda | NobodyCam: probably just something simple I missed...? | 18:54 |
| NobodyCam | devananda: sounds like it | 18:54 |
| NobodyCam | I have not seen that error | 18:55 |
| *** eghobo has quit IRC | 18:55 | |
| *** ifarkas has joined #openstack-ironic | 18:55 | |
| NobodyCam | did you actually run devtest.sh ? | 18:55 |
| NobodyCam | or manyally run thru th steps | 18:55 |
| devananda | NobodyCam: no. following the steps | 18:55 |
| devananda | NobodyCam: this happens during devtest_seed | 18:55 |
| NobodyCam | looking | 18:56 |
| *** dtantsur|bbl is now known as dtantsur | 18:57 | |
| *** lucas-afk is now known as lucasagomes | 18:57 | |
| *** mrda_away is now known as mrda | 18:58 | |
| NobodyCam | devananda node should be created with the power driver https://github.com/openstack/tripleo-incubator/blob/master/scripts/register-nodes#L46 | 18:59 |
| NobodyCam | meeting? | 19:00 |
| romcheg | + | 19:00 |
| devananda | meeting time! | 19:00 |
| lucasagomes | devananda, that's a new error, wondering how the node was created with the driver null | 19:00 |
| *** zdiN0bot has quit IRC | 19:01 | |
| *** zdiN0bot has joined #openstack-ironic | 19:02 | |
| openstackgerrit | A change was merged to stackforge/pyghmi: Add more missing sensor specific event information https://review.openstack.org/87351 | 19:03 |
| *** zdiN0bot has quit IRC | 19:06 | |
| *** vkozhukalov has quit IRC | 19:11 | |
| *** vkozhukalov has joined #openstack-ironic | 19:12 | |
| *** max_lobur has joined #openstack-ironic | 19:20 | |
| *** zdiN0bot has joined #openstack-ironic | 19:31 | |
| *** zdiN0bot has quit IRC | 19:33 | |
| *** zdiN0bot has joined #openstack-ironic | 19:33 | |
| *** pcargnel has quit IRC | 19:37 | |
| *** pcargnel_ has joined #openstack-ironic | 19:41 | |
| pcargnel_ | Hi, can someone could help me understand this one? https://bugs.launchpad.net/ironic/+bug/1284781 | 19:41 |
| NobodyCam | we're in meeting right now | 19:43 |
| NobodyCam | can you give us 2- moinutes | 19:43 |
| NobodyCam | minutes even | 19:43 |
| NobodyCam | 20 | 19:43 |
| NobodyCam | :-p | 19:43 |
| pcargnel_ | no prob. thanks :) | 19:44 |
| NobodyCam | :) | 19:44 |
| *** zdiN0bot has quit IRC | 19:47 | |
| adam_g | dtantsur, ah! i think it may be iptables in devstack gate causing issues | 19:50 |
| dtantsur | adam_g quite possible | 19:51 |
| dtantsur | adam_g afaik IPv6 issue if Fedora-specific, you should not encounter it | 19:51 |
| dtantsur | adam_g, could you point, how I can try the test scenario? It's 10pm here, and my brain does not want to work at all :( | 19:53 |
| adam_g | dtantsur, #openstack-infra just confirmed that devstack-gate nodes have strict iptables rules applied to $HOST_IP, which is where the nodes are attempting to tftp from | 19:53 |
| dtantsur | aha, I see | 19:53 |
| adam_g | dtantsur, deploy devstack /w ironic. in tempest.conf, make sure 'allow_tenant_isolation = False' and flavor_ref points to the correct baremetal flavor id | 19:54 |
| adam_g | dtantsur, then run the single scenario test (if you run the others you will have a bad time) | 19:54 |
| adam_g | dtantsur, i usually run it with nose... 'nosetests tempest/scenario/test_baremetal_basic_ops.py' | 19:54 |
| dtantsur | ok, now it's time for me to finally figure out how tempest work :) | 19:54 |
| * dtantsur is a new guy at openstack | 19:54 | |
| dtantsur | adam_g, thanks | 19:55 |
| *** tatyana has joined #openstack-ironic | 20:00 | |
| NobodyCam | good meeting all | 20:00 |
| NobodyCam | brb | 20:01 |
| lucasagomes | JayF, anyhoo, a short tutorial about how u guys create the images now would be very appreciate | 20:01 |
| JayF | lucasagomes: there's a readme in the agent repo itself about imagebuild for coreos | 20:01 |
| *** vkozhukalov has left #openstack-ironic | 20:01 | |
| jroll | lucasagomes: https://github.com/openstack/ironic-python-agent/tree/master/imagebuild/coreos | 20:01 |
| lucasagomes | JayF, ah, cheers | 20:01 |
| jroll | it doesn't document dependencies but yeah | 20:01 |
| lucasagomes | jroll, thanks will take a look :) | 20:01 |
| jroll | :) | 20:01 |
| lucasagomes | jroll, np its grand | 20:01 |
| *** gmatefi has joined #openstack-ironic | 20:01 | |
| JayF | lucasagomes: but right now I'm pulling it out of a docker container and putting it into a systemd-nspawn container instead, since we don't want to be abstracted as much from the hardware | 20:01 |
| lucasagomes | +1 | 20:02 |
| dtantsur | What I also wanted to discuss is a lot of really large review requests (e.g. test refactoring) that can't be just merged, because any other change breaks them | 20:02 |
| JayF | lucasagomes: I'll make sure to update the docs as that changes, but I'll continue to abstract out the build commands into Makefile | 20:02 |
| *** iron1 has joined #openstack-ironic | 20:02 | |
| dtantsur | but this can wait | 20:02 |
| lucasagomes | JayF, good stuff nspawn is cool! chroot on steroids heh | 20:03 |
| JayF | yeah exactly | 20:03 |
| lucasagomes | aye... will eat something, it's late here and I'm pretty hungry | 20:04 |
| lucasagomes | have a good night everyone! | 20:04 |
| JayF | lucasagomes: Have you built pxe images/ramdisks with coreos before? It's pretty great. You should look at how we're doing stuff | 20:04 |
| JayF | once you've eaten and slept and things of that nature :) | 20:04 |
| jroll | night lucas | 20:04 |
| lucasagomes | I will take a look at the agent stuff later/tomorrow | 20:04 |
| JayF | have a nice day :) | 20:04 |
| JayF | thanks | 20:04 |
| lucasagomes | JayF, I haven't | 20:04 |
| lucasagomes | looking fwd, tomororw I will give it a go | 20:04 |
| jroll | devananda: so I understand the main answer about all of those agent patches is time; if there's anything at all I can do to enable that please let me know :) | 20:04 |
| *** lucasagomes is now known as lucas-dinner | 20:05 | |
| dtantsur | good night, 10pm already | 20:05 |
| matty_dubs | See ya, dtantsur | 20:05 |
| adam_g | cheers, dtantsur | 20:05 |
| *** dtantsur is now known as dtantsur|zzzZZZz | 20:06 | |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Auto degrade sessions to operator in environments that forbid administrator https://review.openstack.org/87370 | 20:06 |
| romcheg | adam_g: I've just thought to move our chat to tomorrow, because it's 11PM here and this can take much time | 20:06 |
| *** ifarkas has quit IRC | 20:07 | |
| romcheg | adam_g: if you don't mind | 20:07 |
| adam_g | romcheg, no problem. i think it may be firewall related, actually. working on a test fix now | 20:07 |
| russell_h | anyone know if this is a real failure? http://logs.openstack.org/19/81919/7/check/check-tempest-dsvm-ironic/69dcb90/ | 20:07 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Auto degrade sessions to operator when admin is forbidden https://review.openstack.org/87370 | 20:07 |
| russell_h | "EC2ResponseError: EC2ResponseError: 400 Bad Request" | 20:08 |
| romcheg | adam_g: I hope it's firewall or selinux. But if the problem still exists tomorrow, we can take a loot together. Two pairs of eyes is always better :) | 20:08 |
| adam_g | romcheg, definitely. have a good night! | 20:09 |
| jbjohnso_ | take a loot tomorrow | 20:09 |
| jbjohnso_ | guess that could be more profitable | 20:09 |
| romcheg | Good night guys! | 20:09 |
| romcheg | Have a nice day/evening | 20:09 |
| openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Auto degrade sessions to operator when admin is forbidden https://review.openstack.org/87370 | 20:10 |
| jbjohnso_ | one thing about split personality, you always have someone to review your code | 20:11 |
| *** BadCub01 has quit IRC | 20:15 | |
| matty_dubs | linggao: Just tested 64100 in my environment; works A-OK! | 20:16 |
| linggao | matty_dubs, thanks for testing it. Good to know it's working! | 20:16 |
| jbjohnso_ | matty_dubs, speaking of serial console.. | 20:17 |
| jbjohnso_ | matty_dubs, https://www.youtube.com/watch?v=G_lDaktYnsQ | 20:17 |
| linggao | matty_dubs, then I'll check in the fix for your comments. | 20:17 |
| * matty_dubs readies +1 | 20:18 | |
| matty_dubs | jbjohnso_: Ooh! | 20:18 |
| *** gmatefi has quit IRC | 20:18 | |
| openstackgerrit | linggao proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 20:21 |
| jbjohnso_ | matty_dubs, the code is in the world now, open for mocking | 20:23 |
| matty_dubs | Ooh, where? | 20:24 |
| *** jdob has joined #openstack-ironic | 20:24 | |
| jbjohnso_ | matty_dubs, https://sourceforge.net/p/xcat/confluent/ci/master/tree/ | 20:24 |
| jbjohnso_ | I want the health resource showing the explanation for a 'bad' grade | 20:24 |
| jroll | woohoo! | 20:25 |
| jbjohnso_ | and then I'll probably move on to making it properly daemonize | 20:25 |
| jbjohnso_ | and then packaging | 20:25 |
| jbjohnso_ | currently it has the raw data to show, but I haven't defined a message object to hold it yet, got distracted by some other sutff | 20:25 |
| matty_dubs | "got distracted by some other stuff" | 20:26 |
| matty_dubs | The story of my life | 20:26 |
| matty_dubs | ;) | 20:27 |
| jbjohnso_ | oh, and then I saw that if I tried to connect with a bad password, the automatic retries don't get attempted to see if the password got changed to match the server. If the server changes password it does know to check immediately | 20:27 |
| jbjohnso_ | oh, I also need to reuse the sdr object so that health is trivial to fetch | 20:27 |
| matty_dubs | Capability-wise, it looks pretty slick | 20:28 |
| jbjohnso_ | taking 1.7 seconds to get health is tortuous | 20:29 |
| jbjohnso_ | "health": {"value": "warning"} | 20:30 |
| jbjohnso_ | the detail missing that I need to cram: {'badreadings': [{'name': 'SEL Fullness', 'imprecision': None, 'value': None, 'states': ['Event log nearly full'], 'health': 1, 'units': '', 'type': 'Event Log Disabled'}], 'health': 1} | 20:30 |
| Shrews | devananda: Whenever you have some time, please check out my comment on this: https://review.openstack.org/#/c/86993/1 | 20:31 |
| Shrews | devananda: open to any suggestions you may have re: complete code-path testing | 20:31 |
| linggao | devananda, what makes more sense v1/node/uuid/console_info or v1/nodes/uuid/consoe given that PUT v1/nodes/uuid/states/consle turns console on and off? | 20:33 |
| openstackgerrit | Jay Faulkner proposed a change to openstack/ironic-python-agent: Minor fixes to image build https://review.openstack.org/87378 | 20:33 |
| jbjohnso_ | matty_dubs, yes, that new terminal code is *worlds* better than shellinabox's | 20:33 |
| linggao | jbjohnso_, do you know that is the protocol used by shellinabox? | 20:35 |
| linggao | that _> what | 20:35 |
| jbjohnso_ | linggao, sure, http ;) | 20:35 |
| linggao | I need to put the protocal name in the 'type' attribute for the console info. | 20:36 |
| jbjohnso_ | specifically, I did do 'net' tab dissection of their behavior to replicate the server half when I thought I was going to be endorsing shellinabox's client js code | 20:36 |
| linggao | for example: in nova, the type for vnc console is 'novnc' | 20:36 |
| jbjohnso_ | I changed the way the implementation sends data to the server because shellinabox's was kind of weird... | 20:36 |
| jbjohnso_ | when I switched to term.js and had to write the networking piece entirely anyway | 20:37 |
| matty_dubs | I haven't actually looked at how it works... I'm kinda afraid to ;) | 20:37 |
| jbjohnso_ | matty_dubs, it *could* behave more quickly using websockets, but I stuck with Ajax because in my experience web sockets and proxies get weird and for text consoles the difference is not too bad | 20:38 |
| jbjohnso_ | linggao, dunno if it has a name, just a particular JSON encoded scheme over http requests | 20:38 |
| jbjohnso_ | linggao, nothing used outside of shellinabox itself | 20:39 |
| jbjohnso_ | matty_dubs, the real smarts is in a quality vt100 javascript implementation. That is magical. | 20:40 |
| jbjohnso_ | thankfully, I didn't need those brains since term.js exists and is MIT licensed | 20:41 |
| matty_dubs | :) | 20:41 |
| matty_dubs | Sometimes that is my favorite part of open-source -- being able to hook together pieces that people way smarter than me wrote and build something really neat | 20:42 |
| jbjohnso_ | linggao, so I'd call it 'shellinabox' protocol I guess for lack of a better term | 20:42 |
| jbjohnso_ | linggao, just like I'd call my interface 'confluent' protocol | 20:42 |
| linggao | jbjohnso_, I'll put shellinabox there in the type. | 20:43 |
| jbjohnso_ | though in my case that's ambiguous, since there is an http based and non-http based variante to the same data | 20:43 |
| jbjohnso_ | speaking of which, I might want some guidance on how openstack would ideally present my daemon with authentication data for a given console | 20:44 |
| jbjohnso_ | today shellinabox doesn't pretect things at all, and confluent requires some sort of authentication | 20:45 |
| jbjohnso_ | It could either be a way to authenticate 'users' or else a way for a particular user to request tokens to delegate console authentication and authorization | 20:47 |
| linggao | jbjohnso_, maybe the user part needs to be considered at highter level. | 20:51 |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Minor fixes to image build https://review.openstack.org/87378 | 20:53 |
| *** jdob_ has quit IRC | 20:56 | |
| jbjohnso_ | linggao, not sure I follow.. | 20:58 |
| *** tatyana has quit IRC | 20:58 | |
| linggao | jbjohnso_, I am not sure I understand this part yet. I thought nova decides who gets what access. | 21:00 |
| matty_dubs | linggao: Oh, one more nit -- it's literally os.makedirs, not os.mkdirs (even though that makes more sense) -- https://docs.python.org/2/library/os.html#os.makedirs | 21:05 |
| linggao | matty_dubs, let me fix it now..... Thanks. | 21:06 |
| matty_dubs | linggao: BTW, where are you located? The .cn in your email address made me think China, but you're online at US-like hours. | 21:09 |
| linggao | matty_dubs, I am located in upstate New York. I am working on the sjing's patch. The email address is hers. | 21:10 |
| linggao | Maybe I should change the owner of this patch to make it clear :) | 21:10 |
| matty_dubs | Haha, oops... or I should learn to read ;) | 21:11 |
| *** jbjohnso_ has quit IRC | 21:12 | |
| matty_dubs | Though it'd probably be fair to add a Coauthored-By or whatever it is | 21:13 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 21:13 |
| linggao | yes, not sure how to do that. | 21:14 |
| linggao | :) | 21:14 |
| matty_dubs | Haha, ditto. Someone had recommended it back when I was working on this, but I ended up not adding a lot | 21:15 |
| linggao | :) | 21:15 |
| *** jdob has quit IRC | 21:17 | |
| linggao | I have to go. good night matty_dubs, jbjohnso_ and all. | 21:18 |
| matty_dubs | G'night, linggao! | 21:19 |
| *** linggao has quit IRC | 21:19 | |
| NobodyCam | night lifeless | 21:19 |
| NobodyCam | doh sorry lifeless shaould have been linggao | 21:19 |
| matty_dubs | Did someone tell me it's a faux pas to -1/+1 before Jenkins? | 21:19 |
| matty_dubs | Well, a more literal question -- is that the case? | 21:20 |
| lifeless | NobodyCam: :P | 21:20 |
| lifeless | matty_dubs: not really | 21:21 |
| NobodyCam | oh brb | 21:25 |
| *** max_lobur has quit IRC | 21:25 | |
| openstackgerrit | Pablo Fernando Cargnelutti proposed a change to openstack/ironic: Overwriting node_is_available in IronicDriver https://review.openstack.org/84214 | 21:26 |
| openstackgerrit | linggao proposed a change to openstack/ironic: Support serial console access https://review.openstack.org/64100 | 21:28 |
| *** linggao has joined #openstack-ironic | 21:29 | |
| *** linggao has quit IRC | 21:29 | |
| devananda | anyone looking for a quick patch to write? | 21:33 |
| *** dkehnx has quit IRC | 21:33 | |
| NobodyCam | should we ask why | 21:33 |
| devananda | cause ya'll like to write code? :) | 21:34 |
| comstud | i prefer time consuming patches | 21:37 |
| rloo | devananda. depends. when did you want it written by? | 21:39 |
| rloo | devananda. actually, re-read your question. no, I'm not looking ;) | 21:39 |
| devananda | rloo: :) | 21:40 |
| rloo | (feeling bad about all those unreviewed patches...) | 21:40 |
| devananda | rloo: as you should :) | 21:41 |
| rloo | ha ha. | 21:41 |
| NobodyCam | what would this quick entail | 21:42 |
| * devananda hacks together a quick patch | 21:42 | |
| NobodyCam | gah s/quick/quck patch/ | 21:42 |
| rloo | ooo, a possible bite ;) | 21:42 |
| *** dkehn_ has joined #openstack-ironic | 21:45 | |
| *** dkehn_ is now known as dkehnx | 21:46 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Add configdrive generation to Nova driver https://review.openstack.org/86192 | 21:50 |
| *** chuck_ has joined #openstack-ironic | 21:50 | |
| openstackgerrit | Chris Krelle proposed a change to openstack/ironic: Fix for tripleO undercloud gate tests DO NOT MERGE https://review.openstack.org/85529 | 21:50 |
| *** zul has quit IRC | 21:51 | |
| *** chuck_ has quit IRC | 21:53 | |
| *** zul has joined #openstack-ironic | 21:53 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic: Fix conductor.manager test assertion order https://review.openstack.org/87035 | 21:55 |
| *** zul has quit IRC | 21:57 | |
| openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Fix leaking DB details to API on error https://review.openstack.org/73121 | 21:57 |
| openstackgerrit | Jim Rollenhagen proposed a change to openstack/ironic: Support for v2 agent payloads https://review.openstack.org/86984 | 22:01 |
| NobodyCam | be back in a bit. | 22:05 |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Organize agent extensions https://review.openstack.org/87057 | 22:09 |
| matty_dubs | I'm headed out as well. G'night, #openstack-ironic! | 22:10 |
| devananda | g'night! | 22:10 |
| *** matty_dubs is now known as matty_dubs|gone | 22:11 | |
| NobodyCam | night matty_dubs|gone | 22:11 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add ephemeral support to nova.virt.ironic https://review.openstack.org/87396 | 22:16 |
| *** zul has joined #openstack-ironic | 22:17 | |
| *** zul has quit IRC | 22:21 | |
| *** zul has joined #openstack-ironic | 22:21 | |
| openstackgerrit | Russell Haering proposed a change to openstack/ironic-python-agent: Add a @sync_command() decorator https://review.openstack.org/87144 | 22:22 |
| Shrews | matty_dubs|gone, lifeless: actually, i believe sdague sent an email out a while back asking for people to not +1/-1 reviews until jenkins had voted. not sure if that still applies | 22:23 |
| *** athomas has quit IRC | 22:24 | |
| *** zul has quit IRC | 22:24 | |
| *** chuck_ has joined #openstack-ironic | 22:25 | |
| openstackgerrit | A change was merged to openstack/ironic-python-agent: Organize agent extensions https://review.openstack.org/87057 | 22:26 |
| lifeless | Shrews: you certainly shouldn't +A things | 22:27 |
| lifeless | Shrews: this was because you can +A things that failed validation and they go into the integrated gate, and then fail which is hugely disruptive. | 22:27 |
| Shrews | lifeless: i know, but i'm pretty sure his request was to NOT +1/-1. i no longer have the email though | 22:30 |
| Shrews | likely not much of an issue now anyway | 22:30 |
| JayF | it's really hard to not want to vote on a review at all when you get half-day jenkins build wait times :) | 22:30 |
| JayF | although I wouldn't +A something until jenkins passed, ofc | 22:31 |
| JayF | just seems crummy that the person would have to wait a long time for any feedback at all | 22:31 |
| *** epim has quit IRC | 22:32 | |
| *** radsy has joined #openstack-ironic | 22:34 | |
| *** radsy has joined #openstack-ironic | 22:34 | |
| *** chuck_ is now known as zul | 22:34 | |
| *** derekh has joined #openstack-ironic | 22:39 | |
| *** epim has joined #openstack-ironic | 22:39 | |
| *** agordeev2 has quit IRC | 22:41 | |
| Shrews | JayF: i wouldn't worry about at this point. it was back when the gate was wedged all the time. i'm probably mis-remembering anyway | 22:42 |
| *** pcargnel_ has quit IRC | 22:45 | |
| *** epim has quit IRC | 23:03 | |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Nova.virt.driver updates driver_info more correctly https://review.openstack.org/87407 | 23:09 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add ephemeral support to nova.virt.ironic https://review.openstack.org/87396 | 23:09 |
| *** epim has joined #openstack-ironic | 23:09 | |
| *** epim has quit IRC | 23:10 | |
| russell_h | rloo: fixed up https://review.openstack.org/#/c/87035/ | 23:10 |
| *** romcheg has quit IRC | 23:15 | |
| NobodyCam | devananda: line 276 https://review.openstack.org/#/c/87407/1/ironic/nova/virt/ironic/driver.py (white space) | 23:28 |
| *** blamar has quit IRC | 23:31 | |
| *** lucas-dinner has quit IRC | 23:41 | |
| jroll | has anyone run into image_meta == {} in virt.ironic.driver.spawn? assuming image-show gives me valid info for my image id? | 23:45 |
| devananda | Shrews: hi! around? | 23:53 |
| devananda | jroll: that doesn't sound familiar to me, no | 23:54 |
| jroll | ok, it may just be my glance setup | 23:54 |
| jroll | thanks | 23:54 |
| devananda | NobodyCam: d'oh | 23:55 |
| NobodyCam | :) but ++ I like it :) | 23:58 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Nova.virt.driver updates driver_info more correctly https://review.openstack.org/87407 | 23:58 |
| openstackgerrit | Devananda van der Veen proposed a change to openstack/ironic: Add ephemeral support to nova.virt.ironic https://review.openstack.org/87396 | 23:59 |
| devananda | i'm not sure if those will conflict with lucas' patch sets | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!