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

ansible subversion module silently fails on network problems #29824

Closed
ansibot opened this issue Sep 12, 2017 · 4 comments
Closed

ansible subversion module silently fails on network problems #29824

ansibot opened this issue Sep 12, 2017 · 4 comments
Labels
affects_2.0 This issue/PR affects Ansible v2.0 bug This issue/PR relates to a bug. module This issue/PR relates to a module. source_control Source-control category support:core This issue/PR relates to code supported by the Ansible Engineering Team.

Comments

@ansibot
Copy link
Contributor

ansibot commented Sep 12, 2017

From @Petrox on 2016-04-11T20:00:01Z

ISSUE TYPE
  • Bug Report
COMPONENT NAME

subversion

ANSIBLE VERSION
ansible 2.0.1.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides
CONFIGURATION

nothing special

OS / ENVIRONMENT

ubuntu 14.04.4 trusty fully upgraded

SUMMARY

sometimes - because of network errors - subversion can't finish the checkout, but this should raise an error, ansible though moves on believing evertyhing is okay

I could also say, that if I do an "svn update" after a successful ansible checkout zero changes should happen (if the server side did not change of course).

STEPS TO REPRODUCE

create or find a large svn repo
checkout with subversion module
stop your network interface while downloading

EXPECTED RESULTS

ansible should exit with an error

ACTUAL RESULTS

ansible moves on like if checkout was a success

Copied from original issue: ansible/ansible-modules-core#3405

@ansibot
Copy link
Contributor Author

ansibot commented Sep 12, 2017

From @ansibot on 2016-04-11T20:00:01Z

@dsummersl ping, this issue is waiting for your response.
click here for bot help

@ansibot ansibot added the affects_2.0 This issue/PR affects Ansible v2.0 label Sep 12, 2017
@ansibot ansibot added bug_report module This issue/PR relates to a module. support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Sep 12, 2017
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 1, 2018
@ansibot ansibot added the source_control Source-control category label Feb 17, 2019
@ansibot ansibot added the needs_triage Needs a first human triage before being processed. label May 16, 2020
@mkrizek mkrizek removed the needs_triage Needs a first human triage before being processed. label May 18, 2020
@s-hertel
Copy link
Contributor

s-hertel commented Jun 4, 2021

This seems like a problem with svn hanging/not handling errors properly. The subversion module checks the return code after checking out the repo, so the module should fail if svn is accurately reporting failures.

@s-hertel s-hertel closed this as completed Jun 4, 2021
@Petrox
Copy link
Contributor

Petrox commented Jun 5, 2021 via email

@bcoca
Copy link
Member

bcoca commented Jun 7, 2021

2.11 has a timeout keyword for the task, no matter what the action is.

Suibversion itself does have a built int timeout now (defaults to 1h iirc).

@ansible ansible locked and limited conversation to collaborators Jul 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.0 This issue/PR affects Ansible v2.0 bug This issue/PR relates to a bug. module This issue/PR relates to a module. source_control Source-control category support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
None yet
Development

No branches or pull requests

5 participants