[jira] [Commented] (TS-1006) memory management, cut down memory waste ?

2013-03-27 Thread jaekyung oh (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13614983#comment-13614983
 ] 

jaekyung oh commented on TS-1006:
-

yes. with last final patch i always put --enable-reclaimable-freelist=yes

same error happens on both 3.2.0 and 3.2.4

 memory management, cut down memory waste ?
 --

 Key: TS-1006
 URL: https://issues.apache.org/jira/browse/TS-1006
 Project: Traffic Server
  Issue Type: Improvement
  Components: Core
Affects Versions: 3.1.1
Reporter: Zhao Yongming
Assignee: Bin Chen
 Fix For: 3.3.3

 Attachments: 
 0001-TS-1006-Add-an-enable-reclaimable-freelist-option.patch, 
 0002-TS-1006-Add-a-new-wrapper-ink_atomic_decrement.patch, 
 0003-TS-1006-Introduce-a-reclaimable-InkFreeList.patch, 
 0004-TS-1006-Make-InkFreeList-memory-pool-configurable.patch, 
 Memory-Usage-After-Introduced-New-Allocator.png, memusage.ods, memusage.ods


 when we review the memory usage in the production, there is something 
 abnormal, ie, looks like TS take much memory than index data + common system 
 waste, and here is some memory dump result by set 
 proxy.config.dump_mem_info_frequency
 1, the one on a not so busy forwarding system:
 physics memory: 32G
 RAM cache: 22G
 DISK: 6140 GB
 average_object_size 64000
 {code}
  allocated  |in-use  | type size  |   free list name
 |||--
   671088640 |   37748736 |2097152 | 
 memory/ioBufAllocator[14]
  2248146944 | 2135949312 |1048576 | 
 memory/ioBufAllocator[13]
  1711276032 | 1705508864 | 524288 | 
 memory/ioBufAllocator[12]
  1669332992 | 1667760128 | 262144 | 
 memory/ioBufAllocator[11]
  2214592512 | 221184 | 131072 | 
 memory/ioBufAllocator[10]
  2325741568 | 2323775488 |  65536 | 
 memory/ioBufAllocator[9]
  2091909120 | 2089123840 |  32768 | 
 memory/ioBufAllocator[8]
  1956642816 | 1956478976 |  16384 | 
 memory/ioBufAllocator[7]
  2094530560 | 2094071808 |   8192 | 
 memory/ioBufAllocator[6]
   356515840 |  355540992 |   4096 | 
 memory/ioBufAllocator[5]
 1048576 |  14336 |   2048 | 
 memory/ioBufAllocator[4]
  131072 |  0 |   1024 | 
 memory/ioBufAllocator[3]
   65536 |  0 |512 | 
 memory/ioBufAllocator[2]
   32768 |  0 |256 | 
 memory/ioBufAllocator[1]
   16384 |  0 |128 | 
 memory/ioBufAllocator[0]
   0 |  0 |576 | 
 memory/ICPRequestCont_allocator
   0 |  0 |112 | 
 memory/ICPPeerReadContAllocator
   0 |  0 |432 | 
 memory/PeerReadDataAllocator
   0 |  0 | 32 | 
 memory/MIMEFieldSDKHandle
   0 |  0 |240 | 
 memory/INKVConnAllocator
   0 |  0 | 96 | 
 memory/INKContAllocator
4096 |  0 | 32 | 
 memory/apiHookAllocator
   0 |  0 |288 | 
 memory/FetchSMAllocator
   0 |  0 | 80 | 
 memory/prefetchLockHandlerAllocator
   0 |  0 |176 | 
 memory/PrefetchBlasterAllocator
   0 |  0 | 80 | 
 memory/prefetchUrlBlaster
   0 |  0 | 96 | memory/blasterUrlList
   0 |  0 | 96 | 
 memory/prefetchUrlEntryAllocator
   0 |  0 |128 | 
 memory/socksProxyAllocator
   0 |  0 |144 | 
 memory/ObjectReloadCont
 3258368 | 576016 |592 | 
 memory/httpClientSessionAllocator
  825344 | 139568 |208 | 
 memory/httpServerSessionAllocator
22597632 |1284848 |   9808 | memory/httpSMAllocator
   0 |  0 | 32 | 
 memory/CacheLookupHttpConfigAllocator
   0 |  0 |   9856 | 
 memory/httpUpdateSMAllocator
   0 |  0 |128 | 
 memory/RemapPluginsAlloc
   0 |  0 | 48 | 
 memory/CongestRequestParamAllocator
   0 |  0 |128 | 
 memory/CongestionDBContAllocator
 5767168 | 

[jira] [Commented] (TS-1006) memory management, cut down memory waste ?

2013-03-27 Thread Yunkai Zhang (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13614985#comment-13614985
 ] 

Yunkai Zhang commented on TS-1006:
--

1) The format seems error,

it should be ./configure --enable-reclaimable-freelist, need not =yes.

 memory management, cut down memory waste ?
 --

 Key: TS-1006
 URL: https://issues.apache.org/jira/browse/TS-1006
 Project: Traffic Server
  Issue Type: Improvement
  Components: Core
Affects Versions: 3.1.1
Reporter: Zhao Yongming
Assignee: Bin Chen
 Fix For: 3.3.3

 Attachments: 
 0001-TS-1006-Add-an-enable-reclaimable-freelist-option.patch, 
 0002-TS-1006-Add-a-new-wrapper-ink_atomic_decrement.patch, 
 0003-TS-1006-Introduce-a-reclaimable-InkFreeList.patch, 
 0004-TS-1006-Make-InkFreeList-memory-pool-configurable.patch, 
 Memory-Usage-After-Introduced-New-Allocator.png, memusage.ods, memusage.ods


 when we review the memory usage in the production, there is something 
 abnormal, ie, looks like TS take much memory than index data + common system 
 waste, and here is some memory dump result by set 
 proxy.config.dump_mem_info_frequency
 1, the one on a not so busy forwarding system:
 physics memory: 32G
 RAM cache: 22G
 DISK: 6140 GB
 average_object_size 64000
 {code}
  allocated  |in-use  | type size  |   free list name
 |||--
   671088640 |   37748736 |2097152 | 
 memory/ioBufAllocator[14]
  2248146944 | 2135949312 |1048576 | 
 memory/ioBufAllocator[13]
  1711276032 | 1705508864 | 524288 | 
 memory/ioBufAllocator[12]
  1669332992 | 1667760128 | 262144 | 
 memory/ioBufAllocator[11]
  2214592512 | 221184 | 131072 | 
 memory/ioBufAllocator[10]
  2325741568 | 2323775488 |  65536 | 
 memory/ioBufAllocator[9]
  2091909120 | 2089123840 |  32768 | 
 memory/ioBufAllocator[8]
  1956642816 | 1956478976 |  16384 | 
 memory/ioBufAllocator[7]
  2094530560 | 2094071808 |   8192 | 
 memory/ioBufAllocator[6]
   356515840 |  355540992 |   4096 | 
 memory/ioBufAllocator[5]
 1048576 |  14336 |   2048 | 
 memory/ioBufAllocator[4]
  131072 |  0 |   1024 | 
 memory/ioBufAllocator[3]
   65536 |  0 |512 | 
 memory/ioBufAllocator[2]
   32768 |  0 |256 | 
 memory/ioBufAllocator[1]
   16384 |  0 |128 | 
 memory/ioBufAllocator[0]
   0 |  0 |576 | 
 memory/ICPRequestCont_allocator
   0 |  0 |112 | 
 memory/ICPPeerReadContAllocator
   0 |  0 |432 | 
 memory/PeerReadDataAllocator
   0 |  0 | 32 | 
 memory/MIMEFieldSDKHandle
   0 |  0 |240 | 
 memory/INKVConnAllocator
   0 |  0 | 96 | 
 memory/INKContAllocator
4096 |  0 | 32 | 
 memory/apiHookAllocator
   0 |  0 |288 | 
 memory/FetchSMAllocator
   0 |  0 | 80 | 
 memory/prefetchLockHandlerAllocator
   0 |  0 |176 | 
 memory/PrefetchBlasterAllocator
   0 |  0 | 80 | 
 memory/prefetchUrlBlaster
   0 |  0 | 96 | memory/blasterUrlList
   0 |  0 | 96 | 
 memory/prefetchUrlEntryAllocator
   0 |  0 |128 | 
 memory/socksProxyAllocator
   0 |  0 |144 | 
 memory/ObjectReloadCont
 3258368 | 576016 |592 | 
 memory/httpClientSessionAllocator
  825344 | 139568 |208 | 
 memory/httpServerSessionAllocator
22597632 |1284848 |   9808 | memory/httpSMAllocator
   0 |  0 | 32 | 
 memory/CacheLookupHttpConfigAllocator
   0 |  0 |   9856 | 
 memory/httpUpdateSMAllocator
   0 |  0 |128 | 
 memory/RemapPluginsAlloc
   0 |  0 | 48 | 
 memory/CongestRequestParamAllocator
   0 |  0 |128 | 
 memory/CongestionDBContAllocator
 5767168 | 704512 |   2048 

[jira] [Comment Edited] (TS-1006) memory management, cut down memory waste ?

2013-03-27 Thread Yunkai Zhang (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1006?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13614985#comment-13614985
 ] 

Yunkai Zhang edited comment on TS-1006 at 3/27/13 7:05 AM:
---

The format seems error,

it should be ./configure --enable-reclaimable-freelist, need not =yes.

  was (Author: yunkai):
1) The format seems error,

it should be ./configure --enable-reclaimable-freelist, need not =yes.
  
 memory management, cut down memory waste ?
 --

 Key: TS-1006
 URL: https://issues.apache.org/jira/browse/TS-1006
 Project: Traffic Server
  Issue Type: Improvement
  Components: Core
Affects Versions: 3.1.1
Reporter: Zhao Yongming
Assignee: Bin Chen
 Fix For: 3.3.3

 Attachments: 
 0001-TS-1006-Add-an-enable-reclaimable-freelist-option.patch, 
 0002-TS-1006-Add-a-new-wrapper-ink_atomic_decrement.patch, 
 0003-TS-1006-Introduce-a-reclaimable-InkFreeList.patch, 
 0004-TS-1006-Make-InkFreeList-memory-pool-configurable.patch, 
 Memory-Usage-After-Introduced-New-Allocator.png, memusage.ods, memusage.ods


 when we review the memory usage in the production, there is something 
 abnormal, ie, looks like TS take much memory than index data + common system 
 waste, and here is some memory dump result by set 
 proxy.config.dump_mem_info_frequency
 1, the one on a not so busy forwarding system:
 physics memory: 32G
 RAM cache: 22G
 DISK: 6140 GB
 average_object_size 64000
 {code}
  allocated  |in-use  | type size  |   free list name
 |||--
   671088640 |   37748736 |2097152 | 
 memory/ioBufAllocator[14]
  2248146944 | 2135949312 |1048576 | 
 memory/ioBufAllocator[13]
  1711276032 | 1705508864 | 524288 | 
 memory/ioBufAllocator[12]
  1669332992 | 1667760128 | 262144 | 
 memory/ioBufAllocator[11]
  2214592512 | 221184 | 131072 | 
 memory/ioBufAllocator[10]
  2325741568 | 2323775488 |  65536 | 
 memory/ioBufAllocator[9]
  2091909120 | 2089123840 |  32768 | 
 memory/ioBufAllocator[8]
  1956642816 | 1956478976 |  16384 | 
 memory/ioBufAllocator[7]
  2094530560 | 2094071808 |   8192 | 
 memory/ioBufAllocator[6]
   356515840 |  355540992 |   4096 | 
 memory/ioBufAllocator[5]
 1048576 |  14336 |   2048 | 
 memory/ioBufAllocator[4]
  131072 |  0 |   1024 | 
 memory/ioBufAllocator[3]
   65536 |  0 |512 | 
 memory/ioBufAllocator[2]
   32768 |  0 |256 | 
 memory/ioBufAllocator[1]
   16384 |  0 |128 | 
 memory/ioBufAllocator[0]
   0 |  0 |576 | 
 memory/ICPRequestCont_allocator
   0 |  0 |112 | 
 memory/ICPPeerReadContAllocator
   0 |  0 |432 | 
 memory/PeerReadDataAllocator
   0 |  0 | 32 | 
 memory/MIMEFieldSDKHandle
   0 |  0 |240 | 
 memory/INKVConnAllocator
   0 |  0 | 96 | 
 memory/INKContAllocator
4096 |  0 | 32 | 
 memory/apiHookAllocator
   0 |  0 |288 | 
 memory/FetchSMAllocator
   0 |  0 | 80 | 
 memory/prefetchLockHandlerAllocator
   0 |  0 |176 | 
 memory/PrefetchBlasterAllocator
   0 |  0 | 80 | 
 memory/prefetchUrlBlaster
   0 |  0 | 96 | memory/blasterUrlList
   0 |  0 | 96 | 
 memory/prefetchUrlEntryAllocator
   0 |  0 |128 | 
 memory/socksProxyAllocator
   0 |  0 |144 | 
 memory/ObjectReloadCont
 3258368 | 576016 |592 | 
 memory/httpClientSessionAllocator
  825344 | 139568 |208 | 
 memory/httpServerSessionAllocator
22597632 |1284848 |   9808 | memory/httpSMAllocator
   0 |  0 | 32 | 
 memory/CacheLookupHttpConfigAllocator
   0 |  0 |   9856 | 
 memory/httpUpdateSMAllocator
   0 |  0 |128 | 
 memory/RemapPluginsAlloc
   0 |  0 |

[jira] [Created] (TS-1783) Eliminate the wpad.dat configuration option (it's unused)

2013-03-27 Thread Leif Hedstrom (JIRA)
Leif Hedstrom created TS-1783:
-

 Summary: Eliminate the wpad.dat configuration option (it's unused)
 Key: TS-1783
 URL: https://issues.apache.org/jira/browse/TS-1783
 Project: Traffic Server
  Issue Type: Improvement
  Components: Configuration
Reporter: Leif Hedstrom




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Assigned] (TS-1783) Eliminate the wpad.dat configuration option (it's unused)

2013-03-27 Thread Leif Hedstrom (JIRA)

 [ 
https://issues.apache.org/jira/browse/TS-1783?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Leif Hedstrom reassigned TS-1783:
-

Assignee: Leif Hedstrom

 Eliminate the wpad.dat configuration option (it's unused)
 -

 Key: TS-1783
 URL: https://issues.apache.org/jira/browse/TS-1783
 Project: Traffic Server
  Issue Type: Improvement
  Components: Configuration
Reporter: Leif Hedstrom
Assignee: Leif Hedstrom
 Fix For: 3.3.2




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-1770) Unable to create remap rule for SSL sites when accessed as a forward proxy

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1770?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13615653#comment-13615653
 ] 

Leif Hedstrom commented on TS-1770:
---

Mark, let us know (and submit a new patch if appropriate), and I'll commit it :)

 Unable to create remap rule for SSL sites when accessed as a forward proxy
 --

 Key: TS-1770
 URL: https://issues.apache.org/jira/browse/TS-1770
 Project: Traffic Server
  Issue Type: Bug
  Components: HTTP
Affects Versions: 3.3.1, 3.2.4
Reporter: Mark Harrison
Assignee: Leif Hedstrom
Priority: Minor
 Fix For: 3.3.2


 When connecting to https sites using ATS as a forward proxy, the CONNECT 
 method is used, and the URL doesn't have a scheme (http/https) present. When 
 using remap.config to limit which sites ATS will proxy for (remap_required 
 set to 1), there is no rule that can be made to match an a request without a 
 scheme present, and so no way to allow requests to a https site.
 Example:
 {code}
 # curl -x 127.0.0.1:8080 -o /dev/null -v -s https://example.com/
 * About to connect() to proxy 127.0.0.1 port 8080 (#0)
 *   Trying 127.0.0.1... connected
 * Connected to 127.0.0.1 (127.0.0.1) port 8080 (#0)
 * Establish HTTP proxy tunnel to example.com:443
  CONNECT example.com:443 HTTP/1.1
  Host: example.com:443
  User-Agent: curl/7.19.7 (x86_64-redhat-linux-gnu) libcurl/7.19.7 
  NSS/3.13.1.0 zlib/1.2.3 libidn/1.18 libssh2/1.2.2
  Proxy-Connection: Keep-Alive
 
  HTTP/1.1 404 Not Found
  Date: Fri, 22 Mar 2013 21:41:25 GMT
  Connection: close
  Server: ATS/3.3.1-dev
  Cache-Control: no-store
  Content-Type: text/html; charset=utf-8
  Content-Language: en
  Content-Length: 309
 
 * Received HTTP code 404 from proxy after CONNECT
 * Closing connection #0
 {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (TS-1782) CENTOS 5.8 x64 autoreconf fail

2013-03-27 Thread Leif Hedstrom (JIRA)

 [ 
https://issues.apache.org/jira/browse/TS-1782?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Leif Hedstrom updated TS-1782:
--

Fix Version/s: 3.3.2
 Assignee: James Peach

 CENTOS 5.8 x64 autoreconf fail 
 ---

 Key: TS-1782
 URL: https://issues.apache.org/jira/browse/TS-1782
 Project: Traffic Server
  Issue Type: Bug
  Components: Build
Reporter: helaku
Assignee: James Peach
 Fix For: 3.3.2


 [root@nbs35 trafficserver]# autoconf -i
 configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
   If this token and others are legitimate, please use m4_pattern_allow.
   See the Autoconf documentation.
 configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
 configure.ac:75: error: possibly undefined macro: dnl
 configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
 configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
 configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
 configure.ac:548: error: possibly undefined macro: AM_PROG_AS
 configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL
 [root@nbs35 trafficserver]# cat /etc/redhat-release 
 CentOS release 5.8 (Final)
 2.6.18-308.11.1.el5 #1 SMP Tue Jul 10 08:48:43 EDT 2012 x86_64 x86_64 x86_64 
 GNU/Linux
 [root@nbs35 trafficserver]# git show
 commit 4e53792f2740de507b6fd51f0d34f6bf4f780852
 Author: Igor Gali[7m87 i.ga...@brainsware.org
 Date:   Wed Mar 27 00:47:21 2013 +0100
 TS-1781: Make clang happy and esi compile
 diff --git a/plugins/experimental/esi/lib/EsiProcessor.cc 
 b/plugins/experimental/esi/lib/EsiProcessor.cc
 index ed4868b..96a647e 100644
 --- a/plugins/experimental/esi/lib/EsiProcessor.cc
 +++ b/plugins/experimental/esi/lib/EsiProcessor.cc
 @@ -422,7 +422,7 @@ EsiProcessor::flush(string data, int overall_len) {
const Attribute url = (*node_iter).attr_list.front(); 
  
string raw_url(url.value, url.value_len);
attemptUrls.push_back(_expression.expand(raw_url));
 -if (!_getIncludeStatus(*node_iter) == STATUS_ERROR) {
 +if (_getIncludeStatus(*node_iter) != STATUS_DATA_AVAILABLE) {
attempt_succeeded = false;
_errorLog([%s] attempt section errored; due to url [%s], 
 __FUNCTION__, raw_url.c_str());
break;

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (TS-1782) CENTOS 5.8 x64 autoreconf fail

2013-03-27 Thread Leif Hedstrom (JIRA)

 [ 
https://issues.apache.org/jira/browse/TS-1782?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Leif Hedstrom updated TS-1782:
--

Description: 
{code}
[root@nbs35 trafficserver]# autoconf -i
configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
  If this token and others are legitimate, please use m4_pattern_allow.
  See the Autoconf documentation.
configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
configure.ac:75: error: possibly undefined macro: dnl
configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
configure.ac:548: error: possibly undefined macro: AM_PROG_AS
configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL

[root@nbs35 trafficserver]# cat /etc/redhat-release 
CentOS release 5.8 (Final)

2.6.18-308.11.1.el5 #1 SMP Tue Jul 10 08:48:43 EDT 2012 x86_64 x86_64 x86_64 
GNU/Linux


[root@nbs35 trafficserver]# git show
commit 4e53792f2740de507b6fd51f0d34f6bf4f780852
Author: Igor Gali[7m87 i.ga...@brainsware.org
Date:   Wed Mar 27 00:47:21 2013 +0100

TS-1781: Make clang happy and esi compile

diff --git a/plugins/experimental/esi/lib/EsiProcessor.cc 
b/plugins/experimental/esi/lib/EsiProcessor.cc
index ed4868b..96a647e 100644
--- a/plugins/experimental/esi/lib/EsiProcessor.cc
+++ b/plugins/experimental/esi/lib/EsiProcessor.cc
@@ -422,7 +422,7 @@ EsiProcessor::flush(string data, int overall_len) {
   const Attribute url = (*node_iter).attr_list.front();  
   string raw_url(url.value, url.value_len);
   attemptUrls.push_back(_expression.expand(raw_url));
-if (!_getIncludeStatus(*node_iter) == STATUS_ERROR) {
+if (_getIncludeStatus(*node_iter) != STATUS_DATA_AVAILABLE) {
   attempt_succeeded = false;
   _errorLog([%s] attempt section errored; due to url [%s], 
__FUNCTION__, raw_url.c_str());
   break;
{code}

  was:
[root@nbs35 trafficserver]# autoconf -i
configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
  If this token and others are legitimate, please use m4_pattern_allow.
  See the Autoconf documentation.
configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
configure.ac:75: error: possibly undefined macro: dnl
configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
configure.ac:548: error: possibly undefined macro: AM_PROG_AS
configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL

[root@nbs35 trafficserver]# cat /etc/redhat-release 
CentOS release 5.8 (Final)

2.6.18-308.11.1.el5 #1 SMP Tue Jul 10 08:48:43 EDT 2012 x86_64 x86_64 x86_64 
GNU/Linux


[root@nbs35 trafficserver]# git show
commit 4e53792f2740de507b6fd51f0d34f6bf4f780852
Author: Igor Gali[7m87 i.ga...@brainsware.org
Date:   Wed Mar 27 00:47:21 2013 +0100

TS-1781: Make clang happy and esi compile

diff --git a/plugins/experimental/esi/lib/EsiProcessor.cc 
b/plugins/experimental/esi/lib/EsiProcessor.cc
index ed4868b..96a647e 100644
--- a/plugins/experimental/esi/lib/EsiProcessor.cc
+++ b/plugins/experimental/esi/lib/EsiProcessor.cc
@@ -422,7 +422,7 @@ EsiProcessor::flush(string data, int overall_len) {
   const Attribute url = (*node_iter).attr_list.front();  
   string raw_url(url.value, url.value_len);
   attemptUrls.push_back(_expression.expand(raw_url));
-if (!_getIncludeStatus(*node_iter) == STATUS_ERROR) {
+if (_getIncludeStatus(*node_iter) != STATUS_DATA_AVAILABLE) {
   attempt_succeeded = false;
   _errorLog([%s] attempt section errored; due to url [%s], 
__FUNCTION__, raw_url.c_str());
   break;


 CENTOS 5.8 x64 autoreconf fail 
 ---

 Key: TS-1782
 URL: https://issues.apache.org/jira/browse/TS-1782
 Project: Traffic Server
  Issue Type: Bug
  Components: Build
Reporter: helaku
Assignee: James Peach
 Fix For: 3.3.2


 {code}
 [root@nbs35 trafficserver]# autoconf -i
 configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
   If this token and others are legitimate, please use m4_pattern_allow.
   See the Autoconf documentation.
 configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
 configure.ac:75: error: possibly undefined macro: dnl
 configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
 configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
 configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
 configure.ac:548: error: possibly undefined macro: AM_PROG_AS
 configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL
 [root@nbs35 

[jira] [Commented] (TS-382) socket option cleanup (and bug fixes)

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13615687#comment-13615687
 ] 

Leif Hedstrom commented on TS-382:
--

I prefer #1 (lets stop putting bitmask nonsense which none one can understand 
:).

 socket option cleanup (and bug fixes)
 ---

 Key: TS-382
 URL: https://issues.apache.org/jira/browse/TS-382
 Project: Traffic Server
  Issue Type: Bug
  Components: Network
Reporter: Leif Hedstrom
 Fix For: 3.5.0


 This is a bug moved from Y! Bugzilla, I'm posting the original bug 
 description and a few comments separately. Note that the bug description is 
 fairly limited, but while looking at this code, I noticed a lot of oddities 
 with the socket option support (lots of hardcoded stuff, and most of it is 
 not configurable).
 Note that the original bug should have been fixed already in Apache TS, but 
 the other comments are still applicable.
 From Bugzilla (posted by Leif):
 We have two socket option config options in records.config:
 proxy.config.net.sock_option_flag_in
 proxy.config.net.sock_option_flag_out
 With accept thread enabled, at least the _in option isn't honored. There are 
 possibly other cases in UnixNetAccept.cc
 that we don't honor these flags either.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-382) socket option cleanup (and bug fixes)

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13615690#comment-13615690
 ] 

Leif Hedstrom commented on TS-382:
--

I do think it's useful to support these (and as many as possible) of these TCP 
options.

 socket option cleanup (and bug fixes)
 ---

 Key: TS-382
 URL: https://issues.apache.org/jira/browse/TS-382
 Project: Traffic Server
  Issue Type: Bug
  Components: Network
Reporter: Leif Hedstrom
 Fix For: 3.5.0


 This is a bug moved from Y! Bugzilla, I'm posting the original bug 
 description and a few comments separately. Note that the bug description is 
 fairly limited, but while looking at this code, I noticed a lot of oddities 
 with the socket option support (lots of hardcoded stuff, and most of it is 
 not configurable).
 Note that the original bug should have been fixed already in Apache TS, but 
 the other comments are still applicable.
 From Bugzilla (posted by Leif):
 We have two socket option config options in records.config:
 proxy.config.net.sock_option_flag_in
 proxy.config.net.sock_option_flag_out
 With accept thread enabled, at least the _in option isn't honored. There are 
 possibly other cases in UnixNetAccept.cc
 that we don't honor these flags either.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-1366) cquc does not work as documented: Logging the remapped URL

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1366?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13615691#comment-13615691
 ] 

Leif Hedstrom commented on TS-1366:
---

I'd say just commit it :)

 cquc does not work as documented: Logging the remapped URL
 --

 Key: TS-1366
 URL: https://issues.apache.org/jira/browse/TS-1366
 Project: Traffic Server
  Issue Type: Bug
  Components: Documentation
Reporter: Igor Galić
Assignee: Alan M. Carroll
 Fix For: Doc 3.4

 Attachments: ts-1366.diff


 From the documentation 
 (http://trafficserver.apache.org/docs/trunk/admin/working-log-files/log-formats#SquidFormat)
  :
 cquc
 The client request canonical URL; blanks and other characters that might 
 not be parsed by log analysis tools are replaced by escape sequences. The 
 escape sequence is a percentage sign followed by the ASCII code number of the 
 replaced character in hex.
 But in fact it logs the remapped URL.
 This may or may not be related to the {{records.config}} setting:
 {noformat}
 CONFIG proxy.config.url_remap.pristine_host_hdr INT 1
 {noformat}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (TS-1586) spdy plugin doesn't build under clang (from trunk)

2013-03-27 Thread JIRA

 [ 
https://issues.apache.org/jira/browse/TS-1586?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Galić updated TS-1586:
---

Environment: 
Linux with llvm/clang from trunk + libstc++
(I don't have libc++ on Linux)

  was:llvm/clang from trunk (on Linux, haven't tested on other platforms yet)


 spdy plugin doesn't build under clang (from trunk)
 --

 Key: TS-1586
 URL: https://issues.apache.org/jira/browse/TS-1586
 Project: Traffic Server
  Issue Type: Bug
  Components: Build, Plugins
Affects Versions: 3.3.1
 Environment: Linux with llvm/clang from trunk + libstc++
 (I don't have libc++ on Linux)
Reporter: Igor Galić
Assignee: Igor Galić
 Fix For: 3.3.2

 Attachments: restrict.patch, swap_restrict.patch


 {noformat}
 Making all in spdy
 gmake[3]: Entering directory 
 `/home/igalic/src/asf/trafficserver/CLANG/plugins/experimental/spdy'
 /bin/sh ../../../libtool  --tag=CXX   --mode=compile clang++ -DHAVE_CONFIG_H 
 -I. -I../../../../plugins/experimental/spdy -I../../../lib/ts  
 -I../../../../plugins/experimental/spdy/lib -I../../../proxy/api 
 -D__STDC_FORMAT_MACROS=1 -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 
 -D_GNU_SOURCE -D_REENTRANT -Dlinux -DDEBUG -D_DEBUG  -std=c++11 -ggdb3 
 -Werror -Qunused-arguments -Wno-invalid-offsetof -MT message.lo -MD -MP -MF 
 .deps/message.Tpo -c -o message.lo `test -f 'lib/spdy/message.cc' || echo 
 '../../../../plugins/experimental/spdy/'`lib/spdy/message.cc
 libtool: compile:  clang++ -DHAVE_CONFIG_H -I. 
 -I../../../../plugins/experimental/spdy -I../../../lib/ts 
 -I../../../../plugins/experimental/spdy/lib -I../../../proxy/api 
 -D__STDC_FORMAT_MACROS=1 -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 
 -D_GNU_SOURCE -D_REENTRANT -Dlinux -DDEBUG -D_DEBUG -std=c++11 -ggdb3 -Werror 
 -Qunused-arguments -Wno-invalid-offsetof -MT message.lo -MD -MP -MF 
 .deps/message.Tpo -c 
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc  -fPIC -DPIC -o 
 .libs/message.o
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:72:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extract(const uint8_t __restrict * ptr) {
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:80:32: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extractuint8_t(const uint8_t __restrict * ptr) {
  ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:85:30: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 insert(const T val, uint8_t __restrict * ptr) {
  ^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:91:33: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extract_stream_id(const uint8_t __restrict * ptr)
   ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:97:46: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 insert_stream_id(uint32_t stream_id, uint8_t __restrict * ptr)
  ^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:104:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:133:44: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const message_header msg, uint8_t __restrict * ptr, size_t len)
^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:153:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:170:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:185:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:200:48: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const rst_stream_message msg, uint8_t __restrict * ptr, size_t len)

[jira] [Commented] (TS-1586) spdy plugin doesn't build under clang (from trunk)

2013-03-27 Thread JIRA

[ 
https://issues.apache.org/jira/browse/TS-1586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13615747#comment-13615747
 ] 

Igor Galić commented on TS-1586:


updated description and environment to reflect this:

Can't use libc++ yet - not without pain: http://libcxx.llvm.org/

 spdy plugin doesn't build under clang (from trunk)
 --

 Key: TS-1586
 URL: https://issues.apache.org/jira/browse/TS-1586
 Project: Traffic Server
  Issue Type: Bug
  Components: Build, Plugins
Affects Versions: 3.3.1
 Environment: Linux with llvm/clang from trunk + libstc++
 (I don't have libc++ on Linux)
Reporter: Igor Galić
Assignee: Igor Galić
 Fix For: 3.3.2

 Attachments: restrict.patch, swap_restrict.patch


 {noformat}
 Making all in spdy
 gmake[3]: Entering directory 
 `/home/igalic/src/asf/trafficserver/CLANG/plugins/experimental/spdy'
 /bin/sh ../../../libtool  --tag=CXX   --mode=compile clang++ -DHAVE_CONFIG_H 
 -I. -I../../../../plugins/experimental/spdy -I../../../lib/ts  
 -I../../../../plugins/experimental/spdy/lib -I../../../proxy/api 
 -D__STDC_FORMAT_MACROS=1 -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 
 -D_GNU_SOURCE -D_REENTRANT -Dlinux -DDEBUG -D_DEBUG  -std=c++11 -ggdb3 
 -Werror -Qunused-arguments -Wno-invalid-offsetof -MT message.lo -MD -MP -MF 
 .deps/message.Tpo -c -o message.lo `test -f 'lib/spdy/message.cc' || echo 
 '../../../../plugins/experimental/spdy/'`lib/spdy/message.cc
 libtool: compile:  clang++ -DHAVE_CONFIG_H -I. 
 -I../../../../plugins/experimental/spdy -I../../../lib/ts 
 -I../../../../plugins/experimental/spdy/lib -I../../../proxy/api 
 -D__STDC_FORMAT_MACROS=1 -D_LARGEFILE64_SOURCE=1 -D_COMPILE64BIT_SOURCE=1 
 -D_GNU_SOURCE -D_REENTRANT -Dlinux -DDEBUG -D_DEBUG -std=c++11 -ggdb3 -Werror 
 -Qunused-arguments -Wno-invalid-offsetof -MT message.lo -MD -MP -MF 
 .deps/message.Tpo -c 
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc  -fPIC -DPIC -o 
 .libs/message.o
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:72:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extract(const uint8_t __restrict * ptr) {
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:80:32: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extractuint8_t(const uint8_t __restrict * ptr) {
  ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:85:30: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 insert(const T val, uint8_t __restrict * ptr) {
  ^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:91:33: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 extract_stream_id(const uint8_t __restrict * ptr)
   ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:97:46: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 insert_stream_id(uint32_t stream_id, uint8_t __restrict * ptr)
  ^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:104:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:133:44: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const message_header msg, uint8_t __restrict * ptr, size_t len)
^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:153:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:170:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:185:23: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const uint8_t __restrict * ptr, size_t len)
 ~~^~
 ../../../../plugins/experimental/spdy/lib/spdy/message.cc:200:48: error: 
 restrict requires a pointer or reference ('uint8_t' (aka 'unsigned char') is 
 invalid)
 const rst_stream_message msg, uint8_t __restrict * 

[jira] [Commented] (TS-1540) better handle of the over connection warnings

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13616061#comment-13616061
 ] 

Leif Hedstrom commented on TS-1540:
---

Yongming: Any updates on this? Are you running with tracers on ?

 better handle of the over connection warnings
 -

 Key: TS-1540
 URL: https://issues.apache.org/jira/browse/TS-1540
 Project: Traffic Server
  Issue Type: Improvement
  Components: HTTP
Affects Versions: 3.3.0
 Environment: taobao own issue here
Reporter: Zhao Yongming
Assignee: Leif Hedstrom
 Fix For: 3.3.2


 on a high load server, we get many WARNINGs in the traffic.out, which result 
 into about 500iops disk write, we may need to decrease the logging speed or 
 convert it to some stats instead.
 {code}
 [Oct 18 12:30:29.015] Server {0x2b0b5e24d700} WARNING: [874028856] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5e44f700} WARNING: [874058392] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5e752700} WARNING: [874102720] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5e853700} WARNING: [873849436] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5d1b8c00} WARNING: [874091635] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5e954700} WARNING: [874102614] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.015] Server {0x2b0b5e34e700} WARNING: [874008652] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.016] Server {0x2b0b5e550700} WARNING: [874102763] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.016] Server {0x2b0b5e550700} WARNING: [874099447] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.016] Server {0x2b0b5d1b8c00} WARNING: [874124453] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e853700} WARNING: [874119636] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e44f700} WARNING: [874119708] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e853700} WARNING: [874034770] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e954700} WARNING: [874107245] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e44f700} WARNING: [874024156] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e24d700} WARNING: [874118129] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e44f700} WARNING: [874123046] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e954700} WARNING: [874121399] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e651700} WARNING: [873939682] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e550700} WARNING: [874043351] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e651700} WARNING: [874116643] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e34e700} WARNING: [874072524] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e752700} WARNING: [874031734] over the 
 number of connection for this host: 253840494
 [Oct 18 12:30:29.020] Server {0x2b0b5e651700} WARNING: [874107262] over the 
 number of connection for this host: 253840494
 {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-1290) the init script patch in RPM

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13616064#comment-13616064
 ] 

Leif Hedstrom commented on TS-1290:
---

So, hmmm, what is this patch about? Is it about including the .spec file in the 
source ? Something to the init script?? ming_zym?

 the init script patch in RPM
 

 Key: TS-1290
 URL: https://issues.apache.org/jira/browse/TS-1290
 Project: Traffic Server
  Issue Type: Improvement
  Components: Portability
Affects Versions: 3.3.0
Reporter: Zhao Yongming
Assignee: Leif Hedstrom
Priority: Minor
 Fix For: 3.3.2


 we have a patch in the RPM, should we put it in the git master?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-1782) CENTOS 5.8 x64 autoreconf fail

2013-03-27 Thread Leif Hedstrom (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13616067#comment-13616067
 ] 

Leif Hedstrom commented on TS-1782:
---

Have you installed all the prerequisites ?

{code}
sudo yum install make automake libtool pkgconfig gcc-c++ openssl-devel 
tcl-devel expat-devel  pcre-devel
sudo yum install libcap libcap-devel
{code}


Please update, so we can take action on this bug.

 CENTOS 5.8 x64 autoreconf fail 
 ---

 Key: TS-1782
 URL: https://issues.apache.org/jira/browse/TS-1782
 Project: Traffic Server
  Issue Type: Bug
  Components: Build
Reporter: helaku
Assignee: James Peach
 Fix For: 3.3.2


 {code}
 [root@nbs35 trafficserver]# autoconf -i
 configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
   If this token and others are legitimate, please use m4_pattern_allow.
   See the Autoconf documentation.
 configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
 configure.ac:75: error: possibly undefined macro: dnl
 configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
 configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
 configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
 configure.ac:548: error: possibly undefined macro: AM_PROG_AS
 configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL
 [root@nbs35 trafficserver]# cat /etc/redhat-release 
 CentOS release 5.8 (Final)
 2.6.18-308.11.1.el5 #1 SMP Tue Jul 10 08:48:43 EDT 2012 x86_64 x86_64 x86_64 
 GNU/Linux
 [root@nbs35 trafficserver]# git show
 commit 4e53792f2740de507b6fd51f0d34f6bf4f780852
 Author: Igor Gali[7m87 i.ga...@brainsware.org
 Date:   Wed Mar 27 00:47:21 2013 +0100
 TS-1781: Make clang happy and esi compile
 diff --git a/plugins/experimental/esi/lib/EsiProcessor.cc 
 b/plugins/experimental/esi/lib/EsiProcessor.cc
 index ed4868b..96a647e 100644
 --- a/plugins/experimental/esi/lib/EsiProcessor.cc
 +++ b/plugins/experimental/esi/lib/EsiProcessor.cc
 @@ -422,7 +422,7 @@ EsiProcessor::flush(string data, int overall_len) {
const Attribute url = (*node_iter).attr_list.front(); 
  
string raw_url(url.value, url.value_len);
attemptUrls.push_back(_expression.expand(raw_url));
 -if (!_getIncludeStatus(*node_iter) == STATUS_ERROR) {
 +if (_getIncludeStatus(*node_iter) != STATUS_DATA_AVAILABLE) {
attempt_succeeded = false;
_errorLog([%s] attempt section errored; due to url [%s], 
 __FUNCTION__, raw_url.c_str());
break;
 {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (TS-1782) CENTOS 5.8 x64 autoreconf fail

2013-03-27 Thread helaku (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1782?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13616079#comment-13616079
 ] 

helaku commented on TS-1782:


{code}
[root@nbs35 ~]# for i in  make automake libtool pkgconfig gcc-c++ openssl-devel 
tcl-devel expat-devel  pcre-devel  libcap libcap-devel; do  rpm -q $i; done  
make-3.81-3.el5
automake-1.9.6-2.3.el5
libtool-1.5.22-7.el5_4
pkgconfig-0.21-2.el5
gcc-c++-4.1.2-52.el5_8.1
openssl-devel-0.9.8e-22.el5_8.4
openssl-devel-0.9.8e-22.el5_8.4
tcl-devel-8.4.13-4.el5
tcl-devel-8.4.13-4.el5
expat-devel-1.95.8-11.el5_8
expat-devel-1.95.8-11.el5_8
pcre-devel-6.6-6.el5_6.1
pcre-devel-6.6-6.el5_6.1
libcap-1.10-26
libcap-1.10-26
libcap-devel-1.10-26
libcap-devel-1.10-26

{code}

 CENTOS 5.8 x64 autoreconf fail 
 ---

 Key: TS-1782
 URL: https://issues.apache.org/jira/browse/TS-1782
 Project: Traffic Server
  Issue Type: Bug
  Components: Build
Reporter: helaku
Assignee: James Peach
 Fix For: 3.3.2


 {code}
 [root@nbs35 trafficserver]# autoconf -i
 configure.ac:43: error: possibly undefined macro: AM_INIT_AUTOMAKE
   If this token and others are legitimate, please use m4_pattern_allow.
   See the Autoconf documentation.
 configure.ac:50: error: possibly undefined macro: AC_DISABLE_STATIC
 configure.ac:75: error: possibly undefined macro: dnl
 configure.ac:154: error: possibly undefined macro: AC_MSG_RESULT
 configure.ac:198: error: possibly undefined macro: AM_CONDITIONAL
 configure.ac:516: error: possibly undefined macro: AC_MSG_NOTICE
 configure.ac:548: error: possibly undefined macro: AM_PROG_AS
 configure.ac:552: error: possibly undefined macro: AC_PROG_LIBTOOL
 [root@nbs35 trafficserver]# cat /etc/redhat-release 
 CentOS release 5.8 (Final)
 2.6.18-308.11.1.el5 #1 SMP Tue Jul 10 08:48:43 EDT 2012 x86_64 x86_64 x86_64 
 GNU/Linux
 [root@nbs35 trafficserver]# git show
 commit 4e53792f2740de507b6fd51f0d34f6bf4f780852
 Author: Igor Gali[7m87 i.ga...@brainsware.org
 Date:   Wed Mar 27 00:47:21 2013 +0100
 TS-1781: Make clang happy and esi compile
 diff --git a/plugins/experimental/esi/lib/EsiProcessor.cc 
 b/plugins/experimental/esi/lib/EsiProcessor.cc
 index ed4868b..96a647e 100644
 --- a/plugins/experimental/esi/lib/EsiProcessor.cc
 +++ b/plugins/experimental/esi/lib/EsiProcessor.cc
 @@ -422,7 +422,7 @@ EsiProcessor::flush(string data, int overall_len) {
const Attribute url = (*node_iter).attr_list.front(); 
  
string raw_url(url.value, url.value_len);
attemptUrls.push_back(_expression.expand(raw_url));
 -if (!_getIncludeStatus(*node_iter) == STATUS_ERROR) {
 +if (_getIncludeStatus(*node_iter) != STATUS_DATA_AVAILABLE) {
attempt_succeeded = false;
_errorLog([%s] attempt section errored; due to url [%s], 
 __FUNCTION__, raw_url.c_str());
break;
 {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira