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

[chore] Add Docker test pipeline #837

Merged
merged 3 commits into from
Mar 5, 2025

Conversation

mowies
Copy link
Member

@mowies mowies commented Feb 25, 2025

This PR adds steps to the CI pipeline setup that run all built container images with a small default config that is mounted into the container as a volume.
The pipeline then check if the Everything is ready. message is logged to make sure that the collector started up correctly.

Fixes #786

@mowies mowies changed the title Add Docker test pipeline [chore] Add Docker test pipeline Mar 4, 2025
mowies added 3 commits March 4, 2025 09:43
Signed-off-by: Moritz Wiesinger <[email protected]>

try to build images again

Signed-off-by: Moritz Wiesinger <[email protected]>

ignore darwin ppc64le build

Signed-off-by: Moritz Wiesinger <[email protected]>

try with conditional

Signed-off-by: Moritz Wiesinger <[email protected]>

fix expression

Signed-off-by: Moritz Wiesinger <[email protected]>

try again

Signed-off-by: Moritz Wiesinger <[email protected]>

debugging

Signed-off-by: Moritz Wiesinger <[email protected]>

and again

Signed-off-by: Moritz Wiesinger <[email protected]>

disable windows/ppc64le

Signed-off-by: Moritz Wiesinger <[email protected]>

remove ppc64le from darwin and windows builds

Signed-off-by: Moritz Wiesinger <[email protected]>

more debugging

Signed-off-by: Moritz Wiesinger <[email protected]>

try agian

Signed-off-by: Moritz Wiesinger <[email protected]>

again

Signed-off-by: Moritz Wiesinger <[email protected]>

more

Signed-off-by: Moritz Wiesinger <[email protected]>

try using tojson and not fromjson

Signed-off-by: Moritz Wiesinger <[email protected]>

extract data from files directly

Signed-off-by: Moritz Wiesinger <[email protected]>

finally getting somewhere

Signed-off-by: Moritz Wiesinger <[email protected]>

more jq changes

Signed-off-by: Moritz Wiesinger <[email protected]>

more variables

Signed-off-by: Moritz Wiesinger <[email protected]>

remove newlines

Signed-off-by: Moritz Wiesinger <[email protected]>

remove quotes

Signed-off-by: Moritz Wiesinger <[email protected]>

try removing quotes in jq commands

Signed-off-by: Moritz Wiesinger <[email protected]>

extract goos and goarch from tar.gz artifacts

Signed-off-by: Moritz Wiesinger <[email protected]>

use correct path

Signed-off-by: Moritz Wiesinger <[email protected]>

try with different conditional

Signed-off-by: Moritz Wiesinger <[email protected]>

adjust paths again

Signed-off-by: Moritz Wiesinger <[email protected]>

try adding quotes

Signed-off-by: Moritz Wiesinger <[email protected]>

try different conditional

Signed-off-by: Moritz Wiesinger <[email protected]>

dfebugging

Signed-off-by: Moritz Wiesinger <[email protected]>

sanity check

Signed-off-by: Moritz Wiesinger <[email protected]>

fix conditional, try to fix quotes

Signed-off-by: Moritz Wiesinger <[email protected]>

copy binary before building image

Signed-off-by: Moritz Wiesinger <[email protected]>

add first docker test steps

Signed-off-by: Moritz Wiesinger <[email protected]>

fix matrix reference

Signed-off-by: Moritz Wiesinger <[email protected]>

reduce pipeline debugging demand

Signed-off-by: Moritz Wiesinger <[email protected]>

exclude darwin and windows docker tests

Signed-off-by: Moritz Wiesinger <[email protected]>

remove darwin for now

Signed-off-by: Moritz Wiesinger <[email protected]>

debugging of paths

Signed-off-by: Moritz Wiesinger <[email protected]>

fix path

Signed-off-by: Moritz Wiesinger <[email protected]>

change from docker ps to docker logs

Signed-off-by: Moritz Wiesinger <[email protected]>

set entrypoint manually

Signed-off-by: Moritz Wiesinger <[email protected]>

change from import to load

Signed-off-by: Moritz Wiesinger <[email protected]>

almost there

Signed-off-by: Moritz Wiesinger <[email protected]>

fix version

Signed-off-by: Moritz Wiesinger <[email protected]>

set name explicitely

Signed-off-by: Moritz Wiesinger <[email protected]>

more output

Signed-off-by: Moritz Wiesinger <[email protected]>

redirect stderr to stdout

Signed-off-by: Moritz Wiesinger <[email protected]>

better output

Signed-off-by: Moritz Wiesinger <[email protected]>

more debug output

Signed-off-by: Moritz Wiesinger <[email protected]>

fix newlines

Signed-off-by: Moritz Wiesinger <[email protected]>

use special test config file

Signed-off-by: Moritz Wiesinger <[email protected]>

forgot backslash...

Signed-off-by: Moritz Wiesinger <[email protected]>

add pwd

Signed-off-by: Moritz Wiesinger <[email protected]>

do loopback

Signed-off-by: Moritz Wiesinger <[email protected]>

Revert "reduce pipeline debugging demand"

This reverts commit b858523.

remove maybe unneeded steps

Signed-off-by: Moritz Wiesinger <[email protected]>

add back download step

Signed-off-by: Moritz Wiesinger <[email protected]>

add back qemu action

Signed-off-by: Moritz Wiesinger <[email protected]>

add back checkout step

Signed-off-by: Moritz Wiesinger <[email protected]>

add back qemu step

Signed-off-by: Moritz Wiesinger <[email protected]>

revert removal of windows/ppc64le variant

Signed-off-by: Moritz Wiesinger <[email protected]>

revert matrix configs

Signed-off-by: Moritz Wiesinger <[email protected]>
Signed-off-by: Moritz Wiesinger <[email protected]>
@mowies mowies marked this pull request as ready for review March 4, 2025 08:46
@mowies mowies requested a review from a team as a code owner March 4, 2025 08:46
Copy link
Member

@mx-psi mx-psi left a comment

Choose a reason for hiding this comment

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

Looks reasonable, it may flake if the Collector takes too long to start up but I guess it's also good to be aware of that possibility. Thanks!!

@mx-psi mx-psi added this pull request to the merge queue Mar 5, 2025
Merged via the queue into open-telemetry:main with commit ce90044 Mar 5, 2025
75 checks passed
@mowies mowies deleted the docker-tests branch March 5, 2025 10:53
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.

Add test for running Docker images
2 participants