-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 support for s390x #2622
Comments
@ncdc can you please assign this to me? I have a PR in progress. |
@e-desouza Any update on your PR? |
Replicating what the other platforms did, I'm looking for a place to upload the restic binary to. This part specifically needs it. |
It needs the restic supports s309x platform, ref to restic/restic#2780. |
Closing, but feel free to reopen if/when you make the PR. |
What is the status of s390x support? The is closed but I'm not finding any mention or packages for Z. |
The PR was never submitted. If you need this, please reopen the issue and vote for it. |
I don't have permission to re-open this issue. |
@hoytk Reopened for you! |
I mentioned in #4047 (comment) that Restic now has a release with s390x support. In order to support this architecture we would need to upgrade to this version. |
Thanks @zubron - for now it can be built on |
I'm also looking forward! What's the status of it right now? |
Hi @zubron - just wanted to follow-up on this issue. I also noticed that in order for testcases (
|
Hi @rposts - Sorry for the delay in getting back to you here! For non-amd64 platforms that we support, we don't maintain the build image for building/running tests, and instead rely on Go's cross-compiling and Docker buildx for releasing on those targets as I mentioned in #4047 (comment). I don't think it will be possible to actually build on s390x as not all of the tools listed above have s390x binaries available. Are you using s390x as your development platform? Another option for testing on this platform would be to use our E2E tests. These run the velero binary and deployment on a cluster so that would enable testing of s390x assets. We recently upgraded the version of restic we are using (see #4274) which has a s390x build so it should just be a matter of adding this platform to the targets that we build for. |
Thanks @zubron for your comments. Yes, my development platform is I will try to run E2E test if that offers sufficient coverage. Would you have any rough idea when we can expect this platform to be added to the cross-compile/release scripts? Thanks again for your continued help. |
Hi @zubron - just a quick follow up on adding this platform to the build targets. I think we have pretty much everything in place now. Thanks for your patience and help. |
@reasonerjt is this resolved now? |
@rposts I'm removing the |
@reasonerjt as far as I can see nothing has changed here so there is no reason to believe |
@reasonerjt - just wanted to check how I can further assist in making |
Hello, I am also looking for s390x support for velero, is there any timeline for adding this? I see this issue is over 3 years old now and that the version of restic is at least 0.15.0 since #5784 (above the necessary 0.12.1 according to #4047 (comment)). What needs to happen to support s390x? |
Hi Team, |
Hi Team, |
It should be possible to produce a binary using only GitHub actions. However, it will not be officially tested in any continuous manner, so its functionality cannot be guaranteed at all times. If I understand the sentiment of the maintainers correctly, it is that we should make it easy and perhaps even document how to create such a CLI for s390x, but we will not ship anything that we do not have CI/dev infrastructure to test and debug. Are you OK with documentation and any fixes required to unblock self builds? Or do you require maintainers to produce unvalidated cross compiled s390x velero binary? |
#7505 is going to address this |
Describe the problem/challenge you have
Take OpenShift backups on IBM Z/IBM LinuxONE systems
Describe the solution you'd like
s390x support similar to arm64, ppc64lee
Environment:
velero version
): v1.4.0 and abovekubectl version
): Kubernetes 1.16/etc/os-release
): linuxVote on this issue!
This is an invitation to the Velero community to vote on issues, you can see the project's top voted issues listed here.
Use the "reaction smiley face" up to the right of this comment to vote.
The text was updated successfully, but these errors were encountered: