Skip to content

feat: add slug property to app, use in URLs #4573

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 18 commits into from
Oct 28, 2022
Merged

feat: add slug property to app, use in URLs #4573

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
6889450
feat: add slug property to app, use in URLs
deansheather Oct 16, 2022
ec5ecfe
fixup! feat: add slug property to app, use in URLs
deansheather Oct 16, 2022
e6ddf3c
fixup! feat: add slug property to app, use in URLs
deansheather Oct 17, 2022
43df27e
feat: app slugs pt.2
deansheather Oct 18, 2022
c658572
feat: app slugs pt.3
deansheather Oct 18, 2022
d174a90
Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 19, 2022
7a79f4a
fixup! Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 19, 2022
4efc389
chore: upgrade coder tf provider to 0.6.0
deansheather Oct 21, 2022
fecef1e
chore: add terraform provider test for slug validity
deansheather Oct 21, 2022
1a5eabc
chore: add app slug uniqueness check to coderd
deansheather Oct 22, 2022
4086202
Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 22, 2022
9d41a46
chore: fix and improve app slug migrations
deansheather Oct 22, 2022
5b607e8
Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 25, 2022
b53611f
fixup! Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 25, 2022
b06dcad
Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 28, 2022
3e07cf1
fixup! Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 28, 2022
6aac60a
fixup! Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 28, 2022
a423ce8
fixup! Merge branch 'main' into dean/app-urls-use-slug
deansheather Oct 28, 2022
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
chore: add terraform provider test for slug validity
  • Loading branch information
@deansheather
deansheather committed Oct 21, 2022
commit fecef1e42a593b9992b9169816602be4b9d744b9
6 changes: 6 additions & 0 deletions provisioner/terraform/resources.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -220,6 +220,7 @@ func ConvertResources(module *tfjson.StateModule, rawGraph string) ([]*proto.Res
}

// Associate Apps with agents.
appSlugs := make(map[string]struct{})
for _, resource := range tfResourceByLabel {
if resource.Type != "coder_app" {
continue
Expand Down Expand Up @@ -248,6 +249,11 @@ func ConvertResources(module *tfjson.StateModule, rawGraph string) ([]*proto.Res
return nil, xerrors.Errorf("invalid app slug %q, please update your coder/coder provider to the latest version and specify the slug property on each coder_app", attrs.Slug)
}

if _, exists := appSlugs[attrs.Slug]; exists {
return nil, xerrors.Errorf("duplicate app slug, they must be unique per template: %q", attrs.Slug)
}
appSlugs[attrs.Slug] = struct{}{}

var healthcheck *proto.Healthcheck
if len(attrs.Healthcheck) != 0 {
healthcheck = &proto.Healthcheck{
Expand Down
Loading