Skip to content

feat: implement runtime configuration package with multi-org support #14624

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 21 commits into from
Sep 9, 2024
Merged

feat: implement runtime configuration package with multi-org support #14624

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
5c92688
Implementation
dannykopping Aug 28, 2024
03ecbac
Support zero values
dannykopping Sep 3, 2024
f97ca89
Add database queries against site_configs
dannykopping Sep 3, 2024
b2b43c8
Document usage in test
dannykopping Sep 3, 2024
e67da12
Touchups
dannykopping Sep 3, 2024
127f1cc
make gen to generate dbmock functions
Emyrk Sep 3, 2024
43e9e85
make lint/fmt
dannykopping Sep 4, 2024
fa7193e
Initializer interface, rename "key" to "name"
dannykopping Sep 4, 2024
31f8cbf
Manager interface
dannykopping Sep 4, 2024
048506d
Manager, with scoping
dannykopping Sep 4, 2024
1940ca0
linting errors
Emyrk Sep 4, 2024
d8ad4d4
chore: add runtimeconfig manager to options for plumbing (#14562)
Emyrk Sep 4, 2024
f44f1c7
linting, remove unused ctx
Emyrk Sep 5, 2024
7452a23
chore: refactor entry into deployment and runtime (#14575)
Emyrk Sep 5, 2024
13ea99a
chore: comments, cleanup, error fixes
Emyrk Sep 6, 2024
7d08255
increase test coverage:
Emyrk Sep 6, 2024
4ec16cd
linting complained about an unhandled Println error
Emyrk Sep 6, 2024
9f546d0
drop manager interface, wrap errors
Emyrk Sep 6, 2024
56f4df2
rename NewStoreManager -> NewManager
Emyrk Sep 6, 2024
8eb46cb
chore: remove DeploymentEntry and Coalesce
Emyrk Sep 9, 2024
081fcbf
fmt
Emyrk Sep 9, 2024
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
make lint/fmt 
Signed-off-by: Danny Kopping <danny@coder.com>
  • Loading branch information
@dannykopping @Emyrk
dannykopping authored and Emyrk committed Sep 6, 2024
commit 43e9e85cce856420f2b79f20e438f2b9f02dd1e0
2 changes: 1 addition & 1 deletion coderd/runtimeconfig/resolver.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ func (r OrgResolver) GetRuntimeSetting(ctx context.Context, key string) (string,

// NoopResolver will always fail to resolve the given key.
// Useful in tests where you just want to look up the startup value of configs, and are not concerned with runtime config.
type NoopResolver struct {}
type NoopResolver struct{}

func NewNoopResolver() *NoopResolver {
return &NoopResolver{}
Expand Down
2 changes: 1 addition & 1 deletion coderd/runtimeconfig/util.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,4 @@ func create[T any]() T {

func orgKey(orgID uuid.UUID, key string) string {
return fmt.Sprintf("%s:%s", orgID.String(), key)
}
}