Monday, 2015-02-09

*** achanda has quit IRC00:00
*** achanda has joined #openstack-ironic00:02
*** s8weber_ has quit IRC00:10
*** s8weber__ has joined #openstack-ironic00:10
*** smoriya has joined #openstack-ironic00:24
*** jiangfei|2 has quit IRC00:38
*** Nisha has joined #openstack-ironic00:51
*** achanda has quit IRC00:52
*** Nisha has quit IRC00:56
*** mlsoule has joined #openstack-ironic01:03
*** mlsoule has quit IRC01:09
*** ijw_ has joined #openstack-ironic01:37
*** jiangfei has joined #openstack-ironic01:38
*** ijw has quit IRC01:40
*** victor_lowther has quit IRC01:40
*** naohirot has joined #openstack-ironic01:43
naohirotgood morning ironic01:43
*** achanda has joined #openstack-ironic01:52
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Management module for iRMC Driver  https://review.openstack.org/14680301:52
*** victor_lowther has joined #openstack-ironic01:53
*** achanda has quit IRC01:57
*** zhenzanz has joined #openstack-ironic02:01
openstackgerritNaohiro Tamura proposed openstack/ironic: Add iRMC Virtual Media Deploy module for iRMC Driver  https://review.openstack.org/15195802:06
*** ijw_ has quit IRC02:29
openstackgerritNaohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic  https://review.openstack.org/13486502:31
*** wuhg has joined #openstack-ironic02:39
*** jiangfei has quit IRC02:40
*** jiangfei has joined #openstack-ironic02:41
*** chenglch has joined #openstack-ironic02:42
*** ramineni has joined #openstack-ironic02:43
*** zyu4_ has joined #openstack-ironic02:44
*** zhidong has joined #openstack-ironic02:44
*** zhidong has quit IRC02:45
*** jerryz has joined #openstack-ironic03:03
*** kan_ has joined #openstack-ironic03:05
*** pcrews has quit IRC03:30
*** dttocs has quit IRC04:00
*** coolsvap_ is now known as coolsvap04:02
*** Marga_ has joined #openstack-ironic04:03
*** Marga_ has quit IRC04:04
*** Marga_ has joined #openstack-ironic04:04
*** lnxnut has quit IRC04:04
*** lnxnut has joined #openstack-ironic04:13
*** achanda has joined #openstack-ironic04:39
openstackgerritYuiko Takada proposed openstack/ironic: Unuse deprecated auth headers  https://review.openstack.org/15392004:42
*** lnxnut has quit IRC04:43
*** lnxnut has joined #openstack-ironic04:49
*** lnxnut has quit IRC04:53
openstackgerritRamakrishnan G proposed openstack/ironic: ipmitool drivers fail with integer passwords  https://review.openstack.org/15270204:55
*** coolsvap is now known as coolsvap_04:55
*** coolsvap_ is now known as coolsvap04:56
*** spandhe has joined #openstack-ironic04:57
*** Marga__ has joined #openstack-ironic05:18
*** Marga_ has quit IRC05:18
*** killer_prince is now known as lazy_prince05:32
*** Marga__ has quit IRC05:32
*** Marga_ has joined #openstack-ironic05:36
*** achanda has quit IRC05:38
*** lnxnut has joined #openstack-ironic05:50
*** achanda has joined #openstack-ironic05:57
openstackgerritSirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images  https://review.openstack.org/15014206:20
*** achanda has quit IRC06:22
*** pcaruana has quit IRC06:28
*** s8weber__ has quit IRC06:38
*** lnxnut has quit IRC06:41
*** spandhe has quit IRC06:44
*** yog_ has joined #openstack-ironic06:52
*** ndipanov has joined #openstack-ironic07:01
*** yuanying has quit IRC07:07
openstackgerritAnusha Ramineni proposed stackforge/proliantutils: Initial Support to RIS  https://review.openstack.org/15278007:07
*** yuanying has joined #openstack-ironic07:10
*** Nisha has joined #openstack-ironic07:12
*** yuanying has quit IRC07:13
*** achanda has joined #openstack-ironic07:15
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Add hpssa module to proliantutils  https://review.openstack.org/15351907:15
*** ukalifon1 has joined #openstack-ironic07:17
*** yuanying has joined #openstack-ironic07:20
*** yuanying has quit IRC07:21
*** yuanying has joined #openstack-ironic07:26
*** yuanying has quit IRC07:30
*** romcheg has joined #openstack-ironic07:32
*** yuanying has joined #openstack-ironic07:33
*** yuanying has quit IRC07:34
*** chlong has quit IRC07:37
*** yuanying has joined #openstack-ironic07:39
*** lazy_prince has quit IRC07:43
*** yuanying has quit IRC07:43
*** yuriyz has quit IRC07:43
*** lazy_prince has joined #openstack-ironic07:43
*** yuanying has joined #openstack-ironic07:46
*** achanda has quit IRC07:47
openstackgerritSirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images  https://review.openstack.org/15014207:48
*** achanda has joined #openstack-ironic07:51
*** pradipta has joined #openstack-ironic07:58
*** Marga_ has quit IRC08:01
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Fix boot mode exceptions for Gen 7 servers  https://review.openstack.org/15394508:05
*** romcheg1 has joined #openstack-ironic08:07
*** romcheg has quit IRC08:07
*** dtantsur|afk is now known as dtantsur08:10
dtantsurMorning Ironic08:10
*** datajerk has quit IRC08:11
*** datajerk1 has joined #openstack-ironic08:11
*** gridinv has joined #openstack-ironic08:13
*** soren has quit IRC08:13
*** zer0c00l has quit IRC08:14
*** soren has joined #openstack-ironic08:19
*** gridinv has quit IRC08:20
*** yuriyz has joined #openstack-ironic08:25
*** zer0c00l has joined #openstack-ironic08:27
*** Marga_ has joined #openstack-ironic08:28
*** bradjones has quit IRC08:31
*** bradjones has joined #openstack-ironic08:32
Nishamorning dtantsur08:35
Nisha:)08:35
dtantsurNisha, o/08:35
Nishadtantsur, need your reviews on https://review.openstack.org/#/c/141012/ (Nova FFE request patch) and introspection patches08:36
dtantsuryeah, sure, once I wake up and get rid of urgent emails :)08:37
Nishadtantsur, ok08:37
*** jcoufal has joined #openstack-ironic08:40
*** Marga_ has quit IRC08:40
*** andreykurilin_ has joined #openstack-ironic08:40
*** jerryz has quit IRC08:41
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Fix boot mode exceptions for Gen 7 servers  https://review.openstack.org/15394508:43
*** jistr has joined #openstack-ironic08:49
*** achanda has quit IRC08:52
*** achanda has joined #openstack-ironic08:56
*** lucasagomes has joined #openstack-ironic08:56
Nishadtantsur,09:02
Nishai saw your comments on https://review.openstack.org/#/c/141012/18/nova/virt/ironic/patcher.py09:03
*** Marga_ has joined #openstack-ironic09:06
*** achanda has quit IRC09:08
openstackgerritRamakrishnan G proposed openstack/ironic-specs: New driver interface for RAID configuration  https://review.openstack.org/13589909:10
*** achanda has joined #openstack-ironic09:18
*** derekh has joined #openstack-ironic09:18
*** zyu4_ has quit IRC09:20
*** mgoddard has joined #openstack-ironic09:20
*** zhenzanz has quit IRC09:25
*** rameshg87 has joined #openstack-ironic09:25
*** MattMan has joined #openstack-ironic09:29
*** athomas has joined #openstack-ironic09:30
*** mgoddard1 has joined #openstack-ironic09:32
*** mgoddard has quit IRC09:33
Nishadtantsur, i reposted the patch https://review.openstack.org/#/c/141012/09:39
Nishadtantsur, addressed your comments....it will be good if you can look at it again09:39
*** andreykurilin_ has quit IRC09:41
*** achanda has quit IRC09:46
*** dtantsur is now known as dtantsur|brb10:00
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add support for local boot  https://review.openstack.org/14618910:04
openstackgerritMerged stackforge/proliantutils: Initial Support to RIS  https://review.openstack.org/15278010:06
Nishalucasagomes, please review https://review.openstack.org/#/c/141012/10:11
lucasagomesNisha, ack10:26
*** Nisha has quit IRC10:30
*** pelix has joined #openstack-ironic10:32
openstackgerritShivanand Tendulker proposed openstack/ironic: Changes for secure boot implementation for ILO drivers  https://review.openstack.org/15397410:36
openstackgerritShivanand Tendulker proposed openstack/ironic: Code changes for secure boot implementation for ILO drivers  https://review.openstack.org/15397410:39
openstackgerritMerged openstack/ironic: ipmitool drivers fail with integer passwords  https://review.openstack.org/15270210:39
*** mgoddard1 has quit IRC10:45
*** mgoddard has joined #openstack-ironic10:45
*** mgoddard1 has joined #openstack-ironic10:47
*** mgoddard has quit IRC10:47
*** Marga_ has quit IRC10:47
*** ifarkas has joined #openstack-ironic10:55
*** dtantsur|brb is now known as dtantsur10:59
dtantsurifarkas, hey hey, I have some patches for you to review :) https://review.openstack.org/#/c/153240/1 and https://review.openstack.org/#/c/153261/ please10:59
ifarkasdtantsur, sure thing ;-)11:00
*** pradipta has quit IRC11:01
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Fix boot mode exceptions for Gen 7 servers  https://review.openstack.org/15394511:05
*** ramineni has quit IRC11:05
*** lazy_prince has quit IRC11:07
*** rameshg87 has quit IRC11:10
*** killer_prince has joined #openstack-ironic11:10
*** killer_prince is now known as lazy_prince11:10
*** kbs1 has joined #openstack-ironic11:13
*** kbs has quit IRC11:14
*** Marga_ has joined #openstack-ironic11:14
openstackgerritMerged stackforge/ironic-discoverd: Add eDeploy plugin  https://review.openstack.org/14659911:20
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add support for local boot  https://review.openstack.org/14618911:26
lucasagomesdtantsur, gotcha, I will fix that in the next patchset11:32
dtantsurcool11:32
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add support for local boot  https://review.openstack.org/14618911:36
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Bump requirements to Kilo  https://review.openstack.org/15399811:47
*** romcheg has joined #openstack-ironic11:56
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Get rid of thread=False monkey-patching hack  https://review.openstack.org/15400411:57
*** romcheg1 has quit IRC11:58
openstackgerritMerged stackforge/ironic-discoverd: Bump requirements to Kilo  https://review.openstack.org/15399812:00
*** romcheg1 has joined #openstack-ironic12:01
*** romcheg has quit IRC12:02
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Get rid of thread=False monkey-patching hack  https://review.openstack.org/15400412:07
openstackgerritMerged stackforge/ironic-discoverd: Refactor ValidateInterfaces plugin  https://review.openstack.org/15324012:15
*** lucasagomes is now known as lucas-hungry12:21
openstackgerritMerged stackforge/ironic-discoverd: Get rid of thread=False monkey-patching hack  https://review.openstack.org/15400412:22
*** chlong has joined #openstack-ironic12:25
*** BadCub01_ has joined #openstack-ironic12:27
BadCub01_morning Ironic12:28
*** Marga_ has quit IRC12:29
openstackgerritMerged stackforge/ironic-discoverd: Allow ramdisk to specify PXE boot interface  https://review.openstack.org/15326112:34
*** chlong has quit IRC12:35
*** chlong has joined #openstack-ironic12:35
openstackgerritMerged openstack/ironic: Fix argument value for work_on_disk() in unit test  https://review.openstack.org/15307212:40
*** dlpartain has joined #openstack-ironic12:51
*** lnxnut has joined #openstack-ironic12:53
*** Marga_ has joined #openstack-ironic12:56
*** dprince has joined #openstack-ironic12:57
*** lnxnut has quit IRC12:58
*** BadCub01_ has quit IRC13:01
*** BadCub_ has joined #openstack-ironic13:02
*** erwan_taf has quit IRC13:05
*** yuanying_ has joined #openstack-ironic13:05
openstackgerritSirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images  https://review.openstack.org/15014213:08
*** yuanying has quit IRC13:09
*** smoriya has quit IRC13:09
*** romcheg has joined #openstack-ironic13:15
*** romcheg1 has quit IRC13:17
*** erwan_taf has joined #openstack-ironic13:19
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Fix detection of PXELINUX-provided boot interface  https://review.openstack.org/15402713:25
dtantsurifarkas, mind also having a quick look at ^^^ (needed to unbreak our env)?13:26
ifarkasdtantsur, yeah, I will check it13:26
dtantsurthanks13:27
*** chlong has quit IRC13:36
*** yog_ has quit IRC13:37
openstackgerritNaohiro Tamura proposed openstack/ironic: Fix chown invalid option -- 'p'  https://review.openstack.org/15404213:49
*** Nisha has joined #openstack-ironic13:50
*** stendulker has joined #openstack-ironic13:57
*** rloo has joined #openstack-ironic14:00
*** david-lyle_afk is now known as david-lyle14:01
*** jcoufal has quit IRC14:02
*** jcoufal has joined #openstack-ironic14:02
*** krtaylor has quit IRC14:03
*** r-daneel has joined #openstack-ironic14:04
*** mjturek1 has joined #openstack-ironic14:04
*** Haomeng has joined #openstack-ironic14:05
Nishalucas-hungry, dtantsur addressed coments on https://review.openstack.org/#/c/141012/14:06
Nisha and posted the patch. Need re-review.14:06
*** lucas-hungry is now known as lucasagomes14:07
lucasagomesNisha, thanks will do in a few14:07
openstackgerritNaohiro Tamura proposed openstack/ironic: Fix chown invalid option -- 'p'  https://review.openstack.org/15404214:07
Nishalucasagomes, thanks14:07
*** dlpartain has quit IRC14:08
openstackgerritRamakrishnan G proposed stackforge/proliantutils: Fix boot mode exceptions for Gen 7 servers  https://review.openstack.org/15394514:10
*** rameshg87 has joined #openstack-ironic14:11
rameshg87NobodyCam, lucasagomes, can you please have a look at raid configuration spec: https://review.openstack.org/#/c/135899/2614:12
lucasagomesrameshg87, yes I was support to do that14:16
lucasagomeschecking now14:16
*** krtaylor has joined #openstack-ironic14:16
rameshg87lucasagomes, thanks :)14:16
*** lnxnut has joined #openstack-ironic14:17
stendulkerNobodyCam, JayF, can you please review secure boot management interface related spec https://review.openstack.org/#/c/135845/14:18
*** chenglch has quit IRC14:20
*** lnxnut has quit IRC14:21
*** ramineni has joined #openstack-ironic14:25
*** wuhg has quit IRC14:29
*** Nisha has quit IRC14:30
lucasagomesNobodyCam, https://review.openstack.org/#/c/135899/ when you get some time14:30
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Fix get_kernel_parameter fixture  https://review.openstack.org/15405814:30
*** jcoufal has quit IRC14:34
*** jcoufal has joined #openstack-ironic14:34
openstackgerritMerged stackforge/ironic-discoverd: Fix get_kernel_parameter fixture  https://review.openstack.org/15405814:38
*** yog_ has joined #openstack-ironic14:41
*** s8weber has joined #openstack-ironic14:43
*** stendulker has quit IRC14:45
*** stendulker has joined #openstack-ironic14:47
NobodyCamGood MOrning Ironic14:55
*** lnxnut has joined #openstack-ironic14:58
dtantsurNobodyCam, morning!14:58
NobodyCammorning dtantsur14:58
lucasagomesNobodyCam, morning14:59
NobodyCamrameshg87: lucasagomes: stendulker: will take a look in a couple of minutes...15:00
NobodyCamjust powering on atm15:00
NobodyCammorningn lucasagomes :)15:00
stendulkermorning NobodyCam15:00
stendulkerNobodyCam: Thank you :)15:01
*** lnxnut_ has joined #openstack-ironic15:01
*** lnxnut has quit IRC15:02
*** derekh has quit IRC15:02
*** derekh has joined #openstack-ironic15:03
NobodyCam:)15:05
*** ChuckC has joined #openstack-ironic15:06
*** ChuckC has quit IRC15:06
*** absubram has joined #openstack-ironic15:06
*** ChuckC has joined #openstack-ironic15:07
*** ijw has joined #openstack-ironic15:07
*** ijw has quit IRC15:08
*** ijw has joined #openstack-ironic15:09
*** kejiahu has joined #openstack-ironic15:11
NobodyCamdo we still need an Agenda item to  Discuss per driver sensor meters spec?15:22
* dtantsur thinks no15:25
dtantsuralso, did we get a status email last week?15:25
*** jcoufal_ has joined #openstack-ironic15:26
*** vdrok is now known as vdrok_afk15:27
NobodyCamdid zapping spec land...15:29
*** jcoufal has quit IRC15:29
rameshg87NobodyCam, yes15:29
NobodyCam:)15:30
rameshg87NobodyCam, https://github.com/openstack/ironic-specs/blob/master/specs/kilo/implement-zapping-states.rst15:30
*** gridinv has joined #openstack-ironic15:30
*** lazy_prince has quit IRC15:30
NobodyCamI thought it did... just early in the morning for /me :-p15:30
rameshg87NobodyCam, :)15:30
*** ramineni has quit IRC15:30
*** ramineni has joined #openstack-ironic15:31
*** Haomeng has quit IRC15:31
rloomorning ironickers15:31
rloodtantsur: just saw your comment wrt https://review.openstack.org/134866. is that worth asking/discussing at the meeting?15:32
dtantsurrloo, maybe, but deadline for topics was on Fri :)15:32
NobodyCammorning rloo :)15:32
rloodtantsur: it was?15:33
dtantsurwe can discuss it in open discussion (though I'm REALLY surprised we have to discuss it)15:33
dtantsurrloo, and morning :)15:33
rloodtantsur: oh, for meeting topics. gotcha.15:33
dtantsuryep15:33
*** killer_prince has joined #openstack-ironic15:33
*** killer_prince is now known as lazy_prince15:33
rloodtantsur: fine with me. It was just a thought. I have no stake in that spec ;)15:34
dtantsurheh15:34
rloodtantsur: just to save time for people, if those shouldn't be specs, etc.15:34
dtantsurrloo, I think it should not be in Ironic :)15:35
dtantsurwould really love to hear from NobodyCam why it's an appropriate change btw...15:35
*** achanda has joined #openstack-ironic15:35
dtantsurI mean, if we bring in a new feature, let us make it a feature, at least some vendor-passthru15:36
dtantsurhere we have something that won't be tested, won't be discoverable by users, won't be consumable via API (and thus via UI), probably won't become generic15:37
dtantsurNobodyCam, rloo ^^^15:38
NobodyCamhuh15:39
NobodyCamlookin15:39
devanandamorning, all15:39
dtantsurdevananda, morning15:40
rloodtantsur: I saw it as a tool/script that might have value to other folks, but yeah, I'd like to know what others think too ;)15:40
rloomorning devananda15:40
BadCub_morning devananda15:40
*** Haomeng has joined #openstack-ironic15:40
dtantsurrloo, nobody prevents them from having this tool in e.g. proliantutils package...15:40
NobodyCamrameshg87: lucasagomes: 135899 +a'd15:41
*** achanda has quit IRC15:42
NobodyCamoh the tool script. ya, I saw that a a tool that lives in tools/ dir and works for chassis bases servers.15:42
NobodyCamI just thought it was a cool Idea that if some one wanted to could expand upon15:42
NobodyCamwith out impacting the code15:42
*** jcoufal_ has quit IRC15:43
*** jcoufal has joined #openstack-ironic15:44
dtantsurNobodyCam, this "with out impacting the code" actually bothers me. if we don't like it enough to expose to our users, why have it in tree at all?15:44
lucasagomesNobodyCam, o/!15:45
NobodyCammorning devananda15:45
NobodyCamdtantsur: I don't understand the "if we don't like it" I like it its a good (and our only) real use for chassis object15:47
dtantsurNobodyCam, then why aren't we exposing it to users (= creating API for it)?15:47
openstackgerritMerged stackforge/ironic-discoverd: Fix detection of PXELINUX-provided boot interface  https://review.openstack.org/15402715:47
dtantsuronly minority of users will actually get access to this utility15:47
dtantsurbecause it's 1. won't get packaged; 2. is not exposed via HTP API15:48
devanandadtantsur: "wont be tested" -- why wouldn't it be tested? "wont be discoverable" -- it'll be installed along with other commands, documented, etc, so how is this not discoverable?15:48
openstackgerritMerged openstack/ironic-specs: New driver interface for RAID configuration  https://review.openstack.org/13589915:48
dtantsurdevananda, current proposal is to keep it in ./tools, not create a proper executable15:48
devanandadtantsur: huh. I'd expect it to be in cmd/15:49
dtantsurdevananda, are we starting to provide vendor-specific tools without exposing them via HTTP API?15:49
dtantsurafter decision to split away discoverd it sounds really strange to me15:50
devanandaok, clearly i'm missing some context here15:50
devanandait's early in the morning, i'm deal with a cold, and have a LOT of things to do today, so I"m going to just back away from this conversation15:50
*** zz_jgrimm is now known as jgrimm15:51
*** ChuckC has quit IRC15:51
*** ChuckC has joined #openstack-ironic15:52
*** pcrews has joined #openstack-ironic15:59
*** beekneemech is now known as bnemec15:59
NobodyCamdtantsur: my impression is that the chassis based systems can make do this type of discovery where standard systems (non chassis based) could not? Is your imppression different then mine?16:00
dtantsurNobodyCam, maybe. My main point is not about existing of such API, my point is about bad form it comes in.16:01
openstackgerritDevananda van der Veen proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173716:01
dtantsurOf course I'm talking from standpoint of package distribution with Tuskar UI16:02
dtantsur1. something from ./tools won't get packaged; 2. script is of no use for Tuskar or any other UI16:02
NobodyCamahh this is true, I have only thought about it from a command line standpoint16:02
NobodyCamdtantsur: but it is a great example of how a operator could expand on their internal tooling if they have systems that can support such functions16:04
dtantsurheh16:04
dtantsurwell, yeah, it's a good example16:04
NobodyCamalso with out something like this why do we have the chassis object16:05
dtantsurbut my expectation from something we merge into Ironic is somewhat bigger (read: proper HTTP API)16:05
NobodyCamThe thing I liked about this was that there was no api changes16:05
dtantsuraha, that's exactly what I hate ;-)16:06
NobodyCamI think thats where we are not seeing eye to eye16:06
NobodyCam:)16:06
NobodyCamwhich is okay and good16:06
*** Marga_ has quit IRC16:07
* dtantsur is thinking of 'ironic-receipts' repo with good tooling for Ironic16:07
*** Marga_ has joined #openstack-ironic16:07
rloodtantsur: your objection isn't to the script/tool, but where it is located?16:08
dtantsurrloo, to both. my objection is not to the usefulness.16:08
* NobodyCam thought about a (tools/)contrib dir16:08
dtantsurrloo, even if we make a proper executable, it's of no use for UI16:08
rloodtantsur: curious about the UI thing. Does everything we put in ironic repo, have to be accessible by a UI?16:10
dtantsurrloo, I hope so (replace UI with any other HTTP API consumer actually)16:11
Shrewshrm, i sort of get dtantsur's point... it feels a bit "strange" to have a tool doing something that looks like it _could_ be modeled within our API.16:11
Shrewsthough, i also see the usefulness of the tool16:11
* Shrews adds no real value to the conversation :(16:12
rlooShrews: good point!16:12
devanandadtantsur: ironic-dbsync does not have any HTTP API16:13
NobodyCamthat was my thought... it that it has some value in a couple of areas, useful for folks that have hardware that can support it, good example of how an operator could expland on our default tooling,16:13
*** ramineni has quit IRC16:14
dtantsurdevananda, I'm not sure I get the parallel. dbsync is pure maintenance thingy, not something we provide as functionality16:14
dtantsuryou can also say we don't have HTTP API to call ironic-api script :D16:15
*** coolsvap is now known as coolsvap_16:15
* jroll tries to read and figure out wheat we're talking about16:16
* lucasagomes same as jroll16:16
jroll:P16:17
jrollmorning everybody16:17
lucasagomesdevananda, jroll rloo morning16:17
dtantsurjroll, lucasagomes, https://review.openstack.org/#/c/134866/16:17
dtantsurjroll, morning :)16:17
NobodyCamlucasagomes: jroll: https://review.openstack.org/#/c/134866/16:17
NobodyCamlol16:17
* dtantsur was faster16:17
jrollyeah, just clicked, morning everyone :)16:17
rloohi lucasagomes, morning jroll16:17
*** Nisha has joined #openstack-ironic16:18
Shrewsmorning jroll. hi lucasagomes.16:18
*** Marga_ has quit IRC16:18
NobodyCammorning jroll :)16:18
jrolloh snap we have a meeting today16:18
*** sdake has joined #openstack-ironic16:19
*** Marga_ has joined #openstack-ironic16:19
sdakehey folks - shouldn't a nova boot with pxe+ssh agent try to boot the deploy ramdisk/kernel first?16:19
sdakeI am getting a "Failed to load ldlinux.c3216:19
sdakeBoot failed: press a key to retry, or wait"16:19
Nisharloo, dtantsur lucasagomes addressed the commensts on https://review.openstack.org/#/c/141012/16:19
Nisha and posted the new patch16:19
jrollsdake: yes, it should16:19
sdakeit seems to fail to load the deploy ramdisk or kernel for some reason16:20
sdakeany tips jroll?16:21
rloothx Nisha, will look later today16:21
sdakeI can see the deploy ramdisk and kernel in tftpboot directory so its definately there16:21
Nishadtantsur, saw your comment on states patch16:21
lucasagomessdake, https://review.openstack.org/#/c/118194/ see if that helps16:21
sdakelcuasgomes applying thanks16:22
lucasagomesthat's just docs16:22
sdakeoh docs16:22
sdakelet me read then :)16:22
lucasagomessdake, so for syslinux 6, you have to copy the syslinux modules to the tftpfolder16:22
Nishadtantsur, i would like to get others opinion on dropping INSPECTED state from inspection transient states16:22
sdakehas devstack been updated with this change?16:22
lucasagomessdake, I'm not sure, I can take a look16:23
lucasagomesif not we should do it16:23
lucasagomescause next fedora (21) and ubuntu will require that step16:23
sdakethat is odd, because it was working for me a month ago and I don't think I've upgraded since16:23
lucasagomessdake, I see... hmm not sure, can you check the version ur using?16:23
Nishalucasagomes, jroll NobodyCam dtantsur has posted a comment on https://review.openstack.org/#/c/147857/ (add states for inspection patch) to drop the state INSPECTED from the state model.16:24
NishaIt would be great to have you guys also put in your opinion on that16:24
sdakelucasgomes I am definately using f2116:25
sdakemaybe  I upgraded and it flashed out of mind :)16:25
sdakehow would I ge t the right files into a devstack setup?16:25
lucasagomessdake, we need to modify the lib/ironic to copy those files (if they exist)16:26
lucasagomesso it would work with syslinux < 4 too16:26
sdakewhere do the files come from?16:26
lucasagomes<=*16:26
lucasagomesfrom the syslinux package16:26
sdake(presently, the pxelinux.0 file comes from where?)16:26
lucasagomesalso from that package16:26
sdakecool I'll sort it out16:26
lucasagomesbut before pxelinux.0 was a standalone binary16:26
lucasagomeswhere after version > 4 they splitted the arch modules out16:27
NishaNobodyCam, there?16:27
sdakeyup got it - although there is no pxelinux.0 in the syslinux packagei n fedora 21 :)16:27
sdake /usr/share/syslinux it appears16:27
lucasagomessdake, http://www.syslinux.org/wiki/index.php/Library_modules16:28
lucasagomesno? hmm can you check "yum provides" ?16:28
lucasagomeslemme see if I can find16:28
sdakeloosk like it just needs a couple files from that directory rather then the 50 or so in /usr/share/syslinux16:29
lucasagomessdake, syslinux-tftpboot16:29
openstackgerritShivanand Tendulker proposed openstack/ironic: Code changes for secure boot implementation for ILO drivers  https://review.openstack.org/15397416:29
NobodyCammaybe16:29
naohirotlucasagomes: good morning, I happen to be working on the pxe/tftp setting and have question16:30
*** ukalifon1 has quit IRC16:31
NobodyCamNisha: Whats up?16:31
lucasagomesnaohirot, whats up?16:31
NishaNobodyCam, First request review for https://review.openstack.org/#/c/14101216:32
Nisha:)16:32
Nishalucasagomes, ^^^16:32
naohirotlucasagomes: I'm using Ubuntu 14.04 and dpkg --list syslinux says 3:4.05+dfsg16:32
*** derekh has quit IRC16:32
sdakelucasgomes copying /usr/share/syslinux/* to /opt/stack/data/ironic/tftpboot fixes it16:32
sdakethanks for the tips :)16:32
naohirotlucasagomes: it seems syslinux is ver 4.05, but I could find /usr/lib/syslinux/modules/*/ldlinux.*16:33
lucasagomessdake, cool, mind filling a bug on devstack?16:33
sdakewill do16:33
lucasagomessdake, ta much!16:33
stendulkerHow do I pass multiple capabilities in the  command line using 'ironic node-update'. Somehow it interpets passed dict as a single value.16:33
lucasagomesnaohirot, yea, it's only for versions > 416:33
lucasagomesnaohirot, http://www.syslinux.org/wiki/index.php/Library_modules16:34
jlvillalJoshNang: Question on cleaning.  So CLEANFAIL can go to AVAILABLE?  https://review.openstack.org/#/c/153444/3/ironic/common/states.py16:34
NishaSecondly wanted an opinion on "to drop state INSPECTED from inspection transient states". This is a comment posted on states code patch. https://review.openstack.org/#/c/147857/16:34
naohirotlucasagomes: yes ver 4.05 > 4 right?16:34
lucasagomesstendulker, capabilities in Ironic is a string with key:value,key:value format :(16:34
lucasagomesnaohirot, I mean, major version > 416:34
lucasagomeslike 5.*+16:35
*** dlpartain has joined #openstack-ironic16:35
lucasagomesnaohirot, for ur version only copying pxelinux.0 should be enough16:35
naohirotlucasagomes: I see, ver >= 5.0, I got it.16:35
lucasagomesyes sorry for the confusion16:35
naohirotlucasagomes: thinks :)16:35
naohirotlucasagomes: s/thinks/thanks/16:36
lucasagomesstendulker, I would love to treat capabilities as a dict, and have a fallback to string for backwards compat16:36
lucasagomesstendulker, didn't have time to work on that yet16:36
JoshNangjlvillal: my thought there was if the operator doesn't actually think the failure should prevent the node from being scheduled, they can manually put it in available. otherwise nodes could get stuck16:36
lucasagomessince our API support dicts and so on :) (before it didn't)16:36
*** gridinv has quit IRC16:36
lucasagomesnaohirot, yvw16:36
stendulkerlucasgomes: Thanks you. I think same is true for instance_info as well16:36
jlvillalJoshNang: Okay, that makes sense.  thanks :)16:36
lucasagomesstendulker, yeah, we just recently added JSON fields in the API, this cycle16:37
sdakelucasgomes thanks for troubleshooting that for me : https://bugs.launchpad.net/devstack/+bug/141986716:37
openstackLaunchpad bug 1419867 in devstack "Ironic is busted with Fedora 21" [Undecided,New]16:37
lucasagomesso some of the options still not using it16:37
lucasagomessdake, no problem :) yvw! thanks for filling that out16:37
JoshNangjlvillal: no problem!16:38
lucasagomessdake, I will put that on my list of things to fix if nobody assign to it16:38
openstackgerritMerged openstack/ironic: Fix chown invalid option -- 'p'  https://review.openstack.org/15404216:38
*** ndipanov has quit IRC16:39
stendulkerlucagomes: so does, instance_info field will accept dict from ironic cli now?16:40
stendulkerlucasgomes: so does, instance_info field will accept dict from ironic cli now?16:40
*** derekh has joined #openstack-ironic16:41
NobodyCamNisha: I will take a look after the meeting16:41
Nishaok16:41
*** lazy_prince is now known as killer_prince16:42
lucasagomesstendulker, yeah the fields does accept dicts... like $ node-update <node uuid> add instance_info/foo='{"foo": "bar"}' for e.g16:42
*** rwsu has joined #openstack-ironic16:44
lucasagomesstendulker, any json primitive really... could be a list too16:44
stendulkerlucasgomes: thank you. I'll it  out.16:45
lucasagomescool np :)16:45
*** Marga_ has quit IRC16:45
*** Marga_ has joined #openstack-ironic16:46
openstackgerritLucas Alvares Gomes proposed openstack/ironic: Add support for local boot  https://review.openstack.org/14618916:47
rameshg87lucasagomes, do you have the corresponding change in dib for local boot support out for review ?16:47
*** ijw has quit IRC16:47
lucasagomesrameshg87, yup, it's merged on DIB already16:47
* lucasagomes finds the review16:47
lucasagomesrameshg87, https://review.openstack.org/#/c/146097/16:48
lucasagomesbut we still need that patch in nova to pass the capabilities16:48
rameshg87lucasagomes, oh great .. thanks16:49
lucasagomesnp16:49
rameshg87lucasagomes, had one question16:50
rameshg87lucasagomes, when we do try_set_boot_device(task, boot_devices.DISK) to get the node booted from disk16:50
rameshg87lucasagomes, we would hit the same problem as what we tried to address in root-device-hints16:50
NobodyCamten minutes16:50
rameshg87lucasagomes, i mean we are not saying which bare metal disk to boot from16:51
NobodyCamrameshg87: still want raid spec on the agenda?16:52
rameshg87lucasagomes, so if we insalled image on disk B. there is image installed on some other disk A, and the bare metal might actually boot from A16:52
lucasagomesrameshg87, right, not sure if our interface to set the boot device actually supports telling which disk to boot from specifically16:52
lucasagomesI suspect that machines will fallback, only one disk is bootable16:52
rameshg87NobodyCam, no :), but still discuss if someone has some opinion against it16:52
NobodyCam:)16:52
stendulkerlucasgomes: Is the patch for supporting  instance_info as dict is merged?16:52
lucasagomesstendulker, yeah, are u on master?16:52
rameshg87lucasagomes, sirushti did something similar but i think pretty useful in https://review.openstack.org/#/c/150142/16:53
lucasagomesalso you may need to update the python-ironicclient16:53
NobodyCammjturek1: rady for your topic, is it still vaild for this next meeting?16:53
rameshg87lucasagomes, oh i am talking about localboot16:53
lucasagomesrameshg87, yeah the full disk image will have the same problem right?16:53
* lucasagomes checks the code16:54
rameshg87lucasagomes, but we boot from network in that case16:54
mjturek1NobodyCam, yeah shouldn't take much time. If we can't get to it, no worries16:54
rameshg87lucasagomes, and find the mbr identifier of disk and use it in the pxe template16:54
lucasagomesrameshg87, oh right, so boot from network and tell in the pxe config to local boot?16:55
lucasagomesI see16:55
rameshg87lucasagomes, yeah, but i guess here the problem statement is different16:55
NobodyCammjturek1: your really the only topic on the agenda :-p16:55
rameshg87lucasagomes, we need the machine to boot locally and not from net16:55
lucasagomesyeah that would work too, tho what I want to achieve on the local boot is to not depend on network boot at all (think about ur control plane)16:55
*** dlpartain has quit IRC16:55
lucasagomesrameshg87, +116:55
mjturek1NobodyCam hahaha okay, well I'll still keep it brief :)16:56
NobodyCam:) hehehehe16:56
lucasagomesrameshg87, so it boils down to, whether the interface supports specifying which disk we should boot from16:56
lucasagomesidk how I can do that via ipmitool, which is what sets the boot device for my case ;/16:56
lucasagomesthe machines will fallback, until a disk which is bootable (contians a bootloader etc) is found16:57
rameshg87lucasagomes, yeah, i don't think any bios machine will allow to specify which disk to boot from16:57
rameshg87lucasagomes, does bios fw loop over all disks until it finds one bootable ?16:57
rameshg87lucasagomes, any idea ?16:57
lucasagomesrameshg87, yeah it does, or should do at least16:57
lucasagomesi can try that out16:57
rameshg87lucasagomes, okay, i will just see what happens on our machines too16:58
lucasagomesrameshg87, cool yeah, so in conjunction with cleaning state16:58
lucasagomeswe should be safe guaranteeing that only one machine is bootable16:58
*** Guest69804 is now known as dan_16:58
rameshg87lucasagomes, just wondering if this is as predictable to work as all our other deployment mechanisms16:58
jrolllucasagomes: for local boot stuff, some hardware will fall back to pxe, jfyi16:58
rameshg87lucasagomes, ah okay ...16:58
*** dan_ is now known as dank_16:58
lucasagomesjroll, right, if network boot is enabled right?16:59
lucasagomesand part of the list of boot devices16:59
*** Marga_ has quit IRC16:59
lucasagomeswhich I think we only enable one at time16:59
jrolllucasagomes: enabled in bios, yes, which it needs to be for ironic to work :P16:59
lucasagomeswhen we set_boot_device16:59
jrollno, it will fall back16:59
lucasagomes:(16:59
lucasagomeswell16:59
jrollthere's some ipmitool magic that can be done16:59
lucasagomesI clean the pxe config files16:59
rameshg87jroll, without looking at all the disks ?16:59
*** Marga_ has joined #openstack-ironic16:59
lucasagomesafter I deploy with local boot, so the machine won't boot anyway16:59
jrollwhich involves force_pxe and persistent=false, but it might be just our hardware17:00
jrollah, cool17:00
lucasagomesand I don't generate pxe config files at take_over17:00
jrollrameshg87: I assume it looks at all disks first17:00
lucasagomesso the boot will fail, but yeah it won't boot into the image or any other image via network17:00
lucasagomeswhich I think is good enough17:00
jrollyeah, cool17:00
jrollI haven't looked at code yet so thought I would point that out :)17:00
* jroll wonders what happened with partition images for the agent driver17:01
lucasagomesjroll, :) yeah it's a valid point, thanks17:01
lucasagomesjroll, also wanted to talk to u about IPA17:01
jrollindeed17:01
jrollmeeting is right now, want to chat after or?17:01
lucasagomesquick q... after the machine boots, is the heartbeat function that triggers it to continue the deployment right?17:01
lucasagomescool yeha17:01
lucasagomesnp17:01
jrolloh, yep17:02
jrollthe heartbeat works like the pxe ramdisk callback17:02
lucasagomesgotcha17:02
lucasagomesyeah I gotta refactor that out of the agent17:02
*** coolsvap_ is now known as coolsvap17:02
lucasagomes(lookup as well)17:02
jrolllucasagomes: why's that?17:02
lucasagomesto a common place so other drivers can use it17:02
lucasagomesjroll, IPA as the default ramdisk17:02
jrollah, right17:03
NobodyCammeeting time17:08
*** ijw has joined #openstack-ironic17:10
*** jistr has quit IRC17:10
*** rwsu has quit IRC17:10
*** mrda-away is now known as mrda-sfo17:11
*** rwsu has joined #openstack-ironic17:11
* jroll makes a note to wonder about localboot being a flavor property17:11
jrollmrda-sfo: are you here?17:13
*** EmilienM is now known as EmilienM|afk17:15
*** Marga_ has quit IRC17:18
mrda-sfojroll: yes17:22
jroll:P17:23
*** jerryz has joined #openstack-ironic17:24
*** ijw has quit IRC17:24
Shrewsmrda-sfo: where are you staying?17:24
mrda-sfoI'm at the Pickwick17:25
*** ijw has joined #openstack-ironic17:26
*** jmank has joined #openstack-ironic17:27
*** athomas has quit IRC17:27
* jlvillal thinks there are some big patch sets to review today :)17:28
*** zigo has quit IRC17:29
openstackgerritRamakrishnan G proposed openstack/ironic-specs: In-band RAID configuration using agent ramdisk  https://review.openstack.org/14780317:34
*** ijw has quit IRC17:35
*** ijw has joined #openstack-ironic17:38
*** kbyrne has quit IRC17:40
*** krtaylor has quit IRC17:41
dtantsurone more spec touching management interface: https://review.openstack.org/#/c/135845/17:41
openstackgerritSirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images  https://review.openstack.org/15014217:42
*** kbyrne has joined #openstack-ironic17:44
*** ijw has quit IRC17:49
*** ijw has joined #openstack-ironic17:50
*** spandhe has joined #openstack-ironic17:51
*** krtaylor has joined #openstack-ironic17:54
*** eghobo has joined #openstack-ironic17:55
*** jcoufal has quit IRC17:57
openstackgerritDmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for boot interface detection  https://review.openstack.org/15415717:57
*** ifarkas has quit IRC18:00
NobodyCamgreat meeting all18:00
NobodyCamThank you18:00
* mrda-sfo is glad he could make this fortnight's meeting :)18:00
NobodyCam:)18:00
lucasagomes:)18:01
lucasagomesright and I will call it a day18:01
lucasagomeshave a good night everyone18:01
lucasagomestalk to you all soon18:01
jrollnight lucasagomes :)18:01
lucasagomesenjoy the meeting in SF tomorrow18:01
dtantsurg'night18:01
rameshg87goodnight all :)18:01
*** dtantsur is now known as dtantsur|afk18:01
lucasagomes(is it tomorrow right?)18:01
devanandaI need to run to the russian consulate today ... will be offline for a few hours18:01
devanandamrda-sfo: i'm glad you made it too -- and looking forward to seeing you on wednesday18:02
*** lucasagomes is now known as lucas-dinner18:02
lucas-dinnerdevananda, g'luck18:02
rameshg87jroll, JayF, JoshNang, just put up a spec for in-band raid configuration using ipa using hardware manager: https://review.openstack.org/#/c/147803/. please have a look at it when you get some time18:02
mrda-sfoI'm here in SFO now, and am looking forward to it18:02
jrollrameshg87: cool, thanks :)18:03
NobodyCamhave a good night lucas-dinner18:03
*** rameshg87 has quit IRC18:04
*** harlowja has joined #openstack-ironic18:04
NobodyCambrb... popping out for quick smoke18:04
*** derekh has quit IRC18:05
*** opelina has joined #openstack-ironic18:07
*** Nick_A has joined #openstack-ironic18:07
*** opelina has quit IRC18:07
mrda-sforloo: review 141737 would welcome a re-+2'ing :)18:12
rloojroll: wrt the weekly status reports, how would you like me to help you? remind you ... do it?18:13
mrda-sforloo: jroll is afk for the moment, please hold18:13
rloomrda-sfo: will do. with the next 2 hours I think. gotta eat lunch and have a meeting.18:14
mrda-sfocool, thanks rloo18:14
rloomrda-sfo: and maybe shovel some snow18:15
mrda-sfooh, no snow here in... where I am... SFO18:15
*** jxiaobin has joined #openstack-ironic18:17
NobodyCamgoing to be 85F in palm springs today :-p18:17
jrollrloo: if you want to do it go ahead :D18:17
jrollI'm not in a good email situation at the moment, that would be awesome if you did18:18
rloojroll: ok, i'll do it sometime today.18:18
jrollthanks!18:19
NishaNobodyCam, jroll reviews for https://review.openstack.org/#/c/141012/ :) ....18:20
Nisharloo, thanks for the review18:20
*** krtaylor has quit IRC18:27
* naohirot good night > all18:28
*** naohirot has quit IRC18:29
*** regebro has joined #openstack-ironic18:29
stendulkerjroll: Hi18:30
jrollstendulker: hey, what's up18:30
stendulkerjroll: is configdrive passing mandatory in ironic cli to put node into active state?18:31
jrollstendulker: no18:31
NobodyCamNisha: +1'd with a comment18:31
stendulkerjroll: I get this error upon updating python-ironic-client18:32
stendulker'Namespace' object has no attribute 'configdrive'18:32
*** korekhov has joined #openstack-ironic18:32
stendulkerjroll: 'Namespace' object has no attribute 'configdrive'18:32
jrollstendulker: I didn't work on the client side...18:32
regebroStupid newbie question: Where does ironic logs end up? I can't find anything...18:33
NishaNobodyCam, sure will file a bug once this is in18:33
stendulkerjroll: Looks like there is an explicit check for that in cli side18:34
jrollregebro: using devstack?18:34
NishaNobodyCam, thanks18:34
jrollstendulker: pointer to the code?18:34
regebrojroll: instanck, actually.18:34
regebrostanck, lol. instack, I mean.18:34
jrollregebro: idk then... /var/log/ironic?18:35
regebroNop, /var/log/messages apparently...18:36
regebroOr well, i haven't seen any errors yet. :-)18:36
jrollstendulker: so what are you looking at, I don't see anything that looks bad to me on the client side18:36
stendulkerjroll: yes tht chk looks fine18:37
jrollstendulker: can you paste the error and usage etc18:37
jrolllike, the full traceback with --debug18:38
*** stendulker_ has joined #openstack-ironic18:39
stendulker_jroll: Here it is http://paste.openstack.org/show/170128/18:40
stendulker_jroll: facing this after updating  ironic and python-ironicclient18:41
*** Marga_ has joined #openstack-ironic18:42
jrollstendulker_: huh.18:42
regebroyeah, the nodes are in provision state "wait callback" and never get out of that.18:42
*** stendulker has quit IRC18:42
*** achanda has joined #openstack-ironic18:43
jrollregebro: check the console for the baremetal nodes18:43
*** zer0c00l has quit IRC18:43
*** zer0c00l has joined #openstack-ironic18:44
regebroooooh. No bootable device.18:44
*** coolsvap is now known as coolsvap_18:45
NishaNobodyCam, when we say *ED be no-op state means how do we add in states.py? we dont add any transition to them? does it mean that?18:45
*** eghobo has quit IRC18:46
*** coolsvap_ is now known as coolsvap18:46
*** coolsvap is now known as coolsvap_18:47
*** achanda has quit IRC18:47
stendulker_jroll: do i need to update any other component for it?18:47
jrollstendulker_: I don't think so, it should default to None...18:48
jrollstendulker_: I'd love a bug filed for this, and you could try to play with the code and find the problem18:48
jrollstendulker_: does it work if you pass in a configdrive?18:48
stendulker_jroll: Never tried creating one... Is there a instruction to create one and how to pass it ?18:49
jrollstendulker_: just pass in any string and see if that gets you past this error, that could help debug18:53
*** ChuckC has quit IRC18:53
*** Marga_ has quit IRC18:53
*** achanda has joined #openstack-ironic18:53
stendulker_jroll: tried that. it does not work18:53
jrollstendulker_: same error?18:53
jrollstendulker_: what was your command when you did that?18:53
jrollOH18:54
stendulker_jroll: ironic  node-set-provision-state  --config-drive=None $NODE65 active18:54
jrolllol, wow18:54
jrollI think I see it, if you file a bug I'll post a patch18:54
*** ChuckC has joined #openstack-ironic18:54
Nishajroll, even i faced this issue18:54
jrollNisha: did you report a bug?18:55
Nishajroll, no :) i was trying inspection code18:55
stendulker_jroll: ok.will file a defect18:55
jrollthanks18:55
* jroll makes a patch18:55
stendulker_jroll: Defect is https://bugs.launchpad.net/ironic/+bug/141994018:58
openstackLaunchpad bug 1419940 in Ironic "ironic CLI gives error when --config-drive feature enabled bits are used. " [Undecided,New]18:58
jrollthank you!18:58
jrollI need to restack and then I'll fix this18:59
stendulker_jroll: thanks for your help :) Will test with your patch.18:59
jrollno problem :)18:59
jrollactually, I will post the patch now and test shortly19:00
*** ijw has quit IRC19:01
stendulker_jroll: Pls update patch details in bug. Will test it tomorrow morning. Feeling sleepy.. Its past midnight here19:01
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Fix metavar for configdrive argument  https://review.openstack.org/15420019:01
jrollstendulker_: there's the patch19:01
jrollpretty simple :)19:01
jrollhave a good night, see you tomorrow :)19:01
stendulker_jroll: cool :)19:02
stendulker_jroll: will test it right away19:02
jrolloh, thank you!19:03
*** zigo has joined #openstack-ironic19:03
*** Marga_ has joined #openstack-ironic19:04
*** ijw has joined #openstack-ironic19:04
*** gridinv has joined #openstack-ironic19:05
stendulker_jroll: still same error19:06
jrolloh?19:06
jrolldid you install from source? (python setup.py install)19:06
stendulker_jroll: yes19:06
*** eghobo has joined #openstack-ironic19:06
jrollhrm19:06
jrollok, I'll play around19:06
*** jerryz has quit IRC19:09
* jroll waits for devstack19:11
*** PaulCzar has joined #openstack-ironic19:11
*** EmilienM|afk is now known as EmilienM19:12
*** Marga_ has quit IRC19:12
*** Marga_ has joined #openstack-ironic19:13
*** pelix has quit IRC19:17
*** regebro has quit IRC19:18
*** gridinv has quit IRC19:18
*** Nisha has quit IRC19:19
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Fix argument for configdrive usage  https://review.openstack.org/15420019:24
jrollstendulker_: ^ should be good now19:24
*** achanda has quit IRC19:25
stendulker_jroll: ok.let me try it out19:25
jrollI tested locally19:26
*** achanda has joined #openstack-ironic19:27
*** ChuckC_ has joined #openstack-ironic19:28
*** ChuckC has quit IRC19:29
stendulker_jroll: Thanks it works. :)19:30
*** blinky42 has joined #openstack-ironic19:31
jrollno problem :)19:32
openstackgerritVictor Lowther proposed openstack/ironic: Add DRAC BIOS config vendor passthru API -- DO NOT MERGE  https://review.openstack.org/15048819:37
*** andreykurilin_ has joined #openstack-ironic19:38
*** openstackgerrit has quit IRC19:41
*** openstackgerrit has joined #openstack-ironic19:41
*** gridinv has joined #openstack-ironic19:43
rloomrda-sfo: reviewed 141737 (again)19:51
openstackgerritJosh Gachnang proposed openstack/ironic: Implement Cleaning States  https://review.openstack.org/15344419:51
*** regebro has joined #openstack-ironic19:52
*** sdake is now known as sdake__19:58
*** gridinv has quit IRC19:59
*** krtaylor has joined #openstack-ironic19:59
*** andreykurilin_ has quit IRC20:00
*** ijw has quit IRC20:03
*** stendulker_ has quit IRC20:05
*** dprince has quit IRC20:10
*** ijw has joined #openstack-ironic20:34
*** gridinv has joined #openstack-ironic20:36
*** Guest46399 has joined #openstack-ironic20:37
*** BadCub_ has quit IRC20:38
rloojroll: sent out subteam status reports. When do you usually clear out the status from the etherpad? Is it OK to do it now?20:39
*** BadCub_ has joined #openstack-ironic20:40
*** anderbubble has joined #openstack-ironic20:51
NobodyCamrloo: anytime after email is good for /me :)20:52
rlooNobodyCam: that's what I was thinking, but thought I'd check with jroll. If it is different, I may have to deprecate what we used to do :D20:52
NobodyCamhehehe20:53
*** korekhov has quit IRC20:54
*** korekhov_ has joined #openstack-ironic20:55
*** ijw has quit IRC21:06
mrda-sfothanks rloo21:09
*** dmellado has quit IRC21:16
*** dmellado has joined #openstack-ironic21:16
*** Marga_ has quit IRC21:19
*** Marga_ has joined #openstack-ironic21:20
*** achanda has quit IRC21:20
*** ChuckC_ has quit IRC21:20
*** ChuckC_ has joined #openstack-ironic21:21
*** ChuckC_ is now known as ChuckC21:21
openstackgerritMichael Davies proposed openstack/ironic: Add logical name support to Ironic  https://review.openstack.org/14173721:21
*** dmellado has quit IRC21:25
*** ChuckC has quit IRC21:38
*** ChuckC has joined #openstack-ironic21:39
NobodyCambbiafm21:40
*** s8weber has quit IRC21:40
*** ChuckC_ has joined #openstack-ironic21:42
*** Marga_ has quit IRC21:45
*** ChuckC has quit IRC21:46
*** Marga_ has joined #openstack-ironic21:46
*** achanda has joined #openstack-ironic21:54
*** ijw has joined #openstack-ironic21:58
*** dmellado has joined #openstack-ironic22:00
*** korekhov_ has quit IRC22:00
*** ijw has quit IRC22:00
*** korekhov has joined #openstack-ironic22:01
*** ijw has joined #openstack-ironic22:01
*** romcheg has quit IRC22:10
*** korekhov has quit IRC22:15
*** korekhov has joined #openstack-ironic22:15
*** mrda-sfo is now known as mrda22:17
*** eguz has joined #openstack-ironic22:19
*** eghobo has quit IRC22:19
*** korekhov has quit IRC22:19
*** dmellado has quit IRC22:20
*** mriedem has joined #openstack-ironic22:25
mriedemanybody known which commit fixed this in the last few days? http://logs.openstack.org/12/153312/1/gate/gate-tempest-dsvm-ironic-agent_ssh/f07b9ab/logs/screen-ir-cond.txt.gz?level=TRACE22:25
mriedemhttp://goo.gl/qSLqmi22:25
mriedembingo https://bugs.launchpad.net/ironic/+bug/141883322:29
openstackLaunchpad bug 1418833 in Ironic "check-tempest-dsvm-ironic-agent_ssh fails due to ConfigDriveWriteError" [Critical,Fix committed] - Assigned to Jay Faulkner (jason-oldos)22:29
*** eguz has quit IRC22:30
*** eghobo has joined #openstack-ironic22:32
jrollrloo: clearing after email is fine, thank you so much for doing that22:37
jrollmriedem: that's correct22:38
mriedemjroll: thanks just wanted to get it tracked with elastic-recheck22:38
jrollah, cool22:39
rloothx jroll. Done!22:40
*** vipul has quit IRC22:42
*** mriedem has left #openstack-ironic22:44
*** digimonde has joined #openstack-ironic22:44
*** vipul has joined #openstack-ironic22:45
*** absubram has quit IRC22:49
*** gridinv has quit IRC22:51
*** dmellado has joined #openstack-ironic22:52
*** dprince has joined #openstack-ironic22:52
*** aweeks has quit IRC22:54
*** dmellado has quit IRC22:56
*** dmellado_ has joined #openstack-ironic22:56
*** dmellado_ is now known as dmellado22:57
*** aweeks has joined #openstack-ironic22:58
*** Haomeng has quit IRC22:59
*** Marga_ has quit IRC23:01
*** Marga_ has joined #openstack-ironic23:01
mrdaJust to confirm, python-ironicclient hasn't been updated for microversions yet?  i.e. If I wanted to use ironicclient to invoke functionality protected by a specific microversion, I would first need to enable this in the client?23:01
*** spandhe has quit IRC23:02
*** Marga_ has quit IRC23:02
*** andreykurilin_ has joined #openstack-ironic23:03
*** Marga_ has joined #openstack-ironic23:03
*** Haomeng has joined #openstack-ironic23:03
*** Ng has quit IRC23:06
rloomrda: yes, I believe that is true. seems like i saw a bug opened for that, let me look23:07
*** cppforlife_ has quit IRC23:07
rloomrda: https://bugs.launchpad.net/python-ironicclient/+bug/141743023:08
openstackLaunchpad bug 1417430 in python-ironicclient "API microversion" [High,Triaged] - Assigned to Tan Lin (tan-lin-good)23:08
mrdarloo: ta23:09
mrdarloo: mrda has a look23:09
*** aweeks has quit IRC23:09
*** enikanorov_ has quit IRC23:09
mrdaI need to consider this so I can get logical-name support it as per my promise to deva :)23:09
rloomrda: maybe ping tan. we should get that in asap.23:10
mrdaya23:10
*** penick has joined #openstack-ironic23:10
*** mjturek1 has quit IRC23:10
rloomrda: hmm, i hadn't thought about the client changes for your stuff. more than just passing the microversion.23:10
rloomrda: dinner bell is ringing. gotta go. ciao.23:11
mrdaHave fun rloo23:11
*** rloo is now known as rloo_afk23:11
*** spandhe has joined #openstack-ironic23:11
mrdarloo: FWIW, we're just using patch, so it does work out of the box via 'ironic node-update $NODE_ID replace name=foobarish' if we had microversion support23:12
*** lucas-dinner has quit IRC23:13
*** ijw has quit IRC23:15
*** s8weber_ has joined #openstack-ironic23:20
*** cppforlife_ has joined #openstack-ironic23:22
*** cppforlife_ has quit IRC23:23
jrolldevananda: https://review.openstack.org/#/c/154200/23:24
jrollwe might want to land that and release a client, node-set-provision-state active is completely broken23:25
*** regebro has quit IRC23:25
*** EmilienM is now known as EmilienM|afk23:28
*** anderbubble has quit IRC23:32
NobodyCamjroll: is 154200 the fix?23:32
*** jgrimm is now known as zz_jgrimm23:35
*** dmellado has quit IRC23:36
jrollNobodyCam: yes23:36
jrollstendulker and myself both tested it23:37
NobodyCam+2'd23:38
*** dmellado has joined #openstack-ironic23:38
digimondejroll: bug #1419940 is not about --config-drive argument or --configdrive23:39
openstackbug 1419940 in Ironic "ironic CLI gives error when --config-drive feature enabled bits are used. " [Undecided,Incomplete] https://launchpad.net/bugs/1419940 - Assigned to Erhan Ekici (erhan-ekici)23:39
digimondethere is no any implementation for configdrive in ironicclient version 0.3.2 which bug reporter uses?23:39
NobodyCamjroll: is our gate busted as result?23:40
jrolldigimonde: there is, and that's the bug23:40
jrollNobodyCam: no, this is cli only23:40
NobodyCamack :)23:40
*** korekhov has joined #openstack-ironic23:40
jrolldigimonde: there is a --config-drive argument in 0.3.2, and it's implemented incorrectly such that all node-set-provision-state commands fail23:40
jrolldigimonde: this same error occurs even with the --config-drive argument23:42
jrollactually, you might be right, 0.3.2 doesn't have this, that's weird that the bug specifies that23:44
digimondejroll: from 0.3.223:45
digimondejroll: def do_node_set_provision_state(cc, args):23:45
digimonde    """Provision, rebuild or delete an instance."""23:45
digimonde    cc.node.set_provision_state(args.node, args.provision_state)23:45
digimondethat code from node_shell.py (0.3.2)23:45
jrollright, I was wrong, sorry23:45
jrollhowever23:45
digimondenp23:45
jrollthe latest version is where this bug is seen23:46
*** andreykurilin_ has quit IRC23:51
*** sdake__ is now known as sdake_23:53
*** dprince has quit IRC23:54
digimondejroll: yes, I see, it is the latest version bug is seen.23:54
*** eghobo has quit IRC23:59

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