Friday, 2015-08-28

*** achanda has quit IRC00:00
*** achanda_ has quit IRC00:01
*** dims has joined #openstack-ironic00:03
*** bnemec has quit IRC00:03
*** dims has quit IRC00:05
*** dims has joined #openstack-ironic00:05
*** enikanorov has quit IRC00:05
*** enikanorov has joined #openstack-ironic00:07
jlvillalclif_h: Not sure how big you want to grow that patch :)00:07
*** tsekiyam_ has joined #openstack-ironic00:09
*** dims has quit IRC00:10
jlvillalclif_h: It does make me wonder if IronicException should define __unicode__  ??00:10
*** mtanino has quit IRC00:12
*** tsekiyama has quit IRC00:13
*** tsekiyam_ has quit IRC00:13
*** dims has joined #openstack-ironic00:17
openstackgerritJohn Trowbridge proposed openstack/ironic-inspector: Store and expose introspection data  https://review.openstack.org/21315900:19
*** yuikotakada has joined #openstack-ironic00:19
*** naohirot has joined #openstack-ironic00:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** meghal1 has quit IRC00:24
mrdatonyb: that's not an easy thing to check00:26
*** garthb has quit IRC00:34
*** penick has quit IRC00:36
*** raghu has joined #openstack-ironic00:36
tonybmrda: It's not that hard either.00:46
*** blakec has joined #openstack-ironic00:53
mrda:)00:56
*** zhenguo has joined #openstack-ironic00:57
*** smoriya has joined #openstack-ironic01:10
jlvillalmrda, I implemented your idea. https://review.openstack.org/#/c/217853/  Not sure how well I implemented it ;)01:12
*** blakec has quit IRC01:21
*** Nisha has quit IRC01:23
openstackgerritlei zhang proposed openstack/ironic: Add minimum and maximum on port option  https://review.openstack.org/21701901:23
*** meghal has joined #openstack-ironic01:33
*** meghal1 has joined #openstack-ironic01:33
*** chenglch has joined #openstack-ironic01:35
*** meghal has quit IRC01:37
*** sinval_ has joined #openstack-ironic01:39
*** praneshp has quit IRC01:43
*** Haomeng|2 has joined #openstack-ironic01:43
*** penick has joined #openstack-ironic01:49
*** Haomeng has joined #openstack-ironic01:57
*** penick has quit IRC01:59
*** Haomeng|2 has quit IRC02:00
*** dims has quit IRC02:06
*** JoshNang has quit IRC02:06
*** sinval_ has quit IRC02:08
*** JoshNang has joined #openstack-ironic02:09
*** boris-42 has quit IRC02:10
*** rloo has quit IRC02:11
*** raginbajin has quit IRC02:23
*** raginbajin has joined #openstack-ironic02:28
*** achanda has joined #openstack-ironic02:28
*** purp has quit IRC02:33
*** dims has joined #openstack-ironic02:33
*** coreycb has quit IRC02:33
*** purp has joined #openstack-ironic02:36
*** coreycb has joined #openstack-ironic02:36
*** penick has joined #openstack-ironic02:40
openstackgerritTan Lin proposed openstack/ironic: Base IronicObject on VersionedObject  https://review.openstack.org/21360202:48
*** hakimo has joined #openstack-ironic02:52
*** penick has quit IRC02:54
*** shoutm has quit IRC02:54
*** hakimo_ has quit IRC02:54
*** baoli has joined #openstack-ironic03:01
openstackgerritTan Lin proposed openstack/ironic: Trusted boot doc  https://review.openstack.org/20843803:06
*** getvasanth has quit IRC03:11
*** dims has quit IRC03:23
*** tideline has quit IRC03:25
*** tideline has joined #openstack-ironic03:26
*** BadCub has quit IRC03:27
*** meghal1 has quit IRC03:27
*** meghal has joined #openstack-ironic03:28
*** comstud has quit IRC03:28
*** jroll has quit IRC03:28
*** mgagne has quit IRC03:28
*** natorious has quit IRC03:28
*** wendar has quit IRC03:28
*** JayF has quit IRC03:28
*** wendar_ has joined #openstack-ironic03:28
*** eglute has quit IRC03:28
*** BadCub has joined #openstack-ironic03:31
*** eglute has joined #openstack-ironic03:31
*** BadCub has quit IRC03:31
*** natorious has joined #openstack-ironic03:31
*** jroll has joined #openstack-ironic03:32
*** praneshp has joined #openstack-ironic03:32
*** mgagne has joined #openstack-ironic03:34
*** praneshp has quit IRC03:37
*** comstud has joined #openstack-ironic03:37
*** JayF has joined #openstack-ironic03:37
openstackgerritDebayan Ray proposed openstack/ironic: Firmware update as part of iLO zapping process  https://review.openstack.org/21101903:37
*** achanda has quit IRC03:41
*** natorious is now known as zz_natorious03:44
*** Nisha has joined #openstack-ironic03:46
*** raghu has quit IRC03:54
*** getvasanth has joined #openstack-ironic03:54
*** baoli has quit IRC03:57
*** zz_natorious is now known as natorious03:59
*** getvasanth has quit IRC04:07
*** natorious is now known as zz_natorious04:09
*** links has joined #openstack-ironic04:09
*** ramineni has joined #openstack-ironic04:21
*** BadCub has joined #openstack-ironic04:32
*** amotoki has joined #openstack-ironic04:33
*** puranamr has joined #openstack-ironic04:37
*** PradeepV has joined #openstack-ironic04:40
*** puranamr has quit IRC04:44
*** meghal has quit IRC04:55
*** Nisha has quit IRC05:05
*** meghal has joined #openstack-ironic05:07
*** Nisha has joined #openstack-ironic05:09
*** valkyrja has quit IRC05:12
*** valkyrja has joined #openstack-ironic05:14
*** getvasanth has joined #openstack-ironic05:17
openstackgerritTan Lin proposed openstack/ironic: Use oslo.versionedobjects remotable decorators  https://review.openstack.org/21700005:19
openstackgerritTan Lin proposed openstack/ironic: Using ObjectRegistry to register class  https://review.openstack.org/21758405:19
*** lazy_prince has joined #openstack-ironic05:21
getvasanthGood Morning All05:32
*** Nisha has quit IRC05:36
*** Nisha has joined #openstack-ironic05:37
getvasanthcan any one help me to know how ironic makes aware of node information to ironic05:37
getvasanthcan any one help me to know how ironic makes aware of node information to nova05:38
Haomenggetvasanth: morning05:39
Haomenggetvasanth: there is nova ironic driver, which will poll the node status05:40
getvasanthHaomeng: Good Morning05:40
Haomenggetvasanth: :)05:40
Haomenggetvasanth: you can check nova compute log to get the informations05:40
*** meghal has quit IRC05:41
getvasanthHaomeng: having a trouble, when i try to boot through nova boot, it fails with "No valid host found", further enabling debug logs, "ironic, 9833d3ad-a282-4c04-9615-a9aa19e28153) ram:-512 disk:0 io_ops:0 instances:0 does not have 2048 MB usable ram, it only has -512.0 MB usable ram. There are 0 hosts available but 1 instances requested to build."05:42
*** meghal has joined #openstack-ironic05:42
getvasanthHaomeng: but the ironic node list show that it has 64GB ram etc..05:42
Haomenggetvasanth: yes, so we can check by nova hypervisor-stats to see the host resource05:43
getvasanthHaomeng: trying to figure out how to make nova aware of it05:43
Haomenggetvasanth: can you show the result of "nova hypervisor-stats" call05:43
getvasanthHaomeng: sure05:44
Haomenggetvasanth: and "ironic node-show $IRONIC_NODE_ID |grep properties"05:44
*** getvasanth__ has joined #openstack-ironic05:45
getvasanth__Haomeng: http://pastebin.ca/313297805:46
Haomenggetvasanth__: ok05:46
HaomengHaomeng: all values are 005:48
getvasanth__Haomeng: yes, all are 005:48
*** getvasanth has quit IRC05:48
HaomengHaomeng: and how about ironic side, try run "ironic node-show $IRONIC_NODE_ID |grep properties"05:48
Haomenggetvasanth__: to see if the prpoerties is correct or not and what the ironic node status is05:49
getvasanth__Haomeng: http://pastebin.ca/313297905:49
PradeepVHi Ironic,05:49
PradeepVGood morning05:49
HaomengPradeepV: good morning:)05:50
Haomenggetvasanth__: I guess you deployed and failed, right, so check nova list05:50
PradeepVHi Haomeng: :) ... how can we generate a whole disk image in ISO format05:50
Haomenggetvasanth__: and try to delete the nova instance05:50
HaomengPradeepV: iso?05:50
PradeepVyes05:50
PradeepViso05:51
HaomengPradeepV: why iso?05:51
PradeepVi need to inject a specific driver in my whole disk image05:51
HaomengPradeepV: for configuration drive?05:52
PradeepVyes Haomeng:05:52
getvasanth__Haomeng: done05:52
HaomengPradeepV: please check http://docs.openstack.org/developer/ironic/deploy/install-guide.html, it covers config drive05:52
Haomenggetvasanth__: run hypervisor-stats again05:52
Haomenggetvasanth__: to see if the resource is restored back05:52
PradeepVok Haomeng, Thank you05:53
Haomenggetvasanth__: ironic will keep the resource even nova instance is in fail status05:53
HaomengPradeepV: welcome05:53
*** Nisha has quit IRC05:53
getvasanth__HAomeng: its all the same again, count 1, rest others are 005:53
*** Nisha has joined #openstack-ironic05:53
Haomenggetvasanth__: did you monitor the nova compute log05:53
Haomenggetvasanth__: if there is something error05:54
Haomenggetvasanth__: and check ironic driver-list05:54
Haomenggetvasanth__: to see if ironic conductor works fine05:54
getvasanth__Haomeng: yes, there is a error, "2015-08-28 11:23:50.263 28848 ERROR nova.openstack.common.periodic_task [req-a0eab617-f449-480e-99b2-695ee3cf0af9 - - - - -] Error during ClusteredComputeManager.update_available_resource: HTTPResponse instance has no attribute 'text'05:55
getvasanth__"05:55
Haomenggetvasanth__: yes, this should be root cause05:55
Haomenggetvasanth__: can you check ironic conductor log05:56
Haomenggetvasanth__: how did you install ironic, via devstack?05:56
Haomenggetvasanth__: did it work before?05:56
getvasanth__I used packstack05:56
getvasanth__Haomeng: I used packstack05:56
Haomenggetvasanth__: sorry, not sure if packstack supports devstack as well05:56
Haomenggetvasanth__: we suggest to run with devstack05:57
getvasanth__Haomeng: this is the first time i am trying ironic05:57
Haomenggetvasanth__: devstack is openstack official deployment tool, which should working05:57
Haomenggetvasanth__: for your " instance has no attribute 'text'" error, there should be some code which not matching I think05:57
Haomenggetvasanth__: or other root cause05:58
Haomenggetvasanth__: so can you follow our iornic official guide to try with devstack ?05:58
Haomenggetvasanth__: this can help you - http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html#deploying-ironic-with-devstack05:58
getvasanth__Haomeng: Ok, will try that too, but is there big difference in using packstack and devstack?05:59
Haomenggetvasanth__: I think if youjust want to play with openstack, there is no differenc between devstack and packstack, because we just want to deploy a openstack dev env, right?06:00
getvasanth__Haomeng: yes.06:00
*** Marga_ has joined #openstack-ironic06:00
*** Marga_ has quit IRC06:00
Haomenggetvasanth__: not sure if packstack supports ironic configurations as well, devstack support ironic as well06:01
getvasanth__Haomeng: Packstack supports ironic06:01
*** Marga_ has joined #openstack-ironic06:01
Haomenggetvasanth__: cool06:01
Haomenggetvasanth__: but after the installation, ironic should work06:01
Haomenggetvasanth__: if you dont care devstack/packstack, try with devstack, it works fine in ubuntu os06:02
getvasanth__Haomeng, it is, but fails to start the instance.06:02
Haomenggetvasanth__: and what os you are runing? redhat?06:02
getvasanth__Haomeng: rhel7.106:02
Haomenggetvasanth__: not sure if devstack has been tested in rhel71, community prefer ubuntu 1406:03
getvasanth__Haomeng: oh my god!! big change to be made.06:03
Haomenggetvasanth__: yes, if you run with rhel, suggest to use redhat's packstack, it should support redhat better than devstack I think06:04
getvasanth__Haomeng: so do you think, anything else i am missing, in my config?06:04
Haomenggetvasanth__: sorry, no idea about packstack, maybe nova ironic driver and ironic client version not matching06:05
Haomenggetvasanth__: from ironic team, maybe some redhat guys who did try with packstack, and they can help you06:06
getvasanth__Haomeng: here is the version info for ironic "0.3.1"06:06
getvasanth__Haomeng: here is the version info for ironic "0.3.1" client06:06
Haomenggetvasanth__: packstack will run with latest openstack code?06:06
getvasanth__Haomeng: created a repo for the kilo release and thats what is being used. not through the code.06:07
Haomenggetvasanth__: 0.3.1?06:07
Haomenggetvasanth__: it is old version I think06:07
getvasanth__Haomeng: ironic client version06:07
getvasanth__Haomeng: Ok i will try to update it to the newer one06:08
Haomenggetvasanth__: latest is 0.7.1 - https://pypi.python.org/pypi/python-ironicclient06:09
getvasanth__Haomeng: saw that now, will update it06:09
Haomenggetvasanth__: but not sure how packstack check same kilo version for both client and server code06:09
Haomenggetvasanth__: yes, if you did not care version, use latest code06:09
getvasanth__Haomeng: yes, will give a try06:09
Haomenggetvasanth__: for kilo version issue, it is hard to check and fix06:09
Haomenggetvasanth__: ok, good luck06:10
getvasanth__Haomeng: thanks a lot for the pointer06:10
Haomenggetvasanth__: welcome:)06:12
getvasanth__Haomeng: :)06:12
Haomenggetvasanth__: :)06:12
*** puranamr has joined #openstack-ironic06:15
*** Nisha has quit IRC06:18
*** yog_ has joined #openstack-ironic06:22
*** puranamr has quit IRC06:28
*** dlpartain has joined #openstack-ironic06:37
*** dlpartain has left #openstack-ironic06:38
getvasanth__Haomeng: are you there?06:47
*** dtantsur|afk is now known as dtantsur06:58
dtantsurMorning Ironic, TGIF!06:58
Haomenggetvasanth__: yes06:58
*** ifarkas has joined #openstack-ironic06:59
getvasanth__Haomeng: thanks, I have updated only the ironic client, and now the error vanished :)06:59
getvasanth__Haomeng: but the instance state is still in spawning06:59
Haomenggetvasanth__: cool06:59
Haomenggetvasanth__: so that is the version mismatch issue07:00
Haomengdtantsur: morning:)07:00
getvasanth__Haomeng: "2015-08-28 12:29:57.946 9745 DEBUG nova.virt.ironic.driver [-] [instance: a376070d-4110-4f09-bd81-eac5751bea1b] Still waiting for ironic node 9833d3ad-a282-4c04-9615-a9aa19e28153 to become ACTIVE: power_state="power on", target_power_state=None, provision_state="wait call-back", target_provision_state="active" "07:00
getvasanth__Haomeng: yes; thanks a lot for the pointers07:00
Haomenggetvasanth__: need some time07:00
getvasanth__Haomeng: "need some time"??07:01
Haomenggetvasanth__: yes, for deployment action07:01
Haomenggetvasanth__: it is waiting for callback from baremetal side07:02
getvasanth__Haomeng: oh!07:02
Haomenggetvasanth__: check the target side07:02
Haomenggetvasanth__: to see console output07:02
getvasanth__Haomeng: its booting..07:02
Haomenggetvasanth__: ok07:03
getvasanth__Haomeng: "system halted"07:04
*** achanda has joined #openstack-ironic07:05
getvasanth__Haomeng: get the screenshot07:05
*** dlpartain has joined #openstack-ironic07:05
*** dlpartain has left #openstack-ironic07:06
getvasanth__Haomeng: http://imagebin.ca/v/2DfblgH8gIqI07:07
Haomenggetvasanth__: what image did you used07:08
Haomenggetvasanth__: suggest to run dib to create07:08
*** meghal has quit IRC07:09
getvasanth__Haomeng: I created it centos707:09
getvasanth__Haomeng: looks like it failed too07:09
Haomenggetvasanth__: is this first boot?07:09
Haomenggetvasanth__: what driver did you use07:09
getvasanth__Haomeng: yes, pxe_ipmitool07:11
Haomenggetvasanth__: centos7 deploy image?07:11
getvasanth__Haomeng: do you have any images built, bootable?07:11
Haomenggetvasanth__: we should folow the guide to create deploy image07:12
getvasanth__Haomeng: i think i need to rebuild the image and try again07:12
getvasanth__Haomeng: i will rebuild the image and test it again, will ping you once done07:14
Haomenggetvasanth__: ok07:15
openstackgerritHarsh Shah proposed openstack/python-ironicclient: [WIP] Add keyring support  https://review.openstack.org/21813207:26
*** Marga_ has quit IRC07:34
*** Marga_ has joined #openstack-ironic07:35
*** mgoddard has joined #openstack-ironic07:45
openstackgerritDmitry Tantsur proposed openstack/ironic: Add deprecation warning to periodic tasks with parallel=False  https://review.openstack.org/21660407:47
*** ndipanov has joined #openstack-ironic07:51
*** ndipanov has quit IRC07:52
*** ndipanov has joined #openstack-ironic07:52
*** achanda has quit IRC07:56
*** stendulker has joined #openstack-ironic07:56
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Pass -w flag to iptables to make it wait for xtables lock  https://review.openstack.org/21672608:02
*** dlpartain has joined #openstack-ironic08:02
*** leopoldj has joined #openstack-ironic08:04
openstackgerritMerged openstack/ironic-inspector: Rename plugin root_device_hint -> raid_device  https://review.openstack.org/21717108:04
openstackgerritMerged openstack/ironic-inspector: Store and expose introspection data  https://review.openstack.org/21315908:05
openstackgerritTan Lin proposed openstack/ironic: Validata input of property  https://review.openstack.org/21550508:05
openstackgerritTan Lin proposed openstack/ironic: Validata input of property  https://review.openstack.org/21550508:06
*** derekh has joined #openstack-ironic08:10
*** jistr has joined #openstack-ironic08:12
*** viktors|afk is now known as viktors08:16
openstackgerritTan Lin proposed openstack/ironic: Validate the input of properties  https://review.openstack.org/21550508:23
*** lucasagomes has joined #openstack-ironic08:25
openstackgerritShivanand Tendulker proposed openstack/ironic: Ensure that pass_deploy_info() always calls boot.prepare_instance()  https://review.openstack.org/21675608:27
*** dlpartain has quit IRC08:41
*** dlpartain has joined #openstack-ironic08:41
*** dlpartain has quit IRC08:46
*** dlpartain has joined #openstack-ironic08:46
*** amit213 has quit IRC08:50
*** amit213 has joined #openstack-ironic08:51
*** e0ne has joined #openstack-ironic08:54
aarefievdtantsur: hi, could you take a look when you have time, please: https://review.openstack.org/#/c/217135/08:57
dtantsurhi, when I have some time..08:58
aarefievdtantsur: thanks08:58
*** marzif has joined #openstack-ironic08:59
*** degorenko has quit IRC09:01
*** VikasC has joined #openstack-ironic09:01
*** e0ne has quit IRC09:03
*** pelix has joined #openstack-ironic09:06
openstackgerritMerged openstack/ironic: Put py34 first in the env order of tox  https://review.openstack.org/21728309:07
*** deray has joined #openstack-ironic09:07
openstackgerritMerged openstack/ironic: Fix logging and exceptions messages in ipminative driver  https://review.openstack.org/21707109:08
*** marzif has quit IRC09:12
*** marzif has joined #openstack-ironic09:12
*** amotoki has quit IRC09:25
*** e0ne has joined #openstack-ironic09:25
openstackgerritSatoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes  https://review.openstack.org/20049609:29
*** degorenko has joined #openstack-ironic09:30
*** degorenko has quit IRC09:39
*** marzif has quit IRC09:41
*** degorenko has joined #openstack-ironic09:44
*** chenglch has quit IRC09:45
*** yuikotakada has quit IRC09:57
*** MattMan has quit IRC10:00
*** MattMan has joined #openstack-ironic10:01
rcarrillocruzhmm10:16
rcarrillocruzTheJulia: when you are around, i have a problem on bifrost provisioning i detected i would like to discuss with you10:17
sambettsMorning everyone TGIF :D10:23
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837510:25
dtantsursambetts, morning, and yeah, super TGIF!10:25
*** karimb has joined #openstack-ironic10:27
*** mgagne has quit IRC10:27
*** wendar_ has quit IRC10:27
*** hakimo has quit IRC10:27
*** purp has quit IRC10:27
*** adam_g has quit IRC10:27
*** jamielennox has quit IRC10:27
*** vishwanathj has quit IRC10:27
*** Ng has quit IRC10:27
*** persia has quit IRC10:27
*** early has quit IRC10:27
*** d0ugal has quit IRC10:27
*** marios has quit IRC10:27
*** yuriyz has quit IRC10:27
*** dguerri` has quit IRC10:27
*** pleia2 has quit IRC10:27
*** Marga_ has quit IRC10:27
*** valkyrja has quit IRC10:27
*** BadCub has quit IRC10:27
*** links has quit IRC10:27
*** comstud has quit IRC10:27
*** jroll has quit IRC10:27
*** zz_natorious has quit IRC10:27
*** eglute has quit IRC10:27
*** davideagnello has quit IRC10:27
*** tteggel_ has quit IRC10:27
*** krotscheck has quit IRC10:27
*** anteaya has quit IRC10:27
sambettsdtantsur: Has it started cooling down where you are? or is it still uncomfortably hot?10:27
dtantsursambetts, it did start cooling down, and now it's starting to be hot again >_< +32 this weekend10:28
lucasagomesdamn10:28
sambettsdtantsur: woah?! Thats insane, we've had a week of storms and heavy rain here10:29
*** Marga_ has joined #openstack-ironic10:29
*** valkyrja has joined #openstack-ironic10:29
*** BadCub has joined #openstack-ironic10:29
*** links has joined #openstack-ironic10:29
*** comstud has joined #openstack-ironic10:29
*** jroll has joined #openstack-ironic10:29
*** zz_natorious has joined #openstack-ironic10:29
*** eglute has joined #openstack-ironic10:29
*** davideagnello has joined #openstack-ironic10:29
*** tteggel_ has joined #openstack-ironic10:29
*** krotscheck has joined #openstack-ironic10:29
*** anteaya has joined #openstack-ironic10:29
*** mgagne has joined #openstack-ironic10:30
*** wendar_ has joined #openstack-ironic10:30
*** hakimo has joined #openstack-ironic10:30
*** purp has joined #openstack-ironic10:30
*** adam_g has joined #openstack-ironic10:30
*** jamielennox has joined #openstack-ironic10:30
*** vishwanathj has joined #openstack-ironic10:30
*** Ng has joined #openstack-ironic10:30
*** persia has joined #openstack-ironic10:30
*** early has joined #openstack-ironic10:30
*** d0ugal has joined #openstack-ironic10:30
*** marios has joined #openstack-ironic10:30
*** yuriyz has joined #openstack-ironic10:30
*** dguerri` has joined #openstack-ironic10:30
*** pleia2 has joined #openstack-ironic10:30
*** mgagne has quit IRC10:30
*** wendar_ has quit IRC10:30
*** hakimo has quit IRC10:30
*** purp has quit IRC10:30
*** adam_g has quit IRC10:30
*** jamielennox has quit IRC10:30
*** vishwanathj has quit IRC10:30
*** Ng has quit IRC10:30
*** persia has quit IRC10:30
*** early has quit IRC10:30
*** d0ugal has quit IRC10:30
*** marios has quit IRC10:30
*** yuriyz has quit IRC10:30
*** dguerri` has quit IRC10:30
*** pleia2 has quit IRC10:30
*** mgagne has joined #openstack-ironic10:32
*** wendar_ has joined #openstack-ironic10:32
*** hakimo has joined #openstack-ironic10:32
*** purp has joined #openstack-ironic10:32
*** adam_g has joined #openstack-ironic10:32
*** jamielennox has joined #openstack-ironic10:32
*** vishwanathj has joined #openstack-ironic10:32
*** Ng has joined #openstack-ironic10:32
*** persia has joined #openstack-ironic10:32
*** early has joined #openstack-ironic10:32
*** d0ugal has joined #openstack-ironic10:32
*** marios has joined #openstack-ironic10:32
*** yuriyz has joined #openstack-ironic10:32
*** dguerri` has joined #openstack-ironic10:32
*** pleia2 has joined #openstack-ironic10:32
*** shoutm has joined #openstack-ironic10:34
*** VikasC has quit IRC10:34
*** zhenguo has quit IRC10:40
openstackgerritVladyslav Drok proposed openstack/bifrost: Fix RST warnings in README.rst  https://review.openstack.org/21820010:42
*** dtantsur is now known as dtantsur|brb10:43
*** lazy_prince has quit IRC10:47
*** Marga_ has quit IRC10:48
*** stendulker has quit IRC10:51
*** dimsum__ has joined #openstack-ironic10:53
*** karimb has quit IRC10:55
*** deray has quit IRC10:55
*** ionutbalutoiu has joined #openstack-ironic10:59
*** ramineni has quit IRC11:02
*** getvasanth__ has quit IRC11:02
openstackgerritMerged openstack/bifrost: Correct bug with default value where required  https://review.openstack.org/21710911:14
openstackgerritMerged openstack/bifrost: Resync with openstack/requirements  https://review.openstack.org/21668111:17
openstackgerritMerged openstack/bifrost: Support for fine-grained distro-related defaults  https://review.openstack.org/21735011:17
*** dlpartain has left #openstack-ironic11:20
*** dprince has joined #openstack-ironic11:30
*** marzif has joined #openstack-ironic11:32
*** PradeepV has quit IRC11:34
*** dtantsur|brb is now known as dtantsur11:34
sambettsdtantsur|brb: I'm just looking at https://review.openstack.org/#/c/212012 and I can't decide whether we need to define __all__ inside client.py, python will only expose not _'d functions, so all the private functions should remain hidden11:36
sambettsis there anything your trying to hide by overriding the defaults with __all__ ?11:36
dtantsursambetts, LOG and all imported modules11:38
dtantsurlucasagomes, did I show you https://github.com/dtantsur/metalsmith ?11:39
lucasagomesdtantsur, hey, not really11:39
* lucasagomes looks now11:39
lucasagomeshah nice one11:39
dtantsur:)11:39
*** lucasagomes is now known as lucas-hungry11:48
TheJuliarcarrillocruz: whats up?11:49
rcarrillocruzso11:50
rcarrillocruzmy servers11:50
rcarrillocruzhave one nic that i set up for static provisioning11:51
rcarrillocruzerm static ip11:51
TheJuliaOkay11:51
rcarrillocruzlet's say the management nic11:51
rcarrillocruzanother one i use for provisioning, which use dhcp/pxe11:51
rcarrillocruzi' m hitting this issue where at times i get servers having the default gw set to the DHCP server IP11:51
rcarrillocruzthat apparently is the default behaviour on dnsmasq per the conf file, it will push the default route as the dns server ip11:52
rcarrillocruzso what i think is happening11:52
rcarrillocruzif glean runs on the nic that does the provisioning after the nic where i want the management ip, it will configure it and pull the default route11:52
rcarrillocruzthus overriding the default route i want11:53
rcarrillocruzthe workaround i did is:11:53
rcarrillocruz(sec, copy pasting on paste.openstack.org)11:53
TheJuliano rush, I'm still asleep11:53
rcarrillocruzhttp://paste.openstack.org/show/430840/11:55
rcarrillocruzthat on dnsmasq.conf.j211:55
rcarrillocruzessentially, also forcing the DHCP server to use the same gateway you put on vars11:55
rcarrillocruznot sure if the right approach, it fits my use case unsure about others, prior to pushing it as a patch11:56
*** trown|outttypeww is now known as trown11:57
*** dlpartain1 has joined #openstack-ironic12:00
TheJuliarcarrillocruz: Please feel free to submit a patch and a bug.  It seems like its a bug in glean as well that needs to be reviewed and addressed.  Pondering your use case,  your still in the realm of "I want to bootstrap a pool of hardware", it is just that your network architecture is a little different and has raised a race in glean.12:01
openstackgerritJohn Trowbridge proposed openstack/ironic-inspector: Add logging to intropspection data storage  https://review.openstack.org/21823212:05
TheJuliarcarrillocruz: I'll take a look at glean, likely this weekend if time permits12:05
dtantsurtrown, hi, typo "intropspection" ^^12:05
trowndtantsur: :)...I really should not submit anything before my second cup of coffee :p12:07
dtantsurhehe12:07
TheJuliaThis reminds me, time to go find coffee12:07
openstackgerritJohn Trowbridge proposed openstack/ironic-inspector: Add logging to introspection data storage  https://review.openstack.org/21823212:08
*** ukalifon has joined #openstack-ironic12:13
*** Haomeng has quit IRC12:17
*** naohirot has quit IRC12:22
*** thiagop has joined #openstack-ironic12:22
thiagopGood morning Ironic12:23
*** dlpartain1 has left #openstack-ironic12:25
sinvalgood morning thiagop and Ironicers12:25
*** Marga_ has joined #openstack-ironic12:29
*** chlong has joined #openstack-ironic12:34
*** alexpilotti has joined #openstack-ironic12:35
*** trown is now known as trown|brb12:36
thiagoplucas-hungry dtantsur yuriyz NobodyCam morning! What do we need to put the OneView driver in the Drivers wiki?12:40
thiagopis it necessary to merge it first or not?12:40
dtantsurI would say, yes12:40
yuriyz+112:40
thiagopok then ;)12:42
dtantsurI was granted a Japanese visa \o/12:44
*** lucas-hungry is now known as lucasagomes12:44
dtantsurnow I have to go to Prague to pick it up :)12:44
*** mgoddard has left #openstack-ironic12:45
thiagopgo to Prague is on my "100 things to do before die" list12:46
lucasagomesdtantsur, w00t!12:46
dtantsurthiagop, it's worth doing!12:46
lucasagomesTheJulia, thiagop yuriyz trown|brb morning12:46
lucasagomesprague is awesome indeed12:46
sambettsdtantsur: w00p!12:50
*** jcoufal has joined #openstack-ironic12:53
openstackgerritMerged openstack/python-ironic-inspector-client: Make sure we expose all API elements in the top-level package  https://review.openstack.org/21201212:53
*** baoli has joined #openstack-ironic12:54
*** marzif has quit IRC13:00
*** baoli has quit IRC13:01
*** baoli has joined #openstack-ironic13:02
*** ukalifon has quit IRC13:03
*** thiagop_ has joined #openstack-ironic13:04
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/21656213:13
*** kbyrne has quit IRC13:13
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837513:13
dtantsurJoshNang, hi! If you have some time, could you please review IPA patch for inspection? https://review.openstack.org/#/c/205587/13:15
*** mgoddard has joined #openstack-ironic13:17
*** nicodemos has joined #openstack-ironic13:17
*** kbyrne has joined #openstack-ironic13:19
*** kbyrne has quit IRC13:20
*** kbyrne has joined #openstack-ironic13:21
dtantsurifarkas, hi! could you please review https://review.openstack.org/#/c/216726 ? It's a pretty nasty bug13:21
ifarkasdtantsur, sure, will do13:21
*** leopoldj has quit IRC13:24
*** trown|brb is now known as trown13:26
*** links has quit IRC13:29
*** amotoki has joined #openstack-ironic13:32
*** puranamr has joined #openstack-ironic13:33
*** dlpartain has joined #openstack-ironic13:35
*** dlpartain has left #openstack-ironic13:36
*** puranamr has quit IRC13:39
*** garthb has joined #openstack-ironic13:45
NobodyCamgood morning dtantsur lucasagomes sambetts jlvillal thiagop TheJulia JayF JoshNang BadCub13:46
*** bnemec has joined #openstack-ironic13:46
dtantsurNobodyCam, morning, TGIIIIIIF!13:46
lucasagomesNobodyCam, hey hey hey! Good ugt morning13:46
NobodyCamdt oh ya BIG time TGIF13:47
NobodyCamhey lucasagomes :) morning :)13:47
thiagop_morning NobodyCam13:47
NobodyCamdt== dtantsur + tab fail13:47
NobodyCam:-p13:47
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Support ironic root device hints  https://review.openstack.org/21827113:48
thiagop_TGIF!13:48
NobodyCamya :)13:49
*** dimsum__ has quit IRC13:49
*** dimsum__ has joined #openstack-ironic13:50
*** degorenko has quit IRC13:52
*** degorenko has joined #openstack-ironic13:56
*** dlpartain1 has joined #openstack-ironic13:57
*** smoriya has quit IRC13:59
*** dlpartain1 has quit IRC14:00
*** dlpartain has joined #openstack-ironic14:00
*** dprince has quit IRC14:01
*** dlpartain has left #openstack-ironic14:03
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837514:06
openstackgerritMerged openstack/ironic-inspector: Pass -w flag to iptables to make it wait for xtables lock  https://review.openstack.org/21672614:09
NobodyCamTheJulia: are you around?14:09
trowndtantsur: I am working on reviewing that rules patch, it is a big one :)14:14
dtantsurtrown, heh yeah :)14:15
dtantsurit took me several weeks to finish (and it still requires some follow up work)14:15
dtantsursambetts, UCS nodes support IPMI as well, right?14:19
*** yog_ has quit IRC14:19
sambettsdtantsur: They do, but in some more recent models its actually disabled by default14:22
sambettsdtantsur: and its not needed if you can talk to BMC directly14:22
*** zhenguo has joined #openstack-ironic14:22
dtantsursambetts, yeah, I'm rather interested in if we should support ucs_address as a valid ipmi_address in inspector14:23
sambettsdtantsur: When IPMI is enabled then you use the same address to talk that or the http api so I think it could be added there14:24
dtantsurcool, I'm adding it14:25
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Do not overwrite the iPXE boot script on every deployment  https://review.openstack.org/21829014:25
sambettsdtantsur: I'm just thinking that ucs_address isn't actually the address I'm thinking of, because the current ucs driver which is the one that implements ucs_address doesn't talk to the BMC/IPMI endpoint it talks to UCSM which is a centralised management server14:27
sambettsIn CIMC driver that I've got a spec cimc_address is the IPMI endpoint14:27
dtantsursambetts, ah, I see. So it's not like ilo/drac where special protocol address is the same as BMC address, right?14:28
sambettsdtantsur: It is in the CIMC case https://review.openstack.org/#/c/217598, but in the UCSM case its not14:29
dtantsursambetts, cool, thanks for clarifying14:29
sambettsalso wasn't there a spec/wish_list bug for standardising all the *_address *_username *_password fields ??14:29
sambettslucasagomes, dtantsur ^14:29
* lucasagomes reads14:30
dtantsursambetts, there was a bug, I guess, but nobody volunteered..14:30
dtantsurit's a big PITA for inspector though, I agree...14:30
lucasagomessambetts, not sure if we have a spec to standardize it, but I think most of us agree with the idea14:31
lucasagomessambetts, and yeah, cimc_address for this case sounds good to me14:31
*** mgoddard1 has joined #openstack-ironic14:32
sambettslucasagomes: I'm sure I saw a patch to do that standardising... maybe I dreamt it :-P eletric sheep and all that haha14:32
*** mgoddard has quit IRC14:33
krtaylorgood UGT morning all14:34
dtantsurkrtaylor, o/14:34
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Make list of ipmi_address-alike driver fields configurable  https://review.openstack.org/21830414:36
dtantsursambetts, suggestions are welcome ^^14:36
trowndo we still support python2.6-? i.e. can we remove discover from test-requirements? https://github.com/openstack/ironic/blob/master/test-requirements.txt14:37
lucasagomessambetts, hah maybe!14:37
trownI can submit a patch to remove it14:37
lucasagomestrown, we don't know14:37
*** dprince has joined #openstack-ironic14:37
lucasagomeswe don't no*14:37
trownlucasagomes: thanks14:37
lucasagomesI think the python-ironiclcient still does, but Ironic itself doesn't14:38
openstackgerritDmitry Tantsur proposed openstack/ironic: Add deprecation warning to periodic tasks with parallel=False  https://review.openstack.org/21660414:39
dtantsurwill appreciate reviews, we need to get it in liberty ^^14:39
thiagop_morning krtaylor14:40
openstackgerritSerge Kovaleff proposed openstack/ironic: Use six.text_type in parse_image_ref  https://review.openstack.org/21831114:41
openstackgerritJohn Trowbridge proposed openstack/ironic: Remove unused dep discover from test reqs  https://review.openstack.org/21831314:42
*** mtanino has joined #openstack-ironic14:44
*** kbyrne has quit IRC14:45
*** degorenko has quit IRC14:51
*** thiagop_ has quit IRC14:54
dtantsursambetts, I'm sorry, I probably didn't get your comment, it seems to me that test_additional_fields does what you're requesting: https://review.openstack.org/#/c/218304/1/ironic_inspector/test/test_utils.py14:54
*** thiagop_ has joined #openstack-ironic14:54
*** meghal has joined #openstack-ironic14:56
*** kbyrne has joined #openstack-ironic14:57
*** meghal1 has joined #openstack-ironic14:58
sambettsdtantsur: Yup your right, I must be tired, I read that as a test checking for a node with a field that didn't match, no idea why ... but maybe thats a test that should be added to ensure it returns None14:58
*** meghal has quit IRC15:00
openstackgerritLilia Sampaio proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279615:02
*** mtanino has quit IRC15:03
*** mgoddard1 has quit IRC15:06
*** garthb has quit IRC15:06
*** mgoddard has joined #openstack-ironic15:07
*** viktors has left #openstack-ironic15:13
*** boris-42 has joined #openstack-ironic15:13
*** zhenguo has quit IRC15:14
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837515:16
*** romcheg has quit IRC15:17
*** romcheg has joined #openstack-ironic15:18
*** romcheg has quit IRC15:19
*** gabriel-bezerra is now known as gabrielb15:19
*** gabrielb is now known as gabriel-bezerra15:20
*** gabriel-bezerra is now known as gbezerra15:22
*** gbezerra is now known as gabezerra15:22
*** gabezerra is now known as gbezerra_15:23
*** gbezerra_ is now known as gabrielb15:23
openstackgerritMerged openstack/python-ironic-inspector-client: Drop comment about changing functional tests to use released inspector  https://review.openstack.org/21200715:23
*** enikanorov has quit IRC15:23
*** gabrielb is now known as gabriel-bezerra15:23
*** enikanorov has joined #openstack-ironic15:25
*** enikanorov has quit IRC15:26
*** tideline has left #openstack-ironic15:29
*** tideline has joined #openstack-ironic15:29
*** puranamr has joined #openstack-ironic15:32
*** enikanorov has joined #openstack-ironic15:33
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Fix logic for internal server error  https://review.openstack.org/21835815:34
*** romcheg has joined #openstack-ironic15:34
*** zz_natorious is now known as natorious15:35
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Fix logic for internal server error  https://review.openstack.org/21835815:37
*** trown is now known as trown|lunch15:37
*** chlong has quit IRC15:39
openstackgerritMerged openstack/bifrost: Cleanup role meta definitions  https://review.openstack.org/21690515:40
openstackgerritVladyslav Drok proposed openstack/ironic: Reset tempdir config option after NestedTempfile fixture applied  https://review.openstack.org/21836315:46
*** mtanino has joined #openstack-ironic15:46
openstackgerritVladyslav Drok proposed openstack/ironic: Revert "Add cleanup in console utils tests"  https://review.openstack.org/21836815:50
openstackgerritVladyslav Drok proposed openstack/ironic: Revert "Add cleanup in console utils tests"  https://review.openstack.org/21836815:52
*** ionutbalutoiu has quit IRC15:54
*** shoutm has quit IRC15:54
*** shoutm has joined #openstack-ironic15:55
*** shoutm has quit IRC15:55
*** puranamr has quit IRC15:57
openstackgerritVladyslav Drok proposed openstack/ironic: Revert "Add cleanup in console utils tests"  https://review.openstack.org/21836816:02
*** mgoddard has quit IRC16:04
*** romcheg1 has joined #openstack-ironic16:05
*** mgoddard has joined #openstack-ironic16:05
*** romcheg has quit IRC16:05
*** linggao has joined #openstack-ironic16:08
dtantsurhave a great weekend everyone! I may be later for the Monday evening, as I have to go to Prague to get my passport back16:09
*** dtantsur is now known as dtantsur|afk16:10
*** romcheg1 has left #openstack-ironic16:11
jlvillaldtantsur|afk: ciao16:18
*** Marga_ has quit IRC16:20
*** chlong has joined #openstack-ironic16:22
*** garthb has joined #openstack-ironic16:23
*** alexpilotti has quit IRC16:23
NobodyCamhave a good weekend dtantsur|afk16:24
NobodyCamw00t: https://review.openstack.org/#/c/217129 approved. this should fix the dib ipa build16:24
JoshNangwoo!16:27
NobodyCammorning JoshNang :)16:27
*** e0ne has quit IRC16:28
JoshNanggood morning!16:29
NobodyCam:)16:29
jlvillalGood morning all16:30
*** alexpilotti has joined #openstack-ironic16:33
NobodyCammornign jlvillal :)16:33
*** jistr has quit IRC16:35
*** derekh has quit IRC16:36
*** alexpilotti has quit IRC16:38
*** penick has joined #openstack-ironic16:41
*** dprince has quit IRC16:41
thiagop_morning jlvillal16:41
NobodyCammorign thiagop_16:42
NobodyCam:)16:42
thiagop_morning NobodyCam ;016:42
thiagop_;)16:42
NobodyCam:)16:42
jlvillalthiagop: Bom Dia!16:44
*** chlong has quit IRC16:45
thiagop_:D16:45
JayFjlvillal: lekha had already done some work w/mocking out the Ironic API using Mimic for a separate project we have (http://github.com/rackerlabs/arsenal) -- I forwarded her your email and said you might be interested in seeing the work she's done16:46
JayFjlvillal: probably a huge head start on functional testing for the client at a minimum16:46
jlvillalJayF: Awesomesauce!  Learned that is an official word ;)16:47
JayFjlvillal: also generally appreciated the "call for interested parties" email, we should do that more16:47
jlvillal:)16:47
lekhajlvillal: hihi16:48
jlvillalSpeaking of which.  krtaylor Did you see my email about functional testing in Ironic?16:48
jlvillallekha: Hello16:48
*** thiagop_ has quit IRC16:48
lekhajlvillal: https://github.com/rackerlabs/arsenal/pull/60 is a PR that begins to do functional testing for Arsenal16:49
sambettsHave a good weekend ironic, see everyone next week :D16:49
lekhajlvillal: also, I am yet to release the latest verison on mimic, hence the travis failure16:49
jlvillalsambetts: Ciao16:50
jlvillallekha: Thanks.16:50
jlvillallekha: Let me look :)16:50
lekhajlvillal: sure, feel free to ping me if you have questions16:51
jlvillallekha: Will do. Thank you very much.16:51
lekhajlvillal:16:51
lekhajlvillal: wc :)16:51
* jlvillal thinks WC first...16:52
jlvillal:D16:52
* jlvillal has traveled too much in europe...16:53
*** alexpilotti has joined #openstack-ironic16:53
*** Marga_ has joined #openstack-ironic16:56
*** enikanorov has quit IRC16:56
openstackgerritJosh Gachnang proposed openstack/ironic: Clean up cleaning error handling on heartbeats  https://review.openstack.org/21376916:57
*** enikanorov has joined #openstack-ironic16:57
*** alexpilotti has quit IRC16:57
JayFlekha: in Mimic, there's no concept of a plugin system or such? all the Ironic support is just "mixed in"?16:58
lekhaJayF: there is a plugin system for services listed within the service catalog, we don't have anything for the others yet16:59
*** dprince has joined #openstack-ironic16:59
*** mtanino has quit IRC17:00
lucasagomesfolks I'm calling it a day17:01
lucasagomesJayF, lekha jlvillal goot ugt morning :-)17:01
lucasagomeshave a great night everyone! Enjoy the weekend17:01
thiagopgood night & weekend lucasagomes17:01
lucasagomescheers! you too17:01
*** lucasagomes is now known as lucas-beer17:01
JayFnight17:02
thiagopgarthb: Hey17:03
garthbthiagop: hello17:03
NobodyCamnight lucas-beer17:03
garthbthiagop I've installed the python-oneviewclient and now I'm starting to see driver load exceptions in ir-cond17:03
*** ifarkas has quit IRC17:03
garthbthiagop exception is: DriverLoadError: Driver fake_oneview could not be loaded. Reason: coercing to Unicode: need string or buffer, NoneType found.17:04
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Refactor list_block_devices to its own function  https://review.openstack.org/21062617:04
openstackgerritHarsh Shah proposed openstack/python-ironicclient: [WIP] Add keyring support  https://review.openstack.org/21813217:04
thiagopgarthb: what? That is strange17:04
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837517:04
thiagopgarthb: paste the full stack for me, please17:05
garthbthiagop: sure...one moment17:05
krtaylorjlvillal, thanks for the ping, not I have not, checking now17:06
garthbthiagop http://paste.openstack.org/show/431134/17:06
garthbthiagop I'm adding fake_oneview, iscsi_pxe_oneview, and agent_pxe_oneview to the enabled_drivers in ironic.conf...I assume thats correct17:07
*** mjturek1 has quit IRC17:08
thiagopgarthb: yes, it is17:09
garthbthiagop good17:10
thiagopgarthb: can you paste your enabled_drivers line on ironic.conf?17:11
*** trown|lunch is now known as trown17:11
garthbthiagop enabled_drivers = fake,pxe_ssh,pxe_ipmitool,fake_oneview,iscsi_pxe_oneview,agent_pxe_oneview17:12
*** alexpilotti has joined #openstack-ironic17:13
krtaylorjlvillal, re: testing - yes, I'll help however I can, super busy at work right now with other things, but that won't stop me17:14
*** aarefiev22 has joined #openstack-ironic17:15
*** e0ne has joined #openstack-ironic17:18
*** penick has quit IRC17:19
jlvillalkrtaylor: That is what the weekends and the time between 1am - 4am is for! ;)17:22
jlvillalkrtaylor: Kidding of course. But any help/input you have would be much appreciated.17:22
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests  https://review.openstack.org/21473517:23
*** wendar_ is now known as wendar17:23
*** lucas-beer has quit IRC17:25
*** Marga_ has quit IRC17:26
krotscheckjlvillal: When you say "functional testing" do you mean "We're going to spin up ironic and run client tests against it" kind of testing?17:32
*** alexpilotti has quit IRC17:32
jlvillalkrotscheck: That's a good question. I'm not exactly sure where the cut-off between functional and integration testing is.  I am not a testing expert.17:33
jlvillalkrotscheck: So hoping to learn from others.17:33
*** e0ne has quit IRC17:33
krotscheckRighto.17:33
krotscheckWell, for me to do webclient gating things I need to do something similar.17:33
jlvillalCool :)17:33
krotscheckSo I'm curious about what you're doing from the Hey I'm a Javascript Client17:33
krotscheckperspective17:34
jlvillalI was planning on seeing how Nova is doing it for their client and base nova.17:34
jlvillalAlso I know dtantsur|afk has done a lot of functional testing with inspector.17:34
JoshNangjlvillal: the line is generally whether you're testing the functionality of one service or the integration of multiple services17:35
trownjlvillal: ya there is the devstack gate for inspector: https://github.com/openstack/ironic-inspector/blob/master/devstack/exercise.sh17:36
krotscheckJoshNang: Huhn. In the UI world, functional testing is the "Clicky Button" testing.17:36
trownjlvillal: that is the main functional test...there is also a tox env, but the devstack one covers much more17:36
jlvillalJoshNang: Thanks for the info. So functional testing should only test Ironic and not require any other services (if possible). If I understand correctly.17:36
JoshNangkrotscheck: if we had clicky buttons in ironic we'd test those too :D17:36
trownhmm...although I guess that is integration testing technically17:37
JayFjlvillal: that's part of why lekha's work to make mimic support Ironic API is so helpful. You have a fake Ironic API to functional test client features against17:37
trownI think there is some overlap in those venn diagrams17:37
*** romcheg has joined #openstack-ironic17:37
JayFjlvillal: and functional testing is esepcially valuable to Ironic b/c it's so damn heavyweight to do full integration tests17:37
jlvillalJayF: Ah, cool.17:37
JoshNangjlvillal: yup, as i understand at least. basically "does the api handle a response correctly and return proper values while causing proper side effects"17:37
krotscheckJoshNang: We have those in the client :-P17:38
*** alexpilotti has joined #openstack-ironic17:39
JoshNangkrotscheck: even the side effect bits? i'm especially concerned with side effects like: if you delete a node, it cleans up, spins up an agent, does some cleaning things, etc17:40
trownhave any of you guys looked at gabbi at all for API testing?17:40
trownI am writing some API tests in it, and it is pretty nice17:40
krotscheckJoshNang: Not yet, but eventually.17:40
JoshNang(obviously mocks the spinning up agent part)17:40
JoshNangkrotscheck: then \o/17:40
trownhttp://gabbi.readthedocs.org/en/latest/gabbi.html17:40
JayFtrown: that sounds *very* similar to mimic -> https://github.com/rackerlabs/mimic17:41
JoshNangtrown: hmm interesting17:41
JayFDespite the docs not being updated yet, the master version has some ironic support17:41
lekhajlvillal: I have mocked the list and get calls in ironic in mimic. I am going to be adding the add and delete nodes this weekend. bUt feel free to add issues for anything you might need17:42
jlvillallekha: Thank you. I think it will take some time to get up to speed. And unfortunately I have other work related activities to do :(  But I will try to look at it as soon as I can.17:43
*** alexpilotti has quit IRC17:43
lekhajlvillal: sure17:43
JoshNangJayF: i don't think those are at all similar. gabbi runs some html queries and expects some response, mimic acts as a fake service and returns specified response given an input (as i understand it)17:43
*** Marga_ has joined #openstack-ironic17:44
JayFJoshNang: oooh, so they're like opposite sides of a coin?17:44
JoshNangso like gabbi sends a request to ironic, ironic talks to mimic to mock out IPA/keystone/etc, ironic returns a response, gabbit makes sure the response is correct (as i understand)17:44
trownya, I am thinking they are not the same...gabbi can just run any wsgi app and you write tests against it17:44
JoshNangyeah :D17:44
JayFoh that's cool, igi17:44
JoshNanggood complements, IMO17:45
trownya17:45
*** praneshp has joined #openstack-ironic17:45
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Add some comparison operators to Serializable class  https://review.openstack.org/21585417:49
trownJayF or JoshNang , jroll is back next week?17:50
thiagopguys, I gotta go17:50
thiagophave a wonderful weekend everybody17:50
thiagopsee you on monday17:50
trownthe project-config patch for IPA merged17:50
JayFtrown: he's only scheduled to be vacationing this week17:50
trownhave a good weekend thiagop17:51
JayFtrown: I can only hope he returns :)17:51
trownlol17:51
*** cdearborn has joined #openstack-ironic17:51
sinvalgarthb: hey17:51
garthbsinval: hello17:52
*** thiagop has quit IRC17:52
trownJayF: I figured out how to create a Fedora package from a pre-release git snapshot, so I am moving forward with that for now, but it would be nice to rebase on an actual release17:52
trownif nothing else to not have such a crazy N-V-R for the package: (0.1.0-0.1gite6439ca1d06a50e72b97975da9b3963ba04ba841)17:53
JayFtrown: ah, it's the "we need to have releases for IPA" conversation17:54
trownya :)17:54
*** e0ne has joined #openstack-ironic17:55
trownJayF: it makes downstream packaging much nicer17:55
*** achanda has joined #openstack-ironic17:56
JayFPart of the reaosn I originally advocated for IPA not having releases17:56
JayFwas b/c I *didn't* want it packaged downstream17:56
JayFbut obviosuly since agent is default method for both popular deploy drivers that's likely not possible anymore :)17:56
JoshNangmaybe we should cut a release to coincide with ironic 4.0.0?17:57
JoshNangand certainly for whatever we release for liberty17:57
*** pelix has quit IRC17:57
JayF+017:58
JayFI'm not excited about the idea, but with the new ironic release model it makes sense to cut an ipa with them soonish as well17:58
*** alexpilotti has joined #openstack-ironic17:59
trownJayF: ya, kind of impossible to adopt it downstream without a package17:59
JayFSadly, I have no idea how to go about such a thing. heh.18:00
JayFThere's a doc somewhere that has been done at least partway18:00
JayFabout making it so IPA can release18:00
JayFwhelp, today was going to be a majority-upstream day anyway. Maybe I've found a thing to od other than reviewing code :)18:00
trownJayF: I think with the patch I got merged to project-config it is as simple as tagging a release in git18:02
JayFtrown: can you link me to that patch?18:03
*** Marga_ has quit IRC18:03
JayFtrown: I think other things are still missing; like pypi and such18:03
trownhttps://review.openstack.org/#/c/213749/18:03
JayFtrown: unless someone did all that work while I was gone18:03
trownJayF: IPA is set up on pypi, just 0 releases18:03
JayFokay cool, so I must have at least gotten that far :)18:03
*** mjturek1 has joined #openstack-ironic18:04
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Dispatch the call to erase_block_device  https://review.openstack.org/21378218:05
*** pelix has joined #openstack-ironic18:06
*** pelix has quit IRC18:07
JayFJoshNang: ^ your commit message didn't wrap18:07
JayFJoshNang: the 2nd full paragraph18:07
JayFJoshNang: lgtm otherwise so if you fix that I'll +218:08
*** pelix has joined #openstack-ironic18:08
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Dispatch the call to erase_block_device  https://review.openstack.org/21378218:08
JayFJoshNang: ^ I just fixed it18:09
*** nicodemos has quit IRC18:11
*** marzif has joined #openstack-ironic18:12
*** mgoddard has quit IRC18:15
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Refactor list_all_block_devices()  https://review.openstack.org/21840418:17
*** alexpilotti has quit IRC18:17
JoshNangJayF: blah. thanks for the fix :D18:18
JayFJoshNang: and as soon as I get my local ipa tests running again, going to +2 your functional test piece18:19
JoshNang\o/18:19
JayFand fix some ironic dev docs, because they don't say you need python3.4-dev :)18:19
JayFwhen you do18:19
JayFat least I think that's what my problem was18:20
*** Nisha has joined #openstack-ironic18:20
jlvillalWow, the gate queue is at about 23 hours!18:21
JayFhttps://gist.github.com/jayofdoom/35d24d4351c15c98c25a I can't seem to get py34 tests passing locally18:22
JayFanyone else seen that error? "db type could not be determined"18:22
JayF(IPA)18:22
jlvillalJayF: rm -rf .testrepository/18:23
jlvillalJayF: From the root of IPA18:23
JoshNang^ works for me for at least one run :/18:23
jlvillalThen: tox -epy3418:23
*** mtanino has joined #openstack-ironic18:23
JoshNangusually breaks on the next run though18:23
JayFso just running "tox" is basically always setup to fail18:23
JayFthat's pretty crappy18:23
JayF:C18:23
jlvillalJayF: Probably should submit a patch to have py34 run first18:23
jlvillalThere was a patch that did that for Ironic18:24
JoshNangorly?18:24
jlvillalNot sure, but for me it is working fine run after run. Once I deleted the directory and ran the py34 test18:24
jlvillalI can then just run 'tox' again and again.18:24
jlvillalJoshNang: I was thinking about proposing it for IPA, but then I started doing other stuff...18:25
JoshNangif i rm that dir, i can do tox -epy34 over and over, but as soon as i do a full tox it fails18:25
jlvillalJoshNang: Hmmm.  I am on Fedora 22 and Ubuntu 14.04 and it seems to work for me.18:25
jlvillalNo idea what is different18:26
JayFgood call jlvillal, that worked for me18:26
JayFin my ubu 14.04 vm18:26
JoshNangyup 14.04 here too18:26
JayFtox -epy34, then just tox18:26
JayFand all things pass18:26
JayFafter deleting the testr dir18:26
JayFnow I'm going to work out the ordering patch and put it up :)18:26
JoshNangheh wouldn't be the first time my dev env is messed up though, which is why i haven't dug in too far :P18:26
*** chadix has joined #openstack-ironic18:26
openstackgerritJohn L. Villalovos proposed openstack/ironic-python-agent: Have tox run py34 as first test  https://review.openstack.org/21840518:27
jlvillalJayF: JoshNang ^^^18:27
JayFI was going to jfdi, but I guess I can land it if you put it in18:27
*** romcheg has left #openstack-ironic18:27
JayFI'm just a little worried I might not have ATC yet this cycle, haha18:27
* jlvillal grumbles about someone add 'pint' to IPA and forcing him to do a '-r' 18:27
JoshNangat least ipa reinstalls fast :D18:28
*** mgoddard has joined #openstack-ironic18:30
*** amotoki has quit IRC18:31
*** alexpilotti has joined #openstack-ironic18:36
*** alexpilotti has quit IRC18:40
*** Nisha has quit IRC18:40
*** chadix has quit IRC18:41
*** e0ne has quit IRC18:42
*** achanda has quit IRC18:44
*** alexpilotti has joined #openstack-ironic18:47
*** pelix has quit IRC18:48
openstackgerritStephanie Miller proposed openstack/bifrost: Get install path for ironic services  https://review.openstack.org/21841618:54
*** jcoufal has quit IRC18:59
JayFJoshNang: https://review.openstack.org/#/c/218405/1 we should land this19:05
JoshNangjlvillal: JayF: tested locally, fixed the bug! +A19:08
JayFA+ +A19:08
JayFlol19:08
*** alexpilotti has quit IRC19:08
JoshNangheh19:10
JayFWhat would folks think about making it configurable in the ipmitool driver what IPMI "protocol" is chosen19:10
JoshNang+A before jenkins finishes, because YOLO19:10
JayFtoday we hardcode "lanplus"19:10
*** achanda has joined #openstack-ironic19:11
JayFIf it was configurable, it'd likely have to be done per node though? in driver_info?19:11
jlvillalJoshNang, Any idea why the gate-tempest-dsvm-ironic-pxe_ipa-src seems to be failing.  Well it did on two of my unrelated patches to IPA19:12
JoshNangpretty sure it's bug 139309919:13
openstackbug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed] https://launchpad.net/bugs/139309919:13
JoshNangoh heh just saw you did a recheck too. must have been after i opened the page19:13
*** afaranha has quit IRC19:15
trownJayF: is there something other than lan and lanplus?19:21
trownJayF: https://github.com/openstack/ironic/commit/f0e9bc63aa903a792a05eaca7059967522c6c88719:21
JayFtrown: mostly vendor-specific things19:21
JayFtrown: After asking that question, I found out it's more that we want to add a "-o vendor_thing" to the ipmi string19:21
JayFbut there absolutely are other options19:22
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Enable --debug for debug logging  https://review.openstack.org/21526519:25
JayFJoshNang: were command line options just not working?19:29
JayFI'm confused as to what actually changed in agent.py there19:29
JoshNangJayF: passing sys.argv[1:] is required for oslo to see the cli opts, i guess?19:31
JoshNang--standalone definitely worked (i've used it a ton)19:32
JoshNangand/or it needs to happen before log.setup19:32
JayFJoshNang: I'm just confused as to why the CONF(args=sys.argv[1:]) line isn't before log.register_options19:32
JoshNang(i tried about 20 combos before i got this working)19:32
JayFahaha19:32
JayFJoshNang: I gave you a +0 with some suggestinos for the doc19:33
JayFJoshNang: I'll leave that +2 to someone who actually understands what's going on ;)19:33
JoshNangheh k19:34
JoshNangthanks!19:34
*** alexpilotti has joined #openstack-ironic19:34
*** devlaps has joined #openstack-ironic19:37
*** achanda has quit IRC19:43
*** achanda has joined #openstack-ironic19:47
*** doublek has joined #openstack-ironic19:52
*** jcoufal has joined #openstack-ironic19:52
openstackgerritMitsuhiro Tanino proposed openstack/ironic: [POC] pxe: support boot from Cinder iSCSI volume using iPXE  https://review.openstack.org/21584919:53
JayFJoshNang: fwiw 'recheck bug x' is deprecated19:54
JayFJoshNang: I've been told basically to make elastic-recheck entries for common things and not to indicate the bug number anymore19:54
JoshNangJayF: ahh thanks19:55
*** Nisha has joined #openstack-ironic19:57
*** alexpilotti has quit IRC19:59
*** penick has joined #openstack-ironic20:04
*** penick_ has joined #openstack-ironic20:05
*** penick has quit IRC20:09
*** penick_ is now known as penick20:09
*** puranamr has joined #openstack-ironic20:11
*** Nisha has quit IRC20:12
*** Nisha_away has joined #openstack-ironic20:12
*** Nisha_brb has joined #openstack-ironic20:14
*** Nisha_away has quit IRC20:14
*** puranamr has quit IRC20:17
JayFJoshNang: any thoughts on how to approach 1490008 (the bug we were pm'ing about)20:19
*** puranamr has joined #openstack-ironic20:20
JayFhmm. I have an idea even though I think it's wrong.20:20
*** puranamr has quit IRC20:21
*** alexpilotti has joined #openstack-ironic20:26
*** achanda has quit IRC20:28
NobodyCamSpamapS: just -1'd 217374 only for doc update. I'm to change my vote if we want to address with a follow up patch. just let me know20:31
NobodyCams/I'm to/I'm happy to/20:32
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: WIP: Preload Hardware Managers  https://review.openstack.org/21846320:32
*** Nisha_brb has quit IRC20:32
JayFJoshNang: ^ that doesn't pass tests, but it's sorta the idea I was thinking about20:32
*** Nisha has joined #openstack-ironic20:33
*** Nisha has quit IRC20:37
JayFaarefiev: welcome to the project; I've been seeing your reviews and don't know if we've met :D20:40
JayFaarefiev: nice to have you around and thanks for reviewing things in ipa20:40
*** krtaylor has quit IRC20:51
*** alexpilotti has quit IRC20:51
JoshNangJayF: that's exactly how I would handle it!20:54
JayFyah, and it breaks tests20:55
JayFwhich is sad20:55
JayFJoshNang: lemme gist you the test failures and see if you have any idea20:56
JoshNangJayF: :|20:57
JayFhttps://gist.github.com/jayofdoom/e2b68a1e2b7fce0dde0320:57
JayFah, you know what it is20:57
JayFbecause we load early now it's got the empty set of managers cached for the multi hardware tests, I think20:57
*** aarefiev22 has quit IRC20:58
JayFthose tests were hard enough to get working once :)20:58
JoshNangmaybe? if so the test seems a bit leaky20:58
JoshNangheh i know i had to change them at one point and it was a bit of a pain20:58
*** natorious is now known as zz_natorious20:59
JayFYou wanna take a look with me real quick? I'm around until my wife gets home20:59
*** dprince has quit IRC21:00
JoshNanguhh, sure21:00
JayFJoshNang: jumped in my vidyo room21:01
JoshNanggimme a sec to look at this first21:01
JayFJoshNang: there's like a 75% chance you're just going to be a rubber duck21:01
JayFlol21:01
*** puranamr has joined #openstack-ironic21:02
SpamapSNobodyCam: ACK, thanks. It's pretty WIP at this point, so can add docs soon.21:02
*** krtaylor has joined #openstack-ironic21:04
openstackgerritMerged openstack/ironic-python-agent: Have tox run py34 as first test  https://review.openstack.org/21840521:07
*** trown is now known as trown|outttypeww21:08
*** linggao has quit IRC21:08
*** alexpilotti has joined #openstack-ironic21:09
JoshNangJayF: oooooooooooooooooo21:10
JoshNangline 71821:10
JoshNangin hardware.py21:11
JayFYeah?21:11
JoshNanglook at the diff :D21:11
JoshNangit's indented21:12
JayF...21:12
JayFif that fixes it though, you're the man21:12
JayFHow did I miss that21:12
JayFI looked at the diff, too21:12
JoshNangyou and me both...i almost +2'ed21:13
JayFWell, in good news, it still fails. Just more tests too21:13
JayFwhy would you +2 a WIP pr, lol21:13
JoshNangwell, that too21:13
JayFhttps://gist.github.com/jayofdoom/733ef660eb2c230d6564 with the indentation removed21:13
JayFalthough the failures are more what I expect now at least21:13
JoshNanghmm? totally passes tests for me. did you change anything else?21:13
JayFoh, really?21:13
*** alexpilotti has quit IRC21:14
JoshNangwell, pep8 error, but py27 works21:14
JayFyeah I didn't drop indent enough21:14
JayFtrying it now21:14
*** puranamr has quit IRC21:15
JayFoh hell yeah21:15
JayF\o/21:15
JoshNangwooo!21:15
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Load Hardware Managers at runtime  https://review.openstack.org/21846321:16
JayFthank JoshNang21:16
JayFvery happy to have that done, and now I'm going to be out for the day21:16
openstackgerritJosh Gachnang proposed openstack/ironic-python-agent: Split functional and py27 tests  https://review.openstack.org/21473521:16
JoshNangJayF: \o/ +221:17
openstackgerritHarsh Shah proposed openstack/python-ironicclient: [WIP] Add keyring support  https://review.openstack.org/21813221:18
*** alexpilotti has joined #openstack-ironic21:19
JayF^ oh, that'd be excellent!21:19
* JayF adds that to list of things to review21:19
*** mjturek1 has quit IRC21:23
*** puranamr has joined #openstack-ironic21:25
jlvillalNobodyCam: Would you say this is closed?  https://bugs.launchpad.net/python-ironicclient/+bug/144117021:26
openstackLaunchpad bug 1441170 in python-ironicclient "ironicclient "Not Acceptable" error" [High,In progress] - Assigned to John L. Villalovos (happycamp)21:26
*** puranamr has quit IRC21:29
*** thrash is now known as thrash|wknd21:31
*** doublek has quit IRC21:38
*** cody-somerville has joined #openstack-ironic21:38
*** dimsum__ has quit IRC21:38
*** alexpilotti has quit IRC21:42
NobodyCamhey jlvillal I have not seen that in a while21:42
NobodyCamso ya21:42
jlvillalNobodyCam: Great :)  I remember working on that...21:43
*** meghal has joined #openstack-ironic21:48
*** meghal1 has quit IRC21:48
*** achanda has joined #openstack-ironic21:48
*** alexpilotti has joined #openstack-ironic21:49
*** alexpilotti has quit IRC21:54
cody-somervilleHey. When I try to set a node into or out of maintenance mode I get 404 despite ironic node-show working just fine for same uuid.21:56
*** dimsum__ has joined #openstack-ironic21:56
NobodyCamcody-somerville: I have not run in to that. are you using the cli?21:58
cody-somervilleYes.21:58
*** cdearborn has quit IRC21:58
NobodyCamcan you run a test with --debug and paste?21:58
*** dims_ has joined #openstack-ironic22:00
zer0c00lHow does one find the interface_type (SATA, SAS) on a running system http://specs.openstack.org/openstack/ironic-specs/specs/approved/ironic-generic-raid-interface.html22:00
zer0c00la22:00
zer0c00lIs the code for IPA side of things done yet?22:01
cody-somervilleNobodyCam: http://paste.openstack.org/show/431407/22:01
zer0c00lDetermining if the disk is whether ssd or hdd is easy. Readling /sys/block/<dev>/queue/rotational can tell you if it is ssd or hdd22:02
NobodyCamzer0c00l: maybe something like sudo lshw -c storage -c disk22:02
zer0c00lyeah, but its not that simple as reading from sysfs22:02
*** dimsum__ has quit IRC22:02
zer0c00land we need to add lshw in the ramdisk22:02
zer0c00lAny way to find it from /sys ?22:03
zer0c00ltried searching engine i am not able to find any22:04
zer0c00lparsing lshw output is cumbersome as well :(22:06
NobodyCamcody-somerville: wow thats strange.. and the node is deleted22:06
cody-somervilleNobodyCam: No. The node is still there.22:07
cody-somervilleAny risk to just modifying the bool in the db directly?22:09
*** meghal1 has joined #openstack-ironic22:09
*** alexpilotti has joined #openstack-ironic22:10
NobodyCamcody-somerville: and the api is listening on that ip and port? ie. it hasn't crashed or been firewalled out22:10
cody-somervilleNobodyCam: No. ironic node-list and node-show works just fine.22:11
*** meghal has quit IRC22:11
NobodyCamhumm22:13
NobodyCamzer0c00l: hummm.... readlink -f /sys/block/sda/device/../../../driver ???22:16
NobodyCamcody-somerville: node-list and node-show only hit the db... check that your conductor is in fact running22:17
jlvillalzer0c00l: lshw seems to support json22:20
jlvillalzer0c00l: Would seem to make parsing simpler22:21
NobodyCamI think that readlink will give him what he's looking for22:21
jlvillalNobodyCam: cool.  Would be better if can do it without calling external program22:22
zer0c00ljlvillal:  i see22:22
jlvillalNobodyCam: So the readlink seems better than calling lshw22:22
NobodyCam:)22:23
* jlvillal thinks he can do that with os or os.path library module :)22:23
jlvillalNobodyCam: Not sure I quite understand all the '..' in the path though...22:25
* jlvillal figures it out22:26
zer0c00lReadlink on my ssd machine shows this22:26
zer0c00l /sys/devices/pci0000:00/0000:00:1f.2/ata1/driver22:27
*** bnemec has quit IRC22:27
zer0c00lReadlink on a hp raid controller looks something like '/sys/bus/pci/drivers/hpsa'22:27
zer0c00l this is on a sata disk22:28
zer0c00l /sys/bus/pci/drivers/ahci22:28
NobodyCamzer0c00l: is htat what you need?22:32
NobodyCamthat*22:32
*** alexpilotti has quit IRC22:32
zer0c00lNobodyCam:  I was hoping to find if the interface_type is SATA or SAS22:33
jlvillalzer0c00l: Maybe you can look at the kernel driver to figure out what information it publishes and where.22:35
jlvillalzer0c00l: I'm not sure if it is standard or not.22:35
jlvillalTo figure out SAS vs SATA22:36
zer0c00lokay22:38
zer0c00llshw does say in the description if it is a SATA controller22:38
zer0c00land other drives are attached to it as children22:38
*** ndipanov has quit IRC22:42
*** alexpilotti has joined #openstack-ironic22:50
*** alexpilotti has quit IRC22:55
*** jamielennox is now known as jamielennox|away23:00
*** alexpilotti has joined #openstack-ironic23:02
*** baoli has quit IRC23:10
openstackgerritHarsh Shah proposed openstack/python-ironicclient: [WIP] Add keyring support  https://review.openstack.org/21813223:10
*** mtanino has quit IRC23:21
*** david-lyle has quit IRC23:24
*** harshs has joined #openstack-ironic23:24
*** zz_natorious is now known as natorious23:25
*** alexpilotti has quit IRC23:28
*** vishwanathj has quit IRC23:28
*** garthb has quit IRC23:32
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements  https://review.openstack.org/21696623:43
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/21748123:43
*** mgoddard has quit IRC23:51
*** dimsum__ has joined #openstack-ironic23:52
*** alexpilotti has joined #openstack-ironic23:54
*** dims_ has quit IRC23:55

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