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

Issue #27 Completing activation of the security TCK. #46

Merged
merged 3 commits into from Aug 24, 2022

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Aug 24, 2022

No description provided.

bstansberry and others added 3 commits August 24, 2022 18:33
…not authentication

Disable running the new security tck to allow test runs to focus on getting the old security TCK to run properly
@darranl
Copy link
Contributor Author

darranl commented Aug 24, 2022

FYI the failures are expected until we have a new release of WildFly.

One failure still needs challenging to see if we can agree to ignore but we have discussed that the reason for the failure is not actually the purpose of that test.

@@ -211,7 +212,7 @@ then
echo "Configuring WildFly for the Old TCK"
pushd $TS_HOME/bin
$ANT_HOME/bin/ant config.vi
Copy link
Contributor

Choose a reason for hiding this comment

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

@darranl It seems the various uses of $ANT_HOME/bin is not necessary. If so we should clean that out having the mix of usage is confusing. That shouldn't block this though.

@bstansberry bstansberry merged commit 75e2344 into wildfly:main Aug 24, 2022
@jamezp
Copy link
Member

jamezp commented Aug 24, 2022

@darranl Just an FYI. This currently runs tests against the latest successful build of the Nightly WildFly CI job. Should it be passing on last nights main branch?

@bstansberry
Copy link
Contributor

@jamezp The nightly job last night had 1 failure and thus didn't publish the artifact. Last successful run was against wildfly/wildfly@62f0e67 which doesn't include wildfly/wildfly#15968

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.

None yet

3 participants