Skip to content

feat(auth):Reducing Image size of docker container image for pullpiri[#116]#135

Merged
youngtaekiim merged 4 commits into
eclipse-pullpiri:refactoringfrom
akshaylg0314:refactoring
Jul 21, 2025
Merged

feat(auth):Reducing Image size of docker container image for pullpiri[#116]#135
youngtaekiim merged 4 commits into
eclipse-pullpiri:refactoringfrom
akshaylg0314:refactoring

Conversation

@akshaylg0314
Copy link
Copy Markdown
Contributor

PR Description
Build and push Container image multi arch using github workflow

🔗 Related Issue
Closes #116

🧪 Test Method
Github Action should run and check based on the Code change.
Release Process will trigger through:-
git tag v1.0.0
git push origin v1.0.0

Below is the link for GitHub Action run for the pull request from the CD-Pipeline branch of my fork to the refactoring branch:
https://github.com/akshaylg0314/pullpiri/actions/runs/16366633243

Dummy uploaded multi-architecture container images to GHCR, using these changes:
🔗 Example overall pullpiri Image(server+agent+player)

This demonstrates the release automation pipeline is functioning correctly — including license report generation, artifact uploads, and QueVee metadata integration and build and push container to ghcr

✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)

@akshaylg0314
Copy link
Copy Markdown
Contributor Author

Hey @youngtaekiim,
I've made the changes as per your requirements to reduce the image size.
Now, I'm able to build and push a single multi-arch Docker image for pullpiri that includes all required binaries:

  • apiserver
  • monitoringserver
  • policymanager
  • nodeagent
  • actioncontroller
  • filtergateway
  • statemanager

Please review the changes and let me know if any modifications are needed.

Here's the resulting image with significantly reduced size:

acrn@acrn-NUC11TNHi5:~$ docker images | grep cicd_pullpiri
ghcr.io/akshayg0314/cicd_pullpiri   v1.0.66   9133788f9d80   2 hours ago   50.8MB

Extracted contents from the image:

acrn@acrn-NUC11TNHi5:~/extracted$ ls
bin  dev  etc  home  lib  lib64  media  mnt  opt  piccolo  proc  root  run  sbin  srv  sys  tmp  usr  var

acrn@acrn-NUC11TNHi5:~/extracted/piccolo$ ls
actioncontroller  apiserver  filtergateway  monitoringserver  nodeagent  policymanager  settings.yaml  statemanager

Thanks!!

@youngtaekiim youngtaekiim self-assigned this Jul 21, 2025
@youngtaekiim youngtaekiim self-requested a review July 21, 2025 00:37
@akshaylg0314
Copy link
Copy Markdown
Contributor Author

Hey @youngtaekiim,
Could you please share the requirements and scope for the following components — just for my reference and better understanding?

  • nodeagent
  • policymanager
  • statemanager

Thanks!!!

@youngtaekiim
Copy link
Copy Markdown
Contributor

Hey @youngtaekiim,

Could you please share the requirements and scope for the following components — just for my reference and better understanding?

  • nodeagent

  • policymanager

  • statemanager

Thanks!!!

very naive text

nodeagent : in each node, collect information (and some other work)
policymanager : before actioncontroller start action, check the operation is suitable
statemanager : collect node information and reconcile operation

the module view diagram will be updated in documents

@akshaylg0314
Copy link
Copy Markdown
Contributor Author

Hey @youngtaekiim,
Could you please share the requirements and scope for the following components — just for my reference and better understanding?

  • nodeagent
  • policymanager
  • statemanager

Thanks!!!

very naive text

nodeagent : in each node, collect information (and some other work) policymanager : before actioncontroller start action, check the operation is suitable statemanager : collect node information and reconcile operation

the module view diagram will be updated in documents

@youngtaekiim Thanks for the basic information i will go through the docs once updated..
Can you please review this PR changes? Is there any changes required

Thanks

Comment thread .github/workflows/build-container.yml
@youngtaekiim youngtaekiim merged commit 84c7d52 into eclipse-pullpiri:refactoring Jul 21, 2025
6 checks passed
chaehee-lim pushed a commit to chaehee-lim/pullpiri that referenced this pull request Sep 3, 2025
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.

3 participants