Commit eb3671e
authored
Feature: Serverless Endpoint (#193)
Issue #, if available:
Description of changes:
This pull request contains code/unit tests for the Serverless Endpoint.
Changes made:
`generator.yaml` - Uncommenting shape and adding hook.
`pkg/resource/endpoint_config/custom_delta.go` - Code that handles server side default. Late initialization isnt possible because this field is an array.
`testdata/` - Unit tests
The Server returns a value for VolumeSizeInGB despite it being a parameter the user cannot even specify while using a serverless endpoint. The same problem is not there for an instance based endpoint config.
Manual testing (Will delete this before merge):
Endpoint Config:
[Serverless] Async Conifg - Validation Exception
[Serverless] Multi Variant - Validation Exception
[Serverless] Data Capture - Validation Exception
[Serverless] Including VolumeSizeInGB in spec: Validation exception
[Serverless] No VolumeSizeInGB: Normal reconciliation, no errors after restart.
[Normal] No VolumeSizeInGB - Normal reconciliation, no errors after restart.
[Normal] VolumeSizeInGB - Normal reconciliation, no errors after restart.
Endpoint:
Launching Serverless: Works, user sees information in PendingDeploymentSummary
Updating Instance→ Serverless: Validation Exception
Updating Serverless → Instance: Production variant must have a different name, works.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.1 parent 05ab27e commit eb3671e
File tree
17 files changed
+426
-69
lines changed- apis/v1alpha1
- config/crd/bases
- helm/crds
- pkg/resource
- endpoint_config
- testdata
- sdkapi/describe
- v1alpha1/readone
- desired
- observed
- endpoint
17 files changed
+426
-69
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | | - | |
| 2 | + | |
| 3 | + | |
4 | 4 | | |
5 | | - | |
6 | | - | |
| 5 | + | |
| 6 | + | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | | - | |
| 10 | + | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
60 | 63 | | |
61 | 64 | | |
62 | 65 | | |
| |||
1017 | 1020 | | |
1018 | 1021 | | |
1019 | 1022 | | |
1020 | | - | |
1021 | 1023 | | |
1022 | 1024 | | |
1023 | 1025 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
170 | 181 | | |
171 | 182 | | |
172 | 183 | | |
| |||
Lines changed: 44 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
268 | 268 | | |
269 | 269 | | |
270 | 270 | | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
| 276 | + | |
| 277 | + | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
271 | 282 | | |
272 | 283 | | |
273 | 284 | | |
| |||
295 | 306 | | |
296 | 307 | | |
297 | 308 | | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
298 | 320 | | |
299 | 321 | | |
300 | 322 | | |
| |||
332 | 354 | | |
333 | 355 | | |
334 | 356 | | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
335 | 368 | | |
336 | 369 | | |
337 | 370 | | |
| |||
359 | 392 | | |
360 | 393 | | |
361 | 394 | | |
| 395 | + | |
| 396 | + | |
| 397 | + | |
| 398 | + | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
| 402 | + | |
| 403 | + | |
| 404 | + | |
| 405 | + | |
362 | 406 | | |
363 | 407 | | |
364 | 408 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
57 | 57 | | |
58 | 58 | | |
59 | 59 | | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
60 | 63 | | |
61 | 64 | | |
62 | 65 | | |
| |||
1017 | 1020 | | |
1018 | 1021 | | |
1019 | 1022 | | |
1020 | | - | |
1021 | 1023 | | |
1022 | 1024 | | |
1023 | 1025 | | |
| |||
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
167 | 167 | | |
168 | 168 | | |
169 | 169 | | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
170 | 181 | | |
171 | 182 | | |
172 | 183 | | |
| |||
0 commit comments