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

workflows: avoid unknown/unknown architecture in ghcr #2971

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

KaloyanTanev
Copy link
Contributor

@KaloyanTanev KaloyanTanev commented Mar 16, 2024

Pin buildkit version in all docker/setup-buildx-action@v2 steps (N.B.: at one place v1 is used, which is using buildkit version <0.10.5, so I haven't pinned it there). Other proposed solutions didn't seem to work currently (tested on private repo).

Included as well .vscode in .gitignore.

category: bug
ticket: #2654

Closes #2654

@KaloyanTanev KaloyanTanev self-assigned this Mar 16, 2024
Copy link

sonarcloud bot commented Mar 16, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link

codecov bot commented Mar 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.13%. Comparing base (8b5c004) to head (d1bed15).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2971      +/-   ##
==========================================
- Coverage   54.21%   54.13%   -0.08%     
==========================================
  Files         195      195              
  Lines       27525    27551      +26     
==========================================
- Hits        14922    14916       -6     
- Misses      10842    10875      +33     
+ Partials     1761     1760       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

.gitignore Show resolved Hide resolved
@gsora
Copy link
Collaborator

gsora commented Mar 18, 2024

Added a "Closes" line to the original PR description, so that #2654 gets closed automagically when this PR is merged!

@gsora
Copy link
Collaborator

gsora commented Mar 18, 2024

Hey @KaloyanTanev! When you feel comfortable, add the merge when ready tag to this PR so that our bot, obol-bulldozer can merge it in main :D

@KaloyanTanev KaloyanTanev added the merge when ready Indicates bulldozer bot may merge when all checks pass label Mar 18, 2024
@obol-bulldozer obol-bulldozer bot merged commit aaed89d into main Mar 18, 2024
16 checks passed
@obol-bulldozer obol-bulldozer bot deleted the kalo/remove-unknown-arch-ghcr branch March 18, 2024 22:11
obol-bulldozer bot pushed a commit that referenced this pull request Mar 22, 2024
Some packages installed from the Dockerfile failed after #2971 was merged. Reason was it used default pre-installed version of qemu - the emulator used from amd64 architecture to build the arm64 images. Qemu is suggested by Dockerhub as well.

[Dockerhub](https://docs.docker.com/build/ci/github-actions/multi-platform/)
[Github Issue](docker/buildx#495 (comment))

category: fixbuild
ticket: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge when ready Indicates bulldozer bot may merge when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate unknown/unknown arch in the Docker builds
5 participants