Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1250383 - Facter fails with "sh: vdsmdummy: command not found"
Summary: Facter fails with "sh: vdsmdummy: command not found"
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Configuration Management
Version: 6.1.8
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: Unspecified
Assignee: satellite6-bugs
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-08-05 09:34 UTC by Shinobu KINJO
Modified: 2020-01-17 15:32 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-11-30 12:46:40 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Shinobu KINJO 2015-08-05 09:34:49 UTC
Description of problem:
When customer issue ifconfig, puppet agent keeps complaining:

    sh: vdsmdummy: command not found

Version-Release number of selected component (if applicable):

facter-1.7.6

How reproducible:

Just issue ifconfig ';vdsmdummy;'

Steps to Reproduce:
1.
2.
3.

Actual results:
[root@nsds1be101t ~]# ifconfig ';vdsmdummy;'
 ...

And now Puppet complains:

[root@nsds1be101t ~]# puppet agent -t
 ...
sh: vdsmdummy: command not found

Expected results:


Additional info:

Since this message comes from executing "ifconfig", ruby file which outputs that message is:

    ip.rb

locating at:

    lib/facter/util

which src.rpm is:

    facter-1.7.6-2.el6sat.src.rpm

Specific line should be to somewhere which executed ifconfig or ip addr show us network interface information.

Where is that, for example.

Let's see line #120. There is a line:

    ip_output = %x{/sbin/ip link show #{interface}}

Since #{interface} is a parameter set in this function:

    self.get_infiniband_macaddress(interface)

It should be surrounded by semicolon like:

    "#{interface}"

So this:

    ip_output = %x{/sbin/ip link show #{interface}}

should be

    ip_output = %x{/sbin/ip link show "#{interface}"}

There are other 2 lines which are:

    Line 130, 190

Please let me know when you can fix it out.
Or if you require more, jest let me know what's required to do quick action.

Sincerely,
Shinobu

Comment 3 Ivan Chavero 2015-08-06 21:48:23 UTC
This appears to be a facter problem, changing to the proper componen.

Comment 5 Shinobu KINJO 2015-08-18 06:18:03 UTC
Customer confirmed that modification I mentioned is working.
Will patch be backported?

Just customer wants to know that.

Comment 10 Craig Donnelly 2016-05-24 20:18:11 UTC
Hey Shinobu,

What change did you make on line #130?

For our el7sat version, we have this by default:
130     output = Facter::Util::IP.exec_ifconfig([interface,"2>/dev/null"])

Are we simply adding double-quotes around 'interface'?

Thanks,
Craig.

Comment 12 Bryan Kearney 2016-07-26 19:11:29 UTC
Moving 6.2 bugs out to sat-backlog.

Comment 15 Ohad Levy 2016-11-30 12:46:40 UTC
Satellite 6.2 delivered Facter 2.4.z which already solved this issue.


Note You need to log in before you can comment on or make changes to this bug.