You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by "Dewayne Richardson (JIRA)" <ji...@apache.org> on 2017/05/15 19:28:04 UTC

[jira] [Created] (TC-291) GenIso can't handle concurrent requests

Dewayne Richardson created TC-291:
-------------------------------------

             Summary: GenIso can't handle concurrent requests 
                 Key: TC-291
                 URL: https://issues.apache.org/jira/browse/TC-291
             Project: Traffic Control
          Issue Type: Bug
          Components: Traffic Ops
    Affects Versions: 2.0.0
            Reporter: Dewayne Richardson


Because GenIso always changes the same files (no temporary directories created), it can't handle concurrent requests. We sometimes send commands to create 10 Edge Caches at the same time and we found out that the wrong IP address shows up on servers.

I've got the code to fix it up (but need to sign our paperwork). But our GenIso.pm now deviates from what's in Traffic Ops because we wanted to keep the 1.0.0 functionality (don't use the install_cfg directory, keep using the simple network line).

Suggestion: We should have a parameter to call different ISO generator scripts. Seems like this is implementation specific and not really a core function of Traffic Ops.

Basic changes :

use File::Temp;
use File::Copy::Recursive;

my $tmpdir = File::Temp::tempdir(CLEANUP => 1);
...
GenIso.pm:        my $tmp = File::Copy::Recursive::dircopy($dir,$tmpdir);
GenIso.pm:  my $cfg_dir = "$tmpdir/$install_cfg";
GenIso.pm:  print STUF "Dir== $tmpdir\n";
GenIso.pm:  my $cmd = "mkisofs -joliet-long -input-charset utf-8 -b isolinux/isolinux.bin -c isolinux/boot.cat -no-emul-boot -boot-load-size 4 -boot-info-table -R -J -v -T $tmpdir";
GenIso.pm:  open(IN, "<$tmpdir/ks.src") || die("$tmpdir/ks.src:$!");
GenIso.pm:  open (OUT, ">$tmpdir/ks.cfg") || die("$tmpdir/ks.cfg:$!");
GenIso.pm:        File::Path::rmtree $tmpdir;



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)