-
Notifications
You must be signed in to change notification settings - Fork 239
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
ci: validate CNS in-memory state after restart #2049
Conversation
kubectl get pod -owide -A | ||
make test-validate-state | ||
echo "restart CNS" | ||
kubectl rollout restart ds azure-cns -n kube-system |
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.
Let's add a wait as well. kubectl rollout status ds azure-cns -n kube-system
to make sure azure-cns has restarted.
@@ -165,6 +165,18 @@ steps: | |||
name: "validatePods" | |||
displayName: "Validate Pods" | |||
|
|||
- script: | |
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.
Should we add cilium agent restarts as well ?
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.
I think that check can be reserved for cilium nightly pipeline runs
@@ -165,6 +165,19 @@ steps: | |||
name: "validatePods" | |||
displayName: "Validate Pods" | |||
|
|||
- script: | | |||
echo "validate pod IP assignment and check systemd-networkd restart" |
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.
why log message says "systemd-networkd" restart
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.
The cmd make test-validate-state
validates the state after restarting systemd-networkd restart for linux as a part of validating state.
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.
lgtm
* validate pod states after cns restart * update job * wait for cns to complete restart * update pipeline log * update pipeline log
* validate pod states after cns restart * update job * wait for cns to complete restart * update pipeline log * update pipeline log
Reason for Change:
Adds check to validate CNS pod IP assignment before and after the CNS daemonset is restarted.
Issue Fixed:
Requirements:
Notes: