Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-23254

DFS - tmp files left if the file size is not identical to the original one

    Details

      Description

      EZP-21825 fix some of the temporary files left in the file system if there's a failure in the copy/rename process between the local and the NFS file systems.

      But, if after the copy the file size of the original and the copied file are different, the tempo files will be left undeleted.

      Steps to Reproduce

      This is a hard issue to reproduce since it will only happen if there are any NFS errors, but it's possible to achieve the same reproduce.
      Edit

      + if( $loopCount != $this->maxCopyTries )
      +{
      +   $dfsFileSize += 10;
      +   $localFileSize -= 10;  
      +}
      $localFileSize = filesize( $filePath );
      if ( $dfsFileSize == $localFileSize )
      {
      	return $filePath;
      }
      

        Issue Links

          Activity

          Eduardo Fernandes (Inactive) created issue -
          Eduardo Fernandes (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to EZP-21825 [ EZP-21825 ]
          Eduardo Fernandes (Inactive) made changes -
          Status Open [ 1 ] Confirmed [ 10037 ]
          Yannick Roger (Inactive) made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Yannick Roger (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ]
          Yannick Roger (Inactive) made changes -
          Summary Clustered NFS environment may leave temporary files if the file size is not identical to the original one DFS - tmp files left if the file size is not identical to the original one
          Show
          Yannick Roger (Inactive) added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1059
          Yannick Roger (Inactive) made changes -
          Remote Link This issue links to "PR (Web Link)" [ 14256 ]
          Yannick Roger (Inactive) made changes -
          Status Development [ 3 ] Development review [ 10006 ]
          Show
          Yannick Roger (Inactive) added a comment - - edited Fixed in master : https://github.com/ezsystems/ezpublish-legacy/commit/bac700aa148188b77d6b02dc4212b96daa990e3a
          Yannick Roger (Inactive) made changes -
          Status Development review [ 10006 ] Development Review done [ 10028 ]
          Affects Version/s 2014.07 [ 13481 ]
          Fix Version/s 5.4 [ 13180 ]
          Fix Version/s 2014.09 [ 13681 ]
          Yannick Roger (Inactive) made changes -
          Status Development Review done [ 10028 ] Documentation done [ 10011 ]
          Paulo Nunes (Inactive) made changes -
          Status Documentation done [ 10011 ] QA [ 10008 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ] Paulo Nunes [ paulo.nunes@ez.no ]
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Tested on master.
          tc-2232

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved Tested on master. tc-2232
          Paulo Nunes (Inactive) made changes -
          Assignee Paulo Nunes [ paulo.nunes@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Yannick Roger (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ]
          Yannick Roger (Inactive) made changes -
          Status Reopened [ 4 ] InputQ [ 10001 ]
          Yannick Roger (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Yannick Roger (Inactive) made changes -
          Affects Version/s 5.3.2 [ 13483 ]
          Affects Version/s 5.2 [ 12582 ]
          Yannick Roger (Inactive) made changes -
          Fix Version/s 5.2 Maintenance [ 12782 ]
          Fix Version/s 5.3.3 [ 13484 ]
          Yannick Roger (Inactive) made changes -
          Status Development [ 3 ] Documentation done [ 10011 ]
          Paulo Nunes (Inactive) made changes -
          Status Documentation done [ 10011 ] QA [ 10008 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ] Paulo Nunes [ paulo.nunes@ez.no ]
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Globally tested on master, 5.3.3 and in 5.2

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved Globally tested on master, 5.3.3 and in 5.2
          Paulo Nunes (Inactive) made changes -
          Assignee Paulo Nunes [ paulo.nunes@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Hide
          Eduardo Fernandes (Inactive) added a comment -

          Reopening for backport to eZP 5.1

          Show
          Eduardo Fernandes (Inactive) added a comment - Reopening for backport to eZP 5.1
          Eduardo Fernandes (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Yannick Roger (Inactive) made changes -
          Status Reopened [ 4 ] InputQ [ 10001 ]
          Yannick Roger (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ]
          Yannick Roger (Inactive) made changes -
          Status Development [ 3 ] Documentation Review done [ 10011 ]
          Affects Version/s 5.1 [ 11280 ]
          Fix Version/s 5.1 Maintenance [ 12301 ]
          Eduardo Fernandes (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Miguel das Neves Jacinto (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ] Miguel das Neves Jacinto [ miguel.jacinto@ez.no ]
          Hide
          Miguel das Neves Jacinto (Inactive) added a comment -

          QA approved on 5.1

          Show
          Miguel das Neves Jacinto (Inactive) added a comment - QA approved on 5.1
          Miguel das Neves Jacinto (Inactive) made changes -
          Assignee Miguel das Neves Jacinto [ miguel.jacinto@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          André Rømcke made changes -
          Workflow eZ Engineering Scrumban Workflow [ 64173 ] EZ* Development Workflow [ 85286 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 85286 ] EZEE Development Workflow [ 124089 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          2m 4s 1 eduardo.fernandes@ez.no 11/Aug/14 3:10 PM
          Confirmed Confirmed InputQ InputQ
          19h 43m 1 yannick.roger@ez.no 12/Aug/14 10:53 AM
          Development Development Development Review Development Review
          6h 38m 1 yannick.roger@ez.no 12/Aug/14 5:32 PM
          Development Review Development Review Development Review done Development Review done
          1d 21h 46m 1 yannick.roger@ez.no 14/Aug/14 3:19 PM
          Development Review done Development Review done Documentation Review done Documentation Review done
          8s 1 yannick.roger@ez.no 14/Aug/14 3:19 PM
          Closed Closed Reopened Reopened
          253d 23h 17m 2 eduardo.fernandes@ez.no 30/Apr/15 4:53 PM
          Reopened Reopened InputQ InputQ
          1m 8s 2 yannick.roger@ez.no 30/Apr/15 4:53 PM
          InputQ InputQ Development Development
          9s 3 yannick.roger@ez.no 30/Apr/15 4:53 PM
          Development Development Documentation Review done Documentation Review done
          4m 47s 2 yannick.roger@ez.no 30/Apr/15 4:57 PM
          Documentation Review done Documentation Review done QA QA
          3d 19h 2m 3 miguel.jacinto@ez.no 30/Apr/15 5:59 PM
          QA QA Closed Closed
          5d 7h 41m 3 miguel.jacinto@ez.no 04/May/15 5:26 PM

            People

            • Assignee:
              Unassigned
              Reporter:
              Eduardo Fernandes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: