Skip to content
This repository has been archived by the owner on Feb 13, 2023. It is now read-only.

Refactor tasks into shim roles #1327

Merged
merged 13 commits into from
Apr 26, 2017
Merged

Conversation

oxyc
Copy link
Collaborator

@oxyc oxyc commented Apr 22, 2017

I also included e84af01 in this PR (sorry) beetboxvm/beetbox#359. I can remove that commit and open up a separate PR if you prefer an easier read.

Tested:

  • Ubuntu 16.04 with PHP 5.6 -> re-provision with 7.1
  • Ubuntu 16.04 with PHP 7.1 -> re-provision with 5.6
  • DRUPALVM_ANSIBLE_TAGS=php vagrant provision
  • DRUPALVM_ANSIBLE_TAGS=xdebug vagrant provision
  • DRUPALVM_ANSIBLE_TAGS=database vagrant provision
  • DRUPALVM_ANSIBLE_TAGS=webserver vagrant provision

- name: Define php_xhprof_html_dir.
set_fact:
php_xhprof_html_dir: "/usr/share/php/xhprof_html"
when: php_xhprof_html_dir is not defined
Copy link
Collaborator Author

@oxyc oxyc Apr 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I almost moved this task but then realized it exists so that provisioning doesnt fail even if xhprof and the variable isn't available. 42d554d

@oxyc
Copy link
Collaborator Author

oxyc commented Apr 22, 2017

Changing the PHP version with

DRUPALVM_ANSIBLE_TAGS=php vagrant provision

works, but you also have to manually empty the twig cache:

rm -rf sites/default/files/php/twig/*

@oxyc oxyc changed the title [WIP] Refactor tasks into shim roles Refactor tasks into shim roles Apr 22, 2017
@geerlingguy
Copy link
Owner

Heh, you've created quite the backlog of Travis CI jobs :)

Thanks so much for the help—main suggestion would be you can update the 'Author' section to mention you instead of me (or if you insist, in addition to), since (a) you did this refactoring work, and (b) I think you probably wrote more of the tasks that found their ways into these particular roles!


## Author Information

This role was created in 2017 by [Jeff Geerling](https://www.jeffgeerling.com/), author of [Ansible for DevOps](https://www.ansiblefordevops.com/).
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update these sections to mention something like "was created in 2017 by Oskar Schöldström".

Also, how do you pronounce your last name?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, but I'll definitely leave you in the meta as I would be the most unreliable maintainer ever hehe. I prefer offloading it on to you 😄

Haha, the "ö" is pronounced like the "i" in bird. I'll listen in the next time there's a recorded session and check how well you do.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just spent the past 5 minutes trying to figure out how to pronounce the 'i' in bird... I just can't do it without saying "ir".

@geerlingguy geerlingguy added this to the 4.5.0 milestone Apr 22, 2017
@geerlingguy geerlingguy merged commit bd6ba69 into geerlingguy:master Apr 26, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants