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

cds-1565 Refactor Shipper code base in preparation for Metrics workflow #116

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

daidokoro
Copy link
Contributor

  • initial refactor, moved logs logic to sub-module logs
  • cargo.lock
  • change function_handler to just handler
  • added placeholder handler function for metrics.

Description

this PR refactors the current AWS Shipper code based in preparaiton for other telemetry workflows (metrics/traces). All logs related logic has been moved to a submodule, and a placeholder module has been created for metrics.

No change log required as no features or functionality have been changed.

How Has This Been Tested?

Checklist:

  • I have updated the versions in the SemanticVersion in template.yaml
  • I have updated the CHANGELOG.md
  • I have created necessary PR to Terraform Module Repository (https://github.com/coralogix/terraform-coralogix-aws) if needed
  • This change does not affect any particular component (e.g. it's CI or docs change)

@daidokoro daidokoro requested a review from a team as a code owner October 23, 2024 17:49
@daidokoro daidokoro added the skip-changelog Skip changelog update check label Oct 23, 2024
@MichaelBriggs-Coralogix

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skip changelog update check
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants