search

drone-plugins / drone-ansible

Drone plugin to provision via Ansible
http://plugins.drone.io/drone-plugins/drone-ansible
Apache License 2.0
30 stars 41 forks source link

Add a no-galaxy-force option #44

Closed hrenard closed 3 years ago

hrenard commented 3 years ago

Related to #43. This add the possibility to remove the --force tag during galaxy install. The end goal is to allow caching of galaxy packages.

hrenard commented 3 years ago

I forgot the PLUGIN_. But are you sure about the no ? I had the feeling it was impossible to have boolean flag defaulting to true. The negation is to keep the current behavior by default.

tboerger commented 3 years ago

BoolTFlag defaults to true.

hrenard commented 3 years ago

My bad. I fixed it.

hrenard commented 3 years ago

The CI reached docker hub pull rate limit but it should be fine now (11h later). If someone with the appropriate permissions could restart the build to get this merged.

techknowlogick commented 3 years ago

CI fail is fine as like you said it is due to docker hub rate limits, the other tests pass. Although if you want to trigger a build you could send an empty commit.

tboerger commented 3 years ago

Thanks for you effort!