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

[WFCORE-4214] VaultPasswordsInCLITestCase fails with Elytron profile #4532

Merged
merged 1 commit into from Mar 19, 2021

Conversation

petrberan
Copy link
Contributor

@wildfly-ci
Copy link

Hello, petrberan. I'm waiting for one of the admins to verify this patch with /ok-to-test in a comment.

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Mar 17, 2021
@petrberan
Copy link
Contributor Author

As discussed with Darran Lofthouse, this test case should not run with -Delytron profile as it is vault related. However, running this test with -Delytron profile causes failures in tests following this one in manualmode module, so a check of whether Elytron is used.

In case of Elytron use, this test case is skipped, allowing following tests to be run properly and not to fail.

@wildfly-ci
Copy link

Core - Full Integration Build 10438 outcome was UNKNOWN using a merge of fc2be03
Summary: Canceled (Tests passed: 6223, ignored: 105; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 02:00:18

@luck3y
Copy link
Contributor

luck3y commented Mar 18, 2021

/ok-to-test

@jmesnil jmesnil merged commit 34dbf24 into wildfly:master Mar 19, 2021
@jmesnil
Copy link
Member

jmesnil commented Mar 19, 2021

@petrberan thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
5 participants