Skip to content

feat: improve resources_monitoring for OOM & OOD monitoring #16241

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.

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 28 commits into from
Feb 4, 2025
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
9c4071e
work on resources monitoring
defelmnq Jan 23, 2025
5609041
work on migrations and db
defelmnq Jan 23, 2025
fd2dbe7
work on database integration
defelmnq Jan 23, 2025
d2b42b7
add metadata logic to db
defelmnq Jan 27, 2025
893c54c
add metadata logic to db
defelmnq Jan 27, 2025
15a3837
revert formatting
defelmnq Jan 27, 2025
de9e73a
improve errors handling
defelmnq Jan 27, 2025
d8568a3
work on fake db
defelmnq Jan 27, 2025
87737eb
work on db
defelmnq Jan 27, 2025
5cd023f
update provisioner version
defelmnq Jan 27, 2025
25aedf0
work on testdata
defelmnq Jan 28, 2025
a5f44aa
merge
defelmnq Jan 29, 2025
6f0373f
work on resources monitor
defelmnq Jan 29, 2025
4bfb4cb
worked on dbmem and dbauthz
defelmnq Jan 29, 2025
68b022f
add dbauthz tests
defelmnq Jan 29, 2025
5bc849e
work on errors message improvement
defelmnq Jan 31, 2025
8a79421
improve database indexes
defelmnq Feb 3, 2025
7d9a05b
Merge remote-tracking branch 'origin/main' into agent_resource_monito…
defelmnq Feb 3, 2025
93609ca
merge main and rename migration
defelmnq Feb 3, 2025
1bf0866
fix resources volumes allocation
defelmnq Feb 3, 2025
ffb6f91
regenerate
defelmnq Feb 3, 2025
aacc294
rename fixture
defelmnq Feb 3, 2025
961ab25
use agent_id uuid
defelmnq Feb 3, 2025
d7f57d7
work on tests
defelmnq Feb 3, 2025
401b000
work on tests
defelmnq Feb 3, 2025
930bfbb
add tests on provisionerserver
defelmnq Feb 3, 2025
fae8dc9
remove unrequired test case
defelmnq Feb 3, 2025
d98ea6f
Merge remote-tracking branch 'origin/main' into agent_resource_monito…
defelmnq Feb 4, 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 metadata logic to db
  • Loading branch information
defelmnq committed Jan 27, 2025
commit d2b42b7d6c3c158cbbbc23f2dafed61909d046dc
16 changes: 6 additions & 10 deletions coderd/database/dbauthz/dbauthz.go
Original file line number Diff line number Diff line change
Expand Up @@ -1391,8 +1391,8 @@ func (q *querier) FavoriteWorkspace(ctx context.Context, id uuid.UUID) error {
return update(q.log, q.auth, fetch, q.db.FavoriteWorkspace)(ctx, id)
}

func (q *querier) FetchAgentResourcesMonitoringByAgentID(ctx context.Context, agentID uuid.UUID) (database.AgentResourcesMonitoring, error) {
return q.db.FetchAgentResourcesMonitoringByAgentID(ctx, agentID)
func (q *querier) FetchAgentResourceMonitorsByAgentID(ctx context.Context, agentID uuid.UUID) (database.WorkspaceAgentResourceMonitor, error) {
panic("not implemented")
}

func (q *querier) FetchNewMessageMetadata(ctx context.Context, arg database.FetchNewMessageMetadataParams) (database.FetchNewMessageMetadataRow, error) {
Expand All @@ -1402,10 +1402,6 @@ func (q *querier) FetchNewMessageMetadata(ctx context.Context, arg database.Fetc
return q.db.FetchNewMessageMetadata(ctx, arg)
}

func (q *querier) FlushAgentResourcesMonitoringForAgentID(ctx context.Context, agentID uuid.UUID) error {
return q.db.FlushAgentResourcesMonitoringForAgentID(ctx, agentID)
}

func (q *querier) GetAPIKeyByID(ctx context.Context, id string) (database.APIKey, error) {
return fetch(q.log, q.auth, q.db.GetAPIKeyByID)(ctx, id)
}
Expand Down Expand Up @@ -2902,10 +2898,6 @@ func (q *querier) InsertAPIKey(ctx context.Context, arg database.InsertAPIKeyPar
q.db.InsertAPIKey)(ctx, arg)
}

func (q *querier) InsertAgentResourcesMonitoring(ctx context.Context, arg database.InsertAgentResourcesMonitoringParams) (database.AgentResourcesMonitoring, error) {
return q.db.InsertAgentResourcesMonitoring(ctx, arg)
}

func (q *querier) InsertAllUsersGroup(ctx context.Context, organizationID uuid.UUID) (database.Group, error) {
// This method creates a new group.
return insert(q.log, q.auth, rbac.ResourceGroup.InOrg(organizationID), q.db.InsertAllUsersGroup)(ctx, organizationID)
Expand Down Expand Up @@ -3226,6 +3218,10 @@ func (q *querier) InsertWorkspaceAgentMetadata(ctx context.Context, arg database
return q.db.InsertWorkspaceAgentMetadata(ctx, arg)
}

func (q *querier) InsertWorkspaceAgentResourceMonitor(ctx context.Context, arg database.InsertWorkspaceAgentResourceMonitorParams) (database.WorkspaceAgentResourceMonitor, error) {
panic("not implemented")
}

func (q *querier) InsertWorkspaceAgentScriptTimings(ctx context.Context, arg database.InsertWorkspaceAgentScriptTimingsParams) (database.WorkspaceAgentScriptTiming, error) {
if err := q.authorizeContext(ctx, policy.ActionCreate, rbac.ResourceSystem); err != nil {
return database.WorkspaceAgentScriptTiming{}, err
Expand Down
24 changes: 10 additions & 14 deletions coderd/database/dbmem/dbmem.go
Original file line number Diff line number Diff line change
Expand Up @@ -2355,7 +2355,7 @@ func (q *FakeQuerier) FavoriteWorkspace(_ context.Context, arg uuid.UUID) error
return nil
}

func (q *FakeQuerier) FetchAgentResourcesMonitoringByAgentID(ctx context.Context, agentID uuid.UUID) (database.AgentResourcesMonitoring, error) {
func (q *FakeQuerier) FetchAgentResourceMonitorsByAgentID(ctx context.Context, agentID uuid.UUID) (database.WorkspaceAgentResourceMonitor, error) {
panic("not implemented")
}

Expand Down Expand Up @@ -2391,10 +2391,6 @@ func (q *FakeQuerier) FetchNewMessageMetadata(_ context.Context, arg database.Fe
}, nil
}

func (q *FakeQuerier) FlushAgentResourcesMonitoringForAgentID(ctx context.Context, agentID uuid.UUID) error {
panic("not implemented")
}

func (q *FakeQuerier) GetAPIKeyByID(_ context.Context, id string) (database.APIKey, error) {
q.mutex.RLock()
defer q.mutex.RUnlock()
Expand Down Expand Up @@ -7517,15 +7513,6 @@ func (q *FakeQuerier) InsertAPIKey(_ context.Context, arg database.InsertAPIKeyP
return key, nil
}

func (q *FakeQuerier) InsertAgentResourcesMonitoring(ctx context.Context, arg database.InsertAgentResourcesMonitoringParams) (database.AgentResourcesMonitoring, error) {
err := validateDatabaseType(arg)
if err != nil {
return database.AgentResourcesMonitoring{}, err
}

panic("not implemented")
}

func (q *FakeQuerier) InsertAllUsersGroup(ctx context.Context, orgID uuid.UUID) (database.Group, error) {
return q.InsertGroup(ctx, database.InsertGroupParams{
ID: orgID,
Expand Down Expand Up @@ -8542,6 +8529,15 @@ func (q *FakeQuerier) InsertWorkspaceAgentMetadata(_ context.Context, arg databa
return nil
}

func (q *FakeQuerier) InsertWorkspaceAgentResourceMonitor(ctx context.Context, arg database.InsertWorkspaceAgentResourceMonitorParams) (database.WorkspaceAgentResourceMonitor, error) {
err := validateDatabaseType(arg)
if err != nil {
return database.WorkspaceAgentResourceMonitor{}, err
}

panic("not implemented")
}

func (q *FakeQuerier) InsertWorkspaceAgentScriptTimings(_ context.Context, arg database.InsertWorkspaceAgentScriptTimingsParams) (database.WorkspaceAgentScriptTiming, error) {
err := validateDatabaseType(arg)
if err != nil {
Expand Down
27 changes: 10 additions & 17 deletions coderd/database/dbmetrics/querymetrics.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

58 changes: 22 additions & 36 deletions coderd/database/dbmock/dbmock.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

28 changes: 14 additions & 14 deletions coderd/database/dump.sql

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

This file was deleted.

This file was deleted.

Loading