Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PLT-1330] CMEK, SA & CIDRs #642

Merged
merged 2 commits into from
Jan 10, 2025
Merged

Conversation

iamjanr
Copy link

@iamjanr iamjanr commented Dec 12, 2024

Description

Soporte CMEK:
Ejemplo:
worker_nodes:
root_volume:
encrypted: true
size: 50
type: pd-standard
encryption_key: "projects/PROJECT_ID/locations/LOCATION/keyRings/KEY_RING/cryptoKeys/KEY_NAME"

Soporte SA:
Ejemplo:
spec:
security:
nodes_identity: "<sa_name>@<PROJECT_ID>.iam.gserviceaccount.com"
Soporte CIDR's:
Ejemplo:
control_plane:
managed: true
gcp:
ip_allocation_policy:
cluster_secondary_range_name: "gkepods-europe-west4"
services_secondary_range_name: "gkeservices-europe-west4"
Ejemplo:
control_plane:
managed: true
gcp:
ip_allocation_policy:
cluster_ipv4_cidr_block: 10.180.0.0/14
services_ipv4_cidr_block: 10.8.32.0/20

Related Pull Requests

cluster-operator: https://github.com/Stratio/cluster-operator/pull/247
cluster-api-provider-gcp: Stratio/cluster-api-provider-gcp#36

Pull Request Checklist:

  • [PR title] Include a title referencing a ticket in Jira (e.g. "[CLOUDS-99] Implement a new funcionality").
  • [PR desc] Add a summary of the changes made in simple terms.
  • [PR desc] List any pull-request related to this change (docs, tests, feature, etc.).
  • [PR labels] Add the corresponding labels (release, skips, cherry-pick, AT-eks-smoke, etc).
  • [Docs] Are changes to the documentation required? (if so, please add references to those PRs).
  • [QA] Are new unit tests required according with the changes? (if so, please add references to those PRs).

JIRA: https://stratio.atlassian.net/browse/PLT-1330

@iamjanr iamjanr requested a review from lreciomelero as a code owner January 9, 2025 09:57
@iamjanr iamjanr added ok-to-merge and removed 0.6.1 labels Jan 9, 2025
Copy link

@rherrero-stratio rherrero-stratio left a comment

Choose a reason for hiding this comment

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

lgtm

@iamjanr
Copy link
Author

iamjanr commented Jan 9, 2025

🙏

@iamjanr iamjanr merged commit a63dc2a into Stratio:branch-0.17.0-0.6 Jan 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants