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

Fix problem when odo deploy processes do not terminate #6716

Merged

Conversation

feloy
Copy link
Contributor

@feloy feloy commented Apr 4, 2023

What type of PR is this:

/kind bug
/area testing
What does this PR do / why we need it:

Which issue(s) this PR fixes:

Fixes #6715

PR acceptance criteria:

  • Unit test

  • Integration test

  • Documentation

How to test changes / Special notes to the reviewer:

@netlify
Copy link

netlify bot commented Apr 4, 2023

Deploy Preview for odo-docusaurus-preview canceled.

Name Link
🔨 Latest commit 00d2060
🔍 Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/642c3fb8613c1900086a7b87

@openshift-ci openshift-ci bot added kind/bug Categorizes issue or PR as related to a bug. area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering labels Apr 4, 2023
@openshift-ci openshift-ci bot requested review from anandrkskd and ritudes April 4, 2023 14:43
@feloy feloy requested review from rm3l and removed request for anandrkskd and ritudes April 4, 2023 14:43
@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

NoCluster Tests on commit 521771c finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

OpenShift Unauthenticated Tests on commit 521771c finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

Unit Tests on commit 521771c finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

Validate Tests on commit 521771c finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

Windows Tests (OCP) on commit 521771c finished with errors.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

OpenShift Tests on commit 521771c finished successfully.
View logs: TXT HTML

@feloy feloy force-pushed the tests/odo-deploy-no-stop branch from 81f9c57 to 00d2060 Compare April 4, 2023 15:18
@feloy feloy closed this Apr 4, 2023
@feloy feloy reopened this Apr 4, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

Kubernetes Tests on commit 521771c finished successfully.
View logs: TXT HTML

@odo-robot
Copy link

odo-robot bot commented Apr 4, 2023

Kubernetes Docs Tests on commit 8876e1a finished successfully.
View logs: TXT HTML

@@ -117,6 +117,7 @@ func (cw *CmdWrapper) ShouldRun() *CmdWrapper {
func (cw *CmdWrapper) Should(f func(session *gexec.Session)) {
cw.Runner()
f(cw.session)
cw.session.Wait()
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to fix the problem, logs point so.

Ran 436 of 788 Specs in 1079.759 seconds
SUCCESS! -- 436 Passed | 0 Failed | 0 Pending | 352 Skipped


Ginkgo ran 1 suite in 19m5.388476487s
Test Suite Passed

But I don't understand why this missing piece created a problem in the first place.

I also noticed that if you check this run on IBM cloud, the duration for Job run was Duration1 hour, 57 minutes, 19 seconds, but the test run duration was Duration23 minutes, 40 seconds, is that normal?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

But I don't understand why this missing piece created a problem in the first place.

The tests were terminating before the processes odo deploy were terminated. Ginkgo probably has problems handling children processes remaining after the tests finish.

I also noticed that if you check this run on IBM cloud, the duration for Job run was Duration1 hour, 57 minutes, 19 seconds, but the test run duration was Duration23 minutes, 40 seconds, is that normal?

The job waited 1h34 before to start.

@valaparthvi
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Apr 5, 2023
@valaparthvi
Copy link
Contributor

valaparthvi commented Apr 5, 2023

/override windows-integration-test/Windows-test

  [oc] ""Unable to connect to the server: dial tcp 158.177.79.34:31700: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
  [FAILED] in [BeforeEach] - C:/Users/Administrator.ANSIBLE-TEST-VS/3617/tests/helper/helper_oc.go:34 @ 04/04/23 12:20:15.12
  Deleting project: cmd-logs-test194pap

Summarizing 1 Failure:
  [FAIL] odo logs command tests when component is created and odo logs is executed when running in Deploy mode [BeforeEach] should successfully show logs of the running component
  C:/Users/Administrator.ANSIBLE-TEST-VS/3617/tests/helper/helper_oc.go:34

@openshift-ci
Copy link

openshift-ci bot commented Apr 5, 2023

@valaparthvi: Overrode contexts on behalf of valaparthvi: windows-integration-test/Windows-test

In response to this:

/override windows-integration-test/Windows-test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-merge-robot openshift-merge-robot merged commit 7aa3369 into redhat-developer:main Apr 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/testing Issues or PRs related to testing, Quality Assurance or Quality Engineering kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Integration test suite taking 30 minutes to terminate
3 participants