Bug #18115

Smart proxy - 1.14 - Windows - uninitialized constant error

Added by Jack Watroba 8 months ago. Updated 8 months ago.

Status:Closed
Priority:Normal
Assigned To:Dmitri Dolguikh
Category:Core
Target version:Foreman - Team Ivan Iteration 9
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/smart-proxy/pull/495
Story points-
Velocity based estimate-
Release1.14.1Release relationshipAuto

Description

When trying to upgrade from 1.13 to 1.14 smart proxy on Windows 2008R2, I receive the following error when trying to run the proxy:

C:\smart-proxy>bundle exec ruby C:\smart-proxy\bin\smart-proxy
C:/smart-proxy/lib/proxy/log.rb:35:in `<class:LoggerFactory>': uninitialized con
stant Proxy::LoggerFactory::Syslog (NameError)
from C:/smart-proxy/lib/proxy/log.rb:22:in `<module:Proxy>'
from C:/smart-proxy/lib/proxy/log.rb:15:in `<top (required)>'
from C:/smart-proxy/lib/launcher.rb:1:in `require'
from C:/smart-proxy/lib/launcher.rb:1:in `<top (required)>'
from C:/smart-proxy/lib/smart_proxy_main.rb:4:in `require'
from C:/smart-proxy/lib/smart_proxy_main.rb:4:in `<top (required)>'
from C:/smart-proxy/bin/smart-proxy:5:in `require'
from C:/smart-proxy/bin/smart-proxy:5:in `<main>'

I've tried Ruby 2.3.3. and ruby 2.2.6 and both have this issue. Smart proxy version 1.13 works fine on this machine.


Related issues

Related to Smart Proxy - Feature #15874: Log X-Request-ID by default Closed 07/28/2016

Associated revisions

Revision 55583fb3
Added by Dmitri Dolguikh 8 months ago

Fixes #18115 - Logger is initialized when Syslog isn't available.

History

#1 Updated by Dmitri Dolguikh 8 months ago

  • Category set to Core

#2 Updated by Dominic Cleal 8 months ago

#3 Updated by Dominic Cleal 8 months ago

  • Release set to 1.14.1

The constant is referenced without checking if it's defined, or if the syslog library loaded successfully. This was added in #15874.

#4 Updated by Dmitri Dolguikh 8 months ago

  • Status changed from New to Assigned
  • Assigned To set to Dmitri Dolguikh

#5 Updated by The Foreman Bot 8 months ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/495 added

#6 Updated by Anonymous 8 months ago

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

#7 Updated by Ivan Necas 8 months ago

  • Target version set to Team Ivan Iteration 9

Also available in: Atom PDF