Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Head last modified time does not match list last modified time #337

Closed
georgebrightinteractive opened this issue Apr 2, 2014 · 1 comment

Comments

@georgebrightinteractive
Copy link
Contributor

Hi,

According to:
https://review.openstack.org/#/c/55488/
head last modified is always a rounded up (in seconds) time now.

The method pyrax.cf_wrapper.client._convert_list_last_modified_to_local rounds the timestamp down, meaning that now there is a 1 second difference between the last modified time for the same item when getting it from a list vs getting it's head data.

eg

original last modified in head Wed, 02 Apr 2014 11:31:34 GMT
last modified in head 2014-04-02T12:31:34
original last modified in head Wed, 02 Apr 2014 11:31:34 GMT
last modified in head 2014-04-02T12:31:34
original last modified in list 2014-04-02T11:31:33.288830 UTC
last modified in list 2014-04-02T12:31:33

@EdLeafe
Copy link
Contributor

EdLeafe commented Apr 2, 2014

That method uses the datetime module to convert the string representation to a datetime value, and that seems to be the source of the "round down".

Would you be able to create a pull request that addresses this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants