Skip to content

test: validate cluster is in desired state #6

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.

Sign up for GitHub

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

Jump to bottom
Open
xoxwaw wants to merge 3 commits into
base: main
Choose a base branch
from
Open

test: validate cluster is in desired state #6

xoxwaw wants to merge 3 commits into from

Conversation

@xoxwaw
Copy link
Contributor

@xoxwaw xoxwaw commented Aug 31, 2022 •
edited
Loading

Please confirm that you have done the following before requesting reviews:

  • I have confirmed that the PR type is appropriate for the change I am making according to the Honest Pull Request and Commit Message Naming Conventions.
  • I have typed an adequate description that explains why I am making this change.
  • I have installed and run standard pre-commit hooks that lints and validates my code.

Description

  • Add consumer and producer to validate the cluster is in desired state.
  • Cluster state verification

This change is Reviewable

@xoxwaw xoxwaw requested a review from a team as a code owner August 31, 2022 07:10
@xoxwaw xoxwaw requested a review from jai August 31, 2022 07:10
@pull-request-size pull-request-size bot added size/L and removed size/M labels Aug 31, 2022
@xoxwaw xoxwaw changed the title test: add consumer and producer to validate cluster is in desired state test: validate cluster is in desired state Sep 1, 2022
@xoxwaw xoxwaw force-pushed the phi/da-593-kafka-terraform-terratest-modification branch 4 times, most recently from 432a2a6 to 3586f48 Compare September 2, 2022 01:53
@xoxwaw xoxwaw force-pushed the phi/da-593-kafka-terraform-terratest-modification branch from 3586f48 to 19b4a24 Compare September 2, 2022 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jai jai Awaiting requested review from jai jai is a code owner automatically assigned from honestbank/devops-engineers

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xoxwaw