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.

Sign up for GitHub

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

Jump to bottom

attempt to run k8s in gha #873

Merged
merged 43 commits into from
Aug 16, 2024
Merged

attempt to run k8s in gha #873

merged 43 commits into from
Aug 16, 2024

Conversation

hahn-kev
Copy link
Collaborator

not sure why I didn't try this sooner

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 10, 2024 •
edited
Loading

C# Unit Tests

54 tests  ±0   54 ✅ ±0   6s ⏱️ +2s
11 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 3306112. ± Comparison against base commit 8a2753d.

♻️ This comment has been updated with latest results.

megahirt
megahirt reviewed Jun 11, 2024
View reviewed changes
Copy link
Contributor

@megahirt megahirt left a comment

Choose a reason for hiding this comment

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

Woohoo for GHA! Glad to see that you have gotten K8s to run in GHA. I hope this will make our automated testing more robust in the future.

myieye and others added 28 commits June 13, 2024 10:34
@myieye
Make local.env creation idempotent in task setup
a67e9ce
@hahn-kev @myieye
initial workflow attempt
4cd0ee2
@hahn-kev @myieye
add pr trigger
fe26583
@hahn-kev @myieye
fix help kind version
c11d3f5
@hahn-kev @myieye
only run codeql when code changes
e267c0f
@hahn-kev @myieye
use local-dev instead of develop since cert-manager is missing
93f0d9a
@hahn-kev @myieye
breakout local.env setup and call in k8s workflow to fix issue where …
8fccda1 
…file was missing
@hahn-kev @myieye
create languagedepot namespace before applying
bb5797d
@hahn-kev @myieye
correct waiit action to wait for pods and select by label, specify na…
a189893 
…mespace since default ns isn't where we want to check
@hahn-kev @myieye
add missing namespace
86a9dbe
@hahn-kev @myieye
use skaffold to setup cluster, run dotnet test with GHA logger
d2bbb69
@hahn-kev @myieye
setup better failure logs, remove skaffold since we don't want to bui…
db4001e 
…ld anything
@hahn-kev @myieye
switch from kind to mini kube because of PVC issues
7e1ace2
@hahn-kev @myieye
introduce a kustomize layer for gha, change storage class used
6fff47c
@hahn-kev @myieye
correct name of kustomize file, change back to kind
6467ead
@hahn-kev @myieye
pull image with docker before pulling with kind
44e278f
@hahn-kev @myieye
donn't attempt to load images via kind
fed1f60
@hahn-kev @myieye
change local-dev-init back to use busybox, increase timeout when wait…
80be6b1 
…ing for deployments
@hahn-kev @myieye
prevent doubl prefixing busybox with the tag
a495240
@hahn-kev @myieye
increase log tail length to see error
f300cfd
@hahn-kev @myieye
add frontend folder to gha for dotnet to write the schema to.
1a1118b
@hahn-kev @myieye
ensure that kind forwards localhost:80 to ingress in gha
65a5673
@hahn-kev @myieye
ensure we wait for ingress to be ready
a373210
@hahn-kev @myieye
wait for app.kubernetes.io/component=controller
7dc0444
@hahn-kev @myieye
use port forwarding to access ingress
258d916
@hahn-kev @myieye
explicitly build and restore to ensure that MercurialExtensions is found
b1db609
@hahn-kev @myieye
disable export to honeycomb in gha
f8dade4
@hahn-kev @myieye
rename workflow
f995fd8
@myieye myieye force-pushed the chore/run-k8s-in-gha branch from b2e1a17 to 653d8ed Compare June 13, 2024 08:37
@megahirt
Copy link
Contributor

megahirt commented Aug 7, 2024

I think seeing this run in GHA is amazing :)

@hahn-kev hahn-kev requested a review from rmunn August 15, 2024 07:54
rmunn
rmunn previously approved these changes Aug 15, 2024
View reviewed changes
Copy link
Contributor

@rmunn rmunn left a comment

Choose a reason for hiding this comment

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

Great idea! Kudos for figuring out how to use kind to get this done.

@hahn-kev hahn-kev merged commit 4e675f0 into develop Aug 16, 2024
9 checks passed
@hahn-kev hahn-kev deleted the chore/run-k8s-in-gha branch August 16, 2024 10:15
@rmunn rmunn mentioned this pull request Aug 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@megahirt megahirt megahirt left review comments

@rmunn rmunn rmunn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hahn-kev @megahirt @rmunn @myieye