Merge ~lucaskanashiro/ubuntu/+source/resource-agents:rename-binary-pkgs into ubuntu/+source/resource-agents:ubuntu/devel

Proposed by Lucas Kanashiro
Status: Merged
Approved by: Lucas Kanashiro
Approved revision: 6e97c30cc566eebc29679ac5c81a36a866355cf2
Merged at revision: 6e97c30cc566eebc29679ac5c81a36a866355cf2
Proposed branch: ~lucaskanashiro/ubuntu/+source/resource-agents:rename-binary-pkgs
Merge into: ubuntu/+source/resource-agents:ubuntu/devel
Diff against target: 391 lines (+124/-44)
11 files modified
debian/changelog (+41/-0)
debian/control (+24/-7)
debian/extract-agents-description (+12/-12)
debian/install-common-files (+7/-7)
debian/install-curated-agents (+22/-0)
debian/install-ldirectord-files (+7/-7)
debian/resource-agents-cleanup (+2/-2)
debian/resource-agents-extra.lintian-overrides (+5/-0)
debian/rules (+3/-3)
debian/tests/control (+1/-1)
dev/null (+0/-5)
Reviewer Review Type Date Requested Status
Sergio Durigan Junior (community) Approve
Canonical Server Pending
Review via email: mp+402678@code.launchpad.net

Description of the change

Create a transitional package called 'resource-agents', and rename the binary containing supported agents to 'resource-agents-base' and the binary containing the unsupported agents to 'resource-agents-extra'.

I am not sure yet about the right terms we want to use in the package descriptions, I am waiting on Rick to define that.

The proposed package is available in this PPA:

https://launchpad.net/~lucaskanashiro/+archive/ubuntu/ha-agents-split

autopkgtest summary:

autopkgtest [18:23:57]: @@@@@@@@@@@@@@@@@@@@ summary
IPaddr2 PASS
command1 PASS
command2 PASS
command3 PASS
command4 PASS
command5 PASS
command6 PASS
command7 PASS
command8 PASS
command9 PASS
command10 PASS
command11 PASS
command12 PASS
command13 PASS

To post a comment you must log in.
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

I ran the tests for the supported agents and they are still passing, no regression.

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks for the MP, Lucas.

I'm leaving a bunch of comments that I think should be addressed, but they're mostly about updating comments/variables names/file names. There is one issue with the package, though: the resource-agents-extra package does not have the proper Conflicts/Breaks directives, which breaks the downgrade of the package. This diff fixed it for me:

diff --git a/debian/control b/debian/control
index 2ae7451..026c27e 100644
--- a/debian/control
+++ b/debian/control
@@ -60,7 +60,10 @@ Recommends: libxml2-utils,
             net-tools,
             resource-agents-base (>= ${source:Version})
 Suggests: python3-googleapi
-Replaces: cluster-agents (<= 1:1.0.4-1), rgmanager (<= 3.0.12-2+b1)
+Replaces: cluster-agents (<= 1:1.0.4-1),
+ rgmanager (<= 3.0.12-2+b1),
+ resource-agents (<< 1:4.7.0-1ubuntu2)
+Breaks: resource-agents (<< 1:4.7.0-1ubuntu2)
 Conflicts: cluster-agents (<= 1:1.0.4-1), rgmanager (<= 3.0.12-2+b1)
 Description: Cluster Resource Agents
  This package contains cluster resource agents (RAs) compliant with the Open

Other than what I pointed here, the changes LGTM.

review: Needs Fixing
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

I think I forgot to mention, but the "debian/supported-agents" file should also be renamed IMO.

Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks for the review Sergio. I believe I addressed all your comments, up for review again.

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thanks, Lucas. This LGTM now (module some very minor details that I'm leaving as comments). I've verified that the package builds, installs, upgrades and downgrades fine.

review: Approve
Revision history for this message
Lucas Kanashiro (lucaskanashiro) wrote :

Thanks Sergio, applied the fixes and uploaded the package:

$ git push pkg upload/1%4.7.0-1ubuntu4
Enumerating objects: 46, done.
Counting objects: 100% (46/46), done.
Delta compression using up to 32 threads
Compressing objects: 100% (34/34), done.
Writing objects: 100% (34/34), 4.50 KiB | 2.25 MiB/s, done.
Total 34 (delta 26), reused 0 (delta 0)
To ssh://git.launchpad.net/ubuntu/+source/resource-agents
 * [new tag] upload/1%4.7.0-1ubuntu4 -> upload/1%4.7.0-1ubuntu4
$ dput ubuntu ../resource-agents_4.7.0-1ubuntu4_source.changes
Checking signature on .changes
gpg: ../resource-agents_4.7.0-1ubuntu4_source.changes: Valid signature from F823A2729883C97C
Checking signature on .dsc
gpg: ../resource-agents_4.7.0-1ubuntu4.dsc: Valid signature from F823A2729883C97C
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading resource-agents_4.7.0-1ubuntu4.dsc: done.
  Uploading resource-agents_4.7.0-1ubuntu4.debian.tar.xz: done.
  Uploading resource-agents_4.7.0-1ubuntu4_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 2f3a429..a05a93e 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,44 @@
6+resource-agents (1:4.7.0-1ubuntu4) impish; urgency=medium
7+
8+ * Rename resource-agents with non-curated agents to resource-agents-extra.
9+ - d/control: rename the resource-agents binary package to
10+ resource-agents-extra.
11+ - d/extract-agents-description:
12+ + Rename the package name used in the substvars filename.
13+ + Rename all the occurrences of the 'unsupported' term to 'non-curated'.
14+ - Rename RESOURCE_AGENTS_DIR to RESOURCE_AGENTS_EXTRA_DIR with the
15+ appropriate path in the following files:
16+ + d/install-common-files
17+ + d/install-common-files
18+ + d/install-supported-agents
19+ + d/resource-agents-cleanup
20+ + d/rules
21+ * Rename resource-agents-supported with curated agents to
22+ resource-agents-base.
23+ - d/control:
24+ + Rename resource-agents-supported binary package to resource-agents-base.
25+ + Update the short and long description to not use the 'supported' term.
26+ + Rename the supportedAgents substvar to curatedAgents.
27+ - d/curated-agents: new file, this is the renamed d/supported-agents file.
28+ - d/extract-agents-description:
29+ + Rename the package name used in the substvars filename.
30+ + Rename all the occurrences of the 'supported' term to 'curated'.
31+ - d/install-common-files: Rename all the occurrences of the 'supported'
32+ term to 'curated'.
33+ - d/install-curated-agents: new file, this is the renamed
34+ d/install-supported-agents script.
35+ - d/rules: call d/install-curated-agents instead of
36+ d/install-supported-agents.
37+ - d/t/control: make IPaddr2 test depend on resource-agents-base instead of
38+ resource-agents-supported.
39+ * Create a transitional binary package called resource-agents.
40+ - d/control: add a new section for the transitional package, adding
41+ resource-agents-{base,extra} as dependencies.
42+ * d/control: add a missing Breaks/Replaces relation to the
43+ resource-agents-base.
44+
45+ -- Lucas Kanashiro <kanashiro@ubuntu.com> Wed, 12 May 2021 16:15:15 -0300
46+
47 resource-agents (1:4.7.0-1ubuntu3) impish; urgency=medium
48
49 * d/control: make resource-agents recommend resource-agents-supported.
50diff --git a/debian/control b/debian/control
51index e2f229f..4265cb2 100644
52--- a/debian/control
53+++ b/debian/control
54@@ -37,6 +37,20 @@ Vcs-Browser: https://salsa.debian.org/ha-team/resource-agents
55 Vcs-Git: https://salsa.debian.org/ha-team/resource-agents.git
56
57 Package: resource-agents
58+Architecture: all
59+Section: oldlibs
60+Depends: ${misc:Depends},
61+ resource-agents-base (>= ${source:Version}),
62+ resource-agents-extra (>= ${source:Version})
63+Description: Transitional package for the resource-agents
64+ This package contains cluster resource agents (RAs) compliant with the Open
65+ Cluster Framework (OCF) specification, used to interface with various services
66+ in a High Availability environment managed by the Pacemaker resource manager.
67+ .
68+ This is a transitional package which provides a set of curated
69+ (resource-agents-base) and non-curated (resource-agents-extra) agents.
70+
71+Package: resource-agents-extra
72 Architecture: any
73 Depends: ${misc:Depends}, ${shlibs:Depends}, ${python3:Depends},
74 cluster-glue,
75@@ -45,9 +59,12 @@ Depends: ${misc:Depends}, ${shlibs:Depends}, ${python3:Depends},
76 resource-agents-common (>= ${source:Version})
77 Recommends: libxml2-utils,
78 net-tools,
79- resource-agents-supported (>= ${source:Version})
80+ resource-agents-base (>= ${source:Version})
81 Suggests: python3-googleapi
82-Replaces: cluster-agents (<= 1:1.0.4-1), rgmanager (<= 3.0.12-2+b1)
83+Replaces: cluster-agents (<= 1:1.0.4-1),
84+ rgmanager (<= 3.0.12-2+b1),
85+ resource-agents (<< 1:4.7.0-1ubuntu2)
86+Breaks: resource-agents (<< 1:4.7.0-1ubuntu2)
87 Conflicts: cluster-agents (<= 1:1.0.4-1), rgmanager (<= 3.0.12-2+b1)
88 Description: Cluster Resource Agents
89 This package contains cluster resource agents (RAs) compliant with the Open
90@@ -57,7 +74,7 @@ Description: Cluster Resource Agents
91 Agents included:
92 ${agents}
93
94-Package: resource-agents-supported
95+Package: resource-agents-base
96 Architecture: all
97 Depends: ${misc:Depends},
98 resource-agents-common (>= ${source:Version})
99@@ -67,16 +84,16 @@ Replaces: cluster-agents (<= 1:1.0.4-1),
100 Breaks: resource-agents (<< 1:4.7.0-1ubuntu2)
101 Conflicts: cluster-agents (<= 1:1.0.4-1),
102 rgmanager (<= 3.0.12-2+b1)
103-Description: Supported Cluster Resource Agents
104+Description: Cluster Resource Agents curated by Ubuntu
105 This package contains cluster resource agents (RAs) compliant with the Open
106 Cluster Framework (OCF) specification, used to interface with various services
107 in a High Availability environment managed by the Pacemaker resource manager.
108 .
109- This package contains the Resource Agents that are officially supported by
110- Ubuntu.
111+ This package contains the Resource Agents that are curated by the Ubuntu
112+ Server team.
113 .
114 Agents included:
115- ${supportedAgents}
116+ ${curatedAgents}
117
118 Package: resource-agents-common
119 Architecture: any
120diff --git a/debian/supported-agents b/debian/curated-agents
121similarity index 100%
122rename from debian/supported-agents
123rename to debian/curated-agents
124diff --git a/debian/extract-agents-description b/debian/extract-agents-description
125index 9c11eb2..9824f4d 100755
126--- a/debian/extract-agents-description
127+++ b/debian/extract-agents-description
128@@ -5,18 +5,18 @@ set -x
129 export LC_ALL=C
130
131 agents=()
132-supported_agents=()
133-unsupported_agents=()
134+curated_agents=()
135+non_curated_agents=()
136
137-readonly RESOURCE_AGENTS_DIR="debian/resource-agents"
138+readonly RESOURCE_AGENTS_EXTRA_DIR="debian/resource-agents-extra"
139 readonly RESOURCE_AGENTS_COMMON_DIR="debian/resource-agents-common"
140-readonly RESOURCE_AGENTS_SUPPORTED_DIR="debian/resource-agents-supported"
141-readonly UNSUPPORTED_HEARTBEAT_DIR="$RESOURCE_AGENTS_DIR/usr/lib/ocf/resource.d/heartbeat"
142-readonly SUPPORTED_HEARTBEAT_DIR="$RESOURCE_AGENTS_SUPPORTED_DIR/usr/lib/ocf/resource.d/heartbeat"
143+readonly RESOURCE_AGENTS_BASE_DIR="debian/resource-agents-base"
144+readonly NON_CURATED_HEARTBEAT_DIR="$RESOURCE_AGENTS_EXTRA_DIR/usr/lib/ocf/resource.d/heartbeat"
145+readonly CURATED_HEARTBEAT_DIR="$RESOURCE_AGENTS_BASE_DIR/usr/lib/ocf/resource.d/heartbeat"
146
147 export OCF_ROOT="$RESOURCE_AGENTS_COMMON_DIR/usr/lib/ocf"
148
149-for agent in $SUPPORTED_HEARTBEAT_DIR/* $UNSUPPORTED_HEARTBEAT_DIR/*; do
150+for agent in $CURATED_HEARTBEAT_DIR/* $NON_CURATED_HEARTBEAT_DIR/*; do
151 name=$(basename "$agent")
152 short_desc=$("$agent" meta-data | xml_grep --root resource-agent/shortdesc --text_only /dev/stdin)
153
154@@ -30,15 +30,15 @@ for agent in $SUPPORTED_HEARTBEAT_DIR/* $UNSUPPORTED_HEARTBEAT_DIR/*; do
155 done
156
157 for agent in "${agents[@]}"; do
158- if grep -qFx $(cut -d: -f1 <<< "$agent") debian/supported-agents; then
159- supported_agents+=("$agent\${Newline}")
160+ if grep -qFx $(cut -d: -f1 <<< "$agent") debian/curated-agents; then
161+ curated_agents+=("$agent\${Newline}")
162 else
163- unsupported_agents+=("$agent\${Newline}")
164+ non_curated_agents+=("$agent\${Newline}")
165 fi
166 done
167
168-echo "agents=${unsupported_agents[*]}" >> debian/resource-agents.substvars
169-echo "supportedAgents=${supported_agents[*]}" >> debian/resource-agents-supported.substvars
170+echo "agents=${non_curated_agents[*]}" >> debian/resource-agents-extra.substvars
171+echo "curatedAgents=${curated_agents[*]}" >> debian/resource-agents-base.substvars
172
173 # Clean up pycache
174 rm -rf debian/resource-agents-common/usr/lib/ocf/lib/heartbeat/__pycache__
175diff --git a/debian/install-common-files b/debian/install-common-files
176index 94dc125..e1c652d 100755
177--- a/debian/install-common-files
178+++ b/debian/install-common-files
179@@ -1,11 +1,11 @@
180 #!/bin/bash
181
182-# This script aims to install all the common files used by resource-agents and
183-# resource-agents-supported into resource-agents-common binary package.
184+# This script aims to install all the common files used by resource-agents-base
185+# and resource-agents-extra into resource-agents-common binary package.
186
187 set -x
188
189-readonly RESOURCE_AGENTS_DIR="debian/resource-agents"
190+readonly RESOURCE_AGENTS_EXTRA_DIR="debian/resource-agents-extra"
191 readonly RESOURCE_AGENTS_COMMON_DIR="debian/resource-agents-common"
192
193 readonly COMMON_DIRS="usr/include/heartbeat usr/lib/heartbeat
194@@ -15,7 +15,7 @@ readonly COMMON_DIRS="usr/include/heartbeat usr/lib/heartbeat
195
196 for dir in $COMMON_DIRS; do
197 mkdir -p "$RESOURCE_AGENTS_COMMON_DIR/$dir"
198- mv "$RESOURCE_AGENTS_DIR/$dir"/* "$RESOURCE_AGENTS_COMMON_DIR/$dir"
199+ mv "$RESOURCE_AGENTS_EXTRA_DIR/$dir"/* "$RESOURCE_AGENTS_COMMON_DIR/$dir"
200 done
201
202 mkdir -p "$RESOURCE_AGENTS_COMMON_DIR"/usr/lib/ocf/resource.d/heartbeat \
203@@ -25,17 +25,17 @@ readonly TARGETS="ocf-binaries ocf-directories ocf-returncodes ocf-shellfuncs"
204 for tgt in $TARGETS; do
205 ln -r -s -T "$RESOURCE_AGENTS_COMMON_DIR/usr/lib/ocf/lib/heartbeat/$tgt" \
206 "$RESOURCE_AGENTS_COMMON_DIR/usr/lib/ocf/resource.d/heartbeat/.$tgt"
207- rm "$RESOURCE_AGENTS_DIR/usr/lib/ocf/resource.d/heartbeat/.$tgt"
208+ rm "$RESOURCE_AGENTS_EXTRA_DIR/usr/lib/ocf/resource.d/heartbeat/.$tgt"
209 done
210
211 readonly FILES="metadata.rng ra-api-1.dtd"
212 for file in $FILES; do
213- mv "$RESOURCE_AGENTS_DIR/usr/share/resource-agents/$file" \
214+ mv "$RESOURCE_AGENTS_EXTRA_DIR/usr/share/resource-agents/$file" \
215 "$RESOURCE_AGENTS_COMMON_DIR"/usr/share/resource-agents/
216 done
217
218 readonly OCFT_FILES="README README.zh_CN caselib helpers.sh runocft runocft.prereq"
219 for file in $OCFT_FILES; do
220- mv "$RESOURCE_AGENTS_DIR/usr/share/resource-agents/ocft/$file" \
221+ mv "$RESOURCE_AGENTS_EXTRA_DIR/usr/share/resource-agents/ocft/$file" \
222 "$RESOURCE_AGENTS_COMMON_DIR"/usr/share/resource-agents/ocft/
223 done
224diff --git a/debian/install-curated-agents b/debian/install-curated-agents
225new file mode 100755
226index 0000000..be9cdd5
227--- /dev/null
228+++ b/debian/install-curated-agents
229@@ -0,0 +1,22 @@
230+#!/bin/bash
231+
232+# This script aims to install all the agents curated by the Ubuntu community
233+# into resource-agents-base binary package.
234+
235+set -x
236+
237+readonly RESOURCE_AGENTS_BASE="debian/resource-agents-base"
238+readonly RESOURCE_AGENTS_EXTRA="debian/resource-agents-extra"
239+
240+mkdir -p $RESOURCE_AGENTS_BASE/usr/share/resource-agents/ocft/configs/ \
241+ $RESOURCE_AGENTS_BASE/usr/share/man/man7/ \
242+ $RESOURCE_AGENTS_BASE/usr/lib/ocf/resource.d/heartbeat/
243+
244+for agent in $(cat debian/curated-agents); do
245+ mv $RESOURCE_AGENTS_EXTRA/usr/lib/ocf/resource.d/heartbeat/$agent \
246+ $RESOURCE_AGENTS_BASE/usr/lib/ocf/resource.d/heartbeat/
247+ mv $RESOURCE_AGENTS_EXTRA/usr/share/man/man7/ocf_heartbeat_$agent.7 \
248+ $RESOURCE_AGENTS_BASE/usr/share/man/man7/
249+ mv $RESOURCE_AGENTS_EXTRA/usr/share/resource-agents/ocft/configs/$agent \
250+ $RESOURCE_AGENTS_BASE/usr/share/resource-agents/ocft/configs/
251+done
252diff --git a/debian/install-ldirectord-files b/debian/install-ldirectord-files
253index 32974fd..39c97d0 100755
254--- a/debian/install-ldirectord-files
255+++ b/debian/install-ldirectord-files
256@@ -4,7 +4,7 @@
257
258 set -x
259
260-readonly RESOURCE_AGENTS_DIR="debian/resource-agents"
261+readonly RESOURCE_AGENTS_EXTRA_DIR="debian/resource-agents-extra"
262 readonly LDIRECTORD_DIR="debian/ldirectord"
263
264 mkdir -p $LDIRECTORD_DIR/etc/ha.d/resource.d \
265@@ -15,12 +15,12 @@ mkdir -p $LDIRECTORD_DIR/etc/ha.d/resource.d \
266 $LDIRECTORD_DIR/usr/share/man/man8
267
268 cp debian/ldirectord.cf $LDIRECTORD_DIR/etc/ha.d
269-mv $RESOURCE_AGENTS_DIR/etc/ha.d/resource.d/ \
270+mv $RESOURCE_AGENTS_EXTRA_DIR/etc/ha.d/resource.d/ \
271 $LDIRECTORD_DIR/etc/ha.d/
272-mv $RESOURCE_AGENTS_DIR/etc/init.d $LDIRECTORD_DIR/etc/
273-mv $RESOURCE_AGENTS_DIR/etc/logrotate.d $LDIRECTORD_DIR/etc/
274-mv $RESOURCE_AGENTS_DIR/lib/systemd/system/ldirectord.service \
275+mv $RESOURCE_AGENTS_EXTRA_DIR/etc/init.d $LDIRECTORD_DIR/etc/
276+mv $RESOURCE_AGENTS_EXTRA_DIR/etc/logrotate.d $LDIRECTORD_DIR/etc/
277+mv $RESOURCE_AGENTS_EXTRA_DIR/lib/systemd/system/ldirectord.service \
278 $LDIRECTORD_DIR/lib/systemd/system/
279-mv $RESOURCE_AGENTS_DIR/usr/sbin/ldirectord $LDIRECTORD_DIR/usr/sbin
280-mv $RESOURCE_AGENTS_DIR/usr/share/man/man8/ldirectord.8 \
281+mv $RESOURCE_AGENTS_EXTRA_DIR/usr/sbin/ldirectord $LDIRECTORD_DIR/usr/sbin
282+mv $RESOURCE_AGENTS_EXTRA_DIR/usr/share/man/man8/ldirectord.8 \
283 $LDIRECTORD_DIR/usr/share/man/man8/
284diff --git a/debian/install-supported-agents b/debian/install-supported-agents
285deleted file mode 100755
286index 0ff0612..0000000
287--- a/debian/install-supported-agents
288+++ /dev/null
289@@ -1,22 +0,0 @@
290-#!/bin/bash
291-
292-# This script aims to install all the agents supported by the Ubuntu community
293-# into resource-agents-supported binary package.
294-
295-set -x
296-
297-readonly SUPPORTED_AGENTS="debian/resource-agents-supported"
298-readonly RESOURCE_AGENTS="debian/resource-agents"
299-
300-mkdir -p $SUPPORTED_AGENTS/usr/share/resource-agents/ocft/configs/ \
301- $SUPPORTED_AGENTS/usr/share/man/man7/ \
302- $SUPPORTED_AGENTS/usr/lib/ocf/resource.d/heartbeat/
303-
304-for agent in $(cat debian/supported-agents); do
305- mv $RESOURCE_AGENTS/usr/lib/ocf/resource.d/heartbeat/$agent \
306- $SUPPORTED_AGENTS/usr/lib/ocf/resource.d/heartbeat/
307- mv $RESOURCE_AGENTS/usr/share/man/man7/ocf_heartbeat_$agent.7 \
308- $SUPPORTED_AGENTS/usr/share/man/man7/
309- mv $RESOURCE_AGENTS/usr/share/resource-agents/ocft/configs/$agent \
310- $SUPPORTED_AGENTS/usr/share/resource-agents/ocft/configs/
311-done
312diff --git a/debian/resource-agents-cleanup b/debian/resource-agents-cleanup
313index 0134280..ad34bde 100755
314--- a/debian/resource-agents-cleanup
315+++ b/debian/resource-agents-cleanup
316@@ -2,12 +2,12 @@
317
318 set -x
319
320-readonly RESOURCE_AGENTS_DIR="debian/resource-agents"
321+readonly RESOURCE_AGENTS_EXTRA_DIR="debian/resource-agents-extra"
322
323 # Those directories were moved to another binary package
324 readonly EMPTY_DIRS="etc lib run usr/include usr/lib/heartbeat usr/sbin
325 usr/share/man/man8 usr/share/pkgconfig usr/lib/tmpfiles.d"
326
327 for dir in $EMPTY_DIRS; do
328- rm -rf $RESOURCE_AGENTS_DIR/$dir
329+ rm -rf $RESOURCE_AGENTS_EXTRA_DIR/$dir
330 done
331diff --git a/debian/resource-agents-extra.lintian-overrides b/debian/resource-agents-extra.lintian-overrides
332new file mode 100644
333index 0000000..b5de12c
334--- /dev/null
335+++ b/debian/resource-agents-extra.lintian-overrides
336@@ -0,0 +1,5 @@
337+# ignore manpage warnings
338+resource-agents-extra: groff-message usr/share/man/man7/ocf_heartbeat_openstack-info.7.gz 37: warning [p 1, 2.3i]: can't break line
339+
340+# ignore notices about usr/lib
341+resource-agents-extra: executable-in-usr-lib usr/lib/ocf/resource.d/heartbeat/*
342diff --git a/debian/resource-agents.lintian-overrides b/debian/resource-agents.lintian-overrides
343deleted file mode 100644
344index 26a2035..0000000
345--- a/debian/resource-agents.lintian-overrides
346+++ /dev/null
347@@ -1,5 +0,0 @@
348-# ignore manpage warnings
349-resource-agents: groff-message usr/share/man/man7/ocf_heartbeat_openstack-info.7.gz 37: warning [p 1, 2.3i]: can't break line
350-
351-# ignore notices about usr/lib
352-resource-agents: executable-in-usr-lib usr/lib/ocf/resource.d/heartbeat/*
353diff --git a/debian/rules b/debian/rules
354index 9e5f9ce..181c55f 100755
355--- a/debian/rules
356+++ b/debian/rules
357@@ -5,7 +5,7 @@ DPKG_EXPORT_BUILDFLAGS = 1
358 include /usr/share/dpkg/buildflags.mk
359 include /usr/share/dpkg/pkg-info.mk
360
361-RESOURCE_AGENTS_DIR="$(CURDIR)/debian/resource-agents"
362+RESOURCE_AGENTS_EXTRA_DIR="$(CURDIR)/debian/resource-agents-extra"
363
364 %:
365 dh $@ --with python3
366@@ -28,12 +28,12 @@ override_dh_auto_configure:
367 REBOOT=/sbin/reboot POWEROFF_CMD=/sbin/poweroff
368
369 override_dh_auto_install:
370- $(MAKE) install DESTDIR=$(RESOURCE_AGENTS_DIR)
371+ $(MAKE) install DESTDIR=$(RESOURCE_AGENTS_EXTRA_DIR)
372
373 override_dh_install:
374 debian/install-ldirectord-files
375 debian/install-common-files
376- debian/install-supported-agents
377+ debian/install-curated-agents
378 debian/resource-agents-cleanup
379
380 dh_install
381diff --git a/debian/tests/control b/debian/tests/control
382index 4905200..4b83b1d 100644
383--- a/debian/tests/control
384+++ b/debian/tests/control
385@@ -1,5 +1,5 @@
386 Tests: IPaddr2
387-Depends: resource-agents-supported
388+Depends: resource-agents-base
389 Restrictions: needs-root, isolation-container
390
391 Test-Command: service ldirectord status

Subscribers

People subscribed via source and target branches