Skip to content

CORE: Disable Setting Type Validation #33660

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

Conversation

original-brownbear
Copy link
Contributor

* Reverts setting type validation introduced in elastic#33503
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@original-brownbear
Copy link
Contributor Author

@simonw I couldn't cleanly revert #33503 because there's a few commits on top of it using the isListSetting functionality introduced => turned off the type checking when getting settings here only and suppressed the test. Hope that's ok?

Copy link
Contributor

@s1monw s1monw left a comment

Choose a reason for hiding this comment

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

This must go into master too. This validation is broken. We also need a test that shows it imo but merge it first to unblock others

@original-brownbear
Copy link
Contributor Author

@s1monw thanks, will merge once green and look into the validation next then :)

@original-brownbear original-brownbear merged commit 928fe3f into elastic:6.x Sep 13, 2018
@original-brownbear original-brownbear deleted the remove-setting-type-validation branch September 13, 2018 10:40
@jasontedor
Copy link
Member

Sorry @original-brownbear for the trouble with the backport.

original-brownbear added a commit to original-brownbear/elasticsearch that referenced this pull request Sep 13, 2018
* Reverts setting type validation introduced in elastic#33503
@original-brownbear
Copy link
Contributor Author

@jasontedor np :)

version for master incoming here #33669 <= put version labels on that PR now

original-brownbear added a commit that referenced this pull request Sep 13, 2018
* Reverts setting type validation introduced in #33503
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Sep 13, 2018
…polling

* elastic/master:
  SQL: Return correct catalog separator in JDBC (elastic#33670)
  [CCR] Add validation for max_retry_delay (elastic#33648)
  [CCR] Add monitoring mapping verification test (elastic#33662)
  CORE: Disable Setting Type Validation (elastic#33660) (elastic#33669)
  Revert "Use serializable exception in GCP listeners (elastic#33657)"
  Adding index refresh (elastic#33647)
  [DOCS] Moves securing-communications to docs (elastic#33640)
  [HLRC][ML] Add ML delete datafeed API to HLRC (elastic#33667)
  Mute testRecoveryWithConcurrentIndexing
  TEST: decrease logging level in the flush test
  DOC: Add SQL section on client applications
  Fix race in global checkpoint listeners test
  Use serializable exception in GCP listeners (elastic#33657)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Core/Infra/Settings Settings infrastructure and APIs >non-issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants