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

s390x CI support #85

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from
Open

s390x CI support #85

wants to merge 3 commits into from

Conversation

Nayana-ibm
Copy link

This pull request adds CI support for s390x. For now we are adding build and unit tests targets for s390x.

@@ -0,0 +1,110 @@
#!/bin/bash -xe
Copy link
Member

@michaelsembwever michaelsembwever Nov 24, 2022

Choose a reason for hiding this comment

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

it's unclear to me how this file is different to the existing build-scripts/cassandra-artifacts.sh ?

it is just the deb and rpm packaging that has been removed?

Copy link
Author

Choose a reason for hiding this comment

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

Yes..deb and rpm packaging is removed for s390x for now as apache cassandra needs few code changes to fix test failures on s390x and we have already provided code changes through https://issues.apache.org/jira/browse/CASSANDRA-17723 . Also, chronicle-bytes code used in Apache Cassandra needs s390x support hence we are going ahead with this workaround for now.

Copy link
Member

Choose a reason for hiding this comment

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

Ok. Is the plan to fix those failures as part of 17723 or 18072 ?

If that is the case then this PR should be marked as a draft, as the intention is never to merge it.

If you want to delay fixing the deb and rpm packing, then I suggest adding a conditional around that in the original script. Something like DEP_PACKAGING_SKIP and RPM_PACKAGING_SKIP.

Copy link
Author

@Nayana-ibm Nayana-ibm Nov 25, 2022

Choose a reason for hiding this comment

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

Ok. Is the plan to fix those failures as part of 17723 or 18072 ?

Issue-17723 will fix 6 tests out of 13 tests which we are skipping for s390x.
Issue-18072 is a part of CI for s390x. I raised different issue as repository was different.

If that is the case then this PR should be marked as a draft, as the intention is never to merge it.

If you want to delay fixing the deb and rpm packing, then I suggest adding a conditional around that in the original script. Something like DEP_PACKAGING_SKIP and RPM_PACKAGING_SKIP.

I think we can go ahead with .deb and .rpm packaging for s390x also. Lets not delay.

@Nayana-ibm
Copy link
Author

Created https://issues.apache.org/jira/browse/CASSANDRA-18072 and provided code patch for Apache Cassandra to add Jenkins CI support for s390x.

@Nayana-ibm Nayana-ibm changed the title S390x CI support s390x CI support Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants