Hi, Dejan
Post by Dejan MuhamedagicHi Yusuke-san,
Post by yusuke iidaHi, Dejan
Post by Dejan MuhamedagicHi Yusuke,
Post by yusuke iidaHi, Dejan
I also tested by rhel6.3 and fedora17.
Since there is no environment, centos is not tested.
- 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!
Post by yusuke iidaPost by Dejan MuhamedagicPost by yusuke iida- 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
http://download.opensuse.org/repositories/network:/ha-clustering/RedHat_RHEL-6/x86_64/
Post by yusuke iidaI 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
Post by Dejan MuhamedagicPost by yusuke iidaIf 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
Post by yusuke iidaRegards,
Yusuke
Post by Dejan MuhamedagicCheers,
Dejan
Post by yusuke iidaRegards,
Yusuke
Post by Dejan MuhamedagicPost by Dejan MuhamedagicPost by Dejan MuhamedagicHi,
Post by yusuke iidaHi, 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
[ 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
Post by Dejan MuhamedagicThanks,
Dejan
Post by Dejan MuhamedagicNo 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
Post by yusuke iidaBest regards,
Yusuke
--
----------------------------------------
METRO SYSTEMS CO., LTD
Yusuke Iida
----------------------------------------
_______________________________________________________
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
_______________________________________________________
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://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
----------------------------------------
--
----------------------------------------
METRO SYSTEMS CO., LTD
Yusuke Iida
----------------------------------------
_______________________________________________________
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
--
----------------------------------------
METRO SYSTEMS CO., LTD
Yusuke Iida
Mail: ***@gmail.com
----------------------------------------