On Mon, Feb 15, 2010 at 11:58 PM, Angus Salkeld <asalk...@redhat.com> wrote: > Hi > > I am using CTS for testing corosync (no pacemaker) and need this > as corosync doesn't know about hostnames only ip addresses. > > So in my Corosync class I can do the following: > > def getPatTheyStopped (self, other, node): > node_ip = socket.gethostbyname (node) > return self["Pat:They_stopped"] %(other, node_ip)
I suspect you'll probably want this elsewhere too... perhaps we want something more like: patterns.append(self.CM["Pat:They_stopped"] %(other, self.CM.key_for_node(node))) where self.CM.key_for_node() does whatever mapping is needed. > > > Regards > -Angus Salkeld > > > diff -Nur Pacemaker-1-0-3043c9539eca/cts/CTS.py > Pacemaker-1-0-1ddefe5f1353/cts/CTS.py > --- Pacemaker-1-0-3043c9539eca/cts/CTS.py 2010-02-16 07:12:09.000000000 > +1100 > +++ Pacemaker-1-0-1ddefe5f1353/cts/CTS.py 2010-02-15 20:16:27.289151904 > +1100 > @@ -404,6 +404,9 @@ > self.ShouldBeStatus={} > self.ns = NodeStatus(self.Env) > > + def getPatTheyStopped(self, other, node): > + return self.CM["Pat:They_stopped"] %(other, node) > + > def errorstoignore(self): > '''Return list of errors which are 'normal' and should be ignored''' > return [] > diff -Nur Pacemaker-1-0-3043c9539eca/cts/CTStests.py > Pacemaker-1-0-1ddefe5f1353/cts/CTStests.py > --- Pacemaker-1-0-3043c9539eca/cts/CTStests.py 2010-02-16 07:12:09.000000000 > +1100 > +++ Pacemaker-1-0-1ddefe5f1353/cts/CTStests.py 2010-02-16 09:43:06.728844951 > +1100 > @@ -490,7 +490,7 @@ > # NOTE: This wont work if we have multiple partitions > for other in self.CM.Env["nodes"]: > if self.CM.ShouldBeStatus[other] == "up" and other != node: > - patterns.append(self.CM["Pat:They_stopped"] %(other, node)) > + patterns.append(self.CM.getPatTheyStopped(other, node)) > #self.debug("Checking %s will notice %s left"%(other, node)) > > watch = CTS.LogWatcher( > @@ -1398,7 +1398,7 @@ > # Make sure the node goes down and then comes back up if it > should reboot... > for other in self.CM.Env["nodes"]: > if other != node: > - self.patterns.append(self.CM["Pat:They_stopped"] > %(other, node)) > + self.patterns.append(self.CM.getPatTheyStopped(other, > node)) > self.patterns.append(self.CM["Pat:Slave_started"] % node) > self.patterns.append(self.CM["Pat:Local_started"] % node) > > @@ -2019,7 +2019,7 @@ > else: > for stopping in stopset: > if self.CM.ShouldBeStatus[stopping] == "up": > - watchpats.append(self.CM["Pat:They_stopped"] % > (node, stopping)) > + watchpats.append(self.CM.getPatTheyStopped(node, > stopping)) > > if len(watchpats) == 0: > return self.skipped() > > > _______________________________________________ > Pacemaker mailing list > Pacemaker@oss.clusterlabs.org > http://oss.clusterlabs.org/mailman/listinfo/pacemaker > > _______________________________________________ Pacemaker mailing list Pacemaker@oss.clusterlabs.org http://oss.clusterlabs.org/mailman/listinfo/pacemaker