You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@brooklyn.apache.org by GitBox <gi...@apache.org> on 2022/05/31 07:10:45 UTC
[GitHub] [brooklyn-server] algairim opened a new pull request, #1318: Prevent application from deployment if the user does not have the req…
algairim opened a new pull request, #1318:
URL: https://github.com/apache/brooklyn-server/pull/1318
…uired entitlement
Signed-off-by: Mykola Mandra <my...@cloudsoft.io>
--
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: dev-unsubscribe@brooklyn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-server] algairim commented on pull request #1318: Prevent application from deployment if the user does not have the req…
Posted by GitBox <gi...@apache.org>.
algairim commented on PR #1318:
URL: https://github.com/apache/brooklyn-server/pull/1318#issuecomment-1143168591
Build has failed with unrelated test failure in the `StreamGobblerTest.java`.
--
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: dev-unsubscribe@brooklyn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-server] jcabrerizo merged pull request #1318: Prevent application from deployment if the user does not have the req…
Posted by GitBox <gi...@apache.org>.
jcabrerizo merged PR #1318:
URL: https://github.com/apache/brooklyn-server/pull/1318
--
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: dev-unsubscribe@brooklyn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [brooklyn-server] algairim commented on a diff in pull request #1318: Prevent application from deployment if the user does not have the req…
Posted by GitBox <gi...@apache.org>.
algairim commented on code in PR #1318:
URL: https://github.com/apache/brooklyn-server/pull/1318#discussion_r885289161
##########
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/ApplicationResource.java:
##########
@@ -18,35 +18,20 @@
*/
package org.apache.brooklyn.rest.resources;
-import static com.google.common.base.Preconditions.checkNotNull;
-import static javax.ws.rs.core.Response.created;
-import static javax.ws.rs.core.Response.status;
-import static javax.ws.rs.core.Response.Status.ACCEPTED;
-import org.apache.brooklyn.api.mgmt.ManagementContext;
-import org.apache.brooklyn.core.config.Sanitizer;
-import static org.apache.brooklyn.rest.util.WebResourceUtils.serviceAbsoluteUriBuilder;
-
-import java.net.URI;
-import java.net.URISyntaxException;
-import java.nio.charset.StandardCharsets;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.concurrent.Callable;
-import java.util.stream.Collectors;
-
-import javax.ws.rs.core.Context;
-import javax.ws.rs.core.Response;
-import javax.ws.rs.core.Response.ResponseBuilder;
-import javax.ws.rs.core.UriInfo;
-
+import com.google.common.base.Optional;
+import com.google.common.base.Preconditions;
+import com.google.common.base.Predicate;
+import com.google.common.base.Predicates;
+import com.google.common.collect.FluentIterable;
+import com.google.common.collect.ImmutableMap;
Review Comment:
My IDE re-shuffled these imports.
--
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: dev-unsubscribe@brooklyn.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org