-
Notifications
You must be signed in to change notification settings - Fork 24
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
Japi 524 ws fileio ws fs regressions #632
Japi 524 ws fileio ws fs regressions #632
Conversation
7788cb0
to
89b0721
Compare
HPCC30117 status: 'open' |
9be82bc
to
a17b1f6
Compare
@jpmcmu please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rpastrana Only one comment, but otherwise looks good.
.github/workflows/httpsUnitTests.yml
Outdated
run: mvn -B --activate-profiles jenkins-on-demand -Dmaven.gpg.skip=true -Dmaven.javadoc.skip=true -Dmaven.test.failure.ignore=false -Dhpccconn=https://eclwatch.default:8010 -Dwssqlconn=https://sql2ecl.default:8510 -DHPCC30117=open install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are your thoughts on creating a github variable to inject maven options here? I don't think it is critical to take that approach, but it seems like it could be useful.
- Ignores invalid fileio tests until HPCC_30117 is addressed - WsFS conditionally provides DZ path and name if available - Adds CUSTOM_MVN_OPTIONS env var as mvn command arg Signed-off-by: Rodrigo Pastrana <[email protected]>
9a7f0d7
to
b8d2e16
Compare
Signed-off-by: Rodrigo Pastrana <[email protected]>
b8d2e16
to
ee846bd
Compare
Type of change:
Checklist:
Testing: