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

[BUG] - Tauticord tries to create a migration file when there are no environment variables #194

Closed
1 of 2 tasks
vember31 opened this issue Mar 31, 2024 · 2 comments · Fixed by #195
Closed
1 of 2 tasks
Labels
bug A provable error exists in the code resolved Has been fixed or completed

Comments

@vember31
Copy link

This bug has not been reported already

  • I have verified that this bug has not already been reported.

You have tried to fix the bug yourself

  • I have tried to fix the bug myself.

Current Behavior

Upon upgrading to 4.2.1, even if no environment variables are present in the deployment, it looks like Tauticord is trying to perform a migration to a config file. I recently transitioned everything to the config file manually (a couple of Tauticord versions ago), so don't need Tauticord to perform the migration. The main issue though is it's running into an issue trying to perform a migration.

After upgrading to 4.2.1 I'm running into the errors below, and the container continues to restart itself. Rolling back to 4.2.0 has solved the issue and Tauticord is able to launch.

2024-03-31 02:17:54,302 - [INFO]: Migration 001: Copying environment variables to /config/migration_data/001_env_var_to_config_yaml.yaml
2024-03-31 02:17:54,303 - [INFO]: WARNING: Environment variable configuration is going away soon! Please use a config file instead.
2024-03-31 02:17:54,307 - [ERROR]: Migration 001 failed: [Errno 2] No such file or directory: '/config/migration_data/001_env_var_to_config_yaml.yaml'
2024-03-31 02:17:54,307 - [CRITICAL]: Migrations failed. Exiting...

Let me know if I can provide any other info about my deployment, etc. Thank you for all of the work on Tauticord!

Expected Behavior

No migration is needed if no env variables are present, and shouldn't see a warning about env variables being deprecated if they're not in use too.

Steps to Reproduce

Be on v4.2.1 without env vars and (I think) you'll get the same error as above.

Supporting Information

I deploy this in a kubernetes cluster and can share the helmrelease.yaml if it's needed.

@vember31 vember31 added bug A provable error exists in the code triage Looking into it. labels Mar 31, 2024
@nwithan8
Copy link
Owner

I'm unable to immediately recreate the issue on my end. Could you share the logs in /logs/Tauticord.logs, it should have more information.

Just to confirm, is your /config folder read-write or read-only?

@nwithan8
Copy link
Owner

I was able to recreate the issue and patch the issue in v4.2.2

@nwithan8 nwithan8 added resolved Has been fixed or completed and removed triage Looking into it. labels Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A provable error exists in the code resolved Has been fixed or completed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants