Skip to content

Commit

Permalink
Merge pull request #34 from openinfradev/main
Browse files Browse the repository at this point in the history
merge from main
  • Loading branch information
ktkfree authored Nov 29, 2022
2 parents 0e62386 + 23667c4 commit 97ebcba
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 1,240 deletions.
6 changes: 4 additions & 2 deletions cmd/server/handlers.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import (

"github.com/golang/protobuf/ptypes/empty"
"github.com/google/uuid"
"github.com/openinfradev/tks-common/pkg/argowf"
"github.com/openinfradev/tks-common/pkg/helper"
"github.com/openinfradev/tks-common/pkg/log"
pb "github.com/openinfradev/tks-proto/tks_pb"
Expand Down Expand Up @@ -74,12 +75,13 @@ func (s *server) CreateContract(ctx context.Context, in *pb.CreateContractReques

workflowTemplate := "tks-create-contract-repo"
nameSpace := "argo"
parameters := []string{
opts := argowf.SubmitOptions{}
opts.Parameters = []string{
"contract_id=" + contractId,
"revision=" + revision,
}

workflowName, err := argowfClient.SumbitWorkflowFromWftpl(ctx, workflowTemplate, nameSpace, parameters)
workflowName, err := argowfClient.SumbitWorkflowFromWftpl(workflowTemplate, nameSpace, opts)
if err != nil {
log.Error("failed to submit argo workflow template. err : ", err)

Expand Down
4 changes: 2 additions & 2 deletions cmd/server/handlers_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ func TestCreateContract(t *testing.T) {
}, nil)

mockArgoClient.EXPECT().
SumbitWorkflowFromWftpl(gomock.Any(), gomock.Any(), gomock.Any(), gomock.Any()).
SumbitWorkflowFromWftpl(gomock.Any(), gomock.Any(), gomock.Any()).
Times(1).
Return(randomString("workflowName"), nil)
},
Expand Down Expand Up @@ -158,7 +158,7 @@ func TestCreateContract(t *testing.T) {
}, nil)

mockArgoClient.EXPECT().
SumbitWorkflowFromWftpl(gomock.Any(), gomock.Any(), gomock.Any(), gomock.Any()).
SumbitWorkflowFromWftpl(gomock.Any(), gomock.Any(), gomock.Any()).
Times(1).
Return(randomString("workflowName"), errors.New("argo error"))
},
Expand Down
2 changes: 1 addition & 1 deletion cmd/server/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ func main() {
contractAccessor = contract.New(db)

// initialize argo client
_argowfClient, err := argowf.New(argoAddress, argoPort)
_argowfClient, err := argowf.New(argoAddress, argoPort, false, "")
if err != nil {
log.Fatal("failed to create argowf client : ", err)
}
Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ require (
github.com/google/uuid v1.3.0
github.com/grpc-ecosystem/go-grpc-middleware v1.3.0 // indirect
github.com/lib/pq v1.10.4
github.com/openinfradev/tks-common v0.0.0-20221121042940-4356d48a950b
github.com/openinfradev/tks-common v0.0.0-20221124045547-fbf60e9529da
github.com/openinfradev/tks-proto v0.0.6-0.20221018052004-85d1b297f865
github.com/stretchr/testify v1.7.0
google.golang.org/genproto v0.0.0-20211013025323-ce878158c4d4 // indirect
Expand Down
Loading

0 comments on commit 97ebcba

Please sign in to comment.