Skip to content
This repository has been archived by the owner on Jul 28, 2024. It is now read-only.

YAML parsing for Liquibase broken in 3.7+ #16

Open
robfletcher opened this issue Jun 29, 2020 · 5 comments
Open

YAML parsing for Liquibase broken in 3.7+ #16

robfletcher opened this issue Jun 29, 2020 · 5 comments

Comments

@robfletcher
Copy link
Contributor

robfletcher commented Jun 29, 2020

Upgrading from 3.6.0 to 3.8.0 I now get a failing Gradle build with the following error:

liquibase.exception.UnknownChangelogFormatException: Cannot find parser that supports /Users/fletch/netflix/spinnaker/workspace/keel/keel-sql/src/main/resources/db/databaseChangeLog.yml

It's not clear if this is a bug or whether I'm now expected to add an additional dependency.

I tried 3.7.0 and I get the same error.

@robfletcher robfletcher changed the title YAML parsing for Liquibase broken in 3.8 YAML parsing for Liquibase broken in 3.7+ Jun 29, 2020
@robfletcher
Copy link
Contributor Author

Adding jooqModelatorRuntime("org.yaml:snakeyaml") also makes no difference

@ayedo
Copy link
Owner

ayedo commented Jun 29, 2020

Thank you for your report.

I can reproduce the problem, and have written a new test for it.

I have a workaround that I could quickly deploy, but I have the feeling that other Liquibase Classloader issues (#12) are related to this, and I'll need some more time to look into them.

@ursjoss
Copy link
Contributor

ursjoss commented Jun 30, 2020

Might be related with https://liquibase.jira.com/browse/CORE-3495, even though in that ticket, it was liquibase 3.8.0 that introduced the issue. Here it was still working with 3.8.0 IMHO.

@ayedo
Copy link
Owner

ayedo commented Jul 6, 2020

Thanks @ursjoss for your hint. I'll keep an eye on that issue.

I still could not find out how to set the classpath correctly for Liquibase. I have added snakeyaml to the plugin classpath as a workaround while I'm figuring this out. The new test I wrote passes like that.

@robfletcher Can you please check with version 3.9 if this resolves your issue as well?

@robfletcher
Copy link
Contributor Author

robfletcher commented Jul 7, 2020

@ayedo yes, 3.9.0 works for me. Thanks for the update.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants