Skip to content

feat: add workspace_sharing attribute to coderd_organization resource#295

Merged
geokat merged 11 commits intomainfrom
geokat/internal-1245-add-coderd-organization-workspace-sharing-attr
Feb 5, 2026
Merged

feat: add workspace_sharing attribute to coderd_organization resource#295
geokat merged 11 commits intomainfrom
geokat/internal-1245-add-coderd-organization-workspace-sharing-attr

Conversation

@geokat
Copy link
Contributor

@geokat geokat commented Jan 21, 2026

Summary

  • Add workspace_sharing to the coderd_organization resource and data source

Testing

  • Organization resource/data source tests now cover:
    • experiments enabled
    • experiments disabled
    • older coderd without the endpoint

Other changes

  • Refactor StartCoder to use functional options (image, version, experiments, license)
  • Upgrade dependencies: cdr.dev/slog/v3, github.com/coder/coder/v2 → v2.30.0
  • CI/tooling updates to support Go 1.25:
    • Release workflow Go toolchain → 1.25.6
    • golangci-lint action -> v8, lint tool -> v2.8.0
    • Migrate .golangci.yml to v2 schema (v1 is not supported by v2.8.0)
  • Lint-driven fixes in template resource (flagged by the new linter):
    • check Close() errors in defers
    • adjust newVersion() return order and update call sites

Related to: coder/internal#1245

- Also adds it to the coderd_organization data source
- Also adds an option to the test harness to test against features
  gated by experiments (enabled via the CODER_EXPERIMENTS env var)

Related to: coder/internal#1245
@geokat geokat linked an issue Jan 21, 2026 that may be closed by this pull request
@geokat geokat requested a review from aslilac January 21, 2026 20:58
Copy link
Member

@aslilac aslilac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one small thing but otherwise looks great!

Gracefully handle older coderd versions or disabled workspace-sharing
experiment, and bump coder dependency to the new official mainline
version (v2.30.0).
@geokat geokat force-pushed the geokat/internal-1245-add-coderd-organization-workspace-sharing-attr branch 2 times, most recently from 354f4ac to 11f12ab Compare February 3, 2026 19:59
go v1.25 is now required by the dependencies we pull in (coder/v2).

- pin golangci-lint to v2.4.0 in CI
- update .golangci.yml to v2 schema (default none, formatter config, settings move)
- bump golangci-lint-action to v8 so v2.x installs correctly
@geokat geokat force-pushed the geokat/internal-1245-add-coderd-organization-workspace-sharing-attr branch from 11f12ab to 77774a6 Compare February 3, 2026 20:51
- check error return values
- return error as last value (ST1008)
Allow integration tests to provide Coder image/version/experiments
while deferring to environment overrides.
Add no-experiment and backward-compat paths for resource/data source
tests, using shared helpers to avoid duplication.
@geokat geokat marked this pull request as ready for review February 4, 2026 17:20
@geokat geokat requested a review from ethanndickson February 4, 2026 17:20
Copy link
Member

@ethanndickson ethanndickson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@geokat geokat merged commit 96ca475 into main Feb 5, 2026
25 of 26 checks passed
@geokat geokat deleted the geokat/internal-1245-add-coderd-organization-workspace-sharing-attr branch February 5, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add coderd_organization.workspace_sharing attribute

3 participants