-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LPD-29337 Resolve test failures in DatabaseUpgradeClient #5982
Conversation
ci:test:upgrade |
@javalosjr96 please send it to database-infra repo to be tested |
@@ -119,7 +119,7 @@ definition { | |||
|
|||
var schemaVersion = ValidateSmokeUpgrade.getSchemaVersion(mysqlStatement = "SELECT schemaVersion FROM Release_ WHERE servletContextName = 'com.liferay.journal.web';"); | |||
|
|||
if (!(contains(${schemaVersion}, "0.0.1"))) { | |||
if (!(contains(${schemaVersion}, "4.1.0"))) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@javalosjr96 from which module you got the 4.1.0 version? In this case we are queryin the journal web module, see JournalWebUpgradeStepRegistrator
if (!_verifyProcessError && | ||
message.contains(VerifyException.class.getName())) { | ||
|
||
if (!_verifyProcessError && loggerName.contains("verify")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@javalosjr96 checking just the verify string is weak, why is this needed?
@javalosjr96 I have sent a new PR: |
ci:close |
No description provided.