Skip to content

bugfix: add plan-time regex validation for telemetry link name and description - #1593

Merged
GokceGK merged 6 commits into
stackitcloud:mainfrom
suedaozkaya:bugfix/STACKITTRO-1041-telemetry-link-validation
Aug 11, 2026
Merged

bugfix: add plan-time regex validation for telemetry link name and description#1593
GokceGK merged 6 commits into
stackitcloud:mainfrom
suedaozkaya:bugfix/STACKITTRO-1041-telemetry-link-validation

Conversation

@suedaozkaya

Copy link
Copy Markdown
Contributor

Description

This PR adds plan-time regex validation to the telemetry link's name and description fields to catch invalid inputs locally and avoid raw 400 API errors.

relates to STACKITTRO-1041

Checklist

  • Issue was linked above
  • Code format was applied: make fmt
  • Examples were added / adjusted (see examples/ directory)
  • Docs are up-to-date: make generate-docs (will be checked by CI)
  • Unit tests got implemented or updated
  • Acceptance tests got implemented or updated (see e.g. here)
  • Unit tests are passing: make test (will be checked by CI)
  • No linter issues: make lint (will be checked by CI)

@suedaozkaya
suedaozkaya requested a review from a team as a code owner July 16, 2026 17:12
Comment thread stackit/internal/services/telemetrylink/link/resource.go
@Fyusel Fyusel self-assigned this Jul 21, 2026
Comment thread stackit/internal/services/telemetryrouter/accesstoken/resource.go Outdated
Comment thread stackit/internal/services/telemetryrouter/destination/resource.go Outdated
Comment thread stackit/internal/services/telemetryrouter/instance/resource.go Outdated
@github-actions

Copy link
Copy Markdown

This PR was marked as stale after 7 days of inactivity and will be closed after another 7 days of further inactivity. If this PR should be kept open, just add a comment, remove the stale label or push new commits to it.

@github-actions github-actions Bot added the Stale PR is marked as stale due to inactivity. label Jul 31, 2026
@Fyusel Fyusel removed the Stale PR is marked as stale due to inactivity. label Jul 31, 2026
Fyusel
Fyusel previously approved these changes Aug 7, 2026
Comment thread stackit/internal/services/telemetryrouter/accesstoken/resource.go Outdated
Comment thread stackit/internal/services/telemetryrouter/destination/resource.go Outdated
Comment thread stackit/internal/services/telemetryrouter/instance/resource.go Outdated
@GokceGK
GokceGK merged commit 6e395ed into stackitcloud:main Aug 11, 2026
3 checks passed
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.

3 participants