Adds cleanDisabled to Flyway configuration

[tioback]

Adds cleanDisabled to Flyway configuration.

Fixes #16561.

[quarkus-bot]

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

• title should not end up with dot
• title should not contain an issue number (use 'Fix Allow * expression for quarkus:add-extension #1234' in the description instead)

This message is automatically generated by a bot.

[quarkus-bot]

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building d9bad8a

Status	Name	Step	Test failures	Logs	Raw logs
✖	Initial JDK 11 Build	Build	⚠️ Check →	Logs	Raw logs
🚫	Devtools Tests - JDK ${{ matrix.java.name }}
🚫	Devtools Tests - JDK 11 Windows
🚫	Gradle Tests - JDK 11 ${{ matrix.os.family }}
🚫	JVM Tests - JDK ${{ matrix.java.name }}
🚫	JVM Tests - JDK 11 Windows
🚫	Maven Tests - JDK ${{ matrix.java.name }}
🚫	Maven Tests - JDK 11 Windows
🚫	MicroProfile TCKs Tests
🚫	Native Tests - ${{ matrix.category }}
🚫	Native Tests - Windows - ${{ matrix.category }}

[gastaldi]

Can you please squash your commits before we merge this? Thanks!

[quarkus-bot]

🚫 This workflow run has been cancelled.

Failing Jobs - Building f12903e

Status	Name	Step	Test failures	Logs	Raw logs
⌛	Devtools Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
⌛	Devtools Tests - JDK 11 Windows	Extract Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	Gradle Tests - JDK 11 Linux	Verify dependencies	⚠️ Check →	Logs	Raw logs
⌛	Gradle Tests - JDK 11 Windows	Extract Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 11	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 11 Windows	Download Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	JVM Tests - JDK 15	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Maven Tests - JDK 11	Build	⚠️ Check →	Logs	Raw logs
⌛	Maven Tests - JDK 11 Windows	Extract Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	MicroProfile TCKs Tests	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Amazon		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Cache		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data1	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data2	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data3	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data4	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data5	Download Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Data6	Download Maven Repo	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - HTTP		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Main	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Messaging1		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Messaging2		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc1		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc2		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc3		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Misc4		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security1	Reclaim Disk Space	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security2		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Security3		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Spring		⚠️ Check →	Logs	Raw logs
⌛	Native Tests - Windows - hibernate-validator	Install cl.exe	⚠️ Check →	Logs	Raw logs
⌛	Native Tests - gRPC		⚠️ Check →	Logs	Raw logs

[geoand]

Thanks for this!

Can you please add a test for this in quarkus-integration-test-flyway as well?
The reason being that we would like to make sure it works in native mode as well

[tioback]

Sure thing. I wasn’t aware of this test before. I’ll try to finish it later on today. Em sex., 16 de abr. de 2021 às 03:59, Georgios Andrianakis < ***@***.***> escreveu:

Thanks for this! Can you please add a test for this in quarkus-integration-test-flyway as well? The reason being that we would like to make sure it works in native mode as well — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#16563 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADKOAKP7VDXS7THU72EYU3DTI7N55ANCNFSM43AFA4QQ> .

-- Enviado do meu telemóvel, o pá!

[gsmet]

@geoand I don't think we mandated a native test for the other options? Or did I miss something? If so, I would leave it at that.

[geoand]

Well, we recently had a case where one of the Liquibase options fail in native.
I would like to avoid repeating that

[gsmet]

Sure but this option is about disabling something. So if you configure the native executable to use this option, you are actually not testing something (as we won't generate a second native executable just for that).

[geoand]

To be honest, I hadn't even read the name of the option, so I guess you are right 😉

[gsmet]

Let's merge! Thanks!

I will backport it to 1.13.3.Final.