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

Update Tomcat to 9.0.67 #7093

Merged
merged 1 commit into from
Oct 4, 2022
Merged

Update Tomcat to 9.0.67 #7093

merged 1 commit into from
Oct 4, 2022

Conversation

zrhoffman
Copy link
Member

@zrhoffman zrhoffman commented Sep 28, 2022

This PR fixes #7092 by updating Tomcat to version 9.0.67.


Which Traffic Control components are affected by this PR?

  • Traffic Router

What is the best way to verify this PR?

Run TR tests

If this is a bugfix, which Traffic Control versions contained the bug?

  • 7.0.1
  • 6.1.0

PR submission checklist

@zrhoffman zrhoffman added Traffic Router related to Traffic Router high impact impacts the basic function, deployment, or operation of a CDN improvement The functionality exists but it could be improved in some way. dependencies Pull requests that update a dependency file labels Sep 28, 2022
@zrhoffman zrhoffman changed the title Update Tomcat to 9.0.66 Update Tomcat to 9.0.67 Sep 28, 2022
Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

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

Code changes look good but I haven't ran TR tests.

Copy link
Contributor

@rimashah25 rimashah25 left a comment

Choose a reason for hiding this comment

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

Tests (TR) pass on local.

[INFO] --- maven-failsafe-plugin:2.19:verify (integration) @ ROOT ---
[INFO] Failsafe report directory: /Development/go/src/github.com/apache/trafficcontrol/traffic_router/core/target/failsafe-reports
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for traffic_router 7.1.0:
[INFO] 
[INFO] traffic_router ..................................... SUCCESS [  0.149 s]
[INFO] traffic_router_shared .............................. SUCCESS [  9.101 s]
[INFO] traffic_router_connector ........................... SUCCESS [ 10.599 s]
[INFO] traffic_router_config .............................. SUCCESS [  0.057 s]
[INFO] traffic_router_geolocation ......................... SUCCESS [  0.454 s]
[INFO] ROOT ............................................... SUCCESS [05:51 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  06:12 min
[INFO] Finished at: 2022-10-03T18:55:27-06:00
[INFO] ------------------------------------------------------------------------```

@ocket8888 ocket8888 merged commit 96a8543 into apache:master Oct 4, 2022
@zrhoffman zrhoffman deleted the tomcat-9.0.66 branch October 4, 2022 20:20
zrhoffman added a commit that referenced this pull request Oct 14, 2022
zrhoffman added a commit to zrhoffman/trafficcontrol that referenced this pull request Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file high impact impacts the basic function, deployment, or operation of a CDN improvement The functionality exists but it could be improved in some way. Traffic Router related to Traffic Router
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade Tomcat in 6.1.x, 7.0.x, and master
3 participants