Skip to content

chore: disable parameter validatation for dynamic params for all transitions #17926

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 16 commits into from
May 20, 2025
Merged

chore: disable parameter validatation for dynamic params for all transitions #17926

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
d4d52e9
chore: unit test to showcase param validation drift
Emyrk May 19, 2025
f6b6485
trying some things
Emyrk May 19, 2025
2b03908
chore: dynamic parameters on workspace builds, not just create
Emyrk May 20, 2025
266b1e0
working on enabling dynamic params for builds
Emyrk May 20, 2025
10fe129
working on enabling dynamic params for builds
Emyrk May 20, 2025
0c648fb
use expierment in detection
Emyrk May 20, 2025
b90650d
add experiments to wsbuilder
Emyrk May 20, 2025
671ef69
linting
Emyrk May 20, 2025
07fc321
make gen
Emyrk May 20, 2025
93b0974
add provisioenr version check too
Emyrk May 20, 2025
8100193
add unit test
Emyrk May 20, 2025
1ce5e7b
Merge remote-tracking branch 'origin/main' into stevenmasley/deleting…
Emyrk May 20, 2025
1e4c183
linting
Emyrk May 20, 2025
cfe2120
PR suggestions, mostly comments
Emyrk May 20, 2025
2cd42e3
Merge remote-tracking branch 'origin/main' into stevenmasley/deleting…
Emyrk May 20, 2025
0944436
make gen
Emyrk May 20, 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
working on enabling dynamic params for builds
  • Loading branch information
@Emyrk
Emyrk committed May 20, 2025
commit 10fe129869501372c8c6b5e5c2d61bb681c26a4a
1 change: 1 addition & 0 deletions coderd/workspacebuilds.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -338,6 +338,7 @@
RichParameterValues(createBuild.RichParameterValues).
LogLevel(string(createBuild.LogLevel)).
DeploymentValues(api.Options.DeploymentValues).
Experiments(api.Experiments).

Check failure on line 341 in coderd/workspacebuilds.go

View workflow job for this annotation

GitHub Actions / gen 

wsbuilder.New(workspace, database.WorkspaceTransition(createBuild.Transition)).Initiator(apiKey.UserID).RichParameterValues(createBuild.RichParameterValues).LogLevel(string(createBuild.LogLevel)).DeploymentValues(api.Options.DeploymentValues).Experiments undefined (type wsbuilder.Builder has no field or method Experiments, but does have unexported field experiments)

Check failure on line 341 in coderd/workspacebuilds.go

View workflow job for this annotation

GitHub Actions / test-e2e 

wsbuilder.New(workspace, database.WorkspaceTransition(createBuild.Transition)).Initiator(apiKey.UserID).RichParameterValues(createBuild.RichParameterValues).LogLevel(string(createBuild.LogLevel)).DeploymentValues(api.Options.DeploymentValues).Experiments undefined (type wsbuilder.Builder has no field or method Experiments, but does have unexported field experiments)
TemplateVersionPresetID(createBuild.TemplateVersionPresetID)

var (
Expand Down
Loading