Wednesday, 2014-10-29

NobodyCamlol closer ... http://paste.openstack.org/show/HPTMqqpPkfxBVk8Ttzad/00:00
jrollweird?00:03
jrollseems wrong.00:04
jrollany logs in conductor?00:04
NobodyCamthat was just prob me00:05
NobodyCami go it to wait call back00:05
NobodyCambut the deploy image is crashing00:05
NobodyCami need to dbl check the basics 32/64 bit00:06
NobodyCamenough mem00:06
NobodyCamblah00:06
NobodyCametc00:06
*** yuanying has joined #openstack-ironic00:06
NobodyCambut it is trying00:06
jrollah hmm00:07
jrolldo you get anything on the console?00:08
NobodyCamya big blow up00:08
NobodyCamlol00:08
jrollorly00:08
jrollcan I see?00:08
jrollI mean... keep in mind nobody has ran this DIB image00:08
jrollso it could be completely f'd00:08
spandhejroll: I saw the same error messages in nova-compute.log.. turns out, I was trying to use nova.network.manager instead of Neutron.. qq on that.. do I have to run neutron at all, or is there a way around it ?00:12
jrollmmm. not sure. I'm using neutron.00:13
NobodyCamjroll: I need to check what I've done before I say its broke00:13
jrollI thihnk you have to00:13
spandheI am just thinking out loud.. nova-compute is going to call allocate_network_async.. and thats going to invoke whichever network API you configure00:13
spandhejroll: ok cool..thanks!00:13
jrollyeah, I feel like it should work on the nova side, but not on the ironic side :(00:14
*** achanda has quit IRC00:15
*** achanda has joined #openstack-ironic00:15
NobodyCamjroll: I have a tiff of the console if you really want it00:15
jrollNobodyCam: sure why not :P00:16
JayFNobodyCam: never hurts to share info00:16
jrollwe may have seen it before00:16
spandhejroll: I am going to use my own dhcp server.. hence, trying to see if I can avoid using Neutron completely :)00:16
jrollaha, that can be done :)00:16
NobodyCamhow you want the tiff00:16
jrollwe do that ourselves00:16
jrollNobodyCam: dunno, as a tiff?00:16
jrollimgur.com?00:16
*** yuanying has quit IRC00:17
spandhejroll: yeah I have read the blogs.. have gone through the new agent you guys proposed.. but that stuff is too fancy for my use case..00:17
*** yuanying has joined #openstack-ironic00:17
jrolllol00:18
jrollit's not that fancy :)00:18
NobodyCamhttp://imgur.com/8nOmR5i00:19
NobodyCamlol first image ever shared00:19
*** achanda has quit IRC00:19
spandhejroll: The Ironic agent is meant to update switch port configuration, right? I am going to use static DHCP allocation.. nothing will change on the network side..00:20
*** rushiagr is now known as rushiagr_away00:20
jrollspandhe: no... that's a downstream thing we buildt, you don't need to use that00:20
jrollit's not upstream yet00:20
spandhejroll: right.. thats what I was talking about.. its mentioned on your OnMetal blog.. interesting stuff!00:21
jrollNobodyCam: weird, maybe pass an init= something?00:21
jrollmaybe init=/path/to/systemd00:22
jrollJayF: ^00:22
jrollspandhe: thanks :)00:22
jrollspandhe: upstream ironic agent is still pretty simple00:22
*** yuanying has quit IRC00:22
jrollNobodyCam: though I think maybe there's a different problem making nothing executable00:23
*** yuanying has joined #openstack-ironic00:28
spandhejroll: havent checked that yet.. let me check..00:29
openstackgerrityunhong-jiang proposed a change to openstack/ironic: Change the force_raw_image config usage  https://review.openstack.org/13088000:29
openstackgerrityunhong-jiang proposed a change to openstack/ironic: Convert qcow2 image to raw format when deploy  https://review.openstack.org/13088100:29
*** todd_dsm has joined #openstack-ironic00:31
*** yuanying has quit IRC00:35
*** yuanying has joined #openstack-ironic00:36
*** yuanying has quit IRC00:40
spandhejroll: did u mean DHCP agent?00:41
jrollspandhe: no, ironic-python-agent and the associated ironic driver00:41
jrollit's a deploy driver00:41
jrollinstead of iscsi + dd we use a python application with a rest api00:42
jroll(in the ramdisk)00:42
*** marcoemorais has quit IRC00:44
*** marcoemorais has joined #openstack-ironic00:44
*** marcoemorais has quit IRC00:46
*** marcoemorais has joined #openstack-ironic00:46
*** marcoemorais has quit IRC00:46
*** marcoemorais has joined #openstack-ironic00:46
*** todd_dsm has quit IRC00:49
spandhejroll: oh, ok.. I was talking about https://github.com/rackerlabs/ironic-neutron-plugin00:53
*** todd_dsm has joined #openstack-ironic00:54
*** yuanying has joined #openstack-ironic00:55
*** hemna has quit IRC00:58
*** ryanpetrello has joined #openstack-ironic01:02
*** todd_dsm has quit IRC01:03
*** Marga_ has joined #openstack-ironic01:06
*** yuanying has quit IRC01:07
*** yuanying has joined #openstack-ironic01:08
*** yuanying has quit IRC01:12
*** ryanpetrello_ has joined #openstack-ironic01:13
*** ryanpetrello has quit IRC01:13
*** ryanpetrello_ is now known as ryanpetrello01:14
*** marcoemorais has quit IRC01:17
*** lucas-dinner has quit IRC01:19
*** ryanpetrello has quit IRC01:25
*** chenglch has joined #openstack-ironic01:26
*** ryanpetrello has joined #openstack-ironic01:30
*** ryanpetrello has quit IRC01:36
*** Marga_ has quit IRC01:37
*** ryanpetrello has joined #openstack-ironic01:39
*** chenglch has quit IRC01:49
*** chenglch has joined #openstack-ironic01:49
*** ryanpetrello has quit IRC01:53
*** ryanpetrello has joined #openstack-ironic01:57
*** todd_dsm has joined #openstack-ironic01:59
*** nosnos has joined #openstack-ironic02:06
*** foexle has quit IRC02:07
*** dlaube has quit IRC02:08
*** yuanying has joined #openstack-ironic02:14
*** lazy_prince has quit IRC02:25
Haomengmorning:)02:30
*** spandhe has quit IRC02:30
*** Poornima has joined #openstack-ironic02:31
*** ryanpetrello has quit IRC02:35
*** Marga_ has joined #openstack-ironic02:39
*** killer_prince has joined #openstack-ironic02:55
*** killer_prince is now known as lazy_prince02:55
*** Marga_ has quit IRC03:02
*** Marga_ has joined #openstack-ironic03:03
*** leopoldj_ has joined #openstack-ironic03:19
*** leopoldj has quit IRC03:22
*** zz_naotok is now known as naotok03:24
*** jrist has quit IRC03:26
*** todd_dsm has quit IRC03:37
*** todd_dsm has joined #openstack-ironic03:41
*** ryanpetrello has joined #openstack-ironic03:43
*** nosnos has quit IRC03:46
*** nosnos has joined #openstack-ironic03:46
*** todd_dsm has quit IRC03:48
*** nosnos has quit IRC03:51
*** todd_dsm has joined #openstack-ironic03:51
*** ujuc has joined #openstack-ironic03:54
*** todd_dsm has quit IRC03:57
*** dlaube has joined #openstack-ironic04:00
*** dlaube has quit IRC04:02
*** marcoemorais has joined #openstack-ironic04:16
*** spandhe has joined #openstack-ironic04:16
*** marcoemorais1 has joined #openstack-ironic04:21
*** harlowja is now known as harlowja_away04:21
*** marcoemorais has quit IRC04:23
*** nosnos has joined #openstack-ironic04:27
*** ramineni has joined #openstack-ironic04:30
openstackgerritjiangfei proposed a change to openstack/ironic-python-agent: modify for CONF Fixes: bug 1386486  https://review.openstack.org/13163204:31
*** ryanpetrello has quit IRC04:34
*** Poornima has quit IRC04:34
*** pcrews has quit IRC04:37
*** todd_dsm has joined #openstack-ironic04:41
*** pcrews has joined #openstack-ironic04:41
*** chenglch has quit IRC04:46
*** achanda has joined #openstack-ironic04:49
*** chenglch has joined #openstack-ironic04:53
*** pcrews has quit IRC04:57
*** Poornima has joined #openstack-ironic05:05
*** todd_dsm has quit IRC05:08
*** todd_dsm has joined #openstack-ironic05:08
*** todd_dsm has quit IRC05:22
*** pensu has joined #openstack-ironic05:24
*** pcrews has joined #openstack-ironic05:26
*** rushiagr_away is now known as rushiagr05:29
*** pensu has quit IRC05:29
*** rakesh_hs has joined #openstack-ironic05:31
*** pcrews has quit IRC05:32
*** pensu has joined #openstack-ironic05:32
openstackgerritHarshada Mangesh Kakad proposed a change to openstack/ironic-specs: Seamicro Serial Console  https://review.openstack.org/13138205:36
*** loki184 has joined #openstack-ironic05:44
*** achanda has quit IRC06:12
*** achanda has joined #openstack-ironic06:14
*** spandhe has quit IRC06:20
*** achanda has quit IRC06:20
*** achanda has joined #openstack-ironic06:25
*** lazy_prince has quit IRC06:38
*** achanda has quit IRC06:39
*** Marga_ has quit IRC06:46
*** killer_prince has joined #openstack-ironic06:46
*** chenglch|2 has joined #openstack-ironic06:46
*** killer_prince is now known as lazy_prince06:46
*** chenglch|2 has left #openstack-ironic06:46
*** lazy_prince has quit IRC06:50
*** killer_prince has joined #openstack-ironic06:59
*** killer_prince is now known as lazy_prince06:59
*** lazy_prince has quit IRC07:20
*** lazy_prince has joined #openstack-ironic07:21
*** nosnos has quit IRC07:25
*** nosnos has joined #openstack-ironic07:27
*** leopoldj_ has quit IRC07:29
*** dtantsur|afk is now known as dtantsur07:36
dtantsurMorning Ironic07:37
*** teju has joined #openstack-ironic07:46
tejuHaomeng : morning :)07:48
*** foexle has joined #openstack-ironic07:51
*** lazy_prince has quit IRC07:58
*** RioS2 has quit IRC08:05
*** killer_prince has joined #openstack-ironic08:05
*** killer_prince is now known as lazy_prince08:06
*** jcoufal has joined #openstack-ironic08:07
*** lazy_prince has quit IRC08:10
*** andreykurilin_ has joined #openstack-ironic08:12
*** killer_prince has joined #openstack-ironic08:19
*** killer_prince is now known as lazy_prince08:19
*** kongfy has joined #openstack-ironic08:22
*** RioS2 has joined #openstack-ironic08:23
*** rushiagr is now known as rushiagr_away08:27
*** k4n0 has joined #openstack-ironic08:28
openstackgerritA change was merged to openstack/ironic: Update install guide to install tftp  https://review.openstack.org/13154708:29
*** ndipanov_gone is now known as ndipanov08:29
*** lazy_prince has quit IRC08:39
*** lazy_prince has joined #openstack-ironic08:39
*** rushiagr_away is now known as rushiagr08:39
*** andreykurilin_ has quit IRC08:47
Haomengteju: morning:)08:51
Haomengteju: how are you08:51
tejuHaomeng : i'm cool..how r u & howz Ironic?08:52
ekarlsomorning :D08:54
Haomengteju: fine:)08:55
Haomengekarlso: morning:)08:55
*** yuanying has quit IRC08:57
*** jistr has joined #openstack-ironic09:06
*** derekh has joined #openstack-ironic09:07
*** ujuc has quit IRC09:12
*** leopoldj has joined #openstack-ironic09:13
*** lucasagomes has joined #openstack-ironic09:24
*** kongfy has quit IRC09:24
*** igordcard has joined #openstack-ironic09:27
*** athomas has joined #openstack-ironic09:28
*** pradipta_away is now known as pradipta09:39
*** achanda has joined #openstack-ironic09:40
*** marcoemorais1 has quit IRC09:41
*** lazy_prince has quit IRC09:43
*** pelix has joined #openstack-ironic09:44
*** achanda has quit IRC09:44
*** mkerrin has quit IRC09:49
*** mkerrin has joined #openstack-ironic09:54
*** chenglch has quit IRC10:07
*** Marga_ has joined #openstack-ironic10:11
yuriyzmorning all10:23
yuriyzlucasagomes, old client bug again https://bugs.launchpad.net/ironic/+bug/138710110:25
yuriyzfresh Devstack install on the clean OS10:26
*** rushiagr is now known as rushiagr_away10:39
lucasagomesyuriyz, morning, will take a look10:45
*** Marga_ has quit IRC10:46
*** Marga_ has joined #openstack-ironic10:46
lucasagomesyuriyz, I think devananda have spotted that problem yesterday https://bugs.launchpad.net/python-ironicclient/+bug/138647010:49
lucasagomesI will take a look at it10:49
*** killer_prince has joined #openstack-ironic10:54
*** killer_prince is now known as lazy_prince10:54
yuriyzthanks lucasagomes10:55
*** sambetts has joined #openstack-ironic10:56
*** athomas has quit IRC10:58
*** ifarkas has joined #openstack-ironic10:58
yuriyzand this bug reproduced with OS_USERNAME=admin in my environment10:59
yuriyzfor error messages11:01
*** loki184 has quit IRC11:01
sambettsGood Morning Ironic!11:03
lucasagomessambetts, morning11:04
*** Marga_ has quit IRC11:05
*** ramineni has quit IRC11:05
*** athomas has joined #openstack-ironic11:06
sambettsmorning lucasagomes11:07
*** boris-42 has quit IRC11:11
*** rameshg87 has joined #openstack-ironic11:17
*** teju has quit IRC11:19
*** kongfy has joined #openstack-ironic11:24
*** rushiagr_away is now known as rushiagr11:25
*** pradipta is now known as pradipta_away11:28
*** Poornima has quit IRC11:37
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add a mechanism to route vendor methods  https://review.openstack.org/12926111:42
lucasagomessambetts, ^ using ur patch as base :)11:42
lucasagomeswill write the bp now11:42
lucasagomes(wanted to know how it would look like first)11:42
sambettsNice!11:42
*** lazy_prince is now known as killer_prince11:44
sambettsI'm just updating my patch with the comments you left11:45
lucasagomessambetts, yeah I applied the functools.wraps on my patch, I will rebase on top of urs again once u fix it there11:46
lucasagomesno rush :)11:46
*** ryanpetrello has joined #openstack-ironic11:53
*** Poornima has joined #openstack-ironic11:54
*** pensu has quit IRC11:57
*** naotok is now known as zz_naotok12:07
*** dprince has joined #openstack-ironic12:10
dtantsurWhat do you folks think about having periodic tasks for drivers? E.g. any driver could declare it's periodic task...12:12
*** nosnos has quit IRC12:15
*** rameshg87 has left #openstack-ironic12:17
*** killer_prince has quit IRC12:20
lucasagomesdtantsur, what is the use case? But to be honest in general my first reaction is urgh12:21
lucasagomesnot because it's a bad idea12:21
lucasagomesbut because periodic tasks in openstack sucks so badly12:21
lucasagomesif they at least could run in parallel12:21
lucasagomesbut they don't, they run one after the other in a single greenthread12:22
dtantsurlucasagomes, it's for DRAC RAID spec (see comments): https://review.openstack.org/#/c/107981/9/specs/juno/drac-raid-mgmt.rst12:22
dtantsurlucasagomes, yeah, implementation sucks, but the very idea looks sane to me12:23
lucasagomesoh I see12:23
lucasagomesyeah, I see it being useful12:23
*** shikui__ has joined #openstack-ironic12:28
dtantsurlucasagomes, btw why aren't periodic tasks fixed? It should not be too hard to move them to different greenthreads...12:34
openstackgerritSam Betts proposed a change to openstack/ironic: Add logging to driver vendor_passthru functions  https://review.openstack.org/12929812:35
lucasagomesdtantsur, no idea, I bet that for now it may be "okish" to be the way it is12:36
lucasagomesalso it's still part of the oslo incubation, it doesn't have it's own library12:37
lucasagomesmaybe someone is working on that and improving it at the same time (just like the oslo.config stuff)12:37
lucasagomesworth investigation, idk exactly, these are just assumptions12:37
dtantsurlucasagomes, one more topic to discuss, if we have some time12:37
lucasagomesyeah, dhellmann might know it12:37
openstackgerritDmitry Tantsur proposed a change to openstack/ironic-specs: DRAC vendor passthru for RAID management  https://review.openstack.org/10798112:39
lucasagomesyuriyz, :/ I did a fresh install but can't reproduce the error12:43
lucasagomesyuriyz, http://paste.openstack.org/show/126464/12:43
yuriyzlucasagomes, try request with missing parameter, ironic node-create for example w/o driver12:45
lucasagomesack12:46
lucasagomesstack@virtual-machine:~/devstack$ ironic node-create -i test=test12:46
lucasagomesAuthentication failure: Could not find user: dsasddsa (Disable debug mode to suppress these details.) (HTTP 401)12:46
lucasagomesyuriyz, I will investigate more, but seems grand12:46
yuriyzwith valid user12:46
lucasagomesyuriyz, ohhhh there you go12:47
lucasagomesstack@virtual-machine:~/devstack$ ironic node-create -i test=test12:47
lucasagomes'unicode' object has no attribute 'get'12:47
lucasagomesyuriyz, cheers12:47
*** Krast has joined #openstack-ironic12:48
*** kongfy has quit IRC12:49
sambettslucasagomes: What do you think about dtantsur's comments on my patch https://review.openstack.org/#/c/129298/9/ironic/drivers/base.py? I like the idea of changing the name to @passthru instead of @register_passthru, and I think we could make the method param optional, if not provided take the func.__name__ minus the leading '_', ideas?12:49
lucasagomessambetts, I like the idea of having a shorter name @passthru and @driver_passthru seems good to me12:50
lucasagomessambetts, right, about making it optional hmm, I kinda like to be explicit because that's the method name people are adding to the HTTP request12:51
lucasagomesv1/nodes/<uuid>vendor_passthru/<method name>12:51
sambettsyeah thats what I had thought12:51
lucasagomessambetts, I wouldn't be against taking it from __name__ but, I think explicit may be clearer12:52
sambettsdtantsur, ideas?12:52
dtantsurit's overexplicit to me12:52
dtantsur@passthru('method') def method()12:52
lucasagomesdtantsur, right, so use __name__ but not remove the prefixed '_' right?12:53
lucasagomesdtantsur, some functions are not like that as well... see pxe driver12:53
dtantsurI don't like leading _ anyway, these methods are not private12:53
dtantsurlucasagomes, if name does change, we can use this optional argument12:54
lucasagomesthe vendor method is called "pass_deploy_info", and the method itself is called "_continue_deploy"12:54
sambettsthere is an example in the ilo driver where its @passthru('pass_deploy_info') def _set_boot_device12:54
dtantsurIMO them we should rename the method, thus making code more clear12:54
lucasagomesdtantsur, right, yeah I'm not against it12:54
dtantsurpass_deploy_info vs  _set_boot_device looks somewhat weird to me :)12:55
lucasagomesdtantsur, yeah, it's because the ilo is inherting from pxe12:55
lucasagomesso all it does it set the boot device, and then call the parents _continue_deploy12:55
dtantsurlucasagomes, mmm, this inheritance thing... I have concerns about it12:56
dtantsurat least we'll have double error handling12:56
lucasagomesdtantsur, yea me too, but if we talk about it we are diverting the conversation focus12:56
dtantsurright12:56
dtantsurso I think we should try to use method name as often as it's possible and leave 'method' as optional argument12:57
lucasagomesdtantsur, sambetts so to be honest I see benefits on both sides, and could argue about both12:57
lucasagomesdtantsur, idea of making it option, and still provide a way to reuse it if needed sounds good to me12:57
sambettsthats why I'm thinking optional12:58
lucasagomesto rename it*12:58
dtantsurlgtm too12:58
lucasagomesright, I will add it to the spec12:58
sambettswhat did we decide about the leading underscore?12:58
lucasagomesI don't like the idea of removing it from the name if it exists12:59
dtantsurI don't like that it exists :)12:59
lucasagomesimo that's where things get too obscured12:59
dtantsurthese are not private methods, which the name implies12:59
dtantsurthey are used from outside12:59
lucasagomesI'm more into renaming those functions and remove the leading _ than removing it as part of the decorator12:59
dtantsurso  @passthru() def pass_deploy_info() right?13:00
lucasagomesdtantsur, well you still can rename it and keep the _continue deploy13:00
sambettsyeah13:00
lucasagomesin case it's going to affect too many tests13:00
lucasagomesbut yeah...13:00
dtantsurlunch time, brb13:01
lucasagomesenjoy, I will eat something quickly too13:01
*** lucasagomes is now known as lucas-hungry13:01
jrolldtantsur, lucas-hungry, sambetts, please make it explicit13:03
jrolldon't couple the method name to the API13:03
jrolla simple refactoring could end up backwards-incompatible13:03
jrollwe went through this discussion in ipa already :|13:04
jrollalso.. dtantsur ++ for drivers defining periodic tasks13:04
jrollalso also... good morning all :)13:04
*** kongfy has joined #openstack-ironic13:11
*** Haomeng has quit IRC13:13
dtantsurjroll, we already have RPC using method names13:13
dtantsurand g'morning of course :)13:14
*** Haomeng has joined #openstack-ironic13:15
*** mjturek has joined #openstack-ironic13:18
jrolldtantsur: not for passthru13:19
dtantsurjroll, right, but there "a simple refactoring could end up backwards-incompatible"13:20
jrollsigh.13:21
jrollhowever, just because one thing sucks, doesn't mean we should make another thing suck13:22
jrolland this is a user-facing api13:22
jrollRPC is only driver-facing13:22
dtantsurwell, maybe you're right and it's not such a big duplication (though it also does not look a huge problem to me)13:23
jrolldtantsur: another example...13:25
jrollilo has a set_boot_device() call13:26
jrollthat is used from passthru, and not from passthru13:26
jrollso the one called from passthru gets the good name if we couple to names13:26
jrollwhereas the one that actually does stuff needs a different name13:27
dtantsurjroll, it anyway needs, because we'll be baking exception handling inside the @decorator. and caller may expect it to raise13:28
jrollneeds what?13:28
dtantsurneeds a different name13:29
jrollright13:29
jrollhttps://review.openstack.org/#/c/129298/9/ironic/drivers/modules/ilo/deploy.py13:29
jrollI guess dropping the _ would be fine13:29
dtantsurjroll, is this call used anywhere else? I would suppose ilo_common.set_boot_device is used13:31
jrollyeah, you're right13:31
jrollI just remembered because we were talking about how to decorate that13:31
openstackgerritSam Betts proposed a change to openstack/ironic: Add logging to driver vendor_passthru functions  https://review.openstack.org/12929813:31
sambettsI've only implemented the shorten decorator name until we decide what we want to do with the method param13:32
jrollI'd really prefer that being explicit13:36
jrollbut that's just me13:36
*** igordcard has quit IRC13:37
sambettsI think I prefer the explictness because the user facing API may wish describe things differently to the code/function name, if its optional it isn't tied so staticly to the function name it just reduces the duplication in instances when they would be the same13:39
jrollI agree13:41
jrolloptional might be ok13:41
sambettsI'll put together a patch which has it as an optional param so we can see what it looks like and if its no good then we can easily revert it13:42
*** Krast has quit IRC13:43
jrollcool, ty13:44
*** rushiagr is now known as rushiagr_away13:47
*** stendulker has joined #openstack-ironic13:53
*** lucas-hungry is now known as lucasagomes13:56
*** romcheg has joined #openstack-ironic13:56
lucasagomesjroll, morning13:56
jrollheya lucas13:57
stendulkerHello13:57
stendulkerI get following error during deploy13:58
stendulkerERROR ironic.drivers.modules.iscsi_deploy [-] Deploy failed for instance None. Error: Unexpected error while running command. Command: sudo ironic-rootwrap /etc/ironic/rootwrap.conf dd if=/var/lib/ironic/images/f785923c-dfea-4d01-af2b-c26af86111ca/disk of=/dev/disk/by-path/ip-10.10.1.217:3260-iscsi-iqn-f785923c-dfea-4d01  -af2b-c26af86111ca-lun-1-part1 bs=1M oflag=direct Exit code: 1 Stdout: '' Stderr: "/bin/dd: error w13:58
jrollstendulker: you cut off at Stderr: "/bin/dd: error w13:58
jrollis there more?13:59
jrollstderr seems relevant here :P13:59
stendulkerjroll : No further error13:59
jrollreally?13:59
jrollseems wrong.14:00
stendulkerIs there anything to be modified in /etc/ironic/rootwrap.conf. I have a default settings in the same14:00
jrollit looks like dd ran, so rootwrap probably worked14:00
jrollbut I can't see why dd failed14:00
jrollso I really can't help14:00
jrollthere must be more14:00
stendulker2014-10-29 22:51:36.823 DEBUG ironic.common.utils [-] Command stderr is: "" from (pid=8835) execute /opt/stack/ironic/ironic/common/utils.py:86 2014-10-29 22:51:36.825 ERROR ironic.drivers.modules.iscsi_deploy [-] Deploy failed for instance None. Error: Unexpected error while running command. Command: sudo ironic-rootwrap /etc/ironic/rootwrap.conf dd if=/var/lib/ironic/images/f785923c-dfea-4d01-af2b-c26af86111ca/disk of14:01
stendulkerNothing more..14:01
stendulker2014-10-29 22:51:36.823 DEBUG ironic.common.utils [-] Command stderr is: "" from (pid=8835) execute /opt/stack/ironic/ironic/common/utils.py:86 2014-10-29 22:51:36.825 ERROR ironic.drivers.modules.iscsi_deploy [-] Deploy failed for instance None. Error: Unexpected error while running command. Command: sudo ironic-rootwrap /etc/ironic/rootwrap.conf dd if=/var/lib/ironic/images/f785923c-dfea-4d01-af2b-c26af86111ca/disk of14:01
stendulkerI think my messages are  getting truncated14:02
jrollright... maybe by your terminal or whatever application you're using to read the logs?14:03
jrollthey shouldn't be truncated on disk14:03
stendulker2014-10-29 22:51:36.823 DEBUG ironic.common.utils [-] Command stderr is: "" from (pid=8835) execute /opt/stack/ironic/ironic/common/utils.py:8614:04
stendulker2014-10-29 22:51:36.825 ERROR ironic.drivers.modules.iscsi_deploy [-] Deploy failed for instance None. Error: Unexpected error while running command.14:04
stendulkerCommand: sudo ironic-rootwrap /etc/ironic/rootwrap.conf dd if=/var/lib/ironic/images/f785923c-dfea-4d01-af2b-c26af86111ca/disk of=/dev/disk/by-path/14:04
stendulkerip-10.10.1.217:3260-iscsi-iqn-f785923c-dfea-4d01-af2b-c26af86111ca-lun-1-part1 bs=1M oflag=direct Exit code: 114:04
stendulkerStdout: ''14:04
stendulkerStderr: "/bin/dd: error writing '/dev/disk/by-path/ip-10.10.1.217:3260-iscsi-iqn-f785923c-dfea-4d01-af2b-c26af86111ca-lun-1-part1':14:05
stendulkerInput/output error\n1128+0 records in\n1127+0 records out\n1181745152 bytes (1.2 GB) copied, 264.387 s, 4.5 MB/s\n"14:05
stendulker2014-10-29 22:51:37.068 WARNING ironic.conductor.utils [-] Not going to change_node_power_state because current state = requested state = 'power off'.14:05
jrollperfect14:05
stendulkerThese are the logs14:05
jrolllet's use paste.openstack.org next time, but that's great14:05
jrollso I see Input/output error\n1128+0 records in\n1127+0 records out14:06
stendulkersure. was not aware if paste.openstack.org14:06
jrollso looks like 1) error writing to the disk, or 2) error with iscsi14:06
jrollor both14:06
stendulkerAnything I can try for further investigation14:08
jrollI am not an expert on testing disks, and have never used iSCSI14:08
*** todd_dsm has joined #openstack-ironic14:08
jrollbut perhaps someone else can help :)14:09
stendulkerjroll : ok. thanks for looking at it.14:09
jrollnp :)14:09
openstackgerritSam Betts proposed a change to openstack/ironic: Add logging to driver vendor_passthru functions  https://review.openstack.org/12929814:14
sambettslucasagomes, dtantsur, jroll: ^ optional method param14:15
*** Marga_ has joined #openstack-ironic14:15
jrollwhee14:15
* dtantsur is looking14:15
*** foexle has quit IRC14:15
NobodyCamgood morning Ironic14:16
jrollmorning NobodyCam14:17
jrollhmm14:17
jrollsambetts: the idea looks fine... but this is breaking APIs14:17
jrolle.g. https://review.openstack.org/#/c/129298/11/ironic/drivers/modules/ipmitool.py14:17
jrolloh wait I'm an idiot14:18
jrollyeah ignore me14:18
jroll:)14:18
dtantsurjroll, sambetts I'm more worried about using new decorator above @task_manager.require_exclusive_lock14:18
jrollyeah, should probably be below14:18
jrollwrap the whole thing in a lock14:19
*** Haomeng|2 has joined #openstack-ironic14:19
dtantsurjroll, 1. yes, 2. task_manager should also be fixed to use @functools.wraps14:19
sambettsreally? I would have thought you want to keep the lock as close to the code that needs the locked code as possible?14:19
dtantsursambetts, I think fixing the lock decorator should be enough14:20
jrollactually14:20
jrollit might be fine14:20
jrollthe new decorator doesn't do anything that needs a lock14:20
NobodyCammorning jroll, dtantsur14:20
*** Haomeng has quit IRC14:20
dtantsurNobodyCam, morning!14:21
*** zz_jgrimm is now known as jgrimm14:22
sambettsjroll: that is what I thought, logging and exceptions are all thread/process safe, so I decided to make my decorator wrap the already locked code14:22
sambettsthats interesting about it losing the __name__ though14:23
jrolldoes it?14:25
*** rloo has joined #openstack-ironic14:26
dtantsurjroll, without @functools.wraps - it should (to "wrapper")14:26
jrollmmm14:26
jrollthat's right14:26
sambettsYeah, I'm just adding the .wraps to the wrapper function14:27
openstackgerritSam Betts proposed a change to openstack/ironic: Add logging to driver vendor_passthru functions  https://review.openstack.org/12929814:29
sambettshows that ^14:29
* dtantsur is looking again14:30
dtantsursambetts, lgtm14:33
sambettswoop :D14:33
jrollsambetts: hmm, looking at https://review.openstack.org/#/c/129298/12/ironic/tests/drivers/test_agent.py14:37
jrollwould doing mock.patch.object(..., 'heartbeat') avoid the need to set __name__ etc14:37
jrolljust curious, change lgtm either way14:38
jrollmaybe we can clean those tests up later14:38
jroll+214:39
jrollbrb14:39
*** Marga_ has quit IRC14:49
*** todd_dsm has quit IRC14:55
*** jcoufal has quit IRC14:58
*** jcoufal has joined #openstack-ironic15:01
*** todd_dsm has joined #openstack-ironic15:03
*** rushiagr_away is now known as rushiagr15:08
lucasagomesdtantsur, I'm seem a odd error with the decorator thing, you may know15:08
lucasagomesdtantsur, when I call the function reference from the map I generated in the decorator, I'm getting  a "_private_method() takes exactly 3 arguments (2 given) (HTTP 500)"15:09
lucasagomesbasically the self parameter is not being automatically passed to that function15:09
lucasagomesany ideas?15:09
jrollmmm15:09
jrollyou need to bind self15:09
jrollin the decorator iirc15:09
lucasagomesjroll, right hmm15:10
lucasagomesjroll, https://review.openstack.org/#/c/129261/11/ironic/drivers/base.py15:10
lucasagomesL37215:10
lucasagomesthat how I was saving the reference15:10
jrollright15:11
jrollso I think passthru_handler can take self and pass it on to func()15:11
jrollbut I guess *args would do that15:11
lucasagomesso I should take self, *args, **kwargs?15:11
lucasagomesme too15:12
lucasagomesapparently it doesn't15:12
lucasagomeslemme see15:12
jrollmaybe15:12
*** pcrews has joined #openstack-ironic15:12
*** shikui__ has quit IRC15:14
lucasagomessame :/15:17
lucasagomesdamn15:17
jrollhuh.15:19
*** kongfy has quit IRC15:21
*** pensu has joined #openstack-ironic15:21
jrolloh lucasagomes15:22
jrollyou might be able to like... bind self in the decorator15:22
jrollhttps://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/base.py#L21115:22
jrollso passthru_handler should take self15:22
jrollfunc = functools.partial(func, self)15:23
lucasagomesoh15:23
lucasagomeslemme try that15:23
*** Marga_ has joined #openstack-ironic15:23
jrollsince you aren't calling it with object.method()15:23
jrollyou're just calling method()15:23
jrollI think that should work15:23
openstackgerritIlya Pekelny proposed a change to openstack/ironic: Test migrations with Alembic, using Oslo.db  https://review.openstack.org/11198415:23
*** hemna__ is now known as hemna15:30
lucasagomesjroll, nop :(15:31
dtantsurlucasagomes, hmmm.. it should work w/o self, as self is already bound...15:34
dtantsurwhere does this error happen?15:34
dtantsurah no, self is not bound15:35
lucasagomesdtantsur, in the worker thread15:35
lucasagomesdtantsur, I could reproduce here making a test code15:35
lucasagomeslemme give it to ya15:35
lucasagomesdtantsur, jroll http://paste.openstack.org/show/126536/15:35
lucasagomesyeah I invoke ha( ... ) same error happens15:36
dtantsurlucasagomes, lines  20 and 22 bound self twice15:36
dtantsuractually line 20 is not needed15:36
sambettslucasagomes: I think you could partial it out in the VendorPassThru init when you set the dict, you just make sure that every element has partial(func, self) instead of func??15:37
lucasagomesdtantsur, yeah that was a experiment, but removing it fails same way15:37
dtantsurlucasagomes, I can hardly belieave it's _the same_ way, it should be at least a bit different15:38
lucasagomessambetts, I don't have access to self at that point :/15:38
sambettslucasagomes: you have self in the init function ...15:38
sambettsdef __init__(self) <-15:39
lucasagomesoh I see what u mean15:39
sambettsinject it at that point15:39
dtantsurlucasagomes, I see several things wrong there, lemme try to fix15:39
lucasagomesyeah this is an experimentation code15:40
lucasagomesplease try :)15:40
dtantsurlucasagomes, http://paste.openstack.org/show/126539/15:44
lucasagomesha(b, 'aaa', abc='mole')15:44
lucasagomesso I need access to the class instance15:45
dtantsurlucasagomes, of course, you need self to call a method :)15:45
dtantsurlucasagomes, or you can store partial in metadata15:46
dtantsurhmm no you can't15:46
dtantsurso yes, you need self to pass15:46
lucasagomesdtantsur, I thought I could get a ref of the class method and self would be passed implicitly15:46
lucasagomesoh dear15:47
lucasagomes:/15:47
dtantsurlucasagomes, problems is: you don't get a ref to an instance (not class) method15:47
dtantsurlucasagomes, decorator work before binding15:47
lucasagomesyeah I see that now15:47
lucasagomeshmm that actually sucks15:47
*** ifarkas has quit IRC15:48
lucasagomeswondering if I should look at __class__15:48
*** todd_dsm has quit IRC15:48
dtantsurlucasagomes, you have an instance at the call place, what's the problem?15:49
dtantsurhttps://review.openstack.org/#/c/129261/11/ironic/conductor/manager.py line 410 you have vendor_iface15:49
dtantsurvendor_iface is self for the method15:49
*** todd_dsm has joined #openstack-ironic15:49
lucasagomesdtantsur, yeah that's fine, but can be trick when it comes to Mixin15:49
dtantsurright15:50
lucasagomesI could do check if that's mixin interface and get the real class instance from the map15:51
lucasagomesbut that looks pretty horrible15:51
dtantsuroh well15:51
dtantsurthat's why I don't like overusing decorators actually...15:52
*** todd_dsm has quit IRC15:52
lucasagomesyeah15:53
lucasagomesdtantsur, sounds a bit odd, but another way to do it would be to use a metaclass to create the routes15:54
dtantsurlucasagomes, I 99% prefer class decorator to a metaclass, but yeah, it's possible15:54
rlooor no decorator (sorry, haven't been following but I see the 'help me' signs)15:55
*** todd_dsm has joined #openstack-ironic15:55
lucasagomesvendor classes are inheriting from a base class15:55
lucasagomesso a metaclass makes more sense imo15:55
lucasagomesrloo, yeah or no decorator and back to manual mapping15:55
lucasagomeswhich is fine too15:55
dtantsurI guess jroll would call metaclass approach too implicit and I tend to agree :)15:56
rloolucasagomes: I'm fine with either. If going with decorators might make code/debugging harder to understand, I would say make it simple.15:56
lucasagomesdtantsur, well it's already implicit15:56
lucasagomesI don't see the diference15:56
lucasagomesrloo, yeah the manual explicity vendor_routes and driver_routes15:57
dtantsurlucasagomes, you see decorator on every function. with metaclass you see nothing (or you want metaclass + decorator?)15:57
lucasagomesis way simpler to debug15:57
lucasagomesdtantsur, functions will have a decorator15:57
lucasagomeslemme try to make a sample of what I'm thinking here15:58
*** jinxly has quit IRC15:58
openstackgerritJim Mankovich proposed a change to openstack/ironic-specs: Support per Driver Sensor Meters  https://review.openstack.org/13007415:59
openstackgerritIlya Pekelny proposed a change to openstack/ironic: Test migrations with Alembic, using Oslo.db  https://review.openstack.org/11198416:03
*** k4n0 has quit IRC16:03
*** Poornima has quit IRC16:06
lucasagomesdtantsur, http://paste.openstack.org/show/126558/16:06
lucasagomesjroll, ^16:06
lucasagomesrloo, ^ u too :)16:07
dtantsurhmm looks nice16:07
jrollmetaclasses make me cry a little16:08
*** todd_dsm has quit IRC16:09
*** dlaube has joined #openstack-ironic16:09
lucasagomeswell16:09
lucasagomesI also dislike metaclasses, but I don't really know a better way to solve it16:09
lucasagomesI may, but I'm tired at this point too :P16:10
jrollyeah, agree16:10
jrollI mean, whatever works16:10
lucasagomesjroll, we could also not use a decorator as rloo pointed out, and have that explicit routes as the spec says16:10
jrolloh, right16:11
rlooto be honest, I'd rather get rid of the decorators then.16:11
rloothe code seems complex to do something that would be simpler with the explicit maps, no?16:11
lucasagomesrloo, yup16:11
lucasagomesI mean it's not super complex, it just uses some more advanced python features16:12
* jroll bbiab16:12
rloolucasagomes: like the metaclass stuff. Seems like if we can avoid it, why not avoid it. BUT I'm not a python expert. Which is more 'pythonic'?16:13
rloowith the explicit routes, the driver writers can easily see which functions are being used. With the decorator, they'd have to scan the file.16:14
NobodyCambrb16:14
lucasagomesrloo, right... ok I prefer the simple case as well16:17
*** jmanko has joined #openstack-ironic16:17
lucasagomessambetts, hey, I think I won't use the decorator then :(16:17
rloois Shrews around?16:17
rloo(trying to remember who asked for decorators)16:18
lucasagomesI think it was Shrews16:18
rlooTo be honest, I think that the use of decorators for what sambetts did, is reasonable. It is just that it doesn't seem to 'extend' easily for what lucasagomes wants wrt the blueprint.16:19
*** marcoemorais has joined #openstack-ironic16:19
lucasagomesmakes sense for the log and exceptions, but if we try to expand it, then it becomes messy16:19
lucasagomesyeha16:19
Shrewshere16:19
*** jmank has quit IRC16:20
Shrewsjust got off the a long conf call. gotta catch up on scrollback16:20
rloohi Shrews. ^^ lucasagomes has been experimenting with the decorators that sambetts added, and I think it would be simpler to not use decorators. wanted your feedback.16:20
lucasagomes(feels sorry for taking the time of ur all to look at it, but I don't want to have to write a spec again and again and again)16:21
Shrewsrloo: lucasagomes: sambetts: i mentioned the use of decorators to make sure we considered using them as what sambetts wanted to do seemed to fit the pattern for that. if it turns out they are not right, then that's fine by me.16:22
rloolucasagomes: no worries. I'd rather the discussion happen now rather than later.16:22
* Shrews trusts lucasagomes's opinion16:22
lucasagomesShrews, they r fine for the logging and converting exceptions... problem is extending it to create the mapping for the vendor functions dynamically16:23
rlooShrews: yeah, that's what I think. Would you take a quick look at lucasagomes' prototype and see what you think? http://paste.openstack.org/show/126558/16:23
rlooShrews: it's taking those decorated functions and creating a dictionary with entries for them.16:24
*** todd_dsm has joined #openstack-ironic16:24
Shrewslooking16:24
*** rameshg87 has joined #openstack-ironic16:28
*** BLZbubba has quit IRC16:29
*** BLZbubba has joined #openstack-ironic16:29
*** dtantsur is now known as dtantsur|afk16:29
openstackgerritJim Mankovich proposed a change to openstack/ironic-specs: Remove Sensor RecordID from IPMI Sensor ID  https://review.openstack.org/13007016:30
Shrewslucasagomes: rloo: that seems acceptable to me (though metaclasses are handled differently in python3).16:32
*** Marga__ has joined #openstack-ironic16:32
rloothx Shrews.16:33
lucasagomesShrews, right so metaclass is it? yeah for py3 we use six16:33
lucasagomesthis is just a prototype16:33
Shrewssure16:34
*** Marga_ has quit IRC16:34
openstackgerritJim Mankovich proposed a change to openstack/ironic-specs: Support per Driver Sensor Meters  https://review.openstack.org/13007416:35
Shrewslucasagomes: what is 'lalau' ??? anything meaningful?16:36
lucasagomesShrews, no, just sounds funny in portuguese16:37
Shrews:)16:37
*** Marga__ has quit IRC16:37
*** victor_lowther__ has quit IRC16:37
lucasagomesnaming things are hard :P specially for prototypes16:37
* Shrews wants to hear "dhsauhdsuhdsaudshudsah" in portuguese16:37
*** rameshg87 has left #openstack-ironic16:38
lucasagomeshah I was a bit mad things weren't working :P16:38
*** spandhe has joined #openstack-ironic16:42
*** rloo has quit IRC16:43
*** marck has quit IRC16:43
*** marcoemorais has quit IRC16:43
*** spandhe_ has joined #openstack-ironic16:46
*** spandhe has quit IRC16:47
*** spandhe_ is now known as spandhe16:47
*** marcoemorais has joined #openstack-ironic16:47
*** jcoufal has quit IRC16:55
*** rakesh_hs has quit IRC17:04
*** Marga_ has joined #openstack-ironic17:05
*** Marga_ has quit IRC17:14
*** harlowja_away is now known as harlowja17:16
*** Poornima has joined #openstack-ironic17:21
*** harlowja_ has joined #openstack-ironic17:21
*** harlowja has quit IRC17:21
*** stendulker has quit IRC17:22
*** achanda has joined #openstack-ironic17:23
*** derekh has quit IRC17:25
*** jcoufal has joined #openstack-ironic17:27
*** leopoldj has quit IRC17:29
*** spandhe has quit IRC17:32
*** romcheg has quit IRC17:32
*** victor_lowther__ has joined #openstack-ironic17:33
*** achanda has quit IRC17:35
*** jistr has quit IRC17:37
*** marcoemorais1 has joined #openstack-ironic17:40
*** marcoemorais has quit IRC17:40
*** marcoemorais1 has quit IRC17:42
*** marcoemorais has joined #openstack-ironic17:43
*** ryanpetrello has quit IRC17:43
*** achanda has joined #openstack-ironic17:48
*** achanda has quit IRC17:51
*** harlowja_ has quit IRC17:55
*** harlowja has joined #openstack-ironic17:56
*** rloo has joined #openstack-ironic17:59
*** marcoemorais has quit IRC18:01
*** marcoemorais has joined #openstack-ironic18:01
*** marcoemorais has quit IRC18:02
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add a mechanism to route vendor methods  https://review.openstack.org/12926118:02
*** todd_dsm has quit IRC18:02
*** marcoemorais has joined #openstack-ironic18:02
lucasagomesrloo, Shrews ^ no metaclass18:04
lucasagomesclearner imo18:04
rloolooking...18:04
* sambetts looking18:05
lucasagomesI just put on the __init__ of the VendorPassthru what I was doing in the MetaClass18:05
*** pcrews has quit IRC18:07
sambettsI think thats nicer, no metaclass, however doesn't that rely on method being the function name???18:08
lucasagomessambetts, nop why?18:08
lucasagomesI don't get the func reference in the decorator anymore, since the decorator will wrap it and return another method18:09
lucasagomeswhat I do is on the Vendor class instantiation, I look for methods that contains the vendor metadata18:09
NobodyCamany ipa folks about?18:09
lucasagomessambetts, see base.py18:09
jrollhi NobodyCam18:09
NobodyCamhey hey jroll18:09
lucasagomessambetts, L42818:09
NobodyCamwould you happen to have a sec to look at: http://paste.openstack.org/show/FymqYB0fz1jkIqKRr75h/18:10
jrollofc18:10
NobodyCamI'm sure I have not set something18:10
sambettsyeah, I was thinking you were using the name field for getattr but thats not it18:10
jrollhuh18:10
jrollNobodyCam: that's trying to download your deploy image18:10
jrollNobodyCam: probably wrong deploy k/r ID in driver info?18:10
rloolucasagomes: I skimmed it. That looks much better!18:12
lucasagomesrloo, :)18:12
lucasagomesyeah, no black metaclass magic18:12
NobodyCamjroll: hum ... yep18:12
sambettslucasagomes: that looks nice, why did you pick to make metadata a set instead of a dict so you could access it by key??18:13
rloolucasagomes: assuming it works, but I does, right? I'll confirm when I really review it ;)18:13
lucasagomesrloo, yeah I'm testing it now and gate will test it too18:13
lucasagomesrloo, I made a basic vendor method and a driver method18:13
lucasagomesit seems fine18:14
jrolllucasagomes: so uh, on thing18:14
jrollone18:14
jrollyou're looping over all of the methods anyway18:14
lucasagomessambetts, a set instead of dict? A tuple?18:14
jrolloh wait18:14
jrollignore me18:14
jrolllucasagomes: with vendor_meta[1], what other keys are there?18:14
lucasagomessambetts, I will extend it later to be (method, {'supported': ['GET', 'POST'], 'async': True})18:14
rlooneed to get the spec approved. maybe we can do some sort of spec-review thing at the summit/pod.18:14
lucasagomesjroll, ^18:14
jrollaha :D18:15
lucasagomesit's like a plumbing to add the rest of the metadata for that method18:15
*** romcheg has joined #openstack-ironic18:15
jrollyeah18:15
lucasagomesjroll, makes sense?18:15
jrollyep18:15
jroll:)18:15
lucasagomesrloo, yup, yeah I will work on the spec now that I've a clear view of how it will wokr18:16
lucasagomeswork*18:16
sambettslucasagomes: I thought that which is why I wondered why you have a tuple wrapping it, you could just have 'method': whatever in the dict?18:16
lucasagomessambetts, yup having a dict there works as well18:17
lucasagomesI did a tuple cause it's easier to access the elements by index18:17
sambettsI thought that it would be more readable to access it by key thats all, either works :-)18:18
lucasagomessambetts, it will be only 1 key, because it's a tuple or dict per method18:19
*** dprince has quit IRC18:20
sambettsOOOOh I see why you've done it now! Ignore me :D18:22
*** marcoemorais1 has joined #openstack-ironic18:23
sambettsIt makes sense once I realised the mapping was like this: vendor_meta[0]: vendor_meta[1]18:23
lucasagomesyup18:23
*** marcoemorais has quit IRC18:24
*** Marga_ has joined #openstack-ironic18:24
sambetts:D lgtm then18:24
lucasagomes\o/18:25
NobodyCamlol jroll so close: http://i.imgur.com/26dpd4H.png18:25
jroll:(18:25
jrollNobodyCam: how much ram do you have?18:25
*** Poornima has quit IRC18:28
*** todd_dsm has joined #openstack-ironic18:28
*** Marga_ has quit IRC18:31
*** marcoemorais1 has quit IRC18:32
*** marcoemorais has joined #openstack-ironic18:32
*** marcoemorais has quit IRC18:32
lucasagomesaight I will take a break, good night everyone18:33
*** marcoemorais has joined #openstack-ironic18:33
lucasagomesta much for all the comments and suggestions!18:33
NobodyCamnight lucasagomes18:33
NobodyCamjroll: not much <currentMemory unit='KiB'>2097152</currentMemory>18:33
sambettsgood night lucasagomes18:33
jrollNobodyCam: try bumping that up18:34
jrollwait18:34
*** marcoemorais has quit IRC18:34
*** lucasagomes is now known as lucas-dinner18:34
jrollthat's 2 gigs, should be fine18:34
*** marcoemorais has joined #openstack-ironic18:34
*** dprince has joined #openstack-ironic18:37
*** athomas has quit IRC18:40
NobodyCambrb18:41
*** marcoemorais has quit IRC18:45
*** marcoemorais has joined #openstack-ironic18:45
*** marcoemorais has quit IRC18:49
*** marcoemorais has joined #openstack-ironic18:49
*** achanda has joined #openstack-ironic18:52
*** marcoemorais has quit IRC18:54
*** marcoemorais has joined #openstack-ironic18:55
*** achanda has quit IRC18:57
sambettsI'm off for the night, see you tomorrow ironic o/18:58
*** marcoemorais has quit IRC18:59
NobodyCamnight sambetts19:00
sambettsnight NobodyCam19:00
*** sambetts has quit IRC19:00
*** pcrews has joined #openstack-ironic19:07
*** andreykurilin_ has joined #openstack-ironic19:08
*** rushiagr is now known as rushiagr_away19:10
NobodyCamjroll: is there a knowen good ipa deploy image I can pull down for testing?19:12
jrollNobodyCam: https://github.com/openstack-dev/devstack/blob/master/lib/ironic#L78-7919:13
jrollShrews: around?19:13
jrollor anyone that knows about devstack(-gate) and firewalls that happen or don't happen to the "baremetal" vms19:13
NobodyCamTY19:14
jrollnp19:15
*** romcheg1 has joined #openstack-ironic19:15
*** romcheg has quit IRC19:16
*** marcoemorais has joined #openstack-ironic19:22
jrollhttps://github.com/openstack-dev/devstack/blob/master/lib/ironic#L52619:22
jrollmuahahaha19:22
jroll:(19:22
jrollthis is why agent jobs are failing19:23
* jroll will fix shortly19:23
Shrewsjroll: i'm not familiar with the devstack iptable magic19:28
*** jcoufal has quit IRC19:30
jrollShrews: no worries, I found it19:42
*** pcrews has quit IRC19:53
*** marcoemorais has quit IRC19:54
*** Marga_ has joined #openstack-ironic19:56
*** pelix has quit IRC19:57
*** pensu has quit IRC19:58
*** achanda has joined #openstack-ironic19:59
*** Marga_ has quit IRC20:05
*** Marga_ has joined #openstack-ironic20:06
*** andreykurilin_ has quit IRC20:08
*** igordcard has joined #openstack-ironic20:10
*** marcoemorais has joined #openstack-ironic20:18
*** dprince has quit IRC20:19
*** pcrews has joined #openstack-ironic20:23
*** adam_g_gone is now known as adam_g20:30
adam_go/20:30
NobodyCammorning adam_g20:30
*** Shrews has quit IRC20:44
*** Shrews has joined #openstack-ironic20:46
*** Shrews has quit IRC20:46
*** achanda has quit IRC20:50
*** marcoemorais1 has joined #openstack-ironic21:02
*** marcoemorais has quit IRC21:02
*** marcoemorais1 has quit IRC21:03
*** marcoemorais has joined #openstack-ironic21:04
*** todd_dsm has quit IRC21:07
*** Shrews has joined #openstack-ironic21:08
jrolladam_g: \o21:09
jrollNobodyCam: get anywhere?21:09
*** Marga_ has quit IRC21:09
NobodyCamlet ya know in a few21:12
*** todd_dsm has joined #openstack-ironic21:14
*** Marga_ has joined #openstack-ironic21:16
jrollJayF, adam_g, https://review.openstack.org/#/c/131876/21:16
jrollNobodyCam: ^ that was my problem in devstack21:16
adam_gjroll, ah!!21:17
jroll:)21:17
adam_gjroll, one small nit21:18
jrollsure21:18
jrolltouché21:18
jrollfixed, ty21:19
adam_gi miss anything exciting the last week or has it been pre-summit crickets?21:20
*** anderbubble has joined #openstack-ironic21:20
jrollyeah, not a lot going on21:21
*** achanda has joined #openstack-ironic21:24
*** todd_dsm has quit IRC21:26
*** todd_dsm has joined #openstack-ironic21:29
NobodyCamjroll: agent boots attempts to download image and errors21:35
NobodyCamfyi: the sample agent you pointed my to21:35
NobodyCamI see Could not download image with id 35804f18-fe1f-4fa2-8223-3e5421375626. in the log21:36
NobodyCamthat id is in glance | 35804f18-fe1f-4fa2-8223-3e5421375626 | testimage          | qcow2       | bare             | 280100864 | active |21:36
*** mjturek has quit IRC21:38
*** eghobo has joined #openstack-ironic21:39
*** eghobo has quit IRC21:39
anderbubbleI admit that this is really just a general openstack question; but I’m going through the instructions at http://docs.openstack.org/developer/ironic/deploy/install-guide.html to set up ironic, and I’m wondering why I have to configure explicit addresses and urls for individual services to configure ironic. Why can’t I just configure rabbitmq and have it find the services via the mq?21:43
jrollNobodyCam: conductor logs may have better errors?21:45
jrollanderbubble: not everything talks over rabbitmq... any config in particular you're thinking of?21:45
jrollanderbubble: e.g. nova uses a rest api to talk to ironic21:45
anderbubblein general, I think it would be nice if all of the services advertised on the mq; so then I could, for example, configure ironic to know where rabbitmq was, and then just have it broadcast a request for “where is nova?” “where is neutron?” “where is glance?” etc21:47
anderbubblewithout having to hardcode the different ip addresses in the config file21:47
anderbubblefull disclosure: this is my first openstack experience; it’s just a thought21:47
NobodyCamjroll: yep... looks like swift proxy configs : http://paste.openstack.org/show/pCsynsXtZY73tlHuhfFr/21:47
NobodyCambrb21:48
jrollNobodyCam: any logs from before this21:48
jrollanderbubble: aha, so21:49
jrollkeystone has a concept of a service catalog21:49
jrollservices register with keystone and other services can query this21:49
jrollit tends to work.. you shouldn't need to configure the neutron or glance url21:49
anderbubbleah21:49
jrollnor the ironic endpoint21:50
anderbubbleso it’s just in the instructions because… bit rot? ;)21:50
jrollbecause it's optional21:50
NobodyCamhttp://paste.openstack.org/show/8arBmdhe9UBhvVrCC8xk21:50
jrollI prefer to be explicit, so I configure them21:50
*** openstackgerrit has quit IRC21:50
jrollI also work at rackspace, where our service catalog is hilariously broken21:50
anderbubbleok; cool21:50
jrollso I configure them21:50
jroll:)21:50
anderbubbleexcept that, then, if they change, you have to change it in *two* places :p21:50
anderbubble;)21:50
anderbubbleanyway: good to know. I have been enlightened.21:51
jrollright :P21:51
NobodyCambrb21:51
jrollI would love to use etcd or something for service discovery21:51
jrollNobodyCam: huh, I wonder if the tempurl expired... kind of doubt it though21:52
anderbubblejroll, to be fair, the instructions say things like “This configuration file must be modified on the Compute Service’s controller nodes and compute nodes."21:52
anderbubblewith emphasis21:52
jrollright21:53
jrollI still don't know what controller/compute nodes are21:53
jrollat any rate21:53
jrollthe service catalog should work for all of those urls afaik21:53
jroll(if it's configured correctly)21:53
jrollwhich are the "keystone service-create" and "keystone endpoint-create" calls21:53
anderbubbleI see that now; thanks21:54
anderbubbleis the same true for the database connection?21:54
jrollno, db needs to be explicitly configured21:56
*** eghobo has joined #openstack-ironic21:57
*** achanda_ has joined #openstack-ironic22:04
*** marcoemorais has quit IRC22:04
*** marcoemorais has joined #openstack-ironic22:05
*** achanda has quit IRC22:08
*** trown is now known as trown|outttypeww22:10
*** igordcard has quit IRC22:15
*** marcoemorais has quit IRC22:15
*** todd_dsm has quit IRC22:17
*** andreykurilin_ has joined #openstack-ironic22:22
*** anderbubble has quit IRC22:22
*** Marga_ has quit IRC22:22
*** ociuhandu has quit IRC22:25
*** todd_dsm has joined #openstack-ironic22:27
*** ociuhandu has joined #openstack-ironic22:28
*** marcoemorais has joined #openstack-ironic22:29
*** ChuckC has quit IRC22:33
*** achanda_ has quit IRC22:34
dlaubehey guys, for those of you running openstack services configured to use rabbitmq, are you using a single vhost or separate vhosts per service?22:41
JayFNobodyCam: did you see jroll's devstack patch?22:41
JayFNobodyCam: I wonder if you're hitting firewall issues like the job is22:41
jrollmayyyyyybe22:42
NobodyCamhummm22:42
jrolldlaube: we use one vhost22:42
*** ChuckC has joined #openstack-ironic22:45
dlaubesame here, just making sure22:45
dlaubethanks jroll22:45
jrollnp :)22:45
*** romcheg1 has quit IRC22:46
*** todd_dsm has quit IRC22:46
*** spandhe has joined #openstack-ironic22:48
*** hemna is now known as hemna__22:57
*** achanda has joined #openstack-ironic22:58
*** ociuhandu_ has joined #openstack-ironic22:59
*** ociuhandu has quit IRC23:00
*** ociuhandu_ is now known as ociuhandu23:00
*** eghobo has quit IRC23:02
*** yuanying has joined #openstack-ironic23:07
BLZbubbawhere can I find a juno-compatible python-ironic?23:16
BLZbubbatesting 1 2 323:16
*** BLZbubba has quit IRC23:16
*** BLZbubba has joined #openstack-ironic23:16
rlooBLZbubba: for ubuntu?23:16
BLZbubbahello there, what does it take to get a juno compatible version of python-ironic?23:17
BLZbubbayes, 14.04 with the cloud-archive juno build23:17
BLZbubbait has a newer version of sql-alchemy that trusty's python-ironic hates23:17
BLZbubbasucks not to be a real openstack project anymore23:18
*** andreykurilin_ has quit IRC23:18
rlooBLZbubba: oh, yeah, I thought I saw someone else mention something about sql-alchemy. I'm not sure what they did. (Or was that you that mentioned it)23:19
BLZbubbait was me, i haven't been able to look at it for a couple of days23:20
BLZbubbai'm going to have to build my diskimage code on the juno baremetal version, i guess23:21
BLZbubbawhy did it get demoted?23:21
rlooBLZbubba: I don't know.23:22
rlooBLZbubba: don't know if this helps or not: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L13223:27
rloog'night all.23:30
*** rloo has quit IRC23:30
jroll23:18:16        BLZbubba | sucks not to be a real openstack project anymore <- eh?23:31
*** shikui__ has joined #openstack-ironic23:34
*** lucas-dinner has quit IRC23:37
*** annegentle has quit IRC23:42
*** annegentle has joined #openstack-ironic23:42
*** yjiang5 is now known as yjiang5_away23:43
*** Shrews has quit IRC23:46
*** annegentle has quit IRC23:46
*** annegentle has joined #openstack-ironic23:47
*** Shrews has joined #openstack-ironic23:47
*** annegentle has quit IRC23:48
*** annegentle has joined #openstack-ironic23:48
*** annegentle has quit IRC23:49
*** annegentle has joined #openstack-ironic23:49
*** Marga_ has joined #openstack-ironic23:50
*** Haomeng has joined #openstack-ironic23:53
*** Haomeng|2 has quit IRC23:54
spandhehey jroll ! I managed to make some progress.. have setup Neutron for now because it seems nova needs it.. so its going well..23:56
*** Marga_ has quit IRC23:57

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