Tuesday, 2013-12-10

*** yfujioka has joined #openstack-ironic00:16
openstackgerritJenkins proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/6097200:17
*** jcooley_ has joined #openstack-ironic00:17
openstackgerritJenkins proposed a change to openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/6098200:19
*** epim has joined #openstack-ironic00:20
devanandarloo: on https://review.openstack.org/#/c/59795/7/ironic/db/api.py, any other suggestions?00:26
devanandai'm not keen on the current name either00:26
devanandabut it was all i thought of at the time ....00:27
*** rongze has quit IRC00:34
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Improve method to get list of active conductors  https://review.openstack.org/5979500:34
rloodevananda: i'm not good with naming. but yes, get_active_driver_dict is clearer.00:38
devanandagreat00:38
*** docaedo has quit IRC00:38
rloodevananda: so there are active drivers and non-active drivers?00:39
devanandawell...00:39
*** docaedo has joined #openstack-ironic00:39
devanandathere can be non-active conductors00:39
rlooyeah, I'm too literal sometimes.00:39
rlooso if you were to describe what you're getting, what would you say? get drivers? get drivers from active conductors?00:40
rloowhat if you wanted drivers from non-active conductor?00:40
devanandawhat determines a non-active conductor?00:40
devanandaone that previously was active, but has since timed out?00:40
rlooi don't know. you tell me. I didn't use the term 'active conductor' :-)00:41
devanandaor one that was intentionally turned off -- and presumably deregistered itself on shutdown (I need to implement taht ....)00:41
rlooif I recall from the code, it was 'previously active, but since timed out'00:41
devanandacorrect00:41
devanandathe other is a case I'm wrestling with in my head00:41
devanandathe issue is mutlifold00:42
*** davidlenwell has joined #openstack-ironic00:42
devananda*multifaceted00:42
rlooand a conductor that just dies...00:42
devanandawill eventually be considered timed out, yes00:42
rlooi guess a dead conductor is taken care of cuz it timed out00:42
devanandato answer your question00:43
rlooa conductor that was turned off will also have timed out?00:43
devananda00:40:04 < rloo> so if you were to describe what you're getting, what would you say? get drivers? get drivers from active conductors?00:43
rlooit doesn't, at least now, seem like you can get drivers from non-active conductors, right?00:43
rlooso it is just get-drivers?00:43
devanandai would say, what i really want is a list (or set) of the hostnames of all the conductors which are part of the hash ring that I am a part of00:44
rloowell, i guess you can get from non-active conductors -- the ones that timed out but are still alive.00:44
devanandawhere "I" == "the conductor issuing the query"00:44
rloothe conductors that are part of the hash ring. what happens to a conductor that times out. is it part of the hash ring still?00:44
rloo(I forgot if I saw it in the code or not.)00:45
devanandain the proposal from lifeless, it would still be part of the hash ring, but it would be added to an "exclude" list00:45
devanandahttps://etherpad.openstack.org/p/IronicConsistentHashingForInstances00:45
* lifeless perks up00:45
devanandafor reference00:45
rlooand I guess you don't want drivers from the excluded conductors?00:45
devanandalifeless: hi! just chatting with rloo about the hash stuff. she's digging into my patch sets and the new revisions thereof.00:45
lifelesscool00:46
*** davidlenwell_ has quit IRC00:46
devanandarloo: so it's not really the drivers i'm interested in00:46
devanandait's the list (or set) of conductors00:46
devanandaif all conductors shared the same list of drivers, there would only need to be a single hash ring00:46
devanandarloo: but since it is possible (and by design) for different cdr's to have different drivers, we need, effectively, one hash ring per driver00:47
lifelesshuh, why did https://review.openstack.org/#/c/59794/8 not land ?00:47
*** jcooley_ has quit IRC00:47
devanandalifeless: rebase clears the +A00:47
devanandalifeless: rev 8 depended on something else00:47
lifelessok00:48
* lifeless resubmits it00:48
devanandalifeless: i split my series into two branches for simplicity. -- thanks!00:49
*** GheRiver1 has joined #openstack-ironic00:49
*** openstack has joined #openstack-ironic00:49
*** SpamapS has quit IRC00:49
lifelessdevananda: we could do one ring and reprobe, but I agree, N rings is better.00:49
*** bigjools has quit IRC00:50
devanandalifeless: ack00:50
*** bigjools has joined #openstack-ironic00:50
*** bigjools has joined #openstack-ironic00:50
*** SpamapS has joined #openstack-ironic00:50
rloolifeless seems to have a way with words. Maybe he can come up with a good name, devananda.00:50
devanandalifeless: also, fwiw, i got some time to whiteboard with krow. he (surprisingly) liked almost all of this design. except for the heartbeat table00:50
*** jcooley_ has joined #openstack-ironic00:51
lifelessheh00:51
lifelessthats good00:51
*** jcooley_ has quit IRC00:51
openstackgerritA change was merged to openstack/ironic: Add config option for # of conductor replicas  https://review.openstack.org/5979400:51
devanandaand suggested we instead use TCP sockets between all the conductors to know when one dies. Not that we'd send anything across them... just keep them open...00:51
lifelessmmm00:52
lifelessand conductor to all APIs then, so that users can query?00:52
devanandapresumably. which is less than ideal.00:53
lifelessor then you get into 'ask a random conductor whats up'00:53
devanandayea, or something like that00:53
lifelessbut maybe that conductor has a different view to the other conductors00:53
lifelessso you'd have to ask them all00:53
lifelessIt's an interesting take, but honestly - if we're going to reinvent presence, lets do that explicitly, or otherwise use a library to do it.00:54
devananda*nod*00:54
devanandapretty much my response -- neat idea, not sure it'll fit (it might) but it would be a lot to engineer well, for minimal returns right now00:54
lifelessI mean, the heartbeat table is also a form of reinvention, but it's easier to write for now and we can revisit later to replac.00:54
devanandaindeed00:54
devanandabut it's a pretty standard way to invent this, with minimal work00:54
devanandathe hash is much more complex00:55
lifelessactually00:56
lifelessI think, technically, the hash is much simpler.00:57
*** GheRivero has quit IRC00:57
lifelessIt doesn't interact with nearly as many variables outside our control.00:57
lifelessLike network splits :)00:57
devanandarloo, lifeless: if you two have time for one more review, https://review.openstack.org/#/c/58894/. I think I've addressed your comments00:57
lifelesscan one conductor run many drivers?00:57
devanandalifeless: hmm. the hash by itself, yes. but when you consider the interaction with drivers to do take over, etc, it gets more interesting00:58
devanandalifeless: absolutely00:58
devanandalifeless: and conductors in the same cluster may have different drivers00:59
lifelessdevananda: I've -1'd https://review.openstack.org/#/c/59795/00:59
lifelessdevananda: very minor fixes00:59
lifelessdevananda: I have a question for you in https://review.openstack.org/#/c/60954/01:00
devanandaeg, this rack is driverA. that rack is driverB. another rack is a mix of C and D. two conductors per rack. all in the same AZ. -- I want to support that kind of topology.01:01
devanandalooking01:01
devanandalifeless: see the commit message for explanation01:01
lifelessdevananda: can you file a bug on it then? And reference that bug (Related-Bug: # IIRC) in the commit ?01:02
devanandalifeless: i know it breaks PEP. as it is, it breaks the sphinxcontrib-pecanwsme doc formatting.01:02
devanandaack01:02
lifelessdevananda: because otherwise folk will 'fix it'.01:02
lifelessdevananda: also a comment on that docstring01:02
lifeless# XXX (devananda) bug XYZ breaks this if formatted correctly.01:03
lifelessdevananda: thanks!01:03
lifelessdevananda: https://review.openstack.org/#/c/60904/ - You might want to see if you agree with my assessment w.r.t pxelinux.0 there.01:06
*** hemna has quit IRC01:10
*** epim has quit IRC01:18
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Fix auto-generated REST API formatting  https://review.openstack.org/6095401:23
devanandalifeless: on your comment, single conductor can serve many different arch, I presume you mean eg. i686, x86-64, and arm?01:25
devanandalifeless: or do you mean linux + windows?01:25
lifelessarch, not OS01:27
lifelessand yes, ARM specifically01:27
devanandaright01:27
devanandawhat would be different, exactly?01:27
*** nosnos has joined #openstack-ironic01:28
lifelessit wouldn't be pxelinux.0, AIUI.01:28
lifelessI'm a little fuzzy here TBH01:29
lifelesse.g. http://rsalveti.wordpress.com/2011/07/11/net-booting-with-tftp-and-pxe-with-pandaboard/01:29
lifelessU-Boot reads the same files as pxelinux.001:30
lifelesshttps://bugs.launchpad.net/ubuntu/+source/u-boot-linaro/+bug/92778101:31
lifelesspossibly we should have a separate driver for U-Boot01:32
lifeless?01:32
devanandaDon’t worry about the ‘filename “pxelinux.0″;’ line at the dhcpd.conf file, you can remove it.01:32
devananda???01:32
lifelessalso https://bugs.launchpad.net/ubuntu/+source/cobbler/+bug/82768101:32
devanandaI suspect U-boot and iPXE will be different drivers than PXE, yes01:33
devanandahowever, if ARM supports (normal) PXE, but requires different config than i686/x8664, then there is some problem ...01:33
devanandaI don't think that pxe_bootfile_name should be set per node. it's shared between all the nodes taht use the PXE driver today.01:34
lifelessso AIUI U-Boot lives in the firmware01:34
lifelessand honours most of the pxe stuff except it doesn't download pxelinux.0, it just implements the things that that reads directly01:35
devanandaif ARM requires a different boot file, I would suggest a separate option in the PXE driver, eg, arm_pxe_bootfile_name01:35
lifelessso you want the same config files, with the same format and same content01:35
devanandaand add a check to the pxe driver that changes its behavior if necessary, based on node metadata (eg, cpu arch)01:36
lifelessdevananda: what are the heuristics for determining when something should be config on disk vs config-in-the-driver vs config-in-the-ndeode?01:36
devanandalifeless: where the source of knowledge of that thing resides01:36
devanandaeg, in this case, knowledge of what boot file this node uses is, IMO, not something the cloud admin should know (therefor not something s/he will set when enrolling the node)01:37
lifelessok; can I bother you to correct me in that review?01:38
lifelessor would you like me to?01:38
devanandanp. if you agree, i'll write this up on the review01:39
lifelessI need to think more about that heuristic01:39
devanandagreat. I'd love a better one :)01:39
lifelessI concede your concern about per-node01:39
lifelessbut wonder if that may be because we don't have an arch abstraction01:39
devanandaah01:39
devanandait may01:39
lifelessI don't see how it really helps between disk-config and driver-config-in-db01:40
lifelessboth are set by the admin01:40
devanandawhat is "disk cnofig" ?01:40
lifelessfoo.conf01:40
devanandak01:40
devanandawhat is "driver config in db" ?01:40
lifelessI thought (IMBW) that driver objects could take parameters01:40
devanandaahh! nope01:40
devanandaor not yet, at least, and not in any current BPs01:41
lifeless /drivers/driver/properties-of-some-sort01:41
devanandawe can pass commands to drivers via vendor-passthru01:41
devanandabut there is no generalized way for drivers to store their own global config or status in a db01:41
lifelessok01:41
devanandaand doing that would require some synchronization between all the conductors that have said driver01:42
lifelessso if there was a vendor driver that should backend onto some API01:42
lifelessthat would require a config file setting today ?01:42
devanandayep01:42
lifelessok01:42
devanandaor per-node settings01:42
devanandalifeless: replied01:49
lifelessthanks01:49
devanandathank you. tis a good point to raise01:50
*** rongze has joined #openstack-ironic01:51
*** rongze has quit IRC01:51
*** rongze has joined #openstack-ironic01:52
*** rongze is now known as 36DABWUXY01:53
*** rongze has joined #openstack-ironic01:53
*** rongze is now known as 45PAAS1G101:53
*** 45PAAS1G1 has quit IRC01:57
*** jimjiang has quit IRC01:57
*** jimjiang_ has quit IRC01:57
*** jimjiang has joined #openstack-ironic01:58
*** 36DABWUXY has quit IRC02:04
*** rongze has joined #openstack-ironic02:04
*** bigjools has quit IRC02:08
*** rwsu has quit IRC02:12
*** michchap_ has joined #openstack-ironic02:15
*** michchap has quit IRC02:17
*** rwsu has joined #openstack-ironic02:26
*** prekarat has quit IRC02:31
*** rongze_ has joined #openstack-ironic02:35
*** rongze has quit IRC02:35
*** vkozhukalov has joined #openstack-ironic02:49
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Add prepare, clean_up, take_over methods to deploy  https://review.openstack.org/5889402:55
openstackgerritDevananda van der Veen proposed a change to openstack/ironic: Improve method to get list of active conductors  https://review.openstack.org/5979502:59
devanandaand on that note, I need to stop for the night! see ya'll tmw03:00
*** michchap has joined #openstack-ironic03:07
*** michchap_ has quit IRC03:09
*** michchap_ has joined #openstack-ironic03:20
*** michchap has quit IRC03:23
*** rloo has quit IRC03:46
dkehnlifeless: int eh review for https://review.openstack.org/#/c/60904/2/ironic/drivers/modules/pxe.py, when you say this should come out of the node driver dict, can you explain a bit more04:05
*** arata has joined #openstack-ironic04:25
devanandadkehn: he means, if you stored the pxe_bootfile_name per-node, it would be stored in the node.driver_info dict04:57
devanandadkehn: but see my response to his comments04:57
* devananda returns to being afk04:57
*** prekarat has joined #openstack-ironic05:01
*** rongze_ has quit IRC05:34
*** rongze has joined #openstack-ironic05:44
*** rongze has quit IRC05:50
*** rongze has joined #openstack-ironic05:59
*** michchap_ has quit IRC06:00
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/5962706:01
*** arata has left #openstack-ironic06:02
*** arata has joined #openstack-ironic06:07
*** vkozhukalov has quit IRC06:12
*** GheRivero has joined #openstack-ironic06:13
*** GheRivero has quit IRC06:13
*** GheRiver1 has quit IRC06:14
*** GheRivero has joined #openstack-ironic06:14
*** GheRivero is now known as Ghe_HPDiscover06:15
NobodyCamyou have to let us know how Discover is Ghe_HPDiscover06:16
*** jcooley_ has joined #openstack-ironic06:33
*** rongze has quit IRC06:35
*** rongze has joined #openstack-ironic06:38
*** urulama has joined #openstack-ironic07:27
*** ndipanov has quit IRC07:46
*** ndipanov has joined #openstack-ironic07:47
*** vkozhukalov has joined #openstack-ironic07:52
*** ifarkas has joined #openstack-ironic07:56
*** ndipanov has quit IRC07:58
*** rongze has quit IRC07:59
*** ndipanov has joined #openstack-ironic08:02
*** rongze has joined #openstack-ironic08:05
*** yfujioka has quit IRC08:11
*** prekarat has quit IRC08:25
*** prekarat has joined #openstack-ironic08:25
*** tatyana has joined #openstack-ironic08:29
openstackgerritFengqian.gao proposed a change to stackforge/pyghmi: Add BMC bridge request extension  https://review.openstack.org/6076808:31
*** jcooley_ has quit IRC08:45
*** ndipanov has quit IRC08:47
*** ndipanov has joined #openstack-ironic09:00
*** jistr has joined #openstack-ironic09:05
*** derekh has joined #openstack-ironic09:09
*** prekarat has quit IRC09:10
*** rongze has quit IRC09:11
*** rongze has joined #openstack-ironic09:15
*** lucasagomes has joined #openstack-ironic09:23
*** prekarat has joined #openstack-ironic09:25
*** romcheg1 has joined #openstack-ironic09:43
*** romcheg1 has left #openstack-ironic09:43
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Rework patch validation on Ports  https://review.openstack.org/5698410:10
*** max_lobur_afk is now known as max_lobur10:15
*** viktors has quit IRC10:33
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: GET /drivers to show a list of active conductors  https://review.openstack.org/6021610:33
*** viktors has joined #openstack-ironic10:48
*** rongze has quit IRC10:52
*** rongze has joined #openstack-ironic10:52
*** rongze has quit IRC10:55
*** arata has left #openstack-ironic11:01
*** yuriyz has joined #openstack-ironic11:09
*** prekarat has quit IRC11:16
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Handle DBDuplicateEntry on Ports with same address  https://review.openstack.org/5995511:21
*** mdenny has quit IRC11:23
*** jcooley_ has joined #openstack-ironic11:28
*** jcooley_ has quit IRC11:33
*** rongze has joined #openstack-ironic11:41
*** jimjiang has quit IRC11:46
*** jimjiang has joined #openstack-ironic11:47
*** rongze has quit IRC11:51
*** arata has joined #openstack-ironic11:56
*** rongze has joined #openstack-ironic11:56
*** arata has quit IRC11:57
*** lucasagomes is now known as lucas-hungry12:20
*** nosnos has quit IRC12:29
*** jcooley_ has joined #openstack-ironic12:29
*** jcooley_ has quit IRC12:35
romchegHi all12:42
romcheglucas-hungry: are you around?12:42
Haomengmorning all:)12:45
romchegMorning Haomeng12:52
romchegHow are you?12:52
Haomengromcheg: find, thank you:)12:53
Haomengromcheg: fine12:53
*** mdurnosvistov has joined #openstack-ironic12:56
*** linggao has joined #openstack-ironic13:17
*** rongze_ has joined #openstack-ironic13:29
*** jcooley_ has joined #openstack-ironic13:31
*** rongze has quit IRC13:33
*** jcooley_ has quit IRC13:36
*** openstackgerrit has quit IRC13:37
*** openstackgerrit has joined #openstack-ironic13:37
*** jdob has joined #openstack-ironic13:43
openstackgerritAlexandra Fomina proposed a change to openstack/python-ironicclient: Use improved exceptions functionality from Oslo  https://review.openstack.org/6052813:47
*** rongze_ has quit IRC13:48
openstackgerritAlexandra Fomina proposed a change to openstack/python-ironicclient: Import exceptions functionality from Oslo  https://review.openstack.org/6052813:50
*** pleia2 has quit IRC14:07
*** linggao has quit IRC14:08
*** pleia2 has joined #openstack-ironic14:09
*** jdob has quit IRC14:10
*** viktors has left #openstack-ironic14:11
openstackgerritAlexandra Fomina proposed a change to openstack/python-ironicclient: Import exceptions functionality from Oslo  https://review.openstack.org/6052814:17
*** rongze has joined #openstack-ironic14:23
*** jdob has joined #openstack-ironic14:23
*** rloo has joined #openstack-ironic14:27
*** jcooley_ has joined #openstack-ironic14:32
*** jcooley_ has quit IRC14:36
openstackgerritMikhail Durnosvistov proposed a change to openstack/ironic: Clean up duplicated change-building code in objects  https://review.openstack.org/6114314:56
NobodyCamgood morning iRonic15:03
*** jdob_ has joined #openstack-ironic15:13
*** jdob has quit IRC15:13
openstackgerritAlexandra Fomina proposed a change to openstack/python-ironicclient: Import exceptions functionality from Oslo  https://review.openstack.org/6052815:21
*** jdob_ has quit IRC15:29
*** jdob has joined #openstack-ironic15:29
*** rongze_ has joined #openstack-ironic15:39
*** rongze has quit IRC15:40
*** lucas-hungry is now known as lucasagomes15:42
lucasagomesromcheg, pong sorry for the delay15:42
lucasagomesmorning NobodyCam15:42
romchegMorning lucasagomes NobodyCam15:42
romchegI was about to drink some coffee15:43
romchegbrb15:43
NobodyCamgood morning lucasagomes15:43
*** rongze_ has quit IRC15:43
NobodyCammorning romcheg15:43
yuriyzMorning all15:46
NobodyCammorning yuriyz15:46
lucasagomesmorning yuriyz15:48
*** linggao has joined #openstack-ironic15:48
romcheglucasagomes, NobodyCam: Can you please take a look at https://review.openstack.org/#/c/53917/15:51
romcheg*at the last comment15:51
romchegWhat do you think?15:52
romchegimo we don't need that because afrer Ironic graduates incubation it will be added to general devstack jobs.15:53
*** urulama has quit IRC15:55
lucasagomesromcheg, right, I'm not expert in it, but I think it's grand as well. Also we could add those templates later on if needed15:55
*** lexx has joined #openstack-ironic15:57
NobodyCamromcheg: ya I dont see a issuse adding later15:58
NobodyCamwhich we will have to do i subspect15:58
romchegI planned to configure Ironic to be installed by default after nomination to core projects15:59
NobodyCam:) sweet15:59
lucasagomesNobodyCam, do you think that, in the future we should be able to unprovision a node?16:03
* lucasagomes almost finishing the deploy trigger, so I want to make sure we can incorporate future changes w/o changing the interface as well16:04
*** vkozhukalov has quit IRC16:05
*** jcooley_ has joined #openstack-ironic16:06
devanandamorning, all16:09
NobodyCamlucasagomes: ??? yes we will need to unporvision a node16:09
NobodyCammorning devananda16:09
lucasagomesNobodyCam, :)16:09
romchegMorning devananda16:09
NobodyCam:-p16:09
lucasagomesjust confirming, so I can do the right thing16:09
lucasagomesdevananda, morning16:09
yuriyzMorning devananda16:09
NobodyCam:)16:09
*** mdurnosvistov has quit IRC16:11
devanandaromcheg: there is some chicken & egg problem which you may be seeing -- folks want ironic tested before graduation, but it can't be added to main jobs until after graduation16:11
NobodyCambrb16:12
*** jistr has quit IRC16:16
*** jcooley_ has quit IRC16:16
yuriyzdevananda, now pyghmi driver supports only *power, *bootdev methods and consoles. Should it to be extended for sensors etc.? What do you think?16:17
*** jcooley_ has joined #openstack-ironic16:17
devanandayuriyz: yes16:17
yuriyzagree16:17
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add trigger to start provisioning the node  https://review.openstack.org/6116016:18
devanandalinggao: ping ^^16:18
*** lexx has quit IRC16:19
devanandayuriyz: i think Haomeng was starting on some of the ceilometer work16:20
devanandalucasagomes: i figured you'd comment on lots of things in my fix-api-autodoc patch :)16:23
lucasagomesdevananda, hehe16:23
devanandalucasagomes: I thought about just multi-lining everything, without adding any :param: lines, which would *just* fix the formatting but nothing else16:23
devanandai agree that updating all the param names to be more consistent is A Good Thing16:24
devanandajust didnt feel like doing that at what, 7pm last night ;)16:24
lucasagomesdevananda, I see, although it's a good oportunity to make the params more consistent16:24
lucasagomeshehe its grand16:24
*** ifarkas has quit IRC16:33
*** jcooley_ has quit IRC16:40
*** jcooley_ has joined #openstack-ironic16:40
*** jcooley_ has quit IRC16:45
devanandalucasagomes: on PatchType, what do you think of how often validate_attrs is being called?16:56
devanandacomments posted16:59
lucasagomesdevananda, sorry for the delay, lemme take a look at the comments17:00
*** max_lobur is now known as max_lobur_afk17:10
*** romcheg has quit IRC17:16
*** lexx has joined #openstack-ironic17:19
*** mdurnosvistov has joined #openstack-ironic17:23
*** blamar has joined #openstack-ironic17:35
*** rloo has quit IRC17:35
*** blamar has left #openstack-ironic17:36
NobodyCammorning walkies. bbiafm17:41
*** jcooley_ has joined #openstack-ironic17:48
*** jcooley_ has quit IRC17:50
*** anniec has joined #openstack-ironic17:51
devanandarloo, yuriyz: I just posted comment on breaking-resource-locks based on our discussions yesterday17:51
devanandagoing to untarget that blueprint17:51
*** lexx has quit IRC17:53
*** lexx has joined #openstack-ironic18:05
devanandaNobodyCam: https://review.openstack.org/#/c/59066/ could probably use your eyes some time18:08
* NobodyCam thats a look18:08
NobodyCamgah18:08
NobodyCamtakes even18:09
devanandalucasagomes: is martyn around?18:10
*** mattray has joined #openstack-ironic18:12
*** mattray has left #openstack-ironic18:13
lucasagomesdevananda, apparently he's offline already, can't find him on the rh channels as well18:13
devanandalucasagomes: ack, np.18:14
* devananda posts a comment on martyn's BP 18:14
NobodyCamdevananda: lucasagomes was there a bug for the pki token?18:15
devanandaNobodyCam: there was a blueprint for the "allow noauth in API service"18:15
devanandaNobodyCam: which I just closed, since taht approach was rejected18:15
devanandaI'm not aware of a bug specifically for the store-pki-token-on-disk18:16
lucasagomesNobodyCam, not that I know, you talking about https://review.openstack.org/#/c/59066/?18:16
* lucasagomes reviews that patch18:16
NobodyCamnope, was thinking it was Public API requires authentication bug: 122126318:17
*** vkozhukalov has joined #openstack-ironic18:18
NobodyCamlucasagomes: I +2'd but did not +a as you are also reviewing18:18
lucasagomesNobodyCam, thanks :) patch seems good, I'm just reading the previous comments to see if things were addressed18:19
devanandaHaomeng: replied to you on https://blueprints.launchpad.net/ironic/+spec/send-data-to-ceilometer18:22
openstackgerritA change was merged to openstack/ironic: Save PKI token in a file for PXE deploy ramdisk  https://review.openstack.org/5906618:25
NobodyCamanyone know how to run just one test with tox18:25
NobodyCam:-p18:25
devanandatestr run <test file or class name>18:26
devanandabut with tox? no18:26
lifelessNobodyCam: tox -- testnameregex18:26
*** hemnafk is now known as hemna18:26
NobodyCamahh -- TY lifeless18:26
*** jistr has joined #openstack-ironic18:26
lucasagomeslifeless, ahh great, didn't know that18:35
NobodyCamand it works :)18:36
NobodyCamlifeless' command not my test :-p18:36
NobodyCamheheheh18:36
lucasagomesI do like deva suggested, testr run ironic.tests...18:37
lucasagomesanyway, I'm done for the day :)18:37
lucasagomeshave a good night everyone18:37
lifelesslucasagomes: you don't need a full object path there18:37
lifelesslucasagomes: testr takes a regex18:37
NobodyCamhave a good night lucasagomes18:37
lucasagomesdevananda, I will address the patchtype problem you pointed out tomorrow morning, thanks18:37
lucasagomeslifeless, ahh, awesome. Lemme test it :D18:38
lucasagomesindeed it does! good stuff :) lifeless thank u again18:40
lifeless:)18:40
lucasagomesg'night everyone18:41
*** lucasagomes has quit IRC18:41
*** rloo has joined #openstack-ironic18:42
*** viktors1 has joined #openstack-ironic18:44
*** rloo has quit IRC18:44
*** rloo has joined #openstack-ironic18:45
*** rloo has quit IRC18:45
*** rloo has joined #openstack-ironic18:45
*** jcooley_ has joined #openstack-ironic18:47
*** jcooley_ has quit IRC18:49
*** tatyana has quit IRC18:49
*** jcooley_ has joined #openstack-ironic18:49
*** rloo has quit IRC18:51
*** rloo has joined #openstack-ironic18:52
*** jcooley_ has quit IRC18:53
*** rloo has quit IRC18:53
*** rloo has joined #openstack-ironic18:54
*** jcooley_ has joined #openstack-ironic18:54
*** anniec has quit IRC18:56
*** rloo has quit IRC18:58
*** rloo has joined #openstack-ironic18:58
*** anniec has joined #openstack-ironic18:59
*** jcooley_ has quit IRC19:02
*** jcooley_ has joined #openstack-ironic19:07
*** romcheg has joined #openstack-ironic19:15
romchegNobodyCam: Are you around19:15
*** jcooley_ has quit IRC19:18
romchegOr devananda :)19:20
devanandaromcheg: some discussion in #openstack-meeting that you would be interested in19:20
devanandare tempest and infra testing19:21
romchegWill join now19:21
*** vetalll has joined #openstack-ironic19:21
devanandaromcheg: tl;dr - mirantis folks also working on trove/tempest integration19:21
*** vetalll has quit IRC19:22
devanandaromcheg: and anteaya volunteered to coordinate conversations between all the folks who are just starting to dogpile on testing the younger projects (ironic, trove, etc)19:22
romchegdevananda: thanks19:22
*** vetalll has joined #openstack-ironic19:22
devanandaromcheg: conversation over there now is also relevant to us :)19:24
romchegdevananda: I just have an intern here who is trying to get familiar with openstack19:26
*** mordred has quit IRC19:26
*** mordred has joined #openstack-ironic19:26
romchegSo meet vetalll19:26
romchegI think he can work on https://bugs.launchpad.net/ironic/+bug/1259346 this bug19:26
devanandaawesome19:27
vetalllHi all.19:27
devanandaprobably best to take it one module at a time19:27
devanandaor module + relevant unit tests19:27
*** anniec has quit IRC19:27
*** anniec has joined #openstack-ironic19:29
vetallldevananda: Can you please point me to the samples of the  code  that need to be rework.19:29
devanandavetalll: all interfaces in here http://git.openstack.org/cgit/openstack/ironic/tree/ironic/drivers/base.py19:30
devanandavetalll: and all the classes which derive from them19:30
devanandavetalll: reason is that the TaskManager object (http://git.openstack.org/cgit/openstack/ironic/tree/ironic/conductor/task_manager.py) contains the node(s) and all relevant port(s) already19:30
devanandavetalll: so we do not need to pass the node separately19:31
devanandavetalll: one thing to keep in mind, which is not yet very consistent in the code, is that a TaskManager may contain references to one or more nodes.19:31
devanandathere are tricks to allow easy access for single-node tasks: http://git.openstack.org/cgit/openstack/ironic/tree/ironic/conductor/task_manager.py#n14419:32
vetalllok19:32
vetallldevananda: Thanks,  i will poke romcheg  or you for some help if I need it.19:33
devanandavetalll: great! and welcome to the team :)19:34
vetalllThanks.19:35
devanandavetalll: also, please assign the bug to yourself19:37
anteayadevananda romcheg no meeting for tempest, sdague would like contributors to be in #openstack-qa19:44
anteayaromcheg: please join #openstack-qa19:44
devanandavkozhukalov: ^19:44
*** lexx has quit IRC19:51
*** jcooley_ has joined #openstack-ironic19:53
*** viktors1 has quit IRC19:54
*** jcooley_ has quit IRC20:04
*** anniec has quit IRC20:10
*** jcooley_ has joined #openstack-ironic20:10
*** jcooley_ has quit IRC20:11
* NobodyCam looks for some food stuffs20:13
*** jcooley_ has joined #openstack-ironic20:13
*** jcooley_ has quit IRC20:16
*** jcooley_ has joined #openstack-ironic20:19
*** jistr has quit IRC20:25
*** vetalll has quit IRC20:30
*** jcooley_ has quit IRC20:31
*** jcooley_ has joined #openstack-ironic20:31
NobodyCambrb20:34
*** jcooley_ has quit IRC20:36
*** rloo has quit IRC20:41
*** rloo has joined #openstack-ironic20:41
*** rloo has quit IRC20:43
*** rloo has joined #openstack-ironic20:44
*** anniec has joined #openstack-ironic20:52
*** vetalll has joined #openstack-ironic20:53
devanandamordred: question trailing out of the TC discussion -- what does "Project must use OpenStack task, defect and design tracker(s)" mean?21:01
mordreddevananda: launchpad, or whatever else we may decide to move to21:01
mordreddevananda: such as - you can't store your crap in github just because you like it in your neckbeardhole21:02
devanandagotcha21:02
devanandathanks21:02
mordredI just said neckbeardhole21:02
*** romcheg has quit IRC21:03
*** anniec has quit IRC21:03
devanandayou did21:03
devanandaNobodyCam: something worth reading - https://review.openstack.org/#/c/59454/2/reference/incubation-integration-requirements21:03
devanandai added it to our agenda for next week21:04
* NobodyCam looks21:04
devanandawe need to grow our core review team21:04
NobodyCammordred: devananda: what is 'reasonably' in ** Project APIs should be reasonably stable21:07
devanandaheh21:08
mordredNobodyCam: they are guidelines - whatever the TC thinks they are21:08
devanandaNobodyCam: when we stop breaking our v1 API.21:08
devanandaand start creating v2 :)21:08
NobodyCamhehehehe21:08
*** kui has joined #openstack-ironic21:20
NobodyCamgood morning kui21:23
kuiNobodyCam: Good afternoon21:23
NobodyCam:)21:25
*** lexx has joined #openstack-ironic21:28
devanandaNobodyCam: ever gotten "Error: unsupported option i386." from step 1 on http://docs.openstack.org/developer/tripleo-incubator/devtest_seed.html21:29
kuiNobodyCam: The wiki page https://wiki.openstack.org/wiki/Ironic#Using_Disk_Image_Builder is the latest, right ?21:29
*** jcooley_ has joined #openstack-ironic21:30
NobodyCamdevananda: no21:30
NobodyCamkui: should be pritty up to date21:30
NobodyCamdevananda: can you paste the error?21:30
NobodyCamkui: found anything that looks like fun to jump in on?21:31
devanandaNobodyCam: that is the error21:34
kuiNobodyCam: No specific,  I want to get feet wet first, then I will find one.21:35
*** anniec has joined #openstack-ironic21:36
NobodyCamdevananda: from the sed command?21:36
NobodyCamor boot seed21:36
devanandaNobodyCam: build seed21:37
devanandaer, boot-seed-vm21:37
NobodyCamhttps://github.com/openstack/tripleo-incubator/blob/master/scripts/boot-seed-vm -a appers to be a valid option still21:37
devanandaright21:39
*** vetalll has quit IRC21:40
NobodyCamhttps://github.com/openstack/tripleo-incubator/blob/master/scripts/boot-seed-vm#L83 is there a "-a"21:41
*** kui has quit IRC21:41
*** kui has joined #openstack-ironic21:41
NobodyCamkui: if you have any questions please don't hesitate to ask!!21:42
kuiNobodyCam: Thanks  ;)21:44
NobodyCamdevananda: https://github.com/openstack/tripleo-incubator/blob/master/scripts/boot-seed-vm#L103-107 ... should catch anything with arch21:49
*** jdob has quit IRC21:52
*** anniec has quit IRC21:53
dkehndevananda: this is referring to the comment last night about the node.driver_info comment, and your comments, at this point does the pxe._get_pxe_bootfile_name(instance_uuid) need to be moved21:54
devanandadkehn: the comments were about whether the value of pxe_bootfile_name should be a CONF option or a node property21:57
devanandadkehn: for now, it should stay a CONF option21:57
dkehndevananda: agreed, then I'll leave the accessor where it is21:59
*** linggao has quit IRC22:01
devanandai need to step away for an hour or so... bbl22:02
*** anniec has joined #openstack-ironic22:02
*** mdurnosvistov has quit IRC22:09
*** rloo has quit IRC22:09
*** lexx_ has joined #openstack-ironic22:09
*** rloo has joined #openstack-ironic22:09
*** rloo has quit IRC22:10
*** rloo has joined #openstack-ironic22:11
*** lexx has quit IRC22:11
*** lexx_ has quit IRC22:15
*** datajerk has quit IRC22:19
*** lexx has joined #openstack-ironic22:27
*** epim has joined #openstack-ironic22:28
*** rloo has quit IRC22:31
*** rloo has joined #openstack-ironic22:31
*** jcooley_ has quit IRC22:34
*** jcooley_ has joined #openstack-ironic22:34
*** michchap has joined #openstack-ironic22:35
*** michchap has quit IRC22:35
*** derekh has quit IRC22:36
*** michchap has joined #openstack-ironic22:36
*** jcooley_ has quit IRC22:38
*** epim has quit IRC22:47
*** epim has joined #openstack-ironic23:02
*** jcooley_ has joined #openstack-ironic23:05
*** jcooley_ has quit IRC23:08
*** jcooley_ has joined #openstack-ironic23:08
kuiNobodyCam: hello23:09
NobodyCamhey kui :)23:09
kuiNobodyCam: I encounter the same error as deva, and I find the root cause in boot-seed-vm23:10
kui getopt -o hcna:o:s:b:e: -l use-existing-vm -n boot-seed-vm -- -a i386 ubuntu neutron-dhcp-agent23:10
kui getopt -o ihcna:o:s:b:e: -l use-existing-vm -n boot-seed-vm -- -a i386 ubuntu neutron-dhcp-agent23:10
kuiIn boot-seed-vm, there is a change in the latest commit, add "-i" opiton23:11
kuithe order of getopt output is changed,23:11
NobodyCamahh nice23:11
kuiIf you are not looking it, I will file a bug to fix it.23:12
*** jcooley_ has quit IRC23:12
NobodyCamhttps://github.com/openstack/tripleo-incubator/commit/444c63128d0354c97d66a3bb5b8e23ea39633bbb23:12
NobodyCamone sec23:13
NobodyCamlet me ask in OoO23:13
kuiYes, it is the commit inducing the error.23:14
kui-TEMP=`getopt -o hcna:o:s:b:e: -l use-existing-vm -n $SCRIPT_NAME -- "$@"`23:14
kui +TEMP=`getopt -o hcnai:o:s:b:e: -l use-existing-vm -n $SCRIPT_NAME -- "$@"`23:14
kuichange to following order will resolve it.23:14
kui getopt -o ihcna:o:s:b:e: -l use-existing-vm -n boot-seed-vm -- -a i386 ubuntu neutron-dhcp-agent23:14
kuiOK, waiting for OoO feedback.23:15
NobodyCamI've asked in OoO23:17
NobodyCamsee if any one is around23:17
NobodyCamwhat time is it there?23:17
devanandakui: confirmed that fixes it23:18
NobodyCamwb devananda23:18
NobodyCam:-p23:18
devanandakui: do you want to file bug & fix, or shall I?23:18
*** jcooley_ has joined #openstack-ironic23:18
kuidevananda: sure, I will do it now.23:19
*** epim has quit IRC23:19
NobodyCamawesome ty kui23:19
kui:)23:19
devanandakui: thanks. let me know the bug # and I'll add comment23:19
kuidevananda: OK23:19
*** jcooley_ has quit IRC23:22
*** jcooley_ has joined #openstack-ironic23:22
NobodyCambrb23:25
*** datajerk has joined #openstack-ironic23:27
NobodyCamback23:27
*** epim has joined #openstack-ironic23:28
kuidevananda: Bug is filed:  https://bugs.launchpad.net/tripleo/+bug/125974023:29
*** jcooley_ has quit IRC23:30
*** epim has quit IRC23:30
*** jcooley_ has joined #openstack-ironic23:30
devanandakui: bug is now triaged :)23:30
NobodyCamvery nice. detailed :)23:30
kui;)23:31
NobodyCamkui: as you list a fix in the report you should file a patch :)23:31
devanandapatch will probably get merged pretty quickly, since it will be a single line change23:32
kuiNobodyCam: I'm doing it23:32
NobodyCamgreat23:32
*** epim has joined #openstack-ironic23:34
*** epim has quit IRC23:34
*** jcooley_ has quit IRC23:35
*** jcooley_ has joined #openstack-ironic23:37
kuihttps://review.openstack.org/6127123:38
kuidevananda, NobodyCam, review is online23:38
devanandakui: thanks much23:39
*** datajerk has quit IRC23:41
openstackgerritJenkins proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/6097223:41
*** epim has joined #openstack-ironic23:43
*** jcooley_ has quit IRC23:44
*** jcooley_ has joined #openstack-ironic23:44
*** jcooley_ has quit IRC23:44
*** jcooley_ has joined #openstack-ironic23:45
openstackgerritJenkins proposed a change to openstack/python-ironicclient: Updated from global requirements  https://review.openstack.org/6098223:45
devanandakui: you'll need to rebase your patch to tripleo-incubator. it looks like a common dependency broke, hence the flood of Jenkins-proposed-a-change to all the projects23:49
kuidevananda: it the sphinx issue,   a fix is in review:  https://review.openstack.org/6127423:50
devanandayep23:50
*** michchap_ has joined #openstack-ironic23:53
*** jcooley_ has quit IRC23:53
*** jcooley_ has joined #openstack-ironic23:54
*** michchap has quit IRC23:57
*** jcooley_ has quit IRC23:58
*** jcooley_ has joined #openstack-ironic23:58

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