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/2078 docker: Some bugs in docker deploy #2125

Merged
merged 4 commits into from
Jan 13, 2025
Merged

Conversation

fv3rdugo
Copy link
Member

These changes fix issues with Docker deployment.

We recommend deploying Docker only for testing or development environments.

Description

  • changed docker mysql to mariadb
  • added ovmf package installation to docker backend
  • updated removal script to clean up

Motivation and Context

Solve this problem with database:

root@8fbc7e8dd185:/ravada# PERL5LIB=./lib ./script/rvd_back --add-user admin
INFO: creating table groups_local
DBD::mysql::db do failed: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(0),  name char(255) NOT NULL )' at line 1 at lib/Ravada.pm line 2488

And this ovmf error:

libvirt error code: 9, message: operation failed: unable to find any master var
 store for loader: /usr/share/OVMF/OVMF_CODE_4M.fd
<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here. -->

- change database image, from Mysql 5.7 to MariaDB 11.4 LTS
- remove version tag it's deprecated

 WARN[0000] /root/src/ravada/dockerfy/docker-compose.yml: the attribute `version` is obsolete, it will be ignored, please remove it
 to avoid potential confusion
- remove testing tags
- fix dockers names
- Add ovmf package, fix this error:

libvirt error code: 9, message: operation failed: unable to find any master var
 store for loader: /usr/share/OVMF/OVMF_CODE_4M.fd
@frankiejol
Copy link
Member

I see the Github automatic check fails, I try to check where could this come from.

@frankiejol frankiejol merged commit f5622cb into main Jan 13, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants