{"action":"created","issue":{"url":"https://api.github.com/repos/apigee/apigeetool-node/issues/25","labels_url":"https://api.github.com/repos/apigee/apigeetool-node/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/apigee/apigeetool-node/issues/25/comments","events_url":"https://api.github.com/repos/apigee/apigeetool-node/issues/25/events","html_url":"https://github.com/apigee/apigeetool-node/pull/25","id":112157867,"number":25,"title":"removed setting base-path when deploying a revision","user":{"login":"antxxxx","id":2149559,"avatar_url":"https://avatars.githubusercontent.com/u/2149559?v=3","gravatar_id":"","url":"https://api.github.com/users/antxxxx","html_url":"https://github.com/antxxxx","followers_url":"https://api.github.com/users/antxxxx/followers","following_url":"https://api.github.com/users/antxxxx/following{/other_user}","gists_url":"https://api.github.com/users/antxxxx/gists{/gist_id}","starred_url":"https://api.github.com/users/antxxxx/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/antxxxx/subscriptions","organizations_url":"https://api.github.com/users/antxxxx/orgs","repos_url":"https://api.github.com/users/antxxxx/repos","events_url":"https://api.github.com/users/antxxxx/events{/privacy}","received_events_url":"https://api.github.com/users/antxxxx/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-19T14:34:15Z","updated_at":"2015-10-19T16:50:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apigee/apigeetool-node/pulls/25","html_url":"https://github.com/apigee/apigeetool-node/pull/25","diff_url":"https://github.com/apigee/apigeetool-node/pull/25.diff","patch_url":"https://github.com/apigee/apigeetool-node/pull/25.patch"},"body":"When using this to deploy an already existing proxy, it created the new revision correctly, but when it did the deploy, it did not undeploy the old revision, so we ended up with two revisions deployed to the same environment. \r\neg we have revision 2 deployed to environment test\r\nwe use apigetool to deploy a new version from source control to environment test\r\nrevision 3 is created and deployed\r\nWhen looking at deployments, revision 2 and revision 3 are both showing as deployed to test\r\n\r\nTo fix this, we had to undeploy revision 2, undeploy revision 3 and then deploy revision 3 again\r\n\r\nIt seems to be setting base-path in the deploy API call that is causing the issue. Removing it seems to fix the issue"},"comment":{"url":"https://api.github.com/repos/apigee/apigeetool-node/issues/comments/149277993","html_url":"https://github.com/apigee/apigeetool-node/pull/25#issuecomment-149277993","issue_url":"https://api.github.com/repos/apigee/apigeetool-node/issues/25","id":149277993,"user":{"login":"gbrail","id":521323,"avatar_url":"https://avatars.githubusercontent.com/u/521323?v=3","gravatar_id":"","url":"https://api.github.com/users/gbrail","html_url":"https://github.com/gbrail","followers_url":"https://api.github.com/users/gbrail/followers","following_url":"https://api.github.com/users/gbrail/following{/other_user}","gists_url":"https://api.github.com/users/gbrail/gists{/gist_id}","starred_url":"https://api.github.com/users/gbrail/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gbrail/subscriptions","organizations_url":"https://api.github.com/users/gbrail/orgs","repos_url":"https://api.github.com/users/gbrail/repos","events_url":"https://api.github.com/users/gbrail/events{/privacy}","received_events_url":"https://api.github.com/users/gbrail/received_events","type":"User","site_admin":false},"created_at":"2015-10-19T16:50:17Z","updated_at":"2015-10-19T16:50:17Z","body":"Can you describe the set of actions that you took in order to create this\nproblem? Were you using the \"-b\" option to set a base path on your original\ndeployment?\n\nIt may be the case that the required fix is in the undeploy code. The\nability to deploy multiple proxies at different base paths in an\nenvironment is a useful feature, which is why the \"-b\" option is there. I'd\nrather not fix this by removing that capability.\n\nOn Mon, Oct 19, 2015 at 7:34 AM, Anthony Brown <notifications@github.com>\nwrote:\n\n> When using this to deploy an already existing proxy, it created the new\n> revision correctly, but when it did the deploy, it did not undeploy the old\n> revision, so we ended up with two revisions deployed to the same\n> environment.\n> eg we have revision 2 deployed to environment test\n> we use apigetool to deploy a new version from source control to\n> environment test\n> revision 3 is created and deployed\n> When looking at deployments, revision 2 and revision 3 are both showing as\n> deployed to test\n>\n> To fix this, we had to undeploy revision 2, undeploy revision 3 and then\n> deploy revision 3 again\n>\n> It seems to be setting base-path in the deploy API call that is causing\n> the issue. Removing it seems to fix the issue\n> ------------------------------\n> You can view, comment on, or merge this pull request online at:\n>\n>   https://github.com/apigee/apigeetool-node/pull/25\n> Commit Summary\n>\n>    - removed setting base-path when deploying a revision\n>\n> File Changes\n>\n>    - *M* lib/commands/deployproxy.js\n>    <https://github.com/apigee/apigeetool-node/pull/25/files#diff-0> (4)\n>\n> Patch Links:\n>\n>    - https://github.com/apigee/apigeetool-node/pull/25.patch\n>    - https://github.com/apigee/apigeetool-node/pull/25.diff\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/apigee/apigeetool-node/pull/25>.\n>\n\n\n\n-- \n*Greg Brail* | *apigee <https://apigee.com/>* | twitter @gbrail\n<http://twitter.com/gbrail> @apigee <https://twitter.com/apigee>\n"}}