Skip to content

chore: remove cloud integration service cascade delete constraint#11480

Open
swapnil-signoz wants to merge 2 commits into
mainfrom
chore/drop-cascade
Open

chore: remove cloud integration service cascade delete constraint#11480
swapnil-signoz wants to merge 2 commits into
mainfrom
chore/drop-cascade

Conversation

@swapnil-signoz
Copy link
Copy Markdown
Contributor

@swapnil-signoz swapnil-signoz commented May 27, 2026

Pull Request

Removing cascade delete constraint on cloud_integration_service table


📄 Summary

Why does this change exist?
Clean up

Closes

https://github.com/SigNoz/engineering-pod/issues/4257

@github-actions github-actions Bot added the chore label May 27, 2026
return err
}

newTable := *oldTable
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I got confused by this. The oldTable should already have foreign key references without the cascade?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, old table HAS the cascade delete fk reference which we want to delete. New table creates the new fk reference without the cascade delete.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The operator does not have support for reading cascade constraints.

Can you check what the create table for the oldtable is?

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.

2 participants