Skip to content

microservices: Optimized Time Series Analytics Dockerfile#347

Merged
sathyendranv merged 2 commits intoopen-edge-platform:mainfrom
vkb1:feature/vkb1/reduce_timeseries_container_size
Jun 16, 2025
Merged

microservices: Optimized Time Series Analytics Dockerfile#347
sathyendranv merged 2 commits intoopen-edge-platform:mainfrom
vkb1:feature/vkb1/reduce_timeseries_container_size

Conversation

@vkb1
Copy link
Copy Markdown
Contributor

@vkb1 vkb1 commented Jun 16, 2025

Description

Optimized the Dockerfile to reduce the container image size from 1.56GB to 1.29GB

Any Newly Introduced Dependencies

None

How Has This Been Tested?

Yes, functionally verified the temperature results

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes.
  • I have not introduced any 3rd party components incompatible with APACHE-2.0.
  • I have not included any company confidential information, trade secret, password or security token.
  • I have performed a self-review of my code.

Optimized the Dockerfile to reduce the container image size
from 1.56GB to 1.29GB

Signed-off-by: B, Vinod K <vinod.k.b@intel.com>
@sathyendranv sathyendranv requested a review from Copilot June 16, 2025 09:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Optimizes the Dockerfile for the time-series analytics service by consolidating layers, minimizing installed packages, and cleaning up caches to reduce image size and improve security.

  • Consolidated and streamlined package installation and cleanup steps
  • Switched to sparse checkout for Kapacitor Python plugin and removed unneeded files
  • Added non-root user setup and improved healthcheck
Comments suppressed due to low confidence (2)

microservices/time-series-analytics/Dockerfile:44

  • [nitpick] The PYTHONPATH includes '/tmp/py_package', but this directory is never created or populated in the Dockerfile; consider removing it or ensuring it's generated.
ENV PYTHONPATH="${PYTHONPATH}:/tmp/py_package:/app/kapacitor_python/" \

microservices/time-series-analytics/Dockerfile:16

  • The 'apt-get purge -y --auto-remove' command is missing package names and will fail; specify which packages to purge or use 'apt-get autoremove -y' instead.
apt-get purge -y --auto-remove && \

Comment thread microservices/time-series-analytics/Dockerfile Outdated
Comment thread microservices/time-series-analytics/Dockerfile Outdated
@vkb1 vkb1 requested a review from xwu2intel June 16, 2025 12:53
@sathyendranv sathyendranv merged commit d251cad into open-edge-platform:main Jun 16, 2025
13 checks passed
@sathyendranv sathyendranv deleted the feature/vkb1/reduce_timeseries_container_size branch June 16, 2025 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants