Hi Yusuke-san,

On Tue, Jul 02, 2013 at 01:08:18PM +0900, yusuke iida wrote:
> Hi, Dejan
> 
> Could you incorporate this patch?

Oh, somehow missed this one. Sorry about that.

Yes, I'll apply this patch.

Anyway, why is it so difficult to install pssh? Or did I already
forget you reasoning?

Cheers,

Dejan

> I want some messages.
> 
> Regards,
> Yusuke
> 
> 2013/6/3 yusuke iida <yusk.i...@gmail.com>:
> > Hi, Dejan
> >
> > 2013/4/18 Dejan Muhamedagic <de...@suse.de>:
> >> Hi Yusuke-san,
> >>
> >> On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote:
> >>> Hi, Dejan
> >>>
> >>> 2013/4/4 Dejan Muhamedagic <de...@suse.de>:
> >>> > Hi Yusuke,
> >>> >
> >>> > On Thu, Feb 21, 2013 at 09:04:45PM +0900, yusuke iida wrote:
> >>> >> Hi, Dejan
> >>> >>
> >>> >> I also tested by rhel6.3 and fedora17.
> >>> >> Since there is no environment, centos is not tested.
> >>> >>
> >>> >> The point worried below is shown:
> >>> >> - I think that %{?fedora_version} and %{?rhel_version} are macro not 
> >>> >> to exist.
> >>> >
> >>> > Those macros work in OBS when rhel6 packages are built. I wonder
> >>> > if that's some build service extension.
> >>>
> >>> In my environment, the macro of rpmbuild is as follows.
> >>>
> >>> rhel6.3
> >>> # rpmbuild --showrc | grep rhel
> >>> -14: rhel       6
> >>>
> >>> fedora18
> >>> # rpmbuild --showrc | grep fedora
> >>> -14: fedora     18
> >>>
> >>> So I want you to revise it as follows at least.
> >>>
> >>> # hg diff
> >>> diff -r da93d3523e6a crmsh.spec
> >>> --- a/crmsh.spec        Tue Mar 26 11:44:17 2013 +0100
> >>> +++ b/crmsh.spec        Tue Apr 16 13:08:37 2013 +0900
> >>> @@ -6,7 +6,7 @@
> >>>  %global upstream_version tip
> >>>  %global upstream_prefix crmsh
> >>>
> >>> -%if 0%{?fedora_version} || 0%{?centos_version} || 0%{?rhel_version}
> >>> +%if 0%{?fedora_version} || 0%{?centos_version} || 0%{?rhel_version}
> >>> || 0%{?rhel} || 0%{?fedora}
> >>>  %define pkg_group System Environment/Daemons
> >>>  %else
> >>>  %define pkg_group Productivity/Clustering/HA
> >>
> >> Patch applied. Thanks!
> >>
> >>> >> - pssh is not provided in rhel.
> >>> >>   I think that you should not put it in Requires.
> >>> >
> >>> > OK, but currently the only RPM built is the one in OBS where the
> >>> > repository includes pssh RPMs for rhel/centos too. See for
> >>> > instance:
> >>> >
> >>> > http://download.opensuse.org/repositories/network:/ha-clustering/RedHat_RHEL-6/x86_64/
> >>> >
> >>> >> I made a patch to solve the above.
> >>> >
> >>> > Note that the .spec file in the upstream may not be perfect or
> >>> > even work on particular distribution. However, it should advise
> >>> > packagers on what it should contain. The pssh requirement is
> >>> > there because history would not work well without it. It is
> >>> > further rather unfortunate that that feature is used very seldom
> >>> > and that it got so little attention.
> >>> >
> >>> > Therefore, I'm reluctant to apply the pssh part of the patch.
> >>>
> >>> hmm ...
> >>> For example, can't it change so that the function in which pssh is
> >>> used may be disabled by the configure option?
> >>
> >> The functionality is still there, even without pssh. For
> >> instance, static reports can also be examined. It's just that the
> >> live updates are going to be quite a bit slower, if somebody
> >> wants to use the history feature to examine changes happening in
> >> the cluster.
> > I made sure the source code.
> >
> > In the environment where pssh is not supported, history was collecting
> > information using the crm_report command.
> > Furthermore, the processing which is using pssh was found. It is rsctest.
> >
> > This processing serves as an error of python in the environment where
> > pssh is not supported.
> >
> > Probing resources .Traceback (most recent call last):
> >   File "/usr/sbin/crm", line 44, in <module>
> >     main.run()
> >   File "/usr/lib64/python2.6/site-packages/crmsh/main.py", line 413, in run
> >     do_work()
> >   File "/usr/lib64/python2.6/site-packages/crmsh/main.py", line 323, in 
> > do_work
> >     if parse_line(levels,shlex.split(' '.join(l))):
> >   File "/usr/lib64/python2.6/site-packages/crmsh/main.py", line 149,
> > in parse_line
> >     rv = d() # execute the command
> >   File "/usr/lib64/python2.6/site-packages/crmsh/main.py", line 148, in 
> > <lambda>
> >     d = lambda: cmd[0](*args)
> >   File "/usr/lib64/python2.6/site-packages/crmsh/ui.py", line 1945, in 
> > rsc_test
> >     return test_resources(rsc_l, node_l, all_nodes)
> >   File "/usr/lib64/python2.6/site-packages/crmsh/rsctest.py", line
> > 300, in test_resources
> >     if not are_all_stopped(rsc_l, all_nodes_l):
> >   File "/usr/lib64/python2.6/site-packages/crmsh/rsctest.py", line
> > 250, in are_all_stopped
> >     drv.runop("probe")
> >   File "/usr/lib64/python2.6/site-packages/crmsh/rsctest.py", line 143, in 
> > runop
> >     from crm_pssh import do_pssh_cmd
> >   File "/usr/lib64/python2.6/site-packages/crmsh/crm_pssh.py", line
> > 24, in <module>
> >     from psshlib import psshutil
> > ImportError: No module named psshlib
> >
> > Since I thought that this was a problem, I added the processing which
> > checks support of pssh.
> >
> > If there is no problem, I want you to apply this patch.
> >
> > Regards,
> > Yusuke
> >>
> >>> If it is possible, can it not exclude pssh from Requires?
> >>
> >> I already reasoned in my previous message (quoted above) why I'm
> >> reluctant to do that.
> >>
> >> Cheers,
> >>
> >> Dejan
> >>
> >>> Regards,
> >>> Yusuke
> >>> >
> >>> > Cheers,
> >>> >
> >>> > Dejan
> >>> >
> >>> >> Regards,
> >>> >> Yusuke
> >>> >>
> >>> >> 2013/2/19 Dejan Muhamedagic <de...@suse.de>:
> >>> >> > On Tue, Feb 19, 2013 at 11:03:53AM +0100, Dejan Muhamedagic wrote:
> >>> >> >> On Fri, Feb 15, 2013 at 10:19:41PM +0100, Dejan Muhamedagic wrote:
> >>> >> >> > Hi,
> >>> >> >> >
> >>> >> >> > On Fri, Feb 15, 2013 at 02:25:41PM +0900, yusuke iida wrote:
> >>> >> >> > > Hi, Dejan
> >>> >> >> > >
> >>> >> >> > > I made a patch for spec file to make rpm of crmsh in rhel 
> >>> >> >> > > environment.
> >>> >> >> > > I want a crmsh repository to merge it if I do not have any 
> >>> >> >> > > problem.
> >>> >> >>
> >>> >> >> This is a problem which I ran into earlier too. Something
> >>> >> >> (probably one of the rpm macros) does a 'rm -rf' of the doc
> >>> >> >> directory _after_ the files got installed:
> >>> >> >>
> >>> >> >> [   29s] test -z "/usr/share/doc/packages/crmsh" || /bin/mkdir -p 
> >>> >> >> "/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh"
> >>> >> >> [   29s]  /usr/bin/install -c -m 644 'AUTHORS' 
> >>> >> >> '/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/AUTHORS'
> >>> >> >> [   29s]  /usr/bin/install -c -m 644 'COPYING' 
> >>> >> >> '/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/COPYING'
> >>> >> >> ...
> >>> >> >> [   30s] Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.6245
> >>> >> >> [   30s] + umask 022
> >>> >> >> [   30s] + cd /usr/src/packages/BUILD
> >>> >> >> [   30s] + cd crmsh
> >>> >> >> [   30s] + 
> >>> >> >> DOCDIR=/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >>> >> >> [   30s] + export DOCDIR
> >>> >> >> [   30s] + rm -rf 
> >>> >> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >>> >> >> [   30s] + /bin/mkdir -p 
> >>> >> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >>> >> >> [   30s] + cp -pr ChangeLog 
> >>> >> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >>> >> >> ...
> >>> >> >> [   32s] error: create archive failed on file 
> >>> >> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/AUTHORS: 
> >>> >> >> cpio: open failed - Bad file descriptor
> >>> >> >>
> >>> >> >> If somebody can shed some light or suggest how to deal with
> >>> >> >> this ...
> >>> >> >
> >>> >> > OK. I think I managed to fix it. The result is already upstream.
> >>> >> > I tested it with rhel6, centos6, fedora 17 and 18. Can you
> >>> >> > please test too.
> >>> >> >
> >>> >> > Thanks,
> >>> >> >
> >>> >> > Dejan
> >>> >> >
> >>> >> >> Thanks,
> >>> >> >>
> >>> >> >> Dejan
> >>> >> >>
> >>> >> >>
> >>> >> >> > No problem. Will test the patch. BTW, did you notice that there
> >>> >> >> > are packages for rhel too at OBS (see the latest news item at
> >>> >> >> > https://savannah.nongnu.org/projects/crmsh/).
> >>> >> >> >
> >>> >> >> > Cheers,
> >>> >> >> >
> >>> >> >> > Dejan
> >>> >> >> >
> >>> >> >> > >
> >>> >> >> > > Best regards,
> >>> >> >> > > Yusuke
> >>> >> >> > > --
> >>> >> >> > > ----------------------------------------
> >>> >> >> > > METRO SYSTEMS CO., LTD
> >>> >> >> > >
> >>> >> >> > > Yusuke Iida
> >>> >> >> > > Mail: yusk.i...@gmail.com
> >>> >> >> > > ----------------------------------------
> >>> >> >> >
> >>> >> >> >
> >>> >> >> > _______________________________________________________
> >>> >> >> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >>> >> >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >>> >> >> > Home Page: http://linux-ha.org/
> >>> >> >> _______________________________________________________
> >>> >> >> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >>> >> >> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >>> >> >> Home Page: http://linux-ha.org/
> >>> >> > _______________________________________________________
> >>> >> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >>> >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >>> >> > Home Page: http://linux-ha.org/
> >>> >>
> >>> >>
> >>> >>
> >>> >> --
> >>> >> ----------------------------------------
> >>> >> METRO SYSTEMS CO., LTD
> >>> >>
> >>> >> Yusuke Iida
> >>> >> Mail: yusk.i...@gmail.com
> >>> >> ----------------------------------------
> >>> >
> >>> >
> >>>
> >>>
> >>>
> >>> --
> >>> ----------------------------------------
> >>> METRO SYSTEMS CO., LTD
> >>>
> >>> Yusuke Iida
> >>> Mail: yusk.i...@gmail.com
> >>> ----------------------------------------
> >> _______________________________________________________
> >> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >> Home Page: http://linux-ha.org/
> >
> >
> >
> > --
> > ----------------------------------------
> > METRO SYSTEMS CO., LTD
> >
> > Yusuke Iida
> > Mail: yusk.i...@gmail.com
> > ----------------------------------------
> 
> 
> 
> -- 
> ----------------------------------------
> METRO SYSTEMS CO., LTD
> 
> Yusuke Iida
> Mail: yusk.i...@gmail.com
> ----------------------------------------
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to