-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
fix webapp test #47546
fix webapp test #47546
Conversation
@@ -4,7 +4,7 @@ azure-cli-core==2.0.35 | |||
azure-cli-nspkg==3.0.2 | |||
azure-common==1.1.11 | |||
azure-mgmt-batch==4.1.0 | |||
azure.mgmt.cdn==3.0.0 | |||
azure-mgmt-cdn==3.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so is it really necessary? as it somehow worked with broken requirement....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it's necessary to do right thing. actually i didn't understand why previous version works :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so are you sure we need this dependency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, perhaps correct version is in test requirements..... but then sanity tests wouldn't pass....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I think you also need to update in test/runner/requirements
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch. fixed. the dependency is for cdn.
Hi @yungezz, thank you for submitting this pull-request! |
@yungezz, just so you are aware we have a dedicated Working Group for azure. |
SUMMARY
Fix webapp test, VSChina/ansible-issue-tracker#524
ISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION