pf tests: Add python3 as a dependency
ClosedPublic
Actions

Authored by jlduran on Tue, Sep 16, 8:33 PM.

Details

Reviewers

emaste
kp
vegeta_tuxpowered.net
ngie

Group Reviewers

tests

Commits

rGa84744caa297: pf tests: Add python3 as a dependency
rGdea1dd669054: pf tests: Add python3 as a dependency

Summary

These tests use pft_ping.py, which requires python3 and Scapy.

Fixes: 65c318630123 ("pf: Add prefer-ipv6-nexthop option for route-to pools")

Test Plan

https://ci.freebsd.org/view/Test/job/FreeBSD-main-aarch64-test/1766/testReport/junit/sys.netpfil.pf/route_to/

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jlduran created this revision.Tue, Sep 16, 8:33 PM

Herald added subscribers: vegeta_tuxpowered.net, glebius, melifaro and 2 others. · View Herald TranscriptTue, Sep 16, 8:33 PM

jlduran requested review of this revision.Tue, Sep 16, 8:33 PM

Harbormaster completed remote builds in B67104: Diff 162199.Tue, Sep 16, 8:33 PM

jlduran added reviewers: tests, emaste, kp, vegeta_tuxpowered.net.Tue, Sep 16, 8:35 PM

This should skip "failing" aarch64 tests due to the python3 package not being available.

jlduran edited the test plan for this revision. (Show Details)Tue, Sep 16, 8:39 PM

jlduran edited the summary of this revision. (Show Details)

ngie accepted this revision.Tue, Sep 16, 10:25 PM

This revision is now accepted and ready to land.Tue, Sep 16, 10:25 PM

Closed by commit rGdea1dd669054: pf tests: Add python3 as a dependency (authored by jlduran). · Explain WhyWed, Sep 17, 12:30 AM

This revision was automatically updated to reflect the committed changes.

jlduran added a commit: rGdea1dd669054: pf tests: Add python3 as a dependency.

jlduran added a commit: rGa84744caa297: pf tests: Add python3 as a dependency.Sat, Sep 27, 10:57 AM

Revision Contents
Changeset List

Path

Size

tests/

sys/

netpfil/

pf/

route_to.sh

15 lines

Diff 162214

View Options

tests/sys/netpfil/pf/route_to.sh

pf tests: Add python3 as a dependencyClosedPublicActions