-
Notifications
You must be signed in to change notification settings - Fork 175
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
kie-tools#2497: Set local cluster as default in the sonataflow-dev-ui #2498
base: main
Are you sure you want to change the base?
Conversation
@@ -108,14 +108,15 @@ public CardPageBuildItem pages( | |||
|
|||
String openapiPath = getProperty(configurationBuildItem, systemPropertyBuildItems, "quarkus.smallrye-openapi.path"); | |||
String devUIUrl = getProperty(configurationBuildItem, systemPropertyBuildItems, "kogito.dev-ui.url"); | |||
String dataIndexUrl = getProperty(configurationBuildItem, systemPropertyBuildItems, "kogito.data-index.url"); | |||
//String dataIndexUrl = getProperty(configurationBuildItem, systemPropertyBuildItems, "kogito.data-index.url"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should remove rather than commenting
@@ -58,7 +58,8 @@ public SonataFlowQuarkusExtensionJsonRPCService(Vertx vertx) { | |||
|
|||
@PostConstruct | |||
public void init() { | |||
isLocalCluster = ConfigProvider.getConfig().getOptionalValue(IS_LOCAL_CLUSTER, Boolean.class).orElse(false); | |||
//temporary, set isLocalCluster always true since sonataflow-dev-ui works in k8s cluster. | |||
isLocalCluster = ConfigProvider.getConfig().getOptionalValue(IS_LOCAL_CLUSTER, Boolean.class).orElse(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As commented in a previous PR, I think this should have been done through injection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wmedvede Please include the issue link on the description of the PR, or provide a good description at least...
Hi! Any updates here? I see some conflicts have built up here too... Thanks! |
No description provided.