Search squid archive

Re: storeurl_rewrite and ICP

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday 24 December 2008 17:01:39 Adrian Chadd wrote:
> Thanks. Be sure to comment on the bugzilla ticket too.
>
> Oh and tell me which bug it is so I can make sure I'm watching it. :)
>
>
> Adrian
>
> 2008/12/23 Imri Zvik <imriz@xxxxxxxxxx>:
> > On Sunday 21 December 2008 10:52:42 Imri Zvik wrote:
> >> Hi,
> >>
> >> On Thursday 18 December 2008 21:57:22 Adrian Chadd wrote:
> >> > Nope, I don't think the storeurl-rewriter stuff was ever integrated
> >> > into ICP.
> >> >
> >> > I think someone posted a patch to the squid bugzilla to implement
> >> > this.
> >>
> >> If you can point me to said patch, I'd be happy to test it under load.
> >>
> >> > I'm happy to commit whatever people sensibly code up and deploy. :)
> >> >
> >> >
> >> >
> >> > Adrian
> >> >
> >> > 2008/12/18 Imri Zvik <imriz@xxxxxxxxxx>:
> >> > > Hi,
> >> > >
> >> > > I'm using the storeurl_rewrite feature to store content with
> >> > > changing attributes.
> >> > >
> >> > > As my traffic grows, I want to be able to add cache_peers to share
> >> > > the load.
> >> > >
> >> > > After configuring the peers, I've found out that all my ICP queries
> >> > > results with misses.
> >> > > It seems like the storeurl_rewrite logic is not implemented in the
> >> > > ICP queries - i.e., nor the ICP client or the server passes the URL
> >> > > through the storeurl_rewrite process before checking if the
> >> > > requested content is cached or not.
> >> > >
> >> > > Am I missing something?
> >> > >
> >> > >
> >> > >
> >> > > Thank you in advance,
> >>
> >> Thanks!
> >
> > I've found the said patch in squid's bugzilla - It seems to be working,
> > but I'm going to test the patch under load (700 mbit~) and report back.

Here is the bug report: http://www.squid-cache.org/bugs/show_bug.cgi?id=2354

The patch works flawlessly so far.


[Index of Archives]     [Linux Audio Users]     [Samba]     [Big List of Linux Books]     [Linux USB]     [Yosemite News]

  Powered by Linux