Skip to content

fix: remove os data as part of reset #1983

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.

Sign up for GitHub

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

Merged
edwinjosechittilappilly merged 4 commits into from
Jun 30, 2026
Merged

fix: remove os data as part of reset #1983

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3fd8b61
remove os data
edwinjosechittilappilly Jun 29, 2026
656be65
Merge branch 'main' into fix-reset-remove-os-data
edwinjosechittilappilly Jun 30, 2026
1a69b59
review comments
edwinjosechittilappilly Jun 30, 2026
8e57f8d
Merge branch 'main' into fix-reset-remove-os-data
edwinjosechittilappilly Jun 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -262,6 +262,8 @@ make clear-os-data # Clear OpenSearch data directory completely
make factory-reset # Complete reset: containers, volumes, and data
```

The reset also removes any legacy `opensearch-data` directory, so old OpenSearch index files do not linger after cleanup.

---

## Makefile Help System
Expand Down
9 changes: 9 additions & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -591,6 +591,7 @@ factory-reset: ## Complete reset (stop, remove volumes, clear data, remove image
echo " - Delete langflow-data directory"; \
echo " - Delete config directory"; \
echo " - Delete data directory (database and session configs)"; \
echo " - Delete opensearch-data directory (legacy OpenSearch bind-mount data)"; \
echo " - Delete JWT keys (private_key.pem, public_key.pem)"; \
echo " - Remove OpenRAG images"; \
echo ""; \
Expand Down Expand Up @@ -621,6 +622,14 @@ factory-reset: ## Complete reset (stop, remove volumes, clear data, remove image
rm -rf data; \
echo "$(PURPLE)data removed$(NC)"; \
fi; \
if [ -d "opensearch-data" ]; then \
echo "Removing opensearch-data..."; \
if rm -rf opensearch-data; then \
echo "$(PURPLE)opensearch-data removed$(NC)"; \
else \
echo "$(RED)Warning: Failed to remove opensearch-data (check permissions)$(NC)"; \
fi; \
fi; \
Comment thread
coderabbitai[bot] marked this conversation as resolved.
if [ -n "$$OPENRAG_DATA_PATH" ] && [ -d "$$OPENRAG_DATA_PATH" ]; then \
echo "Removing $$OPENRAG_DATA_PATH..."; \
rm -rf "$$OPENRAG_DATA_PATH"; \
Expand Down
3 changes: 2 additions & 1 deletion docs/docs/_partial-factory-reset-warning.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ This is a destructive operation that does the following:
* Destroys all OpenRAG containers, volumes, and local images.
* Prunes any additional container objects.
* Deletes the contents of the `~/.openrag` directory _except_ for OpenRAG's `.env` file and the `/documents` subdirectory.
* Removes any legacy `opensearch-data` directory so old index data does not survive a reset.

<p/>Destroyed containers and deleted data are lost and cannot be recovered after running this operation.
:::
Expand All @@ -19,4 +20,4 @@ Neither <br/> nor wrapping the entire line in <p> </p> worked consistently for b
Either the line break was missing or the indentation was incorrect.

This behavior was observed in Docusaurus 3.9.2 on 05 Dec 2025. In a future release, if this is not longer an issue, you can remove the tag and this note. :)
-->
-->
Loading