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

Modify the kind builder script to perform correct kind cleanup #5738

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

KMAnju-2021
Copy link
Contributor

@KMAnju-2021 KMAnju-2021 commented Nov 21, 2023

Signed-off-by: KMAnju-2021 km074btcse18@igdtuw.ac.in

Modify the kind builder script and destroy function in kind-setup.sh to perform correct kind cleanup.
If we create a kind cluster by using kind-setup.sh script then we should need to delete cluster using same script.

@KMAnju-2021 KMAnju-2021 changed the title Updating the kind builder script to perform correct kind cleanup modify the kind builder script to perform correct kind cleanup Nov 21, 2023
@KMAnju-2021 KMAnju-2021 changed the title modify the kind builder script to perform correct kind cleanup Modify the kind builder script to perform correct kind cleanup Nov 21, 2023
@rajnkamr rajnkamr added this to the Antrea v1.15 release milestone Nov 21, 2023
@KMAnju-2021
Copy link
Contributor Author

/test-kind-conformance
/test-kind-networkpolicy

@@ -409,6 +409,7 @@ function clean_kind {
if [[ $diff -gt $timeout ]]; then
echo "=== kind ${kind_cluster_name} present from more than $UNTIL_TIME_IN_MINS minutes ==="
kind delete cluster --name $kind_cluster_name
delete_networks
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO it would make more sense to have this happen in destroy

We could change the destroy implementation to:

function destroy {
  if [[ $UNTIL_TIME_IN_MINS != "" ]]; then
      clean_kind
  else
      kind delete cluster --name $CLUSTER_NAME
  fi
  delete_networks
}
```

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

Signed-off-by: KMAnju-2021 <km074btcse18@igdtuw.ac.in>
@antoninbas
Copy link
Contributor

/test-kind-conformance
/test-kind-networkpolicy

@antoninbas
Copy link
Contributor

/test-kind-networkpolicy

@antoninbas
Copy link
Contributor

Kind / Validate metrics in Prometheus document match running deployment's is failing (I ran it twice), maybe it's somehow related to your PR.
Also, test-kind-networkpolicy keeps timing out, but I don't know if this is related.

@KMAnju-2021
Copy link
Contributor Author

/test-kind-networkpolicy

@KMAnju-2021
Copy link
Contributor Author

/test-kind-conformance
/test-kind-networkpolicy

1 similar comment
@KMAnju-2021
Copy link
Contributor Author

/test-kind-conformance
/test-kind-networkpolicy

@KMAnju-2021
Copy link
Contributor Author

Kind / Validate metrics in Prometheus document match running deployment's is failing (I ran it twice), maybe it's somehow related to your PR.
Also, test-kind-networkpolicy keeps timing out, but I don't know if this is related.

Today i triggered it thrice but I didn't get any error and job finished with success. I think that was a temporary error?

@antoninbas
Copy link
Contributor

Looks like the metrics job failing was a flake.

@KMAnju-2021 as for test-kind-conformance and test-kind-networkpolicy, we should try to investigate flakes and make sure these jobs are as stable as possible

@antoninbas antoninbas merged commit b1fa272 into antrea-io:main Nov 28, 2023
45 of 56 checks passed
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.

3 participants