Repository navigation
Conversation
Explain the GCP setup in GCP terms so a reader can follow the GCP tab without reading the AWS one. AWS comparisons stay as optional asides. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
Claude finished @tooky's task in 1m 17s —— View job Reviewed new commits (
|
| - `kosli_capture_provider`: the full resource name of the workload identity provider | ||
| - `kosli_capture_service_account`: the email of the service account Cloud Capture impersonates | ||
|
|
||
| Share them with Kosli after the deployment completes. You can read them at any time with `gcloud infra-manager deployments describe`. |
There was a problem hiding this comment.
Improvement: this is now a step readers follow, and as far as I know gcloud infra-manager deployments describe doesn't return Terraform outputs. The Deployment resource only points at latestRevision. The outputs are on the revision, under applyResults.outputs. You already list this as unverified in the PR body. If it holds, a reader who runs this command won't find the values this step asks them to share.
If verified, consider spelling out the command, e.g.:
| Share them with Kosli after the deployment completes. You can read them at any time with `gcloud infra-manager deployments describe`. | |
| Share them with Kosli after the deployment completes. You can read them at any time from the deployment's latest revision: | |
| ```bash | |
| gcloud infra-manager revisions describe \ | |
| "$(gcloud infra-manager deployments describe \ | |
| projects/<your-gcp-project>/locations/<your-gcp-region>/deployments/kosli-capture \ | |
| --format='value(latestRevision)')" \ | |
| --format='yaml(applyResults.outputs)' | |
| ``` |
The same wording also appears in administration/cloud_capture/security.md:143 and in the Terraform comment at security.md:260.
Cloud Capture reads GKE pods from Cloud Asset Inventory, as kosli snapshot gke does, not from each cluster's Kubernetes API. Remove the private control plane limitation and replace the Kubernetes API permissions with the Cloud Asset ones. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
| [`kosli snapshot gke`](/client_reference/kosli_snapshot_gke). It never calls a cluster's Kubernetes | ||
| API, so no Kubernetes RBAC objects need to be created inside your clusters, and Cloud Capture needs | ||
| no network access to any cluster control plane. Asset Inventory is eventually consistent, so a | ||
| snapshot can lag behind recent pod changes. |
There was a problem hiding this comment.
I've noticed this comment elsewhere but testing was pretty instant. But I could find what guarantees GCP offers around events coming in there.
|
To answer a couple of the open questions- the Terraform code will not be prepopulated with the correct values; the values are visible in the Kosli UI only (in the new settings page). We have a new API endpoint for registering Cloud accounts, so they "let us know" by making f a POST request there |
Summary
Until now, parts of the Cloud Capture GCP tabs only made sense if you'd already read the AWS tab. For example, the workload identity checks were defined as the "counterpart" of the AWS trust policy and external ID, and the setup steps said "GCP has no equivalent of phone-home". This PR rewrites the GCP content to explain each mechanism in GCP terms first. Where an AWS comparison still helps, it follows as a short optional aside. The AWS tab content is unchanged.
Security page
kosli_role_nameandattribute_conditioncomments in the Terraform are reworded the same way.kosli snapshot gkedoes. So:container.namespaces.*andcontainer.pods.*forcloudasset.assets.listContainerPodandserviceusage.services.usecloudasset.googleapis.comroles/cloudasset.viewer(which can also listk8s.io/Secret)Getting started page, GCP tab:
The overview's first step now says "Grant Kosli read-only access to your cloud accounts" rather than "Create an IAM role".
Still to do
The three items below aren't confirmed, so the pages carry
{/* TODO */}comments for them. These are hidden on the rendered site.container.clusters.listandcontainer.clusters.get? If discovery also goes through Cloud Asset Inventory, those two permissions and the discovery comment in the role need changing.Also worth checking:
gcloud infra-manager deployments describeactually show Terraform outputs? They may only appear on the revision (gcloud infra-manager revisions describe, underapplyResults.outputs). The wording is unchanged from before, but it's now presented as a step users follow.capture-managedorcloud-capture. Wherever the Settings-page download is generated, it needs the same changes: the reworded comments, the Cloud Asset permissions, and enabling the Cloud Asset API.Evidence
pytest tests/: 59 passed.mint dev: both pages render with the new tabs, steps and code blocks, and the TODO comments don't appear.mint broken-links: one broken link, intutorials/working_with_controls.mdx, which this PR doesn't touch.Merge Danger
Door: two-way. This is a docs-only change to two BETA pages and can be reverted cleanly.
Blast Radius: two pages under Administration ▸ Cloud Capture. No navigation, redirect or anchor changes. Customers who already followed the old GKE permissions would have a role without the Cloud Asset permissions, but GCP support is still marked "coming soon".
🤖 Generated with Claude Code