Friday, 2015-01-23

*** smoriya has joined #openstack-ironic00:00
*** igordcard has quit IRC00:11
*** davideagnello has quit IRC00:14
*** achanda has quit IRC00:31
*** achanda has joined #openstack-ironic00:34
*** spandhe has quit IRC00:51
*** rloo has quit IRC00:57
*** Marga_ has quit IRC00:59
*** Marga_ has joined #openstack-ironic01:00
*** Marga__ has joined #openstack-ironic01:03
*** Marga_ has quit IRC01:04
*** jerryz has joined #openstack-ironic01:10
*** eghobo has joined #openstack-ironic01:13
*** rwsu is now known as rwsu-afk01:15
*** killer_prince has quit IRC01:16
*** killer_prince has joined #openstack-ironic01:16
*** killer_prince is now known as lazy_prince01:16
JayFWTB a way to sort gerrit by # of patchsets01:22
JayFNobodyCam: https://review.openstack.org/#/c/148145/2 your concern was fixed, wanna give it a +3 and +A (it takes +1 more since you have a -1 :P)01:24
NobodyCamlooking now JayF :)01:25
*** ChuckC has quit IRC01:26
JayFnaohirot: if you can update https://review.openstack.org/#/c/136020/1301:26
JayFnaohirot: to reflect the driver names settled on in the meeting (without having to implement all four :P) you have my vote01:27
NobodyCamJayF: _a'd01:27
NobodyCam+a's even01:27
JayFnice01:27
JayFany specs you can call out that could especially use my TLC?01:27
JayFI'm reviewing somewhat indiscriminately but will be reviewing for 15m more or so01:27
NobodyCamJayF: easy one 2 +2's https://review.openstack.org/#/c/135228/01:28
naohirotJayF: NobodyCam: good evening, let me read01:28
NobodyCammorning naohirot :)01:28
openstackgerritjiangfei proposed openstack/ironic: Use oslo_utils replace oslo.utils  https://review.openstack.org/14945001:28
JayFNobodyCam: no problems with me landing that if it lgtm?01:28
NobodyCamplease do!!!!01:29
openstackgerritJosh Gachnang proposed openstack/ironic: Agent driver fails with Swift Multiple Containers  https://review.openstack.org/14945101:29
openstackgerritMerged openstack/ironic-specs: Replace requests lib to python-scciclient lib  https://review.openstack.org/14814501:29
JayFJoshNang: ^ well however did you discover that bug01:29
JoshNangJayF: :P01:29
JayF<.< >.>01:29
JoshNangthough \o/ for upstreaming same day01:29
*** ChuckC has joined #openstack-ironic01:29
NobodyCamJoshNang: same of over beer01:29
* NobodyCam bets its a good story01:30
NobodyCamgrrr01:30
NobodyCamsave for over beer01:30
NobodyCam:-p01:30
JayFGood software is never good stories01:30
JayFwhich means our staging broke a little first, and we fixed it01:30
JoshNangNobodyCam: :) can do01:30
JayFwhich is how it's supposed to work01:30
JayFJoshNang: ^ pretty much, right?01:30
JoshNangyeah pretty much01:30
JayFAlthough it's easy for me to say b/c I was out sick :P01:30
NobodyCamno but the discovery of edge case bugs usally are quite good01:31
JayFI mean, if the image isn't there01:31
JayFit's not much of an edge case01:31
JayFit's a 404 deploy failed case01:31
JayFlol01:31
JoshNanglots of head scratching before we figured out this config option had appeared01:31
JayFJoshNang: reviewed that01:32
JayFNobodyCam: going to ask this before marking it on the spec01:32
JayFNobodyCam: nevermind, I just realized this was a spec not just for UEFI, but for secure boot01:33
JayFwhee01:33
naohirotJayF: let me confirm what we agreed in the meeting.01:34
JayFI think we agreed to only implement pxe(iscsi) and agent drivers today, with pxe boot as the transport01:35
JayFright?01:35
naohirotJayF: I believe that we agreed to stay the name as is, I believe.01:35
JayFIMBW though01:35
*** BadCub_ has quit IRC01:35
JayFlet me look at the logs from the meeting; that's not the impression I got01:35
JayFAGREED: iRMC work to proceed with two drivers (pxe_irmc and agent_irmc) and no vmedia support initially, but that may come later (devananda, 05:49:14)01:35
JayFAGREED: correction - agent_irmc will use vmedia, and pxe_irmc will not (devananda, 05:50:24)01:35
JayFnaohirot: ^ so the spec to add the management driver still says you're implementing three drivers01:36
openstackgerritMerged openstack/ironic-specs: UEFI Secure Boot support for iLO drivers  https://review.openstack.org/13522801:36
JayFnaohirot: when that spec should say 1 (pxe_irmc) and the VMedia spec should indicate one (agent_irmc)01:36
JayFnaohirot: make sense?01:36
JoshNangJayF: thanks01:36
naohirotJayF: 05:51:20 <naohirot> devananda: Just make sure there is no misunderstainding. agent_irmc and iscsi_irmc use virtual media, pxe_irmc doesn't use virtual media.01:37
naohirotJayF: http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-01-20-05.00.log.html#l-24501:38
*** alexpilotti has joined #openstack-ironic01:39
JayFnaohirot: it looks like that was never acknowledged and the agenda/summary never updated01:39
JayFnaohirot: so I'm honestly not 100% sure the best way to proceed01:39
JayFdevananda: ^ if you have a moment?01:39
JayFnaohirot: I don't care other than wanting Ironic to be awesome :) I'd just rather not put a +2 on something that runs counter to what was marked as agreed in the meeting :)01:40
naohirotJayF: If you see some problem, I can discuss any time.01:40
JoshNanghuh, just noticed glance has translation enabled for help= in config options.01:40
JayFJoshNang: that's a really, really good idea.01:40
* JoshNang files a bug01:41
JayFnaohirot: I'll land on that wherever the consensus goes :) I'll keep an eye on channel and such and as soon as this is resolved you have my +2 :)01:41
devanandaI would prefer a simpler initial implementation here01:42
devananda"just do what iLO did" isn't a good reason for 3 or 4 drivers01:42
devanandaby contrast, iLO develpers provided clear reasons for why they implemented each driver which they did01:43
openstackgerritJosh Gachnang proposed openstack/ironic: Agent driver fails with Swift Multiple Containers  https://review.openstack.org/14945101:43
naohirotdevananda: If agent deploy doesn't support uefi, I'd like to put off in the first implementation too.01:44
NobodyCamoh devananda I believe several spec's have landed... have you run the update BP thingy in awhile?01:44
devanandaNobodyCam: nope01:44
devanandanaohirot: ++01:44
devanandanaohirot: AIUI, agent_ilo does not support UEFI *only* because the current image build tools for the agent do not provide an appropriately signable image01:45
JayFI believe we could UEFI boot an agent image (coreos)01:45
JayFbut not secure boot it01:45
JayFFWIW01:45
devanandaJayF: I believe UEFI == secure boot. you may mean "but not trusted boot it" ?01:46
JayFThat's not true at all01:47
JayFyou can non-secure UEFI boot01:47
JayFwithout requiring a signed image, on many many machines01:47
JayFbecause I UEFI booted my desktop for years without using the signed grub shim (and got a nasty message in post about how I had secure boot disabled)01:47
JayFI have servers with similar options01:47
devanandaahh heh01:47
JayFI should hack on the agent DIB image to make it usable too :(01:48
JayFjust depressing to redo the same work I've already done01:48
naohirotdevananda: JayF: so what should I do?01:51
NobodyCamwow is review.openstack.org slow or is it my connection01:51
JayFNobodyCam: it seems quick to me01:51
devanandanaohirot: focus on one implementation first01:51
devanandanaohirot: that will be easier for you, easier for us to review, and give your users something concrete to work with sooner01:51
naohirotdevananda: JayF: Okay, thanks:)01:52
* NobodyCam looks at all the "please reboot me" / "update ready to install" windows and thinks maybe on my end .... lol 01:52
JayFI'm leaving now; thanks for getting this cleared up! You all have a good day01:52
NobodyCamhave a good night JayF01:53
naohirotJayF: good night:)01:53
devanandag'night, JayF01:54
openstackgerritjiangfei proposed openstack/ironic: Use oslo_serialization replace oslo.serialization  https://review.openstack.org/14945601:59
*** Marga__ has quit IRC02:02
*** Marga_ has joined #openstack-ironic02:03
*** alexpilotti has quit IRC02:03
*** alexpilotti has joined #openstack-ironic02:05
*** chenglch has joined #openstack-ironic02:05
*** ChuckC_ has joined #openstack-ironic02:06
*** alexpilotti has quit IRC02:06
*** ChuckC has quit IRC02:06
*** alexpilotti has joined #openstack-ironic02:07
*** Marga_ has quit IRC02:08
*** alexpilotti has quit IRC02:08
*** alexpilotti has joined #openstack-ironic02:11
*** zer0c00l has quit IRC02:32
*** zer0c00l has joined #openstack-ironic02:33
*** lazy_prince is now known as killer_prince02:37
*** alexpilotti has quit IRC02:38
*** ramineni has joined #openstack-ironic02:39
*** zer0c00l has quit IRC02:46
*** zer0c00l has joined #openstack-ironic02:46
ramineniJayF, JoshNang : hi02:47
*** zer0c00l has quit IRC02:49
*** zer0c00l has joined #openstack-ironic02:49
*** jerryz has quit IRC02:55
*** harlowja is now known as harlowja_away02:59
*** jerryz has joined #openstack-ironic03:01
*** bauzas has quit IRC03:10
*** bauzas has joined #openstack-ironic03:14
*** jjulien has joined #openstack-ironic03:18
*** killer_prince is now known as lazy_prince03:23
*** eghobo has quit IRC03:23
*** naohirot has quit IRC03:27
openstackgerritjiangfei proposed openstack/ironic: Use oslo_serialization replace oslo.serialization  https://review.openstack.org/14945603:33
*** achanda has quit IRC03:40
raminenijroll : hi , there?03:42
*** jerryz has quit IRC03:52
*** jerryz has joined #openstack-ironic03:55
openstackgerritjiangfei proposed openstack/ironic: Use oslo_utils replace oslo.utils  https://review.openstack.org/14945003:57
*** achanda has joined #openstack-ironic04:00
*** naohirot has joined #openstack-ironic04:04
*** Nisha has joined #openstack-ironic04:08
*** jerryz has quit IRC04:10
*** jerryz has joined #openstack-ironic04:16
*** pcrews has quit IRC04:22
*** achanda has quit IRC04:29
*** achanda has joined #openstack-ironic04:32
*** rameshg87 has joined #openstack-ironic04:32
*** lazy_prince is now known as killer_prince04:33
openstackgerritMichael Davies proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173704:36
*** ijw has joined #openstack-ironic04:41
*** eghobo has joined #openstack-ironic04:41
*** ijw has quit IRC04:45
*** ijw has joined #openstack-ironic04:45
*** rameshg87 has quit IRC04:49
*** achanda has quit IRC04:51
*** rameshg87 has joined #openstack-ironic04:52
*** pradipta_away is now known as pradipta04:54
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Ironic Management Interfaces to support UEFI Secure Boot  https://review.openstack.org/13584504:55
*** achanda has joined #openstack-ironic04:56
*** pensu has joined #openstack-ironic05:01
*** enterprisedc has left #openstack-ironic05:10
*** eghobo has quit IRC05:14
*** Marga_ has joined #openstack-ironic05:16
*** Marga_ has quit IRC05:16
*** Marga_ has joined #openstack-ironic05:16
*** Marga_ has quit IRC05:24
*** Marga_ has joined #openstack-ironic05:25
*** Haomeng|2 has joined #openstack-ironic05:42
*** Haomeng has quit IRC05:43
jiangfeiHaomeng|2: hi05:45
jiangfeiHaomeng|2: I want to use oslo_utils replace oslo.utils, but CI is error.  I solve the error  local, but the same error in CI, so i want  to know how to rebuild the tox's environment in CI.05:53
*** chlong has quit IRC05:54
*** coolsvap|afk is now known as coolsvap05:55
*** chlong has joined #openstack-ironic06:01
jerryzjiangfei: you want to reproduce the failure in your local env?06:11
jiangfeijerryz:my local env is rebuilded, and the error is sloved06:12
jerryzjiangfei: what do you mean by rebuilding the tox's environment in CI?06:13
jiangfeijerryz: yes.06:13
jiangfeijerryz: I mean CI env need to tox rebuild06:15
jerryzjiangfei: you want to use tox locally?06:17
jerryzjiangfei: simply install tox and run tox -epy2706:17
jerryzjiangfei: it will create a clean virtualenv to install requirements and  run tests.06:18
jiangfeijerryz: no, i delete the dir '.tox/py27', and run tox -epy27.  so it can pip install the packages and create .tox/py27 env again06:19
jerryzjiangfei: what do you want to do ?06:21
jiangfeijerryz: I want to clean up the py27 environment in CI, and rebuild it again.06:24
jerryzjiangfei: in CI, there is step to clean tox so you don't need to worry about it06:24
jerryzjiangfei: if you fixed it, simply submit your fix and trigger CI again06:25
jiangfeijerryz: but i pass the unit-test. but failed in CI, and Error message like me before06:27
jiangfeijerryz: error is here:  http://logs.openstack.org/50/149450/1/check/gate-ironic-python27/1a99140/console.html06:28
*** lazy_prince has joined #openstack-ironic06:42
*** Marga_ has quit IRC06:44
*** Marga_ has joined #openstack-ironic06:45
*** Marga_ has quit IRC06:49
*** lazy_prince has quit IRC06:50
*** killer_prince is now known as lazy_prince06:52
jerryzdtantsur: ping07:02
*** rameshg87 has quit IRC07:22
*** rameshg87_ has joined #openstack-ironic07:22
*** mrda is now known as mrda-away07:37
*** killer_prince has joined #openstack-ironic07:42
*** lazy_prince has quit IRC07:43
*** killer_prince is now known as lazy_prince07:43
*** eghobo has joined #openstack-ironic08:03
*** ifarkas has joined #openstack-ironic08:04
gilliardGood morning Ironic!08:10
dtantsurMorning, TGIF!08:10
dtantsurjerryz, pong08:10
gilliardahhh, Friday!08:10
dtantsurgilliard, yep :)08:10
*** gilliard is now known as gillllliard08:11
*** ijw has quit IRC08:11
*** chlong has quit IRC08:14
jerryzdtantsur: i want to enable debug, but flask failed to be spawned with debug=True08:14
dtantsurjerryz, wow. what's the failure?08:14
jerryzdtantsur: http://paste.openstack.org/show/160676/08:15
jerryzdtantsur: with debug set to false, i can connect to 505008:15
dtantsurjerryz, hmm, I don't see anything failed in the logs (everything is DEBUG-WARNING level)08:16
dtantsurso you can't connect to discoverd with debug on?08:16
jerryzdtantsur: the connection got refused08:17
dtantsurstrange. lemme check08:17
jerryzdtantsur: the log i showed to you is one take. so i am curious why it ran init() twice?08:18
dtantsurjerryz, so I did debug=true authenticate=false and I'm able to reach discoverd. what's the version you're using (git commit etc)?08:21
jerryzdtantsur: Merge "Get rid of monkey patching work around for Python 2 in main.py"08:23
dtantsurjerryz, ooops! you were lucky enough to get a broken commit, sorry for that. please update to git master, you problem should be fixed.08:23
*** jerryz has quit IRC08:28
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Ironic Management Interfaces to support UEFI Secure Boot  https://review.openstack.org/13584508:31
*** Marga_ has joined #openstack-ironic08:32
* dtantsur back in a couple of hours08:33
*** dtantsur is now known as dtantsur|brb08:33
*** achanda has quit IRC08:36
*** Marga_ has quit IRC08:36
*** priteau has joined #openstack-ironic08:40
*** rakesh_hs has joined #openstack-ironic08:40
*** jerryz has joined #openstack-ironic08:52
*** dlpartain has joined #openstack-ironic08:57
*** erwan_taf has joined #openstack-ironic08:58
*** zhenzanz has quit IRC08:58
*** jcoufal has joined #openstack-ironic09:10
*** derekh has joined #openstack-ironic09:11
*** lucasagomes has joined #openstack-ironic09:11
*** jistr has joined #openstack-ironic09:15
jerryzdtantsur: updating code fixed the issue09:20
*** Nisha has quit IRC09:21
*** lifeless has quit IRC09:21
*** lifeless has joined #openstack-ironic09:22
*** stendulker has joined #openstack-ironic09:22
*** eghobo has quit IRC09:22
jerryzdtantsur: now dicoverd could update properties but there was an exception while adding port because in my setup, i used vm and mac had to be added into ironic port for discoverd to validate its existence09:23
jerryzdtantsur: so port creation by discoverd failed because ironic already had that.09:24
*** lazy_prince has quit IRC09:31
*** Nisha has joined #openstack-ironic09:31
*** Marga_ has joined #openstack-ironic09:33
*** Marga_ has quit IRC09:37
openstackgerritMerged openstack/ironic: Add ipmitool to quickstart guide for Ubuntu  https://review.openstack.org/14938109:39
*** dlpartain has left #openstack-ironic09:39
*** killer_prince has joined #openstack-ironic09:46
*** killer_prince is now known as lazy_prince09:46
*** andreykurilin_ has joined #openstack-ironic09:55
*** vdrok_afk has quit IRC09:56
*** yuriyz has quit IRC09:56
*** vdrok has joined #openstack-ironic09:59
*** chenglch has quit IRC10:16
*** enterprisedc has joined #openstack-ironic10:16
*** enterprisedc has left #openstack-ironic10:16
*** romcheg has joined #openstack-ironic10:21
openstackgerritNisha Agarwal proposed openstack/ironic: Add states required for node-introspection  https://review.openstack.org/14785710:31
*** naohirot has quit IRC10:31
*** pelix has joined #openstack-ironic10:33
openstackgerritNisha Agarwal proposed openstack/ironic: Enable agent_ilo for uefi-bios switching  https://review.openstack.org/14879210:34
*** Marga_ has joined #openstack-ironic10:34
*** Marga_ has quit IRC10:38
*** Nisha has quit IRC10:39
ifarkaslucasagomes, hey, could you please review https://review.openstack.org/#/c/147203/ whenever you will have time for that?10:55
lucasagomesifarkas, yo, sure!10:55
ifarkaslucasagomes, thanks!10:55
openstackgerritRamakrishnan G proposed openstack/ironic: Add VirtualBox drivers and its modules  https://review.openstack.org/14630510:58
*** stendulker_ has joined #openstack-ironic11:00
*** ramineni has quit IRC11:00
*** athomas has quit IRC11:03
*** Haomeng has joined #openstack-ironic11:09
*** yuriyz has joined #openstack-ironic11:10
*** Haomeng|2 has quit IRC11:10
*** andreykurilin_ has quit IRC11:14
*** athomas has joined #openstack-ironic11:15
*** ndipanov_gone is now known as ndipanov11:21
*** naohirot has joined #openstack-ironic11:24
openstackgerritJulien Danjou proposed openstack/ironic: Remove argparse from requirements  https://review.openstack.org/14958211:31
*** Marga_ has joined #openstack-ironic11:34
*** pradipta is now known as pradipta_away11:34
*** dtantsur|brb is now known as dtantsur11:38
dtantsurjerryz, that's ok. if you turn debug=false, you will no longer see the exception, only a warning11:39
*** Marga_ has quit IRC11:39
openstackgerritShivanand Tendulker proposed openstack/ironic-specs: Ironic Management Interfaces to support UEFI Secure Boot  https://review.openstack.org/13584511:40
*** stendulker has quit IRC11:45
*** jcoufal_ has joined #openstack-ironic11:45
*** stendulker_ has quit IRC11:46
*** jcoufal has quit IRC11:48
*** rakesh_hs has quit IRC11:50
*** chlong has joined #openstack-ironic11:50
openstackgerritMerged openstack/ironic: DracClient to handle ReturnValue validation  https://review.openstack.org/14720312:03
*** rameshg87_ has quit IRC12:08
*** rameshg87 has joined #openstack-ironic12:10
*** alexpilotti has joined #openstack-ironic12:12
*** rameshg87 has quit IRC12:15
*** erwan_taf has quit IRC12:20
*** dlpartain has joined #openstack-ironic12:30
*** Marga_ has joined #openstack-ironic12:35
*** Marga_ has quit IRC12:40
*** lucasagomes is now known as lucas-hungry12:43
*** pensu has quit IRC12:44
openstackgerritSatoru Moriya proposed openstack/ironic-specs: Cinder integration  https://review.openstack.org/14960612:52
*** dlpartain has left #openstack-ironic12:53
openstackgerritGopi Krishna S proposed openstack/ironic-specs: Add pxe_ucs driver to support Cisco UCS B/C/M-series servers  https://review.openstack.org/13951712:56
*** smoriya has quit IRC13:02
*** dprince has joined #openstack-ironic13:19
*** athomas has quit IRC13:35
*** Marga_ has joined #openstack-ironic13:36
*** jerryz has quit IRC13:41
*** Marga_ has quit IRC13:41
*** athomas has joined #openstack-ironic13:41
*** lucas-hungry is now known as lucasagomes13:42
*** jerryz has joined #openstack-ironic13:46
*** ramineni has joined #openstack-ironic13:47
*** erwan_taf has joined #openstack-ironic13:52
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module  https://review.openstack.org/14490113:58
*** coolsvap is now known as coolsvap|afk14:08
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680314:12
NobodyCammorning Ironicers14:12
dtantsurNobodyCam, morning. TGIF!14:13
lucasagomesNobodyCam, g'morning :)14:13
NobodyCamoh big time TGIF!!!!14:13
dtantsur:)14:13
NobodyCammorning dtantsur & lucasagomes14:13
ShrewsNobodyCam: lol. morning14:20
Shrewsmorning dtantsur+lucasgomes14:20
*** krtaylor has quit IRC14:21
dtantsurShrews, morning14:21
lucasagomesShrews, hola, morning!14:22
naohirotgood night & have a nice weekend to all14:22
lucasagomesnaohirot, night! enjoy the weekend14:22
naohirotlucasagomes: dtantsur: Shrews: NobodyCam: you too!14:23
*** naohirot has quit IRC14:23
dtantsurnaohirot, g'night14:23
dtantsurtoo late :(14:23
NobodyCammorning Shrews :)14:29
NobodyCamahh missed Maohirot14:29
lucasagomesSimple spec with PoC patches available needing another +2 https://review.openstack.org/#/c/14363614:31
*** krtaylor has joined #openstack-ironic14:34
*** Marga_ has joined #openstack-ironic14:37
victor_lowthermorning, Ironic14:40
*** Marga_ has quit IRC14:42
NobodyCammorning victor_lowther ... lucasagomes looking14:42
*** igordcard has joined #openstack-ironic14:44
NobodyCambrb14:45
*** pensu has joined #openstack-ironic14:49
*** igordcard has quit IRC14:52
*** chlong has quit IRC14:54
victor_lowtherlucasagomes and dtantsur: Replied to your comments on the BIOS spec.14:56
dtantsurvictor_lowther, morning, cool!14:56
lucasagomesvictor_lowther, ack thanks I will take a look soon14:57
*** MattMan has left #openstack-ironic14:57
*** pcrews has joined #openstack-ironic14:57
jrollmorning all :)15:00
jrollramineni: I'm here now, what's up?15:00
*** BadCub_ has joined #openstack-ironic15:03
BadCub_Morning Ironic15:05
raminenijroll: hi, good morning  :)15:06
raminenijroll: want some clarification regarding cleaning spec actually . couldnt catch JayF/joshNang , will you be able to help out15:08
NobodyCammorning jroll BadCub_ ramineni15:08
jrollramineni: maybe I can help, you can also ask questions on gerrit15:08
jrollramineni: what's your question15:08
jrollheya NobodyCam BadCub_ lucasagomes dtantsur victor_lowther :)15:08
dtantsurjroll, o/15:09
BadCub_Heya everyone (C)15:09
raminenimorning nobodycam :)15:09
raminenijroll: In the spec , https://review.openstack.org/#/c/142401/4/specs/kilo/ilo-cleaning-support.rst, I proposed secure_disk_erase as part of management interface15:10
raminenijroll: I got comments saying that " The way this works is via a mix-in type of method. So the agent will provide this functionality for you, without iLo driver needing to declare it as all AIUI."15:10
raminenijroll: does that mean i dont need to add interface for secure_disk_erase ? i didnt quite get that15:11
jrollramineni: I believe so15:11
raminenijroll: hmm , does the cleaning framework proposed , will take care of all the in-band operations interface?15:12
raminenijroll : cleaning frameowrk proposed by JoshNang15:13
jrollramineni: I'm not entirely sure, you should ask jay or josh on gerrit if you can't reach them on irc15:15
raminenijroll: ok , will check with them ..thanks15:16
jrollnp15:16
*** rwsu-afk is now known as rwsu15:24
jrolldtantsur: lucasagomes https://review.openstack.org/#/c/149038/4/ironic/db/sqlalchemy/api.py15:25
jrollI think warn() is appropriate here because this is indicative of a problem imo15:25
* lucasagomes clicks15:25
dtantsurhmm15:26
jrollwe theoretically shouldn't have any nodes already locked on conductor startup15:26
dtantsurwell, yeah, seems right. though that means that you should not log a new message for every one15:26
jrollyeah agree15:26
jrollI'm going to have one log message, but include the uuids15:27
openstackgerritImre Farkas proposed openstack/ironic: DRAC specific RAID methods  https://review.openstack.org/14546415:27
*** ramineni has quit IRC15:27
lucasagomesjroll, right, yeah it's grand too. Also +1 for having 1 log message15:27
lucasagomesand only log it in case some field were updated15:28
jrollyeah15:29
jrollhere it is15:30
openstackgerritJim Rollenhagen proposed openstack/ironic: Clear locks on conductor startup  https://review.openstack.org/14903815:30
Shrewsjroll: w00t15:31
Shrewsdtantsur: also, i did some triaging of bug https://bugs.launchpad.net/ironic/+bug/1413934 for us, but i can't change the status of it. just fyi15:32
dtantsurShrews, just join Ironic Drivers team on lp15:33
dtantsurShrews, and thanks for looking into this :) so you think we can call it a duplicate of the 1st bug?15:35
Shrewsdtantsur: i believe so, if my analysis looks sensible15:35
*** achanda has joined #openstack-ironic15:35
Shrewsor a dupe of the second bug15:35
Shrewsnot sure which is more appropriate15:36
dtantsurShrews, I believe the first, as it caused the deploy failure15:37
*** Marga_ has joined #openstack-ironic15:37
Shrewsdtantsur: ++15:38
dtantsurShrews, thanks, updated15:38
*** Marga_ has quit IRC15:42
*** achanda has quit IRC15:43
*** Nisha has joined #openstack-ironic15:44
*** ramineni has joined #openstack-ironic15:50
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Driver and its iRMC Power module  https://review.openstack.org/14490115:52
*** jerryz has quit IRC16:01
*** achanda has joined #openstack-ironic16:02
*** booly-yam-4259 has joined #openstack-ironic16:05
*** DenisEgorenko has joined #openstack-ironic16:05
*** booly-yam-4259 has quit IRC16:06
*** achanda has quit IRC16:06
DenisEgorenkoHi all! Does here someone who can help with setup ironic with neutron on baremetal?16:06
*** booly-yam-4912 has joined #openstack-ironic16:07
*** ijw has joined #openstack-ironic16:07
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680316:11
openstackgerritMerged openstack/ironic: Remove argparse from requirements  https://review.openstack.org/14958216:17
*** ramineni1 has joined #openstack-ironic16:20
*** ramineni has quit IRC16:21
JoshNangramineni1: good morning!16:22
ramineni1JoshNang: hi , good morning :)16:24
ramineni1JoshNang: Regarding comment on https://review.openstack.org/#/c/142401/4/specs/kilo/ilo-cleaning-support.rst16:25
ramineni1JoshNang: Could you please explain bit more on the comment "The way this works is via a mix-in type of method. So the agent will provide this functionality for you, without iLo driver needing to declare it as all AIUI."16:25
ramineni1JoshNang: does it mean we dont need to have an interface for inband operations ?16:26
JoshNangthere will be a clean step in the agentdeploy driver called "erase_devices"16:27
*** bauzas is now known as bauwser16:27
JoshNangso if an operator wants to erase their devices inband, they can make sure that's enable. no help needed from the ilomanagement driver16:27
JoshNang*enabled16:27
JayFramineni1: you should only implement steps for clean/zap that are performed *by the iLO driver*16:28
JoshNangramineni1: and correct, you won't need an interface for inband, you could just add it in IPA (if it doesn't exist or doesn't meet your needs)16:28
JayFramineni1: if the clean/zap step is performed by another driver (like the agent, in band) it will be declared and implemented by teh agent16:28
ramineni1JoshNang, JayF : ok16:29
ramineni1so , if i want to do upgrade_firmware in-band , agent has the interface for it also?16:30
JayFyou would have to add that to the agent16:31
JayFbut the interface would go in the agent16:31
JayFso you'd add support for HP Firmware upgrade to Agent16:31
*** Marga_ has joined #openstack-ironic16:32
JoshNangluckily, JayF just refactored the agent to make adding new hardware support even easier!16:32
ramineni1JayF: right, ya have seen the code :)16:32
ramineni1JoshNang: Hope, cleaning code will be up soon , so will get better idea :)16:35
*** david-lyle_afk is now known as david-lyle16:35
JoshNangramineni1: yeah :/ i need to sit down and write it. i've been distracted for the past couple weeks16:35
*** Marga_ has quit IRC16:45
*** Marga_ has joined #openstack-ironic16:46
lucasagomesjroll, comment on the clear locks patchs... I think the logging needs a mapping when using format16:49
lucasagomescorrect if I'm wrong16:49
* lucasagomes goes google about it16:49
lucasagomesTypeError: format requires a mapping (when doing: logger.warn("%(test)s foo foo", "bar"))16:51
NobodyCamlucasagomes: sorry I'm a bit slow on looking at 14363616:52
lucasagomesNobodyCam, oh no worries at all :)16:53
NobodyCamhow nit picky should I bee16:53
NobodyCamI theink local booting dose have a Security impact16:53
JayFHowso?16:53
lucasagomesNobodyCam, hmm well I think you should comment on everything you think it's important16:53
lucasagomesyou mean in case of a electricity outage?16:54
lucasagomesand having the nodes coming up before the control plane or something?16:54
NobodyCamya customer nodes booting before control plane16:54
lucasagomesright, yeah there's nothing that can be done in that case for local boot (or I haven't thought about it)16:55
lucasagomesbut if it's a concern they can use the netboot for that16:55
JayFI really don't think that's a security concern at all16:55
lucasagomesit's optional to the deployer decide how to deploy it, we offer both16:55
JayFIt's just a different functionality16:55
JayFyou can't assume a deployer *wants* that behavior16:55
lucasagomes+ control plane machines may need local boot too, so we need to have it16:55
lucasagomesJayF, +116:56
JayFin fact for all of my use cases; I'd consider control plane outages causing data plane machines to not boot as a major bug16:56
JayFbut it's differring use cases16:56
lucasagomesyeah, it's a per instance configuration. So mixing netboot and localboot is possible16:57
lucasagomeslocalboot for control plane, netboot for the rest16:57
lucasagomesbut it's up to deployer to decide16:57
Shrewslucasagomes: jroll: on variables in translated messages --  http://docs.openstack.org/developer/oslo.i18n/guidelines.html#adding-variables-to-translated-messages16:58
Shrewsleft a note on the review16:59
lucasagomesShrews, ta much!16:59
*** ndipanov has quit IRC17:01
*** ifarkas has quit IRC17:02
*** ijw has quit IRC17:04
*** romcheg has left #openstack-ironic17:04
*** ijw has joined #openstack-ironic17:05
*** jcoufal_ has quit IRC17:07
*** DenisEgorenko has left #openstack-ironic17:08
*** jistr has quit IRC17:16
*** vdrok is now known as vdrok_afk17:16
*** Marga_ has quit IRC17:16
*** eghobo has joined #openstack-ironic17:18
*** bnemec is now known as beekneemech17:21
lucasagomesNobodyCam, lemme know if I should include the rationale about ^ on the review17:33
jrolllucasagomes: oops, you're right, thanks17:38
lucasagomesnp17:38
NobodyCamlucasagomes: reviewed17:39
lucasagomesNobodyCam, thanks!17:40
NobodyCamlet me know if my comments make sense17:40
*** Marga_ has joined #openstack-ironic17:40
lucasagomesNobodyCam, it does, I will put a mention there and fix the nits :)17:42
*** rameshg87 has joined #openstack-ironic17:44
*** Marga_ has quit IRC17:45
*** derekh has quit IRC17:45
NobodyCam:) I'll +2 ot after that17:45
NobodyCamhehehe17:45
openstackgerritAnusha Ramineni proposed openstack/ironic-specs: Implement Cleaning Operations for iLO drivers  https://review.openstack.org/14240117:46
*** Marga_ has joined #openstack-ironic17:46
*** booly-yam-4912_ has joined #openstack-ironic17:46
*** romcheg has joined #openstack-ironic17:46
*** yjiang5 is now known as yginger517:46
*** booly-yam-4912 has quit IRC17:46
*** Marga_ has quit IRC17:47
*** Marga__ has joined #openstack-ironic17:47
lucasagomesNobodyCam, cool ! working on it :)17:47
jrollNobodyCam: currently eating a glorious bagel :P17:47
*** Marga__ has quit IRC17:48
*** romcheg has quit IRC17:51
dtantsurcalling it a day, have a nice weekend!17:57
rameshg87NobodyCam, lucasagomes, can you please have a look at raid configuration spec: https://review.openstack.org/#/c/135899/1918:01
openstackgerritLucas Alvares Gomes proposed openstack/ironic-specs: Local boot support with partition images  https://review.openstack.org/14363618:02
*** adam_g_out is now known as adam_g18:02
lucasagomesrameshg87, hi, I started it, but haven't finished :( (600+ lines). Mind if I publish it on monday18:02
lucasagomesI'm almost finishing work today18:03
NobodyCamjroll: gah and your not sharing18:03
rameshg87lucasagomes, sure :)18:03
NobodyCamrameshg87: looking now18:03
lucasagomesNobodyCam, updated #143636, lemme know if that explains the problem well and all18:03
rameshg87NobodyCam, thanks :)18:03
NobodyCamlucasagomes: reviewed18:05
lucasagomesNobodyCam, thanks!18:05
NobodyCamthank you :)18:05
dtantsurlucasagomes, +A18:08
dtantsurnow really g'night :)18:08
lucasagomes\o/!18:08
lucasagomesdtantsur, thanks! good night enjoy the weekend18:08
NobodyCamnight dtantsur :)18:08
NobodyCamoh ya and weekend too18:08
openstackgerritDmitry Tantsur proposed openstack/ironic: Add support for driver-specific periodic tasks  https://review.openstack.org/14752318:08
dtantsurhope to finish this ^^^ next week, if my oslo patches land...18:09
*** dtantsur is now known as dtantsur|afk18:09
NobodyCamdtantsur: I'll look at that one in a bit18:09
NobodyCamcore's I have a question on the raid spec. would like options on something18:10
openstackgerritMerged openstack/ironic-specs: Local boot support with partition images  https://review.openstack.org/14363618:10
*** romcheg has joined #openstack-ironic18:11
NobodyCamthe spec calls for a delete endpoint $ ironic node-delete-raid-configuration <node-uuid> it also has $ ironic node-set-raid-configuration <node-uuid> --file blah.json what about removing the delete endpoint and having somehitng like $ ironic node-set-raid-configuration <node-uuid> delete18:11
NobodyCamthis was commented on rev 18 of the spec18:12
rameshg87NobodyCam, i have added to the alternatives for now18:12
*** spandhe has joined #openstack-ironic18:13
NobodyCamyea I'm polling folks to see if I can get a sense of the option18:13
rameshg87NobodyCam, also the main difference from the previous revision is that i have added a provision for driver-specific filters and moved model, vendor, firmware_version, etc there ..18:13
NobodyCamyes :) ++18:14
*** harlowja_away is now known as harlowja18:14
*** Marga_ has joined #openstack-ironic18:15
lucasagomesyeah one command seems good :)18:16
lucasagomesand I will call it a day now18:16
lucasagomeshave a good night everyone,  enjoy the weekend!18:16
NobodyCamthank you lucas.. have a great weekend18:16
BadCub_good weekend Lucas!18:16
lucasagomessee y'all!18:16
*** Marga_ has quit IRC18:19
*** rameshg87 has quit IRC18:20
*** lucasagomes has quit IRC18:27
*** Marga_ has joined #openstack-ironic18:34
*** Nisha has quit IRC18:34
*** ramineni1 has quit IRC18:37
*** achanda has joined #openstack-ironic18:40
*** ChuckC has joined #openstack-ironic18:40
*** romcheg has quit IRC18:41
*** achanda has quit IRC18:41
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/14970718:42
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/14863618:42
*** ChuckC_ has quit IRC18:43
*** yuriyz has quit IRC18:43
*** vdrok_afk has quit IRC18:43
*** pensu has quit IRC18:46
*** david-lyle has quit IRC18:46
*** achanda has joined #openstack-ironic18:46
*** Marga_ has quit IRC18:51
*** penick has joined #openstack-ironic18:57
*** Marga_ has joined #openstack-ironic18:59
*** dprince has quit IRC19:12
*** alexpilotti has quit IRC19:12
openstackgerritJim Rollenhagen proposed openstack/ironic: Clear locks on conductor startup  https://review.openstack.org/14903819:13
jrollNobodyCam: lol19:13
NobodyCamhuh19:15
NobodyCamany one happen to know Shivanand's irc handle?19:17
*** ChuckC_ has joined #openstack-ironic19:18
jrollNobodyCam: what's the last name?19:19
NobodyCamor better Shivanand Tendulker19:20
NobodyCamdoh19:20
NobodyCamShivanand Tendulker19:20
NobodyCamjust a basic question on hte UEFI support spec19:21
*** Haomeng has quit IRC19:21
*** ChuckC has quit IRC19:21
NobodyCamthe spec calls out: The secure boot enable/disable operation would get carried out as part of Nova boot flavor ... then later Nova driver impact: None19:22
NobodyCamdo we already pass secure boot setting down?19:22
jrolloh, stendulker19:23
jrolldoesn't seem to be online19:23
jrollidk, I think nisha had a spec for that or something19:23
*** Haomeng has joined #openstack-ironic19:24
*** Marga_ has quit IRC19:24
*** Marga_ has joined #openstack-ironic19:24
*** athomas has quit IRC19:25
*** vdrok has joined #openstack-ironic19:25
*** dprince has joined #openstack-ironic19:28
*** Marga_ has quit IRC19:28
NobodyCambrb ... quick walkies time19:35
*** booly-yam-4912_ has quit IRC19:36
*** lifeless has quit IRC19:37
*** lifeless has joined #openstack-ironic19:37
*** Marga_ has joined #openstack-ironic19:38
*** pelix has quit IRC19:38
*** Marga_ has quit IRC19:39
*** Marga_ has joined #openstack-ironic19:40
*** romcheg has joined #openstack-ironic19:45
JayFNobodyCam: she passes a capability down20:02
JayFNobodyCam: I thought?20:03
*** Marga_ has quit IRC20:05
*** Marga_ has joined #openstack-ironic20:06
*** romcheg has quit IRC20:06
NobodyCamJayF: I think thats it. I left a comemnt on the review20:07
NobodyCamoh easy add driver spec ... with 2 +2's .. anyone else wanta have a look https://review.openstack.org/#/c/13951720:08
*** Marga_ has quit IRC20:10
*** Marga_ has joined #openstack-ironic20:13
JayFNobodyCam: is it strange of me to want them to explicitly say the drivers they're adding in that spec?20:14
JayFnevermind20:14
JayFI'm blind20:14
JayFContinuous integration (CI) support will be added for Cisco UCS B/C/M series20:14
JayFservers.20:14
NobodyCamyea it there... just hidden20:14
* JayF doubts it20:14
NobodyCamlol20:15
* NobodyCam whants a c7000 to play with20:15
NobodyCamwants even20:15
*** Haomeng|2 has joined #openstack-ironic20:15
ijwNobodyCam: come work for us, we're hiring and our lab is full of toys ;)20:15
*** Haomeng has quit IRC20:15
*** jcoufal has joined #openstack-ironic20:15
NobodyCamlol20:16
ijwToo blatant ;)20:16
*** eghobo has quit IRC20:16
NobodyCamwe do have fun toys at hp too20:16
NobodyCam:)20:16
JayFHey, it's not recruiting if it's not shameless20:16
* JayF has fun toys that have open source designs20:16
JayFlol20:16
JayFNobodyCam: I -1'd that pending clarification on the testing bit20:17
JayFNobodyCam: sorry :( I always feel like a party pooper when I do that on a +4'd thing20:17
BadCub_(h)20:17
*** Marga_ has quit IRC20:17
NobodyCam:) JayF not at all20:18
*** Marga_ has joined #openstack-ironic20:18
NobodyCamwe all have different context in mind that why I ask of others to weigh in20:19
*** priteau has quit IRC20:34
*** ijw has quit IRC20:35
*** romcheg has joined #openstack-ironic20:41
*** Alonha has joined #openstack-ironic20:41
*** achanda has quit IRC20:47
NobodyCambrb20:51
openstackgerritJim Rollenhagen proposed openstack/ironic: Clear locks on conductor startup  https://review.openstack.org/14903820:53
*** achanda has joined #openstack-ironic20:58
*** Alonha has quit IRC20:58
openstackgerritDevananda van der Veen proposed openstack/ironic: Rename NOSTATE to AVAILABLE  https://review.openstack.org/14975420:59
devananda^^^ :)21:00
*** Alonha has joined #openstack-ironic21:02
*** dprince has quit IRC21:02
*** booly-yam-6140 has joined #openstack-ironic21:12
*** Marga_ has quit IRC21:22
*** Marga_ has joined #openstack-ironic21:22
*** Marga_ has quit IRC21:24
*** Marga_ has joined #openstack-ironic21:24
devanandaJayF: if ya'll have time, I'd love eyes and some help testing ^21:25
openstackgerritDevananda van der Veen proposed openstack/ironic: Rename NOSTATE to AVAILABLE  https://review.openstack.org/14975421:26
* devananda fixes commit message21:26
*** booly-yam-6140 has quit IRC21:27
*** Marga_ has quit IRC21:29
NobodyCamvictor_lowther: I'm about to post a shocking comment on you latest spec21:33
victor_lowthera shocking comment?21:33
victor_lowtherHow shocking?21:34
victor_lowther9 volt battery shocking?21:34
victor_lowtherStatic electric cat hair shocking?21:34
NobodyCamhave a read21:34
victor_lowtherAlrighty then.21:35
victor_lowtherThat will be an interesting conversation21:35
jrolldevananda: eep21:36
victor_lowtherseeing as how we have a few specs for vendor passthrough apis out there already. :)21:36
NobodyCamyes21:36
jrolldevananda: I also don't see the migration21:36
NobodyCamand shouldn't21:36
victor_lowtherWell, then, I will make popcorn.21:37
devanandajroll: eh?21:37
devanandaoh, hah21:37
victor_lowtherand we have a topic of conversation for the midcycle. :)21:37
victor_lowtherAnyways, back to implementing the spec.21:38
openstackgerritDevananda van der Veen proposed openstack/ironic: Rename NOSTATE to AVAILABLE  https://review.openstack.org/14975421:38
devanandajroll: thanks for catching that >_<21:38
jrolldevananda: just thinking about how our deploys run migrations, this could be sketchy, but that's not something you need to think about :)21:38
devanandajroll: so I dunno how you guys do that, but this should work in two cases21:38
devananda- you roll out new code, then update the db21:38
devanandaer, hit enter too son21:39
devanandasoon21:39
NobodyCam*cores* please see my comment on https://review.openstack.org/#/c/14938321:39
devananda1. stop services, roll out new code, start services, migrate db21:39
devananda2. stop services, roll out new code, migrate db, start services21:39
devanandaeither way, it should work21:39
jrolldevananda: right, I thought about it in case 1, we do case 221:39
devanandawhat you CAN"T do -- migrate db, then roll out new code21:39
jrollshould be fine21:39
devanandak k21:39
jrollwell21:39
jrollnah nevermind, I'm happy21:40
devananda:)21:40
devanandathere's a corresponding change for Nova21:40
devanandahowever21:40
jrollright21:40
jrollthat needs to go first21:40
devanandathis gets into the "which do we upgrade when"21:40
devanandawith this patch, juno-nova will break with kilo-ironic. but kilo-nova should still work with juno-ironic. *I THINK*21:41
jrollyeah21:41
devanandawe alraedy discussed that -- but this needs to get documented when we land these two patches21:41
jrollshould add a dep on top of the ironic patch21:41
jrolland on the first ironic patch note that it should not land until the nova patch lands21:42
* devananda WIPs it21:43
*** trown is now known as trown|outttypeww21:53
*** andreykurilin_ has joined #openstack-ironic21:57
*** Marga_ has joined #openstack-ironic22:00
*** Marga_ has quit IRC22:05
*** jcoufal has quit IRC22:15
*** jcoufal has joined #openstack-ironic22:16
*** Marga_ has joined #openstack-ironic22:23
*** chlong has joined #openstack-ironic22:25
*** Alonha has quit IRC22:30
openstackgerritDevananda van der Veen proposed openstack/ironic: Add a TODO file  https://review.openstack.org/14977722:39
*** jcoufal has quit IRC22:40
*** david-ly_ has joined #openstack-ironic23:07
*** david-ly_ is now known as david-lyle23:11
NobodyCamaweeks: just poke on https://review.openstack.org/#/c/112682 to see if its still a thing23:22
aweeksNobodyCam: it might be, if I ever have time to get back to it :(23:23
NobodyCam:(23:23
aweekswe haven't implemented any of it internally either23:24
NobodyCam:(23:25
*** Marga_ has quit IRC23:32
openstackgerritMerged openstack/ironic: Allow operations on DEPLOYFAIL'd nodes  https://review.openstack.org/14902723:32
*** Marga_ has joined #openstack-ironic23:32
*** Marga_ has quit IRC23:37
*** andreykurilin_ has quit IRC23:40
*** penick has quit IRC23:49
*** Marga_ has joined #openstack-ironic23:57

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