Skip to content
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.

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

Fix restore of RETENTION_DAYS variable #20

Merged
merged 4 commits into from
May 31, 2024
Merged

Conversation

stephdl
Copy link
Collaborator

@stephdl stephdl commented May 30, 2024

The code changes modify the handling of the RETENTION_DAYS environment variable in the clone-module and restore-module actions. The variable is now converted to an integer before being used.

The unused json property restore has been removed

as a side note backup/restore and clone must be tested again

Refs NethServer/dev#6895

The code changes modify the handling of the `RETENTION_DAYS` environment variable in the `clone-module` and `restore-module` actions. The variable is now converted to an integer before being used. This update improves the consistency and reliability of the code.
@stephdl stephdl self-assigned this May 30, 2024
…ept only for restoration/clone action

The code changes remove unnecessary code related to altering relay rules and setting always-bcc address for the mail server in the `configure-module` script. This simplifies the script and improves its efficiency.
@stephdl
Copy link
Collaborator Author

stephdl commented May 30, 2024

@DavidePrincipi some little changes

  • we configure the mail server the first time if MAIL_SERVER does not exist
  • we configure the mail server again only if we are in a restore or clone action (we send "restore": true in additional property)
  • the property retention_days added to clone and restore action

clone action and restoration are good

@DavidePrincipi DavidePrincipi changed the title Update environment variable handling for retention days Fix restore of RETENTION_DAYS variable May 31, 2024
@DavidePrincipi DavidePrincipi merged commit 863b5e7 into main May 31, 2024
1 check passed
@DavidePrincipi DavidePrincipi deleted the retention_days branch May 31, 2024 07:43
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.

2 participants