Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use secret password for admin user on test deployment #1135

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

LukasKalbertodt
Copy link
Member

Unfortunately it is time. We all knew this was coming but last night someone deleted all pages from our test deployment. That's really not a big problem, as they are recreated from a DB dump on every deploy anyway. It's still annoying (and made us think our deployment partially failed :D).

(Not exactly sure in what category in the changelog this belongs... admins surely test stuff on our test deployment as well)

The test deployment for this PR probably fails or won't show the right behavior as some scripts from master are used. But I deployed this manually: https://testlukas.tobira.opencast.org/ (password for admin there is hund123 ... which won't be the password for the actual deployment :D)

Unfortunately it is time. We all knew this was coming but last night
someone deleted all pages from our test deployment. That's really not
a big problem, as they are recreated from a DB dump on every deploy
anyway. It's still annoying (and made us think our deployment partially
failed :D).

(...)
@LukasKalbertodt LukasKalbertodt added the changelog:dev Changes primarily for developers label Mar 6, 2024
@github-actions github-actions bot temporarily deployed to test-deployment-pr1135 March 6, 2024 14:23 Destroyed
Copy link
Member

@owi92 owi92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@owi92 owi92 merged commit 2c83d20 into elan-ev:master Mar 7, 2024
4 checks passed
@LukasKalbertodt LukasKalbertodt deleted the protected-admin-test-deploy branch March 7, 2024 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:dev Changes primarily for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants