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 initial test using bats #110

Merged
merged 1 commit into from
Aug 27, 2020
Merged

Add initial test using bats #110

merged 1 commit into from
Aug 27, 2020

Conversation

ncopa
Copy link
Collaborator

@ncopa ncopa commented Aug 24, 2020

Create a simple test for default config (without mke.yaml). It will
simply create a footloose cluster with a single node and run mke single --test.

Can be invoked with make check

Depends on bats and jq.

Signed-off-by: Natanael Copa ncopa@mirantis.com

@jnummelin
Copy link
Collaborator

@ncopa as discussed, maybe rebase now that #120 is merged. currently it "deadlocks" way too often. :)

Create a simple test for default config (without mke.yaml). It will
simply create a footloose cluster with a single node and run `mke
single --test`.

Can be invoked with `make check`

Depends on bats and jq.

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
Copy link
Collaborator

@jnummelin jnummelin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This is a good starting point

@jnummelin jnummelin merged commit 965674f into main Aug 27, 2020
@jnummelin jnummelin deleted the smoke-test branch August 27, 2020 10:05
@jnummelin jnummelin mentioned this pull request Aug 30, 2020
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.

None yet

2 participants