This repository has been archived by the owner on Nov 14, 2023. It is now read-only.
forked from hypertrace/hypertrace-ingester
-
Notifications
You must be signed in to change notification settings - Fork 4
Test5 #77
Open
suchithrzp
wants to merge
91
commits into
razorpay:main
Choose a base branch
from
suchithrzp:test5
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Test5 #77
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
) * feat: add fallback support for request and response size * added support for content-type tag * feat: adds support for content-length for grpc * fixed nit comment issues * added suport for http request response truncated body attribute * add support for truncated rpc/grpc attributes too * address comments related to repetative constants * address few more comments * refactor a bit for redability * bit refactor for redability
Co-authored-by: Laxman Ch <laxman@traceable.ai>
* feat: adds pull based metrics exporter * feat: adds hypertrace ingester hook for metrics exporter * refactor and cleaned the code * handles the runtime and interrupt exception * clean up the deps and updated with newer deps * handling closing conditions * test: adds test for otlp proto to metric data conversion * test: add unit test for in-memory metrics producer
Co-authored-by: Laxman Ch <laxman@traceable.ai>
* added gethost method * modified * added unit test * resolved PR comments * added unit test * nit * nit Co-authored-by: Sarthak Singhal <sarthaksinghal@Sarthaks-MacBook-Pro.local>
* added condition on abs url * nit * nit * refactored Co-authored-by: Sarthak Singhal <sarthaksinghal@Sarthaks-MacBook-Pro.local>
…pertrace#280) * feat: add forming grpc reqeust url if rpc attributes are present * nit: minor fix on comment
* added span count attribute * resolved PR comment * Update hypertrace-view-generator/hypertrace-view-generator/src/main/java/org/hypertrace/viewgenerator/generators/SpanEventViewGenerator.java Co-authored-by: kotharironak <53209990+kotharironak@users.noreply.github.com> * Update hypertrace-view-generator/hypertrace-view-generator-api/src/main/avro/SpanEventView.avdl Co-authored-by: kotharironak <53209990+kotharironak@users.noreply.github.com> * Update SpanEventView.avdl * Update SpanEventViewGenerator.java Co-authored-by: Sarthak Singhal <sarthaksinghal@Sarthaks-MacBook-Pro.local> Co-authored-by: kotharironak <53209990+kotharironak@users.noreply.github.com>
Co-authored-by: Sarthak Singhal <sarthaksinghal@Sarthaks-MacBook-Pro.local>
* added the utility method for gprc request URL * feat : add support for calc grpc request url * review: address comments of renaming enriched attributes name * fix: snyk failures * updates protobuf version * going back to compatible version for proto libs * snyk issue, adds constrains for google proto * updated .snyk file * fixed last two snyk issue * added constrain block for jersy lib * fixed snyk issues
- Splitting tag key/value match condition based on the first colon
* refactor: moving out avro checks as a seperate job * reverting back to pull_request_target * revert back space too * adds caching layer for avroCompatibilityCheck
* feat: enhance the support for dropping filter * feat: added support for checking filter criteria along with process tags too * addressed comments * addressed tostring and logging related comments * address comments of wrapping around if debuged enabled for log
…ertrace#290) * feat: add global upper limit for max span count limit * fixed code style * renamed config name from upper.max.span.count to default.max.span.count for global limit
…#291) * fix: adds support for default max spans limit helm config * there is no limit as default
* feat: add metrics generator for num_calls * fix the avro compatibility issue * test: add topology test * test: add the topology tests and fixed it * adds more assertions * disabling avro compatibility check, and adding configs for time outs * removed un-used variable from emitter * updated service framework library * handling snyk issues * addressed review comments * refactor: instead of using avro, changed to proto
…race#292) * feat: adding steps for enabling metrics pipeline * refactor the metrics pipeline sub-topology
* fix: update log4j * fix: more vuln fixes
* chore: update log4j * chore: remove duplicate insecure repo * chore: update netty
* upgrade log4j version to 2.17.0 * upgrade log4j version to 2.17.0 * update view-creator-framework version
* chore: extend the helm template for spanDropFilter * updates log4j libs * update snyk file for IONETTY issue * corrected the date * updates framework libs * addressed comments * handles jackson related snyk failure * adding constrains for log4j * fixed mistake of copying @ notation * fixed log4j vuln via fixing view-gen-framework
Fix for extra string "razorpay" in image names
…ber (razorpay#28) * Github action for release having docker Image with Tag as version number * getCommitHash() method removed * Fixes github action syntax issues
* Enable Dependabot * Security scan added * fixing checks
* changes for codecov * codecov code uncommented * removed unnecessary file * nit * indentation fix * codecov branch changed to rzp_main * default br changes * br fix
Hypertrace Release 0.6.40-0.1.0
- Synthetic Monitoring
Hypertrace Release 0.6.40-0.2.0
* Adds resource attribute enricher * Adds test for resource attribute enricher * test * wip * fix * Adds node.selector to infra metadata * Adds node.selector * spotless applied * Adds more tests for node-selector * nit * removes redundant CI workflows * Adrresses PR comments * Adds attributes to match in config * adds description * Adresses PR comments * Adresses PR comments * Addresses PR comments * increase coverage * applies spotless * remove test * addressed PR comments
This reverts commit 86f16ba.
Immediate mitigation to handle malicious PR vulnerability.
Fix workflow
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
test