Monday, 2018-07-16

« Sunday, 2018-07-15 Index Tuesday, 2018-07-17 »
*** cjloader has joined #openstack-ironic00:12
*** trungnv has joined #openstack-ironic00:52
*** cjloader has quit IRC00:55
*** cjloader has joined #openstack-ironic00:55
*** cjloader has quit IRC00:55
*** cjloader has joined #openstack-ironic00:56
*** cjloader has quit IRC01:04
*** tiendc has joined #openstack-ironic01:12
*** zshi has quit IRC01:26
*** zshi has joined #openstack-ironic01:40
openstackgerrittianhui proposed openstack/ironic master: Fix bug to doc:configdrive  https://review.openstack.org/58251402:14
*** phuongnh has joined #openstack-ironic02:24
*** phuongnh has quit IRC02:28
*** phuongnh has joined #openstack-ironic02:29
*** jiapei has joined #openstack-ironic03:23
*** TxGirlGeek has joined #openstack-ironic03:49
*** phuongnh has quit IRC03:49
*** phuongnh has joined #openstack-ironic03:50
*** TxGirlGeek has quit IRC03:56
*** e0ne has joined #openstack-ironic04:59
*** e0ne has quit IRC04:59
*** mjura has joined #openstack-ironic05:14
*** moshele has joined #openstack-ironic05:17
*** moshele has quit IRC05:17
*** jaganathan has joined #openstack-ironic05:24
*** links has joined #openstack-ironic05:50
*** mmethot has quit IRC06:01
*** rbartal has joined #openstack-ironic06:07
*** skazi has joined #openstack-ironic06:21
openstackgerrityatin proposed openstack/ironic master: Add mock object for get_bios_settings  https://review.openstack.org/58283506:27
*** pcaruana has joined #openstack-ironic06:36
*** arnewiebalck__ has joined #openstack-ironic06:42
*** arnewiebalck__ has quit IRC06:42
*** jtomasek has joined #openstack-ironic06:57
openstackgerritMerged openstack/python-ironic-inspector-client stable/queens: Gate fix: Cap hacking to avoid gate failure  https://review.openstack.org/56756707:03
openstackgerritMerged openstack/ironic-ui master: fix tox python3 overrides  https://review.openstack.org/57380707:05
*** serlex has joined #openstack-ironic07:12
*** jcoufal has joined #openstack-ironic07:13
openstackgerritMerged openstack/sushy-tools master: Cleanup usage of 'simulator' and 'cloud-backed' in docs  https://review.openstack.org/58166007:18
openstackgerritMerged openstack/sushy-tools master: Fix ImportError for tox debug environment  https://review.openstack.org/58034807:18
*** tesseract has joined #openstack-ironic07:19
*** moshele has joined #openstack-ironic07:23
*** jiapei has quit IRC07:43
*** zhangfei has joined #openstack-ironic07:47
openstackgerritTuan Do Anh proposed openstack/networking-baremetal master: Add release notes link in README  https://review.openstack.org/58124607:49
*** rpittau has joined #openstack-ironic07:51
openstackgerritMerged openstack/bifrost master: Make sure we have our folders for uwsgi  https://review.openstack.org/58265307:52
*** AlexeyAbashkin has joined #openstack-ironic08:03
*** mgoddard has joined #openstack-ironic08:07
*** lucas-afk is now known as lucasagomes08:12
*** dougsz has joined #openstack-ironic08:14
*** ajya has joined #openstack-ironic08:14
*** tssurya has joined #openstack-ironic08:16
*** athomas has joined #openstack-ironic08:16
*** moshele has quit IRC08:26
*** skazi has quit IRC08:26
openstackgerritMerged openstack/networking-generic-switch master: fix tox python3 overrides  https://review.openstack.org/57391208:27
*** skazi has joined #openstack-ironic08:32
openstackgerritMerged openstack/ironic master: Add mock object for get_bios_settings  https://review.openstack.org/58283508:36
*** skazi has quit IRC08:39
*** skazi has joined #openstack-ironic08:41
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add conductor_group field to nodes and conductors tables  https://review.openstack.org/57962608:42
*** skazi has quit IRC08:51
*** hjensas has joined #openstack-ironic08:55
*** hjensas has quit IRC08:55
*** hjensas has joined #openstack-ironic08:55
*** e0ne has joined #openstack-ironic08:55
*** hjensas has quit IRC09:00
*** skazi has joined #openstack-ironic09:04
*** e0ne has quit IRC09:05
*** racedo has joined #openstack-ironic09:09
*** e0ne has joined #openstack-ironic09:13
*** skazi has quit IRC09:21
*** skazi has joined #openstack-ironic09:22
*** zhangfei has quit IRC09:24
*** sambetts_ is now known as sambetts09:25
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: [WIP] Wrapping Flash with oslo.service  https://review.openstack.org/56182309:31
*** pbourke has quit IRC09:31
openstackgerritAnkit Kumar proposed openstack/ironic-tempest-plugin master: Test cases for iLO drivers in standalone ironic  https://review.openstack.org/53565209:32
*** dtantsur|afk is now known as dtantsur09:32
dtantsurmorning ironic, happy monday!09:33
*** pbourke has joined #openstack-ironic09:33
etingofgood morning Ironicers and dtantsur in particular! ;)09:34
*** zzzeek has quit IRC09:38
*** zzzeek has joined #openstack-ironic09:38
*** mjura has quit IRC10:02
*** e0ne has quit IRC10:28
*** ajya has quit IRC10:28
*** ajya has joined #openstack-ironic10:29
*** mjura has joined #openstack-ironic10:29
openstackgerritDebayan Ray proposed openstack/sushy master: Add storage disk drive  https://review.openstack.org/56153310:30
*** e0ne has joined #openstack-ironic10:34
*** dtantsur is now known as dtantsur|brb10:35
openstackgerritMerged openstack/ironic master: Deploy steps - conductor & drivers  https://review.openstack.org/57864910:38
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add a netmiko device driver for Juniper switches  https://review.openstack.org/53053710:44
openstackgerritIlya Etingof proposed openstack/ironic master: Migrate ironic `snmp` driver to the latest pysnmp API  https://review.openstack.org/57898710:46
openstackgerritMerged openstack/ironic master: Deploy steps - API & notifications  https://review.openstack.org/57996810:49
openstackgerritMerged openstack/ironic master: Update CI jobs for rescue mode  https://review.openstack.org/52870410:51
openstackgerritAleksandra Bezborodova proposed openstack/sushy master: Add a virtual media resource  https://review.openstack.org/57081010:52
*** e0ne has quit IRC11:02
*** e0ne has joined #openstack-ironic11:15
jrollmorning ironic :)11:19
jrolldtantsur|brb: welcome back11:19
*** verdurin has quit IRC11:19
*** phuongnh has quit IRC11:22
*** racedo has quit IRC11:23
*** verdurin has joined #openstack-ironic11:25
*** rh-jelabarre has joined #openstack-ironic11:28
*** tiendc has quit IRC11:28
etingofjroll, o/11:32
*** racedo has joined #openstack-ironic11:38
TheJuliaGood morning ironic11:41
* TheJulia begins to try and wipe sleep from the eyes11:41
etingofgood morning, TheJulia o/11:42
*** dougsz has quit IRC11:44
*** rcernin has quit IRC11:45
TheJuliamgoddard: When you get a chance, can you revise your vote on https://review.openstack.org/#/c/399234/ ?11:54
patchbotpatch 399234 - ironic - Caching of PDU autodiscovery11:54
TheJuliaIt appears to have legitimate failures now11:54
*** dtantsur|brb is now known as dtantsur11:57
dtantsurmorning jroll, TheJulia11:58
dtantsurTheJulia: tbh, I did not quite get what exactly you're suggesting in your comment on https://storyboard.openstack.org/#!/story/200286811:58
dtantsurshould we just change the default behavior when changing a driver? changing it with a microversion?11:58
dtantsurshould we add it to tomorrow's midcycle agenda?11:59
TheJuliaThat was kind of what I was thinking, just allow the user change to override pre-existing set interfaces if they are incompatible11:59
TheJuliasure11:59
dtantsurTheJulia: see the etherpad. which of the options would you prefer?12:03
dtantsurthe same question for jroll and sambetts12:03
dtantsur(https://etherpad.openstack.org/p/ironic-rocky-midcycle line 39)12:04
jrolldtantsur: we have a low-level API, so I like explicitness. I'd go with #112:06
*** MattMan has quit IRC12:06
*** MattMan has joined #openstack-ironic12:06
sambettsI'm leaning towards #1 until we can redesign the API around interfaces properly12:07
TheJuliaI'd suggest we table redesigning until after we have a plan to meet the minimum desired goals and version bump12:08
sambetts100%12:09
* etingof is working on fixing https://review.openstack.org/#/c/399234/ 12:09
patchbotpatch 399234 - ironic - Caching of PDU autodiscovery12:09
TheJuliafor what it is worth, I can agree to #112:10
TheJuliaetingof: awesome12:13
TheJuliaIf anyone is curious about redfish bios interface support, https://review.openstack.org/#/c/580609/ might be a good review12:13
patchbotpatch 580609 - sushy-tools - Add BIOS resource support12:13
dtantsurTheJulia: okay, do we actually need to discuss it then? or are the three of you enough to approve it? or?12:15
dtantsurs/then/on the midcycle/12:16
*** dprince has joined #openstack-ironic12:16
TheJuliadtantsur: I think if we have consensus on #1, then there really is no need to discuss it12:16
TheJuliaHmmm.. must be too early for mjturek12:16
*** dougsz has joined #openstack-ironic12:17
dtantsurokay, so if anybody puts rfe-approved on it, I'll withdraw the topic ;)12:18
dtantsur(assuming that I get it right and #1 is what I have in the RFE already)12:18
jrollI'd wait for rloo, she also had opinions on this12:18
TheJuliashe should be on fairly soon12:19
dtantsurack12:19
dtantsurI only want to fit it in Rocky, meaning, this week (for CLI to be updated)12:20
*** bfournie has quit IRC12:23
*** rloo has joined #openstack-ironic12:24
TheJuliaYeah, we're getting down to the wire of sorts12:24
TheJuliaAnd speak of the rloo!12:24
* TheJulia goes and finds coffee12:25
*** rloo has quit IRC12:26
*** rloo has joined #openstack-ironic12:27
rlooGood morning everyone, TheJulia, sambetts, dtantsur, etingof :)12:28
dtantsurmorning rloo! we're talking about https://storyboard.openstack.org/#!/story/2002868 right now. ppl seem to be in favour of the current proposal, wdyt?12:28
*** dprince has quit IRC12:29
rloodtantsur: give me a few min to get my bearings :)12:30
dtantsursure :)12:30
rloodtantsur: I'm good with the RFE as long as 1. we expicitly mention there, that the --reset-deploy-interface (in REST API, python API & openstack CLI) is only applicable with updating the driver. 2. TheJulia add a comment (Or someone) wrt TheJulia's comment, so we know that it was considered :)12:37
TheJuliaA link to eavesdrop would be better12:38
rlooTheJulia: that would work I guess wrt discussion but the rfe/description itself out to be updated to be more explicit.12:38
*** jistr is now known as jistr|mtg12:38
TheJuliaought to be?12:39
rlooTheJulia: I don't want someone coming back and say I used ''--reset... " when setting the node name, and the interfaces didn't get reset.12:39
* TheJulia gives the bot a minute12:39
rlooTheJulia: yeah, ought to be :)12:39
*** jistr|mtg is now known as jistr12:39
* TheJulia wonders if we need to chat more to get the buffers to flush to the log server12:40
jrollI think it's every 5 minutes12:40
TheJuliathat would make too much sense12:40
*** dougsz has quit IRC12:43
*** dougsz1 has joined #openstack-ironic12:43
dtantsurrloo: updated12:45
dtantsuralso looking for moar reviews on https://review.openstack.org/58037712:45
patchbotpatch 580377 - python-ironicclient - Support resetting interfaces to their default values12:45
TheJuliaThis will be the week of moar reviews I suspect12:48
dtantsuroh yeah12:48
rloodtantsur: thx. Commented. I think we just need one other person and we can approve.12:48
* dtantsur gets to hacking on the API change12:48
rlooas long as no one disagrees :)12:48
dtantsurrloo: I think we have jroll and sambetts on board with it12:48
jroll++12:49
jrollmy signature is near the top of the comments12:49
rloodtantsur: ok, i will approve it :)12:49
TheJuliaalready did12:49
TheJuliawel, the rfe that is12:49
*** trown|outtypewww is now known as trown12:49
rlooThx TheJulia. I commented just to be explicit about it :)12:50
rloobtw, welcome back dtantsur!12:50
dtantsurthanks :)12:50
*** mjura has quit IRC12:54
*** bfournie has joined #openstack-ironic12:55
openstackgerritMerged openstack/ironic master: Add conductor_group field to nodes and conductors tables  https://review.openstack.org/57962612:55
TheJulia\o/12:56
*** mjturek has joined #openstack-ironic12:56
TheJuliaGood morning mjturek!12:57
mjturekgm TheJulia!12:57
TheJuliaOn your ironic-lib change, I spotted the docstring issue causing the test to fail :)  Comment on the patch12:57
mjturekchecking!12:58
mjturekd'oh.12:58
TheJulia:)12:58
mjturekTheJulia: I'm cleaning up the series now (didn't do storyboard numbers at all and fixing some tests on a final patch). I'll fix it asap12:59
mjturekthanks for the review!12:59
TheJuliamjturek: awesome12:59
*** mjura has joined #openstack-ironic13:00
*** mjura has joined #openstack-ironic13:01
*** EmilienM has left #openstack-ironic13:02
rlooTheJulia, dtantsur: did you want to discuss (or have you already discussed): https://review.openstack.org/#/c/579818/13:04
patchbotpatch 579818 - ironic-lib (stable/queens) - Expose GPT partitioning fixing method13:04
rloofwiw, I agree with dtantsur13:05
rlooalthough maybe there is something I am missing13:05
TheJuliaI'm about to drive my car over for an oil change and alignment13:05
TheJuliagive me ~1513:05
rlooTheJulia: ok, later then!13:05
*** markguz has joined #openstack-ironic13:11
* dtantsur makes sad panda face at wsme13:12
*** milan has joined #openstack-ironic13:15
TheJuliadtantsur: sad panda why?13:22
dtantsurTheJulia: having both body and query parameters is weird13:23
dtantsuryou'll see in the patch13:23
jrollor more generally, because wsme13:23
sambettscan we swtich to flask yet ;)13:23
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Add reset_interfaces parameter to node's PATCH  https://review.openstack.org/58295113:24
dtantsurTheJulia: check this out ^^^13:24
dtantsurspecifically, the change to controllers/node.py13:24
dtantsursambetts++++13:24
TheJuliarloo: so I think the key thing to think about is that it originally started as an operator bug fix against ?ocata? because their whole disk images were not being left in a happy state with the secondary gpt partition, so a case was functionally broken by us not fixing the second copy on whole disk images. The consensus seemed to be that it would be useful to expose that, and the original plan was to invoke it from13:24
TheJuliaIPA, but seems like we didn't need to if we put it in the right place13:24
TheJuliaflask++13:24
sambettsthing is in wsme you can actually send all body paramters via query if you want...13:25
sambettswhich ends up with nasty function defs like that13:25
*** e0ne has quit IRC13:25
mgoddardhi rloo13:25
rlooTheJulia: so all we need to do is backport that and release? Seems like something in IPA would also need to be updated?13:26
TheJuliarloo: so from a consume standpoint, I think it is important that we go ahead and backport for those building their packages and deploys from stable branches13:26
rloomgoddard: hi!13:26
mgoddardrloo: congrats on deploy steps merging :)13:26
rloomgoddard: oh, did it all merge? I am still catching up ...13:26
TheJuliarloo: yeah, should just be backport and release, I think olivierb- confirmed he was fixed with just the ironic-lib patch13:26
TheJuliarloo: they did13:26
rlooTheJulia: oh, in that case fine. what about reno?13:27
mgoddardrloo: haven't had as much time to spend on it as I'd hoped - lots of client visits13:27
rlooTheJulia: well, we don't reno ironic-lib. i don't think...13:27
TheJuliathat13:27
dtantsurI approved the deploy steps this morning. was a nice after-holiday reading :D13:27
mgoddardrloo: I do have some time today, I could clean up some nits if that helps?13:27
openstackgerritIlya Etingof proposed openstack/ironic master: Caching of PDU autodiscovery  https://review.openstack.org/39923413:27
rloomgoddard: there is the client patch that needs to be reviewed unless that was merged too. and on m list: staging drivers (I think maybe there is one deploy driver thre) and docs13:28
rloomgoddard: take your pick :)13:28
mgoddardrloo: I'm also still to fully digest the conductor piece, I'll do that also in case anything comes up13:28
dtantsurTheJulia: if we don't need to bump ironic-lib requirements, I'm fine with the backport13:28
TheJuliaolivierb-: you around?13:28
dtantsurrloo: you remind me, staging drivers needs unit test fixed and classic drivers removed :(13:28
rloomgoddard: ++ on understanding conductor etc part!13:29
*** alex_xu has quit IRC13:29
mgoddardrloo: right you are. I'll look at the client now13:29
rloodtantsur: yeah, i think you had a WIP or something wrt staging drivers. or a -1 from CI?13:29
TheJuliadtantsur: I feel like we need to do something with staging drivers...13:29
dtantsurrloo: an early WIP, yes13:29
rloodtantsur: would be good to get that done (staging drivers) although we can do that after feature freeze.13:30
dtantsurI'll try to get it done today13:30
dtantsurthough13:30
dtantsuryeah, I should finish that API patch first, because of FF13:30
rloodtantsur: ++ features first :)13:30
*** arnewiebalck__ has joined #openstack-ironic13:30
jrollaw, now I need to rush to beat dtantsur so I don't need to rebase again13:31
dtantsurlol, yes13:31
rloojroll: do you have stuff ready for review? (yup, still catching up...)13:31
jrollrloo: I'm having trouble wrapping my head around how having two hash rings helps upgrades13:31
*** arnewiebalck__ has quit IRC13:31
* dtantsur hears "two hash rings" and backs off slowly13:31
jrollthough maybe I'm doing the lookup wrong13:31
*** e0ne has joined #openstack-ironic13:32
* dtantsur only accepts The One Ring13:32
jrollor was it to just have it there to switch over in stein?13:32
*** r-daneel has joined #openstack-ironic13:32
*** alex_xu has joined #openstack-ironic13:32
*** arnewiebalck__ has joined #openstack-ironic13:32
jrolldtantsur: heh, friday eavesdrop if you want to see the discussion around it13:32
rloojroll: to be clear, it is one hashring, with two keys per conductor/driver, right?13:32
dtantsurjroll: do I? :)13:32
rloowell, i guess it is two rings13:32
jrolldtantsur: probably not :)13:32
dtantsurgood!13:32
*** r-daneel has quit IRC13:32
jrollrloo: yeah, I have it as two rings13:33
*** mbeierl has joined #openstack-ironic13:33
* jroll re-re-reads friday discussion13:34
TheJuliajroll: my impression was for a switch in Stein13:34
jrollTheJulia: and so I don't need to fall back to the old one13:34
* rloo thinks it is easier to merge dtantsur's stuff than to wrap our heads around jroll's feature :D13:34
jrollthat's what was getting me13:34
dtantsurhehe13:35
TheJuliajroll: not in stein, but I think we would need it for after we make the change in stein to cross over.... or not... wow this is becoming a blur of thought13:35
openstackgerritJulia Kreger proposed openstack/ironic-lib master: Fall back to PARTUUID if UUID returns nothing.  https://review.openstack.org/56915813:36
TheJuliadtantsur: ^^^ debug log entry added13:36
dtantsurTheJulia: what about reno?13:40
dtantsurah, we don't have them for ironic-lib, right13:40
* dtantsur should not have -1'ed it then13:41
TheJuliaI do not believe so13:41
TheJulianow, that doesn't mean we might not want to drop a note in ironic if olivierb- wants to create one :)13:41
jaypipesjroll, TheJulia, dtantsur: morning folks. what was your plan with https://review.openstack.org/#/c/565841? did you want me to rebase it and review?13:41
patchbotpatch 565841 - nova - Ironic: report 0 for vcpus/memory_mb/disk_gb resou...13:41
TheJuliaoh, different fetaure13:41
TheJuliadoh!13:41
dtantsurjaypipes: I'd prefer to get that in. I'm a bit tired of debugging weird 'no valid host' problems that end up being folks using the old way of scheduling..13:42
TheJuliajaypipes: I think jroll was going to try and update it this week, at lest that is the last thing iI saw on it13:42
jrollwe agreed in the nova channel to wait for stein13:42
* TheJulia must have missed that13:42
* dtantsur would not agree to that, but okay13:43
jrollif someone wants to update the patch and convince nova folks to land it this cycle, that's fine13:43
jrollTheJulia: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-07-11.log.html#t2018-07-11T21:27:3513:43
jroll:P13:43
TheJuliaoh! right!13:43
* jroll is fine either way13:43
* TheJulia is only on cup one of two for coffee13:43
dtantsuryeah, I just hope that everyone realizes that we removed all workarounds that helped the old way of scheduling work reliably13:44
TheJuliaYeah, my worry was major change at the last minute13:44
TheJuliabecause we know how well that goes sometimes :(13:44
mgoddardwould be sad to see that one miss another release13:45
TheJuliait would be helpful to land https://review.openstack.org/#/c/563722/ in that case13:45
patchbotpatch 563722 - nova - ironic: add instance_uuid before any other spawn a...13:45
*** mbeierl has quit IRC13:48
*** r-daneel has joined #openstack-ironic13:52
*** mbeierl has joined #openstack-ironic13:53
*** baha has joined #openstack-ironic13:58
*** markguz_ has joined #openstack-ironic13:59
openstackgerritWill Szumski proposed openstack/networking-generic-switch master: Adds ngs_switchport_mode config option to Dell PowerConnect driver  https://review.openstack.org/58163114:00
*** dprince has joined #openstack-ironic14:01
jaypipesjroll: I can hack on it.14:03
*** links has quit IRC14:03
*** markguz has quit IRC14:04
*** openstackgerrit has quit IRC14:04
jrolljaypipes: thanks! I think the convincing people is the hard part :)14:04
*** bnemec has joined #openstack-ironic14:05
*** jaganathan has quit IRC14:08
TheJuliajroll: that is indeed the hardest part14:09
* TheJulia looks to hire a hypnotoad for ramdisk booting14:09
jaypipesjroll, TheJulia: I have a little pull over in nova-land. will do my best.14:09
TheJuliajaypipes: let me know if you need another set of eyes. I don't jump in that code base often, but happy to context switch over to it14:09
jaypipeswill do, thanks14:10
* TheJulia hopes everyone is updating the whiteboard https://etherpad.openstack.org/p/IronicWhiteBoard14:14
*** skazi has quit IRC14:14
*** markguz_ has quit IRC14:14
*** markguz has joined #openstack-ironic14:15
*** openstackgerrit has joined #openstack-ironic14:16
openstackgerritMichael Turek proposed openstack/ironic-python-agent master: Install grub to PreP partition when prep_boot_part_uuid is provided  https://review.openstack.org/58296214:16
*** markguz has quit IRC14:20
*** lbragstad has joined #openstack-ironic14:20
jroll./ironic/api/controllers/v1/node.py:122:1: C901 'hide_fields_in_newer_versions' is too complex (18)14:24
jrollanother reason to start dropping early api versions :P14:24
openstackgerritMichael Turek proposed openstack/ironic-lib master: Add logic to create PReP partition for ppc64* arch  https://review.openstack.org/57210214:27
*** weshay is now known as weshay_mtg14:30
*** cjloader has joined #openstack-ironic14:31
*** r-daneel has quit IRC14:33
openstackgerritMichael Turek proposed openstack/ironic master: Pass prep_boot_part_uuid to install_bootloader for ppc64* partition images  https://review.openstack.org/57915914:34
mgoddardrloo: quick deploy steps question. If I create a custom interface which provides a deploy_step, should I expect it to be picked up in addition to the mega deploy step?14:35
rloomgoddard: no. oh, actually, yes. i didn't test it, but the framework is there :)14:35
mgoddardrloo: cool, that was my impression14:35
rloomgoddard: let me know if it works! (just use priority < 100)14:36
*** arnewiebalck__ has quit IRC14:36
openstackgerritMichael Turek proposed openstack/ironic-python-agent master: Install grub to PreP partition when prep_boot_part_uuid is provided  https://review.openstack.org/58296214:36
mgoddardrloo: which way do the priorities work? low first, high last?14:36
rloomgoddard: high to low14:37
*** dprince has quit IRC14:37
mgoddardrloo: ok, so we can only add steps after the mega deploy step currently14:37
jrolleasy to remember, highest priority task goes first :)14:37
rloomgoddard: in theory, you can add before. but if you look at the mega deploy, i'm not sure it makes sense to add something before it.14:38
rloomgoddard: so you can, if you can thiink of a deploy step that makes sense to do before mega.14:38
mgoddardrloo: ok, so 100 is not a maximum. Not sure where I'd picked that up14:38
rloomgoddard: no, not a max. i just picked a number. and i think they are floats. cuz i didn't want to constrain ourselves.14:39
mgoddardrloo: BIOS or RAID config would go before14:39
*** exodusftw has quit IRC14:39
rloomgoddard: oh yeah. i've already forgotten what the mega step does, but i think we do .prepare before the first step.14:39
*** exodusftw has joined #openstack-ironic14:40
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add conductor_group field to config, node and conductor objects  https://review.openstack.org/57962714:42
openstackgerritJim Rollenhagen proposed openstack/ironic master: Use conductor group for hash ring calculations  https://review.openstack.org/57968414:42
openstackgerritJim Rollenhagen proposed openstack/ironic master: Expose node.conductor_group in the REST API  https://review.openstack.org/58139114:42
jrollokay, I think this works correctly and jives with what we talked about14:42
*** links has joined #openstack-ironic14:47
openstackgerritMerged openstack/ironic-inspector master: Switch to using stestr  https://review.openstack.org/57852414:49
openstackgerritMerged openstack/python-ironicclient master: Support node's deploy_step field  https://review.openstack.org/58152014:49
*** AlexeyAbashkin has quit IRC14:53
*** kaifeng has joined #openstack-ironic14:55
*** stendulker has joined #openstack-ironic14:56
*** rpioso|afk is now known as rpioso14:56
rpiosoGood morning14:57
TheJuliagood morning14:58
*** rloo has quit IRC14:58
*** rloo has joined #openstack-ironic14:59
*** dprince has joined #openstack-ironic14:59
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Jul 16 15:00:11 2018 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
TheJuliao/15:00
*** mjura has quit IRC15:00
kaifengo/15:00
bdoddo/15:00
rlooo/15:00
etingofo/15:00
TheJuliaI hope everyone has had a wonderful weekend!15:01
mjtureko/15:01
TheJuliaAnd now it is time for our weekly meeting!15:01
stendulkero/15:01
jroll\o15:01
TheJuliaThe agenda can be found on the wiki15:01
rpiosoo/15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
TheJulia#topic Announcements/Reminders15:02
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:02
*** pcaruana has quit IRC15:02
TheJulia#info This week is R-6 for the Rocky development cycle15:03
TheJulia#info Final non-client libraries must be released this week.15:03
*** jiapei has joined #openstack-ironic15:03
TheJulia#info For this week, that means ironic-lib and sushy must be released.15:04
TheJulia#info Next week is R-5, which is final release of client libraries, feature freeze, requirements freeze, etc.15:05
TheJuliaAny questions?15:05
TheJulia#info Tomorrow, Tuesday at 12:00 UTC, We will be having a call to try and plan/discuss any last minute details and sort through anything contentious that needs to be discussed before the PTG.15:06
TheJulia#link https://etherpad.openstack.org/p/ironic-rocky-midcycle15:06
TheJuliaAnyone have anything to announce ?15:06
rloothxs to the folks that helped out on the bug day thing last week15:07
* TheJulia secretly replaces everyone's decaf coffee15:07
TheJuliarloo: ++15:08
mjturekooh yeah that might be good to announce - next bug day15:08
* dtantsur is still somewhere on the Baltic see :)15:08
mjturekAugust 2nd @ 1:00 PM - 2:00 PM15:08
rlooand thanks TheJulia for the awesome email wrt ironic15:08
mjtureksorry, 13:00 UTC to 14:00 UTC15:09
TheJuliamjturek: those times in UTC?15:09
TheJuliamjturek: awesome15:09
mjturekTheJulia: yep15:09
TheJulia#info Next Bug Day - August 2nd @ 13:00 to 14:00 UTC PM15:09
TheJuliaAnyone have anything else, or shall we move on?15:10
TheJuliaAnyway, moving on!15:11
*** AlexeyAbashkin has joined #openstack-ironic15:11
TheJulia#topic Review action items from previous meeting15:11
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:11
TheJulia#link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-09-15.00.log.html15:11
TheJuliaIt looks like we have nothing to discuss here, I believe everything was taken care of15:12
TheJuliaSo... moving on!15:12
TheJulia#topic Review subteam status reports15:12
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:12
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:12
TheJuliaStarting around line 15715:12
*** weshay_mtg is now known as weshay15:13
rloothe bugs stats should have been improved after last week's bug day? (L166ish)15:13
dtantsurI expected that as well, but dunno15:13
dtantsurmaybe we're growing more new bugs?15:13
TheJuliaHas anyone updated them?15:13
rloothe BIOS config stuff; are the TODOs as of today (vs 2 July)?15:14
TheJuliarloo: July 2nd, I believe... however I suspect they are still valid15:15
etingofjust a quick note that this script relies on the `ironic-triaged` tag to be present on the triaged task15:15
TheJuliayolanda_: ^^^15:15
TheJuliaI've not updated the bug numbers, if someone else can, that would be awesome15:15
dtantsurwdym by "updating numbers"? applying the tag?15:16
TheJuliaupdate the block of text indicating but counts and status on the etherpad15:16
mjturekTheJulia: trying to find the numbers, but not sure how? https://storyboard.openstack.org/#!/story/list?status=active&project_group_id=75 according to this is 51715:16
TheJuliaWe need to split this apart from general subteam status report15:17
dtantsurTheJulia: yep, I updated it15:17
TheJuliadtantsur: ahh, awesome15:17
dtantsurmjturek: I generate it from https://github.com/dtantsur/ironic-bug-report15:17
mjturekdtantsur: oh cool!15:17
rloomkrai: wrt your graphical console interface; i updated it. please just summarize the latest status; i don't think we want to carry weekly statuses on this; otherwise all of them will get way too long.15:18
TheJulialooks like hshinna is also not on :\15:18
rlooare we done with management interface refactoring? L231ish?15:18
TheJulia(regardingg bios interface)15:18
TheJuliarloo: I believe we are functionally done. etingof is there anything else on your plate for that?15:19
etingofquick note on the new bug panopticon -- https://storyboard.openstack.org/#!/board/8315:19
TheJuliaRegarding rescue: the nova patch landed over the weekend. \o/15:19
rloodtantsur: wrt classic driver removal, is that the latest (today) status? L252ish15:20
rlooWHEEE, great news about rescue!!!15:20
TheJuliaso anything we have held up in ironic pending that, we can merge15:20
jroll\o/15:20
dtantsurrloo: it is15:20
rlooTheJulia: just the CI patch i believe!15:20
rloodtantsur: OH. it is still july, sorry i thought it was august. moar caffeine...15:20
dtantsur:)15:20
dtantsurTheJulia: I remember approving something about rescue CI today15:21
etingofTheJulia, I believe the mgmt interface implementation is done, we may want to run some more tests in the CI perhaps15:21
TheJuliaAlso, with rescue, we landed a little bit of code in nova to gracefully handle the prior API version to enable the upgrade to be more graceful15:21
TheJuliaso \o/15:21
rlooThat was probably TheJulia's favorite part! :D15:22
dtantsur:)15:22
* dtantsur is still too happy the classic drivers are gone - great release!15:22
jroll++15:22
TheJuliaI think that largely means based on current statuses, we only have the conductor locality work and anything of high impact that we would like to see.15:23
* TheJulia feels like we're in a REALLY good place for releasing without any headaches and having a huge improvement this cycle15:23
TheJuliaAnyway, I'm good to move on? Everyone else ready to move on to proprities?15:24
TheJuliapriorities15:24
mjturekthis is probably a stretch (and not sure it's the right place to bring up) but I think I'm finished with the implementation here https://storyboard.openstack.org/#!/story/174905715:24
mjturekif it's too late, I understand15:24
TheJuliamjturek: at least the ironic-lib change didn't seem major so I dont see why not?!?15:25
mjturekTheJulia okay would appreciate some reviews if everyone's not already too swamped15:25
rloowrt priorities. we don15:27
rloo't need deploy steps there?15:27
TheJuliarloo: I was thinking for doc items or follow ups?15:27
rlooTheJulia: yeah, but i don't know that they are priorities for this week.15:27
dtantsurI'd like https://storyboard.openstack.org/#!/story/2002868 on priorities15:27
TheJulia#topic Priorities for the week15:27
*** openstack changes topic to "Priorities for the week (Meeting topic: ironic)"15:27
TheJuliarloo: ack15:28
rlooTheJulia: but i am fine if we leave it there.15:28
rloojust thinking we want to get any features in15:28
TheJuliaAnyone aware of sushy patches that need reviews this week?15:28
rloois there anything in nova-land that we need/want to get in for their FF?15:28
dtantsurTheJulia: there are a few open, I'm not sure how close to landing they are15:29
TheJuliadtantsur: agreed :\15:29
TheJuliabdodd: are you aware of anything that is needed in sushy for this release?15:29
rlooalso, our client freeze/release is next week. is conductor locality the only one that has a client part?15:30
*** tssurya has quit IRC15:30
TheJuliarloo: I'm not actively aware of anything nova side that must land for us this cycle15:30
TheJuliarloo: I believe that is correct15:30
rloowe're good then :)15:30
* dtantsur pokes etingof15:30
rloodtantsur: what about your reset-interfaces stuff?15:31
bdoddTheJulia: Not that I currently know of. Still testing/debugging my Redfish BIOSInterface code. But I haven't hit any issues with the sushy code.15:31
dtantsurrloo: the reset_interfaces work will need a client change15:31
dtantsurwill post tomorrow hopefully15:31
* dtantsur is fighting with unit tests15:31
rloook, i think dtantsur's stuff is a priority. otherwise, it'll make it hard for our users to change hw types15:31
jrollwell, it's already hard :)15:32
rlootrue, but we had classic drivers before.15:32
jrollI guess we are dropping drivers though, let's make it a priority15:32
TheJulia++15:32
* etingof does not see anything important to be merged with sushy atm15:32
TheJuliaI just added an entry for the client only side patch that dmitry posted15:32
jrollwell, the API needs to land too15:33
TheJuliabdodd: awesome!15:33
rloowe're still missing the 'get me hw-type driver properties for non-default interfaces' i think but ...15:33
* etingof shamelessly suggests merging this little sushy-tools patch of his own though -- https://review.openstack.org/#/c/568636/15:33
patchbotpatch 568636 - sushy-tools - Add unittests for OpenStack nova driver15:33
TheJuliaetingof: :)15:33
TheJuliaetingof: we don't need to cut sushy-tools this week :)15:33
* etingof /o\15:34
TheJuliasorry :(15:34
TheJuliaI think the priorities are basically good at this point, sans having the reset interface patches from dmitry. We can add that later in the week if we're in agreement15:35
rloowe might re-eval during mid-cycle tomorrow anyway. (hopefully it won't change)15:35
TheJulia++15:36
TheJuliaAnyway, seems like we can move on15:36
TheJuliaEveryone good with proceeding?15:36
rloothe bios interface support stuff is questionable (seems like it includes status stuff) but anyway, let's move on :)15:37
TheJuliaslightly less statusy now15:38
TheJuliaAnyway! Moving on!15:38
TheJulia#topic RFE Review15:38
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:38
TheJuliaI have an RFE that I wanted to bring up for discussion or raise awareness of because it is an interesting problem and it is something we should consider addressing.15:39
TheJulia#link https://storyboard.openstack.org/#!/story/154694915:39
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add reset_interfaces parameter to node's PATCH  https://review.openstack.org/58295115:39
*** r-daneel has joined #openstack-ironic15:39
TheJuliaAny thoughts?15:39
jroll"why not" is my only thought :)15:40
mjturekimplementation is here https://review.openstack.org/#/c/346723/15:40
patchbotpatch 346723 - ironic-python-agent - Parallel erase disk devices15:40
TheJuliaAnyone up for taking that on?15:40
rloois that an rfe or a bug?15:40
TheJuliadefinitely an RFE15:40
*** d0ugal has quit IRC15:41
*** d0ugal has joined #openstack-ironic15:41
*** d0ugal has quit IRC15:41
*** d0ugal has joined #openstack-ironic15:41
* TheJulia wonders if we have sufficient consensus to mark it approved and move on?15:41
rlooso looking at the patch and the -1 there. the question or a question is whether to make it configurable?15:42
rloooh wait, maybe they did make it configurable.15:42
rlooanyway, if it is an rfe, the description needs to be updated15:42
rlooother than that, i'm good with it being approved.15:42
TheJuliathey did not15:42
TheJuliaI'm good as well, and it seems like jroll is, so I feel like we have consensus15:43
rloooh, no config? i'm only looking at the comments, and saw 'done' but dunno what was done.15:43
* TheJulia goes and updates it15:43
rloosorry, is this configurable or not? (or do we want it configurable?)15:43
TheJuliayeah, it looks not configurable. I think it might be better to not to... but we're also dealing with hardware that could have quarks15:43
TheJuliaso maybe configurable would be best?15:43
jrollI'm fine with it with or without a config - I don't think the config is necessary, but for whatever reason we make everything configurable, so can deal with it15:43
rlooit is an IPA config, not ironic config15:44
jrollwould need to be able to configure it via ironic somehow, though15:44
TheJuliayeah, I think that is an implementation detail we can work through15:45
kaifengi remember ipa know options from command line..15:45
TheJuliaI have updated the rfe accordingly15:45
rlooTheJulia: it doesn't say anything about configuration?15:46
TheJuliarloo: I just put a comment on it stating that configurability is an implementation detail that can be figured out15:46
rlooif we don't care, should we mention that?15:46
rloook, so we want it.15:47
TheJuliayeah, I think so15:47
TheJuliaTime to proceed to Open Discussion?15:47
rloook, i'll update the description then. thx.15:47
TheJuliarloo: k15:47
* TheJulia wonders if it iddn't save15:47
*** ajya3 has joined #openstack-ironic15:48
*** ajya has quit IRC15:48
TheJulia#topic Open Discussion15:48
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:48
mjturekneed to hammer out any details for tomorrow's call?15:49
rlooTheJulia: are we trying to get https://review.openstack.org/#/c/576717/ for rocky?15:50
patchbotpatch 576717 - ironic-specs - Add Boot from Ramdisk spec15:50
rlooor just the spec approved?15:50
TheJuliaSo, one thing worth noting during open discussion. I've had some arm folks reach out about deploying arm gear in noop node. I've already got a grub related knob to help enable that, there are some outstanding questions, but I might post a patch or two more to make things easier/simpler15:50
TheJuliarloo: I'd like to get it merged, as-is I think it is only like 270-something lines15:50
TheJuliaAnd it would be a nice thing to provide the scientific community15:51
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add API tests for the reset_interfaces parameter  https://review.openstack.org/58298515:51
rlooTheJulia: i haven't actually read it; does it have a client component? (I'm guessing not)15:51
TheJuliasince they would be the ones jumping on it and trying to kick the tires as it were15:51
TheJuliarloo: no, no client component15:51
rlooTheJulia: ok thx.15:51
TheJuliaOh, BTW, Get your summit submissions in!15:52
TheJuliaI have nothing else to discuss or mention today15:53
TheJuliaAnyone have anything?15:53
kaifengif time permits, I'd like to know opinions about https://storyboard.openstack.org/#!/story/200297715:53
TheJuliaOh, 6 minues remaining15:53
kaifeng:)15:53
TheJuliakaifeng: Ohh, interesting15:54
kaifengmainly, the boot mode timeout option, I'm not sure if it's mandatory or optional in the IPMI standard15:54
kaifengbut I encountered several models didn't support this option15:55
TheJuliakaifeng: what sort of hardware did you encounter this on?15:55
TheJuliaI _think_ support is optional15:55
TheJuliakaifeng: was it that we never successful set it for long enough, or we never actually set it?15:56
kaifengactually the raw command never succeeds, so the pxe mode is not set15:56
rloobased on the comment, it looks like if the cmd doesn't succeed, it'll time out in 60s.15:57
rlooeiddccfiultrehgnfhtrjnjfglketkleicvvlvdjibjc15:58
kaifengright, but the timeout is an edge case15:58
* TheJulia wonders if rloo just turned on encryption :)15:58
TheJuliaor a cat decided to capture her keyboard15:58
rloooops :D15:59
kaifengin most cases, it would make it within timeout limits15:59
TheJuliaseems like something we ought to fix, exact details on how I personally have no opinion on at present16:00
rlookaifeng: do you mean 'timeout limits = 60s' or some other timeout limit?16:00
*** links has quit IRC16:00
kaifengMy thought is if this bmc option is optional, we can just wrap this raw command into a try catch, log a warning if not bmc doesn't support it16:01
kaifengrloo: I mean the power action timeout (60s)16:01
rlookaifeng: that won't solve your issue though? you want the inspector to stop, not continue?16:01
TheJuliarloo: I'm thinking the same thing16:01
TheJuliaI'm going to end the meeting, but we can keep discussing this16:02
TheJuliaThanks everyone!16:02
rlooa demain!16:02
dtantsurthanks!16:02
jrollthanks \o16:02
TheJuliarloo: you've got me confused :)16:02
*** d0ugal has quit IRC16:03
TheJulia#endmeeting16:03
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75"16:03
openstackMeeting ended Mon Jul 16 16:03:03 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:03
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-16-15.00.html16:03
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-16-15.00.txt16:03
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-16-15.00.log.html16:03
kaifengah, I think they are separate issues :) but I found inspector didn't stop because of the first one16:03
rlooTheJulia: i was trying to understand the problem based on the story description. not sure i do and i have to go eat16:03
TheJuliakaifeng: ahh, if they are separate, we should at least try and figure out the two separate issues then16:03
jrollTheJulia: regex.match('^eiddcc') == 'bumped a yubikey' :P16:04
rloojroll: oh, is that what i did?16:04
TheJuliajroll: hehe16:04
jrollyes :)16:04
TheJuliarloo: I think two separate things being the case might be the source of confusion16:06
TheJuliaAnyway, I think I need some lunch16:06
mgoddardrloo: looks like no staging driver changes are required. The only deploy interface listed in setup.cfg is AMT, and it inherits from iscsi_deploy16:06
rloomgoddard: sweet. thx for looking into it!16:06
TheJuliaIs that hardware even  on the market still?16:07
kaifengTheJulia, rloo, well, we can discuss later, don't keep your stomach waiting :)16:08
mgoddardthere is an AMT coffee stand at our local station, I assume they're using this driver to provision the coffee machine16:08
TheJulia++16:08
mgoddardI heard they wrote a custom UI for it using CoffeeScript16:09
TheJuliaseems like we should get NobodyCam to port his coffee driver to staging drivers repo16:09
*** rbartal has quit IRC16:09
* TheJulia realizes mgoddard might no tbe joking16:09
*** baha has quit IRC16:10
* mgoddard is usually joking, although it's not always obvious :)16:10
rloo:D16:11
*** d0ugal has joined #openstack-ironic16:11
TheJulia:)16:11
mgoddardat least that's what I tell myself when nobody laughs...16:11
TheJulia:(16:11
TheJuliaMaybe they are just not laughing in IRC but in the big blue room or someplace else where laughs are difficult to detect?16:12
*** dprince has quit IRC16:12
* mgoddard is still joking. Told you it's hard to tell!16:13
TheJuliaheh16:13
* TheJulia chuckles16:13
mgoddardthanks :)16:14
*** mjturek has quit IRC16:15
*** AlexeyAbashkin has quit IRC16:15
* TheJulia needs to go find lunch16:17
*** dprince has joined #openstack-ironic16:18
* TheJulia goes and finds lunch16:19
*** e0ne has quit IRC16:21
openstackgerritIlya Etingof proposed openstack/ironic master: Add RedfishInspect Interface  https://review.openstack.org/44942516:22
*** baha has joined #openstack-ironic16:25
*** mjturek has joined #openstack-ironic16:33
*** lbragstad has quit IRC16:33
mgoddardrloo: I've been through the deploy steps conductor patch again, and have a better feel for it now. I can't see anything that needs to change. I'm setting up a devstack environment to poke at tomorrow. I could look at the documentation or nits, let me know which would be best for you16:34
*** stendulker has quit IRC16:42
*** milan has quit IRC16:42
*** kaifeng has quit IRC16:43
*** trown is now known as trown|lunch16:43
*** mgoddard has quit IRC16:43
*** tesseract has quit IRC16:43
rloomgoddard. thx! I'm doing the nits now, so maybe the docs. I was thinking it'd be similar/subset of existing docs for clean steps.16:44
*** dtantsur is now known as dtantsur|afk16:44
dtantsur|afksee you16:44
yolanda_sorry, i was at the doctor. We don't have any pending patches for bios config16:44
*** sambetts is now known as sambetts|afk16:51
*** d0ugal has quit IRC16:52
*** dougsz1 has quit IRC17:01
*** d0ugal has joined #openstack-ironic17:05
*** mjturek has quit IRC17:13
*** mjturek has joined #openstack-ironic17:15
*** mjturek has quit IRC17:18
*** mjturek has joined #openstack-ironic17:19
*** mjturek has quit IRC17:20
*** jcoufal has quit IRC17:23
*** lbragstad has joined #openstack-ironic17:24
*** mjturek has joined #openstack-ironic17:33
*** r-daneel_ has joined #openstack-ironic17:34
*** r-daneel has quit IRC17:35
*** r-daneel_ is now known as r-daneel17:35
*** trown|lunch is now known as trown17:37
*** r-daneel has quit IRC17:40
TheJuliamjturek: your ipa patch has a one character misalignment17:42
mjturekgaaah dangit17:43
mjturekfixing now TheJulia17:43
*** r-daneel has joined #openstack-ironic17:46
*** tssurya has joined #openstack-ironic17:46
*** r-daneel has quit IRC17:50
*** r-daneel has joined #openstack-ironic17:52
openstackgerritMichael Turek proposed openstack/ironic-python-agent master: Install grub to PreP partition when prep_boot_part_uuid is provided  https://review.openstack.org/58296217:53
mjturekthanks for the heads up TheJulia17:53
*** markguz has joined #openstack-ironic17:55
*** milan has joined #openstack-ironic17:58
*** dprince has quit IRC18:01
*** mmethot has joined #openstack-ironic18:01
*** d0ugal has quit IRC18:04
TheJuliaetingof: I noticed something with the redfish inspect interface, take a look when you have a few minutes free :)18:05
*** markguz has quit IRC18:05
*** markguz has joined #openstack-ironic18:06
*** markguz has quit IRC18:11
*** r-daneel_ has joined #openstack-ironic18:12
*** e0ne has joined #openstack-ironic18:14
*** dprince has joined #openstack-ironic18:14
*** r-daneel has quit IRC18:14
*** r-daneel_ is now known as r-daneel18:14
*** d0ugal has joined #openstack-ironic18:16
TheJuliarajinir: It looks like dell ci is broken :(18:20
*** baha has quit IRC18:21
*** mjturek has quit IRC18:21
TheJuliaThey look molteniron related18:21
*** jiapei has quit IRC18:23
TheJuliajroll: you around?18:27
jrollTheJulia: yar18:27
TheJulia3 pep8 errors on the conductor group patch :(18:27
TheJuliaOne I'm wondering if there is a way to make it happy without touching complexity18:27
jrollwhaaaat18:28
* jroll thought he ran it and fixed that18:29
jrolluh, red across the board, wat18:29
TheJuliaparallel realities?18:29
jrolloh, only first patch is all red18:30
* jroll gets confused, attempts to fix18:30
* TheJulia is confused because she hasn't seen all red18:30
* TheJulia looks at gerrit again18:30
jrollhttps://review.openstack.org/#/c/579627/18:30
patchbotpatch 579627 - ironic - Add conductor_group field to config, node and cond...18:30
TheJuliahmm18:31
jrollprobably something silly18:31
TheJuliayeah, just pulled that one up18:31
jrolla syntax error, this feels like an old patch18:32
jrolloh, I modified the wrong patches for certain things, it seems18:34
TheJulia:(18:36
TheJuliasadly it happens18:36
jrollthanks for the heads up18:37
TheJuliaDidn't seem intentional, so I wanted to make sure you were aware of it18:37
*** mjturek has joined #openstack-ironic18:40
*** baha has joined #openstack-ironic18:40
*** mjturek_ has joined #openstack-ironic18:41
*** mjturek has quit IRC18:44
*** d0ugal has quit IRC18:46
*** d0ugal has joined #openstack-ironic18:49
* rpioso will try to ask rajinir about the Dell EMC CI19:00
rajinirTheJulia: Fixed it, will be green soon. rpisoso19:01
rajinirrpioso: Fixed the CI, will be green soon19:02
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add conductor_group field to config, node and conductor objects  https://review.openstack.org/57962719:02
openstackgerritJim Rollenhagen proposed openstack/ironic master: Use conductor group for hash ring calculations  https://review.openstack.org/57968419:02
openstackgerritJim Rollenhagen proposed openstack/ironic master: Expose node.conductor_group in the REST API  https://review.openstack.org/58139119:02
jrollthere we go TheJulia19:02
*** d0ugal has quit IRC19:02
TheJuliarajinir: thanks19:03
TheJuliajroll: thanks19:03
jrollnp19:03
*** d0ugal has joined #openstack-ironic19:04
*** e0ne has quit IRC19:05
*** arnewiebalck__ has joined #openstack-ironic19:14
NobodyCamGood Morning Ironic'ers19:17
*** arnewiebalck__ has quit IRC19:22
TheJuliagood morning NobodyCam19:25
*** e0ne has joined #openstack-ironic19:25
*** e0ne has quit IRC19:26
openstackgerritJulia Kreger proposed openstack/ironic master: Make pxelinux.cfg folder configurable  https://review.openstack.org/58302819:28
NobodyCam:)19:30
TheJuliastepping away for a little while, not feeling that great and hoping stepping away for a little bit helps19:36
NobodyCamTheJulia: Feel better :)19:37
openstackgerritMerged openstack/ironic master: Caching of PDU autodiscovery  https://review.openstack.org/39923419:45
openstackgerritMerged openstack/ironic master: Migrate ironic `snmp` driver to the latest pysnmp API  https://review.openstack.org/57898719:47
*** r-daneel has quit IRC19:57
*** r-daneel has joined #openstack-ironic19:58
etingofTheJulia, I have shamelessly taken over Nisha's patch in hope to advance it a bit (if this is what you have noticed) ;)20:06
*** d0ugal has quit IRC20:16
*** bfournie has quit IRC20:17
*** d0ugal has joined #openstack-ironic20:22
*** AlexeyAbashkin has joined #openstack-ironic20:26
openstackgerritRuby Loo proposed openstack/ironic master: Minor changes for deploy_steps framework  https://review.openstack.org/58304120:31
*** d0ugal has quit IRC20:34
*** AlexeyAbashkin has quit IRC20:36
*** d0ugal has joined #openstack-ironic20:46
TheJuliaetingof: okay20:47
openstackgerritRuby Loo proposed openstack/ironic master: Minor changes for deploy_steps framework  https://review.openstack.org/58304120:55
*** ajya3 has quit IRC21:00
* TheJulia watches the local hamster wheel spin on her laptop21:01
*** bfournie has joined #openstack-ironic21:03
TheJuliamjturek_: network bootloader wise with power, is it only grub that is supported21:06
tonybTheJulia: Sadly no, but on ppc64le that is useful for OpenStack (running to OPAL firmware) theree is a PXE parseer built into the FW so you don't need a bootlaoder like grub in that context21:12
tonybTheJulia: I don't think we're lookign at supporting ironic to manager LPARs so we don't really need to worry about them21:13
* tonyb suspects that is more confusing than halepfule21:13
TheJuliatonyb: the parser, what format does it expect or support?21:13
tonybpxe/syslinux21:13
mjturek_TheJulia: http://jk.ozlabs.org/blog/post/158/netbooting-petitboot/ useful  article21:14
tonybI don't really know what the std is21:14
TheJuliamaybe we should endevour to pickup the pxe/ipxe split... or just make a meta interface that sets instance loads of ipxe to be pxe and depracate the ipxe_enabeld setting21:15
mjturek_petitboot handles netbooting21:15
*** baha has quit IRC21:15
mjturek_TheJulia: so basically there would be a ipxe and pxe module?21:15
* TheJulia is thinking about multi-arch netbooting, specifically pondering arm since it really seems that my only real option is to directly invoke grubaa64.efi and let it roll from there with grub files21:15
mjturek_hardware_type I mean21:16
TheJuliamjturek_: boot interface I guess21:16
tonybTheJulia: for aarch64 I believe that you're correct21:16
mjturek_got it21:16
TheJuliaanyway our behavior for ipxe right now is conductor wide... which doesn't work for.. well... any of it21:17
TheJuliain multiarch land21:17
TheJulia:(21:17
tonybTheJulia: it's true, I disable ipxe on my multiarch environments21:17
mjturek_heading out for the night, adios21:18
*** mjturek_ is now known as mjturek|off21:18
TheJuliagoodnight mjturek|off21:19
TheJuliatonyb: that makes me sad21:19
TheJuliabut I get it21:19
tonybTheJulia: well we *could* build an ipxe interpretter into petitboot ;P21:20
tonybTheJulia: but that doesn't fix the deployed systems out there21:20
tonybHeck it wouldn't even be that hard21:22
* tonyb wonders about the right way to set the default boot type. (pxe vs ipxe)21:22
tonybTheJulia: anyway that isn't goign to save us this year :/21:22
TheJuliaWell, my hope for ipxe was things like loading over to iscsi, but yeah21:27
* TheJulia ponders going, opening a beer, and hacking an overlay interface21:30
TheJuliathen run with ipxe enabled, but an ipxe interface becomes the setting21:30
TheJuliawe kind of have a spec for this too...21:30
TheJuliabut it is super high level21:30
*** milan has quit IRC21:32
*** rh-jelabarre has quit IRC21:33
*** ianw_pto is now known as ianw21:44
*** lbragstad has quit IRC21:47
*** markguz has joined #openstack-ironic21:57
*** harlowja has joined #openstack-ironic21:59
*** dprince has quit IRC22:01
*** lbragstad has joined #openstack-ironic22:04
*** markguz has quit IRC22:06
*** markguz has joined #openstack-ironic22:07
*** tssurya has quit IRC22:07
*** bfournie has quit IRC22:12
*** markguz has quit IRC22:12
*** bfournie has joined #openstack-ironic22:12
*** rcernin has joined #openstack-ironic22:15
*** bfournie has quit IRC22:17
*** r-daneel_ has joined #openstack-ironic22:17
*** r-daneel has quit IRC22:18
*** r-daneel_ is now known as r-daneel22:18
*** ltomasbo has quit IRC22:30
*** radez has quit IRC22:33
*** pmannidi has joined #openstack-ironic22:54
*** rcernin has quit IRC22:54
*** markguz has joined #openstack-ironic23:01
*** markguz has quit IRC23:02
*** markguz has joined #openstack-ironic23:03
*** harlowja has quit IRC23:03
*** rpioso is now known as rpioso|afk23:04
*** markguz has quit IRC23:07
*** rcernin has joined #openstack-ironic23:08
*** r-daneel has quit IRC23:12
*** kzaitsev_pi has quit IRC23:39
*** kzaitsev_pi has joined #openstack-ironic23:41
« Sunday, 2018-07-15 Index Tuesday, 2018-07-17 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!