search

civo / cli

Our Command Line Interface (CLI) for interacting with your Civo resources
Apache License 2.0
178 stars 87 forks source link

https://github.com/civo/cli/blob/master/cmd/kubernetes_create.go #20

Closed manuelh2410 closed 4 years ago

manuelh2410 commented 4 years ago

Hi, I created 51 clusters in sequence, without being notified of having exceeded any limits. Is this by design? Should I not get notified by the API? From my end, I assumed I would be met with a message indicating that I exceeded my limits.

Kindly see the output of my attempt below.

root@Pegasus:/home/manuelh# ./test.sh The cluster manueltest0 (9ee5f513-3d09-4bf3-808d-18ccc775bf1d) has been created The cluster manueltest1 (a5080341-b6b3-48f3-920a-56fbae000ae7) has been created The cluster manueltest2 (5278b8b3-4e6a-4b88-ba82-0cfc0e803a9a) has been created The cluster manueltest3 (12a85bd3-4002-43f4-a16f-9320e4893ba7) has been created The cluster manueltest4 (4d427f6f-604d-477b-a2e9-3e2f80e2bb0d) has been created The cluster manueltest5 (44ac9be3-f432-468d-a2e5-a12aef43dac7) has been created The cluster manueltest6 (1ede3310-be8e-49e5-80e6-325594e1c753) has been created The cluster manueltest7 (13a3adfb-3b2f-41ab-a5f3-e26da782bc4e) has been created The cluster manueltest8 (e490e768-9342-4494-8860-2ad9947f9feb) has been created The cluster manueltest9 (2115b299-3c98-412a-8f85-8ac9fda057be) has been created The cluster manueltest10 (ed28e54e-8b0c-43cc-9d7d-fa188289faec) has been created The cluster manueltest11 (54428a31-d640-4f54-9c31-a3d86fb1e066) has been created The cluster manueltest12 (ded857a2-b1a2-43fa-a76e-3c12760c5743) has been created The cluster manueltest13 (3d21550b-6917-4d3e-93a4-aefe1f56f6f6) has been created The cluster manueltest14 (e550cd90-aad3-449e-9f45-a1afc74e8757) has been created The cluster manueltest15 (ce11141c-654c-4416-a716-790e7ff8ffb2) has been created The cluster manueltest16 (76921e80-063e-49c1-8762-7a26dd04837e) has been created The cluster manueltest17 (5fbd2ac7-8172-4dbf-9891-9a4ab12d729d) has been created The cluster manueltest18 (59cabb0b-d102-4a34-95f3-75dad2a9ebc9) has been created The cluster manueltest19 (14492526-bdcd-47f5-b5bb-ba94f00201a1) has been created The cluster manueltest20 (59ac2c4f-50cc-45c3-8f01-2e79c211859b) has been created The cluster manueltest21 (cbe45ee4-6692-489d-9701-0b3cedbeb626) has been created The cluster manueltest22 (9f048bc0-813d-4f60-8b86-e734bfbb1c14) has been created The cluster manueltest23 (f81261f1-d17d-4aff-bfb7-d77fa9ec3dd8) has been created The cluster manueltest24 (ed9b818c-8721-49de-aeeb-13245849e496) has been created The cluster manueltest25 (5e3aa75c-6e62-4bd2-8d82-3f0cc0ee8f7e) has been created The cluster manueltest26 (c976cbb6-afc5-48a3-86c8-455ed88473c1) has been created The cluster manueltest27 (aa1ddfb1-1178-4fb1-9e20-5d55112cecc8) has been created The cluster manueltest28 (373b829c-35de-44aa-a6c1-59e02a7e0ac0) has been created The cluster manueltest29 (52f4404a-4da9-48fa-93b0-f2f16286f0ed) has been created The cluster manueltest30 (ae71e4b9-8542-4ba8-ae7e-75e04192006c) has been created The cluster manueltest31 (cbb06046-2db2-4529-8b31-1cd93fe20ec7) has been created The cluster manueltest32 (4a15c5a8-a6e6-4e9c-8385-2634c82c71f0) has been created The cluster manueltest33 (6b85c691-db6d-41a6-822b-6f871b2fd4bc) has been created The cluster manueltest34 (99b78f8b-627b-4592-9d09-3e4b40bb4bb6) has been created The cluster manueltest35 (07448609-5963-4298-8809-4b0e4a097303) has been created The cluster manueltest36 (16d0a5b7-c83d-47a9-8ae8-01631d39a7b7) has been created The cluster manueltest37 (654b9ede-a31b-4d05-addd-43e89147e549) has been created The cluster manueltest38 (423fb82d-a4f3-419e-a823-08bf67e85b16) has been created The cluster manueltest39 (dd6ce640-ef09-4f93-919b-c922af7aa63d) has been created The cluster manueltest40 (96297df2-13ee-4760-915f-8368e19f8420) has been created The cluster manueltest41 (092ffe51-8829-4fe4-910b-380f6e7e5d46) has been created The cluster manueltest42 (51a21788-d437-433b-bb4b-654b09a61ff7) has been created The cluster manueltest43 (8d34f6e1-aa7f-40d0-9e23-eb9e7ebd340b) has been created The cluster manueltest44 (d78b980e-71ab-411a-b15c-088f9e88cfb9) has been created The cluster manueltest45 (792b2aa6-351a-4936-b06a-f54fd7813c1c) has been created The cluster manueltest46 (89c2821a-0882-4ef3-aeb3-9dd14ff62947) has been created The cluster manueltest47 (11fdfe67-98e8-437a-b0eb-7347e20edef7) has been created The cluster manueltest48 (cd519175-60eb-4874-b01d-0247eac8e0a6) has been created The cluster manueltest49 (66e9114a-b9b5-4e7e-b634-e94bbe69cf2c) has been created The cluster manueltest50 (b66389bf-fb86-479f-880f-c4a2d54b5f00) has been created

-- Thanks again for your time.

Regards, Manuel

andyjeffries commented 4 years ago

You should have been notified of a quota full, did they all create in your account?

manuelh2410 commented 4 years ago

Hi , Thanks for getting back to me . Yes all clusters were created in one single account

alejandrojnm commented 4 years ago

This is already fixed

manuelh2410 commented 4 years ago

Thanks for fixing this . Was this a bug ? I tested again and was able to create 6 clusters this time and was then met with an error indicating that I exceeded my quota . (If im not mistaken this is normal)

andyjeffries commented 4 years ago

Yes, we did a quota check after launching resources, thinking most people would hit it during the normal operation of the platform. You highlighted that with a malicious or erroneous script a user could literally exhaust our resources. So we've fixed it in our API Server.

manuelh2410 commented 4 years ago

Hi Andy,

I can assure you, that there was no malintent from my end. Nor were there any errors in my script. The intent was just to check how many clusters I would be able to create before the API would throttle me. Happy to see its been fixed.

Regards ,

Manuel

andyjeffries commented 4 years ago

I completely understand that! I wasn't saying "you" but "a user". You highlighted a bug, in a nice way (raising an issue for us, not doing it with 10,000 instead of 50) and we've fixed it. I was saying that someone could do that, so why we fixed it. We appreciate the way you tested and reported it :-)