| *** chas has joined #openstack-ironic | 00:09 | |
| *** chas has quit IRC | 00:14 | |
| *** shobhar_ has quit IRC | 00:20 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Enable OSProfiler support in Ironic https://review.openstack.org/358142 | 00:24 |
|---|---|---|
| *** xavierr has quit IRC | 00:25 | |
| *** xavierr has joined #openstack-ironic | 00:28 | |
| *** rloo has quit IRC | 00:28 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 00:47 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV https://review.openstack.org/454243 | 00:47 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 00:47 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 00:48 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 00:48 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Logic for skipping deployment with BFV https://review.openstack.org/454243 | 00:49 |
| *** frontrunner has joined #openstack-ironic | 00:49 | |
| *** phuongnh has joined #openstack-ironic | 01:01 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/ironic master: Introduce hpOneView and ilorest to OneView https://review.openstack.org/468129 | 01:06 |
| *** hw_wutianwei has joined #openstack-ironic | 01:08 | |
| *** gcb has joined #openstack-ironic | 01:13 | |
| *** arahal_ has quit IRC | 01:32 | |
| *** ianychoi_ is now known as ianychoi | 01:35 | |
| *** baoli_ has quit IRC | 01:40 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic-python-agent master: Copy ldlinux.c32 for building ISO image https://review.openstack.org/469310 | 02:08 |
| *** chas has joined #openstack-ironic | 02:10 | |
| *** shobhar has joined #openstack-ironic | 02:13 | |
| *** chas has quit IRC | 02:15 | |
| *** jrist has quit IRC | 02:20 | |
| *** jaganathan has joined #openstack-ironic | 02:23 | |
| *** shobhar has quit IRC | 02:26 | |
| *** shobhar has joined #openstack-ironic | 02:27 | |
| *** sasha2 has quit IRC | 02:29 | |
| *** SerenaFeng has joined #openstack-ironic | 02:31 | |
| *** litao__ has joined #openstack-ironic | 02:32 | |
| *** jrist has joined #openstack-ironic | 02:34 | |
| *** shuyingya has joined #openstack-ironic | 02:41 | |
| *** tuanluong has joined #openstack-ironic | 02:46 | |
| *** shuyingy_ has joined #openstack-ironic | 02:48 | |
| *** shuyingya has quit IRC | 02:48 | |
| *** vikrant has joined #openstack-ironic | 03:15 | |
| *** vikrant is now known as vikrant|brb | 03:24 | |
| *** frontrunner has quit IRC | 03:26 | |
| *** udesale has joined #openstack-ironic | 03:26 | |
| *** SerenaFeng has quit IRC | 03:26 | |
| *** udesale has quit IRC | 03:31 | |
| *** udesale has joined #openstack-ironic | 03:31 | |
| *** SerenaFeng has joined #openstack-ironic | 03:34 | |
| *** rbudden has quit IRC | 03:38 | |
| *** vikrant|brb is now known as vikrant | 03:41 | |
| *** links has joined #openstack-ironic | 03:45 | |
| *** priteau has joined #openstack-ironic | 04:08 | |
| *** jaosorior has quit IRC | 04:10 | |
| *** chas has joined #openstack-ironic | 04:11 | |
| *** priteau has quit IRC | 04:12 | |
| *** jaosorior has joined #openstack-ironic | 04:13 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 04:14 |
| *** chas has quit IRC | 04:15 | |
| *** SerenaFeng has quit IRC | 04:16 | |
| *** zhurong has joined #openstack-ironic | 04:16 | |
| *** zhurong has quit IRC | 04:19 | |
| *** mtanino__ has joined #openstack-ironic | 04:24 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 04:27 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface https://review.openstack.org/467911 | 04:39 |
| *** mtanin___ has joined #openstack-ironic | 04:45 | |
| *** mtanino__ has quit IRC | 04:46 | |
| *** udesale has quit IRC | 04:47 | |
| *** udesale has joined #openstack-ironic | 04:48 | |
| *** adisky__ has joined #openstack-ironic | 05:03 | |
| *** mtanin___ has quit IRC | 05:07 | |
| *** shobhar has quit IRC | 05:13 | |
| *** jaganathan has quit IRC | 05:18 | |
| *** jaganathan has joined #openstack-ironic | 05:18 | |
| *** karthiks has joined #openstack-ironic | 05:23 | |
| *** SerenaFeng has joined #openstack-ironic | 05:38 | |
| *** mvpnitesh has joined #openstack-ironic | 05:41 | |
| *** karthiks has quit IRC | 05:49 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Add a note for int conversion of permission https://review.openstack.org/468778 | 05:50 |
| *** ChubYann has quit IRC | 05:58 | |
| *** dsneddon has quit IRC | 05:58 | |
| *** karthiks has joined #openstack-ironic | 06:02 | |
| *** SerenaFeng has quit IRC | 06:06 | |
| *** mvpnitesh has quit IRC | 06:08 | |
| *** jaosorior has quit IRC | 06:08 | |
| *** priteau has joined #openstack-ironic | 06:09 | |
| *** mvpnitesh has joined #openstack-ironic | 06:09 | |
| *** rcernin has joined #openstack-ironic | 06:10 | |
| *** priteau has quit IRC | 06:13 | |
| *** jaosorior has joined #openstack-ironic | 06:17 | |
| *** vgadiraj has quit IRC | 06:18 | |
| *** soliosg has quit IRC | 06:18 | |
| *** soliosg has joined #openstack-ironic | 06:18 | |
| *** SerenaFeng has joined #openstack-ironic | 06:22 | |
| *** hjensas has quit IRC | 06:23 | |
| *** vgadiraj has joined #openstack-ironic | 06:25 | |
| *** mvpnitesh has quit IRC | 06:36 | |
| *** mvpnitesh has joined #openstack-ironic | 06:36 | |
| *** e0ne has joined #openstack-ironic | 06:42 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs master: Boot management for in-band inspection https://review.openstack.org/305864 | 06:45 |
| *** Nisha_Agarwal has joined #openstack-ironic | 06:45 | |
| Nisha_Agarwal | morning Ironic'iers!!! | 06:46 |
| *** e0ne has quit IRC | 06:50 | |
| *** jaganathan has quit IRC | 06:52 | |
| *** tesseract has joined #openstack-ironic | 06:53 | |
| *** jaganathan has joined #openstack-ironic | 07:03 | |
| *** e0ne has joined #openstack-ironic | 07:04 | |
| *** Serlex has joined #openstack-ironic | 07:10 | |
| *** pcaruana has joined #openstack-ironic | 07:26 | |
| *** shuyingy_ has quit IRC | 07:27 | |
| *** shuyingya has joined #openstack-ironic | 07:27 | |
| *** ccamacho has quit IRC | 07:31 | |
| *** dtantsur|afk is now known as dtantsur | 07:31 | |
| *** e0ne has quit IRC | 07:31 | |
| *** jaganathan has quit IRC | 07:31 | |
| *** jaganathan has joined #openstack-ironic | 07:32 | |
| dtantsur | morning ironic | 07:33 |
| *** SerenaFeng has quit IRC | 07:38 | |
| *** hjensas has joined #openstack-ironic | 07:40 | |
| *** hjensas has joined #openstack-ironic | 07:40 | |
| *** racedo has joined #openstack-ironic | 07:40 | |
| pas-ha | morning ironic :) | 07:44 |
| pas-ha | morning dtantsur | 07:44 |
| *** ralonsoh has joined #openstack-ironic | 07:50 | |
| *** milan has joined #openstack-ironic | 07:52 | |
| *** aarefiev_afk is now known as aarefiev | 07:54 | |
| *** ralonsoh_ has joined #openstack-ironic | 07:54 | |
| aarefiev | morning all! | 07:54 |
| *** ralonsoh has quit IRC | 07:56 | |
| dtantsur | o/ | 07:56 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** ccamacho has joined #openstack-ironic | 08:03 | |
| *** fragatina has joined #openstack-ironic | 08:07 | |
| *** fragatina has quit IRC | 08:09 | |
| *** fragatina has joined #openstack-ironic | 08:09 | |
| *** priteau has joined #openstack-ironic | 08:10 | |
| aarefiev | milan, Nisha_Agarwal I don't like the idea about adding vmedia to inspector, ideally ironic should care about pxe introspection imo | 08:11 |
| aarefiev | what issue did you discuss? | 08:11 |
| milan | aarefiev, morning! :) | 08:11 |
| aarefiev | morning milan :) | 08:11 |
| milan | aarefiev, yeah, we discussed what about the spec | 08:11 |
| *** ralonsoh_ is now known as ralonsoh | 08:12 | |
| milan | I also prefer ironic handles the boot | 08:12 |
| milan | we concluded, if I'm not mistaken, that the spec required rephrasing | 08:12 |
| milan | TL;DR | 08:12 |
| milan | * inspector has to PXE filter a manage_boot=False flagged node to avoid collisions | 08:13 |
| *** mgoddard has joined #openstack-ironic | 08:13 | |
| milan | * the discovery feature is incompatible with vmedia unless manually set up in node BIOS | 08:13 |
| milan | aarefiev, ^ | 08:13 |
| milan | and morning dtantsur and all Ironic! ;) | 08:14 |
| dtantsur | o/ | 08:14 |
| *** priteau has quit IRC | 08:14 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| aarefiev | milan: yep, what exactly collisions do you mean ? | 08:15 |
| dtantsur | milan: actually, it may be a bit more complex.. we still need to provide DHCP to nodes, just not PXE | 08:15 |
| lucasagomes | morning all | 08:15 |
| dtantsur | hey-hey lucasagomes | 08:15 |
| milan | morning lucasagomes! :) | 08:15 |
| aarefiev | morning lucasagomes | 08:15 |
| lucasagomes | o/ | 08:16 |
| milan | aarefiev, dtantsur I agree, we just have to filter PXE on all the ironic ports, even those that are tagged manage_boot=False in inspector | 08:16 |
| aarefiev | milan: in this case should be any collisions till we use diff dhcp agents in ironic and inspector | 08:18 |
| *** priteau has joined #openstack-ironic | 08:19 | |
| aarefiev | sorry, *should not be | 08:19 |
| milan | aarefiev, you mean IPA-side? | 08:19 |
| aarefiev | milan: I'm trying to figure out what collisions do you mean :) | 08:20 |
| *** fxpester has joined #openstack-ironic | 08:20 | |
| milan | ah :D | 08:21 |
| milan | the same we prevent by iptables currently | 08:21 |
| aarefiev | milan: and if you always blacklist the node | 08:22 |
| milan | so my point is: we just won't set PXE boot on a node but we will maintain the filtering for the nodes ports no matter the manage_boot flag | 08:22 |
| vdrok | good morning all, Nisha_Agarwal pas-ha aarefiev dtantsur milan lucasagomes ! | 08:22 |
| milan | aarefiev, exactly | 08:22 |
| milan | morning vdrok! :) | 08:22 |
| Nisha_Agarwal | vdrok, o/ | 08:22 |
| * Nisha_Agarwal reading the discussion | 08:22 | |
| milan | aarefiev, also we'd white-list only nodes with manage_boot=True | 08:23 |
| aarefiev | milan: ahh, ok, sounds good | 08:23 |
| aarefiev | morning vdrok :) | 08:23 |
| milan | morning Nisha_Agarwal, pas-ha, vdrok! :) | 08:23 |
| Nisha_Agarwal | morning dtantsur lucasagomes aarefiev vdrok milan :) | 08:23 |
| *** chas has joined #openstack-ironic | 08:24 | |
| aarefiev | o/ | 08:24 |
| Nisha_Agarwal | morning pas-ha | 08:24 |
| milan | aarefiev, cool :) so I guess all's good, just the spec needs a bit rephrasing | 08:24 |
| Nisha_Agarwal | milan, i posted the spec today | 08:24 |
| Nisha_Agarwal | see if it addresses all the concerns | 08:24 |
| milan | Nisha_Agarwal, cool! thanks I'll check it out | 08:24 |
| milan | aarefiev, also this should play nicely w/ the filtering reform | 08:25 |
| milan | well I wish it does :D | 08:25 |
| aarefiev | yeah, it should :) | 08:25 |
| Nisha_Agarwal | aarefiev, it would help if you also review the spec | 08:25 |
| aarefiev | sure, I'll take a look | 08:25 |
| Nisha_Agarwal | https://review.openstack.org/305864 | 08:26 |
| patchbot | patch 305864 - ironic-specs - Boot management for in-band inspection | 08:26 |
| *** amarao has joined #openstack-ironic | 08:27 | |
| * milan coffee time O:-) | 08:28 | |
| *** kromanenko has joined #openstack-ironic | 08:30 | |
| *** jaganathan has quit IRC | 08:32 | |
| dtantsur | milan: I think you may underestimate the problem. if we blacklist manage_boot=false nodes completely, they won't receive DHCP, so they won't post back to inspector, no? | 08:32 |
| *** chas has quit IRC | 08:35 | |
| *** chas has joined #openstack-ironic | 08:35 | |
| *** chas has quit IRC | 08:40 | |
| aarefiev | dtantsur: yes, that is what we talked about | 08:40 |
| *** aolwas has joined #openstack-ironic | 08:41 | |
| *** jaganathan has joined #openstack-ironic | 08:42 | |
| aarefiev | dtantsur: as I understand milan was asking for pointing this in spec | 08:43 |
| *** chas has joined #openstack-ironic | 08:44 | |
| *** derekh has joined #openstack-ironic | 08:44 | |
| *** chas has quit IRC | 08:48 | |
| *** chas has joined #openstack-ironic | 08:50 | |
| *** udesale__ has joined #openstack-ironic | 08:52 | |
| *** MattMan_ has joined #openstack-ironic | 08:54 | |
| *** alezil_ has joined #openstack-ironic | 08:54 | |
| *** rama_ has joined #openstack-ironic | 08:54 | |
| *** rama_ is now known as Guest62718 | 08:55 | |
| *** chas has quit IRC | 08:55 | |
| *** hwoarang_ has joined #openstack-ironic | 08:55 | |
| *** ptx0_ has joined #openstack-ironic | 08:56 | |
| *** strigazi_ has joined #openstack-ironic | 08:56 | |
| *** yibo_ has joined #openstack-ironic | 08:56 | |
| milan | aarefiev, dtantsur yeah, that's the limitation we need to mention in the spec | 08:58 |
| milan | it may be OK for some people though | 08:58 |
| *** aNuposic has joined #openstack-ironic | 08:58 | |
| *** oomichi_ has joined #openstack-ironic | 08:59 | |
| *** dtantsur_ has joined #openstack-ironic | 09:00 | |
| *** zzzeek_ has joined #openstack-ironic | 09:00 | |
| *** radez` has joined #openstack-ironic | 09:00 | |
| *** root has joined #openstack-ironic | 09:00 | |
| *** mmedvede_ has joined #openstack-ironic | 09:00 | |
| *** johnthetubaguy_ has joined #openstack-ironic | 09:00 | |
| *** dmellado_ has joined #openstack-ironic | 09:00 | |
| *** root is now known as Guest42630 | 09:00 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic-specs master: Better driver interface isolation https://review.openstack.org/468000 | 09:00 |
| *** marlinc_ has joined #openstack-ironic | 09:00 | |
| *** zzzeek has quit IRC | 09:01 | |
| *** udesale has quit IRC | 09:01 | |
| *** links has quit IRC | 09:01 | |
| *** vikrant has quit IRC | 09:01 | |
| *** Guest56461 has quit IRC | 09:01 | |
| *** hwoarang has quit IRC | 09:01 | |
| *** MattMan_1 has quit IRC | 09:01 | |
| *** athomas has quit IRC | 09:01 | |
| *** alezil has quit IRC | 09:01 | |
| *** ptx0 has quit IRC | 09:01 | |
| *** yibo has quit IRC | 09:01 | |
| *** marlinc has quit IRC | 09:01 | |
| *** mmedvede has quit IRC | 09:01 | |
| *** radez has quit IRC | 09:01 | |
| *** dmellado has quit IRC | 09:01 | |
| *** oomichi has quit IRC | 09:01 | |
| *** anupn has quit IRC | 09:01 | |
| *** strigazi has quit IRC | 09:01 | |
| *** johnthetubaguy has quit IRC | 09:01 | |
| *** jistr has quit IRC | 09:01 | |
| *** dtantsur has quit IRC | 09:01 | |
| *** mmedvede_ is now known as mmedvede | 09:01 | |
| *** dtantsur_ is now known as dtantsur | 09:01 | |
| *** oomichi_ is now known as oomichi | 09:01 | |
| *** alezil_ is now known as alezil | 09:01 | |
| Nisha_Agarwal | dtantsur, aarefiev milan will come back to spec in a minute. meanwhile i need review from dtantsur on https://review.openstack.org/467538 | 09:01 |
| patchbot | patch 467538 - openstack-infra/project-config - disable swift for py3 enablement in ironic | 09:01 |
| *** SerenaFeng has joined #openstack-ironic | 09:01 | |
| Nisha_Agarwal | dtantsur, please could you review the above patch | 09:02 |
| Nisha_Agarwal | dtantsur, Andreas says ironic liasons need to review that first | 09:02 |
| *** jistr has joined #openstack-ironic | 09:02 | |
| dtantsur | Nisha_Agarwal: I don't this it's what they asked you on review, no? | 09:02 |
| *** marlinc_ is now known as marlinc | 09:03 | |
| Nisha_Agarwal | <AJaeger> Nisha_Agarwal: dtroyer's change is not final yet. I think we can take your change and change later as well. | 09:03 |
| Nisha_Agarwal | <Nisha_Agarwal> AJaeger, thanks | 09:03 |
| Nisha_Agarwal | <AJaeger> Nisha_Agarwal: But first, get yourself a review by the ironic infra liaisons. | 09:03 |
| Nisha_Agarwal | <Nisha_Agarwal> AJaeger, sure | 09:03 |
| Nisha_Agarwal | <AJaeger> Nisha_Agarwal: https://wiki.openstack.org/wiki/CrossProjectLiaisons#Infra | 09:03 |
| Nisha_Agarwal | dtantsur, ^^^^ | 09:03 |
| *** _milan_ has joined #openstack-ironic | 09:03 | |
| *** Guest81691 has quit IRC | 09:03 | |
| Nisha_Agarwal | Andreas asked me initially to wait for dtroyer's change to get merged. but today he said above | 09:04 |
| *** adreznec has quit IRC | 09:04 | |
| *** milan has quit IRC | 09:04 | |
| *** adreznec has joined #openstack-ironic | 09:05 | |
| dtantsur | ok, +1 | 09:06 |
| Nisha_Agarwal | dtantsur, thanks | 09:07 |
| *** links has joined #openstack-ironic | 09:07 | |
| *** athomas has joined #openstack-ironic | 09:07 | |
| Nisha_Agarwal | dtantsur, now coming to the spec | 09:07 |
| Nisha_Agarwal | i read your comment " I think you may underestimate the problem. if we blacklist manage_boot=false nodes completely, they won't receive DHCP, so they won't post back to inspector, no?" | 09:07 |
| Nisha_Agarwal | dtantsur, ^^^ | 09:07 |
| Nisha_Agarwal | is this true? | 09:07 |
| *** vikrant has joined #openstack-ironic | 09:08 | |
| Nisha_Agarwal | dtantsur, aarefiev _milan_ when we say manage_boot = false then inspector controls the boot device of the nodes...isnt it? | 09:09 |
| dtantsur | well, I think so. who's going to provide us DHCP, if we disable inspector's DHCP handling | 09:09 |
| dtantsur | ? | 09:09 |
| dtantsur | Nisha_Agarwal: no, inspector only powers it on in this case. | 09:10 |
| _milan_ | in my understanding it would revolve around https://github.com/openstack/ironic-inspector/blob/master/ironic_inspector/introspect.py#L156 | 09:11 |
| Nisha_Agarwal | dtantsur, ohk manage_boot = false means ironic controls the node | 09:11 |
| dtantsur | yes, this was the original intention | 09:11 |
| Nisha_Agarwal | then DHCP is required \ | 09:12 |
| dtantsur | then it's not clear if we should create ports in neutron or we should still leave inspector do DHCP (but not PXE) | 09:12 |
| dtantsur | _milan_: ^^^ | 09:12 |
| _milan_ | dtantsur, hmm so we won't be able to set the pxe_boot flag on any port | 09:13 |
| _milan_ | yeah | 09:13 |
| dtantsur | right, this is also a problem | 09:13 |
| _milan_ | another limitation to mention in the spec? | 09:13 |
| dtantsur | but this is solvable, I'm more worried about DHCP | 09:13 |
| _milan_ | dtantsur, you mean that the node would boot but all the interfaces will be uncofigured | 09:16 |
| dtantsur | _milan_: yes, which will prevent it from posting data back | 09:16 |
| _milan_ | dtantsur, excellent point! | 09:16 |
| Nisha_Agarwal | dtantsur, _milan_ in my POC using manage_boot it does posts the data back to inspector | 09:18 |
| dtantsur | Nisha_Agarwal: how do you do DHCP? | 09:18 |
| *** jaosorior is now known as jaosorior_lunch | 09:19 | |
| Nisha_Agarwal | dtantsur, we use inspector-dhcp service for dhcp | 09:19 |
| openstackgerrit | Hironori Shiina proposed openstack/python-ironicclient master: Add OSC commands for volume connector https://review.openstack.org/427053 | 09:19 |
| dtantsur | ok, so essentially you still use PXE, right? | 09:19 |
| Nisha_Agarwal | dtantsur, because for OOB inspection it doesnt require neutron/DHCP | 09:20 |
| Nisha_Agarwal | dtantsur, i did vmedia boot...how it is PXE? | 09:20 |
| _milan_ | Nisha_Agarwal, so you essentially used inspector-dhcp to provide DHCP, correct? | 09:22 |
| _milan_ | * but no PXE boot | 09:22 |
| Nisha_Agarwal | _milan_, yes | 09:23 |
| dtantsur | Nisha_Agarwal: so, you modified dnsmasq.conf that inspector uses to not provide PXE, right? | 09:23 |
| Nisha_Agarwal | no PXe boot | 09:23 |
| dtantsur | how does it work when some nodes use PXE and some - vmedia? | 09:23 |
| _milan_ | Nisha_Agarwal, did you guys test a mixed vmedia--pxe boot introspection environment? | 09:24 |
| dtantsur | yeah, this ^^^ is my concern | 09:25 |
| dtantsur | with the work _milan_ is doing we may be able to provide different options to different nodes, but I'm not sure | 09:25 |
| Nisha_Agarwal | dtantsur, i did some inspector.conf changes (manage_firewall=false, add_ports set to active) | 09:25 |
| dtantsur | Nisha_Agarwal: manage_firewall=false will prevent nodes requiring PXE from inspecting | 09:26 |
| _milan_ | Nisha_Agarwal, I guess that's a simple yes/no question so my conclusion is no? | 09:26 |
| _milan_ | dtantsur, right, we'd have to delegate the DHCP to "some other" dhcp | 09:27 |
| _milan_ | another limitation for the spec? | 09:27 |
| Nisha_Agarwal | dtantsur, in dnsmasq.conf i changed "interface" to br-eth1 | 09:27 |
| dtantsur | _milan_: this is a critical limitation IMO | 09:28 |
| Nisha_Agarwal | dtantsur, _milan_ i tested vmedia boot and pxe_ilo with pxe but one by one in the same env | 09:28 |
| _milan_ | dtantsur, I agree | 09:28 |
| dtantsur | Nisha_Agarwal: without modifications? how did PXE work with manage_firewall=false? | 09:28 |
| dtantsur | or, if you just enabled DHCP for all nodes: how does it co-exist with neutron for deployment? | 09:29 |
| Nisha_Agarwal | dtantsur, i need to modify "interface" from br-inspector to "br-eth1" in my env which is used by neutron | 09:29 |
| Nisha_Agarwal | which means i need to restart inspector services | 09:30 |
| Nisha_Agarwal | dhcp and inspector service both | 09:30 |
| Nisha_Agarwal | when i restart inspector service it messes up with neutron iptables rules | 09:31 |
| Nisha_Agarwal | with interface change | 09:31 |
| Nisha_Agarwal | which leads to restacking :( | 09:31 |
| dtantsur | Nisha_Agarwal: I'm sorry, you don't seem to answer my question. The question is: how does the following things work in one environment without any configuration changes: 1. inspection with PXE, 2. inspection without PXE, 3. deploy with PXE? | 09:31 |
| Nisha_Agarwal | thats the main reason i did manage_firewall = false | 09:31 |
| dtantsur | I'd like a detailed answer to this question in the spec we're discussing | 09:32 |
| Nisha_Agarwal | dtantsur, to answer above only i told above reasons. even for pxe_ilo to work, we need to set up manage_firewall = false because i need to change the interface | 09:33 |
| dtantsur | this does not look right, we've been using devstack with it for quite some time in the CI | 09:34 |
| Nisha_Agarwal | because when we have manage_firewall = true, it tries to recreate the iptables rules | 09:34 |
| dtantsur | of course it does, this is how it works | 09:34 |
| dtantsur | preventing it from doing that has unexpected consequences on deployment | 09:35 |
| Nisha_Agarwal | dtantsur, when it recreates the iptables rules it messes up neutron | 09:35 |
| Nisha_Agarwal | neutron is working on br-eth1 | 09:35 |
| Nisha_Agarwal | or u can say our main interface is br-eth1 | 09:36 |
| Nisha_Agarwal | with which we connect to the baremetal | 09:36 |
| Nisha_Agarwal | and get data | 09:36 |
| dtantsur | well, I'm not sure, I just know that the default devstack setting work. it may not be the case for devstack+baremetal, of course | 09:36 |
| Nisha_Agarwal | dtantsur, thats exactly i have seen | 09:36 |
| dtantsur | but disabling iptables is not an answer. you're likely to need a bridge connected to br-eth1 or someting | 09:36 |
| Nisha_Agarwal | so when i do in baremetal env, it requires me to chnage the interface to the one which my devstack uses to connect to baremetal | 09:37 |
| dtantsur | I mean, the behavior you see is a misconfiguration, not a normal mode of operation. it may be useful to figure out why it happens *before* jumping into dangerous modes of operating ironic-inspector | 09:37 |
| dtantsur | manage_firewall=false is mostly designed for environments without neutron | 09:37 |
| Nisha_Agarwal | br-inspector doesnt work in our case | 09:38 |
| Nisha_Agarwal | even if change the range to the one we use | 09:39 |
| Nisha_Agarwal | dtantsur, is the inspector not tested on actual baremetals along with ironic? | 09:40 |
| Nisha_Agarwal | your statement above where you say you are not sure it works with devstack+baremetal looks like its not tested in that env | 09:41 |
| dtantsur | Nisha_Agarwal: we've been using it in production for years | 09:43 |
| dtantsur | and we've been running a virtual CI for years | 09:43 |
| dtantsur | this is why I suspect there may some kind of misconfiguration in your environment | 09:43 |
| *** sambetts|afk is now known as sambetts | 09:44 | |
| *** jaganathan has quit IRC | 09:45 | |
| *** dmellado_ is now known as dmellado | 09:46 | |
| *** shuyingya has quit IRC | 09:48 | |
| *** shuyingya has joined #openstack-ironic | 09:49 | |
| *** jaganathan has joined #openstack-ironic | 09:58 | |
| Nisha_Agarwal | dtantsur, what configuration do u use when u use devstack + inspector? u dont require any chnage? | 10:01 |
| dtantsur | Nisha_Agarwal: for VMs - no. I don't use devstack for bare metal, rather TripleO. It's configuration is quite similar though: we have a bridge br-ctlplane which inspector uses | 10:02 |
| Nisha_Agarwal | dtantsur, ok let me look at tripleo then. could you point me to the code for br-ctlplane...just to make my work easy :) | 10:04 |
| *** Nisha_ has joined #openstack-ironic | 10:08 | |
| *** Nisha_Agarwal has quit IRC | 10:11 | |
| *** Nisha_ has quit IRC | 10:12 | |
| *** links has quit IRC | 10:16 | |
| *** jaganathan has quit IRC | 10:21 | |
| *** jaganathan has joined #openstack-ironic | 10:21 | |
| *** e0ne has joined #openstack-ironic | 10:22 | |
| openstackgerrit | Merged openstack/ironic-lib master: Add 'sgdisk -Z' to destroy_disk_metadata https://review.openstack.org/469155 | 10:25 |
| *** tuanluong has quit IRC | 10:27 | |
| *** hshiina has joined #openstack-ironic | 10:30 | |
| *** links has joined #openstack-ironic | 10:36 | |
| *** vikrant has quit IRC | 10:40 | |
| *** jaosorior_lunch is now known as jaosorior | 10:41 | |
| *** chas has joined #openstack-ironic | 10:42 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Allow setting {provisioning,cleaning}_network in node driver_info https://review.openstack.org/460564 | 10:43 |
| *** mvpnitesh has quit IRC | 10:44 | |
| *** mvpnitesh has joined #openstack-ironic | 10:44 | |
| *** racedo has quit IRC | 10:47 | |
| *** chas has quit IRC | 10:47 | |
| *** jkilpatr has quit IRC | 10:47 | |
| *** chas has joined #openstack-ironic | 10:49 | |
| *** chas has quit IRC | 10:53 | |
| *** chas has joined #openstack-ironic | 10:55 | |
| *** chas has quit IRC | 10:59 | |
| *** jkilpatr has joined #openstack-ironic | 11:00 | |
| *** rbartal_ has joined #openstack-ironic | 11:06 | |
| *** vikrant has joined #openstack-ironic | 11:07 | |
| *** phuongnh has quit IRC | 11:08 | |
| *** amarao has quit IRC | 11:16 | |
| *** amarao has joined #openstack-ironic | 11:17 | |
| *** rbartal_ has quit IRC | 11:20 | |
| *** SerenaFeng has quit IRC | 11:23 | |
| *** bfournie has quit IRC | 11:43 | |
| *** mvpnitesh has left #openstack-ironic | 11:52 | |
| *** mvpnitesh has joined #openstack-ironic | 11:52 | |
| *** ricardoas has quit IRC | 11:52 | |
| mvpnitesh | hi sambetts, can you please look into https://bugs.launchpad.net/ironic/+bug/1694693 | 11:53 |
| openstack | mvpnitesh: Error: malone bug 1694693 not found | 11:53 |
| sambetts | mvpnitesh: that link doesn't work | 11:53 |
| _milan_ | aarefiev, qq https://review.openstack.org/#/c/439603/11/ironic_inspector/pxe_filter/base.py wdyt about renaming driver_manager() to _driver_manager() and having a driver() function? | 11:54 |
| patchbot | patch 439603 - ironic-inspector - PXE boot filtering drivers | 11:54 |
| *** jaganathan has quit IRC | 11:54 | |
| *** _milan_ is now known as milan | 11:54 | |
| * milan renamed, something weird w/ IRC today or is it just me/ | 11:54 | |
| milan | ? | 11:54 |
| mvpnitesh | Sambetts: please refresh it | 11:55 |
| *** racedo has joined #openstack-ironic | 11:55 | |
| mvpnitesh | Sambetts: i've a doubt, We have an ironic node whose provision state is active. Now if we try to delete the port which is attached to that node, it is getting deleting. Is it an expected behaviour ? | 11:56 |
| aarefiev | milan: make sense, I don't think we will need manager outside, anyway it would be easy to fix | 11:57 |
| milan | aarefiev, yup, will update in a minute | 11:57 |
| milan | aarefiev, btw any other things? | 11:57 |
| aarefiev | milan: what about magic number 42 | 11:59 |
| aarefiev | you have config option with default value, why do you need it | 11:59 |
| sambetts | mvpnitesh: I don't think we have code that checks if that should be possible or not, however I expect that it shouldn't be unless the node is in maintenance to prevent any mistakes being made | 12:00 |
| *** frontrunner has joined #openstack-ironic | 12:00 | |
| mvpnitesh | ok | 12:00 |
| milan | aarefiev, hmmmmm, cause the periodics decorator doesn't accept 0 even if the periodic task is switched off | 12:03 |
| *** chlong has quit IRC | 12:03 | |
| *** udesale__ has quit IRC | 12:03 | |
| milan | so maybe use float(inf) in the sense of 0? | 12:03 |
| mvpnitesh | Sambetts: I guess it should not delete that port. If some application server is running on the baremetal and if someone deletes the port ,then that application server will be down. | 12:03 |
| milan | aarefiev, ^^ | 12:03 |
| * milan uses 0 default config and float(inf) in patch #15 | 12:04 | |
| patchbot | No data found for patch 15 | 12:04 |
| mvpnitesh | Sambetts: I guess we should be able to delete the port which is associated to node, either when the node is in maintainance mode or when it is in avilable state | 12:05 |
| sambetts | mvpnitesh: we have logic in place in the update_port code for which states to allow port changes and those which we don't so I think that it would be worth copying it that | 12:06 |
| *** jaganathan has joined #openstack-ironic | 12:06 | |
| aarefiev | milan: I see, please add a note | 12:07 |
| milan | aarefiev, will do, thanks | 12:07 |
| *** trown|outtypewww is now known as trown | 12:07 | |
| * milan assembles patches | 12:07 | |
| aarefiev | thanks milan | 12:07 |
| *** lucasagomes is now known as lucas-hungry | 12:09 | |
| mvpnitesh | Sambetts: Thanks, i'll look into this | 12:09 |
| *** links has quit IRC | 12:11 | |
| *** fellypefca has joined #openstack-ironic | 12:13 | |
| *** litao__ has quit IRC | 12:21 | |
| *** bfournie has joined #openstack-ironic | 12:22 | |
| *** ricardoas has joined #openstack-ironic | 12:23 | |
| *** vikrant has quit IRC | 12:23 | |
| *** dprince has joined #openstack-ironic | 12:29 | |
| *** shuyingya has quit IRC | 12:39 | |
| fellypefca | Morning, Ironic :) | 12:42 |
| *** chas has joined #openstack-ironic | 12:42 | |
| *** hshiina has quit IRC | 12:43 | |
| mgoddard | hi sambetts | 12:43 |
| *** rbudden has joined #openstack-ironic | 12:43 | |
| sambetts | hey mgoddard | 12:44 |
| mgoddard | sambetts: I'm looking at adding a create_port method to the conductor manager to handle port/portgroup/physnet validation in a synchronised manner, as discussed on the review | 12:45 |
| mgoddard | sambetts: the diff for this piece is growing quite large and I'm concerned that if I push to the existing review I'm going to (reasonably) get a 'please split this ps' comment | 12:46 |
| sambetts | yeah... tbh it was quite close to that anyway :/ | 12:46 |
| mgoddard | sambetts: any thoughts on whether I should make one or more preemptive splits? there are at least 3 separable changes in there | 12:46 |
| mgoddard | sambetts: 1. DB/object layer | 12:46 |
| mgoddard | sambetts: 2. conductor validation | 12:47 |
| *** chas_ has joined #openstack-ironic | 12:47 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Unit test framework for Ironic-UI API service https://review.openstack.org/469459 | 12:47 |
| *** chas has quit IRC | 12:47 | |
| mgoddard | sambetts: 2a: make a simple (no physnet) create_port method in conductor manager with associated unit test cruft changes | 12:48 |
| mgoddard | sambetts: 3. API & tempest tests | 12:48 |
| *** adarazs has joined #openstack-ironic | 12:48 | |
| *** athomas has quit IRC | 12:48 | |
| sambetts | mgoddard: we normally split a change like this along the API lines, so 1 patch DB, 1 patch conductor api changes and backend changes, 1 patch rest api | 12:48 |
| sambetts | so that sounds like you've got the right idea | 12:48 |
| *** athomas has joined #openstack-ironic | 12:49 | |
| mgoddard | sambetts: right. I considered that initially but back then it was a lot less code so went with no to keep it atomic :) | 12:49 |
| sambetts | it would certainly make it easier to review (although I understand the pain associated with maintaining a patch chain) | 12:49 |
| sambetts | yeah I agree, thats why I didn't push when I inital reviewed it | 12:49 |
| mgoddard | sambetts: I'd rather get it done sooner if it needs to be done. Is is normal to merge some parts of the chain before others? I'm sure some pieces will not be contentious | 12:50 |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Unit test framework for Ironic-UI API service https://review.openstack.org/469459 | 12:50 |
| sambetts | mgoddard: if split down the API lines we can merge parts before others, or we can put a -2 WIP on the bottom of the chain to allow the rest to be reviewed and approved for a bulk merge | 12:51 |
| *** sasha2 has joined #openstack-ironic | 12:52 | |
| *** trown is now known as trown|brb | 12:52 | |
| *** chas_ has quit IRC | 12:52 | |
| mgoddard | sambetts: got it, thanks. I'll get onto this this afternoon | 12:53 |
| sambetts | thanks! | 12:53 |
| *** trown|brb is now known as trown | 13:03 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 13:04 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 13:04 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 13:04 |
| *** mat128 has joined #openstack-ironic | 13:04 | |
| milan | aarefiev, ^^^ | 13:04 |
| *** chas has joined #openstack-ironic | 13:05 | |
| *** cdearborn has joined #openstack-ironic | 13:05 | |
| *** Goneri has joined #openstack-ironic | 13:07 | |
| *** dtantsur is now known as dtantsur|brb | 13:08 | |
| *** lucas-hungry is now known as lucasagomes | 13:08 | |
| *** mat128 has quit IRC | 13:08 | |
| *** baoli has joined #openstack-ironic | 13:09 | |
| *** baoli has quit IRC | 13:10 | |
| *** cdearborn__ has joined #openstack-ironic | 13:10 | |
| *** chas has quit IRC | 13:14 | |
| *** mvpnitesh has quit IRC | 13:14 | |
| *** strigazi_ is now known as strigazi | 13:15 | |
| *** lifeless has quit IRC | 13:15 | |
| *** chas has joined #openstack-ironic | 13:17 | |
| *** rloo has joined #openstack-ironic | 13:19 | |
| fellypefca | Hey, we are facing some problems with jenkins? | 13:19 |
| *** chas has quit IRC | 13:20 | |
| mjturek | fellypefca: https://etherpad.openstack.org/p/IronicWhiteBoard says that SNMP gate is failing. That what you're seeing? | 13:21 |
| fellypefca | Yeah, thanks for the help mjturek | 13:22 |
| mjturek | npp | 13:23 |
| *** cdearborn has quit IRC | 13:25 | |
| *** superdan is now known as dansmith | 13:26 | |
| fellypefca | Oh, actually is not. I noticed that the patches that ran this morning had some problems with Jenkins. Even those who got +1 for workflow. | 13:27 |
| mjturek | ohhh, fellypefca do you have an example patch? | 13:28 |
| *** chlong has joined #openstack-ironic | 13:30 | |
| *** mat128 has joined #openstack-ironic | 13:30 | |
| *** shuyingya has joined #openstack-ironic | 13:30 | |
| fellypefca | mjturek 465484, 468778, 462982 and 468925 for example | 13:31 |
| *** lifeless has joined #openstack-ironic | 13:33 | |
| *** aparnav has joined #openstack-ironic | 13:33 | |
| *** baoli has joined #openstack-ironic | 13:33 | |
| rloo | hi and morning all | 13:34 |
| mjturek | gm rloo! | 13:34 |
| *** adarazs has left #openstack-ironic | 13:34 | |
| rloo | fellypefca: yes, looks like there is a gate failure. am starting to look into it now. something to do with docs | 13:34 |
| fellypefca | Morning, rloo :) | 13:34 |
| *** shuyingya has quit IRC | 13:34 | |
| mjturek | yeah it's a warning being treated as an error | 13:34 |
| fellypefca | Oh, ok | 13:35 |
| rloo | mjturek: which makes me think someone turned some switch on | 13:35 |
| mjturek | :) | 13:35 |
| *** crushil has joined #openstack-ironic | 13:35 | |
| *** baoli has quit IRC | 13:35 | |
| mjturek | sounds likely | 13:35 |
| *** baoli has joined #openstack-ironic | 13:37 | |
| fellypefca | Weird that my patch had a bug in uefi-pxe-ipmitool-tinyipa-ubuntu-xenial at Gate, but it passed at Check. :/ | 13:39 |
| sambetts | sounds like a race or timeout issue | 13:39 |
| *** zhurong has joined #openstack-ironic | 13:41 | |
| *** chlong has quit IRC | 13:43 | |
| *** krtaylor has quit IRC | 13:46 | |
| *** chlong has joined #openstack-ironic | 13:46 | |
| fellypefca | Just a little doubt, if I i put a recheck, will Jenkins Gate run again too? | 13:50 |
| *** mtanino__ has joined #openstack-ironic | 13:53 | |
| *** krtaylor has joined #openstack-ironic | 13:54 | |
| *** krtaylor has quit IRC | 13:55 | |
| mjturek | fellypefca: yeah it should. Are you trying to run something outside of Jenkins? Third party CIs watch for a regex | 13:59 |
| mjturek | ie for powerkvm it's recheck: pkvm | 13:59 |
| mjturek | fellypefca: but I wouldn't recheck jenkins until gate issues are fixed | 14:00 |
| fellypefca | mjturek oh, already did it :/ my bad | 14:01 |
| mjturek | heh, no worries :) basically if there's a gate failure listed on the whiteboard, it means that you shouldn't recheck jenkins until it's resolved (it'll more than likely fail and waste some resources) | 14:02 |
| *** hecliunyx has quit IRC | 14:02 | |
| fellypefca | mjturek Ok! Thanks again for the help :) | 14:04 |
| *** cdearborn__ is now known as cdearborn | 14:05 | |
| openstackgerrit | Peter Piela proposed openstack/ironic-ui master: Consolidate code for setting node maintenance state https://review.openstack.org/465972 | 14:14 |
| *** zhurong has quit IRC | 14:35 | |
| NobodyCam | Good morning ironic'ers. Coffee time | 14:37 |
| *** chas has joined #openstack-ironic | 14:46 | |
| *** rpioso has joined #openstack-ironic | 14:50 | |
| *** chas has quit IRC | 14:50 | |
| *** dsneddon has joined #openstack-ironic | 14:51 | |
| rloo | dtantsur|brb, others: wrt the broken doc gate job, I opened a bug: https://bugs.launchpad.net/ironic/+bug/1694724 | 14:51 |
| openstack | Launchpad bug 1694724 in Ironic "Warning for ironic.drivers.modules.deploy_utils" [Critical,New] | 14:51 |
| rloo | dtantsur|brb: i don't know how to fix it, it all looks ok to me. i'm guessing it has to do with new sphinx version cuz this only started to happen | 14:51 |
| rloo | dtantsur|brb: i need to work on other stuff. the only short-term/quick thing i can think of is to turn off sphinx's warning-is-error flag. if others think that is ok, i can do that. | 14:52 |
| *** baoli has quit IRC | 14:52 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC https://review.openstack.org/462861 | 14:54 |
| openstackgerrit | Merged openstack/ironic stable/newton: python-oneviewclient version bump https://review.openstack.org/468925 | 14:55 |
| *** dtantsur|brb is now known as dtantsur | 14:58 | |
| dtantsur | morning rloo, NobodyCam, mjturek, fellypefca | 14:58 |
| mjturek | morning dtantsur | 14:58 |
| NobodyCam | Good Morning dtantsur, mjturek and rloo :) | 14:59 |
| *** arahal_ has joined #openstack-ironic | 14:59 | |
| dtantsur | rloo: there was (I think) a fix for this problem, I'll check after today's meeting | 14:59 |
| rloo | hi dtantsur. wrt ^^, my other thought is that requirements is blocking sphinx 1.6.1, so maybe we want to block 1.6.2 too... | 14:59 |
| rloo | dtantsur: ah, ok, that would be good... | 14:59 |
| dtantsur | dunno yet, need to dive in it (if someone else can do, I'd be very thankful too) | 14:59 |
| mjturek | I'm taking a look around but don't know too much about sphinx | 15:01 |
| *** baoli has joined #openstack-ironic | 15:02 | |
| *** priteau has quit IRC | 15:02 | |
| *** ricardoas is now known as ricardoas-lunch | 15:02 | |
| rloo | mjturek, dtantsur: i added some comments to the bug https://bugs.launchpad.net/ironic/+bug/1694724 | 15:03 |
| openstack | Launchpad bug 1694724 in Ironic "Warning for ironic.drivers.modules.deploy_utils" [Critical,New] | 15:03 |
| mjturek | thx rloo | 15:03 |
| rloo | mjturek, dtantsur: am wondering if doug hellman (doesn't seem to be online now) can shed some light on it | 15:03 |
| rloo | thx mjturek. it really looks like a bug in sphinx; that code is quite innocent... :) | 15:04 |
| *** hoonetorg has quit IRC | 15:04 | |
| mjturek | rloo: so yesterday the docs job seemed to be using sphinx 1.5.6 | 15:04 |
| openstackgerrit | Merged openstack/ironic-inspector master: Completely remove support for setting IPMI credentials https://review.openstack.org/466234 | 15:04 |
| mjturek | so maybe you're right about blocking 1.6.2 | 15:04 |
| rloo | mjturek: yeah, which is why i think it has to do with 1.6.2 (or 1.6.1?) | 15:05 |
| mjturek | rloo: failure is happening with 1.6.2 | 15:05 |
| mjturek | wondering if any other projects are hitting it | 15:05 |
| *** shuyingya has joined #openstack-ironic | 15:05 | |
| rloo | mjturek: well, we blocked 1.6.1 so dunno if it would have failed there too | 15:05 |
| mjturek | right right | 15:05 |
| rloo | mjturek: fwiw, i also tried to suppress all known sphinx warnings, and that warning was still emitted. | 15:06 |
| rloo | mjturek: guess i can add that to the bug as well. | 15:06 |
| mjturek | oh my | 15:06 |
| mjturek | lol | 15:06 |
| openstackgerrit | Merged openstack/bifrost stable/ocata: Use WORKSPACE for requirements https://review.openstack.org/468043 | 15:08 |
| dtantsur | rloo: that's not a sphinx warning, this is a warning our code gives when imported. maybe we can remove it, if enough time has passed | 15:10 |
| rloo | dtantsur: no, it is a sphinx warning. sphinx is warning about that warning string :-( | 15:10 |
| *** mjturek has quit IRC | 15:11 | |
| rloo | dtantsur: and we have it configured so sphinx warnings are treated as errors, hence job failure | 15:11 |
| dtantsur | yeah, which is not a bad thing to do actually.. we just have to make it ignore our warnings somehow | 15:11 |
| rloo | dtantsur: well, this particular warning doesn't make any sense. i think it is a bug in sphinx. | 15:11 |
| dtantsur | right, I agree | 15:12 |
| rloo | dtantsur: i already tried suppressing all known sphinx warnings and sphinx still emits a warning for that :-( | 15:12 |
| dtantsur | Oo | 15:12 |
| rloo | dtantsur: so we can wait/ask others, eg doug, if we should block eg 1.6.2, or we can unblock now by turning off the sphinx warnings as errors flag short term. i guess we aren't in a hurry so maybe wait? | 15:13 |
| dtantsur | rloo: I think talking to dhellmann is not an unreasonable thing to do | 15:13 |
| *** mat128 has quit IRC | 15:14 | |
| *** aparnav has quit IRC | 15:14 | |
| *** aparnav has joined #openstack-ironic | 15:16 | |
| *** ptx0_ is now known as ptx0 | 15:16 | |
| *** hoonetorg has joined #openstack-ironic | 15:17 | |
| *** mjturek has joined #openstack-ironic | 15:17 | |
| mjturek | sorry laptop died | 15:17 |
| rloo | dtantsur: ++ | 15:17 |
| rloo | mjturek: i think it might be best to talk to dhellman about it, not sure it is worth spending your time on it too. unless you want to :-) | 15:18 |
| mjturek | thanks rloo if I notice anything else I'll let you know :) | 15:18 |
| rloo | mjturek: thx (or add to the bug) | 15:18 |
| mjturek | will do | 15:19 |
| rloo | dhellmann: when you have a minute, sphinx is emitting a warning/error for ironic docs job: https://bugs.launchpad.net/ironic/+bug/1694724 | 15:25 |
| openstack | Launchpad bug 1694724 in Ironic "Warning for ironic.drivers.modules.deploy_utils" [Critical,New] | 15:25 |
| rloo | oops, wrong window | 15:25 |
| *** krtaylor has joined #openstack-ironic | 15:30 | |
| *** aarefiev is now known as aarefiev_afk | 15:34 | |
| *** fxpester has quit IRC | 15:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:54 | |
| TheJulia | o/ | 15:56 |
| openstackgerrit | Mark Goddard proposed openstack/ironic master: Add physical network to port data model https://review.openstack.org/461301 | 15:56 |
| *** ryanender has joined #openstack-ironic | 15:58 | |
| fellypefca | TheJulia Hi! :) Did you have some time to look at the onetime boot patch? | 15:58 |
| *** baoli has quit IRC | 15:59 | |
| TheJulia | fellypefca: I've honestly not had time. I just landed in another city this morning :) | 15:59 |
| TheJulia | If you have the link handy, I can look after my meeting in a minute | 15:59 |
| *** Nisha_ has joined #openstack-ironic | 15:59 | |
| *** Nisha_Agarwal has quit IRC | 16:01 | |
| *** baoli has joined #openstack-ironic | 16:01 | |
| *** amarao has quit IRC | 16:02 | |
| fellypefca | TheJulia Sure! https://review.openstack.org/#/c/436469/ :) | 16:02 |
| patchbot | patch 436469 - ironic - Adds onetime boot in oneview drivers | 16:02 |
| *** kromanenko has quit IRC | 16:02 | |
| rloo | dtantsur: dhellman suggests: my recommendation is to stop having ironic log things when modules are imported (that's not safe anyway) and do it at runtime somewhere in the code path where the option in question is being used | 16:03 |
| rloo | dtantsur: that warning was something you added: https://github.com/openstack/ironic/commit/93f947c852409af2c56a499428b09ff69ab345a1#diff-baffaa4a176617edd66ee5512da7b6d9 | 16:04 |
| *** baoli has quit IRC | 16:04 | |
| dtantsur | I'm to blame \o/ | 16:05 |
| dtantsur | :D | 16:05 |
| rloo | dtantsur: I'm not sure we can actually remove/delete the warning yet? | 16:05 |
| rloo | dtantsur: no, sphinx is to blame :-) | 16:05 |
| lucasagomes | this thing about treating warning as an error is kinda annoying as well :-( | 16:06 |
| *** milan has quit IRC | 16:06 | |
| dtantsur | rloo: it was introduced in Sep 2016, we can totally just remove it and flip the option default | 16:06 |
| rloo | lucasagomes: ++ | 16:06 |
| dtantsur | well, if we have local boot working in the CI, which I'm not sure :) | 16:06 |
| dtantsur | lucasagomes: +++ | 16:06 |
| lucasagomes | the networking-ovn gate is broken due something similar | 16:06 |
| lucasagomes | dtantsur, rloo btw you can disable that in tox, no ? | 16:06 |
| rloo | lucasagomes: disable the sphinx warning? yes, if we turn off sphinx's 'treat warnings as errors' | 16:07 |
| lucasagomes | dtantsur, rloo https://review.openstack.org/#/c/455182/ | 16:07 |
| *** rcernin has quit IRC | 16:07 | |
| patchbot | patch 455182 - networking-ovn - Docs: Do not turn warnings into errors (MERGED) | 16:07 |
| lucasagomes | rloo, yeah | 16:07 |
| rloo | lucasagomes: yes, that was one of the options i mentioned. | 16:07 |
| rloo | lucasagomes: i would be fine with that as short term, but we should probably fix. our. code. | 16:07 |
| dtantsur | the problem is that sphinx treats many errors as warnings otherwise | 16:07 |
| lucasagomes | rloo, indeed | 16:07 |
| dtantsur | e.g. you can totally screw up syntax, and the CI will pass | 16:07 |
| lucasagomes | dtantsur, sigh | 16:07 |
| *** tesseract has quit IRC | 16:08 | |
| rloo | dtantsur: yeah. so you want me to fix it properly for now? i can look into it. i mean, emit the warning from a method :-) | 16:08 |
| dtantsur | rloo: if you don't mind. I'm afraid we still cannot switch to local boon in the CI (right lucasagomes?) | 16:08 |
| lucasagomes | dtantsur, yeah you can not with cirros | 16:09 |
| lucasagomes | we were discussing in the PTG to have our own images | 16:09 |
| lucasagomes | but someone needs to work on that | 16:09 |
| lucasagomes | I honestly thing that would be a good thing cause, I don't foresee cirros including grub2 in it | 16:10 |
| lucasagomes | it would increase the size of the image substantially | 16:10 |
| *** trown is now known as trown|lunch | 16:10 | |
| *** baoli has joined #openstack-ironic | 16:13 | |
| dtantsur | I wonder if we need a job with some other image (fedora/ubuntu/whatever) | 16:14 |
| TheJulia | aiui, it can't be ubuntu | 16:18 |
| lucasagomes | TheJulia, ++ yeah the IP police of ubuntu makes it odd to distribute a customized image | 16:18 |
| lucasagomes | I think either debian or fedora would be ideal | 16:19 |
| lucasagomes | as long as they have a cloud image available it should be good | 16:19 |
| TheJulia | This circles back to the whole discussion of getting the dib element in tree | 16:20 |
| * TheJulia wonders if the duplicate element processing bug/issue ever got fixed | 16:20 | |
| *** aparnav has quit IRC | 16:20 | |
| *** aparnav has joined #openstack-ironic | 16:20 | |
| *** shuyingya has quit IRC | 16:21 | |
| lucasagomes | yeah, that's a very strange case they have to _never_ delete it from their tree | 16:22 |
| lucasagomes | perhaps we should duplicate it (change the name) and add a deprecation method in the DIB tree | 16:22 |
| lucasagomes | s/method/message | 16:22 |
| TheJulia | That is a valid option for velocity | 16:23 |
| lucasagomes | yeah and we could call it "ironic-python-agent" which would be a more correct name than the "ironic-agent" one in DIB (-: | 16:24 |
| TheJulia | I don't remember the argument against doing that at this point | 16:24 |
| TheJulia | +1 | 16:24 |
| lucasagomes | anyhoo, calling it a day here | 16:26 |
| lucasagomes | have a great evening all! | 16:26 |
| *** lucasagomes is now known as lucas-afk | 16:26 | |
| sambetts | dtantsur proposed that solution on the ML, and we also planned to put it all in a new ipa builder repo | 16:26 |
| lucas-afk | sambetts, ++ yeah, that would be great to have | 16:26 |
| *** baoli has quit IRC | 16:27 | |
| lucas-afk | and a post job to create the image and save it at tarball.openstack.org | 16:27 |
| sambetts | yup | 16:27 |
| *** chas has joined #openstack-ironic | 16:27 | |
| sambetts | I think the plan was to move all the IPA builders coreos/tinyipa etc into that repo | 16:27 |
| *** baoli has joined #openstack-ironic | 16:27 | |
| sambetts | because they aren't tied to IPA versions | 16:28 |
| *** Nisha_brb has joined #openstack-ironic | 16:28 | |
| lucas-afk | ++ | 16:28 |
| lucas-afk | yeah I remember that from the PTG as well | 16:28 |
| * lucas-afk will be afk now | 16:28 | |
| *** rbudden_ has joined #openstack-ironic | 16:29 | |
| *** Nisha_ has quit IRC | 16:30 | |
| *** chas has quit IRC | 16:31 | |
| *** rbudden has quit IRC | 16:31 | |
| *** rbudden_ is now known as rbudden | 16:31 | |
| * TheJulia does not feel that great about disjointing build scripts from the actual code since they may need to track together as things change... *shrugs* | 16:33 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Call deploy warning methods from code path https://review.openstack.org/469560 | 16:33 |
| sambetts | TheJulia: we end up backporting 90% of the changes to the build scripts though because the problems affect all branchs | 16:34 |
| TheJulia | sambetts: guess that makes sense as long as we don't end up doing stable versions on that script repo, otherwise we are in the same exact boat | 16:35 |
| sambetts | yeah -1000000 to stable branchs on that repo | 16:35 |
| TheJulia | OR let them be cut, and a giant warning sign hung there saying "not supported" | 16:35 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/469565 | 16:36 |
| * TheJulia goes and checks in to her hotel room | 16:38 | |
| * sambetts -> home | 16:42 | |
| sambetts | night all | 16:42 |
| *** sambetts is now known as sambetts|afk | 16:42 | |
| *** sayantan_ has joined #openstack-ironic | 16:45 | |
| *** rcernin has joined #openstack-ironic | 16:46 | |
| *** jkilpatr has quit IRC | 16:47 | |
| *** pcaruana has quit IRC | 16:50 | |
| ricardoas-lunch | dtantsur: got a sec? now that we adjusted the version of oneviewclient for stable/newton, we can discuss what can be done with agent_pxe_oneview :) | 16:51 |
| dtantsur | ricardoas-lunch: sure, but I'm about to jump on a long meeting.. maybe tomorrow? or feel free to drop me an email | 16:52 |
| ricardoas-lunch | dtantsur: oops... no problem, i´ll raise some options with the team so we can talk tomorrow :) | 16:53 |
| *** ricardoas-lunch is now known as ricardoas | 16:53 | |
| fellypefca | dtantsur btw, besides this discussion about the agent, tomorrow you could take a look at the answers that ricardoas gave about your questions in the ontime boot patch? :D | 16:56 |
| dtantsur | I hope so | 16:57 |
| *** mgoddard has quit IRC | 16:57 | |
| fellypefca | dtantsur ty :) | 16:57 |
| *** racedo has quit IRC | 16:59 | |
| *** e0ne has quit IRC | 17:00 | |
| *** ralonsoh has quit IRC | 17:01 | |
| *** jkilpatr has joined #openstack-ironic | 17:04 | |
| *** baoli has quit IRC | 17:07 | |
| *** aparnav has quit IRC | 17:08 | |
| *** baoli has joined #openstack-ironic | 17:08 | |
| *** ChubYann has joined #openstack-ironic | 17:11 | |
| *** baoli has quit IRC | 17:13 | |
| *** baoli has joined #openstack-ironic | 17:13 | |
| *** fragatin_ has joined #openstack-ironic | 17:15 | |
| *** fragatina has quit IRC | 17:19 | |
| *** Serlex has quit IRC | 17:19 | |
| *** mat128 has joined #openstack-ironic | 17:22 | |
| *** karthiks has quit IRC | 17:24 | |
| *** trown|lunch is now known as trown | 17:25 | |
| *** chas has joined #openstack-ironic | 17:28 | |
| *** racedo has joined #openstack-ironic | 17:42 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 17:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 17:43 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 17:43 |
| *** jaosorior is now known as jaosorior_away | 17:44 | |
| *** athomas has quit IRC | 17:44 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-ui master: Add support for editing node's resource_class https://review.openstack.org/463852 | 17:44 |
| *** chas has quit IRC | 17:48 | |
| *** chas has joined #openstack-ironic | 17:49 | |
| *** Nisha_brb has quit IRC | 17:50 | |
| *** chas has quit IRC | 17:53 | |
| *** fragatin_ has quit IRC | 17:57 | |
| *** fragatina has joined #openstack-ironic | 17:58 | |
| *** chlong has quit IRC | 18:05 | |
| *** fellypefca has quit IRC | 18:13 | |
| *** cdearborn_ has joined #openstack-ironic | 18:15 | |
| *** chlong has joined #openstack-ironic | 18:22 | |
| *** chas has joined #openstack-ironic | 18:29 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Do not fail sphinx-build on warnings https://review.openstack.org/469615 | 18:29 |
| pas-ha | this should unblock the docs job and the gate as while | 18:30 |
| pas-ha | whole | 18:30 |
| pas-ha | ironic cores, please check ^^^ as we have our gate wedged due to docs job and new sphinx | 18:31 |
| *** cdearborn has quit IRC | 18:32 | |
| rloo | pas-ha: oh, i already have a fix to it | 18:33 |
| pas-ha | rloo: will galdly abandon mine | 18:33 |
| rloo | pas-ha: https://bugs.launchpad.net/ironic/+bug/1694724 | 18:33 |
| openstack | Launchpad bug 1694724 in Ironic "Sphinx Warning for ironic.drivers.modules.deploy_utils" [Critical,In progress] - Assigned to Ruby Loo (rloo) | 18:33 |
| rloo | pas-ha: sorry about miscommunication | 18:34 |
| *** mgoddard has joined #openstack-ironic | 18:34 | |
| rloo | pas-ha: that was one of the options, to turn off that warningas error. anyway, see comments in bug | 18:35 |
| pas-ha | yep, that would be a proper kind of fix, although I wondered if we could drop those warnings and just change the default already :) | 18:36 |
| rloo | pas-ha: i discussed earlier today with dtantsur and we decided we wanted to know about those warnings, ie, cause gate failure | 18:37 |
| pas-ha | I also do not like all these globals, so how about we make those checks on conductor start? there are already checks for other deprecated (mis)configuratuons | 18:37 |
| pas-ha | yep, agree | 18:37 |
| rloo | pas-ha: oh, are we doing that in conductor? i can't remember our code. | 18:37 |
| rloo | pas-ha: it was my quick/dirty way to fix. we have to remove those warnings at some point... | 18:38 |
| *** derekh has quit IRC | 18:39 | |
| pas-ha | since they 'default_boot_option' warning is here for 8months already, why not change the default now and stop emitting it? :) | 18:39 |
| rloo | pas-ha: he. you should read irc, sec, let me find the actual 'time'... | 18:39 |
| * dtantsur off for the day, c u | 18:40 | |
| *** dtantsur is now known as dtantsur|afk | 18:40 | |
| rloo | pas-ha: it starts around here, but http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-05-31.log.html#t2017-05-31T13:34:30 | 18:40 |
| rloo | pas-ha: the meat is around here: http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2017-05-31.log.html#t2017-05-31T16:04:04 | 18:41 |
| rloo | pas-ha: i'll add comments ^^ to that bug | 18:42 |
| pas-ha | we can totally change the default in code, but set up 'netboot' explicitly in devstack plugin (and I think we already are doing that).. I'll propose an alternative patch | 18:43 |
| *** krtaylor has quit IRC | 18:44 | |
| *** racedo has quit IRC | 18:44 | |
| rloo | pas-ha: ok, although i'd be happy with my dumb patch going in just to unblock the gate. not sure if yours will cause more discussion or not... | 18:46 |
| rloo | pas-ha: not that it is urgent to unblock, but i am worried about folks 'recheck'in... | 18:47 |
| rloo | pas-ha: i messed up on that patch i did anyway, forgot to run tox on it, etc. should i bother fixing or abandon for your new patch? | 18:55 |
| pas-ha | fixing it right now | 18:59 |
| rloo | pas-ha: thx! | 19:00 |
| *** adisky__ has quit IRC | 19:01 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Flip default_boot_option to local by default https://review.openstack.org/469624 | 19:04 |
| *** rloo has quit IRC | 19:04 | |
| *** rloo has joined #openstack-ironic | 19:04 | |
| *** chas has quit IRC | 19:04 | |
| pas-ha | rloo: ^ let's see how it works. unit tests seem to pass, and we set the 'netboot' in devstack plugin explicitly, so everything should be good now | 19:04 |
| pas-ha | not sure if we need a release note when we remove deprecated stuff though | 19:05 |
| *** chas has joined #openstack-ironic | 19:05 | |
| rloo | pas-ha: ok... | 19:05 |
| rloo | pas-ha: hmm, i think we should put something in the upgrades section? | 19:05 |
| pas-ha | feel free to amend the patch (after it passes CI :) ) as I should be going home now.. | 19:07 |
| * pas-ha heads off, g'night all :) | 19:07 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Remove legacy auth loading https://review.openstack.org/469626 | 19:08 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Improve endpoint_type handling https://review.openstack.org/469627 | 19:08 |
| rloo | pas-ha: i thought lucas-afk said that we couldn't switch to local boot in the CI though? | 19:08 |
| rloo | pas-ha: good night, we can discuss tomorrow | 19:08 |
| *** chas has quit IRC | 19:09 | |
| *** dyasny has quit IRC | 19:12 | |
| *** crushil has quit IRC | 19:14 | |
| *** sayantan_ has quit IRC | 19:17 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 19:17 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 19:17 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 19:17 |
| *** sayantan_ has joined #openstack-ironic | 19:17 | |
| *** crushil has joined #openstack-ironic | 19:17 | |
| *** sayantan_ has quit IRC | 19:21 | |
| *** jaganathan has quit IRC | 19:21 | |
| *** sayantan_ has joined #openstack-ironic | 19:24 | |
| *** sayantan_ has quit IRC | 19:27 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver https://review.openstack.org/466448 | 19:27 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack https://review.openstack.org/468712 | 19:27 |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic stable/newton: Remove agent vendor passthru from OneView drivers https://review.openstack.org/469629 | 19:28 |
| *** dyasny has joined #openstack-ironic | 19:28 | |
| *** sayantan_ has joined #openstack-ironic | 19:30 | |
| *** sayantan_ has quit IRC | 19:33 | |
| *** sayantan_ has joined #openstack-ironic | 19:36 | |
| *** krtaylor has joined #openstack-ironic | 19:37 | |
| *** jkilpatr has quit IRC | 19:41 | |
| *** sayantan_ has quit IRC | 19:41 | |
| *** sayantan_ has joined #openstack-ironic | 19:43 | |
| *** dyasny has quit IRC | 19:44 | |
| *** pcaruana has joined #openstack-ironic | 19:45 | |
| *** sayantan_ has quit IRC | 19:45 | |
| *** dyasny has joined #openstack-ironic | 19:47 | |
| *** dsneddon has quit IRC | 19:48 | |
| *** sayantan_ has joined #openstack-ironic | 19:49 | |
| *** mgoddard has quit IRC | 19:51 | |
| *** mtanino__ has quit IRC | 19:52 | |
| *** sayantan_ has quit IRC | 19:52 | |
| *** jkilpatr has joined #openstack-ironic | 19:54 | |
| *** sayantan_ has joined #openstack-ironic | 19:55 | |
| *** sayantan_ has quit IRC | 19:59 | |
| *** sayantan_ has joined #openstack-ironic | 20:02 | |
| *** pcaruana has quit IRC | 20:02 | |
| *** crushil has quit IRC | 20:02 | |
| *** sayantan_ has quit IRC | 20:02 | |
| *** sayantan_ has joined #openstack-ironic | 20:02 | |
| *** sayantan_ has quit IRC | 20:04 | |
| *** sayantan_ has joined #openstack-ironic | 20:05 | |
| *** e0ne has joined #openstack-ironic | 20:09 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 20:11 |
| *** dprince has quit IRC | 20:13 | |
| *** crushil has joined #openstack-ironic | 20:17 | |
| *** crushil has quit IRC | 20:18 | |
| *** crushil has joined #openstack-ironic | 20:19 | |
| *** mat128 has quit IRC | 20:20 | |
| *** rcernin has quit IRC | 20:21 | |
| *** rcernin has joined #openstack-ironic | 20:22 | |
| *** Goneri has quit IRC | 20:23 | |
| *** crushil has quit IRC | 20:23 | |
| *** crushil has joined #openstack-ironic | 20:24 | |
| *** crushil has quit IRC | 20:26 | |
| *** crushil has joined #openstack-ironic | 20:27 | |
| *** crushil has quit IRC | 20:32 | |
| *** crushil has joined #openstack-ironic | 20:33 | |
| *** rcernin has quit IRC | 20:42 | |
| *** chlong has quit IRC | 20:53 | |
| *** chlong has joined #openstack-ironic | 21:05 | |
| *** e0ne has quit IRC | 21:07 | |
| *** pcaruana has joined #openstack-ironic | 21:12 | |
| *** penick has joined #openstack-ironic | 21:17 | |
| *** harlowja has quit IRC | 21:21 | |
| *** pcaruana has quit IRC | 21:29 | |
| *** sayantan_ has quit IRC | 21:31 | |
| *** ryanender has quit IRC | 21:34 | |
| *** bfournie has quit IRC | 21:39 | |
| *** crushil has quit IRC | 21:39 | |
| *** chas has joined #openstack-ironic | 21:41 | |
| *** chas has quit IRC | 21:46 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 21:46 |
| *** dsneddon_ has joined #openstack-ironic | 21:49 | |
| *** rama_y has quit IRC | 21:53 | |
| *** Guest62718 has quit IRC | 21:53 | |
| *** shuyingya has joined #openstack-ironic | 22:02 | |
| *** harlowja has joined #openstack-ironic | 22:06 | |
| *** shuyingya has quit IRC | 22:07 | |
| *** bfournie has joined #openstack-ironic | 22:14 | |
| *** bfournie has quit IRC | 22:15 | |
| *** bfournie has joined #openstack-ironic | 22:16 | |
| *** cdearborn_ has quit IRC | 22:19 | |
| *** penick has quit IRC | 22:19 | |
| *** aNuposic is now known as anupn | 22:22 | |
| *** jkilpatr has quit IRC | 22:23 | |
| anupn | Hi TheJulia, ppiela: how do you think of adding a new tab for driver interfaces while enrolling node in Ironic UI? | 22:23 |
| anupn | because right now if interfaces fields are added in driver details form then it will be a long form | 22:23 |
| ppiela | anupn: I think a separate tab triggered by the choice of driver makes sense | 22:25 |
| anupn | ppiela: cool | 22:27 |
| ppiela | anupn: I would develop a minimal prototype and do a demo to get feedback | 22:28 |
| anupn | ppiela: yup i can submit a patch with a separate tab for interfaces, and then i can get reviews on it | 22:29 |
| anupn | do you thought of any other way for demo? | 22:29 |
| ppiela | anupn: In the past we have done Google hangout meetings to review early design ideas | 22:30 |
| anupn | ppiela: Ah that sounds fantastic, i can develop locally and then can have a google hangout with you guys | 22:31 |
| anupn | and get feedback that time | 22:31 |
| ppiela | anupn: yes | 22:32 |
| anupn | do we have scheduled google hangouts weekly/bi-weekly? | 22:32 |
| anupn | or is as per the request | 22:32 |
| ppiela | anupn: We dont, it is pretty much by request. We can discuss at our next weekly meeting. | 22:33 |
| anupn | ppiela: cool | 22:33 |
| *** baoli has quit IRC | 22:34 | |
| *** baoli has joined #openstack-ironic | 22:35 | |
| *** jkilpatr has joined #openstack-ironic | 22:38 | |
| *** baoli has quit IRC | 22:39 | |
| *** krtaylor has quit IRC | 22:46 | |
| *** Sukhdev has joined #openstack-ironic | 22:51 | |
| *** ekoorjr has joined #openstack-ironic | 22:59 | |
| *** chlong has quit IRC | 23:00 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 23:17 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 23:18 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Unit test for checking cross-version migrations compatibility https://review.openstack.org/419993 | 23:20 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrade procedure documentation https://review.openstack.org/416565 | 23:21 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Rolling upgrades related dev documentation https://review.openstack.org/419439 | 23:21 |
| *** baoli has joined #openstack-ironic | 23:32 | |
| *** baoli_ has joined #openstack-ironic | 23:34 | |
| *** baoli has quit IRC | 23:37 | |
| *** chas has joined #openstack-ironic | 23:40 | |
| *** chas has quit IRC | 23:44 | |
| *** ricardoas is now known as ricardoas-afk | 23:53 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!