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

Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI#3255

Merged
mitchell852 merged 1 commit intomasterfrom
unknown repository
Jan 23, 2019
Merged

Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI#3255
mitchell852 merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jan 23, 2019

What does this PR do?

Fix Unit Test for PR (#3211) - ADDSSLKeys API - Remove mandatory SKI/AKI

Fixes PR #3211 Unit Test

Which TC components are affected by this PR?

  • Documentation
  • Grove
  • Traffic Analytics
  • Traffic Monitor
  • Traffic Ops
  • Traffic Ops ORT
  • Traffic Portal
  • Traffic Router
  • Traffic Stats
  • Traffic Vault
  • Other _________

What is the best way to verify this PR?

  1. Build and run unit tests for traffic_ops

Check all that apply

  • This PR includes tests
  • This PR includes documentation updates
  • This PR includes an update to CHANGELOG.md
  • This PR includes all required license headers
  • This PR includes a database migration (ensure that migration sequence is correct)
  • This PR fixes a serious security flaw. Read more: www.apache.org/security

Test has been updated to reflect that rootCA is included in certificate chain output.
@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops tests related to tests and/or testing infrastructure labels Jan 23, 2019
@mitchell852 mitchell852 merged commit e9a9fd0 into apache:master Jan 23, 2019
@asfgit
Copy link
Copy Markdown
Contributor

asfgit commented Jan 23, 2019

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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