apigee / apigee-deploy-maven-plugin

Apache License 2.0
80 stars 169 forks source link

apigee-edge-maven-plugin:1.0.3 restarts maven lifecycle #48

Closed seymen closed 6 years ago

seymen commented 8 years ago

Here is an excerpt from maven output where all lifecycle stages before maven execution is repeated once apigee-edge-maven-plugin is initiated:

Note before/after of this line:

[INFO] >>> apigee-edge-maven-plugin:1.0.3:deploy (deploy-bundle-step) > package @ currency-v1 >>>
[INFO] Scanning for projects...
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] Building currency-oseymenv1 -
[INFO] ------------------------------------------------------------------------
[INFO]
[INFO] --- jshint-maven-plugin:1.6.0:lint (jslint) @ currency-v1 ---
[INFO] using jshint version 2.5.6
[INFO]   /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/apiproxy/resources/jsc/ConfigureTargetRequest.js
[INFO]   /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/apiproxy/resources/jsc/LogToLoggly.js
[INFO]
[INFO] --- exec-maven-plugin:1.3.2:exec (unit) @ currency-v1 ---

  feature: loggly integration
    ✓ should push target error response data to loggly

  1 passing (32ms)

[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (auto-clean-init) @ currency-v1 ---
[INFO] Deleting /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/target
[INFO]
[INFO] --- maven-resources-plugin:2.6:copy-resources (copy-resources) @ currency-v1 ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] Copying 32 resources
[INFO]
[INFO] --- replacer:1.5.2:replace (default) @ currency-v1 ---
[INFO] Replacement run on 3 files.
[INFO]
[INFO] --- exec-maven-plugin:1.3.2:exec (npm-install) @ currency-v1 ---
npm WARN package.json currency-v1@1.0.0 No repository field.
npm WARN package.json currency-v1@1.0.0 No license field.
[INFO]
[INFO] --- apigee-edge-maven-plugin:1.0.3:configure (configure-bundle-step) @ currency-v1 ---
[INFO] No config.json found. Skipping package configuration.
[INFO]

=============Checking for node.js app================

[INFO]

=============Now zipping the App Bundle================

[INFO]
[INFO] --- maven-install-plugin:2.4:install (default-install) @ currency-v1 ---
[INFO] Installing /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/pom.xml to /Users/oseymen/.m2/repository/apigee/currency-v1/-/currency-v1--.pom
[INFO]
[INFO] >>> apigee-edge-maven-plugin:1.0.3:deploy (deploy-bundle-step) > package @ currency-v1 >>>
[INFO]
[INFO] --- jshint-maven-plugin:1.6.0:lint (jslint) @ currency-v1 ---
[INFO] using jshint version 2.5.6
[INFO]   /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/apiproxy/resources/jsc/ConfigureTargetRequest.js
[INFO]   /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/apiproxy/resources/jsc/LogToLoggly.js
[INFO]
[INFO] --- exec-maven-plugin:1.3.2:exec (unit) @ currency-v1 ---

  feature: loggly integration
    ✓ should push target error response data to loggly

  1 passing (33ms)

[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (auto-clean-init) @ currency-v1 ---
[INFO] Deleting /Users/oseymen/projects/prof/apigee/accelerator-ci-maven/currency-v1/target
ssvaidyanathan commented 8 years ago

@seymen Try running mvn apigee-enterprise:deploy -Ptest instead of mvn install -Ptest

seymen commented 8 years ago

@ssvaidyanathan - that will only run executions until apigee deployment. Even other executions in the same phase coming after apigee deployment will not work.

What is stopping us adhering to maven lifecycles so we can put executions before or after apigee deployment without them getting executed twice?

ssvaidyanathan commented 7 years ago

I have the fix for this... will push in the next release

ssvaidyanathan commented 6 years ago

This is fixed and released in https://github.com/apigee/apigee-deploy-maven-plugin/releases/tag/apigee-edge-maven-plugin-1.1.6