Monday, 2018-04-09

« Sunday, 2018-04-08 Index Tuesday, 2018-04-10 »
*** rbudden has joined #openstack-ironic00:10
*** kiseok7 has joined #openstack-ironic00:58
openstackgerritKaifeng Wang proposed openstack/ironic master: Fix test_get_nodeinfo_list_with_filters  https://review.openstack.org/55960200:59
*** bfournie has joined #openstack-ironic01:17
*** AlexeyAbashkin has joined #openstack-ironic01:50
*** tonyb has joined #openstack-ironic01:50
*** AlexeyAbashkin has quit IRC01:54
*** tiendc has joined #openstack-ironic01:54
*** alex_xu has joined #openstack-ironic01:58
*** edleafe- has joined #openstack-ironic02:09
*** edleafe has quit IRC02:10
*** edleafe- is now known as edleafe02:10
*** sapd_ has joined #openstack-ironic02:23
*** sapd has quit IRC02:26
*** mikal_ has quit IRC02:36
*** mikal has joined #openstack-ironic02:36
*** AlexeyAbashkin has joined #openstack-ironic02:50
openstackgerritZachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group  https://review.openstack.org/55192202:55
*** AlexeyAbashkin has quit IRC02:55
*** rbudden has quit IRC03:15
*** rbudden has joined #openstack-ironic03:24
*** tiendc has quit IRC03:57
*** tiendc has joined #openstack-ironic03:58
*** zhangfei has joined #openstack-ironic04:00
*** links has joined #openstack-ironic04:25
*** fragatina has quit IRC04:35
*** fragatina has joined #openstack-ironic04:42
*** rbudden has quit IRC04:46
*** vmud213 has joined #openstack-ironic04:46
*** phuongnh has joined #openstack-ironic05:03
*** jtomasek has joined #openstack-ironic05:48
*** d0ugal has quit IRC06:12
*** namnh has joined #openstack-ironic06:29
*** rbartal has joined #openstack-ironic06:33
*** pcaruana has joined #openstack-ironic06:35
*** oanson has quit IRC06:42
*** oanson has joined #openstack-ironic06:44
*** namnh has quit IRC06:46
*** namnh has joined #openstack-ironic06:46
*** hjensas has joined #openstack-ironic07:06
*** hjensas has quit IRC07:06
*** hjensas has joined #openstack-ironic07:06
*** jaganathan has joined #openstack-ironic07:09
*** tesseract has joined #openstack-ironic07:14
*** namnh has quit IRC07:14
*** tiendc has quit IRC07:14
*** tiendc has joined #openstack-ironic07:14
*** namnh has joined #openstack-ironic07:14
*** rcernin has quit IRC07:16
*** d0ugal has joined #openstack-ironic07:32
*** d0ugal has quit IRC07:32
*** d0ugal has joined #openstack-ironic07:32
*** jistr is now known as jistr|dentist07:34
*** athomas has joined #openstack-ironic07:37
*** lucas-afk is now known as lucasagomes07:56
*** hoonetorg has quit IRC07:59
*** hoonetorg has joined #openstack-ironic08:12
*** dougsz has joined #openstack-ironic08:16
*** ktibi has joined #openstack-ironic08:27
ktibihi, ironic support option enable_proxy_headers_parsing for proxy SSL ?? I can see option in doc for ocata but can't find it for pike or queens :/08:28
*** derekh has joined #openstack-ironic08:32
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: The direct deploy interface provisioning with HTTP server  https://review.openstack.org/50403908:38
*** sambetts|afk is now known as sambetts08:55
sambettsMorning all08:56
*** jistr|dentist is now known as jistr09:02
*** alex_xu has quit IRC09:03
*** alex_xu has joined #openstack-ironic09:08
openstackgerritKaifeng Wang proposed openstack/ironic-specs master: The direct deploy interface provisioning with HTTP server  https://review.openstack.org/50403909:27
openstackgerritjiapei proposed openstack/ironic master: Fix paramater name in xclarity driver  https://review.openstack.org/55753309:29
*** gyankum has joined #openstack-ironic09:34
openstackgerritjiapei proposed openstack/ironic master: Add documentation for XClarity Driver  https://review.openstack.org/54965909:38
*** milan_ has joined #openstack-ironic09:40
*** vabada has joined #openstack-ironic09:48
*** tiendc has quit IRC09:58
*** namnh has quit IRC09:58
*** mgoddard has joined #openstack-ironic09:58
*** tiendc has joined #openstack-ironic09:58
*** namnh has joined #openstack-ironic09:58
*** livelace-link has quit IRC10:07
*** namnh has quit IRC10:08
openstackgerritSam Betts proposed openstack/ironic master: Add detail=[True, False] query string to API list endpoints  https://review.openstack.org/43149710:08
*** gyankum has quit IRC10:27
*** parlos has joined #openstack-ironic10:33
openstackgerritSam Betts proposed openstack/ironic master: [devstack] Allow setting TFTP max blocksize  https://review.openstack.org/52873710:38
*** serlex has joined #openstack-ironic10:43
*** livelace-link has joined #openstack-ironic10:45
*** livelace-link has quit IRC10:46
*** livelace-link has joined #openstack-ironic10:48
*** exodusftw has quit IRC10:48
*** livelace-link has quit IRC10:49
*** exodusftw has joined #openstack-ironic10:54
*** zhangfei has quit IRC11:01
*** lucasagomes is now known as lucas-hungry11:05
*** vmud213 has quit IRC11:06
*** phuongnh has quit IRC11:06
*** fragatina has quit IRC11:17
*** dougsz has quit IRC11:21
*** bfournie has quit IRC11:22
*** pbourke_ has quit IRC11:45
*** gyankum has joined #openstack-ironic11:45
*** pbourke_ has joined #openstack-ironic11:45
*** tiendc has quit IRC11:46
*** dprince has joined #openstack-ironic11:47
*** parlos has quit IRC11:47
*** ktibi has quit IRC11:49
*** ktibi has joined #openstack-ironic11:50
openstackgerritjiapei proposed openstack/ironic master: Fix paramater name in xclarity driver  https://review.openstack.org/55753311:54
*** e0ne has joined #openstack-ironic11:56
*** lucas-hungry is now known as lucasagomes11:56
openstackgerritjiapei proposed openstack/ironic master: Fix paramater name in xclarity driver  https://review.openstack.org/55753311:58
*** dougsz has joined #openstack-ironic12:01
*** gyankum has quit IRC12:04
*** bfournie has joined #openstack-ironic12:05
*** hwoarang has joined #openstack-ironic12:12
*** mmethot has joined #openstack-ironic12:13
*** zhangfei has joined #openstack-ironic12:25
*** e0ne has quit IRC12:31
*** e0ne has joined #openstack-ironic12:31
*** e0ne has quit IRC12:36
TheJuliaGood morning12:38
*** mjturek has joined #openstack-ironic12:44
*** pcaruana has quit IRC12:46
*** Goneri has joined #openstack-ironic12:52
*** rloo has joined #openstack-ironic12:54
openstackgerritMerged openstack/ironic-lib master: Add retry attempts for the partprobe command  https://review.openstack.org/54203512:55
openstackgerritMerged openstack/ironic master: Reference architecture: small cloud with trusted tenants  https://review.openstack.org/55698612:55
*** jcoufal has joined #openstack-ironic12:57
*** hwoarang has quit IRC12:57
*** hwoarang has joined #openstack-ironic12:57
*** AlexeyAbashkin has joined #openstack-ironic13:01
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API  https://review.openstack.org/51257913:08
*** AlexeyAbashkin has quit IRC13:10
jrollmorning everyone13:10
*** AlexeyAbashkin has joined #openstack-ironic13:10
*** pcaruana has joined #openstack-ironic13:11
mmethot'morning13:16
*** AlexeyAbashkin has quit IRC13:17
*** AlexeyAbashkin has joined #openstack-ironic13:17
*** baha has joined #openstack-ironic13:19
*** Goneri has quit IRC13:21
*** rbudden has joined #openstack-ironic13:22
*** zhangfei has quit IRC13:26
*** lbragstad has joined #openstack-ironic13:27
*** rpioso|afk is now known as rpioso13:27
rpiosoGood morning, jroll, mmethot, and TheJulia.13:28
*** AlexeyAbashkin has quit IRC13:34
*** hwoarang has quit IRC13:35
*** AlexeyAbashkin has joined #openstack-ironic13:37
rlooHappy Monday morning everyone, rpioso, jroll, TheJulia, mmethot!13:48
rpiosorloo: :-)13:48
rlooAnd... another reason why we should try to land (or not land) patches soon: https://review.openstack.org/#/c/542035/ (see my comment)13:49
patchbotpatch 542035 - ironic-lib - Add retry attempts for the partprobe command (MERGED)13:49
jroll:|13:52
jrollmorning rloo and rpioso13:52
*** r-daneel has joined #openstack-ironic13:54
TheJuliagood morning13:58
TheJuliarloo: fun :(13:58
openstackgerritMerged openstack/ironic master: Copy port[group] VIF info from extra to internal_info  https://review.openstack.org/51163614:00
openstackgerritMerged openstack/ironic-tempest-plugin master: Add [baremetal] available_nodes configuration option  https://review.openstack.org/54138314:00
rlooTheJulia: I'm not quite sure what to do there. we can do nothing and leave the older patch lying around. when we change to use that version of ironic-lib, we may want to add a reno to indicate the new config option (although we don't do that for new configs/change-of-behavior wrt other libraries we pull in)?14:00
openstackgerritMerged openstack/ironic stable/pike: Change pxe dhcp options name to codes.  https://review.openstack.org/52524814:00
TheJuliaI suspect our only real option is to leave it be. I don't think we need any sort of release note. Besides we've not set any o that up on ironic-lib14:04
*** bnemec has joined #openstack-ironic14:05
rlooTheJulia: i think we decided not to have reno for ironic-lib since it was for internal consumption only. do we want a reno in ironic though, when we start using that ironic-lib code?14:05
TheJuliaI suspect for ironic that would be unnecessary because athe config reference and generated config would appear then and it is moderately self explanditory14:07
jrollI suspect that someone upgrading is more likely to read the release notes than to diff the config reference :)14:09
*** cjloader has joined #openstack-ironic14:09
jrollthat said, it's got a sane default, and it isn't anything that should scare someone - it doesn't exactly need a heads up14:09
jrollonly really need to tune it for very weird cases14:09
*** jjohnson2 has joined #openstack-ironic14:11
*** milan_ has quit IRC14:13
rloojroll, TheJulia: so this fixes a bug, and we typically have release notes for bugs. And this changes the behaviour too, with the new config. we typically add a reno for new configs. Otherwise, do we expect folks to do a diff between old/new configs to see what changed?14:14
rloohaving said that and with what jroll mentioned, i think it would be good to add a reno in ironic about the new config option/point to the bug. IF anyone remembers to do this... :)14:15
jrollrloo: I'm not strictly opposed to a release note, I guess, just saying with my deployer hat on, I wouldn't be upset about not having a note. if it's a bug I really care about I'm probably subscribed to the bug tracker notifications.14:15
TheJuliaI do agree it would be good, I'm frustrated that we're discussing and not just doing14:15
jrollrelease notes for every bug fix always surprised me14:15
jrollheh14:16
*** r-daneel_ has joined #openstack-ironic14:16
TheJuliaI'd also be okayw ith not doing it as well14:16
rlooTheJulia: we can't do it yet. or can we? we have to actually release the lib etc.14:16
jrollrloo: you could push a patch and wip it, the patch being in your queue will remind you :)14:16
TheJuliarloo: we would have to release the lib, and we might as well I guess14:16
rloojroll: oh, i thought release notes were supposed to mention every bug fix?14:16
rloojroll: bug fix that affects the user... ?14:17
jrollrloo: not sure we ever declared that as a rule, it just started happening and became a de-facto thing. again it isn't a bad thing, just surprises me :)14:17
*** r-daneel has quit IRC14:18
*** r-daneel_ is now known as r-daneel14:18
rloojroll: oh. well, there is a 'fixes' or whatever section. anyway, i think i just followed... ?? we're you in charge then? :D14:18
rloojroll: ok, i'll do the wip thing...14:18
jrollI was, yeah, I just got the framework going and everyone else took it from there :)14:18
rlooha ha14:18
*** krtaylor has quit IRC14:19
*** jroll has quit IRC14:22
*** jroll has joined #openstack-ironic14:23
TheJuliaDoes anyone agree/disagree with my nits on https://review.openstack.org/#/c/559500/2 ?14:25
patchbotpatch 559500 - ironic - Implement a function to check the image status14:25
openstackgerritMerged openstack/ironic master: Update and replace http with https for doc links  https://review.openstack.org/55742614:26
*** r-daneel has quit IRC14:27
sambettsTheJulia: yeah I agree, show fails earliest for us when these images aren't valid, I just wasn't sure if there was any point when we might want to show an image (fetch its info) without blowing up if it wasn't active14:28
sambettsTheJulia: I couldn't think of one for now so thats why I'm good with where it is14:28
jrollTheJulia: the grammar bits? yeah, makes sense to me14:29
sambettsit felt like it exceeded the scope of the function if you know what I mean14:29
rlooTheJulia: I commented. I'd like to see the actual status, but ... :)14:30
*** kaifeng has joined #openstack-ironic14:33
*** hshiina has joined #openstack-ironic14:34
*** r-daneel has joined #openstack-ironic14:35
mmethotthanks guys14:38
mmethotI honestly wasnt sure about the wording14:39
mmethotAs to why I selected to shove it in show, was due to how quickly it would show us failure14:39
TheJuliasambetts: I feel like we're almost at "notfound" if not active :)14:41
*** hw_wutianwei has joined #openstack-ironic14:41
sambettsTheJulia: yeah there isn't much we can do with it in any state other than active14:41
TheJuliaYeah, actual status might be good, or the id value of the image14:41
TheJuliasomething to trace back/identify14:42
TheJuliasambetts: agreed, I think there are only two statuses really14:42
mmethotTheJulia, 2 that could work?14:42
TheJuliaI think it is "active" or "inactive"14:42
sambettsthere are a bunch of pending states too14:43
TheJuliaso the active only check seems good to me....14:43
mmethothttps://docs.openstack.org/glance/pike/user/statuses.html14:43
TheJuliasambetts: Oh, I've only ever seen the two, but yeah... too much using image_source :)14:43
sambettsan image can be in "pending" when its been created but not uploaded yet14:43
mmethotThere's a bunch of states, all of them would indicate failure minus active14:43
TheJuliayup14:43
sambettsagreeds14:44
mmethotI hit this issue when my image was stuck in pending14:44
mmethotsambetts, TheJulia so do we have a preferred wording? If I include the current status I would have to rephrase altogether14:45
TheJuliaEh, I would just state that the "%(image) is not in an active state"14:46
TheJuliaerr, %(image)s14:46
jrollooo, almost meeting time14:46
rlooTheJulia, mmethot: i'm not sure, i thought i saw the image ID being passed to the exception, so the string there may already spit it out.14:47
TheJuliaindeed14:47
rloojroll: thx for heads up...14:47
jroll:)14:47
mmethotrloo, yeah the iomageid is already in the error msg14:48
*** rbartal has quit IRC14:48
rlooand sigh. if we have launchpad bugs with corresponding storyboard bugs ... and we have a patch that links to launchpad bug, nothing updates the storyboard bug. eg: https://storyboard.openstack.org/#!/story/175676014:48
rloommethot: ok, let's not mention it twice then :)14:49
mmethot_msg_fmt = _("Image %(image_id)s is unacceptable: %(reason)s")14:49
TheJuliarloo: patches using the launchpad syntax?14:49
TheJuliaawesome14:49
TheJuliarloo: or the storyboard syntax?14:50
TheJuliayeah, launchpad syntax14:50
rlooTheJulia: I mean, the launchpad bug got updated to indicate that it was fixed, but the corresponding storyboard one didn't. cuz the patch had 'Closes-Bug' or whatever, instead of 'Story'....14:50
TheJuliayup14:50
rlooTheJulia: so... hmm... :-( we shoudl try to change all patches to have Story...14:51
rloocores should do that ^^ before approving i think14:51
TheJulia++14:51
*** vmud213 has joined #openstack-ironic14:51
jroll:/14:51
rlooTheJulia: is there any way to block launchpad from accepting/updating ?14:52
TheJuliarloo: afaik no14:52
rlooTheJulia: well, if i have any great ideas... wondering if there is anything we can do in bug bash.14:53
jrollwell, we could patch the tooling that updates launchpad14:53
jrollto check if the storyboard version also exists, and update it if so14:54
mmethotTheJulia, reason=_("The image is required to be in an active state."))14:54
mmethotWe good to roll with that then?14:54
TheJuliammethot: I think we are14:54
*** AlexeyAbashkin has quit IRC14:54
mmethotcool14:54
rloojroll: yup, that'd be a win for all the projects i think. an infra question then? or by 'we' you mean us here?14:54
*** Goneri has joined #openstack-ironic14:54
openstackgerritMarc Methot proposed openstack/ironic master: Implement a function to check the image status  https://review.openstack.org/55950014:55
*** namnh has joined #openstack-ironic14:55
TheJuliajroll: projects.yaml knows, fwiw14:55
jrollrloo: 'we' in the sense of the openstack community14:55
jrollTheJulia: perfect14:55
rloo:)14:55
jrollhere's the code, I believe: https://github.com/openstack-infra/jeepyb/blob/master/jeepyb/cmd/update_bug.py14:55
jrollthough that doesn't have a storyboard reference, maybe there's a different codebase for that14:56
*** zhangfei has joined #openstack-ironic14:56
*** stendulker has joined #openstack-ironic14:57
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Apr  9 15:00:18 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
jroll\o15:00
mgoddardo/15:00
TheJuliaGood morning everyone15:00
etingofo/15:00
kaifengo/15:00
hshiinao/15:00
TheJuliaTime for an ironic meeting!15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
rpiosoo/15:00
rlooo/15:00
stendulkero/15:01
TheJulia#topic Announcements / Reminder15:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"15:01
TheJuliaI have created an etherpad to gather topics for the forum. Please add items this week.15:01
TheJulia#link https://etherpad.openstack.org/p/YVR-ironic-forum-sessions15:01
hjensaso/15:01
jrollnice15:02
*** vmud213 has quit IRC15:02
TheJuliaDoes anyone have any other announcements or reminders to bring up?15:02
*** vmud213 has joined #openstack-ironic15:02
* TheJulia hears crickets15:03
mmethotlol15:03
TheJuliaMoving on!15:04
TheJulia#topic Review action items from our previous meeting15:04
*** openstack changes topic to "Review action items from our previous meeting (Meeting topic: ironic)"15:04
TheJulia#link http://eavesdrop.openstack.org/meetings/ironic/2018/15:04
TheJuliaAhh yes, to look at the bugs which was my action item.15:05
jlvillalo/15:05
TheJuliaI looked, and I think we should revise processes... or at least our view or processes. Viewing the single group in launchpad does make it VERY easy though :)15:05
TheJulia\o15:05
TheJuliaAnyway, we can discuss that later15:06
TheJuliaOnward!15:06
rloodo we want to discuss bug bash now or later?15:06
TheJulia#topic Review subteam status reports15:06
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:06
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:06
TheJuliarloo: during open discussion I think :)15:07
rlooTheJulia: ok15:07
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116215:07
openstackgerrityolanda.robla proposed openstack/ironic master: Add bios_interface db field  https://review.openstack.org/52860915:07
rlooTheJulia: we fixed that critical sushy bug, right?15:07
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140215:07
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add RPC object  https://review.openstack.org/51171415:07
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface  https://review.openstack.org/50779315:07
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOS caching  https://review.openstack.org/51220015:07
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API  https://review.openstack.org/51257915:07
TheJuliaSo I went through and updated a few things, but I was somewhat out of touch last week attending a meeting.15:07
rlooTheJulia: ^^ wrt the sushy bug, did a release get done?15:08
TheJuliaLooks like we did15:08
rloothis bug: https://bugs.launchpad.net/sushy/+bug/175451415:08
openstackLaunchpad bug 1754514 in sushy "Base authentication don't work" [Critical,Fix released] - Assigned to deray (debayan-ray)15:08
TheJuliawell, release15:08
TheJuliahmmm15:08
TheJuliaDmitry was going to create them15:08
TheJuliachecking15:08
TheJuliaRelease is still pending https://review.openstack.org/#/c/558799/15:09
patchbotpatch 558799 - releases - Release sushy 1.3.2 for Queens15:09
rlooTheJulia: ok, i'll update the status of that.15:10
TheJuliaI just did15:10
rlooTheJulia: :)15:10
rlooTheJulia: wrt storyboard migration, L235. what needs to be done there?15:11
jrollrloo: do we want to call for volunteers for sighup support in ironic-inspector and networking-baremetal?15:12
rloojroll: oh, yeah, that'd be good.15:13
* jroll added it15:13
TheJuliarloo: bunch of patches are pending updating docs15:13
rloojroll: thx! (how'd he know i wasn't working on it... ? :D)15:13
jrollheh15:13
TheJuliaSounds good15:13
rlooTheJulia: do you have a list or is there a topic. let's just get them done.15:14
jroll++15:14
TheJuliaI raced through them and didn't topic them. I can make a list15:14
*** e0ne has joined #openstack-ironic15:14
*** hwoarang has joined #openstack-ironic15:14
rlooTheJulia: thx.15:14
TheJuliarloo: I'll make it after the meeting15:15
kaifengrloo: i can be the victim for ironic-inspector15:15
TheJuliaeveryone done reviewing the status of all the things15:15
* hjensas can look at sighup support for networking-baremetal.15:15
rlookaifeng: sweet, thx!15:15
jrollthanks hjensas :)15:15
TheJuliakaifeng: hjensas: awesome, thanks!15:15
mjtureko/15:15
jrolland thanks kaifeng :)15:15
TheJulia\o mjturek15:15
rloothx hjensas!15:16
* rloo done with status15:16
TheJuliaOnward to priorities for the week?15:16
jrollditto15:16
kaifengyw :)15:16
TheJulia#topic Priorities for the coming week15:17
*** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)"15:17
TheJuliaSo it is clear we've been stalling out on the remaining of rescue. I've shuffled the text a little so we can hopefully have a little more clarity there15:18
jrollare we stalling or just disagreeing?15:20
jrollthe one patch has a +215:20
TheJuliaCycling slowly15:20
TheJuliavery slowly15:20
jrollyeah15:20
TheJuliaAs for bios interface, the unit tests are failing15:20
TheJuliaspecifically ones that wont fail locally since they require the database engine15:21
TheJuliaIt impacts the entire stack of patches15:21
jrolloh fun!15:21
rlooouch. someone looking at it, or is it 'i don't know how to fix' thing?15:21
jrollwe know that it's possible to connect local tests to a db engine, right? :)15:22
TheJuliaI looked at it earlier, I think it is the first time we've tried using json native fields15:22
rlooa new set of patches was just done during this meeting, maybe that'll fix...15:22
jrolllooks like yolanda put up a new patch15:22
TheJuliajroll: oh it is, but most people don't have their local db setup to support that15:22
TheJuliajroll: oh, awesome15:22
jrollsure, but it's possible15:22
jroll... wait, native json fields?15:22
TheJuliayup, 7 minutes after15:22
yolandajroll, mine is just wip, but is a long chain15:22
*** zhangfei has quit IRC15:23
jrollyolanda: ah, I see15:23
TheJuliajroll: yeah, as in it is stored as json instead of text15:23
jrollTheJulia: yes, I know what they are, I didn't think they were in mysql until very very recently15:23
jrollthough, looking at the patch I don't think they are native json fields15:24
jrollwe can loop back to this later btw15:24
TheJuliajroll: I think a couple years, but I had a similar thought earlier15:24
TheJuliaAnyway, does anyone have any other items for the priority list?15:25
rlooTheJulia: please add your storyboard patches to that list15:25
TheJuliathat works15:26
TheJuliaOkay, I'll populate the list after the meeting15:26
rlooworks for me!15:26
*** vmud213 has quit IRC15:26
TheJuliaAnything else? I think we're good as long as we make progress on rescue this week15:26
*** vmud213 has joined #openstack-ironic15:27
rloojust looking at release schedule. rocky-1 milestone is next week. maybe we should aim to do a release *after* gtting rescue done.15:27
* rpioso must leave early for a downstream group lunch15:27
*** rpioso is now known as rpioso|eat15:27
rloo(I'm good with priorities)15:27
* jroll leaves a comment on the bios patch indicating the problem, easy peasy :)15:28
yolandathx for reviews!15:28
TheJuliarloo: Possibly yeah, I guess only time will tell15:28
TheJuliaAnyway, are we good to move on to Open Discussion at this point?15:28
rlooyes!15:29
* jroll is fine15:29
TheJulia#topic Open Discussion15:29
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:29
rloomjturek: bug smash?15:30
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API  https://review.openstack.org/51257915:30
TheJuliaSo lets see, replaying brain from last week. I was at a baremetal provisioning forum last week, and the thought of federated cross organization ironic came up. I have a couple building block ideas that could help there, so I'll try to toss them up this week.15:30
mjturekyep! The doodle is here https://doodle.com/poll/xa999rx653pb58t615:30
mjturekthink a 2 hr window will work?15:31
*** vmud213 has quit IRC15:32
rlooi only see 7 people participated in doodle. that makes me sad.15:32
rloodoes that mean the rest aren't going to participate at all ?15:32
mjturek:(15:32
TheJuliamjturek: I just added myself15:32
TheJulia#link https://doodle.com/poll/xa999rx653pb58t615:32
TheJuliaI think a two hour window will work15:33
TheJuliaSeems like we should just agree on the most common time and move on?15:34
mjturekUTC 1:00 PM to 3:00 PM seems most popular15:34
rloosold!15:34
TheJulia:)15:34
mjturekhoorraaaay!15:34
mjturekwhere are we going to hold this? hangouts?15:35
mjturekor how we do mid-cycle?15:35
rlooit has been so long since we've had one, that i can't recall what/how we did it.15:35
TheJuliamjturek: I think a hangout might be ?easiest?15:35
jrollI'm fine with either - inclusivity would imply we should use infra's SIP bridge15:35
rloodo we all go through the same bugs at the same time?15:36
TheJuliait would, if it has been paid for recently15:36
rloodmitry used bluejean before? or something like that.15:36
TheJuliayeah, I have access to a bluejeans account. We could also use that15:36
TheJuliaI'm okay with any of the above15:36
rloowell, 6 people signed up. maybe more will join us. i think as long as the 6 can get access to whatever technology, i'm good.15:37
jrollright, we should check with them before scheduling, but I expect bluejeans would be okay15:37
jrollor just say "if you can't access this or object to it bla bla" to the email15:37
*** links has quit IRC15:37
TheJuliaI can host in bluejeans I guess15:38
TheJuliaThat will allow people to dial in easily15:38
mjtureksounds good to me15:38
mjturekdetails on the ML?15:38
rlooin that case, just as good as SIP bridge, so lets use bluejeans.15:39
rloomjturek: yup, email please.15:39
TheJulia#action TheJulia to create a bluejeans meeting for the time.15:39
TheJuliamjturek: will you close out the poll, and I'll go ahead and create the meeting15:39
mjturekperfect15:39
TheJuliamjturek: and then send you the details to send an email15:39
mjturekwill do15:39
rlooso we don't spend lots of time during bug bash, deciding how/what/process, might be good for someone mjturek? TheJulia? to propose something before hand?15:40
rlooeg will we all look at each bug at the same time, to try to deal with it. or do we split the bugs and ping each other if tehre are questions about one of them?15:41
mjturekI can propose something today if that's helpful15:41
TheJuliamjturek: that would be awesome15:41
mjturekand obviously we can have some back and forth to get a final shape of the thing15:41
mjturekcool will do15:41
rloothx mjturek!15:41
mjtureknp!15:41
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387915:43
*** srart has joined #openstack-ironic15:43
TheJuliaDo we have anything else to discuss?15:44
rloocrickets...15:44
*** gyee has joined #openstack-ironic15:45
TheJuliaWell, sounds like we're done for the day!15:46
TheJuliaMeeting wise at least15:46
TheJuliaThanks everyone!15:46
*** stendulker has quit IRC15:46
TheJulia#endmeeting15:46
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"15:46
openstackMeeting ended Mon Apr  9 15:46:48 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:46
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-09-15.00.html15:46
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-09-15.00.txt15:46
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-09-15.00.log.html15:46
jrollthanks TheJulia :)15:47
* jroll finds lunch15:47
* kaifeng finds bed ..15:47
rloogood night kaifeng15:47
TheJuliagoodnight kaifeng15:48
kaifengrloo: TheJulia: good night :) leaving15:48
rlooTheJulia: another hiccup with the partprobe change in ironic-lib (https://review.openstack.org/#/c/542035/). partprobe is called in several places, but we do the retry in only one of them.15:48
patchbotpatch 542035 - ironic-lib - Add retry attempts for the partprobe command (MERGED)15:48
*** kaifeng has quit IRC15:48
*** namnh has quit IRC15:49
TheJuliarloo: those are unlikely to fail if I understand where things were going sideways15:50
rlooTheJulia: would be good to comment in that case, or just add it in since if unlikely to fail, it shouldn't hurt to have it in, in case it does fail. cuz 'unlikely' isn't never.15:51
rlooTheJulia: anyway, what is there is better than nothing.15:52
TheJuliarloo: commented15:54
*** hshiina has quit IRC15:54
rlooTheJulia: thx.15:55
*** srart has quit IRC15:55
*** ktibi has quit IRC15:56
*** baha has quit IRC15:56
*** milan_ has joined #openstack-ironic15:57
openstackgerritIlya Etingof proposed openstack/sushy-tools master: added system boot mode support  https://review.openstack.org/52610115:58
TheJuliaI've updated the whiteboard with the list of storyboard patches that I know are outstanding. I need to revise one16:00
*** jcoufal has quit IRC16:04
*** fragatina has joined #openstack-ironic16:09
openstackgerritRuby Loo proposed openstack/ironic master: Add reno for new config [disk_utils]partprobe_attempts  https://review.openstack.org/55977916:09
*** AlexeyAbashkin has joined #openstack-ironic16:18
*** lucasagomes is now known as lucas-afk16:19
*** milan_ has quit IRC16:25
* sambetts --> home 16:26
sambettsnight all o/16:26
rloociao sambetts16:26
*** sambetts is now known as sambetts|afk16:26
sambetts|afko/ rloo16:26
*** serlex has quit IRC16:27
openstackgerritMerged openstack/ironic-inspector stable/queens: Disable automated cleaning in grenade  https://review.openstack.org/55878416:28
openstackgerritMerged openstack/ironic master: Follow the new PTI for document build  https://review.openstack.org/55347316:28
*** milan_ has joined #openstack-ironic16:28
jrollTheJulia: looping back, they aren't native json fields in the bios thing. oslo.db provides a JsonEncodedDict type that does a translation to TEXT. That said, the code doesn't match the spec, and dtantsur|pto isn't here to help figure out why he asked for a json field: https://review.openstack.org/#/c/511162/39/ironic/db/sqlalchemy/models.py@31216:31
patchbotpatch 511162 - ironic - BIOS Settings: Add DB model16:31
TheJuliaheh16:32
jrollwith the structure we have there, I'm not sure why we'd need json16:32
*** srart has joined #openstack-ironic16:32
TheJuliaI was actually kind of thinking the same thing16:32
TheJuliaand was unsure as to why16:33
jrollI also pointed out how the unit tests were broken, if we do use json, so it becomes a choose your own adventure :)16:33
TheJuliaheh16:33
TheJulia\o/16:33
TheJuliaI had a sneaking suspicion of that when I wsa looking at it, but I was jumping from item to item in advance of the meeting16:34
*** baha has joined #openstack-ironic16:35
jlvillaljroll, Have you seen the jmespath library?  https://github.com/jmespath/jmespath.py16:35
jlvillaljroll, I saw you mention JSON :)16:35
jrolljlvillal: because dictionary syntax isn't good enough? :P16:37
jlvillalI'm using it in a little bot that queries the weather from Yahoo.  tomorrow_desc = jmespath.search('item.forecast[1].text', weather_channel)16:37
jrolllooks interesting16:37
jlvillaljroll, Well the nice part is if the search fails. You get None.16:37
jrollright16:37
jrollreminds me of jq which is one of my favorite shell tools ever16:38
*** trown is now known as trown|lunch16:38
*** baha has quit IRC16:39
*** baha has joined #openstack-ironic16:40
jlvillal:)16:41
jlvillalAnd I noticed it is already in global-requirements16:42
TheJuliajroll: I have always seen jq as favorite and infamous at the same time16:42
jrollheh16:43
jrollwhile we're on weird json utilities, this is relatively new but useful http://jgrep.org/16:44
jlvillaljroll, Yeah I saw that. Looks useful too.16:44
*** mikal has quit IRC16:46
*** mikal has joined #openstack-ironic16:47
*** tesseract has quit IRC16:48
*** jgwentworth is now known as melwitt16:51
-openstackstatus- NOTICE: zuul was restarted to update to the latest code; please recheck any changes uploaded within the past 10 minutes16:51
*** krtaylor has joined #openstack-ironic16:54
*** derekh has quit IRC16:59
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387917:02
*** AlexeyAbashkin has quit IRC17:02
*** mgoddard has quit IRC17:03
openstackgerritIlya Etingof proposed openstack/sushy-tools master: added system boot mode support  https://review.openstack.org/52610117:04
*** dougsz has quit IRC17:06
*** AlexeyAbashkin has joined #openstack-ironic17:09
openstackgerritJim Rollenhagen proposed openstack/ironic-specs master: Add conductor/node locality spec  https://review.openstack.org/55942017:20
jrollokay, this is as ready as it's gonna get without reviews ^17:20
jrollit's on our priority list, so would love some early reviews before I start writing too much code :)17:21
*** _milan_ has joined #openstack-ironic17:24
*** _milan_ is now known as milan17:24
*** milan_ has quit IRC17:26
TheJuliathanks jroll17:26
TheJuliaI'll try to look later today17:26
TheJuliain the mean time... I need to go to the doctor. bbiab17:26
jroll\o17:27
*** pbourke_ has quit IRC17:30
* milan is away: I'm busy17:32
*** r-daneel has quit IRC17:33
*** trown|lunch is now known as trown17:37
*** mgoddard has joined #openstack-ironic17:42
*** rpioso|eat is now known as rpioso17:46
*** jcoufal has joined #openstack-ironic17:50
*** fragatina has quit IRC17:53
e0neTheJulia: hi. I'm sorry. I forgot that we've got holiday today. can we move our conversation about horizon plugins testing later this week?17:56
TheJuliae0ne: sure17:58
e0nethanks. I'll be available here almost evetyday. just ping me if I'm online17:58
*** AlexeyAbashkin has quit IRC18:01
clarkbhey I'm debugging http://logs.openstack.org/46/559446/1/check/ironic-grenade-dsvm/202d831/logs/grenade.sh.txt.gz#_2018-04-07_00_30_27_418 to try and make sure that my change isn't at fault there (have also rechecked it for more data) and not really seeing where ironic does much with that host? it goes from enroll to verifying but target state is I guess manageable? is it just slow?18:17
jrollclarkb: enroll -> manage basically just checks some db attributes and ensures it can control power18:19
jrollwe do power control over ipmi via virtualbmc (ipmi server that translates to libvirt calls)18:19
jrollI suspect virtualbmc is broken here18:19
jroll(or not running or whatever)18:20
clarkbok, change made is to move that stuff into virtualenvs, is virtualbmc a python tool? chances are its broken because whatever is running that uses the wrong path if so18:21
clarkbbut that gives me a bread crumb to follow, thanks18:21
jrollyeah, it's python18:21
jrollclarkb: virtualbmc log files are typically in logs/old/ironic-bm-logs/virtualbmc.txt.gz, which your run doesn't seem to have, fwiw18:22
clarkbjroll: so it runs as a proper service?18:23
jrollclarkb: not sure offhand, poking around our devstack plugin right now but I don't see where it runs immediately18:24
clarkbjroll: ya I don't see it either :). The setup in there should be fine bceause it uses the devstack setup tooling which will put it in the virtualenv udner my change18:25
jrollyep18:25
jrolljust trying to find the execution18:25
*** srart has quit IRC18:26
jrollclarkb: aha https://github.com/openstack/ironic/blob/70a86cd8b591c7ed31151b622bf79a9e2886c1f4/devstack/tools/ironic/scripts/create-node.sh#L13418:27
jrollalso called here: https://github.com/openstack/ironic/blob/70a86cd8b591c7ed31151b622bf79a9e2886c1f4/devstack/tools/ironic/scripts/cleanup-node.sh#L2618:27
jrollthat looks like it18:27
clarkbya thats the problem that script is called with sudo which will break my PATH munging18:28
jrollyeesh18:28
*** AlexeyAbashkin has joined #openstack-ironic18:28
clarkbhrm though it does do sudo -E but then calls su from that which maybe breaks it?18:29
jrollsudo -u $STACK_USER should work though18:29
clarkbya18:30
clarkbif keeping the -E18:30
* clarkb checks if su breaks env18:30
jrollright, which cleanup-node needs added18:30
clarkbya I think you need to tell su to preserve env too18:30
clarkbI'll work on a patch and depends on it to see if that fixes it18:31
jrollawesome, thanks :)18:31
clarkbit will have to go against queens too I think18:32
jrollyeah, I'm happy to handle getting it landed and backported18:32
openstackgerritClark Boylan proposed openstack/ironic master: Preserve env when running vbmc  https://review.openstack.org/55981118:37
clarkbwe'll start with ^ as a sanity check just to make sure it function and if so I'll propose the backport(s)18:37
jroll++18:38
jrollclarkb: we'll need it on the cleanup-node call, I'll update it quickly18:38
jrollactually.18:39
jrollisn't this already running as $STACK_USER?18:39
clarkbjroll: probably?18:40
clarkbI'd ahve to look at logs to see18:40
jrollheh18:40
*** srart has joined #openstack-ironic18:40
jrollI mean, devstack as a whole is supposed to be run as $STACK_USER18:40
jrollI guess we can tackle that later18:42
openstackgerritJim Rollenhagen proposed openstack/ironic master: Preserve env when running vbmc  https://review.openstack.org/55981118:42
jrollclarkb: updated to also catch cleanup-node ^18:42
*** mgoddard has quit IRC18:44
*** Goneri has quit IRC18:52
*** bl0m1_ has quit IRC18:59
*** AlexeyAbashkin has quit IRC19:09
*** bl0m1 has joined #openstack-ironic19:14
*** jjohnson2 has quit IRC19:18
openstackgerritJulia Kreger proposed openstack/ironic master: Update Launchpad references to Storyboard  https://review.openstack.org/55661819:37
clarkbjroll: http://logs.openstack.org/11/559811/2/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/17ce19e/logs/devstacklog.txt.gz#_2018-04-09_19_15_57_868 is curious, I'm beginning to wonder if the su is some sort of workaround19:43
clarkbthe file is +x for everyone19:46
clarkbI bet this is a rootwrap related problem. We clear out the ability to generally sudo19:47
clarkbbut that confirms we are sudoing to user stack from user stack19:48
* clarkb starts by cleaning that up19:48
*** jtomasek has quit IRC19:49
openstackgerritClark Boylan proposed openstack/ironic master: Preserve env when running vbmc  https://review.openstack.org/55981119:50
clarkbjroll: ^ lets see if that is any happier19:50
jrollclarkb: huh, interesting. glad we confirmed it :)20:06
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387920:07
openstackgerritIlya Etingof proposed openstack/sushy-tools master: added system boot mode support  https://review.openstack.org/52610120:09
TheJuliajroll: your spec looks awesome20:10
jroll\o/ thanks20:10
jrollthe notes were missing some pieces I needed, but I think I recalled everything correctly20:11
*** jjohnson2 has joined #openstack-ironic20:11
*** jjohnson2_ has joined #openstack-ironic20:15
*** jjohnson2 has quit IRC20:19
openstackgerritJim Rollenhagen proposed openstack/ironic-specs master: Add conductor/node locality spec  https://review.openstack.org/55942020:19
jrollTheJulia: fixed the things if you want a quick re-review :)20:19
TheJuliaI'll try again in a little bit, in the middle of another spec at the moment (and I think I ought to write two today...)20:20
jrollno worries20:20
*** MattMan has quit IRC20:24
*** MattMan has joined #openstack-ironic20:25
*** dprince has quit IRC20:27
*** hw_wutianwei1 has joined #openstack-ironic20:32
clarkbjroll: http://logs.openstack.org/11/559811/3/check/ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa/d6e5e59/logs/devstacklog.txt.gz#_2018-04-09_20_14_24_592 progress though this particular issue may be why it was a sudo su all along20:34
*** hw_wutianwei has quit IRC20:34
*** hw_wutianwei1 is now known as hw_wutianwei20:34
*** pcaruana has quit IRC20:36
jrollclarkb: hrm, doesn't seem like su'ing to the stack user would do that20:37
jrollis your path munging available as a function?20:37
clarkbjroll: it is and it should be idempotent20:37
jrollclarkb: might be worth just calling that from the scripts being called then20:38
clarkbmight be, except that now I want ot understand how this works at all :)20:39
clarkbthe stack user does get added to the libvirt group specifically so that this stuff will work iirc20:39
clarkband these tests use nova which is what I think triggers that in devstack20:39
jrollah20:40
jrollthis might be pre-nova20:40
clarkbmight also not be regrouping itself20:41
clarkbsince in the nova case it gets run out of systemd it gets a new login there20:41
clarkboh you know what20:41
clarkbsu may be doing that new login20:41
clarkbhrm not with the passed arguments20:42
clarkbmaybe it is sufficient to regroup though20:42
jrollaha20:42
clarkbya I think that is it20:43
jrollthat would make sense20:43
clarkblet me try a patch that passes the env through su as well using -p20:44
clarkband maybe leave a comment for why I think the su is helpful20:44
mmethotTheJulia, btw I made phrasing change you had requested20:47
TheJuliammethot: thanks20:47
TheJuliajroll: reviewing20:47
*** fragatina has joined #openstack-ironic20:47
TheJuliajroll: done20:48
*** Goneri has joined #openstack-ironic20:48
*** fragatina has quit IRC20:48
jrollTheJulia: should your comment on line 105 be up on 97, I assume?20:49
*** fragatina has joined #openstack-ironic20:49
* TheJulia pulls it back up20:49
jrollI just want to make sure I'm not missing some context :)20:50
jrollat any rate, I have nothing to do here, so thanks!20:50
*** trown is now known as trown|outtypewww20:50
TheJuliajroll: nope, your not missing anything20:50
jrollok, cool20:51
openstackgerritClark Boylan proposed openstack/ironic master: Preserve env when running vbmc  https://review.openstack.org/55981120:52
*** r-daneel has joined #openstack-ironic21:00
*** harlowja has joined #openstack-ironic21:03
*** r-daneel has quit IRC21:15
*** baha has quit IRC21:16
NobodyCamtotally missed my "Good Morning today.", So instead:21:16
NobodyCamGood Afternoon Ironic'ers21:16
*** r-daneel has joined #openstack-ironic21:16
TheJuliaGood Afternoon NobodyCam21:29
NobodyCam:)21:39
*** milan has quit IRC21:54
*** jjohnson2_ has quit IRC21:55
*** ianychoi has quit IRC22:10
*** bfournie has quit IRC22:11
*** ianychoi has joined #openstack-ironic22:11
*** rcernin has joined #openstack-ironic22:14
*** r-daneel has quit IRC22:15
*** r-daneel has joined #openstack-ironic22:16
*** rpioso is now known as rpioso|afk22:24
*** jcoufal has quit IRC22:26
*** r-daneel has quit IRC22:32
*** r-daneel has joined #openstack-ironic22:32
openstackgerritMerged openstack/sushy-tools master: Fix a typo in libvirt driver  https://review.openstack.org/55532122:33
openstackgerritClark Boylan proposed openstack/ironic stable/queens: Preserve env when running vbmc  https://review.openstack.org/55985622:34
clarkbjroll: ^ things were looking good so I pushed a backport and https://review.openstack.org/559446 is updated with new depends on to test it22:35
patchbotpatch 559446 - openstack-dev/grenade - Support devstack's global virtualenvs22:35
clarkbjroll: I am going to WIP things as I am not suer this is actually going to be our long term approach but want to get it written down as an option that we can consider22:35
openstackgerritJulia Kreger proposed openstack/ironic master: Remove elilo support  https://review.openstack.org/53315322:37
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Added new spec for L3 based Ironic deployment  https://review.openstack.org/54393622:48
jrollclarkb: nice, thanks! left a +1 for now22:49
*** Goneri has quit IRC22:57
*** patriciadomin has quit IRC23:04
*** e0ne has quit IRC23:04
*** patriciadomin has joined #openstack-ironic23:05
*** r-daneel has quit IRC23:07
*** lbragstad has quit IRC23:17
* TheJulia goes on a trek for dinner and socialization23:34
*** lbragstad has joined #openstack-ironic23:36
*** cjloader has quit IRC23:39
*** lbragstad has quit IRC23:47
*** exodusftw has quit IRC23:58
« Sunday, 2018-04-08 Index Tuesday, 2018-04-10 »

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