Monday, 2019-02-18

*** hwoarang has joined #openstack-ironic00:01
*** trungnv has quit IRC00:07
*** trungnv has joined #openstack-ironic00:11
*** trungnv has quit IRC00:29
*** moshele has quit IRC01:10
openstackgerritMerged openstack/python-ironicclient master: add python 3.7 unit test job  https://review.openstack.org/63726301:25
*** moshele has joined #openstack-ironic01:26
*** moshele has quit IRC01:49
*** sdake has joined #openstack-ironic02:00
*** whoami-rajat has joined #openstack-ironic02:13
*** sdake has quit IRC02:18
*** sdake has joined #openstack-ironic02:22
*** Soopaman has quit IRC02:32
*** mgoddard has quit IRC02:32
*** mgoddard has joined #openstack-ironic02:34
*** sdake has quit IRC02:44
*** hwoarang has quit IRC02:49
*** hwoarang has joined #openstack-ironic02:51
*** hwoarang has quit IRC02:56
*** hwoarang has joined #openstack-ironic03:00
*** hwoarang has quit IRC03:05
*** hwoarang has joined #openstack-ironic03:06
*** hwoarang has quit IRC04:15
*** hwoarang has joined #openstack-ironic04:17
*** _fragatina has quit IRC04:26
*** _fragatina has joined #openstack-ironic04:26
*** hwoarang has quit IRC04:46
*** hwoarang has joined #openstack-ironic04:47
openstackgerritAnkit Kumar proposed openstack/sushy master: Add support for ilo Virtual Media  https://review.openstack.org/63684505:28
*** dnuka has joined #openstack-ironic05:32
dnukagood morning ironic05:32
openstackgerritKaifeng Wang proposed openstack/python-ironicclient master: Support node description  https://review.openstack.org/63293506:17
*** hwoarang has quit IRC06:23
*** hwoarang has joined #openstack-ironic06:29
*** sdake has joined #openstack-ironic06:38
*** e0ne has joined #openstack-ironic06:41
*** e0ne has quit IRC06:47
*** jtomasek has joined #openstack-ironic06:50
*** moshele has joined #openstack-ironic06:52
*** hwoarang has quit IRC07:12
*** hwoarang has joined #openstack-ironic07:16
*** jtomasek has quit IRC07:27
*** jtomasek has joined #openstack-ironic07:28
*** sdake has quit IRC07:30
*** hamdyk has joined #openstack-ironic07:33
*** e0ne has joined #openstack-ironic07:35
openstackgerritMerged openstack/python-ironic-inspector-client master: add python 3.7 unit test job  https://review.openstack.org/63726107:38
*** e0ne has quit IRC07:38
*** dtantsur|afk is now known as dtantsur07:42
dtantsurmorning ironic07:42
dnukagood morning dtantsur o/07:42
*** pcaruana has joined #openstack-ironic07:43
dtantsur\o07:47
openstackgerritKaifeng Wang proposed openstack/ironic master: Follow up to node description  https://review.openstack.org/63748307:49
*** gkadam has joined #openstack-ironic08:06
*** gkadam has quit IRC08:09
*** iurygregory has joined #openstack-ironic08:14
iurygregorygood morning o/08:15
dnukagood morning iurygregory o/08:15
iurygregoryo/08:16
*** rpittau has joined #openstack-ironic08:17
dnukagood morning rpittau o/08:17
*** tssurya has joined #openstack-ironic08:17
openstackgerritKaifeng Wang proposed openstack/ironic-tempest-plugin master: tests for node description  https://review.openstack.org/63748908:17
rpittaugood morning ironic! o/08:17
rpittauhi dnuka :)08:17
iurygregorymorning rpittau o/08:18
rpittauhey iurygregory :)08:18
*** dnuka is now known as dnuka|brb08:19
*** amoralej|off is now known as amoralej08:23
*** arne_wiebalck has joined #openstack-ironic08:24
arne_wiebalckgood morning, ironic!08:24
rpittauhi arne_wiebalck :)08:27
arne_wiebalckhey rpittau o/ back from sardegna?08:28
rpittauarne_wiebalck, no I'm still there, working from there for the next 2 weeks :)08:29
arne_wiebalckrpittau: uh, nice08:29
*** dnuka|brb is now known as dnuka08:33
dnukagood morning arne_wiebalck o/08:33
arne_wiebalckgood morning dnuka o/08:33
iurygregorymorning arne_wiebalck08:42
arne_wiebalckgood morning iurygregory o/08:42
*** stendulker has joined #openstack-ironic08:48
openstackgerritDigambar proposed openstack/ironic master: Modify the iDRAC driver to use realtime RAID creation  https://review.openstack.org/63490308:48
*** diga has joined #openstack-ironic08:49
openstackgerritvinay kumar muddu proposed openstack/ironic master: Documentation update for iLO Drivers  https://review.openstack.org/63687108:58
*** pcaruana|afk| has joined #openstack-ironic09:01
*** pcaruana has quit IRC09:02
*** e0ne has joined #openstack-ironic09:08
openstackgerritMerged openstack/ironic-inspector master: add python 3.7 unit test job  https://review.openstack.org/63725909:08
*** dougsz has joined #openstack-ironic09:09
*** S4ren has joined #openstack-ironic09:10
*** sdake has joined #openstack-ironic09:15
*** ltomasbo has joined #openstack-ironic09:32
*** derekh has joined #openstack-ironic09:37
*** trident has joined #openstack-ironic09:38
*** diga has quit IRC09:38
openstackgerritHarald Jensås proposed openstack/ironic master: Follow up - API - Implement /events endpoint  https://review.openstack.org/63733109:40
*** pcaruana|afk| has quit IRC10:01
*** pcaruana has joined #openstack-ironic10:07
*** MattMan has quit IRC10:29
*** MattMan has joined #openstack-ironic10:30
*** moshele has quit IRC10:50
*** arne_wiebalck has quit IRC10:50
*** moshele has joined #openstack-ironic10:57
openstackgerritAnkit Kumar proposed openstack/sushy master: Add support for ilo Virtual Media  https://review.openstack.org/63684511:06
*** dnuka is now known as dnuka|brb11:06
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master:  Test BM with VM on the same network L2 isolation and L3 connectivity between: BM and VM This test will be skipped if Nova is not running.  https://review.openstack.org/63659811:08
openstackgerritMerged openstack/sushy master: Add support for the `UpdateService` resource  https://review.openstack.org/61382811:13
*** tssurya has quit IRC11:16
*** sdake has quit IRC11:16
*** sdake has joined #openstack-ironic11:21
*** stendulker has quit IRC11:21
openstackgerritMerged openstack/ironic-tempest-plugin master: Allocation API: functional tests  https://review.openstack.org/63588211:25
*** arne_wiebalck has joined #openstack-ironic11:48
*** moshele has quit IRC12:02
*** dnuka|brb is now known as dnuka12:02
*** moshele has joined #openstack-ironic12:03
jrollmorning y'all :)12:05
rpittauhi jroll :)12:06
dnukagood morning jroll12:06
jroll\o12:06
* dnuka has something to celebrate12:12
openstackgerritHamdy Khader proposed openstack/python-ironicclient master: Add is-smartnic port attribute to port command  https://review.openstack.org/62944912:12
openstackgerritHamdy Khader proposed openstack/python-ironicclient master: Add 'hostname' to port's local link connection  https://review.openstack.org/62877312:12
dnukaI'm on top three https://github.com/openstack/sushy/graphs/contributors \o/12:12
*** sdake has quit IRC12:12
dtantsur:)12:14
dtantsurmorning jroll12:14
rpittaunice dnuka :)12:15
dnukathanks rpittau :)12:16
*** sdake has joined #openstack-ironic12:16
jrollhey dtantsur :)12:18
*** dougsz has quit IRC12:21
*** jcoufal has joined #openstack-ironic12:49
*** tssurya has joined #openstack-ironic12:54
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Run all defined jobs defined in check and gate  https://review.openstack.org/63616912:57
iurygregorymorning jroll o/12:58
*** moshele has quit IRC12:58
*** rh-jelabarre has joined #openstack-ironic13:01
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test BM with VM on the same network   L2 isolation and L3 connectivity between: BM and VM This test will be skipped if Nova is not running.  https://review.openstack.org/63659813:02
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test BM with VM on the same network  https://review.openstack.org/63659813:04
*** trown|outtypewww is now known as trown13:06
*** dougsz has joined #openstack-ironic13:07
openstackgerritIury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Move to zuulv3  https://review.openstack.org/63301013:08
openstackgerritIury Gregory Melo Ferreira proposed openstack/networking-baremetal master: Zuulv3 - Use ironic-base job  https://review.openstack.org/63582413:09
dtantsuriurygregory: I think you only fixed nits in ironicclient patch ^^^13:12
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Test BM with VM on the same network  https://review.openstack.org/63659813:12
iurygregorydtantsur, i fixed the 5 comments you added no? O.o13:14
dtantsuriurygregory: the actual -1 was because the functional tests do not run any tests any more (see the top-level comment)13:15
iurygregory<facepalm>13:16
*** amoralej is now known as amoralej|lunch13:17
*** diga has joined #openstack-ironic13:24
*** priteau has joined #openstack-ironic13:26
*** sdake has quit IRC13:28
*** _fragatina has quit IRC13:34
*** _fragatina has joined #openstack-ironic13:34
iurygregorydtantsur, the envlist is just functional or i need some regex do you know?13:36
*** rh-jelabarre has quit IRC13:37
iurygregorybefore this script was used to call https://github.com/openstack/python-ironicclient/blob/master/tools/run_functional.sh so i assume functionall will do the work fine13:37
*** rh-jelabarre has joined #openstack-ironic13:37
openstackgerritIury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Move to zuulv3  https://review.openstack.org/63301013:38
*** leakypipes is now known as jaypipes13:39
dtantsuriurygregory: yeah, but we may need to run this script or populate the config files similarly13:41
iurygregorydtantsur, ops =X let me see how it goes13:43
dtantsurthnx!13:44
*** priteau has quit IRC13:45
iurygregorynp13:45
*** _fragatina has quit IRC13:53
*** _fragatina has joined #openstack-ironic13:53
*** sdake has joined #openstack-ironic13:55
iurygregorydtantsur, IRONIC_RPC_TRANSPORT can be set to empty without problem if we are using default?13:59
iurygregoryor need to be oslo?13:59
dtantsuriurygregory: you can leave it unset probably.14:00
iurygregoryI think we break ironic-tempest-dsvm-functional-python3-rocky because is setting json-rpc XD14:01
iurygregorygoing to add the configs14:01
*** S4ren has quit IRC14:01
* iurygregory still trying to debug the wholedisk failures and the standalone-queens in the tempest repo14:02
dtantsuriurygregory: I'm not sure how we can break it, it should take the devstack plugin from stable/rocky, where this option does not exist.14:02
iurygregorydtantsur, http://logs.openstack.org/69/636169/11/check/ironic-tempest-dsvm-functional-python3-rocky/26cf39e/controller/logs/local_conf.txt =)14:02
*** priteau has joined #openstack-ironic14:02
dtantsuriurygregory: sure, but does it actually have any effect on rocky?14:08
iurygregorydtantsur, the ironic conf is missing the transport-url conf and all tests failed14:08
iurygregoryIm going to override to see14:09
dtantsurthat's weird, why do we end up with master devstack plugin on rocky?14:10
*** tzumainn has joined #openstack-ironic14:10
dtantsurI think it's a bigger problem14:11
iurygregoryhumm14:11
*** jroll has quit IRC14:13
*** jroll has joined #openstack-ironic14:14
*** amoralej|lunch is now known as amoralej14:14
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: DNM - Test  https://review.openstack.org/63755214:16
*** stendulker has joined #openstack-ironic14:36
*** bnemec-pto is now known as bnemec14:37
openstackgerritIury Gregory Melo Ferreira proposed openstack/python-ironicclient master: Move to zuulv3  https://review.openstack.org/63301014:37
*** rh-jelabarre has quit IRC14:43
*** baha has joined #openstack-ironic14:43
*** cdearborn has joined #openstack-ironic14:44
*** arne_wiebalck has quit IRC14:46
*** priteau has quit IRC14:47
*** rh-jelabarre has joined #openstack-ironic14:48
iurygregoryanyone saw [ERROR] /opt/stack/devstack/inc/python:434 The following LIBS_FROM_GIT were not installed correct: ironic-tempest-plugin ironic-python-agent requirements tempest placement cinder devstack ?14:50
*** moshele has joined #openstack-ironic14:53
*** mjturek has joined #openstack-ironic14:56
*** chris_j has joined #openstack-ironic14:56
*** arne_wiebalck has joined #openstack-ironic14:57
dtantsurcurious.. something wrong with required-projects? is LIBS_FROM_GIT set explicitly?14:58
*** sdake has quit IRC14:58
*** priteau has joined #openstack-ironic14:59
iurygregoryits happeing only in the ironic-tempest-dsvm-ironic-inspector-pike14:59
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Feb 18 15:00:17 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
TheJuliao/15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
iurygregoryo/15:00
dtantsur\o15:00
openstackThe meeting name has been set to 'ironic'15:00
rpittauo/15:00
hjensaso/15:00
arne_wiebalcko/15:00
*** kaifeng has joined #openstack-ironic15:00
dnukao/15:00
kaifengo/15:00
stendulkero/15:00
TheJuliaGood morning everyone!15:00
dnukagood morning TheJulia15:00
rpittaugood morning!15:01
*** rh-jelabarre has quit IRC15:01
TheJuliaOur agenda this morning is fairly light, so hopefully we will get this wrapped up nice and quickly. I hope15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting15:01
rpiosoo/15:01
TheJulia#topic Announcements / Reminders15:01
*** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)"15:01
mjtureko/15:02
TheJulia#info This week is R-7, which means next week is our final non-client library release for this cycle.15:02
TheJulia#link https://releases.openstack.org/stein/schedule.html15:02
dtantsurwhich, I guess, includes sushy, ironic-lib and metalsmith (?)15:03
TheJuliasushy and ironic-lib. Metalsmith likely...15:03
TheJulia#info The Forum/PTG idea etherpad has been posted and already has some great topics.15:04
TheJulia#link https://etherpad.openstack.org/p/DEN-train-ironic-brainstorming15:04
TheJuliaI have added a list for people to indicate if they are going on the etherpad. That way we can hopefully logically and efficiently try to plan topics.15:05
TheJulia#info The Cisco UCS drivers have been marked as of deprecated in a proposed review.15:05
TheJulia:(15:05
TheJulia#link https://review.openstack.org/#/c/637241/15:05
patchbotpatch 637241 - ironic - Deprecate Cisco drivers - 1 patch set15:06
rpittau:/15:06
TheJuliaDoes anyone have anything to announce or remind us of this week?15:06
* dtantsur makes a sad manul face at the cisco deprecation15:06
TheJulia:(15:07
* TheJulia takes the crickets as indication we can proceed15:07
dtantsuryep15:07
TheJulia#topic Review action items from previous meeting:15:07
*** openstack changes topic to "Review action items from previous meeting: (Meeting topic: ironic)"15:07
TheJulia#undo15:07
openstackRemoving item from minutes: #topic Review action items from previous meeting:15:07
TheJulia#topic Review action items from previous meeting15:07
*** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)"15:07
TheJuliaI still have not gotten a chance to send a summary email from our midcycle call given how busy I have been. I'm unsure if it really matters at this point :(15:08
*** hamdyk has quit IRC15:08
TheJuliaI'll try to get something sent out in the next few days. Today should... hopefully... be quiet.15:09
dtantsurmaybe we just skip it15:09
TheJuliaWe did keep really good notes15:09
TheJuliaAnyway15:09
dtantsurafter all, the etherpad is public (and next time we should remember to record the call)15:09
mjturekTheJulia: maybe an email pointing to the etherpad would be nice? I can help there if you'd like15:09
TheJuliaPerhaps...15:09
* arne_wiebalck thinks the notes on the etherpad are good enough15:09
cdearborn\o15:09
TheJuliamjturek: that would be awesome15:09
TheJuliaAnyway, time to move on!15:10
TheJulia#topic Review subteam status reports15:10
*** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)"15:10
mjturekyou got it15:10
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:10
TheJuliaStarting at line 28915:10
TheJuliare: py3 and explicit py36 unit testing, I think that is fine, and likely preferred given there is an attempt to land python 3.7 unit tests in various projects.15:11
dtantsur++15:12
TheJuliajroll: any indication if the conductor grouping stuff is going to make it for nova this cycle?15:12
dtantsurFYI I tried our unit tests with py38-alpha1 :)15:12
rpittauI definitely vote for 3.7 :)15:12
jrollsorry, forgot about this meeting >.>15:12
jrollTheJulia: I hope so, it has a +215:12
TheJulia:)15:13
TheJuliaThe data storage for deploy templates is in good shape, looks like the remaining patches are still WIP15:13
dtantsuryeah, left a few reviews on the API one, mostly complaining about /details :)15:14
TheJuliaheh, okay15:14
TheJuliaLooks like smartnics is mostly in a happy state, we have a few small reviews to do on our side of the universe15:14
dtantsuronly two ironicclient patches, right? or is there anything else?15:14
TheJuliadtantsur: one more ironicclient patch and a follow-up patch15:15
TheJuliaAnd some of the neutron event processing, at least the receiver in the API (That does nothing) has merged so that can move forward some before the end of the cycle.15:16
*** w14161_1 has joined #openstack-ironic15:17
TheJuliaI doub't graphical console is going to merge this cycle without rpc/api work. kaifeng raised a concern that should likely be discussed15:17
dtantsur++ no point in hurrying it in15:17
dtantsurI hope to finish the allocation API work (without backfilling) this cyclle15:18
*** priteau has quit IRC15:18
rpittauTheJulia, for the  graphical console I've been working on RPC/API15:18
TheJuliarpittau: could you put that note on the status?15:19
rpittauyes15:19
dtantsurrpittau: awesome! still, without any vendor implementing it, we don't have to hurry just yet15:19
TheJuliaI've been really amazed just how many new things we've merged this cycle15:19
rpittauyeah15:19
*** rh-jelabarre has joined #openstack-ironic15:19
TheJuliaAnyway, I'm done going down the list and I guess we can proceed to priorities for the next week.15:19
TheJuliaEveryone else ready to proceed?15:20
larsksSanity check: I need to manually copy the kernel and ramdisk used for inspection into the ironic httpboot/tftpboot directories, right? The deploy_kernel and deploy_ramdisk driver_info attributes aren't used for that.15:20
dtantsurlarsks: you're right15:21
dtantsurTheJulia: ++ to proceed15:21
larsksdtantsur: Thanks.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
TheJuliaAnd while everyone is running to the list, I'm going to clean it up and remove the items that we merged this past week15:21
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:22
TheJuliaStarting at line 16015:22
dtantsurquite a few things have merged indeed15:23
TheJuliadtantsur: Want any ironic eyes on https://review.openstack.org/636594 ?15:23
patchbotpatch 636594 - openstacksdk - baremetal: support for allocation API - 3 patch sets15:23
dtantsurTheJulia: I would not mind, but I don't dare asking given how busy everyone is :)15:23
TheJuliaAlright15:24
dtantsurI just need to nag mordred at some point :)15:24
TheJuliaLooks like someone moved up a networking-baremetal item15:25
hjensasI added some backports.15:27
TheJuliahjensas: they were already on the list :)15:27
TheJuliaI did move them up15:27
TheJuliaI think we're in good shape list wise. Is there anything anyone feels should be on the list?15:27
hjensasTheJulia: oh, ok. Thanks, I removed the line I added.15:28
*** rh-jelabarre has quit IRC15:28
dtantsurTheJulia: do we need graphical console in the priorities, if we're not merging it until the API is proposed?15:28
TheJuliaI was just typing something out asking if we want to remove it15:28
TheJuliaRemoved15:28
rpittauI don't think there will be time anyway for a review15:28
TheJuliaOkay, I guess we're good to proceed then15:29
dtantsurLGTM15:29
TheJuliaEveryone good to proceed to ?Open Discussion?15:29
rpittauit seems we can move forward :)15:30
* TheJulia guesses that is a yes15:30
TheJulia#topic Open Discussion15:30
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:30
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: DNM - Test  https://review.openstack.org/63755215:30
TheJuliarpittau: Seems like you are seeking some help with fixing python3 support in python-hardware ?15:30
*** rh-jelabarre has joined #openstack-ironic15:31
dtantsurrpittau: note that we can move python-hardware to git.openstack.org without it becoming an official project (or a part of any official project)15:31
rpittauTheJulia, yes, so far dtantsur has been bery helpful with reviewing and merging some PRs in that hardware repo15:31
rpittauoh ok I didn't know that15:31
dtantsurbut we need agreement from all current maintainers, esp. Erwan (?)15:32
dtantsuryeah, e.g. ironic-staging-drivers is an independent repository15:32
TheJuliaOn the subject of the discussed fast-track feature from ?last week?, I've got some code that should work. I've verified via integration test logs that the API change works, still trying to get a tempest scenario to leverage the entire sequence without any reboots. https://review.openstack.org/#/c/635996/15:33
patchbotpatch 635996 - ironic - fast tracked deployment support - 11 patch sets15:33
rpittauthe idea was to be able to keep the hardware repo in shape as much as possible15:33
dtantsurTheJulia: I'm sloooooowly getting to it :) was planning on today, but had to wake up early, and my head doesn't cooperate easily :)15:33
dnuka:)15:34
dtantsurrpittau: I'm all for it. but we probably need a formal poll (e.g. email) among the current maintainers. then I can guide you through the process.15:34
TheJuliadtantsur: no worries. I think once I have an integration test performing discovery -> deployment without rebooting... I'll be super happy15:34
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Run all defined jobs defined in check and gate  https://review.openstack.org/63616915:34
dtantsurTheJulia: a lot of people will15:34
rpittaudtantsur, sounds good15:34
TheJuliadtantsur: next step.. the world?15:35
dtantsurabsolutely15:35
kaifenguh, the fast track means?15:35
dtantsurkaifeng: from enroll to active with only one reboot15:36
*** rh-jelabarre has quit IRC15:36
TheJuliakaifeng: so dtantsur and I have this downstream req to try and speed up discovery to deployment time, and we spent a lot of time rebooting, so in discussion ?last? week, there was a lot of positive feedback to adding such functionality to ironic so we don't have to power off when that is the case15:36
dtantsurboot the node for discovery, leave it running, clean, leave it running, deploy, reboot into instance15:36
*** rh-jelabarre has joined #openstack-ironic15:36
kaifengawesome!15:37
chris_jgood idea!15:37
chris_jto accelerate the deployment .. some users might not be happy about additional power consumption by running servers15:38
kaifengI am trying to understand how discovery -> deployment can happen, a node is enrolled, it can't be scheduled if we don't put it into available state, and how we know it will be chosen?15:38
TheJuliaAnyway, i'd love some reviews, the main thing right now is the tempest test for me15:38
dtantsurkaifeng: the state machine will progress as usual, we just won't shut down the machine in between15:39
jrollchris_j: it's opt-in, so shouldn't be any surprises15:39
dtantsuryep, and totally opt-in15:39
chris_jawesome! thanks for clarifying15:39
TheJuliachris_j: and I'd rather find out my breaker is degraded with the entire rack about to be deployed instead of during some day in production....15:39
kaifenghmm, I guess there is no spec right? maybe I can only find answers from patches :)15:39
TheJuliakaifeng: yeah, no spec. I put together an rfe, and it really seems fairly simple15:40
TheJuliakaifeng: it really doesn't move the node through the state machine, we truly just leave the power on so the ramdisk can continue to heartbeat to the ironic api if so configured. If it does not with in... 300 seconds, then the node is treated as if it needs to be powered on15:41
kaifengokay, I saw this patch https://review.openstack.org/#/c/636778/ but it's not attached to a story yet15:41
patchbotpatch 636778 - ironic-inspector - Add ironic API url to inspector IPA config - 2 patch sets15:41
*** e0ne has quit IRC15:42
arne_wiebalckkaifeng: you’re looking for this one? https://storyboard.openstack.org/#!/story/200496515:42
TheJuliakaifeng: and https://review.openstack.org/63599615:43
patchbotpatch 635996 - ironic - fast tracked deployment support - 11 patch sets15:43
kaifenggot it, thanks15:44
TheJuliaAnyway, do we have anything else to discuss today?15:44
*** rh-jelabarre has quit IRC15:44
*** rh-jelabarre has joined #openstack-ironic15:45
* dtantsur is busy laughing at https://2ctptqj9vf3lafyt2rkh1qto-wpengine.netdna-ssl.com/wp-content/uploads/2017/10/bork.jpg15:45
* TheJulia takes the silence as everyone is ready to go have more coffee15:45
*** sdake has joined #openstack-ironic15:45
iurygregoryyay more coffee15:45
TheJuliadtantsur: I think something is lost in translation :(15:46
TheJuliaOkay everyone, have a wonderful week!15:46
dtantsurTheJulia: rather some history of memes :)15:46
dnuka:)15:46
TheJuliaThanks everyone15:47
rpittauthank you :)15:47
iurygregoryty15:47
rpiosoThank you15:47
dnukathanks you :)15:47
TheJulia#endmeeting15:47
*** 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:47
openstackMeeting ended Mon Feb 18 15:47:45 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:47
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-18-15.00.html15:47
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-18-15.00.txt15:47
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-02-18-15.00.log.html15:47
*** e0ne has joined #openstack-ironic15:48
*** deray has joined #openstack-ironic15:49
stendulkerTheJulia, dtantsur: Hi, have a moment to discuss https://review.openstack.org/#/c/631181/5/specs/approved/ilo-oob-disk-erase.rst ?15:49
patchbotpatch 631181 - ironic-specs - OOB disk-erase for ilo5 based HPE Proliant Servers - 5 patch sets15:49
dtantsuryeah15:49
dnukahey rpioso o/15:49
stendulkerThis is regarding making 'erase_devices' clean-step into management interface15:49
rpiosodnuka: Good morning :)15:50
dnuka:)15:50
*** rh-jelabarre has quit IRC15:50
stendulkerIf we do that then to invoke it one needs to know the underlying hardware is Gen10. As current erase_devices is a 'deploy' clean step15:50
stendulkerhttps://github.com/openstack/ironic-python-agent/blob/cfb62c87658e32af373278a8de655367f3b489af/ironic_python_agent/hardware.py#L1139-L114715:50
dtantsurstendulker: this is what the new ilo5 hardware type indicates, no?15:51
dtantsurand overall, invoking clean steps require knowing the hardware type and its capability15:51
stendulkeryes, but one may be invoking the same clean step irrespective of hardware type, a common list of steps15:52
stendulkerin that case he would invoke deploy->erase_devices and not management->erase_devices15:52
stendulkerAlso we would have to  call the deploy-erase_devices (in-band) in case the logical volumes are present. OOB cannot perform erase on logical volumes.15:54
*** chris_j has left #openstack-ironic15:54
dtantsurwell, that's the argument against shadowing deploy-erase_devices, right?15:55
stendulkerWould it fine to invoke deploy->erase_devices() from management->erase_devices() Or one has to invoke them separately as deploy->erase_devices would also be listed as supported step?15:55
stendulkerNot completely.15:56
*** rh-jelabarre has joined #openstack-ironic15:56
TheJuliaI think separately is fine because steps are automatically determined for automatic cleaning15:56
stendulkerOOB step cannot perform erasure on logical volumes and NVMe disks15:56
dtantsurmy feeling is to expose both to operators and let them decide15:56
TheJuliaThis may mean OOB cleaning could end up being two steps15:56
stendulkerTheJulia: Yes.15:56
dtantsurside note: we need a uniform way to modify priorities for steps.. there was a spec on that, but I guess it got stuck.15:57
TheJuliadtantsur: add to ptg/forum etherpad?15:57
NobodyCamGood Morning Ironic'ers15:57
dtantsurTheJulia: probably yes, although I'm not sure it's contentions enough15:58
dtantsurmorning NobodyCam15:58
stendulkerToday, for Gen9 servers, erase_devices is being overridden through hardware manager which performs array controller based disk erase on raw disks and shred based erase for logocal volumes15:58
NobodyCamHappy Monday dtantsur :)15:58
dnukagood morning NobodyCam15:58
NobodyCammorning dnuka :) happy Monday15:58
stendulkerIf we go by 2 steps concept, the behavior will change for Gen10 servers.15:58
TheJuliastendulker: hardware manager in python-proliantutils?15:58
stendulkerTheJulia: yes15:58
dnukaNobodyCam, happy Monday :)15:58
rpittauhi NobodyCam :)15:59
dtantsurTheJulia: https://review.openstack.org/#/c/362615/15:59
patchbotpatch 362615 - ironic-specs - Generic way to configure clean step priorites in A... - 10 patch sets15:59
NobodyCamhey hey rpittau :) Good morning (UGT ofc)15:59
stendulkerhttps://github.com/openstack/proliantutils/blob/master/proliantutils/ipa_hw_manager/hardware_manager.py#L91-L11015:59
dnukao/16:01
*** dnuka has quit IRC16:01
stendulkerTheJulia, dtantsur : The idea of this step was to do same thing as done for Gen9 but avoid having dependency on the licensed utility 'ssacli' which openstack distros find difficult to package due to legal issues.16:02
*** cdearborn has quit IRC16:02
stendulker* package into their deploy ramdisks16:03
dtantsurstendulker: do we have OOB erasure for Gen9?16:04
stendulkerNo16:04
dtantsurthen I'm not entirely sure I understand the "same thing as done for Gen9" bit16:04
stendulkerIts inband with smart storage utility  'ssacli'16:05
stendulkerI mean behavior here.16:05
dtantsurbut it's not entirely the same (LVM, etc)16:06
stendulkerIn Gen9, 'ssacli' performs disk erase for raw disks (disks invisible to OS) and then invokes shred to perform erase on disks that are visible to OS.16:06
stendulkerIn case of Gen10, one do not need to have 'ssacli' onto ramdisk. The disk erase can be invoked from OOB and iLO would perfomr same operations as done for Gen916:06
stendulkerSo end-behavior after cleaning operation would remain same16:07
dtantsurso in the end, is it a full erase? or only raw disks?16:07
stendulkerfull erase16:08
*** arne_wiebalck has quit IRC16:08
stendulkerIN case of OOB. Step is invoked OOB, then needs a reboot to begin actual erasure operation16:08
dtantsurokay, I think I can an idea. I still don't think it belongs to the deploy interface though16:09
stendulkerinvoke OOB disk_erase, reboot, check erase status, upon completion, invoke inband erase.16:09
*** arne_wiebalck has joined #openstack-ironic16:09
stendulkerIt can go into management interface, it for the legacy reasons and maintaining it at same 'default' interface.16:10
*** pcaruana has quit IRC16:10
stendulkerwith 'management' interface, one needs to explicitly check it by first invoking get_clean_steps() and then invoke it.16:11
stendulkerIdea, was can we avoid it as 'erase_devices' is  a clean  step that is available for all hardware as its part of IPA.16:12
*** andrein has joined #openstack-ironic16:12
dtantsurany clean steps work the same, I'm not sure why you need to explicitly check anything16:13
*** rh-jelabarre has quit IRC16:13
dtantsuryou can enable it by default, and it will run during automated cleaning16:13
dtantsur(give it higher priority than the regular erase_devices)16:14
stendulkerwhile invoking we need to pass interface_name to step params incase of manual cleaning16:14
stendulkeri think 'automatic cleaning' should be ok with any interface name16:14
stendulkerThough I do not have any data to know if anybody invoke 'erase_devices' as a manual cleaning step. But since its possible...16:15
*** Chaserjim has joined #openstack-ironic16:16
stendulkerdtantsur: I understand we should be overridding deploy related behavior, but any reasons for not allowing to override 'erase_devices' clean steps?16:17
*** rh-jelabarre has joined #openstack-ironic16:18
dtantsurstendulker: why do you need to override it if you want to run it anyway?16:18
dtantsurI think something does not click inside me. If you want first OOB cleaning, then IB cleaning, you can have two steps with different priorities.16:19
stendulkerstendulker: I do not know how people use it (automatic or manual)16:19
dtantsurthen the ops can override the priorities if they want to disable one or the other.16:19
dtantsurtripleo runs erase_devices as manual cleaning, but only because they have automated cleaning disabled16:20
dtantsurI don't think it's a popular case16:20
stendulkerok.. then recommendation is to maintain it under 'management' interface but with the same name for the step 'erase_devices'?16:21
dtantsurI'd do that, yes. TheJulia?16:21
stendulkerThank you. Will ask Paresh (spec-owner) to update  the spec accordingly.16:24
dtantsurthanks!16:27
stendulkerTheJulia, dtantsur: proliantutils have dependency on sushy for https://review.openstack.org/#/c/636845/ so this may need proliantutils to be released after sushy release. Any date in mind for sushy release?16:28
patchbotpatch 636845 - sushy - Add support for ilo Virtual Media - 4 patch sets16:28
dtantsurstendulker: the middle of next week, I guess?16:28
andreinHello everyone, I'm new to openstack, is this the right place to get some help setting up bifrost?16:28
dtantsurandrein: hi, it is16:28
stendulkerdtantsur: That is around 27th...16:28
openstackgerritHarald Jensås proposed openstack/networking-baremetal master: Rename agent queue - fixes broken minor update  https://review.openstack.org/63757916:29
dtantsurstendulker: if the patch lands earlier, we can consider an earlier release. that's all negotiable :)16:29
dtantsur(and release numbers are cheap)16:29
stendulkerdtantsur: That should give us sufficient time for proliantutils release.16:29
stendulkerdtantsur: Thank you :)16:29
stendulkerTheJulia16:30
stendulkerTheJulia: Thank you.16:30
*** rh-jelabarre has quit IRC16:30
*** kaifeng has quit IRC16:30
stendulkerHave a good day !!16:31
* TheJulia looks up from her email16:31
*** stendulker has quit IRC16:31
TheJulia++ for release numbers being cheap16:31
*** deray has quit IRC16:31
*** rh-jelabarre has joined #openstack-ironic16:33
andreinthanks dtantsur. I've deployed bifrost using kolla-ansible, I've hit a few minor issues before I could enroll my nodes (details on https://storyboard.openstack.org/#!/story/2005034), but know I've hit a wall. When I try to deploy the nodes I get the following error: http://paste.openstack.org/show/745272/16:34
*** e0ne has quit IRC16:34
dtantsurmgoddard: rings any bells ^^?16:34
hjensasdtantsur: https://review.openstack.org/637579 - Changeing the queue name works around the issue of having to delete the old queue. I think changeing the existing reno makes sense, it would be confusing to have another reno entry negating the other one?16:37
patchbotpatch 637579 - networking-baremetal - Rename agent queue - fixes broken minor update - 1 patch set16:37
dtantsurandrein: I wonder if you miss commit https://github.com/openstack/bifrost/commit/4f0743fa6eb2cf897cbbbb374c4b94334441cbfe16:38
*** diga has quit IRC16:39
dtantsurdo you use ansible 2.6?16:39
dtantsurTheJulia: WDYT about releasing bifrost master, including the fix for ansible 2.6?16:39
TheJulia++16:39
* dtantsur requests16:40
andreindtantsur, yes, I've upgraded to 2.7 when I couldn't get the nodes to enroll. I'll wipe everything and start fresh16:40
dtantsurandrein: okay, so you need that commit. I will request a release from master, but it may take 1-2 days.16:41
andreindtantsur, don't rush it on my account, it'll probably take a while until kolla-ansible picks it up anyway. It'll be faster for me to just revert ansible for now :)16:44
dtantsurandrein: we haven't released anything in a few months, won't hurt doing it now anyway :)16:45
dtantsurif you want to stick with Rocky release, you need ansible < 2.6 though16:45
* dtantsur is not sure how kolla-ansible handles release series16:45
*** moshele has quit IRC16:47
*** w14161_1 has quit IRC16:48
*** arne_wiebalck has quit IRC16:48
*** tssurya has quit IRC16:52
openstackgerritDao Cong Tien proposed openstack/python-ironicclient master: Replace mock.has_calls() with assert_has_calls  https://review.openstack.org/63758616:52
*** andrein has quit IRC16:57
*** andrein has joined #openstack-ironic16:58
*** andrein has quit IRC16:59
*** rh-jelabarre has quit IRC16:59
*** cxc99 has quit IRC17:00
*** baha has quit IRC17:00
*** sdake has quit IRC17:00
*** andrein has joined #openstack-ironic17:01
andreindtantsur, looks like the revert to ansible 2.4 fixed it17:01
*** rh-jelabarre has joined #openstack-ironic17:02
andreinat least, it looks like my servers are doing *something*17:03
dtantsurgood progress :)17:04
*** sdake has joined #openstack-ironic17:05
*** rh-jelabarre has quit IRC17:11
*** rh-jelabarre has joined #openstack-ironic17:12
*** rpittau has quit IRC17:16
*** rh-jelabarre has quit IRC17:16
*** rpittau has joined #openstack-ironic17:19
*** rh-jelabarre has joined #openstack-ironic17:19
*** cxc99 has joined #openstack-ironic17:23
*** rh-jelabarre has quit IRC17:24
*** rh-jelabarre has joined #openstack-ironic17:25
rpittaubye! good night! o/17:25
*** rpittau has quit IRC17:26
*** rh-jelabarre has quit IRC17:29
*** rh-jelabarre has joined #openstack-ironic17:30
*** moshele has joined #openstack-ironic17:32
*** arne_wiebalck has joined #openstack-ironic17:32
openstackgerritDao Cong Tien proposed openstack/ironic master: Update doc for UEFI first  https://review.openstack.org/63122517:33
andreindtantsur, seems like the network guys messed up and my servers aren't booting17:34
*** rh-jelabarre has quit IRC17:34
andreinif you have some time, can you take a look at https://storyboard.openstack.org/#!/story/2005034 ?17:34
andreinIs that also a known issue?17:35
*** rh-jelabarre has joined #openstack-ironic17:35
*** baha has joined #openstack-ironic17:37
*** moshele has quit IRC17:40
*** hwoarang has quit IRC17:47
*** hwoarang has joined #openstack-ironic17:47
*** _fragatina has quit IRC17:48
*** priteau has joined #openstack-ironic17:53
*** andrein has quit IRC17:59
*** dougsz has quit IRC17:59
*** trown is now known as trown|lunch18:01
*** derekh has quit IRC18:02
*** arne_wiebalck has quit IRC18:04
*** fbouliane has quit IRC18:06
*** amoralej is now known as amoralej|off18:07
openstackgerritNisha Agarwal proposed openstack/ironic master: Follow-up patch for I71feefa3d0593fd185a286bec4ce38607203641d  https://review.openstack.org/63760018:09
*** rh-jelabarre has quit IRC18:15
*** rh-jelabarre has joined #openstack-ironic18:15
*** rh-jelabarre has quit IRC18:20
*** rh-jelabarre has joined #openstack-ironic18:20
*** e0ne has joined #openstack-ironic18:24
*** sdake has quit IRC18:29
*** sdake_ has joined #openstack-ironic18:31
*** arne_wiebalck has joined #openstack-ironic18:35
*** rh-jelabarre has quit IRC18:39
*** rh-jelabarre has joined #openstack-ironic18:39
*** dtantsur is now known as dtantsur|afk18:47
*** rh-jelabarre has quit IRC18:49
*** rh-jelabarre has joined #openstack-ironic18:50
*** rh-jelabarre has quit IRC18:54
*** rh-jelabarre has joined #openstack-ironic18:56
*** dsneddon has joined #openstack-ironic18:59
*** _fragatina has joined #openstack-ironic19:00
*** rh-jelabarre has quit IRC19:05
*** rh-jelabarre has joined #openstack-ironic19:06
*** trown|lunch is now known as trown19:06
*** andrein has joined #openstack-ironic19:16
*** rh-jelabarre has quit IRC19:16
*** rh-jelabarre has joined #openstack-ironic19:17
*** priteau has quit IRC19:25
*** moshele has joined #openstack-ironic19:26
*** arne_wiebalck has quit IRC19:26
*** rh-jelabarre has quit IRC19:26
*** rh-jelabarre has joined #openstack-ironic19:27
*** e0ne has quit IRC19:30
openstackgerritMerged openstack/ironic stable/pike: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/61636719:31
*** rh-jelabarre has quit IRC19:32
*** rh-jelabarre has joined #openstack-ironic19:32
rpiosoTheJulia: Thank you :)19:33
rpiosodtantur|afk: You, too!19:33
rpiosoetingof: Thanks for wishing good luck to the Zuul testing with your +1 ;-)19:36
*** e0ne has joined #openstack-ironic19:37
*** andrein has quit IRC19:45
*** sdake_ has quit IRC19:54
*** rh-jelabarre has quit IRC20:21
*** rh-jelabarre has joined #openstack-ironic20:22
*** whoami-rajat has quit IRC20:23
openstackgerritMerged openstack/ironic master: Follow up - API - Implement /events endpoint  https://review.openstack.org/63733120:24
*** rh-jelabarre has quit IRC20:26
*** rh-jelabarre has joined #openstack-ironic20:26
*** rh-jelabarre has quit IRC20:31
*** rh-jelabarre has joined #openstack-ironic20:32
openstackgerritMerged openstack/ironic stable/ocata: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/61636820:33
rpiosoTheJulia: Is the Stein priority "DHCP-less/L3 virtual media boot" entirely covered by https://storyboard.openstack.org/#!/story/1526753 ?20:33
*** rh-jelabarre has quit IRC20:37
*** rh-jelabarre has joined #openstack-ironic20:37
*** andrein has joined #openstack-ironic20:43
TheJuliarpioso: no, nokia's desire to provide static addressing information in some sort of fashion such that a node can boot without DHCP being available to it.20:55
rpiosoTheJulia: Ah, I see. Thank you for clarifying.21:02
*** rh-jelabarre has quit IRC21:08
*** rh-jelabarre has joined #openstack-ironic21:08
*** rh-jelabarre has quit IRC21:22
*** rh-jelabarre has joined #openstack-ironic21:23
*** ijw has joined #openstack-ironic21:27
*** rh-jelabarre has quit IRC21:27
*** rh-jelabarre has joined #openstack-ironic21:28
*** mjturek has quit IRC21:31
*** ijw has quit IRC21:32
*** ijw has joined #openstack-ironic21:32
*** rh-jelabarre has quit IRC21:32
*** ijw has quit IRC21:32
*** rh-jelabarre has joined #openstack-ironic21:33
*** ijw has joined #openstack-ironic21:34
*** rh-jelabarre has quit IRC21:34
*** rcernin has joined #openstack-ironic21:40
*** jtomasek has quit IRC21:40
*** moshele has quit IRC21:54
*** trown is now known as trown|outtypewww22:02
*** e0ne has quit IRC22:03
*** e0ne has joined #openstack-ironic22:04
*** baha has quit IRC22:04
openstackgerritHarald Jensås proposed openstack/ironic master: Devstack - run vbmc as sudo  https://review.openstack.org/63764322:21
*** _fragatina has quit IRC22:25
*** _fragatina has joined #openstack-ironic22:26
*** sdake has joined #openstack-ironic22:27
*** sdake has quit IRC22:27
*** sdake has joined #openstack-ironic22:32
*** sdake has quit IRC22:46
*** sdake has joined #openstack-ironic22:48
openstackgerritJulia Kreger proposed openstack/ironic-tempest-plugin master: Add fast-track testing capability  https://review.openstack.org/63703922:53
*** e0ne has quit IRC23:12
*** e0ne has joined #openstack-ironic23:23
*** sdake has quit IRC23:24
*** sdake has joined #openstack-ironic23:26
*** e0ne has quit IRC23:30
*** sdake has quit IRC23:32
*** jcoufal has quit IRC23:34
*** andrein has quit IRC23:52

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