Friday, 2017-06-16

*** MattMan has quit IRC00:03
*** MattMan has joined #openstack-ironic00:03
*** rloo has quit IRC00:08
*** jkilpatr has quit IRC00:12
*** ijw has joined #openstack-ironic00:18
*** tuanluong has joined #openstack-ironic00:24
*** rama_y has quit IRC00:25
*** ijw has quit IRC00:27
*** shuyingya has joined #openstack-ironic00:28
*** zhurong has joined #openstack-ironic00:34
*** ijw has joined #openstack-ironic00:47
*** amotoki is now known as amotoki_away00:48
*** amotoki_away is now known as amotoki00:52
*** ijw has quit IRC00:55
*** Goneri has quit IRC01:00
*** phuongnh has joined #openstack-ironic01:01
*** ijw has joined #openstack-ironic01:06
*** ijw has quit IRC01:06
*** zhurong has quit IRC01:30
*** vinsh has quit IRC01:32
openstackgerritWang KaiFeng proposed openstack/ironic master: Make IP address of socat console configurable  https://review.openstack.org/47320101:46
*** zhurong has joined #openstack-ironic01:49
*** Sukhdev has quit IRC01:57
*** rama_y has joined #openstack-ironic02:26
*** rbudden has quit IRC02:29
openstackgerritNisha Agarwal proposed openstack/ironic master: py3.5:Workaround fix for forcing virtualbmc installation with pip2  https://review.openstack.org/47391302:40
*** litao__ has joined #openstack-ironic02:44
*** tuanluong_ has joined #openstack-ironic02:50
*** jaganathan has joined #openstack-ironic02:59
*** jaganathan has quit IRC03:09
*** phuongnh has quit IRC03:09
*** phuongnh has joined #openstack-ironic03:09
*** chlong has quit IRC03:12
*** Sukhdev has joined #openstack-ironic03:22
*** sasha2 has quit IRC03:25
openstackgerritChangBo Guo(gcb) proposed openstack/ironic-lib master: Adjust test with option 'backend'  https://review.openstack.org/47406603:26
*** IridiumScaffold has joined #openstack-ironic03:30
*** wrouesnel has quit IRC03:33
*** jaganathan has joined #openstack-ironic03:41
*** Nisha_Agarwal has joined #openstack-ironic03:46
Nisha_Agarwalpmannidi, hi03:47
Nisha_Agarwalpmannidi, regarding https://bugzilla.redhat.com/show_bug.cgi?id=1460915#c303:49
openstackbugzilla.redhat.com bug 1460915 in openstack-ironic "The pxe_ilo driver ignores power requests under certain conditions with HP BL460" [High,Assigned] - Assigned to dtantsur03:49
Nisha_Agarwalpmannidi, dtantsur|afk this looks an issue in firmware.03:50
Nisha_Agarwalpmannidi, dtantsur|afk the hardware team here(who deals with ilo and enclosure) need certain details03:50
*** jaosorior_away is now known as jaosorior03:53
Nisha_Agarwalpmannidi, dtantsur|afk the bugzilla doesnt allwo me to edit the bug as i dont have login to it. could you get "the result of the OA command – “show all”" from customer's system? apart from that complete conductor logs would be required.03:56
*** jaganathan has quit IRC04:14
*** jaganathan has joined #openstack-ironic04:14
*** crushil has quit IRC04:17
*** crushil has joined #openstack-ironic04:18
*** phuongnh has quit IRC04:19
*** phuongnh has joined #openstack-ironic04:20
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392604:23
*** zhurong has quit IRC04:25
openstackgerritNisha Agarwal proposed openstack/proliantutils master: Discover Storage details  https://review.openstack.org/34137004:27
*** zhurong has joined #openstack-ironic04:30
*** adisky__ has joined #openstack-ironic04:31
*** phuongnh has quit IRC04:31
*** phuongnh has joined #openstack-ironic04:31
*** udesale has joined #openstack-ironic04:32
*** hamzy has joined #openstack-ironic04:36
*** mvpnitesh has joined #openstack-ironic04:46
*** samodid has joined #openstack-ironic04:57
openstackgerritNisha Agarwal proposed openstack/proliantutils master: Discover Storage details  https://review.openstack.org/34137005:02
*** Sukhdev has quit IRC05:13
openstackgerritAnshul Jain proposed openstack/proliantutils master: Adds 'activate_license' for redfish systems  https://review.openstack.org/46878305:44
*** tuanluong has quit IRC05:46
*** tuanluong_ has quit IRC05:46
*** mjura has joined #openstack-ironic05:57
*** ChubYann has quit IRC05:57
openstackgerritMerged openstack/proliantutils master: Add upper-constraints to testenv setup  https://review.openstack.org/45458205:59
*** mvpnitesh has quit IRC06:05
*** mvpnitesh has joined #openstack-ironic06:06
*** samodid has quit IRC06:06
*** milan has joined #openstack-ironic06:11
milanmorning Ironic! :)06:19
*** rcernin has joined #openstack-ironic06:19
*** jaganathan has quit IRC06:22
*** jaganathan has joined #openstack-ironic06:23
*** rcernin has quit IRC06:24
rama_ymilan, morning to you :)06:28
milanmorning rama_y! :)06:28
* milan early bird today06:29
*** e0ne has joined #openstack-ironic06:33
*** rcernin has joined #openstack-ironic06:39
*** e0ne has quit IRC06:43
*** pcaruana has joined #openstack-ironic06:43
*** samodid has joined #openstack-ironic06:48
*** mtanino has quit IRC06:51
*** Nisha_ has joined #openstack-ironic06:59
*** Nisha_Agarwal has quit IRC06:59
*** IridiumScaffold has quit IRC07:05
*** wrouesnel has joined #openstack-ironic07:05
*** tesseract has joined #openstack-ironic07:09
*** jtomasek has joined #openstack-ironic07:10
openstackgerritMerged openstack/ironic master: Validate portgroup physical network consistency  https://review.openstack.org/46993207:11
*** ralonsoh has joined #openstack-ironic07:13
*** ccamacho has quit IRC07:14
*** hw_wutianwei has joined #openstack-ironic07:16
*** fragatina has joined #openstack-ironic07:20
*** hoonetorg has joined #openstack-ironic07:31
*** stendulker has joined #openstack-ironic07:35
*** dtantsur|afk is now known as dtantsur07:49
dtantsurmorning ironic07:49
*** jaganathan has quit IRC07:52
*** paramite has joined #openstack-ironic07:55
stendulkerGood morning dtantsur07:58
*** zzzeek has quit IRC08:00
*** thaj has joined #openstack-ironic08:01
*** zzzeek has joined #openstack-ironic08:02
thajHi08:05
*** jaganathan has joined #openstack-ironic08:06
*** mvpnitesh has quit IRC08:06
*** mvpnitesh has joined #openstack-ironic08:06
thajCan anyone give pointer for the error "Expected a MAC address but received  xx.xx.xx.xx.xx (HTTP 400)" while installing "devstack" in our baremetal nodes?08:07
vdrokgood morning all!08:09
dtantsurmorning vdrok08:11
dtantsurthaj: hard to tell without a context. what was happening right before the error?08:12
dtantsurthaj: this is devstack on baremetal, right? so I guess you have an inventory file? are MACs correct there?08:14
dtantsurmorning stendulker also08:14
milanmorning dtantsur, stendulker, thaj, vdrok and all! :)08:14
vdrokmorning dtantsur and milan08:15
stendulkerGood morning milan :)08:15
thajBefore the errror, node_id is created. (ironic port-create --address XXXX)08:15
vdrokand morning stendulker08:15
stendulkerGood morning vdrok and thaj  :)08:15
dtantsurthaj: is "xx.xx.xx.xx.xx" the actual value from your inventory? it does not look right for sure08:15
dtantsurhey-hey milan08:15
thajDmirtry, I have 2 files. One is local.conf and another is hardware_info file, which has BMC details of another node.08:16
dtantsurthaj: right. I'm talking about hardware_info.08:16
thajHi Milan. Good morning.08:16
milanhey dtantsur! Gonna be a decorator patch after all :D (the filter)08:16
*** ccamacho has joined #openstack-ironic08:17
dtantsurmilan: /me runs screaming :D08:17
milanoh and TGIF! I'd almost forget what day it was!08:17
dtantsur!!!08:17
openstackdtantsur: Error: "!!" is not a valid command.08:17
milanlol08:17
dtantsuropenstack: I respectfully disagree :D08:17
milan:DDD08:17
*** e0ne has joined #openstack-ironic08:18
thajDmitry. In hardware_info file, i have ip address, mac address, user and password of another node(BMC )08:18
milanand with that great news, I've run out of coffee -> me errands :P CU ironic in a bit08:19
*** milan is now known as milan|bbl08:19
*** milan|bbl has quit IRC08:20
*** rcernin has quit IRC08:21
thajThe exact error is "Expected a MAC address but received 10.47.80.AE.FA.93. (HTTP 400)"08:21
*** mgoddard has joined #openstack-ironic08:21
*** rcernin has joined #openstack-ironic08:22
thajGood morning stendulker.08:22
dtantsurthaj: MAC address should use : instead of .08:28
dtantsurthaj: try putting 10:47:80:ae:fa:93 there08:28
thajDmitry, u mean all in small case?08:29
*** shuyingya has quit IRC08:33
*** shuyingya has joined #openstack-ironic08:33
dtantsurthaj: more importantly, using semicolons instead of dots08:33
dtantsurI don't this case matters08:33
dtantsurs/this/think/08:33
dtantsurvsaienk0: morning! please check if your concerns were resolved on https://review.openstack.org/#/c/406290/ as it's close to landing08:34
patchbotpatch 406290 - ironic - Wire in storage interface attach/detach operations08:34
thajI think it is (colon , :), if i am not wrong.08:37
*** zhurong has quit IRC08:37
*** lucas-afk is now known as lucasagomes08:39
*** mvpnitesh has quit IRC08:39
*** mvpnitesh has joined #openstack-ironic08:40
*** shuyingy_ has joined #openstack-ironic08:43
*** bcornec1 has joined #openstack-ironic08:44
*** zhurong has joined #openstack-ironic08:44
*** shuyingya has quit IRC08:46
thajHi Dmitry, One more question. What is the use of the parameter "IRONIC_BAREMETAL_BASIC_OPS".  When i set this "True", installation goes successful. But, it is hard-coded as "False" in the Ironic.sh script. Any idea?08:49
dtantsurright, it's colon, my English lets me down this morning >_<08:49
dtantsurthaj: tl;dr set it to True. I don't remember why we disable it by default. It is needed for configuration of the PXE environment, etc.08:51
thajtl;dr ???08:53
dtantsur"short version", sorry08:53
*** athomas has quit IRC08:56
openstackgerritMerged openstack/bifrost master: scripts: install-deps: Add 'make' to the required packages  https://review.openstack.org/47322508:59
*** athomas has joined #openstack-ironic09:09
*** makowals has quit IRC09:14
openstackgerritMark Goddard proposed openstack/ironic master: Unit test consistency: DB base and utils prefix  https://review.openstack.org/47491809:14
*** Nisha_ has quit IRC09:17
*** makowals has joined #openstack-ironic09:17
openstackgerritDao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume  https://review.openstack.org/47274009:22
openstackgerritMark Goddard proposed openstack/ironic master: Remove unnecessary line in docstring  https://review.openstack.org/47492109:24
thajThanks Dmitry. It works!. Awesome. But it is my silly mistake.09:24
dtantsurthaj: no problem :) I'm glad it works now09:28
*** sambetts|afk is now known as sambetts09:28
openstackgerritMark Goddard proposed openstack/ironic master: Physical network aware VIF attachment  https://review.openstack.org/47040609:28
openstackgerritMark Goddard proposed openstack/ironic master: Expose ports' physical network attribute in API  https://review.openstack.org/46993309:28
*** snecklifter_ has joined #openstack-ironic09:35
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add tests for OSC baremetal chassis create command  https://review.openstack.org/38249609:56
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Add basic tests for OSC plugin baremetal driver commands  https://review.openstack.org/36735909:56
*** zhurong has quit IRC09:59
*** thaj has quit IRC10:06
openstackgerritMerged openstack/ironic master: Wait until iDRAC is ready before out-of-band cleaning  https://review.openstack.org/46608610:11
openstackgerritMerged openstack/ironic master: Wire in storage interface attach/detach operations  https://review.openstack.org/40629010:19
openstackgerritMerged openstack/ironic master: Remove unnecessary line in docstring  https://review.openstack.org/47492110:19
*** mvpnitesh has quit IRC10:23
pas-hamorning ironic :)10:24
sambettshey pas-ha10:30
openstackgerritMerged openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/47464310:34
*** amarao has joined #openstack-ironic10:43
*** stendulker has quit IRC10:45
openstackgerritNguyen Hung Phuong proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371710:47
*** hw_wutianwei has quit IRC10:50
*** mvpnitesh has joined #openstack-ironic10:52
*** phuongnh has quit IRC10:59
*** jkilpatr has joined #openstack-ironic10:59
*** bfournie has quit IRC11:02
*** bfournie has joined #openstack-ironic11:03
*** bfournie has quit IRC11:04
paramitedtantsur, hey, just FYI using the latest master instead of what is in delorean-passed-ci, the ironic problem did not occur11:21
*** bfournie has joined #openstack-ironic11:21
*** lucasagomes is now known as lucas-hungry11:22
dtantsurparamite: good to know! I was quite confused by it.11:22
paramitedtantsur, well, concerning is that we have such bugged version of ironic in delorean-passed-ci11:23
paramiteI was assuming that passed-ci means more stable than latest master :)11:23
dtantsurwell, only for aspects that are tested by CI11:23
* dtantsur wonders how old is passed-ci now11:24
openstackgerritDmitry Tantsur proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332411:47
dtantsurclean rebase ^^^11:47
openstackgerritDmitry Tantsur proposed openstack/ironic master: Logic for skipping deployment with BFV  https://review.openstack.org/45424311:48
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add CRUD notification objects for volume connector and volume target  https://review.openstack.org/46393011:48
*** thaj has joined #openstack-ironic11:49
openstackgerritDmitry Tantsur proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458611:49
dtantsurTheJulia: I hope you don't mind ^^ (all rebased cleanly)11:50
*** udesale has quit IRC11:51
*** mgoddard has quit IRC12:01
*** shuyingy_ has quit IRC12:10
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/47464212:12
*** Nisha_Agarwal has joined #openstack-ironic12:12
*** rbudden has joined #openstack-ironic12:15
TheJuliadtantsur: I don't mind at all :) it takes a community12:15
dtantsur:)12:16
*** root4 has joined #openstack-ironic12:19
*** dprince has joined #openstack-ironic12:20
*** Nisha_ has joined #openstack-ironic12:21
*** Nisha_Agarwal has quit IRC12:21
*** root2 has quit IRC12:23
openstackgerritJulia Kreger proposed openstack/ironic master: Enable cinder storage interface for generic hardware  https://review.openstack.org/46390812:24
openstackgerritJulia Kreger proposed openstack/ironic master: Add storage interface field to node-related notifications  https://review.openstack.org/46397212:24
openstackgerritJulia Kreger proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633312:24
openstackgerritJulia Kreger proposed openstack/ironic master: Add a guide for Devstack configuration for boot-from-volume  https://review.openstack.org/46656812:24
TheJuliamore rebases12:25
openstackgerritDao Cong Tien proposed openstack/ironic master: Adds clean step 'retore_irmc_bios_config' to iRMC drivers  https://review.openstack.org/41640312:27
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Introduce oslo.messaging RPC  https://review.openstack.org/46286112:32
*** trown|outtypewww is now known as trown12:37
TheJuliadtantsur: so, I believe the wire-in patch marks the list for the week... maybe add an item or three?12:38
dtantsurTheJulia: we have https://review.openstack.org/#/c/408556/ also12:39
patchbotpatch 408556 - ironic - Add new dbsync command with first online data migr...12:39
dtantsurTheJulia: otherwise yeah.. I added the next BFV patch, hopefully folks won't mind12:41
TheJuliaI think it is understood :)12:42
dtantsurright12:42
* TheJulia has reporting and needs to begin working on a presentation for a conference12:44
*** lucas-hungry is now known as lucasagomes12:44
*** gcb has quit IRC12:46
*** Nisha_ has quit IRC12:53
*** milan has joined #openstack-ironic12:54
*** jaganathan has quit IRC12:56
TheJuliaping me if anyone needs me to look at something this morning, I'm burring myself in documents and presentations12:56
dtantsurgood luck!12:57
*** bcornec1 has quit IRC12:59
*** sasha2 has joined #openstack-ironic13:00
*** jcoufal has joined #openstack-ironic13:03
TheJulialuck has nothing to do with it. Just getting it all out of my brain in a cogent statement is the key :)13:03
*** cdearborn has joined #openstack-ironic13:09
*** jcoufal_ has joined #openstack-ironic13:18
*** jcoufal has quit IRC13:19
*** fragatin_ has joined #openstack-ironic13:21
*** fragatina has quit IRC13:21
*** dtantsur is now known as dtantsur|brb13:25
*** crushil has quit IRC13:25
*** mvpnitesh has quit IRC13:31
*** baoli has joined #openstack-ironic13:31
aarefievmilan: hey, I had to rebase https://review.openstack.org/#/c/453057/13:34
patchbotpatch 453057 - ironic-inspector - Use new oslo db EngineFacade13:34
aarefievmind to check13:34
*** mgoddard has joined #openstack-ironic13:35
milanaarefiev, hi13:36
milanyup will do13:36
aarefievthanks13:36
* milan refactoring the driver patch13:36
milanbut I'll check this one13:36
*** crushil has joined #openstack-ironic13:38
milanaarefiev, a+13:39
*** adisky__ has quit IRC13:39
*** crushil has quit IRC13:39
aarefievmilan: thanks a lot, I'm going to update following refactoring patch13:40
aarefievI'll add more tests13:40
milanaarefiev, cool! thanks! I hope to post mine tonight13:40
milanhad to refactor the filter so lots of test case upadtes13:41
milanbtw there will be a decorator ;)13:41
aarefievmilan: do you remember it's friday night :)13:42
milandtantsur|brb was very happy to hear ;)13:42
milanaarefiev, is it? :D13:42
milanwhere's NobodyCam TGIF?!? :D13:42
milanaarefiev, thing is I'll forget what I wanted till Monday :D13:43
aarefievmilan: :) btw I've added decorators for session creating to my refactoring patch13:43
aarefievI hope you will enjoy13:43
milanaarefiev, oooh nice! :D13:43
milandefinitely13:43
milanlol]13:43
aarefiev:)13:44
*** dtantsur|brb is now known as dtantsur14:01
*** mrchao has joined #openstack-ironic14:10
*** chlong has joined #openstack-ironic14:14
*** vinsh has joined #openstack-ironic14:16
*** mjura has quit IRC14:36
*** jaosorior has quit IRC14:37
*** ricardoas has joined #openstack-ironic14:39
*** jcoufal has joined #openstack-ironic14:40
openstackgerritRodrigo Farias de Macedo proposed openstack/ironic master: [WIP] Migrate oneview driver to use python-hpOneView  https://review.openstack.org/46687214:40
*** jcoufal_ has quit IRC14:41
*** dansmith is now known as superdan14:49
TheJuliamilan: Well, where he is presently located, it is nearing 8AM local, so hopefully soon() :)14:49
milanwow, 8 am? Japan or so?14:50
milanTheJulia, I remember him mentioning travels14:50
TheJuliaYeah, he should be flying home today or tomorrow, but he is pretty good about checking in with folks on IRC before traveling :)14:51
milanbut I guess its Sat already then14:51
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic master: Allow to set image_url to glance API file  https://review.openstack.org/47499614:52
*** cdearborn__ has joined #openstack-ironic14:53
* milan 's brain tries to wrap around having 2 Saturdays in a row14:54
*** hjensas has quit IRC14:54
milanbut I guess there was no Monday14:54
*** hjensas has joined #openstack-ironic14:54
TheJuliamilan: as in monday was a myth, or previously saturday was a myth?14:55
milanTheJulia, I'm still time traveling :)14:55
TheJuliaWait! If you can travel through time, that also must mean that you have a teleportation device14:56
milanand a what if machine14:58
milan...these mocks...15:00
* milan having type 2 fun unit testing decorators15:01
*** skazi_ has joined #openstack-ironic15:02
*** mrchao has quit IRC15:03
*** cdearborn has quit IRC15:08
*** rpioso has joined #openstack-ironic15:15
*** rama_y has quit IRC15:15
*** rcernin has quit IRC15:15
*** jaganathan has joined #openstack-ironic15:16
*** jcoufal has quit IRC15:17
*** jcoufal has joined #openstack-ironic15:18
*** pcaruana has quit IRC15:20
*** samodid has quit IRC15:24
NobodyCamGood Morning Ironic'ers ... and ...15:25
vdroknicodemos: hey, if folks from your team are around, left a comment on https://review.openstack.org/43646915:25
patchbotpatch 436469 - ironic - Adds onetime boot in oneview drivers15:25
NobodyCamTGIF!15:26
vdrokgood morning NobodyCam !15:26
NobodyCammorning vdrok and milan15:26
NobodyCam:)15:26
NobodyCammilan: travel week so was at hotel.15:26
milanhehe :) morning NobodyCam!15:26
vdrokand good morning TheJulia15:26
milanNobodyCam, so is it Saturday or Friday for you? :)15:27
openstackgerritDmitry Tantsur proposed openstack/ironic stable/ocata: Explicitly use python 2 for the unit-with-driver-libs tox target  https://review.openstack.org/47500715:27
*** jaypipes is now known as leakypipes15:27
milanor rather how long has it been Friday?15:27
NobodyCamIt's Friday here!15:27
NobodyCamhehehe15:27
milan:D great! that means: TGIF!15:28
*** SerenaFeng has joined #openstack-ironic15:30
dtantsurmorning NobodyCam :)15:30
NobodyCamGood Morning dtantsur :) happy Friday!15:30
dtantsur\o/15:30
openstackgerritMerged openstack/ironic-inspector master: Use new oslo db EngineFacade  https://review.openstack.org/45305715:32
*** e0ne has quit IRC15:34
*** vinsh has quit IRC15:38
*** aarefiev is now known as aarefiev_afk15:39
*** Goneri has joined #openstack-ironic15:40
*** rama_y has joined #openstack-ironic15:47
*** rama_y_ has joined #openstack-ironic15:48
*** rama_y has quit IRC15:48
*** snecklifter_ has quit IRC15:52
jlvillalmilan: Tell me more about "unit testing decorators"? I'm curious15:53
jlvillalGood morning Ironic :)15:53
milanjlvillal, morning! :D15:53
vdrokgood morning jlvillal15:54
jlvillal:)15:54
NobodyCamGood Morning jlvillal :)15:55
jlvillal:)15:55
*** SerenaFeng has quit IRC15:55
vdrokmilan: are you trying to mock the decorator itself? :)15:56
* milan 'd prefer not to answer, profanity imminent O:-)15:56
NobodyCamRut-Roh...15:57
rama_y_good morning all!15:57
NobodyCammorning rama_y_ :)15:58
rama_y_morning NobodyCam, jlvillal, vdrok, milan!15:58
jlvillal:)15:58
milanmorning rama_y_! :)15:58
vdrokmorning rama_y_15:58
*** samodid has joined #openstack-ironic15:59
dtantsurmorning jlvillal, rama_y_15:59
* milan relies on a log mock assertions, please don't tell dtantsur .review(-2)15:59
* dtantsur -2es before leaving for a day15:59
milanlol16:00
NobodyCamlol16:00
*** bnemec is now known as beekneemech16:01
dtantsursushi'o'clock! see you all, have a nice weekend16:02
milansushi \o/16:02
milandtantsur, cu!16:02
*** dtantsur is now known as dtantsur|afk16:03
NobodyCamnight dtantsur|afk :)16:04
NobodyCamhave a good weekend16:04
rama_y_have a good weekend dtantsur|afk16:05
*** ccamacho has quit IRC16:05
*** ijw has joined #openstack-ironic16:07
vdrokgood night dtantsur|afk !16:08
*** trown is now known as trown|lunch16:13
*** jaganathan has quit IRC16:32
*** samodid has quit IRC16:33
openstackgerritMerged openstack/ironic-lib master: Updated from global requirements  https://review.openstack.org/47464416:38
*** cdearborn_ has joined #openstack-ironic16:39
*** amotoki is now known as amotoki_away16:47
*** mgoddard has quit IRC16:47
*** milan has quit IRC16:48
*** milan has joined #openstack-ironic16:50
*** athomas has quit IRC16:51
openstackgerritMerged openstack/ironic-python-agent master: Updated from global requirements  https://review.openstack.org/47464616:53
*** cdearborn__ has quit IRC16:54
*** lucasagomes is now known as lucas-afk16:56
*** amarao has quit IRC17:12
*** jtomasek has quit IRC17:14
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Refactoring the firewall  https://review.openstack.org/47183117:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: PXE boot filtering drivers  https://review.openstack.org/43960317:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: Introducing a dnsmasq PXE filter driver  https://review.openstack.org/46644817:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector master: TEST check dnsmasq PXE filter in devstack  https://review.openstack.org/46871217:15
jlvillalTheJulia: Would like to get your opinion on my comment: https://review.openstack.org/#/c/408556/52/ironic/db/sqlalchemy/api.py@112817:17
patchbotpatch 408556 - ironic - Add new dbsync command with first online data migr...17:17
* milan calls it a day17:17
milanGood night Ironic! ;)17:17
jlvillalmilan: Ciao!17:18
milano/17:18
*** milan has quit IRC17:18
TheJuliajlvillal: oh yeah, great catch. We need to split and evaluate separately17:18
jlvillalTheJulia: Okay, good. I'm not crazy ;)17:18
*** dprince has quit IRC17:18
TheJuliai could have sworn I saw a utility or library that we already had access to that does version compares from the strings of them17:19
TheJuliasince the db field is a varchar17:19
jlvillalTheJulia: Oh we do. In our version stuff.17:20
TheJulia\o/17:21
* TheJulia goes back to the fun of travel booking17:21
jlvillalI'm unsure how we store the value in the database though. string or decimal.17:21
jlvillalI'm guessing 'Decimal' since the code is using the Decimal class.17:21
*** sasha2 has quit IRC17:21
jlvillalOh it is a string.17:23
*** sambetts is now known as sambetts|afk17:23
*** sasha2 has joined #openstack-ironic17:27
*** mvpnitesh has joined #openstack-ironic17:30
*** jcoufal_ has joined #openstack-ironic17:34
*** jcoufal has quit IRC17:36
*** ijw has quit IRC17:36
*** ijw has joined #openstack-ironic17:36
*** jcoufal has joined #openstack-ironic17:37
*** jcoufal_ has quit IRC17:39
*** ijw has quit IRC17:41
*** jcoufal_ has joined #openstack-ironic17:49
*** jcoufal has quit IRC17:51
*** root has joined #openstack-ironic17:52
*** root is now known as Guest8533217:52
*** mvpnitesh has quit IRC17:52
*** bfournie has quit IRC17:53
*** trown|lunch is now known as trown17:55
*** root4 has quit IRC17:55
*** Guest85332 has quit IRC17:56
*** skazi_ has quit IRC18:00
*** dprince has joined #openstack-ironic18:01
TheJuliajlvillal: adjusted my vote accordingly and added a comment raising the concern over named releases as strings.18:03
*** pcaruana has joined #openstack-ironic18:04
*** jcoufal has joined #openstack-ironic18:04
*** ralonsoh has quit IRC18:05
*** jcoufal_ has quit IRC18:06
*** fragatin_ has quit IRC18:11
*** [2]rpioso has joined #openstack-ironic18:12
*** rpioso has quit IRC18:13
*** [2]rpioso is now known as rpioso18:13
ricardoas:q18:14
ricardoasoops18:14
*** ChubYann has joined #openstack-ironic18:15
TheJulia:)18:17
*** mrtenio has joined #openstack-ironic18:20
*** jaganathan has joined #openstack-ironic18:22
*** cdearborn_ has quit IRC18:22
jlvillalTheJulia, Thanks18:25
*** root2 has joined #openstack-ironic18:32
*** pcaruana has quit IRC18:41
*** persia has quit IRC18:47
*** mkrai has quit IRC18:59
*** retr0h has quit IRC18:59
TheJuliajlvillal: any chance you could look at https://review.openstack.org/#/c/413324 today?19:03
patchbotpatch 413324 - ironic - iPXE template support for iSCSI19:03
jlvillalTheJulia, sure. I think I can do that today.19:04
*** yusef has joined #openstack-ironic19:05
TheJuliajlvillal: much appreciated :)19:05
jlvillalTheJulia, question on: https://review.openstack.org/#/c/413324/35/ironic/drivers/modules/pxe.py@26619:09
patchbotpatch 413324 - ironic - iPXE template support for iSCSI19:09
jlvillalIf those are 'str' why do we need to call the __return_item_or... function?19:09
*** jtomasek has joined #openstack-ironic19:10
jlvillalI think I'm not understanding something.19:10
*** jaganathan has quit IRC19:12
* TheJulia tries to remember the case19:12
*** litao__ has quit IRC19:12
jlvillalTheJulia, I guess I'm not sure how a 'str' becomes a 'list'19:13
TheJuliawell, it can be a list in the db, and even with-in the list, elements need to be torn apart19:13
jlvillalTheJulia, But str(some_object) becomes a str object.19:14
TheJuliayeah, and it needs to be a string to be written out because we assemble parts out of the whole19:15
*** jtomasek has quit IRC19:16
TheJuliaI'm not sure the str(object) stuff is there because there is the get, that is unless a value is set to None (because I think it is a list of dictionaries, and may be totally wrong), at which point I suspect str(None) -> '' instead of get('key') returning None19:17
* TheJulia hopes that makes sense19:17
jlvillalTheJulia, I think I'm still confused19:18
TheJuliawell, my theory is wrong19:19
TheJuliahttp://paste.openstack.org/show/Phr8q4rWto6UV088xQus/19:20
jlvillalTheJulia, Basically I don't think we need that __return_item_or_first_if_list() function. But I could be wrong.19:20
TheJuliajlvillal: https://review.openstack.org/#/c/413324/23/ironic/drivers/modules/pxe.py19:21
patchbotpatch 413324 - ironic - iPXE template support for iSCSI19:21
TheJuliajlvillal: structure wise, we do need it19:22
*** e0ne has joined #openstack-ironic19:22
jlvillal>>> type(a['val'])19:22
jlvillal<type 'NoneType'>19:22
jlvillal>>> type(str(a['val']))19:22
jlvillal<type 'str'>19:22
TheJuliasince it can be a single item, or a list of items if memory serves19:22
jlvillalTheJulia, How. The code is passing str(something) to the function. Or am I not seeing something?19:23
jlvillal>>> str([1, 2])19:23
jlvillal'[1, 2]'19:23
TheJuliaoh!19:23
TheJulianow I see it19:23
jlvillal>>> isinstance(str([1, 2]), list)19:23
jlvillalFalse19:23
TheJuliawhisky tango19:23
* TheJulia is now confused why str at all19:24
jlvillalMe too. Something seems wrong, one way or the other. I don't know what it is exactly though.19:24
TheJuliawhere is michael to explain it to us :)19:24
TheJuliaI don't quite get why str() was introduced sadly19:28
*** sasha2 has quit IRC19:28
*** mrtenio has quit IRC19:35
*** sasha2 has joined #openstack-ironic19:36
*** rpioso has quit IRC19:40
*** rpioso has joined #openstack-ironic19:43
*** tesseract has quit IRC19:53
*** e0ne has quit IRC19:55
*** jcoufal has quit IRC19:58
*** zzzeek has quit IRC20:07
openstackgerritYusef Shaban proposed openstack/sushy master: Implemented Redfish Sessions  https://review.openstack.org/47194220:11
*** e0ne has joined #openstack-ironic20:18
openstackgerritYusef Shaban proposed openstack/sushy master: Implement Redfish Sessions  https://review.openstack.org/47194220:22
*** jtomasek has joined #openstack-ironic20:22
*** zzzeek has joined #openstack-ironic20:31
*** [1]rpioso has joined #openstack-ironic20:39
*** vinsh has joined #openstack-ironic20:40
*** rpioso has quit IRC20:41
*** bfournie has joined #openstack-ironic20:53
anupnHi Folks, do we need to pass any info other than ipmi_address and driver while creating node? I created a node with ipmi_address passed in the driver info and trying to put that node into manage state from enroll but getting "Failed to get the power state of the node"20:53
anupnand my node is put back to enroll state as it failed to verify20:54
*** bfournie has quit IRC20:54
anupnthis is how my node looks like http://paste.openstack.org/show/612894/20:55
*** root2 has quit IRC20:57
*** root2 has joined #openstack-ironic21:02
anupnjlvillal, TheJulia ^21:02
*** ijw has joined #openstack-ironic21:05
*** dprince has quit IRC21:08
*** Dinesh_Bhor has quit IRC21:16
*** Dinesh_Bhor has joined #openstack-ironic21:17
*** paramite has quit IRC21:18
*** e0ne has quit IRC21:24
TheJuliaanupn: you need ipmi_username and ipmi_password21:26
anupnHi TheJulia: i gave username21:29
anupnand I see ipmi_password is optional21:29
rama_y_anupn: I think so too, ipmi_username and ipmi_password are required21:29
anupnwith agent_ipmitool driver and it will have the password as ''21:29
TheJuliadepends on the hardware, if the hardware requires it configuration wise, then it is required21:29
anupnTheJulia, rama_y_ I have devstack21:30
TheJuliaanupn: "password" is the vm password21:30
TheJuliawith virtualbmc21:30
*** chlong has quit IRC21:30
anupnany idea what is the default virtualbmc password that devstack use?21:31
*** jtomasek has quit IRC21:31
* anupn searching in scripts21:31
*** jkilpatr has quit IRC21:31
*** vinsh has quit IRC21:32
*** vinsh has joined #openstack-ironic21:32
*** mkrai has joined #openstack-ironic21:33
anupnprobably just password https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L158421:34
*** fragatina has joined #openstack-ironic21:35
*** vinsh_ has joined #openstack-ironic21:43
*** vinsh has quit IRC21:46
*** ijw has quit IRC21:49
*** ijw has joined #openstack-ironic21:54
*** retr0h has joined #openstack-ironic21:57
*** baoli has quit IRC22:06
*** fragatina has quit IRC22:10
*** ijw has quit IRC22:23
*** SerenaFeng has joined #openstack-ironic22:25
*** SerenaFeng has quit IRC22:27
*** sasha2 has quit IRC22:31
*** [1]rpioso has quit IRC22:41
*** vinsh_ has quit IRC22:43
*** vinsh has joined #openstack-ironic22:44
*** trown|brb has joined #openstack-ironic23:20
*** mcarden has joined #openstack-ironic23:21
*** logan_ has joined #openstack-ironic23:25
*** ddurst has joined #openstack-ironic23:25
*** mmedvede_ has joined #openstack-ironic23:25
*** hoonetorg has quit IRC23:26
*** hamzy has quit IRC23:26
*** aarefiev_afk has quit IRC23:26
*** mcarden_ has quit IRC23:26
*** quakers has quit IRC23:26
*** mmedvede has quit IRC23:26
*** logan- has quit IRC23:26
*** weshay has quit IRC23:26
*** trown has quit IRC23:26
*** Daviey has quit IRC23:26
*** mnaser has quit IRC23:26
*** bradjones has quit IRC23:26
*** mmedvede_ is now known as mmedvede23:26
*** raginbajin has quit IRC23:27
*** aarefiev_afk has joined #openstack-ironic23:27
*** hamzy has joined #openstack-ironic23:27
*** hoonetorg has joined #openstack-ironic23:27
*** mnaser has joined #openstack-ironic23:28
*** Daviey has joined #openstack-ironic23:28
*** logan_ is now known as logan-23:29
*** weshay has joined #openstack-ironic23:31
*** bradjones has joined #openstack-ironic23:31
*** weshay has quit IRC23:31
*** weshay has joined #openstack-ironic23:31
*** bradjones has quit IRC23:32
*** bradjones has joined #openstack-ironic23:32
*** raginbajin has joined #openstack-ironic23:34
*** vinsh has quit IRC23:38
*** vinsh has joined #openstack-ironic23:39
*** makowals_ has joined #openstack-ironic23:40
*** makowals has quit IRC23:41
*** vinsh has quit IRC23:43

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