Bug 209415
| Summary: | iscsi-initiator-utils should create and own /var/lib/iscsi and /var/lock/iscsi | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Mark McLoughlin <markmc> |
| Component: | iscsi-initiator-utils | Assignee: | Mike Christie <mchristi> |
| Status: | CLOSED RAWHIDE | QA Contact: | |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | agrover |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | 6.2.0.695-0.6 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2007-04-27 07:03:16 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: | |||
Doh, how many times am I going to mess up on these dirs? :) At least I am learning what the correrct procedure is. We will not be able to get this into the initial release of FC6 due to the freeze (unless you can tell Jesse it is critical), but I will check in fixes into cvs in a minute or two here. Thanks I fixed this in iscsi-initiator-utils-6_2_0_695-0_6. It is checked in cvs and built into brew. I so not think they are going to let me update the rpm for the initial release of FC6, so as soon as we can do a update I will make sure this gets moved. |
In order for the initscripts to be able to bindmount a tmpfs over /var/lib/iscsi, the directory needs to exist. So, it would be useful if the package created the directories. Also, it's standard practice for packages to own their directories (e.g so rpm -qf /var/lib/iscsi works) suggest: - create /var/lib/iscsi/nodes, /var/lib/iscsi/send_targets and /var/lock/iscsi in %install - add those dirs to the %files list, using the %dir directive You could probably remove the patches that create those dirs at runtime, then.