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

Periodic Reauthentication #117

Merged
merged 14 commits into from
Apr 8, 2019
Merged

Periodic Reauthentication #117

merged 14 commits into from
Apr 8, 2019

Conversation

Bairdo
Copy link
Contributor

@Bairdo Bairdo commented Mar 17, 2019

builds on top of #116.

@Bairdo
Copy link
Contributor Author

Bairdo commented Mar 18, 2019

test_chewie.py:test_port_status_changes() blocks forever.

@Bairdo
Copy link
Contributor Author

Bairdo commented Mar 18, 2019

Skipping broken test, test_chewie.py:test_port_status_changes()

Not particularly sure how to fix it atm. Will have another look tomorrow morning if at work (coming down with man flu).

The case should be tested over on the faucet integration tests, so could potentially be merged, (i don't mind either way).

@gizmoguy
Copy link
Member

Hi @Bairdo, just letting you know we are deferring the faucet 1.9 release until tomorrow now.

On the off chance you are feeling better you have an extra day to get all this wrapped up. Otherwise we can include it in next weeks release.

@Bairdo Bairdo changed the title WIP: Periodic Reauthentication Periodic Reauthentication Apr 2, 2019
@Bairdo
Copy link
Contributor Author

Bairdo commented Apr 2, 2019

So this PR was branched off of PR #116, but contains a bug or two which have been fixed in this PR. So either they can both be merged in or #116 can be deleted as it is contained in this one, without the errors.

@gizmoguy gizmoguy merged commit 0a460e2 into faucetsdn:master Apr 8, 2019
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.

2 participants