Project

General

Profile

Feature #35865

Support new Debian unstable OS facts

Added by Chris Hofstaedtler about 2 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Facts
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

lsb_release / base-files in Debian unstable changed its output data, and thus facter reports different values. Because they contain a "/", facts upload fail with this error:

During the fact upload the server responded with: 500 Internal Server Error. Error is ignored and the execution continues.
{
 "error": {"message":"Validation failed: Major version is not a number, Major version Operating System version is required"}
}

facter output (from puppet-agent 7.21.0-1bullseye deb):

os => {
  architecture => "amd64",
  distro => {
    codename => "bookworm",
    description => "Debian GNU/Linux bookworm/sid",
    id => "Debian",
    release => {
      full => "bookworm/sid",
      major => "bookworm/sid" 
    }
  },
  family => "Debian",
  hardware => "x86_64",
  name => "Debian",
  release => {
    full => "bookworm/sid",
    major => "bookworm/sid" 
  },
  selinux => {
    enabled => false
  }
}
$ /opt/puppetlabs/bin/facter --version
4.2.14
$ lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux bookworm/sid
Release:        n/a
Codename:       bookworm

Debian bug about the change: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1021663

Associated revisions

Revision 70dd4917 (diff)
Added by Evgeni Golov about 1 month ago

Fixes #35865 - Update future Debian release names and versions

Debian dropped the VERSION_ID from /etc/os-release of unreleased
releases (read: testing and unstable), which makes the common tools
report it as 'n/a'. See https://bugs.debian.org/1021663 for details.

The Puppet parser already coped with that by also looking at major
version which gets reported as '/sid' and now the Ansible parser gained similar logic (but looking at the codename) too. The list of upcoming releases is based on https://wiki.debian.org/DebianReleases

History

#1 Updated by The Foreman Bot about 1 month ago

  • Assignee set to Evgeni Golov
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/9556 added

#2 Updated by Chris Hofstaedtler about 1 month ago

  • Found in Releases 3.4.1 added

Suggested PR works for me. Thanks!

#3 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 3.6.0 added

#4 Updated by Evgeni Golov about 1 month ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Evgeni Golov about 1 month ago

  • Target version set to 3.5.1
  • Assignee deleted (Evgeni Golov)
  • Category deleted (Facts)
  • Found in Releases deleted (3.4.1)
  • Fixed in Releases deleted (3.6.0)

#6 Updated by Evgeni Golov about 1 month ago

  • Assignee set to Evgeni Golov
  • Category set to Facts
  • Found in Releases 3.4.1 added
  • Fixed in Releases 3.6.0 added

#7 Updated by The Foreman Bot about 1 month ago

  • Fixed in Releases 3.5.1 added

Also available in: Atom PDF