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

Merge crusoe_update branch to main #10

Open
michal-cech opened this issue Sep 5, 2024 · 1 comment
Open

Merge crusoe_update branch to main #10

michal-cech opened this issue Sep 5, 2024 · 1 comment

Comments

@michal-cech
Copy link
Contributor

crusoe_update branch is a successor branch to issues:

The required student updates were merged together with their corresponding original repository history. I went through Docker deployment and Vagrant & Ansible deployment. It should be deployable. The question is whether the components actually work together.

  • docker-compose deploys everything
  • Vagrant files deploying singular parts of the Crusoe to separate VMs work
  • Common Vagrant file deploying everything in a single VM works

This also applies to the newly integrated component: the Recommender system - Dockerfile, separate Vagrantfile, and integration to common Vagrantfile.

@michal-cech
Copy link
Contributor Author

michal-cech commented Sep 5, 2024

Observations:

One observation:

Everything is super old: both Vagrantfiles and Dockerfiles are based on Debian 10.

This proved to be especially problematic when integrating the Recommender system, which requires a newer version of Python (Debian 10 has mainly Python 3.7, Recommender prerequisites need at least Python 3.10). This was not an issue for the Docker deployment, but had to be respected for Vagrant & Ansible deployment (mainly the common VM) - solvable by installing Python3.10 from source files and setting it as an alternative version.

What is a little more problematic (assuming the Recommender system and CRUSOE (observe) should work over the same Neo4J database). Crusoe observe deploys Neo4J 3.5.x, Recommender system seems to utilize queries with syntax compatible with 4.0+ versions. I did not change this; it was out of the scope of the issues from my PoV. Modernization will be a probably big task on its own.

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

No branches or pull requests

1 participant