This was triggered by a call to admin_up() that was passing self.lfclient_url as an un-named parameter, creating
unanswerable requests formatted like http://localhost:8080/http:/localhost:8080/. There should really only be
one http[s]*:// pattern in a URL, and further instances of colons should be URL encoded to %3A.
Signed-off-by: Jed Reynolds <jed@bitratchet.com>
And remove the logic to create a bridge within the vap profile,
the vap profile does not have enough info to properly create
a bridge (old code used hard-coded port names)
Instead, logic would be to create vap, and then create a bridge
object that holds the VAP using the bridge profile/script.
Signed-off-by: Ben Greear <greearb@candelatech.com>
Logic was broken trying to compare json_response['down'] to "true",
when instead that object is a Boolean variable. So this method
always returned immediately instead of actually waiting until port
was admin up. This was seen testing VAP profile.
Signed-off-by: Ben Greear <greearb@candelatech.com>
Make it look more like macvlans, including logic to set IP addresses,
and to verify qvlans were created properly.
Signed-off-by: Ben Greear <greearb@candelatech.com>
Use base class methods for logging setup, check error codes
for port creation and cx creation.
l4 profile gets TODO note, since it does not verify that cx/endp
are actually created or not.
Try to figure out why this doesn't work on testbed heather,
at the least, I think DUT is probably wrong in the resgresison
script. More work is needed to get this working.
Signed-off-by: Ben Greear <greearb@candelatech.com>
If port cannot be found, then it is down. This fixes race with
creating port and then immediately trying to way for it to go
admin up.
Signed-off-by: Ben Greear <greearb@candelatech.com>
The generic endpoint profile, especially the monitoring code,
was wrong in lots of ways.
Fix this, improve pass/fail and error reporting, and other
improvements for the gen_profile test.
Probably not backwards compat, but I assume due to breakage
no one was using this code anyway.
Signed-off-by: Ben Greear <greearb@candelatech.com>
Clean up monitor logic in gen_cx profile to remove useless sleeps
and rename a variable that indicated it was msec but which was actually
seconds.
test_generic gets better error checking and also fixes bugs in how it
monitors the connection. More work yet is needed for making the
monitor do a good pass/fail logic.
Signed-off-by: Ben Greear <greearb@candelatech.com>
Let user specify the level of logging desired.
Test implementaiton in example_security_connection.py
Signed-off-by: Ben Greear <greearb@candelatech.com>
Check for false value in create_l3_stations and return error code
upon failure.
Add configurable timeout to waiting for cx/endp logic.
Signed-off-by: Ben Greear <greearb@candelatech.com>
In station_profile create method, pay attention to result of
wait_until_ports_appear. If it returns False, we know creation
has failed somehow, so we do not need to do the extra checking
that was in the create() method.
create_l3_stations adds logic to check return code of station
create logic, and exit with error code if it could not
create them as expected.
Signed-off-by: Ben Greear <greearb@candelatech.com>
