Skip to content

feat: qute condition #184

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
package io.javaoperatorsdk.operator.glue.conditions;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import io.fabric8.kubernetes.api.model.GenericKubernetesResource;
import io.javaoperatorsdk.operator.api.reconciler.Context;
import io.javaoperatorsdk.operator.api.reconciler.dependent.DependentResource;
import io.javaoperatorsdk.operator.glue.customresource.glue.Glue;
import io.javaoperatorsdk.operator.glue.templating.GenericTemplateHandler;
import io.javaoperatorsdk.operator.processing.dependent.workflow.Condition;

public class QuteCondition implements Condition<GenericKubernetesResource, Glue> {

private static final Logger LOG = LoggerFactory.getLogger(QuteCondition.class);

private final GenericTemplateHandler genericTemplateHandler;
private final String template;

public QuteCondition(GenericTemplateHandler genericTemplateHandler, String template) {
this.genericTemplateHandler = genericTemplateHandler;
this.template = template;
}

@Override
public boolean isMet(DependentResource<GenericKubernetesResource, Glue> dependentResource,
Glue primary, Context<Glue> context) {

LOG.debug("Evaluating condition with template: {}", template);

var data = GenericTemplateHandler.createDataWithResources(primary, context);
data.put("target", GenericTemplateHandler
.convertToValue(dependentResource.getSecondaryResource(primary, context)));

var res = genericTemplateHandler.processTemplate(data, template, false);

LOG.debug("Qute condition result: {}", res);

return "true".equalsIgnoreCase(res.trim());
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
property = "type")
@JsonSubTypes({
@JsonSubTypes.Type(value = ReadyConditionSpec.class, name = "ReadyCondition"),
@JsonSubTypes.Type(value = JavaScriptConditionSpec.class, name = "JSCondition")
@JsonSubTypes.Type(value = JavaScriptConditionSpec.class, name = "JSCondition"),
@JsonSubTypes.Type(value = QuteConditionSpec.class, name = "QuteCondition")
})
public class ConditionSpec {

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
package io.javaoperatorsdk.operator.glue.customresource.glue.condition;

public class QuteConditionSpec extends ConditionSpec {

private String template;

public String getTemplate() {
return template;
}

public void setTemplate(String template) {
this.template = template;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,14 @@
import io.javaoperatorsdk.operator.api.reconciler.*;
import io.javaoperatorsdk.operator.glue.Utils;
import io.javaoperatorsdk.operator.glue.conditions.JavaScripCondition;
import io.javaoperatorsdk.operator.glue.conditions.QuteCondition;
import io.javaoperatorsdk.operator.glue.conditions.ReadyCondition;
import io.javaoperatorsdk.operator.glue.customresource.glue.DependentResourceSpec;
import io.javaoperatorsdk.operator.glue.customresource.glue.Glue;
import io.javaoperatorsdk.operator.glue.customresource.glue.GlueStatus;
import io.javaoperatorsdk.operator.glue.customresource.glue.condition.ConditionSpec;
import io.javaoperatorsdk.operator.glue.customresource.glue.condition.JavaScriptConditionSpec;
import io.javaoperatorsdk.operator.glue.customresource.glue.condition.QuteConditionSpec;
import io.javaoperatorsdk.operator.glue.customresource.glue.condition.ReadyConditionSpec;
import io.javaoperatorsdk.operator.glue.dependent.GCGenericBulkDependentResource;
import io.javaoperatorsdk.operator.glue.dependent.GCGenericDependentResource;
Expand Down Expand Up @@ -287,6 +289,8 @@ private Condition toCondition(ConditionSpec condition) {
return new ReadyCondition(readyConditionSpec.isNegated());
} else if (condition instanceof JavaScriptConditionSpec jsCondition) {
return new JavaScripCondition(jsCondition.getScript());
} else if (condition instanceof QuteConditionSpec quteConditionSpec) {
return new QuteCondition(genericTemplateHandler, quteConditionSpec.getTemplate());
}
throw new IllegalStateException("Unknown condition: " + condition);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,10 @@ public String processTemplate(String template, Glue primary, boolean objectTempl
return res;
}

public static Map<?, ?> convertToValue(Object value) {
return objectMapper.convertValue(value, Map.class);
}

@SuppressWarnings("unchecked")
public static Map<String, ?> parseTemplateToMapObject(String template) {
return Serialization.unmarshal(template, Map.class);
Expand Down