Skip to content

Commit 59d7ff9

Browse files
authored
Merge branch 'master' into hackathon-restore-check
2 parents c163175 + 059c08b commit 59d7ff9

File tree

8 files changed

+60
-5
lines changed

8 files changed

+60
-5
lines changed

.github/workflows/docker-image.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ jobs:
1515
steps:
1616
- uses: actions/checkout@v3
1717
- name: Build the Debian Docker image
18-
run: docker build . --file Dockerfile --tag backup-utils-debian:${GITHUB_RUN_ID}
18+
run: docker build . --file Dockerfile --tag backup-utils:${GITHUB_RUN_ID}
1919
- name: Build the Alpine Docker image
2020
run: docker build . --file Dockerfile.alpine --tag backup-utils-alpine:${GITHUB_RUN_ID}
2121
- name: Run tests in Debian Docker image
22-
run: docker run backup-utils-debian:${GITHUB_RUN_ID} ghe-backup --version
22+
run: docker run backup-utils:${GITHUB_RUN_ID} ghe-backup --version
2323
- name: Run tests in Alpine Docker image
2424
run: docker run backup-utils-alpine:${GITHUB_RUN_ID} ghe-backup --version
2525

.github/workflows/stale-support-escalation.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ jobs:
1515
if: github.repository == 'github/ghes' || github.repository == 'github/enterprise2' || github.repository == 'github/backup-utils-private'
1616
runs-on: ubuntu-latest
1717
steps:
18-
- uses: actions/stale@v4
18+
- uses: actions/stale@v7
1919
with:
2020
repo-token: ${{ secrets.GITHUB_TOKEN }}
2121
stale-issue-message: 'This support escalation issue is stale because it has been open 30 days with no activity. To make it never stale, add a label never-stale.'

.github/workflows/stale.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jobs:
1111
runs-on: ubuntu-latest
1212

1313
steps:
14-
- uses: actions/stale@v3
14+
- uses: actions/stale@v7
1515
with:
1616
repo-token: ${{ secrets.GITHUB_TOKEN }}
1717
stale-issue-message: "👋 This issue has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the `Keep` label to hold stale off permanently, or do nothing. If you do nothing this issue will be closed eventually by the stale bot."

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
FROM debian:buster-slim
1+
FROM ubuntu:focal
22

33
RUN apt-get -q -y update && \
44
apt-get install -y --no-install-recommends \

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

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@ backup-secret "management console password" "manage-password" "secrets.manage"
7878
backup-secret "management console argon2 secret" "manage-argon-secret" "secrets.manage-auth.argon-secret"
7979
backup-secret "password pepper" "password-pepper" "secrets.github.user-password-secrets"
8080
backup-secret "kredz.credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
81+
backup-secret "kredz.varz HMAC key" "kredz-varz-hmac" "secrets.kredz.varz-hmac-secret"
8182

8283
# Backup external MySQL password if running external MySQL DB.
8384
if is_service_external 'mysql'; then

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

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,9 @@ restore-secret "management console argon2 secret" "manage-argon-secret" "secrets
5050
# Restore kredz.credz HMAC key if present.
5151
restore-secret "kredz.credz HMAC key" "kredz-credz-hmac" "secrets.kredz.credz-hmac-secret"
5252

53+
# Restore kredz.varz HMAC key if present.
54+
restore-secret "kredz.varz HMAC key" "kredz-varz-hmac" "secrets.kredz.varz-hmac-secret"
55+
5356
# Restore SAML keys if present.
5457
if [ -f "$GHE_RESTORE_SNAPSHOT_PATH/saml-keys.tar" ]; then
5558
echo "Restoring SAML keys ..."

test/test-ghe-backup.sh

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -506,6 +506,31 @@ begin_test "ghe-backup takes backup of Kredz settings"
506506
)
507507
end_test
508508

509+
begin_test "ghe-backup takes backup of kredz-varz settings"
510+
(
511+
set -e
512+
513+
required_secrets=(
514+
"secrets.kredz.varz-hmac-secret"
515+
)
516+
517+
for secret in "${required_secrets[@]}"; do
518+
ghe-ssh "$GHE_HOSTNAME" -- ghe-config "$secret" "foo"
519+
done
520+
521+
ghe-backup
522+
523+
required_files=(
524+
"kredz-varz-hmac"
525+
)
526+
527+
for file in "${required_files[@]}"; do
528+
[ "$(cat "$GHE_DATA_DIR/current/$file")" = "foo" ]
529+
done
530+
531+
)
532+
end_test
533+
509534
begin_test "ghe-backup takes backup of Actions settings"
510535
(
511536
set -e

test/test-ghe-restore.sh

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -336,6 +336,32 @@ begin_test "ghe-restore with Kredz settings"
336336
)
337337
end_test
338338

339+
begin_test "ghe-restore with kredz-varz settings"
340+
(
341+
set -e
342+
rm -rf "$GHE_REMOTE_ROOT_DIR"
343+
setup_remote_metadata
344+
enable_actions
345+
346+
required_files=(
347+
"kredz-varz-hmac"
348+
)
349+
350+
for file in "${required_files[@]}"; do
351+
echo "foo" > "$GHE_DATA_DIR/current/$file"
352+
done
353+
354+
ghe-restore -v -f localhost
355+
required_secrets=(
356+
"secrets.kredz.varz-hmac-secret"
357+
)
358+
359+
for secret in "${required_secrets[@]}"; do
360+
[ "$(ghe-ssh "$GHE_HOSTNAME" -- ghe-config "$secret")" = "foo" ]
361+
done
362+
)
363+
end_test
364+
339365
begin_test "ghe-restore with Actions settings"
340366
(
341367
set -e

0 commit comments

Comments
 (0)