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

Update Github CI #1393

Merged
merged 79 commits into from Mar 15, 2023
Merged

Update Github CI #1393

merged 79 commits into from Mar 15, 2023

Conversation

hainest
Copy link
Contributor

@hainest hainest commented Mar 2, 2023 •
edited

Run on pull request to master:

  • Enforce formatting of CMake files, if modified
  • Check for changes in minimum required versions of dependencies
  • Ensure Dyninst, test suite, examples, and external tests all build with gcc on Ubuntu-20.04

Run on push to master:

  • Build and deploy new dev container, ghcr.io/dyninst/amd64-:latest

@hainest hainest added the CI Related to continuous integration testing label Mar 2, 2023
@hainest hainest requested review from kupsch and bigtrak March 2, 2023 18:08
@hainest hainest self-assigned this Mar 2, 2023
@hainest
Update CMake formatting check
13aaba2
@hainest
Make generic base container for Ubuntu distros
c3c9743
@hainest
Add temp Dockerfile to build Dyninst
7b0dafe
@hainest
Rewrite build script
6407f60
@hainest
Remove testsuite stuff from docker images
5aaf881 
This will be done in the CI workflow instead.
@hainest
Update README
1c8b3df
@hainest
Remove test workflow
bb59e07 
The testsuite doesn't run on Github CI because... reasons.
@hainest
Add PR build tests
7331ac7
@hainest
Add dependency version checks
9c67f99
@hainest
Disable release workflow
38d0309 
The release process will (eventually) be a copy of the container created
by the Dockerfile. In essence, just a regular Dyninst image but at a fixed
version.
@hainest
Rename base-containers.yaml to dev-containers.yaml
2ca4e18 
This reflects its new purpose
@hainest
Update dev container creation
19bdc89
@hainest
Add comments about dependency version checks
ca5a49c
@hainest
Use Dyninst base container when running build tests
4f8acb5
@hainest
Check out Dyninst in /dyninst when doing build tests
ca1189e
@hainest
Fix some spelling items in build-tests.yaml
e72ae9e
@hainest
Use checkout@v3 in dev-containers.yaml
7073b5f
@hainest
Install clang in base Ubuntu container
1693f3f
@hainest
Return error when dependency version check fails
3ee32f5
@hainest
Rename 'Build Tests' workflow to 'Pull Request Tests'
bf9a79f 
It better reflects what's happening
@hainest
Use latest Github ubuntu image for cmake formatting job
b5a1c2f 
There's no need for it to be a specific version.
@hainest
Use only dyninst org when building dev containers
bc88424 
If someone forks our repo, they can re-write the workflows if they want them to work for their fork.
@hainest
Remove pull request checks in dev-containers
805695e 
This is only ever run when we push to master, so the checks are redundant.
@hainest
Fix job name in dev-containers
bc9d4df
@hainest
Add debugging to pr-tests
97a1492
@hainest hainest merged commit 0f5558a into dyninst:master Mar 15, 2023
@hainest hainest deleted the thaines/docker_refactor branch March 15, 2023 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kupsch kupsch Awaiting requested review from kupsch

@bigtrak bigtrak Awaiting requested review from bigtrak

Assignees

@hainest hainest

Labels
CI Related to continuous integration testing
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@hainest