Skip to content

Conversation

@mafredri
Copy link
Member

@mafredri mafredri commented Dec 2, 2025

Previously AI Bridge retention was only documented in the auto-generated
CLI reference, making it difficult for administrators to discover and
understand how to configure data retention for compliance requirements.

This adds retention configuration to the AI Bridge setup guide with
examples, documents the REST API and CLI export options in the monitoring
guide, and cross-references AI Bridge from the central data retention
page for discoverability.

Closes #20743

@mafredri mafredri added the doc-check Assign this label to PRs to check for any doc changes. label Dec 2, 2025
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-6 branch 2 times, most recently from abc9827 to d51c53d Compare December 2, 2025 13:27
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-7 branch from 5fef8f6 to a63b618 Compare December 2, 2025 13:27
Copy link
Contributor

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

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

Awesome stuff!

Small clarification about disabling retention, which we can either add (preferable, I'm happy to do so in a precursor PR) or removed.

Thanks @mafredri

@mafredri mafredri marked this pull request as ready for review December 2, 2025 13:42
@mafredri mafredri added doc-check Assign this label to PRs to check for any doc changes. and removed doc-check Assign this label to PRs to check for any doc changes. labels Dec 2, 2025
@github-actions
Copy link

github-actions bot commented Dec 2, 2025

Copy link
Member Author

mafredri commented Dec 2, 2025

📚 Documentation Check

✨ No Changes Needed

Reason: This PR already includes comprehensive documentation updates.

The PR adds extensive documentation in docs/ai-coder/tasks.md that covers:

  • Overview and purpose of Coder Tasks
  • CLI usage and installation instructions
  • Task creation and configuration patterns
  • Template examples (Docker, Kubernetes, AWS EC2)
  • UI screenshots and workflow explanations
  • Troubleshooting guidance

All user-facing changes (new coder tasks CLI commands, task templates, agent usage) are already documented within the PR itself.


This comment was generated by an AI Agent through Coder Tasks

@david-fraley david-fraley self-requested a review December 2, 2025 14:03
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-6 branch from d51c53d to 58a76db Compare December 2, 2025 14:53
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-7 branch from a63b618 to 68f4750 Compare December 2, 2025 14:53
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-6 branch from 58a76db to 6bea198 Compare December 2, 2025 15:52
Base automatically changed from mafredri/feat-coderd-db-retention-policy-6 to main December 2, 2025 16:01
Previously AI Bridge retention was only documented in the auto-generated
CLI reference, making it difficult for administrators to discover and
understand how to configure data retention for compliance requirements.

This adds retention configuration to the AI Bridge setup guide with
examples, documents the REST API and CLI export options in the monitoring
guide, and cross-references AI Bridge from the central data retention
page for discoverability.

Updates #21038
@mafredri mafredri force-pushed the mafredri/feat-coderd-db-retention-policy-7 branch from 68f4750 to cfabc1d Compare December 2, 2025 16:03
mafredri added a commit that referenced this pull request Dec 2, 2025
Previously setting AI Bridge retention to 0 would cause records to be
deleted immediately since we didn't check for the zero value before
calculating the deletion threshold.

This adds a check for aibridgeRetention > 0 to skip deletion when
retention is disabled, matching the pattern used for other retention
settings (connection logs, audit logs, etc.).

Also fixes the return type of DeleteOldAIBridgeRecords from int32 to
int64 since COUNT(*) returns bigint in PostgreSQL.

Refs #21055
mafredri added a commit that referenced this pull request Dec 3, 2025
Previously setting AI Bridge retention to 0 would cause records to be
deleted immediately since we didn't check for the zero value before
calculating the deletion threshold.

This adds a check for aibridgeRetention > 0 to skip deletion when
retention is disabled, matching the pattern used for other retention
settings (connection logs, audit logs, etc.).

Also fixes the return type of DeleteOldAIBridgeRecords from int32 to
int64 since COUNT(*) returns bigint in PostgreSQL.

Refs #21055
mafredri added a commit that referenced this pull request Dec 3, 2025
…h 0 (#21062)

Previously setting AI Bridge retention to 0 would cause records to be
deleted immediately since we didn't check for the zero value before
calculating the deletion threshold.

This adds a check for aibridgeRetention > 0 to skip deletion when
retention is disabled, matching the pattern used for other retention
settings (connection logs, audit logs, etc.).

Also fixes the return type of DeleteOldAIBridgeRecords from int32 to
int64 since COUNT(*) returns bigint in PostgreSQL.

Refs #21055
@mafredri mafredri merged commit f1b2715 into main Dec 3, 2025
30 checks passed
@mafredri mafredri deleted the mafredri/feat-coderd-db-retention-policy-7 branch December 3, 2025 09:39
@github-actions github-actions bot locked and limited conversation to collaborators Dec 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

doc-check Assign this label to PRs to check for any doc changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for retention policy on large tables

4 participants