Perl 6 - the future is here, just unevenly distributed

IRC log for #gluster-dev, 2017-05-03

| Channels | #gluster-dev index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:52 Humble joined #gluster-dev
01:27 vbellur joined #gluster-dev
01:49 ilbot3 joined #gluster-dev
01:49 Topic for #gluster-dev is now Gluster Development Channel - http://gluster.org | For general chat go to #gluster | Patches - http://review.gluster.org/ | Channel Logs - https://botbot.me/freenode/gluster-dev/ & http://irclog.perlgeek.de/gluster-dev/
02:14 purpleidea joined #gluster-dev
02:14 purpleidea joined #gluster-dev
02:35 vinurs joined #gluster-dev
03:05 prasanth joined #gluster-dev
03:30 nbalacha joined #gluster-dev
03:41 amarts joined #gluster-dev
03:52 itisravi joined #gluster-dev
03:55 atinm joined #gluster-dev
03:58 riyas joined #gluster-dev
04:02 Humble joined #gluster-dev
04:17 ppai joined #gluster-dev
04:17 sanoj joined #gluster-dev
04:20 gem joined #gluster-dev
04:27 gyadav joined #gluster-dev
04:30 skumar joined #gluster-dev
04:32 bwerthmann joined #gluster-dev
04:37 karthik_us joined #gluster-dev
04:44 jiffin joined #gluster-dev
05:04 itisravi joined #gluster-dev
05:11 amarts joined #gluster-dev
05:21 aravindavk joined #gluster-dev
05:27 ndarshan joined #gluster-dev
05:29 hgowtham joined #gluster-dev
05:30 mchangir joined #gluster-dev
05:35 itisravi joined #gluster-dev
05:36 nishanth joined #gluster-dev
05:39 pkalever joined #gluster-dev
05:44 skoduri joined #gluster-dev
05:44 sona joined #gluster-dev
05:45 msvbhat joined #gluster-dev
05:52 ankitr joined #gluster-dev
06:00 kdhananjay joined #gluster-dev
06:04 gyadav_ joined #gluster-dev
06:08 gyadav__ joined #gluster-dev
06:11 dlambrig joined #gluster-dev
06:13 Saravanakmr joined #gluster-dev
06:20 rejy joined #gluster-dev
06:23 nishanth joined #gluster-dev
06:31 Humble joined #gluster-dev
06:50 vbellur joined #gluster-dev
06:50 Saravanakmr joined #gluster-dev
07:10 ashiq joined #gluster-dev
07:18 rafi joined #gluster-dev
07:41 ccha joined #gluster-dev
07:41 ccha hello
07:42 ccha the glusterfs chan is limited ?
07:42 ccha can't join the chan
07:42 aravindavk joined #gluster-dev
07:42 ccha #glusterfs: Cannot join channel (+l) - channel is full, try again later
07:42 ccha oh it's #gluster
07:52 Humble joined #gluster-dev
08:24 ndevos https://review.gluster.org/13762 for 3.11 still needs a review (+merge) from someone
08:27 ndevos skoduri: when you do a backport for the glfs_xreaddirp_r() patch, put it on top of https://review.gluster.org/17153
08:27 rastar joined #gluster-dev
08:27 ndevos skoduri: that one corrects gfapi.aliases/.map and will conflict with your backport
08:35 skoduri ndevos, okay thanks
08:39 ndevos hmm, pranithk merged the halo change, and that adds junk through a --with-fbextras configure option :-/
08:41 ndevos kshlm: if that ^ gets backported it needs some corrections in the .spec and stuff too...
08:41 sona joined #gluster-dev
08:42 kshlm ndevos, Taking a note of that.
08:44 skoduri ndevos, kshlm ...why do we change the  glfs_ipc version back to 3.7 for every release..why not fix it in master itself?
08:44 ankitr joined #gluster-dev
08:45 kshlm skoduri, That is a 4.0 feature. So it remains on master as is.
08:45 ndevos skoduri: jbr uses it, and that gets removed from releases too
08:45 skoduri kshlm, then it should ideally be removed from 3.* releases right?
08:45 kshlm skoduri, That's what we do.
08:46 kshlm ndevos, What are the other stuff you mentioned (.spec and stuff) that need changes?
08:46 skoduri kshlm, oh I see an old version of it..sorry hadn't noticed
08:47 ndevos kshlm: anything that uses BUILD_FBEXTRAS or IPV6_DEFAULT
08:49 ndevos kshlm: IPV6_DEFAULT is only useable with a special fbtirpc forked project, and that is not available outside of facebook (I think)
08:49 kshlm ndevos, Yes. That's why it isn't merged yet.
08:49 ndevos kshlm: the Halo patch merged it
08:49 kshlm What?
08:49 ndevos yeah....
08:49 kshlm How come?
08:50 kshlm I had a -2 on the review.
08:50 ndevos not on https://review.gluster.org/16177
08:51 ndevos I only noticed something changed the nfs and packaging bits, and I just checked what it was, finding this weird --with-fbextras and IPV6_DEFAULT
08:51 kshlm I had -2 on https://review.gluster.org/16228 which is the IPv6 patch.
08:51 ankitr joined #gluster-dev
08:52 ndevos sorry, but at least parts of that got sneaked in :-/
08:53 ndevos and in a much uglier way too
08:55 kshlm Ah, crap.
08:55 kshlm I didn't know it was included in the halo patch as well.
08:56 ndevos yes, me neither
08:56 ndevos I also dont think it is required for the halo feature itself, so it should have been stripped out in the 1st place
08:58 kshlm Just checked the patch. It's just some bits that creeped in.
08:59 kshlm An extra rpc bind function to bind on ipv6 interfaces. Can be removed without affecting the rest of HALO.
09:01 ndevos yes, I think so too
09:03 jiffin joined #gluster-dev
09:05 aravindavk joined #gluster-dev
09:05 sona joined #gluster-dev
09:15 ndevos kshlm: is there anything blocking from your side to merge https://review.gluster.org/13762 ?
09:15 * ndevos is very tempted to just merge it himself
09:19 skoduri joined #gluster-dev
09:23 kshlm ndevos, Since it's on master, it can be merged. But I'd rather not do it, I've not reviewed it and don't have context. If you were looking for reviews on the glusterd changes, atinm has already provided the +1.
09:24 nbalacha joined #gluster-dev
09:24 nbalacha joined #gluster-dev
09:26 amarts joined #gluster-dev
09:27 itisravi joined #gluster-dev
09:30 kotreshhr joined #gluster-dev
09:33 ndevos kshlm: well, I guess it has sufficient +1's, I'll just +2 and merge it, jiffin can then do the backport to 3.11
09:35 jiffin1 joined #gluster-dev
09:40 ndevos kshlm: have you guys already started release notes for 3.11? where can I send details to get included?
09:41 aravindavk joined #gluster-dev
09:43 karthik_us joined #gluster-dev
09:44 jiffin ndevos++ for merging https://review.gluster.org/13762
09:44 rastar kdhananjay++
09:45 kdhananjay rastar++
09:45 rastar no bot here?
09:45 amarts joined #gluster-dev
09:46 ndevos jiffin: if you send the backport for that (use bug 1447597
09:46 ndevos ), I will do the backport for the related shell scripts
09:47 jiffin ndevos: sure
09:47 ndevos JoeJulian: glusterbot has left?
09:56 msvbhat joined #gluster-dev
09:58 sanoj joined #gluster-dev
10:00 sanoj joined #gluster-dev
10:12 rastar atinm: nbalacha ping, regarding https://bugzilla.redhat.com/show_bug.cgi?id=1447608
10:12 rastar I need someone from dht/glusterd team to fix this so me to make 3.10.2 release.
10:20 skoduri joined #gluster-dev
10:25 itisravi_ joined #gluster-dev
10:38 atinm kdhananjay, would you be able to send a quick fix for this?
10:39 amarts joined #gluster-dev
10:41 kdhananjay atinm: nope. im working on sth else for d/s
10:48 jiffin ndevos: done https://review.gluster.org/#/c/17159/
10:59 atinm rastar, I'm sending the fix, but this needs to be reviewed by kdhananjay & DHT folks
11:00 rastar atinm: thanks!
11:00 kdhananjay atinm: sure, will do the needful.
11:11 rraja joined #gluster-dev
11:13 atinm kdhananjay, rastar : https://review.gluster.org/17160
11:13 atinm this is on master
11:15 amarts joined #gluster-dev
11:16 ndevos jiffin: mark it Verified=+1, or are you waiting for something?
11:18 kdhananjay atinm: will tkae a look
11:30 jiffin ndevos: smoke test
11:31 chawlanikhil24 joined #gluster-dev
11:32 msvbhat joined #gluster-dev
11:34 jiffin1 joined #gluster-dev
11:52 skoduri joined #gluster-dev
12:03 atinm rastar, I've replied to your comments, have a look
12:03 kshlm Has the maintainers meeting been cancelled for today?
12:05 ndevos kshlm: yes, I think vbellur mentioned that too many maintainers are attending Red Hat Summit
12:07 kshlm Oh. Cool then.
12:07 itisravi joined #gluster-dev
12:08 atinm kdhananjay, rastar : I see a rebalance start force option, so do we want this command to bypass the validation?
12:08 amarts joined #gluster-dev
12:08 atinm what do others think?
12:08 rastar atinm: kdhananjay said that patches which have gone in 3.10 might have already fixed the issue
12:09 rastar atinm: if someone wants to try/test that, it would be useful to have a force option
12:09 atinm If we think from a different perspective, data loss is definitely not something which we'd even be allowing with force option
12:13 kdhananjay rastar: atinm if users using 'force' without knowledge of the consequences is an issue, i can make a src tar ball with the cli patch reverted for users who could help verify that the fix works
12:14 atinm kdhananjay, I am not saying this is wrong, what I am asking is a consensus as this is quite debatable :)
12:18 kdhananjay atinm: hmm .. users would know best
12:19 atinm JoeJulian, what's your feedback here?
12:22 jiffin1 joined #gluster-dev
12:27 karthik_us joined #gluster-dev
12:38 aravindavk joined #gluster-dev
12:47 skoduri joined #gluster-dev
13:12 sona joined #gluster-dev
13:13 atinm rastar, I refreshed the patch, rebal start force would bypass this validation now
13:39 jiffin1 joined #gluster-dev
13:41 vbellur joined #gluster-dev
13:42 vbellur joined #gluster-dev
13:42 ankitr joined #gluster-dev
13:43 vbellur joined #gluster-dev
13:48 sanoj joined #gluster-dev
13:57 misc so, am I the only one with trouble to push to gerrit ?
13:58 misc (trying to refresh my patches)
14:08 misc ok so reading the log, I am not the only on with trouble on that repo
14:08 misc kshlm: I see you are affected as well
14:09 JoeJulian atinm: I think "force" can be honored. The mailing list argument about force being used by default is, imho, a poor one. That shouldn't be done.
14:10 kshlm misc, I was able to push just fine earlier today
14:10 misc kshlm: yeah, bit I see
14:11 misc [2017-05-03 12:51:14,274 +0000] 8f361cc9 kshlm a/1000019 gerrit.query.--format=JSON.--patch-sets.change:16228 0ms 270240ms killed
14:11 misc kshlm: so you had issue as well ?
14:11 kshlm That, I was trying to fetch a review with git-review
14:12 kshlm It had hung for a while.
14:12 misc oh
14:12 misc my patch did pass finally
14:12 misc [2017-05-03 14:12:29,784 +0000] f0fb0fd0 mscherer a/1000638 git-receive-pack./glusterfs.git 382374ms 850ms 0
14:12 kshlm I killed the command and retried, then it worked.
14:13 sona joined #gluster-dev
14:13 misc 6 minutes...
14:14 kshlm Didn't know git would wait that long
14:14 misc git is much more patient than me :)
14:18 sanoj joined #gluster-dev
14:37 msvbhat joined #gluster-dev
14:37 kotreshhr left #gluster-dev
14:37 atinm JoeJulian, we don't default to force
14:41 aravindavk joined #gluster-dev
14:50 nbalacha joined #gluster-dev
14:59 wushudoin joined #gluster-dev
15:08 JoeJulian atinm: no, the email thread suggested that some users do. That's poor practice and doesn't need to be guarded against.
15:24 vinurs joined #gluster-dev
15:33 gem joined #gluster-dev
15:54 sanoj joined #gluster-dev
15:57 ankitr joined #gluster-dev
17:07 atinm joined #gluster-dev
17:18 ndevos JoeJulian: will glusterbot come back one day?
17:19 JoeJulian Gah! got sidetracked.
17:19 ndevos none might be better than two, but having one is quite useful
17:19 ndevos np :)
17:20 glusterbot joined #gluster-dev
17:28 glusterbot joined #gluster-dev
17:42 jiffin joined #gluster-dev
17:42 rraja joined #gluster-dev
18:16 gem joined #gluster-dev
18:24 Humble joined #gluster-dev
18:59 ankitr joined #gluster-dev
19:18 foster joined #gluster-dev
22:21 rraja_ joined #gluster-dev
22:22 ccha2 joined #gluster-dev
22:26 kaushal_ joined #gluster-dev
22:28 tdasilva- joined #gluster-dev
22:52 owlbot joined #gluster-dev
23:23 Acinonyx joined #gluster-dev

| Channels | #gluster-dev index | Today | | Search | Google Search | Plain-Text | summary