Thursday, 2017-01-26

*** sayantani01 has quit IRC00:00
*** sdake has quit IRC00:01
*** srart has quit IRC00:03
*** aNuposic has quit IRC00:08
*** baoli has quit IRC00:08
*** srart has joined #openstack-ironic00:13
*** aNuposic has joined #openstack-ironic00:14
*** vinsh has quit IRC00:18
*** dsneddon has quit IRC00:23
*** Sukhdev has quit IRC00:31
*** bfournie has joined #openstack-ironic00:31
*** Sukhdev has joined #openstack-ironic00:35
*** sayantani01 has joined #openstack-ironic00:43
*** rloo has quit IRC00:43
*** vsaienko has quit IRC00:46
*** trown is now known as trown|outtypewww00:47
*** tovin07_ has joined #openstack-ironic00:49
*** krtaylor has joined #openstack-ironic00:51
openstackgerritOpenStack Release Bot proposed openstack/python-ironicclient: Update reno for stable/ocata  https://review.openstack.org/42547700:53
*** phuongnh has joined #openstack-ironic00:54
openstackgerritOpenStack Release Bot proposed openstack/python-ironic-inspector-client: Update reno for stable/ocata  https://review.openstack.org/42548000:56
*** rama_y has quit IRC00:56
*** sacharya has joined #openstack-ironic01:08
*** [1]rpioso has quit IRC01:20
TheJuliaAww stable/ocata01:23
*** baoli has joined #openstack-ironic01:28
*** baoli has quit IRC01:32
*** sdake has joined #openstack-ironic01:33
*** sdake_ has joined #openstack-ironic01:35
*** sdake has quit IRC01:38
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914101:39
*** sdake_ has quit IRC01:43
*** rajinir has quit IRC01:46
*** sdake has joined #openstack-ironic01:49
*** sdake_ has joined #openstack-ironic01:50
*** sdake has quit IRC01:54
*** aNuposic has quit IRC01:56
*** sacharya_ has joined #openstack-ironic02:07
*** sacharya has quit IRC02:08
*** sdake_ has quit IRC02:09
*** sdake has joined #openstack-ironic02:11
*** Sukhdev has quit IRC02:14
*** noama has quit IRC02:20
*** dansmith has quit IRC02:20
*** noama has joined #openstack-ironic02:20
*** dansmith has joined #openstack-ironic02:21
*** mtanino has quit IRC02:23
*** mitchjameson has quit IRC02:37
*** Syed__ has quit IRC02:45
*** sacharya_ has quit IRC02:50
*** sacharya has joined #openstack-ironic02:50
*** sayantani01 has quit IRC02:50
*** baoli has joined #openstack-ironic02:56
*** sdake has quit IRC02:58
*** d0ugal has quit IRC03:00
*** baoli has quit IRC03:10
*** rama_y has joined #openstack-ironic03:15
*** d0ugal has joined #openstack-ironic03:17
*** newmember has quit IRC03:23
*** rushil has quit IRC03:23
*** rbudden has quit IRC03:27
*** vikrant has joined #openstack-ironic03:30
*** ijw has quit IRC03:35
*** ijw has joined #openstack-ironic03:35
*** ijw has quit IRC03:35
*** rama_y has quit IRC03:39
*** baoli has joined #openstack-ironic03:39
*** rushil has joined #openstack-ironic03:41
*** sdake has joined #openstack-ironic03:46
*** Sukhdev has joined #openstack-ironic03:48
*** gcb has joined #openstack-ironic03:50
*** sasha2 has quit IRC04:07
*** sdake_ has joined #openstack-ironic04:08
*** sdake has quit IRC04:09
*** baoli has quit IRC04:20
*** sdake has joined #openstack-ironic04:31
*** sdake_ has quit IRC04:32
*** ianw is now known as ianw_pto04:34
*** sdake has quit IRC04:49
*** sayantani01 has joined #openstack-ironic04:57
*** Sukhdev has quit IRC04:58
*** Guest58531 has quit IRC05:04
*** comstud has quit IRC05:07
*** mgagne has joined #openstack-ironic05:07
*** mgagne is now known as Guest3353905:07
*** comstud has joined #openstack-ironic05:08
*** Sukhdev has joined #openstack-ironic05:09
*** sacharya has quit IRC05:14
*** moshele has joined #openstack-ironic05:14
*** sacharya has joined #openstack-ironic05:14
*** sacharya has quit IRC05:15
*** sacharya has joined #openstack-ironic05:15
*** jtomasek has quit IRC05:18
*** sacharya has quit IRC05:20
*** sdake has joined #openstack-ironic05:25
openstackgerritNaohiro Tamura proposed openstack/ironic: Update iRMC driver doc for soft reboot and soft power off  https://review.openstack.org/42554105:41
*** sdake has quit IRC05:45
*** Sukhdev has quit IRC05:49
*** rushil has quit IRC06:04
*** sdake has joined #openstack-ironic06:06
*** baoli has joined #openstack-ironic06:10
*** rbudden has joined #openstack-ironic06:10
*** baoli has quit IRC06:15
*** sacharya has joined #openstack-ironic06:16
*** sdake has quit IRC06:16
*** sdake has joined #openstack-ironic06:17
*** vsaienko has joined #openstack-ironic06:19
*** sdake_ has joined #openstack-ironic06:21
*** sacharya has quit IRC06:22
*** sdake has quit IRC06:23
*** sdake_ has quit IRC06:27
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425706:28
*** mjura has joined #openstack-ironic06:34
*** e0ne has joined #openstack-ironic06:45
openstackgerritNguyen Hung Phuong proposed openstack/ironic-specs: Support additional capabilities discovery for iRMC driver  https://review.openstack.org/40904406:46
*** e0ne has quit IRC06:46
openstackgerritNguyen Hung Phuong proposed openstack/ironic-specs: Support additional capabilities discovery for iRMC driver  https://review.openstack.org/40904406:47
*** e0ne has joined #openstack-ironic06:50
*** e0ne has quit IRC06:53
openstackgerritDao Cong Tien proposed openstack/python-ironicclient: Add args to CLI 'node-create' for selecting hardware interfaces  https://review.openstack.org/41835906:53
*** ChubYann has quit IRC06:55
*** e0ne has joined #openstack-ironic06:57
*** e0ne has quit IRC07:00
*** e0ne has joined #openstack-ironic07:04
openstackgerritNaohiro Tamura proposed openstack/ironic: Update iRMC driver doc for soft reboot and soft power off  https://review.openstack.org/42554107:06
*** e0ne has quit IRC07:09
*** tesseract has joined #openstack-ironic07:11
*** e0ne has joined #openstack-ironic07:12
*** e0ne has quit IRC07:14
*** sayantani01 has quit IRC07:15
*** e0ne has joined #openstack-ironic07:18
*** AlexeyAbashkin has joined #openstack-ironic07:22
*** moshele has quit IRC07:23
*** vsaienko has quit IRC07:26
*** lennyb has quit IRC07:27
*** noama has quit IRC07:27
*** noama has joined #openstack-ironic07:28
*** lennyb has joined #openstack-ironic07:28
*** noama has quit IRC07:29
*** vsaienko has joined #openstack-ironic07:29
*** vsaienko has quit IRC07:38
*** vsaienko has joined #openstack-ironic07:46
*** vsaienko has quit IRC07:54
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355607:57
*** e0ne has quit IRC07:59
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] enable only pxe|agent_ipmitool by default  https://review.openstack.org/42557208:00
*** _fatalis has joined #openstack-ironic08:05
*** tovin07_ has quit IRC08:05
*** ralonsoh has joined #openstack-ironic08:16
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add support for DBDeadlock handling  https://review.openstack.org/39441308:20
openstackgerritGrzegorz Grasza (xek) proposed openstack/ironic: Add support for DBDeadlock handling  https://review.openstack.org/39441308:21
*** rbudden has quit IRC08:27
*** phuongnh has quit IRC08:27
*** ccamacho has joined #openstack-ironic08:27
*** jtomasek has joined #openstack-ironic08:50
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-ironic09:00
*** e0ne has joined #openstack-ironic09:01
*** amarao has joined #openstack-ironic09:07
openstackgerritDao Cong Tien proposed openstack/python-ironicclient: Extends driver-list, driver-show supporting new hardware types  https://review.openstack.org/41927409:07
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506809:08
openstackgerritVasyl Saienko proposed openstack/ironic: Validate port info before assume we may use it  https://review.openstack.org/41588109:09
*** pester has joined #openstack-ironic09:10
*** AlexeyAbashkin has quit IRC09:10
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355609:18
*** athomas has joined #openstack-ironic09:21
*** lucas-afk is now known as lucasagomes09:25
lucasagomesmorning all09:25
sambetts|afkMorning all09:26
*** sambetts|afk is now known as sambetts09:26
*** amotoki has quit IRC09:27
*** e0ne has quit IRC09:27
*** mgoddard has joined #openstack-ironic09:27
*** amotoki has joined #openstack-ironic09:28
openstackgerritNaohiro Tamura proposed openstack/ironic: Follow-up iRMC power driver for soft reboot and soft power off  https://review.openstack.org/42559309:30
*** amotoki has quit IRC09:32
lucasagomessambetts, o/09:33
lucasagomesthe mitaka stable branch tests seems to be completely broken judging by https://review.openstack.org/#/c/422855/09:34
patchbotpatch 422855 - ironic-lib (stable/mitaka) - Limit memory usage when running qemu-img09:34
*** derekh has joined #openstack-ironic09:37
*** gcb has quit IRC09:37
*** amotoki has joined #openstack-ironic09:39
*** e0ne has joined #openstack-ironic09:40
openstackgerritSam Betts proposed openstack/ironic-inspector: [WIP] Put logic in place for using new Ironic network simulation  https://review.openstack.org/42128809:42
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable  https://review.openstack.org/42559909:42
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable  https://review.openstack.org/42559909:43
*** gcb has joined #openstack-ironic09:50
*** moshele has joined #openstack-ironic09:50
*** fxpester has joined #openstack-ironic09:54
*** MattMan has quit IRC09:55
*** aarefiev_afk is now known as aarefiev09:55
aarefievmorning all!09:56
*** MattMan has joined #openstack-ironic09:56
*** pester has quit IRC09:57
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable  https://review.openstack.org/42559909:58
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Download both disk and uec images  https://review.openstack.org/42069209:58
*** noama has joined #openstack-ironic09:59
openstackgerritDao Cong Tien proposed openstack/ironic: Add hardware type irmc-s4 for iRMC S4 servers  https://review.openstack.org/42560410:04
vdrokgood morning ironic, lucasagomes sambetts and aarefiev ! :)10:05
lucasagomessup o/10:05
*** hshiina has joined #openstack-ironic10:05
vsaienk0morning lucasagomes, dtantsur, milan, aarefiev, vdrok, sambetts and all ironicers!10:17
lucasagomeso/10:17
vdrokmorning vsaienk010:17
aarefievmorning vdrok, vsaienk0, lucasagomes10:18
*** amotoki has quit IRC10:18
vsaienk0sambetts: I found that we can't use nova hotplug NIC tempest tests. Current implementation check for new device after attach, but in ironic case new device will not be added, we just plug that NIC to needed network https://github.com/openstack/tempest/blob/50ca4b2bcd113a838846389f5d66bf7a39babd59/tempest/scenario/test_network_basic_ops.py#L27410:18
sambettsvsaienk0: hmm, I think we should be able to modify that test to work for both cases right? E.g. ensure that after DHCP at least one NIC has the IP address of the new port we attached or something :/10:21
sambettsvsaienk0: otherwise I would suggest that we move the check_new_nic test function into the class, so we only need to override that specific function but the rest of the logic should work right?10:22
sambettsvsaienk0: I'n fact looking a couple of lines below we don't even need to use DHCP they assign a static IP for us10:23
sambettsvsaienk0: s/I'n/in/10:23
vsaienk0sambetts: we can't inherit from class that contain tests, it will cause them to run twice10:23
*** yolanda has quit IRC10:23
sambettsvsaienk0: ah ... of course...10:23
vsaienk0sambetts: and another problem that to run hotplug test we will need to have a custom regex for tests. hotplug is not in the smoke group10:24
sambettsvsaienk0: in that case I would suggest that we modify this existing test to work for both use cases10:24
sambettsvsaienk0: ultimate goal is that we'll be running the full suite not just smoke10:24
vsaienk0sambetts: I see, but for now we need to run only specific tests, I don't think that we will be able to run full in nearest future10:25
*** yolanda has joined #openstack-ironic10:26
jrollmorning y'all10:28
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Set DEFAULT_IMAGE_NAME variable  https://review.openstack.org/42559910:28
openstackgerritVasyl Saienko proposed openstack/ironic: [Devstack] Download both disk and uec images  https://review.openstack.org/42069210:28
sambettshey jroll10:29
openstackgerritSam Betts proposed openstack/ironic: [WIP] Don't override device_owner for tenant network ports  https://review.openstack.org/42424810:36
vdrokgood morning jroll10:38
lucasagomesjroll, morning10:50
*** pester has joined #openstack-ironic10:53
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712610:56
*** dtantsur|afk is now known as dtantsur10:57
dtantsurmorning ironic10:57
lucasagomesdtantsur, morning10:58
mat128morning everyone10:58
dtantsurjroll, mind approving https://review.openstack.org/#/c/425066/ please?11:01
patchbotpatch 425066 - ironic-inspector (stable/newton) - Make grenade actually upgrade our source code11:01
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506811:03
jrolldtantsur: morning, done11:04
aarefievmorning jroll, dtantsur11:09
aarefievdtantsur, sambetts: what features we want to go in O for inspector?11:09
makowalsHello everyone11:15
makowalsQuestion from beginner about enrolling new node -- I'm constantly getting error "Error: IPMI call failed: power status" for newly added node, even though executing ipmitool manually returns correct result11:16
makowalsAny tips how to debug the problem? The only idea I have now is there is something wrong with the password which is being provided as "-f" to ipmitool11:16
*** openstackgerrit has quit IRC11:17
vdrokmorning dtantsur and mat12811:18
*** vsaienko has joined #openstack-ironic11:19
makowalsI'm using agent_ipmitool, but with pxe_ipmitool behaviour is exactly the same11:19
dtantsuraarefiev, hey! whatever we can land in a week? :) I'd not start anything new, just maybe finish what is up already, maybe do some small clean ups11:20
dtantsurmakowals, try looking in ironic-conductor logs. assuming debug logging is enabled, you can find the exact ipmitool command it executes. try it locally, see if there any problems.11:20
makowalsdtantsur,  indeed I did like this and it worked from the console11:21
makowalsdtantsur, the only difference is conductor provides password as "-f /tmp/tmpnPigQk"11:22
aarefievdtantsur: ok, I thought ff this week, going to review now :)11:22
dtantsurthere is one thing I'd like to fix due to the move to virtualbmc, stay tuned11:23
dtantsuraarefiev, I want to reset https://github.com/openstack/ironic-inspector/blob/master/example.conf#L63 to .* and deprecate it11:24
dtantsurwith virtualbmc we can't really figure out if we're on virt or not11:24
dtantsurand anyway I've heard of jammed DHCP on bare metal as well11:24
dtantsurwdyt? also sambetts ^^^11:24
dtantsurjroll, did we release clients yesterday?11:25
aarefievdtantsur: +1 that makes sense, ssh drivers was deprecated while ago11:26
jrolldtantsur: we did indeed :)11:27
dtantsurw00t11:28
*** pester has quit IRC11:28
aarefievdtantsur: does this mean we can't land feature with required client changes11:29
dtantsuraarefiev, we can land it, but the client change won't go in Ocata11:29
dtantsure.g. the API will available in Ocata, but the client change only in Pike11:30
aarefievok, thanks11:30
*** jkilpatr has quit IRC11:40
*** gcb has quit IRC11:43
*** openstackgerrit has joined #openstack-ironic11:43
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Deprecate introspection_delay_drivers option and make it no-op  https://review.openstack.org/42564111:43
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs maintenance for all api node's requests  https://review.openstack.org/39221311:55
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Deprecated log_bmc_address option  https://review.openstack.org/42564211:55
*** jkilpatr has joined #openstack-ironic11:58
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425712:00
*** pc_m has quit IRC12:01
sambettsdtantsur: +1 from me on deprecating that option12:04
*** jkilpatr has quit IRC12:07
*** pc_m has joined #openstack-ironic12:10
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355612:10
*** xavierr has quit IRC12:13
openstackgerritMerged openstack/ironic-inspector: Make grenade actually upgrade our source code  https://review.openstack.org/42506612:14
dtantsurw00t, working CI on stable/newton!12:15
aarefievdtantsur: when we are going to switch on tempest plugins from separate repo12:18
dtantsurmmm, after Ocata final? dunno, the folks cannot still agree on this as a Pike goal12:19
aarefievI see one issue in grenade with new devstack-tool12:19
*** priteau has joined #openstack-ironic12:19
dtantsurdevstack-tool? what is that?12:19
aarefievdtantsur: https://github.com/openstack/devstack-tools12:20
aarefievit's already used in grenade12:20
dtantsuromg12:20
* dtantsur thinks that if you need a tool for your configuration files, something goes wrong12:20
*** jkilpatr has joined #openstack-ironic12:21
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472012:21
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797012:21
jrollwhew, that was fun12:21
* dtantsur thinks of approving that before most of folks wake up :D12:22
jrolllol12:22
jrolldtantsur: rloo pointed out we forgot about raid things :P12:22
dtantsuroh12:22
jrollalso found a couple bugs that mocking was hiding12:22
openstackgerritGalyna Zholtkevych proposed openstack/ironic: DNM: Test glance version 2  https://review.openstack.org/42506812:22
dtantsurfun indeed12:22
* dtantsur is going to get seriously drunk when we finish this driver composition reform12:23
jroll+112:23
vdrokyeah, seems like "fast review" is not going well :)12:24
jrolldtantsur: here's what I have left:12:24
jrollcode for "conductor will fail to start if there is a name clash between classic and dynamic"12:24
jrollcode to allow no classic drivers to be loaded, vice versa12:24
jrollCI stuff12:24
jrollmissing anything obvious?12:24
dtantsur" code to allow no classic drivers to be loaded"?12:25
dtantsurwhat's it about?12:25
*** pester has joined #openstack-ironic12:25
*** vsaienko has quit IRC12:25
* jroll finds link12:25
dtantsuralso please review the IPMI driver: https://review.openstack.org/#/c/422708/12:26
patchbotpatch 422708 - ironic - Add hardware type for IPMI using ipmitool12:26
jrolldtantsur: https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L9112:26
dtantsuroh, good catch12:26
jrollget rid of that check and instead do "if not drivers and not hardware_types: fail" later12:26
jrolloh, I actually put the todo in code: https://github.com/openstack/ironic/blob/master/ironic/conductor/base_manager.py#L16212:26
jrollwill review ipmi driver today12:27
dtantsurcool, thnx. yeah, I think the TODO list for Ocata is complete12:27
jroll\o/12:27
dtantsurin Pike we should get MOAR HARDWARE TYPES12:27
jrolloh I guess we need those config warnings too12:27
jroll+112:27
* jroll steps away for a while12:30
*** ccamacho is now known as ccamacho|mcdnlds12:34
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712612:36
*** vikrant has quit IRC12:37
*** mjura has quit IRC12:38
*** mjura has joined #openstack-ironic12:40
*** vsaienko has joined #openstack-ironic12:45
dtantsurmmm, moar hardware types already, sweet: https://review.openstack.org/#/c/425604/12:45
patchbotpatch 425604 - ironic - Add hardware type irmc-s4 for iRMC S4 servers12:45
*** vikrant has joined #openstack-ironic12:45
*** vsaienko has quit IRC12:47
*** vikrant has quit IRC12:47
openstackgerritGalyna Zholtkevych proposed openstack/ironic: ETAGs for subresource requests  https://review.openstack.org/42526012:47
cdanielmorning al12:48
cdaniel*all12:48
openstackgerritMoshe Levi proposed openstack/ironic-inspector: Adding InfiniBand Support  https://review.openstack.org/26425712:48
*** fellypefca has joined #openstack-ironic12:48
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42567012:51
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/42567112:51
*** dprince has joined #openstack-ironic12:51
*** lucasagomes is now known as lucas-hungry12:52
aarefievdtantsur: introspection_delay_drivers will match all drivers, so delay is applied for all drivers, right?12:53
dtantsuraarefiev, correct, unless introspection_delay is set to 012:54
aarefievdtantsur: you just saying that delay is not applied in release notes12:55
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [devstack] Stop installing jq, we're not using it in tests any more  https://review.openstack.org/42567512:55
dtantsuraarefiev, s/not/now/, ugh :(12:56
dtantsurlemme fix12:56
dtantsuraarefiev, any more comments to fix?12:56
*** pester has quit IRC12:57
aarefievI don't see any12:58
dtantsurcool, posting an update12:58
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Deprecate introspection_delay_drivers option and make it no-op  https://review.openstack.org/42564112:58
* dtantsur ALWAYS mistypes "not" vs "now" :(12:59
* aarefiev correct dtantsur with s/not/now next time13:00
dtantsurjroll, wdyt about approving https://bugs.launchpad.net/ironic/+bug/1659209 ?13:06
openstackLaunchpad bug 1659209 in Ironic "[RFE] Add hardware type of iRMC S4 supporting ironic dynamic drivers" [Wishlist,In progress] - Assigned to Dao Cong Tien (tiendc)13:06
openstackgerritMerged openstack/ironic: [Devstack] enable only pxe|agent_ipmitool by default  https://review.openstack.org/42557213:08
*** bfournie has quit IRC13:08
jrolldtantsur: seems fine to me :)13:11
jrollnice small patch too13:11
* jroll approves it13:12
*** vsaienko has joined #openstack-ironic13:14
*** galyna has quit IRC13:15
*** lucas-hungry is now known as lucasagomes13:16
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Restrict access to IPA API  https://review.openstack.org/42568213:18
*** trown|outtypewww is now known as trown13:19
openstackgerritDmitry Tantsur proposed openstack/ironic: Add hardware type for IPMI using ipmitool  https://review.openstack.org/42270813:23
dtantsurhad to rebase :(13:23
jrolldtantsur: heh, was just about to ping you for that13:23
*** amotoki has joined #openstack-ironic13:24
openstackgerritYuriy Zveryanskyy proposed openstack/ironic-specs: Restrict access to IPA API  https://review.openstack.org/42568213:24
*** baoli has joined #openstack-ironic13:27
*** baoli has quit IRC13:27
*** baoli has joined #openstack-ironic13:31
*** baoli has quit IRC13:31
*** moshele has quit IRC13:34
*** baoli has joined #openstack-ironic13:38
*** bfournie has joined #openstack-ironic13:39
*** sdake has joined #openstack-ironic13:39
*** baoli has quit IRC13:39
*** baoli has joined #openstack-ironic13:39
*** vsaienko has quit IRC13:43
*** xavierr has joined #openstack-ironic13:49
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Remove heading "Team and repository tags"  https://review.openstack.org/42570013:50
*** jheroux has joined #openstack-ironic13:51
*** ccamacho|mcdnlds is now known as ccamacho13:53
*** pester has joined #openstack-ironic13:56
*** fellypefca has quit IRC13:56
*** rbudden has joined #openstack-ironic13:56
*** fxpester has quit IRC14:05
openstackgerritMerged openstack/ironic-inspector: Do not use loopback BMC addresses for lookup  https://review.openstack.org/42190914:05
*** Goneri has joined #openstack-ironic14:06
*** hw_wutianwei has joined #openstack-ironic14:08
*** vsaienko has joined #openstack-ironic14:10
*** pester has quit IRC14:10
openstackgerritMerged openstack/ironic-inspector: [devstack] Remove bridge only if it's not OVB_PHYSICAL_BRIDGE  https://review.openstack.org/42375814:11
*** rloo has joined #openstack-ironic14:15
openstackgerritAparna proposed openstack/proliantutils: Add HPSUM firmware update support  https://review.openstack.org/42200414:17
vsaienk0sambetts: please add to your review queue https://review.openstack.org/#/c/415881/14:17
patchbotpatch 415881 - ironic - Validate port info before assume we may use it14:17
sambettsvsaienk0: will do :)14:18
vsaienk0thanks!14:18
*** sasha21 has joined #openstack-ironic14:19
*** vsaienko has quit IRC14:20
*** absubram has joined #openstack-ironic14:22
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914114:23
*** jcoufal has joined #openstack-ironic14:24
*** absubram_ has joined #openstack-ironic14:26
*** vsaienko has joined #openstack-ironic14:26
*** absubram has quit IRC14:27
*** absubram_ is now known as absubram14:27
*** vsaienko has quit IRC14:29
*** srobert has joined #openstack-ironic14:30
openstackgerritOksana Voshchana proposed openstack/ironic: WIP: TEST GRENADE DEV  https://review.openstack.org/39712614:30
*** vsaienko has joined #openstack-ironic14:30
*** mrtenio has joined #openstack-ironic14:31
xavierrdtantsur: I have added the traceback to https://bugs.launchpad.net/ironic/+bug/165789214:33
openstackLaunchpad bug 1657892 in Ironic "ValueError when trying to deallocate a non allocated machine in OneView due fail" [Undecided,In progress] - Assigned to Xavier (marcusrafael)14:33
xavierrdtantsur: is that ok? :)14:33
*** hshiina has quit IRC14:34
dtantsuryep, thanks14:34
*** hw_wutianwei has quit IRC14:38
*** ElCoyote_ has joined #openstack-ironic14:39
xavierrsambetts: we were discussing about this patch some time ago. if you have some time to review it :)14:43
xavierrsambetts: https://review.openstack.org/#/c/408298/14:43
patchbotpatch 408298 - ironic - Shutdown server before change boot order for agent...14:43
*** rama_y has joined #openstack-ironic14:43
rloohi and morning all, xavierr, dtantsur, sambetts, vsaienk014:44
*** rama_y has quit IRC14:44
dtantsurmorning rloo14:44
xavierrhey rloo14:44
rloojroll: client release notes look good :) i probably would have combined the two NMI ones into one but no big deal. http://docs.openstack.org/releasenotes/python-ironicclient/unreleased.html14:45
*** cdearborn has joined #openstack-ironic14:50
*** mtanino has joined #openstack-ironic14:51
*** vsaienko has quit IRC14:51
*** mtanino has quit IRC14:52
jrollrloo: awesome, thanks :)14:53
*** rushil has joined #openstack-ironic14:54
vdrokmorning xavierr and rloo14:55
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914114:55
xavierrhey vdrok o/14:55
rloojroll: am reviewing 417970 now. was thinking about what you said about the patch being too big. why can't we split these patches up, and merge them in separately but ONLY enable them 'at the end'.14:55
*** jrist has quit IRC14:56
rloojroll: not to do for this particular patch. but eg with your two patches, the first one, for utils.allow_dynamic_drivers(), just always return false. then only need to bump microversion once in second patch.14:56
lucasagomesrloo, morning14:57
rloohi lucasagomes!14:57
xavierrmorning lucasagomes14:57
dtantsurdunno about this idea, but we can split it into 1. return hw types in drivers API (no microversion), 2. driver filtering (first microversion), 3. node attributes (second)14:57
dtantsurbut at this stage I suspect it's easier to just merge them finally...14:57
mgouldhi rloo lucasagomes dtantsur xavierr vdrok jroll sambetts14:58
rloodtantsur: i don't want to split these two patches. just thinking for future features.14:58
vdrokmorning mgould14:58
rloodtantsur: so that we bump the version only once when the feature is all there.14:59
rloodtantsur: and make the feature avail only when it is all there.14:59
*** rushil has quit IRC15:00
dtantsuryeah, it makes sense, though it complicates unit testing15:01
jrollrloo: yeah, maybe, I'll think about it next time15:01
* jroll half here half prepping dinner15:01
* rloo wonders why jroll is prepping dinner so early in the day15:01
rloodtantsur: good point. i'll have to think about it more the next time i see a feature with more than one API patch/version bump.15:02
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42542615:04
*** rushil has joined #openstack-ironic15:04
jlvillallucasagomes, On the no_ansi patch ^^  I was keeping both versions as the 'no_ansi' version is technically deleting data. So I wasn't sure if someone might need the full version for some debugging.15:05
jlvillallucasagomes, Not sure what you think?15:05
* dtantsur would probably not need the old version15:05
lucasagomesjlvillal, hmm I thought it was just converting the ansi stuff instead of deleting it15:06
*** absubram has quit IRC15:06
vsaienk0morning rloo15:06
lucasagomesjlvillal, so idk now, maybe keep both them... but I looked at both logs and the data seems to be there15:06
jlvillallucasagomes, It just strips out any ansi codes it finds. Deleting them.15:06
lucasagomesand no ansi is def easier to read so, I thought about just keeping it15:06
jlvillallucasagomes, Yeah the no_ansi is a lot easier to read :)15:07
jlvillalThis: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/logs/old/ironic-bm-logs/node-4_no_ansi_2017-01-26-00:46:51.txt.gz15:09
lucasagomesjlvillal, yeah, I guess we keep both then15:09
lucasagomesat least for now idk15:09
jlvillalcompared to this: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/logs/old/ironic-bm-logs/node-4_console_2017-01-26-00:46:51.txt.gz15:09
dtantsuryeah, the latter is completely useless15:09
*** jrist has joined #openstack-ironic15:10
jlvillalOnly makes sense if you cat it into your terminal. At least for me :)15:10
jlvillallucasagomes, Okay. I will leave it for now. If people want to remove the old version it is simple enough.15:11
*** moshele has joined #openstack-ironic15:11
lucasagomesjlvillal, yeah I will change my vote15:11
jlvillallucasagomes, thanks15:13
*** mtanino has joined #openstack-ironic15:14
*** moshele has quit IRC15:15
*** amarao has quit IRC15:22
*** Nisha_Agarwal has joined #openstack-ironic15:27
*** jaosorior has joined #openstack-ironic15:31
*** sacharya has joined #openstack-ironic15:31
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355615:33
*** stevemar has quit IRC15:36
openstackgerritGalyna Zholtkevych proposed openstack/ironic: Migrate to session for glance client  https://review.openstack.org/41076715:37
jrollrloo: low and slow today :P15:37
*** stevemar has joined #openstack-ironic15:37
rloojroll: oooo, will be yummy...15:38
jrollindeed!15:38
jrollI'm here for real now, what needs some help?15:39
rloojroll: if easy and tasty, please share :)15:39
*** rushil has quit IRC15:39
*** crushil has joined #openstack-ironic15:40
jrollrloo: salt/pepper beef roast. quickly sear to give it a crust. put in crockpot. add one large-ish jar of pepperoncini, including juice. set crockpot to low. wait 8 hours. pull apart and eat. :)15:40
*** vsaienko has joined #openstack-ironic15:41
jrollone of the easiest meals I do regularly, but so good15:41
rloojroll: easy except for searing (but expected). will have to look into that pepperoncini, not sure what that is.15:41
soliosgjfyi, for vim users, I've used AnsiEsc plugin to get a more readable view15:42
rloojroll: oh, spicy peppers... guess that won't work in our household...15:42
soliosghere http://www.vim.org/scripts/script.php?script_id=4979, or here https://github.com/vim-scripts/Improved-AnsiEsc15:42
jrollrloo: oh :(15:42
rloojroll: well, one web site sez they are 'mild' spicy peppers :)15:43
soliosgbut without question, jlvillal's script is a great contribution15:43
*** vsaienko has quit IRC15:43
jrollrloo: they're weaker than jalapeno, stronger than bell pepper, heh. the meat doesn't get spicy, though, so you could just toss the peppers after cooking15:44
rloojroll: i'm going to try it!15:46
jrollrloo: let me know how it turns out :)15:46
rloojroll: will do!15:47
*** rpioso has joined #openstack-ironic15:48
*** openstackgerrit has quit IRC15:48
dtantsur$ openstack baremetal port group list15:49
dtantsurResource could not be found. (HTTP 404)15:49
dtantsurehmmmm15:49
jrollwhat api version does osc default to?15:49
* jroll hopes it's latest15:49
dtantsurgood catch. no, it's not latest15:49
jrollle sigh15:50
mariojvpepperoncini is pretty mild, you can usually get it on a pizza if you want a taste15:51
dtantsurthis is pretty bad, now instead of a clear message we're getting HTTP 40415:51
jrolldtantsur: we can easily catch that and print a real error15:51
jrollthough I'd rather default client to latest :|15:51
dtantsurwell, how do we catch it?15:52
jrollexcept NotFound: ?15:52
mariojvhttps://github.com/openstack/python-ironicclient/blob/master/ironicclient/common/http.py#L4815:52
dtantsurjroll, it's fine for 'portgroup list', but what about 'portgroup show'?15:52
jrolldtantsur: yeeeesh15:53
jrolldo we give a different error in that case?15:53
mariojvidk if that's the actually used version though15:53
jrollmariojv: it is and I hate it so much15:53
mariojv(1.9 seems extremely low)15:53
dtantsurjroll, I suspect it's both HTTP 40415:53
mariojvreally??15:53
*** Syed__ has joined #openstack-ironic15:53
mariojvit should at _least_ go to last named release15:53
jrolldtantsur: yeah, but, I believe there's an error code involved15:53
mariojvpreferably negotiate the highest version it knows about15:53
jrollmariojv: people didn't want to ever bump it, because they don't want to break people's bash scripts :( :( :(15:54
* jroll repeats again, the CLI is not an SDK15:54
jrolls/an SDK/a bash SDK/15:54
* dtantsur kind of agrees15:54
mariojvi wonder how hard it would be to just try to roll back to a version where the command is supported if they pass an unrecognized command15:55
jrollI feel like we have tons of cleanup work to do15:55
jroll1) get the cli bumped up15:55
dtantsuryeah..15:55
jroll2) actually use new cli stuff in devstack etc15:55
dtantsurI wonder if we can fix something right now to avoid people confused as hell15:55
jroll3) so many deprecation warnings everywhere :/15:55
mariojvhow would y'all feel about this as a ptg topic, if it isn't one yet?15:55
dtantsurI do feel like that15:56
* dtantsur -> quick snack15:56
jrolldtantsur: I mean, we could hack it, something like: if api version < portgroup version, raise error('bump the version noob')15:56
jrollmariojv: +100015:56
dtantsurjroll, heh, maybe :)15:56
mariojvthere's already a topic about deprecating ironic CLI15:56
mariojvshould we mix this in with that, or have a separate topic?15:56
jrolllet's make it separate15:57
jrollit's gonna get heated15:57
jrollthough15:57
* jroll points at http://specs.openstack.org/openstack/ironic-specs/specs/kilo-implemented/api-microversions.html#new-client-default-microversion15:57
mariojvi think this is a problem that needs to be talked about as a preqreq to deprecating ironic cli15:57
mariojvooh15:57
jrollsecond paragraph15:57
jrollmaybe jfdi15:57
jrollespecially now that we have a good relnotes process15:57
*** openstackgerrit has joined #openstack-ironic15:57
openstackgerritNisha Agarwal proposed openstack/ironic: SNMP agent support for OOB inspection for iLO Drivers  https://review.openstack.org/37242815:57
mariojvi'll link to that in whatever i put in https://etherpad.openstack.org/p/ironic-pike-ptg15:58
jrollthanks15:58
jrollthere's probably some prework to do15:58
mariojvjroll: dtantsur can i put y'all's name on this topic as well?15:58
jrollin devstack and etc15:58
jrollmariojv: yes please15:58
* jroll just plans to yell about every topic anyway15:58
mariojv:)15:58
*** sayantani01 has joined #openstack-ironic15:59
* rloo wonders if jroll can yell once he isn't ptl16:01
jrollrloo: how'd you think I ended up as ptl? :P16:01
rloojroll: being nice? :D16:02
jrollheh16:02
dtantsurmariojv, ditto as jroll :)16:07
mariojvcool; L96, feel free to modify: https://etherpad.openstack.org/p/ironic-pike-ptg16:08
dtantsurrloo, if I become a ptl, I'll appoint jroll as "yelling liaison"16:08
jrollnice16:08
JayFdtantsur: I have a lot of volume to bring to the table16:08
JayFdtantsur: don't make any rash decisions16:08
rloodtantsur: Oh, wow, that's a good promise.16:08
rlooBut JayF has a good point.16:08
jrollI tend to think of yelling more about the ranting than the volume :P16:08
dtantsurlook, we already have a competition :D16:08
jrollmaybe I can pipe my rants through JayF16:09
openstackgerritMerged openstack/ironic: Add hardware type for IPMI using ipmitool  https://review.openstack.org/42270816:09
* mariojv would tune into a ptl election debate16:09
rlootoo bad we couldn't have a candidate-debate (or whatever they are called) at the ptg :)16:09
jrollwoooo ipmi16:09
jrollheh16:09
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Allow diskless nodes introspection  https://review.openstack.org/32662016:09
dtantsurrloo, would not work, we like each other too much ^_^16:09
*** krtaylor has quit IRC16:09
rloodtantsur: hmm, maybe you're all not fit to be candidates then ;)16:10
*** jrist has quit IRC16:10
dtantsurdo we have "against everyone" in our voting?16:10
rloodtantsur: we need someone that can buck the establishment. the peons are restless. let them eat cake i say.16:10
JayFdtantsur: nope, it's just ranked choice of three, but you can rank people equally16:11
dtantsurtoo bad, rloo will have to pick one of us :D16:11
rloodtantsur: IF I decide to be responsible (or is it a right) in this democratic system, and actually vote :)16:12
dtantsurtrue16:12
rloohonestly, i'd prefer if only one candidate ran, would make it easier for me cuz unfortunately i like you all and i think you'd all do a great job.16:13
JayFI was commenting to someone the other day, about how good it is for our communtiy that we have 3 folks running, all from different companies, and all very capable of doing the job16:13
rlooJayF: why?16:13
JayFrloo: just a sign of good and varied participation. As a project, Ironic isn't dominated by a single company or agenda16:14
JayFrloo: and having 3 different folks who could step in and be a good PTL is more than I'd suspect for other projects16:14
rlooJayF: but that would be true (to me) even if one candidate ran. maybe cuz for me, i know the candidates and what they are capable of.16:14
JayFI appreciate when elections aren't unopposed, as a general concept :)16:15
* dtantsur suggests next time all cores to run. we'll know who's the most popular here :D16:15
JayFIf I am elected PTL, I'll be able to draw from the strength of knowing folks had confidence in me to do it16:15
rloowhat would be good is if the candidate companies could sign (not necessarily in blood) that they fully support the candidates and will let them work 99% upstream.16:15
dtantsurJayF++16:15
mariojvrloo: that's a very interesting idea16:16
jlvillalI vote for blood! ;)16:16
rlooJayF: if there was only one candidate and i opposed them, i probably would mention it...16:16
rloooh oh, didn't realize there were some blood-thirsty folks in the crowd16:17
jlvillalo/16:17
JayFI'm usually sweet-tea-thirsty.16:17
JayFlol16:17
rlooJayF: but do you kiss babies?16:17
JayFrloo: I actually kinda hate children16:18
* dtantsur got distracted for a minute and the discussion is already slightly weird16:18
JayFrloo: like I'm glad they exist as a concept, and that the next generation is happening. I just want them to happen away from me.16:18
rloodtantsur: see, we need 100% attention from ptls. otherwise things degenerate!16:18
* dtantsur puts his signature under "I'm glad they exist as a concept, and that the next generation is happening. I just want them to happen away from me."16:19
dtantsurrloo, mm, maybe we need all 3 to become PTLs?16:19
* dtantsur notes that it's not even Friday yet16:20
JayFwe can form a mighty triumvirate, Roman style!16:20
*** aNuposic has joined #openstack-ironic16:20
rloodtantsur: i thought of that, but i suspect the buck has to stop somewhere and i don't think that is something i want to try to change. there are far more important things i think :)16:20
*** jcoufal has quit IRC16:20
dtantsurheh16:20
*** jrist has joined #openstack-ironic16:21
*** krtaylor has joined #openstack-ironic16:22
openstackgerritJohn L. Villalovos proposed openstack/ironic: Devstack: Create a "no ansi" logfile for the baremetal console logs  https://review.openstack.org/42545716:22
* mgould notes that Roman triumvirates tended to collapse into bloody civil war within a few years...16:25
dtantsurwell, jlvillal asked for some blood :)16:26
vdrokmaybe troika then?16:26
mgould[ http://thehistoryofrome.typepad.com/ <- strongly recommended podcast, btw]16:26
dtantsuroh vdrok +10050016:26
jlvillal:D16:26
JayFmgould: I literally just finished that16:26
JayFmgould: like listened to the final episode yesterday16:27
mgouldJayF: awesome! I've got about five episodes to go16:27
mgouldepically long, but totally worth it16:27
*** jtomasek has quit IRC16:27
JayFmgould: assuming you've done Hardcore History by Dan Carlin as well?16:27
jlvillalvsaienk0: vdrok: Not sure if you have time to look at the failure with the grenade-multitenant job: http://logs.openstack.org/49/422149/7/experimental/gate-grenade-dsvm-ironic-multitenant-ubuntu-xenial-nv/d416cdb/16:28
jlvillalvsaienk0: vdrok: My theory is that it has something to do with grenade changing the networks when it is finished. You discussed it in the summit presentation.16:28
mgouldJayF: not yet, but I'll need something to replace HoR in a few days :-)16:29
jlvillalvsaienk0: vdrok: But I don't know the details on how it was fixed.16:29
JayFmgould: it's a huge backlog, and multi-hour long episodes. If you buy the backlog, you'll have podcasts to listen to for a year+ lol16:30
lucasagomessambetts, quick q (possible dumb): For testing the ironic/neutron integration we have a "fake/generic" ml2 driver for neutron, right ?16:30
lucasagomessambetts, if so, what's the code repository for it ?16:30
lucasagomesvsaienk0, ^ maybe you know it ?16:32
vdroklucasagomes: github.com/openstack/networking-generic-switch16:33
lucasagomesvdrok, thanks much!16:33
vdrokjlvillal: yeah, seems like it, we'll take a look, likely tomorrow morning :)16:33
jlvillalvdrok: Woot! :)16:33
jlvillalvgadiraj: ^^^16:33
openstackgerritMerged openstack/python-ironic-inspector-client: Update reno for stable/ocata  https://review.openstack.org/42548016:36
*** baoli has quit IRC16:36
*** dprince has quit IRC16:37
*** jcoufal has joined #openstack-ironic16:37
*** vsaienko has joined #openstack-ironic16:38
jrollTheJulia: dtantsur: would like your eyes on this gerrit acl change: https://review.openstack.org/#/c/425777/16:39
patchbotpatch 425777 - openstack-infra/project-config - Clean up ACLs for some ironic projects16:39
jrolland krtaylor maybe? ^^ (trying to recall who else is molteniron16:39
jroll)16:39
dtantsurjroll, ironic-inspector-core includes ironic-core. do you want this change just for clarity?16:40
krtaylorjroll, sorry on a call16:41
*** e0ne has quit IRC16:41
krtaylormjturek, hamzy, ^^^^ molten iron ping16:41
*** sayantani01 has quit IRC16:41
jrolldtantsur: yeah, that was my goal16:41
*** sayantani01 has joined #openstack-ironic16:42
jrollkrtaylor: no rush :)16:42
dtantsurjroll, one further suggestion, everything else looks good16:42
jrollsoliosg: please don't add to the whiteboard's priorities, rather suggest items in the meeting. especially without a link. outside of the meeting please run things by the PTL or some other cores :)16:42
jrolldtantsur: thanks16:43
hamzythe molteniron bit for that patch seems ok16:43
*** rama_y has joined #openstack-ironic16:43
jrollhamzy: thanks!16:44
mjturekjroll: yep looks good16:44
hamzywe need moar reviewers! :)16:44
*** vsaienko has quit IRC16:53
NobodyCamGood Morning Ironic'ers16:53
dtantsurmorning NobodyCam16:53
NobodyCam:) morning dtantsur :)16:53
*** e0ne has joined #openstack-ironic16:54
*** e0ne has quit IRC16:55
*** tesseract has quit IRC16:55
*** athomas has quit IRC16:56
mgouldmorning NobodyCam16:59
jrollmorning NobodyCam! :)17:00
*** rajinir has joined #openstack-ironic17:00
NobodyCammorning mogul & roll ... almost Friday :)17:01
NobodyCamgah.17:01
NobodyCams/roll/jroll/17:01
dtantsurs/mogul/mgould/ as well I suspect :)17:02
* dtantsur hands NobodyCam some coffee17:02
*** [1]cdearborn has joined #openstack-ironic17:02
NobodyCamlol yep. /me trains autocorrect17:02
mgouldNobodyCam: rather be skiing? :-)17:03
jrollheh17:03
NobodyCamheheheh !!!17:03
soliosgjroll: thanks, removed 'tempest plugin migration' from that section17:03
jrollthanks soliosg17:03
soliosgjroll: will bring the topic in the meeting17:03
soliosg*bring up17:04
jrollthanks17:04
* dtantsur would prefer to delay discussing it to the PTG17:05
jrolldtantsur: yeah, I asked soliosg to bring it up in the meeting mostly as an FYI, we can punt agreeing to the plan to PTG if we like17:07
dtantsurack17:08
dtantsurI know there will be more discussions on the QA track, which I planned to attend17:08
dtantsurs/planned/am planning17:08
joannaJayF: should I change priority of https://bugs.launchpad.net/ironic/+bug/1513005 to something else than wishlist? (just assuming by 'openstack-wide priority)17:09
openstackLaunchpad bug 1513005 in Ironic "Auto deploy devstack with ir-api under apache" [Wishlist,In progress] - Assigned to Joanna Taryma (jtaryma)17:09
JayFjoanna: it's still considered an RFE, in fact, the title should be prefixed with [RFE]]17:09
patchbotError: Spurious "]".  You may want to quote your arguments with double quotes in order to prevent extra brackets from being evaluated as nested commands.17:09
JayFjoanna: I will vote to approve said RFE as long as it's detailed enoughh17:10
dtantsurjoanna, well, it is still a wish list item, not a bug17:10
* dtantsur is also ready to approve it17:10
jrollrloo: should I wait for comments on the second API patch or go ahead and address the first now?17:10
* jroll never wanted to make the [RFE] thing a rule, just a guideline, lol17:10
joannaJayF: do you think it needs additional description now?17:11
JayFjoanna: I'm OK with it as-is, but I understand the problem well. It never hurts to have a more detailed description :)17:11
*** crushil has quit IRC17:11
joannaso... assuming I'll add more details to it - is there anything else I should do to push it through the process of being approved and assigned a priority?17:12
JayFwell not all RFEs have to be a project priority17:13
JayFbut priorities for pike are determined at the ptg, or at least, drafted there17:13
mariojvyes, and usually the community can vote on them afterwards17:13
JayFyou just need another core to be +2 to the rfe being approved, then it gets its tag changed to 'rfe-approved'17:13
mariojvon gerrit17:13
JayFas far as you're concerned, I'd just update the bug to be more detailed, keep working, and when folks are less busy sprinting to the end of ocata, it should be easy to get attention and get the rfe approved17:14
*** sayantani01 has quit IRC17:14
dtantsurJayF, joanna, I'm +2 on it, feel free to add rfe-approved17:14
sambettslucasagomes: sorry I missed your ping, I think vsaienk0 pointed you in the right direction17:14
JayFjoanna: ^ done17:15
lucasagomessambetts, yeah, no worries! Thanks17:15
joannaJayF: are you +2 too? :)17:15
JayFI had already put that in the bug, lol17:15
joannadtantsur: thank you :)17:15
JayFI am a core, if that's what you're asking?17:15
*** persia has quit IRC17:16
joannaJayF: amazing thanks!17:16
dtantsur:)17:16
*** mjura has quit IRC17:16
joannaJayF: Nope, I was just making sure you're fine with current description :)17:16
*** cdearborn has quit IRC17:16
JayFit's pretty straightforward.17:16
* dtantsur now remembers we have to do the same with inspector, sigh..17:16
jrollheh17:17
jrollyeah that will be a project priority, because it's an openstack priority17:18
jrolljoanna: major project priorities like this, by the way, are recorded in the specs repo, e.g. http://specs.openstack.org/openstack/ironic-specs/priorities/ocata-priorities.html17:18
JayFjroll: we can avoid making it a pike priority by joanna JFDI :P17:18
jroll¯\_(ツ)_/¯17:19
lucasagomesfolks, calling it a day17:19
jrollwe should be focusing on ocata priorities right now17:19
lucasagomeshave a great evening!17:19
*** lucasagomes is now known as lucas-afk17:19
jrolllater lucas-afk \o17:19
joannaJayF: thanks, I know that page - the priority thing was a citation of you, yesterday :D17:19
lucas-afklater!17:19
* lucas-afk runs to the train station17:19
mgouldo/17:19
JayFjoanna: https://github.com/openstack/governance/blob/master/goals/pike/deploy-api-in-wsgi.rst17:20
*** persia has joined #openstack-ironic17:20
* dtantsur also goes17:21
dtantsursee you tomorrow17:21
JayFjoanna: properly rendered link: https://governance.openstack.org/tc/goals/pike/deploy-api-in-wsgi.html17:21
*** dtantsur is now known as dtantsur|afk17:21
JayFo/ have a good day17:21
mgouldo/17:21
joannathanks!17:21
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42567017:21
jrollsee ya, dtantsur|afk17:21
*** derekh has quit IRC17:24
*** sayantani01 has joined #openstack-ironic17:24
rloojroll: haven't yet looked at 2nd api patch (lunching)17:25
openstackgerritMerged openstack/ironic: Update iRMC driver doc for soft reboot and soft power off  https://review.openstack.org/42554117:26
jrollrloo: ok cool, updating the first then17:29
jrollthanks17:29
openstackgerritSam Betts proposed openstack/ironic: Rework Ironic devstack baremetal network simulation  https://review.openstack.org/39295917:31
*** aNuposic has quit IRC17:34
*** Nisha_Agarwal has quit IRC17:35
openstackgerritLeif Madsen proposed openstack/bifrost: Make path to ironic-rootwrap dynamic  https://review.openstack.org/42285617:38
*** mgould is now known as mgould|afk17:39
*** baoli has joined #openstack-ironic17:41
openstackgerritMerged openstack/ironic-inspector: Devstack: don't rely on Ironic local vars  https://review.openstack.org/42072517:47
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic interfaces fields to nodes API  https://review.openstack.org/42472017:50
openstackgerritJim Rollenhagen proposed openstack/ironic: Add dynamic driver functionality to REST API  https://review.openstack.org/41797017:50
openstackgerritAparna proposed openstack/proliantutils: Add HPSUM firmware update support  https://review.openstack.org/42200417:50
jrollrloo: vdrok: I think I got everything &17:50
jrolls/&/^/17:50
rloojroll: will look in a few min17:51
*** trown is now known as trown|lunch17:51
jrollyeah no rush :)17:51
* sambetts -> home17:51
sambettsnight all17:51
jroll\o17:51
sambettscya jroll17:51
*** sambetts is now known as sambetts|afk17:51
jlvillalWhy does ironic/db/sqlalchemy/api.py appear to have no unit tests? I'm just wondering.17:56
jrollit does, they're not named the same as that file17:57
jrollI did a wtf about that recently too17:58
jrolljlvillal: they're here: https://github.com/openstack/ironic/tree/master/ironic/tests/unit/db17:58
jlvillaljroll: Ah and :(17:58
jrollI do like that they're split but wonder if they should be in that sqlalchemy directory17:58
*** david-lyle has quit IRC17:59
* TheJulia gets a small sign that says "Certified Zombie" and hangs it on herself17:59
jlvillaljroll: Okay. Yeah I was looking at a function like: add_port_filter_by_node() and only found it in api.py.17:59
jlvillaljroll: I'm guessing we may not have total coverage.18:00
jlvillalThanks18:00
jrolljlvillal: we might not18:00
jrollI suspect that gets tested where it's used, given it isn't used outside of api.py18:01
jroll¯\_(ツ)_/¯18:01
jlvillaljroll: Yeah, I would prefer the tests to be in ironic/tests/unit/db/sqlalchemy/api/* for the tests if it has been split into multiple files18:02
*** aNuposic has joined #openstack-ironic18:02
*** mgoddard has quit IRC18:02
* jlvillal just though of that idea to create a directory based on the file name. Pure genius18:02
* jlvillal strains should trying to pat himself on the back :)18:02
jlvillals/should/shoulder/18:02
jrolljlvillal: yeah, though I think it's a distraction right now when we should be focusing on priorities18:02
rloojlvillal: ++. historical. not enough people. blah. blah :-( and what jroll sez.18:03
*** ralonsoh has quit IRC18:03
jlvillal+118:04
* jlvillal got distracted while reviewing a patch set18:04
TheJuliajroll: Reviewed the project config change.  Looks like the testing doesn't like it though :\18:05
jrollTheJulia: oh, whomp, I'll take a look. thanks.18:05
jlvillalKeep project-config weird, don't normalize it! :)18:06
TheJulialol18:06
* TheJulia clearly needs less blood in her coffee distribution system this morning.18:07
jrollTheJulia: I can add bifrost-core to bifrost stable maintenance, it was unclear what was going on there to me18:07
TheJuliajroll: Up to you.  It is really rare for anything to have to be landed to the stable branches, aside from the normal required stuff18:09
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42567018:09
jrollTheJulia: okay, I don't mind adding it back18:09
TheJuliaCool18:09
openstackgerritMerged openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/42567118:10
openstackgerritMerged openstack/ironic-inspector: Deprecate setting IPMI credentials  https://review.openstack.org/41704118:10
TheJuliapsutil again...18:10
JayFTheJulia: what is psutil doing/18:11
JayFoh18:11
JayFoh that's a beaut18:11
TheJuliaindeed18:11
JayFwill make my psutil patch a lot easier18:11
JayFI'm glad they merged that18:11
TheJuliaI think housekeeping wants me to checkout. :\18:12
jlvillalrloo: On the python-ironicclient queue. https://review.openstack.org/#/q/status:open+project:openstack/python-ironicclient18:13
jlvillalrloo: Any particular ones to focus on?18:13
jrolljlvillal: I should cross those off the whiteboard, we released it yesterday18:13
rloojlvillal: nope, not any more. i mean, the client release was cut yesterday18:13
rloojlvillal: so focus on ironic features18:14
rloojlvillal: driver composition18:14
jlvillalOkay.18:14
rloojlvillal: welcome to the club :)18:14
jrolldriver composition and node tags are the big ones right now18:14
rloojroll: node tags didn't make the client cut but i guess it would be good to get the ironic part in.18:14
rloojroll: after we get the driver comp stuff in :)18:15
jrollrloo: yep, agree18:15
*** crushil has joined #openstack-ironic18:15
* jroll wonders if I should -2 node tags API until driver comp lands, just in case18:15
rloojroll: bet you it won't matter!18:15
jroll:)18:16
JayFdoes LOG.exception just log the exception, if used in an except block, or does it reraise the exception too?18:19
* JayF can't find this in any oslo.log docs18:19
jrollthe former18:19
jrolland it logs the entire traceback18:19
jrollat ERROR level18:19
JayFperfect, that's exactly what I want in this case, I thikn18:19
rloojroll: geez. since we introduce hw types in 1.30, are we allowing operations on them prior to 1.30, like getting RAID disk properties or vendor-interface-related stuff?18:20
jrollLOG.exception() == LOG.error() + traceback.print_exc() or so18:20
jrollrloo: yes, since one can set node.driver to a hw type in any API version18:20
rloojroll: which reminds, me i didn't even check the patches against the spec.18:20
rloojroll: Ok.18:20
jrollrloo: also why we return hardware types in GET /v1/drivers for all versions18:20
jrollrloo: good thing you didn't, the spec forgot raid :)18:20
*** mrtenio has quit IRC18:20
rloojroll: hard to remember to include everything in the spec. review, review, review, that's the secret :-(18:21
jrollhmm, I should probably lunch before meetings18:21
jrollrloo: yep, it's hard18:21
* jroll bbl18:21
*** mrtenio has joined #openstack-ironic18:22
*** xek_ has joined #openstack-ironic18:22
*** xek has quit IRC18:22
*** moshele has joined #openstack-ironic18:24
rloobefore i forget, jroll: is there an api-ref patch for driver composition?18:26
*** jtomasek has joined #openstack-ironic18:26
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Remove support for psutil <3  https://review.openstack.org/42528418:27
*** dprince has joined #openstack-ironic18:28
openstackgerritAparna proposed openstack/proliantutils: Add HPSUM firmware update support  https://review.openstack.org/42200418:31
*** vinsh has joined #openstack-ironic18:32
*** rcernin has joined #openstack-ironic18:32
*** amotoki has quit IRC18:33
*** dsneddon_afk is now known as dsneddon18:35
openstackgerritAparna proposed openstack/proliantutils: Add return code for hpsum firmware update  https://review.openstack.org/42385618:37
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/42567018:44
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements  https://review.openstack.org/42582618:44
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui: Updated from global requirements  https://review.openstack.org/42327218:44
jrollrloo: not yet, no18:48
*** Goneri has quit IRC18:48
rloojroll: on your list though, right? :)18:48
openstackgerritMerged openstack/bifrost: Typo fix: insufficent = insufficient  https://review.openstack.org/42505318:48
jrollrloo: the ever-growing list, yes18:48
rloojroll: what? should be shrinking.18:49
*** jtomasek has quit IRC18:49
*** moshele has quit IRC18:49
*** crushil has quit IRC18:49
jrollrloo: always growing!18:49
jrollTheJulia: updated the thing https://review.openstack.org/#/c/425777/18:50
patchbotpatch 425777 - openstack-infra/project-config - Clean up ACLs for some ironic projects18:50
TheJuliajroll: Thank you18:50
*** ijw has joined #openstack-ironic18:51
jrollnp18:51
* TheJulia goes and finds a taxi18:52
*** trown|lunch is now known as trown18:54
joanna<low priority warning> I'm thinking - there are certain values for e.g. boot_interface field od node, right? Why we don't use enums in the db for such cases?18:55
jrolljoanna: primary reason is, folks can write their own implementations for any interface, and install those into ironic, so we don't know all the values18:56
joannajroll: thanks! :) makes sense18:56
JayFjoanna: a couple of reasons I can think of, you can have 3rd party implementations, and do we want a DB migration anytime we add a new interface18:56
jrollwe have a db migration for that anyway :P18:57
rloojroll: you have my +2 on the two driver comp patches; you should be able to get +A by tomorrow at the latest I think.18:58
joannaJayF: thanks :)18:58
jrollrloo: nice, thanks!18:59
*** harlowja has quit IRC19:02
*** jtomasek has joined #openstack-ironic19:02
mariojvi think JayF might have meant interface implementation19:04
mariojvwrt the db migration19:04
*** derekh has joined #openstack-ironic19:05
*** crushil has joined #openstack-ironic19:05
*** alexpilotti has joined #openstack-ironic19:07
jrollmmm19:08
TheJuliaRe: db enum, I seem to remember there is/was an issue with sqlalchemy and using enum when we run db related unit tests without a real database.19:08
*** alexpilotti has quit IRC19:08
*** alexpilotti has joined #openstack-ironic19:09
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914119:12
*** alexpilotti has quit IRC19:13
*** Sukhdev has joined #openstack-ironic19:16
*** yolanda has quit IRC19:17
*** absubram has joined #openstack-ironic19:18
*** jtomasek has quit IRC19:20
openstackgerritMerged openstack/ironic: Correctly cache "abortable" flag for manual clean steps  https://review.openstack.org/42323519:20
*** jtomasek has joined #openstack-ironic19:23
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Remove support for psutil <3  https://review.openstack.org/42528419:25
*** derekh has quit IRC19:27
JayFTheJulia: ^ pep8 fixes :-C19:28
openstackgerritMerged openstack/python-ironicclient: Update reno for stable/ocata  https://review.openstack.org/42547719:30
TheJulia:(19:30
TheJuliaI feel slightly better that I didn't spot it now that I've looked at it again :)19:32
*** jjohnson2 has quit IRC19:33
*** amotoki has joined #openstack-ironic19:34
*** yolanda has joined #openstack-ironic19:34
*** jheroux has quit IRC19:35
*** jkilpatr has quit IRC19:35
*** crushil has quit IRC19:39
*** Sukhdev_ has joined #openstack-ironic19:44
*** david-lyle has joined #openstack-ironic19:45
*** mgoddard has joined #openstack-ironic19:47
*** sayantani01 has quit IRC19:51
*** jkilpatr has joined #openstack-ironic19:51
*** jheroux has joined #openstack-ironic19:53
openstackgerritChris Krelle proposed openstack/bifrost: remove pxe_amt driver from pxe enable drivers list  https://review.openstack.org/42587119:54
*** mtanino has quit IRC19:55
jlvillaljroll: I'll do some more reviewing of the driver composition patch after lunch. Slow going...19:55
jlvillalA lot of code!19:55
openstackgerritJim Rollenhagen proposed openstack/ironic: Clean up nits on driver composition API changes  https://review.openstack.org/42587319:56
jrolljlvillal: would prefer a single review, but ^ should fix everything so far19:56
*** Guest33539 is now known as mgagne20:01
*** mgagne has quit IRC20:01
*** mgagne has joined #openstack-ironic20:01
*** harlowja has joined #openstack-ironic20:09
*** Goneri has joined #openstack-ironic20:11
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes  https://review.openstack.org/42128520:12
*** mgoddard has quit IRC20:15
openstackgerritMilan Kováčik proposed openstack/ironic-inspector: Find a node by multiple attributes  https://review.openstack.org/42128520:15
*** jtomasek has quit IRC20:16
openstackgerritRuby Loo proposed openstack/python-ironicclient: Use same variable for --[no-]maintenance  https://review.openstack.org/42588920:21
*** dyasny has quit IRC20:22
*** aNuposic has quit IRC20:28
rloojroll: the resource classes -- do we know for sure that it will be used by nova in pike?20:30
rloojroll: 'it' meaning ironic's resource_class :)20:31
*** sayantani01 has joined #openstack-ironic20:31
jrollrloo: I am told this is the goal20:33
rloojroll: ok20:33
jrollI cannot predict the future20:33
jrollkeep in mind docs can be edited20:34
rloojroll: yup, i'm keeping that in mind (and glad the doc didn't go into newton)20:34
jrollI don't think it would have hurt in newton, but ¯\_(ツ)_/¯20:34
rloojroll: too far into the future :)20:36
jrollrloo: OTOH, deployment projects would have had plenty of time to implement it20:37
rloojroll: perhaps. but what if i implemented it based on the description, and then later decide that the resource classes i created/named didn't make any sense cuz it became (more) clearer later, how they were going to be used.20:38
rloojroll: or what if that feature never happened...20:38
*** milan has joined #openstack-ironic20:38
jrollrloo: if this stuff doesn't happen I'm quitting20:39
* rloo hopes no one saw that20:39
*** srobert_ has joined #openstack-ironic20:39
milanmorning Ironic! :)20:40
jrollmorning milan20:40
milanmorning jroll! :)20:41
*** dyasny has joined #openstack-ironic20:41
*** srobert has quit IRC20:42
*** aNuposic has joined #openstack-ironic20:43
joannamorning :)20:43
*** e0ne has joined #openstack-ironic20:44
*** srobert_ has quit IRC20:48
milanmorning joanna! :)20:49
*** srobert has joined #openstack-ironic20:49
NobodyCammorning Joanna :)20:51
joanna:)20:51
milanmorning NobodyCam! :)20:52
NobodyCammorning milan :)20:53
*** e0ne has quit IRC20:54
*** srart has quit IRC20:54
*** e0ne has joined #openstack-ironic20:58
*** hecliunyx has quit IRC20:59
*** ijw has quit IRC20:59
*** ijw has joined #openstack-ironic20:59
*** jcoufal has quit IRC21:03
*** jcoufal has joined #openstack-ironic21:04
*** srart has joined #openstack-ironic21:06
*** e0ne has quit IRC21:09
* jroll is done for the day, see y'all tomorrow21:09
rloohave a good evening (and dinner) jroll!21:10
milannight jroll! :)21:12
milanand hi rloo! :)21:13
*** moshele has joined #openstack-ironic21:14
*** crushil has joined #openstack-ironic21:14
*** jcoufal has quit IRC21:14
*** jcoufal has joined #openstack-ironic21:18
*** moshele has quit IRC21:19
*** mgoddard has joined #openstack-ironic21:30
*** jcoufal has quit IRC21:30
*** sacharya has quit IRC21:34
*** jkilpatr has quit IRC21:35
*** aNuposic has quit IRC21:36
*** vsaienko has joined #openstack-ironic21:40
*** Sukhdev has quit IRC21:41
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355621:42
rloomilan: what are you doing here so late?21:43
milan:)21:44
* milan plans to go to devconf tomorrow O:-)21:44
milanfor a bit21:44
rloomilan: enjoy! :)21:47
*** ppiela has quit IRC21:48
milanrloo, thanks! :)21:49
*** ppiela has joined #openstack-ironic21:50
*** milan has quit IRC21:51
*** ijw has quit IRC21:52
*** moshele has joined #openstack-ironic21:52
*** mgoddard has quit IRC21:54
openstackgerritJoanna Taryma proposed openstack/ironic: Fixed allowing non-UUID value as UUID for port and portgroup patches.  https://review.openstack.org/40391021:55
*** milan has joined #openstack-ironic21:56
*** moshele has quit IRC21:57
*** aNuposic has joined #openstack-ironic22:00
*** trown is now known as trown|outtypewww22:02
*** jheroux has quit IRC22:03
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914122:03
mariojvJayF: does ipa need to update from global requirements before https://review.openstack.org/#/c/425284/ lands ?22:04
patchbotpatch 425284 - ironic-python-agent - Remove support for psutil <322:04
mariojvnvm22:04
mariojvalready there22:04
JayFmariojv: https://github.com/openstack/ironic-python-agent/commit/7ec1f254751e2953ffd1cb132a3a661a4440ed8c22:04
mariojvhttps://github.com/openstack/ironic-python-agent/blob/master/requirements.txt#L1722:04
mariojvyup, thanks22:04
JayFthat was the catalyst to rewrite that patch22:04
mariojvi was thinking about the source from yesterday22:05
mariojvcool22:05
JayFgoing to try and respond to comments real quick before vanessa calls22:05
*** milan has quit IRC22:06
*** priteau has quit IRC22:09
*** mtanino has joined #openstack-ironic22:09
*** zackf has quit IRC22:09
openstackgerritVasyl Saienko proposed openstack/ironic: WIP: ironic standalone tests  https://review.openstack.org/42355622:11
*** ijw has joined #openstack-ironic22:12
*** jkilpatr has joined #openstack-ironic22:13
csmart\o morning22:14
JayFcsmart: you're the smart man mrda has been telling me about22:14
csmartJayF: smart by name only... ;-)22:15
*** dprince has quit IRC22:15
*** rama_y has quit IRC22:15
csmartJayF: I think we have a meeting scheduled for next week, but happy to chat here any time22:16
csmartI'm in UT+1122:16
csmart*UTC+1122:16
* csmart is still getting settled in (first week)22:16
*** milan has joined #openstack-ironic22:16
*** ijw has quit IRC22:17
JayFcsmart: so I heard. Welcome to the group, and we'll get to know each other better next week :)22:18
*** ijw has joined #openstack-ironic22:18
*** ijw has quit IRC22:18
*** ijw has joined #openstack-ironic22:19
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914122:19
csmartJayF: rgr, thanks. sounds good22:20
*** cdearborn has joined #openstack-ironic22:20
* JayF introduces csmart to the channel: Chris is a new Racker who is going to be working in one of our cloud product groups. He has some experience with embedded linux so I want to nerd sni^W^W ask him nicely to help with our agent image stuff :)22:21
*** sayantani01 has quit IRC22:21
*** milan has quit IRC22:21
*** sayantani01 has joined #openstack-ironic22:22
* csmart waves to everyone, thanks JayF, looking forward to getting my teeth into something22:23
*** sacharya has joined #openstack-ironic22:23
JayFcsmart: was that intentional? use of the word teeth?22:23
csmartJayF: yeah, I assumed that would translate, just means to get deep into something22:24
csmart:-S22:24
JayFcsmart: Ot'22:24
JayFcsmart: the team that originally launched OnMetal at Rackspace (powered by Ironic) and started our whole involvement in the project was the "Teeth Team" (see: https://journal.paul.querna.org/articles/2014/07/02/putting-teeth-in-our-public-cloud/)22:25
JayF really the only remnants of that name is the internal irc channel at rackspace for onmetal and this awesome logo https://journal.paul.querna.org/assets/posts/putting-teeth-in-our-public-cloud/teeth-logo.png lol22:26
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client: Add new introspection commands for interface data including lldp  https://review.openstack.org/41914122:27
*** baoli has quit IRC22:28
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Remove support for older psutil versions  https://review.openstack.org/42528422:29
JayFmariojv: jlvillal rloo soliosg ^ your comments have been responded to and/or fixed. TYVM for the reviews.22:29
csmartJayF: ahh, gotchat ;-)22:29
soliosgcsmart: welcome aboard!22:30
*** aNuposic has quit IRC22:31
soliosgcsmart: i'm sure you'll soon find something to work on22:32
mariojvthx22:32
csmartsoliosg: thanks :-)22:34
*** milan has joined #openstack-ironic22:35
*** [1]cdearborn has quit IRC22:35
*** vsaienko has quit IRC22:37
*** crushil has quit IRC22:39
*** hecliunyx has joined #openstack-ironic22:44
jlvillalJayF: +222:44
*** sacharya has quit IRC22:46
*** aNuposic has joined #openstack-ironic22:51
*** sayantani01 has quit IRC22:55
*** sayantani01 has joined #openstack-ironic22:56
*** mgoddard has joined #openstack-ironic22:58
*** srobert_ has joined #openstack-ironic23:03
*** skazi has quit IRC23:06
*** srobert has quit IRC23:06
*** srobert_ has quit IRC23:07
*** absubram has quit IRC23:13
*** pmannidi has joined #openstack-ironic23:14
*** skazi has joined #openstack-ironic23:16
*** mgoddard has quit IRC23:17
Sukhdev_jlvillal : Hey John - are you there?23:24
jlvillalSukhdev_: I am23:24
Sukhdev_jlvillal : cool - have a QQ - see if you can help23:25
jlvillalI'll try :)23:25
*** skazi has quit IRC23:25
* jlvillal hopes it isn't a you work at Intel so you must know about this Intel hardware question :)23:25
Sukhdev_jlvillal : no nothing of that sort - strictly ironic related23:26
jlvillal:)23:26
Sukhdev_jlvillal : I have an Ironic setup (created by devstack), which is all working fine - I have tested using the VM acting a BMs all on a single server23:27
jlvillalokay23:27
Sukhdev_jlvillal: now I want to switch to the real HW - I have wired everything up - i.e. a switch, BM machine, etc and connected the controller running Ironic conductor to the switch as well23:27
jlvillalokay23:28
Sukhdev_jlvillal: instead of booting the VMs (acting as BMs), want boot the real HW BM server23:28
Sukhdev_jlvillal : so the question is -23:28
Sukhdev_jlvillal : If I update the port created by devstack to change the mac address to the real BM's mac address23:29
Sukhdev_will that work?23:29
jlvillalSukhdev_: Hmm. I'm not sure.23:30
* jlvillal hopes someone else will chime in too23:30
jlvillalSukhdev_: What is your deploy driver?23:30
Sukhdev_jlvillal : one second - let me past it23:31
Sukhdev_jlvillal : here is the paste - http://paste.openstack.org/show/596653/23:32
jlvillalSukhdev_: So your bare-metal node. How is it going to be controlled? IPMI?23:33
Sukhdev_jlvillal : yes23:33
jlvillalSukhdev_: So probably want to change the deploy driver from 'agent_ssh' to 'agent_ipmitool'. I think.23:34
*** skazi has joined #openstack-ironic23:34
jlvillalSukhdev_: I am supposed to get my own IPMI box to use for Ironic testing in a few weeks. So I haven't yet set it up in real life.23:34
Sukhdev_jlvillal : :-)23:34
Sukhdev_jlvillal : but, Ironic conductor will need to communicate with IPA over ssh, no?23:35
Sukhdev_jlvillal : so, I thought both will be needed?23:35
jlvillalSukhdev_: Instead of a devstack way. You might consider using Bifrost. I think TheJulia would recommend that.23:35
Sukhdev_jlvillal : do not what is Bifrost way23:36
jlvillalSukhdev_: agent_ssh means use the agent. But control the power via SSH.23:37
* TheJulia reads23:37
aslezilattaching real hardware to  devstack is going to be hard. Devstack has its own limitation23:37
Sukhdev_jlvillal : I thought as long as I have all the nodes created and the ramdisk image created by devstack, I can use all of that as-is and point the mac address and IPMI tool to the correct address, it should do the trick23:38
jlvillalhttps://github.com/openstack/ironic/blob/master/ironic/drivers/agent.py#L64-L7023:38
jlvillalSukhdev_: The agent_ssh driver doesn't use IPMI though.23:39
TheJuliaDevstack has a flag for using it with real hardware though23:39
Sukhdev_jlvillal : I should clarify - this is still test setup, just want to replace the VM with real HW :-)23:39
* jlvillal has never tried that flag23:39
TheJuliaAbout to head into airport security btw23:40
jlvillalSukhdev_: Maybe look for the flag TheJulia mentioned23:40
Sukhdev_TheJulia : what is that flag called?23:40
jlvillalSukhdev_: I have never tried what you are doing. So I'm unsure.23:40
TheJuliajlvillal: two or more of the third party CI systems use it, and it was used at HPE for downstream ionic ci23:40
TheJuliaI don't remember the flag offhand23:40
TheJuliaI can look once through security23:41
jlvillalSukhdev_: TheJulia: I think it is: IRONIC_IS_HARDWARE23:41
*** noama has quit IRC23:42
*** bfournie has quit IRC23:42
jlvillalSukhdev_: Look in openstack/ironic/devstack/lib/ironic23:43
*** noama has joined #openstack-ironic23:43
Sukhdev_jlvillal : I see it in that file - so, set it to true in local.conf and re-run the devstack, will that do the trick?23:44
jlvillalSukhdev_: I imagine have to set a few other settings23:44
Sukhdev_jlvillal : I was afraid you will say so :-):-)23:45
jlvillalSukhdev_: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L63-L6823:45
TheJuliaThat23:46
jlvillalSukhdev_: https://github.com/openstack/ironic/blob/master/devstack/lib/ironic#L1371-L137823:46
jlvillalSukhdev_: Good luck!23:46
Sukhdev_jlvillal : thanks - looks like I am going to need a lots of luck for being a brave soul23:47
Sukhdev_TheJulia : if you happen to have a moment later, would appreciate if you can point me to anything else that I need to get this going23:48
*** noama has quit IRC23:48
*** noama has joined #openstack-ironic23:49
vgadirajSukhdev_: if this doesn't work or you run into a ton of issues, it may be because I think in devstack the baremetal nodes are wired directly into neutron OVS bridge.23:51
*** david-lyle has quit IRC23:51
Sukhdev_vgadiraj : hmm. you may be right about it23:52
*** jaosorior has quit IRC23:53
Sukhdev_vgadiraj : what information related to ipmi needs to be set in the ironic config file - do you have any idea?23:53
*** david-lyle has joined #openstack-ironic23:54
*** rcernin has quit IRC23:56
*** rcernin has joined #openstack-ironic23:57
TheJuliaOkay, through security23:58
TheJuliaSukhdev_: no ipmi related dates needs to go into ironic.conf, it all should go into the driver_info23:59
*** rcernin has quit IRC23:59

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