Revert "Fix http-parser missing package"

inspirehep / inspire-docker

Dockerfiles for inspirehep/inspire-next application

GNU General Public License v2.0

4 stars 12 forks source link

Revert "Fix http-parser missing package" #52

Closed david-caro closed 7 years ago

david-caro commented 7 years ago

This reverts commit 269857081f210bd14b93af0fd3117d0259b0a7c4.

Once the issue on upstream repos is solved, the patch is not needed anymore.

david-caro commented 7 years ago

This is so I don't forget, it should be fixed relatively soon, so once it's fixed, this can be merged.

jacquerie commented 7 years ago

Apparently there's now a workaround using the CR of Centos 7.4: https://bugzilla.redhat.com/show_bug.cgi?id=1487941. Shall we go for it, or should we wait for the release?

BTW, what about pinning more precisely the base Centos image?

david-caro commented 7 years ago

I think we should still wait until the official release, specially if we pin the centos image version (production actually has 7.3). Though I see value on testing the new versions sooner than later. Maybe we should adapt a more fast-forward attitude in prod too, upgrading the machines when able too.

Though, imo, in the short term, I'd say:

pin the centos image to 7.3 (prod version)
wait for official release of the package

jacquerie commented 7 years ago

It's here! \o/

https://lists.centos.org/pipermail/centos-announce/2017-September/022532.html