Project

General

Profile

Bug #2372

importing fails on weird operatingsystemrelease

Added by David Schmitt over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Low
Category:
Facts
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

I'm currently doing some development to get cisco switches under puppet management. Some of those devices report their operatingsystemrelease as '12.2(55)SE5'. When trying to push those facts to foreman with the push_facts script, the first push emits the following message and fails to populate the overview values like OS and version:

Started POST "/fact_values/create?format=yml" for 10.1.105.39 at 2013-04-04 13:47:52 +0000
  Processing by FactValuesController#create as YML
  Parameters: {"facts"=>"[FILTERED]"}
No smart proxy server found on []
Failed to save switch.example.com: Validation failed: Minor is not a number
Rendered text template (0.0ms)
Completed 200 OK in 175ms (Views: 0.9ms | ActiveRecord: 67.2ms)

I've confirmed that (manually) setting the operatingsystemrelease to '12.2.(55)SE5' results in a properly populated host.

aix-facts.txt aix-facts.txt 3.21 KB Josh Baird, 05/13/2013 08:20 AM

Associated revisions

Revision fe838fad (diff)
Added by Greg Sutcliffe over 6 years ago

Fixes #2372 - Strip non-digit characters from non-numeric OS versions

Revision d666ad55 (diff)
Added by Greg Sutcliffe over 6 years ago

Fixes #2372 - Strip non-digit characters from non-numeric OS versions
(cherry picked from commit fe838fad64723c17a4f69d37355927214cba4592)

History

#1 Updated by Josh Baird over 6 years ago

Hitting this problem with AIX as well. Facts are attached.

#2 Updated by Ohad Levy over 6 years ago

  • Category set to Facts
  • Target version set to 1.2.0

#3 Updated by Greg Sutcliffe over 6 years ago

  • Status changed from New to Assigned
  • Assignee set to Greg Sutcliffe

#4 Updated by Greg Sutcliffe over 6 years ago

  • Status changed from Assigned to Ready For Testing

#5 Updated by Greg Sutcliffe over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF