Deprecate snapshots.read_only_directory configuration option#7800
Merged
Deprecate snapshots.read_only_directory configuration option#7800
snapshots.read_only_directory configuration option#7800Conversation
Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/035db06d-7ede-42c6-9ad5-0d943a6c0129 Co-authored-by: eddyashton <6000239+eddyashton@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Deprecate snapshot read only dirs configuration
Deprecate Apr 1, 2026
snapshots.read_only_directory configuration option
eddyashton
reviewed
Apr 1, 2026
…ecation notice Agent-Logs-Url: https://github.com/microsoft/CCF/sessions/52fe1027-e446-4c81-aeaa-b182e4170a47 Co-authored-by: eddyashton <6000239+eddyashton@users.noreply.github.com>
achamayou
reviewed
Apr 1, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Deprecates the snapshots.read_only_directory host configuration option ahead of planned removal, and informs operators via schema notes, startup logging, and changelog entry.
Changes:
- Marked
snapshots.read_only_directoryas deprecated in the host config JSON schema. - Added a startup log when
snapshots.read_only_directoryis configured. - Documented the deprecation under
[7.0.0-rc1]in the changelog.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/host/run.cpp | Logs at startup when deprecated config option is present. |
| doc/host_config_schema/cchost_config.json | Updates schema field description to explicitly state deprecation and suggested alternatives. |
| CHANGELOG.md | Adds release note describing deprecation and recommended replacement behavior. |
achamayou
reviewed
Apr 1, 2026
cjen1-msft
approved these changes
Apr 1, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
snapshots.read_only_directoryis being deprecated ahead of removal in a future release.Changes
doc/host_config_schema/cchost_config.json): Updated theread_only_directoryfield description to note it is deprecated.src/host/run.cpp): Emits aLOG_FAIL_FMTwarning at startup ifsnapshots.read_only_directoryis set in the config.Deprecatedentry under[7.0.0-rc1].