[
https://issues.apache.org/jira/browse/GEODE-6523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16795514#comment-16795514
]
ASF subversion and git services commented on GEODE-6523:
--------------------------------------------------------
Commit 59b9b022135d06ad8a7f8079aa18813b4ff5503e in geode's branch
refs/heads/develop from Peter Tran
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=59b9b02 ]
GEODE-6523 Disables Geode Management API with feature flag (#3306)
Co-authored-by: Jinmei Liao <jil...@pivotal.io>
Co-authored-by: Peter Tran <pt...@pivotal.io>
Co-authored-by: Owen Nichols <onich...@pivotal.io>
* The Geode Management API is disabled
* A feature flag has been added to allow people to use this experimental
feature using System Property `enable-experimental-cluster-management-service`
* turn on feature by default for acceptance tests
> Disable new REST and Java API behind feature flag
> -------------------------------------------------
>
> Key: GEODE-6523
> URL: https://issues.apache.org/jira/browse/GEODE-6523
> Project: Geode
> Issue Type: Improvement
> Components: management
> Reporter: Peter Tran
> Priority: Major
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Discuss: Do we want our new functionality to be disabled by default until
> such time as we are happy to make it publicly available? Other features -
> e.g. protobuf currently do this.
> If we do this, please use a feature-flag system property of the form:
> `geode.feature-cluster-management-service` for consistency with other feature
> flags.
> * consider tests
> * GFSH start option to switch it on
> * Augment Experimental (don't turn off experimental with addition)
> * What happens with Experimental later, leave it on until we are solid on the
> External/public API.
> * Feature flag = MVP adding value, Experimental = public API is solid
> there is one flag to control this feature as Java Runtime properties "--J-D":
> - enable-experimental-cluster-management-service: for features, which is
> just an experimental, just want to get some feedback from early birds.
> ### Why
> We want users to not have access to Experimental API by default because it
> could be in a state that is insecure/incomplete/could cause corruption of a
> system. We need to turn this off by default so that users can opt into this
> functionality.
> ### Acceptance Criteria
> ```gherkin
> Scenario: System doesn't see this flag
> Given geode is unpacked in a directory
> When the system cannot find a flag of
> 'enable-experimental-cluster-management-service' in Java Runtime properties.
> Then the REST API v2 will not be available to the user on any port of the
> locator
> AND a info level log would be added to locator log: "CMS is turned off ,
> because value the experimental flag:
> enable-experimental-cluster-management-service is false."
> ```
> ```gherkin
> Scenario: User default false for opt-in on experimental
> Given the user does not change this flag
> 'enable-experimental-cluster-management-service' and its default value.
> When the user starts a locator
> Then the REST API v2 will not be available to the user on any port of the
> locator
> AND a info level log would be added to locator log: "CMS is turned off ,
> because value the experimental flag:
> enable-experimental-cluster-management-service is false."
> ```
> ```gherkin
> Scenario: User default incorrectly specified flag value
> Given geode is unpacked in a directory
> When the user specifies an incorrect value for this flag:
> 'enable-experimental-cluster-management-service' by Java Runtime properties
> "--J-D"
> Then the REST API v2 will not be available to the user on any port of the
> locator
> AND a info level log would be added to locator log: "CMS is turned off ,
> because value the experimental flag:
> enable-experimental-cluster-management-service is false."
> ```
> ```gherkin
> Scenario: User opts into the experimental flag
> Given geode is unpacked in a directory
> When the user enables this flag:
> 'enable-experimental-cluster-management-service' by Java Runtime properties
> "--J-D"
> Then the REST API v2 will be available to the user at the standard port of
> 7070 or whatever port the user has specified.
> AND a info level log would be added to locator log: "CMS is turned on ,
> because value the experimental flag:
> enable-experimental-cluster-management-service is true."
> ```
> **Notes:**
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
