Update garm-provider-common and go-github
* Updates the garm-provider-common and go-github packages. * Update sqlToParamsInstance to return an error when unmarshaling This change is needed to pull in the new Seal/Unseal functions in common. Signed-off-by: Gabriel Adrian Samfira <gsamfira@cloudbasesolutions.com>
This commit is contained in:
parent
4348999cb1
commit
d700b790ac
310 changed files with 10994 additions and 1955 deletions
|
|
@ -28,14 +28,19 @@ import (
|
|||
commonParams "github.com/cloudbase/garm-provider-common/params"
|
||||
)
|
||||
|
||||
func (s *sqlDatabase) sqlToParamsInstance(instance Instance) params.Instance {
|
||||
func (s *sqlDatabase) sqlToParamsInstance(instance Instance) (params.Instance, error) {
|
||||
var id string
|
||||
if instance.ProviderID != nil {
|
||||
id = *instance.ProviderID
|
||||
}
|
||||
|
||||
var labels []string
|
||||
_ = json.Unmarshal(instance.AditionalLabels, &labels)
|
||||
if len(instance.AditionalLabels) > 0 {
|
||||
if err := json.Unmarshal(instance.AditionalLabels, &labels); err != nil {
|
||||
return params.Instance{}, errors.Wrap(err, "unmarshalling labels")
|
||||
}
|
||||
}
|
||||
|
||||
ret := params.Instance{
|
||||
ID: instance.ID.String(),
|
||||
ProviderID: id,
|
||||
|
|
@ -74,7 +79,7 @@ func (s *sqlDatabase) sqlToParamsInstance(instance Instance) params.Instance {
|
|||
EventLevel: msg.EventLevel,
|
||||
})
|
||||
}
|
||||
return ret
|
||||
return ret, nil
|
||||
}
|
||||
|
||||
func (s *sqlDatabase) sqlAddressToParamsAddress(addr Address) commonParams.Address {
|
||||
|
|
@ -102,7 +107,10 @@ func (s *sqlDatabase) sqlToCommonOrganization(org Organization) (params.Organiza
|
|||
}
|
||||
|
||||
for idx, pool := range org.Pools {
|
||||
ret.Pools[idx] = s.sqlToCommonPool(pool)
|
||||
ret.Pools[idx], err = s.sqlToCommonPool(pool)
|
||||
if err != nil {
|
||||
return params.Organization{}, errors.Wrap(err, "converting pool")
|
||||
}
|
||||
}
|
||||
|
||||
return ret, nil
|
||||
|
|
@ -126,13 +134,16 @@ func (s *sqlDatabase) sqlToCommonEnterprise(enterprise Enterprise) (params.Enter
|
|||
}
|
||||
|
||||
for idx, pool := range enterprise.Pools {
|
||||
ret.Pools[idx] = s.sqlToCommonPool(pool)
|
||||
ret.Pools[idx], err = s.sqlToCommonPool(pool)
|
||||
if err != nil {
|
||||
return params.Enterprise{}, errors.Wrap(err, "converting pool")
|
||||
}
|
||||
}
|
||||
|
||||
return ret, nil
|
||||
}
|
||||
|
||||
func (s *sqlDatabase) sqlToCommonPool(pool Pool) params.Pool {
|
||||
func (s *sqlDatabase) sqlToCommonPool(pool Pool) (params.Pool, error) {
|
||||
ret := params.Pool{
|
||||
ID: pool.ID.String(),
|
||||
ProviderName: pool.ProviderName,
|
||||
|
|
@ -174,11 +185,15 @@ func (s *sqlDatabase) sqlToCommonPool(pool Pool) params.Pool {
|
|||
ret.Tags[idx] = s.sqlToCommonTags(*val)
|
||||
}
|
||||
|
||||
var err error
|
||||
for idx, inst := range pool.Instances {
|
||||
ret.Instances[idx] = s.sqlToParamsInstance(inst)
|
||||
ret.Instances[idx], err = s.sqlToParamsInstance(inst)
|
||||
if err != nil {
|
||||
return params.Pool{}, errors.Wrap(err, "converting instance")
|
||||
}
|
||||
}
|
||||
|
||||
return ret
|
||||
return ret, nil
|
||||
}
|
||||
|
||||
func (s *sqlDatabase) sqlToCommonTags(tag Tag) params.Tag {
|
||||
|
|
@ -207,7 +222,10 @@ func (s *sqlDatabase) sqlToCommonRepository(repo Repository) (params.Repository,
|
|||
}
|
||||
|
||||
for idx, pool := range repo.Pools {
|
||||
ret.Pools[idx] = s.sqlToCommonPool(pool)
|
||||
ret.Pools[idx], err = s.sqlToCommonPool(pool)
|
||||
if err != nil {
|
||||
return params.Repository{}, errors.Wrap(err, "converting pool")
|
||||
}
|
||||
}
|
||||
|
||||
return ret, nil
|
||||
|
|
@ -311,5 +329,5 @@ func (s *sqlDatabase) updatePool(pool Pool, param params.UpdatePoolParams) (para
|
|||
}
|
||||
}
|
||||
|
||||
return s.sqlToCommonPool(pool), nil
|
||||
return s.sqlToCommonPool(pool)
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue