Skip to content

TPT-3807: Added DiskEncryption field for LKE Node Pool creation#917

Merged
ezilber-akamai merged 7 commits intolinode:mainfrom
ezilber-akamai:TPT-3807-lde-for-lke
Mar 27, 2026
Merged

TPT-3807: Added DiskEncryption field for LKE Node Pool creation#917
ezilber-akamai merged 7 commits intolinode:mainfrom
ezilber-akamai:TPT-3807-lde-for-lke

Conversation

@ezilber-akamai
Copy link
Contributor

📝 Description

Added support for DiskEncryption field in LKE Node Pool POST operation.

✔️ How to Test

make test-int TEST_ARGS="-run TestLKENodePool"

@ezilber-akamai ezilber-akamai requested a review from a team as a code owner March 19, 2026 16:57
Copilot AI review requested due to automatic review settings March 19, 2026 16:57
@ezilber-akamai ezilber-akamai added the new-feature for new features in the changelog. label Mar 19, 2026
@ezilber-akamai ezilber-akamai requested review from jriddle-linode and yec-akamai and removed request for a team March 19, 2026 16:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds DiskEncryption support to LKE node pool creation flows and updates integration coverage/fixtures to validate the new field.

Changes:

  • Added DiskEncryption to LKENodePoolCreateOptions and propagated it through GetCreateOptions.
  • Updated LKE node pool integration tests to create pools with disk encryption enabled and assert the returned value.
  • Refreshed integration fixtures to include disk_encryption in request/response payloads.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
lke_node_pools.go Extends node pool create options to include disk_encryption and propagates it from node pool objects.
test/integration/lke_node_pools_test.go Enables disk encryption in node pool create options and asserts it on GET.
test/integration/fixtures/TestLKENodePool_GetMissing.yaml Updates recorded headers for the missing-node-pool test.
test/integration/fixtures/TestLKENodePool_GetFound_k8s.yaml Updates recorded Kubernetes /nodes response used by readiness polling.
test/integration/fixtures/TestLKENodePool_CreateWithLabelsAndTaints.yaml Updates recorded LKE cluster/node pool create flows and responses.
test/integration/fixtures/TestLKENodePoolNode_Recycle.yaml Updates recorded node pool/node recycle flow fixtures to include disk encryption.
test/integration/fixtures/TestLKENodePoolNode_Get.yaml Updates recorded node pool/node get flow fixtures to include disk encryption.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ezilber-akamai ezilber-akamai requested a review from a team as a code owner March 26, 2026 19:20
@ezilber-akamai ezilber-akamai requested review from psnoch-akamai and removed request for a team March 26, 2026 19:20
@ezilber-akamai ezilber-akamai merged commit 69df217 into linode:main Mar 27, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-feature for new features in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants