Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

fixed an issue where TO integration tests hang#3938

Merged
jhg03a merged 1 commit into
apache:masterfrom
ocket8888:test-fix
Sep 29, 2019
Merged

fixed an issue where TO integration tests hang#3938
jhg03a merged 1 commit into
apache:masterfrom
ocket8888:test-fix

Conversation

@ocket8888
Copy link
Copy Markdown
Contributor

What does this PR (Pull Request) do?

  • This PR is not related to any Issue

Fixes an issue where Traffic Ops integration tests would hang for 8000 seconds.

Which Traffic Control components are affected by this PR?

  • Traffic Ops integration tests

What is the best way to verify this PR?

Run the integration tests, verify the container doesn't hang open for 8000 seconds

If this is a bug fix, what versions of Traffic Control are affected?

  • master

The following criteria are ALL met by this PR

  • This PR includes tests OR I have explained why tests are unnecessary
  • This PR includes documentation OR I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update is not necessary
  • This PR includes any and all required license headers
  • This PR ensures that database migration sequence is correct OR this PR does not include a database migration
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

@ocket8888 ocket8888 added bug something isn't working as intended tests related to tests and/or testing infrastructure labels Sep 27, 2019
@asf-ci
Copy link
Copy Markdown
Contributor

asf-ci commented Sep 27, 2019

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4327/
Test FAILed.

@jhg03a jhg03a self-assigned this Sep 29, 2019
@jhg03a jhg03a merged commit 0b538c6 into apache:master Sep 29, 2019
@ocket8888 ocket8888 deleted the test-fix branch September 30, 2019 11:08
@mitchell852 mitchell852 added the Traffic Ops related to Traffic Ops label Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug something isn't working as intended tests related to tests and/or testing infrastructure Traffic Ops related to Traffic Ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants