Skip to content

[WIP] test(flannel): remove ACL tests for unused legacy flannel wrapper - #38

Draft
Garrett Settles (gsettles01) wants to merge 2 commits into
aclmainfrom
gsettles/remove-acl-flannel-tests
Draft

[WIP] test(flannel): remove ACL tests for unused legacy flannel wrapper#38
Garrett Settles (gsettles01) wants to merge 2 commits into
aclmainfrom
gsettles/remove-acl-flannel-tests

Conversation

@gsettles01

@gsettles01 Garrett Settles (gsettles01) commented Sep 2, 2026

Copy link
Copy Markdown

Summary

Remove ACL-only legacy Flannel-wrapper registrations after team review found the wrapper likely unused. Flatcar and kubeadm Flannel coverage remains.

Change Log

  • Remove acl.flannel.{udp,vxlan}.
  • Remove unused ACL firewall/config fixtures; retain cl.flannel.*.

Type of Change

  • Kola test fix/update

Associated Issues

Test Methodology

  • Targeted go test, go vet, gofmt, Dockerfile.azl build, and AMD64/ARM64 kola list checks pass.
  • CI: 1195642

Merge Checklist

  • Targeted build/tests/vet
  • Full CI/relevant kola complete
  • Documentation, if needed
  • Ready to merge

@gsettles01 Garrett Settles (gsettles01) changed the title test(flannel): remove ACL-specific registrations [WIP] test(flannel): remove ACL-specific registrations Sep 2, 2026
@gsettles01 Garrett Settles (gsettles01) changed the title [WIP] test(flannel): remove ACL-specific registrations [WIP] test(flannel): remove ACL tests for unused legacy flannel wrapper Sep 2, 2026
Restore the ACL registrations and fixtures with explicit distro
exclusions, matching the other documented disabled tests.

Document wrapper retirement and deferred replacement coverage in
acl-disabled-tests.md (WI 23756). Keep Flatcar and kubeadm unchanged.
@gsettles01
Garrett Settles (gsettles01) force-pushed the gsettles/remove-acl-flannel-tests branch from 5dc0ca5 to b45293c Compare September 9, 2026 20:46
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.

1 participant