Skip to content

Commit 75eb71d

Browse files
committed
Add server maint role, drop tool indexing
We add the notion of a maintenance role for the audit service, and drop the tool data indexing by default. We add commented out `pbench-server-backup` and `Indexing` sections to the default server configuration file with a comment that they are deliberately omitted from the defaults. This avoids making any code changes to accommodate always having a default. We drop the default "vos" user for sosreports and use the default user. This let's us remove a few unnecessary config files and their symlinks from the unit tests. We also take this opportunity to make sure all the configuration files in the tree a properly sync'd with each other.
1 parent 9acc723 commit 75eb71d

File tree

18 files changed

+105
-122
lines changed

18 files changed

+105
-122
lines changed

server/ansible/Inventory/pbench-server.hosts.example

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,4 @@
1818
pbench_config_url = http://pbench.example.com/server/config/{{ cenv }}
1919

2020
# list of config files to fetch
21-
pbench_config_files = '["pbench-server.cfg", "pbench-server-backup.cfg"]'
22-
23-
21+
pbench_config_files = '["pbench-server.cfg"]'

server/bin/gold/test-24.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77
_PBENCH_SERVER_CONFIG=/var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/config/pbench-server.cfg
88
99
10+
1 3 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-audit-server
1011
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-prep-002.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-server-prep-shim-002
1112
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-prep-003.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-server-prep-shim-003
1213
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-dispatch.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-dispatch
@@ -15,10 +16,8 @@ [email protected]
1516
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-unpack-tarballs-large.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-unpack-tarballs large
1617
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-unpack-tarballs-huge.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-unpack-tarballs huge
1718
1 2 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-cull-unpacked-tarballs.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-cull-unpacked-tarballs
18-
1 3 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-audit-server
1919
23 * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-copy-sosreports.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-copy-sosreports
2020
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-index.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-index
21-
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-index-tool-data.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-index --tool-data
2221
53 4 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-backup-tarballs.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-backup-tarballs
2322
53 5 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-verify-backup-tarballs.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-verify-backup-tarballs
2423
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server/lib/locks/pbench-sync-satellite-ONE.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server/bin/pbench-sync-satellite satellite-one
@@ -27,6 +26,7 @@ [email protected]
2726
_PBENCH_SERVER_CONFIG=/var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/lib/config/pbench-server.cfg
2827
2928
29+
1 3 * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/bin/pbench-audit-server
3030
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/lib/locks/pbench-prep-002.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/bin/pbench-server-prep-shim-002
3131
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/lib/locks/pbench-dispatch.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/bin/pbench-dispatch
3232
* * * * * flock -n /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/lib/locks/pbench-unpack-tarballs.lock /var/tmp/pbench-test-server/test-24/opt/pbench-server-satellite/bin/pbench-unpack-tarballs

server/bin/gold/test-8.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,17 +7,16 @@
77
_PBENCH_SERVER_CONFIG=/var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/config/pbench-server.cfg
88
99
10+
1 3 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-audit-server
1011
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-prep-002.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-server-prep-shim-002
1112
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-dispatch.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-dispatch
1213
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-unpack-tarballs-small.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-unpack-tarballs small
1314
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-unpack-tarballs-medium.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-unpack-tarballs medium
1415
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-unpack-tarballs-large.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-unpack-tarballs large
1516
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-unpack-tarballs-huge.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-unpack-tarballs huge
1617
1 2 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-cull-unpacked-tarballs.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-cull-unpacked-tarballs
17-
1 3 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-audit-server
1818
23 * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-copy-sosreports.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-copy-sosreports
1919
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-index.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-index
20-
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-index-tool-data.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-index --tool-data
2120
53 4 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-backup-tarballs.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-backup-tarballs
2221
53 5 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-verify-backup-tarballs.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-verify-backup-tarballs
2322
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server/lib/locks/pbench-sync-satellite-ONE.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server/bin/pbench-sync-satellite satellite-one
@@ -26,6 +25,7 @@ [email protected]
2625
_PBENCH_SERVER_CONFIG=/var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/lib/config/pbench-server.cfg
2726
2827
28+
1 3 * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/lib/locks/pbench-audit-server.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/bin/pbench-audit-server
2929
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/lib/locks/pbench-prep-002.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/bin/pbench-server-prep-shim-002
3030
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/lib/locks/pbench-dispatch.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/bin/pbench-dispatch
3131
* * * * * flock -n /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/lib/locks/pbench-unpack-tarballs.lock /var/tmp/pbench-test-server/test-8/opt/pbench-server-satellite/bin/pbench-unpack-tarballs

server/bin/state/config-satellite/pbench-server.cfg

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,24 @@
1-
install-dir = %(unittest-dir)s/opt/pbench-server-satellite
21
default-host = pbench-satellite.example.com
2+
install-dir = %(unittest-dir)s/opt/pbench-server-satellite
33

44
###########################################################################
55
## Deployment section
66
###########################################################################
77
[pbench-server]
8-
commit_id = unit-test
9-
environment = unit-test
10-
admin-email = [email protected]
118
pbench-top-dir = %(unittest-dir)s/pbench-satellite
129
pbench-local-dir = %(unittest-dir)s/pbench-satellite-local
13-
# A satellite just unpacks and syncs
10+
# An example satellite might override the default roles to drop the backup role.
11+
roles = pbench-maintenance, pbench-prep, pbench-results
12+
# An example satellite that just unpacks and syncs
1413
dispatch-states = TO-UNPACK, TO-SYNC
15-
# A satellite just needs to prep and process results, no backup necessary
16-
roles = pbench-prep, pbench-results
1714
# This has to be set to the same value as what the mock'd "hostname"
1815
# command returns.
1916
realhost = pbench.example.com
17+
# Unit test environment configuration values
2018
debug_unittest = True
19+
commit_id = unit-test
20+
environment = unit-test
21+
admin-email = [email protected]
2122

2223
[apache]
2324
documentroot = %(unittest-dir)s/var-www-html-satellite
@@ -27,12 +28,11 @@ logger_type = file
2728

2829
###########################################################################
2930
# crontab roles
30-
#
31-
# For a satellite server we typically don't need to index results or handle
32-
# sosreports, so we override the tasks to drop those two.
3331
[pbench-results]
34-
# The results role on a satellite does not need to copy sosreports or index,
35-
# but we do need to make sure we run sync-cleanup.
32+
# The results role on our example satellite does not need to copy sosreports,
33+
# index data, or cull unpacked tar balls, but we do need to make sure we run
34+
# sync-cleanup. As a satellite, we also only need at most one process
35+
# unpacking tar balls.
3636
tasks = pbench-dispatch, pbench-unpack-tarballs, pbench-sync-cleanup
3737

3838
###########################################################################

server/bin/state/config/pbench-server.cfg

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,21 @@ install-dir = %(unittest-dir)s/opt/pbench-server
44
## Deployment section
55
###########################################################################
66
[pbench-server]
7-
commit_id = unit-test
8-
environment = unit-test
9-
admin-email = [email protected]
107
pbench-top-dir = %(unittest-dir)s/pbench
118
pbench-local-dir = %(unittest-dir)s/pbench-local
129
pbench-backup-dir = %(pbench-local-dir)s/archive.backup
1310
# Add role for sync'ing with satellites
14-
roles = pbench-prep, pbench-results, pbench-backup, pbench-sync-satellites
11+
roles = pbench-maintenance, pbench-prep, pbench-results, pbench-backup, pbench-sync-satellites
1512
debug_unittest = True
13+
commit_id = unit-test
14+
environment = unit-test
15+
admin-email = [email protected]
16+
17+
[pbench-server-backup]
18+
endpoint_url = %(unittest-dir)s/pbench-local/s3-backup
19+
access_key_id = ACCESS_KEY_ID
20+
secret_access_key = SECRET_ACCESS_KEY
21+
bucket_name = testbucket
1622

1723
[Indexing]
1824
server = elasticsearch.example.com:9280
@@ -27,11 +33,6 @@ logger_type = file
2733

2834
###########################################################################
2935
# crontab roles
30-
[pbench-server-backup]
31-
endpoint_url = %(unittest-dir)s/pbench-local/s3-backup
32-
bucket_name = testbucket
33-
access_key_id = ACCESS_KEY_ID
34-
secret_access_key = SECRET_ACCESS_KEY
3536

3637
# The definition of the crontab role for sync'ing satellite pbench servers.
3738
[pbench-sync-satellites]

server/bin/state/test-17.config

Lines changed: 0 additions & 1 deletion
This file was deleted.

server/bin/state/test-20.config

Lines changed: 0 additions & 1 deletion
This file was deleted.

server/bin/state/test-23.config/pbench-server.cfg

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,21 @@ install-dir = %(unittest-dir)s/opt/pbench-server
44
## Deployment section
55
###########################################################################
66
[pbench-server]
7-
commit_id = unit-test
8-
environment = unit-test
9-
admin-email = [email protected]
107
pbench-top-dir = %(unittest-dir)s/pbench
118
pbench-local-dir = %(unittest-dir)s/pbench-local
129
pbench-backup-dir = %(pbench-local-dir)s/archive.backup
1310
# Add role for sync'ing with satellites
14-
roles = pbench-prep, pbench-results, pbench-backup, pbench-sync-satellites
11+
roles = pbench-maintenance, pbench-prep, pbench-results, pbench-backup, pbench-sync-satellites
1512
debug_unittest = True
13+
commit_id = unit-test
14+
environment = unit-test
15+
admin-email = [email protected]
16+
17+
[pbench-server-backup]
18+
endpoint_url = %(unittest-dir)s/pbench-local/s3-backup
19+
access_key_id = ACCESS_KEY_ID
20+
secret_access_key = SECRET_ACCESS_KEY
21+
bucket_name = testbucket
1622

1723
[Indexing]
1824
server = elasticsearch.example.com:9280
@@ -26,6 +32,7 @@ logger_type = file
2632

2733
###########################################################################
2834
# crontab roles
35+
2936
# The definition of the crontab role for sync'ing satellite pbench servers.
3037
[pbench-sync-satellites]
3138
host = %(default-host)s

server/bin/state/test-26.1.config/pbench-server.cfg

Lines changed: 0 additions & 10 deletions
This file was deleted.

server/bin/state/test-27.config/pbench-server.cfg

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,6 @@ install-dir = %(unittest-dir)s/opt/pbench-server
33
[pbench-server]
44
debug_ref_datetime = 1970-02-14T00:00:00.000000
55

6-
###########################################################################
7-
## runtime section
8-
###########################################################################
9-
[sosreport]
10-
user = %(default-user)s
11-
host = pbench-sosreports.example.com
12-
dir = /path/to/sosreport/dir
13-
146
###########################################################################
157
# The rest will come from the global state config file and the default config file.
168
[config]

0 commit comments

Comments
 (0)