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

[ST] Enable connector autoRestart test #9507

Merged
merged 1 commit into from Jan 7, 2024

Conversation

jankalinic
Copy link
Contributor

Type of change

  • Refactoring

Description

While going through STs i've encountered this test which seems to be working now as expected, so i've decided to enable the last checks which seemed to be flaky with current behaviour.

Checklist

  • Write tests
  • Make sure all tests pass
  • Check RBAC rights for Kubernetes / OpenShift roles
  • Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally

Signed-off-by: jkalinic <jkalinic@redhat.com>
@jankalinic jankalinic self-assigned this Jan 5, 2024
@jankalinic jankalinic added this to the 0.40.0 milestone Jan 5, 2024
@jankalinic
Copy link
Contributor Author

@strimzi-ci run tests --cluster-type=ocp --cluster-version=4.14 --profile=all --testcase=ConnectST#testConnectorTaskAutoRestart

@strimzi-ci
Copy link

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

@strimzi-ci
Copy link

✔️ Test Summary ✔️

TEST_PROFILE: all
GROUPS:
TEST_CASE: ConnectST#testConnectorTaskAutoRestart
TOTAL: 1
PASS: 1
FAIL: 0
SKIP: 0
BUILD_NUMBER: 307
OCP_VERSION: 4.14
BUILD_IMAGES: false
FIPS_ENABLED: false
PARALLEL_COUNT: 5
EXCLUDED_GROUPS: loadbalancer,nodeport,olm

@jankalinic jankalinic marked this pull request as ready for review January 5, 2024 10:35
@jankalinic jankalinic changed the title [ST]-[Do not merge] Enable connector autoRestart test [ST] Enable connector autoRestart test Jan 5, 2024
@jankalinic jankalinic requested review from im-konge and see-quick and removed request for im-konge January 5, 2024 10:35
@see-quick
Copy link
Member

/azp run regression

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

I'm fine with enabling the check if it works, but I'm really curious what was the cause that it failed all the time before

@jankalinic jankalinic added ready for merge Label for PRs which are ready for merge and removed do-not-merge labels Jan 6, 2024
@scholzj scholzj merged commit a611991 into strimzi:main Jan 7, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge Label for PRs which are ready for merge System tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants