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
Added SMTP configuration and APIInfo Email method #3925
Conversation
Refer to this link for build results (access rights to CI server needed): |
24e4778
to
c84aa2c
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does https://traffic-control-cdn.readthedocs.io/en/latest/admin/traffic_ops/configuration.html need to be updated to include the new configuration option?
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
What does this PR (Pull Request) do?
Which Traffic Control components are affected by this PR?
The configuration file is undocumented. It shouldn't be, but doing so is outside the scope of this PR.
What is the best way to verify this PR?
Create a configuration file with a valid SMTP server configuration and try to start Traffic Ops. Config should parse fine.
The following criteria are ALL met by this PR