Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add "kgoss" kubectl wrapper to test containers in Kubernetes #465

Merged
merged 2 commits into from
Dec 6, 2019

Conversation

joshgav
Copy link
Contributor

@joshgav joshgav commented Jul 11, 2019

In my environment containers must be built and run within a Kubernetes cluster so I ported dgoss to use kubectl [run | exec | cp] ... instead of docker and named the port "kgoss". Thought others might benefit from this too so sharing back upstream.

If you want this, would love to know what your use case is and what other features you seek.

There are refinements to be made and I'd love to add an e2e test, but it seemed a good time to get feedback on the general idea and direction. Thanks!

@aelsabbahy
Copy link
Member

This looks great. However, I'm keeping dgoss as the only semi-official script. By that I mean it's the only one installed by install.sh by default.

If you just remove it from install.sh, it can be treated like dcgoss in the extras folder.

Maybe in the future, it can be split up into goss-extras bundle or something like that.. but for now, I'd prefer to keep them seperate.

Once that's done I'll merge this. Thanks again!

@joshgav
Copy link
Contributor Author

joshgav commented Nov 26, 2019

Thanks @aelsabbahy, glad to contribute! Working on removing the install commands and some minor touchups now.

@joshgav
Copy link
Contributor Author

joshgav commented Nov 26, 2019

Updated and rebased, tests pass, ready for merge @aelsabbahy.

@joshgav
Copy link
Contributor Author

joshgav commented Dec 3, 2019

Hi @aelsabbahy - I think this is ready to go, LMK if you'd like anything further.

.travis.yml Outdated Show resolved Hide resolved
@aelsabbahy
Copy link
Member

LGTM! I'll probably merge this tomorrow along with a few other PRs.

Thank you for this. I'll be opening a new issue soon tagging you and other contributors to extras/ to see how it can be best managed moving forward.

@aelsabbahy aelsabbahy merged commit e213a2c into goss-org:master Dec 6, 2019
@aelsabbahy
Copy link
Member

FYI: Made a slight change to the readme after the merge

BenjaminHerbert pushed a commit to BenjaminHerbert/goss that referenced this pull request May 28, 2020
…g#465)

* add kgoss kubectl wrapper

* remove kgoss from published artifacts
@aelsabbahy aelsabbahy mentioned this pull request Sep 14, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants