-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
Be less agressive and more patient when creating large master. #55794
Be less agressive and more patient when creating large master. #55794
Conversation
Can you explain how this will address the linked issue? If the zone is out of quota, wouldn't waiting long not change anything? |
@bowei see #55777 (comment) - waiting actually did make a difference, because it's not so much about quota, as about binpacking the large master. In the background we're also working towards a better solution, but I think this workaround might be good enough for the time being. |
cluster/gce/gci/master-helper.sh
Outdated
@@ -76,6 +76,12 @@ function replicate-master-instance() { | |||
function create-master-instance-internal() { | |||
local gcloud="gcloud" | |||
local retries=5 | |||
local sleep_sec=10 | |||
if [[ "${MASTER_SIZE}" =~ -64$ ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the biggest MASTER_SIZE?
maybe?
local suffix=$( echo "${MASTER_SIZE}" | sed 's/.*-\([0-9]\+\)/\1' )
if [[ "${suffix}" -ge 64 ]]; then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, this will be more robust.
Done.
3231fa0
to
5352db8
Compare
5352db8
to
bd76307
Compare
Rebased. |
/retest |
/test pull-kubernetes-e2e-gke-gci |
/kind bug |
/lgtm |
/approve no-issue |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: porridge, wojtek-t Associated issue: 55777 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
[MILESTONENOTIFIER] Milestone Pull Request Current @bowei @gmarek @porridge @wojtek-t Note: If this pull request is not resolved or labeled as Pull Request Labels
|
/test pull-kubernetes-node-e2e
|
Automatic merge from submit-queue (batch tested with PRs 54316, 53400, 55933, 55786, 55794). If you want to cherry-pick this change to another branch, please follow the instructions here. |
What this PR does / why we need it:
Workaround for #55777
Release note: