This PR migrates Ditto to Apache Pekko #1730

alstanchev · 2023-08-28T08:52:23Z

Since the change of akka's license to "Business Source License (BSL) v1.1" this PR:

Migrates all the code to use Apache Pekko framework (fork from Akka 2.6.21)
Adapts the documentation, third party notices
And all necessary changes.

Resolves: #1477

…rces controlled via UI * added radiobutton groups so that "Authorize" modal clearly shows/states which authentication to use where * reduced to a single "Authorize" button in modal * simplified environment a bit Signed-off-by: Thomas Jaeckle <[email protected]>

…ed on "main / devops" auth Signed-off-by: Thomas Jaeckle <[email protected]>

Signed-off-by: Thomas Jaeckle <[email protected]>

…n 3.2.0 Signed-off-by: Thomas Jaeckle <[email protected]>

Signed-off-by: Thomas Jaeckle <[email protected]>

Signed-off-by: Stanchev Aleksandar <[email protected]>

… failing Signed-off-by: Thomas Jäckle <[email protected]>

* UI - New Operations tab * added new tab * changed way to control right auth user * extended api to allow devops path * utils to add table row now returns row * crud editor returns explicit cancel action * crud editor now with option without delete or create Signed-off-by: thfries <[email protected]> * UI - Operations Tab: new logger view Signed-off-by: thfries <[email protected]> * UI - operations tab - refresh not scrolling * UI - change header info on new files Signed-off-by: thfries <[email protected]> * UI - Operations tab - make tabs optional - fixed auth header mixing from wrong environment - operations tab not getting right auth header Signed-off-by: thfries <[email protected]> * UI - added option to define new logger for each service in Operations / Logging functionality Signed-off-by: Thomas Jäckle <[email protected]> * UI - operations tab - remove spellcheck from input - split loggerView into smaller functions Signed-off-by: thfries <[email protected]> --------- Signed-off-by: thfries <[email protected]> Signed-off-by: Thomas Jäckle <[email protected]> Co-authored-by: Thomas Jäckle <[email protected]>

* fallback to display connection "id" if no "name" is present Signed-off-by: Thomas Jäckle <[email protected]>

Signed-off-by: Thomas Jäckle <[email protected]>

…t in a merge patch using a regular expression Signed-off-by: Thomas Jäckle <[email protected]>

…hes including a regex: in that case, unrestricted WRITE access on the contained JSON object is required Signed-off-by: Thomas Jäckle <[email protected]>

Signed-off-by: Stanchev Aleksandar <[email protected]>

…rActor#messageDiscarded * to not use `responseDispatchedMonitor`, but getMonitorsForOutboundSignal() with using `responseMappedMonitor` instead for responses Signed-off-by: Thomas Jäckle <[email protected]>

Signed-off-by: Stanchev Aleksandar <[email protected]>

Signed-off-by: Stanchev Aleksandar <[email protected]

Signed-off-by: Stanchev Aleksandar <[email protected]>

thjaeckle

One first finding when roughly scrolling over the changes.
Will have a closer look soon.

Great effort @alstanchev 👍

bom/pom.xml

mongo-java-driver.version 4.3.4 -> 4.10.2 Signed-off-by: Aleksandar Stanchev <[email protected]>

Signed-off-by: Aleksandar Stanchev <[email protected]>

documentation/src/main/resources/pages/ditto/installation-operating.md

thjaeckle · 2023-09-05T09:47:36Z

documentation/src/main/resources/pages/ditto/installation-operating.md

+* -Dpekko-contrib-mongodb-persistence-connection-remember-journal.overrides.metadata-index=akka_persistence_metadata_pid
+```
+
+And also a full cluster recreate is required, rolling update is not supported as there are changes in the management 


@alstanchev do the changes in management URLs require adjusting anything e.g. in the Helm chart?

thjaeckle

@alstanchev I reviewed by also starting locally and (manually) testing if data would still be there with Pekko - seems good as far as I can tell 👍

I had just one remark about the migration guide.

Signed-off-by: Aleksandar Stanchev <[email protected]>

documentation/src/main/resources/pages/ditto/release_notes_340.md

Signed-off-by: Aleksandar Stanchev <[email protected]>

alstanchev · 2023-09-18T12:26:27Z

@thjaeckle i have tested this with the helm chart and it is working. From my point of view it is ready for merge.

thjaeckle

FMPOV also good to merge 👍

Signed-off-by: Thomas Jäckle <[email protected]>

thjaeckle and others added 30 commits March 13, 2023 15:38

[eclipse-ditto#1592] switched grouping of different auth* methods bas…

a0fd0f1

…ed on "main / devops" auth Signed-off-by: Thomas Jaeckle <[email protected]>

prepare Ditto 3.2.0 release

7db85d7

Signed-off-by: Thomas Jaeckle <[email protected]>

post Ditto 3.2.0 release: update binary-compatibility-check to versio…

2c9c189

…n 3.2.0 Signed-off-by: Thomas Jaeckle <[email protected]>

post Ditto 3.2.0 release: update SECURITY.md

66e539d

Signed-off-by: Thomas Jaeckle <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

8613a16

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

0a44e22

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

c38724d

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

1b35510

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

a0aba9a

Signed-off-by: Stanchev Aleksandar <[email protected]>

fixed OverflowStrategy in SupervisedStream to backpressure instead of…

ba9b820

… failing Signed-off-by: Thomas Jäckle <[email protected]>

fix in UI that "bearerDevOps" token is stored to local storage

9d11565

* fallback to display connection "id" if no "name" is present Signed-off-by: Thomas Jäckle <[email protected]>

provide release notes for Ditto 3.2.1

73af3f7

Signed-off-by: Thomas Jäckle <[email protected]>

eclipse-ditto#1593 support removing existing fields from a JSON objec…

75e7280

…t in a merge patch using a regular expression Signed-off-by: Thomas Jäckle <[email protected]>

eclipse-ditto#1593 added special "merge command" enforcement for patc…

915e68b

…hes including a regex: in that case, unrestricted WRITE access on the contained JSON object is required Signed-off-by: Thomas Jäckle <[email protected]>

Merge branch 'bosch-io'

316edf6

Signed-off-by: Stanchev Aleksandar <[email protected]>

eclipse-ditto#1619 adjusted monitor to use in OutboundMappingProcesso…

f57e41f

…rActor#messageDiscarded * to not use `responseDispatchedMonitor`, but getMonitorsForOutboundSignal() with using `responseMappedMonitor` instead for responses Signed-off-by: Thomas Jäckle <[email protected]>

Merge branch 'bosch-io'

6f90710

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

fc6a1ae

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge branch 'bosch-io'

ba39bb0

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge branch 'bosch-io'

e363342

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

e714aec

Signed-off-by: Stanchev Aleksandar <[email protected]>

Merge remote-tracking branch 'bosch-io/master'

bfff8fb

Merge remote-tracking branch 'bosch-io/master'

9e1c1b9

Signed-off-by: Stanchev Aleksandar <[email protected]

Merge branch 'bosch-io'

bb70917

Signed-off-by: Stanchev Aleksandar <[email protected]>

switch dependencies and imports to Pekko

dc948ef

Signed-off-by: Stanchev Aleksandar <[email protected]>

update documentation links

3589537

Signed-off-by: Stanchev Aleksandar <[email protected]>

move repositories from bom to root pom

3688eea

Signed-off-by: Stanchev Aleksandar <[email protected]>

fix some wrong package names

baf14c5

Signed-off-by: Stanchev Aleksandar <[email protected]>

alstanchev added 6 commits August 28, 2023 09:32

update deployment/* files to pekko

c06834b

Signed-off-by: Stanchev Aleksandar <[email protected]>

update legal dependencies files

38096e5

Signed-off-by: Stanchev Aleksandar <[email protected]>

some more akka leftovers in deployment

6556895

Signed-off-by: Stanchev Aleksandar <[email protected]>

switch pekko-persistence-inmemory to sonatype repo

dde02f2

Signed-off-by: Stanchev Aleksandar <[email protected]>

set management port to Pekko's new default 7626

eec6253

Signed-off-by: Stanchev Aleksandar <[email protected]>

remove snapshot repos and fix inmemory plugin version

95c937f

Signed-off-by: Stanchev Aleksandar <[email protected]>

alstanchev requested a review from thjaeckle August 28, 2023 08:52

thjaeckle reviewed Aug 28, 2023

View reviewed changes

bom/pom.xml Outdated Show resolved Hide resolved

thjaeckle added this to the 3.4.0 milestone Aug 28, 2023

thjaeckle assigned alstanchev Aug 28, 2023

alstanchev force-pushed the feature/pekko-migration branch 2 times, most recently from 8258e29 to 34e35f3 Compare August 31, 2023 07:19

alstanchev added 3 commits August 31, 2023 11:13

sync mongo java driver version with pekko-persistence-mongodb

588e7af

mongo-java-driver.version 4.3.4 -> 4.10.2 Signed-off-by: Aleksandar Stanchev <[email protected]>

pekko-persistence-mongodb version to 1.0.1, metadata-index name configs

5926abd

Signed-off-by: Aleksandar Stanchev <[email protected]>

update documentation with migration steps

0a1d2b2

Signed-off-by: Aleksandar Stanchev <[email protected]>

alstanchev force-pushed the feature/pekko-migration branch 2 times, most recently from 5d3c232 to 0a1d2b2 Compare August 31, 2023 08:26

thjaeckle reviewed Sep 5, 2023

View reviewed changes

documentation/src/main/resources/pages/ditto/installation-operating.md Outdated Show resolved Hide resolved

thjaeckle reviewed Sep 5, 2023

View reviewed changes

thjaeckle requested changes Sep 5, 2023

View reviewed changes

alstanchev added 4 commits September 5, 2023 15:47

remove unneeded line from config

73277b5

Signed-off-by: Aleksandar Stanchev <[email protected]>

move migration steps to a release_notes file

a270162

Signed-off-by: Aleksandar Stanchev <[email protected]>

Merge branch 'bosch-io'

d76b94f

Signed-off-by: Aleksandar Stanchev <[email protected]>

Merge branch 'master' into feature/pekko-migration

d90b045

Signed-off-by: Aleksandar Stanchev <[email protected]>

thjaeckle reviewed Sep 11, 2023

View reviewed changes

documentation/src/main/resources/pages/ditto/release_notes_340.md Outdated Show resolved Hide resolved

update changelog for 3.4.0 release

afe36ed

Signed-off-by: Aleksandar Stanchev <[email protected]>

thjaeckle approved these changes Sep 18, 2023

View reviewed changes

alstanchev merged commit 3c4f89d into eclipse-ditto:master Sep 18, 2023
2 checks passed

thjaeckle added a commit that referenced this pull request Sep 18, 2023

#1730 fix test imports after merge of Pekko migration

8bbd417

Signed-off-by: Thomas Jäckle <[email protected]>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

This PR migrates Ditto to Apache Pekko #1730

This PR migrates Ditto to Apache Pekko #1730

alstanchev commented Aug 28, 2023 •

edited by thjaeckle

Loading

thjaeckle left a comment

thjaeckle Sep 5, 2023

thjaeckle left a comment

alstanchev commented Sep 18, 2023 •

edited

Loading

thjaeckle left a comment

This PR migrates Ditto to Apache Pekko #1730

This PR migrates Ditto to Apache Pekko #1730

Conversation

alstanchev commented Aug 28, 2023 • edited by thjaeckle Loading

thjaeckle left a comment

Choose a reason for hiding this comment

thjaeckle Sep 5, 2023

Choose a reason for hiding this comment

thjaeckle left a comment

Choose a reason for hiding this comment

alstanchev commented Sep 18, 2023 • edited Loading

thjaeckle left a comment

Choose a reason for hiding this comment

alstanchev commented Aug 28, 2023 •

edited by thjaeckle

Loading

alstanchev commented Sep 18, 2023 •

edited

Loading