-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Brave ads database failing to migrate from schema 10 (v1.22.35) #17231
Comments
@brave/legacy_qa because of #17331, we won't be able to run through #17231 (comment) using those specific Brave versions. @jsecretan suggested running through an upgrade from |
Verified passed with
Per #17231 (comment), ran through items listed in brave/brave-core#9589 (comment). Upgraded from Confirmed no Logs``` [1178:775:0804/100024.806935:VERBOSE1:ads_service_impl.cc(1390)] Migrating ads preferences [1178:775:0804/100024.807089:VERBOSE1:ads_service_impl.cc(1457)] Migrating ads preferences from mapping version 10 to 11 [1178:775:0804/100024.807195:VERBOSE1:ads_service_impl.cc(1472)] Successfully migrated Ads preferences from version 10 to 11 [1178:775:0804/100025.562487:VERBOSE1:database_migration.cc(136)] DB: Migrated to version 32 ```Confirmed that Logs
Confirmed that Logs
Confirmed that website classification is working after upgrading as per the following: Logs
Confirmed that ad notifications are working as per the following: Logs
Confirmed that Brave successfully redeemed the unblinded payment tokens as per the following: Logs
Also ensured that:
Verification PASSED on
Per #17231 (comment), ran through items listed in brave/brave-core#9589 (comment). Upgraded from Confirmed no Logs
Confirmed that Logs
Confirmed that Logs
Confirmed that website classification is working after upgrading as per the following: Logs
Confirmed that ad notifications are working as per the following: Logs
Confirmed that Brave successfully redeemed the unblinded payment tokens as per the following: Logs
Verification passed on
Per #17231 (comment), ran through items listed in brave/brave-core#9589 (comment). Upgraded from Confirmed no Logs``` [12615:12615:0804/204820.548772:VERBOSE1:ads_service_impl.cc(1390)] Migrating ads preferences [12615:12615:0804/204820.551889:VERBOSE1:ads_service_impl.cc(1457)] Migrating ads preferences from mapping version 10 to 11 [12615:12615:0804/204820.552425:VERBOSE1:ads_service_impl.cc(1472)] Successfully migrated Ads preferences from version 10 to 11 [12615:12615:0804/204825.785272:VERBOSE1:ads_impl.cc(589)] Successfully initialized ads ```Confirmed that Logs
Confirmed that Logs
Confirmed that website classification is working after upgrading as per the following: Logs
Confirmed that ad notifications are working as per the following: Logs
Confirmed that Brave successfully redeemed the unblinded payment tokens as per the following: Logs
Also ensured that:
|
Description
Steps to Reproduce
Actual result:
Failed to initialize database:
appears in the console logExpected result:
Failed to initialize database:
should not appear in the console logReproduces how often:
Easily reproduced
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
The text was updated successfully, but these errors were encountered: