[GitHub] brooklyn-server issue #927: Bundle type endpoint

2018-01-08 Thread tbouron
Github user tbouron commented on the issue:

https://github.com/apache/brooklyn-server/pull/927
  
I did but looks like I messed up my env somehow. Everything is fine now, 
sorry for the noise @ahgittin 


---


[GitHub] brooklyn-server issue #927: Bundle type endpoint

2018-01-08 Thread ahgittin
Github user ahgittin commented on the issue:

https://github.com/apache/brooklyn-server/pull/927
  
v strange.  are you sure it was built correctly @tbouron ?  it maybe had a 
`mvn clean install` pull down a slightly-old snapshot jar?  adding a new 
`*Resource` class needs updates in a few places but adding methods to the 
`*Api` classes is usually straightforward.


---


[GitHub] brooklyn-server issue #927: Bundle type endpoint

2018-01-08 Thread tbouron
Github user tbouron commented on the issue:

https://github.com/apache/brooklyn-server/pull/927
  
Actually @ahgittin, it seems the new endpoint is now available: I get a 404 
and it does not appear on the swagger page. Not sure what's going on here


---


[GitHub] brooklyn-server issue #927: Bundle type endpoint

2018-01-08 Thread tbouron
Github user tbouron commented on the issue:

https://github.com/apache/brooklyn-server/pull/927
  
Yep, the `latest` for bundles is not in the scope of this PR so I'm happy 
if there is another one for this.

Merging


---


[GitHub] brooklyn-server issue #927: Bundle type endpoint

2018-01-08 Thread ahgittin
Github user ahgittin commented on the issue:

https://github.com/apache/brooklyn-server/pull/927
  
+1 to supporting `latest` on bundle version -- i'll add that.  i think all 
other comments addressed.  `latest` bundle support could be new PR if you're 
happy to merge this now @tbouron .


---