50dc83
From 70842c77735a655a053ed4a7cb77fec01028355a Mon Sep 17 00:00:00 2001
50dc83
From: Sunil Kumar Acharya <sheggodu@redhat.com>
50dc83
Date: Mon, 22 Apr 2019 12:48:13 +0530
50dc83
Subject: [PATCH 117/124] spec: Remove thin-arbiter package
50dc83
50dc83
Thin-arbiter is not supported in downstream. Updated the
50dc83
code to avoid RPMdiff warnings. Marked thin-arbiter
50dc83
test cases as bad to avoid nightly runs from reporting
50dc83
expected failures.
50dc83
50dc83
Label: DOWNSTREAM ONLY
50dc83
50dc83
BUG: 1698436
50dc83
Change-Id: Ic36bccdfe1c7039fb7e5ce078a8b64cf71056970
50dc83
Signed-off-by: Sunil Kumar Acharya <sheggodu@redhat.com>
50dc83
Reviewed-on: https://code.engineering.redhat.com/gerrit/168406
50dc83
Tested-by: RHGS Build Bot <nigelb@redhat.com>
50dc83
---
50dc83
 configure.ac                            |  2 --
50dc83
 extras/systemd/Makefile.am              |  6 ++---
50dc83
 glusterfs.spec.in                       | 39 +++++----------------------------
50dc83
 tests/basic/afr/ta-check-locks.t        |  2 ++
50dc83
 tests/basic/afr/ta-read.t               |  2 ++
50dc83
 tests/basic/afr/ta-shd.t                |  2 ++
50dc83
 tests/basic/afr/ta-write-on-bad-brick.t |  2 ++
50dc83
 xlators/features/Makefile.am            |  2 +-
50dc83
 8 files changed, 18 insertions(+), 39 deletions(-)
50dc83
50dc83
diff --git a/configure.ac b/configure.ac
50dc83
index 521671b..3065077 100644
50dc83
--- a/configure.ac
50dc83
+++ b/configure.ac
50dc83
@@ -116,8 +116,6 @@ AC_CONFIG_FILES([Makefile
50dc83
                 xlators/features/Makefile
50dc83
                 xlators/features/arbiter/Makefile
50dc83
                 xlators/features/arbiter/src/Makefile
50dc83
-                xlators/features/thin-arbiter/Makefile
50dc83
-                xlators/features/thin-arbiter/src/Makefile
50dc83
                 xlators/features/changelog/Makefile
50dc83
                 xlators/features/changelog/src/Makefile
50dc83
                 xlators/features/changelog/lib/Makefile
50dc83
diff --git a/extras/systemd/Makefile.am b/extras/systemd/Makefile.am
50dc83
index 61446a9..b849775 100644
50dc83
--- a/extras/systemd/Makefile.am
50dc83
+++ b/extras/systemd/Makefile.am
50dc83
@@ -1,8 +1,8 @@
50dc83
-CLEANFILES = glusterd.service glustereventsd.service glusterfssharedstorage.service gluster-ta-volume.service
50dc83
-EXTRA_DIST = glusterd.service.in glustereventsd.service.in glusterfssharedstorage.service.in gluster-ta-volume.service.in
50dc83
+CLEANFILES = glusterd.service glustereventsd.service glusterfssharedstorage.service
50dc83
+EXTRA_DIST = glusterd.service.in glustereventsd.service.in glusterfssharedstorage.service.in
50dc83
 
50dc83
 if USE_SYSTEMD
50dc83
-systemd_DATA = gluster-ta-volume.service
50dc83
+systemd_DATA =
50dc83
 endif
50dc83
 
50dc83
 if WITH_SERVER
50dc83
diff --git a/glusterfs.spec.in b/glusterfs.spec.in
50dc83
index ba095b7..bf72a55 100644
50dc83
--- a/glusterfs.spec.in
50dc83
+++ b/glusterfs.spec.in
50dc83
@@ -682,18 +682,6 @@ is in user space and easily manageable.
50dc83
 This package provides the glusterfs server daemon.
50dc83
 %endif
50dc83
 
50dc83
-%package thin-arbiter
50dc83
-Summary:          GlusterFS thin-arbiter module
50dc83
-Requires:         %{name}%{?_isa} = %{version}-%{release}
50dc83
-Requires:         %{name}-server%{?_isa} = %{version}-%{release}
50dc83
-
50dc83
-%description thin-arbiter
50dc83
-This package provides a tie-breaker functionality to GlusterFS
50dc83
-replicate volume. It includes translators required to provide the
50dc83
-functionality, and also few other scripts required for getting the setup done.
50dc83
-
50dc83
-This package provides the glusterfs thin-arbiter translator.
50dc83
-
50dc83
 %package client-xlators
50dc83
 Summary:          GlusterFS client-side translators
50dc83
 Requires:         %{name}-libs%{?_isa} = %{version}-%{release}
50dc83
@@ -1045,14 +1033,6 @@ fi
50dc83
 exit 0
50dc83
 %endif
50dc83
 
50dc83
-%preun thin-arbiter
50dc83
-if [ $1 -eq 0 ]; then
50dc83
-    if [ -f %glusterta_svcfile ]; then
50dc83
-        %service_stop gluster-ta-volume
50dc83
-        %systemd_preun gluster-ta-volume
50dc83
-    fi
50dc83
-fi
50dc83
-
50dc83
 ##-----------------------------------------------------------------------------
50dc83
 ## All %%postun should be placed here and keep them sorted
50dc83
 ##
50dc83
@@ -1188,6 +1168,12 @@ exit 0
50dc83
 %exclude %{_prefix}/lib/ocf/resource.d/heartbeat/*
50dc83
 %endif
50dc83
 
50dc83
+%exclude %{_datadir}/glusterfs/scripts/setup-thin-arbiter.sh
50dc83
+
50dc83
+%if ( 0%{?_without_server:1} )
50dc83
+%exclude %{_sysconfdir}/glusterfs/thin-arbiter.vol
50dc83
+%endif
50dc83
+
50dc83
 %files api
50dc83
 %exclude %{_libdir}/*.so
50dc83
 # libgfapi files
50dc83
@@ -1263,19 +1249,6 @@ exit 0
50dc83
 %{_bindir}/fusermount-glusterfs
50dc83
 %endif
50dc83
 
50dc83
-%files thin-arbiter
50dc83
-%dir %{_libdir}/glusterfs/%{version}%{?prereltag}/xlator
50dc83
-%dir %{_libdir}/glusterfs/%{version}%{?prereltag}/xlator/features
50dc83
-     %{_libdir}/glusterfs/%{version}%{?prereltag}/xlator/features/thin-arbiter.so
50dc83
-%dir %{_datadir}/glusterfs/scripts
50dc83
-     %{_datadir}/glusterfs/scripts/setup-thin-arbiter.sh
50dc83
-%config %{_sysconfdir}/glusterfs/thin-arbiter.vol
50dc83
-
50dc83
-%if ( 0%{?_with_systemd:1} )
50dc83
-%{_unitdir}/gluster-ta-volume.service
50dc83
-%endif
50dc83
-
50dc83
-
50dc83
 %if ( 0%{!?_without_georeplication:1} )
50dc83
 %files geo-replication
50dc83
 %config(noreplace) %{_sysconfdir}/logrotate.d/glusterfs-georep
50dc83
diff --git a/tests/basic/afr/ta-check-locks.t b/tests/basic/afr/ta-check-locks.t
50dc83
index c0102c3..c51aa39 100644
50dc83
--- a/tests/basic/afr/ta-check-locks.t
50dc83
+++ b/tests/basic/afr/ta-check-locks.t
50dc83
@@ -66,3 +66,5 @@ TEST ta_start_brick_process brick0
50dc83
 EXPECT_WITHIN $HEAL_TIMEOUT "0" get_lock_count_on_ta
50dc83
 
50dc83
 cleanup;
50dc83
+#G_TESTDEF_TEST_STATUS_CENTOS6=BAD_TEST,BUG=0000000
50dc83
+#G_TESTDEF_TEST_STATUS_NETBSD7=BAD_TEST,BUG=0000000
50dc83
diff --git a/tests/basic/afr/ta-read.t b/tests/basic/afr/ta-read.t
50dc83
index f2b3c38..1b36dba 100644
50dc83
--- a/tests/basic/afr/ta-read.t
50dc83
+++ b/tests/basic/afr/ta-read.t
50dc83
@@ -58,3 +58,5 @@ TEST [ -z $TA_PID ]
50dc83
 # Read must now succeed.
50dc83
 TEST cat $M0/FILE
50dc83
 cleanup;
50dc83
+#G_TESTDEF_TEST_STATUS_CENTOS6=BAD_TEST,BUG=0000000
50dc83
+#G_TESTDEF_TEST_STATUS_NETBSD7=BAD_TEST,BUG=0000000
50dc83
diff --git a/tests/basic/afr/ta-shd.t b/tests/basic/afr/ta-shd.t
50dc83
index bb2e58b..4b1ea85 100644
50dc83
--- a/tests/basic/afr/ta-shd.t
50dc83
+++ b/tests/basic/afr/ta-shd.t
50dc83
@@ -47,3 +47,5 @@ TEST ta_start_mount_process $M0
50dc83
 EXPECT_WITHIN $PROCESS_UP_TIMEOUT "1" ta_up_status $V0 $M0 0
50dc83
 EXPECT "Hello" cat $M0/a.txt
50dc83
 cleanup;
50dc83
+#G_TESTDEF_TEST_STATUS_CENTOS6=BAD_TEST,BUG=0000000
50dc83
+#G_TESTDEF_TEST_STATUS_NETBSD7=BAD_TEST,BUG=0000000
50dc83
diff --git a/tests/basic/afr/ta-write-on-bad-brick.t b/tests/basic/afr/ta-write-on-bad-brick.t
50dc83
index 18cb65b..77cbf5f 100644
50dc83
--- a/tests/basic/afr/ta-write-on-bad-brick.t
50dc83
+++ b/tests/basic/afr/ta-write-on-bad-brick.t
50dc83
@@ -49,3 +49,5 @@ EXPECT_WITHIN $PROCESS_UP_TIMEOUT "1" afr_child_up_status_meta $M0 $V0-replicate
50dc83
 TEST dd if=/dev/zero of=$M0/a.txt bs=1M count=5
50dc83
 
50dc83
 cleanup;
50dc83
+#G_TESTDEF_TEST_STATUS_CENTOS6=BAD_TEST,BUG=0000000
50dc83
+#G_TESTDEF_TEST_STATUS_NETBSD7=BAD_TEST,BUG=0000000
50dc83
diff --git a/xlators/features/Makefile.am b/xlators/features/Makefile.am
50dc83
index 545c02b..537c148 100644
50dc83
--- a/xlators/features/Makefile.am
50dc83
+++ b/xlators/features/Makefile.am
50dc83
@@ -4,7 +4,7 @@ endif
50dc83
 
50dc83
 SUBDIRS = locks quota read-only quiesce marker index barrier arbiter upcall \
50dc83
 	compress changelog gfid-access snapview-client snapview-server trash \
50dc83
-	shard bit-rot leases selinux sdfs namespace $(CLOUDSYNC_DIR) thin-arbiter \
50dc83
+	shard bit-rot leases selinux sdfs namespace $(CLOUDSYNC_DIR) \
50dc83
 	utime changetimerecorder
50dc83
 
50dc83
 CLEANFILES =
50dc83
-- 
50dc83
1.8.3.1
50dc83