[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320685545
 
 
   Thanks @marcaurele please collaborate on #2226 , feel free to directly push 
changes. I'll use your suggestions and incorporate them in the systemd scripts 
now, currently trying to fix centos packaging.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320680234
 
 
   @marcaurele thanks, I started working on this PR based on your branch to fix 
the centos7 issues: https://github.com/apache/cloudstack/pull/2226 this is on 
ASF remote so any committer should be able to commit or send PRs.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320651759
 
 
   @marcaurele with the above fix I could get the build to work, however can 
you share how would you run the management server both using mvn and scripts? 
I'm unable to get it to work.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320638183
 
 
   @marcaurele I've found the fix, can you put this commit in your branch as I 
cannot push changes:
   ```
   diff --git a/tools/apidoc/build-apidoc.sh b/tools/apidoc/build-apidoc.sh
   index 8ab69c8373..4e1f9a639d 100755
   --- a/tools/apidoc/build-apidoc.sh
   +++ b/tools/apidoc/build-apidoc.sh
   @@ -47,7 +47,7 @@ fi

CP=$PATHSEP/

   -java -cp $CP$PATHSEP$TARGETJARDIR/*$PATHSEP$DEPSDIR/* 
com.cloud.api.doc.ApiXmlDocWriter -d "$DISTDIR" $*
   +java -cp $CP$PATHSEP$TARGETJARDIR$PATHSEP$DEPSDIR 
com.cloud.api.doc.ApiXmlDocWriter -d "$DISTDIR" $*

if [ $? -ne 0 ]
then
   diff --git a/tools/apidoc/gen_toc.py b/tools/apidoc/gen_toc.py
   index a261037d80..39e1576422 100644
   --- a/tools/apidoc/gen_toc.py
   +++ b/tools/apidoc/gen_toc.py
   @@ -180,7 +180,8 @@ known_categories = {
'stopNetScalerVpx' : 'Load Balancer',
'deleteServicePackageOffering' : 'Load Balancer',
'destroyNsVpx' : 'Load Balancer',
   -'startNsVpx' : 'Load Balancer'
   +'startNsVpx' : 'Load Balancer',
   +'listElastistorInterface': 'Misc'
}


   diff --git a/tools/apidoc/pom.xml b/tools/apidoc/pom.xml
   index e6804a0e78..61c1ccd593 100644
   --- a/tools/apidoc/pom.xml
   +++ b/tools/apidoc/pom.xml
   @@ -21,8 +21,8 @@
../pom.xml


   -
../../client/target/cloud-client-ui-${project.version}/WEB-INF/
   -${client.config.base}/lib
   +../../client/target/
   +
${client.config.base}/cloud-client-ui-${project.version}.jar

${client.config.base}/classes

   ```
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320617540
 
 
   @marcaurele thanks, I'll have a look this week. Can you fix Travis failures 
as well, are there any changes wrt how a developer builds, runs and tests 
CloudStack?
   
   I've one reservation -- if we put everything in a single jar, it might be 
difficult for users to apply UI changes and add custom jars, anyway to do that? 
Maybe add a custom path (add a classpath) for an override directory from where 
jars may be loaded and maybe split the UI as a separate rpm/deb?
   
   Can you also allow me to push on the exoscale:CLOUDSTACK-10012 branch? 
Alternatively, you may push the changes to the upstream branch so I can 
collaborate and make changes with you.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320617812
 
 
   @blueorangutan package
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320617540
 
 
   @marcaurele thanks, I'll have a look this week. I've one reservation -- if 
we put everything in a single jar, it might be difficult for users to apply UI 
changes and add custom jars, anyway to do that? Maybe add a custom path (add a 
classpath) for an override directory from where jars may be loaded and maybe 
split the UI as a separate rpm/deb?
   Can you also allow me to push on the exoscale:CLOUDSTACK-10012 branch? 
Alternatively, you may push the changes to the upstream branch so I can 
collaborate and make changes with you.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] rhtyd commented on issue #2207: Embedded jetty inside shaded jar

2017-08-07 Thread git
rhtyd commented on issue #2207: Embedded jetty inside shaded jar
URL: https://github.com/apache/cloudstack/pull/2207#issuecomment-320617540
 
 
   @marcaurele thanks, I'll have a look this week. I've one reservation -- if 
we put everything in a single jar, it might be difficult for users to apply UI 
changes and add custom jars, anyway to do that? Maybe add a custom path (add a 
classpath) for an override directory from where jars may be loaded and maybe 
split the UI as a separate rpm/deb?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services