Details

    • Sprint:
      Castor Core S7, Pollux Platform S13
    • Story Points:
      4

      Description

      Generated image variations can't currently be purged as a whole in 5.4. Make it so that running php ezpublish/console liip:imagine:cache:remove --filters=large will remove all the variations for the large alias.

      A new naming scheme of variations should be implemented for newer versions, that makes it much more efficient to purge all variations of an alias.

        Issue Links

          Activity

          Jérôme Vieilledent (Inactive) created issue -
          Jérôme Vieilledent (Inactive) made changes -
          Field Original Value New Value
          Link This issue relates to EZP-22280 [ EZP-22280 ]
          André Rømcke made changes -
          Status Open [ 1 ] Confirmed [ 10037 ]
          André Rømcke made changes -
          Status Confirmed [ 10037 ] Backlog [ 10000 ]
          André Rømcke made changes -
          Rank Ranked higher
          André Rømcke made changes -
          Remaining Estimate 2 days [ 57600 ]
          André Rømcke made changes -
          Original Estimate 2 days [ 57600 ]
          André Rømcke made changes -
          Sprint Castor Core S7 [ 42 ]
          André Rømcke made changes -
          Rank Ranked lower
          Jérôme Vieilledent (Inactive) made changes -
          Link This issue blocks EZP-23369 [ EZP-23369 ]
          André Rømcke made changes -
          Rank Ranked higher
          Jérôme Vieilledent (Inactive) made changes -
          Labels image io candidate-for-next-sprint image io
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Bertrand Dunogier made changes -
          Original Estimate 2 days [ 57600 ] 3 days [ 86400 ]
          Remaining Estimate 2 days [ 57600 ] 3 days [ 86400 ]
          Jérôme Vieilledent (Inactive) made changes -
          Labels candidate-for-next-sprint image io image io y2015s12
          Bertrand Dunogier made changes -
          Labels image io y2015s12 image io s132015-candidate
          André Rømcke made changes -
          Issue Type Bug [ 1 ] Story [ 7 ]
          Story Points 4
          André Rømcke made changes -
          Sprint Castor Core S7 [ 42 ] Castor Core S7, Pollux Platform S13 [ 42, 72 ]
          André Rømcke made changes -
          Rank Ranked higher
          André Rømcke made changes -
          Affects Version/s 5.4.2 [ 13979 ]
          André Rømcke made changes -
          Fix Version/s 5.4.3 [ 14184 ]
          André Rømcke made changes -
          Fix Version/s 2015.05 [ 14187 ]
          Hide
          Bertrand Dunogier added a comment -

          We might have a bit of a problem. Flysystem can't really glob with a wildcard in the middle of a string.It is quite likely that this is because of the lack of this feature on some of the handled filesystems.

          While we have a metadata overlay, one of those metadata overlays uses Flysystem (the one we use for local operations). Maybe this is the root of the issue, but we can hardly go back now. One way would maybe be to have a dedicated service for this, where globbing is taken care of by different handlers.

          Moving the alias to a directory made some things easier, and other more complicated, did it not ?

          Show
          Bertrand Dunogier added a comment - We might have a bit of a problem. Flysystem can't really glob with a wildcard in the middle of a string.It is quite likely that this is because of the lack of this feature on some of the handled filesystems. While we have a metadata overlay, one of those metadata overlays uses Flysystem (the one we use for local operations). Maybe this is the root of the issue, but we can hardly go back now. One way would maybe be to have a dedicated service for this, where globbing is taken care of by different handlers. Moving the alias to a directory made some things easier, and other more complicated, did it not ?
          Bertrand Dunogier made changes -
          Status Backlog [ 10000 ] Specification [ 10002 ]
          André Rømcke made changes -
          Workflow eZ Engineering Scrumban Workflow [ 64546 ] EZ* Development Workflow [ 69797 ]
          Hide
          Bertrand Dunogier added a comment -
          Show
          Bertrand Dunogier added a comment - Pull-request for review: https://github.com/ezsystems/ezpublish-kernel/pull/1276 .
          Bertrand Dunogier made changes -
          Status Specification [ 10002 ] Specification done [ 10003 ]
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Bertrand Dunogier made changes -
          Status Specification done [ 10003 ] Development [ 3 ]
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Bertrand Dunogier made changes -
          Status Development [ 3 ] Development Review [ 10006 ]
          Bertrand Dunogier made changes -
          Status Development Review [ 10006 ] Development Review done [ 10028 ]
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Bertrand Dunogier made changes -
          Summary IOService: Implement a find() method Implement image variations purging
          Hide
          Bertrand Dunogier added a comment -

          Merged to master.

          Show
          Bertrand Dunogier added a comment - Merged to master.
          Bertrand Dunogier made changes -
          Description Implement a {{find()}} method in IOService, accepting glob wildcards.
          This way, it will be easy to cleanup image aliases:

          {code}
          $aliases = $ioService->find( 'path/to/folder/my_image_*' );
          $otherAliases = $ioService->find( 'some/folder/*/foo/bar/image.jpg' );
          {code}
          Generated image variations can't currently be purged as a whole in 5.4. Make it so that running {{php ezpublish/console liip:imagine:cache:remove --filters=large}} will remove all the variations for the large alias.

          A new naming scheme of variations should be implemented for newer versions, that makes it much more efficient to purge all variations of an alias.
          André Rømcke made changes -
          Rank Ranked higher
          Sarah Haïm-Lubczanski (Inactive) made changes -
          Comment [ A comment with security level 'Employees' was removed. ]
          Bertrand Dunogier made changes -
          Remote Link This issue links to "Page (eZ Documentation)" [ 15215 ]
          André Rømcke made changes -
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Ricardo Correia (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Bertrand Dunogier made changes -
          Remote Link This issue links to "PR ezsystems/ezpublish-kernel#1276 (Web Link)" [ 15633 ]
          Show
          Bertrand Dunogier added a comment - Added doc on https://doc.ez.no/display/EZP/Image+alias+generation#Imagealiasgeneration-Purgingaliases , what do you think ?
          Bertrand Dunogier made changes -
          Status Development Review done [ 10028 ] Documentation [ 10010 ]
          Ricardo Correia (Inactive) made changes -
          Link This issue relates to CS-4889 [ CS-4889 ]
          Ricardo Correia (Inactive) made changes -
          Link This issue relates to CS-5087 [ CS-5087 ]
          Ricardo Correia (Inactive) made changes -
          Link This issue relates to CS-4903 [ CS-4903 ]
          Hide
          Bertrand Dunogier added a comment -

          I'm actually wondering if all of those aren't because of this: https://github.com/ezsystems/ezpublish-kernel/pull/1438.

          Could you try the PR, and let me know if it changes the situation ?

          Show
          Bertrand Dunogier added a comment - I'm actually wondering if all of those aren't because of this: https://github.com/ezsystems/ezpublish-kernel/pull/1438 . Could you try the PR, and let me know if it changes the situation ?
          Hide
          Bertrand Dunogier added a comment - - edited

          @QA / 2nd line: creating a separate issue for this, as it was a regression that was released: EZP-25019.

          Show
          Bertrand Dunogier added a comment - - edited @QA / 2nd line: creating a separate issue for this, as it was a regression that was released: EZP-25019 .
          Bertrand Dunogier made changes -
          Link This issue relates to EZP-25019 [ EZP-25019 ]
          Nuno Oliveira (Inactive) made changes -
          Link This issue relates to CS-5211 [ CS-5211 ]
          Yannick Roger (Inactive) made changes -
          Status Documentation [ 10010 ] Documentation Review done [ 10011 ]
          Assignee Bertrand Dunogier [ bertrand.dunogier@ez.no ]
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Rui Silva (Inactive) made changes -
          Status Documentation Review done [ 10011 ] QA [ 10008 ]
          Hide
          Rui Silva (Inactive) added a comment -

          Tested and approved by QA for 5.4 and master.

          Show
          Rui Silva (Inactive) added a comment - Tested and approved by QA for 5.4 and master.
          Rui Silva (Inactive) made changes -
          Assignee Rui Silva [ rui.silva@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Nuno Oliveira (Inactive) made changes -
          Link This issue testing discovered EZP-25185 [ EZP-25185 ]
          Nuno Oliveira (Inactive) made changes -
          Link This issue testing discovered EZP-25185 [ EZP-25185 ]
          Nuno Oliveira (Inactive) made changes -
          Link This issue relates to EZP-25185 [ EZP-25185 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 69797 ] EZEE Development Workflow [ 124180 ]
          Alex Schuster made changes -
          Workflow EZEE Development Workflow [ 124180 ] EZEE and EZP Story Workflow [ 127795 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          2h 37m 1 André Rømcke 24/Sep/14 4:38 PM
          Confirmed Confirmed Backlog Backlog
          3s 1 André Rømcke 24/Sep/14 4:38 PM
          Backlog Backlog Specification Specification
          244d 23h 40m 1 Bertrand Dunogier 27/May/15 4:19 PM
          Specification Specification Specification Done Specification Done
          4d 23h 28m 1 Bertrand Dunogier 01/Jun/15 3:48 PM
          Specification Done Specification Done Development Development
          3s 1 Bertrand Dunogier 01/Jun/15 3:48 PM
          Development Development Development Review Development Review
          3s 1 Bertrand Dunogier 01/Jun/15 3:48 PM
          Development Review Development Review Development Review done Development Review done
          1d 5h 41m 1 Bertrand Dunogier 02/Jun/15 9:29 PM
          Development Review done Development Review done Documentation Documentation
          111d 22h 3m 1 Bertrand Dunogier 22/Sep/15 7:32 PM
          Documentation Documentation Documentation Review done Documentation Review done
          57d 18h 33m 1 yannick.roger@ez.no 19/Nov/15 1:06 PM
          Documentation Review done Documentation Review done QA QA
          3d 19h 50m 1 rui.silva@ez.no 23/Nov/15 8:56 AM
          QA QA Closed Closed
          7h 29m 1 rui.silva@ez.no 23/Nov/15 4:26 PM

            People

            • Assignee:
              Unassigned
              Reporter:
              Jérôme Vieilledent (Inactive)
            • Votes:
              2 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 3 days
                3d
                Remaining:
                Remaining Estimate - 3 days
                3d
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Agile