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

Automatically start e2e testing #105

Merged
merged 2 commits into from
Jul 26, 2022
Merged

Conversation

sonyafenge
Copy link
Collaborator

Automatically start e2e testing:

export GRS_INSTANCE_PREFIX=[yourPreferName]-grs AUTORUN_E2E=true SIM_NUM=5 CLIENT_NUM=1 SERVER_NUM=1
export SERVER_ZONE=us-central1-a   SIM_ZONE=us-central1-a,us-west1-c,us-west2-a,us-west4-a,us-west3-c CLIENT_ZONE=us-central1-a
export SIM_REGIONS="Beijing,Shanghai,Wulan,Guizhou,Reserved1" SIM_RP_NUM=10 NODES_PER_RP=20000 SCHEDULER_REQUEST_MACHINE=25000 SCHEDULER_REQUEST_LIMIT=26000 SCHEDULER_NUM=10
export SERVICE_EXTRA_ARGS="--enable_metrics=false"
./hack/test-setup.sh

more information can be found from #95

@sonyafenge sonyafenge requested a review from q131172019 July 26, 2022 19:11
SCHEDULER_REQUEST_MACHINE=${SCHEDULER_REQUEST_MACHINE:-"25000"}
SCHEDULER_REQUEST_LIMIT=${SCHEDULER_REQUEST_LIMIT:-"26000"}
SCHEDULER_NUM=${SCHEDULER_NUM:-"16"}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any reason default to 16? and also, is 14 scheduler instances per machine too many ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we have default 2 sim and total 400k nodes, CHEDULER_REQUEST_MACHINE is 25000, so set to 16 scheduler to meet with total nodes.
for this setup, scheduler number per machine are dynamic assigned based on total scheduler_num and client machine numbers.
so far, out testing using 3 client machines and 40 schedulers are working well.

fi
fi

echo "Waiting 60 seconds to get simulator running"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that seems too long for the wait.... on my local dev, a few seconds is more than enough to have simulator up to go.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed to 10 seconds

Copy link
Collaborator

@yb01 yb01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@yb01 yb01 merged commit 595c080 into CentaurusInfra:main Jul 26, 2022
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

Successfully merging this pull request may close these issues.

2 participants