bird-house / birdhouse-deploy

Scripts and configurations to deploy the various birds and servers required for a full-fledged production platform
https://birdhouse-deploy.readthedocs.io/en/latest/
Apache License 2.0
4 stars 6 forks source link

Bump cadvisor version to the latest version: v0.49.1 #458

Closed mishaschwartz closed 5 months ago

mishaschwartz commented 6 months ago

Overview

Changes

Non-breaking changes

Breaking changes

Related Issue / Discussion

Additional Information

Links to other issues or sources.

CI Operations

birdhouse_daccs_configs_branch: master birdhouse_skip_ci: false

mishaschwartz commented 6 months ago

The new version is backward compat with the settings we currently have in the docker-compose-extra.yml? And it plays nice with the older existing Prometheus?

Yes it seems to based on my tests on my local test machine

tlvu commented 6 months ago

Just to be sure, the new cadvisor version will still work with older docker-engine as well?

mishaschwartz commented 5 months ago

@tlvu

Just to be sure, the new cadvisor version will still work with older docker-engine as well?

Theoretically yes. I can try to test it out but I don't have any machines that are running the old version, so it might take me some time to configure a VM to test it. If you have VMs that are running an older version please feel free to check it out too

mishaschwartz commented 5 months ago

Just to be sure, the new cadvisor version will still work with older docker-engine as well?

Personally, I would rather the default support the modern version fo docker engine since most new deployments will use a relatively new version anyway.

If you need to support a legacy version of docker engine then you can always change the version of cadvisor in your deployment since the version is easily configurable since #457

tlvu commented 5 months ago

change the version of cadvisor in your deployment since the version is easily configurable since #457

Ah true, forgot about that one. Merge away then. We can override if needed.