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

enabling disabled modes for XML tests #500

Closed
wants to merge 17 commits into from

Conversation

AshithaVijay
Copy link

Enabled skipped modes for SVT XML tests on AIX, Linux and ZOS Platform according to the platform requirements:

Test passing links in various platforms:
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12686/ - ppc_aix
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12687/ - ppc64_aix
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12688/ - ppc_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12689/ - ppc64_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12659/ - ppc64le_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12844/ - s390_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12683/ - s390x_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12691/ - s390x_zos
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12692/ - s390_zos
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12693/ - x86-64_linux
hyc-runtimes-jenkins.swg-devops.com/Grinder_CR/12841/ - x86-32_linux

@karianna
Copy link
Contributor

@AshithaVijay Can you sign the Eclipse CLA please

@AshithaVijay
Copy link
Author

@karianna I have signed the Eclipse CLA.

@llxia
Copy link
Contributor

llxia commented Apr 2, 2024

Recently, we merged a similar PR #499 and had to revert because it caused issues in the regular build. I would prefer to be cautious when merging changes in ottawa.csv, as it's difficult to discern what has been altered in the file. Please list which mode on which platform has changed, and why.

@karianna
Copy link
Contributor

@AshithaVijay can you respond to @llxia please and we can try to close this off

@AshithaVijay
Copy link
Author

Yes, will close this as this has unwanted commits. I will raise a new PR with required changes only.

@AshithaVijay
Copy link
Author

Duplicate of #582

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.

3 participants