Skip to content

Conversation

@zreigz
Copy link
Member

@zreigz zreigz commented Jan 16, 2026

Plural Flow: docs
Plural Preview: docs

@zreigz zreigz requested a review from a team as a code owner January 16, 2026 12:24
Copy link

@plural-copilot plural-copilot bot left a comment

Choose a reason for hiding this comment

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

🔧 Deploy preview for docs-pr-562 is building...

Name Details
⚡ Service docs-pr-562 (4 / 5 ready)
☁️ Cluster plural (eks)
:octocat: Commit sha ee24a1d
Plural URL https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a...
🔍 Logs URL https://console.mgmt.plural.sh/cd/clusters/446acdef-8524-425a-a49f-5336a...
🌊 Flow URL https://console.mgmt.plural.sh/flows/08a17af1-8578-449e-9ae1-28ccd97eafc...
Preview Link Preview

You can learn more about preview environments and Plural Flows here

@linear
Copy link

linear bot commented Jan 16, 2026

| `interval` | `30m` | The base frequency for drift detection runs. |


## Resource Reconciliation Modes Table
Copy link
Member

Choose a reason for hiding this comment

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

i'm pretty sure this table is wrong, for instance stacks and servicedeployments always adopt (globalservice should too, or at least support it)

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, you are right. I suggested read-only mode. I will fix it.

@zreigz
Copy link
Member Author

zreigz commented Jan 16, 2026

@michaeljguarino I’ve reviewed the resources that support update or upsert operations, and in this case, adoption is possible.

Copy link
Member

@michaeljguarino michaeljguarino left a comment

Choose a reason for hiding this comment

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

Mostly lgtm, I think this should probably be included within the continuous deployments section though

@zreigz
Copy link
Member Author

zreigz commented Jan 16, 2026

Mostly lgtm, I think this should probably be included within the continuous deployments section though

I will move it to this section

@zreigz zreigz merged commit e54cb9e into main Jan 16, 2026
11 checks passed
@zreigz zreigz deleted the lukasz/prod-4298-document-controller-reconcilation-logic branch January 16, 2026 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants