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

ruby on rails app #31

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

ruby on rails app #31

wants to merge 13 commits into from

Conversation

arnaudlevy
Copy link

No description provided.

@jdrouet
Copy link
Collaborator

jdrouet commented May 4, 2023

👋 Sorry to answer that late. Thanks for this PR!
Would it be possible for you to add a Dockerfile and a docker-compose.yml like in the other use cases?

@arnaudlevy
Copy link
Author

Salut @jdrouet ! Je ne sais pas le faire malheureusement :/

@arnaudlevy
Copy link
Author

Enfin évidemment je peux copier coller un fichier qui existe ailleurs mais j'imagine qu'il faut plus que ça :)

@jdrouet
Copy link
Collaborator

jdrouet commented May 5, 2023

Je ne connais pas assez bien Ruby mais j'imagine que c'est plus compliqué. Je regarde quand j'ai un weekend libre ;)

@arnaudlevy
Copy link
Author

Sur l'installation de Ruby je peux faire, mais sur Docker je suis une quiche, pardon.
@SebouChu est-ce que tu sais faire les 2 ?

@SebouChu
Copy link

@jdrouet Hello ! J'ai repris le boulot, tu peux jeter un oeil sur l'implémentation dès que tu as un peu de temps ?

J'ai écrit un peu de documentation dans le README, et j'ai remarqué quelques différences entre la documentation Swagger et l'implémentation en Go (et un peu le service validator).

A l'occasion, j'aimerai bien prendre un petit moment avec toi pour essayer de rationaliser l'ensemble et peut-être rendre le validator un peu moins strict ou alors préciser certaines choses, trois points pour exemple :

  • le test du header application/json qui renvoie une erreur si le charset est aussi défini (application/jsonl; charset=utf-8)
  • les attributs qui sont en camelCase dans la documentation, et en snake_case dans l'implémentation
  • le Content-Length qui peut différer selon le format de la date (sûrement besoin de préciser le format de date attendu)

@youenchene
Copy link
Collaborator

Doing the review:

docker build -t benchmark-ruby-on-rails .
OK
`docker run --name=benchmark-ruby-on-rails -d -p 8080:8080 -e DATABASE_URL='postgresql://postgres:[email protected]:5432/postgres' benchmark-ruby-on-rails'

c566f88dabf8563ca13638c74862d2d59a57c23394b16e148733616b5d80df23
docker: Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "/workdir/bin/docker-entrypoint": stat /workdir/bin/docker-entrypoint: no such file or directory: unknown.

Something wrong in the docker file?

@youenchene
Copy link
Collaborator

@SebouChu @jdrouet n'est plus très disponible.

J'ai corrigé les erreurs dans le swagger, l'implementation go est la référence.

dans le dossier test (main) il y a un test de conformité de comportement de l'API (qui est encore plus dure)

@GuillaumeMathieu27140
Copy link

Création #37 en reprise

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

Successfully merging this pull request may close these issues.

5 participants