Bug 617163
Summary: | cman dispatches config changes when there is no new config | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 6 | Reporter: | Fabio Massimo Di Nitto <fdinitto> | ||||
Component: | cluster | Assignee: | Fabio Massimo Di Nitto <fdinitto> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Cluster QE <mspqa-list> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 6.0 | CC: | ccaulfie, cluster-maint, djansa, lhh, rpeterso, ssaha, teigland | ||||
Target Milestone: | rc | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | cluster-3.0.12-18.el6 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2010-11-10 20:00:03 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Fabio Massimo Di Nitto
2010-07-22 11:17:44 UTC
Maybe we just add 'cman_tool configupdate' alias for 'cman_tool version -r0' ? -r0 == Use what's in cluster.conf Or... cman_tool version -r = reload config (e.g. -r0) cman_tool version -rX = reload config, print warning that the 'X' is deprecated and is ignored Created attachment 433731 [details]
Drop version -r#; always use version from cluster.conf
Note that this patch maintains cmdline compatibility except for the warning message. Due to the way getopt() works (shifting args around), it's difficult to catch this case in a sane way: cman_tool version -r -Dnone 123 This produces the warning about -r# deprecated (instead of an error in parsing the cmdline) but I don't really consider this a big issue. The :: operator means an option arg might have an option, but requires: "-oopt" That is, "-o opt" does not work with the getopt GNU extension for optional arguments. Now, the ':' (required arg) option by contrast accepts both "-oopt" and "-o opt". Consequently, the patch checks for both uses. The alternative is to turn of opterr, which is not a good idea in my opinion. Note for QE: the test cases for those fixes are complex. I´ll need to prepare a proper document comparing before and after the fix. Both Lon and I did test the fixes so I have built for Snap9. Stay tuned. Verified while testing 617161. Red Hat Enterprise Linux 6.0 is now available and should resolve the problem described in this bug report. This report is therefore being closed with a resolution of CURRENTRELEASE. You may reopen this bug report if the solution does not work for you. |