Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

Reduced the size of processor image #5

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

aimbot31
Copy link
Member

Instead of using python3 full image, switched to python:3.7-slim-buster.

In order to have compatibility between cloudkitty
and kubernetes, we need to have a repository
for processing the config map.
There where a problem with PyMySQL which
where missing.
Updated the readme to explain where to put the config
@aimbot31 aimbot31 added the enhancement New feature or request label Aug 13, 2019
@aimbot31 aimbot31 self-assigned this Aug 13, 2019
Copy link
Contributor

@lukapeschke lukapeschke left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! Dockerfile LGTM, but please add some information to the README, about where to mount config files.

README.md Outdated
- `/etc/cloudkitty/metrics.yml`
- `/etc/cloudkitty/cloudkitty.conf`

Please, refer to the [documentation of cloudkitty](https://docs.openstack.org/cloudkitty/latest/admin/configuration/configuration.html) if you don't know how to use theses files.
Copy link
Contributor

Choose a reason for hiding this comment

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

"theses" -> these. Also, please keep lines < 80chars.

Instead of using python3 full image, switched to python:3.7-slim-buster.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants