partition test: increase healing time
## Motivation
remove flakiness in the systest
MODIFY
.github/workflows/ci.yml
.github/workflows/ci.yml
@@ -16,6 +16,10 @@ env:
TD_QUEUE_ZONE: ${{ secrets.TD_QUEUE_ZONE }}
DUMP_QUEUE_NAME: ${{ secrets.DUMP_QUEUE_NAME }}
DUMP_QUEUE_ZONE: ${{ secrets.DUMP_QUEUE_ZONE }}
+ CI_CLUSTER_NAME: ${{ secrets.CI_CLUSTER_NAME }}
+ CI_GCP_CREDENTIALS: ${{ secrets.CI_GCP_CREDENTIALS }}
+ CI_GCP_PROJECT_ID: ${{ secrets.CI_GCP_PROJECT_ID }}
+ CI_REGION_NAME: ${{ secrets.CI_REGION_NAME }}
# Trigger the workflow on all pull requests, and on push to specific branches
on:
@@ -279,13 +283,13 @@ jobs:
uses: "google-github-actions/auth@v1"
with:
# GCP_CREDENTIALS is minified JSON of service account
- credentials_json: "${{ secrets.GCP_CREDENTIALS }}"
+ credentials_json: "${{ secrets.CI_GCP_CREDENTIALS }}"
- name: Configure gcloud
uses: "google-github-actions/setup-gcloud@v1"
- name: Configure kubectl
- run: gcloud container clusters get-credentials ${{ secrets.CI_CLUSTER_NAME }} --zone ${{ secrets.CI_ZONE_NAME }} --project ${{ secrets.GCP_PROJECT_ID }}
+ run: gcloud container clusters get-credentials ${{ secrets.CI_CLUSTER_NAME }} --region ${{ secrets.CI_REGION_NAME }} --project ${{ secrets.CI_GCP_PROJECT_ID }}
- name: Login to Docker Hub
uses: docker/login-action@v2
@@ -313,7 +317,7 @@ jobs:
go-version: ${{ env.go-version }}
- name: Run tests I
- timeout-minutes: 70
+ timeout-minutes: 55
env:
test_id: systest-${{ steps.vars.outputs.sha_short }}
label: sanity
@@ -326,7 +330,7 @@ jobs:
run: make -C systest run test_name=.
- name: Run tests II
- timeout-minutes: 45
+ timeout-minutes: 30
env:
test_id: systest-${{ steps.vars.outputs.sha_short }}
label: destructive

MODIFY
systest/tests/partition_test.go
systest/tests/partition_test.go
@@ -169,7 +169,8 @@ func TestPartition_30_70(t *testing.T) {
tctx := testcontext.New(t, testcontext.Labels("destructive"))
cl, err := cluster.Reuse(tctx, cluster.WithKeys(10))
require.NoError(t, err)
- testPartition(t, tctx, cl, 30, 2)
+ // TODO: re-assess the number of epoch required for healing.
+ testPartition(t, tctx, cl, 30, 4)
}
func TestPartition_50_50(t *testing.T) {
@@ -178,5 +179,6 @@ func TestPartition_50_50(t *testing.T) {
tctx := testcontext.New(t, testcontext.Labels("destructive"))
cl, err := cluster.Reuse(tctx, cluster.WithKeys(10))
require.NoError(t, err)
- testPartition(t, tctx, cl, 50, 3)
+ // TODO: re-assess the number of epoch required for healing.
+ testPartition(t, tctx, cl, 50, 5)
}

partition test: increase healing time (#3882)

## Motivation
remove flakiness in the systest


Co-authored-by: andres-spacemesh <106999199+andres-spacemesh@users.noreply.github.com>