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

Arm builder plus split #25

Draft
wants to merge 177 commits into
base: main
Choose a base branch
from
Draft

Arm builder plus split #25

wants to merge 177 commits into from

Conversation

orangewolf
Copy link
Contributor

this is the where we got to before having to stop the self hosted runners because of a cancel build bug

April Rieger and others added 30 commits November 4, 2022 21:14
Updates the actions to pull from the new branch, updates to add new s…
819e8f7 
…elf hosted runner. Updates to add gitguardian api
Updates the actions to pull from the new branch, updates to add new s…
4bb5122 
…elf hosted runner. Updates to add gitguardian api
@aprilrieger
Update gitguardian.yaml
58f6ab4 
update workflow call and workflow dispatch
@aprilrieger
add registry env variable
70c2b81 
env:
  REGISTRY: ghcr.io
update with endpoint defined
876bea7
tweaking per documentation here: actions/actions-runner-controller#89…
1972ee9 
…3 (comment)
tweaks
591e7f6
tweaks
c11f93c
tweaks
3949f96
updates for testing the lint action error
f99ea43
adding tmate debugging session to troubleshoot error with running doc…
3310169 
…ker-compose while using self-hosted runners
@aprilrieger
fix copy pasta
ac45d5b 
fix copy pasta
found typo
65c87b7
gathering more inforamtion to troubleshoot
2d39229
tweaks to see where we are and how we can checkout the package regist…
d350354 
…ry's image
updates to get more information while troubleshooting
37edfb4
more testing and tweaking
d3fcbb6
works locally, lets see if it works in CI now
6c067c1
@aprilrieger
remove tmate
0a52684 
remove tmat
testing fail_on_failure flag and updates the docker compose syntax to…
ecad26a 
… just docker.
remover the confdir
e1128d2
remover the confdir
3966f17
Updatung the retag to print out some information for troubleshooting,…
224c6d6 
… i forgot to take the other docker run exec to just docker run, and adding back the two commas after receiving feedback, thanks Rob, totally thought it was a typo
move the -T flag not seeing that tty is set by default in the docker …
9c34970 
…exec command equivalent
prepping to merge into main
94f451c
Update readme, clean up syntax, add fall back env.tag, replace github…
6503626 
….repository with REPO_LOWER, inputs.worker need to be evaluated since boolean, adds inputs.confdir so we can pass it from the caller, moves the block to be in order of operation to help for visualization
didn't save in order to push
d46a16c
remove the tag since we are tagging it latest.
b1a9114
latest tag, add it to the image versus declaring it as a variable bc …
d0654b1 
…it's not passed as latest
@aprilrieger
Update build.yaml
94209d0 
inputs.worker is already a boolean no need to specify == 'true'
orangewolf and others added 30 commits November 15, 2022 22:51
@orangewolf
make sure .env is present
26c65eb
@bkiahstroud
Merge pull request #14 from scientist-softserv/arm-y_of_one
8a45bdd 
Use the dedicated ARM builder for ARM images
@bkiahstroud
Merge branch 'main' into add-ability-to-specify-confdir
ea5ba82
@bkiahstroud
Merge pull request #11 from scientist-softserv/add-ability-to-specify…
fd2d7c2 
…-confdir

add-ability-to-specify-confdir
@bkiahstroud
move duplicate confdir input from dispatch to call
258d2d8
@bkiahstroud
Merge pull request #15 from scientist-softserv/fix-confdir-in-test
c45636a 
move duplicate confdir input from dispatch to call
@orangewolf
update arm ip
3c099f0
@orangewolf
Make sure schema is writeable
bb93018
@orangewolf
make sure schema is writeable
93e4198
@orangewolf
solr cloud steps on solr cloud only
f3140ba
@orangewolf
no bash, no problems
f0ea115
@orangewolf
ok, some problems
099d481
@aprilrieger
updates to macos-11 github runner testing arm64 build
45de035
@aprilrieger
Updates to add specific platform and services to split out builds to …
02ef9a4 
…try and help speed up the build process for deployments
@aprilrieger
updates syntax to clear error
077f082
@aprilrieger
updates actions to split platforms and web/worker/solr
0f58c61
@aprilrieger
adds repo env variable
3e852fc
@aprilrieger
set env repo
fd7057c
@aprilrieger
syntax
62a6bfa
@aprilrieger
repo_lower
d404266
@aprilrieger
lowercase
4ea5dd5
@aprilrieger
tweaking syntax
9e8a12f
@aprilrieger
macos-11 doesnt like doencase - swap back to ubuntu-latest
92f753d
@aprilrieger
add back qemu on ubuntu-latest
370a686
@aprilrieger
add the right env back
55fe375
@aprilrieger
add the right env back -- for realz
d9b20be
@aprilrieger
setup goes back to main
fce30d1
@aprilrieger
updates to ensure chmod is added to the test file from main
242541d
@aprilrieger
Merge pull request #18 from scientist-softserv/arm-wrestle
ce802ab 
Arm wrestle
@aprilrieger
dont tag image its set automatically to latest
8b4b50c 
removes tag on end of image as the action defaults it to latest which is what we want
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@orangewolf @aprilrieger @labradford @bkiahstroud