trento-project / web / 9f780691609f2b28fa75c0e4e6a51c54905bf980

88%

push

github

Activity log correlation for deregistration entries (#3554)

* Adds a partial implementation

* See details/desc

- Adds new user activity log type: host_cleanup_requested

* See desc

Propagates causation_id and  correlation_id

* Removes unused deps

* More removal

A bunch of deps used to help introspect intermediate states of correlation identifiers are removed with this commit.

* Fixes test

* Removes more unused deps

* Cleanup

* Formatting

* Fixes failing tests

* Refactor

* Implements UI proposal

- Conditionally render the Related Events/Show Events entry in the ActivityLog modal

* Addresses PR comments

* Fix for runtime js warning

* Formatting

* Adds UI tests for modal

* Formatting

* Adds unit tests...

...for PhoenixConnParser

* Addresses PR feedback

- refactors ActivityLogDetailModal
- Shortens log messages to exclude actor name

* Refactors EventParser

According to PR comment. And adjusts tests to accommodate non-empty metadata.

* Removes plug_request_id

plug_request_id was being used to correlate console logs with activity log entries, but its introduction has been deferred.

* Adds plug test

... and some minor refactor

* Formatting

...and typos

* Addresses PR comments

Adds a new resource details field in cleanup action requested entries.

* Improves test assertion

* Implements correlated dispatch...

for sap systems and databases contexts. Some adjustment of related tests too.

* Improves correlation of deregistration of...

...application instances and databases. Now, in addition to the correlation_id we also see additional enriched fields in the corresponding clean up activity log entry.

22 of 22 new or added lines in 6 files covered. (100.0%)

3500 of 4042 relevant lines covered (86.59%)

296.48 hits per line