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

feat: Re-enable image builds for arm64 for redwood #2605

Conversation

Faraz32123
Copy link

  • The maintainers of didkit have released an update that fixes an issue where it would not build on arm64-based systems. This resolves an issue with Devstack and developers running devstack on Apple silicon-based machines.
  • this fix is already merged in master, backporting this fix to work with redwood.

@Faraz32123 Faraz32123 requested a review from a team as a code owner October 17, 2024 12:08
- The maintainers of `didkit` have released an update that fixes an issue where it would not build on arm64-based systems.
This resolves an issue with Devstack and developers running devstack on Apple silicon-based machines.
- this fix is merged in master, backporting this fix to work with redwood.
openedx#2546
Copy link

@cmltaWt0 cmltaWt0 left a comment

Choose a reason for hiding this comment

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

@Faraz32123 are we sure we want to backport a latest master feature deps along with the simple didkit version change?
In the PR we are actually adding new deps like google-cloud-firestore - it's related to a newly introduced Verifiable Credentials feature that doesn't exists in the Redwood release.

I don't feel like it's good for us to introduce in the redwood release any new deps w/o testing.

@Faraz32123
Copy link
Author

[feat: Re-enable image builds for arm64 for redwood](/openedx/credentials/pull/2605/commits/0ae78186071a67c8d398f349f873fddadb72c02f)
[0ae7818](/openedx/credentials/pull/2605/commits/0ae78186071a67c8d398f349f873fddadb72c02f)

  • The maintainers of didkit have released an update that fixes an issue where it would not build on arm64-based systems.
    This resolves an issue with Devstack and developers running devstack on Apple silicon-based machines.
  • this fix is merged in master, backporting this fix to work with redwood.
    openedx#2546

@cmltaWt0 thanks for the timely feedback. I agree with you. We shouldn't merge the master features into redwood. So, closing this PR.

@Faraz32123 Faraz32123 closed this Oct 21, 2024
@Faraz32123 Faraz32123 deleted the feat/backport_arm64_docker_image_support_to_redwood branch October 21, 2024 07:22
@cmltaWt0
Copy link

@Faraz32123 just want to understand - will we merge just a didkit update to unblock the arm64 builds? (at least for local development).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants