| *** Sukhdev has quit IRC | 00:22 | |
| NobodyCam | thiagop: awesome!!!! | 00:33 |
|---|---|---|
| *** boris-42 has quit IRC | 00:34 | |
| thiagop | now, after 2hs filling the DS-160, I'll call it a day | 00:38 |
| *** Sukhdev has joined #openstack-ironic | 00:38 | |
| thiagop | see you tomorrow folks | 00:38 |
| *** thiagop has quit IRC | 00:38 | |
| *** hoangcx has joined #openstack-ironic | 00:42 | |
| *** aNupoisc has joined #openstack-ironic | 00:43 | |
| * jlvillal remembers filing out the DS-160 for his wife. It is a long long form :( | 00:45 | |
| jlvillal | s/filing/filling/ | 00:45 |
| jlvillal | NobodyCam: Are you recovered from being sick? I hope so! | 00:46 |
| *** baoli has joined #openstack-ironic | 00:52 | |
| *** Sukhdev has quit IRC | 00:52 | |
| *** suro-patz has quit IRC | 00:56 | |
| aNupoisc | HI John | 00:57 |
| *** baoli has quit IRC | 00:58 | |
| jlvillal | aNupoisc: Hey there. | 01:00 |
| jlvillal | Welcome to the Ironic channel :) | 01:00 |
| aNupoisc | Thank you. Great we are connected now | 01:00 |
| *** amotoki has joined #openstack-ironic | 01:01 | |
| aNupoisc | jlvillal: How are you doing today | 01:01 |
| aNupoisc | <jlvillal>: Hope you had a fine day | 01:02 |
| jlvillal | aNupoisc: Doing well. Thanks. Just trying to get the miscellaneous work things done today... | 01:02 |
| *** garthb has quit IRC | 01:03 | |
| aNupoisc | jlvilla: Nice. I am upgrading the system bcoz scheduler wasn't working and the reason was no VT-x | 01:05 |
| *** [1]rpioso has quit IRC | 01:06 | |
| *** penick has quit IRC | 01:08 | |
| *** suro-patz has joined #openstack-ironic | 01:13 | |
| *** suro-patz has quit IRC | 01:26 | |
| *** kan_ has joined #openstack-ironic | 01:48 | |
| *** mtanino has quit IRC | 01:55 | |
| *** aNupoisc has quit IRC | 01:55 | |
| *** rloo has quit IRC | 01:57 | |
| *** suro-patz has joined #openstack-ironic | 02:01 | |
| *** ChrisAusten has joined #openstack-ironic | 02:02 | |
| *** dims has quit IRC | 02:04 | |
| *** Sukhdev has joined #openstack-ironic | 02:08 | |
| *** dims has joined #openstack-ironic | 02:08 | |
| *** baoli has joined #openstack-ironic | 02:11 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add support for console ports allocation/de-allocation https://review.openstack.org/257789 | 02:11 |
| *** dims has quit IRC | 02:13 | |
| *** Sukhdev has quit IRC | 02:14 | |
| *** Marga__ has joined #openstack-ironic | 02:16 | |
| *** dims has joined #openstack-ironic | 02:17 | |
| *** baoli has quit IRC | 02:18 | |
| *** baoli has joined #openstack-ironic | 02:18 | |
| *** Marga_ has quit IRC | 02:19 | |
| *** hemna_ has joined #openstack-ironic | 02:19 | |
| *** baoli has quit IRC | 02:22 | |
| *** baoli has joined #openstack-ironic | 02:23 | |
| *** suro-patz has quit IRC | 02:31 | |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 02:34 |
| openstackgerrit | Haomeng,Wang proposed openstack/ironic-python-agent: add new portal_port argument into iscsi.start_iscsi_target api https://review.openstack.org/272420 | 02:37 |
| *** jaybeale has quit IRC | 02:42 | |
| *** deray has joined #openstack-ironic | 02:43 | |
| *** ijw has quit IRC | 02:43 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Follow up nits of I57bf7cebfb6db805b6c521bacfef2993b16ce1ee https://review.openstack.org/286364 | 02:54 |
| *** hemna_ has quit IRC | 02:57 | |
| NobodyCam | jlvillal: getting better may even try and make it to my desk tomorrow :) | 02:57 |
| jlvillal | NobodyCam, Glad to hear you are getting better | 02:57 |
| * jlvillal just watched a music video on tieing a know... | 02:57 | |
| NobodyCam | tieing a know? | 02:59 |
| NobodyCam | knot? | 02:59 |
| *** Nisha_brb has joined #openstack-ironic | 03:00 | |
| jlvillal | NobodyCam, A knot | 03:00 |
| *** baoli has quit IRC | 03:00 | |
| NobodyCam | :) | 03:00 |
| jlvillal | NobodyCam, https://www.youtube.com/watch?v=TUHgGK-tImY Norwegian band or swedish??? | 03:01 |
| jlvillal | LOL | 03:01 |
| NobodyCam | oh cool :) | 03:01 |
| jlvillal | Just dumb enough to be entertaining. | 03:01 |
| NobodyCam | you ever watch the euro star stuff | 03:01 |
| jlvillal | NobodyCam, I have not. | 03:02 |
| jlvillal | And the video within the video: https://www.youtube.com/watch?v=UEXcVvxmV3s | 03:02 |
| * jlvillal is pretty sure the guy is wearing a toupee | 03:04 | |
| NobodyCam | https://www.youtube.com/watch?v=WKNRGc71hjc | 03:05 |
| NobodyCam | euro visoin not euro star later is a train | 03:06 |
| jlvillal | NobodyCam, Thanks | 03:06 |
| cinerama | is eurovision on in the US??? need to watch | 03:06 |
| jlvillal | cinerama, That's usvision that's on in the US :P | 03:06 |
| *** spandhe has quit IRC | 03:07 | |
| jlvillal | NobodyCam, I'll have to ask my wife about that particular Eurovision entry :) | 03:11 |
| *** dims has quit IRC | 03:19 | |
| NobodyCam | jlvillal: that a few years old now :) | 03:20 |
| NobodyCam | butI still like it | 03:20 |
| *** yuanying has joined #openstack-ironic | 03:21 | |
| *** yuanying_ has quit IRC | 03:23 | |
| *** baoli has joined #openstack-ironic | 03:25 | |
| *** Marga__ has quit IRC | 03:26 | |
| *** yuanying has quit IRC | 03:27 | |
| *** Marga_ has joined #openstack-ironic | 03:30 | |
| *** Nakato has quit IRC | 03:31 | |
| *** Marga_ has quit IRC | 03:35 | |
| openstackgerrit | Kan proposed openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 03:36 |
| *** Marga_ has joined #openstack-ironic | 03:43 | |
| *** harshs has quit IRC | 03:45 | |
| *** Nakato has joined #openstack-ironic | 03:45 | |
| *** Marga_ has quit IRC | 03:47 | |
| *** baoli has quit IRC | 03:49 | |
| *** links has joined #openstack-ironic | 03:49 | |
| *** appu103_ has joined #openstack-ironic | 03:53 | |
| *** chlong_ has quit IRC | 04:03 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Add clean step in iLO drivers to activate iLO license https://review.openstack.org/259921 | 04:04 |
| openstackgerrit | Kan proposed openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 04:06 |
| *** yuanying has joined #openstack-ironic | 04:10 | |
| *** Marga_ has joined #openstack-ironic | 04:16 | |
| *** Marga_ has quit IRC | 04:16 | |
| *** Marga_ has joined #openstack-ironic | 04:17 | |
| *** danimal has quit IRC | 04:30 | |
| *** deray has quit IRC | 04:41 | |
| *** deray has joined #openstack-ironic | 04:41 | |
| *** vaishnavi has joined #openstack-ironic | 04:41 | |
| *** jaybeale has joined #openstack-ironic | 04:45 | |
| deray | Good morning all my Ironicers! | 04:47 |
| *** kalpase has joined #openstack-ironic | 04:48 | |
| *** jaybeale has quit IRC | 04:51 | |
| *** phuongnh has joined #openstack-ironic | 04:56 | |
| vaishnavi | good morning deray ! | 04:56 |
| deray | vaishnavi, o/ | 04:57 |
| *** suro-patz has joined #openstack-ironic | 04:57 | |
| *** Haomeng has quit IRC | 05:16 | |
| *** Marga_ has quit IRC | 05:20 | |
| *** spandhe has joined #openstack-ironic | 05:25 | |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic: Updated iLO driver documentation to recommend ipmitool version https://review.openstack.org/274547 | 05:25 |
| *** suro-patz has quit IRC | 05:29 | |
| *** vinm213 has joined #openstack-ironic | 05:29 | |
| *** Marga_ has joined #openstack-ironic | 05:30 | |
| *** Haomeng|2 has joined #openstack-ironic | 05:34 | |
| openstackgerrit | Kan proposed openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 05:41 |
| *** ChrisAusten has quit IRC | 05:42 | |
| *** Sukhdev has joined #openstack-ironic | 05:55 | |
| *** lucasagomes has quit IRC | 06:02 | |
| *** lucasagomes has joined #openstack-ironic | 06:02 | |
| *** spandhe_ has joined #openstack-ironic | 06:06 | |
| *** spandhe has quit IRC | 06:08 | |
| *** spandhe_ is now known as spandhe | 06:08 | |
| *** rcernin has joined #openstack-ironic | 06:09 | |
| *** shu-mutou-AWAY is now known as shu-mutou | 06:10 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Follow up nits of I57bf7cebfb6db805b6c521bacfef2993b16ce1ee https://review.openstack.org/286364 | 06:18 |
| *** suro-patz has joined #openstack-ironic | 06:20 | |
| *** e0ne has joined #openstack-ironic | 06:23 | |
| *** hoangcx has quit IRC | 06:24 | |
| *** lucasagomes has quit IRC | 06:24 | |
| *** lucasagomes has joined #openstack-ironic | 06:27 | |
| *** e0ne has quit IRC | 06:31 | |
| *** thrash|g0ne has quit IRC | 06:33 | |
| *** e0ne has joined #openstack-ironic | 06:33 | |
| *** e0ne has quit IRC | 06:34 | |
| *** trown|outtypewww has quit IRC | 06:34 | |
| *** trown has joined #openstack-ironic | 06:35 | |
| *** suro-patz has quit IRC | 06:43 | |
| *** Sukhdev has quit IRC | 06:45 | |
| *** harlowja_at_home has quit IRC | 06:49 | |
| *** ishant has joined #openstack-ironic | 06:53 | |
| openstackgerrit | Merged openstack/ironic: [Devstack]Add ability to enable ironic node pty console https://review.openstack.org/256272 | 06:56 |
| *** Marga_ has quit IRC | 06:57 | |
| *** achanda has quit IRC | 06:58 | |
| *** pas-ha has joined #openstack-ironic | 06:58 | |
| *** suro-patz has joined #openstack-ironic | 07:07 | |
| *** hoangcx has joined #openstack-ironic | 07:11 | |
| *** spandhe has quit IRC | 07:15 | |
| *** achanda has joined #openstack-ironic | 07:15 | |
| *** hoangcx has quit IRC | 07:26 | |
| *** suro-patz has quit IRC | 07:26 | |
| pas-ha | morning Ironic | 07:28 |
| *** bnemec has quit IRC | 07:36 | |
| *** bnemec has joined #openstack-ironic | 07:39 | |
| *** achanda_ has joined #openstack-ironic | 08:02 | |
| *** achanda has quit IRC | 08:04 | |
| *** achanda_ has quit IRC | 08:04 | |
| *** slagle has quit IRC | 08:06 | |
| *** achanda has joined #openstack-ironic | 08:08 | |
| *** pcaruana has joined #openstack-ironic | 08:09 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic-specs: Move ssh-console-support spec to 4.3 https://review.openstack.org/286432 | 08:09 |
| *** amotoki has quit IRC | 08:18 | |
| *** amotoki has joined #openstack-ironic | 08:19 | |
| *** achanda has quit IRC | 08:27 | |
| *** ifarkas has joined #openstack-ironic | 08:30 | |
| *** vaishnavi_r has joined #openstack-ironic | 08:32 | |
| *** vaishnavi has quit IRC | 08:32 | |
| *** ohamada has joined #openstack-ironic | 08:35 | |
| *** daemontool__ has joined #openstack-ironic | 08:38 | |
| *** daemontool_ has quit IRC | 08:41 | |
| *** ndipanov has joined #openstack-ironic | 08:44 | |
| openstackgerrit | Rui Chen proposed openstack/python-ironicclient: Fix Resource.__eq__ mismatch semantics of object equal https://review.openstack.org/251836 | 08:47 |
| *** openstackgerrit has quit IRC | 08:48 | |
| *** openstackgerrit has joined #openstack-ironic | 08:48 | |
| aarefiev | morning all! | 08:55 |
| *** athomas has joined #openstack-ironic | 09:01 | |
| *** _degorenko|afk is now known as degorenko | 09:02 | |
| *** mbound has joined #openstack-ironic | 09:06 | |
| *** jcoufal has joined #openstack-ironic | 09:10 | |
| *** ndipanov has quit IRC | 09:11 | |
| openstackgerrit | Merged openstack/ironic: Set node last_error in TaskManager https://review.openstack.org/273992 | 09:12 |
| *** amotoki has quit IRC | 09:14 | |
| *** jistr has joined #openstack-ironic | 09:14 | |
| *** amotoki has joined #openstack-ironic | 09:19 | |
| lucasagomes | morning all :-) | 09:21 |
| *** e0ne has joined #openstack-ironic | 09:28 | |
| *** keedya has quit IRC | 09:29 | |
| openstackgerrit | Merged openstack/ironic: Add db api layer for CRUD operations on node tags https://review.openstack.org/232598 | 09:32 |
| *** dtantsur|afk is now known as dtantsur | 09:33 | |
| dtantsur | Morning Ironic | 09:33 |
| aarefiev | lucasagomes, dtantsur g'morning | 09:34 |
| lucasagomes | aarefiev, o/ | 09:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [devstack] set ipa-debug=1 for greater debugability https://review.openstack.org/286015 | 09:35 |
| aarefiev | dtantsur: hey, will you against logging conf options at startup in inspector? | 09:37 |
| aarefiev | as it does for ironic, it helps sometimes | 09:37 |
| dtantsur | aarefiev, not at all (provided they're logged on DEBUG level just as with Ironic) | 09:37 |
| dtantsur | feel free to propose the patch | 09:37 |
| aarefiev | dtantsur: cool, yep only on DEBUG, no options :) | 09:38 |
| *** alineb_ has joined #openstack-ironic | 09:38 | |
| *** e0ne has quit IRC | 09:38 | |
| *** derekh has joined #openstack-ironic | 09:39 | |
| *** hoangcx has joined #openstack-ironic | 09:40 | |
| *** mkovacik has joined #openstack-ironic | 09:42 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Logging configuration options at startup https://review.openstack.org/286472 | 09:45 |
| dtantsur | that was fast :) | 09:45 |
| openstackgerrit | Moshe Levi proposed openstack/ironic: Adding InfiniBand Support https://review.openstack.org/264263 | 09:47 |
| *** mgould has joined #openstack-ironic | 09:47 | |
| *** aarefiev has quit IRC | 09:48 | |
| *** aarefiev has joined #openstack-ironic | 09:49 | |
| aarefiev | it adds only 2 short lines :) | 09:51 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Logging configuration options at startup https://review.openstack.org/286472 | 09:56 |
| lucasagomes | is it only here or gerrit is very slow today? | 09:57 |
| aarefiev | lucasagomes: yeah, it's very slow from yesterday | 09:59 |
| lucasagomes | :-( | 09:59 |
| openstackgerrit | Kan proposed openstack/ironic: API to list nodes using the same driver https://review.openstack.org/264005 | 10:00 |
| mgould | morning Ironic! | 10:05 |
| dtantsur | lucasagomes, very slow, I'd say | 10:06 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Split ironic-related functions from utils to separate common.ironic module https://review.openstack.org/286184 | 10:07 |
| *** ifarkas has left #openstack-ironic | 10:08 | |
| *** ifarkas has joined #openstack-ironic | 10:09 | |
| *** e0ne has joined #openstack-ironic | 10:11 | |
| lucasagomes | :-( | 10:15 |
| openstackgerrit | Vaishnavi proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 10:25 |
| sambetts|afk | Morning all o/ | 10:26 |
| *** sambetts|afk is now known as sambetts | 10:26 | |
| dtantsur | morning sambetts | 10:26 |
| *** ishant has quit IRC | 10:28 | |
| *** ndipanov has joined #openstack-ironic | 10:29 | |
| sambetts | Oh wow gerrit is being super slow today :( | 10:30 |
| *** lucasagomes has quit IRC | 10:30 | |
| *** alineb_ has quit IRC | 10:31 | |
| *** alineb has joined #openstack-ironic | 10:31 | |
| *** lucasagomes has joined #openstack-ironic | 10:31 | |
| *** lucasagomes has quit IRC | 10:32 | |
| *** lucasagomes has joined #openstack-ironic | 10:32 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to poor performance | 10:36 | |
| *** ChanServ changes topic to "Gerrit is going to be restarted due to poor performance" | 10:36 | |
| sambetts | ++ | 10:36 |
| *** daemontool has joined #openstack-ironic | 10:36 | |
| *** lucasagomes has quit IRC | 10:37 | |
| *** lucasagomes has joined #openstack-ironic | 10:37 | |
| *** daemontool__ has quit IRC | 10:40 | |
| *** lucasagomes has quit IRC | 10:41 | |
| *** 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" | 10:42 | |
| -openstackstatus- NOTICE: gerrit finished restartign | 10:42 | |
| *** lucasagomes has joined #openstack-ironic | 10:43 | |
| *** phuongnh has quit IRC | 10:45 | |
| openstackgerrit | Béla Vancsics proposed openstack/ironic: Use assertEqual/Greater/Less/IsNone https://review.openstack.org/286494 | 10:54 |
| *** vaishnavi_r has quit IRC | 10:55 | |
| *** chlong_ has joined #openstack-ironic | 11:02 | |
| *** dims has joined #openstack-ironic | 11:03 | |
| *** hoangcx has quit IRC | 11:06 | |
| *** amotoki has quit IRC | 11:06 | |
| *** caiobo has quit IRC | 11:09 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic client https://review.openstack.org/286510 | 11:22 |
| aarefiev | sambetts: morning | 11:24 |
| aarefiev | dtantsur, sambetts, guys, what do you think about set up new ipmi cred during discovery(via rules?) | 11:25 |
| dtantsur | aarefiev, I thought about it, would be good probably to have a separate spec, as it might have interesting consequences | 11:26 |
| aarefiev | dtantsur: yeah, a lot of question here, I'll try to gather some thoughts in spec | 11:29 |
| sambetts | Its an interesting idea, I'd be a bit wary if we started making rules be able to take actions on the node itself :/ | 11:31 |
| aarefiev | sambetts: not necessary do this from rules | 11:34 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Do not manage power on nodes in 'enroll' state https://review.openstack.org/286514 | 11:34 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Add CMDB support https://review.openstack.org/286515 | 11:34 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port https://review.openstack.org/240944 | 11:35 |
| dtantsur | pas-ha, any plans to update the CMDB spec btw? https://review.openstack.org/253054 | 11:37 |
| pas-ha | yeah, just going back to it now | 11:37 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Port https://review.openstack.org/240944 | 11:38 |
| dtantsur | ifarkas, sambetts, hey guys, there are quite a few inspector patches up, some with 1x +2 already, could you please help cleaning the review queue? :) https://review.openstack.org/#/q/project:openstack/ironic-inspector+status:open | 11:41 |
| dtantsur | we should finish the whole feature work this week | 11:41 |
| sambetts | Sure, I'll run through them in just a moment :) | 11:41 |
| ifarkas | dtantsur, I was looking at them but a few had a +2 and an unanswered comment | 11:44 |
| ifarkas | eg https://review.openstack.org/#/c/279135/ | 11:44 |
| dtantsur | ifarkas, I can follow-up with nit-fixing | 11:44 |
| ifarkas | dtantsur, works for me | 11:45 |
| vinm213 | lucasagomes, regarding https://review.openstack.org/#/c/246036 | 11:54 |
| vinm213 | i have seen you comments | 11:54 |
| *** shu-mutou is now known as shu-mutou-AFK | 11:54 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Follow up nits of 3429e3824c060071e59a117c19c95659c78e4c8b https://review.openstack.org/286364 | 11:55 |
| lucasagomes | vinm213, hi there, yes? | 11:55 |
| vinm213 | But my doubt here is, we actually want to create the RequestContext object for user='admin' and tenant='admin' | 11:55 |
| vinm213 | where as get_admin_context() creates them with None, None | 11:56 |
| vinm213 | I don't have much understanding into what oslo's context does. | 11:57 |
| lucasagomes | vinm213, ok that's fine, but I don't see any explanation in the patch or bug why the user has to be "admin" ? | 11:57 |
| lucasagomes | vinm213, me neither :-/ | 11:57 |
| lucasagomes | it even surprises me, how did that code worked before? And the tests didn't fail | 11:57 |
| vinm213 | hmm... | 11:58 |
| *** amotoki has joined #openstack-ironic | 11:58 | |
| lucasagomes | vinm213, so I'm not against having the "admin" but we need to iron out the why's | 11:59 |
| vinm213 | ok..so it's ok to have an object with default parameters. | 11:59 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add discover nodes exercise https://review.openstack.org/276107 | 11:59 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add enroll_node_not_found hook https://review.openstack.org/281312 | 11:59 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Negative tests for testing actions with port. https://review.openstack.org/259354 | 12:00 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add discover nodes exercise https://review.openstack.org/276107 | 12:00 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add enroll_node_not_found hook https://review.openstack.org/281312 | 12:00 |
| lucasagomes | JayF, we haven't merged a patch of yours that prevented the node to be powered off if the deployment failed? Or I made that up in my head? | 12:03 |
| sinval | morning Ironicers | 12:04 |
| lucasagomes | sinh, morning | 12:04 |
| lucasagomes | ops... sinval ^ | 12:06 |
| sinval | lucasagomes, o/ | 12:06 |
| *** daemontool_ has joined #openstack-ironic | 12:07 | |
| *** amotoki has quit IRC | 12:07 | |
| *** daemontool has quit IRC | 12:10 | |
| *** raildo-afk is now known as raildo | 12:18 | |
| *** vinm213 has quit IRC | 12:24 | |
| openstackgerrit | Merged openstack/ironic-inspector: Logging configuration options at startup https://review.openstack.org/286472 | 12:25 |
| openstackgerrit | Merged openstack/ironic-inspector: Stop storing ramdisk logs with the introspection data https://review.openstack.org/279135 | 12:28 |
| *** daemontool__ has joined #openstack-ironic | 12:34 | |
| deray | g'morning all :) | 12:36 |
| sinval | morning deray | 12:37 |
| *** daemontool_ has quit IRC | 12:37 | |
| deray | sinval, o/ | 12:37 |
| TheJulia | Good morning | 12:38 |
| deray | TheJulia, g'morning | 12:38 |
| sambetts | o/ TheJulia | 12:39 |
| sinval | morning TheJulia sambetts | 12:39 |
| * deray feels there's less chance of firmware update (manual cleaning step) to get in this release | 12:42 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost: Add script and playbook to test dhcp https://review.openstack.org/276177 | 12:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Clarify that we don't maintain compatibility for stored data https://review.openstack.org/286556 | 12:46 |
| dtantsur | deray, I'm afraid so :( we have too many things to finish this week | 12:46 |
| dtantsur | and morning sinval, deray, TheJulia | 12:47 |
| sinval | o/ dtantsur | 12:47 |
| deray | dtantsur, g'morning.. | 12:47 |
| deray | Is this (https://review.openstack.org/#/c/278566) one of the items which requires immediate attention? | 12:48 |
| dtantsur | aarefiev, +A on discovery patch :) | 12:49 |
| dtantsur | (provided that tests pass, heh :) | 12:49 |
| dtantsur | deray, yep | 12:49 |
| deray | dtantsur, yeah.. i agree | 12:49 |
| openstackgerrit | Merged openstack/ironic: Use assertEqual/Greater/Less/IsNone https://review.openstack.org/286494 | 12:51 |
| openstackgerrit | Merged openstack/ironic-inspector: Add invert option to rule conditions https://review.openstack.org/280603 | 12:51 |
| sambetts | \o/ all the mergy merges! | 12:51 |
| *** amotoki has joined #openstack-ironic | 12:52 | |
| *** baoli has joined #openstack-ironic | 12:54 | |
| aarefiev | Yippee! | 12:55 |
| aarefiev | :) | 12:55 |
| deray | dtantsur, will try then to close in on that.. | 12:56 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector-specs: Sync auto-discovery spec with impl https://review.openstack.org/286567 | 12:57 |
| aarefiev | sambetts: could you go over https://review.openstack.org/#/c/286567 when have time | 12:59 |
| sinval | hey sambetts, check this out https://review.openstack.org/286192 dynalloc for OneView drivers | 12:59 |
| * sambetts is popular | 12:59 | |
| aarefiev | today sambetts is popular | 13:00 |
| aarefiev | :) | 13:00 |
| sambetts | I'll add them to my list :D | 13:00 |
| TheJulia | sambetts: just remember that it is okay hide from people too :) | 13:00 |
| *** kalpase has left #openstack-ironic | 13:00 | |
| sambetts | :-P | 13:00 |
| TheJulia | as in "I need to get some work done" | 13:00 |
| sambetts | TheJulia: haha yes :-P these give me something to do while I'll waiting for devstack | 13:00 |
| sambetts | s/I'll/I'm/ | 13:01 |
| TheJulia | :) | 13:01 |
| *** dims has quit IRC | 13:02 | |
| *** mgoddard_ has joined #openstack-ironic | 13:04 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add db migrations tests https://review.openstack.org/284792 | 13:05 |
| *** ndipanov_ has joined #openstack-ironic | 13:05 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [devstack] Improve virtual machines logs https://review.openstack.org/285412 | 13:06 |
| dtantsur | sambetts, should work ^^ but lets see | 13:06 |
| *** ndipanov has quit IRC | 13:06 | |
| sambetts | dtantsur: thanks :D | 13:07 |
| *** mgoddard has quit IRC | 13:07 | |
| *** links has quit IRC | 13:07 | |
| *** dims has joined #openstack-ironic | 13:08 | |
| *** smoriya_ has quit IRC | 13:11 | |
| *** lucasagomes is now known as lucas-hungry | 13:15 | |
| *** appu103_ has quit IRC | 13:19 | |
| *** dprince has joined #openstack-ironic | 13:25 | |
| *** mgoddard_ has quit IRC | 13:26 | |
| *** mgoddard_ has joined #openstack-ironic | 13:27 | |
| trown | dtantsur: do the failures here look related to the patch? https://review.openstack.org/#/c/286069/ | 13:27 |
| trown | dtantsur: I just see a timeout when trying to power on a node in one tempest test | 13:27 |
| * dtantsur checks | 13:28 | |
| dtantsur | trown, I suspect something wrong with Liberty gate, another patch also fails all the jobs | 13:29 |
| *** mjturek1 has quit IRC | 13:30 | |
| trown | dtantsur: ok, I will come back to those patches later | 13:30 |
| openstackgerrit | Moshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support https://review.openstack.org/264257 | 13:31 |
| *** slagle has joined #openstack-ironic | 13:33 | |
| *** mjturek1 has joined #openstack-ironic | 13:35 | |
| *** ndipanov_ is now known as ndipanov | 13:37 | |
| aarefiev | dtantsur: is it possible to merge auto-discovery job before exercise script? | 13:39 |
| *** baoli has quit IRC | 13:40 | |
| aarefiev | sambetts: thanks for review | 13:40 |
| dtantsur | aarefiev, if it's non-voting - yes (at least no objections from me) | 13:40 |
| *** baoli has joined #openstack-ironic | 13:40 | |
| aarefiev | dtantsur: ok, lets try, I'll rebase infra patch | 13:41 |
| *** baoli has quit IRC | 13:43 | |
| *** baoli has joined #openstack-ironic | 13:43 | |
| *** Marga_ has joined #openstack-ironic | 13:46 | |
| *** Marga_ has quit IRC | 13:50 | |
| *** sinval has quit IRC | 13:53 | |
| *** mbound has quit IRC | 13:56 | |
| *** sinval has joined #openstack-ironic | 14:00 | |
| *** derekh is now known as ndipanov_ | 14:04 | |
| *** ndipanov_ is now known as derekh | 14:04 | |
| *** cdearborn has joined #openstack-ironic | 14:06 | |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/275722 | 14:07 |
| sambetts | aarefiev, dtantsur: it would be good to get the nv job merged before the actual exercise so we can see if it works :) | 14:07 |
| dtantsur | yeah | 14:07 |
| aarefiev | sambetts: yeah, it's my plan | 14:08 |
| sambetts | :D coolio | 14:08 |
| *** baoli has quit IRC | 14:08 | |
| openstackgerrit | Merged openstack/ironic-inspector: Add enroll_node_not_found hook https://review.openstack.org/281312 | 14:09 |
| *** lucas-hungry is now known as lucasagomes | 14:09 | |
| dtantsur | \o/ | 14:09 |
| sambetts | \o/ !! | 14:09 |
| *** baoli has joined #openstack-ironic | 14:12 | |
| deray | vdrok_, yuriyz got a chance to look at the new patch set of https://review.openstack.org/#/c/211019/ | 14:13 |
| *** thiagop has joined #openstack-ironic | 14:14 | |
| thiagop | Good morning, Ironic | 14:14 |
| mkovacik | dtantsur, w/r the --dhcp-sequential-ip I've not yet tested that, though | 14:14 |
| yuriyz | deray ok will try to look today | 14:14 |
| dtantsur | mkovacik, let the gate test :D | 14:14 |
| deray | yuriyz, sure.. tyt | 14:14 |
| *** baoli has quit IRC | 14:15 | |
| mkovacik | dtantsur, sounds like broken gate :D | 14:15 |
| *** mbound has joined #openstack-ironic | 14:15 | |
| * mkovacik hopes dnsmasq rotates the IP eventually | 14:17 | |
| *** baoli has joined #openstack-ironic | 14:22 | |
| *** baoli has quit IRC | 14:23 | |
| *** baoli has joined #openstack-ironic | 14:25 | |
| *** jroll has quit IRC | 14:29 | |
| *** jroll has joined #openstack-ironic | 14:30 | |
| *** jroll has quit IRC | 14:30 | |
| *** jroll has joined #openstack-ironic | 14:30 | |
| *** mbound has quit IRC | 14:30 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector-specs: CMDB integration https://review.openstack.org/253054 | 14:33 |
| *** mbound has joined #openstack-ironic | 14:34 | |
| *** mgould has quit IRC | 14:35 | |
| *** baoli has quit IRC | 14:39 | |
| *** d0ugal has quit IRC | 14:39 | |
| *** d0ugal has joined #openstack-ironic | 14:40 | |
| *** d0ugal has quit IRC | 14:40 | |
| *** d0ugal has joined #openstack-ironic | 14:41 | |
| *** baoli has joined #openstack-ironic | 14:41 | |
| *** mgould has joined #openstack-ironic | 14:50 | |
| *** sinval is now known as sinval-afk | 14:50 | |
| deray | jroll, dtantsur lucasagomes there is also a small patch related to ilo license activation as a amanual cleaning step.. | 14:52 |
| deray | if possible can you have a look? https://review.openstack.org/#/c/259921 | 14:53 |
| *** daemontool_ has joined #openstack-ironic | 14:53 | |
| *** harshs has joined #openstack-ironic | 14:55 | |
| *** daemontool__ has quit IRC | 14:56 | |
| *** kan_ has quit IRC | 14:56 | |
| *** e0ne has quit IRC | 14:56 | |
| deray | okay .. g'day ahead folks | 14:56 |
| *** deray has quit IRC | 14:57 | |
| *** [1]cdearborn has joined #openstack-ironic | 14:57 | |
| lucasagomes | dera... | 15:01 |
| *** harshs has quit IRC | 15:02 | |
| *** spandhe has joined #openstack-ironic | 15:03 | |
| *** spandhe has quit IRC | 15:03 | |
| *** rbudden has joined #openstack-ironic | 15:04 | |
| *** jaybeale has joined #openstack-ironic | 15:06 | |
| *** Goneri has joined #openstack-ironic | 15:10 | |
| *** cdearborn has quit IRC | 15:11 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Split ironic-related functions from utils to separate common.ironic module https://review.openstack.org/286184 | 15:13 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: [devstack] Improve virtual machines logs https://review.openstack.org/285412 | 15:15 |
| sambetts | dtantsur: I had to rebase that ^ , I derped a +W'd it | 15:16 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-lib: use wipefs to erase FS meta information https://review.openstack.org/286283 | 15:16 |
| * jlvillal is happy that after about a total of 6-8 hours (including time over the weekend) he finally figured out how to get VLANs working on his Mikrotik Cloud Router Switch | 15:16 | |
| dtantsur | sambetts, it's usually enough to just remove +A :) anyway I hope ifarkas will +A it for us ;) | 15:17 |
| sambetts | dtantsur: ah, i thought only a rebase would stop it after a +A | 15:17 |
| jlvillal | Note to others, only create ONE master port if you want VLANs. Mikrotik needs to work on making their stuff easier to use!!! | 15:17 |
| sambetts | jlvillal: ? | 15:17 |
| ifarkas | sambetts, dtantsur, sure thing, looks good ;-) | 15:18 |
| jlvillal | sambetts, They probably make Cisco look easy to use :P | 15:18 |
| dtantsur | lol | 15:19 |
| jlvillal | sambetts, It is a cool device for the money. 24 port L2 switch with some L3 routing capabilities possible (but performance for the L3 stuff is not great). Cost was $179. | 15:19 |
| sambetts | jlvillal: haha, I've only ever used dumb switches outside of work, although I've been thinking about upgrading my home network | 15:21 |
| jlvillal | sambetts, Yeah that is what this is for. I want to setup a guest wireless on a separate VLAN and isolate it from my home network. Mostly for fun. | 15:21 |
| * jlvillal has a strange idea of fun | 15:21 | |
| * thiagop agrees | 15:22 | |
| sambetts | jlvillal: its not that strange I love tinkering that stuff at home too :-P | 15:22 |
| jlvillal | I was thinking, why does my WiFi enabled scale need to be on my main home WiFi network. Or my Nest Thermostat or various other things. | 15:22 |
| jlvillal | sambetts, It was very frustrating when it didn't work. But it did make it much sweeter when I finally figured it out :) | 15:23 |
| jlvillal | thiagop, Did you finish your DS-160. I remember doing it with my wife and it was a LONG form!! | 15:23 |
| thiagop | jlvillal: yes, I did. I'm only waiting for the invitation letter of the OSF to schedule my visa interview | 15:24 |
| *** daemontool__ has joined #openstack-ironic | 15:24 | |
| sambetts | jlvillal: it always is :-P I ran into annoying issues with my 5Ghz wifi recently and that took ages to work out | 15:24 |
| jlvillal | thiagop, Sweet! | 15:24 |
| *** jaybeale has quit IRC | 15:24 | |
| thiagop | jlvillal: painful, but very funny too | 15:24 |
| jlvillal | sambetts, But you felt good when you solved it! | 15:24 |
| dtantsur | thiagop, oh yeah, a lot of fun :) | 15:25 |
| dtantsur | will you need an interview? | 15:25 |
| *** raddaoui has joined #openstack-ironic | 15:25 | |
| jlvillal | thiagop, The history questions were tough. Trying to figure every place my wife had lived since age 16 :( | 15:25 |
| thiagop | dtantsur: yes, we brazillians need | 15:26 |
| dtantsur | jlvillal, remembering school etc... | 15:26 |
| thiagop | jlvillal: tough was to discover the date I started my undergrad. I don't have it in any documents... haha | 15:27 |
| *** mtanino has joined #openstack-ironic | 15:27 | |
| jlvillal | dtantsur, My wife has moved a fair amount over the last 20+ years. I know that would be tough for me to figure out. | 15:27 |
| jlvillal | A lot easier if you are in your 20s... | 15:28 |
| *** daemontool_ has quit IRC | 15:28 | |
| thiagop | but those questions about organs traffic made me laugh 10min | 15:28 |
| dtantsur | thiagop++ | 15:28 |
| sambetts | man... the benefits of being a UK citizen are insane when it comes to travel... | 15:29 |
| dtantsur | yep :( | 15:30 |
| dtantsur | well, US, EU and UK probably only need a visa to Russia and North Korea :D | 15:30 |
| jlvillal | sambetts, Yep US and UK citizens have it nice for the most part in not needing visas for many countries. | 15:31 |
| sambetts | jlvillal: although I have to have a travel visa thingy for the US :-P | 15:32 |
| sambetts | jlvillal: but its literally 1 web form | 15:32 |
| jlvillal | Really? I thought it was the Visa Waiver Program | 15:33 |
| jlvillal | Ah, so a sort of pre-screen thing. | 15:33 |
| sambetts | jlvillal: its called an ESTA whatever that stands for | 15:33 |
| jlvillal | sambetts, Electronic System for Travel Authorization | 15:34 |
| sambetts | yeah that :-P | 15:34 |
| jlvillal | That is a lot easier than a visa. | 15:34 |
| sambetts | yup | 15:35 |
| * jlvillal needs to go get ready to go into work... | 15:35 | |
| sambetts | I had to go through a full visa application for the US a couple of years ago when I lived in san jose for a year | 15:35 |
| *** mbound has quit IRC | 15:40 | |
| *** mbound has joined #openstack-ironic | 15:40 | |
| lucasagomes | sambetts, ++ for ESTA | 15:40 |
| *** keedya has joined #openstack-ironic | 15:40 | |
| keedya | Good morning All!!! | 15:41 |
| TheJulia | good morning | 15:41 |
| lucasagomes | the questions are quite funny/straight forward tho, e.g: "Are you a terrorist? (Yes/No)" | 15:41 |
| sambetts | lucasagomes: ahaha yes | 15:41 |
| lucasagomes | keedya, TheJulia morning | 15:45 |
| *** amotoki has quit IRC | 15:46 | |
| thiagop | morning keedya TheJulia | 15:46 |
| keedya | 0/ | 15:47 |
| thiagop | lucasagomes: there was one that made me think | 15:47 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 15:47 |
| krotscheck | Anyone here super crazy familiar with how ironic does configgen? | 15:47 |
| thiagop | it versed about "extrajudicial killing". | 15:47 |
| devananda | morning, all | 15:48 |
| NobodyCam | Good morning Ironicers | 15:48 |
| thiagop | morning devananda NobodyCam | 15:48 |
| lucasagomes | thiagop, ouch | 15:48 |
| lucasagomes | devananda, NobodyCam morning | 15:48 |
| NobodyCam | morning :) | 15:48 |
| thiagop | lucasagomes: that's weird. If I'm authorized to kill by a judge, is it ok then?! | 15:48 |
| sambetts | o/ devananda | 15:48 |
| lucasagomes | thiagop, heh I guess if you work to the government in some cases that may be applicable | 15:49 |
| lucasagomes | e.g, death penalty | 15:50 |
| lucasagomes | work for* | 15:50 |
| thiagop | juditially ok, but humanly it is kinda ugly | 15:51 |
| *** jaybeale has joined #openstack-ironic | 15:51 | |
| lucasagomes | agree, not very ethical | 15:51 |
| devananda | dtantsur: china, brazil, india (up until this year). not many, just a few big ones ;) | 15:51 |
| dtantsur | :) | 15:52 |
| devananda | krotscheck: you mean tools/config/generate_sample.py ? | 15:52 |
| dtantsur | morning devananda, keedya, NobodyCam | 15:52 |
| NobodyCam | morning dtantsur | 15:52 |
| krotscheck | devananda: Yep | 15:53 |
| krotscheck | devananda: oslo.config's generate-config just added the ability to override default values for the output config, and... well, it seems like Ironic's not using it at all. | 15:53 |
| devananda | krotscheck: then yes :) | 15:53 |
| NobodyCam | sorry was reading all the emails I have, didn't do the full good morning today | 15:53 |
| devananda | krotscheck: oh, interesting. I wasn't aware of that. you're probably right - we probably raen't using that | 15:53 |
| krotscheck | I have a mitaka blocker bug that may force me to shave that yak. | 15:54 |
| krotscheck | If it's something the team wants. | 15:54 |
| devananda | dtantsur: are there any CLI's that I can use to talk to ironic-inspector besides openstackclient? | 15:54 |
| dtantsur | devananda, nope.. we don't have a dedicated CLI like ironic | 15:55 |
| devananda | dtantsur: hrmm... ok. I'm having difficulty using openstackclient *without* keystone | 15:55 |
| devananda | since I want to use inspector inside of a bifrost env, there is no keystone | 15:56 |
| dtantsur | devananda, I thought it's possible, to be honest, but not sure how | 15:56 |
| * dtantsur researches | 15:56 | |
| dtantsur | devananda, openstack --os-auth-type token_endpoint --os-token fake --os-url http://127.0.0.1:5050 | 15:58 |
| dtantsur | maybe something like that? | 15:58 |
| devananda | dtantsur: I've tried that, but osc is failing. maybe a bug? I dunno. | 15:58 |
| krotscheck | devananda: Is there any reason why I cannot make that switch? | 15:58 |
| devananda | krotscheck: I am unclear on exactly what you're proposing to change | 16:00 |
| dtantsur | devananda, weird, it works for now, but crashes for inspector-client.. I'll take a look | 16:00 |
| krotscheck | devananda: Get rid of generate_sample.py, use oslo's generator instead? | 16:00 |
| devananda | krotscheck: oh! nope, we tried that. it's terrible | 16:01 |
| devananda | krotscheck: more specifically, it makes assumptions about things that are not valid for Ironic | 16:02 |
| krotscheck | devananda: Such as? | 16:02 |
| devananda | krotscheck: we decided not to refactor dozens of files and change where and how we define config options | 16:02 |
| devananda | just because oslo introduced a new way to do it | 16:02 |
| devananda | lucasagomes: IIRC< I believe you did the prototyping work on ^ before we agreed not to go that path? | 16:02 |
| devananda | dtantsur: thanks! | 16:03 |
| krotscheck | devananda: Ok, so, the snowflake argument. | 16:03 |
| devananda | krotscheck: no, the "it works fine the way it is" argument | 16:03 |
| * lucasagomes reads | 16:03 | |
| devananda | krotscheck: is there a compelling reason for us to go through all that work? | 16:03 |
| lucasagomes | devananda, krotscheck oh yes we did that before | 16:04 |
| lucasagomes | devananda, krotscheck the reason we didn't like it was because we had to put _all_ options in a file, and keep maintaing it every time a new option was added | 16:04 |
| lucasagomes | AFAIR | 16:04 |
| krotscheck | devananda: I have a mitaka-blocker bug that says that I need to A) make sure that no cors headers be hardcoded, B) that those same headers need to be set as the config defaults via oslo's new config override, and c) that those same default headers be expressed in the generated config. | 16:04 |
| krotscheck | C) is handled by the new genconfig hooks in oslo. | 16:05 |
| jlvillal | sambetts, dtantsur Funny I open up cnn.com and see this :) http://www.cnn.com/2016/03/01/travel/worlds-best-worst-passports/index.html | 16:05 |
| krotscheck | B) Is handled via set_dfaults, which I already have working. | 16:05 |
| krotscheck | A) is just deleting a few lines from app.py | 16:05 |
| sambetts | jlvillal: haha | 16:06 |
| krotscheck | I simply do not have the time to backport the necessary code from oslo configgen into ironic's generator. | 16:06 |
| devananda | krotscheck: is it possible for C to be done without deleting generate_sample.py | 16:06 |
| *** suro-patz has joined #openstack-ironic | 16:06 | |
| krotscheck | devananda: I have 23 patches to get working. | 16:06 |
| krotscheck | Anything is possible. Not in my timeframe. | 16:06 |
| devananda | krotscheck: removing generate_sample.py and using oslo's configgen will take a lot of work on our side, too | 16:07 |
| devananda | it's not just "start using configgen" | 16:07 |
| dims | devananda : there's a set of reasons for doing what we did, here it is - https://specs.openstack.org/openstack/oslo-specs/specs/juno/oslo-config-generator.html | 16:07 |
| krotscheck | devananda: So what's the best solution? | 16:07 |
| devananda | and it will introduce ongoing maintenance burden on our reviewers and devs | 16:07 |
| lucasagomes | krotscheck, devananda https://review.openstack.org/#/c/247331/ | 16:08 |
| lucasagomes | tanlin also had a patch for oslo.config generation | 16:08 |
| lucasagomes | and this was the original one: https://review.openstack.org/#/c/137447/ | 16:08 |
| lucasagomes | it's huge and messy | 16:09 |
| lucasagomes | IMHO, I dunno why oslo.config generation was created the way it currently is | 16:09 |
| dims | lucasagomes : see spec above | 16:09 |
| lucasagomes | some people pointed that it's because you can split in more than one sample file | 16:09 |
| lucasagomes | dims, lemme read that | 16:10 |
| devananda | dims: I understand the reasoning, but it would require a significant refactoring of our code, and either A) change the way (where and how) we define config options or B) require listing dozens of entry points | 16:10 |
| dims | lucasagomes : that's not the only reason, the old one did not work in many situations | 16:10 |
| *** chlong_ has quit IRC | 16:10 | |
| openstackgerrit | Monty Taylor proposed openstack/python-ironicclient: Do not pass endpoint to constructor in OSC https://review.openstack.org/286681 | 16:10 |
| dims | devananda : that's fair, please don't say "just because oslo introduced a new way to do it " | 16:10 |
| dims | devananda : not trying to create work here | 16:11 |
| dims | trying to fix problems we see | 16:11 |
| krotscheck | devananda: Most of the other projects have adopted it without too much of a dev overhead or gnashing of teeth. I don't think your argument has teeth. | 16:11 |
| * dims back in a bit | 16:12 | |
| *** max_lobur has quit IRC | 16:12 | |
| lucasagomes | dims, interesting... The thing that annoys me was that we had the script to generate the sample for a long time in the incubator | 16:12 |
| lucasagomes | but once it graduated it was completely redesigned | 16:13 |
| lucasagomes | I mean, why incubate something for so long and then just rewrite it at graduation time? | 16:13 |
| krotscheck | lucasagomes: Seems like that's water under the bridge now, no? | 16:13 |
| devananda | dims: full context there was that we looked at what oslo did, it didn't work for us, and "oslo introduced a new way to do X" isn't (in the absence of a compelling reason in Ironic, and in the presense of significant work) a compelling reason | 16:13 |
| lucasagomes | but yeah, reading the spec I understand it better | 16:13 |
| *** e0ne has joined #openstack-ironic | 16:13 | |
| lucasagomes | krotscheck, seems like the wrong way of doing it | 16:13 |
| lucasagomes | and we haven't adopted the new way because of that | 16:14 |
| devananda | lucasagomes: I agree, it's strange that it changed when it graduated. not the way oslo libs have been handled in general | 16:14 |
| krotscheck | lucasagomes: I don't disagree. But that's what happened. | 16:14 |
| lucasagomes | it's bad architected | 16:14 |
| *** jaypipes has joined #openstack-ironic | 16:14 | |
| devananda | krotscheck: I'm not sure what you mean by "it's water under the bridge" | 16:14 |
| krotscheck | lucasagomes: I'm sure the port of seattle would like to extract the billion dollar tunnel boring machine that's stuck under the city too, but fact is the thing can only move forward. | 16:15 |
| devananda | krotscheck: that argument does not apply here. this is code, not a billion dollar physical machine that can't go in reverse | 16:15 |
| keedya | morning dtanstur 0/ | 16:15 |
| krotscheck | devananda: I'm saying that an argument about "This was handled poorly in the past" is not really an argument. The argument is about the present. | 16:15 |
| lucasagomes | krotscheck, I mean the script we have *works* for us so far... now you discover a problem with it | 16:16 |
| krotscheck | devananda: Ok, so, revert the config things in oslo? Or go back in time so things didn't happen the way you don't like it to happen? | 16:16 |
| devananda | krotscheck: and in the present, Ironic has a working config generator. and oslo decided to take an approach that, when we last looked at it, didn't fit | 16:16 |
| lucasagomes | and we have to deal with it, so we can argue either fixing the current script or moving to the new oslo.config | 16:16 |
| devananda | krotscheck: I'm saying, what's WRONG with what we are doing now? | 16:16 |
| devananda | it works FOR US | 16:16 |
| lucasagomes | I'm just saying the oslo.config one is disruptive and IMHO not very well designed | 16:16 |
| krotscheck | devananda: Does it work IN CAPS? | 16:17 |
| dtantsur | devananda, good news for you: we do have a bug and its fixed in master | 16:17 |
| devananda | krotscheck: totally :) | 16:17 |
| dtantsur | devananda, mind trying python-ironic-inspector-client from git master? | 16:17 |
| devananda | dtantsur: oh? great - I'll test | 16:17 |
| krotscheck | Ok, so, I don't have time for this argument, I have 22 other projects to deal with. | 16:17 |
| keedya | dtanstur: Back to our conversation yesterday, where do you think i can start? | 16:17 |
| krotscheck | I will file a mitaka-blocker bug against ironic and let y'all figure it out. | 16:17 |
| lucasagomes | krotscheck, fair enough, I think the bug is the way to go yes | 16:18 |
| devananda | krotscheck: ok. in that bug, please explain why our config generator is unable to do what you need. because I do not understand it yet | 16:18 |
| devananda | the only reason I heard from you was "i dont have time" | 16:18 |
| devananda | but I don't know what you dont have time to do | 16:18 |
| lucasagomes | but we need to discuss it with time and make the right decisions | 16:18 |
| lucasagomes | because your time != time for the project | 16:18 |
| devananda | if there is a compelling reason for Ironic to use oslo configgen -- ok, we'll discuss what that means for us. I just haven't seen it yet | 16:19 |
| dtantsur | it's pretty convenient, but it probably does not justify the whole refactoring.. | 16:22 |
| *** mmedvede has joined #openstack-ironic | 16:22 | |
| *** garthb has joined #openstack-ironic | 16:23 | |
| keedya | dtantsur: Back to our conversation yesterday, u mentionned integrating with CMDB. | 16:24 |
| keedya | is that in ironic inspector? | 16:24 |
| sambetts | that is something we are working towards in inspector | 16:24 |
| dtantsur | keedya, we're discussing it right now in https://review.openstack.org/253054 | 16:25 |
| dtantsur | would be great to have your input as well | 16:25 |
| dtantsur | yuriyz, anything preventing you from W+1 on https://review.openstack.org/#/c/286015/ ? | 16:26 |
| *** danimal has joined #openstack-ironic | 16:26 | |
| dims | devananda : it's funny, you do use the the good things made possible by oslo-config-generator (https://github.com/openstack/ironic/blob/master/ironic/common/config_generator/generator.py#L124) | 16:27 |
| *** danimal has quit IRC | 16:30 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 16:31 |
| openstackgerrit | Devananda van der Veen proposed openstack/python-ironicclient: Do not pass endpoint to constructor in OSC https://review.openstack.org/286681 | 16:32 |
| lucasagomes | dims, (-: yeah, ironic have been maintain that script, adding new things etc... | 16:32 |
| *** absubram has joined #openstack-ironic | 16:32 | |
| dims | lucasagomes : best of luck is all i can say :) | 16:33 |
| *** absubram_ has joined #openstack-ironic | 16:33 | |
| lucasagomes | dims, heh right | 16:34 |
| krotscheck | devananda: I need the generated configuration to include default overrides. | 16:35 |
| krotscheck | devananda: I've included a reference patch in the bug. | 16:36 |
| krotscheck | devananda: Good luck maintaining your own config script! | 16:36 |
| *** suro-patz has quit IRC | 16:37 | |
| *** absubram has quit IRC | 16:37 | |
| *** absubram_ is now known as absubram | 16:37 | |
| *** suro-patz has joined #openstack-ironic | 16:37 | |
| *** e0ne has quit IRC | 16:38 | |
| devananda | dtantsur: confirmed the version in git works. current release does not. | 16:39 |
| dtantsur | devananda, I hope the git master will become the current release tonight :) | 16:40 |
| devananda | dtantsur: :) | 16:40 |
| *** e0ne has joined #openstack-ironic | 16:40 | |
| *** mgoddard_ has quit IRC | 16:40 | |
| *** mgoddard has joined #openstack-ironic | 16:40 | |
| *** vinm213 has joined #openstack-ironic | 16:41 | |
| *** rbradfor has joined #openstack-ironic | 16:42 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-python-agent: iscsi: wipe part table before starting the target https://review.openstack.org/284347 | 16:42 |
| dtantsur | devananda, left you some debugging hints on https://bugs.launchpad.net/python-ironicclient/+bug/1551837 | 16:46 |
| openstack | Launchpad bug 1551837 in python-ironicclient "openstackclient generates traceback with AUTH=token_keystone" [Low,In progress] - Assigned to Devananda van der Veen (devananda) | 16:46 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/283983 | 16:51 |
| *** rcernin has quit IRC | 16:55 | |
| JayF | lucasagomes: that patch is sitting with a moldy -1. I hope to be able to refocus on upstream stuff in the next week or two and that'll be on the top of my list. | 16:55 |
| *** trevor___ has joined #openstack-ironic | 16:55 | |
| lucasagomes | JayF, oh right, no worries... It's because I had an item in my todo list to give it a go | 16:55 |
| lucasagomes | JayF, so I was setting up the env but couldn't find the configuration option | 16:55 |
| lucasagomes | I thought it was merged already | 16:55 |
| devananda | TheJulia: is there a flag to bifrost to tell it to install everything from git (instead of pip) ? | 16:56 |
| trevor___ | Hi everyone, I have a bug fix proposed in IPA but it needs the workflow approved. It hasn't been updated in 6 days, is there something I can do to get it looked at? | 16:57 |
| trevor___ | https://review.openstack.org/#/c/280970/ | 16:57 |
| dtantsur | JayF, ^^ | 16:57 |
| lucasagomes | trevor___, what you just did :-) yeah point it to us here in the channel | 16:57 |
| lucasagomes | it helps it to get attention | 16:57 |
| trevor___ | I thought so, thanks! | 16:58 |
| * lucasagomes looks at the patch | 16:58 | |
| lucasagomes | trevor___, np, sometimes we need to be reminded to do the review job (-: | 16:58 |
| * JayF +2a | 16:58 | |
| *** mgoddard_ has joined #openstack-ironic | 16:58 | |
| *** rloo has joined #openstack-ironic | 16:59 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-lib: use wipefs to erase FS meta information https://review.openstack.org/286283 | 17:00 |
| *** mbound has quit IRC | 17:01 | |
| *** mgoddard has quit IRC | 17:01 | |
| *** ifarkas has quit IRC | 17:05 | |
| *** jistr has quit IRC | 17:05 | |
| *** pcaruana has quit IRC | 17:07 | |
| Goneri | http://logs.openstack.org/83/286283/2/check/gate-tempest-dsvm-ironic-lib-src/3e05852/logs/screen-ir-cond.txt.gz | 17:10 |
| Goneri | I added a wipefs call in https://review.openstack.org/286283 | 17:10 |
| *** achanda has joined #openstack-ironic | 17:10 | |
| mkovacik | aarefiev, dtantsur, guys, do you prefer not saving the logs in the incoming data w/r introspection reapply patch? | 17:11 |
| mkovacik | in swift I mean | 17:11 |
| Goneri | should I also patch ironic itself to add wipefs in ironic-lib.filters | 17:11 |
| dtantsur | mkovacik, good call, we should store them in a file on receiving anyway | 17:11 |
| Goneri | I guess, it's a yes. | 17:12 |
| jroll | Goneri: I guess so, that's odd though | 17:12 |
| jroll | or maybe s/odd/sad/ | 17:12 |
| jroll | you know about depends-on btw? | 17:12 |
| *** stendulker has joined #openstack-ironic | 17:13 | |
| mkovacik | dtantsur, so you want the logs or not in the swift -incoming? ;) sorry, English parser overflow ;) | 17:13 |
| mkovacik | aarefiev doesn't | 17:13 |
| mkovacik | I'd keep | 17:13 |
| Goneri | jroll, yep I do. thanks | 17:13 |
| jroll | cool | 17:13 |
| dtantsur | mkovacik, kill them with fire :D actually, maybe we should just remove them in the ramdisk_error plugin right after saving | 17:13 |
| mkovacik | dtantsur, OK :D | 17:14 |
| *** MattMan has quit IRC | 17:14 | |
| *** trevor___ has quit IRC | 17:14 | |
| *** MattMan has joined #openstack-ironic | 17:14 | |
| lucasagomes | Goneri, very odd... you may have uncovered a bug there | 17:15 |
| lucasagomes | why the filters not being applied :-/ | 17:15 |
| dtantsur | devananda, python-ironic-inspector-client 1.5.0 released | 17:17 |
| *** dims_ has joined #openstack-ironic | 17:17 | |
| Goneri | lucasagomes, https://github.com/openstack/ironic/blob/053584c982d8f47862824f191e6e4e7663b132de/etc/ironic/rootwrap.d/ironic-lib.filters | 17:17 |
| lucasagomes | Goneri, yes, that shouldn't be there IMHO | 17:17 |
| Goneri | lucasagomes, this file is a copy/past of the same ironic-lib.filters from ironic-lib | 17:18 |
| *** raddaoui has quit IRC | 17:18 | |
| lucasagomes | Goneri, yeah... apparently devstack is not copying the ironic-lib one to the right place | 17:18 |
| Goneri | lucasagomes, the error message is confusing too. it says the call comes from ironic.drivers.modules.deploy_utils | 17:18 |
| *** dims has quit IRC | 17:19 | |
| Goneri | but it's actually in ironic_lib.disk_utils | 17:19 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up driver loading in init_host https://review.openstack.org/285850 | 17:19 |
| *** raddaoui has joined #openstack-ironic | 17:19 | |
| openstackgerrit | lokesh s proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 17:19 |
| lucasagomes | Goneri, yeah, something fishy there, these filters should exist in only one place | 17:19 |
| keedya | dtantsur: I read through the proposal and about the IPMI credentials. Is it possible to use the abstraction layer(CMDB) to run all the ipmi cmds? | 17:20 |
| lucasagomes | jroll, what do you think? Should we delete the one in ironic-lib and all the rootwrap filters in Ironic? | 17:20 |
| *** athomas has quit IRC | 17:21 | |
| lucasagomes | that way we don't break ppl, because if we get rid of the ironic one the ironic-lib will need to be copied to the right place too | 17:21 |
| dtantsur | keedya, depending on what exactly you want to do, I guess? inspector needs to boot the machine, so it needs the credentials in advance | 17:21 |
| jroll | lucasagomes: rloo was involved a bit in the rootwrap move stuff, maybe she knows why? | 17:22 |
| lucasagomes | jroll, right, yeah let's wait for her see if she has any thoughts about it | 17:22 |
| keedya | what if u excecute an API request to boot, set pxeboot, get power status ? | 17:22 |
| rloo | jroll, lucasagomes: what's the question? | 17:22 |
| lucasagomes | rloo, we now have 2 rootwrap filters for ironic-lib | 17:22 |
| lucasagomes | rloo, one living in the ironic tree the other one in the ironic-lib tree | 17:23 |
| lucasagomes | apparently, devstack only cares about the one in the ironic tree | 17:23 |
| lucasagomes | https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L352 | 17:23 |
| rloo | lucasagomes: there are diff filter files, right? | 17:23 |
| dtantsur | keedya, that's what ironic is responsible for, not inspector.. we only call ironic API's for that | 17:23 |
| lucasagomes | rloo, they are the same right now | 17:23 |
| * lucasagomes rechecks | 17:24 | |
| keedya | dtantsur: ok got it!! | 17:24 |
| rloo | lucasagomes: they are? sec, let me look. it has been awhile. | 17:24 |
| lucasagomes | rloo, https://github.com/openstack/ironic-lib/blob/master/etc/rootwrap.d/ironic-lib.filters and https://github.com/openstack/ironic/blob/053584c982d8f47862824f191e6e4e7663b132de/etc/ironic/rootwrap.d/ironic-lib.filters | 17:24 |
| lucasagomes | yeah same | 17:24 |
| *** raddaoui has quit IRC | 17:25 | |
| rloo | oh. i think we have to copy the ironic-lib.filters from ironic-lib, to ironic tree. | 17:25 |
| rloo | lucasagomes: that is why it is the same. | 17:25 |
| lucasagomes | rloo, yeah, that's because devstack is currently ignoring the one in ironic-lib | 17:25 |
| *** ohamada has quit IRC | 17:26 | |
| *** athomas has joined #openstack-ironic | 17:26 | |
| lucasagomes | rloo, I wonder how we can fix it tho... if we decide to remove the one from ironic and keep it only in ironic-lib we have to update devstack | 17:26 |
| rloo | lucasagomes: i think it was the easiest way i could figure out, to have things 'set up' to get access to that filter file. | 17:26 |
| lucasagomes | but this may also break other ppl I guess | 17:26 |
| lucasagomes | rloo, I think that for now we should remove the ironic-lib one and only use it in Ironic, this ofc will need to have the dependnecies set correctly when creating a patch in ironic-lib that depends on tools that are not in the filter | 17:27 |
| rloo | lucasagomes: is there a way to tell the rootwrap script where to get the filters? Or can we only give it the directory that contains all the filters? | 17:27 |
| openstackgerrit | Merged openstack/ironic-lib: Add support for choosing the disk label https://review.openstack.org/283532 | 17:28 |
| lucasagomes | rloo, not sure, lemme look... I think we need to point it to a dir with the filters | 17:28 |
| openstackgerrit | Lilia Sampaio proposed openstack/python-oneviewclient: Adding server profile application and removal operations https://review.openstack.org/258699 | 17:28 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for partition images in agent drivers https://review.openstack.org/162008 | 17:28 |
| Goneri | filters_path=/etc/ironic/rootwrap.d,/usr/share/ironic/rootwrap | 17:28 |
| Goneri | rloo, yes you can | 17:28 |
| lucasagomes | rloo, actually we can specificy multiples | 17:28 |
| lucasagomes | filters_path=/etc/ironic/rootwrap.d,/usr/share/ironic/rootwrap | 17:28 |
| lucasagomes | yup | 17:28 |
| dtantsur | going now, see you tomorrow | 17:29 |
| *** dtantsur is now known as dtantsur|afk | 17:29 | |
| Goneri | so we just need to add something like /usr/share/ironic_lib/rootwrap in the path | 17:29 |
| rloo | lucasagomes: the ironic-lib.filters belongs with ironic-lib, cuz that's where those calls are being made. | 17:29 |
| lucasagomes | dtantsur|afk, see you | 17:29 |
| lucasagomes | rloo, yeah | 17:29 |
| Goneri | rloo, lucasagomes I can do this patch since it blocks me. | 17:30 |
| rloo | lucasagomes: but if we only leave it in ironic-lib and not in ironic, upgrading will break folks cuz filters_path will be wrong. | 17:30 |
| lucasagomes | rloo, that's my fear | 17:30 |
| Goneri | good point | 17:31 |
| rloo | i didn't read far enough back. what's the problem? | 17:31 |
| *** degorenko is now known as _degorenko|afk | 17:31 | |
| lucasagomes | rloo, Goneri has a patch updating the filters in ironic-lib, but it's failing because that filter is ignored by devstack | 17:31 |
| rloo | lucasagomes: OH. presumably ironic is using the code in ironic lib that needs the filter. | 17:32 |
| * lucasagomes w00t disk label was merged!! | 17:32 | |
| lucasagomes | rloo, yeah, we thought it were | 17:33 |
| lucasagomes | rloo, but look https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L352 | 17:33 |
| lucasagomes | devstack is not copying the ironic-lib filters anywhere | 17:33 |
| mkovacik | aarefiev, looking at your comment in https://review.openstack.org/#/c/267637/10/ironic_inspector/process.py line #324 | 17:33 |
| rloo | right, i think we need to copy ironic-lib filter to ironic. i wonder if i left any notes anywhere about it. | 17:34 |
| mkovacik | aarefiev, I think to merge the two exceptions, cause the only think different is the log message, wdyt? | 17:34 |
| lucasagomes | rloo, yeah, but when? As part of the installation of the ironic-lib? Or devstack should manually copy it? | 17:34 |
| lucasagomes | cause I'm afraid we may break ppl if we remove the ironic-lib.filters from ironic now | 17:34 |
| rloo | lucasagomes: i think for now, just copy | 17:35 |
| lucasagomes | rloo, hmm... right | 17:35 |
| lucasagomes | I have to think about it | 17:36 |
| Goneri | rloo, ok, so I patch ironic-lib.filters file from ironic to add my new dep, and add a big fat warning to explain it's a copy of the one from ironic-lib | 17:36 |
| lucasagomes | rloo, cause if we change devstack to copy it that's fine for gate... but won't upgrading ironic-lib won't do that | 17:36 |
| lucasagomes | so it will break them | 17:36 |
| rloo | lucasagomes: oh. so it could get out of sync. | 17:37 |
| lucasagomes | rloo, yeah | 17:37 |
| *** trown is now known as trown|lunch | 17:37 | |
| rloo | copy in ironic should only be modified if the requirements change for the new ironic-lib | 17:37 |
| jlvillal | mgould: You around? | 17:38 |
| lucasagomes | rloo, I think we should remove the one from ironic-lib and use only ironic for now | 17:39 |
| openstackgerrit | Merged openstack/ironic-python-agent: Reduced restriction of parsing for dmidecode output https://review.openstack.org/280970 | 17:39 |
| lucasagomes | because that's how it's being used | 17:39 |
| lucasagomes | rloo, then later ironic-lib should have it's own rootwrap | 17:39 |
| lucasagomes | ironic-lib-rootwrap, with it's own filters | 17:39 |
| lucasagomes | and then we can remove the ironic one | 17:39 |
| rloo | lucasagomes: the problem with removing the one from ironic-lib, is that it is a lib meant to be used by not only ironic. | 17:39 |
| rloo | lucasagomes: how is some other user of ironic-lib meant to know which filters to use if we remove it. | 17:39 |
| lucasagomes | rloo, right, but that's a intermediate step | 17:40 |
| Goneri | can we rename the ironic version of the file | 17:40 |
| rloo | lucasagomes: or we could change/comment it. | 17:40 |
| Goneri | this will make the situation more clear for the packages | 17:40 |
| rloo | lucasagomes: i mean, in ironic-lib, put a readme or something saying if you use it, you may want these filters for rootwrap. | 17:40 |
| rloo | lucasagomes: i think that's what the intent was when we put ironic-lib.filters in ironic-lib and copied it in ironic. | 17:40 |
| rloo | lucasagomes: but we didn't document it properly. | 17:41 |
| lucasagomes | rloo, right... I understand that's the ultimate go | 17:42 |
| lucasagomes | rloo, but I think that for that, ironic-lib will need it's own rootwrap? | 17:42 |
| rloo | Goneri: so yeah, maybe a comment in ironic's ironic-lib.filters | 17:42 |
| lucasagomes | other than ppl copying stuff around, because ironic-lib is a requirement for ironic | 17:42 |
| lucasagomes | rloo, https://github.com/openstack/ironic/blob/master/setup.cfg#L29 | 17:42 |
| lucasagomes | goal* | 17:43 |
| *** derekh has quit IRC | 17:43 | |
| Goneri | rloo, I will patch both | 17:43 |
| lucasagomes | Goneri, ty | 17:43 |
| rloo | lucasagomes: i think i might have looked into the lib having its own rootwrap but it seems to me there was some hitch to it that i don't recall. | 17:44 |
| rloo | lucasagomes: or maybe i didn't want to go to too much effort. i was just trying to get the ironic-lib stuff working at the time i think. | 17:44 |
| lucasagomes | rloo, which is fair enough! | 17:44 |
| lucasagomes | we have options to fix that, so it's a good thing :-) | 17:44 |
| *** stendulker has quit IRC | 17:45 | |
| *** harlowja_at_home has joined #openstack-ironic | 17:45 | |
| rloo | lucasagomes: and 'trying to get' == reviews on someone else's patches :) | 17:45 |
| lucasagomes | rloo, heh | 17:45 |
| rloo | lucasagomes: i think we were supposed to deprecate some config too that i don't think got done. | 17:45 |
| lucasagomes | rloo, right, it's ok. We will fix it as we start using ironic-lib more | 17:47 |
| lucasagomes | right now only ironic's using it, Goneri is adding it to IPA too | 17:47 |
| lucasagomes | and so on... | 17:47 |
| rloo | good. thx Goneri. | 17:47 |
| *** [1]cdearborn has quit IRC | 17:48 | |
| Goneri | rloo, my pleasure :) | 17:48 |
| *** absubram has quit IRC | 17:48 | |
| rloo | Goneri: it has to do with this root_helper config in ironic-lib: https://github.com/openstack/ironic-lib/blob/master/ironic_lib/utils.py#L35 | 17:49 |
| openstackgerrit | Merged openstack/ironic-inspector: [devstack] Improve virtual machines logs https://review.openstack.org/285412 | 17:49 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: [WIP] Architectural changes + MAC address from iLO https://review.openstack.org/276703 | 17:49 |
| *** rloo has quit IRC | 17:52 | |
| *** piet has joined #openstack-ironic | 17:54 | |
| *** baoli has quit IRC | 17:58 | |
| *** harshs has joined #openstack-ironic | 18:00 | |
| *** krtaylor has quit IRC | 18:00 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: add wipefs to ironic-lib.filters https://review.openstack.org/286750 | 18:02 |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic-lib: use wipefs to erase FS meta information https://review.openstack.org/286283 | 18:05 |
| *** penick has joined #openstack-ironic | 18:05 | |
| Goneri | lucasagomes ^ | 18:05 |
| *** ndipanov has quit IRC | 18:05 | |
| lucasagomes | Goneri, just reviewed, it's pointing to the wrong patch in the commit message | 18:06 |
| *** rloo has joined #openstack-ironic | 18:06 | |
| lucasagomes | Goneri, other than that it lgtm | 18:06 |
| lucasagomes | Goneri, I left the change id of the wipefs patch in the commit message | 18:06 |
| *** vinm213 has quit IRC | 18:06 | |
| lucasagomes | I mean in the comment* | 18:07 |
| Goneri | Oops | 18:07 |
| * lucasagomes english is going down | 18:07 | |
| *** baoli has joined #openstack-ironic | 18:07 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: add wipefs to ironic-lib.filters https://review.openstack.org/286750 | 18:08 |
| Goneri | lucasagomes, Here it is^ | 18:08 |
| *** e0ne has quit IRC | 18:08 | |
| lucasagomes | Goneri, cool +2 :-) | 18:08 |
| Goneri | lucasagomes, thanks dude :) | 18:10 |
| lucasagomes | Goneri, np, thank you for fixing it | 18:10 |
| *** nathanh has quit IRC | 18:13 | |
| *** baoli has quit IRC | 18:14 | |
| *** openstackgerrit has quit IRC | 18:18 | |
| *** spandhe has joined #openstack-ironic | 18:18 | |
| *** openstackgerrit has joined #openstack-ironic | 18:18 | |
| mgould | jlvillal, just saw your message | 18:19 |
| mgould | I'm around for about another 10 minutes | 18:19 |
| openstackgerrit | Merged openstack/python-ironicclient: Log SHA1 hash of X-Auth-Token value https://review.openstack.org/278951 | 18:20 |
| lucasagomes | I will call it a day | 18:21 |
| lucasagomes | have a great evening all, see you tomorrow | 18:21 |
| thiagop | night lucasagomes | 18:21 |
| lucasagomes | see ya | 18:22 |
| *** raddaoui has joined #openstack-ironic | 18:22 | |
| *** lucasagomes is now known as lucas-dinner | 18:22 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-inspector: Use keystoneauth for Ironic and Swift clients https://review.openstack.org/286510 | 18:24 |
| *** serverascode has quit IRC | 18:26 | |
| *** BadCub has quit IRC | 18:27 | |
| *** BadCub has joined #openstack-ironic | 18:27 | |
| *** serverascode has joined #openstack-ironic | 18:28 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Allow rerunning introspection on stored data https://review.openstack.org/267637 | 18:30 |
| openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/275722 | 18:30 |
| *** permalac has quit IRC | 18:31 | |
| *** evgenyl_ has joined #openstack-ironic | 18:31 | |
| * mgould has to go; see you all tomorrow! | 18:32 | |
| *** mgould has quit IRC | 18:32 | |
| *** suro-patz has quit IRC | 18:32 | |
| sambetts | o/ see ya mgould | 18:33 |
| sambetts | o/ everyone else too, I'm off | 18:33 |
| *** sambetts is now known as sambetts|afk | 18:33 | |
| *** lekha_ has joined #openstack-ironic | 18:33 | |
| thiagop | night mgould sambetts|afk | 18:33 |
| *** permalac has joined #openstack-ironic | 18:34 | |
| *** permalac has quit IRC | 18:34 | |
| *** trown|lunch is now known as trown | 18:35 | |
| sinval-afk | night sambetts|afk | 18:36 |
| *** sinval-afk is now known as sinval | 18:36 | |
| *** lekha has quit IRC | 18:37 | |
| *** evgenyl has quit IRC | 18:37 | |
| *** evgenyl_ is now known as evgenyl | 18:37 | |
| *** lekha_ is now known as lekha | 18:38 | |
| *** ijw has joined #openstack-ironic | 18:42 | |
| *** krtaylor has joined #openstack-ironic | 18:43 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Install python if it's not there https://review.openstack.org/283783 | 18:45 |
| *** dims has joined #openstack-ironic | 18:45 | |
| *** dims_ has quit IRC | 18:52 | |
| *** pas-ha has quit IRC | 18:52 | |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Install python if it's not there https://review.openstack.org/283783 | 18:52 |
| *** pas-ha has joined #openstack-ironic | 18:52 | |
| *** SpamapS has quit IRC | 18:54 | |
| *** openstack has joined #openstack-ironic | 19:16 | |
| *** openstackstatus has joined #openstack-ironic | 19:17 | |
| *** ChanServ sets mode: +v openstackstatus | 19:17 | |
| *** openstackgerrit has joined #openstack-ironic | 19:20 | |
| openstackgerrit | Gonéri Le Bouder proposed openstack/ironic: ipxe: add --timeout parameter to kernel and initrd https://review.openstack.org/283893 | 19:22 |
| *** cppforlife_ has quit IRC | 19:25 | |
| *** clif_h has quit IRC | 19:25 | |
| *** alaski has quit IRC | 19:25 | |
| *** ChrisMead has quit IRC | 19:25 | |
| *** zsmithnyc has quit IRC | 19:25 | |
| *** johnthetubaguy has quit IRC | 19:25 | |
| *** pas-ha has quit IRC | 19:30 | |
| *** cppforlife_ has joined #openstack-ironic | 19:31 | |
| *** clif_h has joined #openstack-ironic | 19:31 | |
| *** alaski has joined #openstack-ironic | 19:31 | |
| *** ChrisMead has joined #openstack-ironic | 19:31 | |
| *** zsmithnyc has joined #openstack-ironic | 19:31 | |
| *** johnthetubaguy has joined #openstack-ironic | 19:31 | |
| *** jrist has quit IRC | 19:31 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 19:36 |
| *** suro-patz has joined #openstack-ironic | 19:37 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Classifying tests into functional and unit https://review.openstack.org/286797 | 19:41 |
| *** Sukhdev has joined #openstack-ironic | 19:43 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 19:45 |
| *** spandhe has quit IRC | 19:50 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: [WIP] OneView client with managers https://review.openstack.org/286803 | 19:50 |
| *** spandhe has joined #openstack-ironic | 19:52 | |
| *** raddaoui has joined #openstack-ironic | 19:55 | |
| *** jrist has joined #openstack-ironic | 19:55 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add documentation for RAID https://review.openstack.org/226330 | 19:57 |
| *** david-lyle has quit IRC | 19:59 | |
| *** raddaoui has quit IRC | 19:59 | |
| *** daemontool__ has joined #openstack-ironic | 20:05 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 20:06 |
| *** ijw has quit IRC | 20:07 | |
| *** jcoufal_ has quit IRC | 20:08 | |
| *** daemontool_ has quit IRC | 20:09 | |
| *** david-lyle has joined #openstack-ironic | 20:09 | |
| Nisha_brb | jroll, there? | 20:10 |
| *** Nisha_brb is now known as Nisha | 20:11 | |
| *** piet has quit IRC | 20:13 | |
| *** raddaoui has joined #openstack-ironic | 20:18 | |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Classifying tests into functional and unit https://review.openstack.org/286797 | 20:22 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Fix 'uptime' time correction attempt https://review.openstack.org/286181 | 20:26 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Facility to get mac address from iLO on DL class https://review.openstack.org/286801 | 20:27 |
| *** suro-patz has quit IRC | 20:30 | |
| *** ChrisAusten has joined #openstack-ironic | 20:33 | |
| JayF | TheJulia: around? have a systems question I'd like to run by you | 20:33 |
| TheJulia | JayF: somewhat around :) | 20:35 |
| TheJulia | whats up? | 20:35 |
| JayF | TheJulia: I have a haproxy in front of a collection of ironic apis | 20:35 |
| JayF | when it throws a 4xx 5xx style error, ironic gets angry because it's a typical html/text style error page | 20:36 |
| JayF | wondering if you've played around at all with sending json docs back to ironic in that case | 20:36 |
| TheJulia | I have not... but I guess your in http mode with haproxy and doing cookie injection for session tracking/stickiness ? | 20:37 |
| *** e0ne has joined #openstack-ironic | 20:38 | |
| *** penick has quit IRC | 20:38 | |
| JayF | we're not doing any session tracking | 20:40 |
| JayF | ironic is not sessioned in that way | 20:40 |
| TheJulia | yeah, maybe tcp mode so haproxy is operating about it agnostically? | 20:41 |
| *** Nisha has quit IRC | 20:41 | |
| JayF | I mean, this isn't a problem | 20:43 |
| JayF | it's just trying to make my logs not explode when one api takes a long time to respond :) | 20:43 |
| *** pcaruana has joined #openstack-ironic | 20:44 | |
| TheJulia | oh... ahh | 20:44 |
| JayF | yeah, I think I'm going to do it, with these: http://cialis.oldos.org/~jay/json_error_pages/ | 20:45 |
| JayF | I wanted to look at openstack errors in particular and try to perfectly mimic them, but that's much better than html and exploding json.parse() | 20:45 |
| TheJulia | okay, now I understand :) | 20:46 |
| TheJulia | ++ | 20:46 |
| *** e0ne has quit IRC | 20:47 | |
| openstackgerrit | Merged openstack/python-oneviewclient: Fixing URL of index variable and refactoring https://review.openstack.org/285402 | 20:54 |
| *** piet has joined #openstack-ironic | 20:55 | |
| *** e0ne has joined #openstack-ironic | 20:55 | |
| *** e0ne has quit IRC | 20:59 | |
| *** raddaoui_ has joined #openstack-ironic | 21:00 | |
| *** raddaoui has quit IRC | 21:00 | |
| mrda | Morning Ironic | 21:07 |
| *** raildo is now known as raildo-afk | 21:07 | |
| *** david-lyle has quit IRC | 21:08 | |
| TheJulia | good morning mrda | 21:10 |
| mrda | Hi TheJulia | 21:11 |
| *** jaybeale has quit IRC | 21:12 | |
| *** david-lyle has joined #openstack-ironic | 21:17 | |
| NobodyCam | morning mrda | 21:20 |
| *** raddaoui_ has quit IRC | 21:29 | |
| *** dims_ has joined #openstack-ironic | 21:31 | |
| *** dims has quit IRC | 21:33 | |
| openstackgerrit | Merged openstack/pyghmi: Fix 'uptime' time correction attempt https://review.openstack.org/286181 | 21:40 |
| *** derekh has joined #openstack-ironic | 21:41 | |
| *** Sukhdev has quit IRC | 21:43 | |
| *** baoli has joined #openstack-ironic | 21:43 | |
| *** baoli_ has joined #openstack-ironic | 21:44 | |
| mrda | hey NobodyCam | 21:47 |
| NobodyCam | :) | 21:47 |
| *** baoli has quit IRC | 21:48 | |
| *** xavier_ has joined #openstack-ironic | 21:49 | |
| *** raddaoui_ has joined #openstack-ironic | 21:49 | |
| *** trown is now known as trown|outtypewww | 21:59 | |
| *** causten_ has joined #openstack-ironic | 22:00 | |
| *** pcaruana has quit IRC | 22:01 | |
| *** dims_ has quit IRC | 22:02 | |
| *** ChrisAusten has quit IRC | 22:03 | |
| *** Sukhdev has joined #openstack-ironic | 22:04 | |
| *** dprince has quit IRC | 22:08 | |
| *** ijw has joined #openstack-ironic | 22:09 | |
| *** dims has joined #openstack-ironic | 22:10 | |
| *** penick has joined #openstack-ironic | 22:16 | |
| *** Sukhdev has quit IRC | 22:19 | |
| *** keedya has quit IRC | 22:21 | |
| *** raddaoui_ has quit IRC | 22:21 | |
| *** suro-patz has joined #openstack-ironic | 22:22 | |
| *** achanda has quit IRC | 22:23 | |
| *** krtaylor has quit IRC | 22:27 | |
| *** david-lyle has quit IRC | 22:29 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/283983 | 22:30 |
| *** achanda has joined #openstack-ironic | 22:36 | |
| *** raddaoui_ has joined #openstack-ironic | 22:36 | |
| *** cdearborn has joined #openstack-ironic | 22:39 | |
| *** cdearborn has quit IRC | 22:41 | |
| *** piet has quit IRC | 22:43 | |
| *** baoli_ has quit IRC | 22:54 | |
| *** baoli has joined #openstack-ironic | 22:54 | |
| *** [1]cdearborn has quit IRC | 22:55 | |
| *** raddaoui_ has quit IRC | 22:55 | |
| *** jaybeale has joined #openstack-ironic | 22:55 | |
| *** dims has quit IRC | 22:55 | |
| *** baoli has quit IRC | 22:58 | |
| *** baoli has joined #openstack-ironic | 22:59 | |
| *** rbudden has quit IRC | 22:59 | |
| *** harlowja_at_home has quit IRC | 23:00 | |
| *** baoli has quit IRC | 23:00 | |
| *** david-lyle has joined #openstack-ironic | 23:00 | |
| *** dims has joined #openstack-ironic | 23:00 | |
| *** baoli has joined #openstack-ironic | 23:01 | |
| *** ianw has quit IRC | 23:07 | |
| *** intr1nsic has quit IRC | 23:08 | |
| *** baoli has quit IRC | 23:11 | |
| *** chlong_ has joined #openstack-ironic | 23:13 | |
| *** krtaylor has joined #openstack-ironic | 23:16 | |
| *** Sukhdev has joined #openstack-ironic | 23:18 | |
| *** harshs has quit IRC | 23:26 | |
| *** spandhe has quit IRC | 23:30 | |
| *** intr1nsic has joined #openstack-ironic | 23:32 | |
| *** spandhe has joined #openstack-ironic | 23:33 | |
| *** harshs has joined #openstack-ironic | 23:37 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Clean up driver loading in init_host https://review.openstack.org/285850 | 23:40 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Refactor driver loading to load a driver instance per node https://review.openstack.org/285851 | 23:41 |
| *** david-lyle has quit IRC | 23:41 | |
| jlvillal | jroll: I'm kind of concerned about this: http://logs.openstack.org/51/285851/5/check/gate-ironic-python27/52cbcfd/console.html#_2016-02-29_17_58_31_822 | 23:47 |
| jlvillal | jroll: I think the code in this patch is touching the error case: https://review.openstack.org/#/c/285850/ | 23:47 |
| jlvillal | jroll: I also saw that error occur locally, but then stopped happening... | 23:48 |
| jlvillal | But I'm not sure why the error happens... :( | 23:48 |
| jlvillal | Maybe it is some kind of race condition | 23:49 |
| jroll | jlvillal: yeah, I think it's related to dmitry's comment here https://review.openstack.org/#/c/285850/3/ironic/tests/unit/common/test_driver_factory.py | 23:49 |
| jroll | which I've since fixed, and just rebased on | 23:49 |
| jlvillal | jroll: ah :) | 23:50 |
| jlvillal | jroll: I will wait for it to be pushed. Thanks! | 23:51 |
| *** smoriya_ has joined #openstack-ironic | 23:51 | |
| jroll | jlvillal: it is pushed already, see bot messages from 13 minutes ago | 23:53 |
| jlvillal | jroll: I only saw change to docstring in from 4 to 5 | 23:53 |
| jroll | jlvillal: oh, that was fixed in 3-4 | 23:54 |
| jroll | but I didn't rebase til now | 23:54 |
| jlvillal | jroll: Ah. Thanks, makes sense. | 23:54 |
| jlvillal | jroll: And that makes sense why I saw it locally because I had checked out the top patch in the series. | 23:55 |
| jlvillal | Refactor driver loading to load a driver instance per node https://review.openstack.org/285851 | 23:55 |
| jroll | ya | 23:55 |
| * jlvillal is brave and votes ahead of Jenkins :) | 23:56 | |
| *** rloo has quit IRC | 23:57 | |
| *** harshs has left #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!