--- Log opened vr aug 08 00:00:35 2014 00:02 -!- danken: has quit [Quit: Leaving.] 00:07 -!- apuimedo: has quit [Ping timeout: 272 seconds] 00:12 -!- osvoboda: has quit [Quit: ChatZilla 0.9.90.1 [Firefox 31.0/20140722064054]] 00:24 -!- phoracek: has quit [Quit: WeeChat 0.4.3] 01:25 -!- bala (purple): has joined #vdsm 03:52 -!- adahms_ (Andrew Dahms): has joined #vdsm 03:53 -!- adahms: has quit [Ping timeout: 264 seconds] 04:02 -!- fabiand: has quit [Ping timeout: 260 seconds] 04:24 -!- nsoffer: has quit [Read error: Connection reset by peer] 04:50 -!- gpadgett: has quit [Ping timeout: 250 seconds] 04:55 -!- gpadgett (Greg Padgett): has joined #vdsm 05:23 -!- adahms_: has quit [Ping timeout: 264 seconds] --- Log closed vr aug 08 05:33:06 2014 --- Log opened vr aug 08 05:33:15 2014 05:33 -!- ewoud: has joined #vdsm 05:33 -!- Irssi: #vdsm: Total of 17 nicks: 05:33 !wilhelm.freenode.net: why register and identify? your IRC nick is how people know you. http://freenode.net/faq.shtml#nicksetup 05:33 -!- Irssi: Join to #vdsm was synced in 39 secs 05:35 -!- adahms_ (Andrew Dahms): has joined #vdsm 06:10 -!- gpadgett: has quit [Quit: Leaving] 06:30 -!- adahms_: has quit [Ping timeout: 240 seconds] 06:43 -!- adahms_ (Andrew Dahms): has joined #vdsm 07:31 -!- fabiand (Fabian Deutsch): has joined #vdsm 07:38 -!- wgao: has quit [Remote host closed the connection] 08:02 -!- sbonazzo (purple): has joined #vdsm 08:13 -!- xaviern (Xavier): has joined #vdsm 08:23 -!- #vdsm xaviern: has quit [Ping timeout: 272 seconds] 08:28 -!- mode/#vdsm: by ChanServ 08:28 -!- danken (purple): has joined #vdsm 08:31 -!- mskrivanek_away is now known as mskrivanek 08:33 -!- pkliczew (Piotr Kliczewski): has joined #vdsm 08:36 -!- fromani (Francesco Romani): has joined #vdsm 08:38 -!- adahms_: has quit [Quit: Leaving] 08:46 -!- xaviern (Xavier): has joined #vdsm 09:11 -!- #vdsm xaviern: has quit [Quit: WeeChat 0.4.3] 09:12 -!- xaviern (Xavier): has joined #vdsm 09:28 -!- fabiand: has quit [Quit: Verlassend] 09:31 -!- fabiand (Fabian Deutsch): has joined #vdsm 09:52 -!- apuimedo (antoni): has joined #vdsm 09:53 -!- osvoboda (osvoboda): has joined #vdsm 10:02 -!- osvoboda: has quit [Remote host closed the connection] 10:17 -!- osvoboda (osvoboda): has joined #vdsm 10:22 -!- #vdsm ybronhei: has quit [Quit: Leaving.] 10:29 -!- ybronhei (purple): has joined #vdsm 10:49 <@danken> fromani: could you help me verify http://gerrit.ovirt.org/31135 by using it when you verify virt-related patches? I'd love to know if you can survive with less info 10:50 < fromani> danken: sure 10:52 <@danken> fromani: on another matter: I see no problem in dumping "hasSysinfo" in favor of hasVDSMChannels - EVERYWHERE. in master, 3.5, and 3.4. 10:52 <@danken> mskrivanek: ^^^^ 10:53 < fromani> danken: mskrivanek I have zero problems with this in the immediate future 10:54 < fromani> danken: for long term, I'll drop my http://gerrit.ovirt.org/#/c/31211/ , but I'd still like to leverage libvirt domain metadata to let VDSM distinguish its own VMs. A much simpler signature would do the job. What do you think about this? 10:56 -!- bala: has quit [Quit: Leaving.] 10:58 < mskrivanek> danken: we would want to move all the .recovery and any other suff into libvirt's xml. so this a good start. For 3.6 we can rely on it completely (one we get EL 6.6, basically) 10:59 <@danken> fromani: hmmm, depending on new signature is problematic when considering upgrade and migration of VMs started on old vdsms. 10:59 <@danken> mskrivanek: moving .recovery to metadata is a noble cause, but it should not be confused with the current urgent bug 10:59 -!- danken is now known as danken_afk 11:00 < fromani> danken_afk: mskrivanek sure, let's just start having the hasVDSMChannels fix everywhere 11:02 < mskrivanek> danken_afk: sure. those are separate efforts. Just putting the id into metadata is a good start anyway - we dont have to rely on it right away 11:06 < evilissimo> it would anyway require so more effort to move it into the metadata, since we need to ensure that the new vdsm is not choking on migrations without the metadata 11:17 <@danken_afk> fromani: ok, so please post a simpler http://gerrit.ovirt.org/#/c/31213/1/vdsm/clientIF.py on master: just replace current sysinfo-based code with hasVDSMChannels. 11:18 < fromani> danken_afk: sure, in a few minutes. 11:44 < fromani> danken_afk: http://gerrit.ovirt.org/#/c/31241/2 11:55 -!- aravindavk (Aravinda): has joined #vdsm 12:06 -!- timothy_ (Timothy Asir): has joined #vdsm 12:20 -!- aravindavk: has quit [Ping timeout: 272 seconds] 12:23 -!- danken_afk: has quit [Ping timeout: 245 seconds] 12:23 -!- mode/#vdsm: by ChanServ 12:23 -!- danken (purple): has joined #vdsm 12:28 -!- apahim: has quit [Quit: Leaving] 12:33 -!- aravindavk (Aravinda): has joined #vdsm 12:41 < fromani> danken: actually I hadn't understood you want to fully replace the sysinfo check with the hasVDSMChannels *entirely*. My bad, but now I got it, at last. But this new check isn't somehow weaker than the former (although hacky?) 12:41 < fromani> the former was hacky I mean 12:53 <@danken> fromani: I can't think of any facet of its being weaker 12:54 <@danken> fromani: hmm 12:54 < fromani> danken: theoretically the channels can be disabled by the user, we expose this tunable 12:55 < fromani> the vmchannel option 12:55 <@danken> fromani: oh dear, you are right. vmchannel can be set to false. 12:55 < fromani> ok, this is highly unlikely, but still... 12:56 <@danken> fromani: yeah, you are right. 12:56 <@danken> fromani: why do we have that configurable 12:56 * danken going to dig git. 12:58 <@danken> it was intentional, but only for debugging. 12:58 <@danken> ebf896b8a1fd5ce8b0ffb37abce4bbf760b3c07b 13:00 <@danken> fromani: this means that we cannot trust vmchannel existence AT ALL. Unless you change its documentation that it's for debug purposes only, and that it must not be used in production. 13:01 <@danken> actually, now that we have hooks and all... we don't need this configurable. let's kill it. in its own patch. 13:02 < fromani> danken: I guess we don't have a real choice here besides that... It's ok to do the update the doc in the same patch (31241)? 13:02 < fromani> plus a follow up patch, master only, to kill the tunable 13:03 <@danken> fromani: you are more careful than I.. probably because I know who introduced this configurable and why 13:03 <@danken> fromani: but yes, your approach is appealing. 13:04 -!- mskrivanek is now known as mskrivanek_away 13:04 -!- mskrivanek_away is now known as mskrivanek 13:04 < fromani> danken: ok, then, updating 31241 to 1. document the tunable should be used for debug only 2. switch isVDSM to use hasVDSMChannels only 13:05 < fromani> in a few minutes 13:05 < fromani> danken: I had my fair share of hard lessons in the past :) 13:10 < apuimedo> pkliczew: thanks for the edit host patch! 13:10 < pkliczew> apuimedo: no problem, let me know if you need anything else 13:10 < apuimedo> ;-) 13:12 < apuimedo> fabiand: do you have access to a machine that reproduces https://bugzilla.redhat.com/show_bug.cgi?id=1128033 13:12 < apuimedo> ? 13:26 -!- apahim (Amador Pahim): has joined #vdsm 13:47 -!- aravindavk: has quit [Quit: Leaving] 14:18 -!- osvoboda: has quit [Quit: ChatZilla 0.9.90.1 [Firefox 31.0/20140722064054]] 14:23 -!- apahim: has quit [Read error: Connection reset by peer] 14:28 -!- nsoffer (Nir Soffer): has joined #vdsm 14:35 < fabiand> apuimedo, hey no 14:35 < apuimedo> :( 14:35 -!- mskrivanek is now known as mskrivanek_away 14:38 -!- apahim (Amador Pahim): has joined #vdsm 14:43 -!- mskrivanek_away is now known as mskrivanek 14:51 < fabiand> apuimedo, I think we need to wait for next week, so QE can give some more details 14:51 -!- fabiand: has left #vdsm ["Verlassend"] 14:52 -!- fabiand (Fabian Deutsch): has joined #vdsm 14:53 < apuimedo> ok 14:53 -!- nsoffer: has quit [Read error: Connection reset by peer] 14:59 -!- santosh_7724_: has quit [Remote host closed the connection] 15:04 -!- mpolednik (Martin Polednik): has joined #vdsm 15:04 -!- mskrivanek is now known as mskrivanek_away 15:05 -!- bala (purple): has joined #vdsm 15:07 -!- mskrivanek_away is now known as mskrivanek 15:16 <@danken> apuimedo: it's smells like you are already aware of https://bugzilla.redhat.com/show_bug.cgi?id=1128140 15:17 < apuimedo> danken: I noticed it in one of the machines I was debugging 15:17 < apuimedo> but I didn't look at why 15:18 < apuimedo> I made a patch for this and thought it was merged 15:18 < apuimedo> odd 15:18 < apuimedo> 9ab9fc13311851595c31ddd9cbbc4d490362fe09 15:18 < apuimedo> that was the patch 15:23 < apuimedo> danken: http://gerrit.ovirt.org/#/c/29312/ 15:23 < apuimedo> you could merge it, yaniv didn't ;-) 15:23 < apuimedo> I'll add the new bug to the bug-url of the 3.5 patch 15:25 < apuimedo> done 15:25 < apuimedo> and the bug is on post 15:27 <@danken> apuimedo: doh, a missing backport! 15:27 < apuimedo> danken: not a missing one, a 15:27 < apuimedo> not-merged one 15:28 <@danken> ybronhei: pretty please - merge this one http://gerrit.ovirt.org/#/c/29312/ 15:28 <@danken> it's in master by now. 15:28 -!- bala: has quit [Ping timeout: 240 seconds] 15:31 -!- gpadgett (Greg Padgett): has joined #vdsm 15:35 -!- timothy_: has quit [Ping timeout: 272 seconds] 15:45 -!- bala (purple): has joined #vdsm 15:58 <@danken> fromani: any chance you can verify your recovery patch both on x86 and ppc? 15:58 -!- bala: has quit [Ping timeout: 272 seconds] 15:59 < fromani> danken: partially. I manually patched a 3.4 rhevm using hunk from my master patch and run the code on a PPC host, and it worked as expected 16:00 < fromani> danken: the problem is on the ppc host I have access on I cannot rebuild VDSM 16:01 < fromani> (lack of development packages) 16:04 < fromani> danken: let me do a try again 16:15 -!- bala (purple): has joined #vdsm 16:27 -!- mskrivanek is now known as mskrivanek_away 16:29 -!- pkliczew: has quit [Ping timeout: 250 seconds] 16:32 -!- #vdsm xaviern: has quit [Ping timeout: 260 seconds] 16:37 -!- mpolednik: has quit [Ping timeout: 260 seconds] 16:51 -!- mskrivanek_away is now known as mskrivanek 16:56 -!- xaviern (Xavier): has joined #vdsm 17:01 -!- fabiand: has quit [Quit: Verlassend] 17:08 -!- sbonazzo: has quit [Quit: Leaving.] 17:08 -!- #vdsm xaviern: has quit [Ping timeout: 264 seconds] 17:20 -!- bala: has quit [Read error: Connection reset by peer] 17:23 -!- apahim: has quit [Quit: Leaving] 17:24 -!- apahim (Amador Pahim): has joined #vdsm 18:17 < fromani> danken: do I have answered to your question here? http://gerrit.ovirt.org/#/c/28036/ (not urgent I'm just dusting off my patches) 18:19 <@danken> fromani: yes. but I understood it myself (after a too-long thinking). what's needed is a fix to the commit message 18:21 < fromani> danken: ok 18:36 <@danken> fromani: apuimedo: enjoy your weeken, I'm gone. 18:37 < apuimedo> danken: enjoy this shorter weekend ;-) 18:44 -!- fromani: has quit [Quit: Leaving] 19:00 -!- mskrivanek is now known as mskrivanek_away 19:21 < ybronhei> danken: in 19:25 -!- mpolednik (Martin Polednik): has joined #vdsm 19:33 -!- mpolednik: has quit [Ping timeout: 272 seconds] 19:48 -!- mpolednik (Martin Polednik): has joined #vdsm 20:02 -!- nsoffer (Nir Soffer): has joined #vdsm 20:05 -!- nsoffer: has quit [Read error: Connection reset by peer] 20:09 -!- nsoffer (Nir Soffer): has joined #vdsm 20:16 -!- nsoffer: has quit [Read error: Connection reset by peer] 20:38 -!- nsoffer (Nir Soffer): has joined #vdsm 21:20 < ybronhei> danken: let me know what you think about - https://bugzilla.redhat.com/show_bug.cgi?id=1111935 .. sorry for all the mess 21:21 -!- mpolednik: has quit [Ping timeout: 240 seconds] 21:46 -!- apahim: has quit [Remote host closed the connection] 21:48 -!- apahim (Amador Pahim): has joined #vdsm 21:55 -!- nsoffer: has quit [Read error: Connection reset by peer] 22:00 -!- mpolednik (Martin Polednik): has joined #vdsm 22:01 -!- nsoffer (Nir Soffer): has joined #vdsm 22:20 -!- #vdsm dougsland_: has quit [Ping timeout: 240 seconds] 22:51 -!- mpolednik: has quit [Ping timeout: 255 seconds] 23:02 -!- nsoffer: has quit [Ping timeout: 240 seconds] 23:02 -!- mpolednik (Martin Polednik): has joined #vdsm 23:03 -!- nsoffer (Nir Soffer): has joined #vdsm --- Log closed za aug 09 00:00:36 2014