Skip to content

feat: Fix Deployment DAUs to work with local timezones #7647

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.

Sign up for GitHub

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

Merged
merged 18 commits into from
May 30, 2023
Merged

feat: Fix Deployment DAUs to work with local timezones #7647

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
f7f5d66
chore: Add timezone param to DAU SQL query
Emyrk May 23, 2023
45e8342
Merge DAUs response
Emyrk May 23, 2023
b502940
Pass time offsets to metricscache
Emyrk May 23, 2023
9b970ae
Fix unit tests
Emyrk May 23, 2023
4eb43c1
Add unit tests for tzs to offsets
Emyrk May 23, 2023
1949217
Add unit test for closest
Emyrk May 23, 2023
b5b5824
Return after http error
Emyrk May 23, 2023
448d5fb
make gen
Emyrk May 23, 2023
5976ac4
add comments
Emyrk May 23, 2023
6ff576b
Fix imports
Emyrk May 23, 2023
a91debd
missing import
Emyrk May 23, 2023
0d19300
add unit test
Emyrk May 23, 2023
d7e8b25
Formatting
Emyrk May 23, 2023
bf1bd47
Fix FE to use singulare DAUsResponse
Emyrk May 23, 2023
dc324a8
Fix unit test
Emyrk May 23, 2023
bb7c392
Make gen
Emyrk May 23, 2023
16c9c8f
Format
Emyrk May 24, 2023
a86bcd6
Format
Emyrk May 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add comments
  • Loading branch information
@Emyrk
Emyrk committed May 23, 2023
commit 5976ac4ac5490133e7ae8393c21d05141486a957
3 changes: 0 additions & 3 deletions coderd/metricscache/metricscache.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,12 +10,9 @@ import (
"time"

"golang.org/x/xerrors"

"golang.org/x/exp/maps"
"golang.org/x/exp/slices"

"github.com/google/uuid"

"cdr.dev/slog"
"github.com/coder/coder/coderd/database"
"github.com/coder/coder/coderd/database/dbauthz"
Expand Down
2 changes: 2 additions & 0 deletions codersdk/deployment.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1756,6 +1756,8 @@ func (c *Client) DeploymentDAUsLocalTZ(ctx context.Context) (*DAUsResponse, erro
return c.DeploymentDAUs(ctx, TimezoneOffsetHour(time.Local))
}

// DeploymentDAUs requires a tzOffset in hours. Use 0 for UTC, and TimezoneOffsetHour(time.Local) for the
// local timezone.
func (c *Client) DeploymentDAUs(ctx context.Context, tzOffset int) (*DAUsResponse, error) {
res, err := c.Request(ctx, http.MethodGet, "/api/v2/insights/daus", nil, DAURequest{
TZHourOffset: tzOffset,
Expand Down
2 changes: 2 additions & 0 deletions codersdk/templates.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,8 @@ func (c *Client) TemplateDAUsLocalTZ(ctx context.Context, templateID uuid.UUID)
return c.TemplateDAUs(ctx, templateID, TimezoneOffsetHour(time.Local))
}

// TemplateDAUs requires a tzOffset in hours. Use 0 for UTC, and TimezoneOffsetHour(time.Local) for the
// local timezone.
func (c *Client) TemplateDAUs(ctx context.Context, templateID uuid.UUID, tzOffset int) (*DAUsResponse, error) {
res, err := c.Request(ctx, http.MethodGet, fmt.Sprintf("/api/v2/templates/%s/daus", templateID), nil, DAURequest{
TZHourOffset: tzOffset,
Expand Down