Monday, 2014-08-11

« Sunday, 2014-08-10 Index Tuesday, 2014-08-12 »
*** bigjools has quit IRC00:33
*** bigjools has joined #openstack-ironic00:44
*** bigjools has joined #openstack-ironic00:44
*** bigjools has quit IRC00:53
*** bigjools has joined #openstack-ironic00:55
*** bigjools has joined #openstack-ironic00:55
*** pcrews has joined #openstack-ironic01:35
*** nosnos has joined #openstack-ironic01:40
*** pcrews has quit IRC01:51
*** lazy_prince has quit IRC02:28
*** killer_prince has joined #openstack-ironic02:37
*** killer_prince is now known as lazy_prince02:38
*** ramineni has joined #openstack-ironic03:16
*** nosnos has quit IRC03:22
*** Poornima has joined #openstack-ironic03:35
*** rameshg87 has joined #openstack-ironic03:38
*** eghobo has joined #openstack-ironic03:56
*** nosnos has joined #openstack-ironic04:10
*** sabah has joined #openstack-ironic04:26
*** mdorman has joined #openstack-ironic04:34
*** vinbs has joined #openstack-ironic04:35
*** mdorman has quit IRC04:41
*** lazy_prince has quit IRC04:42
*** eghobo has quit IRC04:44
*** saripurigopi has joined #openstack-ironic04:49
*** mdorman has joined #openstack-ironic04:49
*** rameshg87 has quit IRC04:49
*** mdorman has quit IRC04:49
*** rameshg87 has joined #openstack-ironic04:50
*** vinbs has quit IRC04:50
saripurigopiIs there any wiki page that describes how to run various test cases after code changes?04:50
*** vinbs has joined #openstack-ironic04:50
*** bmahalakshmi has joined #openstack-ironic04:54
*** lazy_prince has joined #openstack-ironic05:03
*** lazy_prince has quit IRC05:03
*** lazy_prince has joined #openstack-ironic05:03
*** mdorman_ has joined #openstack-ironic05:03
*** mdorman_ is now known as mdorman05:03
*** bmahalakshmi2 has joined #openstack-ironic05:04
*** bmahalakshmi has quit IRC05:04
*** radsy has quit IRC05:13
*** bmahalakshmi2 has quit IRC05:20
rameshg87saripurigopi, may be you are looking for this one ? - http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html05:21
*** rameshg87 has quit IRC05:21
*** bmahalakshmi has joined #openstack-ironic05:27
*** mdorman has quit IRC05:30
*** mdorman_ has joined #openstack-ironic05:31
*** mdorman_ has quit IRC05:33
*** bvivek has joined #openstack-ironic05:35
*** bmahalakshmi has quit IRC05:35
*** bmahalakshmi has joined #openstack-ironic05:36
*** bmahalakshmi has quit IRC05:37
*** k4n0 has joined #openstack-ironic05:38
*** bmahalakshmi has joined #openstack-ironic05:39
saripurigopi@rameshg87, yes, thanks05:45
openstackgerritNisha Agarwal proposed a change to openstack/ironic-specs: iLO Node discovery  https://review.openstack.org/10908805:55
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/11307506:10
*** dtantsur|afk is now known as dtantsur06:12
dtantsurMorning, Ironic06:13
*** rakesh_hs has joined #openstack-ironic06:14
mrdahi dtantsur06:19
vinbsMorning dtantsur06:19
dtantsurmrda, vinbs, hi :)06:22
*** zigo has quit IRC06:31
*** zigo has joined #openstack-ironic06:36
*** saripurigopi has quit IRC06:38
*** mkerrin has quit IRC07:39
*** jistr has joined #openstack-ironic07:43
*** jistr has quit IRC07:44
*** jistr has joined #openstack-ironic07:44
*** Mikhail_D_wk has left #openstack-ironic07:44
*** pradipta_away is now known as pradipta07:44
*** mkerrin has joined #openstack-ironic07:45
*** mrda is now known as mrda-away08:02
*** ndipanov has joined #openstack-ironic08:08
*** lucasagomes has joined #openstack-ironic08:09
dtantsurerrands, bbl08:15
*** dtantsur is now known as dtantsur|bbl08:15
*** nosnos has quit IRC08:19
*** nosnos has joined #openstack-ironic08:21
*** derekh has joined #openstack-ironic08:28
*** Poornima has quit IRC08:34
*** athomas has joined #openstack-ironic08:35
*** jistr has quit IRC08:35
*** jistr has joined #openstack-ironic08:38
*** k4n0 has quit IRC08:42
*** bvivek has quit IRC08:57
*** dguerri`afk is now known as dguerri08:58
*** k4n0 has joined #openstack-ironic08:59
*** jistr has quit IRC09:22
*** jistr has joined #openstack-ironic09:41
*** pradipta is now known as pradipta_away09:42
*** pradipta_away is now known as pradipta09:44
*** pradipta is now known as pradipta_away09:45
*** pelix has joined #openstack-ironic09:46
*** rameshg87 has joined #openstack-ironic09:53
*** pradipta_away is now known as pradipta09:54
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add create() and destroy() to Port object  https://review.openstack.org/11232309:58
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Remove direct calls to dbapi's get_node_by_instance  https://review.openstack.org/11259509:58
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Move code to cleanup ImageCache to a common place  https://review.openstack.org/11056010:01
*** rameshg87 has quit IRC10:24
*** k4n0 has quit IRC10:27
*** sabah has quit IRC10:41
*** ramineni has quit IRC10:58
*** k4n0 has joined #openstack-ironic10:59
*** bvivek has joined #openstack-ironic11:13
*** lucasagomes is now known as lucas-hungry11:19
*** jroll has quit IRC11:34
*** dtantsur|bbl is now known as dtantsur11:45
*** pradipta is now known as pradipta_away11:59
*** vinbs has quit IRC12:00
*** zigo has quit IRC12:17
*** zigo has joined #openstack-ironic12:18
*** jroll has joined #openstack-ironic12:19
*** lucas-hungry is now known as lucasagomes12:21
*** jroll has quit IRC12:24
Shrewslucasagomes: for the sphinx change, see this review: https://review.openstack.org/10862412:29
lucasagomesShrews, hey morning12:30
lucasagomeswill take a look12:30
Shrewsmorning12:30
lucasagomesShrews, a-ha, and this landed only for icehouse?12:30
lucasagomesthat doesn't seem to be a back port, I looked in the git logs for the requirements project12:31
*** nosnos has quit IRC12:31
Shrewslucasagomes: and havana12:31
lucasagomesand that odd version was never there afaict12:31
lucasagomesShrews, I see... alright thanks for pointing me to that12:31
lucasagomesI will change my vote12:31
lucasagomesI was a bit confused when I saw that12:31
*** jroll has joined #openstack-ironic12:32
openstackgerritYuriy Zveryanskyy proposed a change to openstack/ironic: Reduce running time of test_different_sizes  https://review.openstack.org/11324412:34
Shrewslucasagomes: any reason https://review.openstack.org/112595 isn't approved?12:42
lucasagomesShrews, it was, but there's a dependency12:43
Shrewsoh, yeah. just saw that12:43
lucasagomesI fixed the dependency and uploaded a new patch set so it lost the approved status for that patch :(12:43
openstackgerritDan Prince proposed a change to openstack/ironic: Add method for deallocating networks on reschedule  https://review.openstack.org/11304712:50
*** matty_dubs|gone is now known as matty_dubs13:02
*** shardy has joined #openstack-ironic13:05
openstackgerritLucas Alvares Gomes proposed a change to openstack/python-ironicclient: Show port by MAC address  https://review.openstack.org/11325213:05
shardyHi all, can anyone point me to the sources for http://docs.openstack.org/developer/ironic/dev/api-spec-v1.html please?13:06
*** rloo has joined #openstack-ironic13:11
dtantsurshardy, hi! IIRC it's mostly autogenerated, main source is https://raw.githubusercontent.com/openstack/ironic/master/doc/source/webapi/v1.rst13:11
shardydtantsur: Hi! Thanks, but the error I'd like to fix is in the Image section, I don't see that referenced as a controller in the code anywhere?13:14
*** bmahalakshmi has quit IRC13:14
shardyIt's only a minor nit, but figured it may be worth fixing while I'm figuring out where everything is :)13:15
dtantsurshardy, wow, I didn't know we support storing images Oo lemme see...13:15
shardydtantsur: yeah it does seem a little odd :)13:16
shardySays it's just an alias for the glance API13:16
dtantsurshardy, we don't have this endpoint13:18
*** Poornima has joined #openstack-ironic13:18
dtantsurit should be a bug13:18
dtantsurrloo, lucasagomes, Shrews, hi, have any idea what http://docs.openstack.org/developer/ironic/dev/api-spec-v1.html#image is?13:18
lucasagomesdtantsur, hey... lemme see13:18
rloohi dtantsur -- lucasagomes will check :-)13:19
lucasagomeshah well we def have no /images URI seems like an old copy&paste thing from some other project (prolly nova)13:19
dtantsurI suspect it's something seriously outdated, we already have similar problem with installation guide13:19
dtantsurshardy, thanks for reporting13:19
shardydtantsur: np, I'm happy to post a patch fixing it if we can work out where the section comes from :)13:20
dtantsurwe even had a bug for it, but you know... we're using launchpad... I can't find it13:22
rloodtantsur, shardy, lucasagomes: I think that api-spec-v1.html page was replaced by http://docs.openstack.org/developer/ironic/webapi/v1.html13:23
*** pcrews has joined #openstack-ironic13:23
Shrewsrloo: welcome back!13:23
dtantsurrloo, right. we need to get rid of these ancient pages asap, I guess13:23
lucasagomesweird I can't find that section as well13:23
rloohi Shrews, thx! (although hard to be back heh)13:23
Shrewsrloo: hehe, i'm sure. i'm gone next week. can't wait  :)13:24
shardyrloo: Oh, OK, thanks - that's *really* confusing to a new API user :)13:24
rlooshardy: yeah, I can imagine that it would be very confusing. Just not sure how to 'remove' it.13:25
lucasagomesyeah... maybe after it generates the doc it's not updating the page correctly? cause I built the docs locally and I can't find those sections13:25
dtantsurhere's related bug https://bugs.launchpad.net/ironic/+bug/132371213:25
rloodtantsur: is that an infra question? seems like there was another page you noticed awhile ago, that needed to be removed too.13:25
lucasagomesrloo, yeah looks like13:26
rloothanks dtantsur. Maybe importance is medium since it affects new users?13:27
dtantsurrloo, devananda set it to low13:27
rloodtantsur: ah. ok, we'll leave it low then. I guess we could change the page; add a link to the new page. although there should be a way to delete it.13:28
dtantsurpinged infra13:29
lucasagomesseems like our gate docs jobs is not cleaning up stuff correctly so old pages can still be accessed13:30
*** rakesh_hs has quit IRC13:30
dtantsurlucasagomes, see #infra, it must be a general problem13:30
lucasagomesdtantsur, joining13:31
*** linggao has joined #openstack-ironic13:34
dtantsurI was asked, if we have a means of creating a redirect in our docs. Anyone knows?13:35
rloodtantsur: don't you just add a redirect link to the page? (Ie, I don't know.)13:37
dtantsurrloo, I don't think we want to reintroduce these all pages in our sources13:38
rloodtantsur: agree. They can be deleted, right?13:39
dtantsurrloo, yes, manually13:39
rloodtantsur: since ironic is so young, I don't know that we care to have redirects. Redirect would be the right thing to do if eg people bookmarked the page.13:39
*** jgrimm has quit IRC13:42
*** lazy_prince is now known as killer_prince13:45
*** krtaylor has quit IRC13:53
openstackgerritA change was merged to openstack/ironic: Add create() and destroy() to Port object  https://review.openstack.org/11232313:56
openstackgerritA change was merged to openstack/ironic: Remove direct calls to dbapi's get_node_by_instance  https://review.openstack.org/11259513:56
*** krtaylor has joined #openstack-ironic13:58
openstackgerritNisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-discover-properties  https://review.openstack.org/10095113:59
*** Nisha has joined #openstack-ironic14:00
Nishalucasagomes: dtantsur posted the new patch for Discovery spec addressing your comments.14:01
openstackgerritNisha Agarwal proposed a change to openstack/ironic-specs: Discover node properties at node-create/node-discover-properties  https://review.openstack.org/10095114:06
*** killer_prince has quit IRC14:09
openstackgerritJim Rollenhagen proposed a change to openstack/python-ironicclient: Add /nodes/detail support  https://review.openstack.org/11261014:09
jrollmorning ironic :)14:09
*** sabah has joined #openstack-ironic14:09
jrolllucasagomes: good catch on that ^ thank you!14:09
lucasagomesjroll, morning! no problem, thanks for fixing it :) will re-review it soon14:10
jrollcool, ty14:10
jrollI want to land everything today :P14:10
lucasagomesnp for me :D14:13
dtantsurjroll, morning!14:17
jrollheya dtantsur :)14:17
NobodyCamgood morning Ironic14:25
dtantsurNobodyCam, morning :)14:26
NobodyCammorning dtantsur :) have a good weekend?14:26
dtantsuryes, quite :) thanks14:27
jrollmorning NobodyCam :)14:29
NobodyCammorning jroll :)14:30
NobodyCamHow was your weekend? how are you feeling today?14:31
*** Poornima has quit IRC14:32
*** k4n0 has quit IRC14:33
*** rwsu has joined #openstack-ironic14:34
lucasagomesjroll, there's small thing on #112610 that Shrews pointed out14:34
lucasagomesother than that it LGTM14:34
jrollNobodyCam: weekend was good, feeling fine :) how about yourself?14:36
jrolllucasagomes: gah14:36
jrollthanks for that one Shrews :)14:37
*** killer_prince has joined #openstack-ironic14:38
*** killer_prince is now known as lazy_prince14:38
openstackgerritJim Rollenhagen proposed a change to openstack/python-ironicclient: Add /nodes/detail support  https://review.openstack.org/11261014:38
jrolllucasagomes, Shrews ^^14:38
lucasagomesjroll, awesome +2'd14:39
NobodyCammorning lucasagomes :)14:41
lucasagomesNobodyCam, yo morning buddy, how it's going?14:42
jrollthanks lucas14:42
NobodyCammuch better then last week at the same time14:42
NobodyCam:)14:42
lucasagomes:) good14:42
*** ryanpetrello has quit IRC14:43
*** jgrimm has joined #openstack-ironic14:45
*** ryanpetrello has joined #openstack-ironic14:47
*** lazy_prince has quit IRC14:47
*** killer_prince has joined #openstack-ironic14:58
*** zigo has quit IRC14:58
*** killer_prince is now known as lazy_prince14:58
*** zigo has joined #openstack-ironic14:59
*** jasondotstar has joined #openstack-ironic15:14
*** krtaylor has quit IRC15:37
*** sabah has quit IRC15:56
NobodyCambrb15:57
devanandamorning, all15:59
openstackgerritA change was merged to openstack/ironic: Reduce running time of test_different_sizes  https://review.openstack.org/11324416:01
*** matty_dubs is now known as matty_dubs|lunch16:03
NobodyCamgood morning devananda16:04
dtantsurdevananda, morning16:07
lucasagomesmorning devananda16:10
*** krtaylor has joined #openstack-ironic16:18
*** bvivek has quit IRC16:20
*** ellenh has joined #openstack-ironic16:23
*** jistr has quit IRC16:29
*** ellenh1 has joined #openstack-ironic16:30
*** openstack has joined #openstack-ironic16:33
NobodyCamAre there any open Spec's that need looking at before the meeting?16:35
JayFI think all specs we wanted to land for J are in16:36
NobodyCamya!!!!16:36
JayFmaybe saying that in the meeting and seeing if anyone frantically disagrees16:36
JayFis the best way to be certain :P16:36
NobodyCamlol16:37
dtantsurare we trying to land this: https://review.openstack.org/100951 ?16:37
JayFVery much not is my understanding16:38
JayFall the node-update/discovery/hardware interrogation stuff is going to be significantly hashed out at the summit16:38
ShrewsIs it still a no-no to +A something before Jenkins gives it a +1?16:38
JayFShrews: I've heard it's OK16:39
ShrewsJayF: from?16:39
NobodyCamI try and wait until mr J is done16:39
JayFdeva :)16:39
Shrews'cept Mr. J is slooooooow16:39
JayFI've seen him do it (IPA driver at Mid-cycle)16:39
BadCubMorning everyone16:39
NobodyCammorning BadCub16:40
*** ellenh1 has quit IRC16:41
dtantsurFolks, is there interest in thing like Ansible playbook for Ironic quick start guide? I have some here in private and wondering if I should share (requires some clean up though)16:42
JayFI think you should share all code you use --> provisioning, software, anything16:42
JayFbest case scenario you make your life easier (open repos are easier to use) and help others16:43
Shrewsdtantsur: i'm sure there would be interest in it from someone16:43
NobodyCamdtantsur: YES!!!!!!16:43
JayFworst case you still make it easier for yourself :)(16:43
dtantsurit requires some effort to separate it from my private (really private) cookbooks, but I'll do it maybe tomorrow16:43
dtantsurnow food-buying-time, brb16:44
NobodyCamdtantsur: that would awesome!!!!!16:44
NobodyCamoh seems I need to do a quick walkies... brb16:44
JayFThe only closed source stuff (other than all our deploy automation, which is integrated with our cloud stuff) that we have is we use our own CoreOS image builder, that's basically a copy of imagebuild/coreos is IPA with our own hardware manager added and a lot of utilities (like for bios)16:45
*** romcheg1 has joined #openstack-ironic16:47
dtantsurJayF, well it started as: "let's create a cookbook for setting up the VM how I like it" :) and ended up with some good stuff inside16:48
dtantsurthe hardest thing would be to make it work under Ubuntu, though16:48
dtantsurok, really brb for now16:48
JayFjust open it up. if folks wanna make it work better they can :)16:48
*** vinbs has joined #openstack-ironic16:50
devanandadtantsur: yes!16:51
devanandadtantsur: is this a playbook to use ansible to start and configure ironic? or to use ironic to provision?16:52
devanandaboth are useful. just curious which one you've done16:52
dtantsurdevananda, to configure. I have 2 peaces: one for "Exercising the Services Locally" (up to actually starting conductor & API), another small one for "Deploying Ironic with DevStack" up to running ./stack.sh16:53
JayFNobodyCam: I dropped a -2 and a comment on https://review.openstack.org/#/c/100951/3316:54
dtantsurI'm not sure it's super-useful, but I no longer can do it by hand on each VM :)16:54
NobodyCamJayF: ack :)16:54
dtantsur+ in 1st part using mariadb instead of sqlite16:54
*** vinbs has quit IRC16:55
*** ndipanov is now known as ndipanov_gone16:55
* JayF really hopes his and JoshNang's decom talk get in for Paris, watching nodes decom is thrilling16:55
dtantsurehh16:56
* dtantsur has very low chances of getting to Paris16:56
JayFWe were told that if we got a talk in, we'd get sent. I'm going to try hard to go either way because I think there are some design sessions we need to have that I want to be in16:57
devanandadtantsur: ok. neither of those are quite what I'm looking for, but they'd none the less be helpful :)16:57
devanandaJayF: a few of you definitely should go16:57
dtantsurdevananda, heh, one day I can actually end up with what you want. I'm an automation fan :) what were you looking for?16:58
JayFdevananda: we're trying :) pull strings to make sure me+joshnang's talk is approved and jroll's talk is approved and that'll be three os us ;)16:58
*** dtantsur is now known as dtantsur|brb16:59
*** matty_dubs|lunch is now known as matty_dubs17:00
*** derekh has quit IRC17:02
*** lucasagomes is now known as lucas-dinner17:04
*** romcheg1 has quit IRC17:04
*** bearhands is now known as comstud17:07
*** bmahalakshmi has joined #openstack-ironic17:13
NishaHi JayF17:29
NishaJayF: https://review.openstack.org/#/c/100951/ is not node-discovery, it is node-properties discovery17:32
JayFNisha: i.e., I add a node, then Ironic discovers details about the node properties?17:33
JayFNisha: If so, then yes, that's something we talked about at the mid-cycle and decided to table until Kilo because of some of the contenious nature of adding the scope to Ironic :(17:33
NishaJayF: Sorry i didnt get the above ^17:33
JayFNisha: meaning, your spec is about discovering properties of an added node17:34
JayFi.e. hardware inspection/validation17:34
JayFright?17:34
NishaNo, it is discovering properties at the time of registering node17:34
*** pelix has quit IRC17:34
JayFNisha: yes, that's what I meant. That is something that I think we (Ironic) want to talk about at the design summit in paris and talk about what's in scope, how to implement, etc17:35
JayFNisha: we spent the better part of a day talking about it at mid-cycle -- both your specs and ones like them17:35
openstackgerritEllen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable  https://review.openstack.org/11235117:36
NishaJayF: I posted the patch after i got comments from lucas-dinner last week ....for those comments i had discussion with lucas-dinner and dtantsur|brb on IRC too17:37
*** dtantsur|brb is now known as dtantsur17:37
jrolldevananda: after https://review.openstack.org/112610 merges, can we tag a new client? :)17:37
jrolldevananda: the nova caching patches are waiting for a new client version to land17:38
devanandajroll: sure17:38
jrollcool, thanks - I'll ping you if I see it merge17:38
JayFNisha: I understand, and I'm sorry to bear the bad news in this case. You can feel free to bring it up in the meeting if you think I'm wrong, and I'll gladly remove my -2 if the consensus changes that we should try and implement this for Juno :)17:38
dtantsurNisha, your patch is not wrong or bad, we appreciate your work, but 1. core team is already quite busy and won't be able to land more; 2. discovery is a hot topic that is expected to be prototyped more thoroughly during K cycle17:38
Nishadtantsur: JayF Ok.17:39
dtantsurbut yeah, if folks decide to give it a go - I'm ok with it. Bare in mind, that it does not make sense to land only this spec, but also 2 consequent17:39
jrollmrda-away: fyi, we want to get these landed and into the nova patches: https://review.openstack.org/#/c/108545/ https://review.openstack.org/#/c/113036/17:39
jrollmrda-away: waiting on a new client, then they'll be good to land17:40
devanandajroll: mrda-away and I are tracking the nova work onthe whiteboard. I'll add those17:40
jrolldevananda: right on, ty17:40
devanandaalso, I spent several hours on friday merging small things into the patch chain17:41
*** bmahalakshmi has quit IRC17:41
devanandaI'm going to begin arming mrda with sharp pointy objects so he can poke all of you17:42
jrollyay17:42
Shrewsjroll: the '70s called. they want their lingo back17:42
jrolllol17:42
jroll/nick stuckinthe70s17:42
Shrewsrighteous17:43
jroll:P17:45
*** rameshg87 has joined #openstack-ironic17:45
*** romcheg1 has joined #openstack-ironic17:48
NobodyCamlol17:48
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Move code to cleanup ImageCache to a common place  https://review.openstack.org/11056017:50
*** romcheg1 has quit IRC17:52
*** dguerri is now known as dguerri`afk17:56
rameshg87devananda, hi18:16
*** coolsvap has quit IRC18:17
*** harshada_kakad_ has joined #openstack-ironic18:17
jrollhas anyone proposed a spec for notification support, like nova does?18:20
*** coolsvap has joined #openstack-ironic18:20
rameshg87jroll, something similar to this ? - https://review.openstack.org/#/c/99770/18:22
NobodyCambrb18:22
jrollrameshg87: more like https://github.com/openstack/nova/blob/master/nova/notifications.py18:23
jrollrameshg87: it puts notifications for events on the rabbit bus18:23
rameshg87jroll, okay.18:24
devanandajroll: yes. lemme find the link18:29
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Take iSCSI deploy out of pxe driver  https://review.openstack.org/11123218:31
devanandajroll: no. but there's this, which is not what you asked for :) https://review.openstack.org/#/c/72538/44/ironic/conductor/manager.py18:32
jrolldevananda: heh, not at all :)18:33
jrollok18:33
jrollI'm thinking build/destroy notifications etc18:34
devanandajroll: right. afaik, no, because Nova does that alraedy, so it would kinda be redundant?18:35
dtantsur" Fedora support (dtantsur)  There hasn't been much to report in a while. Should we take this off the agenda?"  <-- likely yes. I guess there is no much breaking changes now.18:35
dtantsurdevananda, ^^^18:35
devanandadtantsur: great, thanks for the updaet!18:35
jrolldevananda: for those, yes, but what about e.g. 'decom failed', nova won't know anything about that18:35
jrolldevananda: or 'power state set to off'18:36
jrollidk if that's useful or not18:36
jrollbut likely is18:36
devanandajroll: honestly idk what the notifications are actually used for -- i suspect feeding into billing systems and the like, and you're the only ones (so far) billing for Ironic usage18:37
devanandajroll: I am much more interested in API callbacks between services18:37
russell_hdevananda: we just bill for nova usage18:37
jroll^^18:38
russell_hfor what thats worth18:38
devanandaeg, so that nova doesn't need to poll as often, and Ironic can actively inform nova of resource changes18:38
devanandarussell_h: thanks. that's what I expected18:38
jrolldevananda: I want to follow a stream of events and do things when bad things happen18:38
jroll:)18:38
jrolls/bad/interesting/18:38
devanandajroll: as an operator, creating operational tooling to troubleshoot "interesting" things -- I get that. tail the log file :)18:39
devananda(j/k)18:39
jrollheh18:39
devanandabut really, what's the difference between that kind of notification vs. a DEBUG log?18:39
jrollthat was the other direction I thought about, just stream logs18:39
devanandaor an ERROR log?18:39
jrollthese would be ERROR/WARNING logs18:39
jrollyeah18:39
jrollbecause we already have infrastructure that consumes a rabbit queue of events18:40
devanandaright. so yea, I think that's mroe what you're looking for, whereas the notification bus is there for billing and auditing events18:40
jrollI suspect others may have that18:40
jrollright - auditing may be a good use case18:40
devanandaI dunno - perhaps there's a queue for each18:40
jrollaudit ironic API use18:40
devanandasure18:40
devanandaI can definitely see that18:40
devananda"user 123 created node XYZ"18:40
jrollbut also thinking of seeing issues that nova wouldn't see - e.g. bmc goewdies18:41
jrolldies*18:41
*** harshada_kakad_ has quit IRC18:44
NobodyCamlast chance to get your item on the agenda18:44
*** harshada_kakad_ has joined #openstack-ironic18:45
devanandahmm18:45
devanandawe have an iLO driver in tree right now -- but as it landed, it's non-functional18:46
matty_dubsNot an agenda item for the meeting, but I should mention to you guys... I've accepted a new position, so this will be my last week at Red Hat, and my last week working on OpenStack. :'( It's been terrific working with you all, though!18:46
dtantsurmatty_dubs ;(18:47
devanandait includes a new "ilo" top level driver, which does not implement the required DeployInterface :(18:47
matty_dubsdtantsur: Yeah :(18:47
NobodyCamomg matty_dubs .... best of luck in your new directrion :)18:47
devanandamatty_dubs: congradulations. Good luck on your new adventure, though you'll be missed here18:47
NobodyCamdirection even18:47
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-python-agent: Use relative time sleeping in heartbeat thread  https://review.openstack.org/11334318:47
dtantsurdevananda, oh, and I've seen the same with recent (not yet merged) drac driver - no deploy interface18:47
NobodyCam+1 to matty_dubs will be missed18:48
matty_dubsThanks guys :)18:48
jrollmatty_dubs: :( but grats on the new gig :)18:48
devanandadtantsur: https://review.openstack.org/#/c/89500/2618:48
devanandadtantsur: not implementing required interfaces means it's not really a driver and should not be added to setup.cfg18:49
devanandawe should not have approved that patch :(18:49
dtantsuryeah, shame on me for approving :(18:49
*** dlaube has joined #openstack-ironic18:50
dtantsurrameshg87, ^^^18:50
rameshg87dtantsur, oh !18:52
rameshg87dtantsur, so should we remove it from setup.cfg until deploy driver is merged ?18:52
rameshg87dtantsur, does that make sense ?18:52
devanandaI'm fixing it now18:53
devanandaone sec18:53
dtantsurrameshg87, what's the state of the deploy driver btw?18:54
lucas-dinnerdevananda, oh thanks for the review on the ports client patch, the /detail makes total sense!18:54
*** lucas-dinner is now known as lucasagomes18:54
devanandalucasagomes: yvw18:54
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Replace incomplete "ilo" driver with pxe_ilo  https://review.openstack.org/11334618:54
devanandarameshg87: ^ paves teh way for using PXE for deploy but iLO for power and management interfaces18:55
devanandarameshg87: when you have the minimum "core" interfaces landed, then please re-add the "ilo" driver to setup.cfg18:55
rameshg87dtantsur, WIP - should post the complete set of patches for deploy driver in a couple of days18:55
rameshg87devananda, okay, will add it when the deploy has landed too. thanks for taking care of it.18:56
devanandaalmost meeting time :)18:57
*** mrda-away is now known as mrda18:57
mrdaMorning Ironic18:57
dtantsurmrda, morning :)18:58
devanandamrda: good morning!18:58
mrdadevananda: dtantsur \o18:59
*** iron1 has quit IRC18:59
NobodyCammorning mrda18:59
mrdajroll: ack18:59
jroll:)18:59
mrdaNobodyCam: \o19:00
NobodyCamo/19:00
*** romcheg1 has joined #openstack-ironic19:19
openstackgerritA change was merged to openstack/python-ironicclient: Add /nodes/detail support  https://review.openstack.org/11261019:23
jrolldevananda: friendly reminder to tag a new client after the meeting :) ^19:23
devanandajroll: snooze 40 minutes19:24
devananda:)19:24
NobodyCam:-p19:24
jrollyou weren't supposed to look yet :P19:24
devanandait pinged me19:24
*** romcheg1 has quit IRC19:26
*** bvivek has joined #openstack-ironic19:31
JayFhttps://review.openstack.org/#/c/112351/ link to the external dhcp provider review I'm shilling in the meeting19:32
dtantsurit's on my list for tomorrow19:33
JayFellenh: ^ lucasagomes and dtantsur both agreed to look at that tomorrow19:34
* JayF wants to get it landed before you turn into a pumpkin19:34
jrollyeah, I need to re-review that today19:34
lucasagomes:)19:34
*** rameshg87_ has joined #openstack-ironic19:43
lucasagomesjroll, will talk here because we are out of topic there19:43
lucasagomesI think19:43
lucasagomesjroll, so I just worked around on that patch, I still have to find out why it was calling cleaning up before configure19:43
jrollah, I see19:44
jrollthat works for me, I wouldn't want to dig into devstack19:44
dtantsurBugs I struggle to triage are most of 'New' bugs here: http://ironic-bugs.divius.net/19:45
lucasagomesjroll, yeah... it worked before and out of the blue it stopped, but not many changes in the lib/ironic itself that would cause it afaict19:45
lucasagomesso it should be something deeper19:45
NobodyCamdtantsur: will take a look a few minutes afterthe meeting19:45
*** ellenh has quit IRC19:46
*** rameshg87 has quit IRC19:46
*** bvivek has quit IRC19:50
*** tatyana has joined #openstack-ironic19:58
NobodyCambrb20:01
NobodyCamgreat meeting everyone20:01
dtantsuryeah20:02
devanandarameshg87_: is https://review.openstack.org/#/c/111232/ necessary for the iLO work? or just "nice to have" ?20:02
rameshg87_devananda, it is *required* for ilo work20:02
jrolldevananda: I had the same question about how useful that patch was, as well20:02
devanandarameshg87_: oh, I see.20:02
rameshg87_devananda, unless we separate out iscsi portions, we cannot use it in the ilo driver20:02
rameshg87_jroll, i thought i had succeeded in convincing you :-)20:03
* mrda goes back to bed for a couple of hours20:03
devanandarameshg87_: ahh, and you are going to use the iSCSI functions with iLO boot (not PXE boot)20:03
devanandamrda: g'night :)20:03
* devananda steps afk for 520:03
dtantsurwell, it's already EOD here, g'night everyone :)20:03
rameshg87_devananda, yes, just replace the pxe portions with "ilo virtual media boot"20:03
*** wanyen has joined #openstack-ironic20:03
lucasagomesI'm also going, have a great night everyone20:03
lucasagomesdtantsur, night night20:04
*** dtantsur is now known as dtantsur|afk20:04
*** lucasagomes is now known as lucas-dinner20:04
*** Nisha has quit IRC20:04
*** harshada_kakad_ has quit IRC20:04
rameshg87_devananda, i am just moving around the code in that review.  i didn't have trouble rebasing it after the ipa and ipxe work.20:04
jrollrameshg87_: I wasn't done :)20:05
jrolldevananda: I think it is useful20:05
rameshg87_jroll, :-)20:05
NobodyCamnight mrda lucas-dinner20:09
devanandarameshg87_: http://docs.openstack.org/developer/oslo.config/opts.html#oslo.config.cfg.DeprecatedOpt20:10
rameshg87_devananda, thanks. let me take a look at this.20:11
devanandarameshg87_: think of the impact this has on an existing deployment which upgrades20:13
*** penick has joined #openstack-ironic20:13
devanandarameshg87_: if by mistake they don't update their config file, it will cause significant problems. a "simple refactoring" shouldn't do that.20:13
rameshg87_devananda, yeah, i agree. moving around config option is a messy business. :-(20:14
devanandarameshg87_: when defining a config option, there are some optional params20:14
devanandadeprecated_name and deprecated_group20:14
devanandaI would prefer they not change at all right now20:15
devanandagiven that we want to refactor this in Kilo anyway, we really should only change these config options once20:16
rameshg87_devananda, it's not a must for the options to change, but wouldn't make logical sense with the code20:16
rameshg87_devananda, for example: https://review.openstack.org/#/c/111232/5/ironic/drivers/modules/iscsi_deploy.py20:16
devanandarameshg87_: what about that? you can import options from anotehr file20:17
rameshg87_devananda, it would make sense for images_path to be moved here because it is iscsi_deploy that handles it20:17
rameshg87_devananda, yeah we can still refer the options in other file for now, and later do it once-and-all for K20:18
rameshg87_devananda, is that acceptable ?20:18
*** penick has quit IRC20:18
devanandayes. that would be a much cleaner refactoring now.20:18
rameshg87_devananda, okay. i can do that.  i can add a small note in the code for that.20:19
devanandagreat, thanks!20:19
rameshg87_devananda, i will do it this way then, i won't move the config options.20:19
rameshg87_devananda, thanks.20:19
devanandadon't hesitate to add #NOTE inline explaining the intent20:20
wanyendeva,  I  am thinking to use ipmitool move sensro data to ceilometer and remote console in ilo driver.  Can we file a bug to do this kindof mix and match fnctions?20:21
devanandawanyen: you want to use iLO for deploy and management, but ipmitool for sensor data?20:22
wanyendeva, ilo driver does not implement sensor data and remote console so I am thinkg to leverage those two fuctions in ilo driver.20:23
devanandawanyen: yep, seems reasonable20:23
devanandawanyen: that should just be a 2 line patch, right?20:23
wanyendeva, so do we need to file a bug to do that or can we just do  it in the ilo deploy driver as part of of the ilo deploy drive?20:24
devanandawanyen: just do it as part of the ilo driver20:24
wanyendeva, great!  thanks!20:25
devanandawanyen: IIUC it's just a change to this file: https://github.com/openstack/ironic/blob/master/ironic/drivers/ilo.py20:25
devanandawanyen: which you'll need to make anyway when you land the vmedia deploy driver, so ... just do it then.20:25
wanyendeva, okay.20:26
*** rameshg87__ has joined #openstack-ironic20:29
*** rameshg87_ has quit IRC20:31
*** tatyana has quit IRC20:36
*** notq has joined #openstack-ironic20:42
NobodyCamare folks using the Ironic Priorties & planning Gdaic?20:50
NobodyCams/Gdaic/Gdoc/20:50
jrollyes20:52
JayFdevananda: question then; along the same lines would that seamicro console feature be a bug and not a spec?20:53
rlooNobodyCam: I have in the past, and I may look at it in the future, but I don't look at it frequently.20:53
NobodyCamack TY20:55
*** tatyana has joined #openstack-ironic20:55
jrolldevananda: reminder to tag a new client :)20:56
jrolldevananda: related: do I need to bump requirements.txt to the new version along with my bugfixes?20:56
*** rameshg87__ has quit IRC20:57
*** rameshg87__ has joined #openstack-ironic20:57
jrollwhat's syntax for updating a thing in e.g. driver_info? how does this look?20:59
jrollironic node-update <uuid> replace /driver_info/ipmi_address=10.2.3.420:59
JayF^ that works21:01
*** matty_dubs is now known as matty_dubs|gone21:01
JayFthanks jroll21:01
*** linggao has left #openstack-ironic21:05
devanandaJayF: not sure I grok your question21:05
JayFdevananda: if adding the ipmi stuff to the ilo driver is a 'bug', would adding a serial console support to seamicro driver also be a bug?21:06
JayFi.e. instead of a spec, and that guy could get his stuff into J21:06
devanandaJayF: the ipmi-in-ilo thing isnt' even a bug -- it's two lines of "glue" connecting existing interfaces together when they add the IloDeploy interface class (which is part of an already approved spec)21:07
devanandaJayF: if the seamicro thing was the same -- just glue the existing serial colsole support into seamicro's driver, sure, taht's fine21:08
devanandabut AIUI, https://blueprints.launchpad.net/ironic/+spec/seamicro-serial-console is proposing a new class21:08
JayFI am just trying to learn where the line is. Does it lie in how complex the implementation is then?21:08
devanandayep21:08
devanandathe line between bug and feature is stupidly fuzzy21:09
devanandaand there's a fairly broad concensus we should abandon launchpad to get away from that21:09
devananda(i'm oversimplifying -- don't quote me on that)21:09
devanandaJayF: but for example, IIUC, the seamicro console would add a requirement / dependency on telnet21:10
NobodyCambrb21:10
JayFgotcha. If it pivots on complexity that makes a lot of sense, I just didn't know if there was a general idea of 'implementing an existing ironic feature in a driver that doesn't have it is not spec-worthy' type of feel21:11
devanandawith only ~3 weeks to go to FF, core reviewers should be able to focus on landing code, not discussing design of new drivers or features21:11
JayFseems like it mostly hinges on complexity, which practically makes a lot of sense21:11
devanandait really pivots on that ^21:11
JayFheh21:11
JayFETOOMUCHWORK21:12
devanandaright :)21:12
devanandaalso, fwiw, "implement feature in driver X that already exists in driver Y" totally needs a spec21:12
devanandawhat if driver X does it differently? in a way that totally doesn't scale the same way as driver Y?21:12
jrollis that the same as "implement this already defined api method in driver X"?21:13
JayFthe ilo/ipmi stuff muddied my opinion, but I get how that's not even ^ that :)21:13
jrollI suspect yes21:13
devanandathe reason the iLO console question was such an easy answer -- they're NOT implementing anything new21:13
devanandajroll: yes. same thing. driver wants to implement an optional interface which they didn't previously implement -> spec please.21:13
jrollright, ok21:13
devanandathe iLO console is taking the existing ipmitoolconsole class and just glueing it into their driver21:14
devanandawhich is the whole point of the modular interfaces :)21:14
*** rameshg87_ has joined #openstack-ironic21:14
*** rameshg87__ has quit IRC21:16
*** ellenh has joined #openstack-ironic21:18
openstackgerritRamakrishnan G proposed a change to openstack/ironic: Take iSCSI deploy out of pxe driver  https://review.openstack.org/11123221:39
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Use auth_token from keystonemiddleware  https://review.openstack.org/10719721:56
*** rameshg87_ has quit IRC21:57
*** eghobo has joined #openstack-ironic21:57
*** rameshg87_ has joined #openstack-ironic21:57
devanandarameshg87_: I believe you need to use CONF.import_opt() to access conf options defined in other files21:59
jrollnot necessarily21:59
jrollif the other files get imported before your file it should just work22:00
jrollor something22:00
rameshg87_devananda, ah, i didn't. i referenced them directly22:00
jroll(I didn't import the pxe config options in the agent driver)22:00
devanandajroll: oh, i see22:00
devanandajroll: so that "if" isn't really great22:00
rameshg87_devananda, even it worked for me now referencing them directly :-)22:00
jrolldevananda: right :)22:00
jrollI'm not entirely sure if my "if" is true22:01
mrda.22:01
rameshg87_jroll, what if somebody disabled pxe driver and enabled "only" agent driver ?22:02
rameshg87_jroll, in such a case the the module itself won't be loaded, right ?22:02
jrollrameshg87_: it still works, we import all drivers at conductor startup or something silly22:02
jrollidk how any of this works22:02
jrollbut it does22:02
jroll:|22:02
rameshg87_jroll, i thought driver_factory loaded only drivers that were enabled22:03
devanandajroll: ooh, i just leard you can declare an option as "secret"22:03
devanandahttp://docs.openstack.org/developer/oslo.config/cfg.html near the bottom22:03
jrollwat22:03
devanandaseems like someting to do for the swift token maybe?22:03
jrollrameshg87_: it seems to load them all in my experience. but I might be wrong22:03
JayFthat's pretty sweet22:03
jrolldevananda: true that, let's try it :P22:04
rameshg87_jroll, okay22:04
jrolldevananda: also mysql connection strings etc22:04
rameshg87_jroll, yeah perhaps the tmpurl key is a secret :-)22:04
jrolldevananda: maybe I'll fix them all and put up one review22:04
* jroll just does it22:04
rameshg87_devananda, so do we really need to use import_opt ?22:05
devanandarameshg87_: I believe you should -- http://docs.openstack.org/developer/oslo.config/configopts.html?highlight=import_opt#oslo.config.cfg.ConfigOpts.import_opt22:06
devanandafwiw, there are 312 occurrences of CONF.import_opt in Nova22:07
rameshg87_devananda, okay. will use it then .. thanks22:07
devanandajroll: ^22:07
devanandajroll: oslo.config is a strage sort of global thing22:08
devanandajroll: the biggest gotcha with it that i keep seeing is, you can't trust anything in CONF at parse time22:08
devanandabecause it hasn't loaded the config file yet22:08
*** rameshg87_ has quit IRC22:09
jrolldevananda: right22:09
jrollso like, you can't use a CONF thing at module load time22:10
jrollbut like, at runtime (inside a method or whatever), you can22:10
jrollhmm22:10
mrdajroll: well, at least not in a way that works consistently.  I've heard stories from Nova of intermittent bugs when this was done...22:11
jrollyeah :\22:12
jrollhuh, our conductor in devstack doesn't do a config dump at startup :/22:14
openstackgerritEllen Hui proposed a change to openstack/ironic: Make DHCP provider pluggable  https://review.openstack.org/11235122:15
* NobodyCam is back22:22
devanandajroll: dont define a function default =CONF.something, for example22:22
devanandabecausethat's eval'd at parse time22:22
rloodevananda: I replied to your comments to my comments, in https://review.openstack.org/#/c/107197/. If it doesn't make sense it is cuz I'm still trying to catch up and remember how to code ;)22:24
jrolldevananda: yeo22:24
jrollyep22:24
*** rloo is now known as rloo_afk22:24
devanandarloo_afk: sure. fwiw, this change is trying to update ironic to use the newly requried keystonemiddleware package22:25
devanandarloo_afk: some things changed upstream from us. keystone went and fixed it in all the integrated projects22:26
devanandayou can see the changes they made here https://launchpad.net/bugs/134227422:26
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Make swift tempurl key secret  https://review.openstack.org/11339022:29
jrolldevananda: ^^22:29
jrolltested in devstack too22:29
devanandaand?22:29
jrollalthough it's not printed as-is today due to imports ordering22:29
jrollor whatever22:30
jrollbut when I imported that module, to force parsing those configs22:30
jrollit worked fine22:30
jroll2014-08-11 22:26:37.066 DEBUG ironic.cmd.conductor [-] glance.swift_temp_url_key      = **** from (pid=23769) log_opt_values /usr/local/lib/python2.7/dist-packages/oslo/config/cfg.py:198822:30
devanandanice22:30
*** eguz has joined #openstack-ironic22:30
*** radsy has joined #openstack-ironic22:33
*** eghobo has quit IRC22:33
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Use auth_token from keystonemiddleware  https://review.openstack.org/10719722:34
devanandajroll: new tag pushed22:43
*** jgrimm has quit IRC22:44
*** eguz has quit IRC22:51
*** eghobo has joined #openstack-ironic22:54
jrolldevananda: thanks!22:59
jrolldevananda: do I need to bump requirements.txt to require the latest, if we have code depending on the latest?23:00
openstackgerritGhe Rivero proposed a change to openstack/ironic-python-agent: Sync oslo.incubator modules  https://review.openstack.org/11339723:00
*** eghobo has quit IRC23:00
*** eghobo has joined #openstack-ironic23:00
*** lucas-dinner has quit IRC23:01
« Sunday, 2014-08-10 Index Tuesday, 2014-08-12 »

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