Bug 856670
Summary: | Aeolus-cleanup error:Must pass nfs_mount_point to Aeolus::Profiles::Rhevm::Disable[default] at /usr/share/aeolus-configure/modules/aeolus/manifests/profiles/rhevm/disable.pp: | ||
---|---|---|---|
Product: | [Retired] CloudForms Cloud Engine | Reporter: | Aziza Karol <akarol> |
Component: | aeolus-configure | Assignee: | Mo Morsi <mmorsi> |
Status: | CLOSED WONTFIX | QA Contact: | Rehana <aeolus-qa-list> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | 1.1.0 | CC: | dajohnso |
Target Milestone: | rc | Flags: | morazi:
needinfo+
|
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-08-14 19:04:56 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Aziza Karol
2012-09-12 14:35:06 UTC
This appears to be 2 issues: 1. The first issue looks like you didn't edit /etc/aeolus-configure/nodes/rhevm_cleanup to telli t about nfs_mount_point (there is a section in there that needs to be uncommented out prior to running aeolus-cleanup) 2. vsphere, ec2, mock don't have corresponding *_cleanup descriptions. We limit cleanup pretty severely so we don't accidentally remove provider/provider account info. rhevm has a cleanup to remove the nfs mount, but none of the other providers require a similar setup. I'm not sure we want to dig too much deeper into removal of resources than that. Thoughts? To be more explicit, if you go along with the explanation in 2 and agree, please CLOSE as not a bug. If I'm missing something or there is disagreement on point 2, let's discuss. Pushed a patch addressing #1 so that the nfs unmount will be skipped over if mount point is not specified https://github.com/aeolusproject/aeolus-configure/commit/601ee822a6559ba76abe9ad02e6cb5a2bc449609 And yes, as far as #2 that is the intended behaviour. We have a check that gets run as part of configure and cleanup that ensures the specified profile exists and outputs a "File not found" before exiting if it doesn't. aeolus is no longer being developed |