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: adapt Makefile to work on pro platform #8529

Merged
merged 1 commit into from
Jun 6, 2023
Merged

Conversation

stephanegigandet
Copy link
Contributor

I tried to make "make build_lang" build the pro platform lang file when we have run source set-env.pro.sh, by loading an extra env.pro environment file (otherwise the env variables set by set-env.pro.sh are overloaded by .env when running the make script I believe).

But it fails later on.

Anyone can help?

stephane@stephane-jaguar-NS50-70MU:~/openfoodfacts-server$ (pro) (pro) (pro) (pro) (pro) make build_lang 🥫 Rebuild language docker-compose --env-file=.env --env-file=env.pro run --rm -e GITHUB_TOKEN= backend perl -I/opt/product-opener/lib -I/opt/perl/local/lib/perl5 /opt/product-opener/scripts/build_lang.pl WARN[0000] volume "po_podata" already exists but was not created for project "po". Use external: trueto use an existing volume WARN[0000] volume "po_orgs" already exists but was not created for project "po". Useexternal: trueto use an existing volume WARN[0000] volume "po_users" already exists but was not created for project "po". Useexternal: trueto use an existing volume [+] Running 1/0 ⠿ Container po_pro-memcached-1 Running 0.0s [+] Running 1/2 ⠦ Container po_pro-dynamicfront-1 Starting 0.7s ⠿ Container po_pro-minion-1 Started 0.7s Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/home/stephane/openfoodfacts-server/gulpfile.js" to rootfs at "/opt/product-opener/gulpfile.js": mount /home/stephane/openfoodfacts-server/gulpfile.js:/opt/product-opener/gulpfile.js (via /proc/self/fd/6), flags: 0x5000: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type make: *** [Makefile:171 : build_lang] Erreur 1

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 6, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@stephanegigandet
Copy link
Contributor Author

@alexgarel found that we need a version of docker-compose more recent than Apr 18 2023 for multiple --env-file to be loaded.

Update docker-compose by running:

sudo curl -SL https://github.com/docker/compose/releases/download/v2.18.1/docker-compose-linux-x86_64 -o /usr/local/bin/docker-compose

Copy link
Member

@alexgarel alexgarel left a comment

Choose a reason for hiding this comment

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

Great !

@alexgarel alexgarel changed the title fix: make build_lang work on pro platform - not working fix: adapt Makefile to work on pro platform Jun 6, 2023
@alexgarel alexgarel merged commit 05c75f4 into main Jun 6, 2023
@alexgarel alexgarel deleted the build_lang_pro branch June 6, 2023 15:27
MonalikaPatnaik pushed a commit to aadak99/openfoodfacts-server that referenced this pull request Jun 8, 2023
@teolemon teolemon added the 🏭 Producers Platform https://wiki.openfoodfacts.org/Platform_for_producers label Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏭 Producers Platform https://wiki.openfoodfacts.org/Platform_for_producers
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants