Hi, added PR for SHEDEV Warsaw <https://github.com...

# confetti

[Oleksandr Katrych]

Hi, added PR for SHEDEV Warsaw
https://github.com/joreilly/Confetti/pull/1152

[John O'Reilly]

just merged.....we'll just need successful backend deployment and can run import then (I think that's all @mbonnin?)

[John O'Reilly]

@mbonnin can't remember if you switched from app engine to cloud run that time....can't seem to find view in console to delete instances (the deploy failed)

[John O'Reilly]

ah, think I was looking in wrong project....trying again now

[mbonnin]

There should be no need to delete instances now

[John O'Reilly]

hmm, this seems like different issue.....

ERROR 2024-02-27T19:41:45.451186Z java.lang.NoClassDefFoundError: org/yaml/snakeyaml/inspector/TagInspector at org.springframework.boot.env.YamlPropertySourceLoader.load(YamlPropertySourceLoader.java:50) at org.springframework.boot.context.config.StandardConfigDataLoader.load(StandardConfigDataLoader.java:54) at org.springframework.boot.context.config.StandardConfigDataLoader.load(StandardConfigDataLoader.java:36) at org.springframework.boot.context.config.ConfigDataLoaders.load(ConfigDataLoaders.java:96) at org.springframework.boot.context.config.ConfigDataImporter.load(ConfigDataImporter.java:132) at org.springframework.boot.context.config.ConfigDataImporter.resolveAndLoad(ConfigDataImporter.java:87) at org.springframework.boot.context.config.ConfigDataEnvironmentContributors.withProcessedImports(ConfigDataEnvironmentContributors.java:115) at org.springframework.boot.context.config.ConfigDataEnvironment.processInitial(ConfigDataEnvironment.java:242) at org.springframework.boot.context.config.ConfigDataEnvironment.processAndApply(ConfigDataEnvironment.java:229) at org.springframework.boot.context.config.ConfigDataEnvironmentPostProcessor.postProcessEnvironment(ConfigDataEnvironmentPostProcessor.java:96) at org.springframework.boot.context.config.ConfigDataEnvironmentPostProcessor.postProcessEnvironment(ConfigDataEnvironmentPostProcessor.java:89) at org.springframework.boot.env.EnvironmentPostProcessorApplicationListener.onApplicationEnvironmentPreparedEvent(EnvironmentPostProcessorApplicationListener.java:109) at org.springframework.boot.env.EnvironmentPostProcessorApplicationListener.onApplicationEvent(EnvironmentPostProcessorApplicationListener.java:94) at org.springframework.context.event.SimpleApplicationEventMulticaster.doInvokeListener(SimpleApplicationEventMulticaster.java:172) at org.springframework.context.event.SimpleApplicationEventMulticaster.invokeListener(SimpleApplicationEventMulticaster.java:165) at org.springframework.context.event.SimpleApplicationEventMulticaster.multicastEvent(SimpleApplicationEventMulticaster.java:143) at org.springframework.context.event.SimpleApplicationEventMulticaster.multicastEvent(SimpleApplicationEventMulticaster.java:131) at org.springframework.boot.context.event.EventPublishingRunListener.multicastInitialEvent(EventPublishingRunListener.java:136) at org.springframework.boot.context.event.EventPublishingRunListener.environmentPrepared(EventPublishingRunListener.java:81) at org.springframework.boot.SpringApplicationRunListen…

[John O'Reilly]

https://console.cloud.google.com/logs/query;query=resource.type%3D%22cloud_run_revision%2[…]27T19:41:45.451186Z;duration=P1D?project=confetti-349319

[mbonnin]

Yikes. I'm afk right now, will check in a bit

[mbonnin]

Made an attempt here. TBH I'm not sure what's going on and I don't have much time to dig a lot further right now. 🤞 it works with this if not, I'll dig a bit more tomorrow/

[John O'Reilly]

seemed to work...

[Oleksandr Katrych]

Works, thank you

[mbonnin]

Cool! We have spring boot gradle plugin 3.2.2 but only depend on 3.1.2 so it looks like it created some mismatch or so...

[mbonnin]

Will do proper post mortem tomorrow 😄

[John O'Reilly]

Screenshot_20240227-203334.png

[mbonnin]

Post mortem: the update to spring-web introduced this. It pulled spring-beans:6.1.4 in the classpath that requires snakeyaml:2.0 but somehow does not add it to the runtime classpath 🤷 . Filed this

[yschimke]

Apologies for optimistic dep updates. Some of my stuff needs very current things so I fix forward. But I assume this gets caught by the basic builds and tests

[mbonnin]

Yea, 100% we should add more tests

[mbonnin]

Also weird on spring to do a breaking change in a minor version, I wasn't expecting that

[yschimke]

Going to bump some dependencies including this one https://github.com/joreilly/Confetti/pull/1154, I think this is OK.

[mbonnin]

👍