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] PodSecurityProfilesST, addition of check for KafkaConnect, merge of all of the tests together. #9117

Merged
merged 6 commits into from Sep 20, 2023
Merged

[ST] PodSecurityProfilesST, addition of check for KafkaConnect, merge of all of the tests together. #9117

merged 6 commits into from Sep 20, 2023

Conversation

henryZrncik
Copy link
Contributor

Type of change

  • Refactoring

Description

  • addition of Doc for test
  • merge of test into one
  • fix of consumer check with mirror maker 2. In old version client was communicating with source Kafka Cluster.
  • addition of testing of Security Profiles on Kafka Connect/Connector.

Checklist

  • Write tests
  • Make sure all tests pass
  • Update documentation

@henryZrncik
PodSecurityProfilesST addition of new test for KafkaCOnnect Restricte…
da95b7a 
…dSecurityProfile

Signed-off-by: hzrncik <hzrncik@redhat.com>
@henryZrncik
PodSecurityProfilesST merge of specific Restricted Security profiles …
0244afc 
…test into 1, fixing of bad consumer checks

Signed-off-by: hzrncik <hzrncik@redhat.com>
@henryZrncik
PodSecurityProfileST addition of check of incorrect config of Pod Sec…
df66ea8 
…urity Profile, removal of separated test for doing that

Signed-off-by: hzrncik <hzrncik@redhat.com>
@henryZrncik
PodSecurityProfileST addition of documentation
be35dac 
Signed-off-by: hzrncik <hzrncik@redhat.com>
@henryZrncik henryZrncik self-assigned this Sep 13, 2023
@henryZrncik henryZrncik added this to the 0.38.0 milestone Sep 13, 2023
@henryZrncik
pod security ST doc. usecase fix
66d5f79 
Signed-off-by: hzrncik <hzrncik@redhat.com>
im-konge
im-konge reviewed Sep 13, 2023
View reviewed changes
Copy link
Member

@im-konge im-konge left a comment

Choose a reason for hiding this comment

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

Few comments, thanks for the changes :)

systemtest/src/test/java/io/strimzi/systemtest/security/PodSecurityProfilesST.java Show resolved Hide resolved
systemtest/src/test/java/io/strimzi/systemtest/security/PodSecurityProfilesST.java Outdated Show resolved Hide resolved
systemtest/src/test/java/io/strimzi/systemtest/security/PodSecurityProfilesST.java Outdated Show resolved Hide resolved
@henryZrncik
PodSecurityProfiles fix checking of security contexts
47bcda1 
Signed-off-by: hzrncik <hzrncik@redhat.com>
@henryZrncik
Copy link
Contributor Author

@strimzi-ci run tests --profile=all --install-type=bundle --cluster-type=ocp --cluster-version=4.14 --testcase= PodSecurityProfilesST

@strimzi-ci
Copy link

▶️ Build started - check Jenkins for more info. ▶️

@strimzi-ci
Copy link

❌ Test Summary ❌

TEST_PROFILE: all
GROUPS:
TEST_CASE:
TOTAL: 249
PASS: 229
FAIL: 20
SKIP: 0
BUILD_NUMBER: 235
OCP_VERSION: 4.14
BUILD_IMAGES: false
FIPS_ENABLED: false
PARALLEL_COUNT: 5
EXCLUDED_GROUPS: loadbalancer,nodeport,olm

❗ Test Failures ❗

  • testKafkaManagementTransferToAndFromKafkaNodePool in io.strimzi.systemtest.kafka.KafkaNodePoolST
  • testKafkaQuotasPluginIntegration in io.strimzi.systemtest.kafka.QuotasST
  • testDynamicallySetBridgeLoggingLevels in io.strimzi.systemtest.log.LoggingChangeST
  • testStrimziIdentityReplicationPolicy in io.strimzi.systemtest.mirrormaker.MirrorMaker2ST
  • io.strimzi.systemtest.operators.CustomResourceStatusST
  • testTopicNotAvailable in io.strimzi.systemtest.operators.NamespaceDeletionRecoveryST
  • testCreateAndAlterBigAmountOfScramShaUsers in io.strimzi.systemtest.operators.user.UserScalabilityST
  • testClusterOperatorFinishAllRollingUpdates in io.strimzi.systemtest.rollingupdate.RollingUpdateST
  • testAutoReplaceAllCaKeysTriggeredByAnno in io.strimzi.systemtest.security.SecurityST
  • testTeamBWriteToTopic in io.strimzi.systemtest.security.oauth.OauthAuthorizationST
  • testPasswordGrantsKafkaBridge in io.strimzi.systemtest.security.oauth.OauthPasswordGrantsST
  • testUpgradeWithNoMessageAndProtocolVersionsSet in io.strimzi.systemtest.upgrade.KafkaUpgradeDowngradeST
  • testStrimziUpgrade in io.strimzi.systemtest.upgrade.OlmUpgradeST
  • testDowngradeOfKafkaConnectAndKafkaConnector in io.strimzi.systemtest.upgrade.StrimziDowngradeST

Re-run command:
@strimzi-ci run tests --profile=all --testcase=io.strimzi.systemtest.kafka.KafkaNodePoolST#testKafkaManagementTransferToAndFromKafkaNodePool,io.strimzi.systemtest.kafka.QuotasST#testKafkaQuotasPluginIntegration,io.strimzi.systemtest.log.LoggingChangeST#testDynamicallySetBridgeLoggingLevels,io.strimzi.systemtest.mirrormaker.MirrorMaker2ST#testStrimziIdentityReplicationPolicy,io.strimzi.systemtest.operators.CustomResourceStatusST,io.strimzi.systemtest.operators.NamespaceDeletionRecoveryST#testTopicNotAvailable,io.strimzi.systemtest.operators.user.UserScalabilityST#testCreateAndAlterBigAmountOfScramShaUsers,io.strimzi.systemtest.rollingupdate.RollingUpdateST#testClusterOperatorFinishAllRollingUpdates,io.strimzi.systemtest.security.SecurityST#testAutoReplaceAllCaKeysTriggeredByAnno,io.strimzi.systemtest.security.oauth.OauthAuthorizationST#testTeamBWriteToTopic,io.strimzi.systemtest.security.oauth.OauthPasswordGrantsST#testPasswordGrantsKafkaBridge,io.strimzi.systemtest.upgrade.KafkaUpgradeDowngradeST#testUpgradeWithNoMessageAndProtocolVersionsSet,io.strimzi.systemtest.upgrade.OlmUpgradeST#testStrimziUpgrade,io.strimzi.systemtest.upgrade.StrimziDowngradeST#testDowngradeOfKafkaConnectAndKafkaConnector

@henryZrncik
Copy link
Contributor Author

@im-konge @see-quick Apologise, given fail has nothing to do with new test; I simply referenced all tests (most of them failing due to timeout issues none of them caused by these changes)

@im-konge
Copy link
Member

/azp run regression

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@im-konge
Copy link
Member

@strimzi-ci run tests --profile=regression --install-type=bundle --cluster-type=ocp --cluster-version=4.14 --testcase=PodSecurityProfilesST

@strimzi-ci
Copy link

▶️ Build started - check Jenkins for more info. ▶️

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: regression
GROUPS:
TEST_CASE: PodSecurityProfilesST
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 242
OCP_VERSION: 4.14
BUILD_IMAGES: false
FIPS_ENABLED: false
PARALLEL_COUNT: 5
EXCLUDED_GROUPS: loadbalancer,nodeport,olm

@im-konge im-konge merged commit a35c6d2 into strimzi:main Sep 20, 2023
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

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

@jankalinic jankalinic jankalinic approved these changes

Assignees

@henryZrncik henryZrncik

Labels
System tests
Projects
None yet
Milestone
0.38.0
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@henryZrncik @strimzi-ci @im-konge @see-quick @jankalinic