Re: [Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Vladislav Bogdanov

Hi Dejan,

19.01.2015 16:30, Dejan Muhamedagic wrote:

Hi Vladislav,


[...]
Fix transition start detection.

--- a/modules/constants.py  2014-12-22 08:48:26.0 +
+++ b/modules/constants.py  2014-12-22 13:07:43.945077805 +
@@ -272,7 +272,7 @@
  # r.group(3) file number
  transition_patt = [
  # transition start
-crmd.* do_te_invoke: Processing graph ([0-9]+) .*derived from 
(.*/pe-[^-]+-(%%)[.]bz2),
+pengine.* process_pe_message: Calculated Transition ([0-9]+): 
(.*/pe-[^-]+-(%%)[.]bz2),


Do you know when this changed?


Original message (from do_te_invoke) was downgraded into the 'info' 
priority a long ago (probably during that Andrew's massive logging 
cleanup), while process_pe_message' one still remains at the 'notice' 
level. First my patch has 2012-12-26 as its date (for crmsh-1.2.4), so 
the change was done before that. iirc process_pe_message's message was 
always there, both messages were printed before that cleanup.




The reason I'm asking is that crmsh tries to support multiple
pacemaker versions, so I'm not sure if we can just replace this
pattern.


Make tar follow symlinks.

--- a/modules/crm_pssh.py   2013-08-12 12:52:11.0 +
+++ b/modules/crm_pssh.py   2013-08-12 12:53:32.666444069 +
@@ -170,7 +170,7 @@
  dir = /%s % r.group(1)
  red_pe_l = [x.replace(%s/ % r.group(1), ) for x in pe_l]
  common_debug(getting new PE inputs %s from %s % (red_pe_l, node))
-cmdline = tar -C %s -cf - %s % (dir, ' '.join(red_pe_l))
+cmdline = tar -C %s -chf - %s % (dir, ' '.join(red_pe_l))


Just curious: where did you find links in the PE input
directories?


Ahm, you know, systems are s different around a world ;)
And system administrators sometimes want to do weird things ;)

Actually that one is specific to my diskless clusters, but it wont hurt 
anyways.




And many thanks for the patches!

Cheers,

Dejan



Best,
Vladislav

___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems



___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems


Re: [Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Vladislav Bogdanov

19.01.2015 16:27, Kristoffer Grönlund wrote:

Vladislav Bogdanov bub...@hoster-ok.com writes:


Hi Kristoffer,


there are two patches, one for crmsh and one for parallax.
They make history commands work.


Thanks!

I have created a pull request with the patches for crmsh here:

https://github.com/crmsh/crmsh/pull/77


Thank you very much, I do not have enough will to make myself ride that 
web-2.0 tools ;)




Cheers,
Kristoffer



Best,
Vladislav






___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

Re: [Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Dejan Muhamedagic
Hi Vladislav,

 [...]
 Fix transition start detection.
 
 --- a/modules/constants.py  2014-12-22 08:48:26.0 +
 +++ b/modules/constants.py  2014-12-22 13:07:43.945077805 +
 @@ -272,7 +272,7 @@
  # r.group(3) file number
  transition_patt = [
  # transition start
 -crmd.* do_te_invoke: Processing graph ([0-9]+) .*derived from 
 (.*/pe-[^-]+-(%%)[.]bz2),
 +pengine.* process_pe_message: Calculated Transition ([0-9]+): 
 (.*/pe-[^-]+-(%%)[.]bz2),

Do you know when this changed?

The reason I'm asking is that crmsh tries to support multiple
pacemaker versions, so I'm not sure if we can just replace this
pattern.

 Make tar follow symlinks.
 
 --- a/modules/crm_pssh.py   2013-08-12 12:52:11.0 +
 +++ b/modules/crm_pssh.py   2013-08-12 12:53:32.666444069 +
 @@ -170,7 +170,7 @@
  dir = /%s % r.group(1)
  red_pe_l = [x.replace(%s/ % r.group(1), ) for x in pe_l]
  common_debug(getting new PE inputs %s from %s % (red_pe_l, node))
 -cmdline = tar -C %s -cf - %s % (dir, ' '.join(red_pe_l))
 +cmdline = tar -C %s -chf - %s % (dir, ' '.join(red_pe_l))

Just curious: where did you find links in the PE input
directories?

And many thanks for the patches!

Cheers,

Dejan
 
 
 Best,
 Vladislav
 
 ___
 Linux-HA mailing list
 Linux-HA@lists.linux-ha.org
 http://lists.linux-ha.org/mailman/listinfo/linux-ha
 See also: http://linux-ha.org/ReportingProblems
___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems


[Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Vladislav Bogdanov
Hi Kristoffer,


there are two patches, one for crmsh and one for parallax.
They make history commands work.

--- a/modules/crm_pssh.py   2015-01-19 11:42:02.0 +
+++ b/modules/crm_pssh.py   2015-01-19 12:17:46.328000847 +
@@ -85,14 +85,14 @@ def do_pssh(l, opts):
'-o', 'PasswordAuthentication=no',
'-o', 'SendEnv=PSSH_NODENUM',
'-o', 'StrictHostKeyChecking=no']
-if opts.options:
+if hasattr(opts, 'options'):
 for opt in opts.options:
 cmd += ['-o', opt]
 if user:
 cmd += ['-l', user]
 if port:
 cmd += ['-p', port]
-if opts.extra:
+if hasattr(opts, 'extra'):
 cmd.extend(opts.extra)
 if cmdline:
 cmd.append(cmdline)
---

--- a/parallax/manager.py   2014-10-15 13:40:04.0 +
+++ b/parallax/manager.py   2015-01-19 12:15:47.911000236 +
@@ -47,11 +47,26 @@ class Manager(object):
 # Backwards compatibility with old __init__
 # format: Only argument is an options dict
 if not isinstance(limit, int):
-self.limit = limit.par
-self.timeout = limit.timeout
-self.askpass = limit.askpass
-self.outdir = limit.outdir
-self.errdir = limit.errdir
+if hasattr(limit, 'par'):
+self.limit = limit.par
+else:
+self.limit = DEFAULT_PARALLELISM
+if hasattr(limit, 'timeout'):
+self.timeout = limit.timeout
+else:
+self.timeout = DEFAULT_TIMEOUT
+if hasattr(limit, 'askpass'):
+self.askpass = limit.askpass
+else:
+self.askpass = False
+if hasattr(limit, 'outdir'):
+self.outdir = limit.outdir
+else:
+self.outdir = None
+if hasattr(limit, 'errdir'):
+self.errdir = limit.errdir
+else:
+self.errdir = None
 else:
 self.limit = limit
 self.timeout = timeout
---

Two more patches I use for ages in my builds are:

Fix transition start detection.

--- a/modules/constants.py  2014-12-22 08:48:26.0 +
+++ b/modules/constants.py  2014-12-22 13:07:43.945077805 +
@@ -272,7 +272,7 @@
 # r.group(3) file number
 transition_patt = [
 # transition start
-crmd.* do_te_invoke: Processing graph ([0-9]+) .*derived from 
(.*/pe-[^-]+-(%%)[.]bz2),
+pengine.* process_pe_message: Calculated Transition ([0-9]+): 
(.*/pe-[^-]+-(%%)[.]bz2),
 # r.group(1) transition number (a different thing from file number)
 # r.group(2) contains full path
 # r.group(3) transition status
---

Make tar follow symlinks.

--- a/modules/crm_pssh.py   2013-08-12 12:52:11.0 +
+++ b/modules/crm_pssh.py   2013-08-12 12:53:32.666444069 +
@@ -170,7 +170,7 @@
 dir = /%s % r.group(1)
 red_pe_l = [x.replace(%s/ % r.group(1), ) for x in pe_l]
 common_debug(getting new PE inputs %s from %s % (red_pe_l, node))
-cmdline = tar -C %s -cf - %s % (dir, ' '.join(red_pe_l))
+cmdline = tar -C %s -chf - %s % (dir, ' '.join(red_pe_l))
 opts = parse_args(outdir, errdir)
 l.append([node, cmdline])
 if not l:
---


Best,
Vladislav

___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems


Re: [Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Kristoffer Grönlund
Vladislav Bogdanov bub...@hoster-ok.com writes:

 Hi Kristoffer,


 there are two patches, one for crmsh and one for parallax.
 They make history commands work.

Thanks!

I have created a pull request with the patches for crmsh here:

https://github.com/crmsh/crmsh/pull/77

Cheers,
Kristoffer


 Best,
 Vladislav



-- 
// Kristoffer Grönlund
// kgronl...@suse.com
___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

Re: [Linux-HA] Support for DRDB

2015-01-19 Thread Andrew Beekhof

 On 18 Jan 2015, at 3:45 am, Lars Marowsky-Bree l...@suse.com wrote:
 
 On 2015-01-16T16:25:15, EXTERNAL Konold Martin (erfrakon, RtP2/TEF72) 
 external.martin.kon...@de.bosch.com wrote:
 
 I am glad to hear that SLE HA has no plans to drop support for DRBD.
 
 Unfortunately I currently cannot disclose who is spreading this false 
 information.
 
 Too bad. Do let them know they're quite wrong though ;-)

And ask them to stop repeating it.

___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems


Re: [Linux-HA] Support for DRDB

2015-01-19 Thread Andrew Beekhof

 On 17 Jan 2015, at 4:19 am, Digimer li...@alteeve.ca wrote:
 
 On 16/01/15 10:43 AM, Dmitri Maziuk wrote:
 On 1/16/2015 8:39 AM, Lars Marowsky-Bree wrote:
 On 2015-01-16T11:56:04, EXTERNAL Konold Martin (erfrakon,
 RtP2/TEF72) external.martin.kon...@de.bosch.com wrote:
 
 I have been told that support for DRBD is supposed to be phased
 out  from both SLES and RHEL in the near future.
 
 This is massively incorrect for SLE HA. (drbd is part of the HA add-on,
 not SLES.) We have absolutely no such plans, and will continue to
 support drbd as part of our offerings.
 
 Where did you hear that?
 
 Don't know about RHEL but on Centos you get DRBD from ELRepo, not even
 EPEL, so I expect it has not been in the RHEL for quite some time.
 Unless RedHat's offering it as a separate add-on to their paying
 customers (but the last rumour I heard was they wouldn't do it 'cause
 they didn't want to share the loot with linbit).
 
 Dima
 
 When RHEL 6 was released, Red Hat wanted to reduce their support overhead a 
 lot. So many things that used to be supported were dropped.

To my knowledge, DRBD was never actually supported.
Which would mean Red Hat's co-operation with Linbit has only ever increased 
over time - something I can't imagine changing.

 DRBD, unlike most other dropped programs, is still supported, just not by RH 
 directly. They worked out an agreement with LINBIT to allow officially 
 supported RHEL systems to be fully supported when they ran DRBD. The 
 agreement was that RHEL users who needed DRBD support specifically would be 
 routed to LINBIT for that support.
 
 So though it's changed, it is still supported.
 
 -- 
 Digimer
 Papers and Projects: https://alteeve.ca/w/
 What if the cure for cancer is trapped in the mind of a person without access 
 to education?
 ___
 Linux-HA mailing list
 Linux-HA@lists.linux-ha.org
 http://lists.linux-ha.org/mailman/listinfo/linux-ha
 See also: http://linux-ha.org/ReportingProblems

___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems


Re: [Linux-HA] [Patch] Collection of patches for crmsh

2015-01-19 Thread Dejan Muhamedagic
On Mon, Jan 19, 2015 at 04:56:14PM +0300, Vladislav Bogdanov wrote:
 Hi Dejan,
 
 19.01.2015 16:30, Dejan Muhamedagic wrote:
 Hi Vladislav,
 
 [...]
 Fix transition start detection.
 
 --- a/modules/constants.py  2014-12-22 08:48:26.0 +
 +++ b/modules/constants.py  2014-12-22 13:07:43.945077805 +
 @@ -272,7 +272,7 @@
   # r.group(3) file number
   transition_patt = [
   # transition start
 -crmd.* do_te_invoke: Processing graph ([0-9]+) .*derived from 
 (.*/pe-[^-]+-(%%)[.]bz2),
 +pengine.* process_pe_message: Calculated Transition ([0-9]+): 
 (.*/pe-[^-]+-(%%)[.]bz2),
 
 Do you know when this changed?
 
 Original message (from do_te_invoke) was downgraded into the 'info'
 priority a long ago (probably during that Andrew's massive logging
 cleanup), while process_pe_message' one still remains at the
 'notice' level. First my patch has 2012-12-26 as its date (for
 crmsh-1.2.4), so the change was done before that. iirc
 process_pe_message's message was always there, both messages were
 printed before that cleanup.

Ah, that rings the bell. But there's more than just that one:

https://build.opensuse.org/package/view_file/network:ha-clustering:Factory/pacemaker/bug-806256_pacemaker-log-level-notice.patch?expand=1

 The reason I'm asking is that crmsh tries to support multiple
 pacemaker versions, so I'm not sure if we can just replace this
 pattern.
 
 Make tar follow symlinks.
 
 --- a/modules/crm_pssh.py   2013-08-12 12:52:11.0 +
 +++ b/modules/crm_pssh.py   2013-08-12 12:53:32.666444069 +
 @@ -170,7 +170,7 @@
   dir = /%s % r.group(1)
   red_pe_l = [x.replace(%s/ % r.group(1), ) for x in pe_l]
   common_debug(getting new PE inputs %s from %s % (red_pe_l, 
  node))
 -cmdline = tar -C %s -cf - %s % (dir, ' '.join(red_pe_l))
 +cmdline = tar -C %s -chf - %s % (dir, ' '.join(red_pe_l))
 
 Just curious: where did you find links in the PE input
 directories?
 
 Ahm, you know, systems are s different around a world ;)

True. And I _do_ appreciate that fact.

 And system administrators sometimes want to do weird things ;)

Which is their prerogative too.

 Actually that one is specific to my diskless clusters, but it wont
 hurt anyways.

Diskless? Cool.

Cheers,

Dejan

 
 And many thanks for the patches!
 
 Cheers,
 
 Dejan
 
 
 Best,
 Vladislav
 
 ___
 Linux-HA mailing list
 Linux-HA@lists.linux-ha.org
 http://lists.linux-ha.org/mailman/listinfo/linux-ha
 See also: http://linux-ha.org/ReportingProblems
 ___
 Linux-HA mailing list
 Linux-HA@lists.linux-ha.org
 http://lists.linux-ha.org/mailman/listinfo/linux-ha
 See also: http://linux-ha.org/ReportingProblems
 
 
 ___
 Linux-HA mailing list
 Linux-HA@lists.linux-ha.org
 http://lists.linux-ha.org/mailman/listinfo/linux-ha
 See also: http://linux-ha.org/ReportingProblems
___
Linux-HA mailing list
Linux-HA@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems