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.

Sign up for GitHub

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

Jump to bottom

Corrected RADOS storage issues in validation #136

Merged
merged 1 commit into from
Jan 27, 2020
Merged

Corrected RADOS storage issues in validation #136

merged 1 commit into from
Jan 27, 2020

Conversation

sabre1041
Copy link
Contributor

@sabre1041 sabre1041 commented Jan 22, 2020 •
edited
Loading

Corrected misconfiguration of variables for RADOS for #133

kurtismullins
kurtismullins approved these changes Jan 27, 2020
View reviewed changes
Copy link
Contributor

@kurtismullins kurtismullins left a comment

Choose a reason for hiding this comment

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

LGTM

I am not entirely sure if this will solve the addressed issue because I do not have a ceph environment standing to verify against, but these changes look like they were required regardless.

@kurtismullins kurtismullins merged commit c32464f into quay:master Jan 27, 2020
@sabre1041 sabre1041 deleted the rados-storage-fix branch April 3, 2020 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kurtismullins kurtismullins kurtismullins approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sabre1041 @kurtismullins