Arbitrary args for kubectl

[nlopezgi]

Trying this PR to replace all other PRs discussed in #209.
With this PR:

• users can add args (string list) to their k8s_deploy object
• users can use bazel run <target> -- <extra args>
• modified templates so the kubectl command is printed out when running bazel run , useful for validation in e2e tests

Hope this satisfies everyone's needs.
fyi @chrislovecnm @Globegitter @omadawn

[chrislovecnm]

Can users have both args defined in the BUILD fuel and pass in args in the cli?

[nlopezgi]

Can users have both args defined in the BUILD fuel and pass in args in the cli?

Yes, in the e2e tests I added both verification for use of args in the attrs defined in the BUILD file as well as one for use of bazel run --
(still fixing the tests/templats to get this working properly)

[chrislovecnm]

You gonna rebase? Ping me when the tests are happy and I will merge ;)

[nlopezgi]

alright, that was fun to debug. Tests are passing now, ptal.

[chrislovecnm]

Harping on this, but do we want to squash the commits on merging? I like to keep master clean so we can easily back out a PR.

I prefer when the PR owner squashed there commits into units of work.

[nlopezgi]

its set up for squash and merge, so it'll be 1 commit only

[chrislovecnm]

/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chrislovecnm, nlopezgi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [nlopezgi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment