Master testcontainers-java breaks ReadonlyREST #493

Activity

1. bsideup commented on Nov 14, 2017

   

   bsideup

   on Nov 14, 2017

   Member

   More actions

   @sscarduzio
   From the logs:

   Could not build image: The command '/bin/sh -c sed -i "s|debug|info|g" /usr/share/elasticsearch/config/log4j2.properties' returned a non-zero code: 2

   Such error might be a flaky test or something caused by 2ad57da

   May I kindly ask you to try first this one:
   com.github.testcontainers.testcontainers-java:testcontainers:2ad57da4a4f7b0d3396c4883294aa9c32ef480e7

   And then:
   com.github.testcontainers.testcontainers-java:testcontainers:1c66a143e6513d25b4d60044303b2fabf1c6c0fb

   Thanks for letting us know before it got released! :)
2. sscarduzio commented on Nov 14, 2017

   

   sscarduzio

   on Nov 14, 2017

   ContributorAuthor

   More actions

   Hi @bsideup,
   Just ran this on my laptop. You are right:

   2ad57da <--- test runs
   1c66a14 <--- test fails as above
3. rnorth

   added 3 commits that reference this issue on Nov 26, 2017

   Fix #493: Use null to indicate no overriding file mode instead of `-1`

   ...

   This issue will close once commit 643ddaa is merged into the 'main' branch.

   643ddaa

   Fix #493: Use null to indicate no overriding file mode instead of `-1`

   ...

   This issue will close once commit 263ac20 is merged into the 'main' branch.

   263ac20

   Fix #493: Use null to indicate no overriding file mode instead of `-1`

   ...

   This issue will close once commit 1391d8c is merged into the 'main' branch.

   1391d8c
4. rnorth

   closed this as completedin b7e7d52on Nov 26, 2017
5. rnorth commented on Nov 26, 2017

   

   rnorth

   on Nov 26, 2017

   Member

   More actions

   Thanks for flagging this @sscarduzio - this should be fixed in the latest master. Thanks to your report we'll avoid shipping this bug in a released version 😅 !
6. sscarduzio commented on Nov 27, 2017

   

   sscarduzio

   on Nov 27, 2017

   ContributorAuthor

   More actions

   Thanks @rnorth for fixing this, I can confirm now it works because I reverted the dependency to the master branch of testcontainers-java. 🎉
7. rnorth

   added a commit that references this issue on Dec 24, 2018

   Bump influxdb-java from 2.10 to 2.14 in /modules/influxdb (#1070)

   ...

   f986243