Merged
Conversation
…ncoded numbers (SigNoz#9154) * fix: let clickhouse handle string to number conversion * fix: ignore casting if it's a comparison operator for number key * fix: add integration tests * fix: update comments * fix: convert only if it's actually not a integrer with comparison operator * fix: force convert to float when number * fix: integration tests * fix: correct the comment * fix: update comment --------- Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
* feat(authz): address tenant isolation for authz * feat(authz): handle role module self registry * feat(authz): keep role / user / resource sync in naming * feat(authz): rename orgId to orgID * feat(authz): add the missing / for user * feat(authz): remove embedding for pkgopenfgaauthz service
* fix: logs popover content logic extracted out * fix: logs popover content in live view * fix: destory popover on close * feat: add logs format tests * feat: minor refactor * feat: test case refactor * feat: remove menu refs in logs live view * feat: globalise Preference context and remove async logic * feat: change preference context state structure to support both logs and traces pref * feat: test refactor
* feat: added new datasources * fix: added new datasource
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
This pull request refactors how TTL (Time-To-Live) settings are applied for logs, metrics, and traces in the ClickHouse reader service. The main change is the removal of the dedicated setTTLLogs method and the consolidation of TTL logic to only support metrics and traces. The code now routes TTL requests based on type, and logs TTL is no longer handled.
Signed-off-by: “niladrix719” <niladrix719@gmail.com> Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
…igNoz#9326) * chore(statsreporter): fix vultr platform detection in statsreporter * chore(statsreporter): adds comment for Vultr Detection order
* feat(sql): swap mattn/sqlite with modernc.org/sqlite * feat(sql): revert the dashboard testing changes * feat(sql): enable WAL mode for sqlite * feat(sql): revert enable WAL mode for sqlite * feat(sql): use sensible defaults for busy_timeout * feat(sql): add ldflags
This reverts commit 934c09b.
…9325)" (SigNoz#9338) This reverts commit c62d41e.
…9208) * feat: add bidirectional keys for http.url and net.peer.name * feat: added bidirectional key maps * feat: addressed comments regarding redundancy and batchKeys * feat: addressed comments regarding redundancy and batchKeys * feat: addressed comments regarding redundancy and batchKeys * feat: added test cases for bidirectional key mapping * feat: addressed comments for tests
…o support (SigNoz#9183) ## 📄 Summary - Instead of relying on JWT for session management, we are adding another token system: opaque. This gives the benefits of expiration and revocation. - We are now ensuring that emails are regex checked throughout the backend. - Support has been added for OIDC protocol
fix cache invalidation on delete
This pull request introduces support for setting TTL (Time-To-Live) on logs tables in ClickHouse, including both backend logic and integration tests. The main changes add back method for setting TTL on logs tables, update the TTL API to handle logs, and provide robust test coverage for legacy and new TTL flows.
* fix: change the date format for span hover card * fix: don't display search in events tab of span details side drawer in empty state * fix: unify the handle span click, to set spanId on clicking span overview column as well * Revert "fix: don't display search in events tab of span details side drawer in empty state" This reverts commit c8f56aa. * chore: add 0.5 seconds delay before showing span popover in trace details * chore: write tests * chore: update tests * fix: fix the failing test * chore: remove the mocks for utility functions
…9247) * fix: make the trace details attributes search visible by default * fix: make the attributes search focusable when the search button is toggled, not by default * fix: don't display search in events tab of span details side drawer in empty state * chore: write tests * chore: mock signozhq/button * chore: minor state rename
SigNoz#9421) * perf: optimize tooltip plugin with caching, memoization, and improved DOM operations * perf(uplot): optimize tooltip with focused sorting and O(n²) to O(n) reduction * perf(uplot): optimize threshold rendering with batched canvas operations * chore: pr review changes * chore: removed last index check for tooltip generation * chore: shifted to rendering only one points when hovered --------- Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com> Co-authored-by: Priyanshu Shrivastava <priyanshu@signoz.io>
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
* feat: fix uplot focused series logic selection * fix: stop propogation only if drilldown enabled * feat: minor refactor * feat: minor refactor * feat: minor refactor * feat: minor refactor --------- Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
…#9500) * fix: updated dashboard panel colors for better contrast ratio * chore: preetier fix * feat: added changes for the tooltip to follow cursor
…oz#9418) The issue was with how Mergo Treats Zero values, Mergo only fills **zero-value** fields in the destination. Since `false` is the zero value for `bool`, it always gets **replaced** by `true` from the source. Using pointers doesn’t help—`Merge` dereferences them and still treats `false` as zero.
…ated (SigNoz#9485) Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
* fix(sessions): do not use axios base instance * fix(sessions): fix test cases * fix(sessions): add trailing slashes
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com> Co-authored-by: Priyanshu Shrivastava <priyanshu@signoz.io>
…nup (SigNoz#9496) * feat: synchronise panel type state * feat: refactor explorer queries * feat: use explorer util queries * feat: minor refactor * feat: update test cases * feat: remove code * feat: minor refactor * feat: minor refactor * feat: update tests * feat: update list query logic to only support first staged query * feat: fix export query and saved views change * feat: test fix * feat: export link fix --------- Co-authored-by: Nityananda Gohain <nityanandagohain@gmail.com>
* fix: fix the issue of aggregation incorrectly falling back to count * refactor: add support for v5 queries in endPointDetailsDataQueries of EndPointDetails * chore: add common utility functions * refactor: migrate status code table to v5 * fix: status code table formatting * chore: add tests for status code table v5 migration * chore: add convertFiltersWithUrlHandling helper * chore: remove unnecessary tests * chore: aggregateAttribute to aggregations * fix: remove the aggregateOperator fallback logic changes * fix: fix the failing test * fix: add response_status_code exists to the status code table query
* feat(authz): openfga sql migration * feat(authz): formatting and naming * feat(authz): formatting and naming * feat(authz): extract function for store and model id * feat(authz): reorder the provider
* refactor: PostableRule struct - made validation part of `UnmarshalJSON` - removed validation from `processRuleDefaults` and updated signature to remove error from return type * refactor: updated error message for missing composite query --------- Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com>
…t cache (SigNoz#9581) Move away from unbounded cache for `lastObservedAt` stat, which was powered by BigCache (unbounded), to Ristretto, a bounded in-memory cache (https://github.com/dgraph-io/ristretto). This PR is first step towards moving away from unbounded caches in the system, more PRs to follow.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
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.
Summary
Related Issues / PR's
Screenshots
NA
Affected Areas and Manually Tested Areas