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

Use non-deprecated appengine maven plugin for deploying portfolio #50

Merged
merged 1 commit into from
May 18, 2020

Conversation

andreahu12
Copy link

@andreahu12 andreahu12 commented May 18, 2020

I might be wrong because I haven't set up billing for my experimental project (and therefore haven't deployed successfully), but following the instructions and trying to deploy with mvn appengine:update was failing with "Deployments using appcfg are no longer supported. See https://cloud.google.com/appengine/docs/deprecations"

To get further in the deployment process, I think it needs to use a non-deprecated plugin to deploy. Using mvn appengine:devserver only works for locally testing with the old plugin. mvn appengine:run can be used to test locally for both.

Current plugin (deprecated): https://github.com/GoogleCloudPlatform/gcloud-maven-plugin
New plugin (not-deprecated): https://github.com/GoogleCloudPlatform/app-maven-plugin

@KevinWorkman
Copy link
Collaborator

Wow, thanks for catching this. I'm hoping to find an approach that makes it as easy as possible to deploy a local server, without needing to comment or uncomment anything. I wonder if there's a way to deploy a local server with the newer plugin? (I'm going to look into that on Monday, but let me know if you have any other ideas.)

@andreahu12
Copy link
Author

No problem! The old plugin still works fine for deploying locally. It's deploying to a live server that isn't working with the old plugin.

@andreahu12
Copy link
Author

Also you can deploy to a local server with the newer plugin using mvn appengine:run

@KevinWorkman
Copy link
Collaborator

Thanks again for looking into this. I'm going to merge this in as-is and then make some additional tweaks on top of it.

@KevinWorkman KevinWorkman merged commit bfaca98 into google:master May 18, 2020
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.

None yet

2 participants