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

win_get_url: ignore defender false positive in tests #56812

Merged
merged 3 commits into from May 23, 2019

Conversation

jborean93
Copy link
Contributor

@jborean93 jborean93 commented May 22, 2019

SUMMARY

We are seeing more errors in trying to start this service that is intermittent https://app.shippable.com/github/ansible/ansible/runs/124053/14/tests. This adds a retry on this task to try again instead of failing straight away.

It seems like the binary is being flagged as a false positive virus by Windows. This adds an exclusion for that binary until we get a better solution to replace it.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

win_get_url tests

@ansibot
Copy link
Contributor

ansibot commented May 22, 2019

@ansibot ansibot added affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. needs_triage Needs a first human triage before being processed. support:core This issue/PR relates to code supported by the Ansible Engineering Team. windows Windows community labels May 22, 2019
@ansibot ansibot added core_review In order to be merged, this PR must follow the core review workflow. and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels May 23, 2019
@mattclay mattclay removed the needs_triage Needs a first human triage before being processed. label May 23, 2019
@mattclay mattclay changed the title win_get_url: add retry to service step win_get_url: ignore defender false positive May 23, 2019
@mattclay mattclay changed the title win_get_url: ignore defender false positive win_get_url: ignore defender false positive in tests May 23, 2019
@mattclay mattclay merged commit 124400f into ansible:devel May 23, 2019
@jborean93 jborean93 deleted the tests-win_get_url branch May 23, 2019 06:23
mattclay pushed a commit to mattclay/ansible that referenced this pull request May 23, 2019
…sible#56812)

(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>
mattclay pushed a commit to mattclay/ansible that referenced this pull request May 23, 2019
…sible#56812)

(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>
mattclay pushed a commit to mattclay/ansible that referenced this pull request May 23, 2019
…sible#56812)

(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>
mattclay pushed a commit that referenced this pull request May 23, 2019
…6812)

(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>
abadger pushed a commit that referenced this pull request May 23, 2019
…6825)

* [stable-2.7] win_get_url: ignore defender false positive in tests (#56812)
(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>

* Adapt tests to work without remote_tmp_dir.
abadger pushed a commit that referenced this pull request May 23, 2019
…6826)

* [stable-2.6] win_get_url: ignore defender false positive in tests (#56812)
(cherry picked from commit 124400f)

Co-authored-by: Jordan Borean <jborean93@gmail.com>

* Adapt tests to work without remote_tmp_dir.
@ansible ansible locked and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.9 This issue/PR affects Ansible v2.9 bug This issue/PR relates to a bug. core_review In order to be merged, this PR must follow the core review workflow. support:core This issue/PR relates to code supported by the Ansible Engineering Team. windows Windows community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants