-
Notifications
You must be signed in to change notification settings - Fork 13
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
added support for VCL #16
base: master
Are you sure you want to change the base?
Conversation
I haven't done any tests... |
Thanks a lot for your contribution. |
I've added S3 logging support and updated the README, but I can't figure out the tests, sorry. If someone else wants to complete the tests that'd be grand! |
…s prior to backends to allow setting backend.request_condition
…sponse_condition='' returns error
I figured out the tests! Wrote tests for all new functions - and added a paragraph for how to do the tests for next time I might need it :-) |
It would be lovely if the vcl could be provided from a file instead of only an inline string; that would help keep things tidy when dealing with lots of vcl. But perhaps that's something out of the scope of this pull. |
Hi - sorry - been a while - but I recently just added dictionary support - which required doing a clone instead of a full rebuild. Unfortunately, I don't think I'll get around to doing the tests this time. Its a pretty major rework. |
@xiongchiamiov I'd love to be able to do that too! But I haven't had the time to figure out how to use Ansible's templating calls. I currently load the files using vars and lookup('template') |
No description provided.