Missed in #898 review. runConfigureCommand() is only invoked when library is not found in state.yaml
|
configured, err := r.detectIfLibraryConfigured(ctx) |
|
if err != nil { |
|
return err |
|
} |
|
|
|
if !configured { |
|
if err := r.runConfigureCommand(ctx); err != nil { |
|
return err |
|
} |
|
} |
Then this logic leads into error
|
libraryID := findLibraryIDByAPIPath(r.state, r.cfg.API) |
|
if libraryID == "" { |
|
return errors.New("bug in Librarian: Library not found during configuration, despite being found in earlier steps") |
|
} |
Also, configure should create a LibraryState skeleton to pass down to container via configure-request.json
cc. @cuiy0006
Missed in #898 review. runConfigureCommand() is only invoked when library is not found in state.yaml
librarian/internal/librarian/generate.go
Lines 166 to 175 in 10e3218
Then this logic leads into error
librarian/internal/librarian/generate.go
Lines 442 to 445 in 10e3218
Also, configure should create a LibraryState skeleton to pass down to container via configure-request.json
cc. @cuiy0006