Skip to content

fix: Helm-installed atelet cannot prewarm sandbox assets - #33

Open
supreme-gg-gg wants to merge 1 commit into
mainfrom
jetc/fix/atelet-sa-sandboxconfig
Open

fix: Helm-installed atelet cannot prewarm sandbox assets#33
supreme-gg-gg wants to merge 1 commit into
mainfrom
jetc/fix/atelet-sa-sandboxconfig

Conversation

@supreme-gg-gg

Copy link
Copy Markdown

Upstream added a feature for sandbox asset prewarming in atelet. Our chart, however, was not updated to include the permission for atelet service account to read cluster-scoped sandboxconfigs.ate.dev resources.

Observed atelet log:

SandboxConfig list/watch failed; sandbox asset prewarm degraded until it recovers
failed to list *v1alpha1.SandboxConfig: sandboxconfigs.ate.dev is forbidden:
User "system:serviceaccount:ate-system:atelet" cannot list resource
"sandboxconfigs" in API group "ate.dev" at the cluster scope

The same permission was added to install manifests upstream:

# SandboxConfigs are watched to pre-download sandbox assets into the node's
# cache before the first actor needs them (see cmd/atelet/sandbox_prewarm.go).
- apiGroups: ["ate.dev"]
resources: ["sandboxconfigs"]
verbs: ["get", "list", "watch"]

Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
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.

1 participant