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

Fix for newly introduced sanity tests #58375

Open
wants to merge 2 commits into
base: devel
from

Conversation

Projects
None yet
4 participants
@zikalino
Copy link
Contributor

commented Jun 26, 2019

SUMMARY

Some sanity tests started failing

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

all

ADDITIONAL INFORMATION
@@ -19,7 +19,8 @@ class ModuleDocFragment(object):
password:
description:
- Active Directory user password. Use when authenticating with an Active Directory user rather than service
principal.
principal

This comment has been minimized.

Copy link
@yungezz

yungezz Jun 26, 2019

Contributor

why remove the .?

@yungezz

This comment has been minimized.

Copy link
Contributor

commented Jun 26, 2019

shipit

@ansibot ansibot added shipit and removed community_review labels Jun 26, 2019

@mattclay mattclay added ci_verified and removed needs_triage labels Jun 26, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 26, 2019

The test ansible-test sanity --test validate-modules [explain] failed with 1 error:

test/sanity/validate-modules/ignore.txt:445:1: A102 Remove since "lib/ansible/modules/cloud/azure/azure_rm_manageddisk_facts.py" passes "E337" test

click here for bot help

@ansibot ansibot added needs_revision and removed shipit labels Jun 26, 2019

@ansibot ansibot added the stale_ci label Jul 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.