Skip to content

Commit d7a2335

Browse files
authored
Merge pull request #11131 from NareshKoduru/6519
📖 infrastructure provider security guidelines
2 parents 199e00b + a512d28 commit d7a2335

File tree

4 files changed

+34
-0
lines changed

4 files changed

+34
-0
lines changed

docs/book/src/SUMMARY.md

+1
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@
4747
- [Diagnostics](./tasks/diagnostics.md)
4848
- [Security Guidelines](./security/index.md)
4949
- [Pod Security Standards](./security/pod-security-standards.md)
50+
- [Infrastructure Provider Security Guidance](./security/infrastructure-provider-security-guidance.md)
5051
- [clusterctl CLI](./clusterctl/overview.md)
5152
- [clusterctl Commands](clusterctl/commands/commands.md)
5253
- [init](clusterctl/commands/init.md)

docs/book/src/developer/providers/cluster-infrastructure.md

+4
Original file line numberDiff line numberDiff line change
@@ -187,3 +187,7 @@ Note, the write permissions allow the `Cluster` controller to set owner referenc
187187
"infrastructure cluster" resources; they are not used for general mutations of these resources.
188188

189189
[aggregation label]: https://kubernetes.io/docs/reference/access-authn-authz/rbac/#aggregated-clusterroles
190+
191+
## Security Guidelines
192+
193+
Please refer to [Infrastructure Provider Security Guidance](../../security/infrastructure-provider-security-guidance.md).

docs/book/src/developer/providers/machine-infrastructure.md

+4
Original file line numberDiff line numberDiff line change
@@ -209,3 +209,7 @@ Note, the write permissions allow the `Machine` controller to set owner referenc
209209
"infrastructure machine" resources; they are not used for general mutations of these resources.
210210

211211
[aggregation label]: https://kubernetes.io/docs/reference/access-authn-authz/rbac/#aggregated-clusterroles
212+
213+
## Security Guidelines
214+
215+
Please refer to [Infrastructure Provider Security Guidance](../../security/infrastructure-provider-security-guidance.md).
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# Infrastructure Provider Security Guidance
2+
3+
There are several critical areas that any infrastructure provider implementer must address to ensure secure operations. These include:
4+
5+
- **Management of cloud credentials** assigned to the infrastructure provider, including setting quotas and rate limiting.
6+
- **Ensuring secure access to VMs** for troubleshooting, with proper authentication methods.
7+
- **Controlling manual operations** performed on cloud infrastructure targeted by the provider.
8+
- **Housekeeping** of the cloud infrastructure, ensuring timely cleanup and garbage collection of unused resources.
9+
10+
The following list outlines high-level security recommendations. It is a community-maintained resource, and everyone’s contributions are essential to continuously improve and adapt these best practices. Each provider implementer is responsible for translating these recommendations to fit the context of their specific cloud provider:
11+
12+
1. **Credentials Management**:
13+
Ensure credentials used by Cluster API are least privileged. Apply access control to Cluster API controller namespaces, restricting unauthorized access to cloud administrators only.
14+
15+
2. **Two-Factor Authentication (2FA)**:
16+
Implement 2FA for all maintainer accounts on GitHub. For any privileged actions (e.g., image building or updates to machine images), follow the "second pair of eyes" principle to ensure review and oversight.
17+
18+
3. **Short-lived Credentials**:
19+
Use short-lived credentials that are automatically renewed via node-level attestation mechanisms, minimizing the risk of credential misuse.
20+
21+
4. **Rate Limiting for Cloud Resources**:
22+
Implement rate limits for the creation, deletion, and updating of cloud resources, protecting against potential abuse or accidental overload.
23+
24+
5. **Resource Housekeeping**:
25+
Any cloud resource not linked to a cluster after a fixed configurable period, created by cloud credentials, should be automatically deleted or marked for garbage collection to avoid resource sprawl.

0 commit comments

Comments
 (0)