Skip to content

Conversation

@Ahmath-Gadji
Copy link
Contributor

This pull request updates the smoke test workflow to make sure the environment variables from the .env file are available during the backup and restore steps. In some cases, this is much needed
The main change is adding the --env-file .env option to the relevant docker compose run commands.

@coderabbitai
Copy link

coderabbitai bot commented Dec 10, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch backup_restore_env_file

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Ahmath-Gadji Ahmath-Gadji marked this pull request as ready for review December 10, 2025 16:23
Copy link
Contributor

@dodekapod dodekapod left a comment

Choose a reason for hiding this comment

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

To do :

  • update backup/restore docs
  • update examples in (openrag/scripts)

@Ahmath-Gadji
Copy link
Contributor Author

@dodekapod the documentation and the examples have been updated

Copy link
Contributor

@dodekapod dodekapod left a comment

Choose a reason for hiding this comment

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

LGTM

@Ahmath-Gadji Ahmath-Gadji merged commit fa2d7d7 into dev Dec 11, 2025
4 checks passed
@Ahmath-Gadji Ahmath-Gadji deleted the backup_restore_env_file branch December 11, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants