RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 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 "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". 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 "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-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 737722 - vdsm-reg params are not removed from cmdline after registering to RHEV-M
Summary: vdsm-reg params are not removed from cmdline after registering to RHEV-M
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: vdsm
Version: 6.2
Hardware: All
OS: Linux
high
low
Target Milestone: rc
: ---
Assignee: Yotam Oron
QA Contact: yeylon@redhat.com
URL:
Whiteboard:
: 744511 745412 (view as bug list)
Depends On: 723115
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-09-12 23:24 UTC by Mike Burns
Modified: 2023-04-07 10:41 UTC (History)
16 users (show)

Fixed In Version: vdsm-4.9-107
Doc Type: Bug Fix
Doc Text:
Clone Of: 723115
Environment:
Last Closed: 2011-12-06 07:28:30 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Proposed patch (1.76 KB, patch)
2011-09-14 08:57 UTC, Alan Pevec 		no flags Details | Diff
suggested upsteam patch (2.02 KB, patch)
2011-09-16 20:55 UTC, Dan Kenigsberg 		no flags Details | Diff
View All


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2011:1782 0 normal SHIPPED_LIVE new packages: vdsm 2011-12-06 11:55:51 UTC

Description Mike Burns 2011-09-12 23:24:00 UTC 
+++ This bug was initially created as a clone of Bug #723115 +++

Description of problem:
 This BZ is holder for issue - when I installed rhev-h-6.2-0.5.iso, I used management_server=IP:8443 & management_server_fingerprint=my:fp:... clauses. In this case vdsm-reg fetch wrong ca.crt (it looks it at   http://IP:8443/ca.crt). After approval the install failed. Then I changed port => 8080 in vdsm-reg.conf & restart vdsm-reg service, then host was installed successfully. 
 The problem is, that kernel param management_server & management_server_fingerprint stays there.

ovirt-node has been patched to read files in /etc/ovirt-commandline.d for a list of command line options to keep out of the default boot options.

vdsm needs to create a file with all it's kernel options listed out and place it in this directory.  File naming doesn't matter, but I'd recommend something like vdsm for the name.  Contents are simply one parameter per line in the file.
Comment 2 Yotam Oron 2011-09-14 06:23:31 UTC 
Mike,

vdsm-reg is taking its needed parameters from vdsm-reg.conf.
The values in this file are configured once, either via TUI or kernel params - both on installation. After that, the kernel params are not relevant anymore (at least in the vdsm-reg case).
From your description, it seems you might be using the wrong port (or older 
rhevm ?), which causes the certificate not to be downloaded etc.

Please clarify.
Comment 3 Alan Pevec 2011-09-14 08:57:13 UTC 
Created attachment 523088 [details]
Proposed patch

(In reply to comment #2)
> Please clarify.

This is to avoid management_server* being propagated to grub config, they are installation-time only parameters so they shouldn't be present on subsequent normal boot from disk.
vdsm-config reads /proc/cmdline and changes vdsm-reg.conf on reboot after upgrade, so it might be confusing for users who changed vdsm-reg.conf to see it reverted to the values hidding in cmdline, esp. if there was a typo in the initial value which they wanted to correct.

Proposed patch is attached.
Comment 4 Dan Kenigsberg 2011-09-15 11:03:26 UTC 
Is there any reason to haste this into RHEV 3.0 beta 3? If so, please wave the rhel-6.2.0 flag again.
Comment 5 Alan Pevec 2011-09-15 11:17:45 UTC 
(In reply to comment #4)
> Is there any reason to haste this into RHEV 3.0 beta 3?

I vote for 6.2: patch is simple and low-risk and for justification let's ask original bug submitter Pavel (bug 723115)
Comment 7 Dan Kenigsberg 2011-09-16 20:55:25 UTC 
Created attachment 523623 [details]
suggested upsteam patch

The patch seems harmless, but our typical user is not editing vdsm-reg.conf manually. I'm a bit worried about fixing a low-severity (and very old) bug during beta period.

I attach a version of Alan's patch rebased on upstream Vdsm. I prefer taking it only to there - unless there is a true reason for exception+.
Comment 17 Yotam Oron 2011-10-02 08:13:47 UTC 
Removed release_notes flag (not needed)
Comment 18 Yotam Oron 2011-10-03 13:53:39 UTC 
Changing prio to HIGH.
Seems that this has a very undesired implication.
Scenario:
- Install a node with kernel params
- Change the RHEVM that the node is working with (using the TUI or any other 
  standard way)
- Upgrade the node
- vdsm-reg is trying to register with the old RHEVM (the IP address is in the 
  command line arguments that now seem to be parsed and processed on upgrade)
Comment 19 Yotam Oron 2011-10-04 12:59:57 UTC 
http://gerrit.usersys.redhat.com/#change,988
Comment 22 Dan Kenigsberg 2011-10-09 09:23:59 UTC 
*** Bug 744511 has been marked as a duplicate of this bug. ***
Comment 23 Yotam Oron 2011-10-16 09:42:07 UTC 
*** Bug 745412 has been marked as a duplicate of this bug. ***
Comment 24 yeylon@redhat.com 2011-10-27 06:28:12 UTC 
verified on vdsm-4.9-108.el6.x86_64
Red Hat Enterprise Virtualization Hypervisor release 6.2 (20111019.2)

1. use an old version of rhev-h kernel params set vdc_host to host A
2. use TUI to change vdc_host to host B
3. register & approve to host B
4. upgrade the node from with in the RHEVM to version  6.2 (20111019.2)
5. check in vdsm-reg.com vdc_host param remains host B

same steps as Bug 744511
Comment 25 errata-xmlrpc 2011-12-06 07:28:30 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2011-1782.html
Note You need to log in before you can comment on or make changes to this bug.