Uploaded image for project: 'Libcloud'
  1. Libcloud
  2. LIBCLOUD-373

Add last_modified in S3 iterate_container_objects

        Details

        • Type: Improvement
        • Status: Resolved
        • Priority: Trivial
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 0.13.2
        • Component/s: Storage
        • Labels:
          None

          Description

          Expose the last_modified field for objects in iterate_container_objects.

          1. Text File
            last_modified.patch
            1 kB
            Stefan Friesel
          2. Text File
            last_modified2.patch
            2 kB
            Stefan Friesel

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            sfriesel Stefan Friesel added a comment -

            https://github.com/sfriesel/libcloud/compare/last_modified

            Show
            sfriesel Stefan Friesel added a comment - https://github.com/sfriesel/libcloud/compare/last_modified
            Hide
            Permalink
            kami Tomaz Muraus added a comment -

            Thanks, the patch looks good.

            Can you please also add a test case?

            Show
            kami Tomaz Muraus added a comment - Thanks, the patch looks good. Can you please also add a test case?
            Hide
            Permalink
            sfriesel Stefan Friesel added a comment - - edited

            I just added the assertion in the existing list_container_objects test, because the field is always present. Github branch updated.

            Show
            sfriesel Stefan Friesel added a comment - - edited I just added the assertion in the existing list_container_objects test, because the field is always present. Github branch updated.
            Hide
            Permalink
            kami Tomaz Muraus added a comment -

            Thanks, I've merged patch into trunk.

            Next time please use git format-patch so we can preserve the commit author.

            Show
            kami Tomaz Muraus added a comment - Thanks, I've merged patch into trunk. Next time please use git format-patch so we can preserve the commit author.

              People

              • Assignee:
                Unassigned
                Reporter:
                sfriesel Stefan Friesel
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development