Set up go environment

Merge pull request #89 from jcabrerizo/master

Adding mvn deploy if not PR

Adding mvn deploy if not PR

Merge pull request #88 from jcabrerizo/fix/help-template

Removing offending fields from template

Removing offending fields from template

fix to accommodate change in type of response builder

fix api change

Merge pull request #87 from apache/update-repos

Mvn 3.8.1 will refuse to use http repos

Mvn 3.8.1 will refuse to use http repos

Revert java to 3.5.4

remove mistaken .bak files and gitignore them

Merge pull request #86 from geomacy/update-go

BROOKLYN-629: add go module support

set GOCACHE to avoid permission denied on CI builds

set GOCACHE to avoid
"failed to initialize build cache at /.cache/go-build: mkdir /.cache: permission denied"
on CI builds

Set GOPATH in build script for Go module cache

Set GOPATH to ${outdir} so that Go module cache gets put there.

update README according to latest changes

adjust docker file to use Go 1.15

Upgrade Dockerfile to latest maven:x.y.z-jdk-8-slim,
but even then it uses Go no higher than 1.11, so
add some configuration to explicitly install 1.15.

fix mvn clean to make it work with module cache

remove GOPATH from build script

tidy maven build and remove unneeded files

update to Go modules - add go.mod and fix compilation

See

https://github.com/urfave/cli/blob/09ac54c2/docs/migrate-v1-to-v2.md

for migration instructions for urfave/cli version 2

update to go modules - remove vendor files

Merge pull request #85 from grkvlt/fix/deprecated-api-test

Adds CatalogApi test using equivalent non deprecated method

Adds CatalogApi test using equivalent non deprecated method

Signed-off-by: Andrew Donald Kennedy <andrew.kennedy@cloudsoftcorp.com>

This closes #84

updated LICENSE, NOTICE, DEPENDENCIES

feature poms include the dependencies the feature uses

    and tidy versions and exclusions for maven/osgi consistency

Set version to 1.1.0-SNAPSHOT

Merge pull request #82 from nakomis/notice-update

Updates NOTICE files

Updates NOTICE files

This closes #81

Update Jenkinsfile to:
- add the pipeline trigger on comment only if we are building a PR
- not deploy artifacts when building master. This will be taken care of during the full brooklyn build on master (see Jenkinsfile configuration at the brooklyn root)

Merge pull request #80 from tbouron/feature/jenkins

Reuse local maven cache + add PR comment trigger

Reuse local maven cache + add PR comment trigger

Merge pull request #79 from tbouron/master

Add jenkinsfile for multibranch job on Jenkins

Add jenkinsfile for multibranch job on Jenkins

Merge pull request #78 from geomacy/jsonpath-update

BROOKLYN-586 update br to replace obsolete jsonpath package

do a little more test and make sure to run unit tests during build

try matryer/is instead of testify

looks nice

tidy error handling and variable naming

fix a couple of error returns in (currently unused) functions

use path.Join instead of Sprintf

Remove some duplication in test code

avoid warnings

update notice files with new package

echo Go version in build.sh

switch to using Kubernetes jsonpath package

add tests for Display

Merge pull request #77 from jcabrerizo/feature/multiHeaders

Allow sent user defined headers and omit credentials if not needed

removed second reading of 'authorizationParam'

remove cedentials output

formating files with gofmt

cleared authorization param manage

unitary test

changed open header params for authorization. Networks now manages credentials instead of user:pass

added headers format

added default behaviour if not credentialsRequired param is found

test for headers and noCredential params

removed dev-outputs

headers and noCredentials as global

updated license files for new deps

This closes #76

bump resteasy to be compatible with new jersey

fix need for csrf header now that that is respected in tests

This closes #72

Switch to dep for dependency management

This closes #71

Fix for SUREFIRE-1588

Latest version of Java 1.8.0_191 enforces that Manifest classpath entries be
relative.

https://issues.apache.org/jira/browse/SUREFIRE-1588

This closes #70

fix jenkins docker build
Jenkins build failed writing to /var/maven/.brooklyn

Closes #69

updated LICENSE and NOTICE, using new procedure

Supporting changes for https://github.com/apache/brooklyn-dist/pull/123

updated LICENSE and NOTICE, using new procedure

Closes #68

Temporarily use fork of NodePrime repo

See https://lists.apache.org/thread.html/10a74756dbeb1243928eb87c379b01ba58357a6204149d98676fa025@%3Cdev.brooklyn.apache.org%3E

Until this is resolved we can use John's fork

Add vendor file and remove glide from build/readme

Temporarily use fork of NodePrime repo

See https://lists.apache.org/thread.html/10a74756dbeb1243928eb87c379b01ba58357a6204149d98676fa025@%3Cdev.brooklyn.apache.org%3E

Until this is resolved we can use John's fork

Closes #67

Invoke effector: support timeout

Passes the timeout arg (if supplied) when making the REST api call to Brooklyn. The response from this is written to stdout.

If it times out, the REST api still gives back a 202 response code with a json payload describing the executing task. The `br` tool simply writes that to stdout. For example:

```
cli/target/bin/darwin.amd64/br app rbfvau9i75 effector stop invoke --timeout 1ms

{"id":"kpML04c1","displayName":"stop","description":"Invoking effector stop on server @ localhost with parameters {}","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost","tags":["EFFECTOR",{"entityId":"rbfvau9i75","effectorName":"stop"},{"entitlementContext":{"user":"<snip>","sourceIp":"0:0:0:0:0:0:0:1","requestUri":"/v1/applications/rbfvau9i75/entities/rbfvau9i75/effectors/stop","requestUniqueIdentifier":"rjuSuB"}},{"type":"org.apache.brooklyn.api.mgmt.ManagementContext"},{"wrappingType":"targetEntity","entity":{"type":"org.apache.brooklyn.api.entity.Entity","id":"rbfvau9i75"}},{"entitlementContext":{"user":"<snip>","sourceIp":"0:0:0:0:0:0:0:1","requestUri":"/v1/applications/rbfvau9i75/entities/rbfvau9i75/effectors/stop","requestUniqueIdentifier":"rjuSuB"}},{"wrappingType":"contextEntity","entity":{"type":"org.apache.brooklyn.api.entity.Entity","id":"rbfvau9i75"}}],"submitTimeUtc":1517490162402,"startTimeUtc":1517490162402,"endTimeUtc":null,"currentStatus":"In progress","result":null,"isError":false,"isCancelled":false,"children":[{"link":"/v1/activities/X9j8xcJ1","metadata":{"id":"X9j8xcJ1","taskName":"invoking stop[] on 1 node","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost"}}],"submittedByTask":null,"blockingTask":{"link":"/v1/activities/X9j8xcJ1","metadata":{"id":"X9j8xcJ1","taskName":"invoking stop[] on 1 node","entityId":"rbfvau9i75","entityDisplayName":"server @ localhost"}},"detailedStatus":"Waiting on Task[invoking stop[] on 1 node]@X9j8xcJ1\n\nTask[stop]@kpML04c1\nChildren:\n  Task[invoking stop[] on 1 node]@X9j8xcJ1: Executing 1 child task\n\nIn progress, thread waiting (notify) on java.util.concurrent.FutureTask@61fe3136\nAt: com.google.common.util.concurrent.ForwardingFuture.get(ForwardingFuture.java:63)\n    org.apache.brooklyn.util.core.task.BasicTask.get(BasicTask.java:384)\n    org.apache.brooklyn.util.core.task.BasicTask.getUnchecked(BasicTask.java:393)\n    org.apache.brooklyn.util.core.task.DynamicTasks$TaskQueueingResult.andWaitForSuccess(DynamicTasks.java:195)\n    org.apache.brooklyn.util.core.task.DynamicTasks.get(DynamicTasks.java:363)\n    org.apache.brooklyn.core.entity.trait.StartableMethods.stop(StartableMethods.java:58)\n    org.apache.brooklyn.core.entity.AbstractApplication.doStop(AbstractApplication.java:271)\n    org.apache.brooklyn.core.entity.AbstractApplication.stop(AbstractApplication.java:246)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:984)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:871)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:857)\n    org.apache.brooklyn.util.javalang.Reflections.invokeMethodFromArgs(Reflections.java:852)\n    org.apache.brooklyn.core.mgmt.internal.AbstractManagementContext.invokeEffectorMethodLocal(AbstractManagementContext.java:327)\n    org.apache.brooklyn.core.mgmt.internal.AbstractManagementContext.invokeEffectorMethodSync(AbstractManagementContext.java:366)\n    org.apache.brooklyn.core.mgmt.internal.EffectorUtils.invokeMethodEffector(EffectorUtils.java:274)\n    org.apache.brooklyn.core.effector.MethodEffector.call(MethodEffector.java:153)\n    org.apache.brooklyn.core.entity.trait.Startable$StopEffectorBody.call(Startable.java:68)\n    org.apache.brooklyn.core.entity.trait.Startable$StopEffectorBody.call(Startable.java:60)\n    org.apache.brooklyn.core.effector.EffectorTasks$EffectorBodyTaskFactory$1.call(EffectorTasks.java:82)\n    org.apache.brooklyn.util.core.task.DynamicSequentialTask$DstJob.call(DynamicSequentialTask.java:364)\n    org.apache.brooklyn.util.core.task.BasicExecutionManager$SubmissionCallable.call(BasicExecutionManager.java:565)","streams":{},"links":{"self":"/v1/activities/kpML04c1","children":"/v1/activities/kpML04c1/children","entity":"/v1/applications/rbfvau9i75/entities/rbfvau9i75"}}
```

Invoke effector: support timeout

This closes #66

`br` on Windows generates incorrect ZIP archives

The ZIP archive should use / internally but when on Windows `br` used
\ instead. This affects the `br catalog add` command when given a
directory.

br catalog add: Fix path parsing errors on Windows

br has trouble working with files on Windows due to its attempt to
coerce filenames to file: URLs, which are ill-defined and unreliable
when it comes to backslashes and drive letters. Code revised to handle
files as files instead of URLs to avoid a number of issues.

This closes #65

Add dockerfile for CI build

This closes #64

Default login port to 80/443 if not supplied

Closes #63

Fixes index out of range issue

Wasn't handling calls where there were only flag params e.g.
br -v

Fixes index out of range issue

Wasn't handling calls where there were only flag params e.g.
br -v

This closes #61

Delete old deprecated code

Change version to 1.0.0-SNAPSHOT

This closes #60

Fix tests tearDown (to always stop management context)

Change version to 0.13.0-SNAPSHOT

Closes #59

Move Resteasy dependency management to java module pom

Closes #58

Fix handling of global flags in scope definition

Previously the presence of global flags in a command like `br --flag app a ent b sensor c` caused the CLI to run the command for app, not sensor.

Closes #57

Add flags for verbose output of HTTP request data

New global flags `--verbose` and `--vverbose` (very verbose) configure the network library to write data about API requests to stderr. (I would have used `-v` and `-vv` but the former is already taken as shorthand for `--version`.)

The intention is to make it easier to debug both the working of the CLI and unexpected responses from Brooklyn servers.

Example:
```
$ ./bin/br --verbose deploy ./bp.yaml
POST /v1/applications HTTP/1.1
Host: localhost:8081
User-Agent: Go-http-client/1.1
Content-Length: 0
Authorization: ******
Content-Type: application/json
Accept-Encoding: gzip

HTTP/1.1 201 Created
Transfer-Encoding: chunked
Cache-Control: no-cache, no-store
Content-Type: application/json
Date: Thu, 27 Jul 2017 14:12:56 GMT
Expires: 0
Location: http://localhost:8081/v1/applications/sr9uw9bdbu
Pragma: no-cache
Server: Jetty(9.2.13.v20150730)
Vary: Accept-Encoding

Id:       | sr9uw9bdbu
Name:     | Application (sr9uw9bdbu)
Status:   | In progress
```

```
$ ./bin/br --vverbose application vhkhp6nbbl entity
GET /v1/applications/vhkhp6nbbl/entities HTTP/1.1
Host: localhost:8081
User-Agent: Go-http-client/1.1
Accept: application/json, text/plain
Authorization: ******
Accept-Encoding: gzip

HTTP/1.1 200 OK
Transfer-Encoding: chunked
Cache-Control: no-cache, no-store
Content-Type: application/json
Date: Thu, 27 Jul 2017 14:16:43 GMT
Expires: 0
Pragma: no-cache
Server: Jetty(9.2.13.v20150730)
Vary: Accept-Encoding

503
[{"id":"tqj1a2g883","name":"Tomcat 7 Server","type":"org.apache.brooklyn.entity.webapp.tomcat.TomcatServer","catalogItemId":"org.apache.brooklyn.entity.webapp.tomcat.TomcatServer:0.12.0-SNAPSHOT","links":{"self":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883","parent":"/v1/applications/vhkhp6nbbl/entities/vhkhp6nbbl","application":"/v1/applications/vhkhp6nbbl","children":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/children","config":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/config","sensors":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/sensors","effectors":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/effectors","policies":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/policies","activities":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/activities","locations":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/locations","tags":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/tags","expunge":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/expunge","rename":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/name","spec":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/spec","iconUrl":"/v1/applications/vhkhp6nbbl/entities/tqj1a2g883/icon","catalog":"/v1/catalog/entities/org.apache.brooklyn.entity.webapp.tomcat.TomcatServer/0.12.0-SNAPSHOT"}}]
0

Id           | Name              | Type
tqj1a2g883   | Tomcat 7 Server   | org.apache.brooklyn.entity.webapp.tomcat.TomcatServer
```

The Authorization header is redacted.

Move Resteasy dependency management to java module pom

Add flags for verbose output of HTTP request data

Closes #56

Set ContentType header when adding .yaml and .bom files to the catalog

Using application/x-yaml saves the Brooklyn server the hassle of guessing. At the moment it's too smart for its own good and gives useless errors about zipfiles when there are errors in yaml files, e.g. when trying to add yaml containing an undefined alias.

Before:
```
$ br catalog add mybp.bom
Server error (400): Bundle BasicManagedBundle{symbolicName=null, version=null, url=n
ull} failed preparation: Unable to read /tmp/brooklyn-bundle-transient-null-86366849
75390229868.zip when looking for manifest: ZipException: error in opening zip file
```
After:
```
$ br catalog add mybp.bom
Server error (400): ComposerException: found undefined alias doesntexist
 in 'reader', line 23, column 15:
            name: *doesntexist
```
I'll look to improve the feedback from Brooklyn's guessing endpoint too.

Fix handling of global flags in scope definition

Previously the presence of global flags in a command like
"br --flag app a ent b sensor c" caused the CLI to run the command for
app, not sensor.

Set ContentType when adding .yaml and .bom files to the catalog

Using application/x-yaml saves the Brooklyn server the hassle of
guessing. At the moment it's too smart for its own good and gives
useless errors about zipfiles when there are errors in yaml files.

This closes #42

Improve login handling.

Ensure logout does not prevent help being displayed