Skip to content

Commit b408e20

Browse files
committed
backup
1 parent cc0fe1f commit b408e20

File tree

4 files changed

+27
-5
lines changed

4 files changed

+27
-5
lines changed

share/github-backup-utils/ghe-backup-settings

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@ backup-secret() {
7676

7777
backup-secret "management console password" "manage-password" "secrets.manage"
7878
backup-secret "password pepper" "password-pepper" "secrets.github.user-password-secrets"
79+
backup-secret "Kredz credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
7980

8081
# Backup external MySQL password if running external MySQL DB.
8182
if is_service_external 'mysql'; then
@@ -113,8 +114,6 @@ if ghe-ssh "$host" -- ghe-config --true app.actions.enabled; then
113114
backup-secret "Actions Launch action runner secret" "actions-launch-action-runner-secret" "secrets.launch.action-runner-secret"
114115
backup-secret "Actions Launch service cert" "actions-launch-azp-app-cert" "secrets.launch.azp-app-cert"
115116
backup-secret "Actions Launch service private key" "actions-launch-app-app-private-key" "secrets.launch.azp-app-private-key"
116-
117-
backup-secret "Kredz credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
118117
fi
119118

120119
if ghe-ssh "$host" -- "test -f $GHE_REMOTE_DATA_USER_DIR/common/idp.crt"; then

share/github-backup-utils/ghe-restore-actions

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,6 @@ restore-secret "Actions Launch service private key" "actions-launch-app-app-priv
8585
restore-secret "Actions Launch token oauth key" "actions-oauth-s2s-signing-key" "secrets.launch.token-oauth-key"
8686
restore-secret "Actions Launch token oauth cert" "actions-oauth-s2s-signing-cert" "secrets.launch.token-oauth-cert"
8787

88-
restore-secret "Kredz credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
8988

9089
# Setup the database logins.
9190
ghe_verbose "* Restoring database logins and users to $host ..."

share/github-backup-utils/ghe-restore-settings

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ restore-secret "external MySQL password" "external-mysql-password" "secrets.exte
4040

4141
# Restore management console password hash if present.
4242
restore-secret "management console password" "manage-password" "secrets.manage"
43+
restore-secret "Kredz credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
4344

4445
# Restore SAML keys if present.
4546
if [ -f "$GHE_RESTORE_SNAPSHOT_PATH/saml-keys.tar" ]; then

test/test-ghe-backup.sh

Lines changed: 25 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -470,6 +470,31 @@ begin_test "ghe-backup upgrades transaction backup to full if LSN chain break"
470470
)
471471
end_test
472472

473+
begin_test "ghe-backup takes backup of Kredz settings"
474+
(
475+
set -e
476+
477+
required_secrets=(
478+
"secrets.kredz.credz-hmac-secret"
479+
)
480+
481+
for secret in "${required_secrets[@]}"; do
482+
ghe-ssh "$GHE_HOSTNAME" -- ghe-config "$secret" "foo"
483+
done
484+
485+
ghe-backup
486+
487+
required_files=(
488+
"kredz-credz-hmac"
489+
)
490+
491+
for file in "${required_files[@]}"; do
492+
[ "$(cat "$GHE_DATA_DIR/current/$file")" = "foo" ]
493+
done
494+
495+
)
496+
end_test
497+
473498
begin_test "ghe-backup takes backup of Actions settings"
474499
(
475500
set -e
@@ -507,7 +532,6 @@ begin_test "ghe-backup takes backup of Actions settings"
507532
"secrets.launch.azp-app-cert"
508533
"secrets.launch.azp-app-private-key"
509534

510-
"secrets.kredz.credz-hmac-secret"
511535
)
512536

513537
# these 5 were removed in later versions, so we extract them as best effort
@@ -551,7 +575,6 @@ begin_test "ghe-backup takes backup of Actions settings"
551575
"actions-launch-azp-app-cert"
552576
"actions-launch-app-app-private-key"
553577

554-
"kredz-credz-hmac"
555578
)
556579

557580
# Add the one optional file we included tests for

0 commit comments

Comments
 (0)