Skip to content

(#22665) mounts should autorequire parent mounts#1935

Merged
pcarlisle merged 1 commit intopuppetlabs:masterfrom
adrienthebo:issue_22665-require_mount_parents
Oct 3, 2013
Merged

(#22665) mounts should autorequire parent mounts#1935
pcarlisle merged 1 commit intopuppetlabs:masterfrom
adrienthebo:issue_22665-require_mount_parents

Conversation

@adrienthebo
Copy link
Contributor

Given two mount points:

mount { "/var/log":
  # ...
}

mount { "/var":
  # ...
}

The mount '/var/log' should autorequire '/var' so that the child mount
is not masked by the parent mount.

@adrienthebo
Copy link
Contributor Author

This supersedes GH-1849.

@puppetcla
Copy link

CLA signed by all contributors.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alternately, expect(var_mount.autorequire).to have_exactly(1).item

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing this out, making this change caught a bug in the test.

@adrienthebo
Copy link
Contributor Author

Expectations on array sizes fixed, tyop removed.

Given two mount points:

    mount { "/var/log":
      # ...
    }

    mount { "/var":
      # ...
    }

The mount '/var/log' should autorequire '/var' so that the child mount
is not masked by the parent mount.
pcarlisle added a commit that referenced this pull request Oct 3, 2013
…arents

(#22665) mounts should autorequire parent mounts
@pcarlisle pcarlisle merged commit 8ec622c into puppetlabs:master Oct 3, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants