Monday, 2017-06-19

*** MattMan has quit IRC00:04
*** MattMan has joined #openstack-ironic00:04
*** dyasny has quit IRC00:21
*** ijw has joined #openstack-ironic00:33
*** ijw_ has joined #openstack-ironic00:49
*** ijw has quit IRC00:49
*** ijw_ has quit IRC00:52
*** amotoki_away is now known as amotoki00:54
*** tovin07 has joined #openstack-ironic00:56
*** shuyingya has joined #openstack-ironic00:57
*** alex_xu has quit IRC01:01
*** alex_xu has joined #openstack-ironic01:01
*** tovin07 has quit IRC01:27
*** SerenaFeng has joined #openstack-ironic01:29
*** litao__ has joined #openstack-ironic01:38
*** Nakato has left #openstack-ironic01:46
*** tuanluong has joined #openstack-ironic02:00
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521002:22
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521002:23
*** hw_wutianwei has joined #openstack-ironic02:32
*** jerrygb has joined #openstack-ironic02:33
*** jerrygb has quit IRC02:35
*** jaganathan has joined #openstack-ironic02:45
*** jerrygb has joined #openstack-ironic02:46
openstackgerritHironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation  https://review.openstack.org/21458603:02
*** SerenaFeng has quit IRC03:07
*** Nisha_Agarwal has joined #openstack-ironic03:17
*** kiseok7 has quit IRC03:20
*** udesale has joined #openstack-ironic03:31
*** jerrygb has quit IRC03:33
*** Nisha_ has joined #openstack-ironic03:35
*** Nisha_Agarwal has quit IRC03:35
*** Nisha_Agarwal has joined #openstack-ironic03:44
*** Nisha_ has quit IRC03:44
*** jerrygb has joined #openstack-ironic03:54
Nisha_Agarwalmorning Ironic!!!03:54
*** jerrygb has quit IRC03:59
*** jerrygb has joined #openstack-ironic04:03
*** rama_y has joined #openstack-ironic04:42
*** jerrygb has quit IRC04:44
*** adisky__ has joined #openstack-ironic04:53
*** jaganathan has quit IRC04:56
openstackgerritHironori Shiina proposed openstack/ironic master: Enable cinder storage interface for generic hardware  https://review.openstack.org/46390804:59
*** tuanluong has quit IRC04:59
openstackgerritHironori Shiina proposed openstack/ironic master: Add storage interface field to node-related notifications  https://review.openstack.org/46397205:00
openstackgerritHironori Shiina proposed openstack/ironic master: DevStack: Add configuration for boot-from-volume  https://review.openstack.org/46633305:00
*** mvpnitesh has joined #openstack-ironic05:02
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'get_pending_boot_mode' in ILO redfish  https://review.openstack.org/46798205:04
*** jerrygb has joined #openstack-ironic05:06
openstackgerritparesh sao proposed openstack/proliantutils master: Adding 'get_current_boot_mode' method using redfish  https://review.openstack.org/46669705:12
*** pcaruana has joined #openstack-ironic05:14
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'set_pending_boot_mode' in ILO redfish  https://review.openstack.org/47264205:15
openstackgerritHironori Shiina proposed openstack/ironic master: iRMC: Support volume boot for iRMC virtual media boot interface  https://review.openstack.org/46791105:16
rama_yHello, hoping to get some reviews on this patch: https://review.openstack.org/#/c/358142/05:21
patchbotpatch 358142 - ironic - Enable OSProfiler support in Ironic05:21
*** jerrygb has quit IRC05:21
*** nmathew has joined #openstack-ironic05:22
*** jerrygb has joined #openstack-ironic05:32
*** e0ne has joined #openstack-ironic05:33
*** ChubYann has quit IRC05:39
*** dsneddon has joined #openstack-ironic05:43
*** jtomasek has joined #openstack-ironic05:48
*** e0ne has quit IRC05:48
*** tuanluong has joined #openstack-ironic06:02
*** e0ne has joined #openstack-ironic06:02
*** tuanluong has quit IRC06:02
*** mjura has joined #openstack-ironic06:02
*** dsneddon has quit IRC06:06
*** e0ne has quit IRC06:06
*** mjura has quit IRC06:07
*** jtomasek has quit IRC06:12
*** dsneddon has joined #openstack-ironic06:13
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521006:14
*** athomas has joined #openstack-ironic06:17
*** rcernin has joined #openstack-ironic06:19
*** makowals has quit IRC06:19
*** makowals has joined #openstack-ironic06:19
*** mjura has joined #openstack-ironic06:20
*** samodid has joined #openstack-ironic06:25
*** jerrygb has quit IRC06:33
*** jtomasek has joined #openstack-ironic06:49
*** Nisha_Agarwal has quit IRC06:51
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521006:54
*** jaganathan has joined #openstack-ironic06:54
*** silvrax has joined #openstack-ironic06:55
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/newton: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863106:59
silvraxI'm looking into ironics multi-tenant networking support. What puzzles me is what exactly does Ironic require from the Neutron mechanism drivers? How do I find out if my switch vendors ML2 drivers will work or not?06:59
openstackgerritPavlo Shchelokovskyy proposed openstack/ironic stable/newton: [devstack] Use global requirements for virtualbmc  https://review.openstack.org/44863107:02
*** tesseract has joined #openstack-ironic07:05
*** amotoki is now known as amotoki_away07:05
openstackgerritAparna proposed openstack/proliantutils master: Support to perform erase on all the disks(volumes and drives)  https://review.openstack.org/43645007:09
*** aparnav has joined #openstack-ironic07:09
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'get_persistent_boot_device' in ILO redfish  https://review.openstack.org/47526807:14
*** jaosorior has joined #openstack-ironic07:26
*** e0ne has joined #openstack-ironic07:26
*** Dinesh_Bhor has joined #openstack-ironic07:31
*** jerrygb has joined #openstack-ironic07:33
*** rcernin has quit IRC07:34
*** rcernin has joined #openstack-ironic07:37
*** rcernin has quit IRC07:37
*** rcernin has joined #openstack-ironic07:37
pas-hamorning ironic :)07:37
*** jerrygb has quit IRC07:38
*** paramite has joined #openstack-ironic07:41
*** openstackgerrit has quit IRC07:48
*** ccamacho has joined #openstack-ironic07:50
*** tuanluong has joined #openstack-ironic07:53
*** aarefiev_afk is now known as aarefiev07:54
aarefievmorning all!07:54
*** David_Orange has joined #openstack-ironic07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-ironic08:01
*** SerenaFeng has joined #openstack-ironic08:01
*** jerrygb has joined #openstack-ironic08:02
*** openstackgerrit has joined #openstack-ironic08:02
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Test OPNFV CI verify job. Please ignore.  https://review.openstack.org/36760808:02
*** jerrygb has quit IRC08:06
*** mjura has quit IRC08:06
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'get_persistent_boot_device' in ILO redfish  https://review.openstack.org/47526808:12
*** dtantsur|afk is now known as dtantsur08:16
dtantsurmorning ironic08:16
*** lucas-afk is now known as lucasagomes08:16
lucasagomesmorning dtantsur, all08:17
dtantsuro/08:17
*** mvpnitesh has quit IRC08:18
*** racedo has joined #openstack-ironic08:21
*** mjura has joined #openstack-ironic08:22
*** SerenaFeng has quit IRC08:26
thajMorning all08:26
*** ralonsoh has joined #openstack-ironic08:28
*** amotoki_away is now known as amotoki08:31
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Use declarative reader/writer separation  https://review.openstack.org/46376808:35
*** mgoddard has joined #openstack-ironic08:36
vdrokgood morning ironic, pas-ha aarefiev dtantsur lucasagomes08:38
lucasagomeso/08:38
aarefievo/08:38
*** SerenaFeng has joined #openstack-ironic08:44
*** amotoki is now known as amotoki_away08:44
*** e0ne has quit IRC08:47
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'update_persistent_boot' in ILO redfish  https://review.openstack.org/47529308:50
openstackgerritD G Lee proposed openstack/ironic master: ironic-conductor can't report heartbeat to database  https://review.openstack.org/47521008:51
openstackgerritparesh sao proposed openstack/proliantutils master: Adds 'set_one_time_boot' in ILO redfish  https://review.openstack.org/47529508:53
*** derekh has joined #openstack-ironic09:00
pas-hamorning dtantsur lucasagomes aarefiev thaj vdrok09:03
pas-hadtantsur: have a question about interface stability / deprecations and our network interface code09:04
pas-hawe'd need to pass the whole task context to the common.neutron.get_client() instead of just token.09:04
pas-habut that will require changes to the signatures of couple of public methods of network interfaces and dhcp providers09:05
pas-hasupporting both 'token' and 'context09:05
pas-ha.. would be quite ugly09:05
pas-habut probably duable. Any thoughts/ideas how to proceed?09:06
pas-hathere already was a patch from sdague trying to add global_request_id to our neutron calls, and it faced the same problem and needed some hacks around... https://review.openstack.org/#/c/472666/09:09
patchbotpatch 472666 - ironic - WIP: pass global_request_id to cinder/neutron09:09
openstackgerritKyrylo Romanenko proposed openstack/python-ironicclient master: Tests for OSC baremetal node create command with options  https://review.openstack.org/38235209:09
dtantsurpas-ha: I wonder if we can/should support passing an implicit context somehow09:10
dtantsur(similar to e.g. flask thread context stuff)09:10
pas-hawell, there's such thing as context.get_current()09:11
pas-hanot sure how safe it is though.. and explicit seems better to me09:11
thajHi Dmitry09:12
pas-hahttps://review.openstack.org/#/c/472666/3/ironic/common/neutron.py@4109:12
patchbotpatch 472666 - ironic - WIP: pass global_request_id to cinder/neutron09:12
openstackgerritMark Goddard proposed openstack/ironic master: Expose ports' physical network attribute in API  https://review.openstack.org/46993309:12
dtantsurpas-ha: ditto. well, we can do something similar to what was done when introducing "timeout" to power actions: inspect the method signature looking for a parameter name09:13
dtantsurugly, but will allow handling the transition period09:13
dtantsurhi thaj09:13
thajI got a question09:13
pas-haok, will try to compose something09:13
thaji have 2 nodes. Those are  baremetal nodes(for e.g, node14 and node13). In node14, devstack has been installed. Now i am able to access node13 from node14.09:15
thajNow, i would like deploy an OS in node13, using the controller node14. Can give an idea, how to proceed?09:16
*** e0ne has joined #openstack-ironic09:18
dtantsurvsaienk0: hi! I see someone selling Juniper SRX-100. do you know if it's of any use for home testing?09:18
*** jerrygb has joined #openstack-ironic09:19
dtantsurthaj: please check the remaining steps of https://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack09:20
vsaienk0dtantsur: it supports SSH so by creating appropriate connector in netmiko, or by using networking ansible module you can add support to n-g-s09:21
*** jerrygb has quit IRC09:23
*** leifmadsen has quit IRC09:25
*** huats has quit IRC09:25
pas-hadtantsur: it seems this switch runs JunOS, so netmiko probably supports it https://github.com/ktbyers/netmiko/blob/v1.2.5/netmiko/ssh_dispatcher.py#L5109:25
*** huats has joined #openstack-ironic09:26
pas-hathus as vsaienk0 said, we'd need a device plugin in n-g-s to support it and it should work09:26
*** leifmadsen has joined #openstack-ironic09:26
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before confuring the DB  https://review.openstack.org/47530609:30
dtantsurfun, thanks! /me thinks of just getting it, but he does not have machines to connect to it :D09:33
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before confuring the DB  https://review.openstack.org/47530609:36
vsaienk0dtantsur: vdrok had idea to write plugin to n-g-s to manage openwrt based routers, and I have cisco 2960s09:40
dtantsurlol, awesome09:41
*** tiendc has joined #openstack-ironic09:42
*** sambetts|afk is now known as sambetts09:42
*** milan has joined #openstack-ironic09:43
milanmorning Ironic! :)09:43
dtantsurhey milan09:44
milanhey dtantsur! :)09:44
* milan checks for decorator induced -2 s ;)09:45
openstackgerritDao Cong Tien proposed openstack/ironic master: Tempest scenario test for boot-from-volume  https://review.openstack.org/47274009:45
*** b00tcat has joined #openstack-ironic09:50
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before confuring the DB  https://review.openstack.org/47530609:51
b00tcatHi all, I want to enable web console for my baremetal nodes, and am following https://docs.openstack.org/developer/ironic/deploy/console.html#node-web-console09:51
b00tcatwhat are the parameters (other than node--uid) here supposed to be? ironic node-update <node-uuid> add driver_info/<terminal_port>=<customized_port>09:52
b00tcatI get that I need to execute: ironic node-update <node-uuid> add driver_info/ipmi_terminal_port=<port>09:52
b00tcatbut not sure what this port is09:52
b00tcatis it the SSH port? one device file?09:52
milanb00tcat, imo it's the port you want it (the web console) to listen on, on the conductor10:01
*** lhinds has quit IRC10:02
*** rnoriega has quit IRC10:02
*** rnoriega has joined #openstack-ironic10:07
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before confuring the DB  https://review.openstack.org/47530610:08
*** lhinds has joined #openstack-ironic10:08
*** mrchao has joined #openstack-ironic10:13
*** milan is now known as milan|lunch10:17
milan|lunch-> offline for a bit10:17
*** milan|lunch has quit IRC10:18
*** persia has joined #openstack-ironic10:27
*** amarao has joined #openstack-ironic10:28
*** hwoarang has quit IRC10:28
*** hwoarang has joined #openstack-ironic10:29
*** sai has quit IRC10:39
*** Guest48624 has quit IRC10:39
*** lucasagomes has quit IRC10:39
*** sai has joined #openstack-ironic10:40
*** rook has joined #openstack-ironic10:41
*** rook is now known as Guest5203410:42
*** root2 has joined #openstack-ironic10:42
*** root4 has quit IRC10:45
*** nmathew has quit IRC10:54
*** jkilpatr has joined #openstack-ironic10:57
*** lucasagomes has joined #openstack-ironic10:58
*** udesale has quit IRC11:00
*** lucasagomes is now known as lucas-hungry11:04
*** lucasagomes has joined #openstack-ironic11:14
*** SerenaFeng has quit IRC11:16
*** jerrygb has joined #openstack-ironic11:20
*** ccamacho is now known as ccamacho|lunch11:22
*** jerrygb has quit IRC11:25
openstackgerritparesh sao proposed openstack/ironic master: Added a condition for ILO for pxe hardware type  https://review.openstack.org/47534211:32
*** bigbigbang has joined #openstack-ironic11:37
*** milan has joined #openstack-ironic11:38
*** links has joined #openstack-ironic11:40
*** links has quit IRC11:41
openstackgerritDao Cong Tien proposed openstack/ironic master: Add functional API tests for volume connector and volume target  https://review.openstack.org/47371711:43
thajHi Dmitry11:43
openstackgerritAnton Arefiev proposed openstack/ironic-inspector master: Use declarative reader/writer separation  https://review.openstack.org/46376811:44
thajDmitry: When i run the command "openstack server list --long" , i am not able to see the value in  "Networks" column. I am just following the Developer Quick start link.11:46
*** Nisha_Agarwal has joined #openstack-ironic11:46
*** trown|brb is now known as trown12:02
*** dprince has joined #openstack-ironic12:03
*** litao__ has quit IRC12:08
*** jtomasek_ has joined #openstack-ironic12:09
dtantsurthaj: is your server built in the end? What's the status?12:13
*** lucas-hungry has quit IRC12:16
thajpower state is :off12:16
sambettswhat the nova status?12:16
sambettserror12:17
sambetts?12:17
* dtantsur has to disappear for an hour or two12:17
thajnova state : NOSTATE12:17
thajAfter the following command,12:18
thajopenstack server create --flavor baremetal --nic net-id=$net_id --image cirros-0.3.4-x86_64-disk --key-name test testing12:18
thajNova's status becomes "ERROR"12:18
sambettsif you do nova show <nova uuid> for that node what error do you see?12:19
openstackgerritAparna proposed openstack/ironic master: Add API and conductor methods for [un]rescue  https://review.openstack.org/35083112:20
thajStatus says: ERROR. Fault says " No valid host was found. Code:500"12:21
sambettsthaj: so that a schedulding error https://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#nova-returns-no-valid-host-was-found-error12:22
thajThanks for guiding me. I will go through the link.12:24
*** Nisha_Agarwal has quit IRC12:24
*** jtomasek_ has quit IRC12:26
*** dtantsur is now known as dtantsur|brb12:26
*** jaypipes has joined #openstack-ironic12:28
*** jerrygb has joined #openstack-ironic12:29
*** jerrygb has quit IRC12:33
*** bfournie has left #openstack-ironic12:37
*** tuanluong has quit IRC12:42
*** jerrygb has joined #openstack-ironic12:47
*** bfournie has joined #openstack-ironic12:57
*** sasha2 has joined #openstack-ironic12:58
*** tiendc has quit IRC13:00
*** jcoufal has joined #openstack-ironic13:02
pas-haTheJulia: what if I need to rename an option in [cinder] - do we consider it already exposed to users and need a deprecation period/handling, or just remove/rename right away?13:04
*** ccamacho|lunch is now known as ccamacho13:07
TheJuliapas-ha: That is a really good question. What is the option?13:10
*** jcoufal has quit IRC13:11
pas-hawith my current work on using keystoneauth adapters in ironic for clients, I'd need to remove cinder.url and use 'endpoint_override' exported by keystoneauth.Adapter13:12
TheJuliapas-ha: I _think_ we might be in a relatively safe position to change it since while it is exposed in config, although we could just put it in as deprecated to begin with13:12
TheJuliadeprecated configuration option then13:13
pas-haok13:13
TheJuliait makes sense, but it will help people put the two together13:13
pas-haagree13:13
*** jcoufal has joined #openstack-ironic13:18
openstackgerritAparna proposed openstack/ironic master: Add rescuewait timeout periodic task  https://review.openstack.org/35315613:19
*** jcoufal has quit IRC13:24
*** aparnav has quit IRC13:24
*** makowals has quit IRC13:26
*** jcoufal has joined #openstack-ironic13:28
*** Guest52034 is now known as rook13:30
*** Goneri has joined #openstack-ironic13:30
*** makowals has joined #openstack-ironic13:32
*** baoli has joined #openstack-ironic13:36
*** rloo has joined #openstack-ironic13:37
*** gcb has joined #openstack-ironic13:38
*** dtantsur|brb is now known as dtantsur13:57
dtantsurmorning TheJulia13:57
rloohi and good Monday morning dtantsur, TheJulia13:58
dtantsurmorning rloo13:58
*** dyasny has joined #openstack-ironic13:59
openstackgerritMerged openstack/ironic-lib master: Adjust test with option 'backend'  https://review.openstack.org/47406613:59
rloodtantsur: btw, i was going to look at our previous meeting minutes -- did we talk about resurrecting weekly network meetings?14:00
dtantsurrloo: I remember talking about that. I don't remember if anybody volunteered to actually do it. IIRC the time slot does not work for me.14:00
rloosambetts: ^^ do you recall? (I seem to think sam had volunteered...)14:01
*** shuyingya has quit IRC14:02
*** baoli has quit IRC14:03
*** crushil has joined #openstack-ironic14:03
*** shuyingya has joined #openstack-ironic14:03
*** dyasny has quit IRC14:04
sambettsrloo, dtantsur: I created an etherpad to track the network changes, I put the link in the etherpad, I havn't got around to pushing a patch to update the meeting information for the ironic networking meeting, Sukhdev is still down as the chair14:04
sambettss/etherpad/whiteboard/14:04
rloosambetts: thx. so you are proposing the same day/time as before? Monday before our meeting if i recall?14:05
sambettsyeah, I was just going to take over the existing slot if its not being used for anything else14:05
rloodtantsur: ^^ that is the time you cannot attend?14:05
*** baoli has joined #openstack-ironic14:06
dtantsurI don't quite remember. but Monday before our meeting is in conflict for me every other week14:06
rloodtantsur: if you want to attend this meeting, then i think we should find a day/time that works for you and sambetts14:06
*** shuyingya has quit IRC14:08
dtantsurI'd like to, but it's not critical to have me there14:08
rloodtantsur: i think it'd be good if you were there. who is critical? sambetts, vsaienk0? mgoddard?14:11
dtantsuryep, vdrok maybe. dsneddon from our team my want to stop by from time to time.. or bfournie14:11
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539714:15
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539814:15
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539714:23
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539814:23
*** bfournie has quit IRC14:24
*** rama_y has quit IRC14:24
*** bfournie has joined #openstack-ironic14:25
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Log warning when API version is not specified for the OSC plugin  https://review.openstack.org/44215314:29
*** bfournie has quit IRC14:29
*** milan has quit IRC14:33
*** samodid has quit IRC14:33
*** root2 has quit IRC14:33
*** dyasny has joined #openstack-ironic14:37
TheJuliaGood morning dtantsur, rloo, sambetts14:37
rlooGood morning TheJulia! Travelling this week?14:37
rlooTheJulia: or is the exception not travelling? :-)14:38
TheJuliarloo: free from travel, as far as I'm aware, until early July \o/14:40
rlooTheJulia: :D14:40
*** mjturek has joined #openstack-ironic14:40
*** rpioso has joined #openstack-ironic14:42
TheJuliaI did get pulled into a last minute thing about two hours away last Wednesday... *shrug*14:42
rlooTheJulia: you are amazing; I don't know how you can do so much travelling!14:43
TheJuliaStill trying to figure that out myself14:44
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539714:46
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539814:46
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539714:47
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539814:47
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Log warning when API version is not specified for the OSC plugin  https://review.openstack.org/44215314:47
TheJuliamjturek: o/ have you had a minute to look at the ipxe template revision?14:48
mjturekmorning TheJulia! Nope, I'll take a look right now14:49
mjturekjust getting back from travel myself :)14:49
openstackgerritSam Betts proposed openstack/ironic-python-agent master: Include IPA Version during heartbeat  https://review.openstack.org/36483414:52
dtantsurmorning mjturek14:52
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before configuring the DB  https://review.openstack.org/47530614:52
NobodyCamGood Morning Ironic folk!14:55
mjturekmorning dtantsur!14:56
dtantsurmorning NobodyCam14:56
NobodyCamhey hey Happy Monday dtantsur :)14:56
*** makowals has quit IRC14:57
*** milan has joined #openstack-ironic14:59
*** makowals has joined #openstack-ironic15:00
mjturekTheJulia: ah, I see the problem now.. I'll update soon15:00
mjtureksorry about that15:00
TheJuliamjturek: no worries!15:00
vdrokgood morning TheJulia sambetts rloo and mjturek !15:00
vdrokand good morning NobodyCam :)15:00
NobodyCamGood Morning vdrok :)15:01
*** Nisha_Agarwal has joined #openstack-ironic15:01
*** bfournie has joined #openstack-ironic15:01
mjturekmorning vdrok!15:02
*** mrchao has quit IRC15:02
rloomorning vdrok, NobodyCam, mjturek!15:03
*** chlong has joined #openstack-ironic15:03
NobodyCamGood Morning rloo.. :)15:03
rloovdrok: i see your comments, thx, haven't had a chance to look yet.15:03
*** mjura has quit IRC15:04
vdrokrloo: np :) these are more of clarification questions. also it seems like today I have a couple of free hours, so if you have not started with ovo base refactor I can do it15:05
sambettsdo we have a document for how to deal with changes in the driver API so that we don't break out of tree drivers?? I'm working on my change to add the IPA version to heartbeat and it either requires a change to the defination of the heartbeat function in the deploy drivers or changing where the node lock is taken in the conductor15:06
*** jaganathan has quit IRC15:07
vdroksambetts: don't think so. ideally we just should not break it :) is it a new required parameter?15:08
*** skazi_ has joined #openstack-ironic15:08
sambettsvdrok: yeah, its the agents version information, e.g. heartbeat(self, task, agent_url, agent_version)15:10
*** skazi_ has quit IRC15:10
*** skazi_ has joined #openstack-ironic15:11
sambettsvdrok: the heartbeat etc function for interacting with IPA are normally inherited from the agent base class, so its not too bad unless someone is completely rewriting that logic in their drvier15:11
*** David_Orange has quit IRC15:13
rloovdrok: yes, if you'd like to do the ovo base refactor, that'd be great!15:13
vdroksambetts: or if they have just copied the definition without the agent_version. can we have a default for it? maybe assume that if no agent_version is specified, consider the earliest version we now support?15:14
vdrokrloo: ok, starting now :)15:14
rloosambetts: what vdrok said. no documentation, and we need to support no version (i guess we can view it as deprecating no version)15:15
sambettsvdrok: having a default isn't a problem, the problem is the actual function definition itself, e.g. the condcutor will try to pass agent_version into their driver, but their defintion is heartbeat(task, agnet_url)15:15
rloovdrok: we were discussing earlier (and in one of our weekly meetings) about resurrecting a network meeting. would you be able to attend?15:15
sambettsso it fails with a too many parameters error15:15
rloosambetts: fails with an exception?15:16
sambettsyeah15:16
rloosambetts: catch and retry w/o the extra parameter15:16
vdroksambetts: ah, we have worked around things like this with catching exceptions rloo ++15:16
sambettseww, but yeah that is one solution :/15:16
rloosambetts: that's how we've done it for other cases (where we've added a new param)15:16
*** fragatina has joined #openstack-ironic15:16
rloosambetts: should be temporary, we should be able to delete that code in some future release15:16
*** rama_y has joined #openstack-ironic15:17
Nisha_Agarwaldtantsur, hi15:25
dtantsurNisha_Agarwal: hi15:25
Nisha_Agarwalregarding comments on https://review.openstack.org/#/c/473913/8/devstack/lib/ironic15:25
patchbotpatch 473913 - ironic - py3.5:Workaround fix for forcing virtualbmc instal...15:25
Nisha_Agarwalwe need to use clean name, else that routine fails15:25
Nisha_Agarwali did it without clean name initially15:25
*** mjturek has quit IRC15:26
Nisha_Agarwaldtantsur, ^^^15:26
Nisha_Agarwaldtantsur, check patch set 715:26
*** e0ne has quit IRC15:26
dtantsurNisha_Agarwal: well, clean_name is incorrect. it builds an incorrect URL, and always gets empty classifiers15:27
dtantsurhave you checked the logs I've pasted?15:27
Nisha_Agarwalsee the logs for patch set 715:27
Nisha_Agarwalhttp://logs.openstack.org/13/473913/7/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/77a22c7/logs/devstacklog.txt.gz#_2017-06-15_17_16_24_10915:27
dtantsurNisha_Agarwal: please check the logs you're pasting :) you had an error in bash on line 661 (missing $()), hence it treated "pyghmi" as a command name15:29
dtantsurI don't see how it's related to using or not using clean_name15:29
dtantsurclassifier=check_python3_support_for_package_remote "pyghmi"15:29
Nisha_Agarwaldtantsur, ohk15:30
*** rcernin has quit IRC15:30
dtantsurthis sets variable "classifier" to value "check_python3_support_for_package_remote", then runs command "pyghmi" with this variable set15:30
Nisha_Agarwalyeah15:30
Nisha_Agarwalok15:30
*** amotoki_away is now known as amotoki15:32
*** pcaruana has quit IRC15:33
*** skazi_ has quit IRC15:34
vdrokrloo: oh, sorry, regarding the meeting, I think I should be able to attend, tho having vsaienk0 would be of greater value15:35
*** baoli has quit IRC15:35
*** mjturek has joined #openstack-ironic15:35
vdrokrloo: is it starting today or it's just planning?15:35
*** baoli has joined #openstack-ironic15:35
*** fragatina has quit IRC15:36
*** mtanino has joined #openstack-ironic15:36
*** fragatina has joined #openstack-ironic15:36
*** mjturek has quit IRC15:39
rloovdrok: no it isn't starting yet, i was just asking where we were at with that meeting and who the 'critical' folks are for it. also, i think dtantsur should be there so think the date/time should change.15:40
*** baoli has quit IRC15:40
vdrokrloo: gotcha. I don't think I have some set times when I'm not able to attend, everything happens spontaneously since recently :) so should be able to attend any day 9-17 utc15:42
rloovdrok: same with vsaienk0? do you know?15:42
vdrokrloo: I'll ask him today, currently he's out15:43
rloothx vdrok!15:44
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: bindep.txt: Install the epel-release repo on CentOS  https://review.openstack.org/47544615:45
rama_yGood morning. this patch is very close, imo; reaching out for a core review: https://review.openstack.org/#/c/358142/15:47
patchbotpatch 358142 - ironic - Enable OSProfiler support in Ironic15:47
*** bfournie1 has joined #openstack-ironic15:52
*** bfournie1 has quit IRC15:52
*** bfournie has quit IRC15:52
*** bfournie has joined #openstack-ironic15:53
*** mjturek has joined #openstack-ironic15:53
*** paramite has quit IRC15:54
*** pc_m has quit IRC15:55
*** baoli has joined #openstack-ironic15:55
openstackgerritNisha Agarwal proposed openstack/ironic master: py3.5:Workaround fix for forcing virtualbmc installation with pip2  https://review.openstack.org/47391315:58
*** baoli has quit IRC16:00
*** pc_m has joined #openstack-ironic16:00
*** baoli has joined #openstack-ironic16:02
*** dprince has quit IRC16:06
openstackgerritSam Betts proposed openstack/ironic master: Receive and store agent version on heartbeat  https://review.openstack.org/36486116:07
*** crushil has quit IRC16:08
*** srart has joined #openstack-ironic16:08
*** crushil has joined #openstack-ironic16:09
*** fragatina has quit IRC16:11
*** pcaruana has joined #openstack-ironic16:11
*** fragatina has joined #openstack-ironic16:12
*** fragatina has quit IRC16:16
anupnHi aarefiev: Are you around? I am trying to enroll node using auto-discovery but kinda not clear about the steps, I created a node and it is in enroll state so do i need to call inspect on other node or put this newly created node in manage->inspect state16:19
TheJuliarama_y: Hi, I've added it to my review queue.  I was kind of surprised by the size, then I saw the fact that there is an SVG included :)16:22
*** ijw has joined #openstack-ironic16:22
rama_yHi TheJulia, thanks.16:23
*** lucasagomes is now known as lucas-afk16:25
*** ijw has quit IRC16:27
*** b00tcat has quit IRC16:27
*** gcb has quit IRC16:28
*** mgoddard has quit IRC16:30
*** dprince has joined #openstack-ironic16:33
*** ChubYann has joined #openstack-ironic16:43
*** trown is now known as trown|lunch16:45
*** aimeeu has left #openstack-ironic16:51
*** ralonsoh has quit IRC16:53
*** stendulker has joined #openstack-ironic16:56
*** chlong has quit IRC16:56
*** AndroUser has joined #openstack-ironic16:57
*** crushil is now known as crushil_16:59
*** tesseract has quit IRC16:59
*** AndroUser is now known as crushil17:00
rlooding ding, time for meeting17:00
*** fellypefca has joined #openstack-ironic17:00
*** derekh has quit IRC17:01
* dtantsur tried to start a meeting in a wrong channel first. sigh..17:01
*** crushil__ has joined #openstack-ironic17:03
*** fellypefca has quit IRC17:03
* TheJulia gives dtantsur coffee17:03
dtantsurTheJulia: coffee is something I tend to avoid at 7pm usually..17:04
*** crushil has quit IRC17:07
*** Nisha_Agarwal has quit IRC17:08
*** crushil__ is now known as crushil17:08
*** Nisha_Agarwal has joined #openstack-ironic17:08
*** Nisha_Agarwal has quit IRC17:09
*** Nisha_Agarwal has joined #openstack-ironic17:09
*** crushil_ has quit IRC17:10
*** chlong has joined #openstack-ironic17:11
*** Nisha_Agarwal has quit IRC17:13
openstackgerritMichael Turek proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332417:13
*** Nisha_ has joined #openstack-ironic17:13
*** Nisha_ has quit IRC17:18
*** rcernin has joined #openstack-ironic17:18
*** Nisha_ has joined #openstack-ironic17:20
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Deprecate DHCP providers  https://review.openstack.org/45612717:21
*** amarao has quit IRC17:22
*** stendulker has quit IRC17:28
*** pc_m has quit IRC17:29
*** stendulker has joined #openstack-ironic17:29
*** jcoufal_ has joined #openstack-ironic17:30
*** jcoufal_ has quit IRC17:30
*** pc_m has joined #openstack-ironic17:31
*** jcoufal_ has joined #openstack-ironic17:31
sambettsdtantsur: got to run again, if there is anything you need me for I'll try to follow up in the morning17:31
*** sambetts is now known as sambetts|afk17:32
*** jcoufal has quit IRC17:32
*** crushil_ has joined #openstack-ironic17:34
*** crushil has quit IRC17:38
*** crushil_ is now known as crushil17:45
*** Nisha_ has quit IRC17:46
*** Nisha_ has joined #openstack-ironic17:46
jlvillalmjturek: What do you think of my comment?  https://review.openstack.org/#/c/413324/35..36/ironic/drivers/modules/pxe.py17:46
patchbotpatch 413324 - ironic - iPXE template support for iSCSI17:46
*** rbudden_ has joined #openstack-ironic17:47
TheJuliajlvillal: makes sense to me17:47
mjturekjlvillal: I was considering it but felt like the helper function shouldn't modify type. I guess that's silly because literally everywhere I use it I convert to str17:47
mjturekwill update17:47
jlvillalTheJulia: mjturek I added one more comment17:47
*** fellypefca has joined #openstack-ironic17:49
mjturekjlvillal: when you say "any of these" do you mean the properties themselves or the vars? Because we pass a default of '' to properties.get in case of None17:49
*** Nisha_ has quit IRC17:49
*** rbudden has quit IRC17:49
*** rbudden_ is now known as rbudden17:49
*** Nisha_ has joined #openstack-ironic17:50
jlvillalmjturek: I guess I was wondering if the properties could be none.17:50
*** crushil has quit IRC17:50
mjturekyeah that I'm not 100% on17:50
jlvillalmjturek: But I do see NOW, that the code uses a default value of '' if unset17:50
mjturekcool, will reply inline as well17:50
jlvillalmjturek: But could it be set to None? What can it be set to?17:50
* jlvillal doesn't know17:50
* mjturek refreshes on what actually sets these properties17:51
mjturekone sec17:51
*** stendulker has quit IRC17:53
*** ppiela has quit IRC17:57
*** ppiela has joined #openstack-ironic17:59
*** fellypefca has quit IRC18:01
*** baoli has quit IRC18:01
*** baoli has joined #openstack-ironic18:02
*** baoli has quit IRC18:02
*** zigo has quit IRC18:05
*** baoli has joined #openstack-ironic18:06
*** ppiela has quit IRC18:06
*** ppiela has joined #openstack-ironic18:07
*** zigo has joined #openstack-ironic18:09
*** trown|lunch is now known as trown18:10
mjturekjlvillal: I'm pretty sure they can be None. I believe Volume Targets and their properties are created by the user so there's no guarantee that lun, portal, or iqn will be defined. TheJulia does this sound right?18:10
mjturekwhich leads to the question, should we be failing if their None18:10
mjturekthey're*18:10
*** BadCub_ has joined #openstack-ironic18:11
*** rajinir_ has joined #openstack-ironic18:12
TheJuliamjturek: That does sound right.  Realistically, failing upon it being None is an edge case (but one we likely ought to address at some point) because the happy path involves code populating the data, at least in the cinder case.18:12
TheJuliaI guess I mean, it would have to be an intentional misconfiguration at that point18:13
*** jaganathan has joined #openstack-ironic18:16
mjturekright makes sense18:18
*** kencjohnston_ has joined #openstack-ironic18:18
*** aignatov2 has joined #openstack-ironic18:18
*** patriciadomin_ has joined #openstack-ironic18:18
*** mtanino has quit IRC18:19
*** dyasny has quit IRC18:19
*** BadCub has quit IRC18:19
*** kencjohnston has quit IRC18:19
*** aignatov has quit IRC18:19
*** patriciadomin has quit IRC18:19
*** d0ugal has quit IRC18:19
*** krtaylor has quit IRC18:19
*** rajinir has quit IRC18:19
*** lennyb has quit IRC18:19
*** fungi has quit IRC18:19
*** BadCub_ is now known as BadCub18:19
*** aignatov2 is now known as aignatov18:19
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: test-bifrost: Start MySQL service before configuring the DB  https://review.openstack.org/47530618:19
dtantsuroh, time to run. see you tomorrow!18:19
*** dtantsur is now known as dtantsur|afk18:19
*** rajinir_ is now known as rajinir18:20
mjtureknight dtantsur|afk18:21
*** dyasny has joined #openstack-ironic18:25
*** fungi has joined #openstack-ironic18:25
*** d0ugal has joined #openstack-ironic18:25
*** krtaylor has joined #openstack-ironic18:25
*** lennyb has joined #openstack-ironic18:27
*** Nisha_ has quit IRC18:27
*** Nisha_ has joined #openstack-ironic18:27
*** Nisha_ has quit IRC18:32
*** Nisha_Agarwal has joined #openstack-ironic18:35
*** crushil has joined #openstack-ironic18:40
*** pcaruana has quit IRC18:46
*** dkehn_ has joined #openstack-ironic18:48
*** Nisha_ has joined #openstack-ironic18:51
*** Nisha_Agarwal has quit IRC18:51
*** dkehn has quit IRC18:51
*** Nisha_ has quit IRC18:51
*** Nisha_ has joined #openstack-ironic18:52
openstackgerritMerged openstack/python-ironicclient master: Improve help text for --local-link-connection  https://review.openstack.org/47473219:03
*** racedo has quit IRC19:09
*** alex_xu has quit IRC19:11
*** alex_xu has joined #openstack-ironic19:11
*** baoli has quit IRC19:14
*** baoli has joined #openstack-ironic19:14
*** jaganathan has quit IRC19:18
*** Nisha_brb has joined #openstack-ironic19:21
*** Nisha_ has quit IRC19:21
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Deprecate DHCP providers  https://review.openstack.org/45612719:28
* jlvillal feels foolish when he fixes pep8 and unit test issues but forgets to commit change before sending it off to Gerrit :(19:29
*** baoli has quit IRC19:29
*** baoli has joined #openstack-ironic19:30
*** baoli has quit IRC19:34
NobodyCamjlvillal: doh... I to have done that!19:38
jlvillalNobodyCam: :)19:38
openstackgerritNisha Agarwal proposed openstack/sushy master: Adds EthernetInterface to the library  https://review.openstack.org/45253219:39
TheJuliamy favorite is think i wrote a fixed file out and updated a revision, but never actually saved the file I was editing19:39
TheJulias/is/is where/19:40
TheJuliai tink19:40
TheJuliathink19:40
TheJuliablah, I give up!19:40
TheJulia$words19:40
*** makowals has quit IRC19:43
jlvillalheh TheJulia :)19:43
*** dkehn_ is now known as dkehn19:50
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: bindep.txt: Install the epel-release repo on CentOS  https://review.openstack.org/47544619:57
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539719:57
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539819:57
*** jdwidari has joined #openstack-ironic19:57
*** makowals has joined #openstack-ironic19:57
*** jtomasek_ has joined #openstack-ironic19:59
*** baoli has joined #openstack-ironic20:01
*** adisky__ has quit IRC20:02
*** mtanino has joined #openstack-ironic20:09
*** jtomasek_ has quit IRC20:10
*** Nisha_brb has quit IRC20:16
*** jcoufal_ has quit IRC20:19
* TheJulia hearts people20:26
*** milan has quit IRC20:28
*** chlong has quit IRC20:41
jlvillalTheJulia: mjturek: Another idea :)  https://review.openstack.org/#/c/413324/36/ironic/drivers/modules/pxe.py@26020:47
patchbotpatch 413324 - ironic - iPXE template support for iSCSI20:47
mjtureklookin'20:48
jlvillalmjturek: I might be missing something. But seems like it looks simpler that way.20:49
mjturekjlvillal: I was literally about to do something like this as there's a get('target_iqn')20:49
mjtureka little further down20:49
jlvillalmjturek: :)20:49
TheJuliaheh, awesome20:49
mjturekyeah I'll do something like this, thanks jlvillal :)20:49
jlvillal:)20:49
*** chlong has joined #openstack-ironic20:55
*** Goneri has quit IRC20:55
*** fragatina has joined #openstack-ironic20:58
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-ui master: Set current boot device on a node  https://review.openstack.org/47392621:01
openstackgerritOpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements  https://review.openstack.org/47554321:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector master: Updated from global requirements  https://review.openstack.org/47554421:03
*** dprince has quit IRC21:10
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: bindep.txt: Install the epel-release repo on CentOS  https://review.openstack.org/47544621:13
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: Add bindep.txt file  https://review.openstack.org/47539721:13
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: scripts: install-deps.sh: Use --no-recommends on zypper  https://review.openstack.org/47539821:13
*** bfournie has quit IRC21:14
*** trown is now known as trown|outtypewww21:16
openstackgerritMichael Turek proposed openstack/ironic master: iPXE template support for iSCSI  https://review.openstack.org/41332421:17
mjturekjlvillal: TheJulia ^21:18
mjturekwas able to get rid of str conversion with this too21:19
mjturekbut heading out, night all!21:19
*** dsneddon has quit IRC21:21
*** sdake has quit IRC21:21
*** jmccrory has quit IRC21:21
*** eglute has quit IRC21:21
*** sambetts|afk has quit IRC21:21
*** comstud has quit IRC21:21
*** mgagne has quit IRC21:21
*** ionutbalutoiu has quit IRC21:21
*** jcook has quit IRC21:21
*** lukl has quit IRC21:21
*** pshige_________ has quit IRC21:21
*** jmccrory_ has joined #openstack-ironic21:21
*** sdake has joined #openstack-ironic21:21
*** eglute_s has joined #openstack-ironic21:21
*** comstud has joined #openstack-ironic21:21
*** sambetts_ has joined #openstack-ironic21:21
*** ionutbalutoiu has joined #openstack-ironic21:21
*** jcook has joined #openstack-ironic21:21
*** mgagne has joined #openstack-ironic21:22
*** pshige_________ has joined #openstack-ironic21:22
*** sdake is now known as Guest872221:22
*** lukl has joined #openstack-ironic21:22
jlvillalmjturek: Thanks. Looks simpler :)21:22
*** jmccrory_ is now known as jmccrory21:23
*** dsneddon has joined #openstack-ironic21:26
*** crushil has quit IRC21:29
*** crushil has joined #openstack-ironic21:30
*** ccamacho has quit IRC21:31
*** fragatina has quit IRC21:33
*** crushil has quit IRC21:35
*** amotoki is now known as amotoki_away21:36
*** dsneddon has quit IRC21:43
*** dsneddon has joined #openstack-ironic21:45
*** jkilpatr has quit IRC21:45
*** chlong has quit IRC21:46
*** mjturek has quit IRC21:57
*** rpioso has quit IRC21:58
*** baoli has quit IRC21:59
*** Goneri has joined #openstack-ironic22:00
*** makowals has quit IRC22:03
*** makowals has joined #openstack-ironic22:05
*** ijw__ has joined #openstack-ironic22:15
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode  https://review.openstack.org/43557322:16
*** jkilpatr has joined #openstack-ironic22:18
openstackgerritRamamani Yeleswarapu proposed openstack/ironic-python-agent master: Collect processor, memory and BIOS output of dmidecode  https://review.openstack.org/43557322:21
*** rcernin has quit IRC22:21
*** jerrygb has quit IRC22:25
*** jkilpatr has quit IRC22:29
*** rbudden has quit IRC22:30
rloojlvillal: i have no idea what the 'right' thing is wrt the suffix stuff, we should probably clarify/clean up one day... (wrt https://review.openstack.org/#/c/475521/)22:31
patchbotpatch 475521 - openstack-infra/project-config - Make the Ironic multinode grenade job voting22:31
jlvillalrloo: My gut feeling is that if it is a voting job, make the suffix not be '-nv'22:31
jlvillalrloo: But unsure on what is "required"22:31
*** baoli has joined #openstack-ironic22:32
*** baoli has quit IRC22:32
rloojlvillal: yeah, i think so too. maybe the other grenade one should be fixed. but then, i looked at other ones and it isn't clear to me; i think we're just not consistent.22:32
jlvillalrloo: Yeah, not consistent22:33
*** Goneri has quit IRC22:34
*** jaypipes has quit IRC23:04

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