mlsorensen commented on code in PR #8033:
URL: https://github.com/apache/cloudstack/pull/8033#discussion_r1346230123


##########
pom.xml:
##########
@@ -154,7 +154,7 @@
         <cs.jaxws.version>2.3.2-1</cs.jaxws.version>
         <cs.jersey-client.version>2.26</cs.jersey-client.version>
         <cs.jetty.version>9.4.51.v20230217</cs.jetty.version>
-        
<cs.jetty-maven-plugin.version>9.4.27.v20200227</cs.jetty-maven-plugin.version>
+        
<cs.jetty-maven-plugin.version>9.4.51.v20230217</cs.jetty-maven-plugin.version>

Review Comment:
   Eh, I'm not confident that I know exactly what's going on here, but I think 
the same [PR](https://github.com/eclipse/jetty.project/pull/5131) in jetty that 
I need for this feature is probably the same one that cloudstack's use of 
jetty:run is incompatible with. Or certainly the very next release after this.
   
   May need to resolve the underlying reason why we can't upgrade to newer 
jetty-maven-plugin before we consider adding a plugin dir for devs.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to