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

Copy v3-client API tests to the TO API tests Docker container now that the client directory is a symlink#4983

Merged
mitchell852 merged 1 commit intoapache:masterfrom
zrhoffman:copy-v3-client
Aug 21, 2020
Merged

Copy v3-client API tests to the TO API tests Docker container now that the client directory is a symlink#4983
mitchell852 merged 1 commit intoapache:masterfrom
zrhoffman:copy-v3-client

Conversation

@zrhoffman
Copy link
Member

What does this PR (Pull Request) do?

#4968 made traffic_ops/client a symlink. Because the CiaB traffic_ops_integration_test Dockerfile copies in traffic_ops/client but not traffic_ops/client-v3, it fails to build the v3 API test.

This PR copies traffic_ops/client-v3 in the Dockerfile.

  • This PR is not related to any Issue

Which Traffic Control components are affected by this PR?

  • CDN in a Box
  • Traffic Ops

What is the best way to verify this PR?

Run the API tests in CDN-in-a-Box, verify they pass:

docker-compose up --no-deps -d dns db smtp trafficops-perl trafficops
docker-compose -f docker-compose.traffic-ops-test.yml up --build integration

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

The following criteria are ALL met by this PR

  • This PR fixes tests
  • Intended behavior unchanged, documentation unnecessary
  • An update to CHANGELOG.md is not necessary
  • This PR includes any and all required license headers
  • 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)

Additional Information

@mitchell852 mitchell852 added cdn-in-a-box related to the Docker-based CDN-in-a-Box system Traffic Ops related to Traffic Ops tests related to tests and/or testing infrastructure labels Aug 21, 2020
@mitchell852 mitchell852 merged commit f2c0d3e into apache:master Aug 21, 2020
@zrhoffman zrhoffman deleted the copy-v3-client branch October 4, 2020 08:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cdn-in-a-box related to the Docker-based CDN-in-a-Box system 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.

2 participants