Tuesday, 2014-09-09

jrolldevananda, NobodyCam, I'm debating if we should keep IPA docs in the IPA repo, Ironic repo, or some of both... I think I'd almost prefer Ironic repo as using IPA outside of Ironic would be weird (but maybe dev docs should be in IPA repo)00:11
openstackgerritA change was merged to openstack/ironic-python-agent: Fix minor documentation issues  https://review.openstack.org/11985900:11
*** mitz has quit IRC00:15
JayFjroll: ^ watching the post job now00:16
jrollcool00:17
*** mitz_ has joined #openstack-ironic00:18
JayFhttp://tarballs.openstack.org/ironic-python-agent/coreos/files/ \o/00:22
JayFjroll: ^00:22
JayFnow to check to see if the urls I put in the readme are right, haha00:22
jroll\o/ thanks00:22
JayFhah, they're wrong00:23
JayFhold please00:23
jroll(now how do I ignore the preserve-ephemeral test) :(00:23
jrollwat00:23
* jroll checks devstack00:23
JayFno I mean, the urls I created work00:23
JayFbut the ones in the readme are wrong00:23
jrollneat, my stuff is good00:24
jrollyeah00:24
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Add missing "/files" to IPA image URLs  https://review.openstack.org/11996300:24
JayFjroll: ^00:25
jrollyeah, landed00:26
*** eguz has joined #openstack-ironic00:26
*** eghobo has quit IRC00:29
jrollhave a good night, ironic00:30
mrdathanks jroll, tty tomorrow00:30
*** chuckC has joined #openstack-ironic00:35
*** rch has left #openstack-ironic00:38
*** penick has joined #openstack-ironic00:38
*** yongli has joined #openstack-ironic00:45
*** penick has quit IRC00:47
openstackgerritA change was merged to openstack/ironic-python-agent: Add missing "/files" to IPA image URLs  https://review.openstack.org/11996300:48
*** dlaube has quit IRC00:49
*** anderbubble has joined #openstack-ironic01:02
anderbubbledoes openstack-ironic compete with a system like xcat; or would they be considered orthogonal?01:03
anderbubbleMore to the point: if I want to provision nodes in a cluster, have used xcat (and raw pxe+kickstart) before, but want something better, should I look at extending / improving / replacing xcat, or should I be trying to use openstack-ironic?01:04
*** r-daneel has quit IRC01:22
*** rloo_afk has quit IRC01:37
*** scubacuda has quit IRC01:49
*** scubacuda has joined #openstack-ironic01:50
*** nosnos has joined #openstack-ironic01:51
*** pcrews has quit IRC02:06
*** yuriyz has quit IRC02:09
*** yuriyz has joined #openstack-ironic02:10
*** andreykurilin has quit IRC02:10
*** romcheg has quit IRC02:10
*** andreykurilin has joined #openstack-ironic02:10
*** romcheg has joined #openstack-ironic02:12
*** pcrews has joined #openstack-ironic02:23
*** openstackgerrit has quit IRC02:33
*** dlaube has joined #openstack-ironic02:57
*** dlaube has quit IRC02:59
*** vinbs has joined #openstack-ironic03:01
*** eghobo has joined #openstack-ironic03:05
*** lazy_prince is now known as killer_prince03:16
*** openstack has joined #openstack-ironic03:42
*** sendak.freenode.net sets mode: -o jroll03:46
-sendak.freenode.net- *** Notice -- TS for #openstack-ironic changed from 1410234039 to 136692279803:46
*** sendak.freenode.net sets mode: +c-s 03:46
*** pcrews has joined #openstack-ironic03:46
*** LV-426 has joined #openstack-ironic03:46
*** scubacuda has joined #openstack-ironic03:46
*** dutsmoc has joined #openstack-ironic03:46
*** stevebak` has joined #openstack-ironic03:46
*** NobodyCa1 has joined #openstack-ironic03:46
*** Hefeweizen has joined #openstack-ironic03:46
*** davidlenwell_ has joined #openstack-ironic03:46
*** victor_lowther_ has joined #openstack-ironic03:46
*** vinbs has joined #openstack-ironic03:46
*** romcheg has joined #openstack-ironic03:46
*** andreykurilin has joined #openstack-ironic03:46
*** yuriyz has joined #openstack-ironic03:46
*** nosnos has joined #openstack-ironic03:46
*** anderbubble has joined #openstack-ironic03:46
*** yongli has joined #openstack-ironic03:46
*** chuckC has joined #openstack-ironic03:46
*** mitz_ has joined #openstack-ironic03:46
*** killer_prince has joined #openstack-ironic03:46
*** sbadia has joined #openstack-ironic03:46
*** Shrews has joined #openstack-ironic03:46
*** enterprisedc has joined #openstack-ironic03:46
*** Haomeng|2 has joined #openstack-ironic03:46
*** jrist has joined #openstack-ironic03:46
*** annegentle has joined #openstack-ironic03:46
*** toabctl has joined #openstack-ironic03:46
*** vdrok has joined #openstack-ironic03:46
*** boris-42 has joined #openstack-ironic03:46
*** greghaynes has joined #openstack-ironic03:46
*** GheRivero_pto has joined #openstack-ironic03:46
*** enikanorov has joined #openstack-ironic03:46
*** krtaylor has joined #openstack-ironic03:46
*** notq has joined #openstack-ironic03:46
*** dtantsur|afk has joined #openstack-ironic03:46
*** lsmola_ has joined #openstack-ironic03:46
*** ryanpetrello has joined #openstack-ironic03:46
*** sirushti has joined #openstack-ironic03:46
*** dividehex has joined #openstack-ironic03:46
*** SpamapS has joined #openstack-ironic03:46
*** bigjools has joined #openstack-ironic03:46
*** agordeev has joined #openstack-ironic03:46
*** yuanying has joined #openstack-ironic03:46
*** kevinbenton has joined #openstack-ironic03:46
*** adam_g has joined #openstack-ironic03:46
*** harlowja has joined #openstack-ironic03:46
*** ekarlso- has joined #openstack-ironic03:46
*** zer0c00l has joined #openstack-ironic03:46
*** mrda has joined #openstack-ironic03:46
*** mmitchell has joined #openstack-ironic03:46
*** Ng has joined #openstack-ironic03:46
*** aweeks has joined #openstack-ironic03:46
*** wendar has joined #openstack-ironic03:46
*** dhellmann has joined #openstack-ironic03:46
*** devananda has joined #openstack-ironic03:46
*** slagle has joined #openstack-ironic03:46
*** rushiagr_away has joined #openstack-ironic03:46
*** marios has joined #openstack-ironic03:46
*** viktors|afk has joined #openstack-ironic03:46
*** early has joined #openstack-ironic03:46
*** keekz has joined #openstack-ironic03:46
*** EmilienM has joined #openstack-ironic03:46
*** pleia2 has joined #openstack-ironic03:46
*** russellb has joined #openstack-ironic03:46
*** pradipta_away has joined #openstack-ironic03:46
*** soren has joined #openstack-ironic03:46
*** JoshNang has joined #openstack-ironic03:46
*** zz_naotok has joined #openstack-ironic03:46
*** russell_h has joined #openstack-ironic03:46
*** dvorak has joined #openstack-ironic03:46
*** mikal has joined #openstack-ironic03:46
*** Madasi has joined #openstack-ironic03:46
*** tonycoffman has joined #openstack-ironic03:46
*** marzif__ has joined #openstack-ironic03:46
*** tteggel has joined #openstack-ironic03:46
*** kylestev has joined #openstack-ironic03:46
*** dguerri has joined #openstack-ironic03:46
*** aignatov has joined #openstack-ironic03:46
*** lifeless has joined #openstack-ironic03:46
*** jeblair has joined #openstack-ironic03:46
*** anteaya has joined #openstack-ironic03:46
*** jbjohnso__ has joined #openstack-ironic03:46
*** zigo has joined #openstack-ironic03:46
*** gilliard_ has joined #openstack-ironic03:46
*** BadCub has joined #openstack-ironic03:46
*** lynxman has joined #openstack-ironic03:46
*** JayF has joined #openstack-ironic03:46
*** cinerama has joined #openstack-ironic03:46
*** mgagne has joined #openstack-ironic03:46
*** Isotopp has joined #openstack-ironic03:46
*** mordred has joined #openstack-ironic03:46
*** morgabra has joined #openstack-ironic03:46
*** d0ugal has joined #openstack-ironic03:46
*** sendak.freenode.net changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"03:46
*** coolsvap has joined #openstack-ironic03:46
*** jroll has quit IRC03:46
*** jroll has joined #openstack-ironic03:46
*** antonym is now known as Guest3927003:46
*** killer_prince has quit IRC03:50
*** lazy_prince has joined #openstack-ironic03:50
*** lazy_prince has joined #openstack-ironic03:51
*** lazy_prince has quit IRC03:53
*** killer_prince has joined #openstack-ironic03:53
*** killer_prince has quit IRC03:53
*** lazy_prince has joined #openstack-ironic03:54
*** lazy_prince is now known as killer_prince04:00
*** pcrews has quit IRC04:06
*** HenryG has joined #openstack-ironic04:07
*** Poornima has joined #openstack-ironic04:08
*** eghobo has joined #openstack-ironic04:19
*** pcrews has joined #openstack-ironic04:21
*** rushiagr_away is now known as rushiagr04:29
*** stevebak` is now known as stevebaker04:32
*** dutsmoc is now known as comstud04:33
*** pcrews has quit IRC04:38
*** ramineni has joined #openstack-ironic04:43
*** anderbubble has quit IRC04:54
*** killer_prince is now known as lazy_prince05:01
*** rushiagr is now known as rushiagr_away05:07
*** vinbs has quit IRC05:08
*** vinbs has joined #openstack-ironic05:13
*** pradipta_away is now known as pradipta05:18
*** rushiagr_away is now known as rushiagr05:21
*** vinbs has quit IRC05:36
*** eghobo has quit IRC05:40
*** lazy_prince is now known as killer_prince05:40
*** vinbs has joined #openstack-ironic05:46
*** harlowja is now known as harlowja_away05:47
*** killer_prince is now known as lazy_prince05:56
*** lazy_prince is now known as killer_prince05:59
*** killer_prince has quit IRC06:00
*** lazy_prince has joined #openstack-ironic06:00
*** andreykurilin has quit IRC06:08
*** jcoufal has joined #openstack-ironic06:12
*** bmahalakshmi has joined #openstack-ironic06:12
*** bmahalakshmi has quit IRC06:13
*** bmahalakshmi has joined #openstack-ironic06:14
*** rakesh_hs has joined #openstack-ironic06:35
*** andreykurilin has joined #openstack-ironic06:40
*** ndipanov has joined #openstack-ironic06:42
*** andreykurilin has quit IRC06:47
*** andreykurilin has joined #openstack-ironic06:48
*** Nisha has joined #openstack-ironic06:51
mrdaHey Ironic.  When North America wakes up in a few hours, I'd be great if you could encourage 119944 to get approved and merged over in Nova.  This is the showstopper for the driver.  It's been reviewed, and the revisions have passed Jenkins.  Thanks in advance :)06:56
*** Nisha has quit IRC07:03
*** ifarkas has joined #openstack-ironic07:08
*** bmahalakshmi has quit IRC07:12
*** bluex-pl has joined #openstack-ironic07:16
*** dtantsur|afk is now known as dtantsur07:18
*** dtantsur is now known as Morning07:18
*** Morning is now known as dtantsur07:18
dtantsurMorning Ironic :)07:19
*** vinbs has quit IRC07:29
*** vinbs has joined #openstack-ironic07:30
*** jistr has joined #openstack-ironic07:33
*** jcoufal has quit IRC07:39
*** andreykurilin has quit IRC07:45
mrdahi dtantsur07:50
dtantsurmrda, g'evening07:50
*** derekh has joined #openstack-ironic07:53
*** athomas has joined #openstack-ironic08:00
*** romcheg1 has joined #openstack-ironic08:02
*** romcheg1 has quit IRC08:02
dtantsurfolks, devananda, I we need some clarification on what people treat as "affects ..." in launchpad. I clearly read it as "should be also fixed in ...", not like "has some effect, that users may attribute to ...".08:03
*** jcoufal has joined #openstack-ironic08:03
dtantsurthat means that I'm inclined to status=Invalid for ironic parts of bugs like https://bugs.launchpad.net/ironic/+bug/124802208:03
*** bluex-pl has quit IRC08:03
dtantsurcompare: I don't think Nova folks report libvirt driver bugs to libvirt upstream :)08:03
*** bluex-pl has joined #openstack-ironic08:04
dtantsurotherwise we end up with lots of open bugs which we (as Ironic project, not as individuals who also contribute to Nova) can't deal with at all, except for waiting08:04
dtantsuryour opinions?08:04
sorendtantsur: I typically see the "also affected projects" used in two ways.08:24
*** lucasagomes has joined #openstack-ironic08:24
sorendtantsur: 1. One problem that spans many projects (e.g. we want to apply the same change to translations, tox config, etc, across all openstack projects).08:24
sorendtantsur: That's a no-brainer.08:24
sorendtantsur: The other use is more user-centric. Consider a bug that *appears* to be in Nova, but is really in Ironic.08:25
sorendtantsur: A user experiencing this problem would be looking for a bug in Nova.08:25
sorendtantsur: Marking it as affecting both Nova and Ironic lets developers track it where it needs fixing and users can track it where they see it.08:26
dtantsursoren, right. but then we;08:27
dtantsursorry08:27
sorendtantsur: But the other way around... Yeah, I can't see much use to that.08:27
dtantsurwe'll really need to report libvirt driver bugs to libvirt)08:27
dtantsuralso, users can track bug even if it was closed on it's Ironic part08:27
dtantsurit's still their bug08:27
mrdaOK, enough for tonight.  See you in the morning...08:31
dtantsurmrda, g'night08:31
*** athomas has quit IRC08:42
*** pradipta is now known as pradipta_away08:56
*** andreykurilin has joined #openstack-ironic08:58
*** nikunj2513 has joined #openstack-ironic09:05
*** nikunj2513 is now known as nikunj251209:08
*** dtantsur is now known as dtantsur|lunch09:11
*** openstackgerrit has joined #openstack-ironic09:20
*** rushiagr is now known as rushiagr_away09:21
*** rushiagr_away is now known as rushiagr09:34
*** lazy_prince has quit IRC09:56
*** pelix has joined #openstack-ironic09:59
*** bluex-pl has quit IRC10:03
*** mkerrin has joined #openstack-ironic10:14
*** bluex-pl has joined #openstack-ironic10:21
openstackgerritSzymon Wróblewski proposed a change to openstack/ironic-python-agent: Enhance decorators in agent and automate creation of command_map  https://review.openstack.org/11752910:36
gilliard_dtantsur|lunch: I was just looking at that particular bug.  It seems to me that some code change might be needed in Ironic to fix this, so "affects: Ironic" looks valid.10:41
*** gilliard_ is now known as gilliard10:42
*** lazy_prince has joined #openstack-ironic10:44
*** dtantsur|lunch is now known as dtantsur10:51
dtantsurgilliard, what ironic code do you expect to change?10:52
lucasagomesdtantsur, I think ur patch in the client is failing to merge on master10:59
dtantsurOo10:59
dtantsurit was just rebased10:59
lucasagomesright, lemme check if I did something wrong here then11:00
dtantsurplease check yeah11:00
lucasagomesdtantsur, oh forget... it has a dependency11:02
lucasagomesI tried to cherry pick it11:02
lucasagomesfor tests11:02
dtantsurok11:05
gilliarddtantsur: the fix might be that ironic-api has to notify nova compute-manager that it needs to update its available resources.11:07
dtantsurgilliard, oh well... maybe. I'll leave this bug as it is.11:08
*** gilliard is now known as gillilunch11:08
*** ramineni has quit IRC11:09
openstackgerritVladyslav Drok proposed a change to openstack/ironic: Translator functions cleanup part 2  https://review.openstack.org/11858711:21
*** vinbs has quit IRC11:23
openstackgerritSzymon Wróblewski proposed a change to openstack/ironic-python-agent: Enhance decorators in agent and automate creation of command_map  https://review.openstack.org/11752911:36
*** nikunj2512 has quit IRC11:36
dtantsurlucasagomes, see my comment https://review.openstack.org/#/c/119732/11:40
lucasagomesdtantsur, ack11:40
dtantsurI hate tox >_< is it designed to rebuild the environment on every occasion?11:41
*** lucasagomes is now known as lucas-hungry11:44
*** lazy_prince has quit IRC11:49
*** nosnos has quit IRC11:52
*** nosnos has joined #openstack-ironic11:52
*** nosnos has quit IRC11:57
*** gillilunch is now known as gilliard11:59
*** andreykurilin has quit IRC12:01
*** andreykurilin has joined #openstack-ironic12:05
dtantsurlucas-hungry, what do you think, may I use 'last_error' field in Node for storing discovery error? will it break anything?12:07
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Implement hardware discovery in PXE driver  https://review.openstack.org/11003112:12
openstackgerritVladyslav Drok proposed a change to openstack/ironic: Translator functions cleanup part 3  https://review.openstack.org/11858812:15
*** anderbubble has joined #openstack-ironic12:17
*** Poornima has quit IRC12:21
*** anderbubble has quit IRC12:31
*** lucas-hungry is now known as lucasagomes12:45
lucasagomesdtantsur, hmm... it's an async operation right? I think that's fine12:46
dtantsurok, thanks12:46
lucasagomesasync I mean, after the node post the info back to the api etc12:46
lucasagomesso I think it's grand12:46
Shrewsmorning ironic12:49
dtantsurShrews, morning12:50
Shrewsdtantsur: morning!12:51
* Shrews makes coffee12:51
*** cdnchris has joined #openstack-ironic12:55
jrollmorning Shrews dtantsur lucasagomes and the rest of ironic :)13:01
lucasagomesjroll, morning13:01
lucasagomesmorning Shrews13:01
jrolllucasagomes: 13:00:49        ndipanov | fwiw I will +2 the sys_meta patch if you come up with it13:01
openstackgerritA change was merged to openstack/ironic: Remove XML from api doc samples  https://review.openstack.org/11986813:01
* jroll cries13:01
jrollso hacky13:01
jrollso horrible13:01
* jroll rewrites nova13:02
ndipanovit's really sad13:02
jrollheh, hi :)13:02
jrollndipanov: yeah, I saw comstud talking about sys_meta stuff the other day... this is new to me, but I don't like it13:03
ndipanovjroll, it's horrible - I think tho that at this point people are pissed off with it enough to actually fix it13:04
jrollwoo :)13:04
ndipanovbecause it keeps blocking useful work13:04
jrolljust... not this cycle :P13:04
ndipanovwell you need to have the paperwork right - we can't help you without that13:04
ndipanovbut as soon as you get that - we'll -1 the shit out of it for spelling errors13:05
jroll:)13:05
jrollshould rubber stamp all the "clean up this horrible thing" specs for kilo13:06
ndipanovthan if you win the jenkins lottery - you need impeccable grammar for that of course - it might land a cycle or 5 from now13:06
ndipanovbut yeah - velocity13:06
jrolland then, once you spend a few cycles on patch iterations13:07
ndipanovsoftware defined economy even13:07
jrolljust *try* to get it through the gate, I dare you13:07
ndipanov</rant>13:07
ndipanovyeah that's what I meant by jenkins lottery13:07
jrollaha13:07
*** jasondotstar has joined #openstack-ironic13:07
jrolltis sad13:07
*** Guest39270 is now known as antonym13:13
*** antonym is now known as Guest5417313:14
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Fix agent PXE template  https://review.openstack.org/11986913:14
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Use expected, actual order for PXE template test  https://review.openstack.org/11987013:15
jroll^ rebased on top of fixed conf sample, those should be good to go now13:15
*** cdnchris has quit IRC13:21
openstackgerritVladyslav Drok proposed a change to openstack/ironic: Use oslo.utils instead of ironic.openstack.common  https://review.openstack.org/11973213:22
*** cdnchris has joined #openstack-ironic13:24
jrolllucasagomes: mind looking at this? I think it's fine, just takes a minute to grok13:25
jroll(seems to work, though)13:25
jrollhttps://review.openstack.org/#/c/117529/6/ironic_python_agent/tests/extensions/base.py13:25
* jroll brb13:25
lucasagomesjroll, yup, will do. 1 sec just finishing adding one test here locally13:26
jrollyeah no rush13:26
*** rloo has joined #openstack-ironic13:32
*** athomas has joined #openstack-ironic13:33
ShrewsSo what's the deal with check-tripleo-ironic-undercloud-precise-nonha lately?13:41
*** r-daneel has joined #openstack-ironic13:43
jrollShrews: it's failing like crazy, yeah?13:44
Shrewsjroll: like cray-cray13:44
jrollgot an example?13:46
jrollI thought I had seen that, too, but now I can't find one13:46
*** anderbubble has joined #openstack-ironic13:46
Shrewshttp://logs.openstack.org/09/119809/2/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/237e64e13:46
jrollidk, I just always assume it's broken13:47
*** tatyana has joined #openstack-ironic13:48
slaglehey folks, i'm curious as to why scheduler_use_baremetal_filters is defaulted to False in ironic_host_manager.py? what's the reasoning behind that?13:48
*** cdnchris has quit IRC13:49
slaglei thought the desire was to be using the Exact*Filter(s) by default for baremetal drivers13:49
slaglelucasagomes: ^^ any thoughts?13:49
slaglei'm curious if this is something we should be setting to true for tripleo deployments13:49
jrollslagle: wouldn't defaulting that to true use those filters even if that compute host wasn't using the ironic driver?13:51
jroll(I'm not sure, just wildly throwing around ideas)13:52
* lucasagomes reads13:52
slaglejroll: yes, i assume it would. but why would you be using IronicHostManager if the compute host was not using ironic13:52
slagleso maybe the reason is the case where you have multiple compute hosts with different drivers?13:52
slaglesome ironic, and some not13:52
jrollright, it's only if it's activated, hm13:53
lucasagomeshmm not sure13:54
lucasagomesmaybe it was because the filters were only in the ironic tree13:54
lucasagomesand if that was set to True, we would need an extra step to actually copy13:54
lucasagomesthose filters to the right direct in nova before using it13:54
jrolloh, could be13:55
lucasagomesslagle, but now, with the driver merged, I think that we should set it to true by default13:55
lucasagomess/direct/directory13:55
slaglelucasagomes: good point, possibly13:56
openstackgerritVladyslav Drok proposed a change to openstack/ironic: Remove unused modules from openstack-common.conf  https://review.openstack.org/12009814:00
*** LV-426 has quit IRC14:05
rloomorning, afternoon, evening, night ironickers14:08
rloojroll: qq. Are you OK with https://review.openstack.org/#/c/118326/? You mentioned trying it out.14:09
*** pcrews has joined #openstack-ironic14:11
lucasagomesrloo, morning14:12
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: List resources with detail  https://review.openstack.org/11982514:13
rloohiya lucasagomes14:13
dtantsurrloo, jroll, morning14:14
*** scubacuda has quit IRC14:14
rloohello dtantsur14:14
*** annegentle_ has joined #openstack-ironic14:14
*** scubacuda has joined #openstack-ironic14:14
*** annegentle has quit IRC14:14
NobodyCa1good morning Ironic14:16
dtantsurNobodyCa1, morning14:17
NobodyCa1morning dtantsur rloo lucasagomes Shrews jroll14:17
NobodyCa1:)14:17
ShrewsNobodyCa1: morning14:17
Shrewsand rloo14:17
jrollrloo: I'll try it right now :/14:17
*** NobodyCa1 is now known as NobodyCam14:17
rloothx jroll14:17
jrollrloo: tempest + agent doesn't work any more, so there's some effort required :(14:17
rloomorning NobodyCam & Shrews14:17
NobodyCam:)14:18
jrollmorning NobodyCam!14:18
rloojroll: I think it should work and I'm fine if you don't try it out.14:18
lucasagomesmorning NobodyCam14:20
jrollrloo, yeah, just want to be safe14:21
jrollit should work14:21
jrollbut we STILL don't have CI on this14:21
jrollif I do nothing else today it will be fixing that14:21
jrollso close14:21
* jroll grumbles at tempest14:21
rloothx for persisting jroll!14:21
openstackgerritDan Prince proposed a change to openstack/ironic: Ironic: don't canonicalize extra_specs data  https://review.openstack.org/12010714:26
NobodyCamis the White board correct?Gate Status:  golden14:28
*** jistr has quit IRC14:28
jrollyes14:28
jroll(afaik)14:29
NobodyCamnice14:29
NobodyCam:)14:29
jrolldriver is still broken in nova, but we haven't flipped over to it yet14:29
NobodyCamahh14:30
rloojroll: do you know what the status is, with the broken driver in nova. Is there anything we can do to help?14:31
openstackgerritA change was merged to openstack/ironic: Minor fixes for ipminative console support  https://review.openstack.org/11947514:34
jrollrloo: https://review.openstack.org/#/c/119944/414:34
jrollrloo: they're discussing it in the nova channel right now, I think it's going to land shortly14:35
rloojroll: good.14:36
rloojroll: umm, do we have the same thing (or a similar patch) as 119944, in our tree?14:38
jrollrloo: I had a patch that switched it to get_flavor(), I abandoned thay14:39
jrollthat14:39
jrollso this code is already in our tree14:39
jrollthis is the change in nova that didn't get merged back to our tree14:39
*** rakesh_hs has quit IRC14:40
rloojroll: yeah, I know. Is this code exactly what is in our tree? I see comments and revisions. I thought the first revision was in our tree. I'll look.14:40
jrollI have no idea14:40
* rloo doesn't want to go through the same exercise again if the proposed code doesn't work14:41
rloojroll: it is different :-(14:42
jrollbut how much different?14:43
jrollhave a diff?14:43
rloono, i don't have a diff. just eyeballing.14:43
rloountil we have ironic tests in nova, should we continue to mimic all changes in our tree?14:43
* jroll lands https://review.openstack.org/#/c/118326/14:43
rloothx jroll ^^14:43
jrollrloo: as soon as this is fixed, we will switch devstack to use the driver in nova14:44
NobodyCamjroll: you still want 11986914:44
jrollrloo: we have tests in nova, just not voting14:44
rloojroll: if we have tests in nova, how did we miss the original failure?14:44
jrollNobodyCam: yes please14:45
jrollrloo: because they are using the driver from ironic tree14:45
rloojroll: right. so this new patch isn't being tested anywhere except our unit tests.14:46
jrollrloo: this is the only change we'll be landing to the virt driver until the tests use the driver in the nova tree14:46
jrollah, right14:46
jrollrloo: how much different is our tree14:46
rloojroll: there are some changes wrt passing the context I think.14:46
rloohttps://review.openstack.org/#/c/119944/4/nova/virt/ironic/driver.py14:46
jrollah yeah14:46
rlooit *should* be fine, but I think we should follow process?14:47
jrollrloo: do you want to backport it or shall I?14:47
rlooyou?14:47
jrollmmmmmkay14:47
jrollgive me a moment14:47
rloothx jroll!14:47
rloo(i could do it if you want, just being lazy)14:47
jrollnah, I've got it14:48
rlooreally, we just need a patch up to make sure jenkins is happy.14:48
*** mitz has joined #openstack-ironic14:49
openstackgerritA change was merged to openstack/ironic: Updated from global requirements  https://review.openstack.org/11980914:49
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Backport Nova review 119944  https://review.openstack.org/12011314:50
jrollright14:50
jroll^^ there it is14:50
*** jistr has joined #openstack-ironic14:52
jrollrloo: let me know if I missed anything14:53
NobodyCamdoesn't necessarily need to land <- not the most convincing commit message14:53
NobodyCam:-p14:53
jrollNobodyCam: it's just going to get deleted :P14:53
NobodyCami know14:53
NobodyCam:-p14:53
rloothx jroll. Looks good.14:56
jrollgreat :)14:56
* jroll waits for unrelated test failures14:56
*** jistr has quit IRC14:56
*** jistr has joined #openstack-ironic14:57
openstackgerritA change was merged to openstack/ironic: Use oslo.utils instead of ironic.openstack.common  https://review.openstack.org/11973214:57
openstackgerritChris Krelle proposed a change to openstack/ironic: Fix compute_driver and scheduler_host_manager in install-guide  https://review.openstack.org/11993714:58
NobodyCambrb. have to run to drop off the car14:58
*** cdnchris has joined #openstack-ironic15:06
*** vdrok is now known as vdrok_afk15:08
*** bluex-pl has quit IRC15:12
*** viktors|afk is now known as viktors15:15
NobodyCamand back15:16
lucasagomesNobodyCam, added a comment to ^15:17
openstackgerritRuby Loo proposed a change to openstack/python-ironicclient: Add 'bash-completion' to 'ironic help' response  https://review.openstack.org/12012215:18
* NobodyCam looks15:18
lucasagomesNobodyCam, I'm not sure what we are doing with the ClusteredCompute thing15:19
jrollnova doesn't like it15:20
jrollso we're keeping it in our tree until we have a better solution15:20
lucasagomesjroll, right15:23
NobodyCamack15:24
openstackgerritChris Krelle proposed a change to openstack/ironic: Fix compute_driver and scheduler_host_manager in install-guide  https://review.openstack.org/11993715:26
lucasagomesit's not decided which day will be the design sessions for Ironic right?15:27
jrollI don't think so?15:27
NobodyCamas far as I know ... wed, thrus, fri are the days just not sur which we'll be sloted for15:27
*** mikedillion has joined #openstack-ironic15:27
lucasagomesI think tue as well15:28
lucasagomesafaik15:28
lucasagomesyeah just wanted to know because I'm planning to get some days off after the summit15:28
*** tatyana has quit IRC15:29
NobodyCamnice15:30
*** andreykurilin has quit IRC15:32
*** sabah has joined #openstack-ironic15:32
*** tatyana has joined #openstack-ironic15:33
*** dhellmann is now known as dhellmann_15:38
*** dtantsur is now known as dtantsur|afk15:38
openstackgerritA change was merged to openstack/ironic: Remove unnecessary mapping from Agent drivers  https://review.openstack.org/11832615:44
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: EXPERIMENTAL Implement hardware discovery in PXE driver  https://review.openstack.org/11003115:45
devanandamorning, all15:45
Shrewsmorning devananda15:45
NobodyCammornign devananda15:46
jrollmorning devananda15:46
lucasagomesdevananda, morning15:46
lucasagomesdo we have a tag for ironic in nova bugs already?15:46
*** tatyana has quit IRC15:47
*** cdnchris has quit IRC15:48
devanandalucasagomes: not yet, following up on that again today15:49
lucasagomesdevananda, thanks! right I will just put "ironic" in the subject15:50
lucasagomessummary**15:50
openstackgerritA change was merged to openstack/ironic: Fix agent PXE template  https://review.openstack.org/11986915:53
openstackgerritA change was merged to openstack/ironic: Use expected, actual order for PXE template test  https://review.openstack.org/11987015:53
NobodyCamare we still keeping our tree's driver in sync with nova's? question comes from: https://review.openstack.org/#/c/12010715:56
rlooNobodyCam: no, I don't think so. Once we fix that flavors bug in nova, and switch the tests to use the driver in nova, we'll be deleting our tree's driver.15:57
rlooNobodyCam: so all changes should only go to nova15:58
*** bandicot has joined #openstack-ironic15:58
*** bandicot is now known as bensonmiller15:58
jrollNobodyCam: we should make sure tripleo flips to using the driver in nova before we delete it15:59
jroll(and that patch should get proposed in nova015:59
jrolls/0/)/15:59
*** eghobo has joined #openstack-ironic16:00
*** ifarkas has quit IRC16:00
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Log ERROR power state in node_power_action()  https://review.openstack.org/11749916:01
devanandaNobodyCam: -2'd that16:02
devanandathe canonical location for the nova/virt/ironic driver is now in the nova tree16:02
devanandawe're two patches away from making the switch16:02
devanandahttps://review.openstack.org/#/c/119944/ has been approved16:04
openstackgerritA change was merged to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/11854816:04
devanandaonce that lands, we need to recheck https://review.openstack.org/#/c/119844/16:04
devanandaonce that passes the virtual-ironic-nv job, we should get it landed and then land https://review.openstack.org/#/c/119892/ to delete the driver from our tree16:06
lucasagomesnice! yeah there's also https://review.openstack.org/#/c/120099/16:06
lucasagomesthat unblocks tripleO I think ^16:06
lucasagomes(more arch-related stuff)16:07
Shrewsyuriyz: odd that 117499 wasn't merged after lucasagomes re-approved...i wonder what happened there16:11
lucasagomesShrews, dependency16:12
lucasagomesit has a dependency on a patch not merged yet16:12
Shrewsah! duh16:12
Shrewsthx16:12
devanandaso to clarify, we should also not delete the driver from our tree until TripleO's testing has moved away from it16:12
* devananda adds all this to the whiteboard16:12
openstackgerritA change was merged to openstack/ironic: Translator functions cleanup part 2  https://review.openstack.org/11858716:13
rloomaybe we should -2 or WIP the patch that deletes the driver...16:13
*** bensonmiller is now known as bandicot16:14
Shrewseh, let's just approve it and see what happens16:15
devanandarloo: done16:15
rlooShrews is a risk taker or a ...16:15
rloothx devananda16:15
devanandait in fact couldn't land yet -- it's failing our own gate tests16:16
Shrewsrloo: toublemaker?  :)  yeah16:16
devananda(those will pass once the devstack change lands)16:16
rloodevananda: right, but once it passes, someone may accidentally approve it before tripleO folks have switched.16:16
*** sabah has quit IRC16:16
rloolike Shrews16:17
Shrewsbwah ha ha ha16:17
lucasagomes:P16:18
lucasagomesNo package 'libvirt' found -- why nova ? :(16:19
* lucasagomes manually installs it16:19
NobodyCamno libvirt? thats odd16:23
jrollit's in test-requirements, I think16:24
jrollI -1'd https://review.openstack.org/#/c/120099/16:25
jrollin hopes they don't land it before that devstack change16:25
lucasagomesNobodyCam, libvirt-devel (tho the message says libvirt only)16:25
NobodyCamahh16:26
*** derekh has quit IRC16:27
rloooh, c'mon, jroll, that code works! good idea though ;)16:27
jroll:)16:27
lucasagomesjroll, https://review.openstack.org/#/c/120107/16:28
devanandaso folks, dprince doesn't really want us to break tripleo, and has proposed a slightly different plan16:32
devanandainstead of deleting ironic/nova/* -- we replace it with a shim that subclasses the things from the nova tree16:32
JayFThat prints loud screaming deprecation warnings and gets removed in K?16:32
devanandayes16:32
devanandawe still get a single copy of the driver (in Nova)16:32
NobodyCamdevananda: would that just for the k cycle?16:32
devanandabut it means folks don't need to immediately change their configs after this patch lands16:33
rloojust for J cycle I think?16:33
jrolllucasagomes: still want CI to run with the actual patch16:33
jrolldevananda: sounds reasonable16:34
lucasagomesdevananda, yeah sounds good to me too16:34
lucasagomesjroll, heh ok16:34
jrollcan't hurt :P16:34
jrolland if tripleo isn't using that driver, no rush :P16:35
rloowith that change, folks won't need to immediately change their configs, but... if we have this in final juno, will we have more folks using/configuring to use the ironic version rather than the nova version?16:35
jrollidk, our docs will say "use the nova versino"16:36
rloojroll: who reads the docs?16:36
jrollso it's only existing users16:36
jrolldunno, I usually just come in here "how the heck do I deploy ironic"16:36
jrollor like... read the source.16:36
rlooI'd prefer that it was gone when juno release is stamped.16:37
jrolls/when juno release is stamped/now/16:37
JayFrloo: I don't disagree, but I also think if we have a loud warning it's not worth the fight with nova if they want it to stay through k16:37
rloodon't want folks to think that the deprecation will then be when kilo is released or after that.16:37
rloonova didn't ask for it, tripleo did16:38
jrollJayF: s/nova/tripleo16:38
rlooso does tripleo want it in K?16:38
rlooOpenStack has this 'thing' about users. Ironic WILL graduate in juno ;) Just worried that having the driver in ironic may mean we have to support or something-I-haven't-thought-of in kilo+.16:40
jrollwe aren't keeping the driver in ironic16:40
jrolljust making it accessible via ironic.nova.virt.ironic.driver or whatever16:41
rloojroll: I know, but we're making the driver available FROM ironic16:41
jrollit's just going to subclass everything in nova16:41
jrollright, ok16:41
*** pradipta_away has quit IRC16:41
rlooeg, if the nova-ironic-driver gets new code/new classes, will we have to update our tree to link to the new classes?16:42
jrollI guess?16:42
jrolldunno16:42
rlooso... who's going to notice/add the links or whatever, or do we wait for someone to complain16:42
rloonot that it isn't do-able. I just wonder whether it is necessary to add extra 'work' for us etc.16:43
jrollyeah, I see your point16:43
JayFespecially after the number of hoops many people working on Ironic have jumped through w/r/t that driver for the last months :)16:44
rlooI mean, if tripleO or others really need this in K, that's another story. But if no one needs it in K, why have it.16:44
*** pradipta_away has joined #openstack-ironic16:44
JayFI honestly think it's a config management problem16:45
jrolldeprecation. yay.16:45
jrollit would be cool to not need to change my configs next time I deploy :)16:45
JayFif you're able to continuously deploy Openstack to the level that a breakage *inside juno* would cause you problems16:45
devanandalucasagomes: dtantsur|afk: jogo's asked me to create a new "ironic" bug tag in Nova, so that's what we'll use16:45
*** dprince has joined #openstack-ironic16:45
* devananda edits all the things16:45
JayFyou should also have orcestration/config management good enough to update the configurations16:45
lucasagomesdevananda, awesome, ok I will tag the bug I opened once it's created16:45
* jroll bbl16:45
JayFJust sometimes too easy for pushback hard problems upstream when they could be more expediantly handled by the operator16:46
*** andreykurilin has joined #openstack-ironic16:49
*** rushiagr is now known as rushiagr_away16:49
*** igordcard has joined #openstack-ironic16:52
devanandarloo: aiui, the point is to give users (ie, tripleo) a deprecation period16:53
devanandarloo: rather than pull the rug out and break their config now16:53
rloodevananda: do you know how much time they want?16:53
slagledevananda: hi, i was wondering if there was a reason why scheduler_use_baremetal_filters is defaulted to False in ironic_host_manager.py?16:54
devanandarloo: nope. I'd guess "until the end of Juno"16:54
rloodevananda: I'm fine if it is until (but not in) the Juno release. It worries me if we release with this temporary thing.16:55
devanandarloo: I suspect it will be "deprecated in the juno release" which means we can remove it once K opens16:56
*** viktors is now known as viktors|afk16:57
NobodyCambrb16:57
rloodevananda: hmm, if that works for people. No one should, but what if someone uses J release configured with our tree 'driver', then upgrades to K release and it breaks. (We'll need to remember to document K release about the deprecation.)16:59
*** bandicot has quit IRC17:02
*** bandicot has joined #openstack-ironic17:06
lucasagomesalright folks I'll call it  aday17:06
lucasagomesa day*17:06
lucasagomeshave a good night everyone!17:07
NobodyCamhave a good night lucasagomes17:07
JayFgood night17:07
*** lucasagomes is now known as lucas-dinner17:07
*** davidlenwell_ is now known as davidlenwell17:10
*** coolsvap has quit IRC17:10
*** bandicot has quit IRC17:10
*** cdnchris has joined #openstack-ironic17:11
*** dhellmann_ is now known as dhellmann17:13
*** rushiagr_away is now known as rushiagr17:14
*** coolsvap has joined #openstack-ironic17:15
*** romcheg has quit IRC17:15
*** rushiagr is now known as rushiagr_away17:20
*** rushiagr_away is now known as rushiagr17:21
*** rushiagr is now known as rushiagr_away17:23
*** cdnchris has quit IRC17:25
*** harlowja_away is now known as harlowja17:25
*** eghobo has quit IRC17:30
devanandalucas-dinner, dtantsur|afk: "ironic" tag has been created in Nova17:32
*** jcoufal has quit IRC17:34
*** jistr has quit IRC17:35
*** dtantsur|afk is now known as dtantsur17:35
dtantsurdevananda, cool, thanks17:35
*** eghobo has joined #openstack-ironic17:37
*** athomas has quit IRC17:39
*** andreykurilin has quit IRC17:40
*** pelix has quit IRC17:43
dtantsurok I'm disappearing again :)17:49
*** dtantsur is now known as dtantsur|afk17:50
NobodyCamnice17:50
*** todd_dsm has joined #openstack-ironic17:52
*** linggao has joined #openstack-ironic17:58
openstackgerritDan Prince proposed a change to openstack/ironic: Deprecate Ironic compute driver and sched manager  https://review.openstack.org/11989218:18
dprincedevananda: ^^18:18
jrolldprince: nice18:19
dprincejroll: thanks :)18:20
jrollI'll leave this on the review, but I think we should e.g. s/Ironic scheduler_host_manager is deprecated.../ironic.nova.scheduler.ironic_host_manager.IronicHostManager is deprecated.../18:20
*** pcrews has quit IRC18:24
*** pcrews has joined #openstack-ironic18:25
*** eghobo has quit IRC18:29
*** eghobo has joined #openstack-ironic18:29
*** ndipanov is now known as ndipanov_gone18:31
*** dprince has quit IRC18:31
jrolldpri<tab> :(18:31
* jroll lunches18:31
*** jistr has joined #openstack-ironic18:33
*** jistr has quit IRC18:34
*** jistr has joined #openstack-ironic18:34
*** rloo has quit IRC18:37
*** jogo has joined #openstack-ironic18:45
jogohave a quick question about the grenade patch: "Add Ironic sideways migration testing18:46
jogo"18:46
jogohttps://review.openstack.org/#/c/111859/18:46
NobodyCamI can attempt to answer18:47
NobodyCam:-p18:47
jogoNobodyCam: the subsequent patch: https://review.openstack.org/#/c/118515/18:47
jogoisn't passing18:47
*** todd_dsm has quit IRC18:47
jogoso does that mean there is an issue in 111859 or something else?18:48
jogoit also takes an hour and a half18:48
jogowich is very slow18:48
NobodyCamhumm18:48
* NobodyCam looks in to the logs to see if he can spot where the error is comming from18:48
NobodyCamThe following services are not running after upgrade:  nova-compute18:49
NobodyCamhttp://logs.openstack.org/15/118515/15/check/check-grenade-dsvm/36662f9/logs/grenade.sh.txt.gz#_2014-09-06_00_42_40_56718:50
jogoNobodyCam: so that is expected to fail18:50
jogobut http://logs.openstack.org/15/118515/15/experimental/check-grenade-dsvm-ironic-sideways/d581408 isn't18:50
Shrewshrm. that started going sideways (pun intended) at patch set 8, which set VIRT_DRIVER=ironic in devstack. this may be dependent on devananda's devstack fix18:57
Shrewsjust a guess though18:57
NobodyCamhumm: http://logs.openstack.org/15/118515/15/experimental/check-grenade-dsvm-ironic-sideways/d581408/logs/new/screen-ir-cond.txt.gz#_2014-09-06_20_25_59_92418:58
Shrewsthat's odd19:00
NobodyCamjust a side note: Tc meeting in an hour19:01
NobodyCam:-p19:01
devanandajogo: that it takes 90 minutes is not unexpected19:02
devanandajogo: aiui, this is because it's doing a full tempest parallel run using the nova ironic driver, which is, in the gate's nested virt environment, MUCH slower than libvirt19:02
*** openstackgerrit has quit IRC19:02
devanandajogo: but i believe it passed yesterday, so this failure may be a result of something else? I'm not sure ... adam_g may know19:03
jogodevananda: full tempest?19:03
jogogrenade ususually just does a smoke run19:03
adam_gjogo, commenting in the review19:03
adam_gigonore that dependent patch, basically19:03
devanandajogo: er, right. i mean as opposed to the virtual-ironic[-nv] job that skips all the nova api tests, this one runs them w/ the ironic driver19:04
jogoadam_g: once you add your comment in gerrit let me know and happy to +W19:05
devanandajogo: https://review.openstack.org/#/q/topic:ironic_infra,n,z has some of those changes19:05
adam_gjogo, posted19:06
jogoadam_g: thanks and +Wed19:07
adam_gjogo, great, thanks19:07
NobodyCambb in time for tc meeting19:26
aweeksdevananda: before the TC meeting today, I got a mostly-mocked-out version of a horizon dashboard up https://review.openstack.org/#/c/115050/19:36
aweeksdevananda: its really not in great shape right now, but I'm currently working on integrating it with the API stuff that JoshNang added.  currently, its my top priority--wish it were more done for the meeting but its what I've got19:37
*** jogo has left #openstack-ironic19:38
NobodyCamaweeks: :) Thank you :)19:40
*** openstackgerrit has joined #openstack-ironic19:42
devanandaaweeks: awesome, ty19:43
aweeksdevananda: np :)19:44
NobodyCamdevananda: any last minute items for tc meeting?]19:44
devanandaNobodyCam: TC will spend some time discussing marconi/zaqar first19:44
devanandaNobodyCam: there will probably be some discussion of ironic, and a follow up discussion next tuesday as well19:44
devanandathis gives TC members time to raise questions, investigate for a week, and come back19:45
NobodyCam:) ahh so we're prob not going to get the "word" today19:45
devanandadunno. maybe.19:46
* NobodyCam crosses fingers19:46
*** BadCub has quit IRC19:49
*** BadCub has joined #openstack-ironic19:49
*** rloo has joined #openstack-ironic19:54
*** lucas-dinner is now known as lucasagomes19:59
*** killer_prince has joined #openstack-ironic20:06
*** killer_prince is now known as lazy_prince20:06
aweeksgood luck!20:06
openstackgerritSyed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add uefi boot mode support in IloVirtualMediaIscsiDeploy  https://review.openstack.org/11656120:09
*** jistr has quit IRC20:14
openstackgerritRuby Loo proposed a change to openstack/ironic: Backport Nova review 119944  https://review.openstack.org/12011320:23
rloojroll: ^^20:23
*** mikedillion has quit IRC20:25
jrollrloo: I wasn't worried about unit tests, just tempest :P thanks for doing that though! :)20:25
rloojroll: I need more caffeine. I was thinking that the unit tests failed, so it wouldn't pass tempest. (Although maybe we should merge this anyway, in case someone does a diff)20:27
jrollrloo: no worries :)20:27
openstackgerritSyed Ismail Faizan Barmawer proposed a change to openstack/ironic: Add uefi boot mode support in IloVirtualMediaIscsiDeploy  https://review.openstack.org/11656120:40
JayFWho is supposed to answer questions on ask.openstack.org?20:51
JayFLike seriously, who is supposed to20:52
NobodyCami just signed up yesteday :-p20:52
NobodyCamI have yet to go through and see whats there to answer20:52
JayFLike I'll gladly help people who drop in channel or somehow filter to -dev20:52
JayFbut I don't think it should be a general contributor's job to be frontline support for Ironic20:52
jrollJayF: everybody!20:52
JayFespecially when most of us work for companies that make serious bank selling that expertise in one form or another20:52
jrollgo team20:53
* rloo knows nothing about ask.openstack.org and wants to stay that way20:53
adam_ggah that was fun. need to run for ~1hr.20:53
jrollrloo: it's stack overflow for openstack :P20:53
rloojroll: I don't want to know, I don't see anything or hear anything ... ;)20:53
jroll:P20:53
JayFI mean I'm kinda 100% serious in my questino20:54
JayFif it's required to go to ask.o.o and answer questions to be integrated20:54
NobodyCamok so I'll put up the api proxy code for nova20:54
JayFwho is expected to pay people to answer questions free online?20:54
JayFI don't get it personally20:54
lucasagomesalright so document the migration and deprecation/proxy thing :)20:54
jrollI don't think anyone has an answer for that, JayF20:54
*** Nisha has joined #openstack-ironic20:55
JayFI think the answer is that someone got an expectation on a document somewhere20:55
NobodyCamJayF: looks like it just up to us to take care of it20:55
JayFwithout providing resources for fulfilling said expectation20:55
NishaHi Ruby20:55
Nisharloo: hi, there20:56
jrollJayF: I expect as we get users, they will tend to help each other, there is already some of that going on there20:56
*** jasondotstar has quit IRC20:56
rloohi Nisha20:56
devanandasounds like we need to pony up some upgrade docs ASAP20:56
devanandaand then, depending on what Nova decides, we might also need to dogpile on making the API proxy acceptable for them to merge20:57
devanandabut that's basically it20:57
jrollyep20:57
jrollgood work so far, everybody :)20:57
lucasagomesnice20:58
rlooyay! what is 'ASAP'?20:58
lucasagomesrloo, 1 week20:58
lucasagomesNobodyCam, add me to the review list after you propose the proxy to nova20:58
NobodyCamlucasagomes: ack20:58
russell_hJayF: the same people who pay developers to write software and put it online for free?20:59
rloolucasagomes: 1 week is good/reasonable :-)20:59
lucasagomesyup20:59
russell_hwe should make a rule that Ironic will only support "integrated" Linux distros20:59
russell_hwhere an "integrated" Linux distro is one that has free support21:00
lucasagomesaight, now it's quite late here :) time to go21:01
lucasagomesg'night everybody21:01
jrollrussell_h: :P21:01
NobodyCamthank you lucasagomes ... have a good night21:01
*** lucasagomes is now known as lucas-zZz21:02
*** romcheg has joined #openstack-ironic21:07
*** romcheg1 has joined #openstack-ironic21:09
*** romcheg has quit IRC21:11
devanandalucas-zZz: fyi, "final revs of clients by Sep 18 (same as olso)" is being proposed cross projects21:13
JayFIs anyone aware of a 'superironic' client or effort towards making one? (i.e. like supernova/superglance for ironic)21:16
NobodyCamrunning to pick up the car... brb21:16
devanandaJayF: i'm not even sure what that means21:18
JayFOh really? Here I'll link it21:18
devanandaJayF: there is a unified openstack client effort but taht doesn't sound like what you're referenceing21:18
jrolldevananda: see http://major.io/2012/06/05/supernova-manage-multiple-openstack-nova-environments-with-ease/21:18
jrollalso see https://github.com/major/supernova21:18
jroll(tldr multi-environment nova client)21:19
devanandaah, gotcha21:19
devanandajroll: not that i know of21:19
JayFI want USE_KEYRING support for ironicclient21:19
JayFand superironic21:19
JayFI might work on one of those next hackdaty21:20
*** linggao has quit IRC21:20
*** lucas-zZz has quit IRC21:24
* NobodyCam is back21:26
NobodyCamcd nova21:27
NobodyCamdoh21:27
mrdaMorning Ironic!21:29
NobodyCamgood morning mrda :)21:29
mrdayay, 119944 is in gate21:31
*** romcheg1 has quit IRC21:32
mrdawow, last night gate for 5 jobs long, and check was 20 jobs.  It's a little different 10 hours later :(21:33
devanandaheh21:33
rloomrda: fyi: https://review.openstack.org/#/c/120113/21:35
mrdathanks rloo21:35
devanandamrda: dprince posted another rev on https://review.openstack.org/#/c/119892/4 which appears to be failing things now21:36
devanandamrda: but the upside is we should be able to land it w/o waiting on the devstack change21:36
mrdawhy can't people leave things alone until they are working? :)21:36
rloodevananda: shouldn't we wait for devstack change first before landing 119892?21:37
rloodevananda: to make sure that the driver in nova works?21:37
devanandarloo: nope. this actually will test the driver in nova21:37
devanandarloo: it replaces the driver in our tree with a subclass of the one in nova's tree21:37
rloodevananda: oh yeah. you're right.21:37
devanandawe do need the fix in nova to land first, tho21:37
rlooright, but this will fail w/o that fix.21:38
devanandahttps://review.openstack.org/#/c/119944/21:38
devanandaright21:38
rloodevstack tests will ignore the warnings in the logs, I hope ;)21:38
mrdalooks like everyone's been busy while I've been afk21:39
rloomrda: aren't we always? :-)21:40
mrdasome days yes, other days it's easier to catch up :)21:41
mrdaoooh, Thierry just marked 119944 as critical.21:42
mrda\o/21:42
*** romcheg has joined #openstack-ironic21:42
openstackgerritSyed Ismail Faizan Barmawer proposed a change to openstack/ironic: Support for setting boot mode in pxe_ilo driver  https://review.openstack.org/11857821:44
*** romcheg has quit IRC21:50
*** todd_dsm has joined #openstack-ironic21:56
NishaHello Core reviewers: Request review for the patches https://review.openstack.org/#/c/116561 and https://review.openstack.org/#/c/11857822:00
jrollNisha: what's the use case for PXEAndIloDriver, when we already have IloVirtualMediaIscsiDriver22:03
jrollNisha: the only difference is PXE vs virtualmedia, and I don't see the point in using PXE when virtualmedia is available22:04
JayFjroll: I think that's not a good reason to disallow that driver22:04
JayFjroll: the better reason might be that it is a feature, without a matching spec/bp, post FF22:04
jrollI'm not disallowing22:05
jrollI'm just curious22:05
jrollJayF: the feature here is an ilo/pxe driver (that already existed) that now has uefi (which we have ffe'd)22:05
JayFI'm asking generally why the use case has to be made, honestly22:05
JayFhmm?22:05
jrollso I don't think that's an issue22:05
jrollI'm just curious what is the use case, not trying to block22:06
jrollthey didn't add the driver22:06
jrollthey just changed it to support uefi, aiui22:06
JayFah gotcha22:06
jrollimbw22:06
JayFsorry as you know trying to pay attention to two things at once :/22:06
jrollbut that's how I understood it22:06
jrolleven though the commit message says it adds a driver :P22:06
*** Nisha has quit IRC22:07
NobodyCamhumm seems I'm running in to the same error lucas did this morning: No package 'libvirt' found22:09
devanandaNobodyCam: install libvirt-dev22:20
devanandamrda: I've got a fix for the error on 119892's python tests22:22
devanandamrda: do you want to address the reviewer feedback, or shall I ?22:22
NobodyCamdevananda: on my mac for a tox run?22:23
devanandaNobodyCam: for nova? yes22:23
devanandaNobodyCam: they added python-libvirt as a requirement, which in turn requires libvirt header files22:24
devananda(yay)22:24
NobodyCamlol happy happy joy joy, /me need to check if brew will install dev versions22:26
devanandaNobodyCam: not dev version -- development library for header files. I dont know what it's called on mac22:29
jrollNobodyCam: just use a vm :/22:31
NobodyCamI may have to22:32
jroll(I hate to say that)22:32
jrollthough 'brew install libvirt' might install headers on os x22:32
jrollI think it does22:32
jrollas brew always compiles from source22:33
NobodyCamwill tell ya in a second22:33
jroll:P22:33
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Deprecate Ironic compute driver and sched manager  https://review.openstack.org/11989222:36
mrdadevananda: sorry, was grabbing coffee22:36
mrdabut back now22:36
devanandanp22:36
devanandamrda: that ought to pass, I hope22:36
* mrda looking atthe diffs22:37
devanandait's passing tox for me locally now, at least (it wasn't before)22:37
mrdait'll take all day to get through zuul, even once it is approved22:39
mrdadevananda: you'd like me to babysit 119892 and get it through the gate if possible?22:40
mrda(today)22:40
adam_gdevananda, NobodyCam, all: want to chat about migration docs sometime this afternoon? im going to be out tomorrow and thurs and would like to get a WIP on a pad going today22:41
jrolldevananda: +2'd that, have a quick thing inline but happy either way with it22:41
NobodyCamwishes the migration scripts would auto doc them selfs :-p22:42
adam_gNobodyCam, well, theres only one and its not doing much. there shouldn't be much to document other than the expectations of the migration, which is what im wondering/worried about. that is, making it clear we're migrating nodes and not instances22:44
devanandamrda: that'd be great, thanks. This should actually fail the virtual-ironic-nv job until 119944 lands in Nova, so that needs babysitting too22:44
devanandaadam_g: so, yes, let's start a pad22:45
devanandaadam_g: Nova expects us to migrate instances, even though we're not *testing* that22:45
mrdaso the focus for today will be to get 119944 and 119892 to land if possible.22:45
devanandamrda: if those two land, I will be very very happy22:45
mrdadevananda: me too :)22:46
adam_gdevananda, /me wonders is that even possible? i dont recall migrating instance state from one virt driver to another ever being supported22:46
mrdaAnd then I can go on 4 weeks vacation tomorrow with a clear conscience :)22:46
*** pradipta_away has quit IRC22:46
devanandaadam_g: yea, it's not supported by any other virt driver, but everyone seems to think we're special ;)22:47
devanandamrda: as well you should!22:47
NobodyCammrda: *cough* FOUR weeks... wow... enjoy22:47
devanandamrda: that's basically "see you in paris" eh?22:47
mrdawell, back for 3(?) weeks, then Paris22:48
devanandaadam_g: if you have the env set up, I'd be super curious if it works (even if its unsupported)22:48
mrdathis was planned before i started at Rackspace, so it's great to finally get here :)22:48
adam_gdevananda, i dont have an env up. im trying to think the best way to get a functional nova-bm  up. probably tripleo22:49
NobodyCamjroll: brew installing libvirt did not get rid of my No package 'libvirt' found error22:50
JayFmrda: where are you headed?22:50
NobodyCam:(22:50
*** pradipta_away has joined #openstack-ironic22:50
NobodyCambrb22:51
devanandaadam_g: probably22:51
devanandaadam_g: anyhow, let's start documenting what we know22:51
adam_ghttps://etherpad.openstack.org/p/IronicMIgration22:52
adam_ger22:52
adam_ghttps://etherpad.openstack.org/p/IronicMigration22:52
*** igordcard has quit IRC22:55
mrdaJayF: ORD, NYC, MCO, HNL :)22:56
JayFmrda: I hope you enjoy HI :) I know I did23:03
mrda\o/23:03
JayFI don't know MCO though?23:03
JayFmrda: protip: Buy wardrobe of amazingly comfy HI shirts for dirt cheap  while there23:04
JayFHawaiian shirts: Pro: Super comfy Con: Becoming the guy who wears Hawaiian shirts23:04
* NobodyCam has a closet full of HI shirts23:05
mrdaMCO == Orlando23:05
JayFAha. Visiting any oversized mice?23:06
mrdaI have 3 kids JayF, so the answer is of course!23:06
NobodyCamsooo much walking required at that mouse house. /me likes the west cost house23:07
JayFNiice. I remember my two childhood visits to Disney World very well :D23:07
mikalJayF: the problem is the dirt cheap Hawaiian shirt are poor quality and fall apart23:07
mikalTommy Bahama all the way23:07
* mrda is planning 4x Disney Parks + 2x Universal Parks23:07
* mrda will have sore feet23:07
NobodyCam:-p23:07
JayFI have two of them and am wearing them very spaced out as to avoid the "guy who wears hawaiian shirts" situation :)23:07
mikalWhen do you stop your holiday to write an API proxy?23:07
mikalI get made fun of for my perfectly sensible cow boy shirt23:08
jrollNobodyCam: :(23:08
mrdamikal: Fortunately that is in hand with other ironic devs23:08
mikalIt makes me sad23:08
* mikal secretly hopes we kill it with fire23:10
mikalBut I was also hoping to ruin your vacation23:10
mrdamikal: my manager would not approve of that!23:11
mikalYour manager learned to manage from watchin the simpsons, so you're in trouble here23:11
mrdaA lot of things make sense now23:12
mikalAlso, please go to the hammock district and collect a hammock for me23:12
jrolllol23:12
mikalThat Hank Scorpio episode is full of useful advice23:12
devanandaadam_g: we should probably make this more prose like at some point23:13
NobodyCamieek: http://pavlok.com shocks you if you don't go to the gym, posts on your facebook account and can take money from your bank account..23:13
devanandaanyone else want to take a poke at upgrade docs? https://etherpad.openstack.org/p/IronicMigration23:13
devanandamikal: wait, you know of a 1k node baremetal installation?23:15
jrollo.o23:15
jrolldevananda: this is where we tell you that onmetal is built on nova-bm23:15
devanandahahaha23:16
mrdaO.o23:16
mikaldevananda: yes, I do23:16
devanandao.023:16
mikalThey are brave, let's leave it there23:16
adam_gdevananda, definitely. just notes atm. where should this be proposed in final form, btw? ironic docs?23:16
mikalAnd not a public cloud23:16
jrollfor my curiousity, is this within rackspace?23:17
mikaljroll: nup23:17
mrdaphew23:17
jroll^23:17
mikalSorry, I don't think they authorized me to mention their name is all23:17
mikalThey're a "web company"23:17
mikalIts used hsoting their own stuff internally23:17
jrollyeah23:18
jrollinteresting23:18
devanandamikal: so it's reasonable to expect that they won't respond publicly to your email then?23:18
devanandathat would be unfortunate23:18
mikaldevananda: they've actually said they will repave with ironic23:18
devanandaah, cool23:18
mrda\o/23:18
mikaldevananda: my point there was more that if one group did it, others might ahve too23:18
devanandagotcha23:18
mikalThey have zero interest in a migration for what its worth23:19
mikalThey're big enough to just rebuild it three racks to the left23:19
jrollben makes a good point here23:21
jrollif they don't update their tooling to use ironic, their baremetal service will be read-only23:21
mikalThis is true23:21
mikalBut perhaps the thing they have is a management dashboard23:21
jrollwhich is mostly useless, imo23:21
jrollright23:21
mikalOr something like that they don't want to re-write23:22
jrollyeah23:22
mikalIts a weak argument though23:22
jrollrewrite is exaggerating, I think, but I see your point23:22
mrdajroll: thanks for the +2 on 11989223:34
openstackgerritgreghaynes proposed a change to openstack/ironic: Add conductor_hostname property to nodes  https://review.openstack.org/10968823:56

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