Skip to content

Update to latest MC::Role so we get stale-while-revalidate header #1377

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ranguard
Copy link
Member

No description provided.

@oalders
Copy link
Member

oalders commented Jun 14, 2025

I am running the snapshot updater on this branch, which should reduce the number of lines changed, but I see that action has been failing for a while due to Search::Elasticsearch, so we may need to figure that out before deploy this branch.

@oalders
Copy link
Member

oalders commented Jun 14, 2025

I think this is the issue: elastic/elasticsearch-perl#240

@oalders
Copy link
Member

oalders commented Jun 14, 2025

@haarg any opinion on what we should pin to?

@ranguard
Copy link
Member Author

@oalders I'll leave this branch with you to co-ordinate then - feel free to push to it :)

@oalders oalders force-pushed the leo/update_mc_role branch 2 times, most recently from f16d880 to 935a34b Compare June 15, 2025 02:38
@oalders
Copy link
Member

oalders commented Jun 15, 2025

After pinning to 8.00 it gets stuck on

2025-06-15T02:43:08,2596,Search-Elasticsearch-Client-7_0-8.00| make: *** No rule to make target 'lib/Search/Elasticsearch/Client/7_0/Direct/SearchableSnapshots', needed by 'pm_to_blib'.  Stop.

@haarg haarg force-pushed the leo/update_mc_role branch from d45d44d to 6fe4d73 Compare June 17, 2025 04:47
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

Successfully merging this pull request may close these issues.

2 participants