Skip to content

Commit 73cd449

Browse files
committed
no dependency on branches
1 parent e5ba174 commit 73cd449

File tree

3 files changed

+39
-35
lines changed

3 files changed

+39
-35
lines changed

dgctl/cmd/exec.go

Lines changed: 24 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -94,15 +94,13 @@ func GetUrlContents(url string) (string, error) {
9494
return content, nil
9595
}
9696

97-
func GetSpec(diggerUrl string, authToken string, command string, actor string, projectMarshalled string, diggerConfigMarshalled string, repoFullName string, defaultBanch string, prBranch string) ([]byte, error) {
97+
func GetSpec(diggerUrl string, authToken string, command string, actor string, projectMarshalled string, diggerConfigMarshalled string, repoFullName string) ([]byte, error) {
9898
payload := spec.GetSpecPayload{
99-
Command: command,
100-
RepoFullName: repoFullName,
101-
Actor: actor,
102-
DefaultBranch: defaultBanch,
103-
PrBranch: prBranch,
104-
DiggerConfig: diggerConfigMarshalled,
105-
Project: projectMarshalled,
99+
Command: command,
100+
RepoFullName: repoFullName,
101+
Actor: actor,
102+
DiggerConfig: diggerConfigMarshalled,
103+
Project: projectMarshalled,
106104
}
107105
u, err := url.Parse(diggerUrl)
108106
if err != nil {
@@ -236,17 +234,17 @@ var execCmd = &cobra.Command{
236234
os.Exit(1)
237235
}
238236

239-
defaultBanch, err := getDefaultBranch()
240-
if err != nil {
241-
log.Printf("could not get default branch, please enter manually:")
242-
fmt.Scanln(&defaultBanch)
243-
}
237+
//defaultBanch, err := getDefaultBranch()
238+
//if err != nil {
239+
// log.Printf("could not get default branch, please enter manually:")
240+
// fmt.Scanln(&defaultBanch)
241+
//}
244242

245-
prBranch, err := getPrBranch()
246-
if err != nil {
247-
log.Printf("could not get current branch, please enter manually:")
248-
fmt.Scanln(&prBranch)
249-
}
243+
//prBranch, err := getPrBranch()
244+
//if err != nil {
245+
// log.Printf("could not get current branch, please enter manually:")
246+
// fmt.Scanln(&prBranch)
247+
//}
250248

251249
projectName := execConfig.Project
252250
command := execConfig.Command
@@ -268,7 +266,7 @@ var execCmd = &cobra.Command{
268266
os.Exit(1)
269267
}
270268

271-
specBytes, err := GetSpec(diggerHostname, "abc123", command, actor, string(projectMarshalled), string(configMarshalled), repoFullname, defaultBanch, prBranch)
269+
specBytes, err := GetSpec(diggerHostname, "abc123", command, actor, string(projectMarshalled), string(configMarshalled), repoFullname)
272270
if err != nil {
273271
log.Printf("failed to get spec from backend: %v", err)
274272
}
@@ -307,11 +305,18 @@ var execCmd = &cobra.Command{
307305
}
308306
}
309307

308+
repoOwner, repoName, _ := strings.Cut(repoFullname, "/")
309+
repository, _, err := client.Repositories.Get(context.Background(), repoOwner, repoName)
310+
if err != nil {
311+
log.Fatalf("Failed to get repository: %v", err)
312+
}
313+
310314
inputs := orchestrator_scheduler.WorkflowInput{
311315
Spec: string(specBytes),
312316
RunName: fmt.Sprintf("digger %v manual run by %v", command, spec.VCS.Actor),
313317
}
314318
_, err = client.Actions.CreateWorkflowDispatchEventByFileName(context.Background(), spec.VCS.RepoOwner, spec.VCS.RepoName, spec.VCS.WorkflowFile, github.CreateWorkflowDispatchEventRequest{
319+
Ref: *repository.DefaultBranch,
315320
Inputs: inputs.ToMap(),
316321
})
317322

@@ -321,7 +326,6 @@ var execCmd = &cobra.Command{
321326
log.Printf("workflow has triggered successfully! waiting for results ...")
322327
}
323328

324-
repoOwner, repoName, _ := strings.Cut(repoFullname, "/")
325329
var logsUrl *string
326330
var runId *int64
327331
var jobId *int64

ee/backend/controllers/spec.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,8 @@ func (d DiggerEEController) GetSpec(c *gin.Context) {
3232

3333
actor := payload.Actor
3434
commitSha := ""
35-
defaultBranch := payload.DefaultBranch
36-
prBranch := payload.PrBranch
35+
//defaultBranch := payload.DefaultBranch
36+
//prBranch := payload.PrBranch
3737
issueNumber := 000
3838

3939
config := digger_config.DiggerConfig{}
@@ -54,7 +54,7 @@ func (d DiggerEEController) GetSpec(c *gin.Context) {
5454

5555
workflowFile := project.WorkflowFile
5656

57-
jobs, err := generic.CreateJobsForProjects([]digger_config.Project{project}, command, "manual", repoFullName, actor, config.Workflows, &issueNumber, &commitSha, defaultBranch, prBranch)
57+
jobs, err := generic.CreateJobsForProjects([]digger_config.Project{project}, command, "manual", repoFullName, actor, config.Workflows, &issueNumber, &commitSha, "", "")
5858
if err != nil {
5959
log.Printf("could not create jobs based on project: %v", err)
6060
c.String(500, fmt.Sprintf("could not create jobs based on project: %v", err))
@@ -77,7 +77,7 @@ func (d DiggerEEController) GetSpec(c *gin.Context) {
7777
}
7878
backendHostName := os.Getenv("HOSTNAME")
7979

80-
jobSpec := scheduler.JobToJson(job, scheduler.DiggerCommandPlan, org.Name, prBranch, commitSha, jobToken.Value, backendHostName, project)
80+
jobSpec := scheduler.JobToJson(job, scheduler.DiggerCommandPlan, org.Name, "", commitSha, jobToken.Value, backendHostName, project)
8181

8282
spec := spec.Spec{
8383
SpecType: spec.SpecTypeManualJob,

libs/spec/payloads.go

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,23 @@
11
package spec
22

33
type GetSpecPayload struct {
4-
Command string `json:"command"`
5-
RepoFullName string `json:"repo_full_name"`
6-
Actor string `json:"actor"`
7-
DefaultBranch string `json:"default_branch"`
8-
PrBranch string `json:"pr_branch"`
9-
DiggerConfig string `json:"digger_config"`
10-
Project string `json:"project"`
4+
Command string `json:"command"`
5+
RepoFullName string `json:"repo_full_name"`
6+
Actor string `json:"actor"`
7+
//DefaultBranch string `json:"default_branch"`
8+
//PrBranch string `json:"pr_branch"`
9+
DiggerConfig string `json:"digger_config"`
10+
Project string `json:"project"`
1111
}
1212

1313
func (p GetSpecPayload) ToMapStruct() map[string]interface{} {
1414
return map[string]interface{}{
1515
"command": p.Command,
1616
"repo_full_name": p.RepoFullName,
1717
"actor": p.Actor,
18-
"default_branch": p.DefaultBranch,
19-
"pr_branch": p.PrBranch,
20-
"digger_config": p.DiggerConfig,
21-
"project": p.Project,
18+
//"default_branch": p.DefaultBranch,
19+
//"pr_branch": p.PrBranch,
20+
"digger_config": p.DiggerConfig,
21+
"project": p.Project,
2222
}
2323
}

0 commit comments

Comments
 (0)