Erin Call
499ab6877f
Do repo error-checking in AddRepo.Prepare [ #26 ]
2019-12-30 13:24:57 -08:00
Erin Call
2b802df00f
Merge branch 'master' into license
2019-12-30 11:59:12 -08:00
Erin Call
48b6b3f5b3
Create AddRepo steps when there are repos to add [ #26 ]
2019-12-30 11:57:19 -08:00
Joachim Hill-Grannec
29ee0c53ab
Merge pull request #38 from pelotech/help-by-default
...
Run the help step by default
2019-12-30 11:52:17 -07:00
Erin Call
22e30fea56
The prefix setting is implemented [#19,#9]
...
Just something I noticed while resolving a merge conflict. The "write
some docs" and "implement prefix" branches happened concurrently and
didn't get re-coordinated.
2019-12-30 09:56:47 -08:00
Erin Call
9155d5eeeb
Merge branch 'master' into helm-repos
2019-12-30 09:56:18 -08:00
Erin Call
75c99683b5
AddRepo step that calls helm repo add [ #26 ]
...
As with some of the other commands, I'm not sure `--namespace` is
relevant here. Just rolling with the "at worst it doesn't hurt" theory.
2019-12-30 09:52:00 -08:00
Joachim Hill-Grannec
eb1834df49
Merge branch 'master' into help-by-default
2019-12-28 09:31:04 -07:00
Joachim Hill-Grannec
1b2af0b3ac
Merge pull request #44 from pelotech/update-dependencies
...
Call `helm dependency update` when so instructed
2019-12-28 09:30:32 -07:00
Joachim Hill-Grannec
0f8657b1c2
Merge branch 'master' into update-dependencies
2019-12-28 09:29:53 -07:00
Joachim Hill-Grannec
edb1fb6e27
Merge pull request #47 from pelotech/timeout-formatting
...
Shim bare numbers into duration strings
2019-12-28 09:27:40 -07:00
Erin Call
d5a59590a1
Shim bare numbers into duration strings [ #39 ]
...
Helm2's --timeout took a number of seconds, rather than the
ParseDuration-compatible string that helm3 uses. For backward-
compatibility, update a bare number into a duration string.
2019-12-27 16:20:32 -08:00
Erin Call
89ec9425b0
Mention the chart param for uninstalls [ #25 ]
2019-12-27 15:44:09 -08:00
Erin Call
181165cc51
Call helm dependency update when so instructed [ #25 ]
...
As with Lint, I have no idea whether the --namespace flag actually
matters here. I don't think it will hurt, though!
2019-12-27 15:06:32 -08:00
Erin Call
354dce2e12
Use the apache 2.0 license [ #23 ]
2019-12-27 11:18:13 -08:00
Erin Call
12c8e08d6f
Mention that EKS isn't currently supported [ #5 ]
...
We'd like to support it eventually, but the current state of affairs
doesn't justify the effort.
Also removed some vestigial code that was copy-pasta from the kubeconfig
in drone-helm.
2019-12-27 10:49:22 -08:00
Erin Call
3b78f01b45
Merge pull request #37 from pelotech/github-metafiles
...
Metafiles for the github repo config
2019-12-26 15:38:46 -08:00
Erin Call
24060c18fc
Provide better e2e-testing instructions [ #12 ]
...
The "get a kubernetes cluster" part could use some outfleshing, but this
is certainly more straightforward than it was before.
2019-12-26 15:21:34 -08:00
Erin Call
232bb5eb96
Rely on the PR template for docs/code consistency [ #12 ]
...
These comments were a reasonable attempt at ensuring the documentation
matched reality, but the checkbox in the pull request template is much
more likely to produce results.
2019-12-26 13:03:53 -08:00
Erin Call
c0964ae178
Merge branch 'master' into github-metafiles
2019-12-26 13:02:02 -08:00
Erin Call
818c0246fa
Merge branch 'master' into help-by-default
2019-12-26 13:00:13 -08:00
Erin Call
9f9e83da99
Merge pull request #35 from pelotech/testplan
...
Add omitted plan tests
2019-12-26 12:59:54 -08:00
Erin Call
39aea4c8dd
Merge branch 'master' into testplan
2019-12-26 12:58:46 -08:00
Erin Call
7e25218945
Merge pull request #36 from pelotech/kubeconfig-tests
...
Validation tests for the kubeconfig template
2019-12-26 12:58:27 -08:00
Erin Call
3b85c38714
Test yaml validity without a new dependency [ #15 ]
...
It turns out testify already depends on yaml, so we aren't adding
anything new by using it here.
2019-12-26 12:53:36 -08:00
Erin Call
1422ec77a4
Merge branch 'master' into kubeconfig-tests
2019-12-26 12:50:00 -08:00
Erin Call
2a13fff548
Don't check the generated config's yaml syntax [ #13 ]
...
See discussion on https://github.com/pelotech/drone-helm3/pull/36 --it
doesn't really make sense to add a dependency on yaml just for testing.
2019-12-26 12:39:02 -08:00
Erin Call
d53a1ed942
Merge branch 'master' into testplan
2019-12-26 12:26:07 -08:00
Erin Call
167b53691b
Put HelmCommand in Help, not run.Config [ #15 ]
2019-12-26 12:23:56 -08:00
Erin Call
b1899dee56
Merge remote-tracking branch 'origin/master' into help-by-default
2019-12-26 12:23:14 -08:00
Erin Call
fc3bfc466a
Merge pull request #32 from pelotech/config-fixup
...
Clarify and improve internal/helm.Config
2019-12-26 12:23:00 -08:00
Erin Call
8f2d4bec49
Merge branch 'master' into testplan
2019-12-26 12:08:30 -08:00
Erin Call
b2066961e1
Merge branch 'master' into kubeconfig-tests
2019-12-26 11:55:32 -08:00
Joachim Hill-Grannec
253a4465f8
Merge branch 'master' into config-fixup
2019-12-26 11:36:55 -08:00
Joachim Hill-Grannec
8857782826
Merge pull request #30 from pelotech/noncompulsory-certificate
...
Make the Certificate setting optional
2019-12-26 11:36:38 -08:00
Erin Call
41e9e42239
Emit a trailing newline on execution error [ #15 ]
...
Just something I noticed while testing the help command's error case.
2019-12-26 11:34:07 -08:00
Erin Call
6d28b7b28a
Return an error on unknown commands [ #15 ]
...
I'm probably overthinking this--explicitly calling help is a strange and
unusual case--but it doesn't really hurt, so I'm going for it.
2019-12-26 11:29:33 -08:00
Joachim Hill-Grannec
81c4877a92
Merge branch 'master' into noncompulsory-certificate
2019-12-26 11:27:43 -08:00
Joachim Hill-Grannec
fbfb69f063
Merge pull request #27 from pelotech/useful-readme
...
Put some useful information in the README
2019-12-26 11:26:59 -08:00
Erin Call
34b9ec1c4c
Run the Help step by default [ #15 ]
2019-12-26 10:47:42 -08:00
Erin Call
568f613401
Associate lines of text with their yaml blocks [ #8 ]
...
As I skimmed through that section I noticed it wasn't immediately clear
whether a line of text was referring to the example above it or the one
below it.
2019-12-26 09:44:46 -08:00
Erin Call
dc05855aa5
Mention the settings/environment equivalency [ #8 ]
...
It seems like this needs more information, like why you'd want to put
something in one stanza or the other, but I don't really know enough
about drone to give useful advice.
2019-12-26 09:41:10 -08:00
Erin Call
a71bba71fd
Give contributors some maybe-adequate information [ #12 ]
...
I honestly have no idea how useful this document is, but it's probably
better than nothing?
2019-12-25 16:08:29 -08:00
Erin Call
f00f6a6329
Encourage good issues with issue templates [ #12 ]
...
I've never used github's new multiple-option issue templates before!
I'm excited to see them in action :)
2019-12-25 12:03:36 -08:00
Erin Call
1a70a626ea
Use the Contributor Covenant code of conduct [ #12 ]
2019-12-25 11:47:41 -08:00
Erin Call
4f549ddf2a
Pre-populate pull request bodies with a template [ #12 ]
...
Note that as of this writing, docs/parameter_reference.md (and required-
params documentation in README.md) doesn't actually exist; I created it
in ef66bc0
2019-12-25 11:21:33 -08:00
Erin Call
4cd369b051
Auto-assign rewiewers with a CODEOWNERS [ #12 ]
2019-12-25 10:50:20 -08:00
Erin Call
6b331fdf03
Check the validity of the kubeconfig template [ #13 ]
...
It's a little tricky to find a balance between "brittle" and "thorough"
in this test--I'd like to verify that e.g. the certificate is in
clusters[0].cluster.certificate-authority-data, not at the root. On the
other hand, we can't actually show that it's a valid kubeconfig file
without actually *using* it, so there's a hard upper limit on the
strength of the assertions. I've settled on verifying that all the
settings make it into the file and the file is syntactically-valid yaml.
2019-12-25 10:11:14 -08:00
Erin Call
801598e1c5
Use a clearer filepath for the kubeconfig template [ #13 ]
2019-12-24 16:16:22 -08:00
Erin Call
cb58b5a021
Phrase errors in Execute the same as in Prepare [ #33 ]
2019-12-24 15:49:47 -08:00
