| *** baoli has joined #openstack-ironic | 00:01 | |
| *** jaypipes has quit IRC | 00:02 | |
| *** rama_y has quit IRC | 00:04 | |
| *** baoli has quit IRC | 00:06 | |
| *** yangyape_ has quit IRC | 00:09 | |
| *** yangyapeng has joined #openstack-ironic | 00:10 | |
| *** yangyapeng has quit IRC | 00:14 | |
| *** jkilpatr has quit IRC | 00:16 | |
| *** srobert has quit IRC | 00:22 | |
| *** mrtenio has quit IRC | 00:26 | |
| *** crushil_ has joined #openstack-ironic | 00:28 | |
| *** mikal_ has quit IRC | 00:32 | |
| *** mikal has joined #openstack-ironic | 00:34 | |
| *** crushil_ has quit IRC | 00:38 | |
| *** litao__ has joined #openstack-ironic | 00:53 | |
| *** hw_wutianwei1 has joined #openstack-ironic | 00:55 | |
| *** ijw has quit IRC | 01:02 | |
| *** ijw has joined #openstack-ironic | 01:02 | |
| *** ijw has quit IRC | 01:07 | |
| *** hw_wutianwei1 has quit IRC | 01:09 | |
| *** hw_wutianwei1 has joined #openstack-ironic | 01:09 | |
| *** yangyapeng has joined #openstack-ironic | 01:12 | |
| *** phuongnh has joined #openstack-ironic | 01:14 | |
| *** phuongnh has quit IRC | 01:20 | |
| *** yangyapeng has quit IRC | 01:40 | |
| *** yangyapeng has joined #openstack-ironic | 01:40 | |
| *** zhangfei has quit IRC | 01:42 | |
| *** crushil has quit IRC | 01:50 | |
| *** yangyapeng has quit IRC | 01:50 | |
| *** yangyapeng has joined #openstack-ironic | 01:51 | |
| *** gcb has joined #openstack-ironic | 01:52 | |
| *** yangyape_ has joined #openstack-ironic | 01:56 | |
| *** yangyapeng has quit IRC | 01:56 | |
| *** rbudden has quit IRC | 02:05 | |
| *** yangyape_ has quit IRC | 02:12 | |
| *** yangyapeng has joined #openstack-ironic | 02:13 | |
| *** racedo has quit IRC | 02:15 | |
| *** yangyapeng has quit IRC | 02:15 | |
| *** yangyapeng has joined #openstack-ironic | 02:15 | |
| *** yangyapeng has quit IRC | 02:20 | |
| *** rwsu has joined #openstack-ironic | 02:20 | |
| *** rbudden has joined #openstack-ironic | 02:45 | |
| *** sasha2 has quit IRC | 02:45 | |
| *** rbudden has quit IRC | 02:51 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 02:54 | |
| *** Dinesh_Bhor has quit IRC | 02:57 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 02:57 | |
| *** mikal has quit IRC | 02:59 | |
| *** yangyapeng has joined #openstack-ironic | 03:01 | |
| *** mikal has joined #openstack-ironic | 03:02 | |
| *** yangyape_ has joined #openstack-ironic | 03:03 | |
| *** yangyapeng has quit IRC | 03:07 | |
| *** crushil has joined #openstack-ironic | 03:14 | |
| *** crushil has quit IRC | 03:17 | |
| *** jlabarre has quit IRC | 03:21 | |
| *** Dinesh_Bhor has quit IRC | 03:24 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 03:28 | |
| *** rwsu has quit IRC | 03:30 | |
| *** Dinesh_Bhor has quit IRC | 03:34 | |
| *** rwsu has joined #openstack-ironic | 03:37 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 03:46 | |
| *** udesale has joined #openstack-ironic | 03:59 | |
| *** rwsu has quit IRC | 04:02 | |
| *** aparnav has joined #openstack-ironic | 04:10 | |
| *** livelace2 has quit IRC | 04:10 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:12 | |
| *** rwsu has joined #openstack-ironic | 04:15 | |
| *** mtreinish has quit IRC | 04:24 | |
| *** mtreinish has joined #openstack-ironic | 04:34 | |
| *** crushil has joined #openstack-ironic | 04:38 | |
| *** trungnv has joined #openstack-ironic | 04:42 | |
| *** gcb has quit IRC | 04:48 | |
| *** etingof has quit IRC | 04:55 | |
| *** yangyape_ has quit IRC | 05:09 | |
| *** yangyapeng has joined #openstack-ironic | 05:09 | |
| *** etingof has joined #openstack-ironic | 05:13 | |
| *** adreznec has quit IRC | 05:13 | |
| *** bnemec has quit IRC | 05:14 | |
| *** yangyapeng has quit IRC | 05:14 | |
| *** livelace has joined #openstack-ironic | 05:15 | |
| *** livelace2 has joined #openstack-ironic | 05:17 | |
| *** Nisha_ has joined #openstack-ironic | 05:18 | |
| *** Nisha_Agarwal has quit IRC | 05:18 | |
| *** Nisha_away has joined #openstack-ironic | 05:19 | |
| *** Nisha_ has quit IRC | 05:19 | |
| *** livelace has quit IRC | 05:21 | |
| *** etingof has quit IRC | 05:21 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:23 | |
| *** Nisha_away has quit IRC | 05:23 | |
| *** gcb has joined #openstack-ironic | 05:24 | |
| *** Nisha_Agarwal has quit IRC | 05:24 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:25 | |
| *** crushil_ has joined #openstack-ironic | 05:27 | |
| *** crushil has quit IRC | 05:29 | |
| *** mat128 has joined #openstack-ironic | 05:38 | |
| *** mat128 has quit IRC | 05:41 | |
| *** mat128 has joined #openstack-ironic | 05:43 | |
| *** mat128 has quit IRC | 05:46 | |
| *** livelace has joined #openstack-ironic | 05:46 | |
| *** mat128 has joined #openstack-ironic | 05:47 | |
| *** Nisha_Agarwal has quit IRC | 05:49 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add uWSGI support https://review.openstack.org/507011 | 05:49 |
|---|---|---|
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 05:49 |
| *** livelace has quit IRC | 05:51 | |
| *** ChubYann has quit IRC | 05:55 | |
| *** adreznec has joined #openstack-ironic | 06:01 | |
| *** dpawar has joined #openstack-ironic | 06:01 | |
| *** livelace has joined #openstack-ironic | 06:04 | |
| *** marios has joined #openstack-ironic | 06:05 | |
| *** mat128 has quit IRC | 06:09 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add uWSGI support https://review.openstack.org/507011 | 06:17 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 06:17 |
| *** jaianshu has joined #openstack-ironic | 06:19 | |
| *** pcaruana has joined #openstack-ironic | 06:20 | |
| *** jaganathan has joined #openstack-ironic | 06:20 | |
| *** zshi__ has joined #openstack-ironic | 06:23 | |
| *** mjura has joined #openstack-ironic | 06:25 | |
| *** hjensas has joined #openstack-ironic | 06:26 | |
| *** hjensas has quit IRC | 06:26 | |
| *** hjensas has joined #openstack-ironic | 06:26 | |
| *** etingof has joined #openstack-ironic | 06:27 | |
| *** makowals has joined #openstack-ironic | 06:33 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic stable/pike: Adds more exception handling for ironic-conductor heartbeat https://review.openstack.org/507373 | 06:45 |
| *** stendulker has joined #openstack-ironic | 06:45 | |
| *** crushil_ has quit IRC | 06:45 | |
| *** rcernin has joined #openstack-ironic | 06:47 | |
| *** dpawar has quit IRC | 07:02 | |
| *** karthiks has quit IRC | 07:03 | |
| *** karthiks has joined #openstack-ironic | 07:03 | |
| *** makowals has quit IRC | 07:11 | |
| *** dtantsur|afk is now known as dtantsur | 07:11 | |
| dtantsur | morning ironic | 07:11 |
| *** tesseract has joined #openstack-ironic | 07:14 | |
| openstackgerrit | Anshul Jain proposed openstack/sushy master: Add PUT method to connector https://review.openstack.org/507385 | 07:15 |
| *** dpawar has joined #openstack-ironic | 07:19 | |
| *** e0ne has joined #openstack-ironic | 07:22 | |
| *** jtomasek has joined #openstack-ironic | 07:22 | |
| etingof | o/ | 07:23 |
| *** hw_wutianwei1 has quit IRC | 07:24 | |
| *** racedo has joined #openstack-ironic | 07:30 | |
| etingof | how could one figure out MACs of rdocloud instances? | 07:31 |
| dtantsur | etingof: 'openstack port list'? :) | 07:32 |
| dtantsur | or port show. I think one of these has to show MACs for bound ports | 07:32 |
| etingof | $ openstack --os-cloud rdo-cloud port list | 07:34 |
| etingof | openstack: 'port' is not an openstack command. | 07:34 |
| *** mrch has joined #openstack-ironic | 07:35 | |
| dtantsur | etingof: you have ancient python-openstackclient package, I bet (complain to number80 on #rdo ;) | 07:35 |
| dtantsur | try neutron port-list / port-show | 07:35 |
| etingof | that worked, thanks! | 07:36 |
| *** rbartal has joined #openstack-ironic | 07:37 | |
| *** phuongnh has joined #openstack-ironic | 07:39 | |
| etingof | dtantsur, btw, -1 is resolved on this patch if you have time for a look -- https://review.openstack.org/#/c/498489/ | 07:45 |
| patchbot | patch 498489 - ironic-python-agent - Report /dev/disk/by-path on inspection | 07:45 |
| dtantsur | k | 07:46 |
| *** tiendc has joined #openstack-ironic | 07:59 | |
| *** ccamacho has joined #openstack-ironic | 07:59 | |
| pas-ha | etingof: it is in current openstack client, check your version | 08:18 |
| * pas-ha slowpokes | 08:18 | |
| etingof | it's 2.2.0, but it worked out through neutron | 08:18 |
| dtantsur | etingof: my F26 has 3.2.0, it's much better | 08:19 |
| dtantsur | also approved the by-disk patches | 08:19 |
| * etingof needs to upgrade | 08:19 | |
| etingof | \o/ | 08:20 |
| etingof | it's time for a followup patch then | 08:20 |
| dtantsur | yep :) | 08:29 |
| *** ralonsoh has joined #openstack-ironic | 08:31 | |
| *** TuanLA has joined #openstack-ironic | 08:35 | |
| pas-ha | etingof: on DevStack 'openstack --version' is 3.12.0 (and this is specified as lower bound for ironicclient currently) | 08:38 |
| pas-ha | and indeed, some time ago I remember openstackclient was lacking port operations, while neutronclient already declared itself deprecated o_O | 08:38 |
| *** vmud213 has joined #openstack-ironic | 08:40 | |
| etingof | yeah, I just needed to talk to rdocloud, I am at 3.12.0 at devstack as well | 08:40 |
| *** udesale has quit IRC | 08:45 | |
| *** noam_ has joined #openstack-ironic | 08:48 | |
| noam_ | hi | 08:48 |
| noam_ | I have some time issues during cleaning for ironic nodes server gets Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(0,0) | 08:48 |
| noam_ | someone know how can i work around it? | 08:49 |
| noam_ | http://paste.openstack.org/show/621909/ | 08:51 |
| *** udesale has joined #openstack-ironic | 08:51 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:56 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Replace the usage of 'admin_manager' with 'os_admin' https://review.openstack.org/507012 | 08:57 |
| openstackgerrit | Merged openstack/ironic-lib master: Added `by_path` root device hint https://review.openstack.org/500524 | 08:58 |
| dtantsur | noam_: are you sure it boots IPA, not the OS from a local disk? if there are random DHCP failures, that may be the case | 09:00 |
| *** derekh has joined #openstack-ironic | 09:00 | |
| noam_ | It boot the deploy image | 09:01 |
| *** amarao has joined #openstack-ironic | 09:03 | |
| noam_ | Repating the cleaning result the same | 09:04 |
| *** yangyapeng has joined #openstack-ironic | 09:10 | |
| *** udesale has quit IRC | 09:11 | |
| *** phuongnh has quit IRC | 09:11 | |
| *** udesale has joined #openstack-ironic | 09:11 | |
| *** yangyapeng has quit IRC | 09:12 | |
| *** yangyapeng has joined #openstack-ironic | 09:13 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Report /dev/disk/by-path on inspection https://review.openstack.org/498489 | 09:15 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Remove hard-coded path to grub binaries https://review.openstack.org/504051 | 09:16 |
| *** Nisha_ has joined #openstack-ironic | 09:19 | |
| *** Nisha_Agarwal has quit IRC | 09:19 | |
| *** dtantsur is now known as dtantsur|lunch | 09:21 | |
| *** yangyapeng has quit IRC | 09:30 | |
| *** yangyapeng has joined #openstack-ironic | 09:31 | |
| *** yangyapeng has quit IRC | 09:34 | |
| *** yangyapeng has joined #openstack-ironic | 09:35 | |
| *** Nisha_ has quit IRC | 09:37 | |
| *** Nisha_ has joined #openstack-ironic | 09:37 | |
| *** yangyapeng has quit IRC | 09:39 | |
| vdrok | late morning ironic! | 09:41 |
| *** milan has joined #openstack-ironic | 09:43 | |
| *** vmud213 has quit IRC | 09:49 | |
| *** vmud213 has joined #openstack-ironic | 09:50 | |
| *** Nisha_away has joined #openstack-ironic | 09:53 | |
| *** Nisha_ has quit IRC | 09:53 | |
| *** etingof has quit IRC | 10:00 | |
| *** ijw has joined #openstack-ironic | 10:08 | |
| openstackgerrit | yfzhao proposed openstack/ironic master: Fix wrong links https://review.openstack.org/492876 | 10:10 |
| *** hjensas has quit IRC | 10:11 | |
| *** etingof has joined #openstack-ironic | 10:12 | |
| *** ijw has quit IRC | 10:13 | |
| *** Nisha_away has quit IRC | 10:18 | |
| *** etingof has quit IRC | 10:19 | |
| *** yangyapeng has joined #openstack-ironic | 10:22 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent stable/pike: Remove hard-coded path to grub binaries https://review.openstack.org/507468 | 10:25 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic-python-agent stable/ocata: Remove hard-coded path to grub binaries https://review.openstack.org/507469 | 10:25 |
| *** yangyapeng has quit IRC | 10:26 | |
| *** vmud213 has quit IRC | 10:29 | |
| *** jaganathan has quit IRC | 10:31 | |
| *** etingof has joined #openstack-ironic | 10:33 | |
| *** aparnav has quit IRC | 10:34 | |
| pas-ha | morning vdrok :) | 10:35 |
| *** dsneddon has quit IRC | 10:35 | |
| *** udesale has quit IRC | 10:37 | |
| *** athomas has quit IRC | 10:43 | |
| *** udesale has joined #openstack-ironic | 10:53 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: [devstack] Switch ironic to uWSGI https://review.openstack.org/507067 | 10:54 |
| *** jkilpatr has joined #openstack-ironic | 10:54 | |
| *** hjensas has joined #openstack-ironic | 10:59 | |
| *** hjensas has joined #openstack-ironic | 10:59 | |
| *** jkilpatr has quit IRC | 10:59 | |
| *** athomas has joined #openstack-ironic | 11:04 | |
| openstackgerrit | zenghui.shi proposed openstack/ironic-specs master: New hardware interface for BIOS configuration https://review.openstack.org/496481 | 11:07 |
| *** vsaienk0 has quit IRC | 11:08 | |
| *** livelace2 has quit IRC | 11:13 | |
| *** jkilpatr has joined #openstack-ironic | 11:15 | |
| *** ralonsoh_ has joined #openstack-ironic | 11:21 | |
| *** e0ne has quit IRC | 11:21 | |
| *** fellypefca-afk is now known as fellypefca | 11:24 | |
| *** stendulker_ has joined #openstack-ironic | 11:24 | |
| fellypefca | Good morning :) | 11:24 |
| *** ralonsoh has quit IRC | 11:24 | |
| vdrok | morning pas-ha , fellypefca | 11:25 |
| *** stendulker has quit IRC | 11:27 | |
| fellypefca | Hi vdrok | 11:28 |
| *** e0ne has joined #openstack-ironic | 11:28 | |
| *** TuanLA has quit IRC | 11:29 | |
| *** stendulker_ has quit IRC | 11:29 | |
| *** rbudden has joined #openstack-ironic | 11:29 | |
| *** yangyapeng has joined #openstack-ironic | 11:38 | |
| *** zshi__ has quit IRC | 11:41 | |
| *** bfournie has quit IRC | 11:44 | |
| *** bfournie has joined #openstack-ironic | 11:44 | |
| *** bfournie has quit IRC | 11:48 | |
| *** dprince has joined #openstack-ironic | 11:57 | |
| *** rbudden has quit IRC | 11:59 | |
| *** jlabarre has joined #openstack-ironic | 12:02 | |
| *** jaianshu has quit IRC | 12:03 | |
| *** tiendc has quit IRC | 12:04 | |
| *** livelace2 has joined #openstack-ironic | 12:07 | |
| *** mat128 has joined #openstack-ironic | 12:08 | |
| *** dtantsur|lunch is now known as dtantsur | 12:15 | |
| *** litao__ has quit IRC | 12:16 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Documentation for 'oneview' hardware type https://review.openstack.org/502072 | 12:28 |
| *** udesale has quit IRC | 12:29 | |
| dtantsur | morning fellypefca, vdrok, TheJulia | 12:31 |
| TheJulia | Good morning! | 12:32 |
| *** vsaienk0 has joined #openstack-ironic | 12:33 | |
| *** athomas has quit IRC | 12:33 | |
| fellypefca | Good morning dtantsur and TheJulia :) | 12:34 |
| *** bfournie has joined #openstack-ironic | 12:34 | |
| *** zshi__ has joined #openstack-ironic | 12:36 | |
| pas-ha | morning TheJulia fellypefca | 12:43 |
| fellypefca | Hey pas-ha :) | 12:44 |
| vdrok | morning dtantsur and TheJulia | 12:45 |
| *** trown|outtypewww is now known as trown | 12:45 | |
| *** vsaienk0 has quit IRC | 12:47 | |
| *** rloo has joined #openstack-ironic | 12:47 | |
| *** vsaienk0 has joined #openstack-ironic | 12:47 | |
| *** jaypipes has joined #openstack-ironic | 12:49 | |
| *** athomas has joined #openstack-ironic | 12:50 | |
| *** mat128 has quit IRC | 12:52 | |
| *** bnemec has joined #openstack-ironic | 12:52 | |
| *** srobert has joined #openstack-ironic | 12:54 | |
| *** sasha2 has joined #openstack-ironic | 12:55 | |
| *** mat128 has joined #openstack-ironic | 12:55 | |
| *** mat128 has quit IRC | 12:57 | |
| *** rbudden has joined #openstack-ironic | 12:58 | |
| *** Dinesh_Bhor has quit IRC | 12:59 | |
| *** jcoufal has joined #openstack-ironic | 13:04 | |
| *** mat128 has joined #openstack-ironic | 13:07 | |
| *** Dinesh_Bhor has joined #openstack-ironic | 13:08 | |
| *** mat128 has quit IRC | 13:10 | |
| *** mat128 has joined #openstack-ironic | 13:12 | |
| *** Goneri has joined #openstack-ironic | 13:14 | |
| *** mat128 has quit IRC | 13:15 | |
| *** dpawar has quit IRC | 13:18 | |
| *** mat128 has joined #openstack-ironic | 13:22 | |
| *** baoli has joined #openstack-ironic | 13:23 | |
| *** udesale has joined #openstack-ironic | 13:27 | |
| *** mat128 has quit IRC | 13:33 | |
| *** mat128 has joined #openstack-ironic | 13:40 | |
| *** mat128 has quit IRC | 13:41 | |
| *** mat128 has joined #openstack-ironic | 13:42 | |
| *** srobert has quit IRC | 13:46 | |
| *** srobert has joined #openstack-ironic | 13:46 | |
| *** cdearborn_ has joined #openstack-ironic | 13:47 | |
| dtantsur | fellypefca or someone from oneview, up for a quick chat re https://bugs.launchpad.net/ironic/+bug/1716023 ? | 13:51 |
| openstack | Launchpad bug 1716023 in Ironic "OneView drivers doesn't support multitenancy in ocata" [Undecided,Triaged] | 13:51 |
| fellypefca | Hey dtantsur | 13:51 |
| ricardoas | sure, dtantsur | 13:52 |
| dtantsur | cool! do you think it's going to be easier to just add missing lines to your mixin? | 13:52 |
| dtantsur | if it's really one-two lines, it may be much less invasive | 13:52 |
| dtantsur | that's going to be an ocata-only patch, obviously | 13:52 |
| ricardoas | for sure... we thought we weren´t allowed to commit directly to stable branches | 13:53 |
| dtantsur | ricardoas: it's an exceptional situation, up to the stable team (for ironic, TheJulia, rloo and me) to accept or not | 13:53 |
| dtantsur | but if it's 5 lines of a clean stable-only fix vs hundreds of lines of a complex backport - I'd prefer it | 13:54 |
| dtantsur | please let me know what you think (ditto TheJulia and rloo) | 13:54 |
| rloo | dtantsur, fellypefca: hundreds of lines is a no unless it is urgent | 13:54 |
| rloo | dtantsur, fellypefca: is it a bug, was it meant to support multitenancy in ocata? | 13:55 |
| dtantsur | morning rloo. I think we expect all drivers that can support multitenancy to support it | 13:55 |
| rloo | morning dtantsur and fellypefca! | 13:56 |
| TheJulia | dtantsur: I'm all for minimalist, so a ocata-only patch is fine with me since this is an extenuating circumstance. | 13:57 |
| ricardoas | dtantsur rloo fellypefca: oneview mixin is missing lines 594, 595 from https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L594 | 13:57 |
| dtantsur | two lines are acceptable, assuming you do test the resulting patch manually ;) | 13:58 |
| ricardoas | dtantsur: sure... in fact we´ve been adding them manually for the past couple of months :( | 13:58 |
| dtantsur | wow | 13:59 |
| dtantsur | then please make a patch. if it's small indeed, let's go with it. | 13:59 |
| dtantsur | and sorry for all this back-and-forth with this issue :( | 13:59 |
| fellypefca | We will work on that right now :) thanks dtantsur | 13:59 |
| ricardoas | dtantsur: still in this matter, stable/newton suffers from a similar issue (that´s why agent_pxe_oneview is not working in the ci)... but I guess we should not insist on stable/newton anymore, right? | 14:00 |
| dtantsur | lemme check | 14:00 |
| dtantsur | ricardoas: newtone is still phase II (https://releases.openstack.org/) so we may consider a patch, if it's critical enough (and assuming your CI passes) | 14:01 |
| *** crushil has joined #openstack-ironic | 14:02 | |
| ricardoas | dtantsur: cool! We´ll check if it is also a case of few likes before submitting anything... :D | 14:02 |
| ricardoas | fellypefca: ^ | 14:02 |
| ricardoas | lines* | 14:02 |
| fellypefca | nice :) | 14:03 |
| dtantsur | sure :) | 14:03 |
| *** rloo has quit IRC | 14:04 | |
| *** rloo has joined #openstack-ironic | 14:04 | |
| *** rloo has quit IRC | 14:04 | |
| *** vsaienk0 has quit IRC | 14:04 | |
| *** rloo has joined #openstack-ironic | 14:04 | |
| *** vsaienk0 has joined #openstack-ironic | 14:05 | |
| dtantsur | folks, the first part of my PTG summary: http://dtantsur.github.io/posts/ironic-ptg-denver-2017-1.html | 14:07 |
| ricardoas | rloo: morning! I´ve seen you added comments on oneview docs for hardware type patch for other sections... in fact we have some updates for all sections under internal review. Is is the case for a follow up or you folks prefer us to update it all in a single step? | 14:08 |
| *** mrch has quit IRC | 14:08 | |
| *** jaganathan has joined #openstack-ironic | 14:15 | |
| *** baoli has quit IRC | 14:17 | |
| *** baoli has joined #openstack-ironic | 14:17 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Generate version_id upon add_node https://review.openstack.org/507559 | 14:27 |
| fellypefca | dtantsur Do we need a release note for this Ocata backport? | 14:29 |
| dtantsur | fellypefca: yep | 14:29 |
| fellypefca | dtantsur thanks | 14:30 |
| *** dsneddon has joined #openstack-ironic | 14:31 | |
| *** makowals has joined #openstack-ironic | 14:34 | |
| *** rbartal has quit IRC | 14:44 | |
| *** srobert has quit IRC | 14:46 | |
| openstackgerrit | Merged openstack/ironic-ui master: Set target raid configuration on a node https://review.openstack.org/483205 | 14:48 |
| *** zshi__ has quit IRC | 14:49 | |
| *** trown is now known as trown|brb | 14:50 | |
| *** ralonsoh_ is now known as ralonsoh | 14:52 | |
| *** rpioso has joined #openstack-ironic | 14:54 | |
| openstackgerrit | Merged openstack/sushy master: Add PUT method to connector https://review.openstack.org/507385 | 14:54 |
| TheJulia | anupn: You are now a member of ironic-ui-core. Congrats! | 15:00 |
| *** vsaienk0 has quit IRC | 15:06 | |
| dtantsur | congrats anupn :) | 15:09 |
| dtantsur | yuriyz: hi! your vote is missing on https://review.openstack.org/#/c/505173/ | 15:12 |
| patchbot | patch 505173 - ironic-specs - Proposal for Queens cycle priorities | 15:12 |
| dtantsur | also vsaienk0, but I guess he's out.. | 15:12 |
| yuriyz | hi dtantsur will look | 15:12 |
| rpioso | Good morning, ironicers. | 15:13 |
| dtantsur | pas-ha: do you know where vsaienk0 is? | 15:13 |
| dtantsur | more importantly, when he comes back? | 15:13 |
| dtantsur | morning rpioso | 15:13 |
| *** stendulker has joined #openstack-ironic | 15:13 | |
| rpioso | dtantsur: :) Thank you for approving and merging that python-dracclient change. It should increase the stability/correctness of the Dell 3d party CI. | 15:14 |
| *** mjura has quit IRC | 15:14 | |
| *** penick has joined #openstack-ironic | 15:15 | |
| *** causten has joined #openstack-ironic | 15:15 | |
| NobodyCam | Good Morning Ironic'ers | 15:16 |
| *** rnoriega_ is now known as rnoriega | 15:16 | |
| *** srobert has joined #openstack-ironic | 15:17 | |
| dtantsur | rpioso: I hope so! still, 8 minutes of wait scare me a bit :) | 15:18 |
| *** trown|brb is now known as trown | 15:18 | |
| *** livelace2 has quit IRC | 15:21 | |
| *** rama_y has joined #openstack-ironic | 15:22 | |
| *** tonyb has quit IRC | 15:22 | |
| vdrok | good morning rloo rpioso and NobodyCam ! | 15:22 |
| *** rama_y has quit IRC | 15:23 | |
| *** vsaienk0 has joined #openstack-ironic | 15:24 | |
| *** tonyb has joined #openstack-ironic | 15:24 | |
| stendulker | Good morning rloo, rpioso, NobodyCam, vdrok, dtantsur :) | 15:24 |
| TheJulia | good morning stendulker | 15:24 |
| stendulker | TheJulia: Good Morning :) | 15:24 |
| vdrok | morning stendulker | 15:25 |
| stendulker | One question regarding rescue interface, should it be supported _only_ as a dynamic interface or shouls also be supported as a classic driver interface? | 15:25 |
| stendulker | Spec do not mention anything related to it https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/implement-rescue-mode.html | 15:26 |
| stendulker | May be we should support on classic driver as well as this interface was defined in base class for a quiet a while, though we never had the implemention for the same. | 15:27 |
| NobodyCam | Good Morning vdrok, stendulker, TheJulia, dtantsur :) | 15:27 |
| *** e0ne has quit IRC | 15:28 | |
| *** dsneddon has quit IRC | 15:29 | |
| dtantsur | morning stendulker, NobodyCam | 15:30 |
| dtantsur | stendulker: I'd avoid touching classic drivers, if it's not technically challenging | 15:30 |
| stendulker | dtantsur: It should not be though the patches in review currently, implements it for classic drivers as well. | 15:31 |
| NobodyCam | :) | 15:31 |
| dtantsur | stendulker: we want to motivate people to move off the classic drivers ;) | 15:32 |
| stendulker | dtantsur: Will rework it to make it for dynamic drivers only | 15:32 |
| dtantsur | thanks! | 15:32 |
| TheJulia | so | 15:32 |
| TheJulia | I'm not sure that is really feasible, short of just not defining it on the older drivers, the interface still needs to exist, it is all rooted in the same loader | 15:32 |
| TheJulia | actually, that may be the only way, just not have it as an option on any classic driver, only have it as an interface with driver comp types | 15:33 |
| TheJulia | interface would still be defined, it would map to to what? no-rescue under the hood? | 15:33 |
| *** vsaienk0 has quit IRC | 15:33 | |
| stendulker | I mean we will not define them in classic drivers defined in /drivers directory | 15:33 |
| stendulker | Its 'None' currently | 15:34 |
| TheJulia | but... then None.rescue() gets called.. | 15:35 |
| stendulker | Actually, one cannot call tht. | 15:35 |
| dtantsur | TheJulia: we have this situation with inspect | 15:35 |
| dtantsur | it can be None for classic drivers. and I think RAID too | 15:35 |
| TheJulia | I think classic drivers just need to map to no-rescue and call it a day | 15:35 |
| stendulker | So we would need to change that to no-rescue. | 15:35 |
| dtantsur | TheJulia: that's an option too | 15:36 |
| stendulker | Agree. | 15:36 |
| stendulker | It would be cleaner. | 15:36 |
| TheJulia | no-rescue should raise a nice error for us if users try to use it with classic drivers | 15:36 |
| stendulker | TheJulia: yes | 15:36 |
| TheJulia | stendulker: btw, thank you for updating that patchset. I didn't see if you saw/replied to my comment about landing the coreos patchset in ipa first | 15:37 |
| TheJulia | but I finally got a chance to review that patch last night. | 15:37 |
| stendulker | TheJulia: Have added IPA CoreOS patch as a dependency to Ironic patch | 15:38 |
| stendulker | TheJulia: Anything else needed on that. Have not seen your last night comments as yet. | 15:39 |
| *** dsneddon has joined #openstack-ironic | 15:39 | |
| TheJulia | stendulker: on the IPA rev, it was just release notes related | 15:40 |
| stendulker | TheJulia: Ok. Will update tht patch shortly. | 15:41 |
| stendulker | TheJulia: Thank you for review | 15:41 |
| stendulker | TheJulia: Also had updated your patch related to capabilities https://review.openstack.org/#/c/492114/ | 15:41 |
| patchbot | patch 492114 - ironic - Change [deploy]/default_boot_option to local | 15:42 |
| stendulker | TheJulia: Hope tht is ok with you. | 15:42 |
| *** rbudden has quit IRC | 15:44 | |
| TheJulia | stendulker: that is awesome, I've been crazy busy recently :) | 15:44 |
| TheJulia | anyway, speaking of busy, I need to step away for a couple hours | 15:45 |
| stendulker | TheJulia: :) | 15:45 |
| *** jcoufal has quit IRC | 15:48 | |
| *** jcoufal_ has joined #openstack-ironic | 15:48 | |
| *** amarao has quit IRC | 15:48 | |
| *** ccamacho has quit IRC | 15:51 | |
| *** srobert has quit IRC | 15:58 | |
| *** srobert has joined #openstack-ironic | 15:59 | |
| *** jistr is now known as jistr|mtg | 16:01 | |
| *** dprince has quit IRC | 16:02 | |
| *** dprince has joined #openstack-ironic | 16:05 | |
| *** rcernin has quit IRC | 16:05 | |
| *** udesale has quit IRC | 16:08 | |
| *** jistr|mtg is now known as jistr | 16:18 | |
| *** jaganathan has quit IRC | 16:20 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-lib master: Migrate to stestr as unit tests runner https://review.openstack.org/507272 | 16:22 |
| *** trown is now known as trown|lunch | 16:23 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent master: Migrate to stestr as unit tests runner https://review.openstack.org/507270 | 16:25 |
| *** yufei has joined #openstack-ironic | 16:27 | |
| *** r-daneel has joined #openstack-ironic | 16:27 | |
| *** amarao has joined #openstack-ironic | 16:27 | |
| *** yufei has left #openstack-ironic | 16:28 | |
| *** stendulker has quit IRC | 16:31 | |
| *** marios has quit IRC | 16:33 | |
| jlvillal | rloo: Would you have time to discuss: https://review.openstack.org/#/c/507297/3/ironic/drivers/modules/ipmitool.py ? | 16:34 |
| patchbot | patch 507297 - ironic - Don't fail when ipmitool command fails if power st... | 16:34 |
| *** causten has quit IRC | 16:34 | |
| *** causten has joined #openstack-ironic | 16:34 | |
| *** rbudden has joined #openstack-ironic | 16:41 | |
| * dtantsur -> beers | 16:42 | |
| dtantsur | see you tomorrow | 16:42 |
| *** dtantsur is now known as dtantsur|afk | 16:42 | |
| *** milan has quit IRC | 16:44 | |
| fellypefca | enjoy :) dtantsur|afk | 16:46 |
| *** mrtenio has joined #openstack-ironic | 16:47 | |
| *** yufei has joined #openstack-ironic | 16:50 | |
| *** yufei has quit IRC | 16:51 | |
| *** derekh has quit IRC | 16:54 | |
| *** baoli has quit IRC | 17:01 | |
| *** baoli has joined #openstack-ironic | 17:02 | |
| rloo | hey jlvillal: should i read the comments etc first wrt 507297? | 17:08 |
| jlvillal | rloo: Sure. I think I figured out a better way. In a meeting at the moment. | 17:09 |
| rloo | jlvillal: ok, ping me when you're avail | 17:09 |
| *** rama_y has joined #openstack-ironic | 17:10 | |
| *** rcernin has joined #openstack-ironic | 17:11 | |
| *** racedo has joined #openstack-ironic | 17:14 | |
| *** amarao has quit IRC | 17:15 | |
| *** trown|lunch is now known as trown | 17:24 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver https://review.openstack.org/507632 | 17:24 |
| *** crushil has quit IRC | 17:27 | |
| *** crushil has joined #openstack-ironic | 17:27 | |
| *** rbudden has quit IRC | 17:27 | |
| *** etingof has quit IRC | 17:28 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 17:36 |
| *** noam_ has quit IRC | 17:36 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 17:37 |
| *** tesseract has quit IRC | 17:37 | |
| jlvillal | rloo: ^^^ I think that is a cleaner approach. I took your suggestions into account. | 17:38 |
| rloo | jlvillal: ok | 17:38 |
| * vdrok goes for a walk | 17:39 | |
| vdrok | good night! | 17:39 |
| jlvillal | vdrok: Good plan :) | 17:39 |
| vdrok | :) morning jlvillal | 17:39 |
| jlvillal | vdrok: Good night :) | 17:39 |
| rloo | bye vdrok! | 17:40 |
| *** vsaienk0 has joined #openstack-ironic | 17:42 | |
| pas-ha | dtantsur|afk: Vasyl is out for evening, but will come back tomorrow. pinged him by internal means anyway. | 17:44 |
| * pas-ha --> home | 17:45 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 17:45 |
| *** [1]rpioso has joined #openstack-ironic | 17:50 | |
| *** ralonsoh has quit IRC | 17:51 | |
| *** rpioso has quit IRC | 17:52 | |
| *** ijw has joined #openstack-ironic | 17:53 | |
| *** cdearborn_ is now known as cdearborn | 17:53 | |
| *** vsaienk0 has quit IRC | 17:53 | |
| *** ijw has quit IRC | 17:55 | |
| *** [1]rpioso has quit IRC | 17:57 | |
| *** ijw has joined #openstack-ironic | 17:58 | |
| *** ijw has quit IRC | 17:58 | |
| *** ijw has joined #openstack-ironic | 17:58 | |
| *** baoli has quit IRC | 18:00 | |
| *** baoli has joined #openstack-ironic | 18:01 | |
| *** jkilpatr has quit IRC | 18:03 | |
| *** jkilpatr has joined #openstack-ironic | 18:04 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Documentation for 'oneview' hardware type https://review.openstack.org/502072 | 18:08 |
| *** srobert_ has joined #openstack-ironic | 18:08 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Documentation for 'oneview' hardware type https://review.openstack.org/502072 | 18:09 |
| jlvillal | rloo: How about | 18:11 |
| jlvillal | Fixes a problem when using ipmitool and rebooting a node which would cause | 18:11 |
| jlvillal | a deploy to fail. Now when rebooting a node we check if the node is already | 18:11 |
| jlvillal | powered off, if it is we don't attempt to power off the node. This is | 18:11 |
| jlvillal | because some BMCs will error if the node is already powered off and an | 18:11 |
| jlvillal | ipmitool request is made to power them off. See | 18:11 |
| *** [1]rpioso has joined #openstack-ironic | 18:11 | |
| jlvillal | https://bugs.launchpad.net/ironic/+bug/1718794 for details. | 18:11 |
| openstack | Launchpad bug 1718794 in Ironic "Some BMC/IPMI will error if currently off and told to turn off" [High,In progress] - Assigned to John L. Villalovos (happycamp) | 18:11 |
| rloo | jlvillal: ok. s/them/it/ (in 'made to power them off') | 18:12 |
| jlvillal | rloo: thanks, will do | 18:12 |
| rloo | jlvillal: did i mention how much i hate documentation and release notes? | 18:12 |
| jlvillal | heh :) | 18:12 |
| *** srobert has quit IRC | 18:12 | |
| jlvillal | rloo: I did comment about why I used the lambda. Basically I think they are cleaner/nicer. | 18:13 |
| rloo | jlvillal: alternative is to add a new optional parameter to the reboot() call, but that affects out of tree drivers too | 18:13 |
| jlvillal | rloo: yeah, that seemed much uglier. I saw what they did for the 'timeout' parameter | 18:14 |
| rloo | jlvillal: i must have missed the lambda comment. but am i correct, it is inconsistent wrt the other mocks being used? | 18:14 |
| rloo | jlvillal: uglier, but more efficient (no need to make the power-status call again) | 18:14 |
| jlvillal | rloo: Yes it is different. But I think the better way is worth the less consistency | 18:14 |
| rloo | jlvillal: i have no idea how expensive it is to make the power-status call | 18:14 |
| rloo | jlvillal: i'd make it consistent/ugly, then submit a new patch to clean it all up | 18:15 |
| rloo | jlvillal: personally | 18:15 |
| rloo | jlvillal: makes it easier to review (for me) | 18:15 |
| rloo | jlvillal: but not going to -1 you on that | 18:15 |
| jlvillal | hmmm.... | 18:15 |
| rloo | jlvillal: but you better make it consistent later cuz if you don't, it will make it more confusing for new people/others when looking at those tests | 18:15 |
| jlvillal | rloo: I guess I find it somewhat consistent. For things we don't need to check if they are called or not, then use a lambda | 18:16 |
| rloo | jlvillal: oh, i see your comment now | 18:16 |
| rloo | jlvillal: so, why don't you want to check if it is called or not? | 18:16 |
| jlvillal | rloo: For things where we do care if they are called or not, or we need to setup side_effects then don't use the lambda | 18:16 |
| jlvillal | rloo: I guess it is always called. Not a thing where it could be or could not be. | 18:17 |
| rloo | jlvillal: i mean, i do want to know if it is being called. why don't you? | 18:17 |
| rloo | jlvillal: you could say that about any method that doesn't have any 'if's in it :) the code will be called... dunno. | 18:17 |
| rloo | jlvillal: isn't the power-on call always called too then? (based on my memory, i didn't look at code) | 18:18 |
| *** vsaienk0 has joined #openstack-ironic | 18:18 | |
| jlvillal | rloo: I guess it depends how much we want to test the implementation versus the desired result. | 18:19 |
| rloo | jlvillal: just put yourself in the shoes of someone new learning about ironic and seeing those tests. | 18:19 |
| jlvillal | rloo: I was considering having one of the tests check to see if it is called. | 18:19 |
| jlvillal | rloo: And have the rest as lambdas | 18:19 |
| rloo | jlvillal: did i also mention how much i hate tests? now I wonder what the order is, tests, docs, renos, ...? | 18:20 |
| jlvillal | :) | 18:20 |
| rloo | jlvillal: anyway, not going to comment any more about that, do what you think is best, easy to understand for future ironic'ers. | 18:20 |
| jlvillal | rloo: Thanks. I notice we don't actually mock_on, mock_off is called in at least some of the tests. | 18:21 |
| rloo | it looks to me like the gate is broken, i see a bunch of tests failing for several patches | 18:21 |
| rloo | but i *need* to get back to something so going to stick my head in the sand until I am done | 18:22 |
| jlvillal | Oh :( | 18:22 |
| *** dprince has quit IRC | 18:23 | |
| *** rbudden has joined #openstack-ironic | 18:23 | |
| -openstackstatus- NOTICE: The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information. | 18:24 | |
| *** rbudden has quit IRC | 18:26 | |
| openstackgerrit | Rushil Chugh proposed openstack/ironic-specs master: Add spec for XClarity Driver https://review.openstack.org/476922 | 18:27 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 18:27 |
| *** vsaienk0 has quit IRC | 18:28 | |
| *** e0ne has joined #openstack-ironic | 18:28 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 18:29 |
| *** dpawar has joined #openstack-ironic | 18:32 | |
| *** dpawar has quit IRC | 18:33 | |
| *** livelace has quit IRC | 18:35 | |
| *** rbudden has joined #openstack-ironic | 18:36 | |
| *** e0ne has quit IRC | 18:36 | |
| *** srobert_ has quit IRC | 18:37 | |
| *** rbudden has quit IRC | 18:38 | |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/ocata: Fix multitenancy scenario on OneView Driver https://review.openstack.org/507632 | 18:38 |
| *** fellypefca is now known as fellypefca-afk | 18:39 | |
| anupn | Hi jlvillal: so in my tinycore image with py3.3 i am getting the "no module named udev" with root user as well. | 18:44 |
| anupn | jlvillal, i think so now my env and gate env is in same state and need to figure why root user is able to find the library with py2.7 and not with py3.3? | 18:45 |
| *** dprince has joined #openstack-ironic | 18:53 | |
| *** baoli has quit IRC | 18:57 | |
| *** MattMan has quit IRC | 18:58 | |
| *** MattMan has joined #openstack-ironic | 18:59 | |
| *** baoli has joined #openstack-ironic | 18:59 | |
| *** ianw|pto is now known as ianw | 19:01 | |
| *** livelace has joined #openstack-ironic | 19:03 | |
| *** Goneri has quit IRC | 19:03 | |
| *** amarao has joined #openstack-ironic | 19:03 | |
| *** ijw has quit IRC | 19:03 | |
| *** ChubYann has joined #openstack-ironic | 19:09 | |
| *** srobert has joined #openstack-ironic | 19:22 | |
| *** srobert has quit IRC | 19:22 | |
| *** srobert has joined #openstack-ironic | 19:23 | |
| *** ijw has joined #openstack-ironic | 19:28 | |
| *** etingof has joined #openstack-ironic | 19:29 | |
| *** ijw has quit IRC | 19:33 | |
| *** rcernin has quit IRC | 19:34 | |
| *** ijw has joined #openstack-ironic | 19:35 | |
| *** ijw has quit IRC | 19:40 | |
| *** d0ugal has joined #openstack-ironic | 19:41 | |
| *** d0ugal has quit IRC | 19:41 | |
| *** d0ugal has joined #openstack-ironic | 19:41 | |
| *** jkilpatr has quit IRC | 19:50 | |
| *** baoli has quit IRC | 20:00 | |
| *** baoli has joined #openstack-ironic | 20:04 | |
| *** pcaruana has quit IRC | 20:05 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 20:06 |
| *** amarao has quit IRC | 20:08 | |
| *** amarao has joined #openstack-ironic | 20:09 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 20:21 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 20:21 |
| *** mat128 has quit IRC | 20:23 | |
| *** jkilpatr has joined #openstack-ironic | 20:26 | |
| *** radez has quit IRC | 20:32 | |
| *** Goneri has joined #openstack-ironic | 20:32 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 20:35 |
| *** baoli_ has joined #openstack-ironic | 20:35 | |
| *** radez has joined #openstack-ironic | 20:35 | |
| *** ijw has joined #openstack-ironic | 20:36 | |
| *** baoli has quit IRC | 20:36 | |
| *** penick has quit IRC | 20:39 | |
| *** crushil has quit IRC | 20:42 | |
| *** ijw has quit IRC | 20:43 | |
| *** rbudden has joined #openstack-ironic | 20:43 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 20:44 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 20:44 |
| *** penick has joined #openstack-ironic | 20:44 | |
| * jlvillal hopes he is done :) | 20:46 | |
| * jlvillal discovers he is not :( | 20:48 | |
| *** jcoufal_ has quit IRC | 20:49 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Reduce complexity of node_power_action() function https://review.openstack.org/507668 | 20:52 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: ipmitool: reboot: Don't power off node if already off https://review.openstack.org/507297 | 20:52 |
| jlvillal | Okay, all unit tests passed :) | 20:52 |
| rloo | jlvillal: i had a thought that your change won't work. guess i should actually look at it. | 20:53 |
| jlvillal | rloo: Which change is that? | 20:53 |
| rloo | jlvillal: ^^ | 20:53 |
| jlvillal | rloo: The reduce complexity? | 20:53 |
| jlvillal | rloo: The don't power off? | 20:53 |
| *** ijw has joined #openstack-ironic | 20:54 | |
| rloo | jlvillal: dunno, i didn't look but i saw some comment where you say you are moving the power-status check into the reboot() method. or something like that. or i interpreted it as that. | 20:54 |
| rloo | jlvillal: well, maybe it will work now that i think of it a bit more. but i haven't actually looked at your change. | 20:54 |
| jlvillal | rloo: Right. Yeah, I did move it into the reboot method. And have also made the node_power_action() function smarter so it only gets the power status if it can actually use that info. | 20:55 |
| rloo | jlvillal: thinking i'll maybe look tomorrow. i can't deal with too many changes to one patch. | 20:55 |
| jlvillal | rloo: No worries :) | 20:55 |
| jlvillal | I'd like to see what the CI says. | 20:55 |
| *** trown is now known as trown|outtypewww | 20:55 | |
| rloo | jlvillal: is it ok to move into the reboot? what about reboot for other drivers. | 20:55 |
| jlvillal | and check if the CI is broken for everyone | 20:55 |
| rloo | jlvillal: i think it might be ok, just not sure cuz you've moved functionality from conductor down to drivers | 20:56 |
| rloo | jlvillal: and clearly, i'm not thinking that straight now :) | 20:56 |
| jlvillal | rloo: This is an ipmitool issue. I added getting the power state in ipmitool.py. I didn't remove it from the conductor though. | 20:56 |
| rloo | jlvillal: ahhh. best that i look at the code first. | 20:57 |
| jlvillal | rloo: Thanks for the input though! I think it is much better from my starting point. | 20:57 |
| jlvillal | rloo: The power of good reviews :) | 20:58 |
| rloo | jlvillal: :) | 21:00 |
| *** dprince has quit IRC | 21:06 | |
| *** baoli_ has quit IRC | 21:07 | |
| jlvillal | We're sorry -- the Sourceforge site is currently in Disaster Recovery mode, and currently requires | 21:09 |
| jlvillal | the use of javascript to function. Please check back later. | 21:09 |
| jlvillal | Ouch, I think this is the reason our builds are failing :( | 21:09 |
| jlvillal | rloo: TheJulia: Why I think our gate is broken ^^^ | 21:11 |
| rloo | jlvillal: wow | 21:11 |
| jlvillal | Yeah :( | 21:11 |
| rloo | jlvillal: maybe update our whiteboard | 21:12 |
| jlvillal | rloo: will do | 21:12 |
| *** srobert has quit IRC | 21:12 | |
| rloo | jlvillal: thx! | 21:12 |
| *** amarao has quit IRC | 21:23 | |
| jlvillal | So sourceforge has problems: https://twitter.com/sfnet_ops | 21:25 |
| *** zhangfei has joined #openstack-ironic | 21:27 | |
| * TheJulia takes a look | 21:29 | |
| jlvillal | Sourceforge does have a problem: https://twitter.com/sfnet_ops/status/912764408643940355 | 21:29 |
| jlvillal | I guess we just need to be patient | 21:29 |
| jlvillal | As an FYI, our failure getting ipmitool tarball: http://logs.openstack.org/72/507272/3/check/gate-tempest-dsvm-ironic-lib-wholedisk-agent_ipmitool-ubuntu-xenial/6d8f17d/logs/devstacklog.txt.gz#_2017-09-26_18_01_22_738 | 21:30 |
| TheJulia | fun | 21:31 |
| TheJulia | :( | 21:31 |
| NobodyCam | :( | 21:33 |
| *** zhangfei has quit IRC | 21:42 | |
| *** r-daneel has quit IRC | 21:43 | |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Implement Redfish Sessions https://review.openstack.org/471942 | 21:43 |
| openstackgerrit | Yusef Shaban proposed openstack/sushy master: Implement Redfish Sessions https://review.openstack.org/471942 | 21:44 |
| *** d0ugal has quit IRC | 21:44 | |
| *** ijw has quit IRC | 21:49 | |
| *** cdearborn has quit IRC | 21:49 | |
| *** ijw has joined #openstack-ironic | 21:50 | |
| *** amarao has joined #openstack-ironic | 21:51 | |
| *** bfournie has quit IRC | 21:54 | |
| *** Goneri has quit IRC | 21:58 | |
| *** penick has quit IRC | 22:01 | |
| *** hjensas has quit IRC | 22:11 | |
| *** amarao has quit IRC | 22:12 | |
| *** jaypipes has quit IRC | 22:16 | |
| *** mat128 has joined #openstack-ironic | 22:24 | |
| *** Sukhdev has joined #openstack-ironic | 22:24 | |
| *** rbudden has quit IRC | 22:26 | |
| *** mat128 has quit IRC | 22:29 | |
| *** rloo has quit IRC | 22:33 | |
| *** jlabarre has quit IRC | 22:39 | |
| jlvillal | TheJulia: As a note, I think it is only the IPA gate that is broken. I updated the whiteboard. | 22:40 |
| *** causten has quit IRC | 22:41 | |
| *** causten has joined #openstack-ironic | 22:41 | |
| TheJulia | jlvillal: thanks! | 22:42 |
| *** causten has quit IRC | 22:46 | |
| *** jcoufal has joined #openstack-ironic | 23:31 | |
| *** jcoufal has quit IRC | 23:40 | |
| *** penick has joined #openstack-ironic | 23:48 | |
| *** penick has quit IRC | 23:53 | |
| *** penick has joined #openstack-ironic | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!