-
Notifications
You must be signed in to change notification settings - Fork 3
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
onboarding: docker-compose up errored at pyrrha-simulator #13
Comments
Is this still an issue @albertum1? |
I believe I changed the package.json file from sensor simulator to get this to work, Pyrrha-Platform/Pyrrha-Sensor-Simulator#19 |
@albertum1, can you please pull main again and try. I created a dev dockerfile that removed the The PR you submitted is another way to fix the problem, however that moves nodemon from dev dependency to production. It's not really a production dependency. I remember that's what we did when we solved it locally for you, but this seems like a better way. Ideally, you would have made this fix on your PR, but I know a few people have been waiting for this and blocked on me. So I went ahead and made the fix in a new PR. |
@albertum1 can you please try again when you get a chance? Thank you! |
@upkarlidder confirming receipt |
@upkarlidder, on a fresh ubuntu I am stuck on docker-compose build returning this error |
@dacamposol can you please take a look? Looks like another docker dependency issue in Rules-Decision. Docker-compose simply uses the repo Dockerfile. I have not investigated yet. I will take a look later today as well. Thank you @albertum1 for running through this! |
I just tried it and it worked fine for me. Just adding logs here and will dm @albertum1
|
@upkarlidder you're using your already cached layers on that test, so it doesn't actually show if there is a problem at the Alpine repository level. I will take a look this evening. |
Good catch @dacamposol. I tried from scratch after removing the cache and I do get the same error:
So in order to learn a little more I did some investigation. I ran
Does apt only cache certain versions? If so, will we run into this again next time they update the curl version? I guess one workaround would be to just remove the version and always install the latest curl, but I do like pinning the dependency. |
@upkarlidder I don't think we should unpin the version, we could just pin up to the minor changes and just keep any build or bugfix version to pass through:
Probably the issue is because the build versions ( |
Thank you @dacamposol. PR merged. @albertum1, can you please try again when you get a chance? You will have to pull latest main from Pyrrha-Rules-Decision. |
With a fresh ubuntu on virtualbox, I followed the steps on the docker-compose.md
Got stuck on docker-compose up with looped error as shown below.
The text was updated successfully, but these errors were encountered: