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

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
Format
  • Loading branch information
@Emyrk
Emyrk committed May 24, 2023
commit a86bcd67e2904cf3c09203fcf11c0f3adbcffda1
16 changes: 7 additions & 9 deletions coderd/metricscache/metricscache.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,15 +21,13 @@ import (
"github.com/coder/retry"
)

var (
// timezoneOffsets are the timezones that are cached and supported.
// Any non-listed timezone offsets will need to use the closest supported one.
timezoneOffsets = []int{
0, // UTC - is listed first intentionally.
-12, -11, -10, -9, -8, -7, -6, -5, -4, -3, -2, -1,
1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12,
}
)
// timezoneOffsets are the timezones that are cached and supported.
// Any non-listed timezone offsets will need to use the closest supported one.
var timezoneOffsets = []int{
0, // UTC - is listed first intentionally.
-12, -11, -10, -9, -8, -7, -6, -5, -4, -3, -2, -1,
1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12,
}

// Cache holds the template metrics.
// The aggregation queries responsible for these values can take up to a minute
Expand Down