appengine-pipelines icon indicating copy to clipboard operation
appengine-pipelines copied to clipboard

Fixes Issue #57

Open someone1 opened this issue 9 years ago • 3 comments

Moves pipeline from /_ah/pipeline/ to /_pipeline/. Also fixes up demo code

someone1 avatar Nov 09 '15 22:11 someone1

lgtm, just not sure about the architecture decision, reserving that for the googlers.

fwiw: there is nothing in here to ease the transition of queued tasks that are presently pointing to the _ah prefix.

soundofjw avatar Nov 09 '15 23:11 soundofjw

if you keep both entries in your YAML file it should go through fine. both new and old will end up executing on the correct routes, just new pipes will use the new route.

You can also specify the base URL when starting each pipeline, so that way you can explicitly keep all your current pipelines on the old path, and slowly remove the explicit arg for the ones that are not executing when deployed.

someone1 avatar Nov 09 '15 23:11 someone1

@tkaitchuck , any thoughts here? I'm not sure about the best choice of prefix, but I do agree something like this should be committed to ensure this code works on AppEngine Managed VMs (which is the future of AppEngine, from what I hear). Thanks.

mikelambert avatar Dec 01 '15 22:12 mikelambert