| *** g3ek has joined #openstack-ironic | 00:00 | |
| *** gcb has quit IRC | 00:03 | |
| *** Administrator_ has quit IRC | 00:04 | |
| *** Administrator_ has joined #openstack-ironic | 00:05 | |
| *** zackf has quit IRC | 00:13 | |
| *** crushil has quit IRC | 00:17 | |
| *** sayantan_ has joined #openstack-ironic | 00:18 | |
| *** amotoki has joined #openstack-ironic | 00:20 | |
| *** mtanino_ has joined #openstack-ironic | 00:23 | |
| *** deadnull has joined #openstack-ironic | 00:24 | |
| *** sasha2 has quit IRC | 00:24 | |
| *** mtanino has quit IRC | 00:25 | |
| *** bfournie has joined #openstack-ironic | 00:36 | |
| *** phuongnh has joined #openstack-ironic | 00:42 | |
| *** xavierr has quit IRC | 00:42 | |
| *** deadnull has quit IRC | 00:42 | |
| *** xavierr has joined #openstack-ironic | 00:44 | |
| *** crushil has joined #openstack-ironic | 00:47 | |
| *** mat128 has quit IRC | 00:47 | |
| *** baoli has joined #openstack-ironic | 00:48 | |
| *** baoli has quit IRC | 00:49 | |
| *** MarginHu has joined #openstack-ironic | 00:49 | |
| MarginHu | Good morning , Ironic | 00:50 |
|---|---|---|
| *** baoli has joined #openstack-ironic | 00:50 | |
| *** rloo has quit IRC | 00:53 | |
| *** zhurong has joined #openstack-ironic | 00:55 | |
| *** mat128 has joined #openstack-ironic | 00:56 | |
| *** xavierr has quit IRC | 00:56 | |
| *** xavierr has joined #openstack-ironic | 00:59 | |
| *** xavierr has quit IRC | 01:01 | |
| *** xavierr has joined #openstack-ironic | 01:03 | |
| *** SerenaFeng has joined #openstack-ironic | 01:03 | |
| *** xavierr has quit IRC | 01:09 | |
| *** xavierr has joined #openstack-ironic | 01:10 | |
| *** xavierr has quit IRC | 01:13 | |
| *** gcb has joined #openstack-ironic | 01:16 | |
| *** xavierr has joined #openstack-ironic | 01:18 | |
| *** xavierr has quit IRC | 01:21 | |
| *** xavierr has joined #openstack-ironic | 01:22 | |
| *** MarginHu has quit IRC | 01:23 | |
| *** Wei_ has joined #openstack-ironic | 01:25 | |
| *** xavierr has quit IRC | 01:25 | |
| *** xavierr has joined #openstack-ironic | 01:27 | |
| *** Sukhdev has quit IRC | 01:33 | |
| TheJulia | jroll: at least it doesn't take _that_ long to sift through the features once you have a list of criteria | 01:42 |
| *** jcoufal has joined #openstack-ironic | 01:47 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: WIP: Initial neutron task changes https://review.openstack.org/452515 | 01:47 |
| *** [1]cdearborn has quit IRC | 02:03 | |
| *** yonglihe has joined #openstack-ironic | 02:03 | |
| Wei_ | Hi all, when I create a server, an error appears in conductor log, but the hypervisor host is up and enabled | 02:08 |
| Wei_ | below is conductor log | 02:08 |
| Wei_ | http://paste.openstack.org/show/606094/ | 02:08 |
| Wei_ | it seems there is no valid host to reside | 02:10 |
| *** mtanino_ has quit IRC | 02:11 | |
| *** tuanluong has joined #openstack-ironic | 02:15 | |
| TheJulia | Wei_: Two likely causes, the scheduling failed due to $some unknown reason.... or there are not enough nodes in ironic in available state that match the user criteria | 02:28 |
| Wei_ | you mean the flavor is not suitable? | 02:30 |
| *** jcoufal_ has joined #openstack-ironic | 02:31 | |
| *** jcoufal_ has quit IRC | 02:32 | |
| *** jcoufal has quit IRC | 02:34 | |
| *** fragatina has joined #openstack-ironic | 02:34 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost master: WIP: Experimental Neutron Installation https://review.openstack.org/452514 | 02:34 |
| *** fragatin_ has quit IRC | 02:37 | |
| TheJulia | Wei_: flavor doesn't match the properties of the baremetal node | 02:37 |
| TheJulia | or there are no baremetal nodes to match the flavor | 02:37 |
| *** fragatina has quit IRC | 02:38 | |
| *** fragatina has joined #openstack-ironic | 02:41 | |
| *** fragatina has quit IRC | 02:42 | |
| Wei_ | TheJulia: the flavor must be the same as the properties of the baremetal node? | 02:47 |
| Wei_ | flavor may be lower than properties? | 02:49 |
| *** spartacloud has joined #openstack-ironic | 03:04 | |
| *** MarginHu has joined #openstack-ironic | 03:19 | |
| *** rbudden has quit IRC | 03:20 | |
| *** amotoki has quit IRC | 03:26 | |
| *** crushil has quit IRC | 03:32 | |
| TheJulia | Wei_: As I understand it, it depends on scheduler configuration. | 03:32 |
| TheJulia | Wei_: the examples that I can think of are exact match based filters. | 03:34 |
| *** Nisha_Agarwal has joined #openstack-ironic | 03:34 | |
| *** gkadam has joined #openstack-ironic | 03:39 | |
| *** amotoki has joined #openstack-ironic | 03:40 | |
| *** gkadam is now known as gkadam-afk | 03:44 | |
| *** MarginHu has quit IRC | 03:46 | |
| *** vikrant has joined #openstack-ironic | 03:46 | |
| *** MarginHu has joined #openstack-ironic | 03:47 | |
| *** chlong has quit IRC | 03:47 | |
| *** aparnav has joined #openstack-ironic | 03:49 | |
| *** amotoki has quit IRC | 03:50 | |
| *** amotoki has joined #openstack-ironic | 03:58 | |
| Wei_ | TheJulia: I will check sheduler configuration after lunch and many thanks | 04:08 |
| *** fragatina has joined #openstack-ironic | 04:09 | |
| *** gkadam-afk is now known as gkadam | 04:11 | |
| *** fragatina has quit IRC | 04:13 | |
| *** amotoki has quit IRC | 04:13 | |
| *** mat128 has quit IRC | 04:14 | |
| *** Sukhdev has joined #openstack-ironic | 04:14 | |
| *** baoli has quit IRC | 04:14 | |
| *** Nisha_Agarwal has quit IRC | 04:17 | |
| *** amotoki has joined #openstack-ironic | 04:29 | |
| *** fragatina has joined #openstack-ironic | 04:37 | |
| *** fragatina has quit IRC | 04:37 | |
| *** fragatina has joined #openstack-ironic | 04:38 | |
| *** mjura has joined #openstack-ironic | 04:46 | |
| *** yongjiexu has joined #openstack-ironic | 04:57 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 05:04 |
| *** jaganathan has joined #openstack-ironic | 05:11 | |
| *** Sukhdev has quit IRC | 05:12 | |
| *** rcernin has joined #openstack-ironic | 05:15 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 05:18 | |
| *** udesale has joined #openstack-ironic | 05:20 | |
| *** tuanluong has quit IRC | 05:33 | |
| *** dsneddon has quit IRC | 05:35 | |
| *** ChubYann has quit IRC | 05:38 | |
| *** adisky_ has joined #openstack-ironic | 05:44 | |
| *** fragatina has quit IRC | 05:44 | |
| *** jaosorior_away is now known as jaosorior | 05:55 | |
| *** tuanluong has joined #openstack-ironic | 05:58 | |
| *** yongjiexu has quit IRC | 06:21 | |
| *** Serlex has joined #openstack-ironic | 06:22 | |
| pas-ha | morning ironic :) | 06:22 |
| *** SerenaFeng has quit IRC | 06:26 | |
| *** SerenaFeng has joined #openstack-ironic | 06:26 | |
| *** amarao has joined #openstack-ironic | 06:27 | |
| *** SerenaFeng has quit IRC | 06:30 | |
| *** jtomasek_ has quit IRC | 06:31 | |
| *** yongjiexu has joined #openstack-ironic | 06:34 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add comments re RPC versions being in sync https://review.openstack.org/455536 | 06:34 |
| *** SerenaFeng has joined #openstack-ironic | 06:37 | |
| *** sdake has quit IRC | 06:46 | |
| *** tesseract has joined #openstack-ironic | 06:47 | |
| *** mgagne has quit IRC | 06:52 | |
| *** Serlex has quit IRC | 07:01 | |
| *** pcaruana has joined #openstack-ironic | 07:06 | |
| *** mgoddard has joined #openstack-ironic | 07:33 | |
| MarginHu | hi, <pas-ha> | 07:34 |
| MarginHu | Hi, pas-ha, I met a issue https://bugs.launchpad.net/ironic/+bug/1681665 | 07:35 |
| openstack | Launchpad bug 1681665 in Ironic "How to write a file for ironic inspector and deploy when pxe with iPXE instead of TFTP ? " [Undecided,New] | 07:35 |
| MarginHu | can you help me ? | 07:35 |
| pas-ha | well, the different content in those files is achievable as it is actually a Jinja2 template being rendered with some options by ironic-conductor. What might have different is how you start introspection - by comanding ironic-inspector or by commanding ironic | 07:41 |
| pas-ha | aarefiev_afk: ^ | 07:41 |
| *** aarefiev_afk is now known as aarefiev | 07:41 | |
| aarefiev | morning all! | 07:41 |
| Nisha_Agarwal | morning all | 07:41 |
| *** mgoddard has quit IRC | 07:42 | |
| yongjiexu | evening in china(-- _--) | 07:46 |
| *** alexpilotti has quit IRC | 07:47 | |
| MarginHu | pas-ha, one is for introspection, "ironic node-set-provision-state NODE inspect", another is for deploying a baremetal node with "nova boot " | 07:48 |
| aarefiev | MarginHu: there are 2 diff ipxe configs for ironic and inspector | 07:48 |
| MarginHu | yes, aarefiev> | 07:48 |
| MarginHu | I didn't find a solution in Ironic and Ironic-inspector Document. | 07:49 |
| aarefiev | MarginHu: what is the question then | 07:49 |
| MarginHu | I only can designate one file in dnsmasq.conf , but it needs different content in two scenes. | 07:50 |
| aarefiev | MarginHu: in devstack inspector generates ipxe configs on install stage https://github.com/openstack/ironic-inspector/blob/master/devstack/plugin.sh#L124 | 07:51 |
| aarefiev | MarginHu: aha, actually, there is 2 dnsmasq, ironic uses neutron dhcp which is uses dnsmasq and inspector has separate dnsmasq instance for introspection and discovery | 07:52 |
| MarginHu | 2 dnsmasq ? | 07:53 |
| *** ianychoi has quit IRC | 07:54 | |
| *** derekh has joined #openstack-ironic | 07:54 | |
| MarginHu | neutron dhcp is for deploying , It seems that you're right. | 07:54 |
| aarefiev | MarginHu: you are pointing in bug on inspector dnsmasq config file, in ironic there is template for it, ironic call neutron to configure dhcp | 07:54 |
| *** ianychoi has joined #openstack-ironic | 07:54 | |
| *** Serlex has joined #openstack-ironic | 07:55 | |
| aarefiev | MarginHu: there is config option for ironic https://github.com/openstack/ironic/blob/master/ironic/conf/pxe.py#L93 | 07:56 |
| aarefiev | on ipxe script | 07:57 |
| aarefiev | for inspector you could modify ipxe script on http server your dnsmasq config pointing on | 07:57 |
| aarefiev | MarginHu: in devstack there are in /opt/stack/data/ironic/httpboot/ by default | 07:59 |
| MarginHu | I see, but how to config the file in neutron dhcp agent ? | 07:59 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| aarefiev | MarginHu: ironic does it for you, see config option, there is default script, you could point on your own | 08:01 |
| aarefiev | MarginHu: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/boot.ipxe | 08:02 |
| MarginHu | ipxe_boot_script = $pybasedir/drivers/modules/boot.ipxe | 08:03 |
| aarefiev | yep | 08:03 |
| MarginHu | but the boot.ipxe content seems it is not what I want . | 08:04 |
| aarefiev | MarginHu: what do you want ? | 08:04 |
| *** lucas-afk is now known as lucasagomes | 08:06 | |
| lucasagomes | morning all | 08:07 |
| aarefiev | morning lucasagomes | 08:08 |
| MarginHu | moment. | 08:10 |
| MarginHu | question , how to coordinate 2 dnsmasq ? | 08:12 |
| aarefiev | MarginHu: inspector does it. inspector handles this by managing iptables as a firewall to white/black list nodes, only allowing nodes that should be talking to the inspector's dnsmasq to be served by it. This avoids DHCP races between Ironic and Inspector | 08:17 |
| aarefiev | This due limitation in neutron which we are trying to fix | 08:18 |
| MarginHu | ok , I see, I will try, thanks very much. | 08:18 |
| *** ralonsoh has joined #openstack-ironic | 08:20 | |
| *** ianychoi has quit IRC | 08:20 | |
| *** ianychoi has joined #openstack-ironic | 08:21 | |
| *** racedo has joined #openstack-ironic | 08:22 | |
| *** athomas has quit IRC | 08:23 | |
| *** priteau has joined #openstack-ironic | 08:27 | |
| *** ianychoi has quit IRC | 08:35 | |
| *** athomas has joined #openstack-ironic | 08:36 | |
| *** ianychoi has joined #openstack-ironic | 08:38 | |
| robcresswell | TheJulia: I'm going to be another week or so with decent testing infra for plugins. I'd put time aside to do it, but got thrown some surprise work to solve both upstream and internally thats a priority | 08:39 |
| robcresswell | TheJulia: Happy to hop into the meeting later and repeat that. Again, I apologise for the holdup, but I'm doing what I can | 08:39 |
| *** g3ek has quit IRC | 08:39 | |
| robcresswell | Horizon plugins* that is | 08:39 |
| *** sayantan_ has quit IRC | 08:40 | |
| *** mvpnitesh has joined #openstack-ironic | 08:49 | |
| *** g3ek has joined #openstack-ironic | 08:54 | |
| *** stendulker has joined #openstack-ironic | 08:54 | |
| *** Nisha_Agarwal has quit IRC | 09:03 | |
| *** openstackgerrit has quit IRC | 09:03 | |
| vdrok | good morning ironic! | 09:07 |
| *** mvpnitesh has quit IRC | 09:07 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 09:08 | |
| *** Nisha_ has joined #openstack-ironic | 09:13 | |
| *** Nisha_Agarwal has quit IRC | 09:15 | |
| *** jaganathan has quit IRC | 09:19 | |
| *** Nisha_ has quit IRC | 09:21 | |
| *** openstackgerrit has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Use new oslo db EngineFacade https://review.openstack.org/453057 | 09:23 |
| stendulker | good morning vdrok ! | 09:24 |
| *** sambetts|afk is now known as sambetts | 09:26 | |
| sambetts | Morning all | 09:26 |
| *** l_jacob has joined #openstack-ironic | 09:28 | |
| *** jaganathan has joined #openstack-ironic | 09:31 | |
| aarefiev | morning sambetts, vdrok | 09:32 |
| vdrok | morning stendulker sambetts and aarefiev | 09:32 |
| stendulker | sambetts, aarefiev o/ | 09:33 |
| aarefiev | o/ | 09:33 |
| aarefiev | sambetts: hey, are you fine with this change https://review.openstack.org/#/c/431141/ | 09:33 |
| patchbot | patch 431141 - ironic-inspector - Use processed lldp data, if available, for local_l... | 09:33 |
| sambetts | aarefiev: I'll take a look :) | 09:34 |
| aarefiev | ok :) | 09:34 |
| * aarefiev wondered if we could add dependencies between plugins | 09:35 | |
| sambetts | I have thought about it before | 09:36 |
| openstackgerrit | Ankit Kumar proposed openstack/ironic master: Adding Bootloader in ironic devstack plugin https://review.openstack.org/454026 | 09:38 |
| *** wagiel has joined #openstack-ironic | 09:41 | |
| *** dtantsur|afk is now known as dtantsur | 09:43 | |
| dtantsur | morning ironic | 09:44 |
| dtantsur | aarefiev, we can add simple dependencies, that do not require re-ordering | 09:44 |
| dtantsur | should be a trivial patch | 09:44 |
| *** MattMan has quit IRC | 09:44 | |
| aarefiev | dtantsur: morning, yeah should be easy | 09:45 |
| *** MattMan has joined #openstack-ironic | 09:45 | |
| dtantsur | aarefiev, wanna propose it? I'm thinking of an optional attribute on a hook listing dependencies that much come before it | 09:46 |
| aarefiev | dtantsur: I can file the bug, I'm on oslo.messaging stuff right now | 09:47 |
| dtantsur | aarefiev, yeah, please. I may be able to take it this week | 09:47 |
| *** jaganathan has quit IRC | 09:55 | |
| *** tuanluong_ has joined #openstack-ironic | 10:02 | |
| *** jaganathan has joined #openstack-ironic | 10:02 | |
| dtantsur | https://etherpad.openstack.org/p/ironic-virtual-meetup updated with doodle and request to add timezones | 10:03 |
| *** tuanluong has quit IRC | 10:05 | |
| *** MarginHu has quit IRC | 10:07 | |
| vdrok | morning dtantsur | 10:10 |
| *** zhurong has quit IRC | 10:10 | |
| vdrok | ouch, my vacation covers all the meetup slots :( | 10:11 |
| *** zhurong has joined #openstack-ironic | 10:11 | |
| aarefiev | dtantsur: https://bugs.launchpad.net/ironic-inspector/+bug/1681751 btw rloo asked about rfe-approved tag, should we use it for inspector as well? | 10:12 |
| openstack | Launchpad bug 1681751 in Ironic Inspector "[RFE] Add dependencies between plugins" [Wishlist,Triaged] | 10:12 |
| *** SerenaFeng has quit IRC | 10:17 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Workflow of iPXE usage in Deploy Process https://review.openstack.org/455377 | 10:18 |
| dtantsur | vdrok, ugh :( worry not, if it proves a success, we'll have a few more this cycle :) | 10:20 |
| dtantsur | aarefiev, we did not, but I guess we should | 10:20 |
| stendulker | dtantsur: Is the time mentioned in doodle is UTC? | 10:20 |
| dtantsur | stendulker, doodle should ask you about your timezone, and convert this time | 10:20 |
| dtantsur | I cannot input UTC there | 10:21 |
| dtantsur | sambetts, mind taking a quick look at https://bugs.launchpad.net/ironic-inspector/+bug/1681751 ? I'd like to mark it rfe-approved | 10:21 |
| openstack | Launchpad bug 1681751 in Ironic Inspector "[RFE] Add dependencies between plugins" [Wishlist,Triaged] | 10:21 |
| stendulker | oh ok... I jumped directly to time slots. | 10:21 |
| stendulker | dtantsur: Thank you | 10:21 |
| dtantsur | stendulker, there should be "All times displayed in ..." message in the top of the page. | 10:22 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 10:22 |
| stendulker | dtantsur: No problem. got it | 10:22 |
| sambetts | dtantsur: Yup I think it makes sense | 10:22 |
| dtantsur | k, approving | 10:22 |
| vdrok | dtantsur: oh, good to know | 10:25 |
| *** SerenaFeng has joined #openstack-ironic | 10:29 | |
| *** makowals_ has quit IRC | 10:32 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers https://review.openstack.org/439603 | 10:35 |
| *** ianychoi has quit IRC | 10:36 | |
| *** milan has joined #openstack-ironic | 10:36 | |
| milan | morning Ironic! :) | 10:37 |
| dtantsur | morning milan | 10:37 |
| milan | morning dtantsur! :) | 10:37 |
| *** ianychoi has joined #openstack-ironic | 10:38 | |
| *** SerenaFeng has quit IRC | 10:39 | |
| vdrok | morning milan | 10:39 |
| milan | morning vdrok! :) | 10:39 |
| vdrok | milan: so, wasup with pixie? :) | 10:39 |
| milan | vdrok, ready to send | 10:39 |
| milan | ;) | 10:39 |
| *** jkilpatr has quit IRC | 10:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata https://review.openstack.org/455640 | 10:44 |
| openstackgerrit | paresh sao proposed openstack/ironic master: Change to avoid deploy iso image being built when provided https://review.openstack.org/454625 | 10:45 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Workflow of iPXE usage in Deploy Process https://review.openstack.org/455377 | 10:49 |
| *** jkilpatr has joined #openstack-ironic | 10:59 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Change to avoid building of deploy iso when provided https://review.openstack.org/454625 | 11:03 |
| *** phuongnh has quit IRC | 11:03 | |
| *** mgould|afk is now known as mgould | 11:03 | |
| mgould | hi everyone | 11:03 |
| vdrok | morning mgould | 11:05 |
| *** stendulker has quit IRC | 11:06 | |
| mgould | morning vdrok milan dtantsur sambetts aarefiev | 11:06 |
| milan | morning mgould! :) | 11:06 |
| *** gooduone has joined #openstack-ironic | 11:08 | |
| dtantsur | mgould, o/ | 11:11 |
| *** alexpilotti has joined #openstack-ironic | 11:12 | |
| *** alexpilotti has quit IRC | 11:12 | |
| *** gooduone has quit IRC | 11:13 | |
| *** gooduone has joined #openstack-ironic | 11:14 | |
| *** gooduone has quit IRC | 11:14 | |
| *** gooduone has joined #openstack-ironic | 11:15 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixes multiple chassis delete https://review.openstack.org/455662 | 11:15 |
| aarefiev | morning milan, mgould | 11:19 |
| milan | morning aarefiev! :) | 11:19 |
| *** lucasagomes is now known as lucas-bbl | 11:20 | |
| *** gooduone has quit IRC | 11:23 | |
| *** alexpilotti has joined #openstack-ironic | 11:26 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 11:29 |
| *** patriciadomin has quit IRC | 11:32 | |
| *** rbudden has joined #openstack-ironic | 11:32 | |
| *** livelace has joined #openstack-ironic | 11:33 | |
| *** patriciadomin has joined #openstack-ironic | 11:37 | |
| *** mat128 has joined #openstack-ironic | 11:45 | |
| *** baoli has joined #openstack-ironic | 11:47 | |
| *** baoli has quit IRC | 11:52 | |
| *** ricardoas has joined #openstack-ironic | 11:52 | |
| *** bfournie has quit IRC | 11:55 | |
| *** MarginHu has joined #openstack-ironic | 12:00 | |
| *** e0ne has joined #openstack-ironic | 12:00 | |
| *** SerenaFeng has joined #openstack-ironic | 12:01 | |
| *** MarginHu has quit IRC | 12:01 | |
| *** bcornec has joined #openstack-ironic | 12:04 | |
| *** fellypefca has joined #openstack-ironic | 12:06 | |
| *** tuanluong_ has quit IRC | 12:08 | |
| *** e0ne has quit IRC | 12:11 | |
| *** amotoki has quit IRC | 12:11 | |
| *** wagiel has quit IRC | 12:12 | |
| fellypefca | Morning :) | 12:13 |
| *** lucas-bbl is now known as lucasagomes | 12:16 | |
| *** dprince has joined #openstack-ironic | 12:17 | |
| *** SerenaFeng has quit IRC | 12:19 | |
| *** ianychoi has quit IRC | 12:21 | |
| *** amotoki has joined #openstack-ironic | 12:23 | |
| *** amarao has quit IRC | 12:24 | |
| *** Madasi has quit IRC | 12:24 | |
| *** amarao has joined #openstack-ironic | 12:25 | |
| *** ianychoi has joined #openstack-ironic | 12:26 | |
| *** hshiina has joined #openstack-ironic | 12:27 | |
| *** bfournie has joined #openstack-ironic | 12:28 | |
| *** fellypefca has quit IRC | 12:30 | |
| *** e0ne has joined #openstack-ironic | 12:33 | |
| thomasem | Good morning! | 12:34 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: DevStack: Setup a Redfish environment https://review.openstack.org/445529 | 12:35 |
| lucasagomes | btw folks, if you have some time mind taking a look at https://review.openstack.org/#/c/455255/ ? This is just deleting the tools/ directory from the sushy repo and add some docs about it | 12:36 |
| patchbot | patch 455255 - sushy - Remove the tools/ directory from the sushy tree | 12:36 |
| lucasagomes | I'm just waiting for it to release a version of sushy (the first one) :D | 12:37 |
| *** e0ne has quit IRC | 12:38 | |
| *** amotoki has quit IRC | 12:39 | |
| *** gkadam has quit IRC | 12:40 | |
| *** amotoki has joined #openstack-ironic | 12:43 | |
| *** erolg has joined #openstack-ironic | 12:45 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 12:47 |
| *** e0ne has joined #openstack-ironic | 12:47 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Set pxe_enabled on new and existing ports on introspection https://review.openstack.org/454724 | 12:48 |
| *** amotoki has quit IRC | 12:49 | |
| dtantsur | inspector folks, any objections to marking https://bugs.launchpad.net/ironic-inspector/+bug/1667472 as rfe-approved? I think we agreed on it on the PTG | 12:49 |
| openstack | Launchpad bug 1667472 in Ironic Inspector "[RFE] update PXE enabled" [Wishlist,In progress] - Assigned to Dmitry Tantsur (divius) | 12:49 |
| *** hw_wutianwei has quit IRC | 12:53 | |
| dtantsur | lucasagomes, a few minor comments, if you don't mind | 12:53 |
| lucasagomes | dtantsur, sure thing! Don't mind at all | 12:53 |
| lucasagomes | will take a look and fix 'em | 12:53 |
| *** zhurong has quit IRC | 12:54 | |
| dtantsur | milan, mind checking 2 easy patches please: https://review.openstack.org/#/c/454237/ and https://review.openstack.org/#/c/454648/ | 12:57 |
| patchbot | patch 454237 - ironic-inspector - Add missing test for NodeInfo.create_ports | 12:57 |
| patchbot | patch 454648 - ironic-inspector - Move port creation to validate_interfaces hook | 12:57 |
| dtantsur | they're needed for my pxe_enabled work | 12:57 |
| *** Wei_ has quit IRC | 12:57 | |
| *** bfournie has quit IRC | 12:58 | |
| milan | dtantsur, sure | 12:58 |
| * milan ETOOMANYINTERRUPTS | 12:58 | |
| *** nicodemos has quit IRC | 12:59 | |
| *** sasha2 has joined #openstack-ironic | 13:01 | |
| *** fellypefca has joined #openstack-ironic | 13:04 | |
| *** aparnav has quit IRC | 13:04 | |
| *** jcoufal has joined #openstack-ironic | 13:08 | |
| *** Goneri has joined #openstack-ironic | 13:08 | |
| *** jaypipes has joined #openstack-ironic | 13:10 | |
| *** dyasny has joined #openstack-ironic | 13:11 | |
| *** amotoki has joined #openstack-ironic | 13:12 | |
| *** rloo has joined #openstack-ironic | 13:12 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/sushy master: Remove the tools/ directory from the sushy tree https://review.openstack.org/455255 | 13:16 |
| *** rloo has quit IRC | 13:18 | |
| *** rloo has joined #openstack-ironic | 13:20 | |
| *** nicodemos has joined #openstack-ironic | 13:20 | |
| *** snecklifter has joined #openstack-ironic | 13:20 | |
| *** jaganathan has quit IRC | 13:20 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add functional api tests for node resource class https://review.openstack.org/437963 | 13:21 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add resource classes integration tests https://review.openstack.org/443628 | 13:21 |
| dtantsur | lucasagomes, "Usage" implies production usage, not playing around, hence my request to move emulators to contributing | 13:23 |
| dtantsur | :) | 13:23 |
| lucasagomes | dtantsur, hmm maybe yeah, I wasn't really thinking on the production aspect | 13:24 |
| lucasagomes | dtantsur, do you want me to move it and link one page with another ? | 13:25 |
| * lucasagomes does that, it should be simple | 13:25 | |
| *** xavierr has quit IRC | 13:25 | |
| dtantsur | yep, thanks! | 13:25 |
| *** mat128 has quit IRC | 13:27 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/sushy master: Remove the tools/ directory from the sushy tree https://review.openstack.org/455255 | 13:29 |
| lucasagomes | dtantsur, ^ | 13:29 |
| lucasagomes | and interesting failure on the requirements job: http://logs.openstack.org/55/455255/2/check/gate-sushy-requirements/031c583/console.html#_2017-04-11_13_22_18_194418 | 13:29 |
| lucasagomes | ImportError: No module named openstack_requirements | 13:29 |
| * lucasagomes googles it | 13:29 | |
| openstackgerrit | Merged openstack/ironic-inspector master: Add missing test for NodeInfo.create_ports https://review.openstack.org/454237 | 13:31 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Allow hooks to have dependencies on other hooks https://review.openstack.org/455713 | 13:39 |
| dtantsur | aarefiev, sambetts ^^^ | 13:39 |
| *** udesale has quit IRC | 13:40 | |
| *** jtomasek_ has joined #openstack-ironic | 13:42 | |
| *** vikrant has quit IRC | 13:45 | |
| *** jkilpatr has quit IRC | 13:46 | |
| *** jkilpatr has joined #openstack-ironic | 13:49 | |
| *** crushil has joined #openstack-ironic | 13:50 | |
| *** l_jacob has quit IRC | 13:55 | |
| *** bfournie has joined #openstack-ironic | 13:57 | |
| TheJulia | robcresswell: Thank you, I'm in training this week, if you could mention that in the meeting today that would be awesome. | 13:58 |
| TheJulia | crushil: ^^^ FYI | 13:58 |
| *** jaypipes has quit IRC | 13:58 | |
| robcresswell | TheJulia: Will do | 13:58 |
| crushil | TheJulia, Thanks. Will do | 13:59 |
| *** jaypipes has joined #openstack-ironic | 14:01 | |
| mariojv | morning ironic o/ | 14:01 |
| *** [1]cdearborn is now known as cdearborn | 14:01 | |
| *** xavierr has joined #openstack-ironic | 14:05 | |
| *** ianychoi has quit IRC | 14:05 | |
| *** jaosorior is now known as jaosorior_away | 14:06 | |
| *** ianychoi has joined #openstack-ironic | 14:06 | |
| *** e0ne has quit IRC | 14:08 | |
| fellypefca | dtantsur Hi :) Can you take a look at the one time boot patch (https://review.openstack.org/#/c/436469/) that xavierr commented yesterday in the open discussion? | 14:09 |
| patchbot | patch 436469 - ironic - Introduces onetime boot flag in oneview drivers | 14:09 |
| fellypefca | Could you look at the last comment? The answer we gave to lucasagomes and yuriyz about a possible implementation solution. Do you agree with it? | 14:09 |
| fellypefca | dtantsur ^^^ | 14:10 |
| dtantsur | fellypefca, in a meeting, will try to get to it later | 14:10 |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Deprecate pre-allocation model for OneView drivers https://review.openstack.org/455427 | 14:10 |
| fellypefca | dtantsur ok, thanks! :) | 14:11 |
| *** aparnav has joined #openstack-ironic | 14:12 | |
| *** aparna has joined #openstack-ironic | 14:16 | |
| *** aparnav has quit IRC | 14:16 | |
| *** chlong has joined #openstack-ironic | 14:22 | |
| *** spartacloud is now known as zackf | 14:26 | |
| -openstackstatus- NOTICE: latest base images have mistakenly put python3 in some places expecting python2 causing widespread failure of docs patches - fixes are underway | 14:26 | |
| *** jkilpatr has quit IRC | 14:26 | |
| *** aparna has quit IRC | 14:28 | |
| *** leetpy has joined #openstack-ironic | 14:28 | |
| *** aparnav has joined #openstack-ironic | 14:33 | |
| *** leetpy has quit IRC | 14:33 | |
| *** aparnav has quit IRC | 14:37 | |
| *** aparnav has joined #openstack-ironic | 14:37 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add resource classes integration tests https://review.openstack.org/443628 | 14:39 |
| *** jkilpatr has joined #openstack-ironic | 14:40 | |
| *** aparnav has quit IRC | 14:41 | |
| *** aparnav has joined #openstack-ironic | 14:41 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Rescue implementation for iLO Virtual Media drivers https://review.openstack.org/453012 | 14:45 |
| -openstackstatus- NOTICE: we have rolled back centos-7, fedora-25 and ubuntu-xenial images to the previous days release. Feel free to recheck your jobs now. | 14:47 | |
| *** ianychoi has quit IRC | 14:48 | |
| *** baoli has joined #openstack-ironic | 14:49 | |
| *** mtanino has joined #openstack-ironic | 14:49 | |
| *** adisky_ has quit IRC | 14:49 | |
| *** ianychoi has joined #openstack-ironic | 14:52 | |
| *** jlvillal_pto is now known as jlvillal | 14:52 | |
| *** srobert has joined #openstack-ironic | 14:58 | |
| *** jaganathan has joined #openstack-ironic | 15:00 | |
| *** jcoufal_ has joined #openstack-ironic | 15:01 | |
| *** hshiina has quit IRC | 15:03 | |
| *** jcoufal has quit IRC | 15:03 | |
| *** rcernin has quit IRC | 15:05 | |
| *** Goneri has quit IRC | 15:06 | |
| *** rpioso has joined #openstack-ironic | 15:09 | |
| *** Goneri has joined #openstack-ironic | 15:10 | |
| *** baha has joined #openstack-ironic | 15:18 | |
| *** baoli_ has joined #openstack-ironic | 15:20 | |
| *** aarefiev is now known as aarefiev_afk | 15:22 | |
| *** baoli has quit IRC | 15:22 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: Support oslo.messaging.zmq in ironic https://review.openstack.org/445550 | 15:22 |
| openstackgerrit | Aparna proposed openstack/proliantutils master: Performs IPA erase_devices after Sanitize disk erase https://review.openstack.org/436450 | 15:23 |
| *** gcb has quit IRC | 15:24 | |
| NobodyCam | Good Morning Ironic'ers | 15:24 |
| *** erolg has quit IRC | 15:25 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 15:27 |
| *** chlong_ has joined #openstack-ironic | 15:29 | |
| *** aparnav has quit IRC | 15:30 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add resource classes integration tests https://review.openstack.org/443628 | 15:30 |
| vdrok | morning NobodyCam | 15:31 |
| NobodyCam | Morning vdrok :) | 15:31 |
| lucasagomes | is the gate docs job broken ? http://logs.openstack.org/29/445529/7/check/gate-ironic-docs-ubuntu-xenial/6b9fb28/console.html#_2017-04-11_12_43_39_090247 | 15:31 |
| NobodyCam | should be fixed now lucasagomes ... they (infra ) rolled bak the images | 15:31 |
| lucasagomes | ah awesome! | 15:32 |
| *** chlong has quit IRC | 15:32 | |
| *** crushil has quit IRC | 15:39 | |
| jlvillal | mariojv, I was reading the back-scroll. Did you get any other opinions on the API patch and where it should go in the series? | 15:39 |
| JayF | jlvillal: I told him I didn't think it made any sense to have a driver api implementation without anything for it to plug into | 15:41 |
| jlvillal | JayF, Well that isn't how I remember it being done in the past. | 15:41 |
| jlvillal | JayF, Usually all the code went in and the last thing was the API. | 15:42 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:42 | |
| *** tuan_luong has joined #openstack-ironic | 15:42 | |
| jlvillal | jroll, rloo dtantsur vdrok sambetts vsaienk0: What do you think about patch set ordering? Should API be first patch in series or last patch in series? | 15:42 |
| jlvillal | This is about the rescue patches. | 15:42 |
| *** vinsh has quit IRC | 15:43 | |
| *** tuanla_ has joined #openstack-ironic | 15:43 | |
| jlvillal | I thought in the past we usually did API last. | 15:43 |
| jlvillal | lucasagomes, ^^ | 15:43 |
| rloo | jlvillal: in general, API is usually at the end | 15:43 |
| jlvillal | TheJulia, ^^ | 15:43 |
| mariojv | fwiw, the api/conductor code and rescuewait code have been reviewed more thoroughly and are closer than the agent driver patch, imho | 15:43 |
| dtantsur | jlvillal, last | 15:43 |
| *** vinsh has joined #openstack-ironic | 15:43 | |
| rloo | jlvillal: i don't see how you can expose the API if the stuff isn't working but i haven't looked at rescue. maybe theirs is diff. | 15:43 |
| lucasagomes | rloo, jlvillal yeah usually API is the last so we don't expose an incomplete feature to the users | 15:43 |
| dtantsur | yes, this is why ^^^ | 15:43 |
| *** chlong has joined #openstack-ironic | 15:44 | |
| jlvillal | Thanks for the input. | 15:44 |
| JayF | I wish someone had brought this up in review before mariojv and I put the patches up in a different order and nobody complained until it's close to landing | 15:45 |
| mariojv | is this how we handled/are handling the new volume stuff, too? | 15:45 |
| * mariojv looks | 15:45 | |
| lucasagomes | JayF, yeah sorry for that... tho, I think it should be alright for rescue since it's really a framework, the API will be there but it just won't have any implementation in-tree | 15:46 |
| vdrok | morning jlvillal rloo JayF and mariojv ! | 15:46 |
| mariojv | hi vdrok | 15:46 |
| mariojv | maybe console is a closer example, i'll look at that instead | 15:46 |
| dtantsur | lucasagomes++ we don't have to have *implementations* ready, just the very mechanism should work | 15:46 |
| vdrok | rloo: what was the issue with grenade you mentioned yesterday? was away the whole day | 15:47 |
| lucasagomes | dtantsur, yup | 15:47 |
| *** wagiel has joined #openstack-ironic | 15:47 | |
| rloo | vdrok: the changes to multinode grenade so it can test rolling upgrades. what's the status with that? jroll was asking i think. | 15:47 |
| vdrok | ah, right. vsaienk0 is on sick leave, I'll try to look into this today/tomorrow | 15:48 |
| rloo | vdrok: it's our weekly priority 'make a change to the grenade job to only upgrade conductor' | 15:48 |
| vdrok | yup, seen that | 15:48 |
| *** e0ne has joined #openstack-ironic | 15:48 | |
| rloo | vdrok: ok thx. sorry that vsaienk0 is off and not having fun | 15:48 |
| TheJulia | jlvillal: my 0.02 is last patch. Sorry for the delay, in training this week | 15:49 |
| vdrok | rloo: in case of grenade it is type 3 fun:) | 15:49 |
| rloo | vdrok: :D and Hi! | 15:50 |
| dtantsur | morning rloo, JayF, mariojv, jlvillal | 15:50 |
| *** livelace has quit IRC | 15:50 | |
| dtantsur | and NobodyCam | 15:50 |
| *** e0ne has quit IRC | 15:50 | |
| jlvillal | Hi all :) | 15:50 |
| JayF | o/ | 15:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Allow hooks to have dependencies on other hooks https://review.openstack.org/455713 | 15:51 |
| NobodyCam | hey hey Morning dtantsur jlvillal JayF role and TheJulia | 15:51 |
| jlvillal | :) | 15:51 |
| NobodyCam | :) | 15:51 |
| *** mjura has quit IRC | 15:51 | |
| dtantsur | folks, another reminder about the virtual midcycle planning, and especially the doodle: https://beta.doodle.com/poll/p6rydi6stinqzfrz | 15:51 |
| *** Administrator_ has quit IRC | 15:51 | |
| *** e0ne has joined #openstack-ironic | 15:51 | |
| mariojv | morning dtantsur, everyone i missed | 15:51 |
| dtantsur | (btw, does it show the correct timezone for everyone?) | 15:52 |
| *** e0ne has quit IRC | 15:52 | |
| mariojv | dtantsur: yep, shows us eastern for me | 15:52 |
| jlvillal | dtantsur, Shows my timezone. | 15:52 |
| *** Administrator_ has joined #openstack-ironic | 15:52 | |
| rloo | hi dtantsur, NobodyCam, JayF, mariojv, jlvillal | 15:52 |
| *** e0ne has joined #openstack-ironic | 15:52 | |
| NobodyCam | :) | 15:52 |
| *** e0ne has quit IRC | 15:53 | |
| *** e0ne has joined #openstack-ironic | 15:53 | |
| mariojv | hi rloo | 15:53 |
| *** e0ne has quit IRC | 15:53 | |
| dtantsur | huh, nearly nobody so far is up for the APAC time slot | 15:53 |
| mariojv | so for the rescue patches - i think my strategy will be to see tempest passing first, then do the reordering, unless anyone really wants me to do that sooner | 15:54 |
| rloo | dtantsur: no surprise but maybe wait a couple of days | 15:54 |
| *** e0ne has joined #openstack-ironic | 15:54 | |
| rloo | dtantsur: my recollection is that not much happened during apac time at last midcycle | 15:54 |
| dtantsur | true | 15:54 |
| rloo | dtantsur: (not that i attended) | 15:54 |
| *** e0ne has quit IRC | 15:54 | |
| *** e0ne has joined #openstack-ironic | 15:55 | |
| mariojv | i'm ok with a -2 on the rescuewait/api+conductor before the reorder to prevent them landing before the driver code, also | 15:55 |
| mariojv | jlvillal: ^ | 15:55 |
| *** e0ne has quit IRC | 15:55 | |
| jlvillal | mariojv, Do you want to do it? I trust you :) | 15:55 |
| mariojv | jlvillal: i can do it :) | 15:55 |
| lucasagomes | mariojv, it should be alright tho | 15:55 |
| JayF | so it sounds like we want to have all the conductor code hooked up before the api stuff lands? | 15:56 |
| JayF | but the driver implementation doesn't need to be done/ | 15:56 |
| mariojv | JayF: driver code - api + conductor are together | 15:56 |
| lucasagomes | as long as the mechanism lands together (api + conductor) we can land the implementation later | 15:56 |
| mariojv | 1. driver code 2. rescuewait 3. api/conductor | 15:56 |
| JayF | well someone above said it was OK, as long as the api worked, to not have any implentations | 15:56 |
| JayF | yeah exactly | 15:56 |
| mariojv | api+conductor are already together lol | 15:56 |
| lucasagomes | yeah | 15:56 |
| JayF | lucasagomes: that's what we've done -- API and conductor have always been together | 15:56 |
| lucasagomes | so it's not a broken api | 15:56 |
| lucasagomes | usually api go last cause it depends on the feature, but for rescue | 15:57 |
| lucasagomes | it's a framework | 15:57 |
| *** vinsh_ has joined #openstack-ironic | 15:57 | |
| lucasagomes | it's ok IMHO to have the framework first and then implementation later | 15:57 |
| lucasagomes | the problem would be land the api w/o the conductor bits first | 15:57 |
| mariojv | lucasagomes: that wouldn't happen in the current state of the patches | 15:58 |
| lucasagomes | mariojv, awesome, so I think we can keep the current order | 15:58 |
| *** xavierr has quit IRC | 15:59 | |
| *** jistr is now known as jistr|mtg | 15:59 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: Move port creation to validate_interfaces hook https://review.openstack.org/454648 | 15:59 |
| mariojv | jlvillal: are you ok with that? | 15:59 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/ironic master: Add api tests for portgroup update https://review.openstack.org/450855 | 15:59 |
| *** vinsh has quit IRC | 16:00 | |
| *** jaganathan has quit IRC | 16:02 | |
| NobodyCam | question anyone know any good HOWTO docs out there for python-dracClient? | 16:04 |
| mariojv | i might have to make sure that NotImplementedError is raised appropriately (it might be already), but that can be addressed in the patch itself | 16:04 |
| mariojv | just want to get consensus on this first | 16:04 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 16:04 |
| *** crushil has joined #openstack-ironic | 16:04 | |
| jlvillal | mariojv, So I'm confused. The first patch is API. Is also all conductor work too? | 16:05 |
| jlvillal | s/Is also/Is it also/ | 16:05 |
| JayF | jlvillal: first patch is API + conductor | 16:05 |
| mariojv | jlvillal: yes, it's all together - rescuewait timeout is the only thing that's separate | 16:05 |
| mariojv | https://review.openstack.org/#/c/350831/38/ironic/conductor/manager.py | 16:06 |
| patchbot | patch 350831 - ironic - Add API and conductor methods for [un]rescue | 16:06 |
| jlvillal | mariojv, JayF Okay. I'll trust you and lucasagomes if you think it is good that way :) | 16:06 |
| JayF | I am not an expert on API patches by any means :) | 16:06 |
| JayF | so don't count my vote for much | 16:06 |
| dtantsur | NobodyCam, only https://docs.openstack.org/developer/python-dracclient/ | 16:06 |
| JayF | lol | 16:06 |
| * jlvillal still recovering from a cold and not thinking great today. | 16:06 | |
| lucasagomes | jlvillal, cool! Oh and hope you get well soon | 16:06 |
| mariojv | alright cool. i'll make a note to ensure that the appropriate error is raised if a driver doesn't implement the rescue interface, also | 16:07 |
| jlvillal | lucasagomes, Thanks. It is better today. | 16:07 |
| *** ianychoi has quit IRC | 16:07 | |
| dtantsur | NobodyCam, rpioso, cdearborn or myself can help too, I guess | 16:07 |
| lucasagomes | JayF, if should be fine, as a thumb rule I think API should go last but, I don't see any problems in the case of rescue there (as you said, the whole mechanism is included in one patch) | 16:07 |
| mariojv | it's also fine if we want to encourage folks to review in reverse order | 16:08 |
| jlvillal | My only concern was that once the actual details of implementing things happened, we would discover an issue with the API and need to make a change. | 16:08 |
| mariojv | something like ipa -> ironic driver -> rescuewait -> api/conductor | 16:08 |
| mariojv | then they all land at the same time | 16:08 |
| jlvillal | I think that is one of the reasons for doing the API last, In case figure out that oh it can't work like that. | 16:09 |
| mariojv | jlvillal: the code details within conductor might change, but i'm almost certain that the actual api itself won't change | 16:09 |
| NobodyCam | dtantsur: ya I been looking at that, I am attempting a simple client.list_nics () and just get a empty list ([]) back | 16:09 |
| jlvillal | mariojv, Okay. | 16:09 |
| mariojv | i.e. you'll always have to call /provision with rescue, specifying a rescue password | 16:09 |
| dtantsur | NobodyCam, huh. try catching rpioso then | 16:10 |
| milan | aarefiev_afk, dtantsur, sambetts, guys, please, lemme know wdyt about the shape of the pxe filter interface https://review.openstack.org/#/c/439603/6/ironic_inspector/pxe_filter.py once you've got some time, thx! | 16:10 |
| patchbot | patch 439603 - ironic-inspector - PXE boot filtering drivers | 16:10 |
| NobodyCam | Ack ty :) | 16:10 |
| * milan will amend spec based on reviews here | 16:10 | |
| sambetts | milan: will take a look # | 16:11 |
| dtantsur | milan, I will, but not today - have to leave soon | 16:11 |
| sambetts | same ^ | 16:11 |
| *** ianychoi has joined #openstack-ironic | 16:12 | |
| milan | dtantsur, sambetts sure thing, thx! | 16:12 |
| *** trown is now known as trown|lunch | 16:13 | |
| *** baoli_ has quit IRC | 16:14 | |
| *** d0ugal has quit IRC | 16:18 | |
| openstackgerrit | Beth Elwell proposed openstack/ironic-ui master: Setup JavaScript test environment https://review.openstack.org/453730 | 16:19 |
| *** udesale has joined #openstack-ironic | 16:20 | |
| *** Serlex has quit IRC | 16:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector master: [WIP] Set pxe_enabled on new and existing ports on introspection https://review.openstack.org/454724 | 16:21 |
| *** wagiel has quit IRC | 16:21 | |
| *** derekh has quit IRC | 16:23 | |
| dtantsur | see you tomorrow! | 16:24 |
| mgould | dtantsur: o/ | 16:24 |
| sambetts | cya dtantsur | 16:24 |
| *** dtantsur is now known as dtantsur|afk | 16:24 | |
| *** baoli has joined #openstack-ironic | 16:24 | |
| NobodyCam | night dtantsur|afk | 16:27 |
| *** fellypefca has quit IRC | 16:28 | |
| *** pcaruana has quit IRC | 16:29 | |
| *** xavierr has joined #openstack-ironic | 16:31 | |
| *** jistr|mtg is now known as jistr | 16:37 | |
| *** alexpilotti has quit IRC | 16:39 | |
| *** alexpilotti has joined #openstack-ironic | 16:40 | |
| *** udesale has quit IRC | 16:40 | |
| *** alexpilotti has quit IRC | 16:44 | |
| *** amarao has quit IRC | 16:44 | |
| *** xavierr has quit IRC | 16:44 | |
| lucasagomes | dtantsur|afk, see ya | 16:45 |
| lucasagomes | calling it a day as well, have a great night all! | 16:45 |
| *** lucasagomes is now known as lucas-afk | 16:45 | |
| *** crushil has quit IRC | 16:48 | |
| JayF | https://bugs.launchpad.net/ironic/+bug/1679260 this is an interesting bug | 16:49 |
| openstack | Launchpad bug 1679260 in Ironic "If provisioning network is changed, Ironic conductor does not behave correctly" [Undecided,New] - Assigned to M V P Nitesh (m-nitesh) | 16:49 |
| JayF | I almost feel like what we're doing is what's expected, but we don't communicate well how we handle that | 16:49 |
| JayF | pretty much, if you use a network name as cleaning_network or provisioning_network, delete and recreate that network, ir-cond holds on to the old network uuid | 16:50 |
| *** fragatina has joined #openstack-ironic | 16:51 | |
| *** fragatina has quit IRC | 16:53 | |
| mariojv | oh, we do cache that somewhere | 16:53 |
| *** fragatina has joined #openstack-ironic | 16:53 | |
| JayF | yeah, I'm just confused if we intended that to be dynamic | 16:54 |
| JayF | or just wanted to use the name as "dns" for the uuid | 16:54 |
| pas-ha | that means there is no point really to allow names for those networks | 16:54 |
| JayF | I guess so, I just wanted to verify folks had the expectation of dynamic ability | 16:56 |
| JayF | before I triaged that bug :) | 16:56 |
| *** ralonsoh has quit IRC | 16:56 | |
| thomasem | Can anyone advise on the most salient resources for gaining a better understanding of versioned objects implementation in OpenStack? | 16:59 |
| thomasem | resource(s) :) | 16:59 |
| JayF | thomasem: assuming you've read the code of oslo.versionedobjects? | 16:59 |
| *** mgagne has joined #openstack-ironic | 16:59 | |
| * mgould unsubscribes from openstack-dev | 17:00 | |
| JayF | I think ironic and nova both use that library | 17:00 |
| mgould | end of an era... | 17:00 |
| JayF | mgould: :( | 17:00 |
| JayF | mgould: your mail server thanks you though | 17:00 |
| JayF | lol | 17:00 |
| mgould | yep :-) | 17:00 |
| *** dyasny has quit IRC | 17:00 | |
| sambetts | JayF: I think that the conductor should re-collect the uuid from neutron | 17:01 |
| mgould | now to unsubscribe from all the internal RH mailing lists | 17:01 |
| thomasem | JayF: I have not read the code itself, no. I was looking through here https://docs.openstack.org/developer/oslo.versionedobjects/, but wasn't sure if there was something with more... context, I guess? | 17:01 |
| *** alexpilotti has joined #openstack-ironic | 17:01 | |
| JayF | thomasem: Yeah, I'm not sure -- you could look at how ironic uses it though | 17:01 |
| JayF | mariojv: ^ did you learn any neato details about versionedobjects working on notifications that thomasem could find useful? | 17:02 |
| thomasem | JayF: Okay. I was looking through this review https://review.openstack.org/#/c/412397/43 which prompted me to wonder more about it | 17:02 |
| patchbot | patch 412397 - ironic - Add version column | 17:02 |
| thomasem | Thanks, by the way. :) | 17:03 |
| *** amotoki has quit IRC | 17:04 | |
| thomasem | Ohh, this is probably helpful https://etherpad.openstack.org/p/ironic-online-upgrade-support | 17:05 |
| JayF | np, I don't know a good response to your question :) | 17:05 |
| JayF | yeah, if you havent' read the spec either | 17:05 |
| JayF | that's worthwhile | 17:05 |
| *** wagiel has joined #openstack-ironic | 17:05 | |
| JayF | thomasem: https://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/support-rolling-upgrade.html | 17:05 |
| thomasem | Okay, cool. Haha, no worries. Navigating my way to a good entry point. Looking at the review feels a bit like trying to figure it out in the wrong direction. :P | 17:06 |
| JayF | so a thing I do, when I'm starting review on a new feature | 17:06 |
| JayF | I read the commit message | 17:06 |
| JayF | follow the link back to the bug and find a spec/rfe that spawned it, and read that | 17:06 |
| JayF | try to form a mental idea of what is being attempted | 17:06 |
| JayF | before even reading teh code | 17:06 |
| JayF | because I should be able to assertain what folks are trying to do without havnig to read the code at all | 17:06 |
| *** xavierr has joined #openstack-ironic | 17:06 | |
| thomasem | Yeah, agreed. | 17:07 |
| thomasem | Up the tree we go | 17:07 |
| thomasem | Thanks a 'mil JayF! | 17:08 |
| JayF | pas-ha: sambetts: https://bugs.launchpad.net/ironic/+bug/1679260 I marked this as Triaged/High, after the comments from you all making me see how important it is | 17:08 |
| openstack | Launchpad bug 1679260 in Ironic "If provisioning network is changed, Ironic conductor does not behave correctly" [High,Triaged] - Assigned to M V P Nitesh (m-nitesh) | 17:08 |
| JayF | thomasem: no problem | 17:08 |
| sambetts | JayF: awesome thanks, I've also left a comment pointing to the problem code :) | 17:09 |
| JayF | Is this a JFDI quick fix, then? | 17:09 |
| * JayF wonders if there's any performance considerations to removing that check unconditionally | 17:09 | |
| *** wagiel has quit IRC | 17:09 | |
| JayF | sambetts: I think I might look to patch this today, but I'll change the config description to make it clear UUID is more performant (saves a neutron lookup) | 17:10 |
| sambetts | JayF: one more call to neutron per network action involving neutron I don't think will be too harsh | 17:10 |
| JayF | it's still a change that someone at scale could easily notice | 17:10 |
| sambetts | +1 | 17:10 |
| JayF | so I wanna make sure we release note it | 17:10 |
| * JayF has about a dozen bugs to finish triaging first | 17:11 | |
| sambetts | JayF: alternative is to respond to a "Network Not found" exception with a re-lookup of the UUID | 17:11 |
| JayF | my alternative was going to be put a timer on the cache | 17:11 |
| JayF | but I think not caching it, and letting deployers know putting a name there has a cost makes the most sense | 17:12 |
| JayF | we're going to be calling neutron a bunch to configure the ports anyway | 17:12 |
| sambetts | yeah I think that is the lease likely to break | 17:12 |
| sambetts | exactly | 17:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor API policy and microversion checks https://review.openstack.org/451506 | 17:13 |
| *** xavierr has quit IRC | 17:13 | |
| vdrok | good night everyone! | 17:14 |
| sambetts | night vdrok | 17:15 |
| * sambetts -> home | 17:15 | |
| *** ChubYann has joined #openstack-ironic | 17:15 | |
| sambetts | Night all | 17:15 |
| *** sambetts is now known as sambetts|afk | 17:15 | |
| *** tuanla_ has quit IRC | 17:16 | |
| *** tuan_luong has quit IRC | 17:16 | |
| *** crushil has joined #openstack-ironic | 17:16 | |
| *** mtanino has quit IRC | 17:21 | |
| *** milan has quit IRC | 17:22 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add validation for OneView connections https://review.openstack.org/377106 | 17:24 |
| *** tesseract has quit IRC | 17:26 | |
| NobodyCam | night sambetts|afk | 17:26 |
| NobodyCam | night vdrok | 17:27 |
| *** xavierr has joined #openstack-ironic | 17:27 | |
| NobodyCam | rpioso: if you have a free minute I have a question about python-dracclient | 17:29 |
| * mgould -> home; if I don't make it in tomorrow (there's every chance the courier will turn up to take my laptop away several hours early), it's been great working with you all! | 17:31 | |
| *** mgould is now known as mgould|afk | 17:31 | |
| JayF | mgould|afk: thanks, it's been nice o/ | 17:31 |
| NobodyCam | enjoy your need journey mgould|afk | 17:31 |
| NobodyCam | need = new :p | 17:31 |
| *** dsneddon has joined #openstack-ironic | 17:33 | |
| *** Nisha_Agarwal has quit IRC | 17:38 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 17:38 | |
| rajinir | Having some networking issue with devstack recently. Name resolution fails in the middle of ironic installation. Anyone facing this? | 17:43 |
| JayF | if name resolution fails, that seems like it has to be local, right? | 17:45 |
| JayF | the ironic install happens inside of devstack, so I don't think anything at that point is done with the local network that would break it? | 17:45 |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: Bypass eventlet sendto when detected https://review.openstack.org/455791 | 17:47 |
| *** Goneri has quit IRC | 17:47 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: Bypass eventlet sendto when detected https://review.openstack.org/455791 | 17:49 |
| *** fragatina has quit IRC | 17:57 | |
| *** baoli has quit IRC | 17:59 | |
| *** hshiina has joined #openstack-ironic | 17:59 | |
| *** wagiel has joined #openstack-ironic | 17:59 | |
| *** baoli has joined #openstack-ironic | 17:59 | |
| *** hshiina has quit IRC | 18:03 | |
| *** wagiel has quit IRC | 18:05 | |
| *** mtanino has joined #openstack-ironic | 18:06 | |
| *** sai is now known as sai-afk | 18:08 | |
| *** trown|lunch is now known as trown | 18:20 | |
| *** sai-afk is now known as sai | 18:22 | |
| *** Sukhdev has joined #openstack-ironic | 18:26 | |
| *** priteau has quit IRC | 18:26 | |
| *** crushil has quit IRC | 18:33 | |
| *** mgagne_ has joined #openstack-ironic | 18:35 | |
| *** mgagne_ is now known as Guest89317 | 18:35 | |
| *** jcoufal has joined #openstack-ironic | 18:36 | |
| *** jcoufal_ has quit IRC | 18:38 | |
| *** Guest89317 has quit IRC | 18:46 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 18:48 |
| *** baoli has quit IRC | 18:49 | |
| *** fragatina has joined #openstack-ironic | 18:53 | |
| *** crushil has joined #openstack-ironic | 18:53 | |
| *** mat128 has joined #openstack-ironic | 18:53 | |
| *** wagiel has joined #openstack-ironic | 18:54 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 18:55 |
| *** alexpilotti has quit IRC | 18:57 | |
| *** baoli has joined #openstack-ironic | 18:57 | |
| *** alexpilotti has joined #openstack-ironic | 18:57 | |
| *** wagiel has quit IRC | 18:58 | |
| *** fragatina has quit IRC | 19:00 | |
| *** alexpilotti has quit IRC | 19:02 | |
| openstackgerrit | Merged openstack/ironic master: Add comments re RPC versions being in sync https://review.openstack.org/455536 | 19:04 |
| *** mat128 has quit IRC | 19:04 | |
| *** mat128 has joined #openstack-ironic | 19:05 | |
| *** Goneri has joined #openstack-ironic | 19:16 | |
| *** racedo has quit IRC | 19:16 | |
| openstackgerrit | Merged openstack/pyghmi master: Bypass eventlet sendto when detected https://review.openstack.org/455791 | 19:18 |
| *** mgagne_ has joined #openstack-ironic | 19:18 | |
| *** mgagne_ is now known as Guest4366 | 19:19 | |
| *** Nisha_Agarwal has quit IRC | 19:20 | |
| *** mgagne has quit IRC | 19:24 | |
| *** Guest4366 is now known as mgagne | 19:25 | |
| *** mgagne has quit IRC | 19:25 | |
| *** mgagne has joined #openstack-ironic | 19:25 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add new dbsync command with first online data migration https://review.openstack.org/408556 | 19:28 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: Add version column https://review.openstack.org/412397 | 19:28 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 19:28 |
| *** ianychoi has quit IRC | 19:34 | |
| *** mat128 has quit IRC | 19:34 | |
| *** ianychoi has joined #openstack-ironic | 19:35 | |
| *** alexpilotti has joined #openstack-ironic | 19:37 | |
| *** sayantan_ has joined #openstack-ironic | 19:39 | |
| *** alexpilotti has quit IRC | 19:41 | |
| *** fragatina has joined #openstack-ironic | 19:43 | |
| *** jkilpatr_ has joined #openstack-ironic | 19:45 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Add validation for OneView connections https://review.openstack.org/377106 | 19:47 |
| *** jkilpatr has quit IRC | 19:47 | |
| *** wagiel has joined #openstack-ironic | 19:48 | |
| *** wagiel has quit IRC | 19:52 | |
| openstackgerrit | Hugo Nicodemos proposed openstack/ironic master: Deprecate pre-allocation model for OneView drivers https://review.openstack.org/455427 | 20:00 |
| *** jcoufal has quit IRC | 20:00 | |
| *** sayantan_ has quit IRC | 20:03 | |
| *** sayantan_ has joined #openstack-ironic | 20:05 | |
| openstackgerrit | Anup Navare proposed openstack/ironic master: Config drive support for ceph radosgw https://review.openstack.org/407639 | 20:08 |
| *** srobert_ has joined #openstack-ironic | 20:09 | |
| *** srobert has quit IRC | 20:13 | |
| *** srobert_ has quit IRC | 20:15 | |
| *** sayantan_ has quit IRC | 20:20 | |
| *** sdake has joined #openstack-ironic | 20:24 | |
| *** sdake has quit IRC | 20:24 | |
| *** sdake has joined #openstack-ironic | 20:24 | |
| *** dprince has quit IRC | 20:27 | |
| *** wagiel has joined #openstack-ironic | 20:42 | |
| *** sayantan_ has joined #openstack-ironic | 20:43 | |
| *** wagiel_ has joined #openstack-ironic | 20:43 | |
| *** crushil has quit IRC | 20:44 | |
| *** wagiel_ has quit IRC | 20:44 | |
| *** wagiel_ has joined #openstack-ironic | 20:44 | |
| *** wagiel has quit IRC | 20:47 | |
| *** dmellado has quit IRC | 20:48 | |
| *** wagiel_ has quit IRC | 20:48 | |
| *** athomas has quit IRC | 20:51 | |
| *** jtomasek_ has quit IRC | 20:51 | |
| *** trown is now known as trown|outtypewww | 20:58 | |
| *** sbezverk_ is now known as sbezverk | 21:04 | |
| *** chlong has quit IRC | 21:09 | |
| *** chlong_ has quit IRC | 21:09 | |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 21:13 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 21:13 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: WIP Rescue mode tempest tests https://review.openstack.org/452308 | 21:13 |
| openstackgerrit | Mario Villaplana proposed openstack/ironic master: Add API and conductor methods for [un]rescue https://review.openstack.org/350831 | 21:13 |
| openstackgerrit | Ruby Loo proposed openstack/ironic master: [POC|DNM|TEST] Add node.configdrive https://review.openstack.org/455811 | 21:16 |
| *** g3ek has quit IRC | 21:25 | |
| *** alexpilotti has joined #openstack-ironic | 21:30 | |
| *** jkilpatr_ has quit IRC | 21:30 | |
| *** baha has quit IRC | 21:33 | |
| *** g3ek has joined #openstack-ironic | 21:34 | |
| *** wagiel has joined #openstack-ironic | 21:38 | |
| *** wagiel has quit IRC | 21:42 | |
| *** vills has joined #openstack-ironic | 21:59 | |
| *** jaypipes has quit IRC | 22:02 | |
| *** cdearborn has quit IRC | 22:02 | |
| *** bfournie has quit IRC | 22:02 | |
| *** jkilpatr_ has joined #openstack-ironic | 22:08 | |
| jlvillal | rloo, ^^^ +2 +A | 22:18 |
| jlvillal | ;) | 22:18 |
| *** mtanino has quit IRC | 22:30 | |
| rloo | jlvillal: what?#$# :D | 22:31 |
| *** wagiel has joined #openstack-ironic | 22:32 | |
| *** wagiel has quit IRC | 22:37 | |
| *** r-daneel has joined #openstack-ironic | 22:38 | |
| davidlenwell | has anyone created a flowchart describing the interactions with neutron for the vlan stuff? | 22:43 |
| *** r-daneel has quit IRC | 22:59 | |
| openstackgerrit | Merged openstack/ironic master: Fix some reST field lists in docstrings https://review.openstack.org/451251 | 23:01 |
| *** baoli has quit IRC | 23:21 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Common cinder interface additional improvements https://review.openstack.org/453839 | 23:22 |
| openstackgerrit | Merged openstack/ironic master: Remove translation of log messages from ironic/api https://review.openstack.org/453360 | 23:25 |
| *** vills has quit IRC | 23:25 | |
| *** wagiel has joined #openstack-ironic | 23:26 | |
| *** wagiel has quit IRC | 23:31 | |
| *** rbudden has quit IRC | 23:36 | |
| *** crushil has joined #openstack-ironic | 23:38 | |
| NobodyCam | davidlenwell: I have not seen one ... but it would be really neat | 23:44 |
| *** baoli has joined #openstack-ironic | 23:49 | |
| *** baoli has quit IRC | 23:49 | |
| *** baoli has joined #openstack-ironic | 23:50 | |
| openstackgerrit | Chris Smart proposed openstack/ironic master: WIP: test buildroot IPA image https://review.openstack.org/445763 | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!