[Bug 885998] Re: facter upgrade crashes puppet

2011-11-09 Thread Martijn van Brummelen
** Changed in: puppet
   Status: New => Fix Released

** Changed in: facter (Ubuntu)
   Status: Confirmed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Steve Huff
another confirmed fix in lucid (non-EC2) for 1.5.8-2ubuntu2.3, on a
system that had previously exhibited the issue.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Martijn van Brummelen
Confirm 1.5.8-2ubuntu2.3 fixes the problem in facter and makes puppet
run again.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Launchpad Bug Tracker
** Branch linked: lp:ubuntu/maverick-updates/facter

** Branch linked: lp:ubuntu/natty-updates/facter

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Launchpad Bug Tracker
This bug was fixed in the package facter - 1.5.8-2ubuntu2.2

---
facter (1.5.8-2ubuntu2.2) natty-proposed; urgency=low

  * debian/patches/fix_ec2_metadata_facts.patch: Refreshed to rescue condition
in can_connect() when timeout() actually has a chance to timeout.
(LP: #885998)
 -- Adam GandelmanMon, 07 Nov 2011 10:47:01 -0800

** Changed in: facter (Ubuntu Natty)
   Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Launchpad Bug Tracker
This bug was fixed in the package facter - 1.5.7-1ubuntu1.3

---
facter (1.5.7-1ubuntu1.3) maverick-proposed; urgency=low

  * lib/facter/ec2.rb: Rescue condition in can_connect() when timeout()
actually has a chance to timeout. (LP: #885998)
 -- Adam GandelmanMon, 07 Nov 2011 10:27:58 -0800

** Changed in: facter (Ubuntu Maverick)
   Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Paul Hirst
The natty package fixes the problem too.

I don't have anywhere to test oneric unfortunately.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Paul Hirst
Just tested the maverick package. Again, it's fixed the problem.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Paul Hirst
I tested the lucid proposed package. The regression is fixed (in my non
EC2 environment).

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-08 Thread Launchpad Bug Tracker
** Branch linked: lp:ubuntu/lucid-updates/facter

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Martin Pitt
Skipping the usual 7 day waiting period as this was a regression. I'll
release them as soon as testing reports come in.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Launchpad Bug Tracker
This bug was fixed in the package facter - 1.5.6-2ubuntu2.3

---
facter (1.5.6-2ubuntu2.3) lucid-proposed; urgency=low

  * lib/facter/ec2.rb: Rescue condition in can_connect() when timeout()
actually has a chance to timeout. (LP: #885998)
 -- Adam GandelmanMon, 07 Nov 2011 10:18:18 -0800

** Changed in: facter (Ubuntu Lucid)
   Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Tarjei Huse
Hi, I tested the package on Lucid and confirm that it fixes the bug.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Steve Langasek
Hello majordom, or anyone else affected,

Accepted into lucid-proposed, the package will build now and be
available in a few hours. Please test and give feedback here. See
https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to
enable and use -proposed. Thank you in advance!

** Also affects: facter (Ubuntu Lucid)
   Importance: Undecided
   Status: New

** Also affects: facter (Ubuntu Maverick)
   Importance: Undecided
   Status: New

** Also affects: facter (Ubuntu Natty)
   Importance: Undecided
   Status: New

** Changed in: facter (Ubuntu Lucid)
   Importance: Undecided => Critical

** Changed in: facter (Ubuntu Lucid)
   Status: New => In Progress

** Changed in: facter (Ubuntu Maverick)
   Importance: Undecided => Critical

** Changed in: facter (Ubuntu Maverick)
   Status: New => In Progress

** Changed in: facter (Ubuntu Natty)
   Importance: Undecided => Critical

** Changed in: facter (Ubuntu Natty)
   Status: New => In Progress

** Changed in: facter (Ubuntu Lucid)
   Status: In Progress => Fix Committed

** Changed in: facter (Ubuntu Maverick)
   Status: In Progress => Fix Committed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Jacob Helwig
Adam Gandelman graciously pointed out that he already had branches to
fix this.  Looking at the changes, they're the minimum to fix the
regression and are a better fix than the more invasive back-porting I
was looking at.  I've also verified his changes on lucid running on EC2,
and in a VM.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Jacob Helwig
I just checked the change that David posted and it breaks retrieval of
EC2 facts.

In the upstream repository (as of
d62e079489c07201cb343f2ca109fecd62d6e567, and later refactored in
cc67a0148b97e315572cdb905476df1224a78dd5) the can_connect? method
started only being called if a couple of additional checks passed.

I'm looking for a good way to back-port some of these changes.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread David Brewer
I just ran into this issue this afternoon on my own servers. The issue
is definitely related to the ec2.rb changes in  facter 1.5.6-2ubuntu2.2.
As a temporary hack/workaround I was able to get Puppet working again by
manually editing /usr/lib/ruby/1.8/facter/ec2.rb and reverting the
"can_connect?" method to use the code for that method defined in the
previous version of the page.  Specifically, see below the commented
"new" version of the method and my inserted "old" version of the method.

# This version of the method causes timeouts even if your machine is not using 
EC2
#def can_connect?(ip,port,wait_sec=2)
# url = "http://#{ip}:#{port}";
# Timeout::timeout(wait_sec) {open(url)}
# return true
#rescue
#  return false
#end

# This version of the method seems to work, although I can't guarantee it works 
with EC2
# as my servers are not running there
def can_connect?(ip,port,wait_sec=2)
 Timeout::timeout(wait_sec) {open(ip, port)}
 return true
rescue
  return false
end

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Launchpad Bug Tracker
** Branch linked: lp:~gandelman-a/ubuntu/lucid/facter/732953_fixregress

** Branch linked: lp:~gandelman-a/ubuntu/natty/facter/732953_fixregress

** Branch linked:
lp:~gandelman-a/ubuntu/maverick/facter/732953_fixregress

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Steve Huff
I can confirm that rolling back to facter_1.5.6-2ubuntu2 resolves the
issue on 10.04 LTS.

I'm using puppet 2.6.1-0ubuntu2 from the lucid-backports repo.

In addition, I see the following traceback using facter_1.5.6-2ubuntu2.2
when running `facter --debug`:


# facter --debug
/usr/lib/ruby/1.8/timeout.rb:60:in `rbuf_fill': execution expired 
(Timeout::Error)
from /usr/lib/ruby/1.8/timeout.rb:62:in `timeout'
from /usr/lib/ruby/1.8/timeout.rb:93:in `timeout'
from /usr/lib/ruby/1.8/net/protocol.rb:134:in `rbuf_fill'
from /usr/lib/ruby/1.8/net/protocol.rb:116:in `readuntil'
from /usr/lib/ruby/1.8/net/protocol.rb:126:in `readline'
from /usr/lib/ruby/1.8/net/http.rb:2024:in `read_status_line'
from /usr/lib/ruby/1.8/net/http.rb:2013:in `read_new'
from /usr/lib/ruby/1.8/net/http.rb:1050:in `request'
from /usr/lib/ruby/1.8/open-uri.rb:248:in `open_http'
from /usr/lib/ruby/1.8/net/http.rb:543:in `start'
from /usr/lib/ruby/1.8/open-uri.rb:242:in `open_http'
from /usr/lib/ruby/1.8/open-uri.rb:616:in `buffer_open'
from /usr/lib/ruby/1.8/open-uri.rb:164:in `open_loop'
from /usr/lib/ruby/1.8/open-uri.rb:162:in `catch'
from /usr/lib/ruby/1.8/open-uri.rb:162:in `open_loop'
from /usr/lib/ruby/1.8/open-uri.rb:132:in `open_uri'
from /usr/lib/ruby/1.8/open-uri.rb:518:in `open'
from /usr/lib/ruby/1.8/open-uri.rb:30:in `open'
from /usr/lib/ruby/1.8/facter/ec2.rb:10:in `can_connect?'
from /usr/lib/ruby/1.8/facter/ec2.rb:10:in `can_connect?'
from /usr/lib/ruby/1.8/facter/ec2.rb:33
from /usr/lib/ruby/1.8/facter/util/loader.rb:72:in `load'
from /usr/lib/ruby/1.8/facter/util/loader.rb:72:in `load_file'
from /usr/lib/ruby/1.8/facter/util/loader.rb:38:in `load_all'
from /usr/lib/ruby/1.8/facter/util/loader.rb:33:in `each'
from /usr/lib/ruby/1.8/facter/util/loader.rb:33:in `load_all'
from /usr/lib/ruby/1.8/facter/util/loader.rb:30:in `each'
from /usr/lib/ruby/1.8/facter/util/loader.rb:30:in `load_all'
from /usr/lib/ruby/1.8/facter/util/collection.rb:94:in `load_all'
from /usr/lib/ruby/1.8/facter.rb:91:in `to_hash'
from /usr/bin/facter:138

Note the calls to ec2.rb; I would look first at the changes implemented
in this branch:

https://code.launchpad.net/~gandelman-a/ubuntu/lucid/facter/lp732953_876130

-Steve

** Also affects: puppet
   Importance: Undecided
   Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/puppet/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Colin Watson
** Tags added: regression-update

** Changed in: facter (Ubuntu)
   Importance: Undecided => Critical

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/facter/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


[Bug 885998] Re: facter upgrade crashes puppet

2011-11-07 Thread Launchpad Bug Tracker
Status changed to 'Confirmed' because the bug affects multiple users.

** Changed in: facter (Ubuntu)
   Status: New => Confirmed

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/885998

Title:
  facter upgrade crashes puppet

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/facter/+bug/885998/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs