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

Add use_inline_resources to web_app #507

Closed
SharpEdgeMarshall opened this issue Aug 28, 2017 · 7 comments
Closed

Add use_inline_resources to web_app #507

SharpEdgeMarshall opened this issue Aug 28, 2017 · 7 comments

Comments

@SharpEdgeMarshall
Copy link

Cookbook version

v5.0.0

Scenario:

When using web_app resource is impossible to add a notification because is never executed

Use case

I need to immediately reload apache2, after creating app config, to request a SSL certificate with acme protocol.

@tas50
Copy link
Contributor

tas50 commented Aug 28, 2017

@SharpEdgeMarshall Can you open a PR up to fix this?

@SharpEdgeMarshall
Copy link
Author

Sure i could make it, but i'm still waiting a feedback on the previous one, do you merge PRs or let them sit there forever?

@lamont-granquist
Copy link
Contributor

web_app is a definition not a resource, so use_inline_resources will not work until its converted to a resource

@cheeseplus
Copy link

@lamont-granquist beat me to it but I'll add that Definitions can't send or receive notifications which is why this isn't as simple as editing the existing definition.

@svanzoest
Copy link
Contributor

This should be addressed by #414

@damacus
Copy link
Member

damacus commented Nov 15, 2018

Closing due to inactivity.

If this is still an issue please reopen or open another issue. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help!

Thanks,
Sous-Chefs

@damacus damacus closed this as completed Nov 15, 2018
@lock
Copy link

lock bot commented Nov 15, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants