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

chore: revert change that skip arm64 image build #48

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

Faraz32123
Copy link
Collaborator

@Faraz32123 Faraz32123 commented Oct 22, 2024

In this PR, IMAGES_BUILD_PLATFORM setting is removed. This setting was initially added to prevent the credentials image from being built for arm64 due to issues with the didkit package on arm64 machines. Now, the image can be built for both amd64 and arm64 architectures.

This commit is already merged in nightly in this PR for sumac.
And this PR is made for redwood. Once redwood.3 is available, this PR will be merged.

- As didkit issue is resolved in latest version which is being used in
upstream credential repo. So, reverting this change.
for context, see this comment: #34 (comment)
Copy link
Contributor

@DawoudSheraz DawoudSheraz left a comment

Choose a reason for hiding this comment

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

to be merged once redwood.3 is available

@DawoudSheraz
Copy link
Contributor

Please add PR description and the link to nightly PR.

@Faraz32123 Faraz32123 force-pushed the chore/revert_arm64_build_skip_change_in_redwood branch from f81fcce to 91d56b3 Compare October 22, 2024 11:35
@Faraz32123 Faraz32123 merged commit 1ac031e into master Oct 24, 2024
2 checks passed
@Faraz32123 Faraz32123 deleted the chore/revert_arm64_build_skip_change_in_redwood branch October 24, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants