Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

im/monitor_ds.sh skip broken symlinks #100

Merged
merged 1 commit into from
Jun 9, 2016

Conversation

atodorov-storpool
Copy link
Contributor

Most probably my test setup is too weird but if you have
/var/lib/one/datastores shared on all hosts and want to have
SYSTEM datastore with TM_MAD on one of the hosts, I am doing the following:

mkdir -p /var/lib/one-local/102
ln -s /var/lib/one-local/102 /var/lib/one/datastores/102

This way I have "local" datastore nested in "shared" filesystem.

In the above scenario on all other hosts the symlink is dead.

The proposed patch test the is the $dir directory and continue only if it is indeed directory.

Most probably my test setup is too weird but if you have
/var/lib/one/datastores shared on all hosts and want to have 
SYSTEM datastore with TM_MAD on one of the hosts, I am doing the following:

mkdir -p /var/lib/one-local/102
ln -s /var/lib/one/local/102 /var/lib/one/datastores/102

This way I have "local" datastore nested in "shared" filesystem.

In the above scenario on all other hosts the symlink is dead.

The proposed patch test the is the $dir directory and continue only if it is indeed directory.
@jfontan jfontan merged commit dd493d9 into OpenNebula:master Jun 9, 2016
@jfontan
Copy link
Contributor

jfontan commented Jun 9, 2016

Thanks!

@atodorov-storpool atodorov-storpool deleted the patch-4 branch July 4, 2016 20:00
rsmontero pushed a commit that referenced this pull request Jul 23, 2020
rsmontero pushed a commit that referenced this pull request Oct 6, 2020
rsmontero pushed a commit that referenced this pull request Jan 20, 2021
rsmontero pushed a commit that referenced this pull request Apr 13, 2021
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.

None yet

2 participants