Skip to content

feat!: extract provisioner tags from coder_workspace_tags data source #15578

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 20 commits into from
Nov 25, 2024
Merged

feat!: extract provisioner tags from coder_workspace_tags data source #15578

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
944eb3a
feat: extract provisioner tags from coder_workspace_tags data source …
johnstcn Nov 18, 2024
7bc2fc8
Merge remote-tracking branch 'origin/main' into cj/15087-extract-work…
johnstcn Nov 20, 2024
4653638
add initial framework for API tests
johnstcn Nov 20, 2024
fc8ff56
Merge remote-tracking branch 'origin/main' into cj/15087-extract-work…
johnstcn Nov 20, 2024
85c2c8f
update tests
johnstcn Nov 20, 2024
2e9526b
drop warn log
johnstcn Nov 20, 2024
01c8e99
var naming
johnstcn Nov 20, 2024
5a53d86
adjust control flow in error case
johnstcn Nov 20, 2024
c1e321d
show warning in templates push
johnstcn Nov 20, 2024
ed534bd
bump test timeout
johnstcn Nov 20, 2024
42f39a7
improve test coverage
johnstcn Nov 21, 2024
c6f33f7
fix tests
johnstcn Nov 21, 2024
b22e890
move to client-side checking for warnings
johnstcn Nov 21, 2024
a8a620f
make gen
johnstcn Nov 21, 2024
8ec672c
address indentation
johnstcn Nov 22, 2024
7b7aa40
use CacheDir for tfparse instead
johnstcn Nov 22, 2024
80dc6c2
feat: add codersdk.MatchedProvisioners
johnstcn Nov 22, 2024
5631577
update cli template push
johnstcn Nov 22, 2024
58fae97
cleanup and error handling
johnstcn Nov 22, 2024
90991bd
Apply suggestions from code review
johnstcn Nov 25, 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
cleanup and error handling
  • Loading branch information
@johnstcn
johnstcn committed Nov 22, 2024
commit 58fae9718fecc60005e3d306338db834ee77a83b
8 changes: 6 additions & 2 deletions cli/templatepush.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -417,7 +417,11 @@ func createValidTemplateVersion(inv *serpent.Invocation, args createValidTemplat
return nil, err
}
var tagsJSON strings.Builder
_ = json.NewEncoder(&tagsJSON).Encode(version.Job.Tags)
if err := json.NewEncoder(&tagsJSON).Encode(version.Job.Tags); err != nil {
// Fall back to the less-pretty string representation.
tagsJSON.Reset()
_, _ = tagsJSON.WriteString(fmt.Sprintf("%v", version.Job.Tags))
}
if version.MatchedProvisioners.Count == 0 {
cliui.Warnf(inv.Stderr, `No provisioners are available to handle the job!
Please contact your deployment administrator for assistance.
Expand All @@ -433,7 +437,7 @@ Details:
Provisioner Job ID : %s
Requested tags : %s
Most Recently Seen : %s
`, version.Job.ID, tagsJSON.String(), version.MatchedProvisioners.MostRecentlySeen.Time)
`, version.Job.ID, strings.TrimSpace(tagsJSON.String()), version.MatchedProvisioners.MostRecentlySeen.Time)
}

err = cliui.ProvisionerJob(inv.Context(), inv.Stdout, cliui.ProvisionerJobOptions{
Expand Down
Loading