| *** pmannidi has quit IRC | 00:03 | |
| *** rbudden has joined #openstack-ironic | 00:18 | |
| *** pmannidi has joined #openstack-ironic | 00:25 | |
| *** pmannidi is now known as pmannidi|Trg | 00:25 | |
| *** pmannidi|Trg is now known as pmannidi | 00:25 | |
| *** fragatina has joined #openstack-ironic | 00:44 | |
| *** fragatina has quit IRC | 00:44 | |
| *** fragatina has joined #openstack-ironic | 00:45 | |
| *** links has joined #openstack-ironic | 00:54 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode https://review.openstack.org/435573 | 00:58 |
|---|---|---|
| *** adreznec has quit IRC | 00:59 | |
| *** hw_wutianwei has joined #openstack-ironic | 01:00 | |
| *** adreznec has joined #openstack-ironic | 01:01 | |
| *** phuongnh has joined #openstack-ironic | 01:02 | |
| *** SerenaFeng has joined #openstack-ironic | 01:24 | |
| *** baoli has joined #openstack-ironic | 01:30 | |
| *** rbudden has quit IRC | 01:42 | |
| *** baoli has quit IRC | 01:46 | |
| *** Sukhdev has joined #openstack-ironic | 01:47 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode https://review.openstack.org/435573 | 01:47 |
| *** fragatina has quit IRC | 01:48 | |
| *** gcb has joined #openstack-ironic | 02:04 | |
| *** shuyingya has joined #openstack-ironic | 02:05 | |
| *** rbudden has joined #openstack-ironic | 02:17 | |
| *** shuyingy_ has joined #openstack-ironic | 02:24 | |
| *** shuyingya has quit IRC | 02:28 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume https://review.openstack.org/466568 | 02:29 |
| *** baoli has joined #openstack-ironic | 03:00 | |
| *** jhesketh_ is now known as jhesketh | 03:08 | |
| *** rbudden has quit IRC | 03:42 | |
| *** vikrant has joined #openstack-ironic | 03:46 | |
| *** baoli has quit IRC | 03:48 | |
| *** SerenaFeng has quit IRC | 04:15 | |
| *** adisky_ has joined #openstack-ironic | 04:21 | |
| *** udesale has joined #openstack-ironic | 04:47 | |
| *** jaosorior has joined #openstack-ironic | 04:52 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: Adds parameters to run CI with hardware types. https://review.openstack.org/465484 | 05:23 |
| *** nmathew has joined #openstack-ironic | 05:28 | |
| *** mvpnitesh has joined #openstack-ironic | 05:31 | |
| *** SerenaFeng has joined #openstack-ironic | 05:32 | |
| *** ChubYann has quit IRC | 05:49 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Add rescuewait timeout periodic task https://review.openstack.org/353156 | 06:06 |
| *** aparnav has joined #openstack-ironic | 06:09 | |
| *** shuyingy_ has quit IRC | 06:13 | |
| *** shuyingya has joined #openstack-ironic | 06:13 | |
| pas-ha | morning ironic :) | 06:15 |
| *** chas has joined #openstack-ironic | 06:19 | |
| *** shuyingy_ has joined #openstack-ironic | 06:19 | |
| *** rcernin has joined #openstack-ironic | 06:19 | |
| *** shuyingya has quit IRC | 06:22 | |
| *** litao__ has joined #openstack-ironic | 06:23 | |
| openstackgerrit | Ngo Quoc Cuong proposed openstack/ironic-inspector master: Replace the deprecated tempest.test.attr with decorators.attr https://review.openstack.org/466604 | 06:24 |
| *** athomas has joined #openstack-ironic | 06:29 | |
| *** chas has quit IRC | 06:37 | |
| openstackgerrit | Aparna proposed openstack/sushy master: Add manager https://review.openstack.org/441236 | 06:37 |
| *** chas has joined #openstack-ironic | 06:37 | |
| *** chas has quit IRC | 06:41 | |
| openstackgerrit | Aparna proposed openstack/ironic master: Agent driver rescue implementation https://review.openstack.org/400437 | 06:41 |
| *** chas has joined #openstack-ironic | 06:41 | |
| *** ccamacho has quit IRC | 06:49 | |
| *** rbartal has joined #openstack-ironic | 06:49 | |
| *** mjura has joined #openstack-ironic | 06:54 | |
| *** chas has quit IRC | 06:55 | |
| *** fragatina has joined #openstack-ironic | 06:55 | |
| *** chas has joined #openstack-ironic | 06:55 | |
| *** chas has quit IRC | 06:57 | |
| *** chas has joined #openstack-ironic | 06:57 | |
| *** SerenaFeng has quit IRC | 06:57 | |
| *** ccamacho has joined #openstack-ironic | 07:01 | |
| *** chas has quit IRC | 07:06 | |
| *** SerenaFeng has joined #openstack-ironic | 07:06 | |
| *** chas has joined #openstack-ironic | 07:06 | |
| *** chas has quit IRC | 07:11 | |
| *** pcaruana has joined #openstack-ironic | 07:11 | |
| *** pcaruana has quit IRC | 07:12 | |
| *** pcaruana has joined #openstack-ironic | 07:13 | |
| *** rcernin has quit IRC | 07:14 | |
| *** rcernin has joined #openstack-ironic | 07:14 | |
| *** vmud213 has joined #openstack-ironic | 07:23 | |
| *** e0ne has joined #openstack-ironic | 07:28 | |
| *** bnemec has quit IRC | 07:29 | |
| *** bnemec has joined #openstack-ironic | 07:35 | |
| *** Serlex has joined #openstack-ironic | 07:38 | |
| *** hoonetorg has quit IRC | 07:39 | |
| *** chas has joined #openstack-ironic | 07:47 | |
| *** racedo has joined #openstack-ironic | 07:47 | |
| *** fragatina has quit IRC | 07:47 | |
| *** Sukhdev has quit IRC | 07:50 | |
| *** fragatina has joined #openstack-ironic | 07:50 | |
| *** hoonetorg has joined #openstack-ironic | 07:54 | |
| *** ralonsoh has joined #openstack-ironic | 07:55 | |
| *** GheRivero has joined #openstack-ironic | 07:58 | |
| *** GheRivero has left #openstack-ironic | 07:59 | |
| *** GheRivero has joined #openstack-ironic | 07:59 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Serve boot configs from Ironic API https://review.openstack.org/369438 | 07:59 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Implement Driver API for dynamic boot config https://review.openstack.org/394398 | 07:59 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Add RPC API to get the boot config https://review.openstack.org/394399 | 08:00 |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Allow not writing pxe configs to disk https://review.openstack.org/394397 | 08:00 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: Convert iPXE boot config to proper Jinja template https://review.openstack.org/393462 | 08:00 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic master: DNM test enabled dynamic iPXE https://review.openstack.org/391943 | 08:00 |
| *** zzzeek has joined #openstack-ironic | 08:01 | |
| *** fragatina has quit IRC | 08:06 | |
| *** e0ne has quit IRC | 08:12 | |
| *** e0ne has joined #openstack-ironic | 08:14 | |
| *** lucas-afk is now known as lucasagomes | 08:15 | |
| lucasagomes | morning all | 08:15 |
| *** ralonsoh has quit IRC | 08:15 | |
| GheRivero | morning lucasagomes | 08:17 |
| *** mvpnitesh has quit IRC | 08:19 | |
| *** ralonsoh has joined #openstack-ironic | 08:19 | |
| *** stendulker has joined #openstack-ironic | 08:25 | |
| *** dtantsur|afk is now known as dtantsur | 08:28 | |
| dtantsur | morning lucasagomes, pas-ha | 08:29 |
| dtantsur | oh morning GheRivero, long time no see | 08:29 |
| *** kromanenko has joined #openstack-ironic | 08:30 | |
| GheRivero | dtantsur: yes, it's been awhile. A couple of years since I left this channel :) | 08:32 |
| dtantsur | are you coming back? :) | 08:32 |
| GheRivero | I hope so. Working on it | 08:33 |
| *** derekh has joined #openstack-ironic | 08:34 | |
| *** mvpnitesh has joined #openstack-ironic | 08:45 | |
| dtantsur | cool | 08:46 |
| *** SerenaFeng has quit IRC | 08:52 | |
| *** jaganathan has joined #openstack-ironic | 08:52 | |
| *** SerenaFeng has joined #openstack-ironic | 08:52 | |
| *** yibo has joined #openstack-ironic | 08:56 | |
| vdrok | good morning pas-ha lucasagomes GheRivero and dtantsur ! | 09:01 |
| vdrok | and morning ironic | 09:01 |
| lucasagomes | o/ | 09:01 |
| dtantsur | o/ | 09:06 |
| *** e0ne has quit IRC | 09:08 | |
| *** e0ne has joined #openstack-ironic | 09:09 | |
| *** dtantsur is now known as dtantsur|lunch | 09:14 | |
| pas-ha | morning dtantsur|lunch lucasagomes vdrok | 09:20 |
| vmud213 | morning lucasagomes,dtantsur,pas-ha, vdrok and Ironicers | 09:24 |
| *** fragatina has joined #openstack-ironic | 09:29 | |
| *** milan has joined #openstack-ironic | 09:30 | |
| milan | morning Ironic! :) | 09:34 |
| openstackgerrit | Aparna proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 09:37 |
| milan | racedo, a bit showing-off ;) https://review.openstack.org/#/c/466448/6/ironic_inspector/pxe_filter/dnsmasq.py meet my braindump dnsmasq pxe filter driver code ;) | 09:40 |
| patchbot | patch 466448 - ironic-inspector - WIP Introducing a dnsmasq PXE filter driver | 09:40 |
| * milan considers updating BZs... | 09:40 | |
| vmud213 | morning milan | 09:42 |
| milan | morning vmud213! :) | 09:42 |
| vmud213 | lucasagomes,dtantsur: Just a small qery on sushy redifsh library | 09:43 |
| vmud213 | How do i figure out post data for a particular URI? | 09:43 |
| racedo | oh wow, quiet a "braindump" :) milan | 09:45 |
| milan | racedo, lol yeah :D | 09:45 |
| *** SerenaFeng has quit IRC | 09:45 | |
| milan | has been looping in my brain for a while | 09:46 |
| * milan glad he got rid of it ;) | 09:46 | |
| racedo | milan, looks like a great start | 09:46 |
| vmud213 | for instance the place i see POST data here for power operations: https://github.com/openstack/sushy/blob/master/sushy/resources/system/system.py#L201-L204 | 09:46 |
| milan | racedo, thx, I gonna take some reviews though ;) | 09:47 |
| racedo | milan, is using systemctl directly fine? (asking from my infinite gnorance :-) | 09:47 |
| milan | racedo, I've checked this a bit, should be able to handle 10k ports OK | 09:47 |
| milan | racedo, I doubt :D I need to figure out; I guess there's going to have to be a config option of a start/stop ctl command of choice | 09:48 |
| racedo | ack | 09:48 |
| lucasagomes | vmud213, hi there, well, you have to look at the documentation/schema | 09:50 |
| lucasagomes | or if you have access to a API you can play with it, the power operation for e.g, the resource returns certain "actions" | 09:51 |
| lucasagomes | which you can do with that resrouce | 09:51 |
| * lucasagomes 1 sec lemme find a link | 09:51 | |
| *** sambetts_ is now known as sambetts | 09:51 | |
| lucasagomes | vmud213, http://redfish.dmtf.org/schemas/DSP0266_1.1.html#actions-property | 09:52 |
| sambetts | Morning all | 09:53 |
| * vmud213 looking the URL | 09:53 | |
| lucasagomes | sambetts, o/ morning | 09:55 |
| milan | morning lucasagomes sambetts! :) | 09:58 |
| lucasagomes | o/ | 10:00 |
| *** fragatina has quit IRC | 10:00 | |
| sambetts | hey lucasagomes milan | 10:01 |
| vdrok | morning vmud213 milan sambetts ! | 10:02 |
| milan | morning vdrok! :) | 10:02 |
| vmud213 | lucasagomes: I just went throught the link u suggested | 10:03 |
| vmud213 | Thats an example for power operation.. | 10:03 |
| *** chas has quit IRC | 10:08 | |
| *** chas has joined #openstack-ironic | 10:09 | |
| *** chas has quit IRC | 10:13 | |
| *** e0ne has quit IRC | 10:13 | |
| *** tuanluong has joined #openstack-ironic | 10:15 | |
| *** dtantsur|lunch is now known as dtantsur | 10:23 | |
| vmud213 | lucasagomes: Do u mean to say, the data given in the "Actions" schema give us some hint and we will try to determine the POST data from there | 10:23 |
| lucasagomes | vmud213, those are the allowed values, not hints | 10:26 |
| lucasagomes | vmud213, sushy has a get_allowed_reset_system_values() method which should return it for you | 10:28 |
| vmud213 | lucasagomes: http://paste.openstack.org/show/610148/ In this case do you mean to say there is no post data? | 10:28 |
| lucasagomes | vmud213, idk, you have to look at the documentation/schema | 10:28 |
| lucasagomes | and see what are the allowed values for those | 10:29 |
| vmud213 | SO If there's nothing mentioned in "Actions" other than the target, then i thnk no POST data is needed? | 10:29 |
| vmud213 | lucasagomes: thanks..Will look into that.. | 10:29 |
| vmud213 | The documentation is messy untill we get to know how to read it :) | 10:30 |
| *** jkilpatr has quit IRC | 10:31 | |
| vmud213 | I mean the schema documetation in DMTF. | 10:31 |
| lucasagomes | vmud213, yeah it's a pain, but, that's the only thing available :-( | 10:31 |
| lucasagomes | you can play with the api urself if you have one available too | 10:32 |
| lucasagomes | or use the mockups (not sure if all possible actions is part of it tho) | 10:32 |
| vmud213 | lucasagomes: Thanks, that's the only option now i have :( | 10:32 |
| lucasagomes | np, yeah g'luck | 10:33 |
| dtantsur | folks, wdyt about https://bugs.launchpad.net/ironic-lib/+bug/1690458 ? | 10:36 |
| openstack | Launchpad bug 1690458 in ironic-lib "[RFE] During cleaning, remove also disk labels (not just partitions)" [Wishlist,Confirmed] | 10:36 |
| dtantsur | seems reasonable, though I don't fully understand why wipefs does not clean everything | 10:37 |
| dtantsur | lucasagomes ^^^ | 10:37 |
| lucasagomes | dtantsur, ++ (withtout seem the patch) I assume it did clear the disk labels | 10:37 |
| lucasagomes | we should, otherwise, if we deploy 2 intances on different disks | 10:38 |
| lucasagomes | and the labels are there we can mess up the configdrive | 10:38 |
| *** racedo has quit IRC | 10:38 | |
| *** racedo has joined #openstack-ironic | 10:39 | |
| *** fragatina has joined #openstack-ironic | 10:39 | |
| dtantsur | yep | 10:41 |
| dtantsur | sambetts: when you have a minute, could you please check https://review.openstack.org/#/c/424729/ ? I'd like to release IPA reasonably soon (we haven't had a Pike release yet), and include this one, if possible. | 10:41 |
| patchbot | patch 424729 - ironic-python-agent - NUMA-topology collector | 10:41 |
| sambetts | dtantsur: I'll try to take a look soon | 10:42 |
| dtantsur | thnx | 10:43 |
| *** chas has joined #openstack-ironic | 10:43 | |
| dtantsur | sambetts: also, do you still need https://review.openstack.org/#/c/434385/ ? | 10:44 |
| patchbot | patch 434385 - ironic-python-agent - [LLDP] Skip NICs that say they are ready but are u... | 10:44 |
| * dtantsur tries to figure our the backlog for IPA | 10:44 | |
| sambetts | dtantsur: that would be really helpful to be included actually and probably backported too I think | 10:45 |
| sambetts | I need to fix it up a bit though | 10:45 |
| *** chas has quit IRC | 10:47 | |
| *** chas has joined #openstack-ironic | 10:47 | |
| *** jkilpatr has joined #openstack-ironic | 10:57 | |
| *** amarao has joined #openstack-ironic | 10:57 | |
| *** lucasagomes is now known as lucas-hungry | 10:57 | |
| * dtantsur reviewed | 11:00 | |
| *** tuanluong has quit IRC | 11:04 | |
| stendulker | dtantsur: Hi | 11:09 |
| dtantsur | hi stendulker | 11:09 |
| *** ccamacho is now known as ccamacho|lunch | 11:10 | |
| stendulker | dtantsur: Proliant Gen10 systems would be supporting Redfish and will not support RIS | 11:10 |
| dtantsur | nice (I guess :) | 11:10 |
| *** xavierr-afk has joined #openstack-ironic | 11:10 | |
| stendulker | Need to add a new Redfish boot interface derived from (iLoBoot) | 11:11 |
| stendulker | Functionalty would be same. Would that need a pec or adding new classes should be fine? | 11:11 |
| dtantsur | stendulker: you mean, for iLO-specific virtual media? | 11:12 |
| stendulker | s/pec/spec | 11:12 |
| stendulker | yes | 11:12 |
| stendulker | virtualmedia and pxe | 11:12 |
| stendulker | everything tht IloBoot* interfaces does | 11:12 |
| dtantsur | stendulker: ok, but if it's specific for your hardware, please don't call it RedfishBoot :) iLo10Boot I guess? anyway, please start with a detailed RFE (no spec for now). | 11:12 |
| stendulker | Yes, Will add vendor name. HPERedfish*Boot | 11:13 |
| dtantsur | works for me | 11:13 |
| stendulker | Also, one more thing. There are non-iLo based _HPE_ servers would want to use this redfish interafce | 11:13 |
| openstackgerrit | Merged openstack/python-ironicclient master: Add testcases for OSC baremetal port group commands https://review.openstack.org/385427 | 11:14 |
| stendulker | Not sure if it be better to change hardware type to 'hpe' instead of 'ilo'? | 11:14 |
| dtantsur | stendulker: it depends on whether you plan to use the same hardware type for both | 11:14 |
| dtantsur | are they functionally identical from Ironic point of view? | 11:15 |
| stendulker | Yes. Would it be better as both would use the same. | 11:15 |
| dtantsur | they = iLO and non-iLO servers | 11:15 |
| stendulker | As far as boot/deploy they are similar. I'm investigating further, but wanted to know if that approach is possible | 11:16 |
| stendulker | yes. ilo and non-ilo Redish based servers | 11:16 |
| *** phuongnh has quit IRC | 11:16 | |
| dtantsur | yes, you can use the same hardware type (hpe-redfish?) for all models that expose the same feature set from Ironic point of view | 11:17 |
| dtantsur | actually, we cannot rename the "ilo" hardware type, I guess, because it also covers hardware that is not redfish-enabled | 11:17 |
| dtantsur | so you need ilo and hpe-redfish, something like that | 11:17 |
| dtantsur | stendulker: does it make sense ^^^? | 11:17 |
| stendulker | ok. So HPERedFish would be a boot interface choice for 'ilo' and 'hpe-redish' hardware type | 11:18 |
| stendulker | That works. It would be cleaner and protects backward compatibity wrt classical driver composition | 11:19 |
| dtantsur | stendulker: does *all* hardware supported by ilo support redfish? if not, then HPERedfish will be a boot interface for hpe-redfish only | 11:19 |
| *** gcb has quit IRC | 11:19 | |
| dtantsur | I guess I need to learn more about capabilities of various HPE hardware | 11:19 |
| stendulker | not all ilo. Only Gen10 and beyond | 11:19 |
| openstackgerrit | Aparna proposed openstack/ironic-python-agent master: Rescue extension for CoreOS with DHCP tenant networks https://review.openstack.org/423521 | 11:19 |
| dtantsur | stendulker: then I guess we need to leave "ilo" hardware type for RIS, and create "hpe-redfish" for redfish-enabled models | 11:20 |
| stendulker | gen9 has, but its more of a pre-standard version. RIS is preferred mechanism over there | 11:20 |
| dtantsur | with "ilo" supporting <= Gen 9, and "hpe-redfish" supporting >= Gen 10 + non-iLO redfish hardware | 11:20 |
| dtantsur | wdyt? | 11:20 |
| stendulker | ok, so you mean gen10 ilo based and all redfish based servers would use hpe-redfish hardware type? | 11:20 |
| dtantsur | yes | 11:21 |
| stendulker | That also works :) | 11:21 |
| dtantsur | good :) | 11:21 |
| stendulker | Even better :) | 11:21 |
| stendulker | One last thing. How do we add HPE specific redfish code? Proliantutils woudl not be a good choice as its not restricted to Proliants | 11:21 |
| *** fragatina has quit IRC | 11:22 | |
| * persia has heard rumours (that should be confirmed) that not all redfish-compliant BMCs have the same set of verbs. | 11:22 | |
| stendulker | Can we add a new hpe redfish lib? | 11:22 |
| sambetts | hey persia o/ | 11:23 |
| dtantsur | stendulker: this is a good question. two options we have: 1. create a new library outside of ironic governance (you don't need my permission for that :) 2. create module e.g. sushy.hpe with vendor-specific bits | 11:23 |
| dtantsur | stendulker: I don't know what I like more. Maybe let's ask on the meeting today? | 11:23 |
| dtantsur | persia: sigh.. to be honest, redfish looks designed with incompatible implementations in mind. | 11:23 |
| stendulker | sure. | 11:24 |
| persia | sambetts: o/ | 11:24 |
| stendulker | we could have a another library based on sushy or extensions within sushy | 11:24 |
| *** lennyb_ has quit IRC | 11:24 | |
| *** lennyb has quit IRC | 11:24 | |
| dtantsur | stendulker: right. please add your question to the agenda. | 11:24 |
| *** racedo has quit IRC | 11:24 | |
| persia | dtantsur: That is what I have been hearing, yes. My fear (which it seems you have anticipated) was that there was a potential for Ironic to gain a "redfish" driver, which I think cannot work except for one vendor. | 11:24 |
| stendulker | dtantsur, persia : All vendors are adding their own extensions to redfish... :( | 11:24 |
| *** lennyb has joined #openstack-ironic | 11:25 | |
| stendulker | Base redfish looks very minimalistic if you look at vendor extensions | 11:25 |
| dtantsur | persia: well, rather we'll have "redfish" driver which is limited to basic power and management only. and then "<vendor name>-redfish" with batteries included | 11:25 |
| stendulker | dtantsur: Sure will add this to agenda. Thank you. | 11:26 |
| dtantsur | persia: which is bad, but may still be better than the situation we have now with IPMI + a big set of completely unrelated vendor protocols | 11:26 |
| dtantsur | thanks stendulker | 11:26 |
| persia | dtantsur: That would make sense. May not be very useful, but perhaps provides a road for collaboration. | 11:26 |
| stendulker | dtantsur: Thank you | 11:26 |
| dtantsur | persia: at least (I hope) we start sharing power and boot management between all vendors :) | 11:26 |
| dtantsur | and inspection | 11:26 |
| * persia also shares that hope, but has low expectations | 11:26 | |
| dtantsur | I do realize that RAID and virtual media are probably not going to ever be common.. | 11:27 |
| *** lennyb has quit IRC | 11:27 | |
| dtantsur | stendulker, persia: a funny (not really) data point: I have access to redfish hardware that does not expose processor architecture :( so our (in-progress) inspection won't work for it. | 11:27 |
| *** lennyb has joined #openstack-ironic | 11:28 | |
| persia | dtantsur: Sadly, I have heard a vender touting that as an advantage. | 11:28 |
| dtantsur | a story as old as the world.. | 11:28 |
| dtantsur | well, we'll start returning "your hardware does not support inspection" messages to users, and they will start pressing the vendors :) maybe it's going to help | 11:29 |
| persia | yeah :( | 11:29 |
| *** lennyb_ has joined #openstack-ironic | 11:29 | |
| *** lennyb has quit IRC | 11:29 | |
| *** lennyb_ has quit IRC | 11:29 | |
| *** lennyb has joined #openstack-ironic | 11:30 | |
| dtantsur | on the other hand.. soon enough we'll no longer require processor architecture, due to nova scheduling changes | 11:31 |
| * dtantsur sees sambetts proposing more database-based locks and goes NOOOOOOOOOOOOOOO | 11:31 | |
| dtantsur | :) | 11:32 |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adding 'get_current_boot_mode' method using redfish https://review.openstack.org/466697 | 11:32 |
| sambetts | dtantsur: /me would love to pull in etcd | 11:32 |
| sambetts | ;) | 11:32 |
| dtantsur | sambetts: Just Do It (tm) | 11:33 |
| dtantsur | my dream is to get rid of Ironic database-based locks at all, but that's for the future | 11:33 |
| dtantsur | someone needs to write an RFE, I appoint milan | 11:33 |
| * milan reads scrollback fearing the worst ;) | 11:34 | |
| *** nmathew has quit IRC | 11:34 | |
| sambetts | dtantsur: not sure if you heard/saw at the summit it was decided (at least informally) that etcd is going to be the single "supported" DLM that OpenStack services can use, to avoid the same issues that we hit trying to support 3 different DB archs | 11:35 |
| openstackgerrit | Merged openstack/ironic-python-agent master: Collect NIC name given by BIOS https://review.openstack.org/396406 | 11:35 |
| milan | lol, 10 years later: someone needs to review the RFE milan wrote 5 years ago :P | 11:35 |
| sambetts | ha! | 11:36 |
| dtantsur | sambetts: this is what I concluded from the ML discussions | 11:36 |
| *** udesale has quit IRC | 11:36 | |
| dtantsur | I guess it make sense, given that we're moving closer to containers | 11:36 |
| dtantsur | milan++ | 11:37 |
| milan | ;) | 11:38 |
| sambetts | being opinionated on this stuff just provides less headaches for developers and upgrades etc so I'm all for not making the same mistakes as trying to support postgres and sql etc | 11:38 |
| dtantsur | heh | 11:38 |
| * dtantsur would prefer we used postgres, sigh.. | 11:38 | |
| *** athomas has quit IRC | 11:38 | |
| dtantsur | but yeah | 11:38 |
| sambetts | I've not had much experience with it so can't judge | 11:39 |
| dtantsur | now let's get rid of oslo.messaging, and we're good :) | 11:39 |
| dtantsur | I think mysql-vs-postgres is a less pain point than e.g. zmq-vs-rabbit | 11:39 |
| sambetts | +1 | 11:39 |
| dtantsur | or etcd-vs-redis | 11:39 |
| dtantsur | (for DLM, I mean) | 11:39 |
| sambetts | heh was a guy at the etcd session suggesting using etcd to replace the RPC messaging system | 11:40 |
| dtantsur | it may work, but it def needs some numbers to back it | 11:40 |
| sambetts | +1 | 11:40 |
| sambetts | k8s uses it for RPC from what I can tell so it might be we can share some knowledge with their community on that stuff | 11:41 |
| dtantsur | nice | 11:41 |
| milan | dtantsur, sambetts guys FYI, could use some early feedback on https://review.openstack.org/#/c/466448/6/ironic_inspector/pxe_filter/dnsmasq.py , just going to write unittests for this | 11:48 |
| patchbot | patch 466448 - ironic-inspector - WIP Introducing a dnsmasq PXE filter driver | 11:48 |
| jkilpatr | so I'm getting failures on one specific node when cleaning "failed to set node power state to on" | 11:48 |
| milan | ah, I actually W-1 it :D | 11:48 |
| *** fellypefca has joined #openstack-ironic | 11:49 | |
| sambetts | jkilpatr: most likly there is a problem with your power interface configuration, username, password or address incorrect, or your server isn't following the instructions that Ironic in ask of it | 11:52 |
| sambetts | jkilpatr: do you have access to the ironic-conductor logs? | 11:52 |
| jkilpatr | sambetts, yes I do, I'll go poke at tehm | 11:52 |
| fellypefca | Morning :) | 11:53 |
| jkilpatr | sambetts, if it just failed all the time that would be fine, but its intermittent probably just a DRAC acting up | 11:54 |
| sambetts | jkilpatr: does the server turn on eventually? | 11:56 |
| jkilpatr | sambetts, not until I try again. | 11:57 |
| sambetts | jkilpatr: hmmm so its either a server ignoring the request or taking longer than the power interface is configured to wait for the server to power on | 11:58 |
| *** litao__ has quit IRC | 11:58 | |
| sambetts | the IPMItool power interfaces have retries built in to deal with the case when an IPMI BMC is being intermittent, I assume your using the DRAC driver though? | 11:59 |
| jkilpatr | nope ipmi tool driver haven't gotten around to trying to drac one. | 12:00 |
| sambetts | ok, perhaps you need to play around with these settings, if its a intermittent BMC https://github.com/openstack/ironic/blob/master/ironic/conf/ipmi.py#L22 | 12:01 |
| sambetts | this will only work if an error performing the operation is actually detectable on the ironic side | 12:02 |
| sambetts | e.g. IPMItool gives us feedback that it was unable to perform the operation | 12:03 |
| dtantsur | sambetts: I wonder if we should also retry when we don't see the reported power state changed | 12:03 |
| dtantsur | I've seen such cases in production | 12:03 |
| sambetts | I took over this patch recently which is making changes to how and when retries are doing in the IPMItool driver https://review.openstack.org/#/c/449454/ | 12:04 |
| *** dprince has joined #openstack-ironic | 12:04 | |
| patchbot | patch 449454 - ironic - Don't retry power status if power action fails | 12:04 |
| sambetts | its a nasty onion of code | 12:04 |
| dtantsur | sambetts++ | 12:05 |
| sambetts | perhaps its something we should globally have a configuration for? set_power_state_retries | 12:05 |
| dtantsur | yeah, that what I thought | 12:05 |
| *** lucas-hungry is now known as lucasagomes | 12:07 | |
| ricardoas | morning, ironic! | 12:08 |
| sambetts | hmm interesting, I think the reason we dont have that is because for example in the IPMItool driver it tries to detect retryable errors such as busy BMC vs non-retryable errors for example invalid username and password, and doing that logic is very driver specific so each driver would need to implement it differently :/ | 12:08 |
| sambetts | dtantsur: we have power_state_sync_max_retries already though | 12:09 |
| sambetts | hmmm | 12:09 |
| openstackgerrit | Merged openstack/ironic master: Enable getting volume targets by their volume_id https://review.openstack.org/460250 | 12:11 |
| dtantsur | sambetts: this is about power state sync loop, not every power operation | 12:11 |
| sambetts | yeah, just thinking it would be the same concept though | 12:11 |
| sambetts | set_power_state_max_retries | 12:12 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 12:12 |
| *** vmud213 has quit IRC | 12:13 | |
| dtantsur | just a rebase ^^^ | 12:13 |
| dtantsur | sambetts++ | 12:13 |
| *** jaypipes has joined #openstack-ironic | 12:18 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: [LLDP] Skip NICs that say they are ready but are unreadable. https://review.openstack.org/434385 | 12:19 |
| openstackgerrit | paresh sao proposed openstack/proliantutils master: Adding 'get_current_boot_mode' method using redfish https://review.openstack.org/466697 | 12:19 |
| dtantsur | derekh: when https://review.openstack.org/366197 merges, we can start working on puppet bits for configuring cinder access, similar to e.g. https://github.com/openstack/puppet-ironic/blob/master/manifests/glance.pp | 12:20 |
| patchbot | patch 366197 - ironic - Add Cinder storage driver | 12:20 |
| dtantsur | wanna do it or should I? | 12:20 |
| derekh | dtantsur: I can | 12:22 |
| *** ccamacho|lunch is now known as ccamacho | 12:24 | |
| *** root____5 has quit IRC | 12:24 | |
| *** root____5 has joined #openstack-ironic | 12:25 | |
| dtantsur | thanks! | 12:25 |
| openstackgerrit | Sam Betts proposed openstack/ironic-python-agent master: Add option for building IPMItool into TinyIPA https://review.openstack.org/465932 | 12:26 |
| *** stendulker has quit IRC | 12:26 | |
| *** chlong has joined #openstack-ironic | 12:29 | |
| dtantsur | derekh: oh and please check if we have a dependency on cinderclient in the RDO package | 12:30 |
| derekh | dtantsur: will do | 12:31 |
| sambetts | dtantsur: I want to pull in the buildroot method for building IPA images, but I can't decide if the infra for building the images should be a separate project or not, the reason I think it should be separate is because the image building process is more or less untied from the version of IPA that is being installed into the image, however we'd need to add something like bin-requirements.txt to | 12:31 |
| sambetts | IPA so that we could automate installing any of the binary requirements that differ between releases | 12:31 |
| dtantsur | sambetts: separate tooling with in-tree dependencies sounds good to me | 12:32 |
| sambetts | only thing that concerns me is that being separate makes things harder from the perspective of running the post merge job to build the image, i.e. we'd need to run it on merges to both repos right? | 12:33 |
| dtantsur | mmm, yes, and potentially some zuul magic to handle Depends-On | 12:33 |
| sambetts | I think Depends-On would be easier to manage, I'm thinking more some weird races between something merging in the IPA repo and something merging in the new repo, pushing the image to tarballs | 12:36 |
| sambetts | hmmm... /me needs more coffee and lunch | 12:37 |
| openstackgerrit | Fellype Cavalcante proposed openstack/ironic master: Introduces onetime boot in oneview drivers https://review.openstack.org/436469 | 12:48 |
| *** rbudden has joined #openstack-ironic | 12:50 | |
| *** mvpnitesh has quit IRC | 12:51 | |
| *** mvpnitesh has joined #openstack-ironic | 12:51 | |
| *** chlong has quit IRC | 12:55 | |
| sambetts | dtantsur: RE: your email regarding DIB, what if we created a ironic-python-agent-builder repo to house all the IPA building tooling? the DIB elements, the coreos/tinyipa methods (although I want to kill them eventually with), the buildroot tooling etc? | 12:55 |
| dtantsur | sambetts: I'm fine with that. Please propose in the thread. | 12:56 |
| *** cdearborn has joined #openstack-ironic | 12:59 | |
| *** shuyingy_ has quit IRC | 13:00 | |
| *** racedo has joined #openstack-ironic | 13:00 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revise deploy process documentation https://review.openstack.org/458867 | 13:01 |
| *** jheroux has joined #openstack-ironic | 13:01 | |
| vdrok | sambetts: yeah, seems like a good idea. although at some point we had coreos-image-build repo or something like that that became abandoned at some point :) | 13:02 |
| vdrok | this one https://github.com/openstack/coreos-image-builder | 13:04 |
| sambetts | wow I didn't know that existed and I would have -1'd the name of that repo sooo fast | 13:04 |
| *** udesale has joined #openstack-ironic | 13:05 | |
| *** aparnav has quit IRC | 13:08 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Revise deploy process documentation https://review.openstack.org/458867 | 13:11 |
| openstackgerrit | Merged openstack/ironic-python-agent master: NUMA-topology collector https://review.openstack.org/424729 | 13:13 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add basic tests for OSC plugin baremetal chassis commands https://review.openstack.org/366158 | 13:14 |
| *** zul has joined #openstack-ironic | 13:14 | |
| *** vikrant has quit IRC | 13:17 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Add security tips https://review.openstack.org/466729 | 13:22 |
| *** trown is now known as trown|brb | 13:23 | |
| dtantsur | vdrok, sambetts: we can ask infra to rename it (they won't like it, but sigh..) | 13:24 |
| sambetts | dtantsur: probably easier to start from scratch, that repo isn't even cookie cuttered by the looks of it | 13:24 |
| sambetts | dtantsur: although its not going to be a python project so I don't know if that a requirement | 13:25 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for OSC baremetal chassis create command https://review.openstack.org/382496 | 13:27 |
| *** gcb has joined #openstack-ironic | 13:28 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Update ipmitool installation and usage documentation https://review.openstack.org/466734 | 13:30 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Add basic tests for OSC plugin baremetal driver commands https://review.openstack.org/367359 | 13:32 |
| dtantsur | yeah.. | 13:32 |
| *** baoli has joined #openstack-ironic | 13:34 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Tests for OSC baremetal node create command with options https://review.openstack.org/382352 | 13:34 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Create port with specific port group UUID in OSC https://review.openstack.org/411840 | 13:35 |
| *** trown|brb is now known as trown | 13:37 | |
| *** shuyingya has joined #openstack-ironic | 13:40 | |
| *** shuyingya has quit IRC | 13:44 | |
| *** mvpnitesh has quit IRC | 13:44 | |
| *** shuyingya has joined #openstack-ironic | 13:45 | |
| *** udesale has quit IRC | 13:46 | |
| *** nicodemos has quit IRC | 13:46 | |
| *** huats has joined #openstack-ironic | 13:47 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic master: Remove unneeded lookup policy check https://review.openstack.org/466735 | 13:51 |
| *** nicodemos has joined #openstack-ironic | 13:54 | |
| *** sok3t has joined #openstack-ironic | 13:55 | |
| *** zackf has joined #openstack-ironic | 13:56 | |
| *** links has quit IRC | 13:57 | |
| *** crushil has joined #openstack-ironic | 13:58 | |
| *** mmedvede has quit IRC | 13:58 | |
| sok3t | Hi, i have a problem while trying to introspect baremetal nodes. I created node (with ipmi creds, port mac), saw the port in baremetal port list but no port is created in neutron. When booting the node, no dhcp responds to the request. Could someone tell me at wich step the neutron port should be created by ironic? Thanks | 13:59 |
| dtantsur | sok3t: introspection does not create ports in neutron, it uses a separate DHCP service | 14:01 |
| *** mmedvede has joined #openstack-ironic | 14:01 | |
| *** rbartal has quit IRC | 14:05 | |
| *** rpioso has joined #openstack-ironic | 14:06 | |
| sok3t | dtantsur: Thanks, so if understand the process, i have to install one dhcp and make it managed by ironic, then the neutron port is created when i boot the instance trough nova | 14:06 |
| *** adisky_ has quit IRC | 14:11 | |
| dtantsur | sok3t: you only need your own DHCP for introspection. boot process is fully managed by Neutron | 14:13 |
| openstackgerrit | Merged openstack/ironic-inspector master: Replace the deprecated tempest.test.attr with decorators.attr https://review.openstack.org/466604 | 14:16 |
| *** e0ne has joined #openstack-ironic | 14:16 | |
| sok3t | dtantsur: thanks, its clearer | 14:23 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: [install-guide] explain the defaults calculation for hardware types https://review.openstack.org/466741 | 14:24 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor API policy and microversion checks https://review.openstack.org/451506 | 14:27 |
| *** cdearborn has quit IRC | 14:28 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Create port with specific port group UUID in OSC https://review.openstack.org/411840 | 14:33 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Tests for OSC baremetal node create command with options https://review.openstack.org/382352 | 14:35 |
| *** MattMan_1 has quit IRC | 14:36 | |
| *** MattMan_1 has joined #openstack-ironic | 14:37 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Refactor API policy and microversion checks https://review.openstack.org/451506 | 14:38 |
| *** shuyingy_ has joined #openstack-ironic | 14:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy master: Declarative approach to parsing JSON fields https://review.openstack.org/464617 | 14:41 |
| dtantsur | lucasagomes, vdrok, milan ^^^ | 14:42 |
| *** jaganathan has quit IRC | 14:42 | |
| milan | dtantsur, ack, will check | 14:42 |
| *** crushil has quit IRC | 14:42 | |
| * milan writing coverage atm | 14:42 | |
| vdrok | dtantsur: thx | 14:43 |
| *** shuyingya has quit IRC | 14:43 | |
| *** mat128 has joined #openstack-ironic | 14:44 | |
| *** sok3t has quit IRC | 14:44 | |
| vdrok | dtantsur: also would be awesome if you could check https://review.openstack.org/451506, it's a long read though :) | 14:45 |
| patchbot | patch 451506 - ironic - Refactor API policy and microversion checks | 14:45 |
| dtantsur | nice, putting on my queue | 14:46 |
| *** mtanino has joined #openstack-ironic | 14:46 | |
| *** parlos has joined #openstack-ironic | 14:48 | |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode https://review.openstack.org/435573 | 14:48 |
| *** mat128 has quit IRC | 14:51 | |
| *** crushil has joined #openstack-ironic | 14:55 | |
| *** d0ugal has quit IRC | 14:57 | |
| *** shuyingy_ has quit IRC | 14:58 | |
| *** fragatina has joined #openstack-ironic | 15:00 | |
| *** cdearborn has joined #openstack-ironic | 15:02 | |
| *** d0ugal has joined #openstack-ironic | 15:06 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Fix compatibility with Pint 0.5 https://review.openstack.org/466764 | 15:06 |
| dtantsur | folks, mind checking ^^ please | 15:07 |
| dtantsur | the last merged patch broke RDO packaging | 15:07 |
| dtantsur | sambetts, vdrok ^^^ | 15:07 |
| sambetts | dtantsur: isn't IPA testing with global-requirements and upper-constraints? | 15:08 |
| sambetts | weird that we didn't catch it in our gate | 15:08 |
| dtantsur | 0.5 is the minimum version, I suspect we test with a newer one | 15:09 |
| dtantsur | yeah https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L44 | 15:09 |
| sambetts | +2 from me | 15:16 |
| *** mjura has quit IRC | 15:21 | |
| lucasagomes | dtantsur, sorry for the delay, I'm adding that tot the list here | 15:24 |
| lucasagomes | I will take a look tomorrow | 15:24 |
| dtantsur | thnx | 15:24 |
| jlvillal | vdrok: I was wondering what your thoughts were on: https://review.openstack.org/#/c/427077/11/ironic/conf/pxe.py | 15:26 |
| patchbot | patch 427077 - ironic - Fix directories permission for tftpboot (MERGED) | 15:26 |
| vdrok | jlvillal: morning | 15:27 |
| jlvillal | vdrok: Privyet :) | 15:27 |
| vdrok | jlvillal: we don't set any default there do we? I guess what we might need is to clarify that a conversion should be done from octal notation to an integer | 15:28 |
| jlvillal | vdrok: Yeah, I was thinking maybe a follow-up on documenting that fact. | 15:28 |
| jlvillal | vdrok: I'll mention it to Madhuri | 15:29 |
| *** ccamacho is now known as ccamacho|brb | 15:29 | |
| vdrok | jlvillal: thanks! | 15:29 |
| jlvillal | vdrok: Spasibo! | 15:29 |
| vdrok | jlvillal: seems like soon the russian speaking people will be majority here :D | 15:29 |
| jlvillal | vdrok: I thought they already were ;) | 15:30 |
| *** marlinc has quit IRC | 15:30 | |
| openstackgerrit | Merged openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 15:31 |
| dtantsur | w00t! | 15:32 |
| jlvillal | woo hoo! :) | 15:34 |
| *** chlong has joined #openstack-ironic | 15:39 | |
| *** shuyingya has joined #openstack-ironic | 15:39 | |
| *** dsneddon has joined #openstack-ironic | 15:42 | |
| *** shuyingya has quit IRC | 15:43 | |
| NobodyCam | Good Morning Ironic'ers | 15:54 |
| *** deray has joined #openstack-ironic | 15:56 | |
| derekh | when trying out BFV I'm seeing this in neutron logs "Refusing to bind due to unsupported vnic_type: baremetal {{(pid=18216) bind_port /opt/stack/neutron/neutron/plugins/ml2/drivers/mech_agent.py:94}} , anybody know if I'm missing something? | 15:56 |
| dtantsur | sambetts ^^^? | 15:58 |
| dtantsur | morning NobodyCam, jlvillal | 15:58 |
| sambetts | derekh: are you running ironic multi-tenant mode? | 15:59 |
| *** pcaruana has quit IRC | 15:59 | |
| NobodyCam | Morning dtantsur :) | 15:59 |
| sambetts | derekh: i.e. having your ironic node configured to use the neutron network interface | 16:00 |
| *** e0ne has quit IRC | 16:00 | |
| derekh | sambetts: think so, I got a meeting about to start and will double check after that | 16:01 |
| sambetts | derekh: that error would indicate that you don't have a matching driver enabled / working on the neutron side that can handle the port binding request from ironic | 16:02 |
| derekh | sambetts: ok, will take a look | 16:02 |
| dtantsur | I guess we should try single-tenant first | 16:02 |
| *** udesale has joined #openstack-ironic | 16:05 | |
| vdrok | morning NobodyCam ! | 16:07 |
| *** chas has quit IRC | 16:09 | |
| *** mikal has quit IRC | 16:14 | |
| *** rcernin has quit IRC | 16:14 | |
| *** mikal has joined #openstack-ironic | 16:16 | |
| NobodyCam | Morning vdrok :) | 16:16 |
| *** kromanenko has quit IRC | 16:20 | |
| *** deray_ has joined #openstack-ironic | 16:22 | |
| *** udesale has quit IRC | 16:22 | |
| *** jaganathan has joined #openstack-ironic | 16:24 | |
| *** deray has quit IRC | 16:25 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Move netowrk interface calls to conductor https://review.openstack.org/453139 | 16:25 |
| *** deray_ has quit IRC | 16:28 | |
| *** parlos has quit IRC | 16:28 | |
| *** rama_y_ has joined #openstack-ironic | 16:28 | |
| *** rcernin has joined #openstack-ironic | 16:31 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: Deprecate DHCP providers https://review.openstack.org/456127 | 16:32 |
| jlvillal | Hi dtantsur :) | 16:32 |
| openstackgerrit | Hugo Nicodemos proposed openstack/python-oneviewclient master: Get the first physical ethernet port in Server Hardware https://review.openstack.org/466277 | 16:34 |
| *** lucasagomes is now known as lucas-afk | 16:35 | |
| *** trown is now known as trown|lunch | 16:37 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:37 | |
| *** kjw3 has joined #openstack-ironic | 16:37 | |
| *** jaganathan has quit IRC | 16:38 | |
| anupn | Morning Ironic'ers :) | 16:39 |
| *** amarao has quit IRC | 16:42 | |
| *** stendulker has joined #openstack-ironic | 16:44 | |
| *** jaosorior is now known as jaosorior_away | 16:45 | |
| *** jaganathan has joined #openstack-ironic | 16:50 | |
| vdrok | good morning anupn ! | 16:53 |
| anupn | vdrok: morning vdrok | 16:53 |
| *** Serlex has quit IRC | 16:53 | |
| *** ralonsoh has quit IRC | 16:57 | |
| *** chlong has quit IRC | 16:59 | |
| *** derekh has quit IRC | 16:59 | |
| *** vmud213 has joined #openstack-ironic | 16:59 | |
| *** dprince has quit IRC | 17:00 | |
| *** xavierr-afk is now known as xavierr | 17:01 | |
| *** fellypefca has quit IRC | 17:04 | |
| *** milan has quit IRC | 17:04 | |
| *** deray has joined #openstack-ironic | 17:05 | |
| *** stendulker_ has joined #openstack-ironic | 17:05 | |
| *** stendulker has quit IRC | 17:07 | |
| *** stendulker_ has quit IRC | 17:07 | |
| dtantsur | JayF: around? the spec for https://review.openstack.org/#/c/364834/ was approved: http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipa_api_versioning.html | 17:07 |
| patchbot | patch 364834 - ironic-python-agent - Include IPA Version during heartbeat | 17:07 |
| *** stendulker has joined #openstack-ironic | 17:07 | |
| *** stendulker has quit IRC | 17:09 | |
| *** stendulker has joined #openstack-ironic | 17:09 | |
| JayF | dtantsur: uh problem | 17:10 |
| *** ankkumar has joined #openstack-ironic | 17:10 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Fix compatibility with Pint 0.5 https://review.openstack.org/466764 | 17:10 |
| *** aparnav has joined #openstack-ironic | 17:10 | |
| ankkumar | #Openstack-meeting-3 | 17:10 |
| dtantsur | JayF: please remove your -2 :) | 17:10 |
| JayF | dtantsur: without core access I can't | 17:10 |
| dtantsur | oooooo | 17:10 |
| dtantsur | lemme check if I can kick you from review | 17:11 |
| JayF | dtantsur: I did it | 17:11 |
| JayF | dtantsur: deleting myself from the review dropped the -2 | 17:11 |
| dtantsur | aha, nice. thanks! | 17:11 |
| JayF | np, gl! | 17:11 |
| *** stendulker has quit IRC | 17:12 | |
| *** aparna has joined #openstack-ironic | 17:12 | |
| *** stendulker has joined #openstack-ironic | 17:13 | |
| *** dprince has joined #openstack-ironic | 17:14 | |
| *** chlong has joined #openstack-ironic | 17:15 | |
| *** chlong has quit IRC | 17:15 | |
| vdrok | although there might be one caveat in that patch, iirc we were discussing it with vsaienk0 some time ago, as after attaching an interface, we don't regenerate the configdrive obviously | 17:16 |
| vdrok | dtantsur: ^ | 17:16 |
| dtantsur | vdrok: oh. please comment on the patch then.. | 17:16 |
| *** aparnav has quit IRC | 17:16 | |
| *** aparna has quit IRC | 17:17 | |
| *** jmccrory_away is now known as jmccrory | 17:18 | |
| *** ChubYann has joined #openstack-ironic | 17:18 | |
| *** stendulker has quit IRC | 17:23 | |
| *** stendulker has joined #openstack-ironic | 17:24 | |
| *** jaganathan has quit IRC | 17:24 | |
| *** vmud213 has quit IRC | 17:25 | |
| *** mat128 has joined #openstack-ironic | 17:29 | |
| *** fragatina has quit IRC | 17:37 | |
| *** e0ne has joined #openstack-ironic | 17:37 | |
| *** Sukhdev has joined #openstack-ironic | 17:43 | |
| mjturek | TheJulia: mind if I rebase the wire-in patch now that the cinder driver has landed? | 17:46 |
| *** kjw3 has quit IRC | 17:49 | |
| *** Nisha_Agarwal has quit IRC | 17:50 | |
| *** racedo has quit IRC | 17:54 | |
| *** racedo has joined #openstack-ironic | 17:54 | |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic-specs master: API Evolution - ETAG identifiers https://review.openstack.org/381991 | 17:54 |
| *** racedo has quit IRC | 17:54 | |
| *** racedo has joined #openstack-ironic | 17:55 | |
| *** chlong has joined #openstack-ironic | 17:55 | |
| *** racedo has quit IRC | 17:55 | |
| *** racedo has joined #openstack-ironic | 17:56 | |
| *** racedo has quit IRC | 17:56 | |
| *** racedo has joined #openstack-ironic | 17:56 | |
| *** racedo has quit IRC | 17:57 | |
| *** racedo has joined #openstack-ironic | 17:57 | |
| *** racedo has quit IRC | 17:58 | |
| sambetts | dtantsur: /me had to run, thanks for the meeting, cya later | 17:59 |
| sambetts | night all o/ | 17:59 |
| *** sambetts is now known as sambetts|afk | 17:59 | |
| vdrok | good night sambetts|afk | 18:00 |
| dtantsur | good night, sambetts|afk | 18:00 |
| vdrok | dtantsur: I think there was a similar bug reported by pas-ha | 18:00 |
| vdrok | I'm +1 on that | 18:00 |
| vdrok | (or maybe not reported, but I heard complaints from him :)) | 18:01 |
| *** Sukhdev has quit IRC | 18:01 | |
| dtantsur | ok, good to know. I guess it can be rfe-approved now | 18:01 |
| *** kjw3 has joined #openstack-ironic | 18:02 | |
| dtantsur | this is a low hanging fruit, so anyone can try it :) (otherwise I'll do it eventually) | 18:02 |
| *** e0ne has quit IRC | 18:06 | |
| *** ankkumar has quit IRC | 18:06 | |
| deray | dtantsur, vdrok sambetts|afk rpioso stendulker I have this patch on sushy: https://review.openstack.org/#/c/463829/ | 18:06 |
| patchbot | patch 463829 - sushy - Add backbone for parsing redfish OEM extensions | 18:06 |
| dtantsur | deray: on my way to it. I had some concerns about the overall design, so I need to start with a fresh head (i.e. tomorrow) | 18:07 |
| deray | dtantsur, np.. tyt :) | 18:08 |
| deray | dtantsur, would love to have inputs | 18:08 |
| xavierr | stendulker: hey. could you clarify why ilo's driver deploy interface was removed/moved to boot? :) | 18:08 |
| *** r-daneel has joined #openstack-ironic | 18:09 | |
| *** deray has quit IRC | 18:09 | |
| *** chas has joined #openstack-ironic | 18:09 | |
| vdrok | good night everyone! | 18:09 |
| xavierr | night vdrok | 18:09 |
| *** ijw has quit IRC | 18:09 | |
| *** crushil has quit IRC | 18:10 | |
| dtantsur | xavierr: it was because it contained boot bits, not deploy buts. | 18:10 |
| dtantsur | g'night vdrok | 18:10 |
| dtantsur | I guess I should go as well, actually | 18:10 |
| xavierr | hey dtantsur, I'm testing what you requested on the patch. place the power off before change the set_boot_device | 18:11 |
| xavierr | dtantsur: I'll keep you aware and send a patch set asap :) | 18:12 |
| dtantsur | thanks a lot! | 18:12 |
| dtantsur | I know, the difference between interface is sometimes unclear, this is why I asked vdrok to write spec/docs on it today | 18:12 |
| xavierr | dtantsur: should we remove the deploy also? I don't see a better place to place the allocate/deallocate operations on oneview :( | 18:12 |
| xavierr | in the future, I mean | 18:13 |
| dtantsur | allocate, hmmm | 18:13 |
| *** rama_y_ has quit IRC | 18:13 | |
| * xavierr looking for the links to send to dtantsur | 18:13 | |
| dtantsur | xavierr: boot interface prepare_instance can be a good candidate for it | 18:13 |
| anupn | night vdrok | 18:14 |
| dtantsur | xavierr: I mean https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L454 and https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L465 | 18:14 |
| dtantsur | but I don't know oneview well enough to say for sure | 18:14 |
| *** chas has quit IRC | 18:14 | |
| xavierr | what about prepare cleaning and tear down cleaning? | 18:15 |
| xavierr | for those operations we need some hooks, we use deploy currently | 18:16 |
| dtantsur | xavierr: there are such methods in the boot interface, if that's what you ask | 18:16 |
| xavierr | dtantsur: ok. but is there a why, for the migration? spec or something? :) | 18:18 |
| *** e0ne has joined #openstack-ironic | 18:18 | |
| dtantsur | well, this is what we lack. there is some more or less shared understanding in the core team. | 18:18 |
| dtantsur | the problem is that deploy was never meant to be vendor-specific | 18:18 |
| xavierr | ah | 18:19 |
| dtantsur | and the drivers/hardware types are expected to be deploy-agnostic. when creating the ilo hardware type we found it to not be the case | 18:19 |
| xavierr | didn't know that | 18:19 |
| dtantsur | and we agreed that having a vendor-specific boot interface is more correct than a vendor-specific deploy | 18:19 |
| dtantsur | xavierr: one of the thing we're not good at documenting, sorry :( | 18:20 |
| xavierr | dtantsur: yes :/ I'll explaing that to my team | 18:20 |
| xavierr | ty dtantsur o/ | 18:20 |
| *** trown|lunch is now known as trown | 18:21 | |
| * xavierr thinks we are about the time to move to driver comp | 18:21 | |
| dtantsur | ++ | 18:22 |
| dtantsur | thanks! | 18:22 |
| *** chas has joined #openstack-ironic | 18:31 | |
| dtantsur | see you tomorrow, everyone | 18:33 |
| *** dtantsur is now known as dtantsur|afk | 18:33 | |
| *** fellypefca has joined #openstack-ironic | 18:33 | |
| *** chas has quit IRC | 18:35 | |
| openstackgerrit | Xavier proposed openstack/ironic master: you shall not pass https://review.openstack.org/466872 | 18:36 |
| *** harlowja has joined #openstack-ironic | 18:37 | |
| *** marlinc has joined #openstack-ironic | 18:39 | |
| *** harlowja has quit IRC | 18:43 | |
| *** crushil has joined #openstack-ironic | 18:43 | |
| *** stendulker has quit IRC | 18:46 | |
| *** dprince has quit IRC | 18:47 | |
| *** e0ne has quit IRC | 18:55 | |
| *** mat128 has quit IRC | 18:56 | |
| *** dprince has joined #openstack-ironic | 18:57 | |
| *** mat128 has joined #openstack-ironic | 18:59 | |
| *** chas has joined #openstack-ironic | 19:12 | |
| *** chas has quit IRC | 19:17 | |
| *** mat128 has quit IRC | 19:20 | |
| *** chas has joined #openstack-ironic | 19:33 | |
| *** chas has quit IRC | 19:38 | |
| *** fellypefca has quit IRC | 19:39 | |
| *** chas has joined #openstack-ironic | 19:54 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 19:58 |
| *** chas has quit IRC | 19:59 | |
| *** chlong has quit IRC | 20:07 | |
| *** milan has joined #openstack-ironic | 20:13 | |
| *** chas has joined #openstack-ironic | 20:15 | |
| openstackgerrit | Merged openstack/python-oneviewclient master: Get the first physical ethernet port in Server Hardware https://review.openstack.org/466277 | 20:17 |
| *** chas has quit IRC | 20:19 | |
| *** ijw has joined #openstack-ironic | 20:21 | |
| *** milan has quit IRC | 20:22 | |
| *** milan has joined #openstack-ironic | 20:23 | |
| *** cdearborn has quit IRC | 20:25 | |
| *** milan has quit IRC | 20:26 | |
| *** chlong has joined #openstack-ironic | 20:31 | |
| *** dprince has quit IRC | 20:32 | |
| *** cdearborn has joined #openstack-ironic | 20:35 | |
| *** chas has joined #openstack-ironic | 20:36 | |
| openstackgerrit | Michael Turek proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 20:37 |
| *** chlong has quit IRC | 20:41 | |
| *** chas has quit IRC | 20:41 | |
| *** ijw_ has joined #openstack-ironic | 20:51 | |
| *** ijw has quit IRC | 20:55 | |
| *** chas has joined #openstack-ironic | 20:57 | |
| *** chas has quit IRC | 21:02 | |
| *** chas has joined #openstack-ironic | 21:18 | |
| *** cdearborn has quit IRC | 21:22 | |
| *** chas has quit IRC | 21:22 | |
| *** ijw_ has quit IRC | 21:23 | |
| *** jheroux has quit IRC | 21:30 | |
| *** fragatina has joined #openstack-ironic | 21:35 | |
| *** chas has joined #openstack-ironic | 21:39 | |
| *** chas has quit IRC | 21:43 | |
| *** ijw has joined #openstack-ironic | 21:46 | |
| *** fragatina has quit IRC | 21:47 | |
| *** fragatina has joined #openstack-ironic | 21:48 | |
| *** fragatina has quit IRC | 21:58 | |
| *** chas has joined #openstack-ironic | 21:59 | |
| *** xavierr has quit IRC | 22:00 | |
| *** mtanino has quit IRC | 22:03 | |
| *** chas has quit IRC | 22:04 | |
| *** jkilpatr has quit IRC | 22:08 | |
| *** ijw has quit IRC | 22:10 | |
| *** baoli has quit IRC | 22:13 | |
| *** ijw has joined #openstack-ironic | 22:20 | |
| *** chas has joined #openstack-ironic | 22:20 | |
| *** ijw has quit IRC | 22:23 | |
| *** chas has quit IRC | 22:25 | |
| *** crushil has quit IRC | 22:27 | |
| *** jkilpatr has joined #openstack-ironic | 22:27 | |
| *** ijw has joined #openstack-ironic | 22:28 | |
| *** ijw has quit IRC | 22:34 | |
| *** ijw has joined #openstack-ironic | 22:35 | |
| *** chas has joined #openstack-ironic | 22:41 | |
| *** ijw has quit IRC | 22:42 | |
| *** chas has quit IRC | 22:46 | |
| *** ijw has joined #openstack-ironic | 22:46 | |
| *** r-daneel has quit IRC | 22:47 | |
| NobodyCam | off the wall question ... anyone have a good example of providing credentials when using curl and the inspector api? | 22:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-oneviewclient master: Updated from global requirements https://review.openstack.org/389942 | 22:50 |
| *** ijw has quit IRC | 22:51 | |
| *** ijw has joined #openstack-ironic | 23:02 | |
| *** chas has joined #openstack-ironic | 23:02 | |
| *** chas has quit IRC | 23:06 | |
| *** rpioso has quit IRC | 23:13 | |
| *** ijw has quit IRC | 23:14 | |
| *** chas has joined #openstack-ironic | 23:23 | |
| *** chas has quit IRC | 23:27 | |
| *** pmannidi has quit IRC | 23:31 | |
| *** ijw has joined #openstack-ironic | 23:33 | |
| *** ijw has quit IRC | 23:34 | |
| *** ijw has joined #openstack-ironic | 23:34 | |
| *** causten has joined #openstack-ironic | 23:38 | |
| *** chas has joined #openstack-ironic | 23:44 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 23:47 | |
| *** chas has quit IRC | 23:48 | |
| *** Nisha_Agarwal has quit IRC | 23:51 | |
| *** pmannidi has joined #openstack-ironic | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!