Skip to content

chore: track usage of organizations in telemetry (abandoned) #16307

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

Closed
wants to merge 8 commits into from
Closed

chore: track usage of organizations in telemetry (abandoned) #16307

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
9672202
add IDPOrgSync to deployment in telemetry
hugodutka Jan 28, 2025
0fa4401
report organizations in telemetry
hugodutka Jan 28, 2025
738faf6
ensure relevant resources have the organization id in telemetry
hugodutka Jan 28, 2025
5aa6149
set up an authorization context for OrganizationSyncEnabled
hugodutka Jan 28, 2025
c2bf9dc
lint
hugodutka Jan 28, 2025
24b29e2
add auth context to db call
hugodutka Jan 28, 2025
61ff880
deferred initialization of OrganizationSyncEnabled
hugodutka Jan 28, 2025
67f69d6
test OrganizationSyncEnabled
hugodutka Jan 28, 2025
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 auth context to db call
  • Loading branch information
@hugodutka
hugodutka committed Jan 28, 2025
commit 24b29e25c2d493a5702b774bc0015100bf3e4f9e
5 changes: 4 additions & 1 deletion coderd/telemetry/telemetry.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import (
"github.com/coder/coder/v2/buildinfo"
clitelemetry "github.com/coder/coder/v2/cli/telemetry"
"github.com/coder/coder/v2/coderd/database"
"github.com/coder/coder/v2/coderd/database/dbauthz"
"github.com/coder/coder/v2/coderd/database/dbtime"
"github.com/coder/coder/v2/codersdk"
tailnetproto "github.com/coder/coder/v2/tailnet/proto"
Expand Down Expand Up @@ -536,7 +537,9 @@ func (r *remoteReporter) createSnapshot() (*Snapshot, error) {
// no longer be reported, and there will be no other indicator that it
// was deleted. This requires special handling when interpreting the
// telemetry data later.
orgs, err := r.options.Database.GetOrganizations(ctx, database.GetOrganizationsParams{})
// nolint:gocritic // AsSystemRestricted is fine here because it's a read-only operation
// used for telemetry reporting.
orgs, err := r.options.Database.GetOrganizations(dbauthz.AsSystemRestricted(r.ctx), database.GetOrganizationsParams{})
if err != nil {
return xerrors.Errorf("get organizations: %w", err)
}
Expand Down
Loading