Skip to content
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

WIP injecting component names into activity log #2951

Closed
wants to merge 2 commits into from

Conversation

nelsonkopliku
Copy link
Member

@nelsonkopliku nelsonkopliku commented Sep 9, 2024

Description

This is an exploration about the possibility to enrich Activity log metadata.

The rationale behind is improving human-friendly identification of the resource the activity log entry is related to and improving log searchability.

The thing could be summarized as: adding a human friendly resource name to the activity log metadata when it does not already have one

  • host -> hostname
  • cluster -> cluster name
  • database -> sid
  • sap system -> sid
  • application instance -> instance number
  • database instance -> instance number

Alternative to the approach tried in this PR could be upcasting all necessary events to also carry the extra information needed. However this would not be sufficient for non event sourced related activities: ie when tagging/untagging we still need some sort of metadata enrichment process to get the resource name being tagged/untagged.

@nelsonkopliku nelsonkopliku changed the title WIP injecting component names WIP injecting component names into activity log Sep 9, 2024
@nelsonkopliku nelsonkopliku force-pushed the inject-component-names-into-activity-log branch from 8de8d36 to 8bb40c3 Compare September 10, 2024 06:30
@nelsonkopliku nelsonkopliku added the env Create an ephimeral environment for the pr branch label Sep 10, 2024
@nelsonkopliku nelsonkopliku self-assigned this Sep 16, 2024
@nelsonkopliku nelsonkopliku added the enhancement New feature or request label Sep 16, 2024
@nelsonkopliku
Copy link
Member Author

Followed up by #2992, #2996 and #2997

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request env Create an ephimeral environment for the pr branch
Development

Successfully merging this pull request may close these issues.

1 participant