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.

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

[ST] acceptance profile test count reduction #10528

Merged
merged 2 commits into from
Sep 5, 2024
Merged

[ST] acceptance profile test count reduction #10528

merged 2 commits into from
Sep 5, 2024

Conversation

henryZrncik
Copy link
Contributor

@henryZrncik henryZrncik commented Sep 2, 2024 •
edited
Loading

Type of change

  • Enhancement
  • Refactoring

Description

Acceptance profile contained a bit large number of tests to run. The total number is now reduced by crossed tests/classes

suite/tests included in Acceptance:

  • HttpBridgeTlsST
    • testSendSimpleMessageTls
    • testReceiveSimpleMessageTls
    • testTlsScramShaAuthWithWeirdUsername
    • testTlsAuthWithWeirdUsername
  • CruiseControlApiST
    • testCruiseControlBasicAPIRequestsWithSecurityDisabled
    • testCruiseControlAPIUsers
  • OauthTlsST
    • testProducerConsumer
    • testProducerConsumerConnect
    • testProducerConsumerBridge
    • testMirrorMaker
    • testMirrorMaker2
    • testIntrospectionEndpoint
  • MetricsST
    • testKafkaMetrics
    • testZookeeperMetrics
    • testKafkaConnectAndConnectorMetrics
    • testKafkaExporterMetrics
    • testKafkaExporterDifferentSetting
    • testClusterOperatorMetrics
    • testUserOperatorMetrics
    • testMirrorMaker2Metrics
    • testKafkaBridgeMetrics
    • testCruiseControlMetrics
    • testKafkaMetricsSettings
  • MultipleListenersST#testCombinationOfInternalAndExternalListeners
  • MirrorMaker2ST#testMirrorMaker2TlsAndTlsClientAuth
  • MirrorMakerST#testMirrorMakerTlsAuthenticated
  • LeaderElectionST#testLeaderElection
  • UserST#testUpdateUser
  • RollingUpdateST#testKafkaScaleUpScaleDown
  • PodSecurityProfilesST#testOperandsWithRestrictedSecurityProfile
  • SecurityST#testAutoRenewAllCaCertsTriggeredByAnno
  • OpenTelemetryST#testProducerConsumerStreamsService
  • ConnectBuilderST#testBuildPluginUsingMavenCoordinatesArtifacts
  • ConnectST#testMultiNodeKafkaConnectWithConnectorCreation
  • CruiseControlST#testCruiseControlWithRebalanceResourceAndRefreshAnnotation
  • ListenerST
    • testSendMessagesTlsScramSha
    • testSendMessagesCustomListenerTlsScramSha
    • testLoadBalancerTls
    • testCustomSoloCertificatesForRoute

@henryZrncik henryZrncik self-assigned this Sep 2, 2024
@henryZrncik henryZrncik added this to the 0.44.0 milestone Sep 2, 2024
@see-quick
Copy link
Member

/azp run acceptance

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Frawless
Copy link
Member

Frawless commented Sep 3, 2024

Are tests from ListenersST needed when we have MultipleListenersST#testCombinationOfInternalAndExternalListeners as part of acceptance?

@henryZrncik henryZrncik force-pushed the acceptance-profile-reduction branch from 53169f0 to 81f0829 Compare September 5, 2024 06:10
@henryZrncik henryZrncik requested a review from scholzj September 5, 2024 11:47
@see-quick
Copy link
Member

/azp run acceptance

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@henryZrncik henryZrncik added ready for merge Label for PRs which are ready for merge and removed needs review labels Sep 5, 2024
@scholzj scholzj merged commit e548ae7 into strimzi:main Sep 5, 2024
15 checks passed
@scholzj
Copy link
Member

scholzj commented Sep 5, 2024

Thanks for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scholzj scholzj scholzj approved these changes

@Frawless Frawless Frawless approved these changes

@see-quick see-quick see-quick approved these changes

@im-konge im-konge im-konge approved these changes

Assignees

@henryZrncik henryZrncik

Labels
ready for merge Label for PRs which are ready for merge System tests
Projects
None yet
Milestone
0.44.0
Development

Successfully merging this pull request may close these issues.
5 participants
@henryZrncik @see-quick @Frawless @scholzj @im-konge