fix(query-service): skip preloading non-json files under dashboards

[prashant-shahi]

Summary

• query-service: skip preloading non-json files under dashboards

Resolves the error log:

{"level":"INFO","timestamp":"2025-01-29T17:37:18.635Z","caller":"dashboards/provision.go:26","msg":"Provisioning dashboard: ","filename":".gitkeep"}
{"level":"ERROR","timestamp":"2025-01-29T17:37:18.635Z","caller":"dashboards/provision.go:38","msg":"Creating Dashboards: Error in unmarshalling json from file","filename":".gitkeep","error":"unexpected end of JSON input","stacktrace":"go.signoz.io/signoz/pkg/query-service/app/dashboards.readCurrentDir\n\t/home/sa_100696978840572610735/signoz/pkg/query-service/app/dashboards/provision.go:38\ngo.signoz.io/signoz/pkg/query-service/app/dashboards.LoadDashboardFiles\n\t/home/sa_100696978840572610735/signoz/pkg/query-service/app/dashboards/provision.go:79\ngo.signoz.io/signoz/pkg/query-service/app.NewAPIHandler\n\t/home/sa_100696978840572610735/signoz/pkg/query-service/app/http_handler.go:271\ngo.signoz.io/signoz/ee/query-service/app/api.NewAPIHandler\n\t/home/sa_100696978840572610735/signoz/ee/query-service/app/api/api.go:56\ngo.signoz.io/signoz/ee/query-service/app.NewServer\n\t/home/sa_100696978840572610735/signoz/ee/query-service/app/server.go:287\nmain.main\n\t/home/sa_100696978840572610735/signoz/ee/query-service/main.go:191\nruntime.main\n\t/home/sa_100696978840572610735/go/pkg/mod/golang.org/[email protected]/src/runtime/proc.go:271"}

Related Issues / PR's

NA

Screenshots

NA

Affected Areas and Manually Tested Areas

---

Important

Skip non-JSON files during dashboard preloading in readCurrentDir function of provision.go.

• Behavior:
  • In readCurrentDir function in provision.go, skip files that do not have a .json extension when preloading dashboards.
  • Log a debug message for each skipped non-JSON file.
• Logging:
  • Add debug log for skipped non-JSON files in readCurrentDir.

^{This description was created by for 1aaf4fd. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 1aaf4fd in 28 seconds

More details

• Looked at 23 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. pkg/query-service/app/dashboards/provision.go:24

• Draft comment:
  The error from file.Readdirnames(0) is ignored. Consider handling this error to avoid potential issues if reading directory names fails.
• Reason this comment was not posted:
  Comment was on unchanged code.

2. pkg/query-service/app/dashboards/provision.go:27

• Draft comment:
  Avoid using inline styles in React components. Instead, use external stylesheets, CSS classes, or styled components. This comment applies to the use of inline styles in the code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code is well-structured and follows the rules provided. However, there is a minor issue with the use of inline styles in React components, which should be avoided.

Workflow ID: wflow_ewAusl1pIeh0tD97

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.