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

Disable testingConventions in build tools in fips #57357

Merged
merged 4 commits into from
Jun 2, 2020

Conversation

jkakavas
Copy link
Member

Necessary since we also disable test and testingConventions would
fail in FIPS 140 mode.

Necessary since we also disable test and testingConventions would
fail in FIPS 140 mode.
@jkakavas jkakavas added :Delivery/Build Build or test infrastructure >test-failure Triaged test failures from CI labels May 29, 2020
@jkakavas jkakavas requested a review from breskeby May 29, 2020 13:18
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Build)

@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label May 29, 2020
@jkakavas
Copy link
Member Author

@elasticmachine update branch

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM, although it would be nice if there were a comment explaining why tests were disabled here in the first place

@jkakavas
Copy link
Member Author

Fair ! I should have done this when originally muting these, but will add now

Copy link
Contributor

@mark-vieira mark-vieira left a comment

Choose a reason for hiding this comment

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

👍

@jkakavas jkakavas merged commit ad374ed into elastic:master Jun 2, 2020
jkakavas added a commit to jkakavas/elasticsearch that referenced this pull request Jun 2, 2020
* Disable testingConventions in build tools in fips

Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
jkakavas added a commit to jkakavas/elasticsearch that referenced this pull request Jun 2, 2020
* Disable testingConventions in build tools in fips

Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
jkakavas added a commit to jkakavas/elasticsearch that referenced this pull request Jun 2, 2020
* Disable testingConventions in build tools in fips

Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
jkakavas added a commit that referenced this pull request Jun 2, 2020
Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
jkakavas added a commit that referenced this pull request Jun 2, 2020
Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
jkakavas added a commit that referenced this pull request Jun 2, 2020
Necessary since we also disable unit tests and 
testingConventions would fail in FIPS 140 mode.
@mark-vieira mark-vieira added Team:Delivery Meta label for Delivery team and removed Team:Core/Infra Meta label for core/infra team labels Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team >test-failure Triaged test failures from CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants