Monday, 2019-06-17

*** ianychoi has quit IRC02:25
*** ianychoi has joined #openstack-ironic02:26
*** yedongcan has joined #openstack-ironic02:41
*** rayzhang has joined #openstack-ironic02:43
rayzhangGood morning guys.02:45
chihhsinGood morning Ironic02:46
openstackgerritMerged openstack/ironic master: Update qemu hook to facilitate Multicast  https://review.opendev.org/66522102:54
openstackgerritMerged openstack/ironic master: Remove ipxe tags when ipx6 is in use.  https://review.opendev.org/66526702:56
*** whoami-rajat has joined #openstack-ironic03:12
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent-builder master: Sync package requirements from IPA repo  https://review.opendev.org/66557503:50
openstackgerritRichard G. Pioso proposed openstack/ironic-specs master: Add spec for defining idrac support of Redfish  https://review.opendev.org/66557603:56
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent-builder master: [PoC] Enable creating pdf doc  https://review.opendev.org/66557703:58
*** jtomasek has joined #openstack-ironic05:48
openstackgerritMerged openstack/ironic master: Updates ironic for using ironic-python-agent-builder  https://review.opendev.org/66539905:59
*** rcernin has quit IRC06:02
*** mkrai_ has joined #openstack-ironic06:10
*** mkowalski has joined #openstack-ironic06:15
arne_wiebalckgood morning, ironic!06:38
openstackgerritMadhuri Kumari proposed openstack/ironic master: Add IntelIPMIHardware  https://review.opendev.org/65955006:45
*** gregoryo has joined #openstack-ironic06:52
*** rpittau|afk is now known as rpittau06:56
rpittaugood morning ironic! o/06:56
*** rcernin has joined #openstack-ironic07:00
*** trident has quit IRC07:06
*** trident has joined #openstack-ironic07:08
*** tesseract has joined #openstack-ironic07:24
*** iurygregory has joined #openstack-ironic07:26
*** jtomasek has quit IRC07:28
*** jtomasek has joined #openstack-ironic07:28
iurygregorymorning ironicers!07:29
*** dtantsur|afk is now known as dtantsur07:30
dtantsurmorning ironic07:30
dtantsurrpittau: please check https://review.opendev.org/#/c/665575/07:30
patchbotpatch 665575 - ironic-python-agent-builder - Sync package requirements from IPA repo - 1 patch set07:30
iurygregorydtantsur, morning07:30
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector stable/stein: Stop failing on missing memory or CPU  https://review.opendev.org/66560107:32
*** yedongcan has quit IRC07:33
*** yedongcan has joined #openstack-ironic07:34
rpittaudtantsur: good morning! That looks good to me, I was just wondering about testing Jobs in ipa-B, or lack of them :)07:40
dtantsurrpittau: yeah, we kind of have chicked and egg problem here..07:40
rpittauI'm fine with merging that as it's a trivial change, and I saw there is a task to create jobs, so we should be fine if that is given priority07:46
*** lucasagomes has joined #openstack-ironic07:53
openstackgerritM V P Nitesh proposed openstack/ironic-lib master: Returns disk_identifier for UEFI boot mode nodes  https://review.opendev.org/66485207:54
openstackgerritMerged openstack/ironic-python-agent-builder master: Sync package requirements from IPA repo  https://review.opendev.org/66557508:02
mkraigood morning ironic o/08:04
*** dougsz has joined #openstack-ironic08:08
*** ricolin has joined #openstack-ironic08:12
*** gregoryo has quit IRC08:15
*** hjensas|afk is now known as hjensas08:34
*** derekh has joined #openstack-ironic08:37
*** tssurya has joined #openstack-ironic08:44
*** pcaruana has joined #openstack-ironic08:52
*** tssurya has quit IRC08:55
*** xXraphXx has joined #openstack-ironic09:12
*** ricolin has quit IRC09:14
xXraphXxhello :), question: I was wondering why we don't monkey_patch the os lib to make it greenthread compliant along with other ones ? eventlet.monkey_patch(os=False) in ironic.cmd09:19
*** mkowalski has quit IRC09:20
*** mkowalski has joined #openstack-ironic09:20
xXraphXxI know we are not supposed to call os.system directly (rather use oslo.processutils.execute if needed), but if some drivers mistakenly make os.system, this will likely trigger big locking troubles right ?09:21
xXraphXx* s/make/call/09:21
openstackgerritM V P Nitesh proposed openstack/ironic master: UEFI PXE Provisioning for wholedisk images  https://review.opendev.org/66486409:23
*** rcernin has quit IRC09:36
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Stop logging lshw output, collect it with other logs instead  https://review.opendev.org/66562809:42
dtantsurxXraphXx: hi! honestly, I don't know. I'm not sure anybody can remember that..09:43
xXraphXxdtantsur, ok thanks for your reply.09:46
arne_wiebalckWe're about to add our burn-in & benchmarks to ironic and we're discussing the best way to integrate them: inspector (usually run once, data transport for free) vs clean steps (more flexibility to re-run individual tests).09:49
arne_wiebalckSo I thought both maybe: have a module with the functionality which is exposed via clean steps in our custom h/w manager and via a collector in the inspector. Does that sound  like a sensible approach?09:49
dtantsurarne_wiebalck: yep, why not09:50
arne_wiebalckdtantsur: thx, just wanted someone to confirm this is not an obvious waste of time ;)09:51
dtantsurarne_wiebalck: FYI we have some benchmarks in the extra-hardware collector09:51
arne_wiebalckdtantsur: yes, saw it09:52
dtantsurcool09:52
arne_wiebalckdtantsur: we have some additional benchmarks to run (plus burn-in)09:52
arne_wiebalckdtantsur: will use this as a template, though09:53
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add check on get_endpoint returning None  https://review.opendev.org/66563209:53
*** ociuhandu has joined #openstack-ironic09:57
openstackgerritDmitry Tantsur proposed openstack/ironic master: devstack: add missing variables for ironic-python-agent-builder  https://review.opendev.org/66563510:05
dtantsurgate fix for IPA and ironic-lib ^^10:05
dtantsuralso FYI nishagb ^^^10:05
dtantsursorry for missing it on review, I did not realize we had variables in another project..10:05
rpittauforgot about that too :/10:06
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: [DNM] Testing the CI  https://review.opendev.org/66563610:06
dtantsurtesting ^^10:06
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Remove deprecated authentication options for accessing ironic  https://review.opendev.org/58704110:41
*** stendulker has joined #openstack-ironic10:46
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Remove deprecated authentication options for accessing ironic  https://review.opendev.org/58704110:48
*** mkrai_ has quit IRC10:50
*** ociuhandu has quit IRC11:17
*** stendulker has quit IRC11:26
*** bfournie has quit IRC11:31
dtantsurrpittau: https://review.opendev.org/#/c/665635/ seems to work according to https://review.opendev.org/#/c/665636/11:36
patchbotpatch 665635 - ironic - devstack: add missing variables for ironic-python-... - 1 patch set11:36
patchbotpatch 665636 - ironic-lib - [DNM] Testing the CI - 1 patch set11:36
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Collect sensor data in ``redfish`` hardware type  https://review.opendev.org/66564611:41
*** rh-jelabarre has joined #openstack-ironic11:49
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Stop logging lshw output, collect it with other logs instead  https://review.opendev.org/66562812:00
*** edleafe has joined #openstack-ironic12:02
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Upgraded RAID delete_config cleaning step  https://review.opendev.org/66463312:08
*** sziviani has quit IRC12:13
*** sziviani has joined #openstack-ironic12:17
*** bfournie has joined #openstack-ironic12:29
*** etingof is now known as etingof|brb12:39
*** lennyb has quit IRC12:40
openstackgerritDongcan Ye proposed openstack/networking-baremetal master: Update networking-baremetal installation  https://review.opendev.org/66550812:44
*** ricolin has joined #openstack-ironic12:45
*** yedongcan has quit IRC12:50
TheJuliagood morning13:06
dtantsurmorning TheJulia13:06
* TheJulia goes and makes coffee13:08
rpittauhi TheJulia :)13:09
*** rachit7 has joined #openstack-ironic13:09
*** yedongcan has joined #openstack-ironic13:11
*** lennyb has joined #openstack-ironic13:13
*** ricolin has quit IRC13:17
*** rloo has joined #openstack-ironic13:18
TheJuliaarne_wiebalck: there have definitely been people asking about burnin somewhere or near cleaning, fwiw13:18
iurygregoryTheJulia, good morning o/13:18
*** beekneemech is now known as bnemec13:18
*** sthussey has joined #openstack-ironic13:20
*** mbeierl has joined #openstack-ironic13:20
*** Goneri has joined #openstack-ironic13:20
TheJuliadtantsur: by chance did you have a minute to look at https://storyboard.openstack.org/#!/story/2005896 ?13:21
dtantsurTheJulia: I have it open here, just did not get to. will get by today.13:21
dtantsurTheJulia: btw, do we have a normal meeting today?13:21
TheJuliaI don't think anyone replied to myemail :(13:22
TheJuliaCorrection, you did! :)13:22
dtantsur:)13:23
arne_wiebalckTheJulia: Yeah, we discussed this a year ago or so.13:26
*** pcaruana|afk| has joined #openstack-ironic13:26
*** pcaruana has quit IRC13:26
arne_wiebalckTheJulia: Burn-in may be generic enough to go upstream into the inspector, just as the benchmark that is there atm, as it may use std tools like badblocks, cpuburn, and the like.13:27
TheJuliaCool13:34
*** iurygregory_ has joined #openstack-ironic13:40
*** ricolin has joined #openstack-ironic13:40
*** iurygregory has quit IRC13:42
openstackgerritchenker proposed openstack/python-ironicclient master: Switch to the new canonical constraints URL on master  https://review.opendev.org/66566913:42
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: Remove ipxe tags when ipxe6 is in use.  https://review.opendev.org/66567213:46
*** etingof|brb is now known as etingof13:47
*** pcaruana has joined #openstack-ironic13:48
openstackgerritchenker proposed openstack/ironic-tempest-plugin master: Switch to the new canonical constraints URL on master  https://review.opendev.org/66567413:50
*** pcaruana|afk| has quit IRC13:51
TheJuliaI feel an ironic 12.2 coming on13:53
dtantsur:)13:54
dtantsurTheJulia: btw, IPA and ironic-lib CI need https://review.opendev.org/#/c/665635/13:55
patchbotpatch 665635 - ironic - devstack: add missing variables for ironic-python-... - 1 patch set13:55
TheJuliale sigh13:56
dtantsuryup :(13:56
TheJuliaapproved, hopefully that ci failure was just load13:57
TheJuliaMy current though is if we can release 12.2, then we can go ahead and start merging the major changes for the cycle like removing elilo :)13:58
TheJuliasince that forces us to 1213:58
TheJuliaerr, 1313:58
*** jcoufal has joined #openstack-ironic14:02
* TheJulia goes and exercises14:03
dtantsuryep14:04
*** mkrai_ has joined #openstack-ironic14:05
openstackgerritchenker proposed openstack/ironic-tempest-plugin master: Switch to the new canonical constraints URL on master  https://review.opendev.org/66567414:07
openstackgerritMerged openstack/sushy-tools master: Rename `drivers` to `resources/systems`  https://review.opendev.org/63849214:11
*** mkowalski has quit IRC14:11
*** e0ne has joined #openstack-ironic14:21
*** e0ne has quit IRC14:26
openstackgerritMerged openstack/sushy master: Enhance Storage models to support RAID config  https://review.opendev.org/65768814:27
*** rachit7 has quit IRC14:35
*** e0ne has joined #openstack-ironic14:36
*** rachit7 has joined #openstack-ironic14:44
*** pcaruana has quit IRC14:47
*** cdearborn has joined #openstack-ironic14:48
*** kaifeng has joined #openstack-ironic14:50
* TheJulia feels so much better after exercising14:54
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Jun 17 15:00:23 2019 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
TheJuliao/15:00
openstackThe meeting name has been set to 'ironic'15:00
cdearborn\o15:00
rpittauo/15:00
bdoddo/15:00
kaifengo/15:00
mgoddard\o15:00
etingof\o15:00
arne_wiebalcko/15:00
jungleboyjo/15:01
TheJuliaGood Morning everyone!15:01
rpioso\o15:01
TheJuliaToday's agenda can be found on the wiki page.15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
rlooo/15:01
TheJuliaI don't have much, so hopefully this will proceed quickly15:01
TheJulia*hopes*15:01
TheJulia#topic Announcements/Reminders15:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:01
jroll\o15:02
bahao/15:02
TheJuliaI'm hoping to cut a release of ironic 12.2 today or tomorrow. Afterwards we should be able to go ahead and begin merging the more breaking changes for the cycle.15:02
arne_wiebalckTheJulia: Do you need the s/w RAID docs for this?15:03
TheJuliaDoes anyone have anything else to remind us about today?15:03
TheJuliaarne_wiebalck: to release, no. It would be nice though to get something into master branch soon-ish15:03
mgoddardTheJulia: is that something I can help with? I did offer to help with releases at the PTG, although I've not been much help so far15:03
TheJuliamgoddard: it is!15:03
arne_wiebalckTheJulia: ack15:04
TheJuliaAlthough I think at this point it would just be picking the hash number and putting the commit up for review in openstack/releases15:04
*** dnuka has joined #openstack-ironic15:04
mgoddardTheJulia: great. Just ironic or other repos too?15:04
dnukao/15:04
TheJuliaI think dtantsur was only thinking https://review.opendev.org/#/c/665635/ needed to merge first. I think15:04
patchbotpatch 665635 - ironic - devstack: add missing variables for ironic-python-... - 1 patch set15:04
TheJuliamgoddard: just ironic, dtantsur already cut some releases for libs and other programs so far this cycle15:05
mgoddardok15:05
TheJuliaWell if nobody has anything to announce or remind us of, we can carry on.15:05
TheJulia#topic Review action items from previous meeting15:06
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:06
TheJuliaWe had one action item last week which was to get a release note update to indicate that smartnic functionality is available with a Train Neutron.15:06
TheJuliaI proposed that and it merged so \o/15:07
TheJuliaMoving on15:07
TheJulia#topic Review subteam status reports15:07
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:07
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:07
* TheJulia skims down the list15:08
TheJuliamkrai: thanks for the update :)15:08
dtantsuryep, I did some releases, only ironic (I think) is pending15:08
*** mkrai_ has quit IRC15:09
rloodtantsur: did you see the question from kaifeng wrt bugs (L233)15:09
TheJuliaarne_wiebalck: Any continuation on the retirement discussions?15:09
arne_wiebalcknot yet, will pick this up again15:09
dtantsurkaifeng: the data is pulled from https://storyboard.openstack.org/#!/board/8315:10
* dtantsur did not have time to update it this time - sorry15:10
kaifengoh, I am not sure how to triage a rfe15:10
dtantsurkaifeng: use "rfe" tag (in addition to "ironic-triaged")15:11
dtantsuras always with triaging, make sure it makes any sense :)15:11
*** stendulker has joined #openstack-ironic15:11
dtantsurif some RFE is ready for discussion, feel free to raise it on the next meeting15:11
kaifengso the ironic-triaged tag is a mandatory tag15:11
TheJuliadtantsur: any update on mdns stuffs this week?15:11
dtantsurTheJulia: we're largely done for now. The next patches will come when/if we get IPv6 support in python-zeroconf.15:12
kaifengdtantsur got it, thanks!15:12
TheJuliakaifeng: I think I saw you proposed a few patches for the single doc generation work?15:13
dtantsurFYI, I'm here till Wednesday, then I have a short PTO (for my German exam), then I'm on an onsite meeting for a week, then I'm on PTO for a week15:13
kaifengTheJulia, right, some repos with simple docs15:13
kaifengI tried ironic, inspector, IPA, they have issues currently15:14
TheJuliakaifeng: if you could keep a running list, I think that would help us tracking wise, or if you already have a list :)15:14
TheJuliayay!15:14
TheJuliaOkay15:14
*** trandles has joined #openstack-ironic15:14
*** rajinir has joined #openstack-ironic15:14
kaifengsure, I will update in the whiteboard, since riccardo is also working on it15:14
rpittaupdf docs generation in ironic would work only converting svg to png, and the final result is very bad :/15:14
TheJuliaOne thing worth noting, rpittau has been kind enough to hunt deprecations for Python 3.8. He created an etherpad and listed it under the python3 second on the whiteboard.15:15
TheJuliarpittau: how bad? unusable bad?15:15
rpittauTheJulia: unreadable in some sections, format is completely off15:15
TheJuliahttps://media1.giphy.com/media/g9OLSjrWHJQRi/giphy.webp?cid=790b76115d07aeb07944554663d2e870&rid=giphy.webp15:16
rlooif it is bad for ironic, wouldn't it be a similar situation for other projects' docs? or maybe ours is more sophisticated...15:16
rpittaulol that's exactly my expression15:16
TheJuliaOurs are a bit more complex15:16
kaifengkind of compromise between typesetting and html I guess15:16
rpittaurloo: we're kind of early for that, I didn't see other projects doing it15:16
rpittauso far15:17
rloorpittau: i'm of the 'wait til closer to the end when others have gotten most bugs out of the system' person15:17
TheJuliaBetter to just get it out of the way sooner and get it into the normal review queues than rush during milestone 315:17
rpittaurloo: I was hoping for that too :D15:17
rloorpittau: maybe reach out to the person that is coordinating the pdf thing?15:17
TheJuliaI'm good with updates, are we good to proceed once the current docs discussion wraps up?15:17
kaifengrloo: this is a smart move :)15:17
rpittaurloo: can do that15:17
arne_wiebalcktssurya put up a first patch set for the power state sync with some explicit questions that came up, if anyone wants to have a look (thanks for your feedback, jroll!)15:17
*** bonguardo has joined #openstack-ironic15:18
rlooTheJulia: one quick question. L 37515:18
rlooTheJulia: do we need that? it is from jan?15:18
rpittaufor the python 3.8 hunt please just be aware of any deprecation message when running tox15:18
TheJuliarloo: it is contextual15:18
rpittauespecially for 'collections'15:18
TheJuliaShould likely be re-written since it is still true context although I think the focus rihgt now is power15:18
* TheJulia is lost on words at the moment and thinks she can come back to it after the meeting15:19
rlooTheJulia: ok, just asking cuz it isn't clear. for those in the audience that don't understand. but then, i'm the only one asking so maybe it is clear to others :D15:19
*** e0ne has quit IRC15:20
rlooproceed ++15:21
TheJulia#topic Deciding on priorities for the coming week15:21
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:21
TheJuliaI already glanced at outstanding patches and didn't see anything that really caught my eye outstanding chagne wise, I think we should just remove the merged patches and carry forward the priorities for this next week. That is unless someone has ones to add?15:22
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:22
TheJuliaLine 12415:22
rpiosoDo we have room for a vendor priority?15:22
TheJuliasure15:23
rpiosoTheJulia: Reviews of https://review.opendev.org/#/c/665576/ would be much appreciated.15:24
patchbotpatch 665576 - ironic-specs - Add spec for defining idrac support of Redfish - 1 patch set15:24
*** yedongcan has quit IRC15:24
dtantsurrpioso: there is a section for vendor priorities in the whiteboard15:24
rpiosodtantsur: It's there :-)15:24
TheJuliaIt is also a spec....15:24
dtantsurI could totally use some eyes on https://review.opendev.org/#/c/665632/15:25
patchbotpatch 665632 - ironic - Add check on get_endpoint returning None - 1 patch set15:25
dtantsuroverall, I'm starting clean up of keystone related items. My ideal goal is to move from *client to openstacksdk.15:25
TheJuliagood plan and ugh, yeah I forgot get_endpoint does that15:26
*** ashtempl_ has joined #openstack-ironic15:26
rpiosoTheJulia: Thank you.15:27
TheJuliaAnything else for priorities, or are we good to move on?15:27
rpittaulet's15:28
rloo++ move15:28
TheJulia#topic Discussion15:29
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:29
ashtempl_Sorry for beeing late15:29
TheJuliaWe have one item, and largely it is not in announcements in case anyone wants to discuss it here instead of on the mailing list.... but it is in regards to the topic of changing the format of our meetings.15:29
ashtempl_Did you talked about pathes to review already?15:29
TheJuliaashtempl_: no worries :)15:29
ashtempl_thanks :)15:29
*** trandles has quit IRC15:30
TheJuliaashtempl_: yes, but if there is one that needs to be added, we can look at it15:30
ashtempl_yep, i have two actually15:30
ashtempl_https://review.opendev.org/#/c/650839/15:30
patchbotpatch 650839 - ironic-tempest-plugin - Single tenant tests: BM to BM and BM to VM - 7 patch sets15:30
*** iurygregory_ is now known as iurygregory15:30
TheJuliaAnyway, going back to the discussion topic. I sent out an email to openstack-discuss summarizing an idea.15:30
TheJulia#link http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007038.html15:30
TheJuliaDmitry has been the only person to reply so far and I'm hoping for other people to reply.15:31
rlooi was hoping to get feedback from the folks that cannot make this meeting. hard to decide w/o their input.15:32
arne_wiebalckrloo ++15:32
rlooif they don't like the proposal, i don't see why we'd do it, unless we want to try it out.15:32
jrollI've been trying to reply for a week and I can't find the right words :/15:32
rlooi'm happy with status quo15:32
ashtempl_The other one is: https://review.opendev.org/#/c/638932/15:32
patchbotpatch 638932 - ironic-tempest-plugin - Advanced tests for Networking Ansible feature - 9 patch sets15:32
rloobut also happy to do something that works for others :)15:32
TheJuliajroll: Perhaps beer will bring them forth?15:33
jrollTheJulia: I have drank plenty of beer in that week :P15:33
TheJuliaoh well, it was a thought.15:33
ashtempl_This one is not running by default as you (Julia) and Dmitry have asked.15:33
rlooTheJulia: who/how many people wanted a change? I already forgot. Can someone reach out to them directly, or should we assume that we've communicated and ... ?15:33
TheJuliaAnyway, I concur with rloo, we need their feedback15:33
rlooi can reply to your email thread, saying basically ^^ :)15:34
TheJuliaI think I'll be meeting in-person next week, and I think the other I can just email15:34
jrollTheJulia: in short, I don't think "Additional updates, questions, and topical discussion" should wait for a "mailing list meeting" - people should just send the email :)15:34
jrollbut I'm willing to try the thing15:34
rlooTheJulia: ok thx. if you want to reach out to them, this is extra work on your part. would like to see a bit more from their part.15:34
kaifengI think someone created a doodle, and then silent since then15:35
TheJuliajroll: +++++++++++++++15:35
TheJuliajroll: say that? :)15:35
TheJuliakaifeng: yeah, I'll reach out to them directly15:35
rlookaifeng: the 'silence' is this discussion and past discussion in this meeting time.15:35
jrollTheJulia: it's so much easier in IRC :) I will attempt to write some words today15:35
TheJuliaOkay, I just wanted to bring this back up, and I think we can move on15:35
TheJulia<315:35
TheJuliaashtempl_: I'll circle back to your patches once we're through our agenda15:36
TheJulia#topic Baremetal SIG15:36
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:36
TheJuliahogepodge o/15:36
TheJuliaLooks like he is AFK15:37
TheJuliaclient idle for 37 minutes15:37
hogepodgehi15:37
TheJuliaoh hai!15:37
*** bobmel has quit IRC15:37
hogepodgeI have to compose the email to the mailing list talking about the whitepaper and case studies, and coordinate with the rest of the marketing team on it.15:37
TheJuliaAnything to discuss or bring up regarding the baremetal sig this week?15:37
hogepodgeThat is to say, no movement since last week, but it's on my mind and on my todo list before I head to Shanghai on Saturday.15:38
*** tesseract has quit IRC15:38
hogepodgeWon't be here next Monday because of KubeCon China.15:38
TheJuliaOkay!15:38
TheJuliaSpeaking of next monday, I'll be on a metal tube to Brno15:38
TheJuliadtantsur: can you run the meeting next week?15:38
* TheJulia suspects dtantsur will be on a train15:38
TheJuliaa train train15:39
jungleboyj:-)15:39
* TheJulia giggles and thinks it is time to move on15:39
dtantsurTheJulia: I'm afraid I cannot :)15:39
dtantsuryes, train train15:39
TheJuliaAny other volunteers for next week? We can always cancel the meeting... or change our meeting format! #notsubtlehintstodiscussiontopic15:39
TheJuliaMoving on to RFE review...15:40
TheJuliahogepodge: btw, thanks!15:40
TheJulia#topic RFE review15:40
*** openstack changes topic to "RFE review (Meeting topic: ironic)"15:40
*** pcaruana has joined #openstack-ironic15:40
TheJuliaI posted one potential RFE last week for some very simple enhancements to ironic-inspector and ipa to enable updating introspection data after deployment which may be performed by an operator as part of their operational processes.15:41
TheJulia#link https://storyboard.openstack.org/#!/story/200585315:41
TheJuliaThe idea is to keep it super simple and provide another entrypoint that can be used in case people want to do it manually.15:41
* arne_wiebalck thinks the inspector (backend) is becoming a CMDB 15:42
kaifengI saw this one, but not clear the use case15:42
*** rachit7 has quit IRC15:42
kaifenga manual re-inspect?15:42
TheJuliaAll the code is basically there, it is a conditional, a copy of ipa's main() with the API start removed15:42
TheJuliakaifeng: basically yes15:42
dtantsurmy primary concern is creating/updating ports. this may have unpredictable effect on a running node15:42
rloois there a new API to do this?15:43
TheJuliarloo: no15:43
kaifengindeed, the trick is inspector will update node15:43
TheJuliadtantsur: don't the ports block deletes if there is still a vif?15:43
TheJuliakaifeng: fwiw, the use case is non-openstack where we want to be able to spot hardware changes via introspection data.15:44
dtantsurTheJulia: I think, yes. I guess we should include in in RFE15:44
dtantsurTheJulia: I can imagine TripleO people interested as well15:44
TheJuliadtantsur: I can double check the port code today most likely15:44
TheJuliadtantsur: yeah15:44
dtantsurIt's probably useful for any managed infrastructure (as opposed to baremetal-as-a-service) case15:45
rlooif someone looks at the introspection data, will they be able to distinguish whether it came from nodes that weren't introspected vs nodes that were? (dunno if that distinction is needed, just wondering)15:45
kaifengdo we need to update node to be spotted or just hold in the introspection data backend?15:45
TheJuliadtantsur: yeah, very.15:46
TheJuliarloo: No, this is much more about gathering the data for other things to be able to use the data15:46
rloosorry to ask cuz it isn't clear to me. is this to update data of nodes that had been introspected before. OR update data of any nodes?15:47
TheJuliakaifeng: that is good question.... I think just hold but it is worth pondering....15:47
kaifengrloo it appears the latter one15:47
*** ianychoi_ has joined #openstack-ironic15:48
TheJuliaThe way inspector works, somewhat either because IPA always posts the data anyway. Ironic-inspector may do nothing with it today or it might add a newly discovered node.15:48
*** gyee has joined #openstack-ironic15:49
*** iurygregory has quit IRC15:49
*** gyee has quit IRC15:49
dtantsurTheJulia: we may want to disable discovery in this mode. Warranting a new API flag?15:49
TheJuliawhy would we want to?15:49
TheJuliaThat is a separate piece of functionality15:49
TheJuliamaybe we might need to extend the BMC address match code in inspector at worst15:50
TheJuliabut that would just be a bug fix really15:50
dtantsurimagine this feature is used from a node that is not enrolled15:50
dtantsurwe probably don't want it enrolled after that, do we?15:51
dtantsur(think, your laptop where you're testing the application)15:51
TheJuliahmmmm15:51
TheJuliaI feel like maybe this is a "yes i know this machine may be considered discovered" command line flag15:51
*** ianychoi has quit IRC15:52
kaifengIf it's a re-inspect I guess the node is enrolled in ironic already?15:52
TheJuliayeah15:52
TheJuliaand if it is not known to ironic and someone runs the report-to-inspector command (to be named) then it is going to by design discover the machine if not already known to ironic15:53
dtantsurright15:53
TheJuliawhen discovery is enabled that is, and I can see reason for both, and even actually teh case of "I want to add this machine now"15:53
openstackgerritMerged openstack/ironic stable/rocky: Ansible module: fix configdrive partition creation step  https://review.opendev.org/66487515:53
openstackgerritMerged openstack/ironic master: redfish: handle missing Bios attribute  https://review.opendev.org/66510015:53
dtantsurand the real case is probably running this command via cron, and pre-flashing it on the standard image15:53
TheJulia"Hi, I'm done with this box. Let me give it to ironic and someone can reconcile the password later15:53
TheJuliadtantsur: yeah15:53
TheJuliaand CMDB going "oh, wait! That changed!"15:54
*** trandles has joined #openstack-ironic15:54
TheJulia"Oh, smart data about $device says bad things, trigger the alarm!"15:54
TheJuliaThere doesn't really seem to be opposition, more that case handling of discovery and what if ports change15:55
*** trident has quit IRC15:56
TheJuliaI can double check the ports code. I feel like we just a safety switch on the footgun and we'll be good for the manual invocations.15:56
jrollI agree with arne_wiebalck, so if we merge this, can we start calling inspector or ironic a cmdb?15:56
jroll:)15:56
TheJuliajroll: it only keeps the latest copy... so CMDB data source15:56
* jroll makes a note to reference this in a year when we add audit logging :P15:57
TheJulialol15:57
arne_wiebalcklol15:57
jroll:D15:57
TheJuliaOkay, we have 3 minutes left. Lets go to Open Discussion and glance at ashtempl_'s patches15:57
TheJulia#topic Open Discussion15:57
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:57
dtantsurI hope we keep it as a data source for CMDB :)15:58
TheJulia"user jroll deleted node 001"15:58
*** trident has joined #openstack-ironic15:58
TheJulia:)15:58
TheJuliaashtempl_: just the two templest test additions?15:58
TheJuliaashtempl_: I think there fine for us to add15:59
ashtempl_ok15:59
ashtempl_both of them?15:59
TheJuliaashtempl_: yeah, if you wouldn't mind just adding them to the whiteboard15:59
TheJuliaDo we have anything else to discuss today?15:59
TheJuliaAre we taking over the world yet?15:59
trandlesI'll be testing standalone ironic as a way to provision a 1600 node HPC cluster in the next month. You might be taking over my world.16:00
TheJuliatrandles: ohhhhhh! Ahhhhh!16:00
TheJuliatrandles: good luck, and feel free to ask questions :)16:01
rpittautrandles: that sounds fun :)16:01
trandlesThanks, will do. I know there was a task from the Denver PTG to work on standalone documentation. Is there some place you can point me so I can contribute?16:01
rpittautrandles: this can be a godo starting point: https://docs.openstack.org/ironic/latest/install/standalone.html16:02
rpittau^godo^good16:02
TheJuliatrandles: /doc/source/install/standalone.rst I believe in the ironic repository if you want to make changes or suggestions16:03
dtantsurtrandles: also something unfinished here https://review.opendev.org/#/c/641383/16:03
patchbotpatch 641383 - ironic - [WIP] Reference architecture: expand on networking - 2 patch sets16:03
mgoddardtrandles: we recently improved support for standalone ironic in kolla-ansible. You might find parts of bifrost useful with it also16:03
mgoddardtrandles: I did some experimenting here: https://github.com/markgoddard/bikolla16:03
trandlesmgoddard: thanks16:03
trandlesI was going to cherry pick ansible from bifrost and drop it into the large collection of stuff we already use.16:04
TheJuliaawesome16:04
TheJuliatrandles: patches are always welcome :)16:04
*** stendulker has quit IRC16:04
TheJuliaAnyway, we're over time. Thanks eveyrone16:05
TheJulia#endmeeting16:05
*** 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 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-dev mailing list."16:05
openstackMeeting ended Mon Jun 17 16:05:18 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:05
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-06-17-15.00.html16:05
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-06-17-15.00.txt16:05
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-06-17-15.00.log.html16:05
*** dnuka has quit IRC16:05
openstackgerritIlya Etingof proposed openstack/ironic master: Collect sensor data in ``redfish`` hardware type  https://review.opendev.org/66564616:06
mgoddardtrandles: bifrost is certainly nice and easy to get going. The main reason one might end up going for kolla-ansible is HA/scale out of the control plane which AFAIK bifrost can't do16:07
*** ashtempl_ has quit IRC16:09
NobodyCamGood morning ironic’ers16:10
*** gyee has joined #openstack-ironic16:11
rpittauhey NobodyCam :)16:13
arne_wiebalckNobodyCam: o/16:13
NobodyCamHey hey rpittau and arne_wiebalck happy Monday16:14
*** bobmel has joined #openstack-ironic16:14
openstackgerritMerged openstack/sushy master: Make Manager->Actions field optional  https://review.opendev.org/65888616:15
*** lucasagomes has quit IRC16:15
*** mbeierl has quit IRC16:17
openstackgerritPradip Kadam proposed openstack/ironic master: DRAC: Adding reset_idrac and known_good_state cleaning steps  https://review.opendev.org/65827116:19
arne_wiebalckbye everyone o/16:20
*** kaifeng has quit IRC16:22
*** dtantsur is now known as dtantsur|afk16:27
dtantsur|afk\o16:27
TheJuliagoodnight16:27
TheJuliagood morning NobodyCam16:28
*** whoami-rajat has quit IRC16:29
rpittaubye! good night! o/16:31
*** rpittau is now known as rpittau|afk16:31
*** ricolin has quit IRC16:35
*** mbeierl has joined #openstack-ironic16:39
*** bobmel has quit IRC16:40
*** bobmel has joined #openstack-ironic16:41
*** eandersson has joined #openstack-ironic16:43
*** mbeierl has quit IRC16:44
*** igordc has joined #openstack-ironic16:44
*** whoami-rajat has joined #openstack-ironic16:48
*** Lucas_Gray has joined #openstack-ironic16:53
*** ricolin has joined #openstack-ironic16:53
*** dougsz has quit IRC16:57
*** rachit7 has joined #openstack-ironic16:57
NobodyCamGood morning TheJulia16:59
NobodyCamNight dtantsur16:59
*** derekh has quit IRC17:00
*** bobmel has quit IRC17:01
*** trandles has left #openstack-ironic17:02
*** bobmel has joined #openstack-ironic17:12
*** rachit7 has quit IRC17:13
*** bobmel has quit IRC17:16
*** bonguardo has quit IRC17:22
*** bobmel has joined #openstack-ironic17:23
*** bobmel has quit IRC17:24
*** bobmel has joined #openstack-ironic17:25
*** trident has quit IRC17:27
*** mbeierl has joined #openstack-ironic17:27
*** trident has joined #openstack-ironic17:29
*** rachit7 has joined #openstack-ironic17:29
*** ricolin has quit IRC17:30
*** rachit7 has quit IRC17:38
*** bobmel has quit IRC17:49
*** bobmel has joined #openstack-ironic17:50
*** rh-jelabarre has quit IRC17:51
openstackgerritMerged openstack/ironic-tempest-plugin master: Switch to the new canonical constraints URL on master  https://review.opendev.org/66567417:52
*** rh-jelabarre has joined #openstack-ironic17:52
openstackgerritMerged openstack/python-ironicclient master: Switch to the new canonical constraints URL on master  https://review.opendev.org/66566917:56
*** bobmel has quit IRC17:58
*** mbeierl has quit IRC17:59
*** Lucas_Gray has quit IRC18:01
*** mbeierl has joined #openstack-ironic18:07
*** Goneri has quit IRC18:23
*** niceplace has quit IRC18:30
*** niceplace has joined #openstack-ironic18:31
*** trandles has joined #openstack-ironic18:37
*** pcaruana has quit IRC18:44
*** bobmel has joined #openstack-ironic18:52
*** bobmel has quit IRC18:56
*** rloo has quit IRC18:57
*** bobmel has joined #openstack-ironic19:04
*** mbeierl has quit IRC19:05
*** bobmel has quit IRC19:10
*** bobmel has joined #openstack-ironic19:10
*** pcaruana has joined #openstack-ironic19:17
*** bobmel has quit IRC19:18
*** Goneri has joined #openstack-ironic19:18
*** mbeierl has joined #openstack-ironic19:21
*** rloo has joined #openstack-ironic19:22
*** pcaruana has quit IRC19:27
*** trident has quit IRC19:37
*** trident has joined #openstack-ironic19:39
*** zaneb has quit IRC19:52
*** zaneb has joined #openstack-ironic19:52
*** mbeierl has quit IRC19:59
*** e0ne has joined #openstack-ironic20:07
*** e0ne has quit IRC20:07
*** mbeierl has joined #openstack-ironic20:10
*** Lucas_Gray has joined #openstack-ironic20:11
*** brandor5 has joined #openstack-ironic20:16
*** rloo has quit IRC20:17
brandor5hello everyone: I'm getting the following error... baremetal node is available and responds normally... "The instance is available but not Active state: ERROR" any idea why I'm getting thta?20:18
openstackgerritMerged openstack/ironic master: devstack: add missing variables for ironic-python-agent-builder  https://review.opendev.org/66563520:18
*** whoami-rajat has quit IRC20:19
*** patchbot has quit IRC20:25
*** patchbot has joined #openstack-ironic20:30
TheJuliabrandor5: where are you seeing that?!?20:31
brandor5TheJulia: it's getting returned as the error when trying to provision that node with ansible20:32
brandor5from reading that, it almost makes me think that nova thinks something is deployed to it even though there shouldn't be20:33
TheJuliaor somehow it ende dup in like rescue or something20:33
TheJuliathat is just bizzar20:34
*** rloo has joined #openstack-ironic20:34
brandor5I'm deleting the node out and trying to re-add it to see if that helps20:34
TheJuliasadly I'm not really sure of how your triggering it20:34
TheJuliaAnyway, I need to go to the post office.20:34
brandor5no worries, I'll keep digging and report back if something changes or I figure out what is going on20:35
brandor5thanks!20:35
TheJuliabrandor5: okay, no problem. I should be back in about an hour. At worst... hopefully20:40
*** Lucas_Gray has quit IRC20:46
*** mbeierl has quit IRC21:17
*** bfournie has quit IRC21:21
*** rcernin has joined #openstack-ironic21:43
*** jcoufal has quit IRC21:50
*** mbeierl has joined #openstack-ironic21:50
*** early` has joined #openstack-ironic21:51
*** early has quit IRC21:52
brandor5I'm done for the day but the follow up is that after deleting that baremetal node and re-adding i'm getting "BadRequestException: Unknown error" now21:52
brandor5I'll pick this back up tomorrow21:53
*** mbeierl has quit IRC21:54
*** rh-jelabarre has quit IRC21:58
*** brandor5 has quit IRC22:00
*** mbeierl has joined #openstack-ironic22:08
*** mbeierl has quit IRC22:13
TheJuliaugh22:22
* TheJulia looks at the clock and just sighs22:22
*** mbeierl has joined #openstack-ironic22:36
*** mbeierl has quit IRC22:40
TheJuliadtantsur|afk: circling back. We explicitly block ports from being able to be destroyed in active state... So I think the only thing that we might want to do there is maybe guard deletes by if a vif is attached (actually, reasonable bug fix really...) from a user being able to destroy the port with a vif record on it.22:45
TheJuliadtantsur|afk: and creates are literally just object creations, vifs live on the ports22:45
*** mbeierl has joined #openstack-ironic23:03
*** mbeierl has quit IRC23:08
*** mbeierl has joined #openstack-ironic23:22
*** mbeierl has quit IRC23:26
*** gyee has quit IRC23:28
*** sthussey has quit IRC23:29
*** mbeierl has joined #openstack-ironic23:40
*** mbeierl has quit IRC23:44
*** cdearborn has quit IRC23:57

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