-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
ceph: fix topologyAware support #4099
Conversation
871d42e
to
6c5f456
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a better commit message. Thanks.
6c5f456
to
35b0978
Compare
35b0978
to
3d84f90
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix the unit tests.
3d84f90
to
ffb4c74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could simplify by only using envVars for all containers instead of maintaining a separate configEnvVars, but we can leave that discussion for another day...
add missing topologyAware and location env variables to OSD pods Signed-off-by: Mateusz Los <los.mateusz@gmail.com>
ffb4c74
to
cb52aea
Compare
ceph: fix topologyAware support (bp #4099)
add missing env variables to OSD pods
Signed-off-by: Mateusz Los los.mateusz@gmail.com
Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #4101
Checklist:
make codegen
) has been run to update object specifications, if necessary.[test ceph]