GitLab

Rework e2e tests so they all pass

Merge environment-scoped libraries with environment overrides

* Simplify app code dealing with overrides
* env set will not mutate passed environment, filters out library configration for overrides
* app.Environment, app.Environments return merged views with overrides applied
* Test override environment merging
* remove redundant calls to mergedEnvironment
* Remove name parameter from AddEnvironment
* Fix override flag for merged environments
Signed-off-by: Oren Shomron <shomron@gmail.com>

This is a tech debt cleanup PR that revises all the e2e tests that
where failing.
Signed-off-by: bryanl <bryanliles@gmail.com>

Don't install helm charts with hooks

Don't change cluster in apply dry run modified

If a helm hook is encountered in a YAML file in a helm chart's `templates`
directory, skip it.

Fixes #700
Fixes #703
Signed-off-by: bryanl <bryanliles@gmail.com>

Add JSON format to table printer

Don't drop empty objects/arrays in env params

Add more test cases to rand string gen

* Exercise random string generation by adding larger test cases
* wrap random number generation with a mutex because math/rand isn't goroutine safe
Signed-off-by: bryanl <bryanliles@gmail.com>

There is a case where empty parameter values could get dropped
from environment parameters

In

```
{
    components: {
        component: {
            values: {}
        }
    }
}
```

component.values would be dropped because it contained an
empty object.
Signed-off-by: bryanl <bryanliles@gmail.com>

Allow installing packages into environment scope

* Add --env flag to pkg install cli
* Add support for environment qualifier in app.UpdateLib()
* Refactor CacheDependency to only focus on vendoring, app update is handled by the caller
* Move default app.Environment[s]() implementation to baseApp.

Closes #688
Signed-off-by: Oren Shomron <shomron@gmail.com>

* Explicitly skipRoot if cmd is version
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

* Catch case for help flags; rename ErrNotInApp
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

* Refactored skipRoot logic; wrapped error message
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

* Added tests; change back to original ErrNotInApp
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

* Address cases without cmd;fix test param
Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

Add environment directory to jsonnet path before eval

Add conditionals for object tagging and upserting.
Signed-off-by: bryanl <bryanliles@gmail.com>

Signed-off-by: bryanl <bryanliles@gmail.com>

Create module when child exists

Update environment cli docs to remove old metadata cache structure

This PR adds JSON output to the existing table printer and updates all
actions which use the table printer.

JSON tables are printed out like:

```json
{
  "kind": "moduleList",
  "data": [
    {"module": "/"},
    {"module": "a"},
    {"module": "a.b"}
  ]
}
```

Where:

* `kind` is the kind of data in the table
* `data` contains the table rows matched to the header

Fixes #693
Signed-off-by: bryanl <bryanliles@gmail.com>

Use full module name when evaluating environment parameters

Signed-off-by: GuessWhoSamFoo <sfoohei@gmail.com>

Remove verbose 'app called' debug messages

Signed-off-by: Oren Shomron <shomron@gmail.com>

Create prototypes for helm packages

Signed-off-by: bryanl <bryanliles@gmail.com>

* create helm prototype
* `ks registry add` can add helm registries
Signed-off-by: bryanl <bryanliles@gmail.com>

Signed-off-by: bryanl <bryanliles@gmail.com>

Given the following modules:

```
$ ks module list                                                                                  108ms  Sun Jul  1 10:26:38 2018
MODULE
======
/
nested
nested2
```

When listing parameters, use the full component name

```
$ ks param list --env default                                                                     109ms  Sun Jul  1 10:26:43 2018
COMPONENT  PARAM         VALUE
=========  =====         =====
nested.ds  containerPort 80
nested.ds  image         "gcr.io/heptio-images/ks-guestbook-demo:0.1"
nested.ds  name          "ds"
nested.ds  replicas      1
nested.ds  servicePort   80
nested.ds  type          "ClusterIP"
nested2.ds containerPort 80
nested2.ds image         "gcr.io/heptio-images/ks-guestbook-demo:0.1"
nested2.ds name          "ds"
nested2.ds replicas      1
nested2.ds servicePort   80
nested2.ds type          "ClusterIP"
```
Signed-off-by: bryanl <bryanliles@gmail.com>

To remove the chance for ambiguous environment parameters, use the
full module name when evaluating environment parameters. For example:

Given two modules, nested and nested2, and they both have a
component named `deployment`, when setting environment parameters for
enviromnet `nested2` component `deployment`,
the ksonnet user will have to set the parameters like:

`ks param set nested2.deployment replicas 3`
Signed-off-by: bryanl <bryanliles@gmail.com>

Update ksonnet-lib printer to 0.1.10 to handle unescaping newlines

Allow creation of module `a` when module `a.b` previously exists.
Changes module finding to look for `params.libsonnet` instead of
the module directory.
Signed-off-by: bryanl <bryanliles@gmail.com>

Support environment arguments in module parameters

ks upgrade relocates vendored packages to versioned paths

Signed-off-by: bryanl <bryanliles@gmail.com>

Signed-off-by: bryanl <bryanliles@gmail.com>

Signed-off-by: bryanl <bryanliles@gmail.com>

Signed-off-by: bryanl <bryanliles@gmail.com>

Example:
vendor/<registry>/<pkg> -> vendor/<registry>/<pkg>@<version>

Closes #663
Signed-off-by: Oren Shomron <shomron@gmail.com>