-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: develop deprecation and related changes #6668
Conversation
feat: support trace v3 queries
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 502f311 in 41 seconds
More details
- Looked at
237
lines of code in14
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. CONTRIBUTING.md:339
- Draft comment:
Update the contribution guidelines to reflect that contributions should be made to the 'main' branch instead of the 'develop' branch. - Reason this comment was not posted:
Comment did not seem useful.
Workflow ID: wflow_VQvSWGomEkiMSyzu
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
* nit: update resource id and revert the flag
Co-authored-by: Nityananda Gohain <nityanandagohain@gmail.com>
* feat: api for trace materialization * fix: minor changes and cleanup * fix: minor fixes * fix: update errors * fix: address comments * fix: address comments
c42461b
to
42fefc6
Compare
Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on 42fefc6 in 1 minute and 23 seconds
More details
- Looked at
237
lines of code in14
files - Skipped
0
files when reviewing. - Skipped posting
2
drafted comments based on config settings.
1. deploy/README.md:61
- Draft comment:
The reference to thedevelop
branch should be updated tomain
to reflect the deprecation of thedevelop
branch. - Reason this comment was not posted:
Comment looked like it was already resolved.
2. deploy/README.md:61
- Draft comment:
Update the branch reference fromdevelop
tomain
to reflect the changes made in the PR. - Reason this comment was not posted:
Marked as duplicate.
Workflow ID: wflow_c9jLrO5DC8Z2QW0q
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
…1.15 Signed-off-by: Prashant Shahi <prashant@signoz.io>
…1.16 Signed-off-by: Prashant Shahi <prashant@signoz.io>
Signed-off-by: Prashant Shahi <prashant@signoz.io>
Signed-off-by: Prashant Shahi <prashant@signoz.io>
Summary
Important
Deprecate
develop
branch in favor ofmain
across workflows, scripts, and documentation, with updates to Docker images and code refactoring for field handling.develop
branch frombuild.yaml
,push.yaml
,sonar.yml
, andjest-coverage-changes.yml
.docs.yml
,e2e-k3s.yaml
,staging-deployment.yaml
, andtesting-deployment.yaml
to trigger onmain
.hotrod-install.sh
andhotrod-delete.sh
to usemain
branch.develop
branch check infrontend/.husky/commit-msg
.CONTRIBUTING.md
anddeploy/README.md
to referencemain
branch.docker-compose.yaml
anddocker-compose-core.yaml
to new versions.go.mod
to usesignoz-otel-collector v0.111.16
.removeUnderscoreDuplicateFields()
inreader.go
to usemodel.Field
instead ofmodel.LogField
.GetTraceFields()
andUpdateTraceField()
inreader.go
.http_handler.go
for trace fields.buildTracesFilterQuery()
inquery_builder.go
to handle new filter operations.replaceFieldInToken()
inparser.go
to usemodel.Field
.parser_test.go
andquery_builder_test.go
to reflect changes in field handling.This description was created by
for 42fefc6. It will automatically update as commits are pushed.