Monday, 2018-07-09

*** threestrands has joined #openstack-ironic00:03
*** threestrands has quit IRC00:03
*** threestrands has joined #openstack-ironic00:03
*** hshiina has joined #openstack-ironic00:10
*** markguz has joined #openstack-ironic00:21
*** markguz has quit IRC00:28
*** pc_m has quit IRC00:40
*** shananigans has quit IRC00:40
*** shananigans has joined #openstack-ironic00:40
*** pc_m has joined #openstack-ironic00:40
*** alex_xu has quit IRC00:40
*** alex_xu has joined #openstack-ironic00:41
*** hwoarang has quit IRC00:43
*** markguz has joined #openstack-ironic01:02
*** markguz has quit IRC01:15
*** threestrands_ has joined #openstack-ironic01:18
*** threestrands_ has quit IRC01:18
*** threestrands_ has joined #openstack-ironic01:18
*** threestrands_ has quit IRC01:19
*** threestrands_ has joined #openstack-ironic01:20
*** markguz has joined #openstack-ironic01:20
*** threestrands_ has quit IRC01:21
*** threestrands_ has joined #openstack-ironic01:21
*** threestrands_ has quit IRC01:21
*** threestrands_ has joined #openstack-ironic01:21
*** threestrands has quit IRC01:21
*** markguz has quit IRC01:36
*** markguz has joined #openstack-ironic01:38
*** zhangfei has joined #openstack-ironic01:49
*** markguz has quit IRC01:49
*** tiendc has joined #openstack-ironic01:53
*** skazi has joined #openstack-ironic02:41
*** skazi has quit IRC02:49
*** markguz has joined #openstack-ironic03:20
*** markguz has quit IRC03:25
*** jaganathan has joined #openstack-ironic03:53
*** tiendc has quit IRC03:56
*** tiendc has joined #openstack-ironic03:56
openstackgerritTuan Do Anh proposed openstack/networking-baremetal master: Updating required neutron version  https://review.openstack.org/58090204:14
openstackgerritTuan Do Anh proposed openstack/networking-generic-switch master: Updating required neutron version  https://review.openstack.org/58090404:26
*** vmud213 has joined #openstack-ironic04:28
openstackgerritNguyen Van Trung proposed openstack/ironic master: Implement iRMC BIOS configuration  https://review.openstack.org/53459504:31
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863304:42
openstackgerritNguyen Van Trung proposed openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863304:42
*** links has joined #openstack-ironic05:00
*** trungnv has joined #openstack-ironic05:44
*** hjensas has quit IRC05:46
*** yolanda has joined #openstack-ironic05:46
*** trungnv has quit IRC05:48
*** trungnv has joined #openstack-ironic05:49
openstackgerritparesh sao proposed openstack/ironic master: [WIP]OOB RAID implementation for ilo5 based HPE Proliant servers.  https://review.openstack.org/55566405:51
*** quiquell|off is now known as quiquell05:53
*** hshiina2 has joined #openstack-ironic06:00
*** hshiina has quit IRC06:04
*** mjura has joined #openstack-ironic06:14
*** rbartal has joined #openstack-ironic06:19
*** quiquell is now known as quiquell|bbl06:19
*** trungnv has quit IRC06:26
*** trungnv has joined #openstack-ironic06:26
*** tiendc has quit IRC06:30
*** tiendc has joined #openstack-ironic06:30
*** hshiina has joined #openstack-ironic06:36
*** trungnv has quit IRC06:37
*** phuongnh has joined #openstack-ironic06:37
*** trungnv has joined #openstack-ironic06:37
*** hshiina2 has quit IRC06:38
*** skazi has joined #openstack-ironic06:51
*** jtomasek has joined #openstack-ironic06:53
openstackgerritHarald Jensås proposed openstack/ironic-inspector stable/queens: PXE Filter dnsmasq: manage macs not in ironic  https://review.openstack.org/58093106:57
*** hjensas has joined #openstack-ironic06:57
*** hjensas has quit IRC06:57
*** hjensas has joined #openstack-ironic06:57
*** arnewiebalck_ has joined #openstack-ironic07:06
*** tesseract has joined #openstack-ironic07:06
*** rcernin has quit IRC07:08
openstackgerritNguyen Van Trung proposed openstack/bifrost master: Follow the new PTI for document build  https://review.openstack.org/57136007:19
*** tesseract has quit IRC07:25
*** tesseract has joined #openstack-ironic07:27
openstackgerritNguyen Van Trung proposed openstack/bifrost master: Follow the new PTI for document build  https://review.openstack.org/58094407:28
*** arnewiebalck_ has quit IRC07:30
*** d0ugal_ has quit IRC07:33
*** d0ugal has joined #openstack-ironic07:33
*** d0ugal has quit IRC07:33
*** d0ugal has joined #openstack-ironic07:33
*** quiquell|bbl is now known as quiquell07:43
*** zhangfei has quit IRC07:46
*** stendulker has joined #openstack-ironic07:55
*** rpittau has joined #openstack-ironic07:57
*** zhangfei has joined #openstack-ironic07:58
*** ajya has joined #openstack-ironic07:59
*** racedo has joined #openstack-ironic08:01
*** zigo has quit IRC08:03
*** zigo has joined #openstack-ironic08:05
*** athomas has joined #openstack-ironic08:11
*** arnewiebalck_ has joined #openstack-ironic08:12
*** threestrands_ has quit IRC08:12
*** athomas has quit IRC08:15
*** skazi has quit IRC08:18
*** dougsz has joined #openstack-ironic08:19
*** athomas has joined #openstack-ironic08:20
*** racedo_ has joined #openstack-ironic08:21
*** stendulker_ has joined #openstack-ironic08:22
*** stendulker has quit IRC08:25
*** derekh has joined #openstack-ironic08:29
*** etingof has quit IRC08:31
openstackgerritAija Jaunteva proposed openstack/sushy-tools master: Add BIOS resource support  https://review.openstack.org/58060908:32
*** mgoddard has joined #openstack-ironic08:32
olivierb-Good morning ironic, I do not understand why CI is failing for https://review.openstack.org/#/c/579817/. Could someone please help me understanding this ?08:36
patchbotpatch 579817 - ironic-lib (stable/pike) - Expose GPT partitioning fixing method08:36
*** derekh has quit IRC08:39
*** derekh has joined #openstack-ironic08:40
olivierb-are there some missng patches in pike (and ocata BTW )08:40
olivierb-rsync: change_dir \"/home/zuul//src/git.openstack.org/openstack/ironic-lib/doc/build\" failed: No such file or directory08:40
*** etingof has joined #openstack-ironic08:43
*** serlex has joined #openstack-ironic08:58
*** racedo_ has quit IRC09:06
*** Goneri has joined #openstack-ironic09:08
openstackgerritDao Cong Tien proposed openstack/ironic master: Update the doc regarding the removal of calssic drivers  https://review.openstack.org/58097309:15
*** zhangfei has quit IRC09:16
*** d0ugal has quit IRC09:17
*** pbourke has quit IRC09:20
*** pbourke has joined #openstack-ironic09:22
*** d0ugal has joined #openstack-ironic09:23
*** zhangfei has joined #openstack-ironic09:29
*** stendulker has joined #openstack-ironic09:36
*** stendulker_ has quit IRC09:39
*** hkominos has left #openstack-ironic09:47
*** hkominos has joined #openstack-ironic09:48
*** sambetts|afk is now known as sambetts09:48
*** mjura has quit IRC09:55
openstackgerritvinay kumar muddu proposed openstack/ironic master: iLO BIOS interface implementation  https://review.openstack.org/57680310:07
*** stendulker has quit IRC10:38
*** phuongnh has quit IRC10:40
openstackgerritparesh sao proposed openstack/ironic master: OOB RAID implementation for ilo5 based HPE Proliant servers.  https://review.openstack.org/55566410:52
serlexHi, my instance deployment seems to be stuck at deploy-ramdisk image (Fedora 25 (cloud edition)) - I can see the login prompt via serial console. Best way to troubleshoot this further?10:55
*** milan has joined #openstack-ironic10:58
*** markguz has joined #openstack-ironic11:20
*** zhangfei has quit IRC11:21
*** bfournie has quit IRC11:23
*** bfournie has joined #openstack-ironic11:24
*** markguz has quit IRC11:25
*** rh-jelabarre has joined #openstack-ironic11:52
*** bfournie has joined #openstack-ironic11:54
*** MattMan has quit IRC12:04
*** MattMan has joined #openstack-ironic12:04
*** d0ugal has quit IRC12:06
*** jaypipes has joined #openstack-ironic12:06
*** d0ugal has joined #openstack-ironic12:08
*** jistr is now known as jistr|mtg12:12
*** tiendc has quit IRC12:14
openstackgerritOlivier Bourdon proposed openstack/ironic-lib stable/pike: Fix pip10 broken build-openstack-sphinx-docs for older releases  https://review.openstack.org/58101812:16
*** cjloader has joined #openstack-ironic12:23
openstackgerritOlivier Bourdon proposed openstack/ironic-lib stable/pike: Avoid tox_install.sh for constraints support  https://review.openstack.org/58102212:25
*** cjloader has quit IRC12:26
*** trown|outtypewww is now known as trown12:26
openstackgerritOlivier Bourdon proposed openstack/ironic-lib stable/ocata: Avoid tox_install.sh for constraints support  https://review.openstack.org/58102412:27
*** quiquell is now known as quiquell|off12:28
*** dprince has joined #openstack-ironic12:37
*** cjloader has joined #openstack-ironic12:41
TheJuliaserlex: Does it just timeout from “wait callback” state?12:43
TheJuliaserlex: It kind of sounds like ironic-python-agent is not starting in the ramdisk... or is failing to start up. We try and ship logs up to the conductor (typically to /var/log/ironic/) node, but if IPA does not start in them ramdisk, naturally that won’t happen.12:45
*** efried_off is now known as efried12:46
TheJuliaserlex: most people end up burning users into their their deploy ramdisk to do this sort of debugging12:46
TheJuliaolivierb-: <312:46
*** efried has quit IRC12:46
TheJuliaolivierb-: I take it you figured out your CI issue?12:46
*** cjloader has quit IRC12:47
olivierb-Happy monday morning TheJulia, yep definitely12:47
olivierb-getting more familiar when have to look into error logs and seeking other repos for appropriate fixes ;-)12:48
olivierb-so easy to overlook stuff, and so many stuff to look at ...12:48
TheJuliaolivierb-: Excellent! And a good morning to you as well!12:49
* TheJulia may have had too much coffee before departing the bed and breakfast....12:49
TheJuliaIt is very easy to overlook things :(12:50
TheJuliaAlso, good morning everyone!12:50
olivierb-BTW not very familiar will all the <3 and others IRC "annotations"12:50
TheJulia<3 == heart12:50
olivierb-ah ok thx12:51
TheJuliaWow, lots of stuff to review it looks like12:51
*** quiquell|off is now known as quiquell12:53
olivierb-only 4 small patches from my side  and end of last week when having to re-use venv to cope with some system wide incompatibilities between python modules and global requirements from OpenStack12:54
TheJuliasambetts: Have you tried boartty ?12:54
* TheJulia is about to loose cell service, will be back in time for the meeting (turned out we could not checkout until after 8 AM this morning when we were planning on checking out earlier and driving back.)12:55
olivierb-BTW small question about ironic-lib patches https://review.openstack.org/#/c/579816/ and https://review.openstack.org/#/c/579817/12:55
patchbotpatch 579816 - ironic-lib (stable/ocata) - Expose GPT partitioning fixing method12:55
patchbotpatch 579817 - ironic-lib (stable/pike) - Expose GPT partitioning fixing method12:55
sambettsTheJulia: I didn't realise that existed, looks interesting12:55
olivierb-should I rebase them onto the CI fixes ?12:55
*** cjloader has joined #openstack-ironic12:57
TheJuliaHmm.. I guess so, otherwise they can wait and we can recheck them. I suspect I’lll need to discuss those with the other stable cores12:57
*** toabctl has quit IRC12:59
TheJuliaolivierb-: regarding incompatibilities, seems like we need to look at the requirements... the g-r syncing no longer being a thing means that they can drift... and we then become incompatible if installed in the same space12:59
*** toabctl has joined #openstack-ironic13:00
*** cjloader has quit IRC13:02
*** cjloader has joined #openstack-ironic13:03
*** hshiina has quit IRC13:04
olivierb-just FYI, updating PyMySQL from 0.8.1 to 0.9.0 produced an incompatbility on my Ubuntu Xenial systems where I also have package virt-install installed due to python-ndg-httpsclient which is a dependency of virt-install and which is  incompatible, least to say because after this you can not use pip nor any network related Python function :-(13:07
*** cjloader_ has joined #openstack-ironic13:07
*** cjloader has quit IRC13:07
*** markguz has joined #openstack-ironic13:07
olivierb-by updating PyMySQL I mean taking the new upper requirements of course13:07
*** rloo has joined #openstack-ironic13:07
serlexthanks for the replies TheJulia. I will modify the ramdisk13:07
*** markguz has quit IRC13:09
*** markguz has joined #openstack-ironic13:10
*** markguz_ has joined #openstack-ironic13:12
*** markguz has quit IRC13:14
*** jistr|mtg is now known as jistr13:14
rlooGood Monday morning ironic'ers!13:15
rlooTheJulia, mgoddard: does this need more reviews or are one of you good with +A'ing? https://review.openstack.org/#/c/399234/13:15
patchbotpatch 399234 - ironic - Caching of PDU autodiscovery13:15
*** markguz_ has quit IRC13:17
TheJuliarloo: I’m good with it, I can +A it when I get on more reliable internets13:17
mgoddardrloo: I'll +A13:17
rloothx mgoddard & TheJulia!13:18
TheJuliamgoddard: thanks!13:18
TheJuliarloo: Good morning!13:18
mgoddard-problem13:18
TheJulia:)13:18
*** vmud213 has quit IRC13:21
*** arnewiebalck_ has quit IRC13:29
*** arnewiebalck_ has joined #openstack-ironic13:31
*** jistr is now known as jistr|mtg13:36
*** lbragstad has joined #openstack-ironic13:37
openstackgerritMerged openstack/bifrost master: Fix installation of python-openstack module when using venv  https://review.openstack.org/58009413:38
*** cjloader_ has quit IRC13:40
NobodyCamGood morning ironic’ers13:40
*** mjturek has joined #openstack-ironic13:42
*** skazi has joined #openstack-ironic13:46
*** zhangfei has joined #openstack-ironic13:47
TheJuliaOkay, I'm back13:54
*** skazi has quit IRC13:55
TheJuliaolivierb-: looks like https://review.openstack.org/#/c/580121 needs to be rebased13:55
patchbotpatch 580121 - bifrost - Fix installation of keystone module when using venv13:56
olivierb-TheJulia let me check this13:56
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix installation of keystone module when using venv  https://review.openstack.org/58012114:00
*** baha has joined #openstack-ironic14:00
olivierb-TheJulia done thanks14:01
olivierb-and many thanks also for the merges14:04
olivierb-including you too rloo14:04
*** jistr|mtg is now known as jistr14:05
rlooolivierb-: yw :)14:07
*** rpioso|afk is now known as rpioso14:11
rpiosoGood morning :)14:11
*** zhangfei has quit IRC14:12
*** med_ has quit IRC14:15
*** med_ has joined #openstack-ironic14:16
*** med_ has quit IRC14:16
*** med_ has joined #openstack-ironic14:16
*** med_ has quit IRC14:16
*** med_ has joined #openstack-ironic14:16
*** links has quit IRC14:18
openstackgerritMerged openstack/bifrost master: Fix uwsgi configuration when using venv  https://review.openstack.org/58013114:18
*** markguz has joined #openstack-ironic14:19
TheJuliavdrok: When you have some time, I left a couple questions on https://review.openstack.org/#/c/34368414:20
patchbotpatch 343684 - ironic-specs - Add synchronize-events-with-neutron spec14:20
openstackgerritMerged openstack/bifrost master: Fix openstack CLI calls environment when using venv  https://review.openstack.org/58016414:20
*** markguz_ has joined #openstack-ironic14:20
*** markguz_ has quit IRC14:21
*** markguz_ has joined #openstack-ironic14:21
*** markguz_ has quit IRC14:22
*** markguz has quit IRC14:23
*** zhangfei has joined #openstack-ironic14:24
*** rbartal has quit IRC14:24
TheJuliarpioso: Hey, regarding https://review.openstack.org/#/c/545184, how close to being ready for prime time review is it? Only asking because it is a DNM right now14:27
patchbotpatch 545184 - ironic - DNM Fix iDRAC hardware type does not work with UEFI14:27
rpiosoTheJulia: Probs another week.14:30
*** jcoufal has joined #openstack-ironic14:30
TheJuliastable-cores: quick ironic-inspecto review https://review.openstack.org/#/c/580931/114:32
patchbotpatch 580931 - ironic-inspector (stable/queens) - PXE Filter dnsmasq: manage macs not in ironic14:32
rlooTheJulia: done14:36
*** arnewiebalck_ has quit IRC14:37
*** jcoufal has quit IRC14:39
*** mjura has joined #openstack-ironic14:44
*** cdearborn has joined #openstack-ironic14:45
*** stendulker has joined #openstack-ironic14:52
*** Will_ has joined #openstack-ironic14:54
*** Will_ is now known as Guest8608814:54
*** Guest86088 is now known as willm14:55
*** hshiina has joined #openstack-ironic14:55
*** quiquell is now known as quiquell|off14:58
jiapeiGood morning guys15:00
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Jul  9 15:00:12 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
TheJuliaGood morning everyone!15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
jiapeio/15:00
rpioso\o15:00
TheJuliaOur agenda can be found on the wiki, as always!15:00
TheJuliao/15:00
hshiinao/15:00
*** beekneemech is now known as bnemec15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
stendulkero/15:01
bdoddo/15:01
mgoddardo/15:01
rlooo/15:01
jroll\o15:01
TheJulia#topic Announcements/Reminders15:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:01
TheJulia#Info Ironic version 11 was released last week!15:01
TheJuliaThanks to everyone who has helped contribute to that milestone. Hopefully we'll be able to get another few major things merged in the next couple weeks!15:02
jrollwoot15:02
rloogreat work, lots of goodies in 11 :)15:02
TheJulia#undo15:02
openstackRemoving item from minutes: #info Ironic version 11 was released last week!15:02
TheJulia#info Ironic version 11 was released last week!15:02
hjensaso/15:02
TheJuliaHeh, commands must be case insensitive15:02
TheJuliaAnyway, moving on!15:02
TheJulia#info A "mid-cycle" call is slated for July 17th. Anticipate the agenda to consist of a quick review of the cycle so far and discussion of any topics for the upcoming PTG.15:03
TheJuliaSpeaking of the upcoming PTG!15:04
TheJulia#info Stein PTG planning etherpad has been posted for a ?couple? weeks. Thoughts, Ideas, needs for time with the group should be added to the list before August 23rd.15:05
TheJulia#link https://etherpad.openstack.org/p/ironic-stein-ptg15:05
jrollnice, didn't realize we had one15:06
rlooTheJulia: wrt the mid-cycle, is there going to be an etherpad/agenda/whatever for that, or do we decide at the time?15:06
TheJulia#info This week is R-7, which means we realistically have just a few more weeks to be ready for ironic 11.1.15:07
rlooalso wrt the mid-cycle, there was a doodle-thingy for it; when will the date/time be decided?15:07
TheJulia#info Non-core reviewers are absolutely welcome and encouraged to review items on our weekly priority list.15:08
TheJuliarloo: I'll send out an email and create an etherpad15:08
TheJulia#action TheJulia to send out an email and create an etherpad for the "midcycle" call15:08
rloothx TheJulia15:09
TheJuliarloo: The doodle was posted a week ago, and we've had limited feedback, but the time is based upon the mutually available time selected.15:09
mjtureko/15:09
TheJuliaor, I should say, will be15:09
TheJulia#link https://doodle.com/poll/nu7bf2t8ygxfm6vf15:10
TheJulia\o mjturek15:10
TheJuliaIf there is nothing anyone wishes to raise, we can move on15:10
*** zhangfei has quit IRC15:10
*** hshiina has quit IRC15:11
*** hshiina_ has joined #openstack-ironic15:11
TheJulia#topic Review action items for our past meeting15:11
*** openstack changes topic to "Review action items for our past meeting (Meeting topic: ironic)"15:11
TheJulia#info No action items since the last meeting.15:11
TheJulia#info TheJulia sent out a summit summary email last week, which was an action item from the prior week.15:12
TheJulia#link http://lists.openstack.org/pipermail/openstack-dev/2018-July/131972.html15:12
TheJuliaAnyway, Moving on!15:12
TheJulia#topic Review subteam status reports15:13
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:13
rloothanks for that summary email TheJulia!15:13
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:13
TheJuliaStarting around line 14915:13
TheJuliare: deploy steps, I saw we still ended up splitting the deploy steps patch and I've not had a chance to review it since since I've been on PTO. I'll try to review the series in the next few days.15:14
rloothx TheJulia. I have two questions (so far) about deploy steps. There is only one big deploy step -- I am good with that but want to make sure others are. AND... deprecation period? 2 cycles instead of 1?15:16
TheJuliahshiina_: Do you happen to know the status of the python-scciclient changes presently blocking the irmc raid interface? Curious based on when we should expect those merged15:16
rlooTheJulia: wrt reference arch guide -- i think we should do it at the next PTG; i am highly doubtful it will happen before then.15:17
hshiina_TheJulia, this will be merged this week15:17
TheJuliarloo: I'm good with one big deploy step this cycle to get the framework landed, even though we discussed two. We can expose more as time goes on, the important thing is beginning to offer the flexibility.15:17
TheJuliahshiina_: awesome15:17
jroll+1 for one big deploy step15:17
mgoddard+1 - keep it simple as time is short15:18
rlooTheJulia, jroll: thx. Makes sense to do deprecation as 2 cycles (min) then.15:18
TheJuliarloo: also, w/r/t deprecation, I think two cycles is ideal, especially as the staging drivers is going to need some time and attention.15:18
* TheJulia feels like we have consensus15:18
jrollhow much work is it to switch over for a driver?15:18
rlooTheJulia: oh yeah, staging drivers... will try to get those done in rocky too.15:18
TheJuliaNot too much, but we need review bandwidth for those to change...15:19
rlooshouldn't be much work but I'm not familiar with the staging drivers :)15:19
jrollI'm more thinking about other out of tree drivers15:19
rloonot sure how much/any testing we have with staging drivers15:19
TheJuliaStaging moves very slowly due to limited reviewers15:19
TheJuliaAnd it is outside of our governance...15:19
jrollwe could just remove after staging drivers get done15:20
rloofor the one big deploy step -- there are two main changes, add the decorator and change the 'last part' of the deployment code, to call the conductor. I'll add docs for that... somewhere...15:20
jrollor the people that contributed those staging drivers can come fix it, and if they don't it'll be broken15:20
TheJuliaI know there are some out of tree deploy drivers that operators have created, so it would likely be good to give them time to adapt.15:20
jrollalso this only affects deploy drivers, not sure if we have any of those in staging15:21
TheJuliarloo: re: refarch, I don't know. *sigh*15:21
jrolllooks relatively trivial: https://review.openstack.org/#/c/578649/8/ironic/drivers/modules/iscsi_deploy.py15:21
patchbotpatch 578649 - ironic - Deploy steps - conductor & drivers15:21
TheJuliathat is a good point jroll, I don't think we do15:22
rlooTheJulia: yeah, it is important. so we MUST? set time aside at PTG to do it if it isn't done before hand.15:22
jrolloh and this bit: https://review.openstack.org/#/c/578649/8/ironic/drivers/modules/agent_base_vendor.py15:22
patchbotpatch 578649 - ironic - Deploy steps - conductor & drivers15:22
* jroll thinks one cycle is plenty15:22
TheJuliarloo: ++15:22
TheJuliajroll: Yeah, I'm leaning towards one cycle as long as we do it promptly15:23
*** mjturek_ has joined #openstack-ironic15:23
rloowrt deprecation of non-deploy-steps; we could do one cycle, and change to longer if it becomes necessary?15:23
jrollTheJulia: do what promptly? land the deploy steps code?15:23
TheJuliaWe absolutely can do that rloo15:23
jrollrloo: yeah, we definitely can change later15:23
TheJuliajroll: land... although didnt we also talk about the default being like automatic cleaning so a user could still just asay deploy and things happen. At which point, what exactly are we deprecating?15:24
*** mjturek has quit IRC15:24
* TheJulia senses the call of task.driver.deploy.deploy() in conductor.py15:25
rlooTheJulia: deprecating the code that supports non-deploy-steps15:25
TheJuliaI guess I need to see in-line deprecation warnings to really be able to wrap my head around it properly15:26
rlooTheJulia: maybe we should talk more at midcycle next week?15:26
TheJuliapossibly, I'll need to re-review the patches as well which is already on my list15:26
TheJulia\o/ for PTO15:26
rloo:)15:27
TheJuliaAnyway, Anything else status update wise that we need to discuss?15:27
rlooready to move on :)15:27
TheJuliaAlright15:28
TheJulia#topic Deciding on priorities for the coming week15:28
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:28
* TheJulia massages the list15:28
rloowrt hw-type cleanup. do we need it as a weekly priority?15:28
*** arnewiebalck_ has joined #openstack-ironic15:28
jrollprobably not anymore15:28
jrollwe landed a lot of the final removals last week15:28
rloo(esp since CI doesn't pass and it is a WIP)15:28
jrollthat's in staging-drivers anyhow15:29
TheJuliaI was just about to note that I've seen some doc updates, but that we can remove it from the priority list15:29
rloodtantsur|afk: if you are reading this, ^^ please add the topic 'hw-types' for your doc patches too15:30
* TheJulia lets jroll type15:30
jrollTheJulia: sorry, was just trying to s/4/315:30
TheJuliajroll: Oh, I thought it was 4 when I looked like an hour ago :\15:30
* jroll is done there15:30
TheJuliaawesome15:30
jrollone is the merged spec15:31
* jroll fixes link15:31
rloowhat is the 'Notes for the meeting' section (L117)? Is that for discussion in a few min, not part of weekly priorities?15:31
jrollwell, I'll leave the link15:31
TheJuliarloo: That was for me, that I've not yet deleted15:31
jrolldo we want specs in priorities given the limited time left to land code?15:31
*** hshiina_ has quit IRC15:32
*** hshiina__ has joined #openstack-ironic15:32
TheJuliaThat is a good question15:32
TheJuliaI'd like to try and land a ramdisk interface given the number of operators that have been asking for a very long time for one15:32
jrollin rocky?15:32
TheJuliayeah15:33
rloois there code already for that? (i should probably read the spec first)15:33
jrollthere is15:33
jrollI'm okay with trying for that15:33
TheJuliaOkay, I'll move the rest down in priority since they are also on that list to raise visibility for reviews15:33
rloois the code 'small' ? I'm guessing it is.15:33
rlooi'd like to get the neutron event stuff, but i think the code there is nontrivial.15:34
TheJuliarloo: very15:34
jrollrloo: it's +291, -17, not bad15:34
*** cjloader has joined #openstack-ironic15:35
rlooprobably too late in the cycle to land that anyway, would like it being used a bit first before we cut a release.15:35
TheJuliaAnyway, I'm good with the list, does anyone have anything else to add?15:35
jrollI'm good15:35
*** gyee has joined #openstack-ironic15:35
rloo+115:35
TheJuliarloo: re: neutron event, agreed.15:35
TheJuliaOkay, moving on then!15:36
TheJulia#topic RFE Review15:36
*** openstack changes topic to "RFE Review (Meeting topic: ironic)"15:36
TheJuliaDmitry has asked for us to review and discuss an RFE that he would like to get in ASAP (at least, that is the way I interpreted the discussion from the logs.15:36
TheJulia#link https://storyboard.openstack.org/#!/story/200286815:37
jrollthere's a comment from rloo that it needs more thought, anything specific?15:37
rloois sambetts here? where are they wrt their discussions?15:37
rloojroll: that's what sambetts was discussing with dmitry15:38
jrollIIRC it ended with this being fine for now15:38
TheJuliaI'm good with both options, the first being the fastest and easiest to implement for users.15:38
sambettsHi, yup we decided this was best for now, and i opened a new story (RFE) for improving the API for interfaces later on15:38
rlooso --reset-deploy-interface is *ONLY* avail with --driver15:38
jrollsambetts: awesome, thanks15:39
TheJuliaI feel like we might actually have a bug based on original discussions for driver composition, but I'd have to go back and dig through notes15:39
sambettsrloo: yup, only with --driver, because you can't send an empty patch request for a node15:39
rloosambetts: exactly.15:39
sambettsditto for the reset-all-interfaces one15:40
rlooi think my question before was whether 'reset_interfaces' was the same as 'unset_X_interface' and whether we should rename it 'unset_intefaces' to be more consistent. but ...15:40
*** cjloader has quit IRC15:40
rloooh, i mean 'openstack baremetal node unset --X-interface'15:40
TheJuliaThat doesn't seem super ideal. I thought we agreed a long time ago that we should catch and reset interfaces if someone tried to change the hardware type15:41
TheJuliabecause that change is to a completely different profile of sorts15:41
rlooTheJulia: that's what #1 is for, 'openstack baremetal node set --driver <hwtype> --reset-deploy-interface15:42
sambettsoh yeah thats for the other use case, think doing unset deploy_interface should reset it, the only problem with that is there is no way to discover which interfaces exist today15:42
sambettsfor all API versions15:42
TheJuliarloo: I thought under the hood deep inside of the conductor15:42
jrollIIRC we didn't do it that way because it's so implicit15:42
rloooh wait. we're adding to our REST API: PATCH /v1/nodes/<node>?reset_interfaces=True15:44
TheJuliajroll: yeah, but even then I'm just not sure there is a reason not to honor the requested action as a high level change.15:44
sambettsmy personal preference is for non-driver-changing default resets to be done via unset, but reseting during a set --driver change to be done with a --reset-interfaces15:44
rloowhich allows one to reset all interfaces (except any explicitly specified in the patch). so it has nothing to do with user setting driver=diff-hw-type15:44
sambettsrloo: execpt you can't have an empty patch request so you need to send at least --driver even if its for the same driver15:45
rloosambetts: no, i mean, you could have a patch that eg. changes the node's name to foo, and also reset_interfaces.15:46
rloosambetts: nothing about changing the node's driver.15:46
sambettsyeah, me and dtantsur|afk decided against that15:46
rloooh, it isn't mentioned in the RFE (#2). if i understand what it sez there.15:46
sambettsbut open for opinions15:46
TheJuliaIt really feels like we are overthinking something that should be kept as simple as possible15:47
*** ajya has quit IRC15:47
*** ajya3 has joined #openstack-ironic15:47
rlooTheJulia: what do you suggest?15:47
TheJuliathe simpler it is for the user, the better the experience15:47
sambetts100% I really want this to be as simple as possible because IMO its a bandaid for a bad API15:47
sambettsand this: https://storyboard.openstack.org/#!/story/2002901 is the real fix15:47
rlooi think the proposed openstack CLI part is simple?15:47
TheJuliasambetts: agree15:48
jrollI think we all agree on the basic premise. the current questions are: 1) should the name be different? 2) should the full reset be implicit instead?15:49
TheJuliarloo: I feel like the simplest thing for 90% of the users is a command that just explicitly unsets all interfaces or replaces the value to None which should reset them back to default upon the next task. The pure API side which could land next cycle if needed could then be wired up to the client switch and not have any task fire in the case of an invalid configuration15:49
sambettsthe problem with a command that explictity unsets all the interfaces (my original suggestion to dtantsur|afk) was that older clients won't be able to know about new interfaces15:50
rlooTheJulia: so the command == CLI? I think what is proposed in #2 'openstack baremetal node set --driver xx --deploy-interface foo --reset-interfaces' satisfies that?15:50
TheJuliaAlthough I think dmitry fixed the code that would prevent a patch with an existing invalid configuration from completely preventing changing interfaces15:51
sambettsso they'll reset some not all of the interfaces15:51
*** cjloader has joined #openstack-ironic15:51
rlooOh, does TheJulia mean no one thing that resets more-than-one interface?15:51
TheJuliasambetts: people have to upgrade to get the command anyway and we operate on the lastest known api version now...15:51
TheJuliaI was thinking just complete reset15:52
sambettsTheJulia: but new interfaces after the new command is added will be hidden15:52
TheJuliareset back to what the conducto rwould calculate based on the hardware type15:52
sambettsnew CLI or API for that?15:52
sambettsCLI can't do it because it doesn't have a way to discover all the interfaces to unset them15:52
TheJuliasambetts: ahh, that is true, which is why I'm thinking just reset the entire node back to the calculated default15:53
TheJuliaactually... it could call validate15:53
sambettsnew interfaces are hidden to older clients15:53
TheJulianot in validate if memory serves15:53
rlooso if i understand, TheJulia wants 'openstack baremetal node set --driver xx --deploy-interface direct' to change the driver, set deploy interface, and unset all other interfaces?15:53
TheJuliaregardless, you are right, ideal is an api method to do it15:54
TheJuliarloo: that means we have to understand the precedence of all the set actions, seems better to just offer a reset-driver-interfaces method or something15:55
* jroll is just confused now as to what people want15:55
* TheJulia is too15:55
rloonow i am confused.15:55
*** cjloader has quit IRC15:55
rlooi am only trying to understand TheJulia, thought she said we were overthinking.15:55
rlooi don't see how we can add a reset-driver-interfaces method that is separate from a method to change the driver.15:55
jrollwell, we've 5 minutes left, maybe TheJulia can summarize her take in a comment on the RFE?15:56
rlooi mean we can add, but i don't see how it solves the problem.15:56
rlooand i thought that's what the proposed 'reset_interfaces' etc was meant to address.15:56
TheJuliajroll: I'll do so15:56
jrollthanks :)15:56
rloothx TheJulia!15:56
TheJulia#action TheJulia to summarize possible third option in the RFE15:56
sambettsrloo: yup, think of it as --force-reset-driver15:56
rlooso this is why we need (sorry) an RFE spec.15:57
TheJuliaheh15:57
rlooOR everyone add comments ot that story so we know why we end up choosing what we choose.15:57
jrollspec would also work, yeah, we do better with these async15:57
TheJuliaAnyway!15:57
TheJulia#topic Open Discussion15:57
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:58
TheJuliaAnyone have anything else to discuss today?15:58
mgoddardI'd like to introduce w-miller15:58
TheJuliamgoddard: *looks around*15:58
mgoddardor rather willm, looking at he members15:58
jroll\o willm15:58
rloowelcome willm!15:58
TheJuliaGreetings and welcome to the party willm!15:58
mgoddardhe's just started at StackHPC as an intern and will be with us over the summber15:58
rpiosoHey willm :-)15:58
sambettso/ willm15:59
willmHi all! \o15:59
rloo(how long is your summer?)15:59
stendulkero/ willm15:59
mgoddardworking on inspector and https://review.openstack.org/57958315:59
patchbotpatch 579583 - ironic-specs - [WIP]: Add virtual Bare Metal Clusters spec15:59
NobodyCamwelcome to Ironic Willm15:59
TheJuliamgoddard: willm: Awesome!15:59
willmI’m here until the 4th week of September and am looking forward to getting stuck in :)15:59
jiapeiI'd like to give a brief update on the xclarity ci status: we are blocked the last week, since zuul v3 can't pull ironic from gerrit, we got responses from IRC, but still in debugging15:59
rloojiapei: is it only ironic that it can't pull, or any openstack project?16:00
jiapeiany openstack project16:00
jiapeiwe tried to pull cinder, but failed too16:00
jiapeihowever, CLI worked16:01
TheJuliajiapei: Are logs being posted anyplace for test jobs? Maybe one of us can take a look and possibly offer some insight or ideas?16:01
rloojiapei: so before last week, it was working, then it stopped?16:01
TheJuliarloo: I believe they were unable to get to the gerrit event streamer previously16:01
rloooh, so never worked16:01
jiapeirloo: no, it didn't worked before...16:01
rloocan you go on the test machine and manually try it?16:01
jiapeiIf I manually run the command zuul throws exception with, it can pull the code16:03
rloobzzzz... time is up :-(16:03
jiapeiTheJulia: I'll post the latest logs tomorrow16:03
TheJuliajiapei: Thanks16:03
mjturek_like I mentioned last  week - delayed bug day by a week. Will people be able to attend this thursday 1:00 PM UTC?16:03
TheJuliamjturek_: checking calendar16:03
mjturek_(sorry really wanted to squeak that in)16:04
rloo(i'm not avail, esp with midcycle next week)16:04
TheJuliamjturek_: I am available16:04
mjturek_rloo :( TheJulia \o/16:04
TheJuliamjturek_: lets do it and welcome all those avialable16:04
mjturek_awesome, I'll send out an invite today16:04
TheJuliaAnyway, time is up for today. Thanks everyone!16:04
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"16:05
openstackMeeting ended Mon Jul  9 16:05:05 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:05
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-09-15.00.html16:05
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-09-15.00.txt16:05
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-07-09-15.00.log.html16:05
*** hshiina__ has quit IRC16:05
*** jiapei has quit IRC16:05
*** stendulker has quit IRC16:05
*** serlex has quit IRC16:15
*** trown is now known as trown|lunch16:20
TheJuliashift + enter == post in sotryboard is insanely annoying16:21
sambettsTheJulia: esc to nuke all your hard written story is horrible too...16:22
sambettsor accidently clicking off the new story window16:23
* TheJulia takes a little bit of a break16:26
*** derekh has quit IRC16:28
openstackgerritMerged openstack/ironic master: Detect skip version upgrades from version earlier than Pike  https://review.openstack.org/57555916:29
*** hoonetorg has quit IRC16:32
*** hoonetorg has joined #openstack-ironic16:34
*** phuongnh has joined #openstack-ironic16:36
TheJuliabdodd: Would you be up for taking a look at https://review.openstack.org/#/c/561533/ ?16:38
patchbotpatch 561533 - sushy - Add storage disk drive16:38
*** skazi has joined #openstack-ironic16:39
*** phuongnh has quit IRC16:42
TheJuliamkrai: By chance did you see my comment on the grpahical console WIP?16:44
openstackgerritMerged openstack/ironic master: Add 11.0 to release mapping  https://review.openstack.org/58053616:46
openstackgerritMerged openstack/ironic master: Update boot-from-volume feature docs  https://review.openstack.org/57863316:46
openstackgerritMerged openstack/ironic master: Update API version history with release 11.0.0  https://review.openstack.org/58070616:46
openstackgerritMerged openstack/ironic-inspector stable/queens: PXE Filter dnsmasq: manage macs not in ironic  https://review.openstack.org/58093116:46
openstackgerritMerged openstack/ironic-tempest-plugin master: Ensure we create nodes with valid CPU archs  https://review.openstack.org/57986716:46
*** cjloader has joined #openstack-ironic16:48
*** milan has quit IRC16:52
*** cjloader has quit IRC16:53
TheJuliaLooks like there is also an iLO bios interface that has been proposed16:56
TheJuliaI fixed the tag so it shows up as well16:56
openstackgerritMerged openstack/bifrost master: Fix installation of keystone module when using venv  https://review.openstack.org/58012116:58
TheJuliaAnyone on the ilo team around?16:58
*** dougsz has quit IRC17:01
*** willm has quit IRC17:02
*** cjloader has joined #openstack-ironic17:03
*** skazi has quit IRC17:05
*** mgoddard has quit IRC17:05
*** mbeierl has quit IRC17:06
*** skazi has joined #openstack-ironic17:06
*** cjloader has quit IRC17:07
*** Goneri has quit IRC17:14
*** cjloader has joined #openstack-ironic17:14
*** tesseract has quit IRC17:15
*** skazi has quit IRC17:19
*** cjloader has quit IRC17:19
bdoddTheJulia: Yes, I can take a look at change 56153317:19
*** trown|lunch is now known as trown17:23
*** sambetts is now known as sambetts|afk17:25
sambetts|afknight all17:25
TheJuliasambetts|afk: replied to https://review.openstack.org/#/c/578512/4 and marked -1 as a result. Thank you for raising that point, and goodnight17:29
patchbotpatch 578512 - ironic - Reboot server after 10 tries17:29
TheJuliabdodd: thanks!17:30
*** sambetts|afk has quit IRC17:30
*** mbeierl has joined #openstack-ironic17:30
*** sambetts_ has joined #openstack-ironic17:31
*** cjloader has joined #openstack-ironic17:32
*** cjloader has quit IRC17:37
*** mgoddard has joined #openstack-ironic17:45
TheJuliaRegarding midcycle call... Do we want to try and do SIP, or do we want to use bluejeans?17:47
* TheJulia should have asked earlier, but $omgallthethings17:48
TheJuliaAnyway, with that, I'm going to go take care of a few things since I'm only working a half day today.  Thanks everyone!17:48
*** cjloader has joined #openstack-ironic17:50
rlooTheJulia: bluejeans worked well. SIP had issues. I don't recall anyone complaining about bluejeans.17:54
*** cjloader has quit IRC17:54
*** skazi has joined #openstack-ironic17:56
*** mjura has quit IRC18:03
*** skazi has quit IRC18:24
*** cjloader has joined #openstack-ironic18:32
*** mgoddard has quit IRC18:34
*** cjloader has quit IRC18:37
*** dprince has quit IRC19:08
*** murrdoc has joined #openstack-ironic19:12
*** luzC has joined #openstack-ironic19:18
*** jaganathan has quit IRC19:19
*** Will_ has joined #openstack-ironic19:22
*** Will_ is now known as Guest1237819:23
TheJuliarloo: ack, thanks!19:24
*** jaganathan has joined #openstack-ironic19:30
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add conductor_group field to nodes and conductors tables  https://review.openstack.org/57962619:31
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add conductor_group field to config, node and conductor objects  https://review.openstack.org/57962719:31
openstackgerritJim Rollenhagen proposed openstack/ironic master: Use conductor group for hash ring calculations  https://review.openstack.org/57968419:31
*** cjloader has joined #openstack-ironic19:32
*** ajya3 has quit IRC19:32
*** dprince has joined #openstack-ironic19:33
*** cjloader has quit IRC19:37
*** hkominos_ has joined #openstack-ironic19:48
hkominos_HI all. I have question regarding ironic FS creation. What does ironic expect from a qcow image in order to boot it and deploy it as an openstack node. I am specifically interested in the disk for cloud-config19:50
*** cjloader has joined #openstack-ironic20:32
*** cjloader has quit IRC20:36
*** skazi has joined #openstack-ironic20:47
*** cjloader has joined #openstack-ironic20:49
*** cjloader has quit IRC20:54
*** mjturek_ has quit IRC20:59
*** baha has quit IRC21:02
*** skazi has quit IRC21:08
*** cjloader has joined #openstack-ironic21:32
*** jtomasek has quit IRC21:37
*** cjloader has quit IRC21:37
openstackgerritMerged openstack/ironic master: Bump osprofiler minimum requirement to 1.5.0  https://review.openstack.org/58067321:50
*** cdearborn_ has joined #openstack-ironic22:06
*** bfournie has quit IRC22:08
*** bfournie has joined #openstack-ironic22:08
*** cdearborn has quit IRC22:10
*** cdearborn_ has quit IRC22:12
*** bfournie has quit IRC22:13
*** jaganathan has quit IRC22:15
*** rcernin has joined #openstack-ironic22:20
*** threestrands_ has joined #openstack-ironic22:20
*** threestrands_ has quit IRC22:20
*** threestrands_ has joined #openstack-ironic22:20
*** jappleii__ has joined #openstack-ironic22:23
*** jappleii__ has quit IRC22:24
*** jappleii__ has joined #openstack-ironic22:25
*** threestrands_ has quit IRC22:26
*** murrdoc has quit IRC22:26
*** jaganathan has joined #openstack-ironic22:27
*** murrdoc has joined #openstack-ironic22:27
*** Guest12378 has quit IRC22:30
*** cjloader has joined #openstack-ironic22:32
*** dprince has quit IRC22:33
*** cjloader has quit IRC22:36
*** rybridges has quit IRC22:36
*** murrdoc has quit IRC22:44
*** mjturek has joined #openstack-ironic22:48
*** luzC has quit IRC22:49
*** mjturek has quit IRC22:51
*** luzC has joined #openstack-ironic22:53
*** mjturek has joined #openstack-ironic22:54
*** pmannidi has joined #openstack-ironic23:00
*** mjturek has quit IRC23:02
*** mjturek has joined #openstack-ironic23:06
*** yolanda_ has joined #openstack-ironic23:06
*** mjturek has quit IRC23:07
*** trungnv has quit IRC23:09
*** yolanda has quit IRC23:09
*** trungnv has joined #openstack-ironic23:09
*** ianychoi_ has joined #openstack-ironic23:21
*** rook has quit IRC23:22
*** gyee has quit IRC23:25
*** ianychoi has quit IRC23:25
*** rook has joined #openstack-ironic23:25
*** rook is now known as Guest6789823:25
*** sambetts_ has quit IRC23:26
*** zshi has quit IRC23:28
*** gyee has joined #openstack-ironic23:28
*** bfournie has joined #openstack-ironic23:28
*** mjturek has joined #openstack-ironic23:29
*** sambetts_ has joined #openstack-ironic23:29
*** cjloader has joined #openstack-ironic23:32
*** mjturek has quit IRC23:34
*** cjloader has quit IRC23:36
*** rpioso is now known as rpioso|afk23:42
*** zshi has joined #openstack-ironic23:43

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