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

ci deb: extract confluent test case #643

Merged
merged 1 commit into from
Jun 19, 2024
Merged

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Apr 10, 2024

In the previous versions, testing with confluent is
executed in serverspec:linux.

As confluent test case should be out of scope about basic serverspec
to distinct confluent's failure, so extract them as isolated test case.

  • github ci: added "Confluent Test"
  • Upgrade confluent from 6.0 to 7.6
  • Replace deprecated --zookeeper option to --bootstrap-server to
    create kafka topics

@kenhys kenhys force-pushed the split-confluent branch 7 times, most recently from a1a334a to 46a7d9b Compare April 11, 2024 07:18
In the previous versions, testing with confluent is
executed in serverspec:linux.

As confluent test case should be out of scope about basic serverspec
to distinct failure, so extract them as isolated test case.

* github ci: added "Confluent Test"
* Upgrade confluent from 6.0 to 7.6
* Replace deprecated --zookeeper option to --bootstrap-server to
  create kafka topics

Signed-off-by: Kentaro Hayashi <[email protected]>
@kenhys kenhys marked this pull request as ready for review April 11, 2024 08:48
Copy link
Contributor

@daipom daipom left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@kenhys kenhys merged commit aac4b6b into fluent:master Jun 19, 2024
61 checks passed
@kenhys kenhys deleted the split-confluent branch June 19, 2024 03:44
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