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

align docker merge/pull/push github workflows so that CI/CD does not break #288

Open
3 tasks done
rockett-m opened this issue Jul 21, 2024 · 2 comments
Open
3 tasks done
Assignees
Labels
fix/bug Fixes errant behavior

Comments

@rockett-m
Copy link
Contributor

rockett-m commented Jul 21, 2024

Affected Branch

trunk

Basic Diagnostics

  • I've pulled the latest changes on the affected branch and the issue is still present.

  • The issue is reproducible in docker

Description

On commits 210 and 211, all the 7/7 checks passed but docker merge failed after the commits were merged into trunk. We should never have this happen. Commit 212 resolved it for the time being, but we want to future-proof this.
Our docker pre-merge checks need to have equivalent steps to the docker merge stage to avoid unexpected failures after merge takes place. The docker-pull stage lags docker-merge.

We can see results of the docker merge github workflow on a push to our local branch. It can be replicated - adding an additional check to the 7 standard checks - by adding the last three lines below to .github/workflows/docker-merge.yml

on:
  push:
    branches:
      - 'trunk'
  pull-request:
    branches:
      - 'trunk'

To test github workflows install act and run act --list and then

act -j docker-build --container-architecture linux/amd64

cc discussion with @HalosGhost + @maurermi

Code of Conduct

  • I agree to follow this project's Code of Conduct
@rockett-m rockett-m added the fix/bug Fixes errant behavior label Jul 21, 2024
@rockett-m
Copy link
Contributor Author

@HalosGhost I'm happy to take this one on

@HalosGhost HalosGhost assigned rockett-m and unassigned HalosGhost Jul 21, 2024
@maurermi
Copy link
Collaborator

@rockett-m please feel free, if you could update us here with what you see in your investigation, that would be great!

rockett-m added a commit to rockett-m/opencbdc-tx that referenced this issue Jul 26, 2024
rockett-m added a commit to rockett-m/opencbdc-tx that referenced this issue Jul 27, 2024
@HalosGhost HalosGhost linked a pull request Jul 27, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix/bug Fixes errant behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants