add worker_subnet_uuid to kubernetes docs#1175
Conversation
|
A few questions/suggestions before approving:
Otherwise LGTM — examples coverage is complete and the write/read description split follows the existing pattern nicely. |
1c50544 to
7031de5
Compare
Yes, added a callout
Yes, TY! Required vpc:read, same as vpc_uuid
Didn't realize that was separate from the response examples tbh. Added the field there.
Yes, there is a validation in the API for this. I tried adding |
There was a problem hiding this comment.
- Immutability note added ✅
- vpc:read scope documented on both create and read sides ✅
- Curl create example now demonstrates the new field (and the broken line-continuation fix is a nice bonus) ✅
- Constraint wording softened ✅
Nice call on keeping worker_subnet_uuid in only some of the response examples — showing both the "custom subnet" and "default subnet" cases is clearer than having every example set it.
LGTM
New field was just added to DOKS API.