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

Since #971 the changesets minutely state.yaml is outdated #984

Closed
SomeoneElseOSM opened this issue Oct 24, 2023 · 4 comments
Closed

Since #971 the changesets minutely state.yaml is outdated #984

SomeoneElseOSM opened this issue Oct 24, 2023 · 4 comments

Comments

@SomeoneElseOSM
Copy link

SomeoneElseOSM commented Oct 24, 2023

#971 is ("Enable Redirects for planet.osm.org replication diffs"),

https://planet.osm.org/replication/changesets/state.yaml

which redirects to

https://osm-planet-eu-central-1.s3.dualstack.eu-central-1.amazonaws.com/changesets/replication/minute/state.yaml

contains

---
last_run: 2023-09-24 15:53:31.626863000 +00:00
sequence: 5680235

If it's relevant, I'm in the UK and the last "correct" one is from "2023-10-23 08:20:01.195511" this morning. I'm using a Firefox private window to obtain the above, so I don't think it's a "not following redirects" issue.

@Firefishy
Copy link
Member

Fixed in openstreetmap/chef@8e2b5b8

@Firefishy
Copy link
Member

Thank you for spotting and reporting it.

@tomhughes
Copy link
Member

It's probably better to publish the global state after the new data is fully published rather than in the middle so I've done that in openstreetmap/chef@cfaa167.

@Firefishy
Copy link
Member

I thought about that, but I just copied the order that the script does it to the filesystem. Yeah, likely both orders should be changed.

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

3 participants