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

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

Closed
manuelh2410 opened this issue Aug 24, 2020 · 7 comments
Closed

Comments

@manuelh2410
Copy link

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
Copy link
Contributor

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

@manuelh2410
Copy link
Author

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

@alejandrojnm
Copy link
Member

This is already fixed

@manuelh2410
Copy link
Author

manuelh2410 commented Aug 25, 2020

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
Copy link
Contributor

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
Copy link
Author

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
Copy link
Contributor

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 :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants