Skip to content

Commit

Permalink
Removing suffix from simulated deploy model step
Browse files Browse the repository at this point in the history
Signed-off-by: Joshua Palis <[email protected]>
  • Loading branch information
joshpalis committed Jan 5, 2024
1 parent b970504 commit 8ef77ee
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ public CompletableFuture<WorkflowData> execute(
// Simulate Model deployment step and update resources created
flowFrameworkIndicesHandler.updateResourceInStateIndex(
currentNodeInputs.getWorkflowId(),

Check warning on line 186 in src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java

View check run for this annotation

Codecov / codecov/patch

src/main/java/org/opensearch/flowframework/workflow/RegisterLocalModelStep.java#L185-L186

Added lines #L185 - L186 were not covered by tests
currentNodeId + "-deploy",
currentNodeId,
DeployModelStep.NAME,
id,
ActionListener.wrap(deployUpdateResponse -> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ public void onResponse(MLRegisterModelResponse mlRegisterModelResponse) {
if (Boolean.TRUE.equals(deploy)) {
flowFrameworkIndicesHandler.updateResourceInStateIndex(
currentNodeInputs.getWorkflowId(),
currentNodeId + "-deploy",
currentNodeId,
DeployModelStep.NAME,
mlRegisterModelResponse.getModelId(),
ActionListener.wrap(deployUpdateResponse -> {
Expand Down

0 comments on commit 8ef77ee

Please sign in to comment.