Tuesday, 2015-04-14

*** achanda has quit IRC00:02
*** romcheg has quit IRC00:12
*** naohirot has joined #openstack-ironic00:13
*** mtanino_ has joined #openstack-ironic00:24
*** davideagnello has quit IRC00:24
*** mtanino has quit IRC00:25
*** saripurigopi has joined #openstack-ironic00:26
openstackgerritDevananda van der Veen proposed openstack/ironic: Open Liberty development  https://review.openstack.org/17127400:27
mrda\o/00:35
*** ijw has quit IRC00:35
*** zhenguo has joined #openstack-ironic00:39
openstackgerritjxiaobin proposed openstack/ironic-specs: Mount config drive as loop device to supply data to cloud-init  https://review.openstack.org/17314200:59
*** meghal has quit IRC01:02
*** jmank has quit IRC01:02
*** dttocs has quit IRC01:13
*** kkoski has joined #openstack-ironic01:15
openstackgerritjxiaobin proposed openstack/ironic-specs: Mount config drive as loop device to supply data to cloud-init  https://review.openstack.org/17314201:23
*** chenglch has joined #openstack-ironic01:25
*** jxiaobin has quit IRC01:29
*** kkoski has quit IRC01:33
*** kkoski has joined #openstack-ironic01:38
*** wuhg has joined #openstack-ironic01:53
*** jerryz has joined #openstack-ironic02:02
*** mtanino_ has quit IRC02:04
*** chenglch|2 has joined #openstack-ironic02:04
*** chenglch has quit IRC02:07
*** davideagnello has joined #openstack-ironic02:10
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: (WIP)validate input uuid from client side  https://review.openstack.org/17261802:13
*** achanda has joined #openstack-ironic02:21
*** saripurigopi has quit IRC02:21
*** thrash is now known as thrash|g0ne02:26
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261802:36
*** davideagnello has quit IRC02:41
*** ramineni has joined #openstack-ironic02:44
*** rloo has quit IRC02:53
*** wuhg has quit IRC03:06
*** ndipanov has quit IRC03:06
*** sambetts has quit IRC03:06
*** arif-ali has quit IRC03:06
*** wuhg has joined #openstack-ironic03:13
*** ndipanov has joined #openstack-ironic03:13
*** sambetts has joined #openstack-ironic03:13
*** arif-ali has joined #openstack-ironic03:13
*** achanda has quit IRC03:14
*** dttocs has joined #openstack-ironic03:19
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261803:20
openstackgerritMerged openstack/ironic: Open Liberty development  https://review.openstack.org/17127403:21
*** saripurigopi has joined #openstack-ironic03:22
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261803:23
openstackgerritMerged openstack/ironic: Fix typos in vendor-passthru.rst  https://review.openstack.org/17293403:39
openstackgerritMerged openstack/ironic: Update session headers during initialization of AgentClient  https://review.openstack.org/17215403:39
*** harlowja is now known as harlowja_away03:58
openstackgerritjxiaobin proposed openstack/ironic-specs: Mount config drive as loop device to supply data to cloud-init  https://review.openstack.org/17314204:08
*** chenglch|2 has quit IRC04:21
*** davideagnello has joined #openstack-ironic04:29
*** davideagnello has quit IRC04:34
*** krtaylor has quit IRC04:35
*** krtaylor has joined #openstack-ironic04:38
* naohirot white board cannot be loaded due to error https://etherpad.openstack.org/p/IronicWhiteBoard04:40
* naohirot it seems all etherpad cannot be loaded? I couldn't load it too, https://etherpad.openstack.org/p/IronicReviewDay04:43
mrda:(04:44
mrdanaohirot: both work for me04:46
naohirotmrda: really? It would be my proxy or cache problem, I'll try to check.04:48
mrdaCould be platform?  Are you on Linux?  What browser?04:48
naohirotmrda: error said "Error: Attempt to load undefined module. in https://etherpad.openstack.org/static/js/require-kernel.js (line 1)"04:48
mrdaBoth work for me right away04:49
mrdaAnd https://etherpad.openstack.org/static/js/require-kernel.js loads for me too04:50
pshigeme to. I can access both pages.04:51
naohirotmrda: It worked on another PC, so something wrong in my desktop pc.04:51
mrdaSorry to be the bearer of bad news...04:51
*** achanda has joined #openstack-ironic04:52
*** jcoufal has joined #openstack-ironic04:53
naohirotmrda: thanks! I could notice earlier :) now I'm going to quickly fill in my status.04:54
mrdanaohirot: np04:57
devanandao/04:57
mrdahey devananda!04:57
devanandaanyone around for the meeting? :)04:57
mrdaof course :) naohirot and I are here04:58
naohirotdevananda: good evening :)04:58
devanandak k04:58
*** rameshg87 has joined #openstack-ironic04:58
devanandai'm hanging on to wakefulness ;)04:58
rameshg87good morning ironic04:58
mrdahi rameshg8704:58
rameshg87hello mrda04:58
pshigedevananda: good evening :)04:59
*** oomichi has joined #openstack-ironic05:13
pshigeoomichi: weekly irc meeting is now on #openstack-meeting-305:15
*** dttocs has quit IRC05:15
*** dttocs has joined #openstack-ironic05:23
* devananda goes to bed05:25
rameshg87good night devananda05:26
pshigedevananda: good night05:26
*** lintan_ has joined #openstack-ironic05:26
openstackgerritYuiko Takada proposed stackforge/ironic-discoverd: Use keystonemiddleware auth credentials  https://review.openstack.org/17078205:42
Haomengdevananda: good night:)05:47
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261805:56
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261806:00
lintan_Hi Haomeng06:04
Haomenglintan_: pong:)06:04
lintan_I am a little confused about the state machine, do we still have zapping or not?06:05
*** chenglch has joined #openstack-ironic06:05
lintan_Haomeng, or we only have cleaning state?06:06
Haomenglintan_: I think so - https://github.com/openstack/ironic/blob/master/ironic/common/states.py#L27006:09
Haomenglintan_: :)06:10
*** devananda has quit IRC06:10
*** krtaylor has quit IRC06:11
lintan_thanks :) Haomeng06:11
Haomenglintan_: welcome:)06:11
*** devananda has joined #openstack-ironic06:17
*** krtaylor has joined #openstack-ironic06:22
*** dttocs has quit IRC06:29
*** dttocs has joined #openstack-ironic06:30
*** Marga_ has quit IRC06:33
pshigeHaomeng: we already have both cleaning and zapping state, but we have no real implementation of zapping at this moment, I think. Is it right?06:33
*** chenglch has quit IRC06:34
Haomengpshige: I understand we define *clean* status now, and have no such *zapping* state defination, so we handle both cleaning/zapping for same logic code06:34
*** dttocs has quit IRC06:35
pshigeHaemeng: Thank you06:36
Haomengpshige: welcome06:37
pshigeand there are no real implementation of zapping, not even HP iLO.06:39
*** kozhukalov has joined #openstack-ironic06:39
pshigeI wonder which of them is lintan's question ...06:42
Haomengpshige: :)06:44
*** ukalifon has joined #openstack-ironic06:50
openstackgerritTan Lin proposed openstack/ironic-specs: Bare Metal Trust Using Intel TXT  https://review.openstack.org/13390206:58
*** andreykurilin has joined #openstack-ironic07:00
*** yuriyz has joined #openstack-ironic07:00
*** jamielennox is now known as jamielennox|away07:02
*** chenglch|2 has joined #openstack-ironic07:05
*** Nisha has joined #openstack-ironic07:09
*** dmellado has quit IRC07:10
*** a1exhughe5 has joined #openstack-ironic07:12
*** andreykurilin has quit IRC07:13
*** dmellado has joined #openstack-ironic07:24
*** achanda has quit IRC07:26
*** chlong has quit IRC07:27
*** jistr has joined #openstack-ironic07:29
*** dttocs has joined #openstack-ironic07:30
*** dtantsur|afk is now known as dtantsur07:35
dtantsurMorning Ironic07:35
pshigedtantsur: morning07:35
*** dttocs has quit IRC07:35
mrdahey dtantsur07:40
*** ndipanov has quit IRC07:41
dtantsuro/07:41
*** ndipanov has joined #openstack-ironic07:42
*** ifarkas has joined #openstack-ironic07:42
openstackgerritchenglch proposed openstack/ironic-specs: Add console log support  https://review.openstack.org/16458607:49
*** pas-ha has joined #openstack-ironic07:50
*** yog__ has joined #openstack-ironic07:56
*** mgoddard has joined #openstack-ironic07:57
*** yog__ has quit IRC07:57
*** wuhg has quit IRC07:59
*** jcoufal_ has joined #openstack-ironic08:06
*** jcoufal has quit IRC08:09
*** derekh has joined #openstack-ironic08:10
*** kan__ has joined #openstack-ironic08:14
*** athomas has joined #openstack-ironic08:16
*** lintan_ has quit IRC08:17
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add new boot interface in Ironic  https://review.openstack.org/16869808:22
*** lucasagomes has joined #openstack-ironic08:26
*** edcragg has joined #openstack-ironic08:26
*** dttocs has joined #openstack-ironic08:32
*** yog__ has joined #openstack-ironic08:34
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add generic RAID configuration spec for liberty  https://review.openstack.org/17321408:37
*** mgoddard1 has joined #openstack-ironic08:38
*** dttocs has quit IRC08:39
*** mgoddard has quit IRC08:39
*** romcheg has joined #openstack-ironic08:45
openstackgerritRamakrishnan G proposed openstack/ironic-specs: Add inband RAID configuration spec for liberty  https://review.openstack.org/17321808:55
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261808:55
*** dtantsur is now known as dtantsur|brb09:06
*** pelix has joined #openstack-ironic09:10
kejiahumorning all09:23
a1exhughe5morning!09:24
kan__morning09:24
kan__@lucasagomes. Hi, lucas. I have a issue 1443003. It is about the usage of 'node_uuid' as the sort key in port list command. Could you please take a little time to have a look?09:25
kan__here is the link:https://bugs.launchpad.net/ironic/+bug/144300309:25
openstackLaunchpad bug 1443003 in Ironic "port-list --detail using 'Node UUID' as sort key and node-list --detail using 'Chassis UUID' as sort key fail" [Undecided,In progress] - Assigned to Kan (kansks)09:25
kan__thx09:25
lucasagomeskan__, hi there, oh... I will take a look soonish09:25
lucasagomesthanks for reporting it09:25
kan__thx :-)09:25
*** mgoddard1 has quit IRC09:29
*** mgoddard has joined #openstack-ironic09:32
edcragghi, does anyone have any experience setting the boot device for HP moonshot cartridges using the ironic ipmitool driver? power control works fine using double bridged ipmitool commands, but i haven't seen standard IPMI commands be able to set the boot device for individual cartridges in a chassis, even though ipmitool `chassis bootdev disk` returns with a relatively sensible 'Set Boot Device to d09:33
edcraggisk'. i know this is an ipmi issue rather than ironic, but wondered if anyone had any experience09:33
*** yuanying has quit IRC09:35
*** dttocs has joined #openstack-ironic09:35
*** dttocs has quit IRC09:42
*** ramineni1 has joined #openstack-ironic09:54
*** lifeless1 has joined #openstack-ironic09:54
*** ramineni has quit IRC09:57
*** naohirot has quit IRC09:58
*** lifeless has quit IRC10:02
*** harlowja_away has quit IRC10:02
*** dtantsur|brb is now known as dtantsur10:09
*** chenglch|2 has quit IRC10:13
*** saripurigopi has quit IRC10:14
*** Nisha has quit IRC10:18
tiagogomes_morning ironic, I am seeing this warning on nova-compute "760 WARNING nova.virt.ironic.driver [-] cpu_arch not defined for node '1956aca6-1f68-462c-9386-f92a2933411f'", however it is on the properties of the node http://paste.openstack.org/show/203833/10:22
tiagogomes_ah wait, the keyname shouldn't have the 'properties' prefix10:23
tiagogomes_I guess10:23
dtantsurtiagogomes_, morning, it should be just cpu_arch etc10:24
lucasagomestiagogomes_, yeah, did you define it in the flavor as well?10:33
tiagogomes_lucasagomes yes, it is working fine now10:33
lucasagomescool10:34
*** Haomeng|2 has joined #openstack-ironic10:36
*** dttocs has joined #openstack-ironic10:38
*** Haomeng has quit IRC10:39
*** ijw has joined #openstack-ironic10:43
*** ijw has quit IRC10:43
*** dttocs has quit IRC10:44
*** ijw has joined #openstack-ironic10:44
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Troubleshoot: Do not power off node if deployment fail  https://review.openstack.org/17293210:48
TheJuliaohh10:53
*** ramineni1 has quit IRC10:57
*** lsmola_ has joined #openstack-ironic10:58
*** ijw has quit IRC11:01
*** ijw has joined #openstack-ironic11:01
tiagogomes_I tried to provision a machine using nova, but I got the following error ""message": "No valid host was found. ", "code": 500, "details": "  File \"/usr/lib/python2.7/site-packages/nova/conductor/manager.py\", line 614, in build_instances"11:03
tiagogomes_The ironic node was created using the ssh_pxe driver11:03
tiagogomes_Any ideas?11:04
dtantsurtiagogomes_, 1. something wrong with Ironic nodes (check status); 2. mismatch between flavor and node properties11:06
*** thrash|g0ne is now known as thrash11:14
*** saripurigopi has joined #openstack-ironic11:15
*** ijw_ has joined #openstack-ironic11:15
*** ijw has quit IRC11:16
tiagogomes_dtantsur, those things look fine11:20
openstackgerritGopi Krishna S proposed openstack/ironic-specs: Add pxe_ucs driver spec for liberty.  https://review.openstack.org/17327111:21
openstackgerritGopi Krishna S proposed openstack/ironic-specs: Add pxe_ucs driver spec for liberty  https://review.openstack.org/17327111:24
* rameshg87 goes home 11:26
*** rameshg87 has quit IRC11:26
tiagogomes_I am not using the nova-conductor, is this required?11:27
dtantsurnova-conductor? not sure, but I would support it's something needed. lucasagomes ^^^?11:28
lucasagomesdtantsur, tiagogomes_ hmm nova-conductor is the guy talking to the db. I've never tried use nova without it...11:29
lucasagomesso I don't know11:29
lucasagomesI can give it a go tho11:29
lucasagomestiagogomes_, nothing in the logs?11:30
openstackgerritGopi Krishna S proposed openstack/ironic-specs: Add pxe_ucs driver spec for liberty  https://review.openstack.org/17327111:31
tiagogomes_just a warning:  WARNING nova.compute.manager [-] Bandwidth usage not supported by hypervisor.11:31
TheJulianova conductor performs the scheduling11:31
dtantsurTheJulia, morning11:32
TheJuliadtantsur: good morning11:32
tiagogomes_0 WARNING nova.scheduler.driver [-] [instance: 2ddd4a5b-b143-4d6e-ac71-5c29e6398d95] NoValidHost exception with message: 'No valid host was found.'11:32
TheJuliatiagogomes_: I suspect that might be enough for it to not schedule on the hosts.  Perhaps if we could see the information for the node your attempting to provision in nova and the information on one of the ironic nodes that nova should have scheduled the host on to?11:33
tiagogomes_TheJulia, http://paste.openstack.org/show/203839/11:37
*** dttocs has joined #openstack-ironic11:40
*** saripurigopi has quit IRC11:41
*** yog__ has quit IRC11:41
TheJuliatiagogomes_: Honestly haven't looked at nova hypervisor-show in a really long time, but checkout your memory settings, that doesn't seem quite right to me11:43
tiagogomes_The I didn't set the ram_allocation_ration and some other memory setting, but the error doesn't _appear_ to be related with that11:44
*** davideagnello has joined #openstack-ironic11:44
*** kan__ has quit IRC11:45
lucasagomestiagogomes_, btw, http://docs.openstack.org/developer/ironic/deploy/install-guide.html#configure-compute-service-to-use-the-bare-metal-service11:45
lucasagomessome tweaks we have to do on nova scheduler11:46
tiagogomes_lucasagomes, yes, I followed that guide, except for ram_allocation_ratio and reserved_host_memory_mb11:46
tiagogomes_I'll try with the memory settings11:47
*** dttocs has quit IRC11:47
*** jistr is now known as jistr|class11:47
lucasagomesok11:48
lucasagomeslet us know if that works for u11:48
* lucasagomes will grab a sandwich, bbl11:49
*** davideagnello has quit IRC11:49
lucasagomesTheJulia, morning :-)11:49
*** lucasagomes is now known as lucas-hungry11:49
TheJulialucas-hungry: good morning11:49
tiagogomes_nope, same error11:51
*** trown|outttypeww is now known as trown11:53
*** Marga_ has joined #openstack-ironic11:57
*** ukalifon has quit IRC11:58
TheJuliatiagogomes_: if memory serves, nova does compare all of the parameters, have you tried turning up the logging level at all?12:00
*** romcheg has quit IRC12:01
*** romcheg has joined #openstack-ironic12:02
*** ukalifon has joined #openstack-ironic12:03
tiagogomes_TheJulia, trying....12:03
*** pas-ha has quit IRC12:04
*** dprince has joined #openstack-ironic12:07
*** oomichi has quit IRC12:12
tiagogomes_2015-04-14 14:24:08.000 17623 INFO nova.filters [req-ccb3a40f-2378-43cb-8336-d3e26fe87d5e None] Filter AggregateInstanceExtraSpecsFilter returned 0 hosts12:16
tiagogomes_I assume this happens when the flavor specs don't match the ironic node properties12:17
tiagogomes_but they look the same to me http://paste.openstack.org/show/203839/12:18
*** pas-ha has joined #openstack-ironic12:18
TheJuliaHmm, i forgot the scheduler was a separate process12:27
TheJuliatiagogomes_: Seems like it still might be memory related, the hypervisor shows half it's ram in use, which means it can't schedule the requested flavor on to the machine12:29
Haomeng|2TheJulia: did you set ram_allocation_ratio =112:29
TheJuliatiagogomes_: ^^^12:29
Haomeng|2TheJulia: I see your ironic hypervisor memory_mb               | 1024, but memory_mb_used          | 51212:29
tiagogomes_TheJulia, yes12:29
tiagogomes_ah, sorry12:29
tiagogomes_that's old12:30
TheJuliaohh12:30
Haomeng|2TheJulia: after change nova.conf, have to restart nova process12:30
TheJuliatiagogomes_: ^^^12:30
Haomeng|2Haomeng|2: and check nova compute log to see if the ironic hypervisor is updated12:30
tiagogomes_http://paste.openstack.org/show/203846/12:30
tiagogomes_yes, I restarted every nova process12:31
openstackgerritHaomeng,Wang proposed openstack/python-ironicclient: validate input uuid from client side  https://review.openstack.org/17261812:31
Haomeng|2tiagogomes_: now looks fine, still not working now?12:31
tiagogomes_Haomeng|2 no12:31
Haomeng|2tiagogomes_: now | memory_mb_used          | 102412:32
Haomeng|2tiagogomes_: so need to delete nova instance12:32
Haomeng|2tiagogomes_: to release the resource12:32
tiagogomes_mm12:32
Haomeng|2tiagogomes_: I guess nova instance is in error status12:33
*** ijw_ has quit IRC12:33
Haomeng|2tiagogomes_: delete all nova instance which is in error status, which will still use the ironic hypervisor resource12:33
tiagogomes_ok12:33
tiagogomes_it still fails :(12:36
tiagogomes_| memory_mb               | 1024                                 |12:37
tiagogomes_| memory_mb_used          | 0                                    |12:37
*** edcragg has quit IRC12:41
*** a1exhughe5 has quit IRC12:41
*** tiagogomes_ has quit IRC12:41
*** edcragg has joined #openstack-ironic12:41
*** a1exhughe5 has joined #openstack-ironic12:41
*** tiagogomes has joined #openstack-ironic12:42
*** dttocs has joined #openstack-ironic12:43
*** lucas-hungry is now known as lucasagomes12:46
*** dttocs has quit IRC12:47
*** dttocs has joined #openstack-ironic12:47
Haomeng|2tiagogomes_: can you share the nova compute log to see details why can not match12:47
tiagogomesHaomeng|2, it doen't say the details that don't match: http://paste.openstack.org/show/203849/12:49
tiagogomesthat's nova-scheduler log12:50
tiagogomesnova-compute: http://paste.openstack.org/show/203850/12:52
*** ukalifon has quit IRC12:57
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Only sync power state if node is ACTIVE (DO NOT MERGE)  https://review.openstack.org/17329912:59
lucasagomesjroll, can't add you as reviewer due ur launchpad problem ^13:00
*** mitz has quit IRC13:01
lucasagomesI just want to trigger some discussions about it13:01
*** kkoski has joined #openstack-ironic13:01
*** jistr|class is now known as jistr13:03
*** mitz has joined #openstack-ironic13:04
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Only sync power state if node is ACTIVE (DO NOT MERGE)  https://review.openstack.org/17329913:06
trownlucasagomes: +1 on daredevil references13:06
lucasagomestrown, heh :-)13:06
lucasagomesgood series, I'm following it now13:06
TheJuliaAgreed, although it is surprisingly dark13:07
lucasagomes+113:08
lucasagomesafter miller daredevil became pretty dark13:08
*** romcheg has quit IRC13:14
*** kozhukalov has quit IRC13:20
*** romcheg has joined #openstack-ironic13:21
tiagogomesafter adding some debug prints, I think that my problem is due cpu_arch not being on the metadata dictionary13:33
*** coolsvap|afk is now known as coolsvap13:34
tiagogomesI am not sure how to add it though13:34
*** MattMan has joined #openstack-ironic13:34
tiagogomesdo why need to create an aggregate in addition to a flavor as well?13:36
jrollmornin'13:43
jrolllucasagomes: my thought is that the power state in the DB should always be (close to) up to date, because IPMI is slow13:43
jrolltiagogomes: the aggregate stuff is only if you want to run virt and baremetal in the same environment, afaik, it directs things to the correct compute13:44
jrollthere's other ways to solve it like cells13:44
trownjroll: have you tried out cells?13:46
lucasagomesjroll, so the other way around then?13:46
jrolltrown: yes, we run them in prod13:46
jrolllucasagomes: ?13:47
trownah, sweet, I will need to look into that13:47
* tiagogomes notes that is using Juno13:47
JayFdevananda: I did not make nor am I aware of an ironic-spec-core meeting13:47
lucasagomesjroll, if the node is off, instead of sync. Just update the db saying it;s off13:47
JayFdevananda: I think BadCub was trying to organize something; that's probably it?13:47
jrolllucasagomes: oh, right. so we run with force_power_state_during_sync=False13:48
lucasagomesjroll, right, yeah that's fine13:48
jrolllucasagomes: I think we should always run the periodic task, I think operators can decide whether to force the power state13:48
lucasagomes+113:49
jrollI think if a user powers off the machine, that's fine if that's what they want to do13:49
lucasagomesmy point on that patch is, when does it make sense to sync power state13:49
lucasagomessay node is in zapping state. Should we still run sync on that state?13:49
jrollwell, let's be clear on words13:49
jrollby 'sync' do you mean sync real power state to db, or sync db state to the node and change that real power state13:50
lucasagomessync the db state to the node and change that real power state13:50
jrollok13:50
jrollso I personally think we should never do that :)13:50
lucasagomes(which is what that force_power_state_during_sync does if it's True13:50
lucasagomesjroll, reason? I mean I see some benefits on having it, say the node is deployed and someone just turned it off accidentaly13:51
lucasagomesso ironic bringing it on again seems fair13:52
jrolllucasagomes: then they call nova start or whatever13:52
jrollI don't want my computers turning on by themselves13:52
*** r-daneel has joined #openstack-ironic13:52
lucasagomesright, yeah nova start is fair :-/13:53
* lucasagomes wonders even more about that option now13:53
jrollunless you mean someone in the DC turned it off, in which case I have some four letter words13:53
lucasagomeslol13:53
*** absubram has quit IRC13:54
lucasagomesas I stated in the patch. Right now that config option tries to sync the state (real state on the node) for almost all states13:54
lucasagomesexcept DEPLOYWAIT, or if it's maintenance13:54
lucasagomesnow... it seems wrong. And we can: 1) remove that option, not force sync state anymore.13:55
lucasagomes2) constraint the scope of that, to only act upon some certain states13:55
lucasagomesthat patch does 2)13:55
jrollright13:55
lucasagomesso idk :-) that's why I put it up there. To trigger some discussion13:56
jrollI guess I don't see why other states are a problem?13:56
jrollwhen ironic changes the state it updates the db13:56
jrollso really the only reason to sync is if someone is messing with power state outside of ironic13:56
lucasagomesyeah it seems ok. It just that we were talking about it downstream and I decided to bring it upstream see if more people has opnions on that13:57
lucasagomesother states like enroll, may not even have right credentials13:57
lucasagomesno reason why try to sync it13:57
jrollyeah, true13:57
jrollI guess I just don't have a strong opinion13:57
* lucasagomes same13:57
lucasagomesanyhoo... just food for thought13:58
jrollyeah :)14:00
*** mtanino has joined #openstack-ironic14:04
NobodyCammorning Ironicers14:07
TheJuliaGood morning!14:08
NobodyCammorning TheJulia :)14:08
dtantsurmorning jroll, NobodyCam14:08
NobodyCammorning dtantsur jroll :)14:08
* TheJulia needs a large amount of coffee()14:09
*** dttocs has quit IRC14:12
BadCubJayF: Yes, I am trying to organize the meeting. I put it on the agenda for folks to discuss and say yes or no to14:13
jrollmorning NobodyCam TheJulia dtantsur and BadCub :)14:15
BadCubmorning jroll14:15
tiagogomesI am still struggling with the same error. Is any scheduler_default_filters that I need to use for Ironic14:15
NobodyCamhey hey jroll14:15
jrolltiagogomes: what error?14:17
jrolltiagogomes: there's a scheduler_use_baremetal_filters that may help14:17
*** zhenguo has quit IRC14:17
tiagogomesjroll, http://paste.openstack.org/show/203875/14:18
jrolltiagogomes: have scheduler and compute logs?14:19
kbs1NobodyCam, et. al.: FWIW, I found the source of the auth issue with IPA that I ping'd about yesterday. IPA is looking for the image in swift but it doesn't exist since my glance isn't configured that way. Kicks out an auth error when the object doesn't exist.14:19
jrollkbs1: :/14:20
tiagogomesjroll, http://paste.openstack.org/show/203876/, http://paste.openstack.org/show/203878/14:20
NobodyCamthe new google pony express Doodle is like a complete game!14:21
TheJuliagoodmorning BadCub14:22
BadCubmorning TheJulia14:22
NobodyCamkbs1: sounds like Ironic could use better error messages around that14:22
jrolltiagogomes: looks like your flavor needs extra_specs that match your aggregate14:22
tiagogomesjroll, which extra_specs does it need? I already added cpu_arch to the flavor14:24
tiagogomesbut I didn't create any aggregate14:24
jrolltiagogomes: it's failing on AggregateInstanceExtraSpecsFilter, so it sounds like you have an aggregate?14:26
jrollI really don't know nova well enough to say :(14:26
tiagogomes~ # nova aggregate-list14:26
tiagogomes+----+------+-------------------+14:26
tiagogomes| Id | Name | Availability Zone |14:26
tiagogomes+----+------+-------------------+14:26
tiagogomes+----+------+-------------------+14:26
tiagogomesI am following the Ironic installation guide, and there it doesn't talk about aggregates14:26
jrolltiagogomes: right, ok, so I think you should set scheduler_use_baremetal_filters=True14:27
kbs1NobodyCam: Yeah. Not obvious. Would be good to have ironic vet the url before handing it out. It happily drops the image ID in the url string and returns it for propagation to the agent. Early failure would be good here, IMO.14:27
tiagogomesjroll, it didn't work14:30
jrolltiagogomes: define didn't work14:30
tiagogomesjroll, sorry, exactly same error14:30
jrolltiagogomes: which scheduler filter returned 0 hosts, then?14:31
*** zz_jgrimm is now known as jgrimm14:32
tiagogomesoh, a different one : 2015-04-14 16:39:05.002 4540 INFO nova.filters [req-3ab6b1c8-f7a1-4f90-99b9-0b0d11dbd8e8 None] Filter RetryFilter returned 0 hosts14:32
jrollok, good, we're getting somewhere14:33
tiagogomes:)14:33
jrollI think that means it actually tried to build, can you check compute logs for errors?14:33
tiagogomesno errors on nova-compute14:34
jrolldoes it look like it tried to build something?14:34
*** Marga_ has quit IRC14:34
jrolloh, I wonder if just 0 hosts are going through the filters in the first place14:35
jrollcan you run 'nova hypervisor-stats'?14:35
tiagogomesjroll, http://paste.openstack.org/show/203881/14:37
tiagogomesThere was this on `nova show` before failing:14:37
tiagogomes| OS-EXT-STS:task_state                | spawning                                                   |14:37
tiagogomes| OS-EXT-STS:vm_state                  | building14:37
jrolltiagogomes: hrm, so it must have tried to build, at least14:38
jrollthat means it got to the compute14:38
tiagogomesjroll, cool, it did. I was just looking at the tail of the log : http://paste.openstack.org/show/203882/14:39
jrolltiagogomes: cool, so you're missing those keys in driver_info14:40
jrollthis is juno, correct?14:40
tiagogomesyep, pxe_deploy_ramdisk  pxe_deploy_kernel are empty fields14:41
tiagogomesyes Juno14:41
jrollyeah, so those are the kernel and ramdisk for your deploy ramdisk14:41
jrollwhether that's the DIB ramdisk or IPA14:41
*** saripurigopi has joined #openstack-ironic14:44
tiagogomes| provision_state        | deploying14:49
tiagogomes:)14:49
tiagogomesthanks jroll14:49
jroll\o/14:49
jrollnp tiagogomes :)14:49
*** absubram has joined #openstack-ironic14:53
*** dttocs has joined #openstack-ironic14:56
*** rloo has joined #openstack-ironic14:59
*** coolsvap is now known as coolsvap|afk15:00
*** pas-ha has quit IRC15:02
*** mtanino has quit IRC15:03
*** saripurigopi has quit IRC15:03
*** saripurigopi has joined #openstack-ironic15:05
*** mtanino has joined #openstack-ironic15:14
*** pas-ha has joined #openstack-ironic15:14
*** maurosr has quit IRC15:21
*** jgrimm has quit IRC15:21
*** jmank has joined #openstack-ironic15:24
openstackgerritChris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver  https://review.openstack.org/16745115:26
NobodyCambrb15:26
devanandamorning, all15:34
BadCubmorning devananda15:34
BadCubbrb15:34
lucasagomesdevananda, NobodyCam BadCub good morning15:35
lucasagomesdevananda, https://review.openstack.org/#/c/173379/15:35
lucasagomeswhen you get some time, this is for nova15:35
lucasagomeswe've discussed it a bit in the feature freeze15:36
devanandalucasagomes: why put networking data in node.extra['configdrive_metadata'] and not, say, in ports?15:38
*** jerryz has quit IRC15:39
devanandaalso, I'd like to point out how the keys used to store data in our JSON fields (like node.properties and node.extra) are, in fact, just part of the API15:39
lucasagomesdevananda, that is one example. But the idea is to have 1 endpoint where nova looks to get the data from15:40
lucasagomesnot multiples, which might complicate stuff15:40
lucasagomeshaving in ports is like useless to ironic because that's about instance configuration, not ports properties really15:40
jrollmorning devananda :)15:40
lucasagomesdevananda, right, yes, I would like to document it as such15:41
jrolldevananda: this is a fun one, would love to see your opinion: https://review.openstack.org/#/c/173142/15:41
lucasagomesthe same way we document the capabilities in node.properties for example15:41
NobodyCammorning devananda15:41
lucasagomeswe could document the extra['configdrive_metadata']15:41
devanandalucasagomes: document it - sure. my point is a side bar to the spec itself. we've gone and created non-discoverable portions of our API to "extend" it.15:42
lucasagomesdevananda, oh... I see what you mean15:42
devanandathere is no way to discover, from the REST API, that the node.properties resource should somehow contain a "capabilities" key to trigger some other special behavior15:42
devanandaditto for your proposal here with configdrive_metadata15:43
lucasagomesdevananda, yeah that's true, but again, nova also looks into the capabilties which is also in the json field. So I just keep the same15:43
jrollwe could break those out into tables15:43
jrolle.g. flavor extra_specs in nova is a separate table15:43
devanandathe difference is that, at least in principle, ironic can auto-populate the capabilities field15:43
lucasagomesdevananda, right, we could have it as an attribute for the node15:43
devanandaso the operator never has to actually know about it15:43
devananda(in principle)15:43
devanandabut for this, it sounds like something the operator needs to set in ironic directly15:44
lucasagomesright15:44
*** jgrimm has joined #openstack-ironic15:44
lucasagomesthat's a fair point15:44
lucasagomesbut again, the same for cpus, memory_mb, disk_gb etc15:44
lucasagomesall json field, requires operators to put it directly15:45
lucasagomesnon-discoverable and nova will looks into it15:45
devanandaindeed -- but again, at least in principle, ironic could auto-populate that info via inspection15:45
*** maurosr_ has joined #openstack-ironic15:45
lucasagomesyeas and the port example we do via discover too15:45
devanandayep15:45
*** dprince has quit IRC15:46
lucasagomeswe add it to the configdrive_metadata, if the iface A is connect to the data network we set it to be renamed to "X"15:46
devanandahmm. so again, it sounds like that should be stored on the port15:46
devanandanova driver, if using configdrive, should be able to collate the information ironic has about the hardware. no?15:47
lucasagomesdevananda, right, but then the nova driver will need to look at multiple parts of our api to get the data15:47
lucasagomesand sometimes we may not have a representation of that data in our api15:48
lucasagomessay disks, same example. But say we want to name the disk devices accordingly15:48
lucasagomeswe don't represent disks in our api15:48
dtantsurmorning devananda and g'night to everyone else :)15:48
*** dtantsur is now known as dtantsur|afk15:48
lucasagomesbut I still would like to have consistent naming for such devices15:48
*** ifarkas has quit IRC15:48
*** romcheg has quit IRC15:48
lucasagomesthe idea is be to generic15:49
*** romcheg has joined #openstack-ironic15:49
*** romcheg has quit IRC15:49
lucasagomesand the data there is not relevant for Ironic itself, since we don't care whether the port 1 will be renamed to X when deploying a node15:50
jrolllucasagomes: is this something --user-data could solve?15:50
lucasagomes(that's why I put on the extra field)15:51
lucasagomesjroll, not really, because it's operators that set it15:51
*** jistr has quit IRC15:51
lucasagomesor discover, not users15:51
jrolllucasagomes: right, ok, guess I'm curious why operators care15:51
*** saripurigopi has quit IRC15:51
lucasagomesjroll, we care about the network naming for e.g15:51
lucasagomesso that we now that, across all isntances in my deployment15:51
lucasagomesthe iface name X is connected to the data network for e.g15:52
lucasagomessome might have other use cases15:52
jrollhmm15:52
*** saripurigopi has joined #openstack-ironic15:52
jrollI mean, I agree it could be useful, I just don't see a use case15:52
jrollI might be missing something15:53
lucasagomesjroll, consistently naming devices is useful15:54
devanandalucasagomes: isn't that part of a much larger problem of 1) how ironic interacts with more complex networks, 2) that nova assumes 1 network == 1 port, 3) that we're not storing any information about the physical links, aside from MAC, to supply to neutron?15:54
* lucasagomes otherwise udev perisistent rules wouldn't exist15:54
jrolllucasagomes: I mean, we do that today without this feature15:55
lucasagomesjroll, how ?15:55
* lucasagomes looks for alternatives15:55
jrolllucasagomes: it might just be that we're bonding both interfaces together15:55
jrollwhich doesn't help you15:55
jrollsorry, in a call at the same time15:55
lucasagomesdevananda, yes but that is for the deployment right? that data is post configuration15:56
lucasagomesjroll, right, yeah...15:56
* jroll needs to step away for a bit15:57
devanandalucasagomes: both15:57
*** krtaylor has quit IRC15:57
devanandalucasagomes: in non-flat network environments, we need to be able to supply the instance with enough information for its networks to be configured properly15:58
BadCubdemolition crew just arrived. Will be bouncing in-and-out for the rest of the day15:58
devanandaBadCub: mmm, fun. ttyl!15:58
NobodyCam:-p15:58
TheJuliaBadCub: The archiological dig?15:58
* BadCub wonders if there is a new definition of "fun" lol15:58
BadCubTheJulia: nope. Trees and walls15:59
TheJuliaBadCub: Enjoy!  Oh and yes, there are many definitions of "fun"15:59
lucasagomesdevananda, right, but this is something that is already passed to the config drive by nova? we pass the network_info to it15:59
lucasagomesBadCub, heh enjoy it!15:59
BadCubTheJulia: lucasagomes as long as they don;t damage the power lines, we are good hehehehe16:00
lucasagomes+1 :D16:00
TheJuliadevananda: provide network info to what?  Neutron/Infrastucture? or the host it's self?16:01
lucasagomesdevananda, although I feel that we are going a bit out of scope here. What I'm looking at is https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L605-L60616:01
lucasagomeshow me as an operator of Ironic can give my instances some information about the hardware I've so that it get it configured correctly16:01
devanandalucasagomes: right. and I'm questioning whether you, as an operator, should be telling ironic configdrive-specific formatted data, or whether you should be telling ironic about your hardware16:03
devanandalucasagomes: perhaps an example of what you'd propose to store in node.extra['configdrive_metadata'] would help16:03
devanandaTheJulia: both :)16:03
lucasagomesdevananda, right, we do it here https://github.com/stackforge/ironic-discoverd/blob/master/ironic_discoverd/plugins/edeploy.py#L136-L13816:04
lucasagomesI can get a sample of that16:04
TheJuliaFor the node deploying, I somehow suspect that people should be able to do whatever they really want/need, since they can format the configdrive how they wish and inject whatever they desire, I suspect the more important portion is the infrastructure16:05
devanandajroll: lucasagomes: re the "mount configdrive as loopback" proposal, I don't dislike your proposal (make an extension to IPA) too much, but it does move ironic into the realm of injecting files into instances, which I dislike out of hand.16:05
*** ijw has joined #openstack-ironic16:05
lucasagomesTheJulia, yes, configdrive already supports adding extra metadata16:05
jrolldevananda: right, I don't think we should touch the images, at all16:05
lucasagomesthat interface is an api to do it16:05
lucasagomesthat spec*16:05
jrolldevananda: I linked that to you because I thought we were all on board with that opinion, but maybe not16:06
lucasagomesdevananda, yeah :/ I hated nova when it injected network information16:06
lucasagomesdevananda, tho I see some usefulness on that spec16:06
lucasagomesand since it's optional as well makes me happier16:06
lucasagomesthe fact that IPA is trying to create a primary partition (where you can have only 4) is a real problem16:07
* jroll wonders what happens when the image is ntfs or jayfs or something I've never heard of16:07
lucasagomescause we don't specify how the disk images should be built16:07
lucasagomesjroll, it won't find fstab16:07
lucasagomesand won't do anything I assume16:07
jrollmaybe we do something weird, but we deploy coreos with a configdrive partition just fine16:07
lucasagomesfail deployment probably16:07
jrolllucasagomes: right, and so it's broken16:08
jrollyeah16:08
* TheJulia begins pondering juliafs where everything will be represented by delicious food16:08
lucasagomescause it can't inject the data16:08
jrollTheJulia++16:08
lucasagomesjroll, but again, we are doing some assumptions that breaks too16:08
lucasagomesthe room for an extra primary partition16:08
lucasagomesand MBR16:08
openstackgerritJohn L. Villalovos proposed openstack/ironic: Check temp dir is usable for ipmitool driver  https://review.openstack.org/16038316:08
jrolltrue16:08
jrollidk, just that touching user images seems really scary to me16:09
lucasagomesit does, yeah more discussion is needed whether we really should go that way16:09
lucasagomesbut at least, in that constraint scope, it seems a fair addition IMO16:09
jlvillalNobodyCam, ^^^  I did minor changes to your patch.  I hope you don't mind.16:09
jlvillalNobodyCam, And +1 from me on it16:10
NobodyCamjlvillal: I don't mind, however I think you rebased on an older version of 16038316:10
NobodyCam:-p16:10
jlvillalNobodyCam, really?16:11
*** jcoufal_ has quit IRC16:11
NobodyCamJenkins says the depends on is out of date16:11
*** krtaylor has joined #openstack-ironic16:12
jlvillalNobodyCam, I did: git fetch https://review.openstack.org/openstack/ironic refs/changes/83/160383/24 && git checkout FETCH_HEAD16:12
*** Marga_ has joined #openstack-ironic16:12
NobodyCambut its at rev 25 :-p16:12
jlvillalNobodyCam, That was my push :)16:12
jlvillalNobodyCam, Where does it say that?  I'm looking on Gerrit16:12
NobodyCamoh never mind16:12
* NobodyCam needs more coffee16:12
NobodyCamI was looking at the wrong patch16:13
NobodyCam:-p16:13
jlvillalNobodyCam, Whew! :)16:13
jlvillalNobodyCam, You were making me nervous :D16:13
*** Marga_ has quit IRC16:14
jlvillalNobodyCam, I basically just added some 'autospec=True' lines and change 'Mb' to 'MiB'.  I think of 'Mb' as mega-bits.16:14
*** Marga_ has joined #openstack-ironic16:15
NobodyCamjlvillal: ++16:15
NobodyCamThank you16:15
jlvillalNobodyCam, Now let's get that merged! :D16:16
lucasagomesTheJulia, thanks the review, I will add some docs in the troubleshoot section16:16
*** mtanino has quit IRC16:17
*** saripurigopi has quit IRC16:17
*** saripurigopi_ has joined #openstack-ironic16:18
*** saripurigopi_ is now known as saripurigopi16:18
*** Marga_ has quit IRC16:18
TheJulialucasagomes: excellent!16:19
*** Marga_ has joined #openstack-ironic16:19
* devananda pushes a client release fix16:20
devanandaalso - holy crap folks, that's a lot of bugs ya'll fixed in the last few weeks :) https://launchpad.net/ironic/kilo/kilo-rc116:22
openstackgerritChris Krelle proposed openstack/ironic: Add check for usable tftp directory to PXE driver  https://review.openstack.org/16745116:22
NobodyCam:)16:22
*** Marga_ has quit IRC16:23
*** ijw has quit IRC16:26
lucasagomes:-)16:27
*** Nisha has joined #openstack-ironic16:30
*** a1exhughe5 has quit IRC16:32
*** saripurigopi has quit IRC16:33
BadCubdevananda: yes, the gang pushed pretty hard on the bug list :-)16:34
BadCubdevananda: NobodyCam: jroll: lucasagomes; et al. did we all decide if tomorrow was good to get the Spec Core team together? Or would a different date/time be better?16:36
lucasagomesBadCub, hmm what time is it?16:37
* lucasagomes didn't know it was tomrrow16:37
lucasagomesI think it works to me, but thursday would be ideal16:37
BadCubI put a tentative for 1500hrs UTC. But wanted folks to decide if that was good or not.16:38
BadCubI am available pretty much any day/time, so what ever works best for the whole team works for me :)16:38
devanandaJayF: ^ work for you?16:39
*** maurosr_ is now known as maurosr16:41
*** derekh has quit IRC16:43
lucasagomesBadCub, me and dtantsur|afk has a meeting exactly at this time16:45
lucasagomesbut it shouldn't be long16:45
lucasagomesso after that I think we are good16:45
lucasagomeshave*16:45
*** meghal has joined #openstack-ironic16:46
BadCublucasagomes: okay. I am at y'all's disposal, so what ever works best, I will be there!16:46
lucasagomesBadCub, thanks for that!16:46
*** Marga_ has joined #openstack-ironic16:46
*** Marga_ has quit IRC16:46
BadCublucasagomes: anytime :-) I like things to be easy lol16:47
*** ijw has joined #openstack-ironic16:47
*** meghal1 has joined #openstack-ironic16:47
BadCubI have put up a pad for everyone to gander at. I made some observations during the end of K and put up some ideas for discussion on how we might make things easier too.16:48
rlooBadCub: did you check with dtantsur|afk, wrt the core spec meeting?16:48
BadCubrloo: I put it out last week for folks to discuss and had it on the meeting agenda :)16:48
rlooBadCub: I can verify that I saw it mentioned in the meeting minutes yesterday, although I think it wasn't clear whether it was ON at that date/time, or whether it was up for discussion :-)16:50
*** meghal has quit IRC16:50
BadCubYeah, I intended to be there last night to bring it up for discussion but passed out :(16:51
*** Marga_ has joined #openstack-ironic16:51
rlooBadCub: wonder if we should re-evaluate that meeting time then. Is it working for folks? Isn't it 9pm for you?16:51
BadCubrloo: it is 10pm for me16:52
devanandachrome chrashes again :-/16:52
BadCubdevananda: ack :(16:52
jlvillalOne hour earlier (9pm PT) would be better for me :)16:52
rlooBadCub: oh yeah, due to the time change. midnight-1am for me, all the same. I don't attend :-)16:53
BadCubrloo: I try, but sometimes I am already dead sleeping at 10PM. Depends on when I take meds :p16:53
BadCubbrb16:55
*** Marga_ has quit IRC16:58
*** harlowja has joined #openstack-ironic16:59
lucasagomesdevananda, btw re the configdrive_metadata. R you OK with the idea? It's most the interface that concerns?16:59
devanandalucasagomes: totally17:00
devanandalucasagomes: the loopback one, i'm less OK with even the idea, but if its done as an extensionto IPA, it starts to be a grey area where i'm maybe ok with someone doing it even though I dont like it17:01
lucasagomesdevananda, right... yeah leave the loopback :-)17:01
lucasagomesdevananda, the configdrive... Would it be better to have a field in the node for that?17:01
*** ijw has quit IRC17:01
lucasagomesI understand that putting on the ports for the network case makes a bit of sense. But I would like to have it to be generic17:02
*** ijw has joined #openstack-ironic17:02
devanandalucasagomes: generic how? to me, putting information on th eports *is* generic17:02
lucasagomesit's not, it's related to the ports17:02
devanandathat you want to bundle it up in a special configdrive format is the special case :)17:03
lucasagomesor should be17:03
lucasagomesconfigdrive supports extra data, I'm just using that interface17:03
lucasagomesthe code in nova that generates the config drive supports passing a extra_md= parameter to it17:03
lucasagomesfwiw we do use it, as other drivers, to inject the root password17:04
lucasagomesadmin password*17:04
lucasagomeshttps://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L701-L70317:04
*** dttocs_ has joined #openstack-ironic17:05
*** davideagnello has joined #openstack-ironic17:06
lucasagomesdevananda, btw http://paste.openstack.org/show/175855/17:08
lucasagomesit's from Feb 1317:08
*** Marga_ has joined #openstack-ironic17:08
*** dttocs has quit IRC17:08
* lucasagomes thinks more people will do similar things17:09
devanandalucasagomes: so I understand the problem17:11
devanandalucasagomes: and I agree that passing in some "hints" via configdrive is an acceptable solution17:12
devanandalucasagomes: but I think the canonical representation of the information that Ironic needs to record (how the NICs should be configured within the instance so that they work within the physical network outside the instance) should be done differently17:13
lucasagomesdevananda, right and this is something that http://specs.openstack.org/openstack/ironic-specs/specs/kilo/network-provider.html should handle correct?17:15
devanandaTheJulia and/or mordred has been doing some work, IIRC, on a thing to replace cloudinit that would do just this piece of it17:15
devanandalucasagomes: correct17:15
lucasagomesdevananda, the "hints" is extra that ^17:16
lucasagomeswe use it for consistent for e.g17:16
mordredyes17:16
lucasagomesso scripting can be easier across all instances17:16
mordredhttps://gitorious.org/mordred-temp/glean17:16
*** Nisha has quit IRC17:16
mordredI'll be sucking it into an infra repo later today17:16
lucasagomeswe not creating/removing networks or anything of that sort, and I hope people won't do that too17:17
mordredit handles reading static network config on boot from config drive17:17
mordredwe've got a dib element that installs it properly here: https://review.openstack.org/#/c/167003/17:17
*** dprince has joined #openstack-ironic17:18
mordredI'm currently walking through all of the distros and verifying that it does the right thing17:18
lucasagomescool17:18
devanandaalso, this is odd... why are there two of them?17:18
devanandahttp://specs.openstack.org/openstack/ironic-specs/specs/kilo/network-provider.html17:18
devanandahttp://specs.openstack.org/openstack/ironic-specs/specs/kilo-archive/network-provider.html17:18
mordredthe basic logic is - for every interface that /sys reports as being a real interface, see if there is network config in config drive - if there is, use it, if there is not, write a dhcp config. if there is already a config in place, do nothing17:18
lucasagomesdevananda, no idea17:19
mordredit also, just for convenience, supports ssh keypairs in config drive17:19
mordredbut it does _not_ support any other feature of cloud-init17:19
lucasagomesmordred, does it support renaming network devices?17:19
mordredit does not support any additional management after first boot17:19
BadCubdevananda: that is odd.17:19
mordredit assumes that you are using this to boot something and that you'll manage the system with something else in an ongoing manner17:19
lucasagomesmapping interfaces names to some correspoding abstract name?17:20
mordredit does a mapping between mac address and interface name based on what's reported by the sys filesystem17:20
lucasagomescool17:20
mordredso what it cares about is what network you've said a particualr mac address shuld be connected to17:20
lucasagomesmordred, not sure if you looked or care... but os-net-config does some of it17:20
mordredit does - but os-net-config has too many depends17:20
mordredthis has zero depends other than pure python17:21
mordredbecause I dont' want to install tons of libraries in the base of the system that I personally will be suing to then test openstack installs17:21
mordredthe depends step on each other17:21
lucasagomes+1 sounds good17:21
devanandaBadCub: you have time to investiate that // clean up the old ones?17:21
mordredlucasagomes: we could add a flag to it to always blow away existing config17:21
devanandaI think jayF did the initial clean up / split, but it seems something got left behind17:22
mordredlucasagomes: so that if you know you're runnign in an env where config-drive will always be the source of truth17:22
*** ndipanov has quit IRC17:22
mordredlucasagomes: you could have it skip the part where it avoids blowing away existing network config17:22
lucasagomesmordred, I will def take a look into it17:22
BadCubdevananda: yeah, I can tackle that17:23
* devananda steps afk to food17:23
mordredlucasagomes: please do - and let me know if there's anything it can do to make your life better - I've been trying to keep ironic in mind while working on it17:23
mordredsince I think ironic installs are in a similar boat in terms of what you want and don't want out of this17:24
*** Nisha_brb has joined #openstack-ironic17:24
lucasagomesdevananda, but anyway... my reasoning about using extra. Is as it can be any key/pair data, I don't wanna attach it to any resource in our api. Perhaps the mac naming example we could move to ports yes, but that's is 1 use case only17:24
openstackgerritJohn L. Villalovos proposed openstack/ironic: Fix PEP8 E129 errors  https://review.openstack.org/17342717:25
*** trown is now known as trown|lunch17:26
lucasagomesmordred, cool thanks much! I know you want to keep dependencies low but you may want to take a look at augeas for many of the configurations you generate17:26
lucasagomes(it does have python binds)17:27
*** Nisha_brb has quit IRC17:27
*** Nisha has joined #openstack-ironic17:27
*** Nisha has quit IRC17:28
*** Nisha has joined #openstack-ironic17:28
mordredlucasagomes: taht doens't look too crazy, actually - and I don't mind C-based command line tools that have distro packages17:29
mordredsince they don't tend to actually conflict with things17:30
mordredlucasagomes: I'll play with shelling out to it and seeing if it can do some of the file output we're doing17:30
lucasagomesmordred, yeah, augeas is pretty awesome. And writing lens is not complicated17:30
lucasagomesif you really need some custom configs17:30
lucasagomesmordred, cool stuff!17:30
*** pelix has quit IRC17:32
*** Marga_ has quit IRC17:35
*** Marga_ has joined #openstack-ironic17:35
*** Marga_ has quit IRC17:36
*** meghal1 has quit IRC17:36
*** Marga_ has joined #openstack-ironic17:36
lucasagomesaight I will call it a day folks17:42
lucasagomeshave a good night!17:42
NobodyCamnight lucasagomes17:42
*** lucasagomes is now known as lucas-dinner17:42
lucas-dinnerNobodyCam, you too! see ya tomorrow17:42
*** pas-ha has quit IRC17:44
*** edcragg has quit IRC17:47
*** meghal has joined #openstack-ironic17:50
*** dttocs_ has quit IRC17:50
*** achanda has joined #openstack-ironic17:51
*** dttocs has joined #openstack-ironic17:54
*** dttocs has quit IRC17:56
*** jamielennox|away is now known as jamielennox18:05
*** krtaylor has quit IRC18:12
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Troubleshoot: Do not power off node if deployment fail  https://review.openstack.org/17293218:12
aweeksdevananda: jroll: I'm updating the metrics spec, and wondering about the feasibility of emitting metrics for node state changes.  Initially, I considered just instrumenting the FSM, but it doesn't have information about how long a node was in a given state (for timing deploys, cleans, etc.) but it looks like it may be possible to calculate that by looking at18:19
aweeksNode.provision_updated_at, when Node.provision_state is changed18:19
aweekshowever, that might require some relatively invasive changes to the Node/IronicObject classes18:20
aweeksI'm curious if you have any alternative ideas18:20
*** Marga_ has quit IRC18:29
*** meghal has quit IRC18:32
*** trown|lunch is now known as trown18:32
*** meghal has joined #openstack-ironic18:32
*** meghal has joined #openstack-ironic18:33
*** meghal has quit IRC18:33
*** meghal has joined #openstack-ironic18:34
*** Marga_ has joined #openstack-ironic18:50
*** dttocs has joined #openstack-ironic18:57
* devananda finishes a phone call then goes to lunch18:59
*** krtaylor has joined #openstack-ironic19:04
*** meghal has quit IRC19:16
*** mtanino has joined #openstack-ironic19:18
*** andreykurilin has joined #openstack-ironic19:32
*** kkoski has quit IRC19:35
*** kkoski has joined #openstack-ironic19:35
*** kkoski has quit IRC19:36
*** mtanino has quit IRC19:49
*** achanda has quit IRC20:01
*** jgrimm is now known as zz_jgrimm20:11
*** andreykurilin has quit IRC20:13
*** lucas-dinner has quit IRC20:17
*** Haomeng|2 has quit IRC20:18
*** Haomeng has joined #openstack-ironic20:19
*** ijw has quit IRC20:24
*** achanda has joined #openstack-ironic20:25
*** ijw has joined #openstack-ironic20:25
*** romcheg has joined #openstack-ironic20:25
*** Marga_ has quit IRC20:28
*** ijw has quit IRC20:29
*** ijw has joined #openstack-ironic20:33
*** lifeless1 is now known as lifeless20:37
rloohi devananda and anyone else that -2'd patches until liberty was open -- time to undo your -2's :-)20:40
*** Nisha has quit IRC20:52
*** trown is now known as trown|outttypeww21:05
devanandarloo: all: query string to find anything you've blocked in a project: is:reviewer status:open project:openstack/ironic label:Code-Review=-221:08
devanandanot perfect, but close21:09
rloodevananda: cool, thx.21:09
*** dttocs has quit IRC21:12
rloodevananda: it shows all the patches that are -2'd, that I've ever reviewed, even if I didn't -2 it :-)21:12
devanandaoh hah21:13
devanandayou're right. it's not a great query, but it just so happens that almost all of my first page is just my own -2's so it looked like it worked :p21:14
*** dprince has quit IRC21:14
rloodevananda: as long as it works for you. I didn't -2 any so don't need to look ;)21:15
openstackgerritAlex Weeks proposed openstack/ironic-specs: Add pluggable metrics backend for Ironic and IPA  https://review.openstack.org/13717121:17
aweeks^ that change addresses some, but not all of the comments so far21:19
*** dttocs has joined #openstack-ironic21:29
*** dttocs has quit IRC21:41
*** dttocs has joined #openstack-ironic21:44
mrdaMorning Ironic21:46
NobodyCammorning mrda21:46
mrdao/21:47
BadCubhiya mrda :-)21:49
mrdahey21:50
*** kkoski has joined #openstack-ironic21:58
*** dttocs_ has joined #openstack-ironic22:06
*** dttocs has quit IRC22:10
openstackgerritClif Houck proposed openstack/ironic: Add support for image caching for the agent driver  https://review.openstack.org/16183222:12
devanandaanyone know if kevin fox took this proposal anywhere else? https://review.openstack.org/#/c/137404/22:18
devanandarloo: you'll be pleased to know, i have removed all my procedural blocks :)22:20
*** Marga_ has joined #openstack-ironic22:21
*** kkoski1 has joined #openstack-ironic22:23
*** kkoski has quit IRC22:23
BadCubw00t22:23
*** ijw has quit IRC22:32
*** Sukhdev has joined #openstack-ironic22:32
*** achanda has quit IRC22:36
*** kkoski has joined #openstack-ironic22:41
*** kkoski1 has quit IRC22:41
*** achanda has joined #openstack-ironic22:41
*** romcheg has quit IRC22:43
*** dttocs_ has quit IRC22:44
*** Marga_ has quit IRC22:49
*** kkoski has quit IRC22:50
rloodevananda: thx, the whole world is pleased :-)22:57
*** yuanying has joined #openstack-ironic22:57
*** absubram has quit IRC22:59
*** Marga_ has joined #openstack-ironic23:07
*** chlong has joined #openstack-ironic23:10
*** ijw has joined #openstack-ironic23:11
openstackgerritjxiaobin proposed openstack/ironic-specs: Mount config drive as loop device to supply data to cloud-init  https://review.openstack.org/17314223:13
*** Sukhdev has quit IRC23:13
*** Sukhdev has joined #openstack-ironic23:13
rwsuwhat should notification_driver be set to in /etc/ironic/ironic.conf? I've tried "messaging", but no messages are going through the exchange23:20
Haomengrwsu: I think notification_driver = messaging should be fine, we set it for ceilometer which can consume the ironic ipmi sensor data23:24
rwsuHaomeng: are there any other options that need to be set in /etc/ironic/ironic.conf?23:27
Haomengrwsu: for that time I test the ipmi sensor data sending to ceilometer, one option notification_driver = messaging  is enough, but not sure your case23:28
rwsuHaomeng: ok thanks, yeah there is something else wrong with my setup23:29
Haomengrwsu: ok, welcome:)23:30
*** Sukhdev has quit IRC23:35
*** zer0c00l has quit IRC23:39
*** zer0c00l has joined #openstack-ironic23:39
*** jerryz has joined #openstack-ironic23:42
*** chlong has quit IRC23:43
*** chlong has joined #openstack-ironic23:45
*** Sukhdev has joined #openstack-ironic23:46
*** zhenguo has joined #openstack-ironic23:51
*** dttocs has joined #openstack-ironic23:53
*** dttocs has quit IRC23:56

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