-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Support for onlink routing #19507
Support for onlink routing #19507
Conversation
The build fails. You may need to update the SDK |
@wyunchi-ms i do not think the CI pipeline has the latest Microsoft.Azure.Management.Network package. It does not contain latest changes from 2022-05-01 branch. I built this locally for my testing. The changes I'm referring to specifically is the StaticRoutesConfig struct which causes these failures. In my local build this is present. Will follow up with module owners regd this. |
@wyunchi-ms my checks seem to be passing now. could you please trigger the security tools check? |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi @wyunchi-ms seeing some kind of error in security-tools. Looks like an internal error of some kind. Would it be possible to restart the check? |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
@wyunchi-ms i think the merge introduced some errors, but doesnt seem related to my changes. Would it be possible to redo the merge validation? |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
Description
Checklist
CONTRIBUTING.md
and reviewed the following information:generation
branch.ChangeLog.md
file(s) appropriatelyChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header in the past tense. Add changelog in description section if PR goes intogeneration
branch.ChangeLog.md
if no new release is required, such as fixing test case only.