Merged
Conversation
* feat(authz): remove unnecessary dependency injection for role setter * feat(authz): deprecate role module * feat(authz): deprecate role module * feat(authz): split between server and sql actions * feat(authz): add bootstrap for managed role transactions * feat(authz): update and add integration tests * feat(authz): match names for factory and migration * feat(authz): fix integration tests * feat(authz): reduce calls on organisation creeation
* chore: folder name change + CODEOWNER update * chore: revert multi select file change
If multiple batches are inserted with same trace_id, then trace_summary table can have multiple rows before they are aggregated by clickhouse. Query to get the time range from trace_summary was assuming a single which was create unpredictable behaviour as any random row could be returned.
* test: added test suites for uplotchart component * chore: resolved pr review comments * chore: resolved pr review comments
* chore: use variable select strategy * chore: revert custom multi select * chore: fix tests * chore: fix tests * chore: fix flaky test
…LED (SigNoz#10227) * fix: Service Map environment filter not working with DOT_METRICS_ENABLED When DOT_METRICS_ENABLED is active, resource attribute keys use dot notation (e.g. resource_deployment.environment) instead of underscore notation (e.g. resource_deployment_environment). The whilelistedKeys array only contained underscore-notation keys, causing getVisibleQueries and mappingWithRoutesAndKeys to filter out valid queries on the Service Map. - Add dot-notation variants to whilelistedKeys for environment, k8s cluster name, and k8s namespace - Remove unnecessary onBlur handler from environment Select component - Add unit tests for whilelistedKeys and mappingWithRoutesAndKeys Closes SigNoz#10226 * chore: run perttify --------- Co-authored-by: srikanthccv <srikanth.chekuri92@gmail.com>
* fix(meter): custom step interval support * fix(meter): custom step interval support * fix(meter): custom step interval support * fix(meter): custom step interval support * fix(meter): remove frontend harcoding for step interval * fix(meter): remove frontend harcoding for step interval
…0263) Using FINAL in clickhouse query will trigger the aggregation merge of data while using group by will be more efficient. It's recommended id docs as well - https://clickhouse.com/ docs/engines/table-engines/mergetree-family/ aggregatingmergetree#select-and-insert
… simplify PHP flow (SigNoz#10261) * feat(onboarding): add Docker support for apm data sources, Convex and simplify PHP flow - Add Convex logo asset for integration branding - Add Docker deployment options for Java frameworks (Spring Boot, Tomcat, JBoss, Quarkus) - Update JBoss configuration to include WildFly support and associated keywords - Restructure PHP onboarding flow to prioritize environment selection over framework - Update Java keywords to include JDBC and remove deprecated metrics tag * chore: remove 'angular' from related keywords in the onboarding configuration
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
* test: added test for tooltip plugin * fix: added fix for pr review changes
* test: added tests for uplotv2 utils * fix: added fix for pr review changes
…igNoz#10256) * chore: refactored the config builder and added base config builder * chore: added a common chart wrapper * chore: tsc fix * fix: pr review changes * fix: pr review changes * chore: added different tooltips * chore: removed dayjs extention * fix: added fix for pr review changes
* feat: upgraded the ingestion gateway apis * feat: updated test case and refactored code * feat: refactored the api query hooks usage and other refactoring * feat: refactored code to resolve comments * feat: refactored code to resolve comments
* feat: updated the generated apis * feat: added forgot password feature * feat: handled single org id * feat: correct the success message * feat: removed comments * feat: added test cases * feat: added loading in submit enabled condition * feat: updated styles * feat: removed light mode overrides as used semantic tokens * feat: addressed comments and feedback * feat: changed logic according to new open api spec * feat: addressed comments and used signozhq * feat: added signozhq icon in modulenamemapper * feat: used styles variables from design-token for typography * feat: refactored code to resolve comments
…SigNoz#10242) * feat: replace filter feature * feat: updated parsing field value * feat: pr comments fix
* fix: guide user on empty external api monitoring page Fixes SigNoz#3703 * fix: change language of msg to handle empty query result * fix: hide table header for empty result * fix: add styling for empty state message
* feat: adding tests for cloudintegration api * refactor: updating method * ci: updated logs * ci: fmt fixes * refactor: removing unused vars * refactor: updating test assertion * refactor: worked on review comments * fix: fixing tests * refactor: using yield * refactor: fixing lint issues * refactor: cleaning tests * refactor: review comments * refactor: removing unused imports * refactor: updating fixture name --------- Co-authored-by: Vikrant Gupta <vikrant@signoz.io>
* test: added unit tests for uplot config builders * test: added more tests * test: updated tests * fix: updated tests * fix: fixed failing test * fix: updated tests * test: added test for axis * chore: added test for thresholds with different scale key * chore: pr review comments * chore: pr review comments * fix: fixed axis tests --------- Co-authored-by: Ashwin Bhatkal <ashwin96@gmail.com>
…chart + added thresholds in scale computation (SigNoz#10287) * chore: moved spangaps to old default state in the new timeseries chart * chore: added thresholds in scale builder
* test: added tests for utils + components * fix: added fix for legend test * chore: pr review comments * chore: fixed plotcontext test * fix: added tests failure handing + moved from map based approach to array * fix: updated the way we used to consume graph visibility state * fix: fixed label spelling
* chore: shared utils update + API plumbing * chore: add tests
* refactor: merge roletypes into authtypes * refactor: merge roletypes into authtypes * refactor: update openapi spec * feat: split CI * fix: fix tsc of frontend
…igNoz#10560) * fix: check for metric type without query range constraint * revert: revert check for metric type without query range constraint * chore: move temporality+type fetcher to the case where it is actually used * fix: don't send absent metrics to query builder * chore: better package import name * test: unit test add mock for metadata call (which is expected in the test's scenario) * revert: revert seeding of absent metrics * fix: throw a not found err if metric data is missing * test: unit test add mock for metadata call (which is expected in the test's scenario) * revert: no need for special err handling in threshold rule * chore: add last seen info in err message * test: fix broken dashboard test * test: integration test for short time range query * chore: python lint issue
* fix(user): increase expiry for reset password token for invites * fix(user): increase expiry for reset password token for invites * fix(user): increase expiry for reset password token for invites * fix(user): increase expiry for reset password token for invites
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
* feat: adding cloud integration type for refactor * refactor: store interfaces to use local types and error * feat: adding updated types for cloud integration * refactor: using struct for map * refactor: update cloud integration types and module interface * fix: correct GetService signature and remove shadowed Data field * refactor: adding comments and removed wrong code * refactor: streamlining types * refactor: add comments for backward compatibility in PostableAgentCheckInRequest * refactor: update Dashboard struct comments and remove unused fields * refactor: clean up types * refactor: renaming service type to service id * refactor: using serviceID type * feat: adding method for service id creation * refactor: updating store methods * refactor: clean up * refactor: review comments
* fix: in progress status stuck in logs pipelines * fix: stuck in progress logs pipeline status * fix: changes based on review * revert: comment change * fix: change order of handling updation * fix: check newstatus deploy status
* feat: has JSON QB * fix: tests expected queries and values * fix: ignored .vscode in gitignore * fix: tests GroupBy * revert: gitignore change * fix: build json plans in metadata * fix: empty filteredArrays condition * fix: tests * fix: tests * fix: json qb test fix * fix: review based on tushar * fix: changes based on review from Srikanth * fix: remove unnecessary bool checking * fix: removed comment * fix: merge json body columns together * chore: var renamed * fix: merge conflict * test: fix * fix: tests * fix: go test flakiness * chore: merge json fields * fix: handle datatype collision * revert: few unrelated changes * revert: more unrelated change * test: blocked on pr SigNoz#10153 * feat: mapping body_v2.message:string map to body * fix: go.mod required changes * fix: remove unused function * fix: test fixed * fix: go mod changes * fix: tests * fix: go lint * revert: remvoing unused function * revert: change ReadMultiple is needed * fix: body.message not being mapped correctly * fix: append warnings from fieldkeys * fix: change warning to a const to fix tests * chore: addressing comments from Nitya * chore: remove unnecessary change * fix: shift warning attachment to getKeySelectors * fix: lint error * feat: update message as typehint in JSON Column (SigNoz#10545) * fix: cursor comments * chore: minor changes based on review * fix: message field key search in JSON Logs (SigNoz#10577) * feat: work in progress * fix: test run success * fix: in progress * fix: excluding message from metadata fetch * test: cleared * fix: key name in metadata * fix: uncomment tests * chore: change to method for staticfields * fix: remove confusing comments; remove usage of logical keyword * chore: shift method above business logic * chore: changes based on review * fix: comments in metadata_store.go * fix: fallback expr switch case * revert: remove unused JSON Field datatype * fix: remove the exception checking * chore: keep message contained to field mapper * chore: text search tests * fix: package test fixed * fix: redundant code block removal * fix: retain staticfield implementation and spell fix * fix: nil param lint --------- Co-authored-by: Srikanth Chekuri <srikanth.chekuri92@gmail.com> Co-authored-by: Nityananda Gohain <nityanandagohain@gmail.com>
* fix(identn): identn provider claims * fix(identn): add integration tests * fix(identn): use identn provider from claims
Co-authored-by: primus-bot[bot] <171087277+primus-bot[bot]@users.noreply.github.com>
…10460) * feat: replaced external apis barchart with the new bar chart * fix: tests * chore: fixed tsc
* fix(checkout): pass downstream error meesage to UI * fix(checkout): pass downstream error meesage to UI * fix(checkout): pass downstream error meesage to UI * fix(checkout): pass downstream error meesage to UI * fix(checkout): pass downstream error meesage to UI
…igNoz#10607) * fix: adding migration for fixing wrong cloud integration unique index * refactor: removing std errors pkg * refactor: normalising account_id if empty * feat: adding integration test
…letion (SigNoz#10632) * fix: handle foreign key constraint on rule and planned maintenance deletion * fix: handle foreign key constraint on rule and planned maintenance deletion * fix: handle foreign key constraint on rule and planned maintenance deletion
…Noz#10486) * feat(authz): migrate authorization to authz instead of user.role * fixup! feat(authz): migrate authorization to authz instead of user.role address comments * fixup! fixup! feat(authz): migrate authorization to authz instead of user.role Allow anonymous to go to unauthorized, otherwise it will loop in errors * fixup! fixup! fixup! feat(authz): migrate authorization to authz instead of user.role Improve error message when anonymous * fixup! fixup! fixup! fixup! feat(authz): migrate authorization to authz instead of user.role Format breaking with new css
…SigNoz#10633)" (SigNoz#10644) This reverts commit c8fcc48.
* feat: added section in panel settings * feat: added changes for spangaps thresholds * chore: minor changes * fix: fixed failing tests * fix: minor style fixes * chore: updated the categorisation * feat: added chart appearance settings in panel * feat: added fill mode in timeseries * chore: broke down rightcontainer component into sub-components * chore: minor cleanup * chore: fixed disconnect points logic * chore: added spanGaps selection component * chore: updated rightcontainer * fix: added fix for the UI * chore: fixed ui issues * chore: fixed styles * chore: default spangaps to true * chore: moved preparedata to utils * chore: hidden chart appearance * chore: pr review comments * chore: pr review comments * chore: updated variable names * chore: fixed minor changes * chore: updated test * chore: updated test
* feat(identn): implement an impersonation identn * fix: prevent nil pointer error * feat: dry org code by implementing getbyidorname * feat: add integration tests for root user and impersonation * fix: fix lint
* fix: allow gateway apis on editor access * fix: fmtlint and middleware * fix: fmtlint trailing new lines
…#10630) * feat: added section in panel settings * feat: added changes for spangaps thresholds * chore: minor changes * chore: broke down rightcontainer component into sub-components * chore: minor cleanup * chore: fixed disconnect points logic * fix: added fix for isolated points render with null both side * chore: minor naming fix * chore: fixed ui issues * chore: fixed styles * chore: hidden chart appearance * chore: pr review comments * chore: pr review comments * chore: updated conditions
…Noz#10659) Remove the DeprecatedFlags struct and all associated CLI flags (--max-idle-conns, --max-open-conns, --dial-timeout, --flux-interval, --flux-interval-for-trace-detail, --prefer-span-metrics, --cluster, --gateway-url) that were superseded by environment variable-based configuration. Deprecated environment variable handling is retained. Closes SigNoz#6805
* feat: new service_account page with crud and listing * feat: multiple style and functionality fixes * feat: multiple style and functionality fixes * feat: feedback fix * feat: added pagination and sorter * feat: feedback fix * feat: feedback and refactor * feat: updated with new schemas changes * feat: feedback and refactor * feat: added announcement banner * feat: added IS_SERVICE_ACCOUNTS_ENABLED to hide Service Account feature and announcement banner * feat: added test cases * feat: updated subtitle * feat: feedback, refactor and test enhancments * feat: enhancement in role select component * feat: enhancement and implemented nuqs usage across the Service account feature * feat: refactor drawer page to have api fetch and added cache on the list call for faster lookup * feat: refactor edit key modal to be more url controlled than props * feat: moved onsucess and onerror inside the hook * feat: enhancement in createserviceaccount modal with nuqs and refactor * feat: enhancement in addkey, editkey and disable modal * feat: refactor in revoke key modal to make it isolated and url controlled * feat: updated types after latest rebase change * feat: feedback, refactor and cleanup * feat: udpate test case
* fix(tests): fix flaky rootuser integration tests setupCompleted becomes true before the root user is actually created because the reconciliation runs in an async goroutine. Add a second polling phase that waits for /api/v1/user to return 200, confirming the root user exists and the impersonation provider can resolve. * fix(ci): fail py-fmt check when formatter changes files Add git diff --exit-code after make py-fmt so CI fails if any files are not properly formatted, instead of silently passing. Also includes black formatting fix for 01_rootuser.py. * refactor(tests): remove redundant unauthenticated request test test_unauthenticated_request_succeeds is redundant now that test_root_user_created already polls /api/v1/user without auth and waits for 200.
* chore: deprecate old user invite apis * chore: add back validation for pending user in list user apis for integration tests * fix: allow pending user to be updated * feat: updated members page with new status response and remove invite endpoint api (SigNoz#10624) * feat: updated members page with new status response and remove invite endpoint api * feat: removed deprecated invite endpoint apis * feat: delete orphaned type files * feat: changed text for copy, cancel and ingeneral messaging for invited users * feat: test case and pagination fix * feat: feedback, refactor and test mock update * feat: updated the confirmation dialog description as now the we cant permanently delete the member * feat: refactored the member status mapping * feat: used the open api spec hooks in editmember and updated the test cases * feat: added error handling --------- Co-authored-by: SagarRajput-7 <162284829+SagarRajput-7@users.noreply.github.com> Co-authored-by: SagarRajput-7 <sagar@signoz.io>
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