Skip to content

feat(cas): add managed flag#3092

Open
Piskoo wants to merge 1 commit intochainloop-dev:mainfrom
Piskoo:pfm-5834
Open

feat(cas): add managed flag#3092
Piskoo wants to merge 1 commit intochainloop-dev:mainfrom
Piskoo:pfm-5834

Conversation

@Piskoo
Copy link
Copy Markdown
Collaborator

@Piskoo Piskoo commented May 5, 2026

Added a managed flag on the CAS backend to mark backends as Chainloop-managed, and blocked users from updating or deleting them.

Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 19 files

@Piskoo Piskoo marked this pull request as ready for review May 6, 2026 06:37
Default: in.Default,
Fallback: in.Fallback,
IsInline: in.Inline,
IsManaged: in.Managed,
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'd say that only users with the INSTANCE_ADMIN role can set this flag to true. what do you think?

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.

nodody should do it no? Isn't setting this value out of the scope of the PR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yes, nobody should have access to that, it should be set automatically by the backend during auto provisioning

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