Skip to content

Add readonly_supported to readme-vars.yml#92

Closed
ghost wants to merge 1 commit intomasterfrom
unknown repository
Closed

Add readonly_supported to readme-vars.yml#92
ghost wants to merge 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jun 26, 2025

linuxserver.io


  • I have read the contributing guideline and understand that I have made the correct modifications

Description:

From my own testing it seems like running this container in read_only mode works without any issues so I've updated the readme template to signal this to users. It's possible that my use case for Syncthing does not cover edge cases that causes problems with read_only mode. Please let me know and disregard this pull request if so.

Benefits of this PR and context:

It reflects your efforts to mark containers compatible with read_only mode as such.

How Has This Been Tested?

I added read_only: true to the containers docker compose file and did some basic testing to make sure that no functionality had been lost and that no errors or warnings showed up in the logs. Syncing files worked as expected.

Source / References:

#91

Copy link
Copy Markdown

@github-actions github-actions Bot 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 opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/syncthing/v1.29.7-pkg-c5625171-dev-530aebf4e397a39a169329dbed0825f8285b9b8d-pr-92/index.html
https://ci-tests.linuxserver.io/lspipepr/syncthing/v1.29.7-pkg-c5625171-dev-530aebf4e397a39a169329dbed0825f8285b9b8d-pr-92/shellcheck-result.xml

Tag Passed
amd64-v1.29.7-pkg-c5625171-dev-530aebf4e397a39a169329dbed0825f8285b9b8d-pr-92
arm64v8-v1.29.7-pkg-c5625171-dev-530aebf4e397a39a169329dbed0825f8285b9b8d-pr-92

@LinuxServer-CI
Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/syncthing/v1.30.0-pkg-1a6a0c81-dev-3644b978f0320a9a15a231452f4c5eef42469552-pr-92/index.html
https://ci-tests.linuxserver.io/lspipepr/syncthing/v1.30.0-pkg-1a6a0c81-dev-3644b978f0320a9a15a231452f4c5eef42469552-pr-92/shellcheck-result.xml

Tag Passed
amd64-v1.30.0-pkg-1a6a0c81-dev-3644b978f0320a9a15a231452f4c5eef42469552-pr-92
arm64v8-v1.30.0-pkg-1a6a0c81-dev-3644b978f0320a9a15a231452f4c5eef42469552-pr-92

@ghost ghost closed this by deleting the head repository Aug 5, 2025
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant