Skip to content

[actions] Add build-and-test github action#1054

Merged
JamesMBartlett merged 1 commit intopixie-io:mainfrom
JamesMBartlett:tests_github_actions
May 9, 2023
Merged

[actions] Add build-and-test github action#1054
JamesMBartlett merged 1 commit intopixie-io:mainfrom
JamesMBartlett:tests_github_actions

Conversation

@JamesMBartlett
Copy link
Copy Markdown
Member

@JamesMBartlett JamesMBartlett commented Mar 14, 2023

Summary: Adds a github action to run builds and tests, including bpf tests. The action uses the configs defined in ci/github/bazelrc along with the bazel_build_deps.sh script to decide what tests to run and under what configurations to run them.

Type of change: /kind test-infra

Test Plan: Tested in this PR that all the tests (including BPF tests) work. Also, tested that failing tests are reported properly, by adding an ASSERT_EQ(0, 1) only in opt configs, and saw that only opt configs failed.

@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch 30 times, most recently from dbea812 to ae84263 Compare March 16, 2023 22:13
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch 3 times, most recently from 71ca72c to 6fde2a5 Compare April 10, 2023 17:58
@JamesMBartlett JamesMBartlett changed the title [draft][do not review] Testing k8s actions runner [actions] Add build-and-test github action Apr 10, 2023
@JamesMBartlett JamesMBartlett marked this pull request as ready for review April 10, 2023 22:58
@JamesMBartlett JamesMBartlett requested a review from a team as a code owner April 10, 2023 22:58
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch 6 times, most recently from 0762552 to 7445b27 Compare April 19, 2023 21:17
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch 4 times, most recently from fbb7131 to b2affee Compare April 26, 2023 17:43
ddelnano pushed a commit to ddelnano/pixie that referenced this pull request May 1, 2023
Summary: We don't want to use buildbuddy from github actions directly
because of potential egress/ingress costs from github actions to gcloud.
Instead once pixie-io#1108 lands, self-hosted runners will have access to
buildbuddy. This PR replaces adding the buildbuddy api key with adding
github specific bazelrc options.

Type of change: /kind test-infra

Test Plan: Tested that these configs work in pixie-io#1054

Signed-off-by: James Bartlett <jamesbartlett@pixielabs.ai>
@JamesMBartlett JamesMBartlett marked this pull request as draft May 3, 2023 21:12
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch 4 times, most recently from 40063f8 to 7294a45 Compare May 4, 2023 16:29
@JamesMBartlett JamesMBartlett marked this pull request as ready for review May 4, 2023 16:29
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch from 7294a45 to 6ab92f1 Compare May 5, 2023 23:19
@JamesMBartlett JamesMBartlett requested a review from a team May 5, 2023 23:22
Comment thread ci/github/matrix.yaml Outdated
Run #ci:bpf-build.

Signed-off-by: James Bartlett <jamesbartlett@pixielabs.ai>
@JamesMBartlett JamesMBartlett force-pushed the tests_github_actions branch from 6ab92f1 to 2ad4962 Compare May 9, 2023 18:20
@JamesMBartlett JamesMBartlett merged commit 18611a1 into pixie-io:main May 9, 2023
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