Friday, 2014-01-17

*** rloo has quit IRC00:15
*** rloo has joined #openstack-ironic00:15
*** michchap_ has quit IRC00:43
*** michchap has joined #openstack-ironic00:43
* NobodyCam wanders afk00:50
*** harlowja is now known as harlowja_away00:50
*** harlowja_away is now known as harlowja01:08
*** nosnos has joined #openstack-ironic02:04
*** rloo has quit IRC03:53
*** coolsvap has joined #openstack-ironic04:00
*** harlowja is now known as harlowja_away04:01
*** coolsvap_away has joined #openstack-ironic04:20
*** coolsvap has quit IRC04:21
*** coolsvap_away is now known as coolsvap04:21
*** vkozhukalov has joined #openstack-ironic04:21
*** anniec has joined #openstack-ironic04:32
*** mrda has quit IRC05:03
*** mdurnosvistov has joined #openstack-ironic05:11
*** mrda has joined #openstack-ironic05:13
*** anniec has quit IRC05:53
*** mdurnosvistov has quit IRC06:32
*** vkozhukalov has quit IRC06:52
*** mrda has quit IRC07:01
openstackgerritJenkins proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/6503207:02
*** tatyana has joined #openstack-ironic07:35
*** tatyana has quit IRC07:37
GheRiveromorning Ironic07:40
*** romcheg has joined #openstack-ironic07:50
*** romcheg has quit IRC08:06
*** tatyana has joined #openstack-ironic08:09
*** vkozhukalov has joined #openstack-ironic08:27
agordeevmorning Ironic08:34
*** coolsvap has quit IRC08:35
*** coolsvap has joined #openstack-ironic08:35
Haomengagordeev: morning agordeev, GheRivero: )08:43
agordeevHaomeng: morning :)08:44
Haomengagordeev: :)08:46
*** mdenny has quit IRC09:01
*** mdenny has joined #openstack-ironic09:01
*** vkozhukalov has quit IRC09:03
*** jistr has joined #openstack-ironic09:14
*** vkozhukalov has joined #openstack-ironic09:18
*** ndipanov has joined #openstack-ironic09:20
*** max_lobur_afk is now known as max_lobur09:23
*** romcheg has joined #openstack-ironic09:35
*** mdurnosvistov has joined #openstack-ironic09:45
mdurnosvistovGood morning all! :)09:50
*** martyntaylor has joined #openstack-ironic10:00
*** martyntaylor has quit IRC10:01
*** martyntaylor has joined #openstack-ironic10:03
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Sync apiclient and strutils from Oslo  https://review.openstack.org/6052810:09
openstackgerritAndrey Kurilin proposed a change to openstack/python-ironicclient: Reuse Resource from oslo  https://review.openstack.org/6292110:10
openstackgerritDmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests  https://review.openstack.org/6741510:19
openstackgerritDmitry Shulyak proposed a change to openstack/ironic: Fix duplicate uuid error fo pxe tests  https://review.openstack.org/6713010:19
openstackgerritGhe Rivero proposed a change to openstack/ironic: Fix non-unique tftp dir instance_uuid  https://review.openstack.org/6685810:21
Haomengmdurnosvistov: morning:)10:22
*** mrda has joined #openstack-ironic10:29
openstackgerritZhiQiang Fan proposed a change to openstack/ironic: Replace assertTrue with explicit assertIsInstance  https://review.openstack.org/6742010:32
*** vkozhukalov has quit IRC10:46
*** romcheg has quit IRC10:49
*** romcheg has joined #openstack-ironic10:51
*** nosnos has quit IRC10:53
*** vkozhukalov has joined #openstack-ironic10:58
*** lucasagomes has joined #openstack-ironic11:00
*** derekh has joined #openstack-ironic11:05
*** martyntaylor has quit IRC11:09
*** mrda has quit IRC11:26
openstackgerritDmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests  https://review.openstack.org/6741511:30
*** martyntaylor has joined #openstack-ironic11:57
*** vkozhukalov has quit IRC12:00
*** vkozhukalov has joined #openstack-ironic12:12
*** romcheg has quit IRC12:14
*** romcheg has joined #openstack-ironic12:15
*** vkozhukalov has quit IRC12:17
*** vkozhukalov has joined #openstack-ironic12:32
*** derekh has quit IRC12:46
openstackgerritDmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests  https://review.openstack.org/6741513:02
*** coolsvap has quit IRC13:09
*** jdob has joined #openstack-ironic13:22
*** jbjohnso has joined #openstack-ironic13:49
*** zul has quit IRC13:52
*** zul has joined #openstack-ironic13:55
*** Guest50254 has quit IRC14:04
*** jistr has quit IRC14:33
*** jistr has joined #openstack-ironic14:37
*** matty_dubs|gone is now known as matty_dubs14:45
*** coolsvap has joined #openstack-ironic14:53
*** russellb is now known as rustlebee14:55
NobodyCamgood morning Ironicsays the waiting for coffee14:59
lucasagomesmorning NobodyCam :)15:01
romchegMorning lucasagomes NobodyCam and everyone else!15:01
* lucasagomes thinks about having a cofee as well :D15:01
lucasagomesromcheg, morning15:01
NobodyCammorning lucasagomes romcheg ... TGIF!!!15:03
lucasagomesNobodyCam, oh yea15:03
max_loburmorning/evening All!15:03
lucasagomesNobodyCam, going for some beers tonight?15:04
NobodyCamhey hey max_lobur :) morning over here15:04
lucasagomesmax_lobur, morning :)15:04
mdurnosvistovmorning all! :)15:04
NobodyCambeen on a cider kick of late15:04
max_loburlucasagomes, FYI I proposed a quick fix for race bug https://bugs.launchpad.net/ironic/+bug/125991015:04
max_loburfunctionally it's almost equal to your patch15:04
max_loburstill leaves some place for race15:05
max_loburbut improves the feedback15:05
lucasagomesmax_lobur, thanks, I will take a look (I'm in a google hangout meeting now)15:05
max_loburok :)15:05
lucasagomesyea, the race is actually more annoying than I thought15:05
lucasagomescause I hit it twice already15:05
lucasagomes:(15:05
openstackgerritZhiQiang Fan proposed a change to openstack/ironic: Replace assertTrue with explicit assertIsInstance  https://review.openstack.org/6742015:08
*** anniec has joined #openstack-ironic15:08
GheRiveromorning all15:08
NobodyCammorning GheRivero15:08
lucasagomesmorning GheRivero15:08
* lucasagomes meeting finished, going to grab something to eat15:09
*** lucasagomes is now known as lucas-hungry15:09
*** anniec_ has joined #openstack-ironic15:11
*** anniec has quit IRC15:13
*** anniec_ is now known as anniec15:13
*** jistr has quit IRC15:16
*** jistr has joined #openstack-ironic15:25
*** jistr has quit IRC15:45
openstackgerritDmitry Shulyak proposed a change to openstack/ironic: alembic with initial migration and tests  https://review.openstack.org/6741515:59
*** lucas-hungry is now known as lucasagomes16:01
*** ifarkas has quit IRC16:04
openstackgerritMartyn Taylor proposed a change to openstack/ironic: Remove POST calls in tests for resource creation  https://review.openstack.org/6467416:05
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Fix API inconsistence when changing node's states  https://review.openstack.org/6750616:09
*** aignatov has joined #openstack-ironic16:12
*** tatyana has quit IRC16:19
*** sirusm has joined #openstack-ironic16:20
*** anteaya is now known as tired16:21
*** tired is now known as very_tired16:22
NobodyCamvery_tired: LOL16:26
lucasagomeshah16:27
*** aignatov has quit IRC16:28
max_loburfrom tired to very_tired in a one minute :)16:32
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Use oslo.rootwrap library instead of local copy  https://review.openstack.org/6638916:32
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add missing "Filters" section to the ironic-images.filters  https://review.openstack.org/6639016:32
*** romcheg has quit IRC16:33
lucasagomesman zuul is pretty slow today16:36
max_loburnot only today16:36
lucasagomes+1 :P16:36
lucasagomesit's being a while16:36
*** dshulyak has quit IRC16:45
*** coolsvap has quit IRC16:57
lucasagomesmax_lobur, ping... hmm was thinking about the quick fix you proposed, hmm but idk, I don't think the real fix would take too long to land16:59
*** coolsvap has joined #openstack-ironic16:59
lucasagomesI mean if we get to talk, me, you, devananda, NobodyCam and others interested we might reach a conclusion quickly16:59
lucasagomesthat fix is fine, but there's still a race condition there and we would have to add tests too17:00
lucasagomesand then revert it after17:00
openstackgerritGhe Rivero proposed a change to openstack/ironic: Fix non-unique pxe driver 'instance_name'  https://review.openstack.org/6565717:00
lucasagomesI will add the topic to the meeting again, but not on the fft17:00
*** vkozhukalov has quit IRC17:00
lucasagomesto guarantee we are going to talk about it on the next meeting17:01
mdurnosvistovFolks if you have time please look at this patch https://review.openstack.org/#/c/64336/ Need discuss how to do this patch better or we can discuss it is on meeting.17:01
lucasagomesmdurnosvistov, +117:02
lucasagomesI think we can discuss it on the meeting17:02
lucasagomesI'm ok with the change, I think it's better to have only one consistent way to access the attributes of the objects17:02
lucasagomesI like the idea of the patch17:02
max_loburlucasagomes, ok I agree that it's better to work real fix, let's discuss on meeting. Or I'll try to ping Everyone some time before the meeting so we can start earlier. Because I think for others it will take some time to look over all the solution17:03
very_tiredhello ironic17:03
lucasagomesmax_lobur, +117:03
very_tiredwell actually I went from tired to very tired between about 1:30am last night at 5am last night17:04
very_tiredyay jet lag17:04
very_tireddevananda: you back yet?17:04
max_loburalso I wanted to ask if you have some ideas of how the intent lock will look like17:04
max_loburvery_tired hello :)17:04
very_tiredcheers17:04
max_loburlucasagomes, because without intent lock your solution still leaves a room for race, inbetween to AMQM calls, like you mentioned17:05
lucasagomesmax_lobur, yea17:05
*** viktors has left #openstack-ironic17:05
lucasagomesI didn't give it a deep thought17:05
lucasagomesbut I was thinking about some flag in the acquire that would tell the NodeManager to not unlock that node at the finally17:05
lucasagomesand as we have a routing algorithm now, that guarantees that another rpc call17:06
lucasagomeswould land in the same conductor as before17:06
lucasagomeson the next acquire we would say that the conductor will reacquire the node17:06
*** sirusm has quit IRC17:06
lucasagomesor something like that17:07
max_loburhmm, interesting17:07
max_loburdo you think there are some other features that need intent lock mechanism?17:07
lucasagomesmax_lobur, yes17:07
lucasagomesthe power and provision state changes17:08
lucasagomesand the vendor passthru17:08
lucasagomesthe vendor passthru fist validates the data inputed by the user in a sync way17:08
lucasagomesand then do an async call to do the real work17:08
lucasagomesso I can see these 3 things using that kinda of lock17:08
max_loburso these are a places for the race condition17:09
max_loburI mean if we work out a thread approach for example17:09
max_loburwe will cover them too17:09
lucasagomesmax_lobur, yes17:09
max_loburI wanted to ask if there are others17:09
max_loburthat need intent lock not due to race problem17:10
lucasagomesmax_lobur, hmm not that I'm aware of17:10
max_loburwe'll need to ask everyone else17:11
lucasagomesI mean the intent/two phase lock, would be create to solve race condition problems like that17:11
max_loburyea, I see17:11
lucasagomesso idk if there's any other use case for that17:11
max_loburI thought if there will be other places, we will definitely should choose your solution, but otherwise it might look overcomplicated for this17:12
lucasagomesmax_lobur, yea... it's a bit more complicated to understand having a two phase lock17:13
max_loburok, let's discuss on Monday once everyone here :)17:13
lucasagomes:)17:13
lucasagomesmax_lobur, but implementation wise, I don't think that would be a big change17:13
lucasagomescause things like the routing the rpc messages is already in place17:13
lucasagomesdone by the hashing algorithm17:14
*** jdob has quit IRC17:14
max_loburyes17:14
lucasagomesmax_lobur, but I'm ok with both approach17:14
lucasagomeslet's see what other people thinks17:14
max_loburme too :)17:14
*** anniec has quit IRC17:15
*** matty_dubs is now known as matty_dubs|lunch17:15
devanandavery_tired: yep, back yesterday17:17
devanandalucasagomes: hi!17:17
max_loburhi devananda !17:17
lucasagomesdevananda, hey :)17:22
NobodyCammorning devananda17:28
devanandamorning, all17:28
devanandawhat do you guys think of https://bugs.launchpad.net/nova/+bug/1195073 ?17:29
devanandait was targetd to i2 a while back. we don't have the same loopingCall architecture as nova did, though17:29
devanandathe pxe_deploy_timeout option is still defined in drivers.modules.pxe, but it's not used anywhere today17:30
devanandai think the only way we could implement that is with a periodic_task in conductor.manager17:30
devanandaand a db field to track the time a deploy was started17:31
devanandaand move the timeout out of pxe and into conductor17:31
NobodyCamhumm17:32
*** mdurnosvistov has quit IRC17:33
GheRiveroyeah, i would move it outside pxe and into a periodic_task17:35
GheRiverowe only use pxe deployments for now, but maybe in the future there can be other types, which would need also a timeout17:35
devanandak, i've commented on the bug and untargeted it17:35
lucasagomesdevananda, hmm thinking, yea it's something we would need to fix17:35
NobodyCamwould that allow different drivers to have different timeouts17:36
devanandanope. but that's an interesting point17:37
NobodyCamie ilo side load may take longer then pxe ( <- is only for example)17:37
devanandaheh. well, i think the ilo team would prefer if its faster than pxe... but yes, valid point17:37
NobodyCamonly a example :-p17:37
max_loburwe may start some waiting thread in pxe after deploy is started. The thread will check deploy state after some time, if it's not yet completed then It will reject it. This should be less costly than periodic17:38
max_loburIf I got the problem correctly17:38
* max_lobur pays too much attention to threads :D17:38
lucasagomesheh17:39
lucasagomesalso, the periodic task/thread would start at deploy() or _continue_deploy()17:40
lucasagomes?17:40
lucasagomesor what about using a signal instead of a thread/periodic call?17:42
lucasagomeshttp://docs.python.org/dev/library/signal.html#signal.alarm17:43
lucasagomeshmm, as it says its unix only :/17:44
GheRiverodo we support non linux systems? :p17:45
lucasagomesheh idk in the future17:46
NobodyCampython does run on windows17:46
lucasagomeshttp://docs.python.org/2/library/signal#example17:46
lucasagomesthe example is just what we want17:47
very_tireddevananda: woohoo17:47
*** martyntaylor has quit IRC17:50
*** harlowja_away is now known as harlowja17:51
*** lsmola_ has quit IRC17:54
*** matty_dubs|lunch is now known as matty_dubs17:54
devanandamax_lobur: oh, i meant to ask re: your threading patch - how does this work with eventlet / greenthreads?17:55
max_loburdevananda, good question, I assume it should be fine but need to investigate deeper17:57
*** ndipanov has quit IRC18:00
max_loburAs I know the main feature of eventlet is that it effectively manages relation between greenthread / python thread (OS thread), e.g. several greenthreads can execute within one python thread and so on18:01
max_loburso it should work exactly like usual thread18:02
max_loburjust less expensice18:02
max_lobur*expensive18:02
*** jdob has joined #openstack-ironic18:04
*** hemnafk is now known as hemna_18:05
NobodyCampost bbt walkies... bbiafm18:07
devanandamax_lobur: it doesn't work "exactly like real thread"18:08
devanandamax_lobur: for one, if a greenthread blocks on something, the whole process is blocked18:08
devanandamax_lobur: eventlet only switches between greenthreads at certain calls. low-level IO is not caught, for example,a nd can still block the whole process if called directly18:10
devanandamax_lobur: the main python process is still executing only in a single CPU18:10
max_loburyou right18:18
max_loburI have shallow understanding of greenthreads18:18
*** mdurnosvistov has joined #openstack-ironic18:24
max_loburdevananda, I gotta run, I'll investigate this further this weekend, I also will ask folks around on Monday to know what are the pitfalls18:26
devanandamax_lobur: thanks. have a good weekend :)18:27
max_loburyou too :) Bye Everyone18:28
NobodyCamnight max18:28
NobodyCam:)18:28
mdurnosvistovmax_lobur: bye! :)18:28
lucasagomesmax_lobur, g'night18:29
*** max_lobur is now known as max_lobur_afk18:30
NobodyCamdevananda: at first glance it appers that nova is setting up neutron for dhcp but not setting the pxe boot options18:53
*** vkozhukalov has joined #openstack-ironic18:53
NobodyCamI an nova boot(ing) nodes they are getting dhcp addresses but not pxe booting18:54
NobodyCams/an/am/18:54
devanandaNobodyCam: i'll take a look after this call18:57
NobodyCamjust fyi: http://paste.openstack.org/show/P1ovbUxdeXcavsNkVzvE/18:59
devanandaNobodyCam: and is the vm powering on and all that?19:00
NobodyCamyes gets a ip from dhcp19:01
devanandasweet19:01
NobodyCamjust no pxe19:01
NobodyCamI think that makes logical sense19:01
NobodyCamas we moved that logic in to pxe19:01
devanandawell19:02
devanandanova should be setting the dhcp boot option the first time19:02
NobodyCams/we/dkehn/19:02
devanandai think19:02
NobodyCamyes I am getting a dhcp address19:02
devanandaactually, maybe it shouldnt19:02
NobodyCamjust not the pxe boot options19:02
devanandanova//neutron should be setting the IP19:02
devanandabut different drivers may not need the DHCPBOOT option19:02
devanandaso taht shouldn't be done by nova for everything19:03
NobodyCamya19:03
devanandaour pxe driver should do that19:03
devanandayep19:03
NobodyCamthats why I think it makes sense19:03
NobodyCamI get ip gateway and what nots19:03
NobodyCambut with out pxe options it never tries to pxe boot19:04
*** mdurnosvistov has quit IRC19:04
devanandaright19:04
NobodyCamget its ip and then continues to attempt to boot19:04
* NobodyCam goes to review dkehn's patch19:04
lucasagomesNobodyCam, pxelinux.0 file is in the /tftpboot directory?19:05
NobodyCamlucasagomes: root@undercloud-undercloud-rrjizted2tys:/opt/stack/nova# ls /tftpboot/19:06
NobodyCamc04423ef-d9a1-4dcf-9fdf-2584b1956dd9  map-file  master_images  pxelinux.0  pxelinux.cfg  token-c979cf21-a267-4362-ab94-a4f6d947a0d219:06
lucasagomestripleo is using neutron to pxe boot already, will take a look as well to see if they are doing something extra19:06
NobodyCamlucasagomes: yes but we are not setting the pxe options in neutron yet19:07
devanandaNobodyCam: https://github.com/openstack/nova/blob/master/nova/virt/baremetal/driver.py#L51719:07
devanandaNobodyCam: looks like that's not in your driver19:07
devanandaand that is what tells Nova to set the DHCP BOOT options19:07
devanandaso we can plug in there19:07
devanandaor we can plug in to our pxe driver19:07
NobodyCamI pulled that as I thought pxe driver would be doing it19:09
NobodyCamI will add it back in19:10
NobodyCamit really needs to be in both19:10
NobodyCamthe issue we had we getting the correct conductor ip in nova as a recall19:10
NobodyCamactually yes that was it19:10
devanandaahh19:11
NobodyCamwe will need to conductor to set those options as htey need the conductors ip ... If my memory is correct19:11
devanandawhich we still dont expose19:11
* devananda jumps into conference, bbl19:12
NobodyCamwe talked a 'hack' to set a conf option with a single conductor ip19:12
NobodyCamack19:12
* NobodyCam makes a begal19:12
openstackgerritA change was merged to openstack/python-ironicclient: Drop python2.5 suport in ironicclient.common.http  https://review.openstack.org/6651319:18
openstackgerritA change was merged to openstack/python-ironicclient: Run unittest with python 3.3  https://review.openstack.org/6714619:18
NobodyCamwow that took awhile19:20
*** jistr has joined #openstack-ironic19:24
*** aignatov has joined #openstack-ironic19:28
NobodyCamoh monday is a holiday19:28
NobodyCamhere in the US19:28
lucasagomesNobodyCam, hah yea it took almost 4 hours to merge that19:30
lucasagomesmr jenkins is really slow today19:30
NobodyCamya, the crush befor the cutoff19:30
NobodyCambrb19:38
*** romcheg has joined #openstack-ironic19:38
*** aignatov has quit IRC19:44
EmilienMlucasagomes: gate is currently down afik19:44
lucasagomesEmilienM, :(19:46
NobodyCam:(19:51
*** aignatov has joined #openstack-ironic19:53
NobodyCamGheRivero: still around this friday (night) for you?20:01
lucasagomesI'm done for the day20:03
lucasagomeshave a good night everybody20:03
lucasagomesnight NobodyCam devananda20:03
lucasagomesenjoy the weekend20:03
NobodyCamhave a great weekend lucasagomes20:03
NobodyCamlucasagomes: monday is a holiday here20:03
lucasagomesNobodyCam, right, so no meeting?20:03
NobodyCami'll be around for the meeting but prob about it20:03
lucasagomesok20:03
lucasagomesfair enough :)20:03
NobodyCam:)20:04
lucasagomesenjoy the extended weekend then20:04
lucasagomesdrink a pint for me on monday :D20:04
NobodyCam:) will do20:04
lucasagomesttyl20:04
*** lucasagomes has quit IRC20:04
devanandai'll be around for the meeting too20:20
devanandaglobal team and al lthat ...20:20
NobodyCam:)20:21
*** jistr has quit IRC20:40
GheRiveroNobodyCam: jsut lurking from time to time20:41
*** aignatov has quit IRC20:41
NobodyCamhey ghe one sec20:42
*** anniec has joined #openstack-ironic20:42
*** aignatov has joined #openstack-ironic20:42
NobodyCamlooks like https://review.openstack.org/#/c/66858/ will need rebase20:42
*** aignatov has quit IRC20:42
NobodyCamonce the gate comes back up. we can try and merge it and the dep.20:43
GheRiveroagh! true. Will do this weekend... if jenkins is not in holidays :)20:44
NobodyCam:) thank you GheRivero :)20:45
*** anniec has quit IRC20:51
*** anniec has joined #openstack-ironic21:29
NobodyCamdevananda: https://review.openstack.org/#/c/66677/21:33
*** romcheg has quit IRC21:51
*** rustlebee is now known as russellb21:52
*** derekh has joined #openstack-ironic21:54
*** anniec has quit IRC21:54
*** matty_dubs is now known as matty_dubs|gone21:59
NobodyCamahh starBucks!22:05
*** jbjohnso has quit IRC22:05
*** jcooley_ has joined #openstack-ironic22:10
*** anniec has joined #openstack-ironic22:19
*** vkozhukalov has quit IRC22:24
*** anniec has quit IRC22:24
*** very_tired is now known as anteaya22:28
*** anniec has joined #openstack-ironic22:42
*** jdob has quit IRC22:43
NobodyCamwb anteaya22:48
NobodyCamdevananda: reply sent to Alexander22:48
NobodyCambrb22:48
devanandaanteaya: good morning :)22:49
NobodyCamand back22:53
*** mrda has joined #openstack-ironic22:53
*** mrda has quit IRC22:57
*** anniec has quit IRC23:00
*** mdurnosvistov has joined #openstack-ironic23:06
*** anniec has joined #openstack-ironic23:11
*** jcooley_ has quit IRC23:32
*** jcooley_ has joined #openstack-ironic23:39
*** jcooley_ has quit IRC23:44
NobodyCambrb23:47
*** shadower has quit IRC23:55

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