Follow-up from "Extend certificate-based-cluster feature for certain users via a new table"
The following discussions from !87149 (merged) should be addressed:
-
@ahegyi started a discussion: We could disable the
id
column and use thenamespace_id
as the PK. Since this table is temporary, I'm ok with the current structure. -
@ahegyi started a discussion: Minor optimization, defer the
cluster_enabled_grant
query:(Gitlab.com? || Gitlab.dev_or_test_env?) && root_ancestor.cluster_enabled_grant.present?
Edited by João Alexandre Cunha