Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Automatically sync the configuration on server startup #2432

Merged
merged 2 commits into from
Mar 1, 2024
Merged

Automatically sync the configuration on server startup #2432

merged 2 commits into from
Mar 1, 2024

Conversation

sandhose
Copy link
Member

@sandhose sandhose commented Mar 1, 2024

This does two things:

  • automatically sync the configuration when starting the server (unless the --no-sync option is used)
  • automatically run migrations before running mas-cli config sync

@sandhose sandhose added A-Configuration Related on what is configurable and how it can be configured A-CLI Related to the command line utility labels Mar 1, 2024
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Mar 1, 2024 •
edited
Loading

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 09c680b
Status: ✅  Deploy successful!
Preview URL: https://23b4cf17.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://quenting-config-auto-sync.matrix-authentication-service-docs.pages.dev

View logs

@sandhose sandhose marked this pull request as ready for review March 1, 2024 14:36
@sandhose sandhose mentioned this pull request Mar 1, 2024
Merged
AndrewFerr
AndrewFerr suggested changes Mar 1, 2024
View reviewed changes
Copy link
Member

@AndrewFerr AndrewFerr left a comment

Choose a reason for hiding this comment

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

Thanks for this! I tested it locally and it works for me.

I have only a minor change request to the docs. (The Rust code is beyond my expertise at the moment.)

docs/setup/general.md Outdated Show resolved Hide resolved
@sandhose sandhose merged commit 7f9d62f into main Mar 1, 2024
16 checks passed
@sandhose sandhose deleted the quenting/config-auto-sync branch March 19, 2024 10:55
@matrixbot matrixbot mentioned this pull request Sep 10, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@AndrewFerr AndrewFerr AndrewFerr requested changes

Assignees
No one assigned
Labels
A-CLI Related to the command line utility A-Configuration Related on what is configurable and how it can be configured
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sandhose @AndrewFerr