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

[issue 193] - Fix the KafkaOperatorprovisionerTest #194

Conversation

fabiobrz
Copy link
Member

@fabiobrz fabiobrz commented Sep 1, 2024

Description

Fix the KafkaOperatorprovisionerTest in order to support the latest Kafka version, i.e. 3.8.0. Updating documentation accordingly

Closes #193

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change
  • Chore (non-breaking change which doesn't affect codebase;
    test, version modification, documentation, etc.)

Checklist

  • Code contributed by me aligns with current project license: Apache 2.0
  • Pull Request contains a description of the changes
  • Pull Request does not include fixes for multiple issues/topics
  • Code is self-descriptive and/or documented
  • I have implemented unit tests to cover my changes
  • I tested my code in OpenShift

@fabiobrz fabiobrz self-assigned this Sep 1, 2024
@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 1, 2024

/test

…t the latest Kafka version, i.e. 3.8.0. Updating documentation accordingly
@fabiobrz fabiobrz force-pushed the issue-193.upgrade-supported-kafka-version.380 branch from 1f2f55a to bbf9a6e Compare September 2, 2024 07:28
@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 2, 2024

/test

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 2, 2024

Intersmash PR CI check results

Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see #194 (comment).

CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#92> FAILED:

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#39> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#51> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#57> reported failures.

    Please verify your changes locally or reach out to maintainers in order to investigate the issues.

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 2, 2024

/test

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 2, 2024

Intersmash PR CI check results

Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see #194 (comment).

CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#94> FAILED:

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#40> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#52> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#58> reported failures.

    Please verify your changes locally or reach out to maintainers in order to investigate the issues.

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 2, 2024

Intersmash PR CI check results

Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see #194 (comment).

CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#96> FAILED:

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#42> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#53> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#59> reported failures.

    Please verify your changes locally or reach out to maintainers in order to investigate the issues.

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 3, 2024

/test

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 3, 2024

Intersmash PR CI check results

Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see #194 (comment).

CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#101> FAILED:

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#46> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#57> reported failures.

  • Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#63> reported failures.

    Please verify your changes locally or reach out to maintainers in order to investigate the issues.

@fabiobrz
Copy link
Member Author

fabiobrz commented Sep 3, 2024

Intersmash PR CI check results

Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests, which were triggered by @fabiobrz comment, see #194 (comment).

CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#101> FAILED:

* Intersmash integration tests job <_eap-8.x-intersmash-integration-tests-community#46_> reported failures.

<eap-8.x-intersmash-integration-tests-community#46> reported 1 failure, i.e.:

  • [ERROR] Tests run: 7, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 275.987 s <<< FAILURE! - in org.jboss.intersmash.testsuite.provision.openshift.ProvisionerCleanupTestCase which is an intermittent network issue, reported as fixed by the subsequent run: <eap-8.x-intersmash-integration-tests-community#49> ✅
* Intersmash integration tests job <_eap-8.x-intersmash-integration-tests-products-jboss-eap#57_> reported failures.

<eap-8.x-intersmash-integration-tests-products-jboss-eap#57> reported 2 failures, i.e.:

  • RhSsoTemplateTestCase#deploy
  • WildflyHelmChartExistingValuesProvisionerTest#basicProvisioningTest
    which are both intermittent as fixed in <eap-8.x-intersmash-integration-tests-products-jboss-eap#58> ✅
* Intersmash integration tests job <_eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#63_> reported failures.
  Please verify your changes locally or reach out to maintainers in order to investigate the issues.

<eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#63> Reported 1 failure, i.e.:

  • org.jboss.intersmash.testsuite.provision.openshift.ProvisionerCleanupTestCase.testProvisioningWorkflowCleanup(OpenShiftProvisioner)[1]
    Which is related to the InfinispanOperatorProvisioner and deemed as intermittent (not appearing in the base EAP run mentioned above) and anyway unrelated to the changes in this PR ✅

Based on the above findings, the changes in the PR are deemed to pass the integration tests and can now be reviewed.

@fabiobrz fabiobrz assigned marekkopecky and unassigned fabiobrz Sep 3, 2024
@marekkopecky marekkopecky merged commit 12697c3 into Intersmash:main Sep 3, 2024
1 check passed
@marekkopecky
Copy link
Collaborator

Thank you for this MR, LGTM, merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kafka Operator tests are failing when using community deliverables
2 participants