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

tests: allow docker-test.sh to run under root #6173

Merged
2 commits merged into from Oct 8, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

commented Oct 3, 2015

No description provided.

tests: remove fedora docker files
They are not used (missing version number at the end of the directory).

Signed-off-by: Loic Dachary <ldachary@redhat.com>

@ghost ghost added feature tests labels Oct 3, 2015

@ghost ghost assigned smithfarm Oct 3, 2015

tests: allow docker-test.sh to run under root
Do not attempt to create a user if it is root because it already exists.

http://tracker.ceph.com/issues/13355 Fixes: #13355

Signed-off-by: Loic Dachary <ldachary@redhat.com>
@smithfarm

This comment has been minimized.

Copy link
Contributor

commented Oct 4, 2015

@dachary: The machine I was testing this on suddenly went AWOL (Absent With Out Leave - a military acronym). Will keep you posted.

@ghost

This comment has been minimized.

Copy link
Author

commented Oct 4, 2015

@smithfarm ack, no rush ;-)

@ghost

This comment has been minimized.

Copy link
Author

commented Oct 7, 2015

@smithfarm ping ?

@smithfarm

This comment has been minimized.

Copy link
Contributor

commented Oct 8, 2015

@ghost

This comment has been minimized.

Copy link
Author

commented Oct 8, 2015

@smithfarm updated the documentation: it was indeed misleading.

@smithfarm smithfarm assigned ghost and unassigned smithfarm Oct 8, 2015

@smithfarm

This comment has been minimized.

Copy link
Contributor

commented Oct 8, 2015

LGTM

ghost pushed a commit that referenced this pull request Oct 8, 2015

Loic Dachary
Merge pull request #6173 from dachary/wip-13355-docker-test
tests: allow docker-test.sh to run under root

Reviewed-by: Nathan Cutler <ncutler@suse.cz>

@ghost ghost merged commit 1e8cb9b into ceph:master Oct 8, 2015

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.