Thursday, 2015-02-26

rfchapmanHow can I get the console output,  Can I do that through ironic?00:01
rfchapmanI guess I can do it from the iLo00:01
NobodyCamrfchapman: we do support console but I expect Ilo is easier and faster for you00:02
*** kbs has joined #openstack-ironic00:09
rfchapmanOK now it's not creating an instance at all.00:10
rfchapmanAhh,  there still is an instance uuid in ironic.00:10
rfchapmannobodycam: Can I just remove the uuid in the database?  probably not i would suspect that nova also has the nodes uuid in it's database as well.00:11
*** spandhe has joined #openstack-ironic00:12
*** eghobo has joined #openstack-ironic00:13
*** david-lyle is now known as david-lyle_afk00:14
NobodyCamrfchapman: some thing def going on00:15
*** hj-hp has joined #openstack-ironic00:15
NobodyCamand the reset / delete did not work?00:15
rfchapmannope00:15
*** hj-hp has quit IRC00:16
NobodyCamnova list shows just that one instance in error state00:16
rfchapmanYep00:16
NobodyCamand ironic node-list shows one node with a instance uuid in error state?00:17
*** hj-hp has joined #openstack-ironic00:17
rfchapmanIt shows the instance uuid, but the state is none.00:17
NobodyCamI'd stop the ironic conductor edit the db to remove reservation and instance id, restart the conductor00:18
NobodyCamand attempt to reset and delete thru nova again00:18
rfchapmanOK00:18
rfchapmanNobodyCam: nope no luck.00:20
NobodyCamwhat are you getting?00:21
rfchapmanError: you are not allowed to terminate instance: test00:21
NobodyCamoh00:22
NobodyCamcan you paste nova show test00:22
*** jlvillal_sc has joined #openstack-ironic00:24
rfchapmanhttp://paste.openstack.org/show/182182/00:24
NobodyCamhumm Flavor 13 could not be found.00:25
rfchapmanThat is interesting as I have never had a flavor 13..00:25
rfchapmanhttp://paste.openstack.org/show/182183/00:26
NobodyCamdo you get the error on the reset or delete command00:27
NobodyCamis there a reasonmy-baremetal-flavor is not public?00:27
NobodyCamreason why * :-p00:29
*** mdbooth has quit IRC00:29
rfchapmanRequest to delete server 187c4224-fbf0-416b-b014-73573a5b976a has been accepted.00:30
NobodyCamgive it a minute then check nova list00:31
NobodyCam:)00:31
rfchapmanOK that worked.... hummm why it get's tied up like that is a mistery.00:32
rfchapmanOK, launching new instance00:33
NobodyCamwhat the command you are using?00:33
rfchapmanI'm using the web interface00:35
NobodyCamahh can we use command line?00:35
*** mdbooth has joined #openstack-ironic00:35
NobodyCamif not its ok00:36
rfchapmanYes, I just havent gotten the hange of populating the deploy portion of ironic00:38
rfchapmanWhat is a resouce for posting a screenshot..  I couldn't grab text.00:39
NobodyCamoh. ummm00:41
NobodyCamhttp://imgur.com00:41
NobodyCamdont know but I have used them before00:41
*** Marga__ has quit IRC00:41
rfchapmanhttp://imgur.com/p7U61XM00:43
NobodyCamhumm: looking up that error00:45
NobodyCamhumm lets try just disabling the usb ports?00:46
NobodyCamhttp://www.redhat.com/archives/rhl-list/2006-December/msg03144.html00:47
NobodyCamyou running RH?00:47
rfchapmanYes, fedora 2000:48
NobodyCamthats really strange00:51
NobodyCamesp across three nodes00:51
NobodyCambrb00:51
NobodyCamrfchapman: what time zone are you in?00:57
*** BadCub has quit IRC00:58
rfchapmanpacific00:58
rfchapmanCalifornia00:58
NobodyCamAhh :) by chance could we pick up tomorrow morning :) as I too am in Cali and have an apt after work tonight00:59
rfchapmansure,  thanks for you help...  Have a nice evening and chat with you tomorrow.01:00
NobodyCamI'm usaly around by 7:00 am many days eailer then that01:00
NobodyCamyou to rfchapman :)01:01
rfchapmanI get in about 10-10:30...01:01
rfchapman:-)01:01
*** hj-hp has quit IRC01:02
NobodyCamhit me up will pick up then :)01:02
*** naohirot has joined #openstack-ironic01:02
NobodyCamlol *ping me and will pick up troubleshoting01:03
NobodyCamlol01:03
NobodyCamg'night all01:03
*** oomichi has joined #openstack-ironic01:04
naohirotgood night NobodyCam, good morning all01:04
*** spandhe has quit IRC01:07
mrdahi naohirot01:08
naohirotmrda: good morning :)01:09
*** spandhe has joined #openstack-ironic01:09
*** kbs has quit IRC01:10
*** eghobo has quit IRC01:14
*** jlvillal_sc has quit IRC01:17
*** eghobo has joined #openstack-ironic01:19
openstackgerritJosh Gachnang proposed openstack/ironic: Implement execute clean steps  https://review.openstack.org/15556101:19
*** eghobo has quit IRC01:23
*** cinerama has quit IRC01:23
*** eghobo has joined #openstack-ironic01:25
openstackgerritJosh Gachnang proposed openstack/ironic: Implement execute clean steps  https://review.openstack.org/15556101:26
*** cinerama has joined #openstack-ironic01:27
*** rwsu is now known as rwsu-afk01:27
*** hj-hp has joined #openstack-ironic01:31
*** hj-hp has quit IRC01:33
*** sambetts has quit IRC01:34
*** sambetts has joined #openstack-ironic01:35
*** yog_ has quit IRC01:49
*** ijw has quit IRC01:50
*** deepti-openstac1 has joined #openstack-ironic01:59
*** jlvillal_sc has joined #openstack-ironic02:05
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Fix PEP8 E121,E122,E123,E124,E125,E129 errors  https://review.openstack.org/15927902:13
*** jlvillal_sc has quit IRC02:16
*** deepti-openstac1 has quit IRC02:18
openstackgerritJosh Gachnang proposed openstack/ironic: Add /nodes/<uuid>/cleaning/steps API  https://review.openstack.org/15932202:24
*** ramineni has joined #openstack-ironic02:43
*** chlong_ has joined #openstack-ironic02:43
*** Shrews has quit IRC02:47
*** jiangfei has joined #openstack-ironic02:47
openstackgerritRuby Loo proposed openstack/ironic: Update documentation on VirtualBox drivers  https://review.openstack.org/15932402:49
openstackgerritAnusha Ramineni proposed openstack/ironic: Add localboot support for iscsi_ilo driver  https://review.openstack.org/15660803:03
*** yog_ has joined #openstack-ironic03:04
*** davidlenwell has quit IRC03:05
*** yuanying has joined #openstack-ironic03:10
*** davidlenwell has joined #openstack-ironic03:11
*** eghobo has quit IRC03:22
*** spandhe has quit IRC03:23
*** Nisha has joined #openstack-ironic03:24
*** rloo has quit IRC03:24
*** coolsvap_ is now known as coolsvap03:33
*** shrews has joined #openstack-ironic03:40
*** shrews is now known as Shrews03:46
*** Shrews has quit IRC03:53
*** Shrews has joined #openstack-ironic03:53
*** mgoddard1 has joined #openstack-ironic04:02
*** mgoddard has quit IRC04:04
*** wendar has quit IRC04:04
*** wendar has joined #openstack-ironic04:04
*** jiangfei has quit IRC04:09
*** jiangfei has joined #openstack-ironic04:10
*** spandhe has joined #openstack-ironic04:10
*** pensu has joined #openstack-ironic04:11
*** pensu has quit IRC04:11
*** pensu has joined #openstack-ironic04:12
*** pensu has joined #openstack-ironic04:12
*** killer_prince is now known as lazy_prince04:21
*** anderbubble has quit IRC04:25
*** rameshg87 has joined #openstack-ironic04:25
rameshg87good morning ironic04:31
openstackgerritYuiko Takada proposed stackforge/ironic-discoverd: Add scripts to manage translations  https://review.openstack.org/15898104:34
*** takadayuiko has joined #openstack-ironic04:36
openstackgerritRamakrishnan G proposed openstack/ironic: Add localboot support for iscsi_ilo driver  https://review.openstack.org/15660804:48
rameshg87ramineni, ^^04:48
*** spandhe has quit IRC04:49
raminenirameshg87: sure :)04:49
*** saripurigopi has joined #openstack-ironic04:50
*** coolsvap is now known as coolsvap_04:57
*** jmccrory has joined #openstack-ironic05:03
*** coolsvap_ is now known as coolsvap05:04
*** anderbubble has joined #openstack-ironic05:07
*** jmccrory has quit IRC05:13
*** jmccrory has joined #openstack-ironic05:13
*** jcoufal has joined #openstack-ironic05:18
*** anderbubble has quit IRC05:18
*** spandhe has joined #openstack-ironic05:24
*** jiangfei has quit IRC05:33
*** jiangfei has joined #openstack-ironic05:33
*** jcoufal has quit IRC05:39
*** jcoufal_ has joined #openstack-ironic05:39
*** jcoufal_ has quit IRC05:44
*** jmccrory has quit IRC05:47
*** oomichi has quit IRC05:47
*** achanda has quit IRC05:50
*** ParsectiX has quit IRC05:59
*** ParsectiX has joined #openstack-ironic06:00
*** ParsectiX has quit IRC06:04
*** ParsectiX has joined #openstack-ironic06:04
*** ParsectiX has quit IRC06:05
*** achanda has joined #openstack-ironic06:19
*** stendulker has joined #openstack-ironic06:20
*** spandhe has quit IRC06:21
*** stendulker has quit IRC06:41
*** stendulker has joined #openstack-ironic06:42
*** Marga_ has joined #openstack-ironic06:49
*** Marga_ has quit IRC06:49
*** Marga_ has joined #openstack-ironic06:50
*** yog_ has quit IRC06:51
*** yog_ has joined #openstack-ironic07:04
*** jcoufal has joined #openstack-ironic07:07
*** subscope has quit IRC07:14
*** dlpartain has joined #openstack-ironic07:26
*** Marga_ has quit IRC07:35
*** chlong_ has quit IRC07:42
*** chlong has quit IRC07:43
*** lintan has joined #openstack-ironic07:44
*** lintan has quit IRC07:45
*** lintan has joined #openstack-ironic07:46
lintanHi rameshg87,07:47
rameshg87lintan, hi07:47
lintanThanks for your comments about AMT, but I didn't really get your idea about the  mocked pywsman in third_party_driver_mocks.py07:48
rameshg87lintan, yeah https://review.openstack.org/#/c/152857/5/ironic/tests/drivers/third_party_driver_mocks.py07:49
rameshg87lintan, the reason why we would do reload(sys.modules['ironic.drivers.modules.amt']) is because we changes sys.modules['pywsman']07:50
rameshg87lintan, so it needs to be reloaded only if we changed sys.modules['pywsman'] to a mock07:50
lintanrameshg87, yes07:50
rameshg87lintan, yeah so you can move this into "if not pywsman block"07:51
*** ukalifon1 has joined #openstack-ironic07:51
rameshg87lintan, i meant it could be like this: http://paste.openstack.org/show/182322/07:52
lintanrameshg87, OK, I get your point07:54
lintanrameshg87, Thanks for this07:54
rameshg87wc :)07:54
lintanrameshg87, also great to know how to use paste.openstack :)07:55
rameshg87lintan, yeah, even i started using it recently :-;07:55
rameshg87lintan, it's good if you want to show someone some logs or tell some code07:56
lintanrameshg87, definitely, very helpful07:56
*** dlpartain has left #openstack-ironic08:07
*** achanda has quit IRC08:09
*** takadayuiko has quit IRC08:11
*** athomas has joined #openstack-ironic08:15
*** ifarkas has joined #openstack-ironic08:22
*** dtantsur|afk is now known as dtantsur08:26
dtantsurMorning!08:26
*** lazy_prince is now known as killer_prince08:27
*** jcoufal has quit IRC08:28
*** jcoufal_ has joined #openstack-ironic08:28
rameshg87dtantsur, o/08:28
dtantsuro/08:29
*** yog_ has quit IRC08:35
openstackgerritTan Lin proposed openstack/ironic: Add a note to dev-quickstart  https://review.openstack.org/14569008:46
openstackgerritTan Lin proposed openstack/ironic: Add a note to dev-quickstart  https://review.openstack.org/14569008:48
*** andreykurilin_ has joined #openstack-ironic08:49
*** jcoufal_ has quit IRC08:50
openstackgerritTan Lin proposed openstack/ironic: Add a note to dev-quickstart  https://review.openstack.org/14569008:50
*** yog_ has joined #openstack-ironic08:51
*** jistr has joined #openstack-ironic08:53
*** ndipanov has joined #openstack-ironic09:00
*** SpamapS has quit IRC09:04
*** SpamapS has joined #openstack-ironic09:05
*** romcheg has joined #openstack-ironic09:06
*** erwan_taf has joined #openstack-ironic09:08
*** SpamapS has quit IRC09:10
*** killer_prince has quit IRC09:12
*** jgrimm is now known as zz_jgrimm09:15
*** derekh has joined #openstack-ironic09:17
*** lucasagomes has joined #openstack-ironic09:28
rameshg87lucasagomes, hey09:29
lucasagomesrameshg87, hi09:30
rameshg87lucasagomes, i am looking at localboot + uefi09:30
rameshg87lucasagomes, i see your comment here: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L311-L31309:30
rameshg87lucasagomes, have you tried out some experiments ?09:31
rameshg87lucasagomes, we just need to create one more partition (efi system partition), mark the partition of appropriate type, make a vfat filesystem on it09:31
rameshg87lucasagomes, and then invoke grub with target as x86_64_efi, right ?09:31
erwan_tafhey lucasagomes09:31
*** SpamapS has joined #openstack-ironic09:32
*** SpamapS has quit IRC09:32
*** SpamapS has joined #openstack-ironic09:32
*** andreykurilin_ has quit IRC09:32
*** MattMan has left #openstack-ironic09:33
*** MattMan has joined #openstack-ironic09:33
lucasagomesrameshg87, hi haven't tried, it was out of the scope of the spec09:37
lucasagomeserwan_taf, hi there!09:37
rameshg87lucasagomes, okay09:38
lucasagomesrameshg87, but yeah, seems mostly correct. I think you also have to configure the boot menu via efibootmgr. Because IIRC now the uefi is the one that holds the menu entries09:39
lucasagomesrameshg87, there's a way to fool it too, which is what I think u wanna do09:39
lucasagomeshaving only 1 entry that calls grub and show the grub menu09:39
rameshg87lucasagomes, okay09:39
rameshg87lucasagomes, trying some experiments now09:39
lucasagomesrameshg87, but yeah, I left it out cause I don't have means to do it right now :(09:39
lucasagomesand time :(09:39
rameshg87okay :)09:40
rameshg87lucasagomes, will keep you posted :)09:40
lucasagomesrameshg87, ta much09:40
*** pas-ha has joined #openstack-ironic09:41
*** yog_ has quit IRC09:41
raminenidtantsur, lucasagomes: could you please have a look at https://review.openstack.org/#/c/129529/09:41
lucasagomeswill do, just replying some comments on the reviews09:48
lucasagomescan I get some eyes on https://review.openstack.org/#/c/155727/ someone?09:49
saripurigopiis there any wiki page to write tempest tests? I'm not very familiar with it. does writing new driver require tempest changes also?09:49
*** lintan has quit IRC09:51
*** lintan has joined #openstack-ironic09:52
*** yog_ has joined #openstack-ironic09:54
*** vdrok_afk is now known as vdrok09:55
vdrokmorning ironic09:55
*** naohirot has quit IRC09:56
openstackgerritShivanand Tendulker proposed openstack/ironic: Ilo drivers sets capabilities:boot_mode in node  https://review.openstack.org/15573109:57
openstackgerritAnusha Ramineni proposed openstack/ironic: Add Cleaning Operations for iLO drivers  https://review.openstack.org/15771509:59
*** achanda has joined #openstack-ironic10:01
openstackgerritDmitry Tantsur proposed openstack/ironic: Add lift_lock context manager to TaskManager  https://review.openstack.org/15939210:02
dtantsurlucasagomes, could you have a look at ^^^. One more dependency for discoverd inspection10:02
dtantsurvdrok, morning10:02
vdrokmorning dtantsur10:02
rameshg87saripurigopi, it doesn't require writing tempest tests .. you can do with unit tests for now10:04
lucasagomesdtantsur, ack10:04
rameshg87lucasagomes, removing perl from purge list on https://review.openstack.org/#/c/155727/4/Dockerfile was intentional ?10:04
saripurigopirameshg87, thanks for confirming that.10:04
lucasagomesrameshg87, yes tgt needs it10:05
*** killer_prince has joined #openstack-ironic10:10
*** killer_prince is now known as lazy_prince10:10
*** EmilienM|afk is now known as EmilienM10:11
stendulkerlucasgomes, dtantsur: Can you please review this defect fix https://review.openstack.org/#/c/155731/10:12
dtantsurwill plan it for today10:12
*** yuanying has quit IRC10:13
stendulkerdtantsur: thank you10:14
*** achanda has quit IRC10:17
openstackgerritTan Lin proposed openstack/python-ironicclient: Enable ironicclient with --ironic-api-version 1.x  https://review.openstack.org/15562410:19
openstackgerritMerged stackforge/proliantutils: HPSSA: Add disk allocator  https://review.openstack.org/15805210:21
*** pelix has joined #openstack-ironic10:21
*** yog_ has quit IRC10:43
openstackgerritNisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection  https://review.openstack.org/14982310:46
*** achanda has joined #openstack-ironic10:50
*** Nisha has quit IRC10:50
*** ramineni has quit IRC11:10
*** dtantsur is now known as dtantsur|bbl11:14
*** achanda has quit IRC11:17
*** lazy_prince has quit IRC11:17
*** pcaruana has joined #openstack-ironic11:19
*** rameshg87 has quit IRC11:21
*** killer_prince has joined #openstack-ironic11:21
*** killer_prince is now known as lazy_prince11:22
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add the image extension (for local boot)  https://review.openstack.org/15746911:37
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add iscsi extension  https://review.openstack.org/15572711:37
*** stendulker has quit IRC11:46
*** dguerri is now known as dguerri`afk11:47
*** chlong has joined #openstack-ironic11:51
*** chlong_ has joined #openstack-ironic11:51
*** saripurigopi has quit IRC11:52
*** pensu has left #openstack-ironic12:08
lucasagomesreally it updated both ? >.<12:11
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add the image extension (for local boot)  https://review.openstack.org/15746912:13
openstackgerritLucas Alvares Gomes proposed openstack/ironic-python-agent: Add iscsi extension  https://review.openstack.org/15572712:13
*** rodrigods is now known as rodrigods_12:16
*** rodrigods_ is now known as rodrigods__12:16
*** rodrigods__ is now known as rodrigod`12:16
*** rodrigod` is now known as rodrigods12:17
*** dtantsur|bbl is now known as dtantsur12:31
*** annegentle has quit IRC12:33
*** rameshg87 has joined #openstack-ironic12:39
*** rodrigods is now known as rodrigod`12:51
*** rodrigod` is now known as rodrigods12:53
*** kbs has joined #openstack-ironic13:19
*** chlong has quit IRC13:20
*** chlong_ has quit IRC13:20
*** dprince has joined #openstack-ironic13:23
*** yog_ has joined #openstack-ironic13:36
openstackgerritDmitry Tantsur proposed openstack/ironic: Add lift_lock context manager to TaskManager  https://review.openstack.org/15939213:41
dtantsurlucasagomes, ^^^ improved docstring13:41
lucasagomesdtantsur, +1 thanks, will see13:42
rameshg87dtantsur, one question in ^^^13:46
dtantsuryes13:46
rameshg87dtantsur, actually objects.Node.reserve() is the only problem, right ?13:47
rameshg87dtantsur, i mean we don't need to release anything else, right ?13:47
dtantsurrameshg87, that's the only thing to release :)13:47
dtantsurwhat exactly do you mean?13:47
rameshg87dtantsur, yeah ideally your lift just needs to do objects.Node.release() before yielding13:48
dtantsurrameshg87, well, it's nice to delete especially .node too13:48
rameshg87dtantsur, and do  objects.Node.reserve() after returning back, right ?13:48
dtantsur.. and assign all attributes - that's what I do13:48
rameshg87dtantsur, what all are we permitted to do after lifting the lock ? :-)13:50
rameshg87dtantsur, anything and everything ?13:50
dtantsurrameshg87, doing some unrelated business :D like calling into an external service13:50
dtantsurrameshg87, my use case it to call to discoverd13:51
rameshg87dtantsur, i thought it was for discoverd to do something on the node with ironic13:51
dtantsuryes, right13:52
*** kbs1 has joined #openstack-ironic13:52
rameshg87dtantsur, do you have dependent patch which makes use of this ?13:52
rameshg87dtantsur, so discoverd can add ports to the nodes during this time ?13:53
rameshg87dtantsur, while the lock is lifted ?13:53
rameshg87or rather will discoverd add ports13:53
dtantsurrameshg87, discoverd does some stuff even before starting inspection13:53
*** kbs has quit IRC13:54
dtantsurI'm not sure why exactly it deadlock now, but it makes sense to me to lift the lock anyway13:55
*** mjturek1 has quit IRC13:57
*** anderbubble has joined #openstack-ironic13:58
*** Nisha has joined #openstack-ironic13:58
rameshg87dtantsur, but it seems a strange usage to me13:58
*** mjturek1 has joined #openstack-ironic13:59
rameshg87dtantsur, we do acquire a lock with TaskManager.acquire()13:59
*** anderbubble has quit IRC13:59
dtantsurrameshg87, what is strange? what are your suggestions?13:59
rameshg87dtantsur, within that context we will be able to release it13:59
rameshg87dtantsur, and if required we would be able to acquire within second context again (but nobody would do it :-))14:00
dtantsurnot sure I understand what you suggest14:02
rameshg87dtantsur, didn't suggest anything useful actually14:02
rameshg87dtantsur, is it meant to be used between L111-L126 here: https://review.openstack.org/#/c/156562/6/ironic/drivers/modules/discoverd.py ?14:02
*** rloo has joined #openstack-ironic14:03
*** viktors|afk is now known as viktors14:04
dtantsurrameshg87, only around 113-11514:04
rameshg87dtantsur, so discoverd will immediately get back to ironic api after calling client.introspect(), is that the problem ?14:05
dtantsurrameshg87, yes. Actually it's a _possible_ problem for now, so it's more of precaution.14:05
dtantsurbecause _right now_ discoverd does not do any sync update. but it might14:05
*** Nisha has quit IRC14:06
rameshg87dtantsur, if it's doing sync update, why can't it return back something to client.introspect() itself14:06
rameshg87dtantsur, i assume it's all REST based14:06
*** Nisha has joined #openstack-ironic14:06
rameshg87dtantsur, so it could return in the response imeediately on the return14:07
dtantsurrameshg87, because it's not results14:07
dtantsurimmediately >= 10 minutes in case of bare metal :D14:07
dtantsurbut it does leave some markers in Node.extra (legacy, but I have to keep it for compatibility)14:07
dtantsurright now it does not happen in sync fashion though14:07
rameshg87dtantsur, oh but before that it would have long gone out of task_manager.acquire acquired in do_node_inspect no ?14:08
rameshg87s/it/we14:08
*** priteau has joined #openstack-ironic14:08
dtantsurrameshg87, right now - yes. I want to feel free to introduce sync operations without breaking Ironic.14:09
rameshg87dtantsur, but we can protect only client.introspect() here14:09
dtantsuryes14:10
lintanHi dtantsur14:10
dtantsuro/14:10
rameshg87dtantsur, what about other statements after this until we release the lock acquired (in Nishas patch do_node_inspect())14:10
dtantsurrameshg87, what's wrong with them?14:10
dtantsurthey don't cause a deadlock14:11
dtantsuroooooooooook we only allow issuing 'update' when provision_state is None. awesome.14:13
* rameshg87 still hasn't got the problem completely then14:13
rameshg87oh, i didn't mean this ;-)14:14
rameshg87dtantsur, so you wouldn't be able to update even when state = INSPECTING ?14:14
*** romcheg has quit IRC14:15
dtantsurrameshg87, imagine: ironic calls discoverd while holding lock, discoverd calls back before returning and gets NodeLocked, retries forever (because Ironic is still holding a lock. Ironic can't release lock until call returns) <-- this is the problem14:15
dtantsurrameshg87, and yes, seems like we can't update even in manageable state, because it's not None...14:15
jrollmorning all :)14:16
jrolldtantsur: that's pretty awesome14:16
jrollespecially now that we have available14:16
dtantsurah, no, it's checking on target state14:16
dtantsurso no problem with manageable, but indeed problem with inspecting14:16
jrolloh, ha14:16
dtantsurI have to whitelist it...14:16
rameshg87dtantsur, okay, so ironic calls client.introspect()14:17
rameshg87dtantsur, before client.introspect(), discoverd calls back, right ?14:17
dtantsurduring client.introspect()14:18
rameshg87dtantsur, before client.introspect() returns, discoverd calls back, right ?14:18
*** romcheg has joined #openstack-ironic14:18
rameshg87dtantsur, yeah14:18
dtantsurright14:18
rameshg87hmm ..14:18
*** anderbubble has joined #openstack-ironic14:20
rameshg87dtantsur, i strongly feel then discoverd shouldn't be allowed to talk to ironic back during this time. this could be handled otherwise - don't know that's my opinion. there are many options :)14:21
rameshg87dtantsur, if discoverd is looking for some data, client.introspect() call could provide that14:22
dtantsurrameshg87, discoverd has no means of figuring out when it's allowed to talk to Ironic14:22
rameshg87dtantsur, if discoverd wanted to update something, it can return it back to ironic in client.introspect() and ironic can update it in the same context14:22
dtantsur(especially since it can be and is used standalone)14:22
dtantsurrameshg87, how do you imagine it in standalone case?14:23
rameshg87dtantsur, what does standalone mean here ?14:23
dtantsurrameshg87, imagine with Juno Ironic that has no idea about inspection14:23
openstackgerritRuby Loo proposed openstack/ironic: Update documentation on VirtualBox drivers  https://review.openstack.org/15932414:24
rameshg87dtantsur, can't discoverd apis could be enhanced, right ?14:25
rameshg87dtantsur, i might actually tell something stupid here :)14:25
dtantsurrameshg87, it can, I just don't see point14:25
dtantsurchanging protocols is expensive, changing Ironic internals is cheap14:26
rameshg87dtantsur, v1 version of discoverd api works with ironic juno could make sync calls immediately back to ironic14:26
dtantsuryes, but there's no discoverd driver there, so it's not a problem14:27
rameshg87dtantsur, v2 version of discoverd api which works with ironic kilo has to interactively talk and get the work done14:27
rameshg87dtantsur, no side-channel talks :)14:27
openstackgerritTan Lin proposed openstack/ironic: Add AMT-PXE-Driver Power&Management&Vendor Interface  https://review.openstack.org/15285814:27
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partx after partioning the disk  https://review.openstack.org/14283814:27
rameshg87dtantsur, i don't know14:27
rameshg87dtantsur, temporarily releasing the lock can get someone else to acquire during that point of time14:28
rameshg87dtantsur, and our current context cannot get the lock back14:28
rameshg87dtantsur, i feel we could get to that state14:28
dtantsurrameshg87, so instead of landing 20 lines patch you suggest inventing new API version, updating the ironic spec, etc. What you would answer if you were me? ;)14:28
dtantsurrameshg87, what's the problem of not getting lock back? originally I intended to drop it completely before calling to discoverd :)14:28
rameshg87dtantsur, couldn't that happen ?14:28
dtantsurI changed my mind so that not to break calling code (potentially)14:29
rameshg87dtantsur, i mean imagine we release the lock temporarily during client.introspect()14:29
dtantsurbut I probably should get back to just dropping the lock competely14:29
rameshg87dtantsur, some other thread which tries to acquire during that time will be able to acquire it (image an operator call to power on/off the node)14:30
rameshg87dtantsur, or any other operation which is allowed at that point of time14:30
dtantsurok14:31
*** rameshg87 has quit IRC14:31
*** rameshg87 has joined #openstack-ironic14:32
rameshg87dtantsur: sorry got disconnected14:32
rameshg87dtantsur: i think it's too risky to allow to lift a lock after it was allowed (and expecting 100% to get back the lock)14:33
dtantsurrameshg87, sigh.. who said " expecting 100% to get back the lock"?14:34
rameshg87dtantsur: i meant Nisha's patch https://review.openstack.org/#/c/149823/26/ironic/conductor/manager.py : _do_inspect_hardware() was guaranteed a lock from which it started execution14:36
dtantsurI want to have chances of getting it back, so that I can report errors14:36
rameshg87dtantsur: and it called a driver's task.driver.inspect.inspect_hardware14:36
rameshg87dtantsur: _do_inspect_hardware() expected to retain the lock even when the driver call returned14:37
rameshg87dtantsur: because it has already been allowed the lock14:37
dtantsurI mean, it's all possible things, but I don't see any solution to these problems14:37
rameshg87dtantsur: but task.driver.inspect.inspect_hardware() lifted the lock and gave it to some other thread,14:37
rameshg87dtantsur: which seems very unfortunate to me ;-)14:38
dtantsuryeah. but again, I don't see a sane solution14:38
rameshg87hmm .. yeah14:39
rameshg87dtantsur: i was just saying this might lead to problem, perhaps we can get other's opinions too14:39
dtantsurI mean, I can avoid dropping lock right now, but it can cause discoverd breaking Ironic some time in the future. Or Ironic breaking itself, if it starts taking exclusive locks where it didn't14:40
dtantsurrameshg87, one more options is too move client call into a greenthread. then we'll lose nice error handling, but get rid of lock without breaking people14:42
dtantsurthat may be a viable solution14:42
rameshg87dtantsur: ah ..14:42
rameshg87dtantsur: which means the greenthread spawned for client call should start execution only after it's parent has released the lock, right ?14:43
*** zz_jgrimm is now known as jgrimm14:43
dtantsurrameshg87, it does not matter, because we no longer lock parent from returning. so thanks to retries the whole thing will work14:44
rameshg87dtantsur: ah yes, so the parent should return quickly then14:44
dtantsurexactly14:44
rameshg87dtantsur: but how do you know if you were ever able to contact discoverd14:45
rameshg87dtantsur: you might need to set last_error and states within the spawned greenthread14:45
dtantsurrameshg87, the if client call raises, we will try to acquire lock and record a failure14:45
dtantsuryes14:45
rameshg87great .. :)14:45
rameshg87+114:46
dtantsurrameshg87, thanks for bringing it! I'll abandon lift_lock patch and try approach with greenthread14:46
*** rameshg87 is now known as rameshg87-brb14:46
rameshg87-brbdtantsur: sure ..14:46
*** jgrimm is now known as zz_jgrimm14:51
NobodyCamgood morning Ironic14:56
*** BadCub has joined #openstack-ironic14:58
*** anderbubble has quit IRC14:58
lucasagomesNobodyCam, jroll morning15:00
NobodyCammorning lucasagomes :)15:00
jrollmorning NobodyCam lucasagomes :)15:00
NobodyCammorning jroll :)15:01
*** jrist has quit IRC15:02
lucasagomesNobodyCam, lemme see if I addressed ur comment inhttps://review.openstack.org/#/c/155728/15:05
lucasagomeslemme know*15:05
NobodyCamlucasagomes: looking now :)15:08
lucasagomescool, thanks15:09
dtantsurNobodyCam, morning!15:09
NobodyCammorning dtantsur :)15:13
NobodyCamlucasagomes: I'm good with your reply. thou I like the doc string for the ipmitool method example you used.15:14
lucasagomesNobodyCam, right, yeah I can update the docstring15:15
rameshg87-brblucasagomes: rloo: need your +2s (and hopefully +A) back again on https://review.openstack.org/#/c/156608/15:15
rameshg87-brblucasagomes: rloo: please have a look ...15:15
NobodyCamlucasagomes: ++ that and rameshg87's todo and I'll +215:15
NobodyCamthank you :)15:15
lucasagomesrameshg87-brb, ack will take a look15:16
lucasagomesNobodyCam, magic! ok15:16
lucasagomesthanks15:16
NobodyCam:)15:16
*** rameshg87-brb is now known as rameshg8715:16
NobodyCamawesome work lucasagomes :)15:16
lucasagomesty :)15:17
openstackgerritDmitry Tantsur proposed openstack/ironic: Add find_nodes() helper to the conductor manager  https://review.openstack.org/15910015:17
openstackgerritDmitry Tantsur proposed openstack/ironic: Add module for in-band inspection using ironic-discoverd  https://review.openstack.org/15656215:17
dtantsurrameshg87, ^^^15:18
rameshg87dtantsur: ack .. will have a look ..15:22
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partx after partioning the disk  https://review.openstack.org/14283815:31
*** igordcard has joined #openstack-ironic15:32
*** zz_jgrimm is now known as jgrimm15:35
NobodyCamrloo: want to take a look over https://review.openstack.org/#/c/156608 now has two +2's15:37
BadCubMorning folks15:38
NobodyCammornig BadCub :)15:38
rloorameshg87, NobodyCam: ok, looking now15:39
TheJuliaGood morning15:40
BadCubMorning NobodyCam and TheJulia15:40
rlooNobodyCam: do you know how the microversioning is meant to work?15:42
rlooNobodyCam: is adding localboot support for iscsi_ilo driver considered a new feature?15:43
NobodyCammoorning TheJulia15:43
NobodyCamrloo: it has a blue print/ spec15:44
rlooNobodyCam: too bad. I approved it. I would really like to understand what needs a bump in microversion and what doesn't, and *when* to bump it if there are several patches for a particular feature.15:44
NobodyCamrloo: that I think would be a great topic for the meeting15:45
rlooNobodyCam: I just want someone to write it down so I can refer to it. Too many things to remember when reviewing :-(15:45
NobodyCamrloo: I will see what I do to assist with that15:46
rlooNobodyCam: thank you!15:46
NobodyCam:)15:46
NobodyCamanyone know tan lin's irc handle?15:48
rameshg87BadCub: need some help15:48
devanandaNobodyCam: ^ is lintan15:49
devanandaalso, good morning15:49
rameshg87BadCub: need to set milestone target and priority for https://blueprints.launchpad.net/ironic/+spec/inband-raid-configuration15:49
NobodyCamdevananda: Good morning15:49
rameshg87BadCub: spec is submitting15:49
rameshg87devananda: o/15:50
*** igordcard has quit IRC15:50
NobodyCamlintan: are you around?15:51
dtantsurdevananda, morning15:51
dtantsurmorning BadCub, TheJulia15:51
*** rameshg87 is now known as rameshg87-dinner15:52
*** jmccrory has joined #openstack-ironic15:54
NobodyCamanyone know what happened to the spec for: https://blueprints.launchpad.net/ironic/+spec/uefi-secure-boot-management-interfaces15:54
NobodyCamor what the correct BP for https://review.openstack.org/#/c/152858 is?15:55
openstackgerritNisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection  https://review.openstack.org/14982315:55
NishaNobodyCam, is there any plan to pursue the above blueprint further?15:56
NobodyCamNisha: I was looking at it late yesterday and could find the spec for it15:57
NobodyCam*and could NOT...15:57
*** igordcard has joined #openstack-ironic15:58
Nishaohk15:59
Nishaasked stendulker to login16:00
*** stendulker has joined #openstack-ironic16:00
stendulkerHi NobodyCam16:00
*** ukalifon1 has quit IRC16:00
NobodyCamhi stendulker16:01
stendulkerNobodyCam: You were looking for spec related to management interfaces for secure boot https://review.openstack.org/#/c/135845/16:01
*** kozhukalov has joined #openstack-ironic16:01
NobodyCamahh ha16:01
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE drivers support for IPA  https://review.openstack.org/15572816:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE driver: Deprecate pxe_deploy_{ramdisk, kernel}  https://review.openstack.org/15864416:02
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Local boot support for IPA  https://review.openstack.org/15772316:02
NobodyCamstendulker: ahh its adandoned16:02
stendulkerNobodyCam: We abandoned it on Feb 23 as we were not able to agree to the interfaces16:02
lucasagomesdevananda, rloo morning :)16:02
NobodyCamstendulker: ack16:02
NobodyCamI am going to note that on the BP and reviews16:02
stendulkerNobodyCam: ok16:03
* lucasagomes forgot to update the last patch16:03
NobodyCamstendulker: thank you :)16:03
*** mikedillion has joined #openstack-ironic16:03
rloooh hi lucasagomes, devananda, and everyone else :-)16:03
stendulkerNobodyCam: Have raised code review for secure boot implementation for iLO drivers. Please have a look https://review.openstack.org/#/c/153974/16:03
stendulkerNobodyCam: Do I need to update anyt other place  when spec is abandoned?16:04
stendulkerNobodyCam: Not aware of process around the same.16:05
*** pas-ha has quit IRC16:05
*** saripurigopi has joined #openstack-ironic16:05
NobodyCamstendulker: just to make sure its noted on BP and any reviews that may be up for it.16:05
NobodyCamI'll make sure they get updated today16:06
*** hj-hp has joined #openstack-ironic16:06
stendulkerNobodyCam: ok. Thank you16:07
NobodyCamstendulker: :) thank you ... I hope you repost once "L" opens16:07
openstackgerritLucas Alvares Gomes proposed openstack/ironic: PXE driver: Deprecate pxe_deploy_{ramdisk, kernel}  https://review.openstack.org/15864416:08
dtantsurso apparently https://github.com/openstack/ironic/blob/master/ironic/common/keystone.py#L125 get_admin_auth_token does not return a valid admin token >_<16:08
stendulkerNobodyCam: Sure. Will do that.16:08
dtantsurI wonder if someone really tested it...16:08
*** Marga_ has joined #openstack-ironic16:08
jrolldtantsur: wait, what? we don't rely on that?16:09
dtantsurjroll, it's used in one place... but maybe it's our misconfiguration, we use [keystone_authtoken]admin_tenant_name=service16:10
dtantsurso yes, it's really NOT an admin token16:10
*** mikedillion has quit IRC16:11
lucasagomesdtantsur, idk if it's a bug or a feature hah16:11
dtantsurjroll, looks somewhat related to https://review.openstack.org/#/c/156344/216:11
dtantsurlucasagomes, ^^^16:11
lucasagomesnot being an admin token and working seems good16:11
jrolldtantsur: oh, yeah, it's a config thing16:11
* lucasagomes is jk16:12
jrolldtantsur: e.g. admin_tenant_name isn't admin or service at rackspace16:12
dtantsurlucasagomes, but then we need one more set of credentials to access ironic-discoverd16:12
lucasagomesdtantsur, yeah, I'm kidding about it :)16:12
dtantsurheh16:12
dtantsurthis Keystone thing is the most cryptic peace of software I've ever used...16:12
jrolldtantsur: why is this a bug?16:13
jrollidgi16:13
jrollsomeone being able to misconfigure their admin creds doesn't seem like a bug to me16:13
dtantsurjroll, as explained in https://review.openstack.org/#/c/156344 we meed 2 settings: one for checking tokens (a service user) and one for accessing other services16:13
dtantsurbecause you're accessing other services on behalf of a real user (with 'admin' tenant(16:14
jrolldtantsur: I think I agree, though I thought we basically had different creds for every service16:15
jrollwe have configs for glance/neutron creds16:15
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397416:15
openstackgerritNisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection  https://review.openstack.org/14982316:16
openstackgerritShivanand Tendulker proposed openstack/ironic: Common changes for secure boot support  https://review.openstack.org/15397416:17
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for pxe_ilo driver  https://review.openstack.org/15480816:18
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for iscsi_ilo driver  https://review.openstack.org/15481416:18
openstackgerritShivanand Tendulker proposed openstack/ironic: Secure boot support for agent_ilo driver  https://review.openstack.org/15481616:18
*** jmccrory has quit IRC16:20
openstackgerritRamakrishnan G proposed openstack/ironic: Add driver interface for RAID configuration  https://review.openstack.org/15523016:21
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Fix PEP8 E121,E122,E123,E124,E125,E129 errors  https://review.openstack.org/15927916:21
*** dprince has quit IRC16:22
openstackgerritVladyslav Drok proposed openstack/ironic: Support for non-Glance image references  https://review.openstack.org/13674116:32
dtantsurso I assume ironic-discoverd is the first admin-only API we access, otherwise we would see such problems earlier...16:33
lucasagomesjroll, when you get a time: https://review.openstack.org/#/c/155727/ J's have been invoked there16:33
jlvllal_remdtantsur: Thanks for catching my debug test file I accidentally included in my patch :(16:34
jlvllal_remWhoops!16:34
jrolllucasagomes: lol16:34
lucasagomes:)16:34
*** viktors is now known as viktors|afk16:34
jrolllucasagomes: perl is needed at runtime?16:35
lucasagomesjroll, it's needed for tgt16:35
jrolllame16:35
lucasagomesif u purge it you delete tgt16:35
lucasagomesI know16:35
NobodyCamnice :-p16:35
lucasagomesjroll, https://review.openstack.org/#/c/155727/3/Dockerfile16:35
lucasagomestgt depends on libconfig-general-perl[1], that depends on perl[2]16:35
*** david-lyle_afk is now known as david-lyle16:36
jrollboom16:36
jrollapproved16:36
lucasagomeso/16:36
jroll:)16:36
lucasagomesboom is right sr!16:36
*** Nisha has quit IRC16:37
NobodyCamlucasagomes: 155728 +2'd16:37
lucasagomes\o/16:37
*** Nisha has joined #openstack-ironic16:37
NobodyCamhehehe :)16:38
* rameshg87-dinner goes to sleep16:38
rameshg87-dinnergood night all :)16:38
NobodyCamnight rameshg87-dinner16:38
dtantsurjroll, btw it's not our misconfiguration, the whole tripleo suffers from it: https://github.com/openstack/tripleo-image-elements/blob/master/elements/ironic/os-apply-config/etc/ironic/ironic.conf#L3816:38
lucasagomesrameshg87-dinner, g'night!16:38
dtantsurg'night, rameshg87-dinner16:38
*** rameshg87-dinner has left #openstack-ironic16:39
jrolldtantsur: then it's tripleo's misconfiguration16:39
jrollbut like16:39
dtantsurand git blame shows NobodyCam :)16:39
jrollif you want an admin token, configure your admin creds properly16:39
NobodyCamme16:40
jrollI don't actually understand what the problem is16:40
dtantsurhttps://github.com/openstack/tripleo-image-elements/blame/master/elements/ironic/os-apply-config/etc/ironic/ironic.conf#L3816:40
dtantsurNobodyCam, ^^^16:40
NobodyCamoh thats from Aug 30, 201316:40
openstackgerritJohn L. Villalovos proposed openstack/python-ironicclient: Fix PEP8 E121,E122,E123,E124,E125,E129 errors  https://review.openstack.org/15927916:40
dtantsurwe use these credentials for accessing other services16:40
dtantsurbut you can't access admin API with such credentials...16:40
jrolldtantsur: which services, specifically16:40
dtantsurjroll, glance, neutron and now ironic-discoverd16:41
jrollO.o16:41
jlvllal_remrloo: On the format patch.  I was trying to not make any changes unrelated to the specific errors.   But I justl re-rolled the patch to clean up the one you pointed out.  The other lines violated E128, which I wasn't trying to fix in this patch set.16:41
jlvllal_remrloo: But I agree it was ugly!16:41
dtantsurwell, maybe not always, because sometimes we just proxy token from context16:41
dtantsurso it's only for use e.g. in periodic tasks16:41
*** Marga_ has quit IRC16:42
dtantsurjroll, NobodyCam, here's where we use it: https://github.com/openstack/ironic/search?utf8=%E2%9C%93&q=get_admin_auth_token16:42
rloojlvllal_rem: thx. Yeah, it makes sense to localize your changes, but you have to look at the change in the context it is in.16:42
jlvllal_remrloo: Thanks for pointing it out :)  Looks better now.16:43
dtantsurso only glance (and Ironic itself, which should fail)16:43
jrolldtantsur: neutron too: https://github.com/openstack/ironic/blob/master/ironic/dhcp/neutron.py#L8116:43
BadCubrameshg87 spec adjusted.16:43
rloojlvllal_rem: :D16:43
jrolldtantsur: I guess I might not understand the difference between admin and service accounts, but idk if I see a problem16:43
dtantsurright. so service account does not have admin rights in admin tenant16:44
dtantsurthat's why you can't call admin API with such token (though you can call non-admin like Glance)16:44
*** anderbubble has joined #openstack-ironic16:46
*** pcaruana has quit IRC16:50
*** rwsu-afk is now known as rwsu16:50
NobodyCamdtantsur: humm guess that should have been admin and not service :(16:52
NobodyCamhttps://github.com/openstack/tripleo-incubator/blob/master/seedrc#L516:52
dtantsurwell maybe...16:53
dtantsurI see other services using 'service' as well, don't know16:53
*** stendulker has quit IRC16:53
NobodyCamdtantsur: I believe at one point tripleO used service16:53
dtantsurmaybe we're not checking the credentials in The Right Way, but as no one knows The Right Way, I don't know what to do....16:53
lucasagomesrloo, +1 for docs in a diff patch, I wonder if we could remove the icehouse bits16:54
dtantsurnova is also using 'service'. how came they can access our API??16:55
rloolucasagomes: oh, do we still have icehouse? I think maybe we should leave icehouse until kilo is out.16:55
*** anderbubble has quit IRC16:55
*** Marga_ has joined #openstack-ironic16:55
lucasagomesin the install-guide there's some ref to icehouse16:55
lucasagomesack16:55
rloolucasagomes: can't recall what/how long the deprecation periods are, and how long we're supposed to provide support for releases.16:55
lucasagomesusually 1 cycle16:55
NobodyCamoh interesting: seed and undercloud use admin: overcloud uses demo and admin :/ https://github.com/openstack/tripleo-incubator/blob/master/overcloudrc-user#L5 & https://github.com/openstack/tripleo-incubator/blob/master/overcloudrc#L516:55
*** Marga_ has quit IRC16:56
rloolucasagomes: we can clean up the documentation after kilo-3. don't want to worry about that now.16:56
*** Marga_ has joined #openstack-ironic16:56
lucasagomesrloo, yeah, I gotta a bunch to write already (springs to mind, local boot, root device hints, ipa, configdrive)16:57
dtantsurNobodyCam, jroll, I see Nova has it's own credentials for Ironic (https://github.com/openstack/nova/blob/master/nova/virt/ironic/client_wrapper.py#L67-L70), I guess I need the same for discoverd....16:57
rloolucasagomes: i suppose you could open a bug for yourself so you don't forget ;)16:57
jrolldtantsur: yeah, I think that's the right thing to do16:58
dtantsuroh well, it's still service https://github.com/openstack/tripleo-image-elements/blob/master/elements/nova/os-apply-config/etc/nova/nova.conf#L22816:58
rloolucasagomes: unless all that stuff had DocImpact tags on them.16:58
lucasagomesrloo, they are part of the spec16:58
lucasagomesso idk if it's a bug16:58
lucasagomesI have it in a todo list here16:59
rloolucasagomes: ah, no need to open a bug then. we probably need to review all specs wrt documentation.16:59
lucasagomesthings I started but need to finsish16:59
lucasagomesyeah16:59
lucasagomesthe blueprints assigned to me16:59
lucasagomesI will check it at the end16:59
*** erwan_taf has quit IRC16:59
lucasagomesrloo, but anyway, tracking it is not the problem, writting it is :P17:01
* lucasagomes is not good writting documents in general17:02
rloolucasagomes: both. if we don't remember *what* needs to be written, we won't know if it has been written or not. but yeah, the harder part is writing it ;)17:02
rloolucasagomes: although it should be easier if the info is in the specs. but yeah, still harder than writing code ;)17:02
*** jistr has quit IRC17:02
lucasagomes:) yush17:06
*** lsmola has quit IRC17:06
*** dtantsur is now known as dtantsur|afk17:13
NobodyCamnight dtantsur|afk :)17:13
lucasagomesdtantsur|afk, g'night17:16
*** jmccrory has joined #openstack-ironic17:21
openstackgerritNisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection  https://review.openstack.org/15159617:21
openstackgerritMerged openstack/ironic-python-agent: Add iscsi extension  https://review.openstack.org/15572717:22
NobodyCamdoc update patch: has one +2 already :) https://review.openstack.org/#/c/15932417:23
rlooNobodyCam: just skimmed this wrt microversions, from nova http://docs.openstack.org/developer/nova/devref/api_microversions.html17:24
openstackgerritVladyslav Drok proposed openstack/ironic: Support for non-Glance image references  https://review.openstack.org/13674117:24
rlooNobodyCam: something like that would be good, once we know what we're doing ;)17:24
NobodyCamoh I will take a look at that17:24
*** deepti-openstack has joined #openstack-ironic17:25
*** vdrok is now known as vdrok_afk17:28
*** Marga_ has quit IRC17:38
*** igordcard has quit IRC17:38
*** Marga_ has joined #openstack-ironic17:38
*** jlvillal|afk is now known as jlvillal17:39
openstackgerritNisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection  https://review.openstack.org/15159617:39
NobodyCammorning jlvillal17:39
jlvillalNobodyCam: Morning! :)17:40
*** derekh has quit IRC17:40
NobodyCam:)17:40
*** jmccrory has quit IRC17:40
*** jmccrory has joined #openstack-ironic17:40
yuriyzmorning/evening Ironic17:42
NobodyCammorning yuriyz :)17:43
yuriyzlucasagomes, do you plan to work with https://blueprints.launchpad.net/ironic/+spec/pluggable-credential-storage ? I want write spec for this.17:44
lucasagomesyuriyz, go ahead!17:44
lucasagomesyuriyz, barbican seems cool :)17:44
lucasagomes+1 for the idea17:44
yuriyzthanks, will do17:45
NobodyCamanyone happen to know what the correct blueprint for this is : https://review.openstack.org/#/c/15285817:49
*** anderbubble has joined #openstack-ironic17:51
rlooNobodyCam: is it this: https://blueprints.launchpad.net/ironic/+spec/amt-pxe-driver17:54
*** Marga_ has quit IRC17:56
NobodyCamrloo: yep that looks like it.. now why is the Implements blueprint amt-pxe-driver link not working :-p17:59
NobodyCamrloo: Thank you17:59
rlooNobodyCam: yw. I was wondering the same thing...17:59
NobodyCamrloo: also I'm not seeing that bp listed here: https://blueprints.launchpad.net/ironic/18:00
rlooNobodyCam: I am wondering if it doesn't show bps that are completed18:01
NobodyCamoh18:01
rlooNobodyCam: which is lame. I think.18:01
NobodyCamya could be because the bp is marked Implemented18:02
*** kbyrne has quit IRC18:03
*** spandhe has joined #openstack-ironic18:03
rlooNobodyCam: I did a search for a bp that was implemented, and it didn't show up.18:05
*** kbyrne has joined #openstack-ironic18:05
NobodyCamahh so thats it :)18:05
rlooNobodyCam: although for that review's commit msg, it should be the title of the bp, not the bp 'url'18:05
rlooNobodyCam: but it still wouldn't work regardless.18:06
rlooNobodyCam: seems like a bad thing. There are times when I go back to look at at review that was already merged. Oh well.18:06
NobodyCamrloo: ya. and I try to compaire code to bp / specs and was having to "bug" hunt for it18:07
*** anderbubble has quit IRC18:07
rlooNobodyCam: wrt this particular review -- is the bp implemented or not? since there's a patch up?18:07
NobodyCammaybe we should get BadCub to change the Implemented status18:07
rlooNobodyCam: +1 if it isn't finished.18:08
NobodyCamrloo: based on the commit message I would tend to think it still has open items18:09
rlooNobodyCam: yeah, me too. But I didn't want to start looking cuz then I'd feel like reviewing it ;)18:09
NobodyCamdevananda: BadCub: do we know / when this BP was taged as Implemented? There are still reviews up for it18:10
lucasagomesI will call it a day (and leave it here in case someone wants to take a look https://review.openstack.org/#/c/155728 :) have a good night everyone!18:10
NobodyCamlucasagomes: will add to the list:) have a great night18:10
lucasagomesNobodyCam, you reviewed already :D thanks18:10
NobodyCamlol18:10
lucasagomessee ya!18:10
* lucasagomes runs to the train station18:11
*** lucasagomes has quit IRC18:11
devanandaNobodyCam: which one?18:12
openstackgerritMerged openstack/ironic: Add localboot support for iscsi_ilo driver  https://review.openstack.org/15660818:14
devananda!! ^ woot18:15
openstackdevananda: Error: "!" is not a valid command.18:15
*** deepti-openstack has quit IRC18:16
*** ChuckC has quit IRC18:17
jlvillalLOL18:17
* BadCub reads backscroll18:18
BadCubNobodyCam I will look into that one in a few. I am dealing with another atm18:18
NobodyCamdevananda: https://blueprints.launchpad.net/ironic/+spec/amt-pxe-driver18:18
openstackgerritMerged openstack/python-ironicclient: Remove unused OS_TEST_TIMEOUT variable  https://review.openstack.org/15540818:19
NobodyCamBadCub: Ack :) I think devananda may be looking into it too18:19
openstackgerritMerged openstack/ironic: Implement Cleaning States  https://review.openstack.org/15344418:19
devanandaNobodyCam: oh. I marked it as done. thought it was ...18:19
BadCubkk18:19
NobodyCamhehehe I think that hides it from searchs and stuff18:20
openstackgerritMerged openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680318:20
* BadCub pokes launchpad with a cattleprod18:20
NobodyCamI found it when looking at https://review.openstack.org/#/c/15285818:20
devanandathis is the ... yea, that one18:20
* devananda changes status18:21
NobodyCamThank you :) hehehe18:21
devanandaNobodyCam: do you remember where lucas had an etherpad with standalone-ironic testing info?18:22
*** athomas has quit IRC18:22
devanandaI think it was part of his ipxe work a while back18:22
NobodyCamgah I have 22 etherpads but thats not one of them :(18:23
*** romcheg has quit IRC18:25
*** ijw_ has joined #openstack-ironic18:37
*** ChuckC has joined #openstack-ironic18:40
openstackgerritMerged openstack/ironic: Improve last_error for async exceptions  https://review.openstack.org/15893818:44
*** deepti-openstack has joined #openstack-ironic18:45
*** Marga_ has joined #openstack-ironic18:46
*** romcheg has joined #openstack-ironic18:46
*** Marga_ has quit IRC18:47
*** ChuckC has quit IRC18:47
*** Marga_ has joined #openstack-ironic18:47
*** Marga_ has quit IRC18:48
*** Marga_ has joined #openstack-ironic18:48
*** coolsvap is now known as coolsvap_18:48
NobodyCamlol "g_g_grand_child.attr_add" great_great_grand_child got to luv it18:49
*** ChuckC has joined #openstack-ironic18:49
*** Marga_ has quit IRC18:49
*** saripurigopi has quit IRC18:49
*** Marga_ has joined #openstack-ironic18:49
*** kozhukalov has quit IRC18:50
*** anderbubble has joined #openstack-ironic18:55
NobodyCamoh devananda Nova has this rst file ( https://github.com/openstack/nova/blob/master/nova/api/openstack/rest_api_version_history.rst ) think its worth us having something like it? btw I found it on this doc ( http://docs.openstack.org/developer/nova/devref/api_microversions.html ) which ruby and I both think would be nice for  us to have too19:08
devanandaNobodyCam: yea, that's a good idea19:09
NobodyCam:)19:10
openstackgerritMerged openstack/ironic: Update documentation on VirtualBox drivers  https://review.openstack.org/15932419:15
*** priteau has quit IRC19:15
*** coolsvap_ is now known as coolsvap19:21
*** pelix has quit IRC19:31
devanandarandom heads up to folks -- ubuntu 14.10 decided to move pxelinux.0, so all our install scripts and docs will be wrong19:33
devanandahttp://paste.openstack.org/show/182616/19:33
*** anderbubble_ has joined #openstack-ironic19:36
*** purp has joined #openstack-ironic19:37
NobodyCamhappy happy joy joy19:37
*** anderbubble has quit IRC19:37
*** anderbubble_ is now known as anderbubble19:37
*** hj-hp has quit IRC19:38
NobodyCamthat may also require a update to the correct dib element19:39
devanandahttp://docs.openstack.org/developer/ironic/deploy/install-guide.html#install-the-bare-metal-service is wrong for ubuntu 14.1019:44
*** achanda has joined #openstack-ironic19:45
openstackgerritJim Rollenhagen proposed openstack/ironic: Create new config for pecan debug mode  https://review.openstack.org/15958919:45
NobodyCamdevananda: that section or http://docs.openstack.org/developer/ironic/deploy/install-guide.html#pxe-setup19:46
devanandaer, yea. bad paste ...19:46
NobodyCamlol.. I can add a 14.10 section... thou I need to figure out how to test.19:46
NobodyCamis that the only pachage you found that differs?19:47
NobodyCampackage even19:47
rloojroll: did you want to look at https://review.openstack.org/#/c/155728/ ?19:51
jrollrloo: I wouldn't mind, but can't right this minute19:51
jrollrloo: I trust y'all if you don't want to wait for me, otherwise I can review this afternoon19:52
rloojroll: ok, then I won't +A19:52
rloojroll: I'm not in any hurry ;)19:52
devanandaNobodyCam: so far19:52
NobodyCam:)19:52
*** deepti-openstack has quit IRC20:03
openstackgerritChris Krelle proposed openstack/ironic: Update install-guide for Ubuntu 14.10 package changes  https://review.openstack.org/15959720:13
NobodyCamdevananda: ^^^ I left syslinux-common20:14
*** hj-hp has joined #openstack-ironic20:18
NobodyCambrb20:19
openstackgerritJosh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces  https://review.openstack.org/15720820:21
*** Marga_ has quit IRC20:28
*** bandicot has joined #openstack-ironic20:28
openstackgerritJosh Gachnang proposed openstack/ironic: Implement Cleaning in DriverInterfaces  https://review.openstack.org/15720820:31
*** purp has quit IRC20:42
*** hj-hp has quit IRC20:44
NobodyCamJoshNang: Thank you!!!! Package just arived :)20:46
JoshNangNobodyCam: no prob! Glad it got there safe!20:47
NobodyCam:)20:47
NobodyCamyep looks all here :)20:47
* NobodyCam hopes the return address is correct :-p20:47
NobodyCamlol and only missed 2 calls for the whole time lol20:49
JoshNangheh20:49
*** purp has joined #openstack-ironic20:52
*** hj-hp has joined #openstack-ironic20:56
*** bandicot has quit IRC21:00
mrdaMorning Ironic21:00
NobodyCammorning mrda :)21:00
*** deepti-openstack has joined #openstack-ironic21:03
*** deepti-openstack has quit IRC21:04
*** deepti-openstack has joined #openstack-ironic21:04
*** deepti-openstack has quit IRC21:05
mrdao/21:11
*** purp has quit IRC21:11
*** coolsvap is now known as coolsvap_21:15
jrolldevananda: I suddenly wonder if our sideways job is broken :/ https://review.openstack.org/#/c/159588/21:22
jrollheya mrda :)21:22
NobodyCamjust was just looking at that21:23
*** ifarkas has quit IRC21:23
jroll2015-02-26 21:17:30.252 |     2015-02-26 20:55:23,012 15888 DEBUG    [tempest_lib.common.rest_client] Request (BaremetalNodesAdminTestJSON:test_list_baremetal_nodes): 500 GET http://127.0.0.1:8774/v2/3e619df3eabc47da977223aaff607324/os-baremetal-nodes/c1e1a163-867e-4ea8-9c29-270a6a609544 0.212s21:23
jrollGOOD21:24
jrollglad we caught that failure!21:24
* jroll sighs21:24
NobodyCam:(21:24
* jroll rechecks21:24
NobodyCamand such great msg: The server has either erred or is incapable of performing the requested operation.21:25
devanandajroll: erm...21:26
*** Marga_ has joined #openstack-ironic21:29
*** Marga_ has quit IRC21:33
*** achanda has quit IRC21:34
*** achanda has joined #openstack-ironic21:35
devanandajroll: I dont see any api log entry at that timestamp (20:55:23.027) for an error21:38
openstackgerritVladyslav Drok proposed openstack/ironic: Support for non-Glance image references  https://review.openstack.org/13674121:40
*** Marga_ has joined #openstack-ironic21:40
devanandaoh, found it in n-api21:40
NobodyCamvdrok_afk: I wish that was < 1100 lines hehehehe21:41
devanandaalso, apparently, either nova isn't passing the os request id to ironic, or we're ignoring // not logging it21:44
devanandathat would make debugging easier ...21:44
*** hj-hp has quit IRC21:48
openstackgerritNisha Agarwal proposed openstack/ironic: Generic changes for Node Inspection  https://review.openstack.org/14982321:49
devanandajroll: what makes you think a recheck will pass?21:49
*** dmellado has quit IRC21:52
*** hj-hp has joined #openstack-ironic21:53
*** hj-hp has quit IRC21:54
jrolldevananda: idk, random hoping, I'm tied up in other things at the moment and don't really have time to look :/21:58
*** rwsu has quit IRC22:06
*** spandhe has quit IRC22:13
*** ijw_ has quit IRC22:14
*** ijw has joined #openstack-ironic22:15
*** ijw has quit IRC22:15
*** BadCub is now known as BadCub_Away22:19
*** jgrimm is now known as zz_jgrimm22:20
jlvillalWhat are recommendations for how to run devstack inside a VM?  I've been trying uvt-kvm (an Ubuntu utility).  But I am always failing when trying to create an instance.  So I am looking to try a different method to see if I can succeed.22:22
*** rwsu has joined #openstack-ironic22:30
*** ijw has joined #openstack-ironic22:31
devanandajlvillal: virt-manager + qemu-kvm22:31
jlvillaldevananda: Thanks.  And is virt-manager the GUI front-end to virsh?22:32
jlvillalOr are they unrelated?22:32
devanandajlvillal: yes, virsh22:35
devanandabbiab22:35
NobodyCam:)22:35
jlvillaldevananda: Thanks for the help :)  Hopefully it works better than uvt-kvm22:35
openstackgerritMerged openstack/ironic: Create new config for pecan debug mode  https://review.openstack.org/15958922:39
*** rodrigods is now known as rodrigod`22:42
*** rodrigod` is now known as rodrigods22:43
*** BadCub_Away is now known as BadCub22:44
*** rodrigods is now known as rodrigod`22:51
*** rodrigod` is now known as rodrigods22:51
*** achanda has quit IRC22:51
*** andreykurilin_ has joined #openstack-ironic22:53
*** romcheg1 has joined #openstack-ironic23:00
*** romcheg has quit IRC23:01
*** achanda has joined #openstack-ironic23:02
*** spandhe has joined #openstack-ironic23:03
NobodyCamanyone have the link for the agent prebuild images?23:06
NobodyCamI had it23:06
*** Nisha has quit IRC23:06
JoshNangNobodyCam: i believe it is http://tarballs.openstack.org/ironic-python-agent/coreos/ipa-coreos.tar.gz23:07
*** mgoddard has joined #openstack-ironic23:07
NobodyCamahh TY23:08
JoshNangnp23:08
*** mgoddard1 has quit IRC23:10
openstackgerritJosh Gachnang proposed openstack/ironic: Implement execute clean steps  https://review.openstack.org/15556123:10
*** yuanying has joined #openstack-ironic23:10
*** andreykurilin_ has quit IRC23:15
*** andreykurilin_ has joined #openstack-ironic23:15
NobodyCamdevananda: your playing with Support for non-Glance image references ( https://review.openstack.org/#/c/136741 )23:20
devanandaNobodyCam: have not tested it, no23:21
NobodyCamack23:21
*** dmellado has joined #openstack-ironic23:24
NobodyCamany ipa folk have a free minute for : https://review.openstack.org/#/c/15572823:24
*** andreykurilin_ has quit IRC23:26
*** andreykurilin__ has joined #openstack-ironic23:26
devanandaNobodyCam: that's needed for iscsi-based deploy of non-glance. we're playing with IPA's ability to fetch from any URL directly23:27
*** trown is now known as trown|outttypeww23:27
*** anderbubble has quit IRC23:30
JayFSo much interesting stuff happening here, so little time to review and appreciate it all23:30
NobodyCamdevananda: :) I was hopping for a free test of the 1100+ lines of that patch :-p23:31
NobodyCamhehehe23:31
devanandaahh23:31
NobodyCam:-p23:31
devanandaJayF: iknow ...23:31
devanandaJayF: it's kind of awesome. but also OMGTOOMUCHTODO23:31
JayFI've been at an internal conference all week23:32
JayFpreaching the gospel of doing everything (including downstream patches) in the open23:33
* NobodyCam offers support to JayF :)23:33
JayFNobodyCam: I think you have your own places to preach that gospel :P23:34
JayFand about 10x as many people to tell23:34
JayFlol23:34
NobodyCamlol JayF I do!23:34
* mrda wishes he got to SAT to hear JayF23:38
devanandaJayF: don't suppose there's a recording of you saying that somewhere?23:40
JayFdevananda: no, and I can't share the slides either because it's generally internal23:40
JayFdevananda: but let me show you the tweet that's relevant quoted from the talk23:40
JayF(i.e. I'm not sharing a slide deck that calls out lots of the stuff we do badly :P)23:41
devanandaJayF: fair23:41
JayFdevananda: https://twitter.com/filler/status/57063698650917683223:41
JayF“Youre gonna have code downstream, so prep the path for upstream. Be clear w/ the community, code in Gerrit, talk on IRC.” @jayofdoom #raxio <-- save a click23:42
* mrda hopes the message was listened to :)23:43
*** spandhe has quit IRC23:44
*** spandhe has joined #openstack-ironic23:49
*** EmilienM is now known as EmilienM|afk23:49
NobodyCamgreat Nic JayOfDoom23:50
JayFdooooooooooooooooom23:51
JayFthat's my github name too; shouldn't be the first time you saw it23:51
*** chlong has joined #openstack-ironic23:53
* NobodyCam thinks he should reinstall Doom and Break out the level editor.. Doom - Cloud City 23:54
openstackgerritJosh Gachnang proposed openstack/ironic: Add /nodes/<uuid>/cleaning/steps API  https://review.openstack.org/15932223:58
*** andreykurilin__ has quit IRC23:59

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