Merge pull request #1732 from milas/builder-init-panic

builder: extra init error handling
pull/1755/head
CrazyMax 2 years ago committed by GitHub
commit e14e0521cf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -108,7 +108,7 @@ func New(dockerCli command.Cli, opts ...Option) (_ *Builder, err error) {
// Validate validates builder context // Validate validates builder context
func (b *Builder) Validate() error { func (b *Builder) Validate() error {
if b.NodeGroup.DockerContext { if b.NodeGroup != nil && b.NodeGroup.DockerContext {
list, err := b.opts.dockerCli.ContextStore().List() list, err := b.opts.dockerCli.ContextStore().List()
if err != nil { if err != nil {
return err return err

@ -61,7 +61,10 @@ func GetCurrentInstance(txn *store.Txn, dockerCli command.Cli) (*store.NodeGroup
return nil, err return nil, err
} }
if ng == nil { if ng == nil {
ng, _ = GetNodeGroup(txn, dockerCli, dockerCli.CurrentContext()) ng, err = GetNodeGroup(txn, dockerCli, dockerCli.CurrentContext())
if err != nil {
return nil, err
}
} }
return ng, nil return ng, nil

Loading…
Cancel
Save