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

[Release-1.25] E2E Domain Drone Cleanup #8584

Merged
merged 6 commits into from
Oct 31, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion .drone.yml
Original file line number Diff line number Diff line change
Expand Up @@ -579,6 +579,12 @@ platform:
os: linux
arch: amd64

clone:
retries: 3

depends_on:
- amd64

steps:
- name: build-e2e-image
image: rancher/dapper:v0.5.0
Expand All @@ -604,7 +610,8 @@ steps:
- mkdir -p dist/artifacts
- cp /tmp/artifacts/* dist/artifacts/
- docker stop registry && docker rm registry
# Cleanup any VMs running, happens if a previous test panics
# Cleanup VMs running, happens if a previous test panics
# Cleanup inactive domains, happens if previous test is canceled
- |
VMS=$(virsh list --name | grep '_server-\|_agent-' || true)
if [ -n "$VMS" ]; then
Expand All @@ -614,6 +621,13 @@ steps:
virsh undefine $vm --remove-all-storage
done
fi
VMS=$(virsh list --name --inactive | grep '_server-\|_agent-' || true)
if [ -n "$VMS" ]; then
for vm in $VMS
do
virsh undefine $vm
done
fi
- docker run -d -p 5000:5000 -e REGISTRY_PROXY_REMOTEURL=https://registry-1.docker.io --name registry registry:2
- cd tests/e2e/validatecluster
- vagrant destroy -f
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/cgroup.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,11 @@ jobs:
run: vagrant plugin install vagrant-k3s vagrant-reload
- name: "Vagrant Up"
run: vagrant up
- name: On Failure, Dump VM logs
if: ${{ failure() }}
run: |
logsDir=$(vboxmanage list vms --long | grep Logs | awk '{print $3" "$4}')
cat "$logsDir"/VBox.log
- name: "K3s Prepare"
run: vagrant provision --provision-with=k3s-prepare
- name: ⏬ "K3s Install"
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/snapshotter.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,11 @@ jobs:
run: vagrant plugin install vagrant-k3s
- name: "Vagrant Up ⏩ Install K3s"
run: vagrant up
- name: On Failure, Dump VM logs
if: ${{ failure() }}
run: |
logsDir=$(vboxmanage list vms --long | grep Logs | awk '{print $3" "$4}')
cat "$logsDir"/VBox.log
- name: "⏳ Node"
run: vagrant provision --provision-with=k3s-wait-for-node
- name: "⏳ CoreDNS"
Expand Down
2 changes: 1 addition & 1 deletion docs/adrs/add-dual-stack-support-to-netpol-agent.md
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ The motivation behind keeping a fork is that:

* upstream might ask for implementing dual-stack in all kube-router
components (which would be understandable)
* acceepting a solution upstream might take long time
* accepting a solution upstream might take long time

Our fork of kube-router is going to be used as a vendored library inside k3s
code. And the currently copied code in k3s in `pkg/agent/netpol` is going to
Expand Down
Loading