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 testflinger-testenv container #417

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add testflinger-testenv container #417

wants to merge 6 commits into from

Conversation

plars
Copy link
Collaborator

@plars plars commented Nov 21, 2024

Description

When running the test phase for agents, we currently have it run under a docker image pulled from dockerhub at plars/testflinger-testenv-focal (don't be too bothered that it's focal based, it's just to give some working env on the agent-host and this one is well tested).

Now that we're on GH for everything, we should just use ghcr.io, so that nobody has to switch it over to a new location when that bus hits me some day :)

This is the first step in resolving this. I intentionally didn't change the path pointed to by the tf-test script that the agent calls yet, because we need to make sure firewall rules are all adjusted and everything before rolling this out in production.

Resolved issues

N/A

Documentation

There's a small README there just saying what it is, but this doesn't really do anything different, just moves the location

Web service API changes

N/A

Tests

Successful action run: https://github.com/canonical/testflinger/actions/runs/11961073449
Published container image: https://github.com/canonical/testflinger/pkgs/container/testflinger%2Ftestflinger-testenv

I also tried pulling this container image and running it locally.

@plars plars force-pushed the testflinger-testenv branch 2 times, most recently from e57d60d to ef47185 Compare November 21, 2024 20:39
@plars plars requested a review from a team November 21, 2024 20:46
@plars plars force-pushed the testflinger-testenv branch 6 times, most recently from bd09241 to 8d2d3ce Compare November 22, 2024 16:47
@plars plars marked this pull request as draft November 22, 2024 16:53
@plars plars force-pushed the testflinger-testenv branch from 8d2d3ce to 9165857 Compare November 22, 2024 16:56
@plars plars marked this pull request as ready for review November 22, 2024 17:21
@plars
Copy link
Collaborator Author

plars commented Nov 22, 2024

A few more changes:

  • Added some packages that George requested
  • matrix build so that it makes a tagged version of this for focal, jammy, and noble. This way we could try out newer versions if we want.

nancyc12
nancyc12 previously approved these changes Dec 13, 2024
Copy link
Contributor

@nancyc12 nancyc12 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 making this workflow! I only have a small question :)

push:
branches:
- main
- testflinger-testenv
Copy link
Contributor

Choose a reason for hiding this comment

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

should we remove testflinger-testenv?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

you mean the commit that I literally called "WIP: remove this" so that I would remember to rebase and drop it before opening a PR? yes... yes I should remove that. blush thanks for catching that :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

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.

2 participants