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

Get release path variable #8

Merged
merged 1 commit into from
Jan 7, 2016
Merged

Conversation

magi2c
Copy link
Contributor

@magi2c magi2c commented Jan 6, 2016

To share custom tasks between the deploy and rollback.

Example hook:

ansistrano_after_cleanup_tasks_file

- name: doctrine:cache:clear-metadata
  shell: chdir={{ ansistrano_release_path.stdout }}
    php app/console doctrine:cache:clear-metadata --env={{dp_sf_env}} --no-debug

To share custom tasks between the deploy and rollback.

Example hook:

ansistrano_after_cleanup_tasks_file

```
- name: doctrine:cache:clear-metadata
  shell: chdir={{ ansistrano_release_path.stdout }}
    php app/console doctrine:cache:clear-metadata --env={{dp_sf_env}} --no-debug

```
@ricardclau
Copy link
Member

Why would you need that? If you roll back, your production cache will still be there right?

@ricardclau
Copy link
Member

Now that I think about it, your production cache would be there if you use the default app/cache but not if for instance you override getCacheFolder to something like /var/cache in your AppKernel, right?

And generally speaking, even if this particular task does not need it, you may have tasks shared between deploy and rollback that may need it

I am 👍 about it, what do you think @carlosbuenosvinos @theUniC ???

@magi2c
Copy link
Contributor Author

magi2c commented Jan 7, 2016

Yeah, In my case I have the cache in Redis. And when do rollback I have to clean the cache.

@ricardclau
Copy link
Member

Oh, of course, I forgot about these Doctrine cache adapters, merging and retagging

Thanks @magi2c !

ricardclau added a commit that referenced this pull request Jan 7, 2016
Get release path variable
@ricardclau ricardclau merged commit 6469adf into ansistrano:master Jan 7, 2016
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