Skip to content
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

Add Dockerfile to create an OCI image of agent #179

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

hanhsuan
Copy link
Contributor

Description

Add Dockerfile to create an OCI image includes

  1. testflinger-agent
  2. testflinger-cli
  3. device-connector

and could change the settings by environment variables while starting.

Resolved issues

Documentation

Tests

1. testflinger-agent
2. testflinger-cli
3. device-connector
and could change the set by environment variables while starting.
@nancyc12
Copy link
Contributor

nancyc12 commented Dec 21, 2023

/canonical/self-hosted-runners/run-workflows 802441a

@plars
Copy link
Collaborator

plars commented Dec 21, 2023

Can you tell me more about what this is for? I'd be happy to see something like this as a mechanism for integration testing, but for actual use - it's a bad pattern to have the entire agent stack run inside a container. In fact, most of the device connectors will not work well like this currently. What I would recommend for actual use, is to have only the test phase run inside a container. This way you get a clean, reproducible environment every time, while still allowing the other phases to operate in a less restricted way.

@hanhsuan
Copy link
Contributor Author

As discuss and the conclusion in Riga, OEM team will only use test command for now and ask me to put what OEM team has into this repository.

Copy link
Collaborator

@plars plars left a comment

Choose a reason for hiding this comment

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

After a bit more discussion today, here are some changes we are recommending in order to merge this:

  1. Move this to agent/extra instead of agent/docker - this gives a better indication that this is "extra" and not needed for general use
  2. Document at the top of the README that this is for a specific use case and that most device connectors will not function properly this way, so it's not recommended for use in most cases.

@hanhsuan
Copy link
Contributor Author

May I move to agent/extra/docker ? I think it might be more clear to know it is the extra container related items.

@plars
Copy link
Collaborator

plars commented Jan 11, 2024

@hanhsuan yes, that should work

2. Notice user this is for a specific use case in README
@hanhsuan hanhsuan requested a review from plars January 11, 2024 07:08
@plars
Copy link
Collaborator

plars commented Jan 11, 2024

/canonical/self-hosted-runners/run-workflows 9e10be3

@plars plars merged commit b86ef6d into canonical:main Jan 11, 2024
2 checks passed
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