Perl 6 - the future is here, just unevenly distributed

IRC log for #gluster-dev, 2017-10-06

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

All times shown according to UTC.

Time Nick Message
00:55 vbellur joined #gluster-dev
00:57 nbalacha joined #gluster-dev
01:55 ilbot3 joined #gluster-dev
01:55 Topic for #gluster-dev is now Gluster Development Channel - https://www.gluster.org | For general chat go to #gluster | Patches - https://review.gluster.org/ | Channel Logs - https://botbot.me/freenode/gluster-dev/ & http://irclog.perlgeek.de/gluster-dev/
02:10 decayofmind joined #gluster-dev
02:12 kshlm joined #gluster-dev
02:13 lkoranda joined #gluster-dev
02:15 nh2 joined #gluster-dev
02:17 decayofmind joined #gluster-dev
02:47 amarts joined #gluster-dev
02:49 gyadav joined #gluster-dev
03:08 gyadav_ joined #gluster-dev
03:14 ppai joined #gluster-dev
03:26 gyadav__ joined #gluster-dev
03:44 nbalacha joined #gluster-dev
03:45 psony joined #gluster-dev
03:50 sanoj joined #gluster-dev
03:57 msvbhat_ joined #gluster-dev
03:57 msvbhat joined #gluster-dev
04:09 Saravanakmr joined #gluster-dev
04:13 atinmu joined #gluster-dev
04:14 itisravi joined #gluster-dev
04:49 skoduri joined #gluster-dev
04:55 dkhandel_ joined #gluster-dev
04:57 pkalever joined #gluster-dev
05:13 skumar joined #gluster-dev
05:15 msvbhat joined #gluster-dev
05:15 xavih joined #gluster-dev
05:16 amarts joined #gluster-dev
05:19 msvbhat_ joined #gluster-dev
05:19 gem joined #gluster-dev
05:24 Shu6h3ndu joined #gluster-dev
05:38 itisravi joined #gluster-dev
05:41 itisravi misc: Looks like slave22 has a problem now :-( . https://review.gluster.org/#/c/18430/ is failing regressions.
05:41 kdhananjay joined #gluster-dev
05:43 hgowtham joined #gluster-dev
05:53 luckyg joined #gluster-dev
05:56 aravindavk joined #gluster-dev
05:57 apandey joined #gluster-dev
06:03 rafi joined #gluster-dev
06:05 gobindadas joined #gluster-dev
06:06 humblec joined #gluster-dev
06:07 humblec joined #gluster-dev
06:14 jiffin joined #gluster-dev
06:15 susant joined #gluster-dev
06:24 skoduri joined #gluster-dev
06:30 apandey_ joined #gluster-dev
06:32 apandey_ joined #gluster-dev
06:33 itisravi joined #gluster-dev
06:46 atinmu joined #gluster-dev
06:54 rafi1 joined #gluster-dev
07:07 apandey__ joined #gluster-dev
07:07 rastar joined #gluster-dev
07:10 rafi1 joined #gluster-dev
07:18 hgowtham joined #gluster-dev
07:20 sanoj joined #gluster-dev
07:20 aravindavk joined #gluster-dev
07:33 skumar joined #gluster-dev
07:39 nh2 joined #gluster-dev
07:55 hgowtham joined #gluster-dev
07:56 misc mhh, itisravi left, but I have rebooted slave22, and the problem is likely a rogue patch
08:10 itisravi joined #gluster-dev
08:15 aravindavk joined #gluster-dev
08:32 Saravanakmr joined #gluster-dev
08:46 atinmu joined #gluster-dev
08:53 itisravi joined #gluster-dev
08:54 amarts joined #gluster-dev
09:14 skumar joined #gluster-dev
09:14 sanoj joined #gluster-dev
09:24 kotreshhr joined #gluster-dev
09:27 aravindavk joined #gluster-dev
09:27 amarts joined #gluster-dev
09:30 Saravanakmr joined #gluster-dev
10:26 Shu6h3ndu joined #gluster-dev
10:29 major joined #gluster-dev
10:30 skumar_ joined #gluster-dev
10:53 itisravi joined #gluster-dev
11:00 Shu6h3ndu joined #gluster-dev
11:38 kotreshhr left #gluster-dev
11:44 skumar__ joined #gluster-dev
11:46 Saravanakmr joined #gluster-dev
11:49 vbellur joined #gluster-dev
11:50 vbellur1 joined #gluster-dev
11:52 vbellur joined #gluster-dev
12:04 vbellur joined #gluster-dev
12:05 vbellur joined #gluster-dev
12:08 amarts joined #gluster-dev
12:42 susant left #gluster-dev
12:42 atinmu joined #gluster-dev
12:53 rraja joined #gluster-dev
12:53 rraja_ joined #gluster-dev
12:56 jstrunk joined #gluster-dev
13:09 kkeithley misc, nigelb, dkhandel_: jenkins clang-scan and cppcheck are running on the master branch right? Only the master branch?
13:16 atinmu joined #gluster-dev
13:23 gyadav__ joined #gluster-dev
13:35 * misc verify
13:38 msvbhat joined #gluster-dev
13:39 msvbhat_ joined #gluster-dev
13:40 misc kkeithley: yep
13:40 * misc ponder on opening a bug for adding a skin to jenkins that link to the repo where the jobs are defined
13:41 vbellur1 joined #gluster-dev
13:44 shyam joined #gluster-dev
13:59 kkeithley ndevos: I don't consider  the %init_enable glustereventsd as sneaking it in. It was there, but as restart, which is bad
14:02 kkeithley it's all fixing the systemd/systemctl bits
14:06 kkeithley that aside, I'm not exactly loving the "fix."  I'd love to kill all the init.d cruft
14:11 ndevos kkeithley: I thought about using the systemd macros in the scriptlets, and define macros with the same name for the init.d stuff, that makes stripping them out easier
14:12 kkeithley yeah, I was thinking about that too. It seemed like a more invasive change, but maybe that's really the best solution
14:14 ndevos it would be more work, but it surely would be nice to have it
14:14 ndevos we can then strip the init.d stuff from the .spec in fedora and c7 already
14:19 ndevos oh, we've fixed some memory leaks over the last few releases, but we're also adding more :-( http://termbin.com/mt9e
14:20 vbellur joined #gluster-dev
14:21 kkeithley :-/
14:34 psony joined #gluster-dev
14:37 vbellur joined #gluster-dev
14:42 rafi1 joined #gluster-dev
14:46 ppai rastar, please feel free to add further info to https://github.com/gluster/glusterd2/issues/382
14:49 susant joined #gluster-dev
14:55 aravindavk joined #gluster-dev
15:00 pkalever joined #gluster-dev
15:11 msvbhat joined #gluster-dev
15:11 wushudoin joined #gluster-dev
15:16 msvbhat joined #gluster-dev
15:18 msvbhat_ joined #gluster-dev
15:28 xavih joined #gluster-dev
15:43 psony joined #gluster-dev
15:43 major joined #gluster-dev
16:03 kkeithley ndevos: I'm open to other names. What would you like instead of systemd_{start,stop} ?   service_{start,stop} ?
16:15 ndevos kkeithley: just systemd_post etc. and not define anything if systemd is used, define a systemd_post macro for init.d usage
16:16 kkeithley ugh, then we have to scribble %if %else %endif for the start, stop, and install that aren't there now
16:19 kkeithley this is why I thought this change was too invasive
16:21 psony joined #gluster-dev
16:35 ndevos kkeithley: no, I do not think that is needed, only one '%if %{!?with_systemd}' and define the systemd_post macro as 'chkconfig enable ... \%{nil}' or something
16:37 kkeithley I'm not following you. Latest patch defines systemd_{post,preun,postun_with_restart} only for el6
16:37 kkeithley _and_
16:40 kkeithley foo_{start,stop,install} for both systemd and el6.
16:41 kkeithley there are two foo_start(), one foo_install, and two foo_stop()
16:41 kkeithley not that many in the grand scheme of things.
16:42 kkeithley but each would need %if systemd $dosystemd %else $doinitd %endif
16:42 kkeithley ugly scribbling
16:43 ndevos why is foo_start() needed if systemd_post() takes care of that already? Cant the initd variand for systemd_post() not do that too?
16:43 ndevos and yeah, I definitely would not want '%if systemd $dosystemd %else $doinitd %endif' kind of things
16:43 kkeithley foo_start() is not the same as systemd_post()
16:44 ndevos the whole idea is to use standard systemd_*() macros, and have a in implementation for those to use for init.d systems
16:44 kkeithley right.
16:45 ndevos well, foo_start() is probably not acceptible according to the Fedora packaging anyway, so should that really exist?
16:45 kkeithley systemd_{post,preun,postun_with_restart}
16:45 major joined #gluster-dev
16:46 kkeithley look closely at exactly how foo_start() is used. Only on upgrade, to run glusterd to fixup vol files where necessary
16:46 ndevos yeah, use those, and stick to them, no systemd_start() or other extension
16:46 kkeithley then ugly %if %else %endif scribbling
16:46 kkeithley is required
16:47 kkeithley foo_{stop,install} don't violate fedora packaging guidelines
16:48 ndevos oh right, if that is only the upgrade path... hmm
16:49 ndevos why not just kill the process from the .pid file and systemd_post() to start it again?
16:51 kkeithley systemd_post() does not start it. it's `systemctl --no-reload preset ...`
16:51 kkeithley and see --preset-mode=
16:52 kkeithley afaict that does not start it.
16:52 ndevos if it was enabled it should start it, right? just like 'service cond-restart ...' I think
16:53 ndevos but if you really prefer the foo_start() and foo_stop() bit, I can accept that, but then it should be called service_* as it is not a macro from the systemd RPM
16:53 kkeithley postun_with_restart() does that, not post()
16:55 kkeithley It's either define systemd & init.d macros in one place with one %if %else %endif
16:55 kkeithley Or have five places with %if %else %endif
16:55 ndevos one place!
16:56 * ndevos needs to drop off, its 7pm here
16:56 ndevos have a good weekend!
16:58 kkeithley have a nice weekend. Play a lot of squash
16:59 ndevos only tonight, I'll go to London and annoy sprabhu for a couple of days there
17:01 kkeithley I wish I could just pop over to London or Paris for a weekend. Well, I can, but it's not a one hour "hop"
17:03 ndevos I'm flying back on Wednesday, so it's sort of an extended weekend+work trip ;-)
17:03 kkeithley systemd_postun_with_restart() could be used instead foo_start(). But it might seem strange to use systemd_postun*() in %post based on the name.
17:03 kkeithley It does say it's for package upgrade, not uninstall.
17:04 kkeithley then could eliminate the foo_start()
17:04 ndevos yes, that looks weird, but %postun gets executed when the previous version is uninstalled (on updating to a newer version)
17:04 kkeithley indeed
17:05 ndevos I'm not sure about the order of %post and %postun though, but that may not be important anyway
17:05 major joined #gluster-dev
17:05 ndevos also, I'm REALLY leaving now, cya!
17:05 kkeithley ttyl
17:43 vbellur joined #gluster-dev
18:02 vbellur joined #gluster-dev
18:45 rastar joined #gluster-dev
19:35 rafi joined #gluster-dev
21:06 msvbhat joined #gluster-dev
22:06 shyam joined #gluster-dev
22:06 msvbhat joined #gluster-dev
22:06 msvbhat_ joined #gluster-dev
23:08 msvbhat joined #gluster-dev
23:08 msvbhat_ joined #gluster-dev

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