Monday, 2018-10-22

*** MattMan_ has quit IRC01:46
*** MattMan_ has joined #openstack-ironic01:47
*** tiendc has joined #openstack-ironic01:47
*** rnoriega has quit IRC02:36
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent master: Parallel erase disk devices  https://review.openstack.org/34672303:35
*** zhangfei has joined #openstack-ironic03:54
*** zhangfei has quit IRC04:10
openstackgerritKaifeng Wang proposed openstack/ironic master: Passing thread pool size to IPA  https://review.openstack.org/60886804:15
*** hoonetorg has quit IRC04:48
*** skazi has quit IRC04:51
*** hoonetorg has joined #openstack-ironic05:01
*** hoonetorg has quit IRC05:12
openstackgerritparesh sao proposed openstack/sushy master: Requests session keyword arguments for sushy connector  https://review.openstack.org/60780905:20
*** hoonetorg has joined #openstack-ironic05:33
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent master: Parallel erase disk devices  https://review.openstack.org/34672305:59
*** skazi has joined #openstack-ironic06:14
*** pcaruana has joined #openstack-ironic06:21
*** adrianc has joined #openstack-ironic06:26
*** adrianc_ has joined #openstack-ironic06:27
*** adrianc has quit IRC06:31
*** adrianc has joined #openstack-ironic06:31
*** iurygregory has joined #openstack-ironic06:35
iurygregorygood morning06:41
*** e0ne has joined #openstack-ironic06:45
*** pcaruana has quit IRC06:58
*** rcernin has quit IRC06:59
*** rcernin has joined #openstack-ironic07:05
*** rcernin has quit IRC07:07
openstackgerritDebayan Ray proposed openstack/sushy master: Introduce ``cache_it`` and ``cache_clear``  https://review.openstack.org/58164507:08
jiapeiGood morning Ironicers07:11
*** zhangfei has joined #openstack-ironic07:12
*** jtomasek has joined #openstack-ironic07:13
*** pcaruana has joined #openstack-ironic07:13
*** moshele has joined #openstack-ironic07:16
jiapeiI just tried to run tempest with "tox -e all-plugin -- ironic" according to the doc https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#running-tempest-tests , but got InvocatinError, did I miss something? Log is at paste.openstack.org/show/732569/07:19
*** skazi has quit IRC07:22
openstackgerritDigambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/60998207:23
*** adrianc has quit IRC07:25
*** adrianc has joined #openstack-ironic07:26
*** adrianc__ has joined #openstack-ironic07:26
*** adrianc__ has quit IRC07:29
*** moshele has quit IRC07:30
iurygregoryjiapei, i dont have a setup to test, but i was looking at CI how they run tempest tempest run --regex ironic_tempest_plugin.tests.api --concurrency=1 --black-regex=07:31
iurygregorypython2 http://logs.openstack.org/68/608868/2/check/ironic-tempest-dsvm-functional-python2/8252984/job-output.txt.gz#_2018-10-22_04_38_17_96608107:31
jiapeiiurygregory, I just tried input "tempest run --regex ironic_tempest_plugin.tests.api --concurrency=1 --black-regex=", still not working, :(07:36
iurygregorywow =(07:37
iurygregoryyou have a devstack setup?07:37
jiapeiyes, it's deployed by devstack07:37
iurygregorymaster version or a stable one?07:37
jiapeiand all the services (nova, ironic,...) working07:38
jiapeiI pulled the master07:38
jiapeihopefully the branch shouldn't influence tempest07:38
iurygregoryit shouldnt07:39
iurygregorymaybe the doc is not updated and something changed07:39
openstackgerritDigambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/60998207:48
*** skazi has joined #openstack-ironic07:54
openstackgerritDigambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/60998207:54
*** rpittau has joined #openstack-ironic08:02
*** S4ren has joined #openstack-ironic08:07
*** dougsz has joined #openstack-ironic08:11
*** zhangfei has quit IRC08:20
*** jaganathan has joined #openstack-ironic08:26
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180808:38
*** adrianc_ has quit IRC08:39
*** adrianc has quit IRC08:39
*** stendulker has joined #openstack-ironic08:44
openstackgerritKaifeng Wang proposed openstack/ironic master: Enhanced checksum support  https://review.openstack.org/61089408:47
openstackgerritMadhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface  https://review.openstack.org/54735608:49
*** moshele has joined #openstack-ironic09:10
*** derekh has joined #openstack-ironic09:11
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180809:19
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Support IPv6 address when validating interfaces  https://review.openstack.org/53613909:21
*** stendulker has quit IRC09:24
*** dtantsur|afk is now known as dtantsur09:30
sambetts|afkjiapei, iurygregory: it looks like tempest was unable to find the ironic tests, do you have the ironic-tempest-plugin python package installed??09:30
*** sambetts|afk is now known as sambetts09:30
dtantsurmorning ironic09:30
iurygregorymorning09:31
sambettso/ dtantsur09:31
iurygregorysambetts, i just setup the devstack here let me check09:31
jiapeimorning sambetts|afk, dtantsur09:31
jiapeisambetts, it seems the tempest is enabled09:32
jiapeisince I have "enable_service tempest" in local.conf09:33
iurygregorythe dist-packages have the ironic_tempest_plugin folder too09:33
*** zzzeek_ has quit IRC09:37
sambettsjiapei: if you do `pip freeze` it ironic-tempest-plugin in the list?09:39
*** dsneddon has quit IRC09:39
*** stendulker_ has joined #openstack-ironic09:39
*** zzzeek has joined #openstack-ironic09:40
jiapeiit's not in the pip freeze list...09:41
jiapeibut what's strange is there is "-e git://git.openstack.org/openstack/tempest.git@f46eebed9d0aaeb9e31b7afb1bf8f552ca65ed19#egg=tempest" in the "pip freeze" result09:41
sambettsI think devstack installs the plugins only into the virtualenv that tempest uses09:42
sambettsso I think you need to use `tox -e all -- ironic` now09:42
sambettsbecause all-plugin is deprecated09:42
jiapeilet me try it09:43
iurygregorywe need to update the docs lol09:44
jiapeio/, it begins running09:44
sambettsyeah, looks like a doc update is required09:44
sambettsas tempest have changed their process09:44
iurygregoryalso there is a bronken link for the plugin XD09:44
*** bnemec has joined #openstack-ironic09:45
jiapeiiurygregory, which link?09:45
sambettsyup, that link is broken since the code moved into its own repo09:45
jiapeiyes, I find it iurygregory09:45
iurygregory"To run all tests from the Ironic plugin"09:46
*** hoonetorg has quit IRC09:51
*** bnemec has quit IRC09:53
*** iurygregory is now known as iury|lunch09:56
jiapeisambetts, so the new link is https://git.openstack.org/cgit/openstack/ironic-tempest-plugin/tree/?09:57
sambettsyeah thats the one09:58
jiapeiIt seems I can send a patch to fix these minors :)09:58
jiapeiper iurygregory's advice09:58
*** hoonetorg has joined #openstack-ironic10:08
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180810:10
*** dnuka has joined #openstack-ironic10:14
openstackgerritMerged openstack/ironic-specs master: No IPA to conductor communication  https://review.openstack.org/21220610:20
openstackgerritIlya Etingof proposed openstack/sushy master: Untie `Connection` from authenticator on `.close()`  https://review.openstack.org/61109510:24
*** adrianc has joined #openstack-ironic10:32
*** adrianc_ has joined #openstack-ironic10:32
*** dnuka has quit IRC10:34
*** iury|lunch is now known as iurygregory10:35
*** moshele has quit IRC10:48
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180810:49
*** moshele has joined #openstack-ironic10:54
*** stendulker_ has quit IRC10:56
openstackgerritPierre Riteau proposed openstack/ironic-python-agent stable/queens: Fix BMC IP address detection on CoreOS images  https://review.openstack.org/61234611:12
openstackgerritPierre Riteau proposed openstack/ironic-python-agent stable/rocky: Fix BMC IP address detection on CoreOS images  https://review.openstack.org/61234711:12
*** rh-jelabarre has joined #openstack-ironic11:15
*** priteau has joined #openstack-ironic11:19
*** pcaruana has quit IRC11:26
*** hjensas has joined #openstack-ironic11:31
jrollmorning everyone11:34
dtantsurmorning jroll11:37
*** dtantsur is now known as dtantsur|brb11:37
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180811:47
*** pcaruana has joined #openstack-ironic11:48
*** tiendc has quit IRC12:03
*** pcaruana has quit IRC12:12
*** pcaruana has joined #openstack-ironic12:12
*** skazi has quit IRC12:15
*** mjturek has joined #openstack-ironic12:16
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180812:17
*** rnoriega has joined #openstack-ironic12:19
*** jaypipes has joined #openstack-ironic12:27
TheJuliagood morning everyone12:35
*** jcoufal has joined #openstack-ironic12:36
*** dnuka has joined #openstack-ironic12:36
etingofo/ TheJulia & jroll12:36
jroll\o12:37
iurygregorygood morning TheJulia and jroll =)12:39
* TheJulia grumbles because it looks like one of her monitors is dead12:40
*** jesusaur has quit IRC12:46
*** jroll has quit IRC12:53
*** jroll has joined #openstack-ironic12:54
TheJuliasweet, the no agent comm spec landed12:55
*** edleafe has joined #openstack-ironic12:55
TheJuliaBonus points for anyone who reviews 59667112:59
jroll\o/13:02
*** dtantsur|brb is now known as dtantsur13:04
jroll+W13:05
TheJuliawoot13:05
dtantsurmorning TheJulia13:05
*** rpioso|afk is now known as rpioso13:09
rpiosoGood morning, all13:10
dtantsurhey rpioso13:10
rpiosodtantsur: :)13:10
TheJuliaGood morning o/13:11
rpiosoTheJulia: Morning :)13:11
*** skazi has joined #openstack-ironic13:11
* TheJulia places the gentle reminder that the white board loves to be updated https://etherpad.openstack.org/p/IronicWhiteBoard :)13:12
*** hwoarang has quit IRC13:12
rpiosodtantsur: We've been talking about adding the Redfish management and power h/w interface implementations to the idrac h/w type's supported interfaces. More recently, the idea of adding the IPMI's h/w interfaces implementations came up. Would all of the above be aligned with the intent of ironic's driver composition reform?13:15
*** jesusaur has joined #openstack-ironic13:15
*** hwoarang has joined #openstack-ironic13:16
dtantsurrpioso: I think it does, if it makes sense for your hardware13:17
*** rloo has joined #openstack-ironic13:19
dtantsurfolks, I could use some reviews on https://review.openstack.org/605707 and https://review.openstack.org/61001313:19
patchbotpatch 605707 - ironic-python-agent - Allow streaming raw partition images - 3 patch sets13:19
patchbotpatch 610013 - ironic - Allow streaming raw partition images - 5 patch sets13:19
rpiosodtantsur: The iDRAC supports all of that, and the Dell EMC 3d Party CI has been testing the idrac h/w type, as is, along with the ipmi and redfish h/w types.13:20
* TheJulia will add them to the list for the week13:20
rpiosoI'm not clear on what you mean by "if it makes sense for your hardware."13:20
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180813:21
dtantsurrpioso: it's up to you to define :) if e.g. you want people to use idrac power with ipmi management13:22
rpiosodtantsur: Gotcha :)13:23
rpiosoThank you.13:23
*** adrianc_ has quit IRC13:31
*** adrianc has quit IRC13:31
TheJuliaipa in general needs some review attention13:32
*** munimeha1 has joined #openstack-ironic13:34
TheJuliadtantsur: are you planning on working on https://review.openstack.org/#/c/587041/ this week?13:37
patchbotpatch 587041 - ironic-inspector - Remove deprecated authentication options for acces... - 4 patch sets13:37
*** hjensas has quit IRC13:37
dtantsurTheJulia: not sure. this is a background task with quite low priority for me.13:41
TheJuliaunderstood13:42
*** moshele has quit IRC13:52
*** adrianc_ has joined #openstack-ironic13:56
*** adrianc has joined #openstack-ironic13:56
*** skazi has quit IRC13:57
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180814:02
TheJuliahwoarang: thanks :)14:02
hwoarangTheJulia: not yet...14:02
TheJuliadoh :(14:02
*** baha has joined #openstack-ironic14:08
openstackgerritMerged openstack/bifrost master: Fix bug in dnsmasq config  https://review.openstack.org/61166314:08
openstackgerritJulia Kreger proposed openstack/ironic master: Wrap up PXE private method to pxe_utils move  https://review.openstack.org/60533214:10
openstackgerritJulia Kreger proposed openstack/ironic master: Create base pxe class  https://review.openstack.org/60878614:10
*** SteelyDan is now known as dansmith14:11
openstackgerritJulia Kreger proposed openstack/ironic master: Fix DHCPv6 support  https://review.openstack.org/61160014:11
openstackgerritJulia Kreger proposed openstack/ironic master: Cleanup of remaining pxe focused is_ipxe_enabled  https://review.openstack.org/61193314:11
*** kaifeng has joined #openstack-ironic14:18
openstackgerritMerged openstack/bifrost master: playbooks: Convert venv_python_path to proper role  https://review.openstack.org/61089814:18
openstackgerritJulia Kreger proposed openstack/ironic master: Add note to pxe configuration doc  https://review.openstack.org/58340914:22
dtantsurjiapei: hi! could you check out responses on https://review.openstack.org/#/c/611656/ please? do they make sense to you?14:25
patchbotpatch 611656 - ironic - Refactor API code for checking microversions - 1 patch set14:25
jiapeidtantsur: Yes, I'll reply to it :)14:26
dtantsurthanks!14:28
dtantsurTheJulia: this patch ^^ is another candidate for this week, I think14:28
dtantsurI'd prefer to land it before it gets into a conflict with the next API patch14:28
TheJulia++14:28
*** skazi has joined #openstack-ironic14:34
*** dnuka has quit IRC14:37
jrolldtantsur: I love that patch, but why are we removing tests?14:44
jrolloh never mind14:44
* jroll is dumb14:44
jrollplease don't put that on priorities this week :P14:44
dtantsurheh14:45
openstackgerritjiapei proposed openstack/ironic master: Fix ironic developer quickstart document  https://review.openstack.org/61241014:50
TheJuliajroll: I wondered the same thing the first time I looked at it. /f in web view revealed the truth14:52
jrollyep14:53
*** hshiina has joined #openstack-ironic14:56
*** hjensas has joined #openstack-ironic14:57
*** stendulker has joined #openstack-ironic14:59
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Oct 22 15:00:07 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
TheJuliao/15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
iurygregoryo/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
TheJuliaGood morning everyone!15:00
etingofo/15:00
jiapeio/15:00
stendulkero/15:00
bdoddo/15:00
jroll\o15:00
hshiinao/15:00
dtantsuro/15:00
rlooo/15:00
TheJuliaOur agenda for this week can be found on the wiki.15:00
kaifengo/15:00
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:00
TheJuliaIt looks like there is nothing beyond our normal meeting agenda today, so hopefully we'll be able to wrap up this meeting fairly quickly.15:01
TheJulia#topic Announcements/Reminders15:01
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:01
TheJulia#info We're a few weeks out from the summit in Berlin. If there is anything that anyone wishes to have raised during the Project Onboarding, Project Update, or during the feedback session, please let me know.15:02
* dtantsur wishes to raise a pint15:02
TheJuliadtantsur: ?15:02
jrollheh15:02
* TheJulia is all for a pint, but it is also 9am local :)15:02
dtantsurTheJulia: in Berlin, I mean :) Isn't it a great way to onboard people?15:03
TheJuliadtantsur: ++++15:03
iurygregorylol15:03
jrollcan leave this for open discussion, but should we put together a meetup?15:03
hjensaso/15:03
dtantsurjroll: I was about to ask the same15:03
TheJuliaAnyone have anything else to raise or announce today?15:03
rlooein stein I'm guessing? :)15:03
dtantsurbut yeah, let's wait for the open discussion15:03
TheJuliaOkay, moving on!15:04
stendulkerAll iLO 3rd party CI jobs are moved to python3 except for one that would run on python2.15:04
mjtureko/15:04
dtantsurstendulker: nice!15:04
rloonice stendulker! :)15:04
TheJuliastendulker: That is awesome news, could you share an example?15:04
TheJulia#topic Review action items from previous meeting15:05
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:05
TheJulia#info No action items from the last meeting15:05
TheJulia#topic Review subteam status reports15:05
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:05
stendulkerTheJulia: https://review.openstack.org/#/c/608868/15:05
patchbotpatch 608868 - ironic - Passing thread pool size to IPA - 2 patch sets15:05
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:05
TheJuliaStarting around line 17215:05
rloodtantsur: yup, i care about the bug numbers :)15:06
dtantsurokay, recorded, I'll keep doing it :)15:06
rloowhat i don't understand is why so many are untriaged. is it cuz we're still catching up on storyboard?15:06
dtantsurare we actively triaging them?15:07
dtantsurif no, the growth is expected15:07
TheJuliaI'm only getting to them a couple times a month, but they don't seem to grow that fast15:07
* etingof reminds to tag those with `ironic-triaged`15:07
TheJuliaI guess it would likely be good for me to try and compare the counting logic, maybe I've been setting the wrong flags or something15:07
rloowho's doing the python3 first stuff? (no name there)15:08
TheJuliarloo: I think that is largely falling to the PTL15:08
rlooTheJulia: oh :-( Any volunteers for that?15:09
TheJuliaDerek and myself15:09
TheJuliaI'm really impressed with just the number of things we've got moving forward right now15:10
TheJuliajroll: do you feel that there is any action on our part at this moment for nova supporting conductor grouping in nova-compute processes?15:11
jrollTheJulia: no, just the spec back and forth right now15:11
jrollI'll pull people in ad hoc if I need eyes :)15:11
TheJuliajroll: ack, thanks15:12
*** pcaruana has quit IRC15:12
rlooTheJulia: can we amalgamate/remove some of the stuff under Rocky Priorities (L267) ?15:12
TheJuliaI believe mgoddard is attempting to play the game and looking at pretty buildings in far away places, so I suspect we should just try and review the updated deploy templates spec this week15:13
rlooheh, i think there was a graphical console update from mkrai, under the Rocky part. i didn't catch what it was before it was deleted. probably not a big deal.15:13
TheJuliaOr so suggests twitter :)15:13
TheJuliarloo: It looked outdated to me15:14
rlooTheJulia: ok15:14
TheJuliahshiina: I think your closer to her timezone and have been reviewing. Would you mind following up with her?15:14
*** e0ne has quit IRC15:15
hshiinaTheJulia, sure15:16
TheJuliahshiina: thanks :)15:16
openstackgerritMerged openstack/ironic master: Add version discovery information to the /v1 endpoint  https://review.openstack.org/59667115:17
dtantsurmordred: ^^^15:17
TheJuliaIs everyone good to proceed?15:18
rloo++15:18
mordred\o/15:18
* mordred does a little dance15:18
* TheJulia notes it is rare for a dancing mordred in channel15:18
TheJuliaMoving on to priorities if there are no objections15:19
TheJulia#topic Priorities for the next week15:19
*** openstack changes topic to "Priorities for the next week (Meeting topic: ironic)"15:19
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:19
TheJuliaStarting at line 9315:19
TheJuliaI'd like to remove graphical console for this next week, we have a number of housekeeping and smaller patches that could use eyes this week.15:20
TheJuliahjensas: any more inspector grenade fixes?15:20
hjensasTheJulia: no, they both merged and the configurable timeout made it in.15:21
rlooi was looking at the redfish oob inspection stuff (L270). is there just one patch that is left to do/review?15:21
* TheJulia guesses there are no objections re graphical console for this week?15:22
TheJuliarloo: I _think_ so...15:22
etingofthere were also sushy changes related to OOB, but most of them were merged I think15:22
mjturek+1 since we're waiting on revisions anyway right?15:22
TheJuliaetingof: does it require a sushy release?15:22
TheJuliamjturek: yes15:22
* TheJulia removes the items for the week15:23
* TheJulia merges in new things15:23
etingofTheJulia, let me take a look re sushy release15:23
TheJuliaI've added kind of the logical continuation of things, which yes is a lot of small patches15:24
TheJuliaI feel like the redfish inspect interface would be good for that list as well, which I think is where rloo was going15:24
rlooTheJulia: yeah, if it was ready for reviews etc. or wait a week if we need to do a sushy release first.15:25
rloowrt streaming raw images -- did 'we' approve the RFE? I don't see any approval. are the patches waiting for comments only, but not merge?15:25
rloohttps://storyboard.openstack.org/#!/story/200380915:26
dtantsurI think it was approved..15:26
jrollI have a +1 on the story15:26
TheJuliajroll +1, just didn't tag it looks like15:26
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180815:26
jrolloops15:26
jrolldone15:27
rloono worries, just administrivia. thx jroll! :)15:27
TheJuliajroll: thanks15:27
TheJuliaI'm good with the list if everyone else is15:27
* rloo wonders if we need a meeting to discuss step retrieval.15:28
TheJuliarloo: maybe, or just during open discussion?15:28
rloowell, i think we need a few people to read the spec first...15:28
TheJulia++15:28
*** e0ne has joined #openstack-ironic15:28
openstackgerritManuel Buil proposed openstack/bifrost stable/rocky: Fix bug in dnsmasq config  https://review.openstack.org/61242115:29
rlooTheJulia: we could sched in for another mon meeting15:29
TheJuliarloo: I like that idea!15:29
TheJuliadoes anyone have any objections to scheduling for next monday?15:29
rlooguess i was thinking a video/audio one might be more productive, but we can try irc first.15:29
rloo(who cares about it?)15:30
TheJuliarloo: yeah, we've not been able to really reach consensus on this before, but I was kind of hoping minimalist with the new spec so maybe that will help15:30
* TheJulia cares about it15:30
rlooit is a tough one wrt api etc.15:30
dtantsuryeah, I have some problems with the new one.. ++ for a meeting15:31
TheJuliaOkay, I'll put it on the agenda for next week.15:31
rlooanything else that is a bit contentious?15:31
*** e0ne has quit IRC15:32
rloolet's try to remove blockers (besides 'we need reviews') :)15:32
TheJuliarloo: neutron callbacks, but we need the spec revised first15:32
rlooTheJulia: ok, maybe we're good for this week then :)15:33
TheJuliaI think we're good to move to Open Discussion at this time, any objections?15:33
TheJuliaIs there anything anyone else wishes to raise regarding the priorities for this next week?15:34
* kaifeng has one15:35
* TheJulia takes silence as agreement to move on15:35
*** sthussey has joined #openstack-ironic15:35
TheJuliakaifeng: go ahead15:35
kaifenghttps://review.openstack.org/59007615:35
patchbotpatch 590076 - ironic-inspector - Support manipulating iptables filter under IPv6 ne... - 2 patch sets15:35
kaifengthis patch has some connection with the ipv6 stuff15:35
dtantsurkaifeng: quick ques: does it assume either ipv4 or ipv6 stack but not both?15:36
dtantsurI'm not sure it's a blocker, just trying to understand15:36
kaifengdtantsur, not a blocker15:36
TheJuliaat a glance, it appears we assume one, not both15:36
kaifengyes, ipv4 or ipv6, can't be both15:36
dtantsuris it a reasonable limitation?15:37
* dtantsur suspects yes, but has to ask15:37
TheJuliawe have a hardcoded switch in the conductor code, so I think it is15:37
kaifengis there any way to support both? they use different commands15:37
TheJuliakaifeng: I added to the ironic-inspector list15:38
kaifengexcept we bind multiple interfaces15:38
*** cdearborn has joined #openstack-ironic15:38
TheJuliaYeah, pondering is required15:38
kaifengTheJulia, thanks!15:38
TheJuliaAnyway, Open Discussion time!15:38
TheJulia#topic Open Discussion15:38
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:39
TheJuliaso, raising steins! :)15:39
jrollany reason we couldn't just have the iptables call do calls both for iptables and ip6tables?15:39
TheJuliaI suspect we don't presently have enough information15:40
* kaifeng thanks TheJulia for bringing inspector api/woker from the ground again ;)15:40
TheJuliaI guess we might be able to record the socket type the connection came inbound on15:40
TheJuliamaybe15:40
TheJulia_maybe_15:40
jrollwell15:41
jrolljust check the dnsmasq interface for v4/v6/both, and issue iptables/ip6tables/both as needed15:41
jroll> self.interface = CONF.iptables.dnsmasq_interface15:41
TheJuliait is mac based....15:42
TheJuliaso maybe it should just be a v4, v6, both option15:43
jrollI'll just comment on the patch so we can get to happier things :)15:43
TheJulia\o/15:43
* TheJulia wonders if she needs to install beer taps in IRC15:45
iurygregory++15:45
iurygregory=P15:45
jrollso, meetup in berlin?15:45
dtantsurwho is more or less confirmed to go to Berlin?15:46
TheJuliaI'll be there, I know of dtantsur. Anyone else?15:46
jrollI'm going15:46
jrollas is penick15:46
dtantsurIIRC sambetts15:46
dtantsuretingof: do you have an idea if you're coming or not?15:47
jrollyep, sam will be there15:47
TheJuliadtantsur: know any place that would be good for us to get dinner?15:47
etingofdtantsur, no idea yes, but I hope that I come15:47
rloomaybe email, set up etherpad like we do for ptgs. I can't remember what you normally do for summit/forum...?15:47
etingof*yet15:47
dtantsurTheJulia: I have one idea, not sure what they think about group reservations though15:48
jrollprobably doodle would be good15:48
rloodo they have bier gartens in berlin or is that just south germany?15:48
dtantsuryeah15:48
dtantsurrloo: they do ;)15:48
rloothat might be a good experience then dtantsur!15:48
dtantsurbut a proper Biergarten will not have much food, and unlikely reservations15:48
TheJuliaI guess create a doodle and we can go from there. dtantsur do you want to inquire with the location you have in mind?15:48
dtantsurI'm looking at a middle ground15:49
dtantsurTheJulia: I'll try, given my poor language skills :)15:49
rlooGerman food. hmm.. I suspect the bier is the best thing there...15:49
TheJuliaHeh15:49
etingofaren't those bier gartens typically outdoors...15:50
dtantsurdepends on your tastes :) sausages, cabbage, schnitzel..15:50
rloosome are only outdoors. dunno about what is in berlin.15:50
rlooand pickled salads :)15:50
dtantsuryeah, that's another issue with a pure beer garden15:50
rloohave fun wherever you end up !15:50
etingofkartoffelsalat15:51
dtantsurI'm thinking about https://www.bier-genuss.berlin/en/lindenbraeu-am-potsdamer-platz15:51
rlooif there aren't that many of you, you might be ok w/o a reservation.15:51
*** bnemec has joined #openstack-ironic15:51
* TheJulia suspects we're done with the meeting for today15:51
dtantsurright, the place is big15:51
dtantsurit has semi-outside seating as well15:51
jrollI was hungry enough without all this food talk :P15:52
TheJuliaThanks everyone for attending today!15:52
* etingof is aware of a delicious ice cream booth at potsdamer platz15:52
dtantsurwho is doing the doodle, me?15:52
jrollI guess so :P15:52
iurygregoryprob yes dtantsur15:52
rlooaction item!15:52
dtantsurpotsdamer platz is not bad at all :)15:52
TheJuliadtantsur: I can, but if you want to :)15:52
JayFjroll: as a note, ip6tables rules *can* be created and edited on machines that don't have IPv6 connectivity.15:52
dtantsurTheJulia: I'll do it15:52
TheJuliadtantsur: thanks15:52
TheJuliaJayF: Muchas Gracias15:53
JayFjroll: I didn't get whole context but saw iptables / ip6tables :P15:53
* dtantsur wonders which keyword was highlighted for JayF this time :)15:53
dtantsurah, these two :)15:53
JayFI am at a company retreat... just watching IRC :)15:53
jrollJayF: also good point, thanks :)15:53
TheJulia#action dtantsur to create a doodle for Summit/Forum gathering.15:54
dtantsursend it to the ML then?15:55
jrollyeah15:55
TheJuliadtantsur: yeah15:55
dtantsurcool15:55
TheJuliaThanks everyone!15:55
jrollthanks :)15:55
TheJulia#endmeeting15:55
*** 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."15:55
openstackMeeting ended Mon Oct 22 15:55:54 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:55
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-22-15.00.html15:55
*** adrianc_ has quit IRC15:55
*** adrianc has quit IRC15:55
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-22-15.00.txt15:55
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-10-22-15.00.log.html15:55
*** hshiina has quit IRC15:57
kaifengJayF: so it means we can execute them both anyway?15:57
dtantsurTheJulia: when do the talks end at the Forum, 6?15:57
TheJuliakaifeng: we might want to check if there is os support for each and then execute from there15:57
TheJuliadtantsur: I think they are scheduled to 6:10, double checking15:58
JayFkaifeng: I'd be ready to catch if `ip6tables` wasn't installed or no ipv6 modules loaded, but doing it just to be safe is the right thing15:58
TheJuliaIt could also be compiled in statically15:58
TheJuliamarketplace mixer goes to 7:30 on tuesday15:58
kaifengi think that can be done during module init15:58
kaifengtest if there is a ip6tables command15:59
dtantsurTheJulia: okay, so excluding Tue?15:59
dtantsurany other potential conflicts?15:59
*** bnemec has quit IRC15:59
TheJuliaThe latest I see ending is 5:5515:59
TheJuliadtantsur: yeah, excluding Tuesday or doing tuesday after the mixer would be good15:59
rloodtantsur: qq about https://review.openstack.org/#/c/495752/. Are we good with just getting an api-ref that is half-decent?15:59
patchbotpatch 495752 - ironic-inspector - Add API reference - 8 patch sets15:59
dtantsurokay, so 7pm sounds like good time15:59
dtantsurrloo: depends on your definition of half-decent..16:00
TheJuliaWe should also keep in mind that we can iterate and improve16:00
rloodtantsur: i don't have time to fully review/comment on all the grammatical issues i see. and i don't have time to do a followup myself, to fix things.16:00
*** gyee has joined #openstack-ironic16:00
*** stendulker has quit IRC16:00
TheJuliaThere is no rule saying we have to do it immediately :)16:00
dtantsurrloo: I'm not sure what kind of answer you expect.. I prefer some docs to no docs.16:02
dtantsurTheJulia: https://doodle.com/poll/iw5992px765nthde (just to check I'm not doing something stupid before sending it out)16:02
kaifengapi-ref is a headache but worth to have :)16:02
TheJuliadtantsur: LGTM16:03
kaifengi often see some api-ref error code patches, oh may, you never get them perfect16:03
dtantsurI'd also prefer to iterate on non-critical stuff rather than try to land all at once perfect16:03
rloodtantsur: i've +2 but I won't +A. Will leave it for you or others to do that :)16:04
* dtantsur appoints TheJulia :D16:04
TheJuliadtantsur: This aligns with community standards :)16:04
* TheJulia will review it after #currentcall on other screen :(16:04
* TheJulia looks up when she is supposed to land in berlin16:05
kaifengthanks rloo, I will take your suggestion in, whether in this patch or in a follow-up16:06
dtantsurTheJulia: email sent16:06
TheJuliawoot, I land at 5:15 in Berlin.... so 7PM on sunday actually seems like a good idea adjusting time wise16:07
* TheJulia will leave zombie makeup at home16:08
dtantsurheh16:08
dtantsuryeah, I decided to include adjacent days in case the main days are too busy16:08
TheJuliaI'm only flying in on Sunday because I have meetings Monday16:09
rlookaifeng: yw, some of my comments are questions or personal opinion. i leave it up to you to think about :)16:16
kaifengrloo: sure :)16:17
* kaifeng says night to everyone and continue his adventure in the dreamland.16:18
*** kaifeng has quit IRC16:19
TheJulia:)16:20
*** baha has quit IRC16:30
*** cdearborn has quit IRC16:32
*** jcoufal has quit IRC16:35
dtantsurjroll, sambetts, in case you missed: https://doodle.com/poll/iw5992px765nthde16:45
jrolldtantsur: yep, literally just filled it in :)16:46
dtantsurheh16:47
jrollfood was first16:47
dtantsurfair enough16:47
*** e0ne has joined #openstack-ironic16:52
* etingof just figured that sushy emulator is very slow16:52
dtantsuretingof: what did you try to do with it? :)16:54
etingofI stand up sushy-tools alongside openstackbmc script in OVB16:55
etingofsurprisingly, openstackbmc is much faster because sushy-tools do many nova calls for a single power status query16:56
*** dougsz has quit IRC16:57
TheJuliaAuth, get object and current status, request change in object... adds up16:58
openstackgerritMerged openstack/ironic master: Refactor API code for checking microversions  https://review.openstack.org/61165616:58
openstackgerritMerged openstack/ironic master: update "auth_url" in documents  https://review.openstack.org/56809016:58
etingofipmitool does IPMI auth as well, I guess16:59
*** derekh has quit IRC17:00
*** mbeierl has joined #openstack-ironic17:01
*** moshele has joined #openstack-ironic17:03
openstackgerritDmitry Tantsur proposed openstack/ironic master: Support for protecting nodes from undeploying and rebuilding  https://review.openstack.org/61166217:03
*** e0ne has quit IRC17:06
*** dtantsur is now known as dtantsur|afk17:06
dtantsur|afkg'night17:06
*** sambetts is now known as sambetts|afk17:18
sambetts|afknight all17:18
*** bnemec has joined #openstack-ironic17:19
*** baha has joined #openstack-ironic17:25
*** dsneddon has joined #openstack-ironic17:25
*** S4ren has quit IRC17:27
mosheleTheJulia: hi, I uploaded new PS https://review.openstack.org/#/c/582767/, please review when you have time :)17:27
patchbotpatch 582767 - ironic-specs - Add Support for Smart NIC - 11 patch sets17:27
TheJuliamoshele: thanks!17:27
*** pcaruana has joined #openstack-ironic17:27
*** e0ne has joined #openstack-ironic17:30
*** gabys has joined #openstack-ironic17:32
*** hjensas has quit IRC17:33
*** irclogbot_3 has joined #openstack-ironic17:34
*** dsneddon has quit IRC17:38
*** dsneddon has joined #openstack-ironic17:39
*** moshele has quit IRC17:52
*** jcoufal has joined #openstack-ironic17:54
*** irclogbot_3 has quit IRC17:56
*** pcaruana has quit IRC17:57
*** moshele has joined #openstack-ironic18:03
*** irclogbot_3 has joined #openstack-ironic18:06
*** gabys has quit IRC18:12
*** gabys has joined #openstack-ironic18:12
*** gabys has quit IRC18:17
*** hjensas has joined #openstack-ironic18:22
*** e0ne has quit IRC18:24
*** hjensas has quit IRC18:26
*** hjensas has joined #openstack-ironic18:42
*** moshele has quit IRC18:46
TheJuliaetingof: what was the verdict on a sushy release?18:47
openstackgerritMerged openstack/ironic-inspector master: Rearrange conf and policy configuration file  https://review.openstack.org/60958418:54
*** moshele has joined #openstack-ironic18:58
openstackgerritMerged openstack/ironic master: Remove oneview drivers  https://review.openstack.org/59905118:58
*** moshele has quit IRC19:05
openstackgerritMerged openstack/ironic master: Enable configuration of conversion flags for iscsi  https://review.openstack.org/60479419:07
openstackgerritMerged openstack/python-ironicclient master: Update min tox version to 2.0  https://review.openstack.org/61211419:07
openstackgerritMerged openstack/ironic master: Fix ironic developer quickstart document  https://review.openstack.org/61241019:07
openstackgerritMerged openstack/ironic master: Add note to pxe configuration doc  https://review.openstack.org/58340919:07
*** jcoufal has quit IRC19:09
*** irclogbot_3 has quit IRC19:14
*** jcoufal has joined #openstack-ironic19:15
*** irclogbot_3 has joined #openstack-ironic19:15
*** roger2 has joined #openstack-ironic19:22
roger2I'm confused about "boot_option:local". "openstack overcloud deploy --templates" fails for errors about it. eg. u'Node <uuid> has an incorrectly configured driver_info/deploy_ramdisk. Expected "<uuid>" but got "None".' I used node discovery instead of instackenv.json to set capabilities, which appeared as u'capabilities': u'cpu_hugepages:true,cpu_vt:true,boot_mode:bios'. Should I make an introspection rule to specify boot_o19:22
roger2My env: tripleo/queens/containerized/baremetal19:23
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: tests: Always use virtualenv for tests  https://review.openstack.org/61180819:47
*** irclogbot_3 has quit IRC20:27
openstackgerritMerged openstack/ironic-inspector master: Add API reference  https://review.openstack.org/49575220:31
openstackgerritMerged openstack/ironic-python-agent master: Parallel erase disk devices  https://review.openstack.org/34672320:31
openstackgerritMerged openstack/ironic-python-agent stable/rocky: Fix BMC IP address detection on CoreOS images  https://review.openstack.org/61234720:31
openstackgerritMerged openstack/ironic-python-agent master: Allow using ironic-lib from source when building tinyipa  https://review.openstack.org/60700020:31
TheJuliaroger2: it looks like your missing configuration to set driver_info/deploy_ramdisk and deploy_kernel, which are required to deploy a node.20:32
*** sai_p has joined #openstack-ironic20:33
NobodyCamtotally missed the Good Morning Ironic'ers this morning20:38
NobodyCamso Good AfterNoon Ironic'ers20:38
NobodyCam:p20:38
openstackgerritChristopher Dearborn proposed openstack/ironic-specs master: Add firmware update spec for Dell EMC hardware  https://review.openstack.org/60213220:46
*** jcoufal has quit IRC20:52
TheJuliagood morning (well, afternoon) NobodyCam20:53
NobodyCam:) hehehehe how goes the battle today20:53
priteauhttps://review.openstack.org/#/c/612346/ failed to pass one gate job, but I don't think the failure is directly related to the patch?20:53
patchbotpatch 612346 - ironic-python-agent (stable/queens) - Fix BMC IP address detection on CoreOS images - 1 patch set20:53
TheJuliaNobodyCam: fairly good, just working on slide decks at the moment :(21:04
NobodyCam:)21:04
TheJuliapriteau: tl;dr the coreos jobs were actually removed from the gate on master because they have a high failure rate. I rechecked. We might need to consider adjusting timeouts, but first lets see if it passes on its own.21:05
etingofTheJulia, it seems that the OOB patch depends on two sushy patches that were merged after the latest sushy release happened21:08
TheJuliaetingof: I guess it seems reasonable that we could spend some time focusing on sushy reviews this week and cut a release later this week.21:09
TheJuliawe might be in decent shape to cut a release sooner rather than later actually21:10
* TheJulia thinks fresh brain cells should look at it tomorrow21:11
etingofcoffee boosted cells indeed21:11
etingofActually, I keep an eye on sushy reviews routinely, will go through them tomorrow morning21:12
priteauTheJulia: thank you121:14
TheJuliaetingof: awesome, thanks21:15
openstackgerritIlya Etingof proposed openstack/ironic master: Add Redfish inspect interface  https://review.openstack.org/44942521:16
*** baha has quit IRC21:23
openstackgerritJulia Kreger proposed openstack/ironic master: Add a locking interface  https://review.openstack.org/59919121:28
*** dmellado has quit IRC21:32
*** rh-jelabarre has quit IRC21:41
*** priteau has quit IRC21:49
*** bnemec has quit IRC22:18
*** dsneddon_ has joined #openstack-ironic22:41
*** rcernin has joined #openstack-ironic22:46
*** markguz_ has joined #openstack-ironic22:50
*** gyee has quit IRC23:00
*** markguz_ has quit IRC23:14
*** rcernin_ has joined #openstack-ironic23:28
*** rcernin has quit IRC23:30
*** rpioso is now known as rpioso|afk23:32

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