Skip to content

Conversation

@upbeta01
Copy link
Contributor

@geerlingguy
Copy link
Owner

I haven't had an issue with this in building instances of Drupal VM or other projects that install extensions on 14.04... is this something that only affects one particular version of Ubuntu or can be worked around by running an apt-get update?

I'd rather not have the hundreds/thousands of people using this role suddenly waste an extra ton of bandwidth on every provision just to work around a particular Ubuntu bug.

@upbeta01
Copy link
Contributor Author

I could have had pasted the error in relation to the pull request comment, my bad. Basically it only affects one Ubuntu version but not sure if running "apt-get update" patches a fix for this bug -- but as far as my orchestration goes, it should have run apt-get update prior to installing your roles to the client host.

The objective for this pull request is to cater the ones who'll encounter the bug using the raw roles you have. Say like me, I have been using most of your roles for provisioning client hosts which is called via ansible-galaxy, as much as I would like -- I wanted to have the base form of it and pin it down to a specific version as when you'll be having some huge changes and update the repo, I won't have to re-install those roles again and re-edit the script just to place the " -Z " option in particular (if the new release still encounters the same error).

This is purely to preserve the scripts the way you have released it so the ones using it (if in case) encounters error won't be manually editing it.

@geerlingguy
Copy link
Owner

@geerlingguy geerlingguy merged commit 7fe73d5 into geerlingguy:master Nov 7, 2016
@geerlingguy
Copy link
Owner

Merged, but made the install command a variable so it would stay as-is by default. Thanks!

@upbeta01
Copy link
Contributor Author

upbeta01 commented Nov 8, 2016

Nice. Glad to contribute! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants