Thank you!
Cheers
Andrea
On Fri, Dec 1, 2023 at 3:54 PM Niels Charlier <ni...@scitus.be> wrote:
> Hello Andrea,
>
> You are right on both accounts; that stacktrace should not pollute the
> output.
>
> The other error indeed... shouldn't happen. The spring test runner does
> not change the behavior of the order of before calls. I cannot find any
> information on this happening to other people, and the code of that that
> runner class suggests it should be the same. I am quite perplexed. I am
> looking for a work-around that should prevent this from ever happening.
>
> Pull request is on its way.
>
> Regards
>
> Niels
> On 27/11/2023 19:16, Andrea Aime wrote:
>
> Hi all,
> I was checking a random failure in gs-metadata module, that is breaking
> the builds on Github and the build server (there are others, this one is
> popping up frequently though).
>
> The logs are ridden with 300+ instances of errors similar to this one:
>
> Nov 27, 2023 6:22:16 PM
> org.geoserver.metadata.data.service.impl.ConfigurationServiceImpl
> readConfiguration
> SEVERE: Cannot construct instance of
> `org.geoserver.metadata.data.dto.impl.MetadataConfigurationImpl` (although
> at least one Creator exists): no String-argument constructor/factory method
> to deserialize from String value ('klopt-niets-van')
> at [Source:
> (org.geoserver.platform.resource.FileSystemResourceStore$FileSystemResource$1);
> line: 1, column: 1]
> com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot
> construct instance of
> `org.geoserver.metadata.data.dto.impl.MetadataConfigurationImpl` (although
> at least one Creator exists): no String-argument constructor/factory method
> to deserialize from String value ('klopt-niets-van')
> at [Source:
> (org.geoserver.platform.resource.FileSystemResourceStore$FileSystemResource$1);
> line: 1, column: 1]
> at
> com.fasterxml.jackson.databind.exc.MismatchedInputException.from(MismatchedInputException.java:63)
> at
> com.fasterxml.jackson.databind.DeserializationContext.reportInputMismatch(DeserializationContext.java:1739)
> at
> com.fasterxml.jackson.databind.DeserializationContext.handleMissingInstantiator(DeserializationContext.java:1364)
> at
> com.fasterxml.jackson.databind.deser.std.StdDeserializer._deserializeFromString(StdDeserializer.java:311)
> at
> com.fasterxml.jackson.databind.deser.BeanDeserializerBase.deserializeFromString(BeanDeserializerBase.java:1504)
> at
> com.fasterxml.jackson.databind.deser.BeanDeserializer._deserializeOther(BeanDeserializer.java:197)
> at
> com.fasterxml.jackson.databind.deser.BeanDeserializer.deserialize(BeanDeserializer.java:187)
> at
> com.fasterxml.jackson.databind.deser.DefaultDeserializationContext.readRootValue(DefaultDeserializationContext.java:323)
> at
> com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:4825)
> at
> com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3809)
> at
> org.geoserver.metadata.data.service.impl.ConfigurationServiceImpl.readConfiguration(ConfigurationServiceImpl.java:195)
>
> A few things:
>
> - The exception is per se legit, one cannot load
> a MetadataConfigurationImpl from a YAML file that contains only the string
> 'klopt-niets-van')
> - The code is just logging the error... but making a lot of noise
> about it, especially at the current SEVERE logging level. More so
> considering that the code is blindly looping over files in a directory and
> trying to deserialize 3 different types of clases in turn, try the
> next if one fails
>
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/t1HpaHqGExH-9ePSAuYLZBpMvEabZU7x9qfJnhQt3fsncH5DNYBhkff7wZr8cnJx514Y3lAi5TGhl1xe3Vd78nyzjTIfGs6pUA0dkHSkMxQr6R5Y_LFcpicLknrDJT2oupoHQecvtb47Wa-DFPJ2QmMLoGRBeBKFA3qZhscirXPI9mSa5amMbzWBZlDJIYBYiIp6J47cJ5L4wsUtu8bS2lGQx7XMIFdG0DM_rG5TbLe_GAr8pE0EL1u7OtChKBJIF2IAAY5RPA8mqbI6_c-9Bpdfp-XlyMGpRgferZXdceut6jF7npyZXkH1ircGDwo3B_tq911h9XirowfllsHbmW9TP4ZBMJBT_G2zDqhwmPEmlzQ_Juo6vj5P-FOGKr3B7pjEECcyexzbQ46t9VL8MN9ahEPGpI9Qcb6DbQ5xaonvA35AikpBJneyfoCKfzpXhzy4PfsQNx55bz_s9aJainHB3dHuIUoXYJIqaUQ0nptWedXB67vVLGQIUgiwg61jrqW8mXcEHis_Ekig21ei9SS7Dj_qMd58wqU3ycMNnl5Kuw>.
> This needs to be fixed, we cannot have tests logging so loudly for nothing
> (nor we should have runtime code doing so).
> - For all the noise it's making, it's not the actual cause of the
> failure.
>
> Niels could you please remove this wall of text from our build execution?
>
> Now, going to the actual failure, it's just one:
>
> [ERROR] Tests run: 6, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 4.447
> s <<< FAILURE! - in org.geoserver.metadata.web.TemplatePageTest[ERROR]
> testPageNewSave(org.geoserver.metadata.web.TemplatePageTest) Time elapsed:
> 0.043 s <<< ERROR!java.lang.NullPointerException
> at
> org.geoserver.metadata.web.TemplatePageTest.before(TemplatePageTest.java:37)
>
> At line 37 the only NPE I see
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/poFKpkxYXwj1o_ZrkwAYCmT1G2w897b1bX2LcPQeNQh2ChfzdoQ3rNxu7fPhzlPOlcIIh4lkowayVb8jLtL12vz0kuVg_mqHQLSLM-9XJsvIKTcRFO0_tpPiJL932uJiC1czwda0bbmM8vaVXFqyw0jJRimPkFarIIrA1dg1d67oOKO4xrSsXJYK_8MtkBA8kDfmL3_klBWySsW5E6IwFALtcNF5wifqFutviblEuq5IJlVzkMTDmY0o1bjrJzsH1XZEX_r99WmrMiQo6jkfTPmUtPTYRuiSbEfh>
> happening
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/zVBJFgoWCo5TGU6WOrS0iZ8Cm9h-KzmTpIvUVU4_lserQ5CEDPpFq8mhh6YWHViGw8BiAxD351lMGX-VsCOJo90tJM9p8fuxlTIaOtU2ztznFLiyO_0euLfyo8Q3WWx3VF7Hnh9AZa_jwwz1C-6k3JrBg-7c9UyXF6De6Ws9R85ZwwfqNp0wXY7S7g1Az7XO-LJCc9nTN81gmR4F7HJ4U2alcpZmirbxAX-wVt2fbam30owGf0IME4ZFV2tAPSC8DJZ5UcXjSl_Hw1y0AvDqqTGi8hc>,
> is having the tester field being null.
> That should not be happening, because JUnit ensures that the @before
> methods of a parent class are run before the ones of the children, and we
> have this
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/M_Tezg5XhxmVI-8yrH8nUruz5T7-YExnR4UsdPTzUl-jm1GaSwEZjnRD5CCbueNccNj8L72JutJz2l13KwE5CEaP02RSPGKVXySlM9REEnMhWTuuBP_tnGj8KwjVF29GJemx96dU0XnesJx4RKy78SLqYCpKljJ3FJpCtCofX7xPX33GBeaqamnXbdb5cwuLBDMOMSn3QQ21kRYMWYb2OH0eLwsu15mlJdwLx-SELBC8CtkHBQ9YTW8Vo9I1SDffIj50D7vOk1s2HTolVjHV6ULuLsyb5lTLsUS4m3lQJJeZ-Wr-_gjdW1LU4t1u8LHjdNz6TpsZv1zx5SIE-TCLfDPGu45eIV6hXAIyVj3eas6Q6llF69Qo7NDd2kUly1v0fEVcbDpSIuhKUM0xEr5jSJlRt1fRu1HUlQRQptY2iLSFg8J2uhuKpTziucbgHxCwysP5tYWPMuSf2IEn3LMUPdAikfxhzPE>
> .
>
> However... the tests are not using vanilla JUnit, but Spring own test
> runner
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/z-8gDSo6vfhezOxAMJdO967fygLlAfFCtHZsyjixjuDuN1AQHamSVGcHi-qfPtGey3qPxUkTnRemSTrCjxRHDvhJ9zUIFrTtJmftZTXYS2AeLuT0k5lDhZdNMNvhDVlfPhqpoBT0fH_inKhHKCeUZ0t45jTxITyjBT-kCaBR7dX8imGu0L8mt-Vtgm0EfSTX_rKCJit2VlDeFDjxddBegkeiydUjfqA9mykrRfIZ3uKW-ghH95GLP6D_vk0vo5f6-fKwppsRSoqpKOxkXfyBg52n58cmO69B85C2XO9d6AoMJqvjxB35liKmXlgg1osHQeQxUd_hp8CLwM8NzyBRpznG_fQQ23Db7brIysL7C5Tsp5PoIoG2PbhGiazxCvn7wMw-EQNTrqnEEYlUDkKH1oB3FLmbhfAU52EwpCESigHZv2OPwvSh78cu-P5YZ7YL5Wu_MZ31ry3W_g8N25tocCZUrUShnhmocQ>,
> which runs the tests in a different way
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/o1OIRbp0odpEVAn0-F1YXvIVxnJksFE6vnQRIL712HBPPNE250IlSaPOTMrl_kl9W8tI4OS_97nKpFTZ8yCoQWC9jFtEHvYgjXomHPFge8q0LixcCNC_1shdrjpqH-m9MFJZC77FdT5hImUl_kCHTwaDQs2KgIsCTmeaszlapZV631kFhTeml5YEdySu4XTuSpUFlKHQpvAvQfz865VWWcsk5p-H4fOEk433ARmXg9NtIBwZipYH-zqlDhAveSaPIJhWNI0fzneoAjPy-VYme_4VFanmWyFXe1qRm5mvgzCrLjX5-xHcTZ3VetL4wjZcyB72M_AqpMkXeoqKo6lrmnS1DU0nqrLAPxpnvTzmVV061JpRpXST50TlrR64bFY3Q3DxsKBvNSeP-IwpxrLHqbSjS52BNo1NBIllKdkBZLuyijs6PAO6faEryBzKydHQnttrBQ9RsxgXZU8zHzzesq2SGqUnXkrzhBsQxuP5okdfNktcaZ7PdA>.
> Maybe that's why it's happening?
>
> Thad said, the occurrence of the failure has become so common that we
> should probably just disable these tests, until they can get fixed. Or
> maybe, better, rewrite them to use GeoServer integration test support,
> which is generally better known in this community?
>
> Cheers
> Andrea
>
> == GeoServer Professional Services from the experts!
>
> Visit http://bit.ly/gs-services-us
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/w9KOI-ibkZrfiL-jEqZP8MlMbt9KeLpjwcdpDmJaagH_V_nlLJrw2ZSOvRVWn0JO1xlcl2z72v3-LBYBr1EFNqJM2AbDs72oVUc5mTe7DD-l0gM7OXHoAfoQ7_w6cT2eiIlsMMLyXd4iEvZoti1choATqvmzv6pPb-_GztTP2Np-ppJPlN-1OJwYN2wg4RtBqQkvC8vZfJaP2iJeQPpUL8g4s6kUBI1IrwqeoYistAPV9H4WitqjYQ6VJmgwePr-4RD-s6Fr4g3K9xOl0IWk-P0AO72gJi0>
> for more information. == Ing. Andrea Aime @geowolf Technical Lead
>
> GeoSolutions Group phone: +39 0584 962313
>
> fax: +39 0584 1660272
>
> mob: +39 339 8844549
>
> https://www.geosolutionsgroup.com/
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/FhyCHemTzrjoZ2jg22YICxU8HC5hsWRh9RFIFhdoLJPNNlJZVmxUqTMz53KaFocddTXP6gMnapI2SEsF1r5fGsGlWdYdqbD3rrgqlMu6CLAbsgHEUPz6UcrnWt-Qx2wOk4jNknc843wLTF5CZie-S4aNcnlT3oHUa_gucnljPjX-kwM5qCGVCoaBk06g8IbVs8qRIbPjXe9dSRnQpRzJAFfq4opzx-OAMYF2PVH_uhnNZcgac5SuM0N4gBWBkO7akuD0UsAtAe9WybqWPOFxysZpYyOYaOtmpe0YUCU>
>
> http://twitter.com/geosolutions_it
> <https://dficcfc.r.bh.d.sendibt3.com/tr/cl/0aKe6XCO4gd8xKoYXhBALT9exgRLVvjVkv-nnH2_pQiFyCaacufhO8MXe_Pwr8Qrrk43MRxe8LvLNGTA9xWtuFEa7Ea-yJeMHFMgICIgU7375nG7txiLkIIVQFyTuPt38F-FYvaevHjJTdGxnSMAJljeVBi76aPInxbouoh8TtgTm4Xs3LJ5nV2aNXwRxgNnj8CnXL8AWKR21zjY_wNZDSnC72G9dcKeqo61J6k9k3ZAWQolz2YVqihwEtkdOQuaFoFc3T7r58Rd2crQ9OfZuOzlBCEHZ1C9d60ckyM>
>
> -------------------------------------------------------
>
> Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE
> 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si
> precisa che ogni circostanza inerente alla presente email (il suo
> contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è
> riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il
> messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra
> operazione è illecita. Le sarei comunque grato se potesse darmene notizia.
> This
> email is intended only for the person or entity to which it is addressed
> and may contain information that is privileged, confidential or otherwise
> protected from disclosure. We remind that - as provided by European
> Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail
> or the information herein by anyone other than the intended recipient is
> prohibited. If you have received this email by mistake, please notify us
> immediately by telephone or e-mail
>
>
--
Regards,
Andrea Aime
==
GeoServer Professional Services from the experts!
Visit http://bit.ly/gs-services-us for more information.
==
Ing. Andrea Aime
@geowolf
Technical Lead
GeoSolutions Group
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39 339 8844549
https://www.geosolutionsgroup.com/
http://twitter.com/geosolutions_it
-------------------------------------------------------
Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE
2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si
precisa che ogni circostanza inerente alla presente email (il suo
contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è
riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il
messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra
operazione è illecita. Le sarei comunque grato se potesse darmene notizia.
This email is intended only for the person or entity to which it is
addressed and may contain information that is privileged, confidential or
otherwise protected from disclosure. We remind that - as provided by
European Regulation 2016/679 “GDPR” - copying, dissemination or use of this
e-mail or the information herein by anyone other than the intended
recipient is prohibited. If you have received this email by mistake, please
notify us immediately by telephone or e-mail
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
