Jira (PUP-10286) When a site has no active connections, it should be removed from the pool.

16 views
Skip to first unread message

Melissa Stone (JIRA)

unread,
Feb 7, 2020, 6:02:05 PM2/7/20
to puppe...@googlegroups.com
Melissa Stone created an issue
 
Puppet / Task PUP-10286
When a site has no active connections, it should be removed from the pool.
Issue Type: Task Task
Assignee: Unassigned
Created: 2020/02/07 3:01 PM
Priority: Normal Normal
Reporter: Melissa Stone

Spurred from conversation in https://github.com/puppetlabs/puppet/pull/7968

when closing a connection, if a site has no sessions associated with it, the site should be removed from the pool.

pool with an active connectin:

> pool
=> #<Puppet::Network::HTTP::Pool:0x00007f84fdd21110
 @factory=#<Puppet::Network::HTTP::Factory:0x00007f84fdd210c0>,
 @keepalive_timeout=15,
 @pool=
  {#<Puppet::Network::HTTP::Site:0x00007f84fdd21228
    @host="rubygems.org",
    @port=443,
    @scheme="https">=>
    [#<Puppet::Network::HTTP::Session:0x00007f84fdd20e40
      @connection=#<Net::HTTP https://rubygems.org:443:80 open=true>,
      @expiration_time=2020-02-07 14:58:31 -0800>]}>

versus pool after the only connection has been closed:

> pool
=> #<Puppet::Network::HTTP::Pool:0x00007f84fdd21110
 @factory=#<Puppet::Network::HTTP::Factory:0x00007f84fdd210c0>,
 @keepalive_timeout=15,
 @pool=
  {#<Puppet::Network::HTTP::Site:0x00007f84fdd21228
    @host="rubygems.org",
    @port=443,
    @scheme="https">=>[]}>

notice to access the connections, we need to call pool.pool, which is confusing.

Add Comment Add Comment
 
This message was sent by Atlassian JIRA (v7.7.1#77002-sha1:e75ca93)
Atlassian logo

Josh Cooper (JIRA)

unread,
Feb 7, 2020, 6:56:03 PM2/7/20
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Feb 10, 2020, 12:21:04 PM2/10/20
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Feb 10, 2020, 12:23:04 PM2/10/20
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Feb 10, 2020, 11:46:04 PM2/10/20
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Feb 10, 2020, 11:46:05 PM2/10/20
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Sprint: Coremunity Hopper Platform Core KANBAN

Melissa Stone (JIRA)

unread,
Feb 14, 2020, 2:38:05 PM2/14/20
to puppe...@googlegroups.com

Josh Cooper (JIRA)

unread,
Feb 20, 2020, 2:52:04 PM2/20/20
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Release Notes Summary: Very minor reduction in memory usage when the last connection from the http pool is borrowed and never returned.
Release Notes: Bug Fix

Melissa Stone (JIRA)

unread,
Feb 24, 2020, 12:41:04 PM2/24/20
to puppe...@googlegroups.com
 
Re: When a site has no active connections, it should be removed from the pool.

This has passed ci as a part of puppet-agent 6.13.0.51.gf3fde108

Reply all
Reply to author
Forward
0 new messages