Skip to content

UNOMI-203, UNOMI-632, UNOMI-633, UNOMI-634: improve integrations test with possibility to test migrations#464

Merged
jkevan merged 9 commits intomasterfrom
migrationTests
Jul 28, 2022
Merged

UNOMI-203, UNOMI-632, UNOMI-633, UNOMI-634: improve integrations test with possibility to test migrations#464
jkevan merged 9 commits intomasterfrom
migrationTests

Conversation

@jkevan
Copy link
Contributor

@jkevan jkevan commented Jul 26, 2022

Content of the PR:

  • Silent migration feature, using cmd like: migrate 1.6.0 true, true is for silent (UNOMI-633)
  • Configurable migration using new OSGI conf: org.apache.unomi.migration.cfg (UNOMI-633)
  • Improve reindexation in migrations to reuse mappings configuration provided by Unomi bundles instead of duplicating them. (UNOMI-634)
  • Improve integration to use KarafTestSupport from karaf libraries: (UNOMI-203)
  • Improve integration tests to be able to test migrations (UNOMI-632)
  • Implement integration test for migration from 1.6.x to actually 2.0.0 (UNOMI-632)
  • Migrate remaining indices that need a reindexation by generifying the segment migration script to handle others indices in the same situation like: campaigns, goals, rules, condition types, etc ... (UNOMI-632)

@jkevan jkevan changed the title UNOMI-203, UNOMI-632: improve integrations test with possibility to test migrations UNOMI-203, UNOMI-632, UNOMI-633: improve integrations test with possibility to test migrations Jul 26, 2022
Copy link
Contributor

@sergehuber sergehuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello Kevan,

Thanks a lot for doing this, it is something I've been wanting to have for a long time.
Looks all good to me.

Regards,
Serge...

@jkevan jkevan changed the title UNOMI-203, UNOMI-632, UNOMI-633: improve integrations test with possibility to test migrations UNOMI-203, UNOMI-632, UNOMI-633, UNOMI-634: improve integrations test with possibility to test migrations Jul 27, 2022
@jkevan jkevan merged commit 53cf447 into master Jul 28, 2022
@jkevan jkevan deleted the migrationTests branch July 28, 2022 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants