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

'test=True' does not work with artifactory.download #50558

Closed
angeloudy opened this issue Nov 19, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@angeloudy
Copy link
Contributor

commented Nov 19, 2018

Description of Issue/Question

Even I am running state.apply with test=true. The artifactory download state still downloads the file.
Since the file is huge, and the state is very slow to run.
And I have other states that have onchanges dependency on the artifactory.download.
Because the file was already downloaded when I ran it in real, other states would not run because there's no changes.

Setup

(Please provide relevant configs and/or SLS files (Be sure to remove sensitive info).)

Steps to Reproduce Issue

(Include debug logs if possible and relevant.)

Versions Report

(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)

Salt Version:
           Salt: 2018.3.1

Dependency Versions:
           cffi: 1.11.5
       cherrypy: unknown
       dateutil: 2.7.3
      docker-py: Not Installed
          gitdb: Not Installed
      gitpython: Not Installed
          ioflo: Not Installed
         Jinja2: 2.10
        libgit2: Not Installed
        libnacl: Not Installed
       M2Crypto: Not Installed
           Mako: Not Installed
   msgpack-pure: Not Installed
 msgpack-python: 0.5.6
   mysql-python: 1.3.12
      pycparser: 2.18
       pycrypto: 2.6.1
   pycryptodome: Not Installed
         pygit2: Not Installed
         Python: 3.6.6 (default, Jul 27 2018, 19:19:07)
   python-gnupg: Not Installed
         PyYAML: 3.12
          PyZMQ: 17.0.0
           RAET: Not Installed
          smmap: Not Installed
        timelib: Not Installed
        Tornado: 4.5.3
            ZMQ: 4.2.3

System Versions:
           dist:
         locale: UTF-8
        machine: amd64
        release: 11.2-RELEASE-p4
         system: FreeBSD
        version: Not Installed
@ari

This comment has been minimized.

Copy link
Contributor

commented Nov 19, 2018

Maybe a duplicate of #50155

@garethgreenaway

This comment has been minimized.

Copy link
Member

commented Nov 19, 2018

Yes. Appears to be a duplicate. Closing in favor of #50155.

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.