Skip to content

docs: updated information on operator default value#5382

Merged
akhilmhdh merged 1 commit intomainfrom
docs/operator
Feb 6, 2026
Merged

docs: updated information on operator default value#5382
akhilmhdh merged 1 commit intomainfrom
docs/operator

Conversation

@akhilmhdh
Copy link
Member

Context

Updates information on k8s operator resync interval value based of instant updates

Screenshots

Steps to verify the change

Type

  • Fix
  • Feature
  • Improvement
  • Breaking
  • Docs
  • Chore

Checklist

  • Title follows the conventional commit format: type(scope): short description (scope is optional, e.g., fix: prevent crash on sync or fix(api): handle null response).
  • Tested locally
  • Updated docs (if needed)
  • Read the contributing guide

@akhilmhdh akhilmhdh requested a review from varonix0 February 5, 2026 20:40
@maidul98
Copy link
Collaborator

maidul98 commented Feb 5, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 5, 2026

Greptile Overview

Greptile Summary

Updates the Kubernetes InfisicalSecret CRD documentation to clarify that syncConfig.resyncInterval’s default depends on whether syncConfig.instantUpdates is enabled.

This change lives in the operator CRD reference docs (docs/integrations/platforms/kubernetes/infisical-secret-crd.mdx) and is meant to align user expectations about sync frequency when instant updates are used.

Confidence Score: 4/5

  • This PR is safe to merge once the docs are made internally consistent.
  • Change is limited to documentation and introduces no runtime/security impact, but the updated default description currently conflicts with an existing sentence in the same section and could mislead users about operator behavior.
  • docs/integrations/platforms/kubernetes/infisical-secret-crd.mdx

Important Files Changed

Filename Overview
docs/integrations/platforms/kubernetes/infisical-secret-crd.mdx Updates resyncInterval default-value docs to depend on instantUpdates; verify default is truly 1h when instantUpdates=true and ensure spacing/consistency across page.

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 5, 2026

Additional Comments (1)

docs/integrations/platforms/kubernetes/infisical-secret-crd.mdx
Conflicting default values

This section now states a conditional default (1m when instantUpdates=false, 1h when instantUpdates=true), but the preceding sentence still says it “will default to 60 seconds if not defined” (docs/integrations/platforms/kubernetes/infisical-secret-crd.mdx:73-84). These can’t both be true; please update one so the doc has a single authoritative default (and ideally link/mention the exact operator version/behavior if it changed).

@akhilmhdh akhilmhdh changed the title doc: updated information on operator default value docs: updated information on operator default value Feb 6, 2026
@akhilmhdh akhilmhdh merged commit f5a7dab into main Feb 6, 2026
8 of 9 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