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

Add s390x support for pump #701

Closed
wants to merge 7 commits into from
Closed

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Aug 7, 2023

Description

This pull request adds support for building the s390x architecture when building pump assets for the Tyk Pump project. It applies similar logic to the Tyk Gateway project, which already has support for the s390x architecture.

Summary of Changes

  • Modified ci/Dockerfile.std to add a new build stage for the s390x architecture.
  • Updated ci/goreleaser/goreleaser.yml to include the s390x architecture in the list of supported architectures for the build.
  • Updated .github/workflows/release.yml to add a new job for building the s390x architecture.

Please review the changes and let me know if any further modifications are required.

Fixes #699.


To checkout this PR branch, run the following command in your terminal:

git checkout sweep/add-s390x-support

To get Sweep to edit this pull request, leave a comment below or in the code. Leaving a comment in the code will only modify the file but commenting below can change the entire PR.

@sweep-ai sweep-ai bot requested a review from a team as a code owner August 7, 2023 15:40
@sweep-ai sweep-ai bot added the sweep Assigns Sweep to an issue or pull request. label Aug 7, 2023
@buger
Copy link
Member

buger commented Aug 7, 2023

That's not the way how I wanted it. Rollback the changes you did, and one more time look at this patch file, change which I did for another project, and apply similar changes here https://patch-diff.githubusercontent.com/raw/TykTechnologies/tyk/pull/4777.patch

@buger buger closed this Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep: s390x support for pump
1 participant