[trafficcontrol] branch master updated: Fix ORT to reload traffic server when lua scripts are added or changed. (#5574)

[rob]

This is an automated email from the ASF dual-hosted git repository.

rob pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new a289d9c  Fix ORT to reload traffic server when lua scripts are added 
or changed.   (#5574)
a289d9c is described below

commit a289d9ce008df420b41f4b3ac488e094860b10f0
Author: Joe Pappano <joseph_papp...@cable.comcast.com>
AuthorDate: Thu Mar 18 17:48:02 2021 -0400

    Fix ORT to reload traffic server when lua scripts are added or changed.   
(#5574)
    
    * Updated process_reload_restarts to include .lua files.
    
    * line 1337 removed exclusion for .lua files. so remap.config will be 
touched if lua files are changed.
    
    * Added LUA fix to change log
---
 CHANGELOG.md                       | 2 ++
 traffic_ops_ort/traffic_ops_ort.pl | 6 +++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e63c73f..9263412 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -68,6 +68,8 @@ The format is based on [Keep a 
Changelog](http://keepachangelog.com/en/1.0.0/).
 - Added license files to the RPMs
 
 ### Fixed
+- [#5288](https://github.com/apache/trafficcontrol/issues/5288) - Fixed the 
ability to create and update a server with MTU value >= 1280.
+- [#1624](https://github.com/apache/trafficcontrol/issues/1624) - Fixed ORT to 
reload Traffic Server if LUA scripts are added or changed.
 - [#5445](https://github.com/apache/trafficcontrol/issues/5445) - When 
updating a registered user, ignore updates on registration_sent field.
 - [#5335](https://github.com/apache/trafficcontrol/issues/5335) - Don't create 
a change log entry if the delivery service primary origin hasn't changed
 - [#5333](https://github.com/apache/trafficcontrol/issues/5333) - Don't create 
a change log entry for any delivery service consistent hash query params updates
diff --git a/traffic_ops_ort/traffic_ops_ort.pl 
b/traffic_ops_ort/traffic_ops_ort.pl
index d1ed816..e784f02 100755
--- a/traffic_ops_ort/traffic_ops_ort.pl
+++ b/traffic_ops_ort/traffic_ops_ort.pl
@@ -1342,7 +1342,7 @@ sub check_plugins {
                                                ( my @parts ) = split( /\//, 
$plugin_config_file );
                                                $plugin_config_file = 
$parts[$#parts];
                                                $plugin_config_file =~ s/\s+//g;
-                                               if ( 
!exists($cfg_file_tracker->{$plugin_config_file}->{'remap_plugin_config_file'} 
) && $plugin_config_file !~ /.lua$/ ) {
+                                               if ( 
!exists($cfg_file_tracker->{$plugin_config_file}->{'remap_plugin_config_file'} 
)) {
                                                        
$cfg_file_tracker->{$plugin_config_file}->{'remap_plugin_config_file'} = 1;
                                                }
                                        }
@@ -1461,6 +1461,10 @@ sub process_reload_restarts {
                ( $log_level >> $DEBUG ) && print "DEBUG New/changed header 
rewrite rule, installed in: $cfg_file. Later I will attempt to touch 
remap.config.\n";
                $traffic_ctl_needed++;
        }
+       elsif ( $cfg_file =~ m/(.*)\.lua/ ) {
+               ( $log_level >> $DEBUG ) && print "DEBUG New/changed lua 
script, installed in: $cfg_file. touch remap.config, and traffic_ctl config 
reload needed.\n";
+               $traffic_ctl_needed++;
+       }
        elsif ( $cfg_file eq "plugin.config" || $cfg_file eq "50-ats.rules" ) {
                ( $log_level >> $DEBUG ) && print "DEBUG $cfg_file changed, 
trafficserver restart needed.\n";
                $trafficserver_restart_needed++;