-
Notifications
You must be signed in to change notification settings - Fork 4k
Order of Operations for OVPN test #7185
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
Conversation
@maddieclayton looks like run with wrong API version, I will correct this |
Failure doesn't seem related to test |
@henry416 Hey Henry, the build is failing after CredScan flagged a few files for containing potential passwords, secrets, keys, etc. You can find the results here: https://azuresdkci.westus2.cloudapp.azure.com/job/powershell/7369/artifact/src/Package/CredentialScannerOutput/CredScanIssues.csv Would you mind taking a look at the files that were flagged and ensure that these are values that anyone else can use? (i.e., they are dummy values or have been deleted since the recording) If it turns out this is the case, we can suppress these exceptions (using the method outlined here). |
@cormacpayne This is a false positive cred, I have updated the file to try to suppress it. |
On request of @MikhailTryakhov, updated the route API test record as well |
Description
Fix for ovpn test to go in proper order of operations as expected by preview instructions
Checklist
CONTRIBUTING.md
platyPS
module